Published Javadoc has broken search

Describe the bug
Javadoc linked on the main page does not allow to search.
Test case reproducing the bug

Go to https://assertj.github.io/doc/#assertj-core-javadoc and click the link (http://www.javadoc.io/doc/org.assertj/assertj-core/)
Start typing "ObjectAssert"; window focus should be on the "search" input in the top-right corner, if it's not, click the input
Try to use keyboard (Enter key) or mouse to select any of search results

Expected:

you can open a search result like on https://docs.oracle.com/en/java/javase/17/docs/api/

Actual:

no way to open search results; they appear but are not clickable and Enter key pressed in the search input has no effect

It seems it's a local issue with cookies, clearing them should fix it.
Related: maxcellent/javadoc.io#97

I tried opening JavaDoc in a private window of Chrome (102.0.5005.115) and Firefox (101.0.1) and still nothing works. I think this simulates a case of a new visitor with no cookies opening JavaDoc. Have you even tried if this works for you?

Yes, I did exactly these tries yesterday: https://github.com/maxcellent/javadoc.io/issues/97#issuecomment-1164192984

@pkubowicz @lkucik and even after clearing cookies for javadoc.io, it still doesn't work, right?

it works for me with ffox 101.0.1 (64-bit) on linux

and even after clearing cookies for javadoc.io, it still doesn't work, right?

I don't need to clear cookies. I checked Firefox web inspector and it shows no cookies for domain when opening the page in a private window. I suspect private mode in Firefox rejects all cookies.

I just reproduced this on a different computer. I cleared cookies in the normal mode (although I had no cookies as I have probably never visisted javadoc.io on this machine). When clicking a search result I see an error in JS console:
Uncaught TypeError: parent.classFrame is undefined
    select https://www.javadoc.io/static/org.assertj/assertj-core/3.23.1/search.js:346
    jQuery 37
    _create https://www.javadoc.io/static/org.assertj/assertj-core/3.23.1/search.js:124
    jQuery 7
    <anonymous> https://www.javadoc.io/static/org.assertj/assertj-core/3.23.1/search.js:294
    jQuery 13

I suspect the problem may be in your browser: you can have outdated JS files that work, I load fresh JS files and they are broken.

As mentioned in https://github.com/maxcellent/javadoc.io/issues/97#issuecomment-1165695636, there is an ongoing migration activity on javadoc.io, and not all the versions are working properly. The migration may take another two weeks to complete.

From https://github.com/maxcellent/javadoc.io/issues/97#issuecomment-1165729049:

so this is likely a caching issue from CDN. I have purged everything. Now if you do Ctrl + refresh it should work now.
I have test both Chrome and Firefox against latest version.

@pkubowicz @lkucik could you please check again? I've tried it and it looks good on my side.

Closing as the issue seems to be fixed now. @pkubowicz @lkucik please let us know if this is not the case for you.

Works fine now, thank you.


Emote window bugs

Describe the bug

First line of emotes are shifted a bit on every tab in emote window
Every restart emote window size gets reset (same as in #351)

To Reproduce
Expected behavior

Emotes having the same margin from both sides of a window
Emote window position/size should be remembered same as main window is.

Screenshots
1.

Chatterino version
Chatterino Nightly 2.1.5 (commit 367b1970f, 22.09.2019)

Two things I would appreciate, to make fixing the issue easier for everyone:

Please create a separate issue for the second bug you described (one bug per GitHub issue please 🙃)
Then update the issue title to First line of emotes are shifted a bit in emote window or something similar to that, so the issue title is more descriptive.


Outdated information on global DNS cache

Please update Name resolving, Global DNS cache is bad section. This functionality has been removed.

Thanks for pointing this out @i-ky !


Adding support for autocompletion of operators schema

This PR mainly does two things -

Brings source tables from server and displays it as options in source operators.
propagates schema forward as user creates workflows.

To do these, a service called Autocomplete Service has been created which does the following with the help of other services and components: -
When the service created, it has an observable which beings source table names from the backend and applies them to the operator metadata of the source operators. This leads to the creation of new operator schema which is consumed by property editor component.
An operator's input schema can be changed when a workflow is changed in these ways - link creation/deletion/edit, property change by any operator like adding output attribute by sentiment operator. Therefore, in both of these cases, an invokeAutocomplete API is called which brings schema information for each operator from the backend and stores it in a data member in the autocomplete class. How is this schema information used? Whenever the property editor component shows the properties, it checks if the autocomplete schema info contains info about this operator. If yes, then it edits the schema to include that info and then that schema is passed to the JSONtoForm library that we are using to show the property editor.
Another implementation detail is that invokeAutocompleteAPI() method takes a boolean argument. This argument is used to decide whether the property editor is to be explicitly notified when the new schema is fetched from the backend. So, consider two cases: -

A link is created/deleted/edited - Say the workflow contains source, keyword and result, but no links between them. Also, say that source operator's filename has already been selected and the user has now selected the keyword operator. He sees that keyword operator has no schema and decides to create a link between source and keyword. So, in this case, when the new schema is brought, the property editor needs to reload the currentOperatorSchema. So, we pass true to the invokeAutocompleteAPI() method.
An operator's property is changed - In this case, we needn't reload the schema of the current operator as there will be no change in its input schema. When another operator is selected after this, its schema will anyways be according to the information fetched from the backend (which is the default behavior). So, in this case, the default behavior will work fine and we don't need to explicitly tell property editor about new schema. So, we pass false to the method.

Yes, currently each call to invokeAutocompleteAPI calls backend. It can be optimized like we don't need to call if a link is getting deleted and we just do everything on the frontend, but that isn't there right now.

After using the autocomplete branch for some testing, I realize that when user is editing the property and a link is removed (can be any link after the current operator), the changes that the user made for current operator's property will revert back to the state before the user makes any change.
Steps to recreate this scenario:

Create Scan:Source Operator, Keyword Search Operator, and View Results Operator and connect them together.
Set Scan:Source Operator's property to use twitter_sample table
Set Keyword Search Operator's property to search query related to trump and select text for attributes. Stay at Keyword Search Operator without clicking any other operators.
Remove the link between Keyword Search and View Results. You can see that the property we edit on step 3 has been refreshed.

Based on the new comments, a few changes have been added to this PR:

2 addition methods handleOperatorAddEvent and handleOperatorDeleteEvent have been added to listen to operator add / delete event.

A new library ajv has been added to validate the json schema. Here is the link to the library https://github.com/epoberezkin/ajv. According to its github, this library is popular among the developers (with 4554 stars), is using the MIT license, and has good performance among all the json schema validator library.

When handleExecuteResult is triggered, 2 things can happen:

If the new input attributes propagated from autocomplete API does not contain attributes for the existing operators in the dynamicSchemaMap, then update dynamicSchemaMap to map operatorID to the default operator schema in operatorSchemaList. Also, trigger the operatorSchemaChangedStream event so that the data in the workflow graph can be updated.
If the new input attributes contain attributes for the existing operators but have different operator schema, update dynamicSchemaMap to map operatorID to the new schema and trigger the operatorSchemaChangedStream event.

In PropertyEditorComponent, when the schema event is triggered, it will fetch the dynamic schema using getDynamicSchema and validate the existing operator properties using ajv: Another JSON Schema Validator library.

if the operator that has a schema changed equals to the current operator displaying in the property panel and ajv calls show the operator properties are still valid in the new schema, reload the properties to the UI, otherwise, set the properties to be empty.
if schema changed operator does not equal to the current operator, if json schema is invalid, set the properties to be empty


Add dependency on app-tools-lib-for-java

We're going to need this for a lot of things so I'm splitting it out into an issue of its own:
https://github.com/GoogleCloudPlatform/app-tools-lib-for-java/
Currently in maven central snapshot but not eclipse repositories

I started looking at this for #9.  For now we can import it using the Import Existing Maven project and it should do the right thing.

By "right thing" do you mean update all the various manifest and pom.xml files as needed to include the dependency or simply include the dependency locally in Eclipse?

The app-tools-for-java uses the maven-bundle-plugin to add OSGi metadata to generated .jar.  m2e knows how to generate and populate a MANIFEST.MF so that it will be managed by PDE.

Shouldn't this be linked in to the local run milestone?

We actually need it for pretty much everything: local run and debug, new application, and deploy. Github doesn't let us assign it more than one milestone though. :-(
@briandealwis This is fixed now, right?

Ah ok. I see. I didn't realize it was already assigned to a milestone. My bad!

Fixed by #215


Add comments for k8s volume metrics

Looks good to me. I was planning to add an examples directory with all possible values.yaml use cases. This one is a good candidate to be added there. Feel free to submit another PR for that.


Laravel 8.x Compatibility

This is an automated pull request from Shift to update your package code and dependencies to be compatible with Laravel 8.x.
Before merging, you need to:

Checkout the l8-compatibility branch
Review all comments for additional changes
Thoroughly test your package

If you do find an issue, please report it by commenting on this PR to help improve future automation.

:warning: Shift detected GitHub Actions which run jobs using a version matrix. Shift attempted to update your configuration for Laravel 8. However, you should review these changes to ensure the desired combination of versions are built for your package.

:warning: Shift detected a dependency for the laravel/framework. Unless you have a special requirement for depending on the entire Laravel framework, it's recommended to add dependencies for the specific illuminate packages instead.

:alembic: Using this package? If you would like to help test these changes or believe them to be compatible, you may update your project to reference this branch.
To do so, temporarily add Shift's fork to the repositories property of your composer.json:
{
    "repositories": [
        {
            "type": "vcs",
            "url": "https://github.com/laravel-shift/teamwork.git"
        }
    ]
}

Then update your dependency constraint to reference this branch:
{
    "require": {
        "mpociot/teamwork": "dev-l8-compatibility",
    }
}

Finally, run: composer update

Who did trigger this PR?
There are some points I'd like to change. I have commented these.


Error: UMD and IIFE output formats are not supported for code-splitting builds caused by DatePicker

Really enjoying the framework so far, but one of the key reasons I chose it, the DatePicker, is causing me some problems. When I import it into a file, my project starts failing to compile with Error: UMD and IIFE output formats are not supported for code-splitting builds.
I set up the svelte project with degit - packages installed:
  "devDependencies": {
    "@rollup/plugin-commonjs": "^17.0.0",
    "@rollup/plugin-json": "^4.1.0",
    "@rollup/plugin-node-resolve": "^11.0.0",
    "carbon-components-svelte": "^0.42.2",
    "rollup": "^2.3.4",
    "rollup-plugin-css-only": "^3.1.0",
    "rollup-plugin-livereload": "^2.0.0",
    "rollup-plugin-svelte": "^7.0.0",
    "rollup-plugin-terser": "^7.0.0",
    "svelte": "^3.38.3"
  },
  "dependencies": {
    "carbon-icons-svelte": "^10.36.0",
    "firebase": "^8.7.1",
    "rxfire": "^4.0.0",
    "rxjs": "^7.2.0",
    "sirv-cli": "^1.0.0"
  }

Would appreciate any help with getting this working.

Node v12, npm v6

Try adding inlineDynamicImports: true in your exported rollup.config.js.

This fixed it thank you so much!


⚠ BCR Bot has degraded performance

In ae26d43, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 1124 ms

Resolved: BCR Bot performance has improved in 7590723.


Feature/repairkafka3.7

[ ] If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes #.
[x] Update the CHANGES log.

Please explain what is changing here, and if this is a plugin update, you need to update test scenarios too.

If the new use cases are same here, you should add more versions to support your new implementations for newer versions, and keep compatibility with previous.

ase explain what is changing here, and if th

If the new use cases are same here, you should add more versions to support your new implementations for newer versions, and keep compatibility with previous.

Ok, I'll commit later

About https://github.com/apache/skywalking-java/pull/708#issuecomment-2242063529 I mean the test versions
demo codes https://github.com/apache/skywalking-java/tree/main/test/plugin/scenarios/kafka-scenario
tested versions https://github.com/apache/skywalking-java/blob/main/test/plugin/scenarios/kafka-scenario/support-version.list

Please pick one for 3.4, 3.5, 3.6 as well.

Please pick one for 3.4, 3.5, 3.6 as well.

3.4, 3.5, 3.6 are same to 3.2.3

Logically same. But we should test.

Logically same. But we should test.

OK, Should I create modules kafka-3.6.x-scenario and kafka-3.7.x-scenario like kafka-2.3.x-scenario in dir /test?

Logically same. But we should test.

OK, Should I create modules kafka-3.6.x-scenario and kafka-3.7.x-scenario like kafka-2.3.x-scenario in dir /test?

Those exist because the test codes are different. If you just need to change dependency versions, you don't need to do that.

And please fix CIs.

You should verify all locally rather than costing too much on CI.

You should verify all locally rather than costing too much on CI.
Sry, it's my first time to create PR in github, I'm doing test CI in my private branch now

--- /home/runner/work/skywalking-java/skywalking-java/tools/plugin/md-javaagent-plugin-list.txt	2024-07-25 03:04:25.186078041 +0000
+++ /home/runner/work/skywalking-java/skywalking-java/tools/plugin/genernate-javaagent-plugin-list.txt	2024-07-25 03:04:25.178078015 +0000

I think you did not update docs accordingly, Plugin-list.md and Supported-list.md

--- /home/runner/work/skywalking-java/skywalking-java/tools/plugin/md-javaagent-plugin-list.txt	2024-07-25 03:04:25.186078041 +0000
+++ /home/runner/work/skywalking-java/skywalking-java/tools/plugin/genernate-javaagent-plugin-list.txt	2024-07-25 03:04:25.178078015 +0000

I think you did not update docs accordingly, Plugin-list.md and Supported-list.md

--- /home/runner/work/skywalking-java/skywalking-java/tools/plugin/md-javaagent-plugin-list.txt	2024-07-25 03:04:25.186078041 +0000
+++ /home/runner/work/skywalking-java/skywalking-java/tools/plugin/genernate-javaagent-plugin-list.txt	2024-07-25 03:04:25.178078015 +0000

I think you did not update docs accordingly, Plugin-list.md and Supported-list.md

And please fix CIs.

And please fix CIs.

And please fix CIs.

All the CIs were successfully executed but the DeadLink check

@kezhenxu94 As 403 respond code, should we add this into ignore list?

One more. changes.md should be updated as well.

One more. changes.md should be updated as well.
Changed


modified script

add grocery shop data for 2/april/2021

No conflict with the base branch. Okay to merge.


Added label to checkbox

Mostly checkboxes are accompanied by labels. This feature seems to be missing from the current implementation.

Hi, and thanks for the work!
In general it should be easy to achieve the same using a horizontal LinearLayout with a checkbox and a label. Among other things, this naturally let users chose to put the label to the left or right of the checkbox, and supports styled text.

Yes, I agree with what you're saying.


Create fake /proc/stat to make unpached htop work

That's an interesting approach.
What I can criticize about the implementation is that it can mask a working /proc/stat if login is running under root. Could you condition masking it on unreadability of /proc/stat? Also maybe suffix the fake filename with -fake for extra clarity.

Please review the new changes.

Hm, indeed, that should be enough if you don't need it non-root only =)

Please review the new changes.

Yeah, good thinking. It was also frozen at an old version.

Removed in c1b61fc (should be trivial enough to not warrant a review).


Fixes #13. Update to CDTDatastore version 1.0.

Fixed by migrating to version 1.0 of CDTDatastore.  Bumped the pod version accordingly.

Does s.osx.deployment_target in the podspec need to change like you changed the Podfile osx target? Otherwise +1

Good catch @tfrank64 !  I will fix and then merge.


Fix: comments in files (#86)

Fix: comments in files (#86)

Is that all we need to fix the commenting issue?  Opened PR since 3 months :(

Thanks @danbord for the work around for the time being, would be great if this could be merged 🙏


[Content] Corrected (and simpliffied) Booter Quirks list

Fixed some Booter Quirks for Intel CPUs. Also reduced the list by merging Quirks for 2nd to 8th Gen into one since these CPUs all require the same Booter Quirks.
Just replace Booter-Quirks.txt in /Database/preset
Booter-Quirks.txt

Have been updated, find any questions or have any ideas, please feel free to leave a message, thank you.


Question. Values from durable object

I can't access the values from the durable object.
They become visible only in the response
You have such an example
.get('/', ({ Counter }) => Counter.get('test').toJSON())
I have something like this
 .get('/', async ({ Counter }) => {          const state =  await Counter.get('test').toJSON());          console.log(JSON.stringify(state), state);          return json(state);      }
the response in the cloudflare worker dashboard.

 "message": [
        "{}",
        "[object Object]"
      ],

but the response in the browser is fine, I can see all the values of the object.

So if you don’t use the parse flag (autoParse?) on the withDurables middleware, the response back from a DO method (e.g. toJSON) will be a Response, not the raw data. So you wouldn’t want to stringify it. Your final json method is ignoring it because it’s already a valid Response, so it still return just fine in the end.~KevinOn Jul 23, 2023, at 00:46, Nikita Pilgrim @.***> wrote:
I can't access the values from the durable object.
They become visible only in the response
You have such an example
.get('/', ({ Counter }) => Counter.get('test').toJSON())
I have something like this
.get('/', async ({ Counter }) => { const state =  await Counter.get('test').toJSON()); console.log(JSON.stringify(state), state); return json(state); }
the response in the cloudflare worker dashboard.
"message": [
"{}",
"[object Object]"
],
but the response in the browser is fine, I can see all the values of the object.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>


Use Fog DNSimple API Auth

The Fog Gem now supports DNSimple API Auth here.
For now the cookbook could use the latest fog gem until the cookbook can be changed to the dnsimple 2.0 Ruby gem mentioned on GH Issue #24

Also seems to working for token domains, so long as :dnsimple_domain is passed in as well. I assume we could pass this in from the resource if only the token is supplied, and fog would carry on appropriately.

Or wait, according the link in the OP, we can pass them all in, and so long as there's no password or email set, then it will work out.

Would be nice to get some movement on this. I accidentally just re-implemented @pwelch's PR. (mine doesn't have breaking changes, fwiw)

Friendly ping for @martinisoft @therobot

To simplify things I am going to close this issue since its somewhat of a duplicate of #24 and we have two pull requests to try and get this shipped.


FEAT: Same Gmail account can now be registered to different user Cells

Closes #54

At Office365, same account can be registered to different user Cells without "prompt=consent".
So, I think that option add in this line.
if (params.srcType == "Google") {
    paramsStr = paramsStr + "&access_type=offline&prompt=consent";
}                                                                                ^^^^^^^^^^^^^


DOC: stats.pearsonr: typo in coefficient

What does this implement/fix?
There was a typo in the documentation for the stats.pearsonr function. "coefficent" is now changed to "coefficient".

For future reference please see https://scipy.github.io/devdocs/dev/contributor/rendering_documentation.html, specifically the section relating to skipping CI

Ark sorry @tylerjereddy i am seeing the PR was made against maintenance instead of main. Feel free to revert.

no worries in this case because it is so minor, I guess you can just forward port; this does happen by accident once or twice a year, I agree it is easy to miss

Well apparently it was already on main in #17016 so we are good.

Thank you @wenig. Next time please try to group more changes. We don't encourage single typo-like PRs.
Just a question, is this PR linked to hacktoberfest?

I see, will keep that in mind! No it's not linked to hacktoberfest. I have just read the docu for the pearsonr function and stumbled across the typo.


Update free-courses-en.md

What does this PR do?
Add resource(s)
For resources
Description
Added an awesome youtube video that explains self supervised learning in computer vision
Why is this valuable (or not)?
How do we know it's really free?
Its a youtube video.
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

We don't allow shortened links. Please read CONTRIBUTING.md guidelines!
For youtube, we prefer "playlist" links and the name of the producer/creator. https://github.com/EbookFoundation/free-programming-books/blob/master/CONTRIBUTING.md#platforms-and-access-notes


saving file as png saves the contents as jpg

in coypu v3
when calling SaveScreenshot with filename sample.png, the contents of the file are actual jpg ie the mime type is image/jpg instead of image/png

this method doesn't strip out the leading dot in the file extension. the format will always be Jpeg

    internal static ScreenshotImageFormat GetImageFormat(string fileName)

    {
        var extension = new FileInfo(fileName).Extension.ToLower()
                                              .Replace("jpg", "jpeg");

        ScreenshotImageFormat format;
        if (AreEqual(extension, ScreenshotImageFormat.Bmp))
            format = ScreenshotImageFormat.Bmp;
        else if (AreEqual(extension, ScreenshotImageFormat.Gif))
            format = ScreenshotImageFormat.Gif;
        else if (AreEqual(extension, ScreenshotImageFormat.Jpeg))
            format = ScreenshotImageFormat.Jpeg;
        else if (AreEqual(extension, ScreenshotImageFormat.Png))
            format = ScreenshotImageFormat.Png;
        else if (AreEqual(extension, ScreenshotImageFormat.Bmp))
            format = ScreenshotImageFormat.Bmp;
        else
            format = ScreenshotImageFormat.Jpeg;

        return format;
    }

Hi @joshski @adiel ,
I have a fix for this issue as well as a test case for it.  How do i go about submitting it.
Thanks
B

i figured it out . see pull request #204

@basimc4 thanks, I will take a look on you PR

I just wanted to add that WebDriver on .NET Core only supports PNG image format.  Since PNG files are being unintentionally saved as JPEG image format, that causes WebDriverException every time.  Workaround is to bypass BrowserSession and use native WebDriver ITakesScreenshot functionality.

@basimc4
sorry, I can't reproduce it your issue, I've added a test for PNG and when debugging - the code in ImageFormatParser.cs works correctly

@obstar can this issue please be reopened? I also just ran into it too.
I'm using Coypu in a .NET Core test suite which only supports PNG files. I have my file extension listed as .png, but am getting the following error:

.NET Core does not support image manipulation, so only Portable Network Graphics (PNG) format is supported

This is just like @basimc4  described.
var filename = "filename.png";
var extension = new FileInfo(fileName).Extension; //This will return .png

Coypu's ImageFormatParse.AreEqual() function is:
private static bool AreEqual(string extension, ScreenshotImageFormat imageFormat)
{
    return extension.Equals(imageFormat.ToString(),        //This will be ".png"
            StringComparison.InvariantCultureIgnoreCase);  //This will be "png"
}

@berckoi ok, I will go back to it with proper .net core project

Thanks for the quick turn around @obstar! I updated to Coypu 3.1.1 and can confirm that my screenshots are generating flawlessly!

@berckoi np


Image & Text Clipping in Teams Panel

Describe the bug
Image clips with text inside team panel, also occurs in wider panels when teams have longer names.
Steps To Reproduce
Narrow ish panels for favourited teams on main page
Screenshots

Browser

OS: Windows 11
Browser: Edge Chromium Backend

Additional context
Just need a better way to render text and the logo.

Seems to not be occurring any longer. Closing issue for now.


Removed unnecessary extra logout page

This PR addresses issue #1094. The extra logout page has been removed so that when the user logs out, they are presented with an appropriate link to go back to the defined chaiseConfig property logoutURL. If it is undefined, it will go to the root page.

Good catch. I will update the config doc.


Username generator

When registering the system will give the user a username, that will be firstname.lastname, but only if the username does not exist yet.
Write an algorithm which generates a username in case the default pattern is taken by another user. This is based on the first name and last name, so if username firstname.lastname already exists it would use firstname.lastname1 or something like this. The user would be able to change this later on his profile/settings.
DRI @dmorina @neilthemathguy

This idea comes from a lazy guy, but couldn’t the username be the email address?
On May 23, 2015 at 2:28:13 AM, Durim Morina<EMAIL_ADDRESS>wrote:
When registering the system will give the user a username, that will be firstname.lastname, but only if the username does not exist yet.
Write an algorithm which generates a username in case the default pattern is taken by another user. This is based on the first name and last name, so if username firstname.lastname already exists it would use firstname.lastname1 or something like this. The user would be able to change this later on his profile/settings.
—
Reply to this email directly or view it on GitHub.

Couldn't they pick their own usernames?

@Soroosh129 currently if the username is taken then the email is used as a username.
we want usernames because emails are not good to be used in url lookups
@angelamayx they could do that while registering but I think we should keep registration form as short as possible, they will be able to change it later anyways..

@dmorina Purpose of username is only to give a an SEO-friendly url to the user for his/her profile ?

there are few benefits of using a custom username: multiple ways to login, friendly urls, you can browse other users easily by using a username

Another reason for not using emails is that most of the workers or requesters do not want to share their emails on the public pages of platform so username is used mostly for display purposes.


Mocks for Adapter class

This PR fixes some broken tests by mocking the method from rosie.chamber_of_deputies.adapter.Adapter that downloads stuff from the original source.
For those who are new to unittest.mock:

A live coding I made about unittest.mock (in pt-BR)
PropertyMock (first time I used it)

Coverage increased (+8.3%) to 96.381% when pulling e3ae701be163ccff7e2f64721752b0374dffdfc1 on cuducos-creating-core-module into 5ca30d93a52029a722bd15795c37ef7bbf7582a5 on anaschwendler-creating-core-module.

Checked @cuducos changes to my pull request and aimed to understand what he did.
Seems fine, looks good, I can work with that.
Thanks a lot @cuducos and @jtemporal ! <3

Checked @cuducos changes to my pull request and aimed to understand what he did.

If you need help drop a line and I can do a walkthrough ; )

Coverage increased (+8.3%) to 96.381% when pulling 4fd28b38309a38e2a1a6024426d09bd1a988ff83 on cuducos-creating-core-module into 5ca30d93a52029a722bd15795c37ef7bbf7582a5 on anaschwendler-creating-core-module.

Following this code, it may be necessary to delete temporary classifiers that might be running in your enviroment.
If you used to run rosie before that, by now you need to delete all .pklfiles in /tmp/serenata-dataand run again.
If you don't do that, the error that it introduce is:


set vertex.id and edge.id primary key

Summary by CodeRabbit

New Features

Introduced a TableAttributes class to represent and manage attributes of database tables, including primary keys.
Enhanced SQL sentence generation with the inclusion of table attributes.

Refactor

Updated GraphEdge and GraphVertex to utilize TableAttributes.
Renamed and modified the toSqlSentence method in ColumnList to toSqlSentenceSeq for improved clarity and functionality.

Tests

Added comprehensive test cases for new functionalities in TableAttributes, PrimaryKey, and modified SQL sentence generation.

Documentation

Updated documentation to reflect changes in table attribute handling and SQL sentence generation.

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

Comparison is base (acb2571) 74.62% compared to head (1714197) 75.54%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #116      +/-   ##
==========================================
+ Coverage   74.62%   75.54%   +0.92%
==========================================
  Files          20       22       +2
  Lines         398      413      +15
  Branches       41       36       -5
==========================================
+ Hits          297      312      +15
  Misses        101      101

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


grpc: 0.0.8-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository grpc to 0.0.8-0:

upstream repository: https://github.com/CogRob/catkin_grpc.git
release repository: https://github.com/CogRobRelease/catkin_grpc-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.6.4
previous version for package: 0.0.7-0

grpc
* Install grpcpp headers which was missing.
* Contributors: Shengye Wang

Duplicated: https://github.com/ros/rosdistro/pull/17528


MockGiven willThrow

[x] I have read the CONTRIBUTING.

🚀 Describe the feature you'd like
given(mock).function().willThrow(MyError())
🛠 Describe the implementation you've considered
A clear and concise description of any implementation solutions you've considered.
ℹ Other information

Can you please add details (examples would be ideal) on this feature ?
I would be glad to contribute on this great project.

Hello! Thank you for your interest 😀. I will add more details for next issues from now. For this issue, I have already written the implementation. A pull request will be open today. Don’t hesitate to comment  it.


onEnded reset back to Poster Image

Hi,
I've been trying to reset the player back to the poster image once the video ended. I have tried the following code because I can't see any other method that would offer this functionality.
  const handledOnEnded = useCallback(() => {
    playerRef.current.showPreview();
  }, []);

I am experiencing something alike,  where on the end of the video, the youtube player stops 1  seconds before the video actually end, which causes the onEnded callback to not be called.

ChazUK hallow it is a feature that while video ended it will show you some suggestion.
so you want it to not to suggest right?

so you should use this
`  const playerRef = useRef(null);
const handleEnded = (e) => {
playerRef?.current?.showPreview();
};`

Hello! We have one proposal to include the requested feature under name endWithPreview prop

PR Code: #1517

thanks for the feedback but this feature will not be supported by default.
it's possible to accomplish this by using the event callbacks and overlaying the player with a poster image or resetting the video to it's initial state...


[FEATURE] Better S3 Exception handling

🚨🚨 Feature Request

[x] Related to an existing Issue
[x] A new implementation (Improvement, Extension)

If your feature will improve HUB
All errors related to the S3 storage are now handled by general exceptions, e.g https://github.com/activeloopai/Hub/blob/879efa1d7e8e4a7e91dcc569a83a8f17a2e6560a/hub/util/exceptions.py#L282
Description of the possible solution
Add more specific exceptions and their descriptions, handle boto3 errors, and overwrite them to have more meaningful messages and solutions suggestions.
Difficulty: Medium-hard

I'm a beginner, can anyone please tell how to get started to resolve this.

Greetings, @muhdhisham, and welcome to Hub! Since you've mentioned you're a beginner, it may be a good idea to move to another, easier issue, in case you run into any problems with this issue. You can take a look at some issues marked as "good first issues" (#1182 for example.)

hi @dhiganthrao , if this issue isnt assigned, i can take it up

This one is still up for grabs, folks!


PT-BR Translation update

Added translation to the new features
Fixed inconsistencies

Added https://github.com/bluesky-social/social-app/pull/5690


Email templates improvements

Extracted email subject and last-level <table> element containing the body of the email to the main template. Now the separated email templates will only have their own custom content to avoid repeating common elements.
CR @matallo cc @juanignaciosl

as long as they are tested I'm down to 🔥

This is certainly an improvement :-) Nevertheless a quick scan reveals some room to improvement with partials. Take a look at Rails layout guide, there's some techniques there that we're not currently using but that are easy, "standard" and useful.

@juanignaciosl @xavijam I extracted some markup structures that were duplicated to layouts, using their variable content as partials, this way now we can use the same layout in different files changing the content we want to use on them. What do you think?

I don't know if we should move forward with this in order to do the next mails ASAP, but I think that using content_for and local variables (:locals) should remove all structure duplication. Whatever you and xavijam decide for this PR is ok but please try that from the scratch for the next one.

No no, we agreed to make all changes/enhancement before merging. So let's try that.

Using content_for is not the best idea as it's not possible render the same content (i.e. map_liked.html.erb) in two layouts at the same time (layouts/mail.html.erb and a hypothetical layouts/social_message.html.erb where the content is set).
My proposal is to use the different parts of the mails (title, colored boxes, links, preview images, plain text, etc) as partials, passing the variables we need inside them as :locals. This way we preserve the layout/mail.

@matallo could you review the last changes I made?

awesome partials 👍

✨


When switching to another dry-run in the compliance check view, I can't switch back

The second one remains selected.
reported on iso5-stable

Hey team! Please add your planning poker estimate with ZenHub @andraskvr @arnaudsjs @FloLey @Hugo-Inmanta @sanderr


Custom forms: Field type “Number” generates an error message if no value has been entered.

Used Shopware Version: <IP_ADDRESS>
Description: A separate form with a field (type number) has been created. The field is not a mandatory field, but still does not allow a value to be stored.
The message always appears in the frontend that a numeric value must be included. This contradicts the configuration “No mandatory field”.
Environment: <IP_ADDRESS> / 4.3.0
Steps to reproduce: Create a custom form with one field (typ numeric - not a mandatory field). Try to send the form without fill up the field in frontend.
Expected result: The form can send without a value
Actual result: You get an error
[3_config.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1716925507_6656344336a44_3_config.png]  [2_config.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1716925508_6656344400044_2_config.png]  [1_error.png|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1716925508_6656344463838_1_error.png]

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/CMS-453.
[created from CMS-453, comment 484999]


can you planning support new stm32 cube ide 1.15.0 ?

hi.
very nice your cubeide git actions.
can you planning support new stm32 cube ide 1.15.0 ?

Now supported in version 12 of the docker and the action
Please check if it works for you

I tried 1.15.0 but it fails while building image:
Step 10/12 : RUN unzip -p /tmp/stm32cubeide-installer.sh.zip > /tmp/stm32cubeide-installer.sh && rm /tmp/stm32cubeide-installer.sh.zip
---> Using cache
---> aaa190eb22b7
Step 11/12 : RUN chmod +x /tmp/stm32cubeide-installer.sh &&     /tmp/stm32cubeide-installer.sh  &&     rm /tmp/stm32cubeide-installer.sh
---> Running in 6c21bdde70a9
Creating directory makeself_dir_Dq4fYV
Verifying archive integrity...  100%   All good.
Uncompressing STM32CubeIDE 1.15.0 installer  100%
Installing STM32CubeIDE into /opt/st/stm32cubeide_1.15.0 ...
Creating directory makeself_dir_cfmAyM
Verifying archive integrity... All good.
Uncompressing STLink udev rules  100%
Package installed.
Creating directory makeself_dir_CF037U
Verifying archive integrity...  100%   All good.
Uncompressing STM STLink-Server installer  100%
stlink-server error while installation started.
Already installed version is newer or equal: line 64:
NOT downgrading. Aborting stlink-server installation.
DISPLAY not set. Cannot display license. Aborting.
Installation failed
my temporary workaround is to exit the stm32cubeide-installer.sh with 0 because I don't care about stlink-server:
RUN chmod +x /tmp/stm32cubeide-installer.sh &&     /tmp/stm32cubeide-installer.sh ; exit 0 &&     rm /tmp/stm32cubeide-installer.sh

Interesting. All the versions are specified, so I don't understand why you get a different result than I do. Why not just use the published docker image?
I had it working in my sample application.
Please let me know how you go.
@firepooh as original requester, did you manage to test this?

works like a charme 👍
is there a reason why you did not just use the versioning scheme of stm32cube ide? why 12.0 = 1.1x.xx etc

Because I wanted to make sure that I could have multiple versions with the same IDE. Happy to change it to another format, as long as that requirement is still met. Could do 1.15.0a? I would only need to know if the tag can indeed contain an extra letter like that. If you can find confirmation of this then I'm happy to implement the change for the next 1.15.1 which is now the current version.

As there was no further feedback I'll stick with the current versioning scheme and close this issue.

Hey sorry, totaly forgot.
Was just curious why. For me it's totaly fine as it is 👍

Great. Thanks for the feedback.


Future Sight Bug

Future Sight (I would assume Doom Desire) show that they are super effective, even though they are typeless damages.
KaizoFR.zip

To recreate this bug, pick Hariyama (Right Pokemon) go to Route 1 and one of the two Pokemon are Delcatty who has Future Sight.

@bdjeffyp this could be related to the fix you did for Sonic Boom.

Yep, and OnlySpaghettiCode is already on top of it in #59. Thanks so much, @Tapollo!

You may want to remove the link to the zip file. not sure if it's a good idea to leave these files around.


Issue #423 3D-Box Game

PR Description 📜
There will be 2 types of 3d boxes of color red and green. Red is for danger and green will be our moving block. So we have to avoid all the red boxes and keep the green box on the platform and go as far as we can... It's an amazing game and will really be fun to make. It's gonna use HTML, CSS, JavaScript and Three.js Library.
Fixes https://github.com/kunjgit/GameZone/issues/423

Mark the task you have completed ✅

[ x ] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[ x ] I have performed a self-review of my own code or work.
[ x ] I have commented my code, particularly in hard-to-understand areas.
[ x ] My changes generates no new warnings.
[ x ] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[ x ] I have added screenshot for website preview in assets/images
[ x ] I have added entries for my game in main README.md
[ x ] I have added README.md in my folder
[ x ] I have added working video of the game in README.md (optional)
[x  ] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @AnmolChauhan123 ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

Hey @AnmolChauhan123,
Please make sure to link the relevant issue using the appropriate syntax, such as "#issueNumber" 👀.
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

Thank you  @AnmolChauhan123 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Thank you  @AnmolChauhan123 , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀


“Connect Contacts” screen not displayed after pasting attestation codes successfully from “Confirm” page

Frequency: 100%
Repro on build version: Alfajores build V1.18.3
Repro on devices:  Samsung Galaxy J5 Prime (8.0), Google Pixel 2XL (11.0),  iPhone 12(14.7.1),
Testing Account:<PHONE_NUMBER>/Account Key
Repro Steps:

Launch the application & completing onboarding process
Start verification process & user should received 3 attestation codes
Paste all 3 valid codes & wait for sometime
Observed that,
Current Behavior: User redirects to Home screen after last code accepting successfully
Expected Behavior: User should redirect to “Connect Contacts” screen
Investigation: Also we have  verified same issue while Restoring account with verified phone number
Impact: User unable to sync contacts while Signing up/in
Attachment: ConnectContactsScreenNotDisplayed.wmv

@ValoraQA the "Connect Contact" screen was removed in 1.18.3. Once we get more feedback from the Google Playstore we'll have a better idea of the expected behavior. For now this is expected in 1.18.3.

Thanks for Confirmation:)

https://valora.testquality.com defect D259.


Bug fix/6441 after running a business flow a new is added at the end

Bug fix/6441 after running a business flow a new is added at the end

Yaron is fixing the serilizer and i already fixing the LoadRecentBusinessFlow on diffrent branch so closing this branch and related ticket.

Ravi please delete the branch


Issue with mocks for multiple items in a single file

When generating multiple mocks within single file, imports are being duplicated that causes compilation errors.
I'm using slightly modificated script for this purpose.
OUTPUT_FILE="./${PROJECT_NAME}Tests/CuckooMocks/GeneratedMocks.swift"
INPUT_DIR="./${PROJECT_NAME}/Classes"

find ${INPUT_DIR} -name *.swift | xargs \
cuckoo generate \
    --runtime $CUCKOO_VERSION \
    --testable ${PROJECT_NAME} \
    --output ${OUTPUT_FILE}

As an alternative, you create mirror of your application's target directory tree structure inside testing target directory and generate mocks for each file separately. The only problem with this is that you have to manually add generated files and directories into Xcode project.
INPUT_DIR="./${PROJECT_NAME}/Classes"
OUTPUT_DIR="./${PROJECT_NAME}Tests/CuckooMocks"

# Recursively run through the $INPUT_DIR and find files with .swift extension
find ${INPUT_DIR} -type f -iname "*.swift" -print0 | while IFS= read -r -d $'\0' filepath; do
    # Generate mirror path inside $OUTPUT_DIR
    output=${OUTPUT_DIR}${filepath#$INPUT_DIR}
    # Modify filenames by adding 'Mocks' suffix
    output=${output%".swift"}"Mocks.swift"
    # Create mirror directory tree inside $OUTPUT_DIR
    dir=$(dirname ${output})
    mkdir -p ${dir}

    # Run Cuckoo script to generate mock for each file
    cuckoo generate \
        --runtime $CUCKOO_VERSION \
        --testable ${PROJECT_NAME} \
        --output ${output} \
        "$filepath"
done

IIRC you can give a directory to the --output argument and it should be creating files in that directory, instead of writing into a single file. However, what imports are causing compile errors? I was trying multiple same imports in a single file and did not get a compilation error.

Anyway to solve this, we could strip the header and parse its lines to find all unique imports. Those would then be added at the top of the GeneratedMocks.swift file.

IIRC you can give a directory to the --output argument and it should be creating files in that directory

Awesome, thanks!
Will it overwrite files with similar names?

However, what imports are causing compile errors? I was trying multiple same imports in a single file and did not get a compilation error.

Hmm, indeed when trying to reproduce it within the working environment everything's fine.
Seems like it's different issue with scopes not being closed or else..

That's very strange, I think the problem might be somewhere else, like not closing a scope with }.

Will it overwrite files with similar names?
I'm not completely sure, I only used the single file approach since, so hopefully it will work well. If not, we have more to fix :)

Should be fixed in 0.5.1. Please try it out! :)


GLArea demo?

why isn't there a GlArea demo, i'd personally find this very useful as i can't find much info on using the GlArea

@exoess That's actually a good question. I never thought about that tbh. Gotta add one late, but Idk anything about OpenGL, so... It may take a while.

@exoess uwu Josue te solicitan

@exoess 👍

@exoess done, check commit 3798328


script to move Lambda files into a new folder before deploy

a simple bash script to prevent all files in the folder being uploaded to aws

84a5a9e4aa5fbb2097b5556ce86bc428b1e4fc07 shouldn't be part of this PR.


TensorBoard link names can become too long based on run labels

Guild indiscriminately uses a run's label in the TB link name. Long labels lead to long file names that aren't necessarily helpful.
Approaches:

Don't use labels at all (simplifies the problem but loses useful functionality)
Truncate labels at some max length

As of 0.6.5 Guild no longer includes all flag values in trial labels but only those flags that have changed. This should generally shorten the labels.
It's not clear what the max label length should be for TensorBoard - so closing this until we have a better idea of how to address this. Anyone is free to re-open.

It seems the labels are created by the run flags, even without manually setting labels.
I have the too-long error:
(humanGan) dvir@dvir:~/Documents/repos/desantanglGAN$ guild tensorboard
Preparing runs for TensorBoard
Refreshing flags...
WARNING: cannot import flags from /home/dvir/Documents/repos/desantanglGAN/./train.py: /home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:541: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint8 = np.dtype([("qint8", np.int8, 1)])
/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:542: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint8 = np.dtype([("quint8", np.uint8, 1)])
/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:543: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint16 = np.dtype([("qint16", np.int16, 1)])
/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:544: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_quint16 = np.dtype([("quint16", np.uint16, 1)])
/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:545: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  _np_qint32 = np.dtype([("qint32", np.int32, 1)])
/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/tensorboard/compat/tensorflow_stub/dtypes.py:550: FutureWarning: Passing (type, 1) or '1type' as a synonym of type is deprecated; in a future version of numpy, it will be understood as (type, (1,)) / '(1,)type'.
  np_resource = np.dtype([("resource", np.ubyte, 1)])
INFO: [OpenGL.acceleratesupport] No OpenGL_accelerate module loaded: No module named 'OpenGL_accelerate'
'debug_mode'
Traceback (most recent call last):
  File "/home/dvir/anaconda3/envs/humanGan/bin/guild", line 8, in <module>
    sys.exit(main())
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/main_bootstrap.py", line 38, in main
    guild.main.main()
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/main.py", line 32, in main
    _main()
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/main.py", line 38, in _main
    main_cmd.main(standalone_mode=False)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/external/click/core.py", line 759, in __call__
    return self.main(*args, **kwargs)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/external/click/core.py", line 714, in main
    rv = self.invoke(ctx)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/external/click/core.py", line 1132, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/external/click/core.py", line 951, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/external/click/core.py", line 552, in invoke
    return callback(*args, **kwargs)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/click_util.py", line 146, in fn
    return fn0(*(args + (Args(**kw),)))
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/commands/tensorboard.py", line 78, in tensorboard
    tensorboard_impl.main(args)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/commands/tensorboard_impl.py", line 39, in main
    monitor.run_once(exit_on_error=True)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/run_util.py", line 74, in run_once
    self._refresh_logdir(runs)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/run_util.py", line 81, in _refresh_logdir
    path = self._ensure_run(name)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/run_util.py", line 94, in _ensure_run
    util.ensure_dir(path)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/site-packages/guild/util.py", line 65, in ensure_dir
    os.makedirs(d)
  File "/home/dvir/anaconda3/envs/humanGan/lib/python3.7/os.py", line 221, in makedirs
    mkdir(name, mode)
OSError: [Errno 36] File name too long: '/tmp/guild-tensorboard-kxczw1hy/092bbc42 UnetReconstruction:train 2019-12-31 08:17:59 data_dir=_home_dvir_Documents_repos_desantanglGAN_data_faust_mixamo_faust debug_mode=0 gpu_indecies=1 pool_type=SAGPooling template_mesh=_home_dvir_Documents_repos_desantanglGAN_dfaust_data_registrations_train'

Thank you for the report! A fix for this will land in the next pre-release.

In addition to this fix, 0.7 will support a --run-name-flags option that lets you list a set of flags to use as the run label. In some cases the default label is too long so this option can be used to show only certain flags - or no flags at all if an empty string is specified.

I'm going to close this as I believe the issue is resolved in 0.7.0. Please re-open if you think otherwise and we'll sort it out. Thanks again for the report!


Set path on app launch

What's the proper way to set the navigation path on app launch, e.g. based on whether the user is logged in to some API?

There is no definitive "best way". It all depends on how you implement your logic of users logging in and authenticating etc.
Router can be given an initialPath. If you can figure out whether a user is logged in directly once the app launches, something like this could work:
@main
struct MyApp: App {
	@StateObject private var userData = UserData()

	private var initialPath: String {
		userData.isLoggedIn ? "/user" : "/"
	}

	var body: some Scene {
		WindowGroup {
			Router(initialPath: initialPath) {
				ContentView()
			}
		}
	}
}

Or if it's an asynchronous task, you could programmatically navigate to specific path:
@main
struct MyApp: App {
	@StateObject private var userData = UserData()

	var body: some Scene {
		WindowGroup {
			Router {
				ContentView()
			}
			.environmentObject(userData)
		}
	}
}

struct ContentView: View {
	@EnvironmentObject private var userData: UserData
	@EnvironmentObject private var navigator: Navigator
	@State private var isBusy = true

	private func checkLogin() {
		userData.authorizeUser { success in
			self.isBusy = false
			if success {
				navigator.navigate(to: "user")
			}
		}
	}

	var body: some View {
		Group {
			if !isBusy {
				SwitchRoutes {
					Route(path: "user", content: UserView())
					Route(content: HomeView())
				}
			}
			else {
				ProgressView()
			}
		}
		.onAppear(perform: checkLogin)
	}
}

These are very rough and simplistic examples. But it does demonstrate multiple ways to tackle the challenge.

Thanks, looks good!


EIP cannot be bound and reused

Using the fixed-tag mode, the EIP that has been bound can no longer be bound to the POD after the POD is restarted.

I tested it, and if you're referring to the 'kubectl delete pod' method, it is possible to re-associate. In the case of 'kubectl rollout restart', you need to ensure that you have enough idle EIPs with the corresponding tag in your EIP pool to associate with the new pods. If it's not these issues, can you provide more detailed information?

For me this has started to happen more often.
This is the scenario:

My pod dies
Cannot get loop because "no EIPS can be found"
I delete the tags that the controller has made
Delete the pod again
controller can now again provision the node an EIP, all is good.

@jlehtimaki "My pod dies" would you please clarify the specific situation or how to simulate it?

@jlehtimaki I have two questions about the scenario:
{"time":"2024-07-21T04:50:35.629798872Z","level":"ERROR","msg":"disassociate address for pod: xxx/xxx-1 fail: disassociate address xxx/xxx-1: disassociate address association-id ","component":"handler"}

the association-id does not printed in the logs?
Does the pod name xxx/xxx-1 remain unchanged after a restart?

@jlehtimaki I have two questions about the scenario:
{"time":"2024-07-21T04:50:35.629798872Z","level":"ERROR","msg":"disassociate address for pod: xxx/xxx-1 fail: disassociate address xxx/xxx-1: disassociate address association-id ","component":"handler"}

the association-id does not printed in the logs?
Does the pod name xxx/xxx-1 remain unchanged after a restart?

Only thing that I removed was the pod name + namespace. Replaced with xxx/xxx-1
Yes it does. It is a statefulset so the pods dont have any auto generated strings.

Determined this is a bug, when the node is released, the delete event of the pod is not captured, and the association of the EIP is released.
When the Pod is restarted, the controller attempts to release the original association, resulting in failure.

Yes I think this is exactly what's happening when I was checking the code.
Removing some of the tags will resolve the issue.

@turmind are you looking at this or shall I provide a fix for it?

@turmind are you looking at this or shall I provide a fix for it?

Yep, It will be great if you can provide a fix for this.


[Issue #535] Using positional grams in RegexToGramQueryTranslator

In this pull request, the RegexToGramQueryTranslator module is extended to consider the positions of grams and pass a boolean expresion of phrases to Lucene. This will reduce the number of candidates returned by Lucene and therefore, reduce the cost of regex matching on the candidates.
In connection with issue #535 .
The performance experimental results are in:
https://docs.google.com/spreadsheets/d/1x-dCStWbT_uJLveNLcsd5-aBdYstLh7gQlO8EeToo54/edit#gid=1799191034
Summary of results (performance improvement (%) for 10 queries):
1M medline:
34.47, 64.90, 0.58, 46.94, 97.04, 66.32, 29.64, 21.99, 29.86, 3.16, 81.35
5M medline:
52.12, 71.73, 2.12, 25.49, 27.83, 6.47, 7.61, 11.69, 2.84, 20.30, 24.74
10M medline:
83.81, 46.59, 75.42, 66.11, 25.14, 59.25, 68.66, 78.13, 12.67, 86.27, 49.10
Medline queries are:
(cures|helps|treats) ([A-Za-z'-]+\s*){1,2} (disorder|condition|illness|syndrome)
cases? of ([A-Za-z'-]+\s*){1,2} transplantation
[A-Za-z]+ treatment
[A-Za-z]+ injections? of [A-Za-z]+ drug
the [A-Za-z]* (test|experiment) show(s|ed)
severe cases? of [A-Za-z']+ syndrome
(considerably|very|much|so) [A-Za-z]er than
(advanced|severe|intense) ([A-Za-z'-]+\s){1,2} (disorders?|syndromes?|conditions?|diseases?)
\d{1,2}% of (cases|patients|recipients)
[A-Za-z]+therapy
caused by ([A-Za-z'-]+\s*){1,3} virus infection
=================================================================
1M stackoverflow:
9.84, 39.54, 16.42, 30.93, 20.24, 19.13, 10.78, 7.94, 28.63, 33.76
Stackoverflow queries are:
(the) ([A-Za-z]+\s*){1,3} (librar(y|ies))
(the) ([A-Za-z]+\s*){1,3} (architecture(s))
(dependen(t|cy)) (on) ([A-Za-z-']+\s*){1,3} (module(s))
((pars(e|ing) json)|(json pars(e|ing)))
evaluat(e|ing|ion) (of)? ([A-Za-z-']+\s*){1,3} expression(s)
us(ed|ing|e) ([A-Za-z-\+']+\s*){1,2} (de|en)cod(e|er|ing)
implement(ing) ([A-Za-z-']+\s*){1,3} engine
initializ(e|ing) ([A-Za-z-']+\s*){1,3} (runtime|platform|engine)
((writ(e|ing|ten))|(implement(ing|ed)) (cod(e|ing|ed)))|(script(ing|ed))  (in) ([A-Za-z-']+\s*){1,3} language(s)
(ms|microsoft) ([A-Za-z-\+']+\s*){1,2} (software|product|application)
=================================================================
1M wikipedia:
25.79, 14.60, 4.80, 21.95, 34.74, 10.11, 20.37, 13.63, 19.62, 11.04
Wikipedia queries are:
receive(d|s)|wins|win|won ([A-Za-z']+\s*){1,3} award(s) (for) [A-Za-z']+\s*
(ancient) (king|queen) (of) ([A-Za-z']+\s*){1,3}
the(\s+[A-Za-z']+\s*){1,3} (movement) (started) (in) ([A-Za-z']+\s*){1,3}
([A-Za-z']+\s*){1,2} (married) ([A-Za-z']+\s*){1,2}
the(\s+[A-Za-z']+\s*){1,2} (festival|parade|gethering|ceremony) ((of) ([A-Za-z']+\s*){1,2})? ((in) ([A-Za-z']+\s*){1,2})?
([A-Za-z']+\s*){1,2} ((invent(s|ed))|found|(prove(s|d))|discovered) ((with) ([A-Za-z']+\s*){1,2})? ((in) ([A-Za-z']+\s*){1,2})?
([A-Za-z']+\s*){1,3} (invaded|conquered) ([A-Za-z']+\s*){1,3} ((in) ([A-Za-z']+\s*){1,2})?
([A-Za-z']+\s*){1,3} (established|founded|started) ([A-Za-z']+\s*){1,3} ((in) ([A-Za-z']+\s*){1,2})? ((to) ([A-Za-z']+\s*){1,2})?
the(\s+[A-Za-z']+\s*){1,2} (war|battle|((military) (conflict))|conflict) (in) ([A-Za-z']+\s*){1,2}
([A-Za-z']+\s*){1,2} (processing) (unit|plant|factory|machine|device) ([A-Za-z']+\s*){1,2}

In addition to those pasted numbers in text, can you also have a few screenshots of the performance numbers and include them here?

@jesmaelnezhad Travis failed for this PR.  Please fix it.


I built 3 sites with Eleventy!

https://github.com/11ty/11ty-website/pull/441 is my PR.

https://github.com/11ty/11ty-website/pull/441 Merged so I close this.


Lsp can't install if other ones are active in the current buffer

As discussed on discord, this is related to:
https://github.com/MordechaiHadad/nvim-lspmanager/blob/12241981ef5365001363a21b2ceb7eb45f3fd602/lua/lspmanager.lua#L131-L133
We better check if a certain lsp is installed.

Closed with<PHONE_NUMBER>f97693611831640dc0b63c8e117e02


nautilus-open-in-blackbox: init at 0.1.1

Description of changes
A Nautilus Plugin to add Open in Blackbox in Context Menu

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)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (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.

FYI, there is already a nautilus-open-any-terminal which works for blackbox and actively maintained.

FYI, there is already a nautilus-open-any-terminal which works for blackbox and actively maintained.

Yeah I know it exists, but I couldn't get that to work. So I ported this instead

Maybe we should write a module for that.

Here you are. https://github.com/NixOS/nixpkgs/pull/283778 :)


[Profiles] A couple of bugs with profiles

I'm using the latest addon version: https://github.com/asciidisco/plugin.video.netflix/commit/dfc7738380f5ce833063ce74e2c0ab9b82b84eae, a fresh build compiled for Windows from the latest kodi-agile: https://github.com/FernetMenta/kodi-agile/commit/040db824a3760ee941ef625129dc26c9d6142f17 and the latest inpustream: https://github.com/liberty-developer/inputstream.adaptive/commit/764de78a262091b2bd050b97778d0f30e6501460
Issue 1:
My Netflix account has two profiles. Profile 1 has the language set to Spanish and Profile 2 to English. If I use Profile 2, the categories menu (My List, Continue Watching,...) is in English:

but the titles and descriptions are in Spanish:

If I change the Profile 1 language to English, I can see the titles and descriptions in English with Profile 2 so, clearly, the Profile 2 always uses whatever language Profile 1 has.
Issue 2:
If I select Profile 2 from the profiles selection screen, go back to this screen and select it again, the profile screen I get is the one from Profile 1.
These issues can be easily reproduced but I can provide a log if needed.
Thanks in advance.

@Uukrull Thank you for the feedback (and sorry for the delayed in reply).
Regarding Issue No. 1:
We´re not parsing the language per user profile atm., the language is extracted from the account owner profile. I must admit, I wasn't aware that it is possible to have different languages using one account. Fixing that is now on our list.
Regarding Issue No. 2:
Great observation. That´s a bug in the in memory caching layer, the caching id isn't generated on a per user base, but only on a per category base. Refactoring the cache is the next thing on my list, after we fixed the big problem with the crashing background service that still keeps us occupied.
Btw.: If you do speak spanish, would you mind contributing a spanish gettext file to the project. There are only around 30 messages we need to translate & that would definitely help us :)
Cheers

Btw.: If you do speak spanish, would you mind contributing a spanish gettext file to the project. There are only around 30 messages we need to translate & that would definitely help us :)

Done: https://github.com/Uukrull/plugin.video.netflix/commit/422bc2d2d5516307c6a451c3508c24d991412192 Do I open a PR?
Many thanks to you and all the people involved for bringing Netflix to Kodi. I never thought I could see this day :)

@Uukrull Awesome. Yes, please PR it :)

@Uukrull I pushed a release a couple of moments ago. Issue No. 2 should be resolved.
I also looked briefly into the causes of the behaviour of Issue No. 1, but I can't seem to identify how Netflix does determine the language of the user. I have the feeling that they´ll do it with some cookie magic when calling the "/browse" route (which we don't when switching profiles, as this is really slow & there´s no fresh information for us to use). We only called once at the first startup. I guess that would explain your observations.
I´ll do some more research & curl magic to see if I can reproduce this on the lowest level possible, maybe the trick is for us the fake some cookie setting or such.

Issue No. 2 should be resolved.

Yes, it is. Many thanks!

And yes, it is cookie magic, done by a cookie set from the browse HTML page (which takes ages to load & does only accept GET requests, so no chance to do a HEAD here & fetch it faster).
The cookie looks like this:
lhpuuidh-browse-ZFFNPO5JSVGUHB4WTKI6UYYBSD=DE%3AEN%3A4b59b7af-29a8-482d-a6b4-137c3303ebb8_ROOT;

a) the ZFFNPO5JSVGUHB4WTKI6UYYBSM part of the cookie name is your profiles GUID.
b) the urlencoded(DE:EN:) part of the cookie contents defines the accounts origin & the selected language (origin DE & language EN here in my case), the second part is the so called "lolomo" id (b59b7af-29a8-482d-a6b4-137c3303ebb8_ROOT), which Netflix assigns to your root account view (the one is needed to get your accounts "Suggestions", "Genres", "Watchlist" etc.)
So, long story short:
Fetching that cookie takes too long, we shouldn't go down that route.
Faking it would be more interesting (we have all the data, EXCEPT the language), so basically, we could get the users language settings from Kodi & always fetch the language based on the current Kodi settings.
In the End, if you share your Kodi installation with other people at home, you most probably end up with different profiles that have their own settings in Kodi (including the interface language) as well.

Great! That could also explains why I get Latin American Spanish (es) audio tracks instead of the Castilian (es-ES) ones (my account was created in the USA before Netflix arrived to Spain) with the addon.

i think that actually the audio stream with the language selected in kodi should be selected from the stream? Is there a diffrent setting in kodi for es and es-ES?

This isn't a Kodi problem because I tried a Netflix account created in Spain and it shows the correct Castilian audio track. For example, all the Netflix Originals have the Spanish audio track in Latin American Spanish instead of Castilian if I use the addon, but I have no problems with a browser or the Shield TV app (that shows in the information screen Country = ES and Language = en-US).
Could it be possible to add a hidden setting to generate that fake cookie with the country and language we want?

Issue 2:
If I select Profile 2 from the profiles selection screen, go back to this screen and select it again, the profile screen I get is the one from Profile 1.

Problem is still present.


version problem?

after pip install dwave_ocean_sdk
import dwavebinarycsp as dbc

and_gate = dbc.factories.and_gate(["x1", "x2", "y1"])
and_csp = dbc.ConstraintSatisfactionProblem('BINARY')
and_csp.add_constraint(and_gate)
and_bqm = dbc.stitch(and_csp)
and_bqm.remove_offset()

and_bqm

gives the following error:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-18-2e6477ca2e14> in <module>()
      4 and_csp = dbc.ConstraintSatisfactionProblem('BINARY')
      5 and_csp.add_constraint(and_gate)
----> 6 and_bqm = dbc.stitch(and_csp)
      7 and_bqm.remove_offset()
      8

/opt/conda/lib/python3.6/site-packages/dwavebinarycsp/compilers/stitcher.py in stitch(csp, min_classical_gap, max_graph_size)
    167                 continue
    168
--> 169             if pmodel.classical_gap >= min_classical_gap:
    170                 break
    171

AttributeError: 'NoneType' object has no attribute 'classical_gap'

Successfully installed dimod-0.6.11 dwave-cloud-client-0.4.4 dwave-neal-0.3.1 dwave-networkx-0.6.3 dwave-ocean-sdk-1.0.0 dwave-qbsolv-0.2.7 dwave-system-0.3.2 dwavebinarycsp-0.0.2 enum34-1.1.6 homebase-1.0.0 minorminer-0.1.4 penaltymodel-0.14.0 penaltymodel-cache-0.2.1 penaltymodel-maxgap-0.3.2 pysmt-0.7.0
so the version here is 0.0.2

Also this is copy paste from the factoring demo. runs on leap jupy


Fix build number

Fix build number

Hmm, that's actually not the right thing to do either.  You should reset the build number to 0 only as part of incrementing to the next version number (e.g. rc3).  It doesn't matter in this case but it would not be good if you were to actually make build 0 and upload it to anaconda.org because we already had build 1, which gets higher priority.
This is one reason to move over to conda-forge when if and when you can -- they remind you every time you do a pull request to either increment the build number if the version is the same or reset it to zero if this is a new version.  But I understand that zppy can't go to conda-forge yet, and it might not make sense to ever move over there.

You should reset the build number to 0 only as part of incrementing to the next version number (e.g. rc3)

Ah, that makes sense. Luckily, I don't intend on uploading this build to anaconda.org, so at least it will be in place for rc3.

This is one reason to move over to conda-forge

I don't know too much about conda-forge, but I can look into it.

The only real requirement is that all your dependencies need to be on conda-forge.  I think that's the case, but I think we had thought that zppy was too specialized so maybe we wouldn't move it there.  But mache is also super specialized and it's on conda-forge so maybe it's fine.


[rxjava2] Update common files for branch master

Update common files

The relevant changes here are covered by #118


Add ability to pass attributes to checkbox and radio input elements

Will resolve #184

 Pass attributes to checkboxes


Not rendering well / Overlapping on mobile

Hello!
The Dropcap works great in general, but on any Apple mobile devices, the letter is overlapping and the size is not proportional anymore.
Thanks!

Can you be more specific? The demo page looks fine on my iPhone.

It works on iOS for me as well (v9.x). But I do have an issue with Android...


[Base] Improve the slotProps type overriding DX

As noted in https://github.com/mui/material-ui/pull/33582#discussion_r941577213, the current way of overriding the type of slotProps (formerly componentsProps) is not flexible enough. It is not possible to modify the types of individual slots' props independently because all slots share the same override type:
export interface BadgeUnstyledComponentsPropsOverrides {}

export interface BadgeUnstyledOwnProps {
  // ...
  slotProps?: {
    root?: SlotComponentProps<
      'span',
      BadgeUnstyledComponentsPropsOverrides,
      BadgeUnstyledOwnerState
    >;
    badge?: SlotComponentProps<
      'span',
      BadgeUnstyledComponentsPropsOverrides,
      BadgeUnstyledOwnerState
    >;
  };
}

In the example above it is not possible to augment the type of props passed to the root slot independently of the ones passed to the badge slot. There's a single BadgeUnstyledComponentsPropsOverrides type that covers all the slots.
Two solutions are currently being considered:

Have a separate type for each slot override (see #35662)
export interface BadgeUnstyledRootSlotPropsOverrides {}
export interface BadgeUnstyledBadgeSlotPropsOverrides {}

export interface BadgeUnstyledOwnProps {
  // ...
  slotProps?: {
    root?: SlotComponentProps<
      'span',
      BadgeUnstyledRootSlotPropsOverrides,
      BadgeUnstyledOwnerState
    >;
    badge?: SlotComponentProps<
      'span',
      BadgeUnstyledBadgeSlotPropsOverrides,
      BadgeUnstyledOwnerState
    >;
  };
}

This could be slightly easier to use for developers. They would augment the required interface and the changes will apply only to one slot.
It would be slightly more cumbersome to implement, though, and would create many new types (up to 8 in case of SliderUnstyled).

Use the existing interface on the slotProps level (see #35665)
export interface BadgeUnstyledComponentsPropsOverrides {}

export interface BadgeUnstyledOwnProps {
  // ...
  slotProps?: {
    root?: SlotComponentProps<
      'span',
      BadgeUnstyledOwnerState
    >;
    badge?: SlotComponentProps<
      'span',
      BadgeUnstyledOwnerState
    >;
  } & BadgeUnstyledComponentsPropsOverrides;
}

Using this option would mean less required changes in the code, but slightly more work for developers. Additionally, it would allow to create types that don't make much sense, as for example:
interface BadgeUnstyledComponentsPropsOverrides {
  input: { 'data-testid': string } // BadgeUnstyled doesn't have the input slot
}

cc @mui/core

My gut says that 1 is preferable as the option that would be easier for devs to use. The code feels more readable to me at a glance, and though I don't love all the type it would require for some components, I do like that it adds some "guard rails" to the feature (vs. being able to do things that don't make sense in 2). Since this is a pretty advanced concept to begin with, I think it's better for it to not be too open-ended, as the second option might be.

Due to limitations of option 2 (no intellisense for interface members), that would limit the DX of the solution, we will proceed with option 1.

I'm a bit late for the party here, but I have started the same discussion in MUI X, and personally, I'm leaning toward option 2: https://github.com/mui/mui-x/pull/7968#discussion_r1113245225

Due to limitations of option 2 (no intellisense for interface members), that would limit the DX of the solution, we will proceed with option 1.

Indeed, there's no IntelliSense for the interface members when doing type augmentation, but there's a similar problem with option 1. You won't have IntelliSense for the SwitchUnstyledThumbSlotOverrides interface name either:
// Source: https://codesandbox.io/s/base-cra-typescript-forked-2gm09w?file=/src/App.tsx:1703-1904

declare module "@mui/base/SwitchUnstyled" {
  interface SwitchUnstyledThumbSlotOverrides {
    "data-testid"?: string;
  }
}

So while both do not have full IntelliSense support, I would rather choose option 2 where the interface members reflect the same structure as slotProps - this seems more intuitive to me than figuring out the interface name for each of the slots.
Did you encounter other limitations when deciding between option 1 and option 2?

I haven't found any real limitations. With Option 1 you have the ability to look up the correct interface name (for example by importing it).
I also took https://github.com/mui/material-ui/issues/34906#issuecomment-1293669656 into consideration.

Alright, we'll proceed with option 1 in the data grid for consistency.
Thanks for your input!

And how would we separate the overrides from one table to another if we're using multiple tables through out the app ?


Update READme

@wiwang

Updated "Usage", "selecting the best model" and"Results" section of Readme
Usage
To replicate this analysis, clone this repository, install the dependencies below, and run the following code in your terminal:
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”
Python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
Selecting the Best Model
There is a mix of categorical, numerical, and binary features for which we will apply proper transformations for use in analysis. There is no class imbalance based on the target readmitted column. We have used RBF SVM and LR models and picked them based on the best scores given by the model. We have split the data into 80% training and 20% testing and re-fit the best model on the entire training data set. Next, we have evaluated its performance on the test data set. The output can be found here : link
Results
The written report can be found
Notes:
I will update "usage" for script4 once it is ready
I will recheck the links in Results and selecting the model section
I have run the python file through the command line and it ran fine and will test the others once fixed
@jraza19 - If we are using feather in processingdata.py, then we have to add it in the dependency section of readme, otherwise, we can delete if not using in the script
@rachelywong - final report is in .rmd which is not displaying the data properly, can we change it to .md
Please let me know if I have missed anything.

@sukh2929  This looks really good!
Let's change it to be more similar to Tiffany's example (https://github.com/ttimbers/breast_cancer_predictor/tree/v2.0):

Keep what we have for title and contributors
Keep Intro and Importance
Add Report section (The final report can be found [here]) basically the Results section you have but moved up here and changed to "Report"
Keep Usage section you updated but format it to how Tiffany formatted it in hers (To replicate the analysis, clone this GitHub repository, install the [dependencies] listed below, and run the following commands at the command line/terminal from the root directory of this project:", add comments to each terminal command like how she did it
Keep dependencies
Remove Features
Remove Selecting the Best Model
Remove Citation
Keep License
Add References section (reference the paper and the UCI database link we got the dataset from)

Regarding SCRIPT5, it has to be a .RMD file according to the milestone2 rubric. I will try to update SCRIPT5 with all the figures and data, and we can try again to add it to the READme

Introduction
For this project, we are trying to answer the predictive question: Given a patient’s demographic, medication history and management of diabetes during the hospital stay, can we predict if they will be readmitted to the hospital or not? Analysis of machine learning models will identify features more likely to predict patient readmission. This will be important for the management of hospital care for diabetic patients because it will identify areas where changes can be made to decrease patient readmission and reduce the burden on the healthcare system.
Importance:
Due to Covid-19, it is critical to reduce the burden on the healthcare system and prevent readmission rates from increasing to make space for Covid cases. Our predictor aims to look at diabetes management and diagnosis during a patient’s hospital stay to understand how much this affects their readmission. This will allow us to create and improve patient safety protocols to better manage diabetic patients during their hospital stay to provide effective care and prevent readmission during this critical time.
The data are submitted on behalf of the Center for Clinical and Translational Research, Virginia Commonwealth University, a recipient of NIH CTSA grant UL1 TR00058, and a recipient of the CERNER data. This dataset was collected from 1998-2008 among 130 US hospitals and integrated delivery networks. The dataset can be found here. Research from this collected data was used to assess diabetic care during hospitalization and determine if patients were likely to be readmitted or not. The paper by Strack et al. (2014) can be found here and the descriptions for the features can be found here.
Report
The written report can be found here : link
Usage
To replicate the analysis, clone this GitHub repository, install the dependencies listed below, and run the following commands at the command line/terminal from the root directory of this project:
#download data
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
data processing
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”
eda analysis
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”
tune-test model
Python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
render report
Rscript -e "rmarkdown::render('doc/script5.Rmd', output_format = 'github_document')"
Dependencies

Python 4.8.3 and Python packages:
docopt==0.6.2
urllib3==1.25.11
ChainMap==3.3
os==10.15.6
tarfile==3.3
numpy==1.19.1
pandas==1.1.2
altair==4.1.0
requests==2.24.0
zipfile==3.2.0

License
The materials used for this project are under an open access article distributed under the Creative Commons Attribution License, which permits unrestricted use, distribution, and reproduction in any medium provided the original work is properly cited. If reusing/referencing, please provide a link to this webpage.
References:
The paper detailing the data collection and research can be found here: https://www.hindawi.com/journals/bmri/2014/781670/
Feature descriptions about the data can be found here: https://www.hindawi.com/journals/bmri/2014/781670/tab1/
This dataset was taken from https://archive.ics.uci.edu/ml/datasets/diabetes+130-us+hospitals+for+years+1999-2008#
Todo - add link to report once ready

Introduction
For this project, we are trying to answer the predictive question: Given a patient’s demographic, medication history and management of diabetes during hospital stay, can we predict if they will be readmitted to the hospital or not? Analysis with machine learning models will identify features more likely to predict patient readmission. This will be important for management of hospital care for diabetic patients, because it will identify areas where changes can be made to decrease patient readmission and reduce burden on the healthcare system.
Importance:
Due to Covid-19, it is critical to reduce the burden on the healthcare system and prevent readmission rates from increasing to make space for Covid cases. Our predictor aims to look at the diabetes management and diagnosis during a patient’s hospital stay to understand how much this affects their readmission. This will allow us to create and improve patient safety protocols to better manage diabetic patients during their hospital stay to provide effective care and prevent readmission during this critical time.
The data are submitted on behalf of the Center for Clinical and Translational Research, Virginia Commonwealth University, a recipient of NIH CTSA grant UL1 TR00058, and a recipient of the CERNER data. This dataset was collected from 1998-2008 among 130 US hospitals and integrated delivery networks. The dataset can be found here. Research from this collected data was used to assess diabetic care during hospitalization and determine if patients were likely to be readmitted or not. The paper by Strack et al. (2014) can be found here and the descriptions for the features can be found here.
Report
The written report can be found here : link
Usage
To replicate the analysis, clone this GitHub repository, install the dependencies listed below, and run the following commands at the command line/terminal from the root directory of this project:
download data
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
data processing
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”
eda analysis
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”
tune-test model
Python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
render report
Rscript -e "rmarkdown::render('doc/script5.Rmd', output_format = 'github_document')"
Dependencies

Python 4.8.3 and Python packages:
docopt==0.6.2
urllib3==1.25.11
ChainMap==3.3
os==10.15.6
tarfile==3.3
numpy==1.19.1
pandas==1.1.2
altair==4.1.0
requests==2.24.0
zipfile==3.2.0

License
The materials used for this project are under an open access article distributed under the Creative Commons Attribution License, which permits unrestricted use, distribution, and reproduction in any medium, provided the original work is properly cited. If reusing/referencing, please provide a link to this webpage.
References:
The paper detailing the data collection and research can be found here: https://www.hindawi.com/journals/bmri/2014/781670/
Feature descriptions about the data can be found here: https://www.hindawi.com/journals/bmri/2014/781670/tab1/
This dataset was taken from https://archive.ics.uci.edu/ml/datasets/diabetes+130-us+hospitals+for+years+1999-2008#

Introduction
For this project, we are trying to answer the predictive question: Given a patient’s demographic, medication history and management of diabetes during hospital stay, can we predict if they will be readmitted to the hospital or not? Analysis with machine learning models will identify features more likely to predict patient readmission. This will be important for management of hospital care for diabetic patients, because it will identify areas where changes can be made to decrease patient readmission and reduce burden on the healthcare system.
Importance:
Due to Covid-19, it is critical to reduce the burden on the healthcare system and prevent readmission rates from increasing to make space for Covid cases. Our predictor aims to look at the diabetes management and diagnosis during a patient’s hospital stay to understand how much this affects their readmission. This will allow us to create and improve patient safety protocols to better manage diabetic patients during their hospital stay to provide effective care and prevent readmission during this critical time.
The data are submitted on behalf of the Center for Clinical and Translational Research, Virginia Commonwealth University, a recipient of NIH CTSA grant UL1 TR00058, and a recipient of the CERNER data. This dataset was collected from 1998-2008 among 130 US hospitals and integrated delivery networks. The dataset can be found here. Research from this collected data was used to assess diabetic care during hospitalization and determine if patients were likely to be readmitted or not. The paper by Strack et al. (2014) can be found here and the descriptions for the features can be found here.
Report
The written report can be found here : link
Usage
To replicate the analysis, clone this GitHub repository, install the dependencies listed below, and run the following commands at the command line/terminal from the root directory of this project:
*#*\ download data
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
*#\ data processing
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”

*#\ eda analysis
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”

*#*\ tune-test model
Python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
*#*\ render report
Rscript -e "rmarkdown::render('doc/script5.Rmd', output_format = 'github_document')"
Dependencies

Python 4.8.3 and Python packages:
docopt==0.6.2
urllib3==1.25.11
ChainMap==3.3
os==10.15.6
tarfile==3.3
numpy==1.19.1
pandas==1.1.2
altair==4.1.0
requests==2.24.0
zipfile==3.2.0

License
The materials used for this project are under an open access article distributed under the Creative Commons Attribution License, which permits unrestricted use, distribution, and reproduction in any medium, provided the original work is properly cited. If reusing/referencing, please provide a link to this webpage.
References:
The paper detailing the data collection and research can be found here: https://www.hindawi.com/journals/bmri/2014/781670/
Feature descriptions about the data can be found here: https://www.hindawi.com/journals/bmri/2014/781670/tab1/
This dataset was taken from https://archive.ics.uci.edu/ml/datasets/diabetes+130-us+hospitals+for+years+1999-2008#

Introduction
For this project, we are trying to answer the predictive question: Given a patient’s demographic, medication history and management of diabetes during hospital stay, can we predict if they will be readmitted to the hospital or not? Analysis with machine learning models will identify features more likely to predict patient readmission. This will be important for management of hospital care for diabetic patients, because it will identify areas where changes can be made to decrease patient readmission and reduce burden on the healthcare system.
Importance:
Due to Covid-19, it is critical to reduce the burden on the healthcare system and prevent readmission rates from increasing to make space for Covid cases. Our predictor aims to look at the diabetes management and diagnosis during a patient’s hospital stay to understand how much this affects their readmission. This will allow us to create and improve patient safety protocols to better manage diabetic patients during their hospital stay to provide effective care and prevent readmission during this critical time.
The data are submitted on behalf of the Center for Clinical and Translational Research, Virginia Commonwealth University, a recipient of NIH CTSA grant UL1 TR00058, and a recipient of the CERNER data. This dataset was collected from 1998-2008 among 130 US hospitals and integrated delivery networks. The dataset can be found here. Research from this collected data was used to assess diabetic care during hospitalization and determine if patients were likely to be readmitted or not. The paper by Strack et al. (2014) can be found here and the descriptions for the features can be found here.
Report
The written report can be found here : link
Usage
To replicate the analysis, clone this GitHub repository, install the dependencies listed below, and run the following commands at the command line/terminal from the root directory of this project:
# download data
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
# data processing
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”

# eda analysis
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”

# tune-test model
Python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
# render report
Rscript -e "rmarkdown::render('doc/script5.Rmd', output_format = 'github_document')"
Dependencies

Python 4.8.3 and Python packages:
docopt==0.6.2
urllib3==1.25.11
ChainMap==3.3
os==10.15.6
tarfile==3.3
numpy==1.19.1
pandas==1.1.2
altair==4.1.0
requests==2.24.0
zipfile==3.2.0

License
The materials used for this project are under an open access article distributed under the Creative Commons Attribution License, which permits unrestricted use, distribution, and reproduction in any medium provided the original work is properly cited. If reusing/referencing, please provide a link to this webpage.
References:
The paper detailing the data collection and research can be found here: https://www.hindawi.com/journals/bmri/2014/781670/
Feature descriptions about the data can be found here: https://www.hindawi.com/journals/bmri/2014/781670/tab1/
This dataset was taken from https://archive.ics.uci.edu/ml/datasets/diabetes+130-us+hospitals+for+years+1999-2008#

@sukh2929 Looks awesome!! I think for references we can just add these 2 citations though:
Beata Strack, Jonathan P. DeShazo, Chris Gennings, Juan L. Olmo, Sebastian Ventura, Krzysztof J. Cios, and John N. Clore, “Impact of HbA1c Measurement on Hospital Readmission Rates: Analysis of 70,000 Clinical Database Patient Records,” BioMed Research International, vol. 2014, Article ID 781670, 11 pages, 2014.
Dua, Dheeru, and Casey Graff. 2017. “UCI Machine Learning Repository.” University of California, Irvine, School of Information; Computer Sciences. http://archive.ics.uci.edu/ml.

changes done!
Introduction
For this project, we are trying to answer the predictive question: Given a patient’s demographic, medication history and management of diabetes during hospital stay, can we predict if they will be readmitted to the hospital or not? Analysis with machine learning models will identify features more likely to predict patient readmission. This will be important for management of hospital care for diabetic patients, because it will identify areas where changes can be made to decrease patient readmission and reduce burden on the healthcare system.
Importance:
Due to Covid-19, it is critical to reduce the burden on the healthcare system and prevent readmission rates from increasing to make space for Covid cases. Our predictor aims to look at the diabetes management and diagnosis during a patient’s hospital stay to understand how much this affects their readmission. This will allow us to create and improve patient safety protocols to better manage diabetic patients during their hospital stay to provide effective care and prevent readmission during this critical time.
The data are submitted on behalf of the Center for Clinical and Translational Research, Virginia Commonwealth University, a recipient of NIH CTSA grant UL1 TR00058, and a recipient of the CERNER data. This dataset was collected from 1998-2008 among 130 US hospitals and integrated delivery networks. The dataset can be found here. Research from this collected data was used to assess diabetic care during hospitalization and determine if patients were likely to be readmitted or not. The paper by Strack et al. (2014) can be found here and the descriptions for the features can be found here.
Report
The written report can be found here : link
Usage
To replicate the analysis, clone this GitHub repository, install the dependencies listed below, and run the following commands at the command line/terminal from the root directory of this project:
download data
python src/download.py --local_path=./data/raw --url=https://archive.ics.uci.edu/ml/machine-learning-databases/00296/dataset_diabetes.zip
data processing
python src/processingdata.py --input=“data/raw/dataset_diabetes/diabetic_data.csv” --output=“data/processed”
eda analysis
python src/eda2.py --input=“data/processed/diabetes_with_race.csv” --output=“reports/figures”
tune-test model
python src/script4 --input=“data/processed/diabetes_clean.csv” —output=
render report
Rscript -e "rmarkdown::render('doc/script5.Rmd', output_format = 'github_document')"
Dependencies

Python 4.8.3 and Python packages:
docopt==0.6.2
urllib3==1.25.11
ChainMap==3.3
os==10.15.6
tarfile==3.3
numpy==1.19.1
pandas==1.1.2
altair==4.1.0
requests==2.24.0
zipfile==3.2.0

License
The materials used for this project are under an open access article distributed under the Creative Commons Attribution License, which permits unrestricted use, distribution, and reproduction in any medium provided the original work is properly cited. If reusing/referencing, please provide a link to this webpage.

References:
Beata Strack, Jonathan P. DeShazo, Chris Gennings, Juan L. Olmo, Sebastian Ventura, Krzysztof J. Cios, and John N. Clore, “Impact of HbA1c Measurement on Hospital Readmission Rates: Analysis of 70,000 Clinical Database Patient Records,” BioMed Research International, vol. 2014, Article ID 781670, 11 pages, 2014.
Dua, Dheeru, and Casey Graff. 2017. “UCI Machine Learning Repository.” University of California, Irvine, School of Information; Computer Sciences. http://archive.ics.uci.edu/ml.

add dependecies:
pandas_profiling==2.9.0
altair-saver==0.5.0
EDA code under usage needs to be:
python src/eda2.py --input="data/raw/dataset_diabetes/diabetic_data.csv;data/processed/diabetes_with_race.csv" --output="reports/figures"

changes done!