Dev environment guide

Need easy-to-follow instructions for preparing a dev environment.
This would probably include:

pull MDT (and possibly nbmolviz and pyccc as well)
pip install -e any relevant packages
install docker, pull the images.
install python dependencies locally?
know how to build new docker images

Still a big problem that docker images with MDT code in them will lag behind your dev version. Some sort of solution that lets us mount the code into the containers would be ideal. Possibly need to have a custom "dev" mode in CCC that would let you do this.
We need a better way to do interface development - a JSON interface + sending the code functions every time would probably fix this.

In progress at /DEVELOPMENT.md
TODO:

more Conda, less PyEnv
dev version synchronization with nbmolviz
Encourage py3 as the target, then backport as necessary to py2


Implement relay/proxy mode in the lxc client

This is about implementing the relay/proxy feature in the client, where either on transfer failure or when a remote flag is set, the client will act as a relay between the two LXDs.

+1

+1, Useful feature.

+1, per tkuhlman's reference, to be able to copy image to a remote LXD

As i'd like to copy a container from my notebook at home to an external rootserver, +1.

Marking as blocked on #2402 and #1926

This missing feature of copying/pushing a container from behind a NATed machine is quite a downer.  It seems like it could be easily addressed by modifying the NATing firewall to forward 8443 and setting the config of the NATed LXC source, but
lxc config set core.https_address xx.xx.xx.xx:8443 specifying the address of the firewall's forwarded port does not work:
error: cannot listen on https socket: listen tcp xx.xx.xx.xx:8443: bind: cannot assign requested address.
Any way to force lxc to think it's listening to this forwarded address/port?

You can't listen on an address that doesn't exist. It's not LXD refusing to do it, it's the kernel.


docs: add Mixpanel to argo/USERS.md

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[ ] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.
[x] My builds are green. Try syncing with master if they are not.
[x] My organization is added to USERS.md.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@nojvek how can we please confirm your relationship with Mixpanel please?

@alexec my profile has mixpanel badge

I can also send an email from my mixpanel address to make a verification. Let me know where to send to.


Support for minimax-regret curation

Dear authors,
Thanks for deleloping this brilliant package to accelerate research in UED!
I'm trying to develop a minimax-regret curator, which seems to be not included in the current repo yet. E.g., for the maze task, the plan is to use algorithms like A* to compute the performance upper bound on each task, so that we can curate with the true regret on each task. I was wondering if you have some suggestions on what may be the best way to implement this? Is it sufficient to just add a new runner to support this new function? Thanks!

Thanks! One way to do this would be to extend ued_scores.py with your custom score function, then simply pass --ued_score='your_function_name' as part of train.py's command line args.


Full height of some row

How can do this?
https://monosnap.com/file/kAugaWbteFgRuOkTuHdlVFYe8g7YUk
http://the-echoplex.net/flexyboxes/?fixed-height=on&display=flex&flex-direction=column&flex-wrap=nowrap&justify-content=flex-start&align-items=flex-start&align-content=stretch&order[]=0&flex-grow[]=1&flex-shrink[]=1&flex-basis[]=auto&align-self[]=stretch&order[]=0&flex-grow[]=0&flex-shrink[]=1&flex-basis[]=auto&align-self[]=stretch

If you are trying to make a 100% page height layout you'll need to add your own flexbox wrapper.
Grids are flex themselves and will stretch correctly if you use them as children of a flex container.


feature request: autoscaling scales also down again and/or fixed range for external devices

screencapture.zip
Would it be possible to add different autoscale options? Here is a simple example (as you know I'm heavily into bio): imagine you want to record the tremor on somebody's hand which are tiny changes of the acceleration. Then suddenly the person moves the hand a bit then the signal goes through the roof (see video above where I held the Attys first still and then shook it violently and then kept it still again). The same problem is with ECG or other biosignals which are tiny. A fridge next door might cause a massive spike so that the ECG trace would vanish for the remainder of the experiment. It would be great if you could add a "reset" button for the autoscaling or perhaps a gentle re-scaling towards the initial scales at a timescale of a couple of seconds and/or the width of the plotwindow. I guess you'd want to have such functionality at a minimum and transparent to the user.
As a more local fix (and leave it to the external developers) I'd be happy if just the observer class had an optional argument where the scaling could be controlled and then the external device could either leave it to ScienceJournal or could take care of the scaling by itself. For example an ECG will always be about 1mV so no need to scale or an EEG at 100uV.

These are great suggestions.  One trick is that if you tap the current sensor icon in the sensor tab drawer, it does reset the scale.
Allowing the API sensors to set the scale is a very interesting idea.  Do you expect that you would use it if it were provided?

Thanks. I should have tried tapping the current sensor.
I'd definitely use the feature to set the scale and especially for the analogue inputs I'd like to provide different scalings depending on the applications. ECG for example would do best with a fixed scaling of +/- 2mV (ECG is always around 1mV) but for certain applications I'd love to write a dynamic scaling which is pretty much in the spirit of this app to have it as userfriendly/easy to use as possible. Especially for the analogue inputs that would be really good to have customs ranges which intelligently adadpt so that for example spurious spikes for would be ignored for the scaling. I def would use it.

Just to illustrate it with my ECG (just added custom config for biosignals). I sat still, pressed REC and then at the end I pressed REC again to stop it but because I reached out my finger to press REC I created some nasty artefacts. Hard to avoid when recording it from myself. In this case limiting it to +/- 2mV would be great. I'm aware that this would saturate the min/max readings in this case. Perhaps a region to pinch with both fingers would be great also to measure time.

FYI, we're planning in a release soon to support cropping of trials, so that you could snip out the early and late invalid data.

Excellent. That would be fantastic.

Great that you guys have cropping in your roadmap. Still I think it would be great if the y scaling could be controlled. I was wondering if one could pinch and zoom into the data while it's recording so that one could evaluate if it's worth continuing a trial. I think it's already there. I can pinch it and it scales up a bit but it bounces back. I know I can reset the recording with a new scaling which is good but on a tablet that would be fun being able to use your fingers to zoom in. Many interesting signals are tiny (not just bio-signals but for example temperature from a thermocouple) and from the educational aspect that would be great to see how tiny they are and then zooming in. Also artefacts are often not just at the start / end (the fridge can go off any time) so that even cropping won't prevent the y-scaling getting squashed. Perhaps pinch/zoom is better than expecting the external drivers to provide some dynamic scaling info but rather having the student doing it and experiencing the size of a signal? I've just shown a student her ECG first on full scale (=flat line) and then we zoomed in until it was visible.

Pulled out a separate feature request for zooming Y.  How about we try that first, and see if it resolves most of the concerns?  Thanks.

Sounds good to me. I think in combination with the x cropping in post that should work resolve most of the concerns indeed.

OK, sounds like cropping + Y zooming satisfies this request, so closing this particular issue.  Feel free to re-open if I've misinterpreted!


Design a Logo for this project

Currently, the project is having the name "OnlineBookSales".
Design a logo based on this name or if you have any better idea, kindly share with us.

Hey @Trisha-tech I would like to work on this issue.

Hello @Sarguroh20 , Go for it. BrowseBookHub also looks good.
All the Best

Hey @Trisha-tech I would like to work  . I would like to solve this issue with  "BookshelfExpress"  , Could you assign the issue to me?

Hey @Trisha-tech, I've designed some logos. Can you review them?
Here is my Canva link:
https://www.canva.com/design/DAGE_pm2Hg0/htpgI6nqt3s3fUML4CG0CA/view?utm_content=DAGE_pm2Hg0&utm_campaign=designshare&utm_medium=link&utm_source=editor

Hey @Trisha-tech, I've designed some logos. Can you review them? Here is my Canva link: https://www.canva.com/design/DAGE_pm2Hg0/htpgI6nqt3s3fUML4CG0CA/view?utm_content=DAGE_pm2Hg0&utm_campaign=designshare&utm_medium=link&utm_source=editor

@Sarguroh20 All are good.
First decide the theme of the project. min 2 color and max 4 colors must be there.

Hey @Trisha-tech ,i like  to work with you,i suggest you to choose a logo from here ,
first one is :

2.second on is:

3.Third one is:

Hey @Trisha-tech, I've designed some logos. Can you review them? Here is my Canva link: https://www.canva.com/design/DAGE_pm2Hg0/htpgI6nqt3s3fUML4CG0CA/view?utm_content=DAGE_pm2Hg0&utm_campaign=designshare&utm_medium=link&utm_source=editor

@Sarguroh20 All are good. First decide the theme of the project. min 2 color and max 4 colors must be there.

I usually decide the theme color of my website based on logo. What about you?
https://palettes.shecodes.io/
https://gradients.shecodes.io/

Hey @Trisha-tech, I've designed some logos. Can you review them? Here is my Canva link: https://www.canva.com/design/DAGE_pm2Hg0/htpgI6nqt3s3fUML4CG0CA/view?utm_content=DAGE_pm2Hg0&utm_campaign=designshare&utm_medium=link&utm_source=editor

@Sarguroh20 All are good. First decide the theme of the project. min 2 color and max 4 colors must be there.

These are just sample theme examples:
When it comes to a combination of two colors, this is how it looks:
https://github.com/Trisha-tech/OnlineBookSales/assets/116432233/9ebf5e20-5759-42e2-b7ab-fa2086083050
Combination of 4 colors:
https://github.com/Trisha-tech/OnlineBookSales/assets/116432233/245dc455-232e-4ce9-b0e8-c91dd6387fde

Can I work on this as gssoc'24- Extd contributor.

Can I work on this as gssoc'24- Extd contributor.

Hello @gaurkrutika, logo is already there in the website. If you have any better idea, feel free to share first.


[3.0.1-prepare][cherry-pick]fix status check issue (#11580)

Purpose of the pull request
this pr: #11580

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

Codecov Report

:exclamation: No coverage uploaded for pull request base (3.0.1-prepare@70ddb11). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             3.0.1-prepare   #12030   +/-   ##
================================================
  Coverage                 ?   39.86%
  Complexity               ?     4582
================================================
  Files                    ?      888
  Lines                    ?    35114
  Branches                 ?     3860
================================================
  Hits                     ?    13998
  Misses                   ?    19759
  Partials                 ?     1357

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


On getStarted.html page all heading tag should have same font-style ,it is different after Base Icon Button.

Describe the bug
On getStarted.html page there is different heading font-style ,it should be same for all the heading on the page.
##Here it has correct heading style.

##But here the font style of heading is different it should be similar.

Makes sense. Would you like to work on it @SARVESHKHANDELWAL ?

Hello @shahednasser, I have created a PR that fixes this issue.
Also, do issues like this qualify for hacktoberfest contributions? I am unsure as only some issues have the hacktoberfest tag.


feat: owner nft settings page

A few outstanding questions:

I had to add src/components/v2v3/V2V3Project/V2V3ProjectSettings/pages/ProjectNftSettingsPage/Capsules.module.css to .eslintignore. How can I better handle these eslint errors?
Is there a better way to locally declare a @font-face than a css module?

This is awesome. @filipviz I'm seeing a little UI bug with the preview being too small.
https://juice-interface-next-cic6rvl8c-peel.vercel.app/@mijanidao/settings?page=projectnft


Delete ingress v1beta1 code paths (breaks k8s <1.19)

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
When versions of k8s older than 1.19 go out of support for KIC, we'll be ready to delete the code paths associated with networking.k8s.io/v1beta1 and extensions/v1beta1 Ingress.
The scope of this issue is to drop support for talking to the API server using APIs older than v1.
Note to users: this is not equivalent to dropping support for v1beta1 Ingress resources. As long as k8s accepts v1beta1 Ingress resources (FWIW, until v1.22 not inclusively), they will work with KIC regardless of this change.
Proposed Solution
Drop networking.k8s.io/v1beta1 and extensions/v1beta1 code paths from KIC
Drop the ingress version negotiation mechanism.
Additional information
Marking this issue blocked because this needs to wait until we decide not to support k8s versions older than 1.19.
Acceptance Criteria

[x] Code paths for ingress versions older than v1 have been dropped, and the controllers have been deleted.
[ ] Ingress class version negotiation mechanism has been deleted.
[ ] Documentation clearly says that k8s <1.19 is not supported anymore.

At the moment of writing, this is a breaking change for ~11% of installations. Out of those installations that we'd break, 97% are on KIC older than 2.0.
Also, <1.19 appears to be unsupported by any major cloud provider.
I think that we're ready to make this breaking change.

Considering that KIC 2.6 already removed support for k8s < 1.22 de facto, it is OK to make such a breaking change now.

Unclear whether the cost to benefit ratio justifies deleting the old code paths (in the event we want to work with ancient vendor versions of Kubernetes that aren't v1-capable.
Soft alternatives available:

say explicitly that v1beta1 APIs are unsupported anymore and can be deleted at any point in time
the above + hide it before a feature flag so as to prevent accidental use

Since 2.5 is (going to be?) marked as LTS we'll still have something to offer for those users that run 1.21 and older.
I'd personally try to trim down the code surface of supported k8s object versions especially those that are not served by the respective kubernetes API version anyway.
#3867 aims to drop support for netv1beta Ingress. We can target it for whichever version we want but I'd vote for sooner rather than later since we don't support 1.21 and older at this point anyway, as per https://docs.konghq.com/kubernetes-ingress-controller/latest/references/version-compatibility/#general.

@mflendrich

Ingress class version negotiation mechanism has been deleted.

I was looking for this but can't seem to find anything relevant. Did you mean this https://github.com/Kong/kubernetes-ingress-controller/blob/eb47a6906e583a5b39cbb15038671d7fef472062/internal/controllers/utils/utils.go#L24-L48 which already doesn't take netv1beta1 Ingress into account?

As for

Documentation clearly says that k8s <1.19 is not supported anymore.

Is the table at https://docs.konghq.com/kubernetes-ingress-controller/latest/references/version-compatibility/#kubernetes enough?


i can not use route.navigate after then

I want to move the component after the OK button is pressed.
I use code like this,
result.subscribe((data) => {             swal({                 title: 'Good job!',                 text: data.message,                 type: 'success',             }).then(function () {                 this.router.navigate(['menus/categories']);             });         });
But the result of a console like this,
RROR Error: Uncaught (in promise): TypeError: Cannot read property 'router' of undefined TypeError: Cannot read property 'router' of undefined     at form.component.ts:94
Is there anything wrong or can not be with route.navigate?

Hi @rifkysyaripudin,
You are using swal(), so this is not technically about NgSweetAlert2, but SweetAlert2.
Anyway, that is because you're using function() in the callback of then(), and function() expressions in JavaScript don't capture the this of the scope in which the callback is defined, unlike the other variables. You need to use the arrow notation (() =>).
I've explained it all here, where an user has the same problem you have: https://github.com/limonte/sweetalert2/issues/630#issuecomment-331239542

Oh yeah, it work. Awesome!
Thanks @toverux


Audio Support

We should integrate some sort of audio component to with web audio api.
https://developer.mozilla.org/en-US/docs/Web/API/Web_Audio_API

The support will depend on the Browser Compatibility
https://developer.mozilla.org/en-US/docs/Web/API/Web_Audio_API#browser_compatibility.
What audio api can be support by pynecone is depended on what version of browser pyencone plan to support.

This is supported by the rx.audio component: https://reflex.dev/docs/library/media/audio/


Handle unknown chars in segmentation_upid

Input /DBKAAAAAAAAAP/wBQb+YtC8/AA0AiZDVUVJAAAD6X/CAAD3W3ACEmJibG5kcHBobkQCAsGDpQIAAAAAAAEKQ1VFSRSAIyowMljRk9c=
Actual behaviour: UnicodeDecodeError: 'utf-8' codec can't decode byte 0xc1 in position 12: invalid start byte at https://github.com/futzu/scte35-threefive/blob/f41c8a614abc1c866c5ddb48caf54368c235145a/threefive/bitn.py#L55
Expected behaviour:

Either, having that exception handled, setting segmentation_upid to a default string
Or, allowing user to provide encoding charset

let me check it out, Ill get back to you shortly.

Passing a charset in is a can of worms I do not wish to open, however, I will do,
 return int.to_bytes(stuff, wide, byteorder="big").decode("utf-8", errors="replace")
and the segmentation upid will come out like
    "descriptors": [
        {
            "tag": 2,
            "descriptor_length": 38,
            "name": "Segmentation Descriptor",
            "identifier": "CUEI",
            "components": [],
            "segmentation_event_id": "0x3e9",
            "segmentation_event_cancel_indicator": false,
            "program_segmentation_flag": true,
            "segmentation_duration_flag": true,
            "delivery_not_restricted_flag": false,
            "web_delivery_allowed_flag": false,
            "no_regional_blackout_flag": false,
            "archive_allowed_flag": false,
            "device_restrictions": "Restrict Group 2",
            "segmentation_duration": 180.12,
            "segmentation_duration_ticks": 16210800,
            "segmentation_message": "Not Indicated",
            "segmentation_upid_type": 2,
            "segmentation_upid_type_name": "Deprecated",
            "segmentation_upid_length": 18,
            "segmentation_upid": "bblndpphnD\u0002\u0002\ufffd\ufffd\ufffd\u0002\u0000\u0000",
            "segmentation_type_id": 0,
            "segment_num": 0,
            "segments_expected": 0
        },

Can you live with that solution?

pip install --upgrade threefive
>>>> import threefive
>>>> threefive.version()
'2.3.49'

On second thought, a charset might be a good idea.
In the Stream class, I need a charset sometimes for mpegts descriptors,
I'm fairly sure it could be used in other places as well.
Maybe a threefive.charset var or something. I can default it to utf-8,
but it could be set otherwise if needed.
What do you think?

Thanks for the quick fix.
Yes, I vote for allowing the user to specify a charset.

Thanks @futzu.
That quick fix is good for us.
I also vote for allowing the user to provide the charset.

Here's what I'm testing now,
this would apply to UPID types:

0x01,0x02(Deprecated),
0x03 (AdID) ,
0x07 (TID),
0x09 (ADI),
0x10 (UUID),
0x11 (SCR),
0x0E (ADS Info),
0x0F (URI)
0xFD (Unknown)

>>> from threefive import Cue,upids
>>> i="/DBKAAAAAAAAAP/wBQb+YtC8/AA0AiZDVUVJAAAD6X/CAAD3W3ACEmJibG5kcHBobkQCAsGDpQIAAAAAAAEKQ1VFSRSAIyowMljRk9c="

>>> upids.charset
'ascii'
>>> cue=Cue(i)
>>> cue.decode()
ascii
True
>>> cue.descriptors[0].segmentation_upid
'bblndpphnD\x02\x02���\x02\x00\x00'

>>> upids.charset="big5"
>>> cue.decode()
big5
True
>>> cue.descriptors[0].segmentation_upid
'bblndpphnD\x02\x02���\x02\x00\x00'

>>> upids.charset="utf16"
>>> cue.decode()
utf16
True
>>> cue.descriptors[0].segmentation_upid
'扢湬灤桰䑮Ȃ菁ʥ\x00'

threefive.upids.charset is active as of threefive-2.3.51.


AttributeError: module 'pinocchio' has no attribute 'buildModelFromUrdf'

Is 'buildModelFromUrdf' feature deprecated? If so, are there any alternatives I can use?
I am using this code:
import pinocchio as pin
import numpy as np

model = pin.buildModelFromUrdf('~/home/rshailly/Documents/ur10_urdf.urdf')
data = model.createData()

# Specify joint configuration
q = pin.neutral(model)

# Compute Jacobian at the end-effector
frame_id = model.getFrameId('tool0')  # Adjust as needed
J = pin.computeFrameJacobian(model, data, q, frame_id)

# Compute pseudo-inverse of Jacobian
pseudo_inverse_J = np.linalg.pinv(J)

and I am getting this error:

Hi @RitishShailly , this function should be available. How did you install pinocchio and which version do you have now ?
You could maybe check your conda or pip list or just do in python: import pinocchio; pinocchio.__version__


Update manage-account-members.md

changed step 5 in the 'Add account members' section from5. (Optional) Click Direct add to add the member to your account without requiring an email invitation to 5. (Optional) If the member already has a Cloudflare account, you can click Direct add to add the member to your account without requiring an email invitation.

Hi Team, I believe that's not updated now under the new Domain Scope Roles. Can you check, please?


Nested Function Calls

Hi Team,
Thanks for a great tool.
I got a series of PL/SQL function calls nested within each other like
A->B->C->D->E
But, I can only see A being reported as time consuming. Why, we are not getting details for other nested functions as they are also problematic when it comes to take time.
Thanks,
Owais.

That's not due to pgBadger. pgBadger only analyzes your log files, and PostgreSQL only logs the top query (hence, your A function call).
You need to configure track_functions to pl in your postgresql.conf file, and you'll find what you need in the pg_stat_user_functions system view. But this is out of scope for pgBadger.

Thanks for prompt response.


Sort credits by frequency

Fixes #449
Builds off of the "pass-gltf-copyright" branch because I am testing with credits in the gltf copyright.

Thanks @joseph-kaile!


remove empty vendor in stretchr testify

remove empty vendor in stretchr testify
With the empty vendor in /root/gocode/src/github.com/docker/swarm/vendor/github.com/stretchr/testify, error will occur when building the binary of swarm.
Fixed #2142
Signed-off-by: Sun Hongliang<EMAIL_ADDRESS>

I am getting the same error.

@alexellis
You can do fix by following this pr.

Has anybody tested this? Now I cannot build a Swarm binary via the source code.
I think this makes me a little bit suffering when hacking Swarm.
@nishanttotla @dongluochen @vieux

@allencloud you mean you can't build from source on the branch in this PR? Or do you mean master?

@nishanttotla
No, I cannot build from source code. I describe the issue in #2142.
However, I tried docker build  from Dockerfile, and swarm image can be built seccessfully.
here is the error message from go install .:
root@10-11-1-156:~/gocode/src/github.com/docker/swarm# go install .
vendor/github.com/stretchr/testify/mock/mock.go:12:2: no buildable Go source files in /root/gocode/src/github.com/docker/swarm/vendor/github.com/stretchr/testify/vendor/github.com/stretchr/objx

LGTM

LGTM


3.4.3 serviceResolved not called

I have just tried JmDNS 3.4.3 on a network with airplay and chromecast devices
I have noticed that serviceResolved is not called anymore
On the logcat i see those kind of warnings:
02-23 09:54:42.205 1764-1915/my.package W/j.j.i.DNSIncoming: Could not find record class. domain: local. type: TYPE_IGNORE index 0
                                                                              dns[response,<IP_ADDRESS>:5353, length=125, id=0x0, flags=0x8400:r:aa, answers=1
                                                                              answers:
                                                                              	[Service@172513216 type: TYPE_SRV index 33, class: CLASS_IN index 1-unique, name: Chromecast5471._googlecast._tcp.local. ttl: '119/120' server: 'Chromecast5471:8009']]
                                                                              	answer:        [Service@172513216 type: TYPE_SRV index 33, class: CLASS_IN index 1-unique, name: Chromecast5471._googlecast._tcp.local. ttl: '119/120' server: 'Chromecast5471:8009']
                                                                                0:<PHONE_NUMBER>000001 000000010e436872 6f6d656361737435 3437310b5f676f6f     ........ .....Chr omecast5 471._goo
                                                                                20: 676c656361737404 5f746370056c6f63 616c000021800100 000078001c000000     glecast. _tcp.loc al..!... ..x.....
                                                                                40: 001f490e4368726f 6d65636173743534 3731056c6f63616c 000e4368726f6d65     ..I.Chro mecast54 71.local ..Chrome
                                                                                60:<PHONE_NUMBER>343731 056c6f63616c0000<PHONE_NUMBER>007800 04c0a80126           cast5471 .local.. ......x. ....&
02-23 09:54:44.956 1764-1915/my.package W/j.j.i.c.DNSRecordType: Could not find record type for index: 49220

Was it any different with 3.4.2?

I have just changed the version number on the build.grade (with the package rename)
If i downgrade back to 3.4.2 it works

@kaikreuzer i think that i have found the offending change
I have done some tests and it seems that is all related to the change of USE_DOMAIN_NAME_FORMAT_FOR_SRV_TARGET on the commit of sl4fj https://github.com/jmdns/jmdns/commit/5a8489186254d49141315195644361731007d25c#diff-280355df0b37afdb4aa6561c06374fc1L33

Argh, sh$%&t, this should indeed never have been changed in there!
Ok, I suggest to revoke 3.4.3, fix it and re-release it.

Could you please have a try with https://bintray.com/artifact/download/jmdns/mvn/org/jmdns/jmdns/3.4.3/jmdns-3.4.3.jar and let me know if it fixes the problem?

Sorry, there is still something wrong about the jar... Let me try again :-/

ok, meanwhile i have found another issue regarding serviceResolved (it happens on 3.4.2 too) i will open a separate issue for it

Ok, I have revoked the 3.4.3 release and I will actually better do a 3.5.0 instead as a replacement (feeling that the version fits better to the fact that a new dependency has been introduced).

i will open a separate issue for it

If you could directly come up with a fix and a PR, I would also be fine to integrate it directly in 3.5.0.

Fixed with https://github.com/jmdns/jmdns/commit/cbc80c83fab69008debcad875323804f9ce3aaaf


[CSM-352] Rewrite mkCTxs without isRedemptionTx flag

This is crucial fix since it fixes CSM-392 as well

I broke redeeming with this commit, will fix in several hours

I tested following scenarios

Redeem
Send money withing account (w/ and w/o remainder) - this was required because CTx forming logic has changed
Send money to foreign wallet (w/ and w/o remainder)
I checked result of newPayment and redeemAda endpoints, and after each transaction I checked results of getAccounts (all) and getTxHistory (for each related account) as well.

What differs:

Now transaction from account A to A is displayed with amount equal to sum of balances of input addresses upon the moment transaction was performed (previously we displayed 0)

Everything else works as before


Add version: JetBrains.Kotlin.Compiler version 1.5.10

Pull request has been created with Komac v1.9.1 :rocket:
Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/116208&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Address comments for DoubleVectorOpUtils

Address the comments in https://github.com/apache/pinot/pull/8724#discussion_r882168859
@Jackie-Jiang

Codecov Report

Merging #8777 (c2f42d5) into master (36298f0) will increase coverage by 40.79%.
The diff coverage is 21.21%.

:exclamation: Current head c2f42d5 differs from pull request most recent head 2ffaf13. Consider uploading reports for the commit 2ffaf13 to get more accurate results

@@              Coverage Diff              @@
##             master    #8777       +/-   ##
=============================================
+ Coverage     25.36%   66.16%   +40.79%
- Complexity       45     4432     +4387
=============================================
  Files          1724     1289      -435
  Lines         90820    65632    -25188
  Branches      13592    10290     -3302
=============================================
+ Hits          23037    43424    +20387
+ Misses        65558    19106    -46452
- Partials       2225     3102      +877

Flag
Coverage Δ

integration2
?

unittests1
66.16% <21.21%> (?)

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

Impacted Files
Coverage Δ

...y/aggregation/function/MaxAggregationFunction.java
63.93% <0.00%> (-19.71%)
:arrow_down:

...y/aggregation/function/MinAggregationFunction.java
63.93% <0.00%> (-19.71%)
:arrow_down:

...y/aggregation/function/SumAggregationFunction.java
68.00% <0.00%> (-27.46%)
:arrow_down:

...local/indexsegment/mutable/MutableSegmentImpl.java
59.28% <0.00%> (+59.28%)
:arrow_up:

...ime/impl/forward/VarByteSVMutableForwardIndex.java
66.66% <0.00%> (+66.66%)
:arrow_up:

...ocal/segment/readers/PinotSegmentColumnReader.java
70.21% <0.00%> (+70.21%)
:arrow_up:

...segment/spi/index/mutable/MutableForwardIndex.java
4.44% <0.00%> (+4.44%)
:arrow_up:

...e/query/aggregation/utils/DoubleVectorOpUtils.java
80.00% <87.50%> (+80.00%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 1501 more

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 36298f0...2ffaf13. Read the comment docs.


fix(masthead): add z-index value to address overlapping issue seen within the Drupal

Related Ticket(s)
Issue: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/7675
HC JIRA Ticket: https://jsw.ibm.com/browse/HC-2325
Description

This PR adds in a z-index of 900 to <dds-masthead-container> and <dds-cloud-masthead-container> components to eliminate overlapping stacking context issues seen within Drupal.
See https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/7675 for more context.

Changelog
Changed

Adjust the z-index value to 900 instead of 999 because Drupal has an administrative toolbar that's currently set to 901. See screenshot.

https://user-images.githubusercontent.com/1815714/142263164-bf0de7b8-b86c-4adb-95ff-8976f0c81df3.mov

@annawen1 @jeffchew Can I get your review on this PR? Let me know if there's anything else that we need to account for/adjust within this PR.


build: FullDebugUnitTest fails

gradle build fails with
    [...]
    :app:processFullDebugUnitTestJavaRes UP-TO-DATE
    :app:compileFullDebugUnitTestSources
    :app:mockableAndroidJar UP-TO-DATE
    :app:assembleFullDebugUnitTest
    :app:testFullDebugUnitTest

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryArtistsSongWithArtistWithoutAlbumTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAllSongsSongWithMultipleArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAllSongsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAlbumSongsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryArtistSongsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAllSongsSongWithAlbumWithoutArtistTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryMultidiscAlbumSongsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAllSongsSongWithArtistWithoutAlbumTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryThirdArtistSongWithMultipleArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > querySecondArtistSongWithMultipleArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryFirstArtistSongWithMultipleArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.SongsTest > queryAllSongsSongWithAlbumAndArtistTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAlbumsForArtistTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAlbumTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAlbumsForArtistWithVariousArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAlbumWithMultipleArtists FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAlbumWithoutArtist FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.AlbumsTest > queryAllAlbumsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.GenresTest > queryAllGenresTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.GenresTest > queryAlbumsForGenreTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.ArtistsTest > queryAllArtistsTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    org.xbmc.kore.provider.mediaprovider.ArtistsTest > queryArtistTest FAILED
        org.apache.tools.ant.BuildException
            Caused by: org.apache.maven.artifact.resolver.MultipleArtifactsNotFoundException

    23 tests completed, 22 failed, 1 skipped
    :app:testFullDebugUnitTest FAILED

    FAILURE: Build failed with an exception.

    * What went wrong:
    Execution failed for task ':app:testFullDebugUnitTest'.
    > There were failing tests. See the report at: file:///~/git/Kore/app/build/reports/tests/fullDebug/index.html

    * Try:
    Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

    BUILD FAILED

    Total time: 4 mins 2.121 secs

The release apk works fine though. I'm on Ubuntu 16.04 with latest AndroidStudio, SDK and support libraries installed.

The error you're getting should not occur. The :app:testFullDebugUnitTest should run fine.
Try running a clean before running testFullDebugUnitTest.
Note that running build does not work because we use different flavors to make sure test files and permissions do not get included into release builds as well. The build task will run testFullReleaseUnitTest which will run the local tests without the required assets. These assets are included in the testFullDebugUnitTest task.

Still the same after git pull followed by git reset --hard followed by git clean -dfx followed by ANDROID_HOME=~/Android/Sdk/ /opt/android-studio/gradle/gradle-2.14.1/bin/gradle testFullDebugUnitTest. Still, the assembleRelease target works fine.

So it's still failing at testFullDebugUnitTest? Strange, I just ran the build on a clean clone on the master branch and it succeeds at testFullDebugUnitTest  and fails at testFullReleaseUnitTest, as it should.

Did you manage to get this fixed?

Closing this. Please reopen if the problem persists and you can provide more details.


ci: Fix GHPages deploment

Not sure what changed, but it wasn't working. This more closely replicates the example GH pages example workflow. It works on web branch.

FYI I just force pushed because the last release is so recent


Leading SPACE in EXCEL header/footer when using font-size

Hi,
I've tried to create an excel footer with a different font-size. The result was an unwanted leading space in footer/header.
Coding:
*   Footer Left
    ls_footer-left_value = ls_value.
    ls_footer-left_font-size = 8.
    ls_footer-left_font-scheme = zcl_excel_style_font=>c_scheme_none.
    ls_footer-left_font-name = zcl_excel_style_font=>c_name_arial.

I've adjusted the following coding to solve the problem:

Maybe you can fix it with the next release.
Thanks in advance.
Marcel

Hi Marcel,
you can contribute this fix by forking the project, do the correction and then send a pull request.
Best regards
Gregor

Should be fixed with your correction. Thank you so much.


Could you provide (RESTful)API for other GUI programs to call

This would be quite cool actually. We could expose the same commands as the command line. Is there anything else that should go into the HTTP endpoints?

daemon mode
api

list devices
select/connect device
disconnect/release device
load(and play)
pause/unpause
seek: +/-
volume: get/set
mute/unmute
stop
status/event history

Those seem like a good starting point. Let me take a look at turning these into HTTP API endpoints.

 I will try later.

It's working great, but the headers return the wrong content-type, messing up tools like https://github.com/ducaale/ht that will not trigger json formatting in the response.

It's working great, but the headers return the wrong content-type, messing up tools like https://github.com/ducaale/ht that will not trigger json formatting in the response.

Apologies, that was an oversight. I will push a fix for this.

Apologies, that was an oversight. I will push a fix for this.

Pushed a fix for this to master, will add this to a new release shortly.

Pushed a fix for this to master, will add this to a new release shortly.

Closing this as I believe it is done. Feel free to reopen.

Closing this as I believe it is done. Feel free to reopen.


fix(ci): fix test failing on new default

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

Comparison is base (d30b35f) 33.51% compared to head (4573794) 33.51%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1058   +/-   ##
=======================================
  Coverage   33.51%   33.51%
=======================================
  Files         101      101
  Lines       12457    12457
=======================================
  Hits         4175     4175
  Misses       7875     7875
  Partials      407      407

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


[incubator/solr] Zookeeper status : Unable to mount volumes for pods

After installing, when getting zookeeper status solr/admin/zookeeper/status I see the below error:
"msg":"Index 1 out of bounds for length 1",
"trace":"java.lang.ArrayIndexOutOfBoundsException: Index 1 out of bounds for length 1\n\tat org.apache.solr.handler.admin.ZookeeperStatusHandler.monitorZookeeper(ZookeeperStatusHandler.java:189)\n\tat org.apache.solr.handler.admin.ZookeeperStatusHandler.getZkStatus(ZookeeperStatusHandler.java:99)\n\tat org.apache.solr.handler.admin.ZookeeperStatusHandler.handleRequestBody(ZookeeperStatusHandle`

I used the example command:
helm install --name solr  --set image.tag=7.7.2,javaMem="-Xms1g -Xmx1g",logLevel=INFO,replicaCount=2,livenessProbe.initialDelaySeconds=420,exporter.readinessProbe.periodSeconds=30 incubator/solr
Helm v "v2.16.5"
kubectl "v1.18.1"
Incubater/solr chart
When I run kubectl describe po solr-zookeeper-0 , I see this error:
Unable to mount volumes for pod "solr-zookeeper-0_xxxx": timeout expired waiting for volumes to attach or mount for pod "<namespace>"/"solr-zookeeper-0". list of unmounted volumes=[data]. list of unattached volumes=[data config default-token-swvgs]
Although the pods remain running, zookeeper doesn't look happy.

After waiting a while this error has resolved itself. Zookeeper status is nto accessible via SOLR admin page however, but if I go directly on the pods it is fine


New package: Apache.Lounge version <IP_ADDRESS>904

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Resolve #181427
[x] Resolve #91846

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

[!NOTE]
Further investigation is needed. It seems no response when running httpd.exe via symbolic links. But while running ab.exe, usage is printed as expected. Other executables haven't tested on my device yet.

Note
Further investigation is needed. It seems no response when running httpd.exe via symbolic links. But while running ab.exe, usage is printed as expected. Other executables haven't tested on my device yet.

This is because of the zip-binary

[!CAUTION]
Close with reason: https://github.com/microsoft/winget-pkgs/labels/Blocking-Issue https://github.com/microsoft/winget-pkgs/labels/zip-binary


wrong placement of MathML in SVG with Safari

Issue Summary
On Safari, certain MathML-in-SVG is rendered in the top left corner of the image regardless of its intended position. This is definitely a Safari bug (reasonable guess is https://bugs.webkit.org/show_bug.cgi?id=23113), but I am hoping MathJax can include a workaround for the time being.
For instance: resetting the position and will-change CSS properties of <mjx-container> resolves the problem, although it probably messes up the alignment if the MathML is not the only content within <foreignObject>.
Steps to Reproduce:
Add MathML in the scope of a transform. For example:
<svg width="400" heigth="100" viewbox="0 0 400 100">
  <rect x="0" y="0" width="400" height="100" stroke="black" fill="none" />
  <foreignObject transform="matrix(1 0 0 1 300 0)" width="1" height="1" overflow="visible"><math>
      <mi>a</mi>
    </math></foreignObject>
</svg>

renders as

instead of

Technical details:

MathJax Version: 3.2.2
Client OS: Mac OS X 14.7
Browser: Safari 18.0.1

I am loading MathJax via
<script id="MathJax-script" async<EMAIL_ADDRESS>

I believe you are correct about the Safari bug that underlies this.
Version 4 removes the will-change CSS, and the position is only used when assistive MathML is being generated (off by default in v4).
For v3, you can use
<style>
foreignObject > mjx-container {
  position: unset !important;
  will-change: unset !important;
}
</style>

to reset those CSS styles, which should allow MathJax output to work in your SVGs.

Thanks! That's reassuring.

only used when assistive MathML is being generated

Oh, so that's the actual reason for position: relative. I see now what happens without it (the assistive MathML has position: absolute and goes to the top of the outer container, typically the page). Since foreignObject acts as container, the assistive MathML stays inside, which is fine for the typical scenario (e.g. SVGs generated from TikZ).
Is position: absolute on the assistive MathML really necessary? Or is it just there to prevent an extra invisible pixel? It seems to me I can get the same visual result with -1px margins (and related adjustments, e.g. display: inline-block). In fact, I might just set up my CSS that way within foreignObject, and hope it doesn't blow up in my face later on.

Is position: absolute on the assistive MathML really necessary?

It depends on one's goal.  The reason this is used is so that the assistive MathML will be at there same location as the MathJax typeset math.  Because many screen readers highlight the text being read (this helps some readers who are not visually impaired but have other disabilities follow the reading as it progresses), we want that highlighting to display at the location of the math when it is read.  The easiest way to do that is by using an absolute position for the MathML container within a relatively positioned typeset math container.
It would be possible to do this in other ways, but they are more fragile, such as using a negative margin-left, a vertical-align value, and a width and height on the MathML container.  This might might also allow a breakpoint between the math any any following punctuation, which v4 currently has worked hard to prevent (I would have to check whether this CSS arrangement would prevent that or not).  One could make a configuration that modified the assistive MathML output to work that way instead, though I haven't done that to test it.

Thanks for the detailed answer.

This might also allow a breakpoint between the math any following punctuation

I hadn't thought of that!
At any rate: based on what you said, I will configure my MathJax so that for WebKit, in foreignObject, it uses negative margin-inline-start and similar for the assistive MathML. Screen reader outlines will look like small dots, but that seems like a reasonable price to pay for having the content in the right place. Bonus points if I can keep the punctuation at bay. I'll post my code here in case other people hit the same issue.

OK.  Good luck, and let us know how it works out.

This is the current BookML workaround:
@media (-webkit-transform-2d) {
  foreignObject>mjx-container[jax="CHTML"] {
    position: initial !important;
    will-change: initial;
    &>mjx-assistive-mml {
      width: 1px !important;
      height: 1px !important;
      position: initial !important;
      margin-inline-start: -1px !important;
      vertical-align: 1px !important;
    }
  }
}

This renders correctly even in WebKit browsers.

Thanks for the details!

Perhaps this solution will help you figure out the problem click


fixes #2046 for custom themed buttons not working with pressed/focused action.

+Fixes #2046
+Added DRAW_FOCUSED and related necessary changes.
+Some formatting clean-ups.
+Tested working on x11 x84-64 Linux Mint 17 Qiana 14.04.2 LTS.
AMD R280 Open source driver

There should not be a need for DRAW_FOCUSED, use has_focus() instead. Many button types just draw focus as an overlay but need to know the current state of the button.

I think the problem was that has_focused() with draw call was placed at the end of the Button::_notification, drawing over top of everything. I can remove DRAW_FOCUSED, draw as overlay as requested. (calling draw twice I assume). By overlay, I take it that people could draw translucent focused texture. Would you accept this?

yes, the problem is that the focus state is parallel to the other states
(pressed, hover), it is also drawn over the existing button, most commonly
transparent. I'm not exactly sure what else you want to change though
On Mon, Jun 8, 2015 at 10:18 AM, Kevin Choi<EMAIL_ADDRESS>wrote:

I think the problem was that has_focused() with draw call was placed at
the end of the Button::_notification, drawing over top of everything. I can
remove DRAW_FOCUSED, draw as overlay as requested. (calling draw twice I
assume). By overlay, I take it that people could draw translucent focused
texture. Would you accept this?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/pull/2047#issuecomment-109993161.

--
OkamStudio

Pretty much just moving focus draw call right after other stylebox draw so focus style doesn't clobber everything if it's opaque.

ok, let me check if this works and doesn't cause issues and will push a fix
On Mon, Jun 8, 2015 at 10:26 AM, Kevin Choi<EMAIL_ADDRESS>wrote:

Pretty much just moving focus draw call right after other stylebox draw so
focus style doesn't clobber everything if it's opaque.
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/pull/2047#issuecomment-109995095.


[lightning_utilities] Missed py.typed file causes type hinting errors

Bug description
Hello PyTorch team 👋  Thank you for maintaining this amazing package, the latest releases have been excellent and a joy to use 🙇♂
When upgrading to the latest version of pytorch-lightning (1.8) I've stumbled upon a type checking problem with rank_zero_only. It uncovered a bug in the type checker pyright but it is also related to a missed py.typed file in the lightning_utilities package.
Would you please add this file to lightning_utilities so that its types may be used by other libraries?
How to reproduce the bug
See https://github.com/microsoft/pyright/issues/4253 for context
Error messages and logs
No response
Environment

#- Lightning Component (e.g. Trainer, LightningModule, LightningApp, LightningWork, LightningFlow):
#- PyTorch Lightning Version (e.g., 1.5.0): 1.8
#- Lightning App Version (e.g., 0.5.2): n/a
#- PyTorch Version (e.g., 1.10): n/a
#- Python version (e.g., 3.9): n/a
#- OS (e.g., Linux): n/a
#- CUDA/cuDNN version: n/a
#- GPU models and configuration: n/a
#- How you installed Lightning(`conda`, `pip`, source): n/a
#- Running environment of LightningApp (e.g. local, cloud): n/a

More info
No response

@AlexanderVanEck great finding, would you mind opening a PR on https://github.com/Lightning-AI/utilities to fix it? :rabbit:


use boost-url lib 1.81

when available in appveyor: now 1.77, MacOS is OK with 1.81
1.82 is now in the image, not yet ok
and still merge problem

Done, merged with merge-url branch


[HxMessenger] OnMessageClosed #288

I considered adding the OnMessageClosed event to the IHxMessengerService, however, I concluded that it would be quite chaotic to intermingle our internal logic for adding messages with events users may subscribe to and that's why I decided to simply add an EventCallback parameter to the HxMessenger component.

Rejecting the PR for the reasons described above.


feat: fix athena partitions limit

Description
Resolves: https://github.com/dbt-athena/dbt-athena/issues/87
Models used to test
For table materialization (iceberg):
{{
  config(
    schema='sandbox',
    materialized='table',
    partitioned_by=['DAY(date_column)', 'doy'],
    table_type='iceberg'
  )
}}

SELECT
    CAST(date_column AS DATE) as date_column,
    doy(date_column) as doy,
    random() as rnd
FROM (
    VALUES (
        SEQUENCE(FROM_ISO8601_DATE('2023-01-01'), FROM_ISO8601_DATE('2023-07-24'), INTERVAL '1' DAY)
    )
) AS t1(date_array)
CROSS JOIN UNNEST(date_array) AS t2(date_column)

For table materialization (hive):
{{
  config(
    schema='sandbox',
    materialized='table',
    partitioned_by=['date_column', 'doy'],
    table_type='hive'
  )
}}

SELECT
    random() as rnd,
    CAST(date_column AS DATE) as date_column,
    doy(date_column) as doy
FROM (
    VALUES (
        SEQUENCE(FROM_ISO8601_DATE('2020-01-01'), FROM_ISO8601_DATE('2023-07-24'), INTERVAL '1' DAY)
    )
) AS t1(date_array)
CROSS JOIN UNNEST(date_array) AS t2(date_column)

For incremental materialization:
{{
  config(
    schema='sandbox',
    materialized='incremental',
    incremental_strategy='merge',
    partitioned_by=['DAY(date_column)', 'doy'],
    table_type='iceberg',
    unique_key=['doy']
  )
}}

select * from {{ ref('mat_table') }}

Checklist

[x] You followed contributing section
[x] You kept your Pull Request small and focused on a single feature or bug fix.
[x] You added unit testing when necessary
[x] You added functional testing when necessary

How is handled when we want to redo a certain batch ? Is there a way to delete overlapping partitions as it was done in the insert_by_period made by Jesse ?

How is handled when we want to redo a certain batch ? Is there a way to delete overlapping partitions as it was done in the insert_by_period made by Jesse ? Especially I don't see examples with is_incremental condition.

@Jrmyy all I've changed it the way how the tmp table is created and how the data from it is inserted into target relation. All other logic including is_incremental condition and delete overlapping partitions stay as they were before. I'm now testing these changes on my own project (only iceberg tables to be honest) and everything works OK. But of course, I would appreciate help with testing on different models to catch possible edge cases.

@svdimchenko I did some more tests - let's fix the conflicts and merge ok? - I'm looking forward to having this merge before we release 1.6 :)

hey does this work for (non-iceberg) tables that are bucketed in addition to partitioned?

AFIK for bucketing (hive and not hive) you won't get any partition limitation. Did you face partition limitations with bucketing?
Bucketing allows you to write objects with the same hash value in the same object.


implement removal of styles

Fixes #26

This is basically untested .. I don't think I can trigger this with the TodoMVC app, right?
Either way, I'm pretty sure this is correct (and it's better than a TODO). See the poor MDN docs here or a slightly better / weird docpage here. The only downside to this approach is a strict requirement on IE 9+, but I don't think anyone (myself included) will object to this ;)

This is basically untested

Perhaps it's useful to add a "// TODO: Verify this works, then remove this comment." comment then? That way, if someone runs into an issue here, they'll know to report it immediately instead of wasting time thinking they're doing something wrong.
The logic seems sound to me, so LGTM.

Perhaps it's useful to add a "// TODO: Verify this works, then remove this comment." comment then? That way, if someone runs into an issue here, they'll know to report it immediately instead of wasting time thinking they're doing something wrong.

Edit: To force it to be more actionable and help the above TODO get resolved faster, perhaps it can be an log.Println("TODO: Verify this works as expected, then remove this log statement.") instead. People who run into the message will be motivated to resolve the TODO since it'll litter their stdout. :) This is inspired by goxjs/gl#15 (comment).

Honestly.. this is very off-putting to me. I don't like TODO's in code unless they are purely for markers (if real actionable items) and littering stdout doesn't seem very nice to me either. I'm coming from the stance that this code is most likely correct and a TODO or log line is probably not worth the trouble of just removing it later. If this code doesn't work, anyone can easily open an issue and we'll know right away just as easily (if you remove a style, it should be removed from the browser DOM, just as you'd expect).
I personally feel comfortable committing the code as-is right now (i.e. find it unlikely that it would be incorrect), but if we don't want to go based on feelings I'd rather not merge the change at all (until it can be tested by someone, I mean, rather than littering the code).

I'm coming from the stance that this code is most likely correct and a TODO or log line is probably not worth the trouble of just removing it later.
...
I personally feel comfortable committing the code as-is right now (i.e. find it unlikely that it would be incorrect)

In that case, please don't use my suggestion, since it's not helpful. My LGTM still stands.

LGTM

(just got around to checking back on this)

In that case, please don't use my suggestion (I used the word "perhaps" to indicate it's optional), since it's not beneficial. My LGTM still stands.

@shurcooL hope my comment came across as more of an explanation/justification for me wanting to do it a different way, rather than coming across as rude / or uninterested. If it was the latter, I apologize :smile:

hope my comment came across as more of an explanation/justification for me wanting to do it a different way

Absolutely. It was a good argument, and it convinced me.
When I said:

In that case

What I meant by that was:

I agree with your arguments, and now believe that your suggestion is better, so disregard my previous suggestion since I now think it's actually worse.

coming across as rude / or uninterested

Not at all! :+1: We're both striving to find the best solution, and I think we came to a consensus on what that was. :)

Also, I'm glad that my suggestion of adding a TODO led you @slimsag to think it over and demonstrate confidence in the current code not needing a TODO, which led to better final code. Originally, you weren't as confident about it. :P


Go to specific slide

Is it possible to get to a specific slide to active/first on click? So if I have a button that always goes to a specific index of the slides when clicking?

Yes you can do so using carousel component reference
<template>
  <Carousel ref="myCarousel">
  ...
  </Carousel>
<template>

<script>
{
  ...
  handleButtonClick() {
    this.$refs.myCarousel.slideTo(1);
  }
 }
</script>


Concurrent lookups of instances on a class sometimes result in NullReferenceException

Prerequisites
The issue tracker is used to report bugs and request new features, NOT to ask questions.
Questions should be posted to the users mailing list which can be accessed at
https://ironpython.groups.io/g/users.

[Y] Are you running the latest version?
[Y] Are you reporting to the correct repository?
[Y] Did you perform a cursory search?

Description
When object instances are defined on a class, concurrent lookups of that instance sometimes produce a NullReferenceException in PythonTypeUserDescriptorSlot.GetValue.
Steps to Reproduce
Run the following in C# Interactive. It eventually fails with a NullReferenceException.
#r "Microsoft.Scripting"
#r "IronPython"

using System;
using IronPython.Hosting;
using IronPython.Runtime;
using IronPython;
using Microsoft.Scripting.Hosting;
using Microsoft.Scripting;

var engine = Python.CreateEngine();
var scope = engine.CreateScope();
engine.SetSearchPaths(new[] { @"C:\Program Files\IronPython 2.7\Lib\" });

string script = @"
import threading
from multiprocessing.pool import ThreadPool

class Member(object):
    alpha = 0
    beta = 1

class Thing(object):
    member = Member
    member_instance = Member()

test_thing = Thing()

def query(trigger_event):
    trigger_event.wait()
    return [test_thing.member_instance for _ in range(10)][-1]

def single_test(number_of_threads):
    trigger = threading.Event()

    pool = ThreadPool(number_of_threads)
    results = [pool.apply_async(query, [trigger]) for i in range(number_of_threads)]
    trigger.set()
    [r.get() for r in results]
    pool.close()
    pool.join()

# Including the next line prevents NullReferenceExceptions in the threaded queries.
# print test_thing.member_instance

single_test(16)
";

while (true) {
    engine.Execute(script);
}

Expected behavior: Concurrent lookups each return the object instance, always.
Actual behavior: Concurrent lookups occasionally produce a NullReferenceException.
System.NullReferenceException: Object reference not set to an instance of an object.
  + Microsoft.Scripting.Interpreter.ThrowInstruction.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.Interpreter.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.LightLambda.Run3<T0, T1, T2, TRet>(T0, T1, T2)
  + System.Dynamic.UpdateDelegates.UpdateAndExecute3<T0, T1, T2, TRet>(System.Runtime.CompilerServices.CallSite, T0, T1, T2)
  + IronPython.Runtime.Method.MethodBinding.SelfTarget(System.Runtime.CompilerServices.CallSite, IronPython.Runtime.CodeContext, object)
  + System.Dynamic.UpdateDelegates.UpdateAndExecute2<T0, T1, TRet>(System.Runtime.CompilerServices.CallSite, T0, T1)
  + IronPython.Compiler.Ast.CallExpression.Invoke0Instruction.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.Interpreter.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.LightLambda.Run2<T0, T1, TRet>(T0, T1)
  + System.Dynamic.UpdateDelegates.UpdateAndExecute3<T0, T1, T2, TRet>(System.Runtime.CompilerServices.CallSite, T0, T1, T2)
  + IronPython.Compiler.Ast.CallExpression.Invoke1Instruction.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.Interpreter.Run(Microsoft.Scripting.Interpreter.InterpretedFrame)
  + Microsoft.Scripting.Interpreter.LightLambda.Run1<T0, TRet>(T0)
  + IronPython.Compiler.RuntimeScriptCode.InvokeTarget(Microsoft.Scripting.Runtime.Scope)
  + Microsoft.Scripting.Hosting.ScriptSource.Execute()

Versions
2.7.11 (<IP_ADDRESS>0)
[.NETFramework,Version=v4.5 on .NET Framework 4.8.4300.0 (64-bit)]

A more useful stacktrace from an application where we first hit this:
   at IronPython.Runtime.Types.PythonTypeUserDescriptorSlot.GetValue(CodeContext context, Object instance, PythonType owner)
   at IronPython.Runtime.Types.GetMemberDelegates.UserSlotDict(CallSite site, Object self, CodeContext context)
   at System.Dynamic.UpdateDelegates.UpdateAndExecute2[T0,T1,TRet](CallSite site, T0 arg0, T1 arg1)
   at IronPython.Compiler.Ast.DynamicGetMemberExpression.GetMemberInstruction.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.LightLambda.Run5[T0,T1,T2,T3,T4,TRet](T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4)
   at IronPython.Compiler.PythonCallTargets.OriginalCallTarget4(PythonFunction function, Object arg0, Object arg1, Object arg2, Object arg3)
   at System.Dynamic.UpdateDelegates.UpdateAndExecute6[T0,T1,T2,T3,T4,T5,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5)
   at IronPython.Runtime.Method.MethodBinding`3.SelfTarget(CallSite site, CodeContext context, Object target, T0 arg0, T1 arg1, T2 arg2)
   at System.Dynamic.UpdateDelegates.UpdateAndExecute5[T0,T1,T2,T3,T4,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4)
   at IronPython.Compiler.Ast.CallExpression.Invoke3Instruction.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.LightLambda.Run4[T0,T1,T2,T3,TRet](T0 arg0, T1 arg1, T2 arg2, T3 arg3)
   at IronPython.Compiler.PythonCallTargets.OriginalCallTarget3(PythonFunction function, Object arg0, Object arg1, Object arg2)
   at Microsoft.Scripting.Interpreter.FuncCallInstruction`6.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.Interpreter.Run(InterpretedFrame frame)
   at Microsoft.Scripting.Interpreter.LightLambda.Run6[T0,T1,T2,T3,T4,T5,TRet](T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4, T5 arg5)
   at System.Dynamic.UpdateDelegates.UpdateAndExecute5[T0,T1,T2,T3,T4,TRet](CallSite site, T0 arg0, T1 arg1, T2 arg2, T3 arg3, T4 arg4)
   at CallSite.Target(Closure , CallSite , CodeContext , Object , Object , Object )
   at _thread_operation$718(Closure , PythonFunction , Object , Object , Object ).

@mickp Can you test if https://github.com/IronLanguages/ironpython2/pull/772 resolves your issue?

@slozier Thanks for the quick response.
#772:

does not resolve the issue on .Net Framework 4.8;
does appear to fix it on .Net Core 3.1.

@slozier Apologies - I'm still seeing failures on 4.8. I don't see a DLL build for 4.8, though, so it could be that. It  is definitely resolved in 3.1.

Rebuilt and double checked the paths for the references. It's using the newly-build IronPython.dll for Framework 4.8, and still failing, although the failure rate seems to be lower.
A colleague has done his own tests and sees the same: resolved in Core 3.1, not in FW.

@mickp Hmm, I'm not getting a failure with the sample program here. Are you sure it's not picking up the old IronPython assembly from the GAC? I believe the msi installer has the option to install to the GAC on by default.

I was helping @mickp troubleshoot this issue.
We cloned the repo and added tests project in IronPython's solution. So, no, we were not using GAC assemblies but project references.
I hope this helps.

Just to make sure, I cleared out all IronPython-related DLLs from the GAC. I still see the failures under FW4.8.

Hmm, I haven't been able to reproduce the failure with 4.8 over here, but I guess it was still possible to end up in inconsistent state. Anyway, I pushed an update to the fix.

@slozier - You were absolutely right: it appears there are multiple GAC locations. I dumped debugging output, and saw that the tests are pulling FW4.8 from another folder under %windir%, rather than those set in the project references (which I checked before).
Your fix resolves the issue in both .Net versions.

@mickp Thanks for the update. Which version of the fix did you adopt (so I know which one to merge)?

@slozier I just retested both, now I know my tests are pulling the correct assemblies!
The first commit (8ff8c3db5de9c860163a2dde7ee7967b38a8d14c) resolves the issue in both Framework and Core.


Fixes issue of not propagating events and issue of selected value not a string

This fixes two issues that we have come across:

Selecting a result does not trigger change events for related input values as would be expected
Result values may contain non-string (e.g. in our case integer) values (#11)

We've updated the minified js as well with these fixes.

Does this handle #11? If so, thank you!

yup :smiley:

:+1: Only suggestion, add the plain text Closes #11 to your top comment of this pull request. It will automatically close my issue. :-)

@codenamev Thanks.

@codenamev Is it possible you write some tests proving this behavior? I do not want to update them gem and change my controllers before I know it works.


[DRAFT] More formatters

I'll be adding many more formatters in the next few days. I hope to reach parity with emacs-format-all-the-code. Each one will be a separate commit for easy cherry-picking.
I've read your contributor guide. I'll update the changelog and such when I'm closer to finishing this pull.

Wow, this is phenomenal. Thanks!

Wow, this is phenomenal. Thanks!

I hope you don't mind---I've updated the changelog, merged in master, and included this for the next release. We can always add even more formatters in the future!

Hi, that's somewhat bad timing right now, because I've just been writing an issue about integrating rustfmt. I'm afraid the proposed PR will not work for rustfmt. I'll post the issue in a moment.


Cannot render expression for Meijer G-function

This LaTeX:
        G_{p,q}^{m,n} \left(\begin{matrix}a_1, \dots, a_n & a_{n+1}, \dots, a_p \\
                                        b_1, \dots, b_m & b_{m+1}, \dots, b_q
                          \end{matrix} \middle| z \right).

does not render at https://khan.github.io/KaTeX/.  It works fine in MathJax and regular LaTeX.
The error in the console is
katex.min.js:1 Uncaught Error: KaTeX parse error: Expected & or \\ or \end at position 53: in{matrix}a_1, ̲\dots, a_n & a(…)

However, I can't figure out how to reduce it to an expression that works. It seems it does not like the matrix (I don't know why). If I put the matrix by itself it still fails. Additionally, it doesn't support \middle (even \left(x \middle | y \right) fails with katex.min.js:1 Uncaught Error: KaTeX parse error: Expected '\right', got '\middle' at position 29: \left(x \middle̲ | y \right)}(…)).

Looks like there's two issues, which can both be worked around for now:

We don't support \dots: it depends on observing the environment that it's in, so it's non-trivial to implement correctly. This can be replaced with \ldots, since that's what \dots would turn into in this case.
We don't support \middle: I didn't even know that \middle existed! You can approximate it by doing something like \bigg| or \Big| in your case.

So, with those two things fixed,
  G_{p,q}^{m,n} \left(\begin{matrix}a_1, \ldots, a_n & a_{n+1}, \ldots, a_p \\
                                        b_1, \ldots, b_m & b_{m+1}, \ldots, b_q
                          \end{matrix} \bigg| z \right).

ends up rendering fine. :)

Hopefully this is helpful! I'll file some issues for the missing features.

I didn't notice it was using \dots instead of \ldots. Perhaps there should be a third issue about the error message.  Undefined control sequence.
l.131 ...{p,q}^{m,n} \left(\begin{matrix}a_1, \dts
                                                  , a_n & a_{n+1}, \dots, a_...
!  ==> Fatal error occurred, no output PDF file produced!

And MathJax renders the whole equation with a red \dts.  The KaTeX error message doesn't indicate what went wrong.
At any rate, \dots doesn't sound like something you should ever use.

Also, it's funny, 5 years ago I reported \middle not working to MathJax, and they gave a similar response.


"Corni" is listed in the drop-down menu

Description:
"Corni" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Corni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Corni, GALAȚI".


DO NOT MERGE: CloudFoundry ruby buildpack update

Do not merge until Buildpack issue gets resolved:
https://github.com/UKGovernmentBEIS/beis-opss/pull/1644#issuecomment-579340652
Description
This updates bundler, node and yarn to the new versions supported by the CloudFoundry Ruby buildpack v1.8.8. This is due to be rolled out on PaaS "on or after 2020-01-28", according to the post on the paas-announce mailing list. After this time, the current build will break.
Important: this cannot be merged until the PaaS team have upgraded their buildpack. If it is merged before then deployments will fail. Review app deploy will also fail until the buildpack has been updated
I've also specified the node and yarn versions in package.json to ensure compatibility.
Checklist:

[ ] Automated checks are passing locally.
[ ] CHANGELOG updated if change is worth telling users about.

General testing

[ ] Test without javascript
[ ] Test on small screen

Accessibility testing

[ ] Works keyboard only
[ ] Tested with one screen reader
[ ] Zoom page to 400% - content still visible
[ ] Disable css - does content make sense and appear in a logical order?
[ ] Passes automated checker (automated in build or manual)

@dependabot-bot rebase


fix: fixed file name color in the File Manager

Description:
In the FileManagerModal component. Made file name text color blue only if the file is selected.
Issues:

Issue #591

UI changes

Checklist:

[x] the pull request name complies with Conventional Commits
[x] the pull request name starts with fix(<scope>):, feat(<scope>):, feature(<scope>):, chore(<scope>):, hotfix(<scope>): or e2e(<scope>):. If contains breaking changes then the pull request name must start with fix(<scope>)!:, feat(<scope>)!:, feature(<scope>)!:, chore(<scope>)!:, hotfix(<scope>)!: or e2e(<scope>)!: where <scope> is name of affected project: chat, chat-e2e, overlay, shared, sandbox-overlay, etc.
[x] the pull request name ends with (Issue #<TICKET_ID>) (comma-separated list of issues)
[x] I confirm that do not share any confidential information like API keys or any other secrets and private URLs

/deploy-review


dispense command going to blowout?

Dispense command goes to blowout to pickup, not bottom - is that supposed to happen?  It ends up picking up more volume than it should

app issue, fixed by connecting Aspirate UI button


ENH: Add node function

GEOS has a GEOSNode function (linear geometry in, geometry out) to create a fully noded linestring (eg add intersection point for intersecting lines), which would be useful to expose.
PostGIS has this as https://postgis.net/docs/ST_Node.html

This is a good way to create fully-noded linework suitable for use as input to ST_Polygonize.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 84.187%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

shapely/linear.py
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
2087

Relevant Lines:
2479

💛  - Coveralls


Feature request: Allow resolve libraryDependencies from other project of a Multi-Project Builds.

I maintain an open source project foo, which uses another project bar. Both of the two projects are used by a private project baz. I have directory layout like this:

root/

project/

Build.scala

foo/

project/

Build.scala

build.sbt

bar/

build.sbt

baz/

build.sbt

I add libraryDependencies in foo/build.sbt:
libraryDependencies += "org.bazGroup" %% "bar" % "1.0"

This is necessary because foo is a solid project. I expect users of foo can checkout foo's source and run sbt, then the baz is automatically downloaded.
But I have a forked version of bar. I expect to aggregate foo, bar and baz together, and frequently modify all the three projects. So I write root/project/Build.scala:
import sbt._
import Keys._

object RootBuild extends Build {

  lazy val foo = Project(id = "foo", base = file("foo")).aggregate(bar)

  lazy val bar = Project(id = "bar", base = file("bar"))

  lazy val baz = Project(id = "baz", base = file("baz")).aggregate(foo, bar)

}

But It does not work. There are two problem:

foo/project/Build.scala is ignored.
"org.bazGroup" %% "bar" % "1.0" does not resolved to my forked version of bar, instead, it uses the one in repository.

The second problem is even more terrible than maven. Maven's Multiple Modules can resolve dependencies between sub-modules.

Can you try dependsOn(bar) instead of aggregate(bar) and see what happens?
Something like:
import sbt._
import Keys._

object RootBuild extends Build {

  lazy val foo = Project(id = "foo", base = file("foo")).dependsOn(bar)
  lazy val bar = Project(id = "bar", base = file("bar"))
  lazy val baz = Project(id = "baz", base = file("baz")).dependsOn(foo, bar)
}

dependsOn can add new build dependency between these projects, but the library dependency to the one in repository is still there.
I write some complex setting code to detect dependencies between current projects.
import sbt._
import Keys._

object RootBuild extends Build {

  private def addBuildDependencyIfMatches(matching: Project) =
    buildDependencies <<= (
      buildDependencies in ThisProject,
      libraryDependencies in ThisProject,
      thisProjectRef in ThisProject,
      name in matching,
      organization in matching,
      thisProjectRef in matching) { (origin, moduleIDs, self,
                                     matchingName, matchingOrganization,
                                     matchingProject) =>
      moduleIDs.find { moduleID =>
        moduleID.name == matchingName &&
        moduleID.organization == matchingOrganization
      } match {
        case Some(moduleID) =>
          origin.addClasspath(
            self,
            ResolvedClasspathDependency(matchingProject,
                                        moduleID.configurations))
        case None =>
          origin
      }
    }

  private def removeLibraryDependencyIfMatch(matching: Project) =
    libraryDependencies <<= (
      libraryDependencies in ThisProject,
      name in matching,
      organization in matching) { (moduleIDs, matchingName,
                                   matchingOrganization) =>
      moduleIDs filterNot { moduleID =>
        moduleID.name == matchingName &&
        moduleID.organization == matchingOrganization
      }
    }

  private lazy val defaultSettings =
    Project.defaultSettings ++
    (projects map addBuildDependencyIfMatches) ++
    (projects map removeLibraryDependencyIfMatch)

  private def newProject(path: String) =
    Project(id = path, base = file(path),
            settings = defaultSettings)

  lazy val foo = newProject("foo")

  lazy val bar = newProject("bar")

  lazy val baz = newProject("baz")

}

But the subpath/project/Build.scala is still ignored. this behavior prevent sbt from being used in complex project which contains multiple hierarchical sub-projects.

@indrajitr note that the subprojects define their own project/, so they have to be treated as external builds.
@Atry I see that the documentation describing this particular feature is not as prominent after the getting started guide rewrite, but it is described on the original page.  See the project references section on:
https://github.com/harrah/xsbt/wiki/Full-Configuration
Please use the mailing list if you have further questions.

@harrah thanks for pointing that out. It went off my head, sorry.


path to tophat

hi, I ran the example pipleine for rna-seq and encounter the following problem.
it looks like it can't find tophat at the path /opt/bcbio/tools/bin/
So, i ls -lrht the /opt/bcbio/tools/bin and it shows tophat is there!
      tophat -> ../Cellar/tophat-binary/2.0.14/bin/tophat

any idea?
btw, tool-path for bcbio tool-dir was /opt/bcbio/tools
and I ran the example pipeline at the directory of somewhere other than /opt/bcbio/tools
[2015-07-05T10:01Z] Aligning lane 1_2015-07-01_seqc with tophat2 aligner
Traceback (most recent call last):
File "/opt/bcbio/tools/bin/bcbio_nextgen.py", line 226, in
main(**kwargs)
File "/opt/bcbio/tools/bin/bcbio_nextgen.py", line 43, in main
run_main(**kwargs)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 37, in run_main
fc_dir, run_info_yaml)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 80, in _run_toplevel
for xs in pipeline.run(config, run_info_yaml, parallel, dirs, samples):
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/main.py", line 297, in run
samples = run_parallel("process_alignment", samples)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 28, in run_parallel
return run_multicore(fn, items, config, parallel=parallel)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/multi.py", line 86, in run_multicore
for data in joblib.Parallel(parallel["num_jobs"])(joblib.delayed(fn)(x) for x in items):
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 657, in call
self.dispatch(function, args, kwargs)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 404, in dispatch
job = ImmediateApply(func, args, kwargs)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/joblib/parallel.py", line 142, in init
self.results = func(*args, **kwargs)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/utils.py", line 49, in wrapper
return apply(f, *args, **kwargs)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/distributed/multitasks.py", line 31, in process_alignment
return sample.process_alignment(*args)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/sample.py", line 100, in process_alignment
data = align_to_sort_bam(fastq1, fastq2, aligner, data)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/alignment.py", line 64, in align_to_sort_bam
:
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/alignment.py", line 64, in align_to_sort_bam
names, align_dir, data)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/pipeline/alignment.py", line 99, in _align_from_fastq
out = align_fn(fastq1, fastq2, align_ref, names, align_dir, data)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/ngsalign/tophat.py", line 260, in align
align_dir, data, names)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/ngsalign/tophat.py", line 110, in tophat_align
if _tophat_major_version(config) == 1:
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/bcbio/ngsalign/tophat.py", line 357, in _tophat_major_version
version_string = str(tophat_runner(version=True)).strip().split()[1]
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/sh.py", line 1021, in call
return RunningCommand(cmd, call_args, stdin, stdout, stderr)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/sh.py", line 486, in init
self.wait()
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/sh.py", line 500, in wait
self.handle_command_exit_code(exit_code)
File "/opt/bcbio/anaconda/lib/python2.7/site-packages/sh.py", line 516, in handle_command_exit_code
raise exc(self.ran, self.process.stdout, self.process.stderr)
sh.ErrorReturnCode_1:
RAN: '/opt/bcbio/tools/bin/tophat --version'
STDOUT:
STDERR:
File "/opt/bcbio/tools/bin/tophat", line 1003
except getopt.error, msg:
^
SyntaxError: invalid syntax

Thanks for the report and sorry about the issue. The main tophat executable is a Python script which was using the default Python (first one in your PATH). It looks like the Python you had was incompatible with the script, which I believe requires 2.x. I pushed fixes which will use bcbio's internal Python to call TopHat instead of the PATH one, which should resolve this issue without needing to change your external environment. You can upgrade with bcbio_nextgen.py upgrade -u development.
Thanks again for the report and hope this gets it running for you.


[Reflection] Support OpaqueExistential in RecordTypeInfo::readExtraInhabitantIndex

Support OpaqueExistential in RecordTypeInfo::readExtraInhabitantIndex().
This missing support was identified when calling projectEnumValue() with a type of Any?. In that case, the projectEnumValue was returning false because readExtraInhabitantIndex returned false. For OpaqueExistentials, readExtraInhabitantIndex should look at the type field (named "metadata"), and use that to determine extra inhabitants.

This is an initial working version for macOS x86-64, in a specific case I was debugging in lldb. Still todo are:

[ ] Verify other platform support (!macOS || !x86-64)
[ ] Verify opaque existential + nested optional (Any?? etc)

This is an initial working version for macOS x86-64, in a specific case I was debugging in lldb. Still todo are:

[ ] Verify other platform support (!macOS || !x86-64)
[ ] Verify opaque existential + nested optional (Any?? etc)

@swift-ci test

@swift-ci test

Nice!  This is simpler than I thought it would be.  (To be honest, I've not looked at this code in a little while...).  I see a couple of other unimplemented cases that we might be able to fill in fairly quickly.
I'll sketch out some test cases this afternoon to help push this forward.

Nice!  This is simpler than I thought it would be.  (To be honest, I've not looked at this code in a little while...).  I see a couple of other unimplemented cases that we might be able to fill in fairly quickly.
I'll sketch out some test cases this afternoon to help push this forward.

test cases sound great, thanks

test cases sound great, thanks

I've changed the base branch to main now, as requested.

I've changed the base branch to main now, as requested.

I'm hoping to merge this soon, and then to cherry pick it to release/5.4. Are you ok with doing the other cases in a separate change because of this? I can add some tests for this case. Based on #30635, is validation-test/Reflection still the place to write reflection tests?

I'm hoping to merge this soon, and then to cherry pick it to release/5.4. Are you ok with doing the other cases in a separate change because of this? I can add some tests for this case. Based on #30635, is validation-test/Reflection still the place to write reflection tests?

@swift-ci test

@swift-ci test

Build failed
Swift Test OS X Platform
Git Sha - 340b85fe03009ec9cfd579af9366b0aafed7983f

Build failed
Swift Test OS X Platform
Git Sha - 340b85fe03009ec9cfd579af9366b0aafed7983f

Ugh.  Looks like I made a lot of obvious dump copy/paste errors in the 32-bit version.  I'll work on that this afternoon...

Ugh.  Looks like I made a lot of obvious dump copy/paste errors in the 32-bit version.  I'll work on that this afternoon...

I'm surprised that Linux passed here.   I'll have to double-check, but I thought Linux only had 4k extra inhabitants in 64-bit pointers, where macOS has 4G.

I'm surprised that Linux passed here.   I'll have to double-check, but I thought Linux only had 4k extra inhabitants in 64-bit pointers, where macOS has 4G.

@swift-ci Please test macOS

@swift-ci Please test macOS

I did some fixes in 7ed7122, and I'll run the tests locally to iterate on them faster.

I did some fixes in 7ed7122, and I'll run the tests locally to iterate on them faster.

I see we're both on it haha

I see we're both on it haha

Build failed
Swift Test OS X Platform
Git Sha - 7f8bebb4676a059a3ca51a0282bd19158b9b0c10

Build failed
Swift Test OS X Platform
Git Sha - 7f8bebb4676a059a3ca51a0282bd19158b9b0c10

lookiung

lookiung

@swift-ci Please test macOS

@swift-ci Please test macOS

This looks right to me.

Thanks!

"Just" need to finish filling out the test suite for it.

@tbkka Are you intending for "filling out the test suite" to be done in this PR or in a follow up?

Thanks for pushing this forward…

thanks for your guidance.

This looks right to me.

Thanks!

"Just" need to finish filling out the test suite for it.

@tbkka Are you intending for "filling out the test suite" to be done in this PR or in a follow up?

Thanks for pushing this forward…

thanks for your guidance.

@swift-ci test Linux

@swift-ci test Linux

Sorry, I probably should have said "get this test suite to pass."  I think the test we have is sufficient coverage, but it may still have some warts.  More tests can follow in a subsequent PR, I think.

Sorry, I probably should have said "get this test suite to pass."  I think the test we have is sufficient coverage, but it may still have some warts.  More tests can follow in a subsequent PR, I think.

Th new test in your wrote pass for the different architectures:
PASS: Swift(macosx-x86_64) :: Reflection/reflect_Optional_Any.swift (678 of 14039)
PASS: Swift(iphonesimulator-i386) :: Reflection/reflect_Optional_Any.swift (679 of 14039)
PASS: Swift(iphonesimulator-x86_64) :: Reflection/reflect_Optional_Any.swift (679 of 14039)
PASS: Swift(watchsimulator-i386) :: Reflection/reflect_Optional_Any.swift (679 of 14039)

Th new test in your wrote pass for the different architectures:
PASS: Swift(macosx-x86_64) :: Reflection/reflect_Optional_Any.swift (678 of 14039)
PASS: Swift(iphonesimulator-i386) :: Reflection/reflect_Optional_Any.swift (679 of 14039)
PASS: Swift(iphonesimulator-x86_64) :: Reflection/reflect_Optional_Any.swift (679 of 14039)
PASS: Swift(watchsimulator-i386) :: Reflection/reflect_Optional_Any.swift (679 of 14039)

Removed swift5-branch-managers from reviewers list; you don't need their review for merging into main

Removed swift5-branch-managers from reviewers list; you don't need their review for merging into main

@rjmccall and @jckarter – Is there any reason why different Linux platforms would have different extra inhabitant counts? This new test isn't passing on my Fedora 33 (x86-64) box and the test seems to assume that the maximum number of extra inhabitants for 64-bit platforms are all the same (which isn't true).

@rjmccall and @jckarter – Is there any reason why different Linux platforms would have different extra inhabitant counts? This new test isn't passing on my Fedora 33 (x86-64) box and the test seems to assume that the maximum number of extra inhabitants for 64-bit platforms are all the same (which isn't true).

Now I'm wondering if the tests are executed on the Linux build bots at all (as opposed to just my "unified build" setup). Let's find out: #35512

Now I'm wondering if the tests are executed on the Linux build bots at all (as opposed to just my "unified build" setup). Let's find out: #35512

Ding. This test isn't triggering on the Linux bots but it is on my cmake build:
$ curl https://ci.swift.org/job/swift-PR-Linux/25240/consoleText 2>/dev/null|grep reflect_Optional_Any
UNSUPPORTED: Swift(linux-x86_64) :: Reflection/reflect_Optional_Any.swift (617 of 14002)

I'm going to commit a workaround that unbreaks unified builds: #35513

Ding. This test isn't triggering on the Linux bots but it is on my cmake build:
$ curl https://ci.swift.org/job/swift-PR-Linux/25240/consoleText 2>/dev/null|grep reflect_Optional_Any
UNSUPPORTED: Swift(linux-x86_64) :: Reflection/reflect_Optional_Any.swift (617 of 14002)

I'm going to commit a workaround that unbreaks unified builds: #35513

@swift-ci Please test Linux

@swift-ci Please test Linux

On Linux, pointers only have 4096 extra inhabitants.  that requires some finessing of the test.  I'll work on that.

On Linux, pointers only have 4096 extra inhabitants.  that requires some finessing of the test.  I'll work on that.

Thanks. Do you think you'll have a fix soon or should I merge the workaround?

Thanks. Do you think you'll have a fix soon or should I merge the workaround?

Ding. This test isn't triggering on the Linux bots but it is on my cmake / unified build:

Just in case this is something obvious — AFAIK the reflection tests are part of the validation tests. Is your bot running these?

Ding. This test isn't triggering on the Linux bots but it is on my cmake / unified build:

Just in case this is something obvious — AFAIK the reflection tests are part of the validation tests. Is your bot running these?

Yes, my bot always runs the validation tests. (They just take a few seconds.) I thought some of the Swift bots run the validation tests. Is that still true?

Yes, my bot always runs the validation tests. (They just take a few seconds.) I thought some of the Swift bots run the validation tests. Is that still true?


Serve static files out of the box

In #120 we definitely will have this, but I think it might be useful to have it in every Bailador app and to have a config parameter that contains the name of the static directory. It should default to static/.

I do understand why makes perfectly sense in development environment.
I am thinking of a good way to introduce it in a way that it is (usually) turned off in production mode, because in a productive environment you would probably want to serve static files from a ngnix or apache.
For example we could load B::Feature::* because of certain settings in the configuration. So this would be more a generc feature then a hard coded "just serve static files" thingy.
any ideas are welcome


Shulker boxes

It would be nice if you could see in the overview what is in the shulker boxes

Also opened an Issue in Plugin Repo: Shulkers and bundles support #7
Added this to the Project MCDealer 1.2.0


Add ConformalMetric

Description
Adds a metric that can masquerade as any other metric that uses only standard metric components. This can be used to deserialize custom metrics when the source code of the custom metric is unavailable.  The conformal metric has limited functionality, but should be usable in a read-only profile.
Changes

Adds ConformalMetric
Use ConformalMetric as last-ditch effort to deserialize a non-standard metric that has not registered as a deserializable metric class

Keep the branch around as an example or possible future resurrection, but don't merge to mainline at this time.


Add search engines

Hey @bruno-
I'm creating another PR as you asked. Can you check to see if this one is alright?

Hey, this is merged. I've done it via the command line because I wanted to do some whitespace updates 0e22a2186 (using tabs instead of spaces for indentation).
I'll add you as the repo collaborator so you can make updates to the README so ppl know we have this new feature.

All right @vasconcelloslf, I just invited you to the team!
Let's just agree on one thing: we'll do all the changes via pull request so each of us can look at the changes being made, point improvements etc.
Also if we're about to make a bigger change, we're better off discussing it via an issue first.
Apart from that, welcome to the team! :tada: :tada: (given that you accept the invite lol)

Hey, sorry for the late reply. Thanks for the invitation, off course I will accept it!
All changes via PR, got it!
=)

Can you put an example in README.md ? I didn't get it to work to :(


1.18 Server Crashes on Boot

Will crash when this mod is in the mods folder
latest.log

This was confusing me, so I did some reading up on that, while you are using the newest Fabric Loader, it is necessary to use the newest installer to install it properly.
The indicator for this is this:

Source=file:/E:/Games/Minecraft%20Server%202%20Electric%20Boogaloo/fabric-server-launch.jar

This should look something like this instead:

Source=file:/D:/Games/Minecraft/Fabric%201.18/libraries/net/fabricmc/sponge-mixin/0.10.7+mixin.0.8.4/sponge-mixin-0.10.7+mixin.0.8.4.jar

Please try installing fabric again with the newest installer (0.10+) from https://fabricmc.net/use/

this worked, thank you


Set up BCV and check APIs on CI

https://github.com/Kotlin/binary-compatibility-validator

View the CI running result at https://github.com/Goooler/ytm-kt/actions/runs/10213815526/job/28259985919.

Would you care to explain what any of this is/does and why it should be included?

It would be easier to track API changes and verify something on CI.


Includes all the @bases

When I try to process code like
.clearfix {
	/*
	* @base
	*/
	clear: both;
}

.button {
	/*
	* @base
	 */
	background: red;
}

.wrapper:after {
	/*
	* @include .clearfix
	 */
}

.wrapper .super-link {
	/*
	* @include .button
	 */
}

i get
.wrapper:after {
	/*
	* @include .clearfix
	 */
	clear: both;
	background: red
}

.wrapper .super-link {
	/*
	* @include .button
	 */
	clear: both;
	background: red
}

i.e. I get both generators in both places.
The bug can be reproduced while using postcss-include either through command line call or via atcss installation and call.

Thank you for your report. I added test code, and I will fix soon :)

OK, thank you!