How to ensure a user's email is verified before they can sign in

Related to https://github.com/feathersjs/feathers-authentication/issues/391.

Original bug report by @IBwWG
OK, so, as a newcomer, I really am not sure where exactly this issue fits into this repo, but @eddyystop is pretty involved here so I'm taking his word for it.  :)  (Original issue is at https://github.com/eddyystop/feathers-starter-react-redux-login-roles but I'm assured that it's not about that repo.)
Steps to reproduce

git clone https://github.com/eddyystop/feathers-starter-react-redux-login-roles/
npm install
npm start
curl -X POST -H "Content-Type: application/x-www-form-urlencoded" -H "Cache-Control: no-cache" -H "Postman-Token: 95cf199c-f038-c893-7053-a8a09fbef2ca" -d 'name=i have a name&username=nammmmmmmmmmme&password=what the hey&confirmPassword=what the<EMAIL_ADDRESS>"http://localhost:3030/users"
curl -X POST -H "Content-Type: application/x-www-form-urlencoded" -H "Accept: application/json" -H "Cache-Control: no-cache" -H "Postman-Token: 018a022f-bf1b-7e4f-cf11-db40a5fce490" -d<EMAIL_ADDRESS>the hey' "http://localhost:3030/auth/local"

Expected behavior
Failure, since I never verified with the "e-mailed" token.  (i.e. I didn't use the link that appears in the console at step 4.)
Actual behavior
Success and JWT token given via JSON.  If you scrap the Accept header in step 5, you get a similar result served up in HTML.
System configuration
This is happening both on a Windows box and a Linux box am I testing on.
Module versions (especially the part that's not working):
feathers-authentication 0.7
feathers 2.0.3
NodeJS version:
Windows: node 7.3.0
Linux: node 6.9.2
Operating System:
Windows: 7x64sp1
Linux: Mint 17.3 (32-bit)
Module Loader:
see https://github.com/eddyystop/feathers-starter-react-redux-login-roles

I think the main issue is around documentation or we should have a hook that someone can use to ensure that an email exists and has been verified. Currently the way I see this working is a hook after auth.hooks.authenticate or it is a custom verifier for feathers-authentication-local that upon looking up the user by email also ensures that the email has been verified.

feathers-authentication-management's isverified hook checks if the email address has been verified. Have you included that hook?
feathers-authentication is not aware of the fields maintained by feathers-authentication-management. When that happens, the isVerified code can be moved into feathers-authentication-local.

Could you maybe comment:

Is this scheduled any time soon?
I need this, is this doable? And what is suggested way to implement this using current feathers-authentication and feathers-authentication-management and feathers-authentication-local.

@marshallswain ^^^
The hooks to do this yourself are documanted https://auk.docs.feathersjs.com/api/authentication/local-management.html#hooks
Code: https://github.com/feathers-plus/feathers-authentication-management/blob/master/src/hooks.js#L42

Thanks for the reference to the docs. I am not sure what is best practice:

have users make an account (with limited features) and then get them to verify their emailaddress
make an account only after they verified their email

Currently my setup is 1 and it works, but 2 makes sense as the original issue already mentioned 'bots could easily make lots of accounts etc.'. When choosing 2 I would not know how to do it as the isVerified is a property of user, so you first need to create a user to have isVerified set to true, right?!

Interesting. Most people tend to go with option 2.

@eddyystop  giving this package a try today and also having same problem (i.e. in my case I can get login token  without email is verified).
I can understand when you write above that

The hooks to do this yourself are documanted https://auk.docs.feathersjs.com/api/authentication/local-management.html#hooks

But where should I use that hook to disallow auth token creation or login for thos with email not verified?
Thanks!

Are you using the isVerified hook? That will prevent a successful login regardless of what you get back. This repo works in conjuction with the Feathers authentication routines, it does not, and it cannot, override them.

I have not use isVerified hook yet because no idea where to put them.
But I found a workaround by customizing authentication-local verifier as per this documentation
Since I am using Feathers CLI. My modification is in \src\authentication.js
  class CustomVerifier extends Verifier {
    // The verify function has the exact same inputs and
    // return values as a vanilla passport strategy
    verify(req, username, password, done) {
        // My custom code as per documentation
    }
  }

  app.configure(local({
    Verifier: CustomVerifier
  }));

A bit of a hack. It works, but please let me know if I should do something else.

@HarisHashim You should be using the isVerified and addverification hooks as described in https://github.com/feathers-plus/feathers-authentication-management/blob/master/src/hooks.js rather than hacking authentication-local.

@HarisHashim I believe the article explains what to do.

I don't know if you still need help with this issue but the solution is very simple: add the isVerified hook in the before hook of authentication service.
 Import the verification hooks
const verifyHooks = require('feathers-authentication-management').hooks;

// src/authentication.js
module.exports = function (app) {
  const config = app.get('authentication');

  // Set up authentication with the secret
  app.configure(authentication(config));
  app.configure(jwt());
  app.configure(local());

  app.configure(oauth2(Object.assign({
    name: 'google',
    Strategy: GoogleStrategy
  }, config.google)));

  app.configure(oauth2(Object.assign({
    name: 'facebook',
    Strategy: FacebookStrategy
  }, config.facebook)));

  // The `authentication` service is used to create a JWT.
  // The before `create` hook registers strategies that can be used
  // to create a new valid JWT (e.g. local or oauth2)
  app.service('authentication').hooks({
    before: {
      create: [
        authentication.hooks.authenticate(config.strategies),
        verifyHooks.isVerified() // !!! Add the isVerified hook before authentication
      ],
      remove: [
        authentication.hooks.authenticate('jwt')
      ]
    },
    after: {
      create: [
        context => {
          // Add the user to the result response
          context.result.user = context.params.user;
          // Don't expose sensitive information.
          delete context.result.user.password;
        }
      ]
    }
  });
};

I believe the all issues here have been addressed. If you have any new comments please make them in authenticate-local-management rewrite.
Full details on the rewrite are posted to https://github.com/feathers-plus/authentication-local-management/blob/master/misc/upgrading.md


fix/check-synced.sh

Closing in favor of a larger solution, https://github.com/Start9Labs/electrs-wrapper/pull/19


Make the NDigits and total p.e. distributions at high energy in the new digitizer/trigger scheme agree with the old digitizer/trigger

The combination of the new SKI digitizer and NDigits trigger disagrees with the old WCSim SKI_SKDETSIM combined digitizer+trigger in the NDigits and total p.e. distributions for high energy electron particle guns. In the past, the WCSim SKI_SKDETSIM was compared* with the SKI mode of skdetsim, and good agreement was seen at high energy. Therefore it is important for WCSim's new classes to agree with the old class, in order to perform high energy physics studies.
*@cwwalter it'd be useful to know which configurations were tested (e.g. particle gun particle types, energies, etc.), which distributions were plotted, and how close the agreement was. Could you post a link to slides/documents?

NDigits is calculated as the number of digitized hits in the first subevent in the output file, and so is after the entire digitization + trigger process.

Total p.e. is calculated by summing Q of the digitized hits in the first subevent in the output file.
The above plots are for 5 GeV electrons without dark noise. See #89 verification plots for more configurations.
It is useful to explain how digits are created & saved

New code

Find the first raw hit on a PMT in the simulation. Integrate hits on this PMT for hit time + 200 ns.
Apply the efficiency factor & threshold function. Create a new digit if threshold is passed
Reject hits in the deadtime window (0 ns)
Find the next hit. Integrate for hit time + 200 ns. Go to iv.
Only digits that pass an NDigit trigger are saved

Old code

Trigger on raw hits
Find the first raw hit on a PMT in the subevent window. Integrate hits on this PMT for hit time + 200 ns.
Apply the efficiency factor & threshold function. Create a new digit if threshold is passed
Digitizer is now dead until next subevent window
All digits created are saved

The expected cause of this is the number of digits (digitized hits) allowed per PMT per subevent window - in the old code it is 0 or 1, whereas in the new code >1 is allowed.
There are a few potential solutions

Restrict the new code so that it allows only only 0 or 1 digit per PMT per subevent window.
Change the digitizer deadtime in the new code (verification plots appendix A shows that moving from 0 ns to 500 ns, this works for electron guns)
Change the digitizer integration window in the new code (#116) - a longer integration window will cause a reduction in NDigits, but not total p.e. (unless combined with digitizer deadtime)

I've tried having a look at the skdetsim code to see what values are used, and how many digits per PMT per subevent window are allowed, but have been unable to work this out (I can't run skdetsim, I don't have all the source files, and I don't know fortran). So it'd be useful if someone who knows skdetsim could take a look & report on how it does things, so that the best solution(s) can be found.
This should be solved before #115 is merged

I've done some tests restricting the maximum number of digits per PMT per subevent window is 1 (the first digit in time order is taken). There is good agreement between the old code and the new code with this restriction for both NDigits and average digit p.e.

So the next steps are:

work out what SKDETSIM does
get more information on what WCSim/SKDETSIM checks were done in the past

Then using this we can decide which method (restrict number of digits per PMT per subevent, change default time variables) to use to fix this bug.

@MakotoMiura, do you know the answer to this? Does the SKI electronics have multiple digits per PMT per (sub)event? Does the SKI code in SKDETSIM?

Can you define exactly what you mean by "multiple digits"?
If you mean do we know how many times a PMT was hit by photons in the window, I believe the answer is 'no' for all phases  of SK electronics. In SK-IV we have a QTC which records the start of the timing gate and charge for the time you have integrated up over threshold.  In SKI there were effectively 2 ADC/TDCs which switched over after the 1st one was hit to give you "dead timeless" operation.  @MakotoMiura or Hayato (no GH account) should be able to say more.

To define some terminology:

hit - a photon (or dark noise) creating a signal on a PMT
digit - an integration of hits in an integration window by the electronics (with threshold effect). Therefore there is the possibility of multiple hits per digit, and NHits >= NDigits is always true.
subevent - collection of digits saved around a trigger (e.g. -400 ns to +950 ns around the trigger time)

If you think of the case of a single PMT with 5 hits at 0 ns, and 5 hits at 900 ns (ignoring time and charge smearing):

single digit per PMT per subevent - Digits saved: 0 ns with 5 p.e. (this is the case in the old WCSim trigger)
multiple digits per PMT per subevent - Digits saved: 0 ns with 5 p.e., 900 ns with 5 p.e. (this is the case in the new WCSim NDigits trigger)

I'll send an email to Hayato-san, ccing you all.

After very detailed explanations from Hayato-san regarding how the SKI electronics work, we've decided not to simulate them exactly. But what we do know is:

a maximum of 1 digit per PMT per trigger window is allowed. I'll make a pull request with this limitation added to the NDigits trigger. As shown above, this will make the NDigits and p.e. distributions agree between the old & new code.
SKDETSIM seems to use the same algorithms as the new (post #89) code.
some default WCSim values are not the same as some as-built SKI values. See #112 for more.


feat(FN-1480): remove duplicate bank routes in Central API

Ticket
FN-1480
Introduction
There are two sets of endpoints for the fetching and creating of banks:

bank-routes.js

POST /bank
GET /bank/:id

portal-routes.js

POST /portal/banks
GET /portal/banks/:id

They point to the same controllers, so there is no functional difference. This appears to be some tech debt left over from when Central API / TFM was created (possibly the “bank routes" were added for TFM whilst the portal routes already existed).
Since the bank collection in MongoDB is generic, the “portal routes” version should be removed.
Resolution

Remove the routes from portal-routes.js, keeping the bank-routes.js versions.
The portal routes endpoints do not appear to be used anywhere, so no further refactoring is required.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main-stbr@dd5c94a). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             main-stbr    #2288   +/-   ##
============================================
  Coverage             ?   10.11%
============================================
  Files                ?      655
  Lines                ?    14835
  Branches             ?     1910
============================================
  Hits                 ?     1501
  Misses               ?    12052
  Partials             ?     1282

Flag
Coverage Δ

ui-test
39.80% <0.00%> (?)

unit-test
2.04% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Would it be possible to shade javassist to avoid conflicts with applications using it

Javassist is a popular library and ideally it should be shaded into HikariCP jar using https://maven.apache.org/plugins/maven-shade-plugin/ to avoid any conflicts. Conflicts can arise when hikari jar reside in application WAR as well as when it is deployed on web server level (where web servers do not separate class loaders for server and contexts). Not only it could cause issues for an application it can cause issues for the pool itself if application supplies incompatible javassist version so shading makes it much safer and makes it one less dependency to deploy.

In the upcoming 2.4.0 release javassist has ben eliminated as a runtime dependency completely.


update halloween_countdn app to change display on Halloween

Updated Halloween Countdn app to say "Halloween" on Halloween instead of "0 days".

@arock95 becuase you have safe modules (all 3 green checks) - have you seen if you can auto merge with the old "  :shipit: " comment.  I know it the past it's worked - if you have all 3 green.
I want this update (  #purelyselfish )

I wasn't familiar with that @jvivona so thanks for sharing!  Assuming it works via PR comment, :shipit:

I know it used to work - they may have taken it out.  I think it has to be it's own comment, not inline & it has to be from the author (you).

:shipit:

whomp whomp.   sorry - guess it's been removed by the team.   I'm sure they will get to it here shortly.

No worries, neat to know that it was possible at one time!


ScribblejrDrawNative function misname/manual mismatch

The script resource "ScribblejrDrawNative" names the function inside as ScribblejrNative() at line 21.
My suggestion is to rename the function from ScribblejrNative() to ScribblejrDrawNative() to match the naming conventions of other script assets & functions, and to match the documentation.

https://github.com/JujuAdams/ScribbleJunior/commit/b9005b229fed4bc0aca920f954fe05a044795d92


LuisBlanche/issue362

fixes #362

Can you add the example config given in the issue to inform users that they can reduce verbosity ?

I think there is a problem with the Flake8 pre-commit hook. It raises a lot of :
codecarbon/core/emissions.py:202:76: E231 missing whitespace after ':'. errors on files that were not changed.
And you can see from the following example that we cannot add a space after the ':' in that case.
logger.debug(
            f"We apply an energy mix of {emissions_per_kWh.kgs_per_kWh*1000:.0f}"
            + f" g.CO2eq/kWh for {geo.country_name}"
        )

Maybe something has changed with the Flake8 hook configuration ?

I had to ignore the following Flake8 error codes in .flake8 :
E201 : Whitespace after "("
E231 : Missing whitespace after ',', ';', or ':'
E226 : Missing whitespace around arithmetic operator
NB: these were only raising in the CI , not in local when running pre-commit run . E231 is raised at times it shouldn't be. The other ones are mostly raised because of line breaks introduced by black (which is acceptable imho)

@benoit-cty  what do you think about the error codes i had to ignore ? seems ok to merge ?

The problem was actions/setup-python@v3  that switch to Python 3.12, so I force 3.11.
I open a new issue for that : https://github.com/mlco2/codecarbon/issues/465
You could merge this PR :+1:


Create workshop on 'responsible disclosure'

See for example what GitHub did with https://github.com/blog/2273-incident-report-inadvertent-private-repository-disclosure

Workshop page is here https://github.com/OWASP/owasp-devseccon-summit/blob/master/Workshops/Responsible-disclosure.md
Now it needs some content


[FLINK-3764] [tests] Improve LeaderChangeStateCleanupTest stability

The LeaderElectionRetrievalTestingCluster now also waits for the Flink
resource manager to be properly started before trying to send leader
change notification messages to the LeaderRetrievalServices.

Looks good to me!
Please merge once Travis gives green light...

Merging.


LSP launching language server Elixir-LS on non-Elixir files

For some reason LSP is launching the Elixir-LS language server no matter what non-Elixir file type I open... then promptly crashes.

OS and language server
MacOS Mojave (10.14.4)
ST3 build 3206
Elixir-LS (https://github.com/JakeBecker/elixir-ls)

How you installed LSP (Package Control or from git?)
Package Control

Minimal reproduction steps

Open any random non-Elixir file... like subl -n data.csv
In status bar bottom right, it says Plain Text (CSV) which is the proper file type detection.
In status bar bottom left it says elixir-ls initialized, then an error comes from the language server No mixfile found in project. which isn't surprising since I opened a CSV file.
Finally a popup says "Elixir LS has crashed. See Output panel." and "Language server elixir-ls has crashed, do you want to restart it?"

Log

using gpu buffer for window
server: Started ElixirLS
server: Elixir version: "1.8.1 (compiled with Erlang/OTP 21)"
server: Erlang version: "21"
server:
11:42:47.822 [warn]  Did not receive workspace/didChangeConfiguration notification after 5 seconds. Using default settings.
server: Compiling with Mix env test
server: Compile took 61 milliseconds
server:
11:43:02.921 [error] GenServer ElixirLS.LanguageServer.Server terminating

My config:

{
  "clients": {
    "elixir-ls": {
      "command": ["~/Programs/language_servers/elixir-ls/language_server.sh"],
      "enabled": true,
      "languageId": "elixir",
      "scopes": ["source.elixir"]
    }
  },
  "log_debug": true
}

Perhaps try adding a ”syntaxes” key to your client config, as it seems to be missing. I am unsure what the behavior is when the syntaxes key is missing.

Thanks @rwols that fixed it.  I thought the "syntaxes" key was optional but I guess not.


support sizes/strides in eager mode

Currently, we set torch._dynamo.config.dynamic_shapes = False for simple eager MNIST test case(https://github.com/nod-ai/SHARK-Turbine/pull/92). This is because eager mode does not support dynamic shape at the moment. For that reason, the batch size is set to be divisible by the length of the dataset.
Otherwise, we would get the following error:
torch._dynamo.exc.BackendCompilerFailed: backend='turbine_cpu' raised:
MLIRError: Failure while executing pass pipeline:
error: unknown: failed to legalize operation 'torch.aten.view' that was explicitly marked illegal
 note: unknown: see current operation: %171 = "torch.aten.view"(%169, %170) : (!torch.vtensor<[?,14],f32>, !torch.list<int>) -> !torch.vtensor<[?,1,28,14],f32>

The following tasks are to be addressed to flip the flag to True in the near future.

Fix aten.view in torch-mlir
Support dynamic shapes in eager mode

I don't believe Bruce works on the project anymore.
For something that includes a report, generally, don't nag the reporter: see if the work was ever done.


[Leaflet] Update marker's popUp content without rewriting all HTML

Hi, I would like to know if there's a way to change the content of a marker's popUp without having to rewrite all the html that is already there, I mean, I would like to toggle a class of one of the element inside of the popUp, it appears that the class exist in the DOM, and it toggles right, but after I close the popUp of the markers, returns to it's previous state, as If toggle hasn't applied, I believe is because I need to change the initial HTML of the popUp, I would like to do this without having to rewrite all the html I already built, and since is a pretty heavy html, is complicated to rewrite everything, I tried using update() method, but is not working, hope somebody can help me,
 var objeto = Layerpints._layers;
      for(r in objeto){
        var html = Layerpints._layers[r]._popup._content;
        var html_parse = $.parseHTML(html);
        var up_tick    = $(html_parse).find('.up_tick');
      }
      $(up_tick).each(function(e){
          var id_pint_name = $(this).attr('name');
          if(id == id_pint_name){
            if($(this).hasClass('text-info')){
              console.log("remove class");
              $(this).removeClass('text-info');
              //Layerpints._layers[r]._popup.update();
            }else{
              console.log("add Class");
              $(this).addClass('text-info');
              //Layerpints._layers[r]._popup.update();
            }
          }else{
            console.log("do nothing");
          }
      });

this is the code I'm trying to use hope someone could help, thanks and greetings!

Somehow I managed to fixed it by changing the update() method to setContent() method like this
var objeto = Layerpints._layers;
for(r in objeto){
var html = Layerpints._layers[r]._popup._content;
var html_parse = $.parseHTML(html);
var up_tick    = $(html_parse).find('.up_tick');
}
$(up_tick).each(function(e){
var id_pint_name = $(this).attr('name');
if(id == id_pint_name){
if($(this).hasClass('text-info')){
$(this).removeClass('text-info');
Layerpints._layers[r]._popup.setContent();
}else{
$(this).addClass('text-info');
Layerpints._layers[r]._popup.setContent();
}
}else{
console.log("do nothing");
}
});
the truth is I don't know why this is working since I believed setContent() will empty my popUp but it didn't, perhaps someone could explain further, thanks

Somehow I managed to fixed it

Seems like the problem is gone, so closing this.

I believed setContent() will empty my popUp but it didn't

And it doesn't when there was no content passed, check the source code.
Otherwise, remember that you can use instances of HTMLElement instead of strings containing HTML for the popup contents - using those will allow you to keep their state around.


A brief usage example of how click and pydantic interact would be helpful

I am interested in configuring command-line parsing from pydantic models -- this could be very useful as I switched from standard dataclasses to pydantic, and would love to integrate these models with command line parsing.  However I am having a bit of trouble trying to figure out intended use from your tests.
For example, if I attempt to replicate your test_basic_field function:
import sys
from typing import Dict, List, Tuple, Type, Any
from pydantic import BaseModel, BaseSettings

from cyto.settings import autofill

# from cyto/tests/conftest.py
class Argv:
    def __init__(self, monkeypatch: Any) -> None:
        self._monkeypatch = monkeypatch
        self._sys_argv_first = sys.argv[0]
        self.clear()

    def assign(self, *args: Any) -> None:
        """Clear existing arguments first and append the given ones."""
        self.clear()
        self.append(*args)

    def clear(self) -> None:
        self._argv = [self._sys_argv_first]
        self._update()

    def append(self, *args: Any) -> None:
        self._argv += args
        self._update()

    def _update(self) -> None:
        self._monkeypatch.setattr(sys, "argv", self._argv)

# from cyto/tests/settings/conftest.py
class MyTunesSettings(BaseSettings):
    theme: str
    volume: int = 80
    shuffle: bool = True
    translations: Dict[str, str] = {
        "repeat": "repetir",
        "shuffle": "barajar",
    }

# from cyto/tests/settings/sources/test_source_cli.py
def test_basic_field(
    mytunes_settings: Type[MyTunesSettings],
    argv: Argv,
) -> None:

    # Set the required field
    argv.append("--theme", "dark")
    settings = mytunes_settings()
    assert settings.theme == "dark"
    # Non-required fields get their default value
    assert settings.volume == 80
    assert settings.shuffle is True
    assert settings.translations == {
        "repeat": "repetir",
        "shuffle": "barajar",
    }
....

presumably I could create a mytunes_settings arg with
mytunes_settings = autofill(name="mytunes")(MyTunesSettings)
but I haven't figured out what the 'monkeypatch' arg for Argv should be, eg how to construct argv.  Taking a stab at it, since it requires setattr that can manipulate sys.argv:
class Patch:
    def setattr(self, obj, k, v):
        setattr(obj, k, v)

monkeypatch = Patch()

mytunes_settings = autofill(name="mytunes")(MyTunesSettings)
argv = Argv(monkeypatch)
argv.append("--theme", "dark")
settings = mytunes_settings()
assert settings.theme == 'dark'

print(settings)

theme='dark' volume=80 shuffle=True translations={'repeat': 'repetir', 'shuffle': 'barajar'}

gets me started, but just a toy example would be immensely helpful!  I'll look forward to integrating cyto.

Thanks for raising this issue. :+1: Let me have a look.
It's on my to-do list to make a brief usage example. Currently, we have zero documentation, so I completely understand that you're struggling with the API.
For what it's worth, I envision something like the following high-level API will work for most users:
#!./.venv/bin/python
from typing import Dict

from cyto.app import App, Settings

class MyTunesSettings(Settings):
    theme: str
    volume: int = 80
    shuffle: bool = True
    translations: Dict[str, str] = {
        "repeat": "repetir",
        "shuffle": "barajar",
    }

async def main(settings: MyTunesSettings) -> None:
    """Run the MyTunes app."""
    # Do something with the settings
    print(settings)

App.launch(main)

If you call it like this ./_my_tunes.py --theme dark, you get the following output:
debug=False background=True data_directory=PosixPath('/var/appster') theme='dark' volume=80 shuffle=True translations={'repeat': 'repetir', 'shuffle': 'barajar'}

Of course, there is also a low-level API that gives you more control (and doesn't force you to use cyto.app.App). I'll write that up in the documentation. Hopefully, I'll get some time to work in it this weekend. :)
I hope that helps you. I'll leave this issue open until we have proper documentation for cyto. :+1:
~Frederik

Documentation would be great. I took a look at https://github.com/mpkocher/pydantic-cli and got scared off by HOW MUCH documentation there was (if the readme needs to be that long to explain how it works, it must be too complicated :) ), and I like how succint and clean this codebase looks, but just a few examples would be great.


Question: Install x86 JDK on M1 Mac

Hi all,
for a specific project I need an x86 JDK (version 11 or 13 prefered) installed on my M1 MacBookPro. More precisely I need nd4j and javacpp which are not available for Arm64 yet and maven tries to retrieve macosx-arm64 versions that are not available.
I tried the sdkman_rosetta2_compatible switch as described here (https://itnext.io/how-to-install-x86-and-arm-jdks-on-the-mac-m1-apple-silicon-using-sdkman-872a5adc050d) but the list of available java versions is the same with switch set to true and false.
Is there a x86 version? It does not show in list. And it not how can I find a x86 version to be installed via sdkman.
cheers and best
Ilja

this should work
❯ sdk install java 17.0.3.fx-zulu
java 17.0.3.fx-zulu is already installed.

❯ cat `which java` | file -
/dev/stdin: Mach-O 64-bit executable x86_64

Hi all,
for a specific project I need an x86 JDK (version 11 or 13 prefered) installed on my M1 MacBookPro. More precisely I need nd4j and javacpp which are not available for Arm64 yet and maven tries to retrieve macosx-arm64 versions that are not available.
I tried the sdkman_rosetta2_compatible switch as described here (https://itnext.io/how-to-install-x86-and-arm-jdks-on-the-mac-m1-apple-silicon-using-sdkman-872a5adc050d) but the list of available java versions is the same with switch set to true and false.
Is there a x86 version? It does not show in list. And it not how can I find a x86 version to be installed via sdkman.
cheers and best Ilja

Hello, Ilja
I has been following the same article. The reason it may not be working for you is that when SDKman! is installed locally, it stores its configuration in the user folder, starting at ~/
So try to edit the same property using sdk config command (for me it opens vim editor with actual settings loaded from user home directory (i.e. ~/.sdkman/...)

Do you open your terminal with rosetta compatibility? You should be able to download from it. I had the same case and use to download using it like that

Closing because the last comment mentions how to proceed in this case.


"Are you sure?" dialog box for deregistering

Currently, it is too easy to deregister a manually registered tool by accident

Checked into develop as part of https://github.com/ga4gh/dockstore-ui/pull/140

Works on staging.dockstore.org


Newtonsoft Json Dependency in Nuget

Just a small issue but the release in nuget doesn't allow the used of the latest version of newtonsoft json.

I would also love a fix for this! The dependency of Json just needs to include the current latest version 8.0.2


Make Ui fit new prefix

Codecov Report

Merging #270 into master will not change coverage.
The diff coverage is 0.00%.

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


feat: use salts and encryption in backend to better store passwords

closes  #387
⚠ test this pr thoroughly!!! ⚠

Code looks great, and curly still works (tested on both bob & osoc; can't test trudy afaik because her email isn't stored in normalized form in the DB).
However, small remark: you use the same code in three different files. I recommend making a small function in utility.ts to aggregate this function (so we only have to change this function in one place, and reduce possible technical debt).
In all other aspects, amazing work 🚀

what part do you see as duplicate? I only notice calculating the hash but the difference would be very minor. I would still need to call that function with 2 arguments. The only difference is that await bcrypt.hash would be replaced by await ownFunc


java.lang.IllegalStateException: Cannot remove non-existing user

Describe the bug
https://pastebin.com/raw/6QZqWKbB
To Reproduce
Steps to reproduce the behavior:

Join to the server
Sometimes the error will appear and player is kicked from a server

Server Software and Version:

Vendor: Pufferfish
Minecraft Version: 1.20.1
Output of /version command: This server is running Pufferfish version git-Pufferfish-"979d3a2" (MC: 1.20.1) (Implementing API version 1.20.1-R0.1-SNAPSHOT) (Git: 979d3a2 on HEAD)

It looks like you have another plugin (possibly ProtocolLib?) causing errors when the player joins, it disconnects them (firing quit event) when no join event ever happened. This is an invalid state that TabTPS doesn't expect.


python312Packages.aioraven: 0.6.0 -> 0.7.0

Diff: https://github.com/cottsay/aioraven/compare/refs/tags/0.6.0...0.7.0
Changelog: https://github.com/cottsay/aioraven/blob/0.7.0/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 326934 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.rainforest_raven
python311Packages.aioraven
python311Packages.aioraven.dist
python312Packages.aioraven
python312Packages.aioraven.dist


please fix kconfig warning

such as,
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/peripherals/nrf5x_sdk/Kconfig:63:warning: defaults for choice values not supported
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/misc/crclib/Kconfig:46:warning: 'CRC8_POLY': number is invalid
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/misc/crclib/Kconfig:81:warning: 'CRC16_POLY': number is invalid
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/misc/crclib/Kconfig:116:warning: 'CRC32_POLY': number is invalid
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/misc/TensorflowLiteMicro/Kconfig:32:warning: defaults for choice values not supported
D:\tools\env\tools\ConEmu\ConEmu\..\..\..\packages/packages/misc/TensorflowLiteMicro/Kconfig:36:warning: defaults for choice values not supported

This is new:
C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:33: warning: style: quotes recommended around 'ClassA' in 'bool ClassA'
C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:35: warning: style: quotes recommended around 'ClassC' in 'bool ClassC'
C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:44: warning: style: quotes recommended around 'OTAA' in 'bool OTAA'
C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:47: warning: style: quotes recommended around 'ABP' in 'bool ABP'
warning: SALOF_LOG_LEVEL (defined at C:\Develop\env\packages/packages/iot/kawaii-mqtt/Kconfig:76) has leading or trailing whitespace in its prompt
C:\Develop\env\packages/packages/system/gui_engine/Kconfig:32: warning: style: quotes recommended around '32' in 'bool 32'
C:\Develop\env\packages/packages/system/gui_engine/Kconfig:35: warning: style: quotes recommended around '24' in 'bool 24'
warning: PKG_USING_DRV_INFRARED (defined at C:\Develop\env\packages/packages/peripherals/infrared/Kconfig:24) has leading or trailing whitespace in its prompt
warning: VDEVICE_USING_PIN (defined at C:\Develop\env\packages/packages/peripherals/vdevice/Kconfig:13) has leading or trailing whitespace in its prompt
warning: VDEVICE_USING_LCD (defined at C:\Develop\env\packages/packages/peripherals/vdevice/Kconfig:24) has leading or trailing whitespace in its prompt
warning: VDEVICE_USING_TOUCH (defined at C:\Develop\env\packages/packages/peripherals/vdevice/Kconfig:34) has leading or trailing whitespace in its prompt
warning: style: quotes recommended around default value for string symbol RT_LWIP_IPADDR (defined at rt-thread/components/net/Kconfig:178)
warning: style: quotes recommended around default value for string symbol RT_LWIP_GWADDR (defined at rt-thread/components/net/Kconfig:182)
warning: style: quotes recommended around default value for string symbol RT_LWIP_MSKADDR (defined at rt-thread/components/net/Kconfig:186)
warning: style: quotes recommended around default value for string symbol DHCPD_SERVER_IP (defined at rt-thread/components/net/Kconfig:473)
warning: style: quotes recommended around default value for string symbol DHCP_DNS_SERVER_IP (defined at rt-thread/components/net/Kconfig:485)
warning: style: quotes recommended around default value for string symbol WIZ_IPADDR (defined at C:\Develop\env\packages/packages/iot/wiznet/Kconfig:58)
warning: style: quotes recommended around default value for string symbol WIZ_GWADDR (defined at C:\Develop\env\packages/packages/iot/wiznet/Kconfig:62)
warning: style: quotes recommended around default value for string symbol WIZ_MSKADDR (defined at C:\Develop\env\packages/packages/iot/wiznet/Kconfig:66)
warning: style: quotes recommended around default value for string symbol PKG_USING_ALI_IOTKIT_PRODUCT_KEY (defined at C:\Develop\env\packages/packages/iot/iot_cloud/ali-iotkit/Kconfig:15)
warning: style: quotes recommended around default value for string symbol PKG_USING_ALI_IOTKIT_PRODUCT_SECRET (defined at C:\Develop\env\packages/packages/iot/iot_cloud/ali-iotkit/Kconfig:19)
warning: style: quotes recommended around default value for string symbol PKG_USING_ALI_IOTKIT_DEVICE_SECRET (defined at C:\Develop\env\packages/packages/iot/iot_cloud/ali-iotkit/Kconfig:27)
warning: style: quotes recommended around default value for string symbol PKG_USING_TENCENT_IOT_EXPLORER_DEVICE_NAME (defined at C:\Develop\env\packages/packages/iot/iot_cloud/tencent-iot-sdk/Kconfig:18)
warning: style: quotes recommended around default value for string symbol LORAWAN_ED_STACK_PHY_PARAMETER_CHANNEL_MASK1 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:243)
warning: style: quotes recommended around default value for string symbol LORAWAN_ED_STACK_PHY_PARAMETER_CHANNEL_MASK2 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:246)
warning: style: quotes recommended around default value for string symbol LORAWAN_ED_STACK_PHY_PARAMETER_CHANNEL_MASK3 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:249)
warning: style: quotes recommended around default value for string symbol LORAWAN_ED_STACK_PHY_PARAMETER_CHANNEL_MASK4 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:252)
warning: style: quotes recommended around default value for string symbol LORAWAN_ED_STACK_PHY_PARAMETER_CHANNEL_MASK5 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:255)
warning: style: quotes recommended around default value for string symbol LORA_RADIO0_DEVICE_NAME (defined at C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:33)
warning: style: quotes recommended around default value for string symbol LORA_RADIO0_SPI_BUS_NAME (defined at C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:37)
warning: style: quotes recommended around default value for string symbol LORA_RADIO_RFSW1_PIN_NAME (defined at C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:100, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:122, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:178, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:252, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:326, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:419, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:493, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:567, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:661, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:735)
warning: the string symbol LORA_RADIO_RFSW1_PIN_NAME (defined at C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:100, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:122, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:178, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:252, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:326, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:419, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:493, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:567, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:661, C:\Develop\env\packages/packages/peripherals/lora_radio_driver/Kconfig:735) has ranges, but is not int or hex
warning: the default selection ClassA (undefined) of <choice> (defined at C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:27) is not contained in the choice
warning: the default selection ABP (undefined) of <choice> (defined at C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:38) is not contained in the choice
warning: the default selection REGION_CN470 (defined at C:\Develop\env\packages/packages/iot/lorawan_ed_stack/Kconfig:201) of <choice> (defined at C:\Develop\env\packages/packages/iot/lorawan_driver/Kconfig:50) is not contained in the choice
warning: the default selection ABUP_NETWORKING_MODE (undefined) of <choice> (defined at C:\Develop\env\packages/packages/iot/abup_fota/Kconfig:13) is not contained in the choice
warning: the default selection ABUP_ALGORITHM (undefined) of <choice> (defined at C:\Develop\env\packages/packages/iot/abup_fota/Kconfig:38) is not contained in the choice


Give tap and target developers a "Get it on Meltano" badge

Migrated from GitLab: https://gitlab.com/meltano/hub/-/issues/217
Originally created by @aaronsteers on 2022-03-22 20:30:34

https://meltano.slack.com/archives/C01PKLU5D1R/p1647416381125879?thread_ts=1647388209.417079&cid=C01PKLU5D1R
From @emcp:

I wonder aloud if there is also a “One Click Deploy” or Badge I can add in future to my tap.. like “Get it on MeltanoHub”

In theory, we could add create a default (probably commented-out) badge to the cookiecutter readmes. The developer would just need to update the link to it goes to the correct page on hub.meltano.com.
Something like this from shields.io could work:
[![](https://img.shields.io/badge/-Get%20it%20on%20Meltano-purple)](https://hub.meltano.com/taps/adwords#usage-with-meltano)

Renders as:

View 2 previous comments from the original issue on GitLab


Update mongodb driver

MongoDB.Driver v2.28.0 has been released:
https://www.mongodb.com/community/forums/t/net-driver-2-28-0-released/289745
It now provides strong-named assemblies. This pull request updates the dependencies.
The fact that Hangfire.Mongo still depends on < 2.28.0, which are not signed, causes me to not be able to update MongoDB.Driver in projects that also use Hangfire.Mongo.

Thanks!


Update landscape.yml

Updating Logos

Build failed because of:
No cached entry, and GOPACS has issues with logo: Tennet GOPACS - text.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/Tennet GOPACS - text.svg'
No cached entry, and Smart Cable Guard has issues with logo: Smart Cable Guard.DNV.GL.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/Smart Cable Guard.DNV.GL.svg'
No cached entry, and PostGIS has issues with logo: PostGIS.OSG.svg, ENOENT: no such file or directory, open '/opt/build/repo/hosted_logos/PostGIS.OSG.svg'


issue handling beam in header

I don't know what changed, but I am now unable to print the header information of a cube... and this blocks the use of pyspeckit.
Here is the example code:
In [1]: %paste
In [1]: %paste
OneOneFile = 'L1544/L1544_NH3_11_base1.fits'
import spectral_cube
from spectral_cube import SpectralCube
print(spectral_cube.__version__)
cube11sc = SpectralCube.read(OneOneFile)
cube11sc.header

## -- End pasted text --
0.4.1.dev1305
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-1-398b8ffca5a0> in <module>()
      4 print(spectral_cube.__version__)
      5 cube11sc = SpectralCube.read(OneOneFile)
----> 6 cube11sc.header

/Users/jpineda/code/python_code/spectral-cube/spectral_cube/spectral_cube.pyc in header(self)
   2216
   2217         if 'beam' in self._meta:
-> 2218             header = self._meta['beam'].attach_to_header(header)
   2219
   2220         # TODO: incorporate other relevant metadata here

/Users/jpineda/anaconda/lib/python2.7/site-packages/astropy/units/quantity.pyc in __getattr__(self, attr)
    861                 "'{0}' object has no '{1}' member".format(
    862                     self.__class__.__name__,
--> 863                     attr))
    864
    865         def get_virtual_unit_attribute():

AttributeError: 'Beam' object has no 'attach_to_header' member

which is using the latest version of Spectral Cube  '0.4.1.dev1305' and the beam information is properly loaded:
In [2]: cube11sc.beam
Out[2]: Beam: BMAJ=31.32 arcsec BMIN=31.32 arcsec BPA=0.0 deg

I've been experiencing a related issue not in pyspeckit.  I think something changed upstream in astropy; if we can find a MWE we can git bisect to identify the bad change

problem was with really old radio-beam installation (without version number)... after removing that folder then it found the newer version and the problem was solved.


color-picker-grid not working in Safari

The color-picker-grid won't work properly in the Safari browser. More specifically, when you hit the ng-focus="show()" function which turns your $scope.visible variable from true to false isn't working in Safari. Can you attempt to fix please?

Hi @CarlsBad505,
I don't have immediate access to Mac, but I will try to get a hold of one as soon as possible.

Do you work on a PC? You can download that browser for PC.
On Sep 25, 2016 3:50 PM, "Ruhley"<EMAIL_ADDRESS>wrote:

Hi @CarlsBad505 https://github.com/CarlsBad505,
I don't have immediate access to Mac, but I will try to get a hold of one
as soon as possible.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ruhley/angular-color-picker/issues/114#issuecomment-249448879,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMon7QNtpCPcBLU4W9P-PoWobfXENQySks5qtuyKgaJpZM4KEVua
.

Don't download Safari for PC.  It hasn't been developed/supported for years on PC.  Safari can only be tested on a Mac.

@CarlsBad505 Do you have a specific example (with code): maybe I could help as I have access to a Mac.

@warpdesign According to this website, it might be futile to try since it appears that color input fields are not supported in Safari 9.1, but supported on version 10. Check it out here: http://caniuse.com/#search=color input
This isn't necessarily something that's particular to my code, rather than an issue with browser compatibility. I'd suggest you create a pull request if you want to attempt to fix it! Thanks

@warpdesign Nicolas, it doesn't appear to work in Safari 10 either, unfortunately. I just tried it out.

@CarlsBad505 I got hold of a Mac Mini running OS X v10.9.5 and Safari 9.0 and everything seems to work fine for me. We may need more details of what you are running or some code with it breaking.

@ruhley yes, it seems to work, that's why I asked @CarlsBad505 for an example to reproduce the problem :)

There hasn't been an update on this issue in a while and the exact issue cannot be replicated. If you still need help then please re-open the issue.


the autocode-cli should have compiled into javascript before publishing it to the npm registry

It is too slow when run autocode command. Because everytime it compiles the coffeescript in runtime, which costs lots of time.
You should have compiled it into javascript before publishing it to the npm registry.

It seems that the lib/ folder is the compiled result from src/, but cli/ascii.coffee and cli/help.coffee need to be compiled.
I suggest removing this line require('coffee-script').register(); from bin/autocode. Write the cli files in javascript.

Fixed at adb0a9c66c4eee7ae7221bc7860f5634206c719f. Added cli/src/ and cli/lib/ to compile to JS for CLI. Removed require('coffee-script').register() and added .npmignore so cli/src/ and src/ aren't published to npm.
Thank you so much for your help and support!

You are welcome. I think the project will be a great work. Well done  :+1:


fix add lock when get visible version

Proposed changes
introduced by https://github.com/apache/doris/pull/34615
When get visible version from cache, the visible version may be updated by load, so it is necessary to add lock when get visible version.

run buildall


Add support for git submodules

When fetching rock sources using git, run git submodule update --init --recursive after cloning. When using git >= 1.8.4, also pass --depth=1 to fetch only the last commit of each submodule.

Can't we just pass --recursive to clone?

@daurnimator --recursive is only supported since git 1.6.5, and it ignores --depth (that is, git clone --recursive --depth=1 clones main repo shallowly, but not the submodules.

This can be merged only in the LuaRocks 3.0 tree when we bump up the rockspec_format version number

@hishamhm I see that this change can be breaking, but why rockspec_format must be bumped? I didn't add any rockspec options.


Need pub(readonly) access to struct BigInt and struct Sign

I'm trying to implement an Integer type that supports arbitrary precision integers (similar to Haskell's Integer). When operating on BigInt, I hope to be able to safely shrink it to Int and Int64 types by accessing struct BigInt and struct Sign to check the bound.

We might need to add to_int / to_int64 / to_uint64 methods. But exposing the struct will not be an option.

Thanks for your reply. If the above functions are added, they will perfectly solve my problem. I am looking forward to seeing this new feature.

@KCN-judu What's the semantic of the to_xxx function you want to add? Is it saturating, i.e. overflow -> INT_MAX or mod 2^xx?

overflow -> INT_MAX is better, and also need is_int()/is_int64() to check does this converting cause an overflow

such as
let a : BigInt =<PHONE_NUMBER>N<EMAIL_ADDRESS>
if a.is_int() {
 println("a can convert to Int without overflow")
 b = a.to_int()
}
else if a.is_int64(){
  println("a can convert to Int64 without overflow")
  b = a.to_int64()
}

output: a can convert to Int64 without overflow

this is my impl for Integer and to_integer for Int and Int64
///|
pub(all) enum Integer {
  IntegerInt(Int)
  IntegerInt64(Int64)
  Integer(BigInt)
}

///|
pub(open) trait Integral: Eq + Compare + Show {
  to_integer(Self) -> Integer
}

///|
pub impl Integral for Int with to_integer(self) { IntegerInt(self) }

///|
pub impl Integral for Int64 with to_integer(self) {
  if self >= @int.min_value.to_int64() && self <= @int.max_value.to_int64() {
    IntegerInt(self.to_int())
  } else {
    IntegerInt64(self)
  }
}


Any chance of getting a PDF version?

First, thank you so much, the book looks extremely useful!
The online version looks amazing, but there's one minor issue: I prefer reading books away from computer, using my E-Ink device. I can imaging that a lot of people have similar preferences.
So a PDF version of the book (or at least instructions how to make one) would be very helpful.

This should be possible, and I have created similar setups in the past – where .lagda source compiles to a website and to a PDF – but we're not currently working on this.

Ways to do this are relatively simple. You can use ready-made applications such as a Adobe Acrobat DC, which quickly and easy translates HTML pages into pdf files or even the PdfCrowd tool (https://pdfcrowd.com).

@Kwezan Those tools might work, but they'll be translating the HTML to a PDF. Since we have Markdown source for the book, and we can translate the literate Agda parts to LaTeX using agda, I feel we can do much better if we set up a script which runs the book through LaTeX.

I'd love a PDF version of this!


New API proposal_Subscription Status

What type of PR is this?
New API Proposal
What this PR does / why we need it:
ShutdownServiceStatus proposal from Open Gateway Drop 4
Which issue(s) this PR fixes:
Fixes #

@camaraproject/api-backlog_codeowners
Agreed to be merged. Needs approval from at least one codeowner.


Bugfix:  backport defensive Communicator change and revert constant change in Scheduler

Two PRs from 3.4:

add defensive coding to Communicator.cpp that removes Communicator object dependency from lambda function sent to Scheduler:  https://github.com/arangodb/arangodb/pull/7144

revert accidental change to constant in Scheduler.cpp:  https://github.com/arangodb/arangodb/pull/7208

Will start Jenkins on Monday.

http://<IP_ADDRESS>:8080/job/arangodb-matrix-pr-linux/2971/  ... running


Replace goja by otto which support Default JS function like console.log

I've changed some function to private, because those are no need to expose outside package

 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.Paul Aan seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Any particular reason to switching to otto? goja seems to be more active than otto.

Any particular reason to switching to otto? goja seems to be more active than otto.

goja doesn't support default function like console.log() I've tested and make this PR to fix that issue

Goja is significantly faster than Otto.  If console is the only issue, it can be added to goja using the following:
import (
        ...
        "github.com/dop251/goja_nodejs/console"
        "github.com/dop251/goja_nodejs/require"
)

...
        vm := goja.New()
        new(require.Registry).Enable(vm)
        console.Enable(vm)

Goja is significantly faster than Otto. If console is the only issue, it can be added to goja using the following:
import (
        ...
        "github.com/dop251/goja_nodejs/console"
        "github.com/dop251/goja_nodejs/require"
)

...
        vm := goja.New()
        new(require.Registry).Enable(vm)
        console.Enable(vm)

if its faster than otto, we can get rid of otto

@weburnit You can create a PR with what I suggested above to add console.log() support

@fm-tibco I figured out goja support ECMAScript 5 which doesn't have import statement

Yes, import is a part of ECMAScript 6, which goja doesn't support.  I guess we'll have to evaluate if import makes sense for this activity.  We could support require as an alternative to import with goja.  https://github.com/dop251/goja_nodejs


Need to know module not displaying second Interpretation

This event has two interpretations selected in Prismic: BSL and Relaxed. https://wellcomecollection.org/events/W498dB4AACEAph-V
However, it's only rendering the first interpretation (BSL)

The event is tonight, so no rush, but it will come up again so will need to be fixed.

Having added Relaxed Performance as a second interpretation, it's now showing a second RP label on the card instead

I've now removed the second RP interpretation, just to get rid of the redundant label.


timestamp undefined

I implemented the fabric locally and I did not see any errors.
My member made purchases and redeemed points but the transaction data is incorrect.
For example:
Points Earned Transactions

timeStamp: undefined
partner: Villanova01
member: 123456
points: 30
transactionName: undefined
transactionID: undefined

timeStamp: undefined
partner: Villanova01
member: 123456
points: 80
transactionName: undefined
transactionID: undefined

and
Points Redeemed Transactions

timeStamp: undefined
partner: Villanova01
member: 123456
points: 50
transactionName: undefined
transactionID: undefined

Appreciate suggestions on how to fix.

Hi @jsolderitsch , thanks for pointing out this bug.
It is now fixed, and the data should be able to find its way back in the UI.


Every BLEDescriptor allocates 600 bytes of memory. Is this needed?

On construction of a BLEDescriptor, it allocates 600 bytes for m_value.
I think this is quite a lot, considering most values will be a few bytes (<20) at most in my project
Am I missing something here?
Can I just reduce the #define used for this?

@Bascy Like i answered in other topic, its not this library settings that makes descriptor 600 bytes value length. Its esp-idf bt stack.


Add the name of the page that is absent in the context description of…

… the workflow query whether it is supression or approve
Check these before submitting new pull requests

[ ] I read the guidelines for contributing linked above

[ ] PHP-only files follow our coding style; in order to do that use php-cs-fixer - http://cs.sensiolabs.org/ - as follows:
php-cs-fixer fix --config=<webroot>/.php_cs.dist <filename>

If all the above conditions are met, feel free to delete this whole message and to submit your pull request, and... Thank you, your help is really appreciated!

This looks fine to me, could you change these lines to match PSR2 code style? All you'd need to do is add a space after the commas, IE
Change
t("Page %s Version %s Submitted for Approval.",$c->getCollectionName(), $this->cvID);

to
t("Page %s Version %s Submitted for Approval.", $c->getCollectionName(), $this->cvID);

Done space added

Shouldn’t we already be on the page in question? Do we need its name there?
On Fri, Oct 6, 2017 at 11:01 PM Faker Ben Ali<EMAIL_ADDRESS>wrote:

Done space added
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/6022#issuecomment-334913036,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAgNwR2vZAxauyvUw521rzxX9QX6ZWnUks5spxOdgaJpZM4PwOJH
.

When you see the message in dashboard notification page waiting for approve you will get message notification "page version 1  wiating for approve " !! This because the data saved don t contain page name .try to make workflow and submit page to workflow you will get this message notification ...

could you change these lines to match PSR2 code style?

Tests wouldn't have passed with https://github.com/concrete5/concrete5/pull/5971 :wink:

@aembler we got a site with thousands of pages, showing this message to approvers in mails and in the "waiting for me" page is very helpful

The order of the placeholders in the format string added

Awesome, thanks for making those changes! I'll leave this one up to @aembler

This makes sense @faker-ben-ali – I think I'm going to solve this a different way. We shouldn't be using the in context description in the workflow list - we should be using the regular description. But there are some other things that are wrong as well; I will do a pass on them and open a new pull request today.


docs: improved instructions to install xcode in macOS

A simple alteration to help macOS users to install Xcode command line.

@cla-bot check


implement side effect api

Pull Request Test Coverage Report for Build 45

1 of 8 (12.5%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-19.4%) to 80.556%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/index.js
1
8
12.5%

Totals

Change from base Build 31:
-19.4%

Covered Lines:
20

Relevant Lines:
27

💛  - Coveralls

action overwrite, so it this solution is impossible


Using Docker to build for distribution - question and suggestion

I notice that you have a Dockerfile that builds Supergiant binaries: https://github.com/supergiant/supergiant/blob/master/build/Dockerfile.build. The products in dist/bin appear to work in Alpine but not Ubuntu. What is the purpose of these builds as it doesn't look like they're for general use?
As for the packaging for general distrubtion (as seen in https://github.com/supergiant/supergiant/blob/master/.travis.yml): it would be great if the builds were done in Docker, opposed to being coupled to Travis CI.

Hi, @colin-nolan, the Dockerfile is an old contribution to the project, and we would be interested in knowing how you think it should be set up. There are probably some things we need to clean up in there, we're open to your suggestions.

I'm interested in getting copies of the binaries through the same process as you guys use for distribution, without having to install/configure anything on the build machine (as it will undoubtly get configured differently). Docker therefore seems like a great idea.
The build process could happen as part of the Docker build or on container run + source mount (I've not read up on what best practice is). The ability to define a subset of the platforms to build for would be a nice to have.

1.0 uses a golang container now for a correct build on multiple platform types. 2.0 will follow this same process.


[libvpx] fix cmake target for linux

On x64-linux, the debug libraries are called libvpx.a instead of libvpxd.a - this patch fixes the unofficial cmake target to reflect that. In addition, the upstream vpx build script also installs libvpx_g.a as a duplicate of libvpx.a and this patch removes that duplicate file.

Which triplets are supported/not supported? Have you updated the CI baseline? No changes.

Does your PR follow the maintainer guide? Yes.

LGTM, thanks for the fix!

Thanks for merging!


Changing CheckBox -> CheckButton

Since other button widgets are all *Button, I believe making this change will improve consistency in naming conventions.

Probably it makes sense. But would create incompatibility.
Working solution - create an alias.

Why not make it built in and add a depreciation message for users to switch before its too late.

The reason for loose naming is you never know what the parent class might be in future. Imagine an intermediate "State" class is added for all state-like widgets like the checkbox (silly example). Would it be worth to rename it?
It's better name things according to their purpose not implementation.
Technically "checkbox" seems to be correct: https://en.wikipedia.org/wiki/Checkbox
This quickly becomes a question of preference:
fltk/gtk/... ~ checkbutton
winform/qt/... ~ checkbox

In many other GUI libs this control is called CheckBox, too
Probably, we can leave its name as is

I prefer CheckBox. Closing.


Deprecate: getElement returns null

This moves us into the direction of more consistency with wd, making it easier to switch back-and-forth. Also can help cleaning up some tests that want to chain method calls onto getElement without forcing them to remember the proper null checks.

:+1:

The deprecate module has some weird interactions with strict mode. Switching to util.deprecate.


Major adjustments in the text.

Major adjustments in the text which change meaning.

 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.

@kri99 - haha good catches! I can merge this in when you sign the CLA

Hello @kri99,Thanks again for the PR! We want to show our appreciation by sending                                              you some swag. Could you please fill out                                              this form so we can send it to you? Thanks!                                             Team SendGrid DX


Trouble running pointnet2_classification script

❓ Questions & Help
Hi
First of all, I'm excited to work with this project in relation to my own data. Before I attempt that though, I'd like to run a couple of examples, but when I try the pointnet2 classification script I get the following error:
FileExistsError: [WinError 183] Cannot create a file when that file already exists: 'C:\Program Files\Anaconda3\envs\pointcnn\pointdemo\data\ModelNet10\ModelNet10' -> 'C:\Program Files\Anaconda3\envs\pointcnn\pointdemo\data\ModelNet10\raw'
Do you know of any possible solution to this problem? I have been able to run the gcn.py example, so I am not exactly sure why this fails while extracting the downloaded modelnet10 data.

This seems Windows related :( I fixed it in the master branch. If you do not want to re-install PyG, you can manually download the ModelNet10 data and move it to the raw/ directory.

Thank you very much for the help. It worked perfectly. I do have a more general question, that I hope you might be able to help with.
Having run the Pointnet++ classification example on some input data, i would like to view the classified result as a point cloud with assigned labels. Can you provide some guidelines for how I might go about doing that?
My current understanding would be, that a library like Open3d would be needed for visualization, but that PyG could classify the point cloud using Pointnet++. Any help you can provide as to how I can obtain a classified pointcloud with PyG would be very helpful.

You can convert your point clouds and the computed label to, e.g., numpy, and use tools like Open3d to visualize them.:
pcd = o3d.geometry.PointCloud()
pcd.points = o3d.utility.Vector3dVector(xyz)

Thanks for the quick response. I will look into it :)


shim for go1.11 is named go1.11.0 which does not match binary own opinion on its version

while it feels like bug in go, to omit .0 from version, I believe you should match version numbers exactly. Integration tests that put version into .go-version and try to parse that version back from go version output do fail due to this mismatch.
goenv install 1.11
go-build: definition not found: 1.11

The following versions contain `1.11' in the name:
  1.11.0
  1.11beta2
  1.11beta3
  1.11rc1
  1.11rc2

echo '1.11.0' > .go-version
go version
go version go1.11 linux/amd64

I understand, however keeping it correct according to semver is more important to me.
I actually think we should've had 1.11.0{beta2,beta3,rc1,rc2}` to be consistent.

Didn't mean to close issue w/o consensus.
Since I'm refreshing the test suite in #62 I'll check if we can "hack" around matching 1.11 also a 1.11.0 and/or latest patch version.

Addressed in https://github.com/syndbg/goenv/releases/tag/2.0.0beta9 :tada:


Basic parsing support for feature packages

@danielshaw42,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

Merged.


Fix failing builds on Windows

Coverage remained the same at 90.0% when pulling f612de8daf690b99764008be9b384ab8e7b4301c on fix-windows-build into 2153697fc6fdd2e5fb10792c4f3a9858c86203e6 on master.


airflow-upgrade-db job hangs

Often times when the metadatabase is not ready yet, the initial db check up job hangs and fails to initialize. This results in deployments taking longer due to the fact that it takes time until the pod times out. I find from my experience that using the official Airflow CLI command often time hangs instead of a proper while loop that pokes the database for readiness. It would be beneficial to swap this one out with a normal bash "until" to poke the postgres db.
It's pretty frustrating having all those crash loopbacks.

@davido912 is it specifically the airflow db check part which hangs, as we could just remove that init-container from the upgrade-db job.
It was always something I was thinking about, as clearly the airflow db upgrade part effectively checks if the external DB is up.

@davido912 I think a simple solution is using the linux timeout command in front of the airflow db check

@davido912 this should be fixed after version version 8.0.8 of the chart, as the airflow db check will now timeout after 16 seconds.
Can you confirm?


Add dockerhierarchy parser

This will parse a subfolder of Dockerfile, and add new subdirectories corresponding to tags, each with a Dockerfile based on the last tag known/generated.

This was added with https://github.com/vsoch/uptodate/commit/c17248d6dedf182fdc4cc8ffecf025e83e59c2f5. I'll make tweaks to it after creating the action and testing.


OpenDRIVE data never retrieved when it is the only sensor.

I wanted to check out the opendrive_map sensor but never got any input_data; not setting up other sensors for this purpose.
I tracked this bug down to this line of code that always aborts the data retrieval:
https://github.com/carla-simulator/leaderboard/blob/111a48f9099c08a2f1068ee8aea2ad56ce52ef9d/leaderboard/envs/sensor_interface.py#L230-L232
With only an opendrive_map sensor this is equal to 1 == 0+1 and always breaks -> returns empty dict.
Shortly also noted in #50

@Daraan That does seem to be the case. We'll fix it ASAP


[Feature Request] Support OAuth2 Device code authorisation flow

I would like httpyac to support the OAuth 2.0 Device authorization grant for the openid variable replacer.
Ref. RFC8628
Example of usage: Microsoft Identity platform
Proposal for httpyac:
GET https://graph.microsoft.com/v1.0/me
Authorization: openid device_code msidp

Reason:
This is a great way of authenticating using applications where you cannot/don't want to modify the redirect Url, and don't have access to/don't want to leak the client secret. And contrary to the implicit and hybrid authorization flows, the device code flow hasn't been deprecated by the OAuth standard and best practices.

I would be happy to add the Device Authorization Flow. But I would have to implement this more or less according to the spec. If the Postman examples from Microsoft Identity Server are executable I can test it, otherwise I may need your help.
Is it technically possible to open the verification_uri directly in the browser or does a real device change have to take place? In that case, a QR code might make sense.

And thanks for the site. Here is also PKCE for Authorization_Code Flow described:-)

Yeah, I scripted it manually with on request script hooks. Device code auth is how I usually authenticate with powershell and console apps. So you don't have to have an actual device change.

Nice. Then I would open a Notificatian with the user_code and verification:_uri and a button, which then opens the verifcation_uri.

Can you add another button to copy the user_code to the clipboard?

Unfortunately, I have not been able to fully test the Device_Code Flow. The request after device_code and the user_code works. It also opens the web page and the user_code is in the clipboard. But unfortunately I don't have a valid account for Microsoft and so I can only test the wait loop until expired_token. But I do not have a successful call yet. I would ask you, once I have created the version, to test this case and give me feedback.

Sure, I'd be happy to help!

A code review would be nice. You implemented it already and may spot a mistake.

Sure, while scripting and integrating my scripts with httpyac, I have already familiarised myself with the code base and I am comfortable with reading TypeScript. Just point me to a publicly accessible branch and I'll take a look and test during the weekend or early next week.

I did not make an extra branch. It is about the commits attached to the issue, or actually about the class deviceCodeFlow. This has the task to map the flow and return either false or the AccessToken. I am not sure if I should rather check for the presence of authorization_pending, or rather for the error status. Microsoft has an additional error bad_verification_code which is not described in the RFC. I'd rather wait too long unnecessarily before aborting at a valid status unknown to me.

Yeah I usually just check for auth pending error and would display all other errors to the user. I see you inlcuded the slow down error handling as well. Maybe call showNote or showWarning with the error message and continue polling or cancel based on user input

@AnWeber I think this looks great! I tested it with Microsoft Identity platform and it works like a charm! As expected it keeps the session and even picks up the refresh_token if you specify the offline_access scope.
I also managed to recompile the VS Code extension with the new code and tested the UI features of the flow. Everything checks out fine! 👍

Super. Thanks for the effort in setting up the project locally. I have released version 4.6.0 which includes the change.


vendored library qtls revision doesn't exist

The vendor.json pins the library  https://github.com/marten-seemann/qtls@061d608f3d22ea089c4c9039e1216eea999b9341 here
This commit doesn't exist anymore so I am currently having some trouble pinning my gomodules to these revisions.

That's exactly the reason why we commit the code in the vendor directory.

The purpose of the vendor directory is so that you can keep the code working as upstream moves master along. That all makes sense to me.
In this case you have code vendored to a SHA1 that doesn't exist. It makes it really hard to audit the code/be sure what is actually going on. Since you have pinned to a SHA1 that doesn't exist you could make any changes to the code you wanted and no one would ever know, that seems like a nice way for someone to slip in a backdoor.

The code corresponding to the hash in the vendor directory, so I don’t see how this could be used to introduce a backdoor. When you clone the repository, you get all the code including dependencies.

Does that hash verify anything more then the validity of the contents of the directory? I am not worried about someone making changes after the initial commit of vendoring qtls.
If the SHA1 no longer exists in the qtls repo how can I confirm what you actually copied from the qtls repository? You could have copied and then made a minor change and I would never know. If that SHA1 exists and all the files match I can be sure you actually vendored what you said.

You're right, vendoring only copies the files, not the git history. So the hash is basically worthless.

@marten-seemann the real problem here is that this completely contradicts to how go modules work. Go modules do not use vendor folder when this project is imported as a dependency. So go modules tries to grab stuff from the actual repo which no longer has the right hashes. And since go modules is the future of golang I would raise that it is a real issue. I would recommend to not delete any commits from public dependency libraries going forward.
This is also a definite compatibility problem for #1547 #1548 #1620 because now that old commits are missing from github.com/marten-seemann/qtls, anyone who is dependent on older version of quic will suffer from go modules spitting issues of not being able to find the right dependencies. Really not sure even how to help here.


Investigate instability of test_wasm_end_to_end_matching_engine.

In contrast to other end to end tests, the matching engine test is failing for several instances. It is possibly because of the larger size of the instance or because of another bug.
For service_grpc (a proposed service in a PR under consideration) we obtain Error, message length too large: found 4742535 bytes, the limit is: 4194304 bytes.
For aws_grpc we have a timeout error.

The problem for service_grpc has been addressed by PR https://github.com/linera-io/linera-protocol/pull/1788 .


[Bug]: particles.vue3 Breaks Vite HMR

Contact Details
No response
What happened?
If I use Particles on the app instance the HMR for Vite breaks. Vite continues to work correctly I can see the hot updates in the command line and the console, but the webpage no longer updates. I have traced it down directly to commenting out app.use(Particles);.
import { createApp } from "vue";
import Particles from "particles.vue3";

const app = createApp(App);
app.use(Particles);

"tsparticles": "^2.1.3",
"particles.vue3": "^2.1.3",
"vite": "^2.9.12",
"@vitejs/plugin-vue": "^2.3.3",
"vue": "^3.1.0",
tsParticles Version
2.1.3
Which library are you using?
Vue.js 3.x (particles.vue3)
tsParticles Configuration
{
    "particles": {
      "number": {
        "value": 150,
        "density": {
          "enable": true,
          "value_area": 1500
        }
      },
      "color": {
        "value": "#66b3ff"
      },
      "shape": {
        "type": "circle"
      },
      "opacity": {
        "value": 0.3,
        "random": false,
        "anim": {
          "enable": false,
          "speed": 1,
          "opacity_min": 0.1,
          "sync": false
        }
      },
      "size": {
        "value": 3,
        "random": true,
        "anim": {
          "enable": false,
          "speed": 40,
          "size_min": 0.1,
          "sync": false
        }
      },
      "line_linked": {
        "enable": true,
        "distance": 150,
        "color": "#66b3ff",
        "opacity": 0.3,
        "width": 1
      },
      "move": {
        "enable": true,
        "speed": 1,
        "direction": "none",
        "random": false,
        "straight": false,
        "out_mode": "out",
        "bounce": false,
        "attract": {
          "enable": false,
          "rotateX": 600,
          "rotateY": 1200
        }
      }
    },
    "interactivity": {
      "detect_on": "canvas",
      "events": {
        "onhover": {
          "enable": true,
          "mode": "grab"
        },
        "onclick": {
          "enable": true,
          "mode": "push"
        },
        "resize": true
      },
      "modes": {
        "grab": {
          "distance": 140,
          "line_linked": {
            "opacity": 1
          }
        },
        "push": {
          "particles_nb": 4
        }
      }
    },
    "retina_detect": true
  }

What browsers are you seeing the problem on?
Firefox, Chrome
Relevant log output
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

This is still broken on<EMAIL_ADDRESS>

I need a full project with the issue, I've never used Vite HMR.

@matteobruni I put together a small project but the issue is not occuring, so it seems to be related to my specific repo. Thanks for the time, I will look into my repo.


Unable to run cross binary due to error invalid toolchain name: 'usr'

When I try to run the following, in my Docker container:
RUST_BACKTRACE=1 QEMU_STRACE=1 /root/.cargo/bin/cross run && \
    --target aarch64-unknown-linux-gnu --release --features final

Where the Docker container is running:
$ uname -a
Linux 6b42705e6f67 4.9.184-linuxkit #1 SMP Tue Jul 2 22:58:16 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Where I have rustup installed as follows:
root@6b42705e6f67:/open-ethereum# rustup toolchain list
stable-x86_64-unknown-linux-gnu
nightly-x86_64-unknown-linux-gnu (default)

Error is shown as follows. Note that I get the same error if I switch from Nightly to Stable too:
$ /root/.cargo/bin/cross build

error: invalid toolchain name: 'usr'
error: couldn't install toolchain `usr`
caused by: `"rustup" "toolchain" "add" "usr"` failed with exit code: Some(1)
note: run with `RUST_BACKTRACE=1` for a backtrace

Detailed error log is:
root@6b42705e6f67:/open-ethereum# RUST_BACKTRACE=1 QEMU_STRACE=1 /root/.cargo/bin/cross build --target aarch64-unknown-linux-gnu --release --features final

error: invalid toolchain name: 'usr'
error: backtrace:
error: stack backtrace:
   0: error_chain::backtrace::imp::InternalBacktrace::new
   1: rustup_init::rustup_mode::update
   2: rustup_init::rustup_mode::main
   3: rustup_init::run_rustup_inner
   4: rustup_init::main
   5: std::rt::lang_start::{{closure}}
   6: main
   7: __libc_start_main
   8: <unknown>

error: couldn't install toolchain `usr`
caused by: `"rustup" "toolchain" "add" "usr"` failed with exit code: Some(1)
   0: error_chain::backtrace::imp::InternalBacktrace::new
   1: <std::process::Command as cross::extensions::CommandExt>::status_result
   2: cross::main
   3: std::rt::lang_start::{{closure}}
   4: main
   5: __libc_start_main
   6: _start

So you are running cross itself inside a Docker container? I'm afraid that use case is unsupported or at least untested at the moment. Try if running with -v gives you any more insight.

If I run cross -v it returns:
root@6b42705e6f67:/open-ethereum# cross -v
+ "rustc" "--print" "sysroot"
+ "rustup" "toolchain" "list"
+ "rustup" "toolchain" "add" "usr"
error: invalid toolchain name: 'usr'
error: couldn't install toolchain `usr`
caused by: `"rustup" "toolchain" "add" "usr"` failed with exit code: Some(1)
note: run with `RUST_BACKTRACE=1` for a backtrace

What does rustc --print sysroot output?

root@6b42705e6f67:/open-ethereum# rustc --print sysroot
/usr

That's the problem then. The sysroot should be somewhere in RUSTUP_HOME I believe.

Thanks, I'll see if I can fix that

I resolved it by running source $HOME/.cargo/env,
So now when I run rustc --print sysroot it outputs:
/root/.rustup/toolchains/stable-x86_64-unknown-linux-gnu instead of /usr

So we can close this issue, and I've opened another separate issue here https://github.com/rust-embedded/cross/issues/385 with the error that I'm getting now that cross binary is working within my Docker container. Thanks for your help @reitermarkus
FYI, the contents of the Dockerfile that I was using is:
FROM rustembedded/cross:mips-unknown-linux-gnu

WORKDIR /open-ethereum
COPY . /open-ethereum
RUN dpkg --add-architecture mips-unknown-linux-gnu && \
  apt-get update && \
  apt-get install -y git vim cargo sudo && \
  curl https://sh.rustup.rs -sSf | sh -s -- -y && \
  export PATH=$PATH:/root/.cargo/bin && \
  rustup toolchain install nightly && \
  rustup default nightly && \
  rustup toolchain list && \
  rustup run nightly cargo --version && \
  which rustup && \
  rustup run nightly cargo install cross && \
  uname -a


php artisan blade:validate leaves bunch of empty temp_* directories when PHPStan is enabled

It's hard reproduce the issue, but if Larastan is installed and PHPStan options are enabled, empty temporary directories are not being cleaned up after the validate command.
rmdir temp_* is a workaround for the issue but it obviously is not the intended behavior as validate command was executed successfully (returns 0).  Would you mind taking a look if you have the time?
When I turn off PHPStan integration, temporary directories will be removed as I expected.
Thanks
versions
nunomaduro/larastan: v2.9.1
stillat/blade-parser: v1.7.0

phpstan.neon
includes:
    - ./vendor/nunomaduro/larastan/extension.neon

parameters:
    paths:
        - app

    ignoreErrors:
        - '#Call to an undefined static method Illuminate\\Support\\Facades\\DB::getSequence\(\).#'
        - '#Call to an undefined method Monolog\\Handler\\HandlerInterface::setFormatter\(\).#'
        - '#Call to an undefined method Monolog\\Handler\\HandlerInterface::pushProcessor\(\).#'
        - '#Do-while loop condition is always false.#'

    level: 5

Do you have other dependencies, like Livewire by chance? I'll try and sort this out this upcoming weekend

@JohnathonKoster No, I don't.
Reproduce process:
$ composer --version
Composer version 2.7.1 2024-02-09 15:26:28
$ cat /etc/redhat-release
AlmaLinux release 8.7 (Stone Smilodon)
$ composer create-project laravel/laravel example-app
$ cd example-app
$ composer require larastan/larastan:^2.0 --dev
$ composer require stillat/blade-parser --dev
$ php artisan vendor:publish --tag=blade
$ php artisan blade:validate -vvv
Running 20 validators against 1 files; PHPStan document validation enabled.
/resources/views/welcome.blade.php
[BLADE_V011] Inconsistent casing for [@endauth]; expecting [@endAuth] on line 30
Analyzed 1 files in 00:00:01 (Blade Parser: 00:00:00 PHPStan: 00:00:01)
$ ls -ald temp_*
drwxr-xr-x 2 kojima staff 6  3月  1 11:56 temp_f1180b69d9d773c79bc55e93106cab64434c436d.php
$ test -z "$(ls -A temp_f1180b69d9d773c79bc55e93106cab64434c436d.php)"; echo $?
0

Thanks for the additional info and steps!

Thanks for the PR. Its very much appreciated! I've released that as 1.7.1


dockerfilejson templating empty, not rendered

Hello,
I am running ESO 0.7.1 and I have the following manifest:
apiVersion: external-secrets.io/v1beta1
kind: ExternalSecret
metadata:
  name: some-secret
  namespace: some-namespace
spec:
  refreshInterval: 1h
  secretStoreRef:
    kind: ClusterSecretStore
    name: aws
  target:
    name: secret-name-inside-k8s
    template:
      type: kubernetes.io/dockerconfigjson
      data:
        .dockerconfigjson: |
          {"auths": {
            "{{ .url }}": {
              "username": "{{ .user }}",
              "password": "{{ .password }}",
              "email": "{{ .email }}",
              "auth": "{{ .auth }}"
            }
          }}
  data:
    - secretKey: url
      remoteRef:
        key: aws-secrets-manager-secret
        property: url
    - secretKey: user
      remoteRef:
        key: aws-secrets-manager-secret
        property: user
    - secretKey: password
      remoteRef:
        key: aws-secrets-manager-secret
        property: password
    - secretKey: email
      remoteRef:
        key: aws-secrets-manager-secret
        property: email
    - secretKey: auth
      remoteRef:
        key: aws-secrets-manager-secret
        property: auth

AWS Secrets Manager content:

And this this the "rendered" data:
{"auths": {
  "": {
    "username": "",
    "password": "",
    "email": "",
    "auth": ""
  }
}}

Also, this problem is rather inconsistent (sometimes it happens, sometimes it doesn't, but mostly it does.).
I have tried reinstalling secrets manager, deleting es and secret, etc.
Any insight here?

Can you create a another es without the template block and see if variables are loaded ok there?

Yup that has been tested also, and the variables show up. Also, sometimes it also works with the template, sometimes it doesn't. Pretty inconsistent.

It seems to me that this issue might be related to some inconsistency in fetching the secrets 🤔 when using templates, then that produces a blank one not sure why.
Does this happen in older ESO versions?

Yes, it happens in all the versions. Unfortunately I will have to create those secrets in terraform.

Can you send the logs of your ESO deployment? Maybe there’s an error track there that we can chase on

Small aside: The manifest OP posted here was really useful for getting my own dockerconfigjson setup. This would be a nice example to add to the official site as another dockerconfigjson example

I get the same issue when using Hashicorp Vault as a backend and i am on v0.8.1


Add InteractiveCategoryPlot class

interactive plotting as user selects filename/categories from widgets in jupyter notebooks
add percent_matrix_heatmap as the default plotting callback
add a demo notebook using it

Since the correlation code will change in #15 I'll wait until that is merged and then rebase this


Can't build Nibegw Firmware for ESPHome 2023.10.0 or 2023.10.1

I've been happily using Nibegw on my LILYGO T-CAN485 for about 6 months. However, I am unable to update my Nibegw firmware since I updated Home Assistant to ESPHome 2023.10.0. It looks like it has something to do with my (non-existant) IPv4 configuration, but I would very much prefer my LILYGO board to get its IP from my router.
Top of the install log:
INFO ESPHome 2023.10.1
INFO Reading configuration /config/esphome/nibegw.yaml...
INFO Generating C++ source...
INFO Compiling app...
Processing nibegw (board: esp32dev; framework: arduino; platform<EMAIL_ADDRESS>--------------------------------------------------------------------------------
HARDWARE: ESP32 240MHz, 320KB RAM, 4MB Flash
 - toolchain-xtensa-esp32 @ 8.4.0+2021r2-patch5
Dependency Graph
|-- WiFi @ 2.0.0
|-- ESPmDNS @ 2.0.0
|-- Update @ 2.0.0
|-- ESP32 Async UDP @ 2.0.0
Compiling .pioenvs/nibegw/src/esphome/components/nibegw/NibeGwComponent.cpp.o
Compiling .pioenvs/nibegw/src/esphome/components/socket/lwip_raw_tcp_impl.cpp.o
Compiling .pioenvs/nibegw/src/esphome/components/socket/lwip_sockets_impl.cpp.o
src/esphome/components/nibegw/NibeGwComponent.cpp: In member function 'void NibeGwComponent::callback_msg_received(const byte*, int)':
src/esphome/components/nibegw/NibeGwComponent.cpp:33:72: error: invalid cast from type 'std::__tuple_element_t<0, std::tuple<esphome::network::IPAddress, int> >' {aka 'esphome::network::IPAddress'} to type 'uint32_t' {aka 'unsigned int'}
         if (!udp_read_.writeTo(data, len, (uint32_t)std::get<0>(*target), std::get<1>(*target))) {
                                                                        ^
src/esphome/components/nibegw/NibeGwComponent.cpp: In member function 'void NibeGwComponent::token_request_cache(AsyncUDPPacket&, byte, byte)':
src/esphome/components/nibegw/NibeGwComponent.cpp:59:52: error: conversion from 'uint32_t' {aka 'unsigned int'} to 'esphome::network::IPAddress' is ambiguous
     network::IPAddress ip = (uint32_t)udp.remoteIP();
                                                    ^

logs_nibegw_run.txt

Fixed yesterday. Get a new version of the component


📝 Fix documentation CI?

We update docs on publish
But this job fails:
      - name: "Update Doc"
        uses: peter-evans/repository-dispatch@v2
        with:
          event-type: doc-build

Probably due to a policy change in the org cc @glegendre01  (https://huggingface.slack.com/archives/C02SPHC1KD1/p1722448474979249?thread_ts=1722339176.798599&cid=C02SPHC1KD1)
With the error message: Error: Resource not accessible by integration
eg https://github.com/huggingface/huggingface.js/actions/runs/10736074066/job/29774687075 cc @merveenoyan :)
It doesn't stop publishing thank god, but it'd be nice to avoid errors
cc @Wauplin too

@coyotte508 thanks a lot, I ran the workflow again

@coyotte508 issue persists (I think there's an issue related to a token or something) https://github.com/huggingface/huggingface.js/actions/runs/10736074066


Changes for nightly support

What does this PR do?

Renames cli:latestrc:build to cli:release:build (adds alias)
Allows you to use any npm dist-tag, semver version, branch, or commit ref as a starting point for the release
Adds support for prereleases
Replaces hardcoded references to latest-rc
renames flags to be sf-ified (adds and deprecates aliases)

What needs to be changed elsewhere:

Update the makePrForRc.yml in sfdx and sf to pass in --start-from-npm-dist-tag latest-rc
Rename command from cli:latestrc:build to cli:release:build everywhere
I changed the title of the PR, we need to update rcTagAndRelease.yml that is watching for a very specific string

What issues does this PR fix or reference?
@W-12184428@

Too many changes in some of these files. Github is showing delete/add instead of a file moved. Pull branch locally and compare changes with git diff main HEAD -M10 to change the change threshold

-M[]
--find-renames[=]
Detect renames. If n is specified, it is a threshold on the similarity index (i.e. amount of addition/deletions compared to the file’s size). For example, -M90% means Git should consider a delete/add pair to be a rename if more than 90% of the file hasn’t changed. Without a % sign, the number is to be read as a fraction, with a decimal point before it. I.e., -M5 becomes 0.5, and is thus the same as -M50%. Similarly, -M05 is the same as -M5%. To limit detection to exact renames, use -M100%. The default similarity index is 50%.

@WillieRuemmele I added aliases where needed. But there will be a breaking change because exactlyOne does not work well with defaults. I had to remove the latest-rc default on the rcTag (< old name) flag. I did open two PRs for sf and sfdx to change this though.

QA notes: (testing from main on sfdx-cli)
❓  did you plan for this to be a breaking change?  It should major-bump pjson version, right?
 ../plugin-release-management/bin/dev cli:latestrc:build --build-only --start-from-npm-dist-tag latest-rc
✅  found lastest-rc at 7.182.1
✅  bumped to 7.183.0
Things hangs every time on yarn:install for both the latest and previous (shelljs) commit


Add Bootstrap Responsive Web Development

What does this PR do?
Add Bootstrap Responsive Web Development by Jake Spurlock
For resources
https://pepa.holla.cz/wp-content/uploads/2015/10/Bootstrap.pdf
Description
If you have some background in writing HTML/CSS and JavaScript, this book will help you get off the ground writing some flexible code for responsive websites - From the Author.
Why is this valuable (or not)?
Get familiar with bootstrap file structure, grid systems, navigation, jQuery, breadcrumbs and modal.
How do we know it's really free?
It's readily available for download in the link provided.
For book lists, is it a book? For course lists, is it a course? etc.
It is a book.
Checklist:

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

Follow-up
No warnings, able to merge!

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

This appears not to be an authorized distribution. Please see docs/CONTRIBUTING.md guidelines.

Okay. Thanks


Added TextStyle to CommandCards

I added TestStyle to the CommandCard text to enable the developer to use other background colors that won't interfere with the text. Also, the text is quite small so using TextStyle the developer is free to style the text the way he wants with Size, Weight, Color...

hello @radikris, its been a while, do you think you can push those changes so i can use the features i have implemented please?

@mazenhalawi Hi, sorry for my delay, I am back, and alright, it's merged


Updated to Sierra this morning on MBA mid 2011, prior to updating CAT was working flawlessly, but now i cant get it to work with my iphone6 with ios10, im not sure if its an ios10 issue now as I only updated to that this morning also.

Have you tried these steps?
Is there already an open topic for this issue?
Please include the following information.

OS Version:
Mac model:
Dongle or replaced card?:
Used dongle/card:

Your Issue:
Steps done to try to fix the issue:

Ive tried numerous times over the weekend, tried uninstalling it, and reinstalling but to no avail, Im on very same software versions as you are, also have macbook air mid 2011 and iphone 6 running 10.0.1. I was able to confirm that its not an ios10 issue as I uninstalled sierra and reverted back to El capitan and handoff worked directly from my back up despite my phone now been on ios10.

"Has anyone here gotten it to work fully on their mid-2011 MBA with Sierra GM & iOS 10 GM?"
I have on a 2011 mac mini which has the same card as the 2011 MBA, so it definitely should work.  Are they both on the same iCloud account?  Also i remember someone once reporting a similar issue where creating a brand new Mac user account fixed the problem for them so might be worth signing out of icloud on the Mac, create a new admin user, then log back in again.

@digger99 Ah, so you think it's a new issue with Sierra that the tool needs to be updated for?
@durosity Thank you! Yes, both devices are on the the same iCloud account. You're having success with pairing the Sierra GM & iOS 10 GM with your Mac Mini set up? With Handoff, phone calls, clipboard and everything?

@miguelmanalo Well, thats what it looks like to me anyway, but Im open to trying out other things if others have alternative solutions for me to try

@digger99 Your conclusion makes sense to me, thank you! I'll keep my fingers crossed that after Sierra & iOS 10 are officially released this tool still can work for us with updates.

@miguelmanalo Yeah everything seems to work fine except watch unlocking, i suspect due to the 'time of flight' requirements.  So it's a bit strange that it's not working on your MBA.. guess there must be a minute difference between them that's causing the issue.

@durosity @digger99 OMG I did something weird and now Handoff is working!! https://imgur.com/a/CdzjU
Clipboard sharing, Safari/iMessage handoff!
What I did is:

Settings > iCloud > iCloud Drive > Toggle Off > Toggle On > Stuff works

Have the same issue. MBA mid 2011. MacSierra 10.12 and iOS 10.0.1 iPhone 6s. The Toggle didn't do the trick for me.

Toggle worked for me..MBP early 2011 with BC card.

On 21 Sep 2016, at 09:57, Alecnash<EMAIL_ADDRESS>wrote:
Have the same issue. MBA mid 2011. MacSierra 10.12 and iOS 10.0.1 iPhone 6s. The Toggle didn't do the trick for me.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/dokterdok/Continuity-Activation-Tool/issues/335#issuecomment-248551865, or mute the thread https://github.com/notifications/unsubscribe-auth/ALU-RvoebmgJXpJVjBdqXnGvERD_3Jddks5qsPFhgaJpZM4J5JaY.

Same setup here for me as @alecnash has and the toggle didn't work for me either.

I did "off/on" but failed again. I can't do that everyday.

I am having the same issue, but on 10.12.1 Beta 5 (16B2553a)

Updated my iphone 7 to ios10.2 beta on thursday night and Ive just noticed that handoff is now working again for me, happy days.


DRM Support

Per this article, DRM support isn't fully available for Macs using iGPUs (i.e. iMac20,2). Is there any workaround to get that fixed without using an iGPU-free SMBIOS?

Don't know. I use this system for media creation, not for consuming it.

gotcha, ok.

Basically, any Browser which handles DRM via software can be used as a workaround: FIrefox, Chrome, tec.


About the drone2021 dataset

I would like to confirm that the objects in the drone2021 dataset are indeed birds, correct? Additionally, were the images captured as snapshots taken by a drone?
Thank you!

Hi jet-c-21,
YES for both questions.
Please read the original paper [1] for more information on drone2021 objects.
[1] S. Fujii, K. Akita, N. Ukita, "Distant Bird Detection for Safe Drone Flight and Its Dataset", 17th International Conference on Machine Vision and Applications (MVA), 2021.
Thanks,


Composite unique index on distance table

Cypress tests look to be keycloak related, all are passing except login on local