Lab 04 - Errors when trying to create dynamic legend

Every time I try to run the document it keeps giving me the following error:
ERROR: Operation not allowed without an active reactive context
I went back to my code and saw that it was saying that Error: object input not found when I ran the following code:
# Reactive Object / Legend
one_team <- filter( .data = Teams,
                    name == input$my_team )
points( x = one_team$yearID,
        y = one_team$ave.so,
        type = "b",
        pch = 19,
        col = "darkorange4" )

I am confused as to what to assign for input. In the demo examples, it states:
When using Shiny, a global object called "input" will be created, and it stores the input values the user defines in slots created through the inputId arguments.
I did call out this by doing the following
selectInput( inputId = "my_team",
             label = "Choose a Team",
             choices = c("Cincinnati Reds",
                         "Pitsburgh Pirates",
                         "Philidelphia Phillies",
                         "St. Louis Cardinals",
                         "Chicago White Sox",
                         "Detroit Tigers",
                         "Chicago Cubs",
                         "Boston Red Sox",
                         "New York Yaknees",
                         "Cleveland Indians",
                         "Baltimore Orioles",
                         "Washington Senators",
                         "New York Giants",
                         "Philidelphia Athletics",
                         "St. Louis Browns",
                         "Los Angeles Dodgers",
                         "San Fransisco Giants",
                         "Minnesota Twins",
                         "New York Mets",
                         "Houston Astros",
                         "Atlanta Braves",
                         "Milwaukee Brewers",
                         "Oakland Athletics",
                         "Kansas City Royals",
                         "San Diego Padres"),
             selected = "Los Angeles Dodgers" )

Does anyone know how to work around this?

Hi @pharri14.
You're call to selectInput() looks fine to me, so long as it's in Column {.sidebar} using flexdashboard per the examples.
You're also using input$my_team correctly, so none of that needs adjustment.  The error:

ERROR: Operation not allowed without an active reactive context

...typically indicates that the location where you use input$my_team is not within a reactive function.  It can't simply be within a code chunk.  Try putting it inside renderPlot({}), within both parentheses and curly brackets.  Your call to filter() should precede any plotting functions.  For example:
renderPlot({

one_team <- filter(.data = Teams,
                   name == input$my_team)

plot.new()

# Your plot code with reactive value "one_team"

})

@jamisoncrawford thank you for that tip. I was able to get the error from popping up and to at least show the chart. However, it still says that Error: object input not found inside my Console. Do I need to create a new Shiny Web Application, like was illustrated in the Week 4: How does "Shiny" work? video?

@pharri14 the template should work fine unless you've made any alterations. This is a difficult error to diagnose without seeing the code.
Would you email me your .rmd file?

@pharri14 thanks for sending the .rmd file - posting here for others.
Any operation with the reactive variable input$my_team should occur within renderPlot({}) first, so you'll want to assign the output of filter() to one_team in order to prepare that for later use in the code.
Importantly, plot.new() should be the next step, then any code for the main plot and, lastly, you'll want to include the dynamic part of your plotting code (the code that uses the filtered data one_team).
After switching the order of these around, it functioned as intended - so I do hope that works for you.  I should note that I did not get any errors when I ran it initially, so hopefully you're not experiencing that, either.  Let me know if this works (and, if so, congrats on your first dashboard)!

@jamisoncrawford I made sure to put one_team inside of renderPlot ({}). When I run the one_team input line (as shown below) it produces the following error:
  one_team <- filter( .data = Teams,                        name == input$my_team )
Error in input$my_team : $ operator is invalid for atomic vectors
one_team still shows up in my Data Environment. I went back and watched the Lab 04 Part II video and saw that when you ran that line of code, no error occurred in your Console. For reference, here is a snippet of my code to show the order:
renderPlot({
  one_team <- filter( .data = Teams,
                      name == input$my_team )
  plot.new()
  plot.window( xlim = c(1900, 2012),
               ylim = c(0, 10),
               yaxs = "i" )
  par( mar = c(2, 2, 2, 4),
      mgp = c(2, 0.2, 0.2) )...................

@pharri14 are you knitting the file or running the code chunks individually?
Are you able to open the dashboard?

@jamisoncrawford the entire program will run and the dashboard opens. But, when I try to run that code chunk itself, it gives me that error.

@pharri14 that makes sense. The individual code chunks should throw an error if they contain input$my_team.
This is because input is not made into an object in your working environment using assignment (<-).  It's only made into a reactive object when it can work in concert with the code chunk where inputId =` specifies it - and for that to happen, the document must be knit in its entirety.
So in order to check the result of changes in code, you should knit the entire document, but don't worry about individual chunks.  If you shuffle around the code you have in your renderPlot() call and knit the whole dashboard each time, you will see the real changes your making, and all the code you need is already in your script.
Sorry for prematurely celebrating - does this help?

@jamisoncrawford that makes sense. However, no matter where I move the following code chunk around and run the document, no lines show up on the graph when a team is selected:
points( x = one_team$yearID,            y = one_team$ave.so,            type = "b",            pch = 16,            col = "darkorange4" )
lines( x = one_team$yearID,          y = one_team$ave.so,          col = "darkorange4",          lwd = 2 )
Am I messing something up with the order of other chunks of the code, maybe? Right now I have it ordered as:

renderPlot
one_team filter
plot.new + plot.window + par
I plot all of the points in gray as well as the average line in blue
I establish the X and Y-axis
I set up the title
I set up the legend
I insert all of the text on the chart to highlight specific points in gray
I have the dynamic plotting function

I have also tried to place the dynamic plotting function right before the section where I establish the X and Y-axis without success.

@pharri14 perhaps par() should come before plot.new() since it establishes graphical parameters.  This is the damnedest thing - would you mind resending your .rmd script?
We had an issue that neither Dr. Levy nor I could figure out for the life of us on this particular assignment last semester.  It turned out that argument inputId = had a lowercase "i" and since R is case sensitive, it didn't run.  Too us a hot minute to find that.  It might be something small like that.

@pharri14 I have one more suggestion.  Try creating a new script (Ctrl + N), then copy and past your Lahman data and any other manipulation, then the code to create your plot, including the part that is supposed to be dynamic (the orange line for ave.so for a single team).
Only this time, don't actually use input$my_team.  Instead, simply replace input$my_team with the name of any of the teams.  Does the orange line appear for my_team/``one.team`?
This way, you can test the plot itself and determine if the error lies within your plot code, or if it is an issue with the reactive variable code.

@pharri14 any news?

@jamisoncrawford I figured out what I was doing wrong. For one, instead of listing 10-25 teams in the {.sidebar} chunk, I had made my code include every single team. This was causing issues because I had my timeframe set to start at 1900 and some of the teams were outside of those bounds. So, I switched it to having 25 teams with large datasets within that time frame (see below).

selectInput( inputId = "my_team", # creates slot in input to hold values: input$your_inputId
             label = "Choose a Team", # title of the input widget
             choices = c("Cincinnati Reds",
                         "Pitsburgh Pirates",
                         "Philidelphia Phillies",
                         "St. Louis Cardinals",
                         "Chicago White Sox",
                         "Detroit Tigers",
                         "Chicago Cubs",
                         "Boston Red Sox",
                         "New York Yaknees",
                         "Cleveland Indians",
                         "Baltimore Orioles",
                         "Washington Senators",
                         "New York Giants",
                         "Philidelphia Athletics",
                         "St. Louis Browns",
                         "Los Angeles Dodgers",
                         "San Fransisco Giants",
                         "Minnesota Twins",
                         "New York Mets",
                         "Houston Astros",
                         "Atlanta Braves",
                         "Milwaukee Brewers",
                         "Oakland Athletics",
                         "Kansas City Royals",
                         "San Diego Padres"), # options for the user, all team names
             selected = "Los Angeles Dodgers" ) # sets the default option

Additionally, I organized my main chunk as follows:

renderPlot
one_team filter
par
plot.new
plot.window
I plot all of the points in gray as well as the average line in blue
I establish the X and Y-axis
I set up the title
I set up the legend (which also includes the one_team filter)
I insert all of the text on the chart to highlight specific points in gray
I have the dynamic plotting function

I am not sure if it helped or if I'm tricking myself, but when my code was not reacting properly when run, I took your advice and copied it into a new script, ran that, and then re-ran my original script, and it worked.
Thanks for helping me figure out the issue

@pharri14 fantastic work troubleshooting this whole thing!  Thanks for your patience with my instructions.  Looking forward to seeing your first interactive data product! 🔥 🔥 🔥


XSD Extensions to JSON

Hi. I want to write a Activiti BPMN moddle extension, like the Camunda one.
Do you use some tool to parse the XSD and write the JSON equivalent?
Can anyone any documentation about it?
PS: The camunda extension that is used in test classes in this project uses the Activiti XSD Namespace many times. Camunda have more extensions than Activiti or they are compatible?
Thanks!

Do you use some tool to parse the XSD and write the JSON equivalent?

You can use the XSD as a reference, however it hides important information such as relationships. The Camunda extension is hand made. The repository and the contained test cases should give you a good indication what to build and how to test it.

The camunda extension that is used in test classes in this project uses the Activiti XSD Namespace many times. Camunda have more extensions than Activiti or they are compatible?

Camunda forked Activiti years ago. There is some overlapping in attributes but nowadays non of these projects claims to be compatible with the other.


Extension registry for lapce

I'm working on a registry for lapce so everyone can easily publish and install lapce plugins.
The source code is at coffee-is-power/lapce-extensions
The back end uses rust with rocket.rs, and the front end uses svelte. The API is still a WIP, but it can already login with GitHub.

This was resolved #1643


release for arm containerservice

https://github.com/Azure/sdk-release-request/issues/526

Closing this PR as the @azure/arm-containerservice has since been re-generated with updated swaggers via #10252 and #10720


fix(RHTAPWATCH-660): Smee server IP filter fix

The IP filter configuration string for the Smee server included a trailing newline that caused it to be ignored.
Also adding comments to warn future developers touching this.

/approve
/lgtm

/test appstudio-e2e-tests


Improve search performance.

VSCode Version: 1.7
OS Version: Windows 10

Steps to Reproduce:

Search for text in a large folder
Open a cmd window and search for the same text using "findstr".
Note how findstr is an order of magnitude faster.

Searching for stuff is a key operation. It shouldn't be this slow.

Improvements are in the upcoming 1.8. #15384

FWIW, one way to improve the perceived performance could be to prioritize searches around "known" locations of the tree.
I often open up a huge folder because I want the context, but in practice I spend 99% of my time in some tiny sub-tree. When I search for something, the thing I want is probably going to be "near" some of my currently open files. Perhaps sorting the search order to prioritize folders that are close to currently opened files would mean that all the matches show up instantly (and I don't really care that it keeps searching the rest of the folder for another 10s after I've found what I wanted).

Good point, it would prevent us from showing the results in a deterministic order because the order we search depends on what files are/were open. /cc @roblourens

Interesting idea, also you should see faster search in Insiders now if you want to try it out.


Describe the bug
I was using groq for general completion and openai for embeddings. For some reason it keeps giving me invalid api error in the log, but my api token is actually valid(I test it out with a couple of tasks and they all worked).
It's most likely that this is not caused by groq, since:

there's no usage in my openai dashboard but there is usage in groq dashboard
after switching to the nomic embed model on LMstudio this problem no longer pops up

Steps to reproduce

generate a new full-access open ai api key
insert into .env file GRAPHRAG_API_KEY = "api key"
replace the groq settings as below
run command to start generation task

Expected Behavior
No response
GraphRAG Config Used
encoding_model: cl100k_base
skip_workflows: []
llm:
api_key: ${GROQQ_API_KEY}
type: openai_chat # or azure_openai_chat
model: llama3-8b-8192
model_supports_json: true # recommended if this is available for your model.
#max_tokens: 4000
#request_timeout: 180.0
api_base: https://api.groq.com/openai/v1
#api_version: 2024-02-15-preview
#organization: <organization_id>
#deployment_name: <azure_model_deployment_name>
tokens_per_minute: 20000 # set a leaky bucket throttle
requests_per_minute: 30 # set a leaky bucket throttle
#max_retries: 3
#max_retry_wait: 10.0
#sleep_on_rate_limit_recommendation: true # whether to sleep when azure suggests wait-times
#oncurrent_requests: 1 # the number of parallel inflight requests that may be made
parallelization:
stagger: 0.3
#num_threads: 50 # the number of threads to use for parallel processing
async_mode: threaded # or asyncio

embeddings:
##parallelization: override the global parallelization settings for embeddings
async_mode: threaded # or asyncio
llm:
api_key: ${GRAPHRAG_API_KEY}
type: openai_embedding
model: text-embedding-3-small
Logs and screenshots
{"type": "error", "data": "Error Invoking LLM", "stack": "Traceback (most recent call last):\n  File "D:\python3.11\Lib\site-packages\graphrag\llm\base\base_llm.py", line 53, in _invoke\n    output = await self._execute_llm(input, **kwargs)\n             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File "D:\python3.11\Lib\site-packages\graphrag\llm\openai\openai_embeddings_llm.py", line 36, in _execute_llm\n    embedding = await self.client.embeddings.create(\n                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File "D:\python3.11\Lib\site-packages\openai\resources\embeddings.py", line 215, in create\n    return await self._post(\n           ^^^^^^^^^^^^^^^^^\n  File "D:\python3.11\Lib\site-packages\openai\_base_client.py", line 1805, in post\n    return await self.request(cast_to, opts, stream=stream, stream_cls=stream_cls)\n           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n  File "D:\python3.11\Lib\site-packages\openai\_base_client.py", line 1503, in request\n    return await self._request(\n           ^^^^^^^^^^^^^^^^^^^^\n  File "D:\python3.11\Lib\site-packages\openai\_base_client.py", line 1599, in _request\n    raise self._make_status_error_from_response(err.response) from None\nopenai.AuthenticationError: Error code: 401 - {'error': {'message': 'Invalid API Key', 'type': 'invalid_request_error', 'code': 'invalid_api_key'}}\n", "source": "Error code: 401 - {'error': {'message': 'Invalid API Key', 'type': 'invalid_request_error', 'code': 'invalid_api_key'}}", "details": {"input": [""EMQX CLUSTER":"EMQX Cluster is a group of EMQX nodes that work together to provide a scalable and fault-tolerant messaging service."", ""TOPIC B":", ""GEO":", ""EMQX ENTERPRISE LICENSE":"EMQX Enterprise License is a type of license provided by EMQX Enterprise."", ""ACCOUNT MANAGER":"Account Manager is a person responsible for managing licenses and providing support."", ""LICENSE":"License refers to a permission or authorization to use a product or service."", ""EMQX ENTERPRISE LICENSE UPDATE":"EMQX Enterprise License Update is the process of updating a license for EMQX Enterprise."", ""LOG":"Log refers to a record of events or activities, used for troubleshooting and monitoring purposes."", ""SSL/TLS":"SSL/TLS is a cryptographic protocol used for secure communication over the internet."", ""CERTIFICATE":Here is a comprehensive summary of the data:\n\nThe CERTIFICATE is a digital file used for authentication and encryption purposes. It is employed in various contexts, including the TLS handshake process, to verify the identity of a website or server, and in MQTT connections to authenticate and encrypt data. In essence, the CERTIFICATE serves as a digital proof of identity, allowing entities to establish trust and secure communication with one another.", ""CIPHER SUITE":"Cipher Suite is a set of algorithms used for encrypting and decrypting data."", ""HANDSHAKE FAILURE":Here is a comprehensive summary of the data:\n\n"Handshake Failure" is an error that occurs during the SSL/TLS connection handshake process.", ""UNKNOWN CA":Here is a comprehensive summary of the data:\n\n"UNKNOWN CA" is an error that occurs when the certificate verification fails. This type of error occurs when the certificate verification process is unable to verify the authenticity of a certificate, resulting in a failure to establish a secure connection.\n\nNote: I resolved the contradictions by combining the two descriptions into a single, coherent summary.", ""SERVER":Here is a comprehensive summary of the data:\n\nThe "SERVER" is a device or application that provides MQTT services to clients, offering a platform for message-based communication. Additionally, the server is also a party involved in the TLS (Transport Layer Security) handshake process, indicating its role in securing communication between clients and other entities.", ""INTERMEDIATE CA":"Intermediate CA is a trusted certificate authority that issues certificates to other organizations."", ""ROOT CA":"Root CA is a trusted certificate authority that issues certificates to intermediate CAs.""]}}
Additional Information

GraphRAG Version:
Operating System:
Python Version:
Related Issues:

You're using a llama model, so you shouldn't be using an OpenAI key.

got same error...let me know if you find a solution

It currently dosent support GROQQ_API_KEY, you have to have OPEN AI api key only

In my case only have Openai API key and hot same error.

same here. I tried openai api and together api, both met invalid api issue as above.

@lauraparra28 @maverick001  Hey, I managed to get it work locally with groq+nomic embedding with Ollama. I modified my code as user spacelearner has said in issue #345  I guess there's some compatibility issues.


New build-scan command does not seem backwards compatible and yields 403 responses

Describe the bug
There seems to be a bug with the jf build-scan command where it does not behave the same way as the now deprecated jf rt build-scan command. When we attempt to call the jf build-scan command a 403 Forbidden error is yielded, however when called through the deprecated jf rt build-scan command the scan proceeds and normal and yields expected output. It would therefor seem that the new command is not completely backwards compatible with the older command. We have been unable to use the newer jf build-scan command as we cannot get around the 403 Forbidden error.
Current behavior
In the example below the build info has already been published to Artifactory, and we are trying to trigger a scan of the already published build info.
$ jf c add mgx<EMAIL_ADDRESS>--access-token=<redacted> --url=https://repo.mgx.pensionsmyndigheten.se/ --overwrite=true
$ jf c show
Server ID:			mgx
JFrog Platform URL:		https://repo.mgx.pensionsmyndigheten.se/
Artifactory URL:		https://repo.mgx.pensionsmyndigheten.se/artifactory/
Distribution URL:		https://repo.mgx.pensionsmyndigheten.se/distribution/
Xray URL:			https://repo.mgx.pensionsmyndigheten.se/xray/
Mission Control URL:		https://repo.mgx.pensionsmyndigheten.se/mc/
Pipelines URL:			https://repo.mgx.pensionsmyndigheten.se/pipelines/
User<EMAIL_ADDRESS>Access token:			***
Default:			true
$ JFROG_CLI_LOG_LEVEL="DEBUG" jf rt bs --fail=true "devops :: test-pipeline-maven-jar :: PLFORM-11374-jenkins-artifactory-utvardera-jfrogcli" 57
13:48:33 [Debug] JFrog CLI version: 2.53.2
13:48:33 [Debug] OS/Arch: linux/amd64
13:48:33 [🟠Warn] You are using a deprecated syntax of the command.
	Instead of:
	$ jf rt build-scan ...
	Use:
	$ jf build-scan ...
13:48:33 [🔵Info] Triggered Xray build scan... The scan may take a few minutes.
13:48:33 [Debug] Usage Report: Sending info...
13:48:34 [Debug] Sending HTTP GET request to: https://repo.mgx.pensionsmyndigheten.se/artifactory/api/system/version
13:48:34 [Debug] Sending HTTP POST request to: https://repo.mgx.pensionsmyndigheten.se/artifactory/api/xray/scanBuild
13:48:34 [Debug] Artifactory response: 200
13:48:34 [Debug] JFrog Artifactory version is: 7.55.10
13:48:34 [Debug] Sending HTTP POST request to: https://repo.mgx.pensionsmyndigheten.se/artifactory/api/system/usage
13:48:34 [🔵Info] Xray scan completed.
{
  "summary": {
    "total_alerts": 0,
    "fail_build": false,
    "message": "No Xray “Fail build in case of a violation” policy rule has been defined on this build. The Xray scan will run in parallel to the deployment of the build and will not obstruct the build. To review the Xray scan results, see the Xray Violations tab in the UI.",
    "more_details_url": ""
  },
  "alerts": [],
  "licenses": []
}
$ JFROG_CLI_LOG_LEVEL="DEBUG" jf bs --fail=true "devops :: test-pipeline-maven-jar :: PLFORM-11374-jenkins-artifactory-utvardera-jfrogcli" 57
13:48:51 [Debug] JFrog CLI version: 2.53.2
13:48:51 [Debug] OS/Arch: linux/amd64
13:48:51 [Debug] Sending HTTP GET request to: https://repo.mgx.pensionsmyndigheten.se/xray/api/v1/system/version
13:48:51 [Debug] Usage Report: Sending info...
13:48:51 [Debug] Sending HTTP GET request to: https://repo.mgx.pensionsmyndigheten.se/artifactory/api/system/version
13:48:51 [Debug] Sending HTTP POST request to: https://repo.mgx.pensionsmyndigheten.se/xray/api/v2/ci/build
13:48:52 [Debug] Artifactory response: 200
13:48:52 [Debug] JFrog Artifactory version is: 7.55.10
13:48:52 [Debug] Sending HTTP POST request to: https://repo.mgx.pensionsmyndigheten.se/artifactory/api/system/usage
13:48:52 [🚨Error] server response: 403 Forbidden

403 response code appears in the second call, despite having identical configuration and calling the same REST endpoints.
Reproduction steps
In this case we are using a very simple Hello World Java program, built with Maven, as our project.

Initiate and build the source code. Adaptations need to be made for your Artifactory/Xray server.

$ export JFROG_CLI_BUILD_NUMBER=57
$ export JFROG_CLI_RELEASES_REPO=mgx/jfrog-releases-remote
$ export JFROG_CLI_BUILD_URL=<path to CI server build>
$ export JFROG_CLI_REPORT_USAGE=false
$ export JFROG_CLI_BUILD_NAME="devops :: test-pipeline-maven-jar :: PLFORM-11374-jenkins-artifactory-utvardera-jfrogcli"
$ export CI=true
$ jf c add mgx<EMAIL_ADDRESS>--access-token=<redacted> --url=https://repo.mgx.pensionsmyndigheten.se/ --overwrite=true
$ jf mvnc --repo-deploy-releases pm-mvn-release-local --repo-deploy-snapshots pm-mvn-snapshot-local --repo-resolve-releases pm-mvn-public --repo-resolve-snapshots pm-mvn-public --include-patterns '*.jar, *.pom' --exclude-patterns '*tests.jar, *sources.jar'
$ jf mvn -f pom.xml -Dartifactory.publish.artifacts=true package

Publish the build info and initiate build-scan.

$ jf rt bp
$ jf bs --fail

This fails with 403 Forbidden.
Expected behavior
The jf bs command behaves exactly like the jf rt bs command and produces the same result given the same input/configuration.
JFrog CLI-Security version
1.0.3
JFrog CLI version (if applicable)
2.53.2
Operating system type and version
Reproduced both on Red Hat 8 and Arch Linux.
JFrog Xray version
3.66.6

Hey @PM-JoakimGustavsson
It looks like JFrog CLI is working as intended. We suspect there might be a problem in Xray side (since the error is coming from Xray). To further investigate this issue we need logs from Xray server. Please contact your JFrog Xray support and they will know how to handle it.
Thank you!

Hey @orz25 !
Thank you for your feedback!
I will file a support case with JFrog support instead then :)

For reference looking at the log they seem to be issuing very different API calls under the hood:
2024-04-19T10:42:17.246Z|966d771705bcc8a|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:17.756Z|658e17bb10674f93|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:18.160Z|7a15a01c4213ebf|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:18.743Z|7c0851dd2cf2bac9|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:19.253Z|18f419b1d6cca645|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:19.665Z|5ebdbedf0f8ed56e|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:20.511Z|5fe6879272052dc|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:21.186Z|4eedc8aee943918a|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:21.756Z|50cad1b51d8cbbf2|<IP_ADDRESS>, <IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:21.916Z|29aee8789c274063|<IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>|joagus@ppm.nu|GET|/ui/system/logs/data?file_size=10298141&id=xray-request.log|200|-1|0|0.596000|JFrog-Frontend/1.55.2
2024-04-19T10:42:27.016Z|a6a3eea931d9fbe1|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:27.432Z|34e0697fc58fd7c7|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:27.469Z|42a71c775a81605f|<IP_ADDRESS>, <IP_ADDRESS>|jfrt@01ejxfxnn0xw200b4h3jeh1zxx|GET|/api/v1/repositoriesPolicies?artifactoryId=default|200|-1|0|0.234000|XrayJavaClient/1.0.8
2024-04-19T10:42:27.944Z|c469e6da25143e57|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:28.337Z|a53c6a104731e863|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:28.908Z|84ce996f4719b4c0|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:29.521Z|2cb4884860005711|<IP_ADDRESS><EMAIL_ADDRESS>2024-04-19T10:42:30.134Z|39a2d49db156679f|<IP_ADDRESS><EMAIL_ADDRESS>
The calls to the v2 API are the failing ones from jf bs whereas the v1 calls are from jf rt bs.

I figured out what the issue was - the API token used by JFrog CLI needs to be an admin token. A user token is not sufficient. Switching to an admin token solved the issue.

@PM-JoakimGustavsson Thank you for the update!


⚠ ClamAV has degraded performance

In cfd5b90, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 739 ms

Resolved: ClamAV performance has improved in d6ef22d after 7 minutes.


Retries do not support backoff

When calling a service with the retry option, it will immediately make a retry request after the first failure:
https://github.com/bojand/grpc-caller/blob/549484c9ded7412490b95da9dd9db75afa00cfbc/lib/client.js#L47-L64
General guidance is to implement backoffs when retrying to prevent overloading the server.
The suggestion is to support a function that returns the amount of time to delay before attempting the retry. This approach would support different approaches to backoffs (constant value, linear, exponential, jitter, no jitter).
Something similar to this:
/**
 * @param {number} requestAttempts Amount of times that the request has been made.
 */
function backoff(requestAttempts) {
  return (requestAttempts * 20);
}

The retry option is just async.retry(). So you could do:
const res = await client.sayHello({ name: 'Bob' }, {}, { retry: { times: 3, interval: 200 } })
console.log(res)

and that will do retries with 200 milliseconds between retry intervals.
You can also indeed use a function as well:
const res = await client.sayHello({
  name: 'Bob'
}, {}, {
  retry: {
    times: 10,
    interval: function(retryCount) {
      return 50 * Math.pow(2, retryCount);
    }
  }
})

console.log(res)

Is this not working for you?

That works! Thanks for the reply, and my apologies for not catching this myself.


regexp: issue parse string

Go version
go1.20
Output of go env in your module/workspace:
GOPRIVATE=""
GOPROXY="https://goproxy.cn,direct"
GOROOT="/Users/xxxx/go1.20/go1.20"
GOSUMDB="off"
GOTMPDIR=""
GOTOOLDIR="/Users/xxxx/go1.20/go1.20/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.20"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/jl/_cckn_y96kg7wnz6t3624xkr0000gn/T/go-build4144142847=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
func main() { 	pattern := fmt.Sprintf(\b%s\b, "ch") 	regex := regexp.MustCompile(pattern) 	result := regex.ReplaceAllStringFunc("chácara", func(match string) string { 		trimmedMatch := strings.TrimSpace(match) 		replacement := "replacement" 		return strings.ReplaceAll(match, trimmedMatch, replacement) 	}) 	fmt.Println(result) }
What did you see happen?
actually result is "replacementácara"
What did you expect to see?
except result is "chácara"

sorry， code is：
func main() {
	pattern := fmt.Sprintf(`\b%s\b`, "ch")

	regex := regexp.MustCompile(pattern)
	result := regex.ReplaceAllStringFunc("chácara", func(match string) string {
		trimmedMatch := strings.TrimSpace(match)
		replacement := "replacement"
		return strings.ReplaceAll(match, trimmedMatch, replacement)
	})
	fmt.Println(result)
}

Related Issues and Documentation

regex: ReplaceAllString and ReplaceAll broken group replacement #65000 (closed)
regexp.ReplaceAllString is removing the accents of a normalized string #46108 (closed)
regular expression sub-match not used in replace #42291 (closed)
regexp: confusing ReplaceAllString result when using patterns like `$1_$2` #68712 (closed)
Regex: Underscore (_) makes replace group not found #32885 (closed)
regexp: targetSubString miss match #61465 (closed)
golang regex replace problem #29936 (closed)
regexp: wrong match #68281 (closed)
regexp: In ReplaceAllString, the expand parameter eats the word after it #17469 (closed)
regexp: compiling incorrectly #59007

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

see https://github.com/google/re2/wiki/Syntax
word boundaries use only ascii
Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Failure in ActiveRecord

I got following failure when starting up my local server.
module:ConnectionAdapters': uninitialized constant ActiveRecord::ConnectionAdapters::Mysql2Column (NameError)
Following gems are installed:
GeoRuby 1.3.4
activerecord 3.2.1
spatial_adapter 1.2.0
I am using MySQL Server 5.5.28 and InnoDB.
Shouldnt it be working with InnoDB already?

+1

+1


Implement trace analogical to numpy.trace

Is it possible to have trace explicitly implemented as in numpy.trace?

We could add this. I do not think it would be too difficult to implement in terms of diag and sum. I will mark as enhancement if someone wants to work on it.


bugs-tab removes highlight from "Issues" tab after ajax navigation

Please ensure:

[X] The bug is caused by Refined GitHub. It doesn't happen if I disable the extension.

Description

Click Issues tab
Click Issues tab

One click may be enough.
This doesn't happen if bugs-tab is disabled
Screenshot
https://user-images.githubusercontent.com/1402241/133208841-a2c0bbc3-8fa3-4da4-9c76-a94647d00450.mov
Console errors
No response
Example URL
https://github.com/sindresorhus/refined-github/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc
Browser(s) used
Safari 14

Duplicate of #4579 ?

We've got so many bugs that even @fregante is starting to forget them 😂
Jokes aside, I investigated this a little bit since I reported a similar bug in EnixCoda/Gitako#173.
GitHub code for highlighting is at https://github.githubassets.com/assets/app/assets/modules/github/behaviors/side-navigation.ts:
// Deal with selected_link_to on repository and gist pages
document.addEventListener('pjax:end', function () {
  const meta = document.querySelector('meta[name="selected-link"]')
  const selection = meta && meta.getAttribute('value')
  if (!selection) {
    return
  }
  for (const item of document.querySelectorAll('.js-sidenav-container-pjax .js-selected-navigation-item')) {
    const isSelected = (item.getAttribute('data-selected-links') || '').split(' ').indexOf(selection) >= 0
    isSelected ? item.setAttribute('aria-current', 'page') : item.removeAttribute('aria-current')
    item.classList.toggle('selected', isSelected)
  }
})

It is looking for the data-selected-links attribute:

However this is missing when bugs-tab enabled:

So is it GitHub's code that's removing it? What value should the attribute have?

So is it GitHub's code that's removing it?

No, it's RGH's own code #3001


Remove long literal to maintain python3 compatibility

Does snakebite even work on py3? Also, could you squash those commits?
Thanks!

I'm trying to use snakebite with python 3 and this 1L keeps throwing a syntax error, so I would appreciate if this resolved.
In Python 3, all integers are long. Python 2 switches to the long type transparently as needed anyway. int and long were "unified" a few versions back. So it seems that 1L could be 1 and work with both versions.


Support for #x... constructions to be able to collect groups of children

Subj.

Codecov Report

Merging #44 (9f9424f) into master (cacbd40) will decrease coverage by 0.13%.
The diff coverage is 42.10%.

@@             Coverage Diff              @@
##             master      #44      +/-   ##
============================================
- Coverage     80.50%   80.36%   -0.14%
- Complexity      912      914       +2
============================================
  Files           178      179       +1
  Lines          4155     4167      +12
  Branches        464      467       +3
============================================
+ Hits           3345     3349       +4
- Misses          629      637       +8
  Partials        181      181

Impacted Files
Coverage Δ

...rc/main/java/org/uast/astgen/base/ListBuilder.java
28.57% <28.57%> (ø)

...main/java/org/uast/astgen/interpreter/Creator.java
46.66% <66.66%> (+2.91%)
:arrow_up:

...in/java/org/uast/astgen/interpreter/Converter.java
100.00% <100.00%> (ø)

...main/java/org/uast/astgen/interpreter/Matcher.java
72.97% <100.00%> (ø)

Continue to review full report at Codecov.

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


"When to testing" duplicate

"When to testing" content appears two times. (for "all pages") - PROVIDER

@melikaeft this issue is not related to xls sheet.This is repeated in Json Topic


What does this homepage JS do?

@ascott1 @contolini @mistergone
Why is this JS at the bottom of the homepage?
https://github.com/cfpb/owning-a-home/blob/master/src/index.html#L110
I thought maybe some JavaScript is dependent on it, since otherwise it could just be added using Jekyll, but found no references to the class name "index" outside of the CSS files.

ping @akiany

Removed this JS in https://github.com/cfpb/owning-a-home/pull/378


Link to download installer is broken

File: docker-for-windows/install.md
The line:

If you haven’t already downloaded the installer (Docker for Windows Installer.exe), you can get it from download.docker.com.

links to a file that doesn't exist. Browsing to the root of https://download.docker.com/ doesn't show the referenced file. It appears there are no installers present, just zips of builds.

Hi @Wizarth ,
Thanks for reporting the issue.
Looks like the file you referenced redirects to https://docs.docker.com/desktop/windows/install/ where you will be able to download the Docker for Windows Installer.exe.

Sorry, perhaps I should have attached a screenshot or such, I get:
url: https://download.docker.com/win/stable/Docker%20for%20Windows%20Installer.exe
<Error>
AccessDenied
<Message>Access Denied</Message>
<RequestId>KHZWBZ4Q0AW2R0Q0</RequestId>
<HostId>
nDbjyGG344NmtqDj8VqP21YH0hxcqq2hV77dkhf0g9iWNg40U9r2tYhQwKOp/v+v9mH/rho80MI=
</HostId>
</Error>

and there is no redirection.


[Fix #1337] Update the cop to raise error for insufficient descriptions.

Per issue #1337, I tried to add a generic way to the ExampleWordingCop to call out insufficient descriptions. For now, I've used "works" and "throws errors" as that is a phrase I see used often as well. I didn't use a new cop since insufficient descriptions seemed to be along the same lines as "should not start with should or it" as they are both description related.

Before submitting the PR make sure the following are checked:

[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[X] Added tests.
[X] Updated documentation.
[X] Added an entry to the CHANGELOG.md if the new code introduces user-observable changes.
[X] The build (bundle exec rake) passes (be sure to run this locally, since it may produce updated documentation that you will need to commit).

If you have modified an existing cop's configuration options:

[X] Set VersionChanged in config/default.yml to the next major version.

Thanks @ydah! How do I get this merged? Do I need to wait for the other reviewers to approve?

I would expect the following to pass, but it fails, why?
    expect_no_offenses(<<-'RUBY')
      it "works" \
        "totally fine" do
      end
    RUBY

Looks like I didn't handle that edge case! updated the check to check for the entire description instead, and added some more test cases! Thanks for the catch!

I have renamed some of the required CI checks, which unfortunately means that you will need to rebase this branch to make CI pass.

Rebased and green @bquorning @Darhazer

Thank you for the contribution and your time, @akrox58 !


Pad the status line elements so that they do not move as the build is progressing

Looks good to me.


A little advice

https://github.com/pixijs/pixi.js/blob/f8ed935c26f6c34f5777b5ff5846b4cec2c923e8/packages/sprite/src/Sprite.js#L17
https://github.com/pixijs/pixi.js/blob/f8ed935c26f6c34f5777b5ff5846b4cec2c923e8/packages/sprite/src/Sprite.js#L498
from is a static method, should new be removed?
https://github.com/pixijs/pixi.js/blob/f8ed935c26f6c34f5777b5ff5846b4cec2c923e8/packages/core/src/textures/Texture.js#L20

You are correct. PR welcome to fix this.


GH-40720: [Python] Simplify and improve perf of creation of the column names in Table.to_pandas

Rationale for this change
The pandas_compat.py has over the years grown quite complex and a lot of pandas compatibility code, which probably can be simplified nowadays because of not supporting old pandas and Python versions anymore.
One part of the code where this is the case is in the reconstruction of the .columns Index object of the resulting DataFrame. Right now that always goes through a MultiIndex (even for simple column names), which has quite some overhead of the simple case. And it also has some old Python/pandas compat code that could be removed.
What changes are included in this PR?
The simplification to not go through a MultiIndex for the simple cases gives a nice speed-up as well:
In [1]: table = pa.table({'a': [1, 2, 3], 'b': [0.1, 0.2, 0.3], 'c': [3, 4, 5]})

In [2]: %timeit table.to_pandas()
251 µs ± 1.26 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)   # <-- main
68.1 µs ± 894 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)  # <-- PR

Are these changes tested?
We should have extensive existing tests for this
Are there any user-facing changes?
That should not be the case

GitHub Issue: #40720

@github-actions crossbow submit -g python

In [1]: table = pa.table({'a': [1, 2, 3], 'b': [0.1, 0.2, 0.3], 'c': [3, 4, 5]})

In [2]: %timeit table.to_pandas()
251 µs ± 1.26 µs per loop (mean ± std. dev. of 7 runs, 1,000 loops each)   # <-- main
68.1 µs ± 894 ns per loop (mean ± std. dev. of 7 runs, 10,000 loops each)  # <-- PR

btw, 68 µs still seems non-trivial for such a small table, do we know what the cost factors are?

Based on a cprofile run and for this small example, around 35% comes from the actual conversion in our C++ layer (creating the (copied) numpy arrays), and the remainder is some overhead in our python code, and then mostly the overhead on the pandas' side constructing the objects.
For example, a large part (ca 20-25%) is in the creation of the Index object for the column names. Looking at the profile, there looks to be some easy room for improvement here, but so that's something to do on the pandas side.


kubernetes_io_hostname missing in metrics

What did you do?
Tried to install this grafana dashboard https://grafana.com/dashboards/315
What did you expect to see?
Dashboard working.
What did you see instead? Under which circumstances?
Graphs are showing "no data points"
Environment
Using kube-prometheus with v0.14.1 prometheus-operator release.

Kubernetes version information:
Client Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.5", GitCommit:"17d7182a7ccbb167074be7a87f0a68bd00d58d97", GitTreeState:"clean", BuildDate:"2017-08-31T09:14:02Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"7", GitVersion:"v1.7.5", GitCommit:"17d7182a7ccbb167074be7a87f0a68bd00d58d97", GitTreeState:"clean", BuildDate:"2017-10-04T09:07:46Z", GoVersion:"go1.8.3", Compiler:"gc", Platform:"linux/amd64"}

Kubernetes cluster kind:
Minikube but tried too in an Azure cluster created with acs-engine.

It seems that some labels are missing from a lot of metrics, I don't know if the problem is related to Prometheus v2.0 or some other problem related with prometheus-operator itself, this is one of the metrics are failing now from the linked dashboard:
sum (rate (container_network_receive_bytes_total{kubernetes_io_hostname=~\"^$Node$\"}[1m]))
There was a problem getting the node name in the dashboard but I fixed that, now the next thing is failing and I am don't know how to fix is that container_network_receive_bytes_total doesn't have the label kubernetes_io_hostname available.
I would like to be able to integrate that dashboard, does anyone how to fix that? Thanks!

The metrics on cAdvisor have no stability guarantees unfortunately, so these may have changed. We'll have to investigate.

That dash works fine with me, I expect some of the exporters not to work correctly.

Hey @Morriz, I am using kube-prometheus directly, not helm based installation, maybe the problem is related?
I don't see the label kubernetes_io_hostname in any of the metrics used in that dashboard so it's impossible in my setup for those charts to work.

Then I think you are not correctly deploying the hack/... scripts, or have not generated the correct artefacts. Also, I have only been able to get the manifests to work together by NOT specifying a namespace with kubectl apply. So trying to deploy everything into a 'monitoring' namespace has been no success (I am only deploying on minikube for now, so I assume it's not rbac related, as that is not active on minikube yet).
Anyway, you have to check the 'targets' section of the prometheus server to see if your exporters are scrapable.
Personally, I suggest you do go the helm way, as it packages up what you are trying to deploy atomically, and makes you think about how to do that...

For me, everything is working perfectly fine using the monitoring namespace and the kube-prometheus scripts.
You can see the Prometheus targets here:

All the included dashboards in kube-prometheus work perfectly fine, and other dashboards I integrated directly from grafana are working fine too. The only one is failing for me is this specific dashboard because it makes use of kubernetes_io_hostname.
The other dashboards that use label_values(node_boot_time, instance) as node identification are working fine. I will try a minikube based installation with helm but I had problems using the helm based installation with my acs-engine deployed cluster, that is the reason why I am using kube-prometheus directly.
Miguel.

that is strange, as I just added it and it works nicely :|

The ClusterRoleBindings of the kube-prometheus stack assume the monitoring namespace, if you wish to deploy it to another namespace you'll have to check those manifests and adapt them. The helm charts, are currently in a bit of a rough state, we're working on getting them into shape. I'd recommend this article if you still want to use helm to deploy the kube-prometheus stack with it.

I used monitoring namespace and kube-prometheus scripts, indeed everything is working fine for me, other than the kubernetes_io_hostname label is not inside metrics.

@skuda can you just query for container_network_receive_bytes_total in the Prometheus UI, and paste the "console" output here? I'm guessing that the label was renamed.

I don't think so, but here it is:

container_network_receive_bytes_total{container_name="POD",endpoint="http-metrics",id="/kubepods/besteffort/pod4256e17ced5ddf3c3d2993e020b26592/40b24544b3051051f767fa5cc0aecb27490b4099c17b7c4a220134763b961454",image="gcrio.azureedge.net/google_containers/pause-amd64:3.0",instance="<IP_ADDRESS>:10255",interface="eth0",job="kubelet",name="k8s_POD_kube-scheduler-k8s-master-42737137-0_kube-system_4256e17ced5ddf3c3d2993e020b26592_3",namespace="kube-system",pod_name="kube-scheduler-k8s-master-42737137-0",service="kubelet"} |<PHONE_NUMBER>1

container_network_receive_bytes_total{container_name="POD",endpoint="http-metrics",id="/kubepods/besteffort/pod427d3a4dadff5c4319c62797574c7106/97b8e6fd078dce2663e66a3d6a7c7d086342e6e4f3676e45dafa652f29484c35",image="gcrio.azureedge.net/google_containers/pause-amd64:3.0",instance="<IP_ADDRESS>:10255",interface="cbr0",job="kubelet",name="k8s_POD_kube-apiserver-k8s-master-42737137-0_kube-system_427d3a4dadff5c4319c62797574c7106_0",namespace="kube-system",pod_name="kube-apiserver-k8s-master-42737137-0",service="kubelet"} | 3248871

hmm .. sounds like there was a change upstream then. Could you open an issue on the cadvisor repository and tag me? We can leave this open until we resolve things there.

@skuda I've also followed the same way that you have tried. Even me too ended up with the same problem that you are facing label kubernetes_io_hostname is not present on Prometheus.
But when I tried with helm method I don't see that problem, but it creates other problem like no alerts configurations on Prometheus alerts page.
I'm using Kubernetes 1.6 with RBAC disabled on helm method.

The helm chart is too bloated and should be split up to be able to use the prometheus and alertmanager charts (they are now bundled up with kube-prometheus).
Take a look at morriz/mostack to see what I did. It has a lot of work in figuring out how to deal with helm, rbac and network policies.
On Nov 20, 2017, at 11:21, Prasanna Ranganathan<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:

@skuda https://github.com/skuda I've also followed the same way that you have tried. Even me too ended up with the same problem that you are facing label kubernetes_io_hostname is not present on Prometheus.
But when I tried with helm method I don't see that problem, but it creates other problem like no alerts configurations on Prometheus alerts page.
I'm using Kubernetes 1.6 with RBAC disabled on helm method.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/coreos/prometheus-operator/issues/725#issuecomment-345653064, or mute the thread https://github.com/notifications/unsubscribe-auth/AA5fKTAab8BmbQ88rmdqH3X5BArzXzUdks5s4VLBgaJpZM4QQMcs.

I totally agree @Morriz, I documented what I imagine the helm charts to look one day: #592. We don't use helm for the prometheus-operator, so we rely on the community to maintain them.

@stuxnet78 I am waiting for cadvisor maintainers to check the issue I created to have a proper fix, the only Grafana dashboard that fails for me is the one I linked in the first post so I can live without kubernetes_io_hostname for now.

Closing this as there doesn’t seem to be a problem with the Prometheus operator, I’ll try to follow up with the cAdvisor people.


Fixing README.md

Small change to fix the links and the gifs to show up properly.

Tries to fix the same issue as #2441 right?

yes!

@alvarojoao then let's close this in favour of the other one. Thanks!


AvailabilitySet refinements, javadocs, bug fixes

Can one of the admins verify this patch?

Considering that you have work on top of this we should be able to merge this one and you can address the feedbacks later.

sounds good.

Merging.


Posh-Git special characters not showing up correctly

Windows Terminal version
1.11.3471.0
Windows build number
10.0.19043.0
Other Software
Installed Posh-Git and Oh-My-Posh from the powershell gallery
Steps to reproduce

Install Posh-Git PowerShellGet\Install-Module posh-git -Scope CurrentUser
Install Oh-My posh PowerShellGet\Install-Module oh-my-posh -Scope CurrentUser
Import the git-posh module
Download an oh-my-posh theme and set it in the environment set-poshprompt -Theme "C:\Users\jadelaga\Downloads\agnoster.omp.json"
Navigate to a git repo

Expected Behavior
Git posh symbols are loaded correctly and I can see the special characters in the terminal
Actual Behavior
A lot of special characters end up as boxes with question marks

This might be a deeper issue with powershell as the screenshot shows VS and Powershell having the same issue.

Looks like one needs to install the Caskadya Cove NF fonts to support these special characters.
Closing this ticket


Write a Python program to find average of digits of a number

Description
Write a Python program to find average of digits of a number
Average of digits of a number is the sum of all digits divided by the number of digits.
Input  : 123
Output : 2

How to contribute

Save the solution in program/find-average-of-digits-of-a-number.py file
Add find-average-of-digits-of-a-number.py file in find-average-of-digits-of-a-number folder

!assign


tests: arm_runtime_nmi: Make test build on v8m

The ICSR[NMIPENDSET] bit got renamed to ICSR[PENDNMISET] in the v8m
architecture.  So we map SCB_ICSR_PENDNMISET_Msk to
SCB_ICSR_NMIPENDSET_Msk to the tests builds and functions.
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6553   +/-   ##
=======================================
  Coverage   53.43%   53.43%
=======================================
  Files         437      437
  Lines       41377    41377
  Branches     7931     7931
=======================================
  Hits        22111    22111
  Misses      16048    16048
  Partials     3218     3218

Continue to review full report at Codecov.

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

@galak This is a nice catch, indeed.
I am wondering: since we are touching this, could we do it properly? I.e. define NMI_SetPending(void) in arm/core/cortex_m/nmi.c, and there we can have different-implementations at compile-time, e.g.
#if ARMV8_M
SCB->ICSR |= SCB_ICSR_PENDNMISET_Msk;
#else
SCB->ICSR |= SCB_ICSR_NMIPENDSET_Msk;
#endif
then, the test would just call the API function.

That is OK, then.


group function

The group function allows to combine multiple columns of a dataframe into one PooledDataArray column. A user case is to create groups based on multiple variables before fitting a model.
The function group basically rewraps code from groupby.I'm not sure whether this functionality already exists or not.

Needs tests and docs...

I have added tests, a doc.
Moreover

I have renamed the function to poolall since it completes the pool function.
I have added a skipna option : should observations with some column equal to NA grouped into a NA group?  The option defaults to true, conserving any NA in the original columns. I think it is the behavior one generally wants when fitting models. I don't know the usual name for NA options in Julia, so tell me if skipna does not sound right.

I have updated my commit to solve a bug I spotted in groupby, which fails in the following case:
    df = DataFrame(v1 = pool(1:1000), v2 = pool(fill(1, 1000)))
   groupby(df, [:v1, :v2])

Grouping by v1 and v2 should give 1000 groups. However, only 255 groups are created. pool(fill(1, 1000)) creates a PooledDataArray with refs of type Vector{Uint8}. This type  is not promoted for a higher type when grouping by v1 and v2, while the number of groups becomes > 255. I now promote this vector to Uint32 in all cases. I have added a test for this example.

I think it's good to go now.


KUBECONFIG env var with empty path item leads to "IsADirectoryError: [Errno 21] Is a directory: '.'"

Which project are you reporting a bug for?
kr8s
What happened?
My KUBECONFIG env var has a leading empty path segment, i.e. ":/path/to/config1:/path/to/config2". kubectl ignores this empty path segment, but trying out the following simple kr8s example resulted in an error:
import kr8s

for pod in kr8s.get("pods", namespace="machina"):
    print(pod.status.podIP, pod.metadata.namespace, pod.metadata.name)

Error (trimmed):
[ . . . ]
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_api.py", line 73, in f
    await self.auth
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_auth.py", line 53, in f
    await self.reauthenticate()
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_auth.py", line 65, in reauthenticate
    await self._load_kubeconfig()
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_auth.py", line 100, in _load_kubeconfig
    self.kubeconfig = await KubeConfigSet(
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_config.py", line 35, in f
    await config
  File "/[...]/.venv/lib/python3.9/site-packages/kr8s/_config.py", line 197, in f
    async with await anyio.open_file(self.path) as fh:
[ . . . ]
IsADirectoryError: [Errno 21] Is a directory: '.'

Side note: This is due to the way my bash_profile scripts were setup to always append to existing KUBECONFIG var and initial one is always going to be empty. I have since fixed this, but thinking that other users may have a similar setup and run into this issue.
Anything else?
Potential solution
Update kr8s._auth.KubeAuth._load_kubeconfig to check for empty path segments and ignore them. Could also go a step further and ignore any paths that are directories also as they will cause an error, but that would be inconsistent with kubectl, which does return error if a config path is a directory.
I would be happy to create a PR for this. Is there a Contributing guide I should follow?

Totally agree we should handle this gracefully. I think we should strive to copy the behaviour of kubectl so we should handle empty paths, but raise an error on badly configured paths. Although if kubectl's error is not helpful that doesn't mean ours can't be.
Yep contributing guide is here.


zephyr-ide.get-gdb-path uses backslashes

${command:zephyr-ide.get-gdb-path}" uses backslashes which is incompatible in unix systems.
Current solution: use "${command:zephyr-ide.get-toolchain-path}/arm-zephyr-eabi/bin/arm-zephyr-eabi-gdb" instead

Thanks for the issue. Ill have a look into changing it when I have some time.

This is fixed in v1.0.41. Will reopen if you are still experiencing issues.


Support undici

Undici is the new official node http client.
When trying xray on a Node-18-Lambda while using undici I get an error.
There is an issue on undicis side (https://github.com/nodejs/undici/issues/2190) about not working with xray.
They refer to use the diagnostics channel.
import {captureHTTPsGlobal} from 'aws-xray-sdk';
import * as http from 'http';
import * as https from 'https';

captureHTTPsGlobal(http);
captureHTTPsGlobal(https);

2023-10-22T11:24:52.170Z	undefined	ERROR	Uncaught Exception 	{
    "errorType": "TypeError",
    "errorMessage": "Cannot add property __request, object is not extensible",
    "stack": [
        "TypeError: Cannot add property __request, object is not extensible",
        "    at enableCapture (file:///var/task/index.mjs:34012:25)",
        "    at captureHTTPsGlobal3 (file:///var/task/index.mjs:33900:9)",
        "    at file:///var/task/index.mjs:35587:44",
        "    at ModuleJob.run (node:internal/modules/esm/module_job:194:25)"
    ]
}

undici support is in X-Ray nodejs SDK backlog, but we don't have specific timeline due to resource constrains. Now xray service is able to work with OpenTelemetry, for customers need new libraries support at the moment, we recommend customers onboard opentelemetry
Refer to OpenTelemetry undici support: https://github.com/open-telemetry/opentelemetry-js-contrib/issues/1021


Can't install

Hi, when i try to install this, composer get stuck. When i do composer -vvv, it got stuck when git remote set-url origin 'https://www.devkit.net/thepixeldeveloper/sitemap.git' && git remote update --prune origin. Can you switch git repo from devkit.net to github? thanks

Related: #55

It should have updated, if you're still having issues then let me know.

it works now. thanks


Improve README structure

This Pull Request improves the look of the README.md file by making it overall easier to navigate and better-structured while also not deleting any important info from it. I hope this works well for you, please let me know if something doesn't look right.

Can I make the necessary changes?

@TanushJadhav All the changes have already been made, there is no need for any edits in my opinion.


Statically Link Windows Binaries

Statically Link Windows Binaries #136 and add support for feature branches

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Restoring Git repo encounters case-sensitivity conflicts because the repo contains files with name only differing in casing. Please fix the casing conflicts and try building again. Note that if you are working on a Pull Request, please fix the target branch first if the target branch contains casing conflicts. Detected casing conflicts are with following files: samples/Basic/Xamarin.Forms/SkiaSharpSample.Linux.sln, samples/Gallery/Xamarin.Forms/SkiaSharpSample.Linux.sln
Learn more: https://aka.ms/gitcase

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Inappropriate ioctl for device on MacOS BigSur 11.15.2

Python version
BoAAAA :: ~ » python3 --version
Python 3.8.2
BoAAAA :: ~ »

Os Vesion
BoAAAA :: ~ » sw_vers
ProductName:	macOS
ProductVersion:	11.5.2
BuildVersion:	20G95
BoAAAA :: ~ »

Error Message
...
File "/Users/jas0n/Library/Python/3.8/lib/python/site-packages/hatsploit/hsf.py", line 93, in launch
    self.console.script(script, do_shell)
  File "/Users/jas0n/Library/Python/3.8/lib/python/site-packages/hatsploit/core/base/console.py", line 178, in script
    self.launch_history()
  File "/Users/jas0n/Library/Python/3.8/lib/python/site-packages/hatsploit/core/base/console.py", line 107, in launch_history
    self.enable_history_file()
  File "/Users/jas0n/Library/Python/3.8/lib/python/site-packages/hatsploit/core/base/console.py", line 99, in enable_history_file
    readline.read_history_file(self.history)
OSError: [Errno 25] Inappropriate ioctl for device

Hello, @nikotidar
Thank you for submitting this issue, but next time submit it at HatSploit Framework's repository - https://github.com/EntySec/HatSploit
This is a very strange error, because I have this version of macOS too and did not get this error. Please, provide some additional details like:

Installation steps
Your terminal emulator
Your shell name (sh, bash, zsh and etc.)

Regards,
Ivan Nikolsky (@enty8080)


RunTimeError is misspelled in the test status files

The proper camel case spelling of "run-time error" should be RunTimeError and it should be changed in the test status files.
If not, we should then switch to CompiletimeError and MissingCompiletimeError (which is horrible) for consistency.

TL;DR: "RuntimeError" and "CompileTimeError" are each consistent with current usage even though not consistent with each other.
I have sunk more of my life into hyphenation and concatenation rules that probably anyone should, mainly because I need to make sure I apply a correct, consistent style in my books.
The short answer is that there are no context-free rules for which word pairs are separate words ("real estate"), hyphenated ("three-year-old son"), or single words ("website"). The general rules are:

Over time, a new compound word begins as separate words ("web site"), becomes hyphenated ("web-site"), and eventually coalesces into a single word ("website") if it is frequently used enough. So the answer for how a compound word is punctuated is mostly a matter of history and popularity. The "correct" answer for any given word usually requires consulting word lists in a style guide like the Chicago Manual of Style.

Compound words that have not become a single word ("keyboard") are typically hyphenated when used as an adjective preceding a noun ("up-to-date schedule") but not when used as a standalone noun phrase ("schedule is up to date"). This is not always true, but it usually is. It's "post office box", not "post-office box" even though "post office" is a compound word.

In this specific case, we're in a weird spot because "run time" is in a transition period. [Wikipedia lists both "run time" and "runtime"](https://en.wikipedia.org/wiki/Run_time_(program_lifecycle_phase)) when referring to the execution phase as a noun. When referring to the part of the system, they use "runtime" or "run-time". The examples related to computing in the OED use "run-time" for both the noun and adjective forms.
Since our readers are in a computing context, I think it's best to follow the norms of other tech sources, since the wider English community tends to lag that. We're using "runtime" as an adjective "runtime error", so looking for that form, I see:

Our own core library has a member "runtimeType". (lowercase "t"). ("Lowercase" is itself a compound word that recently transitioned from an open to closed form.)

Python calls the exception "RuntimeError".

Ruby also uses "RuntimeError".

IE reports it as a "Runtime Error".

Given that, I think it should be "runtime", no hyphen or space, when used as an adjective. When CamelCased, the "t" stays lowercase.
"Compile time error" is different. If you Google, most people tend to use "compile error" and omit "time" entirely. I see 2,380,000 results for "compile time error", [2,430,000] results for "compile-time error", and only 16,700 results for "compiletime error".
"Compile time" as a compound word is much less common than "run time" (unsurprising since many more people run programs than compile them), so it doesn't appear to have gone through the coalescing process that "runtime" has yet. Maybe it will at some point, but the idiomatic style today is still open form.
If the discrepancy really bothers you, one way to dodge it is to use the more common term "compile error" and eliminate "time" entirely. It's also shorter. "Static error" would work too.
Now, when are we going to talk about "timeout"? :)

I do think self-consistency is important and that parallel things should use parallel constructions.  It helps the reader because they don't have to wonder if you're making an important distinction or not.  And it helps the writer on a team of writers because they don't have to guess what to write.
Here it's not much effort and I don't think we'd be too far away from common usage if we just adopted the simple rules: (1) "run time" and "compile time" are noun phrases describing times, because they are parallel and "compiletime" is ugly, (2) the adjective forms are "run-time" and "compile-time" because they are parallel, they are a single adjective not two, and "compiletime" is ugly.
Then we can ask how hyphenated words are camel-cased.  I don't really know what the rule is, but it should be consistent and, again, "compiletime" is still ugly.
We do use "runtime" in Dart, but we use it as jargon describing the "run-time system" like the VM or the dart2js JS runtime.  I think that's fine, but here it's parallel to "compiler".  If we use the same nouns "runtime" for the time and "runtime" for a Dart run-time system, then we lose the ability to distinguish the adjective forms "runtime" and "runtime" which I would actually miss.  (The noun forms are easy because "runtime" as in "run-time system" normally has an article and "runtime" as in "run time" normally does not.)
(I also think runtimeType was a mistake because that's a type used in the run-time system, but nobody asked me.)

Or else we get way out ahead of the curve and go with runtimeerror, compiletimeerror, and missingCompiletimeerror.

I don't think we'd be too far away from current usage

The O'Reilly style guide for their books uses "runtime" for both noun and adjective use, "compile time" for a noun, and "compile-time" for adjective.

The Google Developer Docs style guide, which is the style guide we use for our own docs does the same.

I think those are about as authoritative as we could hope to be.


Error on startup with vim 7.2

Thanks for this great plugin. It works well on other machines, but on a server with only vim 7.2 the following error appear at startup when trying to open two files simultaneously:
Error detected while processing function workspace#render..<SNR>62_RenderTab:
line   17:
E706: Variable type mismatch for: prev_buffer
Error detected while processing function workspace#render..<SNR>62_RenderTab:
line   19:
E706: Variable type mismatch for: next_buffer

Hey @fbreitwieser, thanks for reporting the issue and using the plugin. I will have a look asap!

Thanks for the quick reply, @bagrat !

Hey @fbreitwieser, looks like I missed this part:

...trying to open two files simultaneously

is that when you do vim file1 file2, or whenever you open two buffers after starting empty vim?

Closing this as no activity for a pretty long time.


Better error messages when trying to use virtual fields in queries

Following this discussion

@rizasal 💘 thank you so much!

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Sent headObject get stuck after getObject request of alluxio-proxy v2

Alluxio Version:
2.9
Describe the bug
Using go aws sdk, first call getObject, then call headObject, and the request is stuck.
Found exceptions in proxy log.

To Reproduce
import (
    "fmt"
    "os"

    "io"
    "github.com/aws/aws-sdk-go/aws"
    "github.com/aws/aws-sdk-go/aws/session"
    "github.com/aws/aws-sdk-go/service/s3"
    "github.com/aws/aws-sdk-go/aws/credentials"
)
func main() {

     sess, err := session.NewSession(&aws.Config{
      Region: aws.String("default"),
      Credentials: credentials.NewStaticCredentials(
         "xxx", "xxx", "TOKEN"),
         Endpoint:         aws.String("http://localhost:39999"),
      S3ForcePathStyle: aws.Bool(true),
   })
   if err != nil {
      _ = fmt.Errorf(err.Error())
   }

   bucket := "get_test"
   item := "bigfile.txt"

   file, err := os.Create("bigfile.txt")
   defer file.Close()
   object := s3.GetObjectInput{
      Bucket: aws.String(bucket),
      Key:    aws.String(item),
   }

   s3Client := s3.New(sess)
   result, err := s3Client.GetObject(&object)
   if err != nil {
       fmt.Fprintln(os.Stderr, err)
       os.Exit(1)
   }
   defer result.Body.Close()
   _, err = io.Copy(file, result.Body)
   if err != nil {
       fmt.Fprintln(os.Stderr, err)
       os.Exit(1)
   }

   if err != nil {
      fmt.Println("Unable to get item %q, %v", item, err)
   }

   input := s3.HeadObjectInput{
          Bucket: aws.String(bucket),
          Key:aws.String(item),
       }
        _, err2 := s3Client.HeadObject(&input)
        if err2 != nil {
        fmt.Fprintln(os.Stderr, err)
                os.Exit(1)
        }
    fmt.Println("finish")
}

Expected behavior
Requests are going well.
Are you planning to fix it
Yes.

@yyongycy @Jackson-Wang-7 Please help review the issue and solution.

wondering whether this error exists in 2.10?

wondering whether this error exists in 2.10?

This is an issue if async is enabled and the entire large file is uploaded in a single request.

wondering whether this error exists in 2.10?

@yyongycy This issue has been around for a long time.
https://github.com/Alluxio/alluxio/pull/17267


Create animation when player minimized/maximized

Similar to youtube app - you can see the player minimizing vs on Odysee, it just happens right away.

Similar to youtube app - you can see the player minimizing vs on Odysee, it just happens right away.

and also the auto splash screen like for apple, if possible.


MNT: be a bit more cautious about importing numpy

Small follow on to #157
A documented way to prevent a module from being imported is to set sys.modules['foo'] = None which would have passed the previous test and failed on import.  This will only try to use numpy if it is actually already imported.
I know this because we had a similar check for IPython in Matplotlib and get bit by someone who wanted to prevent importing IPython!

Always learning. Thanks @tacaswell.


Inconsistent compiler behaviour, adding an identity function leading to different outcome

Problem
When a locally bound function and a global function have the same name f, then using it as f () or (identity f) () yields different results.
Context
I was trying to implement an ADT representing a circular list. This is what I had so far:
module CircularList exposing (CircularList(..), get, next)

type CircularList a
    = CircularList { val : a, next : () -> CircularList a }

get : CircularList a -> a
get (CircularList { val, next }) =
    val

next : CircularList a -> CircularList a
next (CircularList { val, next }) =
    -- it doesn't work when I just do "next ()"???????
    (Debug.log "next" next) ()

It works as expected and the following test case is green:
test "get" <|
            \() ->
                let
                    s =
                        "bla"

                    a =
                        CircularList { val = s, next = \() -> a }
                in
                    Expect.equal s (CircularList.get (CircularList.next a))

However as soon as I remove the Debug.log and execute next () directly the test never finishes. Looks like the identify call makes the compiler use the locally bound variable, while without, it takes the global function.
See slack discussion for more details.
Environment

Elm Version: 0.18.0
NPM Version: 5.0.3
OS: Mac OS Sierra 10.12.6

Workround
Don't name the local variable the same as the global.
Expected Behaviour

The behaviour should be consistent whether I add identity or not
The compiler should warn because I am shadowing the global name (or not)
If the behaviour cannot be made consistent the code should not compile, forcing you to choose different names

It seems that debug statement is getting hold of the argument-next from the argument, while the execution of the method is calling the self-next method instead.
I think that in this case, the compiler should err with a message that there are two different "next" entities in the scope, and it does not know which one to choose.

The type-checker and the codegen disagree, leading to an incorrect result. The inner-most next should be the one used, which the type-checker does, but the codegen seems to think this is a recursive function and transforms it to a while loop instead.
@FRosner could you close the issue here and reopen it on elm-lang/elm-compiler instead? Thanks for the report!

I think this is a known and reported issue.
https://github.com/elm-lang/elm-compiler/issues/1452

I opened https://github.com/elm-lang/elm-compiler/issues/1636 instead.


Add a separate immediate quality reports queue

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Summary by CodeRabbit

New Features

Introduced a new service for immediate quality reports in the CVAT application, enhancing report processing capabilities.
Added a debugging configuration for immediate quality reports to improve development efficiency.
Implemented a new enumeration for immediate quality reports in the queue management system, allowing for quicker processing.

Bug Fixes

Enhanced the configuration for worker services to ensure proper handling of immediate quality reports.

Documentation

Updated deployment and configuration files to reflect new services and functionalities for immediate quality reports.

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

Project coverage is 83.59%. Comparing base (aaad059) to head (3ac1e0c).
Report is 11 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #8429      +/-   ##
===========================================
+ Coverage    83.58%   83.59%   +0.01%
===========================================
  Files          396      397       +1
  Lines        41906    41902       -4
  Branches      3881     3883       +2
===========================================
+ Hits         35026    35030       +4
+ Misses        6880     6872       -8

Components
Coverage Δ

cvat-ui
79.56% <ø> (+0.03%)
:arrow_up:

cvat-server
87.18% <ø> (+<0.01%)
:arrow_up:


Repeated Pings and No Imbeds

Hi I've just started hosting this bot but it seems to be pinging the new incursions repeatedly every 10 minutes as if the cache doesn't exist I've confirmed the file exists and is being updated. Also the messages aren't showing up with an embed just a constellation name. It was doing this before I made the change but I did make one change in mapping the BotController.ts so that I could remove the everyone ping as not everyone in my group runs incursions.

Hello Jacob, did you manage to fix the lack of caching?

I actually switched over to AA-Incursions for Alliance Auth but I still had the files for this project as well so I've spun it back up running for about an hour and so far so good, it might've been a permissions issue with the cache file. I do see content in the incursions_cache.json file, so it seems to be working. The data is also in an embed now so all of my problems seem to have gone away this time. If it comes back I'll let you know, thanks!


Share task details with other apps. #172

(continuing https://github.com/dmfs/opentasks/pull/282 , cause that was closed accidentally when branch was removed)
TODO German translations
(https://github.com/dmfs/opentasks/issues/172#issuecomment-312687018)

As discussed we are going to use a template engine:
https://codeka.github.io/carrot/
for creating the shared text.
(Moving this back to backlog)

Closing this PR for now. Please open a new one (rebased to master)


Issue running mila code

Make sure you can reproduce the issue with the latest version available
pip install milatools --upgrade
[milatools command e.g. mila code ...]

What command did you run?
[e.g. mila code ...]
Describe the bug
When I run mila code /path/to/remote_dir, all seems to work but vscode never launches, I get this traceback
See screenshot.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Mac Os Ventura Version 13.4

Additional context
Add any other context about the problem here.

Also note: Running from zsh shell, and running code in that terminal gives zsh: command not found: code

Seems to now be working on @ak3ra 's side. I suspect that this error might have been fixed by upgrading milatools.

Issue fixed by adding vscode "code" to path, and upgrading mila-tools


Update 0148-sort-list.java

This code is the implementation of the merge sort, as explained in the video

File(s) Modified: 0148-sort-list.java
Language(s) Used: Java
Submission URL: https://leetcode.com/problems/sort-list/submissions/950945371/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.

Thanks for your contribution. I re-added the old solution below, it's still a valid one. But added your mergesort first since it's the one used in the video.


Fix: mousewheel zoom is not natural

Description

Motivation and Context

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Test Case (changes that add missing tests or correct existing tests)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[ ] Chore (changes that don't modify src or test files)

Self Check before Merge

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Pull Request Test Coverage Report for Build 321199330

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build 316958166:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Add Microsoft Sql Server driver

I've recently needed to integrate SQL Server driver connection on my Project, so I decided to add a new Driver choice.

This is a bit of an opinionated change, adding another database driver.
I am not entirely against it, but, I would perhaps change out one of the current drivers with this one and see what others say.
One other note, we recently are moving away from a .env example and just have the values update directly the .env file


Docker image is huge

The docker image is currently giant considering what's included in it.
Expect a PR with a new Dockerfile soon :)

Indeed, I would gladly accept this :)

hi, seems there is a good proposition here  :
https://github.com/spidercpsf/mongoclient
https://hub.docker.com/r/giangndm/mongoclient/

Hi, thanks trying this one: https://github.com/spidercpsf/mongoclient, but as far as I know, that's an ancient project FROM meteorhacks/meteord:onbuild

just replace meteorhacks with kadirahq to get last version with meteor 1.4 :
https://github.com/kadirahq/meteord
but there is a better way: you can build your app, then create a small image from an official node image, as you can see here :
https://github.com/markoshust/docker-meteor/blob/master/Dockerfile
Just add  in your bundled directory a Dockerfile with only :
FROM markoshust/meteor
then :
docker build -t mongoclient/mongoclient .

gotta try automated build with FROM markoshust/meteor thanks

No chance:
Step 1 : FROM markoshust/meteor # Executing 2 build triggers... Step 1 : ADD . /opt/app Step 1 : RUN cd /opt/app/programs/server   && ([ -f package.json ] || npm init -f)   && npm i --loglevel=silent --unsafe-perm  ---> Running in b41c136997a7 /bin/sh: 1: cd: can't cd to /opt/app/programs/server Removing intermediate container 255fdeb9dcf2 Removing intermediate container b41c136997a7 The command '/bin/sh -c cd /opt/app/programs/server   && ([ -f package.json ] || npm init -f)   && npm i --loglevel=silent --unsafe-perm' returned a non-zero code: 2

Did you 'meteor build' first ?
You can try this, in your app directory :
meteor build /tmp --directory
cd /tmp/bundle
echo 'FROM markoshust/meteor' > Dockerfile
docker build -t mongoclient/mongoclient:latest .
docker push mongoclient/mongoclient:latest

It's an automated build on hub, I don't want to do any manual push

since no help comes out, closing this for now


Conn manager fuzz test improve

Commit Message: Improve state handling in conn manager fuzz test
Additional Description:
The fuzzer tends to repeat actions, but a decoding done already cannot
be repeated or an assert is hit.
Signed-off-by: Andre Vehreschild<EMAIL_ADDRESS>Risk Level: Low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
Platform Specific Features: n/a

@adisuissa @htuch @pradeepcrao @KBaichoo here is the patch as requested.


5.x: Implement worker node taints

https://blogs.oracle.com/cloud-infrastructure/post/auto-tainting-oke-nodes-startup
https://oracle-terraform-modules.github.io/terraform-oci-oke/guide/workers_cloudinit.html
https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/5.x/docs/src/guide/workers.md

Hi @devoncrouse, can you tell me if taints is working now? I see some implementation was done - https://github.com/oracle-terraform-modules/terraform-oci-oke/blob/c177189b4d44e0b1a41bae6e475b7b65b1e3c922/modules/workers/virtualnodepools.tf#L46:L53
However configuration is being ignored, then I try to use it with following example:
  worker_pools = {      n2-pool = {       shape              = "VM.Standard.E4.Flex",       ....       taints = {         dedicated = {           value  = "observability",           effect = "NoExecute"         }       }     }   }
Am I missing something?

Hello, I'm really interested in having this feature, but I fear that the current oci-terraform-provider doesn't support it given the following issue: https://github.com/oracle/terraform-provider-oci/issues/1504
is there any ETA for having it available in the provider and not having to do hacks with cloud-init stuff ?

Hi @thpham
I wish taints were easy to do in OKE too!
I've tried hacking with cloud-init (as discussed by Oracle) but had no luck - have you got a working solution for it?
My last attempt:
...

worker_pool = {
   overhead = {
    size             = 1,
    shape            = "VM.Standard.E4.Flex",
    ocpus            = 1,
    memory           = 4,
    boot_volume_size = 50,
    os               = "Oracle Linux",
    os_version       = "8",
    cloud_init = [
      {
        content = <<-EOT
        #!/bin/bash
        curl --fail -H "Authorization: Bearer Oracle" -L0 http://<IP_ADDRESS>/opc/v2/instance/metadata/oke_init_script | base64 --decode >/var/run/oke-init.sh
        bash /var/run/oke-init.sh --kubelet-extra-args "--node-labels=oke.oraclecloud.com/pool.name=overhead --register-with-taints=notfm-app=true:NoSchedule"
        touch /var/log/oke.done
        EOT
      },
    ]
  },

...

Hi @ajhindle
Have you set worker_disable_default_cloud_init to true in order to use custom cloud init script?
The following works for me
worker_disable_default_cloud_init = true

worker_pools = {
  np1 = {
    shape              = "VM.Standard.E4.Flex",
    ocpus              = 1,
    memory             = 4,
    size               = 1,
    boot_volume_size   = 50,
    kubernetes_version = "v1.29.1",
    cloud_init = [
      {
      content      = <<-EOT
    runcmd:
    - 'curl --fail -H "Authorization: Bearer Oracle" -L0 http://<IP_ADDRESS>/opc/v2/instance/metadata/oke_init_script | base64 --decode >/var/run/oke-init.sh'
    - 'bash -x /var/run/oke-init.sh --kubelet-extra-args "--register-with-taints=notfm-app=true:NoSchedule"'
    - 'touch /var/log/oke.done'
    EOT
      content_type = "text/cloud-config",
      }
    ]
  }
}

@rayeswong Solution worked for me above. Also note, I noticed that you will need to create new worker pools to get this enabled. Also, if you check in the console to see what the cloud init for a worker pool is, it will be base64 encoded. Hope this helps!
More details here

Thanks @rayeswong - worker_disable_default_cloud_init = true worked.
When this flag is set to true, I found that if I want to make node pool 2 without any taint or special kubelet arguments then I still need to explicitly declare boilerplate CloudInit like this, otherwise node pool 2 will fail.
  nodepool2= {
    size             = 1,
    shape            = "VM.Standard.E4.Flex",
    ocpus            = 2,
    memory           = 32,
    os               = "Oracle Linux",
    os_version       = "8",
    cloud_init = [
      {
      content      = <<-EOT
    runcmd:
    - 'curl --fail -H "Authorization: Bearer Oracle" -L0 http://<IP_ADDRESS>/opc/v2/instance/metadata/oke_init_script | base64 --decode >/var/run/oke-init.sh'
    - 'bash -x /var/run/oke-init.sh'
    - 'touch /var/log/oke.done'
    EOT
      content_type = "text/cloud-config",
      },
    ]
  }

setting worker_disable_default_cloud_init = true is like a global change. Suppose i am creating two node pools np1 and np2.
I want taint to be set for np2 only. The way i did it is something like this
  worker_pools = {
    np1 = {
      mode   = "node-pool",
      size   = 1,
      shape  = "VM.Standard.E4.Flex",
      create = true
    },
    np2 = {
      mode                     = "node-pool",
      size                     = 1,
      shape                    = "VM.Standard.E4.Flex",
      create                   = true,
      ocpus                    = 1,
      memory                   = 16,
      node_labels = {
        "project" = "projectA",
      },
      disable_default_cloud_init = true,
      cloud_init = [
        {
          content      = jsonencode({ timezone = "Etc/UTC" })
          content_type = "text/cloud-config",
          filename   = "10-timezone.yml"
        },
        {
          content = jsonencode({
            # https://cloudinit.readthedocs.io/en/latest/reference/modules.html#growpart
            growpart = {
              mode                     = "auto"
              devices                  = ["/"]
              ignore_growroot_disabled = false
            }

            # https://cloudinit.readthedocs.io/en/latest/reference/modules.html#resizefs
            resize_rootfs = true

            # Resize logical LVM root volume when utility is present
            bootcmd = ["if [[ -f /usr/libexec/oci-growfs ]]; then /usr/libexec/oci-growfs -y; fi"]
          })
          content_type = "text/cloud-config",
          filename   = "10-growpart.yml"
        },
        {
          content = jsonencode({
            write_files = [
              {
                content = module.k8s_infra.apiserver_private_host
                path    = "/etc/oke/oke-apiserver"
              },
              {
                content  = module.k8s_infra.cluster_ca_cert
                encoding = "base64"
                path     = "/etc/kubernetes/ca.crt"
              },
            ]
          })
          content_type = "text/cloud-config",
          filename   = "50-oke-config.yml"
          merge_type = "list(append)+dict(no_replace,recurse_list)+str(append)"
        },
        {
          content      = <<-EOT
          runcmd:
          - 'curl --fail -H "Authorization: Bearer Oracle" -L0 http://<IP_ADDRESS>/opc/v2/instance/metadata/oke_init_script | base64 --decode >/var/run/oke-init.sh'
          - 'bash -x /var/run/oke-init.sh --kubelet-extra-args "--register-with-taints=node-role.kubernetes.io/projecta-node:NoSchedule"'
          - 'touch /var/log/oke.done'
          EOT
          content_type = "text/cloud-config",
          filename   = "10-taints.yml"
        }

      ],
    }
  }

i did not set  worker_disable_default_cloud_init as a module input. This way np1 gets whatever the oke module has to offer in terms of cloud init and np2 will get oke base cloud init + taint.

Thanks @ddevadat  - disable_default_cloud_init = true works well for that use case (i.e. different cloud_init for different node pools)


Consider implementing getLogoutUrl

If you use the Google Client API, there is a function called getLogoutUrl() that allows you to revoke the current access token.  The azure third party client for the PHP League's oauth system has this function as well so it would make life easier if both microosft's and google's clients had this.

@netniV a PR implementing this would be welcomed.


Nothing happens upon taping the notification

[ ] Chat sample
[x] P2P Call sample
[ ] Conference Call sample
[x ] Android
[x ] iOS
Device info
Manufacture:
Model: Iphon 7plus , MI 5 A prime
OS version:
Upon receiving call notification , when i tap upon notification my app doesn;t awake plus notification disapears.

Sorry for the delayed answer. Do you reproduce this issue on our sample without any changes (except configs)? If yes, please add detailed steps on how can we reproduce the same?

no activity for a long time, so closing


Setup the Google Play store

The Google Play store needs to be setup so that we can expand the test group. This includes a description of the app, screen shots, and artwork. This can be changed at a later date.

@alexgmetcalf @knguyenBC FYI.

I can deal with the more visual elements of the this. @alexgmetcalf if you want to work with the description of the app.

@jleach Do the screenshots for the other devices (7 and 10 inch tablets) need to be different than the phone ones? Or can we just use the same ones?

@alexgmetcalf @jleach The graphical elements for this are done. Please review and let me know if there's any changes necessary.

Meeting with Jason at 9:30am Feb 8 to discuss.

@jleach Realized there were mistakes in the google play graphics so I've uploaded to more correct versions in the same folder.

See text in App Store ticket.

@jleach Do you want to send me a screenshot of the stores so I can review, or shall I just move this ticket to Done?

I'd just mark it as done. We still have two tickets to track the text changes, those are not done. But since we're using both iTunes and Google Play for distributing beta apps I think they're done.


management client does not support adding custom policies

Package Name: azure-core
Package Version: 1.13.0
Operating System: All
Python Version: All

Describe the bug
Although a PR has added the ability to add the custom policies when no specified policies are passed in through **kwargs to the PipelineClient.
PR link: https://github.com/Azure/azure-sdk-for-python/pull/17340
However, management client still does not support adding custom policies.
For example, the per_call_policies or per_retry_policies is passed in through **kwargs when building ResourceManagementClient. The reason for this issue is as follows:

In method __init__() of ResourceManagementClient, the ARMPipelineClient is created
https://github.com/Azure/azure-sdk-for-python/blob/602537021e0d5a94e0b61419e99cfa8052817c7e/sdk/resources/azure-mgmt-resource/azure/mgmt/resource/resources/_resource_management_client.py#L80

When creating ARMPipelineClient, if the specified policies are not passed in through **kwargs, the default policies will be added to **kwargs before building the pipeline
https://github.com/Azure/azure-sdk-for-python/blob/602537021e0d5a94e0b61419e99cfa8052817c7e/sdk/core/azure-mgmt-core/azure/mgmt/core/_pipeline_client.py#L44-L51

When building pipeline, only when there are no policies in **kwargs, there is logic to add custom policies. Since the default policies are passed in through **kwargs in step 2, so there is no logic to add custom policies.
https://github.com/Azure/azure-sdk-for-python/blob/602537021e0d5a94e0b61419e99cfa8052817c7e/sdk/core/azure-core/azure/core/_pipeline_client.py#L111-L117

Expected behavior
Could we consider that when building pipeline client, if the specified policies are passed in through **kwargs, these policies (in ARMPipelineClient, they are the default policies set by _default_policies) can also support appending custom policies?
ETA
Because this issue affects our migration of track 2 for resource module, and we plan to complete the merge of the migrated code by next Monday.
So this is an urgent issue and we hope to have it resolved by next Monday. Otherwise, we have to add custom policies by modifying private variables which do not conform to the code specification.

I think this functionality should help you. You can pass in policies with per_call_policies and per_retry_policies. This was added in version 1.13.0, so you guys may have to update your azure-core dependency.
cc @xiangyan99 @lmazuel for confirmation

I think the design that "only policies is not specified, we will honor custom policies" is correct.
In your case, I would suggest you supporting custom_policies in your _default_policies(config, **kwargs)
https://github.com/Azure/azure-sdk-for-python/blob/602537021e0d5a94e0b61419e99cfa8052817c7e/sdk/core/azure-mgmt-core/azure/mgmt/core/_pipeline_client.py#L54-L69
Does this make sense?

Fixed.


Created RichExecution.withSeparateCache and used it to solve oom er…

…rors related to Hive partition diff calculation.

Can you bump the minor version please. :+1:

This resolves #459

The squashed commit for this is: c18183d74ce9f2d978e8fcce9b44ec9ce2593d6f
It's not shown here, I guess because I pushed this to master before pushing to the PR.


"F" key Viewport focus problem

version -> http://godot3builds.digitecnology.com/builds/2017-12-06.12:52:48.a8ae46e14/godot-v3.0-osx.fat.zip
Issue description:
When i press F key, wrong viewport is active.
Steps to reproduce:
https://www.youtube.com/watch?v=y5K5xv7UBuo&feature=youtu.be

Also referred here - #12409 , as a more global issue, not only related with shortcut 'F'


log-cache-scheduler would not start on recreate

We recreated the scheduler VMs and failed to start the log-cache-scheduler with repeated failures of the following ...
2019/03/12 13:41:54.755751 Starting Log Cache Scheduler...
FIELD NAME:                    TYPE:          ENV:                      REQUIRED:  VALUE:
Config.HealthAddr              string         HEALTH_ADDR               false      localhost:6064
Config.Interval                time.Duration  INTERVAL                  false      15s
Config.Count                   int            COUNT                     false      100
Config.ReplicationFactor       int            REPLICATION_FACTOR        false      1
Config.NodeAddrs               []string       NODE_ADDRS                false      [14430b85-6375-4d7a-84ee-8459919a499d.doppler.default.yz-stage0.microbosh:8080 187d7f9c-8c09-4d6b-b23b-f7e3c1e8193a.doppler.default.yz-stage0.microbosh:8080 72a066d8-634e-4ffe-a758-6d9e05857534.doppler.default.yz-stage0.microbosh:8080 77aaee4d-7d84-45ba-a000-6e829d9058c3.doppler.default.yz-stage0.microbosh:8080 bd8cb129-a554-4d76-9a48-3f3108e8db84.doppler.default.yz-stage0.microbosh:8080 cd556ff2-111b-464f-ba26-73c3f560fcff.doppler.default.yz-stage0.microbosh:8080 d74c45fe-9fd9-4578-9727-bf2dcd76618c.doppler.default.yz-stage0.microbosh:8080 e56ca6ec-f875-4026-b11b-7bd375623668.doppler.default.yz-stage0.microbosh:8080]
Config.LeaderElectionEndpoint  string         LEADER_ELECTION_ENDPOINT  false
TLS.CAPath                     string         CA_PATH                   true       /var/vcap/jobs/log-cache-scheduler/config/certs/ca.crt
TLS.CertPath                   string         CERT_PATH                 true       /var/vcap/jobs/log-cache-scheduler/config/certs/log_cache.crt
TLS.KeyPath                    string         KEY_PATH                  true       /var/vcap/jobs/log-cache-scheduler/config/certs/log_cache.key
panic: runtime error: index out of range

goroutine 14 [running]:
code.cloudfoundry.org/go-orchestrator.(*Orchestrator).rebalance(0xc00010e500, 0xc0004405a0, 0xc000440570, 0xc0001981b0, 0xc000394280, 0x8)
    /var/vcap/packages/log-cache-scheduler/src/code.cloudfoundry.org/go-orchestrator/orchestrator.go:192 +0x4d0
code.cloudfoundry.org/go-orchestrator.(*Orchestrator).NextTerm(0xc00010e500, 0x988f60, 0xc00002a038)
    /var/vcap/packages/log-cache-scheduler/src/code.cloudfoundry.org/go-orchestrator/orchestrator.go:133 +0x160
code.cloudfoundry.org/log-cache/internal/scheduler.(*Scheduler).Start.func1(0xc00010e480)
    /var/vcap/packages/log-cache-scheduler/src/code.cloudfoundry.org/log-cache/internal/scheduler/scheduler.go:178 +0xd1
created by code.cloudfoundry.org/log-cache/internal/scheduler.(*Scheduler).Start
    /var/vcap/packages/log-cache-scheduler/src/code.cloudfoundry.org/log-cache/internal/scheduler/scheduler.go:172 +0x2e9

Finally resolved this by manually doing a restart of all the log-cache jobs on the dopplers.
This is on log-cache 2.0.2

@andrew-edgar Thanks for reporting this. Is this the first time you've seen this?
We think it might be something to do with the NODE_ADDRS array being out of sync with the actual addresses of the log cache nodes. Had you done any other re-deployments of the doppler VMs before this happened?

He mentioned that he checked that all the IPs resolved to correct doppler addrs. I think we can just add a error log somewhere we are log.Fataling.

@andrew-edgar we are about to release log-cache v1.4.8 with a fix for this and we will port this to the 2.0.x and 2.1.x line pretty soon.

That's cool thanks.

The release are now cut. Please let us know and reopen this issue if the problem reoccurs after the fix.
Thanks for reporting this.