Issue with tutorials/modis-ndvi-time-series-animation/index.md

When I change spatial extend of polygon the video is blank

Can you please share the script that produces the error. It will help me identify the problem. Thank you.

I resolved it.
Thanks!
Sharon
From: Justin Braaten<EMAIL_ADDRESS>Sent: Tuesday, October 20, 2020 9:39 AM
To: google/earthengine-community<EMAIL_ADDRESS>Cc: Bywater-Reyes, Sharon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [External]Re: [google/earthengine-community] Issue with tutorials/modis-ndvi-time-series-animation/index.md (#241)
Can you please share the script that produces the error. It will help me identify the problem. Thank you.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgoogle%2Fearthengine-community%2Fissues%2F241%23issuecomment-712942813&data=04|01|sharon.bywaterreyes%40unco.edu|39aa33424d9044d75a6c08d8750e3530|b4dce27cd088445499652b59a23ea171|0|0|637388051177714510|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=UKjdjOaRoaji54jsJT%2B3ucTlopFeMJuQVpslIsFRfpo%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJPPDIANYXPZ3BIJYXDRIWLSLWVHTANCNFSM4SYLXN4A&data=04|01|sharon.bywaterreyes%40unco.edu|39aa33424d9044d75a6c08d8750e3530|b4dce27cd088445499652b59a23ea171|0|0|637388051177724500|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=lQbjUbpoOGEJulSakTh%2Fua9zihJGzLmKZ7cvYOM9vLU%3D&reserved=0.
This message originated from outside UNC. Please use caution when opening attachments or following links. Do not enter your UNC credentials when prompted by external links.

Glad to hear it!

Was trying to do the same thing from the tutorial I only modified the variables var mask and var region. The script is here.

Great, thanks so much
From: Nomit Rawat<EMAIL_ADDRESS>Sent: Wednesday, November 25, 2020 3:28 AM
To: google/earthengine-community<EMAIL_ADDRESS>Cc: Bywater-Reyes, Sharon<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: [External]Re: [google/earthengine-community] Issue with tutorials/modis-ndvi-time-series-animation/index.md (#241)
Was trying to do the same thing from the tutorialhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgoogle%2Fearthengine-community%2Fblob%2Fmaster%2Ftutorials%2Fmodis-ndvi-time-series-animation%2Fmodis-ndvi-time-series-animation.js&data=04|01|sharon.bywaterreyes%40unco.edu|fbb166d9e03547495cc408d8912cc396|b4dce27cd088445499652b59a23ea171|0|0|637418968742614939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2BsSMBOtHXV0rW2jo3vKD%2FEP%2FGynru4CJ6AzTft%2F2wKg%3D&reserved=0 I only modified the variables var mask and var region. The script is herehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fcode.earthengine.google.com%2F%3Faccept_repo%3Dusers%2Fnomitrwt%2Ftest&data=04|01|sharon.bywaterreyes%40unco.edu|fbb166d9e03547495cc408d8912cc396|b4dce27cd088445499652b59a23ea171|0|0|637418968742614939|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=ynUusF8Q2MuYFrL2rfZwoiQZM9y5zWLufclcQgq7l%2Bk%3D&reserved=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgoogle%2Fearthengine-community%2Fissues%2F241%23issuecomment-733617783&data=04|01|sharon.bywaterreyes%40unco.edu|fbb166d9e03547495cc408d8912cc396|b4dce27cd088445499652b59a23ea171|0|0|637418968742624895|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=Bs7UFlF392DLLRHAB2gr4%2BfeC0aS3YqdIUf9OSvEdNo%3D&reserved=0, or unsubscribehttps://nam02.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAJPPDICGGZDV4WNK4PUKTYDSRTL2NANCNFSM4SYLXN4A&data=04|01|sharon.bywaterreyes%40unco.edu|fbb166d9e03547495cc408d8912cc396|b4dce27cd088445499652b59a23ea171|0|0|637418968742624895|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=prtg%2FiB6JhKoHmG0U3cBB2VcBF0EoM3TDQ2gcttUFpk%3D&reserved=0.
This message originated from outside UNC. Please use caution when opening attachments or following links. Do not enter your UNC credentials when prompted by external links.

Hello @bywaters. Please update you script so that the mask variable is defined as such:
var mask = ee.FeatureCollection('USDOS/LSIB_SIMPLE/2017')
  .filter(ee.Filter.eq('country_na', 'India'));

The problem is that the 'USDOS/LSIB_SIMPLE/2017' feature collection property 'wld_rgn' has no entry for 'India'; you must use the 'country_na' property instead.


More verbose logging needed for Faraday Errors

Is your feature request related to a problem? Please describe.
Faraday Errors in logs just show up like Faraday::BadRequestError (the server responded with status 400), which is not very helpful for debugging!
Describe the solution you'd like
Faraday has the ability to spit out more verbose logs I think. Maybe Faraday options should be customisable in config

This is super important for us too 👍
The error message we care about is normally in the response body, which is JSON with a top level error key, eg:
curl -v https://api.openai.com/v1/chat/completions \
  -H "Content-Type: application/json" \
  -H "Authorization: Bearer [REDACTED]" \
  -d '{
    "model": "gpt-3.5-turbo",
    "messages": [
      {
        "role": "systemMMM",
        "content": "You are a helpful assistant."
      },
      {
        "role": "user",
        "content": "Hello!"
      }
    ],
    "stream": true
  }'

[snip]
< HTTP/2 400
< content-type: application/json
[snip]
{
  "error": {
    "message": "'systemMMM' is not one of ['system', 'assistant', 'user', 'function'] - 'messages.0.role'",
    "type": "invalid_request_error",
    "param": null,
    "code": null
  }
}

Thanks @harlantwood , currently you can do
  begin
    OpenAI::Client.new.models.retrieve(id: "text-ada-001")
  rescue Faraday::Error => e
    raise "Got a Faraday error: #{e}"
  end

does that help in your usecase?
Alternatively, here's PR for adding a Logger (plus any other middleware): https://github.com/alexrudall/ruby-openai/pull/373

Hmm for our usecase I am less interested in logging errors, more interested in acting on errors in specific cases, where we can tell something useful to the user, and in other cases, re-throwing the exception to be handled by our error reporting service.
So something like this should work for us:
begin
  # openai call….
rescue Faraday::BadRequestError => e
  response_body = e.response[:body]
  # if response body is “exceeded context length” then let the user know
  # else Sentry.handle_exception(e, extra: { response _body: })
end

I wonder if there is a way to put this error handling into the middleware.. but this should work!

Hmm maybe the library could wrap these Faraday errors, so the user could do:
begin
  # openai call….
rescue OpenAI::Error => e
  # e.message => “exceeded context length”   # the message body from OpenAI, ie the useful information
  # e.cause => the original Faraday::Error  # so that we don't loose access to the internal error, for advanced cases
end

This seems like it would address both the logging use case and the error service usecase, by putting the relevant details up front in the message.

Docs on Exception#cause: https://ruby-doc.org/3.2.2/Exception.html#method-i-cause


test-branch/execute-job/comment-from-wrong-user-on-pr/10b16d8c-8d71-4e93-8811-3932d0b60de8

PR for testing 1474f6a72d9481cf6b176fb93a74f5a25927bf4d

/canonical/self-hosted-runners/run-workflows 1474f6a72d9481cf6b176fb93a74f5a25927bf4d


Update to 1.22.4

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Generation error on valid RAML file

I'm trying to generate the doc for this RAML: https://api.unbounce.com/raml/v0.4/api.raml
Here is the error I get:

array(1) {
  ["schema"]=>
  string(3008) "{"$schema":"http:\/\/json-schema.org\/draft-04\/schema#","description":"Collection of account entities.","type":"object","additionalProperties":false,"required":["accounts","metadata"],"properties":{"accounts":{"type":"array","minItems":0,"uniqueItems":true,"items":{"$schema":"http:\/\/json-schema.org\/draft-04\/schema#","description":"TODO describe","type":"object","additionalProperties":false,"required":["created_at","id","metadata","name","options","state"],"properties":{"created_at":{"title":"Generic Date","description":"RFC 5322, section 3.4.1, compliant date, which means formatted with: yyyy-MM-dd'T'HH:mm:ss.SSS'Z'","type":"string","format":"date-time","id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/common\/definitions_v2.0.json#\/date"},"id":{"type":"string"},"metadata":{"type":"object","allOf":[{"title":"Entity Metadata","type":"object","required":["location","documentation"],"properties":{"location":{"type":"string","format":"uri"},"documentation":{"type":"string","format":"uri"},"related":{"type":"object"}},"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/common\/entities_v2.0.json#\/entity_metadata"},{"type":"object","required":["related"],"properties":{"related":{"type":"object","additionalProperties":false,"required":["pages","accounts","sub_accounts"],"properties":{"pages":{"type":"string","format":"uri"},"accounts":{"type":"string","format":"uri"},"sub_accounts":{"type":"string","format":"uri"}}}}}],"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/metadata.json#\/account_metadata"},"name":{"type":"string"},"options":{"$schema":"http:\/\/json-schema.org\/draft-04\/schema#","type":"object","additionalProperties":false,"properties":{"api_keys_enabled":{"type":"boolean"}},"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/account_options.json"},"state":{"type":"string","enum":["active","suspended"]}},"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/account.json"}},"metadata":{"title":"Collection Metadata","type":"object","allOf":[{"title":"Entity Metadata","type":"object","required":["location","documentation"],"properties":{"location":{"type":"string","format":"uri"},"documentation":{"type":"string","format":"uri"},"related":{"type":"object"}},"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/common\/entities_v2.0.json#\/entity_metadata"},{"type":"object","required":["count"],"properties":{"count":{"type":"integer"}}}],"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/common\/entities_v2.0.json#\/collection_metadata"}},"id":"file:\/\/\/home\/ddossot\/dev\/projects\/ub-public-api-specs\/resources\/webroot\/raml\/v0.4\/schema\/accounts.json"}"
}
The key application/vnd.unbounce.api.v0.4+json does not exist.
phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php 333
#0 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(336): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#1 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(336): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#2 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(336): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#3 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(336): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#4 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(273): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#5 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(211): Raml\Parser->parseRamlData(Array, '/home/ddossot/d...', true)
#6 phar:///home/ddossot/dev/tools/raml2html.phar/src/Generator.php(55): Raml\Parser->parse('/home/ddossot/d...', true)
#7 phar:///home/ddossot/dev/tools/raml2html.phar/src/Command/GenerateCommand.php(62): Cnam\Generator->parse('/home/ddossot/d...')
#8 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Command/Command.php(257): Cnam\Command\GenerateCommand->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#9 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(874): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#10 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(195): Symfony\Component\Console\Application->doRunCommand(Object(Cnam\Command\GenerateCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#11 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(126): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#12 phar:///home/ddossot/dev/tools/raml2html.phar/index.php(15): Symfony\Component\Console\Application->run()
#13 /home/ddossot/dev/tools/raml2html.phar(10): include('phar:///home/dd...')
#14 {main}

Hello, @ddossot
Your a use mediaType application/vnd.unbounce.api.v0.4+json that is not standard and does not pass validation.
This bug confirmed
If your change media type to application/json then generator will be correct working

Thanks for confirming the bug! Looking forward to a fix, I really like the design of the doc this tool generates :smile:

Hello, @ddossot
We have released version 0.0.4 can be downloaded from the link,
http://raml2html.leanlabs.io/raml2html.phar

Thanks!
Here is the error I receive now:

Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
PHP Notice:  Undefined index: application/vnd.unbounce.api.v0.4+json in phar:///home/ddossot/dev/tools/raml2html-0.0.4.phar/vendor/cnam/php-raml-parser/src/Parser.php on line 331
PHP Fatal error:  Call to a member function setSourceUri() on a non-object in phar:///home/ddossot/dev/tools/raml2html-0.0.4.phar/vendor/cnam/php-raml-parser/src/Parser.php on line 332

I am sorry, please try again.
http://raml2html.leanlabs.io/raml2html.phar

Thank you :+1: I get a different error now:

Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: key: application/vnd.unbounce.api.v0.4+json not found from accept types application/json,text/json,application/(.*?)+json
Warning: Not valid json or file not found.
JSON:'page_group'
Invalid JSON.
phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Schema/Parser/JsonSchemaParser.php 46
#0 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(340): Raml\Schema\Parser\JsonSchemaParser->createSchemaDefinition('page_group', '/home/ddossot/d...')
#1 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(342): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#2 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(342): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#3 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(342): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#4 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(342): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#5 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(342): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#6 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(273): Raml\Parser->recurseAndParseSchemas(Array, '/home/ddossot/d...')
#7 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/cnam/php-raml-parser/src/Parser.php(211): Raml\Parser->parseRamlData(Array, '/home/ddossot/d...', true)
#8 phar:///home/ddossot/dev/tools/raml2html.phar/src/Generator.php(55): Raml\Parser->parse('/home/ddossot/d...', true)
#9 phar:///home/ddossot/dev/tools/raml2html.phar/src/Command/GenerateCommand.php(62): Cnam\Generator->parse('/home/ddossot/d...')
#10 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Command/Command.php(257): Cnam\Command\GenerateCommand->execute(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#11 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(874): Symfony\Component\Console\Command\Command->run(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#12 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(195): Symfony\Component\Console\Application->doRunCommand(Object(Cnam\Command\GenerateCommand), Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#13 phar:///home/ddossot/dev/tools/raml2html.phar/vendor/symfony/console/Symfony/Component/Console/Application.php(126): Symfony\Component\Console\Application->doRun(Object(Symfony\Component\Console\Input\ArgvInput), Object(Symfony\Component\Console\Output\ConsoleOutput))
#14 phar:///home/ddossot/dev/tools/raml2html.phar/index.php(15): Symfony\Component\Console\Application->run()
#15 /home/ddossot/dev/tools/raml2html.phar(10): include('phar:///home/dd...')
#16 {main}

You can check the API Console for my RAML file here: https://api.unbounce.com/console.html It may help you debug the issue?


import from package doesn't work under Python 3

The import statement inside the package's __ init __ .py ( from xbmcjson import * ) doesn't work under Python 3.
In Python3 the relative path must be explicitly stated.
Before the correction, a test app trying to use the packages with from xbmcjson import XBMC would trigger an ImportError exception ( cannot import name 'XBMC' ).
If the app tried to import the whole package with from xbmcjson import *, the import would seem to succeed but the first use of the XBMC symbol would trigger a NameError exception ( name 'XBMC' is not defined ).
I added a tiny dot in front of xbmcjson in __ init __ .py : from .xbmcjson import * to make it more explicit and that seems to fix both problems.
It has been tested on Linux under Python 3.4.2 and Python 2.7.8 and it seems to work under both versions.

Hi.
Thanks for the PR.
Will test it and merge it if everything is ok.


ReceiveAndKeep value in enum ReceiveType is just a peek operation

I was first very confused with the name "ReceiveAndKeep" because nothing like that exists when we are talking about service bus queues. I took a chance and tried to use the default value at first, against a test service bus queue and it worked flawlessly.
The more I play with it, I believe everyone would benefit from using the correct names / verbs for the action that you are trying to expose in module. It makes it easier for people to read about different concepts and follow along other guides, even though they are using this module.
As a mitigation, I suggest that we simply have 2 enums, that does the same. We are keeping the original "ReceiveAndKeep", so we don't introduce any breaking changes. And we add the "PeekOnly" option as well, for people that know about the concepts and knows what they are after.

Hey @Splaxi - thanks for your contribution!  I totally agree with this.  Let me fix the problem with the build and I will review the PR.

PR merged.


[ADH-4303] Add new SSM REST API specification

Added module with SSM web server OpenAPI specification

Superseded by #26


feat: fix peer renewal, change Filter keep alive

Problem
Peer renewal doesn't work as intended.
Current behavior - it renews a peer to a one with which light node has a connection already.
Expected behavior - renews to a peer which we don't have a connection.
This PR also adds decrease for Filter keep alive queries to 30 seconds as was proposed in one of the threads in https://github.com/waku-org/specs/pull/18, and some code improvements.
Solution
Fix the behavior and add some improvements.
Notes
Tested in the dogfooding app - https://github.com/waku-org/lab.waku.org/pull/68

I'm curious how the peer renewal failure was observed/debugged? Any failing test cases? Should we add new tests that would reproduce it?

Peer renewal was working by dropping connection and then taking a peer from those with whom we have a connection. What we actually need to do is to get a peer to which we are not connected yet. Otherwise we need to always have connections to peers that we are not using in order to be able to use them as a replacement.
As for repro - you can do this by running dogfooding app now and seeing that at dome point there are no peers to be used for light push


citrix-receiver Invalid  Checksum

Checking sha256 of url in repo differs from sha256 variable.
curl -o CitrixReceiverWeb.dmg \
http://downloadplugins.citrix.com.edgesuite.net/Mac/CitrixReceiverWeb.dmg \
&& shasum -a 256 CitrixReceiverWeb.dmg

returns sha256:
5138096b25c97cb37d037d27e6151fc96e1af9da5184347f2812b522fe35e8a7
Which differs from citrix-receiver.rb sha256 hash var:  86d376b1b3fdf3a5b53c1c9129b2cbf11087e0c190514a3da24c4f69aa8fa150
EDIT
Should have mentioned I'm on mac if you try running that command.  The SHA returned from shasum matches sha from brew cask install
Michaels-MBP:Downloads mcnichol$ brew cask install citrix-receiver
==> Downloading http://downloadplugins.citrix.com.edgesuite.net/Mac/CitrixReceiverWeb.dmg
Already downloaded: /Library/Caches/Homebrew/citrix-receiver-12.1.dmg
==> Verifying checksum for Cask citrix-receiver
==> Note: running "brew update" may fix sha256 checksum errors
Error: sha256 mismatch
Expected: 86d376b1b3fdf3a5b53c1c9129b2cbf11087e0c190514a3da24c4f69aa8fa150
Actual: 5138096b25c97cb37d037d27e6151fc96e1af9da5184347f2812b522fe35e8a7
File: /Library/Caches/Homebrew/citrix-receiver-12.1.dmg
To retry an incomplete download, remove the file above.

Version updated. Fixed in https://github.com/caskroom/homebrew-cask/pull/19337


⚠ Bofrost.fr has degraded performance

In f520602, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1045 ms

Resolved: Bofrost.fr performance has improved in 97390e8.


installing node-red from repo: MODULE_NOT_FOUND

When I install node-red-contrib-sm-ind from a local repo, it installs, but does not show up in the node-red palette. Under user setting, it is listed along with all of the nodes, but it has a "MODULE_NOT_FOUND" warning. None of the nodes are enabled.
If I remove it, and re-install it directly from node-red, then it works properly.
The only two differences I have identified are

the node_modules folder node-red-contrib-sm-ind is a symbolic link when installed locally.
the dependency listing in the package.json.

local install: "node-red-contrib-sm-ind": "file:../repos/megaind-rpi/node-red-contrib-sm-ind",
Install from node-red: "node-red-contrib-sm-ind": "~1.0.4",

The version shown is 1.04. in both cases.
I am installing on a RaspberryPi
Not the end of the world, but frustrating.

Thank you for the bug report.
We will look into it and if we can not solve it we keep only the second installing method.
One small question, have you change your current folder to the node-Red (.node-red) install folder before installing the node, as in the instructions?
Can also provide node-red and npm versions on your Raspberry?

Yes, I run the following command from within ~/.node-red
npm install ~/repos/megaind-rpi/node-red-contrib-sm-ind
It appears to install correctly, I see everything in the node modules folder. Then I stop/start node-red. None of the sequent nodes are available in the palette, and I see the "MODULE_NOT_FOUND" in the palette settings.
Very strange, I can't tell why this method does not work, but installing from within node-red works fine.
node-red version: 3.1.0
npm -v: 9.6.7

Had the same issue. Thanks for the work around @PattonDynamics


Support RGBP encoding

This PR adds support for a new HDR texture encoding (similar to RGBM).
Instead of storing pixel intensity in alpha, RGBP stores roughly 1.0 - max(1.0, pixel_intensity).
This has some desirable properties, especially when stored as PNG:

should solve most banding issues on safari browsers which still premultiply PNG alpha (since only bright areas now lose bits)
PNG file size is reduced by up to 20% in typical environments due to simpler alpha data in LDR areas
resulting images are easier to manage, since much more colour data is left intact
slightly higher quality HDR areas compared to RGBM
pixel data is filterable (like RGBM, but unlike RGBE)

For example, given this HDR image:

RGBM colour and alpha (3.6MB):

RGBP colour and alpha (3.1MB):

I created an RGBP version of a particularly problematic lightmap originally taken from https://playcanvas.com/project/345310/overview/orange-room.
The result running on iphone 7:

And comparison of the two lightmaps:

encoding
size (gzipped)
lightmap

RGBM
5.8MB

RGBP
4MB

Wow. I'd say that a massive win! 😮

This looks really awesome, simple solution that solves the problem, and likely even faster to encode to!


slot-scope destructuring with alias does not seem to work

Version
2.5.16
Reproduction link
https://jsfiddle.net/50wL7mdz/261895/
Steps to reproduce
1.) We tried to v-bind the slot to an object
2.) On consuming the slot we used slot-scope with a deconstructed and aliased variable.
3.) It says "TypeError: Cannot read property 'title' of undefined"
What is expected?
It should deconstruct the object correctly as it's okay without the alias
What is actually happening?
the alias seems to be not working, returning an "undefined" value

You mixed up the order. it's oldname: newname, not newname: oldname.
Working fiddle: https://jsfiddle.net/50wL7mdz/261938/

my bad. Thanks


include a simple loading indicator that is displayed as long as initialPageLoad is not finished yet

fixes #1624

@quasarchimaere can't merge - there are conflicts

I don't see any change in the owner webapp whatsoever.


Update setup-steps-for-extensible-key-management-using-the-azure-key-…

Suggested doc edits in response to feedback provided from customer on case<PHONE_NUMBER>001033.

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

@Rupp29 - Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
Note that the new links need to point to the live article rather than to the article in GitHub.
#label:"aq-pr-triaged"
#assign:@MicrosoftDocs/public-repo-pr-review-team

The following users don't have valid GitHub IDs, or are not collaborators on this repo: microsoftdocs/public-repo-pr-review-team

#sign-off

#sign-off

The content looks good, but I'm unable to edit the URL to the live URL
https://docs.microsoft.com/en-us/azure/key-vault/general/access-behind-firewall


feat: make $if a type predicate

Currently doesn't look like the $if(cond, ...) narrows the type of cond.
Would be nice if it did, so we could skip the non-null-assertions

You shouldn't use $if for adding optional where expressions. Use normal if statements instead like @igalklebanov showed.

You shouldn't use $if for adding optional where expressions. Use normal if statements instead like @igalklebanov showed.

May I ask when $if should be used then?

@juliusmarminge It was created for conditional selects.

I've tried to make that as clear as possible in the documentation but we probably still need to improve that

My bad for not checking the docs properly here. I ended up doing something like this:

Another (unrelated) quick question when I got you in the thread, what's the "best practice" for breaking out a joined table to it's own object? Is what I'm doing here good/bad?

@juliusmarminge Take a look at the relations recipe.


[BUG][iOS] Could not initialize an instance of the type 'OpenGLES.EAGLContext': the native 'initWithAPI:' method returned nil

Description
SKGLView fails to initialise with the error "Could not initialize an instance of the type 'OpenGLES.EAGLContext': the native 'initWithAPI:' method returned nil"
StackTrace:
NSObject.InitializeHandle (System.IntPtr handle, System.String initSelector) OpenGLES.EAGLContext..ctor (OpenGLES.EAGLRenderingAPI api) <0x100a9c820 + 0x0005b> in <79170d3ca0a34f20a27c5ea33d79161f#cb7c80be70a7684816046182eab111ad>:0 SKGLView.Initialize () SkiaSharp.Views.iOS.SKGLView..ctor () <0x101575dc0 + 0x0001b> in <1dff7dabaada4d38b0d12baee26f046c#cb7c80be70a7684816046182eab111ad>:0
Code
				var view = new SKGLView 				{ 					TranslatesAutoresizingMaskIntoConstraints = false, 					BackgroundColor = UIColor.Clear 				};
Expected Behavior
The GL view works as normal.
Actual Behavior
The above exception
Basic Information

Version with issue:  1.68.0
Last known good version:
IDE:  Visual Studio for Mac
Platform Target Frameworks:

iOS:  13.4.1

Target Devices:

iPad Air (3rd gen)

This is a bit concerning... Does the app run on an older OS? 13.3?
I know Apple was depreciating GL, but I don't thing they were removing it. Especially not in a minor update.

Most affected OS is 13.3.1

Are you able to update to the later versions of SkiaSharp: 1.68.3? I have not seen this happen locally. What happens if you try another device?

Closing due to no response. However, if this issue still persists, feel free to reopen.


Delete orphaned C++ files

Also upgrades one VB project.

Hi @gewarren
Do you have any idea why the first error shows up in the snippets build?

@BillWagner I think it's related to this explanation from Andy: https://github.com/dotnet/docs/pull/34630#issuecomment-1473989525. It's a problem with Snippets 5000.


incomplete kitti_2015_train.txt

Hi! Thank you for your impressive work! But did you notice that kitti_2015_train.txt is corrupted located in mono/utils/filenames/kitti_2015_train.txt? Paper tells there should be 29K rectiﬁed stereo images rather than 19K lines in this file.

pop it up in case you didn't notice that I reopened the issue...

Hi,
we used the KITTI train split available here: https://github.com/mrharicot/monodepth/tree/master/utils/filenames
However, in KITTI should exist a mapping file to filter out such images

Hi,
we used the KITTI train split available here: https://github.com/mrharicot/monodepth/tree/master/utils/filenames
However, in KITTI should exist a mapping file to filter out such images

Understood. Thank you for your time.


Put codestyle check back in after accidental removal

@larrybradley it looks like your package infrastructure PR left out the codestyle check when you moved things to ci_tests.yml. I assume this was accidental, but if it was on purpose let me know.

Wait, the codestyle check is in there, it's just further down. Scrapping this.


Glorfcode (force say on damage/stun/crit)

About the PR

Requires https://github.com/space-wizards/RobustToolbox/pull/4455
forces clients to submit whatever is in their chat box on:

stun
damage of >10 brute or burn
sent from alive->dead/crit

on damage/stun specifically, sends it with a funny suffix like glorf or augh or etc etc etc. taken from tg feel free to suggest more
Why / Balance

very immersive very fun for RP, lets people get out what they were trying to say on crit in a very spooky fashion (for anyone listening), lets you interrupt what people are trying to say by smacking them around. overall it is simply good fun
Technical details

pretty obvious how it works, DamageForceSayEvent gets sent to client which forces them to submit their chatbox lineedit
theres one weird thing here:

AllowNextCritSpeechComponent which functions to give some leeway to clients and allow them to temporarily ignore mobstate checks for a message for a given time, which is calculated based on their ping

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

https://github.com/space-wizards/space-station-14/assets/19853115/bcc8af2b-1083-44ac-99b4-03294080341c
Breaking changes

Changelog

:cl:

add: Being damaged, being stunned, or going into crit will now force you to send whatever message you were typing, if it was in local/whisper/radio. You'll sometimes find yours-AUGH

oop just realized i forgot to implement a cooldown

i dont like the suffix of augh or ock, id prefer it to just have a “-“

you will glorf and you will like it

i dont like the suffix of augh or ock, id prefer it to just have a “-“

die saying ick, it will turn into ick ock
automatically better than -

Extremely minor and not really needed; but I think having slipping not being all caps might add a bit more flavor.

i will consider it

@mirrorcult does this account for narcolepsy

Maybe you already accounted for it, but what if you're typing a command in the chatbox?

it only applies if your active channel is local/whisper/radio

@mirrorcult does this account for narcolepsy

doesnt account for sleep at all rn so i guess ill do that

prepare to glorf in your sleep

we're so fucking b-ACK!

wait so- if i say ;help me i am dyi
does it do [radio] help me i am dyi- AUGH?

yes

epic


[REST API] Performance are lower than Express

Express:

Blaze:

Hono:

Since Blaze is based on Hono, it should have a performance which closest to Hono. The PR #42 are mean to fix this issues but the performance of the REST are still far from what we expected.
After #42 implemented:

Result in the becnhmark:

Update 28/02/2024:
After v2.0.0 release, we got some huge improvement as seen below

Update 04/03/20240:
Results in the benchmark:

Update 22/04/2024 w/ changes on #53 :
[Express]
Statistics        Avg      Stdev        Max
  Reqs/sec     10335.13    1447.90   11739.41
  Latency       12.09ms     2.69ms    92.45ms
  HTTP codes:
    1xx - 0, 2xx - 103441, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     2.84MB/s

[Blaze]
Statistics        Avg      Stdev        Max
  Reqs/sec     26020.97    3352.23   28432.99
  Latency        4.80ms     1.57ms    77.16ms
  HTTP codes:
    1xx - 0, 2xx - 260304, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     5.58MB/s

[Hono]
Statistics        Avg      Stdev        Max
  Reqs/sec     33221.54    4401.20   37051.42
  Latency        3.76ms     1.03ms    56.72ms
  HTTP codes:
    1xx - 0, 2xx - 332295, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     7.13MB/s

Update 25/04/2024 with changes on #54 :
[Express]
Statistics        Avg      Stdev        Max
  Reqs/sec     10642.25    1415.21   11948.79
  Latency       11.74ms     2.50ms    89.29ms
  HTTP codes:
    1xx - 0, 2xx - 106531, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     2.92MB/s

[Blaze]
Statistics        Avg      Stdev        Max
  Reqs/sec     26162.52    3563.90   28526.28
  Latency        4.78ms     1.59ms    77.34ms
  HTTP codes:
    1xx - 0, 2xx - 261732, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     5.61MB/s

[Hono]
Statistics        Avg      Stdev        Max
  Reqs/sec     33822.25    4214.40   36983.55
  Latency        3.69ms     1.02ms    58.09ms
  HTTP codes:
    1xx - 0, 2xx - 338291, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     7.26MB/s

Update 12/09/2024, WIP
[Express]
Statistics        Avg      Stdev        Max
  Reqs/sec     11458.65    1825.28   16791.01
  Latency       10.90ms     3.91ms   131.68ms
  HTTP codes:
    1xx - 0, 2xx - 114650, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     3.15MB/s

[Blaze]
Statistics        Avg      Stdev        Max
  Reqs/sec     40168.13    7241.35   58179.51
  Latency        3.11ms     2.22ms   133.19ms
  HTTP codes:
    1xx - 0, 2xx - 401527, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     8.61MB/s

[Hono]
Statistics        Avg      Stdev        Max
  Reqs/sec     33585.36    5037.15   50599.75
  Latency        3.72ms     2.67ms   146.33ms
  HTTP codes:
    1xx - 0, 2xx - 335813, 3xx - 0, 4xx - 0, 5xx - 0
    others - 0
  Throughput:     7.20MB/s

Need further investigation is this a fluke or not


Feature Request Sign-In with Apple

Is your feature request related to a problem? Please describe.

Firebase "Sign-In with Apple"

Describe the solution you'd like
Something like the google sign-in:
GdxFIRAuth.inst().google().signIn().then...
for example:
GdxFIRAuth.inst().apple().signIn().then...
It would make User Authentication on IOS much easier.

@Trurl101 sure, thanks for a request!
I'd try to bring it to gdx-fireapp next week

sorry for a delay. I was out for a while. I'll try to pick this topic soon.

Hi,
I try to implement Sign-In with Apple but when I change the library version to the latest 3.2.0 Gradle failed to resolve the dependency...
Can you help me with this?

Hello @Daniel-Donev
sure, I need to check, and I'll let you know. There was indeed a problem with shutting down the jcenter() repo. It could be that artefact has not been successfully deployed to maven central.
Pls check if you have mavenCentral() in your repositories block

Hi,
the problem is still here...
this is my repositories block
buildscript {
repositories {
mavenLocal()
maven { url "https://plugins.gradle.org/m2/" }
maven { url "https://oss.sonatype.org/content/repositories/snapshots/" }
jcenter()
google()
maven { url 'https://maven.google.com' }
mavenCentral()
}
dependencies {
classpath 'com.android.tools.build:gradle:3.4.1'
classpath 'com.google.gms:google-services:4.3.4'
classpath 'com.mobidevelop.robovm:robovm-gradle-plugin:2.3.10'
classpath 'com.google.firebase:firebase-crashlytics-gradle:2.3.0'
}
}
allprojects {
apply plugin: "eclipse"
apply plugin: "idea"
version = '1.0'

repositories {
    mavenLocal()
    jcenter()
    google()
    maven { url "https://oss.sonatype.org/content/repositories/snapshots/" }
    maven { url "https://oss.sonatype.org/content/repositories/releases/" }
    mavenCentral()
}

}

Is it possible to download version 3.2.0 with Gradle or the problem is in the maven version?
https://search.maven.org/search?q=g:pl.mk5.gdx-fireapp

@Daniel-Donev I need to check, it's just not deployed into maven-central correctly.
btw, I think you can remove jcenter() from your repositories. https://blog.gradle.org/jcenter-shutdown

when I include iosEntitlementsPList in my robovm.xml like this:
data/Entitlements.plist
Info.plist.xml
I run into this error:
[ERROR] Couldn't start application
org.robovm.libimobiledevice.LibIMobileDeviceException: ApplicationVerificationFailed
at org.robovm.libimobiledevice.util.AppLauncher$1.error(AppLauncher.java:1006)
at org.robovm.libimobiledevice.Callbacks.callInstproxyCallback(Callbacks.java:64)
My Entitlements.plist is like:

com.apple.security.get-task-allow

com.apple.developer.applesignin

Default

can you help me?

@mk-5
Hi, today I'have tried the apple sign in implementation on iPhone 7 with iOS 15 and everything was ok (the sign in popped up asking for password) until after I have entered my pass I get this error:
"Unable to parse the ID Token."
I believe that this is something from the library? What do you think?

@Daniel-Donev can you give me a full stacktrace? or at least some part?

Yes, you have throwable there. You can print the stack trace :)
throwable.printStackTrace();
On Wed, Oct 20, 2021, at 07:46, Daniel-Donev wrote:

Hi @mk-5 https://github.com/mk-5
I didn't have a crash error, just catch this from .fail
GdxFIRAuth.inst().apple().signIn().then(new Consumer<GdxFirebaseUser>() { @Override public void accept(GdxFirebaseUser user) { logged.user = user; } }).fail(new BiConsumer<String, Throwable>() { @Override public void accept(String s, Throwable throwable) { println("firebase error: " + s); // return "firebase error: Unable to parse the ID Token." } });
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mk-5/gdx-fireapp/issues/48#issuecomment-947347712, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHTADFCNQME2CLWSOYVB4OLUHZJTXANCNFSM4YO5G5AA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Pozdrawiam,
Mateusz Kulawik

Hi @mk-5, sorry for the delay I was sick...
the throwable.printStackTrace(); return
java.lang.RuntimeException: Unable to parse the ID Token.
at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:54)
at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:50)
at org.robovm.pods.firebase.auth.FIRAuth$$BlockMarshaler2.invoked(Unknown Source)
at org.robovm.apple.uikit.UIApplication.main(Native Method)
at org.robovm.apple.uikit.UIApplication.main(UIApplication.java:446)
at com.playshiftboy.IOSLauncher.main(IOSLauncher.java:61)
can you help me?

Where do you think is the problem?
Can I track anything else to help you?

@Daniel-Donev mhmm I need to investigate. Unable to parse the ID Token. is nothing comming from our lib.
The error is comming from FIRAuth.auth().signInUsingCredential execution. The execution is here: https://github.com/mk-5/gdx-fireapp/blob/master/gdx-fireapp-ios/src/pl/mk5/gdx/fireapp/ios/auth/AppleAuthProvider.java line 54

Hi @mk-5, I implement all steps from https://github.com/mk-5/gdx-fireapp/wiki/iOS-Guide
and FIRAuth I think is from download required frameworks.
I don't have any problems with Google Sign in.
If I need to update some of the required frameworks or its something else?
Thanks for the support

Hi, @mk-5 sorry to bother you again.
I just wanted to ask did you had the time to look into the problem?

@Daniel-Donev hi hi, no problem. I was out for a short period of time.
mhmmm there is also additional step required in order to make apple sign-in works. It's described here: https://github.com/mk-5/gdx-fireapp/wiki/Apple-Sign-in

Hi, @mk-5 I have implemented all of these steps since the beginning.
and I still have the problem:
java.lang.RuntimeException: Unable to parse the ID Token.
at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:54)
at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:50)
at org.robovm.pods.firebase.auth.FIRAuth$$BlockMarshaler2.invoked(Unknown Source)
at org.robovm.apple.uikit.UIApplication.main(Native Method)
at org.robovm.apple.uikit.UIApplication.main(UIApplication.java:446)
at com.playshiftboy.IOSLauncher.main(IOSLauncher.java:61)
did you successfully try your implementation with apple sign-in?
where else do you think my mistake could be?

@Daniel-Donev yes, I'm able to run this test successfully: https://github.com/mk-5/gdx-fireapp/blob/master/e2e/core/src/pl/mk5/gdx/fireapp/e2e/tests/AuthAppleSignInTest.java

Ok, thank you.
I will check all the steps again.

@Daniel-Donev I just checked, look:

"Unable to parse the ID Token." we need to find what could be the reason for that. Do you have apple sign-in enabled in firebase console?

Yeah, I think so...
These days I will check all of my implementations and will share what I will find.
Thank you!

Hi I still have the same problem :(
I have a configuration on my firebase for ios version and authentication with apple
I configure my App in developer.apple.com to support "sign in with Apple"
I implement all of your steps. Ios version successfully login with "google sign-in" and use firebase.
overall I have this error return:
Domain=FIRAuthErrorDomain Code=17004 "Unable to parse the ID Token." UserInfo={NSLocalizedDescription=Unable to parse the ID Token., FIRAuthErrorUserInfoNameKey=ERROR_INVALID_CREDENTIAL}
but the credential.getEmail() is returning my apple email!
credential.getIdentityToken().toString() return {length = 870, bytes = 0x65794a72 61575169 4f694a6c 57474631 ... 76305771 56344d51 }
nonce  return  bd35fb6eed1d9b4c17c21881f81e5e5390fa3a1ea5fe8f2a066346bcb578d575
and PROVIDER return apple.com
In my Entitlements.plist i remove:
com.apple.security.get-task-allow

because I try to build a deployment version - can this be a problem?
Can you suggest to me what can do?

Mhmm, I will examinate the E2E test configuration, maybe I forgot about something in docs.
Now I’m „out-of-pc”, but I will try to look on it asap ;)
On Sat, Jan 8, 2022, at 13:29, Daniel-Donev wrote:

Hi I still have the same problem :(
I have a configuration on my firebase for ios version and authentication with apple
I configure my App in developer.apple.com to support "sign in with Apple"
I implement all of your steps. Ios version successfully login with "google sign-in" and use firebase.
overall I have this error return:
Domain=FIRAuthErrorDomain Code=17004 "Unable to parse the ID Token." UserInfo={NSLocalizedDescription=Unable to parse the ID Token., FIRAuthErrorUserInfoNameKey=ERROR_INVALID_CREDENTIAL}
but the credential.getEmail() is returning my apple email!
credential.getIdentityToken().toString() return {length = 870, bytes = 0x65794a72 61575169 4f694a6c 57474631 ... 76305771 56344d51 }
nonce return bd35fb6eed1d9b4c17c21881f81e5e5390fa3a1ea5fe8f2a066346bcb578d575
and PROVIDER return apple.com
In my Entitlements.plist i remove:
com.apple.security.get-task-allow
because I try to build a deployment version - can this be a problem?
Can you suggest to me what can do?
—
Reply to this email directly, view it on GitHub https://github.com/mk-5/gdx-fireapp/issues/48#issuecomment-1007975957, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHTADFEZTMV4SHX5CCR5TPTUVAUZTANCNFSM4YO5G5AA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Pozdrawiam,
Mateusz Kulawik

Hi
Thank you for your help.
I downloaded your project and ran e3e the test.
I replaced GoogleService-Info.plist to work with our firebase and changed app.id in robovm.properties.
I had to replace "ios/libs/" I guess because of the difference in the PC architecture.
The other change I had was to remove this:
" com.apple.security.get-task-allow
"true />"
from Entitlements.plist in order to upload the project to the phone.
It still gives me the same error:
Domain=FIRAuthErrorDomain Code=17004 "Unable to parse the ID Token." UserInfo={NSLocalizedDescription=Unable to parse the ID Token., FIRAuthErrorUserInfoNameKey=ERROR_INVALID_CREDENTIAL}
Thanks in advance

@Daniel-Donev interesting ... mhmmm what do you mean by "I had to replace ios/libs/ beacuse of the difference in the PC architecture?

Hi
@mk-5 in your e3e the test project you have all libraries (frameworks) in ios/libs/ .
the libraries (frameworks) in your folder didn't work in our environment and we downloaded them from here -  https://firebase.google.com/download/ios and replaced them in the same folder.

@Daniel-Donev mhmm, for me it works. It really looks like some firebase-console configuration issue.
Have you seen this? https://stackoverflow.com/questions/56712711/login-through-facebook-using-oauth-issue-with-firebase-app-id-in-the-input-toke
Guy got the same error, FIRAuthErrorDomain Code=17004
It's hard to say something, because I cannot see your firebase console. I would go through points here: https://firebase.google.com/docs/auth/ios/apple?authuser=0 and check if everything is configured.

user: Have an Apple ID with two-factor authentication (2FA) enabled.
user: Be signed in to iCloud on an Apple device.
app bundle ID needs to match with bundle id configured in firebase console
enable apple sign in in Firebase Console -> Authentication tab

In addition you can check Entitlements.plist. It should pointed-out in your robovm.xml. This is what I got for e2e project:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0">
<dict>
	<key>com.apple.security.get-task-allow</key>
	<true/>
	<key>com.apple.developer.applesignin</key>
	<array>
		<string>Default</string>
	</array>
</dict>
</plist>

If everything from that is correct on your side, maybe you can create some demo project, with demo firebase app / configure everything as you did, and send it to me via zip? then I will be able to really examinate it.

Hi @mk-5
I try and test so many times my firebase-console configuration -

and all off this points: https://firebase.google.com/docs/auth/ios/apple?authuser=0
user: Have an Apple ID with two-factor authentication (2FA) enabled. - true
user: Be signed in to iCloud on an Apple device. - true
app bundle ID needs to match with bundle id configured in firebase console - true
enable apple sign in in Firebase Console -> Authentication tab - true
Its is
"com.apple.security.get-task-allow
"
important?
because I try to set it and my project can`t deploy on the iPhone and I removed...
Thank you for your help! I am a little sick at the moment, but I will make an archive as soon as possible.

Hi @mk-5 we send you an e-mail with demo project.
Thank you for your help.

@Daniel-Donev  all right, I'll check it as soon as I get it<EMAIL_ADDRESS>

@mk-5, just wondering did you get the email?

@Daniel-Donev nope, I didn't get anything

@mk-5, we have resended the e-mail from another e-mail account - Magic box.
If it`s not in you inbox please check the spam folder.

@mk-5 @Daniel-Donev
I am also stuck at same issue since last month and tried all the things mentioned here and on firebase  page for setting up apple login.
but stuck at errror:
java.lang.RuntimeException: Unable to parse the ID Token.
	at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:54)
	at pl.mk5.gdx.fireapp.ios.auth.AppleAuthProvider$1.invoke(AppleAuthProvider.java:50)
	at org.robovm.pods.firebase.auth.FIRAuth$$BlockMarshaler2.invoked(Unknown Source)
	at org.robovm.apple.uikit.UIApplication.main(Native Method)
	at org.robovm.apple.uikit.UIApplication.main(UIApplication.java:433)
	at com.dalmalgames.IOSLauncher.main(IOSLauncher.java:37
	```

Did you figure something out ?

@prateekchandan @mk-5
Unfortunately, I also can't resolve the issue with "Unable to parse the ID Token"
I repeatedly tried to integrate "Sign-In with Apple" as described without success.
I will follow with interest if there is any development on the subject. If you find some solution or integrate successfully, I will know that there is no problem with the integration.

@prateekchandan  let me run e2e test / and I will merge it right away. Awesome! 🍻

@prateekchandan  I fixed two small fixes directly in master 🙈 as I mentioned in a PR comment - for any reasons pipelines has been not triggered, and there were some syntax issues in the code.
https://github.com/mk-5/gdx-fireapp/commit/e3d64b7c7aeb8b9abb34357e06a02fd70bc39b3d
https://github.com/mk-5/gdx-fireapp/commit/d93904a5d6d048b5e485c7507ebf47517f490b28
Let me verify master after those changes, and I will release a new version

okay, so local e2e tests works

but there were fine for me previosuly as well. I will release a new version 3.3.1 - @Daniel-Donev  @prateekchandan  pls verify if the latest fix works

Awesome!
I will need time to re-integrate "Sign-In with Apple" as described.
Then I will give feedback.

It works for me, and launched the multiplayer version of my game here.
https://apps.apple.com/us/app/classic-callbreak/id6444685243 :D

@mk-5 Please feel free to close this issue.

@prateekchandan  awesome :)


Site Spanning will only send traffic to UDF site and not AWS

Environment

Application Lab

Describe the bug
In the "Site Spanning" section.
When you create additional pools for SPA/API it will default traffic to only one pool and not both.
Current workaround is to add site to existing pool instead of creating a new pool.

This also requires that spa listen on 8081 in both sites (vs. using port mapping).  it is not possible to have a pool with pool members that use different ports.  https://github.com/f5devcentral/spa-demo-app/pull/27

this has been resolved since we are now using one origin pool per service.


ryujinx: 1.1.1102 -> 1.1.1155

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for ryujinx is: Experimental Nintendo Switch Emulator written in C#
meta.homepage for ryujinx is: https://ryujinx.org/
meta.changelog for ryujinx is: https://github.com/Ryujinx/Ryujinx/wiki/Changelog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.1118 with grep in /nix/store/4s2rl3fm7iznklh4k96r0nn3z740ibnx-ryujinx-1.1.1118
found 1.1.1118 in filename of file in /nix/store/4s2rl3fm7iznklh4k96r0nn3z740ibnx-ryujinx-1.1.1118

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ryujinx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/4s2rl3fm7iznklh4k96r0nn3z740ibnx-ryujinx-1.1.1118 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ryujinx https://github.com/r-ryantm/nixpkgs/archive/d89d3e1588d76ecc4103986ff0877e76556b1115.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/d89d3e1588d76ecc4103986ff0877e76556b1115#ryujinx

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/4s2rl3fm7iznklh4k96r0nn3z740ibnx-ryujinx-1.1.1118
ls -la /nix/store/4s2rl3fm7iznklh4k96r0nn3z740ibnx-ryujinx-1.1.1118/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ryujinx

Maintainer pings
cc @IvarWithoutBones @06kellyjac for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for ryujinx is: Experimental Nintendo Switch Emulator written in C#
meta.homepage for ryujinx is: https://ryujinx.org/
meta.changelog for ryujinx is: https://github.com/Ryujinx/Ryujinx/wiki/Changelog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.1141 with grep in /nix/store/wiix8pwm6y45wcx0fsnp37b8r9rlwd2g-ryujinx-1.1.1141
found 1.1.1141 in filename of file in /nix/store/wiix8pwm6y45wcx0fsnp37b8r9rlwd2g-ryujinx-1.1.1141

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ryujinx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/wiix8pwm6y45wcx0fsnp37b8r9rlwd2g-ryujinx-1.1.1141 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ryujinx https://github.com/r-ryantm/nixpkgs/archive/3e28dd67cfada50ad7db1fe833b1d0074dfe8e12.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/3e28dd67cfada50ad7db1fe833b1d0074dfe8e12#ryujinx

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/wiix8pwm6y45wcx0fsnp37b8r9rlwd2g-ryujinx-1.1.1141
ls -la /nix/store/wiix8pwm6y45wcx0fsnp37b8r9rlwd2g-ryujinx-1.1.1141/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ryujinx

Maintainer pings
cc @IvarWithoutBones @06kellyjac for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for ryujinx is: Experimental Nintendo Switch Emulator written in C#
meta.homepage for ryujinx is: https://ryujinx.org/
meta.changelog for ryujinx is: https://github.com/Ryujinx/Ryujinx/wiki/Changelog
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.1.1155 with grep in /nix/store/jgz701bzr65lqd1l76gxzf8mf0f8hv04-ryujinx-1.1.1155
found 1.1.1155 in filename of file in /nix/store/jgz701bzr65lqd1l76gxzf8mf0f8hv04-ryujinx-1.1.1155

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
ryujinx

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jgz701bzr65lqd1l76gxzf8mf0f8hv04-ryujinx-1.1.1155 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A ryujinx https://github.com/r-ryantm/nixpkgs/archive/bb5c2a8f8e633140f00a10f30fa22c5e50f54c6c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/bb5c2a8f8e633140f00a10f30fa22c5e50f54c6c#ryujinx

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jgz701bzr65lqd1l76gxzf8mf0f8hv04-ryujinx-1.1.1155
ls -la /nix/store/jgz701bzr65lqd1l76gxzf8mf0f8hv04-ryujinx-1.1.1155/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

ryujinx

Maintainer pings
cc @IvarWithoutBones @06kellyjac for testing.


About the teleport mines

Is it possible to add a config - which would be responsible for enabling the destruction of mines - after teleportation. It’s just that over the entire time I’ve been using this mod, I’ve noticed that people are abusing the system with mines, constantly teleporting, saving things, and so on. And so once it happened - the teleport mine broke - you travel further (only if the problem with the monsters remains - because of which they teleported by stepping on the mine, then this will not be relevant)

I'm assuming this is talking about Lethal Things? That's a different mod.


Local GPO-listener unable to connect

Hi!
I'm quite new to code so it's probably just a minor issue with my setup. I've tried both localhost and running the server on another computer and i still get the same error. Downgraded the socket.IO to the version recommended. I want to run the server and listener on the same Pi.
I'm getting the error:
Unable to connect to Tally Arbiter server: <IP_ADDRESS> 4455
Traceback (most recent call last):
File "tallyarbiter-gpolistener.py", line 165, in
sio.connect('http://' + server_config['ip'] + ':' + str(server_config['port']))
File "/usr/local/lib/python3.7/dist-packages/socketio/client.py", line 284, in connect
raise exceptions.ConnectionError(exc.args[0]) from None
socketio.exceptions.ConnectionError: Connection refused by the server
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "tallyarbiter-gpolistener.py", line 173, in
time.sleep(15)
File "/usr/local/lib/python3.7/dist-packages/engineio/client.py", line 39, in signal_handler
return original_signal_handler(sig, frame)
File "/usr/local/lib/python3.7/dist-packages/socketio/client.py", line 26, in signal_handler
return original_signal_handler(sig, frame)
KeyboardInterrupt

Did you find a solution to this issue? I am having the same problem.

Did you find a solution to this issue? I am having the same problem.

I was only downgrading the server socket.io and not the client.
Just uninstall the client with
sudo pip3 uninstall python-socketio[client]
and install the last version 4 client with
sudo pip3 install python-socketio==4.6.1

I was only downgrading the server socket.io and not the client.
Just uninstall the client with
sudo pip3 uninstall python-socketio[client]
and install the last version 4 client with
sudo pip3 install python-socketio==4.6.1

Sorry about this. I’ll be working to get everything brought up to the latest socketio ASAP.

Sorry about this. I’ll be working to get everything brought up to the latest socketio ASAP.


Update Security md to point to website

Closes #???
Description

For contributor use:

[ ] Targeted PR against master branch
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work
[ ] Code follows the standards mentioned here
[ ] Updated relevant documentation
[ ] Re-reviewed Files changed in the Github PR explorer
[ ] Added appropriate labels

Codecov Report

Merging #328 (31fc28e) into master (b7fe92a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #328   +/-   ##
=======================================
  Coverage   52.45%   52.45%
=======================================
  Files          36       36
  Lines        1775     1775
=======================================
  Hits          931      931
  Misses        723      723
  Partials      121      121

Flag
Coverage Δ

unittests
52.45% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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


fido-authenticator/Cargo.toml: use github URL for trussed by default

By default the project does not build if trussed repository is not
locally copied to the expected location. It also causes the CI builds
to fail.
Signed-off-by: Maciej Pijanowski<EMAIL_ADDRESS>

Thanks, slightly adapted and merged!


mummy only serves on ipv6 since 0.3.0 without specifying address.

since i updated to 0.3.0 i get 502 bad gateway if i use
let server = newServer(router)
server.serve(Port(8080))

checking in the terminal with ss -l -t -p shows mummy listening on [::]:* only.
If i change my serve proc to server.serve(Port(8080), address = "<IP_ADDRESS>") ss shows mummy listening on  <IP_ADDRESS>:* and it works in the browser again. It's also not listening on the ipv6 address anymore.
I'm running fedora 37 with ipv6 disabled in NetworkManager, btw.
thanks

Hm, well that is a bad default behavior. IPv6 support was contributed in https://github.com/guzba/mummy/pull/48 and tbh maybe I should remove it? Do people actually use IPv6? I will never need it (always behind reverse proxy lb) so I have not learned much about it.

Hi! Yes, I tried to use it in an IPv6-only environment, and found that it didn't work, hence the patch.
Perhaps IPv4 should be assumed when no address is specified, even if IPv6 is available?  Maybe better, it should listen on IPv6 and IPv4 if both are available and neither is explicitly specified?  I think the first option ought to be very easy to do, but the second might be harder as I think Mummy makes the core assumption that it is only ever going to listening on one interface with one configuration.
Another option (as @guzba suggests) would be to drop IPv6 support.  It is a tiny change that would be easy to maintain out-of-tree if I wanted it.  It seems sort of defeatist though - the future is coming whether we like it or not!  Ok, fine, maybe it's coming very, very, very slowly in the case of IPv6... :-)

Just to address the question about the websockets test - I think probably not important - it was just that the test was testing only IPv4, which obviously fails if you're listening on IPv6, so I changed the test to not make that assumption.  The only reason for the comment on it was that I think I had initially neglected to include it in the patch.

Perhaps IPv4 should be assumed when no address is specified, even if IPv6 is available?

I think this is a reasonable default. It could always be revisited later. It might even be safer, in case the user is not aware ipv6 is enabled, and/or that mummy is listening on that address.

PR #76 is ready for testing!

My issue with IPv6 is that I personally have no idea what I'm supporting. I am kind of shocked that localhost would default to IPv6. Is this always the case? When does that happen?
I know its "bad", but I have zero interest in becoming an IPv6 expert given all the other work I want to do, so now that I've seen IPv6 support has implications on default behavior I'm feeling very uneasy about promising to support it for others when I'll never use it myself.
Is it possible to craft a PR such that IPv6 will only be used when a user is explicitly asking for it? I would never ever want anyone to get IPv6 accidentally.

I'm not sure if it is always the case, but I think for some years now IPv6 has been ready to go and actively used in most OSes, it's just that if your ISP only supports IPv4 (and most do only support that), you may never see it in action, except with localhost maybe.
I don't think you'll need to become an IPv6 expert because a web server only really needs to lightly graze that layer to open a socket, but after that isn't it all HTTP?  One of the reasons the original PR was so easy was that it really doesn't affect anything other than the style of IP address when opening the socket.
I've update #76, so that the only way to get an IPv6 socket would be to explicitly specify an IPv6 style IP address, in the case of either not specifying any IP address or specifying "localhost", IPv4 will be forced.
For what it's worth, I think this is the wrong approach.  I think that the web server should work with the environment it is in, wherever possible.  I.e. If the OS would open an IPv6 socket given the parameters provided, that is what the user would expect and that is what the user should get.  The web server probably shouldn't attempt to subvert that.

Actually, I think "what the user would expect" in most cases is going to be whatever Apache, NGINX, et al. would do.
And they would, I think, if "localhost" resolved to an IPv4 and an IPv6 address, open sockets for both.
I don't think mummy is able to do that at present, though?

I am not at PC now so I need to sit down and read the full post but I do agree with this statement:

Actually, I think "what the user would expect" in most cases is going to be whatever Apache, NGINX, et al. would do.

I will be back on this later but that strikes me as eminently reasonable.

Let me rewind the tape and start from the top. This issue was opened because Mummy was only listening on IPv6 localhost even though IPv6 was disabled in NetworkManager. How this situation arise?
It appears Mummy would use Domain.AF_UNSPEC and then create a socket based on what the getAddrInfo syscall returned. Something seems wrong about this outcome. Before I jump to any further conclusions I'd like to start by understanding this.

Potentially related: https://stackoverflow.com/questions/55627377/getaddrinfo-returns-only-ipv6-when-using-af-unspec

I've just re-read the OP and there's a potential issue.  This could be just a couple of typos, but experience tells me to always check anyway!
@ITwrx says that mummy is listening on [::], but if no address is given to serve(), it will use "localhost" (which should be [::1] in IPv6 land).  Again, when the OP says that they specified <IP_ADDRESS>, mummy was listening on <IP_ADDRESS>, which I don't think it should be either (and if it is, isn't that another bug and security problem?).

i just checked again with server.serve(Port(8080)) and now ss -l -t  (less verbose output than ss -l -t -p )  shows:
<IP_ADDRESS>:http and [::]:http for the Local Address: Port, but i still get 502 error. Maybe the ipv4 hid from me last time, but i kind of doubt it, though i have no idea what would cause a variation.
with server.serve(Port(8080), address = "<IP_ADDRESS>") i get:
<IP_ADDRESS>:http and [::]:http but no 502. serves fine. using nginx  as reverse proxy via proxy_pass http://<IP_ADDRESS>:8080/ with <IP_ADDRESS> for local domain in /etc/hosts for nginx server_name and static files.
yeah, i think i forgot to mention, or missed, the <IP_ADDRESS> issue. I'm behind a firewall right now, but not so great for production.
ipv6 is still disabled in NetworkManager and still on mummy 0.3.0. Someone might want to reproduce in case there's something specific to my env.

I'm wondering whether there is a web server already running on your machine?  I'm just wondering why are you seeing something listening on all-interfaces when you specify a specific interface?  What output do you get when Mummy is not running?

Thanks!
!@$% i didn't see the -n option for ss... i knew i didn't like it showing the service names only, but i didn't look well enough for alt options...so ss was showing nginx on port 80 as http too, of course.
so, with nginx stopped...
with server.serve(Port(8080)) i get:
[::1]:8080
what's really strange is if i visit http://localhost:8080, mummy serves the app somehow! If i had nginx running it would 502... For the record ip a doesn't show an ipv6 address for my ethernet interface either, so it's not just NetworkManager lying. :) IDK/remember, maybe ipv6 localhost still works even though ipv6 is disabled otherwise, by design?
with server.serve(Port(8080), address = "<IP_ADDRESS>")  i get:
<IP_ADDRESS>:8080 only. mummy still serves the app on http://localhost:8080

i was looking at the lan interface and not the loopback with ip a when wondering about localhost!
lol, how embarrassing. :)
lo is:
inet6 ::1/128 scope host
so that explains why that works. I guess it's just a question of defaults at this point.

Don't worry - I only knew to ask because I've done the literal exact same thing! :-)
You might be right about Network Manager, although I don't use it myself and the last time I saw it in action was I think over 10 years ago!  It used to be focused on allowing the end user to easily connect to transient networks like WiFi hotspots, and so perhaps the loopback and maybe other interfaces are dealt with elsewhere (certainly that used to be the case back then if I remember correctly).
The behaviour of Mummy that you're describing there sounds correct and as designed at the moment.  It can only listen on one socket at the moment.  I'm not sure if this would align with @guzba's plans for Mummy as a back-end/proxy server, but it might be useful to make it listen on multiple addresses and ports if it was to evolve into a more general purpose web server, if only to deal with IPv4 + IPv6 and/or http + https in one instance.
@guzba What do you think about defaults?  Should PR #76 be discarded or changed?  My vote would be to drop it, as I don't think it will fix anything but it does add a restriction that might cause future issues.

@finitemonkey I think dropping #76 is best. Assuming it is currently behaving in a sensible way, which appears to be the case, I don't think messing around with things is necessary.
Listening on multiple addresses / ports is something I have had working in a branch at one point, I don't recall it being a big deal and it could be revisited at some point.

@guzba  maybe you lost track due to all my mistakes and posts, but it's still doing what i said in the original post, except it's serving over <IP_ADDRESS> like it's supposed to when the ip is specified, instead of <IP_ADDRESS>, as i incorrectly reported.
I don't have a strong opinion on how you decide to handle it, but if it's going to serve over ipv6 by default when no address is specified (the way it's documented to be used), even when ipv6 is disabled in NetworkManager, you might want to update the docs to specify an address, as the default results in a 502 when using with a reverse proxy config (that expects ipv4), and is served over ipv6 lo regardless of reverse proxy, neither of which would be expected. I had to search for the syntax for specifying the address, for instance.
I don't use NetworkManager for non-desktop machines either, but it is probably the most common for Ganoo/Linox desktop OSes, so i doubt i'm an outlier in this regard.

I am genuinely sorry finitemonkey, I really appreciate your patience and helpfulness and I wish this could have worked out better, but I really have no interest in figuring out what is going on here or why, for the sake of IPv6.
I do not understand all the Linux settings and defaults affecting this. I do not know what the defaults for other flavors of Linux are. I do not want to take the time to learn when I have so many other more fun things to work on.
I am going to revert IPv6 support and just do simple IPv4 which I can understand, actively use, and will be able to maintain for the foreseeable future.

Reverted and tagged in https://github.com/guzba/mummy/releases/tag/0.3.1

Thanks @guzba, I understand, and appreciate the consideration toward me about it but it is a totally logical decision to make.
I might have a go at reintroducing the multiple socket listening code in my own fork, and see how that goes!


Update Drush version from 8.3.1 to 8.3.2, fixes #1999

Related #1999

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@truls1502 you don't seem to have given me permission to push into this, so you'll need to bump the version.go to point to the webimage I have pushed for you. All you have to do is change the WebTag in pkg/version/version.go :
commit 9190c9d84918e5ec4fd576102aed170933cd39a8
Author: Randy Fay<EMAIL_ADDRESS>Date:   Thu Dec 12 10:58:11 2019 -0700

    Bump version to point to new container

diff --git a/pkg/version/version.go b/pkg/version/version.go
index 5d43742b..c6172b02 100644
--- a/pkg/version/version.go
+++ b/pkg/version/version.go
@@ -49,7 +49,7 @@ var DockerComposeFileFormatVersion = "3.6"
 var WebImg = "drud/ddev-webserver"

 // WebTag defines the default web image tag for drud dev
-var WebTag = "v1.12.1" // Note that this can be overridden by make
+var WebTag = "20191212_truls_drush" // Note that this can be overridden by make

Did you see that this is number 2000! Celebration!

Did you see that this is number 2000! Celebration!

Ahh, it is a nice number! 🎉👍😍

Thanks so much for this!


Incremental Groovy compilation when a file moves between source roots

When a source file moves between source roots, then the GroovyCompile task will be up-to-date, since the path sensitivity for the sources is RELATIVE. When the moved file changes afterwards, then we won't be able to map the source file to the compiled classes and will not remove the leftover classes, since we use the absolute path as a key for the source to classes mapping.
We probably need to use a relative path for the mapping. That would also allow to share the mapping file via the build cache.

@wolfs is this the same problem as in https://github.com/gradle/gradle/issues/9380?

@isker It is similar, though not really the same since Groovy incremental compilation uses a slightly different mapping from class names to source files than Java incremental compilation. I wouldn't be surprised if the same error occurs though.


Link fix

linksFix checkReference was being past a reference to the container configuration but it was assigning it to another varible rather than using it directly

Looks good. Please squash your commits

I completed the requested squash.


Use getMethod instead of getDeclaredMethod for valueOf reflection

Using getDeclaredMethod requires the privilege accessDeclaredMembers
when running under a SecurityManager. However, this returns private,
protected and package private methods as well as public, while static
valueOf functions are already required to be public. Since public
is all that is needed, getMethod works fine, and doesn't require any
special privileges.

Thanks for this!


Support multiple git worktrees

Scenario: I have two PRs in flight for Kubernetes. They may or may not change vendored deps and require their own GOPATH. I can spare the HD, so I don't mind some of GOPATH dependencies being duplicated on disk.
I'd like to be able to do:
goclone k8s.io/kubernetes feature1
goclone k8s.io/kubernetes feature2

and see this on my FS:
$HOME/workspace/gopath--k8s.io-kubernetes
$HOME/workspace/gopath--k8s.io-kubernetes--feature1
$HOME/workspace/gopath--k8s.io-kubernetes--feature2

where

$HOME/workspace/gopath--k8s.io-kubernetes/src/k8s.io/kubernetes is a full git clone
$HOME/workspace/gopath--k8s.io-kubernetes--feature{1,2}/src/k8s.io/kubernetes are worktree "checkouts".

I have something like this working in a fork, but I was messy. It removes support for passing in args to be exec'd, and makes a couple breaking changes to how the workspace path is calculated in the first place. https://github.com/colemickens/goclone/tree/worktree
If you (or anyone else is interested, I could clean it up...).

I think this can worm without complicating or breaking the existing CLI processing.
If you make the syntax like:
goclone k8s.io/kubernetes@feature1

you can just detect that a @ exists and all you need to do is to append --${branch} to the path, and do an extra git checkout ${branch}.
The only part I'm not sure it whether changing the branch should be done every time you run goclone, or only the first time when the repo is cloned. I think latter is a better option, because you may have a feature1 fork which can have multiple branches for various experiments etc.

The only complicated part of this would be updating the regexps that parse the repo reference. But I think the branch should definitely be part of that argument ($1).

@ahmetb nice idea with the '@' delimiter. I also thought about whether to "ensure on branch" or just switching the first time. I also agree with your conclusion.
I'll do this in a bit.


Add guidance on external collaborators

Addresses https://github.com/ministryofjustice/operations-engineering/issues/143 to update the GH account guidance for external collaborators.

This is great 👍


installing kvm-vdi on osx

hello there,
how can i install on osx ?

you'd be looking for running qemu-kvm on osx and i'm not sure i've seen that before.
in theory i guess it is possible but you would be in for a lot of work i think.

thanks a lot


fix CustomMimeType parameter

hopefully this time i fixed the formatting issue. vscode was "autoformatting on save"

@jonsequitur, I had to cancel my previous PRs. Initially I was not able to figure out what was happening with formatting, and then just luckily sopped how VSCode "auto-formatted" on save. Hopefully it works this time

Thanks!
This looks better.
One thing that I might not have been clear about is that removing the CustomMimeType property entirely, and just using MimeType, would be better still. The extra concept makes things harder to understand, and the MimeType property shouldn't be restricted to just a few types. It's intended to open-ended and this is how it works everywhere else in .NET Interactive.

Thanks!
This looks better.
One thing that I might not have been clear about is that removing the CustomMimeType property entirely, and just using MimeType, would be better still. The extra concept makes things harder to understand, and the MimeType property shouldn't be restricted to just a few types. It's intended to open-ended and this is how it works everywhere else in .NET Interactive.

Well, I agree, but for some reason someone did split them into two different options. And behavior is totally different, as far as I understand. What was the reason for splitting them?

@jonsequitur this time I removed CustomMimeType parameter completely

Thanks, @eosfor!


Removed duplicate quantity on notification

Prevent duplicate quantities in Ground Item Notifications
This fixes #10319 as PR  #10526 doesn't look like it is being worked on currently.
The format for notifications is now is now
[{player}] received a highlighted drop: {name} x {quantity}!

I'm not sure why this failed the build it worked fine for me :(

#10526 was merged in favor of this PR


Support IntelliJ 2020.3

IntelliJ 2020.3 EAP just released.

Plugin 'Lombok' (version '0.31-2020.2') is not compatible with the current version of the IDE, because it requires build 202.* or older but the current build is IU-203.3645.34

Same problem, roll back to IntelliJ 2020.2.2 not working too :-1:

how to build plugin?

"Mateu MDD" is somewhat old fork of this plugin that seems to be working for now.

What is going on with this plugin, I used to run EAP's and the plugin would just work, but in the last 6-12 months worth of EAPs/Releases seems like the plugin is broken for a few days before it gets fixed.

Can the untilBuild field just be configured to not be populated? I tried this locally and installed in the EAP and everything seems to be fine.

I have also tried locally build and install without 'untilBuild' on EAP and it is working fine.
https://jetbrains.org/intellij/sdk/docs/tutorials/build_system/gradle_guide.html#patching-the-plugin-configuration-file
Changed only in build.gradle:
patchPluginXml {
  version artifactVersion

  pluginDescription(file(descriptionFile).text)
  changeNotes(file(changesFile).text)

  untilBuild(null)
}

Can the untilBuild field just be configured to not be populated? I tried this locally and installed in the EAP and everything seems to be fine.

This may work but if there is an issue like #829 then IntelliJ won't warn the user and after upgrade the plugin may not work properly.

Can the untilBuild field just be configured to not be populated? I tried this locally and installed in the EAP and everything seems to be fine.

This may work but if there is an issue like #829 then IntelliJ won't warn the user and after upgrade the plugin may not work properly.

That's very true, but personally, I'd rather risk a chance that it may not work properly than a guarantee that it won't work. Many people are dead in the water while we wait for an upgrade. I'll be building from sources as a workaround.

Right now the plugin just gets disabled and doesn't work at all.

I went ahead and opened a PR

Just release 0.32 build, included one for IntelliJ 2020.3 EAP

The new EAP from today also breaks Lombok.In the IDE everything looks fine but build fails.
Message is:
java: You aren't using a compiler supported by lombok, so lombok will not work and has been disabled.
Your processor is: com.sun.proxy.$Proxy27
Lombok supports: sun/apple javac 1.6, ECJ

@amiram Please make an issue in lombok project here:  https://github.com/rzwitserloot/lombok/issues
This can be fixed only in lombok self.

Did somebody log a bug? Is it being worked on?
I updated IntelliJ IDEA to version 2020.3 yesterday when prompted by IntelliJ, and it warned me that the Lombok plug-in was not compatible. I assumed there would be an updated plug-in available, but there is not. And I am not able to build my project.

Try uninstalling and reinstalling the plugin. Sometimes that helps fix this problem for me.

Did somebody log a bug? Is it being worked on?
I updated IntelliJ IDEA to version 2020.3 yesterday when prompted by IntelliJ, and it warned me that the Lombok plug-in was not compatible. I assumed there would be an updated plug-in available, but there is not. And I am not able to build my project.

Starting from 2020.3 Lombok plugin is bundled with IntelliJ IDEA so uninstall the plugin and update IDEA. Lombok will work out of the box.
Tested in macOs Big Sur.

@seenukarthi Is this only true for Ultimate or does it apply to the community edition as well?  In my case when I updated to 2020.3 (community), it set the plug-in version to 0.32 (had been 0.33).  When I did as you suggested and removed the plug-in entirely then my build broke (nothing with val or any of the annotations was recognized).
This is Windows and my project is Groovy/Java based on Gradle.

@seenukarthi Is this only true for Ultimate or does it apply to the community edition as well? In my case when I updated to 2020.3 (community), it set the plug-in version to 0.32 (had been 0.33). When I did as you suggested and removed the plug-in entirely then my build broke (nothing with val or any of the annotations was recognized).
This is Windows and my project is Groovy/Java based on Gradle.

Yes, you are correct. After fresh install of community version, I can see the plugin version is 0.32-EAP

In the plugin site It's 0.33-2020.2

I'm not sure this is a plugin issue or platform issue.

Using 2020.3 and the 0.32 plug-in I am getting:
java: You aren't using a compiler supported by lombok, so lombok will not work and has been disabled.
  Your processor is: com.sun.proxy.$Proxy32
  Lombok supports: sun/apple javac 1.6, ECJ

Above it says that this is a Lombok issue -- was one ever filed?

OK, that was with Lombok 1.18.4.  If I updated to 1.18.16 I get:
java: Compilation failed: internal java compiler error

And essentially nothing builds.

Logged https://github.com/rzwitserloot/lombok/issues/2670 based on the assertion above that this must be fixed in the core Lombok.


Online expansion support

longhorn/longhorn#1674

Bypassing the code factor check on purpose, but we need to revisit our code style in the future.


CLR failing with KeyError: 'batch_size' during model.fit

Have defined my lr callback as follows
lr_manager = OneCycleLR(max_lr,end_percentage,
scale_percentage,maximum_momentum,
minimum_momentum,verbose=True)
Then have further created the callback
callbackscustomx = [lr_manager,
ModelCheckpoint(filepathx, monitor='val_acc', verbose=1, save_best_only=True)]
But then model.fit is failing ....
modelCLR = model1.fit_generator(Customdatagen.flow(train_features, train_labels, batch_size=64),
steps_per_epoch=len(train_features) / 32,
epochs=nb_epoch,
verbose=1,
shuffle=False,
callbacks=callbackscustomx,
validation_data=(test_features, test_labels)
)
It is failing with following error. Looks like CLR expects batch size but then we cant pass the value?
147
148         self.epochs = self.params['epochs']
--> 149         self.batch_size = self.params['batch_size']
150         self.samples = self.params['samples']
151         self.steps = self.params['steps']
KeyError: 'batch_size'

https://github.com/titu1994/keras-one-cycle/blob/799e7042ac9cad4bee0caca96a964b721f76a823/clr.py#L149

The problem is that params is inherited from keras.callbacks.Callback and it is initialised based on the parameters passed to fit/fit_generator. You are not setting batch_size in your call to fit_generator (because you can't) and therefore can't be found.
I assume the example in the repo manages to find the batch_size argument because it extracts it from datagen.flow?

A possible solution is to use the set_params method inherited from Callback
from clr import OneCycleLR

lr_manager = OneCycleLR(max_lr=10, maximum_momentum=0.9, verbose=True)
lr_manager.set_params({"batch_size":  64})

A possible solution is to use the set_params method inherited from Callback
from clr import OneCycleLR

lr_manager = OneCycleLR(max_lr=10, maximum_momentum=0.9, verbose=True)
lr_manager.set_params({"batch_size":  64})

As explained here, that does not work because 'batch_size' is not included in params when it is built by fit_generator(). The solution I found is to modify the code and add the batch size and number of samples as parameters to OneCycleLR.


chore(website): Update the website to bootstrap 3.1 and make it working on mobile devices

Make the site web responsive to work on mobile devices.
Add meta tag viewport
Update the download modal & remove the modal backdrop
Update the popovers in html and JS files
Remove the video modal and embed videos in the pages
Update Bootstrap to 3.1.0
Update Font Awesome to 4.0.3
Use CDN for faster loading
Update the copyright to 2014
Clean up

We are reworking the site in a different way so sadly this PR is no longer likely to be landed. Thanks @christophe-f for the submission.


Fix NSBackgroundActivityScheduler exception caused by interval value

Resolves #1389

@barijaona: This perhaps also resolves this issue: https://forums.cocoaforge.com/viewtopic.php?f=18&t=27232.


HTTPS certificate check stopped working as intended

I'm not sure if that's connected to Go runtime or my code, but a long time ago this behaved as intended.
The actual bug is that when HTTPS certificate becomes expired, jsonmon does not report it as error until the app is restarted.
Opened a golang-nuts thread hoping for more ideas.

Fixed in v3.1.3


internal/database: check all connections are closed prior to test close

It appears paygate has connection leaks so we need to run through connections and verify/close them. I got the idea for checking this from the link below.

Idea: https://redd.it/4j5ysh

Codecov Report

Merging #344 into master will decrease coverage by 4.43%.
The diff coverage is 53.84%.

@@            Coverage Diff             @@
##           master     #344      +/-   ##
==========================================
- Coverage   63.77%   59.33%   -4.44%
==========================================
  Files          60       60
  Lines        5833     5868      +35
==========================================
- Hits         3720     3482     -238
- Misses       1426     1738     +312
+ Partials      687      648      -39

Impacted Files
Coverage Δ

internal/database/mysql.go
11.9% <0%> (-48.59%)
:arrow_down:

internal/micro_deposit_attempts.go
63.63% <100%> (+3.63%)
:arrow_up:

internal/micro_deposits.go
65.43% <100%> (-1.24%)
:arrow_down:

internal/receivers.go
58.72% <100%> (+0.27%)
:arrow_up:

internal/gateways/storage.go
65.21% <100%> (+1.03%)
:arrow_up:

internal/depositories.go
60.86% <100%> (+0.17%)
:arrow_up:

internal/depository_migration.go
60.41% <100%> (+1.72%)
:arrow_up:

internal/customers/cursor.go
60% <28.57%> (-8.63%)
:arrow_down:

internal/database/sqlite.go
55.55% <33.33%> (-1.31%)
:arrow_down:

internal/refresh_ofac.go
53.03% <60%> (-10.05%)
:arrow_down:

... and 9 more

Continue to review full report at Codecov.

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


Adding a warning message for SIGINT OOM failures

This warning should help identify OOM failures caused by docker killing the service process.
Fixes https://github.com/TypeStrong/fork-ts-checker-webpack-plugin/issues/357

This is great.

One other concern is that when this happens, webpack still exits cleanly. I'm wondering if we should push an error to compilation.errors to trigger a webpack build failure.

That's a great idea to push error to the compilation.errors. It will work only for async: false. Could you add this to this PR?

That's a great idea to push error to the compilation.errors. It will work only for async: false. Could you add this to this PR?

I don't mind doing that, but it will require more manual testing. Would you be ok with merging this and making that a separate PR?

Ok, no problem :)

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


player.html.php: CCA table doesn't list Challenges 6-12

I'll take a look at this one this evening.

Fixed.