"sources created by user" is incomplete

Under "my sources", the "sources created by user" only shows three test sources I made within NewCantus (where I have dozens on OldCantus--see below). This suggests to me that the OldCantus creator information is not being used in NewCantus, and users of the new site could lose their old "created by user" list.

cf OldCantus:

I guess this is really a consequence of #237 .

Maybe it would be worth asking Jan about how to get at this information. Last summer, I recall @jiwoojeongjj, @jwang44 and I doing a fairly thorough search, and it was when we came up empty-handed that Junhao opened #237...

Interesting...today this is empty, and I can no longer find the test sources in question.  I don't remember deleting them...

Is this on staging? @lucasmarchd01 deleted all the old sources and synced from OldCantus - now, the only sources on staging should be ones present on OldCantus as of a couple of days ago.

Ah that probably explains the disappearance.  My desire to see this populated with my old created sources still stands, though.

Regarding getting creatorship information from OldCantus, I don't think there's any possibility of making progress on this issue unless someone is able to figure out #237.
Linking #737, because this seems directly to the discussion of "author" there.

@lucasmarchd01, similar to my request on #644, when you get a chance, can you review the status of this one? Has this particular issue been addressed by the recent changes you've made to these pages?

Linking #825 here as it will most likely resolve this issue as well.

Closing this issue as we have now synced the data from OldCantus with #825 in place


Transform should accept streams of Objects, not just strings

A Little Feature request
i expected it to work out of the box, but apparently i had to stringify my JSON objects, because otherwise Transform threw an exception stating it accepts either strings or buffers. This feature is useful when streaming from database queries (in my case, it was using the streaming capabilities of the mongodb driver), etc.

Include the version of json2csv used: 4.2.1
Include your node version/browser vendor and version: 10.12

Node.js streams work either on strings/buffers mode or in object mode.
Have you read about object mode in the documentation?
I think that that's what you want.
If you want something else. Can you elaborate a bit? And possibly provide a code sample showing what you would like?

Closing since I think that this is resolved. Feel free to re-open if you feel that something is still missing.


feat: added package installation progress

Closes #1049
✅ Checklist

[x] I have followed every step in the contributing guide (updated 2022-10-06).
[x] The PR title follows the convention we established conventional-commit
[x] I performed a functional test on my final commit

Changelog

added package installation progress to the cli working for all 3 supported package managers
npm - output is inherited from stderr
pnpm and yarn - output is redirected from stdout

Screenshots
npm

pnpm

yarn

💯

Can we make the pnpm output prettier and not pushed so far to the right?

Can we make the pnpm output prettier and not pushed so far to the right?

pnpm uses this kind of output by default:

However, I think we can try to either:

parse the progress bar manually and show it somewhat differently
cut the string to the | sign and only show the text with progress info

What do you think is the better option? Of course the second one is easier haha

It looks like that now:


DS-4499 Fix cleanup of test environment in earlier Integration Test

References
https://jira.lyrasis.org/browse/DS-4499
Description
*All IT classes have been reviewed and where the context.restoreAuthSystemState() was missing, it has been added.
*Objects that were not used have been removed.
*Refactoring of the cleanup method in the Builder classes has been done.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[x] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & integration tests). Exceptions may be made if previously agreed upon.
[x] My PR passes Checkstyle validation based on the Code Style Guide
[ ] My PR includes Javadoc for all new (or modified) public methods and classes. It also includes Javadoc for large or complex private methods.
[x] My PR passes all tests and includes new/updated Unit or Integration Tests for any bug fixes, improvements or new features. A few reminders about what constitutes good tests:

Include tests for different user types, including: (1) Anonymous user, (2) Logged in user (non-admin), and (3) Administrator.
Include tests for known error scenarios and error codes (e.g. 400 Bad Request, 401 Unauthorized, 403 Forbidden, 404 Not Found, etc)
For bug fixes, include a test that reproduces the bug and proves it is fixed. For clarity, it may be useful to provide the test in a separate commit from the bug fix.

N/A If my PR includes new, third-party dependencies (in any pom.xml), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.
N/A If my PR modifies the REST API, I've linked to the REST Contract page (or open PR) related to this change.

it is ready to review now. A nice side effect of the refactoring seems to be an increased speedup in the test execution. The server tests took 12minutes less compared to the last build on the master

hi @benbosman
thanks for reviewing that. I can answer your first question

Why is it now required to explicitly delete e.g. items from a builder, this used to be automated

this is not required, no changes in such direction. If we have added an explicit deletion of an item created with an item builder this is most probably superfluous. Can you point to the specific line so that I can eventually double check?

What's the purpose of .andDo(result -> idRef.set(read(result.getResponse().getContentAsString(), "$.id")))
What's the purpose of .andDo(result -> idRefNoEmbeds.set(UUID.fromString(read(result.getResponse().getContentAsString(), "$.id"))))

in general they are used to grab the ID of object created via REST request so that they can be cleaned later. Do you have any specific lines that I need to double check?


Add support for ISafeSerializationInfo interface.

JSON.NET can't seem to deserialize an exception that it serialized with the same settings.  I get an error which suggests it cannot find a protected constructor in various EntityFramework exception classes: "ISerializable type 'System.Data.Entity.Infrastructure.DbUpdateConcurrencyException' does not have a valid constructor. To correctly implement ISerializable a constructor that takes SerializationInfo and StreamingContext parameters should be present."
The following code throws an error on deserialization:
var serializationSettings = new JsonSerializerSettings() {
    DateFormatHandling = DateFormatHandling.IsoDateFormat,
    DateParseHandling = DateParseHandling.DateTime,
    DateTimeZoneHandling = DateTimeZoneHandling.RoundtripKind,
    DefaultValueHandling = DefaultValueHandling.Include,
    TypeNameHandling = TypeNameHandling.All,
    TypeNameAssemblyFormat = System.Runtime.Serialization.Formatters.FormatterAssemblyStyle.Simple,
    ObjectCreationHandling = ObjectCreationHandling.Replace, //Necessary for subclassing list types
    ConstructorHandling = ConstructorHandling.AllowNonPublicDefaultConstructor
};

var json = JsonConvert.SerializeObject( new System.Data.Entity.Infrastructure.DbUpdateConcurrencyException( "hi" ), serializationSettings );
if (json == null)
    return null;
var err = JsonConvert.DeserializeObject<System.Data.Entity.Infrastructure.DbUpdateConcurrencyException>( json, serializationSettings ); //throws error

The DbUpdateException class it inherits from is marked with attribute [SuppressMessage("Microsoft.Design", "CA1032:ImplementStandardExceptionConstructors", Justification = "SerializeObjectState used instead")], which seems intentional.  It has a member that implements ISafeSerializationData, which is new in .NET 4.0 and is designed to work like so:
"In versions previous to.NET Framework 4.0, serialization of custom user data in a security transparent code was accomplished using the GetObjectData() method. Starting with .NET Framework 4.0, that method is marked with the SecurityCriticalAttribute attribute which prevents execution in security-transparent code. To work around this condition, implement the ISafeSerializationData interface."  See: https://msdn.microsoft.com/en-us/library/system.runtime.serialization.isafeserializationdata(v=vs.110).aspx
As you can see from the DbUpdateException class, it very intentionally does NOT implement the constructor, and even provides a justification for not doing so:
        /// <summary>
        /// Exception thrown by <see cref="T:System.Data.Entity.DbContext" /> when the saving of changes to the database fails.
        /// Note that state entries referenced by this exception are not serialized due to security and accesses to the
        /// state entries after serialization will return null.
        /// </summary>
        [SuppressMessage("Microsoft.Design", "CA1032:ImplementStandardExceptionConstructors",
        Justification = "SerializeObjectState used instead")]
        [Serializable]
        public class DbUpdateException : DataException
        {
                /// <summary>
                /// Holds exception state that will be serialized when the exception is serialized.
                /// </summary>
                [Serializable]
                private struct DbUpdateExceptionState : ISafeSerializationData
                {

I'm not sure how one should go about supporting this.  As of now, none of these Errors are deserializable by JSON.NET.

Guys! I am also getting this error. Any solution or workaround?

Are there any plans to fix this?

You can do it with a contract resolver if you accept to use only base exception fields.
public class ISasfeSerializationInfoContractResolver : DefaultContractResolver
{
    protected override JsonContract CreateContract(Type objectType)
    {
        JsonContract contract = base.CreateContract(objectType);

        if (contract is JsonISerializableContract && IsSafeSerializationInfo(objectType))
        {
            JsonISerializableContract serializable = contract as JsonISerializableContract;
            serializable.ISerializableCreator = p =>
            {
                SerializationInfo info = (SerializationInfo)p[0];
                StreamingContext context = (StreamingContext)p[1];

                Exception exception = (Exception)Activator.CreateInstance(typeof(Exception), BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance, null,
                new object[] { info, context }, CultureInfo.InvariantCulture);

                object realObject = Activator.CreateInstance(objectType, BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.Instance, null, null, CultureInfo.InvariantCulture);

                FieldInfo[] fields = typeof(Exception).GetFields(System.Reflection.BindingFlags.NonPublic | System.Reflection.BindingFlags.Instance);

                foreach (FieldInfo field in fields)
                {
                    field.SetValue(realObject, field.GetValue(exception));
                }

                return realObject;
            };

        }

        return contract;
    }

    private bool IsSafeSerializationInfo(Type objectType)
    {
        if (objectType != typeof(Exception) && typeof(Exception).IsAssignableFrom(objectType))
        {
            if (objectType.GetConstructors().Count() < 4)
            {
                return true;
            }
        }

        return false;
    }
}


Patches that are contextmanagers are not registered properly in _nbdev

@pete88b since we merged the stuff with magic, I'm noticing that methods patched to a function that are also context managers are no registered as class.method in the _nbdev.py but just as method.
An example of this in fastai2 is Learner.no_bar in 16_callback.progress.ipynb registered as no_bar now. Do you have bandwidth to take this issue?

Oops, it's actually something that used to be the case and that you fixed, I was reading the diff wrong ;)


[VS Code] Cannot read property 'id' of undefined

Error:
Cannot read property 'id' of undefined
Steps to Reproduce:

Version Data

extensionName
VSLS

extensionVersion
0.3.623

protocolVersion
2.2

applicationName
VSCode

applicationVersion
1.26.1

platformName
Linux

platformVersion
4.15.0-33-generic

@ASEE26 Hey! Do you recall when you say this error? Was it when starting or attaching to a collaborative debugging session?

@lostintangent  Was it when starting to a collaborative debugging session. But, the problem was because the internet connection was bad.

@ASEE26 This issue should be fixed in the current version of the extension. Please let us know if it occurs again.

Hi Team, I am facing this error while doing retrieve from sandbox using vs code from last 15 days and it is very urgent or critical to update a branch on bitbucket with the new changes, please help here. This is my last hope for issue resolution.

Hey @NehaAgrawalTayal , it looks like that error message is coming from the Salesforce CLI - you might want to try asking your question on their repo (this one is for Visual Studio Live Share).

Okay, Thank you @daytonellwanger
If possible can you share forum/plateform for the question post.


Add a small python distribution + dependencies to the docker image?

I was just thinking - it would really simplify the CI/CD process if the Docker image we supply could both build page HTML as well as the full book HTML. Doing this would require us to put a little Python distribution alongside the current emdupre/jupyter-book image.
What do folks think about doing this?

It will make the docker image larger, but I'm not currently optimizing for size :smile_cat:
Could you confirm what we want in the image other than a working python 3.5+ install ? I'll assume all of the requirements.txt installed, but is there anything else ?

hmmm - well, since originally posting this, I realized that it's actually pretty straightforward in CI/CD to just use the Ruby image, and then manually install Python (see https://predictablynoisy.com/automating-jb for some notes). So maybe this issue would be unnecessary complexity that wouldn't help as much? I think I'd be happy to just keep the Ruby image as you've already got it...

@emdupre are you OK with me closing this? I feel like it is not as useful as I thought it'd be and is extra work :-)


Update best-practices.md

Will follow up on teams.

@dyor : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @dyor - If you're the Microsoft PM I see in Outlook, your GitHub account should be linked to your Microsoft account. Also, (Microsoft) PMs should work in the private repository. Information about both of these items is in the internal contributor guide, but I can send you the information in email, if you'd like. Please let me know. Thanks!

This will be handled in a separate PR in the private repository.
#please-close

This will be handled in a separate PR in the private repository.
#please-close


[Homepage] Responsive carousel for zemax training

[Homepage] Responsive carousel for zemax training, which isn't carousel in wide screen, but become carousel in smaller screen, which may show one or two items

done in https://github.com/hlxsites/zemax/pull/92


add Validate User to flask microservices

Pull the validate user url from the env vars
Call the /msapi/validateuser endpoint to validate the userid.

Note:  JWT token contains the userid to be validated.  This token is stored in the cookies from the /api/login end point.  Login needs to be called prior to calling this endpoint
validateuser_url = os.getenv("VALIDATEUSER_URL", "http://localhost:5000")

class CompItem(Resource):

    @classmethod
    def get(cls):

        result = requests.get(validateuser_url + "/msapi/validateuser", cookies=request.cookies)
        if (result is None):
            return None, 404

        if (result.status_code != 200):
            return result.json(), 404

Need to make sure main.py in the following repos are using the ValidateUser code.

ortelius-ms-compitem-crud
ortelius-ms-dep-pkg-r
ortelius-ms-dep-pkg-cud
ortelius-ms-textfile-crud

https://github.com/ortelius/ortelius-ms-dep-pkg-cud/pull/7
https://github.com/ortelius/ortelius-ms-dep-pkg-cud/pull/8


[BUG] Send by QR-code doesn't work on dev web/android

[BUG] Send by QR-code doesn't work on dev web/android
Steps:

go to send by QR-code screen
scan QR-code
enter the amount of GD
observe the result

Actual result:
It redirects you back to the main screen without confirmation
Expected result:
The send by QR-code flow leads to a confirmation screen
Environment:
DEV Android build Version 2.27.0 (204839)
WEB 2.27.0
Device:
Android 10 // Redmi 9
Attachments:
QR-code bug.txt
Private Zenhub Video

fixed here: https://github.com/GoodDollar/GoodDAPP/commit/7a7cf9fb1d7ba8f76417c72106945f72ef43a5e4

verified on prod


Motion controller : near plane and cloning problem in Unity 2017.2.0f3

I met some problem when using the HMD in Unity project.
In the hierarchy of my project, for some reason the parent of my "MixedRealityCameraParent" should be scaled to 0.12 . just like the picture below

But actually I've met some problem when using the HMD because the near plane is not as near as I think. In the Scene window, the plane doesn't clip my controller at all, but in the Game window (or watching in the HMD), my controller is actually clipped.
The main reason is that I change the scale of the parent of "MixedRealityCameraParent".
Is there any way to edit the camera rendering formula or adjust something to fix this problem?
2.
I find that the motion controller should be cloned by the script.
Is there any way that we can put the gameObject directly in the hierarchy of Unity like HTC Vive?
Because I have to change some HTC Vive project into Windows Mixed Reality project.
Thanks a lot!

Why is your camera parented on another game object?

Because in my project, I have to scale the player into a smaller size, to meet the need of the environment's resolution.

I think that's a bad idea.
Instead you should be adjusting the scale of your objects.

Thanks a lot for answering my question!
But as I mentioned above, I cannot adjust the scale of the object around me because of the resolution. Is there any way to edit the formula calculating in the camera?
Because the project is changed from the HTC Vive project. In the HTC Vive project, I didn't met this problem.

It makes sense that it isn't clipped in the scene view, because that isn't showing you things from the perspective of your camera, like the game window/HMD does.
Scaling the camera likely does unexpected things.
What are your camera clipping planes set to in the Vive project?

The resolution of your texutres, or the mesh itself?
The mesh resolution wouldn't make a difference, but the texture scaling can also be done too to make it look the same without scaling your camera.


Add Media Mate

Adds my application, Media Mate, which is a easy to setup and use media center.
Link

Thanks!


Downloadlist: "Cancel download" missing in context menu

Download list: In a few cases the entry "Cancel download" is missing in the context menu.

I cannot reproduce this bug. Are there any steps you have to take to reproduce this? Can you cancel downloads in the detail view?

Apart from filtering the download list with search terms, I have not taken any further steps. The downloads were in waiting mode. In the individual cases where the entry was missing, the download could be cancelled via the detailed view. I have not yet been able to reproduce this bug specifically.


Redesign every input field in ODK-X Surveys to meet ODK-X Design Guidelines

The text fields used to enter texts into the ODK-X Surveys need to be redesigned. The input fields need to be updated to match the current ODK-X Design Guidelines. and to review for usability.
Objectives:

Update the UI to match the current style guide, and new styles introduced in the Outreachy Winter 2022 project.
Look at Surveys and become familiar with the current input fields throughout the app.
Provide a redesign of the input fields that meet the objectives.

Design Guidelines:
Colours, icons and other user interface elements used should conform to the ODK-X Design Guidelines
These guidelines may be updated during the next Outreachy 2024 project.
Sample

Hello @Cveman1 Can I be assigned to work on this?
Would this task require design screens and implementation of the designs to the applications?

Hello @wbrunette could you please assign me to this task? Thank you so much

Hello @Cveman1 Can I be assigned to work on this?
Would this task require design screens and implementation of the designs to the applications?

Yes, Design and Implementation

Hello @wbrunette could you please assign me to this task? Thank you so much

Go ahead

Hello @Redeem-Grimm-Satoshi @Cveman1 I went through the apps and the design guidelines. I have been able to clone all the repositories and set up my environment. I would appreciate if you could point me to where I can edit the style for the text fields. I checked the res folder but couldn't find a folder for the styles. Thank you.
Here is a screenshot of the redesigned screen attached in the issue that was created.

The text fields, fonts and other components follow the outlined styles in the ODK-X Design Guidelines.

@teelda ODK Survey and Tables are currently rendered on a webview, Meaning they use frontend web stack ( html, css, javascript ) not like ODK-X Services written purely in Java ( Android )
You should be looking at this repository: https://github.com/odk-x/app-designer

Thank you @Redeem-Grimm-Satoshi I have cloned the https://github.com/odk-x/app-designer repository. From my devtool, I noticed a CSS preprocessor LESS, was used to compile the CSS. I couldn't find the .less source files from the project structure in my text editor. I can see the minified CSS bootstrap.min.css and the source map bootstrap.css.map.
I would appreciate if you could give me a path to locate the .less source files. Thank you
Here's is what I have done

I figured the class for the input field was .form-control, I made my changes in the bootstrap.css this didn't reflect in my browser.

I searched the project for @import statements to find where the .form-control styles are defined and I didn't find them either.

I checked the Gruntfile.js to look for the configuration for any .less files and I didn't find that either. I have npm,less and grunt installed.

I used grunt adbpush to push my changes to my physical device and there didn't reflect either.

This is the image showing the source file in my devtool.

The image showing the changes I made which are not reflecting the the webview.

"However, I added my styles to the odk-survey.css file and they reflected. Do I push this? "
Can you provide a screenshot of that first?, the changes you made

@Redeem-Grimm-Satoshi Here is the screenshot of where I added the styles for the text input field which reflected on the web view. I made changes to the text colour, border colour, font size, padding etc according to the styles in ODK-X Design Guidelines.

This is what the current text input field looks like.
screen-capture.webm
This is what it looked like before the change

@teelda looks good

"I would appreciate if you could give me a path to locate the .less source files. Thank you" Did you locate it?
"However, I added my styles to the odk-survey.css file and they reflected. Do I push this?" Yes send a PR to the required Repo

@Redeem-Grimm-Satoshi "I would appreciate if you could give me a path to locate the .less source files. Thank you" Did you locate it? No, I have not located the path yet.
"However, I added my styles to the odk-survey.css file and they reflected. Do I push this?" Yes send a PR to the required Repo.
Thank you, I have sent a PR to this regard.

Hello @Redeem-Grimm-Satoshi,
I am an Outreachy applicant and would love to contribute by working on this issue. Could you please assign it to me?
Thank you!

Hello @Redeem-Grimm-Satoshi,
I am an Outreachy applicant and would love to contribute by working on this issue. Could you please assign it to me?
Thank you!

Go ahead

Hello  @Redeem-Grimm-Satoshi ,
I wanted to let you know that I have submitted a pull request for the redesign of the ODK-X Survey, aligning with the ODK-X Design Guidelines.
When you have a moment, could you please review my PR? If you notice anything that needs improvement or adjustment, I would greatly appreciate your feedback.

#214 :https://github.com/odk-x/app-designer/pull/214


includeBuild has unexpected side effects on the "main" build

@cloviss commented on Fri Jun 08 2018
includeBuild changes the build behavior of the composite build, even if the included build has nothing be be built.
Expected Behavior
"Each included build is configured and executed in isolation", as documented.
Current Behavior
Using includeBuild changes the build behavior of the "main" build (and eventually this build might fail).
Steps to Reproduce (for bugs)
Unpack the attached file foo.zip and, in the directory foo, type gradle compileJava . The build will fail.
Remove includeBuild '../includedProject'  from foo\settings.gradle. The build succeeds.
Your Environment
gradle 4.7 and 4.8
.
├───foo
│   │   settings.gradle
│   │
│   └───app
│           build.gradle
│
└───includedProject

foo\settings.gradle contains only
include 'app'
includeBuild '../includedProject'

foo\app\build.gradle downloads a file from github to use it as dependency
apply plugin: 'java'
ext.jgraphxVersion = '3.9.3'

repositories {
    ivy {
        url 'https://github.com/'
        layout 'pattern', {
            artifact '/[organisation]/[module]/archive/v[revision].[ext]'
        }
    }

    flatDir {
        dirs "${buildDir}/unpacked/dist/jgraphx-${jgraphxVersion}/lib"
    }
}

configurations { jgraphx }

dependencies {
    jgraphx "jgraph:jgraphx:${jgraphxVersion}@zip"

    implementation ":jgraphx:"
}

task unzipJgraphx(type: Copy) {
    from zipTree(file(project.configurations.jgraphx.asPath))
    into file("${buildDir}/unpacked/dist/")
}

tasks.compileJava.dependsOn unzipJgraphx

This build resolves dependencies at configuration time which is an error. Composite builds just highlight that problem more clearly, but it's always wrong.


Header updates

This moves header() calls to a class that simply accumulates them and then outputs them when requested.  Sending in a duplicate header replaces an existing ones.  Moved most of redirect to this class as well since that was header work with a maintance task.
The goal is to prevent headers already sent errors and allow an easy way to change a header before it ever gets sent.  Its a static method as where we do final output is not consolidated in one area.
Also updated the attachment area, the headers were a bit messy but I think they are all cleaned up now.  I also changed the compressed file output.  It now looks at the mime type and if its text/xyz it will attempt to compress the file.  This replaces the old file extension sniff.  It also drops the use of the output buffer to compress the file and instead calls gzencode directly on the file and echos that result.  This potentially uses more memory but the old method would not compress larger files due to buffer overflow and might use readfile or might use file get contents or might use fread .. the update simply uses one to keep it simple.
Bonus updates are a couple of bug fixes found while trying to test this.  Also the protection index.php files (which still have standard header calls) were updated to remove that ugly nested dirname stuff

Codecov Report

Merging #3465 (ffebde7) into development (946699e) will increase coverage by 0.08%.
The diff coverage is 33.87%.

@@                Coverage Diff                @@
##             development    #3465      +/-   ##
=================================================
+ Coverage          30.22%   30.30%   +0.08%
- Complexity         15976    16044      +68
=================================================
  Files                417      419       +2
  Lines              74414    74599     +185
=================================================
+ Hits               22489    22610     +121
- Misses             51925    51989      +64

Impacted Files
Coverage Δ
Complexity Δ

sources/DumpDatabase.php
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

sources/ElkArte/AdminController/BadBehavior.php
0.00% <0.00%> (ø)
35.00 <0.00> (ø)

sources/ElkArte/AdminController/ManageErrors.php
0.00% <0.00%> (ø)
41.00 <0.00> (ø)

sources/ElkArte/AdminController/ManageSearch.php
4.41% <0.00%> (+0.12%)
83.00 <0.00> (-7.00)
:arrow_up:

sources/ElkArte/AdminController/ManageServer.php
39.83% <0.00%> (ø)
59.00 <0.00> (ø)

sources/ElkArte/Controller/Announce.php
27.58% <0.00%> (ø)
30.00 <0.00> (ø)

sources/ElkArte/Controller/Calendar.php
5.26% <0.00%> (-0.08%)
48.00 <0.00> (ø)

sources/ElkArte/Controller/Groups.php
25.28% <0.00%> (+0.09%)
86.00 <0.00> (-2.00)
:arrow_up:

sources/ElkArte/Controller/MergeTopics.php
0.00% <0.00%> (ø)
35.00 <0.00> (ø)

sources/ElkArte/Controller/News.php
0.00% <0.00%> (ø)
119.00 <0.00> (-1.00)

... and 60 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 b938078...928fc02. Read the comment docs.

Cool!
I started doing something similar a few times and always given up. :laughing:


Align CEP section with the Tour Guide App

CEP section on the tour guide needs to be aligned with the Tour Guide Application example

closing this. CEP Proton has been discontinued

fiware-cep in StackOverflow stills mentioning Proton:

This should be fixed... in some way. Maybe with a more neutral label description, removing Proton mentions.


[BEAM-10341] Support drain in Java SDK

The majority of the review occurred on #12016 and this PR represents a large portion of that carved off that was almost ready for check in.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.


Fix non-existing version in pom dependency management

There is no v4.9.3 on mvn central (yet), so the project won't compile erroring with:

Could not find artifact de.flapdoodle.embed:de.flapdoodle.embed.mongo:jar:4.9.3 in central (https://repo.maven.apache.org/maven2)

@Philzen thanks:) i made a new release with version 4.9.3 as i had to change this version in more than one place (spring 3.0.x etc) .. so stay tuned. It should be in maven central soon (<24hrs).

Thanks for the reply – works fine now.


merge from konsum-frontend-svelte

package.json

chore(deps): add ^1.2.0 remove ^1.1.1 (devDependencies.@sveltejs/vite-plugin-svelte)

.github/workflows/browser_ci.yml

chore: add ${{ matrix.os }} (jobs.test.runs-on)

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

npm package (@latest dist-tag)
v1.34.11
GitHub release

Your semantic-release bot :package::rocket:


Cannot get direct essay link from parse_list

After call parse_list , I can only get URL like this http://weixin.sogou.com/websearch/art.jsp?sg=CBf80b2xkgZWehj5vWa6p7H14b.... . However, most weixin essay's direct link is something like this http://mp.weixin.qq.com/s?__biz=MjM5NjM4OTAyMA=.... . If you request the first link you can get 302 redirection. Response header:
HTTP/1.1 302 Found
Server: nginx
Date: Sun, 08 Nov 2015 07:52:29 GMT
Content-Type: text/html; charset=UTF-8
Transfer-Encoding: chunked
Connection: keep-alive
x_log_ext: suv=008A25C47B7F4D26563EFED6A708B610&openid=oIWsFt3nvJ2jaaxm9UOB_LUos02k&query=%E7%AE%80%E4%B9%A6&rank=1&from=gzhjs&page=1&snuid=76022F344F557452C69017DF5064A727&dec_art=succ
Location: http://mp.weixin.qq.com/s?__biz=MjM5NjM4OTAyMA==&mid=400225618&idx=1&sn=4560366d10e320d2feddf1ce0e00bf0e&3rd=MzA3MDU4NTYzMw==&scene=6#rd
Set-Cookie: black_passportid=1; domain=.sogou.com; path=/; expires=Thu, 01-Dec-1994 16:00:00 GMT
Expires: Sun, 08 Nov 2015 07:52:29 GMT
Cache-Control: max-age=0

The question is that I cannot get the redirect url using requests package:
r = requests.get(link, headers=headers, cookies=cookies)
print(r.headers)
print(r.url)
for resp in r.history:
    print(resp.status_code, resp.url)

I try to use this code to get response Location in the header. But I always get 200 status code not 302. And get 当前请求已过期，请点击重新加载 error. Did I miss something?

meet the same problem


Rust package handling is broken for packages containing "-"

Xmake Version
2.7.7
Operating System Version and Architecture
Windows 11
Describe Bug
This change introduced a bug, as by default Rust libraries containing a - (for example future-util) will produce .rlib file such as libfuture_util-<hash>.rlib.
With this commit, xmake now searches for a libfuture-<hash>.rlib file which is wrong.
This causes xmake to not link some libs (and rustc to not find some libraries).
I know this change was caused by this discussion: https://www.reddit.com/r/xmake/comments/zq9k9c/xmake_not_installing_all_cargo_dependencies and I checked why obj-rs didn't produce a libobj_rs-<hash>.rlib file and it's because this lib renames its output file.
I don't know if we can detect this when cargo builds the crate rlib file, but for now we should revert the commit as it breaks more libraries than it fixes.
Expected Behavior
That it works with all Rust crates.
Project Configuration
I tested this with:
src/main.rs:
use obj::*;
use futures_util::*;

fn main() {
    println!("Hello, world!");
}

Cargo.toml:
[package]
name = "test_rust"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
futures-util = "0.3"
obj-rs = "0.7.0"

xmake.lua:
add_requires("cargo::test", {configs={cargo_toml=path.join(os.scriptdir(), "Cargo.toml"),version="*"}})

target("test")
	set_kind("binary")
	add_files("src/main.rs")
	add_packages("cargo::test")

with the current xmake version, this project will break on future-util:
lynix@SirDesktop:/mnt/c/Projets/Perso/Tests/test_rust$ xmake.exe
checking for Microsoft Visual Studio (x64) version ... 2022
[ 33%]: linking.release test.exe
error: error[E0432]: unresolved import `futures_util`
 --> src\main.rs:3:5
  |
3 | use futures_util::*;
  |     ^^^^^^^^^^^^ use of undeclared crate or module `futures_util`

with https://github.com/xmake-io/xmake/commit/081dd1de9439e2bf71ab1a1afffac3cb9a1049e1 reverted, obj-rs breaks:
lynix@SirDesktop:/mnt/c/Projets/Perso/Tests/test_rust$ xmake.exe
[ 33%]: linking.release test.exe
error: error[E0432]: unresolved import `obj`
 --> src\main.rs:2:5
  |
2 | use obj::*;
  |     ^^^ use of undeclared crate or module `obj`

however I think it should be reverted for now until we have a better solution, because most Rust libraries don't rename their library file.
Additional Information and Error Logs
No response

you can open a pr.

I tried extracting the libname from cargo.toml and it seems to work for me, despite some hacking.
I don't have a better way to deal with it at the moment.
https://github.com/xmake-io/xmake/commit/8c1523adbcc37c3e020433faada44e8825bf99b6

It works great with #3462 applied, thanks.


[BOARDD] Update QA test's board member profile

This PR was created by @EthanThatOneKid using the boardd slash command. More info.

Updating self’s full name still works 👍


Remove nested zlint package structure

This moves main.go to a cmd/ subpackage, and moves zlint/zlint to the
root package. It further simplifies main.go to operate only on single
certificates. Multiple certificate linting should be accomplished using
the ZCertificate utility.

Thanks for tackling this. This has been bothering me too. :)

Just as a note, we should update zcertificate this weekend. I think we've made several breaking changes this week.

The Go idiom is to put main/cLI packages in a directory structure like cmd/zlint, cmd/zfoo, etc


Icon Request: [name of icon]

I would like icons for identification bands on wrist, adult, pediatric and infant for nurse education modules

Hi,
I realise now the challenges of showing such detail (different ages) in a small ICON. Thanks for your help, and I don’t mind waiting.
Clare
Clare O' Reilly RN, RSCN, Graduate Certificate in Transfusion Practice
Transfusion Safety Nurse Clinician
Children's & Women's Health Centres
Call “transfusion nurse” on Vocera
<EMAIL_ADDRESS>direct line<PHONE_NUMBER>
Pronouns I use: she | her | hers
I acknowledge that the land on which I live and work is that of the unceded territory of the Coast Salish peoples, including the territories of the Musqueam, Squamish and Tsleil-Waututh Nations.
Blood transfusion is a lot like marriage...it should not be entered into lightly, unadvisedly or wantonly, or more often than is absolutely necessary. [Beal, R.W. (1976): Aust N Z J Surg 6:309.]
CONFIDENTIALITY NOTICE: This e-mail and any attachments are confidential and may be protected by legal privilege. If you are not the intended recipient, be aware that any disclosure, copying, distribution, or use of this e-mail or any attachment is prohibited. If you have received this e-mail in error, please notify the Provincial Health Services Authority (PHSA) immediately by replying to this e-mail and delete this message from your system. Thank you.
@.***
From: Daniel Burka @.>
Sent: Monday, June 05, 2023 08:06
To: resolvetosavelives/healthicons @.>
Cc: OReilly, Clare [CWBC] @.>; Author @.>
Subject: Re: [resolvetosavelives/healthicons] Icon Request: [name of icon] (Issue #169)
EXTERNAL SENDER. If you suspect this message is malicious, please forward to @.*** and do not open attachments or click on links.

We just added these as part of a recent update. So sorry it took so long with no communication!
I'm not sure we can represent a difference for age groups... the icons can't have that much detail. Unless you have a great idea to do it?
[Image removed by sender. image]https://user-images.githubusercontent.com/711809/243393423-c12652b9-6c90-4bf8-ba28-d3ee52072a0b.png
—
Reply to this email directly, view it on GitHubhttps://github.com/resolvetosavelives/healthicons/issues/169#issuecomment-1576976213, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A35OEP72D7H345SRLVU7EKLXJXYT5ANCNFSM6AAAAAARTH6S4Y.
You are receiving this because you authored the thread.Message ID: @.***>


Add fallbacks for publishedDate and updatedDate

Use publishedDate and updatedDate as fallbacks for each other, or use the feed date if neither is available. (e.g. if publishedDate is not available, set it to updatedDate and vice versa)
This also adds some tests for the publishedDate and updatedDate properties.
cc: @BernhardPosselt re https://github.com/nextcloud/news/pull/43

Right, you can add the checks at both places but if there's a sensible default it should be done once in the library rather than forcing all users to deal with nulls :D

Ok, the whole thing is a bit of a mess with logic sprinkled into all parser files, so let's re-iterate:
There are 4 types of dates which need fallbacks:

$feed->getDate()
$item->getDate()
$item->getPublishedDate()
$item->getUpdatedDate()

Code should look something like this:
// feed date fallback
if ($feed->getDate() == null) {
    $feed->setDate($this->getDateParser()->getCurrentDateTime());
}

// item date fallback
if ($item->getPublishedDate() == null) {
    $item->setPublishedDate($feed->getDate());
}

if ($item->getUpdatedDate() == null) {
    $item->setUpdatedDate($item->getPublishedDate());
}

if ($item->getDate() == null) {
    $item->setDate($item->getPublishedDate());
}```

There's another issue with the feed date which can produce a null exception: https://github.com/fguillot/picoFeed/blob/master/lib/PicoFeed/Parser/Rss20.php#L138 falls back to the current date while https://github.com/fguillot/picoFeed/blob/master/lib/PicoFeed/Parser/Atom.php#L149 doesn't. The current date fallback should be extracted into the parent parser class (see feed code above).

Another thing that I'm curious about: why do you fall back to the feed date if the updated date is null? Shoulnt it fall back to the item's published date?

Another thing that I'm curious about: why do you fall back to the feed date if the updated date is null?

Stupidity ;-)
Your item date fallback code changes the current fallback logic, is this on purpose? E.g. the current code uses setDate(max($published,$updated)) and not just the published date.
I updated this pr with your code, but keeping the current fallback logic. (without the feed date fixes)

I see, then all is well :)


"xcversion install-cli-tools| and "xcversion install" returns 1 if package is already installed

Commands:
$ xcversion install-cli-tools
and
$ xcversion install 8.2.1
returns error code (1) if Xcode/cli is already installed
Currently:
returns 1 value. It means that the error was introduced. So scripts like "xcversion install 8.2.1 && xcodebuild --version"
Expected:
When xcode or command line tools are already installed it should return 0. If it will be that script "xcversion install 8.2.1 && xcodebuild --version" will work.
It will be also much easier to check if error occured.

Sounds good to me, feel free to submit a pull request changing this behaviour 👍

im new user ! u cen help please


Key "plugins": Key "react-refresh": Expected an object.

Hello, I am getting this error when I run the lint command:
ConfigError: Config (unnamed): Key "plugins": Key "react-refresh": Expected an object.
I am using<EMAIL_ADDRESS>and I have type="module" in my package.json, I import and use the plugin like this:
import reactRefresh from 'eslint-plugin-react-refresh'

  // ...
  {
    files: ['**/*.tsx'],
    plugins: {
      'react-refresh': reactRefresh,
    },
    rules: {
      'react-refresh/only-export-components': [
        'warn',
        { allowConstantExport: true },
      ],
    },
  },
  // ...

I found that it works fine if I remove the 0 && from the below code in the bundled index.js file or if I use require('eslint-plugin-react-refresh')
// Annotate the CommonJS export names for ESM import in node:
0 && (module.exports = {
  rules
});

What's your node version? Do have any issues when running a fresh vite react templates (also using esm + eslint v9 + this plugin)

Do you have more information on your setup? tsconfig option should only impact how the TS import are resolved for type checking, but it doesn't impact node at runtime, expect if you are using something else?

Sorry my bad, I am using unstable eslint.config.ts
You can check this repo I created.
Or you can reproduce the issue using Vite with additional eslint.config.ts setup:

create a new react-ts vite app npm create vite@latest -- --template react-ts
rename eslint.config.js to eslint.config.ts
install jiti npm install -D jiti
run eslint npx eslint --flag unstable_ts_config.

--

What's your node version?

My node version is 20.16.0

This is fixed in the latest version!


add haze

am I supposed to be seeing a haze at the bottom?

am I supposed to be seeing a haze at the bottom?

looks like there was a lcouple lines in the wrong commit on the other PR so when I cherry-picked I missed them. add in d6e4123


feat: escape hatch typings

Hey, this PR adds autocomplete typing support on the __ escape hatch while leaving the possibility to freely use untyped values.

@astahmer what would be the added value here? The original idea with escape hatch properties is to only use them when you need a value that is not in tour tokens, but I'm probably missing something here! (happy to merge, just want to check about the added value)

the main thing that PR changes is to go from any to string|number so the props passed using __ are still somewhat correct
imo keeping the autocomplete accurate (by providing the token autocomplete) even with the __ API (which should definitely be used for free values) makes it easier to visualize what values are available, what kind of values can/should be used, I see it more as a dev helper than it having an actual use

@astahmer got it, that sounds good to me, merging!

will release a new version soon 😄


New Group -"No result found" message is shown while selecting the existing user

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Launch app
Tap fab
Tap new group
Enter an existing user id
Select the user

Expected Result:
"No result found" message should not be shown while selecting the existing user
Actual Result:
"No result found" message is shown while selecting the existing user
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.61-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/c759dd84-ffd4-467b-bd32-ba327ee4e4b6
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Reviewing today while in the air! Currently taxi'd, will get to once I can hop on wifi!

Coming back from conference last week - updating today!


Add Letters

Project URL
https://github.com/klaaspieter/letters
Category
Other
Description
An app to teach your kids the alphabet and how to type.
Why it should be included to Awesome macOS open source applications  (optional)
If “it's an open source mac app” is not reason enough then: it's an open source mac app with some weird functional concepts.
Checklist

[x] Only one project/change is in this pull request
[x] Addition in alphabetical order
[x] Appropriate language icon(s) added if applicable
[ ] Screenshots(s) added if any
Where?
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          2 Warnings

:warning:
Too many changes (when adding, please keep it to one project per Pull Request)

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

116
404
https://raw.githubusercontent.com/alichtman/shallow-backup/master/img/demo_faster.gif

593
301
https://github.com/riboseinc/rvc-mac redirects tohttps://github.com/riboseinc/cryptode-mac

  Generated by :no_entry_sign: Danger

For the record it was saying too many changes because I sorted the entire Other section in vim and it sorts lowercase letters below uppercase letters. Which I believe is technically correct 🤔

Close this because I've added it in #339. Thank you!


added CentOS prod/stg instances to pagure service

added CentOS prod/stg instances to pagure service

recheck

recheck

Packit tests are broken, I'll merge manually.


update README

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.2%) to 96.515%

Files with Coverage Reduction
New Missed Lines
%

moltx/pipelines.py
1
97.84%

Totals

Change from base Build<PHONE_NUMBER>9:
-0.2%

Covered Lines:
637

Relevant Lines:
660

💛  - Coveralls


Can i host my server made using RobotJs library on Heroku?

I was trying to host but there where some issues in deployment.
make: *** [robotjs.target.mk:120: Release/obj.target/robotjs/src/mouse.o] Error 1
make: Leaving directory '/tmp/build_499e5fd9/node_modules/robotjs/build'
gyp ERR! build error
gyp ERR! stack Error: make failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/tmp/build_499e5fd9/.heroku/node/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:197:13)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:254:12)
gyp ERR! System Linux 4.4.0-1088-aws
gyp ERR! command "/tmp/build_499e5fd9/.heroku/node/bin/node" "/tmp/build_499e5fd9/.heroku/node/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /tmp/build_499e5fd9/node_modules/robotjs
gyp ERR! node -v v11.10.0
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: prebuild-install || node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
   npm ERR! A complete log of this run can be found in:
   npm ERR!     /tmp/npmcache.9J3jU/_logs/2021-06-02T06_07_54_834Z-debug.log

-----> Build failed

pretty sure you aren't going to have any mouse or keyboards to control on heroku.

Hey Im just trying to do this too, did you manage to get any instances of robot js running on heroku


kubecolor: 0.3.3 -> 0.4.0, and fix included packages

Description of changes

kubecolor: 0.3.3 -> 0.4.0:
https://github.com/kubecolor/kubecolor/releases/tag/v0.4.0

kubecolor: only include root package:
the repository contains multiple executable Go "main" packages used for things like generating documentation and running tests, but only the root package should be a published executable artifact from the kubecolor repository. They were accidentally included from the beginning, and have no practical use for the end-users.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[x] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[x] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.


xml-body-parser returning errors-

I use xml-body-parser in my middleware-
With version 0.2.2 xml body parser use to work fine- Every time a client use to provide invalid xml. Parser use to work perfectly with our middleware chain. Now as soon as i have update to 0.3.0. I am getting  exception from my application-
Error: Can't set headers after they are sent.
at ServerResponse.OutgoingMessage.setHeader (_http_outgoing.js:356:11)
When i go back to previous version 0.2.2 i no longer get the above exception and errors are handled by my next function in line in the middleware.
Is there any specific change that was done between these versions ?

Never mind- i set the async=false and it worked!

@SameerSiddiqui Glad I could help 😄 Unfortunately, async=true still has some issues, so it's recommended to turn it off.


about FatraGNN, including model,datasets and example

Description

Checklist
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [$CATEGORY] (such as [NN], [Model], [Doc], [Feature]])
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage
[ ] Code is well-documented
[ ] To the best of my knowledge, examples are either not affected by this change,
or have been fixed to be compatible with this change
[ ] Related issue is referred in this PR

Changes

Model file was not well-documented and comments should be in English.
Please add a readme document to the examples/fatragnn folder.

提交对应数据集和模型的pytest文件，数据集pytest内容可参考 GammaGL/tests/datasets/test_acm.py 的内容


Do not select nor display messages when venv created through command by another extension

Today we use the python command to create a new python environment from the Jupyter Kernel picker
At the end of the creation the python extension displays a mesage indidating this has been selected as the active interpreter
Would it be possible to hide this message and not select the env as the active interpreter when this command is called from another extension (optionally provide a flag or new command to make this possible)

can also make the message more useful, i.e. what we installed from, such as a requirements.txt file.

@brettcannon from jupyters perspective i don't think we'd want the requirements.txt file to be read and installed. The expectation is a new env (blank) is created. optionally we could prompt the user to install stuff from requirement files

it's still another optimization we could make to the flow, in terms of allowing users to choose whether to install the requirements or not. @cwebster-99 did some user studies and one of the patterns we saw is that folks wanted the ability to "customize" what was the command was going to do (https://github.com/microsoft/vscode-python/issues/20264 ). I wonder if there's something we could somewhat similar to the dev containers extension, where each box would be a step in the flow?

from jupyters perspective i don't think we'd want the requirements.txt file to be read and installed.

Why not? Are you seeing users with fully independent notebooks from the rest of the workspace the majority of the time? Personally, when I use notebooks I always have a requirements file to list Jupyter and any dependencies I have in order to make the notebook run.

I wonder if there's something we could somewhat similar to the dev containers extension, where each box would be a step in the flow?

I've been planning this from the start for the command (even polled folks on Twitter about it to find out if we needed multi-select or quick pick), but I apparently forgot to open an issue. 😅 Just created #20277 to track this.

Why not? Are you seeing users with fully independe

i take it back,  i don't have any data to backup such a proposal. and it makes sense to do what you're suggesting

@brettcannon @luabud
Just wanted to clarify, the message displayed today indicates the .venv that was created via the Kernle picker in a notebook ends up getting selected as the active interpeter.
I.e.

Open a notebook
Go into kernel picker
Create a new venv or conda env
At the end of taht process, Python extension displays a message indicatin that env was selected
If you create yet another env (e.g if venv was used earlier, and now you use conda for the same folder), once again Python displays the message

I don't think Python should change the interpretr at this stage, and optionally not display such a mesage.

I can understand not changing the selected interpreter when the environment is created as part of the kernel picker workflow. But given we do select it, I think the message is important to indicate to users that the env creation process has been finished, otherwise they don't know (especially if they don't have a Python file on focus, because then the selected environment won't be displayed in the status bar).
Is the problem here the message itself, or that we auto select it?

But given we do select it, I think the message is important to indicate to users that the env creation process has been finished, otherwise they don't know (especially if they don't have a Python file on focus, because then the selected environment won't be displayed in the status bar).

But doesn't the end of the progress message indicate it has completed
Displaying a message on top of closing the progress message seems like an unnecessary step to indicate the completion.

I personally don't find the progress notification disappearing enough because you don't know if it went away automatically or if it was complete. At least on our user studies most users appreciated it as a signal of completion.

Is there still an ask here since we explained why we think the notification is useful to users?

Sorry, will close this issue.
Makes sense


Ghost-Admin - Upgrade ember-cli-mirage

We're currently sitting on a pretty out-of-date version of ember-cli-mirage, mostly because of the time required to go through the upgrade process to move everything and update for the latest APIs.
Upgrading will mean one less package bringing in old babel versions and speeding up our yarn install.

This is a requirement for #9478 now that v0.4.2 has added support for Ember's new test helpers.

Cool - I’ll make sure to get this done this weekend.


💡 [Feature]: Extend sample gallery with filter by supported hosts

🎯 Aim of the feature
The aim is to provide a filtering option by supported hosts. This is kinda tricky as a single solution may have multiple SPFx components and each may have different hosts defined
📷 Images (if possible) with expected result

🤔 Additional remarks or comments
we could research how it is handled here
https://pnp.github.io/sp-dev-fx-webparts/samples/compatibility/

TBH I think we may drop this. It's the same like we could show a filter option by framework. There is some but still little benefit in such a functionality and still the complexity of introducing such a filter option is quite high not to mention the problem with maintaining it.
So for now I will still leave this opened but set this to neverlog milestone and on hold to await more feedback if such a feature would be helpful

Lets close it as I think it is quite low feature yet would be very complex. We may always reconsider in future


xrdp: enable vsock connections

(first time contributor)
If NixOS is run as a Hyper-V guest, enabling vsock transport allows
to use enhanced session mode when connecting from the host operating
system.
Modification is ported from Microsoft's Ubuntu scripts:
https://github.com/microsoft/linux-vm-tools/blob/4dcc278f842ef1de4cc2c7b24ecb1c048415ff07/ubuntu/18.04/install.sh#L47-L55

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @volth

I noticed that setting the default to virtualisation.hypervGuest.enable might break the setup of people who do use RDP but have not set up enhanceded session mode at the moment (some setup is needed on the windows side of things for vsock to work). Therefore false is a much safer default.


Drop In Folder - Install Automatically?

I'm a bit confused about the drop-in folder.  Under settings it says "In checked, archives downloaded into the selected drop-in folder will be imported once the download is finished."  This seems to indicate that it auto-imports items dropped in.  When I add items to the folder, it shows a star next to the Drop-In Assets button, but I still need to click to install them directly.  Should they be auto-installed?

No they don't, the text is misleading. The auto-appear inside the dropdown menu.
They should not be installed automatically.
Since I'm not a native speaker, here is the is the corrected version:

If checked, new files of the selected drop-in folder will be shown in the menu once the download is finished.

Fixed with 3.10.2


No displaying listings after adding more than 12,000 listings

After adding over 12,000 offers to the local OpenBazzar server, the offers disappeared from the website. The offers are physically on disk (the cache files have not been deleted), but the list of offers in the OpenBazzar program is not visible. By adding fewer offers, everything is displayed correctly.
Has anyone tried to add so many offers to OpenBazzar?
What could be the reason for the disappearing listings?

Hi @Majsin I can't think of any explanation for a limit on the number of listings a store can have (aside from maybe computer memory amount and hard drive size). I can try to investigate when I get some free time.


ComfyUI-Manager

https://github.com/ltdrdata/ComfyUI-Manager
An extremely convenient and pleasant thing to expand the popularity and ease of working with this extension. but I couldn’t figure out how to attach it to SD-Webui-comfyui. Sorry in advance I'm new to this

Sorry, I figured it out. enough in the directory
automatic1111\extensions\sd-webui-comfyui\comfyui_custom_nodes
use
git clone https://github.com/ltdrdata/ComfyUI-Manager

We planned on adding a button to install it with comfyui. Will keep this open to track completion.

You can install ComfyUI-Manager like you normally would in your ComfyUI installation under ComfyUI/custom_nodes.
I wouldn't recommend adding nodes to sd-webui-comfyui/comfyui_custom_nodes, because then you won't load these nodes if you decide to run ComfyUI on its own without the Webui.

I wouldn't recommend adding nodes to sd-webui-comfyui/comfyui_custom_nodes, because then you won't load these nodes if you decide to run ComfyUI on its own without the Webui.

I did not read correctly. This is correct, but also in my experience many extensions cannot work outside of the normal comfyui custom_nodes folder.


In SpeechSynthesizer , how to make  AttachmentReader be a mp3/wave  local file?

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.

Provide the steps to reproduce the issue, if applicable:

Tell us about your environment:

ubuntu 14.04 64bit
What version of the AVS Device SDK are you using?
AVS  <1.2.0>

Tell us what hardware you're using:

[V ] Desktop / Laptop
[ ] Raspberry Pi
[ ] Other - tell us more:

Tell us about your OS (Type & version):

[V ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[ ] Other - tell us more:

You want to play a local file instead of a voice command to Alexa right? SpeechSynthesizerIntegrationTest.cpp does that by sending "recognize_whats_up_test.wav". Check the  function setupMessageWithAttachmentAndSend at line 414 to see how local file is used to send to AVS.

Hi @captainpanther
Can we talk more about your use case with the AVS Device SDK? Please fill out this Contact Form and I will get in touch with you.
Thanks

@captainpanther , check if the solution worked.

i have been fillouted the contact form
2017-12-28 6:05 GMT+08:00 Mradula Nayak<EMAIL_ADDRESS>
@captainpanther https://github.com/captainpanther , check if the
solution worked.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/alexa/avs-device-sdk/issues/414#issuecomment-354184481,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfLeawNahVT2t9UF1NMNSTct1veZayvwks5tEr8OgaJpZM4RMlhJ
.

@captainpanther , if the issue is resolved, please close the ticket. Thanks!


Add rustfmt config

These standardized options can make code a little nicer. Right now they don't change anything, though they include an implicit commitment to adopt the # imports_granularity = "Module" and group_imports = "StdExternalCrate" settings.
While these settings are unstable, we can apply them with rustfmt nightly without actually committing to using nightly in CI. We should progressively move parts of the codebase towards that format in future PRs.

Does the newline_style = "Unix" lint have an impact? Git does the conversion for you anyway, right?


Not up to date information

These steps are not up to date:
"Enable monitoring and configure event logging on an existing VM. You can enable diagnostics at the VM level. To enable diagnostics and then configure event logging, perform the following steps:
Select the VM.
Click Monitoring.
Click Diagnostics.
Set the Status to ON.
Select each diagnostics log that you want to collect.
Click OK."
There is Diagnostics Settings under Monitoring , then it is not Status to turn on and Azure diagnostics in a virtual machine is not set this manner any more.
Please update the article.
Regards, Ilkin

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

ID: d4cd31f8-9ffe-9350-ab99-51bfbb7c66b3
Version Independent ID: 63e89e8b-cb06-2e67-5163-1b345ad522db
Content: Use blob storage for IIS and table storage for events in Azure Monitor - Azure Monitor
Content Source: articles/azure-monitor/platform/azure-storage-iis-table.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @bwren
Microsoft Alias: bwren

@ctahok Thanks for the feedback.
Assigning content owner @bwren for visibility, review and document enhancement consideration if appropriate.

Thanks. We actually just updated this article along with a lot of others for the diagnostics extension.  More updates along the way, mainly for the Linux extension.
#please-close


nerdctl run pull logs the incorrect image name

Description
nerdctl run pull logs the incorrect image name.
Steps to reproduce the issue

nerdctl run pull registry.test:5000/example-docker-buildx-go:v1.6.0

Describe the results you received and expected
This is what the log looks like:
PS C:\Users\vagrant> nerdctl run pull registry.test:5000/example-docker-buildx-go:v1.6.0
docker.io/library/pull:latest: resolving      |--------------------------------------|
elapsed: 0.8 s                 total:   0.0 B (0.0 B/s)
time="2022-07-29T19:34:50+01:00" level=info msg="trying next host" error="pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed" host=registry-1.docker.io
time="2022-07-29T19:34:50+01:00" level=error msg="active check failed" error="context canceled"
time="2022-07-29T19:34:50+01:00" level=fatal msg="failed to resolve reference \"docker.io/library/pull:latest\": pull access denied, repository does not exist or may require authorization: server message: insufficient_scope: authorization failed"

Notice that failed to resolve reference \"docker.io/library/pull:latest\":  should mention the registry.test:5000/example-docker-buildx-go:v1.6.0 image name.
What version of nerdctl are you using?
0.22.0
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
No.
Host information
Windows 2019. But it should not matter which OS is used.

ops. my bad... I had a stray run in nerdctl run pull registry.test:5000/example-docker-buildx-go:v1.6.0.


DO NOT MERGE! Fast Forward stable to f469d41877cad9dce05a763666b24d6f886f4828.

This PR will be merged automatically once checks are successful.

Jenkins job istio-testing/pr-stable passed


[Feature Request] Add toast or alert component

Is your feature request related to a problem? Please describe.
**When i want to make my web app shows up a small alert at the top oe the left of the screen container the information that the user was up to , like: uploading done, loading, warning, error, other small tips ...
so i can't find any solution using your library but to get some extra whole library just for getting a toast or alert component. all this just because your library doesn't contain a tiny and easy component to develop, called toast or alert. !**
Describe the solution you'd like
(I could have used the popover component you provided but that bothers me because I can't control its position over the page ).
Describe alternatives you've considered
Probably it's the most easiest component to develop, just simple toast or alert (loading status, done uploading, warning, waiting, ext...). and it's importing too in any web app I suppose.
Screenshots or Videos

Easy to develop? Accessibility is very difficult and a toast is in the same boat of being difficult. It's not that easy to develop a fully accessible beautiful looking toast component. It is also in the backlog which can be found here.

https://github.com/nextui-org/nextui/issues/2560#issuecomment-2309555174

toast: tracked in https://github.com/nextui-org/nextui/issues/2053
alert: tracked in https://github.com/nextui-org/nextui/issues/2250


Remove pytest-cov upper bound

Works just fine with 2.6.* and 2.7.* versions

Ping?

Hi @Infinisil
Looks good. Right now I have a little bit of a problem with a release, which means that I would rather not merge. Hopefully it will be fixed this weekend and I will pull it next week.

Hmm. Ok, so I did a little digging. This upper restriction was introduced with commit 2b17ca0 in #654 to make sure that continuous integration passes. As we can see, the CI passes for this PR but I would like to ensure that the CI environment actually reflects this change and isn't just running in the old environment.


Update manage-windows-copilot.md

corrected grammatical errors.

article link
https://learn.microsoft.com/en-us/windows/client-management/manage-windows-copilot

Thanks @RAJU2529 for catching those serial commas, but I'm going to close this PR. @mestew is working on an update to the article.


return full uuid for getDeviceUrl

connects to https://github.com/resin-io/resin-cli/issues/388 (does waffle work cross repos?)
I didn't have time to run the tests, cause I have to run, but pushed in case you have a chance to look at it today @jviotti

Looks good, I'll merge once the following items are completed:

Re-build the JavaScript to include latest changes.
Some tests are added. There are a few that specifically test shorter uuid vs longer uuid stuff, so you can adapt one of those.

Let me know if I can help with anything!

Good to merge @jviotti?

Yeah, looks great, will merge in a second. Do you need me to publish this ASAP? Thanks a lot!

Nope, I was just running through my open PRs and wanted get them merged. No rush at all :)

Ok, amazing! I'll try to batch it together with other SDK changes, but if you ever need this, just ping me and I'll publish.


a warning when compiling the newest mxnet

src/ndarray/ndarray.cc: In function ‘void mxnet::CopySliceTo(const mxnet::NDArray&, int, mxnet::index_t, mxnet::index_t, mxnet::NDArray*, int)’:
src/ndarray/ndarray.cc:230:39: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   CHECK(slice_dim < from.shape().ndim())
                                       ^
/home/mingzhang/work/dmlc/mxnet/dmlc-core/include/dmlc/./logging.h:62:9: note: in definition of macro ‘CHECK’
   if (!(x))                                                \

should be fixed after #3547 is merged


Use the correct FileIO for all file interaction

#14 split the file-related methods from TableOperations into a separate FileIO class. The new class is serializable so that it can be sent to other JVMs where tasks run. The Spark integration needs to be updated to send a FileIO instance and use it when opening data files.

@mccheah, I think this is needed for you to hook into all file reads.


Update KOBman with latest release.

Update KOBman with latest release.

@asa1997 Please validate if the KOBman works from the TheKochiOrgBook name space as well

@panickervinod Will do.


Make indicator accessible to screen reader users

When recording samples, we have to look for a visual indicator (a circle IIRC) to know when to start speaking.
I have not found how to know when to speak, nothing changes in the interface (Orca in my case) so I spoke randomly and read the counter to know if there was a recorded sample.
Could it be possible to add something which could be read by screen readers to allow E.G. blind users when to start each sample?

Hi @Nardol!
First of all, thank you for your patience in waiting for a reply to this ticket.
I just wanted to reassure you that I am working to fix this by either adding an aria-label to the circle or adding a visual checkbox as an indicator to know when to speak. I expect it to get rolled out next week and I'd love to get your thoughts on if this works with your hardware.

Nice, thanks for your answer and working on a fix. As soon as I know I can test, I will have a try and make a feedback.

Hi @Nardol
I was going to push this out with some other changes next week but thought I'd get the screen reader change out today. It should now tell you as soon as you are required to speak and when it's done capturing your voice.
I'm keen to get your feedback on this as I want all of our future websites and microsites to be accessible for all.
Thanks again!


Conflicts with web-sys when trying to use egui-wgpu with wgpu 0.12

Hi !
I'm trying to add egui to my renderer and I'm having a conflict of the version of web-sys.
Apparently, wgpu 0.12 asks for ^0.3.53 while egui-wgpu asks 0.3.50 because of having wpu 0.9 dependency. I'm not sure but it looks like the crate of egui-wgpu has not been updated on crates.io ?

To Reproduce
Steps to reproduce the behavior:

Create a crate
Specify : egui-wgpu = "0.0.1-alpha.1 and wgpu = "0.12.0"
Try to compile

Expected behavior
Compilation should finished well
Screenshots

I just published egui-wgpu 0.18.0, so this should be fixed now!


[bug] Missing CC 113-0-Home_Security-* in dashboard

Version
Build/Run method

[x] Docker
[ ] PKG
[ ] Manually built (git clone - npm install - npm run build )

zwavejs2mqtt version: master
Describe the bug
Missing the 113-0-Home_Security-* command classes in the dashboard for motion and door sensors.

Expected behavior
I expect to see 113-0-Home_Security-Motion_sensor_status and 113-0-Home_Security-Cover_status in the dashboard under the User section
Additional context

@chilicheech Could you tell me what's the type of that valueId? I think that type may not be supported here

https://github.com/zwave-js/node-zwave-js/blob/3b5d62549b47eb3c6ca2eba311831ff7d6bb5afb/packages/config/config/notifications.json#L642-L648
https://github.com/zwave-js/node-zwave-js/blob/3b5d62549b47eb3c6ca2eba311831ff7d6bb5afb/packages/config/config/notifications.json#L650-L661
both are numbers

So the type of the valueId is number ? If so it's strange it doesn't appear. @chilicheech Could you check please?

They should be... Maybe they are "any", I gotta check

So here's what z2m/ozw shows in the dashboard:

It has all many more values in the user section than zjs2m/zjs. The way it's organized in a single column is also much nicer, with the proper labels/titles.
I'm rebuilding the image right now. I had to upgrade my rpi so I lost my config. I'll post here as soon as I find out what the type is..

It has many more values in the user section than zjs2m/zjs.

That is not always a good thing. zwave-js tries to hide unnecessary ones. The information for the labels is also there:

It has many more values in the user section than zjs2m/zjs.

That is not always a good thing. zwave-js tries to hide unnecessary ones.

My opinion is different. As a user of zwave I want all values to be exposed to me so I can do what I please with them. That is one of the reasons I stopped using home assistant's zwave integration in favor of z2m/ozw. Home assistant hides values it thinks are unnecessary but that prevents users from hacking/doing what they want. So, I switched to z2m/ozw to gain access to those unnecessary values. Just my opinion.. 😄

Here's the MQTT payload for the motion sensor value:
{
  "id": "24-113-0-Home Security-Motion sensor status",
  "nodeId": 24,
  "commandClass": 113,
  "commandClassName": "Notification",
  "endpoint": 0,
  "property": "Home Security",
  "propertyName": "Home Security",
  "propertyKey": "Motion sensor status",
  "type": "number",
  "readable": true,
  "writeable": false,
  "label": "Motion sensor status",
  "ccSpecific": {
    "notificationType": 7
  },
  "genre": "system",
  "min": 0,
  "max": 255,
  "list": true,
  "states": [
    {
      "text": "idle",
      "value": 0
    },
    {
      "text": "Motion detection",
      "value": 8
    }
  ],
  "value": 0,
  "lastUpdate":<PHONE_NUMBER>879
}

Here's the MQTT payload for the cover value:
{
  "id": "24-113-0-Home Security-Cover status",
  "nodeId": 24,
  "commandClass": 113,
  "commandClassName": "Notification",
  "endpoint": 0,
  "property": "Home Security",
  "propertyName": "Home Security",
  "propertyKey": "Cover status",
  "type": "number",
  "readable": true,
  "writeable": false,
  "label": "Cover status",
  "ccSpecific": {
    "notificationType": 7
  },
  "genre": "system",
  "min": 0,
  "max": 255,
  "list": true,
  "states": [
    {
      "text": "idle",
      "value": 0
    },
    {
      "text": "Tampering, product cover removed",
      "value": 3
    }
  ],
  "lastUpdate":<PHONE_NUMBER>881
}

Both of them report type number.
Cheers.

Home assistant hides values it thinks are unnecessary but that prevents users from hacking/doing what they want

zwave-js doesn't hide values when there's no other (intended) way of accessing them. But some values just don't make sense in a vacuum and would just litter the UI. For example associations (which can be managed with an API), CC versions (which are used internally for checks), alarm type when there is a matching notification, root endpoint values that mirror a non-root endpoint value, etc...
If there's something you are really missing, just let me know and we can discuss.

@chilicheech They should be reported as list as they both have "list": true,. Could you check the chrome debugger console if there are errors?

The way it's organized in a single column is also much nicer, with the proper labels/titles.

I can revert to the old disposition using a single column if you think it was better :)

+1
I guess the missing isn't specific for some brands or products but, just in case: I also miss information reported by @chilicheech for a Fibaro Motion Sensor (FGMS001):

I can revert to the old disposition using a single column if you think it was better :)

My 2 cents: current implementation (several columns) makes harder to find things on devices with a lot of objects. For instance:

One column won't make things easier, though, because the scroll becomes too big but, if I may choose, I would say one column is slighty better on these scenarios.

@katiuskt Could we speak about the best way to show values? I'm open to improve UI but I need some ideas :)

I think grouping related values would go a long way.

@AlCalzone Grouping based on which logic?

E.g. grouping by CC (which is how it is done in ioBroker - although that is a treeview):

In zwave2mqtt, I think having a block of cells for each CC with a matching headline could work.

I will ask in slack what users prefer, I'm +1 for everything

+1 for @AlCalzone suggestion

Opened an issue for that #79 , can this be closed?

Opened an issue for that #79 , can this be closed?

I think this issue and the #79 are different ones.
This case was raised by @chilicheech because Home_Security-Motion_sensor_status and Home_Security-Cover_status information aren't shown into User section of the GUI.
Yesterday I've updated the docker image (dev tag) and still that information was missing for my Motion Sensor (Fibaro FGMS001)
Issue #79 seems to be more related with the way to display UI user values (grouped by CC) in general but it doesn't seem to me it will solve this particular issue. Once solved this issue, thanks to #79 the values will be presented in a better way but it needs to be values to shown for that and right now they aren't :-)

Just in case it helps, in ZWave2MQTT the information for the Fibaro FGMS001 is shown in GUI:

I'm a bit lost here, @chilicheech is this fixed oin latest version? is this something on my side or related to zwavejs? Do you see that value added in console logs and it's not showed on UI ?

@robertsLando , I think the first issue reported by @chilicheech is those 113 objects aren't shown into zwavejs2mqtt "User" section.
In fact, I also expect to find them there (as they are there into Zwave2MQTT, see screenshots above). But taking a look deeper I see they are under "System" section! Is that place the expected one?

hey @robertsLando the issue is as @katiuskt describes. I didn't look under the system section so i missed that they were already there. but, as @katiuskt mentioned, in z2m it shows up under the user section, which makes more sense. it would be nice to move them to the user section in zjs2m as well.

In fact, I also expect to find them there (as they are there into Zwave2MQTT, see screenshots above). But taking a look deeper I see they are under "System" section! Is that place the expected one?

I have submitted a patch yesterday that now groups valueids by cc, check master or use dev tag if you are using docker :)
If this is the only one issue you are expecting I'm 99% sure that it's fixed now. Try master and let me know, if not fixed feel free to reopen it

hey @robertsLando , thanks for your work on this. I built a new container from master and this is what i now see:

So the CCs are all grouped which is very nice. But the main issue still persists. The issue is that we believe the CC 113 should show up in the user section instead of in the system section, since those are motion detection and cover tampering values. Also, the CC 128 should be in the user section as well. That's how it was in z2m 😄 . @katiuskt posted a nice picture of what it looks like in z2m with the CCs 48, 49, 113, 128, and 156 in the user section: https://github.com/zwave-js/zwavejs2mqtt/issues/59#issuecomment-747713500

@chilicheech I'm not really sure that is from muster, them are not grouped by CC.

Ah, i forgot to build the image without using the cache. I was able to build a new one and i now see everything grouped by CC. Looks great!

It would be nice to put them in some sort of order, maybe alphabetical?

I'm not really sure that would improve visualization, what's the benefit?


Added project-Extract mp3 from mp4

What have you Changed -Using this project,you can extract audio(mp3) from a video(mp4) file using Python in 4 lines of code. You need to have the source-code in the same location of the video file, the program will generate a new mp3 file in that location.
Issue no.(must) - #94
Self Check(Tick After Making pull Request)

[x] This issue was assigned to me
[ ] One Change in one Pull Request
[x] My file is in proper folder
[x] I am following clean code and Documentation.
[x] I have added README.md with my script

lint your code proper with flake8..

I want to improve this code, please assign me this

@TechTarun Code is not merged yet. once code will merge you can create an issue for improving code functionality. Assigning a task at the same will create conflict in the master branch.

@Shanu1515 Please lint your code proper with flake8.

Code Quality does not meet up to the requirement. please go through the documentation.
docs:- https://py-contributors.github.io/awesomeScripts/


An option for choosing between LIFO and FIFO ordering

Tokio and Rayon need an option for fair scheduling (processing elements in FIFO order rather than LIFO). While we do support FIFO ordering, our performance is not as good as it could be...
Here's what we've learned over time so far:

crossbeam-deque was initially designed as a LIFO data structure (Deque::push() and Deque::pop() work with one end, while Stealer::steal() works with the other end). Later on, we've added support for FIFO ordering (fair scheduling) through the Deque::steal() method. That means our deque now supports both FIFO and LIFO processing at the same time.

A minor observation: the method Deque::stealer() is not that useful. I've searched for all uses of crossbeam-deque and it seems most users simply do let d = Deque::new(); let s = d.stealer() anyway. The old constructor that returns (Worker<T>, Stealer<T>) was serving us just fine.

There have been multiple requests for steal-half operation (i.e. batched steal). Unfortunately, Chase-Lev deques cannot implement it particularly efficiently, but we can still gain something from batching. By batching multiple steal operations together, we can execute epoch::pin() only once rather within each call to steal(). This should give us some performance wins.

CircBuf by @jeehoonkang is faster than crossbeam-deque because it doesn't attempt to support the Deque::pop() operation at all - i.e. it is a FIFO-only data structure. Moreover, the lack of Deque::pop() allows it to implement a very efficient batched steal operation. @carllerche has expressed interest in using it in Tokio.

I'd like to propose the following:

Let's merge crossbeam-deque and CircBuf into a single data structure, but differentiate between the underlying implementations at runtime ("flavors"). The same idea is used in std::sync::mpsc and crossbeam-channel, where we have two completely different implementations for bounded and unbounded channels but behind the same interface. Java's ForkJoinPool supports LIFO and FIFO very similarly (see here).

Let's tweak the interface a little bit for simplicity.

Let's add support for batched stealing.

Finally, here's the interface I'm proposing:
// `Worker::push()`   - bottom
// `Worker::pop()`    - top
// `Stealer::steal()` - top
fn fifo() -> (Worker<T>, Stealer<T>);

// `Worker::push()`   - bottom
// `Worker::pop()`    - bottom
// `Stealer::steal(). - top
fn lifo() -> (Worker<T>, Stealer<T>);

// Pushes and pops elements.
struct Worker<T>;

// Steals an element.
struct Stealer<T>;

impl<T> Worker<T> {
    // Pushes an element.
    fn push(&self, value: T);

    // Pops an element.
    fn pop(&self) -> Option<T>;

    fn is_empty(&self) -> bool;
    fn len(&self) -> usize;
}

impl<T> Stealer<T> {
    // Steals an element.
    fn steal(&self) -> Steal<T>;

    // Steals a batch of elements.
    fn steal_batch(&self) -> Steal<Batch<T>>;

    fn is_empty(&self) -> bool;
    fn len(&self) -> usize;
}

enum Steal<T> {
    // The queue was empty.
    Empty,

    // Some data was stolen.
    Data(T),

    // The operation should be retried.
    Retry,
}

// A batch of elements.
//
// Internally, this should probably be just a wrapper around
// `SmallVec::<[T; 16]>::IntoIter`, or something similar.
struct Batch<T>;

impl<T> Iterator for Batch<T> {
    type Item = T;
}

The advantages of this design are:

By choosing LIFO or FIFO strategy at construction (functions lifo() and fifo()), we can internally use the most optimal implementation (Chase-Lev vs CircBuf) for methods push, pop, steal, and steal_batch.

The interface is simple. One can easily choose the appropriate scheduling strategy (FIFO or LIFO) at runtime - we have a single Worker and Stealer type for both. This will be quite handy in Rayon.

What do you think?
cc @carllerche @jeehoonkang

I would also like to add that CircBuf not having any fences is also a selling point to me. I've been hitting TSAN warnings that I believe are false positives, but I am not 100% sure.
Re: merging within one structure, how do you differentiate the implementations internally? A condition or dyn dispatch? I assume you have measured the performance implications vs. just having separate data structures?

Just writing down another idea for the interface of steal_batch:
impl<T> Stealer<T> {
    fn steal_batch<E: Extend<T>>(container: &mut E);
}

This way we don't have to introduce struct Batch nor allocate needlessly when calling steal_batch.


Hi！after i built this,i met this error when i runtime

How  can i fix it?

Thank you! Your driver doesn't support some extensions. Please update the driver.

well... please tell me more info about GPU and driver?
I think you don't need any configuration.

my GPU is Rtx3060 ,and my GPU Driver is 535.98

This repo doesn't use DirectX, please install latest VulkanSDK.

yep, i have installed the latest VulkanSDK and configure in the project ,but its still have this error


OLS-447: Unit test: default params to None

Description
Unit test: default params to None
Similar to tests we have for other providers already
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[x] Unit test

Related Tickets & Documents

Related Issue #OLS-447
Closes #

/retest

/lgtm

/approve


redefine filtering of "autosome none n-gap" region as a whitlelisted collection of segments

Currently  filtering of "autosome none n-gap" regions is done by (1) hard-coded susetting of (input bed) lines starting with chr[1..22] and (2) filtering a black-listed collection of gap and n-regions segments :
https://github.com/c-BIG/NPM-sample-qc/blob/b49dbed233bd2c018416f8c229ae8ab08bd459d4/main.nf#L158a
It'd be better / cleaner to  filter "autosome none n-gap" regions by filtering against a white-listed collection of autosome and none gap and n-regions segments... something like :
zcat "${biosample_id}.regions.bed.gz" | bedtools intersect -v -a stdin -b ${autosome-non-gap_bed} | gzip -9c > "${biosample_id}.regions.autosomes_filter_n_bases.bed.gz"
To that effect :

 ${autosome-non-gap_bed} is to be pulled from https://github.com/c-BIG/NPM-sample-qc/resources/
A chapter in  https://github.com/c-BIG/NPM-sample-qc/README.rst

Dropped gap regions file and added whitelisted autosomes region bed file https://github.com/c-BIG/NPM-sample-qc/blob/feature/dsl2-v3/resources/autosomes_non_gap_regions.bed for the mosdepth_datamashprocess.
The gap regions file originally downloaded from http://hgdownload.soe.ucsc.edu/goldenPath/hg38/database/gap.txt.gz and processed with autosomes.bed, which is sorted bed format start and end of the autosomes.
zcat gap.txt.gz | cut -f2-4 - | egrep -v '_|-|X|Y' | sort -k1,1V -k2,2n > gap_regions.bed
bedtools subtract -a auotsomes.bed -b gap_regions.bed >autosomes_non_gap_regions.bed

Details updated in the https://github.com/c-BIG/NPM-sample-qc/README.rst


Making the configuration file optional

Right now, it seems that a configuration file is created whenever the program is run. However, sometimes the configurations used are not intended to be reused. Also, it can contain sensitive information such as the password. Therefore I believe it is better if the file was made optional.

@Farzat07 will mark it as future enhancement.

@Farzat07 will mark it as future enhancement.


Impossible to build to root folder whereas I disable "delete-ouput-path" option

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions.
@angular/cli: 1.2.1
node: 6.11.0
os: win32 x64
Repro steps.
After upgrading anglular/cli to version 1.2.1, I could not build my application anymore. Before migration I used version 1.0.6.
Now when I build my application, I get this error message : "Output path MUST not be project root directory!". It's true.. my output path is root BUT I disable "delete-output-path" option.
Desired functionality.
When "delete-output-path" option is disabled, cli should accept to use root path.

May I ask what the reason behind building to root is? That sounds really weird as it would clutter up your root and it sounds really hard to clean.

I started my application with Angular 2 beta 8 based on Umbraco wich is an ASP.Net CMS. Before version 1.2.1 of angular/cli there was no problem with this architecture. The principal reason to build to root is only it was working before ;).
I build to root only for dev. For production, I build to "dist" folder and I don't disable "delete-output-path" option.

I don't think this behaviour is going to change. You'll be better of using a build path and change your server to serve from that instead of root.

Very bad news for me (and certainly for some others). I have to change my development architecture while this one works perfectly for more than 1 year.
To save me you've just to had a test on line 17 of file build.ts :
if (this.project.root === path.resolve(outputPath)) {
becomes
if (this.project.root === path.resolve(outputPath) && runTaskOptions.deleteOutputPath) {
Why prevent deployment on root if you do not delete the contents of the folder?

Feel free to submit a PR for this behaviour.
Please include a warning though to inform the user that they're doing something potentially unsafe.

I have bypassed the problem by reorganizing the folders of my application.
I close the issue.


[Bullet] InputEvent in StaticBody always return index 0

Godot version:

Mono 3.2.3 and 3.2.4
OS/device including version:

Win 10 64
Issue description:

InputEvent in StaticBody always return 0 index shape for ConcavePolygonShape...
Minimal reproduction project:

Example.zip

This is a known issue with Bullet and not specific to Mono, it doesn't happen with Godot Physics.
See #39767.
MRP with gdscript:
ConcavePolygonShapeIndex.zip
In 3.2.3, it actually returns the triangle index instead of shape index when using Bullet.
PR #42882 enforces indices to be 0 (instead of triangle indices) and documents this limitation with Bullet, but it would be nice to fix it (probably upstream).
CC @AndreaCatania

This is a known issue with Bullet and not specific to Mono, it doesn't happen with Godot Physics.
See #39767.
MRP with gdscript:
ConcavePolygonShapeIndex.zip
In 3.2.3, it actually returns the triangle index instead of shape index when using Bullet.
PR #42882 enforces indices to be 0 (instead of triangle indices) and documents this limitation with Bullet, but it would be nice to fix it (probably upstream).
CC @AndreaCatania


Task/CMPS 57

fixed mobile view information release and calendar UI

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

621 of 1286 (48.29%)  changed or added relevant lines in 30 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-20.3%) to 48.585%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

compass/clients/init.py
3
4
75.0%

compass/clients/mock_person_client.py
3
4
75.0%

compass/templatetags/vite.py
40
41
97.56%

compass/urls.py
15
16
93.75%

compass/views/api/access_group.py
6
9
66.67%

compass/views/api/program.py
7
10
70.0%

compass/views/api/support.py
7
11
63.64%

compass/tests/init.py
13
18
72.22%

compass/dao/group.py
9
17
52.94%

compass/views/pages.py
9
17
52.94%

Files with Coverage Reduction
New Missed Lines
%

compass/context_processors.py
3
0%

Totals

Change from base Build<PHONE_NUMBER>:
-20.3%

Covered Lines:
635

Relevant Lines:
1307

💛  - Coveralls


crashes when I repeatedly press and dismiss the sheet

When I repeatedly press and dismiss the sheet, it crashes and throws error "RouterView.swift:483: Fatal error: Did dismiss pushed screen but could not find new root screen"

I noticed that the Sample project also encounters a similar issue when pressing and continuously dismissing the sheet

Hi @xqsadness I'm still not seeing how to reproduce this. You're saying to open the app and click "dismiss screen" 100x?

Hi @SwiftfulThinking I think you need to open the app and click "dismiss screen" 10-20 times, or possibly fewer, to reproduce the issue.
And here is the video I recorded:
"https://drive.google.com/file/d/1T47elnTUTXtLqB-MnkQlpcEYzLU_Im6D/view?usp=sharing"


znap status doesn't work when using znap eval for Zoxide

Describe the bug
When I add this line in my .zshrc
znap eval zoxide 'zoxide init zsh --cmd cd'

znap status gives the following errors:
~ ❯ znap status
zoxide: zoxide: no match foundno match found

""zszsh-snap": hNo such file or directory- (os error a2u)t
osuggestions": No such file or directory (os error 2)
 zoxide: no match found
zoxide: no match found
 %

Steps To Reproduce
As mentioned above
Expected behavior
znap status should be able to list the repos
Environment:

OS: macOS
Terminal: Kitty
Zsh version: 5.9
Repo version: 97e9904

Additional context
Errors are gone if I switch back to the plain eval
eval "$(zoxide init zsh --cmd cd)"

I wonder if this is because the way I'm using znap eval is incorrect and that leads to the zoxide not being correctly initialized?

I have the same problem. Switching back to  eval "$(zoxide init zsh --cmd cd)" didn't fix my problem.

It turns out this is probably because I'm binding zoxide to cd

That is indeed the problem. Try moving znap eval zoxide 'zoxide init zsh --cmd cd' to the very end of your .zshrc. Does that solve the issue?

I think I figured out a workaround that is otherwise useful, too: Znap could defer all alias definitions until the command line has been loaded.


Boilerplate update to latest version

What type of PR is this?
Maintenance
What this PR does / why we need it?
Updates to the latest boilerplate so that we can inherit some necessary fixes.

/lgtm


Create DNNL conda package

To use DNNL in SINGA, it would be convenient to have a DNNL conda package.
However, Intel does not provide the conda package for DNNL.
Therefore, we may need to compile DNNL via conda and upload it to anaconda cloud.

This is uploaded https://anaconda.org/nusdbsystem/dnnl


Possible fix for object type error

This setVerifyCredentialsCallback in the OAuth2ServerServiceProvider class is not happy with the password grant passed in because of the callback function taken from the website placed in the oauth2 config file.
Currently passing this grant type:
http://laravel.io/bin/BLwBJ
produces this error when authenticating:
http://laravel.io/bin/1y6rv
I am getting a successful authentication along with the access token with the modification made below.

You're on the dev branch, I've decided to remove that callback to allow for php artisan config:cache to work correctly. It is not a best practice to have logic inside your config files. The new way of doing it is described in the wiki: https://github.com/lucadegasperi/oauth2-server-laravel/wiki/Implementing-an-Authorization-Server-with-the-Password-Grant


Add AlphaWallet connector

Adding AlphaWallet as an option for onboard interface.

@JamesSmartCell thanks for this! Taking a look 👀

@JamesSmartCell Also if you could update this branch with develop, thanks!

Hi @taylorjdawson thanks for checking our PR :) I merged and applied formatting as directed, can you re-check? Thanks.

Looks good! 👍