(Feature request) Boundaries for y values in solve_ivp

Hi guys and gals,
first of all thanks for the great work in general - makes my life a lot easier.
I have recently been playing around a lot with the solve_ivp function and the related solvers.
When using the stiff solvers (BDF, Radau), I have a lot of grief with the right hand side of my ODE system being called with "unphysical values", which results in exceptions and / or long run times. If I undertsand this correctly, this is done by the solver to approximate the jacobian. Hence I thought it could be useful if one could define lower and upper bounds on the values of t and y to tell the solver in which domain the system of equations is well defined and can be called
Looking forward to your opinion,
cheers
Hannes

Hi Matthew,
thanks for your input. Manipulating the numerical routines to suit my needs sounds like an interesting approach. Unfortunately I currently need to focus on other things, but when I get around to take a look at this again I will keep this thread updated.
Cheers

Hello there,
I am also using solve_ivp in problems for which it does not make much sense to talk about negative values of y . Not sure if it is the same case as @HPLegion , but I guess this problem could be partially solved if it was possible to change the values of y in handle_events, this way when any variable gets very close to 0, say y ~ 1e-12, it would be set to 0, diminishing the change of underflows and keeping the system in some sane region.
Is this a good idea ?
I could help implementing this, with some guidance.

I think this would be a good feature, and I would love to see it in SciPy. However, I am not qualified to implement it. Methods exist, but they are more complex than that proposed by @r-menezes. From the people I have talked to about this, it is not as simple as throwing a y[x < 0] = 0 as calling it a day. The tweaking of the state has to be done in a way that respects the assumptions of the ODE method or you will get crashes or wrong answers.

@drhagen , I see...  presumably these assumptions are also different for different methods?
as a workaround, I redefined my diff. equation so that it would return 0 for variables with values below some threshold.
def function(time, x_values):
    x_values[x_values < 1e-10] = 0.
    return dxdt(time, x_values)

not clean, but its a user tweak that prevents the negative values

presumably these assumptions are also different for different methods?

I don't know enough about the details of these algorithms, but I don't see why the assumptions could not be different.

Hi,

as a workaround, I redefined my diff. equation so that it would return 0 for variables with values below some threshold.
def function(time, x_values):
    x_values[x_values < 1e-10] = 0.
    return dxdt(time, x_values)

I remember doing something similar and I had mixed results (it probably depends a lot on the ODE itself and just how nonlinear it is). What I found dangerous about this is, that sometimes it would solve the problem but the solution was nonsense (and that was not always immediately obvious). However it has been some time since I dealt with this, and I don't really feel qualified to say too much about this. Consider it more of a friendly advice to be cautious.
In my case, iirc, I eventually went down a similar path, but instead of steamrolling negative values, I went through all the terms and functions that made up the RHS of my ODEs and adjusted their return values for negative inputs in a way that seemed most sound in terms of the actual physical interpretation of these quantities.
Good luck with your problem and I am intrigued to see what comes out of this. I imagine that this must be a fairly common challenge, so there would probably be some benefit to solving it, but I also acknowledge that the theory behind the solver's is not exactly trivial.
Cheers

Hi,
I think this is related to #12766, where it was suggested to implement a way to enforce positivity of the variables. I have implemented this for explicit methods and will soon propose a merge with my changes.
Regarding implicit solvers, a limitation of the variables has to be done in the Newton loop (solve_collocation_system for Radau, solve_bdf_system for BDF). Each time a correction dy to the state vector y is computed, the new state vector is norammaly computed as:
y_new = y + dy
Here, we would simply need to allow for a limiter function to be provided and call it in the following way:
y_new = limiter(y+dy)
I can implement this if someone still wants it.
Now this should only be used if we do not counteract the natural behaviour of the ODE, i.e. if the true solution wants to go past the bounds we have set for some variables, then the Newton loop will never converge, and I don't think there's a generic remedy for that. I think in this case the user may do two things:

redefine the ODE function so that the time derivatives of the the limited variables go smoothly to 0 as these variables approach their bounds
use an event to detect when a variable reaches its bounds, and then proceed with a corrected ODE function.

Regards

Hej :)
Thanks for the cross post!
Indeed my scenario (back then and still occasionally haunting me) was exactly about strictly positive inputs, so it's nice to see there is something happening on that front, I think it is a pretty common constraint.
Regarding the idea to

redefine the ODE function so that the time derivatives of the limited variables go smoothly to 0 as these variables approach their bounds

That is the approach I am currently using (with mixed results)

Hi @HPLegion
I have tried Radau and BDF on my test problem and they seem to handle it quite well without variable limitation. Do you have like a toy model that reproduce your issue with implicit solvers ?

Yes Radau generally seems to be extremely stable, have not played around with BDF much, since Radau usually does the trick.
No unfortunately I do not have a simple toy example. It's a pretty messy system of coupled equations made up of numerous (physics model) terms.


Use 0install-solver in tests

Meant to address persistent solvers timeouts causing CI tests to give false negatives.
Suggested by @kit-ty-kate

Motivated by discussion on #26473


Fix #209: Add AllNodes

Add AllNodes property with tests, as well as tests for AllPaths, AllNodes, AllFiles, and AllDirectories.

Note, this is for #209

I'm not really opposed to this feature, it's just a helper. Also adds cases for methods we already have that somehow escaped test coverage.

@cmeeren Could you resolve the merge conflict?

This has been lying dormant for so long that I've long since moved on to other projects. I can't remember making this PR in the first place... :P Could anyone else take over this PR?

Yeah, I'll take it over. Thanks for the ground work @cmeeren

Great, thanks!

Closing for #323


[Firestore] New documents with a persisted ID still creates a new ID

1. Version info

Angular: 5.2.0
Firebase: 4.12.0
AngularFire: 5.0.0-rc.6
Node: 8.9.3
Other (e.g. Ionic/Cordova, Node, browser, operating system): Windows 10
2. How to reproduce these conditions
Find all the source code and credentials to a sanitized Firestore project here: StackBlitz
2.1 Steps to set up and reproduce

Create a new Firestore project and bind to an Angular App
Create a reference to a new collection and try to add a persisted ID as specified in the documentation:

const id = this.afs.createId();
const item: Item = { id, name };
this.itemsCollection.add(item);

2.2 Actual Behavior

2.3 Expected behavior

Credentials are available in the StackBlitz project
2.4 Solution
The desired behavior can be achieved like this. So either there's a real issue, or the docs are outdated 🤔
const id = db.createId();
const user = { name:'juan' };
users.doc(id).set({ name: 'juan' });

Fixed!


Apply "conversion of spacebars from tabs"

Current, tabs in document are stored intactly.
But I think that conversion of spacebars from tabs is better than now.
So I will modify .vimrc

fixed.


Install via npm

Could you add package.json to install it via npm?
Seems, otherwise it's impossible to do. Here is a link on correspondent ticket.

Hi @dmitryt
Use the command npm install simptip --save-dev
Or
sudo npm install --g simptip

$ npm install simptip --save-dev
npm ERR! Linux 4.4.6-201.fc22.x86_64
npm ERR! argv "/usr/bin/node" "/usr/bin/npm" "install" "simptip" "--save-dev"
npm ERR! node v5.11.0
npm ERR! npm  v3.8.6
npm ERR! code E404
npm ERR! 404 Registry returned 404 for GET on https://registry.npmjs.org/simptip
npm ERR! 404
npm ERR! 404  'simptip' is not in the npm registry.
npm ERR! 404 You should bug the author to publish it (or use the name yourself!)
npm ERR! 404
npm ERR! 404 Note that you can also install from a
npm ERR! 404 tarball, folder, http url, or git url.

I think, it cannot be installed via npm in a such way, because it was not added into NPM registry. https://www.npmjs.com/search?q=simptip

'simptip' is not in the npm registry yet. I'll create a package.json in version 2.0

I think this can be closed as its already published to npm


A few new presets for NSI

NSI is depending on this repo for presets now - see https://github.com/osmlab/name-suggestion-index/issues/4718!
Here are some features being tracked in NSI that we didn't find iD presets for in the latest build:

[ ] Warning - no iD source preset found for amenity/motorcycle_rental
[ ] Warning - no iD source preset found for office/security
[ ] Warning - no iD source preset found for shop/repair
[ ] Warning - no iD source preset found for emergency/lifeboat_station
[ ] Warning - no iD source preset found for route/aerialway

Thanks! 👍

@bhousel Sure! Thanks for all your great work on NSI 🚉


Fix DCEGM/calcCrossPoints' return fields

DCEGM's calcCrossPoints, which I implemented a while ago, has an annoying behavior.
The number of things it returns depends on whether or not it finds crossing points in the value function. This produces bugs when one calls the function, as one can not know how many things will be returned. This tiny PR fixes that issue.

Makes sense to make this change.
A nitpick: to keep the type signature consistent, the second returned element should be an ndarray not a list.
This can be done with np.array([]) or np.empty(0)

Makes sense to make this change.
A nitpick: to keep the type signature consistent, the second returned element should be an ndarray not a list.
This can be done with np.array([]) or np.empty(0)

Great point. Fixed in the previous commit.

Great point. Fixed in the previous commit.


Add Dutch (nl) translations

This partially implements #74. What's missing is the glue to activate the translations.

@siebrand Could you look at my suggested changes?

Updated with all comments, but one.

I thought my thing fixed that


Chore/update information files

Updated files to harmonize with the other Sofie repositories.

Updated with Silje's corrected YAML templates.


building tree: batch save/get from state

Tests pending. See https://github.com/iterative/dvc-data/pull/93 for previous attempts/work.
Building objects
Before
$ time dvc-data build mnist
object e42412b82dcab425ce9c7e2d0abdfb78.dir
dvc-data build mnist  12.54s user 3.62s system 90% cpu 17.907 total
$ time dvc-data build mnist
object e42412b82dcab425ce9c7e2d0abdfb78.dir
dvc-data build mnist  2.27s user 0.25s system 99% cpu 2.542 total

After
$ time dvc-data build mnist
object e42412b82dcab425ce9c7e2d0abdfb78.dir
dvc-data build mnist  5.08s user 0.65s system 98% cpu 5.820 total
$ time dvc-data build mnist
object e42412b82dcab425ce9c7e2d0abdfb78.dir
dvc-data build mnist  1.16s user 0.15s system 98% cpu 1.318 total

data status
Before
$ time dvc data status
DVC committed changes:
  (git commit the corresponding dvc files to update the repo)
        added: mnist/
(there are other changes not tracked by dvc, use "git status" to see)
dvc data status  12.41s user 3.57s system 89% cpu 17.863 total
$ time dvc data status
DVC committed changes:
  (git commit the corresponding dvc files to update the repo)
        added: mnist/
(there are other changes not tracked by dvc, use "git status" to see)
dvc data status  2.79s user 0.23s system 99% cpu 3.035 total

After
$ time dvc data status
DVC committed changes:
  (git commit the corresponding dvc files to update the repo)
        added: mnist/
(there are other changes not tracked by dvc, use "git status" to see)
dvc data status  7.53s user 1.01s system 98% cpu 8.638 total
$ time dvc data status
DVC committed changes:
  (git commit the corresponding dvc files to update the repo)
        added: mnist/
(there are other changes not tracked by dvc, use "git status" to see)
dvc data status  1.94s user 0.21s system 99% cpu 2.169 total

The improvements are coming from set_many where we save hashes in a transaction and in one executemany, and get_many where we fetch multiple rows in one go (by chunking to 999 limit).
This required modifications to how we build tree, basically now we:

collect all files
compare all files path with state, to see if some paths are already hashed/unchanged and some paths are new.
we only hash the new files and save those hashes/checksums to the db immediately.
We combine both information from state and hashing to build a new tree.

This of course complicates the implementation a bit. But from what I see, building tree and a single objects are quite different that we should not try to keep them in sync, they have different requirements.

For the record: let me get back to this in a little bit, need to think about indexes, just so we can develop this further later on.

I’m thinking of reviving this PR, it’s almost a year and the problem is still there. I noticed that odb.add is batched and tries to save to state in a loop. So that part should be simple and straightforward to implement and won’t be difficult to maintain.


Add CLI parameters to README

Implements #33.

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

 All committers have signed the CLA.

Awesome, thanks!


403 - Acess denied error while installing elasticsearch alerting plugin

I am trying to install plugins from the plugins page :  https://opendistro.github.io/for-elasticsearch-docs/docs/kibana/plugins/
To install the alerting plugin, I ran the following command for elasticsearch v7.4.2
sudo bin/elasticsearch-plugin install https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-plugins/opendistro-alerting/opendistro-alerting-<IP_ADDRESS>.zip
It fails with error code 403 - Access Denied.
Seems like the file https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-plugins/opendistro-alerting/opendistro-alerting-<IP_ADDRESS>.zip is not public.
This issue seems to exist for other plugins as well.
Thanks.

Hi @srksumanth, apologies about this. The team changed the naming scheme of the plugins at least twice and didn't go back and rename/forward to the older artifacts. Try https://d3g5vo6xdbdb9a.cloudfront.net/downloads/elasticsearch-plugins/opendistro-alerting/opendistro_alerting-<IP_ADDRESS>.zip


Think about how to distinguish different types of data portal

We're using the term "data portal" a bit loosely to encompass both local governments that have standalone CKAN/Socrata/... portals and local governments that just exist as an organisation on their state's data portal?
I pondered making that distinction visible (it's not hard from my side) and then couldn't find a good term to describe the distinction that would make sense to participants and users. After all, most of them don't care about the distinction - they just want at the data.
I'm open to suggestions of how to better represent the "standalone data portal" vs "organisation within someone else's portal" distinction.

Maybe we  remove reference to data potals on the event location pages? Seems like duplication...

4 I guess my key concern is it looks like only adelaide city council has a portal...  when they are a "publisher on a portal" we have over 8 different councils who have thier own.publishing account on.data.sa + local gov association has data for all others... how do we make it fair...   and also not create a bucket.load of.work.for.us to maintain.
Im concerned anything we do we need to be able.to do.next year so i.want things to be as simple as possible.  We promote data portals and.offical data and that is.it.  we do not do.any special mapping
Also how are you going to know which local gov publisher sits with each event? Seems like a lot of work... remove refernce to data on the location event pages.

Think about this one a bit more.
Does a better "No wrong door for data" story solve this next year? Something that involves less education around "Here's how CKAN works to filter by organisation".
Maybe: Frontmatter on state portals that list publishing organisations to highlight?
Consideration: Having to update/repeat some of this next year to get new local publishers included.


Tweak to my-app.js and index.html to allow basic JS unit testing

Moves DOM interactions inside an init function, and calls this function from index.html.
This allows us to include my-app.js in a testing environment without worrying about DOM particulars - sets the stage for basic unit testing.
Also added an .editorconfig file because my four-space default conflicts with the two-space style used in this project :)
Please review @devgeeks !

Made suggested tweaks. Let me know if there's anything else I can do here before merging @purplecabbage or @devgeeks.


Runtime variables mangled by config expression

What happened?
I wrote a config expression which output the name of a runtime variable (to be used by DevSpace after the config expression is evaluated), but the name of the variable was mangled.
What did you expect to happen instead?
The name of the variable to not be mangled.
How can we reproduce the bug? (as minimally and precisely as possible)

Create the following devspace.yaml:version: v2beta1
name: test
dev: |
  $(cat <<EOF
  test:
    imageSelector: \${runtime.images.test}
  test-ok:
    imageSelector: \${runtime.images.test-ok}
  EOF)
images:
  test:
    image: hello-world-1
  test-ok:
    image: hello-world-2

Run devspace print
Note that the runtime.images.test variable is mangled in the output (also impacting runtime):version: v2beta1
name: test
images:
    test:
        image: hello-world-1
    test-ok:
        image: hello-world-2
dev:
    test:
        imageSelector: ${runtime___devspace___images___devspace___test}
    test-ok:
        imageSelector: ${runtime.images.test-ok}

Anything else we need to know?
It looks like this is caused by the code which makes variable names "valid" shell syntax:
https://github.com/devspace-sh/devspace/blob/6c1e41e994785543e16e3d39a27c64de286bff29/pkg/devspace/pipeline/engine/engine.go#L45-L53
The reason that the issue doesn't affect test-ok is that the regular expression does not match variables containing a -: https://github.com/devspace-sh/devspace/blob/6c1e41e994785543e16e3d39a27c64de286bff29/pkg/devspace/pipeline/engine/engine.go#L17
Should this replacement be applied during config expression evaluation? It seems like the runtime. variables aren't available at this point anyway (as the config is still being evaluated).

I've worked around this for now by doing this:
version: v2beta1
name: test
dev: |
  $(
    var() { echo "\${$1}"; }
    cat <<EOF
  test:
    imageSelector: $(var runtime.images.test)
  test-ok:
    imageSelector: $(var runtime.images.test-ok)
  EOF
  )
images:
  test:
    image: hello-world-1
  test-ok:
    image: hello-world-2

@jackwilsdon Hello! Here's another way of escaping the expression that might be a little easier:
version: v2beta1
name: test
dev: |
  $(cat <<EOF
  test:
    imageSelector: \$'{runtime.images.test}'
  test-ok:
    imageSelector: \$'{runtime.images.test-ok}'
  EOF)
images:
  test:
    image: hello-world-1
  test-ok:
    image: hello-world-2

You might also look into using a pipeline instead of config expressions. For dynamic dev config you can use --set to override specific fields. The above works too, but sometimes it's difficult to get everything interpreted by variables / config expressions to work together nicely.

Hey @jackwilsdon
Appreciate your help in improving devspace! 🚀
Closing this issue for now due to inactivity.
Please take a look at @lizardruss s suggestions above and feel free to re-open this if you're still facing issues

This still seems to be a bug in devspace IMO - there's a workaround but it's not ideal.


how can we use different date format in single page?

i tried to update provider value but the owl date picker parse formate is not updated.

This is NOT a issue from the picker. I would close this.
You could learn more about formation from here

@DanielYKPan , I already used the code below to use custom format.
`import { NgModule } from '@angular/core';
import { OwlDateTimeModule, OwlNativeDateTimeModule, OWL_DATE_TIME_FORMATS} from 'ng-pick-datetime';
// learn more about this from
// https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/DateTimeFormat
export const MY_NATIVE_FORMATS = {
fullPickerInput: {year: 'numeric', month: 'numeric', day: 'numeric', hour: 'numeric', minute: 'numeric'},
datePickerInput: {year: 'numeric', month: 'numeric', day: 'numeric'},
timePickerInput: {hour: 'numeric', minute: 'numeric'},
monthYearLabel: {year: 'numeric', month: 'short'},
dateA11yLabel: {year: 'numeric', month: 'long', day: 'numeric'},
monthYearA11yLabel: {year: 'numeric', month: 'long'},
};
@NgModule({
imports: [OwlDateTimeModule, OwlNativeDateTimeModule],
providers: [
{provide: OWL_DATE_TIME_FORMATS, useValue: MY_NATIVE_FORMATS},
],
})
export class AppExampleModule {
}
`
but if i need to update the value of OWL_DATE_TIME_FORMATS provider in ngOnInt, Is it possible?

that was my bad. got the solution.
need to set multi: true in providers section and  pass  @Optional()
@Inject(OWL_DATE_TIME_FORMATS)
protected dateTimeFormats: OwlDateTimeFormats in constructor and update it in ngOnInit.

@rahulgiri could you please help out? I am trying to change the format on button click or on ngOnInit but I can't make it work. Do you have a full example?


Fix DLPack header version

Follow-up of #4517. @tqchen kindly fixed this issue in upstream right after I reported it, so given that we manually bundle the header we can just apply the same fix ourselves.
This PR should be backported (after #4535).

Jenkins, test this please

Jenkins, test this please

Jenkins CI test (for commit b89e1ab3b0b9310fd26eaf44a66689ccd5a7091a, target branch master) succeeded!

Jenkins CI test (for commit b89e1ab3b0b9310fd26eaf44a66689ccd5a7091a, target branch master) succeeded!

@emcastillo This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

@emcastillo This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


Heatmap render Errors on Big Datasets

Found this on rendering big datasets in to a heat map.
From my point of view it looks like that the internal plotting runs into an variable limit (i guess singed short (2^15)) and than stretches the last row of data al the way to the end as seen in pictures,
OK Render less than 2^15 rows in y dimension

Faild renders at more than 2^15  rows

[Test]
[TestCase(3768,"outOK1.png")]
[TestCase(32768,"outOK2.png")]
[TestCase(34000,"outFail1.png")]
[TestCase(60000,"outFail2.png")]
        public void ScottplotBoundary( int traceCount ,string path) {
            Random r           = new Random();
            int    sampleCount = 500;

            var zData = new double[traceCount, sampleCount];
            for (var i = 0; i < traceCount; i++) {
                for (var i1 = 0; i1 < sampleCount; i1++) {
                    zData[i, i1] = r.NextDouble() ;
                }
            }

            var                plt = new Plot();
            CoordinatedHeatmap hmc = plt.AddHeatMapCoordinated(zData);
            plt.AxisAuto(0, 0);
            plt.SaveFig(path);
        }

found on nuget 4-1.6-beta -- running on core 3.1

Hi @Matthias-C,
Heatmap render based on Bitmap. Perhaps this is due to its limitations, then there is nothing to be done..
It is also possible that this is somehow related to the use of x86 mode(Any CPU), if you use it try changing it to x64.

I suspect @StendProg is right and that this is a limitation of System.Drawing.Bitmap.
I haven't tested it, but some information I found on Google suggests the maximum bitmap size is limited by the total number of pixels (not necessarily edge length), and that value is (2^16/2)*(2^16/2) or 1,073,741,824 pixels. @StendProg pointed-out that this value may be different on x86 and x64 systems, but is also worth testing.
When we conclude what the limitations of System.Drawing.Bitmap are, we should check the total number of values when heatmaps are created and throw an exception if more data is passed in than can be represented. If the user really wants to plot more data than this (probably a pretty rare case), they can easily create multiple heatmaps with carefully chosen coordinates so they tile nicely in the data area.

thanks for changing the title ... new one is way better.
for now I did some data compression to limit the amount of input data.
public double[,] RescaleArray(double[,] data, int targetWidth, int targetHeight) {
            int       sourceHeight   = data.GetLength(0);
            int       sourceWidth    = data.GetLength(1);

            double[,] compressedData = new double[targetHeight,targetWidth];

            double[] valuesTmp = new double[targetWidth];
            uint[]   countsTmp = new uint[targetWidth];

            double widthScale  = (double)sourceWidth  / targetWidth;
            double heightScale = (double)sourceHeight / targetHeight;

            int yTarget =0;

            for (int ySource = 0; ySource < sourceHeight; ySource++) {

                int xTmp =0;
                for (int xSource = 0; xSource < sourceWidth; xSource++) {

                    if (xSource >= widthScale * (xTmp + 1))
                        ++xTmp;
                    valuesTmp[xTmp] += data[ySource,xSource];
                    countsTmp[xTmp] += 1;
                }

                if (ySource + 1 >= heightScale * (yTarget + 1))
                {
                    for (int xTarget = 0; xTarget < targetWidth; ++xTarget)
                    {
                        compressedData[yTarget,xTarget] = valuesTmp[xTarget] /  countsTmp[xTarget];
                        valuesTmp[xTarget]               = 0.0;
                        countsTmp[xTarget]               = 0U;
                    }
                    ++yTarget;
                }
            }
            return compressedData;
        }

For the moment it's not fully tested, and not well optimised but gets the job done.

On my system System.Drawing will throw an error if you try to create an image wider than about 10M pixels or with a total number of pixels above 100M. I see from the screenshot above that with only about 30M pixels it won't throw an exception, but it will produce unpredictable behavior.
I added descriptive exceptions which throw if an edge is greater than 1M points or if the total number of points is greater than 10M. I think this will prevent this issue from occurring unexpectedly in the future.
Users wanting to display extremely large 2D heatmap data should probably investigate efficient image analysis libraries which are optimized for this type of math. This topic is definitely pushing the boundaries of ScottPlot's capabilities and intended scope!
Thanks again for pointing out this issue @Matthias-C

Hi @swharden,
I think this exceptions must be part of optional validation.
I use 2k*2k = 4M bitmaps a lot, and it's very close to provided limits...

I came to a compromise and like @StendProg suggested moved the size-checking code into the deep validation method. I also extended the XML documentation for the AddHeatmap() methods to indicate rendering may become unpredictable for large arrays with ~10M or more values.
Thanks again for both of your input!


Binance sending transactions with all kinds of nonces

I am not sure what tooling/wallet they are using. Hoping to get in contact. A user send this transactions which has been pending for 18 hours. This transaction with nonce 23118:
0x412410fd1d1d61ba07f58d214c4c06d0b3492340020780668ec98cb696f929e8
I see there is a nonce gap (23117) which explains why it is not getting processed. What I do not understand is why there are many more of these gaps. In addition some nonces are way off in the hundreds range instead of 23k range. Gaps (in green in the screenshot) commonly occur when transactions get dropped after maturity (256 blocks) that is not the case here. I do not know of the wallet simply skipping nonces. It does not make sense.
Is this the problem (hidden transactions): https://github.com/hirosystems/wallet/issues/3482
Who can analyse the mempool and see if there are indeed transactions pending but not showing on the explorer?
https://explorer.hiro.so/address/SP34EBMKMRR6SXX65GRKJ1FHEXV7AGHJ2D8ASQ5M3?chain=mainnet
Much lower nonces could be caused by some confusion about which account to check the nonce for. Disconnect between active account and signing account? Something like this: https://github.com/hirosystems/wallet/issues/3390

@cuevasm mentioned he believes Binance is using their own wallet, not Hiro Wallet?

+1, I'd be very surprised if they're using our wallet. Their systems are automated.

Not an issue, closing


ARROW-5769: [Release] Ensure setting up test data in dev/release/00-prepare.sh

+1, LGTM.  @kou I don't think this actually tested in CI right?  (I confirmed it works by running the added command on fresh env on my mac).

Thanks for confirming this!
I also confirmed this locally.

I don't think this actually tested in CI right?

Right. I want to test this on CI later.


az login tries to load powershell profile on bash linux/WSL

Describe the bug
Command Name
az login
Errors:
az login is trying to load my powershell profile on windows WSL Ubuntu.

Expected Behavior
az does not try to load my powershell profile when i'm in bash in Ubuntu WSL on windows
Environment Summary
Linux-5.4.72-microsoft-standard-WSL2-x86_64-with-debian-bullseye-sid
Python 3.6.10
Installer: DEB

azure-cli 2.19.1

Extensions:
log-analytics-solution 0.1.1

Account

While searching for possible causes for https://github.com/Azure/azure-cli/issues/16942. I noticed that when I run az login my Ubuntu window title changes to say powershell.
I checked the $PATH and it's long and contains a bunch of paths referencing my host (i.e. /mnt/c/Program Files)
Is this normal?

Could this be the root cause of the referenced bug and this bug too?

@rjfmachado, we encountered this issue before: https://github.com/Azure/azure-cli/pull/15361. I will submit a PR to fix it.

@ThoughtHopper, az login uses powershell.exe to launch a browser
https://github.com/Azure/azure-cli/blob/f92cea11e4e97af64c41ab7d35562639fc9d71b9/src/azure-cli-core/azure/cli/core/util.py#L650
This behavior was introduced by #12851, so yes, this is expected.
The terminal will revert back to its original name after az login finishes.


FizzBuzz Solution

This is my solution to Hacktoberfest. I hope you find it useful.

Thank you for your PR!
Unfortunately a solution similar to yours has already been submitted :/
The goal is to find creative/abstract new solutions to FizzBuzz.
See The top 5 samples here.
Thanks anyway! :)


Come work with us at Lightning AI !!

🚀 Feature
Calling all full-stack, backend, and frontend engineers to come build Lightning AI!!
Motivation
I'm looking for awesome developers to join our team and revolutionize AI with us!
Pitch
If you're seeing this- you already love Lightning and use it. We love to see that passion in the open-source community and would love to have you on the team :) You can do what you love AND get paid for it AND work with other awesome people 🎉
Alternatives
You can join a different company and have less fun 😉
Additional context
Job posts:

Fullstack Engineer
Frontend Engineer
Backend Engineer

Let's connect on linkedin: https://www.linkedin.com/in/noha-alon-9311aa80/
Add me on twitter ✌🏼 https://twitter.com/Noha_Alon

If you enjoy Lightning, check out our other projects! ⚡

Metrics: Machine learning metrics for distributed, scalable PyTorch applications.

Lite: enables pure PyTorch users to scale their existing code on any kind of device while retaining full control over their own loops and optimization logic.

Flash: The fastest way to get a Lightning baseline! A collection of tasks for fast prototyping, baselining, fine-tuning, and solving problems with deep learning.

Bolts: Pretrained SOTA Deep Learning models, callbacks, and more for research and production with PyTorch Lightning and PyTorch.

Lightning Transformers: Flexible interface for high-performance research using SOTA Transformers leveraging PyTorch Lightning, Transformers, and Hydra.

Hi @nohalon, I understand you are hiring software engineers. I saw the job postings, and they are all for senior roles. I am very interested in open-source projects as such, and I also like to contribute to them. I have been a PyTorch user since my undergrad, and now I am a final year master's student at Lakehead University (Canada), and I have been using lightning in my research 😄. I am very interested in what lightning is doing for the whole data community and would like to be a part of it.
I was wondering if you have any openings for new grads (I am graduating in April 2023) in Machine Learning or Software Engineering.
We can connect via  or you check out my some of my work


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1598 ms

Resolved: Bofrost.at performance has improved in 9ba08a7 after 6 minutes.


Compatibility with Screen API of platform version 7

Now, we can use standard actions frame only with AbstractWindow screens.

In v7 screens a new ProcessActionsFragment may be used. It is a copy of the ProcActionsFrame and should work in the same way.
To include the fragment to the screen:
<fragment id="procActionsFragment" screen="bpm_ProcActionsFragment" width="400px"/>

Usage in screen controller:
@Inject
protected ProcActionsFragment procActionsFragment;

@Subscribe
protected void onBeforeShow(BeforeShowEvent event) {
    procActionsFragment.initializer()
            .standard()
            .init("order-approval", getEditedEntity());
}


Phrase search considering synonyms

Would it be possible to use this matching library also for some smarter phrase search which would take into consideration spacy's word vectors?
For instance, if I create a matcher object like this:
import spacy
import spaczz

nlp = spacy.load('en_core_web_lg')
matcher = spaczz.matcher.FuzzyMatcher(nlp.vocab)
matcher.add("my_phrase", [nlp('humorous story')])

Then it would be maybe interesting to see also match for a sentence like in this example:
matcher(nlp('He told me a very funny story.'))

where there is a sub-phrase "funny story" which is a synonym to a phrase "humorous story" we added to the matcher.

Hi @Matt52, this is an interesting idea that I think could be implemented in a similar fashion to the way the fuzzy matcher is currently implemented but it would use spaCy's existing Span.similarity method instead of fuzzy matching. I don't have a definite timeline on getting this implemented but I will definitely start working on it as time allows.

Hi @Matt52 sorry for the really slow development on my end. Between work and the stress of the recent election cycle here in the US I have not been very productive. I have picked this issue back up again and have about 1/2 the work done in the new branch enhancement-similaritysearch. Hoping to have a release ready in the next couple days.
There will likely be more changes as I continue working on enhancing spaczz in general but the initial enhancement should be enough to get you started.

Hi @Matt52 I am publishing a new release (v0.3.1) shortly and will close this issue accordingly. Please check out the new details about the SimilarityMatcher in the readme when the release happens and let me know if you have any additional questions/issues. Thanks!


Volume Populators: fix feature gate name

This MR fixes the typo in recent blog post about Volume Populators.

/assign @mrbobbytables


Restore state crash

Fix crash when restoring state when an initially expanded row has been collapsed
fixes #122

Loses expanded state of parents at the end of the list on configuration change

changed restore instance state to recreate the itemlist vs dancing around it and modifying what is there.

Looks good :+1:


kie-issues#103: Upgrade pnpm from 7.0.0 to latest (7.26.3)

Upgrading pnpm version to latest (7.26.3).

Thanks for the changes! The readme in kie-tools\packages\kn-plugin-workflow\README.md also specifies a pnpm version if you want to update that as well.

The (build) kie-tools-stunner-editors check has failed. Please check the logs.


Change collection count property to method

Create branch from main
issue-35-change-collection-count-property-to-method


avfs: fix compatibility with go 1.23

In go 1.23, filepath.volumeNameLen has moved to internal/filepathlite.volumeNameLen.
Drive-by: fix golangci-lint errors.

@psadac Hi Paul. Please have a look at this PR when you get a chance. It's fixing the go 1.23 compatibility problem.

Looks like this landed upstream...


missing cmake-filesystem package

The platform module is missing cmake-filesystem binary package (from cmake component). We decided we will keep cmake-filesystem in platform, the rest will be in cmake module.
I built cmake-bootstrap module and then when building cmake module, I got this root.log error:
DEBUG util.py:439:    - nothing provides cmake-filesystem(x86-64) needed by cmake-3.9.3-1.module_7c9e41dd.x86_64

This is because platform module does not deliver cmake-filesystem package. Please add it.
See cmake-bootstrap build macros for disabling unneeded feature, especially:
%_with_bootstrap 1

for building it without already having cmake in the build root.

Fixed in the latest platform builds


Add backwards compatibility with old CL enums, bindings improvements

#886 was merged adding significantly nicer enums to Silk.NET.OpenCL, with the caveat that all of the old-style enums were entirely removed/broken. The aim of this PR is to add logic to the generator to minimize the breaking changes as much as possible such that we can release this in the 2.X cycle.
There are also some unrelated changes which I'm unsure what they were triggered by (basically anything to do with calling conventions), but I looked into it and it seems fine don't worry about it.
Also contains the latest bindings seeing as you can't really avoid regenerating the bindings when working on the generator.

Ready for review


Global styles manipulate styles of website

Hi, the global styles e.g. in viewer.css are changing styles of other vue elements outside of your component. (*, html, body, input, button, select,.. which are defined in your viewer.css) Please restrict them to your own container.. e.g. with a custom class on your main container?! Or you could use a shadow dom? Thanks! :)

Sorry the global styles are from pdfjs viewer 😏

Do you have any workaround for this issue or can you recommend other libraries?

I will fix it this weekend
Other libraries: https://github.com/vuejs/awesome-vue#pdf

Четверг, 25 февраля 2021, 10:55 +03:00 от armie-isla<EMAIL_ADDRESS>

Do you have any workaround for this issue or can you recommend other libraries?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly,  view it on GitHub , or  unsubscribe .

С уважением, Андрей Сергеев
e-mail<EMAIL_ADDRESS>

Thank you


Add basic CLI supports

Currently the CLI usage is
pymake <target> <variables>

Where target is defined inside PyMake.yaml file. This issue requests for additional ui features which include:

[ ] Adding pymake --help: showing available commands
[ ] Changing pymake <target> to pymake run <target> <variables> to make it more explicit
[ ] Adding pymake target --list: showing available targets defined in PyMake.yaml
[ ] Adding pymake target <target> --var: showing all variable information about the target. - i.e. required, positional, default
[ ] Adding pymake target <target> --env: showing environment variable information about the target.
[ ] Adding pymake target <target> --cmd: showing commands that will be run by invoking the target
[ ] Adding pymake target <target> --help: showing all information pertaining to the target.

Basic CLI commands provided in the latest PR #17
Current commands include:

list [--target TARGET] list available targets
inspect [-e| --env] [-v|--var] [-c|--cmd] target ...: inspect target
run target ...: run target

Each subcommands has an accompanying help.
Future subcommands to be added:

configure: set config, show configs, etc. Some config ideas:

echo: whether to echo commands
set colour for warning, suggestion, output, error
select pymake file
inherit pymake files

help: display program help information

version: show version number


Coordinate with the open source guild

We need to get in touch with the open source guild to inform them of changes in emails and google group setup.

Email sent to guild. Poked them on HipChat, too.


flutter: Running Gradle task 'assembleDebug'... taking forever after adding hive to pubsec

Steps to Reproduce
I just added both of these to my pubsec:
  hive: ^1.4.1+1
  hive_flutter: ^0.3.0+2

tried stopping and running the application again (full restart with rebuilding), and now the process stuck at:
Running Gradle task 'assembleDebug'...
Version

Platform: Android
Flutter version: 1.12.13
Hive version: 1.4.1+1

flutter doctor:
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.9, on Mac OS X 10.15.4 19E266,
    locale en-US)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.4)
[✓] Android Studio (version 3.6)
[✓] VS Code (version 1.44.2)
[✓] Connected device (1 available)

• No issues found!

Try running "flutter clean"  then retry

@dop3ch3f thank you for your suggestion. I already have. I guess the problem was due to the fact that the process was trying to download some additional files (I even had run "flutter pub get" before trying to build).
thanks anyway.

Well, now I'm getting build errors:
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring project ':path_provider_macos'.
> Could not resolve all artifacts for configuration ':path_provider_macos:classpath'.
   > Could not find builder.jar (com.android.tools.build:builder:3.2.1).
     Searched in the following locations:
         https://dl.google.com/dl/android/maven2/com/android/tools/build/builder/3.2.1/builder-3.2.1.jar
> Could not get unknown property 'android' for project ':path_provider_macos' of type org.gradle.api.Project.

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 9m 42s
Finished with error: Gradle task assembleDebug failed with exit code 1

the build error above seems to happen when including any package which uses local storage. The exact same error happens with "cached_network_image" for example.

the error is made by the hive_flutter package, and not hive itself.

didn't find out what is the exact issue, but I could finally build by deleting the packages from .pub-cache directory. In the above case, deleting path_provider_macos before build made it to build successfully.


fix(sha256): Perform compression per block and utilize ROM instead of RAM when setting up the message block

Description
Problem*
Resolves #5761
Resolution to performance blow-up found with sha256_var.
Summary*
Issue
The crux of the blow-up was the result of calling sha256_compression inside of the same loop where we build the message block. In the current sha256_var algorithm we are looping over the entire message and conditionally checking a msg byte pointer (the pointer into the msg block) to determine whether we have filled up a msg block and should run the sha compression. However, in a circuit this leads to us calling the compression opcode N times where N is the size of the message.
We also were utilize RAM to build our message block when we do not have to do so. We can instead construct our block outside of the circuit and verify that the block has been constructed as we expect with assertion that just require ROM.
Improvements
This code produces >125k ACIR opcodes on master and >500k backend constraints:
fn main(foo: [u8; 95], toggle: bool) {
    let size: Field = 93 + toggle as Field * 2;
    let hash = std::sha256::sha256_var(foo, size as u64);
    println(f"{hash}");
}

Output of nargo info on this PR:
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| Package                    | Function                   | Expression Width     | ACIR Opcodes | Brillig Opcodes |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | main                       | Bounded { width: 4 } | 6984         | 1058            |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | build_msg_block_iter       | N/A                  | N/A          | 294             |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | pad_msg_block              | N/A                  | N/A          | 209             |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | attach_len_to_msg_block    | N/A                  | N/A          | 298             |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | print_unconstrained        | N/A                  | N/A          | 244             |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | directive_integer_quotient | N/A                  | N/A          | 6               |
+----------------------------+----------------------------+----------------------+--------------+-----------------+
| sha256_var_size_regression | directive_invert           | N/A                  | N/A          | 7               |
+----------------------------+----------------------------+----------------------+--------------+-----------------+

bb gates output:
{"functions": [
  {
        "acir_opcodes": 6984,
        "circuit_size": 44663,

Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [For Experimental Features] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

This solves #5761, can you add this as a test program as a regression test?

This solves #5761, can you add this as a test program as a regression test?

Done

I guess we could remove test_programs/execution_success/sha256_var_size_regression if the test in #5761 is included. However, the larger size does show the blow-up more prominently.

Do you know why this version increases the circuit size?
I would imagine that using ROM instead of RAM would be better?
This PR does not fix the underlying issue, it just hides it, so if it degrades the performance, I don't think we should merge it.

Do you know why this version increases the circuit size? I would imagine that using ROM instead of RAM would be better?
This PR does not fix the underlying issue, it just hides it, so if it degrades the performance, I don't think we should merge it.

For the smaller hashes it does look we are still getting an increase and that is why I have left this a draft. Looking at a larger hash though, as per the PR description we have a >15x ACIR opcode improvement and a >10x backend opcode improvement.

Do you know why this version increases the circuit size?

I am also checking the entire msg block after an unconstrained function. In some cases like a constant message size, we can optimize away certain RAM opcodes if we know they will never be hit. Now they have been replaced with assertions that always happen. This leads to an increase for small hashes but not for large ones. I think this is a reasonable trade-off as most meaningful use cases will require larger hashes not hashes that are smaller than a single message block.

The description gives numbers but no comparaison with the previous version, and the benchmarks do not look good.
15x improvement is definitively very good!

The description gives numbers but no comparaison with the previous version, and the benchmarks do not look good.

I have updated the description to include the exact numbers on master and provide comparisons.

Holding off from merging as another review from @guipublic was requested.


Update README.md

earthdata is on conda-forge now!

Good work! Beat me to it! I was waiting for a "full" (not beta/alpha) release to start in on it.


Design/Implement new API docs

Based off of @nsthorat work parsing jsdoc and our new doc annotations. This task will pick up where that leaves off to generate the final layout we want to release.

[ ] Add TOC highlighting
[ ] Add links between symbols
[ ] Fix subsection header
[ ] General look through on vertical spacing

This is pretty much done though there is room for improvement. closing this in favour of more specific issues.


endpoint: move api-related functions to api.go

Fixes: #8987
Signed-off by: Ian Vernon<EMAIL_ADDRESS>

This change is

test-me-please

test-me-please

Coverage decreased (-0.05%) to 44.085% when pulling c694af15961348e492d34b7f3412ddfb403b4a5b on pr/ianvernon/8987 into 663041ef10b587e5ced69ca762da523e432de06b on master.


Add i18n string from TopicsLinkCardGridItem

Bug/issue #108050564, if applicable:
Summary
Add i18n string from TopicsLinkCardGridItem
Dependencies
NA
Testing
Steps:

Assert everything looks correct

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran npm test, and it succeeded
[x] Updated documentation if necessary

@swift-ci test


Crash after updating server/world

Hi, I just updated our server from create 3.1 to 3.2 and therefore installing flywheel in the process.
The server runs without issues and SP also works fine when creating new worlds.
If I try to connect to the server I'll get the following client-side crash: https://mcpaste.io/ad060e0bb8b184a0
It seems like some already existing create contraptions are causing the issue.
Thanks!
p.s. I disabled Optifine to test this, but it didn't change anything :(

Make sure you're using flywheel 0.2.3 and create 0.3.2d

Thanks!
Upgrading Create to 0.3.2d fixed it.
Curseforge defined the 0.3.2d version as "older" than the 0.3.2c version, which created the error while updating.
(just as note if anyone else encounters this issue)


refactor: SEO and Accessibility are 100% for main page

Fixes Issue
This PR closes the issue #649
Changes proposed
there are no major changes in structure or styles, added only necessary attributes and modified some elements to achieve better SEO and accessibility
Screenshots

Note to reviewers
However, there is a major performance issue due to poor core web vitals. This is something I may fix later.

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

If you have any questions feel free to ask, otherwise let me know your satisfaction of the changes with merging this PR and closing issue #649
thanks
skyDe

@kabir0x23 , I don't know about SEO stuff. Could you review it?

hi @Sky-De u have removed this file by mistake
can u fix this...

thank you

hi @Sky-De u have removed this file by mistake
can u fix this...
thank you

this image does not open for me
please let me know what exactly i removed
thanks

file_upload.md
Probably your anti-virus removed this file, the problem is that you staged and commit this file in git changes.
You should never use git add . / git add -a which added all files, you need always to see exactly which files and changes you stage and commit.

@Sky-De can you add this file back? If you have a problem with it , tell us.

@Sky-De can you add this file back? If you have a problem with it , tell us.

sure I can
but I'm busy with some work right now
I will fix it as soon as I have some free time.

ok, thanks, there is no rush :)

ok, thanks, there is no rush :)

You were correct; my antivirus kept removing that file. I added it back and created a new commit, hoping it would resolve the unintended issue.
let me know about it
thanks

@kabir0x23

@kabir0x23 @ArkadiK94 Is there anything else that is wrong with it? Let me know about it. Otherwise, let me know if you are satisfied with merging it.

@Sky-De We will let you know after this pr will be reviewed again.

@kabir0x23 @ArkadiK94  is is ok with you if i check other pages to find available same improvements and create issues and PR s?
Let me know
Thanks

@Sky-De
seo is only available for home page it's the issue with React. go to any other page, click on view source code u will see there is only one code of home page.
@kabir0x23 as you told me before.
@Sky-De this reason, nope , there is no need to check other pages.

@Sky-De still you are welcome to create other issues and we can discuss about the issue (not have to be related to seo).

@Sky-De seo is only available for home page it's the issue with React. go to any other page, click on view source code u will see there is only one code of home page. @kabir0x23 as you told me before.
@Sky-De due to this reason, nope , there is no need to check other pages.

@ArkadiK94 I respectfully disagree with that viewpoint, I will attach screenshots related to explore and feed pages :

Additionally, I believe that the main issue is the significant lack of performance on all pages, including the homepage :

To improve the performance of your app, we need to eliminate the loading time and make some design changes. However, we will discuss this after we have resolved the accessibility and SEO issues. If you are satisfied with the current state of your app, I will not create any further issues. However, I can help improve its SEO and performance significantly which will have a huge positive effect on SEO and UX of this app if you wish. It would be my pleasure to fix these vital issues. Please let me know if you are interested.

@kabir0x23 what do you think?

@Sky-De let's wait for a reply from kabir, but could you explain to me the warning message? Do you have experience with Seo in React apps?

those are getting render in different way, their way of doing is differnt

google this thing site:"thecyberhub.org"

u will not see any single page of our side other than homepage

@Sky-De let's wait for a reply from Kabir, but could you explain to me the warning message? Do you have experience with Seo in React apps?

@kabir0x23 that error is related to my chrome extensions and has nothing to do with Seo or accessibility , i will looking forward to @kabir0x23 response

@Sky-De let's wait for a reply from Kabir, but could you explain to me the warning message? Do you have experience with Seo in React apps?

@kabir0x23 that error is related to my chrome extensions and has nothing to do with Seo or accessibility , i will looking forward to @kabir0x23 response

see
no explore section on google as well
i am directly passing our eaxct url see, still home page only

@Sky-De let's wait for a reply from Kabir, but could you explain to me the warning message? Do you have experience with Seo in React apps?

@kabir0x23 that error is related to my chrome extensions and has nothing to do with Seo or accessibility , i will looking forward to @kabir0x23 response

see
no explore section on google as well i am directly passing our eaxct url see, still home page only

@kabir0x23 I see. Thanks for explaining but what about the lack of performance on the main page? Consider that to fix that it may need to  eliminate the loader to achieve better FCP and LCP

@Sky-De let's wait for a reply from Kabir, but could you explain to me the warning message? Do you have experience with Seo in React apps?

@kabir0x23 that error is related to my chrome extensions and has nothing to do with Seo or accessibility , i will looking forward to @kabir0x23 response

see
no explore section on google as well i am directly passing our eaxct url see, still home page only

@kabir0x23 I see. Thanks for explaining but what about the lack of performance on the main page? Consider that to fix that it may need to  eliminate the loader to achieve better FCP and LCP

Hey @Sky-De
U can do things as you want to improve
We will try to shift to different structure of the app where we will achieve this seo on Google.
Till u can fix those issues u get , It may help us later
Thank you 😄


If you start an OpenSearch cluster with 3 nodes at the same time, check the status of the nodes and the error message OpenSearch Security Not Initialized is displayed. [Bug]:

Describe the bug
If you start an OpenSearch cluster with 3 nodes at the same time, check the status of the nodes and the error message OpenSearch Security Not Initialized is displayed.
To reproduce
If you start an OpenSearch cluster with 3 nodes at the same time, check the status of the nodes and the error message OpenSearch Security Not Initialized is displayed.
Expected behavior
PossibleSolution
Restart the cluster every 30 seconds and restore the cluster to normal
Screenshots
If applicable, add screenshots to help explain your problem.
Host / Environment
No response
Additional context
No response
Relevant log output
No response

[Triage] Hi @hubcomsix, thanks for filing this issue. Could you please provide more information on the scenario you are encountering? For example, could you share any configuration changes you have made from the default config? Could you also share the time frame you are checking and whether you have let the cluster come up successfully before checking for the security index?

[Triage] Going to close this since it appears this issue is no longer relevant.


Words correction in TypeHandlerRegistryTest.

Just litte important function name in test case.

@Fudeveloper Thanks for the cleanup.

My pleasure!

发自我的荣耀手机-------- 原始邮件 --------发件人： Fudeveloper @.>日期： 2021年7月16日周五 10:01收件人： mybatis/mybatis-3 @.>抄送： Subscribed @.***>主    题： [mybatis/mybatis-3] Words correction in TypeHandlerRegistryTest. (#2289)Just litte important function name in test case.
You can view, comment on, or merge this pull request online at:
  https://github.com/mybatis/mybatis-3/pull/2289
Commit Summary
Words correction in TypeHandlerRegistryTest.
File Changes
M
src/test/java/org/apache/ibatis/type/TypeHandlerRegistryTest.java
(4)

Patch Links:
https://github.com/mybatis/mybatis-3/pull/2289.patchhttps://github.com/mybatis/mybatis-3/pull/2289.diff
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or unsubscribe.


Re add Tengam for AmunRa

This reverts commit 2164da46f28cc35898d5ba391f06af9f6dfbee71.
Add when AmunRa is done

missed opportunity to call this 'Magnet on' - reverses no tengam


Feature Request: Command for getting page title via the block uid

Issue:
Several commands, such as <%BLOCKMENTIONS%> and <%CHILDREN%>, have parameters for choosing the output format which includes {page}. However, to my knowledge, no existing command allows you to take a block uid and return the page title it was found on.
Example:
Say I wanted to find all instances of the tag client nested under tag/attribute type in my graph.
I can use <%BLOCKMENTIONS%> to find all instances of client, then <%REPEAT%> to run a callback to query if the <%PARENT%> of the <%ITERATIONVALUE%> contains the tag type.
However, I'm left with a collection of block uids, and no way to return the page titles those block uids were found on.
Current (hacky) solution
Currently, this can be worked around by creating a Query Builder table and returning the column containing the page, but I don't believe this can be done strictly via SB commands.
Proposed solution
The getPageTitleByBlockUid function already exists in the roamjs-components package. A command that hooks into this existing function would return the page title of a given uid

May be able to do this with <%BREADCRUMBS:+((blockuid))%>. The + before the block uid filters out the non-page title elements of the BREADCRUMBS


Handling long filenames of mesos-slave's sandbox dirs

rsyslog crashes if the filename that it has to monitor exceeds 200 chars. To work around that, we create a symlink to the task dir of mesos sandbox and add that path as the file to be watched in rsyslog. This should safely keep it under the 200 char file path limit

Note: I could not test this on my box at all. So, please please test it out before merging.

@ashwanthkumar - incase you weren't notified ;)

@brewkode We've taken your changes and made a few more on the top as part of #4. Thank you for this, closing this now.


mat1 and mat2 shapes cannot be multiplied (1323x9248 and 1568x32)

These are the shapes of my input, input_len, target, target_len where batch size=27

This is the setup I am running (only using first batch to check that is working before training with all the batches)

This is the error I am getting

I need some assistance here please:)

yes i had to transpose the input from [32, 201, 1162] to [32, 1162, 201] (batch, seq_len, dim)


Cewi/Excel Deprecation

I have used the plugin in my cakephp app and I have put in my DigitalOcean server. So my server is running with PHP 7.2.19 and I couldn't find a solution why it affecting to run cake.
The error I get,
"Deprecation warning: require.Cewi/Excel is invalid, it should not contain uppercase characters. Please use cewi/excel instead. Make sure you fix this as Composer 2.0 will error. Do not run Composer as root/super user! See https://getcomposer.orgroot for details."
Every time I try to update composer, this warning comes and tells Cewi/Excel plugin is deprecated.
Can you help me with this problem?

did you try it with "require cewi/excel"? I will check this later and adapt the documentation. HTH

to install the plugin simply use "composer require cewi/excel". I updated the readme.


Get publishing version from build metadata

Pretty self explanatory - get version from build_metadata.json instead of from version.txt.
Figure we'll still keep the version.txt, it's nice to have. But also have it be consistent across products now

noting that this was related to #776


bugfix and support ckpt saved by DDP

model defined in #50 will overwrite the input parameter model passed by main.py which makes --eval_from  out of action.
support checkpoint file saved by DDP

Number of classes are hardcoded again :(
https://github.com/PatrickHua/SimSiam/blob/d22ce2abbfc230e37cd055f14f932afba9360a08/linear_eval.py#L68

Number of classes is hardcoded again :(
https://github.com/PatrickHua/SimSiam/blob/d22ce2abbfc230e37cd055f14f932afba9360a08/linear_eval.py#L68

Sorry about that, but if you only modify the first defined classifier, the later one will overwrite it. Maybe we need another PR now


Support future-proof enums as enums

Description
Related #6061
Type of change
Please delete options that are not relevant.

n/a Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
n/a Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take action on it as appropriate
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Test Environment:

OS:
@graphql-codegen/...:
NodeJS:

Checklist:

[ ] I have followed the CONTRIBUTING doc and the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

This is great, thanks @ertrzyiks . Feel free to ping me if you need any help!

I will push it forward in the next weeks. I'm going to start vacations so I should have more time for it than now.

@dotansimha it's ready for an early feedback. I would love to play with it a bit more though, but from what I see the canary version is not available to forks (very reasonable though).
To sum up the changes:

keep futureProofEnums working for enumsAsTypes as it used to, the future value is added to the enum type
make futureProofEnums work for other types by changing the definition of enum usages
there already was an option tweaking the usage configured with allowEnumStringTypes, so to make it work with both flags I added an extra code branch for that case

@dotansimha can I do anything to push this PR forward?

@ertrzyiks Thank you 🥳


gc: unused variable bound to closure not recognized

gc accepts the following snippet of code, but gccgo and go/types report that: "x declared but not used":
func myfunc() {
  var x int
  go func() {
    x = 3
  }()
}

gccgo and go/types are probably correct here. x is only ever assigned to, but never returned or used.
If I make x a return value, none of one of gc, gccgo, and go/types complain.
func myfunc() {
  var x int
  go func() int {
    x = 3
    return x
  }()
}

Dup of #8560.


[prim] prim_flop_2sync depends on non-synthesizable code

Commit 43a24e6 added lowrisc:prim:cdc_rand_delay as a dependency of prim_flop_2sync.core. This causes lots of lint messages and I suspect will be an issue for the synthesis flow as well.
Would it make sense to depend on this from some DV base class instead?
@sriyerg, @weicaiyang, @cindychip: What do you think?

Ohh. I think we just need an ifndef VERILATOR in the file: the module contents are already stubbed out for synthesis.

Ohh. I think we just need an ifndef VERILATOR in the file: the module contents are already stubbed out for synthesis.

this sounds good.

sounds good to me as well.
I have another PR parallel to this that adds the missing waiver file (which
breaks ascentlint runs).
On Mon, Apr 11, 2022 at 2:36 PM weicaiyang @.***> wrote:

Ohh. I think we just need an ifndef VERILATOR in the file: the module
contents are already stubbed out for synthesis.
this sounds good.
—
Reply to this email directly, view it on GitHub
https://github.com/lowRISC/opentitan/issues/12046#issuecomment-1095590870,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2RSTIDGFIFGIDQFHZC33VESLNDANCNFSM5TEO6KTA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Suggest adding +define+DISABLE_PRIM_CDC_RAND_DELAY to verilator instead.

Sorry, Sri! I just realised that I hadn't replied to this. Yes, I agree: it's in my queue of tidy-ups to do!

Oh, again! It turns out that that's what I'd done in https://github.com/lowRISC/opentitan/pull/12047: apparently I'd come to the same conclusion as you, @sriyerg :-D


fix duplicate memory detection

Addresses #333: uniqueueMems should be constructed in terms of the entire circuit.
@shunshou I might have changed the algorithm with my refactoring. Also eqMems changes in accordance with the recent refactoring.

Hm ok... I don't remember any more... info was always a field of DefMemory, right? o_O Strange that I didn't exclude it...... Now info is just source locator, so I definitely don't want to look at it for uniquness... And other added annotations definitely still need to match.
Not sure why the tests get an UnknownType error... code seems reasonable...

I fix eqMems to pass tests

Is this branch ready for a trial run on my codebase?

@colinschmidt Yeah, let me know if this also works for your case.

Note that due to a change to chisel3, all Travis tests are currently failing. I'm trying to fix it.

Something weird is happening on this branch, I don't actually get any conf file out. I think that this could be due to several recent PRs(rocket-chip is far behind firrtl/master) of  so I'll try to do a bisect and see what I come up with.

Ok so #337 was this issue according to bisect. Once thats resolved I'll test this again.

with help from @chick I got a fix for #337 and was able to verify that this branch fixes my issue. Thanks!

Looks good to me pending Travis pass? Sorry, should've caught the ref myself.


[FEATURE] Display

Problem
Rendering to the window is a problem because we need a way to apply shading to the whole screen, which is hard without a proper virtual representation of a surface. We need to implement a Display to become this intermediate between rendering and displaying the rendered output to the window.

This got implemented as a Screen struct.


6TOPS: Linux is not booting

Linux is not booting on 6TOPS Accelerator, might be becuase of modules missing.

Fixed in v2023.12-6TOPS-rc1 tag


Update Error impl

Remove the implementation of the now-deprecated description method, and
implement source rather than cause.
Closes #94

Thanks :+1:


fix typo in comment

changed "Check is fit had been called" to "Check IF fit had been called"

This has been fixed during the revamp. Thanks @lacava


disallow cygwin on windows builds

we should check for it and disable it since it's subsumed and not worth maintaining nor confusing the other better way under windows using the official subsystem nowadays

What's the "official subsystem"? As far as I know, you can use iov-core from a default Windows cmd as well as bash on Windows.


Issue with train script (translation task) - fairseq v0.7.0

Hi fairseq team, i have cloned latest version of fairseq and use it to train new translation model. The preprocess works fine but with train script, it gave me below error
The train script:

!CUDA_VISIBLE_DEVICES=0 fairseq-train fairseq/data-bin
--dropout 0.2
--max-tokens 4000
--arch lstm
--max-epoch=20
--save-dir checkpoints
--fp16 --encoder-bidirectional
--optimizer nag --lr 0.01 \

And error:

| model lstm, criterion CrossEntropyCriterion
| num. model params: 40918008 (num. trained: 40918008)
| training on 1 GPUs
| max tokens per GPU = 4000 and max sentences per GPU = None
| no existing checkpoint found checkpoints/checkpoint_last.pt
| loading train data for epoch 0
| fairseq/data-bin train en-vi 133318 examples
| epoch 001:   0% 0/970 [00:00<?, ?it/s]Traceback (most recent call last):
File "/usr/local/bin/fairseq-train", line 11, in
load_entry_point('fairseq', 'console_scripts', 'fairseq-train')()
File "/content/fairseq/fairseq_cli/train.py", line 302, in cli_main
main(args)
File "/content/fairseq/fairseq_cli/train.py", line 80, in main
train(args, trainer, task, epoch_itr)
File "/content/fairseq/fairseq_cli/train.py", line 121, in train
log_output = trainer.train_step(samples)
File "/content/fairseq/fairseq/trainer.py", line 236, in train_step
sample = self._prepare_sample(sample, self.args.fp16)
File "/content/fairseq/fairseq/trainer.py", line 496, in _prepare_sample
sample = utils.move_to_cuda(sample)
File "/content/fairseq/fairseq/utils.py", line 53, in move_to_cuda
return apply(_move_to_cuda, sample)
File "/content/fairseq/fairseq/utils.py", line 42, in apply
for key, value in sample.items()
File "/content/fairseq/fairseq/utils.py", line 42, in
for key, value in sample.items()
File "/content/fairseq/fairseq/utils.py", line 35, in apply
if len(sample) == 0:
TypeError: object of type 'int' has no len()

I have tested with previous fairseq (0.6.2) and it works fine

Fix with fairseq v0.7.1


AudioResource memory leak

Voice.createAudioResource() creates memory leak.
There is no AudioResource.destroy() but without destroy() it eats memory.
With creating resource (eats memory):
Voice.createAudioResource(res.stream, {inputType: res.type, inlineVolume: true});

Without creating resource (all memory is fine):
//Voice.createAudioResource(res.stream, {inputType: res.type, inlineVolume: true});

Also if use this resource:
player.play(resource);
player.stop();

Memory will not free up.

If you are using ytdl-core, then it's your coding problem not djs voice issue. Ytdl core explicitly tells everyone to destroy it's stream once you are done downloading.
See this
Don't think that it is issue of voice module.

I also destroy ytdl's stream below but AudioResource still takes memory.

Please send a full code sample

async #playCycle(guild) {
		let playMusic = async () => {
			try {
				connection.rejoin({channelId: guild.channel});

				var music = this.#channels.getMusic(guild);
				let info = await YTDL.getInfo(music.link, {lang: "ru"});
				music.name = this.#getName(info);

				let bitrate = this.#channels.getChannel(guild).bitrate;

				try {
					var format = YTDL.chooseFormat(info.formats, {quality: "lowestaudio", filter: form => form.bitrate >= bitrate});
					bitrate = format.audioBitrate * 1000;
				} catch {
					var format = YTDL.chooseFormat(info.formats, {quality: "highestaudio", filter: form => form.bitrate < bitrate});
				}

				info.formats = [format];

				var stream = YTDL.downloadFromInfo(info, {highWaterMark: 1 << 30});
				var res = await Voice.demuxProbe(stream);
				let resource = Voice.createAudioResource(res.stream, {inputType: res.type, inlineVolume: true});

				resource.encoder.setBitrate(bitrate);
				resource.volume.volume = guild.volume * this.#settings.volume;

				await Voice.entersState(connection, "ready", this.#settings.waitTime);

				guild.player.play(resource);

				await Voice.entersState(guild.player, "playing", this.#settings.waitTime);

				let ann = this.#channels.getAnnouncment(guild);
				if(ann != null)
					sendChannel(ann, this.#settings.messages.info.playing, music.name);

				this.#channels.changeMusic(guild, 1);

				await Voice.entersState(guild.player, "idle", this.#settings.maxTime);
			} catch(e) {
				if(e.message != "Status code: 403") {
					console.log("#playCycle");
					console.log(music.link);
					console.log(guild.player.state);
					console.trace(e);
				}

				let ann = this.#channels.getAnnouncment(guild);
				if(ann != null)
					sendChannel(ann, this.#settings.messages.info.invalid, music.name);

				this.#channels.changeMusic(guild, 1);
			}

			res?.stream.destroy();
			stream?.destroy();
		};

		let canPlay = () => this.#channels.joinable(guild) && guild.musics.length > 0 && guild.channel != null
			&& this.#channels.getChannel(guild).members.size > (this.#channels.getChannel(guild).members.has(this.#channels.getRealGuild(guild).me.id) ? 1 : 0);

		if(canPlay()) {
			this.#channels.stopAwaiting(guild);

			guild.player = Voice.createAudioPlayer();

			var connection = Voice.joinVoiceChannel({
				channelId: guild.channel,
				guildId: guild.id,
				adapterCreator: this.#channels.createAdapter(guild)
			});

			connection.subscribe(guild.player);

			while(canPlay())
				await playMusic();

			connection.destroy();

			guild.player = null;
		}

		this.#channels.await(guild);
	}

Have luck in reading.

var stream = YTDL.downloadFromInfo(info, {highWaterMark: 1 << 30});
var res = await Voice.demuxProbe(stream);
let resource = Voice.createAudioResource(res.stream, {inputType: res.type, inlineVolume: true});

🤔 is demuxProbe important here? You have inline volume enabled (inlineVolume: true) which would obviously convert your stream to pcm

I also destroy ytdl's stream below but AudioResource still takes memory.
res?.stream.destroy();
stream?.destroy();

저 또한 스트림을 파괴했는데도 메모리 사용량이 돌아오지 않는 상황을 겪어서 해당 이슈에 동의합니다.

state setter of AudioPlayer does not clean-up operations about resource.volume (when inlineVolume: true).
Could this be the cause of memory leaks?
if (oldState.status !== AudioPlayerStatus.Idle && oldState.resource !== newResource) {
	oldState.resource.playStream.on('error', noop);
	oldState.resource.playStream.off('error', oldState.onStreamError);
	oldState.resource.audioPlayer = undefined;
	oldState.resource.playStream.destroy();
	oldState.resource.playStream.read(); // required to ensure buffered data is drained, prevents memory leak
}

just operation about playStream

I don't think this is the issue. The volume stream is included in the pipeline that creates the playStream, so ending the playStream will end the volum stream too.
I still think this is an issue specific to your usage of ytdl-core

Even if I use youtube-dl-exec as the code of the example, but memory usage still increase. Is there a solution to the increase in memory usage?

To detect a memory leak you would need to observe it over 24-72h and check if you see a steady increase in memory, solely from playing music (no new members, no new guilds, etc)

Do you wait for the garbage collector?
To detect a memory leak you would need to observe it over 24-72h and check if you see a steady increase in memory, solely from playing music (no new members, no new guilds, etc)

I tried to wait for GC, but bot very slows down when memory is running out.


[Radio] Add regressions tests

[Radio] Add regressions tests
[RadioGroup] Add regressions tests

Same as #5639 the failed test seems related to circleci

Thanks!


Analyzer: Support mixing in classes that have constructors containing optional arguments

As of https://github.com/dart-lang/sdk/commit/63c6851dd01514e9d9e6cb96da0430e15b66d576, this is legal.
class A {
  A([int x = 4]);
}

class B extends Object with A {} // valid

Revealed by mixin_forwarding_constructor4_test
It's also allowed by the front-end, and a nuisance for many folks, see #15101
I'm putting this in 2.1 because it should be so so easy to do, and people bump up against it. Seems like low hanging fruit.

Right, as of 63c6851dd01514e9d9e6cb96da0430e15b66d576 there is support for implicitly inducing forwarding constructors of additional forms into a class created by mixin application: The induced constructors can now have optional arguments, and they can be constant:
class A {
  A([int i]);
  const A.cc();
}

class M {}

class B extends A with M {
  B(int i): super(i);
  B.d(): super();
  const B.cc(): super.cc();
}

The constructors in B used to be an error because the class A with M did not have any constructors (because no constructors were induced when the candidate constructor in the superclass had optional arguments, and the induced ones weren't constant even when they could have been constant). (... OK, maybe the constructor named B used to work as well, but its argument wasn't optional, I'm not quite sure how it used to be, but that shouldn't matter so much here ...)
But that spec update doesn't change any rules about explicitly declared constructors, so they will still make it an error to extract a mixin from any given class. In particular, it is an error to extract a mixin from the class A here, because A contains a constructor declaration (at all, const/not, optionals/not).

Thanks @leafpetersen  and @eernstg, yep, I confused the base class constructor for the mixin constructor. Fixed.


2 answered discussion which are marked as answer

2 answered discussion which are marked as answer
Originally posted by @tanish36 in https://github.com/Schweinepriester/github-profile-achievements/discussions/9#discussioncomment-3659115

yes


Output proper information for the web versions

While exploring home-assistant/core/issues/101713, I got extra confused by the wrong output.

#Hacktoberfest


Importing  intermediate case counts for _EXALLC and _EXMORE endpoints

Copy the results from non-_EXALLC and non-_EXMORE counter-parts.

Making script
Run to get data file
Import data file

_EXALLC and _EXMORE endpoints don't have intermediate counts because the counts are generated by the Endpointter program but _EXALLC and _EXMORE endpoints are not generated by the program. Need to copy the counts from the counterpart endpoints.

Not sure if it's a problem anymore.
Putting in Backlog, maybe the problem will resurface for the next release.


Minimize VTL files when they get above 64KB

We get Invalid request template given, must be less than 64kb in size. on some VTL files.
Looking at the files it appears they contain comments and are fairly verbose, this is great for people who are learning VTL or are going to manually update them, but for large generated files it would be good if we minimized the file size somehow.
One suggestion would be to remove comments if the file hits 64kb, then if the file is still above 64kb run some sort of minimization script on the file...such that $ownerAuthExpressions becomes $o1, additional whitespace is removed.
Another solution would be to add a feature flag "minimizeVTL" that does the same.

https://github.com/aws-amplify/amplify-cli/issues/2698


Update wallet connection GA event

What it solves
Resolves #2969
How this PR fixes it
The wallet connection event is now standardised/located alongside all other project-wide GA events.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.01%) to 31.529%

Files with Coverage Reduction
New Missed Lines
%

src/logic/wallets/store/actions/fetchProvider.ts
1
10.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.01%

Covered Lines:
2975

Relevant Lines:
8408

💛  - Coveralls

@iamacook @katspaugh I can only test this one in the unified app in stg, since low test env are not tracked in GA. Can you just merge this one to stg so I can check it there?


Dev Spaces must be initialized first

"Connect your project..." did not work until I first initialized (which installs the cli extension ) with "az aks use-dev-spaces "

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

ID: a28fe6ff-a564-a867-2f1d-91197e74f22b
Version Independent ID: 2119d269-d3e2-3e8d-4753-e9dfe8a313a0
Content: Debug and iterate with Visual Studio and .NET Core on AKS with Azure Dev Spaces - Azure Dev Spaces
Content Source: articles/dev-spaces/quickstart-netcore-visualstudio.md
Service: azure-dev-spaces
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@philspokas Thank you for your feedback. I will review it and get back to you shortly.

I am going to assign this to the doc author to update accordingly.


Update Italian README.md

Fixed few badly translated Italian sentences and verbs

Grazie mille, @LeoSpyke!


Cannot install Better Bibtex on Zotero Standalone

Hi!
I am using Zotero Standalone version <IP_ADDRESS>, and it shows me an error when I am trying to install Zotero Better Bibtex. It got installed on my firefox plugin though. It is very strange because for a while i could see Better Bibtex options for exporting my references, but I do not anymore.
Could you please help?
Thank you!

What error do you see?

After I installed a firefox plugin, I can see better bibtex options, but when I try to export using them it gives this error:
(1)(+0000007): TypeError: this.titleCaseLowerCase is undefined
===== Stack Trace =====
Translator.initialize/this.titleCaseLowerCase<@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1272:5
Translator.initialize@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1270:49
Translator.initialize@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:18134:5
Translator.nextItem@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1425:3
doExport@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:17683:17
=======================
(2)(+0000000): Translate: Translation using Better BibLaTeX failed:
string => TypeError: this.titleCaseLowerCase is undefined
stack => Translator.initialize/this.titleCaseLowerCase<@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1272:5
Translator.initialize@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1270:49
Translator.initialize@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:18134:5
Translator.nextItem@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:1425:3
doExport@/Users/evgeniiaiakhnis/Library/Application Support/Zotero/Profiles/nyo756n4.default/zotero/translators/Better BibLaTeX.js:17683:17
url => /Users/evgeniiaiakhnis/Desktop/USC_Files/Exported Items.bib
downloadAssociatedFiles => true
automaticSnapshots => true
(5)(+0000000): Translate: Running handler 0 for done

So you can install it correctly, but you cannot export?

Well, as far as i understand, I need to install it separately on my firefox plugin and standalone? It shows as one of the extensions on my firefox, but not on the standalone. And when I try to do "Install add-on from file", it tells me that the version is not compatible. So I am very confused.

OK, but can we keep the two issues apart? It's a little confusing to be dealing with them intertwined. There is apparently a separate issue when you try to export from firefox, please file a new issue for that, and let's try to figure out why it's not installing on standalone.
So you say you get a "not compatible" message, and you're installing the same XPI on both firefox and standalone? I've just installed the latest BBT on standalone <IP_ADDRESS>, and I can just install it. How have you downloaded the XPI? No through a download manager?

I just clicked on the installation file here https://github.com/retorquere/zotero-better-bibtex/releases/tag/1.6.65

And you used this same file for both standalone and firefox?

yes

can you attach the xpi you downloaded to this issue?

it says that that file type is not supported

Just rename it to .zip, I think it will allow that

No, wait, I was wrong. I actually don't have it on my computer. I think it installed automatically when I pressed the installer here https://github.com/retorquere/zotero-better-bibtex/releases/tag/1.6.65

sorry, for the confusion, I guess I was trying to open a different file

does it mean that it actually got installed on my standalone?

I don't know, but it seems unlikely. In standalone, you can go into tools - addons to see which plugins are installed. This is also where you would install the plugin -- if you haven't been in that part of the UI, I don't think you have installed BBT.

could you attach the installation file here for me to try?

Just download https://github.com/retorquere/zotero-better-bibtex/releases/download/1.6.65/zotero-better-bibtex-1.6.65.xpi and select that from Tools > Add-ons menu (gear menu, "Install add-on from file...")

It wouldn't be different from getting it from https://github.com/retorquere/zotero-better-bibtex/releases/tag/1.6.65. Just right-click on the xpi link and choose "save", and then install it in standalone.

cool, it installed correctly now and it all works! Thank you very much for your help!

OK, if there's still a separate issue with exports, please file a new issue


Bug: Annotation overrides for StringSlice arguments are not treated as space delimited  like cmdline and env var.

Problem
Annotation overrides for StringSlice arguments assume a json encoding instead of using the space delimited behavior used in the arguments and environment variables.
Let's make arguments parse in a more consistent manner.
Straw proposal

Arguments of cobra StringSlice  type use CSV styled space delimited in al contexts.
Arguments of cobra StringArray type are parsed in ENV and annotation as a json array of strings, if not parsable as json assume its a single string.

I'll have to look at this one in light of the recent StringArray merge, I don't think this was captured.


Remove Useless / Unused Code

You have instance of

Mandelbrot (Andrew Duncan)
Game of Life

This is because it's a direct copy of the library, it's being updated to a symlink.

You maybe check your license. You have many library from other developer.

Yes, as stated it’s going to be updated to a symlink and the library is
also covered by an MIT license that states its free to use for our use
case. Your input is valued but please try and validate your issues before
adding them. The license is a non issue and therefore did not need
mentioning.
On Sun, 4 Nov 2018 at 01:05, Jin Lucky<EMAIL_ADDRESS>wrote:

You maybe check your license. You have many library from other developer.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Helder1981/OneForAll/issues/1#issuecomment-435633372,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABxLHVM7Tq8IhS9TWTXhsrTraGzqSl1Qks5urj1IgaJpZM4YLV8w
.


Swift problem

Hello,
I use the following code
var errorPointer:NSError?
var number:NBPhoneNumber = phoneUtil.parse(sender.text, defaultRegion: region, error: &errorPointer)
and I get the following error :
fatal error: unexpectedly found nil while unwrapping an Optional value
Do you know where it comes from ?
Thx

I am getting the same error! Not sure what the problem is:
func isPhone(text: String) -> Bool {
    var errorPointer: NSError?
    // Error comes in the line below.
    var number:NBPhoneNumber = phoneUtil.parse(text, defaultRegion: "KR", error: &errorPointer)

    if errorPointer == nil {
        return phoneUtil.isValidNumber(number)
    }
    return false
}

actually the problem is with the example. In your case it works so there is no error and the function doesn't return a nil.
However, when there is an error, the function return a nil and you can't put a nil in a non-optional value such as NBPhoneNumber.
Therefore the correct way to do the example is:
var phoneUtil = NBPhoneNumberUtil.sharedInstance()
var errorPointer:NSError?
var number:NBPhoneNumber? = phoneUtil.parse("01041241282", defaultRegion:"KR", error:&errorPointer)

Thanks~


Work with Consumable products

Hey there.
i might not get the point correctly, but in my case i want to offer Consumable products within my App. So when the user has purchased a consumable product there should be a possibility to mark the purchasedProduct as consumed?! How would that look like?

@BigPun86 I haven't worked with the consumable product flow. I think it should be possible to support this very easily but I'm afraid you will have to do some digging and possible send a PR if something is missing. Let me know what you find.


fix add support for quoteChar and seperatorChar for hive

Proposed changes
add support for quoteChar and seperatorChar .

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


FM Receiver Resampling Issue

I was trying to extend the FM receiver example. I added a function to the audio sink that allows querying the default sample rate of the device https://github.com/bastibl/FutureSDR/commit/8da5ceabd1c835dd6b7f9b1277ee1636d4060ca3. For me, this returns 44.1kHz. Like in the current example, I wanted to resample in two steps with an intermediate rate of 5 * audio_rate. Using this in combination with my HackRF at a sample rate of 8MHz resulted in a rather long filter (~10k taps). With the default buffer size, this stalls the flowgraph, since only 4k samples fit in the buffer. Increasing the buffer size with a config.toml works. However, the audio is broken.
My WIP state is here https://github.com/bastibl/FutureSDR (the master branch). @AndersKaloer do you have any idea? I'm not sure if this is a problem with the resampler or my code.
Here's the diff with debug output: https://github.com/bastibl/FutureSDR/commit/6fcb45c931c85825fd25894361df05ce50c8ca7f

Might be because of the DC offset, will try offset tuning.

Tuning the radio quite a bit away (>0.5MHz) results in some audio. I'm still not sure if this is a bad receiver design of if something is wrong with the resampler. Would be great if you could have a look.

I will try to have a closer look at this tonight.
It sounds a bit like the GCD normalizer in FirBuilder::new_resampling does not work as intended. The filter should produce 2*12*interp taps, and in your case I think the GCD is 80000, so effectively you only need to interpolate by 3 and decimate by 100, which should produce 72 taps. Maybe you could try specifying 3 and 100 directly?
I also don't think you want a complex filter here, since that will remove the negative frequencies, which contain half of the signal power in FM.
In any case, downsampling by 100 is still quite much for a filter of that length, and it might be better to use a chain of resamplers, e.g., a filter that interpolates by 3 and decimate by 10 followed by another filter that interpolates by 1 and decimates by 10.
Also, I had to compile with --release in order to make it perform well (but I guess you are aware of that).

I think the GCD works. It's gcd(44100*5, 8000000) = 500, so it's 2*12*441 = 10584. My feeling is that there is an issue with larger decim/interp rates, but I didn't have a closer look. It's not urgent, I don't want to annoy you :-)

I tried your code, and it worked fine for me both with BladeRF and HackRF, and with 8 MHz bandwidth (the only problem I had was when my bandwidth was smaller than 2*OFFSET).
I run it using (PKG_CONFIG_ALLOW_CROSS and the target flag are just to compile for aarch64-apple-darwin)
PKG_CONFIG_ALLOW_CROSS=1 cargo run --target aarch64-apple-darwin -- --frequency 89700000 --rate 8000000

I also tried to dump the samples to a file both before and after the resamp1.
The input to resamp1 looks like this:

The output looks like this:

Note that my PC actually reports 48000 as the audio rate, which means that I am effectively interpolating by 3 and decimating by 100, so I do not have so many taps in my filter. Actually, the current filter implementation works as intended, but obviously it produces a large number of taps when the interpolation factor is high. Honestly, it is quite inefficient to have such high interpolation/decimation factors in a single filter, so a much better solution would be to adjust the sampling frequency a bit to have a for higher GCD.
What you can do is to design your own filter with a smaller number of taps. Something like this:
let gcd = num_integer::gcd(interp, decim);
let interp = interp / gcd;
let decim = decim / gcd;
let resamp1_taps = firdes::kaiser::multirate::<f32>(interp, decim, 2, 0.0001);
println!("{} taps", resamp1_taps.len());
let resamp1 = FirBuilder::new_resampling_with_taps::<Complex32, Complex32, _>(
    interp,
    decim,
    resamp1_taps,
);

BTW, be careful with https://github.com/bastibl/FutureSDR/commit/6fcb45c931c85825fd25894361df05ce50c8ca7f#diff-d0c711267097b2b1b136ad18d24dbf53f875886e8f713a08be4b816e8ed7fec0R176-R178 since the filter taps are usually designed for specific values of interp and decim.

Thanks for looking into it and for your feedback. I deleted the bogus gcd-minimization and tried to make the receiver smarter selecting the parameters. (Setting the offset to sample_rate / 4, choosing audio_rates based on gcd, automatically selecting the audio multiplier.) I think it's pretty OK now. At some point, one could add dedicated blocks for frequency shifting and demodulation, which should provide better performance compared to the Apply-style blocks.
Thank you very much!


Use only spaces for indentation in addon example

Right now, both tabs and spaces are mixed (in some weird way) in this file, making it look strangely in some editors.

tabs?? evil! thanks.


Fixed argument types in setup.py

This makes the library compatible with newer versions of pip

LGTM and thanks for the change.
Will merge after tests pass.


[BUG]: poetry-lock precommit hook not working

Operating system
Windows 11
Bug description
I tried committing some changes to removing unused frontend tsx code. I didn't change the pyproject.toml and when I commit the change and the poetry-lock pre-commit hook runs, i get a failed build check even though i didn't touch any poetry dependency.
Fix: update our poetry github action build check to make sure the poetry.lock file is up to date with pyproject.toml or identify a better workaround that isn't too slow
Steps to reproduce

make some minor code change
run git add . followed by git commit -m <message>
see the pre-commit hooks running

@karkir0003 already tried this, but I was having same issue. I solved by updating dependencies and regenerating  your poetry lock file using:
poetry update
this is what worked for me in the last commit. Make sure you run poetry update in the correct subdirectories.
if that doesn't work, make sure to update and clean ur pre-commit cache.
pre-commit clean
pre-commit autoupdate

This is personally what worked for me, and I'm not sure why it's not working for @karkir0003. Some more info here:
https://github.com/python-poetry/poetry/issues/8189

hmm. thats weird. im having austin tsst this out @codingwithsurya

@Dashrekker @codingwithsurya is this solved?

@Dashrekker @codingwithsurya is this solved?

this works on my end.
i think this is more of a windows issue so you will have to test it on your end. @karkir0003