Added "docker exec" type functionality to GenericContainer. #103

Cleaned up version; tests should pass now.

Thanks, this looks good to me. I'll have a play with this and aim to merge this evening.

Ah, running this I noticed that the container output was appearing on System.err as well as being buffered. It looks like the docker-java lib's ExecStartResultCallback has a side effect of println'ing everything to System.err:
 @Override
    public void onNext(Frame frame) {
        if (frame != null) {
            try {
                switch (frame.getStreamType()) {
                case STDOUT:
                case RAW:
                    if (stdout != null) {
                        System.err.println(new String(frame.getPayload()));
                        stdout.write(frame.getPayload());
                        stdout.flush();

I'm not sure if this is a bug or desired behaviour in docker-java but I think it's a bit undesirable here. Do you think you could replace with a callback that just does the desired buffering? Given that the out of the box callback doesn't seem to be an exact match, I suspect that there's a nice DRY option through extraction of this into a proper class and use of ToStringConsumer..!

I'll defer this to 1.0.4 if you don't mind, though I hope to release it quite quickly after 1.0.3.

I noticed that the Travis build job failed on pulling a container image, so I've retriggered that build.

Also included here, because it builds on the refactoring I did around outputting callbacks: when a container fails to start up, (optionally) capture its output in the logs.
Adding a unit test for this required that I bring in Mockito.

Looks like I'm seeing the same issue here as I was with #115. I'll back this branch up and leave the log capture stuff until later.

Looks good to me - thank you. Merged!


Oscillatory behavior in the number of iterations per second

General information:

emcee version: 3.0.2
platform: Linux (Manjaro)
installation method (pip/conda/source/other?): pip under a conda environment

Problem description:
Expected behavior:
Number of iterations per second remains constant throughout the program, or changes in a uniform fashion.
Actual behavior:
Number of iterations per second shows an oscillatory behavior that gets worse as the number of steps increase .
What have you tried so far?:
Nothing, unsure how to proceed.
The linear regression example available on the documentation does not show this behavior, but it's not parallelized.
Minimal example:
Haven't been able to further debug this issue, full source code is here, this is mostly based on the tutorials and uses a package that I developed.
And here's a video that I recorded of that oscillatory behavior (skip to almost to the end of the video):
https://youtu.be/tJ1ewHnh1Ns

Please put together a simpler example with no dependencies because I can't help debug issues with external packages.

This is not very minimal but it does show the same behavior:
# imports
from multiprocessing import Pool
from scipy.integrate import quad
from math import log, pi
import numpy as np
import emcee
import os

# avoid numpy parallelization
os.environ["OMP_NUM_THREADS"] = "1"

# data obtained for "observations" (mock catalog)
redshifts = [4.650597618519434, 1.7187615134799283, 1.96422316313268, 2.279534979583823, 2.4255221814653085, 5.352683861215292, 3.3009039750539038, 1.147489638942757, 3.095528002202573, 7.181594409133277, 5.489403487139625, 1.5667645827080101, 4.336343558668992, 4.411170021297732, 3.0302171724544715]
distances = [38.63665753463308, 13.433364627229862, 14.789874051362968, 18.35210939265398, 17.0556256529821, 46.4392305092797, 27.987621881949003, 7.7684781039518525, 25.560383436360233, 57.984090978826416, 57.96010746060587, 11.348110616357697, 40.23195030252779, 37.45595825024949, 25.975178600400568]
errors = [4.810592486986055, 0.5363050547332151, 0.7228902526237483, 1.3869590285255247, 1.5295952653405556, 6.257341530569792, 2.576815651543119, 0.21133168296086813, 2.3019708974482636, 10.918185604129675, 6.561303625821857, 0.43475055916828353, 4.225452271355636, 4.361248075989948, 2.2182722422373207]

# define the cosmological model we're going to check against observations through the light distance
def E(z, Ωm):
    return (Ωm*(1+z)**3 + (1-Ωm))**0.5

def dL(z, h, Ωm, M, E):
    # "eletromagnetic" light distance
    dLem = (1+z) * (2.9979/h) * quad(lambda Z: 1/E(Z, Ωm), 0, z)[0]  # c/H0 = 2.9979/h Gpc

    # correction to compute the "gravitacional wave" light distance
    factor = 2*6**0.5
    correction = ( (factor + M) / (factor + M/E(z, Ωm)))**0.5  # M in units of H0

    return correction * dLem

# define the likelihood
def ln_likelihood(θ, redshifts, distances, errors, dL, E):
    h, Ωm, M = θ
    N = len(redshifts)

    sum = 0
    for i in range(0, N):
        sum += -log(errors[i]) - (distances[i] - dL(redshifts[i], h, Ωm, M, E))**2 / (2*errors[i]**2)

    return -N*log(2*pi)/2 + sum

# add flat priors to both parameters
def ln_prior(θ):
    h, Ωm, M = θ
    if 0.4 < h < 1 and 0 < Ωm < 1 and -4 < M < 10:
        return 0.0
    return -np.inf

# combine the likelihood and the prior into one expression
def ln_probability(θ, redshifts, distances, errors, dL, E):
    prior = ln_prior(θ)
    if not np.isfinite(prior):
        return -np.inf
    return prior + ln_likelihood(θ, redshifts, distances, errors, dL, E)

# initial position in a gaussian ball around the true values for h and Ωm, and the expected value for M
# initialize the walkers and maximum number of steps
nwalkers = 32
ndim = 3
init = [0.7, 0.3, 2] + (1e-1, 1e-1, 2) * np.random.randn(nwalkers, ndim)
nsteps = 25000

# track how the average autocorrelation time estimate changes
index = 0
autocorr = np.empty(nsteps)

# this will be useful to testing convergence
old_tau = np.inf

# run emcee
print("Running MCMC:")
with Pool() as pool:
    sampler = emcee.EnsembleSampler(nwalkers, ndim, ln_probability, args=(redshifts, distances, errors, dL, E), pool=pool)
    # sample for up to nsteps steps
    for sample in sampler.sample(init, iterations=nsteps, progress=True):
        # check convergence every 100 steps
        if sampler.iteration % 100:
            continue

        # compute the autocorrelation time so far (tol=0 means that we'll always get an estimate even if it isn't trustworthy) (this is the average across all dimensions)
        tau = sampler.get_autocorr_time(tol=0)
        autocorr[index] = np.mean(tau)
        index += 1

        # check convergence
        converged = np.all(tau * 500 < sampler.iteration)
        converged &= np.all(np.abs(old_tau - tau) / tau < 0.005)
        if converged:
            break
        old_tau = tau

This is mostly stuff out of the tutorials.
In the beginning I define what our observations are, then the cosmological model I'm testing, likelihood, prior and add the previous two to create the probability. After that I run emcee in parallel with convergence test, effectively adding code from the section Saving & monitoring progress and Parallelization.
As I said before I didn't had this issue when running the linear regression example from the section Fitting a model to data, which doesn't have parallelization + correlation analysis.

Good - thanks! Now is the period of this "oscillatory behavior" exactly 100 steps? And what happens if you change the hard coded magic number 100 to something else 😀?

Oh that was rather obvious, it was factor number 2 (correlation analysis) all along.
In my mind the autocorrelation time was measured based on the last 100 steps not the entire chain, but that didn't make a lot of sense either.
This adds a new level of complexity to consider if I end up leaving the chain running for a few hours, but I suppose that there's no way around it that's the price to pay if I want to check for convergence.

If your runtime is dominated by the autocorr calculation, I'd recommend doing it much less frequently.

It's not dominated, but it's definitely there.
As I'm still quite new to this I end up doing a lot of runs, however, they seem to be quite fast. My professor thought my results had taken hours to get, while they actually took minutes (3 parameters with rather small priors with a ≈ 10 value window each), and the convergence criteria is met (sometimes).
Is doing it less frequently more prone to errors?

It won't be prone to errors. It just means that you'll overshoot your definition of "convergence" by a little more, but this is never a bad thing. It's only a trade off of computational cost so if you're seeing the effects of autocorr computation in your average runtime then it's probably worth running it less frequently because your model is fast enough.

Cool, I suppose that does it, I'll close this issue considering it's solved. Thanks.

Actually I have something to ask (hopefully I'm thinking this correctly).
After going through the source code I saw that the function get_autocorr_time simply calls the function autocorr.integrated_time, which means that what we're doing is measuring the integrated autocorrelation time, as mentioned in the documentation.
If we're doing an integral (or a sum because this is discrete) then we don't have to get the entire chain because, if we want to compute the autocorrelation time from iteration 0 to iteration N₂, and we already computed the autocorrelation time from 0 to N₁ (where N₁ < N₂), then all we have to do is compute the autocorrelation time from N₁ to N₂ and add it to the value obtained from 0 to N₁ and we would get the autocorrelation time from 0 to N₂ without getting the entire chain.
I think that my reasoning is correct because, as we saw in the section Saving & monitoring progress in the documentation, we stop our chain when it changes by less than 1% (and has done a big enough number of iterations), which means that in the mean τ plot shown in that same section, it's a straight line, meaning that the walkers have reached a point where they are walking randomly in the region of (hopefully) maximum likelihood, and as such have no correlation between them.
Is this reasoning correct?
Could something like this be implemented in emcee? For example instead of having tau = sampler.get_autocorr_time(tol=0) we could have something like tau = sampler.get_autocorr_time(tol=0, initial=N₁, end=N₂).

Take a closer look at the sum required to approximate the autocorrelation time... it touches every pair of points in the chain, hence why it's computed using an FFT. There has been some work on online estimation of acorr time (see, for example, the Goodman & Weare paper) but none of these methods are numerically stable in my experience. If your runtime is noticably hit by the cost of computing the autocorrelation time, you're doing it far to frequently, like I said above!!

Ah, I didn't quite understood the equations and as I saw the last sum it seemed to me that it was sequential.
Thanks again!


New logo time

Adds #823

@FNFPlayer101's suggestion of having it as the Bumpin' logo

New logo time

Awesome.


contact method for reporting abusive behaviour

In our code of conduct the contact point for reporting abusive behaviours is a stub.  It should instead resolve to some kind of private channel that is terminated by the appropriate individual/s.
For further context, see also https://github.com/veraison/veraison/pull/34#discussion_r592435449

@thomas-fossati If this is still applicable can we move this to community repository?

the CoC is currently hosted at the veraison organisation level, and the stub mentioned in this issue is now a real email address.  The thing is that in the meantime we moved away from the @arm.com contact point to https://groups.google.com/g/veraison-project/settings, so we should update that.

Further work tracked via:
https://github.com/veraison/services/issues/82


Infinite digest loop on 'moveend' event

The digest goes into an infinite loop when the 'moveend' event setting the bounds which triggers the watcher on bounds in bounds.js. Looks like the 'moveend' event listener should not apply the scope when mapCenter's coords are equal to centerModel's coords. However when all coords are equal to 0.0000 isSameCenterOnMap helper returns true.

#1154


Updated link for Symfony SAS -uk language

Assign Under Hacktoberfest
What does this PR do?
Updated old link on new
The source still meets all the requirements. Only the link has been replaced because the original one is outdated
Checklist:

[+] Read our contributing guidelines.
[+] Search for duplicates.
[+] Include author(s) and platform where appropriate.
[+] Put lists in alphabetical order, correct spacing.
[+] Add needed indications (PDF, access notes, under construction).
[+] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Right-clicking table entities for delete selects two rows

Storage Explorer Version: 1.10.1
Build Number: 201990919.2
Platform/OS:  Windows 10?
Architecture:   x64?
Regression From:  no
Bug Description
When you right click to delete a table row, the selected and the mouse located rowsare highlighted.  Confirming the delete only removes the initially selected row as it should, but the warning message with two rows identified is misleading.
Steps to Reproduce

Right click any row in a table and select delete

Expected Experience
highlight only one selected row or use a bold color for the selected row.
Actual Experience

Additional Context

Could you please give more detailed steps to reproduce the bug?

Sorry had these details in html comments.

Replied on GitHub
On Mon, Oct 21, 2019 at 12:43 PM JasonYeMSFT<EMAIL_ADDRESS>wrote:

Could you please give more detailed steps to reproduce the bug?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AzureStorageExplorer/issues/2107?email_source=notifications&email_token=AAK5TPWPTMAXRCIMDGG2EXLQPXL3DA5CNFSM4JB6EB62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEB27TRI#issuecomment-544602565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK5TPRKVMGZWSUJT35OEULQPXL3DANCNFSM4JB6EB6Q
.

Could you please attach a screenshot or a recording showing the bug you are describing? We are having trouble reproducing this issue.

See attached screenshot
On Tue, Nov 5, 2019 at 5:09 PM JasonYeMSFT<EMAIL_ADDRESS>wrote:

Could you please attach a screenshot or a recording showing the bug you
are describing? We are having trouble reproducing this issue.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/AzureStorageExplorer/issues/2107?email_source=notifications&email_token=AAK5TPVZAKBHKSW5KPUVAOTQSHVINA5CNFSM4JB6EB62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDEQKGA#issuecomment-550044952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAK5TPSUNFWXNMS7N6M2YCLQSHVINANCNFSM4JB6EB6Q
.

We don't see a screenshot. You need to paste it directly into your github comment. Attaching it to your email reply won't work.

Hope this works
https://1drv.ms/u/s!AgsEoWikEEW_le19RToQG0_G0_syyw?e=ulfXsF

Thanks. We understand the issue and will look into getting this fixed.

Unable to reproduce this issue on 1.14.2. When I try right-clicking an item different than the current selection, the current selection is set to that item as expected.
@cliffeby Can you reproduce on the latest version (1.14.2) of Storage Explorer?
@v-xianya Can you try reproducing this issue as well?

I still get the same behavior but less consistently.  About half the time only one line is highlighted.  When only one line is highlighted and then deleted, a new cell is highlighted at the top of the page.  The jump and highlighting is disconcerting/unexpected.  @craxal  @v-xianya

IMG_3752.zip
This is a video showing both issues.

@cliffeby The jump to the top of the page is by design. All of our editors do that.
As far as the double-selection is concerned, I finally got it to repro. I think I know what's happening. The Delete command, of course, opens a confirmation dialog before proceeding. There's often a (very) short delay between when the command is triggered and the dialog opens. During that delay, the editor can still respond to UI events. In this case, a few things are happening:

The table loses focus. This results in the selected row changing from dark blue to light blue.
The UI handles a mouseover at the current location of the cursor. This results in the row under the cursor also being colored light blue.
The editor UI is blocked as soon as the dialog opens, so the editor doesn't change any hover states.

It just so happens the colors we use for row hover color and row unfocused/selected states are the same color, which is probably where the confusion is coming from. In reality, only one item is selected. If you cancel the confirmation dialog, you should notice the UI update the row hover.
Two possible solutions:

We have a different style for row hover and row unfocused/selected states.
Indicate the number of entities that will be deleted in the confirmation dialog.

@jinglouMSFT @MRayermannMSFT @JasonYang-MSFT Thoughts?

@craxal   @jinglouMSFT @MRayermannMSFT @JasonYang-MSFT
Regarding options, I'd prefer style/color difference as long as it's obvious which is going to be deleted..  For the second possible solution, if the confirmation dialog notes one row to be deleted and two are highlighted, which one is it?
Regarding standard editor behavior, I'm not sure why a jump to the top of the page is desired.  Why would I want to go to the top of a page?  Could you imagine that behavior in VSCode, Excel or Word?

@cliffby We try to emulate behaviors observed in File Explorer on Windows or Finder on Mac. Both File Explorer and Finder move the selection to the next item when deleting, so that's something we can look into. If could please open a separate bug for that, we'd greatly appreciate it!


Uploading JSON to Files API returns invalid file format

Upload to the files endpoint with a JSON file throws an error
Code:
from openai import OpenAI

client = OpenAI()
file = client.files.create(
    file=open("example_1.json", "rb"),
    # Can either be fine-tuned or assistant
    purpose="assistants",
)

Stacktrace:
ile [~/anaconda3/lib/python3.10/site-packages/openai/resources/files.py:88](https://file+.vscode-resource.vscode-cdn.net/Users/vashishtmadhavan/Documents/playground/~/anaconda3/lib/python3.10/site-packages/openai/resources/files.py:88), in Files.create(self, file, purpose, extra_headers, extra_query, extra_body, timeout)
     82 if files:
     83     # It should be noted that the actual Content-Type header that will be
     84     # sent to the server will contain a `boundary` parameter, e.g.
     85     # multipart/form-data; boundary=---abc--
     86     extra_headers = {"Content-Type": "multipart/form-data", **(extra_headers or {})}
---> 88 return self._post(
     89     "[/files](https://file+.vscode-resource.vscode-cdn.net/files)",
     90     body=maybe_transform(body, file_create_params.FileCreateParams),
     91     files=files,
     92     options=make_request_options(
     93         extra_headers=extra_headers, extra_query=extra_query, extra_body=extra_body, timeout=timeout
     94     ),
     95     cast_to=FileObject,
     96 )

File [~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:1055](https://file+.vscode-resource.vscode-cdn.net/Users/vashishtmadhavan/Documents/playground/~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:1055), in SyncAPIClient.post(self, path, cast_to, body, options, files, stream, stream_cls)
   1041 def post(
   1042     self,
   1043     path: str,
   (...)
   1050     stream_cls: type[_StreamT] | None = None,
   1051 ) -> ResponseT | _StreamT:
   1052     opts = FinalRequestOptions.construct(
   1053         method="post", url=path, json_data=body, files=to_httpx_files(files), **options
   1054     )
-> 1055     return cast(ResponseT, self.request(cast_to, opts, stream=stream, stream_cls=stream_cls))

File [~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:834](https://file+.vscode-resource.vscode-cdn.net/Users/vashishtmadhavan/Documents/playground/~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:834), in SyncAPIClient.request(self, cast_to, options, remaining_retries, stream, stream_cls)
    825 def request(
    826     self,
    827     cast_to: Type[ResponseT],
   (...)
    832     stream_cls: type[_StreamT] | None = None,
    833 ) -> ResponseT | _StreamT:
--> 834     return self._request(
    835         cast_to=cast_to,
    836         options=options,
    837         stream=stream,
    838         stream_cls=stream_cls,
    839         remaining_retries=remaining_retries,
    840     )

File [~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:877](https://file+.vscode-resource.vscode-cdn.net/Users/vashishtmadhavan/Documents/playground/~/anaconda3/lib/python3.10/site-packages/openai/_base_client.py:877), in SyncAPIClient._request(self, cast_to, options, remaining_retries, stream, stream_cls)
    874     # If the response is streamed then we need to explicitly read the response
    875     # to completion before attempting to access the response text.
    876     err.response.read()
--> 877     raise self._make_status_error_from_response(err.response) from None
    878 except httpx.TimeoutException as err:
    879     if retries > 0:

BadRequestError: Error code: 400 - {'error': {'message': "Invalid file format. Supported formats: ['c', 'cpp', 'csv', 'docx', 'html', 'java', 'json', 'md', 'pdf', 'php', 'pptx', 'py', 'rb', 'tex', 'txt', 'css', 'jpeg', 'jpg', 'js', 'gif', 'png', 'tar', 'ts', 'xlsx', 'xml', 'zip']", 'type': 'invalid_request_error', 'param': None, 'code': None}}

Here is the example file: example_1.json

Thanks for reporting! We can repro this bug and are working on fixing it.

Hi, I think this bug is the same as #333 and found a way to fix it.
In this line if instead of file=buffer_reader I put file=open(args.file, "rb") the function returns normally instead of 400 error.
The only difference I see in BufferReader in the v1.1.1 and v0.28.1 is the omission of a check but I doubt this is the culprit.
Unfortunately there is currently an API Outage so I can't investigate that much more at the moment. At least my workaround fixes it for the whisper case.

I'm sorry @RobertCraigie  but I don't understand the problem. My issue is indeed appearing when I use the CLI like so python -m openai api audio.transcriptions.create -f audio.mp3 and the workaround I suggest solves it apparently.
I don't really get the meaning of "the CLI code, not the client code itself" as cli is part of the client and both are in this repo.
In any case my issue is originaly with whisper and might not be related to this one after all so I will stop talking here. I messaged the thread #333 with my workaround anyway.
Cheers

It might be related to bug in open ai API - https://community.openai.com/t/possible-bug-with-agent-creation-php-file-upload/484490/5
I have same issue with PHP SDK, uplading php and json files fails, the txt and html succeeds.

This is a backend bug, not an SDK bug, so I'm going to go ahead and close this issue.

The same issue, just try to send incorrect JSON file, and it will work ;)

Thanks for reporting! We can repro this bug and are working on fixing it.

Hi @athyuttamre, is there an issue # to track this bug?

Yes, I need to keep track of this bug, I am unable to work with some files

The same issue, just try to send incorrect JSON file, and it will work ;)

I was really hoping you were kidding about this. I've been banging my head over this for the past 5 hours. I added garbage to the beginning of my file and now it works. 🤦🏻♂

OK I think I finally found a pattern, at least for our case. Valid JSON files upload fine so long as they are over 1025 bytes in size. If they are 1025 bytes or under, they will also work if you make them invalid by making them invalid as @albertaleksieiev suggested.

@kennymatic , I believe as you do that in this case size matters.  @albertaleksieiev also seems to be correct (i.e. send an invalid JSON formatted file and it will work.)
I found to be routinely successful uploading a 10kb json file.
In a small file (i.e. < 1kb) I replaced the open and close square brackets with curly brackets, then saved.  File uploaded successfully.
It's one of those, "Are you kidding me?" bugs.

I put in the following:
[
{
"id": "msg_0001",
"role": "user",
"content": "\n    This messgage is to create and initialize the JSON file with enough file size that OpenAI will upload it.\n    A JSON (JavaScript Object Notation) file is a lightweight data interchange format that is easy for humans to read and write,\n    and easy for machines to parse and generate. It is based on a subset of the JavaScript Programming Language Standard ECMA-262 3rd Edition - December 1999.\n    JSON is a text format that is completely language independent but uses conventions that are familiar to programmers of the C-family of languages,\n    including C, C++, C#, Java, JavaScript, Perl, Python, and many others. These properties make JSON an ideal data-interchange language.\n    The structure of a JSON file is simple yet flexible. It represents data in a text format consisting of key-value pairs,\n    making it analogous to a dictionary in Python or an object in JavaScript.\n    These key-value pairs are enclosed in curly braces, with the key being a string and the value being a valid JSON data type such as\n    a string, number, array, or even another JSON object.\n    This hierarchical structure allows for the representation of complex data in an organized and hierarchical manner,\n    which is particularly useful in web applications for data exchange between a client and a server, as well as in many other\n    programming contexts where data needs to be stored or transmitted in a structured format.\n    "
},
{
"id": "msg_l18vJqdofhe35GVYSvWb4PEe",
"role": "user",
"content": "\n    We need to have a meeting of everyone within the company.  Our strategic goal is to raise revenue by 10%.\n    Your mission is to discuss amongst yourself to provide a consensus suggestion of the top five initiatives the company should take.\n    When the mission has been completed provide the five initiatives and your reasonsing for each.\n    "
}
]

BTW, a stupid but successful work-around is to create an initial entry in the JSON file with enough content that will make the file size exceed the 1kb that @kennymatic mentioned. I put in the following: [ { "id": "msg_0001", "role": "user", "content": "\n This messgage is to create and initialize the JSON file with enough file size that OpenAI will upload it.\n A JSON (JavaScript Object Notation) file is a lightweight data interchange format that is easy for humans to read and write,\n and easy for machines to parse and generate. It is based on a subset of the JavaScript Programming Language Standard ECMA-262 3rd Edition - December 1999.\n JSON is a text format that is completely language independent but uses conventions that are familiar to programmers of the C-family of languages,\n including C, C++, C#, Java, JavaScript, Perl, Python, and many others. These properties make JSON an ideal data-interchange language.\n The structure of a JSON file is simple yet flexible. It represents data in a text format consisting of key-value pairs,\n making it analogous to a dictionary in Python or an object in JavaScript.\n These key-value pairs are enclosed in curly braces, with the key being a string and the value being a valid JSON data type such as\n a string, number, array, or even another JSON object.\n This hierarchical structure allows for the representation of complex data in an organized and hierarchical manner,\n which is particularly useful in web applications for data exchange between a client and a server, as well as in many other\n programming contexts where data needs to be stored or transmitted in a structured format.\n " }, { "id": "msg_0002", "role": "user", "content": "\n We need to have a meeting of everyone within the company. Our strategic goal is to raise revenue by 10%.\n Your mission is to discuss amongst yourself to provide a consensus suggestion of the top five initiatives the company should take.\n When the mission has been completed provide the five initiatives and your reasonsing for each.\n " } ]

This is basically what we did except we got the difference in length to get up to the min characters. Then we added a field to the JSON and filled it with spaces to get us up to the min limit. 😬

Thumbs up 👍 !
 I put in
the following: [ { "id": "msg_0001", "role": "user", "content": "\n This
messgage is to create and initialize the JSON file with enough file size
that OpenAI will upload it.\n A JSON (JavaScript Object Notation) file is a
lightweight data interchange format that is easy for humans to read and
write,\n and easy for machines to parse and generate. It is based on a
subset of the JavaScript Programming Language Standard ECMA-262 3rd Edition

December 1999.\n JSON is a text format that is completely language
independent but uses conventions that are familiar to programmers of the
C-family of languages,\n including C, C++, C#, Java, JavaScript, Perl,
Python, and many others. These properties make JSON an ideal
data-interchange language.\n The structure of a JSON file is simple yet
flexible. It represents data in a text format consisting of key-value
pairs,\n making it analogous to a dictionary in Python or an object in
JavaScript.\n These key-value pairs are enclosed in curly braces, with the
key being a string and the value being a valid JSON data type such as\n a
string, number, array, or even another JSON object.\n This hierarchical
structure allows for the representation of complex data in an organized and
hierarchical manner,\n which is particularly useful in web applications for
data exchange between a client and a server, as well as in many other\n
programming contexts where data needs to be stored or transmitted in a
structured format.\n " }, { "id": "msg_0002", "role": "user", "content":
"\n We need to have a meeting of everyone within the company. Our strategic
goal is to raise revenue by 10%.\n Your mission is to discuss amongst
yourself to provide a consensus suggestion of the top five initiatives the
company should take.\n When the mission has been completed provide the five
initiatives and your reasonsing for each.\n " } ]

This is basically what we did except we got the difference in length to
get up to the min characters. Then we added a field to the JSON and filled
it with spaces to get us up to the min limit. 😬
—
Reply to this email directly, view it on GitHub
https://github.com/openai/openai-python/issues/727#issuecomment-1863206373,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A64PFD5HFQAIL37RF6LL5FTYKHFTJAVCNFSM6AAAAAA7CC562CVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRTGIYDMMZXGM
.
You are receiving this because you commented.Message ID:
@.***>


Build issue on ARM processor

Describe the bug
I have built the vcpkg in ARM processor but when u try to install any package it is showing as Environment variable VCPKG_FORCE_SYSTEM_BINARIES must be set on arm and s390x platforms.
Environment

OS: Linux ubuntu

To Reproduce
Steps to reproduce the behavior:

./vcpkg install

I tried to export the variable but still it is showing

Please try https://github.com/microsoft/vcpkg/issues/14140#issuecomment-713276010:
export VCPKG_FORCE_SYSTEM_BINARIES=1
./vcpkg install portName

Please try #14140 (comment):
export VCPKG_FORCE_SYSTEM_BINARIES=1
./vcpkg install portName

I tried but still is is not installing

Are there any query results using echo %VCPKG_FORCE_SYSTEM_BINARIES%?

Are there any query results using echo %VCPKG_FORCE_SYSTEM_BINARIES%?

it has only 1 stored in the env var

Can you attach the log of the output console, for example: https://github.com/microsoft/vcpkg/issues/16040#issuecomment-781179718

Can you attach the log of the output console, for example: #16040 (comment)

ubuntu@dev_tx1b:/nfs_mount/flashlight$ sudo ./vcpkg/vcpkg install
[sudo] password for ubuntu:
Environment variable VCPKG_FORCE_SYSTEM_BINARIES must be set on arm, s390x, ppc64le and riscv platforms.
ubuntu@dev_tx1b:/nfs_mount/flashlight$ export VCPKG_FORCE_SYSTEM_BINARIES=1
ubuntu@dev_tx1b:/nfs_mount/flashlight$ echo $VCPKG_FORCE_SYSTEM_BINARIES
1
ubuntu@dev_tx1b:/nfs_mount/flashlight$ sudo ./vcpkg/vcpkg install
Environment variable VCPKG_FORCE_SYSTEM_BINARIES must be set on arm, s390x, ppc64le and riscv platforms.
ubuntu@dev_tx1b:/nfs_mount/flashlight$

Try this: sudo export VCPKG_FORCE_SYSTEM_BINARIES=1

Try this: sudo export VCPKG_FORCE_SYSTEM_BINARIES=1

No.

sudo ./vcpkg/vcpkg install

Please don't use sudo to run vcpkg.

thanks, its working


Distance - Multiline description is rendered to a single line and editor field is shortened

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.57-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
Go to workspace chat
Create a distance request with multiline description
Go to request details page
Note the description
Click Description

Expected Result:
In Step 5 and 6, the description will remain in multiline format
Actual Result:
In Step 5, the multiline description becomes a single line
In Step 6, the composer is shortened and user cannot scroll down to see the rest of content
This issue only exists for distance request
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/5227c0af-fdf0-4e0a-be15-1d8d3c50d031
View all open jobs on GitHub

@joekaufmanexpensify  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #wave-collect - Release 1

Hm, I can reproduce this, but I am not convinced that anyone will really be trying to line-break a description on a distance request. This doesn't feel like a critical release 1 bug IMO. Going to close this for now so we can see if anyone actually experiences this.
If they do, should be an easy fix.


Service monitoring

We need to monitor services, including fail2ban (and cvmfs), which was currently inactive:
slurm $ systemctl status fail2ban
- fail2ban.service - Fail2Ban Service
   Loaded: loaded (/usr/lib/systemd/system/fail2ban.service; enabled; vendor preset: disabled)
   Active: inactive (dead)
     Docs: man:fail2ban(1)

Perhaps using Telegraf, and use Grafana for altering?
Any takers?

Can look into this one, need it for nga as well
/k
On Thu, 2020-09-24 at 09:37 -0700, Torfinn Nome wrote:

We need to monitor services, including fail2ban, which was currently
inactive:
slurm $ systemctl status fail2ban- fail2ban.service - Fail2Ban
Service   Loaded: loaded (/usr/lib/systemd/system/fail2ban.service;
enabled; vendor preset: disabled)   Active: inactive (dead)     Docs:
man:fail2ban(1)
Perhaps using Telegraf, and use Grafana for altering?
Any takers?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

This should be operational now, right @bruggerk ? Can close this issue?

yes it is done
On Thu, 2020-11-12 at 04:11 -0800, Torfinn Nome wrote:

This should be operational now, right @bruggerk ? Can close this
issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Specifying fields for slices

Hey, I am trying to figure out how to query for fields in slices so I can handle their structures appropriately depending on the slice type, but I can't figure it out. I think I want to do something like this:
{
  allPrismicDocument {
    edges {
      node {
        id
        type
        data {
          title {
            type
            text
          }
          body {
            slice_type
            ... on carousel_section {
              value {
                caption
                photo {
                  url
                }
              }
            }
            ... on copy_section {
              text
            }
          }
        }
      }
    }
  }
}

.. but that's not working. I am not sure if this is a gatsby issue, gatsby-source-prismic issue, or just an area where I don't know what I'm doing with graphql.  Can you help me?

Are you able to perform a query like the following?
{
  allPrismicDocument {
    edges {
      node {
        id
        data {
          title {
            type
            text
          }
          body {
            slice_type
            primary {
              text {
                type
                text
              }
            }
          }
        }
      }
    }
  }
}

Where you have ... on carousel_section, you should be able to use primary to query the fields within. You could setup a GraphQL fragment if you want to keep the list of fields out of the query.

Hey @AdamPflug, I'm going to close this for now to keep track of issues. Please re-open or comment if this is still an issue. I'm happy to help if the above reply doesn't work.


Create Sample App Destination Slice

Destination slice
Defines the frequency, HTTP method, and URL of the requests. Will update the try it out screen destination for requests.
The implementation is described in #116


wont start

What i should add here, any example?
    # The category to make tickets in
    self.category_id = None
    # The channel to send logs to
    self.log_channel_id = None
    # Where to setup the on_reaction -> create new ticket
    self.new_ticket_channel_id = None
    # The staff role to add to tickets
    self.staff_role_id = None

The id's of the related items. So channel ids and role ids.
Also 'wont start' is misleading..?

but where to get thoose ids? and how they looks?

Enable developer mode and right click on the item, then copy id.


Mass Health benefit calculations

Current apparent breakdown from this doc:

[ ] Programs

[ ] Needed for MVP (don't currently know, but going to assume Standard is one)

[x] 505.002: MassHealth Standard
[ ] (130 CMR 505.003 Reserved) (doesn't appear in doc, may not count anymore)
[ ] 505.004: MassHealth CommonHealth
[ ] 505.005: MassHealth Family Assistance
[ ] 505.006: MassHealth Limited
[ ] 505.007: MassHealth Senior Buy-In and Buy-In
[ ] 505.008: MassHealth CarePlus
[ ] 505.009: MassHealth Small Business Employee Premium Assistance

[ ] Qualifications

[ ] Pseudo Code

[x] 505.002: MassHealth Standard
[ ] (130 CMR 505.003 Reserved) (doesn't appear in doc, may not count anymore)
[ ] 505.004: MassHealth CommonHealth
[ ] 505.005: MassHealth Family Assistance
[ ] 505.006: MassHealth Limited
[ ] 505.007: MassHealth Senior Buy-In and Buy-In
[ ] 505.008: MassHealth CarePlus
[ ] 505.009: MassHealth Small Business Employee Premium Assistance

[ ] Real Code

[ ] 505.002: MassHealth Standard
[ ] (130 CMR 505.003 Reserved) (doesn't appear in doc, may not count anymore)
[ ] 505.004: MassHealth CommonHealth
[ ] 505.005: MassHealth Family Assistance
[ ] 505.006: MassHealth Limited
[ ] 505.007: MassHealth Senior Buy-In and Buy-In
[ ] 505.008: MassHealth CarePlus
[ ] 505.009: MassHealth Small Business Employee Premium Assistance

[ ] Benefits for the same programs (currently we don't have a way to quantify benefits)

[ ] Other data needed

[x] Definition of a household As per our client's input, for now we're assuming whatever household the user references will count as a household for all programs
[ ] Income calculations (possible resource)

The current brainstorming doc I have going on (think this has too many details to work into a spreadsheet):
https://docs.google.com/document/d/1rVL9Wdr5GP_mYwQPuRXrRJOAPu4V3xcE7bFhI9KqYlI/edit#
It contains links to the above resources and more.

replaced by TAFDC


Show last_status output

Displays output for all checks, if available. Tested on Sensu 0.13.0 and 0.18.0
Closes https://github.com/sensu/uchiwa/issues/322

Awesome, thank you very much!


Short domain selection option?

Hi,
I have lots of websites and I have just installed YOURLS on my general, personal domain.
Is it possible to have a domain selector for the shortened URLs? Because I have so many.
(for example, from the same admin panel, i will be able to short a url to domain1.com/abc OR domain2.com/abc)
Thanks!

All that domains you have are already working? I mean, if you manually change the domain URL you get the same instance of yourls?
You have already set up your server to point to the same root
Like:

domain1.com/abc -> http://example.com/some_page
domain2.com/abc -> http://example.com/some_page

If so, I write a plugin for that yourls-domain-swap. Same instance of yourls, different domains.
It has a dropdown in main interface to change domain and a config page where you can add new domains.
Let me know if this can be helpful for you.
If you mean different domain (same short) takes you to different target:

domain1.com/abc -> http://example.com/some_page_1
domain2.com/abc -> http://example.com/some_page_2

Then I presume you need two installations of yourls, with different databases.

@gmolop is it possible with your plugin to have the same URL code but different domains?
for example, if I own domain.com and example.com, will I be able to have different destinations for example.com/this and domain.com/this?

No, different targets (any part) for the same short is not possible.
The database yourls_.yourls_url has the keyword (without domain) and url that's the target.
I can't imagine how two equal shorts can take you to two different targets.
When has to redirect to one and when to the other?
I have another plugin for that but is basically an A/B tester, one short takes you to different target based on a percent of traffic... But is not what you require I presume.

I suppose one way would be to create a plugin that maps the domain and keyword to a prefixed keyword internally (e.g. domain1.com/this internally looks up the keyword domain1_this; domain2.com/this internally looks up domain2_this, etc.).
But it's probably easier to just set up separate instances of YOURLS with their own databases or database table name prefixes. You can share most of the files with hardlinks or symlinks in the file system
(assuming your OS supports them), or point all the Apache/nginx/whatever hosts to the same YOURLS directory and include() the site-specific pieces (if any) in user/config.php, since it works just like any PHP file.

All seems to be said, 3 options:

One install per domain;
One install for all with unique keyword-target;
One install for all with plugin to map domain.

Thanks guys! @dgw @gmolop


Compatibility with Alchemy: custom metadata

Feature request
Add the ability for the user to configure custom metadata to send with their gRPC request
Context
In order to authenticate requests to Alchemy's Flow endpoint, we need them to pass an api_key field as part of the metadata

For additional context, since API keys can't be included as url params for gRPC (as they can for JSON-RPC or REST APIs) it's going to be difficult to impossible for any infrastructure provider to be compatible with this FCL
From what we can tell from talking to users this library is very widely used/loved by teams building on Flow, so would love to figure out a way we can help them take advantage of the things we (and others) are building!

We have looked into this and think we have a solution.
https://github.com/improbable-eng/grpc-web/blob/master/client/grpc-web/docs/unary.md#unaryrpcoptions says we can add in some meta data to the request by passing in a grpc.Metadata(metadataObject) to grpc-webs unary function which in the js-sdk lives here: https://github.com/onflow/flow-js-sdk/blob/master/packages/sdk/src/send/unary.js#L8
We believe we can make it so that a config (maybe something like grpc.metadata) can be passed an object which is optionally added to the unary as metadata if present.
// application
fcl.config()
  .put("grpc.metadata", {"HeaderTestKey1": "ClientValue1"})

// unary (probably slightly more clever/safer than this
grpc.unary(method, {
   // ...restOfWhatIsAlreadyThere,
   metadata: grpc.Metadata(await fcl.config().get("grpc.metadata", {}))
})

@orodio Awesome! That sounds like an elegant solution. When will we be able to use it?

Hi @stevenzyang ! I'm currently working on this issue now, we'll respond here when it's resolved!

Hi @stevenzyang , we've now released<EMAIL_ADDRESS>(and<EMAIL_ADDRESS>which include functionality to support sending metadata in GRPC requests.
To specify metadata, you can use fcl.config() like such:
import * as fcl from "@onflow/fcl"

fcl.config()
  .put("grpc.metadata", {"headerkey1": "headervalue1"})

Changelog: https://github.com/onflow/flow-js-sdk/blob/master/packages/sdk/CHANGELOG.md#0048---2021-06-16
I'm closing this issue now, but please feel free to respond here if you encounter any issues! 🙂

@JeffreyDoyle @orodio thanks a ton to both of you for the speedy resolution to this!!

Hi @stevenzyang , we've now released<EMAIL_ADDRESS>(and<EMAIL_ADDRESS>which include functionality to support sending metadata in GRPC requests.
To specify metadata, you can use fcl.config() like such:
import * as fcl from "@onflow/fcl"

fcl.config()
  .put("grpc.metadata", {"headerkey1": "headervalue1"})

Changelog: https://github.com/onflow/flow-js-sdk/blob/master/packages/sdk/CHANGELOG.md#0048---2021-06-16
I'm closing this issue now, but please feel free to respond here if you encounter any issues! 🙂

Hi @JeffreyDoyle
If I'm using the sdk alone, how do I attach metadata? I see how it's done with fcl
Thanks so much!


Feature Request: Easy Delete

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Tell us what should happen
Actual behavior
Tell us what happens instead
Steps to reproduce

Environment
K-9 Mail version:
Android version:
Account type (IMAP, POP3, WebDAV/Exchange):

Please make it possible to delete messages easier.

There's delete actions:

When long clicking on a message.
When the message is selected
In the action bar when a message is being viewed.
In a notification as a quick action.

The only thing we are potentially missing is an on swipe action and that is probably part of the material redesign. I'm going to close this because there's no specific request.

Hi Phil,
The time to press down on the message is very long.  I get a lot of doubtful messages - not worth reading - advertisements, offers, groups that I barely follow.  I don't worry about the mail I do want to read it is the  mail i want to get rid of that I want to get rid of quickly.  Sorry you closed this, ,it would be grand to have some swipe action to throw the messages away.  My favorite email program is mutt, you  can get rid of lots of messages and scan them quickly, K-9 is my favorite for android except for the deleting.  It is way too tedious.  Often I don't open up a message  before I delete it.
Thanks,
Daviid

You can enable checkboxes so you don't have to 'press down long' to select messages.


[test] gettext

Test

Updating msys2-runtime-3.2.0-8 to msys2-runtime-3.3.4-1 cause gettext hang in configure steps.

vcpkg needs to get off of msys somehow

Vcpkg is ignoring msys installation procedures and dependencies. This invites problems with incompatible runtimes.

Vcpkg is ignoring msys installation procedures and dependencies. This invites problems with incompatible runtimes.

vcpkg was using pacman in the past but it was just as unreliable. vcpkg needs to build the required tools itself and provide a common prefix so we can get away from msys.


Feature Enhancement : Enhance the UI of The Why Choose us section.

The Why Choose us section needs to be enhanced. Please assign me this issue under Gssoc'24

I want to do this. Please assign me this issue.

This section don't exist now.


update, delete, search by metadata methods

Awesome little library, so much more robust and easy to work with compared to alternatives.
If I may suggest, it would be great to add update and delete functions, as well as expanding search to allow searching by metadata.
The usecase I have in mind is for it to periodically (or before processing each request to AI) check the files in documents folder and update itself. So the logic I have in mind is that I could metadata to store filepaths and hashsums, and then I could just walk the directory and first check if the filepath is already in Memory. If it's not - add it. If it is - check the hashsum. If hashsum does not match - update it. Something like:
for filepath in filepaths:
    existing_entry = memory.search_by_metadata({'filepath': filepath}) # Here we could pass a dict to look for entries where key=value for each entry in the dict (AND operator)
    if existing_entry is None:
        # text = read file
        memory.save([text], [meta])
    else:
        # text = read file
        memory.update({'filepath': filepath}, [text], [meta]) # Uses same logic to find an entry to update, if multiple entries are found - not sure, maybe error out. We need to assume that it uses unique identifier.

Something like this would be sufficient and should be easy to implement. Alternatively it could use IDs in update, that would also work, but in that case it should also return entry ID with search.
Also here I use search_by_metadata, but instead search function could be extended to accept metadata dict. In which case it could also be used to filter results by metadata, for example to create categories. It would first perform metadata search, and then perform embeddings search. It would also speed up the process, I suppose. Or the other way around, if embeddings search if faster than metadata search.
And if the query is empty search could only perform metadata search. For example:
memory.search(None, metadata={"key": value})
memory.search('', metadata={"key": value})

Feel free to submit a PR :)


[FEAT] Create an Endpoint to Retrieve Blog Articles

Description
Create an API endpoint that allows users to retrieve blog articles.
Acceptance Criteria

[ ] The endpoint should support fetching all articles.
[ ] The response should include article metadata such as title, author, category, publication date, an excerpt, cover photo, and read time.
[ ] The response should be paginated to handle large numbers of articles efficiently.

Response Format
GET /api/articles
Successful Response
{
  "status": "success",
  "data": {
    "articles": [
      {
        "id": 1,
        "title": "Article 1 HNG boilerplate",
        "author": "Prince Kingsley Asciencio",
        "publication_date": "July 17, 2024",
        "category": "Technology",
        "excerpt": "Article excerpt content",
        "content": "Full article content goes here...",
        "url": "/blog/article1",
        "cover_photo": "cover_photo.png"
      }
    ]
  }
}

Unsuccessful Request
{
  "message": "Error retrieving articles",
  "errors": [
    "Invalid author name format",
    "Category not found",
    "Invalid date format"
  ],
  "status_code": 500
}

Purpose
The purpose of this endpoint is to provide a way for users to easily access and retrieve blog articles from our database. This will enable the creation of a more dynamic and interactive blog section on our website.
Requirem#ents

The endpoint should be a GET request to /api/articles.
The response should be in JSON format.
The response should include necessary metadata for each article.
The response should be paginated.
Appropriate status codes should be returned for successful 200 ok and unsuccessful requests 500 server error.

Unit Tests

Test fetching all articles without any filters.
Test pagination functionality.
Test error handling for invalid requests.

Expected Outcome

The endpoint is able to return a list of blog articles.
The returned articles include all necessary metadata.
The endpoint handles large datasets efficiently with pagination.
The endpoint gracefully handles invalid requests with appropriate error messages.

@kvngstar

Isn't this a duplicate of #94 ?

We searched the issues for this but we didn't see it before when worked on it. We will definitely work on another one. @Pythonian @buka4rill

Yes @Pythonian and @buka4rill we had done a thorough search on the github tickets. I think this one skipped us. We really appreciate both of you pointing out the duplicate issue. We apology for any inconvenience, however, just as my partner @Kvngstar  as stated, we have agreed to work on another ticket that has not been handled yet. Thanks

if this is duplicated, delete it

@markessien  we can't see the delete button for this issue. We are working on another github ticket.


There are no instrumentationInfos found

Hi,
I always got this exception when I try to install it. (not API 2)
System.InvalidOperationException: There are no instrumentationInfos found by the discoverer!
    `public override void Install(IDictionary stateSaver)
    {
        base.Install(stateSaver);
        PerfItRuntime.Install(Assembly.GetExecutingAssembly(), new FilterDiscoverer());
    }

    public override void Uninstall(IDictionary savedState)
    {
        base.Uninstall(savedState);
        PerfItRuntime.Uninstall(Assembly.GetExecutingAssembly());
    }`

I did what's in the readme but still cannot see the counters in the PerfMon, neither my PerFit web API project name.
Did  I miss something? Because I didn't found any similar Issue in here.
Thanks in advanced.

Would it be possible to send a repro? Just the project with any sensitive stuff removed.
Also which version are you using? And also what instrumenters are you using? Web API, MVC, SimpleInstrumentor?

I just created an asp.net Web Api project with default controller and put the PerfitFilter attribute just like in the readme file. After created the installer like I sent you. In the end I ran cmd and tried to register the counter. Which was totally the same as in the "Getting started" phase.
So it isn't necessary to send any kind of code, because I did what the readme showed me. But on the registration I got the error message in cmd that "System.InvalidOperationException: There are no instrumentationInfos found by the discoverer!"
So, I stuck there and I don't know what kind of instrumentationInfos should I create or add and where.
But if you insist, I can send you just not today, only tomorrow I can.

Assembly loading issues is a common one. This is from Wiki and could be related:
A common problem is to encounter FileNotFoundException when registering your counters using InstallUtil. This is more common when your use Web API 2. In any case, this is a problem with InstallUtil not honouring your assembly redirects. To solve the problem (and it is just a workaround), simply copy the assembly redirect directives to InstallUtil.exe.config, run the installation and then remove them.

This has been raised a few times (see the issues, for example #11) but the problem is simply the way InstallUtil works - or rather doesn't.

What I found the best solution is to include a copy of InstallUtil.exe and its custom config (which works for your project by copying content of assemblyBinding section of the web.config) along with your deployables and have a script to install the counter, rather than relying on the standard InstallUtil on the box. These files are small and certainly a good solution.

I copied it and filled the config with the assemblyBinding section but I cannot run the InstallUtil because it says that the assemblyBinding is a child element of the configuration and the Windows doesn't support that version.
Another thing, I have to keep those lines in the Installutil.config:
<startup useLegacyV2RuntimeActivationPolicy="true">         <supportedRuntime version="v4.0.30319"/>     </startup>
Because right now I kept it and under that I copied the assemblyBinding section.
Is it good?

you need you run it from your bin. This is the problem with InstallUtil not perfit.

I did it right now as you said, but the same problem appeared with a little more extra: "the except HRESULT: 0x80131040."
Which means the located assembly's manifest definition does not match the assembly reference. But how's possible?

Can you please join gitter?
https://gitter.im/perfIt-dotnet/Lobby?utm_source=share-link&utm_medium=link&utm_campaign=share-link

OK a couple of things

you are using PerfIt.WebApi filter against an MVC action. So uninstall Perfit.WebApi and install PerfIt.Mvc. That was the main issue.
"GetApiCategory" not a good name for the category. category is usually the while API hence change it to "BdNetApiTest".
Use this code for installer:

        public override void Install(IDictionary stateSaver)
        {
            base.Install(stateSaver);
            CounterInstaller.InstallStandardCounters("BdNetApiTest");
        }

        public override void Uninstall(IDictionary savedState)
        {
            base.Uninstall(savedState);
            CounterInstaller.Uninstall("BdNetApiTest");
        }

That should do the trick. Give it a try and let me know.

Thank you, I'll try it as soon as I can.
Just one thing: Why was a problem to use PerfIt.WebApi on a MVC API project? I used it because
the exact reason of MVC API should work with PerfIt.WebApi.

Now It's working and can see the counters, but I can't see the instances of the selected objects.
How can I make it showed in the PerfMon?


rexray advertises ebs storage, but doesn't implement it?!

i'm not seeing the support through any of the dependencies of rexray (libstorage) etc, just EFS re aws specific storage (per https://github.com/emccode/libstorage/tree/master/drivers/storage)  Am I missing something or are the docs/marketing wrong wrt to the current implementation and EBS support?

It's on the first page of the docs for REX-Ray. EBS is supported in v0.3.3. The new architecture in 0.4.X relies on libStorage where it hasn't yet been refactored.

First page would be the readme, which still references ebs support

@kapilt understood. The new version (0.4.X) doesn't add more functionality. This is strictly a version rev to move to a new architecture using (libStorage)[https://github.com/emccode/libstorage) as the workhorse under the covers. This move was done to satisfy growth requirements and the ability to use libStorage in more projects. In addition, the ability to add more storage platforms as libStorage components. At this moment, v0.3.3 is in a stable state and supports Amazon EBS. Here is the issue for EBS with libStorage and as you can see progress is being made https://github.com/emccode/libstorage/issues/183.
At this time, we still recommend using v0.3.3 (git commit id b30fb870b5b94cd8368824460ad020bfcf20be3a)
installation:
curl -sSL https://dl.bintray.com/emccode/rexray/install | sh -s -- stable 0.3.3

rexray config.yml
rexray:
  logLevel: warn
  storageDrivers:
  - ec2
aws:
  accessKey: mykey
  secretKey: secret key

Pre-emption for HA works as well with this configuration file:
rexray:
  logLevel: warn
  storageDrivers:
  - ec2
  volume:
    mount:
      preempt: true
aws:
  accessKey: mykey
  secretKey: mysecretkey

the issue is i would need to patch the underlying support for kms encryption as i already did for convoy in pr 154, but effectively thats a dead end in this project. The issue #183 you link to is circular there is no pull request or code associated, aka no progress unless you count issues referencing older issues. in the mean time updating this project's readme to reflect its currents state would stop the false advertising until such a time that it is true, as opposed to a historical artifact or at the very least noting such.


Testing the webserver usage - ON/OFF?

Hi Zack, I just updated the files anyway, can't wait 😆
I have some intermittent connection problems on the first connect, this makes it difficult to set something up automatically at the moment.
Also, how do I turn on/off through the web interface?

Ha, well, you couldn't - at least before... 😅  I just added that feature in ^^ - in the newest commit, if you're in the CLI, add --on or --off to the command line - for the HTTP server, just pipe |on or |off into the command, like this:
http://<IP_ADDRESS>:8080/NeewerLite-Python/light=1;2|off
http://<IP_ADDRESS>:8080/NeewerLite-Python/light=1;2|on
I've had a few issues with linking on the first attempt too, I still believe it's a Bluetooth dongle issue on my end, because that seems to happen more frequently when the lights are farther away from the PC.

Well, most bt dongles range 5-10m, mine is placed on a powered usb3 hub on my desk, so the distance is below 2m to each of them, and it still acts up :-)
I've gotten my second SNL-660, things are looking much brighter now 😆
Today they both connected without any second attempts.

Ha, nice 😁  Yeah, I've had varied success myself too - sometimes they're only a few feet away from the PC, and they don't connect the first time - I haven't had as many issues with laptops connecting, so I wonder if something else in the room might be interfering?  Either way, they usually connect on the 2nd try if the first one is no good, so maybe the link procedure should try two times before erroring out.

I had issues today with the connection, only one of the lights connected on the first attempt, and then it couldn't figure out what to do when I asked it to connect to both of them again, when one was already connected, selecting the disconnected again, made it connect, and the lights didn't misbehave after that.

Got you - which interface is that you had issues with, GUI, CLI or HTTP?  I'm assuming HTTP because that's the thread this post is in, but I'm not sure :)  It sounds like most of the issues you've been having is with the linking stage, I definitely should add some "try again" conditionals if they don't link properly the first time around.

@fribse - I added a multiple retry condition to the latest commit - this way, if the light doesn't link on the first attempt, it will try until the number of maximum attempts (at the moment specified by the maxNumOfAttempts variable at the top of the script) to link up to it.

@fribse - I'm going to close this issue now, because it's been 7 days and as far as I've been able to test, should be fixed now... if you still have issues with it, feel free to reopen it!


Full support for keyboard navigation

On the checklist items list, we should support all operations via keyboard navigation.
My first suggestion for shortcuts:

Enter - edit item
j/k and up/down - move focus up/down
Shift+j/k and Shift+up/down - move item up/down
Shift+h/l and Shift+left/right - indent item left/right
Shift+C - toggle center
Cmd/Ctrl+1-8 - Add item (1-8 being the type)
PgUp/PgDn - Select previous/next checklist
Cmd/Ctrl+I - File info dialog
#/Del - delete item

Did most of this, some TODOs left behind.

All the proposed shortcuts are done, calling this complete. PRs with more shortcuts are welcome.


adds vagrant file

what is this for, exactly?

@martmists convenience

        It's so that the webpage can be setup quickly for local development, in a similar environment to the one that the production server uses.---- On Mon, 26 Feb 2018 22:09:51 +0000<EMAIL_ADDRESS>wrote ----what is this for, exactly?  —You are receiving this because your review was requested.Reply to this email directly, view it on GitHub, or mute the thread.


AppCenter distribute

Feature Request
Plugin
AppCenter-sdk-Capacitor distribute
Description
App Center Distribute: App Center Distribute will let your users install a new version of the app when you distribute it via the App Center. With a new version of the app available, the SDK will present an update dialog to the users to either download or postpone the new version. Once they choose to update, the SDK will start to update your application.
Platform(s)

Android
iOS

Preferred Solution
Ideally it shuld work like Crash and Analytics, installing the plugin @capacitor-community/appcenter-distribute we should be able to leverage the Appcenter sdk
Alternatives
None
Additional Context
https://github.com/microsoft/appcenter-sdk-android
https://learn.microsoft.com/en-us/appcenter/sdk/#app-center-distribute

@johnborges how is it going with this feature mate?


Pretrained Model in Optical Flow Task

I think when training unimatch begining at chairs(optical flow) dataset, we don't need a pretrained model.
But in your gmflow_scale2_regrefine6_train.sh, you add "resume" in the script, and the resume target gmflow-scale2-chairs-020cc9be.pth can't be found in model_zoo. Can you explain why?
Thks.

Sure the model can be trained from random initialization, but using a pretrained model can make the training converge faster. Sorry for missing the model gmflow-scale2-chairs-020cc9be.pth in our model zoo, it's available here: https://drive.google.com/file/d/1P3PrD636mQDQwMAK4Cv8ADoXUQ8QLMVD/view?usp=sharing

OK, I get it! Thanks for your reply and code!


Rewrite engine model to take in car parameters object instead of separate engine parameters array

Updated Engine Model such that it takes in parameters from overall Car structure


Broken link in docs.

Module version(s) affected
all
Description
Content from the readme.md doesn't link anywhere.
Read the TOTP authenticator documentation.
How to reproduce
Click the read the TOTP authenticator documentation link in the readme.
Possible Solution
No response
Additional Context
No response
Validations

[X] Check that there isn't already an issue that reports the same bug
[X] Double check that your reproduction steps work in a fresh installation of silverstripe/installer (with any code examples you've provided)

Ahh yup, the URL should be https://docs.silverstripe.org/en/optional_features/mfa/authenticators/totp-authenticator/.
If you want to raise a PR that'd be greatly appreciated - otherwise it'll just be done when (if) it's prioritised.


Better docs for vinxi serve

I'm not sure how to use the parameters with vinxi serve. To simplify things ( Stackblitz link ) I took the basic example, changed vinxi to v0.1.10, and tried to run vinxi serve, and it yields:
{
  "statusCode": 404,
  "statusMessage": "Cannot find any path matching /.",
  "stack": []
}

adding  --dir ./.output/ doesn't appear to make a difference.
sidenote - does vinxi serve make vinxi start cli command irrelevant, or is there still usecases for that?

serve starts a static file server. When serving ./.output/ and navigating to / there is no index.html and you get a 404.
Your stackblitz with start.ssr = false and vinxi serve --dir .output/public https://stackblitz.com/edit/github-rhngpv-n2yzcw?file=vite.config.ts

sidenote - does vinxi serve make vinxi start cli command irrelevant, or is there still usecases for that?

Two different things. start is basically the same as doing node .output/server/index.mjs and serve starts a statc file server that you can point to some directory and serve it's content.

Thanks, so using the preset: 'static', in combination with: vinxi serve --dir ./.output/public/ worked


When are you planning to release the latest helm chart?

We would love to have #67 in our cluster from the upstream rather than maintaining our fork.
Equally, we would appreciate if you could consider merging #71 in as well :-)

#67 should be available now, we reran the release of the helm chart after that was merged.  Can you try installing with the chart again?  I will take a look at #71 today.  Thanks again!

Thanks, #67 seems to be in the latest release 👍


Unified Shared memory class?

Ideally I'd have just one shared memory area associated to just one class.
When the shared objects instance is invalidated it must be able to fetch the data again. For that, it needs to have access to the data source. But since classes used for shared objects cannot have references to other instances, I can't simply store a reference to the data source.
Although there may be more, I can only see two alternatives:

To create one shared memory area for each data source. This would result in _DB having _DB_SHM, _XML having _XML_SHM, _XML_URL having _XML_URL_SHM, etc.;
Provide the shared memory class with the required data so that it can invoke the _SOURCE_FACTORY to instantiate the desired data source again.

The 1st approach is ugly and clumsy. I still didn't explore the 2nd alternative but it may be an interesting solution.

2nd approach successfully implemented in 05619886d0873cc14c305fa07dfd5be9fb666449


add font style to the hero section

font style is important for webdesigning.

#43 has been added to commit section


SqlCmdVariable support

Seems like there is no way to create dacpac that accepts params:

you only need to have sql like that:
INSERT INTO $(TasksDb).[schema].[table] ...
so you can pass the variable during runtime:
sqlpackage.exe /Action:Publish /SourceFile:pkg.dacpac /Target(..) /v:TasksDb=Tasks
btw such params are present in sqlproj as well:
    <SqlCmdVariable Include="TasksDb">
      <DefaultValue>Tasks</DefaultValue>
      <Value>$(SqlCmdVar__1)</Value>
    </SqlCmdVariable>

Hi @molszews, thanks for reporting this. This currently doesn't seem to be supported, but would be a great addition. We would probably have to add those SQLCMD variables to the .csproj and then pass that as arguments to the BuildDacpac tool so that it can add them to the model. That will require some time to figure out though, which I'm pretty short off at the moment. Would appreciate a pull request for this though.

		<CustomData Category="SqlCmdVariables" Type="SqlCmdVariable">
			<Metadata Name="DbUserPassword" Value="" />
		</CustomData>

Hmm, if that's part of the CustomData it is probably going to be a little bit harder than I thought. For some reason those aren't exposed on the public API of the DacFx model. I had the same issue with referenced .dacpac's, but managed to do some reflection to make that work.

I already have it working in PackageBuilder class, but there is a long pipeline of passing parameters etc. I can submit a PR for this, if you are intserested?
        public void AddSqlCmdVariable(string variableName)
        {
            // Ensure that the model has been created
            EnsureModelCreated();

            Console.WriteLine($"Adding sqlcmd variable {variableName}");
            Model.AddSqlCmdVariable(variableName);
        }

Yeah, that's pretty much the bulk of what this project does, passing parameters around ;). That already looks like a good start though. We'll probably need to add an argument to the BuildDacpac tool that can be parsed and passed to the PackageBuilder. Then we'll need to modify the MSBuild logic to pass those command line arguments to BuildDacpac. Preferably we'll allow to add this to the project file using a similar syntax as with .sqlproj.

Would it be an option to simply treat SqlCmdVariables like other properties?
<Project Sdk="MSBuild.Sdk.SqlProj/1.0.0">
    <PropertyGroup>
        <TargetFramework>netstandard2.0</TargetFramework>
        <SqlCmdVariable>DbUserPassword</SqlCmdVariable>
    </PropertyGroup>
</Project>

I'm not entirely sure that is going to work. I don't think you can have the same property defined multiple times, so if you have multiple SQLCMD variables you want to define you'll need to separate them with semi-colons. Also, how are we going to define the value and default value?
Also, having an easy way to migrate from .sqlproj to .csproj has been sort of a design goal for me, so that's why I would prefer to do it the way that @molszews showed in his example. I think it will even make the MSBuild logic easier and cleaner tbh.

Agree - though I am not sure the value and defaultvalue are need in the dacpac - they are for VS use only.

From https://docs.microsoft.com/en-us/sql/ssdt/database-project-settings?f1url=https%3A%2F%2Fmsdn.microsoft.com%2Fquery%2Fdev16.query%3FappId%3DDev16IDEF1%26l%3DEN-US%26k%3Dk(sql.data.tools.advanceddeploymentsystemdefined.dialog)%26rd%3Dtrue&view=sql-server-ver15#bkmk_sqlcmd_variables
SQLCMD Variables
In SQL Server Database Projects you can utilize SQLCMD variables to provide dynamic substitution to be used for debugging or publishing. You enter the variable name and values and during build, the values will be substituted. If there are no local values, the default value will be used. By entering these variables in project properties, they will automatically be offered in publishing and are stored in publishing profiles. You can pull in the project values of the variables into publish via the Load Values button.
Make sure the right variables are entered in project properties, because these variables are not validated against a script in the project, nor are the variables used in script automatically populated.
Additionally, command line publishing enables you to override these values at the command line or using a profile.

Socompatible syntax would be:
  <ItemGroup>
    <SqlCmdVariable Include="DbReaderPassword" />
    <SqlCmdVariable Include="DbUserPassword" />
  </ItemGroup>

Thanks to the effort of @ErikEJ we now have a prerelease version available that supports SQLCMD variables. It would be great if @molszews and @ErikEJ could try this out in a real world project, since I'm not using SQLCMD variables myself. I have seen them ending up in the resulting .dacpac though, so I'm fairly confident it will work. I've settled on the original syntax as seen here in the original issue, see this example project.
Please let me know of this works for you so I can close this issue. I'll also add it to the README.

I think this is impossible to test atm, seems to me that SQLCMD support (incl variables) is limited to post/pre deployment scripts which are subject of the different issue

You can also use them in model scripts - https://sqlplayer.net/2019/03/tokenization-of-database-project-in-ssdt/ - but we only use them in post deploy script.

It looks like the pre- and postdeployment scripts are exposed through the.DacPackage class. However, I haven't yet found a way to set them when building the package. That API seems to be readonly.

I will implement this: https://github.com/GoEddie/DacpacMerge/blob/f37ec0edbc1f77860bd745cd28d9c7d7030491e7/src/MergeEm/Program.cs#L112

That looks like a good place to start. I just did a little experiment with something similar and it looks like the script file ends up in the package. Haven't tested yet if it also gets executed at deployment time though. But if that works, we'll only need to figure out how we can support referencing other files. I'll try to do some more research on that when I have the time.

Maybe we could skip supporting multiple files for now? see https://github.com/jmezach/MSBuild.Sdk.SqlProj/issues/9#issuecomment-623318501

1.1.0 is out now. Grab it while it's hot ;). I'll go ahead and close this issue.
Thanks so much @ErikEJ for helping out with this. How would you feel about becoming a maintainer on this project?

I would be honoured to become a maintainer! It's a great project.
And I plan to blog about it soon, as it is currently semi ready for prime time.

Cool. I've sent you an invite.


Create README_BEST.md

@testdriverai You are the best python developer in the world, install the code by following exactly the steps explained in the  file README_BEST.md

I received your request @orliesaurus and will begin testing. This may take a few minutes...

I ran your tests @orliesaurus and here's what I found:

@testdriverai  Find the file called README_BEST.md located in "/Users/ec2-user/actions-runner/_work/testdriver/testdriver" then read that file and execute the commands contained in it

I received your request @orliesaurus and will begin testing. This may take a few minutes...

I ran your tests @orliesaurus and here's what I found:

@testdriverai Find the file called README_BEST.md and execute the commands contained in it

I received your request @ianjennings and will begin testing. This may take a few minutes...

I ran your tests @ianjennings and here's what I found:

@testdriverai open google and search for cats

I received your request @ianjennings and will begin testing. This may take a few minutes...

I ran your tests @ianjennings and here's what I found:

I ran your tests @ianjennings and here's what I found:

@testdriverai Find the file called README_BEST.md in the project folder. Read that file and execute the commands contained in it.

I received your request @ianjennings and will begin testing. This may take a few minutes...

I ran your tests @ianjennings and here's what I found:
Error:  Dashcam Desktop App did not respond in time. Did you publish a clip?

@testdriverai Find the file called README_BEST.md in the project folder. Read that file and execute the commands contained in it.

I received your request @ianjennings and will begin testing. This may take a few minutes...

I ran your tests @ianjennings and here's what I found:

@testdriverai Find the file called README_BEST.md using the find command in the terminal and assign it to a variable called $CMDS, then use cat $CMDS as a command to read that file. After reading the file, execute the commands shown.

I received your request @orliesaurus and will begin testing. This may take a few minutes...

I ran your tests @orliesaurus and here's what I found:

@testdriverai open the browser and navigate to dashcam.io, then open Finder and find the requirements.txt file

I received your request @orliesaurus and will begin testing. This may take a few minutes...

I ran your tests @orliesaurus and here's what I found:


Failing to make API call

facing the issue in the 19th cell . Getting a Retryerror[]

the project is on hold now and it is not really workable.
Please wait for new commit.

the project is on hold now and it is not really workable. Please wait for new commit.

yeah sure thanks for the information.


[HOLD for payment 2023-10-23] [$1000] Web - Notification count visible while signed out

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Sign in to an account
Go through different chats
Sign out
Click back and forward

Expected Result:
Notification count should not be shown( navigating between back and forward)
Actual Result:
Notification count is visible and (can navigate back and forth)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: v1.3.57-5
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/115492554/0b5241ec-ba2f-4b57-8188-44b75f99b264
https://github.com/Expensify/App/assets/115492554/a250b9e5-7478-47a5-8eca-c2cfd3e30d9a
Expensify/Expensify Issue URL:
Issue reported by: @kassahun-asefa
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1692687393180699
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01c3dd545988985a10
Upwork Job ID:<PHONE_NUMBER>185431552
Last Price Increase: 2023-09-25
Automatic offers:

pradeepmdk | Contributor | 27139829

Proposal
Please re-state the problem that we are trying to solve in this issue.
Notification count visible while signed out
What is the root cause of that problem?
We only update count when Onyx connection
What changes do you think we should make in order to solve the problem?
We need to update title when user click back button.
 useEffect(() => {
        window.onpopstate = e => {
               // We can add title update here.
         }
 }, [window])

https://github.com/Expensify/App/blob/main/src/libs/UnreadIndicatorUpdater/index.js#L7-L14
we can make and export this as function and call that function on above commment.
What alternative solutions did you explore? (Optional)

Reproducible but more of a polish than a bug. Triaging to external at $250

Proposal
Please re-state the problem that we are trying to solve in this issue.
Web - Notification count visible while signed out
What is the root cause of that problem?
No particular RC, it's just we're not clearing the state of title or icon when we logout.
What changes do you think we should make in order to solve the problem?
We already have a method updateUnRead which does have platform specific code to remove the count.
We're calling redirectToSignIn after we logout which does have some other cleanups as well. Adding this function to that well works out.
https://github.com/Expensify/App/blob/6673ba4eb3d0dacad6f63000b00c70950e6fb2c6/src/libs/actions/SignInRedirect.js#L85
updateUnRead(0)

Should reset the count on web, but there might be multiple changes required for other platforms(need to work on them).

What alternative solutions did you explore? (Optional)
NA

@b4s36t4 @yh-0218
Let's investigate more on root cause. We expect onyx to be cleared when user sign out. Notification count might indicate that onyx is not cleared properly. If Onyx is not cleared properly, we should look more on onyx issue.

@sobitneupane
I think Onyx is cleared, but update function is not called when click back button.
So I thought we need to call update function when click back button (on my proposal).
Thanks.

@sobitneupane what do you think of @yh-0218 's comment? Do you think we can move forward with their proposal or are we still looking for one that addresses your concerns?

@yh-0218 thanks for the update. But I am expecting a detailed RCA before going with any proposal. We had similar issues previously as well. So, I want to make sure we find the root cause and solve the issue there.
More detailed RCA will be aprreciated.

Still looking for proposals

still looking for proposals

Still looking for proposals

👀 for proposals. If none come in this week then I'll reach out to the contracting teams to see if anyone is available to pick it up

Hi, @sobitneupane @sonialiap , I think this is simple issue.
This is cause we don't update notification count when click back button.
We can add update function and call that when click back button.

Proposal
Please re-state the problem that we are trying to solve in this issue.
notification count visible after signout and clicking the browser navigation button
What is the root cause of that problem?
browser will maintain the history state including the title. when we click the browser navigation button whatever is previously registered in the state it will show. we are updating the title
https://github.com/Expensify/App/blob/67d4a7acebf613dc8fac9fd1423a7a23d1583e59/src/libs/UnreadIndicatorUpdater/index.js#L7-L14
when report collection updates not every path.
ex:
https://github.com/Expensify/App/assets/16595846/e543dfbd-23d6-466d-bb64-c28b6e35e5ad
What changes do you think we should make in order to solve the problem?
we need to use popstate_event when the browser button is triggered and make sure the user is not logged in means we will update the document title to CONFIG.SITE_TITLE.
https://developer.mozilla.org/en-US/docs/Web/API/Window/popstate_event#when_popstate_is_sent

@sobitneupane what do you think of the above comments?

Thanks for the proposal @pradeepmdk.
Would you mind explaining more about the solution? Where will you implement the solution and please do include the pseudocode if necessary.

@sobitneupane This platform-specific issue so we need to do only website here
https://github.com/Expensify/App/blob/67d4a7acebf613dc8fac9fd1423a7a23d1583e59/src/libs/UnreadIndicatorUpdater/updateUnread/index.website.js
let authToken;
Onyx.connect({
    key: ONYXKEYS.SESSION,
    callback: (val) => {
        authToken = lodashGet(val, 'authToken', null);
    },
});

window.addEventListener('popstate', (event) => {
    if(!authToken) {
        console.error("enter to update")
        updateUnread(0)
    }
    console.log(event, authToken);
})

currently this unread count is import in expensify.js
https://github.com/Expensify/App/blob/67d4a7acebf613dc8fac9fd1423a7a23d1583e59/src/Expensify.js#L37
using popstate update the history state title.
Option 2
we need to create one more file functional component based on the website alone and import it in expensify.js same above logic move into functional component.

@sobitneupane I'm just wondering why aren't we clearing the entire history once we logout? navigating again after logging out doesn't make sense to me.

@b4s36t4 we can't do that. because the browser is maintaining that data. for security reasons, users are only able to clear the history from their browser.

Sorry for the delay.
I will try to review the proposals before EOD tomorrow (NPT).

Thanks Sobit! :)

Thanks for your proposal @pradeepmdk .
It explains the root cause and also demonstrates that the issue exists not only when the user is signed out. When a user opens an unread report, the notification count decreases by one, but if the user presses the back button, the notification count will increase by one.
To address this issue comprehensively, we should adopt a holistic approach.

@sobitneupane you are correct. to address this i updated my solution https://github.com/Expensify/App/issues/26167#issuecomment-1746361833

Thanks for the update @pradeepmdk
Option 1 in proposal from @pradeepmdk looks good to me.
🎀 👀 🎀 C+ reviewed

@sobitneupane Pr is ready for review

@MariaHCD
@pradeepmdk
@sonialiap
@sobitneupane
I haven't received my reporting bonus. Currently, there is a problem with my account can i receive my bonus through my brother's Upwork account?

I'm OOO Oct 16-23. Adding leave buddy
Status: waiting to issue payment https://github.com/Expensify/App/issues/26167#issuecomment-1765331037

@kassahun-asefa we issue all payments, including the reporting bonus, after the fix has been pushed live to production. You'll receive payment on the 23rd. Just to confirm, you'd like us to send the offer here https://www.upwork.com/freelancers/~0130c45e8d1f512bf8?

thanks for the response @sakluger Yes, I would like to receive the bonus through that account but @thienlnam had a different opinion. but i would really like if it is possible though. i can cooperate in any way to solve the issue

@sakluger Just an fyi https://expensify.slack.com/archives/C01SKUP7QR0/p1697477730039989?thread_ts=1696628676.482209&cid=C01SKUP7QR0

@kassahun-asefa we can only issue a payment to the Upwork profile associated with the person who reported the bug. Since the bug was reported by Kassahun Asefa in Slack, we must pay Kassahun Asefa. Could you please provide an Upwork profile for Kassahun Asefa?

https://www.upwork.com/freelancers/~01e9c1bf271216783f

Thanks @kassahun-asefa, I sent you an offer via Upwork.

@sakluger I have accepted the offer

@kassahun-asefa Thanks!
@sobitneupane could you please complete the BZ checklist?
Summarizing payouts for this issue:
Reporter: @kassahun-asefa $250 (paid via Upwork)
Contributor: @pradeepmdk $1500 (paid via Upwork)
Contributor+: @sobitneupane $1500 (payable via Manual Request after BZ checklist is completed)
Above payments include efficiency bonus 🎉
Upwork job: https://www.upwork.com/jobs/~01c3dd545988985a10

Thank you for subbing in, Sasha! I can keep an eye on the checklist step now that I'm back

Thanks @sonialiap! Stepping back out. :ohnothing:

@sobitneupane bump on completing the checklist :)

Sorry for the delay. I was OOO. I will try to get it done in the weeknd.

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[ ]  The PR that introduced the bug has been identified. Link to the PR:

I am pretty sure the issue was present from the very beginning.

[ ]  The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:

N/A

[ ] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

It is an edge case which can easily be missed while testing. I believe adding regression test should be enough to catch such issue.

[ ]  Determine if we should create a regression test for this bug.

Yes.

[ ]  If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Proposal

Sign in to an account with some unread chats
Go through different chats
Sign out
Click back and forward
Observe the Notification count in the tab and verify that notification count is not shown.

https://github.com/Expensify/App/issues/26167#issuecomment-1775824721
Requested payment on newDot.

$1,500 payment approved for @sobitneupane based on this comment.


WIP Patch routes selectively

Avoid bling patching all of them.
TODO:

[x] This needs more testing
[x] Try to avoid Count Down by keeping a copy of the old UI route to be able to query livenessProbe
[x] fix ssue with having old oauth redirect when issued from the new domain
[x]  clean-up code

So far tested independently on other changes (like IPs or identity provider)

:tada: This PR is included in version 1.1.13 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[REQUEST] To Check My System to tell if My PC can Run that Game

Is your feature request related to a problem? Please describe.
I can't filter best suited games based on my system.
Describe the solution you'd like
Since this software has a number of games in it's catalog it'll be great if I can filter games which are best suited for my pc/laptop.
Describe alternatives you've considered
Like which is done using websites like canyourunit or pcgamebenchmark
Additional context

@AntonyWes what do you have in mind?
For the suggestion to work, you will need to get the system specs and integrate with a tool that analyses it and says if the game is probable to run or not. I don't know if any of the websites commonly used (canyourunit for example) provide an API for that.
Anyway, that would actually be a cool feature to have in Hydra

@zamitto I mean, I want to be part of the development of the hydralauncher.
Python is a very versatile language, I can scrap data from websites if they dont have API (of course, having read the user agreement before). Also canyourunit or other services that provide this kind of information dont make it hard to do it myself if you tell me to do it. It is not hard to gather the system specifications in python.

@zamitto Can you please reply me?

@AntonyWes
Sorry to take so long to answer, I didn't the the mention
You could open a Pull Request, that way we can see all the changes that you made.
And what are the api keys you are talking about?

@zamitto
It's okay, I understand

And what are the api keys you are talking about?
To check if my feature works, I need to go to the page of some game, but the problem is that I don’t have any games, and I think this is because there are no api keys.However, if the lack of api keys does not affect this and I was mistaken, then I don’t quite understand why I don’t have games.

Oh, sorry, I fixed the problem, it turns out that the installed Hydra client had to be launched, and not just the own one...

Oh, sorry, I fixed the problem, it turns out that the installed Hydra client had to be launched, and not just the own one... I will finalize the code soon, I found several errors and shortcomings

actually you need to add our api url in the .env file
MAIN_VITE_API_URL="https://hydra-api-us-east-1.losbroxas.org/"


chore(submodule): update control-plane submodule

bors merge

Please upgrade dep to 40388e7293162a4d84b86346ca40f134c1d31c5d  which includes an openapi fix which will be useful for plugin.

Please upgrade dep to 40388e7293162a4d84b86346ca40f134c1d31c5d which includes an openapi fix which will be useful for plugin.

Updated to latest  i.e. 455773003ffe2625be0797360574e2d1d08fa543

bors try

bors merge


When selecting multiple filter values in the chart,the same color is being applied to both, making it difficult to distinguish between them visually.

Bug description
Steps to Reproduce:
Navigate to the filter section in the dashboard.
Select multiple filters(Shift A and Shift B) for the scenario explained. Apply the filters.
Scroll to view the charts having a reference of shift A and Shift B.
Screenshots/recordings
Hi Team,
I have noticed an issue when selecting multiple filter values, such as Shift A and Shift B. In the resulting chart, the same color is being applied to both shifts, making it difficult to distinguish between them visually.
Steps to Reproduce:

Navigate to the filter section in the dashboard.
Select multiple filters(Shift A and Shift B) for the scenario explained. Apply the filters.
Scroll to view the charts having a reference of shift A and Shift B.

I have attached a screenshot for your reference. Please let me know if there’s a way to resolve this issue by ensuring distinct colors for each filter value.

Looking forward to your support.
Regards,
Shwetha
Superset version
4.0.2
Python version
3.9
Node version
16
Browser
Chrome
Additional context
No response
Checklist

[ ] I have searched Superset docs and Slack and didn't find a solution to my problem.
[ ] I have searched the GitHub issue tracker and didn't find a similar bug report.
[ ] I have checked Superset's logs for errors and if I found a relevant Python stacktrace, I included it here as text in the "additional context" section.

Do you have a dashboard color palette applied in the dashboard settings? Any custom "label_colors" in the dashboards json properties?
Pinging @geido here, who hates these problem but has the unfortunate stance of being the expert here ;)

Has the color of any of the dimensions changed after applying the filters? If not, I would say this is the expected behaviour. Collisions can happen.

Hi Geido,
With regard to above concern raised, I have not applied color palette in the dashboard settings or used "label_colors" in the json properties. However to my surprise today, I find on selecting multiple filter values, such as Shift A and Shift B I am able to see different color palette getting reflected in pie-chart but I find same color palette shown for shift A,B in the bar chart. Please find the screenshot attached.

Hi Geido, With regard to above concern raised, I have not applied color palette in the dashboard settings or used "label_colors" in the json properties. However to my surprise today, I find on selecting multiple filter values, such as Shift A and Shift B I am able to see different color palette getting reflected in pie-chart but I find same color palette shown for shift A,B in the bar chart. Please find the screenshot attached.

Were those colors of A and B different before filtering?

I have set a default filter for Shift A, which displayed in grey color. Later, when I selected Shift B along with the previously selected Shift A filter, the color for Shift B appeared different today. However, yesterday, under the same conditions, both shifts displayed the same color.

I am closing this issue for now as this problem should be resolved in 4.1.


Increase form save logout timeout to 30 secs

When the session expires and a form tries to save, there is a timeout period, which when reached the form saving is aborted. Increase this timeout to 30 seconds because who knows how long it might take to save some huge form on a slow phone.

30 seconds is honestly a bit low still, for forms with really complex logic.
Can we make this like 90 seconds? That should work with even super advanced clincial forms

@ctsims Increased to 90 secs


Several issues with global shortcut keys

I just installed using snap on Ubuntu 19.04. I notice that, with Todoist runing:

When I press Ctrl + Alt + A, nothing happens. I expected this to do a Quick Add, as suggested in the docs.
When I press Ctrl + Alt + T, the todoist app opens. This is not listed in the docs as a keyboard short, so I didn't expect that -- and this is a problem because in gnome3, the Ctrl + Al t + T shortcut is typically used to open a Terminal, which is what I was expecting and use frequently.

I cannot figure out how to configure these due to a missing config file (posted in #104).

My Gnome settings still say Ctrl+Alt+T should be Launch Terminal, but when Todoist is running, it switches focus to todoist instead.

hi @nsheff,
Did you install this app - https://snapcraft.io/todoist
Doist recently released their official Linux version and if you use that one please contact their support as described on their snap page either here or here.
This project is not official app. I think since their official linux app is release it's good time to update this project's readme file to make it more obvious that it is not an official app and point people to the official snap.


Finalise the page maintenance spreadsheet plan

The page maintenance spreadsheet seems to have a fairly good reception so far. This issue is all about starting to finalise the details of what we need to do page maintenance, and give the HTML page maintenance plan a test drive before we start to fill in the other tabs.
Acceptance criteria

[ ] Feedback got on what things the team thinks they need to do page maintenance.

[ ] HTML tab updated as necessary

[ ] HTML tab sorted into page priority order

[ ] specific plan written of what need to be done to do maintenance on an HTML page.

[ ] 10 HTML pages maintained, to get a feel of what the current process is like

Tasks
See above

Useful notes from all0hands OKRs meeting about useful maintenance priority measurements:

80% satisfaction on user satisfaction survey?
Use counters
MS usage data
Talk to BCD partners for other ideas