Update ALT checking regex to include decimal numbers

@wiedehopf as-per Discord:
The current version of mlat-client seems to accept decimals just fine. I've been testing for the past hour and it seems to be OK. My stats on adsbx.org seem fine (lots of green dots).
@wiedehopf do you have any problems with me allowing decimal numbers for the mlat-client --alt argument in the adsbx image?
Currently, the user input is checked with regex: (^\-{0}\d+(m{0,1}|ft)$|^-\d+$).
I'm proposing to change that regex to: (^\-{0}[\d\.]+(m{0,1}|ft)$|^-[\d\.]+$).
That way it will allow:

A positive number ending in 'm' (for metres) or 'ft' (for feet)'; or
A positive number with no suffix (for metres); or
A negative number in metres (no suffix) for below sea level.


Add support for bidirectional effects

This PR adds the feature of bidirectional control flow to the Effekt language. The feature is inspired by the paper

"Handling bidirectional control flow".
Yizhou Zhang, Guido Salvaneschi, and Andrew C. Myers. OOPSLA'20 (PDF).

Could you maybe add a couple of example files with expected output check files to the examples/pos directory?

@b-studios I applied suggested changes and added an example. Please review it.

closing and reopening to trigger CI

Thank you @long-long-float for your contributions! As you can see, there are still a few tests that do not pass and I cannot merge your feature without the CI going through.
The Effekt language has multiple backends and your bidirectionality feature only supports one of them (JavaScript), which is fine. It is expected that your new test fails on all other backends. You can see how the bidirectionality test can be disabled in the other backends here:
https://github.com/effekt-lang/effekt/blob/581b1113eee97dabe0ab4e9a0a0118cef9d81b3d/jvm/src/test/scala/effekt/ChezSchemeTests.scala#L18-L28
However, all other tests that are not concerned with bidirectionality should still pass.  Maybe you can look into this? If you have any questions, I am happy to assist.

@b-studios The CI fails and says an error [error] examples/neg/existential_effect_leaks.effekt:22:1: Unhandled effects: GiveInt. My modification may effect this. Should I fix this?

Yes, it should warn about an effect leaving its scope as you can see in the check file. Would be great if you could look into this

@b-studios I fixed the test case of examples/neg/existential_effect_leaks.effekt, but other cases are faild.
[info]   - unbound_type.effekt (lift) *** FAILED *** (145 milliseconds)
[info]     "... pos: Position): A
[info]     ^[^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^]
[info]     " did not equal "... pos: Position): A
[info]     ^[]
[info]     " (ChezSchemeTests.scala:62)
[info]     Analysis:
[info]     "... pos: Position): A
[info] ^[^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^]
[info] " -> "... pos: Position): A
[info] ^[]

I Investigated the cause and guess that the position of the type of an operator (ret: Effectful) is not set (but I don't know how to set it.)

Don't worry -- just update the test file to reflect the new position. We can fix and improve positions later.

Actually, while you are at it: You could update the methods resolveTerm, resolveType, and resoleCallTarget to something like:
  private[namer] def resolveType(id: Id): TypeSymbol = at(id) {
  //                                                    ^^^^^
    val sym = scope.lookupType(id.name)
    assignSymbol(id, sym)
    sym
  }

This will improve the error positions of the namer

Update: I just modified Namer in master so you only need to rebase. This should hopefully also solve the problem you described here.

@b-studios All tests are passed! Thanks for your advice.

Thanks for your contributions @long-long-float, which I'll merge now!
If you want, you could add more examples of the feature, potentially translating the examples from the original paper? The paper is presented this week at OOPSLA and having translated their examples could facilitate discussion.

Of course I'm creating more examples for the paper 😄

@long-long-float Just to be clear: What I meant is: could you take examples from the existing paper by Zhang er al., translate them to Effekt and add them to this repository?

Yes, I'm translating examples from the existing paper by Zhang er al., to Effekt.

Generators: I'm translating it.
Async-await: I considered that it cannot be because Effekt doesn't have first-class function.
Ping and Pong: I translated it as examples/pos/bidirectional.effekt.

Hey, thanks! I added a quick translation of the iterator example here to talk about it with people at the OOPLSA conference.

Cool! I have just translated the same example here.

That's great. Since you are the expert on bidirectional effects in the Effekt language, do you want to document it on the website? We have a separate repository for that. I added a quick stub, but there could be much detailed explanation about what bidirectional effects motivates, how they work and how they interact with effect polymorphism:
https://github.com/effekt-lang/effekt-website/blob/master/docs/bidirectional.md

Sure, I want to do. I'll write it with my master thesis.


JupyterHub asking for a password

I got an information that students have done all deployments as in a tutorial and have issue after logging via X2Go live session. Starting JupyterHub starts browser but need to be logged in. Any of the passwords works. I tried to re-deploy it and issue still works - even "hardcode" of a password not helps in that situations.
Case: Machine Learning - Python (Create a Data Science Virtual Machine DSVM), import a  dataset containing on-time arrival information for a major U.S. airline,  and use scikit-learn to build a machine-learning model that predicts  whether flights will arrive on time.)

I will look into it and see if i can replicate the issue.

I will be grateful, because it looks like - Azure staff changed something and forget to inform about :)

YES... Immediately I run it, in a browser I see only login or login/password what is not described in a Lab.

Can you send a screen shot? I am not getting prompted for a login of any sort.

I also can't repro this in this scenario, but have had similar issues when Jupyter was incorrectly configured.
Check out this documentation and this StackOverflow question - hopefully they'll get this sorted.
Feel free to reopen if you think this is an issue with the DSVM or the lab itself 😄


pylint complain on models with validators since version 0.27

Bug
For bugs/questions:

OS: Linux (Debian 9)
Python version import sys; print(sys.version):
3.7.3 (default, May 22 2019, 11:07:29)
[GCC 6.3.0 20170516]
Pydantic version import pydantic; print(pydantic.VERSION): 0.27

Where possible please include a self contained code snippet describing your
bug, question, or where applicable feature request:
the below model causes pylint model.py to complain about
model.py:12:4: E0213: Method should have "self" as first argument (no-self-argument)
model.py:12:4: R0201: Method could be a function (no-self-use)
model.py:7:0: R0903: Too few public methods (1/2) (too-few-public-methods)
from typing import Optional

import pydantic

class DemoModel(pydantic.BaseModel):

    some: Optional[str] = None

    @pydantic.validator("some")
    def check_some(cls, val):
        return val

These complaints does not occur with pydantic 0.26.
Marking the validator as @classmethod will cause it to not execute at all, so the only option I have found is to disable these checks in pylint.

The major change in v0.27 was that pydantic was compiled with cython, I guess this makes introspection by pylint of the pydantic code harder or impossible.
There might be some way to trick pylint into think you're using the non-compiled pydantic code (should behave exactly the same), but I'm afraid that's a question for pylint.
Otherwise I don't really know how to help as I've never used pylint.
Let me know if you find a solution.

The issue seems to be in the published version of pydantic.
To replicate this I cloned the pydantic git and did the following.
Running pylint in a docker container with the source mounted I see the following.
$ docker run --rm -it -v `pwd`:/src -w /src -e https_proxy python:3.7 bash -c "pip install -q pylint && cat demo_model.py && pylint demo_model.py"
# pylint: disable=missing-docstring
from typing import Optional

import pydantic

class DemoModel(pydantic.BaseModel):

    some: Optional[str] = None

    @pydantic.validator("some")
    def check_some(cls, val):
        return val

------------------------------------
Your code has been rated at 10.00/10

But if I use the published pydantic version I get the errors mentioned in the bug report.
$ docker run --rm -it -v `pwd`/demo_model.py:/src/demo_model.py -w /src -e https_proxy python:3.7 bash -c "pip install -q pydantic==0.27 pylint && cat demo_model.py && pylint demo_model.py"
# pylint: disable=missing-docstring
from typing import Optional

import pydantic

class DemoModel(pydantic.BaseModel):

    some: Optional[str] = None

    @pydantic.validator("some")
    def check_some(cls, val):
        return val
************* Module demo_model
demo_model.py:12:4: E0213: Method should have "self" as first argument (no-self-argument)
demo_model.py:12:4: R0201: Method could be a function (no-self-use)
demo_model.py:7:0: R0903: Too few public methods (1/2) (too-few-public-methods)

------------------------------------
Your code has been rated at -1.67/10

This might be related to the cython changes done for this release.
And on a side note. When introducing potentially breaking changes, please step up the major version to indicate this. It's hell to investigate and fix test environments due to breaking changes in minor version bumps.

confirmed with
curl -L https://files.pythonhosted.org/packages/28/06/5faf55ff0fc320c508bbdc6d3a85ca1da7537fb42617863eda1a524e4bcd/pydantic-0.27-py36.py37.py38-none-any.whl > pydantic-0.27-py36.py37.py38-none-any.whl
pip uninstall -y pydantic
pip install pydantic-0.27-py36.py37.py38-none-any.whl
pylint test.py

Doesn't give the error, while default install from pypi does.

You might be able to use something like the above for linting, but make sure you're using the compiled pydantic for tests and production.

@gangefors

And on a side note. When introducing potentially breaking changes, please step up the major version to indicate this. It's hell to investigate and fix test environments due to breaking changes in minor version bumps.

I'm following semvar, we're pre version 1 so everything can be a breaking change, but here I did use a new minor version for this change - the biggest change I could make except for upreving to v1 which I think you'd agree would be more confusing.
I documented it very prominently in the change log.
I created a pre-release to test cython, left the pull request #548 open for a couple of days for feedback and ask people to check it worked.

Is that really not sufficient?
But we should move to version 1, I agree.

I would suggest you use something like pyup.io to upgrade dependencies, the changelog included in PRs will avoid "hell".

I hear you regarding following semvar. But then one could argue that it's easy to stay at 0.x.y for all future releases which would make life easy for the developer but not for the users.
This software is awesome and we have considered it really stable for many versions now sans a few minor breaking changes. We do hope that you decide to "release" it soon. Thanks for your work and fast respones in the community.

thanks, I agree - just trying to work out if there's anything big I want to change before releasing.

This is still an issue in pydantic 1.5.1.

Seeing the same issue on pydantic 1.5.1

I'm also having the same issue :/

The code works, its just the pylint complain that annoys, but in the meantime this error could be disabled if you write this comment on top of the class
pylint: disable=E0213
I did find this information on this post:
https://stackoverflow.com/questions/27965324/how-to-disable-pylint-no-self-use-warning

Please reopen, I also have the same issue. It is very annoying

What's the problem in pydantic? As far as I know this is an issue with pylint.
I need some indication of what pydantic is doing wrong or how we might fix it to reopen this.

If you find the open source code I work on in my free time and give away for free annoying, don't use it.

pydantic isn't doing anything wrong, it's pylint that can't introspect the code properly.
AFAICT pydantic.validator returns a classmethod type object and pylint should be able to pick this up but it doesn't.
There might be ways to rewrite the code to make pylint happy, but I do think that it would be better to make pylint handle the code as it is written since it's totally valid code.

I'm pretty sure the code is typed correctly since it passes mypy and didn't used to, but the typing around validators is pretty complicated, I wouldn't be surprised if it was too complicated for pylint.

In general I find mypy + black + isort + flake8 enough. That more does pylint provide?

@samuelcolvin
Even flake8 complains about the first argument not being self. Furthermore, I don't think that changing linters is something that most people would consider.
Nevertheless, the problem is clearly with pylint which doesn't support custom decorators/descriptors.
A hackish workaround is to add a @classmethod decorator after the pydantic.validator one. For simple cases it seems to work and it does silence the linters. Not sure if there are any side-effects though and I haven't tested it in bigger codebases:
import pydantic

class Model(pydantic.BaseModel):

    value: int

    @pydantic.validator("value")
    @classmethod
    def _get_the_answer(cls, val: int) -> int:
        print(val)
        return 42

instance = Model(value=3)
print(instance)

In my case, I will probably just add an explicit # pylint: disable=no-self-argument,no-self-use though

Even flake8 complains about the first argument not being self.

This still happens, but it does only when the pep8-naming flake8 plugin is installed.
from pydantic import BaseModel, validator

class DemoModel(BaseModel):
    name: str

    @validator("name")
    def check_some(cls, val: str) -> str:
        return val

$ pip list
Package           Version
----------------- --------
click             7.1.2
flake8            3.9.1
flake8-polyfill   1.0.2
mccabe            0.6.1
pep517            0.10.0
pep8-naming       0.11.1
pip               21.1.1
pip-tools         6.1.0
pycodestyle       2.7.0
pydantic          1.8.1
pyflakes          2.3.1
setuptools        56.0.0
toml              0.10.2
typing-extensions <IP_ADDRESS>
wheel             0.36.2

$ flake8 main.py
main.py:8:21: N805 first argument of a method should be named 'self'

To configure pep8-naming flake8 plugin to work with Pydantic @validator's, either set the following CLI flag:
flake8 --classmethod-decorators=classmethod,validator

Or set the following configuration option in setup.cfg / flake8.ini / tox.ini:
[flake8]
classmethod-decorators =
    classmethod
    validator

@samuelcolvin
Even flake8 complains about the first argument not being self. Furthermore, I don't think that changing linters is something that most people would consider.
Nevertheless, the problem is clearly with pylint which doesn't support custom decorators/descriptors.
A hackish workaround is to add a @classmethod decorator after the pydantic.validator one. For simple cases it seems to work and it does silence the linters. Not sure if there are any side-effects though and I haven't tested it in bigger codebases:
import pydantic

class Model(pydantic.BaseModel):

    value: int

    @pydantic.validator("value")
    @classmethod
    def _get_the_answer(cls, val: int) -> int:
        print(val)
        return 42

instance = Model(value=3)
print(instance)

In my case, I will probably just add an explicit # pylint: disable=no-self-argument,no-self-use though

Thanks. This works and looks cleaner for me

@samuelcolvin Even flake8 complains about the first argument not being self. Furthermore, I don't think that changing linters is something that most people would consider.
Nevertheless, the problem is clearly with pylint which doesn't support custom decorators/descriptors.
A hackish workaround is to add a @classmethod decorator after the pydantic.validator one. For simple cases it seems to work and it does silence the linters. Not sure if there are any side-effects though and I haven't tested it in bigger codebases:
import pydantic

class Model(pydantic.BaseModel):

    value: int

    @pydantic.validator("value")
    @classmethod
    def _get_the_answer(cls, val: int) -> int:
        print(val)
        return 42

instance = Model(value=3)
print(instance)

In my case, I will probably just add an explicit # pylint: disable=no-self-argument,no-self-use though

Adding @classmethod will not work before still makes PyCharm complain and putting it after will not call the validator.
I just added # noqa: N805

Pycharm works fine if you use the plugin.

Pycharm works fine if you use the plugin.

I do have the plugin and it works for most cases except this one.

I think you might need to import validator directly.

I am using mypy too @samuelcolvin but i am using disallow_untyped_defs = True and i am def getting that error on self :(

To configure pep8-naming flake8 plugin to work with Pydantic @validator's, either set the following CLI flag:
flake8 --classmethod-decorators=classmethod,validator

Or set the following configuration option in setup.cfg / .flake8 / tox.ini:
[flake8]
classmethod-decorators =
    classmethod
    validator

Saved my freaking sanity (or what's left of it). Thanks!


Invoke color picker with shortcut

Discussed in https://github.com/microsoft/vscode-python/discussions/15697

Originally posted by blaylockbk March 17, 2021
VS code has a built-in color picker that is so handy for some languages but is not useable in a Python script.
Sometimes I have a variable or argument that represents a color RGB or Hex, and it would be wonderful if I could modify these with a color picker. It's probably not possible for the extension to automatically know these are colors, but it would be nice if I could manually invoke the color picker with a short cut, as discussed here https://github.com/microsoft/vscode/issues/52364, and then click the color to insert it.
Color variable as RGB
my_color = (.4, .5, .3)

Color argument as a Hex value
import matplotlib.pyplot as plt
plt.plot([1,2,3,4], color="#6666ff")
```</div>

I found a workaround using the json-color-token vscode extension.
See this for an example of how to invoke the color picker for hex colors in a python file
https://github.com/YECHUNAN/json-color-token/issues/2

Thank you to everyone who upvoted this issue! Since the community showed interest in this feature request we will leave this issue open as something to consider implementing at some point in the future.
We do encourage people to continue 👍 this issue as it helps us prioritize our work based on what the community seems to want the most.

I know this issue is about a keyboard shortcut, I don't know if this kind of comment is helpful (apologies if not), but I wanted to suggest an alternative way this could work.
I don't know how feasible this is as it seems like it would need support in the language server, but it would be nice to be able to define an "RGB" class e.g. as a namedtuple or dataclass, and in the VSCode workspace configuration, specify that the name "RGB" refers to an RGB color, and have VSCode display the color picker and box next to it wherever it appears in source files. For example, it might be used something like RGB(1, 2, 3). If the arguments are not literals, it should not get the color picker and just be treated like regular code.
There might be some variation in the way such a class is defined (are the values floats in the interval [0.0, 1.0], are they ints in the interval [0, 255), or does it take a hex string as an argument, like RGB("#ABCDEF"), or a single int literal e.g. RGB(0xABCDEF)?). I think its possible to differentiate and support most of these cases though.
I also wanted to add -- something that is currently supported, does not require any extensions, and although does not solve the exact problem of RGB color literals in the source, gives a good editing experience if you don't mind defining your color values in a separate JSON file. I noticed that you can actually specify a JSONSchema using "$schema" and use the "color" format -- it will get the color picker and colored box next to it. Windows Terminal settings.json does this (and is a good reference for how to do this, see the schema). Again, needs no extensions, and gives a good editing experience if it is acceptable that the colors are defined in a separate JSON file. And its fairly trivial in many languages to load a JSON file and package arbitrary resources/assets with your code. You also need to host the schema somewhere I think (or possibly it can be a reference to a local file).


volumes appears to require absolute paths

File: get-started/part5.md, CC @johndmulhausen
I hope this isn't in error; I'm new to Docker so this could be entirely off-base, but when I tried to go through all the steps of this tutorial in order and when I got to this part, the redis service wouldn't start and I could see the beginning of 'invalid mount' in the docker stack ps getstartedlab entry for it. I couldn't figure out how to see the full string, but changing the volume directories to absolute paths in both the host and container paths seemed to fix it.

The instructions are not very clear on that part 😢
In the docker-compose.yml file there's this instruction:
volumes:
      - ./data:/data

In this case . refers the directory where the compose-file.yaml is. So you need to make sure you create a data directory in the same place as you have your compose file.

I did, and it didn't work.
On May 8, 2017 12:13 PM, "Joao Fernandes"<EMAIL_ADDRESS>wrote:

The instructions are not very clear on that part 😢
In the docker-compose.yml file there's this instruction:
volumes:
- ./data:/data
In this case . refers the directory where the compose-file.yaml is. So
you need to make sure you create a data directory in the same place as
you have your compose file.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker.github.io/issues/3189#issuecomment-299929505,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE7e7Gf6rIwfcRho0ZA_W9CjG3pEAt6Eks5r300rgaJpZM4NS-5O
.

Can you paste here the docker-compose.yml file you're using, as well as the structure of your project directory tree .?

I can't replicate this again, which means I probably didn't type what I thought I had typed earlier. Sorry about the mishap :sweat_smile:

No worries! Thanks for following-up on this.


Backup AffectedCommits in a separate bucket.

The existing backup doesn't easily distinguish between entity types. This makes it easier for internal consumers to use our AffectedCommits data.
Using a different bucket here so we can set a different lifecycle policy (3 days TTL) than our existing backup bucket (60 days TTL).

Does terraform need to be updated to add the new bucket?

Forgot about that. Thanks for reminding me!


Rebel C-3PO now wrong point cost.

When Republic C-3PO got its points cost corrected to be 6 points from 7, Rebel C-3PO also got listed as 6 points when he should still be 7.

FIxed in #525

Not fixed, R2-D7 Bot still displays Rebel C-3PO crew as 6 points instead of 7.

#525 hasn’t been merged in yet but the fix is in that PR


Code maintenance in utility module

Fix type annotations
Add a unit test
Minor tweaks

We should probably use Optional over Union in other modules as well. I'll create separate issue for this.

We should probably use Optional over Union in other modules as well. I'll create separate issue for this.

That's a matter of taste :smile: But in a lot of places a default value of None is not explicitly annotated.

We should probably use Optional over Union in other modules as well. I'll create separate issue for this.

That's a matter of taste 😄 But in a lot of places a default value of None is not explicitly annotated.

Yeah, but I think it's good practice to keep it consistent in whole package.


[kie-issues#995] Upgrade Spring Boot to 3.2.4

This is a PR to upgrade Spring Boot to version 3.2.4.
kie-issue: https://github.com/apache/incubator-kie-issues/issues/995
Related kogito-runtimes PR: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3463
Related kogito-examples PR: https://github.com/apache/incubator-kie-kogito-examples/pull/1903
Related optaplanner-quickstarts PR: https://github.com/apache/incubator-kie-optaplanner-quickstarts/pull/628

@yurloc see ^


Chef 12 'chef-server-ctl upgrade' fails on RHEL 5.11

I'm migrating from Chef 11 (11.1.3) to Chef 12 (12.0.5) on a RHEL 5.11 server, and running into an issue after running chef-server-ctl upgrade:-
- configure service runit_service[rabbitmq]
* link[/opt/opscode/service/rabbitmq] action create
  - create symlink at /opt/opscode/service/rabbitmq to /opt/opscode/sv/rabbitmq

At this point, it hangs. Running strace against the chef-client process shows it trying to stat /opt/opscode/service/rabbitmq/supervise/ok, which apparently doesn't exist:-
stat("/opt/opscode/service/rabbitmq/supervise/ok", 0x7fffa031f450) = -1 ENOENT (No such file or directory)

I'm assuming this part of the chef-server-ctl upgrade is supposed to create it, but for some reason it isn't. I've attempted the upgrade twice now, and both times it has failed at the same point.
Exiting out of the upgrade leaves Chef 12 in a bad state, and it appears to then be impossible to continue the upgrade:-
<...>
Creating /tmp/chef11-server-data20150312-21492-nblrd0 as the location to save the open source Chef 11 server data
Ensuring Chef 12 server is stopped
warning: rabbitmq: unable to open supervise/ok: file does not exist
Unable to stop the Chef 12 server, which is needed to complete the upgrade

The only way to get the upgrade process to work again is to uninstall the chef-server-core RPM and start again.

Not sure if it's relevant, but I see lots of references to private-chef-ctl, whereas I'm following the instructions for the Open Source version.

This looks like a chef-server issue, I've refiled it over there as https://github.com/chef/chef-server/issues/127.


Even or Odd

Title of programme
Write a Java programme to check a number even or odd.
Additional information
Input  : 3
Output : Odd Number

Input  : 2
Output : Even Number

Code of Conduct

[X] I have read and agree to the Codinasion project's Code of Conduct

Can I take this up? @harshraj8843

Can I take this up? @harshraj8843

Sure
Thanks for showing your interest in this project 😃

I have raised a PR. Could you please check and let me know if I need to change anything?


ciri: default conf (from readme) stopped working

since https://github.com/iotaledger/entangled/commit/1861c9f6d1f4d4a037b88e1fc2df266a83432d7d
running a node with:
bazel run --define network=mainnet -- ciri

shows the usage and exits - reverting the commit, the node starts fine.

Hi @alon-e, it seems to work fine on my side.
However I did change the name of some options so I believe your conf.yml is not up-to-date.
I suspect:

coordinator is now coordinator-address
num-keys-in-milestone is now coordinator-num-keys-in-milestone
Please let me know if that work.

I'll add a better error message than the usage for this kind of scenario.

Hi @alon-e, following https://github.com/iotaledger/entangled/pull/1100, conf.yml is not tracked by git anymore. We provide a conf.example.yml that can be renamed.
Please update us if my previous message fixes your issue.

This has been fixed. Please reopen if you still have issues.


Check if the feedback form can be configured to default to email for sharing options

Related to https://github.com/itchysats/10101/issues/513
On Android we get a Share menu which is not intuitive.
Check if it's possible to configure the feedback form to be more specific to share  the feedback via email.

I found a way to specify a fallback that would at least allow us to send text (and the user could attach additional images). This is obviously not as convenient, but better than nothing.


added skeleton ui for appbar when status is loading

Pull Request Title
Thank you for contributing to our project! Please provide a summary of your changes and reference any related issues below.
Issue Number: Fixes #172
🛠 Type of Change

[x] 🐛 Bug fix
[ ] ✨ New feature
[ ] 🚨 Breaking change
[ ] 📚 Documentation update

✅ Checklist
To ensure a smooth review process, please check off each item as you complete it:

[*] Code Style: My code adheres to the project’s style guidelines.
[*] Self-Review: I have reviewed my own code and made improvements.
[ ] Comments: I’ve added comments to explain complex or non-obvious parts of the code.
[ ] Documentation: I’ve updated the documentation to reflect my changes.
[ ] Warnings: My changes introduce no new warnings.
[ ] Tests: I’ve added tests to verify that my changes work as expected.
[ ] Passes Locally: All new and existing unit tests pass on my local machine.

We appreciate your contributions! If you have any questions or need further assistance, feel free to reach out.
fix :
problem : Session is picked up after the page fully loads and therefore leading to a delay in the rendering of user image and wallet icon
Solution : Added Skeleton when status is loading to avoid delay in rendering of image and wallet icon
Before :
https://github.com/user-attachments/assets/df6063a2-4ddf-4a5f-a25c-4e462bef6b64
After:
When User is not logged in-
https://github.com/user-attachments/assets/d84e105b-f059-4b4d-889c-31ee660d4920
When User is logged in-
https://github.com/user-attachments/assets/6d0b674e-bbca-42b3-a30d-e8bbb89e3b14

@cb7chaitanya what about this pr.

What about it? @Ameerjafar
I am not going to merge it @Bhushan-04 as the issue was assigned to @Ameerjafar and I can only merge it if @Ameerjafar is not going to turn up with a PR

Ok sir let me make the pr @cb7chaitanya
On Fri, 20 Sep, 2024, 8:06 pm cb7chaitanya, @.***>
wrote:

What about it? @Ameerjafar https://github.com/Ameerjafar
I am not going to merge it @Bhushan-04 https://github.com/Bhushan-04 as
the issue was assigned to @Ameerjafar https://github.com/Ameerjafar and
I can only merge it if @Ameerjafar https://github.com/Ameerjafar is not
going to turn up with a PR
—
Reply to this email directly, view it on GitHub
https://github.com/code100x/tiplink/pull/188#issuecomment-2363884639,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZE45SIXG4UCF3WWFOIBNRDZXQXFVAVCNFSM6AAAAABORVQTF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRTHA4DINRTHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

What about it? @Ameerjafar I am not going to merge it @Bhushan-04 as the issue was assigned to @Ameerjafar and I can only merge it if @Ameerjafar is not going to turn up with a PR

Sure , no problem sir !


Fix BDC remember password and reprompting connection

Fixes #7921
Fixed a couple of issues.
First off was the issue that even if you left remember password unchecked the node would still "remember" it and thus not prompt you for the password when you opened the dashboard. But if you restarted ADS then it would correctly not store it and so would fail to open the dashboard.
I've changed it to not save the password to the credential store, and then to re-prompt for the password if it doesn't exist (or the stored password is wrong, such as if it changed). The password will persist for the session though so the user won't be prompted unless an error occurs.
Other fixes :

Fixed error message displayed on Connection dialog to be more friendly (before it mentioned something about endpoints which isn't helpful in that scenario)
Made dashboard open when new controller connection is created
Added error handling for model
Added reprompt connection for model if there's no valid controller (for example if the user cancelled out of the connection dialog when the dashboard was opened)

Follow-up items not included in this PR :

Still need to make the dashboard indicate an error occurred (right now it just sits there spinning)

@kevcunnane @alanrenmsft Could you take a look at this when you get a chance?

@kevcunnane I changed it as discussed earlier so it now keeps the password for the session (re-prompting if the credentials are wrong though).
I also fixed up some no-floating-promises errors


[API] Add 'const' for pointers to input arrays

This PR adds const for pointers to input arrays

@atamazov Please review

Please remove the https://github.com/ROCmSoftwarePlatform/MIOpen/labels/correctness label.


Use qualify  row_number() ... not support *

When I execute the following SQL statement.
select a.* from tablea a qualify row_number() over (partition by a.id order by a.date desc) = 1;
I will receive an exception message.
ERROR 1064 (HY000): Getting syntax error. Detail message: Can't support result other than column..
Adjust the statement to
select a.id,a.name from tablea a qualify row_number() over (partition by a.id order by a.date desc) = 1;
The statement can run normally.

repeat


Anchor Wallet still not allowing Ledger Nano S setup

Hi Aaron, I see you closed the issue, so I'm not sure if I need to reopen as a new issue?  I tried all your suggestions:

I opened a brand new Ledger Nano S chord/connector to use.
I ensured Ledger Live Manager was on and connected
I closed out all other wallets.
I restarted my computer.
I verified all firmware is up to date.

But the Anchor is still saying "reinitializing" endlessly. When I try to connect the EOS app within Ledger, it bounces me out to the main menue.
Any other ideas? I really need to secure the Anchor .
Thanks
Jay

I closed it since the support discussions like this typically happen on the forums and we only keep issues open if there's a bug we need to track and fix within Anchor. We can also continue discussion here if you'd prefer too, even if the issue is closed.
Are you saying that when you open the EOS app on the Ledger, the EOS app on the Ledger closes and it goes back to the menu on the device? I am honestly not sure what is causing that, and sadly we aren't the developers of that software. That app can be found here:
https://github.com/LedgerHQ/app-eos
Anchor itself requires the EOS App to be running on the device before it can connect - so if I'm understanding the issue right, that's why it gets stuck on "reinitializing". It's trying to connect to the running EOS app but since it's not started, it's retrying over and over.

Thanks for continuing the discussion here Aaron.
Yes, when I open the EOS app on the Ledger, the EOS app on the Ledger closes and it goes back to the menu on the device. But it ONLY does this when connected to the Anchor wallet. It works with virtually every other wallet and software I have. The Anchor is the only wallet I experience this issue with.

That's incredibly odd, I've never seen that before. I just updated my Ledger(s) to the latest firmware and latest version of the EOS App, and using Anchor 1.1.12, I'm not seeing the same issue. Few questions:

What version of the Ledger app are you running? You mentioned upgrading the firmware, but figured I'd ask if you had the latest EOS app.
Have you tried other EOSIO wallets with the Ledger, and do those work?

The only other thing I can think of is that the Ledger Live might also be interfering if it's connected.

Yes, it is odd. I just opened another brand new Ledger S and loaded only EOS  app on it.
I have 1.6.1 on the Ledger.
I have not used the ledger with other EOS wallets. Just other coin wallets.
I disconnected the Ledger Live to see if that would help.  It allowed me not to be booted off the EOS in the ledger. But still endless Reinitializing in the Anchor. Then I turned on the Ledger Live again, and it booted me off again.
Any other ideas? Do I need to find another EOS wallet that can be secured by Ledger? If yes, can you suggest one? Thanks Jay

1.3.1? Just confirming, I don't see a 1.6.1.
With Ledger Live turned off, and Anchor stuck on Reinitializing, try opening the Ledger section of the app and stopping it and then restarting it (or just restart Anchor with Ledger Live closed). See if that does the trick.
If you're looking to try other wallets - you could try connecting it with Scatter (https://get-scatter.com/), but I have a feeling whatever is preventing the connection with Anchor will do the same with Scatter. Most EOSIO wallets use the same methods to connect with Ledger devices (HID) and have these same sorts of oddities.

Yes, it's 1.6.1 which is the latest version.
I also updated my Ledger Live to the most recent update.
One thing I noticed is that in the Anchor screen to the right of "Ledger- Public Key"  it says "Load Public Key[0]"  I have let it stay at 0, which may be a mistake? I don't know how to find the Ledger Public Key.
Is this affecting my results?

Very odd. I wonder if that version has something to do with the issue you're experiencing. In Ledger Live, all I see is 1.3.1:

I'll have to do some digging to see why that is.

One thing I noticed is that in the Anchor screen to the right of "Ledger- Public Key" it says "Load Public Key[0]" I have let it stay at 0, which may be a mistake?

That is correct, 0 is the first key derived by the Ledger and most people will just default to using it. You can change the index to any number you want - but for ease of use, typically you'll just want to leave it at 0.
When your Ledger is connected, the Load Public Key[0] button when clicked will load and display the first key returned from the Ledger. There will also be a "Verify on Device" button that when clicked will tell the Ledger to display the first key on the devices screen - which you can verify matches that which Anchor (or any other wallet) is reporting.

Actually the EOS is 1.3.1. I was referring to the Ledger Nano S as 1.6.1
Thanks for explaining the load public key option.
If you happen to find out anything else about the ledger bug please let me know.

We'll keep looking and trying to reproduce the issue. I'll also be curious to know if Scatter is able to connect to your Ledger. If it's not, it's an issue with something on your computer - as opposed to something Anchor specific.

For those coming to this thread with the same issue.
I have found that when Ledger Live Manager is running then I get the same symptoms described by the OP. The EOS app on the ledger would just quit. Making sure Ledger Live Manager is not running allows me to use the ledger as expected.
TLDR;
Don't run Ledger Live Manager when you want to use Anchor.


91 7e (LENGTH_ERROR)

On every example I run I get a 917e (LENGTH_ERROR).
I am using a desfire ev1 4k

Do you mean the examples on the sample directory?
This example is a simple test to see if things are working. Your DESFire EV1 card is assumed to be using the default key and cipher type (key is all zeros and cipher type is AES).


Add read_from option to query creation

When creating a new query, Nakadi SQL would process the entire backlog
of events sitting in the input event types. This lead to a spike in
load and the production of potentially a lot of events.
The decision to process all the events was made when we only had
queries for which this is the only sensible choice, namely FULL OUTER
JOINs.
In a full outer join, not having the complete state of the input event
types makes little sense.
But since then, a lot has changed and currently the most common type
of query are stateless queries, meaning filters and projections. For a
significant amount of users, producing events retroactively is not the
expected behaviour and is the most cost inneficient, thus this field
is introduced with default to 'end'.
There is a chance users might pick 'end' without first understanding
its implications or simply overlooking and later on having their
queries not producing events. But parsing the query to identify if the
input event type is a log compacted event type would be significantly
more complex so we trust users to make a sensible choice.

👍

👍


Document pre-installed Python packages

As part of many customer asks and CSA requests, we have seen a pattern of folks wanting to know what packages are pre-installed in the serverless ecosystem and what a customer needs to add in their requirements.txt.
For eg:  does the existing serverless infra already install latest versions of the UUID library? if so what version does it have?
If someone tries to use the latest version of UUID in their requirements.txt that might not have compatibility with older versions they will get to know through error messages.
Can this be published?
This has also been an ask when comparing competitor products.

Roger just assigned this to you. Please document this in a wiki in this repo.

Somewhat related to azure/azure-functions-python-library#52
I'd like to see the the included packages and versions documented. In particular the expected behavior when installing azure-functions

@anirudhgarg @priyaananthasankar @cecilphillip
I created a Wiki page for the preinstalled libraries: https://github.com/Azure/azure-functions-python-worker/wiki/Preinstalled-Python-Libraries


Message broker integration should be moved to back end app settings

Description
Message broker (RabbitMQ) integration should be moved to application settings rather than configured from Integrations UI
Value
Message broker currently is a 100% service component for Zafira and has no impact on any of user flows; it should be hidden from user interface and encapsulated as part of back end service implementation.
Additional context
From implementation perspective i propose to remove this integration completely (both integration group and type) and move broker settings to application.yml.

@lmikhadyuk you can test it using test running on stage, if logs are good - 👍

@Boniara could you please provide SNAPSHOT version?

Verified in ui v.1.295-SNAPSHOT | server v.1.468-SNAPSHOT | client v.1.0-SNAPSHOT.


Flux example

Hi,
a working flux example would be very nice. I'm now fighting with the notion of a notification store and what it would contain. Especially if I add an alert for every notofication in the store, when would I then remove the notification from the store? Because for every notification added, all notifications in the store would be shown again.
/Herman

+1
The example that is given in the README is very opaque and difficult to understand in context of an actual flux application.

+1
I am using redux and actions ´TOAST_INCOMING´ and ´TOAST_DISPLAYED´ for adding and removing messages from a "queue".  The component reads the queue in ´componentWillUpdate´
componentWillUpdate(nextProps){
     nextProps.toast.items.forEach(item=>{

         this.refs.container.success(
             "test",
             "test", {
                 closeButton:true,
                 timeOut: 5000,
                 extendedTimeOut: 2000
             });

         const {dispatch} = this.props;
         dispatch(Actions.displayed(item.id)); // remove from toast queue
     })
}

The toasts are displayed but the close button and the timeouts are broken so the toasts are not removeable/usable.
any ideas?

Hello!
I've was able to come up with an implementation of react-toastr using redux that can be found here:
https://gist.github.com/jerairrest/bcfd0e6692e90625d8d1
To use the notification, simply dispatch the action 'displayNotification' with the needed title and message.
Let me know if you have any questions,
Jer

@jerairrest Thank you. This is quite the same approach I took, but it does not work with the default message factory (without jquery):
var ToastMessageFactory = React.createFactory(ReactToastr.ToastMessage.animation);

Are your messages not removing? Or is it just the fact the animation doesn't work?

Have you included animate.css?

@kromit I have the same problem too.
What is the solution?


Table border is drawn over the cell content

Steps to Reproduce
Run this snippet:
import 'package:flutter/material.dart';

void main() {
  runApp(
    MaterialApp(
      home: MyWidget(),
    ),
  );
}

class MyWidget extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    final BorderSide borderSide = BorderSide(
      width: 20,
      color: const Color(0xff00ff00),
    );

    return Scaffold(
      body: Table(
        border: TableBorder(
          verticalInside: borderSide,
          horizontalInside: borderSide,
        ),
        children: <TableRow>[
          for (var row = 0; row < 10; row++)
          TableRow(
            children: <Widget>[
              for (var col = 0; col < 10; col++) Text("cell ${row + 1} ${col + 1}", style: TextStyle(fontSize: 20, fontWeight: FontWeight.bold)),
            ],
          ),
        ],
      ),
    );
  }
}

Expected results: actually, I don't know if it is intended behaviour. I expect borders to be drawn between cells, not over them.
Actual results:
https://codepen.io/Dimous/pen/abBeWML

@Dimous
Since the width you set is 20 and there doesn't seem to be any spacing between cells, the borders seems to be over it.
Can you try by setting the space between cells and see if it helps ? Also, what's the expected behavior / experience you are looking for in this use case ?

Hello, @darshankawar!
I think borders in a table should separate the cells (set spacing with themselves -- if I set border with width of 20 pixels, it should make that 20px gap, ok?), not just be decorated on cells (drawn on widgets placed in cells).
How do I set space between rows? I can't find appropriate attribute in Table/TableRow/TableCell.
I tried Padding  with custom logic for the first row but I don't think this is how it should be.

@Dimous
Thanks for the update. I don't see either an explicit property to set space between rows for those widgets.
Keeping this issue open for further investigation.


Worked like a charm on the first try, will be using it again often in the future to collaboratively develop

Product and Version [VS/VSCode]:
OS Version [macOS/Windows]:
Live Share Extension Version:
Target Platform or Language [e.g. Node.js]:
Steps to Reproduce / Scenario:

@lildocta Hey! Thanks for reaching out 👍It's great to hear that Live Share worked well for you, and please don't hesitate to let us know if you run into any issues/questions/feedback after using it more. Thanks!


There is the possibility of activating Third-party cookies on iOS?

There is the possibility of activating Third-party cookies on iOS?  And how could I do it?
In Android-specific options if you have it and it works correctly, but in iOS it does not work.
thirdPartyCookiesEnabled: Boolean value to enable third party cookies in the WebView.

would be nice to see that feature


[Regression] V2 - Corporate Console - CLA Manager role is not assigned to cla manager designee who signed CLA

Create CLA Group at foundation level
Assign cla manager designee
As cla manager designee sign CLA for the project from corporate console
Observed that CLA Manager role is not assigned to cla manager designee who signed CLA. Due to this issue not able to add/delete cla manager and not able to add/delete approval criteria

Tested for Project level and observed same issue

The behavior is very in consistent. Sometimes issue is seen for foundation level and sometimes for project level.

Cleaned up and added a ton of debugging. The assignment of the cla-manager and removal of cla-manager-designee is called from the Database trigger when the Signatures record is updated to indicate signed=true.  Upon this event, the record is evaluated and:

Signed On date field is added
if CCLA

company record is loaded
cla-manager role is assigned to the initial CLA manager

User is searched by via LF Username
User is searched by LF Email
User is searched by Alternative Emails
CLA Group -> Project Mappings are loaded
cla-manager role assigned for each ProjectSFID + CompanySFID

CLA Group -> Project Mappings are loaded
(New) cla-manager-designee roles are removed for all users who were assigned this role for ProjectSFID + CompanySFID

@nirupamav Created PR and pushed to DEV. Needed to deploy since the trigger is accomplished via a DB update event.  Need to construct and validate a test scenario.  I will do this...unless you have a nice automated script.
For this one, the log details will be in: DEV us-east-2 easycla-api cloudwatch log

@dealako
I have automated script to validate this scenario. I ran the scripts today and observed that the issue is still reproducible

@nirupamav I have verified this ticket looks like the issue is resolved so moving this ticket to QA.

Executed the automated scripts and Verified that the issue is fixed


Fix xml entity handling

Registers the xml entities. Therefore they are preserved. Also fixes and issue with Lax option and fragments.
Unfortunately it does not solve the Fix XML option yet. But I have optimized it so much, that we could perhaps remove the option. Somebody has to do a benchmark for that.

@SebastianStehle I was out of time yesterday. I'll need to review this later tonight, sorry! 😀

Closed in favor of the other PR.


Import "helper_functions" could not be resolved(reportMissingImports)

Hi, I am getting this error message on 03. PyTorch computer vision 6.1.
Import "helper_functions" could not be resolved(reportMissingImports)
Any suggestions on how to resolve?

Have you solved your problem? I have same error and after 2 days everything start working. It was a problem with colab itself i guess.

No, it's still not working.

It is not a problem, actually. Google Colab writes this, but you can access functions from helper_functions anyway. I have many similar small problems, but code still works So. Just I just move on.

It is not a problem, actually. Google Colab writes this, but you can access functions from helper_functions anyway. I have many similar small problems, but code still works So. Just I just move on.

Yes, it still works, but the weights are not updating correctly.

Hi @rafaelblevin821 and @PrzemyslawCh,
I'm not sure what's happening here.
If it's just a warning but the code works, feel free to ignore it.
Otherwise I can confirm that downloading/importing the helper_functions.py file works for me.
See more here:
• Answer on downloading helper_functions.py - https://github.com/mrdbourke/pytorch-deep-learning/discussions/308#discussioncomment-5096916
• Example notebook for downloading helper_functions.py - https://colab.research.google.com/drive/1WgEpv0VhI9Wo_dizMMwNuX1AzLtTrDeJ?usp=sharing


github.com/gorilla has been archived

The Gorilla Web Toolkit has been archived. I think it'll be a good thing to look into how this will affect Dapr.

Usually when these things happen (see the JWT library for example), the community will take that up. A few forks will pop up, and one will establish itself as the new one.
It's also possible that the gorilla libraries are in a "done" state and won't really need much maintenance either.

@ItalyPaleAle good point!
On the other hand,  thinking about how quickly this project would react if in the future a major bug or security issue is found in the gorilla libraries.
I guess instead of creating this issue as bug, I should've created it as a question issue.

So far I haven't seen a single "strong" fork for gorilla/mux.
A lot of people seem to be migrating to other routers, actually. The most popular choices (aside from those based on fasthttp like fiber, which we should steer clear of) seem to be Gin and Chi.
Gin is more powerful, but Chi is simpler and perhaps a better choice for us?
https://github.com/go-chi/chi

Chi would be a good choice.

@ItalyPaleAle @robertojrojas @yaron2 @davidkarlsen
pls just use this github.com/cloudwego/hertz
or allow an option to switch default http server

GWT has been revived again https://github.com/gorilla/mux/issues/715


[AutoPR eventgrid/data-plane] Update the MediaServices.json file in the EventGrid DataPlane SDK

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4111

This PR has been merged into https://github.com/Azure/azure-sdk-for-go/pull/3044


build not working

Plz add build info, thanks

Hi! First, thank you for trying to build my project. I would not anticipated that anyone would use it, but since you tried, I added the small doc for build script in README.md.
The build script did not work because I made it to build without CRT from assembly language, that would only work on only one architecture, where I develop it (on a Raspberry pi).
I made that so build script would build with CRT by default, so probably it should work now, but I am not sure. If it can't, please send the error messages you get.
Thank you!


CSV Export in Reports / Lists shows weird characters

@caadler This is on the export of "All Projects"  table I suppose, all the reports exported look fine to me.

Before:

After:


Support more than 16 vCPUs

Currently, HAXM supports max 16 vCPUs per VM.
Considering recent multi-core CPUs (e.g., Xeon scalable processor), I think the current limit  is small. KVM supports max 288 vCPUs.
It seems on NetBSD, HAX_MAX_VMS * HAX_MAX_VCPUS pseudo devices are created at the module initialization,  but on other platforms, a pseudo device is created when creating a vCPU (i.e., in the hax_vcpu_create_host()). So I think except for NetBSD, increasing  HAX_MAX_VCPUS has no side effects.
QEMU uses the HAX_MAX_VCPUS. This is my patch for the QEMU to allocate the hax_vcpu_state as much as necessary (I tested this on the Linux host / Linux guest). I'm not sure other software using HAXM.
Is there anything to consider?

Do you test on Windows host? I checked Windows HAXM codes. It also has limitation that HAX_MAX_VCPUS is defined to 16. Besides the limitation, vpid is a two bytes structure. High byte represents vm_id. So can't allocate vcpu for a VM more than 255.

@hyuan3 Hi, I don't have a Windows test environment and I tested my patch only on Linux.


stellar-core: 18.5.0 -> 19.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/stellar/stellar-core/releases.
meta.description for stellar-core is: "Implements the Stellar Consensus Protocol, a federated consensus protocol"
meta.homepage for stellar-core is: "https://www.stellar.org/"
meta.changelog for stellar-core is: ""
Updates performed

Version update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 19.3.0 in filename of file in /nix/store/vynkbmg691r43yd1qf9zw4rrr13320r3-stellar-core-19.3.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
stellar-core

Instructions to test this update (click to expand)

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

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

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

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


Add support for private class fields (#name) [low-priority]

This allows classes have private variables and methods like the ones you can have in a constructor function.
[ ] JSHint should be able to detect variable & method definitions via their # prefix.
[ ] Also, JSHint should give an error for using # in front of get and set. Getters and setters can be defined with get #name.
[ ] Also, according to a editor note below the Note 2 in the proposal for private fields, JSHint should give an error for trying to delete any private static or instance definition.
[ ] Also, according to the same proposal, having #constructor should give an error.
^This thing is pretty new, even firefox doesn't support it now, so it would be the best to leave this as a low-priority.

Hi there! We're using gh-3139 to track this feature request, so I'm going to close this in the interest of keeping things organized.


rework SMS selector

closes #1661 - new users should not see retired data and a fix for the workflow.finished? check.


Lint fixes

Builds on master are failing due to lint issues. This PR simply fixes the problems.

/gcbrun


Prepare Release and Debug build

Currently we use debug build to build ONE-vscode.
We now need to prepare release build.
What is the difference between debug and release build with ONE-vscode?

Debug build

put everything

Release build

remove '[debug]' log

What else?
Note: tsc compilers (https://www.typescriptlang.org/docs/handbook/compiler-options.html) does not seems to have optimization flgas.

How to implement:

env var? (like https://stackoverflow.com/a/59908595)

it won't work if ONE-vscode works after installing vsix.

What is the difference between debug and release build with ONE-vscode?

Q. Is it meaning that npm run compile manually removes debug log when it is built on release mode? or tsc compiler automatically removes debug log?

@YongseopKim,
For now, as far as release-build in a form of vsix does not show debug log, it would be great.
(Main purpose of running one-vscode by pressing F5 is for debugging... so we could consider it as debug-build.....)

So, can I understand like below in ONE-vscode?

a kind of debug build: run in debug mode -> Logger.debug() prints
a kind of release build: install/run vsix -> Logger.debug() doesn't print

Yes.

And, we don't have to prepare release build? because vsix is done with release build.

I tried to find the difference between release build and debug build in case of JS or TS project. However, it seems that there are not many difference between them.
So I'd like to focus only on Logger.debug() in this issue.

Let me rename this.

logger.debug was handled well. Let me close.


About negative rating(s)

I have a Codeforces account named "BouncyHunter". I have negative rating but it's displayed as "UNRATED" according to this rating badge. Please fix that, thanks.


Twice a day devices become unavailable for a moment

I have a a few Xiaomi BT sensors and plugs connected to ZNDMWG03LM gateway with 1.5.4_0090 firmware.
Every 12 hours (twice a day) ~at 9 AM and 9 PM (+- a few minutes) devices connected to gateway are displayed as unavailable for seconds. This resets some of my homeassistant timers (since i use for my purposes  last_updated states from sensors).
XiaomiGateway3 plugin version is 3.3.6. HA version is latest stable (but this issues persist on any of the latest versions).
Checked my router and it seems Wifi connection of gateway wasn't interrupted. Internet connection itself is stable.
Also everything is fine in MiHome app.
Here I tried to log this issue and can see unusual MQTT disconnect:
2024-03-02 09:02:16 [D] <IP_ADDRESS> [BLEE] a4c138721c24 (1371) recv {'temperature': -0.5}
2024-03-02 09:02:28 [D] <IP_ADDRESS> [BLEE] a4c138102e3b (1371) recv {'temperature': 0.4}
2024-03-02 09:03:15 [D] <IP_ADDRESS> [LUMI] 0x00158d00044ecad8 (0xb6fc) recv {'temperature': 1.36}
2024-03-02 09:03:15 [D] <IP_ADDRESS> [LUMI] 0x00158d00044ecad8 (0xb6fc) recv {'humidity': 91.04}
2024-03-02 09:06:16 [D] <IP_ADDRESS> [BLEE] a4c138721c24 (1371) recv {'battery': 50}
2024-03-02 09:06:57 [D] <IP_ADDRESS> [BASE] MQTT disconnected
2024-03-02 09:06:57 [D] <IP_ADDRESS> [BASE] Prepare gateway mgw with fw 1.5.4_0090
2024-03-02 09:06:57 [D] <IP_ADDRESS> [BASE] openmiio: previous log: b'21:04:34.405 INF openmiio_agent version 1.2.1 linux/mipsle\n21:04:34.500 INF [miio] connected to miio_client\n21:04:35.519 INF [zigb] listen TCP port=8888\n21:04:35.520 INF [zigb] run app=Lumi_Z3GatewayHost_MQTT\n21:04:35.528 INF [miio] accept conn app=gateway\n21:04:35.530 INF [miio] accept conn app=zigbee\n21:04:35.646 INF [cent] using original socket\n21:04:35.670 INF [mqtt] online\n21:04:35.815 INF [miio] accept conn app=automation\n21:04:36.808 INF [miio] accept conn app=hardware\n21:04:38.807 INF [miio] accept conn app=homekit\n21:04:40.562 INF [cent] accept conn\n21:04:40.567 INF [miio] accept conn app=bluetooth\n21:04:43.818 INF [miio] accept conn app=central\n'
2024-03-02 09:06:57 [D] <IP_ADDRESS> [BASE] openmiio: latest version detected
2024-03-02 09:06:57 [D] <IP_ADDRESS> [BASE] MQTT connected
2024-03-02 09:06:57 [D] <IP_ADDRESS> [LUMI] 0x00158d00036b1e36 (0x8b95) recv {'online': True, 'zigbee': datetime.datetime(2024, 3, 2, 5, 57, 37, 794511, tzinfo=datetime.timezone.utc)}
2024-03-02 09:06:57 [D] <IP_ADDRESS> [LUMI] 0x00158d00044ecad8 (0xb6fc) recv {'online': True, 'zigbee': datetime.datetime(2024, 3, 2, 6, 3, 17, 797118, tzinfo=datetime.timezone.utc)}
2024-03-02 09:06:57 [D] <IP_ADDRESS> [LUMI] 0x00158d000363d67b (0xe3e3) recv
2024-03-02 09:06:57 [D] <IP_ADDRESS> [LUMI] 0x00158d000346c0a0 (0x97ed) recv {'online': True, 'zigbee': datetime.datetime(2024, 3, 2, 6, 2, 17, 816071, tzinfo=datetime.timezone.utc)}
2024-03-02 09:06:57 [D] <IP_ADDRESS> [LUMI] 0x00158d00044ecb0b (0x14e0) recv {'online': True, 'zigbee': datetime.datetime(2024, 3, 2, 5, 12, 7, 817380, tzinfo=datetime.timezone.utc)}
2024-03-02 09:07:27 [D] <IP_ADDRESS> [BASE] Run zigbee parent scan process
2024-03-02 09:07:28 [D] <IP_ADDRESS> [BASE] Process zigbee parent scan response
2024-03-02 09:07:28 [D] <IP_ADDRESS> [BASE] Total zigbee devices: 5
2024-03-02 09:07:44 [D] <IP_ADDRESS> [LUMI] 0x00158d000363d67b (0xe3e3) recv {'plug': False, 'chip_temperature': 14, 'power': 0, 'energy': 3129.59, 'new_resets': 0, 'resets': 176, 'fw_ver': 90}
2024-03-02 09:07:46 [D] <IP_ADDRESS> [LUMI] 0x00158d00044ecb0b (0x14e0) recv {'battery_voltage': 2915, 'battery': 43, 'battery_original': 66, 'new_resets': 0, 'resets': 35172, 'temperature': 0.36, 'humidity': 86.87, 'parent': '-', 'fw_ver': 2}
2024-03-02 09:10:28 [D] <IP_ADDRESS> [BLEE] a4c138102e3b (1371) recv {'battery': 50}

MQTT connection breaks. You have problems with your network. Maybe signal, maybe router software, maybe router settings. There can be no other options.
https://github.com/AlexxIT/XiaomiGateway3#network-configuration


Investigate integration with Remix

It should be possible to dynamically load new ABIs (it is missing watching changes in the contracts folder). Search for approaches how to integrate with Remix or other IDEs.

Out of scope for now


Georgia-hca-weekly-updates

Heya @malvikasharan - can you review my PR? Would be great!!! Also will do yours if you like :) My meeting with Kirstie is at 4:30 x G

@GeorgiaHCA This looks fantastic. So many communities and technical stuff done over the last weeks. 🙌 Damn welcome bot for creating such a brain-fish (that's a legit word). I hope someone can help sort that issue - I still think it is something to do with the configuration as that's the only thing we don't have complete info of. All the best with the community strategy and newsletter. I have to do a newsletter for The Turing Way too for early next week. I have schedule 11:00 - 12:30 tomorrow to work on it, so feel free to ping me if you want to co-work 👯♀

🤣 "brain-fish" is such a great word, love it! & yep 11-12:30 sounds great - let's do it!
Thanks for the lovely review and all your support over the last couple of weeks 🌟


⚠ Aither has degraded performance

In f1f7030, Aither ($AIH) experienced degraded performance:

HTTP code: 200
Response time: 4809 ms

Resolved: Aither performance has improved in d6812ed after 14 minutes.


TCD population fix

A few other changes as well.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

26 of 26 (100.0%)  changed or added relevant lines in 13 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 93.11%

Totals

Change from base Build<PHONE_NUMBER>9:
0.04%

Covered Lines:
1527

Relevant Lines:
1640

💛  - Coveralls


Add IP injections to procedures

NOT a Brain Injection, NOT viruses
subject procedure
material
dose
volume?

IP = Intraperitoneal injection

Make it an instance of the Injection class - then I think the only additional field you need is volume and volume unit. We have an open issue to refactor the injection material that will make that work better and cover most of the other things listed here.


Promote and Demote Broken

Description
The promote.js ranks a user higher than what the next rank is, this is the same as demote. Another thing is that it says it does not have permission to promote/demote the user although the bot has all permissions on the group. This might be because it's trying to demote/promote people 3 rolesets lower/higher. Please take a look into this.
To Reproduce
Steps to reproduce the behavior:

Use the promote/demote command.
2.It should rank them higher than the actual next rank in the group, for example: 3 rolesets higher.
Keep doing promote/demote and there should log an error.
See error in bot console.

Expected behavior
To promote/demote the person one roleset instead of multiple.
Screenshots
https://you-shmuck.is-inside.me/GOcnBB1F.png , groupId: 5783745
Additional context
Nothing else, thanks for reading.

I've looked into my code and I see nothing wrong, maybe it's the noblox library.


laravel-shopify can't override the homepage route.

Wiki

404

What is the code that you are writing? What is your expectation of what should happen? What errors are you encountering? Just linking to the wiki of the project page isn't very helpful.

Hi @lyongdee
Does \OhMyBrew\ShopifyApp\ShopifyAppProvider::class, come BEFORE or AFTER App\Providers\RouteServiceProvider::class, in config/app.php?
I believe Laravel is seeing the package's GET / route first and ignoring yours. So, placing the \OhMyBrew\ShopifyApp\ShopifyAppProvider::class, AFTER
App\Providers\RouteServiceProvider::class, should solve your issue :)

@ohmybrew Yes, that's right.
\OhMyBrew\ShopifyApp\ShopifyAppProvider::class  must after
App\Providers\RouteServiceProvider::class
in app.php.
'providers' => [     ...         \OhMyBrew\ShopifyApp\ShopifyAppProvider::class,         App\Providers\RouteServiceProvider::class,     ]


Bump Component Detection version

Bumping Component Detection version in order to support the new env variables to override the pip detection behaviour.
Also resolves #622

@microsoft-github-policy-service agree company="Microsoft"

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

Project coverage is 58.40%. Comparing base (91b1b41) to head (c591527).

:exclamation:  There is a different number of reports uploaded between BASE (91b1b41) and HEAD (c591527). Click for more details.
HEAD has 2 uploads less than BASE

Flag
BASE (91b1b41)
HEAD (c591527)

3
1

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #624       +/-   ##
===========================================
- Coverage   68.47%   58.40%   -10.08%
===========================================
  Files         273      273
  Lines        8445     8445
  Branches      990      990
===========================================
- Hits         5783     4932      -851
- Misses       2148     3085      +937
+ Partials      514      428       -86

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


ARROW-7273: [Python][C++][Parquet] Do not permit constructing a non-nullable null field in Python, catch this case in Arrow->Parquet schema conversion

This was the simplest triage I could think of. Fixes a DCHECK failure in debug / segfault in release builds

+1


No module named 'secrets'

I followed the instructions correctly, including:
python3 -m pip install -r requirements.txt --user

But...
user@Matrix:~/Programs/SocialFish$ python3 SocialFish.py
Traceback (most recent call last):
  File "SocialFish.py", line 7, in <module>
    from core.dbsf import initDB
  File "/home/user/Programs/SocialFish/core/dbsf.py", line 3, in <module>
    from core.genToken import genToken, genQRCode
  File "/home/user/Programs/SocialFish/core/genToken.py", line 1, in <module>
    import secrets
ImportError: No module named 'secrets'

You're saying the secrets module isn't installed.
You will try to install manually:
user@Matrix:~/Programs/SocialFish$ pip install secrets --user
Collecting secrets
  Could not find a version that satisfies the requirement secrets (from versions: )
No matching distribution found for secrets

The python-secrets' official  @documentation has it installed as follows:
python3 -m pip install python_secrets

Still does not work
Desktop:

OS: [Linux Elementary OS 0.4.1 Loki, Built on "Ubuntu 16.04.5 LTS"]
Python Version [Python 3.5.2]
Pip Version [pip 19.0.3]

PREREQUISITES to run:
Python 3.6 or >

Hello,
Please upgrade your python version to a compatible one.

i am running secrets on python version 3.10.6, still i am facing the same issue.


ModuleNotFoundError: No module named 'external.metrics.StructuralLosses'

Hi, I have updated the installation guide for StructuralLosses.
Please let me know if it works.

sovled it.

Solved it.


Add http client factory and user agent to identity provider requests

When using MSAL or ADAL authentication libraries no attempt is done to populate the UserAgent header when making requests, and no great API is provided to easily set this. The only mechanism is to implement their IHttpClientFactory interface and customize the HttpClients being used by the library. As such I'm taking this opportunity to use this factory to return a singleton instance for use in all places that we're making HTTP requests.
Additionally I'm updating the UserAgent header we're sending to include more information about the platform and runtime, which now produces a header that looks like the following: User-Agent: CredentialProvider.Microsoft/1.0.4+92842bd4d33ce1b547f8f8dd2a7463bc8049eab1 (Windows; X64; Microsoft Windows 10.0.19045) CLR/6.0.16 (win10-x64; .NET 6.0.16)

Can you file an issue in MSAL? There should be some default...


Error in NavigationRail code sample

There is an error  in the NavigationRail code sample https://flet.dev/docs/controls/navigationrail#examples.
The code
destinations=[
    NavigationRailDestination(
        icon=ft.icons.FAVORITE_BORDER, selected_icon=ft.icons.FAVORITE, label="First"
    ),
    NavigationRailDestination(
        icon_content=ft.Icon(ft.icons.BOOKMARK_BORDER),
        selected_icon_content=ft.Icon(ft.icons.BOOKMARK),
        label="Second",
    ),
    NavigationRailDestination(
        icon=ft.icons.SETTINGS_OUTLINED,
        selected_icon_content=ft.Icon(ft.icons.SETTINGS),
        label_content=ft.Text("Settings"),
    ),
],

has to be changed to
destinations = [
    ft.NavigationRailDestination(
       icon=ft.icons.FAVORITE_BORDER, selected_icon=ft.icons.FAVORITE, label="First"
    ),
    ft.NavigationRailDestination(
       icon_content=ft.Icon(ft.icons.BOOKMARK_BORDER),
       selected_icon_content=ft.Icon(ft.icons.BOOKMARK),
       label="Second",
    ),
    ft.NavigationRailDestination(
       icon=ft.icons.SETTINGS_OUTLINED,
       selected_icon_content=ft.Icon(ft.icons.SETTINGS),
       label_content=ft.Text("Settings"),
    ),
],

The ft. is missing in front of the NavigationRailDestination.

@FeodorFitsner I just saw in the Card sample (https://flet.dev/docs/controls/card#examples) there is the same problem. There is the TextButton("Buy tickets") button which is also missing .ft.

you can contribute by opening Pull requests with the appropriate changes.

@MMartin09 it's been fixed too, thank you!


Crosswalk for ROI Qual Interviews Study

Creating a github issue for the creation of a crosswalk between the sampling frame masked IDs and their corresponding Connect IDs for the ROI Qualitative Interviews Study. Selection criteria and requirements discussed between Nicole and Jake.
ROI Qualitative Interviews Requirements + Invitation Strata: https://nih.app.box.com/folder/288120201144
@FrogGirl1123 @jacobmpeters  please add additional info or questions here, if needed.
Also tagging @m-j-horner here, as I was not able to add her as an assignee.

RE: Masked ID's
I set up a table called connect_id_crosswalk in BigQuery that we will use for tracking the "alternative Connect_ID's" or uuids for this use case and all future data-sharing use cases. I've pasted the schema below.
Once @BrittanyCrawford405 has a complete list of Connect_ID's I will add them to this connect_id_crosswalk table and generate the uuids.

NOTE: I will be on PTO 10/17-10/18 and will be at the OHDSI Symposium 10/22-10/24. So the I will need the final Connect_ID list by Monday, 10/21.

BigQuery Table Schema
table_id: nih-nci-dceg-connect-prod-6d04.connect_id_crosswalk.connect_id_crosswalk

Field Name
Type
Mode
Description

uuid
STRING
REQUIRED
Unique identifier assigned to a participant-request pair using GENERATE_UUID().

Connect_ID
INTEGER
REQUIRED
Unique identifier used for a given Connect Participant.

service_now_request_id
INTEGER
NULLABLE
Unique identifier associated with a Service Now ticket in which a dataset was requested by a stakeholder. If github_issue_url is NULL, then service_now_request_id is REQUIRED.

github_issue_url
STRING
NULLABLE
URL associated with the GitHub issue in which a data request is documented. If service_now_request_id is NULL, then github_issue_id is REQUIRED.

snapshot_id
STRING
NULLABLE
An identifier associated with the BigQuery snapshot that was live at the time the data request was fulfilled.

creation_date
DATE
REQUIRED
Timestamp of when the UUID was generated.

crosswalk_id
STRING
NULLABLE
Unique identifier assigned to a data request using GENERATE_UUID().

Hello @jacobmpeters DCCPS has approved the selection and this is ready for the masked IDs and Crosswalks to be run and finalized. I have created boxed folders for these to be placed here below. Brittany's selection list for the initial batch can be found here: https://nih.app.box.com/folder/291418005167
For the masked ID list, please make sure there are no Connect IDs or identifiers in the list prior to placing in the folder as I have shared this folder with DCCPS team already.
Masked ID List (restricted access folder for DCCPS, NORC, CCC): https://nih.app.box.com/folder/291416584548
Crosswalk (to be shared with NORC and CCC only) in NORC-DCEG folder: https://nih.app.box.com/folder/291417376177

@robertsamm Thanks for your notes here. I can do this this afternoon, but I have some questions.

Is the "selection list" the same as the "sampled_data.csv" in this box folder you linked to? There are two files in that folder, so I want to be certain.
The "Masked ID List" is simply the sampled_data.csv file but with a uuid instead of a Connect_ID, correct?
My plan for the crosswalk was to store all of this information in a BigQuery table (as I outlined above) and not share this outside of the CCC. Is this still ok? This is the plan that Davin, @FrogGirl1123 , Tony and I agreed to when we initially discussed the Concept ID Crosswalk solution.

Hi @jacobmpeters see my responses below

Yes, the selection list is the 'sampled_data' file. Thanks for checking!
Yes, I believe the masked ID list is the uuid unique identifier (without a Connect ID)
For the crosswalk, we will need to share the crosswalk with NORC as they will be sending the recruitment invitations for everyone invited to the Qual ROI study using the contact info they provided for Connect. They will need both the masked IDs and the Connect IDs for this. I think the updated plan was to provide the crosswalk on both Box (with limited access) and GCP. Once the study is over, the crosswalk would be removed from Box and would be stored indefinitely on GCP. @FrogGirl1123 @m-j-horner @brotzmanmj can you confirm the plan for this?

@robertsamm Thanks for the clarification on this. This makes sense to me. I'm working on this now.

Once the study is over, the crosswalk would be removed from Box and would be stored indefinitely on GCP. @FrogGirl1123 @m-j-horner @brotzmanmj can you confirm the plan for this?
Yes- when the interview study is over, the crosswalk should be removed from NORC's Box folder as they will have no further use for it.

The Crosswalk file has been delivered to the appropriate folder on Box: https://nih.app.box.com/file/1686092574162
The data with the masked ids has been uploaded to the Box folder containing Brittany's original file: https://nih.app.box.com/file/1686091418050
If it looks ok to everyone, I can move it to the folder that DCCPS has access to.

Hi @jacobmpeters thanks for creating! Just to clarify, once we finalize and the masked IDs and crosswalk are approved, we'll only be providing a list with the masked IDs field (and no other data) to the folder DCCPS has access to as we won't provide any demographic data to DCCPS until recruits are consented to the study.
@FrogGirl1123 can you please review and approve the crosswalk and masked ID participation list Jake created above? One note I have is for the UUIDs/masked IDs. This ID will be put on the invitation email NORC sends out and recruits will be asked to verbally share the ID with DCCPS at the time of consent (this will be how we link them back). Given this, do you think the number of characters in the UUID should be shortened?

@robertsamm Thanks for clarifying that point! This part of the requirements document was difficult to understand. I will only provide a single column in that file containing the uuid and no other data.
@FrogGirl1123 Regarding the uuid's, we agreed to use the GENERATE_UUID function in BigQuery to generate the uuid's so that we could easily ensure that they are universally unique across all crosswalk use cases. I'd like to use the same method for every use case so that we don't produce a series of one-off solutions which would become unmanageable over time.

@jacobmpeters @FrogGirl1123 are the first 4 digits unique? If yes, could we provide recruits with just the first 4 as their invitation ID/masked ID?

Hi @jacobmpeters yes, that does make sense and agree we should retain the entire UUID as the official masked ID, since we will have many of these in the future. A small edit to the digit length that we give recruits, should we make it 5 digits instead of 4 to differentiate from the length of the PIN they receive at time of Connect study invitation?

@robertsamm That makes sense to me. OK, my current understanding is that I will put the whole uuid to in the data files and NORC will share the first 5 digits with the participants. Do we agree?

@/all The plan makes sense. I do prefer the 5 digits as the improves the odds of any given data set having unique ids. @jacobmpeters I wonder if we should add a column for the "Invite IDs", just in case we ever have to move to 6 digits or change it in some way?

Hi @jacobmpeters and @FrogGirl1123 That all sounds good to me and Michelle. I like the idea of having a column for "Invite IDs" to limit confusion on DCCPS end so that the list they have matches with what is provided to the recruit.

@FrogGirl1123 @robertsamm
The two requirements of the crosswalk that were stated to me are

Ensure that Connect_ID's are not overused so that they do not effectively become PII.
Ensure that the masked ids (i.e., uuids) are unique across all data sharing use cases so that if there is a data breach, we can easily trace it back to the party that we shared the data with.

Both the uuid and the invite_id meet the first requirement.
However, using the shortened invite_id in any data files without the uuid does not meet the second requirement.
So as long as they are both included in the data files, we should be ok.
I'll update the files in Box folder for one more look before moving them to the final folders.

Hello @jacobmpeters DCCPS has approved the selection and this is ready for the masked IDs and Crosswalks to be run and finalized. I have created boxed folders for these to be placed here below. Brittany's selection list for the initial batch can be found here: https://nih.app.box.com/folder/291418005167
For the masked ID list, please make sure there are no Connect IDs or identifiers in the list prior to placing in the folder as I have shared this folder with DCCPS team already.
Masked ID List (restricted access folder for DCCPS, NORC, CCC): https://nih.app.box.com/folder/291416584548 Crosswalk (to be shared with NORC and CCC only) in NORC-DCEG folder: https://nih.app.box.com/folder/291417376177

@robertsamm
I have become a little confused about who is supposed to receive what information. The requirements document was not clear to me, so I discussed with Nicole.
DCCPS should receive in the “masked file” here (https://nih.app.box.com/folder/291417376177):

uuid
invite_id

NORC should receive in “crosswalk file” here (https://nih.app.box.com/folder/291416584548):

Connect_ID
uuid
invite_id
no other information

Hi @jacobmpeters Michelle and I discussed and we would like to restrict the access further for the crosswalk file. Here's the updated paths.
DCCPS + NORC should receive in the “masked file” here (https://nih.app.box.com/folder/291416584548):

uuid
invite_id (a five character truncation of uuid)
no other information

NORC should receive in “crosswalk file” here (https://nih.app.box.com/folder/291611715929):

Connect_ID
uuid
invite_id
no other information

The previous subfolder for the crosswalk in NORC-DCEG folder will be deleted. I'll also update the documentation in the requirements doc to be more clear!

The files have been delivered to the specified folders.
For future reference, the code for generating these files lives here: https://github.com/Analyticsphere/ad-hoc/blob/main/operations/generate_crosswalk_for_roi_qualitative_interviews.R

Awesome, thanks @jacobmpeters!

Hi @robertsamm. Thank you for the notice.
I can do this either way. The second option is likely the simplest (n=100), but either option is doable if you have a preference.

Good morning @jacobmpeters it would be good to keep the list together (n=100) so that we have the full list of recruits in one place. Is it possible to indicate on each list batch 1 and batch 2 so that it's clear to DCCPS and NORC which IDs to use for the 2nd batch of invitations without having to cross-reference the initial list?

Good morning, DCCPS approved the selection demographics for the 2nd batch. The masked ID list and crosswalk list are ready to be created based on the selection here: https://nih.app.box.com/file/1701314204943

@robertsamm Apologies for the delay. I have been OOO unexpectedly since Nov. 14.
@BrittanyCrawford405 Were you able to run the second batch of masked ID's? Do you need any assistance?

Hi @jacobmpeters , yes! I was able to run the second batch of masked IDs and uploaded them to Box on Tuesday.

@BrittanyCrawford405 Thank you for filling in on that! I appreciate it.

@BrittanyCrawford405 Thank you for filling in on that! I appreciate it.

Of course!


du -l actually has a long form on linux, which is now used as well.

On MacOS/BSD it doesn't though, but that platform isn't used for benchmarks.

It's more correct to say GNU's du has long form.


Adjust network timeout for mobile to shorter period

To reduce lag in waiting for slow nodes to respond, we will reduce the network timeout for a direct message to be delivered. This network timeout should only affect the mobile server configuration.
TBD: what the new timeout should be? 10 seconds? @cpacia ?

10 seconds?

Is there a risk then that 95% of messages will fail?
If they fail, is there a retry process?

There's always a risk of failure for every timeout value because this value optimizes between the two extremes of highly-available nodes responding quickly and a responsive UI when the node is not available. We can make a more robust retry method but it would need to be something which retries after the offline message is sent and also doesn't block the UI. Certainly something to discuss, but as a short term solution which would only affect the mobile server this seems like a simple test we can try.

10 seconds is still pretty long for mobile... is there a way we might make this configurable in the mobile package so I can drop down the timeout to 5 seconds?

So does this work where it tries to send an online message for this amount of time and if that fails, it gets delivered as an offline message?

@rmisio speaking for mobile, the node will try and send an online message to the peer (and in most cases fail), before trying to send an offline message; that process takes 30 seconds... and if I understood @cpacia correctly, that 30 seconds is the timeout (as sending the offline message is done in a separate goroutine in the background).
For mobile, reducing it to 5 seconds is probably ok... but ideally this is something we can configure specifically for mobile, as other clients won't have this problem.

Merged.


Changing Virtual Desktop causes TaskbarX to reload

Describe the bug
While changing virtual desktops on windows 10 feels like it is being reloaded. I don't know if it's a windows 10 issue or not. I've linked a video below.
Screenshots
https://user-images.githubusercontent.com/51121313/144092897-00d71559-8a83-4dbd-b805-896b60ca256b.mp4
Desktop (please complete the following information):

OS: Windows 10
Version 21H2 (OS Build 19044.1348)

@isayakmondal This happens because the amount of icons on your desktop(s) are different. It's just centering them again.


add test for ecies

some happy path test are in relay crypto, could be ported from there or moneybutton/bsv

@relayking see branch https://github.com/remjx/relay-browser-extension/tree/ecies-test, the ported relay crypto tests are failing