Excessively long exposure times at Sunrise

For some reason the allsky app is completely ignoring my max exposure times and taking 1-1.5 hours to reach an appropiate exposure time every morning. My max night time exposure is 45secs and max daytime exposure is 10secs. I don't udnerstand why it is jumping to 300+ secs and taking a long time to adjust back down. I've tried adjusting the Aggression, auto gain on or off, tweaking other settings and nothing seems to have an impact on why it starts taking multiple minute long exposures  when going night to day.
Feb 12 06:08:17 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:08:17   @ 45.0 sec
Feb 12 06:09:04 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:09:04   @ 45.0 sec
Feb 12 06:09:50 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:09:50   @ 45.0 sec
Feb 12 06:10:36 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:10:36   @ 45.0 sec
Feb 12 06:11:22 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:11:22   @ 45.0 sec
Feb 12 06:12:09 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:12:09   @ 45.0 sec
Feb 12 06:12:55 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:12:55   @ 357.4 sec
Feb 12 06:19:09 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:19:09   @ 357.4 sec
Feb 12 06:25:22 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:25:22   @ 321.7 sec
Feb 12 06:31:00 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:31:00   @ 289.5 sec
Feb 12 06:36:06 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:36:06   @ 260.6 sec
Feb 12 06:40:43 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:40:43   @ 234.5 sec
Feb 12 06:44:54 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:44:54   @ 211.1 sec
Feb 12 06:48:41 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:48:41   @ 190.0 sec
Feb 12 06:52:08 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:52:08   @ 171.0 sec
Feb 12 06:55:15 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:55:15   @ 153.9 sec
Feb 12 06:58:06 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:58:06   @ 138.5 sec
Feb 12 07:00:41 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:00:41   @ 124.6 sec
Feb 12 07:03:02 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:03:02   @ 112.2 sec
Feb 12 07:05:10 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:05:10   @ 101.0 sec
Feb 12 07:07:07 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:07:07   @ 90.9 sec
Feb 12 07:08:54 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:08:54   @ 81.8 sec
Feb 12 07:10:32 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:10:32   @ 73.6 sec
Feb 12 07:12:02 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:12:02   @ 66.2 sec
Feb 12 07:13:25 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:13:25   @ 59.6 sec
Feb 12 07:14:41 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:14:41   @ 53.7 sec
Feb 12 07:15:51 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:15:51   @ 48.3 sec
Feb 12 07:16:55 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:16:55   @ 43.5 sec
Feb 12 07:17:55 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:17:55   @ 39.1 sec
Feb 12 07:18:50 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:18:50   @ 35.2 sec
Feb 12 07:19:42 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:19:42   @ 31.7 sec
Feb 12 07:20:30 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:20:30   @ 28.5 sec
Feb 12 07:21:15 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:21:15   @ 25.7 sec
Feb 12 07:21:57 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:21:57   @ 23.1 sec
Feb 12 07:22:36 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:22:36   @ 20.8 sec
Feb 12 07:23:13 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:23:13   @ 18.7 sec
Feb 12 07:23:48 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:23:48   @ 16.8 sec
Feb 12 07:24:21 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:24:21   @ 15.2 sec
Feb 12 07:24:53 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:24:53   @ 13.6 sec
Feb 12 07:25:23 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:25:23   @ 12.3 sec
Feb 12 07:25:51 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:25:51   @ 11.0 sec
Feb 12 07:26:19 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:26:19   @ 9.9 sec
Feb 12 07:27:04 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:27:04   @ 3.94 ms (0.00 sec)
Feb 12 07:27:20 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:27:20   @ 3.53 ms (0.00 sec)
Feb 12 07:27:36 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 07:27:36   @ 3.53 ms (0.00 sec)

I tried to disable the version 0.8 exposure method and that made things worse where every other frame was vastly overexposed. The exposure time was flipping between 200-250usecs to 3-4ms.

@havocedge   There's a bug in the current version where "max exposure" isn't really the max exposure.  I think it's the starting exposure instead.  It's fixed in our new release which should be out this month.

@EricClaeys Does that explain why the first image after the endofNight script runs is 6 minutes long?
Feb 12 06:16:12 allsky allsky.sh[17767]: endOfNight.sh: ===== Timelapse complete
Feb 12 06:16:12 allsky allsky.sh[17767]: endOfNight.sh: Deleting old directory /home/pi/allsky/images/20230128
Feb 12 06:18:54 allsky allsky.sh[17767]:   > Sleeping 15.0 sec from histogram exposure
Feb 12 06:19:09 allsky allsky.sh[17767]: STARTING EXPOSURE at: 2023-02-12 06:19:09   @ 357.4 sec
The max Daytime exposure in the config is set to 10 seconds.

@havocedge   It's supposed to never go longer than the maximum auto exposure the camera supports, which for most ZWO cameras is 200 seconds.  I'm pretty sure there's a bug there as well.  If you want to try the new release now, I can let you know how.  It's pretty stable, but if you elect to install it I'd ask you to install the official release when it's out - that will be very easy to do.

Ok I'm using an ASI385MC which has a max exposure time of 2000secs. I willing to try the new version.
Sensor: 1/2″ CMOS IMX385
Diagonal: 8.35mm
Resolution: 2.12Mega Pixels 1936×1096
Pixel Size: 3.75µm
Exposure Range: 32µs-2000s

@havocedge   Go to the Wiki link on any GitHub page; on the bottom of the table of contents, click on the "New documentation" link.  Look at the Installation-->Allsky page and follow the instructions with one exception, Add --branch dev to the git line, i.e., git --dev branch --clone ......
If you have the Allsky Website installed on your Pi or on a remote server, look at the Installation-->Website page.  Again, add --branch dev to the git line.

Here's the lines from this morning logs with debug increased.
Feb 13 06:11:56 allsky allsky.sh[30162]: xxx resetGainTransitionVariables(0, 180) called at DAY
Feb 13 06:11:56 allsky allsky.sh[30162]: Processing end of night data
Feb 13 06:11:56 allsky allsky.sh[30162]: ==========
Feb 13 06:11:56 allsky allsky.sh[30162]: === Starting daytime capture ===
Feb 13 06:11:56 allsky allsky.sh[30162]: ==========
Feb 13 06:11:56 allsky allsky.sh[30162]: Using the last night exposure, old and new Gain to calculate new exposure of 343.1 sec
Feb 13 06:11:56 allsky allsky.sh[30162]: Turning off daytime auto-exposure to use histogram exposure.
Feb 13 06:11:56 allsky allsky.sh[30162]: STARTING EXPOSURE at: 2023-02-13 06:11:56   @ 343.1 sec
My Night Time Gain is 180 and my Daytime is set to 1. Auto Gain is disabled for both. I will install the dev version today.

@EricClaeys The new dev version seems to have fixed the problem after tweaking the settings. This new version is a pretty significant upgrade. I like it a lot. I did find one odd issue. The REMOTE_PORT var in the ftp-settings.sh file is only used for FTP/SFTP. I switched to uploading via SCP with SSH keys and had to manually edit the upload.sh script to leverage the REMOTE_PORT var as I don't use the default SSH port of 22. I'm not sure why all methods that can specify a non-default port don't use this var.

@havocedge. Thanks for the feedback. We are pretty excited by this release.  The next one will be even better!
I don't use scp so did not know it accepted a port option.  I will fix that, thanks for letting me know.
Are there any other protocols that also accept it?

The fix for the "scp" protocol is now in the "dev" release.  Thanks for pointing it out.


how to get the generated queries output?

Issue with current documentation:
below's the code which is using multiqueryretrieval
%%time
# query = 'how many are injured and dead in christchurch Mosque?'

from langchain.retrievers.multi_query import MultiQueryRetriever
from langchain.chat_models import ChatOpenAI

# Define your prompt template
prompt_template = """Use the following pieces of information to answer the user's question.
If you don't know the answer, just say that you don't know, don't try to make up an answer.

Context: {context}
Question: {question}

Only return the helpful answer below and nothing else. If no context, then no answer.
Helpful Answer:"""

# Assuming `pdf_files` is a list of your PDF files
for pdf_file in pdf_files:
    # Load the PDF file
    loader = PyPDFLoader(pdf_file)
    document = loader.load()

    text_splitter = RecursiveCharacterTextSplitter(
                                               chunk_size=1000,
                                               chunk_overlap=200)

    # Split the document into chunks
    texts = text_splitter.split_documents(document)

    # Create a vector database for the document
    # vectorStore = FAISS.from_documents(texts, instructor_embeddings)

    vectorstore = FAISS.from_documents(texts, embeddings)

    # vectorstore = Chroma.from_documents(texts, embeddings)

    llm = OpenAI(temperature=0.2)

    retriever = MultiQueryRetriever.from_llm(retriever=vectorstore.as_retriever(), llm=llm)

    # docs = retriever.get_relevant_documents(query="how many are injured and dead in christchurch Mosque?")

    # print(docs)

    # Create a chain to answer questions
    qa_chain = RetrievalQA.from_chain_type(llm=llm,
                                           chain_type="stuff",
                                           retriever=retriever,
                                           return_source_documents=True)

    # # Use the chain to answer a question
    # llm_response = qa_chain(query)
    # process_llm_response(llm_response)

    # Format the prompt using the template
    context = ""
    question = "how many are injured and dead in christchurch Mosque?"
    formatted_prompt = prompt_template.format(context=context, question=question)

    # Pass the formatted prompt to the RetrievalQA function
    llm_response = qa_chain(formatted_prompt)
    process_llm_response(llm_response)

I just want to know what are the queries multiqueryretrieval generated. How do i get those?
Idea or request for content:
No response

🤖
Hey @nithinreddyyyyyy! Great to see you back here. Diving into another intriguing puzzle, I see? Let's get to it!
To retrieve the queries generated by the MultiQueryRetriever in the LangChain framework, you can use the verbose attribute of the SelfQueryRetriever class. When verbose is set to True, the generated query is logged using the logger.info function.
Here is the relevant code snippet:
if self.verbose:
    logger.info(f"Generated Query: {structured_query}")

This code is present in both the _get_relevant_documents and _aget_relevant_documents methods of the SelfQueryRetriever class. These methods are responsible for generating the structured query, preparing the query, and retrieving the documents relevant to the query.
To retrieve the generated queries, you can set verbose to True when creating an instance of SelfQueryRetriever and then check the logs for the "Generated Query" log messages.
Please note that you need to have your logging level set to INFO or lower to see these log messages. You can find this code in the base.py file of the self_query directory in the retrievers directory.
Additionally, the LangChain framework provides a method to directly access the generated queries from the MultiQueryRetriever. The method is called generate_queries and it takes a user query and a run manager as arguments. It returns a list of queries generated by the language model that are similar to the user input.
Here is the method:
def generate_queries(
    self, question: str, run_manager: CallbackManagerForRetrieverRun
) -> List[str]:
    """Generate queries based upon user input.
    Args:
        question: user query

    Returns:
        List of LLM generated queries that are similar to the user input
    """
    response = self.llm_chain(
        {"question": question}, callbacks=run_manager.get_child()
    )
    lines = getattr(response["text"], self.parser_key, [])
    if self.verbose:
        logger.info(f"Generated queries: {lines}")
    return lines

This method uses the llm_chain to generate the queries based on the user's input question. The generated queries are then logged (if verbose is set to True) and returned. You can find this method in the multi_query.py file of the retrievers directory.
I hope this helps! If you have any other questions, feel free to ask.
Sources

libs/langchain/langchain/retrievers/self_query/base.py
libs/langchain/langchain/retrievers/multi_query.py

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.

below's the code
%%time
# query = 'how many are injured and dead in christchurch Mosque?'

from langchain.retrievers.multi_query import MultiQueryRetriever
from langchain.chat_models import ChatOpenAI

# Define your prompt template
prompt_template = """Use the following pieces of information to answer the user's question.
If you don't know the answer, just say that you don't know, don't try to make up an answer.

Context: {context}
Question: {question}

Only return the helpful answer below and nothing else. If no context, then no answer.
Helpful Answer:"""

# Assuming `pdf_files` is a list of your PDF files
for pdf_file in pdf_files:
    # Load the PDF file
    loader = PyPDFLoader(pdf_file)
    document = loader.load()

    text_splitter = RecursiveCharacterTextSplitter(
                                               chunk_size=1000,
                                               chunk_overlap=200)

    # Split the document into chunks
    texts = text_splitter.split_documents(document)

    # Create a vector database for the document
    # vectorStore = FAISS.from_documents(texts, instructor_embeddings)

    vectorstore = FAISS.from_documents(texts, embeddings)

    # vectorstore = Chroma.from_documents(texts, embeddings)

    llm = OpenAI(temperature=0.2)

    retriever = MultiQueryRetriever.from_llm(retriever=vectorstore.as_retriever(), llm=llm)

    # Set logging for the queries
    import logging

    logging.basicConfig()
    logging.getLogger("langchain.retrievers.multi_query").setLevel(logging.INFO)

    docs = retriever.get_relevant_documents(query="how many are injured and dead in christchurch Mosque?")

    print(len(docs))

    # Create a chain to answer questions
    qa_chain = RetrievalQA.from_chain_type(llm=llm,
                                           chain_type="stuff",
                                           retriever=retriever,
                                           return_source_documents=True)

    # # Use the chain to answer a question
    # llm_response = qa_chain(query)
    # process_llm_response(llm_response)

    # Format the prompt using the template
    context = ""
    question = "how many are injured and dead in christchurch Mosque?"
    formatted_prompt = prompt_template.format(context=context, question=question)

    # Pass the formatted prompt to the RetrievalQA function
    llm_response = qa_chain(formatted_prompt)
    process_llm_response(llm_response)

and below's the output
INFO:langchain.retrievers.multi_query:Generated queries: ['1. What is the total number of casualties in the Christchurch Mosque attack, including both injured and deceased individuals?', '2. Can you provide the exact count of injured and deceased individuals in the Christchurch Mosque tragedy?', '3. How many people were affected by the Christchurch Mosque shooting, in terms of injuries and fatalities?']
6
INFO:langchain.retrievers.multi_query:Generated queries: ['1. How many casualties were there in the Christchurch Mosque attack?', '2. Can you provide the number of injured and deceased individuals in the Christchurch Mosque tragedy?', '3. What is the total count of injured and dead individuals in the Christchurch Mosque shooting?']
I don't know.
Sources:
/content/documents/2015_Crime_prediction_using_twitter_sentiment_and_weather_11.pdf
INFO:langchain.retrievers.multi_query:Generated queries: ['1. What is the number of casualties in the Christchurch Mosque attack?', '2. How many people were injured and killed in the Christchurch Mosque shooting?', '3. Can you provide the total count of injured and deceased individuals in the Christchurch Mosque tragedy?']
5
INFO:langchain.retrievers.multi_query:Generated queries: ['1. How many casualties were there in the Christchurch Mosque attack?', '2. Can you provide the number of injured and deceased individuals in the Christchurch Mosque incident?', '3. What is the total count of injured and dead individuals in the Christchurch Mosque tragedy?']

51 dead and 49 injured.

Sources:
/content/documents/2022_Crime_prediction_using_a_hybrid_sentiment_analysis_approach_10.pdf

INFO:langchain.retrievers.multi_query:Generated queries: ['1. What is the total number of casualties in the Christchurch Mosque incident, including both injured and deceased individuals?', '2. Can you provide information on the number of people who were injured and killed in the Christchurch Mosque attack?', '3. How many individuals were harmed or killed in the Christchurch Mosque tragedy?']
3
INFO:langchain.retrievers.multi_query:Generated queries: ['1. How many casualties were there in the Christchurch Mosque attack?', '2. Can you provide the number of injured and deceased individuals in the Christchurch Mosque tragedy?', '3. What is the total count of injured and dead individuals in the Christchurch Mosque incident?']
 I don't know.

Sources:
/content/documents/Pre-proposal PhD students - Nithin.pdf

CPU times: user 2.21 s, sys: 40.7 ms, total: 2.25 s
Wall time: 12.6 s

why it has generate 6 queries for one pdf file and which query will it select to get the output?


Unable to manipulate image

Issue Summary
Since the last ghost update (5.53.3), I'm unable to upload a gif (its size is 6MB) to my blog.
In my container logs, I have the error you see in the screenshot.
How can I configure the limitInputPixels or resolve this issue please ?
Steps to Reproduce

Adding a Gif via the Ghost Editor

Expected : Showing the gif in the blog like before

What Happened 👇🏾

In ghost editor

Ghost container Logs

Ghost Version
5.53.3
Node.js Version
16.20.1
How did you install Ghost?
Self-Hosted in a container on a Ubuntu 20 OS
Database type
MySQL 8
Browser & OS version
Brave & Mac OS Monterey
Relevant log / error output
No response
Code of Conduct

[X] I agree to be friendly and polite to people in this repository

How did you host your images? Directly? First of all, check permissions and ownerships, then if the script needs some special library in the container

@ngeorger If Ghost is throwing the error "Input image exceeds pixel limit" when there's a permissions problem, I'd say that's a bug in Ghost.

This seems to be due to the limit within Sharp - limitInputPixels: https://sharp.pixelplumbing.com/api-constructor
We'd probably consider accepting a PR that would make this configurable 🙂

How did you host your images? Directly? First of all, check permissions and ownerships, then if the script needs some special library in the container

It's not a permission issue, like @markstos stated, it's a bug in ghost

Also experiencing this issue with GIF's. When added:
"imageOptimization": {
  "resize": false
}

To my Ghost config, it allows me to upload GIF's. But obviously not ideal as it disables optimizations of all other non-GIF images.

Also experiencing this issue with GIF's. When added:
"imageOptimization": {
  "resize": false
}

To my Ghost config, it allows me to upload GIF's. But obviously not ideal as it disables optimizations of all other non-GIF images.

Does the difference "before/after" is huge for non-GIF images ?

For some context, the default for limitInputPixels is 268402689 pixels. Some conversions:

That's 268 Megapixels
If the image was square it would have dimensions greater than 16,000 pixels per side.
A high resolution 4k screen has dimensions of 4096 x 2160

In other words, it appears this limit is set quite high for typical images. GIFs are not a typical image because it's more a video format-- it contains many frames and thus many pixels.
The GIF format is also horribly inefficient. The format is so bad,  that when large website display "GIFs", they are not actually using the GIF format, but they have instead converted the files to a video format where they be compressed much better. Like,  possibly more than 90% better.
I'm not not sure it makes sense to adapt Ghost to better support this ancient and bloated format that works like video but is actually an image format.
Consider converting the image to another format like webp or mp4 before uploading it.
https://www.pixx.io/en/blog/gif-alternative

yes @markstos, I finally use mp4 format for video, but I compress it before uploading in my articles.

Hey @1kevinson , Please assign this issue to me.

This seems to be due to the limit within Sharp - limitInputPixels: https://sharp.pixelplumbing.com/api-constructor
We'd probably consider accepting a PR that would make this configurable 🙂

Thats one lazy Core Developer comment, right there...

Same here, using docker. Probably has to do with some unsupported operation by my vps cpu. Would it be possible to add an option to disable sharp completely?

Would it be possible to add an option to disable sharp completely?

@MyWay It's a modern blog system, it's fairly essential to generate images of different sizes-- for desktop and mobile. It's also a major convenience to have the blog system downsize a huge original photo to the right size that works.
Unless you have some specific logs to implicate a missing CPU feature, I doubt that's the issue in your environment.

Hey @1kevinson , Please assign this issue to me.

Unfortunately I cannot

Please assign this to me

@Sahil-K123 @Pratik1603 You don't have to wait for an issue to be assigned to you, just leave a comment that "I'm working on a PR" for this and start on one. Describe how you'd solve the problem if you'd like. Whether the issue is assigned to you or not, there's not guarantee that the pull request you submit later will accepted-- that will depend on the solution you submit and the quality of code.
By declaring that you are working on it, anyone else reading the issue can decide they want to wait for your PR to be done or pursue another solution for themselves.

Would it be possible to add an option to disable sharp completely?

@MyWay It's a modern blog system, it's fairly essential to generate images of different sizes-- for desktop and mobile. It's also a major convenience to have the blog system downsize a huge original photo to the right size that works.
Unless you have some specific logs to implicate a missing CPU feature, I doubt that's the issue in your environment.

No useful logs, unfortunately. I moved the exact same installation on another server (still with docker) and everything works fine.
I bet it's due to some cpu instruction needed by sharp, in my case (my issue wasn't depending on the size of the image, it was crashing for every image I tried to upload).

OK, I recommend closing this issue then, since it's not happening on your new server, and there no logs to confirm if might be a bug in Ghost or some environmental issue.


fix: add images which are needed post-upgrade into retaining list

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

Some of the pods like LH instance-manager will continue to use the old container image after the upgrade ended, such images should not be cleaned up. Otherwise, it could break the service especially for air-gapped environment.
Solution:

Find out those critical images and adding them into the retaining list so that they won't get purged.
Related Issue:
#5749
Test plan:

Build an ISO image containing this fix
Prepare a Harvester cluster (v1.3.0) in air-gapped environment
Start the upgrade
After the upgrade ends, check for the following images. Both the old and new versions should exist on the node
Check if any pod fails to run

@mergifyio backport v1.2 v1.3


Unexpected VSCode Intellisense Behavior in TypeScript with Object interface augmentation

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.83.1
OS Version:
Edition	Windows 11 Home
Version	21H2
Installed on	1/16/2022
OS build	22000.1098
Experience	Windows Feature Experience Pack 1000.22000.1098.0

When augmenting the interface for Object, augmentations are not shown as suggestions when the cursor is placed over a valid JavaScript object. This is certainly a very niche use case (most developers probably would be hesitant to prototype-pollute Object), but I believe it extends to simpler, more common use cases; such as getting a suggestion for .toString()
Steps to Reproduce:

Clone my repro repository: https://github.com/hubol/ts-intellisense-repro
Install the necessary dependencies using npm install (Should only install typescript!)
Open the repository in VSCode
Open the file index.ts, which looks like this:

interface Array<T> {
    myArrayAugmentation: number;
}

interface Object {
    myObjectAugmentation: number;
}

// Using the <Trigger Suggest> command (Ctrl-Space), I see .myArrayAugmentation suggested
// But not .myObjectAugmentation
const array = [].myArrayAugmentation

// Using the <Trigger Suggest> command, I do not see anything suggested
const object = {}.myObjectAugmentation

In index.ts, try creating a new array literal and getting a suggestion for one of the augmented properties: Type const array2 = [].my and you should see something like this:

You'll notice that myArrayAugmentation is correctly suggested, but not myObjectAugmentation

Now try creating a new object literal and getting a suggestion for myObjectAugmentation: Type const object2 = {}.my and you should see something like this:

And well... it did suggest it, but not in the way I would expect. It clearly is not detecting the type information for these properties for these suggestions.

If you mouse over the existing array and object variables in the file, you'll see that they have the correct type information. It is just that properties from the Object interface never seem to be suggested:

Hiding methods from Object in the completion list is 100% intentional, since for the vast majority of property accesses, you're not looking for e.g. hasOwnProperty


Solid Popover Component

I'm not sure this could be made much more readable by a mixin to handle the caret styling. I think this is probably easier to debug that if we put it all in too-clever if statements that might reduce the line count.


Re-organize documentation, add user guide

Closes #250
Re-organizes the documentation to be consistent with grantami-jobqueue. Adds a user guide with some general information about record lists and how the package can be used.

Minor changes, primarily based on guide style recommendations

@PipKat Thanks!


Gas fee problem

When using the minting dapp, Connect wallet > Mint > Metamask transaction pops up > The confirm transaction button is null and until you hit > Market and select a gas type low / med / high ... it will not allow you to make the mint/transaction without selecting the Market link?

Hi @hrnft, I think you enabled the new Gas Fees UI from MetaMask. We are not doing anything strange on the DAPP side... I'm afraid there is nothing we can do about it, that's a new thing and I think you can disable it from the settings.


build: ignore TestingMacrosTest on Windows

This relies on symbolic renaming which is not available on Windows. This allows us to build the test suite on Windows.

@swift-ci please test


Clarify rounding semantics

See https://github.com/gunnarmorling/1brc/pull/5#discussion_r1440578784; CC @AlexanderYastrebov

I find that Math.round((double) x / y) / 10.0 produces the same result as the reference - the important part is to do the division in double - addition is guaranteed to be loseless.
Also, I have redone the reference impl as BigDecimal with rounding half-even, and the results don't change (for my file).
CalculateAverage_BigDecimal.java.txt

Done via #392.

Done via https://github.com/gunnarmorling/1brc/pull/392.

Unfortunately no, see https://github.com/gunnarmorling/1brc/pull/392/files#r1451734941


Relative links on "build" with link_to helper?

How can we link pages with each other in relative way within generated projects. Currently the "link_to" helper doesn't computes relative urls, which would be very helpful.
The use case:
Project can have its structure with files in base folder and some subfolders as well. Let's imagine that all these files shares for example navigation partial.
When middleman generates project into static HTML, these pages aren't properly connected in relative way (by relative I mean ../ reference to parent folder).
Here is an example
Cheers

Sorry, I'm using 3.3.12 and I cant get the correct path into a subdirectory for images. For the resto of the assets the path is ok.
I get this URL for images:
"/images/dummy-image-01-6457c7c6.png"
When I need:
"images/dummy-image-01-6457c7c6.png"
I have this setting into build block:
activate :relative_assets
And outside the build block:
set :relative_links, true
Whats wrong?

are you using an image helper like image_tag?

Yes, but in a wrong way. I just discover it:
<%= tag :img, :src => "/images/dummy-slideshow-color.png" %>
I let in this way (without slash) and Boom!!:
<%= tag :img, :src => "images/dummy-slideshow-color.png" %>
Thanks tdreyno, with your help I could find it.  ;-)

Should use: <%= image_tag "dummy-slideshow-color.png" %>

Much better and much simple! ;-)
Thanks tdreyno,

If you want to do it with link_to as well, I was able to work around it with:
= content_tag :a, 'Home', :href => 'index.html'


ADD EMPLOYEE

ADD EMPLOYEE
Acceptance Criteria
GIVEN a command-line application that accepts user input
WHEN I choose to add an employee
THEN I am prompted to enter the employee’s first name, last name, role, and manager, and that employee is added to the database

Add employee


[Exiftool] Error: [minor] Will wrap JXL codestream in ISO BMFF container for writing

Solution: Use the -m option to ignore the minor error and allow ExifTool to wrap the JXL codestream in a BMFF container.
Source: How to add a comment to a JPEG XL image? - exiftool.org

Do you have an example file / URL on one of Grabber's source where this can be reproduced?

This is for literally any images converted to jxl.

Without the -m it doesn't change the metadata of the JXL files.

This is for literally any images converted to jxl.

It's now working fine with the https://github.com/Bionus/imgbrd-grabber/commit/fef4a9405dee2fb5110dbc569339be2cfe487588  nightly build.


Update fluentd example

This PR updates the fluentd sample that we have in the docs.  The current sample passes through the timestamp as a string which means you cannot do time-based queries like "before x" or "between x and y" or "in the last 15 minutes".
The PR updates the sample so that the special @timestamp field will be added, with type time and with the same value as the string timestamp.  This will allow users to do time based queries.

Looks good to me


Cover IR 76

The Link View Service operation shall be handled by the INSPIRE Discovery Service [INS DSTG].

Requirement for the Metadata ATS

Closed, nothing to do in the context of VS ATS


Remove all Vimeo controls

First of all great library. This isn't a bug as such but seeking clarification.
When using Plyr and Vimeo is it possible to get rid of all Vimeo controls. I've created a codepen, you'll noticed that when play is pressed the native Vimeo play button comes up in the center of the screen. Is it possible to hide or remove this?

Yes, it's now possible now, you can add this config on the fly:
new Plyr('player', {
  vimeo: {
    sidedock: 0,
    controls: 0,
  }
});

@sampotts would be good to implement it by default in Plyr

Yes, it's possible now, you can add this config on the fly:
new Plyr('player', {
  vimeo: {
    sidedock: 0,
    controls: 0,
  }
});

@sampotts would be good to implement it by default in Plyr
This is the latest list of iframe parameters:
https://vimeo.zendesk.com/hc/en-us/articles/360001494447-Using-Player-Parameters
P.S. Seems work on for videos, which hosted by a Plus account or higher. But still better than nothing.

Easy enough to add this. Will get it into the next release.

@attenzione Thanks for posting this! Fixed it for us. (Plyr v3.7.2).


DateTimeField validation error

Minimal working example:
from wtforms import DateTimeField, Form
from wtforms.validators import DataRequired
from wtforms_json import MultiDict

class MyForm(Form):
    timestamp = DateTimeField(validators=[DataRequired()])

data = {'timestamp': 'asd'}
formdata = MultiDict(data)

form = MyForm(formdata=formdata)
success = form.validate()

print('Success:', success)
print('Errors:', form.errors)
print(form.timestamp.data)

Execution environment: Docker container, image based on python:3.6.
Output:
Success?,  False
Errors: {'timestamp': ['This field is required.']}
None

You can see there is the wrong error message. It should mention wrong date format, not the missing field.

Thats because you're trying to validate a type string as a Datetime. The field doesn't accept the input so the form validation ends up being an empty field.

No, the field accepts the string and transforms it to the datetime.datetime during validation.
If we transform the data into the following format: data = {'timestamp': '2018-04-13 10:03:08'}, it works fine:
Success: True
Errors: {}
2018-04-13 10:03:08

On master the behavior is this:
>>> import wtforms
>>> class MyForm(wtforms.Form):
...     timestamp = wtforms.DateTimeField()
>>> MyForm(timestamp='2018-04-13 10:03:08').validate()
True
>>> MyForm(timestamp="asd").validate()
True

The last case should raise an exception.

I could reproduce with:
>>> import wtforms
>>> class F(wtforms.Form):
...     timestamp = wtforms.DateTimeField(validators=[wtforms.validators.DataRequired()])
...     F(data=dict(timestamp='asd')).validate()
True


Lib broken on Python 3.12

py_grist_api uses python-future which is broken on Python 3.12 because imp module was removed. Even though this issue was fixed a month and half ago, no release fix has been published yet.
Is  python-future still necessary? Support for Python 2.7 has ended nearly 3 years ago now and the oldest still supported Python version is 3.8.

I just ran into this now with Python 3.12.  For what it's worth it seemed OK when I just blindly commented out those imports and the standard_library.install_aliases() line, but I actually ended up just using the json and requests packages directly, since that was so simple for read-only access.  (This repo feels like an example for how you might edit tables within a document over the API, but it doesn't seem to give you anything over using the API directly when it comes to other use cases.)


Documentation: Databases

Clear and concise description of the problem
The sqlite database woodpecker uses by default (especially with helm installs) does not handle multiple incoming webhooks well.
Case in point, if I made multiple commits to different repos within a minute, I'd get some tasks failing to start with 'Database is locked' in the server log.  They would not show up as pending jobs.
Running fewer, or re-running them would work without issues, provided concurrency was low.
Suggested solution
It would be valuable to include a note in the documentation encouraging setup with a real database for anything beyond an initial prototype - I would even say that starting with a real database saves the efforts required in later migration and the associated hassles.
Alternative
Alternatively, the helm chart/docker examples could come with mysql/postgresql configured out of the box, but that puts the onus on your side, as well as increasing the installation requirements.
Additional context
No response
Validations

[X] Checked that the feature isn't part of the next version already [https://woodpecker-ci.org/faq#which-version-of-woodpecker-should-i-use]
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

That might also be sth we can try to solve in code 😅  (Just a short brainstorm from what I had in mind) I guess this is related to the way we create pipeline entries. To be able to have an increasing pipeline number we first do sth like select count(id) from pipelines where repo=... and probably lock the table til we created that entry.

SQLite evolves. Now it's better than before.
Grafana works well with SQLite. So, problem in Woodpecker.

Database is locked

#1905, #2827


random: Clarify the semantics of TEST_RANDOM_GENERATOR

The TEST_RANDOM_GENERATOR Kconfig symbol indicates that a non-random
number generator (i.e. a random number generator that is not truly
random) is allowed to be used for testing purposes, and not
necessarily that a non-random number generator is selected -- that is
indicated by the RNG_GENERATOR_CHOICE choices that depend on
TEST_RANDOM_GENERATOR (i.e. only TIMER_RANDOM_GENERATOR as of now).
This commit updates the TEST_RANDOM_GENERATOR Kconfig symbol
description to reflect and clarify that.
It also removes the TEST_RANDOM_GENERATOR's dependency on
ENTROPY_HAS_DRIVER because the act of allowing a non-random number
generator to be used does not depend on the availability of an entropy
driver -- when an entropy driver is available, by default, the
ENTROPY_DEVICE_RANDOM_GENERATOR will be selected; otherwise,
TIMER_RANDOM_GENERATOR, a non-random generator, will be selected.

Closes #53019

@ceolin @joerchan could you please take a look?

The bluetooth IPSP sample is an example of this being enabled where it should not have been.

@joerchan If you are referring to samples/bluetooth/ipsp, I am not sure if I understand the rationale. That sample runs on qemu_x86 and qemu_cortex_m3 and therefore requires CONFIG_TEST_RANDOM_GENERATOR=y.
In general, tests and samples should set CONFIG_TEST_RANDOM_GENERATOR=y so that they can run on a wide range of devices (i.e. on the devices without an entropy device) -- that is what CONFIG_TEST_RANDOM_GENERATOR is for after all. Even if CONFIG_TEST_RANDOM_GENERATOR=y, the test/sample will run with a true RNG if an entropy device is available.
The NCS samples are rather a special case in the sense that they only need to run on the nRF devices, which comes with an entropy device, and not any other devices, in which case you can get away with CONFIG_TEST_RANDOM_GENERATOR=n.

The bluetooth IPSP sample is an example of this being enabled where it should not have been.

@joerchan If you are referring to samples/bluetooth/ipsp, I am not sure if I understand the rationale. That sample runs on qemu_x86 and qemu_cortex_m3 and therefore requires CONFIG_TEST_RANDOM_GENERATOR=y.
In general, tests and samples should set CONFIG_TEST_RANDOM_GENERATOR=y so that they can run on a wide range of devices (i.e. on the devices without an entropy device) -- that is what CONFIG_TEST_RANDOM_GENERATOR is for after all. Even if CONFIG_TEST_RANDOM_GENERATOR=y, the test/sample will run with a true RNG if an entropy device is available.
The NCS samples are rather a special case in the sense that they only need to run on the nRF devices, which come with an entropy device, and not any other devices, in which case you can get away with CONFIG_TEST_RANDOM_GENERATOR=n.

This is a bluetooth sample. It should be able to use the entropy of the bluetooth controller ENTROPY_BT_HCI.

The sample itself does not have any entropy requirements, so not sure it needs it.

sys_rand_get requires a RNG and that function is used at least once in that sample. For instance, building with CONFIG_TEST_RANDOM_GENERATOR=n results in build failures for qemu_x86:
/opt/zephyr-sdk-0.15.2/x86_64-zephyr-elf/bin/../lib/gcc/x86_64-zephyr-elf/12.1.0/../../../../x86_64-zephyr-elf/bin/ld.bfd: zephyr/subsys/net/ip/libsubsys__net__ip.a(net_if.c.obj): in function `sys_rand32_get':
/home/stephanos/Dev/zephyrproject/zephyr/twister-out/qemu_x86/zephyr/samples/bluetooth/ipsp/sample.bluetooth.ipsp/zephyr/include/generated/syscalls/rand32.h:32: undefined reference to `z_impl_sys_rand32_get'

I will leave the sample as is for now.

Ok. It could still work with the BT entropy source though.
cp ./tests/drivers/entropy/api/entropy_bt_hci.overlay ./samples/bluetooth/ipsp/entropy_bt_hci.overlay
west build -b qemu_x86 samples/bluetooth/ipsp -- -DCONFIG_TEST_RANDOM_GENERATOR=n -DDTC_OVERLAY_FILE=./entropy_bt_hci.overlay

Leaving it as is is OK with me though. Adding overlay files is a very complicated way to get an entropy source in your application.


Cross-arch IRQ priority get/set

I'd like Zephyr to have, in include/irq.h:
 unsigned int irq_get_priority(unsigned int irq);
 int irq_set_priority(unsigned int irq, unsigned int priority);

So one can get and set the priority of an interrupt without having to resort to arch-specific code (eg. NVIC_SetPriority() which to make matter worse doesn't take into account the offset introduced by the kernel).

In addition of this, it would also be very valuable if there's a generic API for affining interrupts to particular CPUs. Unlike the priority API, the get_irq_affinity / set_irq_affinity will only be applicable for SMP / AMP scenarios..

@carlescufi While this may not be exactly what you describe, dynamic IRQ priority configuration (i.e. irq_set_priority equivalent) can be done in an arch/intc-agnostic manner through irq_connect_dynamic.

@carlescufi While this may not be exactly what you describe, dynamic IRQ priority configuration (i.e. irq_set_priority equivalent) can be done in an arch/intc-agnostic manner through irq_connect_dynamic.

@stephanosio yeap, we've discussed that, too. I suppose @carlescufi would like to have this option for statically-configured IRQ lines (i.e. not dynamic), as well. This is, currently supported for Cortex-M via a kernel-private API, so I guess, the point is whether to expose this as a public API, and how exactly we can do it. (Might be desired, as well, for direct IRQs, @carlescufi , is this correct?)
Before going with an implementation, I 'd like to see if there are any hidden dangers of exposing this functionality to the app developer.

I suppose @carlescufi would like to have this option for statically-configured IRQ lines

IRQ_CONNECT accepts the priority parameter, so one can statically configure IRQ priority in arch/intc-agnostic manner too.
If the goal is to dynamically re-assign the IRQ priority of an interrupt (both static and dynamic) that is already configured (can't really think of any sane usage cases for this though), one could do that using irq_connect_dynamic.

OK, so let me re-phrase this: this API should give the option to a user to dynamically change the IRQ priority of a non-dynamic IRQ.

If the goal is to dynamically re-assign the IRQ priority of an interrupt (both static and dynamic) that is already configured (can't really think of any sane usage cases for this though), one could do that using irq_connect_dynamic; just need to irq_disable the particular interrupt before doing so (as required by z_isr_install.

This is doable but you need to have CONFIG_DYNAMIC_INTERRUPTS enabled (correct?) - I suppose the point here is to be able to change IRQ priorities, without enabling this Kconfig, which affects memory footprint and run-time performance.

This is doable but you need to have CONFIG_DYNAMIC_INTERRUPTS enabled (correct?)

That is indeed correct.

I suppose the point here is to be able to change IRQ priorities, without enabling this Kconfig, which affects memory footprint and run-time performance.

Ok, that makes more sense. The rationale given above was "So one can get and set the priority of an interrupt without having to resort to arch-specific code," so I assumed the primary goal was to provide an arch-agnostic way of configuring IRQ priority, which is already supported.
I am still not sure how practical and/or useful this API would be; also whether this would be considered a good practice. The question I would like to ask is: when would one need to re-configure IRQ priority on-the-fly?

I am still not sure how practical and/or useful this API would be; also whether this would be considered a good practice. The question I would like to ask is: when would one need to re-configure IRQ priority on-the-fly (other than connecting a new dynamic IRQ, that is)?

i agree, @carlescufi needs to answer this question, since he has requested this enhancement :)
(My assumption is that you want to have as minimal overhead as possible (i.e. CONFIG_DYNAMIC_INTERRUPTS=n), use the standard ISR wrapper, and have the option to just change the IRQ priority on the fly)
IMO, adding an additional API, suitable only for rare use-cases, is not a big deal: as I said above, it feels more important to investigate whether the usage of such API will not have negative impact(s) to the kernel.

I think it should be possible to reconfigure priority even for static build-time interrupts.
I'd need to look at the code further, but on x86 IIRC the priority is used to determine the position in the IDT for the installed interrupt, it's not handled by the APIC. It's been a long time since I looked at this in detail though (and I was never fully clear how it worked). But it should be possible to write a little code to move entries to different IDT vectors if we want to, but we'll have to make sure the IDT is in RAM and not ROM.

@stephanosio yeap, we've discussed that, too. I suppose @carlescufi would like to have this option for statically-configured IRQ lines (i.e. not dynamic), as well. This is, currently supported for Cortex-M via a kernel-private API, so I guess, the point is whether to expose this as a public API, and how exactly we can do it. (Might be desired, as well, for direct IRQs, @carlescufi , is this correct?)

That's correct @ioannisg. Interrupt priority in DT or in Kconfig or in the code should be the initial value, but why not be able to query it and modify it later?

I am still not sure how practical and/or useful this API would be; also whether this would be considered a good practice. The question I would like to ask is: when would one need to re-configure IRQ priority on-the-fly (other than connecting a new dynamic IRQ, that is)?

I'm not sure what @carlescufi had in mind, but here is a practical use case:

As a mechatronic device developer, I need to build one FW image that supports the series of prototypes along with the minor variants that exist, so that I can minimize the variability in the results of testing done by other engineers.
As a result, the FW needs to determine the electro-mechanical prototype revision and variant at "start" time and adjust behavior as appropriate.  This adjustment may require lowering the interrupt priority of a hardware source for later revisions (e.g., EVT requires a servo to run at 4KHz, while DVT needs the servo to run at only 700Hz).  The electro-mechanical prototype revision may only be discoverable through exercising a mechanical sequence.

Now this introduces complications in non-interrupt code accessing data shared with the interrupt code.  In order to keep the periodic interrupt code running with minimal latency impact, the non-interrupt code must lock out all interrupts at the priority of the periodic interrupt, do the minimal necessary access, and restore the previous interrupt lockout level.  When the priority of the interrupt changes dynamically, it is more difficult to determine what the interrupt lockout priority needs to be.
The required logic is a bit more complicated than described above, but it points out the critical constraints.

Just saw this come by.  FWIW: the reason this API doesn't exist is because the concept isn't easily portable.  Every architecture does priority assignment differently, and most interrupt controller drivers have their own schemes that would need to be integrated.  Some systems like Xtensa have hard-wired priorities that can't be configured at all (though in many cases the systems then include a second level interrupt controller that does have features like this).  Systems like x86 encode the priority in the vector number, so an API like the one requested would have to decouple vectors from "interrupts" in a recoverable way (i.e. have some kind of internal table to do the mapping).  It gets even weirder for setups like MSI, where the "priority" is encoded in the target vector and thus involves the driver generating the interrupt (i.e. if you want to change your USB xhci interrupt priority, you need to talk to the xhci controller!).
Add to that the fact that most systems are able to emulate part of this (via masking off interrupts at shared priorities inside the ISR so that they can only interrupt each other in known ways) and things get harder still.  All that code has size and performance overhead.  Do we enable it for all these platforms just because some of them want runtime-mutable interrupts?
I'm not saying that no clean API is possible, just that it requires a bunch of thought that no one has attempted yet.  And IMHO it doesn't seem like a huge hardship to have platform-local APIs for problems like this.  Interrupt priority tuning is always going to be low-level and probably device-specific anyway.


Add grant access to camera functions

Project settings updated to Xcode 8.3
Grant access to camera functions added

Thank you 👍


Textmate doesn't understand python typehints

Though semantic token highlighting is able to pick up some elements from quoted type annotations, overall string is not marked as a typeHint:

Ideally, it would be nice to have the same scopes for quoted typehints as a regular ones, but with a proper "typeHint" modifier:

VS Code version: Code 1.67.2 (Universal) (https://github.com/microsoft/vscode/commit/c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:20:57.384Z)
OS version: Darwin arm64 21.5.0

@Bobronium is there a reason that you want the textmate scope to reflect that it's a typehint instead of the semantic token?

@alexr00, good question! Essentially I'm coming from the point that typehints have their own syntax.
Semantic token surely can add context to expression in that syntax, like saying if something is a builtin type or not, which it already does.
But from my perspective, dealing with plain syntax should be in textmate scope.


Mod Conflict marking error

When conflict manager is on,
It should be un-marked when it is disabled, but it still marked.

Enable mod
Disable mod in ordering space in right side.

It happened in "Stellaris >= 2.4" and I am running on Windows.

Is it still appears in the right column?

no. disappeared in the right column but still marked in left column

So it might be only a visual bug.


Is map type unordered?

Spec suggests encoding json as an example and json is unordered, however it doesn't explicitly states this.
Ordered hash maps would complicate implementation for some languages, like go(map) or python(dict).
copy from https://github.com/antirez/RESP3/issues/30

Let me point out Python dict is specified as ordered (since Python 3).

@dumblob see this comment https://github.com/antirez/RESP3/issues/30#issuecomment-557050925


Revert "Revert "Corrected terminology and other things""

Reverts MicrosoftDocs/sql-docs#6998

@SyntaxC4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Fetch wrong instance

I need to do this trick to avoid this error: 'intl-tel cannot read property '3' of undefined'
It fetch a wrong instance in the data of the element. Any help?
trick:
// get the data element
var d = angular.element(tel).data("plugin_intlTelInput");
// check that the countryCodes not be null
if ((telNumber != null) && !_.isUndefined(d.countryCodes)) {
return angular.element(tel).intlTelInput("setNumber", telNumber);
}

I'm afraid I'm going to need much more information to be able to help. Before you talk about your trick to avoid the error, can you describe the problem you're having? And how I can reproduce it?

I try to create a plunker to reproduce the problem. Anyway I have the problem trying to set the phone number multiple times (angular.element(tel).intlTelInput("setNumber", telNumber);). Probably is my environment that bring your component to works in the wrong way. Thank you.

Closing due to inactivity.


Add documentation to explain how to use a STREAM sender.

Tutorial/Walk-through in Quilt Wiki

Tell person to go xpring.tech to generate ILP account (this will provide them an outgoing bearer token to perform ILP-over-HTTP with the Rust connector running in the Xpring ILP testnet).
Show java code should show how to call sendMoney from one account to another.

Closing this as done. @nhartner keep me honest.


testing(run): Update all HTTP1 testing to new retry facility

Description
This change updates remaining Run sample e2e tests to use the cloudrunci retry facility.
Exceptions have more manual retry implementations (gRPC and h2c).
Checklist

[x] I have followed Contributing Guidelines from CONTRIBUTING.MD
[x] Tests pass:   go test -v ./.. (see Testing)
[x] Code formatted:   gofmt (see Formatting)
[x] Vetting pass:   go vet (see Formatting)
[ ] These samples need a new API enabled in testing projects to pass (let us know which ones)
[ ] These samples need a new/updated env vars in testing projects set to pass (let us know which ones)
[ ] This sample adds a new sample directory, and I updated the CODEOWNERS file with the codeowners for this sample
[ ] This sample adds a new Product API, and I updated the Blunderbuss issue/PR auto-assigner with the codeowners for this sample
[x] Please merge this PR for me once it is approved

Tests passed (https://fusion2.corp.google.com/invocations/8825d1c7-3a5c-48bc-8d9d-cd8b279bb9b2/targets/cloud-devrel%2Fgo%2Fgolang-samples%2Fpresubmit%2Flatest-version;config=default/log). Not going to wait for Kokoro.


Possibility to display a dropdown menu centered

This is updated PR related to changes described here: https://github.com/AhmedLSayed9/dropdown_button2/pull/180

Thanks!


Question about training from pretrained model for reranking

Hi, I noticed that in Line 2 of train_reranking.sh, the model does not load the weights trained in the global retrieval part. Does it mean that the reranking model is trained by the pre-trained weight of the backbone? Thank you!

You may check the --resume. It loads the model.

Thanks for your reply, but I mean Line 2 of train_reranking.sh does not have --resume, which means it does not load the global retrieval model weights as I understand. Do I misunderstand something?

Is that the size of the dataset you're  @xjh19971

Is that the size of the dataset you're useing @xjh19971

Yes

I have followed the training configuration provided by the author twice for training, but the accuracy now does not meet the accuracy provided by the author in  paper.the first time in A800 R@1: 87.4, R@5: 93.8, R@10: 94.5, R@20: 95.4, R@100: 97.0,the second time in RTX3090 R@1: 84.5, R@5: 91.5, R@10: 93.2, R@20. 93.8, R@100: 95.5 What is the accuracy you have trained? Did the author use data augmentation when training the backbone? @xjh19971

Thanks for your reply, but I mean Line 2 of train_reranking.sh does not have --resume, which means it does not load the global retrieval model weights as I understand. Do I misunderstand something?

Yes, it has the --resume command. Please check the script file carefully.

@Jeff-Zilence thanks for your answer!

@LKELN I get a similar performance with you but have not fine-tuned (the second command in train_reranking.sh).


[indexer] Modify PruningOptions to point to a toml file of epochs_to_keep and optional per-table overrides

Description
Modify PruningOptions to contain a config field that points to a toml file that must specify epochs_to_keep and an optional [overrides] portion of per-table overrides.
Test plan
How did you test the new or updated feature?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:
[ ] REST API:

Whoa.... how'd that happen


Can not build or serve code. Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

I am trying to npm install and it fails by giving following main error

<EMAIL_ADDRESS>install /Users/zulfiqaralisoomro/Documents/Projects/ago-assistant/node_modules/fsevents
node install

node-pre-gyp ERR! Tried to download(404): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.2/fse-v1.1.2-node-v67-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v67 ABI) (falling back to source compile with node-gyp)
node-pre-gyp ERR! Tried to download(undefined): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.2/fse-v1.1.2-node-v67-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v67 ABI) (falling back to source compile with node-gyp)
What is the fix for this?

hi @soomro30 could you give a node -v and npm -v output so we can check? And share your OS version?

node -v = v11.3.0
npm -v  = 6.4.1
My OS Version is MacOS Mojave 10.4

I would suggest the following:

try npm cache clean and reinstall
try re-cloning the repository to a new folder and re-npm installing

hi @soomro30 could you give a node -v and npm -v output so we can check? And share your OS version?

I would suggest the following:

try npm cache clean and reinstall
try re-cloning the repository to a new folder and re-npm installing

Hi Slibby,
I tried doing again a clone and re npm installing but still the below error:
node-pre-gyp ERR! Tried to download(undefined): https://fsevents-binaries.s3-us-west-2.amazonaws.com/v1.1.2/fse-v1.1.2-node-v67-darwin-x64.tar.gz
node-pre-gyp ERR! Pre-built binaries not found for<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(node-v67 ABI) (falling back to source compile with node-gyp)

@soomro30 I think it must be either an environment-specific issue or missing a compiler of some kind. I would suggest trying to install fsevents by itself to figure out if the issue is specific to this application or the module.

Hi @soomro30 I also have this issue, did you figure out how to solve it? Thank you!


Add More +CR Bonuses

The field should support common Raid leaders (and combinations thereof) instead of arbitrary numbers.

The code actually describes the approach as "cheese".  In the file, it's currently stored as 1 = 15, 2 = 30, and 3 = 45.  We probably need to introduce a file version so we can migrate the indexes to proper, natural values.

It may make sense to not fix this and instead implement CR buff and multiple leaders.  An additional leader could be used to simulate any other odd cases (like elemental advantage).


Please add a license

Thanks for providing the code on github.  Can you please add a license?

Hi,
the README already mentioned MIT license, now I explicitly added a LICENSE file with MIT license.
Please also note that parts of the code are based on Tensorpack (https://github.com/tensorpack/tensorpack/tree/master/examples/FasterRCNN) and a signle file for logging is based on RETURNN (https://github.com/rwth-i6/returnn/blob/master/Log.py).


Replace rusoto with custom implementation (#2176)

Which issue does this PR close?

Closes #2350
Part of #2176
Rationale for this change

See #2176
What changes are included in this PR?

Replaces rusoto with a custom implementation based on reqwest
Are there any user-facing changes?
Not that I'm aware of, but there is a possibility

Looks like I found a bug in the azure implementation :sweat_smile:

https://github.com/influxdata/influxdb_iox/pull/5342 has been deployed and appears to be working correctly, testing both the Instance Metadata credentials and the WebIdentity

I would like to request holding off merging this PR until we have done the official 0.4 object store release from this repo, tracked by https://github.com/apache/arrow-rs/issues/2338
My rationale is that there are enough moving parts and we are (effectively) still testing some of this code in production so I don't want to accidentally introduce a regression that would require a new release quickly after the previous one

Sounds like the IOx 'testing in production' has gone very well ❤
So to move this PR forward:

It seems like like there are a few more comments from @crepererum  to address.
The 0.4.0 release is going along nicely,
I think we are on track to merge this over the weekend or early next week

I intend to get this in this afternoon unless anybody would like longer to review

I don't have any objections, though I haven't reviewed this PR in detail either. I will confirm we deployed these changes in our IOx environment and they seem to be working great
cc @roeap

I don't have any objections, though I haven't reviewed this PR in detail either. I will confirm we deployed these changes in our IOx environment and they seem to be working great
cc @roeap


Nested list unity lag

I'm using a nested reorderable list within a reorderable list for my AI editor, something like this:

public ActionTrigger[] actions;

[Serializable]
public class ActionTrigger {
     [Reorderable(paginate = true, pageSize = 1)] public ActionSetList actionSetList;
}

[Serializable]
    public class ActionSet {

}

When there are 4+ elements within the nested list, it starts to lag unity significantly. I've tried to tackle this somewhat by enabling paging, but it's still noticeable.
Is there a way to address this?

Does the issue still occur if you create a CustomEditor for your Actions class?
i.e manually create the ReorderableList?
Also, have you tried profiling the editor to see where the slowdown might be occuring?
If you want to post the full code for your Actions class I can take a look as well

Figured out this was due to all the conditional hiding of attributes I was doing in the nested list. Closing this out, sorry!


gke: Not possible to detach getpeername{4,6} BPF progs on <= 5.4.89+ kernel

The backport [1] of "bpf: Add get{peer, sock}name attach types for sock_addr" didn't append the new attach types to bpf_prog_detach() which results in a failure when detaching a prog of any of the types. E.g.:
bpf(BPF_PROG_DETACH, {target_fd=3</run/cilium/cgroupv2>,
    attach_type=0x1d /* BPF_??? */}, 120) = -1 EINVAL (Invalid argument)

This became visible when we disabled the kube-proxy-replacement feature on Cilium which was running on GKE COS 5.4.
[1]: https://cos.googlesource.com/third_party/kernel/+/b6b28282e02f4f816346136a94de601bacf91a17
A patch has been provided to the Google kernel team.

If you get hit by that, AFAIK the only solution is to reboot the node.

Just tested - the issue is still present :disappointed:

@jtaleric reported that this is still present today.


Strict ident convert

This is a follow-up to https://github.com/Snapchat/djinni/pull/58 and the rational didn't change.
The solution, however, did.
There, I introduce so-called strict ident styles, that is, normal ident style suffixed with an exclamation-mark.
Specifying ident-style "FooBar!" will apply the CamelCase ident-style strictly, i.e. it will convert snake_case identifiers as usual, but ALL_CAPS will be converted correctly too.

Change looks good.  There is a build failure though.

One question, what is the motivation of using full caps in djinni files instead of just all_snake_case and let the the ident style take care of each language?

The motivation is 2 folds:

avoid surprising behavior: we write djinni files in CamelCase and everything happens as if no transformation is done, except for constants that we write in all caps.
having the exact same identifiers across languages allows easier grep/find. If we use all_snake_case for constant, that will be the only place in our repo where the identifier is written like that.

I'll fix the test later.


Fix thumbnail image

Fixes #762
Fixes a bug in some cases the search grid images loading the full res image and not the thumbnail

Developer Certificate of Origin Version 1.1
Copyright (C) 2004, 2006 The Linux Foundation and its contributors. 1 Letterman Drive Suite D4700 San Francisco, CA, 94129
Everyone is permitted to copy and distribute verbatim copies of this license document, but changing it is not allowed.
Developer's Certificate of Origin 1.1
By making a contribution to this project, I certify that:
(a) The contribution was created in whole or in part by me and I have the right to submit it under the open source license indicated in the file; or
(b) The contribution is based upon previous work that, to the best of my knowledge, is covered under an appropriate open source license and I have the right under that license to submit that work with modifications, whether created in whole or in part by me, under the same open source license (unless I am permitted to submit under a different license), as indicated in the file; or
(c) The contribution was provided directly to me by some other person who certified (a), (b) or (c) and I have not modified it.
(d) I understand and agree that this project and the contribution are public and that a record of the contribution (including all personal information I submit with it, including my sign-off) is maintained indefinitely and may be redistributed consistent with this project or the open source license(s) involved.

@akmadian the image data size depends more on the API than on the frontend. In production the MET thumbnails seem to be much smaller. I still think this quick fix is valid.


Including postcss-rtlcss with Angular12 with SCSS base styles later compiled to CSS

I am trying to use this library in my Angular 12 project with a custom webpack. I am getting this error: "Module build failed (from ./node_modules/sass-loader/dist/cjs.js): SassError: Invalid CSS after "i": expected 1 selector or at-rule, was 'import api from "!.' on line 1 of src/app/... from sass-loader/dist/cjs.js file"
module.exports = (config, options) => {
    addRtlConfig = (config) => {
        return {
            ...config, *//has all the existing default configs*
            module: {
                ...config.module,
                rules: [
                    ...config.module.rules,
                    {
                    test: /\.scss$|\.sass$/,
                      use: [
                        'style-loader',
                        'css-loader',
                        {
                          'loader': 'postcss-loader',
                          'options': {
                            postcssOptions: {
                                plugins: () => [require('postcss-rtlcss')({})],
                            }
                          }
                        },
                        'sass-loader'
                      ]
                    },
                  ]
                }
            }
        }
    return addRtlConfig(config);
}

Please guide if I am using this incorrectly!

Hi @Kavshree,
It is hard to support you without having your complete set up. Please, provide me with the link of the repo in which you are having your issue or create a minimum reproducible git repo in which the issue can be reproduced.
But before that, consult this this issue. There, @MatasRut posted a git repo of an angular project using SASS and postcss-rtlcss, maybe it can help you.
Regards

Hi @Kavshree, did you resolve your issue?
If the issue is not resolved, please, provide me with a repo in which the issue can be reproduced.
Regards


BugsnagInstallJniLibsTask setup should be moved to different plugin class so it can be applied to android library module

With fixes in version 5.1.0 we were able to integrate v5 plugin though some regressions were spotted.

In version 5 of plugin support for library modules was dropped. While it make sense to require this for variant based configuration, the BugsnagInstallJniLibsTask should be allowed to be added at any level.

It's not clear why would BugsnagInstallJniLibsTask setup depend on uploading NDK symbols while we just need it for compilation. E.g. explicitly setting bugsnag.uploadNdkMappings to false will disable BugsnagInstallJniLibsTask setup.

So we would like to see BugsnagInstallJniLibsTask setup moved to different plugin class so it can be applied to android library module.

Hi @pbajurko,
Adding support for library modules is on our roadmap. Although we don't have any confirmed timeframes for this as yet.
The BugsnagInstallJniLibsTask copies the JNI libraries and is usually only required if you are linking against Bugsnag's SO files. Are you using Bugsnag's NDK API, perhaps you could explain your use case for doing this? Separating out this from the the upload task could mean the symbols then are not uploaded. Is there a reason you would not want to upload the symbols for your use case?

Hey @johnkiely1,

Are you using Bugsnag's NDK API, perhaps you could explain your use case for doing this?

e.g. https://github.com/bugsnag/bugsnag-android/issues/904#issuecomment-663687119

Is there a reason you would not want to upload the symbols for your use case?

It's rather simply in daily development we want to compile app without need to upload symbols. We can do it with SDK API but not NDK API for some reason(?). Also we want keep Bugsnag related code at library level for better modularity we can do it with SDK API but not NDK's one.
So it's rather simply change just please move BugsnagInstallJniLibsTask setup to different plugin class that can be applied to library as it was in version 4 of plugin

Hi @pbajurko,
Thanks for that. It seems pretty reasonable, so we are going to look into making this change back to something like how it was in v4. I will let you know of any updates.

@pbajurko we have now released a new version of our Gradle plugin v5.3.0 which should fix this issue.

@phillipsam thanks although it #323 fixes only one from listed regressions.
NDK dependencies still can only at application module level not library one like it was in v4

Hi @pbajurko,
Prior to v5 this worked in some circumstances although it was never officially supported by us. In v5 using a later version of AGP meant this didn't work anymore. Adding back in the library module support is on the roadmap as we do want to officially support it but it does require more work. To that end I have reopened this ticket, and will let you know of any updates.

@johnkiely1
As mentioned in original post, we don't need main plugin class to support library modules.
What we need is just task that will deliver native dependencies on library level and this can be done in separate plugin class.
Also not sure why this plugin has this responsibility, they coming from https://github.com/bugsnag/bugsnag-android/blob/master/bugsnag-plugin-android-ndk. So they should be distributed there? Also you mention newer AGP version which supports native dependencies from aars so maybe simply publish them next to SDK's AAR?

Thanks for the suggestions.
At this stage we are happy with the structure of plugin and splitting it out to that level of granularity could become confusing. While you are waiting for us to implement library module support, you could by all means write your own gradle task to copy the files in to the library module or indeed adapt ours to your requirements.
Putting NDK dependencies into the AAR is something that we are considering and do have this on our roadmap.

We've released a further fix in v7.1.0 which covers this case.


Update VitamioViewManager.java

@abbasfreestyle for which version of RN this change is necessary? I know RN team iterate quickly with their 2 weeks release cycle, so we need to have some clarity here.

Sorry, it's for version 0.24
Im going to test it on 0.25 within the next couple of days.


Add option to decompress into a triangle only

[x] Add uplo = :U/:L/:UL parameter to decompress! and decompress_single_color! to allow picking the triangle for symmetric matrices
[x] Implement the corresponding methods
[x] Add tests for full decompression and for single color, structure into subtests

@amontoison you asked why uplo is not a kwarg. The reason is that I want to be able to dispatch on whether it is present or not, and you cannot dispatch on kwargs (not even on their presence or absence).
My reasoning is that between the following methods, the first one is more efficient (leads to predictable memory accesses) and should thus be preferred when uplo is not specified. What do you think?

function decompress!(
    A::SparseMatrixCSC{R}, B::AbstractMatrix{R}, result::StarSetColoringResult
) where {R<:Real}
    @compat (; S, compressed_indices) = result
    check_same_pattern(A, S)
    nzA = nonzeros(A)
    for k in eachindex(nzA, compressed_indices)
        nzA[k] = B[compressed_indices[k]]
    end
    return A
end

function decompress!(
    A::SparseMatrixCSC{R}, B::AbstractMatrix{R}, result::StarSetColoringResult, uplo::Symbol
) where {R<:Real}
    @compat (; S, compressed_indices) = result
    check_same_pattern(A, S)
    rvA = rowvals(A)
    nzA = nonzeros(A)
    for j in axes(S, 2)
        for k in nzrange(S, j)
            i = rvA[k]
            if in_triangle(i, j, uplo)
                nzA[k] = B[compressed_indices[k]]
            end
        end
    end
    return A
end

You will not gain a lot if you do that:
function decompress!(
    A::SparseMatrixCSC{R}, B::AbstractMatrix{R}, result::StarSetColoringResult; uplo::Symbol=:F
) where {R<:Real}
    @compat (; S, compressed_indices) = result
    check_same_pattern(A, S)
    nzA = nonzeros(A)
    if uplo == :F
      for k in eachindex(nzA, compressed_indices)
          nzA[k] = B[compressed_indices[k]]
      end
    else
      rvA = rowvals(A)
      for j in axes(S, 2)
        for k in nzrange(S, j)
          i = rvA[k]
          if in_triangle(i, j, uplo)
            nzA[k] = B[compressed_indices[k]]
          end
        end
      end
    end
    return A
end
``

The issue is that sometimes uplo is a kwarg and sometimes not.

You will not gain a lot if you do that:

True but with dispatch I avoid compiling the useless part of the code.

The issue is that sometimes uplo is a kwarg and sometimes not.

There was one spot where it was (erroneously) a kwarg, I fixed it in #85. I think now it should be a positional argument everywhere.

You code too fast for me. It's perfect in that case.


Having trouble finding distribution on pip?

I've tried running sudo pip install amazon-paapi5 and sudo pip install amazon-paapi5==1.1.1 without success

Never mind, lists itself as 3.6 :)


feat(core_reconciler): add partial rebuild support in control plane

This PR adds Partial/Log-based rebuild support in control plane. Currently control pane removes Faulted child and adds new child to nexus children list. Post which nexus uses one of the healthy child to Full rebuild the empty child.
With this change depending on number of healthy replicas in nexus we wait for the time defined by the rebuild policy. For eg: If we have atleast 2 Online child in nexus we will wait for faulted child node to come back. If one more goes down we do Full rebuild straight by removing 2nd Faulted child while the 1st Child is going through TimedWait stage.
If after TimedWait period child doesnt come back we do Full rebuild of the child.

@tiagolobocastro  Will handle all unwraps. WIP as of now.

bors try

bors try

bors try

bors try

bors try

bors merge


[REQ ITS MORE OF A QUESTION] is there a fastflag that allows u to see your ping without having performance stats on

Acknowledgement of preliminary instructions

[X] I have read and acknowledged the preliminary instructions.

What idea do you have?
idk thats it, its a question and/or a request sorry

No


depfile,dyndep: reconcile relative and absolute paths with build manifest

A depfile (or dyndep file) may contain relative and/or absolute paths based on compiler
behavior and the include directory flags given.  It is not always
possible to make these exactly match corresponding paths in the build
manifest.  If Ninja does not find an exact match for a depfile path, try
converting it to the opposite kind of path (absolute or relative) and
look again.  If neither matches, add a node with the path as-is.
Fixes: #1251

Based on #1917.

Closing for now.  I have some ideas on a more general solution.


Saving and loading patches

Great work on this editor! It's very intuitive and much more stable than the alternatives that I have tried.
In the README.md, I see that Saving patches to file and Loading patches from file are planned. This would be hugely appreciated.
I had a look and it should not be too difficult to add this capability to the Patch.cs files, but I don't feel ready to take the plunge and start implementing it. For now, just see this as a thanks for great work so far and hopefully some motivation.

Thanks a lot! That was the reason I made this. Every editor I've tried basically was grabbed. Either looked like it or behaved like it ;-)
Currently I don't have access to my JDXI so cannot update the all with features. Maybe will try to do a blind update with patch saving as this should not require device, but don't know when I will have time for that. Anyways if you can do something with code go on, make a PR, and that wi be probably a better kickstart for me to finish it.

Actually, made the UI changes already in the master branch. Maybe I'll finish it this week.

@Zalastax
Loading and saving for almost every patch is currently on master branch, you can compile it yourself and try, FX tab loading data is currently not working as there are some problem with polymorphism. Will try to fix that, but in the meantime you can try other tabs ;-)

Wow! That was amazingly fast! Very kind of you @Magiczne 🥇💟🥇
I've tested it and it works great 😃


Set up the structure for trainer.py

Structure the trainer.py file, add comments and etc.

Implemented, closed


Feat: adding signin/signup with google

Describe the feature
It will make user easy to login and signup directly through their google account
Add ScreenShots
I will implement it using NodeJS and firebase. I had previously worked on it.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@VanshKing30 kindly assign this issue to me

@VanshKing30 kindly assign this issue to me

Ok, start working on this

hey @hustlerZzZ  Can I Work On this issue As its been 2 weeks and she has not raised pr

hey @hustlerZzZ Can I Work On this issue As its been 2 weeks and she has not raised pr

Ok you can work on this issue


Upgrade to Hugo Modules

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Upgrade hugo to use modules and use the latest versions.
Also provide make rules for running locally.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/hold
some styles changed

/assign @PBundyra

/hold cancel

/lgtm

/lgtm

/cherry-pick website

FYI @moficodes @danielvegamyhre @kannon92, in case you want to do the same for jobset


how I do put this code?

hi I have googling how to handle transparent background on phantomjs and found this one but still don't know where I put this code on responsive mockups

page.evaluate(function(){
document.body.bgColor='white';
});

Hi!
Is the problem related to the screenshot images or the generated mockup image?

thank for reply, the problem is related to the screenshot images, it come with transparency background on some website

This is where the screenshots are rendered: https://github.com/fabrik42/responsive_mockups/blob/master/index.js#L18
So I think you need to put your code right above the page.render call.


fix: pages "command" can consist of multiple words

On Windows, the following command wrangler pages dev -- foo bar would error
saying that bar was not a known argument. This is because foo and bar are
passed to Yargs as separate arguments.
A workaround is to put the command in quotes: wrangler pages dev -- "foo bar".
But this fix makes the command argument variadic, which also solves the problem.
Fixes #965

Not sure if I can come up with a reasonable test that is not overkill for this small fix...


Proper parsing of invoice lines

The current parser does not take into account that invoice lines can contain newline characters. Instead of using a custom parser for this the inbuilt .NET CSV parser is used.

I have had good luck using CsvHelper in the past, have you considered using that? Good for if you want it to have a strong model. http://www.nuget.org/packages/CsvHelper/ and https://github.com/JoshClose/CsvHelper

I am thinking that adding a third party library would be unnecessary when there is a inbuilt library that handles it pretty well.

@DanielDestino I definitely appreciate the concern over adding an external dependency. I would say it would be unnecessary today, but TextFieldParser is not portable and is not included in the .NET Core foundational libraries, meaning this code would prevent compilation targeting .NET Core. At some point in the future this will need to be rewritten to use an external portable CSV library like CsvHelper, unless TextFieldParser is added to CoreFX (not likely at this point for RTM).

I don't have a problem merging this in, but there are some conflicts you need to resolve first.
We can upgrade to another library later if it becomes necessary.

Conflicts fixed, please merge :)

This merge solves a problem when parsing an invoice with no itemlines (CsvLineItems == null), which throws an NullPointerException in Invoice.cs line 235


Add CRLite support

We have CRLSets support but they are limited and cover a little part of all CRLs, so we don't use them.
CRLite might be better solution since it covers much more.
https://blog.mozilla.org/security/2020/01/09/crlite-part-1-all-web-pki-revocations-compressed/

Closed due to low significance


Multi nodes database data not consistency

Run multi nodes testing with redis database, there some keys on different database with same key name but with different value information.

Close due to design


[Question] how do you insert nofollow to the example post?

I see <meta name="robots" content="noindex,nofollow"> in the example posts but it is harmful to insert this lines to the real sites then I am investigating how to toggle this line but still no idea.
the line is inserted in-between these two blocks from partials/head.html but it seems impossible to insert the line automatically.
<meta name="renderer" content="webkit" />
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1"/>

<meta http-equiv="Cache-Control" content="no-transform" />
<meta http-equiv="Cache-Control" content="no-siteapp" />

do you insert the line manually for the example site?

I override the layout, see Hugo's Lookup Order. This is an expected behaviour. I will remove it, after host the example-site elsewhere.


Different behaviour with bootstrap-datetimepicker 4.17.42 vs. 4.17.37

Hi!
Based on your Simple example with one datetimepicker http://plnkr.co/edit/n8L8UZ I discovered a problem, and I'm not sure if this is more a problem of the bootstrap-datetimepicker or with this angular directive. Nevertheless, I thought I should report it.
Expected behaviour (works with 4.17.37):
(Open the plunkr above and run it)
Press Add +1w about 5 times (it MUST jump you to at least one month later then the current month)
Press the calendar icon
The selected day and the selected month should appear
With 4.17.42 this does not work.
See also the two uploaded screencasts (as GIF)

<EMAIL_ADDRESS>has the same issue as 4.17.42.
Also, 4.17.37 can NOT be installed using npm...

npm ERR! notarget No compatible version found<EMAIL_ADDRESS>npm ERR! notarget Valid install targets:
npm ERR! notarget 4.17.43, 4.17.42, 4.15.35, 4.14.30, 4.7.14, 3.1.3

This pretty much leaves this angular library in a spot where it does not work as expected (when using npm), unless you are willing to use a CDN version of the datetimepicker or download it manually or use npm install git+https://...

Well cutting long story short... if by changing the version of date-time picker you observe different behaviour it is not the problem of the directive.
Myself I am using bower for dependencies because they rely on GitHub tags.
To sum up, I understand your problems, thought all of them should be posted on library's GitHub and are really not in my decision to be fixed or not. The library itself has a maintenance problem and was not updated for a long time. I also suggest using master version directly from GitHub. The pull requests there were occasionally accepted. I use 4.17.37 from bower.

Then you still run into the problem of new users that install your package will get version 4.17.43 automatically:
bower install angular-eonasdan-datetimepicker --save

angular-eonasdan-datetimepicker#0.3.7 bower_components/angular-eonasdan-datetimepicker
├── angular#1.5.8
├── bootstrap#3.3.7
├── eonasdan-bootstrap-datetimepicker#4.17.43
├── jquery#2.2.4
└── moment#2.15.2

jquery#2.2.4 bower_components/jquery

angular#1.5.8 bower_components/angular

bootstrap#3.3.7 bower_components/bootstrap
└── jquery#2.2.4

eonasdan-bootstrap-datetimepicker#4.17.43 bower_components/eonasdan-bootstrap-datetimepicker
├── jquery#2.2.4
└── moment#2.15.2

moment#2.15.2 bower_components/moment
...

It's definately good that one can get 4.17.37 by using bower, though with so many people abbandoning bower in favour of npm (npm is even the first installation method mentioned in the README) I believe we should find a fix.

To be honest I totally understand you but still I have no answer I could help you with other than manually selecting the version. I was not even aware of the bugs you have just  pointed out in the newest version and probably most of the users did not use those "corner cases" as well :smile:
I wonder if the bug above could be anyhow connected to

Could be connected. One way to "fix" this issue was to call issue an extra call to setDefault in ngmodel.$render, which would overwrite the default value.
Though I'm pretty sure this is not the way it's intended to work.

Tested with 0.3.8 but the library version 4.17.42 still shows wrong month.

@anx-ckreuzberger It is a feature :smile_cat:
The calendar was designed in a manner that if the selected day is still visible on the previous month's page it does not change the page. In your example:

On the November's page you can see two rows of days from December and in 4.17.42 until the 12.12.2016 is reached the next page is not loaded. Maybe there is an option for that, but it is not a bug.


[AppStore Metadata] Figure out process to localize

After applying #1272, we now currently don’t have localized content.
keywords, description, etc

thanks @alloy looking into translation services and cost $$. Will close when I find a solution

@steipete recommends http://t.co/a0K5IuJE7V


Updating rcbops/openstack-ansible submodule sha

This is to include the linux-image-extra name change
from https://github.com/rcbops/openstack-ansible/pull/2
in the rpc-openstack's openstack-ansible submodule.

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.

Canceling.  Looks like these tests use an old version, so we might need to fine tune.


Improved UI of worker-signup page

🛠 Fixes Issue

Closes #98
👨💻 Changes proposed
Improved the UI of worker-signup page.Made it very good looking and synchronous with other pages.

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📷 Screenshots

hey @KaizenGirl1111
Can you please review this PR?


looks like switching between namespaces on the apps/services/controllers/pods pages are broken

just tried it and changing the selected namespace doesn't seem to have an effect

it works on the Diagram tab mind you!

This works fine now with the new watcher service...  Except on the Diagram/overview tab!  :-)  Will open a separate issue for that...


Storage directory getter for Files

Overview

This makes the storage directory from a File accessible.
Checklist

[x] tests/ added or updated
[x] CHANGELOG.md's Unreleased section is updated
[x] Documentation is updated

Coverage increased (+0.1%) to 88.152% when pulling bf2d0a376f222ab64f804e9f0dc134a97c5598ea on storage-dir into d9c3040b4ed96f13237bc5e4449dbc6e77eed2cd on develop.

Coverage increased (+0.1%) to 88.152% when pulling bf2d0a376f222ab64f804e9f0dc134a97c5598ea on storage-dir into d9c3040b4ed96f13237bc5e4449dbc6e77eed2cd on develop.


Remote attribute's function behavior added

Get options from service as a deferred object by remote attribute.
HTML
<select selector
    model="country"
    remote= "vm.remote(search)"
    label-attr="name">
</select>

JS
vm.remote =
	function (search) {
	    return MyAppService
	    		.getCountries({search:search})
	};

For more example
Plunkr1
Plunkr2

Closes #17

Thank you @Nishchit14 for this nice feature!

This is now available in v0.1.1.


[REGRESSION]: Failed to obtain the Json Web Token(JWT) using service principal client ID

New issue checklist

[X] I searched for existing GitHub issues
[X] I read pipeline troubleshooting guide
[X] I checked how to collect logs

Task name
PublishToAzureServiceBusV2
Breaking task version
2.243.0
Last working task version
2.243.0
Regression Description
As part of security work at Microsoft, we switched our pipelines from PublishToAzureServiceBusV1 (which uses a connection string to publish messages to an Azure Service Bus) to PublishToAzureServiceBusV2 (which uses a federated workload identity).
At around 8/5 at 5pm UTC, these tasks started failing with the following message:

Since then, the errors have been increasing in frequency:

Environment type (Please select at least one enviroment where you face this issue)

[X] Self-Hosted
[ ] Microsoft Hosted
[ ] VMSS Pool
[ ] Container

Azure DevOps Server type
dev.azure.com (formerly visualstudio.com)
Azure DevOps Server Version (if applicable)
No response
Operation system
Agentless
Relevant log output
Failed to obtain the Json Web Token(JWT) using service principal client ID

Full task logs with system.debug enabled

UNSUCCESSFUL RUN
"Message payload signature was not generated."
Failed to obtain the Json Web Token(JWT) using service principal client ID

SUCCESSFUL RUN
"Message payload signature was not generated."
2024-08-09T17:41:53.0426621Z Received instance ID of: xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx
2024-08-09T17:41:53.5453278Z Successfully started orchestration with ID 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx.xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx'.
2024-08-09T17:41:53.6093454Z Getting build status for build ID 10019795
2024-08-09T17:41:54.3673798Z Found build ID '10019795' with status 'Completed'
2024-08-09T17:41:54.8858050Z 'DD-CB-PR' with run ID '10019795' completed with result 'Succeeded'.

Repro steps
- task: PublishToAzureServiceBus@2
  inputs:
    azureSubscription: 'VSEng/DartLab'
    serviceBusQueueName: 'deploymachines-prod1'
    serviceBusNamespace: 'vsengtradtl'
    signPayload: false
    waitForCompletion: true
    useDataContractSerializer: false

Please refer this icm for more updates https://portal.microsofticm.com/imp/v5/incidents/details/530278277/summary


Webpack 3

Upgrade to Webpack 3
Webpack 3 has been released. Most interesting it introduces rollup style scope hoisting via the webpack.optimize.ModuleConcatenationPlugin.
Scope hoisting means that the scope of each module will be hoisted to the top of the bundle, potentially greatly reducing size and parse time of the code.
Even if it's a major release, it shouldn't introduce any breaking changes, so I will make a PR when I get time, unless someone beats me too it.

😢 webpack.optimize.ModuleConcatenationPlugin doesn't decrease the file size as much as I expected, but still I think it's good for ReactGo to apply that option.

@ZeroCho yeah true. From what I can see the size goes from 337.32KB to 332.46KB so that is certainly a modest amount. Reportedly, there is also the benefit of faster browser evaluation time. I haven't had time to test this though, and I think the application is really too small of a sample to get any conclusive eveidence.
Having said that, I don't think there's any appearant drawbacks to upgrading!

#903


[Question] VecEnv implementation reasoning

Hi,
It seems that for vectorized environments, the design for this library (and others) is to sample as following: if n = # environments, one sample is stored as a (n x obs_size) tuple into the replay buffer, and the model consumes the n-tuple. Why is it done this way, as opposed to storing the n-tuple as n separate tuples, and having the model consume one obs at a time? Thanks!
Bryan

Hello,

Why is it done this way, as opposed to storing the n-tuple as n separate tuples, and having the model consume one obs at a time?

Why would you do it the other way?
(btw, it also support dicts but at the end, the behavior is the same)

The other way would allow you to test your model on 1 observation

The other way would allow you to test your model on 1 observation easily.

You have the predict() method for that (cf doc).
The VecEnv is useful for making the training faster.


Dark Theme of Website Updated Issue no #290 Resolved

Fixes #290

I have Enhance the Dark theme of the website
@Trisha-tech  Please Merge this pull request with the labels attached.
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[ x] I have performed a self-review of my code
[ x] I have commented my code, particularly in hard-to-understand areas
[x ] My changes generate no new warnings
[x ] I am working on this issue under GSSOC

Fixes #290

I have Enhance the Dark theme of the website @Trisha-tech Please Merge this pull request with the labels attached.
Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.
* [ x] I have performed a self-review of my code

* [ x] I have commented my code, particularly in hard-to-understand areas

* [x ] My changes generate no new warnings

* [x ] I am working on this issue under GSSOC

Nice work ! Could you please send the before vs after screenshots for better clarity.

Before
After

@Suman373 Please merge the PR with labels attached

Thanks ! Well done 👍

@Trisha-tech @Suman373 @rajatsinghal02

I think dark mode logo would look better if in png format, the above one looks odd

@Trisha-tech @Suman373 @rajatsinghal02
 I think dark mode logo would look better if in png format, the above one looks odd

Yes, if you want to work on this small fix u may create an issue regarding this.


Nested attributes cannot convert correctly.

api response:

transformed json:

expected:
startAt:
Sun Oct 19 2024 016:00:00 GMT+0900 (日本標準時)
endAt:
Sun Oct 20 2024 03:30:00 GMT+0900 (日本標準時)
root json attribute is transformed correctly.
e.g.
{ endAt: "2024-10-20T04:00:00+09:00" }

nested json attribute is not transformed correctly.
e.g.
{ shift: { endAt: "2024-10-20T04:00:00+09:00" }}

It would be helpful if you could investigate.

Sorry. this problem is own degrade.
please close this.


Split Krew side functionality into different files

In krew we are seeing a lot of functionality added and in the future, it will be more,
So if we can split the functionality into different files keeping the base as kubectl-rook-ceph.sh would a good idea,
for maintaining reviewing and implementing, will become closer to object-oriented code:)

Krew allows for multiple files to be installed with a plugin?

not valid anymore since we moved to golang.


Learn Basic CSS by Building a Cafe Menu - Step 64

Describe the Issue
In step 64 you are asked to create an anchor element, the anchor element in step 65 has a target attribute, but there is no test for this in step 64.
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-basic-css-by-building-a-cafe-menu/step-64
Your code

Visit our website

Expected behavior
The target element in step 65 should not come as a surprise. Step 64 should instruct to add it.
Screenshots
No response
System
NA
Additional context
No response

Okay, let's make sure the user adds the target="_blank" in step 64 by adding the following sentence:

Make sure that the link opens in a new tab by adding a target attribute with the value _blank.

After adding that to the description make sure to add a test that tests for a target attribute on the anchor element.

In step 64, an anchor element is required in a paragraph element on line 54.

In step 65 on line 56 the anchor element does not contain a target attribute.

However in step 65 there is a target attribute in the anchor element one line 9

In summary, there are three anchor elements in the final Cat Photo App.
The first one has a target attribute, the other two do not.
The footer > p > a does not appear to need a target attribute or test.

Hey @Supravisor
that appear to be different project.


Fix transposed context values.

Fixes bug introduced during recent context refactor in commit https://github.com/constabulary/gb/commit/4097af980ccf60c79b4c8350369192982f744bad.

Thanks. I'll submit this once wercker goes green.


Can users set permissions and roles for projects?

Can users set permissions and roles for projects?

Currently, there is no this feature.