error[E0277]: the trait bound *const str: diesel::deserialize::FromSql<diesel::sql_types::Array<diesel::sql_types::Text>, diesel::pg::Pg> is not satisfied

Setup
Versions

Rust:
Running rustup show gives:

active toolchain
----------------

stable-x86_64-unknown-linux-gnu (default)
rustc 1.41.1 (f3e1a954d 2020-02-24)

Diesel: 1.4.3
Database: PostgreSQL 12.2
Operating System: Ubuntu 18.04 (Windows Subsystem for Linux)

Feature Flags

diesel: { version = "1.4.3", features = ["postgres"] }

Problem Description
What are you trying to accomplish?
I am trying to call the load() function to return a Vec of a struct that implements Queryable.
Instead of compiling, running cargo build gives me error[E0277]: the trait bound *const str: diesel::deserialize::FromSql<diesel::sql_types::Arraydiesel::sql_types::Text, diesel::pg::Pg> is not satisfied.
There are two issues that have described similar problems to mine, but neither are quite the same.
The first is https://github.com/diesel-rs/diesel/issues/330, and the issue was simply that the user was trying to use a Nullable<Integer> SQL value with a model that had an i32 type instead of an Option<i32>.
The second is https://github.com/diesel-rs/diesel/issues/2011, with the issue being that the struct in models.rs had a different order than the table! macro in schema.rs.
Neither of these solutions apply to my problem, which is why I suspect it might be a bug.
Steps to reproduce
The load() function is called with the following code:
use actix::{Message, Handler};
use std::io;
use crate::models;
use crate::db::DbExecutor;

use diesel::prelude::*;

pub struct Webpages;

impl Message for Webpages {
    type Result = io::Result<Vec<models::Webpage>>;
}

impl Handler<Webpages> for DbExecutor {
    type Result = io::Result<Vec<models::Webpage>>;

    fn handle(&mut self, _: Webpages, _: &mut Self::Context) -> Self::Result {
        use crate::schema::webpage::dsl::*;

        let w = match webpage.load::<models::Webpage>(&self.conn) {
            Ok(items) => items,
            Err(_) => {
                return Err(io::Error::new(io::ErrorKind::Other, "Database error"));
            }
        };

        Ok(w)
    }
}

The DbExecutor is just a connection to Postgres, but I am including it here:
/// Database [SyncArbiter](https://actix.rs/book/actix/sec-6-sync-arbiter.html)
/// with a connection field to Postgres
pub struct DbExecutor {
    conn: PgConnection
}

/// Create Sync Actor
impl Actor for DbExecutor {
    type Context = SyncContext<Self>;
}

/// Implement new `DbExecutor` with a live Postgres connection
impl DbExecutor {
    pub fn new(db_url: &str) -> DbExecutor {
        let conn = PgConnection::establish(db_url)
            .expect(&format!("Error connecting to {}", db_url));
        // Run database migrations
        diesel_migrations::run_pending_migrations(&conn).unwrap();
        DbExecutor {
            conn
        }
    }
}

My Webpage model in models.rs is as follows:
#[allow(non_snake_case)]
#[belongs_to(Website)]
#[derive(Serialize, Queryable, Debug, Associations, Identifiable)]
#[table_name="webpage"]
pub struct Webpage {
    pub id: i32,
    website_id: i32,
    scraped_data_ids: Vec<i32>,
    url: Option<String>,
    title: Option<String>,
    meta: Option<String>,
    canonical: Option<String>,
    h1s: Option<Vec<String>>,
    h2s: Option<Vec<String>>,
    h3s: Option<Vec<String>>,
    h4s: Option<Vec<String>>,
    h5s: Option<Vec<String>>,
    h6s: Option<Vec<String>>,
    ps: Option<Vec<String>>,
    images: Option<String>,
    is_secure: bool
}

My schema.rs webpage table is as follows:
table! {
    webpage (id) {
        id -> Int4,
        website_id -> Int4,
        scraped_data_ids -> Array<Int4>,
        url -> Nullable<Varchar>,
        title -> Nullable<Varchar>,
        meta -> Nullable<Varchar>,
        canonical -> Nullable<Varchar>,
        h1s -> Nullable<Array<Text>>,
        h2s -> Nullable<Array<Text>>,
        h3s -> Nullable<Array<Text>>,
        h4s -> Nullable<Array<Text>>,
        h5s -> Nullable<Array<Text>>,
        h6s -> Nullable<Array<Text>>,
        ps -> Nullable<Array<Text>>,
        images -> Nullable<Array<Text>>,
        is_secure -> Bool,
    }
}

My up.sql for the migration that generated the schema is:
CREATE TABLE webpage (
     id SERIAL PRIMARY KEY,
     website_id INTEGER NOT NULL,
     scraped_data_ids INTEGER[] NOT NULL,
     url VARCHAR,
     title VARCHAR,
     meta VARCHAR,
     canonical VARCHAR,
     h1s TEXT[],
     h2s TEXT[],
     h3s TEXT[],
     h4s TEXT[],
     h5s TEXT[],
     h6s TEXT[],
     ps TEXT[],
     images TEXT[],
     is_secure BOOLEAN NOT NULL DEFAULT 'f'
)

In the code above, the model, schema, and SQL all have the same order, and every value that is not specified as NOT NULL is wrapped in an Option<T>.

Checklist

[x] I have already looked over the issue tracker for similar issues.
[x] This issue can be reproduced on Rust's stable channel. (Your issue will be
closed if this is not the case)

Your struct implementing Queryable uses a Option<String> as 15. field (images). This type is not compatible with the Nullable<Array<Text>> returned by your query at this position.

@weiznich I literally just noticed that and was about to close this ticket. Sorry for adding noise to the issue triage here.


Add lambda

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 68.53%. Comparing base (cec06f1) to head (b496669).

:exclamation:  There is a different number of reports uploaded between BASE (cec06f1) and HEAD (b496669). Click for more details.
HEAD has 4 uploads less than BASE

Flag
BASE (cec06f1)
HEAD (b496669)

5
1

Additional details and impacted files
@@             Coverage Diff             @@
##           master     #112       +/-   ##
===========================================
- Coverage   84.87%   68.53%   -16.34%
===========================================
  Files          68       68
  Lines        6863     6865        +2
  Branches      726      726
===========================================
- Hits         5825     4705     -1120
- Misses        783     1942     +1159
+ Partials      255      218       -37

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


Accessibility/QoL: Read-only checkboxes in Dendron Preview

Please select if your request is either something new or an enhancement

[ ] Enhancement of an existing Feature.
[x] Request of a new feature.

Please select the area your request applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace - VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup - Dendron's Lookup Command
[x] Views - Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema - Dendron Schemas
[ ] Pod - Data import from / export to Dendron
[ ] Publish - External Site Publish
[ ] Markdown - Markdown features, such as syntax support and features

Is your feature request related to a problem? Please describe
Because checkboxes are greyed/read-only in the preview pane, you have to check off items using markdown which is inconvenient. Additionally, the greyed boxes make it difficult to distinguish checked from unchecked items.
Describe the solution you'd like
Editable checkboxes in the preview pane would be nice. Clicking a checkbox would put an [x] in the corresponding markdown.
Describe alternatives you've considered
For the visibility/accessibility issue: Some other way (with themes perhaps?) to make read-only checkboxes more visible and distinguishable from their unchecked counterparts
For the convenience/qol issue: An option in the editor, via the right-click menu or lightbulb tips, when focused on a checklist item, to toggle between check/uncheck
Additional context
Read-only checkboxes in the Dendron editor:

Related: https://github.com/dendronhq/dendron/issues/1571


Changed AFNetworking dependency version to latest one possible

Anyway, since NSURLConnection-based APIs are deprecated, seems like it would be good to move to AFNetoworking 3.X but this looks like needs rewriting of PDKClient-related stuff.

@Austinate If it would require a bunch of rewriting, I don't think we can merge this PR without that rewriting done.

This PR updates only to 2.6~ which actually has no functional changes that brake ios-pdk as i see

@Austinate we're not requiring users to use the old version, by raising the version, we're requiring them to use that. While it's obviously beneficial that they get that security fix, the issue wasn't in the version we require it to be above.
I don't think there's an advantage to requiring users to upgrade, they can do so on their own if they choose.

Sorry, maybe we're just misunderstood each other.
When i said that something will require rewriting — i said about usage of 3.x AFNetworking, which dropped NSURLConnection-based APIs support.
Using latest 2.x version is fine and doesn't break any functionality while gives us numerous bugfixes and improvements.
As for users ability to choose version — since AFNetworking is a dependency of ios-pdk users can't use another version of AFNetworking in same app, which as for me brings a bit more responsibility of which version ios-pdk will require anyone to use.

@garrettmoon the legacy version of AFNetworking dependency PDK Client is preventing us from oving our app to the current version of AFNetworking


Replace FFTS with vkFFT In all library (non-test) code

FFTS is old and not as fast as I'd like.
We're moving towards Vulkan compute being a fundamental requirement for the platform, so it might be possible to replace all of our FFTs with it. We need to think about how this will work in the ATE use case though: can vulkan compute run on integrated GPUs / headless platforms? Will we need to put a discrete GPU in an ATE system?

The only remaining use of FFTS is DegradeSerialData() in TestWaveformSource.

Are you on x86 or ARM?
At one point, not sure if fixed, we had a bug where FFTS was disabled on apple silicon builds for macos but not for x86 builds.
FFTS is currently only used for the demo oscilloscope and some unit tests. It's not far from being completely gone but I've been too busy to completely remove it. If you feel adventurous and want to poke around the code, building without tests (I think this is default) and disabling DemoOscilloscope should get you a working binary in the meantime.

At the moment attempting to install on x86 (but I do have an M2 Mac I would also like to install it on).
Ok, I'll poke around a bit.  I already tried eliminating it from the cmake file but the build failed because it was not included.

Actually, not even the whole demo scope. Just making TestWaveformSource::DegradeSerialData be a no-op would probably be sufficient.

Seems to also be used in "Filter_FFT.cpp" as well.
Appears to be building ...

Seems to also be used in "Filter_FFT.cpp" as well.
Appears to be building ...

That's one of the unit tests. You should be able to turn off all of the tests with a CMake option.

It built with no errors.  When I run the app, it throws an error.  Not sure if this is a problem with my Homebrew installation of Vulkan or if there is some other issue.  There was no app bundle, just a "ngscopeclient" executable.  Is that the normal build of ngscopeclient for macOS?  Or, is there another step to build the app bundle that I missed?
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[BronzeMtlRenderCmdEncoder setVertexBuffer:offset:attributeStride:atIndex:]: unrecognized selector sent to instance 0x7fcc1f8abc00'
*** First throw call stack:
(
	0   CoreFoundation                      0x00007ff81607d2ee __exceptionPreprocess + 242
	1   libobjc.A.dylib                     0x00007ff815b6e21b objc_exception_throw + 48
	2   CoreFoundation                      0x00007ff81612265d -[NSObject(NSObject) __retain_OA] + 0
	3   libMoltenVK.dylib                   0x0000000181252f75 _ZNSt3__110__function6__funcIZN39MVKGraphicsResourcesCommandEncoderState10encodeImplEjE3$_6NS_9allocatorIS3_EEFvP17MVKCommandEncoderR19MVKMTLBufferBindingEEclEOS7_S9_ + 69
	4   libMoltenVK.dylib                   0x000000018124f704 _ZN39MVKGraphicsResourcesCommandEncoderState14encodeBindingsE14MVKShaderStagePKcbNSt3__18functionIFvP17MVKCommandEncoderR19MVKMTLBufferBindingEEENS4_IFvS6_S8_11MVKArrayRefIKjEEEENS4_IFvS6_R20MVKM	5   libMoltenVK.dylib                   0x0000000181250110 _ZN39MVKGraphicsResourcesCommandEncoderState10encodeImplEj + 272
	6   libMoltenVK.dylib                   0x00000001811bd4ec _ZN17MVKCommandEncoder17finalizeDrawStateE16MVKGraphicsStage + 76
	7   libMoltenVK.dylib                   0x00000001811b8193 _ZN17MVKCmdDrawIndexed6encodeEP17MVKCommandEncoder + 595
	8   libMoltenVK.dylib                   0x00000001811bb0ca _ZN17MVKCommandEncoder6encodeEPU27objcproto16MTLCommandBuffer11objc_objectP25MVKCommandEncodingContext + 250
	9   libMoltenVK.dylib                   0x00000001811bb52c _ZN16MVKCommandBuffer6submitEP31MVKQueueCommandBufferSubmissionP25MVKCommandEncodingContext + 236
	10  libMoltenVK.dylib                   0x0000000181229676 _ZN35MVKQueueFullCommandBufferSubmissionILm1EE20submitCommandBuffersEv + 102
	11  libMoltenVK.dylib                   0x0000000181227d76 _ZN31MVKQueueCommandBufferSubmission7executeEv + 198
	12  libMoltenVK.dylib                   0x00000001812261a9 _ZN8MVKQueue6submitI12VkSubmitInfoEE8VkResultjPKT_P9VkFence_T13MVKCommandUse + 329
	13  libMoltenVK.dylib                   0x0000000181171e7a vkQueueSubmit + 74
	14  ngscopeclient                       0x000000010e183b46 _ZN12VulkanWindow6RenderEv + 2742
	15  ngscopeclient                       0x000000010e1cf842 main + 850
	16  dyld                                0x00007ff815ba2366 start + 1942
)
libc++abi: terminating due to uncaught exception of type NSException
zsh: abort      ./ngscopeclient

All use of FFTS outside of unit tests is now gone as of 1a704437bf094bbcdfa214c1e5d1f0778fed149a. Closing this ticket (finally!)


enable fortran

Checklist

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

Enable fortran. Used for dftbplus.

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

@conda-forge-admin, please rerender


Different colors in tmux

vim-snazzy works great for me when I open vim directly in iterm2, however when opening vim in iterm2 in a tmux session, it has a weird hue to it.
Outside tmux (correct):

Inside tmux (incorrect):

Does this happen to anyone else?

I figured it out.
I had to put the following in my .tmux.conf (don't ask me why):
set -g terminal-overrides ',xterm-256color:Tc'
set -g default-terminal "tmux-256color"
set -as terminal-overrides ',xterm*:sitm=\E[3m'

@tim-phillips Glad you got it figured it out! Good work


Feature/totp integration

This pull request implements the issue https://github.com/jenkinsci/openstack-cloud-plugin/issues/332

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[x] Link to relevant issues in GitHub or Jira
[x] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

The fork of the repository was transferred to the @paydirekt-developer organisation. This organisation will handle further implementation changes and fixes of this Pull Request instead.


Display current desktop number in the systray icon

It was mentioned in #6 that you were working on running WinJump in the systray. A feature to improve this would be to display the current desktop number in the WinJump systray icon. I found a couple other projects that implement this feature. Since systray icons are so small, I lean towards solutions that don't have fancy icons but instead show the current desktop number as large as possible.

https://github.com/dankrusi/WindowsVirtualDesktopHelper
https://github.com/esvres/vdn
https://github.com/ridvanaltun/virtual-desktop-tray-icon
https://github.com/m0ngr31/VirtualDesktopManager

I have this working, just polishing up the code a bit should be up soon.

Done! Take a look at the new release @PhilipNelson5

This looks awesome and seems to be working so far!


Layout/LineContinuationLeadingSpace has a autofix logic error

Expected behavior
Describe here how you expected RuboCop to behave in this particular situation.
Nothing - string-splitting is "well defined" at the \n, or
eq(
  "    0:    1    2    3    4         5    6    7    8      \n    " \
  "8:    9   10   11   12        13   14   15   16      \n   " \
  '16:   17   18   19   20',
)

I would also not force changing the quotes here. "It doesn't look as pretty here".
Actual behavior
Describe here what actually happened.
Please use rubocop --debug when pasting rubocop output as it contains additional information.
eq(
  "    0:    1    2    3    4         5    6    7    8      \n    " \
  "8:    9   10   11   12        13   14   15   16      \n      " \
  '16:   17   18   19   20',
)

Steps to reproduce the problem
This is extremely important! Providing us with a reliable way to reproduce
a problem will expedite its solution.
eq(
  "    0:    1    2    3    4         5    6    7    8      \n"\
  "    8:    9   10   11   12        13   14   15   16      \n"\
  "   16:   17   18   19   20"
)

RuboCop version
Include the output of rubocop -V or bundle exec rubocop -V if using Bundler.
If you see extension cop versions (e.g. rubocop-performance, rubocop-rspec, and others)
output by rubocop -V, include them as well. Here's an example:
$ [bundle exec] rubocop -V
1.66.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.32.3, analyzing as Ruby 3.3, running on ruby 3.3.5) [x86_64-linux]
  - rubocop-performance 1.22.1
  - rubocop-rspec 3.1.0

The

Nothing - string-splitting is "well defined" at the \n,

is "a personal preference of mine". Call it a feature if you want.
I didn't get the exact intrinsics of Layout vs Style - maybe it is a feature too (and breaking change)
However, autofix is wrong:

That is a clear bug

If you want this to be understood as a clear bug, you need to change how you report these. Git diffs and unrunnable code snippets are not optimal. Instead show how it changes meaning:
puts "    0:    1    2    3    4         5    6    7    8      \n    " \
  "8:    9   10   11   12        13   14   15   16      \n      " \
  '16:   17   18   19   20'

    0:    1    2    3    4         5    6    7    8
    8:    9   10   11   12        13   14   15   16
      16:   17   18   19   20

========================================================================

puts   "    0:    1    2    3    4         5    6    7    8      \n"\
  "    8:    9   10   11   12        13   14   15   16      \n"\
  "   16:   17   18   19   20"

    0:    1    2    3    4         5    6    7    8
    8:    9   10   11   12        13   14   15   16
   16:   17   18   19   20

Indentation output changes.

If I had just reported the string, then the indentation, given the \ would've already disoriented the reader.
And if I wrote a whole rspec for it (hence the eq), it would not have been a MWE from a repro standpoint.
I didn't know that there was a requirement for the code to be executable - just valid Ruby

All of the autofix bugs I have opened so far have been dealt in about 1d-2d time.
Is this perhaps forgotten @koic? 🙏

FYI, the following configuration would probably give you the behaviour you're looking for here (keep string quotes consistent in a multiline string, and put spaces at the beginning of a string instead of the end in a continuation):
Style/StringLiterals:
  ConsistentQuotesInMultiline: true

Layout/LineContinuationLeadingSpace:
  EnforcedStyle: leading

FYI, the following configuration would probably give you the behaviour you're looking for here (keep string quotes consistent in a multiline string, and put spaces at the beginning of a string instead of the end in a continuation):
Style/StringLiterals:
  ConsistentQuotesInMultiline: true

Layout/LineContinuationLeadingSpace:
  EnforcedStyle: leading

Eeeh ... I'd have to evaluate what changes either does, if I really get into it.
Yes, in this case leading spaces would be best (and assuming \n is not considered space). But maybe the default looks better in other places.
This will probably be one disable I won't remove (if I was going to merge rubocop, which won't be happening apparently) - unless the block-style disable gets flagged as useless.


feat: Add global redis for registry cache

Description
Add a redis instance for caching registry requests.
Resolves #34
How Has This Been Tested?

Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

How did you test this?

This one is just for the infra, so tf plan basically.

How did you test this?

This one is just for the infra, so tf plan basically.

We have dev and staging environments. Please feel free to test your changes on one or the other (ideally dev). The README has instructions.

How did you test this?

This one is just for the infra, so tf plan basically.

We have dev and staging environments. Please feel free to test your changes on one or the other (ideally dev). The README has instructions.

Good thing you wanted to test it more thoroughly because the apply eventually failed.
I changed how the ECS containers are declared so that the JSON syntax can be checked instead of an opaque heredoc.


Update: PeterBienstman.Mnemosyne version. 2.8

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Ahh cool, @palenshus broke another PR by requesting a change but not changing the metadata.. (When a maintainer request a change it have to be applied)

All three labels are there but still not merging?

@palenshus Please approve changes so that it can be merged.


fix: carousel should stop immediately after interval is set to false

[x] Bug fix
[ ] New feature
[ ] Chore
[ ] Breaking change
[x] There is an open issue which this change addresses
[x] I have read the CONTRIBUTING document.
[x] My commits follow the Git Commit Guidelines
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.

[ ] I have updated the documentation accordingly.

[ ] My change requires a change to Typescript typings.

[ ] I have updated the typings accordingly.

[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Closes #2779

Thank you so much for the PR.
A better solution, in my opinion, would be to clear the interval if the props has changed.
Something like:
 componentDidUpdate(prevProps, prevState) {
    if (prevProps.interval !== this.props.interval) {
      this.setInterval();
    }
    if (prevState.activeIndex === this.state.activeIndex) return;
    this.setInterval();
  }

Currently, there is another bug which is once the carousel is stopped, it does not restart even if you toggle the interval back to a number. the above fixes that issue also. (We are calling setInterval again because it has a clearInterval inside it which takes care of clearing the interval and running the interval again only if needed).
Also the second thing is currently we are deprecating enzyme so it would be great if you can change the tests to run using react testing library instead.


remove breaking decode call

I started to experiment with this addon and it seems that on a mac, pages that are not already encoded in UTF-8 will fail to import and throw an exception.
I was testing the addon using this site, which is Shift_JIS encoded.
http://hukumusume.com/douwa/betu/world/07/22.htm
On load, this error is produced.
Error
An error occurred. Please start Anki while holding down the shift key, which will temporarily disable the add-ons you have installed.
If the issue only occurs when add-ons are enabled, please use the Tools>Add-ons menu item to disable some add-ons and restart Anki, repeating until you discover the add-on that is causing the problem.
When you've discovered the add-on that is causing the problem, please report the issue on the add-ons section of our support site.
Debug info:
Anki 2.1.7 (a6c34fd7) Python 3.6.7 Qt 5.12.0 PyQt 5.11.3
Platform: Mac 10.14
Flags: frz=True ao=True sv=1

Caught exception:
  File "/Users/jyore/Library/Application Support/Anki2/addons21/935264945/importer.py", line 105, in importWebpage
    webpage = self._fetchWebpage(url)
  File "/Users/jyore/Library/Application Support/Anki2/addons21/935264945/importer.py", line 52, in _fetchWebpage
    html = urlopen(url, context=context).read().decode('utf-8')
<class 'UnicodeDecodeError'>: 'utf-8' codec can't decode byte 0x83 in position 23: invalid start byte

After inspecting the code and testing, I think that the strict decode call should just be removed. Reason being that BeautifulSoup intrinsically transforms the content to UTF-8 during parsing (if possible). Because of this, it is not  necessary to do strict decoding before passing the document. This is basically what the non-mac version does, in the else statement as well.
After removing the decode call, the import works fine:

no problem. glad i could help :)
good work on the add-on so far. i am looking forward to using it more in the future.


Add Zip Codes to table: zip-code-data.csv

Problem statement
Some valid zip codes are not present in the zip-code-data table.  As a result, county lookups for those zip codes are failing.
Acceptance criteria

[ ] county lookups for the added zip codes are successful

To do

[ ] add zip codes to table

Completed and merged into Production.


Less Template, More Tekton

Expected Behavior
As said by @xihaLong:
Tekton going forward, as of last week's working group meeting, is following the template style of K8S, which can be interpreted as specifically avoiding a few things called out in the K8S community document here: https://github.com/kubernetes/community/blob/master/contributors/design-proposals/architecture/declarative-application-management.md
2/3rds of that K8S document may easily apply directly to how Tekton will want to pursue templates and reuse of items. I am trying to find exceptions to the rules I have found in that document, though I have none currently. Everything that can be expressed with a template within our API/controller could instead be pre-processed outside of Tekton's view and applied as API calls to Tekton.
In fact, if following a couple of the models for apply -f/ apply -k/ and merging YAML/JSON in a way transparent to K8S, there is a chance to additionally unroll patches and recover original values if desired - though that is not going to be required for now.
Tekton may avoid templating most forms of variable interpolation at the API layer for Tekton.
This will also mean avoiding creating any objects named "XYZTemplate" and/or "XYZTemplateType" in favor of allowing external tools to be chosen by a developer for generating the API calls/YAML/JSON files.
I propose any templating options be pushed to a CLI wrapper layer that makes the user experience for Tekton nicer, though leaves the controllers and API simpler for Tekton itself. (or tutorials that include Jinja/et al as a tool for manipulation)
So, I think I am not going too far out there to state: Let us not add too many intelligent forms of template logic into the controllers.
It may be argued that Pipelines and PipelineRuns also follow a similar template logic (PipelineRun is just providing a patch of values over the Pipeline template)
It follows to me that Tekton can be simpler and still achieve its goals. What to consider for removal? Anything performing variable interpolation/variable substitution that the user could have provided directly instead. Additionally, most other items that might be interpolated can potentially be generated and turned into a full API call/fully filled out YAML/filled out JSON. If not, then the extra state can be considered exposed via ConfigMap, Secret, or creating a resource/file/envvar for ingestion further along (each with different implications for debugging).
Filed an issue in Triggers as well for consideration. May file a small set of issues that refer to this issue in the near future to show a few practical implications of removing these templating and interpolation concepts from the controllers.
As far as variable interpolation itself is concerned, I believe it falls into the same set of issues as a DSL. There are many competing algorithms for interpolation.
Separately, providing a plethora of named parameters is directly discussed in the K8S community spec in this area referred to as parameterization. The design takes a negative approach to injecting named parameters in general vs. providing ConfigMaps, resources, or other debuggable separate pieces: https://github.com/kubernetes/community/blob/master/contributors/design-proposals/architecture/declarative-application-management.md#what-about-parameterization Tekton may also want to avoid adding too many bespoke dotted string names (or too many underscore_separated_names) that give deferred determination of concrete values, if convinced of other arguments in that document
See "Less Template, More Tekton" proposal https://docs.google.com/document/d/11QWLYPP-yj_K_ghQqHr3LFonFE9bfHwtooYq6qPhn8w/edit
Actual Behavior
We support variable replacement in Tasks and Pipelines.
Additional Info
This originally came up in #850, this issue exists to keep the discussion going and address just the syntax parts of #850.

@skaegi who I think has been looking into this recently


Added more information how to use Publish-DSCModuleAndMof cmdlet and …

Added more information how to use Publish-DSCModuleAndMof cmdlet and samples. Also updated the cmdlet to directly package DSC resources from the local depot $Source without copying them to powershell module path.

Hi @NarineM, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Narine Mossikyan). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


Invalid token q.js?v=1588706333081:854 Invalid token

I'm running into an error and I'm not sure what's causing it. The call is resulting in an "Invalid token" console message coming from line 854 from q.js. I'm running<EMAIL_ADDRESS>The value of value in the following code appears to be "Invalid token". I'm at a loss as to how to debug this issue. Any thoughts would be appreciated.
Promise.prototype.then = function (fulfilled, rejected, progressed) {
    var self = this;
    var deferred = defer();
    var done = false;   // ensure the untrusted promise makes at most a
                        // single call to one of the callbacks

    function _fulfilled(value) {
        try {
            return typeof fulfilled === "function" ? fulfilled(value) : value;
        } catch (exception) {
            return reject(exception);
        }
    }

q doesn't log anything to console, so it's likely an uncaught exception? You'll have to paste the entire stack to know.
You might be better off switching your code to Promise, which has native support and therefore better error management.


M3-5091: Bump postcss version

Description
We want to upgrade postcss to v8.2.10 or higher. This proved to be a bit tricky because it is not a direct dependency and we have several packages that had various different versions as a dependency:

Even the most recent versions of many of these packages use versions of postcss older than v8.2.10.
Before the changes (two instances of<EMAIL_ADDRESS>and one instance of<EMAIL_ADDRESS>listed):

Given the situation, I opted for selective dependency resolutions (also see our "Managing Dependencies" doc) by adding the desired postcss version number to the resolutions object in package.json and packages/manager/package.json. This defines the specified version as an override.
How to test
Test the app thoroughly to make sure no functionality has been impacted. Additionally, please check the console to make sure no new errors are being logged to the console from this change.

As discussed in standup today, going to close this PR out. Listed out the details of steps taken to try to get this working in a comment on the ticket.


Update navigation steps

Today the exports option is only available below the "Cost Management" blade. Added a step explicitely saying you have to navigate in that blade as this wasn't clear from the instructions.

@tvuylsteke : Thanks for your contribution! The author, @bandersmsft, has been notified to review your proposed change.

@tvuylsteke Thanks a bunch Thomas.
Contribution looks great.
#sign-off


Adv Duplicator 2 + Wiremod crash.

When spawning a contraption that uses wiremod with advanced duplicator 2, it causes a server crash.
Errors Produced:
Error, Could not find the output entity

This is all the information I have unfortunately.

@Ripmax unless you have a dupe we can use to test this, there's nothing we can do. Feel free to reopen with more resources for us.


Mobile view next challenge button

A camper relayed this to me:
There is some problem with the first challenge ending button on android with firefox browser. And in this moment I tried in chrome too, the issue is the same, the button is not active after the challenge is done.
Xperia Z1 compact, android: 5.0.2, firefox: 39.0
Chrome: 43.0.2357.93

This is an issue whenever the screen size is small. It will also happen on a desktop if you resize the window to half screen or less.


Concat crashes issue #170

Issue #170 and Issue #212 should now both be fixed.
Concatenated LDFC spectrograms do not show file joins but Concatenated summary index images do show file joins. Have previously included image showing what file joins now look like.

Yes there are certainly conflicts in Sandpit.cs.
Those in IndexProperties.cs are just to do with max and min index bounds.


Lsp finder window overlapping

Describe the bug

After the latest update, the lsp finder window UI is overlapping
Steps to reproduce
:Lspsaga lsp_finder
Expected behavior
windows should not overlap, like previous version
Neovim version (nvim -v)
0.8.3
lspsaga commit
9e58b1a
Terminal name/version
kitty

provide the reproduce step. i can't reproduce it.

provide the reproduce step. i can't reproduce it.

@glepnir Issue persists, even after 79f5a08. I can reliably reproduce by opening 3 vertical splits and sending sending lsp_finder in the middle split.

fix in tomorrow


PS4 server "Wow this server is popular!"

I am trying to connect to my dedicated server on PS4, it will appear as a lan connection but every time I try to connect it gives me the message "Wow this server is popular! Check back later to see if space opens up." despite the fact that there are no people currently on the server.

yeah i started seeing the same thing on PS4 with current version of minecraft 1.17.10

I have the same problem. Did you manage to solve the problem?

Afraid not, never got any response and not clever enough to figure it out
on my own!
On Fri, 4 Nov 2022, 12:40 leshkol, @.***> wrote:

I have the same problem. Did you manage to solve the problem?
—
Reply to this email directly, view it on GitHub
https://github.com/jhead/phantom/issues/145#issuecomment-1303454430, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ATNRIXCO3G33CODB536RXCLWGT74NANCNFSM4ZR5B3PQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Sadly.

Hey all, sorry about the delay. The Server offline message shown there in the LAN Games list means that phantom cannot connect to your remote server. If you have any logs from phantom, you can share them here to help troubleshoot.
Other things to consider:

Ensure the server is running and accessible from another device, like the iOS/Android app or a PC.
Check and make sure phantom is allowed through your PC's firewall

I'm getting this issue now on a Mac M1 connecting from PS4 and iOS. The iOS debug is:
7:27PM INF Opening connection to ***.***.***.***:**** for new client <IP_ADDRESS>:56232!
7:27PM INF Opening connection to ***.***.***.***:****
7:27PM INF Received LAN ping from client: <IP_ADDRESS>:56232
7:27PM DBG Received Unconnected Pong from server: [28 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 255 255 0 254 254 254 254 253 253 253 253 18 52 86 120 0 60 77 67 80 69 59 112 104 97 110 116 111 109 32 194 167 99 83 101 114 118 101 114 32 111 102 102 108 105 110 101 59 51 57 48 59 49 46 49 52 46 54 48 59 48 59 48 59 59 59 67 114 101 97 116 105 118 101 59 49 59]
7:27PM DBG Unconnected Pong: &{[0 0 0 0 0 0 0 0] [0 0 0 0 0 0 0 0] [0 255 255 0 254 254 254 254 253 253 253 253 18 52 86 120] {MCPE phantom §cServer offline 390 1.14.60 0 0<PHONE_NUMBER>660344355  Creative 1  }}
7:27PM INF Sent server offline pong to client: <IP_ADDRESS>:56232
7:27PM INF New connection from client <IP_ADDRESS>:56232 -> <IP_ADDRESS>:19132
7:27PM WRN read udp <IP_ADDRESS>:60789->***.***.***.***:****: recvfrom: connection refused

and PS4/5
7:25PM INF Opening connection to ***.***.***.***:**** for new client <IP_ADDRESS>:56232!
7:25PM INF Opening connection to ***.***.***.***:****
7:25PM INF Received LAN ping from client: <IP_ADDRESS>:56232
7:25PM DBG Received Unconnected Pong from server: [28 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 0 255 255 0 254 254 254 254 253 253 253 253 18 52 86 120 0 60 77 67 80 69 59 112 104 97 110 116 111 109 32 194 167 99 83 101 114 118 101 114 32 111 102 102 108 105 110 101 59 51 57 48 59 49 46 49 52 46 54 48 59 48 59 48 59 59 59 67 114 101 97 116 105 118 101 59 49 59]
7:25PM DBG Unconnected Pong: &{[0 0 0 0 0 0 0 0] [0 0 0 0 0 0 0 0] [0 255 255 0 254 254 254 254 253 253 253 253 18 52 86 120] {MCPE phantom §cServer offline 390 1.14.60 0 0<PHONE_NUMBER>660344355  Creative 1  }}
7:25PM INF New connection from client <IP_ADDRESS>:56232 -> <IP_ADDRESS>:19132
7:25PM INF Sent server offline pong to client: <IP_ADDRESS>:56232
7:25PM WRN read udp <IP_ADDRESS>:51185->***.***.***.***:****: recvfrom: connection refused

Hope that helps!


firebase-app-distribution 0.0.1

New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

PR with such step already exists: #2151

Unfortunately, due to Firebase App Distro not being public at the time, we could not disclose, that we've been working together with a contributor on this step earlier.
As stated on the linked PR, we decided to go with this PR because of the following reasons:

Step test CI is set up for both platforms
Step does input validation
Step employs our usual step practices, such as config printing, colored logging

closing this PR in favor of version 0.1.0, recommended changes applied.


Provide longer audio window for better context

When a segment is hard to transcribe (either fully or the first word), it sometimes helps to hear a couple of more seconds earlier.
Would be good to have an ability to hear those too when needed.

Reported by @yoavcaspi.

Another option: link to original content, with exact timestamp.

Is your feature request related to a problem? Please describe.
The current tagging interface lacks context for auto-transcribed segments. Accurate tagging of short audio segments sometimes requires additional context. Insufficient context can lead to transcription errors.
Describe the solution you'd like
The tagging interface should include an option to listen to a few seconds of audio before and after the segment that needs to be tagged. This additional context will help ensure accurate transcription.
Describe alternatives you've considered
An alternative solution could be to provide a direct link to the original audio file, indicating the time from which the segment to be transcribed is extracted. This would allow users to listen to the surrounding context manually.
Additional context
Implementing this feature will require changes to both the backend and the frontend of the tagging interface. This enhancement is expected to improve the accuracy and efficiency of the transcription process.


Products doesn't add to cart if user is not authorized

I get bug which I don't understand how fix it. Products doesn't add to cart if user is not authorized. Although user is created:

but products don't add to cart. The screenshot above show than user create more than one time. I try add product to cart and UNRECOGNIZED same user is created every time.

Are you sure that there's a Customer for the given User object?
There is a long section about Customer and User

Yes, I use demo tutorial. Maybe my my models.py for email_auth app is wrong?
# -*- coding: utf-8 -*-
from __future__ import unicode_literals

import re
from django.contrib.auth import get_user_model
from django.contrib.auth.models import AbstractBaseUser, PermissionsMixin, UserManager as BaseUserManager
from django.core import validators
from django.core.exceptions import ValidationError
from django.core.mail import send_mail
from django.db import models
from django.utils import timezone
from django.utils.encoding import python_2_unicode_compatible
from django.utils.translation import ugettext_lazy as _

class UserManager(BaseUserManager):
    def get_by_natural_key(self, username):
        return self.get(is_active=True, **{self.model.USERNAME_FIELD: username})

    def _create_user(self, email, password,
                     is_staff, is_superuser, **extra_fields):
        now = timezone.now()
        if not email:
            raise ValueError('The given email must be set')
        email = self.normalize_email(email)
        is_active = extra_fields.pop("is_active", True)
        user = self.model(email=email, is_staff=is_staff, is_active=is_active,
                          is_superuser=is_superuser, last_login=now,
                          date_joined=now, **extra_fields)
        user.set_password(password)
        user.save(using=self._db)
        return user

    def create_user(self, email, password=None, **extra_fields):
        is_staff = extra_fields.pop("is_staff", False)
        return self._create_user(email, password, is_staff, False,
                                 **extra_fields)

    def create_superuser(self, email, password, **extra_fields):
        return self._create_user(email, password, True, True,
                                 **extra_fields)

@python_2_unicode_compatible
class User(AbstractBaseUser, PermissionsMixin):
    USERNAME_REGEX = re.compile('^[\w.@+-]+$')

    username = models.CharField('Логин', max_length=30,
        help_text=_("Required. 30 characters or fewer. Letters, numbers and @/./+/-/_ characters"),
        validators=[validators.RegexValidator(USERNAME_REGEX, _('Enter a valid username.'), 'invalid')])

    # copied from django.contrib.auth.models.AbstractUser
    last_name = models.CharField('Фамилия', max_length=30, blank=True)
    first_name = models.CharField('Имя', max_length=30, blank=True)
    middle_name = models.CharField('Отчество', max_length=30, blank=True)

    # this is the only field, which differs from the default implementation
    email = models.EmailField(_("Email address"), null=True, default=None, blank=True, max_length=254)
    is_staff = models.BooleanField(_("staff status"), default=False,
                                   help_text=_("Designates whether the user can log into this admin site."))
    is_active = models.BooleanField(_("active"), default=True,
                                    help_text=_("Designates whether this user should be treated as active. "
                                                "Unselect this instead of deleting accounts."))
    date_joined = models.DateTimeField(_("date joined"), default=timezone.now)

    objects = UserManager()

    USERNAME_FIELD = 'email'
    REQUIRED_FIELDS = []

    class Meta:
        db_table = 'auth_user'
        verbose_name = 'Пользователь'
        verbose_name_plural = 'Пользователи'

    def get_username(self):
        if self.is_staff:
            return self.username
        return self.email or '<anonymous>'

    def __str__(self):
        return self.get_username()

    def get_full_name(self):
        full_name = ' '.join([self.last_name, self.first_name, self.middle_name])
        full_name = full_name.strip()
        if full_name:
            return full_name
        return ''

    def get_short_name(self):
        if self.first_name:
            return self.first_name
        return self.email

    def email_user(self, subject, message, from_email=None):
        send_mail(subject, message, from_email, [self.email])

    def validate_unique(self, exclude=None):
        super(User, self).validate_unique(exclude)
        if get_user_model().objects.exclude(id=self.id).filter(is_active=True, email__exact=self.email).exists():
            msg = 'Пользователь с e-mail адресом "{}" уже существует'
            raise ValidationError({'email': msg.format(self.email)})

what is the difference to the original email_auth.models.py? Why don't you just add a django.po file to your locals for Russian translations?

Yes, django.po is better approach. Later will be so.
Now I saw many little difference in original email_auth.models.py and my 'email_auth' (not only in UserManager). This issue is still opened for me. And I will find error independently as soon as I can and write about bug. Please, while don't close this issue.

email_auth.py shouldn't be there in the first place. However, I had to add a different implementation to django/contrib/auth.py because nobody nowadays logs in using a self assigned user password.
Therefore, if you want to add additional fields or methods, add them to the Customer model. This is the place intended for it.

Thank you for hints! I rethink my opinion about email_auth. And I cannot find my bug in this issue. I totally copied demo project with its settings to my project and bug hide. Where bug was I don't know. Now my project work and issue may be closed. Jacob, thank you for help!


docs: polish translation

Hi @Dekier   thanks for taking care of this. We recently merged the Cookbook section https://github.com/Tresjs/tres/issues/557 so we will need to update the PR.
Steps:
The .md files under the examples directory are the same ones used in the cookbook, so just rename the directory from pl/examples to pl/cookbook.

Add a pl/cookbook/index.md like this https://github.com/Tresjs/tres/pull/558/files#diff-65c3e40a0b2bdcfae6c3be42b004bfa4e079a250fefed0fca56af95b33a209ca
Add the correspondent translated frontmatter meta to each one of the .md files inside the cookbook, like this https://github.com/Tresjs/tres/pull/558/files#diff-587026bcdc81388b1c6b391cdbdd7c2184a16b752a489c544a9d91c9e4b8f674
Update the config links https://github.com/Tresjs/tres/pull/558/files#diff-d9b1069ba7dab3215b86ba2b70b13a13977d3c09c11bdd8d468a0d7e8c686641

If you have any questions or you need help with this, let me know 🙏🏻


"Could not start video source" error on Android 7 and Chrome 109

Hey, first of all, thank you for this lib, it's awesome.
Unfornutelly I found one glitch. When I run Storybook demo on the Android 7 and Chrome 109.0.5414 video is not starting. Instead, I see the "Could not start video source" error.

I have the same issue in my react app using your library. On Android 8 it works well.
This Android device can run camera video (check other websites to test if it's device related issue).
Is there a chance to look at it from your side? Thank you in advance!

Could you send me the output of Media Device List. This will give me a better idea of the camera on that device.


TSC meeting for January 30, 2024

This should be an all-sigs meeting.
Please update your boards and consider putting anything in here you'd like to be read to anyone present even if you cannot yourself be present.

No updates from SIG-Core in the past month.


Not working on macOS 12.3

Code 127: /bin/bash: python: command not found
https://www.macrumors.com/2022/01/28/apple-removing-python-2-in-macos-12-3/

This workflow is based on the top of https://github.com/deanishe/alfred-workflow. Once alfred-workflow adds support for macOS 12.3, we can move forward.

fixed in 2.0.0


Generalize the fitting procedure

It may be interesting to leave to the user the possibility of choosing the fitter.
See https://docs.astropy.org/en/stable/modeling/reference_api.html#module-astropy.modeling.fitting
https://github.com/Gaiana/nirdust/blob/76d8591300afe43dcf6d6ff87191f91184064040/nirdust.py#L103

I'm leaving LevMarLSQFitter hardcoded because it's the only fitter that provides the uncertainty value for the fit. LinearLSQFitter can't be used because the black body model is not linear on the temperature.


Improve tooltips in the editor profiler to mention the script name

Port of https://github.com/godotengine/godot/pull/49247 to master, which can be cherry-picked to 3.x.
This closes https://github.com/godotengine/godot/issues/48835.

Thanks!

Cherry-picked for 3.4.


[TextField] add ref callback

Problem description
The TextField component ref's callback returns null instead of the root node of the component.
More globally, it would be great to have this callback on every Material UI's components.
Link to minimal working code that reproduces the issue
https://codesandbox.io/s/kzJWNlX5

Versions

Material-UI: 1.0.0-alpha.14
React: 15.5.3
Browser: Chrome 58 on Windows 10 x64

This behaviour is explained by the fact that TextField is a  stateless functional component. What's the use case for that reference? I think that good rule of dumps is to add rootRef callback property whenever needed. Also, we could add a inputRef callback property to access the underlying input.

Following the React docs... works for me, but I get warnings.
https://facebook.github.io/react/docs/refs-and-the-dom.html
// TextField component
import React, { Component } from 'react'
import TextField from 'material-ui/TextField'

export default (props) => (

        <TextField {...props} ref={props.inputRef}  />

)

import React, {Component} from 'react'

import { TextField } from '../TextField'

class LoginBox extends Component {

    constructor() {
        super()
    }

    send(event) {
        event.preventDefault()
        console.log(this.email.input.value
        console.log(this.password.input.value})
    }

    render() {
        return (

                    <form onSubmit={this.send.bind(this)}>

                            <TextField
                                name="email"
                                type="email"
                                inputRef={el => this.email = el}
                            />

                            <TextField
                                name="password"
                                type="password"
                                inputRef={el => this.password = el}}
                            />
                            <input type="submit" value="Send" />

                    </form>

        );
    }
}

warning.js?9398f22:36 Warning: Unknown prop inputRef on  tag. Remove this prop from the element. For details, see https://fb.me/react-unknown-prop
"material-ui": "^0.18.1",
"react": "^15.5.4",
"react-dom": "^15.5.4"

@maisumbruno My issue concerns the next branch.
@oliviertassinari I wanted to use it for my Autocomplete component which doesn't work anymore without the ref.

@jgoux What about?
<TextField
  inputProps={{
    ref: (node) => { console.log('node', node); },
  }}
/>

Well, that won't work yet. I'm on it.

I think that exposing a inputRef property would be more appropriate as React is handling ref differently from the other properties.

@jgoux Which Autocomplete component?

Thanks for the inputRef addition!
I'm still missing the ref to the root node of TextField. 😄
@Nagle This one : https://codesandbox.io/s/oQ0nkl5pk

You can try innerRef of withStyles().

innerRef did the trick! 👍
I'm curious, why adding a new prop like innerRef instead of making the component a class and relying of React's native ref ?

@jgoux The component is already a class so you can get a ref/

I'm not sure I follow.
TextField seems to be a functional component : https://github.com/callemall/material-ui/blob/next/src/TextField/TextField.js#L9
I tried adding a ref callback to it first, but it always return null

I tried adding a ref callback to it first, but it always return null

Yes, innerRef wouldn't be working if it was using a functional component. Functional component do not support ref callback.

Use can do this by using inputRef
Look at the my answer
https://stackoverflow.com/questions/31446751/how-to-get-password-field-value-in-reacts-material-ui/47329368#47329368

I was needing ref to get the input value , i did the simple trick after passing the ref to material ui text field.
<TextField
id="required"
ref={"password"}
defaultValue={"password"}
/>;
I used the this to get the value
let sPassword = this.refs.password.input.value;

Not sure if I had the same issue as the original issue but with RootRef as a wrapper I got the DOM ref I needed.


Add a GCE health checks example

This change is

Coverage remained the same at 45.914% when pulling 801d256e1e4f47dc05609b4541bb30db4a8b1b03 on bprashanth:gce_health_checks into cb60085b17d95b0e8e0f24eee59e396851419dd7 on kubernetes:master.

Merging, only docs


Register yourselves to the club.

You can fork this repo and add in your details, then send a PR to confirm your registration.

Add myself in member.json

Hey @RudrakshK please assign this to me


Feature JSON type

I'm not sure if this is even possible, but it would nice to have a JSON type.
Which validates that the value of this type contains just primitives or objects with properties consisted of primitives, arrays or objects (no functions, or Date objects etc.)
I tried to implement it but I keep hitting the recursive type limitation, however you have solved it somehow since DeepReadonly should have same problem.

@Ciantic Did you mean a type called something like JSONSerializable ? This sounds pretty cool :) One thing to mind that we can serialize an object to JSON even when there are functions or non-primitive objects(e.g. Date objects) though. As long as there are corresponding toJSON method defined and the returning object doesn't contain these JSON-unserializable values.

I know about toJSON but the JSON type I want could be used to validate e.g. a REST api result types that they contain only JSON primitives etc. Maybe it could be named JSONSerializable, or simply JSONType.
I think if it is possible to do, it could be highly usable and I think a good contribution to TypeScript standard library also.

I think I know what you mean. Often I mark json loaded objects as Object where in fact it could be a little bit more specific.
Here's discussion + additional comments in TS repo https://github.com/Microsoft/TypeScript/issues/1897

Okay, looks like the issue is not novel, but somehow I thought it would be possible using similar things as DeepReadonly. I'm closing this because it's probably not possible, since that issue has existed for so long, someone would have found a way already.


Re-Create VM that was manually deleted.

If you deploy a vm using terraform proxmox provider and delete it manually in the proxomox interface terraform will crash.
│ Error: vm '100' not found
Expected behaviour would be to re-create the vm if missing from the infra and present in the code.
At least this is what happen on AWS.

+1

Error: vm '161' not found
In my case a LXC container

By deleting VM's from the ui, you create an inconsistency between the state files and the real world. The intended solution by HashiCorp is to remove the ressource from the state file by using terraform state rm {ressource} .
I am not sure if it's realy a great idea to differ from Terraforms default behavior here.

I am not sure if it's realy a great idea to differ from Terraforms default behavior here.

Are you sure that it is default? I expect that I can 'fix' the infrastructure by terraform if someone made a manual mistake by clicking delete in AWS. Same should be with Proxmox


chore(ci): add codecov.yml

Codecov Report

Merging #254 (a672ed8) into main (04f6e23) will decrease coverage by 10.35%.
The diff coverage is n/a.

:exclamation: Current head a672ed8 differs from pull request most recent head 25f9d4c. Consider uploading reports for the commit 25f9d4c to get more accurate results

@@             Coverage Diff             @@
##             main     #254       +/-   ##
===========================================
- Coverage   95.95%   85.60%   -10.36%
===========================================
  Files          35       35
  Lines         396      396
  Branches        5        5
===========================================
- Hits          380      339       -41
- Misses         14       56       +42
+ Partials        2        1        -1

Flag
Coverage Δ

unittests
85.60% <ø> (-10.36%)
:arrow_down:

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

Impacted Files
Coverage Δ

libs/components/src/lib/button/button.template.ts
17.64% <0.00%> (-80.40%)
:arrow_down:

Continue to review full report at Codecov.

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


[v1] Warning if Bootstrap 3 classes are used

Check for some Bootstrap 3 classes, at least those where I assume that are unlikely to be used for anything else, if they are detected then a new warning with the is id W018 is printed.
Status

Code: Done
Tests: TBD, help appreciated
Wiki: TBD, help appreciated

For the wiki it would be nice to give some hints on what the Bootstrap 4 replacements for some of the things, see also https://github.com/twbs/bootlint/pull/410#issuecomment-327455304
/cc @XhmikosR @Johann-S

I've changed it now to more specific selectors where it makes sense.
/cc @Johann-S @XhmikosR

Isn't this ready to be merged?

Tests and wiki missing


Auth

[ ] - Auth

[ ] - Authorize Attribute on EndPoints
[x] - Authorize Attribute on Hub
[x] - Authorize Attribute on Hub methods

Hub and hub method auth done.
Rest of the work is punted to RTM: https://github.com/aspnet/SignalR/issues/583


Changes for publish code coverage task

Testing done : manual, unit tests added.

Hi @AshwiniChalla, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Ashwini Challa). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;


improvements to build process

fix issue w/ package versioning post-deploy
fix badge issues on pypi page
auto-build & deploy a new docker file when a new Liu is published

#1 fixed

completed


Captain's Log

I am creating this issue, mainly for writing how some things are progressing

I've been trying lately to write a successful test for different CRS. It kind of works but I've more or less ended up writing parts of Proj4Leaflet which, even the version for leaflet 1.0 beta does't really properly work. All the vectors are now displayed fine in a couple of different coordinate systems (that I am familiar with) (that's the 2770 and 2100). I have some issues though mainly with tile requests. I hope everything will be resolved soon.

Obviously I won't make the 1.3 milestone. So I am going to move all milestones a month. There are a lot of features available on the developer branches but I don't wont to release them without proper testing. So I'll take my time, any help will be appreciated.

I managed to write a test for EPSG:2100 it can be found at https://elasticrash.github.io/Angular.io.MapViewer/example/#/prj

documentation is completely, out of date. so before I make any changes to the repository, I will first bring that up to date.

My plans before moving to any changes are:

more tests
fix any bug that might occur
create a small risk like game, to extend my examples

The past week I moved houses, the new house is still full with unopened boxes. My pc is still not plugged. So development will be slowed down significantly for the next two weeks

examples were updated to use the next release, in order to demonstrate custom crs projections

I've been busy lately consulting for a really big Angular 2 project. So I am only spending like 1/5 of the time I used to spend on this project. This will propably go on for the next three months. I am really close on releasing a really good update. But I need extented testing before I publish it.

currently building the library's website while improving angular cli compatibility issues

Minor release but huge update, I got around to properly build my library in both umd and esm. Structure changed a lot as well as some minor logic and depedencies. Function wise is identical


Fixed and added warnings.

This is very useful if you have multiple Tileset images and don't know which one is giving the warning.
The second commit will save an hour-long headache if someone accidentally applies Phaser.Physics.Arcade to a group instead of Phaser.Physics.ARCADE and can't understand why their sprite.body property is null (which just happened to me).

Nice one.


Node debug processes hang

VSCode Version: 1.0.0-alpha
OS Version: Windows 10

Steps to Reproduce:

clone vscode-eslint
cd eslint && npm install && code-alpha .
cd eslint-server && npm install && code-alpha .
press Ctrl+Shift+B in both instances
Install eslint npm package into a workspace folder (npm install eslint)
Switch to the one having eslint open
Press F5. In the dev instance open the workspace folder created in step 5
Switch to code having eslint-server open
Attach the debugger to the server by pressing F5
Go to the dev instance and reload the instance
Go back to code having eslint-server open
Try to reattach the debugger. It fails

I am pretty sure that this worked in the past.

@dbaeumer is this before or after the electron update?

I saw this before the electron update.

In the end, I could follow steps but I am running first into #5313

With latest changes from Ben regarding #5313 I can no longer repro this on OS X.
@dbaeumer will also try on windows with latest
@egamma let me know if you still see it with latest

Closing this. Please reopen @dbaeumer if you still see it

Didn't see it again.


Unbreak comment links

The current code links comments with ?context=1000 appended, in an attempt to show the full context tracing back to the original comment on the post. This doesn’t work, as the current interface (definitely on mobile, and I believe I tested on desktop as well) just never loads any comments.
Some testing has revealed 3 to be the largest context supported. More than 3 results in three levels of context being displayed, starting n levels above the linked-to comment, meaning that the comment itself is absent, and more than 8 or so (don’t remember the exact number) just breaks comment loading altogether.

Alternatively, we could use the superior old UI, which still supports context=1000.

Alternatively, we could use the ~superior~ old UI, which still seems to support context=1000 from my brief testing.

That’s not the default Reddit experience that many users are familiar with, nor is it especially mobile-friendly.

That’s not the default Reddit experience that many users are familiar with, nor is it especially mobile-friendly.

Yeah… too bad that it's not possible to allow this to be user-configurable (since the link is obviously the same for everyone).

Yeah I've never had issues with this since I still use old Reddit but I guess it's time to bite the bullet.


Test

Testing Mac host to Linux

@lfpino Should this PR be closed? The last change in here was 12 days ago.


Remove temporary logging and adjust test column names

Fixes #2494.
This removes the debug logging added in #2490.  It also adjusts the column names in test_round_robin_sort_fallback to disambiguate the schema from a very similar query being built in test_repartition_df.  As seen in the Dataproc test failures, the test that executes just before the one that fails ends up having the same schema, so it's not easy to tell if the test accidentally captured the plan from the previous query.  From the Dataproc test log before this change:
06:46:47  integration_tests/src/main/python/repart_test.py::test_repartition_df[4096-2345-[('a', Array(String))]][IGNORE_ORDER({'local': True})] PASSED [ 81%]
06:46:47  integration_tests/src/main/python/repart_test.py::test_round_robin_sort_fallback[[('a', Array(String))]][IGNORE_ORDER({'local': True}), ALLOW_NON_GPU(ShuffleExchangeExec,RoundRobinPartitioning)] FAILED [ 81%]

build


Highlight only till the task description ends rather than the whole line

Hi,
I have been using this extension for a while and discovered there is an option called "isWholeLine" which we can enable but that enables BG color to the whole line which can seem a little out of the place when going through the code (see image below).
Feature:
Can we build a feature where background color can be enabled only for the words/the sentence?

Thanks
Aditi

"isWholeLine" works well where you really need something to stand out.
Background color will just be applied to the words/sentence that the regex
defines if "isWholeLine" is off. So I'm not sure what new feature is
required. Please elaborate if I have missed something.

Message ID: @.***>

@jgclark Hi. Is it possible to mark only the commented part, not the whole line? Just like it works in Webstorm?

Background color will just be applied to the words that the regex defines if "isWholeLine" is off.


Can't find enclosing method void onClick when enabling Proguard

Hi, I was trying to assembleRelease in Gradle and I get the following error
can't find enclosing method 'void onClick(android.content.DialogInterface,int)' in program class
and we are using version apt 'com.jakewharton:butterknife-compiler:8.4.0'

What makes you think this is related to Butter Knife?

Because it is annotated in @OnClick(R.id.button_delete)

Nevermind, it seems I need to rebuild project first before generating a release. Thanks! Apologize!


Primary colored radio buttons

Currently, your radio buttons look like this:

Change the color of the radio buttons to your primary color.

Use the custom input component from here. https://reactstrap.github.io/components/form/#Custom-Inputs
Use inspect element to determine how to override the colors for these - you switched components because browser radios are unstylable, unlike these. As me if you need help with this - I've done it for ymath before.

the radio button acts a bit strange when using the space key


Load more home page cards

Who is this PR for?
educators, HS NGE/10GE to start
What problem does this PR fix?
The home page only loads a fixed number of cards on page load; there's no way to browse back further.
What does this PR do?
Adds a "see more" link to load more cards into the feed.
This PR also adds a time_now method to TestPals, consistent with what's in NowContainer in the JS specs.  This is to help make it easier to make correct timestamps, etc. across tests by fixing test time to a particular moment.
Screenshot (if adding a client-side feature)

Checklists

[x] Author checked latest in IE - Home page
[x] Author improved specs for code in need of better test coverage
[x] Author included specs for new code

selfie


Azure Cosmos DB API for MongoDB 4.0 Updates

We'll need to update the CLI for the new supported ServerVersion: 4.0

Cosmos

code checked in dev


File Manager issue

Hello @zainawanworktech,
Open your config file "config/filesystems.php" and check the key "mediamanager", make sure it contains 'driver' => 'local' or any other driver that Laravel support and run below artisan command.
php artisan config:clear

Hello @zainawanworktech,
Open your config file "config/filesystems.php" and check the key "mediamanager", make sure it contains 'driver' => 'local' or any other driver that Laravel support and run below artisan command.
php artisan config:clear

i am already this code try but they have still error come in my project

i am already this code try but they have still error come in my project

Hi @zainawanworktech,
You have added 'mediamanager' disk outside the "disks" setting, which is incorrect.
Please correct and move this under "disks" key.

Hi @zainawanworktech,
You have added 'mediamanager' disk outside the "disks" setting, which is incorrect.
Please correct and move this under "disks" key.

kindly attached the correct file

kindly attached the correct file

Hi @zainawanworktech,
My humble request to you that please read Laravel documentation before setup. This issue is related to the wrong configuration.
Please see below screenshot to setup new disk.

Hi @zainawanworktech,
My humble request to you that please read Laravel documentation before setup. This issue is related to the wrong configuration.
Please see below screenshot to setup new disk.

Hi @zainawanworktech,
Didn't receive any feedback from your side. So, I am closing this issue. You can re-open this if you face the same issue again.
The solution is already mentioned in the above response.

Hi @zainawanworktech,
Didn't receive any feedback from your side. So, I am closing this issue. You can re-open this if you face the same issue again.
The solution is already mentioned in the above response.

Saturday Sunday is not working day so I can not check feedback

Saturday Sunday is not working day so I can not check feedback

ok done they have worked fine

ok done they have worked fine

Glad to know that your problem has been solved.

Glad to know that your problem has been solved.


srt: 1.4.2 -> 1.4.3

Motivation for this change
https://github.com/Haivision/srt/releases/tag/v1.4.3
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 127890 at 951939e4 run on aarch64-linux 1

7 packages marked as broken and skipped:

gtkd
odpdown
pulseeffects-legacy
pulseeffects-pw
termplay
tilix
yaxg

228 packages skipped due to time constraints:

MIDIVisualizer
adapta-gtk-theme
almanah
apostrophe
aravis
astroid
audio-recorder
balsa
bookworm
bubblemail
...

90 packages built successfully:

baresip
brasero
brasero-original
enlightenment.rage
evince (gnome.evince)
farstream
fcitx5-gtk
fcitx5-with-addons
ffmpeg-full
gajim
gmrender-resurrect
gnome-network-displays
gnome.gnome-chess
gnome.gnome-sound-recorder
gscrabble
gst_all_1.gst-editing-services
gst_all_1.gst-plugins-bad
gst_all_1.gst-rtsp-server
gst_all_1.gst-vaapi
gtk4
helvum
imagination
kaidan
kooha
libadwaita
libsForQt5.phonon-backend-gstreamer (libsForQt515.phonon-backend-gstreamer ,plasma5Packages.phonon-backend-gstreamer)
libsForQt514.phonon-backend-gstreamer
lollypop
mailnag
megapixels
mopidy-iris
mopidy-local
mopidy-mopify
mopidy-mpd
mopidy-mpris
mopidy-musicbox-webclient
mopidy-podcast
mopidy-scrobbler
mopidy-somafm
mopidy-soundcloud
mopidy-subidy
mopidy-tunein
mopidy-youtube
mousai
notejot
pantheon.elementary-camera
pantheon.elementary-videos
parlatype
picard
pidgin
pidgin-carbons
pidgin-latex
pidgin-mra
pidgin-msn-pecan
pidgin-opensteamworks
pidgin-osd
pidgin-otr
pidgin-skypeweb
pidgin-window-merge
pidgin-xmpp-receipts
pithos
pitivi
purple-discord
purple-facebook
purple-hangouts
purple-lurch
purple-matrix
purple-slack
purple-xmpp-http-upload
python38Packages.gym
python38Packages.moderngl-window
python38Packages.pydub
python38Packages.pyglet
python38Packages.pytmx
python39Packages.gym
python39Packages.moderngl-window
python39Packages.pydub
python39Packages.pyglet
python39Packages.pytmx
qt-video-wlr
qutebrowser
qxmpp
recapp
skype4pidgin
sound-juicer
srt
srt-live-server
telepathy-farstream
telepathy-haze
toxprpl

Thank you nice bot.


message edit/deletion counts

i.e. so we can rank people based on how many messages they sneakily delete, possibly with deleterious social results.
would be nice! Possible?

It should be possible using the GuildMessageDeleteEvent.
Sounds like a good idea :)

Just implemented with bcc86a8aea837e86afa2a2a21d90c161c1118a84 ;)

Turns out my implementation (the deletesinquiry feature) is currently incorrect as it increments a user's count even if someone else deletes their message. It works fine if everyone just deletes their own messages, but that obviously doesn't happen all the time. I'll edit this comment if/when I delete it.
The problem: You can't identify who deleted a message within GuildMessageDeleteEvent. Only the author of the message. There is a way to check audit logs with hikari (so you can see who the deleter is), however it's apparently only available using some cursed method using this: https://www.hikari-py.dev/hikari/api/rest#hikari.api.rest.RESTClient.fetch_audit_log
And even then audit logs do not create an event for people who delete their own messages. Audit logs only show message deletes when someone deletes someone else's message. So you'd have to use this audit logs method in combination with a method identifying when a user deletes their own message (possible with GuildMessageDeleteEvent -- you could cross-check it with the audit log check).

So, it's currently still available as a command, but I might remove it. I'm not reopening this issue as it seems based on the above this feature is not easy to make (idek if it's possible -- if you find a way do say!). I wanted to add these comments here as I put them in a different issue by mistake.


ikhal crashes when translating event TZs across date boundaries

First, congratulations on khal, a great application.
Now, the bug.  I created the following event:
BEGIN:VCALENDAR
PRODID:-//Mozilla.org/NONSGML Mozilla Calendar V1.1//EN
VERSION:2.0
BEGIN:VTIMEZONE
TZID:Europe/London
BEGIN:DAYLIGHT
TZOFFSETFROM:+0000
TZOFFSETTO:+0100
TZNAME:BST
DTSTART:19700329T010000
RRULE:FREQ=YEARLY;BYMONTH=3;BYDAY=-1SU
END:DAYLIGHT
BEGIN:STANDARD
TZOFFSETFROM:+0100
TZOFFSETTO:+0000
TZNAME:GMT
DTSTART:19701025T020000
RRULE:FREQ=YEARLY;BYMONTH=10;BYDAY=-1SU
END:STANDARD
END:VTIMEZONE
BEGIN:VEVENT
CREATED:20150701T095323Z
LAST-MODIFIED:20150706T105425Z
DTSTAMP:20150706T105425Z
UID:5507-5593B880-1-15FFE820
SUMMARY:airport transfer :: LHR t5 - home [plega]
ORGANIZER;CN=[...]com
ATTENDEE;PARTSTAT=ACCEPTED;RECEIVED-SEQUENCE=1;ROLE=REQ-PARTICIPANT;RECEIVE
D-DTSTAMP=20150701T095323Z;CN=plega:mailto:[...]
DTSTART;TZID=Europe/London:20150710T180500
DTEND;TZID=Europe/London:20150710T200000
LOCATION:[...]
CLASS:PUBLIC
TRANSP:OPAQUE
SEQUENCE:1
X-MOZ-GENERATION:1
END:VEVENT
END:VCALENDAR
This displays correctly in ikhal when "local_timezone = Europe/London".  However, if I set "local_timezone = Australia/Sydney" it crashes.  From testing it, I believe (though I haven't tested it exhaustively) that this will happen with any value for local_timezone which results in the above event being split across two days.
Traceback (most recent call last):
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/init.py", line 653, in start_pane
loop.run()
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 273, in run
self.screen.run_wrapper(self._run)
File "/usr/lib64/python2.7/site-packages/urwid/raw_display.py", line 267, in run_wrapper
return fn()
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 338, in _run
self.event_loop.run()
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 668, in run
self._loop()
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 705, in _loop
self._watch_filesfd
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 389, in _update
self.process_input(keys)
File "/usr/lib64/python2.7/site-packages/urwid/main_loop.py", line 489, in process_input
k = self._topmost_widget.keypress(self.screen_size, k)
File "/usr/lib64/python2.7/site-packages/urwid/wimp.py", line 642, in keypress
return self._current_widget.keypress(size, key)
File "/usr/lib64/python2.7/site-packages/urwid/container.py", line 1123, in keypress
return self.body.keypress( (maxcol, remaining), key )
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/base.py", line 32, in keypress
return urwid.Columns.keypress(self, size, key)
File "/usr/lib64/python2.7/site-packages/urwid/container.py", line 2264, in keypress
key = w.keypress((mc,) + size[1:], key)
File "/usr/lib64/python2.7/site-packages/urwid/container.py", line 1123, in keypress
return self.body.keypress( (maxcol, remaining), key )
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/calendarwidget.py", line 174, in keypress
return super(CListBox, self).keypress(size, key)
File "/usr/lib64/python2.7/site-packages/urwid/listbox.py", line 986, in keypress
key = focus_widget.keypress((maxcol,),key)
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/calendarwidget.py", line 127, in keypress
key = super(DateCColumns, self).keypress(size, key)
File "/usr/lib64/python2.7/site-packages/urwid/container.py", line 2278, in keypress
self.focus_position = j
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/calendarwidget.py", line 97, in _set_focus_position
self.on_date_change(self.contents[position][0].original_widget.date)
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/init.py", line 629, in show_date
self.eventscolumn.current_date = date
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/init.py", line 221, in current_date
self.eventcount = self.events.update(date)
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/init.py", line 163, in update
eventcolumn=self.eventcolumn),
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/ui/init.py", line 62, in init
super(U_Event, self).init(self.event.compact(self.this_date))
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/khalendar/event.py", line 295, in compact
rstring = self._compact_datetime(day)
File "/usr/lib/python2.7/site-packages/khal-0.5.0-py2.7.egg/khal/khalendar/event.py", line 333, in _compact_datetime
'please supply a day this event is scheduled on')
ValueError: (u'Something went wrong while displaying "5507-5593B880-1-15FFE820.ics"', u'please supply a day this event is scheduled on')

I think this bug will only appear if the event is scheduled on two different days in the different timezones. I believe this is fixed with the new database layout (see branch event_rewrite). I hope to merge that code rather sooner than later.

I'm looking forward to that.
Meanwhile, I've run some specific tests and here is some more detail, which suggests that the problem isn't exactly as you have described it:

If I create an event which falls across midnight in my local TZ, it displays in ikhal on both relevant days.  [That is the desired behaviour.]

If I create an event, say, UK BST 14:00-19:00, and then change local TZ to Pacific/Auckland or Pacific/Samoa it will crash.  [Bad]

Using the above example, if I set local TZ as Australia/Sydney it displays the event as "23:00-".  If I move to the next day, however, it does not display the rest of the event. [Bad]

So it looks as though it is the evaluation of DTSTART which is causing the crash, although incorrect handling of event length is causing the truncation.

Sorry, didn't mean to close it (!)

Could we re-open this ?  I get what seems to be the same problem on v0.6.0.
TEST 1: enter event as 14:00-15:00 BST, then change localtime to Aust/Sydney.  Event shows as "23:00-" OK.
TEST 2: event 15:00-17:00 BST, but change localtime to Aust/Sydney and it crashes.
TEST 3: event 15:00-16:00 BST, also crashes.  Sample output (TEST 3):
Traceback (most recent call last):
File "/usr/bin/ikhal", line 9, in
load_entry_point('khal==0.6.0', 'console_scripts', 'ikhal')()
File "/usr/lib/python2.7/site-packages/click/core.py", line 610, in call
return self.main(*args, **kwargs)
File "/usr/lib/python2.7/site-packages/click/core.py", line 590, in main
rv = self.invoke(ctx)
File "/usr/lib/python2.7/site-packages/click/core.py", line 782, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "/usr/lib/python2.7/site-packages/click/core.py", line 416, in invoke
return callback(*args, **kwargs)
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/cli.py", line 315, in interactive_cli
controllers.interactive(build_collection(ctx), ctx.obj['conf'])
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/controllers.py", line 193, in interactive
description='do something')
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/ui/init.py", line 596, in init
self.eventscolumn = EventColumn(pane=self)
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/ui/init.py", line 191, in init
self.events = EventList(eventcolumn=self)
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/ui/init.py", line 140, in init
self.update()
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/ui/init.py", line 163, in update
eventcolumn=self.eventcolumn),
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/ui/init.py", line 62, in init
super(U_Event, self).init(self.event.compact(self.this_date))
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/khalendar/event.py", line 295, in compact
rstring = self._compact_datetime(day)
File "/usr/lib/python2.7/site-packages/khal-0.6.0-py2.7.egg/khal/khalendar/event.py", line 333, in _compact_datetime
'please supply a day this event is scheduled on')
ValueError: (u'Something went wrong while displaying "KBBZP5B2TIEYZQOTF8DX87WJ3T6L4WI8B63W.ics"', u'please supply a day this event is scheduled on')

Sure, I'll have a look tomorrow

It looks like you are using khal v.0.6 now, right? This version does not yet contain the fix for the above probvem. Please test with the latest version from git (either by downloading from git (git clone https://github.com/geier/khal.git khal) or by downloading and than creating a file version.py by hand, see #230).

Now running khal, version 0.6.1.dev52+ng57fa447
It looks good - I can't break it. ;-)
You can close this bug.
Thanks
Philip
On Tue, Jul 21, 2015 at 12:49:17PM +0100, Christian Geier wrote:

It looks like you are using khal v.0.6 now, right? This version does not yet contain the fix for the above probvem. Please test with the latest version from git (either by downloading from git (git clone https://github.com/geier/khal.git khal) or by downloading and than creating a file version.py by hand, see #230).

Reply to this email directly or view it on GitHub:
https://github.com/geier/khal/issues/222#issuecomment-123278950

--
Philip Allen, Executive Chairman, CoreFiling Limited
http://www.corefiling.com
Phone: +44-1865-203192  Mobile: +44-7956-160959

alright and thanks for reporting this.


Raw field data seems to be arbitrarily trimmed

Describe the bug
We recently upgraded to 8.157.14 and noticed a regression in the data stored within our raw fields. We have a plain and simple raw type defined in our schema:
field source_data type raw {}

When we PUT a new document to Vespa, we assign a base64 encoded string as the value for this field. The base64-decoded data within this field is a JSON blob, which we store as a representation of data from the "source" (original API we consumed from). The field is for internal replay/testing, mostly. We don't represent it in summaries but we do have some applications bootstrapped from the data within this raw field.
Since the version upgrade, we are seeing a case where some source_data representations are partial. I speculate, but in my (non exhaustive) tests I did see base64 strings without padding get serialized correctly, however those without, not so.
I also found this commit, which exists within the range where I've noticed the error: https://github.com/vespa-engine/vespa/commit/1d818e96e1d6002726f6b89967e0835116251f15
When I downgrade to 8.153.19 I no longer see the issue.
To Reproduce
I wrote a simple bash script which replicates the issue. Note you'll need to swap Vespa versions between runs as my script simply queries the document API.
Versions to consider:

8.155.19 (the first version I see the issue on)
8.153.19 (the newest tag where the issue is not present)

Here's the script:
#!/bin/bash

set -e

docid=test-b64-plain
source_data='{"user":{"_expandable":{"personalSpace":""},"_links":{"base":"https://cyberdyne-dev.atlassian.net/wiki","context":"/wiki","self":"https://cyberdyne-dev.atlassian.net/wiki/rest/api/user?accountId=5ed690ff037d930c16c01680"},"accountId":"5ed690ff037d930c16c01680","accountType":"atlassian","displayName":"Bartek Ciszkowski","email":"bartek@atolio.com","profilePicture":{"height":48,"isDefault":false,"path":"/wiki/aa-avatar/5ed690ff037d930c16c01680","width":48},"publicName":"Bartek Ciszkowski","type":"known"},"groups":[{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/confluence-users"},"id":"34b261a0-30db-4326-8290-de2cb5f50da8","name":"confluence-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/jira-software-users"},"id":"81155928-2c39-421f-905e-62969093d5c8","name":"jira-software-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/site-admins"},"id":"a2feb27a-f7ef-4c73-9092-102c9ad2270b","name":"site-admins","type":"group"}]}'

b64_source_data=$(echo $source_data | base64)
echo ""
value='{"fields": {"source_data": "'$b64_source_data'","active":false}}'

echo "🦁 Deleting existing document to cleanup test"
curl -X DELETE http://localhost:8080/document/v1/lumen/coredoc/docid/$docid

echo ""
echo "🦁 Creating document";
echo $value;

curl -X POST -H "Content-Type: application/json" -d "$value" http://localhost:8080/document/v1/lumen/coredoc/docid/$docid

echo ""
echo "🦁 Getting source_data from document";
out=$(curl -s http://localhost:8080/document/v1/lumen/coredoc/docid/$docid | jq -r .fields.source_data)
decoded=$(echo $out | base64 -d)
jq . <<< $decoded

The script POSTs a payload like so to the document API (base64 encoding the source_data JSON blob):
{"fields": {"source_data": "eyJ1c2VyIjp7Il9leHBhbmRhYmxlIjp7InBlcnNvbmFsU3BhY2UiOiIifSwiX2xpbmtzIjp7ImJhc2UiOiJodHRwczovL2N5YmVyZHluZS1kZXYuYXRsYXNzaWFuLm5ldC93aWtpIiwiY29udGV4dCI6Ii93aWtpIiwic2VsZiI6Imh0dHBzOi8vY3liZXJkeW5lLWRldi5hdGxhc3NpYW4ubmV0L3dpa2kvcmVzdC9hcGkvdXNlcj9hY2NvdW50SWQ9NWVkNjkwZmYwMzdkOTMwYzE2YzAxNjgwIn0sImFjY291bnRJZCI6IjVlZDY5MGZmMDM3ZDkzMGMxNmMwMTY4MCIsImFjY291bnRUeXBlIjoiYXRsYXNzaWFuIiwiZGlzcGxheU5hbWUiOiJCYXJ0ZWsgQ2lzemtvd3NraSIsImVtYWlsIjoiYmFydGVrQGF0b2xpby5jb20iLCJwcm9maWxlUGljdHVyZSI6eyJoZWlnaHQiOjQ4LCJpc0RlZmF1bHQiOmZhbHNlLCJwYXRoIjoiL3dpa2kvYWEtYXZhdGFyLzVlZDY5MGZmMDM3ZDkzMGMxNmMwMTY4MCIsIndpZHRoIjo0OH0sInB1YmxpY05hbWUiOiJCYXJ0ZWsgQ2lzemtvd3NraSIsInR5cGUiOiJrbm93biJ9LCJncm91cHMiOlt7Il9saW5rcyI6eyJzZWxmIjoiaHR0cHM6Ly9jeWJlcmR5bmUtZGV2LmF0bGFzc2lhbi5uZXQvd2lraS9yZXN0L2V4cGVyaW1lbnRhbC9ncm91cC9jb25mbHVlbmNlLXVzZXJzIn0sImlkIjoiMzRiMjYxYTAtMzBkYi00MzI2LTgyOTAtZGUyY2I1ZjUwZGE4IiwibmFtZSI6ImNvbmZsdWVuY2UtdXNlcnMiLCJ0eXBlIjoiZ3JvdXAifSx7Il9saW5rcyI6eyJzZWxmIjoiaHR0cHM6Ly9jeWJlcmR5bmUtZGV2LmF0bGFzc2lhbi5uZXQvd2lraS9yZXN0L2V4cGVyaW1lbnRhbC9ncm91cC9qaXJhLXNvZnR3YXJlLXVzZXJzIn0sImlkIjoiODExNTU5MjgtMmMzOS00MjFmLTkwNWUtNjI5NjkwOTNkNWM4IiwibmFtZSI6ImppcmEtc29mdHdhcmUtdXNlcnMiLCJ0eXBlIjoiZ3JvdXAifSx7Il9saW5rcyI6eyJzZWxmIjoiaHR0cHM6Ly9jeWJlcmR5bmUtZGV2LmF0bGFzc2lhbi5uZXQvd2lraS9yZXN0L2V4cGVyaW1lbnRhbC9ncm91cC9zaXRlLWFkbWlucyJ9LCJpZCI6ImEyZmViMjdhLWY3ZWYtNGM3My05MDkyLTEwMmM5YWQyMjcwYiIsIm5hbWUiOiJzaXRlLWFkbWlucyIsInR5cGUiOiJncm91cCJ9XX0K","active":false}}

In this example, there is no padding and when I read the source_data field back from the document API (This is the Getting source_data from document line in the script), it looks correct:
{"user":{"_expandable":{"personalSpace":""},"_links":{"base":"https://cyberdyne-dev.atlassian.net/wiki","context":"/wiki","self":"https://cyberdyne-dev.atlassian.net/wiki/rest/api/user?accountId=5ed690ff037d930c16c01680"},"accountId":"5ed690ff037d930c16c01680","accountType":"atlassian","displayName":"Bartek Ciszkowski","email":"bartek@atolio.com","profilePicture":{"height":48,"isDefault":false,"path":"/wiki/aa-avatar/5ed690ff037d930c16c01680","width":48},"publicName":"Bartek Ciszkowski","type":"known"},"groups":[{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/confluence-users"},"id":"34b261a0-30db-4326-8290-de2cb5f50da8","name":"confluence-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/jira-software-users"},"id":"81155928-2c39-421f-905e-62969093d5c8","name":"jira-software-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/site-admins"},"id":"a2feb27a-f7ef-4c73-9092-102c9ad2270b","name":"site-admins","type":"group"}]}

However, I make a small amendment to source_data (simply added another k to my email in the JSON blob .. any valid change which will cause an addition of padding)
The POST body looks like this now (has padding):
{"fields": {"source_data": "eyJ1c2VyIjp7Il9leHBhbmRhYmxlIjp7InBlcnNvbmFsU3BhY2UiOiIifSwiX2xpbmtzIjp7ImJhc2UiOiJodHRwczovL2N5YmVyZHluZS1kZXYuYXRsYXNzaWFuLm5ldC93aWtpIiwiY29udGV4dCI6Ii93aWtpIiwic2VsZiI6Imh0dHBzOi8vY3liZXJkeW5lLWRldi5hdGxhc3NpYW4ubmV0L3dpa2kvcmVzdC9hcGkvdXNlcj9hY2NvdW50SWQ9NWVkNjkwZmYwMzdkOTMwYzE2YzAxNjgwIn0sImFjY291bnRJZCI6IjVlZDY5MGZmMDM3ZDkzMGMxNmMwMTY4MCIsImFjY291bnRUeXBlIjoiYXRsYXNzaWFuIiwiZGlzcGxheU5hbWUiOiJCYXJ0ZWsgQ2lzemtvd3NraSIsImVtYWlsIjoiYmFydGVra0BhdG9saW8uY29tIiwicHJvZmlsZVBpY3R1cmUiOnsiaGVpZ2h0Ijo0OCwiaXNEZWZhdWx0IjpmYWxzZSwicGF0aCI6Ii93aWtpL2FhLWF2YXRhci81ZWQ2OTBmZjAzN2Q5MzBjMTZjMDE2ODAiLCJ3aWR0aCI6NDh9LCJwdWJsaWNOYW1lIjoiQmFydGVrIENpc3prb3dza2kiLCJ0eXBlIjoia25vd24ifSwiZ3JvdXBzIjpbeyJfbGlua3MiOnsic2VsZiI6Imh0dHBzOi8vY3liZXJkeW5lLWRldi5hdGxhc3NpYW4ubmV0L3dpa2kvcmVzdC9leHBlcmltZW50YWwvZ3JvdXAvY29uZmx1ZW5jZS11c2VycyJ9LCJpZCI6IjM0YjI2MWEwLTMwZGItNDMyNi04MjkwLWRlMmNiNWY1MGRhOCIsIm5hbWUiOiJjb25mbHVlbmNlLXVzZXJzIiwidHlwZSI6Imdyb3VwIn0seyJfbGlua3MiOnsic2VsZiI6Imh0dHBzOi8vY3liZXJkeW5lLWRldi5hdGxhc3NpYW4ubmV0L3dpa2kvcmVzdC9leHBlcmltZW50YWwvZ3JvdXAvamlyYS1zb2Z0d2FyZS11c2VycyJ9LCJpZCI6IjgxMTU1OTI4LTJjMzktNDIxZi05MDVlLTYyOTY5MDkzZDVjOCIsIm5hbWUiOiJqaXJhLXNvZnR3YXJlLXVzZXJzIiwidHlwZSI6Imdyb3VwIn0seyJfbGlua3MiOnsic2VsZiI6Imh0dHBzOi8vY3liZXJkeW5lLWRldi5hdGxhc3NpYW4ubmV0L3dpa2kvcmVzdC9leHBlcmltZW50YWwvZ3JvdXAvc2l0ZS1hZG1pbnMifSwiaWQiOiJhMmZlYjI3YS1mN2VmLTRjNzMtOTA5Mi0xMDJjOWFkMjI3MGIiLCJuYW1lIjoic2l0ZS1hZG1pbnMiLCJ0eXBlIjoiZ3JvdXAifV19Cg==","active":false}}

And the resulting source_data field after querying from the document API is a partial JSON representation:
{"user":{"_expandable":{"personalSpace":""},"_links":{"base":"https://cyberdyne-dev.atlassian.net/wiki","context":"/wiki","self":"https://cyberdyne-dev.atlassian.net/wiki/rest/api/user?accountId=5ed690ff037d930c16c01680"},"accountId":"5ed690ff037d930c16c01680","accountType":"atlassian","displayName":"Bartek Ciszkowski","email":"bartekk@atolio.com","profilePicture":{"height":48,"isDefault":false,"path":"/wiki/aa-avatar/5ed690ff037d930c16c01680","width":48},"publicName":"Bartek Ciszkowski","type":"known"},"groups":[{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/confluence-users"},"id":"34b261a0-30db-4326-8290-de2cb5f50da8","name":"confluence-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/jira-software-users"},"id":"81155928-2c39-421f-905e-62969093d5c8","name":"jira-software-users","type":"group"},{"_links":{"self":"https://cyberdyne-dev.atlassian.net/wiki/rest/experimental/group/site-admins"},"id":"a2feb27a-f7ef-4c73-9092-102c9ad2270b","n

This is not what I expect. I am unclear what in particular about the referenced commit (https://github.com/vespa-engine/vespa/commit/1d818e96e1d6002726f6b89967e0835116251f15) may be causing this, or maybe there are other recent changes.
In another test, I post a padding base64 string:
#!/bin/bash

set -e

docid=test-b64-plain
b64_source_data="RG9uJ3QgYmVsaWV2ZSBoaXMgbGllcw=="

echo ""

value='{"fields": {"source_data": "'$b64_source_data'","active":false}}'

echo "🦁 Deleting existing document to cleanup test"
curl -X DELETE http://localhost:8080/document/v1/lumen/coredoc/docid/$docid

echo ""
echo "🦁 Creating document";
echo $value;

curl -X POST -H "Content-Type: application/json" -d "$value" http://localhost:8080/document/v1/lumen/coredoc/docid/$docid

echo ""
echo "🦁 Getting source_data from document";
curl -s http://localhost:8080/document/v1/lumen/coredoc/docid/$docid | jq -r .fields.source_data

The resulting output is RG9uJ3QgYmVsaWV2ZSBoaXMgbGllcw, which seems to suggest that Vespa is stripping the padding on read?
Please let me know if I can provide more detail!
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

OS: macOS
Infrastructure: Self hosted (local)
Versions 8.153.19 <-> 8.155.19

Looking at the linked commit, this seems to be an unintentionally breaking change.  Java's base64 decoder will successfully decode a base64 string whether padded or not, but other languages are stricter.
For Go, the standard decoder will fail with an error if the padding is missing.  It offers a base64.RawStdEncoding instead which conversely expects there to be no padding and will error if it sees it, so is only a partial solution as the code is now dependent on specific versions of Vespa to decode succesfully.
Other languages such as Python don't offer an easy alternative and (afaik) will always fail if the padding is missing.

Thanks for the report. This indicates that we need to put back padding.
As we are now making raw fields a first class citizen we tried to clean up the different rendering we had of raw fields.
We decided to go for no padding, assuming most decoders handled both equally well.
According to the comments here that does not seem to be the case.
ETA for 8.161

Now all changes has been merged and both unit and system tests have passed.
Part of vespa-8.161.12 and on. Will be released tomorrow if release pipeline works as expected.

@baldersheim Super! Thank you for the quick turn around on this. FWIW, we resolved internally by creating a new type which can handle padded/unpadded b64 data, but makes sense to use what is most common across decoders.


The production docker image uses dev dependencies

The command in the docker image executes the start script which requires nodemon and pino-pretty which are both dev dependencies

nodemon does not seems needed to me as it is meant to reload changes in the code in the fly

Maybe it should not do that, we can have a seperate command to start in production

https://github.com/threefoldtech/tfchain_activation_service/pull/22

4954a5cb49450550812141349fcb7f74174cf993


[Word] (Desktop only) Add moniker

Related to https://github.com/OfficeDev/office-js-docs-reference/pull/1301

Docs Build status updates of commit bd328ef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 88b1f88:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel