Make all DateTime inputs timezone-aware

DateTime inputs are not timezone-aware by default. There are two parts to getting this right:

Display the correct timezone in the form field (see 0e9b49c82a9daabe0f4a1581f7f222e0ec7e6845)
Interpret as the correct timezone when converting from form data (see 205607301eb2bc14fba35ab8665daeb024f3eee9)

We've already fixed this for ItemOptions, but we still need to get Discounts and any other DateTime inputs.

Finances view... order- / attendee-related timestamps...


fix add item success msg

Codecov Report
Base: 0.00% // Head: 0.00% // No change to project coverage :thumbsup:

Coverage data is based on head (60e207e) compared to base (d452654).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@          Coverage Diff           @@
##           master     #48   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files          87      87
  Lines        1473    1473
  Branches      146     146
======================================
  Misses       1473    1473

Impacted Files
Coverage Δ

...va/seedu/waddle/logic/commands/AddItemCommand.java
0.00% <0.00%> (ø)

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


NYUv2 Filled depth map

I downloaded NYUv2 from your link. However, I found that the filled depth map is the same as the raw depth map. Is there any problem?
I am new to this area, I wonder if people just calculate metrics using only raw depth maps or dense depth maps.
If raw nyuv2 depth map, everything is okay. If dense nyuv2 depth map, there might be a problem.
Looking forward to your reply. This problem confuses me a lot.

Hi, the NYUv2 dataset I uploaded only contains the raw depth predictions. The evaluation was conducted on these raw depth predictions only, since the dense version was obtained by naive bilateral interpolation.

Hi bingxin, Thanks a lot!


feat: add support for ARCHIVE storage class

Closes #2425
cc @tritone for review.

Looks good to me; will give approval once the feature is no longer behind a flag. Also needs approval from a repo owner.

We are clear to merge this now with a plan to release the change between 1/2 and 1/8. It looks like there are two remaining things now to be resolved:

@dwsupplee , we need repo owner approval from you on this PR.
Looks like the code coverage check is also currently blocking the merge. @jdpedrie could you look into that?

Let me know if either of you see any issue with this timeline.

@tritone just rebased this, should fix the coverage check.

Thanks @dwsupplee. Went back to including the double-quotes, since I think that's more in line with the general formatting we've preferred til now.


Remove database availability hard dependency at boot

Summary
This PR removes the need for a Postgres database availability for the service to start. This hard dependency was related to the database migrations execution, and not to the service startup itself. According to that, only postgres-database.migration.hook.ts was modified.
Changes

Prevent postgres-database.migration.hook.ts from throwing an error that blocks the service startup.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>9
Details

1 of 7 (14.29%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.001%) to 46.92%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/datasources/db/postgres-database.migration.hook.ts
1
7
14.29%

Files with Coverage Reduction
New Missed Lines
%

src/datasources/db/postgres-database.migration.hook.ts
1
30.77%

Totals

Change from base Build<PHONE_NUMBER>0:
0.001%

Covered Lines:
4551

Relevant Lines:
7829

💛  - Coveralls


ControllerTest: use file stream for cache + log dir

@HellPat I tried a solution with cache + logs in memory. 🔥


Assertion errors print poorly

The following code:
class JS {
  final String name;
  const JS([this.name]);
}

void main() {
  print(JS());
}

triggers this runtime error on the console:
Error: Assertion failed: file:///tmp/dartpadKBZCRA/main.dart:4:18

If break in the devtools debugger, it appears that:
(a) the entire error is:
Error: Assertion failed: file:///tmp/dartpadKBZCRA/main.dart:4:18
name != null
"A null value was passed into a non-nullable parameter name"

(b) there is code is dartpad that truncates everything but the first line:
              var msg = error.toString().split("\n")[0];
              console.log(msg);

I'm marking this as a duplicate of https://github.com/dart-lang/dart-pad/issues/1966, where we are tracking issues related to assertion error messages.


Add a feature request

Title of the Pull Request

[x] Have you renamed the PR Title in a meaningful way?

Related Issue

Closes: #1060
Description

I add a feature request
Screenshots 📷

Type of change
What sort of changes have you made:

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

Checklist

[x] My code follows the code style of this project.
[x] I have followed the contribution guidelines
[x] I have performed a self-review of my own code.
[x] I have ensured my changes don't generate any new warnings or errors.
[x] I have updated the documentation (if necessary).
[x] I have resolved all merge conflicts.

Merge it @vishanurag


resnet18 pre-trained model

Could you provide  pre-trained model of resnet18？

We use 10-fold cross validation for the CK+ dataset. There are 10 pre-trained models.

Thanks for your reply，could you share one if possible？ I just want to run the demo.

OK!
https://pan.baidu.com/s/161Rvz25gudQLvXqQwp_QEw
key：bbpg


fix(csharp): StackProps are renamed when no stack props are present

We should only be renaming StackProps when there are props present. If none are, we just won't add the structure or rename the stack props.
This is also how Java is structured.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Should have provided more context on my pr https://github.com/cdklabs/cdk-from-cfn/pull/279/files. Can you explain how you plan to handle the stack props in the CDK CLI?

Pulled down the PR. Creates the previous error:
/Users/bobertzh/work/deleteme/temp/myTemp/src/MyTemp/Program.cs(13,44): error CS1503: Argument 3: cannot convert from 'Amazon.CDK.StackProps' to 'MyTempStack.MyTempStackProps' [/Users/bobertzh/work/deleteme/temp/myTemp/src/MyTemp/MyTemp.csproj]

If we are going to implement java's prop overrides instead of using an empty props class can we do that here as well?

Yes, I did the half here that would fix the use case we're testing against in the CLI integ tests to unblock upgrading on that end and then was working on doing the other half as a fast follow up.

I am going to take a harder line and say this is also not consistent with CSharp documentation https://docs.aws.amazon.com/cdk/v2/guide/stack_how_to_create_multiple_stacks.html#cdk-how-to-create-multiple-stacks-extend-stackprops


Issue 435 display sensor charts on asset page

This PR puts sensor data to the forefront of the asset page. The form for editing the asset (which used to be front and center) has been moved to a new sidepanel.

Right now, 2 sensors are shown by default (as seen above), but it is possible to customize which sensors are shown by defining a generic asset attribute (which isn't supported yet by API or UI at this point):
{
    "show_sensors": [3, 11, 4, 5]
}

In this example, sensors 3, 11, 4 and 5 would be shown from top to bottom, in the given order.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

41 of 107 (38.32%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.4%) to 67.965%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

flexmeasures/api/dev/sensors.py
8
10
80.0%

flexmeasures/api/v3_0/assets.py
13
19
68.42%

flexmeasures/data/models/charts/belief_charts.py
2
14
14.29%

flexmeasures/data/models/generic_assets.py
13
59
22.03%

Totals

Change from base Build<PHONE_NUMBER>:
-0.4%

Covered Lines:
7127

Relevant Lines:
9938

💛  - Coveralls

This is great!
Let's create a follow-up ticket where one can alter the plotted sensors
on the asset page as well, by dragging the sensor names into a list.
Solving the UI challenge here is the difficult part, but I feel that
there might be compontens for this in bootstrap or a bootstrap plugin.

There are more ways to make that possible, and I'd prefer another way than a dragging interaction, which would indeed be a challenge to implement such that it works nicely across devices. Some other solutions:

Adding a "Show" column to the sensor list with checkboxes.
Adding a selection list to each chart, from which to choose which sensors to display, plus buttons to remove or add a plot.
An additional form field allowing to set a comma separated list of sensor ids. This doesn't really need to be that UXy, in my opinion it's something one would do only once when setting up the asset.

Additional considerations:

It is useful to be able to show relevant public sensors (that belong to a different asset).

Only a few of these support choosing the order of plots. It might be that the selection list is easier than drag and drop. It depends on available Bootstrap widgets.
I agree it would be very cool to be able to add public sensors.
Am 23. Juni 2022 19:58:48 MESZ schrieb Felix Claessen @.***>:

There are more ways to make that possible, and I'd prefer another way than a dragging interaction, which would indeed be a challenge to implement such that it works nicely across devices. Some other solutions:

Adding a "Show" column to the sensor list with checkboxes.
Adding a selection list to each chart, from which to choose which sensors to display, plus buttons to remove or add a plot.
An additional form field allowing to set a comma separated list of sensor ids. This doesn't really need to be that UXy, in my opinion it's something one would do only once when setting up the asset.

Additional considerations:

It is useful to be able to show relevant public sensors (that belong to a different asset).

--
Reply to this email directly or view it on GitHub:
https://github.com/FlexMeasures/flexmeasures/pull/449#issuecomment-1164709228
You are receiving this because you commented.
Message ID: @.***>
-- Gesendet von /e/ Mail.

Browsing http://localhost:5000/assets/3/ leads to a spinner and no results. The 500 error is surprising, not sure what to make of it [1]. It's the call to /api/dev/assets/3 which errors. If I select a date, it works, and it's calling /api/v_3?assets/3/chart_data

Have my latest commits resolved this issue? I could reproduce the spinner for an asset without sensors, but that exact 500 error I couldn't reproduce.

As a user, I find it a bit confusing to select a larger date range, and then getting a data view with data, which is shorter than what I wanted (only one day of data available in the toy data), but no indication which day(s) I'm now looking at. The date selector still shows my original date selection, and the plot shows no date.

I agree that this is confusing behaviour. It was already there before this PR, so maybe I'll open a separate issue? I had also expected the plot to show a date, but it doesn't because the toy schedule that is created falls entirely within a single day, and the x-axis only shows date labels at midnight.

I can test this tomorrow evening or thursday, hopefully.

I still have the eternal spinner when asset 3 (from toy tutorial) is loaded on a date range with no data.
I looked into the 500 error locally. If I uncomment @with_appcontext_if_needed() on GenericAssetField, it goes away, but the investigation was difficult afterwards. Some weird way calls click's get_current_context without silent=True. Something for later I guess.

Can you verify that running python run_local.py instead of flexmeasures run "resolves" the issue? Hopefully that takes us one step closer to getting to the underlying problem.

Yes, python run-local.py does not have this problem. No spinner staying, no 500.
I don't know what the difference really is. Or why this API call runs into problems. I found a point to start investigating, though. Us mixing click into web serving code might be a bit unconventional.

Apparently the with_appcontext_if_needed decorator finds an active click context when flexmeasures run triggers that decorator on the deserialize method, but no active click context is found anymore when the server receives requests. When python run_local.py is used, no active click context is found at the time the decorator on the deserialize method is triggered.
A possible fix might be to change this line as follows:
- if get_current_context(silent=True):
+ if get_current_context(silent=True) and not ctx.invoked_subcommand == "run":

Ah yes I keep forgetting to consider when a decorator is called.
Very good next step of investigation.
Your suggested fix would need at least a comment because we've not addressed the root.
Is ctx available when there is no context?

One small problem: On the asset page, when viewing a plot with partial data, if I move the cursor into non-data territory, the last data tooltip shown will keep being visible. The sensor page does not seem to have this.

Can you elaborate on the problem? Do you think that the last data tooltip should not be visible in that case, or rather that the sensor page should get the same tooltip behaviour? FYI, underlying the datapoint selection for the tooltip is a hidden voronoi that extends to the sides of the chart area.

If I'm not hovering over data, I should not see a data tooltip anymore. So I feel the sensor page has it correct.

I recall it was possible in other projects we did to zoom in by selecting a range directly on the Altair plots. Is the plan that we'll get that possibility here, as well?

I haven't planned it as a FlexMeasures feature right now.

If I'm not hovering over data, I should not see a data tooltip anymore. So I feel the sensor page has it correct.

I disagree. The ability to select the nearest data point this way was advised to me by a UX designer. I agreed with that advice, because it can be quite cumbersome to select individual data points. The sensor page currently shows bar graphs, which makes that easier, except for bars showing values near zero.
A weaker version of your statement I would consider, though: "If I'm not hovering over a period with data, I should not see a data tooltip anymore."
The ability to limit the voronoi to something other than the edges of the chart does not exist right now, however. I think it would require a front-end developer to make that work.

I can't manage to record my screen properly right now, but I believe we might actually already agree.
In my example, I select a month (I wasn't sure which days actually had data), and so my result has only a few days with data, the rest of the days are blank. If my pointer moves sideways from days with data to days without data, the tooltip stays. Even if I'm two weeks earlier in the blank zone, even if I then leave the plot to the top and come back.

A weaker version of your statement I would consider, though: "If I'm not hovering over a period with data, I should not see a data tooltip anymore."

I think this matches my grief.

Alright, then I understand the grief now. Personally I like the current behaviour, so I can quickly get to see from when the last data point was. But in any case, I can't change it.
Another issue I encountered is that the date selection resets to the most recent available data when you load an asset or sensor page. I was thinking of adding the date selection to the session, so that the same period will load when a user switches to a different page. Would that be better than the current behaviour?
Anything else for this PR?

I understand you get some value out of this UX as a power user, but first-time users will be quite confused, I feel.

I can't change it.

I thought the weaker version of my statement is in the realms of possible fixable?
As for the date range selection in the session, I like a selected range to persist across these data plot pages. We have start_time and end_time in the session variables (see ui/utils/view_utils.py::render_flexmeasures_template) - maybe that is where this should be going?
If tht is a lot of work, we can also make a new issue for that.

I thought the weaker version of my statement is in the realms of possible fixable?

I managed to create another invisible layer myself, as I imagined it might be possible to create data-encoded bars that extend from bottom to top, and assign the tooltip to that.

As for the date range selection in the session, I like a selected range to persist across these data plot pages. We have start_time and end_time in the session variables (see ui/utils/view_utils.py::render_flexmeasures_template and set_time_range_for_session) - maybe that is where this should be going? If that is a lot of work, we can also make a new issue for that.

I looked into it, and it seems like work for a separate PR. We don't want it to be necessary to reload the page in order to set a session variable, so I was thinking we could have relevant API endpoints (like /chart or /chart_data), or a new dedicated API endpoint (called when selecting a calendar period), to set the session variable, instead of doing that when the server is called to load the page.

I think you are right about the session date range deserving its own PR (make a new issue for now?).
I believe the /chart_data endpoint is a good place.


Fixed issue in patching config file to update repo. If the repo was o…

…n the
last line and the file had no new line at the end, the repo would become psij-pytho.

This looks correct to me. Another approach might be to sanitize input with line = line.rstrip('\n') and then do the replace.

Yes, that sounds much better.


G-CLI 3.0.0-beta1 prefers 64 bit version

Needs more testing but saw on a local run that despite having 32 bit and 64 bit installed and detected and no -x64 flag set.
Confirm if this is the case - also consider whether we should have a more specific override flag which selects one or the other as 64 bit gets more popular

The problem is if you don't provide --lv-ver it will just load a default, ignoring the bit flag. Or if you provide one it cant find

Confirmed the old behaviour was at https://github.com/JamesMc86/G-CLI/blob/20706b209ab61fdc0c2619aca3792f18c351a733/C Sharp Source/LabVIEW CLI/Program.cs#L235
Basically if no --lv-ver then just get the "CurrentVersion" which is the last version launched. This also ignores the bit flag so the behaviour is consistent but I still find it a little odd.


Loading owned object fails

Lazy Loading owned object and Entry(entity).Reference(e => e.OwnedObject).LoadAsync() are failed.  Only dbContext.DbSet.Include(e => e.OwnedObject).FirstOrDefaultAsync() works.
BTW, they fail on  EF Core version: 3.0.0-preview6.19304.10 but work on EntityFramerworkCore 3.0.0-preview5.19227.1.
Exception message:
Cannot create a DbSet for 'UserProfile' because it is configured as an owned entity type and should be accessed through the defining entities.
Stack trace:
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.get_EntityType()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.get_EntityQueryable()
   at Microsoft.EntityFrameworkCore.Internal.InternalDbSet`1.System.Linq.IQueryable.get_Provider()
   at System.Linq.Queryable.Where[TSource](IQueryable`1 source, Expression`1 predicate)
   at Microsoft.EntityFrameworkCore.Internal.EntityFinder`1.Query(INavigation navigation, Object[] keyValues)
   at Microsoft.EntityFrameworkCore.Internal.EntityFinder`1.Load(INavigation navigation, InternalEntityEntry entry)
   at Microsoft.EntityFrameworkCore.Internal.LazyLoader.Load(Object entity, String navigationName)
   at Microsoft.EntityFrameworkCore.Proxies.Internal.LazyLoadingInterceptor.Intercept(IInvocation invocation)
   at Castle.DynamicProxy.AbstractInvocation.Proceed()
   at Castle.Proxies.UserProxy.get_UserProfile()

Further technical details
EF Core version: 3.0.0-preview6.19304.10
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system:
IDE: (e.g. Visual Studio 2019 16.1.3)

I ran into the same issue and am waiting for the resolution (hopefully soon).
While at it, I think it might be useful to remove the current requirement (which I think is a bug) to mark Owned type properties as virtual if Lazy Loading is enabled. I think the bug might even be related to treating Owned properties as Nav properties when lazy loading is enabled.
Just my 2 cents...

Just wondering if this issue has been fixed. It seems to be still open.

@ibasin This hasn't been fixed yet and we currently can't provide an estimate as to when it will be.

Related to https://github.com/aspnet/EntityFrameworkCore/issues/16186

@AndriySvyryd It works with "EF Core 3.0.0-rc1.19456.14" .  But Entry.Reload doesn't reload owned objects #16186 .

Good to hear

Thank you!


Version incompatibility

In v0.3.0 it was possible to call the async del method without a callback, in v0.3.1 this throws an error due to the callback being undefined.
I can provide a pull request if desired.

I don't think it was intentional for the async method to be used without a callback. This was a side effect of passing the callback directly into async.each. I think this was a bug that was fixed since the documented example in the readme shows passing in a callback.
I don't think we should allow not passing in the callback since this can lead to more errors and hard to track down bugs. But to keep from breaking code written without the callback, we should probably bump to 0.4.0.

Makes sense, +1 to a minor version bump.

Ok, so I'll need to add some code to emulate 0.3.0, publish to 0.3.2 then bump to 0.4.0. How about if I do this in the del function for the 0.3.2 code:

function del(patterns, options, cb) {
  if (typeof options === 'function') {
    cb = options;
    options = {};
  }
  // if no callback is given, return the sync method
  if (typeof cb !== 'function') {
    return del.sync.apply(del, arguments);
  }

  // ...
}

@jonschlinkert yea looks good


Adding namespace scope to webhook config

This PR explicitly defines the rule scope for the admission-webhook configurations.
Since the rules are only defining one resource type per rule, I chose to explicitly define the scope instead of providing variables that would allow customization.
This fixes #939
Thanks in advance!

Not sure this change would impact operator's Cross namespace instrumentation feature.

@povilasv I thought already had, at least running it on my branch doesn't produce any changes.

Not sure this change would impact operator's Cross namespace instrumentation feature. Setting namespaceselector is enough?

I tested the cross-namespace instrumentation in real-world tests, and it workes well.
tested with yaml:

Instrumentation CR:

apiVersion: opentelemetry.io/v1alpha1
kind: Instrumentation
metadata:
  name: opentelemetry-autoinstrumentation
  namespace: scope-ns-test
spec:
  resource:
    addK8sUIDAttributes: true
  sampler:
    # Enum: always_on, always_off, traceidratio, parentbased_always_on, parentbased_always_off, parentbased_traceidratio, jaeger_remote, xray
    type: always_on

demo app :

apiVersion: apps/v1
kind: Deployment
metadata:
  name: my-deployment-with-aotu-instrumentation
spec:
  selector:
    matchLabels:
      app.kubernetes.io/name: my-deployment-with-aotu-instrumentation-kuberntes
  replicas: 1
  template:
    metadata:
      labels:
        #app: my-deployment-with-aotu-instrumentation-app
        app.kubernetes.io/name: my-deployment-with-aotu-instrumentation-kuberntes
      annotations:
        #sidecar.opentelemetry.io/inject: "false"
        ##instrumentation.opentelemetry.io/inject-sdk: "default/insight-opentelemetry-autoinstrumentation"
        instrumentation.opentelemetry.io/inject-java: "scope-ns-test/opentelemetry-autoinstrumentation"

    spec:
      serviceAccountName: instrumentation-app
      automountServiceAccountToken: false
      securityContext:
        runAsUser: 1000
        runAsGroup: 3000
        fsGroup: 2000
      containers:
      - name: myapp
        image: ghcr.m.daocloud.io/open-telemetry/opentelemetry-java-instrumentation/smoke-test-spring-boot:jdk11-20211213.1570880324
        #image: docker.m.daocloud.io/library/nginx:1.23.3
        ports:
          - name: http
            containerPort: 8080

and java auto-instrumentation injected in init-container.
So, LGTM

@b3ndill I merged another operator PR first, so now we need to bump the patch number again and regenerate the examples.   Once you do that I'll make sure this is merged next.

@TylerHelmuth Done, thanks for letting me know.


feat: move to blob_sidecars_by_range/root methods

Continuing

https://github.com/ChainSafe/lodestar/pull/5181
integration

this PR moves the req/resp methods to the decoupled blobs spec

:tada: This PR is included in v1.9.0 :tada:


fix: opt physics code

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

Codecov Report

Merging #519 (3d45dcb) into dev/0.6 (fa92e1c) will increase coverage by 0.15%.
 Consider uploading reports for the commit f6467c9 to get more accurate results

@@             Coverage Diff             @@
##           dev/0.6     #519      +/-   ##
===========================================
+ Coverage    46.22%   46.37%   +0.15%
===========================================
  Files          379      379
  Lines        19689    19621      -68
  Branches      2602     2589      -13
===========================================
- Hits          9101     9100       -1
+ Misses       10588    10521      -67

Impacted Files
Coverage Δ

packages/core/src/physics/Collider.ts
11.86% <0.00%> (-1.35%)
:arrow_down:

packages/physics-physx/src/PhysXCollider.ts
0.00% <ø> (ø)

packages/draco/src/decoder.ts
0.00% <0.00%> (-0.92%)
:arrow_down:

packages/loader/src/scene-loader/Oasis.ts
0.00% <0.00%> (ø)

...ackages/loader/src/scene-loader/ResourceManager.ts
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


gosund smart led light strip

Hello,
is it possible to flash gosund smart led light strip?
To get light working i have to registrade it in gosund cloud.

install gosund app
register gosund account
type in wifi credentials for gosund device
gosund app magically transmit wifi credentials over wifi to gosund led stripe

There is no accespoint opend by the led stripe
Here some pics of PCB maybe you have seen it in other devices

Can always manually flash ESP8266 chip by soldering to pins, though I have not had luck using tuya-convert OR manual serial flash on gosund or treatlife devices, only success has been sonoff so far.


Construct collector by java programming, no content returns

Hi, I am trying to constructing collector and crawler by java programming, everything looks good, except the problem with missing content. something is missing ?
public static void main(String[] args) {
        HttpCollectorConfig collectorConfig = new HttpCollectorConfig();
        final String domain = "www.jszwfw.gov.cn";
        collectorConfig.setId(domain);
        collectorConfig.setProgressDir("./progress/" + domain);// ./progress/+domainName

        List<HttpCrawlerConfig> httpCrawlerConfigs = new ArrayList<>();
        httpCrawlerConfigs.add(buildCrawlerConfig(domain, "http://www.jszwfw.gov.cn/col/col28/index.html", "^http://www\\.jszwfw\\.gov\\.cn/art.*"));

        collectorConfig.setCrawlerConfigs(httpCrawlerConfigs.toArray(new HttpCrawlerConfig[httpCrawlerConfigs.size()]));
        HttpCollector collector = new HttpCollector(collectorConfig);

        collector.start(true);
    }

    private static HttpCrawlerConfig buildCrawlerConfig(String domain, String startUrl, String targetRegex) {
        HttpCrawlerConfig httpCrawlerConfig = new HttpCrawlerConfig();
        httpCrawlerConfig.setId(startUrl.substring(7, startUrl.length()));
        httpCrawlerConfig.setStartURLs(startUrl);
        httpCrawlerConfig.setWorkDir(new File("./work/" + domain));

        RegexReferenceFilter regexReferenceFilter = new RegexReferenceFilter();
        regexReferenceFilter.setCaseSensitive(false);
        regexReferenceFilter.setRegex("(" + Pattern.quote(startUrl) + ")" + "|" + "(" + targetRegex + ")");
        regexReferenceFilter.setOnMatch(OnMatch.INCLUDE);
        httpCrawlerConfig.setReferenceFilters(regexReferenceFilter);

        ImporterConfig importerConfig = new ImporterConfig();

        httpCrawlerConfig.setImporterConfig(importerConfig);

        ElasticsearchCommitter committer = new ElasticsearchCommitter();
        committer.setIndexName("opportunities");
        committer.setTypeName("document");
        committer.setClusterName("elasticsearch");
        httpCrawlerConfig.setCommitter(committer);

        httpCrawlerConfig.setMaxDepth(1);
        httpCrawlerConfig.setIgnoreRobotsMeta(true);
        httpCrawlerConfig.setIgnoreRobotsTxt(true);
        httpCrawlerConfig.setIgnoreSitemap(true);

        return httpCrawlerConfig;
}

What do you mean by "missing content"?  Nothing gets crawled?  Do you have any error in your logs?  Can you attach a copy of your log?
I tried running your code with the FileSystemCommitter instead and documents were crawled properly.  Maybe it is an issue specific to Elasticsearch? Which version of Elasticsearch are you using?

I am using Elasticsearch1.5, I meant the content meta was missing from committed documents.

I did not set target content field, so content field was missing.
committer.setTargetContentField("content");


Unpacking Epics data

In TUnpackingLoop.cxx, in the function ProcessMidasEvent, case 5 is commented out. This case is for unpacking the Epics data.
In the future, this should be re-included. I'm not sure why it was commented out and I'm hesitant to do such un-commenting on the current experimental account without that knowledge.

I think this is done with PR #815.


Move version.txt inside spinalcordtoolbox package

Context: https://github.com/neuropoly/spinalcordtoolbox/pull/2814#discussion_r466113927
In spinalcordtoolbox.utils, the version is fetched using os.path.join(__sct_dir__, 'version.txt'), which, for a pip-installed package, is pointing to envs/sct-env/lib/python3.6/site-packages/version.txt, but the file does not exist there.
This causes the RTD script function to not show the usage:

The solution implemented in this PR is to:

move version.txt inside spinalcordtoolbox/, so it is automatically copied with pip install
edit spinalcordtoolbox.utils to point to this new location

Partly addresses https://github.com/neuropoly/spinalcordtoolbox/issues/1526#issuecomment-411509189
Fixes #2832

@joshuacwnewton i guess we can close this PR as https://github.com/neuropoly/spinalcordtoolbox/pull/2837 does include my changes-- right?

@jcohenadad I was planning on updating #2837 when this one gets merged, just to keep things organized.

@kousu travis-windows fails: https://travis-ci.org/github/neuropoly/spinalcordtoolbox/jobs/716982434
do you think it is related to changes in this PR?

@jcohenadad I don't believe so.
It looks like there was a change on Microsoft's end. The install script Chocolatey package (wsl-ubuntu-1804) fetches WSL as an .appx file from Microsoft at this link: https://aka.ms/wsl-ubuntu-1804 (More information here.)
On the previous successful test, the downloaded .appx file was CanonicalGroupLimited.Ubuntu18.04onWindows_1804.2018.817.0_x64__79rhkp1fndgsc.Appx. With this new failed test, the .appx file is now Ubuntu_1804.2019.522.0_x64.appx. So, it looks like MS updated the version of WSL they distribute, and that caused the Chocolatey package to fail.
More information about this error can be found here: https://www.gep13.co.uk/blog/chocolatey-error-hashes-do-not-match. It recommends:

The best way to fix this problem would be to reach out to the maintainers of the package and ask them to push a new package version that includes the correct checksum. In the case of this package, there is actually a new version of the application available, so this package is due to be updated. If there wasn't a new version available, then the maintainer could push a new package version using what is known as the package fix notation.


Show project name in header of intellij

As an IDEasy user, I want intellij to display my project name in the header like we currently do for eclipse.
Currently intellij shows the workspace name only.

currently unclear if possible at all. IF we can pass a title via CLI this will be easy to implement.


refactored validate-signup, changed code climate settings

Description
A refactor of some middleware and codeclimate settings.
Fixes # (issue)
Hopefully improves codeclimate maintainablity score.

Refactor only lowered cognitive score to 10.

Passes maintainability locally, all existing tests passing.


PLAT-62228: Set a 5way key mode when pressing a key to focus an item

Issue Resolved / Feature Added
When pressing a page up or down key over the item in a VirtualList with a pointer mode, VirtualList could not move focus properly.
Resolution
VirtualList called Spotlight.focus(). But the VirtualList could not move focus because it was in a pointer mode.
As we did in a VirtualList with the #1778, I forced to call Spotlight.setPointerMode(false)  when pressing a key.
Additional Considerations
When pressing a 4 direction key over the page control in a VirtualList with a pointer mode to move another page control, VirtualList could not move focus properly. So I fixed it in a same way.
Links
PLAT-62228
Comments
Enact-DCO-1.0-Signed-off-by: YB Sung<EMAIL_ADDRESS>

LGTM


Allow requesting money

Once #3 is implemented, a page can be made (Or repurpose Receive) to request a payment showing an invoice qr code

Implemented in ee23b39a7df60e645279c694c0b3a4adc8f4e497


gitbook not found

gitbook link is died (by https://github.com/idevz/vanilla#more-information)

you can use https://github.com/idevz/vanilla-zh


refs #12 upgrade to AngularJS v1.4.8

Upgraded AngularJS to v1.4.8, as referred in issue#4 and issue#12.
Changelog:

Updated Angular.js
Added angular-route and related settings.
Default placeurl in route parameters is set to 'atm' in ZipCodeFrmCtrl as updated angular-route won't match against /search/:zipcode/:place when :place is '' .
Removed default handling in SearchCtrl as it is already handled in ZipCodeFrmCtrl.
The syntax in use right now for Dependency Injection (DI) will cause error in production environment (when minification is used), hence using the updated dependency Injection syntax for angular modules. eg: here
Updated karma configuration file.

Thanks @anuja-joshi . I'll go through the changes in the evening and will merge if everything is good.


Access control

There needs to be a permissions model. :)

[ ] Ensure each object is owned by a group
[ ] Every user is a member of one or more groups
[ ] Groups should be groupname+[add,change,delete,view]?
[ ] Access maps to groupname + permission level

Another option, especially if we integrate with LDAP or similar... At this point we'll rely on group-based access, which suggest a model where each object is owned by a group and we can (simply) check group memberships for access.
This would probably suggest dropping Guardian as there is no need for individual per-object permissions, as permissions can be extracted from the owning group.
To revisit the model suggested above:

[ ] Ensure each object is owned by a group
[ ] Every user is a member of one or more groups
[ ] Access maps to groups
[ ] There is the option to extend this model with a role map so we can have multiple groups acting on objects, where their roles (read, write, delete, create(?)) differ.
[ ] Does everyone have read access to everything? Probably? Maybe?

Fixed in #28.
More tests are needed, and some models are exempt, but otherwise good.


WIP : Cni install change

Just testing initially - don't merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: john-a-joyce
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sebastienvas
If they are not already assigned, you can assign the PR to them by writing /assign @sebastienvas in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

prow/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@john-a-joyce: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/istio-integ-k8s-tests.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/istio-integ-k8s-tests.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/istio-integ-k8s-tests.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-simpleTests-cni

/test e2e-simpleTests-cni

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/istio-integ-k8s-tests.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/istio-integ-k8s-tests.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test e2e-simpleTests-cni

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test release-test

prow/e2e_pilotv2_auth_sds.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-multicluster-e2e.sh
849ddaab1635c668dd82dcdb27d7b7438bd7db02
link
/test istio-pilot-multicluster-e2e

prow/e2e-simpleTests-cni.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio_auth_sds_e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@john-a-joyce: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests-cni.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test e2e-simpleTests-cni

prow/istio-integ-k8s-tests.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio-integ-k8s-tests

prow/e2e_pilotv2_auth_sds.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio_auth_sds_e2e

prow/istio-pilot-multicluster-e2e.sh
73bf1a2e5b587c36a46085eae6de2ea202690603
link
/test istio-pilot-multicluster-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

This was a PR to do some gate testing.  It was superseded by another PR for merge purposes


Cannot exit until 2048 epochs (#675 item 21) [redo]

See #675 item 21
(Supersedes #736)

[transfered comment from #736]
This is supposed to "(i) ensure stability of persistent committees" but there are other ways a validator might be exited -- slashing and balance ejections. Not sure if the balance ejection could happen in the time frame, but a slashing certainly could.
Wanted to make sure the intention is to just provide more stability but not to prevent quick exits all together.
Also, we should probably just rebuild this PR from scratch against current dev. The merge conflict is nasty for such a simple PR

Wanted to make sure the intention is to just provide more stability but not to prevent quick exits all together.

Correct.

**
**


Decodoku

Summary
The Decodoku class has been moved from an external repo to qiskit_qec.codes. This implements abstract versions of qubit and qutrit codes and can present them interactively using Jupyter notebooks. The required tools using Jupyter widgets have been moved to qiskit_qec.utils.visualizations.
Details and comments
ipython and ipywidgets have been added to the requirements.

I moved everything to utils. This is the version I'll use for the live stream on Monday (whether it eventually gets merged or not).


Add wrapper for Android.

I also started a README and added a LICENSE file.
/cc @chris-erickson

Neat.  Now I gotta figure out why I couldn't get it to work in Xcode. :bar_chart:


ebot_description not found error

Whenever I run the part:
ros2 launch ebot_description ebot_gazebo_launch.py
The terminal gives the error saying:
Package 'ebot_description' not found: "package 'ebot_description' not found, searching: ['/opt/ros/humble']"
Tough I have the entire source file downloaded . It keeps saying this. I haven't pulled yet

Yes, I reinstalled everything and it is fixed now. Thank you


Add string pattern found count to find/replace tools.

When I refactor my code using the find and replace tools, it would be extremely helpful to know how many instances of the search string the tool has found.
Sometimes I need to create const string references in a huge source file and there is no point in making them if the string is used only once, so I create the constant strings only if there are multiple instances of the string used.
I could use the "up" or "down" arrow to check if the editor jumps to another instance if there exists one, but that's not an elegant solution...

Duplicate of #14513?

Indeed, closing as a duplicate of the above-mentioned #14513.


Variable explorer still auto-refreshes even if disabled

Hi,
this issue is similar to #1958 .
I have disabled Spyder's autorefresh feature but the variable explorer still refreshes automatically. The IDE is unusable when I have series of loops and complicated nested data structures because the variable explorer freezes the execution of my code frequently, causing it to take much longer to complete than when executed on an ipython console.
I'm using Spyder <IP_ADDRESS> distributed via conda.
Please let me know if you need more information.

Refreshing should only occur at the end of a given computation. The auto-refresh option means that refreshing is done at a certain amount of time, but that's disabled for IPython consoles.
So, please post a reproducible example to see what's happening in this case :-)

Thanks for the quick reply.
First, I have disabled the autorefresh by unticking the checkbox. When I run the following simple script (F5/runfile(...)), I can see in the variable explorer the value of n being updated.
In particular:

If I have the refresh interval set to 4000ms, I see the value being incremented in steps of four (with approx. +-1 variations)
If I have the refresh interval set to 5000ms, I see the value being incremented in steps of five (with approx. +-1 variations)
etc

I don't think this is intended behavior of the autorefresh feature and, furthermore, it can be problematic with complicated variables in real scenarios.
test script:
import time
for n in range(100):
time.sleep(1)

I'm not able to reproduce your bug when both the variable explorer and the IPython console are focused, and I wait until the computation is over.
Please be sure that the Auto-refresh button is not pressed in the Variable Explorer either, i.e. this button:

There are two situations when the variable explorer gets refreshed (but no auto-refreshed):

If you leave focus from the Variable Explorer to see other panes (e.g. the Object Inspector) and then come back to it again.
If you focus a window different than Spyder (e.g. Firefox) and then come back to Spyder again, and the Variable Explorer is focused.

I've looked at this issue again and it seems like the variable editor starts acting funny after restarting the ipython console kernel.
If it is not too much trouble, please watch this screen recording https://www.youtube.com/watch?v=VKRvX8C3IgE
This video shows how:

it behaves correctly right after starting spyder, but for some reason the auto-refresh button is disabled
after restarting the console kernel the button is not pressed, but it auto-refreshes
then if I press and unpress the button, auto-refresh is disabled as expected
it does a single refresh of the variable explorer if I focus the console even if auto-refresh is disabled.(perhaps this is intended behavior, if I understood you correctly)

@geancarlo, thanks for the video and for narrowing down this problem. Now I can reproduce it on my side and I know how to fix it.
This will part of 2.3.6, to be released next week.


update fixes for cloud shell and formatting

Move copyright to README.md . Fix terminology clarifying Docker Hub vs Store (redirects). Fix markdown formatting in ordered lists. Test with Cloud Shell

@jzaccone would be cool if you could look over and merge. Not urgent.

LGTM


Add support for DOODS Image Processing

Description: This change adds support for DOODS Object Detection service. It's basically an easy way to run Tensorflow remotely.
Pull request with documentation for home-assistant.io (if applicable): home-assistant/home-assistant.io/pull/10228
Example entry for configuration.yaml (if applicable):
image_processing:
  - platform: doods
    scan_interval: 1000
    url: "http://<my docker host>:8080"
    detector: default
    file_out:
      - "/tmp/{{ camera_entity.split('.')[1] }}_latest.jpg"
    source:
      - entity_id: camera.front_yard
    confidence: 50
    labels:
      - name: person
        confidence: 40
        area:
          # Exclude top 10% of image
          top: 0.1
          # Exclude right 15% of image
          right: 0.85
      - car
      - truck

Checklist:

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

If user exposed functionality or configuration variables are added/changed:

[X] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[X]The manifest file_]manifest-docs has all fields filled out correctly. Update and include derived files by running python3 -m script.hassfest.
[X] New or updated dependencies have been added to requirements_all.txt by running python3 -m script.gen_requirements_all.
[X] Untested files have been added to .coveragerc.

If the code does not interact with devices:

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

It does not allow to use requests directly. You need to create a small wrapper library.

@pvizeli can you give me an example of a library that implements a wrapper for requests? I'm not great at python and would prefer to copy one. I looked at a handful and it appears that everyone uses requests directly.

@snowzach now you need address all CI issues.

@MartinHjelmare or @pvizeli do I need to do anything else to get this accepted? I may have messed up the process. I committed the changes by hand vs accepting yours because I ended up fixing another problem.

@pvizeli are you able to complete your review?

Hi @pvizeli just checking in if you are able to review this? Sorry if there's a formal process for this. I'm not familiar.

@MartinHjelmare are you able to keep this moving since I address @pvizeli 's request to wrap server interaction in a library?

Ok. We have a whole release cycle to address things if needed.

I think we did not allow in past to write out images. I think we should enable that as entity picture and use it on component itself, because it work for all platform all time in same way. Maybe @snowzach have the knowlage to address this for all image processing platforms?


JWT Token not found

Hi,
When I call /api/token/refresh, the response is 401.
The request:
curl -X POST \   http://<IP_ADDRESS>:8000/api/token/refresh \   -H 'Content-Type: application/x-www-form-urlencoded' \   -H 'Postman-Token: 484cd44f-3646-4052-b9d4-2a9ce1c4fb0b' \   -H 'cache-control: no-cache' \   -d 'refresh_token=xxxRefreshTokenxxx'
composer.json:
"require": {         "php": ">=7.2.0",         "ext-json": "*",         "doctrine/doctrine-bundle": "^1.6",         "doctrine/doctrine-fixtures-bundle": "^3.0",         "doctrine/doctrine-migrations-bundle": "^1.3",         "doctrine/orm": "^2.5",         "friendsofsymfony/rest-bundle": "^2.4",         "fzaninotto/faker": "^1.8",         "gesdinet/jwt-refresh-token-bundle": "^0.5.4",         "incenteev/composer-parameter-handler": "^2.0",         "kriswallsmith/buzz": "0.17.1",         "lexik/jwt-authentication-bundle": "^2.6",         "nelmio/cors-bundle": "^1.5",         "sensio/distribution-bundle": "^5.0.19",         "sensio/framework-extra-bundle": "^5.0.0",         "sonata-project/admin-bundle": "^3.42",         "sonata-project/doctrine-orm-admin-bundle": "^3.6",         "symfony/monolog-bundle": "^3.1.0",         "symfony/polyfill-apcu": "^1.0",         "symfony/requirements-checker": "^1.1",         "symfony/swiftmailer-bundle": "^2.6.4",         "symfony/symfony": "3.4.*",         "twig/twig": "^1.0||^2.0"     }
could you help me?,

@cristhianDt did you resolve your issue?

Same too me. How do you resolve this issue ?

In my case, it happens because wrong pattern on security.yaml
i wrote:
pattern:  ^token/refresh
should be:
pattern:  ^/token/refresh

Hello everyone!
In my case everything fixes after putting token-refresh firewall rule before api rule.
`
    token-refresh:
        pattern:  ^/api/token/refresh
        stateless: true
        anonymous: true

    api:
        pattern:   ^/api
        stateless: true
        guard:
            authenticators:
                - lexik_jwt_authentication.jwt_token_authenticator

`


swift 5 support

What's the plan to support Swift 5.0
New Issue Checklist

[x] I have read and understood the CONTRIBUTING guide
[x] I have read the Documentation or CocoaDocs
[x] I have searched for a similar issue in the project and found none

Issue Info

Info
Value

Platform Name
ios

Platform Version
12.0

CocoaLumberjack Version
3.4.2

Integration Method
cocoapods

Xcode Version
Xcode 10.2 beta

Repro rate
all the time (100%)

Swift version
5.0

Issue Description and Steps
Please fill in the detailed description of the issue (full textual output of any stack trace, compiler error, ...) and the steps to reproduce the issue.

Fixed by #1044

The above commit did not cover DDAssert.swift, which has similar compiler errors when compiling on Xcode 10.2.


Create Oracle Scripts

Create the Oracle create and DefaultData scripts for Marketplace.

The Oracle scripts can be found in Pull Request #4 at

All database scripts have been completed and are part of v7.17.0-BETA-2.


Get status from script

Hello!
There is  state = get_state() from Environment API,But I looked up the play_game.py script and I didn't find the usage of the environment API, was it in another script?
Are there any relevant documents or tutorials?
For EXAMPLE:
Is there a minimal  easy script:

Choose reinforcement learning algorithm

Start the engine

Get the list and description of environment variables, such as player's position, status and all other variables that can be obtained?

THANK YOU!

get_state / set_state is meant to save/restore environment state. State is opaque and immutable. Simple use case is when you want to verify outcomes of different actions given current environment state. You can get_state(), do a sequence of actions, set_state() to restore to the previous state and do a different sequence of actions.
See documentation here: https://github.com/google-research/football/blob/master/gfootball/doc/api.md

Please reopen if you have followup questions.


inconsistent behavior with atoms as labels in schema and JSON

jsx can parse labels to atoms and also serialize atoms in labels as strings. When using this for the schemas and/or the JSON, the validation results are inconsistent and in many cases plain wrong.
When using atom labels in the schema, validations seems be to totally broken. In this sample, schema validation should return invalid types for a, b, and c, but it doesn't. There is no difference in whether the labels in the JSON are atoms or binary.
2> Schema = jsx:decode(<<"{\"properties\": {\"a\": {\"type\": \"integer\"}, \"b\": {\"type\": \"string\"}, \"c\": {\"type\": \"boolean\"}}, \"additionalProperties\": false}">>, [{labels, atom}]).
#{additionalProperties => false,
  properties =>
      #{a => #{type => <<"integer">>},
        b => #{type => <<"string">>},
        c => #{type => <<"boolean">>}}}
3> jesse:validate_with_schema(Schema, #{a => true, b => 2, c => 3}, [{allowed_errors, infinity}]).
{ok,#{a => true,b => 2,c => 3}}
4> jesse:validate_with_schema(Schema, #{a => true, b => 2, <<"c">> => 3}, [{allowed_errors, infinity}]).
{ok,#{a => true,b => 2,<<"c">> => 3}}

When using binaries as labels in the schema and atom in the JSON, validation results are "strange":
1> Schema = jsx:decode(<<"{\"properties\": {\"a\": {\"type\": \"integer\"}, \"b\": {\"type\": \"string\"}, \"c\": {\"type\": \"boolean\"}}, \"additionalProperties\": false}">>).
#{<<"additionalProperties">> => false,
  <<"properties">> =>
      #{<<"a">> => #{<<"type">> => <<"integer">>},
        <<"b">> => #{<<"type">> => <<"string">>},
        <<"c">> => #{<<"type">> => <<"boolean">>}}}
2> jesse:validate_with_schema(Schema, #{a => true, b => 2, c => 3}, [{allowed_errors, infinity}]).
{error,[{data_invalid,#{<<"additionalProperties">> => false,
                        <<"properties">> =>
                            #{<<"a">> => #{<<"type">> => <<"integer">>},
                              <<"b">> => #{<<"type">> => <<"string">>},
                              <<"c">> => #{<<"type">> => <<"boolean">>}}},
                      no_extra_properties_allowed,
                      #{a => true,b => 2,c => 3},
                      [a]},
        {data_invalid,#{<<"additionalProperties">> => false,
                        <<"properties">> =>
                            #{<<"a">> => #{<<"type">> => <<"integer">>},
                              <<"b">> => #{<<"type">> => <<"string">>},
                              <<"c">> => #{<<"type">> => <<"boolean">>}}},
                      no_extra_properties_allowed,
                      #{a => true,b => 2,c => 3},
                      [b]},
        {data_invalid,#{<<"additionalProperties">> => false,
                        <<"properties">> =>
                            #{<<"a">> => #{<<"type">> => <<"integer">>},
                              <<"b">> => #{<<"type">> => <<"string">>},
                              <<"c">> => #{<<"type">> => <<"boolean">>}}},
                      no_extra_properties_allowed,
                      #{a => true,b => 2,c => 3},
                      [c]},
        {data_invalid,#{<<"type">> => <<"integer">>},
                      wrong_type,true,
                      [<<"a">>]},
        {data_invalid,#{<<"type">> => <<"string">>},
                      wrong_type,2,
                      [<<"b">>]},
        {data_invalid,#{<<"type">> => <<"boolean">>},
                      wrong_type,3,
                      [<<"c">>]}]}

It does pick up the wrong data types for a, b and c, but it also reports all the fields with an no_extra_properties_allowed error.

something went wrong when creating this ticket and it ended up being created twice, closing the duplciate


Update TestOutputListenerProvider.java to fix [NETBEANS-635]

[NETBEANS-635] Fixed parsing of test results for surefire 2.19 or later to restore the results navigation via links. For details see https://issues.apache.org/jira/browse/NETBEANS-635
This fix guarantees proper parsing of test results output for surefire 2.19 and later. It has been tested with 2.18.1, 2.19.1, 2.22.1, 3.0.0-M1
I really hope this very annoying bug will be included in the 10 release.

@jmborer Please don't change the permission (644 -> 755). Could you please add unit tests?

I really hope this very annoying bug will be included in the 10 release.

Unfortunately, this is not a critical or blocker issue. So, I think it would be included after NetBeans 10.
@lkishalmi Can we include this in NB10?
@mkleint Could you please review this if possible?

should you be adding new tests to TestOutputListenerProviderTest.java to cover the new output as well? this regexp matching has always been a tricky business.

I’ll try to extend the existing unit test, but the problem is that currently the reports path is deducted from the project context, because the surefire plugin no longer logs the path on the console (I have filed a change request for that on the surefire Jira). This context is not easily mockable. I would have loved to be able to use Mockito in this cas, but because of my lack of time, I didn’t have time to check if it is available and if I have the « right » to add such kind of stuff to the NB build system.

I've prepare a unit test. How shall I add it to this PR? Create a new one with both files?

Just update your branch on GitHub it will automatically update your PR.

Thanks for the tip (not yet used to work with community on github ;-)
I just pushed the change.

@jmborer Thank you adding them :)
@mkleint Thank you for reviewing it! It seems that unit tests were added. So, if it's OK with you, we can (squash and) merge this :)


missing support of shared mailboxes

Microsoft Support just confirmed that the usage of the report message add-in is not working / supported for shared / team mailboxes.
Can you add such a note to this article?
I would assume the same applies to the report phishing add-in

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

ID: 445892a6-5948-33e4-ce60-72670829d8ef
Version Independent ID: a7407a6f-6563-9a4b-4a95-7ed908ab322b
Content: Enable the Report Message add-in - Office 365
Content Source: microsoft-365/security/office-365-security/enable-the-report-message-add-in.md
Product: m365-security
Technology: mdo
GitHub Login: @siosulli
Microsoft Alias: siosulli

Hi @sebhe and update on this was actually made yesterday to reflect this and should be available shortly!

Ha, that is funny coincidence. Maybe this was due to my support ticket :-)
looking forward to the updated version

Closing this now as this following is now in the article:
The Report Message add-in is not available for shared mailboxes or mailboxes in on-premises Exchange organizations.


Task: rewrite Solid spec in W3C template

The current Solid specification has a couple of issues;

It is a collection of Markdown documents, as opposed to a single authoritative document
The specification contains parts that are (purposely) not implemented
Some parts that are implemented (and necessary) are not in the spec
The spec is insufficiently precise to be implemented in an unambiguous way

As such, I propose to start from scratch with a Solid spec in a W3C template, where we pull in curated parts of the current Solid v0.7 spec, with the aim of reaching a v1.0.

https://www.w3.org/StyleSheets/TR/2016/readme.html

I suggest https://tabatkins.github.io/bikeshed/ many specs on REC track use it and it still allows using convenient markdown.

As such, I propose to start from scratch with a Solid spec in a W3C template, where we pull in curated parts of the current Solid v0.7 spec, with the aim of reaching a v1.0.

I'm in 100% agreement with this proposal.

Perhaps in a new repo GitHub.com/solid/specification?

+1 to converting the format
+1 to fixing points where the text diverts from current practice
+1 to fixing ambiguities
-1 to "pull in curated parts" if by that you mean making changes to the meaning of the spec
-1 to " the aim of reaching a v1.0" if by that you mean changing its technical details into different ones
In this phase (I would say from now to roughly the end of 2019), we should only change the spec to make it better describe the current practice, or to fix (security) bugs.

-1 to "pull in curated parts" if by that you mean making changes to the meaning of the spec
-1 to " the aim of reaching a v1.0" if by that you mean changing its technical details into different ones

conflict with

+1 to fixing points where the text diverts from current practice

but I agree with the overall sentiment.

But really, we should move away from the idea that v0.7 is untouchable, or even a spec. It's mainly a set of documentation written together, without any consensus process or anyone wondering whether something is a good idea. Let's have a minimum of oversight here.
Regarding timeline specifically:

In this phase (I would say from now to roughly the end of 2019),

Why wouldn't we want a v1.0 by the end of 2019?

Follow-up in https://lists.w3.org/Archives/Public/public-solid/2019May/0009.html and https://github.com/solid/specification


refactor(miniapp): optimize entry name in runtime miniapp

Codecov Report

Merging #544 (881c09d) into release-next (da754ff) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff              @@
##           release-next     #544   +/-   ##
=============================================
  Coverage         87.80%   87.80%
=============================================
  Files                21       21
  Lines               779      779
  Branches            196      196
=============================================
  Hits                684      684
  Misses               86       86
  Partials              9        9


Added analytics-zoo, updated README

Note for @blublinsky : I have updated apps/anomaly-detection/README.md. But it needs update for your modules too.
Some more stuff can be removed from analytics-zoo, which I will work on gradually. Just wanted to have the PR out so that @blublinsky can start updating the documentation parallely.

Made more changes today. Code structure organized like the other sample apps, used our common sbt plugins for dockerization. And changed the project / docker image names based on the convention that we followed earlier. Pls take a look and review ..

Add chart and missing Grafana jsons, updated Readme.
Had to remove .json from gitignore. See if this creates a problem

👍

fixed it, you can merge now


Issues with Apple Watch - Authenticator app integration

Hi team,
there seems to be a limitation around Apple Watch:
it seems as if the MS authenticator app on an Apple Watch is not able to integrate with password less. The 3 numbers to select from are not displayed on the Apple Watch, you have to use the Authenticator app on the iphone instead.
Thanks

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

ID: ee6ac70d-5853-6a3a-0964-23b46910b4c4
Version Independent ID: a5beda87-9845-8807-6221-7fdc64079587
Content: Passwordless sign-in with the Microsoft Authenticator app - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-authentication-passwordless-phone.md
Service: active-directory
Sub-service: authentication
GitHub Login: @Justinha
Microsoft Alias: justinha

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

Hi @sebhe,
There are some known connectivity issues with Apple Watch. Please refer to our FAQ article for mitigation steps. There was a known bug on watch OS 7. This was reported to Apple but I'm unsure if they have resolved this yet.
One thing that has worked for others is following these steps:
Open Authenticator in your iPhone. Go to Settings in the Authenticator application and disable the application lock in security. By deactivating the application lock in the app the approval should work again.
Hope this helps! If you have further questions I would recommend reporting this to the Apple team. https://discussions.apple.com/


Remove buf-setup action

We don't use buf-setup action in this repository, let's remove it.

Actually, we need this for testing.


Added the ability for the shiphawk client class to be used as a singl…

…eton

I updated the PR to resolve the merge conflict.


Incomplete values for some tags

I'm trying to read the english wikipedia data dump using spark-xml. My setup is as follows:

zeppelin 0.5.6 (using built it Spark 1.6.0)
using spark-xml 0.3.1 on scala 2.10

I'm loading the wikipedia pages as follows
val items = z.sqlContext.read
                        .format("com.databricks.spark.xml")
                        .option("rowTag", "page")
                        .load("<PATH>/enwiki-latest-pages-articles-sample.xml")
which succeeds and infers a schema that seems reasonable
org.apache.spark.sql.DataFrame = [id: bigint, ns: bigint, redirect: struct<#VALUE:string,@title:string>, revision: struct<comment:string,contributor:struct<id:bigint,ip:string,username:string>,format:string,id:bigint,minor:string,model:string,parentid:bigint,sha1:string,text:struct<#VALUE:string,@space:string>,timestamp:string>, title: string]
If I try to items.select("revision.text.#VALUE") it gives me a value that's only a prefix to the whole text available in that page meaning the value is not read fully.
Another thing I tried is to do
// changing the rowTag from page to text
val items = z.sqlContext.read
                        .format("com.databricks.spark.xml")
                        .option("rowTag", "text")
                        .load("<PATH>/enwiki-latest-pages-articles-sample.xml")
and I now get
org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 4.0 failed 1 times, most recent failure: Lost task 0.0 in stage 4.0 (TID 4, localhost): scala.MatchError: ENDDOCUMENT (of class com.sun.xml.internal.stream.events.EndDocumentEvent)
	at com.databricks.spark.xml.parsers.StaxXmlParser$.checkEndElement(StaxXmlParser.scala:94)
	at com.databricks.spark.xml.util.InferSchema$.com$databricks$spark$xml$util$InferSchema$$inferObject(InferSchema.scala:218)
	at com.databricks.spark.xml.util.InferSchema$$anonfun$3$$anonfun$apply$2.apply(InferSchema.scala:95)
	at com.databricks.spark.xml.util.InferSchema$$anonfun$3$$anonfun$apply$2.apply(InferSchema.scala:83)
	at scala.collection.Iterator$$anon$13.hasNext(Iterator.scala:371)
...
which seems to suggest that for some reason it's not able to find the end_tag for text even though it is there.
I'm assuming something is wrong with the XML parsing here but I'm not really sure what's special abt the text tag.
I've attached the input file at https://gist.github.com/humanzz/e2e296fd82dccaa5109c

using the current spark-xml snapshot (spark-xml_2.10-0.3.2-SNAPSHOT.jar) seems to have loaded the full context of the text tag.
However,
// changing the rowTag from page to text
val items = z.sqlContext.read
                        .format("com.databricks.spark.xml")
                        .option("rowTag", "text")
                        .load("<PATH>/enwiki-latest-pages-articles-sample.xml")
still fails. Maybe deeper level tags cannot be used as rowTags? I'm not sure what's the intended behaviour here.

@humanzz As far as I remember this was fixed in master branch but a release has been not published. I will try to reproduce this first with the master branch one. Thanks for reporting this.

I am running into scala.MatchError: ENDDOCUMENT. For me it seems because my input contains self-closing row tags, e.g. <ROW attr="foo" />. I see some self-closing tags in your input too, but not for your text row tag, so I don't know if it's the same problem.
@HyukjinKwon Is self-closing tags something that you want to support? I had a quick look and don't directly see a quick way to do that with how XmlInputFormat currently works. If you do I can open a separate issue for it.

@HyukjinKwon I confirm it's fixed in master branch. Just tried it yesterday.
I was running the spark-xml on a sample of a few wikipedia pages in the dump... I'm currently trying to read the whole dump enwiki-latest-pages-articles.xml.bz2 but now failing with scala.MatchError in com.databricks.spark.xml.parsers.StaxXmlParser$$anonfun$com$databricks$spark$xml$parsers$StaxXmlParser$$convertObject$1.apply$mcVI$sp(StaxXmlParser.scala:218).
As I'm no expert on spark internals, Is there a way to try the spark-xml reading independently so that I can reproduce the error but with more details this time?

@sgvandijk Yes, this should be supported. Thanks for looking into this deep. Could you please create another issue for this?
@humanzz Actually, I recently tested this library with the wiki dump (only a small dataset) and it looked fine but not with a large data set due to the lack of resources (or my laziness to set up a cluster in AWS). I think I should test it again with the file you just said.
Also, If you meant testing this within IDE such as IntelliJ or Eclipse, you can simply "open" this directory and some configurations would be automatically set up (but maybe you might have to install Scala 2.10.x or 2.11.x). And then, you can just run some tests codes in XmlSuite within the IDE. This is what I usually do.
Alternatively, you can build this by sbt package and then start the spark-shell including the built jar file by --jars.

@humanzz Maybe the issue you just met is related with https://github.com/databricks/spark-xml/issues/80. I will try to resolve that one first.

@humanzz Would you please try this with 0.3.2? I just released a new one and it looks working fine with wiki dumps.

@HyukjinKwon Sorry for the late response. I just ran it today and it now works just fine :)

@humanzz thanks! I am closing this.


ci(docs): add jazzy doc generation to pipeline

Generates Jazzy docs after deployment and merges them back to the gh-pages branch.
(* need to add an ssh key for amplify-ios to the circleci project settings in order for the pipeline to be able to push changes to the repo)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Closing in favor of merged https://github.com/aws-amplify/amplify-ios/pull/1658.


Determine which elements to include in header

How much customization do we want each of us to have over what appears in the header? Is differentiation by logo, brand colors and typography enough? Should all things be formatted similarly?

Potential header elements:

Logo
Institution name separate from logo?
Site Title?
Search
Primary Navigation
Secondary Navigation?
Breadcrumbs in header? Or, in main content, adjacent to page title?
Login/Identity Management/Dashboard
Call to action to donate to University of Nebraska Foundation
Call to action to Apply Now

Just thought I'd share a mock up I made a while ago trying to figure out what to do with our headers. Nebraska Medicine just went through a site redesign so we're trying to align with them a bit more. We also have some sites, like facilities, that are both UNMC and Nebraska Medicine. None of it has at all been finalized.
UNMC header examples
I still need to talk to our branding team, but Im hoping we don't need to spell out "University of Nebraska Medical Center" on every page and that the UNMC in the top left will be enough. Otherwise these titles will get pretty ugly, especially on mobile.
Also using a secondary navigation for some action items like Apply, Visit, etc. The"Audience" link would be a drop down that would lead you to our current "audience gateway" links that we have on every page. These lead to pages that house resources for those particular audience members. Not sure if "Audience" is the best term to use here or something like "Resources for" but you get the idea.


Better training.py enhancement

add more!

you do it

fine

@changcheng967 good

@changcheng967 good

yep

hi

hi

hi

@changchengdouletmedia #41 Why do you revert this?

I just want to let my name on that


Introduce DriverErrorType.usageError to driver-definitions

This PR introudces a new enum member DriverErrorType,usageError with the value driverUsageError. This was done to help replace DriverErrorType.genericError which has been deprecated (see https://github.com/microsoft/FluidFramework/issues/8272 for more info).
Closes https://github.com/microsoft/FluidFramework/issues/8489

@markfields  and @vladsud it seems quite odd to have a driver error be a usage error. usage errors are usually the result of an invalid api call by the consumer, this usually isn't a driver api. This also already exists on (ContainerErrorType0[https://github.com/microsoft/FluidFramework/blob/01c841910f48d4b8d27cbddcd7aa61fc2621d744/common/lib/container-definitions/src/error.ts#L35]. which seems like the right place for this. i worry just shoe horning this into driver error is just a generic error by another error and as it does not semantically make sense

After further discussion with @markfields, we decided to keep genericError and we will not be adding in a new error type.

A bit more context - My original concern (see https://github.com/microsoft/FluidFramework/pull/6489) was the reuse of the value. But what I've observed since then is that code all over the place (even in partner repos!) will set the errorType to genericError. So its interpretation is very broad, and doesn't indicate any particular source of the error.
Furthermore, attempting to leverage errorType to track back to where an error originated could be seen as a mild abuse of the field, whose primary purpose is signaling the class of error to the host or component so it can react at runtime (e.g. showing different error UX).
So instead of removing, replacing or renaming this errorType, we will leave it be and remove the deprecated tag.


BREAKING CHANGE: Possible build break in ADO pipelines coming

Commit 47b3594878bb2a3db1c967ad6ad1876d152ee61d reorganizes how tests are run. I attempted to reflect this properly in the dotnet ADO build. We shall see if it's successful.
If not, I'll tweak until it works.
If so, I'll update the other pipelines.
@RecursiveGeek

Ok, it passed. Yay. Only problem is that Azure Pipelines' code coverage report generator can't seem to handle multiple code coverage output files.
This is tracked as:

Bug AB#1451: Azure Pipelines Can't handle multiple code coverage files


Add protobuf libraries to the linker search path

Signed-off-by: Violeta Sebe<EMAIL_ADDRESS>

@jdekonin @sxa555 please review, thank you!

@sxa555 - I removed the role from ansible/playbooks/AdoptOpenJDK_Unix_Playbook/main.yml.

Thanks - are the changes still needed in the local_srcinstall role?

are the changes still needed in the local_srcinstall role?

@sxa555 - no, they're not (removed). Protobuf role should be used instead of local_srcinstall role.

Fixed merge conflict.
@sxa555 ready for your review, thank you!

@vsebe Can you resolve merge conflicts please. We're out of lockdown now so once that's done I can merge

@sxa555 - merge conflict fixed, thx


Focused Player image

Is there a way to add specific images for the "focused player" in the HUD?

Do you mean like an avatar or a facecam? Not currently, but hoping to add this feature eventually, cf. #109.
Let me know if you meant something else :D

I was referring more to the fact that in the same way that there is a "team-logos" folder, you can do the same thing for player avatars. For example, you could download the PNG images from HLTV to a specific folder called "player-avatars". The PNG files could be named using the same names as the ones previously set in the "player name overrides". For instance, if I associate tabsen's SteamID with the name "tabsen" in "player-name-overrides", I could download tabsen's image from HLTV, name it tabsen.png, and put it in the "player-avatars" folder. The software should then be able to detect the similarity between the player's name and the image's name.

Yeah, that's what I meant :D Not a feature currently, but should eventually be added as part of #109.


Merge in latest changes

Merge in latest changes

Thanks for the feedback

Yes, and my latest develop branch has this
Not sure, I can't see the actual comment in context
I have renamed all the interfaces, as <name>able, doesn't work for things like Table and GridItem, they are now Implements<XXX>, still not perfect but better (I think).

I will leave the PRs open for longer as feedback is welcome


Support option to use Slf4j/Commons Logging/JUL when using the command line tool

Currently, when using the command line tool, Flyway only supports logging to standard out. It would be useful if the command line tool could optionally leverage Flyway's existing ability to use Slf4j or Commons Logging if their jars are on the classpath.

Thanks for your suggestion. What would be the real world use case for this?
Cheers
Axel

It would allow users of the command line tool to log the output of the tool to a file on disk if a jar with a logging configuration file and the logging jars were on the classpath. You could also do things like configure the logger to print the info statements to the console, but only log the debug statements to a file, so you don't have to run the tool with the debug flag and you still have the ability to consult the logs if there's an issue.
Thanks,
Eric

I ended up implementing it with an automatic switch instead.
Cheers
Axel


V2.0

Merging v2.0 to main. This PR contains the following features:

Email service
API service for a time frame
CVS report download option

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: leihchen, skanthed
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


devtools: Improve the appearance of dev_panel.

It's primarily for style changes to improve the look of the integration devtool.
GIFs or screenshots:
Before

After

Merged, thanks @PIG208!


请教下，为啥pg不支持从lsn拉取，是有什么额外的考虑吗

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Could you use English for better communication? feel free to reopen once updated the description.


[BUG][Native/Android] The design of the Claim screen does not match the mockup

Description: The design of the Claim screen does not match the mockup
Steps to reproduce:

Launch the app
Login into the account
Go to the Claim screen
Press the Claim button
Pay attention to the Counter and Gooddollar Stats

Expected result:
The design of the Claim screen matches the mockup
Actual result:
The design of the Claim screen does not match the mockup
The user is new: No
Environment:  App Version 1.13.0-0 (983)
Devices list: Xiaomi Mi8 // Android 8.1.0
Additional info:
The user is registered with self custody method
Backup words: bottom duty enroll crew fabric window warrior cabbage goat potato fine neither
Mockups:https://xd.adobe.com/view/35096551-9e0b-415a-91ea-810935803331-605a/screen/3db9e90f-fe12-4b7b-bb2e-27664d87598c/
Attachment:

@sirpy The issue is not reproduced. The design of the Claim screen matches the mockup
Checked on the Android App 1.22.13-1 (1204)
Device: Xiaomi Mi 8 // Android 8.1.0
Attachment:


add timestamp to each log

I believe this is the best place to add time to each log message. Log4j does this automatically for the logs it outputs but we need to add one manually so that juttle-viewer receives the right time.

@mstemm @go-oleg @demmer

changed it to subprocess.

lgtm!


How about insert some html into content? If insert simple div it works. But if you want to add something like this

How to solve this issue... I am also having the same below problem.. How about insert some html into content? If insert simple div it works. But if you want to add something like this

it add only div
or
embedding tags like  etc inside  is not working..
Originally posted by @arrchi-style in https://github.com/davidroyer/vue2-editor/issues/63#issuecomment-532582430

This issue is stale because it has been open for 60 days with no activity.

This issue was closed because it has been inactive for 7 days since being marked as stale.


⚠ Bofrost.be has degraded performance

In 44f8996, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1049 ms

Resolved: Bofrost.be performance has improved in 2d1bec1.


TypeError in /

Error in People Counter
TypeError in /
undefined is not an object (evaluating 't.user.spaces.length')
View on Bugsnag
Stacktrace
resources/js/components/HomePage.vue?c8f2:1 - null
resources/js/app.js:27 - null

View full stacktrace
Created by RoboJackets Admin via Bugsnag

The affected line is:
e(),_vm._v(" "),_c('hr'),_vm._v(" "),_c('b',[_vm._v("Default Space:")]),_vm._v(" "),(_vm.user.spaces.length === 0)?[_vm._v("\n                            Not Set\n                        ")]:[_vm._v("


Another library like this.

Hi there,
i curate the https://github.com/h4cc/awesome-elixir list and found another library similar to yours:
https://github.com/michihuber/mixpanel_ex
Maybe you could start working together, like this:

Thanks for the feedback and awesome picture.
Our livs serve two seperate purposes. His is to track events. Mine is to interact with the mixpanel data api.


How to deal with stored procedures that return multiple rows, in golang 1.9 or later (sql.Out)?

So in golang 1.9, sql.Out was introduced, allowing this sort of thing:
  var outArg string
  _, err := db.ExecContext(ctx, "ProcName", sql.Named("Arg1", Out{Dest: &outArg}))

This is relatively straightforward and easy to use if you are getting a single value from a query.
However, if you are getting multiple rows of multiple columns, it is a pain to deal with, because you usually have to use driver.Rows or []driver.Value
(and lets say I can't change my workplace to not use stored procedures).
I would like to be able to scan the results into a struct (and anything else that sqlx package supports).
Here is a concrete example.
The procedure:
PROCEDURE get_regexs( o_csr OUT o_as_cur) IS
BEGIN
  IF (o_csr%ISOPEN) THEN
    CLOSE o_csr;
  END IF;
  OPEN o_csr FOR
    SELECT
      t1.regexp_id,
      t1.regexp,
      t1.other_id,
      t1.regexp_match_type
    FROM
      my_regexp_t t1
    WHERE
     t1.status = 1;;
END;

The code I am currently using:
var rset driver.Rows
res, err := conn.ExecContext(ctx, `begin myuser.get_regexs(:v1); end;`, sql.Out{Dest: &rset})
// Deal with err and defer rset.Close()

rows := make([]driver.Value, len(rset.Columns()))
for {
	if err := rset.Next(rows); err != nil {
		if err == io.EOF {
			break
		}
		return err
	}
	// Manually copy each element in the `rows` slice into a slice of structs...
}

There are a few different ways that sqlx could help.
One way would be to have some sort of ScanStructFromValues method, that takes a driver.Rows and scans it into a struct.
Another way would be to have the current Select method automatically take any sql.Out that is provided as an argument, and scan them into the provided struct interface.  Though on second thought, this would be incomplete because Select only allows one destination interface, while the procedure may have multiple outputs.
Maybe there could be a sqlx.Out that includes the destination interface in it, and sqlx automatically scans it into that struct.
Just some ideas, but I'd like to know if this something you'd consider adding to sqlx.
I would much rather hook into the tried and tested struct scanning that sqlx provides, than recreate the wheel.

Hey @jmoiron, I'm curious to hear what you think?

Having the same thought here in 2020. Any recommended solution?


decode minified stacktrace

Description
This pull request addresses issue #1524. It adds a helper function that fetches and utilizes the source .map files to decode the stack trace and implements it when a user clicks on "Copy Error Information."
Checklist
Are your changes backwards compatible? Please answer below:
yes
On a scale of 1-10 how well and how have you QA'd this change and any features it might affect? Please answer below:
9 - This implementation looks for a specific stack trace pattern, but I am unsure if it may vary. However, it does return the original stack trace intact, so any failure in pattern matching will not lead to losing information in the report.

Mhh, I just read the NextJS docs about productionBrowserSourceMaps again. I read:

When the productionBrowserSourceMaps option is enabled, the source maps will be output in the same directory as the JavaScript files. Next.js will automatically serve these files when requested.

So this means these source maps aren't automatically requested on errors? I hoped we just need to use NextJS more / properly instead of writing a custom stack trace decoder. But fine for me if there's no better solution. Will pull code and test today.

Mhh, I just read the NextJS docs about productionBrowserSourceMaps again. I read:

When the productionBrowserSourceMaps option is enabled, the source maps will be output in the same directory as the JavaScript files. Next.js will automatically serve these files when requested.

So this means these source maps aren't automatically requested on errors? I hoped we just need to use NextJS more / properly instead of writing a custom stack trace decoder. But fine for me if there's no better solution. Will pull code and test today.

I might be wrong, but i did some digging and my understanding is that they are just there ( ie. https://a.stacker.news/_next/static/chunks/pages/_app-f2a569540a712597.js.map ) for whatever logging/monitoring tool or code you want to use.

This should be a huge improvement and hopefully someday we'll have a better means of alerting us to clientside errors/context.


Issue in rendering path having space

Prerequisites

[ x ] I have read and understand the CONTRIBUTING guide
[ x ] I looked for duplicate issues before submitting this one

Description
[Description of the bug or feature]
Environment

Oh my Posh version: 3
Theme: zash
Operating System: windows 10 2003
Shell: powershell
Terminal: windows terminal

Steps to Reproduce
installed oh-my-posh according to the steps mentioned in the readme
Expected behavior: The pathname on the terminal should be C:\Users\Jibin Thomas\dev
Actual behavior: The path is being truncated till the space C:\Users\Jibin
Attaching a screenshot of the problem

That's silly. Let's see if we can fix that.

A new version will be available soon. The path wasn't provided in full to the binary it seems, causing spaces to drop everything that came after. Thanks for noticing this!

A new version will be available soon. The path wasn't provided in full to the binary it seems, causing spaces to drop everything that came after. Thanks for noticing this!

Thanks for the quick fix :)


Duplicate integration?

How is this different from https://github.com/jlamendo/ha-sensor.owlet by @jlamendo?

The integration you linked doesn't use a separate API library, the API is baked into the integration itself. This is the first point in HA's development check list stating that all external communication should be wrapped in a separate library https://developers.home-assistant.io/docs/development_checklist.
The integration I created allows setup from the UI, reauth and allows to change options from UI. I believe my integration also exposes more sensors and conforms more to the standard structure of core HA integrations whereas the integration you linked has most of the code in one file.
Both have their place I'm sure. I created mine as a pet project and just added it to HACS as I believed other people would find it useful and easier to use that some of the other Owlet integrations I saw (including the one you linked)


Sysmon 11.10 stops capturing network events after a period of time

Issue Description:
After updating Sysmon  10.42 to 11.10, I have unfortunately recognized, that the Sysmon service stops capturing network events (Event ID: 3) after a period of time of round about 45 minutes (sometimes less than 45 minutes). To fix the issue I have tried to re-install the sysmon service and also to update the configuration file. However, the service stops capturing network events after a short period of time.  Furthermore, to verify that the issue is not on my system only, my fellows and I checked also their Sysmon logs and noticed the same behavior.
We also have checked if the network monitoring is enabled and it is.
Current configuration:
 - Service name:                  Sysmon64
 - Driver name:                   SysmonDrv

 - HashingAlgorithms:             SHA1,MD5,SHA256,IMPHASH
 - Network connection:            enabled
 - Image loading:                 enabled
 - CRL checking:                  enabled
 - DNS lookup:                    disabled
 - Filter archive directory:      \Sysmon\

System:
Windows 10.0.18362.904
Sysmon 11.10

Hello
could you contact me offline at<EMAIL_ADDRESS>and I will try to assist you with this.
MarkC(MSFT)

I have the same issue. I've seen this on Windows Server 2016 10.0.14393

Hello
I am looking at this today. Would you happen to have a memory dump from an affected endpoint that you could share with us? If so could you contact me at marcook<EMAIL_ADDRESS>and I will provide you with an upload location.
Regards
Mark
From: super0xbad1dea<EMAIL_ADDRESS>Sent: 09 July 2020 09:40
To: MicrosoftDocs/sysinternals<EMAIL_ADDRESS>Cc: Mark Cook<EMAIL_ADDRESS>Assign<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/sysinternals] Sysmon 11.10 stops capturing network events after a period of time (#295)
I have the same issue. I've seen this on Windows Server 2016 10.0.14393
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fsysinternals%2Fissues%2F295%23issuecomment-655993433&data=02|01|marcook%40microsoft.com|28f75eda632f40c21c1a08d823e3a39a|72f988bf86f141af91ab2d7cd011db47|1|0|637298807893782289&sdata=Y6V5epEUfMZxOhlEqM99rYc%2BWkjNm1Aqf9qBeAxzw%2Bs%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAKJ6QD7XZQP4QT53NFEVZITR2V65HANCNFSM4OJKE76A&data=02|01|marcook%40microsoft.com|28f75eda632f40c21c1a08d823e3a39a|72f988bf86f141af91ab2d7cd011db47|1|0|637298807893782289&sdata=xBzTCcoCa%2FGdi57iB9h3RSkeFHEA2MiOXZZr00SPZTs%3D&reserved=0.

Resolved in Sysmon 11.11, closing.


Lordaeron changes

Can we just get rid of the "Arthas Menethil is at King Terenas" bit? Seems a bit silly considering it would require Arthas to go back home before the Frozen Throne is destroyed.

This new ability shares an icon with Transmute.

This quest says that Arthas can "no longer permanently die", but actually it doesn't affect his permadeath status at all.

 Can we just get rid of the "Arthas Menethil is at King Terenas" bit? Seems a bit silly considering it would require Arthas to go back home before the Frozen Throne is destroyed.

Couldnt come up with any more fitting Icons if u can think of any i can change it

 This new ability shares an icon with Transmute.

I couldnt come up with any other icons if u can think of any more fitting i can change it.

 This new ability shares an icon with Transmute.

I couldnt come up with any other icons if u can think of any more fitting i can change it.

As the person adding the ability it's up to you to find and implement the icon. This is a very generic ability so just look up Holy on Hive Workshop and you'll find something.


Unable to create TensorRT engine

Expected Behavior

Actual Behavior
creates tensor
Steps to Reproduce
wont work on my desktop manjaro, gtx 1060, 32gb ram, 6gb vram wont generate for sdv1.5 nor turbo even though it stays well below 5gb when starting with sdv1.5....
Debug Logs
in image
Other
No response

Probably an error to post to https://github.com/comfyanonymous/ComfyUI_TensorRT


Auto position does not work correctly when dropdown is in nav tag #v.5

Operating system and version (Windows)
Browser and version (All)
https://codepen.io/alexshnur/pen/qBrOPNb?editors=1010

The problem is we don't initialize Popper for dropdowns in the nav element, tho I'm not sure why (I wasn't aware of this).
As a workaround you could add .dropdown-menu-end to the dropdown-menu element to have it right-aligned.

this does not solve the problem for dynamic content

I was running in to the same issue. The following workaround solves the problem for me to render some right-aligned dropdown menu:
<div
  class="dropdown-menu dropdown-menu-end"
  :class="{ show: showMenu }"
  data-bs-popper
>

This works because the CSS selector include data-bs-popper. Not sure if that causes other side effects though.


Improve test coverage

can #17 be closed with this?

@edgurgel would this be more or less ok for merging? and I noticed that there are endpoints added that are not yet released - looks like time for a release. :)

Sorry for the delay! I just got back from a trip. This looks amazing. I will release a new version today! Thank you again for such amazing contribution :)