Unable to :Neogit & :q or :bd w/o error

When I :Neogit

After running :Neogit the current buffer is cloned in a new tab. When I then :bd or :q I see

NVIM v0.6.0-dev+155-g15b736d6e on Neogit 3b5fa08

Hmm, I have suspicions that this is related to an update from another library...

@jc00ke this seems to be related to my recent update in diffview. Do you have diffview installed? If so, diffview and neogit will be incompatible until #188 is merged.
But still, this error is strange. It seems like you only partially updated diffview. Because these errors indicate that you have the updated autocommands, but not the updated lua code. I would try to reinstall diffview if I were you, because the update should not cause these types of errors regardless. It should only mean that you can't open diffview from neogit until my PR is merged.

Thanks @sindrets! I tried reinstalling diffview and it did not work. NBD, I'll disable it for now and I'll watch #188.

Just merged #188.


Was not included in the mobile the description of the incident.

I handled this, in my last pull request.


Jedi: Add return type hint to function hover

Right now, the hover over function calls does only show type hints for its arguments and the docstring, but not its return type hint.

It would be very helpful if the returning type would also be shown in the hover if there is a return type hint.

@tweakimp  Are you using Jedi or Language server? Let us know what you have set for jediEnabled setting.

"python.jediEnabled": true
With "python.jediEnabled": false the hover shows the return type.

For anyone addressing this issue, we probably need to switch to the newer signature APIs in completion.py.

This should fixed in our insiders


Round of general improvements for docs!

Working on my next big contribution I detected some widespread inconsistencies throughout the docs and I'm addressing them in this first PR. It should be a quick one to review/approve, hopefully.

Looks good, @jorgeorpinel ! Merged.

That was fast! I may have another commit with small improvements like these 😋  But I can bundle it with the coming big one (complete dvc cache and dvc cache dir refs).


Add dependabot support

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

Comparison is base (924310f) 95.19% compared to head (c49146c) 95.19%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #50   +/-   ##
=======================================
  Coverage   95.19%   95.19%
=======================================
  Files          23       23
  Lines         749      749
  Branches       29       29
=======================================
  Hits          713      713
  Misses         13       13
  Partials       23       23

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


TS4.1

Hi, with typescript 4.1 we can now type path from objects. i.e. generate a string union of path from an object structure.
This would be really helpful to type the array of keypath.
Here is a post on stackoverflow for the same use:
https://stackoverflow.com/questions/64046160/ts4-1-is-there-a-way-to-define-this-property-path-type-that-isnt-excessively-d
enjoy

I just gave a try and yes… it looks pretty good

I just get a weird error Type instantiation is excessively deep and possibly infinite.
but the POC seems to be functional


remove WM_CHAR message from the queue when peeking

This fixes https://bugs.openmpt.org/view.php?id=1635.

Applied in r18140.


Does baby-beamup server currently/will support Persistent Disk?

Stremio is currently fixing issues with the baby-beamup server, so good, but every addon is restarting and losing data. It would be great to have Persistent Disk.

it doesn't support it, and i don't think it will ever do. that's just the way it's built and changing that won't be easy.


Not able to click or identify any element when logged into the application. Third party application is involved for the authentication purpose.

Hello,
We are automating the Angular application using Protractor ver- 5.3.2.
About application - The application evolves third party authentication media during login.
Issue: When tried running any script , the login is successful every time , but when we landed on the Home page, Protractor is not able to click or identify any element. We tried browser.sleep(); for that matter , but still not result. Tried $timeout and browser.wait() as well. Also when tried to get current URL from Homepage, we did not get any output, but when when we tried to get URL from Login page its working there. Need your help here.

Hi, @pranesh517 ! Please provide your config file and test example.

@IgorSasovets I'm seeing a similar scenario in the tests I'm trying to set up - After it goes through the third party auth process (which is being bypassed, but it still has to pass through it), when I get to the sites home page, the test just hangs indefinitely.  Tried browser.wait() and browser.sleep().  Theres nothing in the console, as it seems to just stop running entirely.  Below is my config and test  (with specific values omitted).  Sorry if I'm hijacking this incorrectly @pranesh517 , please let me know if so.
conf.js:
exports.config = {
seleniumAddress: 'http://localhost:4444/wd/hub',
getPageTimeout: 60000,
allScriptsTimeout: 500000,
framework:'jasmine',
jasmineNodeOpts: {
showColors: true,
defaultTimeoutInterval: 3600000
},
capabilities: {
'browserName': 'chrome'
},
specs: [
'../specs/loginTests.js'
],
};
describe('login', function() {
var Main = require('../pages/main.js');
var Login = require('../pages/login.js');
var Dashboard = require('../pages/dash.js');
var lp = new Login();
var mp = new Main();
var dash = new Dashboard();

beforeEach(function() {
    browser.get('*/Site URL/*')
});

it('should display all page elements', function(){
    expect((mp.Banner).isDisplayed());
    expect((mp.idlogo).isDisplayed());
    expect((lp.introText).isDisplayed());
    expect((lp.introTitle).isDisplayed());
    expect((lp.introDesc).isDisplayed());
    expect((lp.username).isDisplayed());
    expect((lp.password).isDisplayed());
    expect((lp.loginButton).isDisplayed());
    expect((lp.forgotPassword).isDisplayed());
    expect((mp.footer).isDisplayed());
});

it('should allow active admin to log in with valid password', function(){
    lp.username.sendKeys('/*USERNAME*/');
    lp.password.sendKeys('invalidpass');
    expect((lp.loginError).isPresent()).toBe(false);
    lp.loginButton.click();
    expect((lp.loginError).isDisplayed());
    lp.password.clear();
    lp.password.sendKeys('/*PASSWORD*/');
    lp.loginButton.click();
    browser.wait(protractor.ExpectedConditions.visibilityOf(dash.greeting), 5000);
    mp.logOut.click();
});

});

});
Thanks in advance for any insight you can provide!

Thank you @IgorSasovets !  This solution worked perfectly.

@CrispusDH I kinda hijacked the issue, wasn't mine to begin with, so I can't close.

@cswanPWW , not at all)


travis update

In an effort to speed up travis testing:
Removed some intermediate puppet versions that weren't in PE
Enabled travis's container-based testing
FUTURE_PARSER also includes STRICT_VARIABLES (and neither parameter exists in 4.x)
In all the travis tests I've watched run errors are typically either on all tests, old version of ruby, or future parser/strict_variables.  Testing every version seemed like it just made things take longer without any real benefit, your experience may be different though.  (I'd also vote to drop 2.7 testing/support - it's 2 major versions behind)

Thanks!


version numbers are missing on macOS

Checklist

[X] I'm reporting a problem with Chatterino
[X] I've verified that I'm running the most recent nightly build or stable release
[X] I've looked for my problem on the wiki
[X] I've searched the issues and pull requests for similar looking reports

Describe your issue
i've downloaded version 2.3.4 but when selecting it in the macOS Finder, the version numbers are missing.
it seems both the CFBundleVersion and CFBundleShortVersionString entries in your Info.plist file are missing. those are mandatory.

Screenshots
No response
OS and Chatterino Version
Chatterino 2.3.4 on macOS 12

Duplicates #3283


Option to remove dashes

Sometimes there is a requirement to get 32 character UUID (may be I shouldn't call that UUID :) ).
However the UUID generated by this library is 36 chars length. Because of dashes.
Is it possible to add an option to omit dashes? Something like this:
function v4(options, buf, offset) {
    ...
    return buf || bytesToUuid(rnds, 0, options.nodash);
}

function bytesToUuid(buf, offset, nodash) {
  var i = offset || 0;
  var bth = byteToHex;
  return  bth[buf[i++]] + bth[buf[i++]] +
          bth[buf[i++]] + bth[buf[i++]] + (nodash ? '' : '-') +
          ...
}

The dashes are part of the RFC4122 spec:
The formal definition of the UUID string representation provided by the following ABNF [7]:

      UUID                   = time-low "-" time-mid "-"
                               time-high-and-version "-"
                               clock-seq-and-reserved
                               clock-seq-low "-" node

If you need to remove them, uuid().replace(/-/g, '') should do the trick.

@LeoDupont: I have zero interest supporting non-standard id formats.

Just to add to this, this might be useful for downstream client libraries like express-request-id which allow users to pass params to uuid

@jakesyl to re-iterate: This library focuses on what's covered by RFC4122, we have no interest in supporting non-standard formats.
If you are looking for more compact unique identifiers to be used in URLs and don't need UUID compliance you could look into some of the alternatives:

nanoid
flake-id
cuid
ulid

The fact that some of these libraries use base64 encoding instead of hex encoding allows them to encode the same amount of entropy in shorter strings which can be handy when storing & using IDs as opaque strings.


Check handling of concurrent session modifications

Add a test that does this with a session store that rejects concurrent modification, for example by failing a transaction.
How should this be handled? The request then can only fail, but should it be repeated? Or is there some "temporary failure, please try again" HTTP error?

Turns out that typed-session already handles this case and returns an error. We now forward this error out of the session layer, and let the application handle it.


Remove space in kernel spec path

On IPython 3.1, the space was causing problems with loading the logo (even though the kernel spec itself gets loaded).
[W 20:07:18.060 NotebookApp] 404 GET /kernelspecs/julia%200.3/logo-64x64.png

Perhaps this bug exists in IPython notebook itself for kernelspecs with a space in the name.

Ended up talking to @minrk of IPython/Jupyter, he had this to say:

I think we are going to exclude spaces from valid kernelspec directory names, so we don't have to deal with URL escaping.

This fixes up the issue then.

It would probably be more readable if we replaced the space with a hyphen.

It would probably be more readable if we replaced the space with a hyphen, though.

Done. I also removed a fairly outdated comment, now that IJulia is compatible with the current release of IPython and is right on par with the 4.0 release.


Is there a way to supply additional JDBC properties?

I'm trying to get PostgreSQL native enum types working with Hibernate, and the most straightforward solution is to set the stringtype=unspecified property on the JDBC connection (to make the server infer the enum type for the column). However, the JDBC URI is being provided in my VCAP, and the DataSource (and pool) by Connectors. I know there's the MapServiceConnectorConfig, but IIRC there's no hook to add custom entries to it.
Is there any way to insert additional properties to be set on the JDBC DataSource created by Connectors?

@chrylis There are three types of configuration that can be supplied when customizing the DataSource created by Connectors. When using Java configuration like this:
@Bean
public DataSource dataSource() {
    PoolConfig poolConfig = new PoolConfig(5, 30, 3000);

    ConnectionConfig connConfig = new ConnectionConfig("useUnicode=yes;characterEncoding=UTF-8");

    Map<String, Object> properties = new HashMap<String, Object>();
    properties.put("driverClassName", "test.driver");
    properties.put("validationQuery", "test validation query");
    properties.put("testOnBorrow", false);

    DataSourceConfig dbConfig = new DataSourceConfig(poolConfig, connConfig, properties);
    return connectionFactory().dataSource("my-own-personal-sql", dbConfig);
}

the DataSource will be configured as follows:

PoolConfig will be used to set well-known properties on the DataSource bean related to connection pools (min pool size, max pool size, max wait time). Connectors knows the names of the properties for each concrete DataSource implementation (Apache Commons DBCP and DBCP2, Tomcat JBCP Pool, HikariCP) and sets the appropriate properties from this object.
ConnectionConfig will be used to set the value of a connectionProperties property on a concrete DataSource object that supports this property (Apache Commons DBCP and DBCP2, Tomcat JBCP Pool). See Commons DBCP2 as an example. This string is typically passed on to the JDBC driver.
Map<String, Object> properties will be used to set properties on the concrete DataSource object. The keys in this map correspond to bean properties/setter methods on the DataSource object, and are specific to the concrete implementation being used.

See Detection and Prioritization of DataSource Implementations in the documentation for details on how a concrete DataSource implementation is chosen.
It looks to me like ConnectionConfig is what you need, but I can't say for sure.
The documentation should include this level of detail. I've created #203 to address that.

As far as I remember (I was involved back around 1.1), the JavaConfig you showed has to live in an extends AbstractCloudConfig in order to have access to the connectionFactory() (which is loaded with the ServiceInfos), and if I supply my own I have to include explicit @Beans for every service I want bound--I can't simply add the DataSource override and have everything else still autocreated. Am I correct on this?

the JavaConfig you showed has to live in an extends AbstractCloudConfig in order to have access to the connectionFactory()

You can either extend AbstractCloudConfig and use connectionFactory().dataSource(...); or you can avoid extending AbstractCloudConfig and create the necessary beans yourself - something like:
@Configuration
@Profile("cloud")
public class DataSourceConfiguration {
  @Bean
  public Cloud cloud() {
    return new CloudFactory().getCloud();
  }

  @Bean
  @ConfigurationProperties(DataSourceProperties.PREFIX)
  public DataSource dataSource() {
    ....
    DataSourceConfig dbConfig = new DataSourceConfig(poolConfig, connConfig, properties);
    return cloud.getSingletonServiceConnector(DataSource.class, dataSourceConfig)
  }
}

if I supply my own I have to include explicit @Beans for every service I want bound

Unfortunately, that's still true - auto-configuration is all-or-nothing.

@chrylis I'm going to close this issue since it's been inactive for a while. Feel free to re-open if there's something you'd like to see done on this.


insert video command for document to video

introduces a link to the gallery wheere the video can be found. warning sign next to the video (can cause sync issues)


tenantServerAction acts on instance without checking for tenant

When an instance list is given to act on, tenantServerAction should check to see if an instance belongs to a tenant before allowing the action to proceed.

I worked on this, I still need to make a test before sending pull request,  meanwhile you can see the commit  in my fork https://github.com/leoswaldo/ciao/commit/1a1caa5d81fe82edcf06ecca404138cdf1c446d1


Some typos in JulianDate

and I added some files to the gitignore for people like me who use IntelliJ as IDE, if you don't want that let me know.
FYI I use this package to find the optimal angle for the solar panels at my home, as they can move in one direction. Thanks for all the work and good job at making it as fast as Mathematica (proprietary language I previously used the sun position of)  :)

Coverage remained the same at 91.492% when pulling 8b50cf1fb14e98a30417d9357de3ee3076a706f2 on PHPirates:typos into 3c62c2544a47b3a5716b381c4780e4f684ee968e on aligusnet:master.

thanks!


null lists

this question was raised by @chrdebru
how do we handle the generation of null lists?
this issue might be related to kg-construct/rml-fno-spec#16

The spec now added information and examples on how to deal with empty lists. Empty containers are easier.

I have added a small note on empty lists when they are identified with an IRI or blank node identifier (they should be replaced with rdf:nil).
Should I add examples of those with containers?

Guess the examples you added are illustrative enough.

Addressed in the current spec.


Add documentation related build and checks to GitHub workflows

There's nothing in the pull request verification that checks if the command-line references are up-to-date. Add a job to the pull request update workflow that verifies this.
There's nothing in the release build workflow that generates the documentation and uploads the documentation artifacts so that they can be published. Add a step to the build release job that generates the documentation so that it can be published to swift.org.

@swift-ci test macOS


Add another troubleshooting step

Learn Build status updates of commit 814ba24:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/active-directory/app-proxy/application-proxy-troubleshoot.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@kenwith,
Can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Thank you for this information. I have captured in a larger update that will be going out.
#please-close


Update Version OS Chart to Include .Net 6

The chart designating which tech stacks for Azure Functions work for different hosted OSes should reference .Net 6 in addition to .Net Core 3.1 and .Net 5.

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

ID: 06436739-8586-23b1-6b69-1960e93c26be
Version Independent ID: 8f7abfae-cf6d-56cb-1d20-f02041a6f6d9
Content: Azure Functions scale and hosting
Content Source: articles/azure-functions/functions-scale.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

@cwelliotte Thanks for your feedback! We will investigate and update as appropriate.

Linked PR addresses the issue. Thanks for the heads-up about this issues. #please-close


Custom Formly Model

Question
Is it possible to have a custom formly model?
For example, for the following formly config:
[
    {
        "key": "a1",
        "type": "repeat",
        "fieldArray": {
            "fieldGroup": [
                {
                    "type": "textarea",
                    "key": "ta1"
                }
            ]
        }
    },
    {
        "key": "ta2",
        "type": "textarea"
    }
]

The corresponding formly model would be this:
{
    "a1": [
        {"ta1": "value for ta1 index 0"},
        {"ta1": "value for ta1 index 1"},
        {"ta1": "value for ta1 index 2"}
    ],
    "ta2": "value for ta2"
}

I would like to have custom Ids associated with each node, especially for array nodes. I have a requirement to relate fields in an array to a external model, and since formly does not keep an ID with array item, it makes it difficult in many cases to find which array items have been deleted/modified.
I would like to store the formly model as something like this
[
    {
        key: "a1",
        id: "UNIQUE_GUID",
        type: "repeat",
        children: [
            {
                id: "UNIQUE_GUID",
                key: "",
                type: "group",
                children: [
                    {
                        id: "UNIQUE_GUID",
                        key: "ta1",
                        type: "textarea",
                        value: "value for ta1 index 0"
                    }
                ]
            },
            {
                id: "UNIQUE_GUID",
                key: "",
                type: "group",
                children: [
                    {
                        id: "UNIQUE_GUID",
                        key: "ta1",
                        type: "textarea",
                        value: "value for ta1 index 1"
                    }
                ]
            },
        ]
    },
    {
        id: "UNIQUE_GUID",
        key: "ta2",
        type: "textarea",
        value: "value for ta2"
    }
]

Please let me know if this is possible.  Thank you!

I'm assuming that you want to associate an id for array node, so todo that you just need to add an extra field which contains the value of id:
{
  "key": "a1",
  "type": "repeat",
  "fieldArray": {
    "fieldGroup": [
      { "type": "textarea", "key": "ta1" },
+     { "key": "id", "defaultValue": "UNIQUE_GUID" }
    ]
  }
},


Possible bug in capturing: Frame didn't arrive within 5000 / Error occured during execution of the processing block!

Required Info

Camera Model
D415

Firmware Version
<IP_ADDRESS>

Operating System & Version
Debian 8

Kernel Version (Linux Only)
4.9.0-8-amd64

Platform
PC

SDK Version
2

Language
python

Segment
others

Issue Description
Hello everyone. I have been working lately with the python bindings of the realsense2 library with the D415 model in Linux, and found something that might be a bug (or likely underdocumented error) when accumulating frames.
When I append the outputs from pipeline.wait_for_frames() to different python lists, it looks like it blocks the depth or color frame object and after 16 frames of appending it throws an error "Frame didn't arrive within 5000". If we try to align to the color frame, the error is "Error occured during execution of the processing block!".
However, this is easily overcome when appending a copy of said frames, which looks like a good solution and not too costly performance-wise. Before thinking of any PR to the main repo, I wanted to know whether this behavior was expected.
I am yet to understand completely the nature of the bug, but here I leave you the gists that trigger both errors, and the copy workaround.
If it helps, I found is not time-related but rather number of frames related, and it also is not dependent (I haven't checked all the range though) on the resolution.
Pablo
https://gist.github.com/pabloalcain/65610225deb0ae9335e66e1a6c755c57

@pabloalcain Sorry for the late reply. What's the SDK version when you get this issue? Did you get chance to try the latest SDK version and FW to see if you still get the issue? Thanks!

@pabloalcain Could you please update? Thanks!

@pabloalcain Did you still have issue with the latest SDK and FW? Could you please feedback?Thanks!

@pabloalcain Thanks for the feedback! Looking forward to your update. Thanks!

@pabloalcain Any update from your side now? Thanks!

@pabloalcain Did you get the HW now to test? Looking forward to your update. Thanks!

@pabloalcain Sorry that we didn't get response from you for weeks. Will close this if no other questions. Please feel free to create new one if you still have any other questions/issues. Thanks!


Crash fix for Netflix decrypt

Summary: Crash fix for Netflix decrypt
Type: Fix
Test Plan: None
Jira: None

https://jenkins.code.rdkcentral.com/job/build-component/65812/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/9f1c686600eb3b91fe66b29078bcaaa2

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141733/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141734/


Requiremets.txt file

Hi
Is there any requirements.txt file present for flumberbuckets?

Hi
No I actually forgot to include requirements.txt file in the repository, to
be honest I don't think it's needed as you can see in the installation
instructions that only a few dependencies are required for flumberbuckets!
threading, argparse, subprocess, os, sys, queue are part of standard
library of python, they're pre-installed
While botocore is not part of default python installation but on most Linux
systems you'll get it when you install aws-cli package.
Once you get the aws-cli tool working that is you can run command like
aws s3 ls s3://flaws.cloud
and get output like the following which shows the files currently in the
bucket then you can start using the script
2017-03-14 08:30:38       2575 hint1.html
2017-03-03 09:35:17       1707 hint2.html
2017-03-03 09:35:11       1101 hint3.html
2018-07-10 22:17:16       3082 index.html
2018-07-10 22:17:16      15979 logo.png
2017-02-27 07:29:28         46 robots.txt
2017-02-27 07:29:30       1051 secret-dd02c7c.html
Currently I'm using arch linux and I have following version of aws-cli
installed
bash_command > aws --version
aws-cli/1.16.303 Python/3.8.0 Linux/5.4.3-arch1-1 botocore/1.13.39
In Arch Linux you can install aws-cli using following command
sudo pacman -S aws-cli
For other Linux distributions you can just google on how to install aws-cli
on them or refer to this article
https://www.slashroot.in/how-to-install-and-configure-aws-cli-on-ubuntu-16-04
And yes you'll have to install aws-cli because after flumberbuckets is done
you'll have to manually check if contents of buckets are sensitive or not
for that you'll end up using aws-cli so go ahead and install aws-cli and
botocore will come along with it. So naturally all the dependencies and
requirements would be taken care of :)
Hope this resolved your query.
On Tue, Dec 24, 2019 at 5:29 PM AkshayJainG<EMAIL_ADDRESS>wrote:

Hi
Is there any requirements.txt file present for flumberbuckets?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fellchase/flumberboozle/issues/1?email_source=notifications&email_token=AC25Z3H5AKSV2J4EES4EOKTQ2H2TVA5CNFSM4J65X5N2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ICP4X5A,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AC25Z3BB7Y7TYDRVP2MEPEDQ2H2TVANCNFSM4J65X5NQ
.

Thanks for bringing this issue to my notice
Check out new installation instructions on the blogpost :) it has been updated. If you encounter any new problem then create new issue


Feature idea : picture gallery

It would be pretty awesome to have a nice way of fitting 3 to 5 images on a slide...
Or maybe you have a way to do it simply?
Thank you.

You can do it in different ways :)

ul class="flexblock gallery" + .overlay
img class="alignleft", img class="alignleft size-80", img class="alignright size-50"...

Logos (clients): ul class="flexblock clients"

Thanks @jlantunez
I'll try the different options. I'm basically looking to do something clean without having to resize all the different pictures, like this:


Date Forward Error

@yegor256, Looks like you forgot to change the year in this commit in deadline for paper/abstract submission from 2023 to 2024

@advancedAlpaka indeed, my fault, fixed


[Help] Issue pulling in svgr types.

I recently updated to module version 2.2.1 and noticed that I now have a hard-to-figure-out error in my tsconfig.json in relation to this plugin.
Interestingly, this is fine in my vite.config.ts. Is it expected to include tsconfig.json in it's own includes?
tsconfig.json
Cannot find type definition file for 'vite-plugin-svgr/client'.
  The file is in the program because:
    Entry point of type library 'vite-plugin-svgr/client' specified in compilerOptions

at
{
  "compilerOptions": {
    "target": "ESNext",
    "lib": ["dom", "dom.iterable", "esnext"],
    "types": ["vite/client", "@types/vite-plugin-svgr/client", "@types/react", "jest"],
    ...
    }
  "files": [ // is it expected to include tsconfig.json here?
    "vite.config.ts",
    "public/source-env.js",
  ],
  "include": [
    "src",
    "env/*.js",
    "src/**/*.test.ts",
  ]
}

package.json
"devDependencies": {
    "vite-plugin-svgr": "^2.2.1",
    ...
}

@mikeymop
Is this resolved?
Seems like changing @types/vite-plugin-svgr/client -> vite-plugin-svgr/client in compilerOptions.types might help.


Refactor/bybit parpetual latest standards

Before submitting this PR, please make sure:

[X] Your code builds clean without any errors or warnings
[X] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
This PR refactors Bybit perpetual to the latest standards. It also creates the PerpetualDerivativePyBase, which brings the ExchangePyBase approach to the perpetual exchanges.
Tests performed by the developer:

Test cases for the new code

Tips for QA testing:
Everything about this connector should be tested, plus please ensure that the other perpetual connectors can start correctly — no need to thoroughly test them.
Proposal link: https://snapshot.org/#/hbot-prp.eth/proposal/0x9694fa477470dcae142476786db58b3860024e19f77165729b377ceb2bfa52da

Test performed:

clone and Install featured branch = OK
Connect a valid Bybit Perpetual API = OK
Balance output = OK
Create a strategy = OK
Import a strategy = OK
Starting strategy = OK
Status (--live) command = OK
Order Cancelation = OK
Data integrity = OK
Broker ID = OK
Data aggregate = OK
Order filled events = OK for trade to open
Verify fee calculation = OK
pending test:
Order filled events - pending for trades to close
Funding time
Run a quick test on other Perpetual connectors
test bybit perpetual testnet

Hi @petioptrv, please resolve conflicts in this PR. Thanks!

@JeremyKono, I've updated the PR. There were non-trivial changes done to the code to bring it up to speed with changes in the connector base class, so I would suggest a thorough re-test by QA

PR update:

Tested latest commit
Did same test for Spot-perpetual arbitrage and Perpetual market making (some ongoing issues spotted)

Cloned and compiled feature branch = OK
Created docker image and container = OK
Started client and created password = OK
Connected API with no issues = OK
Created/started PerpetualMM strategy = OK
Checked status and status --live = OK
Data integrity verified synced with exchange = OK
Filled and cancel orders work accordingly = OK
Fees matched with the exchange = OK
Filled orders are tracked via datadog = OK
Handlig Funding time = OK
Checking history = OK
Filled orders on History, CSV Data file match with the exchange = OK
Quick test on other Perpetual connectors = OK
Tested on AWS Source and Docker Build = OK
Spot-perpetual arbitrage - had an issues that also replicated on dev (not related to this PR):
Failed to perform an arbitrage using live exchange as a spot connector #5541,
bot showing low balance for base asset on derivative exchange #5533

Just minor checking when setting up markets for bybit_perpetual. I noticed there are some tokens that should not be listed sample is BTC-USD, XRP-USD, etc.. and it seems these tokens is part of Inverse contracts. Crosschecked with dev branch and the results are below:
Screen recording from this PR
https://user-images.githubusercontent.com/73840223/182344203-e77ab1f9-9509-4ef6-9b66-aec6bc3713a5.mp4
Screenshot from development branch:


Fixes a11y error due to missing label for errors pane

Description
Fixes an accessibility error in TSDoc Playground due to missing <label> for <textarea> since it is an input type and therefore required. Providing a label helps users that are using screen-reader technology navigate the page.
The Change
I replaced the in-line text node with a <label> and removed the <br> to preserve the same spacing.
Before

After

Reference: https://www.w3.org/TR/WCAG20-TECHS/H44.html

:shipit: :


All sites are currently appearing in the iCal export

https://www.pillarpointstewards.com/shifts/calendar-instructions/ is the page linking to the shift calendars.

URLs: https://github.com/natbat/pillarpointstewards/blob/b0768349ddcb30ce2cd678fc5914aba8f6fcae01/pillarpointstewards/config/urls.py#L36-L37
Python code: https://github.com/natbat/pillarpointstewards/blob/b0768349ddcb30ce2cd678fc5914aba8f6fcae01/pillarpointstewards/shifts/views.py#L276-L299
Tests: https://github.com/natbat/pillarpointstewards/blob/b0768349ddcb30ce2cd678fc5914aba8f6fcae01/pillarpointstewards/shifts/tests.py#L70-L88

Yeah, neither of those ics view functions attempt to filter by membership.

The fix is now live. You will now see all shifts from all teams you are a member of - so if you are only a member of one team you'll only get one set of shifts.
Maybe we should have separate URLs for each team though? Natalie and I are members of multiple teams but we only want Pillar Point shifts in our calendars.


fix(macos): properly set icon_as_template and menu_on_left_click

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes
[ ] No

Checklist

[ ] This PR will resolve #___
[x] A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
[ ] I have added a convincing reason for adding this feature, if necessary
[ ] It can be built on all targets and pass CI/CD.

Other information
This is a regression from #793

https://github.com/tauri-apps/tauri/issues/7925
Also need to fix set_menu to this
  pub fn set_menu(&mut self, tray_menu: &Menu) {
    unsafe {
      (*self.tray_target).set_ivar(TRAY_MENU, tray_menu.menu);
      self.ns_status_bar.setMenu_(tray_menu.menu);
      let () = msg_send![tray_menu.menu, setDelegate: self.ns_status_bar];
    }
    self.tray_menu = Some(tray_menu.clone());
  }

Sorry I'm outside can't push it right now


Handle edge case where contract ends before it starts

Coverage decreased (-0.01%) to 32.185% when pulling d5f90bcda225c95bd4b9765e0c476b8c22a46bba on qingweibinary:qingwei/fix into e9690f319eb1f95805605f63fb85fa4c99996039 on binary-com:master.

Coverage decreased (-0.01%) to 32.185% when pulling d5f90bcda225c95bd4b9765e0c476b8c22a46bba on qingweibinary:qingwei/fix into e9690f319eb1f95805605f63fb85fa4c99996039 on binary-com:master.


Couldn't find UUID for device iPhone Retina (4-inch) with SDK 8.3

I am trying to run iOSDriver script against iOS iPhone simulator.
Configurations:

iOS Driver server: ios-server-standalone-0.6.6-SNAPSHOT-e1bb7351e221c753ae72895cbd509337ace395a5-jar-with-dependencies.jar (Compiled from source)
iOS Driver client: ios-client-0.6.6-SNAPSHOT.jar (Compiled from source)
XCode version: 6.3.2
iOS version: 8.3
Target app iOS version: 8.3

I get the following error log when I trigger my script:
INFO NewSessionNHandler.safeStart  The server cannot run Capabilities [{simulator=true, timeHack=false, language=en, CFBundleIdentifier=com.ebayent.iphone, locale=en_GB, platform=MAC, variation=Retina4, CFBundleVersion=654, instruments=true, acceptSslCerts=false, sdkVersion=8.3, CFBundleName=eBay, device=iphone}] at the moment.Couldn't find UUID for device iPhone Retina (4-inch) with SDK 8.3
Can someone help? Do I need to pass the UUID for the simulator from somewhere?

I am running into same issue on Xcode 6 SDK  8.4


Should still run when DNS fails to resolve

for whatever reason my server can't resolve itself foo.example.com and so caddy bails instead of listening on <IP_ADDRESS> to handle vhost for foo.example.com.

I think this is valid. To see how this is handled in nginx, for example, I added this to nginx.conf:
server {
    listen 8081;
    server_name foobar.fizz;
    root /Users/matt/Sites/matt.life;
}

I ran nginx then added foobar.fizz to my /etc/hosts file. When I loaded foobar.fizz:8081 in my browser, the server responded with the site. I then tried using <IP_ADDRESS>:8081 and localhost:8081 in my browser, and both also worked. It seems that nginx listens on the wildcard host.
We can still print a warning if the name does not resolve. Any objections? If not, I can go forward with this.

(I bet we can knock out this and #136 at the same time)


Charms fail install hook if python3-setuptools > 60

On machines that have (system-wide) setuptools version 60 or higher, installation fails with:
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install Traceback (most recent call last):
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/var/lib/juju/agents/unit-ovn-chassis-0/charm/hooks/install", line 8, in <module>
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     basic.bootstrap_charm_deps()
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/var/lib/juju/agents/unit-ovn-chassis-0/charm/lib/charms/layer/basic.py", line 202, in bootstrap_charm_deps
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     _update_if_newer(pip, pre_install_pkgs)
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/var/lib/juju/agents/unit-ovn-chassis-0/charm/lib/charms/layer/basic.py", line 362, in _update_if_newer
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     installed = _load_installed_versions(pip)
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/var/lib/juju/agents/unit-ovn-chassis-0/charm/lib/charms/layer/basic.py", line 278, in _load_installed_versions
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     pip_freeze = check_output([pip, 'freeze']).decode('utf8')
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/usr/lib/python3.10/subprocess.py", line 421, in check_output
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     return run(*popenargs, stdout=PIPE, timeout=timeout, check=True,
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/usr/lib/python3.10/subprocess.py", line 503, in run
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     with Popen(*popenargs, **kwargs) as process:
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/usr/lib/python3.10/subprocess.py", line 971, in __init__
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     self._execute_child(args, executable, preexec_fn, close_fds,
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install   File "/usr/lib/python3.10/subprocess.py", line 1863, in _execute_child
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install     raise child_exception_type(errno_num, err_msg, err_filename)
unit-ovn-chassis-0: 16:07:11 WARNING unit.ovn-chassis/0.install FileNotFoundError: [Errno 2] No such file or directory: '/var/lib/juju/agents/unit-ovn-chassis-0/.venv/bin/pip'

I think it's connected to virtualenv using different layouts, influenced by different versions of setuptools as described here https://github.com/pypa/setuptools/issues/3278. Charm fails to find pip because instead of usual <venv>/bin/pip it's located in <venv>/local/bin/pip

The problem seems to go deeper than just difference in layout. Virtual environments created on machines with virtualenv 20.13.0+ds and python3-setuptools 68.1.2-2ubuntu1 do not seem to isolate at all. Creating clean venv and running pip freeze shows system-wide dist packages and using pip from the virtual environment to install packages attempts to mess with the system packages as well:
pip install -U --force-reinstall --no-index --no-cache-dir -f wheelhouse PyYAML==6.0.1
Looking in links: wheelhouse
Processing ./wheelhouse/PyYAML-6.0.1-cp310-cp310-linux_x86_64.whl
Installing collected packages: PyYAML
  Attempting uninstall: PyYAML
    Found existing installation: PyYAML 6.0.1
ERROR: Cannot uninstall PyYAML 6.0.1, RECORD file not found. Hint: The package was installed by debian.


Add getframe

This PR depends on the new IO system -- that should be merged first.
This PR adds a new command-line too, dcm-getframe, which reads a single frame out of a WSI DICOM. This is supposed to be useful for testing DcmBOT and libdicom's ability to handle the various flavours of WSI that are being used.
Sample output with the largest level in the 3D Histech sample WSI DICOM (this file is 1.4GB and contains over 10,000 frames):
$ time dcm-getframe -o x.jpg Histech\^Theresa\ \[1473843\]/2.25.111454577819918515926489701058493557552.dcm 5000

real	0m0.011s
user	0m0.000s
sys	0m0.004s

So it's able to parse the DICOM file, extract and parse the basic offset table, fetch the tile, and write it to disk in 11ms. Pretty good I think!
If you use -v (verbose) it dumps some extra extra information:
$ time dcm-getframe -v -o x.jpg Histech\^Theresa\ \[1473843\]/2.25.111454577819918515926489701058493557552.dcm 7777
INFO     [Tue Feb 14 13:57:19 2023] - Read file 'Histech^Theresa [1473843]/2.25.111454577819918515926489701058493557552.dcm'
INFO     [Tue Feb 14 13:57:19 2023] - Read metadata
INFO     [Tue Feb 14 13:57:19 2023] - Read BOT
INFO     [Tue Feb 14 13:57:19 2023] - Read Basic Offset Table value.
INFO     [Tue Feb 14 13:57:19 2023] - Read frame 7777
INFO     [Tue Feb 14 13:57:19 2023] - frame number = 7777
INFO     [Tue Feb 14 13:57:19 2023] - length = 220848 bytes
INFO     [Tue Feb 14 13:57:19 2023] - rows = 1024
INFO     [Tue Feb 14 13:57:19 2023] - columns = 1024
INFO     [Tue Feb 14 13:57:19 2023] - samples per pixel = 3
INFO     [Tue Feb 14 13:57:19 2023] - bits allocated = 8
INFO     [Tue Feb 14 13:57:19 2023] - bits stored = 8
INFO     [Tue Feb 14 13:57:19 2023] - high bit = 7
INFO     [Tue Feb 14 13:57:19 2023] - pixel representation = 0
INFO     [Tue Feb 14 13:57:19 2023] - planar configuration = 0
INFO     [Tue Feb 14 13:57:19 2023] - photometric interpretation = YBR_FULL_422
INFO     [Tue Feb 14 13:57:19 2023] - transfer syntax uid = 1.2.840.10<IP_ADDRESS>.50

real	0m0.006s
user	0m0.006s
sys	0m0.000s

The written JPEG files seem correct.
It fails for the sample Leica DICOMs:
$ dcm-getframe -o x.jpg ~/Leica_Dicom.dcm/<IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm 1
ERROR    [Tue Feb 14 14:08:14 2023] - Parse error: Reading of Data Element failed - Encountered unexpected Value Multiplicity 4 for Data Element '0040A160'

They don't use TILED_FULL so I guess the issues is in there somewhere. I noticed what look like some problems in extended BOT handling, which could also be the issue. But we can come back to that.
With libdicom reading frames from at least some slides nicely, I'll move on to adding a DICOM vendor to openslide and circle back to libdicom later.

@hackermd you might be interested in the timings above ^^

@hackermd you might be interested in the timings above ^^

Very impressive! Glad to see that the implementation is that fast. Great work @jcupitt!!

It fails for the sample Leica DICOMs:
dcm-getframe -o x.jpg ~/Leica_Dicom.dcm/<IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm 1
ERROR    [Tue Feb 14 14:08:14 2023] - Parse error: Reading of Data Element failed - Encountered unexpected Value Multiplicity 4 for Data Element '0040A160'

They don't use TILED_FULL so I guess the issues is in there somewhere. I noticed what look like some problems in extended BOT handling, which could also be the issue. But we can come back to that.

The error appears to be due to a standard conformity issue. The attribute indeed has an incorrect Value Multiplicity.
@dclunie do you have an image with TILED_SPARSE dimension organization handy that is fully standard compliant and would be suitable for testing frame-level data access?

(0040,A160) is Text Value, which has a VR of UT, which by definition does not have multiple values (any embedded backslash in the text is not a value delimiter), so whatever parser is saying "unexpected Value Multiplicity 4" is incorrect.

Yes, looks like UT is trying to parse the argument. I've made an issue for this:
https://github.com/ImagingDataCommons/libdicom/issues/39

... I fixed it on this branch. It now reads the Leica DICOMs correctly, though it can't fetch frames as there's no BOT.
With a largeish (710mb) Leica DICOM level I see:
$ time dcm-getframe -v -o x.jpg <IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm 1
INFO     [Thu Feb 16 10:27:48 2023] - Read file '<IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm'
INFO     [Thu Feb 16 10:27:48 2023] - Read metadata
INFO     [Thu Feb 16 10:27:49 2023] - Read BOT
INFO     [Thu Feb 16 10:27:49 2023] - Basic Offset Table is empty.

real	0m1.017s
user	0m0.472s
sys	0m0.545s

Parsing the sparse tile metadata is extremely painful. We could probably speed the parser up a bit more.

... I fixed it on this branch. It now reads the Leica DICOMs correctly, though it can't fetch frames as there's no BOT.

You can't depend on the presence of BOTs - you need to be able to build your own if absent (with lazy instantiation on the 1st get frame call if you want to defer the time it takes to do this unless it turns out to be necessary).
This is not too hard if there is one fragment per frame - just walk all the Items in the PixelData noting their offset as you go. If there is not a match between the NumberOfFrames and the number of items (+1 for the empty BOT item), then you have fragmentation (more than one item per frame) and you need to figure out which items are the start of frames (or which is an end item meaning the next is a start item - I do this for JPEG-family Transfer Syntaxes by walking back from the end of each item looking for JPEG EOI marker segments (accounting for trailing padding to an even byte length) rather than walking forward parsing the JPEG bitstream.

Thanks for the tips, David.

This is not too hard if there is one fragment per frame

Yes, I do this with fo-dicom to read Leica files, then cache the offset maps for later reuse.

I do this for JPEG-family Transfer Syntaxes by walking back from the end of each item looking for JPEG EOI marker segments

Ah good idea! Thanks.

... I fixed it on this branch. It now reads the Leica DICOMs correctly, though it can't fetch frames as there's no BOT.

You can't depend on the presence of BOTs - you need to be able to build your own if absent (with lazy instantiation on the 1st get frame call if you want to defer the time it takes to do this unless it turns out to be necessary).

@jcupitt We have implemented the dcm_file_build_bot() for this purpose. The getframe program will need to check for the presence of a BOT and none is found, try to build it instead of failing right away.

The getframe program will need to check for the presence of a BOT and if none is found, try to build it instead of failing right away.

Ah gotcha, I'll change it. There seem to be a few problems with build bot, I'll try to fix them too.

Oooof that was painful, but it seems to work now, with and without a BOT. On a TILED_FULL image with a BOT I see:
$ dcm-getframe -v -o x.jpg ../Histech\^Theresa\ \[1473843\]/2.25.231896281549030369487042280447616527925.dcm 2
INFO     [Sat Feb 18 17:22:48 2023] - Read file '../Histech^Theresa [1473843]/2.25.231896281549030369487042280447616527925.dcm'
INFO     [Sat Feb 18 17:22:48 2023] - Read metadata
INFO     [Sat Feb 18 17:22:48 2023] - Read BOT
INFO     [Sat Feb 18 17:22:48 2023] - Read Basic Offset Table value.
INFO     [Sat Feb 18 17:22:48 2023] - Read frame 2
INFO     [Sat Feb 18 17:22:48 2023] - frame number = 2
INFO     [Sat Feb 18 17:22:48 2023] - length = 221482 bytes
INFO     [Sat Feb 18 17:22:48 2023] - rows = 1024
INFO     [Sat Feb 18 17:22:48 2023] - columns = 1024
INFO     [Sat Feb 18 17:22:48 2023] - samples per pixel = 3
INFO     [Sat Feb 18 17:22:48 2023] - bits allocated = 8
INFO     [Sat Feb 18 17:22:48 2023] - bits stored = 8
INFO     [Sat Feb 18 17:22:48 2023] - high bit = 7
INFO     [Sat Feb 18 17:22:48 2023] - pixel representation = 0
INFO     [Sat Feb 18 17:22:48 2023] - planar configuration = 0
INFO     [Sat Feb 18 17:22:48 2023] - photometric interpretation = YBR_FULL_422
INFO     [Sat Feb 18 17:22:48 2023] - transfer syntax uid = 1.2.840.10<IP_ADDRESS>.50

And on a BOT-less Leica dcm I see:
$ dcm-getframe -v -o x.jpg <IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm 17
INFO     [Sat Feb 18 17:23:24 2023] - Read file '<IP_ADDRESS>.4.1.36533.992031277215326518414019311377114211255162.dcm'
INFO     [Sat Feb 18 17:23:24 2023] - Read metadata
INFO     [Sat Feb 18 17:23:25 2023] - Read BOT
INFO     [Sat Feb 18 17:23:25 2023] - Basic Offset Table is empty
INFO     [Sat Feb 18 17:23:25 2023] - Build BOT
INFO     [Sat Feb 18 17:23:25 2023] - Read frame 17
INFO     [Sat Feb 18 17:23:25 2023] - frame number = 17
INFO     [Sat Feb 18 17:23:25 2023] - length = 5506 bytes
INFO     [Sat Feb 18 17:23:25 2023] - rows = 256
INFO     [Sat Feb 18 17:23:25 2023] - columns = 256
INFO     [Sat Feb 18 17:23:25 2023] - samples per pixel = 3
INFO     [Sat Feb 18 17:23:25 2023] - bits allocated = 8
INFO     [Sat Feb 18 17:23:25 2023] - bits stored = 8
INFO     [Sat Feb 18 17:23:25 2023] - high bit = 7
INFO     [Sat Feb 18 17:23:25 2023] - pixel representation = 0
INFO     [Sat Feb 18 17:23:25 2023] - planar configuration = 0
INFO     [Sat Feb 18 17:23:25 2023] - photometric interpretation = YBR_FULL_422
INFO     [Sat Feb 18 17:23:25 2023] - transfer syntax uid = 1.2.840.10<IP_ADDRESS>.50

Thanks for your efforts @jcupitt. I can image that it was painful, but am glad that you figured it out. At some point, we will need to take the extended offset table into account, too. However, we may want to leave that for another day.


Support for building Debian packages

It would be useful if it was possible to make Debian packages from a project using ros2_rust. There's official documentation for doing this, but bloom-generate claims to not support ament_cargo builds, along with rclrs and rosidl_runtime_rs not being in rosdep.
My project at least doesn't seem to have ended up with any dynamic dependencies outside the standard ROS libs, so if that's the common case it might not be that complicated to support (dependencies can mostly be ignored?)
There are other potential distribution methods too:

Copying source and building on-target - rough for robots with weak CPUs
Just copying binaries around - as mentioned, seemingly only standard runtime deps, so this is ugly but functional
Docker - works fine, though I had some weird issues with --net host and FastDDS (unrelated to ros2_rust)

Of these, I think debian packages are the best option. What work has to be done for bloom to support cargo; if that's even possible before being in the core distribution?

@ashquarky thanks for your interest in ros2-rust. Adding support for building Debian packages out of ros2-rust is not currently possible, we'd need at least these two before we can do that:

Add support for Cargo to rosdep. No easy feat, we talked with the maintainers of the ROS buildfarm and they raised several concerns about it,
Include the Rust toolchain in the ROS Buildfarm. The ROS buildfarm builds packages without access to the internet, as a security measure, we'd need to find a way to "freeze" dependencies so that Cargo does not download them

As a first step, we want to submit rosidl_runtime_rs to the ROS buildfarm, as it's just a regular ROS package written in Python, and eventually enable it as one of the default message generators. Before that we want to add support for actions (see https://github.com/ros2-rust/ros2_rust/pull/295), which would bring it to pretty much feature parity with other message generators. Once we have that included in the standard ROS distribution, all messages will be generated as Rust structs and although rclrs will still be distributed as a crate, the barrier to entry for having ROS 2 applications written in Rust will be much lower.


Add d key in commit view for diffview integation

I didn't see any tests regarding the diffview integration, that's why this commit doesn't add any.

Ahh, thanks for this. Didn't notice it was missing. But, it shouldn't go to the diffview integration directly, but the diff popup. I took care of this on the nightly branch, so no worries. Thanks for the heads up :)


replace twitter endpoints with V2

now twitter start to only support new clients registered in v2; we need to move all the endpoints to version 2 then
authentication part is already coded in twitter_auth_oauth2.js

https://developer.twitter.com/en/docs/twitter-api/tweets/lookup/introduction

V2 means we might need to pay. Need to study the pay plan

maybe keep v1 since it's still working but roll out the v2 simultaneously


The repository does not have a release file for Mint.

W: The repository 'https://zencashofficial.github.io/repo serena Release' does not have a Release file.
N: Data from such a repository can't be authenticated and is therefore potentially dangerous to use.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: Failed to fetch https://zencashofficial.github.io/repo/dists/serena/main/binary-amd64/Packages  404  Not Found

Change your mint sources zencash release to xenial if you do want to use the wallet for now, until the repository is fixed.

I think the right place to report this issue is the ZEN APT repository project:
https://github.com/ZencashOfficial/repo/issues
The problem reported is not specific to the ZENCash Desktop GUI Wallet but concerns the entire APT repository. I am sure @cronicc will look into it...

Resolved, see https://github.com/ZencashOfficial/zencash-swing-wallet-ui/issues/73#issuecomment-368325988


Can't view live Streams

https://www.youtube.com/watch?v=bnJedJEI3I4
See also #1973

Same here. Tested on the link. Gave the error screen for like a second then switched to flash.

Thing is I don't have flash at all

Can confirm. Youtube Center dev build does not seem to support HTML5 live streaming. Disabling YTC fixes it and plays stream using HTML5 player. Regular videos play fine in HTML5 player.

I have this problem too. HTML5 live streaming only work if I disable YouTube Center.

This bug is still active and if YTC enabled live videos play with flash player.

Can confirm that this behavior is still present in current dev build.


PageField acts strange on form validation in latest Firefox versions

Whenever you try to save a form that has cms.forms.fields.PageField in it, the value of the widget resets if form validation fails because of other fields in it.
The issue happens in Firefox, and doesn't in Chrome. When I dug into the code, I've noticed this snippet:
(function($) {
    var handleSiteChange = function(site_name, selected_id) {
        $("#id_%(name)s_1 optgroup").remove();
        var myOptions = $("#id_%(name)s_2 optgroup[label='" + site_name + "']").clone();
        $("#id_%(name)s_1").append(myOptions);
        $("#id_%(name)s_1").change();
    };
    var handlePageChange = function(page_id) {
        if (page_id) {
            $("#id_%(name)s_2 option").removeAttr('selected');
            $("#id_%(name)s_2 option[value=" + page_id + "]").attr('selected','selected');
        } else {
            $("#id_%(name)s_2 option[value=]").attr('selected','selected');
        };
    };
    $("#id_%(name)s_0").change(function(){
        var site_label = $("#id_%(name)s_0").children(":selected").text();
        handleSiteChange( site_label );
    });
    $("#id_%(name)s_1").change(function(){
        var page_id = $(this).find('option:selected').val();
        handlePageChange( page_id );
    });
    $(function(){
        handleSiteChange( $("#id_%(name)s_0").children(":selected").text() );
        $("#add_id_%(name)s").hide();
    });
})(django.jQuery);

Apparently, the widget has one invisible <select> tag that affects the initial value passed to widget when validation fails. What happens (and I've checked it by making the third <select> visible) is that in FireFox, the value of this <select> doesn't get populated on page load.
I had a similar problem lately in another JS application, I've followed an advice from here:
http://stackoverflow.com/a/15656897/345811
Basically, attr("selected") doesn't work as intended in latest FF and Safari builds.
I am following this up with a MR containing a possible fix that works in Django 1.6+ (since jQuery 1.6+ is required and Django 1.5 ships jQuery 1.4.2).

Fixed in #3687


cannot debug in vs code

Description:

With a simple hello-world ballerina file and  the default launch.json, the "internal error" pop up window occurred.
The problem can be regenerated in multiple windows 10 machines.
The  launch.json is :
{
"version": "0.2.0",
"configurations": [
{
"type": "ballerina",
"request": "launch",
"name": "Ballerina Debug",
"script": "${file}",
"networkLogs": true
},
.....
]
}
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, Browser, other environment details and versions:
windows 10, vscode 1.46.1
Steps to reproduce:
Related Issues:

Closing as this has been already fixed with #24735 and #24745.


[Logging v2] Control Plane logs - cloud

[ ] EKS
[ ] AKS
[ ] GKE

Closing In favor of
https://github.com/rancher/rancher/issues/28572
https://github.com/rancher/rancher/issues/28573
https://github.com/rancher/rancher/issues/28574


Add MathJax Support

For block math:
$$
.... math content
$$

For inline math:
$items$

It handles inline math in a way the same as pandoc:

Anything between two $ characters will be treated as TeX math. The opening $ must have a character immediately to its right,
while the closing $ must have a character immediately to its left. Thus, $20,000 and $30,000 won’t parse as math.
If for some reason you need to enclose text in literal $ characters,
backslash-escape them and they won’t be treated as math delimiters.

Hello and thank you for your pull request. Support for math sequences is one of the longest standing and most highly requested features. There have been long discussions on what the delimiters for inline and block math sequences should be, but we have unfortunately not yet come to a decision. The latex-style you've implemented is certainly a contender, but there are some drawbacks relating to unintended sequences and graceful fallback.
Before we can move forward with any implementation, we'd have to agree on a format (read: delimiters) and a spec. That said, I did have a quick look at the code and the implementation generally looks good. If we do choose to go with dollars, this PR should make an excellent basis.
You have prompted me to try and form a proposal so we can hopefully finally get this sorted. I will include you in future discussion. Thanks again!

Hi @marcusklaas, actually I've noticed the discussions. I desperately need the math support, so I rushed to implement a basic working version for my personal needs. It's true that I haven't consider too much about the edges cases and this one shouldn't be a complete and sound solution.
I am a heavy markdown user and this is certainly one of the best implementation. It would be perfect if it has any kind of math support.
Thanks

Any content on a line that starts with $$ is currently ignored, and something like this, which I think is relatively common:
$$ \LaTeX $$

breaks the rest of the document.
Just mentioning to keep track. Hopefully we can get this merged soon!

Hey @marcusklaas did you ever put that proposal together? Or what is the status of planning for this?
#6 is a mess of bikeshedding over the delimiter. $...$ and $$...$$ is basically ubiquitous at this point (used by StackExchange, Pandoc, Jupyter, and now GitHub $E=mc^2$) so should definitely be included. Choosing other delimiters shouldn't block implementation work since they could be added later.
Assuming that logic is satisfactory, what is blocking this from being merged? There hasn't been too much of a discussion on KaTeX vs MathJax and pre- vs. post- rendering, but the work here is already mostly done, and could be adapted later if desired.
@litao91 are you interested in doing work to make this mergable, if needed? I see you are at least keeping this branch up to date with upstream

Closed in favor of #734.


chore: bump Gateway API to 0.6.0

What this PR does / why we need it:
The Gateway API has been bumped to 0.6.0

Which issue this PR fixes:
Fixes #3214

Special notes for your reviewer:

The Gateway API has been bumped to 0.6.0. A subset of conformance tests has been disabled to keeping this PR as small as possible. In the next iterations, all the currently disabled conformance tests will be fixed and re-enabled.
PR Readiness Checklist:
Complete these before marking the PR as ready to review:

[ ] the CHANGELOG.md release notes have been updated to reflect any significant (and particularly user-facing) changes introduced by this PR

Note for the reviewers: a subset of conformance tests has been disabled to keeping this PR as small as possible. In the next iterations, all the currently disabled conformance tests will be fixed and re-enabled.


mapErrors vs flatMapErrors

mapErrors maps error to a new error
flatMapErrors maps error to a stream of values
I find this very counter-intuitive. Why it's made like that?
My expectation was to have both converting errors to values. I see little value in converting errors to another errors because errors are mostly new Error objects i.e. instances (prototype is important and immutable x -> y-like transformations aren't possible in general).

The general idea of error related methods is to be symmetrical to similar value related methods. E.g. mapErrors is the same as map but for errors, skipErrors is the same as skip but for errors, etc. Hopefully, this makes API more intuitive, once you get that idea.

I think I get the idea but what does "for errors" mean. It may mean two things:
1) error -> error2
2) error -> value

mapErrors maps errors to errors (1)
flatMapErrors maps errors to values (2)
It's not symmetrical to value related methods where both map and flatMap map values to values.

"error -> error2" this one.

flatMapErrors maps errors to values

flatMap and flatMapErros can do both:
// values -> values
s.flatMap(x => Kefir.constant(x))

// values -> errors
s.flatMap(x => Kefir.constantError(x))

// errors -> values
s.flatMapError(x => Kefir.constant(x))

// errors -> errors
s.flatMapError(x => Kefir.constantError(x))

Ouch. Now it makes sense to me. Closing...


[OTEL-1977] Fix new OTel stats API to respect _dd.measured key

What does this PR do?
Fix new OTel stats API OTLPTracesToConcentratorInputs so that spans with _dd.measured key get APM stats. Also when the new top level feature is enabled, make sure non-root client and producer spans also get APM stats.
Describe how to test/QA your changes
No QA is needed because this API is not used in Agent, only in datadog connector.

Please write QA instructions for otel agent

This does not impact otel agent for now. Only after datadog connector is updated, new collector version is released then deps are updated.
QA should be done in OSS collector

/merge


Bump socrata-http version.

socrata-thirdparty-utils #64 FAILURE
Looks like there's a problem with this pull request

Looks like it can't find the repo.

socrata-thirdparty-utils #65 SUCCESS
This pull request looks good


issue with input element in forms when the input has a wave effect

Trying to implement materialize for an e-commerce solution...
One of the forms has the following structure:
<form id="edd_purchase_470124-2" class="edd_download_purchase_form edd_purchase_470124" method="post">

		<input type="submit" class="edd-add-to-cart edd-no-js button red waves-effect waves-light btn radius" name="edd_purchase_download" value="Free – Download" data-action="edd_add_to_cart" data-download-id="470124" data-variable-price="no" data-price-mode="single">

			Checkout

	<input type="hidden" name="download_id" value="470124">
	<input type="hidden" name="edd_action" class="edd_action_input" value="add_to_cart">
</form>

waves.js transforms that form to this:
<form id="edd_purchase_470124-2" class="edd_download_purchase_form edd_purchase_470124" method="post">

			<input type="submit" class="waves-button-input" name="edd_purchase_download" value="Free – Download" data-action="edd_add_to_cart" data-download-id="470124" data-variable-price="no" data-price-mode="single">

			Checkout

	<input type="hidden" name="download_id" value="470124">
	<input type="hidden" name="edd_action" class="edd_action_input" value="add_to_cart">
</form>

As a result I'm seeing a button inside a button.
The problem seems to be located on https://github.com/Dogfalo/materialize/blob/9e41e8bf72e34e5b325a4ee915a7168a54ae1785/js/waves.js#L197-L235 but unfortunately my JS skills are limited so I can't figure this one out..

Use the <button> tag instead. Take a look at the "Submit Button" section on http://materializecss.com/buttons.html

waves effects not working in self closed elements..


Remove a?.b = c and delete a?.b

Those are merely by-products of the spec mechanics (Nil reference), rather the intended semantics of optional chaining.
The proposed semantics may also be somewhat confusing for users, and it makes more difficult to have simple understanding of what short-circuiting does.
Those could be re-added later in case clear semantics and compelling use cases are found.
Strategy for fixing the issue:

use proper grammar and/or use static semantics in order to have SyntaxError where possible;
PutValue on a Nil Reference should throw a TypeError. Ditto for the delete operator.

I have kept delete a?.b because:

it has a reasonable use case, https://github.com/tc39/proposal-optional-chaining/issues/14#issuecomment-316475290;
it could hardly be confusing;
it is consistent with delete a.b not throwing when a evaluates to a primitive value different from null/undefined.


bugfix

@jemus42 can you check if this fixes things for you?

On TeX Live 2023: Produces a PDF without obvious (to me) problems, but I get this warning
Reference `endlect:@@1' on page 1 undefined on input line 24

Not sure what \endlecture does, reset slide numbers? https://github.com/slds-lmu/lecture_i2ml/blob/764a16f62b23a3bd71a7c43c274c2d7a4d66bcc9/style/lmu-lecture.sty#L140-L144
On 2022, I get a similar error
! Package calc Error: `\calc@A ' invalid at this point.

As I understand it, \endlecture writes out the current slide number to the aux file, which is then used as the y in the x / y footer.

As far as I can tell this works now - thing is I don't understand the problem (or solution).
In general though, what's happening there, anyway? We're redefining \ref (or rather \lectureref) due to beamer redefining that, to ensure slide numbers are displayed correctly? But why wouldn't they be?

I think what is supposed to happen here is that we want the footer to display "page X of Y" for each lecture, even if there are multiple \lecture{...} .... \endlecture in a single tex file -- this way you can \input many lectures to get the total collection of slides, and each lecture has its own page counter. One way to solve this would be to do \label{lect:<lecturenumber>} on the first page and \label{endlect:<lecturenumber>} on the last page and do some calculations by subtracting \pageref{lect:<lecturenumber>} from the current frame number etc.
My impression is that the code in our .sty file is a copy-pasted version of the Latex source code that re-implements the \ref / \pageref logic for this purpose. I think the problem is that we copy-pasted and re-implemented some, but not all, of the \ref logic. So when this commit in this PR changed how \@setref works, it broke our code (added some \@empty tokens that our code doesn't anticipate). My change here should work regardless of the presence of \@empty tokens by simply looking for the \null token at the end.

Since this works now, I am going to merge. As a medium term task, we should probably clean up this code. I think it does a lot of things that we don't actually need, and the other things we should probably solve "the recommended way".


More events

Not sure if it would be good idea to add more events to e.g. check when the sw has been updated or become redundant.
Check this code for more details

Yeah, that'd be very useful! PRs welcome!

PR on #4 :+1:

Merged, thanks :heart:


Enforce new connection when last activity > than (idle interval + TTL)

Enforces a new connection when the last activity is > than (_maxIdleInterval + _connectionStateTtl)
Mimics this behavior in JS, called when a connection gets started.
cc @paddybyers

Took another stab https://github.com/ably/ably-ios/pull/719/commits/deb79a77d896ecead2bf222a2e033bb205626c70

Have a look at the tests in https://github.com/ably/ably-ruby/pull/146 for the cases we need to test

@paddybyers Will do

ricardopereira approved these changes 32 minutes ago

I do think we should have test cases included before approving the PR.

@funkyboy I think we should get into the habit with spec changes of writing tests that fail, then making them pass.  This approach IMHO is in most cases wrong.  If you look back at how we worked with @ricardopereira in the past, he wrote tests that failed but met the spec, then Toni wrote the code.  I am not sure anyone can review this code in any meaningful way without tests.

@mattheworiordan will do. Won't merge this without having tests, which I am working on.

@mattheworiordan you can read
delay(reconnectAfterInterval) {
  client.connect()
}

as the counterpart of the following Ruby
sleep(reconnectAfterInterval)
client.connect

you can read
delay(reconnectAfterInterval) {
client.connect()
}
as the counterpart of the following Ruby
sleep(reconnectAfterInterval)
client.connect

Ok, so then if the client is running in it's own thread, whilst the test is sleeping for X time, what's stopping it from reconnecting itself automatically?

It puts to sleep the whole test context, on the main thread: https://github.com/ably/ably-ios/blob/develop/Spec/TestUtilities.swift#L498. The client also belongs to that thread.

Ready for another look.

@funkyboy I don't think that is sufficient.  See https://github.com/ably/ably-ruby/pull/146/files#diff-6994e351b062efe17b0b5c88ac75991f for comparison.
I test both positive & negative conditions, as well as the channel attach requirements in the specification.  I don't believe this is comprehensive enough.

@mattheworiordan @paddybyers @ricardopereira ready for another look

@mattheworiordan If you don't have any further observations, I'd merge this.

After talking to @paddybyers and after working on the Java version of this feature I realized that these tests are wrong, in that they should wait for suspended state (after disconnecting) and then perform checks. Here's a new todo list:

[ ] Change tests to follow this sequence:

connect
extract data / mock values
disconnect
wait for the connection to go into suspended mode
reconnect and wait for the connection to reconnect
verify assertions

[ ] Change implementation accordingly

After another discussion about RTN15g it turns out that this (fresh connection after connectionTtl + idle timer has passed) doesn't need to go through the suspended state.
I tweaked the tests so that the reconnection happens via the state machine instead of calling connect().
That surfaced a bug because the freshness check was performed only when connect() was called directly.

Ok, logic seems good

@mattheworiordan Made an issue with your feedback on this PR after merge.


Platform Test Suite Failed To Install on Apple's new Mac ARM machine

Attempt to install Platform Test Suite failed on Apple's new Mac ARM machine.
Expected Behavior
Successfully install Platform Test Suite
Current Behavior

Steps to Reproduce (for bugs)
npm i
Your Environment

Platform Test Suite v0.21.0
Machine: Apple M1 Mac

Fixed the issue in a PR here:
https://github.com/dashevo/node-merk/pull/14


LDAP Question

When we work with LDAP and user information is synced with it, if user information changes in ldap, this information will not be included in subsequent syncs and will only be included in the first sync?

@dacongda

YES


Remove Control Dashboard version 1.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest

Note: <path> is the name of the directory containing the manifest you're submitting.

Removed Control Dashboard version 1.0 as latest version is 1.1
Microsoft Reviewers: Open in CodeFlow

URL: https://www.japplis.com/control-dashboard/versions/ControlDashboard-1.0.exe
Status Code: 200
(Automated message - build 861)


RSS feeds should always include channel’s icon

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
My RSS feed looks disorganized because all the odysee icons don’t reflect the channels I’m subscribed to
Describe the solution you'd like
A clear and concise description of what you want to happen.
when a RSS feed is subscribed to, the icon should automatically be the channel’s icon
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Some channels do have icons, but most don’t, I’m not sure if this is an opt-in feature, but I don’t think it should be opt-in

I believe there is no field for us to define a favicon in the rss spec.  RSS readers will grab the favicon from the "site link" field.
If your creator has a website defined in the About page (claim.value.website_url) and the page has a favicon, then it will work. Otherwise, it uses the default Odysee favicon.

Can you show us a screenshot of what you are seeing also?

this is NetNewsWire on iOS, but he strange thing is that YouTube channels I’m subscribed to show the channels icon in their favicon

Think it would work if we forced the odysee link into the website URL if they don't have one, or maybe the url link there. Can experiment soon and have you try again.

I can confirm this issue has been fixed, thank you for the great work, keep it up!


Sketch plugin updater does not work

Looks like the update from Sketch enters an infinite loop, because the appcast says there's a v1.1 available, but the version on your manifest.json is 1.0. So Sketch downloads the "update" and doesn't install it (its version is not higher than the currently installed 1.0) and the Update button does nothing.
Bumping the version on manifest.json to 1.1 should fix it.

Thanks, just did that
https://github.com/timuric/Content-generator-sketch-plugin/commit/5ccbbfb37a02a6bcf9144b466257e0ddd1d7387e


web/assets: AssetHandler: fix dynamic Content-Type handling

when embed.FS hasn't pre-computed it

<EMAIL_ADDRESS>tagged


pull request for e2e test at 12/12/2016 17:56:03

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG] BepInEx update 5.4.2101

Jotunn Version:

2.11.2

Repeatability:

100%

Problem Description:

Jotunn does not load because it is not compatible with BepInEx 5.4.2101 (released 28 march)

I already noticed earlier that in the commits for the v2.11.2 release  (also 28 march), nothing actually was changed besides the changelog and Jotunn's own version number. Not sure if related!
https://github.com/Valheim-Modding/Jotunn/compare/v2.11.1...v2.11.2
https://github.com/Valheim-Modding/Jotunn/commit/fbdc2080c4291b339a727b26975fc343acf467dc
The bepinex version does not show correctly in the console output, but I know this is the cause because I have just updated and that is what broke everything :)

Never mind, this is actually a bug in Vortex - it's installing 5.4.19..


Blue kerbin dynamics Mk1 Fuel Tanks Plus

Closes #4539

Can you make the identifier just "BKDMk1FuelTanksPlus"?

Spacedock was down, please try this later

Looks good to me, merging with thanks


DESIGN: Should we use PostgreSQL instead of MS SQL Server

I would like to use arrays or list of value types in our models.
@hassanhabib and @ElbekDeveloper What should I do ?

Create a string conversion in  OnModelCreating to handle this ?

pros: we can keep SQL Server
cons: it adds complexity

Change EF config to use PostgreSQL instead of SQL Server
Use a secondary class to hold a value and make a list of those entities in my primary entity?

keep it MSSQL
what does the array of string hold? just create a separate data class to hold that one type.


Update to xenial

This change is

Coverage remained the same at 97.26% when pulling 265782283d9acaca04ba6c14741d6c3e0853cf68 on update-to-xenial into 0afc66f30954aaba32035c23a0f7ed77a6c7e418 on master.


guard String() in console-breadcrumbs plugin

In the console-breadcrumbs plugin, String() is used on each argument to console.log without a guard.
This will throw when the argument is an object which doesn’t implement toString/@@toPrimitive (most commonly this would occur when objects have null prototype).
This fix does not address the fact that various parts of Bugsnag will throw if Symbol values are logged when this plugin is enabled. I believe that may be a broader issue concerning breadcrumbs metadata, not specific to the console-breadcrumbs plugin.

Hey @bathos-wistia, thanks for supplying this PR. I've made this change over on another branch, as the current codebase has been rejigged for universal JS support. I'm closing this in favour of that (#429) because what's currently in master is no longer in active development (universal will be switched over to be the "new" master this week).
I explored a couple of other options, including doing Object.prototype.toString.call(val) but arrived back at your suggestion as the best thing to do for console breadcrumbs specifically.
With regards to your other comment:

This fix does not address the fact that various parts of Bugsnag will throw if Symbol values are logged when this plugin is enabled. I believe that may be a broader issue concerning breadcrumbs metadata, not specific to the console-breadcrumbs plugin.

I think this is something we will need to support in the serialization module we maintain our own fork of: @bugsnag/safe-json-stringify. If you have a concrete example that you can create a test or quick example of, it would be great if you could post it over on that repo. Much appreciated!


Update Space Engineers Generic to add Config Directory

This commit adds the ConfigDirectory update step to Space Engineers generic template due to the directory not existing until the first server start.

This should not be merged yet. I need to correct the checkbox values being reversed after the patch was put in.

This one should be ready for publishing now. Sorry for any issues it's caused!


Added composed-task-runner to the AllSpringCloudTaskAppStarterJobs

Also added it to RELEASE_PHASE1_JOBS.  Please verify that is correct.

LGTM - merging.


tf.math.reciprocal lacks support of int input on CPU, inconsistent with the documentation

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.15.0
Custom code
Yes
OS platform and distribution
No response
Mobile device
No response
Python version
No response
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
tf.math.reciprocal fails to support integer input on CPU, which is inconsistent with its documentation https://www.tensorflow.org/api_docs/python/tf/math/reciprocal?hl=en.
Instead, it can support integer input on GPU.
Standalone code to reproduce the issue
import os
os.environ['CUDA_VISIBLE_DEVICES'] = ''
import numpy as np
import tensorflow as tf
x = tf.constant(np.random.randint(-50, 50, ()), dtype='int64')
tf.math.reciprocal(x)

### Relevant log output

```shell
tensorflow.python.framework.errors_impl.NotFoundError: Could not find device for node: {{node Reciprocal}} = Reciprocal[T=DT_INT64]
All kernels registered for op Reciprocal:
  device='XLA_CPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_INT16, DT_INT8, DT_COMPLEX64, DT_INT64, DT_BFLOAT16, DT_COMPLEX128, DT_HALF]
  device='XLA_GPU_JIT'; T in [DT_FLOAT, DT_DOUBLE, DT_INT32, DT_INT16, DT_INT8, DT_COMPLEX64, DT_INT64, DT_BFLOAT16, DT_COMPLEX128, DT_HALF]
  device='GPU'; T in [DT_COMPLEX128]
  device='GPU'; T in [DT_COMPLEX64]
  device='GPU'; T in [DT_INT64]
  device='GPU'; T in [DT_DOUBLE]
  device='GPU'; T in [DT_FLOAT]
  device='GPU'; T in [DT_HALF]
  device='GPU'; T in [DT_BFLOAT16]
  device='CPU'; T in [DT_COMPLEX128]
  device='CPU'; T in [DT_COMPLEX64]
  device='CPU'; T in [DT_DOUBLE]
  device='CPU'; T in [DT_BFLOAT16]
  device='CPU'; T in [DT_HALF]
  device='CPU'; T in [DT_FLOAT]
 [Op:Reciprocal] name:

@drewshark Before applying tf.math.reciprocal, could you try to explicitly cast your integer input tensor to a supported floating-point data type using tf.cast as follows;
import os
os.environ['CUDA_VISIBLE_DEVICES'] = ''
import numpy as np
import tensorflow as tf
x = tf.constant(np.random.randint(-50, 50, ()), dtype='int64')
x = tf.cast(x, dtype=tf.float64)
tf.math.reciprocal(x)

It might resolve the issue. Please find the gist here for reference.
Thank you!

@sushreebarsa Sure, thanks for your effort in fixing it.

@drewshark Glad it helped you!
Thank you for your response here.
Closing the issue for now.


Problem running a matrix completion algorithm using convex optimization

I tried to use cvxpy to run a matrix completion algorithm

The code goes like this

The code doesn't run, getting a warning that 'UserWarning: Forming a nonconvex expression (affine)*(affine).", and throwing an exception that 'Cannot evaluate the truth value of a constraint or chain constraints, e.g., 1 >= x >= 0.'
Anyone could help me figure out where the problem is?
Is that my way for passing the constraints goes wrong, or my specification or multiplying two cyxpy-typed variables(matrices)?
It seem that this way of writing constraints dynamically does not work, then what can I do?

I tried a faked instance to pass the constraints and it seems not work.

Could anyone help me figure out how to dynamically arrange the constraints (i.e., according to the inputs)

If you multiply two variables together, that's not a convex problem. You could use the DMCP package, but CVXPY alone won't work. You'll be better off constructing a boolean matrix B that is 1 wherever A is not nan. Then add the constraint X[B] == A[B].


fix: add jquery in ProvidePlugin

add jquery in ProvidePlugin
Description of the Change
angular.js looks for window.jQuery (cf: https://github.com/angular/angular.js/blob/master/src/Angular.js#L1890 )
Before angular.element used jqLite, with this fix it's now jQuery.
This solution is recommended in the webpack documentation
7a0e47e - fix: add jquery in ProvidePlugin
/cc @jleveugle @frenautvh @antleblanc

This solution is recommended in the webpack documentation

I'm a bit curious about why you've decided to close this pull request?
Could you be more precise of such a decision? Your feedback will be well appreciate:) :)


Update Recipes page

Changed recipes to use "kubejs" and "minecraft" namespaces.

You made a mistake here

Thank you for pointing that out.


hash/anchor-link scrolling in standardScroll

I am not certain this is the best approach (basically option 1 from https://github.com/taion/scroll-behavior/issues/22#issue-121245846), but wanted to open something up for discussion.
Changes

Check for hash and scroll to relevant element on next frame

Closes #22

This change is

Coverage decreased (-3.8%) to 96.154% when pulling 358ac26577ce7db57b757539fb5d01d8c164c9eb on SpainTrain:anchor-scroll into c5f17e4f27095cf880021600add0fc510780c603 on taion:master.

Let me think a bit about what I want to do with this library.
I need to also get https://github.com/mjackson/history/issues/163 into a good direction first as well.

How would it affect people that use smoothScroll in some specific cases? It would override/cancel the animation, wouldn't it?

I have no idea.
This PR is lacking the logic to only scroll to the hash on a new transition (e.g. we shouldn't scroll to the hash on a pop transition – we should just restore the scroll position).

It would override/cancel the animation, wouldn't it?

Probably.  I'm using my fork in production with react-scroll.  I will switch over to smoothscroll (which is better for my use case anyway) and see about a solution / try your suggestion.

we shouldn't scroll to the hash on a pop transition

:+1: This is the type of discussion I wanted to open.  I created a TODO checklist in the PR description to track outstanding items.

By default, this should be on all actions other than POP, BTW.

At this point, this change is sufficiently far behind head that it won't be directly mergeable. I'm going to close this out for now.


Problem with fetching not a json features

We have faced a problem in the lib code caused by the marked string. If the features receive not a json string (for some reason) we fallback to crash right after initialization.

Not a json features but what the features?
My feature type was String and it was fetched successfully

Yeah I just figured out one more thing, it gets crashed even if we got 500.
Because onSuccess is still being called after error, so check does not stop the block actually.

And the features request was a second step of getting malformed answer.

It should be checked.
Do you have ability to use NetworkDispatcher based on Ktor instead of OkHttp?

I came to a solution to write my custom NetworkDispatcher with the fixes in consumeGETRequest and consumePOSTRequest methods:

Great!

Yeah I just figured out one more thing, it gets crashed even if we got 500. Because onSuccess is still being called after error, so check does not stop the block actually.  And the features request was a second step of getting malformed answer.
About the features response: still there is a possibility to get something unexpected even with 200 code, so handling the possibility of bad json when decoding is still a good practice imo

It is screenshot from OkHttpDispatcher v1.0.0; v1.0.2 is available on MavenCentral


Chainlit is unrecognized error in MacOs M1 chip

chainlit run chainlit_mac_app.py -w

Loaded .env file
usage: chainlit [-h] [--hide-source] [--mute-stream]
chainlit: error: unrecognized arguments: run chainlit_mac_app.py -w

That is weird?
I am also using M1 chip (Apple M1 Pro) but I can run the command chainlit run xxx.py -w fine.
Maybe sharing more info about your environment could help solving this issue?


Trying to work with aave v3 but getting this error (VM Exception while processing transaction: reverted with reason string '31')

Hi @PatrickAlphaC
Please help me with this code!
I'm trying to work with aave V3 but I'm stack on borrowing section
const { formatUnits } = require("ethers")
const { getNamedAccounts, ethers } = require("hardhat")
const { getWeth, AMOUNT } = require("../scripts/getWeth")

async function main(){
    // the protocol treats everything as ERC20 token
    await getWeth()
    const {deployer} = await getNamedAccounts()
    // Pool Address Provider : 0x2f39d218133AFaB8F2B819B1066c7E434Ad94E9e
    const pool = await getPool(deployer)
    console.log(`Pool Address : ${pool.address}`)
    // deposit!
    const wethTokenAddress = "0xC02aaA39b223FE8D0A0e5C4F27eAD9083C756Cc2"
    // approve!
    await approveERC20(wethTokenAddress, pool.address, AMOUNT, deployer)
    console.log("Depositing...")
    await pool.deposit(wethTokenAddress, AMOUNT, deployer, 0)
    console.log("Deposited!");

    let { availableBorrowsBase, totalDebtBase } = await getBorrowUserData(pool, deployer)
    // borrow
    // how much we have borrowed, how much we have in collateral, how much we can borrow
    // // what is the conversion rate on DAI
    const daiPrice = await getDaiPrice()
    const daiAmount = await getEthPrice(availableBorrowsBase)
    const amountDaiToBorrow = daiAmount.toString() * 0.8 * (1/daiPrice.toNumber())
    // console.log(`You can borrow ${daiAmount.toString()} DAI`)
    const amountDaiToBorrowWei = ethers.utils.parseEther(amountDaiToBorrow.toString())
    const daiAddress = "0x6B175474E89094C44Da98b954EedeAC495271d0F"
    await borrowDai(daiAddress, pool, amountDaiToBorrowWei, deployer)
    await getBorrowUserData(pool, deployer)
}

async function borrowDai(daiAddress, pool, amountDaiToBorrow, account){
    const borrowTx = await pool.borrow(daiAddress, amountDaiToBorrow, 1, 0, account)
    await borrowTx.wait(1)
    console.log("You've borrowed!");
}

async function getDaiPrice(){
    const daiEthPriceFeed = await ethers.getContractAt("AggregatorV3Interface", "0x773616E4d11A78F511299002da57A0a94577F1f4")
    const price = (await daiEthPriceFeed.latestRoundData())[1]
    console.log(`The DAI/ETH price is ${price.toString()}`);
    return price
}
async function getEthPrice(basePrice){
    const EthPriceFeed = await ethers.getContractAt("AggregatorV3Interface", "0x773616E4d11A78F511299002da57A0a94577F1f4")
    const price = (await EthPriceFeed.latestRoundData())[1]
    // console.log(`The ETH/USD price is ${price.toString()}`);
    const ethPrice = await price * basePrice / 10**8
    return ethPrice
}

async function getBorrowUserData(pool, account){
    const { totalCollateralBase, totalDebtBase, availableBorrowsBase } = await pool.getUserAccountData(account)
    console.log(`You have ${(await getEthPrice(totalCollateralBase)).toString()} worth of ETH deposited.`)
    console.log(`You have ${(await getEthPrice(totalDebtBase)).toString()} worth of ETH borrowed.`)
    console.log(`You can borrow ${(await getEthPrice(availableBorrowsBase)).toString()} worth of ETH.`)
    return { availableBorrowsBase, totalDebtBase }
}

async function getPool(account){
    const poolAddressesProvider = await ethers.getContractAt("IPoolAddressesProvider", "0x2f39d218133AFaB8F2B819B1066c7E434Ad94E9e", account)
    const poolAddress = await poolAddressesProvider.getPool()
    const pool = await ethers.getContractAt("IPool", poolAddress, account)
    return pool
}

async function approveERC20(erc20Address, spenderAddress, amountToSpend, account){
    const erc20Token = await ethers.getContractAt("IERC20", erc20Address, account)
    const tx = await erc20Token.approve(spenderAddress, amountToSpend)
    await tx.wait(1)
    console.log("Approved!")
}

main()
    .then(()=> process.exit(0))
    .catch((error) => {
        console.error(error)
        process.exit(1)
    })

I do not what's wrong with it everything seems fine to me but I got the error below
Got<PHONE_NUMBER>0000000 WETH
Pool Address : 0x87870Bca3F3fD6335C3F4ce8392D69350B4fA4E2
Approved!
Depositing...
Deposited!
You have<PHONE_NUMBER>3781440 worth of ETH deposited.
You have 0 worth of ETH borrowed.
You can borrow<PHONE_NUMBER>2811468 worth of ETH.
The DAI/USD price is<PHONE_NUMBER>69495
Error: VM Exception while processing transaction: reverted with reason string '31'
    at <UnrecognizedContract>.<unknown> (0x87870bca3f3fd6335c3f4ce8392d69350b4fa4e2)
    at <UnrecognizedContract>.<unknown> (0x87870bca3f3fd6335c3f4ce8392d69350b4fa4e2)
    at <UnrecognizedContract>.<unknown> (0x87870bca3f3fd6335c3f4ce8392d69350b4fa4e2)
    at processTicksAndRejections (node:internal/process/task_queues:95:5)
    at HardhatNode._mineBlockWithPendingTxs (/home/astro/Astro_sol_js/Defi/node_modules/hardhat/src/internal/hardhat-network/provider/node.ts:1840:23)
    at HardhatNode.mineBlock (/home/astro/Astro_sol_js/Defi/node_modules/hardhat/src/internal/hardhat-network/provider/node.ts:517:16)
    at EthModule._sendTransactionAndReturnHash (/home/astro/Astro_sol_js/Defi/node_modules/hardhat/src/internal/hardhat-network/provider/modules/eth.ts:1532:18)
    at HardhatNetworkProvider.request (/home/astro/Astro_sol_js/Defi/node_modules/hardhat/src/internal/hardhat-network/provider/provider.ts:123:18)
    at EthersProviderWrapper.send (/home/astro/Astro_sol_js/Defi/node_modules/@nomiclabs/hardhat-ethers/src/internal/ethers-provider-wrapper.ts:13:20)
error Command failed with exit code 1.

Any Help guys please !!!!!

Can you:

Make this a discusson on the full repo? https://github.com/smartcontractkit/full-blockchain-solidity-course-js/
Follow this section for formatting questions? https://www.youtube.com/watch?t=19846&v=gyMwXuJrbJQ&feature=youtu.be

Hello,
According to Aave, this error corresponds to the following message "STABLE_BORROWING_NOT_ENABLED". In other words, for some tokens that are considered high volatility, you cannot borrow at a fixed rate. DAI is probably one of them.
One way to get around that is to check if a token has stable rate borrowing enabled by calling the getReserveData() function.
You can then adjust the loan rate type depending on the result.
I have worked on a version of the project that uses both Aave V3 and Uniswap V3 for token swap.
You can check my repo for more details : https://github.com/MMtis/Hardhat-Defi-AaveV3-UniswapV3
Hope this helps !


Support access/trunk mode and managing native vlan on bond for cisco/dell

This is a partial improvement that only allow adding/removing native vlan and changing access/trunk mode on bonds.  It does not support listing or creating or deleting bonds.

:+1:


Save before run

Ability to save before run bat file

I like the idea, will look into adding it for the next update!

With the 1.2.0 release you can now set the setting
{
    "batch-runner.saveFileBeforeRun": true
}

to automatically save the batch files before running them.
Full changelog here: https://github.com/nils-soderman/vscode-batch-runner/releases/tag/1.2.0

Is there a reason you didn't make this the default? As far as I know, that's the behaviour in the vast majority of code editors. I can't personally think of any reason why someone would want an easy way to run the current file without making sure it's saved first. I also just think it's more intuitive for an easy command to run the file the user is looking at to run the version of it they're looking at (including any unsaved changes) (i.e. save it first).

Hi,
No, no real reason, chose to default it to false just because that was the current behavior of the extension.
But you raise some valid points, and I agree that we should default this to true instead.

With the latest release (1.3.0) batch-runner.saveFileBeforeRun now defaults to true.
Thanks for the suggestion!


modify recommendation CQL to replace recommendation text with keys

per conversations we had before the holiday break, would like to replace recommendation text blocks in CQL with unique keys, as a matter of improved separation of concerns, and to simplify future modifications to recommendation text, and to make possible the provision of alternate text given certain scenarios (e.g. during A-B testing).
as a product of this process, please compile a basic spreadsheet of key-value pairs (key=recommendation text block) so that I can use that to build out the COACH database table where those associations can be made.
this issue is a companion to issue https://github.com/OHSUCMP/coach/issues/24

@mattStorer I had been hoping for more discussion before we jumped on this idea. We will need to figure out how to deal with CQL values that were previously dynamic. For example, goal suggestions use a unique id to avoid conflict with other goals, so it looks something like this:

And when it's time to update a goal, the CQL returns the text of the goal as part of the suggestion:

Either COACH will have to do substitutions of this dynamic information or the CQL will have to pass "metadata" associated with the key. Since COACH has the information, I'd vote for the first option, but I need to give it some more thought.
I'm starting to compile a document starting with the easier workflows. I'm going to keep the pipe-delimited convention for details with individual keys for rationale, suggestions, etc. And I'm only considering keys for summary and detail right now, but can do them for indicator and source if you want a fully generic implementation. So a cards response might look something like this:
{
    "cards": [
        {
            "summary": "Hypertension.5-ConsiderStage1.Summary",
            "indicator": "info",
            "detail": "Hypertension.5-ConsiderStage1.Rationale|Hypertension.5-ConsiderStage1.Suggestions|Hypertension.SelectionBehavior|Hypertension.5-ConsiderStage1.Links",
            "source": {}
        }
    ]
}

@aeyates ah, okay thanks for this information.  please note that there is no set timeline on doing any of this, I'm just trying to document plans for future development and get the ball rolling, even if that just means exploring what it might take to accomplish it.  no stress, no rush.
I think that mostly all we will really want to adjust is the label component, we probably don't want to try to reduce entire recommendations down to a single key.  there is probably a viable middle-ground to be found here.

Thanks for the added context. It will be good to get some specifics on what type of variation the team wants in messaging for the clinical trial. If it would be sufficient to provide keys for the summary and details in the graphic below, but leave suggestions, links, and other things static, that would be an easier lift.