Duplicate check fails against the example question?

I tried creating a MD file with the following text:
# Test Deck
## What's the Markdown?

Markdown is a lightweight markup language with plain-text-formatting syntax.
Its design allows it to be converted to many output formats,
but the original tool by the same name only supports HTML.

## Who created Markdown?

John Gruber created the Markdown language in 2004 in collaboration with
Aaron Swartz on the syntax.

I copied the questions directly our of the README.md. The "What's the Markdown?" question gets duplicated every time I run Anki: Send to Deck.
A quick diff check of the default notes's deck's JSON files for the questions shows that the duplicated questions only differ in the "id" field.

A minimal repro of this bug can be found by using the following test deck:
## Test'
Answer

## Test
Answer

The first question "Test'" will be duplicated but the second question "Test" will not. It looks like the apostrophe throws off the duplicate check somewhere and causes it to be re-added every time the question set is added to Anki.
Inspecting the generated card JSON file, I suspect that this is because the apostrophe converts to ' which triggers https://github.com/FooSoft/anki-connect/issues/194 but I haven't done any deeper dive into this yet.

I think this is a duplicate (no pun intended) of https://github.com/jasonwilliams/anki/issues/13
Certain characters cause he duplicate check to fail on AnkiConnect. I think it’s because they’re htmlEncoded and that causes problems.

Closing as duplicate

Closing as duplicate


upgrade empirica and attempt to change consent and mturk id language

I followed Nicholas' instructions in https://github.com/empiricaly/empirica/issues/107 and https://github.com/empiricaly/empirica/issues/101 but nothing seems to be changing and I'm not sure why. Storybook also won't run now at all. Could you take a look and see if upgrading empirica broke any dependencies or docker or anything?

I don't know where things are going wrong, but version 1.0.0-beta.3 of @empirica/player is the correct version, which does contain the fix in GameFrame. Maybe something to do with how Docker Compose is building, I don't know. Maybe just a matter of deleting node_modules and rebuilding it before running the build. Maybe stop mounting node_modules into the container.

I suspect now a vite issue. Try to run rm -rf client/node_modules/.vite.
https://vitejs.dev/guide/dep-pre-bundling.html#caching

Closing in favor of: https://github.com/Watts-Lab/deliberation-empirica/pull/131


Lightmaps in FM 2024

In the latest version of Football Manager there are lightmaps with the suffix [lm] for every interior room. The current version of this addon does not seem to import these and so the lightmap data is lost when exporting an edited .sia

I got a bit more work left, cleanup and such, but this should work so that you can progress in the meantime.
minimum blender version is now 4.
io_scene_sia.zip

I added a new release now :)


Sync Error - Attempt to start an already started synchronizer.

besu-config.txt
Description
2 boot nodes, 4 validator nodes, 3 member nodes, 1 rpc node.
Steps to Reproduce (Bug)

Running tests against the BESU network over 3-4 hours

Expected behavior:
All validators continue to mine at the same rate
Actual behavior:
One validator stops mining and hangs indefinitely.
Frequency:
Happens when running tests after a 3-4 hours (12 hours max). This happens every-time we run tests against the network.
Logs (if a bug)

Besu has identified a problem with its worldstate database.
│ # Your node will fetch the correct data from peers to repair the problem.
│ # Starting the sync pipeline...
####################################################################################################   │
│ validator-4-besu Throwable summary: java.lang.IllegalStateException: Attempt to start an already started synchronizer.  │
│ validator-4-besu     at: org.hyperledger.besu.ethereum.eth.sync.DefaultSynchronizer.start(DefaultSynchronizer.java:218) │
│ validator-4-besu 2024-05-28 23:58:01.418+00:00 | EthScheduler-Services-7 (importBlock) | ERROR | PipelineChainDownloade │
│ validator-4-besu java.util.concurrent.CompletionException: java.lang.IllegalStateException: Attempt to start an already started synchronizer.

Versions

Software version: [24.5.2]
Java version: [openjdk-java-21]
OS Name & Version: [AKSUbuntu-2204gen2containerd-202405.03.0]
Kernel Version: [5.15.0-1061-azure]
Kubernetes Version: [1.28.5]
Cloud VM, type, size: [Standard_D4as_v4]

Additional Information (Add any of the following or anything else that may be relevant)

Besu setup info - genesis file, config options
`genesis.json:

{
"nonce": "0x0",
"timestamp": "0x58ee40ba",
"extraData": "0xf87aa00000000000000000000000000000000000000000000000000000000000000000f8549452dfcfb57df309cb83a2a5873ab9e02b5286adc294c78292173664af0153c943b67d6b7371c5bbbbcf94130ee989acf35492f16ac796c2672a7ae085bf7f940f96dea61aa4303a35f5f0c29938973fc0d0bf52c080c0",
"gasLimit": "0x989680",
"gasUsed": "0x0",
"number": "0x0",
"difficulty": "0x1",
"coinbase": "0x0000000000000000000000000000000000000000",
"mixHash": "0x63746963616c2062797a616e74696e65206661756c7420746f6c6572616e6365",
"parentHash": "0x0000000000000000000000000000000000000000000000000000000000000000",
"config": {
"chainId": 1337,
"homesteadBlock": 0,
"eip150Block": 0,
"eip150Hash": "0x0000000000000000000000000000000000000000000000000000000000000000",
"eip155Block": 0,
"eip158Block": 0,
"byzantiumBlock": 0,
"constantinopleBlock": 0,
"petersburgBlock": 0,
"istanbulBlock": 0,
"muirglacierblock": 0,
"berlinBlock": 0,
"londonBlock": 0,
"zeroBaseFee": true,
"qbft": {
"blockperiodseconds": 2,
"epochlength": 30000,
"requesttimeoutseconds": 20
}
},
"baseFeePerGas": "0x0",
"nonce": "0x0",
"alloc": {
"0xa1c0d3e39e5524dd0284f7fe768f9f5aebfbae8a": {
"balance": "1000000000000000000000000000"
},
"0x8d5903e624be9cddd0a23ea5d1dc3efc077ea438": {
"balance": "1000000000000000000000000000"
},
"0xceee8609e8950c22edbf0adca5a36b52a350dba5": {
"balance": "1000000000000000000000000000"
},
"0x29f9c20171de55c323e95070c4eea1ccaec2f5ec": {
"balance": "1000000000000000000000000000"
}
}
}`

System info - memory, CPU
Total memory: 15.62 GB
CPU cores: 4

Hi there - Bonsai and Tessera can have issues together - have you tried FOREST mode for your data storage format?
https://besu.hyperledger.org/development/public-networks/concepts/data-storage-formats

In my case, I encountered the same issue, but changing the node's option from data-storage-format="BONSAI" to "FOREST" resolved the error.


Grafana JSON datasource broken on Grafana 9.1.2, JSON plugin 1.3.1

Hello All,
I'm trying to create a variable on a REST endpoint that has the following (simple example):
{
  "all": {
        "children": [
            "linux",
            "osx",
            "pi",
            "ungrouped",
            "windows"
        ]
    }
}

The endpoint is running on http://localhost:3001/test
Method: GET
Path: /query
JSONPath field 1: $.all.children[*]
I was expecting to get:
[
  "linux",
  "osx",
  "pi",
  "ungrouped",
  "windows"
]

But the UI shows nothing on the autocomplete. I do not see any errors either. Also when I try to add this as a variable on the dashboard I get "none" for some reason.
Can any of you point out a way to debug this? That will allow me to fix the issue (in case is a user error) or to create a better ticket here with more details.
Thanks in advance,
--Jose

Gets more interesting, Grafana is making the query and is getting a response (I'm using Mockoon for the REST endpoint)
{
  "request": {
    "url": "api/datasources/proxy/4/test",
    "method": "GET",
    "hideFromInspector": false
  },
  "response": {
    "all": {
      "children": [
        "linux",
        "osx",
        "pi",
        "ungrouped",
        "windows"
      ]
    }
  }
}

Tried downgrading to '8.3.11-ubuntu' and error is the same. Points to the plugin it seems.

I noticed the URL that your request is being sent to is a deprecated API endpoint.
I wonder whether the plugin needs to be updated to use the newer /api/datasources/proxy/uid/:uid/ endpoint to fix this issue.


libchdr shared library

Would it be possible that it could be made so that a libchdr shared library could be built so it can be installed system wide? My main interest is that I would be able to reuse it in downstream projects instead of requiring libchdr to be built statically. This would greatly help in comparing upstream libchdr with downstream implementations.
Additionally it would also be nice to be able to build libchdr with system version of libraries like flac. At least with flac, a system version works fine with libchdr in RetroArch.

Well there's no reason not to.
I will look into this and community submitted fixes I need to merge.
Le ven. 27 avr. 2018 à 20:44, orbea<EMAIL_ADDRESS>a écrit :

Would it be possible that it could be made so that a libchdr shared
library could be built so it can be installed system wide? My main interest
is that I would be able to reuse it in downstream projects instead of
requiring libchdr to be built statically. This would greatly help in
comparing upstream libchdr with downstream implementations.
Additionally it would also be nice to be able to build libchdr with system
version of libraries like flac. At least with flac, a system version works
fine with libchdr in RetroArch.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rtissera/libchdr/issues/7, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC3LBfqhsjvnUq2M5I7o8JAn82Y32lHVks5ts2b-gaJpZM4Tq1ys
.

Thanks! My familiarity with cmake is limited and I'm not sure I can implement this in a good way soon.

Fixed in latest commit.


Detect URL with IPs as unsafe-link

URL with raw IPs should be tagged as unsafe links (except <IP_ADDRESS>).
See the following code<EMAIL_ADDRESS>(do not use this as example, this code is dangerous)

Hi Thomas can you please assign me this issue? Please. Thanks.

Package was removed, you can still see the code here: https://socket.dev/npm/package/u-workflow.auth/files/2.0.0/index.js#T221-228
Example of dangerous pattern:
var url = "http://<IP_ADDRESS>/burpcollaborator.txt";


Support for .babelrc?

Does the transpiler respect the .babelrc file for including extensions?
I am using react-rails and i would like to include https://github.com/gaearon/react-transform-catch-errors as well. This is possible through adding it to the .babelrc file. I tried but it is not doing anything. I am expecting to much? :)
Would like to have some thoughts on my issue.

I'm pretty sure that .babelrc works only for babel-cli. Ruby-babel-transpiler works in isolated ExecJS environment and doesn't take .babelrc for configuration.


[Feature request] Add squiggles under filenames that have issues

Basically something like this. For some reason I couldn't find an existing feature request for this.  Apologies if it does indeed exist.

Ping.  Looks like this had the assignee removed without a replacement.

Support for this has been added with VS Code 1.18 (the current insiders builds). Try setting: problems.decorations.enabled": true to enable marking files with problems in the explorer


[release-1.9] Allow, with warning, users using first-party-jwt with Kubernetes 1.21+

Manual cherrypick required.
#32012 failed to apply on top of branch "release-1.9":
Applying: Warn, but allow, first-party-jwt with Kubernetes 1.21+
Using index info to reconstruct a base tree...
M	security/pkg/server/ca/authenticate/kubeauth/kube_jwt.go
M	tests/integration/helm/install_test.go
M	tests/integration/security/mtls_first_party_jwt/main_test.go
Falling back to patching base and 3-way merge...
Auto-merging tests/integration/helm/install_test.go
CONFLICT (content): Merge conflict in tests/integration/helm/install_test.go
Auto-merging security/pkg/server/ca/authenticate/kubeauth/kube_jwt.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Warn, but allow, first-party-jwt with Kubernetes 1.21+
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

Chery-pick merged


typo

there is a typo in kubeadm-upgrade-1-11.md

thanks
/sig cluster-lifecycle
/lgtm

/lgtm cancel
actually @WisWang please squash your commits. there is no need to have 3 commits for a single word typo fix.
thanks

It will get squashed on merge.
/approve

/lgtm


Having Trouble Using Plugin

I had a friend who is in the Apple Dev program download the font directly from Apple's site and fwd me the zip file, but the plugin doesn't seem to be working with it. Font sizes are at the appropriate size within my sketch file for the different font types. I've tried uninstalling and reinstalling the plugin and that didn't help either. Any advice?

Are you designing at 1x or 2x? :)
I check the plugin every update and haven't had any issues personally but I only design in 1x. The font tracking table was built for 1x only.

Yeah I'm using the standard iPhone 6 artboards - 375 x 667 px, so yeah I'm designing at 1x

I assume you're using the latest version of Sketch?

up to date, running 39.1

idk if it matters but the fonts downloaded as "San Francisco Text", "San Francisco Display" as opposed to "SFUI Text", "SFUI Display"

Can you check Font Book and see what they're font-family name is? It should be SF UI Text and SF UI Display like so:

They are named San Francisco Text and San Francisco Display. I assume this is the problem?

Yeah thats not the correct font.  From that image it looks like you actually have SF Compact Display and SF Compact Text fonts, which are the watch fonts.  Those were originally the only SF fonts we had because it was first released with watchOS.
You'll have to download the official ones from https://developer.apple.com/fonts/ They come in the file "SFUI.zip".

would it work to rename the fonts?

Not unless you edit the actual font data. The postscript names are written to the file itself, the file name really has no effect on it.

ok, thanks


serif font support, editing adjustments, red underline

prettify red underline, fix DomTextEditor height settings, use serif font-family for error popup message

I think this should be several different commits.

Yes, but they were close (by time) and you told me to do such in one. I can split them if you want

The rule is the following: one logical change - one commit. There are 3 logical changes there.


Update patch-glibc-CVE-2015-0235.yml

cosmetic patch for glibc_search:, adding ssh to affected services

SSH is already on line 30

oops :p mybad


[Bug] Buttons on show not primary color

Bug report
What I did
I used tabs on show page. The buttons are blue insted of the primary color purple.

What I expected to happen
Buttons pruple
What I've already tried to fix it
Is related to this class but i don't know how or when is compiled
.btn-link {
    color: #206bc4;
    background-color: transparent;
    border-color: transparent;
    box-shadow: none;
}

Context to replicate the bug - Backpack, Laravel, PHP, DB version
In the backpack demo in the show monster page

Yes, I can confirm this from the demo.
https://demo.backpackforlaravel.com/admin/monster/140/show
Thanks for reporting @adriallongarriu

Thanks for the report @adriallongarriu
it's now fixed, a composer update should get you the fix.
Don't forget to clear caches as it was a css change 👍
Let us know if you find other issues.
Thanks 🙏

I updated and is working. @pxpm Thanks the quick fix.


Query on table from multiple databases

I want to join tables from multiple databases, is it possible?

Please use Drill or Trino as federated query data warehouse

Please use Drill or Trino as federated query data warehouse

Thanks,but can you tell me, would it be configured with company specific(internal) superset?
and wouldn't there be any data privacy issue?


make gen tries to clone non-existent repo

Running make gen as per the instructions fails when trying to clone https://github.com/InjectiveLabs/injective-core.git (which does not exist).

Please read the README.md carefully. That instruction you mention (make gen) is in the "Development" section, and is intended only for the SDK developers

@aarmoa a bit quick to jump to conclusions. I think development instructions should also be kept up to date and functional.
In our case, we needed to add support for parsing a message type that wasn't supported by python-sdk. I think that falls under development. But we managed to do so without following the development steps in the readme, thank you.


DOC: update jmlr + cosmit

Codecov Report

Merging #186 (77b8897) into master (4610641) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #186   +/-   ##
=======================================
  Coverage   85.95%   85.95%
=======================================
  Files          13       13
  Lines         904      904
  Branches      119      119
=======================================
  Hits          777      777
  Misses         97       97
  Partials       30       30

Impacted Files
Coverage Δ

celer/dropin_sklearn.py
96.40% <ø> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #186 (77b8897) into master (4610641) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #186   +/-   ##
=======================================
  Coverage   85.95%   85.95%
=======================================
  Files          13       13
  Lines         904      904
  Branches      119      119
=======================================
  Hits          777      777
  Misses         97       97
  Partials       30       30

Impacted Files
Coverage Δ

celer/dropin_sklearn.py
96.40% <ø> (ø)

Continue to review full report at Codecov.

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


KML geometry incorrect compared to Google Earth

As reported on the forum, load the attached KMZ file into Cesium and you'll notice small gaps between the walls and roof.  These gaps do not exist if you load the same file into Google Earth.
test_geometry.kmz.txt

Matt, I have 153K Kml file that loads fine in Google Earth. In Cesium, one hemisphere loads fine, and the other hemisphere contains a few lines which should be an entire grid, but it appears the processing stops. Strangely, i can't upload the file here, it errors. I can get you the file somehow if you are interested in using it to debug. I wish I had more js skills, I would debug it for you. Thanks

Thanks, that is very weird.  You can email me the file directly to
<EMAIL_ADDRESS>Thanks for the bug report.
On Wed, Oct 14, 2015 at 11:02 AM, billyweisberg<EMAIL_ADDRESS>wrote:

Matt, I have 153K Kml file that loads fine in Google Earth. In Cesium, one
hemisphere loads fine, and the other hemisphere contains a few lines which
should be an entire grid, but it appears the processing stops. Strangely, i
can't upload the file here, it errors. I can get you the file somehow if
you are interested in using it to debug. I wish I had more js skills, I
would debug it for you. Thanks
[image: ge]
https://cloud.githubusercontent.com/assets/9979211/10487647/2edba34c-7252-11e5-90c9-3a30f7aaac55.JPG
[image: cesium]
https://cloud.githubusercontent.com/assets/9979211/10487655/3a5f3b02-7252-11e5-9c99-7ba93e809cce.JPG
—
Reply to this email directly or view it on GitHub
https://github.com/AnalyticalGraphicsInc/cesium/issues/3082#issuecomment-148077525
.

The bug was fixed in 1.17 and the gaps are gone.  I believe an unrelated fix to the underlying Polygon geometry code fixed this as well.


Simplify even more?

Having used Iron Router in the past 12 months, I dreamed about a simpler client-only router package. Here it is, thanks a lot!
In my dream, the new router was decoupled from template rendering. You did it!
In my dream, the new router was decoupled from subscription management. Is there a reason you didn't go this far? What would be the difference between:
FlowRouter.route('/blog/:postId', {
    subscriptions: function(params, queryParams) {
        this.register('myPost', Meteor.subscribe('blogPost', params.postId));
    }
    ...
});

and:
FlowRouter.route('/blog/:postId', {
    action: function(params, queryParams) {
        this.register('myPost', Meteor.subscribe('blogPost', params.postId));
        ...
    }
});

and finally:
FlowRouter.route('/blog/:postId', {});
Tracker.autorun(function() {
    Meteor.subscribe('blogPost', FlowRouter.getParam('postId'));
});

?
In my dream, the new router would even have no action function. Why not just have the following?:
FlowRouter.route('/blog/:postId');
Tracker.autorun(function() {
    // Use the FlowRouter API the way I want
    ...
});

You should not use this.register inside the action. I you can that's a bug in Flow Router.
There is a reason for that we've a subscriptions in the router.

We believe subscriptions are aligned to a route most of the time.
For fast-render we need to run subscriptions on the server

That's why we have something like this.
But, we don't need to wait for subscriptions. That's why we only register it. Later you can check for the status.

Thanks @arunoda, understood.
My feeling is that maybe there is a simpler way to use your package that what is explained in the doc (for those who don't use fast-render and don't attach subscriptions to routes). I'll try to write an alternate manual if I have time.
Regarding the ready API, since I never attach subscriptions to a route, I use Template.subscriptionsReady instead.

You need to have action method for the route. Otherwise you need to do some logic to pick the route and do the rendering.

Yes. You can leave it alone. Both subscription and action are optional. May
be I can add it to docs.
On 2015 මාර්තු 12, බ්‍රහස් at ප.ව. 7.25 steph643<EMAIL_ADDRESS>wrote:

Thanks @arunoda https://github.com/arunoda, understood.
My feeling is that maybe there is a simpler way to use your package that
what is explained in the doc (for those who don't use fast-render and don't
attach subscriptions to routes). I'll try to write an alternate manual if I
have time.
Regarding the ready API, since I never attach subscriptions to a route, I
use Template.subscriptionsReady
https://github.com/meteor/meteor/blob/devel/History.md#blaze instead.
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/flow-router/issues/34#issuecomment-78483389
.

Yeah. I am quite happy to add a link to that manual on the read me.
On 2015 මාර්තු 12, බ්‍රහස් at ප.ව. 7.30 Arunoda Susiripala <
<EMAIL_ADDRESS>wrote:

Yes. You can leave it alone. Both subscription and action are optional.
May be I can add it to docs.
On 2015 මාර්තු 12, බ්‍රහස් at ප.ව. 7.25 steph643<EMAIL_ADDRESS>wrote:

Thanks @arunoda https://github.com/arunoda, understood.
My feeling is that maybe there is a simpler way to use your package that
what is explained in the doc (for those who don't use fast-render and don't
attach subscriptions to routes). I'll try to write an alternate manual if I
have time.
Regarding the ready API, since I never attach subscriptions to a route,
I use Template.subscriptionsReady
https://github.com/meteor/meteor/blob/devel/History.md#blaze instead.
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/flow-router/issues/34#issuecomment-78483389
.

I was going to ask something similar: since you can run any code in subscriptions, you can run rendering as well. Then what is the point of action?
FlowRouter.route('/blog/:postId', {
    subscriptions: function(params, queryParams) {
        this.register('myPost', Meteor.subscribe('blogPost', params.postId));
        FlowLayout.render('componentLayout', {content: ...});
    }
    ...
});

I'm new to Flow-Router, so please pardon my ignorance :)

subscriptions is different. I runs inside a autorun for managing
subscriptions (auto unsub.)
It also runs on the server.
Actions is a simply a function.
On Thu, Jun 4, 2015 at 4:36 PM Dan Dascalescu<EMAIL_ADDRESS>wrote:

I was going to ask something similar: since you can run any code in
subscriptions, you can run rendering as well. Then what is the point of
action?
FlowRouter.route('/blog/:postId', {
subscriptions: function(params, queryParams) {
this.register('myPost', Meteor.subscribe('blogPost'
, params.postId));
FlowLayout.render('componentLayout', {content: ...});
}
...
});
I'm new to Flow-Router, so please pardon my ignorance :)
—
Reply to this email directly or view it on GitHub
https://github.com/meteorhacks/flow-router/issues/34#issuecomment-108842836
.


create separate modules for excel stats calculations and excel utilities

good job.
let's write some unit tests next.


Update browserlist and issue templates

closes #202
prerequisites:

[x] branch is up-to-date with the branch to be merged with, i.e. develop
[x] build is successful
[x] code is cleaned up and formatted
[ ] tested with Firefox 52, Firefox 57+, Chrome 64+, MS Edge 16+

Summary
update the browser list along with the PR template to reflect it. Moreover, I added a bunch of issue templates for better issue management

circleci fails with an unknown: permission denied for unpacking cache error

@sgratzl It seems that restoring the cache for the /root/.npm directory does not work (anymore?). Should I wait for a fix for the CircleCI error?

@sgratzl It seems that restoring the cache for the /root/.npm directory does not work (anymore?). Should I wait for a fix for the CircleCI error?

seems like we have a faulty machine cause another build (https://app.circleci.com/pipelines/github/lineupjs/lineupjs/460/workflows/93ea315b-d9f4-4ed1-a2ce-4b98e43531f4/jobs/834) works like a charm

@sgratzl I merged this PR and now the lineup-v4 branch has CircleCI errors...


Issue #1148 - Updated URL from goodworkfor.life to usegood.work

Make sure you've reviewed the checklist. (DO NOT REMOVE)
🚨Please review the guidelines for contributing to this repository.

[X] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
[X] Make sure you are making a pull request against the dev branch. Also you should start your branch off dev branch.
[X] Check you have included issue number in your pr (e.g #23).
[N/A] Check you have added tests that prove your fix is effective or that your feature works.
[N/A] Check your code additions will fail neither code linting checks nor unit test.
[N/A] Check you added any notable changes to the CHANGELOG.md file.

N.B Only push atomic PR meaning single feature/update/fix in a PR. Don't combine multiple feature/update/fix in a single PR.
Description
Please describe your pull request.
Updated project URL in readme following domain change.
Ref to issue #1148
💔Thank you!

Hello!
By checking https://usegood.work, it doesn't look to me that #1148 is resolved since in the demo page it still links the broken domain. Am I wrong?

demo is available at new domain htttps://demo.goodwork.cloud, thanks for the PR though.

@Hasnayeen The demo is still not functional:

Can you please have a look at it when you have time?
Thanks for your work! :)

@ffranchina fixed, thanks for notifying

@Hasnayeen no problem, thanks to you!
The 404 errors are gone but it's still not possible to interact with the platform, is that the intended behavior?

@ffranchina redesigning the whole app, so most of the things are just placeholder for now


doc: Fix links to anchors of the same page.

Affected page: http://vitess.io/user-guide/horizontal-sharding-workflow/
This broke when I changed the URL format in https://github.com/youtube/vitess/pull/3183.
I fixed it by removing the file name from the link. Note that the links should not have included the file name to begin with.
Fixes https://github.com/youtube/vitess/issues/3238.

Trivial doc change. I'm merging this now.


Minecraft - Move ownership to startup

First, thanks for the wonderful and very complete image, it is well done and works well.
Setting ownership to 1000:1000 at image build time raises issues with using external volume providers. I believe it would be beneficial to move the setting of ownership defined at https://github.com/itzg/dockerfiles/blob/master/minecraft-server/Dockerfile#L25 to the startup scripts.
When using an external volume like something from the host, or glusterfs, amazon, or anything else, you start to run into issues with the default uid/gid set to 0 or 0 and a random number, but since the container starts as root and then starts minecraft as 1000:1000 it would be more beneficial to during startup to set all the volumes to the user 1000:1000.
Alternatively move this https://github.com/itzg/dockerfiles/blob/master/minecraft-server/Dockerfile#L22-L25 to the startup script so you could make the UID/GID settable via environment variables.

Just in the past couple commit I actually restored the ability to adjust the runtime user via the UID and GID variables
https://github.com/itzg/dockerfiles/blob/master/minecraft-server/start
It also turns out the chown in the Dockerfile is not honored by external volume providers, so setting the runtime user to match the established volume ownership is the best/only solution.

Correct it is not honored. However I still think the best bet would be to add a chown -R $UID:$GID just before the su-exec to all the defined volume mounts because unless the volume driver is specifically design to allow you to change the GID, generally the volumes will get mounted in as 0:0.
Since the container initially run's as root just before the su-exec you can call a recursive chown to ensure proper ownership of all directories needed regardless of how the content makes it in.

Good point. I was actually pondering reviving that feature also, so you have helped me decide :)

Awesome, glad I could help!

I added an init container to the container spec in the stable/minecraft helm chart.  This fixed the issue.  I can send it as a MR if you like? @itzg

@mumbley that would be great to integrate that into the image here for non-k8s scenarios.

This is now pushed to Hub.


Add support for arguments in manifest

Description of the new feature/enhancement
Oculus and some other programs need a / argument to run silent mode. This can not be done with winget right now
Proposed technical implementation details (optional)
When using the wingetcrete the should be a argument option were i can add / commands

WinGet CLI and Manifest does have support for Switches. https://aka.ms/winget-manifest.installer.1.0.0.schema.json
Switches prompt will be added for wingetcreate once its out of its alpha stage.

oh right cool i will try that.


add basic menu to header

for #271
The menu is still unstyled and the button to open the menu isn't clickable yet (css issue I think).
Once we have all the links in there we need, I'll make them open the target in a browser, as expected.

@juliangruber What's missing here? Tested visually, LGTM!


Problem with two lines cells in tables

I see tool has problem with rows in tables where row has two lines. Something:
first cell |second cell | third cell
with two lines
CSV it will produce will be:
first cell, second cell, third cell
with two lines

In "Stream" mode, this is expected behavior. Stream mode is unable to distinguish multi-line cells from separate rows. If your PDF has lines separating each cell, you can try selecting the Lattice mode, which handles multi-line rows as you expect.


blockpages_generator lambda sporadic failure due to "No space left on device" error

We got paged from time to time due to blockpages_generator lambda failures.
Upon checking the logs, it was caused by errors similar to below
[Errno 28] No space left on device: '/tmp/tmp7cjkc3ey': OSError
Traceback (most recent call last):
File "/var/task/aws_lambda.py", line 288, in blockpages_generator
target_dir = mkdtemp()
File "/var/lang/lib/python3.6/tempfile.py", line 368, in mkdtemp
_os.mkdir(file, 0o700)
OSError: [Errno 28] No space left on device: '/tmp/tmp7cjkc3ey'

The failure is sporadic and it usually doesn't happen very often, but for the recent 4 days, it has happened twice.
Is it possible to either clean the temporary space (before or after work), or catch the exception and ignore it?
This will help reduce false alerts, besides the lambda usually correct itself after a few re-runs, so it's basically when this happens, it's not actionable by ops.

Thanks for the fix, @leplatrem
v4.3.0 lambda deployed to -stage without issues.
I'm currently waiting for https://bugzilla.mozilla.org/show_bug.cgi?id=1495493 before deploying v4.3.0 to -prod (since no urgency for us to get this out).


Unzip unfinished run (ask Node(3-unzip/2-htigs/chunk_000591F) failed with exit-code=256)

Hi,
I tried to run unzip on my latest falcon assembly (12 SMRT cell of nematode ~120Mb genome) and it gets this error :
pypeflow.simple_pwatcher_bridge:94 - ERROR - Task Node(3-unzip/2-htigs/chunk_000487F) failed with exit-code=256
It then deletes all heartbeats files and stop.
here is the report for my input falcon assembly :
{
"genome_length": 120000000,
"length_cutoff": 4000,
"preassembled_bases":<PHONE_NUMBER>,
"preassembled_coverage": 55.341,
"preassembled_esize": 10469.713,
"preassembled_mean": 6986.596,
"preassembled_n50": 9607,
"preassembled_p95": 16350,
"preassembled_reads": 950529,
"preassembled_seed_fragmentation": 1.465,
"preassembled_seed_truncation": 1736.009,
"preassembled_yield": 0.718,
"raw_bases":<PHONE_NUMBER>,
"raw_coverage": 80.094,
"raw_esize": 14299.757,
"raw_mean": 10324.589,
"raw_n50": 13193,
"raw_p95": 22348,
"raw_reads": 930912,
"seed_bases":<PHONE_NUMBER>,
"seed_coverage": 77.036,
"seed_esize": 14758.064,
"seed_mean": 11952.704,
"seed_n50": 13500,
"seed_p95": 23261,
"seed_reads": 773409
}
Here is my fc_run.cfg :
_[General]
input_fofn = input.fofn
input_type = raw
#openending = True
length_cutoff = 4000
genome_size = 120000000
seed_coverage = 60
length_cutoff_pr = 7000
sge_option_da = -pe smp 5 -q bigmem
sge_option_la = -pe smp 20 -q bigmem
sge_option_pda = -pe smp 6 -q bigmem
sge_option_pla = -pe smp 16 -q bigmem
sge_option_fc = -pe smp 24 -q bigmem
sge_option_cns = -pe smp 12 -q bigmem
pa_concurrent_jobs = 50
cns_concurrent_jobs = 50
ovlp_concurrent_jobs = 50
pa_HPCdaligner_option =  -v -B128 -t16 -M24 -e.70 -l1000 -s400 -k14 -h80 -w8
ovlp_HPCdaligner_option = -v -B128 -M24 -h100 -e.96 -l1000 -s400 -k14
pa_DBsplit_option = -x500 -s400
ovlp_DBsplit_option = -s400
falcon_sense_option = --output_multi --min_idt 0.70 --min_cov 2 --max_n_read 100 --n_core 50
falcon_sense_skip_contained = False
overlap_filtering_setting = --max_diff 100 --max_cov 1000 --min_cov 2 --n_core 50_
And my fc_unzip.cfg:
_[General]
job_type = local
[Unzip]
input_fofn= input.fofn
input_bam_fofn= input_bams.fofn
smrt_bin=/media/F/Falcon-Unzip-Juin2018/fc_env_180619/bin/
jobqueue =
sge_phasing=
sge_quiver=
sge_track_reads=
sge_blasr_aln=
sge_hasm=
unzip_concurrent_jobs = 50
quiver_concurrent_jobs = 50_
I read that exit-code=256 could mean a lack of memory but I check my system monitor and never got close to use all of my 512G of RAM...
Please guide thru this "not easy to understand" log and config files.
thank you very much,
PY

I am having a very similar error

We have fixed a lot of Unzip errors in the latest binary tarball. Please try that. If it fails again, we may ask for more details on the crash.


Add JavaScript for Keyboard focus tracking

A global utility for tracking the current input method (mouse, keyboard or touch):
https://github.com/ten1seven/what-input

What Input uses event bubbling on the window to watch for mouse, keyboard and touch events (via mousedown, keydown and touchstart).
By default, What Input uses session storage to persist the input and intent values across pages. The benefit is that once a visitor has interacted with the page, subsequent pages won't have to wait for interactions to know the input method.

This feature will allow to display (or not) a unique highly visible focus indicator for all focusable elements using a combination of a highly contrasting style.
Check out the demo to see What Input in action:
https://ten1seven.github.io/what-input/

This enhancement will be available with the next release 2.7.


PathEscape artifact name to process artifacts with slash in the name

e.g. to get the minder helm chart, we need the API to call:
https://api.github.com/orgs/stacklok/packages/container/package/minder%2Fhelm%2Fminder

but we've been seeing a bunch of:
GET
https://api.github.com/orgs/stacklok/packages/container/minder/helm/minder:
404 Not Found

in the logs.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Can we add a unit test?

Can we add a unit test?

done!


feat: show keyboard shortcuts modal instead of link from help and men…

…u panel
This PR...
Changes

Fixes

How to test it

Screenshots

scrolled

Checklist

[x] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

Grouped navigation vs tools shortcuts at my own discretion, wondering what you think guys

@topliceanurazvan , thank you for your feedback, updated my PR

This is super cool! I'm still reviewing the code but I already have a product request. It's a common shortcut that shift + / (so basically typing ? character) opens this kind of menu. Could we add this here as well?

@WitoDelnat,
You mean, shift + / functionality to open a shortcuts itself?

@jmelnych I tried playing around with both Gmail and Linear app and they use "?" and not "shift + /" (for my international keyboard layout that just happens to be "?" but if I change to Italian layout the hotkey changes. Not sure if we can do that kind of behaviour. Otherwise shift + / should already cover most Qwerty users. What do you think?

@WitoDelnat, I like your idea, I think we can go with shift + /

Great work 👍🏻 - just one more thing I've noticed, the scrollbar styling seems to not be applied

Updated


AWS_DEFAULT_REGION respected, and availibility zones not needed

AWS_DEFAULT_REGION was always set as a GitHub secret when qhub init auto-provisions a GitHub repo, but the region was never set in the qhub-config.yaml file. It was always left as a hardcoded us-west-2.
Additionally, availability_zones of us-west-2a and us-west-2b were hardcoded.
This PR:

respects AWS_DEFAULT_REGION and inserts that into qhub-config.yaml
no longer requires availability_zones to be set in qhub-config.yaml
if qhub-config.yaml does contain availability_zones those will be used

Note that availability_zones is still allowed in qhub-config.yaml:

for backwards-compatibility
in case the user does have a preference

However, I do not believe it is possible to use qhub deploy to migrate an existing qhub to newly-specified availability zones (or regions, for that matter?)
This new approach could therefore cause problems if AWS starts to return different availability zones, or even just in a different order. (Maybe there is a known solution in this situation - I haven't studied it yet.)
If this PR is approved, we need to create similar issues for the other clouds if they have similar data sources available.

By the way, I really don't understand why terraform fmt is throwing this error, or rather why Jinja2 is dropping that whitespace!

This new approach could therefore cause problems if AWS starts to return different availability zones, or even just in a different order. (Maybe there is a known solution in this situation - I haven't studied it yet.)

I think the way we can mitigate this is to sort the availability zones returned by AWS. We can't have deployed on zones that does not already exist.

However, I do not believe it is possible to use qhub deploy to migrate an existing qhub to newly-specified availability zones (or regions, for that matter?)

Terraform will do this i believe which will in the process destroy nearly everything and recreate it.

By the way, I really don't understand why terraform fmt is throwing this error, or rather why Jinja2 is dropping that whitespace!

It had to do with cookiecutter. I was not removing the newline e.g.
a
{% if cookiecutter.b is defined %}
{{ b }}
{% endif %}
d

Will render if b=1 is defined to
a

1

c

Preserving the newline.


Compact option for messages

Instead of sending two messages for AFK and PROTECTION status, be able to define just one message.

The way the mod is coded currently, it's not quite straight forward, because they are totally separate functions with separate hooks.  In the future, I do plan on re-writing a lot of the mod with a more modern coding style / interface / config format.

And to be clear, I do have plans for doing somewhat of a mod re-write / overhaul in the coming future.


Pre-GSoC Tasks 2024

The following are Pre-GSoC tasks for 2024
Task 1:
Design a Notifications Service System for Codelabz
Objective:
Design a robust system for the Notification Service for Codelabz, ensuring effective communication of necessary actions and events to users.
Tasks:

Document an Entity-Relationship (ER) Model based on the existing database model.
Create an Architecture/System Flow Diagram illustrating the notification system's workings.

Task 2:
Design a recommendation service to Generate User Feed
Objective:
Design a basic and robust recommendation system to generate user feed for Codelabz, suggesting relevant codelabzs (tutorials) to users.
Tasks:

Design a recommendation service to populate user home feeds with recommended codelabzs (tutorials).
Document an Architecture/System Flow Diagram depicting the recommendation system's structure.
Show How the ranking system will work, and explain the integration and working of the recommendation model.
Contributors can add code snippets.

Extra Tasks:
Objective:
To assess contributors' implementation skills and coding practices, candidates are requested to fork the project repository and solve specified tasks. This method prevents PR flooding on the main repository and ensures confidentiality of contributors' solutions.
Procedure:

Fork the repository.
Resolve/Implement the designated tasks.
Raise PRs to your forked repository.
Include PR links in your proposal.

Tasks:
Github Task 1 :

Like/Dislike Feature Implementation:
Implement database interactions for user likes and dislikes.
Implement them for tutorials and comments.
Ensure the feature aligns with the existing database design.

Github Task 2:

Add Image Feature for tutorials:
Users currently cannot upload images while creating tutorials
Fix this issue to upload images in tutorial.

Github Task 3:

Fix Bugs in Tutorials comments
Comments are not shown when posted, needs to be reloaded to view the comment.

Note:
Kindly refrain from submitting PRs to the main repository, as these tasks are not intended for merging into the main codebase. Your contributions should be confined to your forked repository for evaluation purposes.

@shivareddy6 ( sorry for tagging)
can I work on task 1 by creating an ER model or anyone else working on it?

Hi can you assign this task to me ? I want to work in this issue.

Hi can you assign this task to me ? I want to work in this issue.

Hey @ishanguptaofficial  these are tasks related to GSOC selections . You have to make the prs in your own cloned repo and mention them in your proposal . Hope this helps :)


Add an option to change the EFS performance mode

There are two EFS performance modes, and some users will benefit from using the Max I/O Performance Mode
https://aws.amazon.com/premiumsupport/knowledge-center/linux-efs-performance-modes/
https://docs.aws.amazon.com/efs/latest/ug/performance.html#performancemodes

Changes merged to master branch.


Add FAQ Section to Documentation

Description
Enhance the user experience and provide additional support by incorporating a Frequently Asked Questions (FAQ) section into the documentation. This section will serve as a valuable resource for users, addressing common queries and concerns. Please contribute your insights and frequently encountered questions to help create a comprehensive and user-friendly FAQ that enhances the usability of the documentation. Let's collaborate to make the documentation more accessible and informative for everyone.
-->

Hi Sigstore Team
I would like to work on this issue I have a good understanding of writing documentation


Fresh Install 'npm run dev' fails on Mac

Found an issue or bug with electron-vue? Tell me all about it!
Questions regarding how to use electron or vue are likely to be closed as they are not direct issues with this boilerplate. Please seek solutions from official documentation or their respective communities.
Describe the issue / bug.
# On a fresh install, I run npm run dev and it gives the following error:
  SyntaxError: Unexpected token {       at Object../node_modules/electron-localshortcut/index.js (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:249:1)       at __webpack_require__ (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:21:30)       at eval (webpack:///./node_modules/electron-debug/index.js?:3:23)       at Object../node_modules/electron-debug/index.js (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:177:1)       at __webpack_require__ (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:21:30)       at eval (webpack:///./src/main/index.dev.js?:2:1)       at Object../src/main/index.dev.js (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:505:1)       at __webpack_require__ (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:21:30)       at eval (webpack:///multi_./src/main/index.dev.js_./src/main/index.js?:1:1)       at Object.0 (/Users/jbedford/Sites/lando-gui/dist/electron/main.js:528:1)
How can I reproduce this problem?
# Follow the instructions given to install and start this.
`# Install vue-cli and scaffold boilerplate
npm install -g vue-cli
vue init simulatedgreg/electron-vue my-project
Install dependencies and run your app
cd my-project
yarn # or npm install
yarn run dev # or npm run dev`
I utilized npm install and npm run dev
Tell me about your development environment.

Node version: 12.10.0
NPM version: 6.13.0
vue-cli version: (if necessary) 2.9.6
Operating System: macOS 10.14.6

I also encountered this Bug

try to this way:  yarn add<EMAIL_ADDRESS>

Works like a charm!

npm install<EMAIL_ADDRESS>

@yangjiakun326 @derekzhan Does it ship with a broken version of electron-localshortcut or something?

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

What is your node version？

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

What is your node version？

6.13

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

What is your node version？

6.13

I think you should change node version  to  10.16.0

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

What is your node version？

6.13

I think you should change node version to 10.16.0

I apologize. My NPM version is 6.13. My node version is v12.13.0.

It is now just giving another error: 'Process is not defined'
  ERROR in Template execution failed: ReferenceError: process is not defined

  ERROR in   ReferenceError: process is not defined

    - index.ejs:11 eval
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:11:2

    - index.ejs:16 module.exports
      [.]/[html-webpack-plugin]/lib/loader.js!./src/index.ejs:16:3

    - index.js:284
      [lando-gui]/[html-webpack-plugin]/index.js:284:18

    - task_queues.js:93 processTicksAndRejections
      internal/process/task_queues.js:93:5

What is your node version？

6.13

I think you should change node version to 10.16.0

I apologize. My NPM version is 6.13. My node version is v12.13.0.

never mind.you can try to change node version to 10.16.0

I also encountered the problem, and changing node version to 10.17 fix this

Found in latest electron@8, the default for nodeIntegration changed from true to false. You can enable it:
new BrowserWindow({
    webPreferences: {
        nodeIntegration: true
    }
})

Then you can run electron with Node12 or 13.


Compilation failed on ios

When I'm running tns build ios .. compilation failed and show this error messages below:
Analyzing dependencies
Downloading dependencies
Using GoogleMaps (1.12.3)
Using IQKeyboardManager (4.0.0)
Generating Pods project
Integrating client project
Sending stats
Pod installation complete! There are 2 dependencies from the Podfile and 2 total
pods installed.
Processing node_modules failed. Exception: Pod install command failed. Error output: 2016-03-24 02:02:41.086 ruby[2692:42624] Failed to save project: "/Volumes/Data/<ProjectDir>/platforms/ios/Pods/Pods.xcodeproj"

I thought there is a problem with my internet connection so the pod can't be installed but after check my connection, everything is okay
And I'm using:
El Capitan 10.11.3
XCode 7.2

Any clue?

Don't know if this is an issue with this plugin @patuku-roy . Try uninstalling the video player plugin and see if your project builds. If it does then it might be the plugin but we don't use any pods with this plugin. It's all baked in controls provided by android/iOS. So this is likely not an issue with the video player.
You might try pinging the slack channel for nativescript to see if someone more familiar with ios can help you with this.

hi @bradmartin I think I found the problem, I think this is related to NS 1.7 file structure changes ... let me check first about this and I think so, this plugin does not use any pod at all, so maybe this is my bad, but thanks anyway for your fast reply, really appreciated.


Batch of bugfixes for Kit

This PR aims to address a multitude of bugs related to Drizzle Kit.

#1272, #2184 & #1765: (Pg) Escape single quotes in enum values.
#2122: Escape single quotes in default value.
#2390: (MySQL) Add columns before indexes in migrations.
#1804: (Pg) Don't add quotes around time and vector types when config is provided.
#2396: Primary key name not surrounded with quotes on table alteration.
#2344: (Pg | MySQL) Correctly recreate primary key on table rename.
#2899 & #2905: (Pg) Handle sql values for date, time and timestamp.
#2623: Use run instead of execute on SQLite queries that don't return anything.
#2598: Enum name when defined is now wrapped in quotes.
#2935: Add index operator even if it isn't in the internal vectorOps array.

Some issues that have already been solved in latest and added tests to confirm such. These include: #2299, #2326, #998.

I'd love for 0e392d6 to get shipped. How can I help move this forward?

Apologies for the delayed review on our side. It would be great if you could resolve all the conflicts so I can merge it. I'm currently spending a few weeks focusing on PR merges and bug fixes rather than working on new features

Apologies for the delayed review on our side. It would be great if you could resolve all the conflicts so I can merge it. I'm currently spending a few weeks focusing on PR merges and bug fixes rather than working on new features

I'll update this PR soon.


UI-View is not showing a default content before state takes effect

If I need a landing page, the default content between the the element is not showing until the link is clicked. " This is my landing page. You will see this content changing  when you click on menu. This really sucks as this text is not showing up until a state is activated "

Did you mean to paste an example?
If you want a landing page then make a state with one in it and go to that first.

You might get this right,
How would you do this simple page with ng-ui when I DO NOT WANT TO or NEED TO chop all the < div > into a separate .html file (I really don't understand why should I have to have 100 .html files?)
http://www.codeply.com/render/DRBmOoChfM

Thats really one page and you don't need states so much as a directive that looks to see what tab should be active (should be in the url somewhere) and then scrolls to it.

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


[FEATURE] Add linkfree-cli support

Description
Changes required
1) Add linkfree-cli as devDependencies
Steps:

Fork the repo
Then run

npm run linkfree-cli

2) Add a script in the package.json file.
"create": "npx linkfree-cli"

Additional information
Link to CLI repo - https://github.com/Pradumnasaraf/LinkFree-CLI
Link to NPM rej - https://www.npmjs.com/package/linkfree-cli

hi, can i work on this?
Also adding it as a dev dependency should mean right? correct me if i'm wrong
npm i linkfree-cli --save-dev

hi, can i work on this?
Also adding it as a dev dependency should mean right? correct me if i'm wrong
npm i linkfree-cli --save-dev

+1


Fix multiple connections for a single user in Redis and MemoryLeak

Version of https://github.com/aspnet/SignalR/pull/2272 with the memory leak fixed. (TODO: Still need to manually verify that the leak is gone)
Used the _connectionLock because it was only used for connecting the Redis and the areas I added it shouldn't be under high contention. Can change it to separate locks if we want.
@anurse

Confirmed memory leak gone, creating unique connections and closing them over and over again, capped at ~184MB, attaching debugger shows only current connection in the dictionary

I'm not sure about this lock. We have to take it through the whole add/remove action. The only place we don't take it is when the subscription fires, in which you capture the UserData object anyway...

Why do we have 2 different fixes here? The RedisSubscriptionManager should be in the release branch. It's just a better overall fix with less duplicate code.

Changed the base to be the other PR. We have 2 PR's because this one is focused on fixing the memory leak and the other is fixing the bug. If shiproom is ok with the bug but not the memory leak fix then we have the change ready.

I changed the base back to release/2.1, so we can review this as the complete change.
I think the lock is limited enough that the throughput impact would be minimal. It is only take during connection/disconnection, so throughput would be mostly affected by scenarios where lots of connections are being established/terminated rapidly. Most SignalR usage involves medium-to-long-running connections.


redhat-lsb not installing on GCP Centos 7

Attemping to run invidious_update.sh on Centos 7 without redhat-lsb, won't install.
sudo ./invidious_update.sh
✗ Looks like redhat-lsb is not installed!
Do you want to download redhat-lsb? [y/n]? y
➜ Installing redhat-lsb on CentOS Linux release 7.7.1908 (Core)...
✗ Error: could not install redhat-lsb!
✔ Done
✗ Looks like redhat-lsb is not installed!
Do you want to download redhat-lsb? [y/n]? n
./invidious_update.sh: line 177: lsb_release: command not found
./invidious_update.sh: line 177: lsb_release: command not found
./invidious_update.sh: line 194: lsb_release: command not found
./invidious_update.sh: line 210: lsb_release: command not found
✗ Error: Sorry, your OS is not supported.
workaround, install it manually with yum

Hello,
might be because you ran the script with sudo.
Try again with ./invidious_update.sh as root.
E.g su root then ./invidious_update.sh
Or enter root password when prompted running ./invidious_update.sh


update readme

OPERATOR_IMAGE_VERSION is now required with the new featuregate approach. If version is not set the operator starter fails with:
cluster failed with : unable to determine features: missing desired version "" in featuregates.config.openshift.io/cluster
Same change as for Azure File operator: https://github.com/openshift/azure-file-csi-driver-operator/pull/70

/lgmt
/retest-required

/lgtm

/label docs-approved
/label px-approved
/label qe-approved

/retest-required


Backspace at end of empty line doesn't delete whole line and go to end of above line

VSCode Version: 1.19.2
OS Version: W10LTSB

Steps to Reproduce:

Create a new line of a block of multi-line code like an opening div tag or JavaScript or PHP function, then hit Enter.
If you have an indent, continue to step 3. If not, create an indent with Tab or 4 spaces (whatever your religion is).
Hit the Backspace key; notice it merely removes an indent each time instead of deleting the whole line and putting your cursor at the end of the previous line like awesome JetBrains software does. Realize how many unnecessary keystrokes you'll now have to deal with (and Ctrl+Shift+Up-End each line is still a lot).
Continue to use VS Code because nobody can afford to buy another JetBrains license for your new job.

This should be standard logic in all editors; there's zero functional or stylistic reason to want to just delete indents on an empty line with Backspace (unless there's too many indents there which is rare, or your editor auto-indented wrong), and even if for some Pan-like reason you need to use that regressive space, you can just use Ctrl+Home to delete the line then indent where you want (or Shift+arrow, or just arrow if you're a weirdo).

Does this issue occur when all extensions are disabled?: Yes

I like this feature idea, but one other thing to think about is the interaction with the editor.trimAutoWhitespace setting. If editor.trimAutoWhitespace is false, then I might want to delete some whitespace after hitting Enter, while still keeping the new line. And in those situations, reaching for the Backspace key would feel like the most natural way to do it.
So if this is implemented, I'd like to see a setting to turn it off for the rare cases when I might actually need to backspace over some auto-inserted whitespace (e.g., if I've turned editor.trimAutoWhitespace off for some reason).

Auto-adding whitespace should happen only after hitting Enter on an indented line, and only be added up to the beginning of the previous line or one indent more if it's inside a nesting (or maybe two more such as inside an HTML tag or PHP array so that the inner contents aren't on the same line as the following block segment), and I could see how it would be useful to just delete the empty whitespace on the line (instead of deleting the line), but this means two Backspaces to delete the line.
It's easy to see how Eclipse, JetBrains, and NetBeans editors handle Enter indentation and Backspace on empty or indent-only lines, though the JetBrains way has been the most useful and avoided the most keystrokes in my usage of all code editors I've used.

That is something that should be implemented natively in VS Code. But for now I found this extension really useful https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete
Saves up hours a day :)

If #71506 is implemented, that would probably solve this one as well.

This extension seems like it would solve your issue. It worked for me.
https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete

+1

Still need this added!

@edgarwideman Have you tried this extension yet? It worked almost as good as how the JetBrains IDEs do it.
https://marketplace.visualstudio.com/items?itemName=jasonlhy.hungry-delete

I did find it after dropping previous comment. Thanks anyway. 👍

Related on Stack Overflow: Move cursor up after deleting a line in VSCode

I posted a possible solution on SO: https://stackoverflow.com/a/78290422/836330

It is just a keybinding, no extension neccessary.


Added GitHub action to automatically build PyTTa distribution package. If it succeeds, it is uploaded to PyPI

Currently PyTTa version from PyPI repository is too distant from the GitHub repo. This problem is addressed with a GitHub action. There is a full explanation at .github/workflows/README.md

At https://pypi.org/project/pytta-deploy-test/0.1.0b9/ there's an example of PyTTa where the build and deploy was fully automated by this workflow. I will delete it latter.

To my understanding, it is really useful to have some automations, although this one would imply that each and every modification commited to this branch would automatically update PyPI, which, in it's side, would only accept uploads from new versions, which would then imply that for the automation to work, every push to GitHub must have a new version number on setup.py file.
Deploying the software continously is not what I see as a higher priority right now, since we are currently on heavy documentation process and we don't have (yet) a contribution workflow that can address the update-version-before-push process that this automation requires.
This rolling versioning wasn't the original plan, but it indeed is a way of versioning software, and lots of open source projects work like this. Still, good guidelines on how to contribute with code are needed to have a better understanding of which versioning fits PyTTa better.

I'm glad to hear that! You're right, the only thing needed is to setup PyTTaMaster secrets for PyPI and it should be good to go.

@jvitorcarli , one more consideration. Only the most stable version should go to pip. For now, we could tag the most recent beta version. After v0.1.0 we won't tag beta versions anymore (hell yeah).


Error: Require is not defined

idk what this means or how to fix it but Desmodder does not work at all (menu does not open) (even though it did in the past) on Firefox 90.0 (64bit)

That error shouldn't be a problem. Have you tried updating Firefox? Latest version is 108, so your browser is a year and a half old.
If the update doesn't work: what error do you get when you click the menu?

The error should no longer appear. Desmos has removed that script tag.


Why are passwordless methods showing in the AAL1 table?

The table under "Permitted authenticator types" says "The following table contains those not covered in AAL2 and AAL3." but it contains the passwordless credentials (FIDO, WHFB) which meet higher AAL standards. It's confusing and conflicting, and I'm not even sure how to best edit it for a fix.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c6238f33-50fe-ea10-ca62-1bc070b3fea4
Version Independent ID: 56a9868b-6c21-c347-7cd1-efb68c81c0b6
Content: Achieve NIST AAL1 with Azure Active Directory
Content Source: articles/active-directory/standards/nist-authenticator-assurance-level-1.md
Service: active-directory
Sub-service: fundamentals
GitHub Login: @BarbaraSelden
Microsoft Alias: baselden

@libbro2006
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@JamesTran-MSFT I updated the article to clarify the confusion. Please close this issue. Thank you!


ApplicationEventEmitter

Summary
ApplicationEventEmitter can be used to create and emit custom events to facilitate communications between the emitting component (emitter) and the listening component (listener). The behavior of the event emitter is similar to the "pub/sub" model. When ApplicationEventEmitter emits an event, listeners that have registered to listen to this event can act upon receiving the event.
Added the lightweight eventemitter3 dependency to leverage some performance gains.
Closes: #23
Deployment Link
https://terra-applic-event-emit-smhshu.herokuapp.com/application/terra-application/components/application-event-emitter
Testing

Additional Details

Thank you for contributing to Terra.
@cerner/terra

@benbcai could we write up a guide walking through adding an event emitter to a react component and cleaning it up when the component goes away?

@benbcai could we write up a guide walking through adding an event emitter to a react component and cleaning it up when the component goes away?

Yes, having one would be even more helpful for consumers.


[refurb] Implement single-item-membership-test (FURB171)

Summary
Implement no-single-item-in as single-item-membership-test (FURB171).
Uses the helper function generate_comparison from the pycodestyle implementations; this function should probably be moved, but I am not sure where at the moment.
Update: moved it to ruff_python_ast::helpers.
Related to #1348.
Test Plan
cargo test

Thanks, looks great.


ebml2xml change

Changed options in ebml2xml util to reflect standard use, and updated the documentation.
Making --pretty and --min options default to on, and adding the reverse --single and --max, for folks who want the output in a single line flooded with debug info.

Codecov Report

Merging #95 (eafc969) into development (f7da7ef) will increase coverage by 0.05%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development      #95      +/-   ##
===============================================
+ Coverage        80.00%   80.05%   +0.05%
===============================================
  Files                6        6
  Lines             1155     1153       -2
===============================================
- Hits               924      923       -1
+ Misses             231      230       -1

Impacted Files
Coverage Δ

ebmlite/core.py
76.97% <0.00%> (+0.08%)
:arrow_up:

Continue to review full report at Codecov.

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


Update rush version to latest to build transitive dependency correctly

Pass all dependency packages when testing any dependent package as a work around to get non-GA version required project to be built.

Rush 5.39 doesn't support Node 8. I am going to try rush 5.38 and check

@mikeharder @sadasant Unfortunately test has failed on Node 8 since it is not support by rush from 5.36.2 onwards. I had offline chat with Daniel and we cannot update dev dependency of other packages to identity beta version due to the nature of changes. I think we will have to go ahead with my previous work around until we figure out what needs to be done with Node 8 support.

This is short term fix until we we figure out Node 8 support in rush.


new-command: yammer report activitycounts

implements #1383

Coverage remained the same at 100.0% when pulling 4ae8f49e8b2a07fcb005eee2cde4a4a3320602f6 on plamber:yammer-report-ActivityCounts into f7f194d36ec571e84edc736ac0c7d821534abd54 on pnp:dev.

Awesome! We'll review it shortly! 👏

Merged manually. Thank you! 👏


fix: use konflux image

In the https://github.com/RedHatInsights/yuptoo/pull/107 , the Konflux pull request CI seems was not triggered in Konflux, no record for that PR in Konflux. And the re-runaction in github side triggered nothing to Konflux pipeline: Red Hat Konflux Failing after 2596m — pipelinerun start failure
Re-open for PR https://github.com/RedHatInsights/yuptoo/pull/107 to try to re-trigger the connection for Konflux pull request CI .

/ok-to-retest

Since this PR was opened with the same commit id from https://github.com/RedHatInsights/yuptoo/pull/107, the Konflux related CI build was not retriggered. Closed here.


Inheritance

https://github.com/Zarel/Pokemon-Showdown/pull/649
I have come to agree that getCategory is a better approach over the time though.


Turn in our app idea to teachers

Title:
Find Your Spirit Dog
Project Description:
A simple, image based quiz clicking yes or no if that trait describes you. The quiz takes about 90 seconds to take. At the end the user will be presented what dog breed fits their personality, for if they were a dog. It will also include a description of traits of that dog breed.
User Story:
As a dog lover I’ve always been curious what breed.
I want to put in traits about myself.
So that I can figure out what dog breed has those same type of traits, giving me my spirit dog.
Wireframe/Sketch of Design:
APIs to be used:
Traitify
Rough breakdown of Tasks:


Short solution needed: "Wine dataset" (python-scikit-learn)

Please help us write most modern and shortest code solution for this issue:
Wine dataset (technology: python-scikit-learn)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to load wine dataset


Swap Callback and Context positions in TS generics

Allow using useLongPress<Element, ContextType>(...) syntax instead of useLongPress<Element, LongPressCallback<Element, ContextType>, ContextType>(...)

Implemented in v3 alpha version use-long-press@alpha


Configuration option restrict_orgs does not fully restrict assignable organisations/funders in plan update controller

Please complete the following fields as applicable:
What version of the DMPRoadmap code are you running? (e.g. v2.2.0)
3.0.4
Expected behaviour:
If you set the configuration option config.x.application.restrict_orgs = true,
one should not be able to create an unmanaged organisation, not?
Actual behaviour:
This line
allows you to create a new (unmanaged) organisation.
Now we have a long list of unmanaged organisations of type "organisation",
because people tend to fill in a custom name in the field "funder" on the
"project details page". Afterwards they cannot find their name in the lookup
because that custom organisation was not marked as funder (and also because
it is unmanaged).
Not sure if that is intended behaviour?
These unmanaged organisations tend to pollute the list of organisations.
Is it worthwhile to add an organisation that is only used in one plan?

Hi @nicolasfranck we were just discussing this issue last week. You are correct, the code on this page should be using the 'restrict_orgs' flag. I think we just missed it when implementing.
We have not yet issued a patch for this.
 In dmponline we are patching in the meantime to stop it. @briri is currently working on a more comprehensive solution to this issue as there are multiple options for orgs and funders related to user and plan creation.

Possibly connected to: https://github.com/DigitalCurationCentre/DMPonline-Service/issues/650

This has been fixed and in latest release.


Improve error messages for API responses season

Rabbit season

Failed Test
Failure Summary
Logs

[chromium] should show the privacy policy correctly.
The test timed out waiting for the privacy policy to be displayed correctly.
Logs ↗

Test_BadRequest
failure_message: Failedfailure_text: === RUN   Test_BadRequest    api_test.go:53: Bad request sent to API--- FAIL: Test_BadRequest (0.00s)
Logs ↗

Flaky Test
Failure Summary
Logs

[chromium] should display user profile information correctly.
The test timed out waiting for the element 'User ID: eli3cruz' to become visible.
Logs ↗

[chromium] should display error message for invalid login correctly.
The expected HTTP status code 401 was not received, instead a function was returned.
Logs ↗

[chromium] should return list of users correctly.
The API call to get the list of users failed with a 404 Not Found error.
Logs ↗

[chromium] should login with credentials correctly correctly.
The login request to the server failed with a 404 Not Found error.
Logs ↗

... and 2 more

View Full Report ↗ ⋅ Docs


How to send the sensor-timing measurements in good way?

All timing-values always (no matter having gotten a trigger or not)?
Only triggered channels/sensors (e.g. filtering which is to be sent over to Unity).
Only max of xx sensors? (xx then decided upon by some criteria...)

Not sure what trigger you mean here, could you write full questions please?
Max number of sensors depends on the microcontroller in use and method of
capturing pulses. Currently a teensy can serve 3 sensors using comparator
method, plus 6 more using hardware timers (in works now). Photon can serve
6 sensors if I remember correctly.
On May 22, 2017 12:07 PM, "GameFaceLabs"<EMAIL_ADDRESS>wrote:
All timing-values always (no matter having gotten a trigger or not)?
Only triggered channels/sensors (e.g. filtering which is to be sent over to
Unity).
Only max of xx sensors? (xx then decided upon by some criteria...)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ashtuchkin/vive-diy-position-sensor/issues/28, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAmVHWRyFrYpYLzzwHzLbNDGYCYzmiEVks5r8dzfgaJpZM4NiwlH
.


update dependency and interface for bytes

Running sanity tests locally.   This PR is open now for CI parallelization.

This seems merge ready to me based on this sanity test result:
        PASS [ 335.898s] libtonode-tests::concrete slow::witness_clearing
------------
     Summary [1147.899s] 332 tests run: 332 passed (109 slow), 31 skipped
1731623057 || shofth nattyb bump_bytes[ ]
REDACTED/bump_bytes: 0 $
git show -s
commit a6b24a2b443173a6e62b63538ca9bfa638e9b258 (HEAD -> bump_bytes, zancos/bump_bytes)
Author: zancas<EMAIL_ADDRESS>Date:   Thu Nov 14 15:00:56 2024 -0700

    update dependency and interface for bytes
1731623063 || shofth nattyb bump_bytes[ ]

Obsoletes #1173


support terms of service

as raised in #18

TODO: A demand for proof should be able to include terms of service that the verifier commits to upon receipt of the data. We don't need to define that in great detail. I know that TOS was a rat hole in VC spec work. But we need to allow for it, or else the spec is unusable in many situations.

Good idea, our folks have something similar, called consent in their one-off formats in use today. Can someone propose a PR that covers TOS, consent statements, etc.?

@symorton We have a consent field in our MSFT-centric data format, and this seems very close to that in terms of a feature request. Do you want me to add something to the PE spec that can cover these two related needs?

I remember discussing this in the context of GDPR. In our (Workday's) flow we have a reason field that must be provided when requesting data. IANAL but it seems like an easy step to include a TOS/Reason field to be complaint.

Feeling of the WG is that this is out of scope for the PE data model, but we may want to describe why and what an implementation might do orthogonally to fulfill this need.

I am 100% not a lawyer, but as I understand it, consent and purpose are to separate GDPR values-- you can consent to giving a company X piece of data, in accordance with Y terms of service, for Z purpose.  That company can turn around repurpose X without breaking the terms of service Y or your consent, but this repurposing is also (separately) something you'd be liable for under some interpretations of GDPR. In the GDPR OAuth work I've seen ( @mehdim , care to chime in?), they are separate parameters sometimes used in combination...

I wasn't a participant in the discussion as to why purpose made it in but something like TOS did not. Could someone summarise to me the rationale of the group?
Our presentation definition (equivalent) contains a terms - it must be a uri. Submission can only occur if consent is obtained.
On a related note it also contains an optional duration field which is used to clearly inform the holder the period of time that the credentials will be retained by the verifier


Ps move controller firmware

Just curious as ive read a bit about a 4.5 psvr firmware update, would it be wise to try and update the firmware of the move controllers? Would it brick the current psmovesteamvrbridge?

Probably not... and then you won't be able to calibarte the magnotometer.


Colorize docs are wrong

Take a look at the docs:
https://crystal-lang.org/api/1.3.2/Colorize.html
For example:
require "colorize"

"foo".colorize(0, 255, 255) # => "foo" in aqua

# This is the same as:

"foo".colorize(Colorize::ColorRGB.new(0, 255, 255)) # => "foo" in aqua

^ None of the above work
require "colorize"

"foo".colorize(Colorize::Color256.new(208)) # => "foo" in orange

Doesn't work!
The only thing that did work was:
"foo".colorize.fore(Colorize::ColorRGB.new(0, 255, 255))

Color256 is doing the right thing:
"foo".colorize(Colorize::Color256.new(208)).to_s.chars
# => ['\e', '[', '3', '8', ';', '5', ';', '2', '0', '8', 'm', 'f', 'o', 'o', '\e', '[', '0', 'm']

It works on my Visual Studio Code terminal which has TERM=xterm-256color.
I think we only need to add the missing RGB overload(s).

To compensate that fact that the docs are wrong, I'd like to thanks whoever added this to colorize. I wanted to colorize a text with rainbow colors and after I found out how to do it it was a breeze!


Implement query serialization for UUID ClickHouse type

To close this issue you need to uncomment test
t1 :: Connection -> IO ()
t1 conn = do
  ...
  -- ToDo: runTestForType @ChUUID conn [minBound, toEnum 0, maxBound]```

which causes compilation error
No instance for ‘ToQueryPart ChUUID’

and implement
instance ToQueryPart ChUUID where
  toQueryPart (MkChUUID w128) =  undefined

which passes the Test1
You can take uuid-types implementation as a reference

If this task is actual, I would like to do it.

Hi @lambdajon!
Every issue is relevant, so feel free to open a PR!
Also feel free to suggest any improvements to the documentation or project structure
I see in your repository that you are familiar with Nix, so I think you can start with testing documentation

Hi @lambdajon!
Okay, I'll try to help with this too!
I'd also like to contribute to your repository for practice in Haskell, if you have any tasks for beginners, please post them in the issues!
Thanks for answer!

My personal goal is to create friendly contribution environment. I always add  good first issue to issues I find suitable for newcomers in Nix/Haskell/ClickHouse. I'll put more effort to create issues, especially for newcomers.
Thank you for your interest! Feel free to ask for any help and please leave as much feedback about your experience as your can. I was working with this project a long time and there are might be a lot of opinionated decisions and missing information. Any conversation will help you to learn project too.


Better API for Triggers - WithTrigger(event, detail, timing)

I didn't expect to do 2 PRs in one day but here it is 🥳 My development team has been using this library with their more recent builds and the biggest grumble they have is what follows, so good work and thank you for building something that is just right for working with HTMX!
The current API for triggers (.Trigger(value)) provides the ability to set the header once, meaning that in its most basic form a trigger with no parameters can be specified:
h.Trigger("event-name");

For more control, a manual string would have to be constructed:
h.Trigger("{\"event-name\":{myVal:42,anotherVal:12}}");

For multiple events, the interaction only gets more complex and the user would typically resort to manually constructing JSON or serialising a dictionary:
h.Trigger("{\"event-name\":{myVal:42,anotherVal:12},\"another-event\":{what:false}}");

With that I propose a new API for adding triggers to the response which does the dictionary serialisation method internally, the above details with parameters could now be rewritten as follows:
h.WithTrigger("event-name", new { myVal = 42, anotherVal = 12 })
   .WithTrigger("another-event", new { what = false });

Of course this works the same with or without detail being provided for the event, and timing can also be overridden (to push to an alternative header):
h.WithTrigger("event-name", timing: HtmxTriggerTiming.AfterSettle);

Notes:

The Readme has been updated with this (and also a note about the Antiforgery tag, but feel free to remove that)
The header builder, if there is a single event with no params, will only output the event name (rather than full JSON) in the spirit to save a few characters. Every little helps!
There are tests that cover off all scenarios, I think, so let me know if you need to see more

Major note:
I built this against .NET 6, rather than 5.0. I appreciate it's a big change (and the Action will need updating) however given that 5 is out of support it's probably worth switching to the LTS. (That's why the action is currently failing)

I like this idea. Let me look at updating the action, so it builds successfully.

Glad you approve - we're actively using it and it just feels natural 👍
Fairly primitive change but happy to make any amends.

Tests added to detect usage of legacy / new APIs together (which wouldn't work as they would override each other)

I will merge and deploy this tomorrow. Should I bump this package up to v1.0 at this point?

Feels stable to me, so v1 makes sense.

There's not much missing from the surface either, so it's probably a good candidate now that it's aligned with the latest version of HTMX.
We added another couple of helpers internally (things such as a new tag helper outputting elements that request via HTMX on load) that may be useful, but I may propose later if they prove useful and general purpose.

LFG!!!!!!!

In the subsequent releases, I'll start removing the obsolete methods. Thanks, everyone. The package in its v1.0.0 should be out on NuGet soon.


Installing with npm to Laravel project

How am I supposed to get this to work with NPM and my Laravel project?
I've tried to add it into my app.scss file with
@import "~animate.css/animate.min.css";

or
@import "node_modules/animate.css/animate.min.css";

Even tried to add this to my bootstrap.js with
require('animate.css');

None of these worked, how can I get this to work?

Nevermind, had to run npm run dev to get it to work!


Greenkeeper/@nartc/automapper 6.3.3

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Dev

Codecov Report

Merging #32 (e369955) into main (2fbbd9a) will increase coverage by 1.85%.
The diff coverage is 94.02%.

@@            Coverage Diff             @@
##             main      #32      +/-   ##
==========================================
+ Coverage   78.22%   80.08%   +1.85%
==========================================
  Files          10       11       +1
  Lines         418      477      +59
==========================================
+ Hits          327      382      +55
- Misses         91       95       +4

Impacted Files
Coverage Δ

ghga_service_chassis_lib/config.py
92.45% <81.81%> (-7.55%)
:arrow_down:

ghga_service_chassis_lib/object_storage_dao.py
87.01% <100.00%> (+5.53%)
:arrow_up:

ghga_service_chassis_lib/postgresql_testing.py
100.00% <100.00%> (ø)

ghga_service_chassis_lib/s3.py
73.29% <100.00%> (+2.74%)
:arrow_up:

Continue to review full report at Codecov.

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


[#13251] Allow for a way to limit request value sizes in the server

Fixes #13251
Supports RESP, HotRod and Memcached server protocols.
Adds a max byte array size and max key count limits. The former prevents any byte[] read from a client request to never be be allocated if it is larger than the configured size. The max key count limits any collection from being allocated if it is said to contain more than that number of entries.
Also added a check to the RESP number parser to not allow it to go longer than 8 bytes (size of a long).

I also toyed with making a new command for the client to retrieve what the configured maximum values are.. but I am second guessing that feature. Especially as our client would have to then submit in parallel (defeats most of this feature) or we have to do them sequentially. I am still open to how we may want to handle this though.

Added parser and docs, but rethinking again I think we should have a default value that is greater than 0 for these values. Our REST endpoint currently limits an entire request to 10MB, so I think that should work for the byte array at least. And maybe 500 key entries.


CT: Entire Test suite test with latest

@swe12345 Take latest official release from the link provided
Take both latest BMC and PNOR from the website.

Closing as it completed for the given sprint


Clean up local flake8 (and any other lint config) files

I noticed a few subdirectories had some local configs, I'd like to clean these up and just use the top level one if possible. It looks like the only thing that's failing for mivideo is the line length which we are overriding at the top level.

This was a good change; it's better to have only one file here. We just need to accommodate @lsloan's preference for camelCase in the mivideo code going forward. Marking as Done.

PEP-8 dosen't accommodate for camelCase. If we have any for function names they should probably be renamed.
https://docs.quantifiedcode.com/python-anti-patterns/readability/using_camelcase_in_function_names.html

I personally don't think this is worth fighting over, but PEP does seem more rigid about it then I remember. flake isn't going to catch these unless we add a package, according to this StackOverflow post: https://stackoverflow.com/questions/28725116/flake8-doesnt-report-mixed-case-function-names


Add ability to subset from mesh coordinate system in PhotogrammetryCameraSet.get_subset_ROI

The get_subset_ROI method only works when lat-lon information is provided about each camera. I'm in a situation where some of the cameras are missing this information. It would be nice to subset based on the mesh coordinate system instead, which is the value recorded in the camera transforms. This option could be added as a flag to this method.

More specifically, would we get this info from the cam_to_world_transform matrix? I recall we can get the position (x,y,z,0) from the right hand column of the matrix.

This issue is a bit stale so things have changed a little. Yes, you're on the right track with how to parse the camera location. Since this issue was written, the default way to get the lat/lon for a MetashapeCameraSet was changed from using the image EXIF to using the camera location, transformed into lat/lon coords. You can see the parser code here.
However, a related issue still remains. We shouldn't require that the camera set can be represented in geospatial coordinates to allow this ROI operation to happen. For example, the concept_figures scene is entirely synthetic and has no notion of a coordinate reference system (CRS). It would be good to stop explicitly using lat/lon for the ROI and instead use a more general notion of "2D location". If the camera set doesn't have geospatial information, the 2D information would just be X,Y.
There would be a fair bit of logic here to ensure that the ROI input matches what's expected for the scene. For example, we should probably start supporting shapely.Polygon/MultiPolygon ROIs, since these would be a good choice for scenes without a CRS.

Based on the discussion, I think what we want to do is start subsetting the camera set using the internal coordinate system of the camera set, which is not geospatial. In some cases we may have the additional information to compute the geospatial location from each camera in other cases we might not.
To support this, we should probably have a convenience method that gets returns the location of the camera in the internal coordinate system. This could be 2D or 3D or have an argument to switch between them.
Then, we'd want to support non-geospatial ROIs. As described, the shapely.Polygon/MultiPolygon would be a good choice for this. But you could also consider using geopandas objects that don't have a CRS. You'll have to do some digging to see if that exists. I think checking whether the ROI has geospatial information (a meaningful CRS) would be a good way to check if the ROI should be in geospatial or internal coordinates. But you could also have this be specified by the user.


Generate completely static page

Currently, the cli tool generates an html file with embedded javascript, which in turn creates the dark mode toggle, github corner, and maybe some other stuff affecting layout/design.
This creates a worse site when viewed with scripts disabled.
Could the generator instead fully generate the final html so the final state will always be loaded directly, without having to run scripts for it to be finished?

@Regenhardt Currently, the dark/light theme switch, Markdown preview (as a web component), and code copying features all rely on JavaScript. Is it possible to modify only the Markdown preview part? If so, I can make the changes quickly.

That could be enough I think. I guess it depends on what other elements are already in the layout before running the scripts, in order for the markdown part to be properly displayed.

@Regenhardt Upgrade v5.0


Error Code: 50058
Request Id: bec4125b-a529-4dc7-be37-1e1dc5c13a00
Correlation Id: eabec10c-7885-4e5b-91ce-4c66b3830f9d
Timestamp: 2023-09-13T02:10:07.739Z

Before reporting a bug, please see our FAQ in FAQ.md!
Description

Failed Step

More Details

Error Code: 50058
Request Id: bec4125b-a529-4dc7-be37-1e1dc5c13a00
Correlation Id: eabec10c-7885-4e5b-91ce-4c66b3830f9d
Timestamp: 2023-09-13T02:10:07.739Z

What is the origin of this error? Can you share more information on your proxy/client?

Error Code: 50058
Request Id: bec4125b-a529-4dc7-be37-1e1dc5c13a00
Correlation Id: eabec10c-7885-4e5b-91ce-4c66b3830f9d
Timestamp: 2023-09-13T02:10:07.739Z

Pınar metiner

What is the origin of this error? Can you share more information on your proxy/client?


Auto scroll bottom

When press enter and cursor goes down scroll bar not going down too
what i do?

This has been fixed in a separate branch. I'll have to release a new version.

Can you let me now when you released new version?

I guess no news on this front!???


Asking for downloading something

Hi there,
When I ran MISIC for the first time, it asked me to 'https://github.com/pswapnesh/Models/raw/master/MiSiDC04082020.h5'. However, there is no such file in that rep. So I got stucked there. Could someone help?
Thank you!
Xinxing

Hello Xinxing, of course we can try to help you. let me a litle time to reproduce it. Maybe could you post the full error message ?


Remove boto related settings that aren't used

Since upgrade to boto3 in commit 12db5176d6c4c we don't need AWS_HEADERS anymore

@browniebroke can you check my commit? I followed the previous style, but I don't know if the bytes workaround is needed in boto3.

I tested this on a Django deployment I own. I've set the headers using a normal string, with spaces and I uploaded a file.
AWS_MAX_AGE = 60 * 60 * 24 * 7  # 1 Week in seconds
AWS_S3_OBJECT_PARAMETERS = {
    'CacheControl': f'max-age={AWS_MAX_AGE}, s-maxage={AWS_MAX_AGE}, must-revalidate',
}

> curl -I $url
HTTP/2 200
...
cache-control: max-age=604800, s-maxage=604800, must-revalidate
...

Therefore, the bytes workaround isn't needed anymore. Thank you for flagging this.

Thanks for your contribution!


Feature support alpine stretch, reduce nacos-server image size

Purpose

Reduce nacos-server docker image size
Support more arch without change main docker image

Support alpine

Add Dockerfile-alpine
Add package bash
Remove ENV JAVA_HOME JAVA from dockerfile
Refine Docker file reduce layer

Support stretch

Add Dcokerfile-stretch
Remove ENV JAVA_HOME JAVA from dockerfile
Refine Docker file reduce layer

Startup scripts modify

Alpine/stretch image ENV JAVA_HOME provider by base image. java bin is in system PATH.
Add ${java} default value java to build/bin/docker-startup.sh,

Testing

build image at dockerhub
Docker hub images
Test run standalone mode in local

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: rong-a-zhu:x: fdiskbrainYou have signed the CLA already but the status is still pending? Let us recheck it.


Dom-Xss Vulnerability

One of our snyk vulnerabilities is a dom-xss potential attacks with the use of innerHtml. This includes a cdn link to dompurify to be able to use a method to sanitize the html. Need to confirm the using this library will pass snyk scans.

I'm inclined to merge it to keep Snyk quiet, then see what shakes out in the v3 rewrite.


SV variants BND: CHROM1:POS1/CHROM2:POS1 is displayed

when looking at the BND structural variants, the POS1 is used as the POS2:
e.g.
https://gnomad.broadinstitute.org/variant/BND_1_13?dataset=gnomad_sv_r2
https://gnomad.broadinstitute.org/variant/BND_1_21?dataset=gnomad_sv_r2
https://gnomad.broadinstitute.org/variant/BND_1_31?dataset=gnomad_sv_r2
for the position: the notation CHROM1:POS1|CHROM2:POS1 is used instead of something like CHROM1:POS1|CHROM2:POS2

As @RCollins13 said, the browser does display position links for BND structural variants as chr:pos - chr2:end.
https://github.com/macarthur-lab/gnomadjs/blob/0fc1abf11dddb56742c80803021277db476c214b/projects/gnomad/src/client/StructuralVariantPage/StructuralVariantAttributeList.js#L59-L71
https://github.com/macarthur-lab/gnomadjs/blob/0fc1abf11dddb56742c80803021277db476c214b/projects/gnomad/data/prepare_gnomad_svs_for_browser.py#L76-L77


Julia Icon

It would be nice to support Julia, since it's a growing language. I don't have any experience with vector graphics, but I hope someone could help me with this. Here is their logo:

The 3 dots is usually used as a compact logo, but I don't know what it would be like in the monochromatic color scheme.

v2.0 is out and most importantly there are instructions for building & contributing to the font directly.  This project is 100% a hobby and I'm happy to publish builds, but I'll rely on the open source community to add more icons.  Please see README.md and DEVELOPMENT.md for more info.


Use as? for dynamic check

Currently as is used for two different things, which is confusing in combination with the implicit lifting

Fxied in e9027b4c676c0