text:
Variable types #10

Adding variable types as parameters to auto_entityset, make_entityset
Test in tests/test_normalize
Updated README
Resolves #10

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Jaskirat Grover seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Generally I think this PR looks pretty good, but I have a question about how we want this to behave for the ID columns. If you look at the original entityset (entityset) generated in test_variable_types before normalizing, the customer_id is of type Numeric. However, if you look at this same column of the session_id entity in the normalized entityset, this same customer_id column now is now of type Id.
@rwedge Is this behavior acceptable? I know we will have some differences in variable types where a column in the original non-normalized entityset gets set as an index in a normalized entity, but wasn't sure how we wanted non-index columns to be treated throughout.
We probably should also update the test to test all of the columns we expect to have the same variable types throughout instead of just the single zip_code column.

@thehomebrewnerd changing the variable we normalized on to Id type is standard featuretools behavior.  I think the logic is that we now know this variable can be treated as a foreign key, so we label it as a special categorical type, Id.
I agree the test should test all of the columns, and I think we should test that make_entityset and auto_entityset use variable_types as expected

@j-grover sorry for the delay on the review, are you interested in updating the tests?

@rwedge Updated test to check all columns

@j-grover Thanks for the quick response and updates to the test. Since we have now modified the parameters for the auto_entityset and make_entityset to add the optional variable_types parameter, it would also be good to add specific tests for these two functions to make sure they behave properly and return the expected result when using this optional parameter.
Is that something you would be able to do as well?

@j-grover Thanks for the quick response and updates to the test. Since we have now modified the parameters for the auto_entityset and make_entityset to add the optional variable_types parameter, it would also be good to add specific tests for these two functions to make sure they behave properly and return the expected result when using this optional parameter.
Is that something you would be able to do as well?

What sort of things are we looking to test for these two methods. For example one case with default values and one with custom args?

@j-grover Thanks for the quick response and updates to the test. Since we have now modified the parameters for the auto_entityset and make_entityset to add the optional variable_types parameter, it would also be good to add specific tests for these two functions to make sure they behave properly and return the expected result when using this optional parameter.
Is that something you would be able to do as well?

What sort of things are we looking to test for these two methods. For example one case with default values and one with custom args?

Yes, that is what I was thinking...tests very similar to the test you added for normalize_entity, just to make sure those metohds return the expected results when passing a variable_types parameter. Since these methods are part of the public API, it would be good for test coverage to have specific tests that cover their possible use cases as well.

@thehomebrewnerd
I've added the initial tests for make_entityset and auto_entityset. There is overlap between the tests as auto_entityset makes use of make_entityset internally. When testing auto_entityset, I've found that for entity 0 (refer to line 330 in test_normalize.py) the name changes between 'jersey_num_team' and 'team_jersey_num' for different runs. This causes the following to fail occasionally:
assert normalized_entityset.entities[0].variable_types['jersey_num_team'] == Index
How would you go about testing this?

@thehomebrewnerd
I've added the initial tests for make_entityset and auto_entityset. There is overlap between the tests as auto_entityset makes use of make_entityset internally. When testing auto_entityset, I've found that for entity 0 (refer to line 330 in test_normalize.py) the name changes between 'jersey_num_team' and 'team_jersey_num' for different runs. This causes the following to fail occasionally:
assert normalized_entityset.entities[0].variable_types['jersey_num_team'] == Index
How would you go about testing this?

@j-grover Thanks for creating these tests. If we expect this behavior - where the name is not deterministic - we could set a variable for the name that is actually returned and then use that variable in the tests. One way that comes to mind would be to check if team_jersey_num is in the variable_types dictionary keys and if it is not we would use the other option of jersey_num_team. Something like this:
# Index name is not always the same - checking what was returned
index_vname = "jersey_num_team"
if index_vname not in normalized_entityset.entities[0].variable_types.keys():
    index_vname = "team_jersey_num"
assert normalized_entityset.entities[0].variable_types[index_vname] == Index

I think you would also need to rename one of the dataframe columns for the df.equals test to pass reliably as well.
I don't know enough about the details of this code to know if expect to get the same name back every time, but I can look into that a bit more in the meantime to make sure this isn't highlighting some other issue.

@j-grover After reviewing the code with @rwedge , the non-deterministic nature of column names is to be expected as the new names are created by joining an unsorted list of column names. Issue #24 was created to fix this problem, so for this PR I suggest we go ahead and implement the tests as we have described above, and then we can update later after issue #24 is closed.

@j-grover After reviewing the code with @rwedge , the non-deterministic nature of column names is to be expected as the new names are created by joining an unsorted list of column names. Issue #24 was created to fix this problem, so for this PR I suggest we go ahead and implement the tests as we have described above, and then we can update later after issue #24 is closed.

Sorting the list sounds good. I have change the index names accordingly to jersey_num_team.
@rwedge I've added a pytest fixture for that particular example

authors: CLAassistant j-grover rwedge thehomebrewnerd

repo: FeatureLabs/autonormalize


text:
Dask vs Pandas: Predict Next Purchase Comparison

We need to make two versions of the predict next purchase demo using both Dask dataframes and Pandas dataframes.  There should be a clear starting point where the two demos diverge and we can compare timings from that point forward.

A comparison was made between the original predict next purchase notebook using a Dask Bag approach and the new featuretools approach. The comparison includes only the primitives that are currently supported in the Dask implementation, and does not use the training_window parameter which is not currently supported for Dask dataframes. The output was compared to confirm the two feature matrices were equal after accounting for row/column sort order differences.
A summary of the total times for each approach are listed below:
Original Notebook

Preprocessing files: 555 seconds
Loading sample data to generate features: 11 seconds
Creating label times: <1 second
Running DFS: <1 second
Computing feature matrix and save to CSV: 402 seconds
TOTAL: 969 seconds

New Dask Implementation

Preprocessing files: 766 seconds
Creating entityset, normalizing, and adding last time indexes: <1 second
Creating label times: 43 seconds
Running DFS: 1 second
Computing feature matrix and save to CSV: 476 seconds
TOTAL: 1287 seconds

source: gharchive/issue

authors: rwedge thehomebrewnerd

repo: FeatureLabs/featuretools

license_type: permissive

license_source: github-api


text:
(suggestion) add snake

no fucking way this gets accepted

:skull: reaction stax

what   does the snake do

bro has no childhood :moyai:

what does  the snake do

its a game dumbass

@everyone

snake is an animal

snake is an animal

fr

repo: Fedoraware/Fedoraware

license: WTFPL


text:
Release v0.2-alpha.2

Gotta get another release out!

[x] https://github.com/Fenyx4/U4DosRandomizer/issues/34 - Hythloth Fix
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/46 - Spoiler Log
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/48 - Start with quest items
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/47 - Starting virtues
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/45 - U4 Remastered (C64)
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/43 - U4 v1.01 fixes
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/38 - Fix healer in Serpent's Hold
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/32 - Map randomization optional

Avatar.exe changes

[x] https://github.com/Fenyx4/U4DosRandomizer/issues/50 - Output randomizer flag values
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/44 - Decrease chance of sleep lock
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/42 - Hit Chance -> Apple II
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/41 - Fix Sacrifice - Last coin
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/29 - Set active player
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/57 - Diagonal attack

Integrated Avatar.exe changes

[x] https://github.com/Fenyx4/U4DosRandomizer/issues/50
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/44
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/42
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/41
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/29
[x] https://github.com/Fenyx4/U4DosRandomizer/issues/57

It's done and out!

It's done and out!

license: MIT


text:
While adding social media links special character '/' should also be allowed.

Open "https://staging.fightpandemics.work"
Login using valid email/password
Click on 'Profile' -> select Org-> 'EDIT Profile icon'
Click 'EDIT Profile'
EDIT the Organisation Linkedin/twitter link [as shown in below screenshot]

Issue: You will observe that:
Following errors are displayed if there are max characters or special character like '/'

Invalid entry: only alphanumeric characters and _ are allowed
Max. length is 15 characters [Not Critical]

Please note earlier #795, was also fixed similar to this one, as now I am able to user '. DOT, - hypen' in the URLs.
Marking it as critical because, as an Organisation, unable to add my Organisation Linkedin URL.

@vikjoshi : Feel free to change the labels

@rabbi08 - I do not think you can have company/fight pandemics in the url because they dont allow for / in setting the public URL. So I actually do not think that needs to be modified.
We definitely should not have a character limit though if twitter and linkedin do not have one

@rabbi08 - I do not think you can have company/fight pandemics in the url because they dont allow for / in setting the public URL. So I actually do not think that needs to be modified.
We definitely should not have a character limit though if twitter and linkedin do not have one

@vikjoshi Actually on linkedin every company URL is like this
company/google/
Below are the examples/reference of the linkedin links of the Organizations:
https://www.linkedin.com/company/google/ [Complete URL]
so if we do not enter 'company/......' the link don't work
https://www.linkedin.com/company/time/
https://www.linkedin.com/company/fightpandemics/
https://www.linkedin.com/company/cisco/
https://www.linkedin.com/school/rutgersu/

Ahh @rabbi08 thanks, so I guess the public url link settings are different for org vs individual.
@jyclam, looks like you assigned yourself to this- thank you! Lets make the following changes:

Go ahead and update the permissions here to allow for '/' and remove from the linkedin URL, it should be: https://www.linkedin.com/

Lets remove the character max limit on all fields for now

For Apple Store and Google Play, can you remove the invalid error permissions altogether? It looks like special characters are part of those url so I do not think we should have any error handling for those two fields

FYI- @joshmorel and @idjevm this is needed for soft launch

@vikjoshi Noted, will do

@jyclam @idjevm
As stated by @vikjoshi please fix google and apple links, as per example I got for google play store:
Sample test URL for google play:
https://play.google.com/store/apps/developer?id=Mutual+Aid
and again issue with ? =
Invalid entry: only alphanumeric characters are allowed
So I think special characters should be allowed, don't have sample TEST URL for apple.
But reference URL creation, shared by vik
https://support.google.com/admob/answer/3086746?hl=en

@rabbi08 noted, thanks for details

Discussed with team to add LinkedIn prefixes /in and /company for individuals and companies respectively.

@jyclam
but what about, universities/schools as they have category as '/school/' like
https://www.linkedin.com/school/rutgersu/

WEIRD! issues are not being automatically closed :O not sure why (might be because master is not default)

Also @rabbi08 there is not company/ for twitter.

Thank you @jyclam and @idjevm ! It looks good to me- @rabbi08 feel free to test as well

This is working fine I am able to add following links for the linkedin:
https://www.linkedin.com/school/rutgersu/
https://www.linkedin.com/company/fightpandemics/
Thanks

authors: idjevm jyclam rabbi08 vikjoshi

repo: FightPandemics/FightPandemics


text:
🛑 logs.mmattbot.com is down

In 6369c24, logs.mmattbot.com (https://logs.mmattbot.com/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: logs.mmattbot.com is back up in 0711070 after 5 minutes.

repo: Fijxu/justlog-instances-uptime


text:
🛑 logs.lucas19961.de is down

In 8aadd64, logs.lucas19961.de (https://logs.lucas19961.de/) was down:

HTTP code: 502
Response time: 24159 ms

Resolved: logs.lucas19961.de is back up in 34d9eb5.


text:
Please remove dependencies on rgdal, rgeos, and/or maptools

This package depends on (depends, imports or suggests) raster and one or more of the retiring packages rgdal, rgeos or maptools (https://r-spatial.org/r/2022/04/12/evolution.html, https://r-spatial.org/r/2022/12/14/evolution2.html). Since raster 3.6.3, all use of external FOSS library functionality has been transferred to terra, making the retiring packages very likely redundant. It would help greatly if you could remove dependencies on the retiring packages as soon as possible.

https://github.com/FilgueirasR/cropZoning/issues/3.
You will be aware, for example from:
https://r-spatial.org/r/2022/04/12/evolution.html,
https://r-spatial.org/r/2022/12/14/evolution2.html,
https://r-spatial.org/r/2023/04/10/evolution3.html and
https://rsbivand.github.io/csds_jan23/bivand_csds_ssg_230117.pdf and
perhaps view https://www.youtube.com/watch?v=TlpjIqTPMCA&list=PLzREt6r1NenmWEidssmLm-VO_YmAh4pq9&index=1
that rgdal, rgeos and maptools will be retired this
year, in October 2023.
Strong dependency on rgdal must be fixed quickly, best before June 2023. Replace with rgdal::readOGR with terra::vect and coerce to sp if need be using methods in raster; see this week's blog.

Only three months remain to save this package. Please act now!

I will remove the dependencies. Thank you for remembering.

@FilgueirasR Less than three weeks remain to fix this.

@FilgueirasR R spatial infrastructure packages maptools, rgdal and rgeos will be archived by CRAN on Monday, October 16, 2023. Your package does not pass CMD check when these packages are not available. Expect your package to be archived by CRAN October 17-18 as CRAN checks feed through and your package fails, if not updated by Monday, October 16, 2023.
No grace period is anticipated, as you have had sufficient time to update your package to remove dependencies on maptools, rgdal and/or rgeos. It remains the case that many packages importing the raster package needlessly depend on retiring packages, as raster stopped using them a year ago.

@rsbivand, I will fix the cropDemand package this week.

authors: FilgueirasR rsbivand

repo: FilgueirasR/cropDemand


text:
:bug: Omitted logs in release mode

This PR makes sure logs are omitted in release mode. Based on the logger's recommendation

I did a reimplementation of the filter class

This has been fixed in the latest package

source: gharchive/pull-request

repo: FilledStacks/stacked


text:
mkcert failed adding cert: Access is denied.

Hi,
I tried to use mkcert on Windows.
However when I use mkcert -install, it is failed because access is denied.
I dont know the problem because I use cmd as administrator.

On windows mkcert -install must be executed under elevated Administrator privileges. Open the command prompt as Administrator and try again.

Hi
From the image it doesn't look like that cmd is being run as admin; did you right-click on the "command prompt" entry in the menu and choose "run as administrator"? When I do that, it looks like the image attached - note the "Administrator:" in the window title.

@arafel @Strandedpirate Hi, thanks for your respond. My bad, I forgot to give image with cmd run as administrator.
I couldn't find any source with help.

I have the same issue.

I also have this issue.

same

same

i have this issue too.

Looks like this is a permission issue with the keys stored in C:\ProgramData\Microsoft\Crypto\RSA\MachineKeys. When I try to modify the permission of the parent folder to give the administrator full control for not just the folder, but also subitems, I get the following simialr error:

I haven't been able to resolve the situation yet. I even deleted the files altogether and rebooted, to no avail.

Until more is known about this issue, you can use the following workaround to install the CA:

Press Windows+R and run certmgr.msc
Right-click on Trusted Root Certification Authorities > All Tasks > Import...
(At this point, the Store Location was greyed out, and Current User was preselected for me)
Click Browse... > point it to the rootCA.pem file usually located in %localappdata%\mkcert > Next. When in doubt, double-check the output of mkcert -install to find out the root CA path.
Select Place all certificates in the following store > Browse... > Check Show physical stores > Expand Trusted Root Certification Authorities > Select Local Computer > OK > Next > Finish
The message The import was successful. should appear > Click OK
Verify that your CA is recognized:

Note: If you don't explicitly select the physical store Local Computer, you'll very likely run into the following error message, so make sure not to skip that part. This might be linked to the mkcert -install issue we run into.

I have a similar issue. I was able to mkcert -install, but now I'm unable to do anything related to mkcert:

@cheslijones that's mkcert being unable to reaad the CA that's in your own home directory. You need to change the permissions on that directory (AppData\Local\mkcert, or consider deleting the whole directory and doing mkcert -install again.  Or is C:\Users\work possibly another user's homedir?

Ok, I'll delete and retry. That is my homedir.

Until #453  is merged I created a PowerShell script a while back that'll install the RootCA certs for you and replace it if the thumbprints are different/tell you if its already installed. Feel free to use if it if its to your liking. Just run it in the same directory as the rootCA pem files
Gist link

I was able to resolve this by adding the mkcert executable to the exclusions list in Windows Defender.

authors: JohnKarel LichLord91 Strandedpirate adem arafel bstiffler582 cheslijones davidsiagian fabianberisha hifull julian-code panlazy rfay

repo: FiloSottile/mkcert


text:
Rock Crystals Not Found

Are rock crystals from Astral Sorcery actually enabled in the ore gen?
Based on astralsorcery.cfg, they are disabled:
# Generate rockcrystals [default: true]
B:Generate=false
Now, there could be custom gen somewhere, but I don't know how to find that =)
I tried looking at the "orespawn" files, but didn't see any mention of them there.
I've tried using the resonating wand to locate them, but that doesn't work (even if I place them in creative).
I've tried clearing out dozens of chunks worth of blocks near bedrock, but found none that way either.
Am I wasting my time?

Astral Rock Crystals do spawn, they are in deep coal veins between Y levels 0-20.
Config is located at: \config\cofh\world/05_new_ore_gen.json.

Looks like this

Ah, cool. I've found those, but can't mine diamond yet, so maybe it's stuck inside.
Thanks!

No problem, thanks for the report! I'm looking into why Rockcrystal ore doesn't glow.

My guess is the glow is tied to the default ore gen, marking the locations as it generates chunks.
Since that's disabled, nothing is "marked" along the way.
No idea if it's possible to hook into that in "configuration" without it being built into to the custom ore gen mod (or maybe do a post-gen scan).

Seems to be the case. I’m testing turning it back on, as well as keeping the crystals in the deep ore veins

authors: Filostorm tpaukovi

repo: Filostorm/Multiblock-Madness


text:
Esp is not connecting to the firebase. This is my project code i am not able to receive the data in the firebase database. what my be the error.

#include <FirebaseArduino.h>
#include <ESP8266WiFi.h>
#define FIREBASE_HOST "arduinotest-f6d5e.firebaseio.com"
#define FIREBASE_AUTH "u65MmIKJqyvHoUmG6hMMK0KoCuMYl7ZRF06Y4JFG"
#define WIFI_SSID "kick"
#define WIFI_PASSWORD "12345678"
float fire;
void setup() {
Serial.begin(9600);
pinMode(D5, OUTPUT);
pinMode(D6, OUTPUT);
pinMode(A0, INPUT);
// connect to wifi.
WiFi.begin(WIFI_SSID, WIFI_PASSWORD);
Serial.print("connecting");
while (WiFi.status() != WL_CONNECTED) {
Serial.print(".");
delay(500);
}
Serial.println();
Serial.print("connected: ");
Serial.println(WiFi.localIP());
Firebase.begin(FIREBASE_HOST, FIREBASE_AUTH);
Firebase.set("LED_STATUS", 0);
}
int n = 0;
void loop() {
int analogValue = analogRead(A0);
float millivolts = (analogValue/1024.0) * 3300; //3300 is the voltage provided by NodeMCU
float celsius = millivolts/10;
Serial.print("in DegreeC=   ");
Serial.println(celsius);
delay(1);
if(celsius>35){
digitalWrite(D6,HIGH);
Firebase.setInt("fire_sensor_status",1);
}
n = Firebase.getInt("LED_STATUS");
// handle error
if (n==1) {
Serial.println("LED ON");
digitalWrite(D5,HIGH);
return;
delay(10);
}
else {
Serial.println("LED OFF");
digitalWrite(D5,LOW);
digitalWrite(D6,LOW);
Firebase.setInt("fire_sensor_status",0);
return;
}
}

Your problem in Fingerprint, the fingerprint of firebase is changing every short period, So the library doesn't work and you need to change fingerprint with a new one.
I advise you to use the best version of this Library and the new one not based on Fingerprint so if changed any time you will not worry about that.
https://github.com/mobizt/Firebase-ESP8266
but if you want to use this old library you can follow the instructions here to change fingerprint to a new one #439.

repo: FirebaseExtended/firebase-arduino

license_source: bigquery