Design of monitor/notifier messages

Not sure which format is the best...
Monitors
Failed to detect IPv4 address

Failed to set: me.hello.org, you.hello.org, hi.hello.org, hey.hello.org

Set: me.hello.org, you.hello.org

Notifiers
Updated x.x.x.x and y.y.y.y with the detected IPv4 address <IP_ADDRESS>.

Failed to detect the IPv4 address.

Possibly failed to update xxx.xx with the detected IPv4 address <IP_ADDRESS>.

Possibly failed to update xxx.xx with the detected IPv4 address <IP_ADDRESS>; yyy.yyy and zzz.zzz were updated.

Implemented by #762.


make a NewSpan... function that accepts context.Context

Here is an attempt at it. How does this look?
https://github.com/bketelsen/buffet/pull/5


Move log processing and database operations to a worker thread.

Review status: 0 of 5 files reviewed at latest revision, 7 unresolved discussions.

angular_scripts.js, line 108 [r2] (raw file):
nit: Could we break a line here? Consistency with the rest of the project. At the end of the array as well

angular_scripts.js, line 186 [r2] (raw file):
nit: A comment might help here telling us how the the SELECT query uses the variables minTestId, maxTestId and limit are used. Not a compulsory issue, it's just that I couldn't understand this code without reading LovefieldService.js especially due to the importance of the null value in those variables

angular_scripts.js, line 190 [r2] (raw file):
Could we define resultsView.firstTestId as null in the first initialization as well?

angular_scripts.js, line 192 [r2] (raw file):
This won't be a sufficient condition right? Haven't really worked it out, but what if the total number of results are a multiple of the limit or something on those lines?

index.html, line 107 [r2] (raw file):
A small complaint here. I observed that the UI is responsive while the SELECT query is running. Shouldn't we have a $scope.busy = true in the fillTable() function?

LovefieldService.js, line 360 [r2] (raw file):
nit: Missing ; here and below

LovefieldService.js, line 363 [r2] (raw file):
No issues with the functionality, but an issue with the clarity. Could you add a comment here? Took me sometime to get this, as I was constantly thinking that the final results would be in DESC as well

Comments from the review on Reviewable.io

Review status: 0 of 5 files reviewed at latest revision, 7 unresolved discussions.

angular_scripts.js, line 192 [r2] (raw file):
Right, but we can't know that without doing a limit + 1 query and displaying only limit rows, which I suppose is possible but seems quite complicated.

Comments from the review on Reviewable.io

Review status: 0 of 5 files reviewed at latest revision, 13 unresolved discussions.

angular_scripts.js, line 15 [r2] (raw file):
nit: Missing ;

angular_scripts.js, line 27 [r2] (raw file):
What is the reason for moving this to the LovefieldService?

angular_scripts.js, line 35 [r2] (raw file):
Well more of a doubt rather than a fix. How does this Promise resolve when the Promise inside the Worker API finishes (I mean the services.apply() Promise) ?
From what I've understood, you are using the resolve object cleverly by storing it as a WorkerService data member and executing it (resolve(data)) when service.apply() posts a message to WorkerService after completion.

angular_scripts.js, line 79 [r2] (raw file):
nit: Trailing ;

logcruncher.js, line 12 [r2] (raw file):
Can we make this a part of the LogReader prototype?

LovefieldService.js, line 496 [r2] (raw file):
Could we move this to the beginning of the file? Like in logcruncher.js

workerApi.js, line 7 [r2] (raw file):
nit: Maybe we could improve the logs by saying <Service> got command ...

Comments from the review on Reviewable.io

Reviewed 4 of 5 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 13 unresolved discussions.

Comments from the review on Reviewable.io

This PR attempts to fix #18
It also tries to do the finishing touches for #25 and #26

Review status: all files reviewed at latest revision, 13 unresolved discussions.

angular_scripts.js, line 35 [r2] (raw file):
Right, So what is going on is that we have a bridge between the event-based API for communicating with workers, and the promise based API we previously implemented both on both the main thread and now inside the workers.
In this part of the code we first post a message to the worker telling it to do something and then create a promise that will be resolved later when we get result back from a worker. In the workerApi.js file we receive the message posted here. Then we use the content of that message to select the function to run; that function returns a promise, which we add a then to which posts a message back to the main thread with the results of this function. The final part of the puzzle is the this.worker.onmessage function above, which receives the reply from the worker, looks up the correct resolver and calls it with the data returned from the worker thread so that the promise we originally created is then resolved with this value. From the point of view of the code on both sides this just creates a normal promise-based API; they don't know that there's async event stuff happening in the middle (indeed that kind of abstraction is rather the point of promises).

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 13 unresolved discussions.

angular_scripts.js, line 35 [r2] (raw file):
Oh alright makes sense now. Just one more fundamental question, How can you say the actual process is off the main thread? Is it because it is executed inside a postMessage() which is a part of Worker and hence off the main thread?

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 13 unresolved discussions.

angular_scripts.js, line 35 [r2] (raw file):
Not directly; one can postMessage on a single thread (or between different windows, for example). The fact that we run the code in a worker is determined by the fact that we call new Worker(worker_script) above, for both LovefieldService.js and logcruncher.js, rather than including them via a HTML script element. After that postMessage is the only available method of communication.

Comments from the review on Reviewable.io

Review status: all files reviewed at latest revision, 3 unresolved discussions.

angular_scripts.js, line 27 [r2] (raw file):
You can't return a db connection object from a worker, and it creates a simpler API if all the LovefieldService methods implicitly create the database connection if it isn't already established.

angular_scripts.js, line 186 [r2] (raw file):
I've added some JSDoc to the getResults method.

logcruncher.js, line 12 [r2] (raw file):
Well we could, but this way the internals are private to the worker and can't be accessed directly via the postMessage API.

Comments from the review on Reviewable.io

r+ if the code is working. Can't test it right now :(


add wobbly

wobbly is a piece of java script, which implements a simple gemini browser in your html browser.
https://www.warmedal.se/~wobbly/

@erwaelde Please open a PR and add it to the README.md.

Fixed with #141


Make schema loader beta

ecschema-metadata

Made SchemaLoader, SchemaJsonLocater and SchemaPropsGetter beta
Add getter to SchemaLoader for context

ecschema-locaters

Change SchemaFileUtility class to SchemaXml namespace and make beta
Rename functions to be shorter and clearer

Added docs for loading schemas from an iModel, creating a new schema with the editor and inserting back into the iModel
Added docs for creating and using code snippets

Please summarize these changes and promotions with links to documentation in NextVersion.md.

done


Support for portable PDBs

With the new csproj format, the default is the superior portable PDB format for all .NET projects, including traditional .NET Framework apps.
This is somewhat of a integral issue. Where should this capability be on the roadmap?

Duplicate of #182

On mac/linux only ppdb can be used.

Closing as duplicate of #182.


Fixing Cython doctests

With the release of pytest 6, the pytest-cython module doesn't work anymore. This can be reproduced by running pytest --doctest-cython. An issue is open here. It might be worthwhile to send a pull request to pytest-cython in order to fix this issue.

Hi,
I am not able to reproduce the error.
On running pytest --doctest-cython --disable-warnings
I get the following error:
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --doctest-cython --cov-config=.coveragerc --doctest-cython
  inifile: /Users/ashish/Downloads/projects/creme/pytest.ini
  rootdir: /Users/ashish/Downloads/projects/creme

Have you installed pytest-cython?

yes it is installed already


freeze the environment by default when it is activated

Add a freeze=true argument to activate! (or something like this) to freeze the environment until it is deactivated.
The effect of this is that resolve will refuse to uninstall or modify any already-installed packages - i.e. you can only add new packages, and in this case we use the conda install --freeze-deps, only passing the extra packages to conda install and pip install.
This will let the user safely add new packages even if PythonCall is already running, and prevents disrupting anything already there (which might already be depended upon).
Could even add a --fast flag to resolve which uses this scheme anyway (if possible).

I'm not sure how to mark an environment as frozen. Perhaps just set an env var JULIA_CONDAPKG_FROZEN=true, and check for this in resolve()?

This would be very useful...


Nselib is not fetching data for anything

The issue started today at around 12:25

yfinance was also facing this problem on 18th October, for Nse stock data, but they fixed it by evening

Looks like the NSE website is going through some maintenance activity.. we will wait till Monday.. if it is not fixed .. then we will fix it .
Thanks for informing us.

Please upgrade to nselib 1.4,  all issue resolved


Individual pages for providers and update UI

Overview
This could be merged after testing, but I would love to have anyone make any potential tweaks and modifications. I want to hear from a few people to make sure they think this works.

Uses TailwindUI logo cloud for homepage to select provider, adds logos
Separates each provider to individual page
Creates a provider lib file that streamlines adding new providers
Switches providers with select box on provider page
Creates badge component using TailwindUI badge to show if authenticated or not
Adds Redwood logo that links to app's home page
Uses TailwindUI form layout as wrapper for the providers
Adds general styles to reduce the need to style every component, includes adding Redwood red color
Adds HeadlessUI and Heroicons

Closes #29
Notes
I went with a select box instead of tabs because my thought was if there are tons of providers in the future the tabs might get too cluttered. I originally tried the HeadlessUI Listbox component but it seemed easier to do a standard select box because it switches the page.
The redirect URIs would need to be updated for the providers, but it would stay consistent going forward as each provider has its own page.
I cleaned up some of the code to make it easier to add providers. Ideally, there should only need to be minimal Tailwind styles for providers as the new styles in index.css should have it covered.
I added Ethereum and Nhost, but I know we don't have those examples yet. It shows an "Example coming soon!" message instead of the provider and links to the Redwood auth docs (see screenshot). We could remove this, but I thought it might be nice to let people know even if we don't have an example.
Screenshots
Home page

Provider page

Successful Auth

Coming soon

The workflow with this layout for future providers would be:

Create a provider component
Add provider logo to src/lib/images
Add provider name, slug, photo, and component to arc/lib/providers

That's it, everything else should work as expected!

I've only looked at the screenshots so far, but visually this looks great Morgan! Only thing I'd change/add is to add a callout for help with adding the missing provider example. So instead of just saying "coming soon", I'd add something about help wanted or PRs welcome or something like that. Or "here's how to help add this" and then a link to our contributor guidelines.

@morganmspencer This is wonderful!
I turned on the Deploy Previews (as they had to be approved since you have not issued a PR before) so you should be able to see this site live now in a Netlify preview.
Let's  (1) add @Tobbe's suggestion and then (2) format the "polling refresh token" section similar to the way you did the userMetadata to preview the text overflowing the box:

and (3) add the how-to to the readme or contributing:
The workflow with this layout for future providers would be:

* Create a provider component
* Add provider logo to src/lib/images
* Add provider name, slug, photo, and component to src/lib/providers

and then I say we merge -- and we can think of some nice ways to improve the usermetadata or more social provider oauth logins etc in other issues.
Looks so good I bet we can link from Auth docs now!

@dthyresson where is the contributing doc located? I was going to add to it, but I can't seem to find it

I was going to add to it, but I can't seem to find it

@morganmspencer Sorry about that, I thought it was there, but it isn't. Please do add thanks!

@morganmspencer Let's also do 2 small other additions:

Add a site title (eventually we'll use React Helmet) via a new web/config/webpack.config.js

See:  https://redwoodjs.com/docs/webpack-configuration.html#configuring-webpack
module.exports = (config, { _env }) => {
  config.plugins.forEach((plugin) => {
    if (plugin.constructor.name === 'HtmlWebpackPlugin') {
      plugin.options.title = 'RedwoodJS Auth Playground'
    }
  })

  return config
}

2 - Add a TW footer so we can link to this GitHub repo and other sites eventually

Add a site title (eventually we'll use React Helmet)

Did you want me to just add React Helmet? It should be fairly easy as there are only two pages

Add a site title (eventually we'll use React Helmet)

Did you want me to just add React Helmet? It should be fairly easy as there are only two pages

Sure! ;)

For react Helmet you may want to look into adding a Layout if there is not one and using the new Router Set to learn how it can wrap pages.
See: https://redwoodjs.com/docs/redwood-router.html#sets-of-routes

All changes have been implemented!

That was fast! Nice!

Thanks! I am approving the PR to make a deploy preview. One moment ...

@Tobbe and @morganmspencer the Preview looks great
https://607c38f1e786c80007d2f36e--redwood-playground-auth.netlify.app/
Once you both are happy with @Tobbe suggestions, @Tobbe please approve and merge.
This is great. Excited to share with the rest of the team today and let them know how you've made this so much better.

@dthyresson I just resolved all of @Tobbe's suggestions

@morganmspencer I think a "home" link to get back to that most excellent list of all providers view, too, would be a nice quick add ;)

@morganmspencer I think a "home" link to get back to that most excellent list of all providers view, too, would be a nice quick add ;)

Oh, it is the logo. Found it. Sorry.

@dthyresson and @Tobbe, do any of the redirect URLs need to be changed in the provider settings now that this is merged?

@dthyresson and @Tobbe, do any of the redirect URLs need to be changed in the provider settings now that this is merged?
Oh they likely do. I got too excited and merged.
Let me create a new issue as I've closed and merged.


Adding DareU to the Showcase

@ide Just created a new one. Hope this works.

@darkrishabh - you need to sign the CLI as indicated above

@brentvatne Just did it.


Chat alerts show message content instead of "Chat alert triggered"

I asked this a while ago lol

What were


estimateGas hits an out-of-gas error for a transaction that works with hardcoded gas

I'm trying to send a transaction with web3.js, which I believe does an internal call to estimate gas before sending the transaction. During that estimateGas call, TestRPC errors with VM Exception while executing eth_estimateGas: out of gas. If I hardcode the gas to 188561, TestRPC mines the transaction successfully. I suspect a different gas limit is being used for estimation and action transaction processing.
The transaction I'm using is a contract deployment with this data:
{
  "from": <any address with ether, e.g. `0x90f8bf6a479f320ead074411a4b0e7944ea8c9c1` for `testrpc -d`>,
  "data": "606060405260008054600160a060020a031916331790556101ab806100246000396000f3606060405260e060020a60003504631a695230811461003c5780632f54bf6e1461004b5780638da5cb5b14610066578063d7f31eb914610078575b005b61003a6004356100fc33610052565b6100cd6004355b600054600160a060020a0390811691161490565b6100df600054600160a060020a031681565b604080516020600460443581810135601f810184900484028501840190955284845261003a94823594602480359560649492939190920191819084018382808284375094965050505050505061012433610052565b60408051918252519081900360200190f35b60408051600160a060020a03929092168252519081900360200190f35b15610121576000805473ffffffffffffffffffffffffffffffffffffffff1916821790555b50565b156101a65782600160a060020a03168282604051808280519060200190808383829060006004602084601f0104600302600f01f150905090810190601f1680156101825780820380516001836020036101000a031916815260200191505b5091505060006040518083038185876185025a03f19250505015156101a657610002565b50505056"
}`

I suspect a different gas limit is being used for estimation and action transaction processing.

interesting hypothesis!

https://github.com/ethereumjs/testrpc/issues/96

I'm also currently running into out of gas while executing eth_estimateGas. Shouldn't estimateGas never throw this error? Our contract does have a lot of code, but the point of testrpc should be to be able to estimate and deploy in any case, right?
As the VM throws this error, this issue should probably be in ethereumjs-vm?

@graup hmmm... yeah I would agree that its unnexpect to see estimateGas OOG... I think this is a problem with the estimateGas implementation, meaning its over in provider-engine.
Guess we can set some crazy high gasLimit there, then its up to the rpc method consumer to determine whether or not that gas usage is safe e.g. below the block limit

I remember there were some more comments in this issue, and someone pointed to https://github.com/MetaMask/provider-engine/issues/76 which has since been deleted. Can someone check if this issue has been fixed? I'm a little confused now.

which has since been deleted.
not sure what has been deleted? maybe you can clarify

The issue has not been resolved yet. As a fix for now you can remove the gas or gasLimit parameter from your params provided to eth_estimateGas

@kumavis The link I mentioned 404'd. All your other comments here were hidden, too! Now everything is back. Thanks!

I'm looking into this. One big thing while I determine if this is an error:

I'm also currently running into out of gas while executing eth_estimateGas. Shouldn't estimateGas never throw this error?

It should throw this error. If it doesn't, this leaves nodes open to a DoS attack. If you get an out of gas issue during estimateGas, then it's likely your request uses more than the available gas. That, or there's a bug.
I'm working now to determine if there is actually a bug. I'll do so by trying to fill up the gas limit to the max available (0x47E7C4). If I can get to the max in a simple contract using estimateGas, then there's likely no issue here.

I agree with @tcoulter that an OOG is appropriate in certain situations. now the most important question is "what does geth go"

@niran One of the important questions we should have asked what: What is the solidity code related to this call? Do you still have it? It's very likely that the constructor does, in fact, run out of gas.

Hmm, scratch that. Would help if I read your initial writeup better.

Alright, so just figured this out. The issue is that your data field's value is not prefixed with 0x. If you prefix it with 0x, estimate gas works correctly.
I wrote a test that you can verify yourself. Stick this in the TestRPC's test/request.js file, in the tests() function. Remove the 0x from the data field, and you'll see that it fails with an OOG error.
describe("gas limit", function() {
    it.only("aaa", function(done) {
      web3.currentProvider.sendAsync({
        "method": "eth_estimateGas",
        "params": [{
          "from": accounts[0],
          "data": "0x606060405260008054600160a060020a031916331790556101ab806100246000396000f3606060405260e060020a60003504631a695230811461003c5780632f54bf6e1461004b5780638da5cb5b14610066578063d7f31eb914610078575b005b61003a6004356100fc33610052565b6100cd6004355b600054600160a060020a0390811691161490565b6100df600054600160a060020a031681565b604080516020600460443581810135601f810184900484028501840190955284845261003a94823594602480359560649492939190920191819084018382808284375094965050505050505061012433610052565b60408051918252519081900360200190f35b60408051600160a060020a03929092168252519081900360200190f35b15610121576000805473ffffffffffffffffffffffffffffffffffffffff1916821790555b50565b156101a65782600160a060020a03168282604051808280519060200190808383829060006004602084601f0104600302600f01f150905090810190601f1680156101825780820380516001836020036101000a031916815260200191505b5091505060006040518083038185876185025a03f19250505015156101a657610002565b50505056"
        }],
        jsonrpc: "2.0",
        id: 27
      }, function(e, r) {
        console.log(e, r);
        done();
      })
    });
  });

really this is etheruemjs-tx's fault for ever accepting utf8 as a valid encoding mechanism for string


Rename GuildMemberNickChangeEvent to GuildMemberNicknameUpdateEvent

Pull Request Etiquette

[x] I have checked the PRs for upcoming features/bug fixes.
[x] I have read the contributing guidelines.

Changes

[ ] Internal code
[x] Library interface (affecting end-user code)
[ ] Documentation
[ ] Other: _____

Closes Issue: NaN
Description
This PR renames GuildMemberNickChangeEvent to GuildMemberNicknameUpdateEvent and some related methods. I discussed this change in the JDA discord about a two months ago and forgot to create a PR.

The naming convention is EntityUpdatePropertyEvent

@MinnDevelopment do you think that we should go ahead and add the GenericGuildMemberUpdateEvent hierarchy in preparation for there possibly being other member update events?
There haven't been thus far, but to do so later would be a breaking change.

@DV8FromTheWorld yes that sounds reasonable, however we don't need to do that in this PR.

Agreed. I'll put an issue into the backlog so we handle before v4 alpha is done.

Just noticed I forgot to update the method name.


ENC: Import/Export nbextension settings

I'm setting up nbextensions on a second computer.  It would be great to simply import my current settings from computer one.  I imagine the settings are saved in yamls, but I am unfamiliar with the details. Can an import/export option be added to the nbextensions menu?

The config for most nbextensions, along with their enabled/disabled status, is stored via the jupyter notebook's config API, in the appropriate section (which is notebook for almost all of them, though the javascript for nbextensions.py stores a single value in the common section, and the tree-filter extension's enable status is stored in the tree section). As a result, the config for nbextensions, either form this repo or others, isn't readily separable from anything else in the config, but I guess if you're trying to duplicate your original setup, that isn't too much of a  worry.
The config files are json files stored in jupyter's config directory, which you can find using
jupyter --config-dir

Or, if you'd like a list of the ones available and (potentially) relevant to nbextensions, you can use the following python snippet form the jupyter server machine:
import os.path
from notebook.services.config import ConfigManager
cm = ConfigManager()
for section in ('edit', 'tree', 'notebook', 'common'):
    conf_path = cm.file_name(section)
    if os.path.exists(conf_path):
        print(conf_path)

As for an import/export option on the nbextensions menu - I guess you mean something that'd let you download the existing config as json, or upload a new one? I guess that'd be possible, certainly, but whether it would make sense to put it in the nbextensions config menu, I'm not sure, since it can potentially affect other things?

Also, for anyone interested, the yaml files contain information for the config extension (that's the one in this repo that provides the nbextension configuration/enabling page at /nbextensions) to allow it to know the default values for, and update the correct keys in, the json config files through the jupyter config service javascript. The yaml files themselves never get altered.

Thanks for the info.  Locating the config file should work for future endeavors.

An update to this thread, nbpresent is the next iteration of RISE https://github.com/Anaconda-Platform/nbpresent.  Check  it out.

Thanks for the pointer, but I think you may have meant this for another thread?

@par2 This link is very interesting. Thanks for the information.

Correct jc91.  I removed the comment and reposted elsewhere. Thank you.

Thanks @par2. For anyone wondering, it's now at https://github.com/ipython-contrib/IPython-notebook-extensions/issues/565#issuecomment-217563820


Join form code

Related Issue
#569
Proposed Changes

Change in code of Join us form by removing two form tag and using 1 tag instead.

Additional Info

Any additional information or context

Checklist

[x] ✅ My code follows the code style of this project.
[ ] 📝 My change requires a change to the documentation.
[ ] 🎀 I have updated the documentation accordingly.
[x] 🌟 ed the repo

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

As beacause of two tags only half information was sent at a time so this issue was needed.

Output Screenshots

Screenshot #1
Screenshot #2

Title goes here
Title goes here

Image goes here
Image goes here

it doesn't look good in mobile view, also has conflicts while merging


⚠ Bofrost.es has degraded performance

In 708cdd4, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1677 ms

Resolved: Bofrost.es performance has improved in 3718893 after 6 minutes.


[core-lro] Prepare release

With a fix for an ICM

API change check
API changes are not detected in this pull request.


[Pocket] Extension doesn't work

Extension
https://www.raycast.com/vimtor/pocket
Raycast Version
1.85.2
macOS Version
15.1
Description
When I try to use the extension after downloading it, using the "save current tab" command, I get this error:

Error:
RequestError: self-signed certificate in certificate chain

ClientRequest.<anonymous>:index.js:792:106

---
789:             this._aborted = true

### Steps To Reproduce

1. Download the pcoket extension
2. Run any command

### Current Behaviour

_No response_

### Expected Behaviour

_No response_

@raycastbot close this issue


Shortcut for inputing predefined SQL from table/column view

Provide shortcuts to input predefine sql template into editor

table shortcut

SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;

column shortcuts

primary key column

SELECT count(*) FROM %TABLE% GROUP BY %KEY_COLUMN%;
SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC WHERE %KEY_COLUMN% = ?;

field column

SELECT %FIELD_COLUMN% FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;
SELECT max(%FIELD_COLUMN%) FROM %TABLE%;
SELECT min(%FIELD_COLUMN%) FROM %TABLE%;

time index column

SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;
SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC WHERE %TIME_INDEX% < ? LIMIT 100;

Label for each shortcut:

Query table SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;
Count by SELECT count(*) FROM %TABLE% GROUP BY %KEY_COLUMN%;
Filter by SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC WHERE %KEY_COLUMN% = ?;
Query column SELECT %FIELD_COLUMN% FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;
Query max SELECT max(%FIELD_COLUMN%) FROM %TABLE%;
Query min SELECT min(%FIELD_COLUMN%) FROM %TABLE%;
Query table SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC LIMIT 100;
Filter by SELECT * FROM %TABLE% ORDER BY %TIME_INDEX% DESC WHERE %TIME_INDEX% < ? LIMIT 100;


Allow use of JSONPath selectors

Where a mapper supports object queries like:
mapper.offsets[("people", 0, "name")]

Add polymorphic support for JSONPath strings as well:
mapper.offsets["people.0.name"]

I think this would need the annotation:
def get_position(self, key: JSONKeyTuple | str) -> Position:

It may or may not go directly into the offset dictionary for issues of memory usage, but I think it would be completely reasonable to add something akin to def get_offset(self, key: JSONKeyTuple | str) -> Offset to the interface. The only thing that will cause a bit of grief is JSON objects with the key "0" vs array indices, but I think that's a very solvable problem.


empty cell not correctly managed

Tested with 1.5.13:
A = cell(1);
save('empty_cell.mat', 'A', '-v7.3')

load with matio:
matvar_t * elementMatVar = Mat_VarGetCell(matVariable, (int)k);

elementMatVar->class_type with k = 0 returns: MAT_C_EMPTY and not MAT_C_DOUBLE (expected)
elementMatVar->data_type MAT_T_UNKNOWN
same code with a value works:
A = {1} // or  A = {[]}
save('cell.mat', 'A', '-v7.3')

Is this only the case for v7.3 MAT-files or can you also reproduce when saved with -v6 or -v7?

same problem with v6 or v7

save using matio works but not an original .mat file

Can you please attach all three file here. Thanks.

>> A = cell(1);
>> B = {[]};
>> isequal(A, B)

ans =

     1
>> whos
  Name      Size                    Bytes  Class

  A         1x1                         4  cell array
  B         1x1                        60  cell array
  ans       1x1                         1  logical array

Though the two variable are equal, they are not. The difference is in the size. Variable A is en empty cell, variable B is a cell containing an empty array (of type double). This is also what matdump reports:
$ matdump -f whos -d empty_cell.mat
Name                       Size           Bytes          Class

A                          1x1                   4  mxCELL_CLASS
B                          1x1                  60  mxCELL_CLASS

$ matdump.exe empty_cell.mat A
      Name: A
      Rank: 2
Class Type: Cell Array
{
    Empty
}

$ matdump.exe empty_cell.mat B
      Name: B
      Rank: 2
Class Type: Cell Array
{
}

For variable A there simply is no data type information available. That is why you get MAT_T_UNKNOWN. Changing it to MAT_T_DOUBLE by default doe not seem straight.

But I have to admit that this kind of empty variable is not yet covered by the test. In matio_test_cases.m we have the following cases for empty cells
%% Cell-Array Variables
var51 = {}; // or var51 = cell(0);
var52 = {[] single([]) int64([]) uint64([]) int32([]) uint32([]) int16([]) uint16([]) int8([]) uint8([])};


Incorrect instructions in docs

In docs it mentioned this instruction

Start a 0-db locally, in sequential mode:
zdb --mode seq
Then start zdbfs with autons option, to mount filesystem on /mnt/zdbfs:
./zdbfs -o autons /mnt/zdbfs
That's it.

This gives these errors:
❯ zdbfs -o autons /mnt/zdbfs
[+] initializing zdbfs v0.1.10
[+] blocks cache size: 4096 KB
[+] zdb: auto creating namespace
[-] zdb: (no namespace selected): nsset: This server instance is configured to use a single runtime mode
[-] zdb: could not auto set metadata namespace mode: zdbfs-temp
[-] zdb: (no namespace selected): nsset: This server instance is configured to use a single runtime mode
[-] zdb: could not auto set data namespace mode: zdbfs-data
[-] zdb: (no namespace selected): nsset: This server instance is configured to use a single runtime mode
[-] zdb: could not auto set temporary namespace mode: zdbfs-temp

Debugging this, the autons option will let 0dbfs calls zdb_nsset function which try to use zdb nsset command to set the namespace mode to seq
https://github.com/threefoldtech/0-db-fs/blob/df690b44561e8b7bc7d9ab8e78eba0ddc8ce179f/src/zdb.c#L191-L202
Which cause the error reported above, as setting namespace mode disabled from zdb side unless the zdb instance started without --mode option.
https://github.com/threefoldtech/0-db/blob/08cd0831889d55aa151b18343461ad10949d9128/zdbd/commands_namespace.c#L429-L436
Also I noticed that these errors are mostly harmless, setting namespaces mode failed but they are already created using the correct mode (zdb started in seq mode) so yes still working but the errors still confusing unless you dive into the code to understand what is wrong.
So I suggest instruction could instead referring to start the local zdb instance in mixed mode (without --mode option) to get rid of these errors.
As alternative to this suggestion, we can display (from 0dbfs side) this error as a warning and note alongside that this warning can be ignored if zdb instance started in seq mode.

Thanks for the report ! That's indeed a legacy part, it was a new feature to support mixed mode (first because it's the default mode and second because zdb on the grid are running mixed mode). Documentation were not updated since this get applied.
There is a way to know in which zdb mode is running and based on that, code can be adapted to apply the NSSET or not. I'll implement that to not discard the error but apply the correct settings :)

https://github.com/threefoldtech/0-db-fs/blob/master/src/zdb.c#L533-L553
Just saw that this was already commented in the code, haha


Use pshazz ssh agent in git-bash or putty?

Storing the ssh decryption key in the windows credential manager is a brilliant setup.
Do you know if its possible, or which part of pshazz could be reused to enable an ssh-agent that works with git-bash or putty?

Just found https://github.com/lukesampson/askpass
Looks like the following gets this working in git-bash
~/.bashrc
# Note: ~/.ssh/environment should not be used, as it
#       already has a different purpose in SSH.

env=~/.ssh/agent.env

# Note: Don't bother checking SSH_AGENT_PID. It's not used
#       by SSH itself, and it might even be incorrect
#       (for example, when using agent-forwarding over SSH).

agent_is_running() {
    if [ "$SSH_AUTH_SOCK" ]; then
        # ssh-add returns:
        #   0 = agent running, has keys
        #   1 = agent running, no keys
        #   2 = agent not running
        ssh-add -l >/dev/null 2>&1 || [ $? -eq 1 ]
    else
        false
    fi
}

agent_has_keys() {
    ssh-add -l >/dev/null 2>&1
}

agent_load_env() {
    . "$env" >/dev/null
}

agent_start() {
    (umask 077; ssh-agent >"$env")
    . "$env" >/dev/null
}

add_keys() {
    export SSH_ASKPASS=~/AppData/Local/scoop/apps/pshazz/0.2014.11.25/libexec/askpass.exe
    export DISPLAY=localhost:0.0

    /dev/null | ssh-add
}

if ! agent_is_running; then
    agent_load_env
fi

# if your keys are not stored in ~/.ssh/id_rsa or ~/.ssh/id_dsa, you'll need
# to paste the proper path after ssh-add
if ! agent_is_running; then
    agent_start
    add_keys
elif ! agent_has_keys; then
    add_keys
fi

unset env

Would you be open to shimming pshazz/*/libexec/askpass.exe so that its easier to get access too?

Cool, you found askpass. Yeah I don't mind the idea of including a shim for askpass.exe with pshazz. Do you want to submit a pull request?

Sure, I can do that.  You don't understand how happy this makes me :laughing:

Hahaha!

Closing for now.  Maybe I don't even need pageant support.


Removed ".playground" suffix from the Downloads

Your downloads link always have a ".playground" suffix:

I think this made sense in the beginning when your downloads were always Playgrounds. But recently, with the composable architecture series, that's not longer the case.
Maybe is just me, but I was a little confused when clicking on a link that says "playground" and found a GitHub page with no playground in it :P
Maybe you can consider this small change 😉

(as described here https://github.com/pointfreeco/pointfreeco/issues/561#issuecomment-592128341 I was having issues with the test, but I think the snapshots are updated properly now.)

Ha that's true! We'll chat about it

Thanks for taking the time to get things running and making this PR 👍 Welcome to the PF team 😂

Thanks, I'm glad I could help :)


Swift 5.0

This also resolves #141.
Swift 4.2 or later is required.

Rebased to #141 and force pushed.

Thanks for taking care of this!


[Source Request]

PLEASE READ THIS
I acknowledge that:

I have checked that the extension does not already exist:

In the list of existing extensions in the application
In the multi-source spreadsheet: https://tachiyomi.org/extensions
By searching the GitHub repository for the extension and verify it does not appear in the code base

I have searched the existing GitHub issues and this extension does NOT have an open request
I will fill out the title and the information in this template

Note that the issue will be automatically closed if you do not fill out the title or requested information.
DELETE THIS SECTION IF YOU HAVE READ AND ACKNOWLEDGED IT

Source website

Name: Samehadaku
Link: https://<IP_ADDRESS>/
Language(s): Indonesia

Other details
Streaming Anime

Link not working anymore


DSD-611: Button NoBrand

Fixes JIRA ticket DSD-611
This PR does the following:

Add the NoBrand variant for the Button component.

How has this been tested?
Storybook
Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] All new and existing tests passed.

Front End Review:

[ ] View the example in Storybook

Tugboat has finished building the preview for this pull request!
Link:

https://pr770-p5pvfy8vwqah7b49zztpo5umuggtv0iw.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61966d96de4970639ae404a2

Tugboat has finished building the preview for this pull request!
Link:

https://pr770-p5pvfy8vwqah7b49zztpo5umuggtv0iw.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61966d96de4970639ae404a2

Tugboat has finished building the preview for this pull request!
Link:

https://pr770-p5pvfy8vwqah7b49zztpo5umuggtv0iw.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61966d96de4970639ae404a2

Tugboat has finished building the preview for this pull request!
Link:

https://pr770-p5pvfy8vwqah7b49zztpo5umuggtv0iw.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61966d96de4970639ae404a2

Tugboat has finished building the preview for this pull request!
Link:

https://pr770-p5pvfy8vwqah7b49zztpo5umuggtv0iw.tugboat.qa

Dashboard:

https://dashboard.tugboat.qa/61966d96de4970639ae404a2


Update version number and my contributions page

Fixes #229
Update version number to v1.5 and update my contributions page

LGTM!


Detect changes between null output file and non-existent output file

This is a fix for #3073.

@lptr Please have a look.

@lptr I addressed your comments, please have another look.

If we decide to not compare declared output file paths, we also should not persist them in the task history: https://github.com/gradle/gradle/pull/3090

@lptr Could you have another look?

@lptr I addressed your comments, please have another look.


Read Me

this issue is not from me personally just an idea, you could add a readme file for people who want to install your repo or app, to show them how to start the project,  because not all people are used to working with flutter,
happy coding, it's a good content

Thanks for the idea thought there's already a ReadMe file in the repository.


Create 2021-05-19-PancakeBunny

Hello, would you review please?

It's done. And I changed the article a bit, hope everything is done right

/qualitycheck

address relevant issues from bot fact checker.  add citations/references to the file as needed

status?

address relevant issues from bot fact checker. add citations/references to the file as needed

It's corrected, i just didn't correct much the items on Timeline section, but the link to the source is added via md syntax now, so i supposed it would work. Is it okay?

Please remove the article that has the wrong file name.

It's done

status?

waiting for the reviews:)

Please, resolve conflicts with the main branch

I think i was wrong from the beginning because used your file as a parent for mine https://github.com/Annyvichi/dn-institute/commit/6727b03130980873a83ede4458df2b71516eb444
and then failed trying to merge them https://github.com/Annyvichi/dn-institute/commit/5db92cc82739a11ed3f2bcb4c5ad73715a974f75
I never wanted to mess it all up, may I try to create a new pull request using a file on my computer?

I think i was wrong from the beginning because used your file as a parent for mine Annyvichi@6727b03
and then failed trying to merge them Annyvichi@5db92cc
I never wanted to mess it all up, may I try to create a new pull request using a file on my computer?

Sure. Branch from actual main and add your file with new PR, this can be closed when done.

I think i was wrong from the beginning because used your file as a parent for mine Annyvichi@6727b03
and then failed trying to merge them Annyvichi@5db92cc
I never wanted to mess it all up, may I try to create a new pull request using a file on my computer?

Sure. Branch from actual main and add your file with new PR, this can be closed when done.

Thank you! I hope i did better this time


Dropdown does not recognize active links

Environment

Operating System: Linux
Node Version:     v18.16.1
Nuxt Version:     3.7.1
CLI Version:      3.8.0
Nitro Version:    2.6.3
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      app, build, colorMode, css, devtools, experimental, modules, nitro, runtimeConfig, typescript, content, image, supabase
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>vue-debug-queryparam/nuxt
Build Modules:

Version
2.10.0
Reproduction
https://stackblitz.com/edit/nuxt-ui-fna9js?file=app.vue
Description
Links + VerticalNavigation mark active links with the specified active classes.  Dropdown does not.
In this example I would expect:

On the homepage, the homepage link in the dropdown to be red, even without hovering
On page1, the page1 link in the dropdown to be red, even without hovering
On page2, the page2 link in the dropdown to be red, even without hovering

There is indeed no active styles defined on the Dropdown links, what would you expect?

Because item.active and item.inactive exist in the UI property.
Because VerticalNavigation is a very similar component and it works this way.

If I have a dropdown that has a Settings link, but the user is already on the settings page, I would like to communicate that with something more than just disabled.  I'd like to communicate it exactly as I do in the VerticalNavigation (with the active style).

Will look into it!

I took a look at this issue and it appears that there were two problems in play. In the reproduction you linked, I was able to get some of your styling to be applied by adding dark: variants to the UI definition:
const ui = {
  active: 'before:bg-red-500 bg-red-500 dark:before:bg-red-500 dark:bg-red-500',
  inactive:
    'hover:before:bg-red-500 hover:bg-red-500 dark:hover:before:bg-red-500 dark:hover:bg-red-500',
};

This works, but it applies disabled styling to every item even if the item matched the active page. I took a look at the source and determined that I could pass a variable from the parent NuxtLink component and it would provide the functionality you were looking for without modifying the existing behavior.
I opened a PR to address the issue, but I believe the UI definitions would still need dark variants to function as expected. I am unsure if this is expected functionality of the functions that merge the tailwind classes or if a separate issue should be opened to investigate.


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1517 ms

Resolved: Bofrost.de performance has improved in 41a5cc3 after 53 minutes.


Add another point in note about using more than one subscription

Add a new point on the note that talk about using more than one subscription.
It's required to have the Microsoft.Sql provider registered on both subscriptions or else customer get an error talking about not having authorization.

@vtpombei : Thanks for your contribution! The author, @DhruvMsft, has been notified to review your proposed change.

@Mike-Ubezzi-MSFT please assign to oslake
@oslake per Joachim

#sign-off

@oslake: I'm sorry - only the author of this article, @DhruvMsft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

@CarlRabeler: I'm sorry - only the author of this article, @DhruvMsft, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.


Feature Request: Support multiple grouping objects

Support multiple grouping objects, e.g. users can create another grouping object in a sub dir.

Do we have a concrete use case for this? I would assume that the grouping objects here would correspond with kpt packages that have been pulled in as part of a composite kpt package. So then I can think of two reasons why we want to do this:

We want to in the future be able to apply changes to only a part of the composite package. So if we pull in an update from the upstream repo for on of the subpackages, we want to only apply that one package. But is there a reason why we shouldn't just call apply on the full top-level package here?
We want to keep some representation in the live state of the cluster that a set of resources actually belong to subpackages of a bigger package. So if we have a wordpress package that contains the mysql package, we have those two logical groupings in the cluster state. In this case I don't think we should use the grouping object for this. This is where something like the App CRD or ResourceGroup would be a better choice.


Feature Request: Problems view "source/owner" does not give sufficient information as to the source/owner of the problem report

Could either the "owner"/"source" in "Problems View" have the extension name or product name appended to a "Problems View report" such as:
[VSCode Code] _generated_diagnostic_collection_name_#3"
or
[Ext:ProblematicExtension] _generated_diagnostic_collection_name_#3"
so it is much clearer as to where the problem came from so mis-reports/bugs in the appropriate extensions etc can be reported to the appropriate maintainer.
Being able to right-click on the problem report to "Open/Go to Extension" would be very nice as well.
Reason for requesting
In the "Problems" view the output is:
[{ 	"resource": "/myresource", 	"owner": "_generated_diagnostic_collection_name_#3", 	"severity": 4, 	"message": "Syntax error: expected \"word\" somewhere in the file", 	"startLineNumber": 74, 	"startColumn": 51, 	"endLineNumber": 74, 	"endColumn": 51 }]
So I know  "_generated_diagnostic_collection_name_#3" was the source/owner of this problem report - but "who" are they? Which extension was it (or was it VS Code its self?) that generated this problem report? It is very unclear.
Platform:
Version: 1.69.2 (user setup)
Commit: 3b889b090b5ad5793f524b5d1d39fda662b96a2a
Date: 2022-07-18T16:12:52.460Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044

If you mean it is coming from an extension passing a bad model back to VSCode, then yes I agree that is probably the root cause: but it's difficult to tell which extension it is.
Could a feature such as:
If a problem is matched, but no owner is set, then grab the extension name/id (or use tasks.json if the problem matcher was in that) and set the owner to that, but also add an message/debug entry to Output or similar.
be possible to help these issues? (I suspect this may have been done to the Bash IDE extension, but I have definitely seen it with the Shellcheck extension (luckily, that report had an source field so I managed to track down and raise that fault with them )

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

/duplicate of #141987

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


assessment 1

drift video
screenshots:

approach

Metric
Value

alert_status
:white_check_mark: OK

bugs
1

code_smells
28

reliability_rating
E (at least 1 Blocker Bug)Note: A being best, E being worst

security_rating
A (0 Vulnerabilities)Note: A being best, E being worst

vulnerabilities
0


Ingest from Pulsar: relying on/ compatible to EOL version of Pulsar

Ingest from Pulsar: relying on/ compatible to EOL version of Pulsar
Details:
Pinot ingest streaming from pulsar currently relies on Pulsar client version 2.7.2
https://docs.pinot.apache.org/basics/data-import/pinot-stream-ingestion/apache-pulsar#supported-pulsar-versions
Java Client version numbering seems to be in sync with Pulsar (bundle)
https://pulsar.apache.org/docs/next/client-libraries/#language-specific-client-libraries
Apache Pulsar 2.7.2 was released on 2021-05-11
https://pulsar.apache.org/release-notes/versioned/pulsar-2.7.2/
Active and Security support for this version ended 03 Dec 2021
https://pulsar.apache.org/contribute/release-policy/
Current Pulsar/Pulsar Client version is 3.2.1 from 08 Mar 2024
https://pulsar.apache.org/download/
=> hard to start / to use the "pulsar+pinot dream team"

We have upgraded Pulsar to 2.11.* in release 1.1.0. Can you please try it out?

yep, good to see there was an upgrade with the shiny new 1.10!
Looking into Pulsars Release Policy, even for 2.11 active and also security support were already terminated
on 11 Jan 2024
https://pulsar.apache.org/contribute/release-policy/

Just for the overview and to put the work in the right order, here are some open pull request in the field of pulsar..
https://github.com/apache/pinot/pulls?q=is%3Apr+is%3Aopen+pulsar
at least one of them looks ready to merge (approved+ checks ok)

latest version is 3.2.1.
depend bot seems to have been canceled for this https://github.com/apache/pinot/pull/12123

From what I can tell, Pulsar is actively maintaining 2 major versions, and 3.2.1 and 2.11.4 are released on the same day (03/08/2024). We can evaluate 3.2.*, but I guess it is backward incompatible with 2.11.*

hmm when looking into their release policy it looks like this:

https://pulsar.apache.org/contribute/release-policy/

I see. Interestingly they still release on non-active support version. We will try to upgrade it to 3.2.1

fyi: v3.2.2 was just released, including some client fixes:
https://github.com/apache/pulsar/releases/tag/v3.2.2


Add dump with default fk name

Default name foreign key (fk_rails_XXXXXXXXXX) would not be dumped.
https://github.com/rails/rails/blob/v<IP_ADDRESS>/activerecord/lib/active_record/schema_dumper.rb#L227-L229
So will add --dump-with-default-fk-name option.

Hello,
Sorry, I know it's been a long time ago, but I am curious why caching hasn't been enabled before this (i.e., since December 17, 2014, when it became available to open source projects).
Thank you.


Leaderboard needs to handle underscores

Describe the bug
When displaying the leaderboard using !lb, italics mess up the formatting slightly.
Steps To Reproduce

!lb in chat

Expected behavior
An underscore in a name on the leaderboard should not affect the formatting.
Screenshots

Additional context
May need to look into other formatting changes that might screw up the leaderboard.

Probably can just do something akin to this using .replace()
https://stackoverflow.com/questions/35827838/how-to-show-underscores-symbol-in-markdown

Using several instances of backslashes like in markdown / discord to avoid the italicization didn't work.
What did work was changing from a md code block to a asciidoc code block. About the same formatting and ignores the underscores and other special characters.


Use TOML Config (WIP)

This PR starts to introduce a more robust way of configuring relevant build options and environment.
Currently there's a big surface area for options, and we still have some hardcoded leftovers. This means we currently have to do things like this: https://github.com/oskarth/mopro/pull/66/files/8e7d9dafaf0538ab84d739ec8a9f42437772df1a#diff-10095d80195138814b66f1d6f5b769b0763cd537d9ef563a28c6d75444a151d5 which makes things very confusing, especially when we have dealing with multiple circuits.
Big idea
Specify options in a toml file:
# config-example.toml

[build]
device_type = "simulator" # Options: x86_64, simulator, device
build_mode = "release"    # Options: debug, release

[circuit]
dir = "examples/circom/keccak256" # Directory of the circuit
name = "keccak256_256_test"       # Name of the circuit

[dylib]
use_dylib = false        # Options: true, false
name = "keccak256.dylib" # Name of the dylib file, only used if use_dylib is true

Then call this with build scripts like so:
./scripts/build_ios.sh config-example.toml
It is then easy to introduce multiple configs (e.g. for Anon-Aadhaar), to debug etc.
TOML config is also a good way to prepare for Rust API. I made it work for shell scripts too since that's the current base.
TODO before merge

[ ] Sanity check of initial API (feedback welcome)
[ ] Get rid of hardcoded stuff in build.rs and use TOML config instead
[ ] Update Android script
[ ] Make sure CI passes
[ ] Update README

Later on we can also add arrays to build for multiple platforms etc... starting simple though and just replacing existing positional argument and hardcoded stuff.


Make syntax and explanatory text consistent across DNA, RNA, AA

All changes were merged into main.


[Unity][Analysis] Add utility for collecting compile-time bindings

Whether an optimizations should be performed may depend on when the variables in an expression are known.
For example, consider a LoRA-adjusted model, with base weights W of shape [m,n], LoRA components A and B with shapes [r,n] and [m,r] respectively, and activations x with shape [n,1].  The LoRA-adjusted matmul could be computed either as (W + B*A)*x or as (W*x + B*(A*x)).
If A and B are provided at run-time, then computing (W + B*(A*x)) requires significantly fewer computations.

(W + B*A)*x: m*n*(2*r + 3) operations

B*A: 2*m*n*r operations using a naive matmul
Adding W to (1): m*n operations
Multiplying x by (2): 2*m*n operations

(W*x + B*(A*x)): (2mn + r*(2n + 2m + 1))

W*x: 2*m*n operations
A*x: 2*r*n operations
Multiplying B by (2): 2*m*r operations
Adding (1) and (3): m` operations

However, if A and B are known at compile-time, then computing `(W

B*A)*xgroups all compile-time values together, allowing them to be computed earlier (i.e. usingLiftTransformParams`)

(W + B*A)*x: 2*m*n operations

B*A: 0 operations, computed at compile-time
Adding W to (1): 0 operations, computed at compile-time
Multiplying x by (2): 2*m*n operations

Since the choice of optimized expression depends on which parameters can be computed at compile-time, it is useful to have a utility that identifies values that can be computed at compile-time.

The logic seems fine here, but what is the kNumInput attribute used for? Distinguishing weights from inputs?

The logic seems fine here, but what is the kNumInput attribute used for? Distinguishing weights from inputs?

That's correct.  The attribute was initially added for LiftTransformParams, to distinguish between parameters available at compile-time, and parameters not available until runtime.  It was later expanded to be available for more transforms in https://github.com/apache/tvm/pull/15736, and to be automatically generated for the nn.Module interface in https://github.com/apache/tvm/pull/16247.


fix: no extension able to load the configuration

fix: no extension able to load the configuration for "sylius_pay_pal_plugin" instead "sylius_pay_pal"

Q
A

Branch?
1.4 (bug fixes)

Bug fix?
yes

New feature?
no

Related tickets

Thank you, @Ezar101! 🏅


[Bug] sp_helpme & sp_sizeoptimiser SQLSTring & msg Variable Case Sensitivity

Describe the bug
sp_helpme and sp_sizeoptimizer have variables without proper casing so deployments to case sensitive collations fail. Correction on Line 209 and Line 435 within sp_helpme. Corrections on lines 1325 through 1330 within sp_sizeoptimiser
To Reproduce
Install-DbaMultiTool on Case Sensitive Instance/Database
Expected behavior
Install should complete
Versions(please complete the following information):

OS: N/A
SQL Server: N/A
SSMS: v18

Additional context
N/A

Thanks for the bug report and fix! Merged to development and will move to master soon.


Elemental validation and improvements when using SL Micro 6.0

This PR changes the way Elemental is handled depending on the API Version of the image definition. The reason being: 1.1 only supports SL Micro 6.0 and 1.0 only supports 5.5 so we can operate under that assumption when working with Elemental.
What happens now is that when using 1.0, the Elemental handling is the same and it uses the repositories found in artifacts.yaml, however, when using 1.1, those repositories are no longer used and instead the necessary Elemental packages are found through RPM resolution inside of the SL Micro 6.0 image. This now means that when using Elemental with 6.0, you need to specify a package registration code.
Additionally, to supplement this, I added validation for the Elemental configuration directory and for the registration code.

Quick update: The proper RPMs are now available in the SL Micro 6.0 repo, so concern #2 from Atanas' comment is addressed.

Please drop the unused properties from artifacts.yaml.


Add animated gif support

Perhaps gifs can be omitted from the resizing pipeline entirely as they are usually quite specifically configured.


Unable to generate image

It keeps on buffering. I tried on Edge, firefox browser and Firefox mobile. The results were identical.

I debugged problem by myself and found that html-to-image requires same origin. no-cors breaks everything. Same for html2canvas and I guess there is no way to fix it without bypassing cors.
I don't familiar with Next.js, but maybe there is possible solution with rewrites. Something like:
module.exports = {
  async rewrites() {
    return [
      {
        source: '/cors/:path*',
        destination: 'https://s4.anilist.co/:path*' // Proxy to Backend
      }
    ]
  }
}

And then replacing s4.anilist.co with /cors.

You could do
const downloadImage = async (): Promise<void> => {     setLoadingVisible(true);     if (refElement) {       try {         const dataUrl = await toPng(refElement.current, { cacheBust: true });         const link = document.createElement('a');         link.download =calendar-${year}-${month}.png;         link.href = dataUrl;         link.click();         link.click();       } catch (error) {         console.log(error);       }       setLoadingVisible(false);     }   }
which is in my pull request


Implement UB1 practical exercises

fixes #1

If I read this correctly the run script is not yet independent of the current working directory, right?

I think it boils down to do not change cwd because we will pass you relative path names (?)

But won't exec relative_path_to_compiler_binary fail?

Shouldn't exec ${0%/*}/target/debug/comprakt do it? I think we can assume that $0 contains the full path to the script

And isn't this a problem in the build script also? E.g. cargo build won't work if the cwd is not our project dir

bors r+

sorry I want to try something :smile:
bors r-

Well 4/5 approvals. Let's try this:
bors r=flip1995,joshuabach,reiner-dolp

4: Implement UB1 practical exercises r=flip1995,joshuabach,reiner-dolp a=problame

Nice it works!


Add a Filter for "Contains Person"

we want to allow users to search for what RCs they are involve in, useful for handovers between shifts. Please can we add a filter that pics up all the FRs / tasks assigned to individuals.

PR: https://github.com/equinor/lighthouse-client/pull/1714


[Bug] Do not have some badges at all, the badge count displayed incorrectly.

hello. Your open source project is great. However, I found one problem while using it.
The point is that the badge count is inaccurate.
For example, if user have 4 bronze badges, 0 gold buttons, and 0 silver buttons, the count for the 4 gold buttons, silver buttons, and bronze buttons count will not be displayed.
This is a situation in which the badge count is pushed back when some badges do not exist at all in the logic.
Inaccurate results are output for numbers in the five cases below.

only have gold badges
only have gold and bronze badges
only have silver badges
only have silver and bronze badges
only have bronze badges

I hope this can be improved. thank you :)

@soonhankwon Thank you for observing this and raising the issue, I had not tested this use-case. Will look into this and fix at the earliest :)

This is definitely an issue,

https://stackoverflow.com/users/3774282/faizan-anwer-ali
But in my profile I don't have golden badges

showing wrong stats(https://stackoverflow.com/users/16456741/basit?tab=profile)


fix: build output is now truly esm friendly and works well with cjs too

I've put in the work to truly fix what this PR intended to fix: https://github.com/yann510/ss-search/pull/92

:tada: This PR is included in version 1.10.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Add support for host operations to executor

This adds the GKO_REGISTER_HOST_OPERATION macro which can be used to annotate a costly core-side computation for benchmarking and (later on, once #1036 is merged) profiling. Additionally, it adds the OperationLogger to the host executor in GPU executions, which will make host-side operations visible in detailed benchmark results.
TODO:

[ ] Add tests

Closes #1211
Closes #1113

Yes, I agree this is a bit unergonomic. Do you see a way how to make that work considering that template functions can't be virtual?

Which virtual function do you refer to, Executor::run, HostOperation::run? I think both of them don't need to be virtual.

Having an overloaded function that is partially virtual and partially templated seems a bit dangerous, so I think we should choose only one of the two. Note that there is no such thing as a HostOperation ATM, as any potential later addition to what constitutes an Operation would need to be mirrored with HostOperation (e.g. work estimates, relocation and asynchronous execution for runtime systems)

We could use Executor::run_host, which might also be helpful to add some verbosity. I think we could derive HostOperation from Operation, then changes that apply to Operations could be easily integrated.
But of course that would not help with return values for device operations. Perhaps with some CRTP magic it could be possible, but that is a bigger head scratcher.

I will not hold up this PR anymore, because it is fairly small change and even though it adds the interface for host operations, I think removal of these requires update of code in develop. Additionally, as @upsj needs it for benchmarking purposes, from my side we can probably merge this once approved.

@upsj Before I approve it, add test is still left in TODO. Did you already finish it?


Swap communityAPI call for SNS

Agreed to split out the test tech debt work into
https://dsdmoj.atlassian.net/browse/MAS-375.


Update dial-pad-configuration.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/5686
Added NOTE.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@CarolynRowe PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

@scanum:  Did we check with PMs on this verbiage?  If not, I recommend we do so.  Let me know and I can follow up.

The wording in this PR could be misleading.  I will follow up with the PMs on the correct wording.  Closing this PR for now.


Added "lawnmower" to "activities" category.

Does not currently have character associated with it, since I don't know how to do that.

The emoji needs to be on https://unicode.org/emoji/charts/full-emoji-list.html to be added.


Can't upload SVG files

In the past I found I could upload SVG files and they'd successfully get uploaded to the file manager. It would display an error when they were uploaded (which would give the impression it failed), but it would still actually do the upload. Referenced here #2277
Now I can't upload SVGs at all, even if they're on the allowed filetype list they simply can't be uploaded.
I get the same error message/dialog as per the original issue.

I can't replicate this bug anymore. I think it was a one-off issue.

Cool – yeah I was about to try and replicate too


Some more examples

As Glanceclock finally killed their service, I am looking into using the clock at least for some basic notifications.
I have not done much using protoBuffer in the past and started playing around with https://www.protobufpal.com/, without getting anywhere so far.
I was able to connect to the clock and tried
gatttool --sec-level=high -b CLOCK_MAC -t random --char-write-req -a 0x001f -n<PHONE_NUMBER>03120141
with the correct MAC.
Tells me that it wrote to the characteristic, but see nothing happening on the clock itself.
Thanks for any additional help or recources you can provide.
There are a bunch of us looking for ways to use the clock and not send it to the bin ;)
Thanks
Alex

Did you pair the clock to the host you're running that command on or did you just connect to the clock?
Because iirc (it has been a while), without pairing them, the clock just silently ignores any commands.

Yes I paired it and connection works.

Seems, that we will also be getting more info from the company itself soon.
They seem to plan to share more  on Facebook this week (https://www.facebook.com/ClockGlance/).
Posted on XDA: https://xdaforums.com/t/glance-clock.4655490/post-89335420

Works now after disconnecting and connecting the clock again. I think I am getting the hang of it :)
gatttool --sec-level=high -b MAC -t random --char-write-req -a 0x001f -n<PHONE_NUMBER>061204416c6578
-> sends Alex ;)

Trying to get the digital watchface displaying again after the reconnect. Any pointer for that ?
Thanks for this so far and I will keep digging a bit more ...

The clock probably has no synced time and thus hid the digital clock among other stuff.
I'm using my glance clocks with my software that provides the service needed so that the clock can fetch the time
At some point, I threw some rough notes on how to set that up into an issue there:
https://github.com/Hypfer/Cybele/issues/12
However, those are just my notes so don't expect a finished product.
I'd also ask you to slow down please. I see that you're excited and all, but even though I reply quite quickly, you should not be doing the same. Take your time, think about the problem at hand. Try finding a solution and if you struggle feel free to ask but please do so in a way that shows that time and thought went into the exchange.
Thank you

Thanks for taking the time to reply.  I am not expecting anything and I am happy to be able to use the clock in some way again.
Already connected it via Node-Red to get most of my features back.
Not expecting a finished product or anything ales from you, just sharing my finding as many others are looking for a solution as well.
I know that this is not a support forum ;) and just a passion project of yours.
Thanks again for your time and I will try to internalize my excitement a bit more.
Cheers
Alexander


Unknown error occurred

Is this integration only for the TL 100?
I tried to integrate the WL75. But always get an Unknown error. Thanks Thomas

I have tried this with TL100 but apparently the integration cannot conect to my device after providing the MAC and name. The connection attempt just never ends.
As of a basic troubleshooting I was curious if the integration does indeed connect to it or not, so I have connected to the TL100 with my phone and the blue led was on and retried the connection from integration which instantly failed, disconnected my phone and retried connecting with integration but it seems to be a neverending story with connecting..


Add option for full ISO date format timestamps

Add TimestampISO as a boolean value in config defaulted to false.
If TimestampISO is true, timestamps will use time.DateTime format, otherwise it defaults to original time.Kitchen.

Just add a new configuration field "timestamp_format" (string) that accepts Go time layouts.

Just add a new configuration field "timestamp_format" (string) that accepts Go time layouts. The default value of the field is time.Kitchen.

Thank you! Pretty new to all this and just dabbling as I appreciate this project! The addition should be cleaned up and good to go, but if there's anything else please let me know :)

:rocket: LGTM! Thank you for the contribution!


FredBoat bot example usage not cited

Defect: "Dscirdo authenication" page uses FredBoat as an example. FredBoat is not properly cited, and should be as the
FredBoat bot is under copyright protection. It is highly recommended this is fixed before filing this software
artifact, as this (while highly unlikely) may be viewed in the future as copyright infringement.
Location of defect: "Dscirdo authenication" page
Recommended Fix: I recommend using a fake mockup example, or editing out any fields using the FredBoat logo or name. Alternatively, the bot could be cited properly.
Criticality: Moderate

We will look into this thanks.


Opening a second powershell and immediately closing it crashes terminal

Environment
Windows build number: Version 10.0.18362.175
Terminal Version: 0.2.1715.0

Steps to reproduce
Open Windows Terminal which defaults to an open tab with powershell.
Open a second tab with powershell, and immediately close it.
Terminal crashes/freezes.
Expected behavior
The recently opened tab simply closes
Actual behavior
Tab and the complete Terminal freezes. Need to ALT + Ctrl + Delete.

I had similar issue when I set "commandline" : "powershell -c dir" (which caused terminal to run dir and then close), but after a while it stoped crashing..

That seems like an issue within the terminal itself depending on how fast your computer is. But the issue really is visible when you open powershell.

Probably the same as #1445


fix(plugin/caddy): handle disable_query directive

We added the option to disable query string in keys (https://github.com/darkweak/souin/pull/319), which seemed to work according to the tests in context/key_test.go, but in real life situation it doesn't seem to work.
For example, a test in plugins/caddy/httpcache_test.go still appends the query string to the key.
func TestQueryString(t *testing.T) {
	tester := caddytest.NewTester(t)
	tester.InitServer(`
	{
		admin localhost:2999
		order cache before rewrite
		http_port     9080
		https_port    9443
		cache {
			key {
				disable_query
			}
		}
	}
	localhost:9080 {
		route /cache-default {
			cache
			respond "Hello, default!"
		}
	}`, "caddyfile")

	resp1, _ := tester.AssertGetResponse(`http://localhost:9080/cache-default?query=string`, 200, "Hello, default!")
	if resp1.Header.Get("Cache-Status") != "Souin; fwd=uri-miss; stored; key=GET-http-localhost:9080-/cache-default" {
		t.Errorf("unexpected Cache-Status header %v", resp1.Header)
	}

}

Results:
unexpected Cache-Status header map[Cache-Control:[] Cache-Status:[Souin; fwd=uri-miss; stored; key=GET-http-localhost:9080-/cache-default?query=string]

Closed by #328


Sponsorship Packet

Purpose: Linking sponsorship packet enables website visitors to access and view sponsorship packets without leaving the website.

Implemented PDF embedding feature for sponsorship packets.
Integrated two new sponsorship packet designs.
Ensured smooth display of PDFs on the web pages.

This looks awesome! After you get the font updated I'll resolve the issues from my code I merged into main (it breaks everyone's pages for the most part) and then I can get this merged in!

Just updated this branch to be consistent with the routing on the main branch.


Improve incremental update support for JDBC source

From @sabbyanandan on May 6, 2016 17:13
From @ctatineni on February 8, 2016 19:41
Hello,
just want to check if the current JDBC module supports Incremental data updates, for example, if i am reading the data from MySQL table , i want to make sure i only read the data that is added from the last read, either using offset columns or any other approach, also does it support if i am reading from multiple tables? does the same support available for NoSQL data from Mongo or Cassandra?
Copied from original issue: spring-cloud/spring-cloud-stream-modules#181
Copied from original issue: spring-cloud/spring-cloud-stream-app-starters#76

From @artembilan on February 8, 2016 22:27
Hello, @ctatineni !
Yes it does. The JDBC Source has options like:

query - to read data (the SELECT in SQL terms). Right it can be from several tables. Or with other words it may be SELECT with any complexity. And even the stored function call.
update - to update data, for example, for the next read to skip. For this purpose you even can use SELECT ... FOR UPDATE in the query. BTW with the SELECT my_func() FROM DUAL style you even don't need update since you can simple utilize it in the stored function.

We don't have such a CassandraMessageSource implementation yet. (Feel free to fill out an appropriate JIRA ticket.) Therefore we don't have concrete plan about Cassandra Source.
Yes, we have MongoDbMessageSource, but I see it doesn't have findAndModify ability...
Feel free to raise an appropriate JIRA as well.

From @ctatineni on February 9, 2016 19:48
Thanks, just to be clear,this is my understanding, if i use the existing JDBC module and provide a query, it will incrementally read the data from database, for example if it is trying to read from database every 5 seconds, if it reads 1 to 500 records during first read and next time it reads from 500.
Let me know if my assumption is correct.

@ctatineni: See below few stream definitions from Spring XD, but in essence these would apply for Spring Cloud Stream or Spring Cloud Data Flow model, too.
case 1: read from a table continuously (once every 5s)
stream create foo --definition "jdbc --fixedDelay=5 --split=1 --url=jdbc:hsqldb:hsql://localhost:9101/mydb --query='select * from testfoo' | log" --deploy

case 2: incremental reads (once every 5s)
stream create foo --definition "jdbc --fixedDelay=5 --split=1 --url=jdbc:hsqldb:hsql://localhost:9101/mydb --query='select * from testfoo where tag = 0' --update='update testfoo set tag=1 where fooid in (:fooid)'|log" --deploy

From @artembilan on February 9, 2016 20:15
No, it isn't so smart. :smile:
Although it seems for me I got it. You would like to do some paging without update:
SELECT * FROM
(
    SELECT a.*, rownum r__
    FROM
    (
        SELECT * FROM my_table WHERE
    ) a
    WHERE rownum <= :pageNumber * :pageSize)
)
WHERE r__ > ((:pageNumber-1) * :pageSize)

where those :pageNumber and :pageSize query parameters should be updated after each poll.
I can confirm that it is possible with the setSelectSqlParameterSource(SqlParameterSource sqlQueryParameterSource) for the JdbcPollingChannelAdapter, but it is a feature request for the SCS JDBC Source module. And we can consider to implement it here though.

From @ctatineni on February 11, 2016 15:32
Thanks for the clarification, i will use this approach for now, is there any plan to add this feature to existing SCS JDBC source?

From @artembilan on February 11, 2016 16:9
I think we will have them eventually.
It's a bit tricky to provide the paging SELECT wrappers for different vendors (that one above is for Oracle), but I think we can come up with something, since Hibernate can do that :smile:
CC @trisberg


Update cats-effect to 1.0.0

Updates org.typelevel:cats-effect from 0.10.1 to 1.0.0.

Already completed via another PR.


refactor: switched to arrow functions

Hi @Pizzo15, thanks for your PR. Unfortunately we can't accept it as<EMAIL_ADDRESS>does not support es6 syntax.


Add user option elfeed-search-remain-on-entry

When non-nil, keep point at entry after performing a command.
When nil, move to next entry.

Wait, sorry I mixed up the use-region-p... please stand by...

And fixed. Sorry for the noise.

Great PR, thanks!


Allow popover to be imperatively closed from within with a const { close } = usePopoverController();

I was surprised to find that Radix UI popover doesn’t have an imperative way to close the popover from within. In the example below I have added a const {close} = usePopoverController() and added a controlled state management for the root popover. Although this might not be desired as the main example, I thought I add it as an issue for other folks to potentially find it.
Note that this is some of our internal class names and do not match the default shadcn/ui!
"use client";

import * as React from "react";
import * as PopoverPrimitive from "@radix-ui/react-popover";
import clsx from "clsx";

const PopoverController = React.createContext<{
  close(): void;
} | null>(null);

function usePopoverController() {
  return React.useContext(PopoverController);
}

function Popover({
  defaultOpen,
  open,
  onOpenChange,
  ...props
}: React.ComponentProps<typeof PopoverPrimitive.Root>) {
  const [isOpen, setIsOpen] = React.useState(defaultOpen ?? false);

  return (
    <PopoverController.Provider
      value={React.useMemo(
        () => ({
          close() {
            setIsOpen(false);
          },
        }),
        []
      )}
    >
      <PopoverPrimitive.Root
        {...props}
        open={open ?? isOpen}
        onOpenChange={(open) => {
          if (onOpenChange) {
            // Implies a controlled component.
            onOpenChange(open);
            return;
          }

          setIsOpen(open);
        }}
      />
    </PopoverController.Provider>
  );
}
Popover.displayName = PopoverPrimitive.Root.displayName;

const PopoverTrigger = PopoverPrimitive.Trigger;

const contentStyles = clsx(
  [
    "bg-white",
    "text-popover-foreground",
    "z-50",
    "w-72",
    "rounded-md",
    "border",
    "p-4",
    "shadow-md",
    "outline-none",
  ],
  [
    "data-[state=open]:animate-in",
    "data-[state=closed]:animate-out",
    "data-[state=closed]:fade-out-0",
    "data-[state=open]:fade-in-0",
    "data-[state=closed]:zoom-out-95",
    "data-[state=open]:zoom-in-95",
    "data-[side=bottom]:slide-in-from-top-2",
    "data-[side=left]:slide-in-from-right-2",
    "data-[side=right]:slide-in-from-left-2",
    "data-[side=top]:slide-in-from-bottom-2",
  ],
  ["dark:bg-metal-900"]
);

const PopoverContent = React.forwardRef<
  React.ElementRef<typeof PopoverPrimitive.Content>,
  React.ComponentPropsWithoutRef<typeof PopoverPrimitive.Content>
>(({ className, align = "start", sideOffset = 4, ...props }, ref) => {
  return (
    <PopoverPrimitive.Portal>
      <PopoverPrimitive.Content
        ref={ref}
        align={align}
        sideOffset={sideOffset}
        className={clsx(contentStyles, className)}
        {...props}
      />
    </PopoverPrimitive.Portal>
  );
});
PopoverContent.displayName = PopoverPrimitive.Content.displayName;

export { Popover, PopoverTrigger, PopoverContent, usePopoverController };

in case you don't know, the popover has a Close part to close it, but if you want to close after async code, so you must control

The use case I have found is mostly for comboboxes or menus where a selection inside the popover should also close the popover. I ended up simply using a controlled popover without this hook, but I think it could still be useful for some use cases.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.


Api next

Hmm not sure if we want to mege to master already.
I'm pretty confident that current API is close to the final one, but there will be some changes for sure and I don't want ppl jump onto this until we can guarantee an API that won't change on a daily basis =)
Maybe there is way to have a "preview" release channel? So you do something like yarn add @stoqey/ib@next to get next instead of the master? So ppl can try it, but it's not on official release yet.
Don't do if this make more than 5min of work - it's not worth it - just need some time for testing / playing arround make is sure public APIs are stable before we go to master.

Totally agree with you @mfrener. Lemme know when you think it's ready. I'll try to implement my order functionality on it.
And go for a test run.
It's a draft branch, so it's not even ready for review ;)
Let's use the draft branches for easier reviewing while still in development.

Jep - major for focus now should to establish consistent design patterns for all type of functions, what work fast are reliable and easy are to understand for dev. When we have that it should be (no, must be) easy to missing the functions.
i.e. here is very good point https://github.com/stoqey/ib/issues/54 so there will be one rework on how we handle reqId's - fixing the mentioned issue and introducing the Subscription-cache / lookup at the same time. So better not have too many functions yet, because we need to change it all we change the pattern (but adding the order functions is definitly worth it).

Did a pass through read on this, just offering a few comments:

I think I understand correctly in that, generally, "cached" values are cached in the Subscription created for a specific request; so, invoking the word cache here does not mean that values from one call to something like reqContractDetails would necessarily affect values in a concurrent request to reqContractDetails ( just as example ). If that's true, cool, that seems good. Upon reading cache at first, I was worried that there might be an attempt to marshal data into a global cache of some form, which seems like it would have some nasty surprising behaviors in wait for anyone using this library.
reqMarketData however, has an odd effect. The cache essentially acts as an accumulation of values seen by prior ticks. This may just be a matter of documentation, but it seems like it would be possible to receive a tick update caused by something like an option computation tick update or what have you, and you, the receiver of updates, might also assume that the bid/ask is up to date because you did in fact receive a tick update. However, it's only a subset of the tick that has been updated, and if you were automating interactions w/ the market based on the ticket update, your view of the world could be pretty stale. Maybe allowing a user specified expiry on the data, such that, if the data received is older than some amount, the prior tick is thrown away as it is no longer considered valid. As mentioned tho, that could just be the concern of the caller, and the documentation tells them such.
I do not often work on web frontend things, so I cannot say w/ any real authority, but it seems like the majority of the JS ecosystem is adopting Promises as a way to handle asynchrony. I only mention it to say you may find yourself unnecessarily explaining the philosophy behind your choice to use RxJS / subscriptions in the future more than you like, as the community expectation further drifts towards expecting a Promise. I have no comment on the merit of the decision, I should note, just on the disconnect between what seems to be the current JS community trend and this.

Did a pass through read on this, just offering a few comments:

I think I understand correctly in that, generally, "cached" values are cached in the Subscription created for a specific request; so, invoking the word cache here does not mean that values from one call to something like reqContractDetails would necessarily affect values in a concurrent request to reqContractDetails ( just as example ). If that's true, cool, that seems good. Upon reading cache at first, I was worried that there might be an attempt to marshal data into a global cache of some form, which seems like it would have some nasty surprising behaviors in wait for anyone using this library.
reqMarketData however, has an odd effect. The cache essentially acts as an accumulation of values seen by prior ticks. This may just be a matter of documentation, but it seems like it would be possible to receive a tick update caused by something like an option computation tick update or what have you, and you, the receiver of updates, might also assume that the bid/ask is up to date because you did in fact receive a tick update. However, it's only a subset of the tick that has been updated, and if you were automating interactions w/ the market based on the ticket update, your view of the world could be pretty stale. Maybe allowing a user specified expiry on the data, such that, if the data received is older than some amount, the prior tick is thrown away as it is no longer considered valid. As mentioned tho, that could just be the concern of the caller, and the documentation tells them such.
I do not often work on web frontend things, so I cannot say w/ any real authority, but it seems like the majority of the JS ecosystem is adopting Promises as a way to handle asynchrony. I only mention it to say you may find yourself unnecessarily explaining the philosophy behind your choice to use RxJS / subscriptions in the future more than you like, as the community expectation further drifts towards expecting a Promise. I have no comment on the merit of the decision, I should note, just on the disconnect between what seems to be the current JS community trend and this.

The "cache" (haven't found any better wording so far) servs multiple purposes:
First is for aggreation purpose. Want to make it easy to differential or full-syncs, depending what you need. So on e.g. getPositions, the cache contains the your most up to date list positions. They aggrgate into the cache on position event until all positions collected, than position event will do incremental updates on that list. So we can have a PositionsUpdate.added, PositionsUpdate.changed, PositionsUpdate.removed and PositionsUpdate.all - we handle increment diffs already on the API.
Next reason is to be a ReplaySubject to the outside. It makes not much sense yet, but it will after next push / the introductoin of the subscription registry. This will make sure we never request same thing from TWS twice. So you can call getPositions() a 100x times, but instead of creating 100 subscriptions, you will create 100 observers on 1 subscription. Those new subscriber will never receive the "initial event" if we not rebuild a ReplaySubject, replaying currently cached value.

It's an ugly side-effect of behaving like a ReplaySubject, but otherwise new Observers won't get those ticks at all (e.g. sent after request and than never again - if you create a new Observer later on it will never see it until we replay it).
How about adding an ingressTimstamp to market data ticks?

I build an Angular web frontend, in parallel to the node REST backend. Angular is rxjs all the way, the controllers on serice are rxjs, ... all is rxjs... almost, the damn IBPlugin.ts is not. That's why I started IBApiNext, want to get rid of those ugly EventEmitter hacking accross the whole stack :D

Thanks for feedback! :)

Pushed a new feature - the subscription registry.
What it does (from IBApiNext doc)

[[IBApiNext]] does return RxJS Observables on most of the functions.
The first subscriber will send the request to TWS, while the last un-subscriber will cancel it.
Any subscriber in between will get a replay of the latest received value(s).
This is also the case if you call same function with same arguments multiple times ([[IBApiNext]]
will make sure that a similar subscription is not requested multiple times from TWS, but it will
become a new observers to the existing subscription).
In case of an error, a re-subscribe will send the TWS request again (it is fully compatible to RxJS
operators, e.g. retry or retryWhen).

It also changes pattern of the get-function on IBApi, down to calling this.subscripton.register().
Callback implementations have been moved out of the function for code readability reason.
Nex stepst: bring consistency to APIs that provide updates.
Example: PositionsUpdate has added/changed/removed/all while AccountSummariesUpdate only has changed/all and getContractDetails does not return updates at all, while internal  it gets result incrementally (e.g. you might want show first results already on UI while TWS is still delivering new ones).

Pushed a major update. It includes a whole bunch of fixing optimization work.
Change and the interface is abou that Map is ReadyonlyMap.
There also UnitTests for all RxJS wrapper up to now https://github.com/stoqey/ib/tree/api-next/src/tests/unit/api-next
They will run also w/o a TWS connection, because it tests the Event -> RxJS code, generating the event my itself.
Run it with "yarn test-next".
Next is to implement more functions.

The current state is available for preview on master now.
IBApiNext tests on CI are enabled and README has been updated to cover IBApiNext as well.


chore: allow manual dependency workflow trigger

:tada: This PR is included in version 1.1.8 :tada:
The release is available on:

v1.1.8
GitHub release
v1.1.8

Your semantic-release bot :package::rocket:


[struct_pack] Crossplatform support?

Hey guys!
First of all - thank you for pub useful stuff.
This is no issue but question, sorry for this contact method but here is no many ways.
For me very interesting if struct_pack will works well with different platforms and STL vendors, as example:
System 1 (Sender)

OS: Windows (x86)
STL vendor: Microsoft (MSVC)

System 2 (Receiver)

OS: Linux (x64)
STL vendor: GNU (GCC)

I will get correct data after deserialize at the end point if size_t defined different and if inner layout of STL containers not restricted?
With best regards, thank you.

Good question!
Here are some tips for cross-platform:

make sure CHAR_BIT == 8.
the machine should be all little-endian or big-endian. If not, struct_pack will detect the error with high probability (about 1 - 2^-16) by md5. But if hash collision happened, the bahavior is undefined.
the wchar_t and the type used wchar_t (e.g. std::vector<wchar_t>, std::wstring) is not cross-platfrom. struct_pack can't check this problem so the bahavior is undefined. But struct_pack won't enable those type without define marco STRUCT_PACK_ENABLE_UNPORTABLE_TYPE
In other case, struct_pack is cross platform. Thanks for you issue, I will add some test & document.

For your case, there is no any problem if you don't use wchar_t.

Thank you for response!
Importance of endianness nice tip, i will take into account the information.

Oh, I forget something.
For example, long is 32 bit in x86 msvc, but is 64 bit in x64 msvc. struct_pack will detect the error by md5, but it's better for your data struct to use int64_t, int32_t instead of long, int when cross-platform.

All right, thank you!
Will be nice too see in future full compatible without binding to system / bitness / endianness, sure you know that protobuf or flatbuffer already compensate these problems, but they not concurrents to struct_pack by speed or usage comfortable.
With best regards.

Will be nice too see in future full compatible without binding to system / bitness / endianness, sure you know that protobuf or flatbuffer already compensate these problems, but they not concurrents to struct_pack by speed or usage comfortable.
In common i see good future with struct_pack in game-dev, but hard to make point on it while i must care about end-point systems (it's can be mobile, laptop, pc and etc.).
With best regards.

Well, the problem of little-endian/big-endain convert is in my todolist.

I have add a test for cross platform, It works well when cross linux/macos/win32/win64. See #471

Now struct_pack support big-endian. So I think i can close this issue.


Tab to Modals

On the admin "wars" page, clicking a photo brings up a new tab with the war in it, change this to bring up a modal.

Not sure if I still want to do this. Once the site is a MVP I will revisit features like this.


Irc connect with ii

I'm a little confused on how to connect with a command line irc client. I'm trying to use ii so I can pipe chat from a costume rtmp service. Any advise would be appreciated.

I ended up using weechat. It supported sasl and has a fifo plugin to do what I need. I'm using another project to connect Instagram and multistreamer. It is grabing the comments from Instagram Live and sending to the IRC chat via weechat running as a service. It's not pretty but works.

Oh nice! For future reference (not sure if ii supports SASL), you can also connect by sending your password with PASS


Log data structure needs rewrite

I have a sketch for this... I'll give it a go this week, but may need help finishing the implementation.

I'm now working on this.
I may push a partially working version, but expect bugs for a few days until this is all sorted.

Here is a bit of code (using the examples data) to try the new log:
`
data = importcsv( "Epi_relax.csv", t_col =1, σ_col = 2, ϵ_col = 3)
sls_fit = modelfit(data, SLS2, strain_imposed, verbose=true)
d2=rheolog_run(data.log)
`
d2 would be a new version of data, reload and reprocessed.
There is a bit of work left to update all the other uses of the log throughout the code... but we'll get there!

Only arithmetic operators on RheoXData needed at this stage (I believe...). I never needed these operators, so I'm wondering how much complexity we need there. The new data generation format is quite flexible to provide complex data sets.

This is now completed. :+ and :- operators are classified as sources of new data sets, and can be reconstructed from the logs of each data sets added together.


Scale image

Hi,
My mockup image size 375 X 812.  How can I scale the image to fit the screen on any devices?

Technicly you can streitch you image like that...
PixelPerfect.extended(
image: Image.asset(
'assets/element.png',
scale: 2,
width: MediaQuery.of(context).size.width,
fit: BoxFit.fill,
),
..
)
but I recommend you to not to do it, instead of it create simulator for phone with the same screen size, for example many of iPhones have this size...
I'm closing the issue, but we can continue our dialog here.

But we should add the scaling feature in the plugin in future right?
I was testing it on chrome and wanted to design the UI for my phone, but the image was not scalable.
I'd say that we should maintain the image aspect ratio and just give an option to scale.

Sure, it's already there..
return PixelPerfect(
  assetPath: 'assets/design.png', // path to your asset image
  scale: 1 // scale value (optional)
  ..
  )
)


Argument of type 'null' is not assignable to parameter of type 'MappingContextValue'

🐛 Bug Report
For these 2 lines in theme.service.ts:
  static MappingContext = React.createContext<MappingContextValue>(null);
  static ThemeContext = React.createContext<ThemeContextValue>(null);

I get the tslint error:
Argument of type 'null' is not assignable to parameter of type 'MappingContextValue'

To Reproduce

Start a project with React community Typescript template
copy theme.service.ts to the newly created project

Steps to reproduce the behavior:
Expected behavior
No TS errors
UI Kitten and Eva version

Package
Version

@eva-design/eva
5.x

@ui-kitten/components
5.x

Environment information
React native community template Typescript

Please follow the Getting Started guide

Can you be more specific?
I did follow the Manual installation guide, the only step I did not run was this one (because I had not yet started the bundle, so I figure I did not need to reset it)
npm start -- --reset-cache

Sorry, missed it is a tslint error.
Yes, it probably could be, but it depends on your configuration. We use the custom one in this application as you may notice


Revert "Fix parse matmulinteger 611"

Reverts ROCm/AMDMIGraphX#2922

do not merge, this will be deleted by me