PAMA - Handle insufficient info / no rating scenario

Scenario 1 gives the example that given a code = CPT 71275 (CT angiogram of chest with contrast...) and reasonCode = SNOMED 267036007 (Dyspnea), there should be no rating due to insufficient information.
According to Argonaut guidelines, this means that the service should not return a system action to apply a pama rating, but rather offer a card with an app link or suggestion.  i.e. the first step is not returning a system action.
Testing this scenario in the sandbox produces a pama rating of 'no-guidelines-apply' rather than not having a rating..
Request:
{
  "hookInstance": "4654f34f-36e7-43df-8cd2-22491647817a",
  "hook": "order-select",
  "fhirServer": "https://launch.smarthealthit.org/v/r2/fhir",
  "context": {
    "patientId": "smart-1288992",
    "userId": "Practitioner/COREPRACTITIONER1",
    "selections": [
      "ServiceRequest/example-request-id"
    ],
    "draftOrders": {
      "resourceType": "Bundle",
      "entry": [
        {
          "resource": {
            "resourceType": "ServiceRequest",
            "id": "example-request-id",
            "status": "draft",
            "intent": "plan",
            "code": {
              "coding": [
                {
                  "code": "71275",
                  "display": "Computed tomographic angiography, chest (noncoronary), with contrast material(s), including noncontrast images, if performed, and image postprocessing",
                  "system": "http://www.ama-assn.org/go/cpt"
                }
              ],
              "text": "Computed tomographic angiography, chest (noncoronary), with contrast material(s), including noncontrast images, if performed, and image postprocessing"
            },
            "subject": {
              "reference": "Patient/smart-1288992"
            },
            "reasonCode": [
              {
                "coding": [
                  {
                    "code": "267036007",
                    "display": "Dyspnea (finding)",
                    "system": "http://snomed.info/sct"
                  }
                ],
                "text": "Dyspnea (finding)"
              }
            ]
          }
        }
      ]
    }
  }
}

CDS Service Reply:
{
  "cards": [],
  "extension": {
    "systemActions": [
      {
        "resource": {
          "resourceType": "ServiceRequest",
          "id": "example-request-id",
          "status": "draft",
          "intent": "plan",
          "code": {
            "coding": [
              {
                "code": "71275",
                "display": "Computed tomographic angiography, chest (noncoronary), with contrast material(s), including noncontrast images, if performed, and image postprocessing",
                "system": "http://www.ama-assn.org/go/cpt"
              }
            ],
            "text": "Computed tomographic angiography, chest (noncoronary), with contrast material(s), including noncontrast images, if performed, and image postprocessing"
          },
          "subject": {
            "reference": "Patient/smart-1288992"
          },
          "reasonCode": [
            {
              "coding": [
                {
                  "code": "267036007",
                  "display": "Dyspnea (finding)",
                  "system": "http://snomed.info/sct"
                }
              ],
              "text": "Dyspnea (finding)"
            }
          ],
          "extension": [
            {
              "url": "http://fhir.org/argonaut/Extension/pama-rating",
              "valueCodeableConcept": {
                "coding": [
                  {
                    "system": "http://fhir.org/argonaut/CodeSystem/pama-rating",
                    "code": "no-guidelines-apply"
                  }
                ]
              }
            },
            {
              "url": "http://fhir.org/argonaut/Extension/pama-rating-qcdsm-consulted",
              "valueString": "example-gcode"
            },
            {
              "url": "http://fhir.org/argonaut/Extension/pama-rating-consult-id",
              "valueUri": "urn:uuid:1720661a-6cd4-4b46-91a8-1b139ff7db96"
            }
          ]
        },
        "type": "update"
      }
    ]
  }
}

Thanks @dennispatterson! We should add a test case for this and fix. @barabo will be good to look at this during the connectathon.

Closing based on https://github.com/cds-hooks/sandbox-cds-services/pull/19


Local fs: metadata handling and share persistence

@ishank011 can we have the recycle and versions on the .shadow directory also?

@labkode Cool. Will update the paths

@labkode We would have to modify the namespace resolution in that case. When unwrapping paths such as /root/.shadow/einstein/home/MyShares/a.txt, there will be longer matches with the versions and recycle bin paths. So should I create a separate directory for the references as well? Eg: /root/.shadow/.references/einstein/home/MyShares/a.txt


Generate summary for the class

Is there a way to generate summary of the whole class with this work?

It should but you need to train the model first with a class-level summary dataset.


Blender Color Management and IT

I have been using Blender own Window to display Render Result so far, and I am also using the Filmic Color Management.
Just today, I started using It and with It, the view render matches the last setting from Filmic Color Management, however, when I save the image use Ctrl+S and save PNG, I am not getting the same result. As if, there is no Color Management being applied.
How is the best way to deal with this? Rendering with It, the advantage is the IPR. While I like the to also use the Filmic Color Management.

Switching Color Space from Linear to sRGB and color seems to be matching in It. But anyhow It does not connect to Blender's Color Management (Filmic).

Blenders filmic color management is a post process on a rendered image. IT gets its image directly from RenderMan itself (completely bypassing Blenders image editor).  So there's really no way to have the filmic color management performed when using IT as the viewer
Sent from my iPhone
On Jan 11, 2017, at 12:32 AM, Jimmy Gunawan <notifications<EMAIL_ADDRESS>wrote:
Switching Color Space from Linear to sRGB and color seems to be matching in It. But anyhow It does not connect to Blender's Color Management (Filmic).

You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/bsavery/PRMan-for-Blender/issues/435#issuecomment-271789864, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AM5RFNA1ECX7t7MbmYQl7f8NeBayFx_kks5rRHeSgaJpZM4LgPTx.

Actually, I think you can use OpenColorIO in It to add the filmic color LUTs.
See:

https://renderman.pixar.com/resources/current/RenderMan/itFeatures.html#remapping
https://github.com/sobotka/filmic-blender

It should also be noted that there is a display filter (which you can add
in the scene settings tab) called "FilmicToneMapper" which does almost
exactly the same thing as the Filmic Color management.
On Wed, Jan 11, 2017 at 5:07 AM, Philippe Leprince<EMAIL_ADDRESS>
wrote:

Actually, I think you can use OpenColorIO in It to add the filmic color
LUTs.
See:

https://renderman.pixar.com/resources/current/RenderMan/
itFeatures.html#remapping
https://github.com/sobotka/filmic-blender

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bsavery/PRMan-for-Blender/issues/435#issuecomment-271863921,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABzffCzA0RX8R8SWB63nnNOMW5YOUHhbks5rRNQogaJpZM4LgPTx
.

--
<EMAIL_ADDRESS>508-274-8700

Thanks for suggestions and answers! I'll give that OpenColorIO and FilmicToneMapper a try!


Switching from Bitbucket to GitHub, Installing Setuptools documentation not updated

The installation shell commands here
https://pypi.python.org/pypi/setuptools/3.4.1
have links to bitbucket but now the repository is on GitHub. You get a 404 error.
https://bootstrap.pypa.io/ez_setup.py is the new URL.
I believe that the documentation should be updated.

Your link references an old version of Setuptools. I believe the current releases include current references. I'm sorry the old notes contain old, broken references, but I don't believe it's possible to update those references, and even if it was, I don't think we have the resources to update all of them. Thanks anyhow for the report.

This was the first result on Google.  on the top.


Unable to generate Cucumber reports or json file If we configure test repository in POM

Archive.zip
download the file and run mvn clean install
you will get an error and remove the test resource tag from the pom and everything works as expected

src/test/resources

works fine for me when I follow the documentation here: https://github.com/karatelabs/karate#folder-structure
    <testResources>
        <testResource>
            <directory>src/test/java</directory>
            <excludes>
                <exclude>**/*.java</exclude>
            </excludes>
        </testResource>
    </testResources>

next time try to be more specific as to what you mean by an error


Responsive tailwind site

TODO: Only home page is rewritten, other pages need to be worked.

Site updated to use Astro


[doctest] Update to 2.4.8

What does your PR fix?
Updates doctest from 2.4.6 to 2.4.8

Which triplets are supported/not supported?

Yes

LGTM, thanks for the update!


Plugin was removed ( require-vuejs )

I saw that the plugin require-vuejs ( created for me ) was removed in commit b6296be86fed0de3ed238aa5398ae6f068d60a24.
There are any reason for this ?
There are any thing that a can do to solve it?

@edgardleal, I think at that time it was breaking some of the rules, probably it was considered unmaintained or it was not supporting Vue 2.0. I am sorry if that's not true and if it respects the rules feel free to add it again.

Ok. Thank you for feedback. Can you send me the link of these rules?

I found it here https://github.com/vuejs/awesome-vue/blob/master/CONTRIBUTING.md.
I will check all the rules and if the plugin don´t breaking any of these rules, I will send a new pull request.

A created a pullrequest https://github.com/vuejs/awesome-vue/pull/1303 and I will wait for merge.


validate: allow stable-3.2 to run with ansible 2.4

Although this is not officially supported, this commit allows
stable-3.2 to run against ansible 2.4.
This should ease the transition in RHOSP.
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test pipeline


Permissions module - first iteration

This PR is for a long-running feature branch, permissions. ACL-related code (to read and write .acl resources from the client) will go into here first. Once we have consensus on the first iteration (agreement with the spec, feedback from devs, etc), it'll make its way into master.

(rebased this branch on latest master)

@ghanemabdo - don't forget to rebase any further code on this branch.
Specifically, assuming you have an upstream remote that you set up for this repo, via
git remote add upstream https://github.com/solid/solid.js.git
then you should do:
git fetch upstream
git rebase upstream/permissions

Ping me if you run into any conflicts or anything.

(Since it's relevant to this feature) Here's a PR to a solid-specific Web Access Control spec: https://github.com/solid/web-access-control-spec/pull/1

Friendly reminder that this PR is still failing to pass tests.

Force-rebased permissions on latest master. @ghanemabdo - be sure to check out a fresh copy of permissions, and work off of that.

@ghanemabdo - I added several sections to the WAC spec yesterday that are relevant to this PR, especially the ACL Inheritance Algorithm section.

(Rebased on master)

@ghanemabdo - I'm taking a look at the Permissions code currently, with an eye towards eventually merging this PR into master. Feedback as I go along:

Let's rename Owner permission type to Control, since we don't have a notion of Owners in WAC.

@dmitrizagidulin I added a lot of modifications to this merge. Briefly:

Added support to read and write the predicate defaultForNew. This is also reflected in the demo page.
Added Agent object to encapsulate the information of an agent WebID, permissions, is a list or an individual, this authorization is defaultForNew
Several bug fixes.

Remaining items:

Conforming to the new inheritance way in getPermissions and getAclResource where GET on an individual acl resource url always returns result.
Modifying the README file to reflect the most recent changes in permissions.js
Fix the bug when accessTo predicate has absolute url which doesn't work in some cases. Needs deeper investigation.

Just pushed some refactoring of the new code from today's PR.

Wrapped some more lines to 80 chars
Removed unneeded .catch() statements
Removed redundant return new Promise(...) usage (functions that return promises can be returned directly).

Rebased on current master/HEAD, squashed commits into one for easier rebasing in the future.

@ghanemabdo I have some thoughts for refactoring this permissions api. Let's 1) make the permissions list keyed by agent (instead of keyed by permission mode), and 2) make the API cascading. For example.
Proposed API:
// First, we have our current getPermission() method.
solid.getPermissions('https://www.example.com/file1')  // <- already implemented
  .then(function (permissionSet) {
  })
// ^ is equivalent to:
solid.web.head('https://www.example.com/file1')  // or .get(...)
  .then(function (resource) {  // resource is an instance of the usual SolidResponse
    return resource.permissions()  // this figures out the .acl resource and fetches it, or throws a 401
  })
  .then(function (permissionSet) {
  })

// Ok, so, once we have our set of permissions, we can edit them via a chaining API:
var acl = solid.permission.acls
solid.getPermissions('https://www.example.com/file1')
  .then(function (permissionSet) {
    return permissionSet
      .addPermission(webId, acl.READ)
      .addPermission(webId2, [acl.READ, acl.WRITE])
      .addPermission(acl.EVERYONE, acl.READ)  // public / agentClass foaf:Agent
      .addGroupPermission(groupWebId, acl.READ)  // adds an agentClass entry
      .removePermission(webid3, acl.READ)
      .save()
  })

// Couple things to notice:
// - add/remove methods accept either a single value or an array
// - each add/remove is just queueing the operation,
//    no request is made until save() is called

// Deleting all permissions / resetting them would be done like so:
solid.getPermissions('https://www.example.com/file1')
  .then(function (permissionSet) {
    return permissionSet
      .clear()
      .save()
  })

// For existing permissions, clients can iterate through authorizations
// each authorization belongs to only one user or group (or everyone),
// and has multiple permission modes
solid.getPermissions('https://www.example.com/file1')
  .then(function (permissionSet) {
    permissionSet.forEach(function (authz) {    // loop through every authorization in an .acl
      if (authz.everyone()) { /* this is agentClass foaf:Agent */ }
      if (authz.group) { /* this is a group / agentClass */ }
      if (authz.agent) { /* this is for an individual agent */ }
      authz.iri  // -> webId of an agent or a group/agentClass
      // More than one of the following can be true:
      authz.allowsRead()  // -> true if authorization contains acl:Read mode
      authz.allowsWrite()  // -> true if acl:Write
      authz.allowsAppend()
      authz.allowsControl()
    })
  })

Note that there's no API for defaultForNew -- if the permission set is for a Container, then all authorizations are implicitly defaultForNew. If the permission set is for a regular non-container Resource, then it cannot have defaultForNew. Either way, the client doesn't need to know about it.

Currently, after the last refactoring, the permissions are keyed by authorization hash.
I totally agree, your new design to have cascading method calls for adding/removing auths is neat. I will have it on top of my list this week.

Currently, after the last refactoring, the permissions are keyed by authorization hash.

You're right. I look at it as essentially keyed by user, though. It uses a hash instead of the full agent's webId because it's shorter. And the accessTo resourceUrl is concatenated to the webId on the off chance that an .acl file will contain permissions for some other file (that's not its primary target). And so, in that one case, it makes sense to have multiple entries for one user. So the resourceUrl concatenation is to provide uniqueness.

Ok, merged the refactoring into this main permissions branch.
Heads up - I added another dev dependency, so be sure to run npm install when you fetch this branch.

Squashed commits, rebased on latest master.

Initial functionality implemented, README updated. Merging into master.
Thank you @ghanemabdo for all your help and hard work on this!


ivi-input-controller: ivi-controller: Copy over weston internal functions

Both ivi-controller and ivi-input-controller access functions which are
internal to weston, which means the ABI of these functions may change or
they may even be completely removed. So far, this wayland-ivi-extension
was able to link against libweston and use those functions by sheer luck,
but that may eventually run out. Copy over the internal functions as of
weston commit 2c1ed289 ("build: bump to version 9.0.92 for the beta release")

@efriedrich  if you could have a look at those few new MRs, that would be nice, thanks.

@marex, CC: @efriedrich,
Apologies for the late reply.
We had raised a PR in Weston a while back to make the below functions accessible out of core compositor(https://gitlab.freedesktop.org/wayland/weston/-/merge_requests/503), but it is not yet merged.

weston_touch_has_focus_resource
weston_output_damage
weston_compositor_read_presentation_clock

We will try to get it merged.
Also checking if we could use weston_surface_damage() instead of weston_surface_schedule_repaint() which is available in libweston.h.

I tested this pull request on Weston 8 with my list of test cases. It works great.

Just fyi, Haven't checked the others but the output damage is probably not needed. Dirtying and inflicting damage on surface would be sufficient to cause a proper repaint. Also, there's weston_compositor_damage_all() if you really want that.

Hi @mv0,
thanks for you comment!
we are using it only for screenshot functionality, and it is implemented output specific. only required output will be touched.
I'm in favor to have copy of this functions in wayland-ivi-extension temporary to avoid warnings and then rework it once we are compatible with the weston master release


Add instrumentation for OpenTelemetry API itself

Initial implementation for #10
The instrumentation hijacks OpenTelemetry.getTracerFactory() and returns a bridge that points at the shaded OpenTelemetry API.
Note: this could lead to ClassCastException if user calls this and then casts the result to a concrete implementation, e.g.:
(TracerSdkFactory) OpenTelemetry.getTracerFactory()

This initial implementation does not attempt to solve this problem.

Yes, great point, I just pushed comments, let me know if those make sense.


feat(console): add support for multiple input files

Description
Supports multiple OSCAL files as input as lula console -f <comma separated list>
This will merge the same model types (where possible) and create an output file name for any models that will be written (currently only component is supported in write, but assuming future models will be added)
Also added an option to specify output file names (again, only component is supported, but as more models get added for "write" capabilities, assuming the flags will grow)
Related Issue
Fixes #728
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Other (security config, docs update, etc)

Checklist before merging

[x] Test, docs, adr added or updated as needed
[x] Schema Updates applied
[x] Contributor Guide Steps followed

Conflict needing resolve otherwise looks good.


Add support for os-families-exclude in extensions composer.json

Fix #106
Allows to specify OS families which the extension is incompatible with:
{
    "name": "myvendor/myext",
    "php-ext": {
        "extension-name": "myext",
        "configure-options": [
            // ...
        ],
        "os-families-exclude": ['windows', 'darwin']
    }
}

When the syntax is validated, I'll create a PR in Composer to allow the new key in the Package::$phpExt property.
Related to https://github.com/ThePHPF/pie-design/pull/29

Revamped the whole PR to address your comments.

Things are done in ResolveDependencyWithComposer instead of CommandHelper
An enum holding OS families is added
It is added to the TargetPlatform
PhpBinaryPath is used to determine the OS family, just like the OS

This is great, love to exclude windows.

Remaining CS warnings should be fixed once https://github.com/composer/composer/pull/12218 is merged

PR composer/composer#12218 is now merged; please can you update the composer req, that should fix the static analysis

Here you go, all green! 🙂

It's now tagged as part of composer 2.8.4

Thanks!


all the points are predicted to be the same class

hi，thanks for your contribution！
when i using my own s3dis like dataset for training an Instance Segmentation model，i just got an error output：all the points are predicted to be the same class（background）。
here are my trainning logs and my dataset。can you give me some advice？

Hi, could you provide a more detailed train.log?

Hi, could you provide a more detailed train.log?

train.log

Hi, I checked the log,

It seem that background should be treated as a ignore segment?
Also to ensure that trainining augmentation is aligned with validation alignment, maybe add an centershift after crop is better;
Some scale parameter for PointGroup should be adjust, but I am not an expert, just do more experiment.


Failed to create MMAL component b'vc.camera_info': Function not implemented

After I installed libraspberrypi-bin (and change permissions to /dev/vchiq/) I was able to use $ raspistill -o output.jpg without error. I did not build userland from source so I think this error is different to Error #334 and Error #540.
I used pip (in virtual environment) to install picamera and have version 1.13.
I attempt to run a simple script that calls the PiCamera class and I get the following error

$ python3 cameratest.py
mmal: mmal_component_create_core: could not find component 'vc.camera_info'
Traceback (most recent call last):`
  File "cameratest.py", line 4, in <module>
    camera = PiCamera()
  File "/home/ubuntu/environments/test_env/lib/python3.8/site-packages/picamera/camera.py", line 367, in __init__
    with mo.MMALCameraInfo() as camera_info:
  File "/home/ubuntu/environments/test_env/lib/python3.8/site-packages/picamera/mmalobj.py", line 2346, in __init__
    super(MMALCameraInfo, self).__init__()
  File "/home/ubuntu/environments/test_env/lib/python3.8/site-packages/picamera/mmalobj.py", line 631, in __init__
    mmal_check(
  File "/home/ubuntu/environments/test_env/lib/python3.8/site-packages/picamera/exc.py", line 184, in mmal_check
    raise PiCameraMMALError(status, prefix)
picamera.exc.PiCameraMMALError: Failed to create MMAL component b'vc.camera_info': Function not implemented

$ uname -a
Linux ubuntu 5.4.0-1028-raspi #31-Ubuntu SMP PREEMPT Wed Jan 20 11:30:45 UTC 2021 aarch64 aarch64 aarch64 GNU/Linux

I'm running ubuntu 20.04 server on RPi4
$ vcgencmd get_camera
supported=1 detected=1

$ ldconfig -p | grep libmmal
	libmmal_vc_client.so (libc6,AArch64) => /lib/aarch64-linux-gnu/libmmal_vc_client.so
	libmmal_util.so (libc6,AArch64) => /lib/aarch64-linux-gnu/libmmal_util.so
	libmmal_core.so (libc6,AArch64) => /lib/aarch64-linux-gnu/libmmal_core.so
	libmmal_components.so (libc6,AArch64) => /lib/aarch64-linux-gnu/libmmal_components.so
	libmmal.so (libc6,AArch64) => /lib/aarch64-linux-gnu/libmmal.so

$ ldd /lib/aarch64-linux-gnu/libmmal.so
	linux-vdso.so.1 (0x0000ffffa48de000)
	libmmal_core.so => /lib/aarch64-linux-gnu/libmmal_core.so (0x0000ffffa4879000)
	libmmal_util.so => /lib/aarch64-linux-gnu/libmmal_util.so (0x0000ffffa4858000)
	libvcos.so => /lib/aarch64-linux-gnu/libvcos.so (0x0000ffffa483c000)
	libc.so.6 => /lib/aarch64-linux-gnu/libc.so.6 (0x0000ffffa46c9000)
	/lib/ld-linux-aarch64.so.1 (0x0000ffffa48ae000)
	libpthread.so.0 => /lib/aarch64-linux-gnu/libpthread.so.0 (0x0000ffffa4699000)
	libdl.so.2 => /lib/aarch64-linux-gnu/libdl.so.2 (0x0000ffffa4685000)

Please advise on how to solve the mmal: mmal_component_create_core: could not find component 'vc.camera_info' error

MMAL isn't currently supported on a 64bit userspace.
Ubuntu have taken some patches for 64bit userspace that were reverted due to observing issues with them. That decision really falls on them to support.

@6by9 thanks. I'm not familiar with this domain but if 64bit processor supports 64bit and 32bit then couldn't I (somehow) build MMAL for 32bit?

Generally multiarch isn't supported. It's possible, but not simple.

Ok, thanks! I guess then I have to wait till they update the firmware...

Same exact issue, same kernel, now four months passed - any hopes for soon?
Where should we subscribe to be alerted to the fix when it happens?
Reason this is a big issue:  The ROS community is pushing for adoption of ROS2 (Foxy LTS) which requires Ubuntu Server 20.04 LTS.  ROS 2 robots want to use Python3.8 with picamera to "see".

Is there any update on this issue?

It seems we all need to learn to use libcamera and the standard V4L libraries.
https://www.raspberrypi.com/news/an-open-source-camera-stack-for-raspberry-pi-using-libcamera/


EUR-Wallet: APIError (id=invalid_request): Can't buy with this account

Dear guys,
i am trying to do a trade on coinbase via script. I can pull the prices, i get the account data, the payment methods and so on....finally i would like to buy some coins with my EUR-Wallet. There is some amount of money on this account for testing purposes.
The ID of the payment method etc shall be chosen correctly but nevertheless the error above in the headline arises. I would be glad for your experiences and help;) with kind regards, John
In the following ,i will try to describe all the actions done in my script for reproducing this error:
accounts=client.get_accounts()       #get my accounts (i have EUR-Wallet, ETH, LTC and BTC Accounts)
payment_method=client.get_payment_methods()[0]     #Chooses the EUR-Wallet as my paymentmethod

account=client.get_account("abcdefghihklmnoqrstuvwxyz") # Argument is the ID of EUR-Wallet

buy = account.buy(amount='0.00001', currency="LTC", payment_method=payment_method.id)

print payment_method

shows:
{
  "allow_buy": true,
  "allow_deposit": true,
  "allow_sell": true,
  "allow_withdraw": true,
  "created_at": "2017-10-09T19:12:29Z",
  "currency": "EUR",
  "fiat_account": {
    "id": "abcdefghihklmnoqrstuvwxyz",
    "resource": "account",
    "resource_path": "/v2/accounts/abcdefghihklmnoqrstuvwxyz"
  },
  "id": "1234567891011121314151617",
  "instant_buy": true,
  "instant_sell": true,
  "limits": {
    "buy": [
      {
        "description": "\u20ac9,000 of your \u20ac9,000 weekly Coinbase account limit remaining",
        "label": "Total EUR limit",
        "next_requirement": {
          "type": "identity_verification"
        },
        "period_in_days": 7,
        "remaining": {
          "amount": "9000.00",
          "currency": "EUR"
        },
        "total": {
          "amount": "9000.00",
          "currency": "EUR"
        }
      }
    ],
    "name": "Coinbase Account",
    "sell": [
      {
        "description": "\u20ac9,000 of your \u20ac9,000 weekly Coinbase account limit remaining",
        "label": "Total EUR limit",
        "next_requirement": {
          "type": "identity_verification"
        },
        "period_in_days": 7,
        "remaining": {
          "amount": "9000.00",
          "currency": "EUR"
        },
        "total": {
          "amount": "9000.00",
          "currency": "EUR"
        }
      }
    ],
    "type": "fiat_account"
  },
  "name": "EUR Wallet",
  "primary_buy": true,
  "primary_sell": true,
  "resource": "payment_method",
  "resource_path": "/v2/payment-methods/1234567891011121314151617",
  "type": "fiat_account",
  "updated_at": "2017-10-09T19:12:29Z",
  "verified": true
}

So as you can see i have the correct payment_method ID as well as the correct EUR-Wallet ID. And all my accounts are allowed to deal with the whole set of requests that coinbase API can offer. this is done in the configuration on my coinbase account.
I would be glad for support.
This is the full error message:
Traceback (most recent call last):
  File "/home/pi/Desktop/First_Script.py", line 265, in <module>
    payment_method=payment_method.id)
  File "/usr/local/lib/python2.7/dist-packages/coinbase-2.0.6-py2.7.egg/coinbase/wallet/model.py", line 200, in buy
    return self.api_client.buy(self.id, **params)
  File "/usr/local/lib/python2.7/dist-packages/coinbase-2.0.6-py2.7.egg/coinbase/wallet/client.py", line 381, in buy
    response = self._post('v2', 'accounts', account_id, 'buys', data=params)
  File "/usr/local/lib/python2.7/dist-packages/coinbase-2.0.6-py2.7.egg/coinbase/wallet/client.py", line 132, in _post
    return self._request('post', *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/coinbase-2.0.6-py2.7.egg/coinbase/wallet/client.py", line 116, in _request
    return self._handle_response(response)
  File "/usr/local/lib/python2.7/dist-packages/coinbase-2.0.6-py2.7.egg/coinbase/wallet/client.py", line 125, in _handle_response
    raise build_api_error(response)
InvalidRequestError: APIError(id=invalid_request): Can't buy with this account

and this the relevant section in the package that throws the error:
def _request(self, method, *relative_path_parts, **kwargs):
    """Internal helper for creating HTTP requests to the Coinbase API.

    Raises an APIError if the response is not 20X. Otherwise, returns the
    response object. Not intended for direct use by API consumers.
    """
    uri = self._create_api_uri(*relative_path_parts)
    data = kwargs.get('data', None)
    if data and isinstance(data, dict):
      kwargs['data'] = encode_params(data)
    if self.VERIFY_SSL:
      kwargs.setdefault('verify', COINBASE_CRT_PATH)
    else:
      kwargs.setdefault('verify', False)
    kwargs.update(verify=self.VERIFY_SSL)
    response = getattr(self.session, method)(uri, **kwargs)
    return self._handle_response(response)

  def _handle_response(self, response):
    """Internal helper for handling API responses from the Coinbase server.

    Raises the appropriate exceptions when necessary; otherwise, returns the
    response.
    """
    if not str(response.status_code).startswith('2'):
      raise build_api_error(response)
    return response

  def _get(self, *args, **kwargs):
    return self._request('get', *args, **kwargs)

  def _post(self, *args, **kwargs):
    return self._request('post', *args, **kwargs)

  def _put(self, *args, **kwargs):
    return self._request('put', *args, **kwargs)

Hey @jojohnjo, strongly recommend you contact our support team to understand what is going on with your account. Thanks!


Add ferma-extension API code to main ferma codebase

Codecov Report

Merging #23 into v2.2.x will decrease coverage by 0.19%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             v2.2.x      #23     +/-   ##
===========================================
- Coverage     46.09%   45.89%   -0.2%
  Complexity      446      446
===========================================
  Files            45       46      +1
  Lines          2076     2085      +9
  Branches        241      241
===========================================
  Hits            957      957
- Misses          974      983      +9
  Partials        145      145

Impacted Files
Coverage Δ
Complexity Δ

src/main/java/com/syncleus/ferma/tx/Tx.java
0% <0%> (ø)
0 <0> (?)

...ncleus/ferma/traversals/AbstractEdgeTraversal.java
27.04% <0%> (-0.46%)
20 <0> (ø)

...leus/ferma/traversals/AbstractVertexTraversal.java
60.6% <0%> (-0.75%)
65 <0> (ø)

Continue to review full report at Codecov.

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

@Jotschi I'm going to approve this, looks good. Thanks a lot.

@Jotschi Feel free to add yourself to the contributors list if you arent already there.

@freemo  Okay. Sounds good. Thanks for the merge :-)


Solution/Project Naming Convention

I was curious as to why the root namespace is not following the typical PascalCase naming conventions was this intended or just a mistake and if so would you be open to some convention changes?

As for original intent, you'd have to ask the original developer @rickbutton that question :smile:
As for now, I think changing it will mean we break lots of configs out there relying on the casing being as it is. And I don't really like making breaking changes when they're not needed.
My 2 cents.

If you want to clean up minor things here and there to improve the overall consistency of the code-base in non-breaking ways, I don't think you'll find any objections there. :smile:

I'm not sure whats going on with these extras newlines maybe he fell asleep with his face on the enter button?

I care not for white space in a side project I wrote to a specific problem. White space is for the tooling to figure out, if it becomes a problem, @josteink feel free to install some kind of C# linter.
On namespaces; I didn't use Workspacer because the project is called workspacer, not Workspacer. I don't see any reason to change it.


Do not call slack if SLACK_URL is not set.

Hi @KrauseFx !
I've tried to look into the dsym issue #31 in gym an have bump into a No SLACK_URL given. error when running fastlane test.
So my guess is to prevent slack from being called in this generic Fastfile if the environment variable is not set because it can prevent the local build of all Fastlane tools.

Hi @Liquidsoul, thanks for your pull request. This Fastfile is the Fastfile which is used to deploy fastlane itself. Why do you want to update it? Did you want to edit the slack action instead?

Maybe my intent was not express clearly enough here, sorry :smile:
It's when I tried to launch fastlane test on the gym repository that I found that it was failing because of the slack action in this included Fastfile.
I do not want to change the way the slack action behaves when the SLACK_URL is not present because this is an expected failure for other users.
However, when contributing to fastlane or its associated tools, I find that it could be nice not to force every contributor to have a valid SLACK_URL environment variable if they want to use fastlane test (or any other lane).

@Liquidsoul Perfect, thanks! Sorry I didn't understand it immediately, but this makes perfect sense. I should also update all the repositories to explain how to run tests :+1:


Add locations

Summary
Add all needed to handle locations.
Ticket Link
None

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   58.18%   58.18%
=======================================
  Files           3        3
  Lines          55       55
=======================================
  Hits           32       32
  Misses         23       23

Continue to review full report at Codecov.

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

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   58.18%   58.18%
=======================================
  Files           3        3
  Lines          55       55
=======================================
  Hits           32       32
  Misses         23       23

Continue to review full report at Codecov.

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

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #18   +/-   ##
=======================================
  Coverage   58.18%   58.18%
=======================================
  Files           3        3
  Lines          55       55
=======================================
  Hits           32       32
  Misses         23       23

Continue to review full report at Codecov.

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

Also heads up about the large amount of conflicts due to another PR 🙂


HueyException  not found in TaskRegistry

When using django-huey, the consumer is able to correctly find the registered tasks, however when running the Django server and trying to enqueue a task, I get an "HueyException  not found in TaskRegistry" error. From doing wayy too much debugging, I finally tracked this down to the fact that the huey package initializes itself with RegistryA, and then django_huey initializes itself with a separate RegistryB. So when trying to enqueue a task, it attempts to find the task in RegistryA which is empty, because all of the tasks are in RegistryB.
I use import django_huey as huey everywhere, and then use @huey.task(queue="test"), but I'm still getting this error. I'm not sure what I'm doing wrong 😞

Hi, could you please attach a minimal code to reproduce the issue and specify django, huey versions

Hi @wgordon17 any update on the issue?

Sorry @pablop94, I had a few other issues on huey (like error mgmt, etc), so I've switched over to just using celery. Thank you anyway!

I have the same issue and two queues, configured to use the same Redis server.
So the name parameter is identical (not first_tasks and email_tasks).
If I put twice the same name, it seems that the second consumer get the tasks (and may be the wrong consumer).
If I put different names, the right consumer get the task.
How to have the same name, but things working?
Thanks

Hi, could you please attach your DJANGO_HUEY setting?
Why do you want two queues with the same name?

    @property
    def DJANGO_HUEY(self):
        prefix, suffix = Production.REDIS_URL["default"]["LOCATION"].split("//")
        url = f"{prefix}//:{Production.REDIS_PASSWORD}@{suffix}"
        pool = BlockingConnectionPool.from_url(
            url,
            max_connections=50,
            timeout=20,
            health_check_interval=30,
        )
        return {
            "default": "category_1",
            "queues": {
                "category_1": {
                    "huey_class": "huey.RedisHuey",  # Huey implementation to use.
                    "name": "default",  # Use db name for huey.
                    "results": True,  # Store return values of tasks.
                    "store_none": False,  # If a task returns None, do not save to results.
                    "immediate": False,  # If DEBUG=True, run synchronously
                    "utc": True,  # Use UTC for all times internally.
                    "blocking": True,  # Perform blocking pop rather than poll Redis.
                    "connection": {
                        "connection_pool": pool,
                    },
                    "consumer": {
                        "workers": 1,
                        "worker_type": "thread",
                        "initial_delay": 0.1,  # Smallest polling interval, same as -d.
                        "backoff": 1.15,  # Exponential backoff using this rate, -b.
                        "max_delay": 10.0,  # Max possible polling interval, -m.
                        "scheduler_interval": 1,  # Check schedule every second, -s.
                        "periodic": True,  # Enable crontab feature.
                        "check_worker_health": True,  # Enable worker health checks.
                        "health_check_interval": 1,  # Check worker health every second.
                    },
                },
                "category_2": {
                    "huey_class": "huey.RedisHuey",  # Huey implementation to use.
                    "name": "default",  # Use db name for huey.
                    "results": True,  # Store return values of tasks.
                    "store_none": False,  # If a task returns None, do not save to results.
                    "immediate": False,  # If DEBUG=True, run synchronously
                    "utc": True,  # Use UTC for all times internally.
                    "blocking": True,  # Perform blocking pop rather than poll Redis.
                    "connection": {
                        "connection_pool": pool,
                    },
                    "consumer": {
                        "workers": 2,
                        "worker_type": "thread",
                        "initial_delay": 0.1,  # Smallest polling interval, same as -d.
                        "backoff": 1.15,  # Exponential backoff using this rate, -b.
                        "max_delay": 10.0,  # Max possible polling interval, -m.
                        "scheduler_interval": 1,  # Check schedule every second, -s.
                        "periodic": True,  # Enable crontab feature.
                        "check_worker_health": True,  # Enable worker health checks.
                        "health_check_interval": 1,  # Check worker health every second.
                    },
                },
            },
        }

I want two queues. But use only once Redis instance (I am limited in Redis instances and would not create a second instance).
Maybe is it possible to use a prefix or something like that?

You can delete the name property or change it to different values, it will work.
If you want multiple tasks pointing to any of these queues (sometimes category_1, sometimes category_2), multiple queues is not what you need, you need only one queue and if you need it, you can add more workers.


Add hover effect to the community guide button.

Is your feature request related to a problem? Please describe.
If you want to give your page a little twist, putting CSS button hover effects is ideal. It will help improve your visitors' dwell time.
Describe the solution you'd like
I would like to add a hover effect on the community guide button, when hovering it changes color and button size increases.
Additional context
Before:

After: (Before hovering):

(After hovering): The color changed and size increased.

i would like to work on this issue, please assign this to me.

i would like to work on this issue, please assign this to me.

i have already raised a pr against this


Information on auth parameters information not clearly given

The parameters Oauthredirectmethod and authId are placeholders that are automatically replaced by teams whike making the authentication call. This information is missing in the documents and leads to error while testing the app wihtout using these parameters as placeholders
Below section needs to be changed
https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/how-to/authentication/auth-oauth-provider#add-authentication-to-external-browsers

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: b608448b-3dd7-d5be-128c-da2859067140
Version Independent ID: b608448b-3dd7-d5be-128c-da2859067140
Content: Use external OAuth providers - Teams
Content Source: msteams-platform/tabs/how-to/authentication/auth-oauth-provider.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Thanks for rising this issue. we will update the doc accordingly


Move whitelist processing to email dispatch

Move whitelist application from business logic - i.e. the composition of the Notification - to email dispatch.
The primary affect of this is that whitelists specified in notification.json should be specified using bare email addresses, not mailto: URIs or Fedora resource URIs.  Full whitelist processing is logged at DEBUG level.  Whitelist application is logged at INFO level.
Notification business logic attempts to be agnostic with regard to the underlying dispatch and transport of notifications.  Initially the whitelist was applied on the business logic side, which means that it ought only operate on URIs.  This is inconvenient and opaque when a human is configuring a whitelist for User resources in Fedora.  The whitelist would have to reference Fedora URIs rather than simple email addresses or mailto URIs, making it difficult as a human to 1) compose a whitelist, and 2) interpret a whitelist and see which actual email address are whitelisted.
If whitelist processing were to remain on the business logic side, but be updated to operate on bare email addresses, that would be a nod to the underlying dispatch and transport of notifications as emails (i.e. the abstraction provided by the Dispatch API leaks).  Another option would have whitelist processing remain on the BL side, but be updated to resolve URIs to email addresses, and apply the whitelist.  But again, that breaks the abstraction if another form of notification transport were to be added (e.g. SMS or flashes in the view layer).
Therefore, this commit moves whitelist processing to the dispatch side of the house.  Instead of whitelist processing occurring in the Composer on BL side, it now occurs in the EmailDispatchImpl EmailComposer.  Whitelists are composed of bare email addresses and not opaque URIs.  If multiple dispatch transports are added later, the model (and format) of notification services configuration will need to be updated so each dispatch implementation can process its own form of whitelist, if recipient filtering is desired for these new implmentations.
There are some consequences of this move.  The logging aspect which operates at the Dispatch API join point is unaware of whitelist processing.  So when it logs the successful or failed delivery of a Notification, the recipients listed on the Notification may have been filtered inside the dispatch implementation.  This PR addresses this by logging the application of the whitelist at INFO level.  DEBUG level reveals more information about whitelist processing.
Finally, there is a dichotomy in the current BL/Dispatch impl split which has been there from the beginning, which is the "global cc" configuration field.  It is a bare email address, not a mailto address, and that should be addressed to be consistent with the whitelist processing (e.g. the "global cc" should either be updated to carry URIs, or processing of the "global cc" should be moved to the dispatch side of the house).
Addresses #55.

This PR also includes fixes for the SimpleImapClient and its management of the IMAPStore


support to configure extra blacklist of iface in "up" command

Describe your changes
we can configure extra blacklist of iface in "up" command instead of editing config file
Issue ticket number and link
Checklist

[*] Is a feature enhancement

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Yes. we use netbird client as Daemonset in Kubernetes so that the selected nodes will work as gateways of the cluster. the change in PR is enough for us and works well.
For your point, I'm happy to see to make it be configurable too in traditional way when we run netbird up  in a "normal" node for instance a bare-metal node.  I would like to check the codebase to see how to make it possible with gRPC.
But before that, do you think it worth a try?

Yes. we use netbird client as Daemonset in Kubernetes so that the selected nodes will work as gateways of the cluster. the change in PR is enough for us and works well.
For your point, I'm happy to see to make it be configurable too in traditional way when we run netbird up in a "normal" node for instance a bare-metal node. I would like to check the codebase to see how to make it possible with gRPC.
But before that, do you think it worth a try?

I will sync with the team tomorrow morning and get back to you after. Thank you!

Update the PR to configure in daemon mode

@braginini hey man, thanks for you sync. I have update the pr to support the daemon mode.
Just find another PR not merged #1586. Seems it's going to support to configure InterfaceBlacklist too.
The ExtraIFaceBlacklist option make help people avoid to provide the default blacklist in the command line. But use InterfaceBlacklist will be more explicit.
Which do you think is better?


IntTimesPowerOfTenFormatter browser console warnings

When interacting with a dashboard, the browser console fills up with warnings about IntTimesPowerOfTenFormatter:

To reproduce:

Go to this dashboard: https://ui.rilldata.io/rilldata/rill-demo-partner-filtered-dashboards/c_column_security
Click on one of the dimension values in "Cost by Region"
See the warnings in the console

This indicates that some function is using the IntTimesPowerOfTenFormatter incorrectly. That function is designed to accept numbers that must be a single digit integer multiplied by a power of ten, and return a simplified string like those given in the examples below:
1  -->  "1"
30_000_000  -->  "30M"
10_000  -->  "10k"
10  -->  "10"
0.005  -->  "5e-3"
0.000_000_200  -->  "200e-9"

This is designed for situations where it was desired to have minimally cluttered numeric representations with only one digit of precision (and an order of magnitude if required) . the down stream formatters built around this assume that behavior, so it's possible that formatting may be mangled if that assumption is violated.
I'm not sure where exactly this is happening (I can't reproduce locally) but I suspect it might be occurring during re-scaling of the y axes, possibly when the axes flatten out to zero. I also suspect that updates during tweening might be involved.


Folders with too many files take too long to delete

Storage Explorer Version: 1.8.1
Platform/OS: Windows 10
Architecture: x64
Regression From: Not sure
Bug description
I have a folder with a ASP.NET Core website published, so inside it there are some .NET Core DLLs and some folders like wwwroot, which in turn has bootstrap, jquery, etc.
If I try to delete that folder, ASE gets stuck on "Discovering blobs for deletion from [...]". I've waited more than 3min and it was still stuck on that.
Steps to Reproduce

Publish a SCD ASP.NET Core website with some 10 popular JS libraries installed (or anything like this) and upload it to a folder on a blob storage container.
Try to delete the folder.

Expected Experience
My files are deleted in a reasonable time.
Actual Experience
My files are not deleted in a reasonable time.

@andre-ss6 About how many blobs are in this directory?

Maybe something like three thousand files.

Well, a friend tried to delete on his machine using this same version and it worked normally. 🤷♀
Although it still took some time to actually delete, the "Discovering blobs for deletion" message didn't even appear. If this happens again I'll be sure to report.

@andre-ss6 That'll work. Out of curiosity, did you try this with AzCopy turned on?

@craxal Yea, I forgot to mention that. Yes, I did.  For reference, my friend also used AzCopy on.


Feature/switch placer cost function

@srini229 @soneryaldiz This is just an idea, not something that you have to agree to do.
A simple way to normalize things with widely different units is to optimize the product (power-delay product is an typical example.)
What this amounts to for us is optimizing: exp(ln(area) + ln(hpwl)) or ln(area) + ln(hpwl) since exp is monotonic non-decreasing.
What is nice about this is the a 1% change is area is traded off with a 1% change in hwpl without having to invent scaling functions. You can see this by:
cost = ln a + ln h
0 = da / a + dh / h
da / a = -dh / h

If you want to tradeoff a 3% change in hwpl for a 1% change in area, then you'll want to optimize this cost function:
cost = 3 ln a + ln h = ln( a**3 * h)

This might be an easy way to control the optimizer to find different solutions along the Pareto frontier by sweeping the exponent.
I made some changes in this PR to show this in action. You can see in the GUI the that the existing normalization scheme is producing similar results to this one. I think using log  like this is more general and removes some tricking normalization code that people can argue about whether it is doing the right thing or not.

Added command line control for setting lambda_coeff. This allows steering the result to either area preferred (small positive l) or hpwl preferred (large l).
cost = a * h**l
ln cost = ln a + l ln h
0 = da / a + l dh / h
da / a = -l dh / h

A 1% change in area trades off with a 1/l % change in hpwl

@stevenmburns any function that brings the area and hpwl to a deterministic range should work fine. The problem we had before the normalization : cost function progression with temperature was dependent on circuit size. It was converging too quickly and not exploring other areas in the search space because of the area term dominating the cost.
ln() function alleviates the same problem. As you mentioned, it is better because of its smoothness as well.
The da/a = k * dh/h is what we tried to achieve with the normalization as well.
I can plot the cost progression with temperature with this cost and verify its behaviour.

@soneryaldiz I'm scaling the units from 0.5 nm to scale_factor units in placement_verilog_d before calling the Z3-based checker. The units in the GUI are still 0.5 nm, but I'm thinking that changing them to microns (and floating point) makes the most sense.

@srini229 @854768750 @Lastdaysends Please review and approve it this new cost function is okay.

@stevenmburns the cost progression with temperature looks good.
An example from the telescopic_ota

We might need to tweak the temperature setting so that the convergence is delayed. We might need some dynamic setting for the temperature or fast out if we see convergence in cost.

@soneryaldiz I'm scaling the units from 0.5 nm to scale_factor units in placement_verilog_d before calling the Z3-based checker. The units in the GUI are still 0.5 nm, but I'm thinking that changing them to microns (and floating point) makes the most sense.

microns for GUI sounds good.


small issue: OSError: Using the --html flag requires node v14 to be installed, but it was not found in your system.

when running this comamnd:
doc-builder preview audio-transformers-course ../audio-transformers-course/chapters/en --not_python_module
we got this:
Traceback (most recent call last):
File "/home/silvacarl/.local/lib/python3.8/site-packages/doc_builder/commands/build.py", line 31, in check_node_is_available
p = subprocess.run(
File "/usr/lib/python3.8/subprocess.py", line 493, in run
with Popen(*popenargs, **kwargs) as process:
File "/usr/lib/python3.8/subprocess.py", line 858, in init
self._execute_child(args, executable, preexec_fn, close_fds,
File "/usr/lib/python3.8/subprocess.py", line 1704, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
PermissionError: [Errno 13] Permission denied: 'node'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/silvacarl/.local/bin/doc-builder", line 8, in
sys.exit(main())
File "/home/silvacarl/.local/lib/python3.8/site-packages/doc_builder/commands/doc_builder_cli.py", line 47, in main
args.func(args)
File "/home/silvacarl/.local/lib/python3.8/site-packages/doc_builder/commands/preview.py", line 158, in preview_command
check_node_is_available()
File "/home/silvacarl/.local/lib/python3.8/site-packages/doc_builder/commands/build.py", line 40, in check_node_is_available
raise EnvironmentError(
OSError: Using the --html flag requires node v14 to be installed, but it was not found in your system.
any ideas?

cc @mishig25 can you take a look please?

Hi, downloading Node.js > 14.0 worked for me: https://nodejs.org/en/download/


Added a new Bridge class

Hi!  I added a Bridge class in order to view/change the Hue Bridge properties, and to programmatically "push" the link-button remotely.  It needs unit-test coverage, but works well so far in my own usage.  I added simple usage-documentation to your readme.  Thanks for your awesome gem!

Firstly this is great!
May I suggest adding a single method just for pushing the link button/linking:
def link!
  linkbutton = true
  save
end

That way its a simpler interface to auto linking..

Thanks for the suggestion robert2d; I added this method, as well as updated the bridge attributes since the Hue team has added quite a lot to the API (latest v1.5 I think).

Any chance of this getting merged :bulb:


Error resolving the running build's version number on non-default branch

Wanted to start a discussion prior to a pull request - not sure if this has been identified already.  Receiving the error below (sorry about formatting) when running TeamCityBuildChanges from a build off a branch called 'develop'.
Unhandled Exception: System.ApplicationException: Could not resolve a build number for the running build. [22:26:33][Step 1/4]    at TeamCityBuildChanges.Commands.AggregateBuildDeltaResolver.ResolveToVersion(String buildType) in d:\Work\TeamCityBuildChanges\TeamCityBuildChanges\Commands\AggregateBuildDeltaResolver.cs:line 198 [22:26:33][Step 1/4]    at TeamCityBuildChanges.Commands.AggregateBuildDeltaResolver.CreateChangeManifest(String buildName, String buildType, String referenceBuild, String from, String to, String projectName, Boolean useBuildSystemIssueResolution, Boolean recurse) in d:\Work\TeamCityBuildChanges\TeamCityBuildChanges\Commands\AggregateBuildDeltaResolver.cs:line 88 [22:26:33][Step 1/4]    at TeamCityBuildChanges.Commands.AggregateBuildDeltaCommand.Run(String[] remainingArguments) in d:\Work\TeamCityBuildChanges\TeamCityBuildChanges\Commands\AggregateBuildDeltaCommand.cs:line 37 [22:26:33][Step 1/4]    at ManyConsole.ConsoleCommandDispatcher.DispatchCommand(IEnumerable1 commands, String[] arguments, TextWriter consoleOut) in c:\src\ManyConsole\ManyConsole\ConsoleCommandDispatcher.cs:line 70
[22:26:33][Step 1/4]    at TeamCityBuildChanges.Program.Main(String[] args) in d:\Work\TeamCityBuildChanges\TeamCityBuildChanges\Program.cs:line 13
[22:26:42][Step 1/4] Process exited with code 255`
According to the api documentation if no branch is specified in the build locator then only builds from the default branch are returned.
branch: - since TeamCity 7.1 limit the builds by branch.  can be branch name (displayed in UI, or "(name:,default:<true/false/any>,unspecified:<true/false/any>,branched:<true/false/any>)". If not specified, only builds from default branch are returned.
Tempted just to use
branch:(default:any)

Interested in thoughts around the best solution prior to opening the code.

Shall we close this one off?


Add Reentry flag to contract invocation

What
Resolves https://github.com/stellar/rs-soroban-env/issues/336.
Add a reentry: RawVal argument to the try_call function. If set, it checks the host context for frames with the same contract id.
Why

Why only add it in try_call?  The call host function is the recommended (safe) path of invocation, where any failure will results in an host error.  The try_call is the more loose version that allows failure to be returned as a status. It makes sense to only enable re-entry in the try_call invocation path since that is a further loosening feature (@graydon has suggested it).
Why there is no checking function name? We assume if reentry is disabled, the same contract cannot be entered at all, even if the entry point is a different function than what had been called.

Known limitations
[TODO or N/A]

I haven't bumped the env version because I haven't figured out how to make the sdk changes (proc-macro) in order to regen the wasm tests.

It makes sense to only enable re-entry in the try_call invocation path since that is a further loosening feature (@graydon has suggested it).

I don't think it makes sense to couple reentry as a capability with error handling, as there are disconnected reasons to choose one and the other.
In the SDK I hope to add support for "invocation options" on functions that call both call and try_call, and it will muddy the SDK API if these options are inconsistent between the two functions.
At the moment these options will only include the reentry flag, but I imagine them including other options in the future.
cc @graydon

@jayz22 I think we should add new call and try_call functions that take the additional option. I think we can simply call them call_2 and try_call_2, and each time we add a new option we can add a new host function.


CMake is unable to find_package(Ruby) because of unusual with-soname in ruby package

CMake has certain expectations when running FindRuby.cmake. The ruby nixpkg does not meet those. The problem is that the .so file is named libruby_v2_6_5.so instead of libruby-2.6.5.so, which is one of the values in _RUBY_POSSIBLE_LIB_NAMES as shown below (one of values in _RUBY_POSSIBLE_LIB_NAMES CMake variable).
To Reproduce
set(_RUBY_DEBUG_OUTPUT TRUE)
find_package(Ruby)

This prints
-- --------FindRuby.cmake debug------------
-- _RUBY_POSSIBLE_EXECUTABLE_NAMES: ruby;ruby2.4;ruby24;ruby2.3;ruby23;ruby2.2;ruby22;ruby2.1;ruby21;ruby2.0;ruby20;ruby1.9;ruby19;ruby1.8;ruby18
-- _RUBY_POSSIBLE_LIB_NAMES: ruby;ruby-static;ruby2.6;ruby26;ruby-2.6;ruby-2.6.5
-- RUBY_ARCH_DIR: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/lib/ruby/2.6.0/x86_64-linux
-- RUBY_HDR_DIR: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/include/ruby-2.6.0
-- RUBY_POSSIBLE_LIB_DIR: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/lib
-- Found RUBY_VERSION: "2.6.5" , short: "2.6", nodot: "26"
-- _RUBY_REQUIRED_VARS: RUBY_EXECUTABLE;RUBY_INCLUDE_DIR;RUBY_LIBRARY;RUBY_CONFIG_INCLUDE_DIR
-- RUBY_EXECUTABLE: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/bin/ruby
-- RUBY_LIBRARY: RUBY_LIBRARY-NOTFOUND
-- RUBY_INCLUDE_DIR: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/include/ruby-2.6.0
-- RUBY_CONFIG_INCLUDE_DIR: /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/include/ruby-2.6.0/x86_64-linux
-- --------------------
-- Could NOT find Ruby (missing: RUBY_LIBRARY) (found version "2.6.5")

$ ls /nix/store/idjl5n00a8wr16zhwn3mrk1f3qr4bxkx-ruby-2.6.5/liblibruby_v2_6_5.so
libruby_v2_6_5.so.2.6  libruby_v2_6_5.so.2.6.5  pkgconfig  ruby

For example, FindPython works fine. The libraries there are
$ ls /nix/store/n8nviwmllwqv0fjsar8v8k8gjap1vhcw-python3-3.7.6/lib/
libpython3.7m.so  libpython3.7m.so.1.0  libpython3.so  pkgconfig  python3.7

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"x86_64-linux"`
 - host os: `Linux 5.4.22, NixOS, 20.03beta321.be346a1f4bd (Markhor)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.3.2`
 - channels(root): `"nixos-20.03beta321.be346a1f4bd, nixos-hardware"`
 - channels(jdanek): `"nixos-unstable-20.09pre218613.ae6bdcc5358"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I am looking at the following, with suspicion,
https://github.com/NixOS/nixpkgs/blob/84cf00f98031e93f389f1eb93c4a7374a33cc0a9/pkgs/development/interpreters/ruby/default.nix#L118-L120
CC maintainers @vrthra @manveru

cc: @Izorkin https://github.com/NixOS/nixpkgs/pull/73491/commits/7d0fcb8acc282577aa1bf5fb9bb91e9359f4d6b3

With this variant worked?
diff --git a/pkgs/development/interpreters/ruby/default.nix b/pkgs/development/interpreters/ruby/default.nix
index 1756c13b53d..faad826e46a 100644
--- a/pkgs/development/interpreters/ruby/default.nix
+++ b/pkgs/development/interpreters/ruby/default.nix
@@ -116,7 +116,7 @@ let
         # https://github.com/ruby/ruby/commit/97a5af62a318fcd93a4e5e4428d576c0280ddbae
         buildFlags = lib.optionals atLeast27 [ "REVISION_LATEST=0" ];

-        configureFlags = ["--enable-shared" "--enable-pthread" "--with-soname=ruby_${tag}"]
+        configureFlags = ["--enable-shared" "--enable-pthread" "--with-soname=ruby-${tag}"]
           ++ op useRailsExpress "--with-baseruby=${baseruby}/bin/ruby"
           ++ op (!docSupport) "--disable-install-doc"
           ++ ops stdenv.isDarwin [
diff --git a/pkgs/servers/sql/mariadb/default.nix b/pkgs/servers/sql/mariadb/default.nix
index ea11faf1efc..d3cc08eaadd 100644

@Izorkin I think not. Assuming that --with-soname really sets the filename of .so files, and not just the soname filed in the .so ELF object (something I did not check), then there still would be the leading v and underscores where periods are expected by CMake. That is, libruby-v2_6_5.so instead of libruby-2.6.5.so

-- Running cmake version 3.16.5
-- --------FindRuby.cmake debug------------
-- _RUBY_POSSIBLE_EXECUTABLE_NAMES: ruby;ruby2.4;ruby24;ruby2.3;ruby23;ruby2.2;ruby22;ruby2.1;ruby21;ruby2.0;ruby20;ruby1.9;ruby19;ruby1.8;ruby18
-- _RUBY_POSSIBLE_LIB_NAMES: ruby;ruby-static;ruby2.6;ruby26;ruby-2.6;ruby-2.6.6
-- RUBY_ARCH_DIR: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/lib/ruby/2.6.0/x86_64-linux
-- RUBY_HDR_DIR: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/include/ruby-2.6.0
-- RUBY_POSSIBLE_LIB_DIR: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/lib
-- Found RUBY_VERSION: "2.6.6" , short: "2.6", nodot: "26"
-- _RUBY_REQUIRED_VARS: RUBY_EXECUTABLE;RUBY_INCLUDE_DIR;RUBY_LIBRARY;RUBY_CONFIG_INCLUDE_DIR
-- RUBY_EXECUTABLE: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/bin/ruby
-- RUBY_LIBRARY: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/bin/ruby
-- RUBY_INCLUDE_DIR: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/include/ruby-2.6.0
-- RUBY_CONFIG_INCLUDE_DIR: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/include/ruby-2.6.0/x86_64-linux
-- --------------------

RUBY_LIBRARY: /nix/store/9zbif4pzi5qkdhh8sqn87w11jvw95r42-ruby-2.6.6/bin/ruby
This looks wrong. It should find the .so file there, not the interpreter binary.

with
diff --git a/pkgs/development/interpreters/ruby/default.nix b/pkgs/development/interpreters/ruby/default.nix
index 1756c13b53d..dbfea17437d 100644
--- a/pkgs/development/interpreters/ruby/default.nix
+++ b/pkgs/development/interpreters/ruby/default.nix
@@ -116,7 +116,7 @@ let
         # https://github.com/ruby/ruby/commit/97a5af62a318fcd93a4e5e4428d576c0280ddbae
         buildFlags = lib.optionals atLeast27 [ "REVISION_LATEST=0" ];

-        configureFlags = ["--enable-shared" "--enable-pthread" "--with-soname=ruby_${tag}"]
+        configureFlags = ["--enable-shared" "--enable-pthread" "--with-soname=ruby-${version}"]
           ++ op useRailsExpress "--with-baseruby=${baseruby}/bin/ruby"
           ++ op (!docSupport) "--disable-install-doc"
           ++ ops stdenv.isDarwin [

-- Running cmake version 3.16.5
-- --------FindRuby.cmake debug------------
-- _RUBY_POSSIBLE_EXECUTABLE_NAMES: ruby;ruby2.4;ruby24;ruby2.3;ruby23;ruby2.2;ruby22;ruby2.1;ruby21;ruby2.0;ruby20;ruby1.9;ruby19;ruby1.8;ruby18
-- _RUBY_POSSIBLE_LIB_NAMES: ruby;ruby-static;ruby2.6;ruby26;ruby-2.6;ruby-2.6.6
-- RUBY_ARCH_DIR: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/lib/ruby/2.6.0/x86_64-linux
-- RUBY_HDR_DIR: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/include/ruby-2.6.0
-- RUBY_POSSIBLE_LIB_DIR: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/lib
-- Found RUBY_VERSION: "2.6.6" , short: "2.6", nodot: "26"
-- _RUBY_REQUIRED_VARS: RUBY_EXECUTABLE;RUBY_INCLUDE_DIR;RUBY_LIBRARY;RUBY_CONFIG_INCLUDE_DIR
-- RUBY_EXECUTABLE: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/bin/ruby
-- RUBY_LIBRARY: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/lib/libruby-2.6.6.so
-- RUBY_INCLUDE_DIR: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/include/ruby-2.6.0
-- RUBY_CONFIG_INCLUDE_DIR: /nix/store/5nvmryah49jrzqcsx07v5dbchhzybznk-ruby-2.6.6/include/ruby-2.6.0/x86_64-linux
-- --------------------

Thank you for looking into this. Yes, now it looks correct to me.

I had a look how Ubuntu does it. They have /usr/lib/x86_64-linux-gnu/libruby-2.7.so, that is, they drop the minor number from filename. Doing that would only complicate things in nixpkgs and it would not help anything, so no need to follow them exactly. https://packages.ubuntu.com/focal/amd64/ruby2.7-dev/filelist

Fixed in #87239


Request/Response Builder with String targetURI

Please add a constructor in the WarcRequest.Builder and the WarcResponse.Builder that accepts a String instead of a URI class.
The internet is full of URI's that are difficult or impossible to parse to convert them to a URI class. Jwarc only calls targetURI.toString(), so this would be a small change.

Feel free to submit a pull request with this change. :-)

I have created a pull request at https://github.com/iipc/jwarc/pull/64


thuang-fix-code-ql-scan

Reviewers
Functional:
Readability:

Ran into this error in a different PR. So hopefully this change fixes it!
https://github.com/chanzuckerberg/corpora-data-portal/actions/runs/655456069

1 issue was detected with this workflow: git checkout HEAD^2 is no longer necessary. Please remove this step as Code Scanning recommends analyzing the merge commit for best results.

Changes

add
remove
modify

Definition of Done (from ticket)
QA steps (optional)

Awesome! Thanks so much, @Bento007 🎉 !!


Error with np.str (NumPy >= 1.24)

Hello,
In NumPy 1.20, the use of np.str was deprecated and it now creates an error in some parts of the code for versions >= 1.24. The error can easily be reproduced with the first few lines of the Quick Example in the README:
import numpy as np
from ase.build import molecule
from dscribe.descriptors import SOAP
from dscribe.descriptors import CoulombMatrix

# Define atomic structures
samples = [molecule("H2O"), molecule("NO2"), molecule("CO2")]

# Setup descriptors
cm_desc = CoulombMatrix(n_atoms_max=3, permutation="sorted_l2")
soap_desc = SOAP(species=["C", "H", "O", "N"], rcut=5, nmax=8, lmax=6, crossover=True)

Gives:
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
Cell In[4], line 3
      1 # Setup descriptors
      2 cm_desc = CoulombMatrix(n_atoms_max=3, permutation="sorted_l2")
----> 3 soap_desc = SOAP(species=["C", "H", "O", "N"], rcut=5, nmax=8, lmax=6, crossover=True)

File ~/partycls_dev/lib/python3.8/site-packages/dscribe/descriptors/soap.py:201, in SOAP.__init__(self, r_cut, n_max, l_max, sigma, rbf, weighting, crossover, average, species, periodic, sparse, dtype, rcut, nmax, lmax)
    198 super().__init__(periodic=periodic, flatten=True, sparse=sparse, dtype=dtype)
    200 # Setup the involved chemical species
--> 201 self.species = species
    203 # Test that general settings are valid
    204 if sigma <= 0:

File ~/partycls_dev/lib/python3.8/site-packages/dscribe/descriptors/soap.py:1025, in SOAP.species(self, value)
   1017 """Used to check the validity of given atomic numbers and to initialize
   1018 the C-memory layout for them.
   1019
   (...)
   1022         numbers or list of chemical symbols.
   1023 """
   1024 # The species are stored as atomic numbers for internal use.
-> 1025 self._set_species(value)
   1027 # Setup mappings between atom indices and types
   1028 self.atomic_number_to_index = {}

File ~/partycls_dev/lib/python3.8/site-packages/dscribe/descriptors/descriptor.py:118, in Descriptor._set_species(self, species)
    109 """Used to setup the species information for this descriptor. This
    110 information includes an ordered list of unique atomic numbers, a set
    111 of atomic numbers and the original variable contents.
   (...)
    115         numbers or list of chemical symbols.
    116 """
    117 # The species are stored as atomic numbers for internal use.
--> 118 atomic_numbers = get_atomic_numbers(species)
    119 self._atomic_numbers = atomic_numbers
    120 self._atomic_number_set = set(self._atomic_numbers)

File ~/partycls_dev/lib/python3.8/site-packages/dscribe/utils/species.py:68, in get_atomic_numbers(species)
     64         raise ValueError(
     65             "The given list of species contains negative integers. Please use only non-negative integers"
     66         )
     67     atomic_numbers = species
---> 68 elif all(isinstance(x, (str, np.str)) for x in species):
     69     atomic_numbers = symbols_to_numbers(species)
     70 else:

File ~/partycls_dev/lib/python3.8/site-packages/dscribe/utils/species.py:68, in <genexpr>(.0)
     64         raise ValueError(
     65             "The given list of species contains negative integers. Please use only non-negative integers"
     66         )
     67     atomic_numbers = species
---> 68 elif all(isinstance(x, (str, np.str)) for x in species):
     69     atomic_numbers = symbols_to_numbers(species)
     70 else:

File ~/partycls_dev/lib/python3.8/site-packages/numpy/__init__.py:305, in __getattr__(attr)
    300     warnings.warn(
    301         f"In the future `np.{attr}` will be defined as the "
    302         "corresponding NumPy scalar.", FutureWarning, stacklevel=2)
    304 if attr in __former_attrs__:
--> 305     raise AttributeError(__former_attrs__[attr])
    307 # Importing Tester requires importing all of UnitTest which is not a
    308 # cheap import Since it is mainly used in test suits, we lazy import it
    309 # here to save on the order of 10 ms of import time for most users
    310 #
    311 # The previous way Tester was imported also had a side effect of adding
    312 # the full `numpy.testing` namespace
    313 if attr == 'testing':

AttributeError: module 'numpy' has no attribute 'str'.
`np.str` was a deprecated alias for the builtin `str`. To avoid this error in existing code, use `str` by itself. Doing this will not modify any behavior and is safe. If you specifically wanted the numpy scalar type, use `np.str_` here.
The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at:
    https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations

It still works for versions <= 1.23, despite the deprecation in 1.20. This should be easy to fix, but I thought I should let you know.
Cheers.

Thanks @jorisparet for this. Now fixed in v2.0.0.


Add "BookZZ" as a search source

BookZZ lets you find any book you want. It has 2.5 millions of books indexed. I think we don't have right now a book search source on FrostWire. It can be very useful.
http://bookzz.org/

@gubatron


Slow symbol searching in PHP

Sublime vs VSCode:

These are my extensions:

If I open the file with the method I want sometimes VSCode will find it, other times it will seem to load indefinitely even after a successful hit, I'm using a project wide search for methods in both editors on the same code base

The following extension might provide the ux that you want https://marketplace.visualstudio.com/items?itemName=bmewburn.vscode-intelephense-client

Thank you this helps, is it possible to get a symbol match after every letter typed?
VSCode seems to wait until I stop typing for a short period before giving me results

You'll have to file this on the repo for the intellisense extension you're using.


Delete shortcut doesn't work on gridmap node

Godot version:
3.2 since commit https://github.com/godotengine/godot/pull/25406
Issue description:
The delete shortcut doesn't work anymore on gridmap nodes to delete them since it is now used to clear the cell selection. If there is no selection it should still work though.
I was trying to fix the problem myself since it seemed an easy enough task for a first pull-request but I am not too sure about what to do anymore...
Basically the problem is the InputEventKey never reach the _unhandled_key_input() method from scene_tree_dock.cpp since it is intercepted and consumed before in
menu_button.cpp
void MenuButton::_unhandled_key_input(Ref<InputEvent> p_event) {
[...]
		if (popup->activate_item_by_event(p_event, global_only))
                        // Should NOT be called if there is no selection in the gridmap
			accept_event();
	}
}

And the PopupMenu just  uses the "id_pressed" signal to tell the gridmap to clear the selection by calling _menu_option() so there is no return value to be able to NOT call accept_event()
grid_map_editor_plugin.cpp
void GridMapEditor::_menu_option(int p_option) {
[...]
case MENU_OPTION_SELECTION_CLEAR: {
	    	    if (!selection.active)
                                // HERE we should ignore the input event so it reaches the scene tree dock
				break;

So I don't really know how to proceed without changing everything... any thoughts?

If you disable the item popup->activate_item_by_event(p_event, global_only) should return false. So if there is no selection, you could do something like: options->get_popup()->set_item_disabled(options->get_popup()->get_item_index(MENU_OPTION_SELECTION_CLEAR), true); and it will disable that option. If you don't want to disable it, you could also disable the shortcut, which should also work because of this line: if (is_item_disabled(i) || items[i].shortcut_is_disabled) inside the PopupMenu::activate_item_by_event() method.

@codecustard Simply deactivating the item when there is no selection certainly did the trick and is a lot simpler than the other solutions that I had in mind. Don't know why I didn't think of that. Thanks! I will soon make a PR.
For homogeneity I will also deactivate the other selection actions (cut, fill, duplicate)


[CT-1768] [Docs] API docs glow-up

We have some MVP docs for dbt-core's nascent Python API. It's possible to generate & view these locally:
# working dir: cloned version of dbt-core repo
# within python virtualenv
$ pip install -r editable-requirements.txt -r dev-requirements.txt
$ cd core/dbt/docs
$ make clean
$ make build
$ open build/html/index.html

Before we go live with a dbt-core Python API, we should make these docs fuller & user-friendlier.
Exit criteria
We can close this ticket when we have high-quality, user-friendly documentation for programmatic invocations of dbt-core via its top-level API.
There's more work to do as we dig deeper into the high-level API, and pick which objects should be documented (= public/contracted, with caveats as needed): #5540
In addition to that work, we'll also want to make sure these docs are:

[ ] Working for nested commands. Currently, docs generate and source freshness do not appear (nor do their supported parameters).
[ ] Accessible to community users. The topline "how to invoke" instructions offer a solid start. They could use a bit more narrative, and more working examples.
[ ] Hosted for easy public reference. After some quick experimentation (https://github.com/dbt-labs/docs.getdbt.com/pull/2693), it seems most promising to generate Markdown and distribute it over to the docs repo. We'd have one Markdown file per minor version of dbt Core. We'd want to create a GHA that automates generating, opening a PR, etc.

There will be a lot of additional work involved in documenting the high-level API (internal classes, attributes, methods). https://github.com/dbt-labs/dbt-core/issues/5540 is the ticket encompassing this work right now, which I think is quite broad. Ultimately, we'll want this to include both:

internal objects used during initialization (Project, Profile, ...)
important contracts (nodes, results) that are sorta documented at schemas.getdbt.com today, and which we're in the process of proto-ifying (e.g. #6391)

Queueing this up for Execution team to discuss during BLG. I would understand if we're not ready to point a points estimate on this just yet (given other work that needs to happen first), but I would really like us to make progress here during the 30% of API-ification Phase 2 that's dedicated to UX / documentation improvements. Let's discuss more if the exit criteria listed above don't make sense, or if this should be split into multiple narrower tickets.

@jtcohen6 Let discuss the differences between our higher-level API versus dbt as a library this Friday during our design discussion and what we actually want to document.

Let's make sure to discuss this as part of today's follow-up conversation:

What's the API we want to have by v1.5 / April?
What's the API documentation we want to have by v1.5 / April?


Grammatical Error

I think the text "By default, an attribute applies to the element that it precedes." should be "By default, an attribute applies to the element that follows it."

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 23c570f1-2f76-be1b-410a-ed4bd67e33cc
Version Independent ID: dd329ad3-ca29-61e5-8753-d57792a88516
Content: Attributes (C#)
Content Source: docs/csharp/programming-guide/concepts/attributes/index.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for writing this @jason-hess
I think both wordings are correct, but your suggestion is more clear.
I've added this to our backlog to address when we next update this article.
In the meantime, if you want to submit a PR with your suggestion, I'll watch for it.


Rewrite Instruction Format Configuration Window

The current window using the drawing mechanism; but its broken.  Rewrite this function to follow the traditional mechanisms.  Related to Issue #125

In progress; Need to add support to display instruction format image.

fixed in 5fc27cf

passed all devel tests


Quest-reinit time indicator

I aborted my 3 quests yesterday evening. They are "failed". It seems that I have to wait for 50'000 seconds. It could be better (more professional) to say 13.9 hours, and if less than an hour, say it in minuts, because 50000 seconds doesn't really make sense...
Another thing: it doesn't refresh, so you could think that the timer is stopped when you see how much time is left. It should refresh regularly

It's a temporary value and it's easily tweakable, just wait the quest mod first complete stable version !
Just wait or follow the quest mod issue https://github.com/MinetestForFun/minetest-minetestforfun-server/issues/145

@Ombridride The quests fork is supposed to be stable. Stable doesn't mean complete. The #145 todo list is now complete about the things I wanted to work on, and you'll notice I nowhere made mention of this missing feature. The remaining tasks in it are related to mff_quests, not quests core. Letting this issue open would be a good idea.

You can comment without Reopened the issue Gravgun, i think this issue can be closed because you already planned this feature...

@Ombridride I did think about it, but did not plan it.


ESP32/8266 - MQTT - http

Is there a reason, why I can't use the http client with an ESP?
I don't nee the additional functionality. And I'm restricted to http with our installation.

@imbeacon I think this issue can be closed, ThingsBoardHttp can be used with the ESP32 and ESP8266 as well for a while now.


Security Vulnerability: Join and relpath path traversal vulnerability

When using os.join or os.repath, path traversal is possible. Path traversal allows an attacker inappropriate access to a file.
Issue Example:
def function():
    name = request.args.get('name')
    return send_file(os.path.relpath(os.getcwd(), name))

Issue Workaround:
Do not to rely upon user input when dealing with file system access. If such a scenario is unavoidable, make sure the file is in an expected directory and the application throws an error if it is not. Alternatively, check the file name being requested by using a map and an ID in the request.
def verify_path(basedir, path):
    return os.path.realpath(path).startswith(basedir)

def function():
	name = request.args.get('name')
	if (verify_path(os.getcwd(),name):
		return send_file(os.path.relpath(os.getcwd(), name))
	else:
		print("Throw an error")

Closing, not a vulnerability. Your bug report is simply not applicable. node-gyp is a build tool, there is no concept of untrusted input.


Create Navigation Bar

Make sure the first line is formatted as follows.
As a developer, I will want to create a navigation bar that allows the user to select products by category.
Include link(s) to relevant tutorial(s)
Please include at least one link to a relevant tutorial for the issue to give developers a starting point.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Navigation Drawer:

Toolbar with Tabs on top:

A drop down menu of categories when on the products page to search by categories you are not currently viewing.
Is this issue related to any other issues?
Issue for User Story 74: Navigation
Additional context
Add any other context or screenshots about the feature request here.
Story Points
Please estimate amount of story points. Each story point is defined as 30 minutes.

Closed with PR 114

@Annelisebx I reverted the branch within minutes of you saying that you did not want it merged. Please let me know what you want.


[Multiple Tabs]: When using send to text2img, the prompts will be sent to all tabs that are opened.

Hi, thanks for the amazing image browser. This is definitely one of the best if not THE best out there!
Good work!
I just noticed for a while, that when using send to text2img, (same goes for send to img2img, inpaint, extra, etc), the prompts and all settings will be send to ALL the stable diffusion tabs that are currently opened. It would be great if this function can be applied with the restriction of affecting only the active tab that are using the infinite image browser.
Otherwise, thanks for the amazing work! Xiexie!

In the latest version, if you are not using IIB in a separate browser tab, then IIB will only send the file to the page it is on.

Many thanks! I've updated it to the latest version and it works like a charm now! 10/10 tool!


Update PaginatorHelper.php

Codecov Report

Merging #112 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff            @@
##             master    #112   +/-   ##
========================================
  Coverage      81.4%   81.4%
  Complexity      333     333
========================================
  Files            17      17
  Lines          1038    1038
========================================
  Hits            845     845
  Misses          193     193

Impacted Files
Coverage Δ
Complexity Δ

src/View/Helper/PaginatorHelper.php
5.33% <100%> (ø)
25 <2> (ø)
:x:

Continue to review full report at Codecov.

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


Update PostgresStorageAdapter.js

simplifying one call to the database
adding a note for something that looks like a bug: the code uses .map, but inside the callback nothing is returned.

Not sure about the comment you added as there is an object returned inside the loop.

I read the code wrong, sorry.


Fix regression: Wait until release becomes 1.

Host sets release to 1 to signal waiting threads to continue to completion.
https://github.com/Microsoft/openenclave/blob/master/tests/ecall_ocall/host/host.cpp#L182-L184
Therefore the enclave threads need to wait for release to become 1.

Fixes #1507

@anakrish - Why not revert to the previous builtin function call -
__atomic_add_fetch(args.counter, 1, __ATOMIC_SEQ_CST);
ATOMIC_SEQ_CST Enforces total ordering with all other __ATOMIC_SEQ_CST operations.
We know that this test worked reliably on Linux before the recent change to use EDL.

@anitagov It is preferably to rely on reading the code/reason about multi-thread behavior rather than relying whether tests passed previously or not. If there is something wrong with the current approach we need to be able to clearly describe what is wrong. Otherwise, how will a user of our SDK trust us if we cannot even reason about a simple test?
Do you have any specific concerns about issues with the current implementation?

bors try

bors r+


Don't pass get_timezone_aware_datetime() to Query.filter()

reverts one hunk from f04b3737
The comparison is done in the database so the function is not needed.
Fixes #1752

recheck

Zuul is sleeping, tests pass locally, merging.


Power toys settings UI won't load

Environment
Windows build number: Internal
PowerToys version: 0.18.1-x64
PowerToy module for which you are reporting the bug (if applicable): C:\WINDOWS\SYSTEM32\twinapi.appcore.dll

Steps to reproduce
Installed the latest version 'PowerToysSetup-0.18.1-x64' and it partially works.
I can see the launcher and shortcuts but cannot use fancy zones. The setting UI errors when i launch settings from the taskbar.
Expected behavior
Expecting setting UI.
Actual behavior
Faulting application name: Microsoft.PowerToys.Settings.UI.Runner.exe, version: <IP_ADDRESS>, time stamp: 0x5e222541
Faulting module name: twinapi.appcore.dll, version: 10.0.18980.1, time stamp: 0xfa2d8987
Exception code: 0xc0000005
Fault offset: 0x00000000000433d7
Faulting process id: 0x4378
Faulting application start time: 0x01d6391a2923eb62
Faulting application path: C:\Program Files\PowerToys\SettingsUIRunner\Microsoft.PowerToys.Settings.UI.Runner.exe
Faulting module path: C:\WINDOWS\SYSTEM32\twinapi.appcore.dll
Report Id: ec839bce-aae4-49cf-9bb5-346421c70f8d
Faulting package full name:
Faulting package-relative application ID:
Screenshots
App UI crashes before i can grab a screenshot.

We're debugging the issue with him

@madansr7 we did a bunch of adjustments, can you see if 0.19 fixed this?

@crutkas, I installed the latest version 0.19 and still have the same issue. :-(

Does it pop up if you type
WinKey+`

Yes, it does and i can define zones but they don't seem to work.

hi @madansr7, is this still reproducible in 0.19.2?

@saahmedm , still an issue with 0.19.2.

@crutkas , @saahmedm , @laviusmotileng-ms, this issue was resolved with the latest build v0.20.1
i am going to close this issue.

I have this problem too with 0.27.1. Settings UI opens and closes only on second try (means 2 times click on tray->settings to open and two times on ui "Close" X button to close). .NET Core is installed. I am trying to install them on my office laptop with win10 1803 (OS build 17134.1792).
Are there any operations within PowerToys that could be limited by the network group policies in my company?
Windows Event Viewer Entry:
Faulting application name: PowerToys.exe, version: <IP_ADDRESS>, time stamp: 0x5fcfc216
Faulting module name: Microsoft.Launcher.dll, version: <IP_ADDRESS>, time stamp: 0x5fcfbcee
Exception code: 0xc0000005
Fault offset: 0x0000000000004623
Faulting process id: 0x4ef0
Faulting application start time: 0x01d6d44e5b9b7796
Faulting application path: ...\PowerToys\PowerToys.exe
Faulting module path: ...\PowerToys\modules\Launcher\Microsoft.Launcher.dll
Report Id: 904bb105-f427-4d4b-af92-24a95426bee1
Faulting package full name:
Faulting package-relative application ID:


Create one-pager design doc for Media Gallery state

Task Description
Create initial draft of one-pager design doc in the AMP folder and send it to the team for review.

Done:
https://docs.google.com/document/d/1S9jTh6Lz3UEhj9Wkb7S7x_kJGvRVpEaNOqk2aCyb4y8


"Mărinești" is listed in the drop-down menu

Description:
"Mărinești" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Mărinești" in the search bar.

Expected results
The location is listed in the drop-down menu as "Mărinești, Alba (Întregalde)".


Bug Report - forced to change password weekly

For the past ~4 months I have had to reset my password every 3-7 days. This is frustrating. I don't think this is expected behavior (checked Issues, handbook, and in-app documentation says change pw every "180 days"), hence a bug report. Steps to reproduce:

Every 3-7 days I try to log in with the password that has been working
I get a "Authentication failed. Additional Information: Account inactive or misconfigured." report.
I use "Forgot password" to get a new temporary password via email.
I can log in
I change the password to another password
I continue work, and am able to log in as usual for 3-7 days.

Browser: Firefox on Linux

also updated https://handbook.arctosdb.org/documentation/users.html
@camwebb please make sure you have an active email in your agent record.


doc: Document how to restrict OLM installs to a single namespace

What type of PR is this?
/kind documentation

What this PR does / why we need it:
Our installation and usage document tells you how to restrict the
operator to a single namespace when installing from the upstream
deployment manifests, but not when installing from OLM. Let's fix that.
Which issue(s) this PR fixes:
None

Does this PR have test?
N/A

Special notes for your reviewer:
N/A
Does this PR introduce a user-facing change?

NONE

Codecov Report

Merging #1211 (a917e77) into main (7cfe230) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1211   +/-   ##
=======================================
  Coverage   45.42%   45.42%
=======================================
  Files          48       48
  Lines        5360     5360
=======================================
  Hits         2435     2435
  Misses       2805     2805
  Partials      120      120


Expose configuration and location service

This PR depends on #119 to be merged in first.

Coverage decreased (-0.8%) to 40.288% when pulling bb58fc196bdf04835548a26463c95ab29c5654ab on suda/ch55034/expose-configuration-and-location-service into ac9909fb6e9756103c6fb67f38ac4d915a139b97 on master.


add complex.mod

Does complex.__mod__ exist in CPython?

Removed in 3.9
my bad, I am sorry @HyunP-dev


return to original pdfmake (fixes #2769)

sets the source of pdfmake to the following:
https://github.com/bpampuch/pdfmake/commit/3b44129d599424cd0e306fc5014ba3a129220c64

To test it, make sure to remove the old pdfmake
rm -R bower_components/pdfmake*

run again
npm install

make sure everything looks fine

and clear your browser cache

I tested some stuff rather briefly and everything seemed to work just fine.

forgot to mention:
@emanuelschuetze
@normanjaeckel


Add HTML support for title and message

Changed: use the v-html prop instead of the {{dialog.title}}, allowing the use of HTML tags.
There is an issue, marked as an enhancement - /issues/15 - that I think that this PR solves.
Note: Also bumped the version to 1.1.0 instead of 1.0.3 since I think this is a feature and not a bug fix.

Please merge :)

Second vote to have this merged, please.


beta4: What is the correct upgrade procedure?

$ lxc --version
2.0.0.beta3
$ apt-get update && apt-get install lxd
Ign http://au.archive.ubuntu.com trusty InRelease
Get:1 http://au.archive.ubuntu.com trusty-updates InRelease [65.9 kB]
Get:2 http://ppa.launchpad.net trusty InRelease [20.9 kB]
...
Ign http://au.archive.ubuntu.com trusty/restricted Translation-en_US
Ign http://au.archive.ubuntu.com trusty/universe Translation-en_US
Fetched 5,053 kB in 38s (133 kB/s)
Reading package lists... Done
Reading package lists... Done
Building dependency tree
Reading state information... Done
Suggested packages:
  criu lxd-tools
The following packages will be upgraded:
  lxd
1 upgraded, 0 newly installed, 0 to remove and 8 not upgraded.
Need to get 3,994 kB of archives.
After this operation, 28.7 kB of additional disk space will be used.
Get:1 http://ppa.launchpad.net/ubuntu-lxc/lxd-stable/ubuntu/ trusty/main lxd amd64 2.0.0~beta4-0ubuntu2~ubuntu14.04.1~ppa1 [3,994 kB]
Fetched 3,994 kB in 32s (123 kB/s)
Preconfiguring packages ...
(Reading database ... 58202 files and directories currently installed.)
Preparing to unpack .../lxd_2.0.0~beta4-0ubuntu2~ubuntu14.04.1~ppa1_amd64.deb ...
lxd stop/waiting
Unpacking lxd (2.0.0~beta4-0ubuntu2~ubuntu14.04.1~ppa1) over (2.0.0~beta3-0ubuntu4~ubuntu14.04.1~ppa1) ...
Processing triggers for ureadahead (0.100.0-16) ...
Processing triggers for man-db (<IP_ADDRESS>-1ubuntu1) ...
Setting up lxd (2.0.0~beta4-0ubuntu2~ubuntu14.04.1~ppa1) ...
lxd start/running, process 17290

$ lxc --version
2.0.0.beta3

$ lxc list
error: json: cannot unmarshal string into Go value of type int

This happened separately on two ubuntu machines (14.04, 15.10). I'm guessing I just didn't upgrade properly. What is the the correct procedure? I feel like it needs an internal DB migration...

I've just hit the same bug.

Have you got same version between lxc-client and lxd ?
lxc --version
lxd --version
You must have same version.
apt-get install lxd lxd-client

yups..!  it works on me. thanks @aarnaud
I just confirm lxd and lxc  version is not match with latest lxc update. Stop lxd daemon service and then reinstall it again.
user@lxd-pool-0:~$ lxc version
2.0.0.beta4
user@lxd-pool-0:~$ lxd --version
2.0.0.beta3

user@lxd-pool-0:~$ sudo service lxd stop
user@lxd-pool-0:~$ sudo service lxc stop

user@lxd-pool-0:~$ sudo apt-get install lxd

This issue can be closed, i thinks

Thanks @aarnaud that was it. Opened PR #1649 to document this.


Can't find processing-java

I followed instructions, tried Tools>Build, but ST2 console says it can't find file or directory.
See the error here: https://www.evernote.com/l/AZQ1T9k27VhAu4wpjSai6YzxJu_Dx8h53Iw
Notice that I do have processing-java installed as requested.
I am just alerting, because other users might find the same error. But I solved my problem.
Solved by going to ~Library/Application Support/Sublime Text 2/Packages/Processing/Build Systems and altering the builder with the complete path of my processing-java install.

:) Thanks for the hints!

welcome!
Em qui, 3 de set de 2015 às 03:16, Benedikt Groß<EMAIL_ADDRESS>escreveu:

:) Thanks for the hints!
—
Reply to this email directly or view it on GitHub
https://github.com/b-g/processing-sublime/issues/81#issuecomment-137346750
.

--
Fred Guth

@dvschultz @levitabris I have looked into this more and am now able to replicate the issue. The problem is how Sublime does not use your user shell's PATH configuration on OSX. To fix this, use Package Manager to install the "Fix Mac Path" package.
This occurs with Processing 3 and not Processing 2, because Processing 3 installs processing-java in /usr/local/bin whereas Processing 2 installed it in usr/bin.

Thanks!