text:
What is the right way to close connection to server

I am seeing too many connections on the server. Most of the times when meteor is rebuilding the app, the old connections are staying there increasing the thread-count on the server and wasting resources. What is the right way to close these connections after client disconnect (either within code or through some configuration setting)?
I would like to have no more than 10 threads/connections idle and any other new threads created should wait for a max of 5 minutes and close the connection. How to achieve it? Right now, I have 133 threads below created and aborted by client

Never mind, found this online tool to generate my.cnf file automatically based on input https://tools.percona.com/wizard Sharing it here just so others can also benefit. Thank you.

Actually the configurations does not deal with closing the idle connections - perhaps we should figure a way of closing the connections once the publishing is done or on some time out.

Hmm this is an interesting issue that I had not thought about. When Meteor does a hot code push, refreshing the server, the application is not notified.
In order to not send duplicate result sets on server restarts, I had to add a DDP message to reset the client side results before sending the initial results. But this requires no knowledge of whether the server was restarting, it runs on all initializations.
I'm not sure of any way to tell from inside the application when Meteor decides to restart. It must be possibile somehow. I definitely agree that this is a major issue.

I have determined that this is not a bug but in fact, simply a matter of adding the necessary code to close connections on exit to your application.
On the server-side of your Meteor app add the following event handler:
var liveDb = new LiveMysql(Meteor.settings.mysql);

process.on('SIGTERM', function() {
  liveDb.end();
  process.exit();
});

I cannot add this code to the library directly because you can only effectively have one listener on the SIGTERM event per application. I will be adding this code the readme though.
Thanks for the issue, please let me know of any more you come across. :smile:

Wait a second...that doesn't quite work. Seems to be a bug with Zongji.

@KrishnaPG I've just made the pull request for this bug on Zongji. I will release a new version as soon as the updated zongji module is released.

Cool - thanks for addressing this @numtel . This certainly will improve overall usability of the package and certainly will save resources on shared hosting infrastructures.

BTW, I hope this fix in zongji, takes the scenario of multiple clients into consideration. I am not an expert on binlog or mysql - but just in case there are multiple meteor clients for the same server and one client goes away - hope this killing the binlog does not affect the other connected clients (who may still need that binlog for updates).
In other words, if mysql's binlog is in one-one relation with clients, the fix of killing it may be fine. Otherwise, if binlog is 1-to-many, then we may have to let it run till all other clients are done.

@KrishnaPG Don't worry, your clients will be safe :sunglasses:

@KrishnaPG Ok this is now fixed and I have updated the leaderboard example application to illustrate how to close connections on code push and exit (ctrl+c).
You will need to be updated to the latest version of this package, 0.1.3, for this to work.
https://github.com/numtel/meteor-mysql-leaderboard/blob/master/leaderboard.js#L73-80
  var closeAndExit = function() {
    liveDb.end();
    process.exit();
  };
  // Close connections on hot code push
  process.on('SIGTERM', closeAndExit);
  // Close connections on exit (ctrl + c)
  process.on('SIGINT', closeAndExit);

Cool - thanks. This looks good.

authors: KrishnaPG numtel

repo: numtel/meteor-mysql


text:
thumbnails not rendered when multilingual content in same folder

Describe the bug
When content in different languages is placed in the same folder, for one of the languages the thumbnails are not rendered.
In Blowfish documentation and samples it is assumed content for different languages is placed in different sub-folders. However, when most of the pages and  posts come translated in several languages, duplicated assets like feature* - files does not make much sense. Hugo itself allows separating languages by filename, say, index.en.md  - and it works fine with blowfish for blog posts which do have all languages.
However for posts or pages which for various reasons come only in one language - the post in other language is generated, but thumbnails are not.
To Reproduce
Disable contentDir in config, add a blog post for one langauge only, using language code in filename to distinguish between languages.
Expected behavior
Well, expect of course to see thumbnails for all languages for which index files exist.
Desktop (please complete the following information):

OS: Ubuntu 23.10
Browser Firefox
Version 124.0

(well files are not in public folder, so it clearly doesn't have to do with FF :)
Hugo & Blowfish versions
Hugo v0.124.0 , Blowfish  2.61.1

not 100% sure if it has to do with the theme though, or with Hugo itself...

Yep, it might be a Hugo issue
https://discourse.gohugo.io/t/issue-with-images-in-a-multilingual-multihost-configuration/48501

there's a group of us working on translating the site and we kind are doing that with no issues
https://github.com/nunocoracao/blowfish/pull/1324

Closing this one. If there is an issue it doesn't seem to be theme related. Also in the above PR there is a working preview of multiple languages with single templates.

source: gharchive/issue

authors: nunocoracao undertherain

repo: nunocoracao/blowfish

license: MIT

license_type: permissive

license_source: github-api


text:
Checking the fields for the valid values on tour creation

Check

title is not empty
description is not empty
image is selected
dates are selected

Do opening another widget to create a new route

repo: nurSaadat/RemoteGuide


text:
Exercise 4 Puzzle class

"Define a constructor Puzzle(int n) that takes in a integer value n and creates a (n^2 − 1) puzzle in a solution configuration."
My Puzzle class will take in 2 parameters, a Grid2D and an integer, n.  but they want to create the  (n^2 − 1) puzzle in a solution configuration. Are we supposed to use a loop to do so? are loops even allowed in constructors?

Hi, I am wondering the same thing, do you have the answer to this?

authors: ZhuLeYao zandratay


text:
[T4A3][W14-A2] Ellango Vesali

Ready for Review

Some comments added. Feel free to add more comment if you have any questions. Once you are done, you may close the PR after reading comments.

source: gharchive/pull-request

authors: mauris vesaliE


text:
[W6.1e][W13-A1]Raynold Ng

LO-JUnit

@raynoldng
Some comments added. Please ack & close the PR after reading comments.

authors: okkhoy raynoldng


text:
test(e2e): Add self-hosted e2e test

Creating as draft until kubernetes-sigs/cluster-api-addon-provider-helm#189 is merged and released.
Depends on #494.

To merge this, will need to update required status checks (make e2e-test (Docker) non-required and make the e2e-quick-start and e2e-self-hosted checks required) via Canaveral, merge it, and then all other open PRs will need to rebased to include this self-managed e2e.


text:
Add support for MongoDB provisioning

What this PR does / why we need it:

Adds changes in https://github.com/nutanix-cloud-native/ndb-operator/pull/74
Fix ndb_api_helper test that fails in the above PR

How Has This Been Tested?:

Implementation of mongo db provisioning has been tested as part of the PR https://github.com/nutanix-cloud-native/ndb-operator/pull/74
Failing tests have be fixed:

mongo db provisioning is getting initiated

postgres provisioning is successful

Changes look good, few comments -

Can you please update/cleanup the PR description to remove unused sections like "Which issue(s) this PR fixes", "Special Notes" and "Release Notes".
Can you also verify the provisioning of the existing DB engines - PostgreSQL?

Thanks!

Changes look good, few comments -

Can you please update/cleanup the PR description to remove unused sections like "Which issue(s) this PR fixes", "Special Notes" and "Release Notes".
Can you also verify the provisioning of the existing DB engines - PostgreSQL?

Thanks!

Addressed!

authors: manavrajvanshi shenoypritika

repo: nutanix-cloud-native/ndb-operator


text:
refactor: update module

Update module
Increate coverage 100% 😃
Update readme (badges, setup, options)
Add commitlint
Add husky
Add renovate
Resolve #5

ping @dohomi

@ricardogobbosouza sorry I havent been around for a while.. can you solve the conflicts then I will merge them.

@dohomi resolved
If you want you can help keep this module :smile:

@ricardogobbosouza sure just join the party :-)

authors: dohomi ricardogobbosouza

repo: nuxt-community/google-gtag


text:
Feature request: Custom route keys

Heyho! First of all: Thanks for marking this module 🙏
I'm currently building my companies page with "German" (de) as the default language. So far so good. The only thing that annoys me a bit is that the keys in the routes object should match the base routes from Nuxt. I think it'd be better to use them as custom/general keys.
Currently, my route file would look like this:
routes: {
    'about-us': {
      de: '/über-uns'
    },
    work: {
      de: '/referenzen'
    },
    'some-other-long-route-here': {}
   // ...
  },

It's a bit annoying because I can't keep the route keys short and often I need to "escape" them with quotes (because of the hyphens).
The best case for the routes file would be
routes: {
    about: {
      de: '/über-uns',
      en: '/about-us'
    },
    work: {
      de: '/referenzen',
      en: '/work'
    },
    whatever: {
      en: ' some-other-long-route-here'
    }
  },

Short and meaningful keys, and clear routing that doesn't depend on the actual components.
But I guess that's a lot of work because you'd need to find out the component for each custom route as well. I'm not sure if that's even possible (except you set it in the config)

This question is available on Nuxt.js community (#c5)

Using Nuxt's base routes feels like the safest thing to do for this feature to prevent any confusions.
Ideally, I'd like to have an option to configure custom routes right in the .vue files rather than putting them all in a config file but I'm lacking time to give it a try atm.

Closing this as paths customization has changed a bit since this issue was open.
https://nuxt-community.github.io/nuxt-i18n/routing.html#custom-paths

authors: manniL paulgv


text:
fix(docs): sticky toc on mobile

closes #1412

This does indeed fixes the issue, I was planning to fix this upstream in @nuxt/ui-pro but haven't figured it out yet so we can merge this in the meantime. Thanks! 😊

authors: arashsheyda benjamincanac


text:
path should be a path.relative()d string, but got "../../store/users.js"

Version
v2.13.2
Reproduction link
https://codesandbox.io/s/github/zabavnikov/nuxt-store
Steps to reproduce
Run codesandbox
What is expected ?
Nuxt will find store files
What is actually happening?
path should be a path.relative()d string, but got "../../store/users.js"
Additional comments?
Prior to version 2.13.0, such a path to the store worked.
I have a basic Nuxt configuration and there are two Nuxt configurations (which inherit from the basic), one for the mobile version, the other for the desktop.
The applications/mobile/nuxt.config.js file will merge with the main config, but an error appears immediately after the server starts - path should be a path.relative()d string, but got "../../store/users.js"

This bug report is available on Nuxt community (#c10868)

I'm having the same issue. I have two configurations and my application started presented this issue.

authors: hjardines zabavnikov


text:
how to host nuxt+koa  with multi instances by pm2?

I wanna to start nuxt+koa by pm2 with max instances.
case 1
restarting all  the time.
//pm2 start pm2.json
{
  "apps": [
    {
      "name": "software-measurement-platform-with-json",
      "script": "server/index.js",
      "instances": "1",
      "exec_mode": "cluster",
      "watch": false,
      "ignore_watch": ["node_modules", "lib/chromium", "test", "logs", "data", ".git", ".vscode"],
      "instance_var": "INSTANCE_ID",
      "env": {
        "DEBUG": "starlab:*",
        "NODE_ENV": "production"
      },
      "log_date_format": "YYYY-MM-DD HH:mm Z"
    }
  ]
}

comes with error:
0|software |
0|software | 2021-02-10 14:50 +08:00: [HPM] Proxy created: /api/  -> http://127.0.0.1:3010
0|software | 2021-02-10 14:50 +08:00: [HPM] Proxy rewrite rule created: "^/api/" ~> "/api"
0|software | 2021-02-10 14:50 +08:00: › baseURL: http://localhost:3000/api/v1
0|software | 2021-02-10 14:50 +08:00: › browserBaseURL: /api/v1
0|software | 2021-02-10 14:50 +08:00:
0|software |  READY  Server listening on http://localhost:3000
0|software |
0|software | 2021-02-10 14:50 +08:00: [HPM] Proxy created: /api/  -> http://127.0.0.1:3010
0|software | 2021-02-10 14:50 +08:00: [HPM] Proxy rewrite rule created: "^/api/" ~> "/api"
0|software | 2021-02-10 14:50 +08:00: › baseURL: http://localhost:3000/api/v1
0|software | 2021-02-10 14:50 +08:00: › browserBaseURL: /api/v1
0|software | 2021-02-10 14:50 +08:00:
0|software |  READY  Server listening on http://localhost:3000
0|software |

/root/.pm2/logs/software-measurement-platform-with-json-error-0.log last 15 lines:
0|software |     at processEmit [as emit] (/starlab/www/nodejs/software-measurement-platform/node_modules/signal-exit/index.js:161:32)
0|software |     at emit (internal/child_process.js:832:12)
0|software |     at process._tickCallback (internal/process/next_tick.js:63:19)
0|software | 2021-02-10 14:50 +08:00:
0|software |  WARN  mode option is deprecated. You can safely remove it from nuxt.config
0|software |
0|software | Error: bind EADDRINUSE null:3000
0|software |     at listenOnMasterHandle (net.js:1347:16)
0|software |     at rr (internal/cluster/child.js:126:12)
0|software |     at Worker.send (internal/cluster/child.js:93:7)
0|software |     at process.onInternalMessage (internal/cluster/utils.js:45:8)
0|software |     at process.emit (events.js:203:15)
0|software |     at processEmit [as emit] (/starlab/www/nodejs/software-measurement-platform/node_modules/signal-exit/index.js:161:32)
0|software |     at emit (internal/child_process.js:832:12)
0|software |     at process._tickCallback (internal/process/next_tick.js:63:19)

0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00:
0|software-measurement-platform-with-json  |  WARN  mode option is deprecated. You can safely remove it from nuxt.config
0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00: [HPM] Proxy created: /api/  -> http://127.0.0.1:3010
0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00: [HPM] Proxy rewrite rule created: "^/api/" ~> "/api"
0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00: › baseURL: http://localhost:3000/api/v1
0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00: › browserBaseURL: /api/v1
0|software-measurement-platform-with-json  | 2021-02-10 14:50 +08:00:
0|software-measurement-platform-with-json  |  READY  Server listening on http://localhost:3000
0|software-measurement-platform-with-json  | Error: bind EADDRINUSE null:3000
0|software-measurement-platform-with-json  |     at listenOnMasterHandle (net.js:1347:16)
0|software-measurement-platform-with-json  |     at rr (internal/cluster/child.js:126:12)
0|software-measurement-platform-with-json  |     at Worker.send (internal/cluster/child.js:93:7)
0|software-measurement-platform-with-json  |     at process.onInternalMessage (internal/cluster/utils.js:45:8)
0|software-measurement-platform-with-json  |     at process.emit (events.js:203:15)
0|software-measurement-platform-with-json  |     at processEmit [as emit] (/starlab/www/nodejs/software-measurement-platform/node_modules/signal-exit/index.js:161:32)
0|software-measurement-platform-with-json  |     at emit (internal/child_process.js:832:12)
0|software-measurement-platform-with-json  |     at process._tickCallback (internal/process/next_tick.js:63:19)
PM2                                        | App name:software-measurement-platform-with-json id:0 disconnected
PM2                                        | App [software-measurement-platform-with-json:0] exited with code [0] via signal [SIGINT]
PM2                                        | App [software-measurement-platform-with-json:0] starting in -cluster mode-
PM2                                        | App [software-measurement-platform-with-json:0] online

case 2
I saw "ecosystem.config.js" mentioned in doc:https://nuxtjs.org/docs/2.x/deployment/deployment-pm2
page shows correctly, error log shown in pm2 log, no more logs any more.
//pm2 start
module.exports = {
  apps: [
    {
      name: 'software-measurement-platform-with-eco',
      exec_mode: 'cluster',
      instances: 'max', // Or a number of instances
      script: 'npm', // './node_modules/nuxt/bin/nuxt.js',
      args: 'run start',
      env: {
        NODE_ENV: 'production'
        // HOST: '0.0.0.0',
        // PORT: 3010
      }
    }
  ]
}

only the first instance is correctly, others are with error below:
/root/.pm2/logs/software-measurement-platform-with-eco-error-1.log last 15 lines:
1|software | npm ERR! code ELIFECYCLE
1|software | npm ERR! errno 1
1|software | npm ERR! software-measurement-platform-service@1.0.0 start: `cross-env NODE_ENV=production node server/index.js`
1|software | npm ERR! Exit status 1
1|software | npm ERR!
1|software | npm ERR! Failed at the software-measurement-platform-service@1.0.0 start script.
1|software | npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

case 3
I saw "ecosystem.config.js" mentioned in doc:https://nuxtjs.org/docs/2.x/deployment/deployment-pm2
module.exports = {
  apps: [
    {
      name: 'software-measurement-platform',
      exec_mode: 'cluster',
      instances: '1', // Or a number of instances
      script: 'npm', // './node_modules/nuxt/bin/nuxt.js',
      args: 'run start',
      env: {
        NODE_ENV: 'production'
        // HOST: '0.0.0.0',
        // PORT: 3010
      }
    }
  ]
}

service is ok, pm2 log shows in /root/.pm2/pm2.log correctly.

@bi-kai I appreciate you replying with more information. Could you provide a minimal reproduction - ie all the code needed to replicate these issues, ideally in a github repo?

case 4
pm2 start npm --name "my-nuxt" -- run start

Start 1 instance with fork mode, logs shows in pm2 log, not in /root/.pm2/pm2.log.
pm2 start npm --name "my-nuxt" --cwd /starlab/www/nodejs/software-measurement-platform/ -i max -- run start

comes error:
READY  Server listening on http://127.0.0.1:3010

events.js:174
      throw er; // Unhandled 'error' event
      ^

Error: listen EADDRINUSE: address already in use :::3010
    at Server.setupListenHandle [as _listen2] (net.js:1280:14)
    at listenInCluster (net.js:1328:12)
    at Server.listen (net.js:1415:7)
    at Application.listen (/starlab/www/nodejs/software-measurement-platform/node_modules/koa/lib/application.js:82:19)
    at start (/starlab/www/nodejs/software-measurement-platform/server/index.js:172:7)
Emitted 'error' event at:
    at emitErrorNT (net.js:1307:8)
    at process._tickCallback (internal/process/next_tick.js:63:19)

@bi-kai Is there any chance you are hard coding the port in your koa server entrypoint?

@danielroe no, I am using the default code generated by nuxt-app.

@bi-kai create-nuxt-app no longer generates any custom server presets. So you are likely using an old version.
It would be difficult to help with this unless you provide a full minimal reproduction. However, from looking at your error messages I would suggest a misconfiguration as it seems to be starting all your instances with the same port.
You may also find joining the Nuxt Discord to be helpful.

@danielroe yes, I am confused with the latest nuxt, I need koa as a server, so I selected nuxt@2.0.0.

@danielroe Is there any official doc could guide me, how to use koa with the latest nuxt?

The summary is as bellow:
pm2 start ecosystem.config.js # 1 个实例，日志输出到/root/.pm2/pm2.log
pm2 start npm --name "software-measurement-platform-with-cmd" -- run start # 1 个实例，日志输出到控制台，不会记录到 pm2 日志文件

authors: bi-kai danielroe


text:
Nuxt pushes intial route twice to browser history

Versions

nuxt: v2.14.0/v2.15.0
node: v14 LTS

Reproduction
Hitting the browser back button does not work if you just navigated to your nuxt application.
Steps to reproduce

Start your project
Navigate to it on the browser
Refresh the page
Hit back Button

What is Expected?
You get back to the last page.
What is actually happening?
Nothing. You have to click again.
Related
https://github.com/nuxt/nuxt.js/issues/9111

closing in favour of #9111

authors: danielroe medokin


text:
Security update postcss-url

#2074

Thanks!

Codecov Report

Merging #2155 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2155   +/-   ##
=======================================
  Coverage   96.48%   96.48%
=======================================
  Files          15       15
  Lines         968      968
=======================================
  Hits          934      934
  Misses         34       34

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 750db84...3188100. Read the comment docs.

authors: codecov-io pi0 yashha


text:
types: add cross-origin property to NuxtOptionsRender

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

I found a missing property crossorigin in NuxtOptionsRender Type. This PR fixes it.
According to the MDN page, this property is enumerated.
Now, this property is in the official document and available.
Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly. (PR: #)
[ ] I have added tests to cover my changes (if not applicable, please state why)
[ ] All new and existing tests are passing.

Thanks!


text:
No documentation on how to unit test a simple component with data prop

Environment
nuxt latest
node 22
happy-dom
jest
Reproduction
Hard to reproduce something that isn't there but I can at least show you what I am trying to do, the error I get for context.
import { expect, test } from "vitest";
import { mountSuspended } from "@nuxt/test-utils/runtime";
import ToDo from "~/components/ToDo.vue";

it('can mount todo component', async () => {
    const todo = await mountSuspended(ToDo, {
        data() {
            return {
                completed: true,
                title: "Clean Walrus"
            }
        }
    })
    expect(todo.html()).toEqual("<></>")
})

TypeError: Cannot read properties of undefined (reading 'completed')
 ❯ Proxy._sfc_render components/ToDo.vue:2:28
      1| <template>
      2|  <div :class="`task ${task.completed ? 'is-complete' : ''}`">
       |                            ^
      3|
{{ task.title }}

      4|

 ❯ Proxy.clonedComponent.render node_modules/@nuxt/test-utils/dist/runtime-utils/index.mjs:142:37
 ❯ renderComponentRoot node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:6246:16
 ❯ ReactiveEffect.componentUpdateFn [as fn] node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:4921:46
 ❯ ReactiveEffect.run node_modules/@vue/reactivity/dist/reactivity.cjs.js:181:19
 ❯ instance.update node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:5052:16
 ❯ setupRenderEffect node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:5062:5
 ❯ mountComponent node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:4830:7
 ❯ processComponent node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:4784:9
 ❯ patch node_modules/@vue/runtime-core/dist/runtime-core.cjs.js:4313:11

This error originated in "tests/components/ToDo.vue.test.js" test file. It doesn't mean the error was thrown inside the file itself, but while it was running.
The latest test that might've caused the error is "can mount todo component". It might mean one of the following:
- The error was thrown, while Vitest was running this test.
- If the error occurred after the test had been completed, this was the last documented test before it was thrown.

I have tried various things along this line of thought from various scraps of information I can find on blogs and stack overflow but at least 50% of it is for nuxt2 and out of date.
Describe the bug
I would expect a mature framework to have better unit test examples and especially a simple unit test of a simple view component with a data prop
Additional context
No response
Logs
No response

Latest attempt again from scraps of info that doesn't work
import { expect, test } from "vitest";
import { mountSuspended } from "@nuxt/test-utils/runtime";
import ToDo from "~/components/ToDo.vue";

it('can mount todo component', async () => {
    const todo = await mountSuspended(ToDo, {
        propsData:
            {
                task: {
                    completed: true,
                    title: "Clean Walrus"
                }
            }
        }
    )
    expect(todo.html()).toEqual("<></>")
})

For the unit under test
<template>
	<div :class="`task ${task.completed ? 'is-complete' : ''}`">

{{ task.title }}

			<button @click="toggleDone">{{ task.done ? 'Undo' : 'Done' }}</button>
			<button @click="removeTask" class="delete">Delete</button>

</template>

@suityou01 Check out documentation for @vue/test-utils: https://test-utils.vuejs.org/guide/essentials/passing-data.html#Using-props-to-set-a-minimum-length

Hello, without a minimal reproduction, we can't do much to help you.
@nuxt/test-utils is using @vue/test-utils under the hood.

I included my code above so you do have a minimal reproduction already.

@suityou01 Check out documentation for @vue/test-utils: https://test-utils.vuejs.org/guide/essentials/passing-data.html#Using-props-to-set-a-minimum-length

This is exactly what I needed, but in the nuxt documentation. If the nuxt documentation considers this is documented in the vue documentation then a short explainer with the above link would be sufficient to take out the guesswork for people new to vue/nuxt.

Minimal reproductions are needed so people can play with your reproduction and provide the best answer/solution for you.
https://nuxt.com/docs/community/reporting-bugs explains how to contribute by creating issues. The GH-actions bot also gives you some informations about why we need and how to create a minimal repro.

How does one create a minimal reproduction for gaps in documentation?

In general, it's helpful to provide runnable code for anything that's not working. It makes it much easier to help.
Definitely makes sense to improve docs to link out to the relevant docs, and I've opened a PR to do that. Feel free to open a PR to improve further if you see any opportunity to do so.
Best of luck on your unit testing journey!

repo: nuxt/nuxt


text:
fix(nuxt): if href in nuxt-link starts with hash, render a tag directly

🔗 Linked issue
resolves #30074
📚 Description
When a NuxtLink component's href starts with # or ?, the rendered  tag's href includes the current route's path prepended to it. This issue only occurs when the component is located within the pages folder.
To address this issue, I followed the suggestion from the comments section of the open issue. I have implemented a solution specifically for cases where the href starts with '#'. In such cases, the component will render an  tag directly instead of using RouterLink.
Currently, this PR focuses on fixing the issue with #. I am open to feedback and to extend the solution to handle cases where the href starts with '?', provided there's guidance on the best approach for tackling that problem.
Additional note
This is my first PR here, so while I’ve done my best to follow the contribution guidelines, I welcome any suggestions for improvement.

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Enhanced handling of hash links without hash mode in the NuxtLink component.
Improved link rendering logic to accommodate specific link types.

Bug Fixes

Corrected trailing slash behavior based on user settings.

Tests

Added test cases for identifying external links, including those starting with a hash (#).

Documentation

Updated comments and documentation for clarity on new functionalities.

@danielroe  I've added test cases for that and made the changes you suggested. Just let me know if they need any changes, and I'll update them

(no need to merge in the latest main branch unless there are conflicts)

@danielroe I've made several changes to resolve the issues that were causing tests to fail.

I wonder if this would cause any unexpected gotcha's with transitions between links. But as all links where just the hash changes should only move to certain anchors on the same page, this seems fine to me.
For query parameters, proper handling might be needed (but happy to be proven wrong otherwise).

thank you very much @vmrjnvc ❤

authors: TheAlexLichter danielroe vmrjnvc


text:
fix(Pagination): correct displayedPages()

🔗 Linked issue
#723
❓ Type of change

[ ] 📖 Documentation (updates to the documentation or readme)
[X] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Before :

After :

Resolves #723

Another use case is when the maximum displayed pages are equal to or greater than the total pages.

Before :

After :

Hi @Haythamasalama !
Thanks for the PR! Indeed there is no correct handling of max < 5. This is due to the fact that max is not intended to be set below 5 as it technically has no sens, based on its initial purpose. Let me explain what was thought in the first place 🙂
max has the goal to keep a max number of buttons (not only "pages" buttons but also "ellipsis" buttons), to keep consistency in its integration/display.
See how the number of buttons don't change on production:

And see how the changes affect this UX:

I'm fully open to rewrite that algorithm, I'm not really satisfied of what I've written initially and of how maintainable it is. But it should keep the initial behaviour, to allow some users to restrict the Pagination width and have a consistent UI.
Also about your PR, it looks good when page 1 is selected, but not fine when moving to page 2/3 in this case:

If you can fix that, we are good to go! 🚀

Thanks ❤ @smarroufin for explaining the Max props. The first time, I thought max was only for buttons without dividers
However, after the explanation of what max is, I agree that it doesn't make sense for it to be max > 5 but we still need to handle cases when max > 5 as 5 and when max is equal to the total number of pages.
So, I made a refactor for it using the same algorithm to handle those cases.

@smarroufin
I agree that we need to set the maximum value on the validator to 5, but should we remove the number from here?

 const maxDisplayedPages = Math.max(props.max, 5)

Do you mean 5 as minimum value? Then yes. From 5 to anything.
And I guess your fallback is to be kept anyways, otherwise we would end up with the issue we currently have, meaning that the docs number select can "disrespect" the validator by giving an unappropriated number.

Sure, I've updated to 5.

All good for me, thanks a lot @Haythamasalama!

You're welcome, @smarroufin ❤. Thank you for reviewing it.

authors: Haythamasalama smarroufin

repo: nuxt/ui


text:
Rename master branch

It was hard to explain why I had a folder on my computer called "the fuck master"  to my girlfriend

2333

2333333333333333333333333

authors: JixunMoe TusharaFernando willin

repo: nvbn/thefuck

license: mit

license_source: bigquery


text:
Added quotation_marks rule

Added rule for fixing uneven ' and " usage

Thanks!

authors: diezcami nvbn


text:
Android AutoSuggestBox Suggestion Stops Working

Android AutoSuggestBox Suggestion Stops Working
Current behavior
private void AutoSuggestBox_TextChanged(AutoSuggestBox sender, AutoSuggestBoxTextChangedEventArgs args)
{
if (args.Reason == AutoSuggestionBoxTextChangeReason.UserInput)
{
Suggestions.Clear();
Suggestions.Add(sender.Text + "1");
Suggestions.Add(sender.Text + "2");
}
Control2.ItemsSource = Suggestions;
}
Only in Android, this code hang  adding new suggestions, first Keypress Work, Second KeyPress hang the app
AutoSugges.zip
Expected behavior

How to reproduce it (as minimally and precisely as possible)

Environment

Nuget Package:
"Uno.UI" Version="1.45.0-dev.1707"
"Uno.Wasm.Bootstrap" Version="1.0.0-dev.281"
Package Version(s):
Affected platform(s):

[ ] iOS
[X ] Android
[ ] WebAssembly
[ ] Windows
[ ] Build tasks

Visual Studio

[ ] 2017 (version: )
[X ] 2019 (version: 16.1 )
[ ] for Mac (version: )

Relevant plugins

[ ] Resharper (version: )

Anything else we need to know?

A workaround can be to copy the following template:
https://github.com/nventive/Uno/blob/94f551bf21793c87fff1f88173ac1ad488fd8043/src/Uno.UI/UI/Xaml/Style/Generic/Generic.xaml#L8309
But with a non-virtualizing panel for the list view in the popup:
<ListView.ItemsPanel>
  <ItemsPanelTemplate>
   <StackPanel />
  </ItemsPanelTemplate>
</ListView.ItemsPanel>

This will prevent the app from freezing (into a recycling loop, @davidjohnoliver that may interest you), but the popup is stealing the focus, so you have to tap the textbox after the searchbox appears.

authors: iosub jeromelaban

repo: nventive/Uno


text:
🛑 NVIDIA AI Enterprise Hub is down

In b20c82b, NVIDIA AI Enterprise Hub (https://nvaie.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NVIDIA AI Enterprise Hub is back up in c54b4fd.

repo: nvgsg/lab-upptime


text:
🛑 PufferAI HPC Portal is down

In a7c8064, PufferAI HPC Portal (https://hpc.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PufferAI HPC Portal is back up in 8cf20c9.


text:
🛑 NVIDIA AI Enterprise Hub is down

In 7dcc40c, NVIDIA AI Enterprise Hub (https://nvaie.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NVIDIA AI Enterprise Hub is back up in 3168d03.


text:
🛑 NVIDIA AI Enterprise Hub is down

In 652aab7, NVIDIA AI Enterprise Hub (https://nvaie.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NVIDIA AI Enterprise Hub is back up in 877c007.


text:
🛑 NVIDIA AI Enterprise Hub is down

In 474de1e, NVIDIA AI Enterprise Hub (https://nvaie.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NVIDIA AI Enterprise Hub is back up in e116e58.


text:
🛑 NVIDIA AI Enterprise Hub is down

In f7393f5, NVIDIA AI Enterprise Hub (https://nvaie.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NVIDIA AI Enterprise Hub is back up in 54f421f.


text:
🛑 PufferAI HPC Portal is down

In b2175fb, PufferAI HPC Portal (https://hpc.lab.novaglobal.com.sg) was down:

HTTP code: 0
Response time: 0 ms

Resolved: PufferAI HPC Portal is back up in 091c671.


text:
fix(source_selector): fix bug with tabs_layout

Fixes #848.
The function text_layout doesn't work with the whole tab representation, which relies on vim.fn.strdisplaywidth. It seems that this function considers the tab representation string literally, and gives a length of more than 400 for just 3 tabs. The text_layout then truncates the tab representation, which results in weird values.
To fix this, I moved the text_layout logic to truncate tab.text instead.

Actually I realized that I made the wrong implementation but haven't got time to redo it, so this shouldn't be merged yet.😓

Actually I realized that I made the wrong implementation but haven't got time to redo it, so this shouldn't be merged yet.

Can you mark this as a "draft" PR to make it clearer that it's not ready?

@cseickel This should be ready for review.

authors: cseickel nhat-vo


text:
Enable to change icon width

Neovim now has setcellwidths() (doc) and we can set arbitrary codepoint to 1 or 2 widths despite of the 'ambiwidth' setting.
I set devicon's glyphs can have 2 widths and found all icons become … characters. This patch enables to change this for such cases.

before
after

I set devicon's glyphs can have 2 widths and found all icons become … characters. This patch enables to change this for such cases.

Do you set that explicitly? I recall that width of glyphs may be terminal emulator dependent (I don't find it super fast, but kitty changes width depending on suffix space of glyph IIRC). Apologies I'm also uninformed on ambiwidth and setcellwidth because haven't used them and not clear to me what value they add for devicons.
I've also checked upstream telescope and there we concatenate the filename and devicon into one string. I suppose this wasn't done in the original file-browser code I carved the extension out of due to the folder icon. See
https://github.com/nvim-telescope/telescope.nvim/blob/28dc08f614f45d37ad90f170935f1f4e12559aeb/lua/telescope/utils.lua#L435-L463
All in all, I feel this should not be configurable and be handled by telescope-file-browser internally just like telescope, but for that we'd kind of need a custom variant of the above linked function, if I see that correctly.
Would you be up to make a PR for that?

Do you set that explicitly? I recall that width of glyphs may be terminal emulator dependent

Yes, I call setcellwidths() to set the widths of icons. Now Neovim in HEAD determines widths of East Asian WIdth Class Ambigous characters with process below.

If it find an entry from setcellwidths(), use it.
Use 1 or 2 width from the 'ambiwidth' setting.

Surely many terminals (including kitty) have settings for this, but the process above takes priority over them.
Many “Ambiguous” characters, such as box-drawing character, should have single width to draw a valid UI with plugins (including telescope). But I want to draw devicons' emoji characters as double widths in such case.

I have checked the implementation and the usage of utils.get_devicons and utils.transform_devicons.

utils.get_devicons

This returns an icon and its highlight.
This is used here and it uses strings.strdisplaywidth to determine the width.
This is for showing rows containing several items with an icon.

utils.transform_devicons

This returns a string including an icon and a filename, and a highlight of the icon.
This func supposes that all icons have single width.
This is used here and here. They also suppose icons have single width.
This is for showing rows containing an icon and a filename only.

For telescope-file-browser, it should show rows containing an icon, a filename and some stats. So we should use utils.get_devicons, I think.
I have reverted commits and add one to use strings.strdisplaywidth to determine the width. I will rebase if you want only the latest commit for this PR.
(By the way, utils.transform_devicons should be fixed to use strings.strdisplaywidth. I will PR to telescope later, maybe.)


text:
Keymap issues after fix for #232

Describe the bug
Two keymap issues after that commit is applied:

Visual maps no longer work (for example, can change around functions with caf, but can no longer select the function with vaf)
Running doautocmd FileType <ft> triggers "No such mapping" (just like in #169)

To Reproduce
Try to execute a visual mapping.
Expected behavior
Object is seleted.
Output of :checkhealth nvim_treesitter

nvim-treesitter: require("nvim-treesitter.health").check()
========================================================================
## Installation
  - OK: `tree-sitter` found 0.20.6 (parser generator, only needed for :TSInstallFromGrammar)
  - OK: `node` found v18.7.0 (only needed for :TSInstallFromGrammar)
  - OK: `git` executable found.
  - OK: `cc` executable found. Selected from { vim.NIL, "cc", "gcc", "clang", "cl", "zig" }
    Version: Apple clang version 13.1.6 (clang-1316.0.21.2.5)
  - OK: Neovim was compiled with tree-sitter runtime ABI version 14 (required >=13). Parsers must be compatible with runtime ABI.
Parser/Features H L F I J

rust           ✓ ✓ ✓ ✓ ✓
nix            ✓ ✓ ✓ . ✓
todotxt        ✓ . . . .
ql             ✓ ✓ . ✓ ✓
heex           ✓ ✓ ✓ ✓ ✓
scheme         ✓ . ✓ . ✓
bibtex         ✓ . ✓ ✓ .
kotlin         ✓ ✓ ✓ . ✓
ocaml          ✓ ✓ ✓ . ✓
gitignore      ✓ . . . .
yang           ✓ . ✓ ✓ .
scss           ✓ . . ✓ .
zig            ✓ . ✓ ✓ ✓
julia          ✓ ✓ ✓ ✓ ✓
pioasm         ✓ . . . ✓
jsonc          ✓ ✓ ✓ ✓ ✓
rego           ✓ . . . ✓
tiger          ✓ ✓ ✓ ✓ ✓
markdown_inline✓ . . . ✓
fusion         ✓ ✓ ✓ ✓ .
php            ✓ ✓ ✓ ✓ ✓
tlaplus        ✓ ✓ ✓ . ✓
m68k           ✓ ✓ ✓ . ✓
proto          ✓ . ✓ . .
http           ✓ . . . ✓
rnoweb         ✓ . ✓ . ✓
teal           ✓ ✓ ✓ ✓ ✓
glsl           ✓ ✓ ✓ ✓ ✓
gleam          ✓ ✓ ✓ ✓ ✓
elm            ✓ . . . ✓
regex          ✓ . . . .
query          ✓ ✓ ✓ ✓ ✓
cmake          ✓ . ✓ . .
rasi           ✓ ✓ ✓ ✓ .
hjson          ✓ ✓ ✓ ✓ ✓
vala           ✓ . ✓ . .
gdscript       ✓ ✓ . ✓ ✓
norg           . . . . .
hack           ✓ . . . .
foam           ✓ ✓ ✓ ✓ ✓
sql            ✓ . . . ✓
solidity       ✓ . . . .
graphql        ✓ . . ✓ ✓
ninja          ✓ . ✓ ✓ .
devicetree     ✓ ✓ ✓ ✓ ✓
llvm           ✓ . . . .
hocon          ✓ . . . ✓
lalrpop        ✓ ✓ . . .
json5          ✓ . . . ✓
help           ✓ . . . .
meson          ✓ . ✓ . ✓
vim            ✓ ✓ ✓ . ✓
cuda           ✓ ✓ ✓ ✓ ✓
cooklang       ✓ . . . .
commonlisp     ✓ ✓ ✓ . .
c              ✓ ✓ ✓ ✓ ✓
haskell        ✓ . ✓ . ✓
fortran        ✓ . ✓ ✓ .
latex          ✓ . ✓ . ✓
clojure        ✓ ✓ ✓ . ✓
wgsl           ✓ . ✓ . .
sparql         ✓ ✓ ✓ ✓ ✓
agda           ✓ . ✓ . .
hlsl           ✓ ✓ ✓ ✓ ✓
r              ✓ ✓ . ✓ ✓
svelte         ✓ . ✓ ✓ ✓
prisma         ✓ . . . .
toml           ✓ ✓ ✓ ✓ ✓
scala          ✓ . ✓ . ✓
qmljs          ✓ . ✓ . .
slint          ✓ . . ✓ .
json           ✓ ✓ ✓ ✓ .
java           ✓ ✓ . ✓ ✓
racket         ✓ . ✓ . ✓
beancount      ✓ . ✓ . .
jsdoc          ✓ . . . .
css            ✓ . ✓ ✓ ✓
fennel         ✓ ✓ . . ✓
javascript     ✓ ✓ ✓ ✓ ✓
html           ✓ ✓ ✓ ✓ ✓
comment        ✓ . . . .
yaml           ✓ ✓ ✓ ✓ ✓
surface        ✓ . ✓ ✓ ✓
embedded_template✓ . . . ✓
swift          ✓ ✓ . . .
pascal         ✓ ✓ ✓ ✓ ✓
lua            ✓ ✓ ✓ ✓ ✓
verilog        ✓ ✓ ✓ . ✓
gomod          ✓ . . . ✓
dot            ✓ . . . ✓
astro          ✓ ✓ ✓ ✓ ✓
tsx            ✓ ✓ ✓ ✓ ✓
supercollider  ✓ ✓ ✓ ✓ ✓
dart           ✓ ✓ . ✓ ✓
d              ✓ . ✓ ✓ ✓
org            . . . . .
python         ✓ ✓ ✓ ✓ ✓
glimmer        ✓ . . . .
make           ✓ . . . ✓
elvish         ✓ . . . ✓
gowork         ✓ . . . ✓
ruby           ✓ ✓ ✓ ✓ ✓
vue            ✓ . ✓ ✓ ✓
dockerfile     ✓ . . . ✓
v              ✓ ✓ ✓ ✓ ✓
elixir         ✓ ✓ ✓ ✓ ✓
eex            ✓ . . . ✓
go             ✓ ✓ ✓ ✓ ✓
typescript     ✓ ✓ ✓ ✓ ✓
c_sharp        ✓ ✓ ✓ . ✓
erlang         ✓ . ✓ . .
rst            ✓ ✓ . . ✓
godot_resource ✓ ✓ ✓ . .
cpp            ✓ ✓ ✓ ✓ ✓
ledger         ✓ . ✓ ✓ ✓
ocaml_interface✓ ✓ ✓ . ✓
turtle         ✓ ✓ ✓ ✓ ✓
bash           ✓ ✓ ✓ . ✓
ocamllex       ✓ . . . ✓
perl           ✓ . ✓ . .
hcl            ✓ . ✓ ✓ ✓
pug            ✓ . . . ✓
fish           ✓ ✓ ✓ ✓ ✓

Legend: H[ighlight], L[ocals], F[olds], I[ndents], In[j]ections
+) multiple parsers found, only one will be used
x) errors found in the query, try to run :TSUpdate {lang}

Output of nvim --version
NVIM v0.8.0-dev-1005-g6484c5687
Build type: RelWithDebInfo
LuaJIT 2.1.0-beta3
Compilation: /Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin/cc -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_TS_HAS_SET_MATCH_LIMIT -DNVIM_TS_HAS_SET_ALLOCATOR -O2 -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wdouble-promotion -Wmissing-noreturn -Wmissing-format-attribute -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=3 -I/Users/fsouza/Projects/os/3/neovim/build/cmake.config -I/Users/fsouza/Projects/os/3/neovim/src -I/Users/fsouza/Projects/os/3/neovim/.deps/usr/include -I/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk/usr/include -I/opt/homebrew/opt/gettext/include -I/Users/fsouza/Projects/os/3/neovim/build/src/nvim/auto -I/Users/fsouza/Projects/os/3/neovim/build/include
Compiled by fsouza@xikinbookpro.local

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/Users/fsouza/opt/neovim/share/nvim"

Run :checkhealth for more info

Additional context
Pinning nvim-treesitter-textobjects to https://github.com/nvim-treesitter/nvim-treesitter-textobjects/commit/775c5dbcb6937954d5861465e7c3ec23b855af93 "fixes" the issue for me.

@fsouza I noticed something similar, I think #272 should fix this

@theHamsta I still have the same issue after #272. Visual doesn't work, and also module textobjects.move doesn't work at all. The keybindings I had set in the config don't get recognized at all.

authors: AckslD fsouza marko-cerovac


text:
regression(test): diagnostics handler handler changedtick tracking should call handler only once if buffer changed in between callbacks

Fail	||	diagnostics handler handler changedtick tracking should call handler only once if buffer changed in between callbacks
            test/spec/diagnostics_spec.lua:126: Expected to be called 1 time(s), but was called 2 time(s)

            stack traceback:
            	test/spec/diagnostics_spec.lua:126: in function <test/spec/diagnostics_spec.lua:116>

Minimal repro:
make test-file FILE=test/spec/diagnostics_spec.lua

Seems a plenary / nvim update broke the async generator

addressed by commit 16bdde6a30b9611711e131e593a8fd418c429b09 for now

bisected to this commit: https://github.com/nvim-lua/plenary.nvim/commit/dd5a2f6d677f9c31dc55f0ab1ec2e9e43c408052

license: Unlicense


text:
typo: In README.md regarding base docker image

In Dockerfile, it is using Ubuntu 16.04 as base image but in README.md it mentions that it is using Ubuntu 14.04 as base image

Typo Link

repo: nvm-sh/nvm


text:
Add dupReadCounts info to getStatistics() output

Hello,
It would be nice to add a field dupReadCounts (or similar name) in the object from getStatistics() to have the number of marked reads duplicates that we have in the input bam. It would be very useful.
Thank you in advance.
Best regards.

Thank you for the feature request! I'll definitely look into this!

authors: gloubsi nvnieuwk

repo: nvnieuwk/nft-bam


text:
Add link to vimmer-ps Vim plugin

This PR mentions my vimmer-ps plugin which configures LanguageClient_neovim to use this language server.

Great, thanks!

authors: nwolverson sriharshachilakapati

repo: nwolverson/purescript-language-server


text:
🛑 Harmony Bot Website is down

In 0fb8b22, Harmony Bot Website ($HARMONY_WEB) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Harmony Bot Website is back up in 39aee5d.

repo: nxvvvv/uptime


text:
🛑 Harmony Bot Website is down

In 7771229, Harmony Bot Website ($HARMONY_WEB) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Harmony Bot Website is back up in 9344c1f.


text:
Twigpack::$plugin->manifest->getFileFromManifest() equivalent

Question
I have an endpoint with a static url (e.g.: domain.com/api/v1/script.js), which serves the contents of one file from the manifest (e.g.: script.2040123.js).
With Twigpack I used the following code for this:
    public function actionJs()
    {
        Craft::$app->response->format = Response::FORMAT_RAW;
        Craft::$app->response->headers->set('Content-Type', 'application/javascript');

        return Twigpack::$plugin->manifest->getFileFromManifest('shoprenter.js.map', 'modern');
    }

What is the equivalent of this with Vite?

You're trying to return the .map file?

Yes, and also the js file.
A little explanation: I have a SaaS site, where one of the generated files can be embedded/invoked by a third party website. So Vite bundles the JS & sourcemap files, but the path of it is constantly changing, so I serve the content of these files on an endpoint.
Thanks, I will check it out right away.

@khalwat
Getting unknown property: nystudio107\vite\Vite::viteService


text:
Make hashbash /bin/sh

Fixes #82

And fixes #90

Yeah, I'm aware of the issue, it's because they changed the base image of Alpine to no longer include bash

authors: dGilli khalwat timkelty


text:
Is master stable enough for use in a workshop?

First of all, thank you for NightCode! We used it successfully for the first ClojureBridge Berlin workshop last July, and will use it again for our second workshop this weekend. The ease of installation and the minimal, beginner friendly interface are simply perfect for our use.
Would you recommend sticking to the latest stable release (0.4.7), or could we just as well build an uberjar from master? Is there anything in master that would be helpful, vs are there things that might not be 100% stable?

That's great to hear you found it useful for your workshop. The latest
stable release is 1.0.0 which I released a few days ago.
On Jan 21, 2016 5:23 PM, "Arne Brasseur" notifications@github.com wrote:

First of all, thank you for NightCode! We used it successfully for the
first ClojureBridge Berlin workshop last July, and will use it again for
our second workshop this weekend. The ease of installation and the minimal,
beginner friendly interface are simply perfect for our use.
Would you recommend sticking to the latest stable release (0.4.7), or
could we just as well build an uberjar from master? Is there anything in
master that would be helpful, vs are there things that might not be 100%
stable?
—
Reply to this email directly or view it on GitHub
https://github.com/oakes/Nightcode/issues/145.

:100:

authors: oakes plexus

repo: oakes/Nightcode

license: unlicense


text:
ENG-489 feat: handle preview errors

Description

I'd wanted this work to result in the user (whoever is editing content on CMS) to be shown nicely displayed errors telling them what's wrong
However, this wasn't possible, so this simply stops such errors being reported to BugSnag
@mcky it turns out that the errors we were getting in BugSnag were valid errors (i.e. schema in OWA did't match config in Sanity). So maybe it's best to not merge this and instead to actually treat BugSnag errors from PreviewMode seriously?

How to test

locally you can change a schema and check a preview version and see that it doesn't attempt to report

Superceded by 2050

authors: kimon-satan mantagen

repo: oaknational/Oak-Web-Application


text:
[GLUTEN-2894][CH] Enable output rows metric for native write

What changes were proposed in this pull request?
(Fixes: #2894
How was this patch tested?
manual tests

===== Performance report for TPCH SF2000 with Velox backend, for reference only ====

query
log/native_3061_time.csv
log/native_master_09_11_2023_c93fea040_time.csv
difference
percentage

q1
44.36
44.05
-0.311
99.30%

q2
24.70
24.42
-0.286
98.84%

q3
37.30
37.16
-0.139
99.63%

q4
41.32
41.39
0.065
100.16%

q5
69.51
69.52
0.009
100.01%

q6
6.65
6.57
-0.081
98.78%

q7
83.05
83.17
0.112
100.13%

q8
78.18
80.30
2.124
102.72%

q9
115.25
117.13
1.882
101.63%

q10
47.18
47.78
0.601
101.27%

q11
19.49
21.01
1.520
107.80%

q12
24.83
26.08
1.252
105.04%

q13
50.98
50.52
-0.460
99.10%

q14
14.99
13.64
-1.351
90.99%

q15
29.53
29.03
-0.505
98.29%

q16
16.13
16.10
-0.026
99.84%

q17
120.03
119.95
-0.084
99.93%

q18
163.45
162.64
-0.812
99.50%

q19
12.17
12.83
0.662
105.44%

q20
29.09
29.04
-0.057
99.80%

q21
238.21
236.97
-1.233
99.48%

q22
15.46
15.62
0.162
101.05%

total
1281.88
1284.93
3.042
100.24%

authors: GlutenPerfBot exmy

repo: oap-project/gluten


text:
[NSE-326] update scala unit tests to spark-3.1.1

What changes were proposed in this pull request?
This pr updates the scala unit tests to spark-3.1.1.
fixed https://github.com/oap-project/native-sql-engine/issues/326.
How was this patch tested?
under test

@github-actions scala-unit-test

@rui-mo just amend one commit to enable scala unit tests in github action

authors: rui-mo zhouyuan

repo: oap-project/native-sql-engine


text:
modifying ci_scripts/install_pmem_common.sh oap-ape/README.md

What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

please change your target branch to ape

authors: jikunshang yspMing

repo: oap-project/sql-ds-cache


text:
Playlists

Playlist support. Based around liked songs code

Added ages ago

repo: obiwanconobi/jel-music-app


text:
Is there a way to ignore properties when inserting or updating?

The Azure client SDK allows you to add [JsonIgnore] attributes on properties you want to ignore from reading and writing.  I am able to do that for ODATA reading by using the .Select clause, but how can I ignore certain properties on insert or update?

It looks like there may be support for NotMapped attributes, will try that.

Since I am using a PCL, I had to create my own NotMappedAttribute class, but it worked.


text:
automatic project version and EO version

closes #95

@rultor merge

@rultor merge

@mikhaillipanin OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@mikhaillipanin Done! FYI, the full log is here (took me 3min)

authors: MikhailLipanin rultor

repo: objectionary/dejump


text:
feat(#642): matchers/SameXml.java

In this pull I've transit SameXml.java from test package to main/org/eolang/jeo/matchers in order to reuse it in https://github.com/objectionary/opeo-maven-plugin.
@volodya-lombrozo take a look, please
closes #642

PR-Codex overview
This PR updates package imports, adds a new hamcrest dependency, and refactors XML comparison matchers.
Detailed summary

Updated package imports in test files
Added hamcrest dependency in pom.xml
Refactored XML comparison matchers to a new package for better organization

The following files were skipped due to too many changes: src/test/java/org/eolang/jeo/matchers/SameXmlTest.java

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@volodya-lombrozo some workflows fail on windows, in master as well.

@h1alexbel Ok, I'll take a look a bit later

@volodya-lombrozo updated, take a look, please

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 10min)

@volodya-lombrozo Thanks for the review! You've earned +42 points for this: +25 as a basis; +27 for the 274 hits-of-code that you reviewed; -10 for too few (4) comments.  Your running balance is +185.

@h1alexbel Thanks for the contribution! You've earned +23 points for this: +30 as a basis; -7 for too many hits-of-code (326 >= 100). Please, keep them coming. Your running balance is +40.

@h1alexbel Thanks for the review! You've earned +5 points: +25 base, -40 for self-review (not recommended), +20 for hits-of-code and comments (capped at minimum 5). While self-reviews aren't ideal, your contribution is valued. Remember, more external reviews and comments can boost your points next time! Your balance is now +136. Keep up the good work! 💻👍

repo: objectionary/jeo-maven-plugin


text:
Specifying multiple columns with Attributes does not work

To reproduce:

Start the PropertyGridDemo project
Click the Collections example and switch to Custom tab

There should be two columns as defined in the example library, but only one is shown.
I think the problem is that the PropertyDescriptor only returns one attribute of a kind. The ColumnAttribute class tries to solve that by specifying a unique type id - but only if it's not built as a PCL, which it is in the current PropertyTools configuration.

Using this derived attribute fixes the problem:
    public class ColumnExAttribute : ColumnAttribute
    {
        public ColumnExAttribute(int columnIndex, string propertyName) : base(columnIndex, propertyName)
        {
            this.TypeId = Guid.NewGuid();
        }

        public ColumnExAttribute(int columnIndex, string propertyName, string header) : base(columnIndex, propertyName, header)
        {
            this.TypeId = Guid.NewGuid();
        }

        public ColumnExAttribute(int columnIndex, string propertyName, string header, string formatString, string width = "Auto", char alignment = 'C', bool isReadOnly = false) : base(columnIndex, propertyName, header, formatString, width, alignment, isReadOnly)
        {
            this.TypeId = Guid.NewGuid();
        }

        public override object TypeId { get; }
    }

But it seems that this is not possible in a PCL configuration...

Apparently, the Attribute.TypeId property is not available in a portable class library. So it is not possible to retrieve multiple instances of an Attribute set to a single property using the PropertyDescriptor.
The only way I can see fixing this, while keeping the PCL configuration and the PropertyDescriptor, would be to define a bunch of Column-derived subclasses, like
public class Column1Attribute : ColumnAttribute {}
public class Column2Attribute : ColumnAttribute {}
public class Column3Attribute : ColumnAttribute {}
...

It's neither elegant nor ideal, because the number of available columns is always finite...
Apparently, with regular reflection (PropertyInfo) it's possible to retrieve multiple instances of attributes, but this would be different to all other parts of PropertyTools, and there is probably a reason to use PropertyDescriptor.
Thoughts @objorke ?

I don't really like the last idea... :-) There must be a way to read attributes with AllowMultiple = true for pcl libraries! Can PropertyInfo be used?
Related links

https://github.com/objorke/PropertyTools/blob/develop/Source/Examples/PropertyGrid/ExampleLibrary/Examples/CollectionsExample.cs#L49
https://github.com/objorke/PropertyTools/blob/develop/Source/PropertyTools.Wpf/PropertyGrid/PropertyGridOperator.cs#L486
https://github.com/objorke/PropertyTools/blob/develop/Source/PropertyTools.Wpf/PropertyGrid/PropertyGridOperator.cs#L386
https://msdn.microsoft.com/en-us/library/system.attribute.typeid(v=vs.110).aspx
https://msdn.microsoft.com/en-us/library/system.attributeusageattribute.allowmultiple(v=vs.110).aspx
https://msdn.microsoft.com/en-us/library/system.reflection.propertyinfo.attributes(v=vs.110).aspx
https://social.msdn.microsoft.com/Forums/windows/en-US/e6bb4146-eb1a-4c1b-a5b1-f3528d8a7864/propertydescriptorattributes-doesnt-return-all-attributes?forum=winforms

If AllowMupltiple is impossible, what about splitting the current attribute into

ColumnsAttribute(params string[] properties)
ColumnHeadersAttribute(params string[] headers)
ColumnFormatStringsAttribute(params string[] formatStrings)
ColumnWidthsAttribute(params string[] widths)
ColumnAlignmentsAttribute(params string[] alignments)

From what I've read, PropertyInfo should be able to retrieve multiple Attributes of the same type. But as far as I'm aware, everywhere else in PropertyTools the TypeDescriptor is used. I feel like we should stay consistent here, e.g. apparently the TypeDescriptor allows Attributes to be changed at runtime, whereas PropertyInfo doesn't:
http://stackoverflow.com/a/129369
How about something like
public class ColumnDefinition
{
    string PropertyName {get; set;}
    string Header {get; set;}
    string FormatString {get;set;}
...
}

ColumnsAttribute(params ColumnDefinition[] columnDefinitions)

Would be more convenient to insert/delete columns and closer to the current syntax I think.

What do you think about the last proposal @objorke ? If you're ok with this I could put together a PR.
Or would you prefer something with PropertyInfo?

I don´t think the last ctor is possible to use as an attribute.
See
https://github.com/objorke/PropertyTools/blob/develop/Source/PropertyTools/DataAnnotations/ColumnsPropertyAttribute.cs#L24
can´t this attribute be used?
[Browsable(false)]
public IEnumerable<ColumnDefinition> ColumnDefinitions { get { ... } }

[ColumnsProperty(nameof(ColumnDefinitions))]
public IEnumerable<Item> { get { ... } }

You are absolutely right regarding the constructor.
I made a commit implementing the column definition your proposed way. The one thing I don't like about it is the possible ambiguity between the "portable" DataAnnotations.ColumnDefinition and the PropertyTools.Wpf.ColumnDefinition - do you have a clever idea for solving this? I didn't want to use something "clunky" like PortableColumnDefinition, but maybe this would be better?
Once you are happy with the implementation I will make a PR.

Nice! Great improvement of the example. Could we then remove the ColumnAttribute?
Agree the name can be a bit confusing.
What about simply Column since the attribute is named ColumnsPropertyAttribute?
[Browsable(false)]
public IEnumerable<Column> Columns { get { ... } }

[ColumnsProperty(nameof(Columns))]
public IEnumerable<Item> Items { get { ... } }

Yes, the ColumnAttribute is already removed.
Made the rename as suggested.
Some examples in the Collections category are still broken, but I think this is because of DataGrid bugs unrelated to this issue.

Is it possible to push the current develop build to nuget so I can use the recent fixes in my project? Thanks!

I just pushed the latest build: 3.0.0-unstable0094
From https://ci.appveyor.com/project/objorke/propertytools/build/3.0.0-unstable.94.build.271/artifacts

authors: Jonarw objorke

repo: objorke/PropertyTools


text:
tooth neck

abnormal tooth neck morphology
DEF: any structural anomaly of the slightly constricted part of a tooth between the crown and the root
SYN: abnormal cervix dentis morphology
SYN: abnormal collum dentis morphology
SYN: abnormal dental neck morphology
SYN: abnormal tooth cervix morphology
SOURCES: UBERON:0015181, https://medical-dictionary.thefreedictionary.com/tooth+neck
Child term:
abnormal cementoenamel junction morphology
DEF: any anomaly of the interface between the cementum and enamel at the cervical region of a tooth; it represents the anatomic limit or line of division between the crown and root surface
SYN: abnormal cemento-enamel junction morphology
SYN: abnormal cervical line morphology
SYN: abnormal CEJ morphology
SOURCE: https://medical-dictionary.thefreedictionary.com/cementoenamel+junction

Added:
abnormal tooth neck morphology MP:0030503
abnormal cementoenamel junction morphology MP:0030504


text:
Add memory and CPU metric

This PR adds CPU and memory metric to measures
(also adding generic Measure function that allows to call any of the metric functions generically)

Do you intend to complete scraping CPU/MEM metrics for all components or keep it only for ingesters? If latter, why not?

For now, just for ingesters ... the problem is that we have "too much info" in the results folder ... maybe it makes sense to organize the results in sub-folders and this will make things more understandable

As per adding cavidsor, we can use container_cpu_user_seconds_total now, WDYT?

Added function with "that (renamed the functions to "container_YYY_" )

repo: observatorium/loki-benchmarks


text:
Import kcas_data as a part of saturn

kcas_data contains a nice set of compositional lock-free structures built with kcas. This is in alignment with the goal of saturn, to provide parallelism-safe data structures for OCaml 5+ users. It might be useful to expose these structures in kcas_data as a part of saturn_lockfree or even a separate sub-package.
cc @polytypic @lyrm

This plan has been cancelled.

repo: ocaml-multicore/saturn

license: ISC


text:
No price for pool algos

Published new algo, used dynamic price, result:

did:op:d855538c659038bedbdb094417391ac63a9d22084900cf510bb78ed66f2ac6da
lots of weirdly formatted numbers in the transaction for it for all ERC-20 tokens but that might be another issue: https://rinkeby.etherscan.io/tx/0x9465e0f040323faedfa94ea0e938cab382c9a9ea60c7ba831ef2e868c6c3ee5e

Did you check in the c2d branch? It works for me

authors: kremalicious mihaisc

repo: oceanprotocol/market


text:
Fix published assets

closes #877

yep the assets are fetched now but there seems to be a problem on the pagination with this changes, trying to figure it out now

it looks like we can't rely on queryResult?.page since it has wrong values, for eg it's returned 1 for the first two pages and on the third page it's two and so on.

i'll take a look

hmm seems to work with the update from aquarius helper, I'll revert my prev update

authors: bogdanfazakas mihaisc


text:
Disconnecting components

Fixes oceanprotocol/multi-repo-issue#138.
Changes proposed in this PR:

I ran couple of tests with barge and then I have turned it off to see the behavior of pytest.
At the beginning, it froze couple of seconds and then it threw error log (for not finding the connection to the components).

After these errors occurred, I started again barge, but nothing seemed to re-establish and the tests were still failing. The problem is that pytest couldn't re-establish the connection with the other components after they were turned off couple of seconds.


text:
#332: Added default fallback language to English

#332 Default fallback language was never set and setting a default language did not request for the translations file if the language was not recognised.
Also added a version to the package.json since:

The most important things in your package.json are the name and version fields. Those are actually required, and your package won't install without them.

Source: https://docs.npmjs.com/files/package.json#version
Without this version I could not reference to my fork version on GitHub from my own project's package.json file. If (for some reason) you do not want it, you may remove it of course.
Let me know what you think!

The version is automatically generated on Travis when I push / merge commits (based on the commits messages), that's why it's empty in package.json :)

@ocombe Ahhh right, my bad! Shall I revert the commit?

Just remove it and push the changes. We can squash commits here.

Done. :)

So my point of opinion on this PR:

We should not set the default language to "en" automatically, we just don't know how people want to name it, and maybe for some reason they don't want a default language
Is it really necessary to have a DefaultLangChangeEvent ? I don't see any use case, you define the default lang at bootstrap and then you forget about it, it's not really something that you should change
If we don't use the DefaultLangChangeEvent, then changeDefaultLang becomes unecessary as well

So in the end, the only thing that you should do in this PR is: get the default lang file when the current translation is missing (if the default lang is missing too)

And sorry if I seem a bit blunt here, I'm just making sure that we don't bloat the code too much by adding stuff that isn't necessary. But rest assured that I appreciate the time that you took to do this PR, and it's a good one (clean code and tests!)

@ocombe Thank you for your feedback! No worries, this is the kind of feedback I can use. We all want the same thing, to make it as good as we can! :)
To get back on your 3 points of attention:

You are completely right, didn't think of that. Will update the PR!
2 & 3) I see what you mean and I tried to do it without the default lang changed event. The problem is though that I need to inform the component to update all keys when the default language changes (like with a normal use updates all translations as well). I could "abuse" the event that is already there for the use to update all translations. But this could lead to some strange behaviour (e.g. waiting for a use to finish, but getting a default lang changed event in between).

Removed the default language to be 'en'. Moved the logics that I put into setDefaultLang (which is not really functionality for a setter function) to the getParsedResult function. Also updated the unit tests to reflect the changes.

Ok good, a few last changes and I'll merge it:

fix the conflicts (merge master into your branch)
update the new directive with the same thing that you did in pipe
add documentation on the new event in the README.md

Conflicts are merged
Updated new directive and added some extra unit tests (also for the new subscription). This made me realise that moving the get default language translations from setDefaultLang to getParsedResult was wrong. Setting the default language should emit an event with the translations for the default language. In theory the setDefaultLang function should work about the same way as the use function. So I moved it back to setDefaultLang (but left out the part from getParsedResult where it set the default lang to 'en' of course).
Added documentation about the new event in the README.md

Thanks for your help!!

@ocombe Updated PR, removed the translations argument from the DefaultLangChange events in the pipe and the directive.

It's ready to be merged. Let me know if I am missing something!

@ocombe Can this be merged?

@ocombe Is this on hold/waiting for something?

Sorry, been a bit busy, I'll try to do it this week

No worries, not trying to insult or push you. I was just wondering what the status was.

No problem :)

Ok I fixed the bug in the directive, the failing test will pass now.
I also fixed a few other things at the same time: it sets the default lang to the first lang that receives translations (if the default lang is undefined at that time), and the onTranslationChange event now emits for all setTranslation calls (not just for merges, I don't know why this was just for merge).
Also the directive now listens to onTranslationChange events.

All of this is available in the new 5.0.0 release.

Nice, great work!

authors: SamVerschueren ocombe pndewit


text:
Update OVN-Kubernetes default network policy test scenario

Fixing the issue https://github.com/ocp-power-automation/ocp4-playbooks-extras/issues/5 and refactored the code for default network policy
Signed-off-by: Varad Ahirwadkar varad.ahirwadkar@ibm.com

/approve

authors: pravin-dsilva varad-ahirwadkar


text:
Updating kong setup script, Dockerfile

@tushardwivedi  please verify the Dockerfile from our last discussion.

Dockerfile is good.

authors: pranny tushardwivedi

repo: octoai/apihandler


text:
🛑 Ahora+ (rrhh.octubre.org.ar) is down

In d020126, Ahora+ (rrhh.octubre.org.ar) (https://rrhh.octubre.org.ar/api/health) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Ahora+ (rrhh.octubre.org.ar) is back up in 54f5249.

repo: octubre-softlab/octubre-upptime


text:
No sohkd in root directory

Hello.
I discovered this project and want to try it.
First thing, you don't have to run it as root, will be enough to add your user to the odilia group and install as a regular user. Odilia does not want to start as root, because of no at-spi daemon detected.
Second thing, I can't locate sohkd to properly use shortcuts assigned to Odilia. I don't have it in my main repository directory of the odilia project. Where i can retrieve it?

The readme no longer contains references to sohkd or permissions...

Huh?
https://github.com/odilia-app/odilia/blob/a06ab246b6de96cd4cebea82bad17ee4ef2f1245/README.md?plain=1#L47-L54

Oh shoor! I missed that one!
I removed a few references, I'll remove this one as well.

Odilai should not run with root privilege, since it won't be able to attach to the accessibility session. Is there a reason you are wanting to run Odilia as root?

Look at readme, and something about /etc config locations.

All of that should be removed now as well. This was mentioned by somebody else on a Reddit post :)

OK, so only one question.  Someone use irc or this channel should be closed?

I beleive the channel is still active, if that's what you're asking. It should be bridged to our Matrix server.

Great.

authors: TTWNO jcaesar zywek123

repo: odilia-app/odilia


text:
TeamfightEvent sometimes renders no heroes on losing side

steps to reproduce:

go to https://www.opendota.com/matches/3557630134/story
Scroll to the bottom to see this:

Logging out the TeamfightEvent props reveals that some heroes have deaths set to zero, so that the deaths variable only contains the winning team heroes (relevant line: https://github.com/odota/ui/blob/d4076333a26471cd3ebc2958e3fb584cd025b362/src/components/Match/MatchStory.jsx#L567)
Not sure exactly where the data is coming from so I can't trace it further, but that's probably a good place to start.

@mdiller shouldn't it say "for no heroes"?

So it looks like the reason why this is happening is I only accounted for the winning team having no deaths. I didn't realize you could win a teamfight when four of your players died and none of theirs did lol.
Guess I'll make a new string for when that happens.

I'm guessing it was gold from the buildings, or maybe a hero died weirdly (shared kill credit?) and doesn't get picked up

I'm really not sure what happened (https://www.opendota.com/matches/3557630134/teamfights)
Looks like some kind of fountain farming, but if someone wants to look at the replay maybe we can get some answers :)

Hmmm yeah maybe. I'm tempted to watch the replay and look. I thought I've been in some one sided games but heres this game with 139 - 5 kills at 43 minutes in.

probably the dark willow fountain camped the dire team, killed 4 heroes, got killed by the fountain, and gave a ton of gold to everyone on dire.  I'm not sure why the death doesn't show up on the teamfight log though

Hmmmm yeah maybe, that'd make sense. I'm going to check the replay.

What a strange game.
This happened near the end of the game, and the entire radiant team is afk in dire's fountain, killing anyone who spawns instantly.
Ancient Apparition (on dire's team) is farming in middle lane for the whole "teamfight." his farming plus the gold he gets from his three disconnected teammates means he gains about 1400 gold during the fight. That gold gain is more than the gold the radiant team got for killing four of his teammates as they spawned, and so i guess technically he "won" the "teamfight"

Looks like they went to the fountain at 30 minutes in and didn't leave until the end of the game

The second one was pretty much the same thing except the AA took radiant mid rax and t3 so its a bigger gold swing to his team

The AA ended up just pushing and winning the game because I guess the radiant just didn't care?

https://www.youtube.com/watch?v=6DGNZnfKYnU
I just noticed that dire won haha

lol yeah

authors: blueish howardchung mdiller

repo: odota/ui


text:
Update column type does not get reflected into Egeria

Update column type does not get reflected into Egeria. The type was change from string length 20 to string length 25 but the change to not go through to Egeria.
It looks like code in PostgresDatabaseConnector around line 883 should be uncommented then any follow-on issue debugged
                       if (egeriaColumn.getDatabaseColumnProperties().getQualifiedName().equals(postgresColumn.getQualifiedName()))                             {                             /*                             we have found an exact instance to update                              */                                 found = true;                                 //TODO                              //   updateColumn(postgresColumn, egeriaColumn);                                 break;                             }

That doesn't look like the latest release of code @davidradl or it doesn't match my version.

That doesn't look like the latest release of code @davidradl or it doesn't match my version.

See https://github.com/odpi/egeria-database-connectors/blob/main/egeria-connector-postgres/src/main/java/org/odpi/openmetadata/adapters/connectors/integration/postgres/PostgresDatabaseConnector.java . This is the latest in master and matches my comment. Is there more code that you have locally that could help here?

That piece of code looks like it's from when I was unit testing the "addDatabase" branch which makes it at least two major release behind what I have.
I unit tested the update and delete branches after this .
I've just downloaded the latest version from git hub and it's not what your looking at ?
`                  for (DatabaseElement egeriaDatabase : egeriaDatabases)
{
                    String egeriaQN =  egeriaDatabase.getDatabaseProperties().getQualifiedName();
                    String postgresQN = postgresDatabase.getQualifiedName();

                    if (egeriaQN.equals(postgresQN))
                    {
                    /*
                    we have found an exact instance to update
                     */
                        found = true;
                        updateDatabase(postgresDatabase, egeriaDatabase);
                        break;
                    }
                }
                /*
                this is a new database so add it
                 */
                if (!found)
                {
                    addDatabase(postgresDatabase);
                }
            }`

@wbittles I see that code fragment you mentioned in 'UpdateViewColumns' method
The fragment @davidradl mentioned is in 'UpdateTableColumns' method
Can you clarify how the connector/postgres works with respect to tables/columns? Maybe the code is the same, and it's just our misunderstanding (I've not tested that code path myself)

@planetf1 @davidradl  oops , I was looking at the doc for the database not updating
I have no idea why that code is like that , especially as the other call to that method isn't commented out.

I'm using the connector currently, so
Happy to test the change for column update if you plan on updating, or I can try the code change.

Closing - this repository is now being archived.

authors: davidradl planetf1 wbittles

repo: odpi/egeria-database-connectors


text:
🛑 urbamonde.org is down

In 3d60673, urbamonde.org (https://www.urbamonde.org) was down:

HTTP code: 429
Response time: 574 ms

Resolved: urbamonde.org is back up in b8b6e07.

repo: odqo/upptime


text:
🛑 urbamonde.org is down

In 3f2b1c1, urbamonde.org (https://www.urbamonde.org) was down:

HTTP code: 429
Response time: 630 ms

Resolved: urbamonde.org is back up in 734fb51.


text:
Obtaining signed URL and S3 OPTIONS succeed, S3 PUT fails with 403 Forbidden

I've embedded the uploader in a web application. It works when testing locally but when we deploy to EC2 uploading no longer works. The uploader is able to obtain a signed S3 URL and send an OPTIONS request to it, but the subsequent PUT fails.
These are the requests.
The only difference I can notice between the local and EC2 signed URLs is that the EC2 URL has a x-amz-security-token parameter. Could the OPTIONS request be consuming the security token?

Apparently it just needs s3::PutObjectAcl permissions to work (as described here). Problem solved.

@PatSmuk360 I am also getting this error, even with those permissions. Are there any additional settings that you have on your bucket policy?
Also, any chance you're using NextJS/now? This was working fine for me until I moved to now v2


text:
Identifier clashes in DOM when multiple score containers contain renderings of the same MEI file

... and more generally, whenever different MEI files share identifier names.
Problem: The fragment URI's prefix up to the # is stripped in-application before rendering annotations. We should instead replace the prefix with the relevant score container's React key, and make a method for "getByOriginalID" for those containers.

To clarify different levels of issue:

ID clashes: Lots of components turn IDs in source files (especially MEI and TEI) into DOM IDs. There is no guarantee that these will be unique, making the resulting app invalid HTML and it's functionality, potentially, unreliable.
URI reuse: The same file, rendered twice may have the same IDs duplicated because of 1., but also, depending on how component keys are generated, may have clashing react component keys. We provide no clear mechanism to distinguish between elements from multiple renderings of the same file within the app. This may have an impact on preserving or recreating state.
Where it is important to be able to refer to state specifically, we will need react keys to be reproducible (rather than random). It's probably worth noting that we've got relatively little currently for saving and loading state anyway.

To be clear, there is no problem keeping track of URIs referring to external materials or parts of them within MELD, this is largely (entirely?) an issue of the DOM and of the application state.

authors: DILewis musicog

repo: oerc-music/meld-clients-core


text:
RuntimeError: Invalid parameters for enforcer.

in core_enforcer, there are these code:
class CoreEnforcer:
        #...
        def init_with_model_and_adapter(self, m, adapter=None):
        """initializes an enforcer with a model and a database adapter."""

            if not isinstance(m, Model) or adapter is not None and not isinstance(adapter, Adapter):
                raise RuntimeError("Invalid parameters for enforcer.")

so the async adapter is not instance of Adapter，then throw this error.
how to fix this problem?

i got it. if use AsyncAdapter, the enforcer must be use AsyncEnforcer. So the README.md example is incorrect.

@kingshuaishuai can you make a PR and fix the README?

Of course, I'd be happy to do

authors: hsluoyz kingshuaishuai


text:
Niepoprawne formatowanie po implementacji szablonów

Po implementacji tych szablonów rozjechały się trochę divy na wysokości.

I do that in my branch so it's waiting together with pull request. Yes it's done in html style attribute, but if we will make some other file with custom styles and add then below bootstrap.css we will easy cut and paste that. So it's no problem, and looks good now.

Do momentu kiedy nie poszedł merge trzymamy issue otwarte. Kiedy coś będzie już w masterze wtedy można zamykać.

authors: marcin-pwr ofilipowicz

repo: ofilipowicz/owndb


text:
Ignoring Carriage Returns

I have an issue where within my csv file I have random lines where the input is quoted, but the end quote ends with a carriage return. So the value is essentially coded "value\r". When Hive ingests this file, it begins a new line where really, the carriage return should be ignored. Does your library offer a solution for such a problem? Thanks,

Sorry @mikekeith52 , by the time it gets to the SerDe the lines are already split.

repo: ogrodnek/csv-serde


text:
Improve the Facility details page UI

Describe the bug
When we add multiple facility features and link several spoke facilities to a single facility, the data on the respective facility detail page is displayed collectively. However, the layout is inefficient, with a lot of unnecessary white space, making it difficult to present the information effectively.
To Reproduce
Steps to reproduce the behavior:

Open any facility card
Click on manage facility and update the facility
Add multiple features, link spoke facilities to the facility, and submit the form.
See error

Expected behavior

[ ] Improve the UI layout properly to effective utilise the spaces and show all the relevant details

NOTE: Whoever is working on this issue, please ensure that the mock UI design is approved either in the Slack care_general channel or during the EOD call discussion before starting any coding work.
Screenshots

@nihal467 this will be addressed while we work on the HMIS


text:
🛑 Reportes is down

In 02c7876, Reportes (https://repository.ohelit.co/jasperserver/login.html) was down:

HTTP code: 502
Response time: 606 ms

Resolved: Reportes is back up in 14e2894.

repo: ohelitco/status


text:
Issues accessing presigned s3 URLs

I can create Funnel tasks reading/writing to s3 without issues.
If I change a source URL from s3://[...] to a presigned URL https://{...}.s3.amazonaws.com[..] the task will fail. I see that in the task log the URL is listed correctly, but in the debug execution log is is changed. For example, [...]5DT&Signature[...] gets changed to [...]5DT\u0026Signature[...], which causes the failure.
Is there anything I need to add to allow for URLs with special characters?

What is the error message being returned and what version of Funnel are you using?

This is the version I am using (I am running this on a Mac, I used the Homebrew instructions from here https://ohsu-comp-bio.github.io/funnel/download/):
git commit: 3eaec933
git branch: master
git upstream:
build date: 2018-06-20T16:17:18Z
version: 0.8.0

This is an example of the error I get (I shortened the URL a bit, but the relevant parts are there):
funnel task get bpnlopovlv353e62s680
{
  "id": "bpnlopovlv353e62s680",
  "state": "SYSTEM_ERROR",
  "name": "Hello world",
  "inputs": [
    {
      "url": "\"https://[...].s3.amazonaws.com/ga4gh_demo/README?AWSAccessKeyId=[...]5DT\u0026Signature=cUa[...]g%3D\u0026Expires=158[...]02\"",
      "path": "/inputs/README"
    }

If I look at the task in the Web UI (http://localhost:8000/v1/tasks/bpnlopovlv353e62s680) I get the correct URL:
"https://[...].s3.amazonaws.com/ga4gh_demo/README?AWSAccessKeyId=[...]5DT&Signature=cUa[...]g%3D&Expires=158[...]02"

What is the error message that gets logged by the storage backend when the task is run?
The encoding stuff you are seeing is probably related to https://github.com/golang/protobuf/issues/407, but I am not sure that it is the underlying issue.

The storage back (s3) end simply responds with an authorization error:
<?xml version="1.0" encoding="UTF-8"?>
<Error>AccessDenied<Message>Access Denied</Message><RequestId>2F[...]87</RequestId><HostId>ss[...]0K/w=</HostId></Error>

But I think this is due to the access credentials not being formatted correctly.

I don't think it is an issue with the access credential formatting - Funnel tries to execute a HEAD request on URL prior to downloading which is going to fail for a pre-signed URL that authorizes a GET request.
I will see what I can do to resolve this.

Thank you!

Are you using presigned URLs just for the inputs or for the outputs as well?
I was able to add download support, but upload support would be a bit more work.

I would like to do both, eventually. But being able to access data first would be OK.
(Related to this: if I could specify s3 URLs for input/output formatted like s3://[...]/ga4gh_demo/ via cwl-tes that would also be helpful; but that is a separate issue).

I think it works! I just tested the commit locally, and I do no longer get an error. Thank you.

authors: AlexanderSenf adamstruck

repo: ohsu-comp-bio/funnel


text:
Android godot reports 0 surfaces for mesh data

Android godot reports 0 surfaces for mesh data from MeshInstance.
On PC it behaves normally. Any ideas?

This is expected. OpenGL ES 2.0 does not allow retrieval of mesh data once
you gave it to the API
On Sat, Jul 11, 2015 at 1:20 AM, Sergey Lapin notifications@github.com
wrote:

Android godot reports 0 surfaces for mesh data from MeshInstance.
On PC it behaves normally. Any ideas?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2259.

--
OkamStudio

Probably a workaround is possible, as manipulation of mesh material is
useful feature,
and I think it is quite possible to get these data using Godot API instead
of GLES.
On Sat, Jul 11, 2015 at 8:27 AM, Okam Studio notifications@github.com
wrote:

This is expected. OpenGL ES 2.0 does not allow retrieval of mesh data once
you gave it to the API
On Sat, Jul 11, 2015 at 1:20 AM, Sergey Lapin notifications@github.com
wrote:

Android godot reports 0 surfaces for mesh data from MeshInstance.
On PC it behaves normally. Any ideas?
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2259.

--
OkamStudio
—
Reply to this email directly or view it on GitHub
https://github.com/okamstudio/godot/issues/2259#issuecomment-120575059.

Is a workaround as mentioned by @slapin possible, or should we just close this one as wontfix?

no, workaround is not possible, closing. This will eventually work in GLES3 backend.

authors: akien-mga okamstudio reduz slapin

repo: okamstudio/godot