text:
FEATURE REQUEST: Custom Functions

It would be useful to have custom functions, I am attempting to mod this, but the parse method isn't exactly built for dynamic customization, it uses exact pre-defined string lengths.
an example like
->addFunction('min', function($value, $arg) {
    return min($value, $arg)
});

While this is a basic example, it would be very useful to have functions passing in the value and a single argument.

Well, what you're suggesting is a good thing, Timothy, thanks. But, you know, the architecture of this library was designed 6 years ago, and, frankly, now I don’t have time for changing it (and for adding new features in general) due to work in the company. I'm always ready to consider pull requests, though.

Well, what you're suggesting is a good thing, Timothy, thanks. But, you know, the architecture of this library was designed 6 years ago, and, frankly, now I don’t have time for changing it (and for adding new features in general) due to work in the company. I'm always ready to consider pull requests, though.

Yeah, I totally understand. I just wanted to put this here in case anyone could figure out the best way to handle this.

authors: denissimon timothymarois

repo: denissimon/formula-parser


text:
Subapp error handler is not respected

Reproduction:
import { App } from 'https://denopkg.com/deno-libs/tinyhttp@master/mod.ts'
import { makeFetch } from 'https://deno.land/x/superfetch@1.0.4/mod.ts'

const app = new App({
  onError: (err, req) =>
  new Response(`Ouch, ${err} hurt me on ${req?.url} page.`, {
    status: 500,
  })
})

const subApp = new App({
  onError: (err, req) =>
  new Response(`Handling ${err} from child on ${req?.url} page.`, {
    status: 500,
  })
})

subApp.get('/route', async (req, res, next) => await next('you'))

app.use('/subapp', subApp)

const server = app.handler
const fetch = makeFetch(server)

const res = await fetch('/subapp/route')
res.expectStatus(500).expectBody('Handling you from child on /subapp/route page.')

shouldn't the last test throw a 404?
  it('lets other wares handle the URL if subapp doesnt have that path', async () => {
    const app = new App()

    const subApp = new App()

    subApp.get('/route', (_, res) => void res.end(subApp.mountpath))

    app.use('/test', subApp)

    app.use('/test3', (req, res) => void res.end(req.url))

   // const fetch1 = makeFetch(app.handler)

    //const res1 = await fetch1('/test/route')
    //res1.expect('/test')
    const fetch2 = makeFetch(app.handler)
    const res2 = await fetch2('/test3/abc')
    res2.expect('http://localhost:8080/test3/abc')
  })

source: gharchive/issue

authors: aarontravass talentlessguy

repo: deno-libs/tinyhttp

license: MIT

license_type: permissive

license_source: github-api


text:
Failed to lookup address information: nodename nor servname provided, or not known

I have this error
error: Uncaught (in promise) Error: failed to lookup address information: nodename nor servname provided, or not known     at processResponse (core.js:223:11)     at Object.jsonOpAsync (core.js:240:12)     at async Object.connect (deno:cli/rt/30_net.js:224:13)     at async MongoClient.connect (client.ts:20:18)
Trying connect with Atlas using the following code:
const client = new MongoClient();
await client.connect(
mongodb+srv://${env.dbUser}:${env.dbPass}@${env.dbHost}/,
);
deno 1.5.4
mongo@v0.20.0

I found this, but not sure how to use it...

The protocol mongodb+srv don't surpport now, but may surpport the next version.

I also tried to use mongo atlas and it was throwing the same error
connecting to mongodb locally seems ok, but looks like theres no support for auth atm

This seem to be a issue related to macOS, I also met it.
Related: https://github.com/instructure/canvas-data-loader/issues/3

closed by #198


text:
Update github action runner

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
I use GitHub actions extensively as well as dependabot to update packages such as the actions used within workflows. While dependabot does a great job at updating actions, I do not believe it updates the runner versions the workflow is currently using. For example, if I was using the ubuntu-20.04 runner and a new runner was released, ubuntu-22.04, I would like dependabot to notice that and submit a pull request informing me to update it. While it is possible to simply update the runner to used the latest, I would like my builds to be as stable as possible and would like to test the workflow before blindly updating the runner.

Agreed, this would be helpful for a lot of folks.
Closing as a duplicate of:

https://github.com/dependabot/dependabot-core/issues/5541

See also:

https://github.com/dependabot/dependabot-core/issues/5819
https://github.com/dependabot/dependabot-core/issues/6247

@jeffwidman I don't think this should have been closed. This is different than the duplicate since this is looking at the runs-on value for jobs in workflows.

Good point. I'm no longer on the Dependabot team but perhaps @Nishnha or @jurre could reopen.

authors: jeffwidman jlosito krische

repo: dependabot/dependabot-core


text:
[Docker] Update docker images declared in action.yaml files

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Add option in dependabot to parse docker images declared in action.yaml files
Example file

runs:
  using: 'composite'
  steps:
    - name: Run
      shell: bash
      run: >
        docker run ubuntu@sha256:7a57c69fe1e9d5b97c5fe649849e79f2cfc3bf11d10bbd5218b4eb61716aebe6

@jeffwidman Do you consider this a new ecosystem or an improvement to the existing docker implementation?
Would you accept a PR for it?

Related:

https://github.com/dependabot/dependabot-core/issues/5541
https://github.com/dependabot/dependabot-core/issues/5819

As I mentioned in https://github.com/dependabot/dependabot-core/issues/7189, there's probably some work we need to do first for the config schema...
I'd see this as a part of adding support for bumping docker images in  GitHub Actions, with this as an advanced feature of that support probably.

authors: jeffwidman yeikel


text:
pipenv: handle installs with missing system deps

Raise a DependencyFileNotResolvable error when pipenv fails to install
dependencies require missing system level dependenceis.
The error message parsing is pretty gnarly as the only reference of the
offending dependency is in a tmp build path in the error message.
In the test case pipenv failed to install rtree because it requires a
system level dependency: libspatialindex.
We don't want to install this in our ever growning dockerfile so attempt
to explain the error to users.
The underlying cause might be obscured by this open issue in pipenv:
https://github.com/pypa/pipenv/issues/2791
Similarly, there's an open issue to fix the system requirement in rtree:
https://github.com/Toblerity/rtree/issues/147
This is the error pipenv raises:
pipenv.patched.notpip._internal.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/tmp8jk2atvgbuild/rtree/
File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/resolver.py", line 126, in <module>
    main()
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/resolver.py", line 118, in main
    _main(parsed.pre, parsed.clear, parsed.verbose, parsed.system,
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/resolver.py", line 78, in _main
    results = resolve(
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/resolver.py", line 61, in resolve
    return resolve_deps(
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/utils.py", line 718, in resolve_deps
    resolved_tree, hashes, markers_lookup, resolver = actually_resolve_deps(
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/utils.py", line 480, in actually_resolve_deps
    resolved_tree = resolver.resolve()
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/utils.py", line 385, in resolve
    results = self.resolver.resolve(max_rounds=environments.PIPENV_MAX_ROUNDS)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/resolver.py", line 102, in resolve
    has_changed, best_matches = self._resolve_one_round()
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/resolver.py", line 206, in _resolve_one_round
    for dep in self._iter_dependencies(best_match):
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/resolver.py", line 301, in _iter_dependencies
    dependencies = self.repository.get_dependencies(ireq)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/repositories/pypi.py", line 234, in get_dependencies
    legacy_results = self.get_legacy_dependencies(ireq)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/repositories/pypi.py", line 426, in get_legacy_dependencies
    results, ireq = self.resolve_reqs(download_dir, ireq, wheel_cache)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/piptools/repositories/pypi.py", line 297, in resolve_reqs
    results = resolver._resolve_one(reqset, ireq)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/resolve.py", line 260, in _resolve_one
    abstract_dist = self._get_abstract_dist_for(req_to_install)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/resolve.py", line 211, in _get_abstract_dist_for
    abstract_dist = self.preparer.prepare_linked_requirement(
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/operations/prepare.py", line 294, in prepare_linked_
    abstract_dist.prep_for_dist(finder, self.build_isolation)
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/operations/prepare.py", line 127, in prep_for_dist
    self.req.run_egg_info()
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/req/req_install.py", line 470, in run_egg_info
    call_subprocess(
  File "/usr/local/.pyenv/versions/3.9.0/lib/python3.9/site-packages/pipenv/patched/notpip/_internal/utils/misc.py", line 703, in call_subprocess
    raise InstallationError(
pipenv.patched.notpip._internal.exceptions.InstallationError: Command "python setup.py egg_info" failed with error code 1 in /tmp/tmp8jk2atvgbuild/rtree/

@jurre it's not a dependency of tensorflow, installing rtree is the problem, just used another dep to make sure the error does include the offending dependency name.
Don't think it's worth installing it just for rtree https://github.com/Toblerity/rtree

Will clarify this in the test setup!

source: gharchive/pull-request


text:
Remove unused GH_TOKEN env var

GH_TOKEN is necessary when using the gh CLI.
We use gh over in images-branch.yml when doing some stuff around forks, but not here in images-latest.yml.
I considered leaving it for consistency, and in case we ever add back gh. But the consistency arg doesn't make sense since images-updater-core.yml doesn't have this. And if we add back gh, we'll get an obvious error message about the problem per https://github.com/github/docs/issues/21930#issuecomment-1310122605.

does the token need to be removed from the secrets store too?

does the token need to be removed from the secrets store too?
Good Q, I hadn't even thought of it.

In this case no:

We use gh over inimages-branch.yml when doing some stuff around forks

Also it's set to GITHUB_TOKEN which is set automagically by GitHub Actions, so not something we explicitly set/manage.

authors: abdulapopoola jeffwidman


text:
Fix incorrect vulnerable manifest path check

Closes #185

@SalimBensiali I don't believe that this PR addresses the issue you reported:

This https://github.com/dependabot/fetch-metadata/pull/186/commits/b31caa427946df808241624a181d6966a88fa5c8 does expose a bug in how getAlert works. There was no coverage for when the manifest is at the root of the repo

@mwaddell I run the dry-run command to verify on my end

Thanks for this @SalimBensiali - it looks like the dist/ folder may be out of sync - can you run npm run build and check in any generated changes?

Done ✅

authors: SalimBensiali brrygrdn mwaddell

repo: dependabot/fetch-metadata


text:
Error: Bad configuration option: ControlMaster

| ----------------- | ---
| Issue Type        | Error: Bad configuration option: ControlMaster
| Deployer Version  | N/A
| Local Machine OS  | Windows 10
| Remote Machine OS | Ubuntu 16.04
Description
I have made all preparations for deploying and run command 'dep deploy production'.
What's the problem?
Content of deploy.php
namespace Deployer;
require 'recipe/laravel.php';
// Project name
set('application', 'my_project');
// Project repository
set('repository', 'git@github.com:pisarevaa/hello-world.git');
// [Optional] Allocate tty for git clone. Default value is false.
set('git_tty', true);
set('ssh_type', 'native');
set('ssh_multiplexing', true);
// Shared files/dirs between deploys
add('shared_dirs', [
    'storage',
]);
add('shared_files', [
    '.env'
]);
// Writable dirs by web server
add('writable_dirs', [
    'bootstrap/cache',
    'storage',
    'storage/app',
    'storage/app/public',
    'storage/framework',
    'storage/framework/cache',
    'storage/framework/sessions',
    'storage/framework/views',
    'storage/logs',
]);
// Hosts
host('picloudtech.ru')
    ->stage('production')
    ->user('deployer')
    ->set('deploy_path', '/var/www/prometei');
// Tasks
desc('Restart PHP-FPM service');
task('php-fpm:restart', function () {
    // The user must have rights for restart service
    // /etc/sudoers: username ALL=NOPASSWD:/bin/systemctl restart php-fpm.service
    run('sudo systemctl restart php7.2-fpm.service');
});
after('deploy:symlink', 'php-fpm:restart');
task('upload:env', function () {
    upload('.env.production', '{{deploy_path}}/shared/.env');
})->desc('Environment setup');
// [Optional] if deploy fails automatically unlock.
after('deploy:failed', 'deploy:unlock');
// Migrate database before symlink new release.
task('deploy', [
    'deploy:prepare',
    'deploy:lock',
    'deploy:release',
    'deploy:update_code',
    'upload:env',
    'deploy:shared',
    'deploy:vendors',
    'deploy:writable',
    'artisan:storage:link',
    'artisan:view:clear',
    'artisan:cache:clear',
    'artisan:config:cache',
    'artisan:migrate',
    'deploy:symlink',
    'php-fpm:restart',
    'deploy:unlock',
    'cleanup',
]);

Output log
In Client.php line 99:
The command "rm -f /var/www/prometei/.dep/deploy.lock" failed.
Exit Code: -1 (Unknown error)
Host Name: picloudtech.ru
command-line: line 0: Bad configuration option: ControlMaster

Turn ssh multiplexing off.

antonmedv, hi, turn ssh multiplexing off on local machine or remote server?

antonmedv, thanks a lot! it works! But I have a new error:
In Client.php line 99:
The command "rm -f /var/www/prometei/.dep/deploy.lock" failed.
Exit Code: -1 (Unknown error)
Host Name: picloudtech.ru
no kex alg

Google for no kex alg
Can't tell what it is.

Thanks a lot!

authors: antonmedv pisarevaa

repo: deployphp/deployer


text:
Host selector test

Q
A

Test Features
Yes

Do not forget to add notes about your changes to CHANGELOG.md

Closing. No work going on.

authors: antonmedv douria


text:
Task/create proxy mappers

This handles the initial state of adding proxy mapping - further mappers are very simple, though I'd like to make them even simpler.
Depends on #167

Superceded by #169

repo: derail-valley-modding/custom-car-loader


text:
🛑 Derailed API is down

In 1e92895, Derailed API (https://derailed.one/api) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Derailed API is back up in 79eb004.

repo: derailed-org/derailed-oss-status


text:
🛑 Derailed Home is down

In fdb4019, Derailed Home (https://derailed.one) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Derailed Home is back up in 3175dbe.


text:
Jammy - Ubuntu22.04 support ?

This version fails on Jammy because the nfsv4_servicehelper is set to the default idmapd in params, which does not exist in ubuntu.
Is there any view on when Jammy tests are finished and the module is supported?
error:
Error: Systemd start for idmapd failed!
journalctl log for idmapd:
-- No entries --
Error: /Stage[main]/Nfs::Server::Service/Service[idmapd]/ensure: change from 'stopped' to 'running' failed: Systemd start for idmapd failed!
journalctl log for idmapd:
-- No entries --

Hey, are there any differences between 20.04 and 22.04?

Thx for the reply, I will clone, test and get back to you.

I will test it in #157

The patch in #157 works for me at least, but I haven't studied in details if there are any changes between focal and jammy.

This issue can probably be closed now?

Yes, thanks!

authors: JoeriBouchet derdanne tobixen

repo: derdanne/puppet-nfs

license_source: bigquery


text:
Feature Request: Step over should not include deferred functions

Hi,
Here's a feature request that came up on Gogland's tracker:
What happens:
The "Step over" function "steps into" defer functions
E.g. Stepping through the Copy function below "Step over" steps twice into Close function.

What I would prefer:
"Step over" should not include deferred functions. (Only "Step Into" should include deferred functions)

The code example:
package main

import (
	"fmt"
	"io"
	"os"
)

func Copy(dst, src string) error {
	in, err := os.Open(src)
	if err != nil {
		return err
	}
	defer in.Close()
	out, err := os.Create(dst)
	if err != nil {
		return err
	}
	defer out.Close()
	_, err = io.Copy(out, in)
	cerr := out.Close()
	if err != nil {
		return err
	}
	return cerr // step over - defer
}

func main() {
	fmt.Println(Copy("demo", "demo"))
}

I'm happy to discuss this further as needed.
Thank you.

I think I agree. The only question I have is whether this new behavior should replace the old one or be selected by a flag in the DebuggerCommand packet.
I think the change is simple, it should suffice to change this lines in pkg/proc/threads.go:
			if bp != nil {
				bp.DeferReturns = deferreturns
			}

into this:
			if bp != nil && stepInto {
				bp.DeferReturns = deferreturns
			}

From a IDE users perspective I think there is no need for a flag. The user can use step-into or set a break point in the deferred function. (Without this change the user has no choice - defered  function are always included)

I also think this shouldn't be yet one more flag but rather just the default behavior. Too many flags would make delve lose it's easy to use feeling it has now imho. I'm happy to send the PR if you think that's a good solution.

Fixed via https://github.com/derekparker/delve/commit/5c9b2009cab214c1f912a2d8c6c89eec9f3ada2b.

authors: ThomasKrenn aarzilli derekparker dlsniper

repo: derekparker/delve

license: mit


text:
Authz GetModified targets and resources

Description
Expose the modified targets and resources for local cache control
Must

[ ] Tests
[ ] Documentation (if applicable)

Missing README and CLI

But not everything should be in the README and CLI, right? This is somewhat advanced and initially in normal usage you should not worry about it

repo: descope/go-sdk


text:
🛑 dico-mediadesign.de is down

In a39051c, dico-mediadesign.de (https://dico-mediadesign.de) was down:

HTTP code: 0
Response time: 0 ms

Resolved: dico-mediadesign.de is back up in 97459f6 after 10 minutes.

repo: desisn/upptime


text:
Create "randomImage" method

[!IMPORTANT]
Read issue #2 first

Create the randomImage method in the class FakeTestUtil and you must use “image()” from the datafake to generate a url.
And use it to replace the text on the line https://github.com/despensa-app/despensa-rest-api/blob/b03897dcd0cc8ba04d11e46e561a9c041ed0a014/src/test/java/dev/nmarulo/depensaapp/app/products/ProductServiceTestUtil.java#L72

Hello @nmarulo, I would like to contribute!

@JavierABarreto It's all yours 👍

Thanks!

authors: JavierABarreto nmarulo

repo: despensa-app/despensa-rest-api


text:
Update seed.json and database schema

Update product descriptions in the seed.json file and also update the product schema to include stock count and number of reviews.
TASKS

[x] Update seed.json and product images
[x] Add stock count and number of reviews field

✅ All done

repo: dev-xero/molla-backend


text:
Unsupported  distribution version 'bookworm'

docker in docker

Are you referring to Debian-bookworm? If so, then it is still in testing and we are waiting for an official release to provide support for it.
https://wiki.debian.org/DebianBookworm

yes

We will support bookworm once it's officially released. Thanks!

authors: samruddhikhandale sukeyisme

repo: devcontainers/features


text:
10.0.0-alpha.1 - Passing tagName as a (redundant) prop to an HTML node causes an error

Passing tagName as a prop like:
export default function () {
  return <button tagName='button' />
};

Will result in an error like:
E-v1.js:formatted:10882 Uncaught (in promise) TypeError: Cannot assign to read only property 'tagName' of object '#<HTMLButtonElement>'
    at w (E-v1.js:formatted:10882)
    at g (E-v1.js:formatted:10859)
    at A (E-v1.js:formatted:10999)
    at _ (E-v1.js:formatted:10953)
    at _ (E-v1.js:formatted:10948)
    at _ (E-v1.js:formatted:10948)
    at d (E-v1.js:formatted:10816)
    at A (E-v1.js:formatted:11001)
    at _ (E-v1.js:formatted:10953)
    at _ (E-v1.js:formatted:10948)

I noticed this because I have a (sloppy but allowed) pattern in a couple places where we have custom elements (not shown in example), and specify tagName like so:
class CustomElement extends Component {
  render () {
    const TagName = this.props.tagName;
    return <TagName {...this.props} />;
  }
}
render(<CustomElement tagName='button' class='the-class'>...</CustomElement>, parentEl);

To accommodate this, I need to explicitly remove tagName from props before passing it down. This can be solved like const propsToPass = Object.assign({}, this.props); delete propsToPass.tagName, but it's not a convenient pattern. Still, I think it might still be okay to leave it as is, as long as it's mentioned as a breaking change. Thoughts?

Well my first thought would be you can solve this much shorter like this:
  render () {
    const { TagName, ...props } = this.props;
    return <TagName {...props} />;
  }

I think I need to update my webpack config to support object rest like that, but point taken. 😄
On the lib side, I kinda agree, and in the spirit of keeping things small for preact, I agree that it could prob be left to the consumer. I guess it might just be worth flagging this for a look since, if there are no special checks, then there may be other footguns like this that didn't exist before. This particular issue is the kind of thing that I expect to see mild pain for upgrades and almost no pain for new usage.

This is probably fixed with #1377 . My guess is that our minifier turned this._force into this.t because we set that to false on each state update.

This is probably fixed with #1377 . My guess is that our minifier turned this._force into this.t because we set that to false on each state update.

Isn't this meant for the other issue? :D

@JoviDeCroock you're right, I messed up and linked the wrong issue :man_facepalming:

We could introduce a delete tagName before we stick everything to the native node but that decision is not up to me

repo: developit/preact


text:
Add typescript child type checking

Closes #1086
It fixes the original issue from #1008, it will now type check correctly
It will also allow type inference, a basic example is provided in the tests that infers the type of num but it will be useful for anything like developit/preact-router#269

Sure thing, changed it to object @marvinhagemeister

Coverage remained the same at 100.0% when pulling d7d7b62b3efe21cf486f747f8e34ea7a0b0a2126 on Alexendoo:typed-children into feb96b50832f55abd9ac55ec549bd813feac9fc0 on developit:master.

Yeah this is correct @valotas it is always an array at runtime, this definition is necessary to appease the assumptions TSX makes, it's summed up nicely here https://github.com/developit/preact/pull/1116#discussion_r191113051
It may be possible in TypeScript 3 using LibraryManagedAttributes to do this correctly on both sides however, but I haven't explored this yet

@Alexendoo thanks a lot for the explanation! I'll have a look at ts3

authors: Alexendoo coveralls valotas


text:
GH Integration

Context
Optionally add support for a repo/GitSha to link back to for each iteration for models that have been automatically uploaded from Github events

GH Links have been implemented and are deployed to production.

repo: developmentseed/ml-enabler


text:
[WIP] test equivalency against mercantile.tile

The mercantile.tile and morecantile.tms.get("WebMercatorQuad").tile functions are returning slightly different values given the same lat/lon coordinate.  This is because mercantile uses slightly more precision when defining the bounds of the Web Mercator tiling scheme.  I updated the bounds of the TMS defined in morecantile/data to match the return of mercantile.xy_bounds(mercantile.Tile(0,0,0)).  You'll notice that the first build fails, and tests are fixed when making this change.
This change means our WebMercatorQuad deviates slightly from what is defined by OGC (https://docs.opengeospatial.org/is/17-083r2/17-083r2.html#62) but I think its worth it to align with mercantile.
I also had to move the antimeredian test case into its own function, for some reason it fails when its included in the larger test case 🤷

doing this over #81

authors: geospatial-jeff vincentsarago

repo: developmentseed/morecantile


text:
Add use_epsg parameter to WMTS endpoint

What I am changing
Adds a use_epsg parameter to the WMTS endpoint, which changes the format of the CRS for the tile matrix set. This is needed to support ArcMap, which doesn't like the opengis.net-style CRS in the SupportedCRS.
Also adds an escape to the URLs in the WMTS template, as multiple query parameters were breaking XML parsing b/c of the &.
How you can test it

I've added unit tests. To manually test, you can hit a titiler WMTS endpoint with the parameter, e.g. http://localhost:8000/cog/WMTSCapabilities.xml?url=your-url-here&use_epsg=true and check out the SupportedCRS field.
Related Issues

Should resolve https://github.com/developmentseed/titiler/discussions/732

Manual testing with ArcMap
This is an ArcMap WMTS configuration against titiler endpoint without any modifications:

The layer doesn't load:

Adding the use_epsg parameter:

It works!

thanks a lot @gadomski, really nice PR 🙏

authors: gadomski vincentsarago

repo: developmentseed/titiler


text:
Switch to arch linux

Automate arch installation procedure.
Problems

root perm might cause some issues

https://wiki.archlinux.org/title/Installation_guide

Update system clocktimedatectl set-ntp true

Formatting and partitioning disk

GPT partition, 300MB FAT32, rest - 15GB padding ext4 (no swap)
Give the option to go with the default setting or do it manually
mount fat32 partition to /efi

mkfs.fat -F32 /dev/sdxY
mkfs.ext4 /dev/sdxY

mount root partitionmount /dev/sdXn /mnt

Install essential packagespacstrap /mnt base linux linux-firmware vim NetworkManager grub efibootmgr

generate file system tablegenfstab -U /mnt >> /mnt/etc/fstab

chrootarch-chroot /mnt

set time zone to Asia/Seoul and generate /etc/adjtimeln -sf /usr/share/zoneinfo/Asia/Seoul /etc/localtime
hwclock --systohc

uncomment en_US.UTF-8 UTF-8 in /etc/locale.gen and run:locale-gen

create /etc/locale.conf and write LANG=en_US.UTF-8
create /etc/hostname and write pomp
create /etc/hosts and write:127.0.0.1  localhost
::1        localhost
127.0.1.1  pomp

set passwordpasswd

create bootloadergrub-install --target=x86_64-efi --efi-directory=/efi --bootloader-id=GRUB

edit /etc/default/grub and generate /boot/grub/grub.cfggrub-mkconfig -o /boot/grub/grub.cfg

reboot

Related material:
https://github.com/MatMoul/archfi
https://gitlab.com/anarchyinstaller/installer

Root permission might cause some issues. Imma just use archfi for installation lmao.

repo: developomp/setup


text:
Admin user can delete himself when there are no other admin users

Admin user can delete himself from the admin UI when there are no other admin users.
I think expected behavior would be returning a "bad request" error when the API call is executed in such a case.

Wow, this looks like a very rare case. I would not mark it as a bug specifically. It is a foolproof enhancement I think.
Let's make it a lo priority ticket.

And you also can delete default access key in playground. And after it your playground will be broken :) Next log in attempt through - "Can't create and init a User"

This one I'm going to address. The playground should recreate key if it's missing.

@SorJEF @demon-xxi @Nikolay-Kha @zubrabubra
Hello, guys. I believe we should forbid users to delete themselves and their last default (means token type here, not token, initially created by playground) access token instead of recreating it.
Admin console does not require particular access token to authenticate but it needs at least one access token of default type with any set of permissions to authenticate user.
I implemented the following logic:
(current something means an entity used to authenticate request)

Deleting current user is forbidden
Deleting owner of current access token is forbidden
Deleting the last default access token of current user or someone else is forbidden, even for Admin role
Default access token can be deleted if its owner has at least 2 default access tokens

One of the following messages with HTTP code 403 is returned if user encounters these constraints:

You can not delete current user or owner of current access key
You can not delete your last default access key

I covered these changes and some related logic with mocha tests.
I think we should also hide \ disable "Delete" button in admin console if user has 1 access token in total.
Please, let me know if you have any objections or suggestions regarding this solution.

authors: Nikolay-Kha SorJEF demon-xxi mdolbin

repo: devicehive/devicehive-java-server


text:
YouTube auth tokens can't be retrieved

I have been getting this error since yesterday.
15:02:13.728 lava-daemon-pool-info-loader-2-thr            YoutubeAccessTo WARN   YouTube auth tokens can't be retrieved because email and password is unset in YoutubeAudioSourceManager, age restricted videos may throw exceptions.
15:02:14.114 lava-daemon-pool-info-loader-2-thr            YoutubeSignatur DEBUG  Parsing player script /s/player/018e9916/player_ias.vflset/en_GB/base.js
15:02:14.172 JDA Gateway-Worker 1                          WebSocketClient DEBUG  Sending voice request {"op":4,"d":{"self_deaf":false,"guild_id":750554274865479723,"channel_id":1152627695688105984,"self_mute":false}}
15:02:14.509 lava-daemon-pool-info-loader-2-thr            YoutubeAccessTo INFO   Updating YouTube visitor id (current is null).
15:02:14.547 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Using encryption mode xsalsa20_poly1305_lite
15:02:14.578 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Unknown Audio OP code.
{"op":18,"d":{"user_id":"692598549484929052","flags":2}}
15:02:14.578 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Unknown Audio OP code.
{"op":18,"d":{"user_id":"1024478956952502312","flags":0}}
15:02:14.578 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Unknown Audio OP code.
{"op":20,"d":{"user_id":"692598549484929052","platform":0}}
15:02:14.578 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Unknown Audio OP code.
{"op":20,"d":{"user_id":"1024478956952502312","platform":0}}
15:02:14.608 JDA AudioWS-ReadThread (guildId: 7            AudioWebSocket  DEBUG  Audio connection has finished connecting!
15:02:14.646 lava-daemon-pool-info-loader-2-thr            YoutubeAccessTo INFO   Updating YouTube visitor id succeeded, new one is CgtySHBLT2hvMDFNUSjG-7OyBjIKCgJDQRIEGgAgQzoMCAEg-6To2OS4v6Zm, next update will be after 600 seconds.
15:02:14.646 lava-daemon-pool-info-loader-2-thr            DefaultYoutubeT DEBUG  Loading track info with payload: {"context":{"thirdParty":{"embedUrl":"https://google.com"},"client":{"androidSdkVersion":30,"clientName":"ANDROID","clientScreen":"EMBED","clientVersion":"18.11.34","visitorData":"CgtySHBLT2hvMDFNUSjG-7OyBjIKCgJDQRIEGgAgQzoMCAEg-6To2OS4v6Zm"},"user":{"lockedSafetyMode":false}},"racyCheckOk":true,"contentCheckOk":true,"videoId":"VYJGMftYtY4","playbackContext":{"contentPlaybackContext":{"signatureTimestamp":"19863"}},"params":"CgIQBg"}
15:02:14.709 lava-daemon-pool-info-loader-2-thr            DefaultAudioPla DEBUG  No matches for track with identifier https://music.youtube.com/watch?v=VYJGMftYtY4.
no match
15:02:15.075 JDA RateLimit-Elastic-Worker 2                Requester       DEBUG  Received response with following cf-rays: [887729bb8f362cd1-ORD]
15:02:15.076 JDA RateLimit-Elastic-Worker 2                RestRateLimiter DEBUG  Caching bucket hash POST/interactions/{interaction_id}/{interaction_token}/callback -> 432f6223265d156004ac81366c5a948f
15:02:27.086 JDA RateLimit-Scheduler-Worker 2              RestRateLimiter DEBUG  Removed 2 expired buckets

Testing a fix now, hoping to PR soon

Testing a fix now, hoping to PR soon
UPDATE:
Looks like their API is now expecting "device_code" instead of just "code" in the Auth Token Request Constant. Still working on it here to see if this would also fix non-logged in users.

Thank you for the update and your hard work on this fix!

Does that mean i can't play yt videos until the code gets updated?

The built-in source manager is deprecated as of 1.9.0 (release still pending). It will not be receiving any further updates as its replacement is youtube-source. To use with my Lavaplayer fork, you will need the common package. Logging into accounts is not supported with that source due to issues with authentication, security concerns and the maintenance needed to support such a feature.

authors: Splayfery devoxin lMolMol pablo67340

repo: devoxin/lavaplayer


text:
style: (frontend)Added background image issue-125

close #125
Fixes Issue
Added background image only
Do check out , and inform if any changes required.   : |)

@AbhiPatel10 I feel this background in good but this should cover the whole website what do you say?

@AbhiPatel10 I feel this background in good but this should cover the whole website what do you say?

In my opinion img covering whole website will look terrible. (Just suggestion: In other parts gradients  colors can be applied)

I had added full background to the website in recent push , is it ok or previous one was better .
Do inform if  changes req.

There was no response so I thought to put on chats ...I had added full background to the website in recent push , is it ok or previous one was better .
Do inform if  changes req. @himanshu1221

Hey @ManishaSwain8 as i am a bit occupied with my exams so wont be able to Review it
@AbhiPatel10 will do it shortly or else I'll try to do it by the end of the weekend

There was no response so I thought to put on chats ...I had added full background to the website in recent push , is it ok or previous one was better .
Do inform if  changes req. @himanshu1221

Hey @ManishaSwain8 as i am a bit occupied with my exams so wont be able to Review it @AbhiPatel10 will do it shortly or else I'll try to do it by the end of the weekend

ohh that's not a problem at all , I was just making sure.
Best of luck : |)

We do not need to add any background image as it will slow down the website and it's not as per our website design

repo: devs-in-tech/DevsInTech


text:
Attach events to textarea

I want to handle an event @keyup.enter, but the component doesn't pass along attributes. Can this be added to the textarea tag in the component:
<textarea
    ...
    v-bind="$attrs"
></textarea>

This allows me to use the component in my code like so:
<textarea-autosize
    class="message-text"
    v-model="messageInput"
    :max-height="100"
    @keyup.enter="sendMessage"
  ></textarea-autosize>

I think you should use @keyup.enter.native
https://vuejs.org/v2/guide/migration.html#Listening-for-Native-Events-on-Components-with-v-on-changed

authors: Saymon-biz jwynveen

repo: devstark-com/vue-textarea-autosize


text:
Proposal: Support to download the plugin files from a proxy server

Description
When I try to use dtm to do something. I got the connection timeout errors. It might be pretty common for the Chinese users, the connection to GitHub release URL is not stable in China.
Describe the Proposal
So, I am thinking if it's possible to let dtm downloads those files from a proxy server. Of course, it would be better to let users choose this behavior. For example, we can provide a flag to choose whether to use a proxy server. This flag would be a global flag:
dtm --gh-proxy xxx init xxx
I have already done that in this project http-downloader. Please feel free to write your comments below. I'd be happy to improve this if you believe this is ok.

By the way, in case of anyone wants to try download dtm using the tool which I mentioned. You can try this command: hd install stream

Has the Feature been Requested Before?
Nope.
Describe the Alternatives You Have Considered
Nope.
Additional Context
Nope.

You are right, using dtm in China will indeed face network problems, and we have also thought about whether to store all the plugins in a domestic cloud storage.
If we are using a proxy, there may be two ways.

One is to add a flag to specify the proxy as you said,
And the other is to directly export http_proxy ... to implement the proxy mode without intrusion.

For users who do not have proxy software in hand, they may need the cloud storage I mentioned earlier to solve the download problem.
For the other users, maybe the export ... method simpler?

As for the download of dtm(DevStream cli) itself, maybe hd is a good way, I don't know what other people think. @devstream-io/pmc

There are a few free proxy servers that we can use. See also here.

There are a few free proxy servers that we can use. See also here.

It seems cool!

I think it's ok to add a --gh-proxy or --proxy flag to let dtm downloads plugins from a proxy server. We could set a default free proxy server and reserve the right to choose which proxy to use with this flag.

This is actually a great idea.
I think we should use --proxy instead of --gh-proxy because in the near future we might need to publish the artifacts to AWS S3 or equivalent instead of GitHub. So the --proxy fits future plans better.
@daniel-hutao please create a simple and small design for this feature, and thanks a lot for @LinuxSuRen this great contribution!

Hi Rick @LinuxSuRen , Thank you for the great proposal!
I wrote some considerations here, maybe not being completely accurate. But I hope these suggestions will help you.

Flag name: proxy(--proxy), maybe -p is not needed, because -p is easy to confuse with port
This flag should be global, not used by init alone, because other commands may also need to download some "dependencies". (rootCMD.PersistentFlags())
The default value with this flag should be a free proxy address mentioned in your comment above.

hd install dtm is indeed a good solution to resolve the network problem of dtm itself, what is your opinion? If you agree with this approach, perhaps we can document this approach in the "installation step" at README.md in a separate pr.
If we decide to do it, of course the most suitable performer for this task is @LinuxSuRen . But hd itself needs to be downloaded from github, so, you know what I mean.

hi @daniel-hutao , thanks for your details. That's great.
Consider the use case of downloading a file from the GitHub release is pretty common. I'm thinking about if it's possible to reuse http-downloader project. That means if it's possible to do the most work in the project hd, than reuse it in the devstream project.

Wait a minute, just hang up first, maybe my previous answer was not rigorous enough, let's discuss internally more for the best practices, and I'll update the "final" suggestion here later.

After second thought, I don't think depending on a few fixed third-party proxies like "gh.api.99988866.xyz", "ghproxy.com", "mirror.ghproxy.com", etc., is the best we can do because they might be subject to change.
I think a more generic solution would be to publish *.so files in an object store inside the GFW. What do you guys think?

It's ok to publish those plugin files to any place. But for a contributor, it is best to let all people can see the files easily. This can allow more people to be able to improve it. And I don't recommend having multiple places for different people who are in different countries. The potential users and contributors might come from any place. To make it be short, I think the GitHub release is good enough.
Of course, this is just my personal perspective.

@LinuxSuRen after a discussion with Daniel, we decided to wait a little bit before implementing this feature, mainly because of another high-priority task. Please keep this thread open, and let's revisit it again soon (hopefully in 2 weeks).

See also #23

See also #23

So, I guess we can track this issue in #23 . And I can close this issue now. Right?

/close #444

authors: IronCore864 LinuxSuRen daniel-hutao

repo: devstream-io/devstream


text:
⚠ Mobitopup has degraded performance

In a74e2f9, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3669 ms

Resolved: Mobitopup performance has improved in 807f742 after 7 minutes.


text:
Clients can not view player sneaking state

Players can not view others sneaking.

This seems to be because the sneaking/sprinting actions aren't being handled in the PlayerAuthInput handler

That's odd though, because these actions are supposed to be handled currently.

authors: DaPigGuy JustTalDevelops Sandertv

repo: df-mc/dragonfly


text:
Where is the WCFace.dll file ?

Dear,
According this line : Place the WCFace.dll file inside your ...\AppData\Roaming\VRCFaceTracking\CustomLibs folder. You can get here by pressing Windows+R and typing in %appdata. You can create the folder if it's missing.
I still didnt find the file WCFace.dll anywhere.
Thanks for your help.

Hello, please find WCFace.dll inside of the latest release. The source does not contain this

Thanks

repo: dfgHiatus/NoVRCFT


text:
Radio toggle component

Motivation
A radio group component to be used first on the proposals page.
Changes

RadioToggle compoent

Test

RadioToggle

Screenshots

Can we perhaps use Segment and SegmentButton, like we use here to switch between ckBTC and Bitcoin on the receive modal?

I think we should be careful about adding new design elements every time we add a new feature and try to reuse components as much as possible. Would be could if you could check with PM/UX if this can be reused. Maybe by adjusting either the existing component or the new feature. Thanks!

Closing this PR as it’s no longer relevant.

authors: dskloetd mstrasinskis

repo: dfinity/gix-components


text:
feat: sns deploy network flag

Description
Currently dfx sns deploy works only on localhost.  This PR enables the standard --network flag and passes it through.
How Has This Been Tested?

The dfx sns deploy test is currently disabled in CI to to version alignment issues between the IC and SDk repos, however I have added a comment to exercise the --network flag when that test is re-enabled.
I have tested dfx sns deploy --network local using this code in the snsdemo5.  At least 3 other people have gone through snsdemo5 successfully.
I could not test the same against testnets at the time due to boundary node issues.  Once all the relevant parts are working and merged, probably in one week, the snsdemo can be exercised against testnets.

Checklist:

[x] The title of this PR complies with Conventional Commits.
[x] I have edited the CHANGELOG accordingly.
[x] I have made corresponding changes to the documentation.

Closing as new work on this is planned to be in another repository.

repo: dfinity/sdk


text:
Feature/speedup base64

TODO:

[x] look for better variable name (over NUM_CACHED_BUFFERS)

Beside the base64 speedup, I added

an example for compressor selection in 9ee73e3
example traits implemetations in 195ecc2 (see https://github.com/dglaeser/gridformat/blob/feature/speedup-base64/docs/pages/traits.md)

repo: dglaeser/gridformat


text:
Add way to drop data.

Today in Ratel there's a "DROP ALL" button to drop all the data and schema in Dgraph.
Dgraph also has a way to drop just the data (not the schema). This should be doable in the Ratel UI.
Here's how drop data would be done via curl:
curl -X POST localhost:8080/alter -d '{"drop_op": "DATA"}'

Fixed in 60d5c

authors: danielmai paulftw

repo: dgraph-io/ratel


text:
Add Glycerine into the "Data Annotation" section

Add "Glycerine" to the "Data Annotation" section.
Short pitch
Glycerine provides a suite of annotation tools and end-to-end workflows for researchers, curators and students to collaborate on projects across repositories. Images can be organized into collections and shared with collaborators. Sets of annotations can combine semantic tags from domain-specific vocabularies with critical analysis in multiple languages. Annotated images can be published as research outputs in immersive and engaging visualizations and archived in sustainable formats.
Checklist

[x] I have read and understood the contribution guidelines.
[x] Table of contents has been updated (if a section is added / removed).
[x] Contents have been sorted alphabetically.

@yangli0516 Out of curiosity, are there any research projects using Glycerine that you could share with us? I haven't heard of it or used it yet.

Hi @maehr   Glycerine was developed over the last year with funding from https://ardc.edu.au/services/ardc-community-data-lab/  and was launched just last month https://groups.google.com/g/iiif-discuss/c/jkknN6SJK4U.
It's being used on 3 projects so far: Gandharan Buddhas https://gandhari-texts.sydney.edu.au/collections/gandhari-inscribed-buddhas/,  Upper Indus https://upper-indus.performx.com.au/ and Opening Australias Multilingual Archive https://omaa-arts.sydney.edu.au/.   The initial focus has been on the GLAM sector in Australia, but we host the platform for international researchers as well accessed from https://glycerine.io/.  Best Ian

Hi @maehr apologies in advance for chasing this, but I presume we are just waiting on a reviewer.  Is there anything else you might need from us?  All the best Ian

Hi @maehr apologies in advance for chasing this, but I presume we are just waiting on a reviewer.  Is there anything else you might need from us?  All the best Ian

Hi Ian
I requested reviews from other collaborators. I hope this will speed up the process.

Thanks @maehr

authors: IanMcCrabb maehr yangli0516

repo: dh-tech/awesome-digital-humanities


text:
[Feature]: Integrate in nvim and lazygit workflow possible?

Description
I really like to use lazygit from within nvim. Can opencommit be integrated into this workflow? Or am I missing a git feature?
Suggested Solution
Maybe a custom comand in lazygit is feasable?
Alternatives
No response
Additional Context
No response

havent used lazygit with nvim, you could fork -> investigate -> make a pr?

authors: di-sukharev rafo

repo: di-sukharev/opencommit


text:
Webhook for slot filling with inline parameters in training phrase

What is the proper way for using webhook for slot filling. It seems something is missing (we're missing official documentation on this the most). Apart from that, how to do webhook slot filling on parameters when they are in the training phrase. Seems like I cannot set the value to #context.param_value when the parameter is in the training phrase. So how should I handle this situation?

I realize now that on slots that are filled by training phrase and I can do processing on them in the final webhook call when parameter slots are filled. Hence, closing this here.


text:
[NO-MERGE] DLT-1689 add overlay scrollbars example

Custom scrollbars POC
Jira: DLT-1689
Adds the dependency Overlay Scrollbars to Dialtone to see how it works and what functionality it allows.
I created and exported a new directive DtScrollbarsDirective, that can be used similar to the DtTooltipDirective, just adding v-dt-scrollbars to the component that would normally have overflow: auto to allow scroll.
I tested it inside Dialtone in the Message input component, and it works both in Vue 2 and Vue 3, and also in Ubervoice in the leftbar component.
Check the code for Ubervoice here.
Testing in the Emoji Picker component I realized that it doesn't work correctly with components that add listeners to the scroll. To be able to use it we would have to use a Vue component instead of a directive and attach any listeners to the scrollbars component rather than the original component and that is probably overly complicated.
Message Input ✅

With custom scrollbars
Native scrollbars

Leftbar ✅

With custom scrollbars
Native scrollbars

Emoji Picker ❌

With custom scrollbars
Native scrollbars

Advantages of using Overlays Scrollbars over native scrollbars

Allows custom styling and ensures that the look and feel will be the same across browsers and platforms. Everything can be customizable: the size, color, border, transitions, etc.
Allows overlay scrollbars (in fact there is no option for not overlay scrollbars)
Offer a lot of options to customize them, like when they should appear (on scroll, on hover or all the time)
It would be very simple for developers to use them in simple components (below there's a note on the Conversation View component*)

Disadvantages

It's adding an external dependency for something that exists natively (although we don't have full control of the styles).
Doesn't work as expected with components that listen to the scroll event, like the Emoji Picker an the Conversation View.

Accessibility

They give control over the contrast and can work both in light and dark mode.

The size can increase on hover (imitating the native behavior) for better accessibility. This is customizable too.

Keyboard navigation: the native behavior is supported. These are the guidelines (from https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles/scrollbar_role):

When focus is within the viewport controlled by a scrollbar, the Up Arrow and Down Arrow (or Left Arrow and Right Arrow for a horizontal scroll bar) should move the scroll bar thumb proportionally. Additionally, the Page Up, Page Down, Space, and Shift + Space keys must move the content and the scroll thumb the height (or width) of the viewport for each key press until the bottom or top (or left or right) of the content is in view.

Aria role: scrollbar

Because native scroll bar styling has historically been limited, you may come across a scrollbar implemented in JavaScript that you need to support and make fully accessible. For this, you can use the scrollbar role to inform assistive technologies that a UI control is an interactive scrollbar.

I don't think this role is added, and I didn't find a way to add it so this might be a problem for screen readers.
Next Steps

*Test the directive in the Conversation View. I tried to do this but when adding it, the feed was no longer populated, because it's an InfiniteList component and it's listening to scrolling events, so it might be more tricky to use it here.
Test in different browsers and platforms, and different inputs, like touch on a mobile device
Decide if the overlay is the style we want for the scrollbar
Decide if we want to go with this solution and not add it to every component (ir can't be used with components that listen to the scroll). Or if we want to find a workaround to make it work with these components (that would make it more complicated to use).
Investigate other libraries. https://binaryify.github.io/vue-custom-scrollbar/en/ was one I looked at but discarded since it hasn't been updated for two years

Are we able to show the scrollbar on mouseover of the scrollable region rather than when you actually start scrolling? I think this would make it a bit more obvious that the region is actually scrollable.

Are we able to show the scrollbar on mouseover of the scrollable region rather than when you actually start scrolling? I think this would make it a bit more obvious that the region is actually scrollable.

Yes, that is posible by setting the option scrollbars.autoHide to leave:

Disadvantages
It's adding an external dependency for something that exists natively (although we don't have full control of the styles).

Not really a problem at all, we've investigated native functionality before and found it was actually impossible to do what we needed to do.

Doesn't work as expected with components that listen to the scroll event, like the Emoji Picker an the Conversation View.

We can certainly release this initially the way it is, however it would be nice if we had sort of alternative way to properly use events. Would it work if we had a directive as well as a component like we do for tooltip? The conversation view is a place where we would like to use this scrollbar if possible. (i know it's a mess)

Not really a problem at all, we've investigated native functionality before and found it was actually impossible to do what we needed to do.

Yes, I meant the functionality, not the styles.

We can certainly release this initially the way it is, however it would be nice if we had sort of alternative way to properly use events. Would it work if we had a directive as well as a component like we do for tooltip? The conversation view is a place where we would like to use this scrollbar if possible. (i know it's a mess)

I would need more time to explore this possibility with a Scrollbar component, I wanted to first get some feedback to decide if it's worth to explore that. But yes, I could certainly continue with the investigation to add that possibility.

The implementation was merged here: https://github.com/dialpad/dialtone/pull/391

authors: braddialpad ninamarina

repo: dialpad/dialtone


text:
add string.split function for empty string delimiters

PR for #280
added functionality to split string into individual characters if empty string is passed as delimiters

Perfect, thank you very much!

repo: dictu-lang/Dictu


text:
R plugins not found

R elements are not loading in many browsers. I have checked Firefox's console output and it looks like R plugins cannot be found:
console-export-2021-4-29_10-51-51.txt
German hands up:

Die Ressource von "file.js" wurde wegen eines MIME-Typ-Konfliktes ("text/plain") blockiert: The resource von "file.js" has been blocked due to a MIME-type conflict ("test/plain")
Laden fehlgeschlagen: Loading failed

Can you please recheck? I think now is fixed (tested on Chrome, Firefox, Edge and mobile iOS: Chrome and Safari)

I am trying something else... let me know if is fixed and if not I can roll back to the previous solution

authors: dieghernan rodrihgh

repo: dieghernan/RKI-Corona-Atlas


text:
[gha] migrate changes and matches actions from diem/diem

From latest main at https://github.com/diem/diem/commit/3481b5650d394d46eb89b9e17bfe080cecf32bdb
After this lands, need to change the actions in diem/diem to refer to these

/land

:(( no bors

repo: diem/actions


text:
[diemdb_bench] add prune window

Motivation
add pruner
Have you read the Contributing Guidelines on pull requests?
Y
Test Plan

@bors-libra land

repo: diem/diem


text:
Error compiling

The command I'm running is mvn clean install as stated in the README.md
Eventually the following error is thrown
[ERROR] Failed to execute goal org.codehaus.mojo:findbugs-maven-plugin:3.0.3:findbugs (check) on project smtpsampler: Unable to parse configuration of mojo org.codehaus.mojo:findbugs-maven-plugin:3.0.3:findbugs for parameter pluginArtifacts: Cannot assign configuration entry 'pluginArtifacts' with value '${plugin.artifacts}' of type java.util.Collections.UnmodifiableRandomAccessList to property of type java.util.ArrayList -> [Help 1]

I'm not familiar with maven and I'm not a java expert. Any help getting this compiled would be appreciated.
Thanks!

I was able to resolve the issue.
For anyone else who faces this.

Download Maven 3.0.3 https://repo.maven.apache.org/maven2/org/apache/maven/apache-maven/3.0.3/
Add the following inside the <mirrors> tag.

<mirror>
        <id>internal-repository</id>
        <name>Maven Repository Manager running on https://repo1.maven.org/maven2</name>
        <url>https://repo1.maven.org/maven2</url>
        <mirrorOf>*</mirrorOf>
</mirror>

repo: diennea/smtpsampler


text:
Filters for messages should show correct amount of messages

Description
After applying one filter, another filter still shows the amount of messages as if no other filters applied.
Better user experience would be to show how many messages will be shown if that filter applied considering other filters.

Additional Information
No response
Tasks
No response
Acceptance Criterias
No response

Test;
Søk + Filter = OK

Kun et filter = OK

Filter + Filer = OK

Filter + Filter + DatoFiler = Failed

Det samme skjer med Filter + DatoFilter.

@mbacherycz kan du sjekke i koden om Dato filter ikke tar hensyn til resultatet av de andre filterne?

Observasjonen er logget som #1494
Så lenge 1494 er uløst holdes 1383 på teststatus failed.

Test:
Basert på siste dagers diskusjoner på hvordan dato som intervall skiller seg fra andre felter som "avsender" og "status" som filterkriterie er det tydelig at feil som går på dato skiller seg fra mer generelle feil.
Det generelle med telling av antall dialoger et filtervalg vil presentere er løst generelt sett.
Det er en utfordring knyttet til dette for datovelgeren. Men #1494 er kun en av flere utfordinger med datovelger.
Ny konklusjon er at #1383 løser problemet med telling av dialoger i filtervalg generelt og teststats er derfor passed.

authors: LeifHelstad mbacherycz

repo: digdir/dialogporten-frontend


text:
[Minor] The Cruisers steering wheel does not sync to other clients

Slightly nitpicky but something I did notice, for anyone who's not the driver, regardless if they have this mod installed or not, will still see the steering wheel desync'd on their end.
Additionally, there also appears to be an issue where only the hand animations of turning the wheel play for other clients while the vehicle is stationary/not moving at all, the wheel will desync further if you start moving and start turning any direction, whether this is related to the problem above I've no idea

I've also noticed that the steering wheel (except its animations) do not update if the Cruiser is not moving if that helps

I've noticed this too and looked into it, unfortunately it's not an easy fix and I don't think it's possible without requiring all players to have the mod.
The RPC the game uses to sync the steering angle (VehicleController.SyncCarPositionClientRpc) only sends steering control input instead of the actual wheel position, and only every time the car is moved a certain distance from it's last spot. This just isn't enough information to attempt to synchronize the steering angle over the network.
Could be done by requiring other players to also use this mod but currently the goal is to keep this mod clientside.

I've noticed this too and looked into it, unfortunately it's not an easy fix and I don't think it's possible without requiring all players to have the mod.
The RPC the game uses to sync the steering angle (VehicleController.SyncCarPositionClientRpc) only sends steering control input instead of the actual wheel position, and only every time the car is moved a certain distance from it's last spot. This just isn't enough information to attempt to synchronize the steering angle over the network.
Could be done by requiring other players to also use this mod but currently the goal is to keep this mod clientside.

Yeah, I even tried to look into it a little myself although I am no coder, and I suppose it's fair if the goal is to keep it as client-sided as possible

This has been implemented in v1.2.0 with the addition of synced configs and networking. If both host and client have the mod, the steering wheel will now sync properly.


text:
Remove JQuery from Widget media

I see that you are loading JQuery with the widget media. I wonder if this is the ideal thing to do because, I think, there is a huge possibility that JQuery have already been loaded before on the template and therefore JQuery would be downloaded multiples times. What do you think?, maybe saying in the docs that JQuery must be loaded before the widget would be enough.

https://github.com/digi604/django-smart-selects/issues/135#issue-161036050

Done in f474f47874cda15fb2a58a90473530b81ab73ce5.
JQUERY_URL = False

Closing.

authors: blag lpalomo webknjaz


text:
Execution directives tests

Porting all execution directives tests from graphql-js.
Improve collectFields method

Pull Request Test Coverage Report for Build 774

12 of 13 (92.31%) changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.3%) to 81.338%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Execution/ExecutionStrategy.php
12
13
92.31%

Files with Coverage Reduction
New Missed Lines
%

src/Execution/ExecutionStrategy.php
2
74.55%

Totals

Change from base Build 772:
0.3%

Covered Lines:
3914

Relevant Lines:
4812

💛  - Coveralls

Pull Request Test Coverage Report for Build 774

12 of 13 (92.31%) changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.3%) to 81.338%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/Execution/ExecutionStrategy.php
12
13
92.31%

Files with Coverage Reduction
New Missed Lines
%

src/Execution/ExecutionStrategy.php
2
74.55%

Totals

Change from base Build 772:
0.3%

Covered Lines:
3914

Relevant Lines:
4812

💛  - Coveralls

authors: coveralls hungneox

repo: digiaonline/graphql-php


text:
HYP 158 What We Do Section

name: What We Do Section
about: Added the what we do pop up section
Summary
This is in connection to the Jira ticket HYP-158

Motivation
To make the GUI as close as the design as possible

Describe alternatives you've considered
Close button without X

Additional context
A pop up / dialog box will appear when the top navigation bar “What We Do” is clicked.
The pop up box will have green header, and the box will be light background, with text on top.
A close button on the bottom to close the dialogue box OR a X on the top right. Added BOTH close and x-button

Updated styles and layout to make things simpler
https://github.com/digicatapult/sqnc-hyproof-client/assets/29942957/06191831-b879-45f1-8399-1f1c16f2d20e

authors: dc-andysign mattdean-digicatapult

repo: digicatapult/sqnc-hyproof-client


text:
Revert "remove links to deleted examples page (#209)"

This reverts commit 5bb797397999fa184c4687720b1d97023409e10f.
The link is no longer dead.

https://daml.com/examples looks dead again :thinking:

https://daml.com/examples looks dead again 🤔

Thank for reporting! We're looking into it.

authors: garyverhaegen-da stas-sbi


text:
search_daily(date, sortkey)

model.mdate: date와 값이 같다면 반환.

0614 로만 검색어를 넣었을 때. 작동됨.


text:
Error when platform has bcmath but not gmp extension

@digitaldonkey digitaldonkey
If I run the sample code in Readme.md it causes the following error if the server/platform running the code does not have the php-gmp extension installed, but does have php-bcmath installed.
The dependency simplito/elliptic-php  requires either gmp or bcmath to be installed as you can see in BigInteger.php.
First it tries to do the operation using gmp but if gmp is not installed it uses bcmath as a fallback.
This problem can be fixed (within this repo) by removing the '0x' that is prefixed to the $message_hash in EcRecover.php on line 20.
So this line
$message_hash =  '0x' . Keccak::hash(self::personalSignAddHeader($message), 256);

changes to this
$message_hash = Keccak::hash(self::personalSignAddHeader($message), 256);

Once that change has been implemented the sample code in Readme.md runs fine using both the gmp and bcmath methods.
I'm not sure if the proper solution here is for this repo to be modified as described above or if simplito/elliptic-php should somehow be modified instead. However if you look at the the README.md for simplito/elliptic-php under "Verifying Ethereum Signature" you can see that the example does not prefix the hash with '0x'.
For reference here is the exact code I am running that causes an error. This is the same code from Readme.md, just with a couple typo fixes.
        use Ethereum\EcRecover;

        $address = '0xbe93f9bacbcffc8ee6663f2647917ed7a20a57bb';
        $message = 'hello world';
        $signature = '0xce909e8ea6851bc36c007a0072d0524b07a3ff8d4e623aca4c71ca8e57250c4d0a3fc38fa8fbaaa81ead4b9f6bd03356b6f8bf18bccad167d78891636e1d69561b';

        // Verify known address
        $valid = EcRecover::personalVerifyEcRecover($message,  $signature,  $address);

        // Recover unknown address
        $recoveredAddress = EcRecover::personalEcRecover($message, $signature);
        if ($recoveredAddress === $address) {
          echo 'Jay! it was a long way here. ';
        }

This is the error I am getting:
Exception
Invalid characters
vendor/simplito/bigint-wrapper-php/lib/BigInteger.php:328

@brianrlewis Thank you for your contribution. I'm having very little time for this. I leave it open for reference, or till someone provides a PR and I find some time to get back into it.

authors: brianrlewis digitaldonkey

repo: digitaldonkey/ecverify


text:
Change sentence "Falls Sie denken, die Angaben seien fälschlicherweise, hätten wir Sie gemacht, die Angaben zu widersprechen."

There is a wrong sentence reported by a user (maybe in one of the mail templates): Falls Sie denken, die Angaben seien fälschlicherweise, hätten wir Sie gemacht, die Angaben zu widersprechen.
Don't know where is occurs but needs to be changed to a correct (easy understable) German sentence

@dkehne i really don't know where they got that from but in our verification template the following is written:
"Sie wurden gebeten, die Angaben eines Antrags auf Ehrenamtskarte zu bestätigen. Die Antragsstellerin oder der
Antragssteller hat Sie als Kontaktperson der Organisation {verification.organizationName} angegeben. Im
Folgenden können Sie den zugehörigen Antrag einsehen. Wir bitten Sie, die enthaltenen Angaben, welche die
Organisation {verification.organizationName} betreffen, zu bestätigen. Falls Sie denken, die Angaben wurden
fälschlicherweise gemacht, bitten wir Sie, den Angaben zu widersprechen."
I don't think we have to change anything. If you have a proposal for an easier sentence just let me know

we don't use this sentence in any mail

repo: digitalfabrik/entitlementcard


text:
ITMS-90683: Missing purpose string

Check this task from AppStoreConnect,
Note that we also get this message for other apps even we provide this string
ITMS-90683: Missing purpose string in Info.plist - Your app’s code references one or more APIs that access sensitive user data, or the app has one or more entitlements that permit such access. The Info.plist file for the “Runner.app” bundle should contain a NSLocationAlwaysAndWhenInUseUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. If you’re using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. For details, visit: https://developer.apple.com/documentation/uikit/protecting_the_user_s_privacy/requesting_access_to_protected_resources.

already fixed in 3.2.3


text:
Birthday Surprise

As a user I want to be positively surprised by the app, if I open it.
Acceptance Criteria:

[ ] Be happy

Technical Tasks:

[ ] Don't worry

I can still close and reopen!

I don't have birthday but it is still a really nice suprise, thank you guys 😊 I am definitely happy and don't worry :)

We need virtual balloons for when it's the user's birthday (as taken from their sozialpass). however, we should definitely also include an Easter egg on Steffen's birthday 🎂🎉🥳

authors: maxammann michael-markl steffenkleinle


text:
2140: Foreground push notifications

Short description
Letting push notifications appear in the foreground too.
Proposed changes

Installed Notifee which seems quite well-supported
Used that to show a notification in the foreground if one comes in while the app is open
Added a notification icon for Android (it needs to be black-and-white, and with a transparent background)

Side effects
Should be nothing
Notes for testing
I have not been able to receive notifications in emulated iOS, only on my real iPhone.
On Android, I have only been able to test on emulated Android, it would be cool if we could have at least one test on a real Android device.
And in both cases, the test CMS takes too long until the notification is in the news, so you see the error screen for a good second or two before the actual content appears. This seems to not happen in the real CMS though.
Resolved issues

Fixes: #2140

@f1sh1918 The test CMS takes too long until the notification is in the news, so you see the error screen for a good second or two before the actual content appears. This seems to not happen in the real CMS though.

@f1sh1918 The test CMS takes too long until the notification is in the news, so you see the error screen for a good second or two before the actual content appears. This seems to not happen in the real CMS though.

hm i don't think this is the issue i get also an error
The 'navigation' object hasn't been initialized yet. This might happen if you don't have a navigator mounted, or if the navigator hasn't finished mounting. See https://reactnavigation.org/docs/navigating-without-navigation-prop#handling-initialization for more details.

Hm i really have no idea how to fix this. Pushed some time into it.
I mean if the cms needs some time to get the push notification in the api. Shouldn't it help to setTimeout around the notifee.displayNotification to just wait until we display it instead of timeout the onForegroundEvent
Nevertheless even more timeout doesn't help
I even checked the onReady function of the NavigationContainer and passed a prop to check if its ready which seem to be the case.
I also tried to set the ref but looks like i can not do it in combination with linking (deeplinking)

You're right, it's more than just the CMS taking some time. I'm currently trying to handle the tapping the same way as background notification is handled, by opening the redirect route but that screen somehow never opens. It's a weird issue. I'll keep trying 🤷

You're right, it's more than just the CMS taking some time. I'm currently trying to handle the tapping the same way as background notification is handled, by opening the redirect route but that screen somehow never opens. It's a weird issue. I'll keep trying 🤷

hm maybe the solution is good enough. Even i think this timeout is not very stable :/

Oh. All good, thank you! But it's frustrating to not know why it doesn't work with the Test CMS

I just tested removing the timeout fully and using the real CMS, and it did show the error message. So it looks like something like the timeout is still necessary 😞

repo: digitalfabrik/integreat-app


text:
Add dev tool for ruff

Short description

This PR adds a dev tool for ruff
Proposed changes

Add dev tool for ruff

Side effects

At the moment we use ruff still in parallel to the other formatters and linters. AFAIK over time we want to move towards using only one?
Yet only check is enabled also with fix as extra parameter. Maybe over time we want to include formatting and also include the option to pass arguments such as --fix manually

Resolved issues

Fixes: One part of #2442

Pull Request Review Guidelines

Hmm, as described in #7242, I think adding more dev tools to perform tasks that already can be done by one pre-commit command is kind of superfluous...

Okay. Do you suggest to close this then? :)

Hmm, as described in #7242, I think adding more dev tools to perform tasks that already can be done by one pre-commit command is kind of superfluous...

Invoking the pre-commit manually in order to check for things you need to fix is a bit burdensome though, + the same could be said for most existing dev tools

Okay. Do you suggest to close this then? :)

Yes, I personally would suggest that, but I'm open for other opinions.

Invoking the pre-commit manually in order to check for things you need to fix is a bit burdensome though, + the same could be said for most existing dev tools

Hmm, I'd say that
pre-commit run ruff

is not much longer than:
./tools/ruff.sh

If you still think it's inconvenient, we could set bash aliases for this command rather than separate bash scripts?

My two cents: I basically ignored the pre-commit hooks entirely, always running the code style tools individually or as code_style.sh, because it took way too long and got in the way especially during rebases, if I remember correctly. So I at least got very accustomed to tools/. But my opinion on the pre-commit hooks might very well change once rust fully replaces most tools.

Hmm, I'd say that
pre-commit run ruff

is not much longer than:
./tools/ruff.sh

True, but then why not do the same for eslint, prettier, black, isort,...?
I'd wager most of us are used to using the dev tools for these tasks, and I am having a little trouble seeing the benefit in changing this workflow.

My two cents: I basically ignored the pre-commit hooks entirely, always running the code style tools individually or as code_style.sh, because it took way too long and got in the way especially during rebases, if I remember correctly. So I at least got very accustomed to tools/.

👍🏼

I think we can just merge this, tools/ became kind of the default location I look for, well, tools like that. If it's not in a script, I just straight up assume it's not set up for the project.
If we organize this differently in the future, I guess it makes little difference whether to also have to delete this ruff.sh or not.

authors: JoeyStk PeterNerlich charludo timobrembeck

repo: digitalfabrik/integreat-cms


text:
Plugin getting slower

I've been noticing that the plugin has been feeling sluggish and it seems to be getting slower as I add more cards. There is a significant lag between the time I press an answer and the next question appears. I think the problem may be when the extension starts saving changes, because during the first card of the day there doesn't seem to be any issues. It seems to be happening when it starts syncing in the background.
Is there any way this could be addressed in future updates? The plugin is great in any case, thank you.

@jagr21 Just curious, how many cards did you have on review?

Currently I have to review approx. 20 to 30 cards daily out of a total of 370.

That's weird! It shouldn't be slow at all. Thanks for flagging tho - once I'm done with the old roam/sr migration tool I'll take a look at why this is happening.
In the mean time I'd make sure there isn't something else causing the slowdown (ie. Another plugin).
🙏🏽

Hi - quick check in here - are you still having this issue? If so mind sharing a bit more about exactly what's happening (i'm unable to repro any performance degradations on my end 😪). A screen recording of the issue would be super helpful 🙏🏽 Also, if anyone else coming through this issue has also been experiencing performance issues please also share <3

Hi - quick check in here - are you still having this issue? If so mind sharing a bit more about exactly what's happening (i'm unable to repro any performance degradations on my end 😪). A screen recording of the issue would be super helpful 🙏🏽 Also, if anyone else coming through this issue has also been experiencing performance issues please also share <3

Ok, here is a screen recording where you can see the problem. https://www.loom.com/share/1bbd10ce1135428faeed947e299987ae
See how long it takes to react after pressing the button to show the answer starting from the second flashcard. The issue has to do with the background sync, because if I just Skip flashcards there is no delay at all.

OH wow this is terrible 😆 ... Will take a look at what might be causing.. this is super helpful so thank you for posting 🙏

Re-opening until this fix attempt is released and verified. 🙏🏽

v12 has now been released... do a hard refresh and re-open this issue if it's still slow.
If it is I have a few other things I can try in my next release 🙏🏽

v12 has now been released... do a hard refresh and re-open this issue if it's still slow.
If it is I have a few other things I can try in my next release 🙏🏽

Sorry for not replying sooner, I haven't been able to do my reviews for a couple of days. I updated to v12 but I don't think there is a big difference :( As always, the first card doesn't have any problem, it's after the changes start to be saved that it slows down.
Thank you for taking the time to address this issue

Another hypothesis is that you're on a low-powered device. Do you have access to another computer that you can test this on?
Also, have you already tried temporarily disabling all other plugins and restarting the app? This will confirm that it's not another plugin causing this.

Another hypothesis is that you're on a low-powered device. Do you have access to another computer that you can test this on?

I also thought that could be the issue, but then I tried it on my phone and the problem is the same. And for example, roam/sr, with all its problems, worked very fast.
I'm sure that, at least in my case, the issue is due to uploading changes to the server. The problem definitely occurs when the program starts saving changes after giving that first answer. I don't know if it could be because it has to save too many changes after each answer. But is very clear: When the dot is red/orange the plugin has a noticiably lag. When the dot is green ("Everything is synced"), it works without a problem.

Also, have you already tried temporarily disabling all other plugins and restarting the app? This will confirm that it's not another plugin causing this.

I tried this but the problem persists

I also thought that could be the issue, but then I tried it on my phone and the problem is the same. And for example, roam/sr, with all its problems, worked very fast.

Got it. Thanks for checking. Can I ask, are you on windows or mac?

I'm sure that, at least in my case, the issue is due to uploading changes to the server.

I'm also sure that the issue is correlated to the save/sync operation (the hardest evidence I think is that skipping is smooth). Our challenge now is in finding the exact cause 🕵🏽♂.

I don't know if it could be because it has to save too many changes after each answer.

It's still strange because that operation should be non-blocking 🤔... My guess is that despite that, it could still be such a drag on the thread resources that it blocks the UI from rendering the next card block efficiently. In fact, now that I write this I have seen the entire UI freeze up when whenever I make to many changes at once (ex: if you paste in lots of data all it once.)
Let's test this hypothesis by delaying this saving event until after the UI transitions. 🤓

Got it. Thanks for checking. Can I ask, are you on windows or mac?

Windows.

Let's test this hypothesis by delaying this saving event until after the UI transitions. 🤓

Hoping to give it a try, sounds promising :)

Ok so this change just went live... refresh and lmk anything is different .. if not will keep digging (I have access to windows laptop I can test with) 🙏🏽

Ok so this change just went live... refresh and lmk anything is different .. if not will keep digging (I have access to windows laptop I can test with) 🙏🏽

Ok, just tested the new version and sadly I have to say I don't notice much of a difference :( The lag after clicking "Show answer" is the same. Maybe a little faster when selecting an answer, but I'm not sure. Thank you for trying to find a solution, in any case

No worries at all. This officially eliminates the hypothesis that it's directly related to the saving operation (just correlated, but not causal). You can confirm this by noticing the delay between when you grade a card and when the sync indicator turns from green to orange. If that's not happening then you're not on the latest version.
So my next hypothesis is that this has something to do with the Windows OS (likely a performance issue). BDW, I know you said you tested on a different device, but have you tested this on a non-window machine?

So my next hypothesis is that this has something to do with the Windows OS (likely a performance issue). BDW, I know you said you tested on a different device, but have you tested this on a non-window machine?

Sorry but I don't have access to a non-Windows pc. Only Windows and Android (phone).

No worries at all. This officially eliminates the hypothesis that it's directly related to the saving operation (just correlated, but not causal). You can confirm this by noticing the delay between when you grade a card and when the sync indicator turns from green to orange. If that's not happening then you're not on the latest version.

It's very strange. It still puzzles me that the first card to review does not present any problem. I also found that if between flashcards I wait (takes almost a minute to save changes from a single flashcard) until all changes are saved (sync indicator turns green) the next card works without any lag.

Ok, I just discovered something that may have solved the problem for me. Normally, I have always opened memo (in the side panel) from the Daily Notes page. Today I happened to launch it from another page and it turns out that there is no trace of lag or slowdown. I don't know why, but if you launch the extension from another page other than the Daily Notes page, it works smoothly 😆

Sorry but I don't have access to a non-Windows pc. Only Windows and Android (phone).

Now worries. Tested it on my partners PC and it worked smoothly.

... if you launch the extension from another page other than the Daily Notes page, it works smoothly

Interesting. I wonder if this has something to do with too many memo references being rendered. 🤔.. I collapse it so that it doesn't pollute your Daily Notes page with all these due cards but maybe that still expensive 🤷🏽♂.. not sure.
But glad you found a workaround. 🥳

Closing for now but I have a feeling i'll be coming back to this one at some point. :)

authors: digitalmaster jagr21 prampey

repo: digitalmaster/roam-memo


text:
Setting an IP as Primary via API

Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version:  3.6
NetBox version:  2.2.8

Description
I would like to set an IP as the primary IP when creating/updating an IP via the API.

Thank you for your interest in NetBox. GitHub issues are intended for reporting reproducible bugs and requesting features, and must be submitted using the template provided here. For general discussion, questions, or assistance with installation issues, please post to our mailing list.

🤔
Now that I look at the API again it seems, than what I requested is possible through the /api/dcim/devices/ endpoint.

authors: cimnine jeremystretch

repo: digitalocean/netbox


text:
Ability to collapse/minimize DHCP addresses in IP address view

Would like to see a way to collapse, hide, or minimize the IP's which are listed as DHCP for a prefix.  While it's important to track our DHCP scope(s), it's much less important to list all of them when 99% of the time we're working on entries for those outside of DHCP.

#834 would introduce IP ranges, replacing a consecutive set of discrete IPs with a single range object. But in the interim, it would make sense to include a simple filter panel to allow hiding IPs by status/role.

This would be a very handy feature!

Is there any word on when this change will be implemented? We have just started using NetBox in our environment and adding a DHCP range for a large number of IP addresses is messy when you need to scroll through multiple pages considering not all IP addresses are always in use.

Not sure why this issue is still open. We already have status and role filters for IP addresses. You can hide DHCP IPs by selecting only the other three statuses in the list.

Hi Jeremy,
Can you please be more specific regarding the ability to hide DHCP IPs?
The only option I can see is to sort by status.
Is the option to condense DHCP ranges such as in phpIPAM still in development and do you know if there is an ETA on this yet?
Thanks again for your help with this, it is much appreciated!

Would it be possible to hide all of the dhcp addresses when you look at the prefix's without having to filter them out each time, it is very tedious and messy atm. Or make it possible to condense all dhcp addresses into one object so you dont have to scroll through multiple pages. Thanks

authors: fourg jeremystretch mikeysouthwell netpanic snazy2000


text:
phpmyadmin

how to connect phpmyadmin
Thankyou

I haven't used phpmyadmin in ages, but if I were you, I'd start with their setup documentation.
https://docs.phpmyadmin.net/en/latest/setup.html

authors: kalamell mirorauhala


text:
Design fixes for RU language pack

Type of Change

Tool Source: Styling

What issue does this relate to?
N/A
What should this PR do?
Fixes some minor issues that were observed once the new Russian translations were added.
What are the acceptance criteria?
With the Russian language pack selected:

Consistent font-weight across Cyrillic and ASCII chars in the tool header
Consistent font-weight across Cyrillic and ASCII chars in the labels of checkbox options
Contribution callout (visible on scroll) button text wraps, and does not overflow

With the English language pack selected:

Ensure these changes haven't obviously caused any issues

I can commit more slim translations after successful review to make RU UI looks better :)
In Russian a lot of letters hah
For example Присоединяйтесь к проекту на GitHub to Перейти на GitHub = Get involved on GitHub to ~Come to GitHub ¯_(ツ)_/¯

@powerdot Go for it -- I don't think changes to the strings will have any conflicts with anything being changed here, so you're welcome to open a PR whenever and I'll take a look :)


text:
Documentation Isn't Accurate

I'm trying to use the examples from pynetbox, but the examples are very inconsistent.
It appears to be a mixture of the API from NetBox v1.x and 2.x but I can't seem to make any sense out of it.  (See code under heading of filter at the URL /en/latest/endpoint.html)
For example, I think that

nb.devices.filter(role='leaf-switch')

should be

nb.dcim.devices.filter(role='leaf-switch')

Also, I have a feeling that it might be due to the release of pynetbox v3.0.0 that caused this documentation error.  I'm wondering if it would be possbible to still be able to view the documentation for previous versions of pynetbox until this is resolved?

Sorry about that, thought I caught them all. Let me know if you see anymore.

Yeah, no worries, thanks for fixing that.
On a side note, am I incorrect in thinking that the documentation used to be more fleshed out than it is right now?  I thought I looked into this back in July sometime and was impressed with the available functionality but not in a position to attempt to use it then.  But now as I'm attempting to use the library, I see that the examples (for a newbie to python) aren't very understandable.
Also, I think I may have found a bug with nb.ipam.prefixes.create not handling passed list objects correctly, but not sure.  Should I open a new bug that or just post what I've come across on here?

I split them up into multiple pages in, I think, the 3.0.0 release. Before that it was just one big page, but they're still the same docs. They can certainly be better though. Let me know the places they're unclear and I'll see about fixing them.
As far as your issue with .create(), I started a thread on your mention of it in the NTC slack, but go ahead and open up a new issue and we can work it out there.

Yeah, I replied to your thread in the NTC slack channel. Thanks.
On Wed, Dec 20, 2017 at 10:09 AM, Zach Moody notifications@github.com
wrote:

I split them up into multiple pages in, I think, the 3.0.0 release. Before
that it was just one big page, but they're still the same docs. They can
certainly be better though. Let me know the places they're unclear and I'll
see about fixing them.
As far as your issue with .create(), I started a thread on your mention
of it in the NTC slack, but go ahead and open up a new issue and we can
work it out there.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/pynetbox/issues/26#issuecomment-353123690,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHbDihEwIuLXHLkvcvmNbSo8jrLN6Qzcks5tCT9TgaJpZM4RHhP0
.

authors: bdlamprecht zachmoody

repo: digitalocean/pynetbox


text:
Create jjojo

jjoj

ojojo

ojo


text:
Make it possible to update colliders

This adds support for updating colliders by re-inserting a ColliderBuilder on an entity which already has a ColliderHandleComponent.
Please note that the change to the debug rendering code which updates the collider's mesh on change will not work on bevy 0.4 but will on 0.5 due to the new reliable change detection feature.

Thanks!

authors: alec-deason sebcrozet


text:
🛑 babis Apartments is down

In 37ec8b9, babis Apartments (https://www.babisapartments.gr) was down:

HTTP code: 0
Response time: 0 ms

Resolved: babis Apartments is back up in 6a71bd1.

repo: dimitrisdervas/upptime


text:
Creating first sublime snippet and utils folder

I've just created the first sublime snippet as I told you.
Take a look, please.

I forgot to add the () in the end of first snippet (factory snippet), please fix it :)

repo: dineshTrivedi/angularjs-styleguide


text:
Multiple instances

If I have multiple instances of od-virtualscroll is there a way to determine which instance triggered an observable on ScrollObservableService (scrollWin$, createItem$, etc.)?

AH, thank you serguei, you're right! I did not think of this.. will come up with a solution soon.

Turns out that it is already supported. You have to make use of the Hierarchical Injectors concept in Angular.
The ScrollObservableService is registered on the VirtualScrollModule by default, so it's available on the root injector.
However, if you have multiple instances of the scroll component a singleton instance of the ScrollObservableService is not enough.
You can register the service on the wrapping component, via providers property in the @Component decorator, so that the injector bubbling will stop bubbling up to the root injector and will serve the right instance of the ScrollObservableService.
Please feel free to open up this issue again, if you find something else.

authors: dinony zy-serguei

repo: dinony/od-virtualscroll


text:
optional parameters avoid append comma

Hey,
I started a driver for the sim7020 NBIoT modem using at-commands. Super helpful!
Unfortunately, the handling of optional parameters seems to be different in this chip. Instead of providing empty comma separated parameters the chip seems to also make the delimiting comma optional.
e.g. creating a http session (section 8.2.1 in the AT reference pdf:
        at_commands::builder::CommandBuilder::create_set(buffer, true)
            .named("+CHTTPCREATE")
            .with_string_parameter("http://blabla")
            .with_optional_string_parameter(None)
            .with_optional_string_parameter(None)
            .finish()

// should produce: AT+CHTTPCREATE=http://blabla,,

The chip will return an error. Same happened with all other commands on that IC using optional parameters. If, instead, I use:
        at_commands::builder::CommandBuilder::create_set(buffer, true)
            .named("+CHTTPCREATE")
            .with_string_parameter("http://blabla")
            .finish()

// should produce: AT+CHTTPCREATE=http://blabla

it works fine.
Is this a different flavor of AT commands in general? If not, maybe a feature flag e.g. add_comma_on_optional_commands or alike to switch the delimiters on/off?
Looking forward hearing your thoughts on that.
Cheers
Marius

Hi there!
I see your problem. Yeah that's a bit annoying.
Adding a feature is a bad idea, because say you've got two drivers using this crate it'd force both of them to have the same behavior.
I think it'd be better to add two new functions that exhibits the behavior you want. If None, it would do nothing, if Some, it would add the parameter like normal. Should be simple to implement. The hardest part is naming the things.
The current optional parameters always leave behind a comma to signify their absence.
So what would you call one that doesn't leave behind the comma?
with_skipped_int_parameter?
I don't know... I can't come up with anything.
I'm open to making a breaking change to make it have good names!
If you want, you can make a PR and I'll review, merge and release it.

I got a suggestion from someone to add an option to remove all trailing comma's from a command.
I think that'd help in your case.
However, what also may work is splitting the builder so it's not a single line:
let mut builder = at_commands::builder::CommandBuilder::create_set(buffer, true)
    .named("+CHTTPCREATE")
    .with_string_parameter("http://blabla");

if let Some(foo) = foo {
    builder = builder.with_int_parameter(foo);
}

builder.finish()

Thanks for the proposed solution to split the builder. I guess I will use this for the time being.
How about a method .trim() to remove trailing commas. .with_skipped_int(...) would also be fine but I find the name confusing. How about .with_optionally_skipped_int(...)?

I think I can get behind .with_skipped_int_parameter(...).

I will send a merge request in the coming days

authors: HerrMuellerluedenscheid diondokter

repo: diondokter/at-commands


text:
🛑 Iris Development API is down

In 8143149, Iris Development API (https://api.dev.iris.dioptra.io/status) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Iris Development API is back up in 6086df7 after 27 minutes.

repo: dioptra-io/iris-uptime


text:
[Book] Examples

Added custom arguments to basic
Added comments to test basic test program
Created a script to generate documentation based on comments + test_files folder structure
A bit of cleanup in test data generation scripts + support for arguments

While I was doing this I realised that input arguments for programs are not in json format. But it was too late.
I had to regenerate execution artifacts for all programs as part of testing (for hashing it was not necessary). Pardon me for that.
Here is a link to result
This resolves issue #32
Checklist:

[x]  Cairo sources
[x] Trace
[x] Memory
[x] Input
[ ] Output
[x] Prover params
[x] Prover config
[x] Proof
[x] Args
[x] Verbose description
[x] Documentation Page

I didn't quite understand what you meant when you meant by outputs could you elaborate, please?

repo: dipdup-io/stone-packaging


text:
Implement ShangMi-3 (SM3)

ISO/IEC 10118-3:2018 => https://www.iso.org/standard/67116.html
https://webstore.iec.ch/preview/info_isoiec10118-3{ed4.0}en.pdf

https://datatracker.ietf.org/doc/html/rfc8998

http://www.oscca.gov.cn/sca/xxgk/2010-12/17/1002389/files/302a3ada057c4a73830536d03e683110.pdf

repo: dipu-bd/hashlib


text:
lineproto: support zlib

This is hard but necessary.
Zlib is used in both NMDC and ADC:

for transferring files
for hub-to-client communication, between $ZOn| and a zlib EOF.

Zlib is also (IMHO) the main feature that indicates that the protocol is old and conceived a long time ago: its activation and deactivation on a certain message depends on the content of the previous message.
Parallel parsing and processing of messages (basically the main feature of Go) is not possible, messages have to be FIRST parsed and then processed, because the following message could be compressed and require deflating.
This PR replicates the solution i implemented in dctk, and introduces a 3-levels line reader:

in the first level, bytes are buffered
in the second level, bytes are deflated if compression is active
in the third level, bytes are read until the delimiter, byte by byte, STRICTLY without buffering

Please test carefully this patch with the hub and feel free to propose changes.

This change is

I did everything including the double buffer procedure, but i did not use a custom read buffer, just the standard one.

When EOF is encountered, zlib reader may still have some (uncompressed) data remaining

i checked, and if the buffers are implemented correctly, the zlib reader never have uncompressed data remaining. Any uncompressed data is left in the previous buffer. So it is possible to skip the building of a custom buffer.

ReadByte() and the buffer can be replaced by ReadBytes(delim) https://golang.org/pkg/bufio/#Reader.ReadBytes that surely improves performance.
if a custom buffer further improves performance, we can use it instead.

Regarding file transferring, we’ll see another day if it’s better to expose Read() or to bypass lineproto

repo: direct-connect/go-dc


text:
draft: [DIR-548] webkit workaround

Please do not merge, we should discuss this first.
See the comments in the file.
This will "force" clicks only in the context where this is needed (when tests run locally on webkit).
However, I am not really comfortable with the logical complexity this adds to the tests. I think I'd rather keep them simple and live with tests failing locally in webkit until we know more.

Closed, we decided against implementing this as it would add too much complexity to the tests.

repo: direktiv/direktiv-ui