sorry for all the testing spam

Hi @HCasler,
You have proposed changes to files in these packages:

JobConfig

which require these tests: build.
@Mu2e/write have access to CI actions on main.
:hourglass: The following tests have been triggered for 0747ccc95f991e290cdc6cfdf70bf41fdd49493b: build  (Build queue has 2 jobs)
About FNALbuild. Code review on Mu2e/Offline.

This comment should not result in any response from the bot

@FNALbuild this should get a :confused: response from you

:umbrella: The tests failed for 0747ccc95f991e290cdc6cfdf70bf41fdd49493b.

Test
Result
Details

merge
:white_check_mark:
Merged 0747ccc95f991e290cdc6cfdf70bf41fdd49493b at c947ccaf2b0e4e6d66091c258cad8a58a49169d0

build (prof)
:white_check_mark:
Log file. Build time: 12 min 51 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:x:
Log file. Return Code 9.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 0747ccc95f991e290cdc6cfdf70bf41fdd49493b after being merged into the base branch at c947ccaf2b0e4e6d66091c258cad8a58a49169d0.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

@FNALbuild run build tests with Mu2e/Offline#550

:hourglass: The following tests have been triggered for eea41d8d67343da84ee16485f559fb5917bff58e: build  (Build queue has 3 jobs)

:umbrella: The tests failed for eea41d8d67343da84ee16485f559fb5917bff58e.

Test
Result
Details

test with
:white_check_mark:
Included Mu2e/Offline#550 @ d44b7323998339b12d4cee84886ba2c8ca4ea334 by merge

merge
:white_check_mark:
Merged eea41d8d67343da84ee16485f559fb5917bff58e at c947ccaf2b0e4e6d66091c258cad8a58a49169d0

build (prof)
:white_check_mark:
Log file. Build time: 14 min 00 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:x:
Log file. Return Code 9.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at eea41d8d67343da84ee16485f559fb5917bff58e after being merged into the base branch at c947ccaf2b0e4e6d66091c258cad8a58a49169d0.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


PSCore6 - Not available for Ubuntu version 18.04 Bionic Beaver Release

Just an FYI!
I wonder when is going to become available for this new OS Release of Ubuntu.
Steps to reproduce
I had to try the following:
sudo curl https://packages.microsoft.com/keys/microsoft.asc | sudo apt-key add -

sudo curl -o /etc/apt/sources.list.d/microsoft.list https://packages.microsoft.com/config/ubuntu/18.04/prod.list

sudo apt update

sudo apt-get install -y powershell

Expected behavior
To install PowerShell Core.

Actual behavior
It can't find the package.
:
Reading state information... Done
E: Unable to locate package powershell

Environment data

maxt@ubuntu:~$ cat /etc/os-release
NAME="Ubuntu"
VERSION="18.04 LTS (Bionic Beaver)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 18.04 LTS"
VERSION_ID="18.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=bionic
UBUNTU_CODENAME=bionic
maxt@ubuntu:~$

That one should be coming soon. For now, it will work by installing libssl1.0.0 and libicu60 and then installing the binary archive.
We actually already have the packaging done for Ubuntu 18.04, the remaining work is publishing the repo.

Awesome!!  Thanks @rjmholt!!
:)


[10.x] Fixes $request->user() or Auth::user() run query each time if token doesn't exists

fixes laravel/framework#49201

Don't think we should fix - probably could break some other use case. I think if $request->user() returns null once (meaning unauthenticated) you wouldn't keep calling it over and over again in your app.


--noImplicitAny error not reported for variable declaration

TypeScript Version:  3.5.0-dev.20190412

Search Terms:
Code
Compile the following ts with --noImplicitAny
let x;
if (Date.now()) {
    x = 1;
} else {
    x = { obj: true };
}

foo(x);

function foo(y) { }

Expected behavior:
The declaration let x should produce an implicit any error
Actual behavior:
No error reported. After the assignment, the type of x is number | { obj: boolean }
Playground Link:
https://www.typescriptlang.org/play/#src=let x%3B
if (Date.now())%20%7B%0D%0A%20%20%20%20x%20%3D%201%3B%0D%0A%7D%20else%20%7B%0D%0A%20%20%20%20x%20%3D%20%7B%20obj%3A%20true%20%7D%3B%0D%0A%7D%0D%0A%0D%0Afoo(x)%3B%0D%0A%0D%0Afunction%20foo(y)%20%7B%20%7D
Related Issues:
From https://github.com/Microsoft/vscode/issues/72193

There is no any behavior displayed here. Writes to x are tracked, and if the read type doesn't match its use, an error is issued.
Intuitively, this is entirely consistent with you writing
let x = Date.now() ? 1 : { obj: true };

which is not an error either.

That does not seem correct to me. The programmer can assign x to just about anything, which makes it the same as if they had written: let x: any:
let x;

for (const a of []) {
    x = 1;
}
if (Date.now()) {
    x = new Date();
}
switch (Date.now()) {
    case 1: x = ''; break
   case 2: x = () => {} break
}

foo(x);

function foo(y) { }

Type inference will make sure x has the correctly inferred type when it is used

but the assignments themselves are the problem we are trying to catch

Why should that code have an error, but not this code? The problem here is foo accepting an any.
for (const a of []) {
    foo(y);
}
if (Date.now()) {
    foo(y);
}
switch (Date.now()) {
    case 1: foo(''); break
   case 2: foo(() => {}) break
}

function foo(y) { }

Thanks for the link to the issue. I can better understand why this decision was made but still I'm not convinced from a language user perspective that this let x; should not be treated as an implicit any in cases where x is assigned to different types.
For example, thy should be an implicit any error:
function foo(x?) {
    if (Date.now()) {
        x = 'str';
    }
    return bar(x);
}

function bar(x: any) { }

while this is not:
let x;
if (Date.now()) {
    x = 'str';
}
bar(x)

function bar(x: any) { }

Because one of them creates a usable any target site where no any was specified (foo({ asdf: 10 })), and the other doesn't.

You should think of it in these terms: Was I allowed to access a property of an any-typed variable without writing : any ?
function foo(x?) {
    x.asdf; // yes
}

vs
let x;
x.asdf; // no
if (Date.now()) {
    x = 'str';
    x.asdf; // no
}
bar(x)
x.asdf; // no

function bar(x: any) {
  x.asdf; // I *did* write :any
}

Consider an example like:
const parseError = (err: Error) => {   let errData;   try {     errData = JSON.parse(err.message);     // log errData   } catch (e) {     // log failure   }   if (errData['some-key'].includes('403')) {     throw new Error('some error message');   }   throw err; };
When we go to access errData['some-key'] when errData wasn't set, Node will throw the Error Cannot read properties of undefined (reading 'some-key')
errData is implicitly of type any here, which is why TS won't protect me from accessing properties on it even though it can clearly be undefined. I try really hard to watch out for these issues, but on a team with 10s of engineers they just keep making it into the codebase. Is there no way to get protection from this kind of "implicit any"?

@RyanCavanaugh OK, yeah, I see how it is different now. Of course I can cast that type, I just wish TS could enforce this because it is all over our repo now. I think the correct solution is probably that JSON.parse and the like should return type unknown instead of type any.

I also think it is strange, because the options noImplicitAny should work like it named, help find the any which is not explict set

For anybody find a eslint solution for this: https://stackoverflow.com/questions/73440042/typescript-prevent-inferring-types-except-at-variable-declaration


Update WPT upstream webhook.

This integrates https://github.com/servo-automation/upstream-wpt-sync-webhook/pull/19.

This change is

@bors-servo r+

:pushpin: Commit 0e7e87d has been approved by jdm

:hourglass: Testing commit 0e7e87d6125f352dbc7778487dbbfa09cfbe4ebe with merge 73e2b22ec582907a4775c0950be8ffc24c2fa968...

:sunny: Test successful - checks-travis
Approved by: jdm
Pushing 73e2b22ec582907a4775c0950be8ffc24c2fa968 to master...


Check for Condition within Message Headers

Can you please tell me how to check for a string within the message headers?  Specifically, I need to see if the original message originated from a particular IP address.

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

ID: a38b2975-5eb6-1aa4-c1fd-d0f033ace5ca
Version Independent ID: 85899731-9939-a779-29c4-4da652667b74
Content: Run flows based on email properties - Microsoft Flow
Content Source: flow-docs/email-triggers.md
Service: flow
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

@macWyo Thanks for submitting docs feedback. Sorry for the delay in responding. If you haven't found the answers you were looking for, please submit your question on the Power Automate Community forums. As I don't see a documentation issue here I am going to close this one. Thank you.


Python 3.6 EOL, fixing CI

This seems to fail in parsing a WFS response
=================================== FAILURES ===================================
____________________ [doctest] wfs_MapServerWFSFeature.txt _____________________
001
002 Imports
003
004     >>> from owslib.wfs import WebFeatureService
005
006 Test the getfeature method
007
008     >>> wfs = WebFeatureService('http://nsidc.org/cgi-bin/atlas_south?', version='1.0.0')
009     >>> response = wfs.getfeature(typename=['glaciers'], maxfeatures=5)
UNEXPECTED EXCEPTION: ParseError('junk after document element: line 123, column 0')
Traceback (most recent call last):
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/doctest.py", line 1337, in __run
    compileflags, 1), test.globs)
  File "<doctest wfs_MapServerWFSFeature.txt[2]>", line 1, in <module>
  File "/home/runner/work/OWSLib/OWSLib/owslib/feature/wfs100.py", line 300, in getfeature
    headers=self.headers, auth=self.auth)
  File "/home/runner/work/OWSLib/OWSLib/owslib/util.py", line 220, in openURL
    se_tree = etree.fromstring(req.content)
  File "/opt/hostedtoolcache/Python/3.7.15/x64/lib/python3.7/xml/etree/ElementTree.py", line 1315, in XML
    parser.feed(text)
  File "<string>", line None
xml.etree.ElementTree.ParseError: junk after document element: line 123, column 0
/home/runner/work/OWSLib/OWSLib/tests/doctests/wfs_MapServerWFSFeature.txt:9: UnexpectedException

When added Python 3.10 we got this error:
=================================== FAILURES ===================================
___________________________ [doctest] namespaces.txt ___________________________
039     # 'om300' does not exist as a namespace, so the below will return nothing
040     >>> ns.get_namespace('om300')
041
042     # CSW 3.1.1 doesn't exist, so the below will return nothing
043     >>> ns.get_versioned_namespace('csw','3.1.1')
044
045
046 Invalid Usage Tests
047
048     >>> ns.get_namespace()
Differences (unified diff with -expected +actual):
    @@ -1,3 +1,6 @@
     Traceback (most recent call last):
    -...
    -TypeError: get_namespace() ...
    +  File "/opt/hostedtoolcache/Python/3.10.8/x64/lib/python3.10/doctest.py", line 1350, in __run
    +    exec(compile(example.source, filename, "single",
    +  File "<doctest namespaces.txt[15]>", line 1, in <module>
    +    ns.get_namespace()
    +TypeError: Namespaces.get_namespace() missing 1 required positional argument: 'key'

The 3.10 issue is rooted in how exceptions are emitted. The following works (and maintains test integrity):
diff --git a/tests/doctests/namespaces.txt b/tests/doctests/namespaces.txt
index 1c484e3..970328a 100644
--- a/tests/doctests/namespaces.txt
+++ b/tests/doctests/namespaces.txt
@@ -48,7 +48,7 @@ Invalid Usage Tests
     >>> ns.get_namespace()
     Traceback (most recent call last):
     ...
-    TypeError: get_namespace() ...
+    TypeError: ...

     >>> ns.get_versioned_namespace()
     Traceback (most recent call last):

The test in tests/doctests/wfs_MapServerWFSFeature.txt is caused by a broken / misconfigured WFS. Given this is out of our control, this can moved to tests/_broken/doctests for the time being.


[BUG] Cannot visualise/show RGB semantic segmentations masks

Instructions
Hi,
I'm trying to visualise imported RGB masks (.png) on 51, but while the masks looks like they are loaded, for some reason the image doesn't show up at all.
System information

OS Platform and Distribution: Windows 11, 22H2, Build: 22621.1555) using WSL2: Ubuntu-20.04
Python version (python --version): 3.10.9
FiftyOne version (fiftyone --version):
FiftyOne installed from (pip or source): pip, version: 22.3.1
Virtual Env Conda:  22.9.0
Chrome Version: 112.0.5615.138

Commands to reproduce
Steps:

Create a new virtual env with the specks above
Update the dataset paths to point to where the image and mask is stored on your local env.
Run the code below on a Jupyter env on VS Code
Open 51 on browser using Chrome on http://localhost:5151

Describe the problem
Fiftyone doesn't display correctly RGB masks stored as png image files.
The only way this can work is by first converting the mask to greyscale, save it, and then load that mask.
Code to reproduce issue
RGB code:
import fiftyone  as fo
import fiftyone.utils.labels as foul

dataset_rgb = fo.Dataset.from_dir(
    data_path="path/to/images",
    labels_path="path/to/masks_rgb",
    dataset_type=fo.types.ImageSegmentationDirectory,
    label_field="segmentations",
)

foul.segmentations_to_detections(
    dataset,
    "segmentations",
    "instances",
    mask_targets={
        "#000000": "__background__",
        "#32ff32": "big_dog",
        "#ff3232": "small_dog"
    },
    mask_types="thing"
)
dataset.save()
session = fo.launch_app(dataset)

Greyscale mask code:
import fiftyone  as fo
import fiftyone.utils.labels as foul

dataset = fo.Dataset.from_dir(
    data_path="path/to/images",
    labels_path="path/tomasks",
    dataset_type=fo.types.ImageSegmentationDirectory,
    label_field="segmentations",
)

foul.segmentations_to_detections(
    dataset,
    "segmentations",
    "instances",
    mask_targets={111: "big_dog", 170: "small_dog"},
    mask_types="thing"
)
dataset.save()

session = fo.launch_app(dataset)

Other info/logs
While converted masks to detections seem to have happenned correctly, the image doesn't load:

Example of detection:

What areas of FiftyOne does this bug affect?

[x] App: FiftyOne application issue
[ ] Core: Core Python library issue
[ ] Server: FiftyOne server issue

Willingness to contribute
The FiftyOne Community encourages bug fix contributions. Would you or another member of your organization be willing to contribute a fix for this bug to the FiftyOne codebase?

[ ] Yes. I can contribute a fix for this bug independently
[x] Yes. I would be willing to contribute a fix for this bug with guidance from the FiftyOne community
[ ] No. I cannot contribute a bug fix at this time

Data used for this use case/but.

Any suggestions, advice, or guidance on how to tackle this is welcomed :).
Thanks

Hmm, thanks for reporting this. I am able to produce the issue. The images fail to render specifically when working with the RGB masks.
I placed the original image and RGB masks here:
/tmp/dogs/images/dogs.jpg
/tmp/dogs/masks-rgb/dogs.png

and then ran this:
import fiftyone  as fo
import fiftyone.utils.labels as foul\

mask_targets = {
    "#000000": "__background__",
    "#32ff32": "big_dog",
    "#ff3232": "small_dog"
}

dataset = fo.Dataset.from_dir(
    data_path="/tmp/dogs/images",
    labels_path="/tmp/dogs/masks-rgb",
    dataset_type=fo.types.ImageSegmentationDirectory,
    label_field="segmentations",
)

dataset.default_mask_targets = mask_targets

foul.segmentations_to_detections(
    dataset,
    "segmentations",
    "instances",
    mask_targets=mask_targets,
    mask_types="thing",
)

# Looker fails to render
session = fo.launch_app(dataset)

# Everything works
session.view = dataset.exclude_fields("segmentations")

@benjaminpkane @sashankaryal can one of you track down why Looker doesn't like the RGB dog mask?


Seperate core functionality from app logic

[x] #72
[x] #73
[x] #74
[ ] #75
[ ] #94

The core application logic is now separated from the applications. Initially, the logic was combined in a core crate. It may make sense to split this core crate further and divide it into the sender, receiver and server components.


fixed bugs

Variable declaration statement missing var.
api.dom.email get email id bug in latest gmail page.

Thank you so much for the contribution and sorry for the extremely slow response!
As of now this bug has already been fixed separately: https://github.com/KartikTalwar/gmail.js/commit/1f57072c1ccdc3e65835832a80f2744b2b7a4b6b
The remaining code-cleanups are definitely welcome, but I think now obsolete.
For code-quality, I'm proposing a more aggressive clean-up here: https://github.com/KartikTalwar/gmail.js/issues/327 . You can chime in there if you feel this is important.
In the meantime I'm closing this PR, since most of these issues has already been fixed.
Again: Sorry about the slow response and thanks for contributing!


Handle import failure

Failure to import will result in an exception.

Nope. Throw exception is the right way.


file_picker ?

Hi, is there currently a way to prompt to pick any type file? Or just images?

image_picker

mate, you'll need to know both how to code for Android and IOS to do that. I've been able to do it on just Android!

@dwaindibbly ah ok. thanks


the function readWithCheck() is strange.

Sometimes , data is too long to read at one-time.
mBufferWrap.readWithCheck(local, len);
may fail.

This method will be used only in blocking mode(parse header and connect to remote). I add this since sometimes the socket is closed and the read return value is -1.
BTW, this class will be removed soon.

Plan to refine this in this weekend(or next :-x).  For client site, read the header together. then send package one by one.  For server, try to read 512 bytes in header, the non-header data will be stored and send to remote in pre-tcprelay stage.


Unable to Access fstogendata-ext Data via gsutil

Dear Project Team,
Thank you for the work you’ve done with this project!
I encountered a small issue when trying to follow the steps under Table Download to retrieve the data from the bucket gs://fstogendata-ext. Upon running the following command:
gsutil -m cp gs://fstogendata-ext/approved/REQSEC-61982/2024-08-16-labs-fst-datasets-2.tar _temp.tar

I received the following error message:
AccessDeniedException: 403 HttpError accessing https://storage.googleapis.com/download/storage/v1/b/fstogendata-ext/o/approved%2FREQSEC-61982%2F2024-08-16-labs-fst-datasets-2.tar?generation=1724896698249446&alt=media&userProject=fst-dev-project: response: <{'status': '403'}>, content <The billing account for the owning project is disabled in state absent>

It appears that the billing account associated with the owning project of the storage bucket has been disabled, preventing access to the data even though I am specifying a valid billing project using --billing-project. Since the bucket uses Requester Pays, I am unable to proceed with downloading the datasets needed for the workflow.
Could you kindly assist in resolving this issue? It would be helpful to verify the billing account associated with the bucket’s project (fstogendata-ext) and ensure it is active.
Thank you very much for your support and for this contribution!
Best regards,
Hui

Hi hui, can you try running the same command using gcloud storage? It seems like user-defined billing project's aren't enabled, per this source.
Let me know if that works

Yeah! That works! Thank you!


Use fmt::Display repr for ExitStatus when showing benchmark failures

This way unix signals don't get shown as just None, and windows errors will get printed out as hexadecimal

Hello! Thanks for the pull request.


Refetch first page on success mutation

Describe the bug
Hi there. I faced with the issue when trying to refetch first page for the infinite query. Here is my hooks
export const useInfiniteListMessagesQuery = (
  options: ListMessageOptions = {},
) => {
  const jwt = useSelector(selectAuthToken);
  const headers = new Headers();
  headers.set('Authorization', `Bearer ${jwt}`);
  const query = useInfiniteQuery<IList<Message>, HttpException, Message[]>(
    ['message', 'message_list', options],
    async ({ pageParam }) => {
      if (pageParam) {
        options = {
          ...options,
          page_after: pageParam,
        }
      }
      const queryParams = new URLSearchParams(options as any);
      const response = await fetch(
        'http://localhost:3000/v1/messages?' + queryParams,
        {
          method: 'GET',
          headers,
        },
      );
      return await response.json();
    },
    {
      select: (data) => ({
        pages: data
          .pages
          .map((list) => list.data)
          .flat()
          .reverse(),
        pageParams: [...data.pageParams].reverse(),
      }),
      getNextPageParam: (lastPage) => {
        if (!lastPage.has_more_next) {
          return;
        }
        const lastResource = lastPage.data[lastPage.data.length - 1];
        if (!lastResource) {
          return;
        }
        return btoa(
          JSON.stringify({
            id: lastResource.id,
            created_at: lastResource.created_at,
          }),
        );
      },
    },
  );
  const fetchNextPage = useCallback(() => {
    return query.fetchNextPage({ cancelRefetch: false })
  }, []);

  return {
    ...query,
    fetchNextPage,
  };
};

export const useCreateMessageMutation = () => {
  const jwt = useSelector(selectAuthToken);
  const queryClient = useQueryClient();
  const headers = new Headers();
  headers.set('Authorization', `Bearer ${jwt}`);
  headers.set('content-type', 'application/json');

  return useMutation<Message, HttpException, CreateMessageBody>('message', async (payload) => {
    const response = await fetch(
      'http://localhost:3000/v1/messages',
      {
        method: 'POST',
        headers,
        body: JSON.stringify(payload),
      },
    );
    return await response.json();
  }, {
    onSuccess: () => {
      return queryClient.refetchQueries(['message', 'message_list'], {
        refetchPage: (_, index) => index === 0,
      });
    },
  });
};

and here is my component where I'm using useInfiniteQuery
export const MessengerChat: FC = () => {
  const {
    data: { pages: messages = [] } = {},
    isLoading: isLoadingMessageList,
    hasNextPage,
    fetchNextPage,
    refetch,
  } = useInfiniteListMessagesQuery({
    limit: 50,
    ticket: router.query.ticket as string,
  });
}

Your minimal, reproducible example
Sorry, I can't provide reproducible example due to using external API
Steps to reproduce
When I call mutation, refetching queries works in inconsistency way, sometimes it refetches first page, but more often it refetches last page with page param. Also, I have tried to use useIsMutating to listen when mutation is fully completed
export const MessengerChat: FC = () => {
  const shouldRefetch = useRef(false);
  const {
    data: { pages: messages = [] } = {},
    isLoading: isLoadingMessageList,
    hasNextPage,
    fetchNextPage,
    refetch,
  } = useInfiniteListMessagesQuery({
    limit: 50,
    ticket: router.query.ticket as string,
  });
  const mutationMessages = useIsMutating(['message']);
  useEffect(() => {
    if (shouldRefetch.current) {
      refetch({ refetchPage: (_, index) => index === 0 });
    }
    if (mutationMessages) {
      shouldRefetch.current = true;
    }
  }, [mutationMessages]);
}

in this case refetching first page is working perfectly and every time
Expected behavior
I'm expecting queryClient.refetchQueries will always refetch only first page
How often does this bug happen?
Sometimes
Screenshots or Videos
No response
Platform

OS: macOS

react-query version
v3.39.1
TypeScript version
v4.6.4
Additional context
Does mutation call onSuccess not right after completes all updating? Probably mutation callbacks and updating query cache have some race condition which I need to avoid in another way?

Sorry, I can't provide reproducible example due to using external API

I've explicitly mentioned in the issue template how to solve this:

To stub out real api requests - Promise.resolve and Promise.reject are good options for easy reproduction

I don't have a compiler in my head, so I can't automatically see from a bunch of code what is going wrong. The only thing I do see is that in the queryFn, you are mutating options which are also part of the query key, which is never a good idea.
also, you are reversing the data in select, which is an observer level property. What is stored in the cache will always be what you return from the queryFn, and that is also what you get into getNextPageParam and refetchPage, because it's the only thing that exists in the central cache.

Wow, incredible. You say you don't have a compiler in the head, but seems you have :) First suggestion is working, the problem was in query key mutation, I didn't expect that.
P.S. Sorry, that I didn't read issue template good. Next time, I will do it. Thanks!

Doesn't work on V5.


Explore the use and value of a tool like CloudCheckr or DataDog

In order to have better visibility about resource consumption and costs of our cloud infrastructure, it would be useful to have a tool like CloudCheckr or DataDog available to us so that we can better track our cloud resource utilization.
We have used CloudCheckr in the past, but whatever we end up considering we should note the following:

It would need to be FedRAMP (JAB?) authorized
We'd have to consider cost as we'd likely have to buy it

If there's a buy involved, it might also be worth exploring if other parts of TTS could benefit from it.
Security Considerations

Having access to such a tool could help us stay on top of cloud resources that require updates/patching

This ticket is stale and does not reflect an active team need
There has been a lot of work around tagging resources more effectively and using AWS Cost explorer to break down costs by tag, which reduces the need for this ticket


Simulator Detection is not working with MEmu Android Emulator

Describe the bug
MEmu Android Emulator is not being detected as a simulator
Please complete the following information:

Emulator: MEmu
Version: 9.1.0 - 2024.02.2022
Version of freeRASP: 3.6.1

Hello @bwjctan1999,
Thank you for reporting. We'll look into that.
Best,
Tomas, Talsec dev

Hellp @tompsota, are there any updates regarding this issue or does this only occur on my end? Thanks


pdo_sqlsrv failed to report an out-of-bound column index to fetchColumn()

Problem description
Tested with the latest pdo_sqlsrv driver. When an out-of-bound index is passed to PDOStatement::fetchColumn(), the driver failed to report the error. PDO ended up throwing an exception.
Expected behavior and actual behavior
The pdo_sqlsrv driver is expected to report a more informative error like An invalid column index was passed for fetching but instead, we got a PDO exception as follows:
array(2) {
  [0]=>
  string(5) "HY000"
  [1]=>
  int(0)
}

Repro code or steps to reproduce
$database = 'master';
$server = 'myserver';
try {
    $conn = new PDO("sqlsrv:server=$server ; Database = $database", $uid, $pwd);
    $conn->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);

    $tsql = 'SELECT name FROM sys.objects';  // select only one column
    $stmt = $conn->query($tsql);
    $result = $stmt->fetchColumn(1); // or any out of bound index like -1
} catch (PDOException $e) {
    var_dump($e->errorInfo);
}

If we do not set PDO::ATTR_ERRMODE to PDO::ERRMODE_EXCEPTION we got PHP Warning:  PDOStatement::fetchColumn(): SQLSTATE[HY000]: General error: Invalid column index

Upon further investigation, PDO does the boundary check before passing the control to our pdo_sqlsrv driver. Yet, a new comment has been added to the existing bug report 77118 that is somewhat related.

The issue is considered fixed with PHP 8 RC1. When appending a new catch block like this to the repro script above, we will get the error message "Invalid column index"
} catch (ValueError $ve) {
    var_dump($ve->getMessage());
}


Changed JS hash private fields to TS private

During previous performance exploration (https://github.com/medplum/medplum/pull/1813) we found that using the JavaScript hash # private modifier caused measurable performance penalties when used in inner loops.
Because we use a relatively conservative JS version target, the # modifier was transpiled to a WeakMap with a whole bunch of additional boilerplate code.
In contrast, the TypeScript private modifier is only used at compile-time, and does not require transpiling, and does not have any runtime penalty.
There should be no functional changes in this PR.  Everything was done with a handful of search-and-replace calls.

Coverage: 94.002% (-0.01%) from 94.013% when pulling 598085cefca6ff131da6511740085783645eff97 on cody-change-hash-prefix-to-private into 8c7457fde2d7daae773c5ddaada4a0fa5a9d4979 on main.


Cannot build react-native project with Xcode 9 server

Is this a bug report?
Yes
Have you read the Contributing Guidelines?
yes
Environment
Environment:
OS: macOS High Sierra 10.13.1
Node: 9.2.0
Yarn: Not Found
npm: 5.5.1
Watchman: Not Found
Xcode: Xcode 9.1 Build version 9B55
Android Studio: 2.3 AI-162.4069837
Packages: (wanted => installed)
react: ^16.1.1 => 16.1.1
react-native: github:facebook/react-native#master => 1000.0.0
Steps to Reproduce

Create a new project with react-native init
Open the project in Xcode and verify that it will build locally.
Setup a build bot on an Xcode server for the project and manually start an integration.

Expected Behavior
The project will build the same as it did locally.
Actual Behavior
The project fails with the following:
    AppDelegate.m:12:9: fatal error: 'React/RCTBundleURLProvider.h' file not found
    #import <React/RCTBundleURLProvider.h>
            ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

Reproducible Demo
Any vanilla project created with react-native init will do.

I have the same problem for xcodebuild command. But Xcode 9 run build success.

I have to and XCode run and build successfully. After first buil the red message dissapiar

Surprisingly I can build with xcodebuild just fine. This lead me to create a custom make build target to run xcodebuild to see if that got it to work. It did not.  For some reason running on ccode build server causes it not to reference the React project for includes. I do have some ideas and will post when I’ve tried them out.

Nothing I have tried has worked.  It cannot resolve the React/ dependencies.  Looking at the project structure, I don't see any headers under a "React" folder.  They all seem to be in sub folders.  I'm not sure what the major difference is between running on a build server vs running in the editor but I really don't want to have to trigger builds manually.  Sort of breaks CI.

i am running into the same scenario. Would love to chat, maybe we can debug this. I have everything else working but the react native dependency cannot be found.

Yeah that is also what I am seeing.  I can't figure out why.  I'd love to chat :)

Have you found a way around this?

@TStrauch Not yet.  Still digging around.  I feel like it probably has to do with this new parallel build stuff in Xcode.  I've notice it doesn't seem to save to the project file.  This might not be the issue, but I've seen similar errors when parallel builds are turned on.

I'm also running into this issue where my CI XCode Server bot cannot resolve the React dependencies and fails at the first header file found in AppDelegate.m. Building/Archiving manually with the same scheme works perfectly. Anyone find a solution to this?

I have kind of given up on this I spent half a day with no one having any progress. I would love to see what people come up with. I am sure the ci builds for react native have some solutions.

To be honest, I've found that if you manually link react native, following the add to an existing project steps, the metro garbage doesn't run and you can get it to a point where I think, haven't tested, it would build with an integration system.  Honestly at this point I'm finishing up the projects that I have in react native and going back to programming in pure native code.  I just can't rely on react native as a viable solution at this point.

+1 any progress on solving this issue? This error has begun appearing for me today and cannot get rid of it. Tried all suggestions here https://github.com/facebook/react-native/issues/12077 but no luck. react-native init new project, build in XCode and error persists.

@garethiv Funny you should ask; I actually got my project building last night.  Here's how:
The first thing you should know is about the un/under-documented environment variable RCT_NO_LAUNCH_PACKAGER which will disable metro bundler from launching when your project is built.  You can set it to anything; if it exists then react won't launch the bundler.  If you're curious, the setting is in the Start Packager build phase of the React target in Libraries/React.xcodeproj

The next thing that is needed in order to get the bundler to work properly is to add node to the path before the script is called.  The only reliable way I found to do this is to modify the Bundle React Native code and images build phase of your product's target to append an environment variable to the end of path if that environment variable exists.

Then when you create your bot you can set your environment variables:

Lastly add a pre-integration script for your bot.  Something to keep in mind is that the default folder you get dropped in is the folder your repository is checked out into.  So the first thing you need to do is to cd into the name git clone will create, indicated in the screenshot as <repo-name>.  Then perform npm install and anything else you need to do.  Note: although I am using cocoapods for some libraries here, cocoapods does not build or link React Native as it will clash with the existing React.xcodeproj inside Libraries.

Hopefully facebook/react-native-website#370 will clear some of this stuff up.

so @Rojuinex did you write up anything on how to get this setup or links to help get it going. I gave up but would love to have this work.

@myusuf3 I haven't written anything up on how to do it, but I've actually personally moved to using GoCD + fastlane instead of Xcode Server and it's been working decently well.  The big gotcha's are RCT_NO_LAUNCH_PACKAGER=yes so metro server doesn't start (but it will still bundle the resources), and making sure Facebook's TeamID is removed from the React Xcode project files.  I submitted a PR that appears to be slated for 0.58 that fixes the TeamID issue.
If you can get fastlane to build under these two conditions it's pretty easy to set up GoCD's build environment to run fastlane.
The biggest caveat is when the app-specific password for your App Store Connect account expires and your build seemingly breaks for no reason.  As a build server isn't interactive fastlane will just hang forever; or at least I killed it before it reached any sort of time out (and it had been sitting in this state for very many minutes).
Maybe when I have some breathing room I'll put something together on how I finally got automated builds to work with React Native ;D


Response of Face - Identify

According to the document https://westus.dev.cognitive.microsoft.com/docs/services/563879b61984550e40cbbe8d/operations/563879b61984550f30395239, the response is a json object that contains a json array. However, when I use java to invoke the api, the response is a json array.

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

ID: 4cc723d9-668d-fb4e-817a-8530e4ed2ba2
Version Independent ID: e23b1bf7-210a-3772-41bb-03cd1ea41919
Content: Face API Service overview - Microsoft Cognitive Services
Content Source: articles/cognitive-services/Face/Overview.md
Service: cognitive-services
GitHub Login: @SteveMSFT
Microsoft Alias: sbowles

@PatrickPan2012 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@SteveMSFT Hi Steven, this is a doc-bug, can you please check it and also update the document as necessary? Thanks a lot!

XiXi & Sean from the Face team

Error in our Face API V1.0 doc.
From: YutongTie-MSFT<EMAIL_ADDRESS>Sent: Friday, September 7, 2018 7:41 AM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Steven Bowles<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Response of Face - Identify (#14641)
@PatrickPan2012https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FPatrickPan2012&data=02|01|sbowles%40microsoft.com|1088a94491aa4d619f1308d614cffd35|72f988bf86f141af91ab2d7cd011db47|1|0|636719280876891817&sdata=Q46PD1WAnsIP68v3ERPmJxG0LaCcVSeI4tjfkEnARGE%3D&reserved=0 Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@SteveMSFThttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSteveMSFT&data=02|01|sbowles%40microsoft.com|1088a94491aa4d619f1308d614cffd35|72f988bf86f141af91ab2d7cd011db47|1|0|636719280876891817&sdata=2CbBCZHnL7YnI%2BWYtI2IUrQJRNMBLgA2GMJfUe3Fsd0%3D&reserved=0 Hi Steven, this is a doc-bug, can you please check it and also update the document as necessary? Thanks a lot!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F14641%23issuecomment-419461001&data=02|01|sbowles%40microsoft.com|1088a94491aa4d619f1308d614cffd35|72f988bf86f141af91ab2d7cd011db47|1|0|636719280876901831&sdata=gxEnIju1HTxH2KkKckkNSEyIV%2Bm8QAvzxTvPCCERADk%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAjSdlQv9iqXG4x8Uz2jmtbFXGI0lwiN2ks5uYoWUgaJpZM4WeRxV&data=02|01|sbowles%40microsoft.com|1088a94491aa4d619f1308d614cffd35|72f988bf86f141af91ab2d7cd011db47|1|0|636719280876901831&sdata=dDoH9VLVKhazw6xwjbBckkrUIEQgaEvxbH9R%2FdsozKk%3D&reserved=0.

@PatrickPan2012 We have updated the API reference here: https://westus.dev.cognitive.microsoft.com/docs/services/563879b61984550e40cbbe8d/operations/563879b61984550f30395239. For other regions, the update is under deploying and may have some delay.
I think this is OK to be closed.

Thanks!


Add activity indicator category methods for UIButton

UIImageView+WebCache has support for displaying an activity indicator while loading an image, but this functionality is missing from UIButton+WebCache.

This is resolved by some refactoring on the 4.x branch that moved all the activity indicator code from UIImageView to UIView.


Request: completion for .net objects

Regular powershell completion works fine with the plugin (eg Write-H<tab>), but even namespace-cast .net objects don't provide completion for properties and methods. Consider:
[Void][Reflection.Assembly]::LoadWithPartialName("Microsoft.Web.Administration")
$serverManager = [Microsoft.Web.Administration.ServerManager]::OpenRemote("<IP_ADDRESS>")

at this point, I'd expect to be able to do $serverManager. to list properties and methods, like I can in pwsh, or via get-member

Or should this be working, and I have a problem locally?

For me, the completion works (even on an older plugin version that was available at the time, v2.0.10):

Could you please clarify if this is the right kind of completion?
And if it still doesn't work for you on a modern plugin version, could you please upload the IDE logs to uploads.jetbrains.com and share the upload id?
Thanks!

In the above image, there are a few occasions where I would have really appreciated some auto-completion - and you can see the "No suggestions" popup that comes when I try to; in particular:

[Void] completes, but there was no help for Reflection.Assembly
methods off of that assembly complete, but when I get back the $serverManager, no completion - this is the initial cause of the issue I raised: not knowing the ServerManager api well, I really wanted completion here to solve the issue at hand. Members on an object can be listed with Get-Member, so, whilst I'm not sure how difficult it would be to provide completions for other parts, it seems like a call to completion after a . on an object should be solvable with a call to Get-Member on the provided object. Though, please bear in mind, I don't know what kind of model is followed in language plugins for intellij, so this may seem really simple to me, and yet, really difficult to implement irl.

I'll leave a note to check this issue after updating PSES (#51).
After that, it'll either be closed, or I'll file a feature request to PSES.


Fixed issue where we weren't checking entry key on a search operation.

CODE REVIEW
Fixes a bug in the skip list code where we would return the next value on a get instead of a matching value.
@alexandercampbell-wf @beaulyddon-wf @rosshendrickson-wf @ericolson-wf @stevenosborne-wf @tylertreat-wf @tannermiller-wf

+1

+1

+1


Better feedback about why a given name is invalid

At the moment ts-engine only allows package names with whatever npm's valid package naming system is. ts-engine says:
<name> is not a valid npm package name
Some better feedback would be good to resolve this - perhaps giving guidelines on what the system is, linking to the system or simply giving common pitfalls like "only snake case, no camelCase or PascalCase"
Cheers!

This is merged into master now, expect it in the next release


Flaky Test: Kueue when Creating a Job With Queueing Should readmit preempted job with priorityClass into a separate flavor

What happened:
Flaky Test for End To End Suite: kindest/node:v1.29.4: [It] Kueue when Creating a Job With Queueing Should readmit preempted job with priorityClass into a separate flavor.
{Timed out after 5.001s.
The matcher passed to Eventually returned the following error:
    <*errors.errorString | 0xc0008604b0>:
    NotFoundError expects an error
    {
        s: "NotFoundError expects an error",
    } failed [FAILED] Timed out after 5.001s.
The matcher passed to Eventually returned the following error:
    <*errors.errorString | 0xc0008604b0>:
    NotFoundError expects an error
    {
        s: "NotFoundError expects an error",
    }
In [AfterEach] at: /home/prow/go/src/sigs.k8s.io/kueue/test/e2e/singlecluster/e2e_test.go:125 @ 07/16/24 09:35:23.309
}

What you expected to happen:
Errors have never been seen.
How to reproduce it (as minimally and precisely as possible):
https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/2619/pull-kueue-test-e2e-release-0-7-1-29/1813144006800969728
Anything else we need to know?:
Environment:

Kubernetes version (use kubectl version):
Kueue version (use git describe --tags --dirty --always):
Cloud provider or hardware configuration:
OS (e.g: cat /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/kind flake

/assign


Make the trap name for unreachable traps more descriptive.

Following up on WebAssembly/wasi-sdk#210, this makes the trap message
for unreachable traps more descriptive of what actually caused the
trap, so that it doesn't sound like maybe Wasmtime itself executed a
unreachable!() macro in Rust.
Before:
wasm trap: unreachable
wasm backtrace:
     [...]

After:
wasm trap: wasm `unreachable` instruction executed
wasm backtrace:
     [...]

This sort of makes me idly wonder if wasm should have 2 abort instructions. One is "it's a programmer error we got here, help me debug" (aka unreachable as-is today) and another is "please exit this thread of execution quickly" but presumably debug information was already printed. For example assert might use the latter instead of the former (unsure, but something like that). In any case not exactly something we'll be changing here...


NullPointerException

I was trying out the newest release but I noticed it was giving null exceptions.
This seems to completely prevent pages from loading.
The log says that it is a problem with your autocomplete code. The exception happens on line 93.
(I like to keep eyes on this because I like browser dev, and it happens to resemble my own attempts xD)
EDIT: It actually doesn't keep the page from loading, it seems, I just have to click the page. It's still an exception though, meaning that something is broken somewhere.

Hello @JasonTheKitten thanks for report.
Issue is fixed in last commit and update release Lobo Evolution 2.0


TimeText fadeAway not working with offset

Todo: Demo
Report internally that fadeAway with an initial offset, doesn't correctly scroll.
Should be
if (isInitial && centerItemScrollOffset > initialOffset) {
  val y = (centerItemScrollOffset - initialOffset) / LocalDensity.current.density

https://github.com/google/horologist/pull/429


Deleted not used paths and fix agent-port and Location for mesos-agent logs in dcos-config-windows.yaml

High-level description
Deleted not used paths and fix agent-port and Location for mesos-agent logs in dcos-config-windows.yaml
Corresponding DC/OS tickets (required)

DCOS-ID JIRA title / short description.

Related tickets (optional)

DCOS-ID JIRA title / short description.

Checklist for component/package updates:
If you are changing components or packages in DC/OS (e.g. you are bumping the sha or ref of anything underneath packages), then in addition to the above please also include:

[ ] Change log from the last version integrated (this should be a link to commits for easy verification and review): example
[ ] Included a test which will fail if code is reverted but test is not. If there is no test please explain.
[ ] Test Results: [link to CI job test results for component]
[ ] Code Coverage (if available): [link to code coverage report]

@mesosphere-mergebot changelog-not-required

@mesosphere-mergebot label Ready For Review

@mesosphere-mergebot request-review @jongiddy

@mesosphere-mergebot request-review @DominikDary

Re-running Windows build due to temporary borking of the colorama package: https://github.com/tartley/colorama/issues/240


[Request] Autoruns

https://docs.microsoft.com/en-us/sysinternals/downloads/autoruns

This utility, which has the most comprehensive knowledge of auto-starting locations of any startup monitor, shows you what programs are configured to run during system bootup or login, and when you start various built-in Windows applications like Internet Explorer, Explorer and media players. These programs and drivers include ones in your startup folder, Run, RunOnce, and other Registry keys. Autoruns reports Explorer shell extensions, toolbars, browser helper objects, Winlogon notifications, auto-start services, and much more. Autoruns goes way beyond other autostart utilities.

It's basically a .zip file with executables in it. I couldn't find a repo.

Autoruns and Autorunsc are available in the sysinternals package here.

The issue with that hack is that no one wants to download, extract and install the whole SysInternals suite just for one tool they're probably going to uninstall in a few minutes. That package goes against a third of scoop Scoop's goals.

Path Pollution from installing lots of programs
Unexpected side-effects from installing and uninstalling programs

It's okay if the bucket's maintainers don't want SysInternal's tools, the choice is theirs, but hey shouldn't justify it with something so invalid.

Another goal is to keep each bucket under 1,000 packages, due to GitHub's display limit, and to distribute the maintenance of apps & buckets across multiple maintainers.


Reduce the level of nesting in our docs homepage (evaluate)

When I visit our docs homepage, most sections have 1-2 levels of nesting.
For example, "About Citus" has at most 3 levels of nesting: What is Citus? -> When to use Citus? -> Real-time Applications and Multi-tenant Applications.
The primary exception to this is our References Section. This section has 4 levels of nesting and includes repetitive information on our main page. In particular, we have the following repetition in our docs homepage:

User Defined Functions Reference

Table and Shard DDL

create_distributed_table

Arguments
Return Value
Example

create_reference_table

Arguments
Return Value
Example

master_create_distributed_table

Arguments
Return Value
Example
... similar for all our public APIs

We probably don't need to repeat Arguments, Return Value, and Example headers for each public APIs in our docs homepage. We may therefore want to reduce the level of nesting in our Reference section.

I'll incorporate this feedback into the ongoing menu redesign.


Add skipped/included labels to -show-keys-info

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (146a223) 62.50% compared to head (6b53649) 63.19%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
+ Coverage   62.50%   63.19%   +0.69%
==========================================
  Files          13       13
  Lines        1528     1538      +10
==========================================
+ Hits          955      972      +17
+ Misses        421      414       -7
  Partials      152      152

Flag
Coverage Δ

unittests
63.19% <82.35%> (+0.69%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files
Coverage Δ

flatjsonl/processor.go
67.37% <82.35%> (+1.32%)
:arrow_up:

... and 2 files with indirect coverage changes

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


[prod]add monitoring script for terminating pod

merge this as https://github.com/openshift/openshift-tools/pull/4113 +1ed


fix: github_runner_matrix timeout

remove NEWEST_GITHUB_ACTIONS_MACOS_RUNNER as no one will remember to update it
Brew always supports the latest version of the macOS release.
no need to half the time on github runners
It is still possible for arm runners to hit the time limit. https://github.com/Homebrew/homebrew-core/actions/runs/8553455945

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

NEWEST_GITHUB_ACTIONS_MACOS_RUNNER is the latest x86_64 macOS runner image that GitHub provides, not the latest macOS release that brew supports. Currently that's still sonoma, as macos-14 is an arm64 image.

Yes, the latest macOS runner on Github is Sonoma(macos-14). However, the NEWEST_GITHUB_ACTIONS_MACOS_RUNNER was set to Ventura (macos-13) before this pull request.
I cannot see any reason to use a different time for the newest macOS release if it is hosted by Github.

Yes, the latest macOS runner on Github is Sonoma(macos-14). However, the NEWEST_GITHUB_ACTIONS_MACOS_RUNNER was set to Ventura (macos-13) before this pull request.

Sorry, there was a typo; edited: the latest x86_64 runner image is Ventura (macos-13), not Sonoma (macos-14). All Sonoma CI is hosted by Homebrew.

Sorry, there was a typo; edited: the latest x86_64 runner image is Ventura (macos-13), not Sonoma (macos-14). All Sonoma CI is hosted by Homebrew.

I see. macos-14 runner is available on Github now. Should we change NEWEST_GITHUB_ACTIONS_MACOS_RUNNER  to sonoma?

Should we change NEWEST_GITHUB_ACTIONS_MACOS_RUNNER  to sonoma?

Probably worth trying, yea. Side note: running arm64 Sonoma CI on GitHub-hosted runners would require some additional changes as the architecture of macos-14 runner is different.

I see. macos-14 runner is available on Github now. Should we change NEWEST_GITHUB_ACTIONS_MACOS_RUNNER  to sonoma?

This is done in #17042.


fix(sidenav): remove dependency on OverlayModule

Removes the dependency between the sidenav and the CDK overlay, because the overlay isn't being used for anything.

Rebased.


Feature/IFU-873 gdpr log

This should make it possible to get logs for people with permissions to view from /admin/reports/dblog when any of these pages has been accessed:

user page
user edit page
user listing

It is also saved in public/sites/default/files/logs/infofinland_gdpr.log

For testing in dev


Add operators to Lookup

includeKeys - A predicate on keys that should ALWAYS be cached
excludeKeys - A predicate on keys that should NEVER be cached
Combining lookup functions?

orElse (fallback)
race (first success)

Unary operators

onSuccess(v => ZIO(...))
onFailure(e => ZIO(...))

val lookup2 = lookup.includeKeys(List("SPECIAL_KEY1", "SPECIAL_KEY2") contains _).excludeKeys(_ == "SPECIAL_KEY3")

Another useful operator would be something that builds a chain of cache lookups, so you can first check an in-memory cache, then check a remote cache such as redis, and finally fallback to calculating the value.
A cache the is upstream could populate with the downstream values. This is very similar to the orElse operator, but fills upstream caches w/ downstream values.

Another useful operator would be something that builds a chain of cache lookups, so you can first check an in-memory cache, then check a remote cache such as redis, and finally fallback to calculating the value.
A cache the is upstream could populate with the downstream values. This is very similar to the orElse operator, but fills upstream caches w/ downstream values.


WebComment: comment submission deadline (CERN)

Displays the comment submission deadlinem, when present.
(CERN-specific)

Co-Authored-By: Nikolaos Kasioumis<EMAIL_ADDRESS>

@kasioumis Thanks for the update.  Is this on CDS already?  Are we maintaining "someday" v1.x deadline or promoting it sooner?

@tiborsimko Yes, this is already deployed on production. Let's maintain the v1.x deadline for now. The latest updates on comments need a couple of pending features that might take a couple of weeks. Once the picture is clearer I'll assign the next available milestone.


Ensure all domains are SSL enabled

Checked all following are SSL enabled, https only is turned on for all regions

https://azurespeed-eastasia.azurewebsites.net/
https://azurespeed-westus.azurewebsites.net/
https://azurespeed-trafficmanager.net/
https://www.azurespeed.com/


Newly created accounts all have -1 as accountNumber

This was probably introduced in cd01aec9d8097eb97d593800f95e35ee603580cb
To verify:

Set a breakpoint here https://github.com/k9mail/k-9/blob/1552c151a565f5ff44a46d2becd35c9033d6bb3b/app/core/src/main/java/com/fsck/k9/Preferences.java#L208
Create a new account

Thanks for doing testing like this, it's very helpful! :1st_place_medal:


Fix: exp type & overriding default exp value

Technically, jose supports passing timey strings, e.g. '1h', '10days' etc. (here)
This PR updates the type of exp to be string as well as a number for that.
Also, this fixes overriding exp value when calling jwt.sign() method.
Example:
const token = await jwt.sign({
  userId: 'qwbfo21b9d',
  exp: '1h', // <~ this now works fine. Previously this was ignored and threw a type error
})

I was so confused why my exp field was not being passed into my token; this would be a great QOL feature, and certainly would save others from hours of debugging.


Implementation of Queue

Kindly add this pr to your repository

Spam

Hi, @crweiner,  thank you!


Fixing Module CR deletion during node reboot

In case node it rebooted(for any reason) and the Module CR is being deleted at the same period of time, the following sequence happens: 1) node becomes NotReady
2) module-nmc controller removes Spec from NMC
3) nmc controller does not schedule any unloader pod, since the node is
not ready
4) node becomes ready
5) nmc controller does not schedule unloader pod, since after node
reboot it ignores scheduling unloader pod, since the kernel module is
not loaded anyway
6) no unloader pod running, meaning Status in the NMC is not deleted,
meaning Module CR cannot be finalized, since Status in NMC is present
Solution:
When nmc controller processes Modules without Spec, and the node has been rebooted, the nmc controller will delete the Status from NMC.

/assign @ybettan

/test operator-upgrade

/test e2e

/test e2e-hub

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/test operator-upgrade

/override operator-upgrade

/override ci/prow/operator-upgrade

/lgtm

operator-upgrade fails on access/bootup of internal registry, not related to the changes in nmc controller

/cherry-pick release-2.2


Stuck compiler path for Visual Studio 2017 after migrating to Visual Studio 2019

Brief Issue Summary
Trying to use Visual C++ 2019 compiler with multiple projects after migrating to Visual Studio Community 2019 from Visual Studio Community 2017, but cmake tools keeps using the path to the old Visual C++ 2017 compiler which was uninstalled with Visual Studio Community 2017.
Tried resolutions:

Un+reinstalling CMake Tools extension.
Un+updating to newest CMake version.
Deleting prior .vscode + build folders in projects.
Removed CMAKE_C(XX)_COMPILER from VSCode settings.json.
Removed the Visual C++ 2017 compilers from saved kits.
Adding compiler path to Windows system environment variables under CC + CXX.

What works:

Visual Studio Community 2019's CMake builds the projects fine.
Using the kit 'Clang 8.0.0 for MSVC with Visual Studio Community 2019 (amd64) works as usual.
CMakeTools 'scan for kits' finds the Visual Studio Community 2019 variants.
On my laptop where I never installed Visual Studio 2017; I've had no issues with the same setup using Visual Studio 2019.

I'm guessing it has something to do with the Ninja generator, as if I open CMake-GUI browse to the project and choose a different generator (Visual Studio) it will build using CMakeTools and the 2019 kit (This is not ideal since I wish to use Ninja, which I use for my Linux builds).
The Visual Studio 2017 path is apparently also cached in some settings somewhere other than my project directories, however I'm not sure where and how to clear this cache, which is why I'm opening this issue.
Expected:

CMake Tools finds the Visual C++ 2019 compiler.
Builds using the Visual C++ 2019 compiler with Ninja.

Apparent Behavior:

Selecting Visual Studio Community 2019 - amd64 kit for building.
Yields errors as shown below.

CMake Tools Log
[cmake] The C compiler identification is unknown
[cmake] The CXX compiler identification is unknown
[cmake] CMake Error at CMakeLists.txt:9 (project):
[cmake]   The CMAKE_C_COMPILER:
[cmake]
[cmake]     C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x64/cl.exe
[cmake]
[cmake]   is not a full path to an existing compiler tool.
[cmake]
[cmake]   Tell CMake where to find the compiler by setting either the environment
[cmake]   variable "CC" or the CMake cache entry CMAKE_C_COMPILER to the full path to
[cmake]   the compiler, or to the compiler name if it is in the PATH.
[cmake]
[cmake]
[cmake] CMake Error at CMakeLists.txt:9 (project):
[cmake]   The CMAKE_CXX_COMPILER:
[cmake]
[cmake]     C:/Program Files (x86)/Microsoft Visual Studio/2017/Community/VC/Tools/MSVC/14.16.27023/bin/Hostx86/x64/cl.exe
[cmake]
[cmake]   is not a full path to an existing compiler tool.
[cmake]
[cmake]   Tell CMake where to find the compiler by setting either the environment
[cmake]   variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
[cmake]   to the compiler, or to the compiler name if it is in the PATH.

Platform and Versions

Operating System:  Windows 10 -1903
CMake Version: 3.15.5
VSCode Version: 1.39.2
CMake Tools Extension Version: 1.1.3
Compiler/Toolchain: Visual C++ 2019

If you set your kit to the 2019 compiler and then run the "Delete cached build settings and reconfigure" command, does it still look for the 2017 compiler?

Well I started up VsCode and the problem is no longer persisting.
Unfortunately I didn't keep a detailed log on my attempts to fix, but I believe the last thing I tried was removing the CMAKE_C(XX)_COMPILER from settings.json. I hadn't restarted VsCode when I checked to see if that fix so, a restart might have been needed.
I had tried the "Delete cached build settings and reconfigure" command before, and it didn't resolve the issue. I'd tried all the cmake commands I thought where relevant.
Thanks for the assistance and sorry for the unnecessary issue creation.


implement review method for CSP user

for all services in state "APPROVAL_PENDING", the CSP can either approve or reject the service

[ ] create new service -> reviewServiceRegistrationRequest - /review/result/${ServiceReviewResult }
[ ] create enum -> ServiceReviewResult -> Accepted, Rejected
[ ] if accepted -> set service status to "Approved", otherwise to "REJECTED"

let me do it

@WangLiNaruto This issue is based on https://github.com/eclipse-xpanse/xpanse/issues/1378, which I have completed. Let me take this. @swaroopar

@swaroopar Is this new service only for those service templates in state APPROVAL_PENDING? Is it invalid for those templates in state Approved or Rejected?

@baixinsui yes you are right. For other states, the service must return a 400 with exception - ServiceTemplateAlreadyReviewed

@swaroopar The necessary parameter serviceTemplateId is lost in the new service with URI /review/result/{ServiceReviewResult}.
I have come up with two modification plans as follows,

Update the GET request URI with  /service_templates/{id}/review/result/{serviceReviewResult}
Use POST method with URI /service_temlates/review  and request body as follows:
{
"id":"serviceTemplateId", (required)
"serviceReviewResult": "value of ServiceReviewResul ",(required)
"reviewComment": "comment"(optional, maybe we can save this comment into the DB)
}
Please help me take a look and give some suggestions.

@baixinsui good point.
We can make the service a PUT instead of post.


Minimal read-only InfluxDB client using the REST API in Julia

Every ReferenceProfile in an ESDL file has to retrieved from InfluxDB. There's no Julia client, so we need to write one using the REST API (as recommended by the vendor).  The client should be minimal with only the features we need.  For now:

[ ] Read ReferenceProfile from InfluxDB

@suvayu Should this also move to TulipaIO?


feat(ui): show nested fields in groups as separate columns in table

Description
Group fields are shown as one column, this PR changes this so that the individual field is now shown separatly.
Before change:

After change:

[X] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[X] New feature (non-breaking change which adds functionality)

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[ ] Existing test suite passes locally with my changes
[ ] I have made corresponding changes to the documentation

@JarrodMFlesch Ref the comment you made in #7331, that this was not for now. I wanted to work more on it.
Please let me know what you think about the approach, if it's fine I will work further on checking any side effects and work on tests to ensure it will be covered and look at any failing test.
I also wanted to know what you think about the column header/label. In filters this has a > for nested fileds. Like this:

I think that should also apply to the column header. What do you think?

@andershermansen I think this is a very welcome improvement 😃

@JarrodMFlesch Great! I’m on leave for the rest of this week, will pick this up and implement the rest next week.

@JarrodMFlesch This is now ready for review. The lint error is in a file I have changed, but it's not my change. All other tests are green and I added test case for the new feature.

@JarrodMFlesch Did you (or anybody else in payload team) had a chance to take a look?
I merged in latest beta branch now, the lint error is gone as it was fixed in another merged PR. But live-preview test broke. I think it's a flaky test, passes fine locally.

Maybe I can interest you with some more screenshots ;)
From dev fields
Before:

After:

Maybe | instead of > would be a bit more clear in the columns select toolbar?

Maybe | instead of > would be a bit more clear in the columns select toolbar?

I think the > looks better and is more clear. Also consistent with how the filters is implemented.
Here is how it looks with the |

That's a great addition, thanks @andershermansen! I hope this gets reviewed and merged soon :)


[1275] Renames for post-desktop world

https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/1275
Reviewers
Functional:
@MDunitz
Readability:
@danieljhegeman
Changes
Re-organise backend module to reflect new world of hosted-only code:

backend/czi_hosted -> server
Moved tests to -> server/tests/
Merged the backend/common directories into server/common
Update imports, make commands etc. to reflect changes
Run black

Validation

Ran unit tests
Ran build commands
Ran some other make commands

Next steps

Remove make commands for desktop
update/simplify remaining make commands
remove desktop references from github actions
remove compatibility tests
remove desktop references from documentation

Removed refs to 'hosted' and equivalents (from docs/scripts, not from code yet)
Deleted docs directory

ok everything should be good now. I updated the fmt command to exclude that directory.
Made some minor changes to readme too, we'll likely want to make more changes later


1.16 support

Where can I get Waterdog for 1.16? When I try to connect, I get message "Old server"

Hi, we are unable to build 1.16 working jar on official Jenkins server for permission issues. You can use my personal build, here.
For feature questions, please visit our Discord server.


[BUG] : UI Bug

Bug Description 🐞
Some bugs on UI
Bug Location 🗺
Frontend
Steps to Reproduce 🔄
frontend
Expected Behavior ✔
responsiveness
Actual Behavior ❌
responsivenesss
Proposed Solution

[X] I will provide a pull request to fix this bug.
[X] I need help to fix this bug.

Adherence to Code of Conduct

[X] I adhere to the CONTRIBUTING GUIDELINES of this project.

Hey @shuvadeepmondal👋! Thanks for raising this issue and demonstrating your interest in contributing to our project!🚀
We value your engagement and eagerly anticipate your contributions. Should you wish to get involved, please use the /assignme command to self-assign the issue.
This issue provides an excellent opportunity for us to enhance our project, and we're excited to have you join us! Please take a moment to review our CONTRIBUTING GUIDELINES before getting started.
If you have any questions or need assistance, feel free to reach out to us on LinkedIn.❤

/assignme


Is this official ripple api?

Is this official ripple api?
and What can i do when i use this??

True


ItsyCal to 0.10.9

Thank you for the contribution. It was merged directly as 18eadc0e645480d793f52109113df72479f8ce02 to keep commit history cleaner. Your contribution is still credited to you.


Please push latest version (0.0.8 or later)

The marketplace version is 0.0.8, but the repository version looks like 0.0.7.
Would you please push the latest version?

The market place version is still newer than this repo.... Would be good to sync.

@ko1 Do you have a newer version on your box?

Sorry I uploaded.


Effects support

Is it possible to add support for WLED effects? Let's say in the form of changing the HDMI outputs of the TV?

Yeah more effects are added soon.
The plugin is currently only in "Beta", so it isn't completed and bug free yet.
Which app do you currently use to change the effects (On your screenshot)

Yeah more effects are added soon.
The plugin is currently only in "Beta", so it isn't completed and bug free yet.
Which app do you currently use to change the effects (On your screenshot)

If there are effects, this plugin will have no equal! Screenshot from this article
https://sprut.ai/client/article/2760

Yeah more effects are added soon.
The plugin is currently only in "Beta", so it isn't completed and bug free yet.
Which app do you currently use to change the effects (On your screenshot)

If there are effects, this plugin will have no equal! Screenshot from this article
https://sprut.ai/client/article/2760

Oh ok, this is already a HomeKit Plugin.
So what do you want the plugin to do that it should be able to do?

Yeah more effects are added soon.
The plugin is currently only in "Beta", so it isn't completed and bug free yet.
Which app do you currently use to change the effects (On your screenshot)

If there are effects, this plugin will have no equal! Screenshot from this article
https://sprut.ai/client/article/2760

Oh ok, this is already a HomeKit Plugin.
So what do you want the plugin to do that it should be able to do?

What is in the video from the link I gave is achieved using NodeRed. I would like the homebridge plugin to also be able to change effects. Is it possible?

Yes i can work on this. But it will take some time.
Before adding new effects, i have to fix some other bugs.

Yes i can work on this. But it will take some time.
Before adding new effects, i have to fix some other bugs.

This is great news! I'm looking forward to it! I wish you success!

thanks for your work on this - wled has been missing from homebridge - the current effect implementation is cool, with individual switches - that way we can integrate it with security system or motion sensors.

How about this for an idea instead of having a full list of all available effects, a comma separated list of your favourite effects?

Any chance of multiple switches for multiple effects - great update btw

Any chance of multiple switches for multiple effects - great update btw

I still have to see how and if this is possible at Homebridge. But I'll take a closer look at it this week.
Btw. thanks for your donation: D

I still have to see how and if this is possible at Homebridge. But I'll take a closer look at it this week.
Btw. thanks for your donation: D

I published a new version...
Now you can configure a own list with effects and use it like on the screenshot above.
!! INFO !!
The Plugin is now a Platform-Plugin and has to be moved to the "platforms"-section in the config.json. (See the examples)
If you have any questions, feel free to ask....

I published a new version...
Now you can configure a own list with effects and use it like on the screenshot above.
!! INFO !!
The Plugin is now a Platform-Plugin and has to be moved to the "platforms"-section in the config.json. (See the examples)
If you have any questions, feel free to ask....

So, when we add the the device through + > add accessory > "I dont have a code or cannot scan" > then add the tile .. It requests an 8 figure setup code, correct?

So, when we add the the device through + > add accessory > "I dont have a code or cannot scan" > then add the tile .. It requests an 8 figure setup code, correct?

I published a new version...
Now you can configure a own list with effects and use it like on the screenshot above.
!! INFO !!
The Plugin is now a Platform-Plugin and has to be moved to the "platforms"-section in the config.json. (See the examples)
If you have any questions, feel free to ask....

You're just super cool! Your work is done perfectly! Be happy!

I published a new version...
Now you can configure a own list with effects and use it like on the screenshot above.
!! INFO !!
The Plugin is now a Platform-Plugin and has to be moved to the "platforms"-section in the config.json. (See the examples)
If you have any questions, feel free to ask....

You're just super cool! Your work is done perfectly! Be happy!

Better than Home Assistant's version

Better than Home Assistant's version

Any particular reason you have the effects statically defined in getAllEffects()? Perhaps just grab them from the /json/effects endpoint...? I run the WLED audio reactive fork, and they have their own added effects.

Any particular reason you have the effects statically defined in getAllEffects()? Perhaps just grab them from the /json/effects endpoint...? I run the WLED audio reactive fork, and they have their own added effects.

@RomanGz Oh no... It was just provisional. (because i didn't wanted to request the effects everytime.. at the beginning)
But i will add the "feature" to grab them from the JSON-API and cache them, to not use unnecessary bandwidth.

@RomanGz Oh no... It was just provisional. (because i didn't wanted to request the effects everytime.. at the beginning)
But i will add the "feature" to grab them from the JSON-API and cache them, to not use unnecessary bandwidth.

Ok I tried to fetch the effects dynamically, but it is very difficult due the fact I can only fetch it asynchronously...
So I have to find a perfect solution for this, but for now the static defined effects will do it...

Ok I tried to fetch the effects dynamically, but it is very difficult due the fact I can only fetch it asynchronously...
So I have to find a perfect solution for this, but for now the static defined effects will do it...

Also I changed the config.json layout with the new version 1.1.1
Now you can add multiple WLED instances with the new dynamically platform plugin.

Also I changed the config.json layout with the new version 1.1.1
Now you can add multiple WLED instances with the new dynamically platform plugin.

Ok I tried to fetch the effects dynamically, but it is very difficult due the fact I can only fetch it asynchronously...
So I have to find a perfect solution for this, but for now the static defined effects will do it...

Haha. I tried myself after I asked and had the same problem with async. Wasted a whole day on it. Tried with axios & request. I may revisit it soon. Thanks for trying.

Ok I tried to fetch the effects dynamically, but it is very difficult due the fact I can only fetch it asynchronously...
So I have to find a perfect solution for this, but for now the static defined effects will do it...

Haha. I tried myself after I asked and had the same problem with async. Wasted a whole day on it. Tried with axios & request. I may revisit it soon. Thanks for trying.

Also I changed the config.json layout with the new version 1.1.1
Now you can add multiple WLED instances with the new dynamically platform plugin.

I am very interested, is it possible to add several separate tapes with one accessory?😅🙏🏻

@sergoo-lbt So you mean you can control multiple WLED-Strips with one HomeKit accessory?

So you mean you can control multiple WLED-Strips with one HomeKit accessory?

I ask, is it possible? I currently manage three wleds with a single homekit accessory using native wled synchronization. But this does not work very reliably. I would like to be able to group them into one honekit accessory, but this is not possible since they contain a piece of the TV))

Ok, yes i could add an option, to configure the host/s as a list and then you can simple control your devices with one HomeKit accessory.

Ok, yes i could add an option, to configure the host/s as a list and then you can simple control your devices with one HomeKit accessory.

This is incredible! I will look forward to it!🙏🏻🔥

@sergoo-lbt  In the new version (1.2.1) I added the option to control multiple WLED-hosts with one accessory.
For further infos check the README.md

@sergoo-lbt In the new version (1.2.1) I added the option to control multiple WLED-hosts with one accessory.
For further infos check the README.md

I updated the plugin. You've done an incredible job! Right now, one accessory controls my three wleds, but there is a small problem with the accessory status. When I turn off the backlight with the brightness slider, after a few seconds, the accessory in homekit turns on again, while the LEDs themselves remain off!

In order not to get lost, a list of all effects. Choose who needs what.
"effects": [
"Solid",
"Blink",
"Breathe",
"Wipe",
"Wipe Random",
"Random Colors",
"Sweep",
"Dynamic",
"Colorloop",
"Rainbow",
"Scan",
"Dual Scan",
"Fade",
"Theater",
"Theater Rainbow",
"Running",
"Saw",
"Twinkle",
"Dissolve",
"Dissolve Rnd",
"Sparkle",
"Sparkle Dark",
"Sparkle+",
"Strobe",
"Strobe Rainbow",
"Strobe Mega",
"Blink Rainbow",
"Android",
"Chase",
"Chase Random",
"Chase Rainbow",
"Chase Flash",
"Chase Flash Rnd",
"Rainbow Runner",
"Colorful",
"Traffic Light",
"Sweep Random",
"Running 2",
"Aurora",
"Stream",
"Scanner",
"Lighthouse",
"Fireworks",
"Rain",
"Merry Christmas",
"Fire Flicker",
"Gradient",
"Loading",
"Police",
"Police All",
"Two Dots",
"Two Areas",
"Circus",
"Halloween",
"Tri Chase",
"Tri Wipe",
"Tri Fade",
"Lightning",
"ICU",
"Multi Comet",
"Scanner Dual",
"Stream 2",
"Oscillate",
"Pride 2015",
"Juggle",
"Palette",
"Fire 2012",
"Colorwaves",
"BPM",
"Fill Noise",
"Noise 1",
"Noise 2",
"Noise 3",
"Noise 4",
"Colortwinkles",
"Lake",
"Meteor",
"Meteor Smooth",
"Railway",
"Ripple",
"Twinklefox",
"Twinklecat",
"Halloween Eyes",
"Solid Pattern",
"Solid Pattern Tri",
"Spots",
"Spots Fade",
"Glitter",
"Candle",
"Fireworks Starburst",
"Fireworks 1D",
"Bouncing Balls",
"Sinelon",
"Sinelon Dual",
"Sinelon Rainbow",
"Popcorn",
"Drip",
"Plasma",
"Percent",
"Ripple Rainbow",
"Heartbeat",
"Pacifica",
"Candle Multi",
"Solid Glitter",
"Sunrise",
"Phased",
"TwinkleUp",
"Noise Pal",
"Sine",
"Phased Noise",
"Flow",
"Chunchun",
"Dancing Shadows",
"Washing machine",
"Candy Cane",
"Blends",
"TV Simulator",
"Dynamic Smooth"
]

@sergoo-lbt I released a new version, where this problem should be fixed.


Ideas welcome!

As part of Hacktoberfest, this repository has been growing at a steady rate. If you have additional ideas of what we should add, feel free to open an issue and make a suggestion! There is a template for writing a feature suggestion, and there are plenty of issues already prepared you can read for inspiration.
Suggestions could include e. g.:

Additional badges
Additional README sections
Additional templates for other markdown files

Closing for now as Hacktoberfest has ended. Obviously, ideas are still welcome ;)

Greetings @dbartholomae, I am back for this year's Hacktoberfest! Just wondering if you have any ideas on mind to further improve this project? I would like to make some contributions to this project.

I've reopened a couple of issues that still exist from last year. They are a bit more complicated than the average issue from last year, though.


Feature/system testing

PR Checklist
Please check if your PR fulfills the following requirements:

[x] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?
This PR introduces integration and unit testing which did not exist previously 🥇

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[] Other... Please describe:

What is the current behavior?

Issue Number: #2
What is the new behavior?
There is now a test sutie for the application
Does this PR introduce a breaking change?
[ ] Yes
[x] No

Other information

Changes Unknown when pulling a2d90d1b7787078ff9325627979701e8b72fd170 on feature/system-testing into ** on master**.

Changes Unknown when pulling f914a3d1b2ddb48c1d7d4098ba335be2fe78e6e4 on feature/system-testing into ** on master**.

Changes Unknown when pulling d26aa11253ec2a472173a9db27b785c4edf8175a on feature/system-testing into ** on master**.

Changes Unknown when pulling d26aa11253ec2a472173a9db27b785c4edf8175a on feature/system-testing into ** on master**.

Approved.

Changes Unknown when pulling 14563b97de3fe755cfbc1a7e426dcbb2f29d024a on feature/system-testing into ** on master**.


[Keyvault] Fix#22457: az keyvault key decrypt&az keyvault key encrypt: Fix output vault without b'' wrap

Related command

fix #22457
Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] The PR title and description has followed the guideline in Submitting Pull Requests.

[ ] I adhere to the Command Guidelines.

[ ] I adhere to the Error Handling Guidelines.

Keyvault


Event async

mpm_event changes/improvements for async, debugging and pass pytests (see each commit's message).

close/reopen to restart ci

Now in #294


Cool!

This is only praise, this converter works really fine!
Thx for good work, save my time before handmade

Thanks, good to hear!


SyntaxError: Unexpected token 'export'

Thank you for creating this package!
I ran into this error whenever I refresh/re-render the webpage:

The error leads to this:

My code:

I'm using Next js & the latest tts-react.

It works again when I hide/unhide my TextToSpeech code, but throws the same error when I refresh the page.
Thank you!

Thanks for pointing this issue out.
You are looking for a CJS build which is currently not available, but should be. I'll have to publish a new version of tts-react that includes a CommonJS build.
In the meantime, you can try using transpileModules from Next.js. and add tts-react. Basically you need to convert the import/export's from this package into require/module.exports during your app's build.

Ok! thank you so much!

Just fyi,
I followed the direction to implementing tts-react via Next.js' Module Transpilation.

It produced an error of "window is undefined"

I then used useEffect and useState to implement the package. It works now. (perhaps there's a better way)

Since tts-react uses the browser's Web Speech API, you need the standard window checks for Next.js before rendering TextToSpeech or calling useTts. It is not possible to render on the server since the necessary API is not implemented in Node as far as I know.
Your approach seems standard for Next.js apps. You could also load this package dynamically.

Ok, thank you!


Increase spacing of cards on community landing page

What it says on the tin.
Also clears up the markup on the landing page by removing unnecessary grid html and html where we don't need it because it's handled by the markup.
Visual changes

Before
After

Notes
I had some trouble where our use of caller macros rub up badly against (what I assume is) marked's processing. It thinks that a caller macro's generated markup is part of the preceding element, in this case a markdown paragraph. It then tries to merge the elements, resulting in some unhinged generated markup.
Probably we shouldn't be using callers inside a markdown file and should find an alternative. Maybe just a set of partials where the markup for the grids live.

@domoscargin Good spot! Snuck it in.


Change aiohttp.server default parameters

ServerHttpProtocol is constructed with keep_alive=None and timeout=15.
From my understanding that means no keep-alive connections and 15 sec timeout for web handlers.
The both defaults are wrong: I'd like to have keep-alives enabled by default and no cancelling for slow requests: websocket technically is implemented as very slow web-handler.
Thoughts?

Slow request is different, it is not related to handler, it is just for
parsing status line and headers
On Wednesday, January 14, 2015, Andrew Svetlov<EMAIL_ADDRESS>wrote:

ServerHttpProtocol is constructed with keep_alive=None and timeout=15.
From my understanding that means no keep-alive connections and 15 sec
timeout for web handlers.
The both defaults are wrong: I'd like to have keep-alives enabled by
default and no cancelling for slow requests: websocket technically is
implemented as very slow web-handler.
Thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/KeepSafe/aiohttp/issues/241.

+1 for keepalive, -1 for timeout
On Wednesday, January 14, 2015, Nikolay Kim<EMAIL_ADDRESS>wrote:

Slow request is different, it is not related to handler, it is just for
parsing status line and headers
On Wednesday, January 14, 2015, Andrew Svetlov<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

ServerHttpProtocol is constructed with keep_alive=None and timeout=15.
From my understanding that means no keep-alive connections and 15 sec
timeout for web handlers.
The both defaults are wrong: I'd like to have keep-alives enabled by
default and no cancelling for slow requests: websocket technically is
implemented as very slow web-handler.
Thoughts?
—
Reply to this email directly or view it on GitHub
https://github.com/KeepSafe/aiohttp/issues/241.

Done in da972ad


[Feature Request] Use integrated window for password resets

It would be good if XCreds could spawn a password reset window (rather than opening in the default browser), and grab the new password from the fields, as it does with signing in, and instantly reset the password, without the need to refresh.
I know this might not work with every auth providers, but it should be simple enough to do with entra/azure/microsoft password resets

Hmm. Not sure there is a common way to do this. can you explain a bit more about what "the need to refresh" is?

we have seen issues with the general user, who resets their password via the menu item, thinking it also changes the local password automatically, causing them to instantly forget their old password, and getting locked out of their mac. If there were a way to instantly change the local and online passwords in 1 go, it would save a lot of time.

can you explain a bit more about what "the need to refresh" is?
refresh > Sign In forcing XCreds to grab the new password, rather than waiting for next check

is there a standard password reset window? Or that you could put in a URL that has the password reset type fields and that xcreds could get the password from that? XCreds already has the functionality if the password is marked as requiring to change at the next login. Can you give a sample URL?


genpy: 0.5.5-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository genpy to 0.5.5-0:

upstream repository: git@github.com:ros/genpy.git
release repository: https://github.com/ros-gbp/genpy-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.20
previous version for package: 0.5.4-0

genpy
* fix handling of dynamic message classes with the same name (#37 <https://github.com/ros/genpy/issues/37>)
* fix Duration.abs() when sec is zero (#35 <https://github.com/ros/genpy/issues/35>)

Waiting for other releases before merging...


403 Forbidden

I try to open it in the browser on the result thumbnail, I can't access it

{
  coder: 'Loli Killers',
  result: [
    {
      name: 'Monkey D., Luffy',
      alias_name: '(Mugiwara, Straw Hat, Lucy, Luffytaro)',
      url: 'https://myanimelist.net/character/40/Luffy_Monkey_D',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/9/310307.jpg?s=3a27ab33bee665febfba970f24f203ba',
      anime: 'One Piece: Glorious Island',
      manga: 'One Piece: Episode of Merry - Mou Hitori no Nakama no Monogatari'
    },
    {
      name: 'Yamamoto, Luffy',
      alias_name: '',
      url: 'https://myanimelist.net/character/207833/Luffy_Yamamoto',
      thumbnail: 'https://cdn.myanimelist.net/images/questionmark_23.gif',
      anime: '',
      manga: ''
    },
    {
      name: 'Shishido, Jouichirou',
      alias_name: '(Fluffy)',
      url: 'https://myanimelist.net/character/208064/Jouichirou_Shishido',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/13/469123.jpg?s=ae43b5040d133c9d7b5d01cbd3f59cec',
      anime: '',
      manga: ''
    },
    {
      name: 'Piwi',
      alias_name: '(The Green Bird of Happiness, Fluffy)',
      url: 'https://myanimelist.net/character/155084/Piwi',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/3/441386.jpg?s=d69f5a01aa1cca37680ada6848f64240',
      anime: 'Helck',
      manga: ''
    },
    {
      name: 'Wada, Akinori',
      alias_name: '(Sweet Pretty Lonely Heart Fluffybro)',
      url: 'https://myanimelist.net/character/152561/Akinori_Wada',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/11/333941.jpg?s=ef6ccd0fbeac8670adecf9c50a16614a',
      anime: '',
      manga: ''
    },
    {
      name: 'Koyama, Nami',
      alias_name: '',
      url: 'https://myanimelist.net/character/207834/Nami_Koyama',
      thumbnail: 'https://cdn.myanimelist.net/images/questionmark_23.gif',
      anime: '',
      manga: ''
    },
    {
      name: 'Helmeppo',
      alias_name: '',
      url: 'https://myanimelist.net/character/12362/Helmeppo',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/14/48661.jpg?s=ea8c21d6e7b875553a5b547f7d231a5d',
      anime: 'One Piece: Yume no Soccer Ou!',
      manga: 'One Piece'
    },
    {
      name: 'Makino',
      alias_name: '',
      url: 'https://myanimelist.net/character/21210/Makino',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/6/49465.jpg?s=359c7644512a10d7cefef3f03bda327f',
      anime: 'One Piece: Episode of East Blue - Luffy to 4-nin no Nakama no Daibouken',
      manga: 'One Piece: Episode of Luffy - Hand Island no Bouken'
    },
    {
      name: 'Gina',
      alias_name: '',
      url: 'https://myanimelist.net/character/22414/Gina',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/6/53322.jpg?s=21e974e1d4a04aaac7a92ae4984e8376',
      anime: 'One Piece',
      manga: ''
    },
    {
      name: 'Daigin',
      alias_name: '',
      url: 'https://myanimelist.net/character/161873/Daigin',
      thumbnail: 'https://cdn.myanimelist.net/r/42x62/images/characters/13/369186.jpg?s=ac234c957c96c9aa52c6464acca318c0',
      anime: 'One Piece',
      manga: ''
    }
  ],
  warning: 'It is strictly forbidden to repackage this web API into another web API, copyright © 2022 by Loli Killers'
}

with testing
var search = 'Luffy'

lol.anime.mal_search_character(search)
  .then(res => {
    console.log(res)
  })
  .catch(err => {
    console.log(err)
  })

it works fine for me, maybe your ip address can't access the link/site


add callback for transferred sockets

I was in need of a fix for #385 myself, and implemented this PR as simple solution

I made a similar solution in the end as well. It really is needed for multithreading to be interesting in a real world scenario.

I personally prefer a callback more like the one in send (not tied to the group) but hey this is needed so let's add it

Is it possible to use this from NodeJS as well?


Could not create modpack directory

Hello,
when i start the server with this command:
./factorio-server-manager --dir /opt/factorio-community/.factorio
it shows this error:
2017/01/15 19:00:46 Could not create modpacks directory: mkdir /opt/factorio-community/.factorio/modpacks: no such file or directory
2017/01/15 19:00:46 error opening log file: open /opt/factorio-community/.factorio/factorio-server-manager.log: no such file or directory
Can you please help me?

The user has no read/write permission to the folder?!

Does the /opt/factorio-community/.factorio directory exist and have read and write permissions for the Factorio user?

Thanks i forgot to give the user the permissions

Glad to hear it, cheers


waffle tips + waffle stroke + waffle unit: auto

[x] unit: auto (closes #2214)
[x] #2132 (merge commit, closes https://github.com/observablehq/plot/issues/2129)
[x] #2204 (merge commit closes https://github.com/observablehq/plot/issues/2186)

I’d prefer to understand these three contributions separately rather than reviewing it all in this PR. I’m going to continue the reviews of the tip & stroke PRs before looking at this.


Cake5 - fixed CI pipeline & php8.2 cleanup

This PR builds upon the PR into the muffin webservice repository by MolbioUnige. It solves most of the warnings associated with updating the php version to 8.2, which is in line with what cakephp 5 recommends. It also updates the github pipeline to test php versions 8.1, 8.2 and 8.3, as this is what cakephp 5 supports. There is no use in testing 7.X anymore. Hopefully this PR can give the final push to update muffin webservice to support cakephp 5.
Some typing is made a bit stricter, where the code allows for it.

This is by no means the nicest solution and definitely not my best code. This is meant as more of a fix to get cake 5 to work and not much more. Additional work might be required to actually update the API to cake 5 standards. For an actual non-pre release, a bit of additional work needs to be done.

Both our PR's will be merged as one into the Muffin Webservice project: https://github.com/UseMuffin/Webservice/pull/112


Ensure terms create methods that play nice with ruby

Some terms in the Identities vocab end up with string names:
https://github.com/ruby-rdf/rdf-vocab/blob/develop/lib/rdf/vocab/identifiers.rb#L174-L176
Expected Behavior
I'd hope for a nicer term like:
::RDF::Vocab::Identifiers.issue_number
Actual Behavior
To use this I need to send or call it:
::RDF::Vocab::Identifiers.send(:"issue-number")
::RDF::Vocab::Identifiers.method("issue-number").call

Sorry for the delay in responding.
I'm afraid that the ability to use the term directly, such as RDF::Vocab::DC.created is a convenience, because the term is compatible with Ruby procedure names. For other terms, such as issue-number it is incompatible. Creating some other lookup mechanism such as issue_number is not something anticipated, and the send or call is the best way to handle things where there may be such an issue. I'd consider a PR on RDF.rb RDF::Vocabulary that might support something like this, but it gets into many different aspects.


[terra-framework] Removed inline style for dev-site components Part 6

Summary
Removed all inline styles and replaced the same styles with external CSS for the following packages.

HookShot
Navigation Layout

Partially resolves #657

I will be creating a new PR due to some build issues.


Loading list from appsettings.xml fails

From @sandersaares on February 1, 2018 10:4
I haven an appsettings.xml file with the following structure:
<config>
    <aaa>value1</aaa>
    <aaa>value2</aaa>
</config>

I wish to load this using the XML configuration loading pattern documented at https://docs.microsoft.com/en-us/aspnet/core/fundamentals/configuration/?tabs=basicconfiguration
Expected behavior: a list is loaded, accessible as aaa:0 and aaa:1
Actual behavior: System.FormatException occurred, Message=A duplicate key 'aaa' was found.
Copied from original issue: aspnet/Home#2842

@sandersaares you will need to provide some sort of indices to your array for it to be populated in the Configuration.
Example:
<config>
  <aaa name="s">
    value1
  </aaa>
  <aaa name="t">
    value2
  </aaa>
</config>

you can then call it by Configuration["aaa:s"] . Replace s,t with 0 based index and call Configuraiton["aaa:0"]
Here (https://github.com/aspnet/Configuration/issues/115#issuecomment-94975165) is the only place I could find a relevant spec for this.

More discussion on this issue can be found here: https://github.com/aspnet/Configuration/issues/745

Duplicate of #745


[WIP] Logging

This PR should address #82 by adding logging capabilities to CUPiD instead of using print statements. This should accomplish the following three tasks:

[x] update print statements to logger statements
[x] add log file to .gitignore
[ ] move ploomber output to log file instead of printed to console

It is useful to implement logging instead of using print statements because log files can be retrieved in the future (whereas print statements are not saved) and this reduces the amount of printing to the console that can get overwhelming for users.
All Submissions:

[x] Have you followed the guidelines in our Contributor's Guide (including the pre-commit check)?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

New Feature Submissions:

[ ] Have you lint your code locally prior to submission?

Changes to Core Features:

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you successfully tested your changes locally?

This page provides some examples of basic pipelines that can be used to set up logging for ploomber; however, this calls dag in a slightly different manner and will require a bit more looking into.

Given that within the CESM workflow, log files such as cupid.o#### and cupid.e#### where #### is a timestamp will be automatically generated, we can remove the filename from the logger instantiation.

[ ] It would be useful to set up logging levels in the configuration file so that users can choose what amount of output they want printed to the console.

I'd like to test this a bit more thoroughly to make sure it didn't break cupid-run or timeseries generation. Thus, the testing might be on hold until HPC systems are back up, but this should be ready for testing/review next week.

Thanks for this review, @mnlevy1981 !
I addressed those comments, except for the one print statement in cupid/util.py-- the get_control_dict function is needed prior to logging implementation (because logging uses info from the config file). Thus, I think this one would have circular implementation if both functions call one another.