update fails with a NPE using maven 3.9.0

Describe the bug
When using maven 3.9.0 updating the database fails
java.lang.NullPointerException: Cannot invoke "java.util.List.stream()" because the return value of "org.apache.maven.project.MavenProject.getCollectedProjects()" is null

Version of dependency-check used
The problem occurs using version 7.4.4 of the maven plugin
Log file
Apache Maven 3.9.0 (9b58d2bad23a66be161c4664ef21ce219c2c8584)
Maven home: /mnt/c/users/sellersj/Downloads/apache-maven-3.9.0
Java version: 17.0.5, vendor: Private Build, runtime: /usr/lib/jvm/java-17-openjdk-amd64
Default locale: en, platform encoding: UTF-8
OS name: "linux", version: "<IP_ADDRESS>-microsoft-standard-wsl2", arch: "amd64", family: "unix"
...
[ERROR] Cannot invoke "java.util.List.stream()" because the return value of "org.apache.maven.project.MavenProject.getCollectedProjects()" is null
java.lang.NullPointerException: Cannot invoke "java.util.List.stream()" because the return value of "org.apache.maven.project.MavenProject.getCollectedProjects()" is null
    at org.apache.maven.lifecycle.internal.LifecycleDependencyResolver.getProjectAndSubModules (LifecycleDependencyResolver.java:102)
    at org.apache.maven.lifecycle.internal.LifecycleDependencyResolver.getProjects (LifecycleDependencyResolver.java:93)
    at org.apache.maven.lifecycle.internal.MojoExecutor.ensureDependenciesAreResolved (MojoExecutor.java:395)
    at org.apache.maven.lifecycle.internal.MojoExecutor.doExecute (MojoExecutor.java:327)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:213)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:175)
    at org.apache.maven.lifecycle.internal.MojoExecutor.access$000 (MojoExecutor.java:76)
    at org.apache.maven.lifecycle.internal.MojoExecutor$1.run (MojoExecutor.java:163)
    at org.apache.maven.plugin.DefaultMojosExecutionStrategy.execute (DefaultMojosExecutionStrategy.java:39)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:160)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:105)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:73)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:53)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:118)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:260)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:172)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:100)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:821)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:270)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:192)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at jdk.internal.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:77)
    at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:568)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:282)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:225)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:406)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:347)
[ERROR]

To Reproduce
Steps to reproduce the behavior:
Download and extract maven 3.9.0
run
./apache-maven-3.9.0/bin/mvn -X -V org.owasp:dependency-check-maven:7.4.4:update-only -e -Dhttps.protocols=TLSv1,TLSv1.1,TLSv1.2

It also happens if you use version 8.0.2 or using java 8.
Expected behavior
For it to update the database.

Note: I typically run the update-only task by itself without a pom file in CI.
I think that if you run the command in the same directory as a pom.xml it will work.

Yikes... maven returning null instead of an empty list on a minor release... sounds like a regression to report there as well.

Will do. Sorry I didn't look at the stack trace very closely - I don't see any owasp code in it. Will create a ticket there and link it from here.

Seems this is a known maven bug
https://issues.apache.org/jira/browse/MNG-7679
Since this isn't an dependency check issue, I'll close the ticket.

Will do. Sorry I didn't look at the stack trace very closely - I don't see any owasp code in it. Will create a ticket there and link it from here.

I Also didn't spot that detail, thought it would've been triggered somewhere in our codebase, but indeed it's fully isolated to Maven itself. And thanks for digging up the maven ticket. When I quickly tried to see if the NPE was already reported in maven JIRA (for some plugin) I failed to find it.


pin editor when breakpoints change, fix #55283

I rebased onto master.

Thanks for the PR however this is not the correct place to do this.
I would prefer if the DebugService would somehow do this. Closing this PR as I do not like the approach.
Thanks again and sorry for not providing more guidance


Slow Performance

Hi,
First I would like to thank you for a great product.
Every time I switch between tabs (Tables, Views) it take about 30 seconds to switch. In the chrome console I see the following:
viewedit:1 Unchecked runtime.lastError: Could not establish connection. Receiving end does not exist.
foreground.js:13 Error occurred when fetchNoteList
(anonymous) @ foreground.js:13
(anonymous) @ foreground.js:1
viewedit:1 Unchecked runtime.lastError: Could not establish connection. Receiving end does not exist.
I use docker image out of the box with postgresql backend.
Everything looks well only very very slow.
Please advice.
Thanks
Amir

I also installed the package on a freash ubuntu docker and got to the same point where every click takes about 20 sec to process.
Please help.

Hi @amirhadar , I've never seen anything like that. I think the problem must be your postgresql connection, Either the connection parameters to saltcorn are incorrect, the postgresql backend is not configured correctly, or the problem is a version mismatch. Can you tell me:

What version postgresql are you running?
how are you connecting?
can you connect with psql?

I'll see if I can think of anything else that could be going wrong.

@mshiner any ideas? have you ever seen anything like this?

Unfortunately I also tried the sqlite and I got the same performance issue.
I used sqlite db within the docker (not shared) just for the test.
I can add the logs, where are they?

docker logs  e.g. docker logs saltcorn
I guess it is also possible that there is a problem connecting from your browser to the docker server. Have you tried running another web server in the same docker host?

You can also try streaming the log as the server is responding. try docker logs -f saltcorn

So the only log in the docker is :
Saltcorn listening on http://<IP_ADDRESS>:3000/
pgAdmin work well (a bit slow but I have no reference)
DB connection is ok:

Maybe it related to docker on windows with Hyper-V. Have anyone tried this?

not that i know of. saltcorn runs very well on WSL, but i never tried docker on windows

could be https://github.com/docker/for-win/issues/1936 and https://forums.docker.com/t/very-slow-web-requests-in-hyper-v/27031 ?

Guys
Just picked this up (Sorry to be tardy, Tom).
Admittedly I'm not pushing the envelope in terms of database size or app complexity (yet!!) but I haven't seen slow performance from the docker image I've run using Podman (basically because that is what we use for uzERP development) on an Ubuntu 20.04 server - all seems to work OK.


Added edgesWithoutTop, edgesWithoutLeft, edgesWithoutRight, edgesWithoutBottom constraints

I had a lot of use cases when I need to setup 3 of 4 edges to superview.
Made to replace this:
[topView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.top.and.leading.mas_equalTo(20);
    make.trailing.mas_equal(-20);
}];
[bottomView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.leading.mas_equalTo(20);
    make.bottom.and.trailing.mas_equalTo(-20);
    make.top.equalTo(topView.mas_bottom).with.offset(20);
}];

with this:
[topView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.edgesWithoutBottom.mas_equalTo(UIEdgeInsetsMake(20,20,20,20));
}];
[bottomView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.edgesWithoutTop.mas_equalTo(UIEdgeInsetsMake(20,20,20,20));
    make.top.equalTo(topView.mas_bottom).with.offset(20);
}];

hi @k06a :)
Not sure if you are aware but you can pass UIEdgeInsets to when using the make.left.and.right chaining.  This means you can achieve the same using the following
[topView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.top.left.right.mas_equalTo(UIEdgeInsetsMake(20,20,20,20));
}];
[bottomView mas_makeConstraints:^(MASConstraintMaker *make) {
    make.left.right.bottom.mas_equalTo(UIEdgeInsetsMake(20,20,20,20));
    make.top.equalTo(topView.mas_bottom).with.offset(20);
}];

@cloudkite wow! Will right constraint automagically become -20 value by this code?

@k06a yup thats correct, behind the scenes make.edges.equalTo is just make.top.left.right.bottom.equalTo.

@cloudkite thank you, amazing library! I have nothing to contribute right now, but I'll be back :)

@k06a glad you are finding it useful, look forward to your future contributions! :+1:


repeated item id values should also be updated

there should be way to update id="some-id" values in repeated elements

hmm the repeater has no concept of id's so this might be best handled in userland
you could do this using the custom show callback.
$('.repeater').repeater({
  show: function () {
    // not sure if I got the jquery exactly right here, but you get the idea.
    // btw: "this" in this context is a reference to the newly created element,.
    $(this).find('[id]').attr('id', 'some-new-id');
  }
})


fix link in 06.Artifact-creation/04.State-scripts/docs.md

Based on the following report: https://www.deadlinkchecker.com/reports/docs-mender-io-20200831052911.html


Add targets to links at the bottom

Fixes: Add target="_blank" hrefs.

Add target="_blank" to API, Code & Help at the bottom of each page hrefs

This would force the links to always open in a new window, which the user might not want. Leaving them as they are allows the user to open them in a new window if he or she chooses (by ctrl/shift-clicking).


Fix grammar in strings.json

Changes "This not" to "This is not" in strings.json.

Thanks. The change actually won't be propagated since we pull translation on release through lokalize. So the fix needs to be there.  I'll merge this in so next time we make any changes to tokens it's pulled into lokalize though.


Add forced() method and middleware

This PR is a draft. I'm still trying to figure out how to redirect to the 2FA page.

I believe I already built this in a project of us previous of building this package. I can take a look at it if you want @CodeWithDennis

Yeah, go for it! Just a heads-up, there are a few PHPSTAN issues. =(

In the project I have this code:
class RedirectWhen2faNotEnabled
{
    /**
     * Handle an incoming request.
     *
     * @param  \Closure(\Illuminate\Http\Request): (\Symfony\Component\HttpFoundation\Response)  $next
     */
    public function handle(Request $request, Closure $next): Response
    {
        if (!$request->user()?->hasTwoFactorEnabled() && !$request->routeIs('filament.admin.pages.two-factor')) {
            return redirect()->route('filament.admin.pages.two-factor');
        }

        return $next($request);
    }
}

And in my bootstrap/app.php:
    ->withMiddleware(function (Middleware $middleware) {
        $middleware->group('panel:admin', [
            \Illuminate\Session\Middleware\StartSession::class,
            RedirectWhen2faNotEnabled::class
        ]);
    })

I needed the StartSession middleware to be able to have the user in the request I guess.
Will take a further look tonight. @CodeWithDennis

You should be able to edit this PR, so feel free to make any changes! Unless you prefer to rewrite it. =)

Already fixed redirecting in my lunch break :) @CodeWithDennis

Already fixed redirecting in my lunch break :) @CodeWithDennis

Dedication 🔥

@Baspa Does PHPStan command work for you in my branch? Seems to crash on me.

Already fixed redirecting in my lunch break :) @CodeWithDennis

One issue with this when using multi-tenancy:
Missing required parameter for [Route: filament.customer.pages.two-factor] [URI: customer/{tenant}/two-factor] [Missing parameter: tenant].

Already fixed redirecting in my lunch break :) @CodeWithDennis

One issue with this when using multi-tenancy:
Missing required parameter for [Route: filament.customer.pages.two-factor] [URI: customer/{tenant}/two-factor] [Missing parameter: tenant].

Can you try again maybe? :)

@Baspa Does PHPStan command work for you in my branch? Seems to crash on me.

Seem to work for me but it was very slow, I also gave it full memory so that might be the reason why it worked for me..

Must be something on my side.

Can you try again maybe? :)

Same problem, i think this is where i got stuck because filament()->getTenant() is not available in the middleware. Because you could do something shorter like;
$tenant = TwoFactor::getUrl();

TwoFactor::getUrl($tenant);

But we also need to check if the panel is multi tenancy.

Can you try again maybe? :)

Same problem, i think this is where i got stuck because filament()->getTenant() is not available in the middleware.
Shorter:
$tenant = TwoFactor::getUrl(); // Still same issue

TwoFactor::getUrl($tenant);

But we also need to check if the panel is multi tenancy.

Hmmm good one.. I have to think about that

Is it not available in any middleware at all? Or just after a specific middleware? Then we could maybe load that middleware before our middleware? @CodeWithDennis

Is it not available in any middleware at all? Or just after a specific middleware? Then we could maybe load that middleware before our middleware? @CodeWithDennis

I mean it returns null.

Maybe we could use something like this: https://filamentphp.com/docs/3.x/panels/tenancy#applying-middleware-to-tenant-aware-routes ? @CodeWithDennis
https://filamentphp.com/docs/3.x/panels/tenancy#using-tenant-aware-middleware-to-apply-global-scopes

Won't this be the same problem where we can't access the tenant parameter? I think this PR kinda done if we can figure out how we can access it. (Besides the PHPStan issues) @Baspa

@Baspa I haven't had time to check it again yet. Did you come up with any ideas that might solve the problem?

@CodeWithDennis unfortunately I haven't been able to take a look at this as well.. hopefully later this week

It should be fixed right now, can you test it to be sure? @CodeWithDennis

It should be fixed right now, can you test it to be sure? @CodeWithDennis

Yes it works! Couple of questions:

Will it also work without tenancy?
Should we display a message that its forced.

It should be fixed right now, can you test it to be sure? @CodeWithDennis

Yes it works! Couple of questions:

Will it also work without tenancy?
Should we display a message that its forced.

Hmm, when you're not using tenancy the tenantMiddleware is not triggered. We should add another middleware when not using tenancy I guess. Displaying a message would be a idea. @CodeWithDennis

I can imagine that if users are constantly being redirected to this page, it could be confusing, and they might not understand what's happening. It would be good to inform them that the app requires 2FA before they can use it.
I don't have time right now, but if I get a chance and you haven't added it yet, I'll take a look.

I can make a difference in which middleware to use now by also adding withTenancy(). Don't know if that's the best solution but it's quite clean I guess. Didn't have time to checkout the message yet. @CodeWithDennis

I think you can check if the Panel uses Tenancy and grab that.

Unfortunately $panel->hasTenancy() returns false in the TwoFactorAuthPlugin class..


android app cannot load specific 3d tiles data

I try to package a android app with Cesium. 3d tiles model is true origin coordinate,  I choose True Origin mode in CesiumGeoreference, and seturl use local path "file:///c:/tileset/tileset.json".  It is displayed in Windows but not in Android.
this is the 3d tiles:
tileset.zip

Sorry, this is my descrption mistake .  I also tried to provide the 3D tiles file on the web or upload it to Cesium ion, but it still couldn't load


Add CaseIterable conformance for string, int enums

Motivation

Fixes - #548.

Modifications

Add another value in conformances array.

Result

String and Int generated enums will conform by default to CaseIterable protocol.

Test Plan

Adjust OpenAPIGeneratorReferenceTests.

@swift-server-bot test this please

Ah let me look at that failing compat test in a separate PR, I don't believe this was caused by your PR.

Yes that was strange, I was already trying to figure out why the test is failing.

@swift-server-bot test this please


Open local address in default browser

Open local address in default browser by using option --local.
serve --local

👍 I'd rather call it --open

Yeah 😀. It should be `--open'. I will update it.

@leo I have changed --local to --open. Need to update anything else?

@leo Updated all the changes you asked.

Thanks! :v:


User badges system. Catch the 'no data' case

The user badges thing: http://www.missingmaps.org/users/#/  ..should catch the 'no data' case, and say something like
"Sorry. No data for this user.  Since we started gathering data (Feb 2016) we haven't picked up any edits by this user.
Currently it silently fails logging a js error TypeError: undefined is not an object (evaluating 'user.id') rethrowCaughtErrorbundle.js:50119

+1

@kamicut can you have an error message and with a button/link to /contribute

PR open for this currently in the User pages.

That PR was merged. Was it deployed? This basic issue doesn't seem to be fixed (re-open?)

OK now I see it's fixed on the live site. (Seeing this)

Yay!
but ...it's OpenStreetMap not OpenStreetMaps


fullPageservice is calling methods on second click. Single ng-click doesn't call a method.

It got fixed. I was trying o use reBuild() but now understood that before making a ajax call i need t use destroy() and later after storing data in scope variable I need to call reBuild()


show connected message when web phone has connected

Coverage decreased (-0.008%) to 41.581% when pulling 0a24759b247b49939cb2e258ab2ffaca98ad1981 on embbnux:feature/webphone-register-alert into e53b1e5f98b959d8d2c2ed7e6432a9905550b779 on ringcentral:master.


Curried reducers

The idea is to use curried "action-first" reducers like in Elm.
I propose calling them updaters to differentiate from regular reducers. (in Elm the reducing function is by convention called update)
Instead of taking the current state and action, and returning the next state,
an updater takes the action and returns a function that takes the current state and returns the next state.
nextState = updater(action)(currentState)

Compare the signature of a normal reducer to that of an updater:
type Reducer<C, A, S> = (currentState: C, action: A) => S
type Updater<A, C, S> = (action: A) => (currentState: C) => S

The reason why updaters are better suited for Redux, which arguably embraces functional programming, is that they can be composed with any curried "context-last" function, allowing point-free style and declarative reducer creation with libraries like lodash/fp or Ramda.
Consider this example, where we call the updating function with state:
import { update, concat } from 'lodash/fp'

const reducer = (state, action) => {
  if (action.type === 'PUSH')
    return update('list', concat(action.payload))(state)

  return state
}

Same example, as a point-free updater - we return the updating function itself:
import { update, concat } from 'lodash/fp'

const updater = (action) => {
  if (action.type === 'PUSH')
    return update('list', concat(action.payload))

  return state => state
}

Should createStore expect an updater or a reducer by default?
Converting reducers to updaters and vice versa is trivial:
const fromReducer = (reducer) => action => state => reducer(state, action)
const fromUpdater = (updater) => (state, action) => updater(action)(state)

const applyReducerEnhancer = (reducerEnhancer) => (createStore) => (reducer, ...args) => {
  return createStore(reducerEnhancer(reducer), ...args)
}

if the API for createStore is not to be altered, a store enhancer should be bundled:
const adaptStoreForUpdaters = applyReducerEnhancer(fromUpdater)

Usage:
const store = createStore(updater, adaptStoreForUpdaters)

if createStore is to accept updaters by default, regular reducers would have to be decorated with fromReducer to work:
const store = createStore(fromReducer(legacyReducer))

Is this a good idea for Redux core?

Related

redux-fp - helpers for dealing with updaters

It sounds like your library achieves this without modification to the core. I don't think the main library needs to support this directly, as it's mainly a boilerplate reducer and there are varying opinions on how to reduce boilerplate and deal with your reducers as a whole.


[DROOLS-1175] infer numeric type for sum expression in an accumulate …

…pattern

All optaplanner's non-turtle tests succeed, but this DRL fails:
rule "requiredCpuPowerTotal"
    when
        $computer : CloudComputer($cpuPower : cpuPower)
        $requiredCpuPowerTotal : Integer(intValue > $cpuPower) from accumulate(
            CloudProcess(
                computer == $computer,
                $requiredCpuPower : requiredCpuPower),
            sum($requiredCpuPower)
        )
    then
        scoreHolder.addHardConstraintMatch(kcontext, $cpuPower - $requiredCpuPowerTotal);
end

with this exception:
Caused by: Exception executing consequence for rule "requiredCpuPowerTotal" in org.optaplanner.examples.cloudbalancing.solver: java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.Integer
	at org.drools.core.runtime.rule.impl.DefaultConsequenceExceptionHandler.handleException(DefaultConsequenceExceptionHandler.java:39)
	at org.drools.core.common.DefaultAgenda.fireActivation(DefaultAgenda.java:1100)
	at org.drools.core.phreak.RuleExecutor.fire(RuleExecutor.java:121)
	at org.drools.core.phreak.RuleExecutor.evaluateNetworkAndFire(RuleExecutor.java:74)
	at org.drools.core.common.DefaultAgenda.fireNextItem(DefaultAgenda.java:1007)
	at org.drools.core.common.DefaultAgenda.fireLoop(DefaultAgenda.java:1351)
	at org.drools.core.common.DefaultAgenda.fireAllRules(DefaultAgenda.java:1289)
	at org.drools.core.impl.StatefulKnowledgeSessionImpl.internalFireAllRules(StatefulKnowledgeSessionImpl.java:1335)
	at org.drools.core.impl.StatefulKnowledgeSessionImpl.fireAllRules(StatefulKnowledgeSessionImpl.java:1326)
	at org.drools.core.impl.StatefulKnowledgeSessionImpl.fireAllRules(StatefulKnowledgeSessionImpl.java:1307)
	at org.optaplanner.core.impl.score.director.drools.DroolsScoreDirector.calculateScore(DroolsScoreDirector.java:86)
	at org.optaplanner.core.impl.solver.scope.DefaultSolverScope.calculateScore(DefaultSolverScope.java:109)
	at org.optaplanner.core.impl.phase.scope.AbstractPhaseScope.calculateScore(AbstractPhaseScope.java:159)
	at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.processMove(ConstructionHeuristicDecider.java:133)
	at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.doMove(ConstructionHeuristicDecider.java:122)
	at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.decideNextStep(ConstructionHeuristicDecider.java:95)
	at org.optaplanner.core.impl.constructionheuristic.DefaultConstructionHeuristicPhase.solve(DefaultConstructionHeuristicPhase.java:64)
	at org.optaplanner.core.impl.solver.DefaultSolver.runPhases(DefaultSolver.java:237)
	at org.optaplanner.core.impl.solver.DefaultSolver.solve(DefaultSolver.java:198)

This rule:

rule "requiredCpuPowerTotal"
when
$computer : CloudComputer($cpuPower : cpuPower)
$requiredCpuPowerTotal : Integer(intValue > $cpuPower) from accumulate(
CloudProcess(
computer == $computer,
$requiredCpuPower : requiredCpuPower),
sum($requiredCpuPower)
)
then
scoreHolder.addHardConstraintMatch(kcontext, $cpuPower - $requiredCpuPowerTotal);
end

fails with

Caused by: Exception executing consequence for rule "requiredCpuPowerTotal" in org.optaplanner.examples.cloudbalancing.solver: java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.Integer
at org.drools.core.runtime.rule.impl.DefaultConsequenceExceptionHandler.handleException(DefaultConsequenceExceptionHandler.java:39)
at org.drools.core.common.DefaultAgenda.fireActivation(DefaultAgenda.java:1100)
at org.drools.core.phreak.RuleExecutor.fire(RuleExecutor.java:121)
at org.drools.core.phreak.RuleExecutor.evaluateNetworkAndFire(RuleExecutor.java:74)
at org.drools.core.common.DefaultAgenda.fireNextItem(DefaultAgenda.java:1007)
at org.drools.core.common.DefaultAgenda.fireLoop(DefaultAgenda.java:1351)
at org.drools.core.common.DefaultAgenda.fireAllRules(DefaultAgenda.java:1289)
at org.drools.core.impl.StatefulKnowledgeSessionImpl.internalFireAllRules(StatefulKnowledgeSessionImpl.java:1335)
at org.drools.core.impl.StatefulKnowledgeSessionImpl.fireAllRules(StatefulKnowledgeSessionImpl.java:1326)
at org.drools.core.impl.StatefulKnowledgeSessionImpl.fireAllRules(StatefulKnowledgeSessionImpl.java:1307)
at org.optaplanner.core.impl.score.director.drools.DroolsScoreDirector.calculateScore(DroolsScoreDirector.java:86)
at org.optaplanner.core.impl.solver.scope.DefaultSolverScope.calculateScore(DefaultSolverScope.java:109)
at org.optaplanner.core.impl.phase.scope.AbstractPhaseScope.calculateScore(AbstractPhaseScope.java:159)
at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.processMove(ConstructionHeuristicDecider.java:133)
at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.doMove(ConstructionHeuristicDecider.java:122)
at org.optaplanner.core.impl.constructionheuristic.decider.ConstructionHeuristicDecider.decideNextStep(ConstructionHeuristicDecider.java:95)
at org.optaplanner.core.impl.constructionheuristic.DefaultConstructionHeuristicPhase.solve(DefaultConstructionHeuristicPhase.java:64)
at org.optaplanner.core.impl.solver.DefaultSolver.runPhases(DefaultSolver.java:237)
at org.optaplanner.core.impl.solver.DefaultSolver.solve(DefaultSolver.java:198)
at org.optaplanner.examples.common.business.SolutionBusiness.solve(SolutionBusiness.java:306)
at org.optaplanner.examples.common.swingui.SolverAndPersistenceFrame$SolveWorker.doInBackground(SolverAndPersistenceFrame.java:311)
at javax.swing.SwingWorker$1.call(SwingWorker.java:295)
at java.util.concurrent.FutureTask.run(FutureTask.java:266)
at javax.swing.SwingWorker.run(SwingWorker.java:334)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.ClassCastException: java.lang.Double cannot be cast to java.lang.Integer
at org.optaplanner.examples.cloudbalancing.solver.Rule_requiredCpuPowerTotal1027060274DefaultConsequenceInvokerGenerated.evaluate(Unknown Source)
at org.optaplanner.examples.cloudbalancing.solver.Rule_requiredCpuPowerTotal1027060274DefaultConsequenceInvoker.evaluate(Unknown Source)
at org.drools.core.common.DefaultAgenda.fireActivation(DefaultAgenda.java:1089)
... 25 more

It fails against master too, but I 'd expected it to work with this change.

I just figured out why: there's a BigDecimalSumAccumulateFunction, but no IntSumAccumulateFunction or LongSumAccumulateFunction yet. I 'll try implementing those latter and send you a PR for those :)

If you've seen any previous mails with stacktraces: those don't count - I had another local branch vs your branch name conflict issue locally. I deleted all those from the Github page, so only relevant comments are still on this github PR page.
There's still an issue: with a sum of integers, the call to IntegerSumAccumulateFunction.accumulate(Serializable context, Object value) actually delivers a Long value (which is nicely cast to an int, except in my drools fork (which fixes some bugs) where it triggers a ClassCastException).
It shouldn't be deliver an Long value. For example, in this rule:
rule "requiredCpuPowerTotal"
    when
        $computer : CloudComputer($cpuPower : cpuPower)
        $requiredCpuPowerTotal : Integer(this > $cpuPower) from accumulate(
            CloudProcess(
                computer == $computer,
                $requiredCpuPower : requiredCpuPower),
            sum($requiredCpuPower)
        )
    then
        scoreHolder.addHardConstraintMatch(kcontext, $cpuPower - $requiredCpuPowerTotal);
end

with this class
public class CloudProcess {
    private int requiredCpuPower;

    public int getRequiredCpuPower() {
        return requiredCpuPower;
    }
}

That sum value should definitely be an int.

I've send in a pull request for your branch that fixes the BigDecimal rounding issue and exposes the Long casting issue: https://github.com/mariofusco/drools/pull/1

@ge0ffrey I merged your PR into mine but I still have a doubt: can we safely assume that an accumulator of Numbers of a given class will be exclusively fed by instances of that class? E.g. is it correct that an accumulator on BigDecimal eventually throws a CCE if for some reason is passed with a double? I must admit that at the moment I have no evidence of how this could happen, I just wanted to be sure that this is acceptable. Also what happens if it is passed with a null? I'm afraid this is far more probable and at the moment it will throw a NPE. What should we do in that case? Propagate the NPE? Considering it as a zero?
That said if I have understood correctly you're saying that this current implementation triggers an error in that requiredCpuPowerTotal rule? If so how could I reproduce this issue?

This PR fails, probably because of the ClassCastExceptions that are happening. Before we revert those casting and head back to doubleValue() (which is a rounding error when done some BigDecimals and even some longs), we might want to check if we fix that and keep the casts.
About a double being passed to a BigDecimal accumulate: isn't that impossible because the compile time model guarantees that the parameter is of type BigDecimal? It's like saying that the method AtomicLong.addAndGet(long delta) should internally deal with delta not being a long but a double, no?
Yes, our unit tests now break with ClassCastException, but isn't that normal as they're trying to put ints in a Double accumulation etc? Maybe for backwards compatility we should allow the double accumulation to avoid the cast if it's not instanceof and fallback to doubleValue().
About nulls: if for backwards compatibility you want to keep null + null = null and null + 7 = 7 etc, I wouldn't mind, but otherwise I 'd just fail fast on that too.
Fail fast isn't bad: it's just saying we're not allowing you to do that now because we believe you'll only use it unintended in error - and once someone can deliver a use case that does really need it, we 'll put it in there in the next version. Josh Bloch's advice for any feature: When in doubt, leave it out. (especially edge cases are good fail fast candidates until proven useful).
Let's discuss Wednesday afternoon or Friday when I back in the home-office?

Ok, let's discuss this when you'll be back at home. For now I just want to give a look at that CCE you mentioned (and possibly fix it if I'll have time). How can I reproduce it?

To reproduce run AccumulateTest or any of the others that are failing on Jenkins.

Replaced by https://github.com/droolsjbpm/drools/pull/867


We should always close the browser

Remote browser or not, we should always close the browser.
Closing the browser will actually tell the remote browser provider that the session is to be ended (logs to be consolidated and session terminated), whereas only disconnecting might keep the session hanging for no reason.
Motivation/Description of the PR

Description of this PR, which problem it solves
When executing Codeceptjs/Puppeteer test suite using the browserstack grid, the session hang for 15min after the test suite finished executing and logs are not properly uploaded to the plateform.
Resolves #issueId (if applicable).
I don't have an issue ID as I identified the bug at the same time, I fixed it.

Applicable helpers:

[ ] WebDriver
[X] Puppeteer
[ ] Nightmare
[ ] REST
[ ] FileHelper
[ ] Appium
[ ] Protractor
[ ] TestCafe
[ ] Playwright

Applicable plugins:

[ ] allure
[ ] autoDelay
[ ] autoLogin
[ ] customLocator
[ ] pauseOnFail
[ ] coverage
[ ] retryFailedStep
[ ] screenshotOnFail
[ ] selenoid
[ ] stepByStepReport
[ ] wdio
[ ] subtitles

Type of change

[ ] :fire: Breaking changes
[ ] :rocket: New functionality
[X] :bug: Bug fix
[ ] :clipboard: Documentation changes/updates
[ ] :hotsprings: Hot fix
[ ] :hammer: Markdown files fix - not related to source code
[ ] :nail_care: Polish code

Checklist:

[ ] Tests have been added
No test added as it would require credentials for a remote browsers provider.
[x] Documentation has been added (Run npm run docs)
[x] Lint checking (Run npm run lint)
[x] Local tests are passed (Run npm test)
npm test -> 2 tests fail but they fail anyway on branch 3.x on my system

Actually, I'm actually arguing with myself whether disconnect should be handled by the remote browser provider as a proper termination just like close...
The documentation is not very clear on that point.

I don't know. I would try to use it with servers like Selenoid or Moon to check what is the right command here.
They suggest close even they previously used connect
https://github.com/aerokube/moon-cloud-puppeteer-example/blob/master/index.js#L20

Thanks! Merging!


vue-router 2.0 Docs

Inspired by @chrisvfritz https://github.com/vuejs/vuejs.org/issues/319

A new branch next-doc will be created from next
Guidelines for writing official Vue docs
This is a draft of a proposal for vue-router 2 documentation. If I'm missing something
you'd really like to see, or I missed anything, please leave a comment and let me know :smile:
Also, I'd love your help with titles :wink:
For better consistency, we should be using the same rules as https://github.com/vuejs/vuejs.org/issues/319
Contributing

Edit this comment to change unclaimed to @yourUsername so everyone knows you're working on  a file.
Commit directly to the next-doc branch
Try to edit only one file (the one you claimed) at a time.
When ready to push, git fetch origin next-doc, git rebase origin/next-doc, then git push origin next-doc. (Yes, we're skipping code review for now. Let's just get something in place for this first pass.)
Update this comment to put a check next to the file you updated.

Guide Update
I think we could reorganize the first section to match the style of Vue.js Guide.
Links refer to doc's previous version. Some of them may have to be rewritten from scratch like the Router Hooks.
A missing link means a new page

[ ] @posva Installation
[ ] unclaimed Getting Started

[ ] @fnlCtrl Basic Usage
[ ] unclaimed Route Object & Route Matching
[ ] unclaimed Imperative Navigation (About history manipulation)
[ ] unclaimed Declarative Navigation with router-link
[ ] unclaimed router-view

[ ] unclaimed Advanced Routing

[ ] unclaimed Nested Routes
[ ] unclaimed Named Views
[ ] unclaimed Lazy loading Routes
[ ] unclaimed Route Resolving

[ ] unclaimed Navigation

[ ] unclaimed Router Hooks
[ ] unclaimed Routes Hooks
[ ] unclaimed Views Transitions

API Update
Even though this is separated from the Guide, it's intended to be on the same page as it is right now.
TODO

[ ] API Reference

[ ] Router constructor

I've edited the Contributing part, changing branch name 2.0 to next-doc

Btw I think examples in the Guide should be directly adapted from https://github.com/vuejs/vue-router/tree/next-doc/examples if a corresponding one exists, for consistency

@fnlctrl Absolutely. We should inspire from them as much as possible for consistency.
Did you push your contribution? There're no new commits on the next-doc branch

Just started writing, haven't pushed yet :)

@posva The Getting Started section is complete :smile:.
Would you like to review Imperative Navigation And History Manipulation since it's written from scratch, and I don't know if there's anything I missed :joy:.

Sure, I'll give it a read. I think we should rename it to Programmatic Navigation (without History Manipulation)
I may give it a read tomorrow. I'll try to advance on advanced routing section
Also, I think we should add a section about Best/Recommended Practices. People have been asking some guidance when building large apps

Yeah, Best Practices would be useful!
I'll continue to update Router Hooks tomorrow then.

Forgot to ask, should I move the History Manipulation part to Advanced Routing?

Not, let's keep it. Removing it from the title because it's redundant imo

Ok then, doint it now.

Since docs are online now, I ticked off the relevant tasks that were still open and unclaimed.
@psova, you started working at the types, is that still ongoing? Will those actually be added to the docs? Or can we close this ticket as docs are online now?

@LinusBorg I totally forgot about this! We can close it now 😄


Implement toArray method in the core Vector class to support prototype.js Array.from overload

The lates version on IAB TCF library is not working well with prototype.js library that some of our clients are still using.
The issue is  Array.from function that prototype.js is overloading with there following code
function $A(iterable) {
  if (!iterable) return [];
  if ('toArray' in Object(iterable)) return iterable.toArray();
  var length = iterable.length || 0, results = new Array(length);
  while (length--) results[length] = iterable[length];
  return results;
}
Array.from = $A;

In this case  the core library Vector class object produce the empty array in createVectorField method of TCData class
return [...vector].reduce<BooleanVector>((booleanVector, keys: IdBoolTuple): BooleanVector => {

      booleanVector[keys[0].toString(10)] = keys[1];
      return booleanVector;

}, {});

Because the core Vector class does not have toArray and length properties the expression [...vector] that will call Array.from function with prototype.js code will always produce empty array
[...vector] -> Array.from(vector) -> []

The solution that this PR is implementing is to introduce  toArray public method in Vector class in IAB TCF core library
public toArray(): Array<IdBoolTuple> {

    const result = new Array<IdBoolTuple>(0);

    for (let i = 1; i <= this.maxId; i++) {

      result.push([i, this.has(i)] as IdBoolTuple);

    }
    return result;
}

This will be a resurfacing issue when trying to implement the official library in the future, correct?
The thing is that we currently don't have a way to override what prototype.js does to Array.from and this is why we are going this way?

This will be a resurfacing issue when trying to implement the official library in the future, correct?
The thing is that we currently don't have a way to override what prototype.js does to Array.from and this is why we are going this way?

This is our fork of the official library that we have updated recently to the latest version. Unfortunately after the deployment to the production at leat one client that is using prototype.js and __tcfapi reported the issue - TCData created with empty vendor and purpose objects (consents and legitimateInterests are {}).
After the debugging of the official library we have found the issue -- prototype.js & Array.form. The  code with Symbol and synchronous iteration (yield) that called by [...vector] is not working good in this case.
export class Vector extends Cloneable<Vector> implements Iterable<IdBoolTuple> {
...
  public* [Symbol.iterator](): Iterator<IdBoolTuple> {

    for (let i = 1; i <= this.maxId; i++) {

      yield [i, this.has(i)] as IdBoolTuple;

    }
  }
}

We can't override what prototype.js is doing in WEB SDK but what we can is to update our fork of IAB TCF core library to be compatible with prototype.js. This is what the PR is doing.
We have another task to eliminate the Didomi fork and start using the official library but in this case we need to create issues and PRs for all our changes including this one.
Hope it makes sense and answer your questions.


Enable full build in Docker container

Add a target to build everything (even glide stuff) via Docker
Also adds "--strip-vendor" to glide stuff.
Signed-off-by: Doug Davis<EMAIL_ADDRESS>

I actually would prefer to have a single top-level makefile but that's for a different time.
I'm also not convinced this is the best way to do this with the current structure, so if someone can offer an alternative I'm open to it.
My goal is to allow people to build w/o installing anything - no glide, no go, no lint.... - just git and docker.

All my concerns addressed. LGTM (LGTM1 label)

ok I used that RUN cmd instead - thanks
-Doug

On Dec 8, 2016, at 10:51 AM, Kent Rancourt<EMAIL_ADDRESS>wrote:
@krancour commented on this pull request.
In hack/Dockerfile https://github.com/kubernetes-incubator/service-catalog/pull/122:

@@ -0,0 +1,3 @@
+FROM golang:1.7.3
+RUN curl https://glide.sh/get | sh
this isn't root on your machine, its just root in the container - so its safe

Docker containers share the host machine's kernel. Do not overestimate how safe it is to run things as root in a container.
So, I agree with @pmorie https://github.com/pmorie's point. Here's a safer way to install a specific version of glide (which is important for the sake of determinism).
I've cribbed this from the Dockerfile for Deis' containerized development environment https://github.com/deis/docker-go-dev/blob/master/rootfs/Dockerfile:
...
ENV GLIDE_VERSION=v0.12.3
...
RUN curl -sSL https://github.com/Masterminds/glide/releases/download/$GLIDE_VERSION/glide-$GLIDE_VERSION-linux-amd64.tar.gz
| tar -vxz -C /usr/local/bin --strip=1
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/kubernetes-incubator/service-catalog/pull/122, or mute the thread https://github.com/notifications/unsubscribe-auth/AB2sXzm5fiHt-pf1yOcUCU_7WzHIfX_2ks5rGCecgaJpZM4LGpkE.

What's progress on this? Can we pull out the --strip-vendor into a separate PR? I do not see any discussion on that point. It seems uncontroversial.

This looks fine to me. adding LGTM2 label

LGTM, going to merge this to keep it from getting stale.  Open a PR if there are follow-ups needed re: strip-vendor.

For the record, here are the docs on the k8s docker build: http://releases.k8s.io/HEAD/build-tools/README.md


Create automated action that updates fall back image when renkulab-docker has a new release

If I launch a session now with the default image (because the image build failed, for example) I get renku/singleuser:latest which is super old. We should update this to start with the image that is the base image for the current template version.

Addressed by #803
@rokroskar if the image that comes with the request to launch a session is not available the notebook service cannot do that much. We could clone the repo and check the Dockerfile and then try to use the base image but I don't think this will be very efficient to do with every request to launch a session.
So I think we are stuck with having a hardcoded value.

yes, my point was just that the hard-coded value should be updated, ideally by CI triggered in the renkulab-docker repo.

@rokroskar then I will rename this issue to say something like: "Create automated action that updates fall back image when renkulab docker has a new release".

yeah and tbh it shouldn't even be in this repo then... I just really wanted a one-liner that updated the image now and handle the full solution later. But we can just move the issue to renkulab-docker if you want


Add authentication example

Part of #42.

I think this is done in terms of code. It just needs more documentation.


change image

👷 Deploy Preview for bochen1993 processing.

Name
Link

🔨 Latest commit
7a6956986f664df5488f9ac155085eca5f3cd7fb

🔍 Latest deploy log
https://app.netlify.com/sites/bochen1993/deploys/650efbd50315710008e89300


Fix 174

Overview
This PR fixes #174
Changes:

Added a notebook that uses ipyleaflet and the RAVEN hydrobasins selection WPS
Better handling for lonlat tuples sent via WPS

Related Issue / Discussion
Additional Information
Thanks to @davidcaron for his examples on how to set up ipyleaflet!
Links to other issues or sources.

Nice !
Suggestions:

Add a text intro to the notebook: This is what we're going to do here.
remove print(version) in first cell unless there is a reason to do so.
Add text cell instructing users to move marker in Lac St-Jean before going further
the dash-array contour is confusing, maybe use a smooth contour
Remove plotting of additional watersheds
Display the total area on the map ?

Done.


Automatically open project after creation

This isn't currently happening. Actually, I don't know if this should or should not happen, but it's indeed very useful(if not always, at least an option that lets you decide wheter or not do it).
I took a look at the source and found out that I probably could add this feature myself(eventually a PR) but since I'm not very familiar with sublime plugins I'm firstly reporting here this thought.

@Ball-Man Have many thanks for your interest in this plugin and also for offering a PR.
Indeed this would be a highly desirable feature.
Hence it is implemented via #53
Also duplicate of #44


Non-docker installation guide?

Is it too much trouble to outline the steps to take if a user wanted to install signalserver without using Docker?

I started running manage.py to install, and I installed django, celery but hit a wall when trying to import a module named "rest_framework."

I remember we had a discussion about whether to use Python 2.7 or 3, since they're quite different, but I don't remember which we settled on so I installed under both.

Oh! Okay I installed everything that I found in requirements.txt and could run manage.py, but unsure what to do next.

I honestly don't think it is a good idea to install SignalServer without Docker. I also designed the app by using docker in mind too. Please let me know what your thoughts are after you get it up running the signalserver without docker.

I can understand this for people wanting to implement signalserver but it's very hard to develop if I have to rebuild the docker container (which takes around 5-8 minutes) every time!

Alternatively could frontend.sh be optimized so that components that could be skipped are skipped rather than re-running all of frontend.sh at each call.

I've found that I very rarely need to run frontend.sh and the .css file seems to re-render without running anything with a hard reload. The html pages are usually what get restructured.

Instead @yayoiukai can you test and provide instructions for a way to edit as a developer by running the container with docker run?

I imagine it should go something like this:

Run .quickstart.sh
Run docker images --all
Find hash for signalserver_worker
Run docker run my_image_hash -p 8000:8000

but I don't really know much about Docker or this particular framework setup...

Now I am investigating if running the first command in frontend.sh and then only having to run the second will work!

No, this doesn't update the HTML or seem to do anything new.

frontend.sh and quickstartdev.sh is for developing (kind of). I will update the README.

Hey Ashley, if that helps, here is how I develop on the application:
if changes are in the Django. (anything other than inside frontend/), I just run quickstartdev.sh to rebuild the docker container and it's been reasonably fast. (I would say less than 2 minutes.) Also,
One other thing: don't delete containers or images every time, otherwise you're losing the cache which helps make builds faster. Notably, Postgres or RabbitMQ containers almost never need to be rebuilt.
For frontend.sh, unfortunately, it doesn't seem as reliable as I would like to. (Ideally the frontend directory gets updated, I need to track why this sometimes isn't the case).
Let me know if that helps - let me know if there's parts you think we can improve as far as developer experience is concerned, and I'll be happy to look into it.

I think this  related, but I have to rerun quickstart.sh if I want to see my changes in the localhost and it takes a little over 2 minutes to do (I used a stopwatch!).
It's definitely a drag on keeping a steady pace, especially on the small updates I'm making.

Considering I'm not making Postgres or RabbitMQ, isn't there a way I can use the docker run command to run the application so we can get dynamic (via page refresh) updates of the rendered views and frontend css/js components?

./quickstartdev.sh is a lot faster, but we still definitely need documentation on what all the dependencies of the application are and I'd still much prefer to just run a local setup. I don't expect all users to do this or want to do this, but I do expect all developers will want to.

+1 on non-docker installation guide. It might be hard to find the time to develop a fleshed-out manual at this juncture, but is there a way just to simply list the steps for now?  Or even just the dependencies?

Yes. I think it is too much to write out how to install Singalserver without docker. So for deployment, I don't consider dropping docker nor it would be any useful to write instruction without docker for just to install in a machine. Given said that, there is nothing stopping you trying out deploying Signalserver on your server or machine without docker. So please feel free to try it out. If you think it is easier than using docker, I actually like to know.
Here is the software used for SignalSever. (It should work with all the latest version if it is not noted)
Backend
Python 3.5
Django
Postgres- 9.5
Celery
RabbitMQ
Radis
Frontend
Node
Bower
D3
jquery
jquery date time picker
bootstrap

I wanted to reopen this so we can work on a more complete guide moving forward. I just learned Docker isn't going to work on any of our capture stations - which is fine in general as I realize Sig is meant for server installation. Assigning to Laura B to facilitate.

I don't understand the use case. Having separate installation on capture stations doesn't sound like something worth recommending. The capture stations should use qctools/qcli to send reports to a signalserver installation which lives on a server.

I can remove the use case from the ticket if that doesn't make sense - is
it still worth  working towards a more complete non-docker installation
guide?
On Fri, May 12, 2017 at 1:03 PM, Dave Rice<EMAIL_ADDRESS>wrote:

I don't understand the use case. Having separate installation on capture
stations doesn't sound like something worth recommending. The capture
stations should use qctools/qcli to send reports to a signalserver
installation which lives on a server.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/bavc/signalserver/issues/166#issuecomment-301172718,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI_8Ud6ivsim_Q7o23mVafj6QmZtHRGGks5r5LsLgaJpZM4L-nMY
.

--
Kelly Haydon
Preservation Manager
Bay Area Video Coalition
415-558-2158 |<EMAIL_ADDRESS>twitter/instagram: @bavcpreserve
Where do you find a VR demo, career advice, a discount on SF's most popular
classes and spiked lemonade? BAVC's Open House! May 13th 1-4pm
Black Lives Matter! I stand with BLM!

Mostly my personal situation just prompted me to reconsider this
On Fri, May 12, 2017 at 1:08 PM, Kelly Haydon<EMAIL_ADDRESS>wrote:

I can remove the use case from the ticket if that doesn't make sense - is
it still worth  working towards a more complete non-docker installation
guide?
On Fri, May 12, 2017 at 1:03 PM, Dave Rice<EMAIL_ADDRESS>wrote:

I don't understand the use case. Having separate installation on capture
stations doesn't sound like something worth recommending. The capture
stations should use qctools/qcli to send reports to a signalserver
installation which lives on a server.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/bavc/signalserver/issues/166#issuecomment-301172718,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI_8Ud6ivsim_Q7o23mVafj6QmZtHRGGks5r5LsLgaJpZM4L-nMY
.

--
Kelly Haydon
Preservation Manager
Bay Area Video Coalition
415-558-2158 <(415)%20558-2158> |<EMAIL_ADDRESS>twitter/instagram: @bavcpreserve
Where do you find a VR demo, career advice, a discount on SF's most
popular classes and spiked lemonade? BAVC's Open House! May 13th 1-4pm
Black Lives Matter! I stand with BLM!

--
Kelly Haydon
Preservation Manager
Bay Area Video Coalition
415-558-2158 |<EMAIL_ADDRESS>twitter/instagram: @bavcpreserve
Where do you find a VR demo, career advice, a discount on SF's most popular
classes and spiked lemonade? BAVC's Open House! May 13th 1-4pm
Black Lives Matter! I stand with BLM!

I'm not sure about non-docker installation, but I kinda feel like
with-Docker installation is complicated. I don't know anything about web
apps, but is there some way to package the signal server part of it
(frontend.sh and quickstart.sh) for more user-friendliness? I know this is
prob not a priority, and that the nature of signalserver entails a certain
level of tech support, but there's barrier to entry here that shouldn't be
ignored. I imagine that some of the organizations signalserver was designed
to help might not ever have that kind of tech support; maybe they could
scrape together enough money for a decently-sized vendor project and some
storage, but they're on their own in many ways.
just a thought...
On Fri, May 12, 2017 at 4:10 PM, metacynic<EMAIL_ADDRESS>wrote:

Mostly my personal situation just prompted me to reconsider this
On Fri, May 12, 2017 at 1:08 PM, Kelly Haydon<EMAIL_ADDRESS>wrote:

I can remove the use case from the ticket if that doesn't make sense -
is
it still worth  working towards a more complete non-docker installation
guide?
On Fri, May 12, 2017 at 1:03 PM, Dave Rice<EMAIL_ADDRESS>wrote:

I don't understand the use case. Having separate installation on capture
stations doesn't sound like something worth recommending. The capture
stations should use qctools/qcli to send reports to a signalserver
installation which lives on a server.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
<https://github.com/bavc/signalserver/issues/166#issuecomment-301172718
,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AI_8Ud6ivsim_
Q7o23mVafj6QmZtHRGGks5r5LsLgaJpZM4L-nMY>
.

--
Kelly Haydon
Preservation Manager
Bay Area Video Coalition
415-558-2158 <(415)%20558-2158> <(415)%20558-2158> |<EMAIL_ADDRESS>

twitter/instagram: @bavcpreserve
Where do you find a VR demo, career advice, a discount on SF's most
popular classes and spiked lemonade? BAVC's Open House! May 13th 1-4pm
Black Lives Matter! I stand with BLM!

--
Kelly Haydon
Preservation Manager
Bay Area Video Coalition
415-558-2158 <(415)%20558-2158> |<EMAIL_ADDRESS>twitter/instagram: @bavcpreserve
Where do you find a VR demo, career advice, a discount on SF's most popular
classes and spiked lemonade? BAVC's Open House! May 13th 1-4pm
Black Lives Matter! I stand with BLM!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bavc/signalserver/issues/166#issuecomment-301174161,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIkiXp53X6motB3qzi0ClKbpNCOyXQPxks5r5LyhgaJpZM4L-nMY
.

--
Benjamin Turkus | The New York Public Library
Assistant Manager for Audio and Moving Image Preservation
Barbara Goldsmith Preservation Division
40 Lincoln Center Plaza, New York, NY 10023
T.<PHONE_NUMBER> <(212)870-1609> |<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.nypl.org/preservation
https://twitter.com/NYPLPreserve
Lifelong Learning | Advancing Knowledge | *Strengthening Our
Communities *

docker is intended to automate the building of the environment. If a user thinks that a docker-managed installation is complicated, then a non-docker installation will be VERY complicated as you'll have to install all the separate components. A non-docker installation would more likely be for users that want to customize the environment or do testing, not really for users seeking simplicity.
The goals of making the docker-based installation more foolproof and having a non-docker installation should be separated issues as they don't really overlap.

Hi all, this issue seems to have fizzled out. It's my opinion that documentation for non-docker installation in beyond the scope of this project. Unless anybody strongly disagrees with me on this notion this ticket will be closed by Laura B.

+1, doing all the work to recreate what docker does is a very large effort


[Cognitive Services Text Analytics API ] API Review

New API Review meeting has been requested.
Service Name: Cognitive Services Text Analytics API
Review Created By: Quentin Robinson
Review Date: 10/22/2024 01:00 PM PT
Release Plan:
PR:
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: Not Provided
Description: Preview API for Analyze-Conversations: Mask PII with entity type labels
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Meeting updated by Quentin Robinson
Service Name: Cognitive Services Text Analytics API
Review Created By: Quentin Robinson
Review Date: 10/22/2024 01:00 PM PT
Release Plan:
PR: https://github.com/Azure/azure-rest-api-specs/pull/30954
Hero Scenarios Link: Not Provided
Core Concepts Doc Link: here
Description: Preview API for Analyze-Conversations: Mask PII with entity type labels
Detailed meeting information and documents provided can be accessed here
For more information that will help prepare you for this review, the requirements, and office hours, visit the documentation here

Notes from API Review 10/22/24

Planning to split Language into multiple services, maybe early next year
Offset and length of masked entities should use our new guidance for string index/length

api-guidelines/azure/Guidelines.md at vNext · microsoft/api-guidelines

Both the GA and preview look good to go once the above have been addressed.


Rc/8.2.1

Release severity: Normal
Release notes:
Support for new runtime upgrades and security improvements are here in Nova Wallet v8.2.1
– Added support for new runtime upgrade affecting nomination pools
– Added security improvements for the DApp browser – Nova now rejects access to certain top level domains and warns users to be careful when connecting to DApps which are not already in the DApp catalog
– Fixes & Optimizations


Efficient of Multi-scale?

Dear Kamnitsask,
For multiple scale scheme, you used both original resolution and down-sample resolution to improve segmentation. I have a question related it
Let's assume input resolution is 1x1x1 mm. If we down-sample it 2x times, then the resampled image has resolution 2x2x2 mm. In your architecture, both resolutions are used. The label to compute the loss still used original resolution 1x1x1 mm. My question is that do you think your strategy will work well when input size is 2x2x2 mm, instead of 1x1x1 in the testing phase. I meant that given an image 2x2x2 mm, does your network still provide a good result? Because 2x2x2 mm is already trained using your strategy.
Thanks in advance

Hi John,
The network "looks" at the input via different scales (resolutions) "simultaneously".
By default, it looks at the "normal" resolution (whatever the input is) AND the downsampled (x2 by default) (AND downsampled by x3 in the current version of the code, AND any more scales if you specify more).
If the original resolution of the input is 1x1x1, this means the networks "looks" at it at the same time both as a 1x1x1, and as a 2x2x2 and as a 3x3x3.
If at test time you input an image with spacing 2x2x2, then it will be "looked at" as a 2x2x2 AND as a 4x4x4 (AND as a 6x6x6 in current default).
Your input can be of any spacing. What you need to ensure is that TRAIN and TEST images are of the same resolution. If you train with 1x1x1 spacing and test with 2x2x2 it wont work. You can train with 2x2x2 and test with 2x2x2. Or upsample the (2x2x2) to (1x1x1) and train & test with them.
I hope this clarifies things.
K


Update about-your-personal-dashboard.md

Why:
Closes:

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[ ] A subject matter expert (SME) has reviewed the technical accuracy of the content in this PR. In most cases, the author can be the SME. Open source contributions may require a SME review from GitHub staff.
[ ] The changes in this PR meet the docs fundamentals that are required for all content.
[ ] All CI checks are passing.

Unsubscribe
On Mon, 02 Dec 2024, 12:25 github-actions[bot], @.***>
wrote:

Closed #35499 https://github.com/github/docs/pull/35499.
—
Reply to this email directly, view it on GitHub
https://github.com/github/docs/pull/35499#event-15490716115, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BNL3BCCH2Z64Y7GYTJXE3GL2DQYS7AVCNFSM6AAAAABSZBWRW2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGQ4TANZRGYYTCNI
.
You are receiving this because you commented.Message ID:
@.***>

For example, the following workflow triggers on the repository_dispatch event and uses information from the event payload to build the matrix. When a repository dispatch event is created with a payload like the one below, the matrix version variable will have a value of [12, 14, 16]. For more information about the repository_dispatch trigger
Update Actions discussion category template ·<EMAIL_ADDRESS>


forcePlainTextOnPaste doesn't work

Text copied while this option is set to true is still copied with styling.

+1

+1

+1

Just found out that the functionality was added to the repository on commit: 3ac688fba67c06d5bbc722d4e452b842f93dee75
But the npm package was never updated.

+1

+1

When will this be added to the NPM package? otherwise - is there a way to work around this?

When will this be added to the NPM package? otherwise - is there a way to work around this?

I guess you could have NPM refer to the specific commit. A workaround would be that you implement the required code yourself. When you look into 3ac688f, you'll see that basically the paste-method must be somehow overwritten.
Without diving into the code too much I wouldn't know how to add the eventListener to the component, but instead you could add an eventListener to the document, since you're only required to call execCommand on document.
So instead of using
import wysiwyg from "vue-wysiwyg";
Vue.use(wysiwyg, {
    forcePlainTextOnPaste: true
});

You could end up with something like:
import wysiwyg from "vue-wysiwyg";
Vue.use(wysiwyg);
document.addEventListener('paste', function (e) {
    e.preventDefault();
    document.execCommand("insertHTML", false, e.clipboardData.getData("text/plain"));
});

Hi @chmln
could you kindly add the commit 3ac688f (#72)  to the npm package or should we find a workaround?
All the best

Use this if you want: https://www.npmjs.com/package/vue-wysiwyg-fixed

@fayt81  @Latentimagedesign my bad for taking a while, I will push a release out very soon.

@chmln, any time now would be sweet!

Yes, please update https://www.npmjs.com/package/vue-wysiwyg if you are able to @chmln! I also need forcePlainTextOnPaste working for my project. Thank you!


osxphotos export error when'd destination folder becomes empty when using  --ramdb --cleanup --update [low priority]

Before submitting a bug report, please ensure you are running the most recent version of osxphotos and that the bug is reproducible on the latest version
osxphotos, version 0.68.5
Python 3.11.5 (main, Aug 24 2023, 15:23:30) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
macOS 12.7.6, x86_64

Describe the bug

Not a major issue as it pertains to a very specific outlier case ;)
osxphotos export error when'd destination folder becomes empty when using  --ramdb --cleanup --update
The key being --ramdboption:

with it: it crashes as it tries to write .osxphotos_export.db into the export folder, which is no longer there (was deleted via the --update/--cleanup options)
without it: it does not crash as --update/--cleanup do not delete the folder given they see .osxphotos_export.db file in it.

To Reproduce

Run an export command which results in 0 pics being exported without ramdb

Without --ramdb

$ rm -fr bursts ; mkdir -p bursts && caffeinate osxphotos export bursts --update --cleanup  --to-date "2015-12-22T23:59:59" --library /Users/XXX/Pictures/Import.NAS2Library.photoslibrary/
(...)
$  ls -a bursts/
.			..			.osxphotos_export.db

With --ramdb

$  rm -fr bursts ; mkdir -p bursts && caffeinate osxphotos export bursts --ramdb --update --cleanup  --to-date "2015-12-22T23:59:59" --library /Users/XXX/Pictures/Import.NAS2Library.photoslibrary/
(...)
Did not find any photos to export
Cleaning up /Users/XXX/Documents/GitHub/Export/recreate/bursts
Deleted: 0 files, 1 directory
Something went wrong and osxphotos encountered an error:
RetryError[<Future at 0x118056fd0 state=finished raised OperationalError>]
Writing export database to /Users/XXX/Documents/GitHub/Export/recreate/bursts/.osxphotos_export.db
╭─────────────────────────────── Traceback (most recent call last) ────────────────────────────────╮
│ /Users/XXX/Documents/Apps/osxphotos/venv3.11/lib/python3.11/site-packages/tenacity/__init__.py:3 │
│ 82 in __call__                                                                                │
│                                                                                │
│   379 │   │   │   do = self.iter(retry_state=retry_state)                                        │
│   380 │   │   │   if isinstance(do, DoAttempt):                                                  │
│   381 │   │   │   │   try:                                                                       │
│ ❱ 382 │   │   │   │   │   result = fn(*args, **kwargs)                                           │
│   383 │   │   │   │   except BaseException:  # noqa: B902                                        │
│   384 │   │   │   │   │   retry_state.set_exception(sys.exc_info())  # type: ignore[arg-type]    │
│   385 │   │   │   │   else:                                                                      │
│                                                                                │
│ /Users/XXX/Documents/Apps/osxphotos/venv3.11/lib/python3.11/site-packages/osxphotos/export_db.py │
│ :1189 in write_to_disk                                                                           │
│                                                                                │
│   1186 │   │   │   │   sqlite_delete_dbfiles(self._dbfile)                                       │
│   1187 │   │   │                                                                                │
│   1188 │   │   │   # write the new database to disk                                              │
│ ❱ 1189 │   │   │   conn_on_disk = sqlite3.connect(                                               │
│   1190 │   │   │   │   str(self._dbfile), check_same_thread=SQLITE_CHECK_SAME_THREAD             │
│   1191 │   │   │   )                                                                             │
│   1192 │   │   │   conn.backup(conn_on_disk, pages=1)                                            │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
(...)
$ ls -a bursts
ls: bursts: No such file or directory

Expected behavior
No crash. Either:

keep the destination folder with the .osxphotos_export.db file in it -- could be good to save historical info from the DB.
or remove the destination folder entirely without an error.

Screenshots
N/A
Desktop (please complete the following information):
osxphotos, version 0.68.5
Python 3.11.5 (main, Aug 24 2023, 15:23:30) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
macOS 12.7.6, x86_64

Additional context
N/A

Interesting -- I think the real issue is that osxphotos deleted the export directory with --cleanup if it was empty (nothing exported), which it should not do.

Yes. And possibly is a cleaner and easier solution yes.


adjust minSplitSize to 64K

I'm using reedsolomon for UDP packet(<1500Bytes for each packet), I don't want multiple goroutines started for each packet I received, it's too expensive.  I think setting the minSplitSize to 64K will be appropriate.

any news on this?

I am hesitant to accept without more investigation. I don't like the fact that this will mostly kill any level 1 cache that is probably present on the CPU. However, since access is likely only read-once, it might not have a negative impact after all.
I will do a few tests. There have been requests to be able to control goroutines in a more finegrained manner, so maybe it could be handled with some creation options.

I have added a way of specifying options in #46 - if possible, please take a look.

https://godoc.org/google.golang.org/grpc#DialOption
how about using closure style option like grpc-go.
(current modification is enough for my use case, just wondering if it may better. )

I tried something similar, but didn't have the unexported function. Nice touch. I will try to see if I can mimmick that.

Thanks for the suggestion @xtaci - it made it a lot neater. I have updated the PR.  Updating default is gone. Having package level state is bad anyway.

much neater!
close this for now, waiting for your merge.


Misc. fixes

Product Version(s):

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@girishshankaran Can you take a look?


No problem !


Console warning in firefox "mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create"

I'm seeing this in the console in firefox: "mutating the [[Prototype]] of an object will cause your code to run very slowly; instead create the object with the correct initial [[Prototype]] value using Object.create".
This can be observed in the Shepherd demo http://github.hubspot.com/shepherd/docs/welcome/
Firefox version: 47.0
OS: Windows 8.1

Note, this doesn't happen after updating firefox to 48.0.1


need more details for report_batch.cc:78] Mixer Report failed with: UNAVAILABLE

We see quite a lot of :
W0908 06:04:25.765] [libprotobuf ERROR external/mixerclient_git/src/report_batch.cc:78] Mixer Report failed with: UNAVAILABLE
eg https://k8s-gubernator.appspot.com/build/istio-prow/pull/istio_istio/699/e2e-suite-rbac-no_auth/360/
it would be great upon error to print which ip etc. we were connecting to and more details and granularity about the failure (did we fail to connect, did we timeout, did we not get a response, did we get a weird response, read error etc... (did we read anything, if so print that)
cc @mandarjog

This is the gRPC call status.   Report API API did not return any response.  gRPC status could mean both network error or sever error.  UNAVAILABLE most likely from mixer server.
We don't know mixer IP.  Only Envoy cluster manager know.

Thanks !
is there some verbose or debug we can turn on for grpc c++ client to get more info for failure ?
are you sure there is nothing useful/printable in the response for instance (before delete) ?
sorry for silly question but how do we connect if we don't know to what ? is there a debug level in envoy that would print more info

We are using Envoy gRPC async client code.   If debug is on, it may print more debug info.
Mixer server ip is specified in "ClusterManager" in the envoy config generated by Pilot.  Again, Envoy gRPC async client code use it.  The caller just specifies the "cluster name".

funnily we still have that issue right now

If the issue is that the Error should not be there, I think we need to check Mixer log.


Consolidate retrieval of controlled/tokens with assigned character fallbacks

loot-npc-popup seems to have unrelated changes, but they're good changes. Seems to be an attempt to fix a bug maybe.


PT-161722828 Add network_id to /status endpoint

It seems unrelated, but I restarted 2 times already?! I will have last try before I investigate this docker stuff...

...superseded by https://github.com/aeternity/epoch/pull/1891, branch name with / does not work...


how to extend webpack config?

I'm using this create-react-app to start a new react project, want to use components from material-components-web; while a first problem met is the transpiled script contains import and export which browser doesn't understand, I guess it's because the builtin webpack.config has an exclusion of all code under ^node_modules/, it transpiles only user's app code; is this true?
https://github.com/material-components/material-components-web#javascript
I tested the similar NYTimes/kyt project, that exposes a modifywebpackconfig interface for users to easily tweak webpack config somewhat; wonder if this create-react-app has some similar easy way of webpack customization? I feel like the eject to make copies of all builtin webpack config stuff is the only option here?
https://github.com/NYTimes/kyt/blob/master/docs/kytConfig.md#modifywebpackconfig
/* 272 */
/*!***************************************!*\
  !***<EMAIL_ADDRESS>***!
  \***************************************/
/***/ function(module, exports) {

	/**
	 * Copyright 2016 Google Inc. All Rights Reserved.
	 *
	 * Licensed under the Apache License, Version 2.0 (the "License");
	 * you may not use this file except in compliance with the License.
	 * You may obtain a copy of the License at
	 *
	 *      http://www.apache.org/licenses/LICENSE-2.0
	 *
	 * Unless required by applicable law or agreed to in writing, software
	 * distributed under the License is distributed on an "AS IS" BASIS,
	 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
	 * See the License for the specific language governing permissions and
	 * limitations under the License.
	 */

	import {MDCComponent} from '@material/base';
	import MDCCheckboxFoundation from './foundation';

	export {MDCCheckboxFoundation};

	export class MDCCheckbox extends MDCComponent {
	  static attachTo(root) {
	    return new MDCCheckbox(root);
	  }

    ...

Hi @c0b
You can get all the configuration to the project itself by running
npm run eject
And then you are free to edit the webpack configurations
https://github.com/facebookincubator/create-react-app#converting-to-a-custom-setup

We don't currently transpile dependencies. This is intentional because otherwise build times would be slower. It has long been a convention to precompile packages to ES5 before publishing to npm, and CRA only works well with such packages. There is a discussions about this in https://github.com/facebookincubator/create-react-app/issues/1125.
No, there are no currently no plans to make Webpack configuration configurable before ejecting. This hurts our ability to change things internally as people come to depend on specific versions of Webpack, loaders, etc. This also has been discussed before so I encourage you to use issue search before creating a new issue: #99 #145 #460 #481 #1060 #1103 #1111.
If these limitations don't work for you, indeed I recommend checking out alternatives like nwb and kyt which provide these features. I hope this helps!

thanks

@c0b import transpiled ES5 files. Instead
import { MDCSnackbar } from '@material/snackbar'

use
import { MDCSnackbar } from<EMAIL_ADDRESS>

@c0b
I just published a library called  'create-react-scripts'.
The underlying concept is same as what "react-app-rewired" did but you are able to re-publish it like your version of react-scripts which "react-app-rewired" not allow you to do so.
You are allowed to rewire the env, paths, webpack.config.dev, webpack.config.prod, webpackDevServer.config and createJestConfig yourself.
I am still a newbie on open source community, looking for feedback on this approach.
https://medium.com/@szer0601/create-react-scripts-create-your-own-react-scripts-without-fork-e73d384c7369
https://github.com/raymondsze/create-react-scripts


Change: Notification Copy

Description
This copy change got lost in the MUI rebase.
https://github.com/linode/manager/pull/5056/files
Type of Change

Non breaking change ('update', 'change')

Applicable E2E Tests
N/A

@johnwcallahan good call. addresed


Update<EMAIL_ADDRESS>

Changed "Verlauf" to "Chronik" to match the official GNOME translation

Can you add back the NBSPs or give me push access to your branch<EMAIL_ADDRESS>

Looks good, thanks!


Update for parity with analytics.js integration INT-568

The mixpanel library sends browser version as a number parsed from major.minor. In addition, it sends the OS parsed from the useragent. The server-side integration will now send the same, but the OS will be overwritten by context.os, which is sent from the mobile libraries.

@tsholmes code lgtm -- did you do any e2e testing? Want to make sure that the data shows up as expected in the end tool


[FLINK-8047] [http connector] Add HTTP Sink to allow sending output to HTTP server

What is the purpose of the change
Add HttpSink to enable sending events to an Http server
Brief change log

Added a new connector flink-http-connector
Created HttpSink that allows configuration of the HttpClient and the ResponseHandler
The sink sends each element of the stream as a separate HTTP POST request to the configured server

Verifying this change
This change added tests and can be verified as follows:

Added HttpSinkTest that starts a local cluster and an http server

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): yes
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): don't know
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? JavaDocs

Hi @genged , I will close this PR as there does not seem to have much activity.


Able to publish posts without content

If you go to 'my posts' and save a new post without any content, it publishes a blank post.
See example: http://bigboringsystem.com/post/post!1420495453-c1

Can I take this one?

Do eeeet!


Sun shadow disappears in 2D view

When in 2D mode the suns shadow disappears when translating in longitude.
If I center the display on the US, the suns shadow is visible

If I center the display on the UK, the suns shadow disappears

I would expect the suns shadow to be visible wherever the display is centered.
Kind Regards
Craig

Thanks @craigp1231, this is a known issue.  I'm going to close this as a duplicate of #4122.  I'll make a comment in #4122 so we can notify you when we have a chance to fix this


29 Restrict other authors from modifying my data

David, could you clarify what this user story does?
I'm not sure how to verify that it is working correctly. Is there a url I can go to test each possible error code template?

This user story would encompass everything to do with user data. This user story would also extend to the posts feature as well. So the code in this PR only relates to the rules for restricting other authors from modifying my profile. @ookmm posts feature will need to be updated to fully fulfill this user story.
Great question! See the the dashboard/debug/urls.py for some possible targets. For example: http://<IP_ADDRESS>:8000/debug/404

Okay, thanks for explaining :)

I know this one doesn't request my approval, but I approve!


update Get Started to use simplified example

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azurermps-5.4.0/get-started-azureps.md
:white_check_mark:Succeeded

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

@TaraMeyer Please merge

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.4.0/azureps-vm-tutorial.yml
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.4.0/get-started-azureps.md
:white_check_mark:Succeeded

docs-conceptual/azurermps-5.4.0/toc.yml
:white_check_mark:Succeeded

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


Queue accepted content

Problem
Currently, when trin handles an OFFER message, it cycles through every offered content_key and checks with storage if the key is within radius and not already present in the data store. This strategy ignores content keys that have already been "accepted" but are still being transported over utp. This can lead to a node initiating multiple utp streams with different peers for the same content key.
Given the relatively small size of our network, it seems quite reasonable that our bridge nodes will be offering the same content key (eg. latest) to the same node in a relatively short timeframe. Due to the large size of some of these content values (eg. bodies/receipts), a node might accept the same content key from each bridge node before it has a chance to complete a single utp transfer & store the content value.
Running multiple utp streams for the same content key adds bloat to the network.
Sidenote
While running trin, it's fairly common that I've seen the Accepted content outside radius or already stored log, which is a solid indicator to me that we are duplicating our accepted content keys.
Solution
For the sake of this issue, a content key is considered in "processing" from the instant it is Accepted until the moment it is saved into local storage. This covers the timeframe that it takes to transport the actual content and validate it. This can take some time depending on how large the content value is or how many network requests are needed to fetch the relevant validation data.
Introduce some kind of queue to the OverlayService that maintains a record of every content key which is currently "processing".
The order of operations for deciding whether or not to Accept an Offered content key looks something like this...

Check with storage if key is within radius & not already present in the data store (same as before).
Check with the queue if the content key is currently "processing"
A. If yes, respond to the Offer that we do not want to Accept this content key.
B. If no, save it as "processing", and respond to the Offer that we want to Accept this content key.
After successful "processing" or if any step of "processing" fails we want to be sure we clear the content key from the queue.

This is the simplest solution I can think of, and certainly not optimal. But it seems to me like starting with the simplest solution is best.
Possible Improvements

Trigger a recursive find content lookup if any step of "processing" fails.
Store the peer's enr in the queue for all content keys that go to step 2A, so that if "processing" fails we can send that peer a FindContent request, with a relatively high degree of confidence that they will have that content

Hello. I would like to take this one on


fixfix all avro,hudi and jdbc scanner class loader dependencies conflicts

Proposed changes
add preload-extensions module, put all conflict dependencies to pom.xml in preload-extensions

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

run buildall

run external

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.95 seconds
stream load tsv:          542 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          30 seconds loaded 861443392 Bytes, about 27 MB/s
insert into select:          29.1 seconds inserted 10000000 Rows, about 343K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run external

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.21 seconds
stream load tsv:          535 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.3 seconds inserted 10000000 Rows, about 341K ops/s
storage size:<PHONE_NUMBER>1 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.83 seconds
stream load tsv:          539 seconds loaded<PHONE_NUMBER>9 Bytes, about 132 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.5 seconds inserted 10000000 Rows, about 338K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run p0

what diffrents about "fe/fe-core/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java" and "fe/be-java-extensions/preload-extensions/src/main/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java",???


MXNet Error: Cannot use gluonts because MXNet does not install on Apple M1 MacOS chip

Description
I cannot use gluonts on MacOS because I cannot install MXNet. I have tried all I could from the documentation but it does not work.
@lostella @jaheba any help on this?
pip install --upgrade mxnet~=1.7
pip install gluonts

Error message or code output
ERROR: Could not find a version that satisfies the requirement mxnet~=1.7 (from versions: 1.6.0)
ERROR: No matching distribution found for mxnet~=1.7

Environment

Operating system: MacOS Monterey 12.2.1 (21D62)
Python version:3.8.9
GluonTS version:
MXNet version: 1.7

@babaniyi if you don't strictly need to use MXNet-based models, you don't need to install MXNet. For example, you can use the PyTorch version of DeepAR in case that works for you and pytorch installs on M1 (which I don't know).
Which model(s) are you interested in working with?
If you really need to use MXNet ones, the best thing to do at the moment is to open an issue with MXNet, I believe

@babaniyi you can import the estimator class with
from gluonts.torch.model.deepar import DeepAREstimator

However we don't have the PyTorch version of the zero-inflated distributions, we mostly rely on distributions directly offered by PyTorch. You can still use a NegativeBinomial with
from gluonts.torch.distributions import NegativeBinomialOutput

@babaniyi you can import the estimator class with
from gluonts.torch.model.deepar import DeepAREstimator

However we don't have the PyTorch version of the zero-inflated distributions, we mostly rely on distributions directly offered by PyTorch. You can still use a NegativeBinomial with
from gluonts.torch.distributions import NegativeBinomialOutput

Thanks for your replies.
After several hours, I think the solution is to use MacOS that are not M1 otherwise there will be lots of issues.


Provide way to show more of top of stack trace

Currently rollbar only shows up to 100 lines of the stack in the case that the payload sent to Rollbar is too large, and there are more than 100 lines. It will also show only the top 50 and bottom 50 lines of the stack trace, which makes debugging SystemStackErrors more difficult in the case that the recursion doesn't show up in the top 50 lines of the call stack.
I would like a configuration option you can set that will let you customize the portion of stack frames that show up at the top of the trace.

Another tack here may be to tweak the algorithms used to trim the payload when it's too large - i.e. maybe we can start by trying to keep more than 100 stack frames, or have options to prioritize the stack trace ahead of other parts of the payload.


Lowercase string defaults should show up as lowercase in the doc string

This is no longer an issue.


push-impersonationcontext

Please see the following error, it happens on several environments:

Hi @lelloz0
Can you check version 2.2.1 of the module, should be fixed there.
Regards,
Claudio9

Thank you Claudio, I don't get the error anymore. However, after using ### Push-ImpersonationContext and issuing a whoami I still get the username of the account used to log on the server and used for that powershell session. Is there a way to confirm that, after using that cmdlet, all the commands will use the account pushed by the impersonation? Thank you

This is how impersonation works, I guess. The process is still running as the same user account in Windows, but the interaction is performed as the impersonated used. This is not a runas function. To get the current impersonation context and the current user, use the following command:
Get-ImpersonationContext

Yeah, the problem is that any consequential cmdlet that I run after pushing the impersonation with another user, does not seem to be executed with the impersonated user. I will look into it, thank you.

Maybe, the behavior is different for command that will be executed locally or against a remote system. I use this cmdlets to execute Active Directory cmdlets with another user against the Domain Controllers.

yeah, that works but I was trying to adopt it to primarily try to access a remote server with another set of credentials in the same PS Session (this is because I'm using Jenkins). I found that New-PSSessionConfiguration works for the scope.

Locks like New-PSSessionConfiguration is not a well known cmdlet. Do you maybe mean Register-PSSessionConfiguration or New-PSSessionOption?
Would be great if you can post the few lines how you achieve you goal, so that other with the same or similar issue have a solution. Thank,

sorry: https://docs.microsoft.com/en-us/powershell/module/microsoft.powershell.core/register-pssessionconfiguration


docs: fix some typos in the docs

Description
Some typos are fixed in the documents.
PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[x] Updated relevant documentation (docs/ or spec/) and code comments

Hi @jmalicevic, I've found and fixed several typos in the documents, could you please kindly review this PR? Thanks in advance!


[Bug] Type error in series line option, the scale option in emphasis

Version
5.3.2
Link to Minimal Reproduction
No response
Steps to Reproduce
create chart with echarts.init(),and setOption to set the option,here is the part of my option,and I assign a number for scale, the typescript report an error which is type 'number' is not assignable to type 'boolean',but the doc show number has been supported since v5.3.2
const option:EChartsOption = {
    series: [
      {
        name: mainTitle,
        type: 'line',
        yAxisIndex: 0,
        data: mainData,
       emphasis:{
          scale:1.5
        },
        areaStyle: {
          color: {
            x: 0,
            y: 0,
            x2: 0,
            y2: 1,
            colorStops: [{
              offset: 0, color: 'rgba(0, 100, 255, .2)',
            }, {
              offset: 1, color: 'rgba(255, 255, 255, .2)',
            }],
          },
        },
      },
    ],
  };

Current Behavior
assign a number for scale, report an error
Expected Behavior
correct type detection
Environment
- OS:win10
- Browser: Chrome 103.0.5060.114
- Framework:React@17

Any additional comments?
No response

Similar to https://github.com/apache/echarts/issues/17072#issuecomment-1173256689. This feature was introduced in #16688 but the type definition was ignored. You are welcome to open a PR to fix it.


test error !

# Thanks for you code ！　could  you give me some advice ?
sun@sunwin:~/tf-openpose$ python3 realtime_webcam.py --camera=0 --model=mobilenet --zoom=1.0
Caused by op 'save/RestoreV2_25', defined at:
File "realtime_webcam.py", line 64, in
net, _, last_layer = get_network(args.model, input_node, sess)
File "/home/sun/tf-openpose/networks.py", line 45, in get_network
loader = tf.train.Saver()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1140, in init
self.build()
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 1172, in build
filename=self._filename)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 688, in build
restore_sequentially, reshape)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 407, in _AddRestoreOps
tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/training/saver.py", line 247, in restore_op
[spec.tensor.dtype])[0])
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/ops/gen_io_ops.py", line 640, in restore_v2
dtypes=dtypes, name=name)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/op_def_library.py", line 767, in apply_op
op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 2511, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/ops.py", line 1267, in init
self._traceback = _extract_stack()
DataLossError (see above for traceback): Unable to open table file ./models/trained/mobilenet_368x368/model-release: Failed precondition: models/trained/mobilenet_368x368/model-release: perhaps your file is in a different file format and you need to use a different restore operator?
[[Node: save/RestoreV2_25 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_arg_save/Const_0_0, save/RestoreV2_25/tensor_names, save/RestoreV2_25/shape_and_slices)]]
[[Node: save/RestoreV2_45/_793 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/gpu:0", send_device="/job:localhost/replica:0/task:0/cpu:0", send_device_incarnation=1, tensor_name="edge_1658_save/RestoreV2_45", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/gpu:0"]]

@zilesazoyi2  I have same error,
did you find a solution?

Your solution is posted in #23