ESM Import Default does not work

Hi! First of all, thank you so much for your effort on this project.
For such a code example like this:
import buffer from 'node:buffer';
I get this kind of error:
com.caoccao.javet.exceptions.JavetExecutionException: SyntaxError: The requested module 'node:buffer' does not provide an export named 'default'
as far as I read from docs, javet is supposed to support it.
this kind of codes are supported from commonjs:
const fs = require('fs'); but why not like import fs from "fs"
Thank you for your attention.

Thank you. Yes, it's supported. Please refer to this doc for details.

I believe there is supposed to be a solution for this rather than a hacky module since nodejs is directly able to run the same code without an issue. Because I'm trying to run a pre-built node app. Such as the output of the SvelteKit application. It provides a simple build folder and under it pre-compiled js files. Wouldn't like to touch pre-compiled files :sweat_smile:

That's not a hacky solution, but the current best way of exposing the Node.js built-in modules as ESM. It's just one line of code dealing with Javet. There are no code changes required to your JS files.

Well, then v8Runtime.setV8ModuleResolver(new JavetBuiltInModuleResolver()); this code does not work for this line in JavaScript script file:
import buffer from 'node:buffer';
it gives the error I provided above:
com.caoccao.javet.exceptions.JavetExecutionException: SyntaxError: The requested module 'node:buffer' does not provide an export named 'default'
this was the issue already in the beginning I had. :sweat_smile:
if I change the line in that pre-compiled JavaScript file like this: import * as buffer from 'node:buffer' it works but this is what I'm trying to avoid.

That's interesting. The unittest works well at import * as fs from 'node:fs'. I'm not aware that import buffer from 'node:buffer' may fail. I'll check it out later. Thank you for reporting this issue.

I think that is a bug of JavetBuiltInModuleResolver. I'll fix it later. In the meanwhile, you may create your own resolver to expose default to make it work so that you don't need to wait for the patch.
I wasn't aware of such bug until you reported it. Thank you.

Thank you for your extra attention.

It's fixed. Please wait for the next release. And you may borrow that few lines of change to your own resolver.

This was fast! And I tried now it works! Thanks again.


No such agreement is required if the software is free

This is not balanced at all: the employees do not have any ownership over the product of their labor. A better approach is to make your software (i.e. GitHub) free software and do not assign IP ownership at all, leaving the product collectively owned by all past, present, and future employees.

That is sadly not possible in some jurisdictions.

@ddevault it depends on the employment contract and jurisdiction. In the US, employers typically own work product; however, contractors can often maintain ownership - especially 1099 contractors - of their work product even as they grant licenses to the companies that contracted them (their employers in some respects).
Many open source projects also try to simplify the legal sides of things using Copyright Assignments (CLAs) - Qt, FSF/GNU, OpenOffice (historically, though that may have changed with Apache taking over) are some of the big ones. This lets the organization migrate licenses easier.
Further, as more and more employees engage with the open source world outside of work there is a new balance that needs to be struck. If someone signs onto an employer and their employment agreement assigns all IP of the employee to the employer whether its work product or not then that creates a large hurdle for the employee to work on open source since they technically have no legal ability to grant licenses of the IP they create during the active term of that employment agreement. And yes, I very recently saw an employer try to have those terms; leaving it to their discretion of when to enforce it - something no open source community would want to have as a risk.
So in the end, it's a really complex issue.


Problems filter tab keeps stealing focus.

This issue is really killing me. I can't find anyone having the same problem, but it's making me much less productive. The simplest way to reproduce it is within my project (typescript), if I save my file with a compilation error in it, focus will immediately jump to the filter line on the problems tab. It varies with frequency and sometimes will even occur without saving being necessary, which means that my cursor keeps jumping around as I type. I've disabled all extensions and still have the issue. Please help. Happy to do whatever I need to in order to provide logs etc.

I opened this with extensions disabled, so it didn't provide the correct system info. #177274 is opened instead.


Re-entry

Need to make sure hitting a breakpoint while in the debugger works fine. If it happens during expression evaluation, we should reset the stack. If it happens during stepping, we should probably break and properly chain the stack. In either case, we should have an option to override the default behavior.

No options in the first version.


Keep contentOffset when data has been changed

I've researched a lot regards to fix contentOffset after table dataSource has been updated. I couldn't find a way to do it with Dwifft. When diffCalculator.row = self.data is invoked with new values (in my case, new values will be added at front of previous data), it changes contentOffset. I attempted to use tableView.setContentOffset(offset, animated: false) after providing new data to diffCalculator which didn't work. Also, when new values (in this case, append to previous data) are added while scrolling, scroll somehow jumps on and off at some weird position. I also noticed, when same values were set to diffCalculator, it treated there were diff. I'd appreciate any advices. Thanks

Unfortunately I don't think this is an issue with Dwifft, but rather UIKit
itself - Dwifft doesn't do anything particularly special w/r/t the way it
inserts/deletes rows. A quick Google search suggests that the issue may
have to do with dynamcially-sized cells and estimated row heights. There is
also the new performBatchUpdates method on UITableView in iOS 11. Would
you please try building your app with the performBatchUpdates branch I
just pushed (which uses this new API when available) and see if it helps?
If not, would you please link to a small project that reproduces the
contnetOffset issue you're seeing? Otherwise it'll be pretty impossible for
me to help. Thanks!
On Wed, Nov 29, 2017 at 8:54 AM, Wooyoung Chung<EMAIL_ADDRESS>wrote:

I've researched a lot regards to fix contentOffset after table dataSource
has been updated. I couldn't find a way to do it with Dwifft. When diffCalculator.row
= self.data is invoked with new values (in my case, new values will be
added at front of previous data), it changes contentOffset. I attempted to
use tableView.setContentOffset(offset, animated: false) after providing
new data to diffCalculator which didn't work. Also, when new values (in
this case, append to previous data) are added while scrolling, scroll
somehow jumps on and off at some weird position. I also noticed, when same
values were set to diffCalculator, it treated there were diff. I'd
appreciate any advices. Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jflinter/Dwifft/issues/80, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtM5yYMTZX4O3v_7gaZwTT3E39Hmn1Fks5s7Yw1gaJpZM4QvSdB
.

Just tried performBatchUpdates branch and it didn't help.

I think that Dwifft should have method  func setSectionedValues(_ values: ..., animated: Bool). This will help people using the library in case of having the same issue. I'm having the same issue, but I want using the library. I can create PR with the proposed functionality.

I also think that performBatchUpdates is reasonable to merge into master. It just uses new iOS 11 functionality.

Can you explain why a setSectionedValues:animated: method would help you
here? I am reluctant to introduce a change like that without a really clear
understanding of the underlying UIKit issue.
On Tue, Dec 19, 2017 at 2:17 PM larryonoff<EMAIL_ADDRESS>wrote:

I also think that performBatchUpdates is reasonable to merge into master.
It just uses new iOS 11 functionality.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jflinter/Dwifft/issues/80#issuecomment-352858941, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAtM56R-dITWgAYDL-i6J-4ylaIecjjsks5tCAu1gaJpZM4QvSdB
.

I completely agree that all changes must be reasonable.
But I think there’re be cases when we don’t need any animations to update table view / collection view, like the current issue or when you need just full table view refresh when you know that completely new data came.
I also want to use sometimes the library as a simple data source without batch updates. But there’s no functionality to disable animations when updating the data.
Sent from my iPhone

On 19 Dec 2017, at 22:44, Jack Flintermann<EMAIL_ADDRESS>wrote:
Can you explain why a setSectionedValues:animated: method would help you
here? I am reluctant to introduce a change like that without a really clear
understanding of the underlying UIKit issue.
On Tue, Dec 19, 2017 at 2:17 PM larryonoff<EMAIL_ADDRESS>wrote:

I also think that performBatchUpdates is reasonable to merge into master.
It just uses new iOS 11 functionality.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/jflinter/Dwifft/issues/80#issuecomment-352858941, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAtM56R-dITWgAYDL-i6J-4ylaIecjjsks5tCAu1gaJpZM4QvSdB
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@intoxicated it looks that solutions is easy, but hacky. Try doing the following after settings sectionedValues tableView.layer.removeAllAnimations()

@larryonoff - upon further consideration, I don't think your suggestion belongs in Dwifft - it's not, and should not be, responsible for how animations to collection views take place - that's what UICollectionViewLayout, etc is for. As a really simple example, you can get un-animated behavior today by doing something like
UIView.performWithoutAnimation {
  diffCalculator.sectionedValues = ...
}

What this thread needs a cogent explanation of how to preserve a table view's contentOffset without any visual artifacts when inserting rows/sections of variable height.  I'm not really interested in pursuing any solutions until that is properly understood.

larryonoff's solution doesn't work for me, and moreover, it seems like I lose scroll contentOffset position unexpectedly when updating data while tableview has scrolled..

Does anybody solve this ?

larryonoff's solution doesn't work for me, and moreover, it seems like I lose scroll contentOffset position unexpectedly when updating data while tableview has scrolled.. I'm investigating more on this topic atm tho

Fix you find any solution?

One of the possible solution - perform update in [.default] runloop mode
RunLoop.current.perform(inModes: [.default]) { /* Perform update */ }
But in that case your batchUpdates will be async. In my unusual case its not the best solution

@danya61 I don't use Dwifft anymore. I switched to DifferenceKit

@danya61 I don't use Dwifft anymore. I switched to DifferenceKit

Well, I suppose that the issue doesn't associated with the especial library, I've reproduce it even without diff framework

@danya61 hmmmm. I don't remember that I've faced with the current issue. But this not 100%

I had the same issue, and resolved with setting collectionView.contentInsetAdjustmentBehavior = .never

For UICollectionView you might use StableCollectionViewLayout.
This issue is normal behaviour for UITableView/UICollectionView.


python311Packages.pontos: 23.7.4 -> 23.7.5

Diff: https://github.com/greenbone/pontos/compare/refs/tags/v23.7.4...v23.7.5
Changelog: https://github.com/greenbone/pontos/releases/tag/v23.7.5
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 242226 run on x86_64-linux 1

14 packages built:

faraday-agent-dispatcher
faraday-agent-dispatcher.dist
gvm-tools (python310Packages.gvm-tools)
gvm-tools.dist (python310Packages.gvm-tools.dist)
python310Packages.pontos
python310Packages.pontos.dist
python310Packages.python-gvm
python310Packages.python-gvm.dist
python311Packages.gvm-tools
python311Packages.gvm-tools.dist
python311Packages.pontos
python311Packages.pontos.dist
python311Packages.python-gvm
python311Packages.python-gvm.dist


kubectl-explore: init at 0.7.1

Description of changes

https://github.com/keisku/kubectl-explore - Plugin for kubectl
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

When I tried to run nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD" from the "Things done", I got the error during "running tests" phase:

I was getting the same error on my machine (when I was testing build with nix-build -E 'with import <nixpkgs> { }; callPackage ./default.nix { }'). But I fixed that by adding my user to trusted-users in /etc/nix/nix.conf (through nix.settings.trusted-users in /etc/nixos/configuration.nix)
Should I be worried about the error?

I just noticed that running the tests requires the network to be enabled.
Since tests are in a sandbox, they can't access the network.
Therefore, I would suggest to disable them by adding:
  doCheck = false;

Just after the vendorHash attribute.

Okay, thanks for the suggestion - I've just pushed updated code with doCheck = false;

hey, I just noticed this commit randomly. Adding a small comment on why the tests are disabled is helpful for the next person that wonders why this is.
No need to make a separate PR for this. Just in case you think about it in your next package update (no emergency or rush here).
Thanks to both for your contribution!

See my comment just before yours , I added the justification there.

Sure, having it in the PR is nice. I was talking about leaving a comment in the code directly to make it easier to understand at a glance without having to go through the commit history.

Oh right, fair enough, will do it next time!


Update to a newer version of Golang

This is on Go 1.20, which is fine for now, but based on their release history, Go 1.23 should be out sometime this month and might as well jump ahead to that if we can. Should be as simple as updating the go mod and dockerfile(s), but may lose support for golint and all that stuff. Don't have to do this right away but making a note of it so I can remember to do it later.

Just checked and 1.23.3 is currently available, so probably want to upgrade before the end of the year. Will look into it when I get time


Property 'includes' does not exist

Hi,
I'm trying to use ng2-breadcrumb, but I got this error in the console :
ng2-breadcrumb\app\components\breadcrumbService.ts:90:29
Property 'includes' does not exist on type 'string[]'
Any idea to solve this ?
Thanks in advance.
EDIT : I'm working with angular-cli. In my tsconfig.json file, in the compilerOptions, there was :
"lib": ["es7", "dom"].
But I need to put es6 instead of es7, else my project is not working any more. With es7, no more errors for your lib, but with es6 I got the error reported below.
Can you fix this ?

If I google your issue I can only find a link to the inverse of your problem (ES7 failing): https://github.com/Microsoft/TypeScript/issues/2340. I personally compile to es5, and that has no issues either..

I also compile to es5...
If it can help, my tsconfig.json is here :
"compilerOptions": {
"declaration": false,
"emitDecoratorMetadata": true,
"experimentalDecorators": true,
"lib": ["es6", "dom"],
"mapRoot": "./",
"module": "es6",
"moduleResolution": "node",
"outDir": "../dist/out-tsc",
"sourceMap": true,
"target": "es5",
"typeRoots": [
"../node_modules/@types"
]
}
}

Dude, this isnt an issue I can debug. You could try using the below config, its whats working on my project:
{   "compilerOptions": {     "target": "es5",     "module": "commonjs",     "declaration": false,     "removeComments": true,     "noLib": false,     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "lib": ["es6", "es2015", "dom"],     "sourceMap": true,     "pretty": true,     "allowUnreachableCode": false,     "allowUnusedLabels": false,     "noImplicitAny": false,     "noImplicitReturns": true,     "noImplicitUseStrict": false,     "noFallthroughCasesInSwitch": true,     "typeRoots": [       "./node_modules/@types",       "./node_modules"     ],     "types": [       "node"     ]   },   "exclude": [     "node_modules",     "dist",     "src"   ],   "compileOnSave": false }

still didn't work.


[Linux] Flatpak

Support deployment for flatpak.

This one's a tricky one

Is there any update on this issue, on cross distribution packaging the most promising packaging solution is flatpak. Supporting flatpak would be great.

Use flatpak/flatpak-github-actions/flatpak-builder in GitHub Actions to build Flatpak packages.
https://github.com/picguard/picguard/blob/main/.github/workflows/test-linux-flatpak.yml
https://github.com/picguard/picguard/blob/main/.github/workflows/test-linux-flatpak-arm64.yml


Sections

What's missing:

Access section by name, so I can do sections | find_section(name="posts") for example. It can't be sections.posts as some section name might not be valid tera identifiers. Can be used with the new {% set .. %} tera feature. A end-user can iterate on sections instead if necessary instead but that feels a bit clunky

Warnings for ignored pages: those are pages that can't be sorted in their section. The build and serve command should display the name/path of the problematic file.

Display the number of orphan pages when building/serving the site in the -> Creating 11 pages and 5 sections line

First point can be made to work by allowing indexing in the Tera parser or adding a generic get filter


feat: support external ca from cloud-integrator

Issue
This PR partially is in tandem with the PR https://github.com/canonical/grafana-cloud-integrator/pull/21 and should be merged after.
It also fixes (partially) https://github.com/canonical/grafana-cloud-integrator/issues/18 (I'm also opening a PR in the machine charm).
⚠ Do NOT merge before the other one, it contains the (yet) unreleased version of the cloud_config_requirer library.
Solution
grafana-cloud-integrator#21 (the PR for the CA issue) is addressed by saving the CA coming from relation data to file, and then calling update-ca-certificates as usual. We do this on the custom cloud-config-available event, which is emitted whenever there is a relation joined/changed/broken involving the cloud-integrator relation.
grafana-cloud-integrator#18 (the config issue) is solved by populating the _loki_config not only if there are endpoints coming from self._loki_consumer, but also considering the ones coming from the cloud integrator (self._cloud.loki_ready). The endpoints themselves are already included in self._loki_endpoints_with_tls().
Testing Instructions
Remember to use the Loki FQDN, not the IP, or TLS won't work.

Deploy the necessary charms:

# all the juju deploy commands are from edge and with --trust
juju deploy (the grafana-agent-k8s charm from this branch) agent
juju deploy (the grafana-cloud-integrator from the linked PR) cloud-integrator
juju deploy self-signed-certificates ca
juju deploy flog-k8s flog
juju deploy loki-k8s loki

Relate the charms:

juju relate agent:logging-provider flog:log-proxy
juju relate agent cloud-integrator
juju relate loki ca

juju config cloud-integrator loki-url="https://<loki-fqdn>:3100/loki/api/v1/push"
Make sure there is no agent series in Loki (no CA is configured in cloud-integrator yet)

curl -k https://<loki-fqdn>:3100/loki/api/v1/series | jq | grep flog

Extract the CA from Loki and set it in cloud integrator:

juju scp --container=loki loki/0:/usr/local/share/ca-certificates/cos-ca.crt cos-ca.crt
juju config cloud-integrator<EMAIL_ADDRESS>

Observe the configuration contains the Loki endpoint, the CA has reached grafana-agent and that the series is now in Loki:

# agent config contains the endpoint
juju ssh --container=agent agent/0 cat /etc/grafana-agent.yaml
# agent contains the certificate from cloud-integrator
juju ssh --container=agent agent/0 cat /usr/local/share/ca-certificates/cloud-integrator.crt
# series is now in loki
curl -k https://<loki-fqdn>:3100/loki/api/v1/series | jq | grep flog

I have tested this manually and everything works fine :)


Support custom SSH ports in URL of SFTP-repo

Allow supplying an override for the default port (or the one configured in SSH config) similar to the way the user override is possible. When needing to specify a custom port,  the sftp.//... format must be used.

This is a great PR, thanks a lot for your work!
For all other options for the SSH connection to the sftp server, the ssh config file should be used. Alternatively, I'll add free-form options for the backend soon^TM (in the form restic -o sftp.port=1234)


The 'history' used in intro should be hashHistory instead of browserHistory to avoid 404s

Currently the intro doc uses {browserHistory}.
This means that unless someone has configured their server explicity, when they hit refresh on a url in the browser they will get a 404.
Having just run into this myself, I cant tell you its a terrible first experience.
Please use hashHistory in the intro so it works for everyone when they hit refresh in the browser

I agree, probably just an oversight when updating the docs. Go ahead and click "edit" in github and I'll merge the change.
Thanks.


Update deno dependencies

SSIA
checks
$ deno lint ./**/*.ts
Checked 1 file
$ deno test ./**/*.ts
Check<EMAIL_ADDRESS>running 4 tests from<EMAIL_ADDRESS>patchDduOptions ... ok (0ms)
mergeDduOptions ... ok (0ms)
sourceArgs ... ok (0ms)
isParentPath ... ok (0ms)

ok | 4 passed | 0 failed (2ms)

Thanks!


Cleanup Cargo registry data folders

Description
This PR add cleanup of Cargo cached registry data folders Index and Src in Rust provisioners for Windows and Linux images.
Related issue:
Old cargo cache left over from the build in Windows and Linux #895
Check list

[x] Related issue / work item is attached
[ ] Changes are tested and related VM images are successfully generated

Please be aware that removing a pre-populated ~/.cargo/registry/index may make cargo jobs run slower due to the need to clone the index repository's master branch from scratch. However, when the baked-in index checkout gets too old, it may actually run slower on Windows than a clean clone. The maintainers of crates.io-index also periodically reroot the master branch on a squashed commit, so a single-branch clone or fetch performed by cargo should not get exceedingly large.
But again, the right answer to speeding up cargo index sync is caching.

This workflow run simulates the effects of the cleanup on cargo build speeds, against a caching scheme that has all dependencies for the build restored prior to running cargo. The crate's Cargo.lock lists 71 dependencies, a medium-sized amount. There is surprisingly little difference in build speeds. Furthermore, on Windows the cache-assisted build actually takes longer. This is probably a bug in cargo.

There is surprisingly little difference in build speeds. Furthermore, on Windows the cache-assisted build actually takes longer.

Solved this by adding the --offline flag to build commands.

I've got a test case that can be used to verify the fix.

Here's a workflow to verify the fix with. It's poorly suitable for a regression test though, as it spends a lot of time on cargo syncing the index over the network.


#2419. Try parse SemVerInfo from .nupkg file name for localFolder nuget source

fixes #2419

wow this looks very good!

travis failures seem unrelated.
I guess the timeouts are because NuGet forbi to use their fastest API. ;-)

For the travis failure we will soon have https://github.com/fsprojects/Paket/pull/2423
Interestingly it seems that NuGet is not to blame but mono (who would have thought from the error message)...
For now we can just hammer the repeat build button until we get a fixed mono version :/


Card: Spiritual Society, Property Is Theft

Needs ruling on Spiritual Society.

Answer was something like:

homes should do nice things. I'd say just have the card

So I will update to have the additional card draw on sundown even if there was no card drawn.


Fails to predict large fasta file ie Q8WZ42

Jackhammer fails to predict a huge fasta file 34kb of uniprotId Q8WZ42 , https://www.uniprot.org/uniprot/Q8WZ42.fasta , using reduced-dbs preset , throws error
I0914 18:27:30.618590<PHONE_NUMBER>85728 run_docker.py:194] I0915 01:27:30.618098<PHONE_NUMBER>39232 run_alphafold.py:268] Have 3 models: ['model_1', 'model_2', 'model_3']
I0914 18:27:30.618706<PHONE_NUMBER>85728 run_docker.py:194] I0915 01:27:30.618213<PHONE_NUMBER>39232 run_alphafold.py:280] Using random seed<PHONE_NUMBER>945985241 for the data pipeline
I0914 18:27:30.619273<PHONE_NUMBER>85728 run_docker.py:194] I0915 01:27:30.618959<PHONE_NUMBER>39232 jackhmmer.py:130] Launching subprocess "/usr/bin/jackhmmer -o /dev/null -A /tmp/tmpend58zz0/output.sto --noali --F1 0.0005 --F2 5e-05 --F3 5e-07 --incE 0.0001 -E 0.0001 --cpu 8 -N 1 /mnt/fasta_path_0/Q8WZ42.fasta /mnt/uniref90_database_path/uniref90.fasta"
I0914 18:27:30.762149<PHONE_NUMBER>85728 run_docker.py:194] I0915 01:27:30.761583<PHONE_NUMBER>39232 utils.py:36] Started Jackhmmer (uniref90.fasta) query
I0914 18:27:44.662442<PHONE_NUMBER>85728 run_docker.py:194] I0915 01:27:44.630804<PHONE_NUMBER>39232 utils.py:40] Finished Jackhmmer (uniref90.fasta) query in 13.868 seconds
I0914 18:27:44.665832<PHONE_NUMBER>85728 run_docker.py:194] Traceback (most recent call last):
I0914 18:27:44.665909<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/run_alphafold.py", line 310, in <module>
I0914 18:27:44.665965<PHONE_NUMBER>85728 run_docker.py:194] app.run(main)
I0914 18:27:44.666018<PHONE_NUMBER>85728 run_docker.py:194] File "/opt/conda/lib/python3.7/site-packages/absl/app.py", line 312, in run
I0914 18:27:44.666068<PHONE_NUMBER>85728 run_docker.py:194] _run_main(main, args)
I0914 18:27:44.666118<PHONE_NUMBER>85728 run_docker.py:194] File "/opt/conda/lib/python3.7/site-packages/absl/app.py", line 258, in _run_main
I0914 18:27:44.666167<PHONE_NUMBER>85728 run_docker.py:194] sys.exit(main(argv))
I0914 18:27:44.666216<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/run_alphafold.py", line 292, in main
I0914 18:27:44.666265<PHONE_NUMBER>85728 run_docker.py:194] random_seed=random_seed)
I0914 18:27:44.666313<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/run_alphafold.py", line 129, in predict_structure
I0914 18:27:44.666361<PHONE_NUMBER>85728 run_docker.py:194] msa_output_dir=msa_output_dir)
I0914 18:27:44.666410<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/alphafold/data/pipeline.py", line 134, in process
I0914 18:27:44.666459<PHONE_NUMBER>85728 run_docker.py:194] input_fasta_path)[0]
I0914 18:27:44.666507<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/alphafold/data/tools/jackhmmer.py", line 163, in query
I0914 18:27:44.666556<PHONE_NUMBER>85728 run_docker.py:194] return [self._query_chunk(input_fasta_path, self.database_path)]
I0914 18:27:44.666604<PHONE_NUMBER>85728 run_docker.py:194] File "/app/alphafold/alphafold/data/tools/jackhmmer.py", line 140, in _query_chunk
I0914 18:27:44.666653<PHONE_NUMBER>85728 run_docker.py:194] 'Jackhmmer failed\nstderr:\n%s\n' % stderr)
I0914 18:27:44.666702<PHONE_NUMBER>85728 run_docker.py:194] RuntimeError: Jackhmmer failed
I0914 18:27:44.666752<PHONE_NUMBER>85728 run_docker.py:194] stderr:
I0914 18:27:44.666800<PHONE_NUMBER>85728 run_docker.py:194] b''
I0914 18:27:44.666849<PHONE_NUMBER>85728 run_docker.py:194]

Hi, unfortunately AlphaFold can't handle sequences that are so long, we have validated AlphaFold for sequences up to 2,700 residues.
However, since this is a human protein, we folded all of its 1,400-residue fragments (with 200-residue overlaps) and it is available for download from the AlphaFold Protein Structure Database here: https://alphafold.com/download -> https://ftp.ebi.ac.uk/pub/databases/alphafold/UP000005640_9606_HUMAN.tar
You can then align and combine the predictions into a single structure using e.g. ChimeraX, see the tutorial here: https://rbvi.github.io/chimerax-recipes/big_alphafold/bigalpha.html
Note also that this has already been done for titin: https://mobile.twitter.com/UCSFChimeraX/status/1435760774824169474


Upgrade JGit to 6.3.0 (needs Java 11).

See https://projects.eclipse.org/projects/technology.jgit/releases/6.3.0/.
Upgrading to Java 11 might introduce issues (see https://github.com/melix/includegit-gradle-plugin/issues/13#issuecomment-1321872779), but IMO it is counterbalanced with all fixes and new features that JGit 6.3.0 is coming with.

Sorry I realized a bit too late about this PR. Done in #32.


feat: add access list transaction type, add transaction api txn_hash, fix block transaction query

What I did

correctly instantiate transactions coming from BlockTransactionQuery, previously they were missing data and signature
add txn_hash property to TransactionAPI
add AccessListTransaction type and optional access list for DynamicFeeTransaction

How I did it
How to verify it
chain.blocks[height].transactions should yield transactions with correct calculated txn_hash
Checklist

[ ] All changes are completed
[ ] New test cases have been added
[ ] Documentation has been updated

also caught an interesting off-by-one problem
import rich
from hexbytes import HexBytes

def test_wtf(networks, chain):
    with networks.ethereum.mainnet.use_provider("geth"):
        for height in range(chain.blocks.height, chain.blocks.height - 100, -1):
            print(height)
            web3_transactions = chain.provider.web3.eth.get_block(height, True).transactions
            ape_transactions = chain.blocks[height].transactions
            for a, b in zip(web3_transactions, ape_transactions):
                rich.print(f"[red]{a.hash.hex()}")
                rich.print(f"web3={a}")
                rich.print(f"ape={b}")
                assert HexBytes(a.input) == HexBytes(b.data), "data mismatch"
                rich.print(f"[red]{b.txn_hash.hex()}")
                assert a.hash == b.txn_hash, "hash mismatch"

NOTE: Breaking change since txn_hash is abstract
I think we should do 0.3.0 next anyway?


Build clean ups

Actually, can you explain why assets can be removed?

I don't know what this function is still used for since we compile assets into the css. It was unused so I removed it.


When iOS lock button is pressed and app is in foreground it does not remain active in background (RN 0.31)

Issue Description
Playing with silent push notifications and trying to get them to work when iOS is locked, I discovered that when a RN app is in foreground state and the lock button is pressed the app goes to inactive state and does not end up in background state.
Steps to Reproduce / Code Snippets
Create a RN app (v0.31)
Log changes to new RN app states
Modify RCTAppState.m startObserving method to log which notification triggered the method.
situation A:

launch app
press home button (app goes to background)
press lock button (phone locks)
notice that the state switched active->inactive->background

situation B:

launch app
press lock button (phone locks)
notice that state is switched from active->inactive ...but not to background! In native code 'UIApplicationDidEnterBackgroundNotification' does get triggered but this does not put RN in background state.

Expected Results
App should be in background state when a user presses the lock button.
Additional Information

React Native version: 0.31.0
Platform(s) (iOS, Android, or both?): iOS

It looks like this is fixed now as per https://github.com/facebook/react-native/issues/524

When I test it with RN 0.41.0-rc.0 I still get the same behaviour: when the lock button is pressed on ios the status goes to inactive, when you push the home button the status goes to inactive then ends up in background. (So when the lock button is pressed the javascript code does not run, but when the app was put to the background first and then the lock button is pressed the javascript code gets triggered correctly)

When I test it with RN 0.41.0-rc.0 I still get the same behaviour: when the lock button is pressed on ios the status goes to inactive, when you push the home button the status goes to inactive then ends up in background. (So when the lock button is pressed the javascript code does not run, but when the app was put to the background first and then the lock button is pressed the javascript code gets triggered correctly)

I have the same problem. @fast4ever , did you fix the problem?


add font selector

FYI: @fmiccolis I've extracted some code for the font selection in the admin gui and made it merge ready
https://github.com/PhotoboothProject/photobooth/pull/762

will push this in as soon as the PR gets green and rebase #762


Add mapping to toggle show/hide of excluded files

Add mapping to toggle show/hide of excluded files

I was able to add a key map to a function and saved the current exclude list to a temp variable and set settings.exclude to nil, but couldn't figure out how to trigger a full refresh.

Hello @ironsigma, I've also just implemented this feature. Hidden files and directories are now treated as regular entries except they just won't be rendered. See :h carbon-setting-actions-toggle-hidden.
Similar to #131 I'm going to close this but feel free to reopen if I missed something.

Very nice, than you.

This was a good suggestion, regular file explorers also have the ability to toggle such behavior :)


Allow configuring OpenID Connect Discovery endpoint path

Expected Behavior
The path of the OpenID Connect Discovery endpoint can be configured. (e.g. in ProviderSettings like done for jwkSetEndpoint)
Current Behavior
The path is hard-coded to /.well-known/openid-configuration
Context
We want to provide the endpoints of the spring-authorization-server library within a servlet which listens to a specific path. The old deprecated spring library currently runs with another path. Both of this versions need a separate path for the Discovery endpoint. This means for us the Discovery Endpoints should be available in e.g. /oidc/v2/.well-known/openid-configuration.
Right now this path seams to be hard-coded and cannot be replaced. (OidcProviderConfigurationEndpointFilter, OidcConfigurer)

Closing as duplicate of gh-512. See comment


asOptional should only annotate a predicate instead (breaking)

Related to #132.

https://github.com/denoland/deno/issues/11889#issuecomment-2360821816
it seems the option (exactOptionalPropertyTypes) is not enabled by default


fix: fix name resolution for duplicate models

Codecov Report

Merging #39 (4738b32) into main (ea608b3) will decrease coverage by 0.17%.
The diff coverage is 93.24%.

@@            Coverage Diff             @@
##             main      #39      +/-   ##
==========================================
- Coverage   98.13%   97.95%   -0.18%
==========================================
  Files         175      179       +4
  Lines        5532     5636     +104
  Branches      639      663      +24
==========================================
+ Hits         5429     5521      +92
- Misses         56       63       +7
- Partials       47       52       +5

Impacted Files
Coverage Δ

tests/__init__.py
100.00% <ø> (ø)

...t_docs/tutorial/cookie_params/test_tutorial_001.py
100.00% <ø> (ø)

...t_docs/tutorial/header_params/test_tutorial_001.py
100.00% <ø> (ø)

...st_docs/tutorial/query_params/test_tutorial_001.py
100.00% <ø> (ø)

...st_docs/tutorial/query_params/test_tutorial_003.py
100.00% <ø> (ø)

...st_docs/tutorial/query_params/test_tutorial_004.py
100.00% <ø> (ø)

tests/test_openapi/params/test_cookie_openapi.py
100.00% <ø> (ø)

tests/test_openapi/params/test_header_openapi.py
100.00% <ø> (ø)

tests/test_openapi/params/test_query_openapi.py
100.00% <ø> (ø)

tests/test_security/test_security_oauth2.py
100.00% <ø> (ø)

... and 28 more

Note: this is actually a 35 LOC drop on the implementation side, the increase is all from tests


Hero background does not display in outlook 2016

Describe the bug
The hero background does not display in outlook2016
To Reproduce
Steps to reproduce the behavior:
using
<mj-hero background-color="#f1f2f5" background-url="https://www.abcdefg.com/img-email/banner-help.png" padding="20px" background-height="220px">
Expected behavior
Expect the hero image to be displayed in outlook 2016. It isn't.
MJML environment (please complete the following information):
Using version 4.3.0 in the try it live editor.
Email sending environment(for rendering issues):

piping html through a java based email service. Custom code.

Affected email clients (for rendering issues):
Outlook 2016
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Can't reopen item #1526, however this is the exactly the same behaviour called out there.

Hi,
Can you try with putsmail.com and send us the result ?

sorry, i changed the domain deliberately ..
Screen shots attached - sent through putsmail.com
this is in gmail:

this is in outlook:

note the second banner at the bottom is a background on a section, as opposed to a hero tag.

You can just drag the background here so we can try to reproduce & see what's going on

there you go mate. hope that helps.

Adding a background-width to 600px on the mj-hero look to solve the issue, @kmcb777 the doc state that the background-width should take the size of parent mj-body, can we patch this for the 4.4 release ?

actually it looks like you need to have BOTH background-width and background-height set to resolve the issue.

@iRyusa background-width will default to container width now
background-height is still mandatory because we can't guess the ratio otherwise
it's ok for fixed-height but fluid-height doesn't seem possible on outlook with this vml method, the method used by mj-section seems the only way, so maybe we should drop fluid-height mode, since mj-section does the same ?

hey, i found a little hack to solve this issue. In the vml tags, you have to play with these attributes:
origin="0.5, 0" position="0.5, 0"
after some tests, its gonna work with (depends also on image-size):
origin="1, 0" position="1, 1"
Additionally, you also have to remove the background-color #fffff on the next couple div tags.

Facing similar issue when hero is in wrapper with background color – the background color overlays the background image.


Wrong location of the SQL scripts for the jdbcjobstore in the documentation

The documentation at https://www.quartz-scheduler.org/documentation/quartz-2.3.0/tutorials/tutorial-lesson-09.html says

You can find table-creation SQL scripts in the “docs/dbTables” directory of the Quartz distribution.

But this directory does not exists in any distribution of Quartz. The correct location can be found in GitHub:
https://github.com/quartz-scheduler/quartz/wiki/How-to-Setup-Databases

The SQL files are located in quartz-core/src/main/resources/org/quartz/impl/jdbcjobstore directory.

This location is the correct one and works.
As with the other broken links in #1123 and #938, this issue exists since 2020.

Fixed, thanks.


pdfview not working in safari

Link to PDF file (or attach file here):

Configuration:

safari 5.1.7():
https://mozilla.github.io/pdf.js/web/viewer.html   - pdefview not working in safari

The console shows the following error: "cannot declare a parameter href in strict mode".
Looks like a bug in Safari.

what i have to  do to solve this isssue

I submitted a patch at #8544. I confirmed that the viewer works again in Safari 5.1 with that patch.


gpio: gpio_cmsdk_ahb: Return error for not supported flag

The code had GPIO_INT_DOUBLE_EDGE siliently ignored, instead lets run
-ENOTSUP so caller knows its not going to work.
Fixes: #12764
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

Merging #14149 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #14149   +/-   ##
=======================================
  Coverage   52.68%   52.68%
=======================================
  Files         307      307
  Lines       45473    45473
  Branches    10530    10530
=======================================
  Hits        23956    23956
  Misses      16640    16640
  Partials     4877     4877

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 8624797...d9bccfb. Read the comment docs.


Hide command line instructions when merge form is visible

Hides element introduced by https://github.com/go-gitea/gitea/pull/13840 when merge form is visible.
Before:

After:

This is done to avoid anchor link right below the button which looks bad and can cause mis-click/tap. There is no easy way to make the element flow inline when form is visible because the neighboring element must also be inline, which form is not (and can not be).

This is a draft because .db applies !important which screws with dynamic hiding of element.
@silverwind any proposed solutions? I imagine this will crop up more and more as these helpers are being used.

Codecov Report

Merging #13956 (1e6a46b) into master (3285bab) will decrease coverage by 0.10%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13956      +/-   ##
==========================================
- Coverage   42.21%   42.11%   -0.11%
==========================================
  Files         710      710
  Lines       77226    77226
==========================================
- Hits        32600    32522      -78
- Misses      39249    39354     +105
+ Partials     5377     5350      -27

Impacted Files
Coverage Δ

models/issue_comment.go
43.50% <0.00%> (-8.77%)
:arrow_down:

modules/notification/ui/ui.go
84.72% <0.00%> (-8.34%)
:arrow_down:

modules/notification/mail/mail.go
33.33% <0.00%> (-5.75%)
:arrow_down:

modules/git/commit.go
49.67% <0.00%> (-3.95%)
:arrow_down:

modules/notification/base/null.go
74.28% <0.00%> (-2.86%)
:arrow_down:

modules/notification/notification.go
83.92% <0.00%> (-2.68%)
:arrow_down:

services/mailer/mail.go
60.21% <0.00%> (-1.08%)
:arrow_down:

services/pull/pull.go
40.68% <0.00%> (-0.50%)
:arrow_down:

modules/charset/charset.go
73.03% <0.00%> (+4.49%)
:arrow_up:

modules/queue/unique_queue_channel.go
64.51% <0.00%> (+6.45%)
:arrow_up:

... and 1 more

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 3285bab...1e6a46b. Read the comment docs.

This is a draft because .dib applies !important which screws with dynamic hiding of element.

How so? Is this on Firefox? - as it appears to work fine on Chrome.
Would it work better if the element was wrapped in a div that was hidden instead ?

Chromium, the .hide() applies style="display: none;" to element but display: inline-block !important; overrides it hence it will always be shown.

Would it work better if the element was wrapped in a div that was hidden instead ?

Won't that screw up with inline styling, which is the whole point of adding dib?

ci failed is unrelated and I have restarted it.


Add appVersion key for etcd

The key "appVersion" is needed for ci testing, it's missing in this yaml. That sometimes will cause testing failure.

/assign @viglesiasce

/ok-to-test
/lgtm


DRY the Admin UI middleware a bit

There are separate routines for prod and dev middleware.  The prod
mode path had some bugs in redirect handling.
I've sucked some of the code out of the separate paths so more prod code
is covered by tests.  The redirects work now.

Looks like some integration tests are no longer able to run. Can you see what went wrong there?


Run validation against running, not intended

Right now we are running the validation against intended. We should however also take running into consideration.
Probably we can thereby fully neglect intended. Needs to be discussed.

This was implemented in #167.


Replacing ssh agent with gpg-agent

Is there any way to replace ssh-agent with gpg-agent? I'd like to do this to get my smart card working properly for OpenSSH.

It should be possible in theory, but I don't think anyone has done this yet. You would need to create a plugin that handles the gpg-agent calls (you could use the SSH Plugin as a starting point). And then you would need to modify your Pshazz theme to use the new plugin.
Sorry I haven't had any experience with gpg-agent, but I'd love to get a Pull Request if you can work out how to do it.


Unable to suppress Upgrade Prompt

Hi,
Describe the problem you're observing:
To disable the upgrade prompt, I placed the file "org.jkiss.dbeaver.core.prefs" in specified location
[ %HOME%.dbeaver4.metadata.plugins\org.eclipse.core.runtime.settings] after installation. But after opening the application, I'm still getting the update prompt.
Can you please tell why I'm receiving the upgrade prompt even after placing that file?

That's the wrong location for the dbeaver workspace. Use:
%userprofile%\AppData\Roaming\DBeaverData\workspace6.metadata.plugins\org.eclipse.core.runtime.settings\org.jkiss.dbeaver.core.prefs

Hi @andrelinslima,
I was following the same path for Dbeaver community that was mentioned in https://github.com/dbeaver/dbeaver/issues/3299, %HOMEPATH%.dbeaver4\.metadata\.plugins\org.eclipse.core.runtime\.settings\ and I tried using the path you mentioned, but I could still the update prompt.

Did you try to disable Automatic updates check in preferences?


bad argument #1 to 'ipairs' (table expected, got nil)

I am being inundated with this error:
"~/projects/nvim-candy/candy/lua/marshmallow/lsp.lua" 130L, 3649B written
Error detected while processing CursorMoved Autocommands for "*":
Error executing lua callback: ...art/vimplugin-supermaven/lua/supermaven-nvim/textual.lua:22: bad argument #1 to 'ipairs' (table e
xpected, got nil)
stack traceback:
        [C]: in function 'ipairs'
        ...art/vimplugin-supermaven/lua/supermaven-nvim/textual.lua:22: in function 'is_all_dust'
        ...art/vimplugin-supermaven/lua/supermaven-nvim/textual.lua:76: in function 'can_delete'
        ...art/vimplugin-supermaven/lua/supermaven-nvim/textual.lua:83: in function 'check_state'
        ...supermaven/lua/supermaven-nvim/binary/binary_handler.lua:316: in function 'poll_once'
        ...supermaven/lua/supermaven-nvim/binary/binary_handler.lua:287: in function 'provide_inline_completion_items'
        ...supermaven/lua/supermaven-nvim/binary/binary_handler.lua:109: in function 'on_update'
        ...gin-supermaven/lua/supermaven-nvim/document_listener.lua:52: in function <...gin-supermaven/lua/supermaven-nvim/documen
t_listener.lua:46>

:version

NVIM v0.11.0-nightly+738a84d
Build type: Release
LuaJIT 2.1.1713773202
Run ":verbose version" for more info

With the latest change this issue should no longer be possible, though I'm not sure why it was occurring in your case to begin with. Please let me know if something similar occurs in this newest version.

Seems to be fixed


Swagger refactor

Summary
Refactor after #1510 to include all the swagger references into the v1 folder. Most of the changes are related with the files end-point because we didn't have them initially in that folder.

This PR is important to continue with #1509


Always use spaces inside curly brackets

Nearly every project I've been on uses spaces inside curly braces
regardless of context.
Before:
foos.each {|foo| foo.bar }
{foo: "bar", baz: "what comes after baz?"}

After:
foos.each { |foo| foo.bar }
{ foo: "bar", baz: "what comes after baz?" }

For blocks, we use a space after do, so why wouldn't we
use a space after an opening bracket instead?
For hashes, I don't know why, but I personally find the spaces
comforting. Curly brackets demand personal space. I think it's
because there's a lot more syntax happening inside a hash literal
than you'll usually see  inside arrays and method arguments, so
the extra spaces give each key/value entry a bit more space to
clearly stand out.

I'm personally against spaces inside hashes.
I don't like them in blocks either, but care less about that one.

I strangely prefer the "before" each, and the "after" hash:
foos.each {|foo| foo.bar }
{ foo: "bar", baz: "what comes after baz?" }


feat: Add support for message type generation from Json schemas

What this PR changes/adds
This PR generates HTML tables containing relevant type information (e.g., required and optional properties) for all DSP messages. The HTML tables are generated by a custom Gradle Plugin that parses the DSP schemas at build-time. The generated tables may then be directly included in ReSpec markdown.
The PR introduces the plugin and Json Schema Object Model parser thatis used to generate the tables. Subsequent PRs will include the generated tables in the specification text and remove the relevant PNG images and manually-created type information.
Support for $comment or description still needs to be added. That can be done in another PR.
The plugin could be enhanced to provide hyperlinks for referenced type information. For example, a MessageOffer references the Offer type. The plugin could include a hyperlink directly to the Offer definition. We need to decide on the conventions for doing this.
Why it does that
Enhances spec consistency and reduces the chance for errors.
Further notes
List other areas of the documents that have changed but are not necessarily linked to the main feature. This could be editorial changes or mistakes in example files that were encountered and were fixed inline, etc.
Linked Issue(s)
Closes #62

@arnoweiss

The basis is I think great.
What kind of features do we want to ensure before merging this? Like you already mentioned are the links to the other schemas/tables. But apart from that some of the things I see when generating the tables:

profile is in the table any but in the schema it is a string or string array. (e.g. for Offer)
Enums are now just strings, but shouldn't the enum values themselves be mentioned in the table.
timestamp contains a format which is relevant. But might be difficult to present in a table, this can be mitigated by descriptions/comments.

I would suggest to make sure the tables are a bit more elaborate before including them in the spec, which can be done in separate PRs.

The basis is I think great.
What kind of features do we want to ensure before merging this? Like you already mentioned are the links to the other schemas/tables. But apart from that some of the things I see when generating the tables:

profile is in the table any but in the schema it is a string or string array. (e.g. for Offer)
Enums are now just strings, but shouldn't the enum values themselves be mentioned in the table.
timestamp contains a format which is relevant. But might be difficult to present in a table, this can be mitigated by descriptions/comments.

I would suggest to make sure the tables are a bit more elaborate before including them in the spec, which can be done in separate PRs.

Yes I agree. @arnoweiss and I talked about this and came to the same conclusion. How about this approach:

Merge the PR as a basis but do not include it in the specification text yet
File issues (Arno and I have identified some, as well as the ones you outlined) and fix them
Arno has a suggestion for improving the table format that he will file as a separate issue which we can implement
Do a thorough review for correctness before including the generated tables in the specification text
Include the tables and remove the PNGs

Let me know what you think.

Sound like a good approach

Filed https://github.com/eclipse-dataspace-protocol-base/DataspaceProtocol/issues/80, https://github.com/eclipse-dataspace-protocol-base/DataspaceProtocol/issues/81 and #82


Add ability to do a point-in-polygon analysis with data from OpenSGID

Give users the ability to add data from polygons from OpenSGID to the output.

Laura Dahl from UDRC is very interested in this feature. She talked about being able to append political districts to students and also does a lot with small health areas. She asked that we notify her once this enhancement is complete.

More interest in such a feature from Mark Mitchell of Lt. Gov's Elections Office:

This is a very cool tool!  With it, I'm able to get a matched address and geocoding coordinates.  Is there anything like it that can return a voting precinct based on address or X/Y?


GH-488 Update InspectorDock when ProjectSettings change

Fixes #488

Applied to main (=2.1) and branch 2.0.


Add GitHub CI

Over the past weekend, I added support for SDL satallite libraries to libsdl-org/setup-sdl.
(it currently only supports SDL3)
This seems like a good opportunity to add ci to SDL_rtf.
https://github.com/madebr/SDL_rtf/actions/runs/7532633083

Looks like actions are not activated in this repo.

Maybe @slouken should activate it??

Done!


Move IFeatureCollection and friends to Microsoft.Extensions.Primitives

From @davidfowl on Saturday, May 20, 2017 12:46:19 AM
The IFeatureCollection isn't tied to HTTP and we'd like to use it for non http things. Ideally it would be part of the primitives package and Microsoft.AspNetCore.Http.Features would contain http specific features.
Copied from original issue: aspnet/HttpAbstractions#845

I'd love to see this done in the near future as I have a framework I'm working on that could really use it. Is there anyway I can help expedite this?

Massive breaking change. We might need to live with the legacy....

So this is not going to happen I presume? It's a shame though, super useful outside of HTTP as part of other host models and abstractions

@NinoFloris can you file an API request on https://github.com/dotnet/runtime? It'll help if it's coming from a real customer scenario.


Launchpad filters are counterintuitive

Students think they should click Scratch to see Scratch untutorials. Instead, when you click Scratch, Scratch untuts are filtered out.

I like how this works: https://www.education.com/games/second-grade/
Would love the ability to add more categories in addition to Levels, potentially.

the launchpad at that link works the same way as our site. At that link is a bubble called second grade and when you click it, second grade is removed from the filter. And when there are no filters it shows all. We work the same way. The difference is that, we load in with three filters already there (scratch, web, and woof) so when they click scratch, web and woof are still filtering. If we just load in with no filters (or one filter, but you'd have to pick which) it'll be exactly the same. I think you actually already figured this out, because you commented out woof and web in the most recent version. Would you like me to do set it to zero filters?
Regarding categories; we already have the backend for it. Untuts have categories. There just isn't any way to filter by them. We can do that too.

Oh yeah, it does work the same. I didn't even notice the bubble on top. What I noticed was the side menu which made it clear to me what was selected.
In our site, instead of the "select" menu we have now could we make them links that show/hide? I will also add the "x" to the bubbles so it's clear clicking means closing.
Yeah, I made the filter just Scratch for now. Probably the way to go.
Yes, definitely want to filter for levels and categories. Will "Add Category" be added to New Untutorial?


Remove VirtualMachineFlavor and VirtualMachinePreference SharedInformers

What this PR does / why we need it:
The use of SharedInformers to watch for and cache both flavors and preferences has proven problematic. There have been examples in CI of virt-api and virt-controller workers both encountering issues either seeing the latest generation of a resource or even seeing the resource at all during the initial creation of a VirtualMachine.
For the time being we are going to revert back to client calls and might revisit the use of SharedInformers in the future.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cc @rmohr @davidvossel @akrejcir

/retest

/retest

/hold

/unhold

/retest

/retest


Django 2.0 breaks field.help_text containing lists

TypeError (not all arguments converted during string formatting) caused by field.help_text containing lists.
Fixed by converting field.help_text value to str after copying its value to the widget.
Will be pushed in next update along with fix for<EMAIL_ADDRESS>

Are you still planning on pushing this ?


Email Notifikation only works with Gmail

I like to use my own email provider (smtp.strato.de:465).
Same settings work with thunderbird but on bazaar i get an timeout.
an test google account works fine.
sidenote: eaven the openbazaar deamon is in tor mode, it cant use hidden smtp servers (name resolv issue)

Thanks for reporting. Wasn't aware of the tor issue. Afaik it does work with other email providers. Could it be a wrong port or some other setting?

At the moment i can just guess

close it becaues time passed and i dont had time to test it at the moment -


fix: use correct feature flags for CreateCommandOpts

Self-explanatory, these flags were missed during the latest transition.

I think the current code is correct, on 0.9 we should still use the old ordering of the struct fields. https://github.com/neovim/neovim/commit/efb0896f21e03f64e3a14e7c09994e81956f47b9 is the commit that changed them and it came after the 0.9 release.

Oh, I see. There may be something weird going on with my setup then. I'm on neovim/neovim@f1b415b which I thought was the latest 9.0 tag. But now that you mention it, yeah, it does say 0.10 in the version prompt.

Can I close this?

Yep, my bad.


[$250]  Group chat - Custom avatar reverts to default avatar after promoting member to Admin

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

Version Number: 1.4.63-7
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Issue reported by: Applause - Internal Team
Action Performed:

Go to staging.new.expensify.com
[User A] Go to FAB > Start chat
{User A] Create a group chat with User B
{User A] Send a message to chat
[User A] Upload a group chat avatar
[User B] Open the group chat
[User A] Promote User B to Admin
[User B] Wait for a while to receive the update

Expected Result:
Group avatar will not revert to default avatar after promoting User B to admin
Actual Result:
Group avatar reverts to default avatar after promoting User B to admin. However, it changes back to custom avatar after a while
Workaround:
Unknown
Platforms:

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

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/d590b410-a1e6-4df5-893e-82cc04cc2cb2
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01ee2daa5a2c3e8900
Upwork Job ID:<PHONE_NUMBER>184792576
Last Price Increase: 2024-04-26

We think that this bug might be related to #vip-vsp

Demoting as I don't think this is an actual blocker. Fairly minor. Also wasn't able to reproduce it.

@marcaaron unless you feel like this should be handled as part of group chats I'd say we open this open to contributors and see if anyone else can reproduce and figure out the issue

Yeah it could be - I'll add it to the GH Project I have set up to track these. Definitely, agree not a blocker.

Group avatar reverts to default avatar after promoting User B to admin. However, it changes back to custom avatar after a while

Seems like we could possibly be removing it in the optimistic data somewhere.

@blimpich lmk if you want me to take this one off your hands.

Will do, I'm good to be the CME for this I think 👍

WAiting for weekly retesting to confirm if we want to close

Issue not reproducible during KI retests. (First week)


NEXT: File Button / Dropzone

[!WARNING]
This issue is a work in progress.

This will act as a hub to centralize this information.
Maintainer Requests
The following requests are coming straight from the Skeleton team. These are highly likely be implemented:

Let's consider combining the docs for these into "File Selection"

Community Requests
The following requests have come from the community and are under consideration:

n/a

Bugs and Issues

n/a

Feedback
If you have additional updates or requests for this feature, please do so in the comments section below.

For some reason, the FileDropzone & FileButtons can't be reset via Javascript, reproduction can be found here: https://github.com/Sarenor/reset-reproduction
A vanilla input can be reset though. It's very confusing, since the forwarded input references seem to be identical (in the sense that they are both forwarded input references)


Adds vpc_id to ec2_group, from subnet facts, and allows to specify SG ip cidr

Added VPC ID to EC2 Group (otherwise, it creates an EC2-Classic Security Group)
Based on #2230
Added support for other IP CIDRs different than <IP_ADDRESS>/0

How to use it
VPC ID

Create role using EC2 driver: molecule init role -r test-ec2 -d ec2
Include vpc_subnet_id to your platform on molecule.yml

IP CIDR

Create role using EC2 driver: `molecule init role -r test-ec2 -d ec2
Either include allowed_ip_cidr to the platforms (to specify a CIDR) or auto_public_ip (to make it use your current public IP). If you include none of these, the default will be <IP_ADDRESS>/0.

It would be really awesome to do it. I'm willing to help :)

Thanks @ssbarnea

@vinicyusmacedo Don't be too happy about it, removal comes after but I wanted to avoid having you recreate the patch on the new repository. I am relying on your help testing the plugin once I initialize the new repository.

Also, come chat with us on IRC if you are not already there and we organise ourselves to get this done!

@decentral1se joined.


Add demo section to README

Fixes #87

Thanks a lot for that. I am planning to elaborate more on how to develop xterm.js, but this should do the work for now.


docker-from-docker: allow installation alongside docker-in-docker, as non-default docker configuration

#171 prevents me from using docker-from-docker as the primary docker configuration for my devcontainer (since I need to share docker ports with internal processes). I realized, however, that there was nothing preventing me from mounting both sockets and switching off between them. This allowed me to get what I really wanted out of docker-from-docker, which was the ability to update the devcontainer image on the host machine. This is useful for two reasons, both pertaining specifically to locally-running devcontainers:

I can use the devcontainer CLI to iterate on builds without shutting down the active dev container -- so if I'm making changes, I don't need to lose access to my project while I wait to see the there are any errors.
I can implement a "check for updates" process, where I use the host's docker daemon to check for the newest built version of the devcontainer image, download it, and then encourage the user to Rebuild Container at their earliest convenience if and only if the image has actually changed since the one they're currently using. My image is privately hosted and I can't access my private registry from the bootstrap container (because doing so would require access to the host's credential helpers), so this is the only way to ensure that users start a Rebuild with an up-to-date cache.

I'm accomplishing this right now by vendoring some pieces of the docker-from-docker feature into a custom feature, but it seems like it would make more sense as a toggle for the existing feature: let me use my devcontainer configuration to set the target socket to e.g. /var/run/docker-host-proxy.sock instead of /var/run/docker.sock. Then I can use DOCKER_HOST=/var/run/docker-host-proxy.sock from scripts inside my devcontainer to interact specifically with the host's Docker daemon.

What is preventing you using docker-in-docker as your primary means of using docker with dev containers? Do you have containers running on your local machine that you want to use inside the dev container? Could you do all of your work inside of the dev container (docker-in-docker style), or with docker-compose orchestrated by the dev container CLI, where everything would be on the same docker network and could be configured how you'd like it?
I may not totally understand the use case yet, but it seems reasonable to me to allow an override option to configure the path to socket in docker-from-docker

@joshspicer I am currently using docker-in-docker as my primary means of running Docker, yeah -- and it works great for almost everything. The reason I want docker-from-docker as well is that I want users to be able to pull the newest build of the devcontainer from the registry from inside the devcontainer while it's running, then do "Rebuild From Container" and have it Just Work™. Of course, if I do that while running docker-in-docker, the pulled build won't be present in the host's docker daemon, so the Rebuild will result in the entire container build running from scratch (which can take 10+ minutes).
To give more context: my CI builds/pushes my devcontainer on every commit for ARM and AMD both, using the devcontainer CLI (I use earlier commits as cache-from, so this is always a trivial operation if it hasn't changed). It then creates a manifest joining the ARM and AMD builds and pushes it under the current commit SHA and the branch name. The devcontainer.json specifies "cacheFrom": ["myPrivateRegistry/devcontainer:master"], so when a user opens the application in a devcontainer, it's built against the most recent master build -- which is nearly guaranteed to be up-to-date with whatever the actual devcontainer config is in the state of the repository at the time, unless they're actively hacking on the devcontainer config.
The problem is that this scheme assumes that the host machine has an up-to-date version of myPrivateRegistry/devcontainer:master. In theory, I could just do a docker pull myPrivateRegistry/devcontainer:master in my initializeCommand, but initializeCommand runs inside of a bootstrap container when you're doing Clone Repository In Container Volume -- and I have to instruct my users to do that because the filesystem performance when I have the whole repo in a bind mount isn't good enough yet. The bootstrap container doesn't have credentials for my private registry, so I can't pull there.
I do, however, ask my users to log into gcloud as part of the standard environment setup instructions, which allows them to access the private registry from their host. And I bind-mount the gcloud credentials into the container so that they're automatically logged in there as well. So inside the devcontainer, where I have a bunch of scripts and automation running already, I have the opportunity to pull the latest devcontainer image -- I can even check if the hash of the image has changed and give the developer a heads up that they should rebuild! But to do all that, I need docker-from-docker -- so I hacked up the thing I have now, with both sockets available, and on the rare occasion I want the host I call it out in my scripts.


New web services - data products

Produce web services showing summary of all data hold within the JCDP database, aggregated to C-Square scale.

Effort in km per C-square grid cell; showing two metrics 1. showing total effort in km in all sea states, and 2. where the sea state is <= 3
Number of sightings/observations; as total number for all species and per species; where the sea state is <= 3
Number of unique species recorded per grid (observed species richness); where the sea state is <= 3

@NikiClear  are these queries per year or in total?
For example, what filters do you want to specify when asking the query?
Year?
Species?

@NikiClear are these queries per year or in total?
For example, what filters do you want to specify when asking the query? Year? Species?

That's a good question.
Would it be possible to let the user select a date range (choose all data or between dates)? If not then perhaps to cover all bases, include all data and also per year?

Hi Niki!
It is possible for the users to select which year(s) they would like to retrieve data for, so we can do it that way.
Another question that appeared is regarding the unique species. There are some sightings that are not identified at a species level. There is an example in the excel file attached here (this is for one Csquare, only for year 2008).
How do you think we should approach this?
cetaceans_webservices.xlsx

Hi Maria and Carlos
I think we would need to ignore the records which aren't recorded at species level, as the others chould be the same species or different but we would never know. So for the example you send the species list would be

Balaenoptera physalus
Delphinus delphis
Globicephala melas
Tursiops truncatus
Hyperoodon ampullatus

or 5 unique species

An example of this webservice "Number of sightings/observations; as total number for all species and per species; where the sea state is <= 3" as a product:
https://cetaceans.ices.dk/API/GetTotalSightingsperCsquareAndSpeciesAndYear

Webservice of the data product: GetTotalSightingsperCsquareAndYear

Two of the requested product are on-line:
https://cetaceans.ices.dk/webservices

Let me know if there are any comments or questions.

@MariaMakri to test please

@NikiClear I had a look and it seems that it is working, except for a typo that we need to remove in a all of them ('number' is duplicated). However, could you also have a look and confirm that this is what we were going for? It seems that you can select a string of several years, but not necessarily a range of them like (2014-2018). Would that be an issue?


How to turn off backquote when column auto-complete

How to turn off backquote when column is auto-complete when using mariaDB
SELECT A.`column_1` FROM table

Same problem here using MySQL. Did you found out any solution to disable that?

For now, suggested columns aren't backquoted by default.
Seems the issue is obsolete

I am using the  <IP_ADDRESS>112130652 version, and backquotes are enabled by default, and I cannot disable it!


fix: Remove the blindness certification upload step

Update Automated Testing Accordingly
This upload field is no longer on the form.

@krisrjohnson21 Good question, but no, that test skipped that field.


Comments require Spaces at the beginning

Issue Ref: [Issue number related to this PR or None]
None
Description:
Comments require Spaces at the beginning (Obsessive compulsive disorder)
[PR Description]
Comments require Spaces at the beginning
TODOs:

[ ] Ready to review
[ ] Automated Tests
[ ] Docs

I am very confused. i am very confused, I did not modify any function, I just modified the comments.
./script/validate-vet
# command-line-arguments
cmd/kubeless/autoscale/autoscale.go:38:27: undefined: autoscaleCreateCmd
cmd/kubeless/autoscale/autoscale.go:38:47: undefined: autoscaleListCmd
cmd/kubeless/autoscale/autoscale.go:38:65: undefined: autoscaleDeleteCmd
Makefile:78: recipe for target 'validation' failed
make: *** [validation] Error 1
Exited with code 2

hi @xichengliudui,
It seems that go vet behavior changed and we didn't notice since we haven't touched this file in a while. Do you mind changing ./script/validate-vet to execute go tool vet instead?
That seems the recommended way to run go vet against files from this issue: https://github.com/golang/go/issues/23916


Strange cursor behaviour around prefix

There is an issue with cursor behaviour when inserted before prefix - first entered digit is placed after prefix, next one and following ones land at the end of string.
Here's an example:

I think that all entered digits should be appended to the end of string - so the value should be 1234.
I have used Example 2 from https://cchanxzy.github.io/react-currency-input-field/ and Chrome browser.

Oh that is a bit weird! I've never noticed that.
Thanks for raising, I'll have a look into it.

Thanks! I've also modified example by removing prefix and it seems that inserting characters on the beginning of the field works fine until comma separator is added:

Thanks for raising this issue, this was really useful. I had never used it in that way, so I didn't know about this bug.
I've just merged a PR that should fix the cursor issue, please try using v2.3.5
Feel free to reopen if you find it's still broken.


The phar file created by --prefer-lowest doesn't work with newer PHP

I should have tested it...
The example build is:
https://travis-ci.org/GoogleCloudPlatform/getting-started-php/jobs/97521412
It fails with:
Submitting to https://coveralls.io/api/v1/jobs
Client error occurred. status: 422 Unprocessable Entity
JSON file not found or failed to parse.
elapsed time: 0.360 sec memory: 5.25 MB

Now I re uploaded the phar files manually. I'm going to change the box build without --prefer-lowest. This will result in the phar file not supporting PHP 5.3.3, but this is far better than the previous state.

What is the reason that it fails?
If it fails with lowest deps then probably you should just update one of dep and still using --prefer-lowest. Also, remember to add utest showing the problem

Closing as cannot reproduce nowadays


Ignore defaults does not work properly

Here is example of config file
# check https://github.com/cseeger-epages/mail2most as example

[General]
  File = "data.json"
  TimeInterval = 10

[Logging]
  Loglevel = "debug"
  LogType = "text"
  Output = "stdout"
  Logfile = "mail2most.log"

[DefaultProfile]
  [DefaultProfile.Mail]
    ImapServer = "my def server"
    Username = "my def uname"
    Password = "my def upwd"
    ReadOnly = true
    ImapTLS = true

  [DefaultProfile.Mattermost]
    URL = "my MM url"
    Team = "my team id"
    Username = "mail2most"
    Password = "m2m pwd"
    #Channels = ["#default-channel"]
    #Users =<EMAIL_ADDRESS>    # SubjectOnly will post only the mail subject
    SubjectOnly = false
    # StripHTML will remove all HTML tags bevor sending a msg to mattermost
    StripHTML = true
    # HideFrom allows you to hide the From Address in mattermost messages
    HideFrom= false

  [DefaultProfile.Filter]
    #Folders = ["some-default-email-folder", "some-other-default-email-folder"]
    Unseen = true
    #From =<EMAIL_ADDRESS>    #To  =<EMAIL_ADDRESS>    #Subject = ["example subject"]
    TimeRange = "24h"

[[Profile]]
  IgnoreDefaults = false

  Channels = ["#my-board"]
  Users = ["roman"]

but when I try tun the docker image i get:
level=error msg="Error reaching mailserver" Error="dial tcp: missing address" Server=

As far as I understand IgnoreDefaults should use defaults values from each section e.g. Mattermost and others when in false, but unfortunately it does not work. When set up it to true get the same error. In order to fix it I used redefinition of each section.
Would be really grateful for any help here. Thank you

hi @tillroy what version of mail2most do you use ?

Hi @cseeger-epages, to be honest I use docker image virtomize/mail2most:latest

@tillroy oh my fault i forgot to update the docker image. Please download the latest again and test.

@cseeger-epages, thank you for updates, I will play with updated image and let you know if find something new, thank you for the help.


No email address in Service accounts during installation

I am installing the newest release for the first time and the installation went very well until I got to the "Manage Service Accounts" step after putting in the client ID and client secret. There are no email addresses on the screen to edit.

Any ideas on what to do next?
Thanks,
Neal

OK, the service account eventually showed up. Clicking on the three dots does not give a choice to edit.

Thanks!

OK,  Opened link in Chrome instead of whatever Microsoft browser opened automatically. Got the "Edit button" and completed the step.
Thanks for the help :)


[sqlite3] Update to 3.34.1

Update sqlite3 port to 3.34.1: https://sqlite.org/releaselog/3_34_1.html

Hi @past-due
Thanks for your PR.
LGTM. Have you tested these features in this PR?

All features have passed with the following triplets:

x86-windows
x64-windows-static

LGTM, thanks for the PR!

/azp run

/azp run

1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\broker.h(11,10): fatal error C1083: Cannot open include file: 'Poco/Data/Session.h': No such file or directory (compiling source file broker_impl_writer.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
     1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\broker.h(11,10): fatal error C1083: Cannot open include file: 'Poco/Data/Session.h': No such file or directory (compiling source file broker_impl_reader.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
     1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\broker.h(11,10): fatal error C1083: Cannot open include file: 'Poco/Data/Session.h': No such file or directory (compiling source file broker_impl.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
         isam_impl_database.cpp
         isam_impl_databaseconn.cpp
         isam_impl_errorhelper.cpp
     1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\logger.h(10,14): fatal error C1083: Cannot open include file: 'Poco/Logger.h': No such file or directory (compiling source file callstacktracer.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
     1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\cmdline.h(6,10): fatal error C1083: Cannot open include file: 'boost/regex.hpp': No such file or directory (compiling source file cmdline.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
         isam_impl_instance.cpp
         isam_impl_recordreader.cpp
     1>D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\xml.h(6,10): fatal error C1083: Cannot open include file: 'rapidxml/rapidxml.hpp': No such file or directory (compiling source file configuration.cpp) [D:\buildtrees\3fd\x64-windows-rel\c67c0b275c-accfc8aba3.clean\3FD\3FD.vcxproj]
         isam_impl_session.cpp

/azp run

After rerun, all CI checks have passed now.


Support for #400, handling response headers after upload request

I found it pretty useful to retrieve a whole XHR object, so user can easily do call file.rawRequest. getAllResponseHeaders() or file.rawRequest.getResponseHeader('X-MY-HEADER').

I think it's not so good, to expose the complete XMLHttpRequest cause it's implementation specific. If we (maybe) change the implementation in the future then all users code, which are using the xhr raw request, is broken. So please change your code to expose the headers only. That was your intention, right?

@retailify Reasonable, got it. Wrote a little parser for ByteString which is returned by getAllResponseHeaders() method.

I'll check it tomorrow ;)

adding this to README would be also cool.

Jus relized that typescript is showing me an error
Error:(208, 11) TS2322:Type '{} | null' is not assignable to type '{ [key: string]: string; } | undefined'.   Type 'null' is not assignable to type '{ [key: string]: string; } | undefined'.. However, it compiles without problems. Somebody know how to prevent this?

I cannot reproduce this bug @77bo, seems like everything is okay.


No more releases?

Yep


Can you upload your .env?

Forgive me for asking such a silly question, I want to know which directory this file should be in

no problem


Add a variable to pass in a pre-generated SchemaTemplate object inste…

…ad of generating before spreadsheet generation.

Codecov Report

Merging #60 into master will decrease coverage by 0.05%.
The diff coverage is 33.33%.

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   73.48%   73.43%   -0.06%
==========================================
  Files          35       35
  Lines        2874     2876       +2
==========================================
  Hits         2112     2112
- Misses        762      764       +2

Impacted Files
Coverage Δ

ingest/template/spreadsheet_builder.py
55% <33.33%> (-1.42%)
:arrow_down:

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 b769486...60ae0d6. Read the comment docs.


Can only run CLI when on app root

CLI assumes that you are on the app root directory.
possible solutions

~/.mantrarc file to keep paths to all apps generated by CLI. Add command to register/deregister apps
Make users set env variable MANTRA_ROOT before using CLI. Maybe add a command to do so.
Walk up the directory until it finds .meteor directory and execute command

Suggestions welcomed.

I like the walk up the directory idea.  Then as long as you're somewhere in a Meteor project it just works, with zero extra config.

I'd like to propose a fourth way. Create .mantra directory in the root of application and walk.
Pros.

It's extensible and also allow to add mantra config or something else (for example custom scaffolding templates, for additional coherence logic)
Like .mantrarc can say that is already mantra project, to avoid possible circumstances
Walk up the directory until it finds .meteor or .mantra directory and execute command

It's extensible and also allow to add mantra config or something else

I agree but I am not sure if we need to store any config at this stage. Let's wait and see if there is any need.

I can definitely see a need to allow people to configure the output.  For instance, I prefer 2-space tabs, and have my editor set to that.  Actually, it was already set to that as the default, as was ESLint.  So, every time I create a new artifact I have to manually fix all the tabs.  thankfully I have found some plugins to make it easier, but it's still a manual step I must perform each and every time, else ESLint is complaining profusely.
Past that simple annoyance of 2-space vs. 4-space vs. Tabs, I could see the desire to set "defaults" for a project.  For instance, I prefer my components to use the -c option all the time.  Or, maybe we finally get Astronomy support in, and I prefer my ORM to use Astronomy instead of Collections2/SimpleSchema.  Allowing people to customize their experience greatly improves the workflow when using Mantra CLI.


Merge devel in main

This PR implements a first draft of the library.
For now, I implemented:

[x] Center of Mass position
[x] Jacobians
[x] Forward kinematics
[x] Mass Matrix via CRBA
[x] Centroidal Momentum Matrix via CRBA

@traversaro I add just you as a reviewer since the list is restricted to just 4 people.

Thanks for the precious comments! :) I'll address the suggestions.

Thanks guys! I really learned a lot :)


Observation format

For tic tac toe, the observation array returned by the game at each move is
[3x3 array to indicate crosses on the board, 3x3 array to indicate circles on the board, 3x3 array fill with 1 if it's cross player turn else -1 if it's circle player turn]
Why do we need to return this format? If I want to customize for chess, what should I do?
Can the model work if I return the observation is the chessboard?

Hi,
If you want to take a look there is a pull request which adds chess. (not tested).
For how to encode a chess board, the Alpha Zero paper gives details. https://arxiv.org/abs/1712.01815


[TASK] Nightly tests: add support for k3s and RKE2 for amd64 architecture.

What's the task? Please describe.
Currently we're running nightly integration tests using RKE1 for amd64 arch, and k3s for arm64 arch.
We need to also add support for k3s and RKE2 for amd64 as well
Describe the items of the task (DoD, definition of done) you'd like.

[x] add k3s support for amd64 and arm64 arch
[x] add RKE2 support for amd64 and arm64 arch

Additional context
N/A

k3s support added for both amd64 and arm64 architecture.
https://github.com/longhorn/longhorn/issues/3243

Expecting rke2 :)

RKE2 support is added


python311Packages.emoji: 2.4.0 -> 2.5.0

Diff: https://github.com/carpedm20/emoji/compare/refs/tags/v2.4.0...v2.5.0
Changelog: https://github.com/carpedm20/emoji/blob/v2.5.0/CHANGES.md
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 236723 run on x86_64-linux 1

2 packages marked as broken and skipped:

python310Packages.dalle-mini
python310Packages.dalle-mini.dist

40 packages built:

deltachat-cursed
deltachat-cursed.dist
dinghy (python310Packages.dinghy)
dinghy.dist (python310Packages.dinghy.dist)
home-assistant-component-tests.google
home-assistant-component-tests.local_calendar
home-assistant-component-tests.rainbird
komikku
ntfy
ntfy.dist
octoprint
octoprint.dist
opsdroid
opsdroid.dist
python310Packages.emoji
python310Packages.emoji.dist
python310Packages.gcal-sync
python310Packages.gcal-sync.dist
python310Packages.ical
python310Packages.ical.dist
python310Packages.locationsharinglib
python310Packages.locationsharinglib.dist
python310Packages.pyrainbird
python310Packages.pyrainbird.dist
python310Packages.stanza
python310Packages.stanza.dist
python311Packages.dinghy
python311Packages.dinghy.dist
python311Packages.emoji
python311Packages.emoji.dist
python311Packages.gcal-sync
python311Packages.gcal-sync.dist
python311Packages.ical
python311Packages.ical.dist
python311Packages.locationsharinglib
python311Packages.locationsharinglib.dist
python311Packages.pyrainbird
python311Packages.pyrainbird.dist
terraform-compliance
terraform-compliance.dist


Custom uploader URL parameters value should use curly braces in ShareX v1.4.1

Hi,
Custom uploader URL parameters value should use curly braces in ShareX v1.4.1, filename should be {filename}, response URL should be {json:image}.
ref: https://getsharex.com/docs/custom-uploader#filename

Thanks for submitting this issue! Closed as of #8


Reject PRs if unit test coverage percentage falls below certain percentage

Set up the gate to reject the PRs if the unit testing coverage falls below a certain threshold ( 70-80 % may be?)

For R2 time frame we will not reject the PR but display informational data.

Rado will submit the PR this week .

https://github.com/nephio-project/test-infra/pull/215


Can't use ${selectedText} as a optional variable for Tasks

VS Code Version: 1.53.2
OS Version: Ubuntu 18.4 LTS

I want to create a task that takes either from the selected text or asks for input. something like this
{
  "label": "Blah Blah",
  "type": "shell",
  "command": "echo ${selectedText} || ${input:get-string}",
  "problemMatcher": [],
  "group": "build",
}

but it throws error when text is not selected

Variable ${selectedText} can not be resolved. Make sure to have some text selected in the active editor.

is it possible to get seleted text lenght that we can check conditionally? ${seletedTextLenght} == 0
or by default ${selectedText} return empty string? ${selectedText} == ''

See https://github.com/microsoft/vscode/issues/92854 and https://github.com/microsoft/vscode/issues/94060

Thanks for linking @ArturoDent . Duplicate of https://github.com/microsoft/vscode/issues/92854, since that's how we'd solve this problem.


mute httpsched debug logging

Repurpose the existing debug package-local flag to suppress some debug logs that are currently not suppressed.
Fixes https://github.com/mesos/mesos-go/issues/333

Coverage decreased (-0.05%) to 58.439% when pulling ec8cc79eee6dc95402b3c07414d85fcee2bc2348 on rboyer:mute-debug-logging into a414b0462f4d867d07990e633664bed52dc7cb21 on mesos:master.

thanks for the PR! LGTM


attributeFilter: expects array, not comma-seperated list

See docs/examples here:
https://msdn.microsoft.com/en-us/library/dn265034(v=vs.85).aspx?f=255&MSPPError=-2147217396
https://developer.mozilla.org/en/docs/Web/API/MutationObserver

@webinista, could you please review and confirm this PR?

@mathiasbynens, could you please review this PR?

Thanks!


Missing user @BaaaaaM

Appears the user @BaaaaaM is missing. They are listed as a maintainer. Did they change their username or delete their account? This appears to be blocking the team update script. In either event, could we please have the maintainer list updated?

Yes I change my username yesterday to bam241
Sorry about it
Can I do something to fix that?

I PR a fix for that on all repo I am listed as maintainer.
@scopatz @gonuke could you take look ?
#8
conda-forge/cymetric-feedstock#4
conda-forge/cyclus-build-deps-feedstock#5
conda-forge/cyclus-feedstock#19

Thanks @bam241. That would be the way to fix it.
No need to do this now, but it is fine to add [ci skip] in the commit messages for these sorts of changes as they don't strictly need to run on CI as we aren't changing the build in any way. Just for future reference. 😉

Friendly nudge @scopatz. When you have a chance, could you please take a look at @bam241's PRs listed above? Thanks.

Sorry about the delay. I was obsessed with cyclus/cyclus#1342

No worries. Thanks @scopatz.
Just trying to avoid the steady stream of @ core mentions that often follow things like the team update script breaking. Maybe this is just more motivation to implement a better solution ( https://github.com/conda-forge/conda-forge-webservices/issues/63 ).
Also please don't forget PR ( https://github.com/conda-forge/cyclus-feedstock/pull/19 ).

Think we have resolved this FWICT. So am going to close it out. Thanks @bam241 and @scopatz for helping straighten this out.


Godot 4 rc3: Rigid 3d automatically moves without any impact

Godot version
v4.0.rc3.official (7e79aead9)
System information
Mac Air M1 2020
Issue description
3D Physics: Rigid 3d automatically moves without any impact
https://user-images.githubusercontent.com/57259604/220741554-e9c3b5b1-f9f7-455c-85f1-25fb1bc4311d.mp4
Steps to reproduce
Create a base 3d scene which rigid cylinder and static map. Turn on axis_lock_angular_x and axis_lock_angular_y on rigid body. Run scene the cylinder will automatically moves without any impact.
I converted my project from 3.5 to 4 rc and got this error. So I created another project to test to see if it appeared, the result was like this
Minimal reproduction project
GodotProject.zip

Duplicate of https://github.com/godotengine/godot/issues/61827 and/or https://github.com/godotengine/godot/issues/62819 (the MRP uses a thin floor collision shape).
As a workaround, make your floor's collision shape thicker (i.e. make it extend below the floor).

Duplicate of #61827 and/or #62819 (the MRP uses a thin floor collision shape).
As a workaround, make your floor's collision shape thicker (i.e. make it extend below the floor).

Thanks, i tested with big floor, that does improve a bit. But the cylinder still moves on its own.
https://user-images.githubusercontent.com/57259604/220759867-c432950c-63c9-41f3-9435-a10c3cde0078.mp4

But the cylinder still moves on its own.

Cylinder collisions have many issues in GodotPhysics (also in 3.x – you didn't notice them there because Godot 3.x uses Bullet by default). I recommend using box or capsule collisions instead.

But the cylinder still moves on its own.

Cylinder collisions have many issues in GodotPhysics which are already tracked on GitHub. This is also the caes in 3.x, but you didn't notice them there as Godot 3.x uses Bullet by default (while Godot 4.0 no longer has Bullet physics).
I recommend using box or capsule colliders instead (or a cylinder converted to trimesh, for static bodies).

Thanks for the details, that why my project physics look weird when I switch to godot 4.

Have the same issue with thin items. Making things thicker doesn't help if the object, in this case a candy bar, needs a thin collider.


Fix bug with overlapping applications for leave

Avoid that applying for leave is possible if there is already
a full day vacation.

refs #257

Coverage increased (+0.01%) to 68.701% when pulling 12cf8ee874712cb4bb2ed1e602d8886e186955be on fix-overlapping-vacation-bug-257 into 5bb1ef70fb6607231ff6f9071a709108c8f82539 on master.


[In-meeting][Updatecontent][1615882]

Docs Build status updates of commit b30d04d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/apps-in-teams-meetings/design/designing-apps-in-meetings.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@Trinetra-MSFT - gentle reminder to review and provide comments / approval for the PR.
Thank you.