Update Dockerfile and requirements.txt

Modify Dockerfile and requirements.txt to include more dependency installation in order to support running pipeline jobs remotely.

[Current coverage][cc-pull] is 67.90%

Merging [#16][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #16   diff @@
==========================================
  Files            24         24
  Lines           729        729
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            495        495
  Misses          234        234
  Partials          0          0

Powered by Codecov. Last updated by [ee4a7bd...7fc764a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/googleapis/artman/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/googleapis/artman/compare/ee4a7bd110f440f2d87f2d5c02872969e09589b2...7fc764aba6deed59e84aefb2c37b15c0ee2c3c63
[cc-pull]: https://codecov.io/gh/googleapis/artman/pull/16?src=pr

cool, I think it's good to go -- LGTM.


Newsletters: add 224 (2022-11-02)

[x] @bitschmidty
[x] Bitcoin Core 23927 @Xekyo
[x] Bitcoin Core 25957 @glozow
[ ] Other stuff @harding

I guess you may wish to cover https://twitter.com/brqgoo/status/1587397646125260802?t=ihdFNLORbmxCa-6NyFUFMQ&s=19 as a crucial and exploited bug in this weeks newsletter?

I guess you may wish to cover https://twitter.com/brqgoo/status/1587397646125260802?t=ihdFNLORbmxCa-6NyFUFMQ&s=19 as a crucial and exploited bug in this weeks newsletter? (lnd nodes seem to be affected and their funds may be at risk because they don't sync with the chain anymore)

I imagine this would certainly be covered in next week's newsletter. Not to downplay the importance, but this is happening <24hr before publishing is scheduled and the newsletter usually covers things from the week before.

I know that it was short notice but I decided to drop/suggest it here as I recall @harding once mentionig that for critical security upgrades the newsletter can have last minute hotfixes and updating lnd/btcd seems critical otherwise peers could steel funds.

Ah wasn't aware, ignore me heh

Ah wasn't aware, ignore me heh

But, just to be clear, we should not ignore you when you say "ignore me" 😸

FYI, our publication policy is here: https://gist.github.com/harding/0dffdde00761262ca094219c33b8ff22  (yeah, I should probably move that into the repo at some point).  We'll add important security announcements at any time, e.g. we added this less than an hour before publication.
In cases like the previous LND vulnerability where I also didn't learn about it until the day before publication, we announce the availability of a security critical fix immediately and then go into details in the subsequent week's newsletter.  I'm going to do the same for this week.
Thanks @renepickhardt for mentioning and everyone else for discussion!

Edits made, thanks everyone!  Also added lede, updated releases/RCs with LND info, and added topic links.  Cheers!

ACK https://github.com/bitcoinops/bitcoinops.github.io/pull/909/commits/8a50ac2b25d5714101a8539a6118d0b4a3533cfb sans a couple topic description fixups

Another great newsletter thanks @harding @glozow @Xekyo for authoring and for @adamjonas @LarryRuane @glozow for reviewing! 🚀


Database Feature

Now App is just Fetching Data from Yahoo REST API. We need make fetched stocks offline

added in #13


Feedback: Provide guidance on support for GCCH and DoD

Hey Team!  It says here that Power Platform CLI is not approved for GCCH and DoD yet when you look at the CLI command it appears it should work.

CLI: https://learn.microsoft.com/en-us/power-platform/developer/cli/reference/auth#--cloud--ci

I think it will help provide guidance to our customer to clarify the support.  Thanks!

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

ID: 5446bda4-9257-9a56-ff6b-b62a82b79132
Version Independent ID: 5446bda4-9257-9a56-ff6b-b62a82b79132
Content: Microsoft Power Platform CLI - Power Platform
Content Source: power-platform/developer/cli/introduction.md
Service: power-platform
Sub-service: developer
GitHub Login: @snizar007
Microsoft Alias: snizar

@snizar007
Unless there are some regions we do not support, I think it may be clearer to remove the whole paragraph?
Or just change it to say that all regions are supported.
It seems to be an artifact of a time when only partial support was offered? If all regions are supported, then there shouldn't be a need to call out a subset.

Hello @brmcmill -
Please see REMARKS for clarity: https://learn.microsoft.com/en-us/power-platform/developer/cli/reference/auth#remarks
Thank you.


Support for DDNS in Kea

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[X] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[X] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Is your feature request related to a problem? Please describe.
I am currently using ISC DHCPD which has support in OPNsense for dynamically updating my DNS servers with assigned IP addresses.
Describe the solution you like
I would like for support for kea-dhcp-ddns to be added for the purposes of feature parity with the existing ISC DHCPD.
Looking at the documentation relating to this part, the required configuration interface is

Able to enter one or more TSIG keys, identified by TSIG key name and containing algorithm and secret.
Able to enter one or more forward domain names, along with specifying the DNS servers to use and the TSIG key name.
The exact same as (2), but for reverse domain names.

Describe alternatives you considered
The only alternative for now is to stay with ISC DHCPD. Or to move to a separately hosted DHCP server.
Additional context
See relevant Kea documentation: https://kea.readthedocs.io/en/latest/arm/ddns.html

I'm not sure it's necessary but this comment only to say I would like the same very much, and I have read comments and posts online of many others that would like too

I assumed, clearly falsely, that the kea server would've already supported this as much as migration is being encouraged. Please prioritize this feature as I now have to un-migrate.


JIRA webhook view has no authentication

Issue Type

[x] Bug
[ ] Enhancement

Description
The JIRA webhook view, i.e. the view to which the JIRA server will post updates, does not have any authentication so that anybody can call this webhook and manipulate data in the DefectDojo database.
See:  https://github.com/DefectDojo/django-DefectDojo/blob/master/dojo/jira_link/views.py#L26

Thanks for bringing that to our attention. Originally, when that was created the intent was to whitelist that endpoint with an Nginx proxy. While not ideal, at the time that's all that Jira supported. Two checks have been added:

If Jira is not enabled the call to the webhook will return denied
A system setting option has been introduce, enable_jira_web_hook which is disabled by default along with a suggestion to whitelist.

Since then it looks like there are some other options in later versions such as supporting JWT and possibly an add on to add an API key. I'll add to the documentation and continue reviewing options that Jira might support.


Issue getting ControlPlane to move to proper context

I find that Control Plane is not locking on to the right context. I default to a "Public" context. I have a rule that says if core location services say you are "here" use "Bennett". I have another rule that says if you are on this BSSID, use "Bennett". Core location indeed has my location correct. I am in fact on the designated WiFi BSSID. Control Plane puts me in Public, instead of Bennett. If I switch it manually to Bennett, it pops back to Public. (If I "stick" it on Bennett I lose the whole point of the app.) This is the case no matter where the confidence sliders are set.

This issue was reported by a user


Various editorial updates

Main changes:

Renaming the base points of G1/G2
Removed the Definitions (we are explicitly defining everything in the operation)
Updated the ProofGen/ProofVerify explanatory section

+ couple of terminology and operation description updates

Discussed on the WG call on the 10 of July. Will wait for reviews before resolving

Discussed on the WG call of the 10th of July. Multiple approvals. Editorial updates. Merging


use AWS Region from the set secrets

If we currently set the AWS_REGION Okteto Secret to anything other than us-west-2, the app will not function properly. This is because the Queue will always be created in the us-west-2 region, while the command to create the secret – which the application code relies on – uses the value of the region from the AWS_REGION Okteto Secret.
  - name: Create the AWS secret
    command: |
      kubectl create secret generic aws-credentials --save-config --dry-run=client --from-literal=AWS_REGION=$AWS_REGION --from-literal=AWS_DEFAULT_REGION=$AWS_REGION --from-literal=AWS_SECRET_ACCESS_KEY=$AWS_SECRET_ACCESS_KEY --from-literal=AWS_ACCESS_KEY_ID=$AWS_ACCESS_KEY_ID -o yaml | kubectl apply -f -

By making this change, the Queue will be created in the correct region as specified by the AWS_REGION Okteto Secret.

@rberrelleza if you check the Preview Environment you will see the same error message for the check service that I was talking about on Slack.


BagInfo make SourceOrganisation optional

What is this PR trying to achieve?
Make the SourceOrganisation entry in bag-info.txt optional

Closing in favour of https://github.com/wellcometrust/storage-service/pull/22


Some questions about BasicVSR++

Hi，I have some doubts when reading the BasicVSR++ paper.
For the formula in the paper ： o(i→i−p)= s(i→i−p)+ Co(c(gi,f¯i−1,f¯i−2))
1、The feature channel of gi、f¯i−1 and f¯i−2 is 64，why the number of input channels of the offset estimation module and mask estimation module is 196？
2、Why the number of output channels of the offset estimation module is 288？ It needs to be added to s, shouldn’t it be the same as the number of channels in s(i→i−p)? Or the number of channels of s(i→i−p) is also 288？
Looking forward to your reply, thank you！

Sorry to ask another question under closed issue.

Sorry to ask another question under closed issue.
In second-order propagation with deform_groups=16, the predicted offsets have 2 x 9 x 16 = 288 channels, the predicted offsets are chunked into offset_1 and offset_2 each of which has 144 (2 x 9 x 8) channels.
Does it mean deform_groups // 2 groups of offsets for time step t-1 and another deform_groups // 2 groups of offsets for time step t-2?  I think in such case, deform_groups in second-order propagation is different from it in vanilla DCN (deform_groups groups of offsets for single time step).
Thanks in advance.


Fix broken link

Looks like the account link is no longer valid, this one seems to work

Thanks for the fix :)


fix(docs): add warnings regarding no_prefix strategy

Added warning to the custom path and ignore routes section of the routing documentation
I've been spending a bit of time why my page was not found until I noticed that the custom path feature was in conflict with the no_prefix strategy, but nothing of that was mentioned in the corresponding section.

Thanks.


Disk labels may not be described correctly

I believe the disks are named differently in the file system than how they are described in the documentation above. Currently, this VM, a Data Science Virtual Machine (DSVM), is configured with a 50GB OS disk and 100GB data disk. Here's what I see when I run the linux command: df -h
hws@VM1:/data/home/hws/src/DLW-V1$ df -h
Filesystem      Size  Used Avail Use% Mounted on
udev            3.9G     0  3.9G   0% /dev
tmpfs           796M  9.0M  787M   2% /run
/dev/sda1        49G   30G   19G  61% /
tmpfs           3.9G     0  3.9G   0% /dev/shm
tmpfs           5.0M     0  5.0M   0% /run/lock
tmpfs           3.9G     0  3.9G   0% /sys/fs/cgroup
/dev/sdc1        99G   23G   71G  25% /data
/dev/sdb1        16G   44M   15G   1% /mnt
tmpfs           796M     0  796M   0% /run/user/1000
It appears that the data disk is mapped to /data and the OS disk is mapped to /. Is this so? It would be helpful if the disk to file system mapping was explicitly shown on my Azure portal. I'm very confused but maybe I'm missing something. Also, it's also unclear to me how an additional disk would be named in the file system if I attached it to this VM. Would it be the original name of the disk?

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

ID: b61b1427-1941-9831-e823-e2d114decb4a
Version Independent ID: 1ad9a4b7-0190-5f31-835c-fcf020cb39ad
Content: Azure Disk Storage overview for Linux VMs - Azure Linux Virtual Machines
Content Source: articles/virtual-machines/linux/managed-disks-overview.md
Service: virtual-machines-linux
Sub-service: disks
GitHub Login: @roygara
Microsoft Alias: rogarana

@henry-schneiderman Thank you for your feedback! We will review and provide an update as appropriate.

@henry-schneiderman This document describes the process of adding a data disk to a linux VM.
Lets go through your questions

It appears that the data disk is mapped to /data and the OS disk is mapped to /.

By default os disk is mapped to /.  data disk mapping can be any folder. I think in you case the custom script extension of any other script would have mounted the data disk to /data

It would be helpful if the disk to file system mapping was explicitly shown on my Azure portal. I'm very confused but maybe I'm missing something.

Currently  the mapping is not shown in portal.  portal shows the lun number.  You can use the below command to list the disks with lun numbers
root@desk:/home/jakaruna# tree  /dev/disk/azure/
/dev/disk/azure/
├── resource -> ../../sdb
├── resource-part1 -> ../../sdb1
├── root -> ../../sda
├── root-part1 -> ../../sda1
├── root-part14 -> ../../sda14
├── root-part15 -> ../../sda15
└── scsi1
    ├── lun0 -> ../../../sdc
    ├── lun0-part1 -> ../../../sdc1
    └── lun1 -> ../../../sdd

i have a disk with lum num 0 (sdc) and another one with lun number 1(sdd). Both of them are data disks.  Command taken from here https://www.singhkays.com/blog/how-to-identify-azure-linux-vm-disks/

Also, it's also unclear to me how an additional disk would be named in the file system if I attached it to this VM. Would it be the original name of the disk?

It wont be the original name of the disk from the portal.  First data disk will be sdc, second will be sdd , and goes on like this sd*

@henry-schneiderman is there any update

@henry-schneiderman
I will close this out for now. If you need additional help please open a new issue


Implements mkdir for Windows

Closes #755.

@VocalFan can you test this?

Ah, you reached the same point I did, sadly.
@ultimaweapon
++++++++++++++++++ E [00:00:00:00:250]:0x0000000000004eb0: kernel\src/main.rs:670
Error while running kernel: filesystem initialization failed -> cannot lookup /dev -> couldn't open the specified file -> Access is denied. (os error 5).

It's mainly due to the folder already existing.
If I delete the empty dev folder, it creates the folder and continues to mnt... Which since mnt already exists:
++++++++++++++++++ E [00:00:00:00:211]:0x0000000000004f90: kernel\src/main.rs:670
Error while running kernel: couldn't mount /mnt -> fspath is not found -> cannot lookup 'mnt' from component #0 -> couldn't open the specified file -> Access is denied. (os error 5).

This is bad. Why Windows report access is denied instead of something like file already exists.

I think I found a solution now.

I think I found a solution now.

...Got 8 minutes then I gotta go :P

@ultimaweapon Before I test this when I go home... I basically had an identical implementation, however, the error code number for when I got access denied was (-1073741790)... Don't know if that helps

++++++++++++++++++ E [00:00:00:00:366]:0x0000000000003124: kernel\src/main.rs:670
Error while running kernel: filesystem initialization failed -> cannot lookup /dev -> couldn't open the specified file -> Access is denied. (os error 5).

>Err_6.4.5.exe -1073741771
# for decimal -1073741771 / hex 0xc0000035

  STATUS_OBJECT_NAME_COLLISION                                   ntstatus.h
# Object Name already exists.
# as an HRESULT: Severity: FAILURE (1), FACILITY_NULL (0x0), Code 0x35

Okay, so I confirmed that STATUS_OBJECT_NAME_COLLISION is the actual error code, but... That doesn't work with your ERROR_FILE_EXISTS unwrapping, it just errors anyways with access denied.

Wait, the previous version already correct. The error you got came from another issue so I will revert my last commit.


Polish language file invalid

Looks like something happened to pl-PL.json, because it isn't a valid JSON file anymore.
(Reported here: http://forum.scssoft.com/viewtopic.php?f=34&t=178742&start=170#p653061 )

Whoops..... looks like I failed at merging something https://github.com/mike-koch/ets2-mobile-route-advisor/blob/i18n/pl-PL.json#L25-L30

Fixed.


Fix delayed sampling TP>1

Fix delayed sampling when TP > 1.
Rank 0 generates the next input id, and broadcast to other ranks.
BEFORE SUBMITTING, PLEASE READ THE CHECKLIST BELOW AND FILL IN THE DESCRIPTION ABOVE

 PR Checklist (Click to Expand)
Thank you for your contribution to vLLM! Before submitting the pull request, please ensure the PR meets the following criteria. This helps vLLM maintain the code quality and improve the efficiency of the review process.
PR Title and Classification
Only specific types of PRs will be reviewed. The PR title is prefixed appropriately to indicate the type of change. Please use one of the following:

[Bugfix] for bug fixes.
[CI/Build] for build or continuous integration improvements.
[Doc] for documentation fixes and improvements.
[Model] for adding a new model or improving an existing model. Model name should appear in the title.
[Frontend] For changes on the vLLM frontend (e.g., OpenAI API server, LLM class, etc.)
[Kernel] for changes affecting CUDA kernels or other compute kernels.
[Core] for changes in the core vLLM logic (e.g., LLMEngine, AsyncLLMEngine, Scheduler, etc.)
[Hardware][Vendor] for hardware-specific changes. Vendor name should appear in the prefix (e.g., [Hardware][AMD]).
[Misc] for PRs that do not fit the above categories. Please use this sparingly.

Note: If the PR spans more than one category, please include all relevant prefixes.
Code Quality
The PR need to meet the following code quality standards:

We adhere to Google Python style guide and Google C++ style guide.
Pass all linter checks. Please use format.sh to format your code.
The code need to be well-documented to ensure future contributors can easily understand the code.
Include sufficient tests to ensure the project to stay correct and robust. This includes both unit tests and integration tests.
Please add documentation to docs/source/ if the PR modifies the user-facing behaviors of vLLM. It helps vLLM user understand and utilize the new features or changes.

Notes for Large Changes
Please keep the changes as concise as possible. For major architectural changes (>500 LOC excluding kernel/data/config/test), we would expect a GitHub issue (RFC) discussing the technical design and justification. Otherwise, we will tag it with rfc-required and might not go through the PR.
What to Expect for the Reviews
The goal of the vLLM team is to be a transparent reviewing machine. We would like to make the review process transparent and efficient and make sure no contributor feel confused or frustrated. However, the vLLM team is small, so we need to prioritize some PRs over others. Here is what you can expect from the review process:

 After the PR is submitted, the PR will be assigned to a reviewer. Every reviewer will pick up the PRs based on their expertise and availability.
 After the PR is assigned, the reviewer will provide status update every 2-3 days. If the PR is not reviewed within 7 days, please feel free to ping the reviewer or the vLLM team.
 After the review, the reviewer will put an  action-required label on the PR if there are changes required. The contributor should address the comments and ping the reviewer to re-review the PR.
 Please respond to all comments within a reasonable time frame. If a comment isn't clear or you disagree with a suggestion, feel free to ask for clarification or discuss the suggestion.

Thank You
 Finally, thank you for taking the time to read these guidelines and for your interest in contributing to vLLM. Your contributions make vLLM a great tool for everyone!

@libinta did you verify the accuracy with this change? From my perspective it doesn't work -- the hidden_states tensor after first decode is filled with nan's.

After further debug and sync with @kzawora-intel, I found out that the Nan's issue is not related to your change. BlockManagerV2 allocates blocks starting from id 0, which causes undefined behaviour in reshape_and_cache. When I modified blocks allocator to start from id 1, the issue was not observed and your change looks fine.


Make sure configs are upcased.

Does this imply that release tags need to be ALL CAPS when used in the PURL fetcher API?

No. The assumption is that data is dirty and we can't control it.

checked and updated all existing targets in sw-indexing-dev, -stage and -prod, and revs-indexing-dev, -stage and -prod


Remove broken links from README

Remove the Travis Badge as that pipeline doesn't exist anymore, replace it with the Github Actions CI workflow badge
Remove the Go Walker link, Go Walker doesn't seem to work anymore

Sorry for the delay. Apparently I've overlooked the syntax and my suggestion broke the image link.
I've removed the commit (using force-push), now everything should be fine.

Thanks for your contribution!

Thanks for your contribution!


Add doc about collector account minimum permssions

This PR adds documentation about the collector account's minimum required permissions.
Resolves: ECOPROJECT-2163
Signed-off-by: Cosmin Tupangiu<EMAIL_ADDRESS>

@tupyy hi, this is a dev doc only, it will not be copied to https://docs.redhat.com/en/documentation/migration_toolkit_for_virtualization , if you want this documentation to be visible to users that are not developers in this repo, you can do a similar PR to https://github.com/kubev2v/forklift-documentation
cc:// @RichardHoch

Oooo, sory, this is a wrong repo ... @RichardHoch please ignore last comment, this is not the Forklift repo


Message gating and natural replication over the broadcast

Message gating for the destination – aka routes known messages.

Natural replication or broadcast needed instead of the ordinary round-robin.

This is perfectly done.


SwagUpdate: Throw a Exception, if curl is missing

We need a Exception to display a error on extjs =)

I am not sure if we should merge this.
The curl extension is mentioned and checked multiple times as required dependency.

https://github.com/shopware/shopware#shopware-server-requirements
https://github.com/shopware/shopware/blob/v5.1.5/engine/Shopware/Components/Check/Data/System.xml#L38
https://github.com/shopware/shopware/blob/5.1/composer.json#L19

We usually don't verify required dependencies during runtime.
I'll schedule this ticket anyway for further discussion.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/#/issues/SW-14780.
Please use this issue to track the state of your pull request.

It's checked as requirement, but shopware is installable without curl, because the error flag in the system.XML is missing (diff with zip extension).
I guess we should amend the error flag in system.xml to solve this.

This exception here is useless because you'll receive an error message regarding the batch size. https://github.com/shyim/shopware/blob/ffb8a0fe125eb9b46cf00806f9144f04e2809700/engine/Shopware/Plugins/Default/Backend/SwagUpdate/Views/backend/swag_update/controller/progress.js#L143-146

Done

Merged with 049f1f206a71a518c72d1932dfd2cab3a8bd4595.


Fixed Improper tags for Meshery Docker Extension #6396

Replica of https://github.com/meshery/meshery-consul/pull/327

Thanks for this @aditi5050!


0.6.0 release

[x] Added branding logo to top of readme
[ ] Added chroma embedding demo
[x] PGvector embedding fixes (#45)
[x] Fixes to model introspection

[x] Move OAI response structure handling helpers
[x] api_base management polishing; made it easier to switch between local and OpenAI APIs

[x] Fixed testing mocking for test/test_llm_wrapper.py
[x] Land #40. Don't forget to update the CHANGELOG.md

Kicking this can: Added chroma embedding demo

Released.. Just needs PyPI release, then we can close this, @choccccy?

PyPI released


TypeError: leaflet__WEBPACK_IMPORTED_MODULE_3___default.a.Control.Draw is not a constructor

Not sure if this one is bug or nor.
`
    var editableLayers = new L.FeatureGroup();
    this.map.addLayer(editableLayers);

    var MyCustomMarker = L.Icon.extend({
        options: {
            shadowUrl: null,
            iconAnchor: new L.Point(12, 12),
            iconSize: new L.Point(24, 24),
            iconUrl: 'link/to/image.png'
        }
    });

    var options = {
        position: 'topright',
        draw: {
            polyline: {
                shapeOptions: {
                    color: '#f357a1',
                    weight: 10
                }
            },
            polygon: {
                allowIntersection: false, // Restricts shapes to simple polygons
                drawError: {
                    color: '#e1e100', // Color the shape will turn when intersects
                    message: '<strong>Oh snap!<strong> you can\'t draw that!' // Message that will show when intersect
                },
                shapeOptions: {
                    color: '#bada55'
                }
            },
            circle: false, // Turns off this drawing tool
            rectangle: {
                shapeOptions: {
                    clickable: false
                }
            },
            marker: {
                icon: new MyCustomMarker()
            }
        },
        edit: {
            featureGroup: editableLayers, //REQUIRED!!
            remove: false
        }
    };

    var drawControl = new L.Control.Draw(options);
    this.map.addControl(drawControl);

    this.map.on(L.Draw.Event.CREATED, function (e) {
        var type = e.layerType,
            layer = e.layer;

        if (type === 'marker') {
            layer.bindPopup('A popup!');
        }

        editableLayers.addLayer(layer);
    });

`

Its throwing with 1.0.4 version of Leaflet.draw

This was my problem, i should have included the js explicitly for its availability to the react dom

In case anyone is wondering, this is the way i import leaflet, leaflet-esri, and leaflet-draw in CRA, should work on other webpack configuration once you have css-loader set up.
import L from "leaflet"
import "leaflet/dist/leaflet.css"
import * as esri from "esri-leaflet"
import "leaflet-draw"
import "leaflet-draw/dist/leaflet.draw.css"

import marker from "leaflet/dist/images/marker-icon.png"
import marker2x from "leaflet/dist/images/marker-icon-2x.png"
import markerShadow from "leaflet/dist/images/marker-shadow.png"

delete L.Icon.Default.prototype._getIconUrl
L.Icon.Default.mergeOptions({
  iconRetinaUrl: marker2x,
  iconUrl: marker,
  shadowUrl: markerShadow,
})

L.esri = esri


Should be a link to information regarding profiles

There is a command line option to specify a profile file. There is no information however about the profiles. There should at least be a comment along the lines of :
"For more information about profiles , see . . . . . " and point to the docs in mbed-os

https://github.com/ARMmbed/mbed-os/blob/master/docs/Toolchain_Profiles.md

It might be too long to include the link from above, but you could say something along the lines of "Please see the mbed OS documentation for more details" on the command line, and provide a link to the actual doc in the mbed-cli README.
If that makes sense to you, would you be able to add that?

Now implemented. Thanks @adbridge !


[WIP] Allow GpuWindowExec to partition on structs

Fixes #4626.
This updates the type check for WindowExec's partitionBy parameter to ensure that the code will run on the GPU. The underlying requirements for these queries to run on the GPU has already been implemented previously (See https://github.com/NVIDIA/spark-rapids/issues/2877), so this code just unblocks the check.  Also, the allow_non_gpu mark has been removed from the relevant time window tests, since these are no longer blocked from running on the GPU.

build

build

build

build


#47 See the current Angular version in the side menu

Tab in side menu shows current angular version
Icon will be red if needs update, green if up to date
Tab has dropdown which will show the latest and next versions of Angular
Tab has play button which will run the check for updates function

resolves #47

this should follow #46 and #51

yeah thats weird, it was marking the changes for this branch as conflicts, i went through and resolved


fixed compiler warning on unused variable when using -DELPP_THREAD_SAFE

With unused parameter warning enabled and ELPP_THREAD_SAFE defined, I get compiler warnings on unused parameter ms in msleep(). asynchronous logging is disabled.

Looks like this is fixed in the develop branch..

This is fixed in develop as mentioned - will be available in 9.81 release - closing


Make Z3 4.4.1 the recommended version?

Is there any reason for not making Z3 4.4.1 the recommended version? There are just minor differences in the output of two test cases (livevars/daytona_bug2_ioctl_example_2.bpl and test15/CaptureState.bpl). In one case the model is slightly different and in the other case the trace is slightly different (for a STORM-generated file that supposedly has quite some non-determinism due to thread interleavings).

@wuestholz +1 for moving to a newer version of Z3. There is one minor blocker right now which is I need to package a newer version of Z3 to use for the TravisCI build. I need to do this for a few other projects of mine as well so I'll be doing this quite soon.

Great! I would do the same for the Jenkins build server once we have updated the expected outputs of the test cases.

@wuestholz Great. Could you make a branch for the test changes required for Z3 4.4.1? we can then each do our own local testing and when we are ready to make the switch we can merge the branch into master.

Sure. I have created the branch and I committed the changes to the test output.

@wuestholz Just to let you know I am working on this.
I'm waiting for two PRs to merged into Z3 that fix various packaging problems.
https://github.com/Z3Prover/z3/pull/338
https://github.com/Z3Prover/z3/pull/340
Hopefully they will be accepted and then another release can be made so I can package it up.

#27 implements this.


set up end of questions input

after last question, display input
setup if time has run out message to stop/return end of quiz message

completed


Simplify down to queue-based git credentials

After a long discussion with @toolmantim in https://github.com/buildkite/elastic-ci-stack-for-aws/pull/469#, we decided to radically reduce the surface area offered by these hooks.
The addition of queue as a prefix allows for IAM rules to restrict access to a given elastic stack to just that queues secrets. Otherwise all stacks in an AWS account would have access to all secrets.
We removed env, as once the pipeline is checked out, plugins can be used for secrets management. This keeps these hooks focused on checkout.

After this lands, I'd also propose we rename the repo to aws-secrets-manager-agent-hooks or similar @toolmantim.

Nice! I'd even call it aws-secrets-manager-checkout-credentials-hooks

I've updated the README.

Nice! I'd even call it aws-secrets-manager-checkout-credentials-hooks

That is a mouthful! I reckon I might go with aws-secrets-manager-agent-hooks for the minute, as it leaves us open for expanding on it in future if things change.

I'm going to merge this and rename. @toolmantim happy to take feedback and I'll update master, I just want to reduce the possibility of folks depending on the previous prototype schema.

⛸


Don't use getenv(), use $_SERVER instead

See https://github.com/symfony/recipes/pull/156

Thanks @yceruto.


Update stable version in repo from 0.0.8 to 0.1.1

Think about update stable version to 0.1.1
Version: stable=0.0.8 (353 days old) => latest=0.1.1 (15 days old)
Installs: stable=1409 (84.52%), latest=150 (9%), total=1667
Click to use developer portal
Click to edit
Note: This is an automatically generated message and not personally authored by bluefox!

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 0.1.1 but in the meantime an update to version 0.2.0 is suggested.
So this issue will be closed and replaced by an updated one.


[Bug] Incorrect currency shown.

Not matter what currency is set, the list of DLC for a game (GOG) will always show as USD (have tried deleting getting the data again).

Do you have try to change your currency in plugin settings?

Yes, both DKK and EUR show USD prices.

I have the same issue


Docs crash

[2017-05-23 14:18:23,635: ERROR/PoolWorker-2] Task quality_manager.tasks.task_docs_coverage_check[a34d4531-5efb-4691-855c-79a736c49b2a] raised unexpected: TypeError('exceptions must derive from BaseException',)
Traceback (most recent call last):
File "/mnt/c/Users/joche/Documents/GitHub/MOOCworkbench/docs_manager/sphinx_helper.py", line 93, in get_coverage_data
coverage_list = pickle.load(open(coverage_pickle, 'rb'))
FileNotFoundError: [Errno 2] No such file or directory: 'github_repositories/jlmdegoede/Workbench-Python-Experiment/docs/_build/coverage/undoc.pickle'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/jochem/venvmooc/lib/python3.5/site-packages/celery/app/trace.py", line 367, in trace_task
R = retval = fun(*args, **kwargs)
File "/home/jochem/venvmooc/lib/python3.5/site-packages/celery/app/trace.py", line 622, in protected_call
return self.run(*args, **kwargs)
File "/mnt/c/Users/joche/Documents/GitHub/MOOCworkbench/quality_manager/tasks.py", line 52, in task_docs_coverage_check
docs_measure.measure()
File "/mnt/c/Users/joche/Documents/GitHub/MOOCworkbench/quality_manager/measurements/docs_measurement.py", line 27, in measure
coverage = sphinx_helper.get_coverage_data()
File "/mnt/c/Users/joche/Documents/GitHub/MOOCworkbench/docs_manager/sphinx_helper.py", line 104, in get_coverage_data
raise "Coverage data not found: {0}".format(e)
TypeError: exceptions must derive from BaseException

Fixed


Split RiboViz repository into Python, R and workflow repositories

Split RiboViz repository into 4 repositories:

Python: Python scripts as a Python package and pip-installable.
R: R scripts, as an R package.
Workflow: prep_riboviz.py as a Python package, requiring the above to have been installed.
Website/Shiny.

Cloning the RiboViz repository 4 times then removing content is not appropriate. The 4 new repositories would be of a significant size due to the Shiny data files in the RiboViz repository. The historical versions of these will remain in the .git folder of each repository even if deleted from a current version.
Instead, 4 new repositories will be created and the relevant content copied from the RiboViz repository into each new repository.
Challenges:

Naming of the new repositories.
Migrating the appropriate issues from the RiboViz repository to each new repository.
Synchronising the switchover within the team so that everyone begins to work on the new repositories.

Use git modules as intermediate stage until Python and R components have been packaged? https://git-scm.com/book/en/v2/Git-Tools-Submodules

Shiny and its data files lead to significant bloat compared to the rest of the code/data:
$ du -sm --total data/ docs install/ LICENSE  README.md  rmarkdown/ rscripts/ vignette/
18      data/
1       docs
1       install/
1       LICENSE
1       README.md
2       rmarkdown/
1       rscripts/
91      vignette/
# 117 total

$ du -sm website/
414     website/

Shiny has been purged - see #139 - but other modularisations may be worth considering.

Agree that these modularisations are worth doing. Certainly splitting R workflow into its own R package.
Where would we put the python functionality, and where the nextflow workflow?

An update of the proposed split in the issue description is to split RiboViz repository into 3 repositories:

pyriboviz: Python code and tools, and tool-specific test code, installable as a package via setup.py/pip.
r_riboviz: R code and tools, and tests, as an R package/library.
``riboviz`: Nextflow workflow plus Nextflow-specific Python tests and regression tests.

The Python workflow goes nowhere as it is immediately consigned to history post 2.0.

For R see https://r-pkgs.org/. It would require a redesign of our R scripts into those that have functions only (and which would go in a package's R directory) and those that are the command-line tools (it isn't clear where these go, but an exec directory seems to be a likely candidate - certainly the optparse package developers put their example command-line tools there - https://github.com/trevorld/r-optparse/tree/master/exec).

Arising from #310, notes on the minimal requirements to install our R functions as a package. With the help of https://kbroman.org/pkg_primer/pages/minimal.html and https://kbroman.org/pkg_primer/pages/build.html.
Create folder with desired contents:
rpackage/
  R/
    collate_tpms_functions.R # Removed if-block that imports provenance.R
    provenance.R
  DESCRIPTION

DESCRIPTION is:
Package: riboviz
Version: 2.1

Build and install:
$ cd rpackage
$ R CMD build .
* checking for file 'rpackage/DESCRIPTION' ... OK
* preparing 'riboviz':
* checking DESCRIPTION meta-information ... OK
* checking for LF line-endings in source and make files and shell scripts
* checking for empty or unneeded directories
* creating default NAMESPACE file
* building 'riboviz_2.1.tar.gz'
$ R CMD INSTALL riboviz_2.1.tar.gz
* installing to library '/home/ubuntu/R/x86_64-pc-linux-gnu-library/3.6'
* installing *source* package ‘riboviz' ...
** using staged installation
** R
** byte-compile and prepare package for lazy loading
** help
No man pages found in package  'riboviz'
*** installing help indices
** building package indices
** testing if installed package can be loaded from temporary location
** testing if installed package can be loaded from final location
** testing if installed package keeps a record of temporary installation path
* DONE (riboviz)
$ Rscript ../riboviz/rscripts/list-r-packages.R | grep riboviz
              riboviz      2.1

In another window:
$ R

> library(riboviz)
> packageVersion("riboviz")
[1] '2.1'
> LoadTpms
...
<bytecode: 0x56106f61d778>
<environment: namespace:riboviz>
> write_provenance_header
...
<bytecode: 0x56106f649708>
<environment: namespace:riboviz>

Alternative approach using devtools in R:
$ R

> remove.packages(riboviz)
> build(".")
[1] "/home/ubuntu/riboviz/riboviz_2.1.tar.gz"
> install(".")
...
* DONE (riboviz)

Try to use a function. In another window:
$ R

> CollateTpms("collated_tpms.tsv", NA, c(WTnone="WTnone_tpms.tsv", WT3AT="WT3AT_tpms.tsv"))
Error in get_Rscript_filename() :
  could not find function "get_Rscript_filename"

Try via source:
> source("rscripts/provenance.R")
> source("rscripts/collate_tpms_functions.R")
> CollateTpms("collated_tpms.tsv", NA, c(WTnone="WTnone_tpms.tsv", WT3AT="WT3AT_tpms.tsv"))

OK, need to specify dependencies. Edit DESCRIPTION:
Package: riboviz
Version: 2.1
Imports:
 dplyr,
 here,
 purrr,
 readr,
 tibble,
 tidyr,
 Biostrings,
 getopt,
 git2r

Rebuild and reinstall and retry:
> library(riboviz)
> CollateTpms("collated_tpms.tsv", NA, c(WTnone="WTnone_tpms.tsv", WT3AT="WT3AT_tpms.tsv"))
[1] "Loading ORFs from: WTnone_tpms.tsv"
...
[1] "Loading TPMs from: WTnone_tpms.tsv"
...
[1] "Loading TPMs from: WT3AT_tpms.tsv"
...

$ head collated_tpms.tsv
# Created by: RiboViz
# Date: 2021-04-27 03:23:56
# File: NA
# Version: unknown
ORF	WTnone	WT3AT
YAL001C	116.7	953
YAL002W	212.8	365.8
YAL003W	199013.8	100458.6

OK
Note that file and version are unknown. provenance.R functions would need to be updated to get this information some other way e.g. use packageVersion("riboviz") instead of the Git hash. Arguably only point versions, not Git commit versions, should be used for published results anyway.


#142 - Support for byte array

Added very basic byte array, with
SETBIT, GETBIT and BITCOUNT support to prove functional capabilities.
Please share feedback.

Please review @arpitbbhayani @JyotinderSingh

This is amazing. Loved the implementation of popcount.


test: lazy-oopif-frame.html uses an invalid src for the lazy loaded iframe

As we started implementing lazy iframe loading in Firefox, the [oopif.spec] OOPIF-debug OOPIF should support lazy OOP frames test started failing.
I looked a bit into it, and while I'm not sure why it passed before, the lazy loaded iframe looks odd:

With src='www.example.com' the iframe will not attempt to load the actual example.com, but just hit a relative path from the base url of the top page. I'm not sure what the intent was, but I guess we should rather use a valid URL which resolves to something on the test server?

cc @Lightning00Blade


Modal and Header

So im havin some troubles trying to render both a custom Header and the Modal on the same screen. When the Modal is supposed to be visible, he appears off-screen. My code is something like this:
<View style={{ flex: 1 }}>

<ScrollView>
    ...
<ScrollView>

Any idea how to style this?

👋 the best place for this kind of questions is StackOverflow. I would suggest you to post it there, tag it as react-native and add a reproducible example of your problem.


ReDoc's SupressWarnings doesn't work

Despite setting ...ConfigObject.SupressWarnings = true, ReDoc still shows warnings at the top of the page.

Can you provide a set of repro steps - specifically I want to know what constitutes a warning and how do I trigger one to test out the flag?

Here's what I get once I add options.ExampleFilters(); inside services.AddSwaggerGen:

Not sure yet what exactly in my examples triggers it.

OK I found a reason of those warnings, it's a problem with Swashbuckle.AspNetCore.Filters. It generates an incorrest JSON for a request example, so should be easy for you to reproduce.

Looks like I found the reason for those warnings. It's a problem with Swashbuckle.AspNetCore.Filters. It generates an incorrest JSON for a parameter example, so should be easy for you to reproduce.

@Leon99 , not sure if you still want suppressWarnings to work. PR 1001 is approved but not deployed nuget to use it. Mean while, there is an alternate to make it work with Version="4.0.1". Instead of using SupressWarnings use AdditionalItems property in ConfigObject. The below code worked for me.
      app.UseReDoc(c =>
            {
                c.SpecUrl = "../swagger/v1/swagger.json";
                c.ConfigObject = new Swashbuckle.AspNetCore.ReDoc.ConfigObject() {
                    AdditionalItems = { { "suppressWarnings", true } }

                };
            });

Fixed as part of https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/1001


irc: turn off smartparens mode for circe buffers

Users probably don't want ":-(" to be autocompleted as ":-()" so let's
turn of smartparens for chatting.

Sounds good to me. Thanks for the help (and Merry Christmas)!

Thanks! (Merry Christmas, too!)


doc(kfp): bump version to 1.0.3

/assign @8bitmp3 @joeliedtke
Part of https://github.com/kubeflow/pipelines/issues/4540

/approve

Thank you
/lgtm
@Bobgy @joeliedtke @jlewi (When) shall we consider removing the "Beta" banners from KFP docs? I assume the version number in the master branch - v1.0.x - corresponds to KFP and not a subcomponent (https://www.kubeflow.org/docs/pipelines/pipelines-quickstart/ among others). WDYT?

cc @RFMVasconcelos

@8bitmp3 that's tracked in https://github.com/kubeflow/pipelines/issues/4617


Added Cosmos java team members to cosmos code owner list

Add @simplynaveen20 @xinlian12 @milismsft to cosmos code owners

/check-enforcer override


Memory leak in PixelReader.h

Hi guys,
today we noticed that we run out of memory when simulating with a camera.
A quick look with UnrealInsights showed that the memory tagged with EngineMisc is increasing steadily.

In my local version, I have the possibility to deactivate sending the data to python. i.e. to not execute the code in PixelReader.h.
When I use that option, the memory leak does not occur, so it seems like some problem with PixelReader.h.
Best Robert

Hi Robert,
I've been looking into a similar problem for a while now.
Are you using synchronous mode from python by any chance?
Kind regards,
Marcel

Hi Marcel,
yes, I am. Exclusively that mode, tbh. :)
Best Robert

Hi, is this memory leak problem arised since 0.9.14 ?
Recently, I encountered memory leak in 0.9.14 and 0.9.15, but not in 0.9.13 with the same python scripts.


send content-disposition HTTP header for PDFs

makes Firefox PDF downloads robust across machines and platforms
predefines sane local download filenames with .pdf extension
includes cosmetic improvements to the README

Side Effects:

Chrome will download PDFs now instead of directly previewing
adding a npm dependency also pulled puppeteer up to the new version 1.1.0 in the lockfiles.

Thanks for your PR!!

thanks for liking it :)


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/112132

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/116158

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/116673

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/119218


Add values to document matcher fluent interface

After merging this into salsify-master we'll have to rebase https://github.com/agileanimal/grape-entity-matchers/pull/18.
@tjwp - you're prime

:shipit:


direnv watch flake.lock

nix environment should update if lock file is changed.
watch_file is a direnv command which does that automatically: https://github.com/direnv/direnv/blob/9f22a3ea7e9fc835bb0569f14b35d756d02a4b7c/man/direnv-stdlib.1.md?plain=1#L368

It looks like watch_file flake.lock is already included by use flake.
https://github.com/direnv/direnv/blob/9f22a3ea7e9fc835bb0569f14b35d756d02a4b7c/stdlib.sh#L1276

It looks like watch_file flake.lock is already included by use flake.
I see. Then I have another problem since it is not updating env unless I reopen the terminal.


Adding Tab press for showing available methods and autocompletion for objects

Describe the feature and the current behavior/state.
Pressing the tab button does not show methods or autocomplete already defined objects when using Jupyter notebooks.
I tried this solution https://stackoverflow.com/questions/54281719/how-to-get-code-completion-in-a-jupyter-notebook-running-in-docker. It didn't do anything when I tried it though.
%config IPCompleter.greedy=True
Example of how it works in Jupyter notebook when installed with conda.
import pandas as pd
pd.<tab_here>   # should show all availabe methods e.g. pd.read_csv(), pd.read_excel(), etc.

System information
I used the following below and the feature was missing. I don't know if it has been addressed or fixed in the new versions.

TensorFlow Docker Image:  tensorflow/tensorflow:2.1.0-gpu-py3-jupyter
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 18.04
GPU model and memory: 1080Ti
Are you willing to contribute it (Yes/No): No because I don't know how to implement it.

Will this change the current api? How?
I don't think so. It's just an add-on I think.
Who will benefit with this feature?
The showing of methods is a behavior that works when installing jupyter from conda and I would say is a standard feature in Jupyter notebooks. I've never seen anybody that couldn't hit tab and pull up methods or had autocomplete. It helps the user so they can see what options they have to call. This can save a massive amount of time and help the user be much more efficient.

Hi @pdubz-sudo !
We see that you are using old version of Tensorflow which is officially considered as end of life, We recommend that you upgrade to 2.6 version and let us know if the issue still persists in newer versions .Please open a new issue in case you face any errors, we will get you the right help .Thanks!

@mohantym Hey there! I tried the nightly build but the jupyter notebook didn't have the method list and autocomplete.
For example, doing tf.__ and then hitting tab will have a drop down list of all the available methods which one can then choose from.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Optimize readme.md

currently it is ugly. we must make it look hot, images, badges, links, contributor list and anything else!

Hello @FireCubeStudios , I'd love to take this up for hacktoberfest. Kindly assign it to me. Thanks

Done!

Done!

Thank you


Does not start automatically after boot

The VolumeKeyLongPressListener isn't set automatically after booting up the device. Currently it is necessary to open the app once after a reboot to make it work properly, but ideally it should just work without having to open the app.
I think there might be 2 reasons for this happening: A) the auto start feature is not yet included in the app or B) my Samsung device is killing the process after it auto starts, but not when you manually open the app. However, all battery saving or RAM-killing features of my phone are already turned off in my settings.
I included a logcat: logcat.txt. Around 23:52 I rebooted the device and some skiptrack activity is present. At 23:57 I tried a long press volume up, but nothing happens. At 23:58 I opened the skiptrack settings for a few seconds and the VolumeKeyLongPressListerer gets set. At 00:00 I tried again a long press volume up, and now it works properly.
P.S. I also have a little feature request: adding support for play/pause when you long press both volume up and volume down at the same time. And releasing the app to the Play Store to increase the number of users of this app.

Fantastic!
This fork lets me change songs in Youtube Music on Android 12 when it was previously not compatible. Strangely, I can no longer change media using the long hold volume buttons when the screen is off, where I previously could. Funky.
Thanks a ton for pointing this out and creating the apk! Much appreciated.


Community plugins

[ ] give a base a repo or yeoman generator which has pre-set configuration, file structure & theme
[ ] create a shared empty-state plugin for webdash plugins
[ ] define plugin design guidelines (this will be available as a demo once the generator is completed)

Generator progress

[ ] api.js default template
[ ] setup polymer 2 element
[ ] setup readme
[ ] setup Backend if possible
[ ] Setup sample design guidelines in app

Updated docs
🎉


Add vertical scrollbar to a Grid Column

This could be a noob Q, but I'd appreciate some help with this one.   I have the following grid which is basically a master / detail. The left column is a pick item list and the right is the detail for the corresponding selected item.
Q) The pick item list is getting rather long and I'd like to enable a vertical scrollbar if the list gets over a certain vertical height.  I'd like the detail column to not be effected.  Is there a semantic technique for this?

                                <input type="text" placeholder="Search apps" ng-model="query"/>

                            {{ item.label }}

                                    {{selectedApp.label}}

                                        {{selectedApp.description}}

                                        Additional Details

closed. Gonna try #2206


displayManager slim (and auto) not loading display-manager

Issue description
no errors, just 3 attempts to load display-manager and fallback to system console, very similar to this issue (logs same)
I can provide logs if needed
Steps to reproduce
uncomment this, comment lightdm
Technical details

System:  (NixOS: nixos-version, Ubuntu/Fedora: lsb_release -a, ...) 17.09.1880.ac2bb5684c (Hummingbird)
Nix version:  (run nix-env --version) nix-env (Nix) 1.11.15
Nixpkgs version: (run nix-instantiate --eval '<nixpkgs>' -A lib.nixpkgsVersion) "17.09.1880.ac2bb5684c"
Sandboxing enabled: (run grep build-use-sandbox /etc/nix/nix.conf) build-use-sandbox = false

SLIM is no longer in nixpkgs: ce26b3e


[stable/keycloak] fix readinessProbe when using empty basepath

When the basepath is set to "" to host keycloak on keycloak.host/ instead of keycloak.host/auth the readinessProbe fails because the url is set to //realms/master
Signed-off-by: Michael Dop<EMAIL_ADDRESS>

/assign @unguiculus

/ok-to-test

/lgtm


Sign compiled program

I'd like the ability to sign a compiled teal program that will act as a delegate signature

Another issue I've had is translating back and forth between the official js sdk and algosigners txn format.  As a possible solution, can I just pass their sign function to algosigner and it partially applies the arguments to the function with with the secret key?
I'm terrible with js so this may be a horrible idea

In other words I pass this https://github.com/algorand/js-algorand-sdk/blob/develop/src/logicsig.ts#L281 and you give me back a closure with the sk applied or I pass it + the args necessary and you give me back the result?

You can see in the develop branch our work on v2 signing. How it will work is the developer will pass in the transaction in a msgpack'd format, and we'll unpack it on the far side using the JS SDK. This way you won't need to do anything special with encodings and custom transaction building.
I'll ask one of our devs to check in here to review the logicsig signing request.

I too would like to be able to supply some compiled TEAL and have the user sign it to act as a delegated signature. Any update on this?


baxter_interface: 1.1.0-0 in 'hydro/distribution.yaml' [bloom]

Increasing version of package(s) in repository baxter_interface to 1.1.0-0:

upstream repository: https://github.com/RethinkRobotics/baxter_interface.git
release repository: https://github.com/RethinkRobotics-release/baxter_interface-release.git
distro file: hydro/distribution.yaml
bloom version: 0.5.11
previous version for package: 1.0.0-0

baxter_interface
* Updates baxter_interface to ROS Indigo
* Upgrades to the Indigo robot required a reworking of how cameras power **on** / **off** - Three cameras can no longer be powered at the same time, and closing cameras turns power **off** to the specified camera and **on** to the other two
* Updates joint trajectory action server to use new Inverse Dynamics Feed Forward Commands and configurations for smoother & more accurate MoveIt trajectory execution
* Updates joint trajectory action server default control mode from *position* to *position_w_id* which uses 'raw' joint position control
* Updates joint trajectory action server to replace linear, cubic, and quintic spline fitting with *Cubic Bezier Spline* interpolation. These splines can use Position, Velocity, and/or Acceleration to more accurately interpolate supplied trajectories
* Adds head trajectory action server interface for controlling Baxter's head pan joint (thanks to @aginika for contributing this)
* Updates verification of Gripper software versions to check firmware Build Date instead of Build Version (SDK gripper version 1.1 conflicted with Manufacturing versions)

This PR reverts some recent commit. Please redo the PR.


pin cuda-nvcc as a temporary workaround

fix #387 (potentially, pending more testing)

@ngam @dhruvbalwada is this only to support k80? We can just say we no longer support k80s and switch just to T4s. In https://github.com/2i2c-org/infrastructure/pull/1772 I make T4s the default

This PR will provide support for T4s only. The problem with K80s was not resolved and requires turning multi-threading off, so maybe just switch over to T4 entirely.

@dhruvbalwada yeah, https://github.com/2i2c-org/infrastructure/pull/1772 makes T4s the default - K80s are still available there. We only have quota for 4 T4s, should we get more?

I don't know the answer to that one yet. I personally probably won't be using more than that (probably will be using only 1 GPU) for the next few months. Maybe if we need more, then we can increase at a later point.
Also, based on looking at the logs, do you see any other users reaching the limit often? If so, maybe we can ask them personally if they would like more?

Edit: I actually didn't even realize that more than one GPU can be used. How do we access them?

These quotas are for the whole project. It means only 4 people can be using T4s at once on the entire M2LInES hub. We don't support multiple GPUs per user, but could potentially try that if people thought it would be useful.
In terms of general usage, we have a bit of chicken vs. egg problem here. We have not done much work to explain to users how to take advantage of these resources. We don't have demos, examples, training, etc. So it is not surprising that usage is low.
In the short term, it's fine to drop K80s and use only T4s. I doubt that we would have more than 4 simultaneous GPU users in the project.

We could definitely try multiple GPUs on the same pod if folks think that would be useful. I've made T4s the default now, but I think the suggestion is that we drop K80s completely. https://cloud.google.com/compute/docs/gpus#nvidia_gpus_for_compute_workloads is the list of GPUs available, let me know if we should make any other ones available. It is fairly trivial to do so!


feat: Add WhereHas Method to ORM

Closes #
📑 Description
This PR introduces a feature inspired by Laravel's ORM: the WhereHas method. The WhereHas method allows users to efficiently query records based on the existence of related records and apply conditions to those related records.
Why it is important ?

Simplified Complex Queries: With WhereHas, developers can easily perform complex queries involving relationships without the need for multiple database queries or manual filtering.
Consistency: Aligning with Laravel's ORM conventions makes it easier for developers familiar with Laravel to work with this ORM.

Example
Suppose you have a database with two main models: User and Book, and there is a one-to-many relationship between users and books, where each user can have multiple books. You want to find users who have books with a specific condition, such as books with the word "fantasy" in their names.
query.WhereHas("books", func(subQuery ormcontract.Query) ormcontract.Query  {
    // Apply conditions to the related "books" model.
    subQuery.Where("Name", "like", "%fantasy%")
})

✅ Checks

[ ] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x ] All the tests have passed

ℹ Additional Information

If the PR can be reviewed again, please pin me, thanks!

Hello @hwbrzzl , The PR is now ready for review. However, it seems that the last two tests are failing. I'd like to understand the reason behind these failures and address any issues accordingly. The main message of this failing are "Added line #xxx was not covered by tests"

Hi @sCuz12 Are you still interested in working on this, please?

@sCuz12 any update?

Close this first, please reopen it if have any update.


Excluding js (false) renders "true"

Digging through Paloma this afternoon, and noticed when I exclude js(false) for a controller or specific action, it renders "true" within the insert_paloma_hook div tag.  Running Paloma's test_app (http://localhost:3000/main/prevent) renders "true" too.  Is this intended?
Thanks ahead of time!

@theredcoder what version is this? Thanks.

@kbparagua the app is using:
paloma (4.2.0)
jquery-rails (4.0.3)
rails (4.2.1)
ruby (2.1.2)

+1 have run into this today. Not certain, but it seems to occur when js is defined more than once in the same action. For example:
def create
  js :new # we want to run the js from the #new action here too

  # .... does something ...

  js false # decided that we no longer require js, eg. after successful save of an object
end

Fixed as of 4.2.1

Awesome, thank you!


fix checkstyle version

Description
Guava has one CVE , we need to bump to 30.0. https://nvd.nist.gov/vuln/detail/cve-2020-8908
For ml-commons, the
Before this PR
checkstyle - The Checkstyle libraries to be used for this project.
\--- com.puppycrawl.tools:checkstyle:8.37
     ...
     +--- com.google.guava:guava:29.0-jre
     ...

After this PR
checkstyle - The Checkstyle libraries to be used for this project.
\--- com.puppycrawl.tools:checkstyle:8.38
    ...
     +--- com.google.guava:guava:30.0-jre
     ...

Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I thought I already did that here

I thought I already did that here

This PR just move the version block to before the line using it. Check the description about the difference.


How to  Set Up CI/CD in a Vue Project with Firebase Hosting and GitHub Actions

Article Checklist
NOTE: (1st time contributors - submit ONLY one article to get familiar with the process, quality, and style-once published, you can then submit any number of articles.)
Preliminary Checks - Formatting and Structure

[x] Does your article follow any of the suggested structure formats? - see example formats
[x] Is your article properly formatted in Markdown? - see Github Markdown guide
[x] Have you used the correct folder and file structure? - see Contribution Guidelines
[x] Is your article over 750 words? - Tool: Wordcounter.net
[x] Have you used our preferred heading structure? - H3 (###) and up
[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.
[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.
[x] Have you included a hero image that is 600x400 pixels and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images
[x] In your PR - add a comment that states "This fixes #(enter your Idea Content Form #)" to link and close your Idea Content (issue) when the PR is merged.
This fixes #1160

Grammar Checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid -  Grammarly
[x] Have you checked your article for readability? - Tool: Hemmingway
[x] Have you added sources for quotes and images that aren't yours?

Technical Checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide

Contribution Guidelines
For first-time contributors and for more details, see Contributing Guidelines

@hectorkambow , I would like to review this article?

@hectorkambow , I would like to review this article?

@katungi  will be reviewing this article 🚀

@katungi  will be reviewing this article 🚀

plagiarism check done

plagiarism check done

@Worawat101 Is this article ready or is it still a draft?

@Worawat101 Is this article ready or is it still a draft?

@katungi I wondering that your can help me getting win this week by publishing this post.

@katungi I wondering that your can help me getting win this week by publishing this post.

@hectorkambow, I am done with the tech review and everything looks good. Also, could you advice on the image link issue as you proceed with the next step?

@hectorkambow, I am done with the tech review and everything looks good. Also, could you advice on the image link issue as you proceed with the next step?

@Worawat101  great overall article/tutorial .
Please go over the changes I made during final to help improve future articles.
One example - the images paths
cc @katungi please do the same to improve revisions.


Desktop

-Add index.html for desktop
-Add stylesheet for desktop
-Fix linter errors

@frempongdev there's a problem with the linter and its not from my local project, it needs to be updated on the documentation.


Apply remaining changes from upstream post-open-source

#5 missed a couple of bug fixes and a minor cosmetic change that was implemented upstream after open-sourcing AD; this PR ports these changes over.
As of the time of writing, after this PR, there are two upstream commits left to port:

https://github.com/IvarK/AntimatterDimensionsSourceCode/commit/bcf3448a9fa8360e696e87985faf2b4359c44e45 (off-by-one in glyph inventory space check in Reality modal)
https://github.com/IvarK/AntimatterDimensionsSourceCode/commit/acc5b974ce00e937c1ad5d764a8da2ee08e9047c (incorrect time variable in Automator timings)

However, I'm waiting for the complete decimalization of real time and Realities to port them over.

Shop shouldn't exist anyways in code, so respecIAP is useless

I have removed the corresponding commit from this PR.

UnscaledDeltaTime needs to be trueDeltaTime iirc

This refers to the commit fixing the time units for the automator interval, which I mentioned, but only ported over and pushed after reading the message.

The simulated reality code will probably have a look at in the near future, but it works rn so ill push briefly


Is 3D convolution supported?

From the release notes, it seems supported.
But in the porting guide it says that MIOpen only supports 2D Convolutions. It also says this " all the “Nd” APIs in cuDNN do not have a corresponding API in MIOpen". Is this information incorrect ?
From the API reference, is miopenSetTensorDescriptor analogous to cudnnSetTensorNdDescriptor ? There are also these APIs: miopenInitConvolutionNdDescriptor, miopenGetConvolutionNdDescriptor .

@akankshabaranwal

Is this information incorrect ?

It is outdated.

From the API reference, is miopenSetTensorDescriptor analogous to cudnnSetTensorNdDescriptor ?

Yes.

[Recommendation] @junliume @JehandadKhan https://github.com/ROCmSoftwarePlatform/MIOpen/labels/documentation https://github.com/ROCmSoftwarePlatform/MIOpen/labels/urgency_normal
Let's assign an engineer to fix the docs.

@atamazov Internal ticket has been created to fix the docs. Thanks!


Unallowed State Transition Response Codes

Description
When requesting an unallowed state transition on a document on the CRUD service (ie: from PUBLIC to DELETED) the CRUD service responds with a 404 NOT FOUND status code and the following response body:
{
    "error": "not found"
}

However, I find this response not helpful, as it is the same that is returned when the document is not found. I believe that the appropriate response would be a 400 BAD REQUEST with a response body similar to something like this:
{
  "error": "transitions from state 'PUBLIC' to 'DELETED' are not allowed"
}

Environment
I'm using CRUD Service version 4.0.0, but I suspect that this issue is also relevant for newer versions
Minimal Reproduction

Create a document with __STATE__ set to PUBLIC
Request a state change by performing a POST request to /your-collection/your-document-id/state with the request body set to:

{"stateTo":"DELETED"}

Observe the returned response status code

I would not consider this a bug, but rather an enhancement of the interface. We can work on it to improve CRUD Service interface, but let's keep in mind that it will be a breaking change, since CRUD Service users may have built a logic considering the current HTTP response.

#122 closes this issue


[DOCFIX] Update Tensorflow Doc

What changes are proposed in this pull request?

remove redundant info, add redirect link to section on local cache quick start doc
update external links to open in new tab

Why are the changes needed?
better user journey
Does this PR introduce any user facing changes?
n/a

alluxio-bot, merge this please


Variable was declared but not used

Minor issue is that defining shell variables within the shell part of the process and then using them within the command string is not supported. The server complains about variable being declared but not used. Example:
process example {
    shell:
    njobs = Math.ceil((task.cpus as int) / 4) as int
    '''
    echo "Number of jobs to be used: !{njobs}"
    '''
}

I think we will deprecate the shell block at some point in the future, because it was designed to remove confusion between nextflow and bash variables, but this is largely addressed with syntax highlighting and the language server. For this reason I did not try to parse the variables in the shell string, which would be complicated


SDK does not auto register device for Push Notifications

I am trying to setup push notifications for Android but the SDK is not able to register the device successfully. I've set up the Android Notification Settings with my Firebase Cloud Messaging Server ID and Key. Please note I already have notifications working on my app outside of GetSocial. I've tried on both simulator and physical device.
With autoRegister flag to true I get the following error:
[ +344 ms] D/GetSocial_upgqDBbsrL( 8781): GetSocial DeviceId: 375e8eac-5dc7-48d9-91fe-2b868fea0bc7
[+7038 ms] W/GetSocial_XdbacJlTDQ( 8781): Failed to obtain push token

When I switch the configuration autoRegister to false and call registerDevice() manually I get:
[  +56 ms] I/flutter (17421): Notifications.registerDevice
[        ] I/flutter (17421):
[   +6 ms] W/GetSocial_JSON(17421): Call failed: Notifications.registerDevice, body:
[   +1 ms] W/GetSocial_JSON(17421): Unsupported method: Notifications.registerDevice, body:
[        ] W/GetSocial_JSON(17421): java.lang.RuntimeException: Unsupported method: Notifications.registerDevice, body:
[        ] W/GetSocial_JSON(17421): 	at im.getsocial.sdk.json.GetSocialJson.callAsync(GetSocialJson.java:1553)
[        ] W/GetSocial_JSON(17421): 	at im.getsocial.GetSocialPlugin.onMethodCall(GetSocialPlugin.java:67)
[        ] W/GetSocial_JSON(17421): 	at io.flutter.plugin.common.MethodChannel$IncomingMethodCallHandler.onMessage(MethodChannel.java:233)
[        ] W/GetSocial_JSON(17421): 	at io.flutter.embedding.engine.dart.DartMessenger.handleMessageFromDart(DartMessenger.java:85)
[        ] W/GetSocial_JSON(17421): 	at io.flutter.embedding.engine.FlutterJNI.handlePlatformMessage(FlutterJNI.java:818)
[        ] W/GetSocial_JSON(17421): 	at android.os.MessageQueue.nativePollOnce(Native Method)
[        ] W/GetSocial_JSON(17421): 	at android.os.MessageQueue.next(MessageQueue.java:335)
[        ] W/GetSocial_JSON(17421): 	at android.os.Looper.loop(Looper.java:183)
[        ] W/GetSocial_JSON(17421): 	at android.app.ActivityThread.main(ActivityThread.java:7664)
[        ] W/GetSocial_JSON(17421): 	at java.lang.reflect.Method.invoke(Native Method)
[        ] W/GetSocial_JSON(17421): 	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:592)
[        ] W/GetSocial_JSON(17421): 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:947)

Using version 7.4.11 due to: https://github.com/getsocial-im/getsocial-flutter-sdk/issues/1
getsocial {
    appId "xyz" // Dev App Id
    version "7.4.11"
}

GetSocial Android Configuration: getsocial.json
{
    "autoInit": true,
    "pushNotifications": {
        "android": {
            "titleColor": null,
            "messageColor": null,
            "channelName": null,
            "icon": null,
            "backgroundImage": null,
            "channelDescription": null,
            "largeIcon": null
        },
        "autoRegister": true,
        "customListener": true,
        "foreground": false
    },
    "disableFacebookReferralCheck": false,
    "uiConfig": null
}

$ flutter doctor -v
[✓] Flutter (Channel stable, 2.2.2, on Mac OS X 10.15.7 19H524 darwin-x64, locale en-US)
    • Flutter version 2.2.2 at /Users/mesrob/Library/flutter
    • Framework revision d79295af24 (6 weeks ago), 2021-06-11 08:56:01 -0700
    • Engine revision 91c9fc8fe0
    • Dart version 2.13.3

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0-rc4)
    • Android SDK at /Users/mesrob/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0-rc4
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.4, Build version 12D4e
    • CocoaPods version 1.10.1

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 51.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.58.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.24.0

[✓] Connected device (3 available)
    • Pixel 3 (mobile)   • 8C7X1K15P                            • android-arm64  • Android 11 (API 30)
    • iPhone 11 (mobile) • B7184D89-6AC3-4FE6-B5B2-39107ACC6284 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-13-5 (simulator)
    • Chrome (web)       • chrome                               • web-javascript • Google Chrome 91.0.4472.164

• No issues found!

Hi,
Please update GetSocial plugin version to 7.5.1, it fixes the issue with Notifications.register method.
To understand the other issue you reported, please share the complete ADB logs of your app session.

Hi @gaborvass, updating to Flutter plugin version 7.5.1 resolved the register() issue. I have removed specifying the version field in the android/app/build.gradle file as well.
Still seeing the error Failed to obtain push token. ADB Log attached.
adb_log.txt

Thanks for the logs!
Looks like you're using Flutter-FirebaseCloudMessaging package and it causes some issue in our plugin.
If you want to use only GetSocial Push Notifications, I would suggest removing Flutter-FCM.
If that's not possible, we'll investigate what is the issue if both plugins are added.

I see, thanks for the update. I have my own notification systems for other parts of my app so definitely I'll still need FCM. If you can please find a way to make both work, great appreciate it. Let me know if I can be of help!

Found the solution: please add the following dependency to your app's build.gradle file:
implementation("com.google.firebase:firebase-iid")

That fixed it! Thanks for addressing the issue. Will let you know if anything comes up.


[SPARK-37438][SQL] ANSI mode: Use store assignment rules for resolving function invocation

What changes were proposed in this pull request?

Under ANSI mode(spark.sql.ansi.enabled=true), the function invocation of Spark SQL:

In general, it follows the Store assignment rules as storing the input values as the declared parameter type of the SQL functions
Special rules apply for string literals and untyped NULL. A NULL can be promoted to any other type, while a string literal can be promoted to any simple data type.

Why are the changes needed?

Currently, the ANSI SQL mode resolves the function invocation with Least Common Type Resolution based onType precedence list. After a closer look at the ANSI SQL standard, the "store assignment" syntax rules should be used for resolving the type coercion between the input and parameters of SQL function, while the Type precedence list is used for "Subject routine determination"(SQL function overloads).

I have also done some data science among real-world SQL queries, the following implicit function casts are not allowed as per Least Common Type Resolution but they are commonly seen:

Numeric/Date/Timestamp => String, e.g. tableau generated query CONCAT(DATE_ADD(%1, CAST(%2 AS INT)), SUBSTR(CAST(%1 AS TIMESTAMP), 11)) AS TIMESTAMP)
Timestamp => Date, e.g date_sub(now(), 7) < ...
Double => Long, e.g.  from_unixtime(updated/1000.0)

The changes in this PR is ANSI compatible and it is good for the adoption of ANSI SQL mode.
Does this PR introduce any user-facing change?

Yes, Use store assignment rules for resolving function invocation under ANSI mode.
How was this patch tested?

Unit tests

cc @cloud-fan @allisonwang-db @entong

Test build #145508 has started for PR 34681 at commit 8d0b522.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49980/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49980/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49980/

Test build #145508 has finished for PR 34681 at commit 8d0b522.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145508/

Test build #145513 has started for PR 34681 at commit 65a8758.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49987/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49987/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49987/

Test build #145513 has finished for PR 34681 at commit 65a8758.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145513/

Test build #145528 has started for PR 34681 at commit b7d383e.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50000/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50000/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50000/

Merging to master

Test build #145528 has finished for PR 34681 at commit b7d383e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145528/


Fix incompatible logerrors type in pg_exttable

closes: #21

Change logs
This commit is directly cherry-picked from https://github.com/greenplum-db/gpdb/pull/14123.
logerrors type in pg_foreign_table has been changed from bool to char by design in GPDB7 to support log error persistence for external tables, and originally the logerrors of extension gp_exttable_fdw was changed accordingly. This PR changes the type back to preserve backward compatability.
Because the factual data are stored in pg_foreign_table and this PR only changes the display of the data, it should have no compatability implication.
Does this PR introduce any user-facing change?
No.

 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.

I reset the author of this PR from the original author of https://github.com/greenplum-db/gpdb/pull/14123 to me. Don't know if this will cause problems.

I reset the author of this PR from the original author of greenplum-db/gpdb#14123 to just me. Don't know if this will cause problems. If so I can change it back.

Hi @Ray-Eldath, thanks for you to point this out. If we just cherry-pick the commits for the upstream without any changes, we should keep the original author and copyright information.

I reset the author of this PR from the original author of greenplum-db/gpdb#14123 to just me. Don't know if this will cause problems. If so I can change it back.

Hi @Ray-Eldath, thanks for you to point this out. If we just cherry-pick the commits for the upstream without any changes, we should keep the original author and copyright information.

but then there's a CLA issue.

but then there's a CLA issue.

@Ray-Eldath: Sorry to confuse you. The CLA is optional, and not required, especially for the upstream contributors. Their code is already under Apache License v2.0, for us, we have got the grant to redistribute them, so It's ok to ignore the CLA. Let's keep the original copyright information, thanks!


Bug fix and support for timestamps

Added support for getting timestamp columns from ResultSet and for setting timestamp values in PreparedStatement.  Decided to use the string representation of the timestamp as the interchange format as it seemed the most reliable way to represent the value and avoid timezone issues.
Also fixed minor bug in ResultSet where getInt() was returning a string.
As fyi, I'm using this module in a script to copy large amounts of data from 1 database over to another.  I'll send additional PRs if I need to make any more changes.  Thanks for writing a great module!

Hi ceyhunkerti, thanks.  You may want to regen the doc set as well. Do you need to do something special for the updates to appear on npm?

Hi there,
I've published to npm too. thanks for the reminder.
I'll have a look at the docs in detail tomorrow.

Hey, thanks again.  This is my first time contributing code to an open
source project - it's been an nice experience.
I just tried a fresh install from npm, and it seems to not match what's in
the repo master branch (the batch methods are there, but the new timestamp
changes aren't).  Did you publish an earlier commit to npm?
On Tuesday, March 8, 2016, ceyhunkerti<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:

Hi there,
I've published to npm too. thanks for the reminder.
I'll have a look at the docs in detail tomorrow.
—
Reply to this email directly or view it on GitHub
https://github.com/blue-color/nodejdbc/pull/3#issuecomment-193978901.

sorry my mistake, i've published the latest version now.

Perfect - everything looks good now.  Thanks for collaborating!
On Tuesday, March 8, 2016, ceyhunkerti<EMAIL_ADDRESS>wrote:

sorry my mistake, i've published the latest version now.
—
Reply to this email directly or view it on GitHub
https://github.com/blue-color/nodejdbc/pull/3#issuecomment-194018874.


Extract multi-search out of pg_search

I think that this gem does too much. Have you guys considered splitting pg_search into two gems? It seems that the multi-search functionality could be extracted to a separate gem, or even to a wiki page (since it seems to me that multi-searchable can be implemented in terms of pg_search_scope)

I agree. I've thought about replacing multisearch with a generator so that there's a way to do more than one multisearch, and so that it would be easier for people to modify the way it works. Naturally I would want to have a deprecation window where both are possible.


Update alpine to 3.8

For https://github.com/giantswarm/giantswarm/issues/4197

Hmm. We have some unsatisfiable constraints:
ERROR: unsatisfiable constraints:
  build-base-0.5-r1:
    breaks: world[build-base=0.5-r0]
  git-2.18.0-r0:
    breaks: world[git=2.15.2-r0]
  iptables-1.6.2-r0:
    breaks: world[iptables=1.6.1-r1]
  jq-1.6_rc1-r1:
    breaks: world[jq=1.5-r5]
  less-530-r0:
    breaks: world[less=520-r0]
  openssh-client-7.7_p1-r3:
    breaks: world[openssh-client=7.5_p1-r8]
  openssl-1.0.2o-r2:
    breaks: world[openssl=1.0.2o-r1]
  sudo-1.8.23-r2:
    breaks: world[sudo=1.8.21_p2-r1]

BTW, if somebody wants to take this over, please do.

Hmm. We have some unsatisfiable constraints:

You just need to upgrade to the given new versions, eg. build-base=0.5-r0 -> build-base=0.5-r1

@marians should be fixed with https://github.com/giantswarm/e2e-harness/pull/124/commits/142b3bb5d0c0abe78e5757f274f3490677c5881a

Cool, thanks!


Using fine-tuned model for inference

Hi @iusztinpaul,
Love the course so far!
I have a question: Shouldn't us use our fine-tuned model for inferenced instead of using Paul's peft model here?
https://github.com/iusztinpaul/hands-on-llms/blob/5c1887b4872df749e8863d2fb85f2f456b6af9ac/modules/training_pipeline/configs/dev_inference_config.yaml#L2
If yes then how should we publish our model from experiment to Comet Model Registry? Is it done manually via the Register Model button in the Comet experiment console view?

Thanks!

Hello,
Happy to hear that!
Yes, you got that right! I added my own version to help you test things out, but ideally, you should use your own version.
You should pick the fine-tuned model and register it manually, as you suggested in the SS.
Just be careful to add a name and version that make sense and change them in the yaml file. When using your own fine-tuned model, it will be something like:
yourname/your-model-name:your-version

Got it working.


demos: On Android, treat 1st-time new window different from nth-time.

This fixes an issue if cube ever got a new window, it would leak objects and grow large.

 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.

+1

Hey Ian, commit looks good, but please take a look at the commit message guidelines from the Vulkan Layer Implementation Guidelines and Features.  Thanks!

Hi @mark-lunarg.  That's a big document:-).  I'll change the commit ID before pushing.  BTW, there's a reference to "the Gerrit Automated Build system".  Is that something that you have at LunarG?

Cherry-picked and pushed to master

Yes, it's something that Dustin put together for pre-checking validation and code reviews.  If we can ever catch our breath, the idea is to move it beyond our firewall and share it with you guys.


Refactor radar scales to expose label position data

A common request for the radar chart type is the ability to locate the labels in order to interact (hover, click, etc). For example:
https://github.com/chartjs/Chart.js/issues/5344
https://github.com/chartjs/Chart.js/issues/3049
https://stackoverflow.com/questions/50470784/make-chart-js-radar-labels-clickable
https://stackoverflow.com/questions/37028924/how-to-add-onclick-event-on-labels-in-chart-js-v2-0
As suggested by @etimberg here a good solution would be for the library to expose label position data via a public API. This would make it possible for event handlers to get the data they need without having to duplicate the internal calculations.
I was able to hack together a solution for what I need by replicating parts of the core code:
https://jsfiddle.net/simoncoggins/7r08uLk9/
But worry about having to maintain it as chartjs moves forward. Therefore I’m considering writing a patch to:

Abstract the label positioning logic out of drawPointLabels() into a new method (say calculatePointLabelInfo())
Expose the data via a public API.
Update drawPointLabels() to use the new method when rendering the labels

The idea would be to provide an object something like this:
pointLabelInfo = [
    {
        labelText {
            x: 10,
            y: 10
            textAlign: ‘left’,
        },
        boundingBox {
            width: 75,
            height: 20,
            top: 200,
            bottom: 220,
            left: 300,
            right: 375,
        }
    }
    // repeat for other labels. Array indexes match scale.pointLabels
]

drawPointLabels() would make use of the labelText, while event handlers would be more likely to be interested in the boundingBox data, but they would be calculated together so consistent with each other.
If accepted it would provide a step towards support for label event handlers, and a similar solution could be applied to other scale types.
Before I start just wanted to check if that patch would be likely to be considered for integration?

Would love to have this feature. Had to hack my own solution to detect click on horizontalBar labels by passing a fake event object to getElementAtEvent() with an altered clientY value.


🌱 Improve ipv6 address in bound check performance in ippool webhook

What this PR does / why we need it:
Cherry-pick form main to get the fix into release 1.6
The current mechanism of checking if an ip address is within the bound of a given ippool loops through the domain of all ip addresses in the ip pool one by one. A subnet in ipv6 can easily be a very large cidr that involves millions or billions of addresses. The validation webhook could time out (default 10 seconds) before the check is complete, if the ip address being checked is at the end of the cidr range or even out of the range at all.
Which issue(s) this PR fixes (optional, in fixes #(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes https://github.com/metal3-io/ip-address-manager/issues/430

/ok-to-test

/test-ubuntu-e2e-integration-release-1-6

/cc @kashifest @lentzi90


Added Women in Video Games Database

🛠 Fixes Issue

👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

Good one @priyam99
Congrats 🎉, your PR got merged. To be eligible for the swags from ClueLess. Follow these steps.

Star this repository. ⭐
Follow Clueless on GitHub
SignIn / SignUp in [ClueLess Website](https://www.clueless.tech/) .
Go to the profile section from the top right-hand corner.

Comment your profile URL under this comment tagging the person who merged your PR.

That’s it. Follow the leaderboard!


⚠ Bofrost.de has degraded performance

In b2a24c1, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1400 ms

Resolved: Bofrost.de performance has improved in 7cecbca.


Unable to destroy resources created with terraform when changing ownership

Provider Version
snowflake = {
source = "Snowflake-Labs/snowflake"
version = "0.35.0"
}
Terraform Version
Terraform v1.2.2 on linux_amd64
Describe the bug
When executing terraform destroy, I get the following error :
Error: error dropping database MY_DB: 002003 (02000): SQL compilation error: Database 'MY_DB' does not exist or not authorized.
I create the database 'MY_DB' and the role 'MY_DB_DBA' then I grant the privilege 'OWNERSHIP' of the database to 'MY_DB_DBA'. Everything is OK with the terraform apply. But when I want to delete the ressources by executing 'terraform destroy', I get the error.
To grant the ownership of the database, I use the "SECURITYADMIN" role (Only "ACCOUNTADMIN" AND "SECURITYADMIN" can do it). But the revert action set the ownership to "SECURITYADMIN" instead of "SYSADMIN"  and consequently, the "SYSADMIN" used to create the database cannot execute the DROP DATABASE.
Expected behavior
All resources created would be deleted.
Code samples and commands
terraform {
required_providers {
snowflake = {
source = "Snowflake-Labs/snowflake"
version = "0.35.0"
}
}
}
provider "snowflake" {
alias = "account_admin"
role  = "ACCOUNTADMIN"
}
provider "snowflake" {
alias = "sys_admin"
role  = "SYSADMIN"
}
provider "snowflake" {
alias = "security_admin"
role  = "SECURITYADMIN"
}
resource "snowflake_role" "my_db_dba" {
provider = snowflake.security_admin
name     = "MY_DB_DBA"
}
resource "snowflake_role_grants" "grant_my_db_dba" {
provider               = snowflake.security_admin
role_name              = "${snowflake_role.my_db_dba.name}"
roles                  = ["SYSADMIN"]
}
resource "snowflake_database" "my_db" {
provider = snowflake.sys_admin
name     = "MY_DB"
}
resource "snowflake_database_grant" "grant_database_ownership" {
provider          = snowflake.security_admin
database_name     = "${snowflake_database.my_db.name}"
privilege         = "OWNERSHIP"
roles             = ["${snowflake_role.my_db_dba.name}"]
with_grant_option = true
}

I'm facing exactly the same issue. How can we modify the fact that "revert action set the ownership to "SECURITYADMIN". It is really blocker for my project, who can help? Perhaps with a workaround at least?

I tested again with the last release 0.47.0. Same problem than with the 0.35.0 that @ZigZag59  used:

I created a Snowflake database for example with terraform
This database is owned by SYSADMIN
If I launch a destroy, the provider Snowflake-Labs change the owner to ACCOUNTADMIN. Terraform can't destroy this ressource with SYSADMIN as now it is owned by ACCOUNTADMIN

It is really a blocker bug, is it possible to work on it asap?
If needed, I can provide example to reproduce it easily.
Pieces of code used:
terraform {
required_providers {
snowflake = {
source = "Snowflake-Labs/snowflake"
version = "0.47.0"
}
}
resource "snowflake_database" "db_src" {
count = var.no_of_users
provider = snowflake.sys_admin
name     = join("", [var.snowflake_db_prefix, data.aws_caller_identity.current.account_id, format("USER%02d", count.index + 1)])
data_retention_time_in_days = 0

}
resource "snowflake_database_grant" "database_grant_ownership" {
count             = var.no_of_users
provider          = snowflake.account_admin
database_name     = snowflake_database.db_src[count.index].name
privilege         = "OWNERSHIP"
roles             = ["SYSADMIN"]
with_grant_option = true
depends_on    = [snowflake_database.db_src]

}
After deploy, all is good, owner is SYSADMIN as expected:

Then during destroy, the provider change it for ACCOUNTADMIN and I encounter error for destroy as:

Because the database can't be accessed with SYSADMIN now.


Models with same index but different document type

Hello,
I have a few models that all belong to the same index but they have different document type. My plan was to create a base model, which includes all the ES attributes and inherit from that and make something like this:
class Base
  index_name 'my_index'

  attribute :title,  String, mapping: {fields: {raw: {type: 'string', index: 'not_analyzed'}}}
  attribute :name, String
end

class Foo < Base
  document_type 'foo'
end

The above unfortunately will not inherit  the ES attributes of the base model. Is there a workaround for this? I do not want to copy paste all attributes of the Base to all my models.

I think this might be related with https://github.com/elasticsearch/elasticsearch-rails/issues/279

I used a base module and it seems that it's working properly now. If there is a better way please let me know.
And while we are on the subject, how do I push the mappings for each model to the same index? Model.create_index! will only overwrite the ones I have.

@alolis Have a look here, https://github.com/elasticsearch/elasticsearch-rails/blob/master/elasticsearch-persistence/examples/music/index_manager.rb and the corresponding example Rails application.

Thanks @karmi


v3 and light theme

I like react-alert.
But where is light theme??? The dark theme is too gloomy....

You can create your own templates for alerts now! You can use the dark one as example of how to build your own.
I would happily accept a PR to include the link in the README.


Inconsistency for json objects

envelopeName
recipientName
senderName
username
email
password1
password2
profilepic
handle
imageId
url
filename
action
dnum
token
success
error

@kristojorg

Ok this is what it will be in the end?

Yes I'll make sure I ask for these and return these from backend. Thank you


'cargo ios/app' might be much better than 'cargo xcodebuild'

Hello there!
I just wanted to share a thought with you:
I've been searching for a tool just like this one for quite some time now.
Essentially an cargo apk but for ios.
But I never found this project due to its 'weird' naming.
I always searched for "cargo ios" and similar tools and only by pure chance did I found a blog article talking about this tool while trying to figure out how to use xcodegen  with Rust.
My suggestion would be to rename this tool to cargo ios (following the same schema as cargo apk) or maybe cargo app. This would make it much more clearer what this project is about (building iOS apps with Rust!), without needing to know much about xcodegen.
Furthermore, it would greatly improve the search index.
If wanted/required I can assist and contribute as well :)
Have a nice day!

Hi, thank you! The name cargo xcodebuild was chosen because xcodebuild is the main tool for building iOS/MacOS apps. It's a part of Xcode's bundled command-line tools package.