Negative Derivative Term in Rate PID Controller!

Describe the bug
In the PID implementation of the multicopter attitude rate controller (https://github.com/PX4/Firmware/blob/master/src/modules/mc_att_control/RateControl/RateControl.cpp#L83), the derivative term is negative  - _gain_d.emult(rate_d),
Vector3f torque = _gain_p.emult(rate_error) + _rate_int - _gain_d.emult(rate_d) + _gain_ff.emult(rate_sp);

I don't understand why it's not a positive term just like the P and I terms!
Is this a bug, or there is an explanation for it?
Thanks.

Hi @mzahana. Think of the proportional and integral terms like a gas pedal to speed up an error resolution and the derivative term like a brake pedal to keep the error from being resolved so fast that it results in overshoot. Much like pressing the gas pedal results in the opposite acceleration sign as the brake pedal, you need the derivative controller effort to act opposite from the P and I terms. The rate of change of the error is what the D term acts on, and this is so that when large errors are present and the P and I terms would otherwise lead to overshooting the desired value, the D term keeps the rate of change of the error value reasonable. This also allows higher P and I values than would otherwise be possible. Hopefully this makes sense!

Thanks @mcsauder for the explanation. I am still thinking from a mathematical point of view. There are different implementations of discrete PID. The simplest, approximates the integral and derivative terms by first order approximations. Then, the terms are added together (after multiplying by corresponding gains). I still don't understand the negative sign in the derivative term.

Hi @mzahana , Have a look at how the rates derivative is being calculated:
if (dt > FLT_EPSILON) {
		rate_d = (rate_filtered - _rate_prev_filtered) / dt;
	}

If the computation were instead the previous - current, then, as you are thinking, the sign of the derivative contribution would need be positive.  It can formulated either way, it's mostly a matter of how the rate of change is calculated.  It is also fairly straightforward to prove out in a real system or simulation that the math is correct by forcing a negative D gain to be saved and observing the resulting unstable control behavior, but following the math should also lead you to the same conclusion.  Hopefully this helps.!

@mzahana Normally you are used to seeing a positive sign as the D gain is multiplied by a pre-computed rate error. The reason why we don't use the error but just the rate is because it avoids introducing a zero into the system. I'm speaking about a zero in terms of control systems (https://en.wikibooks.org/wiki/Control_Systems/Poles_and_Zeros).
If you write the equation down as follows:
float rate_error = rate_sp - rate
Now zero rate_sp in order to avoid a zero introduced.
float rate_error = 0 - rate = -rate
This is where the negative sign comes from.

Thanks @mcsauder @RomanBapst. Now it makes sense!
I am asking this because I am doing system identification and LQR-based PID tuning. So, I needed to know what PID implementation is done in the attitude/rate loops.
So just to make sure that I am designing gains for the right controller, is there anything else that is different from a traditional PID implementation?
Cheers.


dns.Msg with only header

A dig @<IP_ADDRESS> +norec . NS returns a packet with only the header and no question section.
Unpack() returns an Truncated error on this packet, which may or may not be in line with the DNS standard.
Should such a packet be considered valid?

Yes it should be considered valid. It may or may not be a sensible response in context, but in isolation it is a valid message.

closed via #608


Repair kit rework

Taken from the Forgero Discord:

I also think repair kits need an overhaul, potentially even removed for lower tier items. Either that or on the go repairing needs to be removed
I like the idea of having them for on the go repairing while caving or fighting mobs, but then you can just repair wooden/stone tools with the material it's made of (which I don't like much)

How about this:
we remove the repair kit repairing recipes from the crafting table. We make it possible to load a repair kit with x amount of the same material, depending on the stage/tier (we could also do something with how much it actually repairs) of the repair kit.


Support Gmail API Notification Source

Proposed Functionality
Add Gmail API as a supported Notification Source, supporting OAuth authentication.
See: https://www.geeksforgeeks.org/how-to-read-emails-from-gmail-using-gmail-api-in-python/
Because of the OAuth nature, the "Application", the Plugin, will require:

the first time we set it up a manual login verification, later we could keep refreshing the token. But, we should offer a way to the user to "activate" this source. Maybe, a "TEST/VALIDATE" option could serve this goal and also provide an easy way to validate the credentials for other methods, such as IMAP.
the way that we identify to the API is more complex than just an account id (i.e. email) and password, so it contains a "client_id", and "project_id", and a "client_secret". This means that the model should be modified to support these more complex cases.
it should work across service restarts

Use Case
Offer an alternative to IMAP Notification Source

Fixed in #26


SandcastleCore

version of the sandcastle without Parts, just dll.
Suppose to be used without EL, just printer plugin for ScienceLab
is it ok, @Angel-125 ?

added ExtraPlanetaryLaunchpads to the conflict of the SandcastleCore, for clarity:
with the EL user need Sandcastle, but not the SandcastleCore

Rather than leaving this open indefinitely, please let us know if you hear from @Angel-125 and we can re-open this. In the meantime, perhaps you can depend on Sandcastle, which is already indexed.


Fix bug: Handle data entry with invalid service end date

Rollbar: https://rollbar.com/somerville-teacher-tool/somerville-teacher-tool/items/69/
Code:
       if service.save
>>>      if estimated_end_date.present? && estimated_end_date.to_time < Time.now
           service.update_attributes(:discontinued_at => estimated_end_date.to_time, :discontinued_by_educator_id => current_educator.id)
         end
         render json: serializer.serialize_service
       else

Thoughts: debug and then move this into the serializer object above.

Tried to reproduce this locally. Nothing seems broken about service recording, was able to record a few fake services locally with the datepicker UI.
Then I tried entering an invalid date, and was able to reproduce the error:
UI (inputs invalid service)

Server logs

What was unexpected for me was that even though the UI shows "error" at the end and doesn't update the list of services, when the user refreshes the service appears in the list, just without any expected end date:
UI (result)

So this is my working theory for what cause this bug right now — a user (educator) entering invalid service end date.

Couple ideas here...

We should validate data entered into the service datetime inputs more strictly.
We should push some of the code in students controller that's triggering this error down to the service model. It's ultimately the service model's job to determine is a service is valid or not, not the students controller.

Client-side validation asymmetry (GIF)
Looks like we have client-side validation that would catch this for date_started, but no such validation for estimated_end_date.


Svelte(Kit) Support?

First of all, kudos on what looks to be a great library 🎉
Noticed the Vite docs that show how to add the Babel plugin for React, any chance this currently supports Svelte / SvelteKit?

Hi. I tried to make Sveltekit samples, but it's not easy. 😢

I think the main thing blocking sveltekit and solid-start is ESM support. Typewind currently only ships commonjs bundles, but both of these frameworks require the libraries to support esm

This would enable support for Qwik as well


SP4 WPF: Strong randomized data slows windows resize down

Bug Report
Issue:  Strong randomized data slows windows resize down.
I need a scatter or signalXY plot with a lot of data points (> 3,000,000). Drawing the chart is ok, but if the windows is resized (e.g. maximized), it takes very long, depending of the displayed data.
Reproducing:

Create a new WPF App
Add WpfPlot Control to XAML
Add code to Loaded event
Resize the window by clicking the maximize button several times
Increase the randFactor and resize again

The computer needs ~ 5s for a resize with a randFactor of 10, but ~ 20s with a randFactor of 1000.
Update: I added Benchmark(true) and got this measures for a randFactor = 1000:

initial render: 1,99 s
maximize: 30,1 s
back to normal size: 9,24 s

<WpfPlot HorizontalAlignment="Stretch" VerticalAlignment="Stretch"
         x:Name="ResultChart" Loaded="ResultChart_Loaded"/>

private void ResultChart_Loaded(object sender, RoutedEventArgs e)
{
    // create some random data
    int numberOfWells = 1536;
    int numberOfDatapoints = 3000;
    double randFactor = 10.0;  // increase this number for example to 1000.0
    for (int well = 0; well < numberOfWells; well++)
    {
        double[] xs = new double[numberOfDatapoints];
        double[] ys = new double[numberOfDatapoints];
        for (int index = 0; index < numberOfDatapoints; index++)
        {
            xs[index] = index;
            ys[index] = Random.Shared.NextDouble() * randFactor + index + index;
        }
        ResultChart.Plot.AddScatterLines(xs, ys, lineStyle: ScottPlot.LineStyle.Solid);
    }

    const bool lowQuality = true;
    ResultChart.Refresh(lowQuality);
}

System Details

ScottPlot Version: 4.1.64?
Operating System: Windows 10 21H2
Application Type: WPF
.NET Version: .NET 7

Hi @mikeKuester, thanks for reporting this!
Unfortunately the poor performance is a limitation of System.Drawing which ScottPlot 4 uses to draw lines. As the data becomes more random the lines become longer, cover more pixels, and take longer to render.
ScottPlot 5 offers great performance improvements by dropping System.Drawing in favor of SkiaSharp which uses OpenGL for much more performant charting. It's still in preview, but public so you can give it a try. I'm ramping-up development on it now and hope to have it reach feature parity with ScottPlot 4 soon.
Hope it helps!
Scott

Hi @swharden ,
thanks for your fast response (and for ScottPlot of course).
I tried ScottPlot 5.0.5-beta and it's much faster:

initial render: 0,5 s
maximize: ~2 s
back to normal size: ~2 s

This looks very promising for the future :D
It takes me a bit of time to find the right methods, so if anyone look into this issue:
<Window x:Class="ScottPlot5.MainWindow"
    ...
    xmlns:scottPlot="clr-namespace:ScottPlot.WPF;assembly=ScottPlot.WPF"
    ... >
    <Grid>
        <scottPlot:WpfPlot HorizontalAlignment="Stretch" VerticalAlignment="Stretch"
            x:Name="ResultChart" Loaded="ResultChart_Loaded" />
    </Grid>
</Window>

        private void ResultChart_Loaded(object sender, RoutedEventArgs e)
        {
            ResultChart.Plot.Benchmark.IsVisible = true;
            ResultChart.Plot.AutoScale();

            // create some random data
            int numberOfWells = 1536;
            int numberOfDatapoints = 3000;
            double randFactor = 1000.0;  // increase this number for example to 1000.0
            for (int well = 0; well < numberOfWells; well++)
            {
                double[] xs = new double[numberOfDatapoints];
                double[] ys = new double[numberOfDatapoints];
                for (int index = 0; index < numberOfDatapoints; index++)
                {
                    xs[index] = index;
                    ys[index] = Random.Shared.NextDouble() * randFactor + index + index;
                }
                var scatter = ResultChart.Plot.Add.Scatter(xs, ys);
                scatter.MarkerStyle.IsVisible = false;
            }

            ResultChart.Refresh();
        }


css package dependency causing issues with yarn

Related:
https://github.com/yarnpkg/yarn/issues/1619#issuecomment-352188088
package.json:
https://github.com/gulp-sourcemaps/gulp-sourcemaps/blob/master/package.json#L30

We are open to adopting a new module that generates CSS sourcemaps but we're not actively working to replace that module to make one unsanctioned package manager work.
Please send a PR if you want to fix this.


Readiness probes for endpoints

Is your feature request related to a problem? Please describe.
Devfile component's is able to describe an endpoint that is supposed to expose some service.
A user may be interested in the endpoints state ( if services behind them) are available or not to:

do not open plugin's endpoint (Theia) when it's not available;
do not open container endpoints, after command which lauch server is run but before server is fully started;

Describe the solution you'd like
The plugin is able to declare a readiness probe for its endpoints. So, the implementation (odo, devworkspace operator) is able to evaluate endpoint info and expose it if needed.
components:
  - name: "mycontainer"
    container:
      image: "busybox"
      memoryLimit: "128M"
      mountSources: true
      endpoints:
        - name: term-websockets
          exposure: public
          protocol: ws
          attributes:
            type: terminal
          targetPort: 4000
          readinessProbe:
            path: /readiness
            periodSeconds: 3 # ?
            failureThreshold: 3 # ?
            # ?something else?

If it's the main devworkspace endpoint - workspace is not going to be marked as Running until that endpoint readiness probe passed.
It even may make sense to introduce liveness probe as well, if endpoint lifecycle bound to container lifecycle, user may want to have container automatically restarted when liveness is failed.
Describe alternatives you've considered
Probably one probe per container can be enough, if so - we can reuse K8s interface here:
  containers:
  - name: cache-server
    image: cache-server/latest
    readinessProbe:
      httpGet:
        path: /readiness
        port: 8888
      initialDelaySeconds: 300
      periodSeconds: 30

Additional context
N/A

The readinessProbe can now be set with container-overrides:
attributes:
    container-overrides:
        readinessProbe:
            httpGet:
                path: /readiness
                port: 8888
            initialDelaySeconds: 300
            periodSeconds: 30

With this being the case, I feel that this issue can be closed now.


Terminal does not re-focus after Accessibility Help

Testing #196124

Open a terminal.
alt-f1 -> gets terminal accessibility help
alt-f1 again -> gets another accessibility help menu.
escape -> closes terminal
Focus isn't on terminal. :bug: I thought that this might be confusing if a user wants to run alt-f2 right after this.

Yeah this flow is unlikely to occur. We should disable alt+f1 when in a help dialog


Use headless chrome to print to pdf from command line

Idea discarded as font rendering in Chrome/Linux is not optimal right now

Add command line scripts to print to pdf using firefox and chrome

(Project archived, development moved to https://github.com/jsidrach/resume-generator)


Update doc to reflect Azure Cognitive Search Extension is no longer maintained

[Enter feedback here]
Update doc to reflect Azure Cognitive Search Extension for VSCode is no longer maintained.
https://github.com/microsoft/vscode-azurecognitivesearch#azure-cognitive-search-for-visual-studio-code-preview
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 22cde966-aff6-4bc4-1cfa-a03fd17de5ce
Version Independent ID: f0cdf4c2-b641-bdb8-1919-1bfef6b5801a
Content: JavaScript tutorial: Add search to web apps - Azure Cognitive Search
Content Source: articles/search/tutorial-javascript-create-load-index.md
Service: cognitive-search
GitHub Login: @diberry
Microsoft Alias: diberry

Hi @JGroot, as far as I can tell Azure Cognitive Search extension is available. I see that it hasn't been updated in a while and I'll check with the team to see what future plans there are.

@HeidiSteen How would you like me to fix this? Do you have an example I can use across the 3 language docs in this series?
#in-progress

@diberry and @RyanHill-MSFT, I changed the workflow to use Azure portal.  The PR is now merged and the docs will be updated on the next publish.  As soon as I see the changes are live, I'll respond to the customer and close this issue. Thanks for bringing this to my attention.

@JGroot the docs are now updated. Thanks for raising the issue!
#please-close

@HeidiSteen Thank you for updating!!


DisplayTitle: update to newer 1.39 version

Specifically include [1] so that section links are not broken.
MN-11
[1] https://gerrit.wikimedia.org/g/mediawiki/extensions/DisplayTitle/+/f5de9daf9db92d4fa74a6991269525266f2a857b

Not sure why the bot isn't reporting it here but the updated image is 1.39.2-20230806-29d81af


Blocking whole feed

@lucboruta Luc, thank you for your plugin. I'm not sure if I misunderstood the description but isn't it supposed to block only promoted posts ? Unfortunately in my case it's blocking all feeds (personal and companies' feeds).
I'm using Brave browser (Version 1.22.70 Chromium: 89.0.4389.105 (Official Build) (64-bit))

Hi, thank you for your interest in the extension. It was indeed designed to block all posts in the feed, as illustrated by the screenshot on the Chrome web store.
The extension is very simple by design, and I have no plans to add code for some types of posts to be blocked and not others. That being said, I'd be happy to review a pull request if you want to give it a shot.
I'll leave this issue open to rework the description next time I submit an update to the Chrome web store.

If you want to filter the feed rather than block it, check out this extension.

This extension is no longer maintained. Please consider using @njelich's LinkOff instead.


bin/setup does not create REGION

There is no reason to setup a file with a value of 0 when if the file is missing, the code will default it to 0 anyway.
Extracted from #6295
/cc @jrafanie think you have the strongest opinion here. I can provide more information if you want, but to be honest, just merge or close. No real reason to discuss this any further.
/cc @carbonin

:+1:


ClassicPress support

Hello,
It's possible to show ClassicPress like WordPress sites usage? several users are migrating because Gutenberg and the stats doesn't show how many users are using themes and plugins with ClassicPress.
Regards.

This project is not affiliated with ClassicPress. Did you perhaps mean to post this issue in a different repository?

Our users are using WordPress and ClassicPress at the same time, for example, the stats show 68.6% of wordpress 5.1 instalations and 30.2% of "Other" but cant determinate if is "Other" means other WP versions or ClassicPress instalations.
I added the "cms" to the $UpdateChecker:
$url = ...
$url .= '&cms=' . self::detect_cms();
$UpdateChecker = Puc_v4_Factory::buildUpdateChecker($url,$target,$slug);

private static function detect_cms() {
    return (function_exists('classicpress_version_short')) ? 'ClassicPress': 'WordPress';
}

And the logs look like this:
[2019-04-05 09:56:16 -0500] <IP_ADDRESS> GET get_metadata	{SLUG}	1.0.1 5.1.1 {WEB_URL} action=get_metadata&slug={SLUG}&token={TOKEN}&cms=WordPress&installed_version=1.0.1&php=7.0.33-1+ubuntu16.04.1+deb.sury.org+1&locale=en_US
Until here everything works fine, but the wp-update-server-stats does not show other non-wordpress cms, I understand that it was the expected behaviour, but now ClassicPress entered to the market.
ClassicPress is a fork of WP 4.9, and some users are migrating.
It's possible to expect an update for wp-update-server-status which shows WordPress and ClassicPress data?

I see, thank you for the explanation.
That sounds like a reasonable idea, but unfortunately I don't have the time to add that feature right now. If you'd like to give it a shot yourself, feel free to send a PR.


sarif-quick-fixes

@Spoor2709 LGTM. Please bump the version and publish to npm

@Spoor2709 LGTM. Please bump the version and publish to npm

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Defect: Update Sidebar styling and format

Topic
Currently the sidebar has some styling issues that need to be fixed, such as gray space on the side and too much padding on the right (not enough space for content).
Service
Frontend
Steps

[ ] Fix styling and formatting

Fixed the edge (grey) padding in 5740bf5f5931e9fa002a7e4ba0eda955c7ae9b8f
Not sure what you mean about too much "padding on the right".
Adding all the content, filters, dropdowns, etc. won't be part of an issue related to format and styling.  That will build as we built out associated features with charts and such.  Should probably remove that task on this issue.
Issues should be about a small and defined piece of work (feature, debt, risk, defect).  Styling would be either feature or defect.  Adding the sidebar content would usually be a feature.  I don't think debt is the right classification here.

Aw bummer, i had already started on this. The padding on the right refers to the right of the sidebar. I think the area for the actual chart content is too small, too much whitespace in there. Just wanted to play around with it and see what it looked like with a bit more real estate.
I'll update the issue to reflect all those changes. I still think we need to adress thee content of the sidebar though, because currently it doesn't make a ton of sense. We don't need to provide the options for the dropdown, but shouldn't we add a placeholder in there? Right now the sidebar has like, "Report", "Team", etc.

I created a pull request where i just updated the padding for the central content. If you don't think it looks better feel free to discard and close everything out.

I think I get your point, but that dotted line wasn't the actual content... that was just there from Tailwind.  That wasn't really the zone for our charts.
Also the placeholders in sidebar won't actually be links like that.  Those are also leftover from the component in tailwind.  I left them just to remind us of the styling that looks good.  There won't be traditional links there (probably).  I'm find with the changing of the dotted line box but that'll disappear once we have real content.


[Pagination dropdown] Dropdown is missing on the "Мої гуртки" page in the SP cabinet

Environment: macOS Big Sur 11.5.1; Chrome version 99.0.4844.51.
Reproducible: always.
Build found: gcp-0.0.94
Preconditions

Log in as SP (e.g. Email -<EMAIL_ADDRESS>Password - Qwerty1!) There are more than 12 workshops in the cabinet

Steps to reproduce

Go to "Мої гуртки"
Scroll down to the bottom

Actual result
There is no pagination dropdown
Expected result
Pagination dropdown should present on the "Мої гуртки" page
User story and test case links
User story - https://github.com/ita-social-projects/OoS-Frontend/issues/1115
Test case
Labels to be added
"Bug", Priority ("pri: medium"), Severity ("severity:minor")

@DmyMi we need pagination for provider workshops


MXNet-to-CoreML module: Fixed KeyError for Reshape

Description
While converting your saved params and symbol.json file to Apple CoreML mlmodel file, for the Reshape layer, there used to come KeyError. This has been fixed with this commit.
 I finished coding on this PR)
[x] All changes have test coverage
[x] For user-facing API changes, API doc string has been updated.
[x] To my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Feature1, tests, (and when applicable, API doc)
[x] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Intersting edge cases to note here

It should be attrs now

@tqchen right?

Yes

@tqchen , @piiswrong , I am making using of MXNet 0.12.0 version. When I am saving the symbol file in JSON format, I get the output in this format for various types of layers:
      {
      "op": "Convolution",
      "name": "convolution5",
      "attr": {
        "kernel": "(3, 3)",
        "num_filter": "16",
        "stride": "(1, 1)"
      },
      "inputs": [[6, 0, 0], [7, 0, 0], [8, 0, 0]]
    },
    {
      "op": "Activation",
      "name": "activation7",
      "attr": {"act_type": "relu"},
      "inputs": [[9, 0, 0]]
    },
     {
      "op": "Activation",
      "name": "activation8",
      "attr": {"act_type": "relu"},
      "inputs": [[15, 0, 0]]
    },
    {
      "op": "FullyConnected",
      "name": "fullyconnected5",
      "attr": {"num_hidden": "2"},
      "inputs": [[16, 0, 0], [17, 0, 0], [18, 0, 0]]
    },
    {
      "op": "Reshape",
      "name": "reshape1",
      "attr": {"shape": "(0, 2)"},
      "inputs": [[19, 0, 0]]
    },
    {
      "op": "transpose",
      "name": "transpose0",
      "attr": {"axes": "(1, 0)"},
      "inputs": [[20, 0, 0]]
    },
    {
      "op": "SoftmaxOutput",
      "name": "softmax",
      "inputs": [[22, 0, 0], [23, 0, 0]]
    }

So all this layers currently are making use of keyword attr while generating the symbol file in MXNet 0.12.0 version. I checked the _layers.py in various branches like dev, master, ci-test, mli-patch-1, all the branches for other layers (i.e. transpose, softmax etc) seem to be using attr only. I am not getting why you want it to be replaced with attrs.
Or else please do let me know if you are referring to some other attr which has to be replaced with attrs.

Why don't you use _get_attrs()? It supports both attr and attrs key.

@Prasad9 Is this still relevant? Could you rebase and test on master?


Tiny fixes: printf error messages and gitignore

Two tiny fixes, please merge :).
hashindex: Fix printf size specifiers in error messages
gitignore: all cython generated files

isn't %ju format and casting all these values to (uintmax_t) a more general solution?

This' a bit less noisy and should work on Windows :).
I saw elsewhere Windows isn't supported.  So"%j" (as in borg) is fine, and I agree uintmax is nice and explicit.

@sourcejedi is there a problem with %j and Windows? Or Cygwin?

It's a problem with native Windows.  http://stackoverflow.com/a/22537979/799204
Apparently Cygwin is a whole 'nother library, so if you're using that then I don't know.  I'd like to think it supports a 15-year old standard which has no fully-portable alternative :).


Date on info popup seems to be broken

Describe the bug
Showing Invalid Date instead of the formatted datestamp; most likely from one of my changes of data source objects to simplify the FE API calls and data use.
To Reproduce
Steps to reproduce the behavior:

Go to Staging
Hover over the (i) in the chart legend or in the profile dropdown in the top right of the screen
Date at bottom displaying incorrectly

Expected behavior
Should should the formatted datestamp, as before.
Additional context
Hopefully is just a mismatch in the input format or similar from the forecast data as it goes into the formatting function.

@brad if you let me know which data this is showing, I could check in the API that this is being made correctly

Did this get solved? I found sometimes you had to wait until the GSP is loaded, then its fine


Fix issue with setting Content-Type on S3 upload. Set order of clinical insights sections. Update package versions

Remove pre-Nov 2023 clinical insights component

This PR looks to have broken the build. The IEvidenceNew and IAuraClinicalDocOutputSectionNew might be missing. It fails locally and if I build in Amplify. If can build 838da88863a470995690c98ddf009959e81a4f8c in Amplify.


DEFAULT_WORKER_SRC version now matches pdfjs-dist version

If the workerSrc version and the pdfjs-dist version are mismatched, the following error will occur:
The API version "4.5.136" does not match the Worker version "4.4.168".

This error may occur whenever npm installs the latest incremental version of pdfjs-dist that has an API incompatibility with the worker version.
This change ensures the default workerSrc version matches the pdfjs-dist version.
Main Changes:
See lines 15 / 16 and 32 / 33. Sorry for the formatter changes.

Can confirm that this bug occurs when installing/using the latest version of react-pdf-highlighter-extended from npm.

Created a new PR that has the correct DEFAULT_WORKER_SRC string (without the "asdfasdf") and which pulls the version from the base package rather than the dist file. See #31


Why is xnxx.com added by default to the hosts file?

Is there a reason xnxx.com is added to the default hosts file? Is that site malicious? Asking for a friend.... My friend doesn't remember it being blocked in previous versions. Other similar sites are not added by default so my friend was curious as to why that site is being blocked in the main hosts file and is not part of a porn extension.

https://github.com/mitchellkrogza/Badd-Boyz-Hosts/blob/master/hosts#L3358
@StevenBlack @mitchellkrogza

Reference: #248

Picked it up in logs sending phishy redirects and strange behavior. Very much a heavy porn site and blocked as a result.

Busy running a script against a years worth of logs, probably going to take till tomorrow to complete running. If I don't see enough results for xnxx it will be removed along with anything else that scores a low score

@benlowry meanwhile, your friend may like to know that Tor (Onion) browsers bypass hosts files.
Friends don't let friends browse edgy sites without Tor.

Thanks @StevenBlack, glad to know it but that's actually a different Ben heh.

I know this article is from 2013 but nbc news says that xnxx.com is safe
"The results were more encouraging than Longmore expected. Porn sites xvideos.com, livejasmin.com, redtube.com, xnxx.com, youporn.com, and adultfriendfinder.com earned a 0 percent risk rating."
http://www.nbcnews.com/id/51498128/ns/technology_and_science-tech_and_gadgets/t/are-all-porn-sites-really-riddled-malware/
It's not really a tor-worthy site in my opinion, pretty standard content on par with other popular clearnet sites like pornhub or youporn.

Whoops sorry @benlowry.

Will be sorted in today's commit. Thanks for the heads up guys.

Just my two-cents.. I would say that there should not be any porn sites in a blacklist for adware/malware (unless the direct URL is known to be a hotbed of viruses and malware). Treading a fine line between safety and censorship there... besides, there are already lists for those that want to (for some weird reason) block porn on their networks.

Fixed in https://github.com/mitchellkrogza/Badd-Boyz-Hosts/commit/5443b12687c8ee9b1462b308d4311f77b894864e along with a few others.

@PromoFaux A lot of the porn sites listed in my list come from my nginx logs where they are sending referrer spam. I'm going to of course continually revise the list thanks to all your inputs here and any sites marked safe by Google Safe Browsing etc should not be in here. Still tweaking a script I wrote yesterday to grep through an entire year's worth of logs and anything that scores below a certain number of attempts against my sites will probably be dropped unless they start re-appearing again.

New host files are up!
Closing.

I find a few domains which must not be here. Where I can report this list?

Please report issues at https://github.com/mitchellkrogza/Badd-Boyz-Hosts

On 01 May 2017, at 6:56 PM, Erotic Videos<EMAIL_ADDRESS>wrote:
I find a few domains which must not be here. Where I can report this list?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@evansoner please don't comment on an issue that's 2 months old, closed, and is irrelevant. Open a new issue with your concerns.

@evansoner please log issues at https://github.com/mitchellkrogza/Badd-Boyz-Hosts as you can see with all issues logged with me I am very responsive to fixing them.


Android - React 0.21.0 - INTERNAL_ERROR

Getting this error on Android. This shows when I click a button that runs GoogleSignin.signIn(). It doesn't make it to the login view at all, just errors out. I have my debug token set as the fingerprint, not sure what else to check?
Error: INTERNAL_ERROR
    at GoogleSigninError.Error (native)
    at new GoogleSigninError (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:93823:188)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:93875:8
    at EventEmitter.emit (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:50043:23)
    at MessageQueue.__callFunction (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:48673:23)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:48577:8
    at guard (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:48531:1)
    at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:48576:1)
    at DedicatedWorkerGlobalScope.onmessage (http://localhost:8081/debuggerWorker.js:39:56)

can you try with latest NPM version please ?
also make sure the json config file is accessible. The sample app is running fine if you want to have a look.

Same error on the newest version (0.4.3) after a clean build. Also, I incorrectly put RN as version 0.20.0 in the title, I'm running on 0.21.0.

I just tried the example app:
WRONG SIGNIN Error: INTERNAL_ERROR
    at GoogleSigninError.Error (native)
    at new GoogleSigninError (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:46246:188)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:46298:8
    at EventEmitter.emit (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:9114:23)
    at MessageQueue.__callFunction (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:7872:23)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:7776:8
    at guard (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:7730:1)
    at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:7775:1)
    at DedicatedWorkerGlobalScope.onmessage (http://localhost:8081/debuggerWorker.js:39:56)

have you installed google apps on your simulator ?

@devfd , hi!
I've also also got an error WRONG SIGNIN Error: INTERNAL_ERROR on Genymotion simulator and
Error: A non-recoverable sign in failure occurred on my Nexus 5.

you get this error when you run the sample app ?

@devfd, no, it's mine. I'm using RN 0.20. So, I'm going to try your sample and come back with feedback.

@devfd, I've tried your example and have the next results:

Genymotion - WRONG SIGNIN Error: INTERNAL_ERROR
Nexus 5

WRONG SIGNIN Error: unknown status code: 12501
    at GoogleSigninError.Error (native)
    at new GoogleSigninError (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:15923:188)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:15975:8
    at EventEmitter.emit (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:18551:23)
    at MessageQueue.__callFunction (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17181:23)
    at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17085:8
    at guard (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17039:1)
    at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.android.bundle?

@devfd Couldn't get Goople apps on my emulator, but was finally able to get an android device to test on and the error didn't occur, so I think it's working as intended.

@m0ngr31  and @devfd
I have the same issue which you  are previously  facing
WRONG SIGNIN Error: unknown status code: 12501     at GoogleSigninError.Error (native)     at new GoogleSigninError (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:15923:188)     at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:15975:8     at EventEmitter.emit (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:18551:23)     at MessageQueue.__callFunction (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17181:23)     at http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17085:8     at guard (http://localhost:8081/index.android.bundle?platform=android&dev=true&hot=false:17039:1)     at MessageQueue.callFunctionReturnFlushedQueue (http://localhost:8081/index.android.bundle?
Have you fixed the issue?
Please  suggest  the solution if it worked fine .

@Hari70a I only had problems when it was trying to login without Google Services installed.

OK @m0ngr31
BUT I have google apps installed in both device and emulator(genymotion)
I  have also tested  the example project in the plugin. Still I have the same issue.
But suggest me some idea to fix it.
Thanks in advance

I am facing the same issue, but only when I include a webClientId and offlineAccess field in the configuration.
It was all working fine with an empty object in the configuration. Since my server needs to make some calls on the backend, I included webClientId and offlineAccess in the configuration. And it started giving the following error:-
A non-recoverable sign in failure occurred
Any ideas?

I think the issue here is about the signing configuration of the app and whether it matches with the SHA1 used to  generate the google configuration files. Note that the if you might be having different keystores for debug and release, so make sure the google-services.json file you are using was generated using the SHA1 of the right keystore file.


Transfer project back to personal GitHub

It looks like we're going to run out of free Netlify credit in just a few days, and I need to transfer the project back to a personal GitHub so we can use Vercel without having to buy a paid team plan, and a lot of other services require a paid plan for projects under the organization, which I don't have the money to pay for at the moment.

I'm surprised this project is transferred back to personal account for this reason, actually:

You can apply https://www.netlify.com/legal/open-source-policy/
You can link org project with your personal Vercel account, and use https://github.com/amondnet/vercel-action for deployment, take https://github.com/un-ts/lib-boilerplate/blob/main/.github/workflows/vercel.yml as example

@Rel1cx Or you can just use Cloudflare Pages:

Unlimited bandwidth
500 build minutes monthly (200 more than Cloudflare)

Also, we can just do builds on GitHub Actions (unlimited build minutes for open-source projects under acceptable usage) and deploy the dist to Cloudflare Pages.

@JounQin @SukkaW Thanks for the suggestions but I'll keep it under personal account until there's an obvious need to transfer it to the organisation.

@Rel1cx The greatest part of organization is collaboration and permission model, but if want to maintain this project solo, that's totally fine, while I'll never consider.


Set up documentation using Sphinx

[ ] Copy the content from the wiki to https://github.com/onnela-lab/forest/tree/develop/docs
[ ] Set up a Read the Docs project
[ ] Optionally set up a custom domain
[ ] Retire the wiki

@biblicabeebli please set up a CNAME record forest.beiwe.org with target readthedocs.io

okay i've set up the cname


actions.restart() not working with popupBridge.js

Description
When using a native SDK (e.g. https://github.com/braintree/popup-bridge-ios) alongside checkout.js to support a native experience launched from a WebView, there are certain issues arising. The main one is support for actions.restart which currently throws an error.
logger.js:65 ppxo_unhandled_error Object {stack: "Error: TypeError: Cannot read property 'then' of u…://www.paypalobjects.com/api/checkout.js:3080:13)", errtype: "[object Error]", timestamp:<PHONE_NUMBER>380, windowID: "f1710985cf", pageID: "117ed7732a"…}
print @ logger.js:65
log @ logger.js:175
error @ logger.js:228
(anonymous) @ setup.js:29
dispatchPossiblyUnhandledError @ exceptions.js:18
(anonymous) @ promise.js:111

types.js:119 Uncaught Error: TypeError: Cannot read property 'then' of undefined
    at Object.restartFlow (https://www.sandbox.paypal.com/webapps/hermes/static/js/button.js?build=2.0.2_20180209175845136.unx:4322:40)
    at https://www.paypalobjects.com/api/checkout.js:3712:40
    at Function.ZalgoPromise.try (https://www.paypalobjects.com/api/checkout.js:9080:37)
    at Object.handler (https://www.paypalobjects.com/api/checkout.js:3711:77)
    at https://www.paypalobjects.com/api/checkout.js:3132:32
    at Function.ZalgoPromise.try (https://www.paypalobjects.com/api/checkout.js:9080:37)
    at Object._RECEIVE_MESSAGE_TYPE.(anonymous function) [as postrobot_message_request] (https://www.paypalobjects.com/api/checkout.js:3128:70)
    at receiveMessage (https://www.paypalobjects.com/api/checkout.js:3059:77)
    at messageListener (https://www.paypalobjects.com/api/checkout.js:3080:13)
    at Object.restartFlow (https://www.sandbox.paypal.com/webapps/hermes/static/js/button.js?build=2.0.2_20180209175845136.unx:4322:40)
    at https://www.paypalobjects.com/api/checkout.js:3712:40
    at Function.ZalgoPromise.try (https://www.paypalobjects.com/api/checkout.js:9080:37)
    at Object.handler (https://www.paypalobjects.com/api/checkout.js:3711:77)
    at https://www.paypalobjects.com/api/checkout.js:3132:32
    at Function.ZalgoPromise.try (https://www.paypalobjects.com/api/checkout.js:9080:37)
    at Object._RECEIVE_MESSAGE_TYPE.(anonymous function) [as postrobot_message_request] (https://www.paypalobjects.com/api/checkout.js:3128:70)
    at receiveMessage (https://www.paypalobjects.com/api/checkout.js:3059:77)
    at messageListener (https://www.paypalobjects.com/api/checkout.js:3080:13)
    at Object._RECEIVE_MESSAGE_TYPE.(anonymous function) [as postrobot_message_response] (https://www.paypalobjects.com/api/checkout.js:3160:122)
    at receiveMessage (https://www.paypalobjects.com/api/checkout.js:3059:77)
    at messageListener (https://www.paypalobjects.com/api/checkout.js:3080:13)

paypal.version="4.0.173"
Test site: https://ppxoab.herokuapp.com/test.html
Happens consistently and on the developer site
Steps to reproduce
Try to use a native SDK with checkout.js and try actions.restart() in onAuthorize
Affected browsers

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge
[ ] IE
[ ] Chrome Mobile/Tablet
[ ] Safari Mobile/Tablet
[x] Web View / Safari ViewController
[ ] Other

Thanks for the report, I'll take a look.


Upgrade to JupyterHub 5.0.0b2

Reference Issues or PRs

Follow up to https://github.com/nebari-dev/nebari/pull/2427
https://github.com/nebari-dev/nebari/pull/2468
Sibling to https://github.com/nebari-dev/nebari-docker-images/pull/145 (may need to be merged and pre-released first)

What does this implement/fix?
Put a x in the boxes that apply

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds a feature)
[ ] Breaking change (fix or feature that would cause existing features not to work as expected)
[ ] Documentation Update
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build related changes
[x] Other (please describe): version update

Testing

[ ] Did you test the pull request locally?
[ ] Did you add new tests?

Any other comments?
For reference the helm chart - JupyterHub version mapping is available at https://hub.jupyter.org/helm-chart/

The failure is unrelated, it comes from the vscode button being elsewhere now that we use jupyterlab-new-launcher:

I will open a PR to fix this.
As for JupyterHub upgrade, I wonder if styling of top navigation bar is off. I think it was not like this before, but I need to check:


unmark GuestProxyAgent service before start it

After we uninstall the azure-proxy-agent package, our GuestProxyAgent service is marked. If we need to install/start that service it again, we need unmark it first.

Hello, I cannot find systemctl unmark in the manpage https://www.freedesktop.org/software/systemd/man/latest/systemctl.html# -- what exactly should this do? Was this perhaps a typo for unmask, which does exist, but which shouldn't be used directly in maintainer scripts? Or is this an unreleased systemd feature that you're using?
I strongly recommend using the debhelper framework for managing these files, as it handles many subtleties for you https://manpages.debian.org/testing/debhelper/dh_installsystemd.1.en.html
I also strongly recommend putting code reformatting changes in their own commits, so that meaningful code changes can be more readily discovered separate from mechanical changes. As far as I can tell, this is 380 lines of changes, which break down into a few categories:

changing thirty or so comments into echo'd output
inserting a bug in the Debian postinst
some test changes that are hard for me to quickly describe
hundreds of lines of whitespace changes that appear to do nothing

I suspect the bug might have been easier to see if it were one of thirty lines rather than one of 380 lines. (I say bug, but maybe you're just using an unreleased feature of systemd? If so, my apologies.)
Thanks


Redisson mapwriter

I am using RMapCache<String,Object> with map option with write mode as WRITE_BEHIND
now when i put something in cache after particular delay it is getting executed after dealy provided,
but what i need is map writer should pick entry from map unless it is cache but it is not doing so it is only getting executed once.
Can anyone help what can i do in this case or there is any alternative?

MapWriter writes cache entry during put operation execution only.

MapWriter writes cache entry during put operation execution only.

is there any alternative way in which i can loop my cache entry till the value is not getting evicted?

is there any alternative way in which i can loop my cache entry till the value is not getting evicted?

I'm afraid no.

I'm afraid no.


[OSPP] [Feature] Unify query planner and executor in BanyanDB

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Currently, metrics from OAP and TopN generated from source metrics are both stored as a Measure. However, they are using different paths to read the data from the storage due to some historical reasons. Measure query is more complicated and comprehensive while TopN query is somehow ad-hoc. Then after finishing TopN streaming modules, we decide it is a good timing to fully unify the query planner and executor for Measure and TopN.
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Hello @lujiajing1126 , I find this issue from ospp-2023 and be interested in this task.
I hope to receive guidance on how to get started, because banyand/measure seems complicated(maybe i found the wrong package) and there seems to be little comment. I would greatly appreciate it if I could receive your guidance

Hello @lujiajing1126 , I find this issue from ospp-2023 and be interested in this task. I hope to receive guidance on how to get started, because banyand/measure seems complicated(maybe i found the wrong package) and there seems to be little comment. I would greatly appreciate it if I could receive your guidance

Hello, can you reach me thru email<EMAIL_ADDRESS>


Experimental: Mariadb 10.2

The Problem/Issue/Bug:

We haven't been able to use debian-based containers for mysql/mariadb in the past on windows
The mariadb library container, which we'd like to use is debian-based
Since we are trying to get away from bind-mounting in https://github.com/drud/ddev/pull/987 and using a docker volume instead, it just might work now. This is based on #987.

Experimental only.
How this PR Solves The Problem:
Manual Testing Instructions:
Automated Testing Overview:

Related Issue Link(s):
Release/Deployment notes:

WIll try this again later, have to get the docker volume PR in first.


Add option to open chrome in incognito mode

Just to avoid polluting the history with all the searches made.
Thank you for this little tool, very useful and easy to use.
Edit: I just realized that by doing this, you are not logged in lol. Nvm then

Hi @flesnuk , this would be a nice feature. The thought had occurred to me, but I came to the same conclusion you did. Operating in incognito mode would require signing into bing every run, which didn't seem ideal.
Glad you get some use out of this :D

Yeah, at least I see there is no problem in setting the exe for the Edge browser (which I don't use) so for me that already solves it 😃
Also, it's a shame in Europe the rewards are lower than US. (150pts vs 90pts for PC and 100pts vs 60pts for mobile). But still it is 6~7€ monthly for free.


Clarify idemptotency behaviour on POST /MeterReadings*

When an API users POSTs one or more MeterReading, the anticipation is that this meter reading is "added". This means that the operation is not idempotent - issuing the same request twice will create two items.
For file import, this semantic does not always make sense. Typically one uploads data for one day. Later on the data might have changed (or perhaps the user did not get a reply from the server and does not know whether the request succeeded or not), and want to upload the same data file. In that case entries will be replaced, not added.
The same could happen for a simple POST to /MeterReading: If there already is a meter reading for the given resource and the given timestamps, servers will typically overwrite the old value.
The server should therefore return either 204 NoContent (if content was replaced) or 201 Created if content was added.
For file imports, the result should typically be 204.
The documentation for these methods should specify this behaviour, and response codes should be added to the OpenAPI document.

@cdmNSIDE @ArnaudDebray what do you think, can we close this issue now that #67 has been merged?


[577]:... typography and font responsiveness

New breakpoint file created
New fontBreakpoint file created to cater for the different media queries for heading tags
Responsiveness is slated for headers h1 through h6

@osamajandali there was no particular reason. I just thought it to be a good seperation of concern.

@Darrellbor From what I understand there is no separation because you re-include it again in font.ts
It's good idea to keep it simple in font.ts

Alright noted

@osamajandali adjusted appropriately.


Update user guide with editLoan command and command summary.

Codecov Report

Merging #93 (d9ac7c0) into master (80954a6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #93   +/-   ##
=========================================
  Coverage     69.49%   69.49%
  Complexity      567      567
=========================================
  Files            95       95
  Lines          1849     1849
  Branches        218      218
=========================================
  Hits           1285     1285
  Misses          480      480
  Partials         84       84

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Add unread notification indicator, clean up notifications

Adds unread notification indictor badge to the nav bar
move indicator badges to top right and change color from red to blue
Fix notification items icons
Update visuals for notification drawer action buttons
Move toast notification list to be below the nav bar
Update administration section’s icon

Indicators and Toast Positioning:

Notification drawer items, Administration section icon:

@serenamarie125 @beanh66 @chriskacerguis

@jeff-phillips-18 does this need to go the euwe?

@chriskacerguis Yes. How should we be indicating that?

@jeff-phillips-18 just need to add the "euwe/yes" label.

@chriskacerguis OK, is there some way we should indicate it when submitting the PR so that you (or whomever else) knows to add the label?

@jeff-phillips-18 you can add labels yourself with the MIQ bot, see the readme here https://github.com/ManageIQ/miq_bot/ for how to do that.

Very cool, thanks @chriskacerguis !

Euwe Backport conflict:
$ git cherry-pick -e -x -m 1  497b1a2
error: could not apply 497b1a2... Merge pull request #218 from jeff-phillips-18/notifications
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'

$ git status
On branch euwe
Your branch is up-to-date with 'upstream/euwe'.
You are currently cherry-picking commit 497b1a2.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:

	modified:   client/app/components/navigation/_navigation.sass
	modified:   client/app/components/notifications/_notifications.sass
	modified:   client/app/components/notifications/notification-footer.html
	modified:   client/app/config/navigation.config.js
	modified:   client/app/layouts/application.html

Unmerged paths:
  (use "git add <file>..." to mark resolution)

	both modified:   client/app/components/navigation/navigation-contoller.js

$ git diff

diff --cc client/app/components/navigation/navigation-contoller.js
index 3dcc8c3,e0e9f53..0000000
--- a/client/app/components/navigation/navigation-contoller.js
+++ b/client/app/components/navigation/navigation-contoller.js
@@@ -182,9 -186,12 +182,18 @@@

      function refreshNotifications() {
        vm.notificationGroups = EventNotifications.state().groups;
++<<<<<<< HEAD
 +      vm.newNotifications = angular.isDefined(vm.notificationGroups.find(function(group) {
 +        return group.unreadCount > 0;
 +      }));
++=======
+       vm.newNotifications = EventNotifications.state().unreadNotifications;
+       vm.unreadNotificationCount = 0;
+       angular.forEach(vm.notificationGroups, function(group) {
+         vm.unreadNotificationCount += group.unreadCount;
+       });
+       vm.notificationsIndicatorTooltip = __(vm.unreadNotificationCount + " unread notifications");
++>>>>>>> 497b1a2... Merge pull request #218 from jeff-phillips-18/notifications
      }

      function refreshToast() {

@jeff-phillips-18 can you please address this?

@chriskacerguis Put up a PR for this cherry-pick to euwe. Let me know if this was not the right thing to do.

Backported to Euwe via #223


Added some cosmetics to autoconf related code

Started to build the v3/master, and found some configure errors - fixed them.

Merged. Thanks ;)


Crash using Carthage install in Objective-C Project

When installed with Carthage in an Objective-C project, this crashes when the list cells are shown:
let fetchedList = (try? managedObjectContext.fetch(fetchRequest))
in the
fetchOGData
method.

It is fixed greater than  0.12.0
Please use latest version

Carthage pulls 0.12.0. I am new to Carthage so I haven't found a way to specify the version. I just found this info: "Carthage does not support prerelease versions." Is this a prerelease version? Thanks.

Can you possibly push the next version after 0.12.0 to Carthage?

I am using 0.12.0 and this is where I have been getting this error.

This is Objective-C sample project's screen capture.
It works fine with Carthage update.

If you can not launch app, please try to add Dummy.swift(this is empty swift file) and bridging-header.h(this is created by xcode).

I'm not trying to be difficult, but I do think your sample project is a different case. In your sample project, you are importing Objective-C into a Swift Project. In my project, I am importing Swift into an Objective-C project.

Bridging-header.h is what is used in a Swift project to import objective-c code. So unfortunately, that makes the sample you have above a Swift project. I am so sorry to bother you, I am just so excited about this code and really want to get it to work in my objective-c app. I wish I knew Swift, but this is an old app and I don't know Swift yet.

I think your understanding is wrong.
This comment is new Objective-C base project.
This is no uploaded example.
First of all, libswiftCore.dylib is needed to use Swift.
Objective-C base project dose not contain libswiftCore.dylib, therefore you need to add Dummy.swift(this is empty swift file) to your Objective-C project.
In addition, I've uploaded URLEmbeddedViewSampleObjc.zip which is Objective-C base URLEmbeddedVIew project.
To read this project, you will notice your wrong understanding.

Thanks for the sample project. I can't get it to run even though it does build. It crashes each time with this error:
dyld: Library not loaded: @rpath/libswiftSwiftOnoneSupport.dylib
Referenced from: /Users/home/Library/Developer/Xcode/DerivedData/Build/Products/Debug-iphonesimulator/URLEmbeddedView.framework/URLEmbeddedView
Reason: image not found

What is your Carthage version?

0.26.2

Does that library require cocoapods?

dyld: Library not loaded: @rpath/libswiftSwiftOnoneSupport.dylib
Referenced from: /Users/home/Library/Developer/Xcode/DerivedData/Build/Products/Debug-iphonesimulator/URLEmbeddedView.framework/URLEmbeddedView
Reason: image not found
(lldb)

dyld: Library not loaded:

@rpath/libswiftSwiftOnoneSupport.dylib

Referenced from: /Users/home/Library/Developer/Xcode/DerivedData/Build/Products/Debug-iphonesimulator/URLEmbeddedView.framework/URLEmbeddedView
Reason: image not found
(lldb)

That sample is work perfectly in my environment with Carthage 0.26.2

The libswiftSwiftOnoneSupport library seems to be the issue. I ran your code as is and I get the above error. Crazy!

I am using Version 9.0.1 (9A1004) of XCode by the way

And running this in iPhone X Simulator

Please try carthage update.

Yes, I had done that already. Still get the libswiftSwiftOnoneSupport error. Ugh.

Dose it work your environment?
URLEmbeddedViewSampleObjc2.zip

So odd. Once I deleted the bad (missing) framework at the top it does build. But the same error when it runs:
dyld: Library not loaded: @rpath / libswiftCoreData . dylib
Referenced from: /Users/home/Library/Developer/Xcode/DerivedData/Build/Products/Debug-iphonesimulator/URLEmbeddedView.framework/URLEmbeddedView
Reason: image not found

Even when I replace that 'bad' library at the top by dragging the Carthage/Build/iOS library there, I still get the same error

I've updated xcode 9.0.1 and try to run that sample. It works fine.

I think this is the issue I am having:
https://github.com/Carthage/Carthage/issues/1394#issuecomment-233958908

Do you try to make "Always Embed Swift Standard library" to "true"?

I can't get the swift libraries to show up in my project

Yes, I have tried that and it doesn't help unfortunately : (

I feel like maybe I need to add it as an embedded library

Embedded library sample project is already in [this comment].
That is not use carthage, but It references URLEmbeddedView.xcproject.

okay, I did get it to embed, but this is the error I am getting in my project which I actually think is causing the problem
/Carthage/Checkouts/URLEmbeddedView/Resources/URLEmbeddedViewOGData.xcdatamodeld: Missing current version declaration

Can you tell me the steps you took to embed it in the sample project?

Now I'm back to the original error - a crash on
let fetchedList = (try? managedObjectContext.fetch(fetchRequest))
in the
fetchOGData
method. Banging my head on the wall.

OK - getting there. It's a multithreading violation. I have something turned on in my code which crashes on multithreading violations. Don't know how to fix it though. Tried running in the background but I get the feeling the violation may possibly be in the project itself - is this possible?

CoreData`+[NSManagedObjectContext Multithreading_Violation_AllThatIsLeftToUsIsHonor]:

This is what I have turned on:
https://oleb.net/blog/2014/06/core-data-concurrency-debugging/

Yup! That was it! When I turn off that setting it works. When I turn on that setting in the sample project it crashes. So the issue is a multithreading violation in URLEmbeddedView.

So I have continued to test and ultimately I do think your Carthage installation is fine. "Always Embed Swift Standard Libraries" does not have to be set to YES (it is no in your sample project and I find I do not need it to be set to YES either). The error is a multithreading violation in URLEmbeddedView. Mystery solved - although the problem still remains! I don't know Swift (obviously) or I would try to help you debug. I suspect maybe a missing performBlock? https://developer.apple.com/library/content/documentation/Cocoa/Conceptual/CoreData/Concurrency.html

Thank you for your investigation!
Let me make sure one thing, this code is crashed in your project?
NSURL *url = ...
[[OGDataProvider sharedInstance] fetchOGDataWithUrlString: url.absoluteString, completion: ^(OGData *ogData, NSError *error) {
    NSLog(@"ogData = %@", ogData);
    NSLog(@"any error = %@", error);
}];

This code can get OGData directly.

Yes - that is actually what I am using. That crashes in the project. Were you able to reproduce by changing that setting in the sample project?

And thank you for your patience in working through things with me!

Do you think you will have any time to work on a fix for this. I understand time is limited. Again, I wish I knew Swift so I could help!

I've released 0.13.0!
Please try it😀

Yay! Latest version in Carthage is 0.12.0 though, so I can't update until it has been changed to 0.13.0 unfortunately...

I will look forward to updating. I also had a couple of other thoughts. 1) Have you ever thought of searching for the Twitter: share tag as well as the og: ? 2) Have you considered creating a non-core-data cache (more of a short term cache)? When I shut down my project with this code in it in the simulator, it also crashes upon next startup (which I think might be related to the issue you are solving). But I use Core Data in my project and I would rather not utilize it in a library. Is there more of a short-term way to cache the links without using Core Data? Thanks again for this great library and I look forward to trying the fix when Carthage gets updated.

I could update via Cartahge.
.
URLEmbeddedView creates another SQLite file (CoreData uses SQLite inside) that does not affect your project's SQLite file.
So I think another caching system is not needed.

Hi Marty! I couldn't come back to this because I was busy, but I am back now! I am nervous about adding anything core data because it is SO touchy (as we have seen) and just prone-to error. If I install the repository in my app, is there any way to do it without a core data database being installed in the app?


Convert APK Pipeline to GitHub Action

what

Preserve the ability to run steps in parallel (as much as possible)
Build the docker images and use them in subsequent steps (is it possible?)
Support github PR comments from contributors (or trusted team) to trigger builds on PR comments matching regex (e.g. /\/ok-to-test/)

why

We have had trouble running builds for Codefresh PRs on forks.
Good first project.

@osterman I think we can close this as done now, don't you?

Closed in #392


Fix 1620: cookiecutter templates failing to clone

Resolve symbolic link in the git folder path.
In case of a process error, always print out the executable and exit code.
Check strictly for exit code 0 for our python scripts. Cookiecutter will return positive value for error. We now have much better error reporting for iknite/cookiecutter-ansible-role template which fails during post hook, and other templates now report errors when they were ignored before.

👍

Wanted to merge my logging fix first so I can see why the test run was failing so bad.


phoebemirman/ch15100/change-header-styles-to-match-mock

https://app.clubhouse.io/augur/story/15100/change-header-styles-to-match-mock

Coverage remained the same at 66.502% when pulling cc8c4501a26ac58d91f667b266331c17f138e331 on phoebemirman/ch15100/change-header-styles-to-match-mock into 0eafd96f130d70cf6e4b111061e28fd808255391 on master.


build fails

Describe the Bug
I can't build astra-sim after installing dependencies in different OS (Mac and Ubuntu)
Steps to Reproduce
At this step I get the following error:
./build/astra_analytical/build.sh
/Users/jose/CLionProjects/astra-sim/build/astra_analytical/../../extern/graph_frontend/chakra/schema/protobuf: warning: directory does not exist.
Could not make proto path relative: et_def.proto: No such file or directory

Expected Behavior
Build to succeed.

Could you check if you have cloned the submodules?

This issue can be closed. I was able to build it based on the ASPLOS 2023 tutorial. Thanks!


Adding animation

Using AOS Library for animation to elements to improve the user experience.
I would love to work on this issue.

Go Ahead @Geetisha17

I will animate the submit button and the radio button too!


Define the resulting package metadata

package_url:
package_type: [pypi | ...]
source_url:
source_type: [github | gitlab]
package_name:

maturity_score:
health_risk_score:

Etc what other fields should be included?
Acceptance criteria
The project readme is updated with the data structure

General List of Fields we will need to collect
bugtrack_url:
docs_url:
project_urls:
home_page:
maintainer:
maintainer_email:
release_url: [{version, date}, ...]
first_release
last_release
license
requires_dist:
version:
monthly_downloads_pypi
monthly_downloads_conda
github_stars
github_forks

These fields will be removed from the data since all projects return -1 for all 3 fields
downloads.last_day:
downloads.last_month:
downloads.last_week:

@karamba228 I think fields like github_stars and info should be gotten 'live' from the frontend i.e. when the user views the site. what if the source is hosted on sourceforge or gitlab?
These fields will be required as intermediate to calculate the scores, but They should not persist. eg what if we do npm, we would need to add all npm metadata like monthly_downloads_npm
here my a revised 'final' output of the scores project
{
    "packages" : [
        {"package_type": "pypi",
        "package_url": "https://pypi"
    }],

    "source_url": "https://github.com",
    "homepage_url": "foo.com"
    "source_type": "github",
    "package_name":  "asdf",

    "maturity_score": "mature",
    "maturity_score_notes": [
        "Package is over 2 years old"
    ],

    "health_risk_score": "healthy",
    "health_risk_notes": [
        "Package has a low number of contributors"
    ]
}

The source_url should be a primary key

@srossross @karamba228
How is this json format to start with I have considered source type as pypi here.
{
    "project": {
        "name": "asdf",
        "source_type": "pypi",
        "source_url": "https://pypi.org/project/python-asdf/",
        "homepage_url": "https://foo.com"
    },
    "packages": [
        {
            "package_type": "pypi",
            "package_id": "python-asdf",
            "latest_version": "2.7.1",
            "release_date": "2023-09-15"
        },
        {
            "package_type": "conda",
            "package_id": "conda-forge/asdf",
            "latest_version": "2.7.1",
            "release_date": "2023-09-15"
        }
    ],
    "scores": {
        "maturity": {
            "score": "mature",
            "notes": [
                "Package is over 2 years old",
                "Regular updates and releases"
            ]
        },
        "health_risk": {
            "score": "healthy",
            "notes": [
                "Package has a low number of contributors",
                "No known vulnerabilities",
                "Good test coverage"
            ]
        }
    }
}

Looks good to me


added missing deviations

Pull Request Functional Test Report for #2960 / 890857cde4a3224d642b54973355e305f65df0c9
Virtual Devices

Device
Test
Test Documentation
Job
Raw Log

Arista cEOS

RT-1.27: Static route to BGP redistribution
a2acf22c
Log

Cisco 8000E

RT-1.27: Static route to BGP redistribution
11de731b
Log

Cisco XRd

RT-1.27: Static route to BGP redistribution
23ffef73
Log

Juniper ncPTX

RT-1.27: Static route to BGP redistribution
07552229
Log

Nokia SR Linux

RT-1.27: Static route to BGP redistribution
088f9d19
Log

Openconfig Lemming

RT-1.27: Static route to BGP redistribution
47a6d9f4
Log

Hardware Devices

Device
Test
Test Documentation
Raw Log

Arista 7808

RT-1.27: Static route to BGP redistribution

Cisco 8808

RT-1.27: Static route to BGP redistribution

Juniper PTX10008

RT-1.27: Static route to BGP redistribution

Nokia 7250 IXR-10e

RT-1.27: Static route to BGP redistribution

Help

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 55.859%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1983

Relevant Lines:
3550

💛  - Coveralls


Create patch function

This PR introduces the patch function to convert Pydantic exceptions to a custom exception class.

The proposed changes look really nice! Couple of ideas and suggestions on how to finalize.


about ubuntu 'version `GLIBC_2.33' not found' compilation problem

Because of the  version 'GLIBC_2.33' not found problem
the official said that compiling the old version of Ubuntu 18 can solve it
but the oldest Github Action is only Ubuntu 20.
How to solve this problem?

Solved with ubuntu 20.04


installation errors in docker

....
IRISAPP>
ERROR! 'isc-dev' not found in any repository.
IRISAPP>
DO ##CLASS(dev.code).workdir("/irisdev/app/src")
^

 *dev.code
IRISAPP>
...
Load started on 09/06/2021 11:28:08
Loading file /opt/irisapp/module.xml as xml
Imported document: iris-PromJet.ZPM
Load finished successfully.
ERROR! Could not find satisfactory version of dsw in any repositories. Required by: iris-promjet: 3.*
IRISAPP>
IRISAPP>
%SYS>
1
%SYS>
Modifying MDX2JSON application security...
%SYS>
%SYS>
%SYS>
%SYS>
%SYS>
ERROR #869: Application /mdx2json does not exist1
%SYS>
%SYS>
Removing intermediate container 80213879e261
---> 16a93f9b9eed
Successfully built 16a93f9b9eed
Successfully tagged promjet-stats_iris:latest

@NjekTt look at this log, please.


Refactor to HOC

This includes fixes from suggestions mentioned in PR #117

remove extraneous .binds() from render
bind class functions used to instance in constructor.
add new line between {...props} and {...state}
add default props and proptypes to CropSelection
add documentation to readme

Some additional notes:
eslint did not complain the same as @DominicTobias eslint, so I think you should check and see if you have eslint globally installed?  If so, it might be superceding the eslint configuration for this repo.  As when I ran eslint with this repo there were many issues, to which I've fixed all of them with this final commit :+1:
Thanks again, let me know if any issues!

Good to know RE eslint, I do have it globally installed for me IDE

Published 2.1.0

Hey @MattMcFarland, there was a bug with the 2.1.0 release so I had to roll back, webpack would need to output 3 files to dist to get it working so I fixed it quickly by rolling back. After that I ended up doing some refactoring for v3 (main things being not keeping crop state in the component and fixing a long standing issue that was bugging me to do with the crop area continuing to expand when hitting an edge on the opposite side).
Anyway I've come to accept that I like the simplicity of it all being in one file, I think it's easier to reason about not being a HOC. Thanks for all your hard work but please make a fork instead :)

Hey no problem! It was a fun challenge nonetheless. I'm trying to use it
without a hoc, and i really like the changes you put in v3. As they are,
I'm finding it much easier to work with so I'm going to go without it being
a hoc anyway.
I might try to contribute again but this time unit tests.
On Oct 7, 2017 10:38 AM, "Dominic Tobias"<EMAIL_ADDRESS>wrote:

Hey @MattMcFarland https://github.com/mattmcfarland, there was a bug
with the 2.1.0 release so I had to roll back, webpack would need to output
3 files to dist to get it working so I fixed it quickly by rolling back.
After that I ended up doing some refactoring for v3 (main things being not
keeping crop state in the component and fixing a long standing issue that
was bugging me to do with the crop area continuing to expand when hitting
an edge on the opposite side).
Anyway I've come to accept that I like the simplicity of it all being in
one file, I think it's easier to reason about not being a HOC. Thanks for
all your hard work but please make a fork instead :)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DominicTobias/react-image-crop/pull/118#issuecomment-334939491,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmDuZmFRv_04MHBl7eThi8f1h5NFRADks5sp4zqgaJpZM4Pmixy
.


BUG: Update TileMergeImageFilter signature in example script

Updates the SimpleMontage.py example script to reflect itk.TileMergeImageFilter type wrapping updates that were introduced in 5b77535871525f917b9cce8d2e6fdbeb3e47fb82.
Resolves issue where the example would fail to instantiate the filter and halt. Tested with RGB input images.
xref https://github.com/InsightSoftwareConsortium/ITKMontage/commit/5b77535871525f917b9cce8d2e6fdbeb3e47fb82

We should probably add a Python test to make sure this doesn't regress again.


Carthage Support?

https://github.com/Carthage/Carthage#supporting-carthage-for-your-framework

I can add carthage support.
Thanks for this beautiful tool. @alberdev

archived framework added too ;)


dynamic cursorShape for shell vi mode

Hello Terminal team 👋
Description of the new feature/enhancement
When normal mode is active in vim or shell vi mode, the cursor stays in the same style ("bar"). On mac and linux terminals the cursor would typically change to something like the "emptyBox" shape for normal mode and "bar" for insert mode.
To reproduce this behavior, we can use Windows Terminal to run cygwin or access a linux machine via ssh and run the following:
set -o vi

Then, pressing escape will enter normal mode where we might expect to see a box highlighting the current letter. Instead we still see the "bar" style, moved one character back. The "box" appearance is helpful to see which character is selected and as a visual aid to know when insert or normal mode is active.
I found a few workarounds for vim using vimrc (i.e. in issue #68) but have not yet seen a suggestion for shell vi mode.

Maybe some combo of #4106, #1604 - I could have sworn there was a more specific dupe...

Windows Terminal already supports the DECSCUSR control sequence with parameters from 1 to 8.  There's nothing else that Windows Terminal has to do, here, pace the bug about it not taking effect until flushed.  The rest is all at the host end.
The rest is a matter of having the Ss and Se capabilities in your terminfo database records, and a shell that understands those (relatively new, as they were invented in 2011) terminal capabilities.  It is almost certainly terminal type and terminfo that is your problem here.

Indeed! Thanks @jdebp.
In addition, somebody's put together a solution for versions of readline that do not take Ss and Se into account: https://stackoverflow.com/a/48449104

Thank you for the responses. In my case, cygwin with bash 4.4.12 did not work with show-mode-in-prompt, and (as expected) this did not work in an older linux bash 4.2.46 I had available for testing. It will take some more investigation on my part to troubleshoot.


How to use  Chat App Using SSE

I don't use Chat App Using SSE in example  project

What are you trying to do?

Chat

https://htmgo.dev/docs/pushing-data/examples/chat
404

ah, will fix, it should be https://htmgo.dev/examples/chat

@maddalax
There is a bug. I create a chat, copy the link. I open it in a private window. The chat opens, but there are no messages, no user list, and nothing is sent.


Add REGEX for Hex-Values

Done. See: https://github.com/Petschko/RPG-Maker-MV-Decrypter/commit/3cb6852488b474a2aa7e4653fac07c08821d9fd7


Conflicting info on ability to change Minimum TLS

https://docs.microsoft.com/en-us/azure/synapse-analytics/security/connectivity-settings#minimal-tls-version
Synapse Dedicated pools do not allow configuration of minimum TLS being used.
https://docs.microsoft.com/en-us/azure/synapse-analytics/security/connectivity-settings#minimal-tls-version
"These settings apply to all SQL Database and dedicated SQL pool (formerly SQL DW) databases associated with the server."
The Get-AzSQLServer command is for Azure SQL only right? Not an expert, but I cannot change minimum TLS on Synapse and their documentation says it's not possible.

@thesqlpro : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Note that publishing in this repository is suspended until Tuesday, November 2. Merging will resume Tuesday morning. Authors can still review and sign off on PRs in the interim. Thanks.

#assign:williamdassafmsft

Hi @thesqlpro! I think this might be a little product name confusion between the current Azure Synapse Analytics dedicated SQL pools and the pre-Synapse, legacy dedicated SQL pools (formerly SQL DW) only. Key is "in Azure Synapse Analytics" here, but I'll take this under advisement and see if we can be more clear in future updates.
We currently don't support setting the minimum TLS version for servers that are part of Azure Synapse Analytics. The minimum is TLS 1.2, we don't have any update on features to enable support for less secure connections.
#please-close


Unable to create VM with un-managed disk from custom image

I created a linux VM through the Azure SDK and added a couple of managed disks to it. I created a custom image out of it after deprovisioning, deallocating and generalizing it through the azure java SDK closely following this sample.
However, I am unable to spawn a VM that uses un-managed disk from this image. The moment I select withLinuxCustomImage() in the VM definition I can no longer specify withUnmanagedDisks() . Is this by design ? Implying I can never create a VM with un-managed disks from a custom image.

@ashishdhar, it's a service constraints that a virtual machine created from custom image (which is managed by azure as a resource) can't use un-managed (storage account based) disks.

@anuchandy this seems a bit strange because it is possible to create a VM out of a custom image using the ComputeManagementClientImpl() by creating a new OSDisk() and assigning it the the blob uri of a storage account.

@ashishdhar this observation look interesting. Reopening the issue. I remember this was not supported at the time of implementing managed disk and custom image. Do you mind sharing the complete repro code? That includes:

Creating VM
Deallocating and Generalizing the VM
Creating custom image from the VM
Code that you have written using Impls which uses the custom image and storage account

Tried to repo this - below is the payload used to PUT a VM based on custom image with un-managed os-disk
{
    "location": "westcentralus",
    "tags": {},
    "properties": {
        "hardwareProfile": {
            "vmSize": "Standard_D3_v2"
        },
        "storageProfile": {
            "imageReference": {
                "id": "/subscriptions/<subscription-id>/resourceGroups/<rg-name>/providers/Microsoft.Compute/images/<image-name>"
            },
            "osDisk": {
                "name": "anuqwer",
                "vhd": {
                    "uri": "https://<stg-name>.blob.core.windows.net/vhds/<vhd-name>.vhd"
                },
                "caching": "ReadWrite",
                "createOption": "fromImage"
            }
        },
        "osProfile": {
            "computerName": "vm2d05040035",
            "adminUsername": "<uname>",
            "adminPassword": "<pwd>",
            "linuxConfiguration": {
                "disablePasswordAuthentication": false
            }
        },
        "networkProfile": {
            "networkInterfaces": [
                {
                    "id": "/subscriptions/<subscription-id>/resourceGroups/<rg-name>/providers/Microsoft.Network/networkInterfaces/<nic-name>",
                    "properties": {
                        "primary": true
                    }
                }
            ]
        }
    }
}

This failed with error:
{
  "error": {
    "code": "OperationNotAllowed",
    "message": "VM created from Image cannot have blob based disks. All disks have to be managed disks."
  }
}

which clearly says you cannot blob to back the disk of the virtual machine based on custom image.
Closing this now, feel free to reopen with specific report steps (with examples) and payload on the wire.


Find a  way to play with string (toLowerCase function) in Google closure

Solution could be with

Add/Modify  function toLowerCase() (Compiler doesn't accept parenthesis()) #165
Regex
Changing type
compiler validation  https://github.com/google/closure-templates/issues/116
call a js function

Project was restructured. Closing issue.


Test heteroskedastic bootstrap types "21" and "31" against fwildclusterboot

As soon as fwildclusterboot allows these variants (currently in the dev branch).

Should by now be in the master branch.

@s3alfisc  should this be closed now?


unread_inbox_emails with new UI

HI @all
after switching to gmail 2018 the number of unread mail is stop working. Back to old version and work again.
i found the call in the api.get.unread_inbox_emails
if(dom[0].text.indexOf
where to focus the problem. For me is not working, because the new UI in the a href text report only "Inbox" and not "Inbox (1)" like the older one.
I found a solution switching from text to title like this
if(dom[0].title.indexOf
Anyone have the same issue?

Thanks for the report and thanks for digging into it!
I agree with your assesment. Looking at the DOM for the new and old Gmail, it seems title is the attribute currently used for this value.
Based on that, changing from .text to .title across all api.get.unread_*_emails-functions looks safe to me.
Would you mind trying that, testing some different configurations (new/old gmail, regular gmail-account, gsuite-account, etc)?
If it turns out all that works, I'll be happy to accept a PR.
Please note gmail.get.unread_inbox_emails() has some special code which differs from the other unread-functions.
I'm not sure if this stuff is still required or not. All I know is that it was introduced by @Thomas101  in this commit. Since I'm not sure if it's needed or not... Let's just keep it to be safe.
Sounds good?

@josteink Thanks.
I forgot to mention i have already tested on new and old interface on gsuite. I have to do it into regular-gmail.
I know about the code, and i havent changed the part introduced by @Thomas101  because i think the problem is before that part.
I have not test other api call, like draft and so on, because i'm not use it, but i'll made some test on it

When you're ready, just issue a PR and I'll be quick to review it :)


Abasive Tech Customer Story

@Hasmin-AC @robmarcer
Do we have a time for when we want to promote?

In the summary can we add a line or two on how FlowFuse helped too.

@robmarcer Yes, last question from my side is if you updated the PDF for download too?

I've updated the PDF @ZJvandeWeg, thanks for the reminder.


event.context.cloudflare not available within the plugin due to Nuxt routing fetch to the server function rather than making the HTTP request

Environment

Operating System: Darwin
Node Version:     v18.14.0
Nuxt Version:     3.8.2
CLI Version:      3.10.0
Nitro Version:    2.8.0
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      app, nitro, modules, runtimeConfig
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Either with wrangler pages dev or a published pages deployment.
I'm using D1 as an auth adapter via Drizzle:
const D1DB: D1Database = event.context.cloudflare.env.DB
const DB: DrizzleD1Database = drizzle(D1DB)
authOptions.adapter = DrizzleAdapter(DB)

This auth configuration is set within /server/api/auth/[...].ts
You can call the above endpoint via HTTP and the request works fine.
Describe the bug
When the plugin calls the /api/auth/session endpoint, it fails as Nuxt hooks into $fetch and routes the request to the server function itself, rather than making a HTTP request. This means event.context.cloudflare is not available. This means I can't use my database adapter. This breaks many things.
Additional context
To temporarily fix this, I've adjusted the plugin to use the baseUrl from my runtimeConfig in the request. This forces fetch to make the HTTP request, rather than just calling the code.
Logs
No response

Workaround available in #138 🙏🏽


Switch to irlba to avoid Spectra dependencies.

This uses the diagonal shift trick to convert the problem of "finding the smallest eigenvalues" into the largest ones.

@LTLA was the motivation for this PR to remove Spectra as a dependency because of the weirdness reported at https://github.com/yixuan/spectra/issues/126?
I am also toying with removing RSpectra as a dependency mainly because it causes PREPERRORs with rhub when using UBSAN/ASAN sanitizers and with valgrind (a pretty important safety net for me). The irlba approach seems good enough, although it is noticeably slower for some artificial cases like a 1D line embedded in 3D.
As an aside, the maximum eigenvalue that can be attained for the normalized laplacian is 2, so I think you could pre-shift your matrix without having to extract the largest eigenvalue first (the approach I used to test irlba is outlined here and almost the same as what you are currently doing).

Edit: I have convinced myself. Needed to look it up but eventually worked it out.

Great. For reference, I have seen this asserted in several places, but the canonical place including a proof seems to be Spectral Graph Theory. Part 5 of Lemma 1.7 in Section 1.3, 'Basic facts about the spectrum of a graph'.

Just did the change for the max eigenvalue - seems to check out, we'll see how it goes.


Toast component close all method

Summary
It would be nice to have the functionality to just call close() method and close all instances of toast component.
For example if I want to close it from another file and another effect in store I would have done it like this:
this.toastService.close();
But currently I need to put this in a constant first like this (and then closing cannot be done from another file or effect without additional implementation or wrapper service):
const toast  = this.toastService.create({...});
toast.close();

Just development requested.
Justification
Currently in premium calculators we have 3rd party library ngx-toastr and closing can be done programmatically. Use-case would be if the toast appears for some input error then if I correct it and successfully submit the contract, I want to clear the toast and not to have to do it manually.

@stefanbukovski just use this.toastService.dismissAll()

@hirsch88 I looked it up but did not find it in BalToastService. There is only create() method there. Am I missing something?


How difficult will be to have a Code Generator for the MVC rails like ?

From @dinooz on September 6, 2017 15:46
I wonder how difficult will be to bring a C# code generator to automagically create all the mehtods, relationships generated from a basic models to speedup the creation of projects ?
Perhaps this needs to build a new project almost like yeoman or rails generator to based on some DB Model create and wire all the required project.
Copied from original issue: aspnet/Mvc#6775

We do have something like this: the Code Generator Scaffolding feature that has command line tools and a Visual Studio UI for generating controllers and pages.
Here's a doc on how to do it with Razor Pages: https://docs.microsoft.com/en-us/aspnet/core/tutorials/razor-pages/model
Though the scaffolding works with regular MVC apps as well.
Is that what you're looking for?

Ah, I see, so this is about scaffolding an entire database (all tables + all relationships) in one command?

From @dinooz on September 8, 2017 13:54
Close... have been working with K2 Forms and is really awesome, the way it works.
Define the SQL Data
Setup Data Sources as Service Instanace / then Smart Objects. [ Build CRUD of the SQL or Data Source]
Use the graphical interface to build the form and put some buttons calling actions and rules.
That's all the form is ready.
Taking that to the command like rails like, will be also cool, since will wire everything together. making super easy and simple get thing moving. Making it Simple is complex, but a lot more can be achieved.
Dinooz

Currently Scaffolding is only supported for ASP.NET Projects.


purifyCss is useless when .js() presented

Laravel Mix Version: 2.1.2
Node Version (node -v): 8.10.0
NPM Version (npm -v): 5.6.0
OS: Windows 10 Pro 1803

Description:
purifyCss is working but doesn't change the content of resulted .css IF .js() is presented (chained or not).
Steps To Reproduce:
dist/index.html
<div class="shouldStay">
    LOL
</div>

src/app.scss
.shouldStay {
    color: green;
}

.shouldBeGone {
    color: red;
}

app.js is empty
webpack.mix.js
let mix = require('laravel-mix');
let htmlPath = ['dist/index.html'];

mix.options({
    purifyCss: {
        paths:         htmlPath,
        purifyOptions: {
            minify:   true,
            info:     true,
            rejected: true,
        }
    }
})

mix.js('src/app.js', 'dist')
    .sass('src/app.scss', 'dist')
    .setPublicPath('dist');

production script is cross-env NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --config=node_modules/laravel-mix/setup/webpack.config.js
Output:
[...]
    ________________________________________________
    |
    |   PurifyCSS has reduced the file size by ~ 50.0%
    |
    ________________________________________________

    ________________________________________________
    |
    |   **PurifyCSS - Rejected selectors:**
    |   **.shouldBeGone**
    |
    ________________________________________________

 91% additional asset processing
 94% asset optimization
 95% emitting DONE  Compiled successfully in 2206ms21:59:29

   Asset       Size  Chunks             Chunk Names
 /app.js  532 bytes       0  [emitted]  /app
/app.css   **48 bytes**    0, 0  [emitted]  /app, /app

Process finished with exit code 0

webpack.mix.js
[...]
mix.sass('src/app.scss', 'dist')
    .setPublicPath('dist');

Output:
[...]
   Asset      Size  Chunks             Chunk Names
/app.css  **24 bytes**       0  [emitted]  mix
[...]

Looks like in Laravel Mix v2.1.0 it works as intended. Probably a regression after this path: https://github.com/JeffreyWay/laravel-mix/commit/95aa7d2990d43e770ad43a0289d94ec5e4a84f93

Confirmed. Fixing this now.

Tagged in v2.1.3.


With identify 1.6.2 ps1 files no longer correct recognized

See commit 31b2ced where support was added.
Currently when running identity-cli I now get:
["Executable", "file", "text"] and not .... "PowerShell"]

show the command you ran and the output?

that output you've written is impossible fwiw so I suspect pebkac

Yup sorry looks like pip-review "updated" identify from 2.5.35 to 1.6.2. reinstalled version 2.5.35 and it now works. I am guessing that it was not introduced until after 1.6.2. sorry about that. Thanks for the quick reply!


[Small] Enable Java 8 Compatibility

Description
This enables java 8 compatibility.
Review checklist

[x] PR is split into meaningful commits for the ease of reviewing
[x] Description contains clear instructions on how to test the feature (where applicable)
[x] Tests have been written
[x] Appropriate labels have been applied

Codecov Report

Merging #41 (3b44e1b) into main (ab8fe07) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main      #41   +/-   ##
=========================================
  Coverage     100.0%   100.0%
  Complexity       39       39
=========================================
  Files             9        9
  Lines            61       61
  Branches          9        9
=========================================
  Hits             61       61


Compute size of text in advance.

I could not find a way to compute the size of a text in advance, without rendering it on screen.
As far as I know, ImGui has a CalcTextSize function that facilitates this functionality.

Did you not find ImGui.calcTextSize()?

No. Does it exist?
On a similar note, is there a way to

Nope. Quick search returns nothing.

It's a generated file, so it's not in the repo. You'll have to rely on auto compete, but it does indeed exist.

Autocomplete doesn't return anything either.

Odd, I'm fairly certain it should be there. I'll confirm when I get time.
Are you able to Ctrl+Click on any ImGui functions? What are the functions near ImGui.calcTextSize(), (the functions are sorted)

calcListClipping & captureKeyboardFromApp

Ah it returns Vec2, which I don't handle yet, so it is indeed missing. I had begun work on this in the im_vector branch.
In the meantime you can use the cinterop apis to workaround this.


PromptProductPurchaseFinished is no longer deprecated

PromptProductPurchaseFinished had the deprecated tag added then removed but the description in the wiki still mentions it is deprecated
Checks
By submitting your pull request for review, you agree to the following:

[ ] This contribution was created in whole or in part by me, and I have the right to submit it under the terms of this repository's open source licenses.
[ ] I understand and agree that this contribution and a record of it are public, maintained indefinitely, and may be redistributed under the terms of this repository's open source licenses.
[ ] To the best of my knowledge, all proposed changes are accurate.

Hey @Aspecky, this looks really good, thanks for contributing. I need you to check the three boxes before I can merge, though -- I've added them back to your initial comment.

Hey @Aspecky, this looks really good, thanks for contributing. I need you to check the three boxes before I can merge, though -- I've added them back to your initial comment.

Sorry I didnt think they were checkboxes, i've checked them now 😅

No worries at all, we just appreciate the contribution. Thanks again!


fix slack report colors

Slack colours seem to have been broken since f9e582bca3c40727ee7fb86cb6f6c3da26e5fab2 and always return the default colour (blue) for all channels.
The example config shows channels is a list (array in js?)
but colors is a dict.
  - file: reportSlack
    channels:                      <- List []
      - hijack
      - newprefix
      - visibility
      - path
      - misconfiguration
      - rpki
    params:
      showPaths: 0 # Amount of AS_PATHs to report in the alert
      colors:                       <- Dict {}
        hijack: '#d60b1c'
        newprefix: '#fa9548'
        visibility: '#fad648'
        path: '#42cbf5'
        rpki: '#d892f0'
      hooks:
        default: _YOUR_SLACK_WEBHOOK_URL_
        noc: _YOUR_SLACK_WEBHOOK_URL_

https://github.com/nttgin/BGPalerter/blob/8b447d698d70291678f357c29694f99342cfe042/src/reports/reportSlack.js#L53
this line sets channel as the index of the list item, rather than the value of the list item.
after the for loop, the template ends up looking something like this
{
"0":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"1":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"2":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"3":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"4":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"5":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"6":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}",
"default":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}"
}

where the keys in the template are integers, rather than the channel names.
this results in the channel never being found in the template and default being used.

this simple change of in to of will set channel as the value in the list.
{
"hijack":"{\"attachments\":[{\"color\":\"#d60b1c\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}","
newprefix":"{\"attachments\":[{\"color\":\"#fa9548\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}","
visibility":"{\"attachments\":[{\"color\":\"#fad648\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}","
path":"{\"attachments\":[{\"color\":\"#42cbf5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}","
rpki":"{\"attachments\":[{\"color\":\"#d892f0\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}","
default":"{\"attachments\":[{\"color\":\"#4287f5\",\"title\":\"${channel}\",\"text\":\"${summary}\"}]}"
}

i have tested this with the -t test parameter and the results are as follows:

(note the correct colour being displayed for a hijack #d60b1c (red) vs #4287f5 (blue)).

Thank you very much for this catch!


Remove Polymer.GestureEventListeners

The same as https://github.com/vaadin/vaadin-text-field/pull/108

This change is

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

Comments from Reviewable