text:
Not detecting numerous Plugins updates available

I love this plugin, but recently noticed it is missing numerous updates that are available to plugins I'm using. If you use this pom.xml: https://github.com/nahsra/antisamy/blob/d0b3fcc99637eed5decd4a3bdc1e66adb9941fa6/pom.xml (and upgrading to versions-maven-plugin v2.7 didn't change anything), I noticed that these plugin updates that are available are not being identified:
org.apache.maven.plugins:maven-dependency-plugin:3.1.1 has a 3.1.2 available
org.apache.maven.plugins:maven-enforcer-plugin:3.0.0.M2 has a 3..0.0.M3 available
com.github.spotbugs:spotbugs-maven-plugin:3.1.12.2 has a 4.0.4 available
and I suspect there are more.
I'm generating the plugin-updates-report by simply using: mvn site with this pom.xml.
Can you confirm this is a bug? Or am I simply using the plugin wrong? I have no idea if this problem is still present in the master branch. I see there hasn't been a new release in almost 2 years, but there is lots of activity. When do you expect a new release to be released (i.e., if this issue was fixed)?
I'm about to update this pom manually to fix these issues, so if you want to test, make sure you use the version I reference above.

Did you execute Maven with -U? I've occasionally seen this plugin miss updates if I don't use that flag to force checking for updates on remote repositories.

Thanks Mark!  That definitely worked and fixed the issue. My question now is that if you are running this plugin, seems like you'd want that flag on (by default) so you don't miss updates that are available.  Does it make sense that this plugin should force that on automatically by default? If so is there a way to do that? And if it can, then if there are projects that don't want that on by default they could turn it back off with a parameter setting for that plugin.

I think it does make sense that you'd always want to use that flag. Unfortunately, I don't personally know if there's an API in Maven for forcing remote repository checks... Anyone else know?

I was thinking that if you could pass -U into the mvn command, there would a way for a plugin to do the same thing. Or if the plugin can't do that directly, is there a way to specify the -U command in the configuration of the versions-maven-plugin in the  block of my pom, so I manually force that flag on whenever the versions-maven-plugin is invoked when I run: mvn site?

@io7m - Really? - this got marked 'stale' and then auto closed?  Can this be reopened and someone figure out if there was an easy way, or not, to get the plugin to automatically invoke -U or its equivalent?

@io7m - Really? - this got marked 'stale' and then auto closed?

Nothing to do with me, to be clear. I'm not even a committer. :slightly_smiling_face:

@io7m - Thanks Mark - But it did have the desired affect. @slachiewicz - reopened the ticket :-). @slachiewicz - can someone on the plugin team research this issue to see how hard it would be to make this improvement?

repo: mojohaus/versions-maven-plugin


text:
Cannot invoke "Object.toString()" because "artifactVersion" is null

Using version-maven-plugin 2.11.0.
Trying the following invocation:
mvn -U -N '-DparentVersion=[8.0.0-20487-5b5e876085]' -DforceUpdate versions:update-parent -e

yields the following result:
[ERROR] Failed to execute goal org.codehaus.mojo:versions-maven-plugin:2.11.0:update-parent (default-cli) on project skeleton-parent: Execution default-cli of goal org.codehaus.mojo:versions-maven-plugin:2.11.0:update-parent failed: Cannot invoke "Object.toString()" because "artifactVersion" is null -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.codehaus.mojo:versions-maven-plugin:2.11.0:update-parent (default-cli) on project skeleton-parent: Execution default-cli of goal org.codehaus.mojo:versions-maven-plugin:2.11.0:update-parent failed: Cannot invoke "Object.toString()" because "artifactVersion" is null
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:215)
...
Caused by: org.apache.maven.plugin.PluginExecutionException: Execution default-cli of goal org.codehaus.mojo:versions-maven-plugin:2.11.0:update-parent failed: Cannot invoke "Object.toString()" because "artifactVersion" is null
at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:148)
...
Caused by: java.lang.NullPointerException: Cannot invoke "Object.toString()" because "artifactVersion" is null
at org.codehaus.mojo.versions.UpdateParentMojo.update (UpdateParentMojo.java:128)
at org.codehaus.mojo.versions.AbstractVersionsUpdaterMojo.process (AbstractVersionsUpdaterMojo.java:339)
at org.codehaus.mojo.versions.AbstractVersionsUpdaterMojo.execute (AbstractVersionsUpdaterMojo.java:274)
at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:210)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:156)
at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:148)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
at org.apache.maven.cli.MavenCli.execute (MavenCli.java:972)
at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:293)
at org.apache.maven.cli.MavenCli.main (MavenCli.java:196)
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)

Using version 2.5 might shed some light on the problem:
[WARNING] Not updating version: could not resolve any versions

The problem is caused by the wrong order of checks in AbstractVersionsUpdaterMojo
    protected boolean shouldApplyUpdate( Artifact artifact, String currentVersion, ArtifactVersion updateVersion, boolean forceUpdate )
    {
        getLog().debug( "Proposal is to update from " + currentVersion + " to " + updateVersion );

        if ( forceUpdate )
        {
            getLog().info( "Force update enabled. LATEST or RELEASE versions will be overwritten with real version" );
            return true;
        }

        if ( updateVersion == null )
        {
            getLog().warn( "Not updating version: could not resolve any versions" );
            return false;
        }

I think checking updateVersion against null should happen before the forceUpdate check.

PR are welcome

give me a sec, I'm about to make one

source: gharchive/issue

authors: ajarmoniuk slawekjaranowski

license_type: permissive

license_source: bigquery


text:
F-Droid

Hi,
Since this app is Apache v2 licensed, are you considering the fact to add it to F-Droid? There is a strong community behind.
Your app has already been requested here.
You can find some documentation here.
Otherwise, if you don't want to work on the inclusion of this app by yourself, would it be ok for you if someone else do it for you?

Thank you very much!
Tomorrow, I am going to do these.

Replace prebuilt jar with source code
Add Twitter API key not being public
Add tags.

I'm sorry. I am not able to do it now.
Because it's still night here and I have to sleep.
Please wait until tomorrow.

Take your time, @moko256 – nobody expects you to "jump on command". We're glad you're supporting us, and we definitely can wait another day (or a few, if needed) :wink:

In a few days, I am going to upload Twitter API key encoded base64 and decoding code in this repository. So it works without secrets.

You are aware that base64 encoding is no encryption, and thus that's the same like uploading the API key "plain text"? I'm not a dev, but this feels strange to me. I could be wrong of course…

It's same.
But that will make it a little difficult for simple BOT to steal API key.
But some people may steal it...

@moko256 I've asked one of the F-Droid maintainers to come over here so you can discuss directly what steps are required or how to approach it. I simply lack the development background. I hope he checks with you soon.

I fixed known problem.

Thanks, @moko256 – I've asked our maintainers to "try a build".

There still seem to be some issues. Quoting a maintainer:

gradle assembleRelease failed for me with

Could not determine the dependencies of task ':app:lintVitalRelease'.
> Could not resolve all task dependencies for configuration ':app:releaseRuntimeClasspath'.
   > Could not resolve project :japng-android:android_api.
     Required by:
         project :app
      > Unable to find a matching configuration of project :japng-android:android_api: None of the consumable configurations have attributes.
   > Could not resolve project :mastodon4j.
     Required by:
         project :app > project :mastodon4j_to_twitter4j
      > Unable to find a matching configuration of project :mastodon4j: None of the consumable configurations have attributes.
   > Could not resolve project :mastodon4j:mastodon4j.
     Required by:
         project :app > project :mastodon4j_to_twitter4j
      > Unable to find a matching configuration of project :mastodon4j:mastodon4j: None of the consumable configurations have attributes.

Also there is some junk in the repo that I had to remove manually (beause binaries):

twitter4j/..git
twitter4j/twitter4j-core/build
twitter4j/twitter4j-http2-support/build

Can you take a look and (hopefully) fix this?

I understood it.

Thanks for saving me another crosspost :rofl: Direct conversation is better. Hope we get the bird flying soon :wink:

The maintainer said it will be available on F-Droid in a few days in this .
Thank you!

The maintainer said it will be available on F-Droid in a few days in this .
Thank you!

Why, thank YOU for supporting this :+1: :tada:
Can't wait to give Twilatte a try :innocent:

authors: IzzySoft Poussinou moko256

license_source: github-api


text:
[TS] Add types for "req" and "res"

This PR adds types for req and req objects.
Usage:
const ApiGateway = require("moleculer-web");
const { IncomingRequest, GatewayResponse } = require("moleculer-web");

Related issue: moleculerjs/moleculer-template-project-typescript#20

Pull Request Test Coverage Report for Build 482

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

Totals

Change from base Build 471:
0.0%

Covered Lines:
665

Relevant Lines:
687

💛  - Coveralls

Pull Request Test Coverage Report for Build 482

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

Totals

Change from base Build 471:
0.0%

Covered Lines:
665

Relevant Lines:
687

💛  - Coveralls

source: gharchive/pull-request

authors: AndreMaz coveralls

repo: moleculerjs/moleculer-web

license: MIT


text:
Route nesting is not working as per docs

I'm following the docs on Route nesting and having difficulties on understanding how it really works
https://github.com/molefrog/wouter?tab=readme-ov-file#route-nesting
Steps to Reproduce:

Create a new React application (using pnpm create vite).
Install Wouter (v3.2.1)
Implement the following routing configuration

import { Route, Router } from "wouter";
import Orders from "./pages/orders";
import SingleUser from "./pages/singleUser";

export default function App() {
  return (
    <Router>
      <Route path="/app" nest>
        <Route path="users/:id" component={SingleUser} nest>
          <Route path="/orders" component={Orders} />
        </Route>
      </Route>
    </Router>
  );
}

Navigate to /app/users/:id/orders in your application.

Expected Behavior
The Orders component should render on the path  /app/users/1/orders
Actual Behavior
The SingleUser component renders instead of the Orders component. even though the docs mention following

Finally, the inner-most route will only work for paths that look like /app/users/1/orders. The match is strict, since that route does not have a nest prop and it works as usual.

Hi @nivethan-me, Route component can't use children when the component prop is given, they are mutually exclusive. What might solve your problem is this:
  return (
    <Router>
      <Route path="/app" nest>
        <Route path="users/:id" nest>
          <Route path="/" component={SingleUser} />
          <Route path="/orders" component={Orders} />
        </Route>
      </Route>
    </Router>
  );

authors: molefrog nivethan-me

repo: molefrog/wouter

license: Unlicense


text:
Possible performance degradation

Quoting @patook

I have done some testing over the past couple of hours. The XF2.2 version compiles/runs for me.
However, I am seeing some perf degradation in the alpha nugets compared to the original nugets. My observations:
1- Loading the images on start-up (from DiskCache I'm guessing) seems particularly slow as well. If I turn the app off/on there is always a long lag at the beginning until the images show up. This lag feels slower than what I had in the previous FF nugets.
2- Are the images loaded in a different order than before? Looks like some of the last ones now show up first. This could be what's causing the slowness perception.
Is this expected? Is it possible to revert to the "old" method of caching within this nuget?

@patook yes the scheduler changed:

it supports priorization
it loads latest tasks first

I expected that loading latest tasks first would increase speed perception therefore you say the opposite.
You can see the change here: https://github.com/molinch/FFImageLoading/blob/master/source/FFImageLoading.Common/Work/WorkScheduler.cs#L344
ThenByDescending(t => t.Position)
Could you test using git sources & change the following line so it becomes ascending?

OK that makes sense. In my case (which is a common scenario I think), I have a list of 16 images top-to-down and they get loaded in reverse order so the top images stay white for the longest time.
Yes, I can get the source code from git but it will be on Monday, is that OK? Might not be a bad idea to make this configurable regardless (i.e. let the user pick between descending/ascending based on scenario).
Also, you mentioned we can now prioritize images, quickly looking at your code it looks like this is the parameter: public Work.LoadingPriority LoadingPriority? I will play around with that as well

Sure Monday is totally fine, many thanks for your help!
And don't worry: if the sorting change makes no sense we will revert it.
Yes you can now add a priority: lowest, low, high, highest or even a custom value since the enum maps to an int.
Please note that DiskCache implementation changed. So cached images will have to be redownloaded.
Basically we no longer rely on a journal file. Instead the implementation is much much simpler, hence easier to maintain/debug.

@patook have you managed to play with it?

Hi @molinch
Yes - I have just finished testing after replacing ThenByDescending with ThenBy.
I believe the performance appears as good as the original nugets now so I am happy.
Personally I can work around this by customizing the priorities but I think ThenBy might be optimal for a larger number of common scenarios than ThenByDescending.
Thank you

Actually please wait a couple more hours as I am still seeing some low perf behavior after more testing.

Out of curiosity was there any change to the way local images (i.e. those under Resources) are loaded? I am perceiving a small difference in perf for them but I've tested it so many times now that I'm not 100% sure anymore.

Many thanks for the feedback !
There are a couple of changes but I don't see a particular one that would harm loading local resources.
If you confirm the performance degradation I will make deeper tests.
Biggest changes are:

journal based disk cache to file system only
the lib uses targets rather than specific views, a target receives the bitmap data and uses it, this allows more fine control & specific cases (like Preload)
a few scheduler minor changes
a couple of extra checks (null, target valid, ...)

OK thank you @molinch . I did some more testing and I cannot conclusively say anything about local resources.
For the moment I just moved the local resources (for example the icons) to LoadingPriority.Highest and that looks good. Before that, sometimes they would be the last to load which maybe contributed to the "slowness" perception.
Since you've replaced with "ThenBy", I am fine with closing this bug if you'd like to. If I see something bad in the next week I will re-open it (or if our users will yell at us :p).

Ok let's do this way :)

authors: molinch patook

repo: molinch/FFImageLoading

license: mit


text:
🛑 fresh is down

In f5c095e, fresh (https://fresh.haccp.or.kr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: fresh is back up in 25b4c1b.


text:
add back missing terms

these terms were accidentally deleted:
MONDO:0005713 and MONDO:0024812
in these commits:
https://github.com/monarch-initiative/mondo/commit/024a35cc932965ea0b6e3f0f2d4002fd96e267d3#diff-889e0a9ba62d62cc6ee7c[…]90ef0dcf4786904ac6768L37362
https://github.com/monarch-initiative/mondo/commit/be7d3053e4f4b8aa00b43260458e10c7a5ebf702

ROBOT template

repo: monarch-initiative/mondo


text:
Idea: add colors to files in the sidebar

I imagine that with a true/false option
for example, instead of having this kind of files (see image), you would have each file with the color you've defined in settings.json

Thanks for the suggestion, few other people have asked for this feature too, I'll definitely put it on the todo list

authors: mondersky sqmasep

repo: mondersky/tabscolor-vscode


text:
Dockerize

Adds dist script to compile for multiple architectures & pushes into docker registry

¯_(ツ)_/¯

repo: mondough/phosphor


text:
reduce CLS when lazy-loading images

Requires #1508
Adds width and height attributes to lazy-loaded images. Allows browsers to calculate the aspect ratio and allocate adequate space to avoid layout shifting.
https://www.youtube.com/watch?v=4-d_SoCHeWE

Please rebase on #1508 (had a small conflict after the recent merges)

authors: erciccione wirrickelliot

repo: monero-project/monero-site


text:
L2 constructs for Cluster, Project, ServerlessInstance, NetworkPeering and PrivateEndpoint etc.

Is your feature request related to a problem? Please describe.
At this moment, we are still missing a lot of frequent L2 constructs including Cluster, ServerlessInstance and Project.
The general idea of the L2 construct is:

It allows developer to create MongoDB Atlas resources with sensible defaults with very minimal required properties only for a single CfnResource.
It allows to create a "reference" object from existing resource with  the from methods such as Xxx.fromAttributes() or Xxx.fromXxxId().
It offers opinionated construct methods such as Xxx.addCluster() or Xxx.addComponent().
As all L2s are extended from CfnResource, they inherit some build-in capabilities as static methods defined in the CfnResource class.

Let's take CfnProject for example, the L2 Project should look like this:
import { Resource, ResourceProps } from 'aws-cdk-lib';
import { CfnProject } from 'awscdk-resources-mongodbatlas';
import { Construct } from 'constructs';
import { ProjectOptions } from './types';

export interface IProject {
  readonly clusterCount?: number;
  readonly projectId: string;
  readonly projectName: string;
  readonly created?: string;
  readonly projectOwnerId: string;
}

export interface ProjectAttributes {
  readonly projectId: string;
  readonly projectName: string;
  readonly projectOwnerId: string;
}

export interface ProjectProps extends ResourceProps, ProjectOptions {
  readonly profile: string;
}

export class Project extends Resource implements IProject {
  public static fromProjectAttributes(scope: Construct, id: string, attrs: ProjectAttributes): IProject {
    class Import extends Resource {
      public projectId = attrs.projectId;
      public projectName = attrs.projectName;
      public projectOwnerId = attrs.projectOwnerId;
    };
    return new Import(scope, id);
  }
  readonly clusterCount?: number;
  readonly projectId: string;
  readonly created?: string;
  readonly projectOwnerId: string;
  readonly projectName: string;

  constructor(scope: Construct, id: string, props: ProjectProps) {
    super(scope, id);

    this.projectName = props.name ?? `project-${id}`;
    const resource = new CfnProject(this, 'Resource', {
      ...props,
      profile: props.profile,
      orgId: props.orgId,
      name: this.projectName,
    });
    this.clusterCount = resource.attrClusterCount;
    this.projectId = resource.attrId;
    this.created = resource.attrCreated;
    this.projectOwnerId = resource.attrProjectOwnerId;
  }
}

And customer can create a single Project by
const newProject = new Project(scope, id, {
  profile: 'default',
});

Or simply import an existing one as a reference object like
const existingProject = Project.fromProjectAttributes(scope, id, {
   projectId,
   projectName: 'my-manual-created-project',
   projectOwnerId: 'the-owner-id',
});

And when we create a single cluster in an existing project we should be allowed to
new Cluster(scope, id, {
    project: existingProject,
    ...

Similarly, if we need to build a VPC Peering with L2 construct, the DX should be like
// define my existing VPC
const vpc = ec2.Vpc.fromLookup(scope, 'Vpc', { isDefault: true});

// create a new VPC Peering for an existing project with an existing VPC from AWS.
new NetworkPeering(scope, id, {
    project: existingProject,
    vpc: existingVpc,
};

which implicitly assume routeTableCidrBlock as props.vpc.vpcCidrBlock and awsAccountId the deploying AWS account ID by default unless explicitly defined.
This makes the L2 experience consistent with AWS CDK if customers are running both AWS services such as Amazon API Gateway, AWS Lambda, Amazon ECS, Amazon EKS with MongoDB Atlas resources altogether in a single CDK application.
Describe the solution you'd like
As mentioned above, we should provide some most frequently used L2 constructs such as Cluster, Project, ServerlessInstance, etc.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
I will start working on PRs for some L2s mentioned above. The goal is to build seamless and consistent developer experience to create MongoDB Atlas resources in AWS.

Hello @pahud this is a grate suggestion, we added this to the backlog hoping to target them soon!

Hi @Adelmar92
I will create some samples in my repo this week and share with you.

repo: mongodb/awscdk-resources-mongodbatlas


text:
(EAI-346): Radiant integration

Jira: https://jira.mongodb.org/browse/EAI-346
Changes

Clients to connect to embedding and LLM models hosted on radiant

Notes

Support MongoDB corp security cookie if necessary
Before merge, wait for https://jira.mongodb.org/plugins/servlet/desk/portal/48/KANOPY-708

closing this PR b/c we've decided not to proceed with the radiant integration for the time being.
the code in this PR actually does work in kanopy when you update the:

OPENAI_API_KEY and OPENAI_ENDPOINT in kanopy
set up a sidecar service to work with istio
also not in the PR, but you'd need to set up the vanity domain https://knowledge.staging.corp.mongodb.com/ to work with the new istio-backed app. this'd be a kanopy ticket

for more information on setting up radiant across namespaces, refer to https://docs.google.com/document/d/1U_t1D4999b59FsUmhYMJyahCVttx1D_bs-v6jlYQqsw/edit#heading=h.trrflxo7p9nk (the content of which might move to the wiki at some point)

closing this PR b/c we've decided not to proceed with the radiant integration for the time being.
the code in this PR actually does work in kanopy when you update the:

OPENAI_API_KEY and OPENAI_ENDPOINT in kanopy
set up a sidecar service to work with istio
also not in the PR, but you'd need to set up the vanity domain https://knowledge.staging.corp.mongodb.com/ to work with the new istio-backed app. this'd be a kanopy ticket

for more information on setting up radiant across namespaces, refer to https://docs.google.com/document/d/1U_t1D4999b59FsUmhYMJyahCVttx1D_bs-v6jlYQqsw/edit#heading=h.trrflxo7p9nk (the content of which might move to the wiki at some point)

repo: mongodb/chatbot


text:
Added ulimit with command to start mongod when building; removed lega…

When first checking out the project + running the commands to build and test the driver several tests fail with "Too many open files" errors causing the suite to hang and server to crash.
This pull request helps new users get up and running quickly by stating a ulimit command that can be used. I also provided the subsequent command to start a local mongod using this limit.
There was a broken build status image/link pointing to 10gen which looks old/legacy so I removed it.
Also fixed intellij image showing extra braces due to an extra line break

The travis CI build seems to be stuck 'in progress', so I tried closing and re-opening the pull request to re-trigger it. It doesn't seem to have helped :(

Hi @jsonking thanks for the PR.
I don't think we should blanket advise to set ulimits to 1024, however, I like the idea of pointing out that they may cause the test suite to fail.
So instead could you update to add a paragraph describing the error and how to fix? It would be good if it linked to the manual: https://docs.mongodb.com/manual/reference/ulimit/
Thanks again,
Ross

Hi @rozza I have pushed a fresh commit with the change suggested.
The reason I thought it would be ok to set the ulimit explicitly is that when running the test suite needs it to be above the default of 256 (on macOS) and when logging in via the shell I get the following: "WARNING: soft rlimits too low. Number of files is 256, should be at least 1000". Agree the single command is cleaner and should work for most people.
Thanks
Jason

Hello,
Is this pull request ok to be merged or would you like further changes?
Thanks
Jason

All good - thanks @jsonking 👍

authors: jsonking rozza

repo: mongodb/mongo-java-driver


text:
PHPLIB-300 Test suite should not query featureCompatibilityVersion for server versions before 3.4

https://jira.mongodb.org/browse/PHPLIB-300

Travis is reporting one more error:
There was 1 error:
1) MongoDB\Tests\DocumentationExamplesTest::testChangeStreamExample_1_3
MongoDB\Driver\Exception\RuntimeException: Unrecognized pipeline stage name: '$changeStream'
/home/travis/build/mongodb/mongo-php-library/src/Operation/Aggregate.php:238
/home/travis/build/mongodb/mongo-php-library/src/Operation/ChangeStream.php:142
/home/travis/build/mongodb/mongo-php-library/src/Collection.php:964
/home/travis/build/mongodb/mongo-php-library/tests/DocumentationExamplesTest.php:933

I think that just needs a skip like the other change stream functional tests, and then we're good to go here.

authors: jmikola kvwalker

repo: mongodb/mongo-php-library


text:
Support for Hashed Indexes

There is no support in morphia annotations for Hashed Indexes. Anybody has any idea when it will going to be added (if it is at all in the roadmap)?

+1

Looks like it might be as simple as a new enum value.  I'll play with it.

authors: evanchooly fedjo hrasadi

repo: mongodb/morphia


text:
Support aggregation pipelines in updates

Hi there!
This PR adds support for usage of aggregation pipelines in updates.
Fixes #733 .
Best regards,
Gustavo Sousa

Support for aggregation pipelines would be really useful!

Any news on this one?

authors: guludo jobrachem michele-amati

repo: mongomock/mongomock

license: ISC


text:
Allow custom ClientClass argument on patch

Hi, this came up as a solution to deal with limitations with the mongomock api - namely not supporting transactions and sessions. I understand the decision to raise an error from the lib side, but I still think it should be possible to use the other features if the user chooses to. In my case I created a class that "pretends" to initiate a transaction for testing:
class MongoMockClientWithSession(mongomock.MongoClient):
    """Mongomock does not implement sessions, so we mock it here. This means that
    the atomicity is not actually tested.
    """

    @contextmanager
    def start_session(self, *args, **kwargs):
        yield self

    @contextmanager
    def start_transaction(self, *args, **kwargs):
        yield

And used it in my tests with mongomock.patch:
@pytest.fixture(autouse=True)
def mock_mongo():
    with patch_mongoclient("testhost", "create", ClientClass=MongoMockClientWithSession):
            yield

So the transactions are not actually tested but can still perform the tests on the other code functionality.
If this is not wanted no problem, just thought it could be a nice feature.

The code was not right. Sorry for the spam.


text:
Update sbt to 1.9.3

About this PR
📦 Updates org.scala-sbt:sbt from 1.5.8 to 1.9.3
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Superseded by #424.

repo: monix/monix-kafka


text:
Introduce the FoldableT type-class

Similar in spirit with FoldableF, the FoldableT type-class will be for left folding data-structures into an asynchronous summary value.

No, not yet. Will have to think this through.

repo: monixio/monix


text:
Github workflow to build and push Docker images on tag

This PR adds a GitHub workflow to automatically build amd64 and arm64 Docker images and push them to the Docker registry (with a tag, and to also push/update the latest image tag).
This PR is largely based on this: https://github.com/getumbrel/umbrel-manager/blob/fc823490591ea55e26734d144a0527fe9618166d/.github/workflows/on-tag.yml
You need to create the necessary Github secrets (for the Docker registry credentials): https://docs.github.com/en/actions/security-guides/encrypted-secrets#creating-encrypted-secrets-for-a-repository
These need to be created:

DOCKER_USERNAME
DOCKER_PASSWORD
DOCKER_HUB_USER

You can create a tag easily by following these instructions: https://stackoverflow.com/a/18223354
The workflow is setup to look for tags using semver. versioning with v as a prefix. i.e. v1.0.0 works, 1.0.0 doesn't.

its working! thank you so much for creating this pr, would have taken me ages to figure out.

this is going to make my deployments much easier, can't thank you enough

repo: monlovesmango/astral


text:
[objc] Add support for compiling to frameworks.

Add support for compiling to frameworks.
I've only tested iOS frameworks, tvOS and watchOS may or may not work.
macOS is not supported (and will show errors).

Build and tests passed.

@rolfbjarne I'm fine with disabling the build/test of the C generator (until fixed).

Build and tests passed.

authors: monojenkins rolfbjarne spouliot


text:
[FEATURE] Expose existing RTree optimization option on SkPictureRecorder.BeginRecording()

We're looking to pan and zoom some SkiaSharp's SkPictures that consist of a reasonably large number of simple operations (~25K, text, lines, rect fills). We're finding a zoomed and translated DrawPicture is rendering on the order of 5 fps on recent model iOS devices, which is clearly unacceptable.
But yay! It turns out Skia has had built in support for RTree-based operations culling for a long time, and it looks very promising:
https://groups.google.com/g/skia-discuss/c/EPOuhhloIF4/m/rZyakh1fBwAJ
https://bugs.chromium.org/p/chromium/issues/detail?id=485597#c22
But oh no! SkiaSharp doesn't expose this functionality, either in C# or the C wrappers in the Mono fork of Skia.
But yay! The fix looks very straightforward - just exposing the constructor of an RTreeFactory instance, and passing it as an extra parameter to BeginRecording().
I've created forks of both SkiaSharp and mono/skia, and implemented a swag at a simple C# api, C wrappers, and some smoke tests. You can find the (fairly small, simple, but unvetted) changes here:
https://github.com/mono/SkiaSharp/compare/main...kcoop:SkiaSharp:kcoop-rtree
https://github.com/mono/skia/compare/xamarin-mobile-bindings...kcoop:mono-skia:kcoop-rtree
This seems like a valuable, big bang for the buck PR. Very small change, and who wouldn't want significant rendering improvements for large-ish documents by simply adding a bool? I'm happy to do what's needed to make this happen.
[Edited to remove noise - I've now discovered this repo uses submodules to build mono/skia]

I would be interested in that feature as well. I want to record draw calls and then replay them with different zoom and translation. Right now this is very slow when only rendering a tiny part of the actual recorded calls (i.e. when zooming in). Any chance to get this feature merged?

I have added this to the list of things for v3. I'll have a look and merge some things.
Thanks!

When do you plan 3.0? Any chance to get this merged earlier? Can we do something to help?

When do you plan 3.0? Any chance to get this merged earlier? Can we do something to help?

If there is a PR, we can easily backport a smaller change like this. SkiaSharp is less scary than all these repos and files.

authors: TwinkyDaniel kcoop mattleibow

repo: mono/SkiaSharp


text:
Bind Skottie's Animation Builder

Description of Change
This PR adds a few of the essential APIs to enable loading embedded files in the Lottie animation. Currently, there is just the support for file-based and data URI-based assets.
Bugs Fixed

Fixes #2173

API Changes
class Animation {
  public static AnimationBuilder CreateBuilder (AnimationBuilderFlags flags = AnimationBuilderFlags.None);
}

public sealed unsafe class AnimationBuilder : SKObject, ISKSkipObjectRegistration {
  public AnimationBuilder SetFontManager (SKFontManager fontManager);
  public AnimationBuilder SetResourceProvider (ResourceProvider resourceProvider);
  public AnimationBuilderStats Stats { get; }
  public Animation? Build (Stream stream);
  public Animation? Build (SKStream stream);
  public Animation? Build (SKData data);
  public Animation? Build (string path);
}

public struct AnimationBuilderStats {
  public readonly TimeSpan TotalLoadTime { get; }
  public readonly TimeSpan JsonParseTime { get; }
  public readonly TimeSpan SceneParseTime { get; }
  public readonly int JsonSize  { get; }
  public readonly int AnimatorCount  { get; }
}

public abstract unsafe class ResourceProvider : SKObject, ISKSkipObjectRegistration {
  public SKData? Load (string resourceName);
  public SKData? Load (string resourcePath, string resourceName);
}

public class CachingResourceProvider : ResourceProvider {
  public CachingResourceProvider (ResourceProvider resourceProvider);
}

public class DataUriResourceProvider : ResourceProvider {
  public DataUriResourceProvider (bool preDecode = false);
  public DataUriResourceProvider (ResourceProvider? fallbackProvider, bool preDecode = false);
}

public class FileResourceProvider : ResourceProvider {
  public FileResourceProvider (string baseDirectory, bool preDecode = false);
}

Behavioral Changes
None.

Required skia PR
https://github.com/mono/skia/pull/114
PR Checklist

[x] Has tests (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[ ] Merged related skia PRs
[x] Changes adhere to coding standard
[ ] Updated documentation

@mattleibow When is this going to released?

Hi @mattleibow any idea when will it be released? i have done migration of an app and i am only blocked by lottie animations which uses files.  if there is not plan for release, then will it be fine if i just checkout this branch and create build and use the dlls?

authors: imms20 mattleibow thisisthekap


text:
Wait for async validations to finish

Is there a straight-forward way to wait for async validations to finish before continuing in a function? For example, if I hit a submit button in a form I want to first validate and submit only if valid. But $touch() doesn't return a promise so I can't wait for the async validations to finish.
An ugly workaround I wrote polls $pending on a fixed interval and returns a resolved Promise once $pending is false.

As the validation is simply a computed value, you can use watches to do this.
methods: {
  validPromise () {
    return new Promise((resolve) => {
      const unwatch = this.$watch(() => this.$v.$valid, (isValid) => {
        if (isValid) {
          unwatch()
          resolve()
        }
      }
    })
  },

  async myButtonClicked () {
    this.$v.$touch()
    await this.validPromise()
    doSomethingWhenValidationPasses()
  }
}

You can possibly cache that promise too, which would create just one watcher when function is called multiple times.

Genius! Thanks!

Actually I can't seem to get that to work either.
I'd love to be able to do something like this:
`
async submit () {
const isValid = await this.validate()
if (!isValid) {
return console.log('validation failed')
}
// else keep going
`

Just change your watch to !this.$v.$progress

For the record - unwatch is undefined when using {immediate: true} since the callback is called before the unwatchFn is returned - see $watch

The issue with that solution is that myButtonClicked() will continue immediately after the user fixes the validation issues which I think results in a really strange user experience. I simply want this flow to be possible:

Fill out form
Click Submit button
Disable button
Wait for async validations to complete
Submit form-data to API
Wait for API to respond
Enable button

I totally agree with @d-levin. It'd be ideal for touch() to return a promise.

Agreed with the other commenters, please consider reopening this issue.

Agreed this seems like a really normal workflow.
Fill out form,
submit,
validate,
send to server
touch() returning a promise would be very useful.

Is there any updates on this? I am struggling to figure out how to allow my form to be submitted with async validations. huge +1 for touch() returning a promise here, or adding another method that does return a promise.
What's the cleanest way to solve this problem now as it stands?

authors: Frizi archSeer d-levin flybayer hooblei jnarowski pkraus109

repo: monterail/vuelidate


text:
Adibags compatibility with Auto Junk Frame option

Hi
I see my last issue (#144 ) was closed and I've just updated to 1.4.0 but I'm having an issue with the 1st function that was requested, (auto open the junk frame when the bags open).
It seems my bag addon, Adibags is interfering with this, and the junk frame wont open when my bags open.
Any chance you could test this functionality and get the addons to work together if possible?

Oof 😞 . Yeah, this is a little bit tricky, but I'm working on it. Whatever solution I come up with might be a bit brittle, but we'll see.

This should be fixed in #160 (Dejunk version 1.4.1), but the code is subject to breaking if AdiBags, ArkInventory, Bagnon, or ElvUI change the name of their bag frames.

authors: Vettic moody

repo: moody/Dejunk


text:
Review the Jupyter and Python lecture notes page

I have a first draft of:
https://moorepants.github.io/learn-multibody-dynamics/jupyter-python.html
It needs a look over for details. Feel free to open a pull request to improve this file:
https://github.com/moorepants/learn-multibody-dynamics/blob/master/jupyter-python.rst

Final draft is live.

Thanks. I'll change the %paste example to something else.

The examples in this page with hyperlinks for every concept for example lists, tuples and dictionaries are useful and clear. In the introduction of Jupyter notebook should we specifically mention to install anaconda for people working in windows?

I think I need an install page added to the notes. Just haven't gotten around to it. The course website has some install instructions.
I think that I want to suggest installing miniconda instead of anaconda and then to configure to install from conda forge. This mimics what I personally do.

@akshath03 Can you try out these instructions and tell me if it works for you: https://moorepants.github.io/learn-multibody-dynamics/install.html
It would be nice to try on WIndows and Mac by you all, because I am on Linux and can't test the other OS's.

Yes, I already tried running jupyter notebook from this link. I think both are similar and works fine.
https://moorepants.github.io/me41055/software.html

One thing I was worried about is whether the conda update --all is extremely slow in Anaconda vs Miniconda. If you add the conda forge channel to an anaconda install and then run update all, it basically has to uninstall all the anaconda packages and download new versions from conda forge. Anyone experience that?

Calling this one done!


text:
undefined: parser.ReadFrom

Can't seem to install mdfmt. Receiving error:
go/src/github.com/moorereason/mdfmt/main.go:64:15: undefined: parser.ReadFrom

Seems that hugo's parser API has changed and parser.ReadFrom no longer exists. Unfortunately, I don't have the bandwidth to dig into this further or contribute a fix at the moment.

Thanks for the report.  You should be able to build mdfmt now since I've added go modules (thanks for that report, too).  I've pinned the hugo module to a version prior to the API change.  I'll leave this issue open as a reminder to update to the latest hugo API.

still getting this error. i used to have a workaround recipe but seem to have misplaced it
go get -u github.com/moorereason/mdfmt

# github.com/moorereason/mdfmt
go/src/github.com/moorereason/mdfmt/main.go:64:15: undefined: parser.ReadFrom

something like this seems to have worked
GO111MODULE=on go get -v github.com/moorereason/mdfmt

go version go1.12.10 linux/amd64

authors: idcrook moorereason pbnj petermbenjamin

repo: moorereason/mdfmt


text:
hosted ACH solution for api.moov.io and sandbox.moov.io

Extending from https://github.com/moov-io/ach/issues/254 we'd like to serve the ACH product from api.moov.io.
There's no infra setup for this currently, but I'm familiar and can do it quick.

sandbox.moov.io (sbx.moov.io?) should be deployed to first and a quick set of sanity specs ran against.

repo: moov-io/ach


text:
0.7 and 60.3

gvm install 60.3 fails!
Seems its not actually downloading to where its looking for it during compile...

My apologies 0.0.7 required a few post "stable" fixes. This was a major rewrite that switched from symlinks to environment files ($PATH). This should be working now and 0.0.8 is going to sport some major package handling updates.

authors: hcatlin jbussdieker

repo: moovweb/gvm


text:
Custom Websites Don't Launch

When I add a custom website it doesn't launch.  I click "Play" and nothing happens.  This happens in both formats I can use for the site in question "www.github.com" and "https://www.github.com" (the sites I'm trying to access are of the format xyz.domain.tld).
Am I doing something wrong here?  I noticed that the launch options are running Chrome.  Is it running the Steam internal Chromium build or do I have to have Chrome installed?  I'm only running Firefox on my Steam Deck right now.

Yeah you need Google Chrome installed, also you can just put github.com no need for www

@moraroy I was only using Github as an example, I have a web game that I'm hosting at a personal domain that is in the form xyz.my-domain.tld.  I just didn't want to post a domain hosted from my home internet publicly.

@CardcaptorRLH85 oh ok..OK... what you could do if it doesn't work initially..... Is Create a dummy shortcut like input in the text field... google.com... and then in the shortcut that is made edit the launch options.. and input your actual url that you want. Make sure Chrome is installed first though

@moraroy I'm still seeing the same issue both before and after editing the launch options.  Chrome is installed from Flatpak on my deck and when I click Play it flashes to cancel then stop and then back to play as though it had just crashed (or closed).

@CardcaptorRLH85 are you forcing comptabability? Make sure it's unchecked and check your url in the launchoptions no reason why it wouldn't work.. make sure chrome is functioning and working on its own

@moraroy Chrome is running by itself and everything looks right in the launch options.  Here's my launch options line run --branch=stable --arch=x86_64 --command=/app/bin/chrome --file-forwarding com.google.Chrome @@u @@ --window-size=1280,800 --force-device-scale-factor=1.00 --device-scale-factor=1.00 --kiosk https://github.com/ --chrome-kiosk-type=fullscreen --no-first-run --enable-features=OverlayScrollbar

@CardcaptorRLH85 yeah that's the issue then you need to install chrome via the Discover Store on arch linux

@moraroy That's strange, I did install from the discover store.  Pacman isn't currently useable on the Steam Deck without modifying system files.

@CardcaptorRLH85 wow that is weird, maybe Chrome is no longer being installed there? This may be a new problem..

@moraroy I see the com.google.Chrome folder in /home/deck/.var  Is this where it should be to work with NonSteamLaunchers?

@moraroy Sorry for the double-post, it looks like we posted our last comments at about the same time.  I've tried changing the Target and Start In based on the location of Chrome on my deck. It's still not running. Then I tried to change the command since it still mentions /app/bin/chrome and still got nothing. I'm not sure what has to change for this feature to work again but it looks like either SteamOS or Discover has moved where Flatpak apps are stored and it has broken your web shortcuts.

@CardcaptorRLH85 interedting...is the install directory moved just for chrome? Or does this happen to any new flat packs now installed?

@moraroy as far as I can tell, all flatpak apps are in the new location. However just Chromium based apps Chromium and Chrome fail to load. (Edge actually loads just fine.)

repo: moraroy/NonSteamLaunchers-On-Steam-Deck


text:
Add AreCollectionsSame

It's pretty useful and widely used feature, to check if two unordered collections are the same, probably with default and nondefault comparers

you can do something like
var seq1 = list1.OrderBy(t => t);
var seq2 = list2.OrderBy(t => t);
return seq1.SequenceEqual(seq2);

or maybe
var set1 = list1.ToHashSet();
var set2 = lits2.ToHashSet();
return set1.SetEquals(set2);

@leandromoh I understand that I can, I actually created test for myself. I was just hoping to see the most efficient way in this library available for everyone.
|           Method |     source |           Mean |        Error |       StdDev |         Median |
|----------------- |----------- |---------------:|-------------:|-------------:|---------------:|
|  SortThenCompare |    Size 10 |       766.6 ns |     22.31 ns |     63.28 ns |       748.9 ns |
| ConvertToHashset |    Size 10 |       624.6 ns |     10.01 ns |      9.36 ns |       623.3 ns |
| OneExceptAnother |    Size 10 |       871.5 ns |      9.11 ns |      8.52 ns |       872.0 ns |
|  SortThenCompare |   Size 100 |    10,697.1 ns |    188.36 ns |    334.82 ns |    10,578.2 ns |
| ConvertToHashset |   Size 100 |     3,730.6 ns |     74.27 ns |    151.71 ns |     3,661.1 ns |
| OneExceptAnother |   Size 100 |     6,319.3 ns |    115.18 ns |    102.10 ns |     6,320.1 ns |
|  SortThenCompare |  Size 1000 |   179,022.0 ns |  2,306.86 ns |  2,157.84 ns |   178,539.6 ns |
| ConvertToHashset |  Size 1000 |    36,351.3 ns |    721.78 ns |  1,951.38 ns |    35,832.8 ns |
| OneExceptAnother |  Size 1000 |    58,811.9 ns |    843.54 ns |    789.05 ns |    59,027.8 ns |
|  SortThenCompare | Size 10000 | 2,428,770.1 ns | 47,772.12 ns | 81,120.69 ns | 2,397,744.7 ns |
| ConvertToHashset | Size 10000 |   345,496.7 ns |  6,631.58 ns | 10,518.39 ns |   344,402.7 ns |
| OneExceptAnother | Size 10000 |   598,397.5 ns | 11,831.15 ns | 16,967.88 ns |   599,364.8 ns |

        [Benchmark]
        [ArgumentsSource(nameof(Datasource))]
        public bool SortThenCompare(TestSource source)
        {
            var x = source.XValues;
            var y = source.YValues;

            return x.OrderBy(t => t).SequenceEqual(y.OrderBy(l => l));
        }

        [Benchmark]
        [ArgumentsSource(nameof(Datasource))]
        public bool ConvertToHashset(TestSource source)
        {
            var x = source.XValues;
            var y = source.YValues;

            return x.ToHashSet().SetEquals(y.ToHashSet());
        }

        [Benchmark]
        [ArgumentsSource(nameof(Datasource))]
        public bool OneExceptAnother(TestSource source)
        {
            var x = source.XValues;
            var y = source.YValues;

            return !x.Except(y).Any() && !y.Except(x).Any();
        }

@Lonli-Lokli This has been implemented in SuperLinq, and will be released with v4.1.0 in the next few weeks.

authors: Lonli-Lokli leandromoh viceroypenguin

repo: morelinq/MoreLINQ


text:
Underscores inserted in enum name with mixed lowercase and numbers

Given an enum definition like
enum Format {
    mp3 = 1;
    m4a = 2;
}

the enum names in Java will be "MP_3" and "M_4A"
Expected result was "MP3" and "M4A".
If the definition is uppercased this works as expected.
This may be a regression introduced when fixing https://github.com/morimekta/providence/issues/46

Fixed, now with much more tests.

authors: morimekta zhaakhi

repo: morimekta/providence


text:
Overflow hidden of the parent container makes the suggestions list cropped

I use Ionic with react. I put the autosuggest in IonHeader.
he problem is: suggestions list is cropped.
I tried: overflow: visible and even position: fixed and it didn't help. Overflow: visible for all parent have side effects. So I appended react-autosuggest__suggestions-container to the root (e.g. body) with code like this:
let root = document.getElementById('root');
if (elm.parentElement === root) {
return;
}
let top = this.getOffsetTop(elm);
let left = this.getOffsetLeft(elm);
Object.assign(elm.style, {
position: 'absolute',
left: left + 'px',
top: top + 'px',
'z-index': 10000
});
root.appendChild(elm);
It looks good now, but: click on suggestion doesn't work. Please help me. I spent a plenty of time trying to make it work... =)

I don't know anything about Ionic I'm afraid. In general though, I'd strongly recommend against doing manual DOM manipulation like this. Even if it works at first, you'll often find that React updates later conflict with your changes and your whole render crashes completely.
If I understand the problem, your autosuggest is inside a header where the suggestions container can't be shown, so you need it to appear elsewhere, is that right?
I think this is possible. Instead of manually editing the DOM, you can use React's built-in support for doing this kind of thing with portals: https://reactjs.org/docs/portals.html. You get a reference to a DOM element elsewhere where you would like your autosuggest to appear, you pass that reference through to your renderSuggestionsContainer, and then you render the container into a portal using your DOM element, and the suggestions container will appear over there.
You should be able to use that to render your suggestions container anywhere else on the page you would like, without manually moving and editing DOM elements. I don't have a full example I'm afraid, but if you get that working it'd be great if you could share the code here, for others hitting the same issue in future.

authors: pimterry predictron-cloud

repo: moroshko/react-autosuggest


text:
Move to Haskell2010

Not sure if there's any deeper reason why this was written in Haskell '98...
Also not sure if this pull request will look like I want it to look (I just intended to PR 26c70980dc9168c75c79623eaad9994aa1f52363, but it only accepts this PR with #18 included).

Maybe you can rebase this now that I merged the other pull request?

Hm. Here it shows the PR as acceptable ("This branch is up-to-date with the base branch"). What happens if you just try to pull this?

Ah, yes, git is smart enough to figure out what to do. I've merged this now, thanks for contributing.

authors: mortberg tulcod

repo: mortberg/cubicaltt


text:
Installing, licencing, usage

Hi,
I've read your article on codemonth.dk
I would be interested in trying the testdata_ninja as a helper library for mock data setup for unit testing using utPLSQL v3
So few questions:

[x] what is the license for testdata_ninja?
[x] what other libraries are required ad what is their licensing?
[x] How do I install it all
[ ] where can I read about how to use it? Is there a user guide?

Thanks
Jacek

        Hi Jacek,License is MIT, and if not updated to that I will do that after work todayTo really be useful you should use it together with random_ninja also written by me, although it is not a requirement. Should also be MIT but I will make sure today after work. Should be an install guide for both libraries in the blog entry on codeMonth. Look for the gist link within the article http://www.codemonth.dk/code_is_good/dev_qa_prod.assert?condition=codemonth::::11370:Updated-random_ninja-and-testdata_ninja-packagesI have not written an actual user guide although the libraries should be pretty much self explanatory if you read through the different blog entries on codemonth website:Random_ninja:http://codemonth.dk/code_is_good/dev_qa_prod.assert?condition=codemonth:::19::Random-data-in-OracleTestdata_ninja:http://codemonth.dk/code_is_good/dev_qa_prod.assert?condition=codemonth:::20:If you need a more detailed explanation or have any additional questions let me know. Thanks,Morten---- On Mon, 19 Jun 2017 03:39:59 +0800  Jacek Gębal<notifications@github.com> wrote ----Hi, I've read your article on codemonth.dk I would be interested in trying the testdata_ninja as a helper library for mock data setup for unit testing using utPLSQL v3 So few questions: -[ ] what is the license for testdata_ninja? -[ ] what other libraries are required ad what is their licensing? -[ ] How do I install it all -[ ] where can I read about how to use it? Is there a user guide? Thanks Jacek  —You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

I will update the readme later tonight with a simple user guide.

Awesome!
Thank you @morten-egan.
I will try to spend some time trying out your test data for data setups using utPLSQL v3.
Looks like you did a piece of amazing work.
I'm truly surprised I didn't find it any sooner.
It seems that combining utPLSQL v3 with tesdata_ninja can give a huge value to unit testing of ORacle databases.

I'm writing a user guide at the moment. Should be done within this week.

User guide written: https://github.com/morten-egan/testdata_ninja#user-guide

authors: jgebal morten-egan


text:
WandBLogger throws an Exception in any mode except 'online'

Exception has occurred: RuntimeError
get_file_artifact can only be called after running init()
File "/usr/lib/python3/dist-packages/composer/loggers/wandb_logger.py", line 282, in download_file
raise RuntimeError('get_file_artifact can only be called after running init()')
File "/usr/lib/python3/dist-packages/composer/utils/file_helpers.py", line 514, in _get_file
object_store.download_file(
File "/usr/lib/python3/dist-packages/composer/utils/file_helpers.py", line 474, in get_file
_get_file(
File "/usr/lib/python3/dist-packages/composer/trainer/trainer.py", line 1491, in _try_checkpoint_download
get_file(
File "/usr/lib/python3/dist-packages/composer/trainer/trainer.py", line 1535, in _get_autoresume_checkpoint
self._try_checkpoint_download(latest_checkpoint_path, save_latest_remote_file_name, loggers,
File "/usr/lib/python3/dist-packages/composer/trainer/trainer.py", line 1390, in init
autoresume_checkpoint_path = self._get_autoresume_checkpoint(
File "/workspaces/houghnet/houghnet/train.py", line 78, in train
trainer = Trainer(
File "/workspaces/houghnet/houghnet/train.py", line 156, in main
train(train_loader, test_loader, accelerator)
File "/workspaces/houghnet/houghnet/train.py", line 163, in
main()
RuntimeError: get_file_artifact can only be called after running init()
Expected behavior
Wandb supports three modes: online, offline and disabled. It's expected from composer.loggers.WandBLogger to support these modes in the same manner.

Thanks for flagging this! We will fix soon :)

@anastasia-spb Can you please provide a bit more info so I can reproduce and add a unit test for this?
I'm a bit confused how this error was raised since I think it should be covered in all the paths... If you can describe your scenario, I can reproduce it and fix

@anastasia-spb  bumping this! would love to get some help reproing!

Closing as I'm not able to reproduce. Please feel free to open a PR with a repro if possible!

repo: mosaicml/composer


text:
Fix loss reporting

Need to add to total_loss before call to scaler.scale().

Yup, I can confirm that this reports the appropriate value. LGTM!

authors: jbloxham moinnadeem


text:
Move pydantic pin to deepspeed

What does this PR do?
Only deepspeed has errors with pydantic 2. Moving the pin down to there as we don't actually use in composer normally

The pydantic error comes from Deepseed. They failed to pin and error. We don't use pydantic


text:
Remove demo column

While we sort out exactly which models will have demos and how they will be hosted, lets remove the link.

Talked to Abhi... we will get the 7b demo up and just keep this

repo: mosaicml/llm-foundry


text:
add formatting for llama2chat

This adds formatting classes for ChatML (MPT) and Llama2 chat. In hf_chat.py we automatically infer the formatting needed based on model name for these two formatting types.
Why? This makes it a lot easier to test MPT-chat and llama2 chat models with our hf_chat.py script. Also, these formatters can be useful for storing chat data in a syntaxless way (list of lists) and easily reformatting for ChatML or llama2 style chat finetuning. It also allows for easy translation from llama2 formatting to ChatML formatting.

tokenizers added functionality that covers this behavior


text:
Support for OpenMM

I am considering to use Foyer as tool for a tool to generate OPLSAA forcefields from smiles string definitions of molecules and run them OpenMM.
Currently, I can only find examples that write in the GROMACS format.
Are there examples, that I am missing that describe different file formats?
I am most interested in OpenMM, and I thought that since you are using there file format to describe forcefields, that should be an easy step.
Or am I mistaken and Foyer does not support this?

Hey @InnocentBug , thank you for the issues raised today and welcome! At the moment, this should be totally possible with our workflow, using ParmEd as the backend data structure (currently the default option).
Here is a link to an example in the ParmEd docs that loads in a GRO and TOP file as the starting point. Also, it should be possible to take the example there, and once you have parametrized a system with foyer to go straight to an openMM system. See the snippet below.
# OpenMM Imports
import sys
import simtk.openmm as mm
import simtk.openmm.app as app

# ParmEd Imports
from parmed import load_file
from parmed.openmm.reporters import NetCDFReporter
from parmed import unit as u

# mosdef imports
import mbuild as mb
import foyer

eth = mb.load("CC", smiles=True)
eth_box = mb.packing.fill_box(eth, n_compounds=100, box=mb.Box(lengths=[1,1,1]))

oplsaa = foyer.forcefields.load_OPLSAA()

param_struct = oplsaa.apply(eth_box)

# Create the OpenMM system
print('Creating OpenMM System')
system = param_struct.createSystem(nonbondedMethod=app.NoCutoff,
                          constraints=app.HBonds, implicitSolvent=app.GBn2,
                          implicitSolventSaltConc=0.1*u.moles/u.liter,
)

# Create the integrator to do Langevin dynamics
integrator = mm.LangevinIntegrator(
                        300*u.kelvin,       # Temperature of heat bath
                        1.0/u.picoseconds,  # Friction coefficient
                        2.0*u.femtoseconds, # Time step
)

# Define the platform to use; CUDA, OpenCL, CPU, or Reference. Or do not specify
# the platform to use the default (fastest) platform
platform = mm.Platform.getPlatformByName('CPU')

# Create the Simulation object
sim = app.Simulation(param_struct.topology, system, integrator, platform,)

# Set the particle positions
sim.context.setPositions(param_struct.positions)

# Minimize the energy
print('Minimizing energy')
sim.minimizeEnergy(maxIterations=500)

# Set up the reporters to report energies and coordinates every 100 steps
sim.reporters.append(
        app.StateDataReporter(sys.stdout, 100, step=True, potentialEnergy=True,
                              kineticEnergy=True, temperature=True)
)
sim.reporters.append(
        NetCDFReporter('dhfr_gb.nc', 100, crds=True)
)

# Run dynamics
print('Running dynamics')
sim.step(10000)

The major changes I added were: building system using mbuild, parametrizing using foyer's oplsaa xml, using the parametrized parmed structure from foyer to then set the rest.

Thanks for the explanation.
I think I know where the confusion cam from.
I wanted to write files first that describe the topology and forcefields first. (Preferably in the native XML format of OpenMM.)
And have those files on record for reproducibility.
But your approach should work too.
I can always write a state with OpenMM and read that one.

authors: InnocentBug justinGilmer

repo: mosdef-hub/foyer


text:
Is "append" to index an option

I am looking through both docs, code, and test usage, and perhaps I am missing something, but it appears the options are A. Index does not exist, and on a single run of elasticsearch_loader, you can load an entire index or B. Index does exist and you specified --delete therefore the existing index will be deleted.  When you try to run elasticsearch loader on an index that already exists, but do not specify --delete, it fails with error: elasticsearch.exceptions.RequestError: TransportError(400, u'index_already_exists_exception')
Thus, my request, if possible, is "append" to existing index.

Sure, this should be the default behaviur, maybe are you passing index-settings-file when you trying to update the existing index?

Ah, I am AM doing that! I didn't know that is what is causing that... perhaps a note in the help screen or readme that states that the index settings only needs to be provided when first creating the index.
Alternatively, if the index is exists, and --delete is NOT specified, perhaps we ignore the index-settings file and pass a warning to the user that since the index exists the settings are ignored. (or perhaps we grab the current settings and if they don't match what is passed, we fail out, otherwise, just append as is?) Lots of options with different levels of complexity, but thanks for pointing out my base mistake.

authors: JohnOmernik moshe


text:
Status Code : 403

Describe the bug
Downloads returns Status Code : 403
To Reproduce
Steps to reproduce the behavior:
Paste url, click download, and error
Add logs from DevTools
Full logs added
Expected behavior
Should download
Screenshots

OS: Win 10
Version --
App Version 2.6.1

Additional context
Complete Log:
-1727458919302.log

Thank you for reporting. Seems like there is a problem with the library that fetches the video
I'll push a fix as soon as they deliver a fix on their side
https://github.com/fent/node-ytdl-core/issues/1313

Released the fix under 3.0.0

authors: moshfeu rommansabbir


text:
Click search icon cancel current downloading file

Describe the bug
I am downloading a youtube video 2 mp3 by the app and I want to download another video to mp3 so i copied the url and past it then click search icon then the previous video which was still downloading disappeared and only the last url video showed.
To Reproduce
Steps to reproduce the behavior:

Go to 'Home'
Past a youtube  Url
Click  on 'Download.'
Past another youtube  Url
Click  on 'Download.'

Expect :new video appended to list of videos while the previous still downloading
What happened: old video disappeared and only the new one showed.

Yah, I don't satisfy this either. You idea is interesting about adding videos to the list instead of replace them. The thing is that the user will not see any difference in the list so he will not know that something happened.
My vision about that is to hold 2 lists, one for the search results and second for the download queue.
What do you think?

i made this design for you as a quick ui as i imagine it should be
when user add more urls its add to the top of the list.

Nice design! Are you a professional UX?
About adding the videos in the top, after some searches, the list will become very long..

so provide a button to delete row and delete all btn depend on user decision  he can delete one or all list.

authors: iahmedhendi moshfeu


text:
Invalid OTP's error message is not proper while VC activation

Describe the bug
Invalid OTP's error message is not proper while VC activation in both android and ios
To Reproduce

Download a new VC
Open it in detailed view
Click on the activate button and click on "yes, i confirm"
Enter a invalid OTP

Expected behavior
Should prompt an error message as in below screenshot and should allow you to retry for another attempt

Actual behavior
The error message prompted is wrong, and it is not letting to retry
Screenshots
https://user-images.githubusercontent.com/102220709/228553669-881088ab-6371-4d85-bd88-9740e816aa0d.mp4
Add the screenshot of the profile page with commit id

Tested on:
Phone make/model: Google Pixel 6
OS: Android 13
BLE version : 5.2
MOSIP Version : 1.2.0.1
MOSIP server : qa-1201-b2.net
APP link : https://github.com/mosip/inji/actions/runs/4511525622

This bug has been moved to Jira.
Link to Jira bug: https://mosip.atlassian.net/browse/MOSIP-26854

The root cause for this issue is the wallet binding API is returning an generic error instead of specific errors.
RESPONSE FOR INVALID OTP - {"errors": [{"errorCode": "binding_auth_failed", "errorMessage": "binding_auth_failed"}], "id": null, "metadata": null, "response": null, "responsetime": null, "str": null, "version": null}.
This was confirmed by @lsivanand .
The response should contain specific error codes with proper error messages then only we can proceed with fixing this issue. There is a dependency from Mimoto server. Once the issue is fixed from the server side then only we can proceed with this. Hence this is not going in the Hotfix release.

verified as fixed in the build with commit id in the below screenshot.

authors: Pragya279 Raginikrishnamurthy adityankannan-tw santhoshsunder

repo: mosip/inji


text:
🛑 api.buymezobo is down

In 9bc7349, api.buymezobo (https://core.buymezobo.ng/api/healthcheck) was down:

HTTP code: 0
Response time: 0 ms

Resolved: api.buymezobo is back up in ebf6afd after 46 minutes.

repo: mossgrid/sentinel


text:
🛑 Site is down

In 97bca84, Site (https://motionarium.top) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Site is back up in d891453 after 19 minutes.

repo: motionarium/upptime


text:
[Feature] Performe actions in timeline

Is your feature request related to a problem? Please describe.
I don't know if there is already a way to do that but I would like to run some js code (whatever) between steps in a timeline
Describe the solution you'd like
const sequence = [
  ["nav", { x: 100 }, { duration: 1 }],
  // run some code here
  [{ action: ()  => {
  // code
  }],
  ["nav li", { opacity: 1 }, { duration: 0.3, delay: stagger(0.1) }],
]

timeline(sequence)

related to #130

authors: Ladvace marpe

repo: motiondivision/motionone


text:
Create Rastaxico.py

Describe tus cambios
(Opcional) Sobre todo aconsejable si la "Pull Request" se corresponde con una corrección adicional y no con la presentación de un ejercicio.
Comprobaciones
Asegúrate de cumplir los siguientes puntos antes de realizar la "Pull Request":

[ ] El título de mi Pull Request sigue este formato: "#[número] - [lenguaje_utilizado]". (Ej: #00 - Python")
[ ] El nombre el fichero que se corresponde con el de mi usuario en GitHub más la extensión del lenguaje. (Ej: mouredev.py)
[ ] El fichero de corrección se encuentra dentro del directorio del ejercicio y en una carpeta con el nombre del lenguaje de programación utilizado en minúsculas. (Ej: 00/python/mouredev.py)
[ ] He revisado que el nombre del directorio del lenguaje no es conflictivo:

c#, no csharp
c++, no cplusplus
go, no golang
javascript, no js

[ ] Únicamente he incluido los ficheros de ejercicios. No se aceptarán Pull Requests que contengan archivos adicionales asociados a editores de código o semejantes.

Información

Tienes toda la información sobre los retos semanales en retosdeprogramacion.com/roadmap.
Cada semana se realizará la corrección en directo y publicación de un nuevo reto en twitch.tv/mouredev.
Recuerda que tienes un grupo de apoyo llamado "reto-semanal" en Discord.

PR duplicada, solución aceptada en #6277

repo: mouredev/roadmap-retos-programacion


text:
T1K for PGx

Hi,
I am trying to use T1K for PGx, following the step-by-step plan described in the vcf_database. Unfortunately, I am not getting the expected results for my samples (for example, I get for the CYP2D6 gene, *4/*86 as output, where I expect *1/*4).
This is the case for both the reference file I created for CYP2D6 according to the step-by-step plan and the reference files in the cyp2d6_idx folder on Git.
What could be possible reasons for not getting the expected outputs?
(The data I am using is from the Genetic Testing Reference Material Coordination Program (GeT-RM). These reference materials contain mutations of clinical importance that have been confirmed by multiple volunteer laboratories using different testing platforms, including for the CYP2D6 gene.)

Do you mean you did not get CYP2D6*1 series in the output? Could you please share the .dat generated from the procedure? Thank you.

The txt file looks fine, and I can generate the reference fasta files containing the CYP2D61 or CYP2D61.XXX . So for the *4/*86 and *1/*4 is the genotyping results?
One possible reason is that CYP2D6 is highly homologous to CYP2D7, and you may need to put in some CYP2D7 gene sequences in the reference.

Thank you for looking into the file!
CYP2D6 is not the only gene I have looked at; I have also examined CYP2C9, CYP2C19, CYP3A5, and CYP4F2. For these genes as well, I do not get the expected output for the 16 samples I tested. If the .dat file looks good, is there another possibility for why I am not getting the expected output for these other genes?

Can you show me your running commands and your genotype.tsv file? Is your data RNA-seq or other sequencing platform?

The WGS files are available at: https://www.ebi.ac.uk/ena/browser/view/ERR1955327
The command I am using is: run-t1k -f T1K/vcf_database/cyp2d6_idx/cyp2d6_dna_seq.fa -1 ERR1955327_1.fastq.gz -2 ERR1955327_2.fastq.gz --od ERR1955327/cyp2d6 --alleleDigitUnits 1 --alleleDelimiter . -t 16
The output that results from this is:
T1K_ERR1955327_1_genotype.ods
Thank you very much for your effort.

I would recommend concatenating all the dna_seq.fa from cyp genes into a combined fasta file. This way it may resolve reads that are aligned to multiple cyp genes. Another parameter to tune is the "-s" option, the default 0.8 might be to lenient. You may consider trying values like 0.9 and 0.97.

authors: mourisl nbiesot


text:
support for ONNX

please support for ONNX

No found device
I try to install devic with ncsdk 1.xx, as follows error:
when make examples  for No found device
Thanks

This PR to old to merge, showing 344 files changed.  There is currently no support for ONNX planned for NCSDK, but if you would like to use ONNX models with the NCS or NCS2 devices you can use Intel's distribution of the OpenVINO toolkit:
https://software.intel.com/en-us/neural-compute-stick/get-started
closing PR, please submit new PR if think changes are still needed with the latest release of NCSDK.

authors: molyswu nealvis zsp1197

repo: movidius/ncsdk


text:
Implement Web3 Wallet Connection

Follow the instructions here https://docs.family.co/connectkit

@Baah-Danso-Kenneth Are you up for this?

@mikehammond Yes please

@Baah-Danso-Kenneth Please take it away

Here’s a more concise and improved version of your message:

Hi @mickeymond,
I've finished the wallet connection but accidentally deleted the .env.local file, so I’m missing the URL and API keys. Should I create a PR for you to add them, or could you send the details to my Gmail?
kwadwobaahdanso@gmail.com
Thanks!

@Baah-Danso-Kenneth I attempted locating you on Telegram and sending you the .env.local content.
Check if you have received it.

authors: Baah-Danso-Kenneth mikehammond

repo: mowblox/payflow


text:
Schema/UiSchema errors crash the Playground on React 16

Prerequisites

[x] I have read the documentation;
[x] In the case of a bug report, I understand that providing a SSCCE example is tremendously useful to the maintainers.

Description
When passing certain Schema or UiSchema documents as props, the Form component throws an exception on the render method. On the Playground, these are caught by a try/catch in the Editor component: https://github.com/mozilla-services/react-jsonschema-form/blob/e4378c2a0b738eeee151e82c4533d2a79b80346c/playground/app.js#L195-L199
This works because the setState call inside the try/catch and the render call happen inside the same event loop iteration. This has never been guaranteed to happen, and the behaviour changed in react 16.
Steps to Reproduce

Compile the Playground on React 16
Make uiSchema reference a widget that doesn't exist

Expected behavior
Application works. Form goes into the invalid state. When the error is fixed, it goes back to normal. No unhandled exception is thrown.
Actual behavior
Application immediately crashes as an uncaught exception is thrown in Form.render().
Version
react-jsonschema-form: 1.0.2
react: 16.2.0
react-dom: 16.2.0

It wouldn't surprise me to discover that there were enormous problems with how we handle updating the state. For instance, see https://github.com/mozilla-services/react-jsonschema-form/issues/446. I don't know React well enough to try to clean this up, but patches would be welcome!

I think the original idear of "setImmediate", "safeRenderCompletion" stuff is "trying to catch exceptions thrown by getWidget".
I would highly recommend that "getWidget" should return a default widget, or similar to "UnsupportedField" for unsupported schema, return an "UiSchemaError" component to display error message in the form, instead of throwing an exception. So that those "setImmediate" "safeRenderCompletion" workaround can be removed.
Here is an official doc about Error Handling in React 16. I haven't tried but I think adding "componentDidCatch" into "SchemaField" should work and the outcome - ’display a “Something went wrong” message to the user‘ is the same as above.

authors: ferranmesas glasserc knilink

repo: mozilla-services/react-jsonschema-form


text:
Book suggests installing cargo-vet from crates.io, which is currently just a placeholder crate

https://mozilla.github.io/cargo-vet/install.html says Installing cargo vet can be done through Cargo $ cargo install cargo-vet but https://crates.io/crates/cargo-vet is just a "hello world" placeholder.
Please either release something on crates.io, or change the book to instruct building from git.

See-also #148
The book does include a note telling the user to install through cargo install --path . on the page you linked, however perhaps it could be made more visible.

Sure, I can move the note further up.

authors: bholley mystor vincentdephily

repo: mozilla/cargo-vet


text:
Update cubeb-coreaudio-rs.

Pull in version of cubeb-coreaudio-rs that fixes a bug with downmixing stereo input to a mono channel.

cc @padenot @Pehrsons as people who commented on the coreaudio PR. :)

repo: mozilla/cubeb


text:
Leaving the NDA-STAFF group removes you from the old NDA group and other access groups

[Precondition]:
Mozillians account with MFA - login via Github with Gmail
User is in NDA group( the old one), NDA-STAFF group and other 2 access groups ( in this order)
STR:
1.Leave the NDA-STAFF group
[Expected result]:
The user should still be a member of the NDA group and other 2 access groups
[Actual results]:
All groups are removed (e.g. NDA, NDA-STAFF and the other 2 access groups)
@akatsoulas

Fixed in #3868

Verified as fixed!

authors: akatsoulas rleitan

repo: mozilla/mozillians


text:
sign invalid addons so we can run tests in automation

I'm working on helping the developer productivity team run some try runs running our tests with signed addons.  This is to prepare for when we run tests on branches where the signed addon preference is enabled by default.  Recently, I was talking to kumar in #amo and was mentioning some of my problems where we need to have addons signed so the tree doesn't show up as red when the pref is enabled.  However, many of these addons are invalid so we can test scenarios.  He advised me to open an issue for this since the addon validation doesn't sign invalid addons.  I attached a zip of addons that were problematic
baddons.zip
#bad
corrupt.xpi
corruptfile.xpi
empty.xpi
#bad id
multi_badid.xpi
#empty id in install.rdf
multi_broken.xpi
multi_signed.xpi
#Error: ID was empty in install.rdf
multi_unsigned.xpi
#version already exists
test_bug526598_2.xpi
#did not get signed
test_bug526598_1.xpi

Well, huh, this might be tricky. I'll work on this to see if we can come up with something.

@jasonthomas can sign add-ons directly I believe against trunion. Given that these are broken and should never go through validator, that might be our best bet.

The signing will be done with this FYI https://github.com/jasonthomas/sign-addon

We also discussed the end goal for releng in IRC. They need to protect against regressions in code that handles corrupted add-ons. Since signed add-ons are required in the real world, it won't be as easy to install corrupted add-ons -- thus, running these tests with the default prefs might not be helpful. One idea presented was to run all these regression tests on the non-public build which would still accomplish the goal of catching code regressions.

We visited this issue earlier in the project and we don't want to have a second set of builds running tests for unsigned addons in automation.  We thought there were security issues with this approach and a lot of duplicated content/load on our CI farm for little benefit.  I added this topic as a discussion item for the next addons meeting :-)

baddons-signed.zip
I've signed the ones that I could, which were test_bug526598_1.xpi, test_bug526598_2.xpi and empty.xpi. The empty.xpi was signed by providing a guid (I used 'empty@tests.mozilla.org) since it was just a empty file.
I didn't try to sign the mutl package since only the inner add-ons need to be signed https://github.com/mozilla/olympia/blob/12f806169a5b62e79ccf1e7fa88ee7e609dd173b/lib/crypto/packaged.py#L128

authors: andymckay jasonthomas kmoir kumar303

repo: mozilla/olympia


text:
Fix pull request #12131 so it can be merged again

It looks like the only problem that required us to do a revert of the patch is the fact that the unit tests in #12131 don't pass locally. However, they don't even run at all on the bots, hence we thought this was safe to merge given that all tests passed.
@timvandermeij Investigate why the bots did not run the test so this won't happen again.
@jsg2021 Could you improve the unit test so that we can try merging the patch again? Note the comment on the PR about what fails. In particular, we need to make sure that the tests work locally too; you can do that by running gulp unittest to test in the browser and gulp unittestcli to test in Node.js. Both should pass in order for the patch to be merged again. Thanks!

yep, will do.
shouldn't npm test run both those? I was using that to run and validate locally.

Ah, now I understand. npm test runs the unit tests in Node.js and it was made this way so that Travis CI has a direct entrypoint for running the tests. The gulp unittest command instead runs the unit tests in the browser instead. There is a difference because we cannot run the same tests in the browser and in Node.js due to platform differences.

In short, the npm commands should not be used for development. Only the gulp tasks are valid for that. See https://github.com/mozilla/pdf.js/wiki/Contributing for more information.

I did find the root cause for the fact that the test failure was not reported on the bots, so I'll make a patch for that so that such problems don't go unnoticed again.

@timvandermeij I've fixed the tests and pushed the branch. Should I issue a new pull request?

Yes, indeed. We have just merged the fix for the test reporter so that such problems don't go unnoticed anymore in the future, so please rebase onto the current master branch and create a new PR. Thank you!

Closing since the test reporter issue is fixed and there is a new PR for this now.


text:
Retain vectors and the fonts when rendering

Appears PDF.js rasterizes a vector-based PDF.
Is there a way, to retain the vectors and the fonts when rendering?
example file: https://www.dropbox.com/s/mpcoiaoeud5f4al/XSC - Drawings.pdf?dl=0

The PDF.js paints everything on canvas (however we set limitation on size of the canvas). Since you did not provide enough details about the issue, I assuming you are talking about the page picture looking rasterized at large zoom (for letter/a4 size page it happens at about 400%) -- that where we stop repainting and using CSS zoom.
Closing as answered and duplicate of #6419.
If you are interested in general about PDF.js ability to respresent information as vector graphics, see our work-in-progress at https://github.com/mozilla/pdf.js/tree/master/examples/svgviewer.

Thanks Yury!

authors: theoryshaw yurydelendik


text:
Problem with some letters when zoom value is 100% in IE

Dear brothers,
I am using pdf.js at link http://mozilla.github.io/pdf.js/web/viewer.html to open the PDF file and I have a problem that the letter 'i' and 'r' were upper case(or higher) when the zoom value of pdfjs is between 96% to 103% and the zoom value of browser is 100% on IE. I am using IE version 11.0.9600.17031.
Please see the attachments for more informations.
Thanks!

PRB-DataOverview-2012.pdf

Closing IE11/Edge (non-Chromium-based) issues in response to #11211. Please note that, as outlined in https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-support, only bugs which completely prevent the library and/or the default viewer from running will be accepted.

authors: haql timvandermeij


text:
Photon design

It would be nice if PDF.js's design would be in line with Firefox's.
To do that, here are some quick changes that would be possible:

Remove background gradients/textures
Use photon colors: https://design.firefox.com/photon/visuals/color.html#grey (use visual layering to pick shades of gray: https://mozilla.invisionapp.com/share/J6GQCZR2PZY#/screens/289803032_Desktop_Common)
Use photon icons: https://design.firefox.com/icons

It's true that the reader looks very outdated... compared to the browser

Fixed by #11077.

authors: Huluti nt1m timvandermeij


text:
Add Math.round to get border width comes in float for issue 14203

The border width is expected to be 3 but tracing Class Annotation, I found the width actually comes as float 2.99998 so when the width passed to Number.Isinteger in Class AnnotationBorderStyle:setWidth() the width gets lost.
Number.Isinteger makes sure that the value should be integer.Taking it into account,adding Math.round can solve this issue I think.

This is my first time Pull request in my life time.I read code contribution guide carefully but I apologize if there are any mistakes in this PR.

For the commit message, I'd suggest the following wording:
Handle non-integer Annotation border widths correctly (issue 14203)

The existing code appears to be wrong, since according to the PDF specification the border width of an Annotation only has to be a number and not specifically an integer. Please see:
 - https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#page=392
 - https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#G11.2096210
 - https://www.adobe.com/content/dam/acom/en/devnet/pdf/pdfs/PDF32000_2008.pdf#G6.1965562

/botio test

Note that the test results above look good, despite the failures (those tests unfortunately fail intermittently).
https://github.com/mozilla/pdf.js/pull/14391#discussion_r774094303 also asked for a new unit-test as well (since you're new to the code-base I'll make a suggestion to help speed this up), and I'd thus suggest changing https://github.com/mozilla/pdf.js/blob/41dab8e7b6c1e2684d4afabb8f02e40a874d8e85/test/unit/annotation_spec.js#L417-L420
to the following instead:
const borderStyleInt = new AnnotationBorderStyle();
borderStyleInt.setWidth(3);
const borderStyleNum = new AnnotationBorderStyle();
borderStyleNum.setWidth(2.5);

expect(borderStyleInt.width).toEqual(3);
expect(borderStyleNum.width).toEqual(2.5);

Thank you for the suggestion and approved my commit.
I understand where to add unit tests and  learned great example of commit message.

/botio-linux preview

Thank you for fixing this!

/botio makeref

authors: KouWakai Snuffleupagus timvandermeij


text:
Improve the loadingParams functionality in the API

Move the definition of the loadingParams Object, to simplify the code.

Add a unit-test, since none existed and the viewer depends on this functionality.

/botio-linux integrationtest

Sorry, I ran the tests here but I thought I were in an other PR...

/botio-linux test

authors: Snuffleupagus calixteman


text:
Convert the files in the /src/core folder to ES6 modules

Depends on PR #8364.
Please note that the glyphlist.js and unicode.js files are converted to CommonJS modules instead, since Babel cannot handle files that large and they are thus excluded from transpilation.
Much smaller diff with https://github.com/mozilla/pdf.js/pull/8284/files?w=1.

Sorry, I did not catch this before but we need to use cjs paths e.g. '../shared/util.js'

Blocked by #8285

/botio test

/botio test

/botio-linux test

/botio-linux test

/botio-linux test

/botio test

/botio-linux test

/botio-linux test

I'm thinking we a hitting some performance issue on botio-linux/chrome with the patch. Let's try convert src/shared files in the separate PR.

Also let's try larger timeout.

Also let's try larger timeout.

I've doubled all the timeout values, and also removed the src/shared/ patch, let's see if that helps.
/botio test

OK, once more with feeling; let's try a 10 minute timeout.
/botio-linux test

I reverted the patch that increased the timeout values, since it unfortunately didn't seem help at all :-(

/botio unittest

/botio unittest

/botio-linux unittest

/botio-linux test

/botio-linux test

/botio-linux test

/botio-linux test

/botio test

/botio test

Since Travis seem to be having problems, let's run linting once more just to check that there's no issue.
/botio-linux lint

Hooray! 💥


text:
Move the XRefMock in the unit tests to a central location

This patch helps to avoid code duplication for this mock since more unit tests are depending on it.

/botio unittest

/botio unittest


text:
[leave-open-no-activity] check some keywords in whiteboard to avoid to nag

https://bugzilla.mozilla.org/show_bug.cgi?id=1216304#c11

Fixed in https://github.com/mozilla/relman-auto-nag/commit/674a7090f1d2371d66d87c4637396d8617e0fd15

repo: mozilla/relman-auto-nag


text:
Store test

The intention here is just to show that we can test the main process actions and reducers.

Coverage decreased (-27.3%) to 9.913% when pulling 6364f72f895842d076b2d4780d9cf23ad1fefd3c on ncalexan:store-test into cf2fb22f6532ba9e40d6ae82a0b2dec43fbee3d1 on mozilla:master.

Coverage decreased (-30.4%) to 6.854% when pulling 5e5882f02b8cb6831046ee4935a4ad9df4ede238 on ncalexan:store-test into 23974541f03785912f10cfced7b8dd4d5803132c on mozilla:master.

Coverage increased (+5.6%) to 42.885% when pulling 77fa79b8784a1626ed756fafb8742094df06eb1e on ncalexan:store-test into a00cafb228a2ecd01d604c930d06b752f22bd11a on mozilla:master.

Coverage increased (+5.6%) to 42.885% when pulling 77fa79b8784a1626ed756fafb8742094df06eb1e on ncalexan:store-test into a00cafb228a2ecd01d604c930d06b752f22bd11a on mozilla:master.

authors: coveralls ncalexan

repo: mozilla/tofino


text:
Is there a way to change a scene image and reload it?

Hi,
Is there a native way (without using dynamic=True), to change a scene config, changing it's image, and instruct panellum to reload that scene with the new image url? I need it because I am pixel editing some images with python given different factors and user input and I need to reload the scene with the new image that the user is in, at the moment. If the url must stay the same, it could work, but I wanted to change the url to append a dummy urlparams to force the cache reload.
Thanks,

You can use viewer.addScene('currentSceneId', {new config}); followed by viewer.loadScene('currentSceneId');. Using addScene with an existing scene ID will overwrite the previous configuration, and calling loadScene will then reload the scene with the new configuration.

Ahá.
Simple and direct. I didn't know this was the default behaviour. Thanks a lot, I will try it out!

authors: martinlombana mpetroff

repo: mpetroff/pannellum


text:
Fix typo which meant more parameters were needed and inference bug

#59 introduced a bug which added redundant type parameters on methods.
Fixed an inference issue which stopped methods such as first[A, B, C] working by explicitly adding the method arguments.

👍

authors: mpilquist yilinwei

repo: mpilquist/simulacrum


text:
Typo in correcting roughtail stingray?

https://github.com/mpinsky/OceanAdapt/blob/master/complete_r_script.R#L247
@JWMorley
When doing this for my version of the trawl data set if I do X[SPECIESCODE==8713050104] I get output (340 lines).
When I do X[haulid==20010106] # my haulid is the original COLLECTIONNUMBER I get 26 lines of output.
But if I do X[haulid==20010106 & SPECIESCODE==8713050104], I don't get anything.
Is this a typo of sorts?

Note that this relates to rBatt/trawlData#6

'haulid' is made from EVENTNAME, not COLLECTIONNUMBER, which I think you
know. The first and last line of code makes sense, but I'm not sure why
`X[haulid==20010106] would produce anything, because there is no EVENTNAME
with that number.....

https://github.com/mpinsky/OceanAdapt/blob/master/complete_r_script.R#L247
@JWMorley
When doing this for my version of the trawl data set if I do
X[SPECIESCODE==8713050104] I get output (340 lines).
When I do X[haulid==20010106] # my haulid is the original COLLECTIONNUMBER I get 26 lines of output.
But if I do X[haulid==20010106 & SPECIESCODE==8713050104], I don't get
anything.
Is this a typo of sorts?

Reply to this email directly or view it on GitHub:
https://github.com/mpinsky/OceanAdapt/issues/45

Actually, I didn't know. I must have messed that up. Crap.
I probably never noticed because I was only using the inner strata (are the 2 collections per event reflecting 1 inner and 1 outer per event??).
I'll be in today and ask in person. Thanks for helping me sort this out.

Given that I may have used the wrong column for haulid, this still doesn't explain why I don't get any output for that last line.

The two COLLECTIONNUMBERS are basically replicate tows within an
EVENTNAME. The catches of each COLLECTIONNUMBER are conducted
independently, but all other variables (depth, strata, temperature, gps,
etc.) are identical within an EVENTNAME.
You most likely will want to drop the outer strata of seus entirely. They
were only sampled for 10 years in spring and fall, and were of limited
latitude coverage.

Actually, I didn't know. I must have messed that up. Crap.
I probably never noticed because I was only using the inner strata (are
the 2 collections per event reflecting 1 inner and 1 outer per event??).
I'll be in today and ask in person. Thanks for helping me sort this out.

Reply to this email directly or view it on GitHub:
https://github.com/mpinsky/OceanAdapt/issues/45#issuecomment-154114002

OK, gotcha. I did drop the Outer strata for those reasons.
Is this a situation where there's a net hanging off of each side of the boat, or are the towing through the area two separate times?

They're towed at the same time, and I think the two nets are connected to
eachother (ie share a center trawl-door or something like that).

OK, gotcha. I did drop the Outer strata for those reasons.
Is this a situation where there's a net hanging off of each side of the
boat, or are the towing through the area two separate times?

Reply to this email directly or view it on GitHub:
https://github.com/mpinsky/OceanAdapt/issues/45#issuecomment-154125430

authors: JWMorley rBatt

repo: mpinsky/OceanAdapt