Create a personal network for each users

Right now, there is the pterodactyl0 network that make containers ables to speak to each others inside the same machine.
The problem is you can reach other users containers that way, and for thoses who rely on the feature that "nobody can be in my localnetwork" (like... every end-user who dosn't setup the panel / aren't really in tech and just want to play).
I think that is a "major" problem, because anybody that have access to the pterodactyl0 network is able to access everybode that have access to this network too.
So, well, this is possible to get per-ser network ? So they can with ease link containers between them, and not passing trought the interwebz.

Switching this to low priority, as we already isolate from the host system to some degree, and properly implementing this would take quite some time to manage and test everything. Its a good idea though.

Closing this due to limited development time and capabilities. This feature would require extensive re-working of daemon code, and a lot of work to manage and test properly across systems.
Additionally, I can see this flooding setups and creating a lot of networking nightmares for those with firewall setups. Additionally, unless you used ICC limiting I believe users would still have access to other containers on the host system.


[Internal] Adds support for Prefix Partition Key searches for sub partitioned containers

Pull Request Template
Description
Support for prefix partition key for sub partitioned containers using ChangeFeed and Query in preview.
Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change.
Type of change
New method FeedRange.CreateFromPartitionKeyAsync accepts arguments container and partition key and determines whether to create a new FeedRangeEpk or FeedRangePartitionKey. Prefix partition key accepts only FeedRangeEpk on the backend for multi-hash containers.
Please delete options that are not relevant.

[] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[] This change requires a documentation update

Closing issues
To automatically close an issue: closes #IssueNumber

/azp run

/azp run

/azp run


Inline import CSS bug

I have inline import .css file  (main-style.css) in my .scss file like
@import "../main-style";
and sass compiles without errors, but watch task throws info about missing file
Could not resolve '../main-style' from file 'C:\patch\to\file\style.scss': file not found, and watch not work on .css files

Thanks for raising it, I'll investigate.

@Sylwekqaz The @import statement will be fixed, but you still won't be able to watch for CSS files changes. Why would you like to watch for CSS files changes actually? Because if it's a file within your codebase, then you should probably convert it into a SCSS file, and if it's a file from an external library, then it should not change, am I wrong?

Theoretically: you are right (maybe its will be use full on changing external library version, but its rare use case)
Practically: sometimes bad things happen in projects, and there no time (money) to fix it.

OK, I'll see what I can do. BTW, in the meantime, you can probably just rename your file from .css to .scss as SCSS is a superset of CSS:

Every valid CSS stylesheet is valid SCSS as well.

Keep you posted soon!

Yes I know, and meantime I use this workaround before I fix problem in our
project.
Thanks
W dniu śr., 26.07.2017 o 15:52 Christophe Maillard<EMAIL_ADDRESS>napisał(a):

OK, I'll see what I can do. BTW, in the meantime, you can probably just
rename your file from .css to .scss as SCSS is a superset of CSS
http://sass-lang.com/documentation/:
Every valid CSS stylesheet is valid SCSS as well.
Keep you posted soon!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/sp00m/gulp-watch-sass/issues/3#issuecomment-318059315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHsdiumPINmf4uaHCumaQsel4rseTwpDks5sR0ShgaJpZM4OhG6V
.

This has been fixed in version 1.2.0 (you can now watch for CSS files modifications in addition to SASS ones).


Question about trimRange non-textual startContainer handling 🤔

In the method trimRange there's a handling for the cases when either the startContainer or the endContainer are not the Node.TEXT_NODE type:
https://github.com/recogito/text-annotator-js/blob/80c2f9235ef27b2e57ff84a23d1f100a7b377e83/packages/text-annotator/src/utils/trimRange.ts#L11-L31
And on line 19 I noticed a small "weirdness". The comments say we're looking for the first text child.
Shouldn't we use range.setStart and not the rangeEnd? Am I missing something?🤔

Good catch. Yes, I think you are correct - that might be an error on my part. Would need testing though... The purpose of the method is to handle edge cases where the selection starts/end on all sorts of non-text (images, layout block element etc.) nodes. Doesn't often happen if the annotated document is mostly text. But happens if markup is more complicated, and users click around frantically (which the usually do ;-) I'll try to test this on some of our own more complex docs.

I think I noticed a few issues with the current trimRange method...

In #65 I noticed that the trimRange cannot be applied to the ranges that were created after spitting on .not-annotatable elements
It creates weird collapsed ranges when the selection ends on an image:

W/o the trimRange - the selector is correct!

And then there's a conceptual question about the role of the trimRange in general 🤔

The purpose of the method is to handle edge cases where the selection starts/ends on all sorts of non-text (images, layout block element etc.) nodes.

So the trim will discard the non-text nodes only if they are at the edges of the selection... but not in the middle... Although in both cases user selected those nodes.
For example, if I select the text + img, the latter will get removed. But if the selection is text + img + text - the image is kept in! Then why was it discarded in the first place 🤷🏻♂
The same can be said for any other non-textual nodes that are trimmed on the edges, but not excluded from the middle of the selection.
I feel like the concept of not annotatable elements and "selection with holes" fits such elements more 🤔

Temporary closing this issue in favor of a more broad one - #66


add better version handling and a helper method for blinding xpubs

One of the main drivers of this was that if you parse a Vpub like Vpub5mvQbnmqKfpPjWfAZEw5Xjdr6UjnjyZEirzrhNMSuKjL8Qfd3nqLBkrBrVXNeMgKCjPXbyLnSCn6qcD8fHQCkNnNLnkpQtY3sh4MHmywvbe, traverse() it and then return the resulting xpub(), you will lose your version byte and display an xpub instead of a Vpub.
One day this won't matter as slip132 will be deprecated, but for now these version bytes are still essential. Software like Specter-Desktop decides on the use of a key (p2wsh) and network (main/test) based on the version byte, and application developers shouldn't need to even know things like that.
Also, there's some general cleanup this enables, like getting rid of the default_version_byte() @proprerty and collapses the _prv/_pub methods.

@jimmysong OK to merge?

lgtm


Owin connect-src not working as expected

I am using the owin middleware like this:
app.UseCsp(config =>
        {
            config
                .DefaultSources(c => c.None())
                .ConnectSources(c => c.Self())
                .FontSources(c => c.Self())
                .ImageSources(c => c.Self())
                .ScriptSources(c => c.Self())
                .StyleSources(c => c.Self());
        });

Still, websocket connections from self are refused:
Refused to connect to 'ws://localhost:41698/signalr/connect?transport=webSockets&clientProtocol=1.5&connectionToken=OPTrgor3M3NOmxP%2B4WF0jpKPRmKaS2zHtIXxSnWHv7EIXGmmBcv8jC1eq1yKUrege55GHJU6uQARKaEbzvFIl%2BmxEC3Oc68qlEei6EEIbkwBcTXVPFPF%2Fng83pv%2F66Wp&connectionData=%5B%7B%22name%22%3A%22systemhub%22%7D%5D&tid=6' because it violates the following Content Security Policy directive: "connect-src 'self'".

I have found no way to configure with CustomSources() to tell him to allow "ws://" + Self.
Thanks.

Yes, it's probably refused because 'self' is same scheme/host/port. Have you tried adding ws://localhost:41698 as a custom source?

This works:
.ConnectSources(c => c.Self().CustomSources("ws://localhost:41698"))

Although it's no good because I have to hardcode the url, if someone else checks out the repository from Git and uses a different port on IIS express it won't work. I couldn't find a way to avoid hardcoding it in Startup.cs though since I don't have context information to get the host name.

Yes, there's currently no way to do that automagically. A workaround for the localhost scenario would be to configure it with a wildcard port like so:
.ConnectSources(c => c.Self().CustomSources("ws://localhost:*"))

I'll leave this issue open and have another look at it for an upcoming release, we can probably handle this particular case in a more elegant manner.

I thank you for your time. If I find a solution I'll be sure to send in a pull request.

This particular scenario will be considered for the next iteration (3) of CSP, it might mandate a change in browser behaviour. I've opened an issue for it: w3c/webappsec#489.
AFAIK they'll get CSP 3 pretty soon, so I'll hold of any changes until I see where the discussion lands.

The CSP issue was just moved to another repo: w3c/webappsec-csp#7

This has been a real pain. Browser Link also causes this problem. I have had to add pre-processor directives and localhost:* to get around this:
filters.Add(
    new CspConnectSrcAttribute()
    {
#if DEBUG
        // Allow Browser Link to work in debug mode only.
        CustomSources = "localhost:*",
#endif
        // Allow all AJAX and Web Sockets calls from the same domain.
         Self = true
    });
filters.Add(
    new CspImgSrcAttribute()
    {
#if DEBUG
        // Allow Browser Link to work in debug mode only.
        CustomSources = "data:",
#endif
        // Allow images from the same domain.
        Self = true
    });
filters.Add(
    new CspScriptSrcAttribute()
    {
        // Allow scripts from the CDN's.
        CustomSources = string.Join(
            " ",
#if DEBUG
            // Allow Browser Link to work in debug mode only.
            "localhost:*",
#endif
            ContentDeliveryNetwork.Google.Domain,
            ContentDeliveryNetwork.Microsoft.Domain),
        // Allow scripts from the same domain.
        Self = true
    });

I've raised this issue on ASP.NET 5 here. Mads Kristensen has been assigned to fix this.

Yes, I see the browserlink pain. Hopefully they'll move the inline stuff to a separate file at least. It is an option to introduce an "allowSameHostWs" configuration option, but that would introduce som unwanted complexity. If the CSP gods decide to not do any adjustments to the semantics of connect-src 'self', I'll definitely add it. If they decide otherwise, well then I'll just wait until the problem disappears by itself. :)
Doing different configs for different environments is a pain when using the attributes. I would suggest using two different configs, either through web.config, or per environment Owin startup classes. That should ease some of the pain.

ASP.NET 5 fixes the multiple environments problem. You no longer need to rely on web.config file transforms, so you could remove the ugly pre-processor directives and do something more civilized like this:
if (environment.IsDevelopment())
{
    // Add CSP filters to get around browser link here.
}

So, they've decided to alter CSP and allow same host/port web sockets when you declare the 'self' source. We'll just wait them out.

@RehanSaeed is the ContentDeliveryNetwork.Microsoft.Domain a construct you created yourself or one you can access from a library? What's the value of it?

They are just constants, sorry I should have made that clear:
    public static class ContentDeliveryNetwork
    {
        public static class Google
        {
            public const string Domain = "ajax.googleapis.com";
            public const string JQueryUrl = "//ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js";
        }

        public static class MaxCdn
        {
            public const string Domain = "maxcdn.bootstrapcdn.com";
            public const string FontAwesomeUrl = "//maxcdn.bootstrapcdn.com/font-awesome/4.2.0/css/font-awesome.min.css";
        }

        public static class Microsoft
        {
            public const string Domain = "ajax.aspnetcdn.com";
            public const string JQueryValidateUrl = "//ajax.aspnetcdn.com/ajax/jquery.validate/1.14.0/jquery.validate.min.js";
            public const string JQueryValidateUnobtrusiveUrl = "//ajax.aspnetcdn.com/ajax/mvc/5.2.3/jquery.validate.unobtrusive.min.js";
            public const string ModernizrUrl = "//ajax.aspnetcdn.com/ajax/modernizr/modernizr-2.8.3.js";
            public const string BootstrapUrl = "//ajax.aspnetcdn.com/ajax/bootstrap/3.3.5/bootstrap.min.js";
        }
    }


how to programmatically select a row in MubTable?

I am editing some table row (in dialog window) by clicking it and then after the edit I would like to persist the row selection.
As I am using Table server after the edit I am doing following steps:
table.ReloadServerData();  //to get new data from the api
table.SetSelectedItem(editedItem); //do make a selection of changed item
selectedItems.Clear();
selectedItems.Add(editedItem); // to save the selection of changed item
Value is changed and visible in the table but the selection is gone.

I think this will help.

Closing since it's been answered.


FIX: Namespacing in documentation was out of date

Namespace fixes in one of the docs

Codecov Report

Merging #668 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #668   +/-   ##
=========================================
  Coverage     46.69%   46.69%
  Complexity     1560     1560
=========================================
  Files           115      115
  Lines          4551     4551
=========================================
  Hits           2125     2125
  Misses         2426     2426

Continue to review full report at Codecov.

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

Thank you.


NEW DOCZ THEME - ZTOPIA

Home page: http://docz-theme-ztopia.surge.sh/
Why Ztopia Theme?
For a long time, Docz has been lonely with just one look, the default theme. Now, let's embrace its new look with the Ztopia theme! Well, creating a new look is just one of many reasons I want to start this project. Below are more reasons why you should switch to the Ztopia theme:

Aesthetically pleasing design: inspired by Ant Design React and Material UI, 40% of my time was devoted to the Ztopia UI&UX when I built this theme. Why? because I believe great UI&UX can attract people reading documentation :)

One theme, two looks: Ztopia theme comes with light and dark mode. You can click the light bulb in top right corner to see the difference. If your component also supports light and dark theme, you can import ThemeConsumer from this library,  and use it to theme your components. See Button.mdx for an example

Gracefully handle complex prop shape: it is common to have component props that accept complex shapes. Inlining them as plain text like this PropTypes.shape({ firstName: PropTypes.string.isRequired, lastName: PropTypes.string.isRequired, ...}) is probably not a best way to show. Fortunately, Ztopia handles complex shapes automatically for you. Whenever you see Shape type in props table, it means you can hover it and see your complex shape. Check out examples in Design page

Description with inline code: when you write descriptions for props, you can embed inline code like this /** This is code */

Playground with built-in iframe support: the default Docz <Playground> is really cool, but it does not serve your components in an isolated browsing context. This makes developers hard to test component responsiveness and lead to possible CSS collisions between Docz default theme style and component style. There is a GitHub issue discussing about it. With the above reasons in mind, Ztopia theme <Playground> is built with <iframe> support in mind, so that you can build your components in isolation. See Button page for an example. (If you are not thinking about switching to Ztopia theme, but still need <iframe> Playground support, check out this plugin I wrote)

Star Ztopia Theme on GitHub
This project is completely open source and free of charge. If you like it, star Ztopia theme on GitHub. Your support is my biggest motivation :)
@pedronauck would it be possible to add this theme to the Docz site theme page here: https://www.docz.site/themes
Thank you all!

Hi @zicodeng!
Great job on the theme, it looks cool :+1:
We would be happy to accept a pull request that adds this theme to the Docz site 🙂

Thanks, will do!

@Glazy PR is here: https://github.com/pedronauck/docz-website/pull/66


panic after shift tab

Ran into the same issue

As a temporary workaround, I set the Shift-Tab keybind in my terminal emulator (Alacritty) to do nothing.
[keyboard]
bindings = [
  { key = "Tab", mods = "Shift", chars = "" }
]

This disables the keybind from crashing mprocs but it does not resolve the issue.

This is fixed in development (unreleased) version - https://github.com/pvolok/mprocs/blob/a20a4726509592dcb59d0108a2249fa0a9555523/src/encode_term.rs#L554
BackTab is handled in all cases from what I can tell.


[Feature]

Is there an existing issue for this?

[X] I have searched the existing issues

Is your feature request related to a problem? Please describe.
Hi,
I'm using filepond with livewire and it works fine when I use it like this:
<x-input.filepondlogos wire:model="{{$key}}"
It can be a single instance or multiple instances.
however when I use the wire:model related to an array as below
<x-input.file posing options wire:model="optionimages.{{$option->id}}"
it takes only the first image.
and if there are more instances it joins the selected image to another instance.
for example if I have:
<x-input.file posing options wire:model="optionimages.{{$option->id}}"
<x-input.filepondproduct wire:model="{{$key}}"
even if the filepond of options is as multiple when saving it saves only 1 image and joins this image to the filepond of product.
and if there are images with the same name I also have problems saving.
Describe the solution you'd like
I would like to be able to use more than one instance on the same page without problems to save.
and be able to relate the instance to 1 position of an array in the livewire,
Describe alternatives you've considered
I need use a filepond multiple intances

Hi Wallace, I honestly don't know, I'd ask this question either on Stack Overflow or on the FilePond livewire repository


Fix input_double

Input double was using input_integer

Great!

Coverage remained the same at 55.776% when pulling ad4068c0ab8df9d599e5b4da44c5cca2e48aa127 on billsix:master into f7e4716a1d06299b8148257e268a514fceb3dfd0 on swistakm:master.

Released as 1.3.0:

release notes
available distributions


enhancement(CommandMenu): Base internal command handling

🔗 Linked issue

❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[x] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

This function now adeptly manages various command selection scenarios, including single and multiple selections, click actions, navigation actions, and external links, eliminating the need for an external onSelect function from the user.
Updated to accept either a single command or an array of commands, ensuring type safety in TypeScript environments.
Refactored conditional logic enhances readability and maintainability. Each selection scenario is clearly defined and separated, simplifying understanding and modification of behavior if necessary.
Allows easy extension and customization, facilitating seamless integration of additional command selection scenarios in the future without altering the component's interface.

📝 Checklist

[ ] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

It's up to the user to choose what to do when a command is selected, here is an example: https://github.com/nuxt/ui/blob/f4a48f6016ede664e4f46741e7811b0dbe0acfbe/docs/components/content/examples/CommandPaletteExampleGroups.vue#L33
This PR causes a breaking change but also removes the possibility to store a selection and handle click actions at the same time.


fix: disable integration tests

Ongoing auth work has updated the token used by this test runner.
These tests no longer work and block updates to the sdk. Their
asserts should be restored in one way or another, but they need
to not block publish.
Also sneaking in a small update to the keepalive ping timer to help
clarify debug log messages on momento-cli.

Looks good but also looks like there's some more tests that need to be disabled?

@rlinehan I think I got them all now...


Add recursive proofs to Credentials ZkProgram

After #2
Enhance the attestation type with a third variant "proof", in which case the zkprogram verifies as proof (using DynamicProof) and check that its vk hash matches a given value

This currently is annoying to do because DynamicProof in o1js is not an instance of Provable, which we use as the base type for input parameters of an attestation.
There are two ways to resolve this:

Update o1js so that Proof is an instance of Provable. This was planned in o1js long ago but never realized: https://github.com/o1-labs/o1js/issues/1039
Change our own input type to be something more generic than Provable

I think we can punt on this for 1-2 weeks and focus on getting the other stuff done.

unblocked as soon as https://github.com/o1-labs/o1js/pull/1851 lands


Add: implementation MF-PPO

MF-PPO algorithm implemented in the paper:
@inproceedings{algumaei2023regularization,
  title={Regularization of the policy updates for stabilizing Mean Field Games},
  author={Algumaei, Talal and Solozabal, Ruben and Alami, Reda and Hacid, Hakim and Debbah, Merouane and Tak{\'a}{\v{c}}, Martin},
  booktitle={Pacific-Asia Conference on Knowledge Discovery and Data Mining},
  pages={361--372},
  year={2023},
  organization={Springer}
}

Thanks for the PR!
A few big things to do first. Basically, your contribution should be more consistent with the rest of the OpenSpiel code.

Google Python style. Please run this through Google's Python lint because the style is inconsistent with other files: (e.g. follow step 9 here: https://github.com/google-deepmind/open_spiel/blob/master/docs/developer_guide.md#adding-a-game). E.g. lines should not exceed 80 characters, there should be two blank lines between highest-level functions etc.  I'll also make a few quick comments on the code.

Please separate the example from the algorithm. The algorithm should be usable as a separate object, and the example contains just the main driver that uses it. Many examples of this decomposition in here. E.g. dqn.py contains the algorithm, and dqn_test.py is the user of the algorithm. You can keep the example where it is now, but the algorithm should be in python/mfg/algorithms.

In the docstring header of the file containing the algorithm, put a full reference to the paper including the URL.

Thanks for the quick response. I added a few more comments. After your next commit, could you go back to the PR and resolve any comments that were done and/or reply to them if necessary? So it's easy to tell which ones are still open.

Dear Lanctot,
Regarding "Can you add the algorithm to the docs (under the MFG algorithms section) in docs/algorithms.md"
I edited the file on commit da977a9 , right now it is in RL but I am not sure if you wanted under a different category e.g., MFG-RL
Let me know what is best : )


Not displaying embedded images in email body.

This Imap library is giving complete output and helping a lot in my projects but 1 issue i'm facing. It's not displaying embedded images in email body.

Hi. Did you tried to set the third param to true?
$this->imap->get_message(25, TRUE, TRUE);

Uhmmm... Hahah. This is not available yet...
But I did it in my local repo. I will update...

Thanks..I'm waiting for your commit. Its really your great effort. 🥇

Ok. Please do your tests. I was not using this as a general library then something can works wrong.
In the next days I will update this library and launch a version.
If anything is strangerous in your tests, report all bugs.

Now is possible to get the embedded images: https://github.com/natanfelles/codeigniter-imap/blob/master/application/libraries/Imap.php#L234

OK..I'm using now this updated file in my current project. I'll update you in couple of days after complete test on live projects.
Thanks

There is Attachment filename issue, I noticed it in old version too...check this attachment array:
[0] => Array
(
[name] => Thu, 07 Sep 2017 14:28:01 GMT
[size] => 12642
[disposition] => INLINE
[reference] => <3ACC6592-BCA7-483B-A8B9-9059E54B2819>
)
                        [1] => Array
                            (
                                [name] => Thu, 07 Sep 2017 14:28:01 GMT
                                [size] => 38044
                                [disposition] => INLINE
                                [reference] => <2BAB5219-EA1C-409A-B35C-F9C6E66A668F>
                            )

                        [2] => Array
                            (
                                [name] => Thu, 07 Sep 2017 14:28:01 GMT
                                [size] => 1225278
                                [disposition] => ATTACHMENT
                                [reference] =><EMAIL_ADDRESS>                            )

                        [3] => Array
                            (
                                [name] => Thu, 07 Sep 2017 14:28:01 GMT
                                [size] => 41214
                                [disposition] => ATTACHMENT
                                [reference] =><EMAIL_ADDRESS>                            )

Still inline images are not visible in body :(

I tested on Outlook and the attachments names was the correct filename.
With this commit the problem with inline images was fixed here.
The IMAP is a little confusing because it has id, uid and msg_number...
Well, test and let me know.

Hi
Sorry I couldn't check latest file that time...But i'm just checking. Its giving error for me. I think you should update your Readme file too with latest instructions. can you plz? :)
Thnaks


FIX: Exclude unneeded routes from prometheus metrics

Pull Request
Related issue
Fixes #3983
What does this PR do?
Excludes the following routes from being observed for prometheus metrics:

/tasks/{task_id}
/manifest.json
Anything starting with /fonts/
Anything starting with /static/
Anything starting with /favicon

Hi @irevoire
Hope you had great vacations!
I can add groups, let me try to update this PR soon and will let you know.
Best regards,
Dario

Hi @irevoire,
sorry got taken away by some stuff so took me longer than anticipated to get back to this:
I added some of your feedback.

search requests (GET and POST grouped together 🤔) by index
I am against grouping GET & POST together because then either GET requests will be labeled as POST or the other way around. This would be confusing for the end user. Would rather keep it separate and also this allows the end user to quickly spot when someone does GET requests instead of POST ones for searching.

multi-search requests by index
I don't understand this task. Do you want me to parse the payload and then create an event/metric for every index that is being searched? Or make a label multi_search_request_{index_one}_{index_two}_{index_three}.

Registering tasks
I didn't understand this either. Which api route are you talking about?

Best regards & hope you had a good weekend,
Dario

I am against grouping GET & POST together because then either GET requests will be labeled as POST or the other way around. This would be confusing for the end user.

Well, maybe I’m wrong, but in my opinion, we shouldn’t even talk about GET or POST requests and simply say we caught a search request that took n ms.
What I was saying is that we should stop exposing raw HTTP routes to the end users because most routes do the same thing in meilisearch. Like the GET and POST search.
But also, like all the routes linked to the API keys.

I was thinking of getting rid of the method entirely and making something like this;
crate::metrics::MEILISEARCH_HTTP_RESPONSE_TIME_SECONDS
                        .with_label_values(&[metric_group])
                        .start_timer(),

Where the metric_group can be something like search, multi_search, static_files, get_tasks, etc

The only issue with this is that you then group routes together that do different things for instance for https://www.meilisearch.com/docs/reference/api/settings#dictionary we have 1 route with 3 different operation and the metric for those should be tracked separately.
Could also just pre-/postfix the routes with their method for some routes and then something like search does not get prefixed so could have as group:

search
multi_search
static_files
get_tasks
get_dictionary
reset_dictionary
update_dictionary

Only issue is that when new routes get added we need to add some mapping otherwise we would fallback to just their plain route or not render anything at all.

we have 1 route with 3 different operation and the metric for those should be tracked separately.

What is the point of having a lot of routes if they're handled exactly the same by the engine? 🤔

Not every route is handled the same by the engine only the POST & GET for search is, right?
But for instance getting, updating & resetting a settings re different operations. One is just a read while others are changing data so these routes perform differently.
If you then do something like median time of requests per minute and you are heavily skewed to the GET metrics (more GET metrics than POST but POST requests are terribly slow) then you won't easily spot it.

Hey, I'm so sorry for the huge delay. I had some issues and then forgot about your PR :sob:

Not every route is handled the same by the engine only the POST & GET for search is, right?

No, most routes are handled exactly the same by the engine actually.
All the routes that generate tasks are pretty much doing the same thing:

Receive a payload
Write it in the DB

The only exception is when you send a document; we have a third step where we write the content of your update in a different file + update its format to something custom.
Then for the rest, most of the time, it's only one simple access to the DB except for the stats, which are computing something a little bit more complex.
And finally, we have two other categories, it's;

When you get documents by filter
When you get tasks with a filter

And everything else should be ignored I think

Hey @Zarathustra2,
Someone found a better approach to fix this issue: https://github.com/meilisearch/meilisearch/pull/4619
Thanks for your efforts, and sorry, I didn’t know we could do it this way 😔


Retry all watch tests once in case of failure.

Watch tests seem to be fundamentally somewhat flaky as we have currently written them.
Recommend reviewing with ?W=1 because most changed lines are just whitespace changes

I'd like our watch tests to just not be flaky, but this seems like a strict improvement


Exception Flask is not installed

How do you use Sentry?
Sentry Saas (sentry.io)
Version
1.5.6
Steps to Reproduce

installed flask 1.1.2 and sentry-sdk 1.5.6 in python3.7 runtime
Run code

from sentry_sdk.integrations.flask import FlaskIntegration

Expected Result
No errors
Actual Result
File "/<my_path>/sentry_sdk/integrations/flask.py", line 41, in <module>
    raise DidNotEnable("Flask is not installed")

exception thrown here
try:
    from flask import (  # type: ignore
        Markup,
        Request,
        Flask,
        _request_ctx_stack,
        _app_ctx_stack,
        __version__ as FLASK_VERSION,
    )
    from flask.signals import (
        before_render_template,
        got_request_exception,
        request_started,
    )
except ImportError:
    raise DidNotEnable("Flask is not installed")

However, if I jump into my python environment, trying to manually replicate the problem, all imports seem fine
Python 3.7.9 (default, Jan 12 2021, 16:13:14)
[GCC 4.8.3 20140911 (Red Hat 4.8.3-9)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from flask import Markup
>>> from flask import Request
>>> from flask import Flask
>>> from flask import _request_ctx_stack
>>> from flask import _app_ctx_stack
>>> from flask import __version__ as FLASK_VERSION
>>> print(FLASK_VERSION)
1.1.2
>>> from flask.signals import before_render_template
>>> from flask.signals import got_request_exception
>>> from flask.signals import request_started
>>>

Seems like it's related to this issue
https://github.com/pallets/itsdangerous/issues/289
This is the full stack trace
ImportError: cannot import name 'json' from 'itsdangerous'
>>> from sentry_sdk.integrations.flask import FlaskIntegration
Traceback (most recent call last):
  File "C:\sentry-sdk-issue-1351\venv\lib\site-packages\sentry_sdk\integrations\flask.py", line 29, in <module>
    from flask import (  # type: ignore
  File "C:\sentry-sdk-issue-1351\venv\lib\site-packages\flask\__init__.py", line 19, in <module>
    from . import json
  File "C:\sentry-sdk-issue-1351\venv\lib\site-packages\flask\json\__init__.py", line 15, in <module>
    from itsdangerous import json as _json
ImportError: cannot import name 'json' from 'itsdangerous' (C:\sentry-sdk-issue-1351\venv\lib\site-packages\itsdangerous\__init__.py)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\sentry-sdk-issue-1351\venv\lib\site-packages\sentry_sdk\integrations\flask.py", line 43, in <module>
    raise DidNotEnable("Flask is not installed")
sentry_sdk.integrations.DidNotEnable: Flask is not installed

Hey @bityob !
Thanks for reporting this. The problem is with Flask not pinning a requirement, and so a wrong version of a package is installed.
Please have a look here for a solution: https://github.com/getsentry/sentry-python/issues/1348


in run_squad_sp.py what is the file format for train_feature_file?

i can't figure out what should I pass to the example in this parameter

You just need to pass the tfrecord file name, ref to
class FeatureWriter  line self._writer = tf.python_io.TFRecordWriter(filename)


Added calendar

This is just the basic layout of the calendar, I tried using blocks but it just doesn't seem to work. Any ideas??
The events need to be fetched from the database in the form of a JSON object.

@vidit23 did you try putting the <script> tags in the scripts block?

Is this ready for a merge? @abhijithanilkumar @vidit23

@rohitvarkey This is ready for merge now. I've added links to the events. Once the Events page and views are complete, this should work fine. :smile:

@rohitvarkey Is it better to have a url field in the model which is populated at the time of object creation?

@abhijithanilkumar You mean like a slug? That'd be nice I think. We won't be giving out ids then. :stuck_out_tongue:

@rohitvarkey We can use the same urls. A post_save handler can be used to populate the url field with the ids.

If we're using ids itself why bother complicating stuff so much?

So, is this fine? Or do you want me to move the URL assignment to the backend code? (In the views itself)

I'm not sure I fully understand what you're talking about here. :stuck_out_tongue: Use your best judgement.

Here, I've written a loop that iterates over the JSON in the html file. Instead, the same iteration can be done suing python in the views. So, if you want me to change it, I'll change. Otherwise, this is ready for merge. :stuck_out_tongue:

Merged! Good work @abhijithanilkumar and @vidit23.
Now please make one more PR for the day/month/year view.


Add school details for Istanbul University (

Description
yok-ogrenci-belgesi-sorgulama.pdf
This pull request adds the school details for Istanbul University. I am a student in the Management Information Systems program, and I have attached my student certificate as requested. I am learning the software to use it in the future. The file has been unzipped and added following the directory structure of the repository. Please review and approve.

@UniX0834 Please fork SWOT repository, copy file.zip in the root folder of your forked SWOT repository. Then unzip file.zip there with no changes to the directory structure. Then make a new pull request. If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message. I have to close this request without merging. Thank you for your understanding.


Require Alter View Query

OS Info:
Hey,
The last update was to work perfectly and really helpful to our team.
If possible please generate a feature as below regarding operation in order to avoid using other software for changes and all.

Create Table

Alter View

Click this button to open create table sql template, I will not enhance the interaction in a short time.
Alter the view is a good feature. I will add it in the next version.

Thanks man :+1: Alter View feature work great...


Add 'deprecated' shims for removed "old" Task APIs to ease migration

APIs that have been renamed or moved re-gained their old APIs but as deprecated:

Task.runDetached -> detached
group.add -> group.spawn
Task.withCancellationHandler -> withTaskCancellationHandler
Task.withGroup -> withTaskGroup / withThrowingTaskGroup

@swift-ci please smoke test and merge


NNI frameworkcontroller example, data path mount

Describe the issue:
https://nni.readthedocs.io/en/stable/TrainingService/FrameworkControllerMode.html
I followed this above example and found that there are no data path in this example.
The example downloads the data through tensorflow example library.
In real world, training neural networks need bunch of data but this example and nni frameworkcontroller have any no description for data utilization.
Is there any way to add data path to nni config in NNI frameworkcontroller?
This is example config for NNI frameworkcontroller
authorName: default
experimentName: example_mnist
trialConcurrency: 1
maxExecDuration: 10h
maxTrialNum: 100
#choice: local, remote, pai, kubeflow, frameworkcontroller
trainingServicePlatform: frameworkcontroller
searchSpacePath: ~/nni/examples/trials/mnist-tfv1/search_space.json
#choice: true, false
useAnnotation: false
tuner:
  #choice: TPE, Random, Anneal, Evolution
  builtinTunerName: TPE
  classArgs:
    #choice: maximize, minimize
    optimize_mode: maximize
assessor:
  builtinAssessorName: Medianstop
  classArgs:
    optimize_mode: maximize
trial:
  codeDir: ~/nni/examples/trials/mnist-tfv1
  taskRoles:
    - name: worker
      taskNum: 1
      command: python3 mnist.py
      gpuNum: 1
      cpuNum: 1
      memoryMB: 8192
      image: msranni/nni:latest
      frameworkAttemptCompletionPolicy:
        minFailedTaskCount: 1
        minSucceededTaskCount: 1
frameworkcontrollerConfig:
  storage: nfs
  nfs:
    server: {your_nfs_server}
    path: {your_nfs_server_exported_path}

Environment:

NNI version: 2.4
Training service (local|remote|pai|aml|etc): frameworkcontroller
Client OS:
Server OS (for remote mode only):
Python version:
PyTorch/TensorFlow version:
Is conda/virtualenv/venv used?:
Is running in Docker?:

Configuration:

Experiment config (remember to remove secrets!):
Search space:

Log message:

nnimanager.log:
dispatcher.log:
nnictl stdout and stderr:

How to reproduce it?:

You could put your data in nfs path, the path will mount into k8s container, you could read the data in the container.
frameworkcontrollerConfig:
  storage: nfs
  nfs:
    server: {your_nfs_server}
    path: {your_nfs_server_exported_path}

I found that nfs shared folder is mount at /tmp/mnt/ in container. It's cool!


Nimfuzzy Documentation Page is Down

By following the documentation link provided in the README, I land on a 404 error page:
http://nimgen.genotrance.com/nimfuzzy
Thanks for wrapping this library! I had already come across it and it's really cool.

Actually it is at http://nimgen.genotrance.com/nimfuzz/
I had to change the name due to a conflict.
I use it in my text editor but that's a nimterop based wrapper.
https://github.com/genotrance/feud/blob/master/wrappers/fuzzy.nim

Thanks for the correct link, and for the text editor example of its usage (good to have).

Fixed the link.


Switch the CI builds to use OSX machines for mac builds and linux machines for linux builds.

Signed-off-by: Dan Lorenc<EMAIL_ADDRESS>

Ref #243


Automatic Upgrade Not Supported with PTA

Please update the limitations to indicate the PTA is an unsupported configuration for Automatic Upgrades. See here: https://docs.microsoft.com/en-us/azure/active-directory/connect/active-directory-aadconnect-feature-automatic-upgrade

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

ID: c535414b-05a2-d4b5-8f15-5c93274d25ef
Version Independent ID: 0bef709f-d10f-5fe6-68a8-f21d0204ecd5
Content: Azure AD Connect: Pass-through Authentication - Current limitations
Content Source: articles/active-directory/connect/active-directory-aadconnect-pass-through-authentication-current-limitations.md
Service: active-directory
GitHub Login: @swkrish
Microsoft Alias: billmath

I am following up with the PM on this.

@jeremyhagan  thank you for submitting this. It is taking me longer then expected to get a response so I have created a work item for this, as this is going to require me to get some verification from the engineering team. At this point I am going to close the issue and I will re-open this once I have an updated. Thanks again!

#please-close

@jeremyhagan We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[8.x] Schema Dump (MariaDB)

Laravel Version: 8.x
PHP Version: 7.4
Database Driver & Version: 10.4.11-MariaDB

Description:
A schema:dump command is added in Laravel 8.x (PR #32275).
The baseDumpCommand uses --set-gtid-purged=OFF in the MySqlSchemaState class.
https://github.com/laravel/framework/blob/c389f0a4a0dfb411bf7f50edcf543a896299f73b/src/Illuminate/Database/Schema/MySqlSchemaState.php#L78-#L81
MariaDB does not have the --set-gtid-purged option which results in a ProcessFailedException.
Error Output:
mysqldump: unknown variable 'set-gtid-purged=OFF'

This should be fixed.

Encountered this issue and just upgraded to 8.0
MySQL 5.7
mysqldump: unknown variable 'set-gtid-purged=OFF'
mysqldump: unknown variable 'set-gtid-purged=OFF'

   Symfony\Component\Process\Exception\ProcessFailedException

  The command "mysqldump --set-gtid-purged=OFF --skip-add-drop-table --skip-add-locks --skip-comments --skip-set-charset --tz-utc --host="${:LARAVEL_LOAD_HOST}" --port="${:LARAVEL_LOAD_PORT}" --user="${:LARAVEL_LOAD_USER}" --password="${:LARAVEL_LOAD_PASSWORD}" "${:LARAVEL_LOAD_DATABASE}" --routines --result-file="${:LARAVEL_LOAD_PATH}" --no-data" failed.

Exit Code: 7(Unknown error)

Working directory: /app

Output:
================

Error Output:
================
mysqldump: unknown variable 'set-gtid-purged=OFF'

  at vendor/symfony/process/Process.php:252
    248▕      */
    249▕     public function mustRun(callable $callback = null, array $env = []): self
    250▕     {
    251▕         if (0 !== $this->run($callback, $env)) {
  ➜ 252▕             throw new ProcessFailedException($this);
    253▕         }
    254▕
    255▕         return $this;
    256▕     }

      +18 vendor frames
  19  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

Same here !
mysqldump: unknown variable 'set-gtid-purged=OFF'
mysqldump: unknown variable 'set-gtid-purged=OFF'

   Symfony\Component\Process\Exception\ProcessFailedException

  The command "mysqldump --set-gtid-purged=OFF --skip-add-drop-table --skip-add-locks --skip-comments --skip-set-charset --tz-utc --host="${:LARAVEL_LOAD_HOST}" --port="${:LARAVEL_LOAD_PORT}" --user="${:LARAVEL_LOAD_USER}" --password="${:LARAVEL_LOAD_PASSWORD}" "${:LARAVEL_LOAD_DATABASE}" --routines --result-file="${:LARAVEL_LOAD_PATH}" --no-data" failed.

Exit Code: 7(Unknown error)

Working directory: /app

Output:
================

Error Output:
================
mysqldump: unknown variable 'set-gtid-purged=OFF'

  at vendor/symfony/process/Process.php:252
    248▕      */
    249▕     public function mustRun(callable $callback = null, array $env = []): self
    250▕     {
    251▕         if (0 !== $this->run($callback, $env)) {
  ➜ 252▕             throw new ProcessFailedException($this);
    253▕         }
    254▕
    255▕         return $this;
    256▕     }

      +18 vendor frames
  19  artisan:37
      Illuminate\Foundation\Console\Kernel::handle(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))

Client config :
root@592ea3c7e066:/app# mysqldump --version
mysqldump  Ver 10.17 Distrib 10.3.23-MariaDB, for debian-linux-gnu (x86_64)

@driesvints can this be reopened?

I am getting:
mysqldump: unknown variable 'column-statistics=0'

Opne a PR to fix what you want.

I have no idea if this works for MariaDB. I only use MySQL.

@iKlsR are you using the latest Laravel version?

@driesvints Yes, latest everything. Fresh project and had just upgraded, php is 7.4, mysql 5.7.31.

@iKlsR can you post the output of mysql --version ?

$ lando mysql --version
mysql  Ver 14.14 Distrib 5.7.29, for Linux (x86_64) using  EditLine wrapper

Ok so poking around a bit, this seems to stem from the mariadb distrib. If I comment out the flag in baseDumpCommand() The schema gets dumped but then I run into #34303 so I guess that might warrant some looking at again as well.
Reading the entire issue listing with this as well as external fixes on this I propose stripping --column-statistics=0 '--set-gtid-purged=OFF if the dump version has mariadb or maybe entirely and allow access to the underlying dump executable to allow advanced? users to pass these. If I remove those works as expected and I don't think they are relevant either, you just want a schema dump.

I don't have any issue at all with MariaDB 10.5.x and latest Laravel 8.

@taylorotwell This is still not fixed sadly, column stats was just an addition, still errs with --set-gtid-purged=OFF. See https://stackoverflow.com/questions/48876519/mysqldump-stopped-working-with-conversion-to-mariadb-set-gtid-purged-off-issue.
Maybe just add a note in the docs, since this is a rare command, I'm fine commenting out that for now each time.


E_UTM, N_UTM, crs, lat, lon, are always defined at the export -> BAD!

See title

E_UTM, N_UTM, lat, lon are not in the official description anymore

CRS has now its own function with E_UTM;N_UTM; lat and lon (testing pending)

this can be closed


Version bump to Squirrel 1.0.0

This PR bumps us to Squirrel 1.0, release notes are here

Published in 1.0.0


Get Params not creating a query string

When you run a GET with data, for instance:
GET google.com
Data: {"q": "search"}
It doesn't put the data in the URL as a query string. That appears to be a limitation of what Angular does. Is there a way to use GET with data and populate the query string?

#10 fixes

Once you remove that one line in the #10 I'll merge. Thanks!

What are the plans to get this fix into a release and out to Bower?  I need this fix to proceed with my project.


add version Id for undelete

Add VersionId for undelete operation per https://review.docs.microsoft.com/en-us/azure/storage/blobs/versioning-overview?branch=pr-en-us-104390&tabs=powershell#blob-versioning-and-soft-delete

/azp run automation - sdk

Can one of the admins verify this patch?

Let's close this pr for now. I just confirmed with service people and they said we cannot undelete a specific version. The undelete behavior should be the same as snapshot undelete.


Annotations

Get valid labels from server? (Start with hard-coded list)
Add ability to annotate timestep with labels
Enable admin to add valid labels?

Provenance? Display that? ML vs human?
Explicit approval vs. implict (didn't remove)
Numerica value for tag (number of people tagging) vs. boolean.

Probably want a context for all annotations in the system, as this will be used for the admin view and the navigation view


version update,

Decided to do version as such.
1X.2X.3X
1x = Will be a major update (like name change)
2x = Will be features
3x = Will be patches

Coverage remained the same at 100.0% when pulling 69a9a5f9da80d0296c216e09e5ed30042cb4222c on chinese-names into 93682e78e8f6d6789177b16ea66e1bf94862c7bd on master.


Relicensing as Apache 2.0

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Travis failures are spurious SDK installation failures unrelated to this change. Merging.


fix: navbar left alignment fixes

What does this PR do?
Fixes https://github.com/ArslanYM/StarterHive/issues/108
Type of change
Bug fix  navbar left alignment fixes

@ArslanYM what do you think let me know ..


Bugfix/synapse client test/compiler errors

Fixed the compiler errors
Changed the folders to packages
Updated the unit test naming conventions

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: cuztomizer:x: pclaudio3You have signed the CLA already but the status is still pending? Let us recheck it.


ValueError: cannot reshape array of size 20 into shape (11,2)

Hello, thank you for providing the code and a good explanation. I was trying to run the code on Ubuntu and have followed all your instructions. I'm using pre-trained weights and trying to test how the code works. But I'm getting value error and I could not find why. Is it due to the tango key points mat file?  Please let me know how I can solve this issue. Thanks

Hi, can you double check if your tango model has the same number of keypoints (11) as expected from the code?


feature request:  add ability to pass multiple arches using CUPY_NVCC_GENERATE_CODE

It would be very helpful to change this line
https://github.com/cupy/cupy/blob/2804af926cead8d2e80665283473a71212e824b0/cupy_setup_build.py#L765
to use list comprehension in order to pass multiple architectures on the command line.  This is useful for building the list of not included architectures from Jetson such as 53,62,and 72.
Something like this with + sign delimited works great
return ['--generate-code={}'.format(val) for val in envcfg.split('+')]
and allows me to build with CUPY_NVCC_GENERATE_CODE="arch=compute_62,code=sm_62+arch=compute_72,code=sm_72" and everything builds great.

I guess for style consistency with CuPy's companioning project Chainer, we better use semicolon as the delimiter, see https://github.com/chainer/chainer/pull/8370.


FancyZones - cycling through zones on two monitors erratic

ℹ Computer information

PowerToys version: 0.31.1
PowerToy Utility: FancyZones
Running PowerToys as Admin: Yes
Windows build number: Version 1909 (OS Build 18363.1350)

📝 Provide detailed reproduction steps (if any)

… Create multiple zones across two monitors with different resolutions
… When focusses on an application, cycle through Zones with 'Win + [right arrow]'
… Sequence of zones is random and erratic.

✔ Expected result
'Win + [right arrow]' should cycle through the screen zones from 1 to x then move to next screen and cyle through zones 1 to x
❌ Actual result
Sequence of zones is random and erratic
📷 Screenshots
N/A

Hey. I have this exact same issue, got it today just from updating to newest version (0.31.1). I got it resolved by uninstalling and then installing the 0.29.3 version. Hope this helps! Let me know if you find another fix which I may be able to try for myself. Thanks!

@rayaddison @Ville907
thank you for reporting the issue.
The problem is tracked by #9453
We will release a hot fix.
Thanks.


Emmet uses the "references" icon

VSCode Version: Code - Insiders 1.13.0-insider (c2a17a7929259d3d2643cb3f98478e994a39605b, 2017-05-30T05:12:15.312Z)
OS Version: Linux x64 4.10.0-21-generic
Extensions:

Extension
Author
Version

EditorConfig
EditorConfig
0.9.3

lorem-ipsum
Tyriar
1.0.0

sort-lines
Tyriar
1.3.0

theme-sapphire
Tyriar
0.2.1

vscode-svgviewer
cssho
1.4.1

tslint
eg2
0.15.0

git-project-manager
felipecaputo
1.3.2

md-navigate
jrieken
0.0.1

vscode-scss
mrmlnc
0.6.2

seti-icons
qinjia
0.1.3

#27316
The icon has semantic meaning already that doesn't match when emmet is trying to do:

https://code.visualstudio.com/docs/editor/intellisense#_types-of-completions

I need an icon that is different from the default "property" icon to differentiation between the suggestion for expanding the abbreviation and the suggestions for autocompleting the abbreviation itself.
Didn't want to introduce a new icon for just emmet.
Could have used the snippet icon, but that means assigning the kind "snippet" which would push the snippets down the list as per editor.snippetSuggestions which had the default value as "bottom" until last release.
Suggestions?
cc @jrieken

which had the default value as "bottom"

No, the default isn't bottom anymore but inline. We made that change because suggestions are now usually sorted by how well they match the text left of the cursor. The "bottom" and "top" settings only come into play when having no text left of the cursor (in which case emmet won't propose anything I believe)

As part of #28556 both type of suggestions are now expanding abbreviations, so there is no need for a different icon anymore


SAK-31978: Images elements not responsive when inserted via CKeditor

Not sure why the last PR got closed. I didn't close it.  Here it is: https://github.com/sakaiproject/sakai/pull/3623

I think I've addressed all the comments in this PR that need addressing.


chore: use modern deploy

:tada: This PR is included in version 2.25.0 :tada:
The release is available on:

v2.25.0
v2.25.0
GitHub release

Your semantic-release bot :package::rocket:


Feature: additional script dokku-run

Allows to execute a single command.
Note: Remove git version specification, build fails.

You can/should instead use the bin/ci-pre-deploy functionality instead.


Qbiasfinal

Fixes #
Changes proposed in this pull request:

Before submitting

[x ] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[x ] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[ x] If this PR fixes a bug, I've added a test that will fail without my fix.
[x ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 70.11070% with 81 lines in your changes missing coverage. Please review.

Project coverage is 34.59%. Comparing base (c6be5bc) to head (ca7fa71).

Files with missing lines
Patch %
Lines

cosmicfishpie/fishermatrix/cosmicfish.py
20.93%
34 Missing :warning:

cosmicfishpie/configs/config.py
75.00%
20 Missing :warning:

cosmicfishpie/LSSsurvey/spectro_obs.py
62.50%
18 Missing :warning:

cosmicfishpie/cosmology/nuisance.py
90.90%
6 Missing :warning:

cosmicfishpie/LSSsurvey/photo_obs.py
93.54%
2 Missing :warning:

cosmicfishpie/utilities/utils.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
- Coverage   34.64%   34.59%   -0.06%
==========================================
  Files          29       29
  Lines        5117     5197      +80
==========================================
+ Hits         1773     1798      +25
- Misses       3344     3399      +55

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

The way the defaults are set up we compute with a spec error of 0.1% but z-independence. Normally we either used z-dependent error of 0.1% or a constant error of 0.2% right?

The way the defaults are set up we compute with a spec error of 0.1% but z-independence. Normally we either used z-dependent error of 0.1% or a constant error of 0.2% right?

I have fixed this now in the new yamls, but left the testing matrix with 0.1%. It is useful to test as well the overwriting of specs through the user.

@Sefa76  can you do a quick review? I did lots of changes, but always testing, so there should be no bugs. However, as soon as you can, please go through the changes and see if you spot something.
Biggest change is sigma_pv nuisance treatment and the fact that cfg is not bothering us anymore with its global state. From here onwards, this can be fully converted into a config class, but let's keep it like this for the moment to allow for (somewhat) backwards compatibility.
Thanks!


"RuntimeError: unreachable executed" when using wasm to dump symbols from this xul.pdb file

Steps to reproduce:

Download and uncompress xul.pdb from https://storage.googleapis.com/profiler-get-symbols-fixtures/xul-noopt-gerald.pdb.gz
cargo install simple-http-server if you haven't already
In this repo, run simple-http-server
Go to http://<IP_ADDRESS>:8000/index.html
"Initiate Symbol Table Query" with the following settings:

"Pick files individually"
debugName: xul.pdb
breakpadId: 914A047EA5B976C74C4C44205044422E1

Click "Query Symbol Table".
Upload xul.pdb when requested.
Wait.

Expected results:
Output should appear.
Actual results:
The UI doesn't update. If you open the Web Console, you see:
Uncaught (in promise) RuntimeError: unreachable executed

This was an out-of-memory problem. I found a way to remove one copy when reading PDB data, saving 1.6GB of memory in this test case. Now the steps in this issue work.


(#785) Expose TableController filters and helpers

Fix for: #785

@microsoft-github-policy-service agree company="IRISNDT Systems"

I'm disagreeing with the scope here, so putting this PR on hold.
Let's finish the discussion in the issue before continuing.


PropTypes and React.createClass deprecation warnings. Also react dependency issues

Description
When running tests with a newly generated react native app, there are warnings about PropTypes being moved to a separate package and React.createClass no longer being supported.
Steps to reproduce
yarn test
Expected Behavior
Expected the tests to pass with no errors or warnings
Observed Behavior
Got the following warnings although the test did pass
  console.error node_modules/fbjs/lib/warning.js:36
    Warning: PropTypes has been moved to a separate package. Accessing React.PropTypes is no longer supported and will be removed completely in React 16. Use the prop-types package on npm instead. (https://fb.me/migrating-from-react-proptypes)

  console.error node_modules/react-native/Libraries/Core/ExceptionsManager.js:71
    Warning: React.createClass is no longer supported. Use a plain JavaScript class instead. If you're not yet ready to migrate, create-react-class is available on npm as a drop-in replacement. (https://fb.me/migrating-from-react-create-class)

Environment
Please run these commands in the project folder and fill in their results:

npm ls react-native-scripts:

<EMAIL_ADDRESS>/mnt/Storage/Coding/AwesomeProject
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

npm ls react-native:

<EMAIL_ADDRESS>/mnt/Storage/Coding/AwesomeProject
├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└──<EMAIL_ADDRESS>
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

npm ls expo:

<EMAIL_ADDRESS>/mnt/Storage/Coding/AwesomeProject
├─┬<EMAIL_ADDRESS>│ └── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>
npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>

node -v: v6.10.3
npm -v: 3.10.10
yarn --version: 0.24.6
watchman version: not installed

Also specify:

Operating system: Ubuntu 16.04
Phone/emulator/simulator & version: Genymotion

Reproducible Demo
Repo: https://github.com/tamj0rd2/react-native-example

The deprecation warnings are a problem with react-native upstream right now -- some parts were not updated to get rid of the deprecation warnings. This will be fixed in the next release, you can ignore those.
The react dependency warnings are a result of how npm uses semver: >= 15.0.0 will not be satisfied by 16.0.0-alpha.12, so it will show you this warning. You can use yarn if you don't want to see the warning. This will be resolved when React 16 is released, not sure what the timeline is on that though.

the actual problem is -> it's not telling what exact packages are resulting this warning :/
so I don't know if those are the new dependencies I've added or the old ones


segmentation the liver and tumor at the same time

Hello, I want to segmentation the liver and tumor at the same time, how to change it, thank you, look forward to your answer

Hi,
when inferencing stage,you can use same vnet model with different trained model params, just load the two model to segment liver and tumor one by one.
thank you


[#162474][Daily Rate] Handle reservations that end at beginning of day

Notes

Fix reservation checks on schedule rules when a reservation ends exactly at midnight (12:00 AM)
Add tests to move a reservation next to unavailable day and next to another reservation when the availability ends at midnight

This change instruduces an error on another edge case which is kind of the oposite of this one: create a reservation starting at beginning of dat after an unavailable day. Closing this an created issue #162494


DSEGOG-95 search by shot number

Description
Allows the user to search by shot number through a popup window containing min and max values, as per the wireframes. I've refactored the functionality behind searching as a whole so that the existing date-time search functionality works in tandem with it. As a result, there are a couple of breaking changes:

search is launched when the user presses the big shiny SEARCH button. This replaces previous behaviour in the date-time pickers which updated the search parameters whenever the user updated their values. Whatever values are currently in the search fields will be used in the new search query and the table won't be updated until the search button is pressed again with new search entries
a need to tie the search parameters in with the existing filters work mean all the search and filter conditions passed to the backend are wrapped in one big $and statement. As far as I can tell, this works as expected but please test this yourself
The wireframes seem to indicate that min and max fields for searching by a shot number range are always present but I may have this wrong. Therefore, as it stands, a min and max parameter for shot number of 0 and 99999999 respectively are included with every query (unless the user changes them). This has complicated queries and unit tests. It may be that we decide to make these optional, in which case I'll have to change the design a bit

Testing instructions
Test searching and filtering on various fields and check the data being displayed matches what you expect.

[ ] Review code
[ ] Check Actions build
[ ] Review changes to test coverage

Agile board tracking
Closes DSEGOG-95

Following a conversation, we have decided to make the min and max fields optional. They can be greyed-out "default" values in the input boxes but not selected unless the user edits them

@louise-davies ready for a further review with the above changes implemented (optional shotnumber range)


T.proc arity checking assumes it's a lambda, not a proc

Sorbet forces a proc variable to have the same amount of arguments as of the expected proc argument with a signature. The documentation states that:

At present, all parameters are assumed to be required positional parameters—T.proc types cannot declare optional nor keyword parameters.

But this is not the behaviour for inline procs. They should be consistent, ideally, allowing for a parameterless proc to be sent as an argument to a method that expects a proc with arguments (since the languages allows for that).
Input
→ View on sorbet.run
# typed: true
extend T::Sig

sig do
  params(
    block: T.proc.params(current_time: Time).void
  ).void
end
def with_current_time(&block)
  block.call(Time.now)
end

# Procs without an argument
with_current_time do
  puts "Hello again!"
end

parameterless_proc = proc do
  puts "Hello again!"
end

# Sorbet prevents me from using the proc, even though
# it's a valid code, and I can also use it if the block is "inline".
with_current_time(&parameterless_proc)

Observed output
editor.rb:35: Expected T.proc.params(arg0: Time).void but found T.proc.returns(T.untyped) for block argument https://srb.help/7002
    35 |with_current_time(&parameterless_proc)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  Expected T.proc.params(arg0: Time).void for block argument block of method Object#with_current_time:
    editor.rb:6:
     6 |    block: T.proc.params(current_time: Time).void
            ^^^^^
  Detailed explanation:
    Proc0 does not derive from Proc1
Errors: 1

Expected behavior
I would expect the that a variable containing a proc would behave the same as having an inline proc.

The underlying issue here is that Sorbet doesn't have separate types for lambdas and procs. Sorbet tries to get around this by sometimes choosing to back T.proc types by behavior from procs and sometimes back them with behavior from lambdas. In this case, Sorbet has chosen the functionality that would be required for checking lambdas correctly:
# typed: true
require_relative './gems/sorbet-runtime/lib/sorbet-runtime'
extend T::Sig

sig do
  params(
    block: T.proc.params(current_time: Time).void
  ).void
end
def with_current_time(&block)
  block.call(Time.now)
end

# Procs without an argument
with_current_time do
  puts "Hello again!"
end

parameterless_proc = proc do
  puts "Hello again!"
end
with_current_time(&parameterless_proc)

parameterless_lambda = lambda do
  puts "Hello again!"
end
with_current_time(&parameterless_lambda)

❯ ruby foo.rb
Hello again!
Hello again!
foo.rb:25:in `block in <main>': wrong number of arguments (given 1, expected 0) (ArgumentError)
        from foo.rb:11:in `with_current_time'
        from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/call_validation.rb:270:in `bind_call'
        from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/call_validation.rb:270:in `validate_call'
        from /Users/jez/stripe/sorbet/gems/sorbet-runtime/lib/types/private/methods/call_validation.rb:191:in `block in create_validator_slow'
        from foo.rb:28:in `<main>'

To drive home the point: Sorbet's inferred type for the two callable variables in the above snippet are the same: T.proc.returns(T.untyped).
→ View on sorbet.run


get user phone number from new incoming message.

Hello, rubenlagus. Thanks for the development of this api. I really appreciate it.
by the way, how do I know the phone number user, when the user sends a message to us. maybe you can give some simple examples. Thanks.

hi @ilhamiarie ,
You should first get message.getFromId() to get the sender user ID. Then in our UserHandler when onUsers() is triggerred, you look for the user with the same ID. You can check if user shares phone number with you with user.hasPhone() and if yes, get it by user.getPhone().
hope it helps


Updated C3 compiler options to show version, allow check/build/compil…

…e to be invoked.
This relies on using the latest version of C3C (checked out from master). I would appreciate information on how to enable the remaining columns.

Thanks


Remove checkboxes from PR template

Not sure about this, but addresses the plethoric checkbox issue. It assumes that you've done everything you should have and requires you to indicate where you haven't. Thoughts?

I'm happier with the reintroduced-but-much-simplified checkboxes – what do we think? I think being encouraged to check them off would make us more accountable than just leaving a bullet list of tasks for us to look at.


Added suport for ubuntu 22.10

I'm not sure if it's related but seems like there are a few e2e tests failed.


The --flavor option of provision --how minute doesn't seem to work

I tried to request a ci.m5.large flavor, but I got a machine with just 1 core.
$ tmt run tests -n . discover provision --how minute -i 1MT-RHEL-<redacted> -F ci.m5.large -vvv login
/var/tmp/tmt/run-036

/plans/default
[...]
    provision
        how: minute
        order: 50
        image: 1MT-RHEL-<redacted>
        flavor: ci.m5.large
        distro: Red Hat Enterprise Linux <redacted>
        kernel: <redacted>
        login: Starting interactive shell
Warning: Permanently added '<IP_ADDRESS>' (ECDSA) to the list of known hosts.
[root@ci-vm-10-0-137-198 provision]#
[root@ci-vm-10-0-137-198 provision]# nproc
1
[...]

When I run 1minutetip --flavor ci.m5.large -n rhel8, I get a 16-core machine as expected.

Seems we should skip the prereserve check when custom flavor is requested. Fix proposed in #501, @FrNecas could you please review? Thanks.

Seems we should skip the prereserve check when custom flavor is requested. Fix proposed in #501, @FrNecas could you please review? Thanks.


Create FC network - error

Scenario/Intent
Create a FC network
Environment Details

Module Version: 5.3.1
Ansible Version: 2.7.15+
OneView Appliance Version: 4.20.01
OneView Client API Version: 800
Python Version: 2.7
python-hpOneView SDK Version: 5.0.0
Platform: [OS distribution and release version]

Steps to Reproduce

name:  Configure fc networks from csv
hosts: localhost
vars:
config: 'oneview_config.json'
tasks:

name: Create fc network FC-A
oneview_fc_network:
config: '{{config}}'
state: present
validate_etag: False
data:
type:                       'fc-networkV4'
name:                       'FC-A'
description:                'FC network'
autoLoginRedistribution:    True
linkStabilityTime:          20
fabricType:                 FabricAttach

Expected Result
FC network is created but the playbook generates error.
Actual Result
TASK [Create fc network FC-A] **********************************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: TypeError: Value of unknown type: <class 'hpOneView.resources.networking.fc_networks.FcNetworks'>, <hpOneView.resources.networking.fc_networks.FcNetworks object at 0x7ff57d68ba50>
fatal: [localhost]: FAILED! => {"changed": false, "module_stderr": "Traceback (most recent call last):\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567630578.31-68185004924295/AnsiballZ_oneview_fc_network.py", line 114, in \n    _ansiballz_main()\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567630578.31-68185004924295/AnsiballZ_oneview_fc_network.py", line 106, in _ansiballz_main\n    invoke_module(zipped_mod, temp_path, ANSIBALLZ_PARAMS)\n  File "/home/dung/.ansible/tmp/ansible-tmp-1567630578.31-68185004924295/AnsiballZ_oneview_fc_network.py", line 49, in invoke_module\n    imp.load_module('main', mod, module, MOD_DESC)\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/main.py", line 124, in \n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/main.py", line 120, in main\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/ansible_oneview_fc_network_payload.zip/ansible/module_utils/oneview.py", line 306, in run\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/ansible_oneview_fc_network_payload.zip/ansible/module_utils/basic.py", line 2046, in exit_json\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/ansible_oneview_fc_network_payload.zip/ansible/module_utils/basic.py", line 2039, in _return_formatted\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/ansible_oneview_fc_network_payload.zip/ansible/module_utils/basic.py", line 418, in remove_values\n  File "/tmp/ansible_oneview_fc_network_payload_WbS8Cl/ansible_oneview_fc_network_payload.zip/ansible/module_utils/basic.py", line 401, in _remove_values_conditions\nTypeError: Value of unknown type: <class 'hpOneView.resources.networking.fc_networks.FcNetworks'>, <hpOneView.resources.networking.fc_networks.FcNetworks object at 0x7ff57d68ba50>\n", "module_stdout": "", "msg": "MODULE FAILURE\nSee stdout/stderr for the exact error", "rc": 1}

@DungKHoang, I am not able to reproduce the error using the same environment you have mentioned.
Can you please check the Python SDK version and AnsibleOneView version you are using?

I was able to create FC network by using the environment details provided by the customer without any error. Hence, closing this issue.

Closing as per Yuvirani's comment


Nightly wheels

🚀  Motivation
As time between release can be a couple of months, it is important to publish wheels for the different supported infrastructures on a shorter term basis
🛠 What does this PR do?

Create a nightly release once a day each time master has changed
Publish a zip containing all wheels corresponding to the change
Once a new named release is created, the nightly ones are deleted

You made tests it in your repo but AFAICT not with this final version, please test and provide a link.
Code is complex, but if I am not mistaken, I emulated it on my repo between v0.9.2 and master (by moving randomly master) and it would look like https://github.com/dbarbier/scikit-decide/releases/tag/nightly (each 85MB file is replaced by dummy content to not waste resources), it is impossible to know which file is the latest one.

is
Indeed I did test it but get rid of most of my tests once master was reset.
The encoding of the name is made by git on master which has a linear history. In the following pattern nightly-x-aha the higher x the more recent is your file.
It is therefore weird to have multiple similar x Can you detail how you did your tests?

Consider that master and nightly are at the same commit. When master moves, git describe --tags --long will print something like nightly-4-g<hash>, then nightly tag is moved to master, and so forth.

Even if master refers to the last release, it would not work better: https://github.com/dbarbier/scikit-decide/releases/tag/nightly2
Anyway I suggest to first publish only one asset, and when it works, to see how to keep multiple assets.

What does skdecide.__version__ look like?

@galleon What does skdecide.__version__ look like?

with my current env:
➜  tmp git:(master) ✗ ipython
Python 3.8.6 (default, May 10 2021, 12:38:32)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.23.1 -- An enhanced Interactive Python. Type '?' for help.
In [1]: import skdecide
In [2]: print(skdecide.version)
0.9.3-dev
Starting from 3.8 one could use importlib.metadata.version but we are still supporting 3.7

Okay so this commit has nothing to do with nightly wheels and should be moved elsewhere.

Okay so this commit has nothing to do with nightly wheels and should be moved elsewhere.

You are right, it is connected as it can be used to track the version of an installed nightly wheel. It has been moved in an other PR.

Versioning of nightly wheels has been changed to track them using:

distance to latest release (vx.y.z)
sha number

It has been tested on galleon/master:

using a 0, 1, *,  *, * crontab
commiting dummy files

In build.yml:

if nightlytag does not exist it is created
Nighly release is created or updated
New artifact is added to the release

release.yml deletes the nightly tag and release

Wheels are alphabetically sorted and not numerically sorted, which looks weird, see https://github.com/dbarbier/scikit-decide/releases/tag/nightly2

If I understand well this refactorying,

each night, master having changed or not, we update the nightly tag to new master position and add the new wheels to assets while keeping all other ones as well
at new release, we remove all these wheels to start again the process

Which means, between releases, we store wheels for all nights (e.g if there are 2 months between releases, we will store about 60 wheels zip). Am I correct ?

If I understand well this refactorying,
* each night, master having changed _or not_, we update the nightly tag to new master position and add the new wheels to assets while keeping all other ones as well

[...]
Nothing is done if master and nightly are at the same commit (test l. 369)

Actually @galleon added a commit for this test once I made the comment.

Actually the commit for this test was added once I made the comment.

okay this is another example why rebasing before review is finished is annoying.

This time at least, i do not think this was the issue as the commit has been done just after my comment and has not yet been squased or rebased (db0cb64fc56c61781791a7721ce593aadb6ccd48)


Remove applicant IDs from URL paths

Overview
When applicants use CiviForm, nearly all of the URL routes they interact with are prefixed with their applicant ID in the form /applicants/:applicantId. For example, landing on the homepage redirects (after creating a guest account for the visitor) to /applicants/:applicantId/programs.
This is an issue when applicants share URLs to CiviForm, since whoever they share it with will be attempting to access the sharer's applicant and so CiviForm detects unauthorized access and redirects them to the homepage. This prevents, for example, applicants from being able sharing direct links to specific programs with others.
Implications for TIs
The original reason for doing this was for simplicity so that TIs and applicants would have the same routes -- implementation was quicker this way. It also allows multiple TIs who work with the same client to share URLs of a client's application with one another for collaboration.
A possible improvement
One way to improve this could be to create a separate set of routes for the applicant only that do not include the /applicants/:applicantId prefix and retain the original routes for the TIs' use only.
For applicants, rather than referencing the URL for the current applicant, the CiviForm server would detect the account type as an applicant and resolve their ID from session state.

To remove the applicant id from URLs, we obviously need new entries in routes. Play doesn't allow controller methods to be overloaded, so we need new controller methods. (There are some ways of specifying default parameter values, but that's only for parameters that are specified in the query string.)
Each of the applicant controller methods needs to get the applicant id somehow. If it's not in the URL, we should get it from the session. We already store the pac4j user profile in the session, with the invariant that the id of the profile is the same as the id of the associated account. So it's simple to get the account id from the request.
But we need to get the applicant id, not the account id. To avoid doing an extra db fetch, we will also store the applicant id in an attribute of the user profile.
(Note that a new db query is not strictly required, as the account is fetched when we check the authorization for each action. However, this currently happens after we fetch some information about the applicant, so that would require quite a bit of refactoring.)
Now that we have two routes/controller methods for each applicant action, we need to populate the appropriate URLs for different scenarios. Every place that we calculate a route for a controller method, we need to decide if this should be the controller method for applicants or the one for TIs. We can handle this by providing the Role corresponding to the user profile to the appropriate views.

Once applicants get a new profile, it will contain the application id attribute, but we need to ensure that existing user profiles (without the attribute) continue to work with the new scheme.

We decided we can enable this for all deployments in this Slack thread.


Create Gallery Image “hhhhhhhhhhhh”

Automatically generated by Netlify CMS

👷 Deploy Preview for determined-meninsky-f29d45 processing.
🔨 Explore the source changes: eb8f6e7e78a0b04add50f8aef89cb30e98b6dc3b
🔍 Inspect the deploy log: https://app.netlify.com/sites/determined-meninsky-f29d45/deploys/6151db3a536ced00075a6f6f


how to user getchats in libtdjson.so

hey
i send this message to Tdlib via libtdjson but get error
how to set chat_list in json format?
$tdlib->send([
"@type" => "getChats",
"chat_list"=>[
"@type"=>"ChatList",
"ChatList"=>"chatListMain"
],
"offset_order" =><PHONE_NUMBER>854775807,
"offset_chat_id" => 0,
"limit" => 200,
]);
{"@type":"error","code":400,"message":"Failed to parse JSON object as TDLib request: Unknown class "ChatList""}

try
$tdlib->send([
"@type" => "getChats",
"chat_list"=>[
"@type"=>"chatListMain",
],
"offset_order" =><PHONE_NUMBER>854775807,
"offset_chat_id" => 0,
"limit" => 200,
]);

try $tdlib->send([ "@type" => "getChats", "chat_list"=>[ "@type"=>"chatListMain", ], "offset_order" =><PHONE_NUMBER>854775807, "offset_chat_id" => 0, "limit" => 200, ]);

I try this
get new error:
{"@type":"error","code":401,"message":"Unauthorized"}

means you are not logged in? 😟

means you are not logged in?

it's true
the is some problem with session
thank


Interactive Tutorial - Keep the tutorial text visible in Fullscreen Mode

at the moment you can put the terminal window into fullscreen mode, which is great.
The lef pane with the turorial text disappears in though when you go fullscreen,
It would be even greater whether you could keep the tutorial text pane active in fullscreen mode so you can follow through without having to switch back and forth all the time (which makes the advantage of a larger view almost go away because it is not so practical whether you have the tutorial text diappearing)

/kind feature
/area web-development


Adding multiple TCP listeners from Consul

We have been working on providing a solution for TCP load balancing for a potential large number of applications.  Our requirements are that the service can be defined in consul, and the load balancer will automatically start the TCP listener.  Also, because we could have different applications requiring the same TCP port, we need to define the route by IP address.
With the current TCP implementation in Fabio, I am only able to start a listener in the properties file, and define a backend route using :port which doesn't match our current use case.
We have built a new proxy option that we are currently calling "tcp-dynamic" that uses the consul urlprefix tagging to start the TCP listeners and defines the backend route using ip:port.  Our initial testing has been promising, we were able to define 5000 different TCP ports in consul and start the listeners with pretty minimal CPU increases.  We need to do a little more cleanup, but should have something ready for feedback early next week.

@craighazen and I have created pull request #626 for this issue.  Take a look and let us know if there is any feedback.

I'm currently trying to do the same thing using consul-template but I'm not getting very far.
Pseudo code would be:

iterate over every service passing
check the tags include "proto=tcp"
update the proxy.addr in fabio.properties to include the urlprefix- tag and do some cleanup

Something like this (pseudo template):
{{ range services }}
{{ if .Tags | join "," | regexMatch "proto=tcp" }}
<< something here to extract `fabio-:10000 proto=tcp` >>
proxy.addr = :9999,<<:10000;proto=tcp>>
{{ end }}
{{ end }}

It'd be much better to just have this dynamic for sure.


update org.json version

Fix the vulnerability issues

(Jove Github Bot) added it to the current sprint.

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).


Mouse events not working with Win XP and QuickEdit mode

Hi,
As per title, the mouse events gets inhibited in Win XP when QuickEdit mode is active.
It should be possible to inihibit this behavior programmatically through [get/set]ConsoleMode and ENABLE_QUICK_EDIT_MODE. Possibly this needs to be done before entering the main loop and should set it back on exit. A big plus would be to enable/emulate clipboard events during the lack of quick edit mode.
BTW: thanks for sharing such an awesome project :-)

Thanks for using it!  I really didn't expect it to get this level of interest when I started...
As for youe bug?  That also reproduces on Windows 7.  Looks like pywin32 is missing some constants, though, so I've had to code them in directly...  This patch looks to do the trick: https://github.com/peterbrittain/asciimatics/commit/88458fb882b321be30dc15318246f96c02793ce4
You can actually re-enable quickedit mode at any time by going into your command prompt settings, so it's not perfect and I should add a troubleshooting section to address this.  Doing more is going to require mouse drag facilities, which I've not added yet (but is on the queue for V2.0).
Can you please confirm that this patch works for you?  Then I'll update the docs and close...

Confirmed! ;-)

Fab! Now documented and closed...


Fixed the set_frame not plotting issue

Closes #469

I forgot the tests

@Juanlu001 Any changes that you'd suggest?

Apart from the quality issues (I'll leave the answer as an exercise to the reader) this PR needs a rebase on current master.

I could reproduce the docs failure: it's been a regression introduced by matplotlib 3.0.1, released 12 hours ago: https://github.com/matplotlib/matplotlib/issues/12634
@shreyasbapat can you please put matplotlib<3.0.1 in setup.py?

Sure!


Operator starts wrong job types

Describe the bug
During testing the newest RC of the k8up operator running on test1, it's observed that the operator seems to start a prune pod instead of a backup pod when a backup schedule is changed so that the backup runs at a specific time.
it's assumed this behavior happens because a prune hasn't previously completed successfully for this project. This prune pod then starts and fails to run because it is missing the AWS secret key in its env vars. This prune pod will then remain in crashloop backoff state for about 30 minutes due to the missing AWS secret key before it is marked as failed by the operator. After this prune job fails, no backup is attempted afterwards.
Additional context

specifically observed this behavior on both the drupal-example-test1-master and drupal-example-test1-develop environments.
verified that the missing secret key does seem to exist in the syn-backup namespace's global-bucket-credentials secret, so this may be an issue with how the operator is creating that prune pod specification?
As the logs show, the controller correctly parsed out the timeframe the prune should run, but it still tried to schedule a prune instead of a backup. This prune then failed, and the controller did not attempt to perform a backup.
tried to remove the prune key in the schedule object, and a check was attempted instead of a backup, which still failed for the same reason. After removing both the prune and check keys, it was found that a backup was started and the pre backup pods ran through successfully, but the same issue with the missing AWS secret key still persists, causing that backup pod to crashloop.

Logs
If applicable, add logs to help explain your problem.
2020-12-31T17:18:09.973+0100	INFO	controllers.Schedule	registering schedule for	{"schedule": "drupal-example-test1-master/k8up-lagoon-backup-schedule", "type": "backup", "schedule": "19 * * * *"}
2020-12-31T17:18:09.973+0100	INFO	controllers.Schedule	registering schedule for	{"schedule": "drupal-example-test1-master/k8up-lagoon-backup-schedule", "type": "check", "schedule": "28 3 * * 0"}
2020-12-31T17:18:09.973+0100	INFO	controllers.Schedule	registering schedule for	{"schedule": "drupal-example-test1-master/k8up-lagoon-backup-schedule", "type": "prune", "schedule": "28 3 * * 6"}
2020-12-31T17:18:09.973+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Schedule", "controller": "schedule", "name": "k8up-lagoon-backup-schedule", "namespace": "drupal-example-test1-master"}
2020-12-31T17:19:00.000+0100	INFO	controllers.Schedule	running schedule for	{"schedule": "drupal-example-test1-master/k8up-lagoon-backup-schedule", "jb": "prune"}
2020-12-31T17:19:00.026+0100	INFO	controllers.Prune	adding job to the queue	{"prune": "drupal-example-test1-master/prunejob-1609431540"}
2020-12-31T17:19:01.558+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:19:01.558+0100	INFO	observer	new event on observed job	{"event": "create", "jobName": "drupal-example-test1-master/prunejob-1609431540"}
2020-12-31T17:19:01.565+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Prune", "controller": "prune", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:19:01.610+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:19:01.611+0100	INFO	observer	new event on observed job	{"event": "running", "jobName": "drupal-example-test1-master/prunejob-1609431540"}
2020-12-31T17:19:01.617+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:19:01.617+0100	INFO	observer	new event on observed job	{"event": "running", "jobName": "drupal-example-test1-master/prunejob-1609431540"}
2020-12-31T17:25:04.553+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:04.553+0100	INFO	observer	new event on observed job	{"event": "failed", "jobName": "drupal-example-test1-master/prunejob-1609431540"}
2020-12-31T17:25:04.553+0100	INFO	controllers.Prune	cleaning old job	{"prune": "drupal-example-test1-master/prunejob-1609431540", "namespace": "drupal-example-test1-master", "name": "prunejob-1609025280"}
2020-12-31T17:25:04.561+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Prune", "controller": "prune", "name": "prunejob-1609025280", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:04.568+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Prune", "controller": "prune", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:04.575+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Prune", "controller": "prune", "name": "prunejob-1609431540", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:06.006+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609025280", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:06.006+0100	INFO	observer	new event on observed job	{"event": "delete", "jobName": "drupal-example-test1-master/prunejob-1609025280"}
2020-12-31T17:25:06.006+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "batch", "reconcilerKind": "Job", "controller": "job", "name": "prunejob-1609025280", "namespace": "drupal-example-test1-master"}
2020-12-31T17:25:06.006+0100	INFO	observer	deleting job from observer	{"jobName": "drupal-example-test1-master/prunejob-1609025280"}
2020-12-31T17:25:06.027+0100	DEBUG	controller	Successfully Reconciled	{"reconcilerGroup": "backup.appuio.ch", "reconcilerKind": "Prune", "controller": "prune", "name": "prunejob-1609025280", "namespace": "drupal-example-test1-master"}

Expected behavior
To Reproduce
Steps to reproduce the behavior:

Specs

apiVersion: backup.appuio.ch/v1alpha1
kind: Schedule
metadata:
  creationTimestamp: "2020-09-01T20:52:05Z"
  finalizers:
  - k8up.syn.tools/schedule
  name: k8up-lagoon-backup-schedule
  namespace: drupal-example-test1-master
spec:
  backend:
    repoPasswordSecretRef:
      key: repo-pw
      name: baas-repo-pw
    s3:
      bucket: baas-drupal-example-test1
  backup:
    resources: {}
    schedule: 28 0 * * *
  check:
    resources: {}
    schedule: 28 3 * * 0
  prune:
    resources: {}
    retention:
      keepDaily: 7
      keepWeekly: 6
    schedule: 28 3 * * 6
  resourceRequirementsTemplate: {}

Commands

Environment (please complete the following information):

Image Version: v1.0.0-rc2
K8s Version: v1.18
K8s Distribution: Rancher

It's important to note that there are two distinct issues here:

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

Any time a prune key exists in the schedule object, a prune is always attempted first.

All pods created by the operator are missing their AWS secret credentials

This affects all backup, check, and prune pods created by the operator equally.
Since prebackup pods do not actually send data to S3 directly, they are unaffected.

It's important to note that there are two distinct issues here:

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

Any time a prune key exists in the schedule object, a prune is always attempted first.

All pods created by the operator are missing their AWS secret credentials

This affects all backup, check, and prune pods created by the operator equally.
Since prebackup pods do not actually send data to S3 directly, they are unaffected.

Thanks for clarifying. We have identified the AWS secret issue already in the meantime, so we can focus this issue on the job type.

Thanks for clarifying. We have identified the AWS secret issue already in the meantime, so we can focus this issue on the job type.

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

Any time a prune key exists in the schedule object, a prune is always attempted first.

I was not yet able to reproduce this locally. We have decided to proceed with releasing rc3 and re-evaluate this in said version.

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

Any time a prune key exists in the schedule object, a prune is always attempted first.

I was not yet able to reproduce this locally. We have decided to proceed with releasing rc3 and re-evaluate this in said version.

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

I was not able to reproduce and observe this problem with v1.0.0-rc3 locally, neither on the test cluster. @cdchris12 is this still anything you experience/observe? Can you provide additional guidance?

The operator starts prune pods instead of check pods or backup pods, regardless of which type should be ran

I was not able to reproduce and observe this problem with v1.0.0-rc3 locally, neither on the test cluster. @cdchris12 is this still anything you experience/observe? Can you provide additional guidance?

I've tested v1.0.0-rc3 extensively, and I cannot replicate this issue anymore. I think we can call this case closed :)

I've tested v1.0.0-rc3 extensively, and I cannot replicate this issue anymore. I think we can call this case closed :)

Thanks @cdchris12 for reporting back. Let's close this issue for now. If you encounter this problem again, just open a new issue.

Thanks @cdchris12 for reporting back. Let's close this issue for now. If you encounter this problem again, just open a new issue.


Move the configuration of the WindowPlugin to a resource

Objective
It is currently hard to configure the WindowPlugin, as it is added as part of the DefaultPlugins. Ideally this should not be difficult.
Solution
Remove the configuration from the plugin itself and put it as a Resource, similar to how it is done for almost all other plugins.
Migration Guide
If you are currently configuring the behavior of the WindowPlugin, by constructing it manually, then you will need to instead create add the WindowSettings as a resource.

bors r+


Support hosts (e.g. notify) and auth (e.g. noauth)

This makes longpoll work. It fixes #42 and #55.

Just one small comment. Squash and rebase and it should be good to go assuming that you've actually tried this on a longpoll route.

Rebased, tested upload/download (found a typo!), and retested that longpoll actually works. (It does.)