text:
Better error reporting

Hi,
this PR introduces more readable error reporting. It will display message in marble format, like in the screenshot:
  it('should work with a cold observable', () => {
    const provided = cold('-a----b|', { a: 1, b: 2 });
    const expected = cold('--a---b|', { a: 1, b: 3 });

    expect(provided).toBeObservable(expected);
  });

it prints marble comparison and detailed JSON.stringified test messages below
if notification is found in the expected TestObservable, then it will match marble character to this Notification, if not, the character will be equal to ?

Additionally, it fixes not.toBeObservable.
Fixes #51, #11
Please let me know what do you think about these changes :)

after a closer inspection, I think the library was designed to be both used by the jasmine ToBeObservable assertion and the RxJS test Scheduler, but the logic for the two ways is nearly completely separated.
so your PR fix the ToBeObservable way of testing,
and mine fix the "TestScheduler" way.
I think the best would be to merge part of our two work so it uses a single codebase for the two way of testing.
I will try to check your code more closely to see how to fix the TestScheduler from your code

Ok, I checked again, and I realize that there is no link between the two style of marble testing.
It is virtually like if there was 2 library in one.
I don't think there is much value to keep these 2 part together.  It would be more sensible to split the library.
In fact, after closer inspection, I see that the TestScheduler of rxjs already give everythings needed to make marble testing, so I will use it direcly instead of using a third party library.

Thanks @krzysztof-grzybek, apologies for the delay. I will cut a release soon

No worries @brandonroberts , thank you!

repo: synapse-wireless-labs/jasmine-marbles


text:
Bypass login

Create request Bundle for Patient
Adjust patientSlice to patientDataSlice
Add skipLogin feature to bypass auth for dev purposes and mocking patient data when landing on Summary Page
Create local mock Patient Bundle file
Flatten response Bundle to make Vital Signs and Lab Results top level resources
Create fhirReader.js to parse fhir data
Create Collections Tab (unrelated but easy, just wanted to sneak it in)

What screen dimensions are you using?

What screen dimensions are you using?

I'm using the iphone 11 emulator which appears to be 1792‑by‑828‑pixel resolution at 326 ppi
I'll quickly modify this to remove some padding at the top. Then I'll work on introducing a library so we can use % instead of fix pixels so the app can be more responsive.

Refactored using %. Iphone 8 vs 11

That works!

source: gharchive/pull-request

repo: sync-for-science/discovery-mobile-ui

license: MIT

license_type: permissive

license_source: github-api


text:
documentation updates

[x] plots of neuron models
[x] activation module

[x] surrogate gradients
[x] spike functions

[ ] data convention: this is the data format that is expected in this library

@sheiksadique would you say the last point is covered too?

all done

source: gharchive/issue

repo: synsense/sinabs


text:
Data Dictionary

Requested Feature
Is there a data dictionary for the columns generated in Synthea?
For example, I am looking at encounters.csv, and I am not sure about the exact meanings behind the columns of: BASE_ENCOUNTER_COST, TOTAL_CLAIM_COST, REASONCODE.

Let us know if any of the fields are not defined sufficiently.

authors: jawalonoski kennethleungty

repo: synthetichealth/synthea

license_source: bigquery


text:
Ändern eines Antrags von Überstundenabbau auf Erholungsurlaub zeigt weiterhin Überstunden in Urlaubsübersicht

Describe the bug
Wenn man einen Antrag von Überstunden zu Erholungsurlaub ändert, bleiben die Überstunden erhalten. In der Übersicht der Urlaube steht dann nicht die Anzahl der Tage wie bei den anderen Erholungsurlauben, sondern die vormals eingetragenen Überstunden
To Reproduce

Urlaubsantrag mit Überstundenabbau
Urlaubsantrag editieren und zu Erholungsurlaub ändern
Übersicht der Urlaube zeigt zuvor eingetragene Überstunden

Expected behavior
Übersicht der Urlaube zeigt Anzahl der Urlaubstage
Screenshots

Zusätzlich gibt es noch das Problem, dass es möglich ist bei einem Erholungsurlaub Überstunden mitzuschicken, sofern man vor dem Versenden des Formulars zwischenzeitlich die Art auf Überstundenabbau geändert hat.

repo: synyx/urlaubsverwaltung


text:
heroku adjustments

use heroku review-apps for fast feedback

use heroku review-apps for fast feedback
würde es für Sinnvoll halten, wenn die Commits gesquashed werden würden

i think we do not want to add the login parameters to the readme like the email :)

authors: christlang derTobsch grafjo


text:
Upgraded dear imgui and ImGuizmo to the latest version

Upgraded ImGui and ImGuizmo to their lattest version on GitHub.
Fixed the changes on the implementation on "btgui" (some function signature now permit you to pass a `void*  "user context" to functions)
The Gizmo feels a little bit nicer to use in the latest version
Signed-off-by: Arthur Brainville (Ybalrid) ybalrid@ybalrid.info

(I will send you theses changes for NanoSG too)

Mh, don't merge it just yet, there's a little bug I need to squash

This is good to go now

Cool! Merged!

authors: Ybalrid syoyo

repo: syoyo/tinygltf


text:
There are not year dates on the news items on the tellurium.analogmachine.org

The NEWS page doesn't have any year dates, only months and days. Add years to the dates. As it stands it is not possible to determine whether there was a 2017 release or not.

Added the years to all releases:
http://tellurium.analogmachine.org/news/

repo: sys-bio/tellurium


text:
Fix #266/Update NEWS.txt and NOTICE.txt

I see that readthedocs is targeting develop branch to generate Tellurium docs. Can we change it to master?

I also updated NEWS.txt and NOTICE.txt. Instead of deleting NEWS.txt, I simply added link to the NEWS section in tellurium website.


text:
Fix: remove delete intent doesnt need input for fromBaseChangeSet, look on the graph

Also, fire some WsEvents!

Failed the merge queue due to an issue pulling a docker image

repo: systeminit/si


text:
Improve MFM

MFM → htmlで改行が2重になる問題を修正
html → MFMでリンクが解除されてしまう問題を修正

問題があった

ハッシュタグが二重に描画される

あとハッシュタグがハッシュタグと認識されない(リンクとして認識される)

あー、確認不足でしたすみません…

authors: syuilo tamaina

repo: syuilo/misskey


text:
Fix URL after deleting remote file

Resolve #2991
リモートファイル期限切れ削除後は直リンクにするように修正

ありがとうございます！


text:
Sample code in readme.md mentions v1.1, but latest release is 1.0

Hi,
was a little bit confusing for me:
In the readme.md sample code for usage is uses: actions/set-timezone-action@v1.1, but the latest release is v1.0. So the sample code doesn't work.
Should be fixed.

In the case of linux, it could be better:
jobs:
  buid_iso:
    runs-on: ubuntu-20.04
    steps:
      - name: Display date with respect to timezone
        run: |
          ln -sf /usr/share/zoneinfo/Asia/Singapore /etc/localtime
          date

Reference: How to Set or Change the Time Zone in Linux

I've raised a PR to fix this in #3
cc @szenius

this is already done in #1 actually

Sorry everyone for the delay, #1 has been merged! And thanks a lot for pointing it out :-)

authors: Jameskmonger carlosal1015 mherbrich szenius

repo: szenius/set-timezone


text:
added docs for Postgresql setup in FreeBSD jails

Hello @c0psrul3,
first: thanks for the patch!
But i need some more information. Since i'm a strong PostgreSQL user i need to now, with which PostgreSQL version in mind you wrote the patch? At least 9.5 and 9.6 really work without this changes. For 9.4 i would double check this - but its very likely that your update is needed.

thank you for noting this, I hadn't even looked.  It is Pg 9.3
And, you're right, I should upgrade.  However, it was installed along with the FreeBSD Package "gitlab-8.10.12"
Would you mind telling me which versions of Pg (9.5 / 9.6) can be used with Gitlab 8.12 / 8.13?  I think it doesn't matter.
I'll upgrade and come back with any further doc patches.  Thanks!

Hello Mike,
yes, PG 9.3 is the current default PostgreSQL version. We are currently trying to set 9.4 as new default.
The documentation states that GitLab needs PG 9.1 or higher, so it doesn't really matter, if you use 9.5 or 9.6. I use both.

thanks for the info.
take the patch for what it's worth, then  :)

repo: t-zuehlsdorff/gitlabhq

license: mit


text:
Ability to load environment variables at runtime, not build time

I wonder if it is possible to load the environment variables at runtime, as with "https://github.com/expatfile/next-runtime-env". This is not possible in the example with nextjs.

Yeah, I assumed t3-env already did this! 🥲
It's important for self-hostable applications that are packaged into a docker image, end users should be able to edit the runtime env
Would you accept PR for this? I could try to implement a solution

I think we can make t3-env compatible with next-runtime-env at least! I'll take a look.

This works:
// src/env.mjs
import { createEnv } from "@t3-oss/env-nextjs";
import { z } from "zod";

const readVariable = (key) =>  {
  if(typeof window === 'undefined') return process.env['key']
  return window.__ENV['key']
}

export const env = createEnv({
  server: {
    DATABASE_URL: z.string().url(),
    OPEN_AI_API_KEY: z.string().min(1),
  },
  client: {
    NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY: z.string().min(1),
  },
  runtimeEnv: {
    DATABASE_URL: process.env.DATABASE_URL,
    OPEN_AI_API_KEY: process.env.OPEN_AI_API_KEY,
    NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY:
      readVariable(NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY),
  },
});

Would you accept PR for this? I could try to implement a solution

Yes

Would love to hear of any updates on this issue🤔

In the project I'm working we  adopted the t3-env lib with nextjs, to work around this validation problem in the build time I added an env var to skip the validation.
Package.json
{
  "scripts": {
      "dev": "yarn generate && NODE_OPTIONS='--inspect' next dev",
      "build": "yarn generate && SKIP_ENV_VALIDATIONS='true' next build",
      "start": "next start",
    }
 }

env.mjs:
export const env = createEnv({
  isServer: typeof window === 'undefined' || process.env.NODE_ENV === 'test',
  /*
   * Serverside Environment variables, not available on the client.
   * Will throw if you access these variables on the client.
   */
  server: {

  },
  /*
 * Environment variables available on the client and server.
 *
 */
  shared: {
    NODE_ENV: z.enum(['test', 'development', 'production'])
      .optional()
      .default('production'),
    STAGE: z.enum(['local', 'dev', 'stg', 'prod']).default('prod'),
    APP_LOG_LEVEL: z
      .string()
      .optional()
      .default('info')
      .transform((data) => data.toLowerCase()),
  },
  /*
   * Environment variables available on the client.
   *
   * 💡 You'll get type errors if these are not prefixed with NEXT_PUBLIC_.
   */
  client: {
  },

  /*
   * Due to how Next.js bundles environment variables on Edge and Client,
   * we need to manually destructure them to make sure all are included in bundle.
   *
   * 💡 You'll get type errors if not all variables from `server` & `client` are included here.
   * All the env vars must end with the suffix _ENV to avoid a cycle import on the env-nextjs that enters in loop
   */
  runtimeEnv: {
    NODE_ENV: process.env.NODE_ENV,
    STAGE: process.env.STAGE,
    APP_LOG_LEVEL: process.env.APP_LOG_LEVEL,

  },
  skipValidation: process.env.SKIP_ENV_VALIDATIONS === 'true'
});

In this way I'm able to skip the validation for the build

This works:
// src/env.mjs
import { createEnv } from "@t3-oss/env-nextjs";
import { z } from "zod";

const readVariable = (key) =>  {
  if(typeof window === 'undefined') return process.env[key]
  return window.__ENV[key]
}

export const env = createEnv({
  server: {
    DATABASE_URL: z.string().url(),
    OPEN_AI_API_KEY: z.string().min(1),
  },
  client: {
    NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY: z.string().min(1),
  },
  runtimeEnv: {
    DATABASE_URL: process.env.DATABASE_URL,
    OPEN_AI_API_KEY: process.env.OPEN_AI_API_KEY,
    NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY:
      readVariable(NEXT_PUBLIC_CLERK_PUBLISHABLE_KEY),
  },
});

I would like something like this, but for async functions. #146

The correct way is what @rochaalexandre posted but SKIP_ENV_VALIDATION instead of SKIP_ENV_VALIDATIONS

authors: SaadBazaz SunnyAureliusRichard baptisteArno juliusmarminge psycho-baller rochaalexandre timheerwagen


text:
LESS/SASS/Stylus etc preprocess support

Any change of getting some of those preprocessors supporter, or more specifically, the format they use to be supported?
So that running
stylestats path/to/stylesheet.less

would output similar results with additional mixin usage statistics.

+1, I used compass stat to generate stats, and send it to a graphite server to visualize it, but since libsass is faster, I've stopped using it.

It's related to #140
As mentioned above, we think that CSS statistics should be collected from CSS.
However, It might be possible to support SCSS files if we move to PostCSS.

authors: paazmaya t32k woutvdd


text:
Feature: Implement Gfx Navigation system communication

Decode messages going to the Gfx Nav system

Fixed in pull request #68


text:
Prevents undefined use of $::execs

Fixes #102

@jtreminio I have added you as a collaborator, so feel free to make any decisions concerning this repository. All accountability and responsibility is yours.
If you think that a transfer would make sense, then I could do that too. Although you'd have to remove your fork then.


text:
jsgrid returns blank line after update

After giving an update via ajax in jsgrid the line returns blank
In the return of the url update: /app/Play/change.php
I return the updated record data in a normal json, same as the initial json used in the load
controller: {
loadData: function(filter) {
var data = $.Deferred();
$.ajax({
type: "GET",
contentType: "application/json; charset=utf-8",
url: "/app/Play/",
dataType: "json"
}).done(function(response) {
data.resolve(response);
});
return data.promise();
},
updateItem: function(item) {
return $.ajax({
type: "POST",
url: "/app/Play/change.php",
data: item,
});
},
}

Resolved:
`
	controller: {
		loadData: function (filter) {
			var data = $.Deferred();
			$.ajax({
			   type: "GET",
			   contentType: "application/json; charset=utf-8",
			   url: "/app/Jogos/app/",
			   data: filter, //Insert line, this is a problem
			   dataType: "json"
			   }).done(function(response){
				 data.resolve(response);
			 });
			  return data.promise();
		},
			updateItem: function(item){
			   return $.ajax({
				type: "POST",
				url: "/app/Jogos/app/change.php",
				data: item
			   });
			},
	},

`

repo: tabalinas/jsgrid


text:
How can i define decimal type ?

I have used number type but it's would not allow me to save decimal values. for example i want to store 55.55 in some fields. how can i do this ?

You can define your own field like the following:

    function FloatNumberField(config) {
        jsGrid.NumberField.call(this, config);
    }

    FloatNumberField.prototype = new NumberField({

        filterValue: function() {
            return parseFloat(this.filterControl.val());
        },

        insertValue: function() {
            return parseFloat(this.insertControl.val());
        },

        editValue: function() {
            return parseFloat(this.editControl.val());
        }
    });

    jsGrid.fields.floatNumber = FloatNumberField;

Just a little notice - a prototype of a FloatNumberField shold be jsGrid.NumberField since NumberField is hidden behind IIFE. Also text should be formatted to use a proper delimiter.

Hi,
Can i know where to add the code for getting decimal places in the field

If I get you right, you need to redefine itemTemplate something like following:
    ...,
    itemTemplate: function(value) {
        return value.toFixed(2);
    }

Is it what are asking about?

When using the FloatNumberField code mentioned above, pressing enter while in a field still shows the message "Please enter a valid value. The two nearest valid ....." while saving (ie clicking the green V does work.. I've been looking at the code but couldn't find what I needed in jsgrid.field.number.js
Any help would be much appreciated!

authors: C4Grey lodb snagareddy tabalinas thoriyas


text:
[Feature]: Simple git actions in txAdmin web interface

Scope
Web
Feature Description
A way of executing simple git actions like pulling, branch switching, maybe also view commit history, if the server-data folder is a git repository.
Maybe adding a new page like this:

Use Case
This is more a quality of life thing. By having git actions integrated into txAdmin, you can fetch the newest changes from your remote repository without logging into your server either via a remote desktop client or a remote shell.
Proposed Solution
Well, I have already pitched one of my ideas in the Feature description, but an easier way would be to just add a small button to the dashboard page.
Additional Info
No response

If the kids dont understand Git then Git gud

Although this predates #933 by a lot, since that one is more updated I'm closing this one as duplicated.
Continued conversations regarding this feature should be done in #933.

authors: AleksanderEvensen tabarra

repo: tabarra/txAdmin


text:
Error in reading pdf file

Java version is 9, and I have problem reading pdf file. Why is that?
code:
java -jar C:\Users\legend\Anaconda2\lib\site-packages\tabula\tabula-1.0.1-jar-with-dependencies.jar --pages 1 --guess D:\XXXX.pdf
error message below:
Error: Need exactly one filename

Have you made sure the path to the pdf does not contain spaces or is correctly quoted? (i guess the XXXX.pdf is not the real filename.)

Yes, I am certain that the path is correct.

After I changed the path to be a short one, there returns no error.

Also I did not doubt you had a wrong path, I rather was concerned about spaces.
If your path contained spaces, you need to enquote it (put it between "", e.g. "C:\my stuff\important document.pdf").
Anyway, sounds like your issue is resolved? If it is you can and should close the issue :)

repo: tabulapdf/tabula-java


text:
'Clear all selections' button does not work after coming back from Preview

n/t

This, I can't replicate. It works fine for me. There must be something else going?

@jazzido: any help on replication?

Closed for inactivity :D
Obvi reopen if you're able to replicate.

authors: jazzido jeremybmerrill

repo: tabulapdf/tabula


text:
br3 → br4 in docs

The border radius 4 element currently appears as ”br3” instead of “br4” in the “Border Radii” docs.

This only updates the generated file so this change would be deleted the next time site generation happens. For future reference all files are in src/. I'll add this patch in manually.

authors: mrmrs nickcernis


text:
いいところ診断の結果に優しさを追加したい。

以下の結果を追加したい
'{userName}のいいところは優しさです。あなたの優しい雰囲気や立ち振る舞いに多くの人が癒やされています。'

これから対応します

9c8040eba35cdbc37ff770145a911d16dd729bd0 にて対応完了しましたら

repo: taharjun/assessment


text:
Add tests

Just for increase coverage.

Coverage increased (+1.2%) to 99.686% when pulling 1e28777e6e1b21a86e9bf18e1e733572239acbe6 on add-tests into 6d53978936a6fa80a75655d924595e0b9995c4e9 on master.

Coverage increased (+1.3%) to 99.765% when pulling 8541424664aea99c518441d87d9a1ad825ea2399 on add-tests into 6d53978936a6fa80a75655d924595e0b9995c4e9 on master.

Coverage increased (+1.3%) to 99.848% when pulling 472e0f52e8bb16967c75af37b4d395a7a0922dd2 on add-tests into 6d53978936a6fa80a75655d924595e0b9995c4e9 on master.


text:
Change client code accordingly after protocol tokenomics v1 PR merged

https://github.com/taikoxyz/taiko-mono/pull/243/files

https://github.com/taikoxyz/taiko-client/pull/104

repo: taikoxyz/taiko-client


text:
chore(repo): add lefthook rule for guardian ui

enforce linting and formatting on commit

@KorbinianK i renamed the title. it doesnt really matter though in retrospect, because chore commits wont get added to the changelog

repo: taikoxyz/taiko-mono


text:
Better validate Alpine release

If I use !Alpine 3.2 I get the following error:
(1/1) Installing alpine-keys (1.1-r0)
OK: 0 MiB in 1 packages
fetch http://mirror.yandex.ru/mirrors/alpine/3.2/main/x86_64/APKINDEX.tar.gz
ERROR: http://mirror.yandex.ru/mirrors/alpine/3.2/main: No such file or directory
WARNING: Ignoring APKINDEX.36465e51.tar.gz: No such file or directory
ERROR: unsatisfiable constraints:
  alpine-base (missing):
    required by: world[alpine-base]

I should use v3.2 instead, but the error is misleading

Same problem different url:
fetch http://dl-7.alpinelinux.org/alpine/v3.2/main/x86_64/APKINDEX.tar.gz
ERROR: http://dl-7.alpinelinux.org/alpine/v3.2/main: No such file or directory
WARNING: Ignoring APKINDEX.0f345221.tar.gz: No such file or directory
ERROR: unsatisfiable constraints:
  ca-certificates (missing):
    required by: world[ca-certificates]
  wget (missing):
    required by: world[wget]

any news on that?

Hm, the original issue is actually fixed.
The issue that @popravich reported is a different one. Many current mirrors don't mirror v3.2 of alpine any more. But not all of them. So you must either set a preferred mirror in settings:
alpine-mirror: http://dl-cdn.alpinelinux.org/alpine/

Or just upgrade alpine version to v3.3 or v3.4

authors: mediatainment popravich tailhook

repo: tailhook/vagga


text:
No Tailwind CSS output when using postcss-easy-import

What version of @tailwindcss/jit are you using?
v.0.1.3
What version of Node.js are you using?
v15.12.0
What browser are you using?
Chrome
What operating system are you using?
macOS
Reproduction repository
https://github.com/sboerrigter/tailwind-jit-easy-import
👋
When using postcss-easy-import (https://www.npmjs.com/package/postcss-easy-import) the compiler doesn't output any tailwind CSS and doesn't give any errors. You can reproduce this but by running npm run dev in the reproduction repository: https://github.com/sboerrigter/tailwind-jit-easy-import.
If you replace @tailwindcss/jit with tailwindcss in postcss.config.js the CSS output does get compiled correctly and the background of the page should turn green because of bg-green-500.

Hey! In your case just need to move your plugins around in postcss.config.js:
  module.exports = {
    plugins: {
+     'postcss-easy-import': {},
      '@tailwindcss/jit': {},
-     'postcss-easy-import': {},
      autoprefixer: {},
    },
  }

The regular tailwindcss library has some special logic for treating @import "tailwindcss/base" as @tailwind base, but haven't added that to the JIT which is why you see the difference. Going to add that now I think anyways 👍

Thanks. Changing plugins the order fixes it indeed!

authors: adamwathan sboerrigter

repo: tailwindlabs/tailwindcss-jit


text:
Typescript Syntax

What version of @tailwindcss/jit are you using?
v0.1.12
What version of Node.js are you using?
v15.11.0
What browser are you using?
Chrome
What operating system are you using?
Ubuntu
Reproduction repository
https://github.com/Dewbud/tailwind-jit-repro
using any of the following
@tailwind base;
@tailwind components;
@tailwind utilities;

either by importing a .css file
or including in the style tags of a Vue SFC
results in
[vite] Internal server error: Maximum call stack size exceeded
  Plugin: vite:css
  File: /home/anonymous/tailwind-jit-repro/src/styles.css
      at candidatePermutations (/home/anonymous/tailwind-jit-repro/node_modules/@tailwindcss/jit/src/lib/generateRules.js:19:32)
      at candidatePermutations.next (<anonymous>)
      at candidatePermutations (/home/anonymous/tailwind-jit-repro/node_modules/@tailwindcss/jit/src/lib/generateRules.js:41:10)
      at candidatePermutations.next (<anonymous>)
      at candidatePermutations (/home/anonymous/tailwind-jit-repro/node_modules/@tailwindcss/jit/src/lib/generateRules.js:41:10)
      at candidatePermutations.next (<anonymous>)
      at candidatePermutations (/home/anonymous/tailwind-jit-repro/node_modules/@tailwindcss/jit/src/lib/generateRules.js:41:10)
      at candidatePermutations.next (<anonymous>)
      at candidatePermutations (/home/anonymous/tailwind-jit-repro/node_modules/@tailwindcss/jit/src/lib/generateRules.js:41:10)
      at candidatePermutations.next (<anonymous>) (x2)

Example:
defineComponent({
  name: 'HelloWorld',
  props: {
    msg: {
      type: String,
      required: true
    },
    things: Array as PropType<string[]>,
  },
  setup: () => {
    const count = ref(0)
    return {
      count,
      stuff: [] as string[] | undefined,
    }
  }
})

candidatePermutations just yields forever when it hits the line containing:
things: Array as PropType<string[]>
// console.log(candidate, lastIndex)
defineComponent, Infinity
PropType Infinity
Component Infinity
props Infinity
things Infinity
Array Infinity
as Infinity
string[] Infinity
string[] 4
string[] 4
string[] 4

Some other Typescript syntaxes that have caused this
const cats: Cat[] = [
    {}
];

let items = [] as string[] | undefined;

type ExampleContext = {
  testMethod<K extends keyof Example>(
    key: K,
    payload?: Parameters<Example[K]>[1]
  ): ReturnType<Example[K]>;
};

Not sure why this only happen when using @tailwind rules but I'm sure there's regex wizardry involved.

Fixed in v0.1.13, sorry about that!

authors: Dewbud adamwathan


text:
Meal Plans - Completing a meal plan

Clicking Confirm doesn't do anything across any of the options. Like the button isnt attached to anything?

now we allow multiple stores I'll need to update this one to choose the store first

repo: taitdigital/aisle-nine-beta

license: Unlicense


text:
Features & bug fixes

I made a few more changes to the script:

bug fixes
distance text
changed German to English text
made fake profiles invisible

Looks great. Thank you!
I have a nitpicky question though. Why do you insist on using var for variable declaration?

Uhm I used const most of the time, maybe I forgot to use it sometimes 😅

looks good to me

LGTM, life's short, let's merge this 😎

authors: JustixDevelopment tajnymag

repo: tajnymag/tinder-deblur


text:
Wrong path in report used for images on Windows

..\C:\Users\Path-To-My-Project\Project-abc\snapshots\build\outputs\roborazzi\Example-exampleName_compare.png
^^ This path is used in the HTML report
It should be without the ..\ at the start or a relative path

Thank you for reporting this bug. I'll take a look.

authors: ben-gooding-hub takahirom

repo: takahirom/roborazzi


text:
doc change: build.xml -> release/build.xml

The doc references a top level build.xml, which doesn't exist any more.

This is a mess, please ignore...

repo: takezoe/gitbucket


text:
py.test -> pytest

py.test is not recommended since pytest v3.0 (see https://github.com/pytest-dev/pytest/issues/1629)

Thanks!

authors: takluyver wimglenn

repo: takluyver/flit


text:
[question] Execute code after Onboarding.of(context)!.show();

Today if i try to execute something on the next line after Onboarding.of(context)!.show() it will run before the end of the onboarding steps.
Is there any way of "waiting" the end of onboarding steps to execute the code (something like this function becoming async) or the only way is adding some function to onTapCallback of the last OnboardingStep widget?

the Onboarding widget has a callback onEnd, which is called after the last step

Can you give any sample on how to use it?

Onboarding(
  key: onboardingKey,
  steps: getSteps(al, theme, focusNodes),
  autoSizeTexts: true,
  onEnd: (int index) {
    onboardingStore.updateOnboardingCompleted(true); // MobX, Store, ChangeNotifier, Cubit w/e
  },
  globalOnboarding: true,
  child: Scaffold(...)
)


text:
[Marketplace 10x / API] Incorrect returned orders

Some collection returned incorrect orders like following:
https://marketplace-10x.tamago.finance/orders/collection/0x2953399124f0cbb46d2cbacd8a89cf0599974963

This is caused due to the target NFT collection address could be Opensea's Shared Store front address. This means that all nfts minted from this contract address (which also exist in our collections database) will be returned.
This issue could be complicated and will be considered in a future opportunity.

Manually editted the collection's name and description to that of Opensea Storefront.

authors: Moopz-Dev pisuthd


text:
Change 16-21 in story.txt

Changed line 16-21 in story.txt and becomes Harsh's version.

@HarshPatel187
Maybe you can review this and merge this for me please?

I have resolved all the conflicts

I have resolved all the conflicts

Nice one!


text:
Says 'Network Unavailable' for Interactive Comics

Should say 'Open in Browser' and clicking it shall do just that. Sharing it results in an incorrect URL as well.
Example: http://xkcd.com/1608/
Does the API give the required information for this?

The api doesnt provide enough info for this, plus interaction requires special keyboard keys which are unavailable on mobile, so even opening the link in chrome wont help, its just something that cannot be fixed by me.

This works with the accelerometer and touch on my phone in Chrome. Too bad that the API doesn't have enough data for this.

authors: danielhickman tanallnight

repo: tanallnight/Blip


text:
latest version suggestion

anton as you have done latest version might be other users could be problem
case you are inserting 0 value during heartbit ? or sleeping mode ? as latest version ?
so all time but problem is solved from xml  as i did ignore 0 and seems working fine and great
so may be this sample config files users can use it and problem solved ? as 0 filtering so will have great working this last version for all people really
case they people can get consider as status of device
i  have done other class customized and working fine according my need  :)

I don't quite understand what you are talking about. Heartbeat messages are not saved in the database and they won't be.

heartbeat mean device sleeping then just send details without gps lat long got it ?

The idea is that on heartbeat message server would just update device status and timestamp.

yes true and it's working fine great work it could be stable version i have tested more time

authors: perfectwebtech tananaev

repo: tananaev/traccar


text:
ShEnableHooks and ShDisableHooks in shadow_hook.cpp

why
HypercallNumber::kShEnablePageShadowing and
HypercallNumber::kShDisablePageShadowing
are missing?

Hey, did you pull down files recursively?
$git clone --recursive https://github.com/tandasat/DdiMon.git

Yes.
And HypercallNumber in HyperPlatform/util.h there is no kShEnablePageShadowing and kShDisablePageShadowing

That is weird. Can you manually checkout 374db11ecefd1f00acccbc34fec6c1013e6f080c in DdiMon/HyperPlatform? That commit is what the DdiMon expects.

repo: tandasat/DdiMon


text:
[NEW ICON] Webpack

Skill Name
Webpack
Why?
Nearly most used bundle tool for Front-End, and I think it is a skill definitely
Reference Image

Just find a png file not svg😢

Added in 9904dfdfef9bf5d8ae649a85fcb531b5892b0796!

authors: justforuse tandpfun

repo: tandpfun/skill-icons


text:
IOS-2643 Waring row expand on view width

alignment не нужен ведь?

как будто нужен, иначе текст по центру получается )

repo: tangem/tangem-app-ios


text:
здравствуйте как вам идея м

здравствуйте как вам идея м
здравствуйте как вам идея м

გამარჯობა
поддерживаю

хорошая идея, спасибо!

самый адекватный политик

пожалуй, соглашусь. лучшая идея за последние 1000 лет

authors: bit0r1n evtn nitreojs tailsjs tangenx

repo: tangenx/lfdi


text:
missing GO terms in GAF file

Hi,
@dvklopfenstein and I had a conversation in issue #79 about discrepancies in GAF files between goa_human.gaf.gz found on the Gene Ontology site, and the one used for Enrichr.
I looked into using GOrilla instead, and they seem to have a much more complete set of GO enrichments than the one taken from Gene Ontology site. For instance, none of my top 3 GO terms found on GOrilla exist: "GO:0010604", "GO:0048522", "GO:0009893".
I'm using your suggested code to load the GAF file:
from goatools.associations import read_gaf

# Import GO annotations from http://geneontology.org/gene-associations/goa_human.gaf.gz
goatools_annotations = read_gaf("../data/goa_human.gaf")

Any ideas where I can find the most complete annotation file that is human specific?
Thanks,
Johnny

The GO terms you list have hundreds or thousands of GO terms child terms underneath them:

GO:0048522 positive regulation of cellular process
GO:0009893 positive regulation of metabolic process
GO:0010604 positive regulation of macromolecule metabolic process

There are over 2,400 child GO terms for the 3 GO terms that you list.
Of those 2,400+ child GO terms, ~1,200 GO terms are in the human association in goa_human.gaf.
Some examples of more specific GO terms that are both found in the human annotation and are children to  "GO:0010604 positive regulation of macromolecule metabolic process" in your list include:

GO:0046016 positive regulation of transcription by glucose
GO:1903518 positive regulation of single strand break repair
GO:0048295 positive regulation of isotype switching to IgE isotypes

Often, the more specific child terms like the ones found in the second bullet list are preferable results from a GO enrichment analyses than the GO terms found in the first bullet list. So you may prefer using the associations in goa_human.gaf. I would hesitate to say the lack of these extremely broad terms that you list means that the goa_human.gaf is less complete.
Regardless, GOATOOLS is not the source for gene-GO annotations, so the better people to talk to regarding the content of associations is the providers of the associations. The gene ontology consortium help may be able to point you where to go for questions on the human annotation. Their link is https://github.com/geneontology/helpdesk/issues
Thank you for your interest in GOATOOLS and your interesting questions. It was interesting to look into the details of the associations for those GO IDs.
I am going to close this issue. If you feel that the issue is not answered, please feel free to open a new issue.

authors: dvklopfenstein iamjli

repo: tanghaibao/goatools


text:
Non-Mapzen hosted documentation

With the shutdown of Mapzen, are there plans to switch where the documentation is hosted?

https://readthedocs.org? Looks like there is already a project started for it: https://readthedocs.org/projects/tangram-js/

cc @meetar who's started the project.
There's three different items, all related, all which need documentation

Tangram JS
Tangram ES
The Tangram Scene File language

As a cartographer, it's the third I'm interested in. readthedocs is typically used with projects like the first two where there's an API, but I don't know of any reason why it wouldn't work for the third.

https://tangrams.readthedocs.io/en/latest/
✅ ✅ ✅

authors: matteblair meetar pnorman

repo: tangrams/tangram-docs


text:
Segmentation fault while displaying a map

The app was in the foreground but I was not actively using it. Next time I looked over at the device it had crashed with the following error in the logcat.
07-07 17:54:28.306    1469-3069/? A/libc﹕ Fatal signal 11 (SIGSEGV), code 1, fault addr 0x8447ab5 in tid 3069 (GLThread 1215)
07-07 17:54:28.414      193-193/? A/DEBUG﹕ *** *** *** *** *** *** *** *** *** *** *** *** *** *** *** ***
07-07 17:54:28.414      193-193/? A/DEBUG﹕ Build fingerprint: 'google/hammerhead/hammerhead:M/MPZ44Q/1960707:user/release-keys'
07-07 17:54:28.414      193-193/? A/DEBUG﹕ Revision: '0'
07-07 17:54:28.414      193-193/? A/DEBUG﹕ ABI: 'arm'
07-07 17:54:28.414      193-193/? A/DEBUG﹕ pid: 1469, tid: 3069, name: GLThread 1215  >>> com.mapzen.erasermap <<<
07-07 17:54:28.414      193-193/? A/DEBUG﹕ signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x8447ab5
07-07 17:54:28.464      193-193/? A/DEBUG﹕ r0 00000000  r1 00000001  r2 b5d4cd51  r3 08447ab5
07-07 17:54:28.464      193-193/? A/DEBUG﹕ r4 99f19f58  r5 99e969d8  r6 12da3150  r7 6f341804
07-07 17:54:28.464      193-193/? A/DEBUG﹕ r8 12da3150  r9 97ae6d00  sl 12d44700  fp 9ac7caa4
07-07 17:54:28.464      193-193/? A/DEBUG﹕ ip af01ad40  sp 9ac7c9f8  lr aef4c958  pc aef4c9ac  cpsr 200f0010
07-07 17:54:28.517      193-193/? A/DEBUG﹕ backtrace:
07-07 17:54:28.517      193-193/? A/DEBUG﹕ #00 pc 001049ac  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__hash_iterator<std::__1::__hash_node<std::__1::__hash_value_type<unsigned int, GLFONSbuffer*>, void*>*> std::__1::__hash_table<std::__1::__hash_value_type<unsigned int, GLFONSbuffer*>, std::__1::__unordered_map_hasher<unsigned int, std::__1::__hash_value_type<unsigned int, GLFONSbuffer*>, std::__1::hash<unsigned int>, true>, std::__1::__unordered_map_equal<unsigned int, std::__1::__hash_value_type<unsigned int, GLFONSbu
07-07 17:54:28.517      193-193/? A/DEBUG﹕ #01 pc 000fdedc  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (glfonsBufferDelete(FONScontext*, unsigned int)+136)
07-07 17:54:28.517      193-193/? A/DEBUG﹕ #02 pc 000ed1b4  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (TextBuffer::~TextBuffer()+40)
07-07 17:54:28.517      193-193/? A/DEBUG﹕ #03 pc 00109ebc  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__shared_ptr_pointer<TextBuffer*, std::__1::default_delete<TextBuffer>, std::__1::allocator<TextBuffer> >::__on_zero_shared()+124)
07-07 17:54:28.517      193-193/? A/DEBUG﹕ #04 pc 000805f0  /data/app/com.mapzen.erasermap-1/lib/arm/libc++_shared.so (std::__1::__shared_weak_count::__release_shared()+68)
07-07 17:54:28.518      193-193/? A/DEBUG﹕ #05 pc 000d8794  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::shared_ptr<TextBuffer>::~shared_ptr()+44)
07-07 17:54:28.518      193-193/? A/DEBUG﹕ #06 pc 000e10a8  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::pair<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const, std::__1::shared_ptr<TextBuffer> >::~pair()+28)
07-07 17:54:28.518      193-193/? A/DEBUG﹕ #07 pc 000d91c8  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__tree<std::__1::__value_type<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::shared_ptr<TextBuffer> >, std::__1::__map_value_compare<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::__value_type<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::shared_ptr<TextBuffer> >, std::__1
07-07 17:54:28.518      193-193/? A/DEBUG﹕ #08 pc 000d7d10  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__tree<std::__1::__value_type<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::shared_ptr<TextBuffer> >, std::__1::__map_value_compare<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::__value_type<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::shared_ptr<TextBuffer> >, std::__1
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #09 pc 000d5fb8  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::map<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> >, std::__1::shared_ptr<TextBuffer>, std::__1::less<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > >, std::__1::allocator<std::__1::pair<std::__1::basic_string<char, std::__1::char_traits<char>, std::__1::allocator<char> > const, std::__1::shared_ptr<TextBuffer> > > >::~map()+24)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #10 pc 000d68b0  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (MapTile::~MapTile()+56)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #11 pc 000d6928  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (MapTile::~MapTile()+20)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #12 pc 000ce044  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__shared_ptr_pointer<MapTile*, std::__1::default_delete<MapTile>, std::__1::allocator<MapTile> >::__on_zero_shared()+140)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #13 pc 000805f0  /data/app/com.mapzen.erasermap-1/lib/arm/libc++_shared.so (std::__1::__shared_weak_count::__release_shared()+68)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #14 pc 000cc42c  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::shared_ptr<MapTile>::~shared_ptr()+44)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #15 pc 000d5b58  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::pair<TileID const, std::__1::shared_ptr<MapTile> >::~pair()+28)
07-07 17:54:28.519      193-193/? A/DEBUG﹕ #16 pc 000d4628  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (std::__1::__tree<std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> >, std::__1::__map_value_compare<TileID, std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> >, std::__1::less<TileID>, true>, std::__1::allocator<std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> > > >::erase(std::__1::__tree_const_iterator<std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> >, std::__1::__tree_node<std::__1
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #17 pc 000d1394  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (TileManager::removeTile(std::__1::__map_iterator<std::__1::__tree_iterator<std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> >, std::__1::__tree_node<std::__1::__value_type<TileID, std::__1::shared_ptr<MapTile> >, void*>*, int> >&)+1276)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #18 pc 000d08cc  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (TileManager::updateTileSet()+3040)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #19 pc 0010b8c0  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (Tangram::update(float)+152)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #20 pc 0017e8f8  /data/app/com.mapzen.erasermap-1/lib/arm/libtangram.so (Java_com_mapzen_tangram_MapController_update+28)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #21 pc 014e2c05  /data/app/com.mapzen.erasermap-1/oat/arm/base.odex (void com.mapzen.tangram.MapController.update(float)+80)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #22 pc 014e32e5  /data/app/com.mapzen.erasermap-1/oat/arm/base.odex (void com.mapzen.tangram.MapController.onDrawFrame(javax.microedition.khronos.opengles.GL10)+256)
07-07 17:54:28.520      193-193/? A/DEBUG﹕ #23 pc 726e35fd  /data/dalvik-cache/arm/system@framework@boot.oat
07-07 17:54:29.128      193-193/? A/DEBUG﹕ Tombstone written to: /data/tombstones/tombstone_02
07-07 17:54:29.128      193-193/? E/DEBUG﹕ AM write failed: Broken pipe
07-07 17:54:29.208      751-837/? E/InputDispatcher﹕ channel 'f7beba3 com.mapzen.erasermap/com.mapzen.erasermap.view.MainActivity (server)' ~ Channel is unrecoverably broken and will be disposed!

Thanks @karimnaaji !

repo: tangrams/tangram-es


text:
port options is not working in watch command

$ ./easel watch sample.md -p 3001
listening on localhost:3000

Option must be specified first argument.
$ ./easel watch -p 3001 sample.md
listening on localhost:3001

This is the flag package behavior.

repo: tanksuzuki/easel


text:
Clarify setting cacheTime for persistWithLocalStorage to work properly

Clarify the docs around persistWithLocalStorage as overriding the default cacheTime appears to be necessary when creating the QueryClient instance.
Closes #1600

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

authors: melanieseltzer tannerlinsley

repo: tannerlinsley/react-query


text:
call-summits on specified regions?

Hi,
I'm using peaks called by MACS2 from within an existing pipeline that ultimately uses IDR to filter and produce a top-set of peaks. I'd like to use the --call-summits command to identify summits within those post-IDR top-set peaks to use for motif analysis. I can't figure out whether I can use the call-summits command along with prescribed regions (e.g. input bam file & bed file to get summits only on the bed file)?
Thank you for your help!

@erin-a-clark Even if you don't specify --call-summits (or use the subcommand refinepeak), MACS will still report summits based on the pileup values in the output. The only difference is that --call-summits will invoke SPP-like way to 'refine' the summits and sometimes it doesn't provide better results than the default behavior. If you are interested in summits in IDR peaks, you can take the IDR peaks in BED file and overlap them with MACS2 summit output (e.g. NAME_summits.bed) using bedtools.

authors: erin-a-clark taoliu

repo: taoliu/MACS


text:
Error installing MACS2 with Cython installed

Dear all,
I am having a problem with installing MACS2 (v 2.1.0, downloaded on 25-Feb-2015).
When I run
python setup.py install --prefix=/my/fav/folder

I get the following error. Cython, v 0.22 is installed. I know a similar issue was reposrted in issue #36, but no solution to the problem is provided there. Can anybody help? Thanks!
running install
/my/fav/folder/setuptools/lib/python2.7/site-packages/setuptools-12.2-py2.7.egg/pkg_resources/__init__.py:2512: PEP440Warning: 'distribute (0.6.24dev-r0)' is being parsed as a legacy, non PEP 440, version. You may find odd behavior and sort order. In particular it will be sorted as less than 0.0. It is recommend to migrate to PEP 440 compatible versions.
running bdist_egg
running egg_info
writing requirements to MACS2.egg-info/requires.txt
writing MACS2.egg-info/PKG-INFO
writing top-level names to MACS2.egg-info/top_level.txt
writing dependency_links to MACS2.egg-info/dependency_links.txt
writing requirements to MACS2.egg-info/requires.txt
writing MACS2.egg-info/PKG-INFO
writing top-level names to MACS2.egg-info/top_level.txt
writing dependency_links to MACS2.egg-info/dependency_links.txt
reading manifest file 'MACS2.egg-info/SOURCES.txt'
reading manifest template 'MANIFEST.in'
warning: no files found matching 'MANIFEST'
writing manifest file 'MACS2.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
running build_ext
building 'MACS2.cProb' extension
gcc -pthread -fno-strict-aliasing -DNDEBUG -g -fwrapv -O2 -Wall -Wstrict-prototypes -fPIC -I/usr/lib/pymodules/python2.7/numpy/core/include -I/usr/include/python2.7 -c MACS2/cProb.c -o build/temp.linux-x86_64-2.7/MACS2/cProb.o -w -Ofast
gcc: error: MACS2/cProb.c: No such file or directory
gcc: fatal error: no input files
compilation terminated.
error: command 'gcc' failed with exit status 4

Response from Tao that helped me on CentOS:

A simple workaround although I don’t have CentOS to test: open MACS2 ‘setup.py’ file and replace all ‘-Ofast’ with ‘-O3’ or ‘-O2’. If it still doesn’t work, remove ‘-Ofast’ entirely.

Thanks for the tip, but it didn’t work, with either –O3, -O2 or nothing. I don’t have a problem installing MACS (not version 2).
Nicola
From: mdozmorov [mailto:notifications@github.com]
Sent: 26 February 2015 15:20
To: taoliu/MACS
Cc: nicolazilio
Subject: Re: [MACS] Error installing MACS2 with Cython installed (#69)
Response from Tao that helped me on CentOS:
A simple workaround although I don’t have CentOS to test: open MACS2 ‘setup.py’ file and replace all ‘-Ofast’ with ‘-O3’ or ‘-O2’. If it still doesn’t work, remove ‘-Ofast’ entirely.
—
Reply to this email directly or view it on GitHub https://github.com/taoliu/MACS/issues/69#issuecomment-76185357 .  https://github.com/notifications/beacon/ACq66AGmpwKBkSoIYnrhyNLoklIWK0Y_ks5nvyMkgaJpZM4DmMgx.gif

Hi Nicola,
Perhaps you downloaded incomplete source codes. Could you remove your current MACS2 source codes and old installation from your computer then download and try again?
-T

Hey Tao,
I actually solved the issue in a much simpler way, I installed Anaconda to ensure all Python dependencies are satisfied and then did pip install. Worked without an issue.
Nicola
From: Tao Liu (τν) [mailto:notifications@github.com]
Sent: 20 April 2015 20:43
To: taoliu/MACS
Cc: nicolazilio
Subject: Re: [MACS] Error installing MACS2 with Cython installed (#69)
Hi Nicola,
Perhaps you downloaded incomplete source codes. Could you remove your current MACS2 source codes and old installation from your computer then download and try again?
-T
—
Reply to this email directly or view it on GitHub https://github.com/taoliu/MACS/issues/69#issuecomment-94535696 .  https://github.com/notifications/beacon/ACq66FGrssikMuNQbt7BHC8yNQojVpsvks5oBUBOgaJpZM4DmMgx.gif

@nicolazilio Good to know. Now I just closed this ticket.

authors: mdozmorov nicolazilio taoliu


text:
Examine party members?

It'd be nice to examine your own party members, and the main character.
Admittedly most of the info is available on the character sheet, but an important bit of missing info is the spell effects durations ("How long until that PFMW wears off?") which Radar, on the other hand, provides.

It should be working if without SCS, gotta investigate how SCS brakes it

Should be fixed in the latest release

authors: gatperdut tapahob


text:
chore(deps): bump docusaurus-plugin/theme-openapi-docs from 1.2.2 to 1.3.0

bump docusaurus-plugin-openapi-docs from 1.2.2 to 1.3.0
bump docusaurus-theme-openapi-docs from 1.2.2 to 1.3.0

Search is broken :O

Search is still broken :(

Search is still broken...

Search works again!

repo: tapioone/docs


text:
TypesScript support on windows not working

Hi!
Testing TS code is not working at the moment. Current implementation uses ts-node bin field that is not cross-platform.
The right way to use ts-node cross-platform is to pass -r ts-node/register option to node, instead of spawning ts-node child process.
I'll make a PR to fix this specific issue.
This is the full error dump on Windows 10:
 TAP version 13
not ok 1 - ./index.ts # time=20.695ms
  ---
  timeout: 30000
  file: ./index.ts
  childId: 0
  command: >-
    C:\temp\node_modules\ts-node\dist\bin.js
  args:
    - '-r'
    - >-
      C:\temp\node_modules\esm\esm.js
    - ./index.ts
  stdio:
    - 0
    - pipe
    - 2
  cwd: 'C:\temp'
  ...
{

    not ok 1 - spawn UNKNOWN
      ---
      timeout: 30000
      file: ./index.ts
      childId: 0
      command: >-
        C:\temp\node_modules\ts-node\dist\bin.js
      args:
        - '-r'
        - >-
          C:\temp\node_modules\esm\esm.js
        - ./index.ts
      stack: |
        loop (node_modules/function-loop/index.js:35:15)
      stdio:
        - 0
        - pipe
        - 2
      cwd: 'C:\temp'
      at:
        line: 366
        column: 11
        file: internal/child_process.js
        function: ChildProcess.spawn
      errno: UNKNOWN
      code: UNKNOWN
      syscall: spawn
      test: ./index.ts
      ...
    1..1
    # failed 1 test
}

1..1
# failed 1 test
# time=44.501ms
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! temp@1.0.0 test: `tap -J ./index.ts`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the temp@1.0.0 test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     C:\Users\Baz\AppData\Roaming\npm-cache\_logs\2019-03-22T11_30_27_215Z-debug.log

I opened 523 PR.

repo: tapjs/node-tap

license: isc


text:
Add doc alias to recip for inv

The F32 struct has an inv implementation that calculates the reciprocal. In the standard library, this function is referred to as recip, so I added a #[doc(alias = "recip")] to it in order to improve visibility.

I also added a note that this function performs an inverse based on an IEEE 754 float layout assumption.

Looks like this isn't allowed under the current MSRV.
We can potentially bump MSRV. Not sure when it was introduced.

@tarcieri Just checked, it was added in Rust 1.48.0 from Novemver 2020 (see here). recip itself is from 1.10.0, ... not that that matters. :)
Current MSRV was 1.47.0, so I bumped it by one minor. Seems excessive just for one alias, so I'm not sure if that's what you want just for this one. It probably makes more sense in light of other PRs.

(Removed the MSRV bump since #108 went higher anyway.)

authors: sunsided tarcieri

repo: tarcieri/micromath


text:
Enabled HEAD requests + refactoring

enabled head requests
refactored servehttp method from composition handler
go format

Coverage increased (+0.3%) to 92.762% when pulling 53de6ccec4e1a17443b9372936d2ad99ced8f593 on bugfix/1463-enabled-head-requests into d4e78f0255ef78a287680c02e3cdb1c8577cab38 on master.

Coverage increased (+0.3%) to 92.812% when pulling 13f46bbc1d11016913258071426ca493d1eef57d on bugfix/1463-enabled-head-requests into d4e78f0255ef78a287680c02e3cdb1c8577cab38 on master.

Cool, looks very clean and understandable, now!
So, from my side fine to merge it.

authors: coveralls deberth smancke

repo: tarent/lib-compose


text:
New tag

Could you please add a new tag regarding PR #381 ?

Hi, I'm sorry to insist on this, but would this be possible?

Sorry for the delay, will see If I can fix this for you today.

Your change is merged into version 7.1.4.
Greetings Sander

authors: leoditommaso sbotman


text:
Fund ropsten accounts/contracts

We should have 100 assets for sale (parcels + estates)
We should allow 200 ephemeral uses (each one with 0.05 ethers and 10 Mana tokens)
Said that:
Fund Gnosis Safe wallet with at least 10 ethers and 2000 Mana tokens.
The Gnosis safe wallet owner should have enough ether to pay for all fund transactions gas. 2 ethers (200 * 0.01) should be more than enough.

Maybe each asset should cost a lot more than 10 MANA tokens to match real prices on mainnet. Maybe more like 10,000 MANA each?

I've funded Gnosis Safe wallet with a billion mana tokens (tx)
I can create prices using this rule: [a rand number between 10 to 100] * 1000 MANA.
In the worst case, it can allow 10k purchases. What do you think?

I've funded Gnosis Safe wallet with a billion mana tokens (tx)
I can create prices using this rule: [a rand number between 10 to 100] * 1000 MANA.
In the worst case, it can allow 10k purchases. What do you think?

Sounds good!

Gnosis safe wallet is now with 11.8 ethers.
Gnosis safe wallet owner/signer is now with 2 ethers.

Extracted the remaining task to https://github.com/tasitlabs/TasitSDK/issues/285 since that will change a script of the TasitSDK repo.
I think that could close this one.

Extracted the remaining task to https://github.com/tasitlabs/TasitSDK/issues/285 since that will change a script of the TasitSDK repo.
I think that could close this one.

Yep. Great work!

authors: marcelomorgado pcowgill

repo: tasitlabs/tasit


text:
Show difficulty level next to star

PR includes

[x] Refactor

Related issues
Resolve #1064

repo: taskany-inc/hire


text:
Add crontab for tasks

This is currently a draft API. But I think it would be nice to have something like this:
@broker.task(
    schedule={
        "* * * * *": {
            "args": [1],
            "kwargs": {"b": "2"},
        },
    },
)
async def my_async_task(a: int, b: str) -> str:
    """My lovely task."""
    await asyncio.sleep(1)
    return f"{a} {b}"

And we have to start the worker application with --scheduler parameter. This thing will be sending tasks according to crontabs.

This functionality was added in release 0.1.0.

repo: taskiq-python/taskiq


text:
Import to AWS ElasticSearch fails with "Request size exceeded 10485760 bytes"

This is perhaps somewhere between a bug and a feature request.
Context
In order to help us troubleshoot issues with this project, you must provide the following details:

ElasticDump version: 6.12.0
Elasticsearch version: AWS ElasticSearch Service, ES v5.5
Node.js version: not sure
Full command you are having issue with (with inputs and outputs):
gzip -cd data.json.gz | docker run -v /home/ubuntu/.aws/credentials:/root/.aws/credentials --rm -i taskrabbit/elasticsearch-dump --retryAttempts=5 --retryDelay=5000 --concurrency=3 --intervalCap=50 --concurrencyInterval=5000 --carryoverConcurrencyCount=true --throttleInterval=1 --awsChain --type=data --input=$ --output=<AWS ES endpoint URL>
A data-set you can share publicly which reproduces the problem: sorry, but dataset is private

Describe the bug
Attempt to import data from a gzipped data file in to an AWS hosted ES instance.  Import fails with "Request size exceeded 10485760 bytes" message.
To Reproduce
Steps to reproduce the behavior:

Import data.
See error.

Current behavior
Import fails.  Logging shows:
Importing data to <AWS ES endpoint URL>
Fri, 04 Oct 2019 10:09:49 GMT | starting dump
Fri, 04 Oct 2019 10:09:49 GMT | got 100 objects from source file (offset: 0)
Fri, 04 Oct 2019 10:09:51 GMT | got 36766 objects from source file (offset: 0)
Fri, 04 Oct 2019 10:09:52 GMT | sent 100 objects to destination elasticsearch, wrote 100
Fri, 04 Oct 2019 10:09:52 GMT | got 0 objects from source file (offset: 100)
Fri, 04 Oct 2019 10:09:52 GMT | Error Emitted => {"Message":"Request size exceeded 10485760 bytes"}
Fri, 04 Oct 2019 10:09:52 GMT | Total Writes: 0
Fri, 04 Oct 2019 10:09:52 GMT | dump ended with error (get phase) => Error: {"Message":"Request size exceeded 10485760 bytes"}
Fri, 04 Oct 2019 10:09:52 GMT | Error Emitted => {"Message":"Request size exceeded 10485760 bytes"}

Expected behavior
Import succeeds.
Additional context
A quick Google for "Request size exceeded 10485760 bytes" turns up this page which says, "This happens because the default value of the max_chunk_bytes keyword arg for bulk() is 100 megabytes, but Amazon only allows 10 megabytes per chunk (as evidenced by that error message)."
Could this problem be worked around by using a lower value for --limit, or does one of the other options somehow allow the setting of max_chunk_bytes to a different value?  If not, could the ability to set max_chunk_bytes and/or other bulk insert properties be exposed as command line args?

A good bug report shouldn't leave others needing to chase you up for more information. Please try to be as detailed as possible in your report. What is your environment? What was the url? What steps will reproduce the issue? What OS experience the problem? All these details will help to fix any potential bugs as soon as possible.

The default value for limit is 100 records, If your records are large then use --limit to reduce the amount of items imported.

Seems the limit is determined based on the instance type : https://docs.aws.amazon.com/elasticsearch-service/latest/developerguide/aes-limits.html

authors: ferronrsmith pawoolley

repo: taskrabbit/elasticsearch-dump


text:
[BUG] ReferenceError: URL is not defined on Node 8.10.0

Ubuntu 18
Node 8.10.0
Latest version of elasticsearch-dump
Exporting to an AWS ES url results in
Wed, 04 Dec 2019 21:48:08 GMT | starting dump
Wed, 04 Dec 2019 21:48:08 GMT | Error Emitted => URL is not defined
Wed, 04 Dec 2019 21:48:08 GMT | Total Writes: 0
Wed, 04 Dec 2019 21:48:08 GMT | dump ended with error (get phase) => ReferenceError: URL is not defined
Tried this both with input being a non-aws ES cluster and a json file, both result in the above behavior. Tried with AWS keys and with --awsChain=true
Upgraded to Node 10. Same command started working fine after the node version upgrade.

There isn't any information in this issue to help you.

Elasticsearch version
Node.js version (note that nodejs v8+ is required for this tool)
Full command you are having issue with (with inputs and outputs)

Really?
Elasticsearch version is irrelevant.
Node.js version was given. 8.10.0. Doesn't work.
Full command irrelevant, you can reproduce this with any set of inputs or outputs. I'm not giving you my employer's private server URLs. They won't do you any good anyway.
I solved my problem by googling "ReferenceError: URL is not defined" and finding a similar bug in another product that mentioned something not working in Node 8 and being fixed in Node 10 (https://github.com/GoogleChrome/lighthouse/issues/8909).
I solved my problem by upgrading my Node version. This type of response is a middle finger to those trying to help you and your users out. Do better.

A good bug report shouldn't leave others needing to chase you up for more information. Glad your issue was resolved. Thanks for using elasticdump and have a nice day

You don't need to chase me for anything. I gave you everything you need to reproduce and resolve this...if you actually cared to.

You 1) missed the Node version that I did provide, and 2) thought I was the one looking for help -- shows how much attention you actually paid to reading this.
Anyway, thanks for the tool, it's useful now that I've wasted half a day solving something you could now fix with a two-character change in your documentation. Good day indeed.

authors: EugeneFeingold ferronrsmith


text:
Add deita-7b-v1.0 model

Deita 7B V1.0 is a fine-tuned + DPO version of Mistral-7B-v0.1 that was trained on only 6K automatically selected lightweight, high-quality alignment SFT data: Deita 6K V0 and 10K randomly sampled alignment preference data from Ultrafeedback. And deita-7b-v1.0 achieves 90.06% win rate on alpacaeval.

Added results and model_configs of deita
updated the leaderboard

thanks for the contribution!

authors: VPeterV rtaori


text:
window.TAURI.invoke() throws "_this.transformCallback is not a function"

Describe the bug
Whenever I call window.__TAURI__.invoke(), I get this error:
_this.transformCallback is not a function. (In '_this.transformCallback(function (r) {
        resolve(r);
        delete window[error];
      }, true)', '_this.transformCallback' is undefined)

To Reproduce
Code:

Make a custom command
Run that command with window.__TAURI__.invoke() (see also line 4)

Runnable example:

Clone https://github.com/edbrannin/multi-diff (at commit: 70a8f04a78ef8e0effc0052ea98077c0be1c2bb8)
Run yarn tauri dev

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Platform and Versions (please complete the following information):
Operating System - Mac OS, version 10.15.7 X64

Node.js environment
  Node.js - 12.20.1
  @tauri-apps/cli - 1.0.0-beta-rc.2
  @tauri-apps/api - 1.0.0

Global packages
  npm - 6.14.11
  yarn - 1.22.10

Rust environment
  rustc - 1.51.0
  cargo - 1.51.0

App directory structure
/node_modules
/public
/src-tauri
/build
/.git
/src

App
  tauri.rs - 1.0.0-beta-rc.0
build-type - bundle
CSP - default-src blob: data: filesystem: ws: http: https: 'unsafe-eval' 'unsafe-inline'
distDir - ../build
devPath - http://localhost:3000

Additional context
I've been using window.__TAURI__.invoke() because of #1512 -- I'll try using the imported version now that rc2 is out.
Stack Trace
[Error] Error:
TypeError: _this.transformCallback is not a function. (In '_this.transformCallback(function (r) { resolve(r); delete window[error]; }, true)', '_this.transformCallback' is undefined)
user-script:3:101:45
Promise
invoke — user-script:3:100
(anonymous function) — App.js:48
asyncFunctionResume
(anonymous function) — App.js:30
asyncFunctionResume
(anonymous function) — App.js:41
invokePassiveEffectCreate — react-dom.development.js:23487
callCallback — react-dom.development.js:3945
dispatchEvent
invokeGuardedCallbackDev — react-dom.development.js:3994
invokeGuardedCallback — react-dom.development.js:4056
flushPassiveEffectsImpl — react-dom.development.js:23574
unstable_runWithPriority — scheduler.development.js:468
(anonymous function) — react-dom.development.js:23324
workLoop — scheduler.development.js:417
flushWork — scheduler.development.js:390
performWorkUntilDeadline — scheduler.development.js:157
	(anonymous function) (main.chunk.js:86)
	asyncFunctionResume
	(anonymous function)
	promiseReactionJobWithoutPromise
	promiseReactionJob

Can you please update and try again? It seems that tauri.rs is out of date:
https://crates.io/crates/tauri/1.0.0-beta-rc.2

Whoops, I thought tauri deps update would take care of that.  It seems like it's checking, but isn't finding the update...
yarn run v1.22.10
$ tauri deps update
[tauri]: running deps
 dependency:manager Updating dependencies... +0ms
 app:spawn [sync] Running "rustup update" +190ms

info: syncing channel updates for 'stable-x86_64-apple-darwin'

  stable-x86_64-apple-darwin unchanged - rustc 1.51.0 (2fd73fabe 2021-03-23)

info: cleaning up downloads & tmp directories
 dependency:cargo-commands "tauri-bundler" is up to date +942ms
 dependency:crates "tauri" is up to date +298ms
 dependency:npm-packages "@tauri-apps/api" is up to date +2s
 dependency:npm-packages "@tauri-apps/cli" is up to date +1s
✨  Done in 4.99s.

Sorry, I'm a rust newbie; I'll try to figure this out

dang - you're right! it seems that it won't find the beta. @lucasfernog - got an idea here? this will probably follow us for a while

I ran cargo update in src-tauri and it's updated:
    Updating crates.io index
    Updating jobserver v0.1.21 -> v0.1.22
    Updating open v1.6.0 -> v1.7.0
    Updating tauri v1.0.0-beta-rc.0 -> v1.0.0-beta-rc.2
    Updating weezl v0.1.4 -> v0.1.5

I tried again and it's still happening.

It works when I import invoke, so this may just be a problem with window.__TAURI__:
import { invoke } from '@tauri-apps/api/tauri';

I'm going to move on now, but let me know if I can be useful for resolving the root cause somehow. :)

Thanks for the headsup and the speedy resolution. Going to close this, since your issue was resolved.

Sounds good, though I hope you won't forget to check on window.__TAURI__.invoke() :)

Hmmm the issue happened because you changed the context of window.__TAURI__.invoke. I can change it to not rely on this and it'll work.

authors: edbrannin lucasfernog nothingismagick

repo: tauri-apps/tauri


text:
[docs] how to listen application:openURLs: event

wrote an application using tauri2
Now, select a file -> right-click and select "Open with" -> select the application you just wrote
Then check the log and print it as follows
[2023-12-08][01:22:42][tao::platform_impl::platform::app_delegate][TRACE] Trigger application:openURLs:
[2023-12-08][01:22:42][tao::platform_impl::platform::app_delegate][TRACE] Get application:openURLs: URLs: [Url { scheme: "file", cannot_be_a_base: false , username: "", password: None, host: None, port: None, path: "/usr/local/text.txt", query: None, fragment: None }]
Excuse me:
How do I listen to this event application:openURLs: in tauri2

Any solution to help?

https://docs.rs/tauri/2.0.0-beta/x86_64-apple-darwin/tauri/enum.RunEvent.html#variant.Opened / https://github.com/tauri-apps/tauri/blob/dev/examples/file-associations/src-tauri/src/main.rs#L52-L71

thanks

authors: FabianLars RWDai poohlaha


text:
🛑 Server CHARLY is down

In fe7ad58, Server CHARLY (https://charlie.tauronik.de:5001) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Server CHARLY is back up in 1d9c8d7 after 53 minutes.

repo: tauronik/tauro-upptime


text:
🛑 wiki.tauronik.de is down

In 8bef111, wiki.tauronik.de (https://wiki.tauronik.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: wiki.tauronik.de is back up in 4314b17 after 35 minutes.


text:
The reference link to handlebars page no longer works.

I tried visiting the handlebars page to get a guide for using conditions in the HTML, but the page link has been changed.
The link needs to be updated.

I would like to work for this issue. can you assign it to me?. Thanks.

repo: tawseefnabi/pdf-node


text:
Update to Ceres Solver 2.0

Could you guide me how to implement Ceres Solver 2.0 on Window 10 and Visual studio 2017?

Upgrading the Visual Studio version is easy: you can open the latest solution and follow the upgrade instructions within VS.
Upgrading to Ceres Solver 2.0 requires loading the new version into the VS solution: this means that files that no longer exist need to be removed from the solution, and new files that are not compiled should be added. Just add all the new files from the folder to the project. The solution itself contains some macro definitions and settings that should be kept for the build to be successful, but the files can easily change.
Let me know if this works for you, and I'd be happy to accept a PR if it works.

Thank you for your guides. I will implement them.

Hello, I followed instructions at https://www.cnblogs.com/Pyrokine/p/14636228.html. AIso, I used glog 0.5.0 and VS2017 instead. Also,  suggested using vcpkg.

authors: PhuDoTuong tbennun

repo: tbennun/ceres-windows


text:
Move tests into their own directory to cleanup the test directory.

I know that you were not so keen to this change in the cmake branch, but I think it make the test directory cleaner and more readable, so I attempted to isolate this change and propose it separately from the cmake branch. In addition, test/Makefile.am is slightly improved by adding some variables, which make it -- in a minor fashion as I do not expect dirname changes so often -- easier to maintain...
Fully tested on linux fedora-27 x86-64 with no regressions (no matlab tests, but I do not see how those would specially fail when all others are succeeding).

I guess you forgot to adapt testsuite.at.

Strange, it worked on my computer. There is in theory no testsuite.at change as we just specify a test include directory in the Makefile.am. I'll investigate....

Indeed I forgot to update the -I in the autotest command line. The last commit should correct it. I dp not know why it worked when I tested it. Maybe I used a wrong repository (I have many) or forgot to update with autoreconf. Let's see if tarvis passes now. Note that this also shows one advantage of cmake build. When it is finished the testsuite will also work for windows builds.... I just realized yesterday that the testsuite was unix/macos only....

Coverage remained the same at 81.189% when pulling af155c48d690a5705a686bc5f065c6cd34464b6f on papadop:TestsReorg into 6600c3d22cb9a71d1002d98de4656fb5905c4b6b on tbeu:master.

authors: coveralls papadop tbeu

repo: tbeu/matio


text:
Add an example of including overridden Backbone classes/methods(non AMD)

Similar to the last one, it would be nice to have an example of the 'recommended' way to override Backbone classes and methods (i.e. extend them to make new 'base classes' that can be used to instantiate actual object instances).
At the moment, I'm not using the namespace.module() method to create modules for these classes. If there's a nice way to do that, I'd love to see it.

Yeah this is a pain point in the non-amd branch.  I'll mark this for documentation.

authors: kurtmilam tbranyen

repo: tbranyen/backbone-boilerplate


text:
Retry failed dynamic import

TLDR: Retrying a once failed dynamic import is not possible without hacking around the browser and spec and this leads once failed imports to get blocked for the whole session.
In the current version of the proposal it's declared, that a once failed import should always fail in the future (1.1.2 - 3).
In my understanding this came from the fact, that a once resolved import should always return the same module instance so two calls to the same import don't retrigger the module setup.
To this point I'm completely in favour of the proposal since it matches as closely as possible the static import.
But if we now change from successful imports to a failed import I can't see the need to not retry an earlier failed import. A typical usecase would be to have a single page app which loads custom elements as modules as they are needed. If a user now has a short network outage he will never during his session be able to reimport that module (a hacky workaround is to add a nonce to the url, but that also breaks the "each module is only instantiated once" rule since now one module becomes two).
In my opinion a call to import should work the following way:

That way it's possible to retry a failed import and you don't need to hack around the browser and specification like described above. This would not touch static import in any way and there's already the misconception out there that dynamic import is retryable.
If there is any reason except for ease of implementation for the browsers to keep the current behavior for dynamic import, please let me know.

I've added an item to the upcoming TC39 agenda to discuss loosening this restriction. In particular I think it should match the requirements of HostResolveImportedModule so that failures are not required to be cached.
If we get TC39 signoff for allowing hosts (like web browsers) to not cache the results, the bulk of the work here would be done in the HTML Standard repository, which specifies exactly how URLs are fetched and the results are cached. @hiroshige-g and I are exploring this space a bit. Tentatively we have the following thoughts:

This should behave the same for static and dynamic imports. For example we should just not cache failed fetches (like we do currently).
We are unsure how to treat parse errors. Should we continue caching them? Or should we allow retrying failed parses?
How this impacts multiple parallel module graph fetches seems complicated.

Thanks for your answer and I like your idea.
My thoughts on your bulletpoints are the following:

I strongly agree
I think the ideal way would be to reject the promise with a parsing error. That way a developer can decide himself if he wants to retry or give up. I understand the concern by thinking "the file isn't going to change anyway", but I can see some cases where the problem might be fixed before the website reloads.
I think the start of a module fetch needs to be synchronized between two module graph fetches. That way multiple parallel subgraphs might be able to share the same subgraph for module loading. Isn't this also a problem with multiple module graphs fetching the same modules at slightly different times? I think this is a problem that doesn't only apply to this problem.

Personally I think it would be better to only allow retry of modules that failed due to errors during the link stage (but not parsing errors). This would cover cases where if a module failed to fetch but wouldn't allow retry-ing a module that simply throws an error during evaluation (e.g. wrong environment).

From the diagram it seems to imply that given this:
// file1.js
console.log("Hello!");

throw new Error("Failed!");

// file2.js
import("./file1.js");
import("./file2.js");

Would print Hello! twice, but I think it would be better that only linking errors (e.g. failed to even get source text) should be allowed to be retried expect for parsing errors which while in the linking phase are program errors rather than loading errors.

Even though I didn't mean to have this as a possibility during the creation of the graph, I see some usecases where it would be good to retry any failed module which didn't yet resolve successfully.
On the other hand the sideeffects would be really unexpected as a developer and I always see cases where a module might break in weired ways because of a retry after execution start.
Overall I agree that your example should indeed only log Hello! once and once execution of a module started there should'nt be a way back.

throw new Error("Failed!") is not a parsing error. It is an evaluation error.

Closing this since Domenic's issue was closed and this is no a duplicate.
Might be interesting to force the retry on failed by the spec, but in the means of this issue the current solution is fine.

Sorry to add to a close issue... Just double checking.
I fit precisely the use case described by  @Snapstromegon -- I have a SPA; what I am actually doing, is loading the bare minimum of my SPA and then slowly, in the background, load the rest of the app using import(). I made sure that if an import failed, the code would retry after 5 seconds assuming a network error. I was confused when I noticed that retrying just wasn't happening...
Am I right in saying that this behaviour has now been rectified, and that future browsers will indeed re-attempt loading failed imports? And, do you guys know which version of Chrome/Safari is taking on the correct behaviour? Is there a ticket anywhere?
I am trying to decide if I should scrap dynamic imports altogether for my SPA, since this limitation basically renders my code useless -- and dynamic imports for SPAs kind of unworkable...

authors: Jamesernator Snapstromegon domenic mercmobily


text:
standard for nested stack traces (Error.prototype.cause)

Hi, not sure if this can be bolted on to this proposal, but there are libraries like verror out there that allow errors to be stacked. This is handy when you propogate an error through a system and want to attach more context to it as it bubbles up. Could we attach another standard method to the error class that allows us to optionally see if there is a parent error attached to an error instance? E.g.
function x() {
  throw new Error('error in x')
}

function y() {
  try {
    x()
  } catch(e) {
    throw new Error('error in y', e)
  }
}

function printError(e) {
console.log(e.stack)
  if (e.cause) {
    console.log('caused by')
    printError(e.cause)
  }
}

try {
  y()
} catch(e) {
  printError(e)
}

You may be interested in https://www.npmjs.com/package/es-aggregate-error, which will be in the language soon?
This proposal is currently two parts; the first is documenting/specifying the existing stack property; the second is adding static methods to provide a structured representation of the stack.
(Separately, I think that your idea would have to be a separate proposal; I think adding more things onto Error.prototype is unlikely; and I'm not clear on why this needs to be in the language at all)

ah interesting. I will look into it. My point was to standardize the shape of an error cause. This would allow unrelated libraries to consume errors with parent errors. (java has this as a standard feature for instance). If it doesnt fit in this proposal, then it doesnt fit though. Oh well

authors: andykais ljharb