docker-machine env default consumes STDIN

I have placed the following line in my .bash_profile to initialize my environment:
eval "$(docker-machine env default)"

It appears that when docker-machine connects to the VM via SSH to obtain needed information, it is consuming STDIN:
pstengel@HAL-9000 ~ $ echo "echo Hello world" | bash -l
pstengel@HAL-9000 ~ $

With the docker-machine line in my Bash profile commented out:
pstengel@HAL-9000 ~ $ echo "echo Hello world" | bash -l
Hello world

This is unexpected behavior.

In case anyone else is encountering this, a temporary workaround is to change the eval line to:
eval "$(docker-machine env default </dev/null)"

+1
This causes problems when used as a pre-window command with Tmuxinator/Tmuxp. Any following pane commands are swallowed up, resulting in a bunch of empty panes on launch.
Thanks for that workaround @pstengel, nice to have things working as they should again!

+1
For anyone suffering the same problem as me, this workaround fixed my iTerm2 'Send text at start' option, as well as tmuxstart sessions.
Cheers for the solution!

:+1: - I can also confirm that this resolves issues where having eval "$(docker-machine env default)" in my shell profile caused my tmux scripts to seemingly fail. Sending keys would seem to send the commands to the background so my tmux panes would end up sitting in a default state.
Changing it to:
eval "$(docker-machine env default </dev/null)"

Solves the problem. Thanks @pstengel

Closing since this is now fixed since v0.5.4


Fix bug where we error on network creation if no subnet was provided

Bad scoping - each subnet you provide must have a default gateway. If you provide no\empty subnet that is still valid in some cases.

@madhanrm - PTAL

Can you add a test for this easily so we don't miss again?

Done

LGTM. @madhanrm  - PTAL

LGTM


Add pagination to rpc queries and some keeper methods

TRACK MY PROGRESS HERE
https://linear.app/upshot/issue/ORA-715/add-pagination-and-limits-for-emissions-module-queryserver-responses#comment-e116c215
TICKET
https://linear.app/upshot/issue/ORA-715/add-pagination-and-limits-for-emissions-module-queryserver-responses

Still todo:

Currently we value requests in a time order (first in, first active). This should be replaced with a "best price" order, though we must be sure that the MaxRequestsPerTopic param isn't too large so as to break things when a request is made.
a. Perhaps we can have the consumer bring a request id if worse value => a new consumer can steal the spot of a previous one. (but then, really why would someone pay more if the inference is already covered... => maybe this doesn't matter and we could keep the current "time order")
Fix integration tests


SITE Workflow Blueprints

Describe the task
Work with Jackie to get the SITE workflow blueprints (transcribed from pdf?).
Acceptance Criteria

[ ] Current SITE workflow blueprints are in Figma
[ ] If relevant: created a new task to update SITE workflow blueprints with our desired workflow

SME/User Contact
Jackie
Additional context

any additional details that could not be captured above

Hey team! Please add your planning poker estimate with Zenhub @mariaye-aot @midhun-aot @mishraomp @nikhila-aot @RyanBirtch-aot @sakthi-aot


Docs: Add warning about backend APIs not having auth by default

Hey, I just made a Pull Request!
For me the realization that backstage was unauthenticated by default, and thus anyone can access its information, was very surprising and undesired. I could be the only one, but I would very much welcome any warning about this when reading a security section. AFAIK no such section exists so I put the warning into at least the authentication section.
Also, while reading the docs about cookie authentication, I thought that the "Asset requests" was rather confusing, as at first I was not sure what that is, and maybe "Frontend" would be more generic, but rather easier to understand.
I am not sure which package this should belong to, assuming I need to add a change set. Also, this is just a more concrete suggestion compared to an issue, so no hard feelings if this gets closed, and any feedback is welcome!
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

@Rugvip, this looks good to me, can you approve and merge if you feel the same, please?


Implement an example about Logistic Regression in MesaPy

Description
Motivation:
Now, the two examples using Mesapy engine in MesaTEE ( Clef +1 ) are independent, one is about file access, the other one is about matrix_multiply. It is not enough.
Therefore, this pull request implement an example including model training and prediction all in Python. The model is logistic regression. I expect this can provide another way for user to train the model and do the prediction. It is convenient for people who are familiar with Python to do secondary AI development in MesaTEE.
I finish all the algorithm by using Mesapy builtin library numpy. Actually, there are some differences between pypy micronumpy( MesaPy ) and numpy v1.17 (www.numpy.org).
Type of change (select applied and DELETE the others)

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

How Has This Been Tested?
In my machine, I successfully compile and run the example: py_logistic_reg.
The model paramaters and prediction result are the same to those in the Jupyter Notebook!
However, I don't know how to do CI Check.
Checklist (check ALL before submitting PR, even not applicable)

[x] Fork the repo and create your branch from master.
[x] If you've added code that should be tested, add tests.
[x] If you've changed APIs, update the documentation.
[ ] Ensure the tests pass (see CI results).
[x] Make sure your code lints/format.

Thank you so much!
There are still some issues of our CI on the transition to the apache org. Don't worry, we will manually check the compilation and testing. In the meantime, we are working on the CI issue.

There are still some issues of our CI on the transition to the apache org. Don't worry, we will manually check the compilation and testing. In the meantime, we are working on the CI issue.

Okie, I have read the mail-list. It seems that the GitHub Actions is helpful.

LGTM, thanks. We cannot merge PR at this time. Once the CI is setup, we will start to merge it.

LGTM, thanks. We cannot merge PR at this time. Once the CI is setup, we will start to merge it.

Okie :)

Merged, you can safely delete the branch. Thanks for your contribution.


Capture video with audio recording sample does not work correctly

The "wmt-audio" branch sample does not work correctly.

Which configurations have you tried? I just pushed a new commit to fix an issue when no audio source is selected.
This branch is still a work in progress and has some known issues:

Any recording using loopback audio does not work. There's a problem where the audio graph is flooded with quantum started events when the synthetic node is hooked up. If loopback is enabled the entire recording will not work properly.
AudioLoopbackInputNode has a race when shutting down the MF work queue. This code is generally not ready for consumption yet.

Having said that, recording using the microphone/audio input devices should work just fine.

I tried the defaults with "Record system audio".

Unfortunately, that doesn't work in the sample at this time. I'd take a look the official sample for loopback audio: https://github.com/microsoft/windows-classic-samples/tree/main/Samples/ApplicationLoopback
What you'll need to do is feed the audio samples along with the format to the audio graph API. I'm not an expert on the audio side, so I haven't had the time required to untangle my issue with the quantum started events. You may have better luck asking in one of the Microsoft Q&A communities or on one of the Microsoft sample repos.

I see. OK. Thanks for the help.
I am looking forward to your finished sample, whenever that may be.


fix: Unify CRs' descriptions in Kyma dashboard

Description
Changes proposed in this pull request (what was done and why):

Unify Telemetry, LogPipeline, MetricPipeline, and TracePipeline CRs' descriptions to be consistent with all CR descriptions in Kyma dashboard

Changes refer to particular issues, PRs or documents:

https://github.com/kyma-project/kyma/issues/18321

Traceability

[x] The PR is linked to a GitHub issue.
[ ] New features have a milestone set.
[ ] New features have defined acceptance criteria in a corresponding GitHub Issue, and all criteria are satisfied with this PR.
[ ] The corresponding GitHub issue has a respective area and kind label.
[ ] The follow-up issues (if any) are linked in the Related Issues section.
[ ] Adjusted the documentation if the change is user-facing.
[ ] The feature is unit-tested
[ ] The feature is e2e-tested

Please rebase your PR to include https://github.com/kyma-project/telemetry-manager/pull/1096. This should fix the failed lint job.


Add --ignoreDirectoryPattern cli option

Rebased version of the dead https://github.com/mikeal/watch/pull/93.

Let's hope it gets merged before the base changes again!

No worries, I have rights to merge now and most certainly can rebase as needed ;)

Rebased, merging.

Great :) When will the version get bumped so NPM sees it?

Already done, v0.19.1 should be what you're looking for.

My bad; "watch" was indeed updated


Facing one crash while i try to use VersaPlayerControls

private func checkOwnershipOf(object: Any?, completion: @autoclosure ()->()?) {
guard let ownerPlayer = object as? VersaPlayer else { return }
if ownerPlayer.isEqual(handler?.player) {
completion()
}
}
I have added versaplayerview in one XIB and added versaplayercontrs ciew in View controller and connected with outlets to viewcontroller.
My app is getting crashed at this completion() call. I am not getting any other logs in debug.
can you please guide what reason it can be.
Let me know if I can provide any detailed explanation about implementation.
Warnings I am getting in log :
[MediaRemote] [AVOutputContext] WARNING: AVF context unavailable for +[MRAVOutputContext sharedAudioPresentationContext]_block_invoke
[MediaRemote] [AVOutputContext] WARNING: AVF context unavailable for SYSTEM
Desktop

OS: MAC OS 10.15.3

DEVICE:

Device: iPhone11
OS: iOS13.4.1

Hi @Disha896 could you share the stack trace?


[CI] Introduce check pr test

Let's introduce check-pr-test.
ONE-vscode-DCO-1.0-Signed-off-by: Yongseop Kim<EMAIL_ADDRESS>

https://github.com/Samsung/ONE-vscode/runs/5765023472

PTAL @Samsung/one-vscode

Note: xvfb: https://github.com/Samsung/ONE-vscode/pull/360#issuecomment-1068845901


New version: GiulioSorrentino.Backgrounds version 2.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.5 schema?

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

Microsoft Reviewers: Open in CodeFlow

I couldn't verify that the background images were placed on the VM.

The exe is recompiled with the binary specific of the platform (so it takes a while) and then shows the progress bar, there is not silent install.
The backgrounds are putted in -o"C:\windows\web\wallpaper\numeronesoft-backgrounds" -y

Manual Validation ended with:

2023-11-28 13:32:36.287 [CORE] WinINet downloading from url: http://numeronesoft.ddns.net/numeronesoft-backgrounds-0.4.exe
2023-11-28 13:32:57.432 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(122)\WindowsPackageManager.dll!00007FFA71F5395E: (caller: 00007FFA71F55C49) Exception(2) tid(1b74) 80072EFD     Msg:[InternetOpenUrl() failed.]

The host appears to be down:

It's back online.

The exe is recompiled with the binary specific of the platform (so it takes a while) and then shows the progress bar, there is not silent install.
The backgrounds are putted in C:\windows\web\wallpaper\numeronesoft-backgrounds, if runned with administrative priviledges.

 Thanks for helping me to find them.

@wingetbot waivers Add Validation-No-Executables


Update from dirs to dirs-next

dirs is unmaintained now, see RUSTSEC-2020-0053:
https://github.com/RustSec/advisory-db/blob/master/crates/dirs/RUSTSEC-2020-0053.md

Incorporated into #1877. Thanks!


CompleteByteSlice/CompleteStr are invasive and make parsers not reusable

Prerequisites
Here are a few things you should provide to help me understand the issue:

Rust version : 2018-06-15
nom version : 4.0.0
nom compilation features used: none

Test case
Say you have a parser like this
named!(doubles_and_int<&[u8],(f64,i64)>, do_parse!(
    tag!("DI") >>
    d: double >>
    space >>
    n: digit >>
    (d, str::from_utf8(n).unwrap().parse().unwrap())
));

And (for example because of https://github.com/Geal/nom/issues/790) you want to change that parser to use CompleteByteSlice. It turns out that is quite hard, because (a) the code at the end of do_parse also needs changing to (un)wrap the CompleteByteSlice everywhere, and (b) much worse, double works for &[u8] only. Here's the solution I came up with eventually:
fn result_map<'a, T, I1, I2, F: FnOnce(I1) -> I2>(x: IResult<I1, T>, f: F) -> IResult<I2, T> {
    use nom::Context::Code;
    use nom::Err::*;
    match x {
        Ok((rest, t)) => Ok((f(rest), t)),
        Err(Incomplete(n)) => Err(Incomplete(n)),
        Err(Error(Code(i, k))) => Err(Error(Code(f(i), k))),
        Err(Failure(Code(i, k))) => Err(Failure(Code(f(i), k))),
    }
}
fn double_complete(i: CompleteByteSlice) -> IResult<CompleteByteSlice, f64> {
    result_map(double(i.0), CompleteByteSlice)
}
named!(doubles_and_int<CompleteByteSlice,(f64,i64)>, do_parse!(
    tag!("DI") >>
    d: double_complete >>
    space >>
    n: digit >>
    (d, str::from_utf8(n.0).unwrap().parse().unwrap())
));

This is clearly bad. It would help if there was a variant of double that works on CompleteByteSlice shipped with nom, but the problem actually goes deeper: I may have both streaming and non-streaming parsing in my project, and I may want to re-use some of the parsers. E.g., imagine doubles_and_int was used both on a stream, and in another parser combinator that involves many0!. Right now, the only way I found to make that work is to literally duplicate doubles_and_int, have one version that works on CompleteByteSlice and one that works on &[u8]. I tried wrapping the &[u8] one similar to how I wrapped double above, but that doesn't actually work: For combinators like take_until_and_consume! (and, I presume, tag! though I did not test that), they have to be used directly on a CompleteByteSlice or they will not behave correctly.
I get the impression that using the type to encode the EOF behavior was the wrong choice (though there may be a trick I am missing here). Whether more data can come later is a property of the value (the byte stream) I am parsing; the parsers themselves should mostly be generic and usable on both prefixes and entire inputs.

Even if the type is the right way to do it, I was left wondering if it could be made ergonomic. Having to start specifying the input type for all the named! parsers seems unnecessarily invasive. What about having a trait that abstracts over CompleteByteSlice and &[u8] which is used by default?

@djc Personally I just ended up creating own macro which, in turns, calls named! with the types I need. That seems to work pretty well and straightforward.

@RReverser What do you do about combinators like double that are tied to the non-complete types?

here's a gist that shows what I meant in my previous comment.  It's based on your doubles_and_int parser example.

@RalfJung yes, that was my idea...  I think ideally the trait that abstracts over all the input types would be defined in nom:: somewhere and then named! would use that trait itself... then you could go back to using named!
I agree, the Trait bounds are weird.  I didn't see another way to satisfy the nom::ParseTo requirement of the implementation of floating_point (which uses parse_to!).  Maybe someone else has a better way.

This issue was driving me so mad that I switched to combine even though it’s a bit slower for me. I will be very glad if nom’s mainteners find some way to make CompleteByteSlice more ergonomic

The nom::types::Input type should allow implementing streaming and non-streaming input (note, at some point, streaming parsers have to reach eof for an arbitrary parser to complete). But Input doesn't implement InputTakeAtPosition.
It should also be possible to implement a generic parser over &[u8] or CompleteByteSlice, which is what I tried before I realized I could use Input, but that requires that FindToken<T> has impl's where T: AsChar.
Though, it seems that you can write one using where clauses like: InputIter<Item=u8>. And eventually it will work if you add enough clauses. It'd be nice to have a variation of named! that was generic and just allowed this to 'work'.

CompleteByteSlice and CompleteStr are removed in nom 5. Now there are complete/streaming versions of some parsers that make things much easier

nom 5's release is imminent, closing this now


setTimeout

setTimeout
setInterval
clearTimeout
clearInterval

todo, how to deal with interval timing, recalc based on wanted starttime, actual starttime, or endtime?
fix timing resultion in utils (currently 250ms) should be able to calulate based on present intervals

https://github.com/HiRoFa/utils/issues/1


Kubernetes docker config?

Are you working on a kubernetes config? Your postgres-xl setup removed ssh, which was a major obstacle in kubernetizing it. So now that that blocker is gone I can see an approach.
Hopefully I can find some time to work on it.

Hi @fire! :) I'm not currently, because Docker Swarm has been enough for my current main project, but I'm very open to the idea, ideally supporting someone who wants to work on it, but potentially drafting it myself if there's enough interest (and I find time). :)

Closing, because I see nothing complex about using the latest Postgres-XL Docker images in Kubernetes. If anyone tries it and encounters any difficulties, please say. :)


Pad default SVG thumbnail for posts so that it fits in the center of the container

We have a default SVG image we want to use for posts and news items.  Here is the one we have for CNCF.  The problem with the LFPH one is that it doesn't have the right whitespace padding to make it centered within the container.  Can you add padding so that it fits in the center of the thumbnail containers here?

this is done


fail to reproduce experimental results

After downloading the released data, I executed the following command to reproduce the results of ablation study (RQ3):
python typegen/evaluate.py -s data/predictions/typegen.json -t data/testset_transformed.json

However, the obtained top-5 accuracy (81.0%) was significantly lower than the result reported in the paper (87.5%).
I also tried "w/o COT Prompt" by running the following command:
python typegen/evaluate.py -s data/predictions/typegen_no_cot.json -t data/testset_transformed.json

Surprisingly, I got a higher top-5 accuracy (82.1%) than what was reported for the complete-version TYPEGEN.
How to correctly reproduce the results?

Hi! Thanks very much for your interest! Your problem is due to the incorrect usage of the evaluation script. When evaluating the predictions of TypeGen, you need to add the options -m and -c to indicate that the predictions are sampled (the common generation method of ChatGPT) and contain Chain-of-Thought prompts. If you want to evaluate the performance of TypeGen without COT prompts, you should remove the option -c and just use -m.
We have updated the README to avoid confusion.

Thank you very much!


Whitelist requester's IP CIDR

Hello there!
I want to allow anyone from the local network to access all resources without authentication and only require it when accessing from outside. So, I would love to just whitelist my network's entire CIDR to forego the authentication when at home.
There is middleware chaining in Traefik as far as I know, but I am not aware of any "conditional middleware" (i.e. if $ip in <IP_ADDRESS>/24 then next() else end()).
My use-case is with k3s/traefik - ServiceLB is configured and traffic policy is set to local on the LoadBalancer; so Traefik sees the actual IP.
Thanks and kind regards,
Ingwie

I gues you are search for this option: https://github.com/jordemort/traefik-forward-auth/blob/main/README.md?plain=1#L183


Fixed #30521 -- Fixed invalid HTML in default error pages.

Ticket: https://code.djangoproject.com/ticket/30521
Simple solution to have better html responses form default error pages

Thank you, this is what I was looking for. I especially like your new test :-)

Made changes based on @felixxm comments.
I wanted to add a test to check that details in ERROR_PAGE_TEMPLATE is used but I didn't like how it gets. The following works fine but for page_not_found the entire sentence has to be used(repeated) :-1:.
def test_error_pages(self):
    request = self.request_factory.get('/')
    for response, title, details in (
        (bad_request(request, Exception()), b'Bad Request (400)', b''),
        (permission_denied(request, Exception()), b'403 Forbidden', b''),
        (page_not_found(request, Http404()), b'Not Found', b'The requested ...'),
        (server_error(request), b'Server Error (500)', b''),
    ):
        with self.subTest(title=title):
            self.assertIn(b'<!DOCTYPE html>', response.content)
            self.assertIn(b'<html lang="en">', response.content)
            self.assertIn(b'<head>', response.content)
            self.assertIn(b'<title>%s</title>' % title, response.content)
            self.assertIn(b'<p>%s</p>' % details, response.content)
            self.assertIn(b'<body>', response.content)

Or it could be like this which works but then the closing </p> is not present :-1: .
(page_not_found(request, Http404()), b'Not Found', b'The requested'),

self.assertIn(b'<p>%s' % details, response.content)

@alej0varas Thanks for updates :+1: I pushed cosmetic edits and added a separate commit to test details.


Add ScaleInvariantError Loss

This PR will add ScaleInvariantError (SIE)  loss as mentioned in this issue, #17719

@qlzh727 Hi,
While writing test cases for loss, I observed an issue with the mask function and its impact on ReductionV2. Specifically, I noticed that the use of mask (to avoid invalid depth) caused a loss of shape in y_true and y_pred, ultimately converting them into 1D vectors. Additionally, since the loss function contains log, multiplying the mask to zero values is not a straightforward solution.
In the context of Depth Estimation, it is essential to consider the occurrence of invalid depths (i.e., depth = 0) due to sensor limitations. In fact, two of the most popular datasets used for Depth Estimation, KITTI and NYU-Depth-V2, contain instances of invalid depth.
Also not sure if we can avoid mask as in Depth Estimation, invalid depths ( depth = 0) are very likely to happen due to sensor. Two most popular dataset KITTI and NYU-Depth-V2 contains invalid depth.
Any thoughts on what should we do?

@fchollet @rchao could you please check?

@divyashreepathihalli any update?

Hello, Thank you for submitting a pull request.
We're currently in the process of migrating the new Keras 3 code base from keras-team/keras-core to keras-team/keras.
Consequently, merging this PR is not possible at the moment. After the migration is successfully completed, feel free to reopen this PR at keras-team/keras if you believe it remains relevant to the Keras 3 code base. If instead this PR fixes a bug or security issue in legacy tf.keras, you can instead reopen the PR at keras-team/tf-keras, which hosts the TensorFlow-only, legacy version of Keras.


Preference page update to support IAM user and test connection, i10n support

issue #15

Add new IAM fields in preference page
Add Test Connection button in preference page, along with result
dialogs
Apply user changes in preference store
Add new resources for UI
Update helper methods
Update client jar with new authentication support.

based on comment #16 (comment), following had been included.

update readme
strings externalization
update servicestage-client.jar
update dropin plugins jar.

additional request for relabeling CPE-OAuth#18

externalize all label and name strings in plugin artifacts
update readme image

@paulmchen update PR


[SR2022]: 2022-02-18 National Batteries Day

Body
todo
Media
No response
Category
Filler

We did this in discord instead


When using "yes".boolValue , return true - issue #863

isuue #863

When I was using "yes".boolValue , I returned false,suggesting that the array be added "yes""

Checklist - While not every PR needs it, new features should consider this list:

[x] Does this have tests?
[ ] Does this have documentation?
[ ] Does this break the public API (Requires major version bump)?
[x] Is this a new feature (Requires minor version

Summary
I implemented #863 and write test code
Please review in your free time.

Thanks for the pull request! I've addressed this issue in #887 and credit your name in pre-release note. Cheers!


remove duplicate file name deletion. filename is not a unique identifier.

After reading the MongoDB documentation, it does not look like a file's name is supposed to be a unique identifier in GridFS. Looking over the Mongolian code, the MongolianGridFile#save method performs a findOne query, using the filename and id properties. If it locates a file with the same name, it deletes it. How to save files with the same name, then? I believe this is a mistake in the code. I also do not recognized the used id property - is that supposed to be _id? The existing GridFS file record does not have an id record, so that'll never match anyways, as far as I can tell.
I have removed this check-and-delete logic and the code appears to function as I would expect, now. I am able to insert multiple files with the same name, and they're giving their own record in GridFS.
This does break the unit tests, but after looking over the unit tests it looks like they were written from the perspective of a file's name being unique, which I think invalidates the unit tests as a whole for GridFS, and so I did not bother updating them.
Can we review the reasons why you consider filename to be unique, @marcello3d?

Here is the application that I wrote to test this. I'm simply saving the code for this application to the database. Multiple executions should created additional records. Currently, in Mongolian master branch, this does not create multiple records. It deletes the previous record.
var fs = require('fs'),
    util = require('util'),
    mongolian = new (require('/home/ryan/javascript-libs/node-mongolian')),
    database = mongolian.db('test'),
    gridfs = database.gridfs();

// create a new gridfs file
var file = gridfs.create('foo'),
    write_stream = file.writeStream();

// pipe local data into it
fs.createReadStream('app.js').pipe(write_stream);

// save the file
file.save(function () {

    // list all files in this collection, now
    gridfs.find().forEach(function (document) {

        // print this file out to console
        console.log('file: ' + util.inspect(document) + '\n');

    }, function (err) {

        if (err) { console.log('error: ' + err); }

    });

});

You're correct in your understanding. The id bit is definitely a bug.
My reasoning was that if I save a file to /foo/bar.txt, and I read a file from /foo/bar.txt, I would expect to get the same file back.
I went back and re-read the GridFS spec: it's not clear, but I believe you're right (the only hint is that filename is optional). My impression of their specification was that it left the details to the driver implementor.
Digging into the Java driver source also indicates that filename doesn't have to be unique and you can retrieve multiple files from the same filename.
I wonder if it makes sense to make this an option. I don't think I'm alone in thinking that the duplicate file behavior is unexpected. Perhaps we should ask 10gen?
That said, we'll need new unit tests if we're going to switch.

I read the documentation as basically saying that GridFS is simply a specification for splitting file data up within a regular document collection. So, in essence, it's the same as all other collections except it has some more requirements if it's to be used for storing large documents - and that is that documents are to be chunked. With that in mind, by default, _id is the only unique field in a collection.
Even more specific to the case of GridFS, I like to think of it more as a database for files, rather than a file-system for an OS. In this case, _id would be the key, and the document payload would be the value. All other attributes would just be metadata, including filename. I think that the mongofiles tool that ships with mongodb is a good example of this - it allows duplicate file names.
In fact, I've used the Python, Java and nodejs native drivers for MongoDB on projects in the past, involving GridFS, and I don't think any of them imposed the uniqueness of attributes for GridFS, now that I think about it.
In terms of this being an option, personally I'd rather have a decision like that better be left up to logic in my code rather than my database driver's code. It's really not that difficult of a code check to put in a routine that creates a new GridFS document, anyways. It just seems unnecessary to me, personally.
I have asked on IRC, in #mongodb on Freenode, and have not received any response yet. I'll idle there for the next few hours while I code and see if anyone responds.

I agree with your reasoning. I'll see what I can do for the next release.


drivers/interrupt_controller/intc_ite_it8xxx2: ite_intc_isr_clear() t…

…o be global
We need to clear interrupt status, before we enable the interrupt.
So I let ite_intc_isr_clear() to be global function.
Signed-off-by: Ruibin Chang<EMAIL_ADDRESS>

Could any one help me to review this change? thanks

Is this being used in the tree? Or out-of-tree code?

Timer driver: https://github.com/zephyrproject-rtos/zephyr/compare/master...RuibinChang:Timer
Watchdog driver: https://github.com/zephyrproject-rtos/zephyr/compare/master...RuibinChang:WDT
In above two drivers need to call ite_intc_isr_clear() before we enable the interrupt. (Both driver are still writing in progress)
And I think other members of ITE developer will need to call this function, too.

Thank you, Daniel.
Before Merging it needs another 1 approving, could anyone help me? Thanks~

Please fix the compliance check error.

Done


Add GADGET to the integration infrastructure

estimate 5

Parent story: #1278
Parent story describes done criteria and process for this task which is specific to GADGET.
see https://github.com/bgeltz/geopm/blob/task-1081/test_integration/test_gadget.py

Updated this to a 5.  With the FFTW, GSL, etc. dependencies, doing this "right" is going to take a lot of scripting.


Closing a tab using middle mouse button on the 'x' causes the tab to the right to also be clsosed

VSCode Version: Code 1.11.2 (6eaebe3b9c70406d67c97779468c324a7a95db0e, 2017-04-13T08:03:11.395Z)
OS Version: Windows_NT ia32 10.0.15063
Extensions:

Extension
Author
Version

gitlens
eamodio
3.4.5

cpptools
ms-vscode
0.11.0

PowerShell
ms-vscode
0.12.2

trailing-spaces
shardulm94
0.2.11

vscode-todo-highlight
wayou
0.4.18

Steps to Reproduce:

Have 2 or more tabs open
Press the middle mouse button (scroll wheel) on the 'x' of a tab which has a neighboring tab to its right
Both tabs will be closed

Nice catch 👍

While we're at it, is it also possible to close a tab by double-clicking it?

No, currently not.

Btw PR welcome.

@bpasero I'm tackling this one, I can confirm it also happens on MacOS


Making this card transparent as same as crypto cards

What would you like to share?
Making this card transparent as same as crypto cards
Assign me this

Additional information
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@kabir0x23  I can only submit PR for this or other issue when my already submitted PR gets merged otherwise all the commits will be done in the same PR.
So kindly review My already submitted PR @kabir0x23

@kabir0x23 I can only submit PR for this or other issue when my already submitted PR gets merged otherwise all the commits will be done in the same PR.
So kindly review My already submitted PR @kabir0x23

hey @Kritika30032002
you should create different branches for different issues.
but okay, i will merge it now after reviewing

Okay
But how long it takes for you to review any PR?
Just asking!!

Okay
But how long it takes for you to review any PR? Just asking!!

reviewed ✔


Add WinUI window to WPF/WinForms

One of the promoted use-cases of WinUI 3 on the community calls is to modernize existing applications.
Especially interesting to me would be how to add a WinUI 3 window to an existing WPF app. This is a very important scenario.
The WinUI 3 documentation is lacking a "Getting started from WPF/WinForms" page. And trying to get this working on my own has been unsuccessful.
Please support this.

Is there something out there how to do this? I can't find anything. @anawishnoff do you know?

Is there something out there how to do this? I can't find anything. @anawishnoff do you know?

@Marv51 - this is a great question, and I know that this is a gap in our documentation. While WinUI 3 is still in preview, we're recommending creating new WinUI 3 projects rather than adding it to an existing project. This is why there aren't any docs on adding WinUI 3 to an existing app yet.
Our local WinUI 3/Desktop expert @marb2000 may have some more detailed advice here.

@Marv51 - this is a great question, and I know that this is a gap in our documentation. While WinUI 3 is still in preview, we're recommending creating new WinUI 3 projects rather than adding it to an existing project. This is why there aren't any docs on adding WinUI 3 to an existing app yet.
Our local WinUI 3/Desktop expert @marb2000 may have some more detailed advice here.

Any update on this? @marb2000 or @anawishnoff ?

After looking into this a bit more, it does not seem to me like this is a documentation issue, it seems to not work. I have updated the title and the description.
@ranjeshj  Can you remove the "documentation" label?

Understanding a bit more about how WinUI 3 works. I feel like I should close this issue.
It is not possible to have WPF/WinForms and WinUI 3 in the same process. If/When XAML-Islands returns then that solves this issue completely.
I was thinking it would be possible/easier to have a WinUI 3 window in a WPF app without full XAML-Island support, but I think that is not so.

This info is on the WinUi page on docs.microsoft.com "... or gradually migrate your existing MFC, WinForms, or WPF apps using familiar languages such as ... ".

I've got a WinForms application and I'm trying to create a new window using WinUI. Is it possible now?
If not, is it possible to create a new Win32 app (that has the same API access as a .NET6 WinForms app) that uses WinUI instead?

@marb2000 fyi

Sorry but I'm now totally lost, and I'm sure I'm not the only one.
1- I thought the aim for project reunion was to be able to use any Win32 controls in a WinUI project. Wasn't it?
2- Since WinUI, WinRT and .Net6 are the new standards, should all my new projects be WinUI projects? (I aim at Windows 10-11 only). But there are WPF controls I really want to use that aren't available in WinUI (especially when using a 3rd party library like DevExpress); so what project type do your recommend I use in that case?
3- As I think I understand right now, XAML Islands is not working in .Net6?
Thank you it's quite confusing honestly (so much have changed in .Net in a short amount of time; it seems like reading about Project Reunion/Xaml Islands or looking at videos from 1 or 2 years ago doesn't seem relevant anymore).

Anyone please? @Marv51 , @anawishnoff , @marb2000? Thank you.

Pretty depressing to read astramistils pleas for some sanity in the WinUI world, nearly three years on from the OP.
The funeral rites on Winforms and WPF have been pronounced repeatedly over the years but there's still lots of us using them daily to get the job done in the dektop world.
MS helpfully made it easy to host WPF controls in WInforms apps and vice versa. Is that one of the design goals  of WinUI or is MS determined to go on trying to persuade us to torch years of development work and move to what looks like a very shaky, now not-so-new framework ?
Anyone please? @Marv51 , @anawishnoff , @marb2000?


Convert to module (ESM).

BREAKING: Convert to module (ESM).
BREAKING: Require Node.js >=14.
Update dependencies.
Lint module.
Ignore temporary editor files.
Use auto-allocated server port for testing.
Handle updated HTTP errors.

Notes:

Some tests are failing.  Help needed.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@dbd7b87). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #22   +/-   ##
=======================================
  Coverage        ?   94.60%
=======================================
  Files           ?        4
  Lines           ?      760
  Branches        ?        0
=======================================
  Hits            ?      719
  Misses          ?       41
  Partials        ?        0

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 dbd7b87...b07fd9e. Read the comment docs.

@dlongley I think some README edits went to the generated file rather than the template.  Is the sync back to the template here correct? https://github.com/digitalbazaar/ezcap-express/pull/22/commits/767a4dcd792eb036eb44f6f6a727d8ada7b9f489


Random Battle Moveset Additions

Added a lot of moves available to Pokemon in ORAS but for some reason
weren't added to their random battle movesets.
(( Why did I add moves to the pre-evos as well? Why not? Also, Seasonals. ))

Made some changes based on suggestions made over IRC


Update feast transformer docs

Updated feast transformer example
Proposed Changes

docs for example in https://github.com/kserve/kserve/pull/2780

Thanks @sivanantha321 !
/lgtm
/approve


Localize DataAnnotation enums

Fixes #7748.
Since this is a breaking change it's hidden behind a compat swtich. I've included @NTaylorMullen specifically to review the compat switch/options stuff just because I don't have a lot of experience with it.
Otherwise, the majority of this is test changes.

:up::date:

:up::date:


Manifests not sorted by .metadata.generateName

Output of tanka show is non-deterministic when multiple manifests are defined when .metadata.generateName is defined without .metadata.name.
It could be worthwhile to take the value of .metadata.generateName into consideration when sorting.
There could be multiple manifests defined with the same .metadata.generateName for which the sort order can be non-deterministic. When manifests have different values for this field, then it would be nice to have them sorted.

Hi 🙂 I've prepared a PR that should solve this. Can you please give it a look?


refactor(supervisor): refactored this.configuration.format

In the latest version of Yarn, the previously deprecated configuration.format has been removed, this PR refactors supervisor to use formatUtils.pretty, the yarn suggested replacement for configuration.format .
Unfortunately, at present, the code will not compile.
Typescript is reporting errors Argument of type 'string' is not assignable to parameter of type 'null'.ts(2345) when formatting PATH items, for example in:
formatUtils.pretty(this.configuration,
            relativePath,
            formatUtils.Type.PATH

VSCode is incorrectly reporting that the method requires a null for the value argument.

VSCode correctly reports the method signature as value: string|null when used in the yarn repo

I suspect that the root cause is somewhere in the yarn.build configuration of typescript, but I have (as yet) been unable to pinpoint it.
A workaround (that seems to illicit no ill effects) is to comment out the following code.
    PATH: {
        pretty: (configuration: any, val: string) => string;
        json: (val: string) => any;
    };

in formatUtils.d.ts, lines 80:83
This PR also has added a few method return values that lint had raised as issues.

Thanks for getting this draft together.
we might need to update yarn in this repo to the latest as well for those types?

Thanks for getting this draft together.
we might need to update yarn in this repo to the latest as well for those types?

Thanks for getting this draft together.
we might need to update yarn in this repo to the latest as well for those types?

The refactor (barring the typescript errors mentioned) seemed to work OK with both v2.4.0 and set version from sources.
I did try upgrading typescript to latest release (as well as some ancillary packages like eslint etc) but the problem seemed to persist.  I'd be happy to take another stab at bringing all the dependencies up to date  (as a separate PR).

Thanks for getting this draft together.
we might need to update yarn in this repo to the latest as well for those types?

The refactor (barring the typescript errors mentioned) seemed to work OK with both v2.4.0 and set version from sources.
I did try upgrading typescript to latest release (as well as some ancillary packages like eslint etc) but the problem seemed to persist.  I'd be happy to take another stab at bringing all the dependencies up to date  (as a separate PR).

Just FYI, because I didn't check existing PRs, I also worked on fixing this and also ran into the exact same issue. https://github.com/oliversalzburg/yarn.build/commit/54ddc677089f9e85ad21a8a758462d236c7c3d24
I asked for guidance on Discord, but no solution was immediatly apparent.

Just FYI, because I didn't check existing PRs, I also worked on fixing this and also ran into the exact same issue. https://github.com/oliversalzburg/yarn.build/commit/54ddc677089f9e85ad21a8a758462d236c7c3d24
I asked for guidance on Discord, but no solution was immediatly apparent.

I added a specific test (locally) in my yarn berry repo and VSCode/TS had no issues with it ... I suspect that somewhere in the yarn.build config we've something that is causing this ... but I cannot figure out what it is!

I added a specific test (locally) in my yarn berry repo and VSCode/TS had no issues with it ... I suspect that somewhere in the yarn.build config we've something that is causing this ... but I cannot figure out what it is!

Excellent call! I believe this may have something to with it in the package.json:
 "resolutions": {
    "typescript": "^3.8.3"
  },

Excellent call! I believe this may have something to with it in the package.json:
 "resolutions": {
    "typescript": "^3.8.3"
  },

I have a branch with updated development dependencies at https://github.com/ojkelly/yarn.build/pull/68 btw. I'll probably wait for other PRs to be merged before I investigate further.

I have a branch with updated development dependencies at https://github.com/ojkelly/yarn.build/pull/68 btw. I'll probably wait for other PRs to be merged before I investigate further.

I'd started a similar and also trying to set all the configuration parameters to mirror those used by yarn berry ... I believe there is mileage in that but I did not manage to complete the journey (ie I still saw the TS error).

I'd started a similar and also trying to set all the configuration parameters to mirror those used by yarn berry ... I believe there is mileage in that but I did not manage to complete the journey (ie I still saw the TS error).

This must be a problem with the berry typings IMHO. It reproduces in isolation: https://www.typescriptlang.org/play?ts=4.1.3#code/CYUwxgNghgTiAEIB2BXAtvAKgTwA4IG8BYAKHnPgAUBBTACXgF54AiG+lgGlIF9TRIsBGAD2SAM4AXeJJhQJAMxEw04gFzxiZCuzoatFQ-FxxJk7BoAUopAoCWAcxRzJdsRvnZO8AG5QIGlIwdkgOAJRMAHzwQSEOANykRhQAVuLu8JZ+ATGycRGM0Z6J2uQ8JeX84NBwMngI1BAQmHKKyqpMdfgiCjKt4koq4iUCNQjm+PAAyiLOYCAAPJjRzJiIAB6SyMDi8ADWINg98I3N-YMdAPxUsFBoIFsw4gunLfID7eIA2pgAul8AcjSYgBv0iXwADL94AAfeCoJrwQJ5UKw+EoJolCYITAACzs4gAkuIAHIYiCdGZzRY4fAAOl0kXiQA

This must be a problem with the berry typings IMHO. It reproduces in isolation: https://www.typescriptlang.org/play?ts=4.1.3#code/CYUwxgNghgTiAEIB2BXAtvAKgTwA4IG8BYAKHnPgAUBBTACXgF54AiG+lgGlIF9TRIsBGAD2SAM4AXeJJhQJAMxEw04gFzxiZCuzoatFQ-FxxJk7BoAUopAoCWAcxRzJdsRvnZO8AG5QIGlIwdkgOAJRMAHzwQSEOANykRhQAVuLu8JZ+ATGycRGM0Z6J2uQ8JeX84NBwMngI1BAQmHKKyqpMdfgiCjKt4koq4iUCNQjm+PAAyiLOYCAAPJjRzJiIAB6SyMDi8ADWINg98I3N-YMdAPxUsFBoIFsw4gunLfID7eIA2pgAul8AcjSYgBv0iXwADL94AAfeCoJrwQJ5UKw+EoJolCYITAACzs4gAkuIAHIYiCdGZzRY4fAAOl0kXiQA

Awesome investigation!
As a followup (in case there was some subtlety being missed) I tried same approach with the contents of formatUtils.d.ts from berry ... showed the same thing as you demonstrated.  playground
and then I wondered about the validity of using Type as the enum name ... This playground is with a different name ... and the problem seems to go away.

Awesome investigation!
As a followup (in case there was some subtlety being missed) I tried same approach with the contents of formatUtils.d.ts from berry ... showed the same thing as you demonstrated.  playground
and then I wondered about the validity of using Type as the enum name ... This playground is with a different name ... and the problem seems to go away.

I brought this up again on the Discord. arcanis said he would change the type structure so that it compiles to a TSD that doesn't have this problem. It's apparently a know issue.
Let's hope it'll land soon 😊

I brought this up again on the Discord. arcanis said he would change the type structure so that it compiles to a TSD that doesn't have this problem. It's apparently a know issue.
Let's hope it'll land soon 😊

I brought this up again on the Discord. arcanis said he would change the type structure so that it compiles to a TSD that doesn't have this problem. It's apparently a know issue.
Let's hope it'll land soon 😊

I was wrong, simply changing the name does not correct the issue. playground

I brought this up again on the Discord. arcanis said he would change the type structure so that it compiles to a TSD that doesn't have this problem. It's apparently a know issue.
Let's hope it'll land soon 😊

I was wrong, simply changing the name does not correct the issue. playground

In #88 a bunch of updates including this PR we're combined to get a release compatible with Yarn 3 out.
While the main issue of this PR persists, I'm just going to side-step it so it can unblock us.
Just released is this for yarn 3 yarn plugin import https://yarn.build/latest.
With yarn 2 available at yarn plugin import https://yarn.build/v2.
Thanks for your help in digging into the issue.


Batch download molecular data rows

Issue
Fixes #303
Description
Allow the user to download more than one molecular data row at a time
Testing instructions
https://localhost:3000/data/models/JAX/TM00025 > tick "add to batch.." checkbox on multiple rows > scroll down to "download batch ..."

TODO: Add rows by sampleId

TODO:

[ ]  Downloading indicator
[ ] Download on card click instead of when checking box? (ideal)
[ ] Download add button


Store plugin settings in .idea folder for sharing between team members

Hello,
Great plugin! Would it be possible to store plugin settings in .idea folder of the project so that plugin settings can be shared across the team? Plugin settings file would then be automatically picked up by the plugin.
Thanks.

Hi,
Thanks for your suggestion. From what I know the .idea folder is usually not checked in, I think this might not the best way of sharing the settings.
The standard way of sharing IntelliJ settings that I'm aware of is via Settings Sync.
This should already be supported. Here is what you can try and let me know if this works:

Configure the plugin the way you want to
In the top left click File -> Manage IDE Settings -> Export Settings...
Deselect everything and select only CommitMessageStoredConfiguration. This is the plugin configuration (see screenshot)
Export the settings and optionally commit this zip file into the repository
On another Intellij import the settings with File -> Manage IDE Settings -> Import Settings...

Thanks

@boristhuy Any update on this? Have you managed to share the settings in this way?


Template format error: Unresolved resource dependencies [SubnetPrivate1cRouteToNatGateway] in the Resources block of the template

Hello everyone,
I have a problem trying to deploy a Multi-AZ cluster with kube-aws v0.11.1 and v0.12.0.
getting the same error on each "kube-aws validate" run:
$ kube-aws validate

Validating UserData and stack template...
generating assets for control-plane, etcd, network, worker, monitoring
Error: failed to validate etcd plane: invalid cloudformation stack template https://s3.amazonaws.com/cosckoya.lab/config/kube-aws/clusters/mylab/exported/stacks/etcd/stack.json: ValidationError: Template format error: Unresolved resource dependencies [Subpriv1cRouteToNatGateway] in the Resources block of the template
	status code: 400, request id: 4e7bf6f5-ecb6-11e8-8ac3-nohaypatatasfritas

This is my private subnet definition:
[...]
  - name: subpriv1c
    private: true
    availabilityZone: us-east-1c
    instanceCIDR: "<IP_ADDRESS>/24"
    natGateway:
      eipAllocationId: eipalloc-XXX

Any ideas?
Thanks! :)

Looks quite similar to https://github.com/kubernetes-incubator/kube-aws/issues/1332, different resource but still. I ended up removing the dep from the generated CF template and then it worked but the template generated from source as of latest master still does not work.

@c-knowles yep. I have read some othe related issues: #1488 #1494 ... I've made a workaround too, generating the aws network resources by hand; VPC, NGW, EIP, Subnet, RouteTables and ELB.Then adding the VPC, ELB and SubnetIDs into the yaml, takes time but it's not hard work.
I've manage to deploy the ec2 instances paired to their network resources. Im still with it, trying with kube-aws v0.12.0.

Oh my! I was able to deploy Etcd EC2 instances and that's all. No master, no worker, no game :S

I've managed to deploy a MultiAZ cluster with 3 NGW & 1 IGW with kube-aws v0.9.9 (+hotfix).
Cheers!


Environment dependent conda-build

Issue:
Consider an example conda recipe with the following build.sh script:
echo "LDFLAGS=$LDFLAGS"

When running LDFLAGS= conda-build /path/to/example/recipe, the output will be
LDFLAGS=-Wl,-O2 -Wl,--sort-common -Wl,--as-needed -Wl,-z,relro -Wl,-z,now -Wl,--disable-new-dtags -Wl,--gc-sections -Wl,-rpath,$PREFIX/lib -Wl,-rpath-link,$PREFIX/lib -L$PREFIX/lib

However, when LDFLAGS is not empty, for instance, run LDFLAGS="-L/path/to/foo" conda-build /path/to/example/recipe, the output is
LDFLAGS=-Lpath/to/foo

I am not sure if this is a bug or not but the result of building a conda recipe from an environment that defines LDFLAGS will be different from the result of the build the recipe where LDFLAGS is not defined. And it's not obvious to figure it out when there are some build issues because of this.
Possible options to resolve this issue:

When LDFLAGS is defined, conda-build should prepend its flags to LDFLAGS.
When LDFLAGS is defined, conda-build should warn that it will use LDFLAGS as it is without adding its flags to LDFLAGS.
When LDFLAGS is defined, raise an exception.
When LDFLAGS is defined, issue a warning that LDFLAGS in an environment will be ignored.

All the above applies also for environment variables CFLAGS, CXXFLAGS, etc.

I'm fine with 1. PRs welcome.

cc @beckermr

Thanks for the pointer @isuruf!


Files: provide a way (via the UI) to include/exclude a folder

This is the default behavior with the completion of #2168. Closing this one up 📕


database seeder issue

this is example this is Setting module
inside module composer autoload
"autoload": {
"psr-4": {
"Modules\Setting\": "app/",
"Modules\Setting\Database\Factories\": "database/factories/",
"Modules\Setting\Database\Seeders\": "database/seeders/"
}
},
"Database"and "Seeders " both have initials in capital but
inside auto-generated seeder file has the namespace is set to
"namespace Modules\Setting\database\seeders;"
due to this the seeder is not working as it supposed to and as a workaround, I have to manually change namespace to
"namespace Modules\Setting\Database\Seeders;"
I also tried to change module composer as  below from stubs and did composer dump but did not work
"autoload": {
"psr-4": {
"Modules\Setting\": "app/",
"Modules\Setting\database\factories\": "database/factories/",
"Modules\Setting\database\seeders\": "database/seeders/"
}
},

I am also experiencing the same issue.
The only solution that has worked (so far) is to change the name spaces manually

okay got to the bottom of it, adding in a namespace to the generators path corrected this.
I'll update the config in the package, for your own files update:
'seeder' => ['path' => 'database/seeders', 'namespace' => 'Database\Seeders', 'generate' => true],
'factory' => ['path' => 'database/factories', 'namespace' => 'Database\Factories', 'generate' => true],

released https://github.com/nWidart/laravel-modules/releases/tag/11.0.4 will take care of the default config.


Align carousel items to the left.

As you can see the image above, i need the item align to the left, but instead it align to the right, how can we fix guys ?
Thank you for reading.

i understood how library work, thank you.


Repeating response content when using streamed chat API

Using the following code to stream chat responses:
$stream = OpenAI::chat()->createStreamed([
    'model' => 'gpt-3.5-turbo',
    'messages' => [
        ['role' => 'user', 'content' => 'Say "Hello World"']
    ],
    'max_tokens' => 500,
    'temperature' => 0.5,
    'top_p' => 0.5,
    'frequency_penalty' => 0,
    'presence_penalty' => 0,
    'user' => '<some user id>'
]);

/* @var CreateStreamedResponse $response */
foreach ($stream as $response) {
    if (connection_aborted()) {
        return;
    }

    /* @var CreateStreamedResponseChoice $choice */
    $choice = $response->choices[0];
    $answer = $choice->delta->content;

    $this->sendUpdate($answer);

    $finishReason = $choice->finishReason;
    if (isset($finishReason)) {
        $this->sendUpdate("<$finishReason>");
        return;
    }
}

In response I get the following from the API (ids are sanitized but they are different):
{
    "id": "<sanitized>",
    "object": "chat.completion.chunk",
    "created":<PHONE_NUMBER>,
    "model": "gpt-3.5-turbo-0613",
    "choices": [
        {
            "index": 0,
            "delta": {
                "content": "Hello"
            },
            "finish_reason": null
        }
    ]
}

{
    "id": "<sanitized>",
    "object": "chat.completion.chunk",
    "created":<PHONE_NUMBER>,
    "model": "gpt-3.5-turbo-0613",
    "choices": [
        {
            "index": 0,
            "delta": {
                "content": "Hello"
            },
            "finish_reason": null
        }
    ]
}

{
    "id": "<sanitized>",
    "object": "chat.completion.chunk",
    "created":<PHONE_NUMBER>,
    "model": "gpt-3.5-turbo-0613",
    "choices": [
        {
            "index": 0,
            "delta": {
                "content": "Hello"
            },
            "finish_reason": null
        }
    ]
}

// Ad Infinitum...

Any idea what's happening?

Not a client library issue.


Fix golint issue in maya/pkg/task/patch.go

File: maya/pkg/task/patch.go
Line 28: warning: exported type TaskPatchType should have comment or be unexported (golint)
Line 48: warning: comment on exported type TaskPatch should be of the form "TaskPatch ..." (with optional leading article) (golint)

Resources:
-> Contributors guide: https://github.com/openebs/openebs/blob/master/CONTRIBUTING.md
Complexity: Easy

go ahead @galihmelon . Let me know if you need any help.


"Could not find module ..." with nix (NixOS) installation

I've installed and IHaskell on NixOS with the following lines in my configuration.nix:
  environment = {
    systemPackages = with pkgs; [
    (with python3Packages; python3.withPackages (ps: with ps; [
         jupyter notebook
      ]))
      ihaskell
    ];
  };

  nixpkgs.config = {
    ihaskell.packages = (self: with self; [
      ihaskell-charts
      ihaskell-diagrams
      # ihaskell-display
      diagrams
      ihaskell-plot
    ]);
  };

But IHaskell doesn't see Diagrams.Prelude module:
<interactive>:1:1: error:
    Could not find module ‘Diagrams.Prelude’

Is there something I've forgotten to add?
P.S. ihaskell-diagrams is commented since it fails to build on latest nixos-unstable channel.

It doesn't look like you're using the provided ihaskell service, can you enable that instead of adding ihaskell to your systemPackages?

How did you enable the ihaskell service?
    ihaskell = {
      enable = true;
      extraPackages = (self: with self; [
        diagrams
        ihaskell-diagrams
      ]);
    };

This really seems like a question for the nixpkgs issue tracker rather than this one.
Well, I guess you are right, I'll ask it there. Thanks!
P.S. If you don't mind, I would like this issue to be open for some time, until I get everything working together.

Did you try the configuration I provided? You have ihaskell instead of services.ihaskell.

It is indeed services.ihaskell. I just forgot to add this since in my config it is nested.

Have you gotten this working?

Closing this, please reopen if necessary.


one ip per container done

Signed-off-by: kevin xu<EMAIL_ADDRESS>

Current coverage is 42.38% (diff: 100%)

Merging #88 into master will increase coverage by 0.13%

@@             master        #88   diff @@
==========================================
  Files            29         29
  Lines         10863      10863
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           4589       4604    +15
+ Misses         5131       5125     -6
+ Partials       1143       1134     -9

Powered by Codecov. Last update 97f3565...683f3f0


Inconsistent use of "profile" vs. "implementation guide"

Section Number throughout
Issue We still have an MHD "Profile", right?    The "implementation guide" is just the documentation vehicle for specifying it, right?     It's unclear how you decided to use one term vs another in here.
e.g.  https://profiles.ihe.net/ITI/MHD/2_actors_and_transactions.html#transaction-descriptions, you say "This section defines the actors and transactions in this implementation guide."   I think the actors are in the profile.  MHD Doc Source, MHD Doc Recipient.   Yes, the exist "in" the IG, but that's just because you have write about them on these pages.
I know this is a 'dancing on the head of a pin' argument, but we are setting precedent here for future IGs.
Proposed Change    You might say "IG" is a term we are going to adopt now.   Then, is "MHD Profile" even accurate anymore (I hope so).   If so, when?
Priority: medium

We can't continue to think that this is a fight we can win.
There is strong dissonance between IHE and HL7 use of the word "profile", and there is significant marketing pressure around "Implementation Guide".
We can't afford to not use the term "Implementation Guide", especially in the context of the authoring and publishing solution driven by the "IG Publisher".
IHE needs to come up with consistent ways to address and change our own thinking on this. The external community are already understanding the concepts as HL7 uses them. However the external community do think that "Implementation Guide" is something that only HL7 publishes, and they are confused on what "profile" means. Thus I think IHE needs to aggressively market that "IHE Profile" and "Implementation Guide" are the same thing. Therefore I think that IHE Profiles do need to emphasize this.
My approach is to use "IHE Profile" and "Implementation Guide" as the same. The use of upper case "Profile" in the context of an IHE publication can be understood as "IHE Profile". The lowercase "profile" is ambiguous, most of the time referring to what the FHIR community calls a "StructureDefinition constraint" .  I don't think that IHE can stop using "IHE Profile", but I do think we can increase use of "Implementation Guide".
We need consistency rules. We need marketing of this.

I'm not looking to win or lose a fight.   If we believe Profile and Implementation Guide are two terms for the same conceptual thing, OK.   We can write 5 explanatory sentences to say that, put it in a common appendix, and then add a link to that explanation in each fhir-based document/artifact that IHE publishes.  Or, something like that.   What I'd like to avoid is using the both terms randomly (exaggerating with that) throughout the text.

see https://profiles.ihe.net/GeneralIntro/ch-3.html

Any further refinement will happen in final review


Ponder Source Milestone 1

Working on #15 and #21 in this branch, those two together to complete #23


Explore creating a dpp::collector wrapper

would reduce complexity of architecture; docs here

after exploring this, would increase memory usage for program, and DPP doesn't work like the docs say


Allow users to highlight text in different colors

Right now you can highlight text in the editor, but all the text has to be highlighting in the same color. This sucks and it would be great to be able to highlight different text in different colors. Below is an example of three different highlights that were supposed to have different colors, but did not.

This project looks interesting, always wanted to work on a text editor in Python. Can I work on this issue?

This project looks interesting, always wanted to work on a text editor in Python. Can I work on this issue?

Thanks for showing interest. I love a good text editor and I really wanted to create one of my own. You are welcome to work on any of the existing issues, or create a new issue/feature you would like to fix/add.