assert isinstance(polygons[0][0], (list, tuple)), str( IndexError: list index out of range。

❓ Questions and Help
loading annotations into memory...
Done (t=0.17s)
creating index...
index created!
2019-04-29 18:33:58,770 maskrcnn_benchmark.trainer INFO: Start training
Traceback (most recent call last):
File "tools/train_net.py", line 199, in
main()
File "tools/train_net.py", line 192, in main
model = train(cfg, args.local_rank, args.distributed)
File "tools/train_net.py", line 93, in train
arguments,
File "/home/ubuntu/ccf-traffic-sign-identification/maskrcnn-benchmark/maskrcnn_benchmark/engine/trainer.py", line 57, in do_train
for iteration, (images, targets, _) in enumerate(data_loader, start_iter):
File "/opt/anaconda3/envs/maskrcnn_benchmark/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 551, in next
return self._process_next_batch(batch)
File "/opt/anaconda3/envs/maskrcnn_benchmark/lib/python3.6/site-packages/torch/utils/data/dataloader.py", line 577, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
IndexError: Traceback (most recent call last):
File "/opt/anaconda3/envs/maskrcnn_benchmark/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/opt/anaconda3/envs/maskrcnn_benchmark/lib/python3.6/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/ubuntu/ccf-traffic-sign-identification/maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/coco.py", line 83, in getitem
masks = SegmentationMask(masks, img.size, mode='poly')
File "/home/ubuntu/ccf-traffic-sign-identification/maskrcnn-benchmark/maskrcnn_benchmark/structures/segmentation_mask.py", line 467, in init
self.instances = PolygonList(instances, size)
File "/home/ubuntu/ccf-traffic-sign-identification/maskrcnn-benchmark/maskrcnn_benchmark/structures/segmentation_mask.py", line 344, in init
assert isinstance(polygons[0][0], (list, tuple)), str(
IndexError: list index out of range

Hello, I have the same problem with you, after reinstalling maskrcnn-benchmark。

I guess you are not using the original coco annotation, right?
Can we you check if all your polygons annotations are valid?
i.e. are all inside the image and the minimal length is greater than 6?

I also meet the same problem, have you ever solved the problem?

I also have this problem, and all my segmentation annotations are [[]] because I only want to do detection and I used faster-rcnn.yaml, I thought it would not use segmentation annotations but it seemed to be wrong.

My work around is to comment these 3 lines of function __getitem__ in maskrcnn-benchmark/maskrcnn_benchmark/data/datasets/coco.py
       masks = [obj["segmentation"] for obj in anno]         masks = SegmentationMask(masks, img.size, mode='poly')         target.add_field("masks", masks)

Same issue here any workaround
/maskrcnn-benchmark/maskrcnn_benchmark/structures/segmentation_mask.py", line 376, in init
type(polygons[0][0])
AssertionError: <class 'int'>


add contract names for contract context

Brief of your idea
reserved contract needs contract_names field.
What could be better if XuperChain using your idea
Describe the benefits of your idea
Are there any side effects?
Describe what might be worse.
Additional information
If possible, include a minimal, complete, and verifiable example to help us identify the issue.

Use template to support this.


Welsh (Cymraeg, cy-GB) localisation

Describe the Feature
It would be great to have Bluesky available in Welsh.
I can do (most of) the translation work, but am posting this first and will wait a bit in case someone's already started on it somewhere.
Attachments
No response
Describe Alternatives
No response
Additional Context
No response

I gave it a try and confirmed that the Internationalization Framework used by Bluesky supports cy-GB. As far as I know, no one is currently working on Welsh localization, so this could be a great starting point.
Apologies if this is a small or unnecessary question coming from a non-native speaker, but is it possible to directly use cy instead of cy-GB?

Thanks. As it's OK technically, there's no reason why we can't just use cy.

A Welsh localisation would certainly be great to have! Like @auroursa, I'm not aware that anyone is currently working on it.

Thanks. As it's OK technically, there's no reason why we can't just use cy.

In my opinion using cy would be preferable, as my understanding is that the country code is usually only added to the name of a locale to distinguish between different national varieties of a language.
Apologies if you're already aware of this, but I have one quick tip: there's no need to translate any of the strings where the line starts with #~. These are obsolete strings, they no longer appear in the app but they haven't been removed from the message catalog yet.

Great. I understand Bluesky uses Lingui "to manage translations". I'm not familiar with Lingui, but it's apparently the same company as Crowdin, which I am familiar with. Does translation take place on one of those platforms or do we just work on the messages.po file here from Github?

Translators have asked several times if localisation could be managed using a platform like Crowdin:

#2940
#5435
https://github.com/bluesky-social/atproto/discussions/2832

Crowdin's CEO even left a comment saying that they'd be happy to offer support to Bluesky in setting up on the platform. Unfortunately, the devs haven't taken up that offer yet.
In the meantime, some translation teams have taken the initiative and have set up their own projects on Crowdin. There may be others but the ones I'm aware of are Indonesian, Ukrainian and Chinese & Cantonese.
Sorry for the long answer, the short answer is that you can choose whichever way works best for you and other Welsh-speaking collaborators: either setting up a project on Crowdin or you can simply open a PR on GitHub and then respond to review comments here. This is what I do when I'm updating the French localisation.
If you do work directly from the messages.po file from GitHub, you might want to wait until the 1.94 release is cut and the file is updated, as there have been a huge number of string changes since 1.93.

I looked around earlier and as someone else has already said, we are translators when we localise and we don't necessarily have the same coding/tech/developer skills shared by much of the Github community, so those localisation platforms exist for many good reasons.
Thanks for the tip to wait until 1.94. That suits me. Any idea when?

I looked around earlier and as someone else has already said, we are translators when we localise and we don't necessarily have the same coding/tech/developer skills shared by much of the Github community, so those localisation platforms exist for many good reasons.

Very good point 💯

Thanks for the tip to wait until 1.94. That suits me. Any idea when?

One of the devs said "soon", it might be as early as tonight or it could be a little while longer. I would certainly think sometime next week at the latest, but then I've been wrong before 😅

There’s another way to get the latest translation files, but it requires some programming knowledge.
As I mentioned in #6244, you can refer to the recently merged Thai localization (#5879) to add the necessary extra files manually. Once that’s done, set up a local build environment and run intl:extract --locale cy --clean to generate the latest Welsh messages.po file. The --clean flag will automatically discard obsolete translation strings, leaving only the ones that need to be translated.
After that, you can import the file into your preferred localization platform, like Crowdin, use another translation tool, or even just a text editor—whichever works best for you.

In my opinion using cy would be preferable, as my understanding is that the country code is usually only added to the name of a locale to distinguish between different national varieties of a language.

Similarly here, I found that browsers and devices tend to support cy better than cy-GB. The latter might encounter issues like font fallback, although this doesn’t significantly affect Welsh since the default font can fully display it even with a fallback. Still, maintaining compatibility is always a good practice from a technical perspective.

Similarly here, I found that browsers and devices tend to support cy better than cy-GB. The latter might encounter issues like font fallback, although this doesn’t significantly affect Welsh since the default font can fully display it even with a fallback. Still, maintaining compatibility is always a good practice from a technical perspective.

That's good to know, and useful info from your testing, thanks @auroursa :)

As I wait for 1.94 then, it would be useful to have a clearer idea of exactly what needs to be done. How much or what percentage of messages.po needs to have been translated before that localisation is adopted and available to app users?
I really hope it's not 100%. It shouldn't be.

As I wait for 1.94 then, it would be useful to have a clearer idea of exactly what needs to be done. How much or what percentage of messages.po needs to have been translated before that localisation is adopted and available to app users?
I really hope it's not 100%. It shouldn't be.

I'm not aware of any specific threshold of completeness that the devs have given for accepting a localisation PR, it's entirely up to their discretion.
So this is purely my personal opinion as a fellow translator, but I would think if all the main UI elements and messages, accessibility labels, etc are translated then that might be sufficient? For instance, maybe some of the error messages are a bit less important to have translated immediately when a localisation is made available, and could wait for future updates?

Great to know. That's exactly how I normally prioritise the work. It might not be practical in some languages, but as pretty much every Welsh speaker also understands English, focusing on getting the main UI out in Welsh first is the most efficient and sensible way.
Also, it'll raise awareness and hopefully some users will volunteer to help and share the work.

One thing I don't think I mentioned that the devs do definitely require to merge a localisation PR is to have it reviewed by at least one other native speaker.
They seem fairly flexible on how that requirement is met though, so I don't think the person necessarily has to have a GitHub account. It could be a fellow Welsh speaker on Bluesky, for instance :)

That's yet more information that is really good to know in advance. Now I know, there are some people I can ask.
I imagine all this info would be useful to other language translators as well. Apologies if I've missed one somewhere, but it might be good to have an informative guide to localising Bluesky somewhere.

I imagine all this info would be useful to other language translators as well. Apologies if I've missed one somewhere, but it might be good to have an informative guide to localising Bluesky somewhere.

Yes that is a good suggestion, I (and others like @auroursa) have been giving ad-hoc tips and advice but maybe it would be sensible to try and draft a guide 🤔

The 1.94 release has now been cut (#6430) so the updated English message catalog is now available here:
https://github.com/bluesky-social/social-app/blob/main/src/locale/locales/en/messages.po


Missing 3.3.8 builds

What is the version?
3.3.8-3.5.0
What happened?
https://github.com/NVIDIA/DCGM/releases/tag/v3.3.8 exists and yet 3.3.8-3.5.0 tag does not exist in this repository, this yields odd results for those that package the two separately.
What did you expect to happen?
to have 3.3.8-3.5.0 tag available in this repository, or even stop tagging things with duplicate version numbers here. Maybe this repo should simply be tagged "3.5.0"?
You can assemble any tag versions in dockerfiles based on the versions of the two projects separately.
What is the GPU model?
No response
What is the environment?
No response
How did you deploy the dcgm-exporter and what is the configuration?
No response
How to reproduce the issue?
No response
Anything else we need to know?
No response

Today, we don't have release, that uses DCGM 3.3.8.

3.3.8-3.6.0 builds have now been published.


webpack.optimize.DedupePlugin

Hi,
I'm curious that why webpack.optimize.DedupePlugin is not include in production config?
Thanks for awesome work.

@katopz สวัสดีครับ | Thanks for raising this issue. I will have pull request to fix this :)

This is done :) ขอบคุณมากครับ :)


Add document type and service

Adds a new Document type and API bindings for it.
Left type in the service source for now to allow me to keep an unrelated change to the base URL in types.go.

Superseded by https://github.com/Kong/deck/pull/336


Add a unittest to check the license.md file date range

The end year in the license file should be "this year" and this should be checked on every run of the tests. If the end year is no correct, fail the test.
example:

Copyright (c) 2012-2016 SendGrid, Inc.
It is 2017, so this should be:
Copyright (c) 2012-2017 SendGrid, Inc.
and the  test should fail.
--done

I'll give this a try, should have something ready in the next few hours!

Done in  #537


chore: rewrite the postinstall script

Branched off #191
Swaps out simple-get for fetch, and uses streams instead of buffering the whole binary in memory.
$ node ./scripts/postinstall.js
[youtube-dl-exec] Downloading `youtube-dl` binary
[youtube-dl-exec] Download Location: `/var/folders/(...)/T/yt-dl-exec-10fffb83`
[youtube-dl-exec] Ensuring directory exists: `/git/microlinkhq/youtube-dl-exec/bin`
[youtube-dl-exec] Moving binary to `/git/microlinkhq/youtube-dl-exec/bin/yt-dlp`
[youtube-dl-exec] Setting permissions for `/git/microlinkhq/youtube-dl-exec/bin/yt-dlp`
[youtube-dl-exec] Installation complete

Tested with npm, yarn and pnpm, and they all hide std{out,err} logs from post-install scripts on install unless the run fails. You can confirm this as well.

Thanks for this @miraclx.
I modified your code for using debug-logfmt instead of log since it could be controlled via environment variable (added into docs too).
Also I noted we can drop a filesystem operation so I implemented the changes and dropped some unnecessary lines to make the script easier to maintain 🙂


useMutation does not work with form input and formdata

can someone tell me how to use useMutation with form input and FormData? I tried to try it but it didn't work because the value of the input was passed from react-router-dom
const { state } = useLocation<EventData>();
const [event, setEvent] = useState(state.title);

const updateEvent = async (e: React.FormEvent<HTMLFormElement>) => {
    e.preventDefault();
      let formData = new FormData();
      formData.append('imageUrl', images, images.name);
      formData.append('title', event);

      const { data } = await authAx.put(
        `/url/${state.id}`,
        formData,
        { headers: { 'content-type': 'multipart/form-data' } }
      );
     return data
  };

  const [mutation] = useMutation(updateEvent, {
    onSuccess: async () => {
      console.log('ok');
    },
  });

on Form input
<form onSubmit={(e)=>mutation(e)} className={classes.root}>
          <Input
            value={event}
            onChange={(e) => setEvent(e.target.value)}
            type="text"
            variant="outlined"
            label="Title"
          />
          <div className="img-container">
            <Input type="file" variant="outlined" onChange={handleImage} />
            <img
              className={classes.img}
              src={images.name ? URL.createObjectURL(images) : null}
              alt="images"
            />
          </div>
          <Button type="submit" variant="contained" color="primary">
            save
          </Button>
</form>

I hope someone informs me of this problem

event objects are not automatically persisted in react across async operations (which mutations naturally are), thus, you must first e.persist(), then pass the event to the mutation.


Shall I select security device credential?

[Enter feedback here]
Hi team,
I would like to confirm if the expected behavior is to select "Security device credential" instead of input PIN in the WHFB pop-up directly. Because when I input PIN in the default pop-up WireShark will report Kerberos C_name mismatch error and continue using WHFB key for RDP instead of the certificate we deployed.

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

ID: 7e9cc344-d2c1-1f8b-232a-b2cccd8b32a3
Version Independent ID: eb87d132-cc0b-8ab2-4417-51e1bc719c14
Content: Deploying Certificates to Key Trust Users to Enable RDP - Windows security
Content Source: windows/security/identity-protection/hello-for-business/hello-deployment-rdp-certs.md
Product: m365-security
GitHub Login: @Gitprakhar13
Microsoft Alias: prsriva

@xinglai22 Thank you for your feedback. This is not the expected behavior as it should use the certificate deployed. If you continue to have an issue, I'd recommend contacting Microsoft support to troubleshoot this issue.
Hope this helps!
Thanks
Sri

Thanks @xinglai22  for taking the time to share this with the Windows documentation team. I’m sorry that you’re having problems, and I want to make sure it gets to the right people that can help. There are a couple of options available to provide the best level of support for this issue:

Post to Windows on Q&A. These are moderated by community members who are very experienced, knowledgeable, and helpful.
Use Microsoft Support to search for the issue, or open a case with professional support.
Contact your Microsoft Unified Support team, such as a Customer Engineer or Customer Success Account Manager.
I'm closing this issue as there's nothing actionable for docs at this time.

Hi
I am getting the same behaviour when using the Kerberos Cloud Trust model.

When attempting to RDP to any machine the authentication prompt will default to the Windows Hello PIN (which fails as it's the wrong cert for this purpose)
I then have to click "more choices" and select "Security device credential" from the list.
I need to do this every time when attempting to RDP

If this is not expected behaviour, then I would suggest this needs to be raised internally with your development team.
If this is expected behaviour, then it would be helpful if the Docs page could be updated to mention the end user experience when attempting RDP sessions.
Many thanks!
David


Failed to load gRPC binary module

Hi, after the last update I'm getting an error. If I point to the previous commit the error disappears. (        uses: jwulf/zeebe-action@900fbc32f6b3b3901245994f2b81d74a75a6a604)
The error:
Run jwulf/zeebe-action@master
/home/runner/work/_actions/jwulf/zeebe-action/master/dist/index.js:19242
    throw error;
    ^

Error: Failed to load gRPC binary module because it was not installed for the current system
Expected directory: node-v83-linux-x64-glibc
Found: [node-v72-darwin-x64-unknown, node-v72-linux-x64-glibc]
This problem can often be fixed by running "npm rebuild" on the current system
Original error: Cannot find module '/home/runner/work/_actions/jwulf/zeebe-action/master/dist/src/node/extension_binary/node-v83-linux-x64-glibc/grpc_node.node'

Thanks for reporting this.

OK, so I rolled a fix for this, but now I get failures like this:
Run jwulf/zeebe-action@master
/home/runner/work/_actions/jwulf/zeebe-action/master/dist/index.js:19245
    throw e;
    ^

Error: Failed to load /home/runner/work/_actions/jwulf/zeebe-action/master/dist//src/node/extension_binary/node-v83-linux-x64-glibc/grpc_node.node. The module '/home/runner/work/_actions/jwulf/zeebe-action/master/dist/src/node/extension_binary/node-v83-linux-x64-glibc/grpc_node.node'
was compiled against a different Node.js version using
NODE_MODULE_VERSION 83. This version of Node.js requires
NODE_MODULE_VERSION 72. Please try re-compiling or re-installing
the module (for instance, using `npm rebuild` or `npm install`).

The environment for the run is:
Current runner version: '2.277.1'
Operating System
  Ubuntu
  20.04.2
  LTS
Virtual Environment
  Environment: ubuntu-20.04
  Version: 20210302.0
  Included Software<EMAIL_ADDRESS>  Image Release: actions/virtual-environments@ubuntu20%2F (release)

What environment is yours running in?

+1

Version 1.0.0 of the Zeebe Action uses Zeebe Node 1.2.4, which uses the pure JS gRPC implementation.
This removes the binary dependency.
It only works with Camunda Cloud / Zeebe 1.x.


Resource Owner Password Credentials Flow does not work any more?

I follow Resource Owner Password Credentials Flow in this guide but it seems not working for me: https://www.inaturalist.org/pages/api+reference#authorization_code_flow
I received some sort of token like this
POST https://www.inaturalist.org/oauth/token

{
    "access_token": "ffaa8aeba7bf0357100987fexxxxxxxxxxxxxxxxx",
    "token_type": "bearer",
    "scope": "write",
    "created_at":<PHONE_NUMBER>
}

When I'm requesting an auth required API, it just simply return me 500, without any error saying what's wrong.
GET /v1/users/edit HTTP/1.1
Host: api.inaturalist.org
Content-Type: application/json
Authorization: Bearer "the____above_____token"
Cache-Control: no-cache

{
    "error": "Error",
    "status": 500
}

I might missed something, if that is the case would be great if somebody could point me out.
Thanks

If you look at the documentation at http://api.inaturalist.org/v1/docs/#!/Users/put_users_id you'll see the route in the new API to edit a user is PUT /v1/users/:id, not /v1/users/edit. I just made a test call and it worked for me. I used the JWT authentication token https://www.inaturalist.org/users/api_token instead of the oauth Bearer token, but in theory either should work. My call was something like this:
curl -XPUT api.inaturalist.org/v1/users/:id -H "Authorization: __YOUR_JWT__" -d '{
  "user": {
    "description": "Testing"
  }
}'


Make animation inside cell

New issue checklist

[x ] I have reviewed the README and documentation
[x ] I have searched existing issues and this is not a duplicate
[x ] I have attempted to reproduce the issue and include an example project.

General information
My app need to blink a UIView inside each cell. I used this code for UITableViewCell but it doesn't work inside IGList (Animations will stop when scrolling). Have you any suggestions for me
override func cellForItem(at index: Int) -> UICollectionViewCell {
        guard let cell = collectionContext?.dequeueReusableCellFromStoryboard(withIdentifier: "OrderInfoLaterCell", for: self, at: index) as? OrderInfoLaterCell else {
            fatalError()
        }
        cell.order = order
        DispatchQueue.main.async { [weak self] in
            if let this = self {
                if this.order.isAnimation() {
                    UIView.animate(withDuration: 1.0, delay: 0, options: [.repeat, .autoreverse], animations: {
                        cell.lblStatus.alpha = 0.2
                        }, completion: nil)
                } else {
                    cell.lblStatus.alpha = 1.0
                }
            }
        }
        return cell
    }

HI @trieuhellodenteam! So looking at this, it appears that it's most likely not really an IGListKit issue but more of a UICollectionView thing in general. At a glance it's probably not the best idea to try to start an animation in the cellForItem() method since the cell might not be totally set up yet, or on the screen. You'll most likely have better luck sticking the animation code in a spot closer to displaying the cell, in particular into listAdapter(willDisplay:cell:at:)
If you're still having trouble, please provide a sample project that illustrates the issue so we can better help.


Parameterize the display and size

If you parameterize the EPD board and display size (for example, in my case I'm using an 880x528 sized 7x5 HD epd), it would be easier to simply specify a board/size instead of rooting through code to find where the EPD is defined and used.  Granted, this does not work on 'b' variants because of differences in API, but this can also be filtered and alerted upon if someone tries to use a multi-color display.

I've actually simplified this in my fork. All that's needed to specify a display is to change the import.

Forgive me in advanced, I'm a total noob and just learning how to code. I have the 12.48in screen. I am having trouble getting it to work. All of the demos in the screens folder work (12.48in.py and Weather.py). It's telling me it can't find the waveshare module for the screen. I figured you tackled this same issue because of the different screen size. Thanks!

@NickAnsy I don't have the 12.48" screen to test, but try this.
https://github.com/missionfloyd/SlowMovie/tree/12.48inch
Copy the lib folder from the waveshare 12.48" repo to the same folder as slowmovie.py

I'm having similar issues but with the 2.13" v2 screen (yes I know it's tiny but its what I have on hand to play/test with) I've changed the script to reflect my 2.13" screen and pixel parameters but keep getting errors when trying to run the script. I had to change the init and clear to match what is being used in the example script to get it to even flash/clear the screen, but still has other issues with the script.

@Estacaco That screenshot seems to show it being unable to find the video file. Are you certain it exists at /home/pi/SlowMovie/Videos/test.mp4 ?

it sure is, I tried missionfloyds fork and i get the same issue after repairing some things (my screen requires init(epd.FULL_UPDATE) and clear(0xFF) or the script errors)

The only thing that I can think of is different calls/commands with the different drivers.

So if i give it an arbitrary number for the number of frames instead of having it count them (eg:1800) it gets by the line I posted previously but then continues to fail more things having to do with ffmpeg and grabbing the jpeg.
pi@raspberrypi:~/SlowMovie2 $ sudo python helloworld.py /home/pi/SlowMovie2/Videos/test.mp4 Traceback (most recent call last):   File "helloworld.py", line 66, in <module>     generate_frame(inputVid, 'grab.jpg', msTimecode, width, height)   File "helloworld.py", line 25, in generate_frame     .run(capture_stdout=True, capture_stderr=True)   File "/usr/local/lib/python2.7/dist-packages/ffmpeg/_run.py", line 320, in run     overwrite_output=overwrite_output,   File "/usr/local/lib/python2.7/dist-packages/ffmpeg/_run.py", line 285, in run_async     args, stdin=stdin_stream, stdout=stdout_stream, stderr=stderr_stream   File "/usr/lib/python2.7/subprocess.py", line 394, in __init__     errread, errwrite)   File "/usr/lib/python2.7/subprocess.py", line 1047, in _execute_child     raise child_exception OSError: [Errno 2] No such file or directory

So I was able to resolve it wiht reinstalling ffmpeg. now im just having issues with it making and grabbing the jpeg with pil lol
/home/pi/SlowMovie/Videos/test.mp4
Traceback (most recent call last):
File "helloworld.py", line 69, in
pil_im = Image.open("grab.jpg")
File "/usr/local/lib/python2.7/dist-packages/PIL/Image.py", line 2766, in open
fp = builtins.open(filename, "rb")
IOError: [Errno 2] No such file or directory: 'grab.jpg'

Sorry you’re having so many problems - did you use Noobs, or build your own SD Card? I

No worries Tom thanks for doing all the hard leg work! I actually got it working last night. After I reinstalled FFMPEG I didnt reboot, so that seemed to fix the remaining issues! The only note to make would be some of the waveshare screens don't work with blank init() and clear() and require init(epd.FULL_UPDATE) clear(0xFF) to make it work.

Wow @Estacaco I've never seen it running on such a tiny display!

@TomWhitwell I originally got it for a Pwnagotchi, but was looking for other neat things to do with eink displays and stumbled across your project.


Update cats-effect, cats-effect-laws to 3.3.8 in series/0.x

Updates

org.typelevel:cats-effect
org.typelevel:cats-effect-laws

from 2.5.4 to 3.3.8.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.0.0

github:typelevel/cats-effect/v3_0_0?sha=v3.0.0

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.typelevel" } ]

labels: library-update, early-semver-major, semver-spec-major, scalafix-migrations, commit-count:n:2

Superseded by #294.


Adding .input-sm class, no effect

Hi,
when adding .input-sm class to the select, style not change and it kept same height so no effect.
Thanks

Same here. :(

Hey there! I am returning from a longer open source/Select2 hiatus and still not up-to-date with the current state of Select2 development in regards to this issue – maybe @kevin-brown can chime in.
The last I know is that

Select2 no longer copies classes to the dropdown and container

(ref. kevin-brown's comment) – this information is reflected in this note in select2-bootstrap-theme's demonstration pages.

Small update: I took a closer look and found that if you include the full Select2 build – [select2.full.js](https://github.com/select2/select2/blob/master/dist/js/select2.full.js) – you can set the containerCssClass option to :all: to copy the original elements CSS-classes to the select2 container.
I have set up a CodePen to show what it does: The original elements CSS-classes are added to span.select2-selection. However, this is not what select2-bootstrap-theme currently expects – which is that those classes are added to span.select2.select2-container.
@kevin-brown Should I look into rewriting the select2-bootstrap-theme CSS to support the current behavior of containerCssClass: ':all:'?

@kevin-brown Should I look into rewriting the select2-bootstrap-theme CSS to support the current behavior of containerCssClass: ':all:'?

Yes, this is the expected behaviour of containerCssClass.

@kevin-brown & all: I have this fixed (as in "rewrote the CSS to fit the expected behavior of containerCssClass) but untested in anything but Chrome, sitting in a local branch for three weeks now – but can't get around to free some time to test and merge. Please bear with me.

Fixed in v0.1.0-beta.5: https://select2.github.io/select2-bootstrap-theme/4.0.0.html#requires-select2-full-js

I have applyed this... my select now is small in size, but it doesn't open.
Also if I dont put input-sm on my select, and I put this on select2 config
containerCssClass: ':all:',
it doesnt works. The select is not opening.
What can it be?

I have found the problem... my select has a class called "select2"
so when document loads
$(".select2").select2({
width: '100%',
placeholder: function(){
$(this).data('placeholder');
},
containerCssClass: ':all:',
});
it copies the select2 class to the container and that makes all broken.
there is another way to make this without the :all: class ?
thanks!


bump ASP.NET to 2.0

No replacement for ASP.NET 1.0, but we can eventually come back to that road. For now we should support the latest version of ASP.NET available.
closes #325.

Working and manually tested.
><> draft up
Draft Up Started: 'mortal-buffoon'
mortal-buffoon: Building Docker Image: SUCCESS ⚓  (160.2048s)
mortal-buffoon: Pushing Docker Image: SUCCESS ⚓  (16.0084s)
mortal-buffoon: Releasing Application: SUCCESS ⚓  (0.9451s)
mortal-buffoon: Build ID: 01BV7J5T18G8MJZ971M8SEPFF1
><> draft connect
Connecting to your app...SUCCESS...Connect to your app on localhost:56475
Starting log streaming...
      Request finished in 0.2274ms 200
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://<IP_ADDRESS>:80/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 0.0841ms 200
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://localhost:56475/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 0.4508ms 200
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://<IP_ADDRESS>:80/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 0.1819ms 200
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[1]
      Request starting HTTP/1.1 GET http://<IP_ADDRESS>:80/
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]
      Request finished in 0.0944ms 200

><> curl localhost:56475
Hello World!

@bacongobbler let me know when you resolve the conflicts and I'll do a manual test for this one :)

I'm going to hold off until #347 has been merged as that one is a little more difficult to rebase.

Hey cats, I'm sitting here with @spboyer and he'd like to mow over this PR a bit; can we hold off until he attacks it?

Looks like he has. Thanks @spboyer!

Since draft create isn't working for me yet I copy all the files from the PR and skip the draft create but when I run draft up I get the following error
$ draft up
Draft Up Started: 'torrid-hummingbird'
torrid-hummingbird: Building Docker Image: SUCCESS ⚓  (142.9726s)
torrid-hummingbird: Pushing Docker Image: SUCCESS ⚓  (57.0417s)
torrid-hummingbird: Releasing Application: SUCCESS ⚓  (0.3145s)
torrid-hummingbird: Build ID: 01BXCADR9G96NVCPNC8820TXZG

$ draft connect
Connecting to your app...Error: could not find ready pod

I'm running version 0.8.0 and create a simple asp.net core 2.0 project.
is there any documentation for debug? or verbose because since everything is SUCESS I'm wondering at what part is the process breaking.
FYI: I ran the draft create and up with the example for node and everything work perfect.

You can check using kubectl to get an idea on the health of your pods. Everything’s deployed in the default namespace unless you changed that parameter in draft.toml.

It’ll normally take some time for the pods to initially come up, FYI.

also, let's get draft create working for you.

@bacongobbler sorry it was my mistake first time user here and since I hand copy all the files from the PR to my computer I make the mistake, I guess the Chart.yaml name:csharp must have a directory name csharp inside chart now it's working.
@squillace I try but since I'm using asp.net core 2 it didn't recognize or create the files I'm running version 0.8.0 I think support for .net 2.0 coming on version 0.9.0

the draft pack at https://github.com/squillace/draft-packs has one for F# using the dotnetcore2 with giraffe framework. I'll see about getting an aspnetcore2 one working. You can create your OWN draft pack for it right now until .9 has a basic one. :-) I'll help!

Done!


Fix ScriptInstanceExtension::get_property_default_value return value

Seems to be a typo.
Allows property reverting to work in the editor.

Thanks! And congrats for your first merged Godot contribution :tada:


cmd/store: fix filtering order

We should process the deduplication filter first, before sharding to get better load distribution between different shards. Add a test to cover this case.

Hm, what is causing worse distribution? I guess the current situation (without you PR) is that dedup might not be working correctly - as is certain stores might still show and present old blocks (replaced already by other blocks that should force dedup). This is handled fine at the runtime dedup, but maybe not very efficient.
However, the opposite situation (your PR) where dedup is first, then sharding is blocking cases when somebody would put relabel (sharding is just relabel at the end) to e.g. change deduplication logic (e.g they messed up labels). It also forces dedup to be more expensive and might not scale to the number of objects in the bucket... At least time limiting has to go first 🤔

Hm, what is causing worse distribution? I guess the current situation (without you PR) is that dedup might not be working correctly - as is certain stores might still show and present old blocks (replaced already by other blocks that should force dedup). This is handled fine at the runtime dedup, but maybe not very efficient.
However, the opposite situation (your PR) where dedup is first, then sharding is blocking cases when somebody would put relabel (sharding is just relabel at the end) to e.g. change deduplication logic (e.g they messed up labels). It also forces dedup to be more expensive and might not scale to the number of objects in the bucket... At least time limiting has to go first thinking

Yes, exactly, a lot of data is downloaded just to be discarded later on. I agree about time limiting but disagree about your other point. Not sure if the use case of "let's relabel labels just to make the deduplication filter work" seems realistic. Deduplication only matters during compaction and old, uncompacted blocks are deleted very quickly.

Ok, what about deduplication being a bottleneck now for every sync for extremely large buckets where users would like to limit store to touch only blocks with certain label?
It seems like ONLY hashmod blockID is affecting deduplication - as it would be nice to have deduplicate agnostic hashmod block ID instead of this change? 🤔

I feel this could be regression overall for some setup. Ideally we could add perhaps some flags to fine tune maybe label sharding BEFORE dedup and AFTER. Happy to merge this for now and see if there are actual perf issues of sync (not very hot path)

So I would start off by saying that I do not recommend block sharding in store gateways since it break so many things. This is obviously one of them, but downsampling is also unusable since we need to fetch 3 times more blocks. I also think it is just inefficient a concept because series now get scattered in different stores and they need to get stitched together in queriers.

Yes, exactly, a lot of data is downloaded just to be discarded later on

I am not entirely sure about this, each store needs to download each meta.json file in order to apply partitioning filters. So I don't think moving deduplication at the beginning would change anything in terms of bandwidth.
I also see the point about deduplicating blocks at download time being more expensive now. We keep data for more than a year but only serve it for the last few months. So by moving this filter at the beginning we would apply block dedup to blocks which are not even served to users.
But, I don't really know how impactful this change is and I am okay with with trying it out in production. I will report back if I see a significant performance regression.


As a user, I can see all updated stats and information on the About page of the application

-About page needs to be updated to include our # of unit tests per person and total commits, issues, etc.
-Make any refinements necessary to Phase 2
Story points estimation: 1

@stevendiaz updating this to a story point estimation of 5, since we had to import about page data from our backend api endpoint.


select in popper with set disablePortal true due to wrong position

[x] This is not a v0.x issue.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior 🤔

Current Behavior 😯

Steps to Reproduce 🕹

Link:

Context 🔦

Your Environment 🌎

Tech
Version

Material-UI
v3.?.?

React

Browser

TypeScript

etc.

Your issue has been closed because it does not conform to our issue requirements.
Please provide a full reproduction test case. This would help a lot 👷 .
A live example would be perfect. This codesandbox.io template may be a good starting point. Thank you!

@oliviertassinari we're having the same issue with Select inside the Popper component. The menu, when opened, positioned like 20px below the select itself. Here's a reproducible example: https://codesandbox.io/p/sandbox/jovial-pond-t547rk?file=%2Fsrc%2Findex.tsx%3A4%2C27
It looks better with some anchorOrigin and transformOrigin but is still wrong.


feat: Add refresh session method

What kind of change does this PR introduce?
Adds a new client method, to refresh tokens whenever you'd like - regardless of the token's expiry status.
What is the current behavior?
There is currently no way to do this. See #504
What is the new behavior?
You'll be able to use the refreshSession() method to refresh tokens (aka the Session).
Additional context
Add any other context or screenshots.

Thanks for catching that @kangmingtay!

Nice work!


I am not able to build, public method is missing.

I am using cocoapods, trying to pod 1.4.1, 1.4.2 and 1.5.0.
All those versions end up with this message.

Funny thing is code completion works.

Hello,
The TSKPinningValidator methods are class methods on 1.4.2- and an instance methods on 1.5.0+. Also the configuration() method was removed from 1.5.0.


Rule no-unused-modules does not count re-export all declarations

The rule no-unused-modules reports files which only have re-export all declarations for not having exports, eg:
export * from './foo.js';
export * from './bar.js';

This seems to be intentional as the re-exports are deleted from the count before the check.
I did not find any information on the reasoning for this behavior.
My proposed solution is to add an option flag, eg. allowReExportOnly, to allow this.
The rule would still report if not a single re-export has an export.
I am willing to create a PR for this functionality if this change is approved.

The reasoning I cam surmise is that there's no purpose in having this file - the consumers of it can (and should) be directly importing from foo.js or bar.js (I don't see any discussion about it in #1142 tho).
A PR for an option is reasonable, but when enabled, it should actually check that somebody is importing from that file - iow, if the file in your OP is index.js, and nobody is importing from index.js, then that file should be reported as unused.
Similarly, if that option is enabled, and only one of the foo.js exports (let's say "baz") is imported from index.js, and none of the bar.js exports, I would expect the rule to error until index.js has been reduced to export { baz } from './foo.js'.

The reasoning I cam surmise is that there's no purpose in having this file - the consumers of it can (and should) be directly importing from foo.js or bar.js (I don't see any discussion about it in #1142 tho).
A PR for an option is reasonable, but when enabled, it should actually check that somebody is importing from that file - iow, if the file in your OP is index.js, and nobody is importing from index.js, then that file should be reported as unused.
Similarly, if that option is enabled, and only one of the foo.js exports (let's say "baz") is imported from index.js, and none of the bar.js exports, I would expect the rule to error until index.js has been reduced to export { baz } from './foo.js'.


fields.Nested missing in the request returns an empty list []

Hi, I would like to report a weird behavior I've stumbled upon. Consider we have the following schemas..
import marshmallow as ma

class OuterSchema(ma.Schema):
    outer_field_one = ma.fields.Str()
    outer_field_two = ma.fields.Str()
    outer_field_three = ma.fields.Nested('InnerSchema', many=True)

class InnerSchema(ma.Schema):
    inner_field_one = ma.fields.Str()
    inner_field_two = ma.fields.Str()

when I try to pass a request containing only the outer_field_one, I tend to get both the outer_field_one and outer_field_three with second one containing an empty list []. I'm expecting that since I didn't include outer_field_three in the request, it shouldn't be included.
I tried to see if this behavior might be native to marshmallow itself, and do the de-serializing myself, but apparently it returns the correct results, which contains only the outer_field_one.
Was this behavior intentional?? thanks

I wrote a test to attempt to reproduce this issue:
# in tests/test_core.py:TestPassingSchema
#...
    def test_nested_field_missing_in_request(self, web_request, parser):
        class OuterSchema(Schema):
            outer_field_one = fields.Str()
            outer_field_two = fields.Str()
            outer_field_three = fields.Nested('InnerSchema', many=True)

        class InnerSchema(Schema):
            inner_field_one = fields.Str()
            inner_field_two = fields.Str()

        web_request.json = {'outer_field_one': 'foo'}

        result = parser.parse(OuterSchema(strict=True), web_request)
        assert result['outer_field_one'] == 'foo'
        assert 'outer_field_two' not in result
        assert 'outer_field_three' not in result

But this test passes.
What versions of webargs and marshmallow are you using?

@sloria: I've had this same issue using webargs 1.4.0 and marshmallow 2.10.3 — when doing a partial update via PUT request, all nested, many fields were being overwritten with empty arrays if they weren't included in the update. Not sure if this matters, but I'm parsing the OuterSchema equivalent via @use_args and additional parameters via @use_kwargs.

@bdewilde Would you be willing to post example code to reproduce the issue? Or--even better--a failing test in webargs/tests/core.py?

I was able to reproduce this issue in a test. I believe this is fixed in https://github.com/sloria/webargs/commit/f0974de50602100d445f69d7765135cc9ebae32c . @bdewilde or @chavz : Can you confirm this?

@sloria I'll do my best to check it this weekend. Very sorry about the delayed response — I was planning to write you out a full example over the long weekend! Thanks a ton for following up.

@bdewilde No worries. Thanks for reporting. Once the fix is verified, I'll go ahead and cut a release.

I was having the same issue when parsing a POST request containing a list of dictionaries:
# POST http://localhost:3000/api/analyses/
{"parameters": [
    {
        "name":"inputFile1",
        "value":"file1.txt"
    },
    {
        "name":"inputFile2",
        "value":"file2.txt"
    }
]}

Having following schemas:
class AnalysisParameterSchema(Schema):
    analysis_id = fields.Int(dump_only=True)
    name = fields.Str()
    value = fields.Str()

    class Meta:
        strict = True

class AnalysisSchema(Schema):
    id = fields.Int(dump_only=True)
    parameters = fields.Nested('AnalysisParameterSchema', many=True)

    class Meta:
        strict = True

Then parsing like this:
analysis_schema = AnalysisSchema()
...
@use_args(analysis_schema)
def post(self, args):
    ...

The result would be args[parameters] being [{}].
@sloria I can confirm, that in my case, installing the latest dev version of webargs fixed the issue!

@sloria I second that confirmation — changes in the dev version fixed the issue!

Thanks guys--I've released 1.5.1 with the fix.

Thank you guys for reporting/fixing this. Just got hit by this and latest release works a treat.


Is it a good idea to use Camunda Cockpit as the admin UI?

For example based on this plugin.

Probably not because developing a Cockpit plugin is harder than a custom web application and we want to keep it minimal.


Scrolling doesn't work in support page

Preparation

[X] I tested the latest release
[X] I looked at other issues (even the closed ones)
[X] I read the Troubleshooting Guide

FGO server
JP
FGA build number
1871
Describe the bug
I tried reinstalling FGA, force closing the app, etc, but FGA still doesn't scroll through the support list. Sometimes it even just sits there on the node selection screen.
Video
https://youtu.be/9rZRGejYXLM
Device model
Samsung Galaxy S22 Ultra
Android version
13
Screen size
3088 x 1440
RAM
12

Where do the + and - buttons in the bottom left come from? An external recorder?

Volume buttons as my phone's volume up doesn't work. Even with it not enabled, the support list still doesn't scroll when fga is running.

Does no one have any idea on what's going on?

Ah, Android 13, recording is broken.
Use build 1888 and record with the system recorder.

Ah, Android 13, recording is broken.
Use build 1888 and record with the system recorder.

I forgot to update you. This resolved the issue. Thanks!


UI flashing and turning black when upload some pictures

Hi, I have a problem with lor card maker in my Android phone. (Samsung Galaxy A15)
When I upload some picture, UI start flashing and turning black. The observation, that miror's picture is start flashing when I move it, then everything is gonna go beyond.
Follow this link below 👇
https://github.com/muggy8/lor-card-maker/assets/112291848/9c44e17e-fdef-4499-8a29-5737daf208a2

I have a feeling that this is because your device is having a hard time rendering the UI changes. The way the card renderer is implemented might be the cause and is likely unfixable. if you enable "low power device mode" in the settings, it might make things better.

Also, it seems you might be using the google play version. The google play version runs even slower than the website version because it just the website pretending to be an android app. I'd suggest trying on the website version too. that might help as well.


Add support for spherical ("360 degrees") videos

Example: https://youtu.be/S3xOB-Bigc8
The video downloads fine, but it's treated like a planar (flat) video; the resulting file doesn't contain any metadata indicating it's a spherical video, and none are made available through the -j option.
There probably isn't that much for youtube-dl to do here: detecting spherical videos, collecting metadata, putting them into info_dict, and perhaps doing some fixups in a post-processor.
Google has a draft specification for spherical video metadata, which may or may not be helpful: https://github.com/google/spatial-media/blob/master/docs/spherical-video-rfc.md

Apart from the webpage having the tags 360 or spherical(which I guess can be manually added for every video), I haven't found anywhere where it says it is a spherical video. How do you think youtube-dl can detect them?

Here's a (prettified) fragment of html5player.js which seems responsible for detecting it:
    DH.j = function(a) {
        var b = a.X().A.c.toLowerCase(), c;
        c = a.getVideoData();
        a = a.X().experiments;
        if (a.Fa && hz(c) && 3 != c.ga && 4 != c.ga)
            b = "Anaglyph3D";
        else {
            a: {
                if (c.j)
                    for (var d in c.j.j) {
                        var e = c.j.j[d].info.video;
                        if (e && 2 == e.projectionType) {
                            c = true;
                            break a
                        }
                    }
                c = false
            }
            b = c ? "Spherical" : a.$ && vj && "web" == b ? "NoOp" : a.Ha && vj && "web" == b ? "WhiteNoise" : a.P && vj && "web" == b ? "EdgeEnhance" : null
        }
        return b
    };

There's a yt:projectionType attribute in the DASH manifest, and projection_type key buried in the video metadata. The video is fully viewable in Chromium.

Any progress on this issue?

What is the update on this issue guys

Any updates guys?


Trying to fix Windows distributable

Trying to fix windows side of current distributable errors related to Pydantic

@garrettmflynn 3 tests are passing according to log output for Windows here: https://github.com/NeurodataWithoutBorders/nwb-guide/actions/runs/6102263434/job/16560353693?pr=346
But CI itself is failing and says there are 'Errors' without reasons or tracebacks, any idea how to interpret this?

This looks like it's fine. The executable test suite finishes without issue, causing the .exe to close—though this action seems to be something that registers as an error for the workflow.
The only outstanding issue is that an Error Code 1 is thrown by the latter. Not sure why or how we could figure that out—but I'll look into it

This looks like it's fine. The executable test suite finishes without issue, causing the .exe to close—though this action seems to be something that registers as an error for the workflow.
The only outstanding issue is that an Error Code 1 is thrown by the latter. Not sure why or how we could figure that out—but I'll look into it

Fantastic! Exciting.
Yeah if you can figure out how to make the CI give the glorious green check that would be great, then I'll move onto the other platforms

Found a solution on this issue: https://github.com/open-cli-tools/concurrently/issues/187
Turns out this is a common problem, I'd tried using the --success flag originally but it turns out first means the first to exit when I thought it meant the first process in the list of those to run

@CodyCBakerPhD Worked for both Windows and Linux! Unfortunately, we're running into the same Mac issue that @rly  originally solved in #89...

@CodyCBakerPhD Worked for both Windows and Linux! Unfortunately, we're running into the same Mac issue that @rly originally solved in https://github.com/NeurodataWithoutBorders/nwb-guide/issues/89...

Awesome, thanks a bunch! We can tackle Mac in a follow-up then


Environment vars not accessible by service

Hi guys!
I'm trying to fill systemd Environment variables from celery_worker. I saw it has ServiceMixin from poise-service interface but unfortunately I can't find a way to pass my env vars to service environment.
I'm trying to do like this:
application 'blabla' do
   environment ({
     :REMAP_SIGTERM => 'SIGQUIT'
   })

   celery_worker do
     app_module 'tasks'
     user 'celery'
     service_name "myapp"
   end
end

But my Environment entry on systemd service conf still empty.
Am I doing something wrong?
Thanks in advance!
Alisson

Ruby syntax oops, remove the space between environment and (. With that space there it's a block argument. Or just do environment REMAP_SIGTERM: 'SIGQUIT' to use all the various optional syntax bits :)

O.o! Now works (sometimes I forgot that it's Ruby, haha).
And it needs to be just at beginning of application block (It was in the middle here).
Thank you @coderanger !

Ordering shouldn't matter in theory, though I would have to step through the code to see if I screwed up a copy or something.

Yes, strange @coderanger . Here is my full recipe, with this config (environment in the middle) it do not pass my env var to systemd conf, but if it is just after start application block it works.
node['python-app'].each do |app|
  application '/srv/' + app['name'] do
    git do
      repository app['repo']
      revision app['branch'] if app['branch']
      deploy_key data_bag_item('ssh_keys', app['deploy_key'])['key'] if app['deploy_key']
    end

    virtualenv

    pip_requirements

    # Setup environment file
    env_filename = "/srv/#{app['name']}/env.prod.py"

    # Just to not break deploy
    file env_filename do
      content ''
      mode '0777'
      action :create_if_missing
    end

    execute 'create env modue' do
      command "cp #{env_filename} /srv/#{app['name']}/env.py"
    end

    # We need this to do a warm shutdown with Celery, with this it's possible to
    # use auto scalling and other features that needs to send a SIGTERM to celery worker
    # and not lose current tasks in process.
    #
    # For more information about this ENV VAR: https://github.com/celery/celery/issues/2839
    #
    # With this env var it's good to use a DefaultTimeoutStopSec as infinity at default config or a
    # TimeoutStopSec with same value at your service.
    environment REMAP_SIGTERM: 'SIGQUIT'

    # SystemD default conf
    template '/etc/systemd/system.conf' do
      source 'systemdconf.erb'
      mode '0644'
      owner 'root'
      group 'root'
    end

    # Setup celery
    if app.fetch('celery', true)
      celery_config do
        options do
          celeryd_concurrency app.fetch('concurrency', 1)
          celery_default_queue app.fetch('queue_name', 'celery') if app.key? 'queue_name'

          # http://docs.celeryproject.org/en/latest/userguide/configuration.html#worker-prefetch-multiplier
          celeryd_prefetch_multiplier 1
        end
      end

      file "/srv/#{app['name']}/celeryconfig.py" do
        mode '0777'
      end

      celery_worker do
        app_module 'tasks'
        user 'celery'
        service_name "s#{app['name']}"
      end
    end
  end
end

Aha, yep, order does matter because I ||= things here https://github.com/poise/application/blob/master/lib/poise_application/resources/application.rb#L90
So if something tries to use the app state before you set the environment, it would be too late. I should figure out a better way to handle that.

hum, understood @coderanger ! I'm using a lot of this recipes, If I figure out a good way to improve this I'll comment on issue or contribute by other way.
Thank you for your help!

Specifically in this case you need to set the property before using the virtualenv resource because that does a compile time change to the app state.


LMDB file download

Currently  to download LMDB files , the link opens up the lmdb file and we download each content separately. Following this link. Can you share a link where we can directly download the lmdb file as we arent able to compress the contents back into lmdb format? @elliottd @e-bug

Hi, I'm not sure I follow what you mean.
As shown here, you just need to specify a path to the folder ending in .lmdb.
So "refcoco+_unc_dets36_feat.lmdb" is not actually a file but a folder. Does this help?

Thanks for replying. I thought lmdb was a compressed file like zip. Thanks for clearing that up. Also another question, do you have a smaller version of resnet101_faster_rcnn_genome_imgfeats/ for vqa which is 1TB, which as you can imagine is quite unfeasible for downloading as well as storing.

I don't but you can preprocess a subset of the data by following the general instructions.


Generate modified names when they would otherwise be invalid

There are generated variable names that occasionally are invalid method or variable names in the target language.
For example, if the message-declarations-file.xml has a variable named final, this will cause the Java generator to generate invalid code since final is a keyword and thus final() is an invalid method name.
A simple although non-exhaustive proposal is to append something like an underscore if the variable name matches a language keyword.

If a keyword from C++ or Java is used you get an error message like the following:
ERROR: at <sbe:message name="Car"> <field name="final"> name is not valid for C++: final
Exception in thread "main" java.lang.IllegalArgumentException: at <sbe:message name="Car"> <field name="final"> name is not valid for C++: final

Are you saying code is generated and the error is not displayed? I generated the above message by naming a field as "final". No code was generated.

A quick look at the code seems like it's validating against the unmodified variable name "Final" (not a keyword) but then generates based on a call to toLowerFirstChar() resulting in "final".  I believe this lower casing was introduced between 1.1.7 and 1.3 fwiw.

Thanks I see what you mean now. I also found a couple of instances where we are not checking for keywords on type and message names which has been fixed.


testing changes

First change is about making sure the tests are calling dbt-bouncer as any other user installing this package would. More info in the commit description.
About the second change, I'm not sure about the current approach. Currently, the github-specific functionality is part of the dbt-bouncer python package. I think ideally it'd be another package/script that is included in the action alongside the dbt-bouncer package. In any case this is an immediate fix so that tests can run locally without the GITHUB_TOKEN envvar

the second commit is mehing with the coverage but hey


Check build arch

https://github.com/xianyi/OpenBLAS/blob/5dde4e65d321076582a2fafe16949d2160551e81/CMakeLists.txt#L59

Checked:

conan create OpenBLAS/0.2.20/danimtb/testing -s arch=x86
Produces binaries (both dll and test) with x86 arch
CMake working well


Static/ manual Uptime Monitor with no Uptime Check

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor, UI Feature
🔖 Feature description
I think i would be nice to have a function to disable the automatic uptime checks from UptimeKuma for example a site from a organisation and the admin can set the status manuelly.
Another thing is a incident log or multiple incidents on one site or a subsite for only incidents and planed incidents.
✔ Solution
For the monitor without a uptime check you only must add on the dropdown action another field for "Disable Checks" and after this, a function to set the current state of the object manuelly.
❓ Alternatives
No response
📝 Additional Context
No response

Closing as a duplicate of #475

Another thing is a incident log or multiple incidents

This is tracked in #862


Capital Case / Lower-Case

Hi All,
I want to utilize abapGit as deployment verification in our jenkins deployment pipeline. Simple Idea: In case abapGit can not recognize any open changes after a transport is done (pushed via commit), than the transport was correct "on byte level".
In comparison of Classes i have unexpected side effects with our pretty printer and the ABAP transport management system. On the left side you have the target system (e.g. production), on the right side you have the development system (or better git status).

Unfortunately it seems that the capital/lower case settings done in eclipse (e.g. via formatting) are not applied during transport. Instead they are just overwritten by the transport management system. Even after transporting R3TR/CLAS the changes are not taken over to the target system.
"Workaround": Manually adjust the class name to be capital case in the target system. This workaround does not really work, as with the next eclipse pretty print the upper case is restored.
Do you have experienced this issue already and maybe even have a solution? If not: The only solution I could think of is serializing the class name as capital case in the IMPLEMENTATION line (sounds like an extremely ugly solution unfortunately :-(). What do you think of that?
We are using NW ABAP 750 / SP12.
Thanks,
Timo

I would regard this as a bug in the SAP deployment logic. However it will probably not be changed.
See the last exit described in https://docs.abapgit.org/ref-exits.html it can be used to change the serialization logic.
Additionally/another approach is to force the file in git to be upper case by triggering errors via linting.

Hmm ya.. I assume the same answer is valid for the problems:

method order after deployment is different, in case it was changed manually in eclipse in development system
spaces / line breaks added between methods are not deployed, and therefore differ

?

one additional question.. How are you handling auto-generated classes in customer namespace?

I knew someone else would also run into this problem :)
The example implementation linked in the docs for the exit should give you consistent serialization results, even if SAP randomly decides to reorder and reformat stuff. As far as I know the "old" class serializer which is used there has a fixed amount of lines between method implementations and always constructs the (END)METHOD statements in all uppercase. Everything else should just be the source code of the separate includes (okcode SHOW_CLIF in SE24).
One thing to keep in mind is that if you enable it for existing repositories you will get one huge commit which changes the method order in all classes to be alphabetical and therefore complicates your git history quite a bit.

hmm - ok i just should have checked the provided link :-)
I will give the "old" de-serializer a try and close this one, in case it is working 👍

OK - works perfectly.. Thanks a lot.

disadvantage: the code made by the developer does not match what is moved to git


Ubuntu 18.04 installation fails

# pip install geetools
Collecting geetools
  Using cached https://files.pythonhosted.org/packages/51/83/3f6f6cc185902d27468dcbef69736d12244386ec26439cabb6a09a30f471/geetools-0.1.7.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-bitz479n/geetools/setup.py", line 5, in <module>
        from geetools import __version__
      File "/tmp/pip-build-bitz479n/geetools/geetools/__init__.py", line 17, in <module>
        from . import tools, bitreader, cloud_mask, expressions, decision_tree,\
      File "/tmp/pip-build-bitz479n/geetools/geetools/tools/__init__.py", line 3, in <module>
        from . import imagecollection, date, dictionary, image, number, \
      File "/tmp/pip-build-bitz479n/geetools/geetools/tools/imagecollection.py", line 3, in <module>
        import ee
      File "/usr/local/lib/python3.6/dist-packages/ee/__init__.py", line 1, in <module>
        from .main import main
      File "/usr/local/lib/python3.6/dist-packages/ee/main.py", line 10, in <module>
        import StringIO
    ModuleNotFoundError: No module named 'StringIO'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-bitz479n/geetools/

likewise with pip2, or pip3

Hi @alexmyczko, it seems to be an issue in the Google Earth Engine API. Try to install it first and see what happens.

Could you please be more specific what I should do? We have CUDA 10 and Ubuntu 18.04. Is the instructions to be done with Python 2 or 3?

Ok. I'd recommend to use a virtual environment, so I guide you through this.

Make the Virtual Environment
a. open a terminal in the path you want to have the virtual environment
b. we'll name it geepy3, you can change that. Type:

vitrtualenv geepy3 --python=python3

activate your environment. Type:

. geepy3/bin/activate

Install google earth engine python API

pip install earthengine-api oauth2client

Test it

python -c "import ee; ee.Initialize()"

If it is the first time you run it, it will ask you to open a link and copy/paste a code...

Then you can install geetools. Type:

pip install pandas pygal ipyleaflet folium pyshp pillow requests geetools

Let me know how it goes =)

Hi @alexmyczko, did you make it run?

Hi @fitoprincipe
I installed it successfully.
It was me having that problem in the first place and @alexmyczko was trying to help me.
Thanx.

Hi @KonstantinosF
Great! Well, any other issue/comment, I am here =)
Best
Rodrigo


Mount /etc/resolv.conf and /etc/hosts instead of copying

Notes to myself: technically it's a breaking change. There were folks who somehow abused two facts:

Container has some reasolv.conf which might still work
When you run second container, already running container got new resolv.conf

Both hacks are quite unreliable. And second one might also partially work: if resolv.conf was "overwritten" rather than "replaced". In the latter case in may also work as a bugfix, still not 100% reliable though. Another note is that in vagga box we overwrite resolv.conf of the virtualbox on each start of a new container.


web: icons do not follow page ltr/rtl direction

Thanks so much for maintaining this repo!
This is a replicated issue from https://github.com/google/material-design-icons/issues/750
In summary direction:ltr tells the browser where to position an icon, not how the icon is drawn. It's forcing left-to-right (LTR) even on RTL (right-to-left) layouts. The line should be removed.

Let me start of by saying, I'm not talking about glyph/icon mirroring.
When a page is using [dir=rtl] items should flow from right to left. Material Design Icons is forcing the orientation to be left to right, even on pages that should be right to left. This forces you to div wrap the icon for the layout to flow properly.
The line in question, specifically, is this one:
https://github.com/jossef/material-design-icons-iconfont/blob/25db3c0d5825bd09237fad6e4af750b80b7c4fd6/src/material-design-icons.scss#L33
Here's the codepen explaining the issue:
https://codepen.io/shortfuse/pen/RMgrOw
You can see the warning icon isn't properly respecting the page's RTL implementation.
I'm not sure why the direction: ltr since no browser actually mirrors the icons. This is even stated in the guide:

By default on the web, icons are not mirrored when the layout direction is mirrored. You need to specifically mirror the appropriate icons when needed.

Perhaps the usage of direction: ltr was misunderstood when the rule was added. If it is fully intentional, there should be a better explanation as to why it has to be used and how to workaround the layout direction issue.

@clshortfuse nice catch
would you like to make a PR?

I want to remove it.
double-checked with the original font's CSS
it's coming from there.
keeping this issue open for now.

You can either remove it, or use direction: inherit; which allows RTL pages to work properly. I use inherit instead of removing to ensure the rule is changed.
The codepen still propertly shows the issue. I'll make the PR for you.


feat: docs

Motivations

Would you like to implement this feature? [y/n]

Solution

Alternatives

Additional context

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Workaround Platform.localName == "null" (the string)

https://github.com/flutter/flutter/issues/59144
Without this using the iOS simulator seems to be impossible for me.

ping?

Closing as https://github.com/flutter/flutter/issues/55642 is closed as completed.


Disable creating new chats if the server is on Big Sur or up

Fixes #929.
Applies opacity to FAB on conversation list, and disables pressing the "start conversation" and "forward" items in the menu options.
"Forward" is disabled because we can't (atm) prevent users from sending forwarded messages to people they don't already have a chat with.

Ah I was under the impression it didn't work at all. I'll rework this then and add a snack bar when entering the chat creator or something.

@zlshames made the changes, this should be ready to go.


RelativeMouseDrag should not be at a framework level

If another project wants to implement this non-standard behaviour, they should just rewrite said code in my opinion. @Tom94 has already made it clear he thinks this should exist in the framework, but I still disagree.
I am going to go ahead and remove all usage of it in the framework (currently in very few places) because it is non-standard. Pending further discussion, I believe it should be moved to the osu! project where it is only enabled on a user explicitly enabling it from a disabled default.

Never had a strong opinion; see https://github.com/ppy/osu-framework/pull/857 . I just personally like how scrolling with it feels like.


[github] handle response with no errors field from create event

What does this PR do?
Handle case where the response has no errors field
Motivation
Currently failing as the script throw an error, because the response has no errors. Which is ironic.
Describe how to test/QA your changes
Tested locally by running with both valid and invalid DD API key.
On valid key, merged PR:
Event sent to Datadog for PR #30208

On invalid API key
Error while sending pipeline event to the Datadog backend: (403)
Reason: Forbidden
HTTP response headers: {'Date': 'Wed, 06 Nov 2024 09:32:28 GMT', 'Content-Type': 'application/json', 'Content-Length': '169', 'Connection': 'keep-alive', 'x-content-type-options': 'nosniff', 'strict-transport-security': 'max-age=31536000; includeSubDomains; preload'}
HTTP response body: {'status': 'error', 'code': 403, 'errors': ['Forbidden'], 'statuspage': 'http://status.datadoghq.com', 'twitter': 'http://twitter.com/datadogops', 'email'<EMAIL_ADDRESS>
Possible Drawbacks / Trade-offs
Additional Notes

/merge


perf: filter Share Logs options

Details

This PR implements filtering of options in Share Logs list, instead of relying on searchText property from options.
Fixed Issues

$ https://github.com/Expensify/App/issues/37619
PROPOSAL:
Tests

Go to Profile -> Troubleshoot
Enable Client Side Logging
Go to Debug view
Press "Share logs"
Type something that matches an existing item in the option list
Verify the items are returned in the same order as if you type the same on the main
Clean the input and type a random email that does not exist on the list
Verify the email appears on the list if it's valid

[x] Verify that no errors appear in the JS console

Offline tests

QA Steps

Go to Profile -> Troubleshoot
Enable Client Side Logging
Go to Debug view
Press "Share logs"
Type something that matches an existing item in the option list
Verify the items matching the searched text are on the list
Clean the input and type a random email that does not exist on the list
Verify the email appears on the list if it's valid

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/13985840/b3fa1b15-cb25-4fd5-b4e4-0ee0c74acc8e

Android: mWeb Chrome
https://github.com/Expensify/App/assets/13985840/acb275bc-b218-49fe-b873-e9bcb7958795

iOS: Native
https://github.com/Expensify/App/assets/13985840/5ed5c2d0-a08f-442c-ada8-0c479c31d37b

iOS: mWeb Safari
https://github.com/Expensify/App/assets/13985840/2dc00975-2362-4e71-8011-67a81d60a4dd

MacOS: Chrome / Safari
https://github.com/Expensify/App/assets/13985840/f650b5a8-ce32-486b-ad98-6a49d5933a2d

MacOS: Desktop
https://github.com/Expensify/App/assets/13985840/e0f73066-4e43-4c1e-a2a3-834eb5779610

To be opened once #40285 gets merged - it's sharing the same logic for inviting a user as in the mentioned PR

Updated branch with newest main. @mollfpr could you please review it when you have a moment?

@TMisiukiewicz I'll review the PR now. Could you help resolve the conflicts? Thank you!

@mollfpr sorry for late action, I had a very limited capacity before the weekend. Conflicts are resolved ✅

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

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

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/roryabraham in version: 9.0.10-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/mountiny in version: 9.0.10-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.10-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@roryabraham Could you help create the issue for the payment? Thank you!

Created https://github.com/Expensify/App/issues/46079

🚀 Deployed to production by https://github.com/francoisl in version: 9.0.11-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Changes to PATH variable not recognized by the msys2 shell

I am trying to install a Perl module installer called cpanm. Here, is my workflow file (simplified for debugging purposes):
name: windows-msys2
on: [push, pull_request]
jobs:
  build-dist:
    runs-on: windows-2019
    defaults:
      run:
        shell: msys2 {0}
    steps:
      - uses: msys2/setup-msys2@v2
        with:
          msystem: MSYS
          path-type: minimal
          install: base-devel
      - uses: actions/checkout@v2
      - name: setup perl locallib
        run: |
          TOPDIR=$(cygpath "$GITHUB_WORKSPACE")
          PERL_LOCAL_LIB_ROOT="$TOPDIR/perl5"
          >>"$GITHUB_ENV" echo "PERL_LOCAL_LIB_ROOT=$PERL_LOCAL_LIB_ROOT"
          >>"$GITHUB_ENV" echo "PERL5LIB=$PERL_LOCAL_LIB_ROOT/lib/perl5"
          >>"$GITHUB_ENV" echo "PERL_MB_OPT=--install_base \"$PERL_LOCAL_LIB_ROOT/\""
          >>"$GITHUB_ENV" echo "PERL_MM_OPT=INSTALL_BASE=$PERL_LOCAL_LIB_ROOT"
          >>"$GITHUB_ENV" echo "PATH=$PERL_LOCAL_LIB_ROOT/bin:$PATH"
      - name: install cpanm
        run: |
          echo "PATH=$PATH"
          cpan App::cpanminus  # This command installs cpanm
          export PATH
          ls -l $PERL_LOCAL_LIB_ROOT/bin/cpanm
      - name: test cpanm
        run: |
          ls -l $PERL_LOCAL_LIB_ROOT/bin/cpanm
          export PATH
          cpanm -v Path::Tiny

Here is a link to the log for failed workflow run.
As can be seen, the step "install cpanm" installs the executable script cpanm into the directory /d/a/msys2-gha-path-test/msys2-gha-path-test/perl5/bin and in step "test cpanm" ls -l shows that the script exists in the correct directory. Also PATH is exported and contains:
PATH: /d/a/msys2-gha-path-test/msys2-gha-path-test/perl5/bin:/usr/local/bin:/usr/bin:/bin:/opt/bin:/c/Windows/System32:/c/Windows:/c/Windows/System32/Wbem:/c/Windows/System32/WindowsPowerShell/v1.0/:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl

Notice that the first entry in PATH is the directory where the script cpanm is installed.
Still, when the runner executes the next command cpanm -v Path::Tiny it fails with
D:\a\_temp\fee6b146-c300-4100-800d-0ad7b7e626b2: line 3: cpanm: command not found
15
Error: Process completed with exit code 127.

What can be the problem here?

That is the default behaviour of MSYS2: the environment/PATH are not inherited. See https://github.com/msys2/setup-msys2#path-type, #98 and #104.

That is the default behaviour of MSYS2: the environment/PATH are not inherited. See https://github.com/msys2/setup-msys2#path-type, #98 and #104.

the environment/PATH are not inherited.

@eine Thanks! So you cannot export  the PATH to the next step via GITHUB_ENV like I did. You need to reconstruct and set the PATH manually at each step? What is confusing is that you can use GITHUB_ENV to export PATH to next step on linux and macOS runners, but on MSYS2 it does not work.

the environment/PATH are not inherited.

@eine Thanks! So you cannot export  the PATH to the next step via GITHUB_ENV like I did. You need to reconstruct and set the PATH manually at each step? What is confusing is that you can use GITHUB_ENV to export PATH to next step on linux and macOS runners, but on MSYS2 it does not work.

Please, see the references I provided.

You need to reconstruct and set the PATH manually at each step?

This is explained in https://github.com/msys2/setup-msys2#path-type. You don't need to, but it is recommended.

What is confusing is that you can use GITHUB_ENV to export PATH to next step on linux and macOS runners, but on MSYS2 it does not work.

See #98 and #104.

Please, see the references I provided.

You need to reconstruct and set the PATH manually at each step?

This is explained in https://github.com/msys2/setup-msys2#path-type. You don't need to, but it is recommended.

What is confusing is that you can use GITHUB_ENV to export PATH to next step on linux and macOS runners, but on MSYS2 it does not work.

See #98 and #104.


Create cicd_workflow.yml

Resolves #5

Completed by https://github.com/MeltanoLabs/tap-athena/pull/12