Examples don't have the main bar colored anymore

It used to be that the progress bar template style supported coloring, but this behavior seems to have broken by commit 5038ba8019bad296b5a0314770db5d05c050a7dd according to git bisect.
Before that commit, cargo run --release --example download would print a progress bar where the hashes (#) were colored in blue. Starting at that commit, they are printed as white. I don't think this is supposed to happen.

This issue seems to be related with the following codes:
https://github.com/console-rs/indicatif/blob/9cb25a71f466bbef39cf210c2a2ee238f7a02a62/src/style.rs#L241-L260
https://github.com/console-rs/indicatif/blob/9cb25a71f466bbef39cf210c2a2ee238f7a02a62/src/style.rs#L324-L349
If key is "wide_bar" or "wide_msg", buf will be set to \x00, and the step of applying FirstStyle to buf is skipped. (Note that AltStlyle will still take effect)
Here is a test case to show this issue:
// append this to `crate::style::tests`
// and run it by `cargo test style::tests::wide_element_style`
#[test]
fn wide_element_style() {
    const CHARS: &str = "=>-";
    const WIDTH: u16 = 8;
    let pos = Arc::new(AtomicPosition::new());
    // half finished
    pos.set(2);
    let state = ProgressState::new(Some(4), pos);
    let mut buf = Vec::new();

    let style = ProgressStyle::with_template("{wide_bar}")
        .unwrap()
        .progress_chars(CHARS);
    style.format_state(&state, &mut buf, WIDTH);
    assert_eq!(&buf[0], "====>---");

    buf.clear();
    let style = ProgressStyle::with_template("{wide_bar:.red.on_blue/green.on_cyan}")
        .unwrap()
        .progress_chars(CHARS);
    style.format_state(&state, &mut buf, WIDTH);
    assert_eq!(
        &buf[0],
        "\u{1b}[31m\u{1b}[44m====>\u{1b}[32m\u{1b}[46m---\u{1b}[0m\u{1b}[0m"
    );

    buf.clear();
    let mut style = ProgressStyle::with_template("{wide_msg:^.red.on_blue}").unwrap();
    style.message = "foobar".into();
    style.format_state(&state, &mut buf, WIDTH);
    assert_eq!(&buf[0], "\u{1b}[31m\u{1b}[44m foobar \u{1b}[0m");
}

Once the following part of code is commented, wide_bar and wide_msg are colored correctly.
https://github.com/console-rs/indicatif/blob/9cb25a71f466bbef39cf210c2a2ee238f7a02a62/src/style.rs#L324-L328
But I'm not sure it's possible to simply remove this code without causing other bugs. Although all tests pass fine, including style::tests::align_truncation introduced by the commit https://github.com/console-rs/indicatif/commit/5038ba8019bad296b5a0314770db5d05c050a7dd.


Update 12-4

Hi thanks for the pull request. Is this CLRS version 2 or 3?

It's version 3. You just misplaced questions and anwsers 12.3.2 - 12.3.6 in 12.4.md. They're in both 12.3.md and 12.4.md.

Yep. Thanks for fixing it and the pull request. ^_^


Is it possible to filter services by instance-zone tag without custom queries?

Posted my question on stackoverflow after reading project guidelines, was hoping for a response to help shed light on this subject.
Are we misunderstanding the use instance-zone? The assumption is new services registered in Consul can be tagged using spring.cloud.consul.discovery.instance-zone, we assign spring active profile to this key.  SCC clients use FeignClient to disscover and call REST endpoints in this service. Since we have one physical COnsul cluster for all operational environments, we want to ensure Springboot clients discovering services see only those services with instance-zone=client active runtime profile, to avoid a "dev" client discovering QA or Prod services.  Are we on the right path? We're trying to avoid building custom querys to do our own filtering using tags, though I suppose if that's a last resort we'll do it.
# for registered service
spring:
  cloud:
    consul:
      discovery:
        instance-zone: dev
        tags: dev

Please open https://github.com/spring-cloud/spring-cloud-consul/issues

My bad, was not paying attention to where I was posting


Compiling issue with Ruby Sass

I am running a project that is using Ruby Sass to compile SCSS to CSS. When using the terminal command: sass --watch scss:css -t expanded --sourcemap=none
the last column of a grid breaks instead of float left.
When using sass --watch scss:css -t compressed --sourcemap=none
the last column of a grid floats left as expected.
Help.

@oneblackcrayon
What version of Ruby Sass?
What version of bootstrap-sass?
Which web browser and version thereof?
I'm not sure whether we officially support compressed mode. Probably it's better to use a dedicated CSS minifier tool instead.

Closing due to lack of reply to my questions.

My apologies. It may have been due to outdated Ruby Sass.


A11y_Azure Data Studio_Nov2023_Database Properties Preview_Filegroup_VoiceOver: Voice over is announcing all the column header information twice which is present on "Rows" "File Stream" "Memory Optimized data" grid.

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.
Git hub Tags:
#A11yTCS; #A11yMAS; #A11ySev3; #DesktopApp; #WCAG1.3.1; #FTP; A11yTCS, #SH_Azure Data Studio_Nov2023_Mac_Dec2023; #SQL Azure Data Studio; #MAC; #VoiceOver;
Environment Details:
Application Name: Azure Data Studio
Operating System: MAC
Azure data studio 1.47.0
VoiceOver
Repro Steps:

Install Azure data studio application and open it
Make sure to enable "Preview" feature in settings.
Tab till "Database" and press ENTER to expand the tree.
Tab till any database and pressF10 to open context menu.
Tab till "Properties Preview" and press ENTER.
Tab till "FilesGroup" tab and press ENTER.
Tab till "Database files" table.
You will observe that voice over is announcing all the column header information twice which is present on "Rows" "File Stream" "Memory Optimized data" grid.

Actual Result:
When navigating to "Database files" table voice over is announcing all the column header information twice which is present on "Rows" "File Stream" "Memory Optimized data" grid.
Note: Issue seen throughout the application for similar tables.
Expected Result:
When navigating to "Database files" table voice over should not announce all the column header information twice which is present on "Rows" "File Stream" "Memory Optimized data" grid.
User Impact:
Screen Reader users will face difficulties if the voiceover is announcing the information of the column headers twice.
Attachment:

@HindujaArugula the same reported behavior with the table header was previously filed and closed as non-blocking after attending the accessibility office hours. Can this issue be closed since the previous issue with the same behavior was approved to be closed?


Added the first sprint plan.

I've added the first sprint plan. If there's something missing, please let me know!

Yes, this sprint plan is as we discussed and planned, we are going to focus on researching about the topic, product vision and planning for next week.

Looks good to me @ClintonCao !


Godot 3.2 Alpha-2 Glitchy tilemaps

Godot version:
3.2 Alpha-2
OS/device including version:
Windows 10
Possible important?:
GPU: Intel HD 4K
CPU: Core i5
Tried with GLES2 & GLES3
Issue description:
When painting Tile Maps the screen is really jittery, video attatched
Steps to reproduce:
Create a Tilemap in Node2D (I use hexagon tiles so i put the following Cell settings)
Half-Offset: Offset-X
Cell Size X: 122
Cell Size Y: 104
Every other settings on the tilemap are left untouched, im not sure if the issue is caused because of the Offset-X or not haven't tried without itIts not an offset issue just tried it
Minimal reproduction project:
New Game Project.zip
Once i find a good site for uploading a video to ill post it as a comment, for now ping me on discord for the video until i post it (@Notlad | Node (Grr)#8402 )
EDIT: Had to use youtube :(
https://youtu.be/Zh2NxKrMs_A

Add to this... On windows

Saving a scene whilst editing a tilemap causes the tilemap edit to stop working. Does not happen on mac osx.
Changes of the some of the tilemap properties on windows do not take effect immediately. Especially glitchy with the center of the tile property.

I think it has something to do with godot's window size. On smaller displays I get this behavior, but not on my 1080x1920 monitor when godot is maximized.

[In terms of you using YouTube, just an FYI you can upload gifs to github and they'll be rendered properly. Converting the video to a gif and drag-and-dropping it in will probably work just fine]

i have tried a few things, not a larger moniter but i can try that in the next few days, also npip i tried uploading the video raw and github said "Nuh uh" 😂


docs(#1574): Add new language

Hello,
I'm back with the work I did, I added a new language, namely Indonesian, this is not finished yet.
I'll finish it later, thanks.

Thank you for your contribution! As the source repository has been deleted, we are unable to merge this pull request. When you are ready, feel free to submit it again!


Css fixes

Continue to fix font sizes in mirador.css
Moved overlay div so it is a child of content-container instead of view-container.  This means the size of the overlay does not change if the side panel is open.
No longer use table-row and table-cell for items in MetadataView so the text doesn't get really scrunched up when multiple slots are open.
Correctly code for a metadata object.  Still defaults to English, for now.
Display logo image, rather than URL.

Hm. Can you explain 2) above a bit more? Is it the metadata panel that's disrupting the layout of annotations somehow?
I'm ready to merge otherwise.

If you look at an object with a table of contents on http://oculus-dev.harvardx.harvard.edu/demo/ and have the overlay open as well.  Minimize the left side panel and you will see the width of the overlay changes because originally it was a child of content-container and the width is set to 45%.  With this PR, the overlay element is no longer a child of content-container, so its width is not affected by the visibility of the side panel.  Now the overlay's width is basically determined by the size of the slot, not the OSD container.


Extension Support for Download Center

Hi,
I'd like to ask you for Support for the Extension "Oxpus Hotschi's Downloads".
You can find it here: https://phpbb3.oxpus.net/dl_ext/?cat=6
It's a simple Download Center with great usability.
It's working with your Theme but I have issues with the sizes on the table as you can see on the attached screenshot. It's working correctly with the default prosilver theme, so i guess there's some optimation needed with eles style.
I use phpbb 3.1.9 + Hotschi's Download 7.1.14 (both should be the latest Versions)

kind Regards

Hi,
do you have any news about this enhancement?
kind Regards


Please release a set of stock data to allow followers to verify the performance of Finmem, or at least provide a detailed description of the data processing procedure.

Dear Finmen Authors,
Firstly, thank you once again for making this project open source, allowing us to engage with cutting-edge research in this field. We place great importance on the results presented in the Finmen paper and are urgently seeking to replicate the results (with a deviation of less than 3%).
We have strictly followed the instructions and code processes described in the paper for data processing: News data is sourced from the Alpaca News API, manually verified for accuracy, and supplemented with Reuters news to achieve the 1655 entries mentioned in the code. The 10K10Q data comes from official documents, and all model-related parameter settings are aligned with the paper. However, the results we obtained significantly differ from those in the paper.
During this process, other researchers have provided their replication results of Figure 7 (regarding TSLA stock) from the paper, which also shows significant discrepancies. The README document of the data-pipeline mentions that Reuters news is private data ('Gather news content from the Refinitiv Real-Time News (Private Dataset), mainly comprising Reuters articles.'), but the unavailability of source data poses a major obstacle for paper replication, making it highly challenging for followers to successfully replicate the results. We urgently hope that the Finmen authors can release a small set of source data (e.g., NFLX) for followers to verify or at least provide detailed data processing procedures. There are many details in the current data processing workflow that are not disclosed and cause considerable confusion, such as why the 10K reports mentioned in Table 1 of the paper are updated daily ('Deep Processing: 10-K fillings, all summarized by LLM. Updated yearly. Extended reflections for the stock in response to the trading inquiry include FINMEM’s cumulative trading returns, decision-making processes, trade volumes, and underlying reasons. Updated daily.')?
We highly value the achievements of Finmen and, in the spirit of rigorous scientific research, hope to verify the accuracy of its results. We have already made our processed NFLX news data and 10K10Q data public in our Github repository for other followers to check and discuss. We earnestly request that the authors consider our plea and look forward to your response!

Hello,
We understand your concerns but must clarify several points where responsibilities and limitations are being misunderstood. Firstly, the request for us to open-source some data cannot be fulfilled due to strict legal constraints imposed by copyright laws that govern most financial data from most reliable data sources. This is not a choice but a legal requirement within our domain. Providing an open-source code base is the best way we can contribute to the FinAI community, given the legal constraints we face.
We have responded promptly and in detail to your multiple inquiries, providing the necessary clarifications. This statement serves as a formal clarification to prevent any legal misunderstandings. Legally, we cannot provide the data as it would result in a breach of law; however, you are welcome to procure the data through the proper channels independently.
Secondly, regarding your mention of other researchers attempting to replicate our results and observing discrepancies, it is difficult to address these claims without specific details about their research group, the settings employed, and the nature of any differences in their approach. It's important to note that despite updates to LLMs and creative settings, the performance of the ticker 'TSLA' remains robust, underscoring the strength of the FinMem framework.
We hope this clarifies our position and limitations. Unfortunately, given these constraints, we cannot proceed with collaboration under the terms you propose.

Thank you very much for your patient response, and we deeply apologize for any inconvenience our previous requests may have caused. We fully understand and respect considerations regarding data privacy, thus we will no longer pursue access to the original data of the Finmem project. We highly appreciate and respect your work, especially the impressive results achieved with TSLA stock without the disclosure of data.
We would like to understand more about the data processing details to better comprehend and replicate your research outcomes. Here are a few questions we hope to gain further insights on:
We understand that you cannot disclose the raw data, but according to the paper, this data can be obtained through programming or manual efforts. We have tried to process the data as described in the paper, yet the results are not satisfactory. We have made our processed NFLX data public; could you possibly review it and point out any potential issues or deficiencies?
Besides the sources mentioned in the paper, "News data is sourced from the Alpaca News API" and in the README file, "mainly comprising Reuters articles," have you used any other sources for news data?
According to the logic in the Finmem code, the number of effective news articles processed from the Alpaca News API and Reuters exceeds 1655. How did you select and control the number of news articles to exactly 1655? Are there any undisclosed details in the news data compilation process?
Regarding the 10K10Q data, given that the target sections still contain a significant amount of text and the paper does not specify the exact summarization method, we have summarized based on the report titles. How does the Finmem project team handle this type of data?
When changing the target stock, it is necessary to modify the corresponding parts of the code. How is the character_string defined in the config file? (From year 2016 to 2017 September, Tesla's continued growth and solid financial performance over the defined period, despite certain quarters where revenue missed Wall Street expectations.)What was the rationale behind selecting the period from 2016 to September 2017 mentioned in the character_string for TSLA?
We are eager to understand the reasons behind the discrepancies between the replication results and the results reported in the paper, and to find solutions. We admire the high-quality code you have provided and believe that the issue may lie within the details of data processing. We have tried our best but still feel helpless and greatly need your assistance. We are willing to provide any form of support and hope you will consider our request. Thank you very much for your time and help!
张全意
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "pipiku915/FinMem-LLM-StockTrading"                                                                                @.>;
发送时间: 2024年6月21日(星期五) 中午12:26
@.>;
<EMAIL_ADDRESS>主题: Re: [pipiku915/FinMem-LLM-StockTrading] Please release a set of stock data to allow followers to verify the performance of Finmem, or at least provide a detailed description of the data processing procedure. (Issue #18)
Hello,
We understand your concerns but must clarify several points where responsibilities and limitations are being misunderstood. Firstly, the request for us to open-source some data cannot be fulfilled due to strict legal constraints imposed by copyright laws that govern most financial data from most reliable data sources. This is not a choice but a legal requirement within our domain. Providing an open-source code base is the best way we can contribute to the FinAI community, given the legal constraints we face.
We have responded promptly and in detail to your multiple inquiries, providing the necessary clarifications. This statement serves as a formal clarification to prevent any legal misunderstandings. Legally, we cannot provide the data as it would result in a breach of law; however, you are welcome to procure the data through the proper channels independently.
Secondly, regarding your mention of other researchers attempting to replicate our results and observing discrepancies, it is difficult to address these claims without specific details about their research group, the settings employed, and the nature of any differences in their approach. It's important to note that despite updates to LLMs and creative settings, the performance of the ticker 'TSLA' remains robust, underscoring the strength of the FinMem framework.
We hope this clarifies our position and limitations. Unfortunately, given these constraints, we cannot proceed with collaboration under the terms you propose.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Thank you very much for your patient response, and we deeply apologize for any inconvenience our previous requests may have caused. We fully understand and respect considerations regarding data privacy, thus we will no longer pursue access to the original data of the Finmem project. We highly appreciate and respect your work, especially the impressive results achieved with TSLA stock without the disclosure of data.
We would like to understand more about the data processing details to better comprehend and replicate your research outcomes. Here are a few questions we hope to gain further insights on:
We understand that you cannot disclose the raw data, but according to the paper, this data can be obtained through programming or manual efforts. We have tried to process the data as described in the paper, yet the results are not satisfactory. We have made our processed NFLX data public; could you possibly review it and point out any potential issues or deficiencies?
Besides the sources mentioned in the paper, "News data is sourced from the Alpaca News API" and in the README file, "mainly comprising Reuters articles," have you used any other sources for news data?
According to the logic in the Finmem code, the number of effective news articles processed from the Alpaca News API and Reuters exceeds 1655. How did you select and control the number of news articles to exactly 1655? Are there any undisclosed details in the news data compilation process?
Regarding the 10K10Q data, given that the target sections still contain a significant amount of text and the paper does not specify the exact summarization method, we have summarized based on the report titles. How does the Finmem project team handle this type of data?
When changing the target stock, it is necessary to modify the corresponding parts of the code. How is the character_string defined in the config file? （From year 2016 to 2017 September, Tesla's continued growth and solid financial performance over the defined period, despite certain quarters where revenue missed Wall Street expectations.）What was the rationale behind selecting the period from 2016 to September 2017 mentioned in the character_string for TSLA?
We are eager to understand the reasons behind the discrepancies between the replication results and the results reported in the paper, and to find solutions. We admire the high-quality code you have provided and believe that the issue may lie within the details of data processing. We have tried our best but still feel helpless and greatly need your assistance. We are willing to provide any form of support and hope you will consider our request. Thank you very much for your time and help!


Python: Migrate to new service builder

OperationRegistryBuilder will be deprecated soon but we're still using it in generated Python SDKs. This PR migrates codegen and runtime crates to the new service builder.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

This looks good to me.
If nothing here breaks the Python customer facing API then I'm guessing this is fine to merge as is?

I don't expect customers to manually implement PyApp trait so it should be fine as long as they have matching versions of codegen and runtime crate. @crisidev what do you think about it?

I don't expect customers to manually implement PyApp trait so it should be fine as long as they have matching versions of codegen and runtime crate. @crisidev what do you think about it?

Yep, I agree.


delete space :)

This PR goes against the code style that Laravel uses. You can see the check failed. Occasionally, a code style "action" is automatically ran against the repo, so no need to make code style PRs.

Please stop sending these in


Storyboard localization strategy

Noticed in #1268 that storyboards (eg. Settings) do not support localization yet. We should think of how we want to do that.

Bug Report Dump (Auto-generated)

Version 1.16.0<PHONE_NUMBER>)
Device: iPhone X (iOS 11.2)
TestFlight: true

Dupe of #647 nvm
Sent with GitHawk


[bug] animation page and maybe others

learn pages
Description

Version

Platform

Web Version [X]
Windows [X]
Linux [X]

I have hidden the tutorial spine example on purpose, should be restore when i fix it.
Thanks a lot for reporting.

I have hidden the tutorial spine example on purpose, should be restore when i fix it.
Thanks a lot for reporting.

oh ok

Animation page is back running with all examples now!


logging: log: add LOG_WRN_ONCE

Add an equivalent to the linux WARN_ONCE macro. This is intended for
use when the developer should be notified of an event, but may occur a
multitude of times in quick succession.
Using LOG_WRN could result in either cluttered logs or recursive
logging (i.e. in serial drivers).
Proposed in the discussion of #65112

Long time ago there was #41783 but author dropped it and it never get in. There we had LOG_<lvl>_N(n, ...) which was more flexible and supported all severity levels maybe it would be good to resurrect that approach instead?

Long time ago there was #41783 but author dropped it and it never get in. There we had LOG_<lvl>_N(n, ...) which was more flexible and supported all severity levels maybe it would be good to resurrect that approach instead?

I don't mind if that is the approach you want to go with, but I won't be resurrecting it.
IMO only LOG_LEVEL_WRN really makes sense for this macro anyway. DBG should always print, ERR is fatal, so likely won't happen again anyway, and INF is normal operation so shouldn't be spamming the logs in any case. WRN is uniquely "user should be notified, the event is unusual, but because its only a warning, may occur many times in quick succession".


Error while add a new device!

I tried to run mapDriver but it ask me to create a new device, Im using nakating source, and successfully host at my localhost:7890 but when I create a new device, they can't connect to my host, I'm even add port forwarding to 49152 and 65535 , but still can't connect, Im also try <IP_ADDRESS>:7890 , <IP_ADDRESS>:7890 but still, they can't connect. Any ideas?

I can't seem to build the mapDriver in VS2015 =/.

@giaphongdau you couldn't do that helps you connect


ARROW-7213: [Java] Represent a data element of a vector as a tree of ArrowBufPointer

For a fixed/variable width vector, each of its data element can be represented as an ArrowBufPointer object, which represents a contiguous memory segment. This makes many tasks easier and more efficient (without memory copy): calculating hash code, comparing values, etc.
This cannot be achieved for complex vectors, because their values often reside in more than one contiguous memory regions. However, it can be seen that the contiguous memory regions for each data element forms a tree-like structure, whose leaf nodes are the contiguous memory regions. For example, a data element for a struct vector forms a tree, whose root corresponds to the struct vector, while the child vectors corresponds to the child nodes of the tree root.
In this issue, we provide a data structure that represents each data element of a vector as a tree, whose leaf nodes are ArrowBufPointers, representing contiguous memory regions for the data element.
With this data structure, many tasks also becomes easier and more efficient: calculating hash code, comparing vector elements (ordering & equality). In addition, we can do something that could not have been done in the past, like placing data elements into a hash table/hash set, etc.

@pravindra do you want to take a look?  I think you reviewed the other ArrowBufPointer changes.

@praveenbingo @emkornfield Thanks a lot in advance.

@liyafan82 I think given that you are trying to remove this from memory, does the PR still make sense?  In general, I'm not sure if this PR is a direciton we should continue in?

@liyafan82 I think given that you are trying to remove this from memory, does the PR still make sense? In general, I'm not sure if this PR is a direciton we should continue in?

@emkornfield Thanks a lot for your comments.
I don't think it makes sense as it is for now.
Maybe it is inappropriate to place it in the memory package. However, I cannot think of a better way to manipulate individual (complex) vector elements efficiently.
Maybe we should hold this for now, and make a decision when https://github.com/apache/arrow/pull/6208 is finished.

Going to close until the memory PRs are merged and we can figure out if we want this/where we want it.


PROPOSAL: Update the order in which "optional" decorators are evaluated

Addresses https://github.com/RateGravity/type-shift/issues/17
First 3 commits are just housekeeping.
In the current world, decorators involving Noneables will return immediately if the field matches the None type. This means that complex converters such as path, or, etc, will be ignored. This PR changes the order of operations so the decorated converter will be evaluated first, then the fallback undefined/None/default logic if it fails.
This means we can utilize optional converters with defaults and optional forPaths.

This is happening because of our fluent/functional interface mixup.
Here's the behavior you want to be derived from the first principles of logic.
  prop?: string;
}

interface ParentObject {
  child?: ChildObject;
}

const childConverter = t.shape({prop: t.string}).or(t.coerce([t.unknown], {prop: 'works'}));
const parentDefault = {child: { prop: 'defaultFromParent' } };
const parentDefaultConverter = t.coerce([t.none], parentDefault);
const parentConverter = t.shape({child: childConverter}).or(parentDefaultConverter);

describe('nested converter', () => {
  it('parent undefined calls parent default', () => {
    expect(parentConverter(undefined)).toEqual({ child: { prop: 'defaultFromParent' } });
  });

  it('parent empty calls child default', () => {
    expect(parentConverter({})).toEqual({ child: { prop: 'works' } });
  });
});

The issue is that .default and .partial don't follow logic rules properly. I think we're making more of a mess here unless we unwind the convention but that's my 5c.

I think this is making more complex conventions around our mix of fluent vs functional interfaces. My 5c would be to solve this by addressing the root of the problem.
I think this can be solved thru conventio by not using default/optioanl and having a fall thru .or case at the end of the chain.

Annnd wrong button.

I think this is also a breaking change. There is undoubtedly something in the code that unknowningly relies on this logic not making sense.


fix: remove transactions from extension completely

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.73%. Comparing base (4549a45) to head (502dc5d).
Report is 2 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #23   +/-   ##
=======================================
  Coverage   82.73%   82.73%
=======================================
  Files           3        3
  Lines         388      388
  Branches       70       70
=======================================
  Hits          321      321
  Misses         67       67

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


Added support for using ** as a catch-all at the end of a route

This allows you to use ** as a catch-all at the end of a route without breaking its support in the middle of a route. For example, the route /test/** would match all of the following:
/test/one
/test/one/two
/test/one/two/three/four/five/six

This is a potential solution to the scenarios discussed in #405 but won't necessarily work if you're using a variable in the route. I instead use request.path in my handler function to perform any logic based on the matched path.

Any updates on the merging of this PR?

Hey @michaelenger, thanks for contribution! I'll be reviewing this PR in the next days, sorry for the delay 😞.

Can this be merged? This change would enable me to fully transition from GCDWebServer

Sorry for the delay guys and thanks @michaelenger for your contribution!

Hi
I've trying to use the wildcard route to match all the routes and handle them after by the request param. I have tried with server[""] and server["/"] but there is no way to catch any request, even index.html using http://localhost/. Am I doing something wrong? Please, any help would be much appreciated.
Regards.


TypeError on Python 3.11

When running CPython 3.11 workloads on Fargate, CodeGuru Profiler sometimes crashes:
2023-11-14 20:27:16,412 - codeguru_profiler_agent.profiler_runner - INFO - An unexpected issue caused the profiling command to terminate.
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/profiler_runner.py", line 74, in _profiling_command
    sample_result = self._run_profiler()
                    ^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/metrics/with_timer.py", line 26, in timed
    result = fn(self, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/profiler_runner.py", line 98, in _run_profiler
    self._sample_and_aggregate()
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/metrics/with_timer.py", line 26, in timed
    result = fn(self, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/profiler_runner.py", line 103, in _sample_and_aggregate
    sample = self.sampler.sample()
             ^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/metrics/with_timer.py", line 26, in timed
    result = fn(self, *args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/sampler.py", line 46, in sample
    stacks = self._get_stacks(
             ^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/sampling_utils.py", line 34, in get_stacks
    stacks.append(_extract_frames(end_frame, max_depth))
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/sampling_utils.py", line 114, in _extract_frames
    stack_entries = _extract_stack(stack, max_depth)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/sampling_utils.py", line 76, in _extract_stack
    _maybe_append_synthetic_frame(result, last_frame, last_frame_line_no)
  File "/usr/local/lib/python3.11/site-packages/codeguru_profiler_agent/sampling_utils.py", line 100, in _maybe_append_synthetic_frame
    line = linecache.getline(frame.f_code.co_filename, line_no).strip()
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/linecache.py", line 31, in getline
    if 1 <= lineno <= len(lines):
       ^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: '<=' not supported between instances of 'int' and 'NoneType'

It appears as if the issue occurs in sampling_utils.py: last_frame_line_no may become None and that case is unhandled.
Could this be due to changes to the traceback module in Python 3.11?

Thanks for the fix!


Google colab

Thanks for the interesting work,
Please provide a google colab solution.

I will prepare a colab notebook and share it

Hello, any updates on the colab notebook?  Would be greatly appreciated!

@ncarraz where you able to make that colab?

There is one but it is not a cleaned version

@ncarraz if you have a testable version , i can check it out , maybe try to clean it up if you can share it.


Geocoding tools

Start to integrate tools and techniques for translating between different representations of locations.
E.g.:

https://developers.google.com/maps/documentation/geocoding/start
http://www.openstreetmap.org

Dup of #111


[Commands] Fix a bad merge

@swift-ci smoke test

I am just going to skip the CI as I've tested this locally and it will block other Swift PRs.


Deactivate stubs depending on TYPO3 version

Hi,
looking at TYPO3 12.4, all stubs are rather outdated. Their purpose was to teach phpstan extbase generics but this is now fully covered by the core.
I know the tests do fail now but I am not 100% sure why. I guess it's because they rely on the stubs that are now loaded dynamically but I don't know how to adjust the tests accordingly. Maybe anyone with insights can help?
Thnaks in advance.
Relates: #133

@alexanderschnitzler Can we close this one or do we need to discuss this further?

Let's close this.


[Question/Suggestion] How to determine the number of training and evaluation iterations?

Hello,
Is there any rule to determine the number of training and evaluation iterations? I see from the source code of "Trainer" that 1 epoch continues until "train_iter" number of iterations are finished. Since the labelled and unlabelled data loaders do not contain same amount of data, how to determine the number of iteration? I would be appreciated if you could explain these parameters.
As a suggestion, it would be great if you could prepare and publish a docstring regarding parameters in config to clarify the goal of such parameters.
Thank you for making such library open to use! Best.

Also, I assume that epochs are not the standard epochs that we know, increasing epochs could lead to exception in data loader:
num_samples should be a positive integeral value, but got num_samples=0
Do epochs should be decided according to the number of the samples?

You are right. The epochs here is quite different from standard epochs we know.
We follows Google setting. You should set total number of iterations first (10240000 in our case), and then number of training iterations per epoch. Then you calculate the epochs based on these two numbers specified.


⚠ Bofrost.nl has degraded performance

In cac5bac, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 941 ms

Resolved: Bofrost.nl performance has improved in 07393fb.


Accommodation for pds4file, convert is_volume & is_volume_dir

Use Pds4FileTest/pds4/bundles as an example, the pds3 volume level should be the uranus_occ_u0_kao_91cm in our pds4 bundle uranus_occs_earthbased directory.

I agree that uranus_occ_u0_kao_91cm is one example of the PDS4 equivalent of what we called a "volume" in PDS3.


Add Twig convenience function for inline table JS

This is documented in changes to README.md. It adds the required inline JS asset that will render an existing table as a DataTable.

@hughbris  This is a good extension. Thank you. Sorry AGAIN for the delay. (I moved from Hong Kong to the UK in this period. Doesn't excuse the delay, but provides some context)


Missing numbers

Enter your question -
Given an array nums containing n distinct numbers in the range [0, n], return the only number in the range that is missing from the array.
Enter link to the question(if question belongs to any online platform) -
Missing numbers
Tags for the question(eg - Array, Basic, Stack, etc.) -
Array,Sorting

Done.

@RAshid602,
Kindly add your solution to "leetcode" folder.
Deadline - 10/10/2021


The questions in data_source.py

Hello,
Thanks for your great work!
I work on the gflownet-based project, and I wonder the role of following code block:
https://github.com/recursionpharma/gflownet/blob/trunk/src/gflownet/data/data_source.py [Line 254-256]
# Override the is_valid key in case the task made some objs invalid
for i in valid_idcs:
    trajs[i]["is_valid"] = True

Can I know why this code block does not mask the invalid objectives' trajectories as follows?
# Override the is_valid key in case the task made some objs invalid
for i, traj in enumerate(trajs):
    if i not in valid_idcs:
        traj["is_valid"] = False

Best regards,
Seonghwan Seo

Hi Seonghwan, I think you may be right, the True should be a False 🤔
Basically there are two sources of invalidity, the sampling method (e.g. an illegal action is taken) and the task itself. In practice, we've mostly managed to create action masks so that this doesn't happen, and the tasks we use typically don't decide that objects are invalid, so this issue may have gone unnoticed!
I don't have time to test & fix this now so leaving this open. If you have the time, a PR is welcome!


chinese doc translation

Q
A

Is bugfix?
yes/no

New feature?
yes/no

Breaks BC?
yes/no

Tests pass?
yes/no

Fixed issues
comma-separated list of tickets # fixed by the PR, if any

Merged. Thanks!


add Google's jsapi to dashboard.html

This fixes Uncaught TypeError: window.google.load is not a function.

Thanks, @thijskramer! Do you know why this wasn't necessary for me? It isn't used in Google's embed demos, as far as I can tell:
https://ga-dev-tools.appspot.com/embed-api/basic-dashboard/

Your question made me look really good at it, and I just discovered that this error is caused by wagtailgmaps. Wagtailgmaps overrides admin_base.html and includes <script src="https://maps.google.com/maps/api/js"></script>. This inclusion breaks it for Wagalytics probably.
I also discovered that when I put 'wagtailgmaps' below 'wagalytics' in the INSTALLED_APPS section of django, the error doesn't appear. So I'm not sure whether this PR should be merged or not...

Thank you for your thorough investigations, @thijskramer. This would have taken me a long time to work out! On balance, I think we should not merge this PR, but update the README to reference your findings.

I've updated my PR to conditionally load the jsapi if it doesn't exist. After some searching I found out that I'm not the only one (for example this SO question) with a window.google.load is not a function error when trying to implement the Google Analytics Embed API.  And it turns out other people solve this problem with including jsapi as well. So maybe this is the correct way of solving it?

This looks good to me, @thijskramer. Thanks!


docker stack deploy - no such image

Description
I have a swarm cluster (1 manager, 2 workers), when trying to deploy a stack to the swarm using a private registry, the image is not being pulled even with auth credentials being present on each node.
Steps to reproduce the issue:

Create swarm consisting of 1 manager, 2 works + external private registry (not in the swarm)
Deploy stack to swarm using an image from the private registry

Describe the results you received:
The image cannot be found
docker stack deploy --compose-file swarm.yml stack
docker stack ps stack
ID            NAME                IMAGE                                                             NODE          DESIRED STATE  CURRENT STATE           ERROR                             PORTS
7pcl8hwo57ax  stack_consul.1      private-registry.com/consul:latest  swarm-master  Ready          Rejected 4 seconds ago  "No such image: private-regist..."
kdt9jithrmmz   \_ stack_consul.1  private-registry.com/consul:latest  swarm-master  Shutdown       Rejected 4 seconds ago  "No such image: private-regist..."

I have also tried using the --with-registry-auth option when deploying the stack, which has the same result.
version: '3'

services:
  consul:
    image: private-registry.com/consul:latest
    command: ["agent", "-server", "-bind=<IP_ADDRESS>", "-bootstrap-expect=3", "-ui", "-client", "<IP_ADDRESS>", "-dns-port=53", "-log-level=info", "-recursor=<IP_ADDRESS>", "-config-dir=/consul/config", "-data-dir=/consul/data"]
    hostname: consul
    ports:
      - 8500:8500
    environment:
      - CONSUL_ALLOW_PRIVILEGED_PORTS=
    volumes:
      - ./consul/config:/consul/config
      - ./consul/data:/consul/data
    deploy:
      mode: replicated
      replicas: 1
      placement:
        constraints:
          - node.role == manager

Describe the results you expected:
Each node should pull the image and start the container.
Output of docker version:
docker version
Client:
 Version:      1.13.1
 API version:  1.26
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 08:47:51 2017
 OS/Arch:      linux/amd64

Server:
 Version:      1.13.1
 API version:  1.26 (minimum version 1.12)
 Go version:   go1.7.5
 Git commit:   092cba3
 Built:        Wed Feb  8 08:47:51 2017
 OS/Arch:      linux/amd64
 Experimental: false

Output of docker info:
docker info
Containers: 0
 Running: 0
 Paused: 0
 Stopped: 0
Images: 1
Server Version: 1.13.1
Storage Driver: aufs
 Root Dir: /mnt/sda1/var/lib/docker/aufs
 Backing Filesystem: extfs
 Dirs: 3
 Dirperm1 Supported: true
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: bridge host macvlan null overlay
Swarm: active
 NodeID: opxx0ia4dd40msfgn74iij8dt
 Is Manager: true
 ClusterID: 2pkudbtmrrvpvakz7vvhd6zw2
 Managers: 1
 Nodes: 3
 Orchestration:
  Task History Retention Limit: 5
 Raft:
  Snapshot Interval: 10000
  Number of Old Snapshots to Retain: 0
  Heartbeat Tick: 1
  Election Tick: 3
 Dispatcher:
  Heartbeat Period: 5 seconds
 CA Configuration:
  Expiry Duration: 3 months
 Node Address: <IP_ADDRESS>
 Manager Addresses:
  <IP_ADDRESS>:2377
Runtimes: runc
Default Runtime: runc
Init Binary: docker-init
containerd version: aa8187dbd3b7ad67d8e5e3a15115d3eef43a7ed1
runc version: 9df8b306d01f59d3a8029be411de015b7304dd8f
init version: 949e6fa
Security Options:
 seccomp
  Profile: default
Kernel Version: 4.4.47-boot2docker
Operating System: Boot2Docker 1.13.1 (TCL 7.2); HEAD : b7f6033 - Wed Feb  8 20:31:48 UTC 2017
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 995.8 MiB
Name: swarm-master
ID: KH4O:2FID:TNEX:4WIA:2MVD:XZYL:WXCZ:7K2A:KBBP:7PPN:OKGO:S4TV
Docker Root Dir: /mnt/sda1/var/lib/docker
Debug Mode (client): false
Debug Mode (server): true
 File Descriptors: 80
 Goroutines: 249
 System Time: 2017-02-19T21:02:47.168179173Z
 EventsListeners: 0
Registry: https://index.docker.io/v1/
Labels:
 provider=virtualbox
Experimental: false
Insecure Registries:
 <IP_ADDRESS>/8
Live Restore Enabled: false

docker-machine ls
NAME           ACTIVE   DRIVER       STATE     URL                       SWARM   DOCKER    ERRORS
node-01        -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.13.1
node-02        -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.13.1
swarm-master   -        virtualbox   Running   tcp://<IP_ADDRESS>:2376           v1.13.1

docker-machine --version
docker-machine version 0.9.0, build 15fd4c7

docker-compose --version
docker-compose version 1.11.1, build 7c5d5e4

Additional environment details (AWS, VirtualBox, physical, etc.):

If you log in to the node, and manually docker pull and image from that registry, does it work?
the "private-registry.com" name; does an entry for that domain exist in the DNS that's set on the daemon (or does it need an /etc/hosts file)?
if you run the daemon with debug enabled, do the log files show anything useful when pulling the image fails?

Yes, if I "pre-pull" the images on each node, then the stack gets deployed successfully.
It's a valid FQDN, adding hosts entry should not be required but I can try when I am at work again tomorrow.
Same as above, will try when at work tomorrow.

Our private registry was running 2.4.1, upgrading to 2.6.0 fixed this problem.

Still have this problem, docker-compose-yml

version: "3"
services:
web:
# replace username/repo:tag with your name and image details
image: nginx
deploy:
replicas: 5
resources:
limits:
cpus: "0.1"
memory: 50M
restart_policy:
condition: on-failure
ports:
- "80:80"
networks:
- webnet
networks:
webnet:

after running "sudo docker stack deploy -c docker-compose.yml st-0801-1417"
sudo docker stack ps st-0801-1417

ID                  NAME                     IMAGE               NODE                DESIRED STATE       CURRENT STATE            ERROR                           PORTS
imcuq9ylr5qq        st-0801-1417_web.1       nginx               ma                  Running             Running 3 minutes ago
jh9y5vkz22yx         _ st-0801-1417_web.1   nginx               laptop-138          Shutdown            Rejected 2 minutes ago   "No such image: nginx:latest"
2t6uxaaqwksv         _ st-0801-1417_web.1   nginx               laptop-138          Shutdown            Rejected 2 minutes ago   "No such image: nginx:latest"
9qz6efcr57q7        st-0801-1417_web.2       nginx               ma                  Running             Running 3 minutes ago
rlffnlts210q        st-0801-1417_web.3       nginx               ma                  Running             Running 3 minutes ago
wipjojcckla3         _ st-0801-1417_web.3   nginx               laptop-138          Shutdown            Rejected 2 minutes ago   "No such image: nginx:latest"
gc0k18l01w8y         _ st-0801-1417_web.3   nginx               laptop-138          Shutdown            Rejected 2 minutes ago   "No such image: nginx:latest"
3p7bhprvj8hq        st-0801-1417_web.4       nginx               ma                  Running             Running 3 minutes ago
s88t2hjnizqu        st-0801-1417_web.5       nginx               ma                  Running             Running 3 minutes ago
ybqr3xm2ez4a         _ st-0801-1417_web.5   nginx               laptop-138          Shutdown            Rejected 2 minutes ago   "No such image: nginx:latest"

@massoudAfrashteh you're commenting on an issue that was resolved; if you suspect there's a bug, please open a new issue with details (as requested in the issue template)

I will leave a comment for people that might encounter this issue,
I needed to add the flag --with-registry-auth
docker stack deploy --with-registry-auth -c docker-compose.yml servicename

Check out this page:
https://docs.docker.com/engine/reference/commandline/login/#credentials-store

even with --with-registry-auth flag, it won't work for me. the worker node won't run the service with "No such image: xxxx"

@koo9 is the image you're trying to use pushed to a registry, and is that registry accessible for the manager node and nodes in your cluster?

@thaJeztah I can manually pull the images from the manager node. it was working before for so long. it stopped working all of a sudden. I noticed that there are other issues with the swarm. the sql server suddenly won't be able to create new database on the volume. I wonder if it's the permission issue.

turns out that the worker node ran out of space. login from azure portal to clean up the images to free up the space. now it's working again.

I suffered with this problem for a long time, and everything was because of the old version of the docker.


X startup failure: "AddScreen/ScreenInit failed for driver 1"

$ sudo edit-chroot -all
name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20170315143304~master:95589555
release: xenial
architecture: armhf
xmethod: xorg
targets: lxde,cli-extra
host: version 9000.91.0 (Official Build) stable-channel daisy
kernel: Linux localhost 3.8.11 #1 SMP Wed Feb 22 23:16:02 PST 2017 armv7l armv7l armv7l GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Please describe your issue:
Crouton appears to work OK on Samsung XE303C12 chromebook. Everything installs and additional software can be installed and run via sudo apt install .... Crouton and shell can be exited safely and reopened without any issues. However it would seem that upon rebooting the chromebook, typing sudo startlxde fails to start X anymore. Sleep may also cause this issue - at first I thought this was random, but it seems reproducible on reboot. I would test this more but it takes a VERY long time to reinstall crouton.
I have reproduced this issue with lxde, xfce and unity.
Updating crouton with sudo sh ~/Downloads/crouton -n xenial -u does not fix the issue.
This is an example of the output:
$ sudo startlxde
Entering /mnt/stateful_partition/crouton/chroots/xenial...

_XSERVTransmkdir: Owner of /tmp/.X11-unix should be set to root

X.Org X Server 1.18.4
Release Date: 2016-07-19
X Protocol Version 11, Revision 0
Build Operating System: Linux 4.4.0-59-generic armv7l Ubuntu
Current Operating System: Linux localhost 3.8.11 #1 SMP Wed Feb 22 23:16:02 PST 2017 armv7l
Kernel command line: cros_secure console= console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 2539520 verity payload=PARTUUID=d6b9cbba-f8c7-f54f-a3f1-b503bc9fdeb8/PARTNROFF=1 hashtree=PARTUUID=d6b9cbba-f8c7-f54f-a3f1-b503bc9fdeb8/PARTNROFF=1 hashstart=2539520 alg=sha1 root_hexdigest=0f8e46d1190f0b380a736dcc0160f1b5d0787ee9 salt=11b3c6f73efe039531fac6ea08ccfbea76bc7f4476fc063a443c9bdc614ba489" noinitrd vt.global_cursor_default=0 kern_guid=d6b9cbba-f8c7-f54f-a3f1-b503bc9fdeb8
Build Date: 26 January 2017  12:27:19AM
xorg-server 2:1.18.4-1ubuntu6.1~16.04.1 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.33.6
        Before reporting problems, check http://wiki.x.org
        to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
        (++) from command line, (!!) notice, (II) informational,
        (WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Thu Mar 16 13:35:41 2017
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
gbm: failed to open any driver (search paths /usr/lib/arm-linux-gnueabihf/dri:${ORIGIN}/dri:/usr/lib/dri)
gbm: Last dlopen error: /usr/lib/dri/exynos_dri.so: cannot open shared object file: No such file or directory
failed to load driver: exynos
EGL_MESA_drm_image required.
(EE)
Fatal server error:
(EE) AddScreen/ScreenInit failed for driver 1
(EE)
(EE)
Please consult the The X.Org Foundation support
         at http://wiki.x.org
 for help.
(EE) Please also check the log file at "/tmp/Xorg.crouton.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.

/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

The CLI seems to work fine though if I start the "xenial" chroot up using sudo startcli.
If known, describe the steps to reproduce the issue:

install crouton (sudo sh /downloads/crouton -t lxde,cli-extra)
run crouton OK (sudo startlxde)
logout of lxde, exit crouton, exit shell, close tab
Reboot device
Go to terminal (Ctrl+Alt+t)
Enter shell (type shell)
Attempt to start X (type sudo sudo startlxde)

Try a different releases like trusty 14.04 or precise 12.04. xenial 16.04 might still be a little buggy.

Same experience on my Samsung Chromebook Plus.

same experience, New xenial install, crouton updated.
chronos@localhost / $ sudo startxfce4
Password:
Enter encryption passphrase for xenial:
Entering /mnt/stateful_partition/crouton/chroots/xenial...
/usr/bin/startxfce4: Starting X server
_XSERVTransmkdir: Owner of /tmp/.X11-unix should be set to root
X.Org X Server 1.18.4
gbm: failed to open any driver (search paths /usr/lib/arm-linux-gnueabihf/dri:${ORIGIN}/dri:/usr/lib/dri)
gbm: Last dlopen error: /usr/lib/dri/exynos_dri.so: cannot open shared object file: No such file or directory
failed to load driver: exynos
EGL_MESA_drm_image required.
(EE)
Fatal server error:
(EE) AddScreen/ScreenInit failed for driver 1
(EE)
(EE)
Please consult the The X.Org Foundation support
at http://wiki.x.org
for help.
(EE) Please also check the log file at "/tmp/Xorg.crouton.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Hi, I can confirm this startxfce4 error with a new crouton, xenial install on a Samsung Series 3.

Same issue on Samsung 303c

Experiencing same issue on Samsung 303c as @danvocino

I'm also having this issue on a 303c.

I am having the same issue on a Samsung Series 3 as well...

I tried deleting the chroot by resetting the computer, and then creating a new one. I made sure to install both extension and xiwi. (However, I wanted to be able to use xorg specifically, and it was using xorg instead of using xiwi that seemed to cause the problem. I never had the problem with xiwi.)
Now if I start xfce like this:
sudo startxfce4, -t xorg
it does actually work. I don't know if simply retrying it was what solved the problem, or if it was making sure to install extension. I would suggest simply trying to install extension. If that doesn't work, try xiwi. This way you can at least run xfce in a chrome tab. IAC the problem is resolved for me. (Unfortunately, I've found xfce is a lot slower if I run it on xorg than if I run it on xiwi, but this is probably a different issue.) I'm still a newbie, so I hope someone here finds this helpful. : /

@cricketbox5000  can you tell me from stat what to download? I did download the regular xfce using the command sudo ~/downloads/crouton -t xfce and what extensions should I add to download xiwi and org? I already have my laptop on dev mod and powerwashed it. Please tell me the command because I'm a total noob haha. Thanks in advance.

@asadislove What I did was I added xfce, xiwi and extension. I'm a total noob too, so I don't remember exactly what specific command I wrote when I installed my chroot. Sorry about that. I believe it was as follows:
sudo sh ~/Downloads/crouton -t xfce,xiwi,extension
or possibly:
sudo sh ~/Downloads/crouton -t xfce,xorg,xiwi,extension
As far as I know, the first command should work fine (barring syntax errors?). I would recommend doing more research on your own before following my advice, and if anyone out there is reading this and knows more than I do, I'd love to hear your input.
Both xorg and xiwi are ways of displaying a graphical user interface like xfce (1)(2). Xorg is automatically included when you install xenial, so you don't need to worry about adding it when you first create the chroot. Xiwi simply runs xfce inside a new Chrome tab. I've found that xorg runs much slower than xiwi.
Extension is needed to run the Crouton Integration Extension for Chrome. I don't know what it does exactly, but I believe that it works with either xiwi or xorg to integrate Chrome with xenial. Normally extension is installed with xiwi automatically. I think that if you run xorg instead of xiwi, you need to install extension separately from xiwi. This is because before I did that, if I tried to open the browser in xfce I would instead get a new tab in Chrome. This is not supposed to happen. Xfce has its own browser. I'm sure that there is also a way of adding these packages after installing the chroot, but I haven't been able to do that successfully yet.
The command I wrote in my above comment (sudo startxfce4, -t xorg) is to launch xfce using xorg when entering the chroot (when you also have xiwi installed). Alternatively, if you want to launch xfce using xiwi, use:
sudo startxfce4, -t xiwi
Thanks for your patience.

To those having trouble with launching your desktop target after a reboot I'd like to suggest doing a backup of the chroot after it's setup and tweaked and before you reboot. This will save you from re-installing or more accurately installing a new chroot each time something goes awry.  You can either first delete the hosed chroot or restore it over the hosed chroot with the edit-chroot command.
For instance to back up a chroot named xenial use this:
sudo edit-chroot -b -f ~/Downloads xenial
To restore a chroot named xenial on top of an old hosed one use this:
sudo edit-chroot -rr -f ~/Downloads/xenial-2017*.tar.gz
I used the '*' wildcard character to find any xenial backup created this year but you can use it's exact name if you have more than one.
Hope this helps,
-DennisL

@cricketbox5000,
Both of your chroot install commands above should work, I always use the second form that installs both xmethods to give more flexibility, for example -
sudo sh ~/Downloads/crouton -t xfce,xorg,xiwi
And the 'crouton integration' extension is needed when you install the 'xiwi' target/xmethod and installs the 'extension' target automatically like you mentioned but can also be used with the 'xorg' target/xmethod for copy pasta but you do have to specify it on the command line, again like you mentioned.
The target/xmethod you enter first in the install command will become the default xmethod so in your example it will be 'xorg'. If you want to change the default xmethod say from 'xorg' to 'xiwi' just update your chroot specifying only the new default xmethod like this for example -
sudo sh ~/Downloads/crouton -n xenial -t xiwi -u
With both xmethods installed you can launch your desktop in a number of ways -

sudo startxfce4 (that will launch in a separate screen via 'xorg', the default)

sudo startxfce4 -X xiwi (that will launch in a window via 'xiwi')

sudo startxfce4 -X xiwi-fullscreen (that will launch fullscreen via 'xiwi')

sudo startxfce4 -X xiwi-tab (that will launch in a browser tab via 'xiwi' - my favorite)

Hope this helps,
-DennisL

@DennisLfromGA
Thank you very much. All of your suggestions are very helpful. : )

While xenial (16.04lts) was unstable for me, trusty (Ubuntu 14.04lts) has been stable for the last month.
To install:
sudo sh ~/Downloads/crouton -n trusty -u -t xiwi,xfce
I would now label this issue as a regression, since 14.04 works well.

I installed xenial and experienced same issue on Samsung Chromebook 2 503.
#2426 fixed this issue for me.
Current version of pixman: 0.33.6
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Sun May 14 20:15:40 2017
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
gbm: failed to open any driver (search paths /usr/lib/arm-linux-gnueabihf/dri:${ORIGIN}/dri:/usr/lib/dri)
gbm: Last dlopen error: /usr/lib/dri/exynos_dri.so: cannot open shared object file: No such file or directory
failed to load driver: exynos
EGL_MESA_drm_image required.
(EE)
Fatal server error:
(EE) AddScreen/ScreenInit failed for driver 1
(EE)
(EE)
Please consult the The X.Org Foundation support
at http://wiki.x.org
for help.
(EE) Please also check the log file at "/tmp/Xorg.crouton.1.log" for additional information.
(EE)
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

@wowSH, what part of #2426 helped you?

Below answer helped me to fix this issue.
tista500 commented on Feb 8, 2016
OK.
Download this file to your "Downloads" directory in ChromeOS:
20-armsoc.txt
In crosh window, run shell command to launch bash-like shell
Rename downloaded file from .txt to conf:
cd ~/Downloads
mv 20-armsoc{.txt,.conf}
Login into chroot without X11:
sudo enter-chroot -n precise
Move that file into proper place and flush the RAM cache:
sudo mv ~/Downloads/20-armsoc.conf /usr/share/X11/xorg.conf.d/
sync
Logout chroot with "Ctrl + d" or run exit command
Re-login into chroot with X11/freon:
sudo startxfce4
That's all, maybe...
Regards.


Publish to Rubygems

Hello,
Are there any plans to publish this gem to Rubygems?
Thank you!

There is no plan for now.
We may implement https://github.com/red-data-tools/activerecord-adbc-adapter instead.
Because we can use DuckDB via ADBC because DuckDB supports ADBC.


Long pressing the control on an iPhone

Long pressing the control on an iPhone will have a WebView like selection effect, affecting the normal experience


Create widget_driver_test tests

Description

Create unit tests for the widget_driver_test package, improving code coverage.
close #111

Type of Change

[ ] 🚀 New feature (non-breaking change)
[ ] 🛠 Bug fix (non-breaking change)
[ ] ⚠ Breaking change (feature or bug fix which breaks existing behaviors/APIs)
[ ] 🏗 Code refactor
[ ] ⚙ Build configuration change
[ ] 📝 Documentation
[x] 🧹 Chore / Housekeeping

Hey @JanMichaelPeter @dominik089 @ebajrami @4ndrey , could we get some 👀 on this PR?
Thank you :D

@JanMichaelPeter @robert-northmind Thanks! I believe there is the need to add the label "ready for pipeline". I don't have permission on the repository

@robert-northmind Thank you! I don't have permission to merge too


FileStorage perhaps has incorrect typing metadata

Describe how to replicate the bug.
from flask import request
# snip some app-specific logic...
fileitem = request.files["file"]
blob = fileitem.read()

and then:
mypy test.py
test.py:3: error: "FileStorage" has no attribute "read"

Environment:

Python version: 3.9.5
Flask version: 2.0.1

hey, I think this more of werkzeug issue than flask since the request module is a part of werkzeug.
Can any of the devs back me up on that so I know I'm on the right track ?

I am not a dev but as far as I know and also according to the docs it doesn't have a read method, only close and save

It proxies to all methods of the underlying stream. If mypy is showing an error then it's wrong, FileStorage defines __getattr__ so any name should be allowed.


DynResidue exponentiation of larger Uints

I need to raise a DynResidue<U4096::LIMBS> by a U4352. There is currently no way for me to do that.
What I could do is:
impl Pow<U4352> for DynResidue<U4096::LIMBS> {
    fn pow(&self, exponent: &U4352) -> Self {
        let (exponent_hi, exponent_lo): (U4096, U4096) = <U4096 as Concat>::Output::from(exponent).split();

        let lo = self.pow(&exponent_lo);
        let hi = (self.pow(&U4096::MAX) * self).pow_bounded_exp(&exponent_hi, U4352::BITS - U4096::BITS);

        lo * hi
    }
}

Or, in a more generalized form:
macro_rules! impl_pow_cross_sizes {
    (($first_type:ident, $first_bits:expr), ($(($second_type:ident, $second_bits:expr)),+ $(,)?)) => {
        $(
            impl Pow<$second_type> for DynResidue<{nlimbs!($first_bits)}> {
                fn pow(&self, exponent: &$second_type) -> DynResidue<{nlimbs!($first_bits)}> {
                    let mut i = 0;
                    let mut shifted_base = self.clone();
                    let mut res = DynResidue::<{nlimbs!($first_bits)}>::one(self.params().clone());

                    while i < $second_bits / $first_bits {
                        let mut limbs = [Limb::ZERO; nlimbs!($first_bits)];
                        let mut j = 0;

                        while j < nlimbs!($first_bits) {
                            limbs[j] = exponent.limbs[i*nlimbs!($first_bits) + j];
                            j += 1;
                        }

                        let exponent_i: $first_type = Uint { limbs }; // The ith part of the exponent
                        res *= shifted_base.pow(&exponent_i);
                        shifted_base = shifted_base.pow(&$first_type::MAX) * shifted_base; // Shift by Self::bits

                        i += 1;
                    }

                    let mut limbs = [Limb::ZERO; nlimbs!($second_bits % $first_bits)];
                    let mut j = 0;

                    while j < nlimbs!($second_bits % $first_bits) {
                        limbs[j] = exponent.limbs[i*nlimbs!($first_bits) + j];
                        j += 1;
                    }

                    let part: $first_type = Uint { limbs }.into();

                    res *= shifted_base.pow_bounded_exp(
                        &part,
                        $second_bits % $first_bits,
                    );

                    res
               }
            }
        )+
    };
}

Now, this should be a working approach and I could do a PR for this.
However, it feels non-optimal; I'd have wanted the exponentiation (i.e. Montgomery reduction) algorithm to factor in the different sizes instead of wrapping around the currently existing API (similarly to what I did with mul_wide). But I don't know how to do that, or whether it could be done.
What approach should be taken here?

I think the general solution to all of these "cross size" problems is to write the impls generically so different sized values can be supported, rather than trying to generate a combinatorial explosion of size-by-size impls

I think the general solution to all of these "cross size" problems is to write the impls generically so different sized values can be supported, rather than trying to generate a combinatorial explosion of size-by-size impls with macros

How do you suggest doing so? LIke I did with mul_wide? can that be done with pow?

Yes, it might require generic_const_exprs. I'm not sure. You'd have to try it.

Yes, it might require generic_const_exprs. I'm not sure. You'd have to try it.

I have tried it and it required it.
So what's the game plan here? I've gotten extra-trait-impls to a mergable situation afaik, I implemented all your requests - are we going to merge?
And if so, should I also create a similar PR for Pow?

Give me some time to re-review the PRs

Give me some time to re-review the PRs

Of course. Should I open a similar one for Pow, or should we delay this discussion?

I have tried it and it required it.

Also what did you try and why was it required?

I tried to implement Concat between sizes in a generic way, i.e. with an additional RHS_LIMBS const generic parameter, and it failed due to the missing generic_const_exprs.
We had a discussion about it in the extra-trait-impls PR, and I explained it there .

If you can make the generic implementation work by adding a nightly feature, I guess that would be OK.
I'm not sure how stable that's going to be though.

If you can make the generic implementation work by adding a nightly feature, I guess that would be OK.
I'm not sure how stable that's going to be though.

No I can't use unstable Rust. My project is security-oriented and I can't risk that


[IDEA] Code Of Conduct

Suggestion(IDEAS)
Describe the IDEA
Add Code Of Conduct
A small survey- (optional)

1- Would you like to work on this idea?

[x] Yes
[ ] No

2- Is this idea an original idea or something you have seen elsewhere?

[ ] It is original.
[x] No, I saw this somewhere.
[ ] Prefer not to say.

3- If you have seen this idea somewhere, can you state where?

Various opensource projects.
4- How much would you rate the current version of the bot?

[ ] Amazing(5/5)

[ ] Good(4/5)

[x] Not Bad(3/5) - Have not used the bot.

[ ] Meh(2/5)

[ ] It succ(1/5)

[ ] Sad, depressing and makes me want to cry

Additional information

@all-contributors add @AllanRegush for docs, ideas


Permission targets only get partially applied

Describe the bug
When creating a permission target through Terraform the affected permissions only get partially applied with respect to X-Ray infos. I created a simple scenario with a Docker repository and a test user which only has read and annotate permissions on the respective repository. Meanwhile I pushed a hello-world image to this repository with another user. After logging in, the repository and its content are visible to the test user but the X-Ray tab is not.

Once I login as an administrator and simply save the created permission again without changing anything, the X-Ray tab is accessible to the test user.
I'm talking about the X-Ray tab under Artifactory -> Artifacts -> xray-permissions-test-repo -> hello-world -> manifest.json.
Requirements for and issue
resource "random_password" "password_tu" {
  length           = 30
  special          = true
  override_special = "_%@"
}

resource "artifactory_user" "test" {
  name     = "permission-test-user"
  email    =<EMAIL_ADDRESS>  password = random_password.password_tu.result
}

resource "artifactory_permission_target" "technical_user" {
  name = "permission-test"

  repo {
    repositories = [artifactory_local_repository.test_repo.key]
    actions {
      users {
        name        = artifactory_user.test.name
        permissions = ["read", "annotate"]
      }
    }
  }
}

resource "artifactory_local_repository" "test_repo" {
  key             = "xray-permissions-test-repo"
  package_type    = "docker"
  repo_layout_ref = "simple-default"
  xray_index      = "true"
}

JFrog cloud platform
Terraform v0.14.5

Expected behavior
Permissions working like set.
Additional infos
The system logs don't show any more infos which could lead to the source of the issue.

Thanks. I will try to have a look at this today

I am trying to reproduce this at the moment - sorry to take so long

Also, you didn't include what version of the provider you are using.

I am unable to reproduce this. I do exactly as stated:
Use the before TF to create the resources:
# Required for Terraform 0.13 and up (https://www.terraform.io/upgrade-guides/0-13.html)
terraform {
  required_providers {
    artifactory = {
      source  = "registry.terraform.io/jfrog/artifactory"
      version = "2.2.7"
    }
  }
}
provider "artifactory" {
  url = "https://cbongiorno-test-rt.jfrog.tech"
  username = "admin"
  password = "xxxx"
}
resource "random_password" "password_tu" {
  length           = 30
  special          = true
  override_special = "_%@"
}

resource "artifactory_user" "test" {
  name     = "permission-test-user"
  email    =<EMAIL_ADDRESS>  password = random_password.password_tu.result
}

resource "artifactory_permission_target" "technical_user" {
  name = "permission-test"

  repo {
    repositories = [artifactory_local_repository.test_repo.key]
    actions {
      users {
        name        = artifactory_user.test.name
        permissions = ["read", "annotate"]
      }
    }
  }
}

resource "artifactory_local_repository" "test_repo" {
  key             = "xray-permissions-test-repo"
  package_type    = "docker"
  repo_layout_ref = "simple-default"
  xray_index      = "true"
}

push an image to the new repo:
christianb@unifi terraform-provider-artifactory % docker pull hello-world
Using default tag: latest
latest: Pulling from library/hello-world
b8dfde127a29: Pull complete
Digest: sha256:308866a43596e83578c7dfa15e27a73011bdd402185a84c5cd7f32a88b501a24
Status: Downloaded newer image for hello-world:latest
docker.io/library/hello-world:latest
christianb@unifi terraform-provider-artifactory % docker tag hello-world cbongiorno-test-rt.jfrog.tech/xray-permissions-test-repo/hello-world
christianb@unifi terraform-provider-artifactory % docker push cbongiorno-test-rt.jfrog.tech/xray-permissions-test-repo/hello-world
Using default tag: latest
The push refers to repository [cbongiorno-test-rt.jfrog.tech/xray-permissions-test-repo/hello-world]
f22b99068db9: Pushed
latest: digest: sha256:1b26826f602946860c279fce658f31050cff2c596583af237d971f4629b57792 size: 525

And now, login with the test user and go where you say to go:

you can see from the image, I have no issue accessing it.
christianb@unifi terraform-provider-artifactory % curl -snL https://cbongiorno-test-rt.jfrog.tech/artifactory/api/system/version/ | jq -re .version
7.16.3
christianb@unifi terraform-provider-artifactory % terraform -version
Terraform v0.14.7
+ provider registry.terraform.io/hashicorp/random v3.1.0
+ provider registry.terraform.io/jfrog/artifactory v2.2.7

Your version of Terraform is out of date! The latest version
is 0.14.8. You can update by downloading from https://www.terraform.io/downloads.html

Note: I pushed the docker image and ran the terraform script as admin, but accessed it only with the new user in incognito mode.

Sorry I forgot to mention the version of the provider. I was using 2.2.4.
I will give it a try today with 2.2.7.  Thank you very much for your efforts so far.

So that didn't make a difference, it resulted in exactly the same behavior. I noticed I can still open the tab while the scan hasn't been done yet. Once it finishes and I try to open it, I get the unauthorized. If I go into the UI as an admin and save the permission target without changing anything, it works afterwards.
I also checked again the effective permissions and there's just this one permission target. Anything else I could try or check?

I will involve one of our product people and see if he can give some insight. This might be a bug with RT itself or with your installation

Ok, I have tried it on our public partnership portal:
christianb@unifi terraform-provider-artifactory % curl -snL https://partnership.jfrog.io/artifactory/api/system/version/ | jq -re '.version // .errors'
7.15.3

# Required for Terraform 0.13 and up (https://www.terraform.io/upgrade-guides/0-13.html)
terraform {
  required_providers {
    artifactory = {
      source  = "registry.terraform.io/jfrog/artifactory"
      version = "2.2.7"
    }
  }
}
provider "artifactory" {
  url = "https://partnership.jfrog.io"
  username = "christianb"
  password = "yyyyy"
}

resource "artifactory_user" "test" {
  name     = "permission-test-user"
  email    =<EMAIL_ADDRESS>  password = "xxxxx"
}

resource "artifactory_permission_target" "technical_user" {
  name = "permission-test"

  repo {
    repositories = [artifactory_local_repository.test_repo.key]
    actions {
      users {
        name        = artifactory_user.test.name
        permissions = ["read", "annotate"]
      }
    }
  }
}

resource "artifactory_local_repository" "test_repo" {
  key             = "xray-permissions-test-repo"
  package_type    = "docker"
  repo_layout_ref = "simple-default"
  xray_index      = "true"
}

push hello-world docker image
christianb@unifi terraform-provider-artifactory % docker login partnership-xray-permissions-test-repo.jfrog.io
Username: christianb
Password:
Login Succeeded
christianb@unifi terraform-provider-artifactory % docker tag hello-world partnership-xray-permissions-test-repo.jfrog.io/hello-world
christianb@unifi terraform-provider-artifactory % docker push partnership-xray-permissions-test-repo.jfrog.io/hello-world
Using default tag: latest
The push refers to repository [partnership-xray-permissions-test-repo.jfrog.io/hello-world]
f22b99068db9: Pushed
latest: digest: sha256:1b26826f602946860c279fce658f31050cff2c596583af237d971f4629b57792 size: 525

and then, go check:

No issue (for some reason, I am not able to highlight the image as before)

I really think this is a problem with your RT instance and not terraform.  Please open a support ticket with the RT team and if they think it's TF related, then reopen or have them cut a ticket

Alright thank you very much for your help @chb0github


Bug report-azurerm_monitor_metric_alert does not resolve scopes by key for postgresql_flexible_server resource_type

Is there an existing issue for this?

[X] I have searched the existing issues

Community Note

Please vote on this issue by adding a :thumbsup: reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

global_settings = {
  default_region = "region1"
  regions = {
    region1 = "uksouth"
  }
}

resource_groups = {
  postgresql_region1 = {
    name   = "postgresql-region1"
    region = "region1"
  }
  security_region1 = {
    name = "security-region1"
  }
}

postgresql_flexible_servers = {
  primary_region1 = {
    name       = "primary-region1"
    region     = "region1"
    version    = "12"
    sku_name   = "MO_Standard_E4s_v3"
    storage_mb = 131072

    resource_group = {
      key = "postgresql_region1"
      # lz_key = ""                           # Set the lz_key if the resource group is remote.
    }

    # Auto-generated administrator credentials stored in azure keyvault when not set (recommended).
    # administrator_username  = "postgresqladmin"
    # administrator_password  = "ComplxP@ssw0rd!"
    keyvault = {
      key = "postgresql_region1" # (Required) when auto-generated administrator credentials needed.
      # lz_key      = ""                      # Set the lz_key if the keyvault is remote.
    }

    # [Optional] Firewall Rules
    postgresql_firewall_rules = {
      postgresql-firewall-rule1 = {
        name             = "postgresql-firewall-rule1"
        start_ip_address = "<IP_ADDRESS>"
        end_ip_address   = "<IP_ADDRESS>"
      }
      postgresql-firewall-rule2 = {
        name             = "postgresql-firewall-rule2"
        start_ip_address = "<IP_ADDRESS>"
        end_ip_address   = "<IP_ADDRESS>"
      }
    }

    # [Optional] Server Configurations
    postgresql_configurations = {
      backslash_quote = {
        name  = "backslash_quote"
        value = "on"
      }
      bgwriter_delay = {
        name  = "bgwriter_delay"
        value = "25"
      }
    }

    postgresql_databases = {
      postgresql_database = {
        name = "sampledb"
      }
    }

    tags = {
      segment = "sales"
    }

  }

}

# Store the postgresql_flexible_server administrator credentials into keyvault if the attribute keyvault{} block is defined.
keyvaults = {
  postgresql_region1 = {
    name                = "akv"
    resource_group_key  = "security_region1"
    sku_name            = "standard"
    soft_delete_enabled = true
    creation_policies = {
      logged_in_user = {
        secret_permissions = ["Set", "Get", "List", "Delete", "Purge"]
      }
    }
  }
}

monitor_metric_alert = {
  mysql-cpu-utilization = {
    name           = "mysql-cpu-utilization"
    resource_group = { key = "mysql_region1" }
    description    = "Action will be triggered when cpu utilization is greater than 90% in the last 30 min."
    severity       = 2
    frequency      = "PT15M"
    window_size    = "PT30M"

    criteria = {
      metric_namespace = "Microsoft.DBforMySQL/flexibleServers"
      metric_name      = "cpu_percent"
      aggregation      = "Average"
      operator         = "GreaterThan"
      threshold        = 90
    }

    scopes = {
      scope1 = {
        resource_type = "postgresql_flexible_server"
        key           = "primary_re1"
      }
    }

    action = {
      action_group = {
        key = "example"
      }
    }
  }
}

  with module.solution.module.monitor_metric_alert["mysql-cpu-utilization"].azurerm_monitor_metric_alert.mma,
  on /home/vsts_azpcontainer/pildev/modules/solution/modules/monitoring/monitor_metric_alert/module.tf line 14, in resource "azurerm_monitor_metric_alert" "mma":
  14:   scopes = try(flatten([
  15:     for key, value in var.settings.scopes : coalesce(
  16:       try(var.remote_objects[value.resource_type][value.lz_key][value.lz_key][value.key].id, null),
  17:       try(var.remote_objects[value.resource_type][var.client_config.landingzone_key][value.key].id, null),
  18:       try(value.id, null),
  19:       []
  20:     )
  21:   ]), [])

This is due to the fact that for postgresql_flexible_server, there is no id exposed. Its called postgresql_flexible_server_id
https://github.com/aztfmod/terraform-azurerm-caf/blob/18d145588b6d0bba38fef9db71870b52b94e9200/modules/databases/postgresql_flexible_server/output.tf
I havent checked for all the other resource types, but would worth looking
Version of the module you are using
main branch
Rover Version
aztfmod/rover:1.1.7-2203.2311
Terraform Version
1.1.7
AzureRM Provider Version
2.99
Affected Resource(s)/Data Source(s)
azurerm_monitor_metric_alert
Terraform Configuration Files
Not needed

Expected Behaviour
No response
Actual Behaviour
No response
Steps to Reproduce
No response
Important Factoids
No response
References
No response

Have you tried "postgresql_flexible_servers". You have to set the aztfmod variable name.

The resource type are pickup from:
https://github.com/aztfmod/terraform-azurerm-caf/blob/main/local.remote_objects.tf


Feature request: Option to use bash as shell interpreter for commands run in Windows agents.

Hook commands can be written as bash scripts and BuildKite agent will execute them via bash.exe (through a Git for Windows package installed in the machine) as long as they do not include an extension.
Would it be possible to allow the same for regular commands in pipelines' steps? It looks like the agent directly goes for CMD.exe  as the shell for commands in Windows.
Being able to use bash scripting makes life so much easier for those unfamiliar with Windows batch/cmd programs.

Hi @frodera 👋 The agent has a shell configuration (see https://buildkite.com/docs/agent/v3/configuration) which it will use to determine what shell to invoke commands with. You should be able to specify bash in there for windows!

Worked like a charm. Thanks @lox 👍


how to increase png resolution with altair.renderers.enable("png")

I'm using altair in a notebook, I tried
altair.renderers.enable("png", scale_factor=3.0)

it doesn't seem to have any effect on the resolution of the generated png. What's the right way to do so, please?

Facing the same. Installed using the conda method on macOS 11.2.3 running python 3.7 and altair 4.10 and vega-lite-cli 5.17.0. I also tried passing webdriver='firefox' and webdriver='chrome' and it made not impact. Additionally the axis tick labels are a much lighter style than the level visible on my browser (Safari using Jupyter Lab).

Try altair_saver.save(plot, "plot.png", vega_cli_options=["-s 4"]) where 4 is the scaling factor

specify the width and height using the properties I hope it helps.
altair_saver.save(bars.properties(width=800, height=400), "plot.png", vega_cli_options=["-s 4"])

This seems to work when using notebooks without having to use save() function:
alt.renderers.enable('altair_saver', fmts=['png'], embed_options={'scaleFactor': '4'})

I'm trying to install altair_saver, but both pip install altair-saver and pip install git+https://github.com/altair-viz/altair_saver/ fail (the latter says it's only available in python 2.7). Am I doing something wrong?

Since Altair 5.2, the functionality of Altair Saver is now available in Altair via the vl-convert package. Most of the functionality has been available since 5.0, and the main addition in 5.2 was PDF export. See the docs on how to save charts for more details. This includes how to set the resolution https://altair-viz.github.io/user_guide/saving_charts.html#png-figure-size-resolution
We are going to archive this repo, so I'm closing all the open issues and PRs before doing so. Try out the new options for saving charts mentioned above and if you run into issues, please open an issue directly in the altair or vl-convert repo.


github.com/glebarez/sqlite@v1.4.8/sqlite.go:101:22: cannot convert 0 (untyped int constant) to *int

Currently having these issues too
# github.com/glebarez/sqlite
/home/kqzz/go/pkg/mod/github.com/glebarez/sqlite@v1.4.8/sqlite.go:101:22: cannot convert 0 (untyped int constant) to *int
/home/kqzz/go/pkg/mod/github.com/glebarez/sqlite@v1.4.8/sqlite.go:102:24: cannot convert 0 (untyped int constant) to *int
/home/kqzz/go/pkg/mod/github.com/glebarez/sqlite@v1.4.8/sqlite.go:103:21: cannot use -1 (untyped int constant) as *int value in assignment
/home/kqzz/go/pkg/mod/github.com/glebarez/sqlite@v1.4.8/sqlite.go:106:39: cannot use limit.Limit (variable of type *int) as type int in argument to strconv.Itoa

What version of GORM are you using?

What version of GORM are you using?

gorm v1.24.0

This was fixed in v1.5.0 by merging the upstream (they already fixed this).
Please get latest version in your go.mod

Tysm!


[MNOE-1287] Delisted apps

Update tenant config schema to include only active apps

@ouranos Rebased on 4.0

@ouranos Updated specs, looks like they were failing due to updates in nokogiri or sanitize gems. Build was then failing due to bundler failures in travis caused by outdated rubygems. I've added a temp fix for this until it is fixed in travis (added gem update --system to before_install: in travis config).


Create artefact store object

Fixes #280. It just adds a tiny new object (based on Dict) for artefact_store and adds a property to access it.

Closing this for now, it was also based on the wrong branch :(


feat: allow IPv6 networkPrefix specification

Add support to allow different IPv6 interface identifier by same network prefix.
The problem this PR solves is as follows:
When a router used to fetch the hostname from (which should be the common way this middleware is used) the IPv6 address includes the full IP to the router (with interface identifier).
If a device from the network behind this router will access the protected website the network prefix will be the same but interface identifier differs.
Therefore we need some validation if the IPv6 address is within the same network.
With this change the user can specify an prefix (1-128) in which the network prefix will be accepted (subnetmask).
The default should be 64 which will commonly split network prefix from interface identifier:
  middlewares:
    ddns-allowlist-router:
      plugin:
        ddns-allowlist:
          allowedIPv6NetworkPrefix: 64

[x] add tests

closes #41

can not add end2end test with k8s because minikube does not support IPv6


IsGreaterThan works as isGreaterThanOrEqualto in case of date

Date lastReadEmailDate = processedEmailList.get(0).getEmailRecievedDate();
new SearchFilter.IsGreaterThan(ItemSchema.DateTimeReceived, lastReadEmailDate)
finditemresult includes the mail with greater and equal date.

Maybe the search uses more precision than the API can specify.
For example if you could only provide hour and minute precision (e.g. > 2017/09/14 00:00) and the message was received at 2017/09/14 00:00:01, it would appear equal according to your precision but would be > than in practice).

I met the same issue:
SimpleDateFormat formatter = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
Date startDate = formatter.parse("2017-10-08 11:15:38");
IsGreaterThan isGreaterThan = new SearchFilter.IsGreaterThan(ItemSchema.DateTimeReceived, startDate);

I have an email with DateTimeReceived is "2017-10-08 11:15:38" (long<PHONE_NUMBER>000). I use "IsGreaterThan" to search in my inbox. And I get this email. It is not greater, but equal. The result is same with "IsGreaterThanOrEqualTo".
Could someone help to resolve it?


[FLINK-27418][table-planner] fix topN retraction for previously delet…

What is the purpose of the change
Fixing the bug inside RetractableTopNFunction. The bug occurs when the record initially inserted inside topN, then it is dropped out from topN by another record and then deleted by retraction message from upstream operator. In such case the record remains inside the dataState with RowKind.Delete prefix.
Brief change log

added input row serializer to RetractableTopNFunction
added record copying while deleting it from topN
added specific test to RetractableTopNFunctionTest

Verifying this change
Extended unit tests for RetractableTopNFunction
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): yes (inside the RetractableTopNFunction)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

@lsyldliu Can you review this? Or should @wuchong do this?

@rovboyko Thanks for fixing it!  The change looks good to me and I've checked the other two places which also invoke the collectDelete without rownumber(UpdatableTopNFunction#L538 and AppendOnlyTopNFunction#L242), no problem there.
Although this fix adds a bit of serialization overhead, correctness is paramount, and I can't think of a better way to do copy on demand.
cc @wuchong if you have some time to double check,  it would be much appreciated.


Last comma of Object

TypeScript Version:  1.8.10 / Release
Code
// Compile target: es5
let obj = {
    a: 0,
    b: 1,
    c: 2,
};

Expected behavior:
var obj = {
    a: 0,
    b: 1,
    c: 2
};

Actual behavior:
var obj = {
    a: 0,
    b: 1,
    c: 2,
};

Isn't trailing comma valid in ES5?

This is legal in ES3 as well, though we'll still remove it if you target ES3 due to IE8.

I thought "Actual behavior" is legal in ES6/ES2015 not in ES5, although it works on modern browser. Some compression tools will throw an error when I compress this code var obj = { a: 0, b: 1, c: 2, }

No, it was codified in ES5 Section 11.1.5, where trailing commas in the object initialiser are expressly allowed.
I would say those compression tools need to change because they are not supporting the standards then.

@kitsonk thanks


spaces around operators

In section 1 at least
 x*x

most people write
x * x

and with
    n*factorial(n-1)

instead
n * factorial(n - 1)

a minor thing but I figured I'd mention it!

Yes, I generally prefer letting operators breathe as well

Lack of space after some commas probably applies as well.

Yes, I've also put space after the comma after the format string in println.  I'll keep the issue open since breathing is important ;)


wrap-gapps-hook.sh: only wrap links when required

Unless dontWrapGapps is set, the wrap-gapps-hook.sh will currently
wrap all executables (and symbolic links to executables) found under
the target directories: bin and libexec.
As a result, if a symbolic link in a target directory points to an
executable in a target directory, both will get wrapped.  This
causes an extra shell/exec when following the symbolic link,
as well as increasing the size of the final executable's environment.
To avoid wrapping a link to an already wrapped executable, this
commit splits the determination of what gets wrapped into two phases:

All binaries under the target directories are wrapped and logged
with "Wrapping program ..."

All links to executables under the target directories are
identified and checked to see if they reference an executable
under one of the target directories.
If yes, the required wrapping has already been performed on
the associated binary (in phase 1), so no wrapping is done
and "Not wrapping link: ... (already wrapped)" is logged.
If no, the link points at an executable that hasn't been
wrapped, so the link is wrapped and "Wrapping link: ..." is logged.

As an example, the yelp package has a bin directory that contains
an executable "yelp" and a symbolic link "gnome-help" -> "yelp".
Prior to this commit, the bin directory would contain these files
after wrapping:
  gnome-help          -- wrapper to exec .gnome-help-wrapped
  .gnome-help-wrapped -- a symbolic link to yelp
  yelp                -- wrapper to exec .yelp-wrapped
  .yelp-wrapped       -- the original yelp binary

After this commit, the bin directory will instead contain:
gnome-help          -- a symbolic link to yelp
 yelp                -- wrapper to exec .yelp-wrapped
 .yelp-wrapped       -- the original yelp binary

NOTE: The primary motivation for this commit is to avoid obscuring
the fact that two or more paths are simple aliases and expected to
behave identically. It also reduces the likelihood of hitting
limits related to environment variable size.
LIMITATION: The method used above is intended to be conservative
and will still wrap symbolic links to other symbolic links when
the ultimate target is outside of bin or libexec.

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

NOTE: Tested with nix-review. Used grep on logs to find packages with symbolic links that would no longer be wrapped after this commit. Here's a list:
package                         symbolic link -- no longer wrapped
===========================     ===========================================
cozy                            cozy-0.6.7/bin/cozy
emacs25                         emacs-25.3/bin/emacs
emacs                           emacs-26.1/bin/emacs
gksu                            gksu-2.0.2/bin/gksudo
gnome3.gedit                    gedit-3.30.2/bin/gnome-text-editor
gnome3.gucharmap                gucharmap-11.0.3/bin/charmap
gnome3.gucharmap                gucharmap-11.0.3/bin/gnome-character-map
gnome3.yelp                     yelp-3.30.0/bin/gnome-help
gnumeric                        gnumeric-1.12.44/bin/gnumeric
libreoffice-fresh-unwrapped     libreoffice-<IP_ADDRESS>/bin/libreoffice
libreoffice-still-unwrapped     libreoffice-<IP_ADDRESS>/bin/libreoffice
lumina                          lumina-1.4.0-p1/bin/lte
mate.mate-calc                  mate-calc-1.20.3/bin/mate-calculator
mate.mate-utils                 mate-utils-1.20.2/bin/mate-panel-screenshot
xfce.thunar-bare                thunar-build-1.6.10/bin/Thunar

sgtpuzzles                      [ 39 symbolic links not wrapped]

You can build any of these packages to test this commit. Also, there is a default.nix that can be used to test link wrapping more comprehensively here:
https://gist.github.com/tollb/4a21b890058ca3ab7d44618e345d0e28

Thank you.