Problem building on Windows

Hi,
Thanks for this great contribution, I'm trying to build the repo on windows but I found out that NCCL is not supported on Windows system so I can't build it.
If there is a idea to build library for single gpu without support of NCCL and adding support to windows I would like to contribute for that

Hi,
Thanks for your comments, we have added a new compilation option (i.e. -DMEGBA_USE_NCCL) in https://github.com/MegviiRobot/MegBA/pull/28. Check the new updated README.md. If you can help with the compilation on Windows, you can fork the repo and create the pull request. We are happy to involve you in MegBA.
Best regards,
Jie

That's great I will work on it on this weekend

Look forward to your good news :-)
Jie

I have finished and it build on windows but there is some major changes in CMakeLists.txt like:

Added installation and packaging using CPack
Used Target-based function instead of global ones (for ex. use target_include_directory instead include_directories)
Used find_package(CUDAToolKit) instead of deprecated find_package(CUDA)
Added CMakePresets to be easier to automate build and for continuous integration(GitHub/GitLab pipelines)
So Should the pull request be only on the windows part or all the changes that makes building on Windows and Linux consistent ?

Hi,
Nice job. What's the meaning of "building on Windows and Linux consistent", I guess it may be using the if-else statement in one "CMakeLists.txt" to judge the system and execute the relevant code. If so, I think making it consistent would be better.
Best regards,
Jie

Hi,
I have merged your pull request. Nice job!
There are several problems in the building stage, check the newly opened pull request (https://github.com/MegviiRobot/MegBA/pull/31). 1) I have difficulty finding thrust, I have to add flag Thrust_DIR to fix it; 2) there is something wrong with the linking of gflags, I modified "examples/CMakeLists.txt"; 3) the header of Eigen3 is not included, I use "include_directories(${EIGEN3_INCLUDE_DIRS})" to fix it temperately. Please polish my pull request, thanks.
Jie


Skip fbc-related-image-check for on-pullrequest phase

The fbc-related-image-check takes about 30 minutes to finish and is supposed to always fail for on-pullrequest checks. Run this only on-push where the long run time is fine. When the regular images (operands) are released to registry.redhat.io we will re-run this check for the commit that will reference the images and then it should pass.

/lgtm


Expecting member declaration

Flutter run in android emulator and throw error 'barcode_scan-1.0.0/android/src/main/kotlin/com/apptreesoftware/barcodescan/BarcodeScannerActivity.kt: (23, 9): Expecting member declaration'

Does the error is also shown in the latest version? (v2.0.1)


Docs: Add deployment documentation

Summary
Create documentation around deployment of Litestar applications.
This would include deployment via

Manually via uvicorn, hypercorn, daphne, granian, etc.
gunicorn with uvicorn workers
Baremetal/VM with something like systemd unit files
Via supervisor
Containers/LXC
K8s
Behind a proxy (NGINX/Traefik/HaProxy/etc)
Deployment through various cloud providers
Any other deployment mechanism that may be beneficial.

It should probably go in Usage section documentation, or topics.

[!NOTE]
While we are open for sponsoring on GitHub Sponsors and
OpenCollective, we also utilize Polar.sh to engage in pledge-based sponsorship.
Check out all issues funded or available for funding on our Polar.sh Litestar dashboard

If you would like to see an issue prioritized, make a pledge towards it!
We receive the pledge once the issue is completed & verified
This, along with engagement in the community, helps us know which features are a priority to our users.

This can be done in steps by many people. If you know how to deploy with systemd unit files, then right just that section. We can add on in pieces.

It should probably go in Usage section documentation, or topics.

It should go into "Topics"; That's what this section was originally intended for.

Inspiration from various good resources:

https://flask.palletsprojects.com/en/3.0.x/deploying/
https://fastapi.tiangolo.com/deployment/
https://docs.djangoproject.com/en/4.2/howto/deployment/

so let me give this a bit of a stab with supervisor (i stopped using systemctl cause you cant actualy "watch" what its doing)
im on ubuntu and i installed (git pulled lol) the repo with the api into /opt/api so please change your paths accordingly. i call my app "api" as well so again change as you need. all references to api here refer to the service and not the actual json thing
this is assuming the venv is setup (python3.12 install -m venv venv) and the requirements.txt is loaded (pip install -r requirements.txt once activating the venv)

place the actual litestar code in an app.py file. app = Litestar() however your app is setup
create a main.py  file with
import uvicorn

if __name__ == "__main__":
    uvicorn.run(
        "app:app",
        port=80,
        host="<IP_ADDRESS>",
    )

if you use a proxy infront of your api then use this replacing your proxy servers ip
import uvicorn

if __name__ == "__main__":
    uvicorn.run(
        "app:app",
        port=80,
        host="<IP_ADDRESS>",
        proxy_headers=True,
        forwarded_allow_ips='xx.xxx.xxx.xxx'
    )

at this point you should be able to just python main.py and it works (when in the venv)

supervisored file (create a file and place it in /etc/supervisor/conf.d/api.conf
[program:api]
directory=/opt/api/app
command=/opt/api/venv/bin/python main.py
#stdout_logfile=/dev/fd/1
#stdout_logfile_maxbytes=0
redirect_stderr=true
#stderr_logfile=/var/log/munsync2.log
stdout_logfile=/var/log/api.log
stdout_logfile_backups=10
autostart=true
autorestart=true

once done, reload the config with
sudo supervisorctl reread
sudo supervisorctl update

you can start stop the service now with
sudo supervisorctl start api
sudo supervisorctl stop api

get the status
sudo supervisorctl status api

and most importantly of all (for me...) watch the output
sudo supervisorctl tail -f api

your api when started should be accessible on http://yyy.yyy.yyy.yyy:80

for BONUS points... heres a more interesting way of setting it up....
have your code base in a repository somewhere and have git remote add origin blahblahblah so that you can do a git pull origin master yes... im oldschool you wokers
make sure you have already done a git pull
have these files as part of your project (in the repo)

service.conf is the same boring as previous post
alias.sh
api() {
  case $1 in
    start)
      echo "Starting"
      sudo supervisorctl start api || true
      ;;
    stop)
      echo "Stopping"
      sudo supervisorctl stop api || true
      ;;
    restart)
      echo "Stopping"
      sudo supervisorctl stop api || true
      sleep 2
      echo "Starting"
      sudo supervisorctl start api || true
      ;;
    status)
      echo "Status"
      sudo supervisorctl status api || true
      ;;
    watch)
      sudo supervisorctl tail -f api
      ;;

    update)
      echo " > Stopping"
      sudo supervisorctl stop api || true

      echo " > Updating files"
      cd /opt/api
      git reset --hard origin/master
      git pull origin master

      sleep 2

      echo " > Linking supervisord service file"
      sudo ln -sf /opt/api/server/service.conf /etc/supervisor/conf.d/api.conf
      echo " > Linking service alias"
      sudo ln -sf /opt/api/server/alias.sh /etc/profile.d/api.sh
      source /etc/profile.d/api.sh

      sleep 2

      echo " > Updating supervisord services"
      sudo supervisorctl reread
      sudo supervisorctl update

      sleep 2

      source venv/bin/activate
      echo " > Updating dependencies"
      pip install -U -r requirements.txt

      echo "------------------------"
      echo "Done"

      read -p "Start the service? (y/n) " -n 1 -r
      echo    # (optional) move to a new line
      if [[ $REPLY =~ ^[Yy]$ ]]
      then
          echo "Starting"
          sudo supervisorctl start api || true
      fi
      ;;

    help)
      echo "Available options:"
      echo "  api start"
      echo "  api stop"
      echo "  api restart"
      echo "  api status"
      echo "  api watch"
      echo "  api update"
      ;;

    *)
      cd /opt/api
    ;;
  esac
}

setup.sh
echo "Setup"

echo "Creating service alias"
sudo ln -sf /opt/api/server/alias.sh /etc/profile.d/api.sh
source /etc/profile.d/api.sh

api update

this ... kinda adds some awesome..
Available options:
  api start
  api stop
  api restart
  api status
  api watch
  api update

to run setup just >sh setup.sh should do the trick. then just update via api update

to build an actual exe on windows. i did this as a total proof of concept so its guaranteed to be busted... it works.. but...
this uses pyinstaller
in the root of your project
api.spec
# -*- mode: python ; coding: utf-8 -*-

a = Analysis(
    ['app\\main.py'],
    pathex=['app'],
    binaries=[],
    datas=[('./app/static/*', './static')],
    hiddenimports=['app', 'cryptography'],
    hookspath=[],
    hooksconfig={},
    runtime_hooks=[],
    excludes=[],
    noarchive=False,
)
pyz = PYZ(a.pure)

exe = EXE(
    pyz,
    a.scripts,
    a.binaries,
    a.datas,
    [],
    name='api',
    debug=False,
    bootloader_ignore_signals=False,
    strip=False,
    upx=True,
    upx_exclude=[],
    runtime_tmpdir=None,
    console=True,
    disable_windowed_traceback=False,
    argv_emulation=False,
    target_arch=None,
    codesign_identity=None,
    entitlements_file=None,
    icon=['app\\static\\icon.ico'],
)

build.py
import PyInstaller.__main__

PyInstaller.__main__.run(
    [
        'app/main.py',
        '--name=api',
        '--onefile',
        '--clean',
        '--paths=app',
        '--hidden-import=app',
        '--hidden-import=cryptography',
        '--icon=./app/static/icon.ico',
        '--add-data=./app/static/*:./static'
    ]
)

python build.py should give you a dist folder.
im "slightly" too lazy to go through and see what requirements are required for it. i think its just pyinstaller extra but i can be mistaken

docker
Dockerfile
FROM python:3.12

WORKDIR /code

COPY ./requirements.txt /code/requirements.txt

RUN pip install --no-cache-dir --upgrade -r /code/requirements.txt

COPY ./app /
CMD ["uvicorn", "app:app", "--host", "<IP_ADDRESS>", "--port", "80"]

compose.yml
services:
  api:
    build:
      context: ./
      dockerfile: Dockerfile
    container_name: "api"
    depends_on:
      - database
    ports:
      - "6969:80"

  database:
    image: mariadb:latest
    container_name: "database"
    environment:
      MARIADB_USER: api-user
      MARIADB_PASSWORD: api-user-password
      MARIADB_ROOT_PASSWORD: free-willy
      MARIADB_DATABASE: api
    ports:
      - "4001:3306"

obviously the db here isnt needed. again im too lazy to even have removed it...
docker-compose up -d

*drops mic*

uhmmmf rst looks funky :(


V2.19.0 change to dbname

Description

Motivation and Context

Breaking Changes

How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects

[ ] I have executed pre-commit run -a on my pull request

Closing - no information provided in PR, not sure what this is intending to do


[utils] update -- chart-grid-border token for dark theme

Motivation and Context
Fixed --intergalactic-chart-grid-token for dark.json for better-looking charts in dark theme on our website.
Types of changes

[x] Nice improve.

Checklist:

[x] My code follows the code style of this project.
[x] I have updated the documentation accordingly or it's not required.
[x] I have added changelog note to corresponding CHANGELOG.md file with planned publish date.

Merged locally into https://github.com/semrush/intergalactic/pull/967


flexible mlp policies (shared and non-shared)

This PR solves part of #91. The LstmPolicy is untouched but FeedForwardPolicy supports the new format to specify network architectures as discussed in #91.
This PR also includes:

[x] documentation in code
[x] documentation in the user guide
[x] (visual) examples in the user guide
[x] warnings when the old layers parameter is used
[x] update of all the documentation to the new net_arch format instead of the old layers format
[x] update of all test code to use the new net_arch format

Honestly, the only good way to fix this issue, it by fixing the save and load methods, so that loading by name can be done.
In the mean time, it's annoying and ugly, but you can do this:
latent_policy = latent
latent_value = latent
for idx in range(max(len(policy_only_layers), len(value_only_layers))):
    if len(policy_only_layers) < idx:
        assert isinstance(policy_only_layers[idx], int)
        latent_policy = act_fun(linear(latent_policy, "pi_fc{}".format(idx),
                                       policy_only_layers[idx], init_scale=np.sqrt(2)))
    if len(value_only_layers) < idx:
        assert isinstance(value_only_layers[idx], int)
        latent_value = act_fun(linear(latent_value, "vf_fc{}".format(idx),
                                      value_only_layers[idx], init_scale=np.sqrt(2)))

@hill-a can't we rather fix the save function?

@hill-a I implemented your suggestion using zip_longest from the itertools. This makes it slightly more readable. All other requests were resolved I think. Now lets wait for the CI.
@hill-a @araffin thank you for the valuable feedback and debugging the loading issue!

Great!
If it makes matters simpler, I am happy to become a maintainer!

@erniejunior cool =), welcome to the team then ;)

Invitation sent, Merging PR


'aws_iam_role.default' not found

I've used the default assumerole role name
OrganizationAccountAccessRole
data "aws_iam_role" "ecr" {
  name = "OrganizationAccountAccessRole"
}

and got the following error planning with TF 0.11
Error: Error running plan: 1 error(s) occurred:

* module.ecr.output.role_name: Resource 'aws_iam_role.default' not found for variable 'aws_iam_role.default.name'

@FernandoMiguel  thank you for evaluating the module.
Please update to Terraform 0.11.1.
It's a known issue in Terraform 0.11.0.
They made it a fatal error in 0.11.0 to reference a resource with count=0 even if that resource is not used.
https://github.com/hashicorp/terraform/issues/16726
But then switched it to warning in 0.11.1
https://github.com/hashicorp/terraform/blob/v0.11.1/CHANGELOG.md
Since the new feature was already rolled out, and since development for 0.12 depends on the foundational changes that enabled it, we decided to compromise with an opt-out mechanism in 0.11.1, so those with configurations containing problematic output expressions have a means to use 0.11.1 without first fixing all modules. We understand that this is not the most ideal migration path -- if we could do this over again we would've introduced the warning in one of the 0.10 point releases -- but we hope that this compromise is acceptable so that we can continue to make progress towards 0.12.
To enable that behavior, you need to set the variable TF_WARN_OUTPUT_ERRORS=1
They mentioned they will make it an error again in 0.12, so it's a temporarily fix anyway.
But if they fix ternaries (to short-circuit and not evaluate left and right part at the same time), we can work around the issue very easily
https://github.com/hashicorp/hil/issues/50
Please update to 0.11.1 and set TF_WARN_OUTPUT_ERRORS=1.
We'll close the issue for now, please let us know if that's addresses your question and if you have other questions or concerns.

i'll give it a try again.
thanks


Column totals in the Exported file differ from totals displayed on Toolbox->Grid->Tree screen

Navigate to Toolbox -> Grids -> Tree (or Tree w/Checkbox or Grouped Columns) screen
Click "Export" icon
Open file
Review column totals

Navigate to Toolbox -> Grids -> Tree (or Tree w/Checkbox) screen

Navigate to Toolbox -> Grids -> Tree (or Tree w/Checkbox) screen

Peter -- can see if you can reproduce this?  Want to take advantage of your deep love of excel.

I can reproduce this. It occurs if you follow the steps described in the issue with a filter enabled.

Found an additional problem: the exported values are not formatted the same as the values displayed in the browser. For instance, the TreeGrid example for this issue formats market value as millions of dollars, and the header states that the units are millions. However, when exported, the data is not formatted as millions.
Maybe we don't want to format data when exported since that throws out some data. In that case, the headerName of the exported data inaccurate.

PR is here:
https://github.com/xh/hoist-react/pull/1488
This was actually somewhat worse than I thought. I do not know where the data we were previously using came from and it didn't work properly. For instance, it does not reflect the configured aggFunc: if the aggFunc is max, the exported version would still be a sum.

The aggregation data that was being exported before came straight form the PositionService, which calculates the aggregates server-side.

Great fix 👍
Thanks!


pyenv rehash segmentation fault

while installing a package I came acroos the segfault:
[root@billme test]# pip install newrelic
Collecting newrelic
Downloading newrelic-<IP_ADDRESS>.tar.gz (662kB)
99% |███████████████████████████████▉| 659kB 8.2MB/s eta 0:00:01/root/.pyenv/plugins/pyenv-pip-rehash/libexec/pip: line 20:  9261 Segmentation fault      "$PYENV_COMMAND_PATH" "$@"
[root@billme test]# pip --version
pip 6.1.1 from /root/.pyenv/versions/2.7.8/lib/python2.7/site-packages (python 2.7)
No idea what's going on here..

Could not reproduce the problem on my installation.
Basically pyenv just invokes pip from its shell scripts, and the problem would be in the package or pip itself.

can install any package with the global version of PIP. there is no problem. My suspect is there is something wrong with the environment variables in CentOS 7. anyway thanks for having a look, I will try to fix it myself ;)


Implement Async API specification v3.0.0

What this PR does:
Implements version 3.0.0 of the Async API specification
Notes to implementers:

[x] Model components
[x] Implement I/O services
[x] Implement fluent builders
[x] Implement document generation
[ ] Create view to render all components in the UI

[x] Create document view
[x] Create server view
[x] Create channel view
[x] Create operation view
[x] Create message view
[x] Create binding collection view
[x] Create binding view
[ ] Create security scheme view
[ ] Create correlation id view
[ ] Create reply view
[ ] Create reply address view
[ ] Create external docs view
[ ] Create tag view
[ ] Create server variable view
[ ] Create component view
[ ] Create operation trait view
[ ] Create message trait view

[ ] Implement fluent validators

@greg-otterpeak @docknight @mothermetabot it would be awesome if you could already take a look at the StreetLightApi project and tell me what you think of the WIP, so that I can already make changes, if need be.
You need to select the adequate spec version (3.0.0) of the documented API, in the dropdown at the top-right of the UI.


Add test for SNS event w/ no identifying event info

This will be used to fix #19 by adding a "catch all" event processor
which will propogate the JSON values to Slack, as opposed to failing
with No matching processor for event.

Awesome, we should be able to handle it. I think something like this will work (untested):
var handleCatchAll = function(event, context) {
    var record = event.Records[0]

    var subject = record.Sns.Subject
    var timestamp = new Date(record.Sns.Timestamp).getTime() / 1000;
    var message = JSON.parse(record.Sns.message)

    // TODO - check for warning here as well
    var color = message.NewStateValue === "ALARM" ? "red" : "green"

    // Add all of the values from the event message to the Slack message description
    var description = ""
    for(key in message) {
        description = key + ": " + message[key] + "<br/>"
    }

    var slackMessage = {
        text: "*" + subject + "*",
        attachments: [
          {
            "color": color,
            "fields": [
              { "title": "Message", "value": message.Sns.Subject, "short": false },
              { "title": "Description", "value": description, "short": false }
            ],
            "ts": timestamp
          }
        ]
    }
}

Notice how it doesn't check for any specific fields in the event, since they could be anything. It just formats the entire message description and adds it to the Slack message.
I'll try and test this in a bit and see if it works (prob needs a couple fixes).


Change swift-argument-parser dependency from 'upToNextMinor' to 'from'

upToNextMinor is no longer needed since swift-argument-parser reached 1.0. Unblocks downstream projects from adopting a newer swift-argument-parser.

Corresponding SwiftPM change: https://github.com/swiftlang/swift-package-manager/pull/7952
sourcekit-lsp already uses from

@swift-ci please test


Cant install @ngneat/transloco:keys-manager via yarn package mananger

I'm submitting a bug

[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report
[ ] Performance issue
[ ] Feature request
[x] Documentation issue or request
[ ] Support request
[ ] Other... Please describe:

Current behavior
in README there are installation instrudtions for ng, but the package name @ngneat/transloco:keys-manager with : is not working for yarn package manager
Expected behavior
Adding a script on how to add it via yarn package manager.
yarn global add @ngneat/transloco-keys-manager

Minimal reproduction of the problem with instructions

following installation instruction and copying package name to yarn provide error

yarn global add @ngneat/transloco:keys-manager
→ yarn global add @ngneat/transloco:keys-manager
yarn global v1.22.5
[1/4] 🔍  Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@ngneat%2ftransloco:keys-manager: Not found".
info If you think this is a bug, please open a bug report with the information provided in
"/Users/maciejsypien/.config/yarn/global/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/global for documentation about this command.

hen replacing : with - installation succeeded.

What is the motivation / use case for changing the behavior?
trying/installing transloco-keys-manager CLI without a need to have angular project installed (via ng)
Environment
For Tooling issues:

Node version: v10.21.0
Platform: MacOS 10.15.6 (19G2021)

Keys manager config
None

If the issue will be generally approved by authors, I can provide a proposition/solution pull request :)

@egel Thanks for suggesting a PR but I'll take this one and create a manual installation guide 🙂


[Feature Request]: Want to Fix Card Design when we search products And ITS WORKING AS WELL

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description

when we search product card design becomes different i will fix that and when we click on that card it is not redirecting to product detail page even these is no add to cart button
Use Case
it will mainatain consistency of design
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

Hey @dinxsh can you review this issue and assign it to me


Import went wrong

I downloaded the .unityPackage and attempted to import it. An error occurred and now the project will not start. Can you tell me what files I could edit to stop the project from trying to import this please?

Managed to delete the files by comparing the project files with a backup. Tried the Windows- > Package Manager -> URL and it similarly failed. Now I have to go remove that one as well.

Hello @tleylan , sorry to hear you had issues importing VRRefAssist, do you remember if you saw any errors or did the project not open at all? Also, all of the files are in Packages/VRRefAssist (or Packages/com.livedimensions.vrrefassist if you are viewing them outside of Unity like explorer.)


RESTler should resolve additionalProperties

Hi, I have an OpenAPI specification containing $ref  in additionalProperties. Following is a simplified version of my swagger spec file (The original can be found here https://github.com/free5gc/Nnrf_NFManagement/blob/master/api/openapi.yaml)
#...
components:
  schemas:
    NrfInfo:
      properties:
        servedUdrInfo:
          additionalProperties:
            $ref: '#/components/schemas/UdrInfo'
          minProperties: 1
          type: object
#...
components:
  schemas:
    UdrInfo:
      example:
        supportedDataSets:
        - null
        - null
        groupId: groupId
        externalGroupIdentifiersRanges:
        - start: start
          pattern: pattern
          end: end
        - start: start
          pattern: pattern
          end: end
        supiRanges:
        - start: start
          pattern: pattern
          end: end
        - start: start
          pattern: pattern
          end: end
        gpsiRanges:
        - start: start
          pattern: pattern
          end: end
        - start: start
          pattern: pattern
          end: end
      properties:
        groupId:
          type: string
        supiRanges:
          items:
            $ref: '#/components/schemas/SupiRange'
          minItems: 1
          type: array
        gpsiRanges:
          items:
            $ref: '#/components/schemas/IdentityRange'
          minItems: 1
          type: array
        externalGroupIdentifiersRanges:
          items:
            $ref: '#/components/schemas/IdentityRange'
          minItems: 1
          type: array
        supportedDataSets:
          items:
            $ref: '#/components/schemas/DataSetId'
          minItems: 1
          type: array
      type: object
#...

After compiling, the servedUdrInfo is rendered as fuzzable_object instead of an UdrInfo Object:
"servedUdrInfo\":{ \"fuzz\": false }
If I resolve the $ref manually, it seems to work.
My question is now: Does RESTler resolve the $ref for additionalProperties?

Hello @c7h,
RESTler does not support additionalProperties right now. It just skips them, and then assigns the default object.
I'll change this issue to 'ehnancement'.  In the meantime, the workaround is as you have done, to reference these properties directly instead of via 'additionalProperties'.
Thanks,
Marina

Hi @marina-p :)
+1 on this ask
In our case, the keys of the object are arbitrary strings, and the values need to compily with a referenced schema. Here is the relevant part of the API spec:
"/configurations/integration/{integration_id}": {
  "put": {
     "requestBody": {
        "content": {
          "application/json": {
            "schema": {
              "type": "object",
              "additionalProperties": {
                "$ref": "#/components/schemas/PolicyConfigurationProperties"
              }
            }
          }
        },
        "required": true
      }
    }
  }

Hi @c7h,
could you share your version of the yaml-file where you resolved the $refs manually? When I do it, the compilation still does not work.

Hello @hidingturtle,
Could you please try out the update 'mp/330' branch and let me know if this fully addresses your issue with additionalProperties?
Thanks,
Marina

Hello @marina-p,
I tried it out and it addresses my issue with additionalProperties! Thank you 😄
servedUdrInfo is now getting rendered as it is specified by the OpenAPI specification instead of the default object.

Hello @hidingturtle,
Great, thanks for trying it out.  This will be merged shortly.
Thanks,
Marina


Resolve UB in gcc9

fix #387

we need to set return value for every non-void function, even for branches that logically can't be reached.


[Suggestion]: Arrows of Purity

Another way to purify the Horde. These arrows are tipped in the same essence that is in the Flasks of Purity.
To craft Arrows of Purity, you surround an arrow with four Purity Essence, which gives you 16 arrows.
The arrows work like the flasks- the difference is that you can shoot the cure further with a bow or crossbow, but their effectiveness is worse than a Flask of Purity.
This gives the player another means to purify the world, allowing the player to snipe an infected area instead of more direct confirmation with the flasks.
Concept of the Arrows of Purity:

Very good recommendation, perhaps I'll do something like this.


Use prebuilt image in quickstart with Mender 2.3.

We need to highlight:

Prebuilt Raspbian image
Snapshot feature

FYI @michaelatmender @mzedel @lluiscampos @alfrunes - we seem to have "forgotten" to include the Raspbian image and snapshot feature in the quickstart.
This is what everyone reads to evaluate Mender so this omission would render much of the improvements hidden.
I think there in general is a problem that nobody is taking care of the documentation at a higher level, thinks about the user journey and improves it, but this is a bit bigger discussion.


Treatment of absolute paths in (-/-) is questionable

Consider the expression,
"/build" -/- "/usr/lib/hello"

What should this evaluate to? Currently it evaluates to /build/usr/lib/hello, interpreting the second argument as a relative path. This seems a bit surprising. Is this intentional?

To be clear, by "surprising" I mean "differs from System.FilePath.(</>)" (haddocks)

Here is a relevant issue: https://github.com/snowleopard/hadrian/issues/218
@bgamari We don't want to call </> because it messes up slash direction on Windows, but we can make the behaviour less surprising.

I believe this has been fixed.


SyntaxError: The requested module 'permissionless' does not provide an export named 'walletClientToCustomSigner'

node_modules/@zerodev/sdk/utils.ts:1
import { walletClientToCustomSigner } from "permissionless"
SyntaxError: The requested module 'permissionless' does not provide an export named 'walletClientToCustomSigner'

seems to be duplicate of https://github.com/zerodevapp/sdk/issues/88, closing since it's relevant to sdk


Issue Template

We need to add issue templates for;

[ ] Bug Reporting
[ ] Checklist Request
[ ] Questions and Help

I just created a template suggestion to use or improve on #56 .


Aj/event cards

Created the initial set of components that will be required for the MVP. Containing:

Multiple Chips
Event Cards
Looking for thread result

The location chip acting tough, might need to go back and fix the styling to get placed perfectly.

Very clean code, please add some comments though. There is no documentation at all. Simple docs for each component will suffice

In the dev night, I will get you to show me the cards, to double check and then we can merge.


Test: Link protection

Refs: #78125

[ ] macOS @isidorn
[ ] Linux @mjbvz
[ ] Windows @stevencl

Complexity: 2
Description: All outgoing links in workbench should be covered by this link protection mechanism, including hover, completion, terminal, output, debug console, webview...
Please test that:

Any action in workbench that would open a link in browser is covered by this feature
The prompt's description and actions makes sense
Configure Trusted Domains from the prompt or command palette work as expected
You can't bypass the user configured trusted domains list. For example, if https://{authority} is configured, http://{authority}, https://www.{authority} etc are still blocked.
localhost and <IP_ADDRESS> are whitelisted

I like how this behaves, nice job.


Feature Request fasd integration

I'm very happy with your plugin, is it possible to add the same functionality which is described under https://github.com/clvv/fasd - so when I press d it will show me the entries of my most typed in commands?
Thanks for the awesome work that went into this plugin.

Hi, I'm not sure what your request is.
Is it

Integration with fasd
or, is it writing a fasd-like feature using fzf?

it will show me the entries of my most typed in commands

This also confuses me a little, as fasd does not seem to have an option to show the most used command, but only files and directories.
If what you want is some kind of shell function for launching fzf finder for most used commands, I can think of something like the following.
eval $(HISTTIMEFORMAT= history | sed 's/^[0-9 ]*//' | sort | uniq -c | sort -n | fzf +s --nth 2..,1 | sed 's/^[0-9 ]*//')

But note that this only considers the number of occurrences and not the recency of the commands.

It is point 2. writing a fasd-like feature using fzf. So I want to have the recency of mos visited directories. Here is a small example.
Bests
Matthias

I see. But fzf is just a simpler filter, like grep, that you can use it to build larger concepts. Implementing fasd-like "frecency" concept is I believe beyond the scope of this project. But I'd love to hear about it when someone comes up with such a solution which utilizes fzf.
Extending (not replacing) fasd with fzf seems like a better solution to me.
fd() {
  local DIR
  DIR=$(d | fzf +s +m) && cd $(sed 's/^[0-9.]* *//' <<< "$DIR")
}

Thanks!

Agree, I will try your solution.

Okay, also check out z-integration section in wiki.
https://github.com/junegunn/fzf/wiki/Examples#z

Works very nice, thanks for it. Maybe it's better to put this command as well in the README.

We have migrated most of the examples on the README page to the wiki pages. Please note that anyone can edit those pages. If you come up with an useful one that you want to share, feel free to update the page. Thanks.


Improve/replace the password generation method in 'Password Managers'

Consider creating a separate tutorial for creating secure passwords.

The old method is good for regular users -- will work against attackers targeting masses -- but isn't very secure against highly-motivated attackers. Against government-level adversaries you want to use something like Schneier's method. Therefore, this depends on threat model.
TODO:

[ ] Threat model tutorial (a prerequisite for 2)
[ ] Password generation tutorial


How to convert black & white depth channels in recorded .mkv files to color ramp?

If I make a recording using k4arecorder and review it in Azure Kinect Viewer, the depth stream is shown with a color ramp. For a specific project, I need this color ramp version, but if I separate the raw recorded depth stream into its own file using ffmpeg, it appears only in black and white.
According to the official recording file format page, the depth feed is actually kept as 16-bit grayscale.
https://docs.microsoft.com/en-us/azure/kinect-dk/record-file-format
So my understanding is that Azure Kinect Viewer converts it to the color ramp just for playback..? If this is the case, is there a way to access that color-ramped version? Or a pre-made way to convert the depth stream file from black and white to either this color ramp or another kind? As long as I'm not re-inventing the wheel!
Any help is greatly appreciated!

Aaand I just saw the C# sample on this. I'll show myself out.
https://github.com/microsoft/Azure-Kinect-Samples/tree/master/build2019/csharp/2 - TransformDepthToColor


Automatic detection for MinGW/GNU toolchain products for PE

Is your feature request related to a problem? Please describe.
PE executables/objects imported will (apparently) invariably have their default language set to the vstudio/MSVC variant. This can easily result in issues like https://github.com/NationalSecurityAgency/ghidra/issues/2208 (demangling fails). It is immediately obvious for someone familiar with Ghidra's processing quirks, but not so much how to "fix" it. Ironically, this could confuse users into thinking Ghidra is less capable than IDA, as the latter usually auto-identifies the PE and selects the right processing target.
The solution is to manually adjust the language to the GCC variant (32/64bit).
Describe the solution you'd like
Ideally:

Upon detection of DWARF data, Ghidra could verify the existence of a debug_info section.
The data contained in this section could easily be used to identify MinGW/GCC builds:

                             //
                             // .debug_info
                             // ram:...-ram:...
                             //
                             ..                                    XREF[1]:     ...(*)
        .... 47 4e 55        ds         "GNU C99 ...

Do note that a crafty packer/malicious user could purposefully create a fake section to misdirect analysis (although this will not fool any reverse engineer with some experience). At that point you are likely dealing with other tricks.
There should also be assorted _mingw symbols present. (ex. pcinit).
This is a "quality of life" request, if anything.
Describe alternatives you've considered
The manual approach works just fine.
Additional context
N/A

I think this will be fixed by PR #4514 and #4520.


Include link to the gitter message that opened the issue

https://github.com/errbotio/err-backend-gitter/issues/15
Cannot be done unless implemented in the gitter backend :worried:

Do you plan to fix it upstream?
Is it feasible to run our own fork of that plugin if it isnt merged upstream?

Yes, it might be feasible since the development isn't really that much on that repo, we might miss out on small things, but rebasing on regular intervals can work. But let's wait for upstream to respond :D ?

Fixed upstream :+1:


call function fails

Steps to reproduce:

Start a node Navigate to nearcore folder. Switch to staging branch. Pull it. Start a node with:

rm -rf testdir; ./scripts/start_unittest.py --local

Get most recent near-shell Navigate to near-shell folder. Switch to staging. Install it:

rm -rf node_modules; npm install

Create new account Where /Users/zaver/Projects/nearcore should be replaced to your path to nearcore folder.

./bin/near create_account status_message --masterAccount=test.near --homeDir=/Users/zaver/Projects/nearcore/testdir --nodeUrl=http://localhost:3030

Deploy smart contract (get wasm file here: https://www.dropbox.com/s/gtg1ytn8vboe3gj/status_message.wasm?dl=0)

/bin/near deploy --accountId=status_message --wasmFile=/Users/zaver/Projects/near-runtime-rs/examples/status-message/res/status_message.wasm --homeDir=/Users/zaver/Projects/nearcore/testdir --nodeUrl=http://localhost:3030

Call a function on it

./bin/near call status_message set_status "{\"message\": \"Hello\"}" --accountId=test.near --wasmFile=/Users/zaver/Projects/near-runtime-rs/examples/status-message/res/status_message.wasm --homeDir=/Users/zaver/Projects/nearcore/testdir --nodeUrl=http://localhost:3030

EXPECTED:
To get a meaningful output.
ACTUAL:
near-shell returns an error:
WARNING] Didn't find config at /Users/zaver/Projects/near-shell/src/config, using default shell config
Scheduling a call: status_message.set_status({"message": "Hello"}) with attached<PHONE_NUMBER> NEAR
Loaded master account test.near key from /Users/zaver/Projects/nearcore/testdir/validator_key.json with public key = 22skMptHjFWNyuEWY22ftn2AbLPSYpmYwGJRGwpNHbTV
Error:  TypeError: Cannot read property 'length' of undefined
    at Object.getTransactionLastResult (/Users/zaver/Projects/near-shell/node_modules/nearlib/lib/providers/provider.js:14:32)
    at Object.exports.scheduleFunctionCall (/Users/zaver/Projects/near-shell/index.js:98:37)
    at processTicksAndRejections (internal/process/task_queues.js:89:5)
    at async exitOnError (/Users/zaver/Projects/near-shell/bin/near:8:5)

Confirm that the state changed. We want to make sure that the above error is about transaction status not being read correctly, while the contract call was actually correctly executed.

./bin/near view status_message get_status "{\"account_id\": \"test.near\"}" --accountId=test.near --wasmFile=/Users/zaver/Projects/near-runtime-rs/examples/status-message/res/status_message.wasm --homeDir=/Users/zaver/Projects/nearcore/testdir --nodeUrl=http://localhost:3030

OBSERVE:
Result: Hello

it appears to be fixed when pulling latest on everything, restarting node, nuking deps in shell, and retrying


Cleaned up the code I changed in my last commit

There are a couple things that bug me about the code I changed in the last commit. I added an "l" variable for the length of the Lawnchair.adapters array and moved the adapter validation check inside the loop. No functional change. Just cleaner.

much nicer


Locked sprints and backlog

Removes the Add Button when signed.

started this task from scratch after other task was merged


Create API to evict a trace

When a Trace is instantiated onto a CQ, it combines the traced commands into a flat buffer and commits that into device DRAM and caches it there.
The lifetime of the trace cannot be inferred, hence the trace buffer that is in DRAM is effectively cached until explicitly told to be evicted by the user.
We need a dedicated API for that to evict the trace out of DRAM by id, there won't actually be any device side operations we just need to destroy the host side trace buffer which will then trigger the deallocate call.

Marking P2, probably not blocking for single workload tests, where by ending the test we deallocate all buffers and free up DRAM. This will be relevant when we need to actually run multiple workloads back to back and need to free up DRAM stored traces
fyi @davorchap

Tony made a commit on Apr1 which added metal API ReleaseTrace() API and a test TraceInstanceManagement added.
Currently this type of API is not exposed at tt_eager level - open question if it should be.

I went ahead and exposed the API at python level in spirit of this ticket that users should have ability to control trace lifetime, marking closed.


[Adaptive] DCR: New Options Memory Shortcut

We need to support access to the dialog.options, and for adaptive dialogs this could be in 2 locations so it would be very useful to have a shortcut for it.
^xxxx => dialog.this.step.options.xxx || dialog.this.options.xxx

Tracked via #1966


Add badge for godoc to readme

Add a badge to godoc to readme.
godoc is available here https://godoc.org/github.com/jaegertracing/jaeger-operator/pkg/apis/jaegertracing/v1. I am not sure if we push it there or it scans github repositories.

It scans the repo automatically


Update on Windows 2008 Guest

Hi,
I'm trying to the auto update working from my host (Windows 7) to the guest which is Windows 2008 but I receive:
ERROR vagrant: Vagrant experienced an error! Details:
ERROR vagrant: #<VagrantVbguest::Installer::NoInstallerFoundError: translation missing: en.vagrant_vbguest.errors.installer.no_install_script_
for_platform>
ERROR vagrant: translation missing: en.vagrant_vbguest.errors.installer.no_install_script_for_platform
ERROR vagrant: C:/Users/gibbonsd/.vagrant.d/gems/gems/vagrant-vbguest-0.8.0/lib/vagrant-vbguest/installer.rb:87:in `guest_version'
Looks like this isn't supported?  Would I need to write a custom script?
Thanks
Dan

Although it would require pre-installation of chocolatey, this might be part of the solution:
https://chocolatey.org/packages/VBoxGuestAdditions.install
With choco installed, a script could run "choco search -lo VBoxGuestAdditions" to learn which--if any--guest additions were installed and "choco update VBoxGuestAdditions" to update to whatever is latest.


bounding box specification issue

Hi,
I don't seem to have control over the the bounding box for a subscription.  Below I show two commands for subscription (a) no -b specification and (b) specifying a small bounding box.  The downloaded files are the same size (and each has the world-sized number of pixels.)
I can subset the local files, but it would consume fewer resources if we could transfer just the smaller subset.
Thanks!
Ben
(a) the world (bounding box unspecified)
podaac-data-subscriber -c MUR-JPL-L4-GLOB-v4.1 -d world -sd 2021-01-21T00:00:00Z -ed 2021-01-23T00:00:00Z --verbose >> subscriber.log
# btupper@c4-6 tmp $ ll world
#   total 2231200
#   -rw-rw-r--. 1 btupper btupper 759327856 May 12 13:35<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper 757346424 May 12 13:35<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper 759097943 May 12 13:34<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper       197 May 12 13:35 MUR-JPL-L4-GLOB-v4.1.citation.txt

(b) a smaller subset specified by the -b argument.
podaac-data-subscriber -c MUR-JPL-L4-GLOB-v4.1 -d nwa2 -sd 2021-01-21T00:00:00Z -ed 2021-01-23T00:00:00Z -b="-77,36.5,-42.5,56.7" --verbose >> subscriber.log
# btupper@c4-6 tmp $ ll nwa2
#   total 2231200
#   -rw-rw-r--. 1 btupper btupper 759327856 May 12 13:43<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper 757346424 May 12 13:43<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper 759097943 May 12 13:43<PHONE_NUMBER>0000-JPL-L4_GHRSST-SSTfnd-MUR-GLOB-v02.0-fv04.1.nc
#   -rw-rw-r--. 1 btupper btupper       197 May 12 13:44 MUR-JPL-L4-GLOB-v4.1.citation.txt

####
#   version info
####
btupper@c4-6 tmp $ podaac-data-subscriber --version
#   PO.DAAC data subscriber 1.13.1
btupper@c4-6 tmp $ cat /etc/os-release
#   NAME="Rocky Linux"
#   VERSION="8.7 (Green Obsidian)"
#   ID="rocky"
#   ID_LIKE="rhel centos fedora"
#   VERSION_ID="8.7"
#   PLATFORM_ID="platform:el8"
#   PRETTY_NAME="Rocky Linux 8.7 (Green Obsidian)"
#   ANSI_COLOR="0;32"
#   LOGO="fedora-logo-icon"
#   CPE_NAME="cpe:/o:rocky:rocky:8:GA"
#   HOME_URL="https://rockylinux.org/"
#   BUG_REPORT_URL="https://bugs.rockylinux.org/"
#   ROCKY_SUPPORT_PRODUCT="Rocky-Linux-8"
#   ROCKY_SUPPORT_PRODUCT_VERSION="8.7"
#   REDHAT_SUPPORT_PRODUCT="Rocky Linux"
#   REDHAT_SUPPORT_PRODUCT_VERSION="8.7"

Hi Brian,
This is actually working as intended right now. The 'bbox' parameter only filters search results that do not pass through your region of interest. Because MUR data are essentially global data products, any bbox will match, and it downloads the entire file.
We have some work in progress on this exact functionality: https://github.com/podaac/data-subscriber/issues/36
Its still in development, but the good news is your data set (MUR) is high on the list of test cases for it. when this is delivered, an option -subset parameter will be provided to actually subset based on your space (bbox)and time bounds (start/stop dates).
We definitely appreciate the comment and detailed report, and are working hard to get the subset work pushed out to the users.

Oh, I see.  Thank you! Ben


[BUG]

Hi,
while calling the anonymize function:

anonym.anonymize(inplace = False, seed = 100)
I face with the following error:
AttributeError: 'function' object has no attribute 'seed'

could you please let me know, how can I fix this error?
Thanks, Shiva

@ShNadi  Heyyy, many thanks for reporting the issue! Seems like I haven't uploaded the latest version to pypi (pip).
So, what you have to do:

pip install anonympy --upgrade
To verify:

import anonympy
print(anonympy.__version__)
>>> 0.2.6
After that you can run your code with `seed` argument to get reproducible results.

@shukkkur thanks for your quick reply! I have upgraded the anonympy to version 0.2.6 but, the problem is not solved. The same error as before...
Best,

@ShNadi  Can you plase share your code

@shukkkur Here is the code and the error...
Have a nice weekend!

Tested both on my machine and Google Colab. Works.

I would suggest to reinstall the package:

pip uninstall anonympy
pip install anonympy

@ShNadi  Any Updates?

hey @shukkkur,
I reinstalled it in my Jupyter notebook, and it still does not work. But it works perfectly in the Pycharm environment. So, probably there is something wrong with my Jupyter...
Many thanks for the nice implemented package and the support.
Best,


Updated mocks

This PR updates mocks and updateMocks.sh script to allow Overview / scope change in topology

Since Overview is filling metrics with zeros, we should consider mocking the front-end time to the last script execution time.
Any thoughts on that @jotak ?

Let's merge to unlock the situation. We can think about my previous comment in a followup
/approve


Minor documentation update

Motivation
Have an up-to-date documentation
Test Plan
Not needed. Minor documentation change

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.3 out of 6 committers have signed the CLA.:white_check_mark: simsekgokhan:white_check_mark: 0o-de-lally:white_check_mark: hemulin:x: davidiw:x: gelash:x: msmouseYou have signed the CLA already but the status is still pending? Let us recheck it.


docs: fetch external sources

What does this PR do?
making docs as standalone resources without needing to pull S3 assets
pending on https://github.com/Lightning-AI/utilities/pull/142

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

cc @borda @tchaton @carmocca @justusschock @awaelchli

okk, it seems that Sphinx does not include resources linked only in HTML section
.. raw:: html

    <video width="50%" max-width="400px" controls
    poster="https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/thumb/check_val_every_n_epoch.jpg"
    src="https://pl-bolts-doc-images.s3.us-east-2.amazonaws.com/pl_docs/trainer_flags/check_val_every_n_epoch.mp4"></video>


mysql_database_user should not add quotes around database_name of '*'

Given the following resource definition:
mysql_database_user username do
  action        :grant
  connection    root_connection
  database_name '*'
  host          'localhost'
  password      pwd
  privileges    ['SELECT']
end

The mysql_database_user provider should grant global privileges by running:
GRANT SELECT ON *.* TO....

Instead, it quotes the * database name producing the query:
GRANT SELECT ON `*`.* TO...

which only grants permissions on a database named *.
This appears to be an unexpected regression introduced in 03477570b420b13c17278d690ea3d83a0107f872 which added support for the require SSL option. The diff includes this change:
- grant_statement = "GRANT #{@new_resource.privileges.join(', ')} ON<EMAIL_ADDRESS>&& @new_resource.database_name != '*'<EMAIL_ADDRESS><EMAIL_ADDRESS>&& @new_resource.table != '*'<EMAIL_ADDRESS>: '*'} TO `#{@new_resource.username}`<EMAIL_ADDRESS>IDENTIFIED BY "
+ grant_statement = "GRANT #{@new_resource.privileges.join(', ')} ON<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>: '*'} TO `#{@new_resource.username}`<EMAIL_ADDRESS>IDENTIFIED BY "

The new logic always quotes the database name, where the old logic only quoted if it was != *. This change is not mentioned in the commit message or in #62.
The provider has since been refactored but the changed logic has been retained

At this time I'm going to close this issue out as we've deprecated this cookbook. The original intent was to abstract database implementation details away from the end user. In hindsight this was a mistake as it is difficult / impossible to make a generic interface for databases that works across all systems. Instead we plan to move the individual providers into their respective cookbooks (mysql, postgresql, sqlite, sql_server) where they can fully utilize the feature set of the underlying DB systems.


undefined is not a function

Bug description
I have integrated posthog in my expo app successfully . Events are getting recored in dashboard too but when i am trying to use useFeatureFlag hook it throws an error saying undefined is not a function . Error is located in useFeatureFlag,js . Digging more into the error it says getFeatureFlag is not a function . Now this is defined in node_modules folder
Please describe.
How to reproduce

Simply try using useFeatureFlag hook in your component
Pass the feature flag in the hook
3.The app will crash

Related sub-libraries

[x] posthog-react-native

Additional context
Thank you for your bug report – we love squashing them!

@devashray-jumbo Hi,
which version of the SDK are you using? Which version of RN and Expo itself?
Can you provide a few code snippets of the PostHog SDK setup? so I can check things on my end.
Thanks

Hi
Please find the details below
posthog-react-native - 3.1.1
react-native - 0.71.14 expo - 48.0.21
export const projectApiKey =
'phc_cWZmmd0TLwconAucvMlzvv7Byim5iZ3ngILJmYm9VC1';
export const projectServerUrl = 'https://us.i.posthog.com';
export const posthog = new PostHog(projectApiKey, {
host: projectServerUrl,
disabled: false,
defaultOptIn: true,
sendFeatureFlagEvent: true,
preloadFeatureFlags: true,
disableGeoip: false,
captureNativeAppLifecycleEvents: true,
});
export const logPosthogEvent = ({
eventName,
eventAttribute = {},
}: {
eventName?: string;
eventAttribute?: any;
}) => {
if (eventAttribute) {
posthog.capture(eventName, { ...eventAttribute });
return;
}
posthog.capture(eventName);
};
On Mon, Jul 1, 2024 at 11:13 AM Manoel Aranda Neto @.***>
wrote:

@devashray-jumbo https://github.com/devashray-jumbo Hi,
which version of the SDK are you using? Which version of RN and Expo
itself?
Can you provide a few code snippets of the PostHog SDK setup? so I can
check things on my end.
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/PostHog/posthog-js-lite/issues/242#issuecomment-2199283762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEXLYGFSV6SHD47FTY5FOVLZKDUARAVCNFSM6AAAAABKC7EQU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJZGI4DGNZWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

When using feature flag hook the app crashes throwing error "undefined is
not a function"
import { useFeatureFlag } from 'posthog-react-native'
const testFlag = useFeatureFlag('Test')
On Mon, Jul 1, 2024 at 1:10 PM Devashray . @.***> wrote:

Hi
Please find the details below
posthog-react-native - 3.1.1
react-native - 0.71.14 expo - 48.0.21
export const projectApiKey =
'phc_cWZmmd0TLwconAucvMlzvv7Byim5iZ3ngILJmYm9VC1';
export const projectServerUrl = 'https://us.i.posthog.com';
export const posthog = new PostHog(projectApiKey, {
host: projectServerUrl,
disabled: false,
defaultOptIn: true,
sendFeatureFlagEvent: true,
preloadFeatureFlags: true,
disableGeoip: false,
captureNativeAppLifecycleEvents: true,
});
export const logPosthogEvent = ({
eventName,
eventAttribute = {},
}: {
eventName?: string;
eventAttribute?: any;
}) => {
if (eventAttribute) {
posthog.capture(eventName, { ...eventAttribute });
return;
}
posthog.capture(eventName);
};
On Mon, Jul 1, 2024 at 11:13 AM Manoel Aranda Neto <
@.***> wrote:

@devashray-jumbo https://github.com/devashray-jumbo Hi,
which version of the SDK are you using? Which version of RN and Expo
itself?
Can you provide a few code snippets of the PostHog SDK setup? so I can
check things on my end.
Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/PostHog/posthog-js-lite/issues/242#issuecomment-2199283762,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEXLYGFSV6SHD47FTY5FOVLZKDUARAVCNFSM6AAAAABKC7EQU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJZGI4DGNZWGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

@devashray-jumbo If you wanna use the usePostHog, useFeatureFlag, and other hooks, you need to set up PostHog with the PostHogProvider otherwise the hooks won't know about yourposthog instance, or at least do this.
export function MyApp() {
  return <PostHogProvider client={posthog}>{/* Your app code */}</PostHogProvider>
}

If you don't wanna use the PostHogProvider, then for instance of using the hooks, just use the posthog.getFeatureFlag(...) and similar methods.

Hi Manoel Aranda Neto ,
I tried using the Provider as mentioned , but as soon as I used usePostHog
hook , the app crashes saying undefined is not a function.
Digging into the issue , I found out that the getAppProperties function is
causing the issue. If I go into the posthog-react-native package in
node_modules and remove this method getAppProperties , the SDK works fine
in that case.
FYI , i have installed all the other dependency which were mentioned in the
docs
[image: image.png]
Please find the code snippet below

<PostHogProvider
apiKey={projectApiKey}
options={{
host: projectServerUrl,
disabled: false,
defaultOptIn: true,
sendFeatureFlagEvent: true,
preloadFeatureFlags: true,
disableGeoip: false,
captureNativeAppLifecycleEvents: true,
}}

 In Login Component I am using the usePostHog hook
const posthog = usePostHog();
On Mon, Jul 1, 2024 at 1:20 PM Manoel Aranda Neto @.***>
wrote:

@devashray-jumbo https://github.com/devashray-jumbo If you wanna use
the usePostHog, useFeatureFlag, and other hooks, you need to set up
PostHog with the PostHogProvider
https://posthog.com/docs/libraries/react-native#with-the-posthogprovider
otherwise the hooks won't know about yourposthog instance, or at least do
this
https://posthog.com/docs/libraries/react-native#without-the-posthogprovider
.
export function MyApp() {
return {/* Your app code */}
}
If you don't wanna use the PostHogProvider, then for instance of using
the hooks, just use the posthog.getFeatureFlag(...) and similar methods.
—
Reply to this email directly, view it on GitHub
https://github.com/PostHog/posthog-js-lite/issues/242#issuecomment-2199472948,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BEXLYGFBQKGU4UYSSZSH4F3ZKEC3TAVCNFSM6AAAAABKC7EQU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCOJZGQ3TEOJUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@devashray-jumbo hey,
getAppProperties cannot return null or undefined so it's very strange.
We are using this in our samples and I just tested it out and everything is working fine.
The code snippets are incomplete so I cannot spot the issue besides that you are wrapping PostHogProvider with other things I don't know such as Provider, APIProvider, etc, you are probably trying to use usePostHog even before the PostHogProvider is created.
PostHogProvider must be ideally the root so it's created before anything.
Can you provide a minimal reproducible example? Just create a sample app from scratch and try to add PostHog, so you can demonstrate the bug or find out that there's something off in your implementation.
Thanks.
Best,
Manoe,

Closing the issue as a part of large repository cleanup, due to it being inactive and/or outdated.
Please do not hesitate to ping me if it is still relevant, and I will happily reopen and work on it.
Cheers!


Use API RBAC in javascript

This is mostly stop-gap until things are more properly VUE-ized. It is likely we may still use some of this in the pre.inc pages to block sensitive pages from even loading.
Also nothing was touched in the concom module until rewrite and cleanup is finished.

Sorry no  responding directly inline but it is all mixed together in my brain.
First off I am very uncomfortable with our 2 RBAC implementation, one API and one frontend PHP.  This is just a mismatch waiting to happen and really feels like it is going to be a source of pain and confusion.  I want 1 RBAC implementation and I feel like the correct place for that is in the API.
Second, trying to call the API implementation of the RBAC directly in PHP is... messy at best and likely really hard. I am trying to figure out if we can do it without having to spool up a full slim framework container.
Third, we already have the case where the php level oauth token stored in $_SESSION['apiAuth'] expires before the javascript one, resulting in strangely missmatched state where some parts of the system will load but others do not.
I believe we can fix the general slow-down by caching the RBAC checks on the PHP side.  That feels like a good way to try to fix that for now.   Yes the number of back in round trips is troubling but I am not sure of another good way to do it.

First off I am very uncomfortable with our 2 RBAC implementation, one API and one frontend PHP. This is just a mismatch waiting to happen and really feels like it is going to be a source of pain and confusion. I want 1 RBAC implementation and I feel like the correct place for that is in the API.

I agree, I think we're on the same page here. To make sure I understand, "frontend PHP" means, the parts of the UI that are not using the API currently?

Second, trying to call the API implementation of the RBAC directly in PHP is... messy at best and likely really hard. I am trying to figure out if we can do it without having to spool up a full slim framework container.

I agree here, too. This is why I was suggesting that we let the API stuff do it's thing and serve the front-end and then the rest of the older legacy stuff continues to do its thing, whether through the new implementation (directly, not via API calls) or otherwise. Especially because it has to spin up all of the roles again and everything else, since we aren't storing them anywhere, I think it drastically impacts performance overall. It's not a permanent recommendation, but one that is just temporary until we get more of this migrated over to JS and the APIs.

I believe we can fix the general slow-down by caching the RBAC checks on the PHP side. That feels like a good way to try to fix that for now. Yes the number of back in round trips is troubling but I am not sure of another good way to do it.

When you say caching the RBAC checks, do you mean "I've seen this user and I know they have role XYZ, so they're good"? For reasons, I would recommend against this. We would then be introducing difficult to debug behavior for our site and potentially allowing or restricting access to site functions in an inaccurate way.
If you mean "generating the roles based on positions and building up the RBAC model", then yes I do think we should cache that if we aren't already, or find a better long-term solution such as the use of a database for this. This might help with performance overall since we currently don't have them stored anywhere else, and are re-generating all of the roles every single time we are answering requests. I don't think we'd want to cache the actual checks to see whether a user can or cannot access things.
The other option here, which would mean fewer round trips, would be to let the two implementations live separately for now and prioritize moving all of the remaining sections of the site that are not currently mostly leveraging JS and Vue to do exactly that. It's definitely risky, and there will be bugs or misses if we are doing other things with RBAC in the meantime.
It does seem like the best end-result is:

RBAC implementation lives in the API
JS calls into the API to do its thing
The end

Right now, with stuff in an in-between state with some Vue-based components using the API and PHP-based data/HTML rendering, I think our RBAC implementation is similarly stuck. If we try to force the non-Vue parts of the site to use the API-based implementation, I think that will definitely be a source of pain and confusion like you've said.
Would it be a better use of our time to get this to a state where it is acceptable (perhaps that state already exists w/o these changes) and serves the API and non-API portions of the site correctly, and then focus on getting the remaining sections onto Vue-based components that use the API before coming back to remove this stuff?


Implement profile.name, .img, .find() and .findAll()

Implements issue #82.

We chatted offline about deciding on a term to refer to "the profile picture". Right now profile.img will get you the URL. I think that profile.pic or profile.picture might be a bit more intuitive since "profile picture" seems to be more culturally accepted. No strong feelings from me, though!

Awesome @dmitrizagidulin - I love the attention to docs (both in the README and in the jsdoc comments). I have a couple of remaining points that I brought up.


Bump jupyterhub version to 2.3.x

Move bioconductor install to separate file, so we don't end up
busting the general R cache each time we need to fiddle with
bioconductor
Explicitly install an older version of locfit so bioconductor can
install edgeR
Install newer version of jupyterhub, so interface selection works
again
Install textblob

Fixes #33

This doesn't actually block upgrading to z2jh, as jupyterhub is already >1.5.0 on this

Yeah, I'll switch this over to mamba.

All fixed!


Replace the deprecated devel block with head

brew now produces the following warning each time it is used with the prob formula installed:
Warning: Calling 'devel' blocks in formulae is deprecated! Use 'head' blocks or @-versioned formulae instead.
Please report this issue to the hhu-stups/prob tap (not Homebrew/brew or Homebrew/core), or even better, submit a PR to fix it:
  /usr/local/Homebrew/Library/Taps/hhu-stups/homebrew-prob/prob.rb:11

devel blocks are deprecated since version 2.4.0 of Homebrew. This pull request simply replaces the devel block with the head block.
Another possible solution is to create a separate prob-nightly formula.

Thanks!


Mirror Listener abruptly shuts down without any error if JSON parse/stringify is used on the message. Hashgraph SDK v2.0.14

The mirror shutting down does sound like an issue, but I think what you meant to write is
console.log(JSON.parse(new TextEncoder("utf-8").decode(message.contents));`

Can you confirm this is still an issue for you. Currently in our code we have                    try {                         listener(topicMessage);                     } catch (error) {                         errorHandler(topicMessage, error);                     } which will catch all errors thrown by the callback supplied to subscribe and pass the error to the provided error handler. The TopicMessageQuery should not be shutting down, and I cannot seem to reproduce this.


extension cannot be found in vscode

Hello,
I'm new to the vscode community.
I follow the following steps in vscode.
Launch VS Code Quick Open (Ctrl+P), paste the following command, and press enter.
ext install znck.grammarly

But the vscode shows
No extensions found.

Can you help to identify which steps I'm wrong?
Thanks.

by following this suggestion, I could resolve the extension not found problem.
https://stackoverflow.com/questions/37143536/no-extensions-found-when-running-visual-studio-code-from-source


Docs:(upload):update http-request attribute description

     const req = this.httpRequest(options);
      this.reqs[uid] = req;
      if (req && req.then) {
        req.then(options.onSuccess, options.onError);
      }

Please make sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's contributing guide (中文 | English | Español | Français).
[x] Make sure you are merging your commits to dev branch.
[x] Add some descriptions and refer relative issues for you PR.

cascader 文案修改没问题 upload promise 文案待讨论


ORC-1475: [C++] ConvertColumnReader.TestConvertNumericToStringVariant fails when compiled with unsigned char

What changes were proposed in this pull request?
Fix the failure of ConvertColumnReader.TestConvertNumericToStringVariant when char is unsigned.
Why are the changes needed?
The UT fails when compiled with unsigned char.
How was this patch tested?
The UT is passed when compiled both with and without the flag -funsigned-char.

Thanks @guiyanakuang for your review. This commit ports it into branch-1.9.

Merged to branch-1.9, Thanks @wgtmac for making this pr.

Hi, @wgtmac and @guiyanakuang . May I ask how to reproduce this bug in Docker testing?

I created Milestone 1.9.2 and added this there for now.

Hi, @wgtmac and @guiyanakuang . May I ask how to reproduce this bug in Docker testing?

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 33aaabd42..ff0143d9c 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -108,9 +108,9 @@ endif()
 # We require a C++17 compliant compiler
 set(CMAKE_CXX_STANDARD_REQUIRED ON)
 if (NOT MSVC)
-  set(CMAKE_CXX_FLAGS_DEBUG "-O0 -g -fno-omit-frame-pointer")
-  set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-O3 -g -DNDEBUG -fno-omit-frame-pointer")
-  set(CMAKE_CXX_FLAGS_RELEASE "-O3 -DNDEBUG")
+  set(CMAKE_CXX_FLAGS_DEBUG "-O0 -g -fno-omit-frame-pointer -fsigned-char")
+  set(CMAKE_CXX_FLAGS_RELWITHDEBINFO "-O3 -g -DNDEBUG -fno-omit-frame-pointer -fsigned-char")
+  set(CMAKE_CXX_FLAGS_RELEASE "-O3 -DNDEBUG -fsigned-char")
 endif ()
 message(STATUS "compiler ${CMAKE_CXX_COMPILER_ID} version ${CMAKE_CXX_COMPILER_VERSION}")
 if (CMAKE_CXX_COMPILER_ID MATCHES "Clang")
(END)

@wgtmac My question was how to reproduce this bug in Docker testing because it was reported here via ./run-all.sh local branch-1.9 without any change. Apache ORC branch-1.9 doesn't have that flag, unsigned-char, in our source code or Dockerfiles.

https://lists.apache.org/thread/7k735b134j7h20ln13b0ztw7kyd2xtd2

bash-5.1$ ./run-all.sh local branch-1.9
Launching centos7 as 85438
...
FAILED centos7
...

Let me ask on the original PR.

I cannot reproduce this issue. What is your host cpu running the docker test? @mystic-lama

Oh, I posted https://github.com/apache/orc/pull/1579#issuecomment-1680003786 without knowing your comment here. So, is it C++ compiler setting difference on Docker Intel and Docker ARM?

You are right. I reproduced it on M1 Max. Previously, I ran Docker tests on old Intel Mac. So I missed that. Thank you, @wgtmac .


wiki git clone is not working on Windows

Describe the bug
Wiki repository  https://github.com/LinuxForHealth/FHIR.wiki.git  clone fails with  "error: invalid path" message.
Environment
git version 2.37.0.windows.1
To Reproduce
Steps to reproduce the behavior:

Open terminal
execute git clone https://github.com/LinuxForHealth/FHIR.wiki.git
See error
error: invalid path 'Image-and-Container:-Tips.md'
fatal: unable to checkout working tree
warning: Clone succeeded, but checkout failed.

Expected behavior
git clone should be work fine.
Additional context
Don't use the following characters in your wiki page's titles: \ / : * ? " < > |. Users on certain operating systems won't be able to work with filenames containing these characters.
Check out this document  https://docs.github.com/en/communities/documenting-your-project-with-wikis/adding-or-editing-wiki-pages#about-wiki-filenames

@berkant-k  thanks - we will get this one fixed too.

@berkant-k I renamed the pages I think were the problem. Please try again and let me know how it goes.

Thanks. It is working now.


Project refactor

Needed for https://github.com/mobie/mobie-viewer-fiji/pull/301

gets rid of the old FileUtils and combines it with FileAndUrlUtils
removes mobie specific functions
for TableRowsTableView I changed its load columns to be more generic (not relying on mobie specific project setup)

Works for me now on the load_columns branch on imagej-utils, please merge that one.


Make bindUntilEvent() and bind() public?

These methods are actually generic enough that they can be reused for other lifecycles (such as controllers in MVP patterns). Would be nice to use them rather than copying in logic and mixing in activity and controller lifecycles.

Same with OutsideLifecycleException

I'm always hesitant to open things up, since I've always considered that more of an implementation detail, especially bind().
Would using takeUntil + Navi help out? Navi is better designed for generic inputs.

I'm not sure Navi helps here since it looks like it would still require writing our own bind() implementations

I have separate thoughts on bindUntilEvent() and bind().
Looking on it now, it seems like we shouldn't even have bindUntilFragmentEvent() and bindUntilActiviyEvent(). bindUntilEvent() can use generics to figure out the difference. I think we can open that up and deprecate the first two (or even just remove them entirely).
Regarding bind() though (and really, your entire original problem). Suppose we open up bind(). Then we also have to open up ACTIVITY_LIFECYCLE and FRAGMENT_LIFECYCLE, since those are the inputs. And your implementation would have to know which one to pass in. I'm not sure how that saves you any effort, generically speaking, since you still have to know whether you're working with Activities or Fragments. The same applies to bindUntilEvent() since you'd have to know which you're targeting.

You wouldn't necessarily have to open them up, though having public examples might be helpful for what I'm about to suggest. The idea is so that you could pass other lifecycles like CONTROLLER_LIFECYCLE or a custom ACTIVITY_LIFECYCLE (e.g. your base actvitiy has other callbacks you want to expose). What's great about the bind() implementation is that it doesn't care if the lifecycle is of an activity, fragment, or goat. The only implementation change would maybe be to use .equals() rather than == to allow for non-enum parameters, but that would be a separate discussion I think.
Perhaps using an interface that behaves the same way as Func1 could make it more clear? Something like LifecycleMapping#map(event) maybe? Then just create a Func1 internally in bind() that just calls it under the hood.

I see your point. I'll think about it.
It almost seems like this could lead to a split in the library, where RxLifecycle is all about binding generic subscriptions to any given lifecycle and RxAndroidLifecycle could be about binding specifically to Android lifecycles.

Yeah that could work


Using a session name that is a prefix of the first window results in missing windows

I have to following catmux config:
---
common:
  before_commands: []
parameters: {}

windows:
  - name: perception_base
    layout: tiled # One can use the default layouts specified by tmux
    splits:
      - commands:
        - echo "lala"
  - name: lane_detection
    layout: tiled # One can use the default layouts specified by tmux
    splits:
      - commands:
        - echo "lblb"

starting it via catmux_create_session test_config.yaml works as expected with 2 windows being created.
If I use the --session_name with a name that is a prefix of the first window defined, the window appears to not be created.
For example catmux_create_session test_config.yaml --session_name perception creates only one window, but executes the commands from both perception_base and lane_detection in the same window.
It prints the following error message:
Parameters found in session config:
 -
{'before_commands': [], 'name': 'perception_base', 'layout': 'tiled', 'splits': [{'commands': ['echo "lala"']}]}
[{'commands': ['echo "lala"']}]
{'before_commands': [], 'name': 'lane_detection', 'layout': 'tiled', 'splits': [{'commands': ['echo "lblb"']}]}
[{'commands': ['echo "lblb"']}]
no server running on /tmp/tmux-1000/catmux
Using tmux config file: /home/max/.tmux.conf
tmux -L catmux -f /home/max/.tmux.conf new-session -s perception -d
Created session "perception"
create window failed: index 1 in use
Error while calling "%s" ['tmux', '-L', 'catmux', 'new-window', '-t', 'perception']

using a session name that is not a prefix of the first window name (for example perception123 or lane) resolves the issue.

Thank you for reporting this. I think #38 should fix this.


Hardcode the list of pools/gauges that user can stake

Describe the bug
Anyone can create a gauge. I just created a gauge for the 80/20 pool and now the UI shows a call to action for me to stake that pool's token.

The frontend should have a list of pre-approved gauges so we're not inviting users to stake for nothing

So are only intending whitelisted pools to be stakable?

Yes. But keep in mind there may be temporary gauges that are only listed for a few weeks. When they are removed from the whitelist they should still be displayed as staked so that users can Unstake

In other words, only invite users to skake on whitelisted gauges, but show all their staked pools regardless and allow them to Unstake from them

We should probably use the hardcoded voting gauges list for this, right @markusbkoch? We can filter out voting gauges from other networks.

@garethfuller We can use that but we need to complement it with ad-hoc gauges that will be incentivized with other tokens but no BAL, or only BAL from the 10% at the Baller's discretion.

@markusbkoch Ok then I don't think we should mix these up, we should have a separate liquidity gauge allowlist.


Remove para on exFAT support

@wickste to review, fixes https://github.com/eclipse-threadx/rtos-docs/issues/4

Closing this PR as redundant.


One active slide which is changed via model - outside carouselComponent.

Changes for setting the active slide.
With the changes developer can change model active properties on SlideComponent and it will propagate on all related slides.
With lot of fixes for whitespace and reogranize file...

Closing in favour of #1298
Thank you so much!


Migrate lab billing

Move reoccurring lab charges away from Koo’s credit card.

➤ Rebecca Fitzhugh commented:
This is proving to be quite challenging, as expected. Primary issue is that we still are not aligned on lab budget after the issues at beginning of year.

➤ Rebecca Fitzhugh commented:
Have provided billing account numbers to Sveta Shandilya & Anoop Madhavan from IT.

Azure - 2d689783-3072-4fb2-b518-cf95cfbb6279
Office 365 - d0aa11b8-9e93-4ee4-a442-bbc077b1015b
AWS -<PHONE_NUMBER>90
GCP -<PHONE_NUMBER>2

Will follow up early next week to check status.


make LINKME_PLEASE mutable to ensure writable flags on all relevant ELF sections

The proximate problem this PR addresses is a failure on illumos demonstrated in this repo: https://github.com/ahl/linkme-test
In short: a library declares a distributed slice and a consumer of that library adds to that slice. On illumos the component of that distributed slice added by the consumer of the library is not present. We can see this in the absence of output from the test cmd.
Looking at ELF data on illumos reveals something interesting:
$ elfdump -c -N set_linkme_DOERS target/debug/cmd

Section Header[20]:  sh_name: set_linkme_DOERS
    sh_addr:      0x5b3068            sh_flags:   [ SHF_ALLOC ]
    sh_size:      0                   sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0x1b3068            sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

Section Header[28]:  sh_name: set_linkme_DOERS
    sh_addr:      0x5cc128            sh_flags:   [ SHF_WRITE SHF_ALLOC ]
    sh_size:      0x8                 sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0x1bc128            sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

We have not one, but two sections named set_linkme_DOERS. The first is zero-sized and not-writable; the second is 8 bytes and writable.
Looking at the object files and rlibs that went into this build, we see the same:
$ elfdump -c -N set_linkme_DOERS target/debug/deps/liblib-3d689bd64c3873d5.rlib
target/debug/deps/liblib-3d689bd64c3873d5.rlib(lib-3d689bd64c3873d5.2vwi8dtk7ky9hvmq.rcgu.o):

Section Header[7]:  sh_name: set_linkme_DOERS
    sh_addr:      0                   sh_flags:   [ SHF_ALLOC ]
    sh_size:      0                   sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0xc8                sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

$ elfdump -c -N set_linkme_DOERS target/debug/incremental/cmd-w8yn2xhke56c/s-fwzp7udpq1-1agt1jj-121smxhf5scli/*.o
target/debug/incremental/cmd-w8yn2xhke56c/s-fwzp7udpq1-1agt1jj-121smxhf5scli/1m65hci6o5zni2mr.o:

Section Header[17]:  sh_name: set_linkme_DOERS
    sh_addr:      0                   sh_flags:   [ SHF_WRITE SHF_ALLOC ]
    sh_size:      0x8                 sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0xf8                sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

The rlib where the slice is declared is zero-bytes and non-writeable; the .o where we add to the slice is 8 bytes and writable. The illumos linker sees these as two different sections due to the difference in ELF flags so we end up with two different sections in the resulting binary.
On Linux, the test program works, and the ELF data is as expected:
$ readelf -S target/debug/cmd | grep -A1 linkme_DOERS
  [28] linkme_DOERS      PROGBITS<PHONE_NUMBER>047050  00046050
<PHONE_NUMBER>000008<PHONE_NUMBER>000000  WA       0     0     8

Note that the section is writable (W) for reasons not readily apparent. Looking at the .o and .rlib files that went into the final binary we see the same disparity on Linux as we did on illumos:
$ readelf -S target/debug/deps/liblib-dbcc34a6ce485a29.rlib | grep -A1 linkme_DOERS
readelf: Error: Not an ELF file - it has the wrong magic bytes at the start
  [ 7] linkme_DOERS      PROGBITS<PHONE_NUMBER>000000  000000d0
<PHONE_NUMBER>000000<PHONE_NUMBER>000000   A       0     0     8
$ readelf -S target/debug/incremental/cmd-3e55fmylgk0vb/s-fwzozwzwjv-zngw3r-3ojl9o3cuvpk4/*.o | grep -A1 linkme_DOERS
  [13] linkme_DOERS      PROGBITS<PHONE_NUMBER>000000  000000c8
<PHONE_NUMBER>000008<PHONE_NUMBER>000000  WA       0     0     8

Note that lib where the slice is defined is not writeable whereas the .o where we add to the slice is writeable.

With the fix applied:
illumos
$ elfdump -c -N set_linkme_DOERS target/debug/deps/liblib-*.rlib
target/debug/deps/liblib-c98febdfc141550b.rlib(lib-c98febdfc141550b.2o3xp1c7sa5vh54b.rcgu.o):

Section Header[7]:  sh_name: set_linkme_DOERS
    sh_addr:      0                   sh_flags:   [ SHF_WRITE SHF_ALLOC ]
    sh_size:      0                   sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0xc8                sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

$ elfdump -c -N set_linkme_DOERS target/debug/incremental/cmd-*/*/*.o
target/debug/incremental/cmd-3usn3y2iy1h21/s-fwzpq2djgj-1kym4gy-3ohhm8qr14hnn/3jh0d7z62f805s47.o:

Section Header[17]:  sh_name: set_linkme_DOERS
    sh_addr:      0                   sh_flags:   [ SHF_WRITE SHF_ALLOC ]
    sh_size:      0x8                 sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0xf8                sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

$ elfdump -c -N set_linkme_DOERS target/debug/cmd

Section Header[27]:  sh_name: set_linkme_DOERS
    sh_addr:      0x5cc0f8            sh_flags:   [ SHF_WRITE SHF_ALLOC ]
    sh_size:      0x8                 sh_type:    [ SHT_PROGBITS ]
    sh_offset:    0x1bc0f8            sh_entsize: 0
    sh_link:      0                   sh_info:    0
    sh_addralign: 0x8

The final binary has a single section with the right length and size; we get the expected output from the program.
Linux
$ readelf -S target/debug/deps/liblib-*.rlib | grep -A1 linkme_DOERS
  [ 7] linkme_DOERS      PROGBITS<PHONE_NUMBER>000000  000000d0
<PHONE_NUMBER>000000<PHONE_NUMBER>000000  WA       0     0     8
$ readelf -S target/debug/incremental/cmd-*/*/*.o | grep -A1 linkme_DOERS
  [13] linkme_DOERS      PROGBITS<PHONE_NUMBER>000000  000000c8
<PHONE_NUMBER>000008<PHONE_NUMBER>000000  WA       0     0     8
$ readelf -S target/debug/cmd | grep -A1 linkme_DOERS
  [28] linkme_DOERS      PROGBITS<PHONE_NUMBER>047050  00046050
<PHONE_NUMBER>000008<PHONE_NUMBER>000000  WA       0     0     8

The rlib section is now writable; the final binary is unchanged (i.e. same length and flags).

Note that since the modified line is only relevant on illumos and Linux, no other examination seems to be warranted.
I'm not sure how to incorporate a test for this, but would be happy to do so with some guidance.

Published in 0.2.5.


compile fails - Arduino can't find

MacOS 10.11.6, Arduino 1.6.5
Following along with the README, I install all the prerequisites and then open one of the OpenBCI examples.  Compilation fails with this mesage:
Arduino: 1.6.5 (Mac OS X), Board: "OpenBCI 32"

Using library OBCI32_SD in folder: /Users/ashroyer/Documents/Arduino/libraries/OBCI32_SD (legacy)

/Users/ashroyer/Library/Arduino15/packages/chipKIT/tools/pic32-tools/1.42-pic32gcc/bin/pic32-g++ -c -g -O2 -Wall -DARDUINO_ARCH_PIC32 -mno-smart-io -ffunction-sections -fdata-sections -mdebugger -Wcast-align -fno-short-double -ftoplevel-reorder -MMD -fno-exceptions -mprocessor=32MX250F128B -DF_CPU=40000000L -DARDUINO=10605 -D_BOARD_DP32_ -DMPIDEVER=16777998 -DMPIDE=150 -DIDE=Arduino -G1024 -D_USE_USB_FOR_SERIAL_ -I/var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/sketch -I/Users/ashroyer/Library/Arduino15/packages/chipKIT/hardware/pic32/1.3.1/cores/pic32 -I/Users/ashroyer/Library/Arduino15/packages/chipKIT/hardware/pic32/1.3.1/variants/OpenBCI -I/Users/ashroyer/Documents/Arduino/libraries/OBCI32_SD /var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/ReadWrite.cpp -o /var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/ReadWrite.cpp.o
ReadWrite.pde:24:23: fatal error: OBCI32_SD.h: No such file or directory
compilation terminated.
/Users/ashroyer/Library/Arduino15/packages/chipKIT/tools/pic32-tools/1.42-pic32gcc/bin/pic32-g++ returned 255
Error compiling.

I thought it was kind of odd that the SD library is contained in an otherwise empty OpenBCI_32bit_SD folder, so I copied OBCI32_SD into ~/Documents/Arduino/libraries
Now I get this error:
Arduino: 1.6.5 (Mac OS X), Board: "OpenBCI 32"

Using library OBCI32_SD in folder: /Users/ashroyer/Documents/Arduino/libraries/OBCI32_SD (legacy)

/Users/ashroyer/Library/Arduino15/packages/chipKIT/tools/pic32-tools/1.42-pic32gcc/bin/pic32-g++ -c -g -O2 -Wall -DARDUINO_ARCH_PIC32 -mno-smart-io -ffunction-sections -fdata-sections -mdebugger -Wcast-align -fno-short-double -ftoplevel-reorder -MMD -fno-exceptions -mprocessor=32MX250F128B -DF_CPU=40000000L -DARDUINO=10605 -D_BOARD_DP32_ -DMPIDEVER=16777998 -DMPIDE=150 -DIDE=Arduino -G1024 -D_USE_USB_FOR_SERIAL_ -I/var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/sketch -I/Users/ashroyer/Library/Arduino15/packages/chipKIT/hardware/pic32/1.3.1/cores/pic32 -I/Users/ashroyer/Library/Arduino15/packages/chipKIT/hardware/pic32/1.3.1/variants/OpenBCI -I/Users/ashroyer/Documents/Arduino/libraries/OBCI32_SD /var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/ReadWrite.cpp -o /var/folders/27/w1myz2vn78gflt11gb_hydfr0000gp/T/build8231903212779785058.tmp/ReadWrite.cpp.o
In file included from ReadWrite.pde:24:0:
/Users/ashroyer/Documents/Arduino/libraries/OBCI32_SD/OBCI32_SD.h:19:18: fatal error: DSPI.h: No such file or directory
 #include <DSPI.h>
                  ^
compilation terminated.
/Users/ashroyer/Library/Arduino15/packages/chipKIT/tools/pic32-tools/1.42-pic32gcc/bin/pic32-g++ returned 255
Error compiling.

For completeness, after trying this I also tried installing the chipkit hardware libraries via the "automatic" route, but this did not change anything for me.
Any ideas?

I think this is a known problem with Arduino IDE 1.6.5.
Please upgrade to the latest Arduino version and try again.

Ah, I was following the OpenBCI_32Bit readme, which says:

Download Arduino IDE version 1.6.5 software from the Arduino site. Currently, only 1.6.5 works with our software.

Perhaps that readme should be revised. On 1.6.12 it compiles just fine.  Thanks!

Oh,
You should not use that code! The 32bit Libraries have been upgraded.
Go here
https://github.com/OpenBCI/OpenBCI_32bit_Library
and find the file DefaultBoard for our produciton code. You can find some other code in the examples there too.

Nice, thanks!


Opt-in flag to confirm changes before applying

Adds an opt-in --confirm flag, which when present will prompt the user to confirm the change set before applying.

Pull Request Test Coverage Report for Build 434

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.825%

Totals

Change from base Build 433:
0.0%

Covered Lines:
636

Relevant Lines:
1597

💛  - Coveralls


Failed to get broadcast options

type object 'Product' has no attribute 'PRODUCT_EXTRA_CITY' (/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/add_1_monkey_patches.py:144)
Traceback (most recent call last):
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/mcfw/restapi.py", line 289, in run
    result = run(f, args, kwargs)
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/mcfw/rpc.py", line 183, in run
    result = function(*args, **kwargs)
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/mcfw/restapi.py", line 96, in wrapped
    return f(*args, **kwargs)
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/mcfw/rpc.py", line 167, in typechecked_return
    result = f(*args, **kwargs)
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/mcfw/rpc.py", line 145, in typechecked_f
    return f(**kwargs)
  File "/base/data/home/apps/e~rogerthat-server/20180803t110202.411588585533225705/solutions/common/restapi/__init__.py", line 472, in rest_get_broadcast_options
    extra_city_product_key = Product.create_key(Product.PRODUCT_EXTRA_CITY)
AttributeError: type object 'Product' has no attribute 'PRODUCT_EXTRA_CITY'

Fixed in ff6f6e1


Install rsync with tce-load

Ping @brikis98.
So, as said rsync for 64bits is now available for tce-load:
https://github.com/boot2docker/boot2docker/issues/936#issuecomment-120992953
I am creating this separate PR for this. It should be simpler to review and merge.

Thanks for the PR! Just one minor comment.

Thanks!

Great! Thanks.


fixed "error: unexpected token "get_local"" in unit tests

get_local is replaced with local.get and causes error in unit tests with current WABT. See https://github.com/mdn/webassembly-examples/issues/17

Oops. Sorry, fixed it already.
Thanks for your contribution anyway!


init: consider mkdir the mount directories

The init executable is going to mount some filesystems on the guest (proc, sysfs, and so on). If the rootfs doesn't have, say, `/proc' then the operation fails.
It can be reproduced with the chroot_vm program and the rootfs made from a busybox container.
Should some (or all) of those mount point directories be created?

That's an interesting question. I'd say a path that's missing those directory doesn't qualify as a valid root filesystem. On one hand, extending init to create those directories it's just a matter of a adding some simple syscalls, but on the other, that would go against the minimalist approach that I'm trying to maintain in libkrun.
I think I'm more inclined to not add those syscalls for the moment, and reconsider this if we find a use case that justifies the cost.


[WIP] Add environment testing causality in stochastic environments.

This is a draft for adding an environment similar to the one proposed in section 6.3 of B. Ziebart's PhD thesis (2010).
Consists of a risky path with high reward (non-achievable in practice, ends up causing low reward), and achievable paths of medium reward, and tests whether the agent correctly chooses a non-risky path.
This is a draft, but let me know if you have any comments on the design and "hyperparameters" of the environment. I imagine there are better cost values to give, but I have not given much thought to it yet.

Superseded by #20.


Streaming Video url Redirecting to domain

Checklist

[x] I'm asking a question
[x] I've looked through the README and FAQ for similar questions
[x] I've searched the bugtracker for similar questions including closed ones

Question

When I try to download using video URL, it is forcibly redirecting to the domain as shown in the log
Is there any way to avoid redirection of .m3u8 link........Can you please solve this issue to download a video

C:\Users\HP I5>"C:\Users\HP I5\Videos\youtube-dl.exe" -v -U https://d2fgiyp8yuap60.cloudfront.net/Geotech_GTE_Origin_of_Soil_20191226/720p.m3u8
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-U', 'https://d2fgiyp8yuap60.cloudfront.net/Geotech_GTE_Origin_of_Soil_20191226/720p.m3u8']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2020.03.24
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.18362
[debug] exe versions: none
[debug] Proxy map: {'https': 'https://<IP_ADDRESS>:8888', 'http': 'http://<IP_ADDRESS>:8888'}
youtube-dl is up-to-date (2020.03.24)
[generic] 720p: Requesting header
[redirect] Following redirect to https://lms.thegateacademy.com/
[generic] lms.thegateacademy: Requesting header
WARNING: Falling back on generic information extractor.
[generic] lms.thegateacademy: Downloading webpage
[generic] lms.thegateacademy: Extracting information
ERROR: Unsupported URL: https://lms.thegateacademy.com/
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\YoutubeDL.py", line 797, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\common.py", line 530, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\ytdl-org\tmpjwbwqymm\build\youtube_dl\extractor\generic.py", line 3352, in _real_extract
youtube_dl.utils.UnsupportedError: Unsupported URL: https://lms.thegateacademy.com/

Example-1:
Reference URL: https://lms.thegateacademy.com/dashboard/course/83/subject/225/chapter/1147/view/video/595/U2FsdGVkX1%2FT708sw5injyq%2FRi3VpxMhA1XY80Ju9xtV8ec3J8K80QM4QIQW1x%2Bm6ZwzPkoQYk6KKjXdhclgRUAWjbgiMcN5GBFY5nUjrWI%3D
Video URL : https://d2fgiyp8yuap60.cloudfront.net/TypesofPileMethodsforQub-Part1of11541834627960_20190126/720p.m3u8
Example-2
Reference URL : https://lms.thegateacademy.com/dashboard/course/43/subject/78/chapter/161/view/video/7749/U2FsdGVkX18wRmUNrVaHrq9qb1nrcA9I08aN1kp%2Fb5UxDVhcJh5BZQx%2BTW%2FEmwrCyZv%2BO1GPo2licLPw3zb0Mg%3D%3D
Video URL : https://d2fgiyp8yuap60.cloudfront.net/Geotech_GTE_Origin_of_Soil_20191226/720p.m3u8

No, same happens in browser. No difference.

No, same happens in browser. No difference.

So, there is no way to download a video using example URL's


Ansible Playbook

Add Ansible Playbook For Fast Deploy haproxystats on CentOS 7.x
Update README File

Thanks a lot for your work, I am pretty sure a lot of people will appreciate it.

your welcome @unixsurfer


[Enhancement] - Add more options to spo site set

Can we enhance spo site set cmdlet to support additional parameter for the site properties which are currently missing in the command?
For example:

CommentsOnSitePagesDisabled
RequestFilesLinkEnabled
RequestFilesLinkExpirationInDays
ListsShowHeaderAndNavigation
DisableAppViews

I would like to contribute to this library for above enhancement request. Thank you!

Hi @ganesh-sanap, looks like a good idea to me. These options might come in handy.
Before you start working on it, could you specify the options you are going to add and their description?
Example:

Option
Description

--commentsOnSitePagesDisabled [commentsOnSitePagesDisabled ]
Disable comments on all site pages. Allowed values: true, false.


Prepare for web extension

@lostintangent
Prepared code tour extension to be web enabled with following changes:

use codicons instead of svgs and remove them
consolidate node and browser tasks into single webpack task
use browser end point
exclude unwanted files in .gitignore and .vscdeignore
fix task and launch configs

😎! @sandy08 1let's do my data preview ext. like that next ...
this wars 1 crafty PR :)


Currently used version of httpclient contains multiple security vulnerabilities

In our exploration of your project we found that it is currently using version 4.2 of httpclient which is vulnerable to Improper Input Validation, Man-in-the-Middle (MitM), Denial of Service (DoS), and Directory Traversal.  See the links below for the details of each vulnerability.
Recommendation:
Upgrade the version of httpclient in the pom.xml located in ag-player to version 4.5.5 or higher.
For additional details on this vulnerability you can visit the following websites:
Snyk:
Improper Input Validation: https://snyk.io/vuln/SNYK-JAVA-ORGAPACHEHTTPCOMPONENTS-30645
Man-in-the-Middle (MitM): https://snyk.io/vuln/SNYK-JAVA-ORGAPACHEHTTPCOMPONENTS-30646
Denial of Service (DoS): https://snyk.io/vuln/SNYK-JAVA-ORGAPACHEHTTPCOMPONENTS-30647
Directory Traversal: https://snyk.io/vuln/SNYK-JAVA-ORGAPACHEHTTPCOMPONENTS-31517

Thank you for bring that, would you like to prepare a pull request for the changes?

Sure, I'll try to take care of that today.

Great, let me know if you hit any issues.


Finally, your UI..it needs a total revamp, right now it causes my eyes to hurt

If you don't know design, copy from sample sites that look appealing or meet someone .
The UI is not appealing at all, please make it good but fix all other issues and make sure those are working appropriately first.

Still working on this


Execution failed for task ':app:desugarDebugFileDependencies'

Launching lib/main.dart on Android SDK built for x86 in debug mode...

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':app:desugarDebugFileDependencies'.
> A failure occurred while executing com.android.build.gradle.internal.tasks.Workers$ActionFacade
   > /home/(username)/Android/Sdk/build-tools/28.0.3/core-lambda-stubs.jar

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 41s
Gradle task assembleDebug failed with exit code 1
Exited (sigterm)

Hi @Grigary-C-Antony
This seems to be related to build-tools/28.0.3, check this stackoverflow
please delete 28.0.3,  download builds tools 30.0 and try again
Thank you

Hi @Grigary-C-Antony
This seems to be related to build-tools/28.0.3, check this stackoverflow
please delete 28.0.3, download builds tools 30.0 and try again
Thank you

Thank you brother it worked on me after when i started a new project , i think error may be because of build so i flutter cleaned the same and it worked for me and
thank you for the help brother @TahaTesser