Metadata is not generated correctly when pushing a big object index

Summary
When doing sfdx force:source:push to deploy a project that contains a big object index, the metadata that gets generated is not correct.
However this only seems to be happening in our CI instance, that is using Ubuntu 20.04.4 LTS (Focal Fossa).
To illustrate further, when I set up the SFDX_MDAPI_TEMP_DIR environment variable to debug the metadata that gets generated, this is how the package.xml looks like:

As you can see there is an entry created for an EmailServicesFunction with the name of the big object index, as well as the actual big object index. Just for additional context, we do not have any EmailServicesFunction defined in our project, so we wouldn't expect any to be defined in our definition file.
Navigating to the metadata component that is generated for the EmailServicesFunction:
circleci@08572840bdbb:~/metadataPackage_1652479702973$ ls -l
total 172
drwxr-xr-x   2 circleci circleci     74 May 13 22:08 applications
drwxr-xr-x   7 circleci circleci    187 May 13 22:08 aura
drwxr-xr-x   2 circleci circleci     71 May 13 22:08 cachePartitions
drwxr-xr-x   2 circleci circleci  32768 May 13 22:08 classes
drwxr-xr-x   2 circleci circleci     93 May 13 22:08 components
drwxr-xr-x   2 circleci circleci    112 May 13 22:08 contentassets
drwxr-xr-x   2 circleci circleci   4096 May 13 22:08 customMetadata
drwxr-xr-x   2 circleci circleci    208 May 13 22:08 customPermissions
drwxr-xr-x   2 circleci circleci     48 May 13 22:19 emailservices
drwxr-xr-x   2 circleci circleci    285 May 13 22:08 flexipages
drwxr-xr-x   2 circleci circleci     46 May 13 22:08 flows
drwxr-xr-x   2 circleci circleci     46 May 13 22:08 globalValueSets
drwxr-xr-x   2 circleci circleci   4096 May 13 22:08 layouts
drwxr-xr-x 107 circleci circleci   4096 May 13 22:08 lwc
drwxr-xr-x   2 circleci circleci     57 May 13 22:08 notificationtypes
-rw-r--r--   1 circleci circleci 103760 May 13 22:08 package.xml
drwxr-xr-x   2 circleci circleci   4096 May 13 22:08 pages
drwxr-xr-x   2 circleci circleci    158 May 13 22:08 permissionsets
circleci@08572840bdbb:~/metadataPackage_1652479702973$ cd emailservices/
circleci@08572840bdbb:~/metadataPackage_1652479702973/emailservices$ ls -l
total 4
-rw-r--r-- 1 circleci circleci 746 May 13 22:08 CM_Document_Audit_Log_Index.indexe
circleci@08572840bdbb:~/metadataPackage_1652479702973/emailservices$ cat CM_Document_Audit_Log_Index.indexe
<?xml version="1.0" encoding="UTF-8"?>
<Index xmlns="http://soap.sforce.com/2006/04/metadata">
    <fullName>CM_Document_Audit_Log_Index</fullName>
    <fields>
        <name>CM_Document__c</name>
        <sortDirection>ASC</sortDirection>
    </fields>
    <fields>
        <name>CM_Event_Type__c</name>
        <sortDirection>ASC</sortDirection>
    </fields>
    <fields>
        <name>CM_Date__c</name>
        <sortDirection>DESC</sortDirection>
    </fields>
    <fields>
        <name>CM_Related_Record_Id__c</name>
        <sortDirection>ASC</sortDirection>
    </fields>
    <fields>
        <name>CM_Performed_By_Id__c</name>
        <sortDirection>ASC</sortDirection>
    </fields>
    <label>Document Audit Log Index</label>
</Index>

It looks like somewhere along the process sfdx force:source:push is mistakenly parsing the original index as an EmailServicesFunction.
Steps To Reproduce:
Repository to reproduce: N/A (private)

Deploy the project with a big object index using sfdx force:source:push.

Expected result
The entire project, including the big object index, gets deployed.
Actual result
The deployment fails since a EmailServicesFunction component that does not exist gets added to the package.xml.
System Information

sfdx

{
        "cliVersion": "sfdx-cli/7.150.0",
        "architecture": "linux-x64",
        "nodeVersion": "node-v14.19.2",
        "pluginVersions": [
                "@oclif/plugin-autocomplete 0.3.0 (core)",
                "@oclif/plugin-commands 1.3.0 (core)",
                "@oclif/plugin-help 3.3.1 (core)",
                "@oclif/plugin-not-found 1.2.6 (core)",
                "@oclif/plugin-plugins 1.10.11 (core)",
                "@oclif/plugin-update 1.5.0 (core)",
                "@oclif/plugin-warn-if-update-available 1.7.3 (core)",
                "@oclif/plugin-which 1.0.4 (core)",
                "@salesforce/sfdx-plugin-lwc-test 0.1.7 (core)",
                "alias 2.0.0 (core)",
                "apex 0.11.0 (core)",
                "auth 2.0.2 (core)",
                "community 1.1.4 (core)",
                "config 1.4.6 (core)",
                "custom-metadata 1.1.0 (core)",
                "data 0.6.15 (core)",
                "generator 2.0.0 (core)",
                "info 2.0.0 (core)",
                "limits 2.0.0 (core)",
                "org 1.12.1 (core)",
                "salesforce-alm 54.3.0 (core)",
                "schema 2.1.0 (core)",
                "sfdx-cli 7.150.0 (core)",
                "signups 1.0.0 (core)",
                "source 1.9.7 (core)",
                "telemetry 1.4.0 (core)",
                "templates 54.6.0 (core)",
                "trust 1.1.0 (core)",
                "user 1.7.1 (core)"
        ],
        "osVersion": "Linux 5.13.0-1021-aws"
}

Additional information
Running sfdx force:source:convert and sfdx force:mdapi:deploy works as expected and does not generate the EmailServicesFunction.

I ran into the same issue and spend a significant number of hours to track this down. I hope the following details will help to identify and fix the root cause for this behaviour:
The error first occurred in my build server (CircleCI using Node LTS container).
Steps To Reproduce:
Repository to reproduce: https://github.com/j-fischer/rflib

Check out the project
Create new scratch org
run sfdx force:source:push with a version 7.146 or later

I was also able to reproduce the issue locally (Windows - sfdx v7.150.0) after I ran sfdx force:source:tracking:clear and then created a new scratch org and started pushing the source.
@cdelmoral, this might be helpful to you: I was able to resolve the issue by reverting the CLI version on my build server back to 7.142.0. Just making that switch and no other changes fixed the problem. Thanks for posting the detailed description of the problem here.

Thank you for the detail, helped us after many days of struggling with this.

Is there a target version for the fix?

Just thought I'd check on this item, as it is preventing our team from upgrading past 7.143 as well.

@j-fischer I have been trying to reproduce this error using 7.143.o, with no avail. May I ask a question regarding the MD file name of the big index. Is the .index portion of the name .index. or .indexe.? If it is the latter, could you try renaming it to .index. and the try the push again using 7.143.0?

@peternhale I was not able to reproduce the issues in 7.143.0, but I was able to reproduce it with the new push implementation in 7.149.0.
Checking the name of the index, it used the .indexe suffix. I renamed it as you suggested and tried again, which fixed the issue.
 So, renaming the file name did the trick. Thanks for the suggestion.

@j-fischer thanks for getting back to me and I am glad the renaming workaround got you past the error.

I will look at this in the context of creating a new package version.

Thanks for checking the package creation, @cdelmoral , I would certainly be running into this issue in the future. And thanks for continuing to look into this, @peternhale .

Hi @peternhale
Same issue here. only renaming indexe to index in the BO metadata, allows a successful push to the scratch
If i try to restore to original indexe file name, push error appears again.
i need to generate a new version package, but not sure in which version to do, because of previous comments
monitor git:(feature/CHN4-9024-flushThroughApi) ✗ sfdx force:source:status
Source Status
IGNORED STATE         FULL NAME                   TYPE                  PROJECT PATH
─────── ───────────── ─────────────────────────── ───────────────────── ────────────────────────────────────────────────────────────────────────────
false   Local Add     UUid                        EmailServicesFunction force-app/main/default/objects/AteneaEventBO__b/indexes/UUid.indexe-meta.xml
=== Component Failures [2]
Type  Name                  Problem
───── ───────────────────── ──────────────────────────────────────────────────────────────────────────────────────────────────────────────
Error AteneaEventBO__b.UUid DELETE isn't supported for Big Objects indexes: UUid
Error UUid                  An object 'UUid' of type EmailServicesFunction was named in package.xml, but was not found in zipped directory
this is my current sfdx version:
➜  monitor git:(feature/CHN4-9024-flushThroughApi) ✗  sfdx version --verbose
CLI Version:
sfdx-cli/7.167.2
Architecture:
darwin-x64
Node Version:
node-v16.17.0
Plugin Version:
@oclif/plugin-autocomplete 1.3.0 (core)
@oclif/plugin-commands 2.2.0 (core)
@oclif/plugin-help 5.1.12 (core)
@oclif/plugin-not-found 2.3.1 (core)
@oclif/plugin-plugins 2.1.0 (core)
@oclif/plugin-update 3.0.0 (core)
@oclif/plugin-version 1.1.2 (core)
@oclif/plugin-warn-if-update-available 2.0.4 (core)
@oclif/plugin-which 2.1.0 (core)
alias 2.1.0 (core)
apex 1.2.0 (core)
auth 2.2.3 (core)
community 2.0.1 (core)
config 1.4.19 (core)
custom-metadata 2.0.0 (core)
data 2.1.2 (core)
generator 2.0.2 (core)
info 2.0.1 (core)
limits 2.0.1 (core)
org 2.2.0 (core)
packaging 1.6.0 (core)
schema 2.1.1 (core)
signups 1.2.0 (core)
source 2.0.13 (core)
telemetry 2.0.0 (core)
templates 55.1.0 (core)
trust 2.0.3 (core)
user 2.1.0 (core)
@salesforce/sfdx-plugin-lwc-test 1.0.1 (core)
salesforce-alm 54.8.1 (core)
sfdx-mohanc-plugins 0.0.202 (user)
OS and Version:
Darwin 21.6.0
Shell:
zsh
Root Path:
/Users/clemente.sanchez/.local/share/sfdx/client/7.167.2-93d663e
Best Regards
Clemente

@peternhale any update on this, I would like to publish a new version of my package but the command fails due to this error.
ERROR running force:package:version:create:  Unexpected file found in package directory: C:\Users\Johannes\Projects\Salesforce\rflib\rflib\main\default\objects\rflib_Logs_Archive__b\indexes\rflib_Log_Index.index-meta.xml
Warning: Command failed: sfdx force:package:version:create --path rflib --package RFLIB --installationkeybypass -c --wait 30
ERROR running force:package:version:create:  Unexpected file found in package directory: C:\Users\Johannes\Projects\Salesforce\rflib\rflib\main\default\objects\rflib_Logs_Archive__b\indexes\rflib_Log_Index.index-meta.xml

Is there any ETA when we can expect this to be fixed?
Thanks,
Johannes

hey @j-fischer sorry for the radio silence, can you try your packaging workflows using the beta packaging commands (force:package(1):beta:*)? The beta commands are using our standardized library and should be able to handle the MD edge cases much better

@WillieRuemmele Thanks, will give beta commands a try and report back during my next packaging cycle.
Cheers

@WillieRuemmele I finally managed to find the time to work on another version of my library creating another package version. Using the beta commands (2nd gen packaging), I was able to publish the big object without having to rename the index file name.
Looks like the beta commands are the way to go. Thanks for the help and apologies that it took that long to confirm.
Cheers

Hey is there any update for the original issue? I am still getting the problem with push throwing the EmailServicesFunction error because of our big index objects.
Error Index An object 'Index' of type EmailServicesFunction was named in package.xml, but was not found in zipped directory
I have tried the workaround with renaming indexe to index and that has fixed the push command to our local scratch orgs but it has caused issues down the line into our sandbox orgs with a deploy.
ERROR running force:source:deploy:  Unexpected file found in package directory: <directory>/<big-object>/indexes/Index.index-meta.xml.
Here is the output of sfdx doctor
{
  "versionDetail": {
    "cliVersion": "sfdx-cli/7.182.1",
    "architecture": "darwin-x64",
    "nodeVersion": "node-v18.12.1",
    "pluginVersions": [
      "@oclif/plugin-autocomplete 1.3.6 (core)",
      "@oclif/plugin-commands 2.2.1 (core)",
      "@oclif/plugin-help 5.1.19 (core)",
      "@oclif/plugin-not-found 2.3.9 (core)",
      "@oclif/plugin-plugins 2.1.8 (core)",
      "@oclif/plugin-search 0.0.4 (core)",
      "@oclif/plugin-update 3.0.7 (core)",
      "@oclif/plugin-version 1.1.3 (core)",
      "@oclif/plugin-warn-if-update-available 2.0.15 (core)",
      "@oclif/plugin-which 2.2.2 (core)",
      "alias 2.1.12 (core)",
      "apex 1.3.0 (core)",
      "auth 2.3.3 (core)",
      "community 2.0.18 (core)",
      "config 1.4.22 (core)",
      "custom-metadata 2.0.5 (core)",
      "data 2.1.17 (core)",
      "generator 2.0.13 (core)",
      "info 2.2.13 (core)",
      "limits 2.1.3 (core)",
      "org 2.2.19 (core)",
      "packaging 1.10.10 (core)",
      "schema 2.1.19 (core)",
      "signups 1.2.12 (core)",
      "source 2.3.8 (core)",
      "telemetry 2.0.5 (core)",
      "templates 55.1.0 (core)",
      "trust 2.2.2 (core)",
      "user 2.1.22 (core)",
      "@salesforce/sfdx-plugin-lwc-test 1.0.1 (core)",
      "@salesforce/sfdx-scanner 3.7.1 (user)",
      "isvte-sfdx-plugin 1.1.16 (user)",
      "salesforce-alm 54.8.4 (core)",
      "texei-sfdx-plugin 1.17.1 (user)"
    ],
    "osVersion": "Darwin 22.2.0",
    "shell": "zsh",
    "rootPath": "/Users/greg/.local/share/sfdx/client/7.182.1-dad7ae3"
  },
  "sfdxEnvVars": [
    [
      "SFDX_REDIRECTED",
      "1"
    ],
    [
      "SFDX_BINPATH",
      "/Users/greg/.local/share/sfdx/client/bin/sfdx"
    ],
    [
      "SFDX_AUTOUPDATE_DISABLE",
      "true"
    ],
    [
      "SFDX_DISABLE_AUTOUPDATE",
      "true"
    ],
    [
      "SFDX_UPDATE_INSTRUCTIONS",
      "Use \"npm update --global sfdx-cli\" to update npm-based installations."
    ]
  ],
  "sfEnvVars": [],
  "cliConfig": {
    "_base": "<EMAIL_ADDRESS>    "debug": 0,
    "topicSeparator": " ",
    "warned": false,
    "commandPermutations": {
      "validPermutations": {}
    },
    "topicPermutations": {
      "validPermutations": {}
    },
    "_commands": {},
    "_topics": {},
    "root": "/Users/greg/.local/share/sfdx/client/7.182.1-dad7ae3",
    "name": "sfdx-cli",
    "version": "7.182.1",
    "channel": "stable",
    "valid": true,
    "arch": "x64",
    "platform": "darwin",
    "windows": false,
    "bin": "sfdx",
    "dirname": "sfdx",
    "flexibleTaxonomy": true,
    "userAgent": "sfdx-cli/7.182.1 darwin-x64 node-v18.12.1",
    "shell": "zsh",
    "home": "/Users/greg",
    "cacheDir": "/Users/greg/Library/Caches/sfdx",
    "configDir": "/Users/greg/.config/sfdx",
    "dataDir": "/Users/greg/.local/share/sfdx",
    "errlog": "/Users/greg/Library/Caches/sfdx/error.log",
    "binPath": "/Users/greg/.local/share/sfdx/client/bin/sfdx",
    "nodeEngine": ">=14.0.0"
  },
  "pluginSpecificData": {
    "@salesforce/plugin-source": [
      {
        "sourceApiVersion": "56.0",
        "targetDevHubApiVersion": "56.0",
        "targetOrgApiVersion": "56.0"
      }
    ]
  },
  "diagnosticResults": [
    {
      "testName": "salesforcedx plugin not installed",
      "status": "pass"
    },
    {
      "testName": "no linked plugins",
      "status": "pass"
    },
    {
      "testName": "[@salesforce/plugin-source] sourceApiVersion matches apiVersion",
      "status": "pass"
    },
    {
      "testName": "[@salesforce/plugin-source] default target DevHub max apiVersion matches default target org max apiVersion",
      "status": "pass"
    },
    {
      "testName": "[@salesforce/plugin-source] sourceApiVersion matches default target org max apiVersion",
      "status": "pass"
    },
    {
      "testName": "using latest or latest-rc CLI version",
      "status": "pass"
    }
  ],
  "suggestions": [
    "Check https://github.com/forcedotcom/cli/issues for CLI issues posted by the community.",
    "Check http://status.salesforce.com for general Salesforce availability and performance."
  ],
  "logFilePaths": []
}

Thanks,
Greg

Hi @gilgrencho @gstephens123 and others...
I was able to partially recreate the issue, but not identically
 ➜  oc
Successfully created scratch org: 00D8G000000HbwyUAC, username<EMAIL_ADDRESS>➜  RGD  hub:(GLOBAL - DevHub<EMAIL_ADDRESS> ➜  status
Source Status
 STATE     FULL NAME TYPE                        PROJECT PATH
 ───────── ───────── ─────────────────────────── ──────────────────────────────────────────────────────────────────────────────────────────────────
 Local Add test__b   CustomObject                force-app/main/default/objects/test__b/fields/field__c.field-meta.xml
 Local Add test__b   CustomObject                force-app/main/default/objects/test__b/indexes/BOI.index-meta.xml
 Local Add test__b   CustomObject                force-app/main/default/objects/test__b/test__b.object-meta.xml
 Local Add Admin     Profile                     force-app/main/default/profiles/Admin.profile-meta.xml
 Local Add New_Graph RelationshipGraphDefinition force-app/main/default/relationshipGraphDefinitions/New_Graph.relationshipGraphDefinition-meta.xml
➜  RGD  hub:(GLOBAL - DevHub<EMAIL_ADDRESS> ➜  push
Pushing v55.0 metadata to<EMAIL_ADDRESS>using the v56.0 SOAP API
DEPLOY PROGRESS | ████████████████████████████████████████ | 5/5 Components
Updating source tracking... done

=== Pushed Source

 STATE   FULL NAME        TYPE                        PROJECT PATH
 ─────── ──────────────── ─────────────────────────── ──────────────────────────────────────────────────────────────────────────────────────────────────
 Created test__b.field__c CustomField                 force-app/main/default/objects/test__b/fields/field__c.field-meta.xml
 Created test__b.field__c CustomField                 force-app/main/default/objects/test__b/fields/field__c.field-meta.xml
 Created test__b          CustomObject                force-app/main/default/objects/test__b/test__b.object-meta.xml
 Created New_Graph        RelationshipGraphDefinition force-app/main/default/relationshipGraphDefinitions/New_Graph.relationshipGraphDefinition-meta.xml

=== Component Failures [1]

 Type  Name        Problem
 ───── ─────────── ─────────────────────────────────────────────────────────────────────────────────────────────────────
 Error test__b.BOI An object 'test__b.BOI' of type Index was named in package.xml, but was not found in zipped directory

I created a scratch org, and created the big object, added a custom field, and added an index on that object, I pulled that down and everything succeeded. I then created a new scratch org, and when I pushed, I saw the above error, but it's not matching your's. What CLI versions are you experiencing this on? I'm on 7.184.2

@WillieRuemmele, i guess we are running the same cli version: sfdx-cli/7.184.2 darwin-x64 node-v18.12.1
let-me know what details you might need, my big object has 10 fields and 1 index

Thanks for helping @gilgrencho - I've seen multiple people report that changing .indexe to .index seems to fix the problem, I'm just stumped why when you pull it it down it's written as .indexe, mine was written as .index...

Maybe try moving that Big Object to a temporary directory outside of the project, and running the pull again and seeing if it regenerates it in a new format, if it doesn't work you can always restore the BO from that saved location

@WillieRuemmele sorry for late reply, i wasn't able to sort the metadata with push/pull. Manually remove the extra e ended up working.
Thank you so much!

The code that was producing the extra e no longer exists.  If you have a very old project with indexe use the workarounds above to be able to deploy.


Improve terraform examples

@ruzickap thank you for yours pr, but unfortunately we can't merge it without building the documentation. we will add yours example to a new pr with other fixes.


Limit climate device actions

Description:
A device action to set the preset_mode should only be available, if presets are used by the integration.
I used the same mechanism as used by device_condition and not the supported features flag.
Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass
[x] There is no commented out code in this PR.
[x] I have followed the development checklist

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

Hi @MartinHjelmare.
Maybe this PR is also something for the 0.102.x milestone.

Hi @balloob,
i still think this is a valid PR for a minor release to limit the number of device actions to the possible ones.
BR


Use in combination with SoundSwitch

I last tried to use my light control and Unbox at the same time, unfortunately it doesn't work.
Unbox picks up the data first and therefore disrupts the lighting control.
When I quit Unbox it works, when I start it again the error reappears.
Is this bug known, will it be fixed in the future or is it not possible to use both at the same time?
Thanks for the great tool!

Yeah this is a known bug, and unfortunately there isn't a fix at the moment. I'd love to test some more once I can get my hands on denon gear and SoundSwitch, I'll be sure to update you here if I'm able to test this some more!


[FileConnector] Provide a acknowledgment timeout warning in console

Currently when acknowledgment timeout happens, no warning is shown in the console. Please provide a warning so that the user is aware of the timeout.

The component which waits for acknowledgment prints a warn log, but it is not getting printed on the Ballerina console, as Ballerina hides those logs.
This will be fixed in a later Ballerina release.

Created https://github.com/ballerinalang/connector-file/issues/6 to track this


Initial conversion of documentation to sphinx+breathe for readthedocs - [merged]

In GitLab by @perarnau on Apr 26, 2019, 11:23
Merges sphinx-breathe -> master
Fix #36.

In GitLab by @perarnau on Apr 26, 2019, 13:41
added 1 commit
7cc0a591 - [fix] missing doxygen files, naming issues.
Compare with previous version
By Swann Perarnau on 2019-04-26T18:41:25 (imported from GitLab project)

In GitLab by @perarnau on Apr 26, 2019, 14:01
added 2 commits
2c927865 - [doc] convert documentation to sphinx+breathe1eeaeed0 - [fix] remove declaration of removed functions
Compare with previous version
By Swann Perarnau on 2019-04-26T19:01:09 (imported from GitLab project)

In GitLab by @perarnau on Apr 26, 2019, 14:07
@NicolasDenoyelle can you have a look ? It builds for me and should match the doxygen version. I'll merge and fix RTD if needed later.
By Swann Perarnau on 2019-04-26T19:07:11 (imported from GitLab project)

In GitLab by @perarnau on May 1, 2019, 09:30
added 1 commit
e681f6f7 - [doc] working makefiles, configure
Compare with previous version

In GitLab by @perarnau on May 1, 2019, 09:51
added 1 commit
184e9b21 - [doc] working makefiles, configure
Compare with previous version

In GitLab by @perarnau on May 1, 2019, 10:03
added 1 commit
b2322b69 - [doc] working makefiles, configure
Compare with previous version

In GitLab by @perarnau on May 1, 2019, 10:14
added 1 commit
6b9687d2 - [doc] working makefiles, configure
Compare with previous version

In GitLab by @perarnau on May 1, 2019, 11:02
changed the description

In GitLab by @perarnau on May 1, 2019, 11:03
merged

In GitLab by @perarnau on May 1, 2019, 11:03
mentioned in commit b185c9aafe763eb9a9e15cde34172ced482f71b3


Newline in message prevent tag keyword detection

Describe the bug
A message including a newline character will not trigger the keyword recognition and in turn not display the associated tag
To Reproduce
Create a tag, eg /tag create test 'this is a test'
Try to trigger tag with message such as
this
is a test

Expected behavior
The example above should trigger the bot to repeated the tag associated with the test keyword.

Fixed in de180e3745c1d1c50203e3c5afae8189c861b239


Set message text color on mobile

Fixes android font color which was being overridden to undefined in text.native.js via this.props.style.
Before:

After:

The ...this.props.style is overriding color to undefined in text.native.js:
https://github.com/keybase/client/blob/master/shared/common-adapters/text.native.js#L27
@chromakode Was there a reason we are setting undefined there? Are we trying to use the default font color on mobile? If so on Android this is too light.
This PR fixes but not sure if it's the appropriate fix and if other problems exist with other styles.
Should we remove the setting of undefined values for style in markdown.native.js?

@keybase/react-hackers

The commits e8b9c572a393...9236bb64e7a0 introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)

Re-applies this fix: https://github.com/keybase/client/commit/0745ee2a that was lost somewhere

@chromakode I added a warning if color is undefined, I think this is ok...

The commits e8b9c572a393...2988493ec4f1 introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)

The commits e8b9c572a393...4816ed9d42a6 introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)

The commits e8b9c572a393...c1b801b2d63a introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)

The commits e8b9c572a393...b15c340a6c9b introduce visual changes on linux. :mag_right: 1 changed

Changed: Tracker-New_user_Scroll_1 (before) (after)


Evaluation

I was thinking about replacing our docs with something more pleasant to use than raw HTML and stumbled upon VuePress. The docs are in Markdown (though you can use YAML, HTML and even Vue in it) and everything else is handled by a few config files. It supports themes (the default one can pretty much only change colors but since the docs are in Markdown switching themes should be quite painless), search and internationalization (we have Korean docs for Vis Network for those who don't know). Plus there are many plugins with additional features.
I played a bit with it and created a working example:
Code: https://github.com/visjs/experiment-vuepress/tree/master/vuepress
Result: https://visjs.github.io/experiment-vuepress/documentation/
What do you think?

:astonished: AMAZING!!!

WOW!! I LIKE!

Here are a few thoughts on this topic:

I realy like the look and feel of vuepress.
The documentation of each project should be done in each project, so that they can go their own way (if the want/need to).
The documentation would best be done in the code (e.g. tsdocs) and than generated from the code.
Additional documentation should be done using markdown instead of html.
It would be desirable that all projects use a similar/same style.


AppLaunch/Cold is suspiciosly big

Description
I receive AppLaunch/Cold stats of 18-20 seconds. I also have tried Sentry (not at the same time with the NR agent). From Sentry I receive values that do not exceed 2 seconds. I'm not sure whether this is a bug or Sentry just calculates cold launch time differently.
Here are cold launch times I receive from the NR agent:
21.7, 21.35, 29.6, 39.2, 5.52, 55.5 (in seconds)
I wonder what is the finish point of the cold launch timer? Could some background process within the app affect this? E.g. the app screen may be loaded, and user could be able to use the app, but some stuck background request which affects the cold launch time
(warm launch times are ok, and all are less than 0.5sec)

Thank you for reporting this. We've created a ticket and will look into this in a future sprint.
Can you lease contact New Relic support and provide details?
Can you please reply to this with a link to a sample app demonstrating this issue?


ff cleanup: merkle_conflict_duplicate_proofs

Feature has been activated on all clusters:
mrkPjRg79B2oK2ZLgd7S3AfEJaX9B6gAF3H9aEykRUS  | active since epoch 657  | 283824004       | generate duplicate proofs for merkle root conflicts #34270

Fixes https://github.com/solana-labs/solana/issues/34641

This is both removing the check for feature activation and also moving some code around.
I would rather we don't mix these 2, and have a different PR to move blockstore.store_duplicate_slot somewhere else.

This is both removing the check for feature activation and also moving some code around. I would rather we don't mix these 2, and have a different PR to move blockstore.store_duplicate_slot somewhere else.

Fair, I'll address store_duplicate_slot in a follow up PR.


edit readme fixed : #99

Description

Category (If Applicable)

[x] Introduction to Open Source
[ ] Learn Git and GitHub
[ ] Contributing to Open Source
[ ] Beginner-Friendly Repos
[ ] Open Source Programs

Related Issue

Fixes #XX
Checklist

[ ] I have gone through the CONTRIBUTING guide
[ ] The name of the resource is correctly spelled (if applicable)
[ ] The link to the resource is working (if applicable)
[ ] The resource is added in the correct format (if applicable)

Edited readme file

is this ok ? i think it is ready to merge


Add SH_STACK array and tooling

Consider adding an array that sits in the environment scope, something like SH_STACK that can be added to, rotated etc and dumped on error

This has been implemented over time in init.sh and has been largely implemented within import(), see e.g. commit 742d3f0


UI: Click save button again, logging config disappear

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):

Select the fluentd target and enable project logging.
Click save button again without any change.

Result:

Logging config disappear.
Other details that may be helpful:
Environment information
master ( 2019/07/10 )

I wasn't able to reproduce in v2.2-head, or v2.3.0-alpha5; I think at this point we may have lost the ability to reproduce this as I don't have older master. I also reviewed the changes and they look good to me.
Validation:
master-head (7/11/19) (066fcfcc)

At project level, enable Fluentd logging on a fresh cluster (click save once).
Click Save button again. Logging config remains (no longer disappears).


[DOCFIX] Update master-configuration.yml of the Chinese version

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/13330/
Test PASSed.


Layers Trainable in Extraction and Evaluation code

Hello, I want to discuss about the following code you have included in 07.evaluate_CNN_LSTM.py and 05.lstm_features.py
since you are extracting and classifying what is the expected use of training all layers (finetuning) since the weights loaded are already finetuned in cnn file.
for layer in baseModel.layers:
layer.trainable = True
Please guide Thank you.

In extraction and classification, there's no need to keeps basemodel layers trainable as true. Either way, it doesn't have any effect, we are not doing any backprop and updating the weights of the model. We are just loading weights and doing a forward pass. You can pass False. I don't think it's gonna change your performance accuracy.
Since it's not gonna have any effect, that's why I just copied the architecture from training to evaluation.

Thanks for the clarrification. Have you tried passing features from cnn to rnn using global avg pooling instead of FC1 layer for the face forensics++ dataset. I will try to use this approach later on my datset as well I hope it doesn't make any major difference on the accuracy results.

I haven't tried that. You can print out a summary for both of your approaches and compare the number of parameters. It may depend on this. I don't think it's gonna have a major impact on the results though.


unable to ssh into boot2docker

Hi,
sorry if this has been explained somewhere else, but i couldn't find any documentation on this...
running ubuntu 15.04:
While i am able to successfully run docker-machine create -d virtualbox --virtualbox-memory=2048 docker-vm, i can't seem to get a working vm running build by make boot2docker-virtualbox.iso .
If i copy the resulting iso to ~/.docker/machine/cache/boot2docker.iso the vm is started, but docker-machine is unable to ssh into it...
&{/usr/bin/ssh [/usr/bin/ssh -o PasswordAuthentication=no -o IdentitiesOnly=yes -o StrictHostKeyChecking=no -o UserKnownHostsFile=/dev/null -o LogLevel=quiet -o ConnectionAttempts=3 -o ConnectTimeout=10 -i /home/strowi/.docker/machine/machines/docker-vm/id_rsa -p 42957 docker@localhost exit 0] []  <nil> <nil> <nil> [] <nil> <nil> <nil> <nil> false [] [] [] [] <nil>}
SSH cmd err, output: exit status 255:
Error getting ssh command 'exit 0' : exit status 255
Getting to WaitForSSH function...
Testing TCP connection to: localhost:42957
Using SSH client type: external
About to run SSH command:
exit 0
...

Can anyone verify this or help me out?

The ISO built from this repo isn't actually supported by docker-machine yet (the changes had to be reverted before the 0.4.0 release). :confused:

Ok, that explains a lot of my troubles. Thx for clearing things up! So i'll have to checkout boot2docker/boot2docker and figure out how to get my tools into that image.

@tianon Could you point us to the reverted commit ? I am running into the same issue.

https://github.com/docker/machine/pull/1640

ok thanks. so the current boot2docker.iso used in the latest docker-machine is the one from boot2docker/boot2docker ?

That is correct.

@tianon any thoughts to when this will be used in docker-machine ?

Definitely best to ask the maintainers of Docker Machine about that (as you did in https://github.com/docker/machine/pull/1640#issuecomment-148728717 :smile:).


Translate Key Concepts » Identity into simplified Chinese

Documents to be translated:

identity

Am translating identity mixer (and generator), will cross check later.

Am translating identity mixer (and generator), will cross check later.
I am translating network now, so identity can be assigned to you, if it is ok :)

Sure, pleasure.


PROD-83: Fix/optimize burn fee

✅ Add Linear ID (e.g., PROD-123) to PR title to automatically link issue

Description
Added Estimate priority function from helius api to get an approrate fees along with adding compute unit in the transaction to increase the sucess rate of the burn transation with very low fees
What are the steps to test that this code is working?
Low priority fees, high success rate.
Screen shots or recordings for UI changes
There is no ui changes

Replaced with: https://github.com/gator-labs/chomp/pull/426


Normalize column names for 60X and 70X coverage

eg
per_twenty_coverage_bases
From:
pct_of_bases_with_60x_coverage
pct_of_bases_with_70x_coverage
To:
per_sixty_coverage_bases
per_seventy_coverage_bases

Issue fixed in 25a906f


fixed directory deletion failure when ufs is cos

What changes are proposed in this pull request?
fixed directory deletion failure when ufs is cos
Why are the changes needed?

cos bucket name is bucket-appid
COSUnderFileSystem.deleteObjects request is not bucket-appid case  Failed to delete 1 paths from the under file system: /xxx (UFS delete dir failed)
ObjectUnderFileSystem.submitBatch exception should print

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

when ufs is cos can normal exec alluxio rm -r xxx

Thanks for the contribution, @Jackson-Wang-7 do you mind help take a look

alluxio-bot, merge this please

alluxio-bot, merge this please


Typescript development workflow?

This isn't really a bug per se, but I'm trying to understand how one can use this project. Here is a catch-22 scenario I am confronted with almost daily, and I'm not sure how to work around it:
Let's suppose you need to modify the return type of a resolver:

Update resolver.ts
Now, typescript is complaining, because what your new resolver is returning does not match the return type of that resolver in the codegen'ed types file. So, you update your graphql typedefs.ts, and re-run codegen
Gotcha! codegen doesn't care about your new typedefs yet, first it will try to download the latest schema, but error, because typescript is not compiling, so the server is not running... ♻

The only solution I can find is to manually edit the return type of the resolver in codegen's output file, so that the typescript will compile and the server will run again. Obviously, this defeats the entire purpose of using this lib at all.
Can someone please explain what I'm missing? Can codegen run off of my graphql typedefs.ts file instead of the schema? Or is there some other solution?
Thanks!

@sarink can you please share a repository? It will help to see how you constructed the code and schema files.
The workflow we are using, is to do schema-first:

Define you schema in .graphql files.
Run codegen and generate resolvers signature (to resolvers.types.ts)
Implement resolvers in a new file (resolvers.ts) using the Resolvers type signature from the generated file.
Now, if you wish to change a return type or something else, do it first in your schema file - if your codegen runs in watch mode, it will automatically update the resolvers signature file, and it will cause your code to be type checked again, and update your resolvers implementation as needed.

If you wish to put your resolvers implementation and schema definition in the same file, you can still do it, I think you just need to specify noRequire: true because otherwise the codegen will have issues to require the file (because if will cause compilation errors from the changes).
Do it answer you question? Did I miss anything?

The missing piece here was that you can tell codegen to generate a schema from your source files rather than from a running service, ie schema: ./src/**/*TypeDefs.ts
Thanks!

@sarink You can use endpoint, code file, graphql file and some more. See the list of loaders here: https://github.com/dotansimha/graphql-code-generator/blob/d3438740d96c8c716c3af65b73f2b7f7a9e70c3d/packages/graphql-codegen-cli/src/load.ts#L16


Password validation

Issue #76 - Added opensearch.yml to YAML files page and instructions on how to add password validation rules
Check List

[x] 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.

@keithhc2 and @aetter can we add a callout in the security setup process for these too in addition to the config update? I can open a new issue.

Disregard my comment I see the step here: https://docs-beta.opensearch.org/security-plugin/configuration/yaml/#opensearchyml


Add GPU acceleration documentation

Signed-off-by: Naarcha-AWS<EMAIL_ADDRESS>Description
Fixes issue #2340
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
Checklist

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

This was identified as experimental in the release highlights, so adding the experimental tag for release notes.


read length calculation question

Hi, I am using riboWaltz package, it is really a wonderful tool.However, I have a question for the calculation of distribution of read length. In my case, the read lengths were range from 25 to 34, but rlength_distr method gave abnormal distribution of read lengths(21-150). I checked the source code of combine_bam.R script, and found the author considered the width of GAlignments object  as the read length,instead of qwidth. In fact, qwidth is true read length.

Bests,
Charlin

Hi Charlin,
thank you for the warning.
I thought quite a lot to which "width" consider. These are some of the considerations I made:

many features of riboWaltz require the rightmost and leftmost coordinates of mapping reads relative to the beginning of the reference sequence. For consistency's sake, the length I consider is then the one associated to the interval between the start and end of the query covered by the alignment (i.e. "width").
in all samples I have analysed so far, columns qwidth and width almost coincide (same values >99%): default options of the most common aligners (TopHat, STAR) force them to discard reads with very small numbers of insertions/deletions/mismatches. This results in few of "I","D" and "M" in CIGAR strings i.e. in small differences between qwidth and width (usually between -5 and 5);
riboWaltz currently works for alignments based on transcriptome coordinates. This means that, in principle, no long gaps in queries "covered" by mapping reads should be present (e.g. from alignments on splicing junctions). How come in your sample reads of 34 nts (max) align on regions of 150 nts?

What I can do is to include a filter to discard all reads with qwidth-width > n where n is an integer number specified by the user. As I said, Less than 1% of the reads would be discarded (would it be true also for your sample?).
Let me know what you think. Suggestions are welcome.
Bests
Fabio

Hi Fabio,thanks for your reply.
To the best of my knowledge, "width" represent matched size of a read,instead of true read length. In my work, we revised "width" as "qwidth" in combine_bam.R script, and I got the expected results(25-34nt). Despite columns "qwidth" and "width" may be almost same, but there is still a small difference between them. This may be avoided through using "qwidth", and qwidth may be more reasonable. For 150 nts I said before, I did not conduct more detailed check, because I found that "width" represent matched size of a read instead of true read length at the time. Thus, I revised it based on your code, and got my expected results.
Thanks for your good work.
Sincerely,
Charlin

Hi Fabio,
I have already performed the command you gave, the results are as follows,

I also give what width 150 is, and width is the sum of non S characters(in cigar). I have already revised width as qwidth, and the program works well.
Yours,
Charlin

Hi Charline,
thank you for your superfast response.
You persuaded me to switch from one width to the other since riboWaltz seems to work (better). Also because my previous idea to include a filter to discard all reads showing qwidth-width > n is not suitable for dataset as yours. In fact, the percentage of reads with different width and qwidth is very high and to throw everything away would mean to waste lot of material.
One last comment: you can for sure use width to get correct read length distribution. Still, I think it would be worth to investigate the wide discrepancy between read length and length of covered regions. I can see from your results that the difference of 120nts is due to the 120 Ns (in CIGAR). In this document N character is described as "skipped region from the reference" and "for mRNA-to-genome alignment, an N operation represents an intron. For other types of alignments, the interpretation of N is not defined". Even if you aligned on genome, data displayed above are in transcriptome coordinates, so no Ns should be there (or at least would be odd).
I still need to verify few things then I'll proceed to modify the code. Meanwhile I'll close this issue.
Thanks again for your tip.
Bests,
Fabio


SPDX in-toto specification does not include version in predicateType

The in-toto spdx schema/spec does not include the version in the predicateType and forces consumers to parse the predicate and read the value of spdxVersion. It would be nice to have the version included in the predicateType to avoid this. This was something that was discussed when adding the CycloneDX attestation specification to in-toto with the same motivation.
This issue could open a pull request to suggest adding the version to the predicateType.

The linked PR has been merged so closing this issue.


Revert "feat: archive issues"

Reverts 42-student-council/website#120

oops didnt see @itislu 's msg


Pulseaudio setup not works, creating container from linux service

I created linux service, that runs docker run command with pulseaudio setup from your tutorual.
It works good, when started from user terminal, but fails when started from service.
I think it maybe related to "Container user must be same as on host" requirement,
or I need to add something like After=pulseaudio to my service file
Please, give me some hint if you know what it may be.

Solved, by adding 2 env vars to the service file. Solution from this source. You can add this to your docker sound tutorial for completeness.
[Service]
Environment="XDG_RUNTIME_DIR=/run/user/1000"
Environment="PULSE_RUNTIME_PATH=/run/user/1000/pulse/"


configure: improve interpreter detection

Currently, when configure is called with --with-java or --with-java=, auto-detection is performed. When called with --without-java, HAVE_JAVA is still being defined. This is unfortunate when cross-compiling for embedded systems, we would end up with Java on the host being auto-detected (which is not what we have on the target), or with a wrong HAVE_JAVA define.
This implements a way to explicitely disable scripting support for all supported languages with ``without-java` (or the respective options for Perl, PHP and Python)
While we're at it, make the detection a bit more verbose, and do some cleanup.

Ultimately I think I want to eliminate all of the third-party scripting support - only Java "needs" special handing as the others support the #! interpreter mechanism...

Ah ok, yes, that seems reasonable. Feel free to only pick the Java part from this PR.

I pushed a slightly different change:
[master b889560c0] Fixed the script interpreter detection in the configure script (Issue #5122)


[CI] Add GPU-based workflows & testing to CI

This PR adds (or rather, resurrects) GPU-based testing in CI using our own self-host infrastructure at Uni Heidelberg. We had some infrastructure for this in the past, but it turned out to be unstable -- hopefully this new approach remedies the issues.
GPU testing in CI has now become even more important than earlier, because our SSCP flow relies heavily on runtime code compilation - which means we don't even stress the full compiler stack unless we actually run SSCP binaries on a GPU.
While the unit tests are not yet in a shape to allow for SSCP testing, this PR is the first step towards this goal.
Backend software infrastructure
The backend infrastructure to deploy and manage the github runners in a containerized manner is contained in the OpenSYCL/ci repository, which is currently only accessible for members of the OpenSYCL organization.
Container for test environment
The PR here contains the workflow as well as the definition of the singularity container image that is used on our infrastructure.
Currently, when the container definition changes, the container needs to be rebuilt and deployed manually.
This is a bit cumbersome, so ideally we could have some additional mechanism that automatically rebuilds the image when the file is change and maybe stores it on dockerhub or similar... The runners could then just pull that image (although they'd still need to be restarted somehow..).
In the best case, we could then even share the images between the self-hosted flows and the github flows to remove redundancy. If somebody has experience with these kinds of things and wants to look into this, please let me know :-)
Hardware
Currently testing is enabled on a dedicated node with 32 cores and 4 NVIDIA 1080 Ti sm_61 GPUs with the omp and SMCP cuda compilation flow.  Each GPU is used exclusively by one GitHub runner.
I also have AMD and Intel hardware available which I plan to hook up into CI as part of this PR soon.
Security
Important: GitHub very prominently warns that adding self-hosted CI runners to public repositories like ours can pose a security risk, because if someone with malicious intent opens a PR, arbitrary code from that PR will be executed on our own machines.
Because of this, and even though I have taken some safeguards on our machines, I have changed the repository setting such that all PRs from non-members of the organization will require approval from a member before workflows will be run.
I'd appreciate if @OpenSYCL members could help me approve workflows where necessary :-) But please have a quick glance at the code before approving just to check if there is no bogus stuff in there that could be a security hazard. Whoever approves, vouches for the code ;)
Membership in the OpenSYCL organization is not intended to be exclusive; if you want in please let me know. I'm open to adding people as long as we have some idea who that person is, in order to have accountability regarding the security aspect. And bonus points for people who have contributed/we have collaborated with in the past :-)
Workflow todos
The current workflow is pretty bare bones. There are more things we should test:

[x] Explicit multipass testing
[x] AMD hardware testing
[ ] nvc++ testing?

Probably in a separate PR:

[ ] SSCP testing
[ ] Intel testing (depends on SSCP testing)

We might also want to look into whether we can integrate automated performance regression testing, such as https://github.com/benchmark-action/github-action-benchmark

Alright, I think this is good enough for a first stab. I had to remove nvc++ testing because compiling is super slow with it (45min for the tests!) and it also miscompiles some of our tests which I had forgotten about. So we'd first have to address this before we can start testing with nvc++.

Might be interesting to check out testing with arm as well? 🤔
https://resources.github.com/arm-gihub-actions-beta/

Definitely. But it seems we currently cannot do this, as the "larger-hosted runners" feature, which is a prerequisite, seems to be only available for enterprise accounts. In any case I cannot select the OpenSYCL org :(


Colab notebooks not opening on GitHub.

I have pushed the notebooks I wrote on Colab directly from Colab itself, and provided links to it in the README file written as [link-colab], but they are not loading on GitHub.
However, the same notebooks can be accessed directly on Colab by clicking on the corresponding [link-colab] links. For now, please use the Colab platform for viewing the notebooks.
If anyone knows what's wrong let me know, although this seems like an issue with Colab / GitHub integration. I will wait for it to resolve for a while (probably a few days); if it doesn't fix itself, I will download the notebooks on my local machine and push them onto GitHub, after which they should open on GitHub as well.
If anyone knows any alternative solution through which I can open the notebooks on GitHub as well as Colab, please write them down in the comments (or create a pull request with the solution implemented).

Just checked. It's working now. Hence, closing the issue.


Async let mocks causing crashes

We have issue with async let . here is an example :
@Mockable
public protocol PersonProtocol {
    var name: String { get set }
}

public final class TestCrash {
    let person: PersonProtocol

    init(person: PersonProtocol) {
        self.person = person
    }

    private func firstFunc() async -> String  {
        return person.name
    }

    private func secondFunc() async -> String  {
        return person.name
    }

    private func thirdFunc() async -> String {
        return person.name
    }

    @MainActor
    func loadAll() async {
        async let first = firstFunc()
        async let second = secondFunc()
        async let third = thirdFunc()

        let (_,_,_) = await (first, second, third)
    }
 }

import XCTest
import MockableTest
@testable import FinanceUI

final class CrashTestCase: XCTestCase {

    func testCrash() async {
        let person = MockPersonProtocol()
        given(person).name.willReturn("")

        let sut = TestCrash(person: person)

        await sut.loadAll()
    }
}

with that we get a crash

Can you confirm this with the latest version too?

@emadhegab I was able to repro this only in 0.0.3, the crash is not present in 0.0.4.
However, this issue highlights an important shortcoming: the current Mocker implementation was not fully thread-safe.
#31 addresses this by synchronising all state mutations


⚠ PrivateHD has degraded performance

In 7dbb2a7, PrivateHD ($PHD) experienced degraded performance:

HTTP code: 200
Response time: 1774 ms

Resolved: PrivateHD performance has improved in 485165d.


remove config from installer

Remove agent config templates from installer as agent and installer are now distinct
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

It's required to remove those to be installed in parallel with datadog-agent deb/rpm.
Yes it does change the behaviour of agent oci, but since we're not shipping it now to customers it's ok. We'll have to add config templates in it

/merge


Using stroke_color causing memory access violation

When using stroke_color, either the python crashes, or it gets memory access violations. any idea why ?
Traceback (most recent call last):
  File "C:\Users\aaron\AppData\Local\Programs\Python\Python36\lib\site-packages\discord\client.py", line 224, in _run_event
    yield from coro(*args, **kwargs)
  File "F:\bot\cogs\auto.py", line 178, in on_member_join
    draw(frame)
  File "C:\Users\aaron\AppData\Local\Programs\Python\Python36\lib\site-packages\wand\drawing.py", line 1963, in __call__
    return self.draw(image)
  File "C:\Users\aaron\AppData\Local\Programs\Python\Python36\lib\site-packages\wand\drawing.py", line 917, in draw
    res = library.MagickDrawImage(image.wand, self.resource)
OSError: exception: access violation writing 0x0000022B9269FFD8

I'm assuming on_member_join uses await, yield, and/or async Python3+ synchronization features. If so, this might require a mutex lock during drawing operations. Is this access violation occur with other color operations, or just drawing-context with stoke_color?

just drawing context, with stroke_color. I dont get this error using my linux server or when removing stroke_color though. and yes, on_member_join is using async

I'm unable to reproduce this on OS X, Fedora, or Debian. Can you post the output of the following...
python -mwand.version --verbose

Also see if the same error occurs when setting fill_color / text_under_color. They both have identical setter definitions with stoke_color.  Setting these values simply adds MVG instructions to the drawing-context-stack, and doesn't hold any reference pointers. If the error only occurs when invoking draw(image), then issue might be elsewhere.


fix to bom

I was doing some tests with camel-spark when I noticed it's not in the BOM.

Thanks, is it in apache-camel also, eg for the distro when you build the zip? You may need to add more stuff if its not there either.

Yeah, it was missing.. It's missing from karaf too, but I guess it's impossible to make Hadoop osgi-compliant :)

Absolutely impossible :-)
Il 21/lug/2016 22:44, "Nicola Ferraro"<EMAIL_ADDRESS>ha
scritto:

Yeah, it was missing.. It's missing from karaf too, but I guess it's
impossible to make Hadoop osgi-compliant :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/camel/pull/1084#issuecomment-234378439, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AE3r11Cg-xopqAN-JI6_RyufVoOjEOc_ks5qX9pEgaJpZM4JR7FN
.

Thanks for the PR it has been merged. Do you mind closing this?


It seems you are using a legacy version of Ollama although im using the most recent version of Ollama.

Hello,
I'm getting the error: [WARNING]: It seems you are using a legacy version of Ollama. Please upgrade to a newer version for better chat support. but I'm using the most up-to-date version of Ollama (0.1.27)
I tested in Windows and Linux, and both have this problem.
This is the log from Ollama:
llm_load_tensors: offloading 32 repeating layers to GPU
llm_load_tensors: offloading non-repeating layers to GPU
llm_load_tensors: offloaded 33/33 layers to GPU
llm_load_tensors:        CPU buffer size =    70.31 MiB
llm_load_tensors:      CUDA0 buffer size =  3847.55 MiB
..................................................................................................
llama_new_context_with_model: n_ctx      = 2048
llama_new_context_with_model: freq_base  = 1000000.0
llama_new_context_with_model: freq_scale = 1
llama_kv_cache_init:      CUDA0 KV buffer size =   256.00 MiB
llama_new_context_with_model: KV self size  =  256.00 MiB, K (f16):  128.00 MiB, V (f16):  128.00 MiB
llama_new_context_with_model:  CUDA_Host input buffer size   =    13.02 MiB
llama_new_context_with_model:      CUDA0 compute buffer size =   164.01 MiB
llama_new_context_with_model:  CUDA_Host compute buffer size =     8.00 MiB
llama_new_context_with_model: graph splits (measure): 3
time=2024-02-24T13:33:18.359Z level=INFO source=dyn_ext_server.go:161 msg="Starting llama main loop"
[GIN] 2024/02/24 - 13:33:19 | 200 |    8.7230985s |       <IP_ADDRESS> | POST     "/api/generate"
[GIN] 2024/02/24 - 13:33:19 | 404 |            0s |       <IP_ADDRESS> | POST     "//api/chat/api/chat"
[GIN] 2024/02/24 - 13:33:19 | 404 |            0s |       <IP_ADDRESS> | POST     "//api/chat/api/generate"
[GIN] 2024/02/24 - 13:37:14 | 404 |            0s |       <IP_ADDRESS> | POST     "//api/chat/api/chat"
[GIN] 2024/02/24 - 13:37:14 | 404 |            0s |       <IP_ADDRESS> | POST     "//api/chat/api/generate"

Langchain is not generating the code and just printing that warning to the console.
    const stream = await this.chatClient
      .pipe(new StringOutputParser())
      .stream(`Translate "I love programming" into German.`);

    const chunks = [];
    for await (const chunk of stream) {
      chunks.push(chunk);
    }

    this.logger.debug(`Chat response: ${chunks.join('')}`);

I'm using the code from the example and is not working, I'm using the chat, not embeddings

@dosu-bot Langchain is not generating the code and just printing that warning to the console.
const stream = await this.chatClient
  .pipe(new StringOutputParser())
  .stream(`Translate "I love programming" into German.`);

const chunks = [];
for await (const chunk of stream) {
  chunks.push(chunk);
}

this.logger.debug(`Chat response: ${chunks.join('')}`);

I'm using the code from the example and is not working, I'm using the chat, not embeddings

How are you initializing the Ollama client?

Hi, yes, ollama serve.

In code I mean

Oh, yes
this.chatClient = new ChatOllama({
      model: chatModel,
      baseUrl: host,
    });

I can run embeddings, but not chat. Also, when I run the .stream() method there is some activity in the Ollama terminal.

Oh, neverming. I had the wrong host, sorry to bother you.


add zipkin demo

Coverage increased (+0.03%) to 62.056% when pulling 2f6c0fe8ed5fae02264915cdb36256c445c3ef70 on tianxiaoliang:master into b4919cc88df215650f8abfc9aa41117b647d5a08 on ServiceComb:master.


Be less prescriptive with release git basics

All of the discussion about creating and reusing and whatnot for
branches is just to say "base your commit against devel properly".
That's already enforced with automation.
There are other actually-important branching topics covered here, and
this only serves to obscure them, not provide useful clarity.
Depends on #534

Closing in favor of this commit as a part of https://github.com/submariner-io/submariner-website/pull/539, which I will continue adding to.


Fix mobile packaging pipeline builds.

@wenbingl Can you please follow up on and finish this PR while Ed is out?

I will follow up on this PR after the other PR is done.

@wenbingl Can you please follow up on and finish this PR while Ed is out?

I will follow up on this PR after the other PR is done.

Thank you @wenbingl

I can deal with this PR tomorrow if needed.

I can deal with this PR tomorrow if needed.

Yes, please.
I have my hands full with something unplanned and may only be available to do it in the next week.

Packaging builds pass with the changes.


How to build the Wordcloud Data from a Obsidian note ?

x: "Word"
value: 1
category: "words"

How can i get this data from inside a obsidian note it self in a automated way?

x: "Word"
value: 1
category: "words"

How can i get this data from inside a obsidian note it self in a automated way?

You could use CSV file.


About test time config

Hi @tianweiy Thanks for sharing this great work. I have several questions regarding the configuration of test time.

why you choose 83 for nms_post_max_size ? wouldn't it be too large for most of the categories in one frame?
why the post_center_limit_range=[-61.2, -61.2, -10.0, 61.2, 61.2, 10.0] is the larger than pc_range=(-51.2, -51.2, -5.0, 51.2, 51.2, 3.0), wouldn't it be useless for post_center_limit_range?
any general guidance on the configuration if I would like to train CenterPoint on my own dataset, currently I follow nuscenes configuration, but found many false positives, and the detection results generate fastly with the distance between objects center and the vehicle center.

Best,
Xuyang

why you choose 83 for nms_post_max_size? wouldn't it be too large for most of the categories in one frame?

Yes, it will be quite large. I never tune this yet.

why the post_center_limit_range=[-61.2, -61.2, -10.0, 61.2, 61.2, 10.0] is the larger than pc_range=(-51.2, -51.2, -5.0, 51.2, 51.2, 3.0), wouldn't it be useless for post_center_limit_range?

Yes, you can set this to be a little smaller. This is only used at the end (after NMS).

any general guidance on the configuration if I would like to train CenterPoint on my own dataset, currently I follow nuscenes configuration, but found many false positives, and the detection results generate fastly with the distance between objects center and the vehicle center.

For false positives, you need to either increase the score threshold or decrease the NMS iou threshold. This should be able to eliminate FPs.

the detection results generate fastly with the distance between objects center and the vehicle center.

Could you elaborate this a little bit more?

Thanks for your quick reply. For the last question, I breakdown the metric by the distance between object center and vehicle frame origin just like WOD, but I found the AP decrease very fastly when the distance becomes larger, for example, the AP for car is larger than 0.90 for [0-30m) range but decrease to like 0.4 for [30-50m). I know that might because the density and pts_in_gt_box is very small for far regions, but any suggestion to improve the results for far regions?

I see. This is quite normal to my knowledge. If you visualize the prediction, for nuScenes (32 lane lidar), you only get one or two points inside a box (the bottom box in the image I attached). I guess there is no way to detect it ..

Ok I see, thanks a lot :)


feat: rsa signature

- What I did
Implemented semi-automatic "from:atSign" challenge signing (pkam).
- How I did it
I haven't been able for now to decode the PkamPrivateKey with the selfEncryptionKey (AES CTR mode). So I created a tool which does exactly that (and gives as output "n, e, d, p, q" parameters from the PkamPrivateKey) while I find a way to implement AES CTR mode propperly (work in progress).
Update: I have been able to decrypt the AES CTR encrypted PkamPrivateKey inside the Pico W. Now I need to find a way to extract "n, e, d, p, q" from the decrypted key (which is similar to a .pem file) so I can use the RSA library without any external tool
Then I used this library to recover the RSA Private Key from those parameters and sign the challenge provided by the secondary server after a "from:atSign" verb was sent.
- How to verify it
Follow the new steps in the README.md file from this branch
- Description for the changelog
Semi-automatic pkam challenge signing implemented

@realvarx could the packages be installed with mip rather than being copied into this repo?

Closing this PR, as the new one will contain more features and the full functionality. Also ~/lib/ directory weight has been significantly reduced.


AmbientLight contructor

The applications createAmbientLight does some parsing and checking to create an AmbientLight.
This code can more less directly go into the AmbientLight constructor. As it checks for null this means no change to the api. Just easier to use the class.
If you want i can make the change

Not sure if I understand you correctly. The constructor doesn’t do null checking on properties

If you mean adding the null checking in the AmbientLight constructor. Yeah i think it will be better. You can make a pull request about this. Thanks!

Or do the validation and color parsing in the uniformTemlates?

Sorry, what are uniformTemplates?

Had a little look at the uniformTemplates and also the whole extend and default mechanism.
That's a little too deep water at the moment.


User needs to have the ability to send in GenV2 in a CSV format and undergo same validation as HL7 message

@lmcnabb What is the difference between #32 and this ticket #31?  Is one Lyme, Arbo, and then Hepatitis?   There then might be three versions?

@lmcnabb What is the difference between #32 and this ticket #31? Is one Lyme, Arbo, and then Hepatitis? There then might be three versions?

31 was for structural validation (think NIST validator) and 32 is for MMG processor for just the MMG-based model


Cannot grant permission for directories: acl is missing

What happened?
Operator grants permissions for the desired directories for user robolaunch with the command setfacl. Operator fails in images that does not contain acl package.
What did you expect to happen?
I expect images to include acl package or install it while loading environment.
How can we reproduce it (as minimally and precisely as possible)?
Almost every image could verify the problem. Loader job crashes if a directory is described in additional config GRANT_PERMISSION.
Kubernetes version
All Kubernetes versions
Container network interface (CNI) and version
No response

Images include acl after 219ce52b310cbe9f4f78741635529810ea203827.


WIP: Repro ui test bug

This is happening again.

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
     Running `/builds/parity/substrate/target/release/deps/construct_runtime_ui-f676bb38f83afb81`
running 1 test
ERROR: failed to execute cargo: No such file or directory (os error 2)
test ui ... FAILED
failures:
---- ui stdout ----
thread 'ui' panicked at 'tests failed', /ci-cache/substrate/cargo/test-linux-stable/registry/src/github.com-1ecc6299db9ec823/trybuild-1.0.23/src/run.rs:38:13
stack backtrace:
   0: backtrace::backtrace::libunwind::trace
             at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/libunwind.rs:88
   1: backtrace::backtrace::trace_unsynchronized
             at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.40/src/backtrace/mod.rs:66
   2: std::sys_common::backtrace::_print_fmt
             at src/libstd/sys_common/backtrace.rs:84
   3: <std::sys_common::backtrace::_print::DisplayBacktrace as core::fmt::Display>::fmt
             at src/libstd/sys_common/backtrace.rs:61
   4: core::fmt::write
             at src/libcore/fmt/mod.rs:1025
   5: std::io::Write::write_fmt
             at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/io/mod.rs:1426
   6: std::io::impls::<impl std::io::Write for alloc::boxed::Box<W>>::write_fmt
             at src/libstd/io/impls.rs:156
   7: std::sys_common::backtrace::_print
             at src/libstd/sys_common/backtrace.rs:65
   8: std::sys_common::backtrace::print
             at src/libstd/sys_common/backtrace.rs:50
   9: std::panicking::default_hook::{{closure}}
             at src/libstd/panicking.rs:193
  10: std::panicking::default_hook
             at src/libstd/panicking.rs:207
  11: std::panicking::rust_panic_with_hook
             at src/libstd/panicking.rs:471
  12: std::panicking::begin_panic
  13: <trybuild::TestCases as core::ops::drop::Drop>::drop
  14: core::ops::function::FnOnce::call_once
  15: <alloc::boxed::Box<F> as core::ops::function::FnOnce<A>>::call_once
             at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/liballoc/boxed.rs:1022
  16: __rust_maybe_catch_panic
             at src/libpanic_unwind/lib.rs:78
  17: std::panicking::try
             at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panicking.rs:270
  18: std::panic::catch_unwind
             at /rustc/5e1a799842ba6ed4a57e91f7ab9435947482f7d8/src/libstd/panic.rs:394
  19: test::run_test_in_process
             at src/libtest/lib.rs:567
  20: test::run_test::run_test_inner::{{closure}}
             at src/libtest/lib.rs:474
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
failures:
    ui
test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out
error: test failed, to rerun pass '-p frame-support-test --test construct_runtime_ui'
Command exited with non-zero status 101

Yeah this is fixed and can be closed.


ListView renders cursor

Even if there's nothing to type in a listview :)

I really don't know if the cursor can be hidden in the Gui, without overshadowing the color.

Fixed somewhere along the line.


Remove track only undone balance feature

The problem
The feature "Track undone balance only" requires users to click on the checkbox, which is not align with the TP constraint of "Typing Preferred", as there is no equivalent typing command for this, and hence a bug.
Changes

Remove track only undone balance feature. Removing this feature has little impact as user can still filter transaction with undone status.
Add vertical scrollbar on balance overview section. As the balance text is wrapped, a very long amount value will increase the height too big to fit in balance overview section.

closes #406, closes #427, closes #389

Screenshots and relevant details in DG/UG should be update

Done!


Updating the avatar based on changes happened after initial loading

I could not get the avatar updated after some changes in the user data. For example, when I load the avatar initially there is no picture which makes it falls back to initials. Later, when a profile picture is added, I need to update the avatar with that image URL. This is not happening now.
Could anyone please help me in getting this done?
Thanks in advance.

Could anyone please respond on this ? I am still stuck with this requirement.

Hi,
Could you please share some code so I can reproduce your issue ? Actually, updating avatar data asynchronously is supported. You can check the demo folder, it has an example of getting user initials asynchronously after the first loading.

<ngx-avatar [name]="userFullName" [src]="profilePic" size="24" textSizeRatio="2.5" initialsSize="2"></ngx-avatar>
When the component loads initially there will not be any profile picture. So the initials will be displayed. But the user can upload a picture. I need it to reflect in the ngx-avatar in realtime without refreshing the screen by changing the variable profilePic and vice versa. This is not happening now.
Example:
https://stackblitz.com/edit/ngx-avatar-demo-ttvfku
See setting the image after the timeout in app.component.ts. But nothing happens.

Hope you got the issue now.

Here is a quick fix for you.
https://stackblitz.com/edit/ngx-avatar-demo-jqyda6
Simply wrapping module in a div and using a boolean to show and hide it. essentially refreshing the component.
Not nice.... but not a naughty fix.

@manumqln,
You probably should use a service or a store to synchronize the state of the avatar.
From what I read from your issue, it seems like a sync issue that is not the library's responsibility.
I'm assuming you have the avatar in on one component and then the avatar is changed within a user profile, which is in a completely different component.
What I am doing is:
I created a singleton service (until I implement ngrx) provided within the app.module.
Both component inject the avatar service.
When the avatar is successfully uploaded to your application, the httpClient subscription triggers a BehaviorSubject with the new avatar value, which is subscribed on the topnav component (where I display the user avatar).
Due to this is structure, the component is able to properly react when a user change their avatar.
In case you are loading the avatar from Gravatar or Facebook or other external source, there's no way your application will be able to automatically react and therefore, you will need a manual refresh on the page to fetch that data.
I hope this can give you an idea on how to fix your problem...
I modified one snippet I have with a similar pattern to give you an idea of what I'm talking about.
Its not complete and its just to show a basic usage of the service that can be used to sync the state across different components throughout the app.
import { BehaviorSubject } from 'rxjs/BehaviorSubject';
import { Injectable } from '@angular/core';

@Injectable()
export class AvatarStateSrevice {

    avatar$: BehaviorSubject<any>;

    constructor() {
        this.avatar$ = new BehaviorSubject(null);
    }

    getAvatar() {
        return this.avatar$.asObservable();
    }

    setAvatar(value: boolean) {
        this.avatar$.next(value);
    }
}

@Component({
    moduleId: module.id,
    selector: 'app-security',
    templateUrl: 'top.nav.component.html'
})
export class TopNavComponent implements OnInit {
    constructor(private avatarStateService: AvatarStateSrevice ) { }

    ngOnInit() {
        ....
        this.avatar$ = this.avatarStateService.getAvatar();
    }
}

<ngx-avatar src="{{avatar$ | async }}"></ngx-avatar>

Then on the upload component, let's say you have a onSuccess() method:
onSuccess(response){
     this.avatarStateService.setAvatar(response.newAvatar);
}

This should update the value on your top nav component with the new avatar.
Obviously you will need to modify to your structure but it should give you an idea...
I hope this helps...

@minuz Thanks for the well explained solution. I will check this and update.

I think there is an issue with this line where it checks whether there is an existing source of the type before (in this case, it is "custom")
https://github.com/HaithemMosbahi/ngx-avatar/blob/44791e79942f6f05832536b669c56023460fad58/projects/ngx-avatar/src/lib/avatar.component.ts#L272
It is comparing the existing sourceType with an toUppserCase version while we can see that the enum definition is all lower case:
https://github.com/HaithemMosbahi/ngx-avatar/blob/44791e79942f6f05832536b669c56023460fad58/projects/ngx-avatar/src/lib/sources/avatar-source.enum.ts#L1-L12
I've made a pull request in case you want to merge it.

@odahcam Thanks for your reviewing. I've applied your suggestions.

@haivp3010 Thank you for your contribution and for your pull request. you are right we don't need the toUpperCase() there.
I have already started to write unit test to the component and do some refactoring in order to get rid of bugs like that. In the new version I already fixed that.
https://github.com/HaithemMosbahi/ngx-avatar/blob/a5cb9ea21215e50ac1a9a741e4f71a5a4c8ce23c/projects/ngx-avatar/src/lib/avatar.component.ts#L288
I will do my best to get this merged as soon as possible.

I'm not able to change the value of src in the production build. There is any fix for this? My idea is to change one picture to another.

I don't know about this error, but as a workaround you can use multiple instances of ngx-avatar and switch them.

Hey @odahcam it could be great to fix this. Add several instances of ngx-avatar is not a good solution. The image should be updated after changing a src for another one.

Any news???

Sorry, I'm pretty busy at the moment, but I still can happily review some PR if someone wants to send one.

<ngx-avatar
[src]="''"
[name]="username | async"
size="40" initialsSize="2">
Where username is observable. Worked for me

This issue is already fixed.

I will close this for now, feel free to create a new issue if you still have any problem with the component.


⚠ Bofrost.at has degraded performance

In eea9700, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2554 ms

Resolved: Bofrost.at performance has improved in 9dbde42.


chore: return json body under http status 401

I hereby agree to the terms of the GreptimeDB CLA
What's changed and what's your intention?
This pr mainly does

change AsyncRequireAuthorizationLayer to middleware::from_fn_with_state so it's now possible to construct resp body
return json body under status code 401 just like other http response
minor: change unsupported error to debug log level

Checklist

[ ]  I have written the necessary rustdoc comments.
[x]  I have added the necessary unit tests and integration tests.

Refer to a related PR or issue link (optional)

I think we cannot return json body for APIs like influxdb ingestion, prometheus queries. Perhaps we need to check the Accept header to decide which format to return.

I think we cannot return json body for APIs like influxdb ingestion, prometheus queries. Perhaps we need to check the Accept header to decide which format to return.

InfluxDB and prometheus queries now both support json response. We can extract a middleware function later if needed, checking Accept header in one place.


done, working great, next step add format date in appsettings.json

Added Queue feature
Added JSON decode to save as Object in MongoDB
Added appsettings.json file to configure with more options, like list of topics, mqtt config, MongoDB config
Addes Dockerfile, docker-compose.yml, docker-compose.debug.yml for run as container (daemon, and debug).

Hello there @juniorsaldanha !
Thank you for your huge PR! However, I'm currently no longer working on/with (and even archiving) many of my MQTT2(db) repositories, including this one (which I forget to archive), so I'd rather keep it as-is. I propose you to keep your changes in your fork (or even creating a new repository with your version), and I'd link it from my README. What do you think?
Regards!

Hi @David-Lor , thats is great.. and thanks for you base code by the way. 😀


Built in support for running Android native tests in CI

This change enables running Android native tests (instrumentation tests) within CI environment using runner defined by project configuration.
Related to task https://github.com/thaliproject/Thali_CordovaPlugin_BtLibrary/issues/46

This change is

Previously, czyzm wrote…
> #### Built in support for running Android native tests in CI
>
> This change enables running Android native tests (instrumentation tests) within CI environment using runner defined by project configuration.
>
> Related to task https://github.com/thaliproject/Thali_CordovaPlugin_BtLibrary/issues/46
>
>
>
>
>
>
>
>
>

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Launcher crashed

Immediately after installation  SMS Backup+
Launcher 3 crashed on my phone.
It happens without  start SMS Backup+

Android 6.0.1
Lenovo Phab 2 Pro
SMS Backup+ version 1.5.11

version 1.5.11-beta4 is OK
but versions 1.5.11-beta7 and 1.5.11 cause Launchercrashed on my phone


Should not __toString() rewind the stream?

See the example:
$response = $this->client->send($request, $options);

var_dump($response->getBody()->getContents()); // outputs normal response
var_dump((string)$response->getBody()); // outputs normal response
var_dump($response->getBody()->getContents()); // outputs empty string

Shoud not \GuzzleHttp\Psr7\Stream::__toString do a rewind after reading from the stream?
I ran into the problem while attaching Logging middleware, which caused  the response to be empty.
\GuzzleHttp\MessageFormatter::format line 88 does this:
$result = $response ? Psr7\str($response) : '';

And in \GuzzleHttp\Psr7\str there's a string cast on line 38 which causes the stream to be seeked to the end.
return "{$msg}\r\n\r\n" . $message->getBody();

Using Guzzle 6.1.1

I think this also explains my issue. I found that when I used the log middleware with DEBUG log, then I couldn't access the body of the request. I had to do this in my own log middleware...
$response->getBody()->rewind();

Yes, the same workaround also worked for me. Nevertheless, I wouldn't say this is very intuitive.

Guzzle must adhere to PSR-7 here, which does not state that the stream is to be rewound after casting it to a string. You'll need to push for PSR-7 to be amended in order for us to be able to update Guzzle. Guzzle's implementation also matches Zend Diactoros.
http://www.php-fig.org/psr/psr-7/

PSR-7 says about __toString()...

This method MUST attempt to seek to the beginning of the stream before
reading data and read the stream until the end is reached.

That's not what currently happens. For example, if logging middleware is used to log the body of response then it is not available in __toString on the response afterwards.

It does seek https://github.com/guzzle/psr7/blob/master/src/Stream.php#L93

More than a year later I came across this problem again. A simple rule of thumb for anyone having this issue:
Use (string)$response->getBody() instead of $response->getBody()->getContents(). The former returns always the full response, as the stream is rewound inside of __toString().
The latter may suddenly become empty, if someone decides to add some logging to your application and the stream has already been read before this line of code. For example using the built-in log format \GuzzleHttp\MessageFormatter::DEBUG causes this problem.
As this is absolutely not intuitive in any sense, I would strongly suggest adding this little nuance to the manual to save future generations debugging time. :)

5 years all php community struggles with this problem. I used rewind middleware (lol) with logging middle ware and it worked. For some reason on new project with new versions it does not work.
        $stack->push(
            Middleware::mapResponse(function (ResponseInterface $response) {
                $response->getBody()->rewind();
                return $response;
            })
        );

        $stack->push(
            Middleware::log($unilabApiLogger, new MessageFormatter(MessageFormatter::DEBUG))
        );

        $stack->push(
            Middleware::mapResponse(function (ResponseInterface $response) {
                $response->getBody()->rewind();
                return $response;
            })
        );

It is so annoying to get empty logs or empty response body so I simply added 2 rewind callbacks to make sure it would work. But now it does not. What a pity the whole PHP community cant make such a simple thing: http client that works.


best practices for updating?

I noticed this AMI got pulled last month: https://github.com/awslabs/amazon-eks-ami/releases/tag/v20231220 as with all software I understand this EKS AMI is no different and may have bugs. Beyond doing our own testing in dev/nonproduction environments is there any publicly viewable data to see what overall adoption of the newest AMI is like?

I've never seen any data on usage. Some setups will automatically attempt to get the latest versions like Karpenter in its default configuration. Others pin. We've always pinned and then will usually wait a week or 2 after an AMI release and then check this repos issues for any reported issues before we start rolling it in non-prod environments.

We're discussing improvements to be made to our documentation, Karpenter, etc. after this event. We will clarify our criteria for recalling an AMI. I'll update here when I can.


Remove duplicate card checkbox component.

Remove duplicate code for selecting service offerings.

Codecov Report

Merging #99 into master will increase coverage by 0.06%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #99      +/-   ##
==========================================
+ Coverage   14.47%   14.54%   +0.06%
==========================================
  Files          67       66       -1
  Lines         891      887       -4
  Branches       39       38       -1
==========================================
  Hits          129      129
+ Misses        729      726       -3
+ Partials       33       32       -1

Impacted Files
Coverage Δ

.../PresentationalComponents/Platform/PlatformItem.js
0% <ø> (ø)
:arrow_up:

...rc/PresentationalComponents/Shared/CardCheckbox.js
0% <0%> (ø)
:arrow_up:

src/SmartComponents/Portfolio/PortfolioItem.js
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@Hyperkid123 : rebase, please


Adding Variable Delay Timer Based on Arm Status

When the satellite comes out of its deployer pod it needs to wait for a certain amount of time before trying to fire the burn wire, do its first beacon, and all the rest. We only want this to happen one time though, so it doesn't try to wait 30 minutes every single time it reboots.

This is getting closed for now as we rethink our software strategy.


Is it possible to attach a svg file?

Please help. How can I attach a svg file?

Hi! 👋 Thanks for the issue!
We're tracking this issue internally and will let you know when we have a resolution.

This should be fixed now!
Feel free to open another issue if you are still having problems with anything.


Display safeTxHash in tx details

What is this feature about? (1 sentence)
This will display the safeTxHash on the transaction details.
Why is it needed? What is the value? For whom do we build it?

We currently only display txHash. That's the public txhash available after submit/execution.
safeTxHash is needed to sign transactions, including  onchain signing via approveHash
The safeTxHash can be used to verify tx parameters. So that's especially important for validating transactions

High-level overview of the feature

safeTxHash is displayed for all txs details

Includes Queue and history

Regular txHash is still additionally displayed when available

Misc

Mobile ticket: gnosis/safe#352

Verified:
android app version: 2.5.0 -1595


Add "Editor" role to users

There needs to be a specific role that can edit the contents on the site, while not being able to access the backend. All other "normal" users should not be able to create or edit content.

Already included in one of the previous pull requests.


pdfview  can support web link?

pdfview  can support web link?

Unfortunately ther is no support for links and currently I do not plan it.

why pdfview can not supoort link?

It is not implemented

I've implemented support for links and I'll submit PRs for the required changes. Of course it is up to @barteksc to accept them.


shinyTree in python

Hi
Do you plan to add a similar shinyTree package python as we have in R. In fact, I'm looking for a feature to have nested group checkboxes. I would like to replicate the following R code in python:
library(shiny)
library(shinyTree)
server <- shinyServer(function(input, output, session) {
  output$tree <- renderTree({
    list(  'I lorem impsum'= list(
      'I.1 lorem impsum'   =  structure(list('I.1.1 lorem impsum'='1', 'I.1.2 lorem impsum'='2'),stselected=TRUE),
      'I.2 lorem impsum'   =  structure(list('I.2.1 lorem impsum'='3'), stselected=TRUE)))
  })
})
ui <- shinyUI(
  shiny::fluidPage(
    h4('Shiny hierarchical checkbox')
    ,shinyTree("tree", checkbox = TRUE)
  )
)
shinyApp(ui, server)

We (the Shiny team) don't have plans to port shinyTree to Shiny for Python, but the maintainers of shinyTree might. I'd recommend that you follow up with them in their repo: https://github.com/shinyTree/shinyTree


[REQUEST] split zero3 checkpoint files into optim states and master weights

Currently, if one wants to pull out fp32 weights from the zero-3 checkpoint they have to load the full checkpoints that contain 3x of data that the user needs. And for huge models this can take hours.
I have already created a PR to drop the optim states here https://github.com/microsoft/DeepSpeed/pull/4025 since otherwise the user needs 3x cpu memory, but the fix can only discard that unneeded data after loading it, so it's still very slow.
One of the solutions I proposed is to split each of the fp32+state1+state2 shard files into 2 files:

fp32
state1+state2

that way a user not caring about optim states and just wants to retrieve weights will be able to perform the extraction much much faster.
as I mentioned in the PR, the other option is to switch to https://github.com/huggingface/safetensors/ which  allows selective loading of tensors, rather than all-in-one loading.
@tjruwase

Hey I also ran into the same issue. It seems splitting into separate files is a good optimization. I also wonder if saving to fp16 is enough for both resuming training and downstream inference.


Started fixing the migrations

Some of the changes to database files need to be moved out into
migrations.
Currently I have started fixing them but ran into some issues with
SQLite ALTER TABLE.
Related Issues
#57
Describe the Changes
Moved some old database changes into migrations
Still To Do
SQLite only supports a limited subset of ALTER TABLE so I need to find a workaround.
Problems
See above
Acknowledgements
@sommea Wrote the initial database changes.

Abandoning this for a number of reasons.


Invalid code action for unused error variable

Description:
https://user-images.githubusercontent.com/34838308/155460434-eaf3bab0-53ac-4184-aea2-e14a86059ba5.mov
Affected Versions:
Plugin version: 3.0.0
SL 2201.0.0

Same as https://github.com/ballerina-platform/ballerina-lang/issues/34279

Duplicate of https://github.com/ballerina-platform/ballerina-lang/issues/34279


Problem to use history = HistoricalRecords(excluded_fields=['description'])

I would like to exclude some fields of the model.
I use the following to exclude a field to the model:
class Metabolite(BaseModel):
    met_id = models.AutoField(primary_key=True, db_column="met_id")
    abbreviation = models.CharField(max_length=250, db_index=True, help_text="Unique metabolite identifier")
    fullName = models.TextField(help_text="Name of the metabolite")
    description = models.TextField(null=True, help_text="Description of metabolite in HMDB", blank=True)
    history = HistoricalRecords(excluded_fields=['description'])

I made makemigrations
Following file created:
operations = [
migrations.RemoveField(
model_name='historicalmetabolite',
name='description',
),
I run the migration
And when I run python manage.py populate_history --auto
I add the following mistake:
Saving historical records for <class 'reconws.models.Metabolite'>
Traceback (most recent call last):
  File "manage.py", line 10, in <module>
    execute_from_command_line(sys.argv)
  File "/srv/venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 364, in execute_from_command_line
    utility.execute()
  File "/srv/venv/local/lib/python2.7/site-packages/django/core/management/__init__.py", line 356, in execute
    self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/srv/venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 283, in run_from_argv
    self.execute(*args, **cmd_options)
  File "/srv/venv/local/lib/python2.7/site-packages/django/core/management/base.py", line 330, in execute
    output = self.handle(*args, **options)
  File "/srv/venv/local/lib/python2.7/site-packages/simple_history/management/commands/populate_history.py", line 65, in handle
    self._process(to_process, batch_size=options['batchsize'])
  File "/srv/venv/local/lib/python2.7/site-packages/simple_history/management/commands/populate_history.py", line 110, in _process
    utils.bulk_history_create(model, history_model, batch_size)
  File "/srv/venv/local/lib/python2.7/site-packages/simple_history/management/commands/_populate_utils.py", line 28, in bulk_history_create
    ) for instance in model.objects.all()]
  File "/srv/venv/local/lib/python2.7/site-packages/django/db/models/base.py", line 572, in __init__
    raise TypeError("'%s' is an invalid keyword argument for this function" % list(kwargs)[0])
TypeError: 'description' is an invalid keyword argument for this function

Environment (please complete the following information):

OS: Ubuntu
Django Simple History Version: django-simple-history==2.0
Django Version: Django==1.11.10

Additional context

Hey @eliUni thanks for submitting this bug. We have a PR open for this fix (#381) that we're going to merge soon that addresses it, but you can look at that branch for an interim workaround. Will comment on this issue when the PR is merged, and when the merge is included in a new release.

When do you thing the merge will be included in the new release?

Going to try to get that PR merged today. Probably have a release with it in the next week or two @eliUni

Sorry thought this was about #355 which I will merge today. I will get the fix for this bug before the next release in ~2 weeks

Hi @eliUni . Fixed this issue in #410 and the fix is in the latest release (2.1.1)


Invalid definition of CSS <snap-block()> and <snap-inline()>

Can you please tell me if you would be able to parse the definitions of <snap-block()> and <snap-inline()> so that they are snap-block( <length> , [ start | end | near ]? ) and snap-inline( <length> , [ left | right | near ]? ), respectively, or if I should request for a proper value definition block (below the float property definition table) on w3c/csswg-drafts?

Ah, good catch. No need to change the spec, they should have been extracted. That's a bug in the extraction logic, the following RegExp should only match functions defined with parameters (right now, it matches snap-block() and thus code thinks it has the right expanded form and does not look into <dd>:
https://github.com/w3c/reffy/blob/9f3b6884e356652b1356bb6c7346965f87982c4d/src/browserlib/extract-cssdfn.mjs#L232


Better validation for missing i18n keys

We are currently missing some keys in the translation .
We should add some way to detect either in dev or compile time where the missing keys are coming from

@esthercheran I notice this issue is marked as done by SorOps, could we close it?


500 in template when showing repository of deleted user

Description
On this instance we have a repository mistercyb/chip8_emulator that somehow still exists despite their user being deleted. Anytime this repo comes up in the Explore tab or in search we get a 500; for instance here in search.
I'm pretty sure this is the same issue as this Discourse thread.
The log suggests an access to the nonexisting user, with
2022/08/19 16:48:39 ...s/context/context.go:232:HTML() [E] [62ffa2c7] Render failed: template: explore/repo_list:29:19: executing "explore/repo_list" at <.Owner.Visibility.IsPrivate>: nil pointer evaluating *user.User.Visibility

(Sorry I can't get debug logs just yet -- I might try later)
Something of note is that the repo folder not only had mistercyb/chip8_emulator.git but also a rogue chip8_emulator.git outside of any user/org folder with no objects or refs. (I deleted it thinking it was the cause of the issue.)
It seems something went wrong as soon as the repo remained when the user was deleted. Currently it's not possible to remove the repository from the web UI because of the error. Any instructions for how to remove it from the command-line would be helpful.
Gitea Version
1.17.0
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.37.2
Operating System
Arch Linux
How are you running Gitea?
Installed from the official Arch Linux repository.
Database
SQLite

The repositories should be deleted before user deleted according the logic in the current codebase.

That seems reasonable. Setting aside the question of whether/how to fix it for now, is there a documented way for me to properly get rid of the rogue repository to unblock the instance without using the web UI?

I think you have to manually delete the repository from database and disk.
I'm going to close this one as it's impossible to address that situation according normal operations.

That's fine with me, but do you have a pointer on how to do that? I'm not exactly familiar with the way Gitea stores repositories on the disk, or the database format.

Maybe it's easier for you to change the repository's owner and then delete the repository from UI.
Select id from repository where owner_name = '<owner_name>' and name = '<repo_name>';

Update repository set owner_id = <new_id>, owner_name = '<new_owner_name>' where id = <above_id>;

Excellent. The owner field was weirdly blanked out (neither an empty string nor null as far as I can tell, which should explain why I had a repo folder outside of any owner folder in the storage) but the ownership change still worked fine. Fortunately the UI allows deleting the repo even when the files are gone, so that's solved. Thank you!


[fix:tests] Fixed flaky expected date in test_password_expiration_mail

Build the expected date in the same way the django template framework does.

coverage: 97.84%. remained the same
when pulling 5ba728436b96c48294644cd6633d63c3c0e4630f on fix-flaky-date-test
into 57d542bfb94689eeaf06947133a78cb77d345d41 on master.


[New Port Request] OpenXLSX

Library name
OpenXLSX
Library description
OpenXLSX is a C++ library for reading, writing, creating and modifying Microsoft Excel® files, with the .xlsx format.
Source repository URL
https://github.com/troldal/OpenXLSX
Project homepage (if different from the source repository)
No response
Anything else that is useful to know when adding (such as optional features the library may have that should be included)
No response

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.

Hello,
Since we can find other issues opened in the past about the same package. Is it possible to reopen the issue again ?
Thanks

+1 for requesting this port.
Here are some of the other issues that have been opened for this request:
#12914
#11555
#15824


[Crosswalk-12][wrt] Update test for rtcore

Impacted TCs num(approved): New 0, Update 1, Delete 0
Unit test Platform: Android
Android test result summary: Pass 1, Fail 0, Block 0

-->
to
-->

Updated

LGTM


how can i get stream url?

can i copy  XCDYoutubeKit StreamUrl link  to AVPlayer()????
can i get link of MPMovieContoller?

@davut5 check out the OS X Demo project specifically, the AppDelegate

Yes, you can. Please take some time to read the documentation and check the existing examples in the demo projects.


Added CMakelists.txt

Associated Issue
Closes Issue #71
Implemented Solution
Added CMakeLists.txt  which can be used with cmake to compile and build
running
cmake CMakeLists.txt
make all

will compile and generate the binary

Thanks for working on this. On quick review, it looks good.
I will test it on my machine later and if it builds properly, I will approve and merge your changes.

@all-contributors please add @npranav7619 for code and maintenance