What is the correct behavior for repr in Jupyter?

Currently console repr prints the table's structure in the console and the table's contents (as HTML) in Jupyter. This should probably be consistent. It seems wrong to ever print the entire table as the result of a trivial print(table). Possibilities:

Print the table's memory location. (Default repr behavior.)
Print the table's structure. (Current console behavior.)
Print the entire table data. (Current Jupyter behavior.)
Print a subset of the table. (Max x columns, max y rows.)

@ghing @TaurusOlson Would appreciate both of your input on this issue since you hacked on the relevant pull requests.

I suddenly remember that I had all of this working with repr at one point and it felt like the wrong behavior. I think the structure printing should happen when the table is stringified, rather than when it's repr'ed. (Which is what Taurus suggested.) The question then is whether it makes sense for Jupyter to behave differently, given that the HTML rendering hook only works for repr.

I've reverted repr to display memory location for console usage and have implemented __str__ to print the table structure. This functionality seems correct to me.
I'm not sure the right behavior for Jupyter. On the one hand, Jupyter encourages overriding repr to display partial contents (pandas displays a subset of rows and columns).  (Since you can't pass a parameter to repr.)
Also, Jupyter's usage of repr is not what repr is for, so that's a bit frustrating.

Hi @onyxfish,
I haven't thought about that for a while. I'll get the latest version of agate and will look at that tomorrow or the day after.

I think the structure printing should happen when the table is stringified, rather than when it's repr'ed. (Which is what Taurus suggested.)

My initial proposition was to use the method print_structure and you thought the result would make a good output for __unicode__. I don't really know the difference between __unicode__ and __str__ but I think you made the right choice: repr should display memory location for console usage and __str__ should print the table structure.
I'll play with agate in the Jupyter notebook and will let you know what I think.

Hey @TaurusOlson, thanks for the feedback. FYI, __str__ is simply the Python 3 name for __unicode__. The way I implemented it will work for Python 2 as well.

The thing that sucks is that Jupyter conventions and Python conventions seem at odds here.  It's really helpful to be able to see the table quickly with repr in Jupyter, but I'd be open to a compromise where the ability to output the table as HTML is available as a public method (which I believe @onyxfish already implemented) of agate.Table and the repr behavior exists in a plugin.

@ghing I think that's an good compromise. The Table.print_html method can be used to implement this in a plugin with very little overhead. In fact, that's how it's implemented now:
    def _repr_html_(self):
        """
        Print HTML version of the table for Jupyter
        (via :meth:`Table.print_html`).

        See http://ipython.readthedocs.org/en/stable/config/integrating.html?highlight=_repr_html_#rich-display
        """
        html = six.StringIO()

        self.print_html(output=html)

        return html.getvalue()

I'm going to remove the _repr_html_ method from core, for now, but I'm open to bringing it back into core in the future if we can figure out an elegant way.
It seems the ideal solution would be if there was a way to flag for Jupyter that a given string is HTML. (With a subclass of the string object, for instance.) Then we could more easily support this case.

I gave a look to the latest version (commit b891191) and everything we talked about looks good. However, in Jupyter print_html returns the string of html table not the html table I expected. Is this normal?

Yes, there is nothing directly I can do about that. It'll only render the HTML if it's returned from the _repr_html_ method. See above for my thinking around removing that from this release. Ideally, Jupyter would provide some other flag to render that correctly.

I've wasted hours trying to resolve this on at least three separate occasions. It can't be solved to my own satisfaction. The inconsistency between the behavior of repr and str annoys me too much to fold it into core, especially given what a mess it makes of the code. Also, at the end of the day, I'm not sure the formatted HTML tables are actually even more readable than the plain-text ones.
Note to future me: next time work on something more useful instead. You aren't going to find a solution to this that you like.
Closing.


Add cache

Description
Caching dataset helps reduce database calls when replaceWhere is invoked multiple times.
Checklist

[X] I have run this code in development and it appears to resolve the stated issue
[X] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have updated the README.md file

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@fead66f). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             develop      #61   +/-   ##
==========================================
  Coverage           ?   43.56%
==========================================
  Files              ?       22
  Lines              ?      544
  Branches           ?       42
==========================================
  Hits               ?      237
  Misses             ?      307
  Partials           ?        0


The CSS properties to control the padding for dismissible Alerts is not working

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
The vertical padding of the alert can be controlled by the CCS property --bs-alert-padding-y, however, doing so causes odd display effects when using a dismiss button.
E.g.

    Holy guacamole! You should check in on some of those fields below.

    <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close"></button>

Produces the following:

I would expect the dismiss button's padding to be adjusted based on --bs-alert-padding-y to ensure consistent resizing.
Likewise, the dismiss button padding should also adapt when --bs-alert-padding-x is set.
Reduced test cases

    Holy guacamole! You should check in on some of those fields below.

    <button type="button" class="btn-close" data-bs-dismiss="alert" aria-label="Close"></button>

What operating system(s) are you seeing the problem on?
Windows
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
5.2

To solve this issue, one possible solution is to adjust the dismiss button's padding using the same CSS property. You can achieve this by adding the following CSS rule to your stylesheet:
.alert-dismissible .btn-close {
--bs-alert-padding-y: var(--bs-alert-padding-y);
--bs-alert-padding-x: var(--bs-alert-padding-x);
}
This rule sets the --bs-alert-padding-y and --bs-alert-padding-x variables of the dismiss button to the same values as the alert's padding, which ensures consistent sizing.
Alternatively, you could use the calc() function to adjust the dismiss button's padding based on the alert's padding, like so:
.alert-dismissible .btn-close {
padding-top: calc(var(--bs-alert-padding-y) - 0.5rem);
padding-right: calc(var(--bs-alert-padding-x) - 0.5rem);
padding-bottom: calc(var(--bs-alert-padding-y) - 0.5rem);
padding-left: calc(var(--bs-alert-padding-x) - 0.5rem);
}
This rule subtracts 0.5rem from the alert's padding to get the dismiss button's padding, which should maintain consistent sizing.
I hope this helps solve the issue you are facing! Thank you


two warnings

elixir 1.8.x, poison 4
remote: warning: module attribute @api_testnet_host was set but never used
remote:   lib/bitmex/rest/http_client.ex:9
remote:
remote: warning: function Poison.Parser.parse/1 is undefined or private. Did you mean one of:
remote:
remote:       * parse!/2
remote:
remote:   lib/bitmex/websocket/parser.ex:40

The second warning seems to be caused by the undocumented API changes in poison 4.0. I've submitted a PR to it. Meanwhile, I'm updating bitmex to use Poison.decode/1 for now instead of the removed function in Poison.Parser.


Example Code

I am a newbye on this platform and i use adafruit mqtt library first because it has qos1 supportfor publish and subscribe and last will with also qos1,but on arduino mega it doesn't work.
i found your library and it's look very nice but i can't find much information to set lastwill and qos1 for subscribe. can you up an example much detailed? thank you so much.

@intel2004
The subscribe example is available in examples/PubSub/PubSub.ino.
Just change MqttClient::QOS0 to MqttClient::QOS1 in line#157.
I don't have the ready example for LWT.  Take a look at question #3 where I explained how to enable it.
Please let me know if you need my help.

just add following code below before connect to setup LWT :D
/* setup LWT */
      options.willFlag                = true;
      options.will.topicName.cstring  = "topic_WLT";
      options.will.message.cstring    = "your message when disconnect";
      options.will.retained           = 1;
      options.will.qos                = 1;


Combine angle calculation in to a single function

All of the airplane angle calculation should be refactored into its own function. this will make that code reusable for other purposes and help simplify the code in the handleMouseUp function.
Lines 129-143
https://github.com/VarynInc/game-experiments/blob/main/path-animate/demo.html#L129

Done in #21 and #22


Background image no longer available in PageHero

Summary
I was relying on the BackgroundImage in the PagerHero which was on the collection entity, but that has been removed from the component, and removed from the outer scope -- so how is one supposed to get the PageHero background image back without a bunch of work?   Also the new sub categories in the PageHero seem not great and were way better in the menus.  It is disappointing the PRs that are getting merged with no description, because it make the direction of this project and the chosen changed totally unclear.   Seems every week the UI heads in some random new direction.
Steps to reproduce
Merge main branch into a fork that styles the site and watch for breaking changes and weird directions to head in.
A link to a reproduction repository
No response
What OS are you using?
No response
Your next.config.js file
n/a
Builds logs (or link to your logs)
n/a
Function logs
n/a

Ah part of my problem was that the collection and the category objects are in fact different, so I modified those pages to continue passing in those components as that is where the background image comes from.  I am not sure why though a PR from 19 hours ago with no description abandons supporting the background image for the category/collection.

Oh man and the cool load more products thing is gone and it just loads everything from the start now :-/

I suppose I can get used to it loading everything but eventually I'll probably have too many listings for that to be super mobile friendly.  Are there plans to add pagination then instead of the load more thing?   Why show the sub categories in the PageHero though (I removed it and put the back the image which is definitely better) when the nav menus already show the subcategories?  Ref deployment:  www.mattscoinage.com

Yeah I see that before the changes I was getting around a 30-37 in performance score, now I get around a 10:

@matteius the goal of the project is to showcase how Saleor API works and provide guidance for the community. We do not intend this to be a generic theme that provides a UI kit for building online shops. Feel free to use component libraries of your choice.

Are there plans to add pagination then instead of the load more thing?
Yes

Yeah I see that before the changes I was getting around a 30-37 in performance score, now I get around a 10:
To increase the score I would advise you to fetch data on the server and use getStaticProps to cache pages. This is what we are planning to do as well

the goal of the project is to showcase how Saleor API works and provide guidance for the community.

@timuric It still would be nice to have descriptions in the PRs and some kind of roadmap or plan published so that we can understand the direction of the showcase.  I am definitely not the only person who has forked this repository to make an online store and presumably there will be important fixes, even security items that get patched upstream that we would have originally brought in from the example.
The problem though is either the showcase example is going to be useful towards a framework to build an online store which feels like the ideal to me where we extend the storefront and that leads to other features and enhancements being contributed back tot he framework that further lets us build a configurable UI store.  That would be the full potential of the project, not just an example of an ever changing UI showcase and then when someone decides to build a store with saleor they get all of the improvements at that point in time, fork it, and then good luck pulling in upstream changes.  This is definitely the easier path forward to developing a prototype, but it feels less ideal, and at what point is the showcase considered done?

To increase the score I would advise you to fetch data on the server and use getStaticProps to cache pages.

Can you expand upon this idea?   When you say fetch data on the server, are you talking about the pnpm run start server that runs the react-storefront?  I have this deployed in a Docker container running this way and I bypassed it to the CDN for images.  What would it mean to fetch data on the server and use getStaticProps? -- is this similar to trying to build static pages by hitting the graph QL API during the build cycle and deploying that (with the caveat that to get product updates and changes the site has to be recompiled) are you suggesting a more dynamic way of fetching data live in order to get a better performance score?  Ideally there are ways to improve performance without taking away the dynamic aspects that make it useful to edit products and see them in real time.

@matteius
We provide several packages for building an online store:

Payments hooks and Checkout Page that is customizable. This is something that you might want to use without forking to keep up to date with all features.
https://github.com/saleor/saleor-checkout

SDK for building stores, which you can use as a package and update as Saleor core evolves
https://github.com/saleor/saleor-sdk

Saleor storefront as a boilerplate, which is meant as a starting point, is not intended to be used as a theme. We believe each store has different needs and in most cases would work quite different from our example. We have ambitious goals to improve the storefront, there are going to be many changes and commits and we don't think it would be realistic to stay up to date with them.  We do acknowledge that the community also needs themes that would be customizable and upgradable like https://www.reactstorefront.io/ or https://www.vuestorefront.io/ but we don't have that planned any time soon. One thing I can promise is that we would try to better separate presentation and business logic to make at least some parts of the code more portable.

Can you expand upon this idea?

You can use getStaticProps with fallback blocking, which would generate pages on request instead of build time
https://github.com/saleor/react-storefront/blob/1b61fb6373e627eb1383612f0df2e54d75494741/pages/[channel]/[locale]/products/[slug].tsx#L38
Set revalidate to X seconds to keep cache fresh
https://github.com/saleor/react-storefront/blob/1b61fb6373e627eb1383612f0df2e54d75494741/pages/[channel]/[locale]/products/[slug].tsx#L57
Our product page already is using this approach, the issue with the product listing page is that it is using filters that depend on URL query parameters which unfortunately is not supported by getStaticProps. We would be looking for a workaround for that.


Fix wide matrices

The case where the matrix is wider than it is tall wouldn't detect any
words which started outside of the square part of the matrix.
The fix is to invert the loop and make sure to use the length of the
current row as the width.

Coverage remained the same at 100.0% when pulling 085c80ebfe4aa05b7785ace2bf0452070be7e5a4 on kfarnung:patch into e678f5de0a749afa1af1fc598f46634de78256a8 on JHotterbeekx:master.


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3722

Questions ?
Please refer to the Backport tool documentation


Data feeder chunck binning.

Due to the high-rate data flow, the data-feeder should be able to merge a chunk of data before it is fed into the data-reduction.

Here is the plot that will determine the number of pulses we need to merge before binning into pixels.
This is only the cost of binning into pixels. When loading a DREAM file this is about 60% of the total cost (for many events).

@SimonHeybrock

See also scipp/scipp#3130

Demonstrated with testing added via #94.
Should decide how it will be included as a feature of the package.

It's done by #184


OCR0044 Glyph Baseline Marking AI Model

Description:
This Card explains the unet model we have made to mark glyph baseline in detail.
Background:
Currently our OCR team is experimenting with scripture font creation. For this we are running google OCR (OCR0021) on tibetan publication images like derge,pecing,etc. We get an image of the target character along with extra letters on the side.
We give annotators these images to:
crop out the target glyph
draw the baseline.

Since this Job follows a specific pattern of drawing a box on the baseline, ai models can perform well for such problems even on very less data.
Diagram:
Below is the diagram explaining the process in the simplest form. It shows us all the processes involved- dataset creation, training, and inference.

Model:
We use a simple conditional Unet model for this task.
Model has 31 million parameters.
Case Study:
For demonstration we trained this model on Derge dataset.
We have a total of  23,000 Data of derge glyphs that have their baseline marked by annotators.
Training on:
3k image dataset only
50 epochs (10 minutes)
we achieve almost 100% accuracy on visual inspection of models predictions on testing data.
example outputs:

Potential:
From now on the work of glyph baseline marking can be easily automated to save company's resources by using this highly accurate and fast ai model.
New pipeline will look like:

Collect 1-2k data of annotators work
train the model on this data
Use this model to mark glyphs baseline for the rest of the dataset

For the case of derge dataset, we have the baseline as vertices of the rectangle box representing the baseline.
if data is in such form we may have to do some extra steps since model works with just images not vertices info:

use the vertices info to make a red colour rectangle over the glyph image.
then if we wanna train or run the ai model, we do it all on images.
On inference we get our output also as an image with red region representing the baseline.
then to get the info as vertices in our json, we can run a script to approximate the red region to a rectangle and get its vertices coordinates.


How to: Start week from Monday

Hello, dear friend!
Big thanks you for your pod file and for your work. It's very helpful!
It will be great if you help me with few questions:

Is is possible to start the week from Monday?
Is it possible to view the past weeks?
How can I add no scrollable view to the top left corner (upper than TimeRowHeader)?

Hi,

For the week to start on monday, it is really easy but you need to modify the CalendarLayout file, there is a 0 and 6 to know for saturday/sunday, you can easily swap it,
In fact, I believe there is a function that tells you wich is the starting day of the locale that would be perfecte, feel free to PR it if you can fix it

Yeah, it should be pretty easy, in weekview there is a function GroupEventsByDay that adds all the days to show, you can just extend it in your own weekview calling super and then just adding the dates before.

If you want something that wont ever move, just add a subvier on top of the weekview

Can you please supply more detail here. I followed the steps and I am still having issues with step 1. Where is the function that returns starting day?

Hi

Is is possible to start the week from Monday?

yes Just change in  @implementation MSWeekView Method -(void)groupEventsBySection{

NSDate* date = [NSDate parse:NSDate.today.toDateTimeString timezone:@"device"];
to replace with
NSDate* date = [NSDate parse:NSDate.weekStart.toDateTimeString timezone:@"device"];


Typeahead fixes and enhancements

General fixes and enhancements:

Added pass-through of alwaysFloatLabel property to input container.
Tweaked CSS to ensure the input text doesn't overflow the dropdown arrow.
Added optional clear button.
Tweaked logic for scrolling the dropdown during keyboard navigation.

Typeahead (and combobox) mode fixes and enhancements:

Disabled browser autocomplete (otherwise the native browser autocomplete overlaps this component's dropdown).
Fixed a bug where clicking on a dropdown option was ignored if the input had focus.

The blur handler was firing (clearing the result) before the tap handler.
Changed this to use a down handler, since that has higher priority.
Replaced component-level tap handling via updateSelected() with item level tap handling via _selectItem().

Consolidated common logic in setValue(), setValueFromLabel() and clearSelection() to avoid code duplication and ensure consistent UI state changes for all combo types.
Added _findFilteredOptionLabelIndex and _findFilteredOptionValueIndex to allow the correct match to be highlighted (and items to be unhighlighted) while typing in the typeahead.
Changed selector selected state to be lazily initialized/updated when the dropdown is opened and when the input is changed.
Added "No matches" item (with configurable text) for typeaheads.
Tweaked keyboard navigation - only commits value on Enter or when dropdown is closed.

Previously silently updated value during interaction.
Now updates label and applies to value (and notifies via event) when interaction is complete.
This helped resolve typeahead issues with sending duplicate notifications (or failing to send them).

I'm seeing a few issues as I play with the demos.

The dropdown is covering up the input box for me.

The combobox is no longer sending events when applying an input value that is not in the list, unless it is immediately following a list selection.  Example:  click the combobox and select "Steve".  The event fires. Then, remove the last "e" and hit enter.  An event goes out for "Stev".  Remove the "v" and see that no event is fired.

On the typeahead, select "Mary".  Tab to the combobox.  Now, shift-tab back to the typeahead.  Press the right arrow to move to the end.  Backspace over the "y".  The list shows all items when it should only show "Mary", since the text in the input is "Mar".

Good catch! I just pushed fixes for the second and third issues.
The first issue is related to iron-dropdown which changed how it positions menus since the 1.0.6 release we're both using in our shared internal projects. (The bower.json in this project references ^1.0.6, so it pulls in a newer version.)
I'll investigate if there's a positioning tweak I can find that works for both versions of iron-dropdown.

Gross - you can see paper-dropdown-menu currently exhibits the same overlapping menu behavior when using the newer iron-dropdown version.
https://elements.polymer-project.org/elements/paper-dropdown-menu?view=demo:demo/index.html&active=paper-dropdown-menu

So, when using newer releases of iron-dropdown, we can position the menu below the input by adding the following to _openDropdown():
this.$.dropdown.verticalOffset = 60;

However, with the 1.0.6 release referenced by your ambrosia library, this ends up pushing the dropdown an extra 60 pixels too far down.
Pondering a DOM measurement check to detect whether to apply a vertical offset.

DOM measurement as a way of detecting whether the menu is incorrectly positioned does not seem viable because the menu is not positioned during the open() call but asynchronously at some unknown future turn of the event loop. (I tried this.async(..., 1) and measurements were not available, this.async(..., 100) and measurements were available.)

When the iron-overlay-opened event is fired, the coordinates have been set. Added a handler for that event that performs DOM measurement to determine if the menu position needs to be adjusted.
Works for the version of iron-dropdown we're using in ambrosia and in the latest version.

Approved!


Bad performance in DictionaryMatcher

The current logic in DictionaryMatcher.cs is quite slow. I'd like to propose changing it to use a proper algorithm searching for a finite set of patterns. In particular I'd like to suggest the Commentz-Walter, which is the multi-pattern equivalent of the Boyer-Moore and thus extremely fast.
Thank you for considering this enhancement. If you want: I am willing to send a merge request implementing it.

A simpler improvement which limits j - i to the longest word in the dictionary should already improve performance for long passwords (going from O(n^2) to O(n), albeit with a dictionary dependent constant).


add extra color to theme

Closed this issue since I created my own theme with (extra colors)
Many thanks to @raunofreiberg for code inspiration and the youtube tutorial.
Check it out ➝ https://marketplace.visualstudio.com/items?itemName=NERO.rave-and-roses&ssr=false#review-details


Add Rector

@samsonasik See what you think of this. Based on the version from the CI4 repo with the "utility" classes and file references removed. Once PHP 7.4 is the minimum we can update some of these rule sets to be more inclusive.


Sidecar injector precedence should be given to pod labels

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Having a revision label on namespace currently overrides revision label given at pod. This is not the expectation when we think about granular control.
This is problematic while doing granular canary upgrades and migration. Migrating a whole namespace is always not ideal. The expected precedence is not documented currently so I believe this issue to be an ask.
For example:
Namespace ns1 has a label istio.io/rev=1-19 and within ns there is a details deployment which has label istio.io/rev=1-17. In this case the revision of namespace ns1 is always used.
This is due to current MutatingWebhookConfiguration which says something like
  namespaceSelector:
    matchExpressions:
    - key: istio.io/rev
      operator: In
      values:
      - 1-19
    - key: istio-injection
      operator: DoesNotExist
  objectSelector:
    matchExpressions:
    - key: sidecar.istio.io/inject
      operator: NotIn
      values:
      - "false"

and

  namespaceSelector:
    matchExpressions:
    - key: istio.io/rev
      operator: DoesNotExist
    - key: istio-injection
      operator: DoesNotExist
  objectSelector:
    matchExpressions:
    - key: sidecar.istio.io/inject
      operator: NotIn
      values:
      - "false"
    - key: istio.io/rev
      operator: In
      values:
      - "1-19"

Suggestion
Ideally this could have been something like (subjected to feedback):
  namespaceSelector:
    matchExpressions:
    - key: istio.io/rev
      operator: In
      values:
      - 1-19
    - key: istio-injection
      operator: DoesNotExist
  objectSelector:
    matchExpressions:
    - key: sidecar.istio.io/inject
      operator: NotIn
      values:
      - "false"
    - key: istio.io/rev
      operator: DoesNotExist

and

  namespaceSelector:
    matchExpressions:
    - key: istio-injection
      operator: DoesNotExist
  objectSelector:
    matchExpressions:
    - key: sidecar.istio.io/inject
      operator: NotIn
      values:
      - "false"
    - key: istio.io/rev
      operator: In
      values:
      - 1-19

Describe alternatives you've considered
Pod labels should be able to overwrite namespace label revision. Currently edited the MutatingWebhook manually.
Question
If this is done deliberately, I wish to understand why and probably:

Why does the namespace label and pod label be need to be considered mutually exclusive?
What is the impact of ENHANCED_RESOURCE_SCOPING if pod labels can be honoured in precedence of namespace labels?

Ps. I believe this is not for security purpose as someone can always add sidecar.istio.io/inject: false to override adding proxy even today.
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ ] Dual Stack
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[x] User Experience
[x] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Additional context
I can draft a PR if this is accepted as a valid requirement.

@Monkeyanator, may you please take a look

@kanurag94, this was discussed recently in the ambient WG meeting actually. Here's a link to the spot in the meeting recording on youtube.
The TLDR though would be that these semantics were established quite a long time ago with backwards compatibility in mind and the namespace label taking precedence is unlikely to be changed.

Thanks @ilrudie for helping with the WG meet notes.
In the example suggestion I have provided in https://github.com/istio/istio/issues/48317#issue-2039058667, we do not use namespace injection when pod has the label, but in all other cases we do. Also while injecting pod, we inject it irrespective of the fact namespace has the revision label or not. Thus no race condition. We can iterate more on it.
During upgrades I understand this could cause race issues when pod start expecting a different revision and earlier MW tries to inject it with something else based on namespace revision label. But I think that's a fair tradeoff for this correction and can be called out in release notes. This would only impact bad configuration (pod having a different revision label) as per current model.
Meanwhile the drawback of not having this is never being able to take advantage of pod level upgrades. What I mean is, not introducing a way to implement this to avoid bad configuration during upgrade seems being too careful.
What do you think? @howardjohn @zhaohuabing @therealmitchconnors

you can still do per-pod revisions if you just leave the namespace unlabeled

The scenario is that we have the namespace on a certain revision, but I wish to upgrade certain workloads to a different revision. I don't wish to move the whole namespace at once.
There can be 50 workloads in the namespace and I wish to test critical upgrade with 5 pods, and later move the whole namespace.

One of the main issue is almost all organisations namespaces are managed by platform team while application developers manages deployments. To test out something or a new version being rolled out, an application team has to go all in/ or all out. This is inconvenient and kind of makes canary upgrade out of reach of app teams but platform dependent.
Maybe if app team had convenience to test out changes before moving to a newer version, it can give more confidence to both the teams involved.


Proxy Documentation Update Request

Sync agent uses WINHTTP_ACCESS_TYPE_AUTOMATIC_PROXY flag as stated in https://docs.microsoft.com/en-us/windows/win32/api/winhttp/nf-winhttp-winhttpopen. Some customers have WPAD  (DHCP option 252) deployed, causing agent unexpectedly use proxy instead of direct/NAT connection. This causing sync issues such as : 0x80090352, because no access to Internet been granted for Sync Agent. There is no way to explicitly disable proxy but using IE settings under SYSTEM account (used by default) which is tricky for customers. Please update this page to reflect this behavior and specify that Sync Agent will use IE proxy settings, including 'Automatically detect settings' under LAN configuration (case<PHONE_NUMBER>01170)

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

ID: e517794d-c237-f052-8445-0280c2ba9ecd
Version Independent ID: 36721ebd-be22-f1e6-882a-83f6d54a1b12
Content: Azure File Sync on-premises firewall and proxy settings
Content Source: articles/storage/files/storage-sync-files-firewall-and-proxy.md
Service: storage
Sub-service: files
GitHub Login: @roygara
Microsoft Alias: rogarana

Thanks for reporting this. I have assigned the issue to the content author to investigate further and update the document as appropriate.

#assign: jeffpatt24

@the-grayest Thanks for reaching out, we appreciate it!
We think this information is best suited to the file sync troubleshooting guide, so we will add it there and update this issue when that is complete.

#reassign @jeffpatt24

I've updated the documentation to cover the different options to configure proxy settings. Doc update will be live tomorrow.

#please-close


Generate package diff during "analyze" step

During a CI/PR pipeline, it would be nice if we could generate a report showing the differences in all generated packages.  This would allow developers to quickly see the customer impact of their change.
Steps required:

Generate the packages for the current PR or commit (all *.tgz files under artifact packages)
Fetch the packages for the last successful build of master
Extract the packages to two separate folders
Diff the folders and write the results to a text file
Upload the text file as an additional artifact

Until this is implemented, here's a small bash script to extract all packages for a local diff:
for file in *.tgz; do mkdir $file.x; tar -zxf $file --directory $file.x; done

#4786


Android: One element labelling another

Requires API Proposal
This issue requires a new API. An API proposal should be added and discussed before proceeding with implementation. The API proposal can be added in the comments of this issue or linked as a separate issue.
Description
No concept of "labelFor" or "labeledBy" or any way to pass a reference to another element as the accessibilityLabel property of an element. Used most when building complex forms.
React Native version:
v0.63
Expected Behavior
Android has a labelFor property on View, which allows you to pass a reference to the ID of another view, that this view should act as the label for. When Focusing on the other view, it will announce  for <other view's label>.
For example, if you have these two Views
<Text labelFor="123">Wifi</Text>
<Checkbox accessibilityLabel="checkbox" id="123" />

And you focus on the checkbox, it would announce "checkbox for Wifi".
Android Details
Any API to add this feature should manipulate the underlying View's labelFor property.  It will need a reference of the ID of the element that is being labelled to properly work.

There is an existing testID property, I wonder if that could be used for this. I'm not sure though as I'm not sure the underlying implementation.

See the comment here on one potential approach for implementation:
https://github.com/facebook/react-native/issues/26739#issuecomment-819149373


Replace hard coded version number of hessian-lite with unified property variable.

Notice, this work should be done on branch 2.6.x.
https://github.com/apache/incubator-dubbo/tree/2.6.x

https://github.com/apache/incubator-dubbo/blob/69311abe3a84996ec6b1a38a381fd39f48bbfdf3/dubbo-serialization/dubbo-serialization-api/pom.xml#L41
https://github.com/apache/incubator-dubbo/blob/69311abe3a84996ec6b1a38a381fd39f48bbfdf3/dubbo-serialization/dubbo-serialization-hessian2/pom.xml#L41


List all Cloud2 URLs

Describe the issue here.
On https://v2.docs.influxdata.com/v2.0/write-data/use-telegraf/manual-config/, we only list the URL for AWS Oregon (https://us-west-2-1.aws.cloud2.influxdata.com). We should also list the URLs for AWS Frankfurt and Google Cloud.

InfluxDB Cloud 2.0

I would suggest not hard-coding URLs in docs but instead explaining where a user might look for the right address to put in their configuration.
My rationale is that today we have only three clusters, but in time we will have dozens across many providers including multiple clusters per provider and even region.
Further, DNS names may change in time. Any change to address would then require a manual documentation update.

Agreed, DRY is good since it means less to update. Someone needs to create that canonical page of InfluxDB Cloud URLs.


NotImplementedError for larger files in read_raw_nihon()

https://github.com/mne-tools/mne-python/blob/3b69e9b936ee01fcb357e732ee35cb75086a6bf8/mne/io/nihon/nihon.py#L214
Trying to read larger Nihon Koden EEG files, I've been hitting the error raised at this line above.
Any bead on when this function might be implemented? If there are no designs I'm happy to try my hand at it if someone's willing to give me a bit of direction

ping @fraimondo

Which one of the two errors was?
This is not yet implemented because I did not have any large files to test. I will need a large file that raises the NotImplementedError, as well as the same large file but in EDF format to compare.

It was the Control Block error (first one of the two)
I have a few anonymized .EEG files I can let you use to test with; they all run into this same error. I'll convert a few into EDF for you.
If you send me an email at<EMAIL_ADDRESS>I can send you a google drive link to the folder

Send them to @.*** @.***>
I’ll have a look.

On 28 May 2021, at 16:42, Phillip Kinney @.***> wrote:
It was the Control Block error (first one of the two)
I have a few anonymized .EEG files I can let you use to test with; they all run into this same error. I'll convert a few into EDF for you.
If you send me an email at @.*** @.***> I can send you a google drive link to the folder
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mne-tools/mne-python/issues/9429#issuecomment-850469035, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCJDA3QAREJTEU2RNNX7JTTP6TXHANCNFSM45U63DVA.

Thanks Fede, the email you sent is censored so I sent the link to the email I found in your github profile. Let me know if you got them

Got them and developed a solution. However, I can't test it yet.  These are the files I got:
87M FA0750X2.EEG
87M FA0750X2_1-1.edf
107M FA0751EA_2013.EEG
32M FA0751EA_2013_1-1.edf
48M FA07548P.EEG
47M FA0755PV.EEG

FA0750X2.EEG has only one datablock, so it worked before the modification
FA0751EA_2013.EEG has two datablocks, but the EDF export is split by datablock and FA0751EA_2013_1-1.edf is the first one (check the size)
FA07548P.EEG and FA0755PV.EEG have two or more datablocks, but I don't have the EDF files.

Love to hear it :heart:
I've dropped in the drive the two .edf datablocks for FA07548P.EEG and the four .edf datablocks for FA0755PV.EEG for your testing pleasure (sorry it took a minute, my partner with the files was too sick to move yesterday)

I have a couple more of questions:
1- Are you still able to access the system to do a couple of recordings for the testing dataset?
2- Why are this recordings split into several datablocks? Is the recording paused in the middle?
3- If 1 and 2 are positive, can you generate a recording that is just a few seconds but with many pauses in between? With this testing data we can integrate it in the MNE testing data and perform automated tests. The electrodes don't need to be connected.

We're not confident about #1, but my partner is heading to the hospital today to see if she can capture some recordings like you described for these automated tests. If you have any ideas you'd like to test, please let me know soon so she may do what she can while she's at the hospital today.
It seems you are right, that pausing is what is separating these recordings into datablocks, but we are not confident just yet. Will see later today
Again if there are any ideas you'd like to test, please let me know soon! We have limited access to the EEG system

She found a .EEG file confirmed with no pauses, and when converting to .edf it only produces one .edf file. That doesn't confirm that pauses are what's separating the datablocks, but it's evidence.

Looks like we might not have access to generate new files for a while. Are you able to use any of the files in the drive for automated testing?

for automated testing we cannot use big files. I guess here if the code is
tested once and written so that it is covered
by our test it's already a clear progress.

I just pushed to #9437
The PR works on the files sent.

Apologies for the delay @fraimondo , thank you so much for your work, this solves our problem beautifully


Cannot find module '@microsoft/sp-property-pane'.

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
(DELETE THIS PARAGRAPH AFTER READING)

Category

[ ] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

For the above list, an empty checkbox is [ ] as in [SPACE]. A checked checkbox is [x] with no space between the brackets. Use the PREVIEW tab at the top right to preview the rendering before submitting your issue.
If you are planning to share a new feature request (enhancement / suggestion), please use SP Dev UserVoice at http://aka.ms/sp-dev-uservoice. (DELETE THIS PARAGRAPH AFTER READING)
(DELETE THIS PARAGRAPH AFTER READING)

Expected or Desired Behavior

If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.
(DELETE THIS PARAGRAPH AFTER READING)

Observed Behavior

If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Steps to Reproduce

If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
(DELETE THIS PARAGRAPH AFTER READING)

Submission Guidelines

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the documents or articles, please ensure that you have clear > reference on the specific file or URL, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, and > #spfx-tooling on (SharePoint StackExchange)[http://sharepoint.stackexchange.com/]. You can also > alternatively submit your question to (SharePoint Developer group)> [https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

(DELETE THIS PARAGRAPH AFTER READING)

Thanks for your contribution! Sharing is caring.

Can you provide more information, like what SharePoint Version and SPFx Version you are using. In SPFx 1.8, the property pane code was moved from  sp-webpart-base to sp-property-pane, however it wasn't handled properly in the on-premises solution.
If you are using on-premises, can you update the reference from sp-property-pane to sp-webpart-base?
Related Issue #3621

We use SP Online only, but I selected 2016/2019/SP Online while creating SPFx webpart to support in future if we may use 2019 on perm for hybrid. If I select "SP Online" only, gulp serve works fine without any issues. Sometimes I get frustrated with these many packages and different versions and we have to update manually every time.

Ya understood. So this onPrem version of the generator is incorrectly handling the change. So you should be able to re-reference to sp-webpart-base for the time being, and it should work.

Thanks Beau. That error gone, but I get below errors.

Correct.  We've got a bug tracking this, and will release a patch for the generator soon.  For now, changing the references back to sp-webpart-base will fix the issue.  I will resolve this as a dupe of the other github bug tracking this.


proposal: crypto/x509: add functions to download certificates from windows update, and retrieve certificates from windows x509stores

SystemCertPool() hasn't been working on Windows (#16736). That issue was fixed once. But since Windows does not ship with all of its root certificates installed, it caused some unexpected behavior (#18609).
The missing certificates can be downloaded from windows update using certutil.exe present on Windows.
There are also other multiple certificate stores on Windows. There are two store locations, and a bunch of stores in each location.
We could retrieve the certificates from windows update and all of the above locations. Then remove the disallowed certificates from that entire collection. That way SystemCertPool() can be fixed.
I have been working on it - CL 127577. While working on it, I created two new functions. One would retrieve the certificates from any location mentioned in the MSDN docs - windows509Store(). And the other would download the certificates from windows update - downloadWUCerts().
I opened this proposal to welcome a discussion regarding whether or not we should export those two new functions to the standard library. The code would be there regardless. Because there are multiple certificate stores on Windows, these functions could provide the end user a bit more control if needed.

@gopherbot, please add label OS-Windows

Should there be an environment variable to disable this? What if the network is down/slow?

Should there be an environment variable to disable this?

Sorry, but I didn't get that. Disable what? If you mean the way Windows downloads the certificates on demand... then no, there isn't a way to disable that.

What if the network is down/slow?

downloadWUCerts() won't work if the network is down.

If the network is slow that means SystemCertPool() could block, for say a minute (or more)? That's not ideal. What about environments without a network?

I agree, blocking would not be ideal.
I think giving a way to enable/disable fetching certs from windows update would increase the complexity a bit. If we rely on an environment variable, the user would have to change it whenever their network status changes. Or whenever they want to enable/disable downloading certs from WU.
Instead, we could just return an error mentioning that certs were not downloaded from WU. I described a way to do it in the CL.

/cc @FiloSottile

/cc @FiloSottile

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

This proposal has been added to the active column of the proposals project
and will now be reviewed at the weekly proposal review meetings.
— rsc for the proposal review group

I had opened this proposal a long time ago. And honestly, I had forgotten about this. For my work, I remember I made a tool to fetch the windows certs whenever needed. It's been a long time since I last worked with Go.

I had opened this proposal a long time ago. And honestly, I had forgotten about this. For my work, I remember I made a tool to fetch the windows certs whenever needed. It's been a long time since I last worked with Go.

This issue is now pretty old, it hasn't gotten much additional enthusiasm that I see, and the original commenter no longer needs the functionality. It sounds like maybe we should decline this specific proposal until we have an ongoing need that we can use to evaluate solutions.

This issue is now pretty old, it hasn't gotten much additional enthusiasm that I see, and the original commenter no longer needs the functionality. It sounds like maybe we should decline this specific proposal until we have an ongoing need that we can use to evaluate solutions.

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

Based on the discussion above, this proposal seems like a likely decline.
— rsc for the proposal review group

No change in consensus, so declined.
— rsc for the proposal review group


Wrong properties on links

check the images, it should be enough to understand what's about
search by name
search by URL

I am going to work on a update per it.

I started working on it, the update will there in 48 hours.

Hello, this issue was fixed in v.6.2.0 Update - please check and reopen the issue if needed.
NPM link: https://www.npmjs.com/package/discord-music-player/v/6.2.0


idmap bugfixes and tests

Jenkins done, just got one unrelated failure. Travis is backlogged and this doesn't affect any non-Linux code, so I think we're good to merge.
@tych0 @brauner
Then I can tag 2.6.


Document FedEx login and test keys

I spent too much of my life figuring out that login keys were not the same between active_shipping and FedEx. I assume I'm not the first to experience this problem. Hopefully this documentation note will help others.

Thanks.
CI fail is a known SSL problem.


Add basejump to community starters

Hello @tiniscule, as per contributing guidelines

Additions should be added to the bottom of the relevant category.

Please move it accordingly. Thnks!

Whoops missed that.  done.

Cool, thanks!


TextEdit scaled selection

Fixes #15245
I ended up using get_viewport_transform().get_scale() *  get_global_transform().get_scale()  to manually scale mouse_position, for some reason the transform's xfrom didn't give the desired result.
It also fixes word selection and line selection, which also appeared with an offset when resized.
And fixes a similar bug that happened when the TextEdit itself had a scale applied.

Good job, but you should squash these commits together.
git rebase -i HEAD~2, change the other commits from "pick" to "fixup", then force push.

Nice! Thanks! I squashed my other pull request too

This fix is pretty overkill, I'm surprised it got so far without being found/fixed properly. The problem is this:
void TextEdit::_update_selection_mode_pointer() {
	Point2 mp = Input::get_singleton()->get_mouse_position() - get_global_position();

Which is broken for the reasons mentioned in this PR.
However, the right way to fix it is simply replacing it by:
Point2 mp = get_local_mouse_pos();

Hello, I have a similar error.
I just added a textedit node to the scene, I scaled it and I changed the size.
I also changed the window to 2d mode.
Then, When I try to select text on the textedit (On windows), that acts very strange.
Here is my project:
https://mega.nz/#!bd03FTrR!FOg4V7FEonIhtceqkXffoCyRnPXeHRPjuzGVxIFTgZg


"flutter run" crashes on Android

Steps to Reproduce

flutter run

Logs
Flutter crash report; please file at https://github.com/flutter/flutter/issues.

## command

flutter run

## exception

ProcessException: ProcessException: Process "C:\Users\Ranjan Binwani\Desktop\hello_rectangle\hello_rectangle\android\gradlew.bat" exited abnormally:

------------------------------------------------------------
Gradle 4.10.2
------------------------------------------------------------

Build time:   2018-09-19 18:10:15 UTC
Revision:     b4d8d5d170bb4ba516e88d7fe5647e2323d791dd

Kotlin DSL:   1.0-rc-6
Kotlin:       1.2.61
Groovy:       2.4.15
Ant:          Apache Ant(TM) version 1.9.11 compiled on March 23 2018
JVM:          1.8.0_202-release (JetBrains s.r.o 25.202-b03)
OS:           Windows 10 10.0 amd64

The system cannot find the path specified.
  Command: C:\Users\Ranjan Binwani\Desktop\hello_rectangle\hello_rectangle\android\gradlew.bat -v

#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:259:7)
<asynchronous suspension>
#1      _initializeGradle (package:flutter_tools/src/android/gradle.dart:300:9)
<asynchronous suspension>
#2      _ensureGradle (package:flutter_tools/src/android/gradle.dart:281:37)
<asynchronous suspension>
#3      _readGradleProject (package:flutter_tools/src/android/gradle.dart:192:31)
<asynchronous suspension>
#4      _gradleAppProject (package:flutter_tools/src/android/gradle.dart:112:37)
<asynchronous suspension>
#5      getGradleAppOut (package:flutter_tools/src/android/gradle.dart:106:29)
<asynchronous suspension>
#6      AndroidApk.fromAndroidProject (package:flutter_tools/src/application_package.dart:164:23)
<asynchronous suspension>
#7      ApplicationPackageFactory.getPackageForPlatform (package:flutter_tools/src/application_package.dart:46:32)
<asynchronous suspension>
#8      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:359:56)
<asynchronous suspension>
#9      HotRunner.run (package:flutter_tools/src/run_hot.dart:254:39)
<asynchronous suspension>
#10     RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:472:37)
<asynchronous suspension>
#11     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:490:18)
#12     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#13     _rootRunUnary (dart:async/zone.dart:1132:38)
#14     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#15     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#17     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#18     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#19     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:30:15)
#20     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:288:13)
#21     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#22     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#23     _rootRunUnary (dart:async/zone.dart:1132:38)
#24     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#25     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#26     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#27     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#28     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#29     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:30:15)
#30     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:288:13)
#31     AndroidDevice.isLocalEmulator (package:flutter_tools/src/android/android_device.dart)
#32     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#33     _rootRunUnary (dart:async/zone.dart:1132:38)
#34     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#35     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#36     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#37     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#38     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#39     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:552:7)
#40     _rootRun (dart:async/zone.dart:1124:13)
#41     _CustomZone.run (dart:async/zone.dart:1021:19)
#42     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#43     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#44     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#45     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#46     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#47     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

flutter doctor
[✓] Flutter (Channel stable, v1.9.1+hotfix.2, on Windows, locale en-US)
    • Flutter version 1.9.1+hotfix.2 at C:\src\flutter
    • Framework revision 2d2a1ffec9 (4 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[✓] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[✓] VS Code (version 1.38.1)
    • VS Code at C:\Users\Ranjan Binwani\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[✓] Connected device (1 available)
    • SM A505F • R58M45A393B • android-arm64 • Android 9 (API 28)

• No issues found!

[√] Flutter (Channel stable, v1.9.1+hotfix.2, on Windows, locale en-US)
    • Flutter version 1.9.1+hotfix.2 at C:\src\flutter
    • Framework revision 2d2a1ffec9 (4 weeks ago), 2019-09-06 18:39:49 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[√] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
    • Android SDK at C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-28, build-tools 28.0.3
    • ANDROID_HOME = C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[√] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[√] VS Code (version 1.38.1)
    • VS Code at C:\Users\Ranjan Binwani\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.4.1

[√] Connected device (1 available)
    • SM A505F • R58M45A393B • android-arm64 • Android 9 (API 28)

• No issues found!

Hi,

have you checked that this file exists  C:\Users\Ranjan Binwani\Desktop\hello_rectangle\hello_rectangle\android\gradlew.bat

run flutter upgrade

Does this happen with a newly created project too?

if not  try creating a new project and copy your Dart files over to the new project.

This file exists
C:\Users\Ranjan Binwani\Desktop\hello_rectangle\hello_rectangle\android\gradlew.bat

Executed  flutter upgrade

    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 40.0.2
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

Facing the same issue even on a newly created project

Is this new? Could you build Flutter projects before?

Just to make a test, could you move the project at some other place on your machine that has no space ' ' in its path? Do a flutter clean and see if it builds there?

This is my first time building a Flutter project and I am facing the same issue even when creating a project in the following location.
D:\flutter_app\flutter_app\

Thanks for testing, so we can eliminate this reason.

On running flutter packages get in D:\flutter_app\flutter_app\
I get the following error:
Running "flutter pub get" in flutter_app...
pub get failed (1; The system cannot find the path specified.)

Can you please post your pubspec.yaml? Di you have a dependency in it that is defined by a path?

Can you please post your pubspec.yaml? Di you have a dependency in it that is defined by a path?

I don't have a dependency that is defined by a path.
Below is the pubspec.yaml file
name: flutter_app
description: A new Flutter application.

version: 1.0.0+1

environment:
  sdk: ">=2.1.0 <3.0.0"

dependencies:
  flutter:
    sdk: flutter

  cupertino_icons: ^0.1.2

dev_dependencies:
  flutter_test:
    sdk: flutter

flutter:
  uses-material-design: true

This is the output of the log file
[  +20 ms] executing: [C:\flutter\] git log -n 1 --pretty=format:%H
[  +61 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 68587a0916366e9512a78df22c44163d041dd5f3
[        ] executing: [C:\flutter\] git describe --match v*.*.* --first-parent --long --tags
[  +44 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[        ] v1.9.1+hotfix.6-0-g68587a091
[   +9 ms] executing: [C:\flutter\] git rev-parse --abbrev-ref --symbolic @{u}
[  +37 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/stable
[        ] executing: [C:\flutter\] git ls-remote --get-url origin
[  +32 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +76 ms] executing: [C:\flutter\] git rev-parse --abbrev-ref HEAD
[  +37 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[  +36 ms] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[  +71 ms] Running "flutter pub get" in flutter_app...
[   +2 ms] Using C:\flutter\.pub-cache for the pub cache.
[   +1 ms] executing: [D:\flutter_app\flutter_app\] C:\flutter\bin\cache\dart-sdk\bin\pub.bat --verbose get --no-precompile
[  +59 ms] MSG : Resolving dependencies...
[        ] MSG : Got dependencies!
[ +144 ms] Running "flutter pub get" in flutter_app... (completed in 0.7s)
[   +2 ms] "flutter get" took 843ms.

But there is no error in there.

Yeah, there seems to be no error in this, but still can't manage to get flutter run to work...

I upgraded the Gradle and I am still facing the same issue.
This is the current log output after running flutter run
Flutter crash report; please file at https://github.com/flutter/flutter/issues.

## command

flutter run

## exception

ProcessException: ProcessException: Process "D:\flutter_app\flutter_app\android\gradlew.bat" exited abnormally:

------------------------------------------------------------
Gradle 5.4.1
------------------------------------------------------------

Build time:   2019-04-26 08:14:42 UTC
Revision:     261d171646b36a6a28d5a19a69676cd098a4c19d

Kotlin:       1.3.21
Groovy:       2.5.4
Ant:          Apache Ant(TM) version 1.9.13 compiled on July 10 2018
JVM:          1.8.0_202-release (JetBrains s.r.o 25.202-b03)
OS:           Windows 10 10.0 amd64

The system cannot find the path specified.
  Command: D:\flutter_app\flutter_app\android\gradlew.bat -v

#0      runCheckedAsync (package:flutter_tools/src/base/process.dart:259:7)
<asynchronous suspension>
#1      _initializeGradle (package:flutter_tools/src/android/gradle.dart:300:9)
<asynchronous suspension>
#2      _ensureGradle (package:flutter_tools/src/android/gradle.dart:281:37)
<asynchronous suspension>
#3      _readGradleProject (package:flutter_tools/src/android/gradle.dart:192:31)
<asynchronous suspension>
#4      _gradleAppProject (package:flutter_tools/src/android/gradle.dart:112:37)
<asynchronous suspension>
#5      getGradleAppOut (package:flutter_tools/src/android/gradle.dart:106:29)
<asynchronous suspension>
#6      AndroidApk.fromAndroidProject (package:flutter_tools/src/application_package.dart:164:23)
<asynchronous suspension>
#7      ApplicationPackageFactory.getPackageForPlatform (package:flutter_tools/src/application_package.dart:46:32)
<asynchronous suspension>
#8      FlutterDevice.runHot (package:flutter_tools/src/resident_runner.dart:359:56)
<asynchronous suspension>
#9      HotRunner.run (package:flutter_tools/src/run_hot.dart:254:39)
<asynchronous suspension>
#10     RunCommand.runCommand (package:flutter_tools/src/commands/run.dart:472:37)
<asynchronous suspension>
#11     FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:490:18)
#12     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#13     _rootRunUnary (dart:async/zone.dart:1132:38)
#14     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#15     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#16     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#17     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#18     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#19     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:30:15)
#20     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:288:13)
#21     RunCommand.usageValues (package:flutter_tools/src/commands/run.dart)
#22     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#23     _rootRunUnary (dart:async/zone.dart:1132:38)
#24     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#25     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#26     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#27     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#28     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#29     _AsyncAwaitCompleter.complete (dart:async-patch/async_patch.dart:30:15)
#30     _completeOnAsyncReturn (dart:async-patch/async_patch.dart:288:13)
#31     AndroidDevice.isLocalEmulator (package:flutter_tools/src/android/android_device.dart)
#32     _asyncThenWrapperHelper.<anonymous closure> (dart:async-patch/async_patch.dart:71:64)
#33     _rootRunUnary (dart:async/zone.dart:1132:38)
#34     _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#35     _FutureListener.handleValue (dart:async/future_impl.dart:137:18)
#36     Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:678:45)
#37     Future._propagateToListeners (dart:async/future_impl.dart:707:32)
#38     Future._completeWithValue (dart:async/future_impl.dart:522:5)
#39     Future._asyncComplete.<anonymous closure> (dart:async/future_impl.dart:552:7)
#40     _rootRun (dart:async/zone.dart:1124:13)
#41     _CustomZone.run (dart:async/zone.dart:1021:19)
#42     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#43     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#44     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#45     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#46     _runPendingImmediateCallback (dart:isolate-patch/isolate_patch.dart:116:13)
#47     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:173:5)

## flutter doctor

[✓] Flutter (Channel stable, v1.9.1+hotfix.6, on Windows, locale en-US)
    • Flutter version 1.9.1+hotfix.6 at C:\flutter
    • Framework revision 68587a0916 (7 weeks ago), 2019-09-13 19:46:58 -0700
    • Engine revision b863200c37
    • Dart version 2.5.0

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = C:\Users\Ranjan Binwani\AppData\Local\Android\Sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)
    • All Android licenses accepted.

[✓] Android Studio (version 3.5)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 40.2.2
    • Dart plugin version 191.8593
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b03)

[✓] VS Code (version 1.39.2)
    • VS Code at C:\Users\Ranjan Binwani\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.5.1

[✓] Connected device (1 available)
    • SM A505F • R58M45A393B • android-arm64 • Android 9 (API 28)

• No issues found!

I'm facing something similar.
`[   +6 ms] "flutter run" took 63,381ms.


Updated Swagger API documentation file

@boydjohnson I have broken up the original commit into two more detailed commits. Thank you for pointing that out.


Add loading until repo name loads

What type of PR is this? (check all applicable)

[x] Refactor
[ ] Feature
[ ] Bug Fix
[ ] Optimization
[ ] Documentation Update

Description
Add loading until repo name loads, I think spinner is not suitable here!
Mobile & Desktop Screenshots/Recordings (if there are UI changes)

Added tests?

[ ] yes
[x] no, because they aren't needed
[ ] no, because I need help

Added to documentation?

[ ] readme
[x] no documentation needed

Thanks a ton for catching these small things.

It's my pleasure @bdougie :pray:


Fix punctuation in README.md

Fixes #.
Description of the changes:

some punctuation in wrong place, and some of them missing

How changes were validated:

Oh come on! put in a little effort if you want to commit to this repo.

@ben174 Hello,, may be you are right. But there are some mistakes right? Compare to open an issue to tell developer, correct them directly will be better I guess.


Don't apply tonemapping for unlit material

This PR skips the LINEARtoSRGB if the material is unlit. This seemed like a simpler approach than changing the LINEARtoSRGB function to be a no-op for unlit materials. @lilleyse let me know if you agree with this approach.
This also fixes the 2nd failing unit test (from https://github.com/AnalyticalGraphicsInc/cesium/issues/8127) by just updating the color.
You can test that this correctly renders an unlit material with:
var viewer = new Cesium.Viewer('cesiumContainer', {
    infoBox : false,
    selectionIndicator : false,
    shadows : true,
    shouldAnimate : true
});

//viewer.scene.highDynamicRange = true;

function createModel(url, height) {
    viewer.entities.removeAll();

    var position = Cesium.Cartesian3.fromDegrees(-123.0744619, 44.0503706, height);
    var heading = Cesium.Math.toRadians(135);
    var pitch = 0;
    var roll = 0;
    var hpr = new Cesium.HeadingPitchRoll(heading, pitch, roll);
    var orientation = Cesium.Transforms.headingPitchRollQuaternion(position, hpr);

    var entity = viewer.entities.add({
        name : url,
        position : position,
        orientation : orientation,
        model : {
            uri : url,
            minimumPixelSize : 128,
            maximumScale : 20000
        }
    });
    viewer.trackedEntity = entity;
}

createModel('../../../Specs/Data/Models/PBR/BoxUnlit/BoxUnlit.gltf', 5000.0);

I used the browser's color picker to verify it is indeed the green base color.

Thanks for the pull request @OmarShehata!

:heavy_check_mark: Signed CLA found.
:grey_question: CHANGES.md was not updated.

If this change updates the public API in any way, please add a bullet point to CHANGES.md.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.
[ ] Works (or fails gracefully) in IE11.

@lilleyse please review/merge ASAP so we can get the release out.

The linear to SRGB conversion still needs to be applied. Only the tonemapping portion should be skipped if unlit.
This branch (with hdr off by default)

1.60 with HDR off

I'm working on a fix...

Hm just realized that this tileset is not using the unlit extension. I'll have to open a separate issue for that.

My changes are up. I'll open a separate PR for AGI HQ updates.


Enable http2 Support

Mapbox does not currently support http2 however, calls to our own backend. In particular calls to the mvt endpoint should support http2 in order to improve performance by having requests in parallel.

Quick update. Fastify http2 is still flagged.
I got this to work locally but there are a handful of issues.
Most importantly I have no clue how to get http2 in fastify to work in a lambda.
I opened an issue with fastify for this. https://github.com/fastify/fastify/issues/1519


add support for SQL BETWEEN expression

Verified in super commit a985431.
With input data.csv:
CustomerID,Country
1,Germany
8,Spain

Before:
$ super -version
Version: v1.18.0-104-g452f3c61

$ super -c "SELECT * FROM 'data.csv' WHERE CustomerID BETWEEN 5 and 10;"
parse error at line 1, column 43:
SELECT * FROM 'data.csv' WHERE CustomerID BETWEEN 5 and 10;
                                      === ^ ===

Now:
$ super -version
Version: v1.18.0-107-ga9854314

$ super -c "SELECT * FROM 'data.csv' WHERE CustomerID BETWEEN 5 and 10;"
{CustomerID:8.,Country:"Spain"}

Thanks @mccanne!


Fix BuildDate in Windows releases

The BuildDate for Hugo is always Unknown. Is there a reason why you are still using it?

What do you mean by "always"? It is set at build time when we release, and also if you build with mage:
https://github.com/gohugoio/hugo/blob/master/magefile.go#L27

Not on Windows though. I get
Hugo Static Site Generator v0.50/extended windows/amd64 BuildDate: unknown

Doing this locally on PC with mage hugo is working.

I'm confirmed the following:
Does the build date show?
| | Linux | Windows |
| --- | --- |
| Regular Build | yes | yes |
| Extended Build | yes | no |
I think this issue is related to GoReleaser config. The build date is set different for the regular build vs extended build (I don't know why). For the extended build though, Windows is doing something different than Linux and macOS is. I believe that's why this is an issue. I've submitted a PR above to fix it.
I don't have the ability to run the GoReleaser build in a Windows computer right now so I'd need someone to config that my PR fixes this issue.
P.S. Building this with Mage does the correct thing regardless, which is why I think GoReleaser is the issue.

This is now fixed.


Make sure it doesn't install elm or try to use elm-tooling's one

On my tab s9 setup I made sure to use the arm64 version of elm using elm-tooling one. It would be interesting to have this plugin also use it.

just tried to patch the repo to fix and issue a PR but this sadly doesn't work IDK why the message is pretty cryptic.
viniciusataide@Viniciuss-MacBook-Pro todois % pnpm i
Packages: +1 -1
+-
Progress: resolved 158, reused 135, downloaded 0, added 0, done
<EMAIL_ADDRESS>Running postinstall script...
[vite-plugin-elm-watch@1.3.4.patch](https://github.com/user-attachments/files/16239401/vite-plugin-elm-watch%401.3.4.patch)

 ELIFECYCLE  Command<EMAIL_ADDRESS>
this is the pnpm patchfile I tried to implement

new file mode 100644
index<PHONE_NUMBER>000000000000000000000000000000..3ceee7dd78a761c841c68dab03d230f72b6ff6ac
--- /dev/null
+++ b/elm-tooling.json
@@ -0,0 +1,5 @@
+{
+  "tools": {
+    "elm": "0.19.1"
+  }
+}
diff --git a/package.json b/package.json
index 769ed53685d66ae7f04713608fa4c807b4083c94..d64559c87b9651c7f7534c252f6ea1a0cc8699a0 100644
--- a/package.json
+++ b/package.json
@@ -3,6 +3,10 @@
   "description": "Use Vite and Elm with reliable HMR and full-color error messages",
   "version": "1.3.4",
   "main": "src/index.js",
+  "private": true,
+  "scripts": {
+    "postinstall": "elm-tooling install"
+  },
   "directories": {
     "example": "example"
   },
@@ -25,7 +29,7 @@
   "homepage": "https://github.com/ryannhg/vite-plugin-elm-watch#readme",
   "dependencies": {
     "cross-spawn": "7.0.3",
-    "elm": "0.19.1-6",
+    "elm-tooling": "1.15.1",
     "launch-editor": "2.6.1",
     "terser": "5.26.0",
     "tiny-decoders": "7.0.1"

Hi!
You can achieve this by overriding the elm package with something that is already installed.  The plugin should still find your own elm installation.

lovely solution but I don't understand how I can change where to get the elm executable instead of changing its version.

it's still trying to install elm from elm and not elm-tooling that provides the 64bit arm

I had to remove my package-lock.json and have a new one created for it to work, forgot to write that.

I'll try that. Looks like I cant move from pnpm to npm


fix -h and --help discrepancies

Just a tiny commit that addresses the -h and --help differences mentioned in https://github.com/uutils/coreutils/issues/3370 - it seems like there are differences when long help is printed, which either occurred when print_long_help() was called or --help was used on a command that had long_help() description in one of its argument.
I removed all calls to long_help since it appears that other commands output their --help to be the same as -h (argument and description start on the same line). So now the changed commands (like false)  look like
...
OPTIONS:
        --help       Print help information
        --version    Print version information

like other commands, as opposed to
...
OPTIONS:
        --help
            Print help information

        --version
            Print version information

I also moved sort -k's long help to the after help section, since that seemed to be to be similar to how GNU does it while also making sort's --help conform with how the other commands' --help are formatted.

Excellent, it probably removes the need of
https://github.com/uutils/coreutils/blob/main/util/build-gnu.sh#L187=
Do you want to give a try? (just remove this line and the one above)
thanks

I removed the two lines - though it seems like the change has no effect on the output of the test. I did make changes to cp.rs because it appeared that it was outputting its help not because of long_help, but because it was printing to stderr.

Thanks :)


autoload.php missing?

Fatal error: Uncaught Error: Failed opening required '/code/vendor/autoload.php' (include_path='.:/usr/local/lib/php') in /code/miner.php:2
doesnt seem to exist in downloaded zip file or in any directory

@modfiles hey sorry, I didn't add the composer steps in the readme I guess.
If you are running it via docker you can run
$ docker run --rm --interactive --tty \
  --volume $PWD:/app \
  composer install

then
$ docker-compose up

and you should have your miner working.
Otherwise if you are running with PHP 8 locally, you should install composer (https://getcomposer.org/download/) and then run
composer install


Don't empty _session_cache during create on SessionStore

Starting Django 1.7 the create() method of db/SessionStore removes this line.
I noticed an issue when using the SessionStore manually with Django 1.9 and django-user-sessions:
ipdb>session = import_module(settings.SESSION_ENGINE).SessionStore(some_session_key)
ipdb>session['foo'] = 'bar'
ipdb>print session.get('foo')
bar
ipdb> session.create()
ipdb> print session.get('foo')
None

I'm not 100% sure how it'll impact Django 1.6 and older but considering the support was dropped in 2014, maybe it won't impact so many people.

Coverage decreased (-0.03%) to 89.931% when pulling ae5080bbaa6508af40d06399b21b322b9525f7f9 on Remiz:patch-1 into 8b7a336c7ba13db2d58b41cf63ee3d6e181d4ad0 on Bouke:master.

This is the upstream commit where this line was removed on the master branch. And it was also backported in this commit to version 1.8. Seeing that this change was made to the versions that this package currently supports (1.8 and up), this change can be merged. However as there are quite a few upstream changes in those commits, it is prudent to review those changes and see if/how they should be merged.


Implement metaclass based hierarchical drivers

This is a cleaned up and tested implementation of my proposal for hierarchical drivers. I tried to trim Eapii heritage but however everything related to caching and the actual implementation of the Features and how they hook up into the drivers are not definitive and still carry the weight of the heritage. This is rather to have a starting point but I chose to do it that way so that we can start testing.
As I don't have Python 3 running I tested only on Python 2 (I expect to get a new computer soon and will then set up a dual version system).

@hgrecco comments addressed.

Is everybody happy with that ? @LabPy/owners
Any comments ? @p3trus, @crazyfermions

Just a short comment: Maybe it's a good idea to hide BaseDriver methods e.g. with a single underscore to avoid name clashes with Features. E.g. some devices support a initialize commands which would shadow the one of the baseclass.

True @p3trus. However as we expect users to open the connection themselves (@hgrecco was against the opening of the connection on driver _init_) they need to call it so we should rather rename it perhaps. @hgrecco you were the one in favor of having initialize rather than open_connection, should we change back (or find something else).

I think we should have an initialize and finalize method which can overridden by derived classes. Is the responsability of derived classes to call the parents class initialize and finalize.

I can live with that, my only concern is that in some cases the user might want to open the connection but not call the initialize method of the instrument which set it in a kind on default mode (but in that case it should not be called initialized in the first place). So I say we can keep it.

This is good to merge from my end. I think it is a good idea to have the reviewers rather than the proposer of a PR to merge so if any of you feels like pushing the green button he is welcome.


⚠ Bofrost.es has degraded performance

In a347414, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1493 ms

Resolved: Bofrost.es performance has improved in e08f6cf after 9 minutes.


make IPC path configurable

Hi!
On my gentoo with discord-bin package discord IPC path is /tmp/discord-ipc-0.
I made this path configurable via emacs' customize interface.
Here is example from my config with use-package:
(use-package discord-emacs
  :ensure nil
  :quelpa
  (discord-emacs :repo "pkulev/discord-emacs.el"
                 :branch "make-ipc-path-configurable"
                 :fetcher github :upgrade t)
  :custom
  (discord-emacs-ipc-dir "/tmp")
  :config
  (discord-emacs-run "384815451978334208"))

And now we have nice customize-group section

Signed-off-by: Pavel Kulyov<EMAIL_ADDRESS>

Looks good, thanks!


[Azure.Monitor.Query] Add QueryResource

Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.

Updated to stable Swagger, still need to add Samples and CodeSnippets

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Monitor.Query

https://github.com/Azure/azure-sdk-for-net/issues/26638

/check-enforcer override


thanks

Thank you for your contribution to the dog coin ecology

Thank you for your contribution to the dog coin ecology

Thanks ! :)


I'm getting strange bugs with custom config

If I starting app with custom config each time I see "Invalid namespace" in the answer.
Also I can't update local pm2-gui.json (it loads it but writes to ~/.pm2/pm2-gui.json).
Bug repeated with both: 0.10@latest and iojs-1.0.4
Also I'm using installed pm2-gui to node_modules and running ./node_modules/.bin/pm2-gui start --config ./pm2-gui.json (Actually a copy of .file in /.pm2/: cp ~/.pm2/pm2-gui.json ./).
What does that mean? How to fix that?

So, are you playing with io.js? I've never tested on that, it's for node.js.

@Tjatse It's working fine. Troubles only when I specify custom configuration ;-)


bugfix: return error if scale ip already in cluster

Describe what this PR does / why we need it

bugfix: return error if scale ip already in cluster
optimize the delete prompt

Does this pull request fix one issue?

Describe how you did it
Describe how to verify it
Special notes for reviews

Codecov Report
Patch coverage has no change and project coverage change: -0.01 :warning:

Comparison is base (52f25f1) 20.42% compared to head (827be6b) 20.41%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2249      +/-   ##
==========================================
- Coverage   20.42%   20.41%   -0.01%
==========================================
  Files          96       96
  Lines        9103     9106       +3
==========================================
  Hits         1859     1859
- Misses       6989     6991       +2
- Partials      255      256       +1

Flag
Coverage Δ

unit-tests
20.41% <0.00%> (-0.01%)
:arrow_down:

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

Impacted Files
Coverage Δ

cmd/sealer/cmd/utils/cluster.go
35.71% <0.00%> (-0.60%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Text Wrapping

Hi;
Is there a way to have node text wrap?  I have scoured documentation, examples, and the API but don't see if it is possible.
Thanks for any assistance!

if you want wrap node text in zTree, please modify the zTreeStyle.css by yourself.
example:

.ztree li{padding:0; margin:0; list-style:none; line-height:14px; text-align:left; white-space:nowrap; outline:0}
height:17px; height:16px;   (you must remove these 'height', or change some value)

And please set the line hide. (setting.view.showLine = false)


create tile list for updated regions

Is it possible to output a list tiles for features that modified by an update? If not, could you suggest a way how this can be implemented?

I'd be also interested

I'd be also interested

Fixed by https://github.com/omniscale/imposm3/pull/127 ?

Yes, the -expiretiles-dir option should cover this.


Bug on adding/removing descriptors

I found a bug using the code in the example.go file.
I think the library is not handling addition/removal of descriptors very well.
My case is:

I start watching one folder like /foo
Some directories inside foo may be removed/moved
These same directories can be recreated
Removed again
Moved
And so on...

I created a python script that simulates random creation of directories:
import random
from subprocess import call
import time

commands = [
    "mkdir watcher/foo",
    "rm -r watcher/foo",
    "mkdir watcher/lol",
    "rm -r watcher/lol"
]

while True:
    for command in commands:
        print(command)
        try:
            call(command, shell=True)
        except Exception:
            pass
    time.sleep(1)

And then I start the example file:
go run example/example.go watcher

I am receiving the panic: descriptor not found right after a directory removal. Here is some log:
2018/12/18 00:16:21 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:21 Watcher watcher event watcher/foo
2018/12/18 00:16:21 Directory created: watcher/foo
2018/12/18 00:16:21 Watch descriptor created and started for watcher/foo
2018/12/18 00:16:21 Event Name: lol Event Path: watcher/lol
2018/12/18 00:16:21 Watcher watcher event watcher/lol
2018/12/18 00:16:21 Directory created: watcher/lol
2018/12/18 00:16:21 Watch descriptor created and started for watcher/lol
2018/12/18 00:16:21 Event Name:  Event Path:
2018/12/18 00:16:21 Watcher watcher event
2018/12/18 00:16:21 Event Name:  Event Path:
2018/12/18 00:16:21 Watcher watcher event
2018/12/18 00:16:21 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:21 Watcher watcher event watcher/foo
2018/12/18 00:16:21 Directory removed: watcher/foo
2018/12/18 00:16:21 Removing descriptor for path watcher/foo
2018/12/18 00:16:21 Watch descriptor removed for watcher/foo
2018/12/18 00:16:24 Event Name:  Event Path:
2018/12/18 00:16:24 Watcher watcher event
2018/12/18 00:16:24 Event Name:  Event Path:
2018/12/18 00:16:24 Watcher watcher event
2018/12/18 00:16:24 Event Name: lol Event Path: watcher/lol
2018/12/18 00:16:24 Watcher watcher event watcher/lol
2018/12/18 00:16:24 Directory removed: watcher/lol
2018/12/18 00:16:24 Removing descriptor for path watcher/lol
2018/12/18 00:16:24 Watch descriptor removed for watcher/lol
2018/12/18 00:16:24 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:24 Watcher watcher event watcher/foo
2018/12/18 00:16:24 Directory created: watcher/foo
2018/12/18 00:16:24 Watch descriptor created and started for watcher/foo
2018/12/18 00:16:27 Event Name:  Event Path:
2018/12/18 00:16:27 Watcher watcher event
2018/12/18 00:16:27 Event Name:  Event Path:
2018/12/18 00:16:27 Watcher watcher event
2018/12/18 00:16:27 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:27 Watcher watcher event watcher/foo
2018/12/18 00:16:27 Directory removed: watcher/foo
2018/12/18 00:16:27 Removing descriptor for path watcher/foo
2018/12/18 00:16:27 Watch descriptor removed for watcher/foo
2018/12/18 00:16:27 Event Name: lol Event Path: watcher/lol
2018/12/18 00:16:27 Watcher watcher event watcher/lol
2018/12/18 00:16:27 Directory created: watcher/lol
2018/12/18 00:16:27 Watch descriptor created and started for watcher/lol
2018/12/18 00:16:30 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:30 Watcher watcher event watcher/foo
2018/12/18 00:16:30 Directory created: watcher/foo
2018/12/18 00:16:30 Watch descriptor created and started for watcher/foo
2018/12/18 00:16:30 Event Name:  Event Path:
2018/12/18 00:16:30 Watcher watcher event
2018/12/18 00:16:30 Event Name:  Event Path:
2018/12/18 00:16:30 Watcher watcher event
2018/12/18 00:16:30 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:30 Watcher watcher event watcher/foo
2018/12/18 00:16:30 Directory removed: watcher/foo
2018/12/18 00:16:30 Removing descriptor for path watcher/foo
2018/12/18 00:16:30 Watch descriptor removed for watcher/foo
2018/12/18 00:16:30 Event Name:  Event Path:
2018/12/18 00:16:30 Watcher watcher event
2018/12/18 00:16:30 Event Name:  Event Path:
2018/12/18 00:16:30 Watcher watcher event
2018/12/18 00:16:30 Event Name: lol Event Path: watcher/lol
2018/12/18 00:16:30 Watcher watcher event watcher/lol
2018/12/18 00:16:30 Directory removed: watcher/lol
2018/12/18 00:16:30 Removing descriptor for path watcher/lol
2018/12/18 00:16:30 Watch descriptor removed for watcher/lol
2018/12/18 00:16:34 Event Name: foo Event Path: watcher/foo
2018/12/18 00:16:34 Watcher watcher event watcher/foo
2018/12/18 00:16:34 Directory created: watcher/foo
2018/12/18 00:16:34 Watch descriptor created and started for watcher/foo
2018/12/18 00:16:34 Event Name:  Event Path:
2018/12/18 00:16:34 Watcher watcher event
2018/12/18 00:16:34 Event Name:  Event Path:
2018/12/18 00:16:34 Watcher watcher event
2018/12/18 00:16:34 Directory removed:
2018/12/18 00:16:34 Removing descriptor for path
panic: descriptor not found

It is strange that some events have empty names and empty paths. Don't you think?
Any ideias on why it is happening?

I guess it is something related to the directory being used. /tmp may have some unwanted effects leading the library to fail handling these events.

Sorry about the insanely late reply. I was swamped with work.
I'm going to write some unit and behavior driven tests, I'll see if I can replicate this behavior and get a test in for it.

I can't seem to reproduce this bug. I'm gonna go ahead and close this for now, until I run into it in the future.


Docker stop does not seem to fall back to sigkill

Output of docker version:
Client:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Wed Apr 27 00:57:43 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.11.1
 API version:  1.23
 Go version:   go1.5.4
 Git commit:   5604cbe
 Built:        Wed Apr 27 00:57:43 2016
 OS/Arch:      linux/amd64

Output of docker info:
Containers: 14
 Running: 14
 Paused: 0
 Stopped: 0
Images: 88
Server Version: 1.11.1
Storage Driver: overlay
 Backing Filesystem: extfs
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null host bridge
Kernel Version: 4.2.5-300.fc23.x86_64
Operating System: Fedora 23 (Twenty Three)
OSType: linux
Architecture: x86_64
CPUs: 32
Total Memory: 141.6 GiB
Name: [OMITTED]
ID: KLBG:YWJX:R5LM:RL6D:2SPA:O77S:TTHF:K4PI:GHOX:6P4Q:I62T:YG7B
Docker Root Dir: /ssd/docker
Debug mode (client): false
Debug mode (server): false
Http Proxy: [OMITTED]
Https Proxy: [OMITTED]
Registry: https://index.docker.io/v1/

Phyisical node.
Steps to reproduce the issue:

Process inside the container falls into state D (uninterruptible sleep).
docker stop the container; hangs indefinitely, even if -t 1 is specified.
Using ps to find the container shim process, you can see it's in Sl state.  kill -9ing this process works and releases the docker stop command.  The container seems to shut down cleanly, as it can be started again without errors.

Describe the results you received:
docker stop hangs and doesn't appear to use SIGKILL, or at least not on all necessary processes.
Describe the results you expected:
docker stop to SIGKILL all related processes after grace period.
Additional information you deem important (e.g. issue happens only occasionally):
Obviously, having processes within the container falling to state D is not nice and is nothing to do with Docker, it merely highlights the stop problem.
Interestingly, I never seem to get to line 47 in daemon/stop.go:
"Failed to send signal %d to the process, force killing"

Is there a way to reproduce this? I.e., how can I get a container to get into that state?

Mine was happening due to NAS problems (the container was moving files about).  I guess you could try to run something like this in a container:
http://unix.stackexchange.com/questions/134888/simulate-an-unkillable-process-in-d-state

You mention it never reached "Failed to send signal %d to the process, force killing"; do the daemon logs show anything useful? Wondering what path it took, that didn't return an error (making it skip that step)

@thaJeztah, I never see them in the CLI, maybe I'm not meant to?
Here are the logs:
level=info msg="Container 8c79844156c37826289bee236fa2de5fc797acef17ec9a7a19d8ac90c0fd2b36 failed to exit within 10 seconds of signal 15 - using the force"

level=info msg="Container 8c79844156c3 failed to exit within 10 seconds of kill - trying direct SIGKILL"

leve=error msg="containerd: get exit status" error="containerd: process has not exited" id=8c79844156c37826289bee236fa2de5fc797acef17ec9a7a19d8ac90c0fd2b36 pid=init systemPid=57603

thanks! ping @mlaventure @tonistiigi any ideas?

Processes in uninterruptable sleep can't be killed. The log output shows the SIGKILL is being sent, but the process isn't responding to it as expected. Killing the parent might cause docker to think the container is exited, but I'm willing to bet that D process is still there. And as such, it's going to be holding a lot of the container stuff open (the namespaces), so killing the parent is not a good ideas as you'll leave all these orphaned resources about.

@phemmer Yep, that's correct, they are still there (and will be until reboot).  I'm wondering if killing the container and leaving the processes (with a warning) is preferable to just hanigng without any indication?

Well, we shouldn't hang, but killing the container and leaving the processes sounds definitely worse, because then there's no indication at all that there's still container processes running.

If this is NFS, you can use mount options such as soft and intr to allow killing your process. In my personal opinion, I would think that should be the recommended solution here. If the process won't exit, I don't think docker should abandon it.


QA "p" releases are not being found

PS C:\Users\bryce> Find-UnitySetupInstaller -Version 2017.3.0p4 -Components Setup
Could not find archives for Unity version 2017.3.0p4
At C:\Users\bryce\OneDrive\Documents\WindowsPowerShell\Modules\UnitySetup\2.1.43\UnitySetup.psm1:204 char:9
+         throw "Could not find archives for Unity version $Version"
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Could not find ...sion 2017.3.0p4:String) [], RuntimeException
    + FullyQualifiedErrorId : Could not find archives for Unity version 2017.3.0p4

It seems to fail on any "p" releases. I got the version numbers from the Patch Releases page. Other release types seem to be working correctly.

It looks to me like this line should be -ne. I changed it in a local copy and it seems to find the version info properly now.
https://github.com/Microsoft/unitysetup.powershell/blob/develop/UnitySetup/UnitySetup.psm1#L199

Thanks! There was another issue in there too. PR is up for moving fix into develop. Next release to master should happen this week.