text:
Could not delete competition

When trying to remove this competition (as the primary organizer of the competition), we obtain the following error.
https://www.codabench.org/competitions/1002/?secret_key=5781b096-f6a8-49a4-970a-f0f05a3a5d3b

Here is the Django error logs on the platform:
django_1          | Internal Server Error: /api/competitions/1002/
django_1          | Traceback (most recent call last):
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner
django_1          |     response = get_response(request)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 115, in _get_response
django_1          |     response = self.process_exception_by_middleware(e, request)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 113, in _get_response
django_1          |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
django_1          |     return view_func(*args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/viewsets.py", line 116, in view
django_1          |     return self.dispatch(request, *args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 495, in dispatch
django_1          |     response = self.handle_exception(exc)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 455, in handle_exception
django_1          |     self.raise_uncaught_exception(exc)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 492, in dispatch
django_1          |     response = handler(request, *args, **kwargs)
django_1          |   File "/app/src/apps/api/views/competitions.py", line 224, in destroy
django_1          |     return super().destroy(request, *args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/mixins.py", line 93, in destroy
django_1          |     self.perform_destroy(instance)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/mixins.py", line 97, in perform_destroy
django_1          |     instance.delete()
django_1          |   File "/app/src/apps/chahub/models.py", line 143, in delete
django_1          |     super().delete(*args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/base.py", line 921, in delete
django_1          |     collector.collect([self], keep_parents=keep_parents)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/deletion.py", line 224, in collect
django_1          |     field.remote_field.on_delete(self, field, sub_objs, self.using)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/deletion.py", line 22, in PROTECT
django_1          |     raise ProtectedError(
django_1          | django.db.models.deletion.ProtectedError: ("Cannot delete some instances of model 'Competition' because they are referenced through a protected foreign key: 'Data.competition'", <QuerySet [<Data: fnachalearn - submission(15789)>, <Data: fnachalearn - submission(15790)>]>)

I've seen that the organizer was in status "denied" from the competition, but even when approved the bug remains.

I think this log describe the problem:
django_1          | django.db.models.deletion.ProtectedError: ("Cannot delete some instances of model 'Competition' because they are referenced through a protected foreign key: 'Data.competition'", <QuerySet [<Data: fnachalearn - submission(15789)>, <Data: fnachalearn - submission(15790)>]>)

I've seen that the organizer was in status "denied" from the competition, but even when approved the bug remains.

I think this log describe the problem:

When trying to remove this competition (as the primary organizer of the competition), we obtain the following error.

https://www.codabench.org/competitions/1002/?secret_key=5781b096-f6a8-49a4-970a-f0f05a3a5d3b
Capture d’écran 2023-06-09 à 11 41 03

Here is the Django error logs on the platform:

django_1          | Internal Server Error: /api/competitions/1002/
django_1          | Traceback (most recent call last):
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/exception.py", line 34, in inner
django_1          |     response = get_response(request)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 115, in _get_response
django_1          |     response = self.process_exception_by_middleware(e, request)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/core/handlers/base.py", line 113, in _get_response
django_1          |     response = wrapped_callback(request, *callback_args, **callback_kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
django_1          |     return view_func(*args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/viewsets.py", line 116, in view
django_1          |     return self.dispatch(request, *args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 495, in dispatch
django_1          |     response = self.handle_exception(exc)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 455, in handle_exception
django_1          |     self.raise_uncaught_exception(exc)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/views.py", line 492, in dispatch
django_1          |     response = handler(request, *args, **kwargs)
django_1          |   File "/app/src/apps/api/views/competitions.py", line 224, in destroy
django_1          |     return super().destroy(request, *args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/mixins.py", line 93, in destroy
django_1          |     self.perform_destroy(instance)
django_1          |   File "/usr/local/lib/python3.8/site-packages/rest_framework/mixins.py", line 97, in perform_destroy
django_1          |     instance.delete()
django_1          |   File "/app/src/apps/chahub/models.py", line 143, in delete
django_1          |     super().delete(*args, **kwargs)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/base.py", line 921, in delete
django_1          |     collector.collect([self], keep_parents=keep_parents)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/deletion.py", line 224, in collect
django_1          |     field.remote_field.on_delete(self, field, sub_objs, self.using)
django_1          |   File "/usr/local/lib/python3.8/site-packages/django/db/models/deletion.py", line 22, in PROTECT
django_1          |     raise ProtectedError(
django_1          | django.db.models.deletion.ProtectedError: ("Cannot delete some instances of model 'Competition' because they are referenced through a protected foreign key: 'Data.competition'", <QuerySet [<Data: fnachalearn - submission(15789)>, <Data: fnachalearn - submission(15790)>]>)

For more context, the "error comp" cannot be deleted after I delete my submissions by checking the box and select Delete submissions as below:

instead of clicking the red bin:

To reproduce the issue, you can try deleting the submissions by checking the box and select Delete submissions as below:

instead of clicking the red bin:

Trying to reproduce this

New competition without any submission -> Deleted successfully
New competition with one submission uploaded  -> Deletion failed

I think this problem is there because of new addition to submission model here:
https://github.com/codalab/codabench/blob/f55e5ccd15027a29a6068e46ae23cf977e231446/src/apps/datasets/models.py#LL61C12-L61C12
changing models.PROTECT to models. CASCADE  will solve the problem

@ihsaan-ullah wow you are getting quicker to fix issues everyday!
I think our automatic tests should have catch this issue. We should add a "delete competition" test.

authors: Didayolo ihsaan-ullah ktgiahieu

repo: codalab/codabench


text:
[MODULE] - New York times news search

Please describe the module you would like to add to bricks
I want to use the New York times news API to fetch news on specific topics.
Do you already have an implementation?
Additional context
Requires API key.

Implemented with #212

source: gharchive/issue

repo: code-kern-ai/bricks

license_type: permissive

license_source: github-api


text:
Chore/linter tox

similar to the recent change in py42:

no logic has changed,  the formatter and flake8 rules touched a lot of files

I mostly would like this review to be of the CONTRIBUTING document to make sure it all makes sense

one of the formatters that auto-sorts imports also breaks up from imports into multiple lines, and I think I would rather have this be import module as m and then use m.do_the_thing(), m.do_the_other_thing() instead of from module import do_the_thing, do_the_other_thing, but i'll worry about that in a separate PR.

templates for issues and PRs and build / code coverage badges have also been added.

Codecov Report

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

@@            Coverage Diff            @@
##             master     #116   +/-   ##
=========================================
  Coverage          ?   91.72%
=========================================
  Files             ?       31
  Lines             ?     1886
  Branches          ?        0
=========================================
  Hits              ?     1730
  Misses            ?      156
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e2090c...3660f25. Read the comment docs.

source: gharchive/pull-request

license: MIT


text:
Error creating app offering

Hello,
I tried to install a sample application in our QA account but this ends up with error
Unable to create app offering: "Unable to query Create app offering"
Failed to upsert app offering: "Unable to query Create app offering"
After an additional console log output the error seems to be
Errors from GraphQL call: [{"message":"Parse error on \".\" (error) at [3, 8]","locations":[{"line":3,"column":8}]}]
All other GraphQL-Calls were successful (adding configuration items, service instances...).
Any hint on this how to fix?
Regards
Patrick
Edit:
Original error message is:
Unable to create app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Failed to upsert app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Translated message is like: "Reference has already been used"

@pfleckenstein  Thanks for reporting this.
To be able to add a new app offering you have to provide a reference. These references have to be unique across all 4me accounts and it seems like the value you are sending is already in use. So to be able to install the app you will have to select another value to use as reference for your app offering and use that instead of the value you are currently sending.
It seems like the error handling in this project doesn't seem to provide you the error message in a good/clear way. Can you maybe share the full response you get from 4me (which contains the 'Original error message' you refer to)?
I'm not quite sure what you mean by "After an additional console log output the error seems to be". Is that something you already see in the log, or did you add an additional log statement yourself, or ...?

Thanks for your reply.
Neither in our QA nor in our production environment any app offer exist.
Is the reference predefined in the examples (e.g. lansweeper, note-dispatcher...)?
What I understand is that if no app offer exist it will be created.
Otherwise it will be updated. In my scenario both actions failed.
However, I renamed the reference in the json-config file and it seems to work now.
Thank you.
With "additional console log output" I meant I tried to debug the code with console.log().
Please find the complete default output below after running npm run deploy-lansweeper.
> 4me-app-builder@0.0.1 deploy-lansweeper
> node lansweeper/deploy_integration.js

asset ./.aws-sam/build/IntegrationFunction/app.js 630 KiB [compared for emit] [minimized] (name: IntegrationFunction) 2 related assets
orphan modules 674 KiB [orphan] 538 modules
runtime modules 937 bytes 4 modules
built modules 1.34 MiB [built]
  modules by path ../../node_modules/ 1.23 MiB
    cacheable modules 1.11 MiB 175 modules
    + 3 modules
  modules by path ./integration-lambda/ 80.5 KiB
    modules by path ./integration-lambda/*.js 80 KiB 12 modules
    modules by path ./integration-lambda/errors/*.js 490 bytes 2 modules
  modules by path ../../library/helpers/ 28.3 KiB
    modules by path ../../library/helpers/*.js 27.3 KiB 9 modules
    modules by path ../../library/helpers/errors/*.js 1010 bytes 3 modules
  + 14 modules
webpack 5.89.0 compiled successfully in 24572 ms

                Managed S3 bucket: aws-sam-cli-managed-default-samclisourcebucket-usqrxyrder8e
                A different default S3 bucket can be set in samconfig.toml
                Or by specifying --s3-bucket explicitly.
File with same data already exists at app-builder-lansweeper/123 skipping upload

        Deploying with following values
        ===============================
        Stack name                   : app-builder-lansweeper
        Region                       : us-east-1
        Confirm changeset            : False
        Disable rollback             : False
        Deployment s3 bucket         : aws-sam-cli-managed-default-samclisourcebucket-usqrxyrder8e
        Capabilities                 : ["CAPABILITY_IAM"]
        Parameter overrides          : {"4MeDomainParameter": "4me.qa", "BootstrapSecretApplicationParameter": "4me-app-builder", "BootstrapSecretAccountParameter": "myCompany", "OfferingReferenceParameter": "lansweeper"}
        Signing Profiles             : {}

Initiating deployment
=====================

File with same data already exists at app-builder-lansweeper/123.template, skipping upload

Waiting for changeset to be created..

CloudFormation stack changeset
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Operation                                    LogicalResourceId                            ResourceType                                 Replacement
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
* Modify                                     IntegrationFunctionCWSchedulePermission      AWS::Lambda::Permission                      Conditional
* Modify                                     IntegrationFunctionCWSchedule                AWS::Events::Rule                            False
* Modify                                     IntegrationFunctionSecretsEventListenerPer   AWS::Lambda::Permission                      Conditional
                                             mission
* Modify                                     IntegrationFunctionSecretsEventListener      AWS::Events::Rule                            False
* Modify                                     IntegrationFunction                          AWS::Lambda::Function                        False
* Modify                                     ServerlessRestApi                            AWS::ApiGateway::RestApi                     False
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Changeset created successfully.

2024-01-23 10:38:53 - Waiting for stack create/update to complete

CloudFormation events from stack operations (refresh every 5.0 seconds)
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
ResourceStatus                               ResourceType                                 LogicalResourceId                            ResourceStatusReason
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
UPDATE_IN_PROGRESS                           AWS::CloudFormation::Stack                   app-builder-lansweeper                       User Initiated
UPDATE_IN_PROGRESS                           AWS::Lambda::Function                        IntegrationFunction                          -
UPDATE_COMPLETE                              AWS::Lambda::Function                        IntegrationFunction                          -
UPDATE_COMPLETE_CLEANUP_IN_PROGRESS          AWS::CloudFormation::Stack                   app-builder-lansweeper                       -
UPDATE_COMPLETE                              AWS::CloudFormation::Stack                   app-builder-lansweeper                       -
---------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

CloudFormation outputs from deployed stack
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Outputs
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
Key                 DeadLetterQueueUrl
Description         SQS dead-letter queue URL
Value               https://sqs.us-east-1.amazonaws.com/123

Key                 IntegrationFunctionIamRole
Description         Implicit IAM Role created for Integration Function
Value               arn:aws:iam::123:role/123

Key                 RefreshQueueUrl
Description         SQS queue for refresh URL
Value               https://sqs.us-east-1.amazonaws.com/123/123

Key                 RefreshQueueArn
Description         SQS queue for refresh ARN
Value               arn:aws:sqs:us-east-1:123:lansweeper-refresh-trigger

Key                 DeadLetterQueueArn
Description         SQS dead-letter queue ARN
Value               arn:aws:sqs:us-east-1:123:lansweeper-dead-letter

Key                 IntegrationApi
Description         API Gateway endpoint URL for Prod stage for Integration Function
Value               https://123.execute-api.us-east-1.amazonaws.com/Prod/integration/

Key                 IntegrationFunction
Description         Integration Lambda Function ARN
Value               arn:aws:lambda:us-east-1:123:function:app-builder-lansweeper-IntegrationFunction-123
----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------

Successfully created/updated stack - app-builder-lansweeper in us-east-1

Created lambda "arn:aws:lambda:us-east-1:123:function:app-builder-lansweeper-IntegrationFunction-123" at: "https://123.execute-api.us-east-1.amazonaws.com/Prod/integration/"
Used S3 bucket aws-sam-cli-managed-default-samclisourcebucket-123for storage
Unable to create app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]
Failed to upsert app offering: [{"path":["input","appOfferingReference.reference"],"message":"Referenz wurde bereits verwendet"}]

@pfleckenstein the app offering reference must be unique across all accounts, this means you will never be able to create app offerings using the default values in this project (for instance in lansweeper/config/4me/app_offering_input.json) as we already used those values for an app offering in the '4me' account. (In your example the Lansweeper app we offer.)
I believe this issue can be closed now, as changing the reference value worked for you.

authors: fhoeben pfleckenstein


text:
Roman and Integer

Roman and Integer
** I Can add a Def in the RomanToInt.py file for Integer to Roman **
can I work on this code and build my idea?

can u assign me this?

@EmadDeve20 Yes Sure, Go ahead. Please read the contribution guidelines for a successful Merge.
Best of Luck for HacktoberFest.

Can you assign this issue? I have already implemented the program for converting Integer into Roman.


text:
OpenAPI docs

Are there any plans for supporting automatic OpenAPI docs generation or some other method to quickly generate API clients?

Definitely there are at some point.
You are welcome to contribute OpenAPI module as well.

The work has started on the openapi branch, you are welcome to review

openapi module released in 1.6.0

authors: FeldrinH angryziber

repo: codeborne/klite


text:
执行BBDown login的时候m1 报以下错误

A type initializer threw an exception. To determine which type, inspect the InnerException's StackTrace property.

Can you give more details? Sorry, but this is not enough information to start analyzing your problem. 😞
Note: I checked your main repository and saw that you target net6.0. So I can guess that you may have the same problems as described in #354 - at least if you run your app on any other platform than windows. But without more information this is only a guessing.

Hi @ge2009 ,
I made some changes, that possibly solve your issues. Can you please take the latest CI-build and check if it's working for you? CI-Build Packages
If you need help on how to use Github Packages as NuGet package source, let me know (or check this article).

Issue should be solved with the new QRCoder 1.4.3 release: https://www.nuget.org/packages/QRCoder/1.4.3

repo: codebude/QRCoder

license: mit

license_source: bigquery


text:
Update de-DE.coffee

Added translations for "Accounts Subscription View" and some other parts.

Hi, @swoeste! Thank you for contributing! Could you  sign the CLA so that I can merge your pull request?

Thanks.

repo: codecombat/codecombat


text:
Update es-ES.coffee

Some visible strings on the UI for a free account.

Thank you for contributing, @ilo


text:
Update sk.coffee

clans

Coverage increased (+0.01%) to 56.191% when pulling 29f34bcd90d1081b299217fce4eaf2df76452a67 on cundamic:patch-58 into 5fe3a75464a3f42e8f1e810740e4b00691cc902e on codecombat:master.

Thanks!

authors: Bryukh coveralls cundamic


text:
Skipping ssl validation for openstack

Hi,
Is it possible to skip ssl validation for openstack driver. I see gopher has an ignore_cert_warning = "true" but is it applicable in rexray. I tried with insecure flag as true. But didnt work.Also no logs seem to get generated to see exact errors. This is the config file
         rexray:

storageDrivers:
openstack
openstack:
authURL:             https://identity-test.eu-de-1.cc.test.corp:5000/v3
userName:             admin
password:             password
tenantName:           dev07
domainName:           TEST_CF
availabilityZoneName:           rot
insecure:             true

@shashankmjain I just looked through the code and indeed the insecure param was not implemented. Looking at a fix now.

@shashankmjain Please check this PR for fix https://github.com/emccode/rexray/pull/370

Closing due to inactivity. OpenStack can be revisited if new openstack driver is merged via https://github.com/codedellemc/libstorage/issues/182

repo: codedellemc/rexray


text:
Jersey is detected as country on the index page, but it should be UK

Events in Jersey are successfully added under UK by the Google Maps API, but the index page detects Jersey as a different country, so clicking on "List all events in Jersey" returns no results.

To look more into the implementation.

authors: ialja oana-sipos

repo: codeeu/coding-events


text:
Volume conversion tool

Create a Volume conversion tool in html,css and javascript.

Add navbar in your tool's html file. You can copy the navbar from index.html.
Add screenshot of your tool in, PR message and also in the folder of your tool
Before Making the PR Pull the latest changes that have been made to the repository

**Note: Please Don't Make Any Changes In index.html AND contributors.html **
Add the below mention section to your PR message
Title
Number of files/folder added
changes made
I have;

[ ] Read contibuting.md
[ ] Followed all the guidelines
[ ] updated readme.md
[ ] add your name in the footer of your tool.html file (if you created length conversion tool then add your name to length.html
file you created)

@shubham2704 I would like to work on this issue.

Is this still open sir ?

@programmerraja Yes this issue is still open.

@shubham2704 i completed this sir can i pull my request

@shubham2704  sir can you take look my pull request #136
Just 3 days to go sir to complete hacktoberfest

@programmerraja very sorry for the delay but we were facing some bug issues. That's why we didn't merged the new pull request.
Don't worry I will mark both of you pull request as hacktoberfest-accepted if they don't have any conflicts.

@shubham2704  Ok sir thank you so much 👍

repo: codeezzi/utility


text:
Angle conversion tool

Create a Angle conversion tool in html,css and javascript.

Add navbar in your tool's html file. You can copy the navbar from index.html.
Add screenshot of your tool in, PR message and also in the folder of your tool
Before Making the PR Pull the latest changes that have been made to the repository

**Note: Please Don't Make Any Changes In index.html AND contributors.html **
Add the below mention section to your PR message
Title
Number of files/folder added
changes made
I have;

[ ] Read contibuting.md
[ ] Followed all the guidelines
[ ] updated readme.md
[ ] add your name in the footer of your tool.html file (if you created length conversion tool then add your name to length.html file you created)
[ ] provide below mentioned details if you want your name to be added in contributors.html
[ ] mail, twitter, github profile link.

can I work on this issue

thanks a lot

@mrunalvilas read the new guidelines:

Add navbar in your tool's html file. You can copy the navbar from index.html.
Add screenshot of your tool in, PR message and also in the folder of your tool
Before Making the PR Pull the latest changes that have been made to the repository

**Note: Please Don't Make Any Changes In index.html AND contributors.html **

hey I haven't seen anyone updating readme for the features, where should i add the description of the tool i am adding?

@mrunalvilas don't worry on that it will be added by our team.
🎉 Good work on the PR 🥇


text:
1.0.12 fires stop BEFORE drop. 1.0.6 fires stop AFTER drop

Any idea why this changed?  I upgraded to 1.0.12 and it broke my code - clean up I was doing in stop was removing stuff I needed in drop

@poshest A demo would be helpful to understand the problem

Duplicate of #241.

This is not a duplicate of #241.  It has nothing to do with different versions of Angular. Re the "demo" do you have a jsfiddle already set up with the basics, so that I can copy it?

Goto any example and fork the plunkr at http://codef0rmer.github.io/angular-dragdrop/#/

@poshest: BTW I presumed it to be duplicate of #241 because I though you might have run the digest loop in the drop callback which eventually resolve the promise of beforeDrop.
I would be interested to understand the problem by a demo?

This was as far as I got with a Plunkr based on 1.0.6, but I've linked it only for reference.  I was unable to reproduce the drop before stop in the Plunkr that I'm seeing in my code.
In any case, it's clear that stop before drop is the standard, and I should just upgrade to 1.0.12 and handle the error some other way.
As a side note, I don't understand why anyone would want stop before drop - I use drop to process the dropped object, and stop to clean up the dropped object, which I thought would be the most common use case. But there are obviously good reasons to do stop then drop.


text:
Implement user_building spec

Issue #154: Implement User Building model test

The test looks good and works but can you amend the commit to not check in the log file?

Overall looks great. Another thing that I think should change are tabs to double space indentation, just to keep consistent with the project 😄

Pulled down, test passes. Looks great! I will leave it @mzagaja to merge & close since I suspect there are some pipeline issues being worked out as I type 😄

Going to merge this in and then we'll update it when @mahtai updates the unit model.

authors: itiswicked mzagaja veddster


text:
Add unit tests for Flask website.

If you haven't already, go through the "Flaskr" tutorial. Then after you're comfortable with it, check out its unit-tests here Copy as much as is relevant inside the conftest.py file and start writing a unit test that tests the website called test_site.py.

Working on this

submitted a draft PR with comments! one test passes, one doesn't

repo: codeforboston/flagging


text:
About BHPA Mobile Responsive Design

Mobile Responsive site design for About BHPA page.
Figma frame link: https://www.figma.com/file/KbNIbEz1n8R5EVXEQLRSEX/Heat-Pump?node-id=2107-12740&t=IL6Sgo9OwtxHGdBU-0

Done @thiagobardini
Review: https://www.figma.com/file/KbNIbEz1n8R5EVXEQLRSEX/Heat-Pump?node-id=2107-12740&t=HH786kTIiN2hyYmy-0 (Mobile Responsive)


text:
Adding a notification calendar/Progress bar for the user.

The user can get notifications of upcoming national level exams, competitions/Hackathons etc. Links to the competitions will be provided on a single click and the related courses can also be shown there.

Can I start working on this?  @Abhishek-kumar09

Great, fetching hackathons and programs is a part of backend, here yoou can create frontend and share design.

Is someone still working on this? I would like to work on this @Abhishek-kumar09 @vasudev19126

Yes @Abhijay007 I'll be submitting a PR soon.

Ok
On Thu, 22 Apr 2021, 22:49 kritarthmehra, @.***> wrote:

Yes @Abhijay007 https://github.com/Abhijay007 I'll be submitting a PR
soon.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/codeforcauseorg/edu-client/issues/189#issuecomment-825040495,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/APLHP3WK6LQR7H6I53XQICLTKBLDXANCNFSM42HWULPQ
.

@kritarthmehra are you still working on it?  if not, can I open a PR for this?

@Abhijay007 Go for it.

repo: codeforcauseorg/edu-client


text:
User Feedback?

When you demo'd the tool to us at DHCD today you mentioned you were exploring ways for users to provide feedback. An example of instance where that could be useful: I was searching projects and filtered the list down to the three largest projects. I noticed the Wingate Apartments among the largest. The Wingate Apartments was foreclosed on several years ago, and unfortunately affordability covenants do not survive foreclosure. So the affordability restriction ended suddenly.
This is an uncommon situation, but more than a few projects have similar nuances. Certain users (DHCD staff, housing counselors, etc.) may have this anecdotal knowledge and it would be great if there were a simple way to report these issues. For DHCD folks, we can modify our own Quickbase data but it seems like a limited number of fields are pulled from that database. Subsidy end date, for example, comes from the Preservation Catalog (right?).
Let me know if this isn't the right forum to post issues like this, I can email them instead or go another route. Anyways, great tool and I/we are really excited to take advantage of it. Thank you for all your work.

@ChrisDickersinProkopp Thanks for feedback! First, yes absolutely this is the best place to provide feedback - that way all the team members can see it, and we won't lose track of it.
RE: this specific issue, currently we are taking the provided data as-is from relevant sources. As you've identified we are not yet integrating many of the fields from Quickbase - mostly what we get right now from Quickbase is just the existence of a project.

[ ] Determine a way for users to easily provide feedback on specific projects ("Report an issue" button?)
[ ] Do a first-pass over the data with @ChrisDickersinProkopp, @ptatian  and others, and make a list of similar nuanced issues
[ ] Especially figure out how to better mark projects as inactive
[ ] Determine how best to fix the majority of these issues - providing a "patch" of override data, getting upstream data providers to change their data, provide better feedback loops for re-calculating fields provided by the PresCat (e.g. recalculate the subsidy end date in this example)

A related issue is that there are defiitely some additional integration steps that are needed to get the DHCD data and PresCat to be better integrated. I just want to flag this for you as we definitely know there is more to be done on this, so you'll likely see plenty more issues. Don't hesitate to report them here as you see them, but I think sitting down with you to do a once over will be important. Also, note that there is one big outstanding bug that I plan to fix before the 5th that is causing there to be a large number of duplicate records when we fail to find that a DHCD project is already in the PresCat - that issue is #583

@NealHumphrey Just to confirm, issue #583 is dealing with the situation where records from DHCD Quickbase are showing up in the subsidy info for projects already in the Catalog? For example, Tyler House. In this case, I think it would be better to omit the Quickbase entry, since it's not adding any information.
As a separate issue, I noticed that many of the project page subsidy entries say, "Some subsidies have missing information, so this chart cannot be displayed." What information is necessary for the chart to be shown? We might want to adjust the criteria because it seems the chart is rarely being displayed.

repo: codefordc/housing-insights


text:
Change colors of upper left image

Make them match blues in code for durham and code for america logos in the lower left.

Colors updated. Borders removed

authors: Toughee dsummersl

repo: codefordurham/adopt-a-drain


text:
simplegRPC analogue to simplerest

Is your feature request related to a problem? Please describe.
For modern applications gRPC APIs are being used as a substitution for REST APIs (for much better performance, smaller network and storage requirements). It would be nice to have a gRPC server could be tested by botium with a new simplegRPC CONTAINERMODE.
Describe the solution you'd like
botium.json could be configured with this new container mode, a URL, and a stub generated from a .proto file.
Describe alternatives you've considered
Additional context

added to the backlog

@codeforequity-at since it is in the backlog, should this stay opened?

@codeforequity-at there is a sample of loading proto here. Hope that helps.. Dialogflow for example is deprecating REST api in favor to gRPC.

To get things rolling, maybe you could start a botium connector project on your own, and I will add it to the core distribution as soon as it works ? It shouldn't be too complicated to make a first usefull version, it is comparable to the websocket connector I would say - https://github.com/codeforequity-at/botium-connector-websocket

@codeforequity-at I could try to help... since it is binary communication, there is no need for these JSON paths (but objects are serialized as Javascript Objects), HTTP methods or status codes.
The usual application with Botium I have uses:
{
  "botium": {
    "Capabilities": {
      "PROJECTNAME": "rest-bot",
      "CONTAINERMODE": "simplerest",
      "SCRIPTING_ENABLE_MEMORY": true,
      "SIMPLEREST_URL": "http://localhost:5000/reply",
      "SIMPLEREST_METHOD": "POST",
      "SIMPLEREST_HEADERS_TEMPLATE": "{ \"content-type\": \"application/json; charset=utf-8\" }",
      "SIMPLEREST_BODY_TEMPLATE": "{ \"message\": \"{{msg.messageText}}\", \"user_id\": \"botium\" }",
      "SIMPLEREST_RESPONSE_JSONPATH": "$.reply",
      "SIMPLEREST_INIT_TEXT": "reset_conversation"
    },
  }
}

Let's say for a simple .proto:
// botiumgrpc.proto
syntax = "proto3";

package botiumgrpcbot;

// Interface for brain.
service BotiumGrpc {
  // Gets reply from one message
  rpc GetReply(GetReplyRequest) returns (GetReplyResponse) {}
}

// Message request.
message GetReplyRequest {
  string text = 1;
  string user_name = 2;
}

// Message response.
message GetReplyResponse {
  string text = 1;
}

I would say an analogue botium.json would be:
{
  "botium": {
    "Capabilities": {
      "PROJECTNAME": "grpc-bot",
      "CONTAINERMODE": "simplegrpc",
      "SCRIPTING_ENABLE_MEMORY": true,
      "GRPC_URL": "localhost:50051",
      "GRPC_PROTO": "botiumgrpc.proto",
      "GRPC_REQUEST_METHOD": "GetReply",
      "GRPC_REQUEST_MESSAGE": "{\"text\": \"{{msg.messageText}}\", \"user_name\": \"botium\"}",
      "GRPC_RESPONSE_FIELD": "text",
      "GRPC_INIT_TEXT": "reset_conversation"
    },
  }
}

If you create a repo for that as a separate connector as in https://github.com/codeforequity-at/botium-connector-websocket I can also try to contribute with PRs (but I struggle a bit with Javascript).

@codeforequity-at please create a project then we can try some collaboration on this if you may.

https://github.com/codeforequity-at/botium-connector-grpc
You should have received an invitation

@codeforequity-at nice, invitation accepted! How should we proceed? Do you want a call to discuss or already have a plan?

not really having a plan. if you could build some minimum GRPC server and client code I can copy&paste it into the Botium connector code - but you can try yourself to do this as well. The boilerplate is there, and there are some places in the src/connector.js where the actual GRPC request has to be done ...

@staticdev The 1st version of grpc connector is ready.
You can try it out by running the samples/echoserver (grpc server), and then start the samples/echo to run a very simple test.

in root directory:
npm install

samples/echoserver:
npm install npm start

samples/echo:
npm install npm test

Please take a look and don't hesitate to ask. (The readme file will be written soon as well)

@szabi84 sorry for the long delay, I just relocated continent and I will try ASAP. Thanks a lot.

authors: codeforequity-at staticdev szabi84

repo: codeforequity-at/botium-core


text:
New Events are Missing service and service_id

@ramonaspence per the conversation on the HG https://github.com/codeforgreenville/hackgreenville-com/issues/113
I see that the Tech After Five - Charlotte (In Person) and the latest What's Brewing Greenville? event, which is a new org and new event as of last night, are the only two missing the service and service_id values in the events API.
I suspect this is an easy fix, but since the Tech After Five - Charlotte (In Person) was part of the most recent https://github.com/codeforgreenville/upstate_tech_cal_service/pull/54 I wanted to loop you in.

This may be a simple as the  if / elif logic is only defining sevice_id and service for online Eventbrite events.
https://github.com/codeforgreenville/upstate_tech_cal_service/blob/master/update_functions.py#L182
Also, the only difference between that if and elif block is the value set for the venue, service, and service_id
Perhaps we can refactor that code to be less repetitive and only use the if / elseif for setting the differences.
event_dict = {
    'event_name': event.get('name').get('text'),
    'group_name': group_name,
    'url': event.get('url'),
    'time': event.get('start').get("utc"),
    'tags': tags,
    'rsvp_count': None,
    'created_at': event.get('created'),
    'description': event.get('description').get('text'),
    'uuid': unique_id,
    'nid': nid,
    'data_as_of': datetime.datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%SZ'),
    'status': normalize_eventbrite_status_codes(event.get('status'))
}

if type(event.get('venue_id')) == str:

# set the                    'venue': venues[event.get('venue_id')],

}
elif event.get('venue_id') == None: # if event venue is None, it is online/virtual

# set the            'venue': event.get('venue_id'),
}

Here's a git diff of a patch I did on the production events API and it seems to fix the issue.
The question is if there's any reason the service_id values would differ between in-person and online Eventbrite events, which I don't think would be the case since we're using the event's unique ID value.  That value is in the Eventbrite URL, so it's not going to change once the event is created
https://www.eventbrite.com/e/631-tech-after-five-charlotte-in-person-tickets-327548736087
and
service_id | "327548736087"
service | "eventbrite"

diff --git a/update_functions.py b/update_functions.py
index 9beb240..96ab057 100644
--- a/update_functions.py
+++ b/update_functions.py
@@ -224,7 +224,9 @@ def format_eventbrite_events(events_list, venues_list, group_list):
                     'uuid': unique_id,
                     'nid': nid,
                     'data_as_of': datetime.datetime.utcnow().strftime('%Y-%m-%dT%H:%M:%SZ'),
-                    'status': normalize_eventbrite_status_codes(event.get('status'))
+                    'status': normalize_eventbrite_status_codes(event.get('status')),
+                    'service_id': event.get('id'),
+                    'service': 'eventbrite'
                 }
             elif event.get('venue_id') == None: # if event venue is None, it is online/virtual
                 event_dict = {

I see that the format_eventbrite_events function is just missing the service_id and service fields for in-person events. You're right that it needs to be refactored. It'll be easier for me to see what's going on in the function once it's refactored too.
I'll jump on it this week :)

@allella
While I'm refactoring this so it's not repeating itself, I thought I would go ahead and do the same for the format_ld_json function as well. As of right now, it's using a redundant if/else block just like the format_eventbrite_events function.
However, I named this branch "fix/refactor-eventbrite-func" which is more specific. But if I remember correctly (and I'll confirm this), I believe uncommitted changes can be carried over to a new branch, so that may be a non-issue. Either way, I wanted to run this by you before I test and make a PR.

Sure, feel free to clean things up. I'd suggest more, smaller branches and PRs since I'm the only one reviewing these and I'm not a Python guru.
You can also use git stash save if you have uncommitted changes and want to save them and then start a branch and git stash apply the changes to the new branch. Here are notes I have on git stash.

Oh thank you! That's very helpful.
Alright, I'll finish this up tonight or tomorrow!

Alright, just one more question for this. When an event is online, most of the location data is null. Do the null fields serve a purpose or would it be better for the online events to read location: "Online Event"?
In the json+ld output, the location for an online event from meetup for reference:
"location": {
                    "@type": "Place",
                    "name": "Online event",
                    "address": {
                        "streetAddress": null,
                        "addressLocality": null,
                        "addressRegion": null,
                        "addressCountry": "",
                        "postalCode": null
                    },
                    "geo": {
                        "@type": "GeoCoordinates",
                        "latitude": null,
                        "longitude": null
                    }
                }

And eventbrite:
"location": {
                    "@type": "Place",
                    "name": null,
                    "address": {
                        "streetAddress": null,
                        "addressLocality": null,
                        "addressRegion": null,
                        "addressCountry": null,
                        "postalCode": null
                    },
                    "geo": {
                        "@type": "GeoCoordinates",
                        "latitude": null,
                        "longitude": null
                    }
                }

Since I'm not as experienced, I don't want to assume that the null fields aren't doing anything for search engines or other websites and apps that consume the API.

That's a fair question. Since we're using Schema.org markup, it's probably something that's been though through and answered somewhere on the internet.
Whether you'd remove the place, or use some other schema entity is probably the question.
Just quickly searching I see a "pending" type of https://schema.org/VirtualLocation
and  "schema place vs virtuallocation" has this from Google
https://developers.google.com/search/docs/advanced/structured-data/event

Actually, that Google link may address this perfectly. Plus, it may address the case when an event is both physical and online.
https://developers.google.com/search/docs/advanced/structured-data/event

Okay, neat! That seems like the direction we should be moving. However, since this is pending, perhaps it's best for us to bookmark this until it's implemented. I'm reading Issue 1842 where a couple of people have specified that Google doesn't recognize virtualLocation as a valid location yet.
I'm trying to verify that that is no longer true. Schema.org lists Issue 1842 as pending as of v11.0, but I don't see that it's been implemented yet, as of v14.0. I'm making a comment on the issue to find out where they are in that process.
Since, as of right now, we only have the Meetup events listing "Online Event" and it's under "name", I don't think that implementing virtualLocation will hinder search engines' capacity to use the data from our API. At the same time, I don't know how it might affect the other services that consume our API. If we implement this now, and have to change it later when Schema.org might implement a different name, it may become a hassle for those who are consuming the events API. Then again, it might be worthwhile hassle for everyone involved.
Whew, sorry to be a bit long-winded on the topic. What do you think, @allella? Should we go ahead and start using virtualLocation for online events like the examples from Google?

Regarding the question about virtuallocation.
The Google Guidance is here:
https://developers.google.com/search/docs/advanced/structured-data/event
The schema is current with the Google implementaton.
Regards.

@ramonaspence if you're not already too far down this road, then perhaps submit a PR for the initial issue, and then we can create a new Issue, referencing the conversation here, and a new PR for that JSON+LD work.
Thanks

Okay, I'll do that. And as always, thank you for the guidance!

@ramonaspence are you going to create a PR with the service and service_id changes I posted above?
I can do it, but I also don't want to complicate your branches and make you have to merge if you want to deal with it.

@allella Yes! Sorry, I didn't mean to drop off the map.
I'm making separate PR's for adding the service and service_id and for refactoring the function.

authors: WeaverStever allella ramonaspence


text:
Start GeoLocation only on iPhone and Android with GMS enabled

This version only remove the dialog "GMS unavailable" from the onboarding stage due to the lack of GMS on Huawei phone
This PR is similar to https://github.com/codeforpublic/SQUID/pull/56 but uses hasGMS detection function from react-native-device-info instead

It works.
I Tested it On Huawei Y6p (HMS).

It works.
I Tested it On Huawei Y6p (HMS).

Podfile.lock should be updated too.

Podfile.lock should be updated too.

authors: Guutong PaperMonster n3tr

repo: codeforpublic/SQUID


text:
札幌駅前通地下歩行空間（チ･カ･ホ）の人流データ表示

[ ] バグ / Bug
[x] 改善提案 / Suggestions for improvement

起こっている問題 / Occurred problem

混雑度のデータがほしい
札幌駅前通地下歩行空間（チ･カ･ホ）の人流データが使えるかもしれない

スクリーンショット / Screenshot

期待する見せ方・挙動 / Expected behavior

@HIGASHI Kenjiro　さん作成のTableauデータを活用？
埋め込みもできそう

<object class='tableauViz'  style='display:none;'><param name='host_url' value='https%3A%2F%2Fpublic.tableau.com%2F' /> <param name='embed_code_version' value='3' /> <param name='path' value='shared/NKS5D6Q7Q' /> <param name='toolbar' value='yes' /><param name='static_image' value='https://public.tableau.com/static/images/NK/NKS5D6Q7Q/1.png' /> <param name='animate_transition' value='yes' /><param name='display_static_image' value='yes' /><param name='display_spinner' value='yes' /><param name='display_overlay' value='yes' /><param name='display_count' value='yes' /></object>

v0ではまずリンクのみ？
埋め込みはV1で検討？

埋め込むとこんな感じってことでサンプルページです。
https://kken78.github.io

・前年同月比で表現
・緊急事態宣言(2/28）以降の時差出勤状況が分かるかどうか（3月分が出るのは4/10？）

札幌市ICT小笠原さんより
https://codeforsapporo.slack.com/archives/CUKTK3R4K/p1583580461145600

N.Ogasawara  20:27
3月のチカホ人流データ提供時期についてGitHubのコメ見ました。
通常は月初2〜3日でUPするように対応していますので、4/10まではかからないです。

・前年同月比で表現
・緊急事態宣言(2/28）以降の時差出勤状況が分かるかどうか（3月分が出るのは4/10？）

たぶん、こういうことを知りたいのかな？
というあたりで、とりあえず追加作成しました。
【確認事項】
・週単位で通行量平均をとり、時間帯別でどの程度減少しているか（測定ポイントごと）
　→　年明けから減っているんですが、やはり２月最終週の落ち込み大きいね
　　　（時間帯や測定ポイントで違いがありますので、そのあたりも着目）
・時差出勤状況が分かるか？
　→　減少が顕著になった２月分で測定ポイントごとの平均通行量を折れ線グラフにしました
　　　測定ポイントごとに違いが出てます
　　　J1（札幌駅）はあきらかに８時台の集中がなくなっていますが、
　　　他のポイントはあまりはっきりしません
以上まとめると、
【可視化で分かること】
・J1ポイントは、たしかに人数減少とともに時差出勤の影響で分散効果があるように見える
・他のポイントは動線パターンが違う（移動目的が違うとか？）
https://public.tableau.com/views/17905/sheet0_1?:display_count=y&publish=yes&:origin=viz_share_link

本日朝から少々問題が発生し、チカホ人流データの提供を一時的に停止することとなりました。
近日中には今後の対応方針が決まる予定ですが、可視化いただいたグラフの公開は少々お待ちいただきたくお願いいたします。
詳細はこちら → [https://ckan.pf-sapporo.jp/dataset/chikaho-jinryu]

authors: Nar-Ogasawara Yfuruchin kken78


text:
EntryPoint#connection allows you to set faraday middleware after setting the adapter

When passing a block to client.connection without the default: false option, adding any middleware will cause a warning:
api = Hyperclient.new('https://grape-with-roar.herokuapp.com/api') do |client|
  client.connection do |conn|
    conn.request :retry, max: 10, interval: 1.0, interval_randomness: 0.5
  end
end

api.splines

#=> WARNING: Unexpected middleware set after the adapter. This won't be supported from Faraday 1.0.

This warning was introduced in https://github.com/lostisland/faraday/pull/685. We are seeing it here because the default_faraday_block, which adds the adapter, always runs before the custom block.

As a workaround I am specifying default: false and then copying in the default_faraday_block into my custom block so I can put things in the correct order:
api = Hyperclient.new('https://grape-with-roar.herokuapp.com/api') do |client|
  client.connection(default: false) do |conn|
    conn.use Faraday::Response::RaiseError
    conn.use FaradayMiddleware::FollowRedirects
    conn.request :hal_json
    conn.response :hal_json, content_type: /\bjson$/
    conn.request :retry, max: 10, interval: 1.0, interval_randomness: 0.5
    conn.adapter :net_http
    conn.options.params_encoder = Faraday::FlatParamsEncoder
  end
end

api.splines

Faraday Version: 0.15.4
Hyperclient Version: 0.9.0

Care to contribute this to README in some capacity?

It looks like https://github.com/codegram/hyperclient/pull/149 might solve this

Yep, and in #163 I reworked this test.

This is fixed moving forward with Faraday 1+ I believe. Via https://travis-ci.org/github/codegram/hyperclient/builds/687209666:

Faraday 0.9, no warning
Faraday 0.17, WARNING: Unexpected middleware set after the adapter. This won't be supported from Faraday 1.0.
Faraday 1.0.1, no warning

authors: composerinteralia dblock

repo: codegram/hyperclient


text:
fix. filters alias multiple

fix #2831
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPdocs
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

the "Files changed" A bit messy.
I just add
if (is_array($this->config->aliases[$alias]))
{
	$classNames = $this->config->aliases[$alias];
}
else
{
	$classNames = [$this->config->aliases[$alias]];
}

foreach ($classNames as $className)
{
	$class = new $className();
        ......
}


text:
[Rector] Enable AddDefaultValueForUndefinedVariableRector

Description
AddDefaultValueForUndefinedVariableRector now already cover isset variable check, so I think it can be enabled.
Checklist:

[x] Securely signed commits

All green :tada:

Sorry, phpstan-baseline.neon.dist was conflicted.
Can you rebase?

@kenjis rebased 👍

Oooh this is handy!!

authors: MGatner kenjis samsonasik


text:
fix: migrations not using custom DB connection of migration runner

Description
When using custom array connection for migration runner, the migrations ran do not use the custom connection of the runner but instead uses the default shared connection.
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[x] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

I found this bug while working on phpstan-codeigniter in trying to add an extension for properly identifying the property types of Entities.

@paulbalandan The code is indeed something wrong, but I don't understand the significance of passing DB connection to MigrationRunner.

The use case is this: In order to infer the property types of Entities, I thought of using the database tables and the fields. So, I must run migrations in order to get the tables. However, I don't want to mess with the migrations of the end users so I thought of running the migrations in a separate connection. So, regardless of the default connection of the user, I run my own migrations using SQLite3 with a database file. I don't use command('migrate') because it will use the default group. I don't want either to alter the shared Config\Database. So, here comes the array connection. All went smoothly until I am listing all the tables using $db->listTables() to find out that the db file only contains the migrations table. Upon checking, migration files were found but were ran using the shared connection.

Okay, so you want to change the DB connection for testing.
After all, the parameter is for testing.

What should be the group for the added test?
sqlite> select * from migrations;
1|2018-01-24-102301|Tests\Support\MigrationTestMigrations\Database\Migrations\Migration_some_migration|tests|Tests\Support\MigrationTestMigrations|1700128890|1
2|2018-01-24-102302|Tests\Support\MigrationTestMigrations\Database\Migrations\Migration_another_migration|tests|Tests\Support\MigrationTestMigrations|1700128890|1

It appears that resetDataCache() resolves the OCI8 test failures.
See https://github.com/kenjis/CodeIgniter4/pull/138/files#diff-ef065cd29dd2066d73750ad2480b61bf513b037402889ee0c0d9e0fb8b3fca3bR478-R479

Frankly, Migration is complex and difficult to maintain consistency.
When temporarily changing a DB connection for testing purposes, it seems better to regard that the DB connection as the default group because it is easier to manipulate.
However, it seems a bit odd that the DB group name in the migration history is changed to the default group even if you specify the name of the DB group, e.g., db1.
Maybe the following patch is needed.
--- a/system/Database/MigrationRunner.php
+++ b/system/Database/MigrationRunner.php
@@ -141,10 +141,17 @@ class MigrationRunner
         // Default name space is the app namespace
         $this->namespace = APP_NAMESPACE;

-        // get default database group
-        $config      = config(Database::class);
-        $this->group = $config->defaultGroup;
-        unset($config);
+        if (is_string($db)) {
+            $this->group = $db;
+        } else {
+            // We use `defaultGroup` even if $db is ConnectionInterface or config
+            // array. Because this is for testing purposes, and it is easier to
+            // understand and operate as if the migrations were run on the
+            // default group.
+            $config      = config(Database::class);
+            $this->group = $config->defaultGroup;
+            unset($config);
+        }

         // If no db connection passed in, use
         // default database group.
@@ -841,7 +848,7 @@ class MigrationRunner

         /** @var Migration $instance */
         $instance = new $class(Database::forge($this->db));
-        $group    = $instance->getDBGroup() ?? config(Database::class)->defaultGroup;
+        $group    = $instance->getDBGroup() ?? $this->group;

         if (ENVIRONMENT !== 'testing' && $group === 'tests' && $this->groupFilter !== 'tests') {
             // @codeCoverageIgnoreStart
@@ -857,8 +864,6 @@ class MigrationRunner
             return true;
         }

-        $this->setGroup($group);
-
         if (! is_callable([$instance, $direction])) {
             $message = sprintf(lang('Migrations.missingMethod'), $direction);

This is complicated. I'm tracking but let me know if you need me more. Thanks for handling.

authors: MGatner kenjis paulbalandan


text:
In search.html demo, setting the openDialog of closeOnBlur is false, Esc will has some error

I want that persistent search(Alt-F)  is not closed when input loses focus, then I set closeOnBlur false.
Unfortunately, when search input lose focus, new CodeMirror-dialog always  appends to cm by using Alt-F again, then shortcut Esc doesnot work.

That's not a feature that our search dialog supports—you'll have to write your own if you need it.

authors: marijnh soulxy

repo: codemirror/CodeMirror


text:
afterInsert is not called if the data is not array

It seems afterInsert is only called when the data is an array. afterInsert is not called if the data is a javascript object.

Seems to work for me, can you provide me with an example that is failing?

The key point is that, use loadTemplate to load some array into the container, then use loadTemplate to append some object into the same container, and afterInsert didn't get called.
I have attached an example.
jqueryTemplateProblem.zip

My testing shows this is fixed in the latest version from the example you have given. Will close unless you can provide an example that still doesn't work.

authors: codepb elgs hxcan

repo: codepb/jquery-template


text:
Fix response code 411(http error 411 length required)

Done.

Has made changes according to the commentary.

repo: codepine/testrail-api-java-client


text:
Set up Kanban Board

This group is made up of  three people:
@Bigizi
Tasks assigned:
Implement Source, Movie, and Game classes.
@coderacheal
Tasks assigned:
Set up the repo and
Implement Item, Book, and Author classes
@habasm
Tasks assigned:
Implement Movie, Game, and Source classes
Here is the GitHub project link: https://github.com/coderacheal/Catalog

Hi @Team,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Highlights

Descriptive pull request ✔

Required Changes ♻

[ ] Let's be more detailed in tasks. Instead of writing "implement Items, Book and Author classes" as one task you can split it into different achievable tasks following project requirements details.
[ ] Add some missing tasks like Add unit tests for all implemented methods.
[ ] Also, following project requirements some tasks should be assigned to a group.

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100%.
Please, remember to tag me @uwadonat in your question so I can receive the notification.

Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi @Bigizi, @coderacheal, and @habasm ✋
Hello ✋, you have done well reaching this point of the milestone 🤝 . But there are some issues that you still need to work on to go to the next project but you are almost there!
This project will be marked as Invalid and won't affect the number of your weekly reviews

[ ] Kindly ensure the issue forwarded for review contains a link to your GitHub project.

Cheers, and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread and tag me @1GORDON if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think the code review was unfair, request a second opinion using this form.

Hi @Bigizi, @coderacheal and @habasm,
Highlights!

Descriptive Issue ✔

Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Status: Approved ✔
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

authors: 1GORDON Bigizi julie-ify uwadonat

repo: coderacheal/Catalog


text:
Add support for encrypted data bag items v2

Hi,
This is an updated version of my previous pull request (https://github.com/coderanger/pychef/pull/26).
As you suggested, I got rid of m2crypto wrappers over OpenSSL functions by using them directly (via ctypes) - please see chef/aes.py file.
Please let me know if this looks good.
Best regards,
Kamil

Hi Coderanger,
What still needs done on this pull request to get it merged in? I see this is a bit outdated, but I would really like support for encrypted data bags so I can help finish this up.
Benji

Is there anything I can do to help see this merged?

We should try to merge this asap.

authors: benjit89 kamilbednarz scriptwriter tescalada

repo: coderanger/pychef


text:
Set up headless Wordpress CMS

Basic Information
Set up a headless Wordpress CMS backend so that the client can easily edit the content. Useful information here.
Rough task list:

[x] Set up a Wordpress server with Docker (see official Docker image)

[x] Dockerise Next.js frontend (see official Next.js Docker examples here)
[x] Set up nginx reverse proxy

[x] Set up API interface for frontend (e.g. Wordpress REST API or wpgraphql)
[ ] Integrate some frontend pages to query content from Wordpress server

[ ] Write documentation so others understand how to do this too

[ ] Look into ISR (?)
[ ] Before merging: check to see if any more documentation needs to be written / updated, and do so

Other Information

For now I've just set up the default Wordpress REST API and not wpgraphql. We'll see if that needs to be changed when we get around to trying to integrate with the frontend.

Two questiones: 1.Can the client edit the content on the page directly, or do they need to use WordPress?
2.Do we need authentication to ensure that only the client can edit

The motivation for this issue is that the client needs a way to easily edit the content of the website. In a standard Next.js website, there's no easy way to do this (short of editing the code itself and redeploying). The current site is built in WordPress, so the client has some experience updating content through this interface so I think it'd be good to use WordPress as a CMS for our version as well. So, this issue revolves around essentially implementing WordPress into the site such that content is fetched from the WordPress database and can be easily updated via the WordPress admin UI.
So, in short:

They'll edit it through the WordPress admin portal
WordPress already implements an authentication portal for us

I'll try show you in person tomorrow so it'll hopefully be a bit easier to understand. @sersangn has also already been working on this issue a little bit, so I'll make sure you find each other and are able to work together on the issue.

Thanks for the explanation. I've got it now and am ready to proceed. Looking forward to discussing it further tomorrow!😊

A potentially useful plugin that would make it more convenient to edit things in WordPress:
ACF Plugin by WP Engine

authors: berry1104 cjnbennett sersangn

repo: codersforcauses/starick


text:
fixed logo and added styles to about page

fixed #622
IS this UI Ok or you need some changes?

about page is made responsive as well

shift about us to the left align to img

add a button below to the text "see more" then make PR
everything  is perfect +1

kindly share your anyout put after make changeq

@codervivek5 please review these changes

@codervivek5  here is the output

responsive

@krutikajichkar is it possible to make some more changes ?

add this content so that I will look like a about us page...
I will increase the level after finish this task.if you can add it then plz let me know

okay @codervivek5 i will do that

@codervivek5  please have a look , if this ok ,then i will make PR

Ohhhhoooooo @krutikajichkar heading always use upper then paragraph kindly ups heading
so if you are .....................................................this line

@codervivek5

@codervivek5

Actually the whole linkage has broken , on contact page , on about page everywhere , i have raised another issue for that , i am linking this to home for now @codervivek5

Actually the whole linkage has broken , on contact page , on about page everywhere , i have raised another issue for that , i am linking this to home for now @codervivek5

ok but wait fo some time because I think someone is working on it....you can raised a issue and share the link with e in discord or anywhere I will assign it to you tomorrow at evening

you deserve Leve2 instead of Level1 👍

I have linked it to home and contacg


text:
confusing on className="brush"

I could not manage to construct className="brush"
I was wondering what was the proper way to form className="brush" in the example code

please  let me know
regards

Hi there!
Can you show me some more of your code? Right now I don't understand what your are doing exactly.

Thank you first
I am new to React
I try to integrate your brush example to my isomorphic server.
var MyComponent = React.createClass({
..
,.

app.get('/', (request, response) => {
const reactHtml = ReactDOMServer.renderToString();
response.render('index',{reactInitialData: JSON.stringify(contacts),
content: reactHtml });
the graph  is rendered to client but brush is not working.
(my current thought is that the problem has to with rendering style).
how did you inject your css file to client?
thank you

I am not doing isomorphic rendering in my example.
Can you show me how you create the brush?

var MyComponent = React.createClass({
...
    return (

            <LineChart
               data={this.state.data}
               width={400}
               height={400}
               margin={{top: 10, bottom: 50, left: 50, right: 20}}
               xScale={this.state.xScale}
               xAxis={{tickValues: this.state.xScale.ticks(d3.time.day, 2), tickFormat: d3.time.format("%m /%d")}}
            />

            <Brush
               width={400}
               height={50}
               margin={{top: 0, bottom: 30, left: 50, right: 20}}
               xScale={this.state.xScaleBrush}
               extent={[new Date(2015, 2, 10), new Date(2015, 2, 12)]}
               onChange={this._onChange}
               xAxis={{tickValues: this.state.xScaleBrush.ticks(d3.time.day, 2), tickFormat: d3.time.format("%m /%d")}}
            />

    );
},

on the server
app.get('/', (request, response) => {
let componentInstance = ContactsAppFactory({contacts:contacts});
const reactHtml = ReactDOMServer.renderToString();
response.render('index',{reactInitialData: JSON.stringify(contacts),
content: reactHtml });
But, I am not sure whether I am doing right on constructing the brush on the server.
Is brush only client issue?
However, Chrome inspect shows brush structure.
regards

Sorry for correction
const reactHtml = ReactDOMServer.renderToString() will be
const reactHtml = ReactDOMServer.renderToString()

const reactHtml = ReactDOMServer.renderToString(

I never did server side rendering so I cannot say if that is the problem, sorry.
Otherwise this.state.xScaleBrush might be a source of error. Maybe if it's set incorrectly the brush might be rendered incorrectly.
I am sorry I cannot help you more at the moment :(

About the CSS..
I am using the following in the example. Maybe it helps you?
.brush .extent {
  stroke: #000;
  fill-opacity: .125;
  shape-rendering: crispEdges;
}

.brush .background {
  fill: #ddd;
}

thank  you foe all

authors: codesuki orkun


text:
Add character: <5> to font:

Character : 5
Font : amongus

assigned in #907

repo: codewithnick/ascii-art


text:
Write a C program to convert temperature from celsius to kelvin

Description
Write a C program to convert temperature from celsius to kelvin
Input  ( C ) : -40
Output ( K ) : 233.15

Contributed by - @magar51

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/convert-temperature-from-celsius-to-kelvin/ConvertTemperatureFromCelsiusToKelvin.c
Commit the changes
Create a pull request

!assign

repo: codinasion/codinasion


text:
Write a Swift program to find largest number in an array

Description
Write a Swift program to find largest number in an array
Input  : [1, 2, 3, 4, 5]
Output : 5

How to contribute

Save the solution in program/FindLargestNumberInAnArray.swift file
Add FindLargestNumberInAnArray.swift file in find-largest-number-in-an-array folder

!assign

Hey @urlunaticguy,
This issue has been assigned to you for more than 15 days.
We are unassigning you from this issue.
If you are still interested in contributing to this issue, please get auto-assigned on the issue by commenting !assign again.
Thanks for your contributions :)

!assign

repo: codinasion/program


text:
Write a Kotlin program to convert seconds to hours

Description
Write a Kotlin program to convert seconds to hours
1 hour = 60 minutes = 3600 seconds
Input  : 18000
Output : 5

How to contribute

Save the solution in program/ConvertSecondsToHours.kt file
Add ConvertSecondsToHours.kt file in convert-seconds-to-hours folder

!assign


text:
Entrega invalida

Profe! Hice dos entregas el domingo, una a las 23.25 y la otra a las 23.52, me dice que estoy fuera de las fechas, tenía entendido que recibía calificación hasta las 23.59. Creo que a varios compañeros les pasó el mismo caso, ¿hay alguna forma que nos ayude con el tema?

Buenas noches, me pasó igual, realicé la entrega del último taller a las 11:32 PM y el bot respondió a las 12:06 AM informando que la entrega era inválida.

Hola profe @luisvasv-teck @luisvasvv confirmo que me paso la misma situación. Recibí respuesta del Bot hasta las 10;40 pm, después de eso envié un taller a las 11:27 pm y como no había respuesta lo reenvié a las 11:48 pm. El Bot solo volvió a funcionar pasada las 12:00 horas y retorno el mensaje de entrega invalida por la hora.

Buenos días, tengo el mismo issue

Cabe resaltar que, aunque recibí los correos después de las 00:00, los envié antes de las 12

Buenas tardes, ya les dieron respuesta en el canal de Telegram, en avisos y notificaciones, y en calificaciones. Por favor revisen en Telegram para  más detalles.

repo: codingupmyfuture/bootcamplinuxpython


text:
Drop MSVC support in autotools builds. Regenerate with latest autotools.

Original pull request by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Alexey, thanks for the input.
All, how does this sit with pull request #200?
Roy

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

There are no conflicts with pull #17 as I can see.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Alexey,
Presumably your request, which looks very thorough (although some of the diffs are too big to download even when individually selected), will also affect the README files in the Coin root directory. Particularly README.WIN32.  Also, what would happen to the 'build' directories?
Roy

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

What do you mean "what would happen to the 'build' directories"?
I will change README to adjust removing MSVC building from cygwin with autotools.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Sorry Alexey, just having a bit of trouble getting my head around what you are actually proposing (I am currently deep into trying to resolve major issues with my X3D development!).
So, am I correct in thinking that your pull request, judging by the title, is really two elements:

Removing MSVC references from the autotools, and
Regenerating the MSVC specific build directories, using the latest autotools.

And that 1) above is covered below but that 2) would still be outstanding.
Roy

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

My pull request is:

Remove MSVC referencing from autotools
Regenerate scripts and makefiles with latest autotools.
Fixing some cast errors.

This pull request not change any MSVC project files but now it allow to build Coin3D with mingw compilers.
Also upgrading autotools is necessary as used by Coin is very old and not present in most Linux distros.

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Alexey,
I am slowly getting there, and agree with your general comments about the age of Coin, etc. Since I took over the management of Coin I have not yet got into Linux type tools, being a Windows programmer by trade.
I do remember that once, albeit quite a number of years ago, that I did install Cygwin on a Windows system, along with the then current MSVC version, and successfully built Coin using that method. This thought leads me to ask three questions?

Does your pull request, therefore, mean that method is no longer supported?
Can MSVC only be used within the IDE using the appropriate project files?
And that any maintenance, or upgrading, of the said project files can only be done within the IDE environment?

Roy

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

Yeah, I think that using Cygwin with MSVC and autotools method need to be deprecated to simplify build system.
Maybe I'm wrong but I think on present time it not necessary as Visual Studio project files present. But having this method do autotools support more harder.
I think for MSVC is the best to use project files or CMake build system that pulled by #17.
I think all your 3 questions will be answer "YES".

Original comment by Roy Walmsley (Bitbucket: walroy, GitHub: walroy).

Thanks for both your patience and your answers, Alexey.
Butbucket won't show me all your files because, as it says, 'the differences are impressive, but too big!', When I want to view these files directly am I correct in assuming that I can do that here, under your own Coin project?
Roy

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

I think you don't need see such "big" changes because this is autotools regeneration. To see what is changed in autotools you need to see files:
configure.ac, Makefile.am. Basing on this changes autotools regenerate configure and Makefile.in files - this is automatic regenaration.

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

In principle, work like this is valuable. However I am ab bit wary about dropping MSVC support from autotools. It is the way we compiled our Coin binaries and it's also the way the latest official binaries have been compiled by Kongsberg/SIM. In particular, we configured the debugging and release versions of Coin and Quarter respectively like so:
../Coin_Hg/configure --with-msvcrt=mdd --prefix=/cygdrive/c/SARC/Coin --disable-sound --enable-threadsafe --with-alternate=debug --with-suffix=d --with-glu=/cygdrive/c/SARC/GLU_DLLs/GLU_Debug_msvc9/Debug
../Coin_Hg/configure --with-msvcrt=md --prefix=/cygdrive/c/SARC/Coin --disable-sound --enable-threadsafe --disable-debug --disable-symbols --with-glu=/cygdrive/c/SARC/GLU_DLLs/GLU_Release_msvc9/Release
../Quarter_Hg/configure --with-msvcrt=mdd --prefix=/cygdrive/c/SARC/Coin --enable-qt-debug --enable-debug --with-alternate=debug --with-suffix=d
../Quarter_Hg/configure --with-msvcrt=md --prefix=/cygdrive/c/SARC/Coin --disable-debug --disable-symbols
Do we know for sure that other build methods are in production and documented that result in equal binaries? If yes then I am prepared to change our build practice, if no then I think we should either try to keep MSVC support or extend other build methods to support similar options prior to merging this. And not only for Coin, but the other repositories as well.
Thanks,
Bastiaan.

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

I can update all repos if it need. Some times ago I try update autotools without dropping MSVC support but I fail to do it.
Maybe vote about this? Does anybody use this method?

Original comment by Thomas Moeller (Bitbucket: TheHubbit, GitHub: TheHubbit).

Alexey,
Thank you for your contribution to the project! I went through the following source files:

include/Inventor/C/glue/spidermonkey.h
src/base/dict.h
src/fonts/glyph.cpp
src/fonts/glyph2d.cpp
src/fonts/glyph3d.cpp
src/foreignfiles/SoSTLFileKit.cpp
src/glue/freetype.cpp
src/glue/freetype.h
src/threads/thread.cpp
src/threads/thread_win32.icc
src/vrml97/JS_VRMLClasses.cpp

Those changes look good to me. Which platforms did you try to build already? Unfortunately the automatic build of pull requests Christian introduced
seems to be down at the moment.
However, I do share the concerns on removing MSVC from the build system at this time (even though I build my binaries on Windows using the solution file). But the solutions have been added for convenience and not to create releases (I think!). It is furthermore quite hard to change any settings in the project files, for example adding the simage library. To me removing MSVC seems to be premature as the CMake environment is not fully working or verified on all platforms yet. Would it be possible to split the pull request into one for source code changes and another one for updating files with newer automake, but without removing MSVC support?
Thomas

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

That's great if you can update all repos, but what I meant was that alternatives need to be in place for all repos first.
I have no experience with pull request #200. Does it offer similar options? Is there any feedback on use with MSVC? Somebody needs to test that thoroughly. The MSVC project files I am not so fond of because it requires the MS IDE and there is a lot of repeated bloat with every new version.
And yes, I still use MSVC with Cygwin.

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

I'm build under MSYS2 with mingw-w64 GCC-4.9.2.
For example, 64-bit packages:
http://sourceforge.net/projects/msys2/files/REPOS/MINGW/x86_64/mingw-w64-x86_64-simage-hg-r672.8ea3f7b43f03-1-any.pkg.tar.xz/download
http://sourceforge.net/projects/msys2/files/REPOS/MINGW/x86_64/mingw-w64-x86_64-coin3d-hg-r11404.a9b4869bdc0e-1-any.pkg.tar.xz/download
They builded with dependencies from other packages but this is how I do it.
I can split pull request to source changes and to auto tool changes. But as I told I can't yet update autotools having MSVC support.

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

I can try merge locally cmake pull request and test it with mingw-w64 in few days if you want.

Original comment by Bastiaan Veelo (Bitbucket: veelo, GitHub: veelo).

That will be helpful in general, but what we really need is testing with MSVC before it can be dropped from autotools.

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

Sorry I don't use MSVC at all.

Original comment by Alexey Pavlov (Bitbucket: [Alexey Pavlov](https://bitbucket.org/Alexey Pavlov), ).

Will close this pull and create separated pulls.


text:
Add approval storage and approvePermission function

Goal: Unlock a UX for update flows that only requires a single signature to both revoke+approve and can also guarantee atomicity.
Designed an additional way to validate permission approval via storage. This storage gets set through a new approvePermission function which can be called in batch with revokePermission for update flows. It also supports caching a permission that has already received signature approval for improved gas efficiency, enabling apps that may be sponsoring many session key transactions to save on their paymaster costs.
On first pass, the design looks valid and follows suit from storage and signature approval patterns on tokens. However, there is notable concern with allowing permissions to be approved via transaction given it puts more trust in our wallet interface to surface permission details accurately to users. Only enabled permission contracts can be approved, guaranteeing our ability to parse their data, but even still if we have a bug or oversight here, we could accidentally enable a user to approve a malicious permission. Of course we have the cosigner as our safety net, but it is still worth calling out.  I recommend disallowing all calls that emit PermissionApproved and PermissionRevoked on this contract just to discourage apps from this behavior and rely specific screens/RPCs to surface permission-related requests to users.
Note that you can view the call diagrams by turning on the "preview" mode on the markdown files.

Again would really prefer tests in same PR :)

authors: ilikesymmetry wilsoncusack

repo: coinbase/smart-wallet-permissions


text:
Factory function for create object schema with keys passed by args

So, i have this example:
function createSchema(keys: ZodRawShape){

    return z.object(keys);
}

const userSchema = createSchema({

    id: z.number(),
    name: z.string()
});

type UserSchemaType = z.infer<typeof userSchema>;

const u1: UserSchemaType = {
    //Nothing inferred
}

The type inference not works.. any though on this? Thx for help!

Solved by using Generic Type extending ZodRawShape:
function createSchema<T extends ZodRawShape>(keys: T){

    return z.object(keys);
}

const userSchema = createSchema({

    id: z.number(),
    name: z.string()
});

type UserSchemaType = z.infer<typeof userSchema>;

const u1: UserSchemaType = {
    //Inference works
}

repo: colinhacks/zod


text:
fix: failed IP validation test

The tests started failing on the main branch after the following PR(https://github.com/colinhacks/zod/pull/3513) was merged.
This PR removed this case from the test because a6ea::2454:a5ce:94.105.123.75 is not a valid IPv6 address.

This problem was resolved in this commit(https://github.com/colinhacks/zod/commit/f487d74ecd3ae703ef8932462d14d643e31658b3).
Closed 🙏


text:
[Question] Duplicate data?

Since all of the data is mirrored along the diagonal of the resulting image, could there be a better method of displaying the data without repetition?

Maybe! The obvious thing to do would be to show only half of the matrix.
But there's something nice about being able to fix a point on the diagonal and look forward and backward in time for matches by going right/left along the row (or down/right along the column). If it only showed the top-right half of the matrix, then you would look right to go forward in time, and up to look back, which is kind of counterintuitive.
I'm open to ideas though.

I wondered about this as well.
The most common method for visualizing these sort of structures seems to be arc diagrams, which were apparently first popularized by Martin Wattenberg (@wattenberg) in his 2001 project in Shape Of Song.
A similar approach, which can be thought of as a circular wrapping of the arc diagrams, are chord diagrams, which were brilliantly used by Paul Lamere (@plamere) in his 2012 The Infinite Jukebox, using repetitions in the actual sound, rather than the lyrics. (Actually, the experiment was so popular that it even sparked an entire subreddit, and is now hosted at "The Eternal Jukebox".)
I suppose these may be, if not easier to follow, at least prettier than the self-similarity matrices (which btw Wattenberg calls dotplots in his paper).

Also, Wikipedia points out that adjacency matrices of undirected graphs also produce such symmetric square representations. It might be of interest to explore some of the labeled graph representations for such matrices mentioned in the article, such as the Nauru graphs or the Cayley graphs. They certainly seem to be more compact.

Grate links, The adjacency matrices seem to be a good match, for visualization. However the chord diagrams are easier to process at a single glance. With the chord diagrams type representation if the same branching approach is used that was in 'The Eternal Jukebox' by @plamere than I imagine new lyrics could be generated.

I came up with an idea, that would remove the redundancy.
Here is an example for All you need is love
Classic Version

New Version

The way that is done is also the main advantage in my opinion: We have two clear axis. Left to right is the progression on the song. Top to bottom is present to future. So on the top from left to right we  have the actual progression of the song. Then from each pixel down we see the future of that part (word). If it is repeated we see how far in the future it will be repeated.
If several words are repeated in order, they create lines. Diagonal lines only appear when words are repeated directly after each other (thats what creates squares in the classic version).
Also if you scan down an individual column its easier to see the repeating pattern for that individual word (or sentence), which was harder to spot in the classic version.
Now all the important information are either horizontal or vertical, instead of everything being diagonal, which has more resemblance to diagrams or charts most people are used to.
Basically its just a shifted version of the classic visualization, where we drop the 'past'. The disadvantage is it doesn't look so nice (lack of symmetry).

I like the new version. I find it easier to wrap my head around the meaning of the image, but to be honest it still takes some effort to interpret the repeating structures. Probably with the interactive version this would be easier.
On a slightly related note: Have you though about a different coloring mode, with a rainbow progression from start to finish, so that it is easier to identify to what part of a song a given repetition came from? I suspect that may help.

If the chart needs to be interactive as the cursor passes over the image, the color that represents the word could become highlighted everywhere that it occurs. It could incorporate two cursors that move as the song progresses, one vertical and one horizontal. The vertical cursor can move from left too right and the horizontal cursor from top to bottom, both highlighting the current word.

Here is another version, that shows the future of a word, as well as the past.

@essenmitsosse those concepts are really cool! I agree that it is nicer to have repeated sequences appear as horizontal lines rather than diagonals. I do find it a little harder to conceptually relate these visualizations to the overall structure of the strong, but that might be because I've spent so much time with the original orientations.
If anyone feels ambitious enough to make a SongSim fork that uses something like these 'tilted' song matrices, they absolutely have my blessing. (I'd also be open to a P.R. to add this as an optional mode that can be toggled on)

On a slightly related note: Have you though about a different coloring mode, with a rainbow progression from start to finish, so that it is easier to identify to what part of a song a given repetition came from? I suspect that may help.

This is basically how the 'colorful' mode works right now. I use a color map and assign colors to words in order of their first appearance. So words that first appear in the same place will be assigned very similar colors. It's especially clear in a song like this for example.
Or did you have something else in mind?
You could assign a unique color to every point on the main diagonal (i.e. every word of the song), but then how do you decide how to color points off the diagonal?

I don't have a good solution to offer, unfortunately. The colors simply don't look like a sequence most of the time, to my eyes at least. It could be due to the black words breaking the progression.
Maybe a different color scale could help? Say, Viridis or some other color map aimed at displaying continuous data.

authors: colinmorris essenmitsosse japrogramer waldyrious

repo: colinmorris/SongSim


text:
HackMatrix GitHub bot name.

I'm writing a bot to manage issues for hack matrix. This is a great opportunity to give a name to the bot that will run HackMatrix systems by default. Think of this like Ma3a in Tron.
What should HackMatrix' default bot name be?

https://github.com/features/actions

Agent Smith

Wall-E

Mr,robot

repo: collinalexbell/HackMatrix


text:
Make file list horizontal-scrollable on small screen

Existing file list in the HTML report overflows on small screen (see screenshot below):

This patch make the file list scrollable horizontally on small screen:

@amatsuda @colszowka Take a look? 👀  it's a quick fix on CSS styles 🙇 😄

@PragTob Hey, possible to get this small patch merged and release? It'll make the UI much nicer and sparks joy ✨

Fixed in #84

authors: PragTob teohm

repo: colszowka/simplecov-html