bug: vim.g.bigfile_size not loaded before use since 12.20.0

Did you check docs and existing issues?

[X] I have read all the LazyVim docs
[X] I have searched the existing issues of LazyVim
[X] I have searched the existing issues of plugins related to this issue

Neovim version (nvim -v)
NVIM v0.11.0-dev-313+gfc9b70826-Homebrew Build type: Release LuaJIT 2.1.1716656478
Operating system/version
MacOS 14.5 (23F79) and Nixos 24.05
Describe the bug
Since the "bigfile" type was added in release 12.20.0, when opening a file and determining filetype the following error is thrown due to lua/lazyvim/config/autocmds.lua being loaded before lua/lazyvim/config/options.lua.
Error executing lua callback: .../share/nvim/lazy/LazyVim/lua/lazyvim/config/autocmds.lua:125: attempt to compare nil with number
stack traceback:
	.../share/nvim/lazy/LazyVim/lua/lazyvim/config/autocmds.lua:125: in function <.../share/nvim/lazy/LazyVim/lua/lazyvim/config/autocmds.lua:121>
	vim/filetype.lua: in function ''
	vim/filetype.lua: in function 'match'
	...llar/neovim/HEAD-fc9b708/share/nvim/runtime/filetype.lua:14: in function <...llar/neovim/HEAD-fc9b708/share/nvim/runtime/filetype.lua:10>

The "nil" value in this case being vim.g.bigfile_size from options.lua. I've created a quick and dirty fix locally but didn't feel confident enough in my understanding of your caching code to raise a PR with a proper solution (sorry!)
My fix is as follows and hopefully a better lua programmer than me can suggest something that will work in the long term.
add to top of autocmds.lua
if vim.g.bigfile_size == nil then
  vim.g.bigfile_size = 1024 * 1024 * 1.5 -- 1.5 MB
end

Steps To Reproduce

Update to V12.20.0
Attempt to determine a filetype of lua file (:e)

Expected Behavior
:set filetype returns "filetype=lua" and no nil comparison error is thrown.
Repro
-- DO NOT change the paths and don't remove the colorscheme
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({ "git", "clone", "--filter=blob:none", "https://github.com/folke/lazy.nvim.git", lazypath, })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  { "LazyVim/LazyVim", import = "lazyvim.plugins" },
  -- add any other plugins here
}
require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")
-- add anything else here

Could you give a link to your dotfiles on Github?

options.lua should always load before autocmds, so no idea how you get that error

I asked a link to OP's github dotfiles, because all I could find was this, which doesn't seem to be a LazyVim configuration. It seems more like OP tried to incorporate LazyVim into his own configuration. But maybe I'm wrong and the OP just doesn't have his dotfiles available on Github.

Yes, look like it.
What's up with all these people posting issues that have some frankenstein LazyVim setup. Pretty annoying...


Santhoshcharan 001 patch 1

General CV With HTML and CSS
This is a General CV of a person. And utilized HTML, CSS to do this.

Done. Please check again.

and there are some conflicts again

Changes done. Go through it.

resolve conflicts and raise another PR


[release] The last recent release, only iOS release

I found that this tool was recently updated with release, but I clicked on it and found that only iOS and Linux-related resources were available. Other Windows and Android are gone.
Or will Android resources be integrated into Linux in the future? What about Windows resources?
If there is no build for others, could you please send it out after the build?

dobby无论是最新版还是之前的release版，都极其容易闪退。所以，我目前已经放弃用dobby了。


Ignore trailing slashes when running donejs add component

Steps to reproduce:
Within a DoneJS app, run:
donejs add component components/past-year/ past-year
Expected results:
A warning or the following output from the CLI:
   create src/components/past-year/past-year.html
   create src/components/past-year/past-year.js
   create src/components/past-year/past-year.md
   create src/components/past-year/past-year.less
   create src/components/past-year/past-year.stache
   create src/components/past-year/past-year-test.js
   create src/components/past-year/test.html

Actual results:
The following files were created:
   create src/components/past-year/.html
   create src/components/past-year/.js
   create src/components/past-year/.md
   create src/components/past-year/.less
   create src/components/past-year/.stache
   create src/components/past-year/-test.js
   create src/components/past-year/test.html

Environment:

Software
Version

donejs -V
1.0.0

generator-donejs version
1.0.5

node -v
7.9.0

npm -v
4.2.0

Operating system
macOS 10.12

Reopening this issue because @mjstahl ran into it with<EMAIL_ADDRESS>

@chasenlehara I checked the test, it passes, I'm still investigating why it doesn’t work!


Share connection across Vue components using Object API

Hi. I'm new to using @capacitor-community/sqlite and am having some difficulties integrating it into my setup.
I'm building an iOS/Android app with Vue 3 and Capacitor 3. Note that I'm not using Ionic Vue, just a standard Vue CLI scaffolded project. Also, I'm using the Options API in my Vue components, rather than the Composition API (and I'm not using Typescript).
I've managed to create a connection to an SQLite db and can create tables and perform SELECT, INSERT etc. However I'm stuck on how to share a single SQLite database connection across all of my Vue components. I am not using vue-sqlite-hook or $existingConn as I don't know if/how these can be implemented without using Vue's Composition API.
I thought perhaps I could instead create a database connection as a utility that I import into all my components but I've been unable to get this to work. Any suggestions would be much appreciated. Thanks!

@jeremyburton thanks for using the plug-in I am in vacation time right now so I cannot test anything. I do not know why you make your life difficult in not using the latest technologies coming with Vue. The vue-sqlite-app-starter demonstrates how to do that. If you do not want to use Ionic/Vue you can do that by removing all the Ionic features

@jepiqueau thanks so much for getting back to me whilst you're on vacation. I've been using Vue for a few years and have built up a strong mental model (and lots of my own components) using the options API. I'm sure I'll want to learn to use the composition API at some point but until now I haven't needed it and it wasn't an option until quite recently. If, when you're back from your vacation, you have the time, it would be really useful to see an example that demonstrates how to use the plugin in a way that is less framework-specific (e.g. by importing a plain javascript service). Thanks again and enjoy your vacation!

@jeremyburton Did you try to create the connection in the main.js file  with a variable app.config.globalProperties$connection pointing to it and share it in your components like
         const connection = app?.appContext.config.globalProperties.$connection;

i think this may work but it is possible with one connection, otherwise you may do it for the CapacitorSqlite plugin and share the plugin. this is some ideas not tested. Can you revert to me if it is solving your issue

@jepiqueau thanks for replying again. I have managed to share the initial sqlite connection as a global property in main.js:
...
import { CapacitorSQLite, SQLiteConnection } from "@capacitor-community/sqlite";
const sqliteConnection = new SQLiteConnection(CapacitorSQLite);

const app = createApp(App).use(router);
app.config.globalProperties.$sqliteConnection = sqliteConnection;

router.isReady().then(() => {
  app.mount("#app");
});

And I can access this global property in components:
const sqliteConnection = this.$sqliteConnection;
const db = await sqliteConnection.createConnection("MY_DB", false);

However, I now get a "connection already exists" error when I connect in this way from multiple components. I understand why I get this error but I'm not sure how I should access an already open db connection. Sorry, I know this is very basic stuff but I'm new to SQLite. Thanks again for your help.

@jeremyburton on each component you can check if a connection exists isConnection  if yes you do not create it otherwise you create it. It also may happen that you need to run checkConnectionConsistency  if it return false you recreate the connection

@jepiqueau thanks, makes sense, but I'm not sure how to access the connection if it already exists:
const sqliteConnection = this.$sqliteConnection;
let db;
if (sqliteConnection.isConnection("MY_DB")) {
  // Get existing db connection
  db = // WHAT GOES HERE?
} else {
  // Create a connection to the db
  db = await sqliteConnection.createConnection("MY_DB", false);
}

@jepiqueau is this right?
if (sqliteConnection.isConnection("MY_DB")) {
  // Get existing db connection
  db = await sqliteConnection.retrieveConnection("MY_DB");
} else {
  // Create a connection to the lollysticks db
  db = await sqliteConnection.createConnection("MY_DB", false);
}

@jeremyburton yes it is right

@jeremyburton in fact you should do
If (!sqlite.connection.isConnection("MY_DB") || !checkConnectionConsistency()) {
// create connection
} else {
// retrieve connection
}

@jepiqueau I'm trying to get this working but .isConnection always returns false and I've not been able to retrieve the connection.
      let db;
      if (
        !sqliteConnection.isConnection("MY_DB") ||
        !sqliteConnection.checkConnectionConsistency()
      ) {
        // Create a connection
        db = await sqliteConnection.createConnection("MY_DB", false);
      } else {
        // Get existing connection
        db = await sqliteConnection.retrieveConnection("MY_DB");
      }

In the Xcode console I get:
[log] - isConnection false
[error] - {}

@jeremyburton can you share your code or a small replicate which can demonstrate the issue that will avoid me to start from scratch as now I do not understand why it is not working

@jepiqueau thanks once again for your ongoing help with this. I've created a basic version of my code for you to take a look at: https://github.com/jeremyburton/vue-capacitor-sqlite. The initDB() method in the App component is supposed to seed the database and then in the Home component I'm trying to read from the database. I may well have misunderstood the correct sequencing of these database calls but I think that is a separate issue to the problem I'm having with retrieving an exisiting connection.

@jeremyburton I cannot install your app for the time being as I get a timeout in npm install due to a really poor connection where I am right now. I do not think in the App.Vue you need to test if there is a connection as obviously there is none when you mount the app. So just create the connection. As you did not close the connection at the end of App which is right when you enter home the connection should be there and the db is still opened as you did not do db.close () in App which is also right.so in Home in case the connection was lost for any internal reason your test is correct but after createConnection you need a db.open () but when you retrieveConnection ("MY_DB") you do not need to do a db.open () as it was done in App.vue like this it should work

@jepiqueau Thanks, I have made the changes you suggested but this does not resolve all the issues. The initDB() method in App component is being executed AFTER the getTeams() method in the Home component. The initDB() method works as expected but the getTeams() method appears to fail when .isConnection("MY_DB") returns false. I have updated the code so you can try it yourself when you are able to. Please feel free to leave this until you get back from your vacation. I can work on something else for now.
Logging from Xcode:
2021-06-25 12:17:40.912404+0100 App[5233:63111] DiskCookieStorage changing policy from 2 to 0, cookie file: file:///Users/jeremy/Library/Developer/CoreSimulator/Devices/C32C0DCE-1121-4CE7-BDE4-3F9A26A7F26E/data/Containers/Data/Application/568E042B-4808-4890-9435-A96BAAED481B/Library/Cookies/app.clevercrow.vuecapacitorsqlite.binarycookies
⚡  Loading app at capacitor://localhost...
2021-06-25 12:17:41.225690+0100 App[5233:63111] WF: === Starting WebFilter logging for process App
2021-06-25 12:17:41.225857+0100 App[5233:63111] WF: _userSettingsForUser : (null)
2021-06-25 12:17:41.225997+0100 App[5233:63111] WF: _WebFilterIsActive returning: NO
⚡  [log] - Check if we have an exisiting connection...
⚡  [log] - isConnection false
⚡  To Native ->  CapacitorSQLite createConnection 24022815
⚡  [log] - Make initial connection to db...
databaseName: MY_DBSQLite.db
database path /Users/jeremy/Library/Developer/CoreSimulator/Devices/C32C0DCE-1121-4CE7-BDE4-3F9A26A7F26E/data/Containers/Data/Application/568E042B-4808-4890-9435-A96BAAED481B/Documents/MY_DBSQLite.db
⚡  TO JS undefined
⚡  WebView loaded
⚡  [log] - >>> in SQLiteDBConnection dbName MY_DB
⚡  To Native ->  CapacitorSQLite open 24022816
⚡  [log] - >>> in SQLiteDBConnection open dbName MY_DB
⚡  TO JS undefined
⚡  To Native ->  CapacitorSQLite execute 24022817
⚡  TO JS {"changes":{"changes":0}}
⚡  To Native ->  CapacitorSQLite execute 24022818
⚡  TO JS {"changes":{"changes":0}}
⚡  To Native ->  CapacitorSQLite execute 24022819
⚡  TO JS {"changes":{"changes":4}}
⚡  To Native ->  CapacitorSQLite execute 24022820
⚡  TO JS {"changes":{"changes":12}}

@jeremyburton I will come back to you when I will have found the solution it is related to Vue and async component

Hi @jeremyburton - how about leveraging async during the mount step so the connection is available from the beginning?

Use the db everywhere, instead of the SQLiteConnection
Force the app to wait until the connection is established before mounting (and stuff begins to render and be visible to the user)

import { CapacitorSQLite, SQLiteConnection } from "@capacitor-community/sqlite";
const sqliteConnection = new SQLiteConnection(CapacitorSQLite);

const app = createApp(App).use(router);

router.isReady().then(async () => {
  app.config.globalProperties.$db = await sqliteConnection.createConnection("MY_DB", false);
  app.mount("#app");
});

In regards to testing (off device), I can recommend decoupling the use of db directly and instead exposing the regular interactions you make (select, update, etc), so they can be dependency injected during testing. Still figuring this part out myself, though :)

@jeremyburton you may do this but if you want to use several databases in your app it will not work. The problem is more specific to Vue. The Home.Vue is execute before the initialization of the sqlite.connection in the App.Vue this is why it is returning false. It is the problem of async component where you should embedded them into a Suspense Tag. I am pretty sure that if you create a Team.Vue and you route to that Vue after that the Home.Vue is mounted  it will work. You do nothing in the Home.Vue. I saw this when I was pressing on the About.Vue anD come back to the Home.Vue the code was well executed. I think also your code is more Vue2 than Vue3.  You should have a look at the vue-sqlite-app-starter to help you even if it is in typescript I define Vue calling component with the Suspense tag good luck I am still travelling

@jeremyburton you can find the proposed solution at https://github.com/jepiqueau/vue-capacitor-sqlite-issue137.git hope this will help you to find your way for your real app. It was to find the proper way in Vue to deal with async methods and component. The Home setup was run before the database was created that why await sqliteConnection.isConnection("MY_DB") was always returning false. I let you closing the issue. Good luck

@jeremyburton Did you had time to have a look

@jepiqueau Sorry, I've had a few busy days. I haven't had a chance to implement your solution yet but I did read through the code and I can see how it solves the problem. I did try using the solution suggested by @codeluggage and that worked for me. I'm not sure I'll ever need to access multiple databases in this app but I'll probably go with your solution just in case. If you're okay to wait a couple of days, I'll come back and close this issue when I've had the chance to implement the proposed solution and test it out. Many thanks to both of you for your help with this issue and I look forward to getting stuck into using the plugin in my app.

@codeluggage do you have any code you can share with me to show how you have decoupled the plugin implementation from your db calls so that you can test off device? That would be super helpful as I'm too sure of the best way to do this.

@codeluggage do you have any code you can share with me to show how you have decoupled the plugin implementation from your db calls so that you can test off device? That would be super helpful as I'm not too sure of the best way to do this.

Ha, you and me both! 😉
I've been through so many "sqlite in memory" projects recently, but each has had some kind of issue. The final solution I ended up with to TDD with Cypress was to use https://github.com/agershun/alasql for in-memory sqlite when the app is not running on-device.
In general the same schemas will work, but I have not been able to use https://github.com/sql-js/sql.js to read from disk (wasm issues in one way or another - when I did get past loading wasm, loading a basic sqlite file hit memory errors). Instead, I exported data to use for testing to csv, put those in public/assets and loaded them with Alasql.
I can post some relevant snippets if you'd like to go on the same adventure :joy:

@codeluggage that sounds like a whole new world of pain that I could probably do without. But thanks for your overview and I now know who to bug for help if I do choose to go down this route at some point 😉

@jeremyburton i will close the issue. Feel free to re-open if necessary

@jepiqueau I have attempted to implement your solution but the execution of the async Teams component fails. You can see my code here: https://github.com/jeremyburton/vue-capacitor-sqlite
Also, your suggested solution makes use of Suspense which is an experimental Vue 3 feature and the official docs state this should not be used in production.

@jeremyburton i update my app https://github.com/jepiqueau/vue-capacitor-sqlite-issue137.git where i remove the tag Suspense. I also look at https://github.com/jeremyburton/vue-capacitor-sqlite to make it working do the following ( i forgot that we did not use vue-sqlite-hook)

in Home.vue remove Suspense

<template>

    <Teams />

</template>

in Team.vue the method is sqliteConnection.checkConnectionsConsistency()

<template>

Teams

        {{ team.name }}

</template>

with this it works

in Team.vue the method is sqliteConnection.checkConnectionsConsistency()

@jepiqueau thank you so much! I'm delighted to finally have this working. Thanks again for all your help :-)


Remove the CLI gradle build in Travis CI

@LukeFarrell @taylorking @alexsong93
FYI: Once this PR is merge you need to rebase any PR you have to be able to pass travis


Tech/re build

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update README schema.png

Updating schema.png to reflect the new schema in the multitenant system

Coverage remained the same at 96.737% when pulling ec772e8e1537cc7446bbfe099a5bbf676dcd3e30 on update-schema-readme into d5cb14783d39033710be692abcca670caaed966d on master.


[e2e]: nodejs push branch1 default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8611103119
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Tue Apr  9 06:14:15 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/branch1
Run: https://github.com/slsa-framework/example-package/actions/runs/8626752334
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.nodejs.push.branch1.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.nodejs.push.branch1.default.slsa3.yml
Trigger: push
Branch: branch1
Date: Wed Apr 10 06:12:24 UTC 2024
Tests are passing now. Closing this issue.


Implement fee scaling based on the size of the memos

A revision of #3007 to  scale the fees for transactions with many memos which is csauing a spamming problem. It may be better to just change the coe to change by the size of the transaction instead which works for other expensive things too likea lot of paths.

I would like to see some sort of Flag, or something that gets added to the response above and beyond simply bumping the fee. How does a legitimate dev know they may have tripped up on this?
There needs to be at least something say hey, you'r causing the fee's to rise.

I would like to see some sort of Flag, or something that gets added to the response above and beyond simply bumping the fee. How does a legitimate dev know they may have tripped up on this?
There needs to be at least something say hey, you'r causing the fee's to rise.

I'm not quite sure I follow? The fee change here isn't dynamic based on load: that is, it increases the amount of the fee purely based on the size of the memo, so a developer would simply check if the corresponding amendment is activated and, if so, calculate the fee using the algorithm implemented in this pull requests.

I'm not quite sure I follow? The fee change here isn't dynamic based on load: that is, it increases the amount of the fee purely based on the size of the memo, so a developer would simply check if the corresponding amendment is activated and, if so, calculate the fee using the algorithm implemented in this pull requests.

Apologies, I took for granted that there was a point at which it kicked in (exceeds limit). Fee is just dynamic end of story 👍🏻

I'm not certain this requires an amendment? It would make coordination/communication easier, but in the end transactions with higher fees are still valid to every node that doesn't run this code and if enough validators run this code, transactions with lower fees than required by the code would just get rejected.

I'm not certain this requires an amendment? It would make coordination/communication easier, but in the end transactions with higher fees are still valid to every node that doesn't run this code and if enough validators run this code, transactions with lower fees than required by the code would just get rejected.

As implemented, it does: it changes the code that calculates the minimum fee.
A non-amendment based implementation is possible, where servers could require higher local fees, but would accept transactions with lower fees if the network reached consensus about including such a transaction.
Personally, I'm fine with the amendment-based approach, because it makes it easier for everyone to calculate the minimum required fee, although it's hardly an "easy" thing to do, especially when faced with queue congestion/levels.

@a-noni-mousse what are your latest thoughts on this change?

@a-noni-mousse can you resolve the merge conflicts?
Otherwise, this PR may be closed due to inactivity. You can always open a new PR in the future, of course.

@a-noni-mousse if merge conflicts are not resolved, this PR will be closed. But don't lose heart - you can always re-open it or open a new PR with the changes in the future :)

Closing due to inactivity. Feel free to open a new PR with this change (rebased on the head of develop).


F5 to reload, middle click to close

Hello,
There are two shortcuts which I’m really used to from web browsers and missing in lagrange:

F5 reloads the page (same as ctrl+R)
Middle click to close tabs

Middle click to close tabs

Where do you middle click for this? (I don't use regular mice that much myself.)

Middle click to close tabs

Where do you middle click for this? (I don't use regular mice that much myself.)

On the tab «header». The same place you left-click to switch to a tab.

On the tab «header». The same place you left-click to switch to a tab.


Updating HealthcareApis SDK to refer to stable api version

Updating HealthcareApis SDK to refer to stable api version
The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.
REST spec: https://github.com/Azure/azure-rest-api-specs/blob/master/specification/healthcareapis/resource-manager/Microsoft.HealthcareApis/stable/2019-09-16/healthcare-apis.json

Hi @poadhika , the PR looks good to me. It's just that the path of the metadata file healthcareapis_resource-manager.txt is wrong, it should be in eng\mgmt\mgmtmetadata\. Could you run

msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules
then re-run generate.ps1 to see if the metadata is generated in the right place? Thanks

@isra-fel I have updated the PR. Please take a look


[$1000] IOU - IOU is displayed as Merchant in IOU details page when IOU is created without a Merchant

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify,com
Create money request without selecting a merchant
Open the money request report
Click on the IOU preview

Expected Result:
Merchant is not stated because money request is created without selecting a merchant
Actual Result:
IOU (or sometimes, Request) is shown for Merchant when money request is created without selecting a merchant
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome
[ ] MacOS / Desktop

**Version Number:**1.3.56-2
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/4ec6929d-e591-4809-b2c2-e8e7a89b449f
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~019d6df5b1911e3c11
Upwork Job ID:<PHONE_NUMBER>885807104
Last Price Increase: 2023-08-22

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

@deetergp Looks like its coming from here
https://github.com/Expensify/App/blob/999ed938befa8910293cf349a4222314d5a73ccd/src/libs/TransactionUtils.js#L46

Is it necessary to have merchant or we can leave it blank?

Now its showing Request as merchant
Looks like its getting overridden from BE

Confirmed it
IOU is coming due to optimistic transaction data
And now for blank merchant BE will pass Request as merchant

Proposal
Please re-state the problem that we are trying to solve in this issue
IOU - IOU is displayed as Merchant in IOU details page when IOU is created without a Merchant
What is the root cause of that problem?
This is due to during requestMoney call we are using buildOptimisticTransaction to create optimistic report data
And we use CONST.REPORT.TYPE.IOU as the default value for merchant in the places
https://github.com/Expensify/App/blob/999ed938befa8910293cf349a4222314d5a73ccd/src/libs/TransactionUtils.js#L46
and
https://github.com/Expensify/App/blob/999ed938befa8910293cf349a4222314d5a73ccd/src/libs/TransactionUtils.js#L69
What changes do you think we should make in order to solve the problem?
Replace CONST.REPORT.TYPE.IOU with empty strings in buildOptimisticTransaction.
What alternative solutions did you explore? (Optional)

I don't think this issue is big enough to consider it a blocker—it is pretty minor.

@rushatgabhane Can you review my proposal?

Hey guys! I reported this bug in Slack before this issue was created. Could you please change the reporter's name? Thanks! Here's the link to the bug report: Bug Report Slack Link (https://expensify.slack.com/archives/C049HHMV9SM/p1692687129320969)

This is not an issue, all requests should have a default merchant name. We're updating it to the correct merchant name in this PR

Gonna close this one.


I hope to defend against CSRF by customizing the request header

Due to the poor user experience of synchronizer token pattern and double submit cookies, we would like to implement the Custom Request Headers approach mentioned in the linked document.
OWASP Cheat Sheet Series
The document also provides detailed explanations of the synchronizer token pattern and double submit cookies methods.

Now, CSRF token can be carried in Header, Query, and Form. It is recommended to carry it in the Header to separate it from the business logic. For security reasons, it is advisable to prohibit carrying the CSRF token in the Query.

Done


Broken channel: Star Wars Sunday.

The new url should be: https://youtu.be/SIfKANrHfnk
Maybe we could add contribution guide to this project so that other developers could fix simple issues like this?

The new url should be: https://youtu.be/SIfKANrHfnk
Maybe we could add contribution guide to this project so that other developers could fix simple issues like this?

Came here to report the same.
Sometimes, the error handling also seems to fail. Although I haven't yet checked if that's YouTube not reporting the error (usually error 150) in a timely manner, or go-app stalling.
Anyhow. I'm gonna prepare a PR for the new URL tomorrow. Maybe bundling it with one or more other broken ones (the star Wars one isn't the only one, AFAIK)
I can also update the README with build instructions, for those who aren't accustomed to reading makefiles.

The feeds is a little spotty but is currently working

This stream is broken again


Add SqlCe DataProvider  to EF7

sqlserver compact is best .NET compact and lite database and very developers use it and need add to EF7. sqlserver compact better support .NET and very performance.
and very user votes to support in visual studio 2013-14 and work on new version
http://visualstudio.uservoice.com/forums/121579-visual-studio/suggestions/4714470-support-sql-server-compact-in-visual-studio-2013
http://visualstudio.uservoice.com/forums/121579-visual-studio/suggestions/5794792-sql-server-compact-5

@rowanmiller Any news on updated guidance for provider writers? https://github.com/aspnet/EntityFramework/wiki/Writing-an-EF7-Provider

@ErikEJ saw you got started, other folks have just been looking at our providers. We'll have more docs once things stabilize but they are changing quite rapidly at the moment.

@rowanmiller  @mojtabakaviani Implementation of the EF7 SQLCE provider is underway: https://github.com/ErikEJ/EntityFramework.SqlServerCompact - only major stumbling block so far is the Update (SaveChanges) pipeline, due to SQLCE limitations/features

@mojtabakaviani @rowanmiller Feel free to close this now, SQLCE provider fully implemented (RevEng support under consideration)

I think SQLCE rich to end and must migrate to SQLITE. However, Thanks!

@mojtabakaviani Good luck with that!


Error installing Drupal

Issue Type

Bug Report / Support Request

Your Environment

Vagrant 1.8.6
ansible <IP_ADDRESS>

Your OS

macOS (10.12)

Full console output

https://gist.github.com/Fynkymynky/7fc356ec35a5704a36829da95a20304e
Summary
I've downloaded the latest stable build from the website, when I run the build process, Vagrant fails during the Drupal installation stage. I've attempted this out of the box from the website and with using the Drush make method, neither work.

It looks like the actual error is:
[RuntimeException]
Could not scan for classes inside "web/core/lib/Drupal.php" which does not appear to be a file nor a folder

And searching that on Google brought me to https://github.com/drupal-composer/drupal-packagist/issues/66 — it looks like the solution is to update Composer (composer self-update), then delete the contents of the drupal directory and run vagrant provision again.
Can you see if that works?

Thanks, I downloaded the latest build from git and that worked.

I solved this issue by running composer self-update, deleting the vendor directory and then running composer install.

Deleting the vendor directory and running composer install seems to solve it.

It did solve it! Thank you :-)


Roll engine c0e6f6a6e665..23fcc2771884 (1 commits)

https://github.com/flutter/engine/compare/c0e6f6a6e665...23fcc2771884
git log c0e6f6a6e665..23fcc2771884 --no-merges --oneline
23fcc2771 Prepare for stripping and dsyming Flutter.framework (flutter/engine#6247)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff, who should
be CC'd on the roll, and stop the roller if necessary.

@rmistry @liyuqian : New roll has been created by https://autoroll.skia.org/r/flutter-engine-flutter-autoroll

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/6719611168882688 , https://cirrus-ci.com/task/5593711262040064

Commit queue failed; closing this roll.


HAWQ-448. hawq_global_rm_type is case sensitive

This makes HAWQ RM able to recognize string of hawq_global_rm_type in a case insensitive way.

LGTM

LGTM. +1


Severe slowdown for unit tests on macos-latest

Describe the bug
I'm a core developer of the open source project Spack. We are currently using Github Actions to run our unit tests on MacOS and this used to work great. Unfortunately, since a while we experience a severe slowdown (in the order of 4-5x) on the times we were used to get and we still have to track the root cause of it. The slowdown only happens in Github Actions and cannot be reproduced on local machines.
Can you help tracking this down and see if there's a bug on either our code or introduced by any actions/virtual-environments update?
Area for Triage:

Apple Python Testing and code coverage
Question, Bug, or Feature?:

Question
Virtual environments affected

[x] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[ ] Ubuntu 20.04 LTS
[ ] Windows Server 2016 R2
[ ] Windows Server 2019

Expected behavior
The last good run on our develop branch is at https://github.com/spack/spack/runs/758344523, the first run showing the slowdown (and a failure) is at  https://github.com/spack/spack/runs/760158098
It seems that between the two this update https://github.com/actions/virtual-environments/commit/c954a9f815c241711de842c7c71e2938a6d497a3#diff-e9f9822df5b3b9087dd94d4c3c083277 happened on the virtual environments and I wonder if that is taking a part on what we see. I have currently disabled a failing test in https://github.com/spack/spack/pull/17072 but the slowdown is still present.
Actual behavior
The current behavior is something like https://github.com/spack/spack/runs/760158098

Hi @alalazo! Could you please provide some simple sample code affected by this slowdown apart from the whole spack project? It'd help us a lot with the investigation.
Thank you!

@miketimofeev I can try to check if the slowest tests are consistent across runs and see if they share some common part in Spack. Would that be enough or still to much to analyze?

@alalazo we'd like to test everything on our end, so it'd be great if you can provide some sample code that works much faster locally than on macOS VM now because there were no significant changes between those VM images from your builds except macOS version.

@alalazo unfortunately, we can't do it in actions, but can test in another environment with multiple image versions, which also takes some time, if some sample code is provided. Is it enough to fork Spack repo and run some commands to push the tests?

Is it enough to fork Spack repo and run some commands to push the tests?

This is not a separate piece of code, but you can run:
% git clone git@github.com:spack/spack.git
[ ... ]
% cd spack
% bin/spack test -k test_stack_view_multiple_views
============================================================= test session starts =============================================================
platform darwin -- Python 3.7.3, pytest-3.2.5, py-1.4.34, pluggy-0.4.0
rootdir: /Users/culpo/github/spack, inifile: pytest.ini
collected 2651 items

lib/spack/spack/test/cmd/env.py .

========================================================== slowest 20 test durations ==========================================================
10.95s call     lib/spack/spack/test/cmd/env.py::test_stack_view_multiple_views[mock_archive0]
0.02s setup    lib/spack/spack/test/cmd/env.py::test_stack_view_multiple_views[mock_archive0]
0.02s teardown lib/spack/spack/test/cmd/env.py::test_stack_view_multiple_views[mock_archive0]
============================================================ 2650 tests deselected ============================================================
================================================= 1 passed, 2650 deselected in 14.97 seconds =================================================

It takes ~15 sec. locally and in the previous virtual-environment, while it seems to be stable at ~90 sec. in the current MacOS virtual-environment. Let me know if that helps.

@alalazo thanks! @dsame will give it a try

Hello @alalazo,
I was able to reproduce the issue on the previous image version, but currently, it seems the issue is gone with new macos image release.
Could you please verify on your side?
We are looking forward to your reply.

@Darleev I see that most of the runs went back to the usual ~15 mins. for unit tests, except a few ones like this one. Do you have any hint on what was causing the issue?

@alalazo I believe this pipeline is stuck and needs to start a new one. It seems like something strange happened to the agent machine.
Could you please try to simply restart the pipeline?

@Darleev Just did it.

@Darleev Confirmed that it took the usual ~15 mins. after restart. I'm still puzzled by what was possibly causing the slowdown in the previous image.

@alalazo We have briefly reviewed updated software, but no idea what was causing the issue.
As soon as the issue appears again, please let us know.
Since the initial issue has been resolved, I'm going to close the ticket. In case of any questions feel free to contact us


refactor: Resolved alignment of beta

Beta is aligned next to Undo and Redo
refactor #10

@VivekTarun let's add refactor as a prefix to PR title.

@VivekTarun let's add refactor as a prefix to PR title.

yeah i have added

hey @VivekTarun are you still working on it ?

sure i ll pull it to local
On Sun, 25 Feb 2024 at 18:13, Amar Khamkar @.***> wrote:

@.**** commented on this pull request.
In src/components/CanvasEditor/Footer/Help/Help.scss
https://github.com/calcont/whiteboard/pull/13#discussion_r1501813617:

   border-bottom: 1px solid var(--light-border-color);

   &:last-child {
     border-bottom: none;
   }
 }

&-content {

 display: flex;

 align-items: center;

 width: 100%;

}
&-text {

 display: flex;

 align-items: center;

 margin-right: 10px;

Also please take latest pull from master. Bit of things changed from
initial release.
Thanks !
—
Reply to this email directly, view it on GitHub
https://github.com/calcont/whiteboard/pull/13#discussion_r1501813617,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT5WDBLJHPYMMFFUC5GPTKDYVMWY3AVCNFSM6AAAAABDAPPL26VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTQOJZGY3TANJRGU
.
You are receiving this because you were mentioned.Message ID:
@.***>


[bug] [v2] [iOS] Manual Xcode build fails with cargo: command not found

Describe the bug
I generate an Xcode project through cargo tauri. When I then open it and try to build in Xcode manually, the script execution phase fails with the following error (long line, scroll to the end):
/Users/stijn/Library/Developer/Xcode/DerivedData/temp-epvdqgocduaasecmmqocxpzwtttl/Build/Intermediates.noindex/temp.build/debug-iphoneos/temp_iOS.build/Script-6BC118688969AAE7C46FEBA4.sh: line 2: cargo: command not found
Command PhaseScriptExecution failed with a nonzero exit code

I looked at the script it tries to execute, which is only this:
#!/bin/sh
cargo tauri ios xcode-script -v --platform ${PLATFORM_DISPLAY_NAME:?} --sdk-root ${SDKROOT:?} --framework-search-paths "${FRAMEWORK_SEARCH_PATHS:?}" --header-search-paths "${HEADER_SEARCH_PATHS:?}" --gcc-preprocessor-definitions "${GCC_PREPROCESSOR_DEFINITIONS:-}" --configuration ${CONFIGURATION:?} ${FORCE_COLOR} ${ARCHS:?}

That's weird, because cargo is obviously installed on my system. The shell listed is /bin/sh, and when I manually open a terminal by running /bin/sh, I can run e.g. cargo --version just fine.
Replacing cargo in the script with /Users/stijn/.cargo/bin/cargo does the trick, so there's probably something wrong with the PATH.
Reproduction

Create a new project. I used pnpm + Svelte + TS, but I guess this fails with any combination.
I also ran pnpm install for good measure
cargo tauri ios init
cargo tauri ios dev --open --host
When Xcode has opened, hit Cmd + B and see the failure

Expected behavior
I expect the Xcode build to finish
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.6.1 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.80.1 (3f5fd8dd4 2024-08-06)
    ✔ cargo: 1.80.1 (376290515 2024-07-16)
    ✔ rustup: 1.27.1 (54dd3d00f 2024-04-24)
    ✔ Rust toolchain: stable-x86_64-apple-darwin (environment override by RUSTUP_TOOLCHAIN)
    - node: 20.14.0
    - pnpm: 9.6.0
    - yarn: 1.22.22
    - npm: 10.7.0

[-] Packages
    - tauri [RUST]: 2.0.0-rc (no lockfile)
    - tauri-build [RUST]: no manifest (no lockfile)
    - wry [RUST]: no manifest (no lockfile)
    - tao [RUST]: no manifest (no lockfile)
    - tauri-cli [RUST]: 2.0.0-rc.4
    - @tauri-apps/api [NPM]: 2.0.0-rc.1
    - @tauri-apps/cli [NPM]: 2.0.0-rc.4

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../build
    - devUrl: http://localhost:1420/
    - framework: Svelte
    - bundler: Vite

[-] iOS
    - Developer Teams: <redacted>

### Stack trace

```text
N.A.

Additional context
N.A.

This is an oddity with Xcode, and as you said probably related to how PATH is resolved. You can link cargo to /usr/local/bin so Xcode will pick it up. Don't ask me why. Something like ln -s $(which cargo) /usr/local/bin/cargo.
I had a similar issue with pnpm and node, but not cargo which is odd. Did you install via rustup?

Wow, that's super weird. Thanks Xcode!
Linking to my cargo install works perfectly. I guess that resolves the issue, it's a problem in Xcode.
Yeah, I installed with rustup, the usual path.

Seems this can be done in a cleaner way by sourcing a shell profile in the prebuild scripts, e.g.
(src-tauri/gen/apple/project.yml)
# ...
targets:
  app_iOS:
    type: application
    platform: iOS
    preBuildScripts:
+      - script: source ~/.zshrc && source ~/.profile
+        name: Load shell profile

      - script: tauri ios xcode-script
        name: Build Rust Code
        basedOnDependencyAnalysis: false

This could be something that Tauri provides out of the box. Example above assumes zsh, but a more robust shell detection would be needed if this is built-in. Maybe just documenting this would be ok?

same problem


Support reading unicode in the template files

Codecov Report

Merging #31 into master will increase coverage by 0.49%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
+ Coverage   33.08%   33.58%   +0.49%
==========================================
  Files           4        4
  Lines         133      134       +1
  Branches       22       22
==========================================
+ Hits           44       45       +1
  Misses         89       89

Impacted Files
Coverage Δ

jupyterlab_templates/extension.py
73.33% <50%> (+0.91%)
:arrow_up:

Continue to review full report at Codecov.

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


Improved tests assertions in cache tests.

Always assert the result of .add(), .delete(), .touch(), incr(), .decr(), and .get_or_set().
Assert bools using .assertIs() to verify its type is correct, not only its truthiness.

@jdufresne Thanks :+1:


Additional layer above the current system

To be able to handle a virtual ZITADEL instance for each customer, we need a level above the admin api, where we can manage more than one ZITADEL instance.

[x] #3061
[ ] #3062

tracked in #2943


NotfoundError: Key bert/embeddings/LayerNorm/beta not found in checkpoint

When I run the below code which evaluates a model from a saved checkpoint,
model_fn_check = model_fn_builder(
  bert_config=modeling.BertConfig.from_json_file(CONFIG_FILE),
  num_labels=len(label_list),
  init_checkpoint=INIT_CHECKPOINT,
  learning_rate=LEARNING_RATE,
  num_train_steps=num_train_steps,
  num_warmup_steps=num_warmup_steps,
  use_tpu=False,
  use_one_hot_embeddings=True
)

estimator_from_checkpoints = tf.estimator.Estimator(
  model_fn=model_fn_check,
  config=run_config,
  params={"batch_size": BATCH_SIZE}
)

estimator_from_checkpoints.evaluate(input_fn=valid_input_fn, steps=int(len(valid_features) / BATCH_SIZE))

I get the following error:
NotFoundError: 2 root error(s) found.
  (0) Not found: Key bert/embeddings/LayerNorm/beta not found in checkpoint
	 [[{{node save/RestoreV2}}]]
  (1) Not found: Key bert/embeddings/LayerNorm/beta not found in checkpoint
	 [[{{node save/RestoreV2}}]]
	 [[save/RestoreV2/_301]]

Is there a solution?
I checked that these tensors are present in the checkpoint but I still get the error.
from tensorflow.python import pywrap_tensorflow
import os

checkpoint_path = INIT_CHECKPOINT
reader = pywrap_tensorflow.NewCheckpointReader(checkpoint_path)
var_to_shape_map = reader.get_variable_to_shape_map()

for key in var_to_shape_map:
    print("tensor_name: ", key)

tensor_name:  module/bert/embeddings/LayerNorm/beta

Any one has solution for this?

Same error  with TensorFlow 1.14.0

the tensor names are inconsistent, module/bert/embeddings/LayerNorm/beta and bert/embeddings/LayerNorm/beta

I had the same problem, changed the names, it solved.


jenkin collectore

I am having this weird issue.
It says -> c.c.d.collector.CollectorTask - http://localhost:8090/
but
Client exception loading build details: 404 Not Found. URL =http://localhost:8080/job/TeleCloud/17/
why it is taking 8080 and not 8090?
2017-04-07 12:55:00,002 INFO  c.c.d.collector.CollectorTask - Running Collector: Hudson
2017-04-07 12:55:00,023 INFO  c.c.d.collector.CollectorTask - -----------------------------------
2017-04-07 12:55:00,023 INFO  c.c.d.collector.CollectorTask - http://localhost:8090/
2017-04-07 12:55:00,023 INFO  c.c.d.collector.CollectorTask - -----------------------------------
2017-04-07 12:55:00,069 INFO  c.c.d.collector.CollectorTask - Fetched jobs                     0s
2017-04-07 12:55:00,070 INFO  c.c.d.collector.CollectorTask - New jobs                0        0s
2017-04-07 12:55:00,083 WARN  c.c.d.collector.DefaultHudsonClient - Credentials for the following url was not found. This could happen if the domain/subdomain/IP address in the build url returned by Jenkins and the Jenkins instance url in your Hygieia configuration do not match: "http://localhost:8080/job/TeleCloud/17/api/json?tree=number,url,timestamp,duration,building,result,culprits[fullName],changeSets[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind],changeSet[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind,revisions[module,revision]],actions[lastBuiltRevision[SHA1,branch[SHA1,name]],remoteUrls]"
2017-04-07 12:55:00,087 ERROR c.c.d.collector.DefaultHudsonClient - Client exception loading build details: 404 Not Found. URL =http://localhost:8080/job/TeleCloud/17/
2017-04-07 12:55:00,090 WARN  c.c.d.collector.DefaultHudsonClient - Credentials for the following url was not found. This could happen if the domain/subdomain/IP address in the build url returned by Jenkins and the Jenkins instance url in your Hygieia configuration do not match: "http://localhost:8080/job/TeleCloud/16/api/json?tree=number,url,timestamp,duration,building,result,culprits[fullName],changeSets[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind],changeSet[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind,revisions[module,revision]],actions[lastBuiltRevision[SHA1,branch[SHA1,name]],remoteUrls]"
2017-04-07 12:55:00,093 ERROR c.c.d.collector.DefaultHudsonClient - Client exception loading build details: 404 Not Found. URL =http://localhost:8080/job/TeleCloud/16/
2017-04-07 12:55:00,094 WARN  c.c.d.collector.DefaultHudsonClient - Credentials for the following url was not found. This could happen if the domain/subdomain/IP address in the build url returned by Jenkins and the Jenkins instance url in your Hygieia configuration do not match: "http://localhost:8080/job/TeleCloud/15/api/json?tree=number,url,timestamp,duration,building,result,culprits[fullName],changeSets[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind],changeSet[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind,revisions[module,revision]],actions[lastBuiltRevision[SHA1,branch[SHA1,name]],remoteUrls]"
2017-04-07 12:55:00,095 ERROR c.c.d.collector.DefaultHudsonClient - Client exception loading build details: 404 Not Found. URL =http://localhost:8080/job/TeleCloud/15/
2017-04-07 12:55:00,100 WARN  c.c.d.collector.DefaultHudsonClient - Credentials for the following url was not found. This could happen if the domain/subdomain/IP address in the build url returned by Jenkins and the Jenkins instance url in your Hygieia configuration do not match: "http://localhost:8080/job/TeleCloud/14/api/json?tree=number,url,timestamp,duration,building,result,culprits[fullName],changeSets[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind],changeSet[items[user,author[fullName],revision,id,msg,timestamp,date,paths[file]],kind,revisions[module,revision]],actions[lastBuiltRevision[SHA1,branch[SHA1,name]],remoteUrls]"
2017-04-07 12:55:00,101 ERROR c.c.d.collector.DefaultHudsonClient - Client exception loading build details: 404 Not Found. URL =http://localhost:8080/job/TeleCloud/14/

@bhuranideepak82  - share your jenkins properties file. check this value jenkins.servers[0]=http://localhost:8080/  or
jenkins.servers[0]=http://localhost:8090/

Check your Jenkins Configuration screen and make sure your Jenkins URL is setup correctly.

yes it worked. Thanks and appreciate your quick response.

same issue for me aswell, tried by giving 2 URL's but no luck.


Allow naming .excalidraw file before download

Edit: This is now more about how to name the .excalidraw file, we don't want to add naming as suggested originally
Edit: Original suggestion:
Something like this (design can be improved of course):

I'm often just downloading the file and renaming it later, without going into the export dialog.. because I want to download it as .excalidraw and this option doesn't even exist there.
Also if we make that change, we should even consider starting with just Untitled instead of excalidraw-123456789, as the word excalidraw is already in the file extension.

This was the initial proposal for the title but I vetoed it.

I see, asking for a name before downloading it's also a bad UX.. dunno :) If we make it a bit gray or something?! :D

What if we made a share button there that opened a popup with buttons for sharing a link, saving, and exporting? That would enable there to be a reasonable place to add the text field and clear up a couple of spots on the top bar for new features.

@dai-shi I just tried and it seems like there is a Save-as dialog (at least on MacOS), so I guess this can be closed?

@lipis ?

Sure..

Am trying to understanding the intended behavior which seem counter intuitive from what most document oriented systems facilitate.
Right now the UI automatically generated name is proposed as Untitled-${ISO_DATE}, an evaluated (template) value? It seems however that it does only this and it's not possible to allow this to be set by user for UX complexity reasons?   Checking because the consequence is that if the user wants to name the file the same they must re-type the desired name each repeatedly (every time a save?)
Is that the case?
The naming approach implies that the string literal for Untitled could also  be templatized, and a reasonable place for it to be stored in the localStorage, DOM and/or document state where it could be checked/loaded as a very limited change without new IA or UI.
If there isn't any interested in adding this functionality in the user interface; would a pull request we entertained to allow for a default value to be set in local storage or in the excalidraw dom/json representation?
In looking through the json schema, there isn't a name property for the board and if adding a proper 'name' value is controversial; a default value object/entry could be stored in the files collection to and allow this functionality to exist without having to build new UI patterns or sledgehammering a browser plugin to hijack the dom.


Swift Codes out of date

So I really love the gem, makes it super easy to validate swift codes. However, the current yml files are out of date. I was wondering how you acquired this list, then I would be happy to update them so they are current.
Any advice you can give would be great.

@zquestz take a look here: https://www.theswiftcodes.com/


Inconsistent carousel .pause() bahaviour

Whem pausing a carousel using .pause(), I see inconsistent behaviour.
I expect what happens on Desktop Firefox:

Auto-sliding off.
Manual sliding works
Manual sliding does not resume auto-sliding

Under some conditions, manual sliding does resume auto-sliding. So far this happens with one of these conditions:

Touch device (even Firefox simulated or Firefox on Android)
Chromium (Edge and Chrome tested)
Even weirder: jsfiddle on Firefox (even though Firefox Desktop usually is the only "normal" one).

So here are my questions:

What is the intended behaviour?
If manual sliding should always resume auto-sliding, it's a Firefox problem. But how could this be achieved? Or perhaps a "heavier" option to .pause() which disables the manual sliding options too might be handy (.dispose() seems to be very "light")?

When calling .pause() on a Carousel object, mobile users will still unpause after an interaction.
Here's a jsfiddle: https://jsfiddle.net/4nqr1od2/
As mentioned above, I see inconsistencies on jsfiddle vs local server.
Note that the carousel starts as auto-sliding and has a 1s interval. The Button indicates which action should be expected to be performed on click; the actual performed action based on carousel._isPaused is logged and highlighted.

The default behavior is to pause on 'hover focus'. Are you sure you don't want to use 'manual' here?

Pause on hover or focus for mobile users is what I need, coincidentally. Even if not, I would expect the auto-pause to not interfere with a manually triggered pause.
In fact, a hover doesn't interfere with it; only a touch-focus or (touch- / click-) navigation using indicators or the button arrows.
I feel the behaviour is not as expected and inconsistent.
It should either be clarified in the documentation (if this was intended); or adjusted.
Sorry for the weird label btw

I'm experiencing same issue here with bootstrap 4.6.0:

When I initiate carousel, I use an id for scope and defining these options:$('#my-carousel').carousel({
    interval: false,
    pause: true,
    wrap: false
})

I control navigation manually and each time that we finish scroll slid.bs.carousel event, I'm pausing the auto-scroll:$('#carousel').on('slid.bs.carousel', ({ target }) => {
    $(target).carousel('pause');
});

And when user do a mouseleave, touchend or blur:$carousel.on('mouseleave touchend blur', ({ target }) => {
    $(target).carousel('pause');
});

While we do minimal navigation (like scrolling 2 - 3 times), auto-scroll doesn't happen but, the more user interact with the carousel, the change gets higher and higher and ends up in a state where it can scroll between 2-5 times at the same moment, like accumulated yield line codes for this auto-scroll.
This carousel is attached with tab navigation but both functions doesn't seems to collide, saw no event triggering thanks to carousel.
Also tried with a different scope, using always $('#my-carousel') instead of target, same result. Also with/out pause and wrap values, doesn't affect at all and also tried with a different approach--datasets and no, same result.
I have another function that calls carousel(${carouselItemIndex}) but this happens when we trigger slide.bs.carousel and this is before slide completes (there are only 2 carousel items, always) and with that, I'm even pausing it as well and the above event also pause it... doesn't respect it.
And no, I don't have another carousel with a scope of $('.carousel') on another file that goes along with the one that I have this code, even if that is the case, slid.bs.carousel should stop it after auto-scroll.
Any ideas?


Upcasting bug in compiler (0.26.1)

Possible compiler bug in Crystal 0.26.1 while upcasting an abstract struct into a struct. May be similar to the issues #6760, #5496, #1346.
Code to Reproduce
https://gist.github.com/ljuti/d719507523f6c9fc8725672b893b0951
The gist includes console output from a development machine running OSX High Sierra 10.13.6.
Environment
$ crystal -v
Crystal 0.26.1 (2018-09-26)

LLVM: 6.0.1
Default target: x86_64-apple-macosx

Reduced:
abstract struct BaseFactory
end
struct DummyFactory < BaseFactory
end

struct Config
  alias AcceptedFactories = BaseFactory.class | DummyFactory.class

  def initialize(@given_factory : AcceptedFactories)
    @given_factory.new
  end
end

Config.new(DummyFactory)

Please always try to minimize examples by removing everything that's not necessary to reproduce the error.

It could be an issue that the ivar has type BaseFactory.class and an abstract struct can't be instantiated.

This no longer reproduces in Crystal 0.34.


example client/server to transmit data

I want to try write dns server in go, but instead of standard axfr mechanism to transfer data, i want to use mesh.
Where i need to start? Thanks!

Hi @vtolstov,
The WeaveDNS (blog entries, src, docs) server uses mesh to transfer records among multiple peers. Have you taken a look into it?

This is perhaps a more relevant blog post:
https://www.weave.works/weave-gossip-dns/
On Mon, Aug 1, 2016 at 5:21 PM, Martynas Pumputis<EMAIL_ADDRESS>wrote:

Hi @vtolstov https://github.com/vtolstov,
The WeaveDNS (blog entries https://www.weave.works/tag/dns/, src
https://github.com/weaveworks/weave/tree/master/nameserver, docs
https://www.weave.works/docs/net/latest/weavedns/) server uses mesh to
transfer records among multiple peers. Have you taken a look into it?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/mesh/issues/50#issuecomment-236630590, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAbGhaZ1hxryseCgNuR5_F62n8p2iTVwks5qbh0UgaJpZM4JYx17
.


Sentences are using present tense.

https://github.com/kijk2869/discodo/tree/master/example/local/example2.py
Some strings of example2.py is using present tense instead of past tense.
For example
Shuffle the queue.

Importing module on local was deprecated


fix: code completion for nested pascal case names

Fixes #

Codecov Report

Merging #41 (fa98d67) into main (df1bbfc) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #41   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            1         1
  Lines           38        42    +4
  Branches        12        12
=========================================
+ Hits            38        42    +4

Impacted Files
Coverage Δ

src/caseConvert.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Write and use a sorting module

Cryptol 2.13 is released.  We should at least cover sort/sortBy in Basics.md, but could go so far as to write a module that describes these instructions and proves they are stable (for a monomorphic sequence and comparator).
In either case, we should also replace the current partition with a faster, simpler implementation that stably sorts the filter results.

@weaversa Ah, I see you already amended the partition text.


Remove unnecessary rt crate dependencies

Fixes #382

remove dependencies to rt crates and riscv / xtensa-lx from the chip specific HALs, re-export from esp-hal-common
re-export riscv / xtensa-lx in the chip specific HALs
entry is re-exported and added to the prelude
atomic emulation is always included by the HAL for ESP32-S2 (like it's already the case for ESP32-C2 / ESP32-C3)

This PR looks huge but most of the changes are in the examples.
Nice side-effect: fixes #197
Probably we could nit-pick about some names (e.g. trapframe::TrapFrame) - I tried to choose the names carefully but don't have strong opinions about them
BEFORE merging this

[ ] publish a new xtensa-lx-rt version, probably also xtensa-lx
[ ] publish the PACs (with removed rt dependencies)
[ ] optionally wait for a new published esp-backtrace

I will get new PACs published today


Content Security Policy: script-src without 'unsafe-inline' impacts rendering

Which @angular/* package(s) are the source of the bug?
Don't known / other
Is this a regression?
No
Description
I have content security policy header defined as,
default-src 'self'; img-src * 'self' data:; object-src 'none'; child-src * mailto: tel: ms-word:; script-src 'self' 'nonce-loader'; style-src 'self' 'unsafe-inline'; form-action 'self' ; frame-ancestors 'self';
I dont have unsafe-inline for script-src
In app.component.html,

Test

In styles.css
h2 { color: green }
Deployed the application in the tomcat server.
Impact:
CSS rules defined in default styles.css is not rendered in the browser.
In the browser console, following error log found,

Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
Angular CLI: 12.0.0
Node Version: 14.15.0

Anything else?
Does unsafe-inline for script-src is mandatory for an angular application?

Duplicate of https://github.com/angular/angular-cli/issues/20864


Provide some means of setting additional properties on process command

It would be nice to be able to support all additional properties that can be passedin into the start command:
var argvOptions = cli.argvOptions = {
  'command':   {alias: 'c'},
  'errFile':   {alias: 'e'},
  'logFile':   {alias: 'l'},
  'append':    {alias: 'a', boolean: true},
  'fifo':      {alias: 'f', boolean: false},
  'number':    {alias: 'n'},
  'max':       {alias: 'm'},
  'outFile':   {alias: 'o'},
  'path':      {alias: 'p'},
  'help':      {alias: 'h'},
  'silent':    {alias: 's', boolean: true},
  'verbose':   {alias: 'v', boolean: true},
  'watch':     {alias: 'w', boolean: true},
  'debug':     {alias: 'd', boolean: true},
  'plain':     {boolean: true}
};

+1 but it looks like this project has abandoned :/

A bit of investigation and I find that this project depends on forever directly and I'm not sure it supports parameters when used in a script approach (vs cli) and there hasn't been an update to forever in 4 months and there are a LOT of outstanding issues. I have a feeling that project has gone dormant so I'm going to look at grunt-nodemon which isn't very active either, but also doesn't have dozens of open issues on the core project.


fix(build): Fix E2E tests

[x] Fix e2e-tests-with-coverage target in Makefile.
[x] Don't check collection types in InstanceChecker.
[x] Give InstanceChecker access to the non-standard names of certain JSON properties.
[x] Update test data for:

[x] ResourcesRouteV2E2ESpec
[x] InstanceCheckerSpec

[x] Fix ClientApiRouteE2ESpec to load anything-data.ttl, now needed for client code generation.
[x] Add E2E tag to E2E specs that didn't have it.

Fixes #1525.

I assume these values were introduced in #1514 but since the e2e tests were not run the difference in the test data was not detected.

Actually, what happened is that at some point we added ARK URLs and UUIDs to values. After that, some hard-coded data that was supposed to fail validation (e.g. it had a boolean property with a string object, to check whether InstanceChecker would notice) started to fail validation because the hard-coded values in the test didn’t contain ARK URLs or UUIDs. So I had to add ARK UURLs and UUIDs to those values, to make them fail validation for the right reasons.

@subotic I don't understand what's happening. It does:
Setting up npm (3.5.2-0ubuntu4) ...

But then it says:
java.io.IOException: Cannot run program "npm" (in directory "/tmp/clientapi18252897938348417182/src"): error=2, No such file or directory

There is also this strange error, as if GraphDB sent a header saying that its response would be in GZIP format but the client couldn't unpack the file:
java.util.zip.ZipException: Not in GZIP format
2679
	at java.base/java.util.zip.GZIPInputStream.readHeader(GZIPInputStream.java:166)
2680
	at java.base/java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:80)
2681
	at java.base/java.util.zip.GZIPInputStream.<init>(GZIPInputStream.java:92)
2682
	at org.apache.http.client.entity.GZIPInputStreamFactory.create(GZIPInputStreamFactory.java:61)
2683
	at org.apache.http.client.entity.LazyDecompressingInputStream.initWrapper(LazyDecompressingInputStream.java:51)
2684
	at org.apache.http.client.entity.LazyDecompressingInputStream.read(LazyDecompressingInputStream.java:69)
2685
	at java.base/sun.nio.cs.StreamDecoder.readBytes(StreamDecoder.java:284)
2686
	at java.base/sun.nio.cs.StreamDecoder.implRead(StreamDecoder.java:326)
2687
	at java.base/sun.nio.cs.StreamDecoder.read(StreamDecoder.java:178)
2688
	at java.base/java.io.InputStreamReader.read(InputStreamReader.java:185)
2689
	at java.base/java.io.Reader.read(Reader.java:144)
2690
	at org.apache.http.util.EntityUtils.toString(EntityUtils.java:227)
2691
	at org.apache.http.util.EntityUtils.toString(EntityUtils.java:308)
2692
	at org.knora.webapi.store.triplestore.http.HttpTriplestoreConnector.$anonfun$getSparqlHttpResponse$1(HttpTriplestoreConnector.scala:662)
2693
	at scala.util.Try$.apply(Try.scala:213)
2694
	at org.knora.webapi.store.triplestore.http.HttpTriplestoreConnector.getSparqlHttpResponse(HttpTriplestoreConnector.scala:652)

When I run all the tests outside Docker, they all pass.

When I run make e2e-tests on the command line, it does some stuff and then stops with this error message:
make: *** No rule to make target `init-knora-test-unit', needed by `e2e-tests'.  Stop.

When I run the tests using make e2e, is there a way to stop them? I tried Control-C, but it didn't have any effect, the tests kept running. I looked in the Docker Desktop menu but didn't find anything for stopping containers. I tried installing Kitematic, but it doesn't show any local containers either. I ended up finding the sbt process ID and using kill -9.

When I run all the tests outside Docker, they all pass.

Then npm is missing inside the Docker container. I will add it to the image.

When I run the tests using make e2e, is there a way to stop them?

No, the only way would be to kill the created api container from terminal:
$ docker container ls
$ docker container rm <container_id> // you could try 'api' as the container_id

I've updated the Docker image and restarted the tests. Let's see if this all that is needed.

I've updated the Docker image

So do we still need this in main.yml?
      - name: install requirements
        run: sudo apt-get install nodejs npm

I've updated the Docker image

So do we still need this in main.yml?
      - name: install requirements
        run: sudo apt-get install nodejs npm

I don't think so.

OMG the tests passed!

Now it's failing because of this:
Triplestore responded with HTTP code 500:
Query evaluation error:
Insufficient free Heap Memory 240Mb for group by and distinct, threshold:250Mb, reached 0Mb (HTTP status 500)

Now it's failing because of this:
Triplestore responded with HTTP code 500:
Query evaluation error:
Insufficient free Heap Memory 240Mb for group by and distinct, threshold:250Mb, reached 0Mb (HTTP status 500)

Which test changed after the merge? GraphDB doesn't have enough memory.

Which test changed after the merge? GraphDB doesn't have enough memory.

No tests changed, it was the merge of #1527. Can we give GraphDB more memory?

@subotic Does this mean we can't use GitHub CI?

Which test changed after the merge? GraphDB doesn't have enough memory.

No tests changed, it was the merge of #1527. Can we give GraphDB more memory?

I don‘t think so.

Does this mean we can't use GitHub CI?

This means that we need to find a way how it can work with Github CI. Unfortunately, there is no alternative. After doing some calculations, running it on our own hardware is not feasible cost wise.
We could try and isolate the high memory tests with a HighMemory test tag and run them separately?

OK the tests that use a lot of memory are the ones that create large texts with lots of standoff. I think we don't really need to run these in CI anyway, so I set them to ignore. Otherwise everything seems to work. Can we merge this?


Plugin hooks idea

List of some Plugin with would be useful for Depenizen

Shopkeepers (has some citizen support)
Ability to add/remove/edit trades
http://dev.bukkit.org/bukkit-plugins/shopkeepers/
https://github.com/nisovin/Shopkeepers

ban-management (doesn't use defualt ban.json file)
http://dev.bukkit.org/bukkit-plugins/ban-management/

Sorry what?

i posted hit enter before editing sorry

I'll need to know exactly what you'd like from each to be added. E.g. what events, tags, or commands you would like implemented :)

Ban-Management
Uses mysql to store data. (Doesn't use banned-players.json)
http://dev.bukkit.org/bukkit-plugins/ban-management/pages/commands-permissions/
(/tempban & /ban)
<EMAIL_ADDRESS>- Check if banned
<EMAIL_ADDRESS>- How long a tempban has left
<EMAIL_ADDRESS>- Reason for banned
<EMAIL_ADDRESS>- Who banned player
<server.list_banned_players> - List of banned players
(/tempmute & /mute)
<EMAIL_ADDRESS>- Check if muted
<EMAIL_ADDRESS>- How long a tempmute has left
<EMAIL_ADDRESS>- Reason for mute
<server.list_muted_players> - List add muted players
(/warning)
<EMAIL_ADDRESS>- List warns are player has
ShopKeeper
Mostly want cmd to set trades of vilalger/npcs
Tags get trades i guess
http://nisovin.com/shopkeepers/shopkeeper_admin_editor.png

I can see a lot of conflictions with this, e.g. we already have most of these tags.
Will consider ban manager. Will look into ShopKeepers however.

Is ban manager support still wanted?
Also, we have some ShopKeepers support, though I'm not sure what still needs to be done there.
If you're still interested, we could finish up support for both plugins

Not really :) just needed the shopkeeper the most, I works greats thanks.


Adds --dest flag that sets the destBasePath

Implements #162.

Love this, was just digging through the code to find a way to do this!

Was trying to develop a work around for this myself, since I need our template generation to be available as a shared module across projects and can't hard code paths.
Would love to see this merged.


Individual Deck Page

Possible features:

Name
Pilot/Author
Colors
Legality
Cards by maindeck/sideboard
Sort cards by creature/noncreature then cmc in initial view
Cards by type
Legality in other formats
Chart of colors used
Chart of colors produced
Chart of card types
Chart of mana curve
"Buy on Cardhoarder/MTGO Traders"
Similar decks
Deck type
Archetype
Comments
Export (MTGO, others)
Source link
Competition(s)
Finish
Wins/losses
Created date/updated date
Visual view
Other decks from the same tournament
Suggested cards
Sort maindeck/sideboard by rarity/color/cmc/type/type-cmc

This is wotc's visual view:

It's from mtgg:

This is one of the things I'm planning on doing as part of StackIt's featureset.
It might be worth just doing it there, and calling it as an external tool?

Using StackIt, even as-is, would be pretty sweet. But there's something up with the typography and mana symbols in those images. Is it because I have a retina display or do they look jagged/low-res to you too?

It's not the fonts bundled in the app (see screenshot) but something about the manipulation of them in StackIt.

I spent some time on StackIt today and improved the quality, especially if we export larger images and resize them in the browser. Will try and get that working without having to produce larger images.
Before:

After:

So we can consider using this in some kind of "visual view". Or some expansion of it.

The "buy on Cardhoarder" link on mtggoldfish is actually pretty sweet and useful. You can post-hoc remove things you don't need like forests and alter numbers:

I'd want to avoid the suspicion that any money was changing hands, though, or making pdm look like a commercial enterprise so maybe not worth it for those reasons.

I messaged Cardhoarder about this and their affiliate program and expanding sponsorship to PDM!

Hmm ...

Maybe ...

It's kind of the same information as the main list so it should probably either replace it (bad?) or be a link/pop-up/clickthrough.

Hex TCGBrowser uses a badly signposted button.

Maybe just do something like this?

Cardhoarder agreed to sponsor all three tournaments so definitely going ahead with a buy on cardhoarder button.
Referring customers to our deck editor
If you provide decklists on your website or application, we highly recommend using our deck editor integration. Most affiliates that send customers to our deck editor benefit from very high conversion rates and large orders.
When linking to our deck editor, there are three key pieces of information you will need to communicate for each item in the decklist:
id MTGO ID
quantity Quantity
is_sideboard Is it in the sideboard?
This information should be bundled into a POST request, and sent to https://www.cardhoarder.com/decks/upload.
Here is an example of how that request should be formatted:
<form method="POST" action="https://www.cardhoarder.com/decks/upload?affiliate_id=mtgtop8">
  <input type="hidden" name="data[Deck][cards][0][id]" value="20920" />
  <input type="hidden" name="data[Deck][cards][0][quantity]" value="10" />
  <input type="hidden" name="data[Deck][cards][0][is_sideboard]" value="0" />

  <input type="hidden" name="data[Deck][cards][1][id]" value="58796" />
  <input type="hidden" name="data[Deck][cards][1][quantity]" value="15" />
  <input type="hidden" name="data[Deck][cards][1][is_sideboard]" value="0" />

  <input type="hidden" name="data[Deck][cards][2][id]" value="46737" />
  <input type="hidden" name="data[Deck][cards][2][quantity]" value="2" />
  <input type="hidden" name="data[Deck][cards][2][is_sideboard]" value="1" />

  <button type="submit">Buy on Cardhoarder</button>
</form>

In this example, our deck is only three cards: 10x Relentless Rats, 15x Swamp, and a couple Pack Rats in the sideboard. For each of these items, we need to have the MTGO ID, which can be found in our price file.
Try it out!
Buy on Cardhoarder
with auto-optimize
Auto-Optimize Feature
Pass the "optimize" flag to automatically process your decklist and replace items with the cheapest, in-stock alternative. You can include it as follows:

Use Card Names (rather than IDs)
We also support a URL-only deck integration that uses card names rather than MTGO IDs. This feature is a little less robust, since it does not support sideboard or specific card versions, but it can be easier to setup. You just need to setup your deck URL as shown below, with the quantity and card name, and separating each item with a double-pipe, ||.
https://www.cardhoarder.com/decks/upload?affiliate_id=mtgtop8&deck=10 Relentless Rats||15 Swamp||2 Pack Rat
Try it

Can you link me to the mentioned price file?
Hopefully it won't be too hard to integrate CatIDs.

We left price file for "down the road". They've been pretty generous so I want to tread lightly :) I'll circle back about prices soon-ish?

Sounds good :)

"Buy on Cardhoarder" is 60cd7468.
Next challenge: how to declare StackIt as a dependency in requirements.txt.

Oh dear.  I'm going to need to package StackIt up as a pip package, aren't I?

poppu-mtg/StackIt#7

created date is f6450cd3.

PRs to poppu-mtg/StackIt https://github.com/poppu-mtg/StackIt/pull/76 followed by https://github.com/poppu-mtg/StackIt/pull/55 will get us in a place where we can use this for decklist image generation. Although the quality will still not be as good as it could be. I'll try and figure that out.

Additional charts now filed separately as #2926.

All other oustanding work here now logged in separate tickets as #2928, #2929 and #2930.


Container Queries with Tooltip breaks

When I attempted to use tailwindcss container queries, whenever I had @container on a parent div, any child tooltips would just get messed up and not be able to be functional since the tooltip would show somewhere else, not where is being hovered.

This issue has been automatically closed because it received no activity for a while. If you think it was closed by accident, please leave a comment. Thank you.

just ran into this same issue!

I'm running into the same issue.
Whenever I use a @container on a parent component, the tooltip position breaks.

This fixs it:
https://github.com/shadcn-ui/ui/issues/2804
You need your <TooltipPrimitive.Content to be wrapped with:
<TooltipPrimitive.Portal>


analytics: Change subgroup and labels lists to dictionary.

I guess it could be a list of pairs as well, or a special class Label. Not sure what the most readable refactoring would be.

Automated message from Dropbox CLA bot
@rishig, it looks like you've already signed the Dropbox CLA. Thanks!

List of pairs are superior to dictionaries when the order of the keys is important, but for something this small, it's inexpensive to sort the keys, so I think this is fine.
I gave the code a shallow review, and it seems fine, but I don't know this code.

Looks great, merged, thanks @rishig!


Incorrect route is used in getAllNodes() call

Moved from https://github.com/weaveworks/service/issues/1053
The leading slash in the topologyUrlsById is causing the incorrect route to be requested.
Repro steps:

Within cloud, enter some search terms in scope.
JSON.parse() errors will appear in the console, as a result of a call to /api/topology/containers
Notice that the cloud index.html file is served instead of a JSON response.

The request to
https://cloud.weave.works/api/topology/containers
should be
https://cloud.weave.works/api/app/loud-breeze-77/api/topology/containers

What functionality, if any, does this break? Search seems to work fine in Weave Cloud.

Searching triggers a fetch of topologies which is throwing errors in the console. The search functionality itself seems to still work, since it does not rely on the result of the fetch. I'm not sure why searching needs to trigger that request, but its not working.

I'm not sure why searching needs to trigger that request,

This is worth investigating.

@rade Found it!:
    dispatch({ type: ActionTypes.FOCUS_SEARCH });
    // update nodes cache to allow search across all topologies,
    // wait a second until animation is over
    setTimeout(() => {
      getAllNodes(getState, dispatch);
    }, 1200);

We discussed a possible improvement in our cloud-prod sync today. I documented it here: https://github.com/weaveworks/service-conf/issues/500

Closed by #2124


fix: bad indentation in security.php

According to Github Actions, there is an indentation issue to security.php
It's corrected, but I wonder how it is possible that it's not detected locally, if you got an idea of why the behavior is different I can take a look at it later.

Thank you @viviengaetan!


Tuner UI, some metronome fixes

Looks good, though there are a few compiler warnings. I'd suggest:

Adding const to the char* args for plotText() and textWidth(). You use const a few places, which is good! Those two functions don't modify the pointer, so adding const there is smarter than removing (or casting) in tunernome.
Adding void as the argument for recalcMetronome() rather than leaving the parenthesis empty. That's a total C quirk. For further reading, see: https://riptutorial.com/c/example/31657/do-not-leave-the-parameter-list-of-a-function-blank---use-void
1#include <user_interace.h>forsystem_get_time()`
Make tAccumulatedUs a uint32_t. Doesn't look like it's going negative, but you compare it to an unsigned value.
The last warning is pain in the rear. You have default in a bunch of switches, but don't explicitly handle all the enum cases, so what you're doing is perfectly correct, and the compiler is being pissy. Just add a bunch of explicit semitone cases to get the compiler to be quiet.

user/modes/mode_tunernome.c: In function 'tunernomeUpdate':
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_0' not handled in switch [-Wswitch-enum]
             switch(tunernome->curTunerMode)
             ^
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_1' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_2' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_3' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_4' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_5' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_6' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_7' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_8' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_9' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_10' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'SEMITONE_11' not handled in switch [-Wswitch-enum]
user/modes/mode_tunernome.c:396:13: warning: enumeration value 'MAX_GUITAR_MODES' not handled in switch [-Wswitch-enum]

* Adding `const` to the `char*` args for `plotText()` and `textWidth()`. You use `const` a few places, which is good! Those two functions don't modify the pointer, so adding `const` there is smarter than removing (or casting) in tunernome.

Where exactly did you want me to add that?

Turn this:
int16_t plotText(int16_t x, int16_t y, char* text, fonts font, color col);
int16_t textWidth(char* text, fonts font);

into this:
int16_t plotText(int16_t x, int16_t y, const char* text, fonts font, color col);
int16_t textWidth(const char* text, fonts font);

in both the .c and .h files

In the prototypes in font.c/h, I assume, rather than the function calls in my own c/h files?

Correct


Concurrency fixes for junit-interface

This PR attempts to address both issues in #399 (in separate commits).
Regarding 2): I only removed multi-suite support from EventDispatcher and RichLogger, not thread-safety.
Some basic observation suggests that both classes are pinned to a thread, even with asynchronous tests, but I wasn't 100% certain that this is the case in all possible scenarios. Most of the possible simplification is already achieved by dropping multi-suite support, so maybe keeping thread-safety does not hurt.

The tests did not execute when I originally created the PR, I had to push a random change to actually trigger them. GitHub had issues earlier today, and apparently when checks don't execute during PR creation, they never will.


WPF Support

I'm trying to apply this sample on WPF. But the question is I cannot set a MediaStreamSource on the WPF MediaElement.  I read about the issues WPF Support #21 and there is a suggestion is that should be able register the MediaStreamSource using the Media Foundation extension so that it can be invoked when setting a source as a URI on the MediaElement.
but the link in that suggestion is dead now.
https://code.msdn.microsoft.com/windowsapps/MediaStreamSource-media-dfd55dff
Do anyone know how to convert MediaStreamSource to a URI in WPF?
Any suggestion will be help, thanks a lot.

I think I found the sample here:
https://github.com/microsoft/VCSamples/tree/master/VC2012Samples/Windows 8 samples/C%2B%2B/Windows 8 app samples/MediaStreamSource media extension sample (Windows 8)
The sample targets Windows 8 Apps, not WPF, but the same mechanism should be possible with WPF as well. However, this is really complex stuff, all written in native C++ code. Making it work with WPF, fully featured, with multiple streams and formats would be a very huge task, requiring deep knowledge of both MediaFoundation and native code. The sample only uses a single audio stream if I understand correctly. No video, no multi streams, no dynamic formats, no subtitles. Also, the sample probably won't compile on any recent Visual Studio version, because it targets a totally outdated Windows 8 App framework.
Sorry to say, but personally I think this approach is a dead end.
You might have more luck using the MediaPlayer class from Windows.Media.Playback with FFmpegInterop (or better FFmpegInteropX) and using it in frame server mode. This means manually receiving the decoded D3D video frames from MediaPlayer, and rendering them into your application, in your case probably using a WPF D3DImage as target. But I don't know if anyone has ever tried this. You'd probably have to get down a bit with native D3D or SharpDX, and it probably won't be easy. I know that some people have used the frame server approach successfully with FFmpegInteropX and Unity, rendering video files into their Unity 3D applications with great performance.

Frame server mode is the way to go. However, it might be tricky to get CC displayed correctly.

Thanks a lot for the information. It's seem there is no worthy to try this approach...
I will try the other ways, includied FFmpegInteropX or just play the video by native code.
Still, thanks again.


timeline not reflecting change after click

After clicking to change the year there is no visual feedback.

duplicate of #83


由于谷歌翻译已退出中国市场，请将 translate.googleapis.com 设为走代理

由于谷歌翻译已退出中国市场，请将 translate.googleapis.com 设为走代理
Since Google Translate has withdrawn from the Chinese market, please set translate.googleapis.com as a proxy.

这是一封自动回复邮件。已经收到您的来信，我会尽快回复。如有急事，请致电本人，谢谢！Thank you for your email! Please do not reply to this message, as this is an automated response. We will respond as soon as we can!


Changelog?

Where's your changelog?

https://github.com/zeit/release

A changelog would be very valuable to check for breaking changes when upgrading this package.


Game Map-HavenOrbital

Haven Orbital
About the PR
I've added the biggest map yet called Haven Orbital, a map that took me over 600 hours to create and a lot of sweat, tears and frustration, but I finally decided it's done and it's time to PR it again.
Haven Orbital is not just a station, it has a little story that you can see in the video below in the Media field.
Haven Orbital is a 65-80 player map, but I'm pretty sure it could easily hold 200+ players.
It has its own evacuation shuttle called Emergency Bean, also made by me and inspired by the Barratry Shuttle.

It has a zoo, a reporter's studio and a large green recreation area on the east side of the station.
Medical checkpoints to support paramedics help the injured
Interesting and thoughtful spaces of maintenance to discover

Since the last pull request I have implemented the feedback from the maintainers and co and changed a few things myself that I thought might cause complaints I am confident that all the major problems with the map have been fixed and it is ready to go into the map pool
Media
Lore Video
https://www.youtube.com/watch?v=PsucLGNMTIc&t
Full Map Render

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

:cl:MaloTheDuck

add: Your crew meets the requirements to inhabit Haven Orbital, the most luxurious station NT has built to date consider yourself lucky!

Haven my beloved

Absolutely gorgeous map, can't wait to explore it!

It seems like these trees might be misaligned, or is that just an artifact of the whole-map screen capture?

Absolutely gorgeous map, can't wait to explore it!
 It seems like these trees might be misaligned, or is that just an artifact of the whole-map screen capture?

Trees become offset on Map Renders they are the right way on the map tho thanks for the nice comment👍

Hmm yes this map looks like a station 10/10 pls merge

its a dumb station :3
-10/10 silly malo malo

Much big, Very effort.

Why did this suddenly get reviewed now

map needs serious layout changes/rebalancing to fit a specific population target. comment for me to reopen if you are gonna get around to it


[Urgent] Doesn't support the latest builds of React-Native and React

yarn add react-native
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>
after that
yarn add<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x".
warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>warning<EMAIL_ADDRESS>has incorrect peer dependency<EMAIL_ADDRESS>
lets say we can ignore and do not use enzyme, react-test-renderer, reactotron-react-native but react-dom is important.
Steps to reproduce it:
ignite new v3
cd v3
yarn add react-native
yarn add<EMAIL_ADDRESS>react-native run-android

The https://github.com/infinitered/ignite-ir-boilerplate boilerplate has been updated to 0.45.1.  Just waiting a release.
You can also control the version of react native by ignite new MyApp --react-native-version 0.45.1.

Hi @skellock, i have a related issue with Ignite v2.0.0.
I just created a new project with ir-boilerplate and have dependencies warnings.
Command: ignite new TestApp --min
Warning from npm cli:
npm ERR! peer dep missing<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x, required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>required by<EMAIL_ADDRESS>npm ERR! peer dep missing<EMAIL_ADDRESS>|| 16.0.0-alpha-6 || 16.0.0-alpha-12, required by<EMAIL_ADDRESS>
npm test results in 5 failed and 3 passed tests for react-dom dependencies (I think)
Here is my ignite doctor log:
System
  platform           darwin
  arch               x64
  cpu                2 cores                              Intel(R) Core(TM)2 Duo CPU     P7550  @ 2.26GHz
  directory          /Users/edolix/Dev/tests/v2/TestApp

JavaScript
  node               8.1.3        /usr/local/bin/node
  npm                5.0.3        /usr/local/bin/npm
  yarn               0.27.5       /usr/local/bin/yarn

React Native
  react-native-cli   2.0.1
  app rn version     0.45.1

Ignite
  ignite             2.0.0        /usr/local/bin/ignite

Android
  java               1.8.0_131    /usr/bin/java
  android home       -            /usr/local/share/android-sdk

iOS
  xcode              8.3.3

Am I doing something wrong?
Thanks

@edolix I can confirm this on a similar issue. On a new ignite 2.0.0 project I get following on first install:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 0.14.x || ^15.0.0-0 || 15.x but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>|| 16.0.0-alpha-6 || 16.0.0-alpha-12 but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

node: 8.1.2
npm: 4.6.1
yarn: 0.24.6
rn: 0.45.1
Ubuntu 16.04

Apparently the solution is to either downgrade react, or wait for RN 0.16 to become stable, as enzyme does not support alpha releases:
https://github.com/airbnb/enzyme/issues/928

@aschrijver I have downgraded React but the results are a lot of other dependencies error/warning. You have to downgrade almost "all" libraries

Yes, I am also in the process of downgrading. Not happy about it. Getting all kinds of issues.

@skellock Please reopen as the issue is still at play.

@edolix I found the issue with the tests. I had to add -u --no-cache to package.json:
"test": "NODE_ENV=test jest -u --no-cache"

Found it out by accident, while sorting out the nightmare of getting VS Code to hit my breakpoints while debugging. Still working on it..

There’s a lot of drift in this issue. For that reason I’m going to keep this closed.
Let me address what are legit issues tho.
So ya, enzyme is not working with the latest React Native. It is currently being rewritten and will be available soon though. This is frustrating to me because we were happy with how our testing was going, and now we can’t use it short term.  🙁
The warnings you see when installing are because the peer dependencies of those libs don’t match. In the case of reactotron-react-native, I typed the dependency version wrong (-16 vs .16). I will fix that once I get out of bed here. ☺ (harmless warning btw)
As far as red screens are concerned, I’d need to see the specific traces to help. One thing that snags me often when switching React Native versions is caches. watchman watch-del-all can help. rm -rf node_modules helps as well.
With VS Code debugging in Node 8, yes that’s a problem. Both the node and code teams are awesome and we should see a fix soon.
Addressing the horrible dev experience when all this happens, yes.  We (as a community) can do better.  One of my favourite talks at Chain React a few days ago was by Brent Vatne pointing this out. That we need to do better.  React Native can be frustrating to deal with fragile deps.  Even the panel talk pointed this out.
Good news is: we (the community at large) care.
Speaking more to Ignite though. When we get foot swept by our dependencies, we need to get back up faster. We can improve here for sure.
The enzyme breakage was particularly painful.

Thanks for your help and background info!
I understand your frustrations all too well, and I'll also try to contribute to the community by linking issues, answering (my own and others) stackoverflow questions, etc.
For myself I've been busy about a week now trying to set things up, and no productive line of code written yet. Now looking into debugging setup (2 days, and counting).
I thought downgrading node to 7.10.1 and using VS Code fixed things for me, but it runs directly from disk, not the js bundle, so my node shims are all ignored. This is vscode-specific :(
About adding the --no-cache to jest task in package.json. This does seem to work getting rid of the 'Unexpected token import' errors, at least when I run tests in VS Code.

@skellock Thanks for the support and info and thanks @aschrijver too!

Having the same issue


Try different keras backends

Depends on #220, with Keras 3.0, we should be able to use different DL backends. This could theoretically achieve multiple things:

Speed up (TF may not be the fastest)
Ease deployment (TF is not compatible with GPU on Windows, and isn't on conda-forge)
Futureproof

Closing as tracked in #229


support http header Accept-Language

could the plugin support Accept-Language and have appropriate Content-Language in the response ?
ie:
curl -i <IP_ADDRESS>:3333/hello -H 'Accept-Language: fr'

HTTP/1.1 200 OK
content-language: fr

salut

for info:
req.i18n.locale // "fr"
req.i18n.getLocale() // "en"

Did you have a look at https://github.com/ozum/hapi-locale? This plugin supports the extraction of the locale from the request header.


Blurred Title on non-journal pages

I'm using v0.4 and it looks great, but when I visit a page other than journal pages, the heading is blurred.  I don't have anything in custom.css.

Yep, can confirm had that by myself after the recent logseq update.
I pushed a 0.5 Update just now, it take some hours until the update is available in the marketplace, give it a try.