where to get build pre-requistes

Two points I encountered on build pre-requistes:

https://github.com/trento-project/trento/blob/main/README.md#L80-L81
- [`Go`](https://golang.org/) ^1.16

Regarding the required Go compiler, on SLES the Development tools module needs activating, e.g. "SLE-Module-DevTools15-SP3-Updates".   On opensuse Leap Go is available, so fine there.

- [`Node.js`](https://nodejs.org/es/) ^15.x

I can't find nodejs15 on standard Leap or SLES repositories, so we should document where SUSE users should get this from. Maybe https://software.opensuse.org/package/nodejs15 ?
thanks, lee

Actually, we just need to fix the https://nodejs.org/es/ link to https://nodejs.org.
Whoever wants to build the thing from source might not even want to use the package manager, or they might not be using any SUSE distribution.
Users are invited to look up to the upstream project documentation on how to install them in their environment; we don't want to get into the rabbit hole of writing documentation about that (because it has been done already by upstream, for example: https://nodejs.org/en/download/package-manager/#opensuse-and-sle).

Made invalid by container builds. All build dependencies are now provided there.


unwrapper.packAtlas(...) goes wrong

Here is the code

package.json

before unwrapping

here is the result of the unwrap(unwrapper.packAtlas(...))

I pass to unwrapper.packAtlas(...) array with only one geometry:

UPD: meanwhile with the model from this github project https://github.com/lucas-jones/three-lightmap-baker all is ok:

what is the problem?

Hi, can you share the 3d model which has this issue?

Hi!
Thanks for great library, it works mainly very well! I have similar issue unwrapping issue with some geometries. I don't understand why geometry get broken.
The issue occurs only with rounded geometries (usually small triangles). Do you what is the problem?
Example of broken model (single geometry) below.
not_working_geometry_glb.zip


Various bug fixes to allow vr_run to work with soil and abiotic_simple models

Description
This pull request is mainly small bug fixes to make the vr_run function runable for the abiotic_simple and soil models (using data stored on RDS).
The most substantive change is moving the definition of layer roles from the abiotic_simple model into the core utilities. This was necessary as the soil model also needed to make use of these layers roles (to find the index associated with the top soil layer).
This PR also updates the documentation to explain to users how to access the data (if they are part of the VR project), and updates the simulation flow documentation to reflect recent changes to the flow.
Fixes #235
Type of change

[ ] New feature (non-breaking change which adds functionality)
[x] Optimization (back-end change that speeds up the code)
[x] Bug fix (non-breaking change which fixes an issue)

Key checklist

[x] Make sure you've run the pre-commit checks: $ pre-commit run -a
[x] All tests pass: $ poetry run pytest

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[ ] Tests added that prove fix is effective or that feature works

Adding @vgro as a reviewer as I'm not sure of best practice when a pull request is cowritten

Codecov Report

Merging #236 (9cd8139) into develop (299e39b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #236      +/-   ##
===========================================
+ Coverage    95.23%   95.25%   +0.02%
===========================================
  Files           36       36
  Lines         1553     1561       +8
===========================================
+ Hits          1479     1487       +8
  Misses          74       74

Impacted Files
Coverage Δ

virtual_rainforest/core/data.py
88.60% <ø> (ø)

virtual_rainforest/core/utils.py
97.56% <100.00%> (+2.32%)
:arrow_up:

...rest/models/abiotic_simple/abiotic_simple_model.py
100.00% <100.00%> (ø)

...nforest/models/abiotic_simple/simple_regression.py
100.00% <100.00%> (ø)

virtual_rainforest/models/soil/soil_model.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

LGTM - minor comment on using admonitions.
I think we should structure the main simulation document a bit more (and introduce all of the defined steps, even if they are currently lightly used). I was going to make suggestions, but instead I'm going to commit a suggested update. I've updated the Sphinx conf to include a mermaid diagram. I think that is useful here and it is probably something we'll want to use more of.

I like the idea of using the diagrams, however, they don't come out very nicely at the moment (e.g. the lines go over the text).

Yeah - it definitely isn't as I'd like and I am normally really fussy about figures. That is mostly the tool: mermaid doesn't really do fine controls -  or at least not without lots of tweaking of things. It does produce diagrams quickly though! We could replace this with a hand crafted image that we're happier with when things are stable, but this is quick to edit and update for now?


feat(WEG-114): Sort by priority first and then by title or distance

This PR adds a Priority Option field to Grist and uses that in frontend to sort the facilities accordingly.  Items are sorted first by priority, then by name
The geolocation filter is on. Items are sorted first by priority, then by distance

:tada: This PR is included in version 1.4.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Bug] internlm-chat-20-4bit运行时回答乱码

Checklist

[x] 1. I have searched related issues but cannot get the expected help.
[ ] 2. The bug has not been fixed in the latest version.

Describe the bug

Reproduction
python3 -m lmdeploy.serve.turbomind.deploy internlm-chat   --model-path ./internlm-chat-20b-4bit  --model-format awq     --group-size 128
Error traceback
No response

I am facing the same issue

It got solved for me, it is basically fault of nvidia architecture you are using. Check Cuda device capability by torch.cuda.get_device_capability(), if it is greater or equal to 8.0, then it will work perfectly

@deku0818 just as @Tushar-ml suggested, 4bit inference requires the capability of nvidia arch no less than 80.


gitpod.io

I created a .yml file.
Try it out here: https://gitpod.io/#github.com/weaversa/cryptol-course
We should add the VS Code extensions and update the installation instructions. I think this will be the preferred way to take the course. It's so easy!
Thoughts?

I added the pdf extension, had the README file open automatically, and started Cryptol (with some help text) in the terminal.

I'm a heavy gitpod user esp. for cryptol dev so it was quite nice to see it included here!


Doesn't recognize express npm

I get this error below whenever i npm update or upgrade. I also can not run express and mean or mern project in my computer; however when i type npm it brings me commend line for npm. None of them are working at all. I tried to remove nodejs and npm, reinstall everything . It was completely working a while ago.
Francos-MacBook-Pro:test carbon$ npm  update
npm ERR! Darwin 15.4.0
npm ERR! argv "/usr/local/Cellar/node/5.9.1/bin/node" "/usr/local/bin/npm" "update"
npm ERR! node v5.9.1
npm ERR! npm  v3.7.3

npm ERR! No compatible version found: express@linked
npm ERR!  If you need help, you may report this error at:
npm ERR!     <https://github.com/npm/npm/issues>

npm ERR! Please include the following file with any support request:
npm ERR!     /Volumes/HD 2/Projects/tmp/npm-debug.log

What does your package.json file look like in regards to express? Specifically, which version are you trying to pull back?

  "_args": [     [       "express",       ""     ]   ],   "_from": "express@latest",   "_id"<EMAIL_ADDRESS>  "_inCache": true,   "_installable": true,   "_location": "/express",   "_npmUser": {     "email"<EMAIL_ADDRESS>    "name": "dougwilson"   },   "_npmVersion": "1.4.28",   "_phantomChildren": {},   "_requested": {     "name": "express",     "raw": "express",     "rawSpec": "",     "scope": null,     "spec": "latest",     "type": "tag"   },   "_requiredBy": [     "#USER"   ],   "_resolved": "https://registry.npmjs.org/express/-/express-4.13.4.tgz",   "_shasum": "3c0b76f3c77590c8345739061ec0bd3ba067ec24",   "_shrinkwrap": null,   "_spec": "express",   "_where": "",   "author": {     "email"<EMAIL_ADDRESS>    "name": "TJ Holowaychuk"   },   "bugs": {     "url": "https://github.com/expressjs/express/issues"   },   "contributors": [     {       "email"<EMAIL_ADDRESS>      "name": "Aaron Heckmann"     },     {       "email"<EMAIL_ADDRESS>      "name": "Ciaran Jessup"     },     {       "email"<EMAIL_ADDRESS>      "name": "Douglas Christopher Wilson"     },     {       "email"<EMAIL_ADDRESS>      "name": "Guillermo Rauch"     },     {       "email"<EMAIL_ADDRESS>      "name": "Jonathan Ong"     },     {       "email"<EMAIL_ADDRESS>      "name": "Roman Shtylman"     },     {       "email"<EMAIL_ADDRESS>      "name": "Young Jae Sim"     }   ],   "dependencies": {     "accepts": "~1.2.12",     "array-flatten": "1.1.1",     "content-disposition": "0.5.1",     "content-type": "~1.0.1",     "cookie": "0.1.5",     "cookie-signature": "1.0.6",     "debug": "~2.2.0",     "depd": "~1.1.0",     "escape-html": "~1.0.3",     "etag": "~1.7.0",     "finalhandler": "0.4.1",     "fresh": "0.3.0",     "merge-descriptors": "1.0.1",     "methods": "~1.1.2",     "on-finished": "~2.3.0",     "parseurl": "~1.3.1",     "path-to-regexp": "0.1.7",     "proxy-addr": "~1.0.10",     "qs": "4.0.0",     "range-parser": "~1.0.3",     "send": "0.13.1",     "serve-static": "~1.10.2",     "type-is": "~1.6.6",     "utils-merge": "1.0.0",     "vary": "~1.0.1"   },   "description": "Fast, unopinionated, minimalist web framework",   "devDependencies": {     "after": "0.8.1",     "body-parser": "~1.14.2",     "connect-redis": "~2.4.1",     "cookie-parser": "~1.4.1",     "cookie-session": "~1.2.0",     "ejs": "2.3.4",     "express-session": "~1.13.0",     "istanbul": "0.4.2",     "jade": "~1.11.0",     "marked": "0.3.5",     "method-override": "~2.3.5",     "mocha": "2.3.4",     "morgan": "~1.6.1",     "multiparty": "~4.1.2",     "should": "7.1.1",     "supertest": "1.1.0",     "vhost": "~3.0.1"   },   "directories": {},   "dist": {     "shasum": "3c0b76f3c77590c8345739061ec0bd3ba067ec24",     "tarball": "http://registry.npmjs.org/express/-/express-4.13.4.tgz"   },   "engines": {     "node": ">= 0.10.0"   },   "files": [     "LICENSE",     "History.md",     "Readme.md",     "index.js",     "lib/"   ],   "gitHead": "193bed2649c55c1fd362e46cd4702c773f3e7434",   "homepage": "https://github.com/expressjs/express",   "keywords": [     "express",     "framework",     "sinatra",     "web",     "rest",     "restful",     "router",     "app",     "api"   ],   "license": "MIT",   "maintainers": [     {       "email"<EMAIL_ADDRESS>      "name": "dougwilson"     }   ],   "name": "express",   "optionalDependencies": {},   "readme": "ERROR: No README data found!",   "repository": {     "type": "git",     "url": "git+https://github.com/expressjs/express.git"   },   "scripts": {     "test": "mocha --require test/support/env --reporter spec --bail --check-leaks test/ test/acceptance/",     "test-ci": "istanbul cover node_modules/mocha/bin/_mocha --report lcovonly -- --require test/support/env --reporter spec --check-leaks test/ test/acceptance/",     "test-cov": "istanbul cover node_modules/mocha/bin/_mocha -- --require test/support/env --reporter dot --check-leaks test/ test/acceptance/",     "test-tap": "mocha --require test/support/env --reporter tap --check-leaks test/ test/acceptance/"   },   "version": "4.13.4" }

The issue looks like in your package.json you have the version for express set to linked for some reason, which isn't a version or tag in npm. You'll want to fix your package.json.
The easiest way to resolve this issue is probably just running npm install --save<EMAIL_ADDRESS>If you're still having issue or need further help, please drop by the support channel: https://gitter.im/expressjs/express

it still gives me an error when try to run npm install —save<EMAIL_ADDRESS>the version is written at the bottom of my son file though.

On Mar 29, 2016, at 10:56 PM, Douglas Wilson<EMAIL_ADDRESS>wrote:
The issue looks like in your package.json you have the version for express set to linked for some reason, which isn't a version or tag in npm. You'll want to fix your package.json.
The easiest way to resolve this issue is probably just running npm install --save<EMAIL_ADDRESS>If you're still having issue or need further help, please drop by the support channel: https://gitter.im/expressjs/express https://gitter.im/expressjs/express
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/expressjs/express/issues/2961#issuecomment-203216567

npm issue with kik: http://blog.npmjs.org/post/141577284765/kik-left-pad-and-npm
this is looking from the outside (not npm blog, and more drama) http://www.theregister.co.uk/2016/03/23/npm_left_pad_chaos/

Hi @francoeeeen, I'm not sure how that relates to the issue you are having. You se to have something corrupted on your computer. Have you tried on a different computer? Have you tried dropping by our support channel?

Hi @dougwilson, I have actually talked couple contributors; however the only solution i got is ,npm -kik. I have tried using that it didnt work at all. I ended up reinstalling my OS. Now everything is fine.


Implement Ratelimits

At some point in the future we might want to implement ratelimit's on the actual command them self.

https://github.com/z64/rate_limiter

Will be implemented with #110


Remove deprecated cube method assert_valid

The method iris.cube.Cube.assert_valid was deprecated in Iris 0.8 (!)
https://github.com/SciTools/iris/blob/master/lib/iris/cube.py#L2121

Closed by #1595


fix RSS image when no Image is mentioned in the Feed

The library was panicking when no Image were specified in feeds.Feed

Thanks! Looks good, went ahead and merged the changes.


Respect :skip re: timestamp-only updates

When attributes are ignored, either by :ignore or by :skip,
then during updates, timestamp attributes like updated_at should
not be considered notable changes.
[Fixes #569]

I'm looking at the code, it doesn't look like any of the logic has changed... I've re-read it a few times and I'm scratching my head. Is this the difference here? The + self.paper_trail_options[:skip]?

Thanks for looking into this btw!

I'm looking at the code, it doesn't look like any of the logic has changed... I've re-read it a few times and I'm scratching my head. Is this the difference here? The + self.paper_trail_options[:skip]?

Yes, that's the key difference.  For the purposes of whether a change is "notable", we should respect both :ignore and :skip, but without this PR, we do not respect :skip.

+1, good catch on this one! Based on README description of how skip should function this seems right.

Cool, thanks for reviewing.  Do you want to cut a new release candidate with this and https://github.com/airblade/paper_trail/pull/566 ?

I think we probably should cut a new one since there have been a lot of changes to master since rc1, but I'm inclined to wait until the final release to merge in #566, because the warnings say they will be removed for 4.0.0 final.


Grafana dashboard to open inside Manageiq.

Hi @Fryguy
I need a Grafana dashboard to open inside Manageiq.
Actually, we are monitoring VM details In the Grafana application, now i want to open that application inside ManageIQ.

Can you give some more background and detail on what exactly you're trying to do? Where is Grafana running in this architecture and what is it presenting?  Do you want the dashboard embedded or just a hardcoded link / menu item out to where it's running?

Hi @Fryguy
I need a Grafana dashboard open inside the Manageiq application.
Yes, I want the dashboard open with an embedded method or (I-frame) concept.
Grafana application URL: http:///192.168.XXX.XX:3000/
For your reference, I shared the screenshot below.

Could you please guide me on how to proceed from here?

Is it acceptable to just have the menu item open a new tab with Grafana?  If so, that functionality already exists.

Hi @Fryguy
Thank you for your update.
We need a new tab to open inside Manageiq .

Is it acceptable to just have the menu item open a new tab with Grafana? If so, that functionality already exists.

Then guide me how to proceed here.

Hello @Fryguy,
For the scenario mentioned above, we are now able to access the Grafana dashboard successfully. However, we are unable to login to the Grafana application.
To help troubleshoot the issue, I’ve shared a screenshot for your reference:

Hello @Fryguy,
We followed the steps outlined in the Commvault documentation to integrate the Commvault Command Center into the CloudForms menu.
While the integration appears successful, we are facing issues logging in to the Commvault Command Center when accessed through the menu in CloudForms.
Here are the details:
The Commvault Command Center opens as expected in the iframe.
However, the login process does not proceed, and we are unable to authenticate.


Break words only in tables to avoid break words for whole post body.

This is just a temporary fix.
To fix the overflow problem once and for all:

All elements that is possible to flow should have overflow-x: auto; applied to its parent.
To ensure robust there should be JavaScript dynamically apply overflow-x: auto; to elements that overflow the width of the device.

Thanks. :+1:


DBeaver should warn before opening dozens of tables / views etc.

DBeaver Version: 6.1.3
Is your feature request related to a problem? Please describe.
Currently, if I select a few dozens tables in an attempt to e.g. delete them, and if I subsequently accidentally double click the selection, DBeaver opens up all tables / views selected without issuing any warning. If the list of tables was long, I end up with dozens of opened TABs in the DBeaver window. Maybe I have missed it, but there doesn't seem to be any menu option to close all open TABs in one action, so I need to manually close each object (Closing DBeaver and re-opening it will also re-open all open TABs again...)
Describe the solution you'd like
DBeaver should either warn before opening multiple tables / views etc., or DBeaver should offer a menu option to close all tabbed windows in one go.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Good point

DBeaver will show confirmation dialog if you are opening more than 3 editors at once.

on a side note, DBeaver does allow to close all tabs by right-clicking on one of them.

Verified

@p91paul
Thanks for the tip, stupid, I missed that one!
I already tried right-clicking, but not with the mouse hovering exactly over the TAB itself, hence I didn't encounter the context menu.


Switch test runner to pytest

Please use self.assertEqual, here and everywhere:

It's consistent with the rest of the code
It gives more informative messages via the test runner

It is because I'm using py.test everywhere, it is able to generate great debug info from regular assert calls (BTW I strongly recommend as unit test runner here too, it should be able to run current tests). But for now I of course change it.
Originally posted by @mrk-its in https://github.com/RaRe-Technologies/smart_open/pull/383

Is this still a valid issue? I could probably contribute here but I'm not clear as to what the issue is. It looks like pytest is being used to run the tests currently. Does this ticket mean to have the command python setup.py test run pytest? This is a fairly easy thing to do and I could knock it out quickly.

No, I think the idea was to rewrite the tests so they look like py.test ones instead of the standard library's unittest ones. Pytest has its own way of dealing with fixtures, etc.
Leave this item with me, I'm still working out what exactly to do, whether it's worth it, and when to do it.


applySingleConstArgVectorFunction loses exceptions

Summary:
applySingleConstArgVectorFunction in Expr is a highly specialized way of invoking Vector
functions when there is only one non-literal argument to the function and it's constant.  In
this case, it does something similar to peeling where it copies the value into a flat vector if
necessary removing the Constant encoding.
There is a potential bug here because we are not saving the context.  If an error occurs it
happens at row X, where X is the wrappedIndex returned by the ConstantVector.
However, the row we're evaluating is row Y, the single row pointed to by rows.  Since
these don't match up TRY will not NULL out the value at row Y and we'll get garbage
results.
In addition, just saving the context is insufficient because chances are, the Constant was
peeled before the function was applied (this doesn't not remove the Constant encoding, it
only ensures we evaluate one row) this saves its own context, and when reapplying the
Constant wrapping to the result it uses the stored constantWrapIndex_ to figure out which
row the value is stored at, as well as which row to check for exceptions.  When we save
another context in applySingleConstArgVectorFunction it resets constantWrapIndex_ to 0,
we need to save and restore it to the original value to ensure the unpeeling still works.
Differential Revision: D35760752

@kevinwilfong is this PR still relevant? If so, can we work on unblocking it?


Add support binaries with required features

In https://github.com/probe-rs/probe-rs/, we have a couple of binaries which only get built when a feature is enabled. Right now, cargo-dist tries to release these binaries even though they can't be built without the required feature.
It would be great if cargo-dist could enable the required features automatically, or at least have a setting for it in the configuration.
Might be a duplicate of #22

Would you accept a PR implementing support for this?
My plan for implementing it would be to keep track of required features for the binaries, and pass that along to the build step, without exposing any additional config settings.

Oh whoops, sorry I need to be more proactive in doing triage -- I think this is definitely something we want! Ideally we could just detect that your build is Like This so no configuration would be required -- how are you expressing it to cargo?
(PRs are welcome of course, but this requires some non-trivial config/ux work)

So we have a combined lib/bin crate. To not pull all sorts of crates into the lib target, we have a cli feature that is required to build the bin target.
We have no way to tell cargo that it is required. Of course we could make it a default feature, but that would mean all the lib users have to use default-features=false when they use the library.
So the only way I see is having custom cargo metadata or passing it via CLI.
I was snooping a bit in the code and it seems non-trivial to patch this, since we cannot just pass the features to all crates.
If we find some good solution for this, I am happy to PR.

I think that would make perfect sense with the precise-builds config in the 0.1.0 release (coming out tomorrow). If you set a feature on a package we can just force enable precise builds.

I gave it a jab here: https://github.com/axodotdev/cargo-dist/pull/317 it requires to set precise-builds manually atm. Also, I am not sure the code is the cleanest it could be.
Feel free to copy paste/push into the PR however you like :)

Oh lord, sorry ok this feature is needed because required-features doesn't do what I thought.

The required-features field specifies which features the target needs in order to be built. If any of the required features are not enabled, the target will be skipped.

So this is just telling cargo that the build can be skipped without those features. Unfortunately there was an error in prerelease.1, so I'll cut prerelease.2 with a fix in about an hour.

v0.2.0-prerelease.2 should work and has the desired functionality
https://github.com/axodotdev/cargo-dist/releases/tag/v0.2.0-prerelease.2

@Gankra I get the error https://github.com/IgnisDa/ryot/actions/runs/5941550457/job/16114985815#step:7:1148 with v0.0.7 and v0.1.0 (which I tried locally). What can I do to fix it? Thanks.


Correction for raw affiliation Modelling, Information and Systems (MIS) Laboratory, Picardie Jules Verne University, Amiens, France

Correction needed for raw affiliation Modelling, Information and Systems (MIS) Laboratory, Picardie Jules Verne University, Amiens, France
raw_affiliation_name: Modelling, Information and Systems (MIS) Laboratory, Picardie Jules Verne University, Amiens, France
new_rors: 01gyxrk03
previous_rors:
works_examples: W4398183298
contact: edc736acab05189ab9b5e115b147386c:39c910c9c48e06af66f3e1093a142825 @ u-picardie.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Overriding str or repr not honored by outer object

We can derive a class from a NamedTuple to attach methods to it, such as __str__ and __repr__, but if that object is part of an other NamedTuple, and we try to print that object, the overridden __str__ or __repr__ method will not be used. See example below. This might be connected to Issue #156
We may also want to test other type combinations (not just NamedTuple within class derived from NamedTuple).
class A(NamedTuple(secret=str)):
    def __str__(self):
        return "***"

B = NamedTuple(x=A)

a = A(secret="112233")
b = B(x=a)

print(f"a:{a}") # -> a:***
print(f"b:{b}") # -> b:(x=(secret="112233",),)

the last line of the test code described in this issue above still prints the secret field

you should be overriding repr, not str.
On Thu, Dec 5, 2019 at 10:12 AM Alexandros Tzannes<EMAIL_ADDRESS>wrote:

the last line of the test code described in this issue above still prints
the secret field
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/APrioriInvestments/typed_python/issues/279?email_source=notifications&email_token=AB6OHBBTMAW7QUHROAHZWO3QXEK63A5CNFSM4JU5H6UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEGBBBNA#issuecomment-562172084,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB6OHBHKHHACYSQ3DETUHCDQXEK63ANCNFSM4JU5H6UA
.

oh, got it! f"{x}" calls __repr__ on x. I checked that both print(f"{x}") and print(f"{str(x)}") do what's expected when both strandrepr` are overridden. :+1:


Bug Report: week 4 Extremely laggy

Please check for duplicates, similar issues or closed issues with the same problem, as well performing simple troubleshooting steps (such as deleting the export folder if it's an issue with compiling, reinstalling Kade Engine) before submitting an issue.
If you are playing a downloaded version of the game, what operating system are you using?
Windows (x86), Windows (x86_64), Linux, or macOS? Specify below.
What version of the game are you using? Look in the bottom left corner of the main menu.
Have you identified any steps to reproduce the bug? If so, please describe them below. Use images if possible.
Please describe your issue. Provide extensive detail and images if possible.

skill issue :trollface:

dude what are you on?

no info


Warning should be raised when log scale is used for negative values

This is a good issue for newcomers!
The code to change would be around https://github.com/cosmo-epfl/chemiscope/blob/15bdb4147c3bb022d17a8b67d82fa53fd3145d88/src/map/options.ts#L212, adding a check that all values are positive before applying the operation in the 'log' case.

@Luthaf I think a quick array method on the value would do the check. I'm a little stuck on what kind of warning to raise. Should the function return null or would the warning need look like something else?

Yes, sendWarning is the right function for this! You can call it with a string argument containing the warning message, something like sendWarning("this property contains negative values, taking the log will discard them").

@Luthaf I think I've got the right idea for what to do, but I'm having a bit of trouble with the linter. I wrote the check as an arrow function, and as per output I added parentheses and semi-colons. It is passing all other tests. Would you know why it's still failing that one with the linter?

Hi @3dvkr -- looking at #122 right now. I think I intended this issue to apply to the log scaling of the axes, as the sizes are never negative, but rather scaled between the max and min of the property value. This can be seen in the built chemiscope that the log-scaled sizes never make negative values disappear, but rather just get very small. However, when you apply a log scale to the axes, points do disappear. Sorry for the confusion! I should have corrected @Luthaf's original comment.

Right, I should have read the surrounding code as well =/. Really sorry about that.
This makes this issue a bit harder to deal with, the code to modify is now the three functions this._options.x.scale.onchange/this._options.y.scale.onchange/this._options.z.scale.onchange; as well as changeRange.
https://github.com/cosmo-epfl/chemiscope/blob/eecc7349b2c31f992930ff7343086a8d98a28e62/src/map/map.ts#L350-L358
https://github.com/cosmo-epfl/chemiscope/blob/eecc7349b2c31f992930ff7343086a8d98a28e62/src/map/map.ts#L362
You can get the values for the coordinates on one axis using this._coordinates(this._option.x, 0)[0] (and similar with this._option.y and this._option.z). The axis style (linear/log) is stored in this._options.x.scale.value/this._options.y.scale.value/this._options.z.scale.value as either 'linear' or 'log'.

No problem. Would there be a threshold value that we should check for in the options.ts file? If I understand correctly, we could have the check make sure that the values are above this threshold if the scale mode is log instead of checking that the numbers are positive. I'll need a bit more time to comb through and come up with a plan for the three functions, but the check could acknowledge the issue to users for the time-being?

What happen in options.ts is related to the size of the points in the map; but this issue is actually related to using log scale for the x/y/z coordinates of the points in the map.
What @rosecers was saying above is that for the size of the points, scaling happening in options.ts, and we already guarantee that values passed to Math.log are positive by rescaling them between 0 and 1. The code doing this is here: https://github.com/cosmo-epfl/chemiscope/blob/eecc7349b2c31f992930ff7343086a8d98a28e62/src/map/options.ts#L237.
The only thing that needs a check is the corresponding code for x/y/z coordinates of the points. In this case, we use plotly's capabilities to do a logscale plot (setting xaxis.type and friends to 'log'), and do not compute the log ourself.
https://github.com/cosmo-epfl/chemiscope/blob/eecc7349b2c31f992930ff7343086a8d98a28e62/src/map/map.ts#L350-L358
But since taking the log of a negative value results in NaN, the corresponding points are dropped from the map, and that's what the warning would have to be about.
Let me know if I'm unclear 😃

@3dvkr you're right on with making a threshold -- hypothetically if there's one value at 1E-12 and the rest between 1E-4 and 1E-2, we don't want the map to have the majority of the data squashed to one side and one lone point sitting on its own. Ideally it should be a tunable threshold, what do you think @Luthaf ?

There is already a min/max setting for the axis range, which should deal with this. Or are you referring to something different?

No, I'm talking with respect to this. When going to log scale, the minimum should automatically change to some epsilon > 0. Agreed?

I think this is a separate issue from this one 😄. My feeling here is that we should not mess with these value automatically in case the user already set them to something meaningful. If the min is negative, we will get a warning once this improvement is implemented (actually, we don't have to check all values for the warning, only the min). If the min is positive but the data distribution is strange, I'd rather leave it to the user to adapt visualization as they want.

Two notes:

we may also want to check the max (imagine the case where all values are
negative, what's the expected behavior?
if there are negative values we should de facto enforce as a minimum the
smallest positive value.

On Tue, 2 Mar 2021 at 15:10, Guillaume Fraux<EMAIL_ADDRESS>wrote:

I think this is a separate issue from this one 😄. My feeling here is
that we should not mess with these value automatically in case the user
already set them to something meaningful. If the min is negative, we will
get a warning once this improvement is implemented (actually, we don't have
to check all values for the warning, only the min). If the min is positive
but the data distribution is strange, I'd rather leave it to the user to
adapt visualization as they want.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/cosmo-epfl/chemiscope/issues/109#issuecomment-788935607,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAIREZ5VYX4Y4JE6NFRDNOTTBTWU7ANCNFSM4XA32N6A
.

Hi @Luthaf,
I tried going through the map.ts file, and I can find a min value in line 369, but cannot find where the file checks the scaleMode that the user has selected. I'm also unsure that the min in line 369 is the same min from options.ts.
In map.ts there's also a this._options.size.mode.value, line 550, which is a slightly different path than this.size.mode.value which is what scaleMode is assigned to in options.ts.
They are both properties of a MapOptions-type object/class, but can you please confirm if I've found the correct values/properties in map.ts? By checking if they're log and negative, would that check (this._options.size.mode.value === 'log')  be an else if statement around line 553?

Fixed in PR #146


FNF : incorrect use of 'More options' triple dot control (and poorly aligned)

Version

Template: Core bot with language
Menu item:  Create
Area: property pane section, 'Dialog interface'
The default for input/output is empty
When adding a new input, 3 fields appear and one, poorly aligned triple dot icon with a single action: Remove
Typically, triple dot menus apply to the (single) field it is associated with.
If a single action applies to more than one thing, it should visually indicate that.
Note: maybe there will be more then 1 item (remove) in the menu when adding multiple inputs, but I cannot test that as the 'new' button is not working for me.

@mareekuh Since this creates an object, you have to set the key before you can add another item. All keys must be unique, so we cannot add two items with the same key.

(needs a new design)

Agreed. (-:

(changing the title to shift emphasis on the "Dialog interface" section of the UI, rather than the ...)


Add integration tests for slack function

Perhaps using https://github.com/bblimke/webmock?

This stackoverflow answer highlights some problems we may face in adding integration tests to assert our app communicates results back to slack.

Because your main thread doesn't halt execution, most of the time your assertion is run before the mail has been sent.

I think webmock is a good approach, but we'd still face the problems i'm mentioning here. Workarounds might include using Thread.new only in non-test environments or using Sidekiq with the following test configurations:
require 'sidekiq/testing'
Sidekiq::Testing.inline!

Looks like that Thread.new stuff is attempting to do the communication with Slack in a way that it doesn't slow down the web request.  That's a fine goal, but I'm wondering about the tradeoffs.
In a larger application I'd suggest something like delayed_job or sidekiq to do this work.  (And those have good testing stories.)  I'm guessing this application isn't big enough to justify the extra complexity (or a heroku dyno ... how's it hosted?)
Going back to the original premise ... is this web request so important that it needs to be performed in the background?  Keep in mind that you're losing a lot of visibility for managing the performance, capturing exceptions, etc.   If not, removing the threads simplifies both your hosting and testing stories.
If the web request is that important (or the slack callback that likely to be slow), then I'd suggest wrapping all the thread management in a library somewhere, and making that library simply be single threaded in the test environment.  (Or, like sidekiq, have configuration where the tests can make them inlined.)


feat: support mqtt

Improve log, log.Start(ctx) with invoke fn
Add project Mqtt broker support
wasm depends middlewares add Init interface
wasm resource support mqtt.Message

add send mqtt abi

add send mqtt abi


Back 4 Blood 4.25d file support and fix?

The game version is 4.25d and here's the file dump tutorial for this game: https://www.youtube.com/watch?v=oAnQQ06eZRA
Some files always get error like this so the tool cant read it for this game.

Please provide a copy of an asset that does not parse with UAssetGUI

PlayerCards.zip
Some of them can be read, some of them dont.

Fixed in atenfyr/UAssetGUI@7c7191a and atenfyr/UAssetAPI@b025ade


Last update.sh version broken instalation

Hello,
First, thank very much for your work.
I run your script last version and don't realize than now you install tiny version.
The scrip run well but now tailscale is not working and when trying to execute tailscale start it gives a error "illegal instruction"

Could you please share some information about the device you run this script on?

https://www.gl-inet.com/products/gl-xe300/

Could you please run uname -a on this device?

Could you please run uname -a on this device?

with --no-tiny

Please try to run the script like ./update-tailscale.sh --no-tiny to download the official tailscale version and see if that fix it.

Please run
wget -qO /tmp/tailscale.tar.gz https://pkgs.tailscale.com/stable/tailscale_1.68.1_mips.tgz
./update-tailscale.sh --no-download --no-tiny

Please run
wget -qO /tmp/tailscale.tar.gz https://pkgs.tailscale.com/stable/tailscale_1.68.1_mips.tgz
./update-tailscale.sh --no-download --no-tiny

Download the file from a private location. Then:

Thanks! Seems that the tailscale download server does not allow wget to download. I remember about this issue.
Will try to fix it on sunday.

The issue is fixed in 343a8d9b953915e84f38e7f45b6f5360a1861ed0

Can you please test the download by running
wget -U "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" -qO /tmp/tailscale.tar.gz https://pkgs.tailscale.com/stable/tailscale_1.68.1_mips.tgz

Are you using VPN or something?

4g connection on the router

The problem is that I can't figure out why dl.tailscale.com is blocking you. I already implemented a fix for the old wget version so a newer user agent will be sent ... but since you are still blocked it's an issue I can't fix on my side :(

:(

If you use a PC, can you download the file https://pkgs.tailscale.com/stable/tailscale_1.68.1_mips.tgz?

yes
And if i share it on my synology drive i can download it on the gl. inet router

Can you try
curl https://pkgs.tailscale.com/stable/tailscale_1.68.1_mips.tgz --output /tmp/tailscale.tar.gz -v

please?

With curl, works

Well, that's weird. But easy fix then - I'll change from wget to curl.

NICE!

Hopefully fixed in 0b78b6d

Thank you very much


regex error when running the code

Hi there,
I successfully compiled the code. However, when I tried to run the test file, I am getting the following error :
terminate called after throwing an instance of 'std::regex_error'   what():  regex_error
Although there was no issue in compiling step, I also tried to compile with different version of gcc like  gcc/4.9.5  gcc/5.3.0  gcc/7.5.0(default)  gcc/8.3.0   gcc/10.2.0  but it does not resolve the problem.
Appreciate your suggestions to solve this issue. thanks,

I am just following up to check if you got some time to look at the issue
posted on github.
Ali
On Tue, May 31, 2022 at 8:03 PM alipazokit @.***> wrote:

Hi there,
I successfully compiled the code. However, when I tried to run the test
file, I am getting the following error :
terminate called after throwing an instance of 'std::regex_error' what():
regex_error
Although there was no issue in compiling step, I also tried to compile
with different version of gcc like gcc/4.9.5 gcc/5.3.0 gcc/7.5.0(default)
gcc/8.3.0 gcc/10.2.0 but it does not resolve the problem.
Appreciate your suggestions to solve this issue. thanks,
—
Reply to this email directly, view it on GitHub
https://github.com/mkerin/LEMMA/issues/8, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALFGXJQIC6UAPQ4C36OL4JTVM3HITANCNFSM5XPQNDSQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Hi Ali. Sorry, I have only just seen this. I will have time to take a look tomorrow...
I suspect that being able to compare the compilation logs will be useful, so if you have these readily available I would appreciate it if you can upload them.

Thank you !
Here are the files :
CMakeCache.txt
compile.lemma.log.txt

Hmm, I have tried recompiling LEMMA and rerunning the examples but I haven't managed to reproduce any errors. Could you be specific about what you mean exactly by the 'test file' / what input you are using to produce the error?
It would also be helpful if you could run the unit tests and let me know if they pass. You should be able to do this as follows
cd /u/home/a/alipazok/sriram/LEMMA/
cmake --build build --target tests -- -j 4
ctest --output-on-failure

I meant running the following based on here :
mpirun -n 1 build/lemma_1_0_3 \
  --gplemma --random-seed 1 \
  --pheno example/pheno.txt.gz \
  --streamBgen example/n5k_p20k_example.bgen \
  --environment example/env.txt.gz \
  --out example/gplemma.out.gz

(base) [alipazok@login3 LEMMA]$ cmake --build build --target tests -- -j 4
[100%] Built target tests
(base) [alipazok@login3 LEMMA]$ ctest --output-on-failure
Test project /u/home/a/alipazok/sriram/LEMMA
No tests were found!!!
(base) [alipazok@login3 LEMMA]$

can you share your machine's gcc and cmake versions?

I'm currently compiling on a Mac with Clang, but I'll retry with gcc.
Sorry - you need to cd into the build directory for the unit tests.. I have updated the original snippet.
Logs in case helpful:
cmake.log
CMakeCache.txt

I run ctest in build dir, the output is attached.
ctest.txt
In general, It seems there are some issues with different version of gcc and regular expressions.

have you had a chance to run with gcc?  the problem is in line 448 of  parse_arguments.cpp where you are calling regex_replace

I managed to compile locally with GCC 11.3.0, but unfortunately could not reproduce the issue. I think the best way to proceed is just to get some CI running with a couple of compilers..  I’m working on this now but it may take some time as LEMMA is a weekend only project for me.
As a temporary solution to unblock your work, you could just comment out lines 448-457. The use of std::regex is not important - it is just there to infer the number of VB iterations that have passed for a LEMMA checkpoint. If you actually need to restart from a checkpoint then it is possible to set the number of iterations manually via —vb-iter-start. However I believe you are just interested in using gplemma? In which case the regex is irrelevant for you.

Thanks! we managed to run the software !

Great! Thank you for letting me know. I am still (slowly) iterating towards more robust testing / a cleaner install so I will leave this issue open for now.

Just for future reference - I have done some work in the latest release to make compilation cleaner, and also switched on CI to double check that it builds properly / tests pass on linux. To my knowledge this is now resolved, so I'm closing the issue.


Error executing "npm run devserver"

[436] ./~/highlight.js/styles/default.css 873 bytes {0} [built]
[437] ./~/css-loader!./~/highlight.js/styles/default.css 2.94 kB {0} [built]
[438] ./~/angular-highlightjs/build/angular-highlightjs.js 9.8 kB {0} [built]
[439] ./lib/syntaxHighlighter/syntaxHighlighter.filter.coffee 355 bytes {0} [built]
[440] ./lib/syntaxHighlighter/syntaxHighlighter.config.coffee 245 bytes {0} [built]
[441] ./lib/init/init.module.coffee 104 bytes {0} [built]
[442] ./lib/init/init.service.coffee 417 bytes {0} [built]
[443] ./lib/views/ion-menu.html 1.11 kB {0} [built]
[444] ./lib/config/ionic.config.coffee 3.15 kB {0} [built]
webpack: bundle is now VALID.

Where is the error? Try removing the node modules and re install them

when I run the " npm run devserver" command.
reaches that liniea and shows " webpack . bundle is now VALID "
then I go to the browser and want to see the application and not the sample
sorry for my English...
attached image

I'm not changing anything, I'm just trying to run the application as the default.

Go to the console tab to see the error

i found using sudo npm cache makes a big different

closing for lack of activity


use vote_description for all displayed descriptions

The search doesn't give the same result as the vote page, e.g. search minimum wage. Everything we show should just use vote_description.

I left the waterfall in place because vote_description is the first field. The problem before was that downloadVotes wasn't even getting vote_description before so the waterfall was going to other files on the vote page.


Update to Xcode 10

What's new in this PR?
We update the framework to compile with Xcode 10.0.

Migrate to Swift 4.2

Note
Do not merge until Jenkins uses Xcode 10 as well.

Actually not necessary.


Best Practice for Logger using Guice Injection

I have read the guice documentation and its recommendation to avoid static injection.
However, usually Logger instances are initialized as static final - as it makes sense to retain them per class and references to the logger instances remain unchanged till end.
Is there an alternate mechanism if requestStaticInjection is not recommended?
public class Application {

 @InjectLogger
  private static Logger logger;

 @Inject
  private static Application application;

 public static void main(String[] args) throws Exception {
        Module loggingInjector = new AbstractModule() {
            @Override
            protected void configure() {
                bindListener(Matchers.any(), new Log4JTypeListener());
                requestStaticInjection(Application.class);
            }
        };
        Injector injector = Guice.createInjector(loggingInjector);
        logger.info("Succesfully initiallized Logger");
    }
}

Initially I was getting null pointer exception for the logger instance since I have not assigned requestStaticInjection for the class. After adding this, logger instance is initialized properly. But I hope it is not good idea to keep requesting for all classes which uses
@InjectLogger
static Logger logger

What is the best practice for such Logging scenarios?

You probably shouldn't use dependency injection for your loggers (we generally don't). There isn't a one-size-fits-all answer here; I'd suggest posting on Stack Overflow (or a sibling site) for such questions. An issue tracker isn't a great mechanism for best practice requests.


[FEATURE] Intercalary month weekday settings

Some settings and calendars may have intercalary days not have a weekday as such, and be "outside" the standard weekday calculation (e.g. Sunday, Monday, Tuesday, IC MONTH DAY 1, Wednesday, Thursday).  I would like this to be an additional option for Intercalary months.
Currently, I have to manually set the starting weekday for every month to get this effect for my Intercalary months.

Intercalary months shouldn't affect the day of the week subsequent months fall on.
I will look and see if this may be a bug that was introduced.

From all my testing this works as expected.
Month 1 ends on a Thursday
Month 2 is an intercalary month.
Month 3 starts on Friday.
So perhaps I am misunderstanding the issue or feature you are asking?

This is what I'm seeing.  My months are 30 days with 6-day weeks, so every month should start on Sowersday, but I have to set the first month to Sowersday to guarantee that for some reason:

And an example of the intercalary day breaking the flow:

and the config for these three months:

It looks like this may be a side-effect of using "Include Intercalary Month in Total Day Count".  When I unchecked that for all Intercalary months, they returned to the expected behavior.

Ah yes I may need to clarify that setting better.
It is basically means that the days in an intercalary month affect week days for other days as they are counted when doing that calculation.
It really just makes an intercalary month behave like a normal month but be listed as an intercalary month. Its kind of an odd setting haha

I understand now.  I thought it was going to be for "days elapsed/days remaining" calculations for future features or something like that.  Feel free to rename this issue for clarifying the setting instead.

Will do, sorry for the confusion!


[bug] hide taskbar icon on macos side is not working

Describe the bug
Hello developers, the property (skipTaskbar) and method (setSkipTaskbar) for hiding the taskbar icon in macos system are invalid when passed to true or false, and the corresponding method (set_skip_taskbar) is also invalid when written in rust. I don't know if windows is working because there is no windows available for testing! I hope you guys can solve the problem!
Reproduction
If the value of the skipTaskbar property of the windows configuration window in tauri.conf.json is true or false, the effect of hiding is not achieved, and then calling the setSkipTaskbar method of the api will not help either
Expected behavior
I want to hide the icons in the macos dock bar, and of course the same effect for windows
Platform and versions
Environment
› OS: Mac OS 12.4.0 X64
› Node.js: 16.14.1
› npm: 8.5.0
› pnpm: 7.7.1
› yarn: 1.22.17
› rustup: 1.25.1
› rustc: 1.62.1
› cargo: 1.62.1
› Rust toolchain: stable-aarch64-apple-darwin
Packages
› @tauri-apps/cli [NPM]: 1.0.5
› @tauri-apps/api [NPM]: 1.0.2
› tauri [RUST]: 1.0.5,
› tauri-build [RUST]: 1.0.4,
› tao [RUST]: 0.12.2,
› wry [RUST]: 0.19.0,
Stack trace
No response
Additional context
No response

skip_taskbar does not work on macos, not a bug, but "expected" behavior. Also, skip_taskbar is window-specific which wouldn't have an effect on macos, since the dock icon is app-specific (i don't know how to explain it, so just ignore that if you don't understand me)
See https://github.com/tauri-apps/tauri/issues/2358 and https://github.com/tauri-apps/tao/issues/105 for the relevant macos feature requests.

For anyone who comes to this issue through googling, Tauri has already implemented this feature:
See https://github.com/tauri-apps/tauri/blob/8fd79b8fc0b1dc66a06af3b7131ae895808f8ea4/core/tauri/src/app.rs#L731

To clarify, after few trial/error (I'm not familiar with Rust at all) I found this way to hide the default Docker icon and only show the one on the system tray (top nav):
fn main() {
  let tray_menu = SystemTrayMenu::new(); // insert the menu items here
  let system_tray = SystemTray::new().with_menu(tray_menu);
  tauri::Builder::default()
    .setup(|app| {
      // Make the docker NOT to have an active app when started
      app.set_activation_policy(tauri::ActivationPolicy::Accessory);
      Ok(())
    })
    .system_tray(system_tray)
  // ...

To clarify, after few trial/error (I'm not familiar with Rust at all) I found this way to hide the default Docker icon and only show the one on the system tray (top nav):
fn main() {
  let tray_menu = SystemTrayMenu::new(); // insert the menu items here
  let system_tray = SystemTray::new().with_menu(tray_menu);
  tauri::Builder::default()
    .setup(|app| {
      // Make the docker NOT to have an active app when started
      app.set_activation_policy(tauri::ActivationPolicy::Accessory);
      Ok(())
    })
    .system_tray(system_tray)
  // ...

I apologize for getting back to you so late, it does work, thanks!

For anyone who comes to this issue through googling, Tauri has already implemented this feature:
See
https://github.com/tauri-apps/tauri/blob/8fd79b8fc0b1dc66a06af3b7131ae895808f8ea4/core/tauri/src/app.rs#L731

Thank you. Indeed.

For anyone who comes to this issue through googling, Tauri has already implemented this feature:
See
https://github.com/tauri-apps/tauri/blob/8fd79b8fc0b1dc66a06af3b7131ae895808f8ea4/core/tauri/src/app.rs#L731

Thank you. Indeed.

Sorry, where is that and how do you use it? I haven't been able to find it in the docs:


Update INSTALL.md to include local development instructions

I have built my gcp-stack image following the INSTALL.md
But it ends at Install part with TBD. I also tried out installing this using the crossplane repo's way (by running scripts under cluster/local/minikube.sh up) but there are not scripts under cluster/ folder.
Will running the built image as deployment do or is there any canonical/recommended way to install gcp-stack in my minikube?

@vadasambar sorry for just getting to this! You will need to install crossplane before installing stack-gcp (recommend the instructions here). If you want to develop locally and you have your kubeconfig set, you can simply run the stack directly go run cmd/stack/main.go after installing crossplane. Also, the minikube script you are looking for is here. Feel free to reach out in Slack if you have any trouble and we can walk through it there :) I am going to update this issue to reflect that INSTALL.md needs to be updated.

Hey @hasheddan , thank you for the reply!
I had crossplane installed on my minikube using the installation guide. I was stuck at stack-gcp installation.  I will try out running the stack on the cli.

Hi @hasheddan, do I need to install the CRDs before running the stack directly using go run cmd/stack/main.go?

@vadasambar Yes! If the stack is installed via a stack package then the CRDs will be installed for you, but if running the controller manager directly locally then you must manually install the CRDs.


egeria-ui --server.servlet.context-path option

In case user wants to specify --server.servlet.context-path option, ui.yaml needs to be modified to modify container's entrypoint command.  Probably better would be to propagate  this into values.yaml as an option, so user would need just to specify an optional correct value in values.yaml. The change needs to be reflected in ingress.yaml as well.

@sarbull This was originally brought up in relation to hosting the spring based UI.
Given the refactoring that will occur in that UI shortly I think it's best you evaluate if any changes are needed in the k8s deployment after those changes are made specific to this issue

@planetf1 we'll have to see at PR, i think some java spring ENV var will be removed (the required zuul ones), after this the backend it's all by itself with no link to the UI, the UI will be refactored so that it knows about the backend with the default deployment strategy we discussed by adding that ENV file which will provide minimal info for the THEME and API_URL and maybe other properties..
other than this, having the UI completely separated from the backend, anyone can decide what fits best for their needs as deployment

Given the refactoring it may be appropriate to close this issue now, and just ensure relevant docs are in place for deployment after the refactoring -


fix(codebuild): add validation for source when badge is enabled

Badge should not be allow to set to true if source is not GITHUB, GITHUB_ENTERPRISE or BITBUCKET
Fixes #1749

Pull Request Checklist

[Y] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[Y] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated

[Y] Title and Description

Change type: title prefixed with fix, feat will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[N] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

LGTM; @skinny85 I'll let you merge


Enable compatibility with insight-api and bitcore

Added bitcore-node files for proper compatibility with insight-api/bitcore.
Added .jscsrc and .jshintrc files to enforce better code quality

:shipit:


Dockerize kweb

@sebastian-goeldi @joamatab,
with regard to our discussion in the las developer meeting:

It seems to work just fine to run kweb inside a docker container and stream the gds out over websocket to the host computer. It actually also works on codespaces, and even on docker containers inside codespaces (container in container) as seen in the screenshot.
Or did I misunderstand you during the meeting @sebastian-goeldi?
Regards JD

If you run all of it in docker, then yeah, it will work.
What probably doesn't work easily is the following:

gdsfactory in docker
klayout with klive on the host

In that case you have to do the inverse: go from docker to the host system. I am not sure you can do that without a special docker config

If you run all of it in docker, then yeah, it will work.
What probably doesn't work easily is the following:
* gdsfactory in docker

* klayout with klive on the host

In that case you have to do the inverse: go from docker to the host system. I am not sure you can do that without a special docker config

Is there a special reason why we would want to run klayout on the host?

Is there a special reason why we would want to run klayout on the host?

Are you volunteering to support & maintain the graphical forwarding to/from docker for KLayout? :)

I am not quite sure of the use case: do we ever want to look at the klayout gui itself, or do we rather use kweb to avoid having to stream the GUI out of the container?

I mean  kweb is just not powerful enough to substitute the KLayout GUI. So unless your use case is purely to use gdsfactory in combination with kweb, we need to support other possibilities too.
I mean the use case where you need KLayout is when you want to traverse the hierarchy in the GUI for example. kweb currently has no possibility for this. And I am not really well versed in web developlment, so I don't really want to spend tons of time one this, as my current usecases for kweb are quite limited.

Could it be an option to have an instance of klayout running inside the container to do the kweb streaming (as off right now) and if you want to look at a layout in more details you spinn up an instance of klayout on your local machine also?

I fully understand the benefit of having a docker container available. My point was more: This was not the main issue in the last meeting ;). It was about having a one cllick solution to deploy gdsfactory on a system. This is not it, as in it also needs to have a way of interacting with klayout. We can do it in docker, but graphical forwarding (especially platform agnostic) is not that easy. So the way to do it is either make the gdsfactory docker container able to stream to the host system (I think also not straight forward) or do everything in docker. And at that point it might be easier to just offer a VM image.

I'll do a bit more digging :)

When using the correct port (https://github.com/gdsfactory/gdsfactory/pull/1569) it also works fine with the jupyter integration

Do I see it correctly, that if we could provide the gds file from within the docker container to klive on the host machine so that we can view it in our local KLayout than everything is sorted? (and easily install all that with a single click)...

No, you not only need to provide the gds (that's easy, just expose your target volume to the host). You need to connect to 8082 (the klive port, I think it's 8082) port of the host.

So the way to do it is either make the gdsfactory docker container able to stream to the host system (I think also not straight forward) or do everything in docker. And at that point it might be easier to just offer a VM image

Sort of second this tbh. If I were a brand new user with little coding experience, I would find it nicer to just download an iso and run it on VMWare or smth. I've been strugling to get Docker running on Codespaces and running it on Windows, I couldnt even  open the app without adding some user perms through cmd (unless Im running it incirrectly?).

This looks great!
thank you Jan-David

Personally I had bad experiences with VMs, but maybe that's just because my pc isn't beefy enough...


Include version number

Hello,
it would be nice to have the version number for the installed package available. I see that the Package.resolved file also includes a version, depending on the installation method.

Added in 0.4.0

Thank you very much :)


#1045 Fixed port forwarding to pod

Updated logic for target port determination in case of port-forwarding to service and target port on the pod is different.
Test result:
--- PASS: TestTunnelOpensAPortForwardTunnelToPod (9.87s)
--- PASS: TestTunnelOpensAPortForwardTunnelToService (12.82s)
--- PASS: TestTunnelOpensAPortForwardTunnelToServiceDifferentPodPort (18.11s)

Closes: https://github.com/gruntwork-io/terratest/issues/1045

@denis256 are you still working on this?


Updating UI to reflect administrator policies do not override

NIFI-2475:

Updating UI to better suggest that component specific administrators do not override higher level administrators.

+1 LGTM, merging


Github stable release/tag system over all repositories

Goal
We want to be able to check out the complete set of repositories (frontend/middleware/backend/users/accounting) at a state in which we are sure they are compatible.
Acceptance criteria

[ ] Checkout process is documented (see #12)

Out of scope
Implementation notes

For the moment, we can trial science-gateway as being the main repo for synchronisation. We then add git submodules for each component.
We can keep a version number tag in the parent science-gateway repository, and then go through and manually add this tag to each sub-repo.
When you want to run a demo of the entire project, you should be able to do the following:
git clone --recursive git@github.com:alan-turing-institute/science-gateway.git
docker-compose up

We should follow semantic versioning in git tags: https://semver.org/
v0.1.0-alpha.1
v0.1.0-alpha.2
v0.1.0
v0.1.1-alpha.1
...

We could add .beta. and .rc. (release candidate) if needed in the future.


fix: Correct bank codes for Cypriot banks

Fix bank codes for the following banks:

ASTROBANK PUBLIC COMPANY LIMITED
BANK OF CYPRUS PUBLIC COMPANY LIMITED
HELLENIC BANK PUBLIC COMPANY LTD

@mdomke fixed some bank codes for CY


Improve the Travis setup

Test with PHP 5.3 again
Store the Composer and simple-phpunit cache
Test with lowest and dev versions

Closing as kinda outdated, sorry this got lost


Missing types in type-graph.txt

The problem
The following types are missing from type-graph.txt:

[ ] class CompUnit
[ ] role PredictiveIterator
[ ] enum Endian
[ ] class Telemetry (you need to use use Telemetry to instantiate it)
[ ] class Telemetry::Period  (you need to use use Telemetry to instantiate it)
[ ] class Telemetry::Sampler  (you need to use use Telemetry to instantiate it)
[ ] role Distribution
[ ] class Distribution::Locally
[ ] class Distribution::Hash
[ ] class Distribution::Path
[ ] class Perl
[ ] class Collation
[ ] class Encoding
[ ] class Encoding::Registry
[ ] class Compiler
[ ] class UInt (it's a subset)
[ ] class Cursor
[ ] class Pod::Defn
[ ] class CX::Done
[ ] class CX::Return
[ ] class CX::Emit
[ ] class Routine::WrapHandle
[ ] class Lock::Async
[ ]  class Telemetry::Instrument::ThreadPool

Suggestions
Add them.
Related to #2945 .

This extends #2302


Link to pages from empty campaign message is broken

Link within message is broken

from

Fixed with https://github.com/silverstripe/silverstripe-framework/commit/84c0df3db0578ea726314733c3b91ae0080f98d8


refactor: getConnectionForEntity()

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

3 of 3 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage remained the same at 97.394%

Files with Coverage Reduction
New Missed Lines
%

src/entity/utils/getConnectionForEntity.ts
1
92.31%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
252

Relevant Lines:
258

💛  - Coveralls

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

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


attempt to avert disaster issue 246

Coverage remained the same at 97.706% when pulling 14fd48317d3458b917855bc1b6f833fafa8c93a4 on realtymaps:1.X into 93db5f5fd3a05c580b891c41771dc14c1ca585ea on floridoo:1.X.

@floridoo @zacronos anyone some review is appreciated


Is Expect extensible?

Hello, thanks for the bindings!
I've looked into the code and as far as I can see, the Expect module creates jest assertions via assertion variant. This makes the API clean, but doesn't really allow users to introduce custom matchers. E.g. we use jest-dom and testing-library/jest-native and we weren't been able to properly cooperate the custom matchers with Expect module.

Hello, thanks for the bindings! I've looked into the code and as far as I can see, the Expect module creates jest assertions via assertion variant. This makes the API clean, but doesn't really allow users to introduce custom matchers. E.g. we use jest-dom and testing-library/jest-native and we weren't been able to properly cooperate the custom matchers with Expect module.

Same problem here.
I had to vender this library and add the extra needed matchers.
Would love to know if the author recommends a solution better than forking the library and adding the matchers.
Anyways thanks for this work.

It should be possible to create a custom asserter and awkwardly re-exposing and forwarding the original assertions to the default asserter. But it wasn't really designed for extensibility, even if that's long been desired. It would likely require a complete rewrite of the internals, and I don't have time to do that. But I would certainly welcome efforts to do so while at the very least retaining the design philosophy, if not the API.


[UPDATE] Pagination

Figma Link
https://www.figma.com/file/R7ymwKrEI3LF6ssrZe5opG/Desktop-UI-Kit?node-id=1641%3A0
Additional Context
Only for base Pagination & PaginationDetail. PaginationFilter will be tracked on a separate issue.
Tasks

[x] Restyle pagination to follow new theming.
[x] New PaginationDetail component. (Showing xxx-xxx out of xxx)

@DioSyahrizal udah bikin style pagination terbaru di business dashboard; coba di review dan di merge @resir014

Check Figma design ya. Overall, about color, state, and layout, do not refer to Figma.

Border radius using 8px
Missed hovered state
Missed item per page feature
Pagination only has one variant. please recheck Figma,

Need Adjustment

[ ] Hover pagination button style
[ ] Space 4px


Update WebAssembly testsuite

This PR updates the WebAssembly testsuite to use all of the latest tests. It highlights four failures, I think:

elem.wast
function_references/data.wast
function_references/elem.wast
function_references/global.wast

Not sure how critical these are to fix but I thought I would point these out in a draft PR.

Here's where the errors start: https://github.com/bytecodealliance/wasmtime/actions/runs/5490314463/jobs/10005600074#step:16:3205.

I think these failures are related to wasmparser and its validation, so the test suite submodule probably will need to get updated in the wasm-tools repository before it's updated here.

With https://github.com/WebAssembly/testsuite/pull/68, https://github.com/bytecodealliance/wasm-tools/pull/1126, and publishing wasm-tools this should be good to go.

Ok, rebased on top of #6739.

Oh can you also try re-updating the submodule its latest? I think that will surface a single error in tests/spec_testsuite/elem.wast:690 which I think is something we need to implement, but otherwise the other failures I think are fixed by updating the test suite

Ok I think those are legitimate failures where we accidentally don't support global.get in table element segment offset initializers.

It seems like global.get has been a constant expression at least since 2017 (WebAssembly/spec#445). However it was not usable inside element segments until the reference types proposal landed in 2021 (WebAssembly/spec#1287). This made funcrefs/externrefs available as globals, and thus global.get became usable in element segments.
Unfortunately the interaction between this proposal and element segments was initially missed and a test was not added for it until earlier this year in WebAssembly/spec#1640. I think most wasm parsers missed this detail. For instance wabt only added support for this today in WebAssembly/wabt#2288.

I've "rebased" this and implemented the missing features from Wasmtime in https://github.com/bytecodealliance/wasmtime/pull/7996


Can't use scope variables inside a popover/tooltip TemplateReference

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-scmfqv
Steps to reproduce
Create a tooltip inside an *ngFor and try to access the declared variable from inside the ng-template
What is expected?
The scope variable should be accessible and rendered inside the ng-template used by the popover/tooltip
What is actually happening?
The variable is undefined

Environment
Info

ng-zorro-antd
7.4.1

Browser
Chrome

It is not a problem of ng-zorro, angular does not support this:

  <ng-container [ngTemplateOutlet]="popoverVarsContent"></ng-container>

<ng-template #popoverVarsContent>

Item: {{item | json}}

</ng-template>

You should create a new component and use @Input.

@javcode You can move ng-template to the loop body.

  <ng-container [ngTemplateOutlet]="popoverVarsContent"></ng-container>
  <ng-template #popoverVarsContent>

Item: {{item | json}}

  </ng-template>

I've ended up creating a new Component taking the 'item' as an input as I wanted to be able to re-use this code as it's used in several places.
export class ImportPopoverComponent implements OnInit {

  @Input()
  item:{}

  @Input()
  type:string

}

Now the new component html looks like:
Add Item

<ng-template #popover>

{{item}}

</ng-template>

and I call it from the parent using:
<app-import-popover [item]="item"></app-import-popover>

Thanks wenqi73 for the work-around

@hsuanxyz Yeah, Angular can use like this, another way is context: {$implicit: item}. I explained incorrectly before.

@javcode , @hsuanxyz 's work-around is more convenient.

@javcode You can move ng-template to the loop body.

  <ng-container [ngTemplateOutlet]="popoverVarsContent"></ng-container>
  <ng-template #popoverVarsContent>

Item: {{item | json}}

  </ng-template>

Works like a charm.


Signing EULA not working

Describe the bug
on the engine tab i click on the button to sign the EULA, it's open the browser on the unreal website but doesn't do any thing, tried in librewolf, firefox so it's not a browser problem, tried to click the refresh button, nothing happening
Expected behavior
something for me to sign and then be able to download the engine
Screenshots

this webpage i get to after i click the button
Desktop (please complete the following information):

OS: Fedora 41
Browser Librewolf tried also firefox 132.0.2-1
Version 3.8.6

Additional context
Add any other context about the problem here.

Running into the same issue on Arch: The link to sign the EULA works, but after completing this step, EAM doesn't seem to be syncing with Epic to confirm that I've completed the EULA.


update ugly.json

new word added Ubiquitous
The Pull Request resolves Issue #[Issue-Number]
Description:
A brief description about the addition or edits made.
Notes: Delete all the text including this!

The #[Issue-Number] should be replaced with the specific Issue being referenced.

Hey @dibyaprusty
Add the contents of the word Ubiquitous in a separate file named Ubiquitous.json in folder U
Refer Contributions.md for more details


An abstract base class or Interface creates a new Azure Service BusTopic

When you add an interface or an abstract base class to a class that is used to publish data with MassTransit, it causes MassTransit to create an undesirable "extra" Service Bus Topic.  For example:
This is fine:
public EventData {
 public string Text {get;set;]
}

There will be one Azure Service Bus Topic created called "EventData".
However, if you add an interface:
public EventData : IHasText {
 public string Text {get;set;]
}

Then MassTransit will create an extra Topic called "IHasText".
Is this by design?  Is there a way to disable this behavior?  I think it causes some quirky behavior when using temporary queues, but I'd have to verify (I'm using a named queue).

This is by design, to support polymorphic message consumers - consumers that subscribe to IHasText will get a copy of the message when published.

What about a declarative attribute to ignore that interface or class?

Sounds confusing and complicated.

Confusing and complicated to implement?  It definitely makes looking at the queues and topics more confusing.  There's no business-driven reason to ever publish something to an IHasText consumer, so it ends up just being extra clutter in Azure Service Bus.

I'd suggest reading the documentation, and look at the examples for RabbitMQ, to understand the why.
http://masstransit-project.com/MassTransit/understand/default-topology.html
If you don't intend to subscribe to IHasText, why do you even have it in your message contract? Message contracts use types to be explicit, having unused interfaces on them just creates confusion. If your interface IHasText isn't adding value to allow message consumers to subscribe to part of a message, then get rid of it.

In our implementation our messaging objects are not a unique set of interface implemented objects. I know this is contrary to your preferred design, but we have a lot of reasons for the implementation. We have created a lot of software surrounding and extending MT.
Our classes are concrete classes that perform transport functions in a couple of different scenarios - similar to a Command pattern. We also use nesting of a "message" object wrapping an "operation" object. The underlying interface and class that is implemented is for other functionality, not receive functionality, for inheritance purposes. The change in this version of MT does change the queues created.
For this reason:

Is it possible to add suppress interface syntax of attribute for use during discovery
Do the queues for the interfaces require subscribers or will the only be send to if a subscriber is hooked up? i.e. are they like meta-data and won't impact performance if you don't subscribe to them specifically?

The shouldn't impact performance if there aren't any subscribers. And I'm not planning to add anything to suppress anything.

Understood. No worries. Thanks for considering it.
From: Chris Patterson<EMAIL_ADDRESS>Sent: Tuesday, April 30, 2019 9:18 PM
To: MassTransit/MassTransit<EMAIL_ADDRESS>Cc: brianro<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [MassTransit/MassTransit] An abstract base class or Interface creates a new Azure Service BusTopic (#1430)
The shouldn't impact performance if there aren't any subscribers. And I'm not planning to add anything to suppress anything.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/MassTransit/MassTransit/issues/1430#issuecomment-488169525, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AA3BCDBCFEZJI2MNHYHRBNTPTDVULANCNFSM4HI2HCZQ.

Happy to see this resolved in the latest 7.0.4 release.

Oh yeah @joshmouch would
Be happy.
Should have tagged this issue in the commit.


Java 1.1: Need of Java #1105

Description 📜
Done Audio Explaining the Need of Java #1105

Type of change 📝

[x] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)

Domain of Contribution 📊

[x] Java

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Need Of Java #1105  : Adding Need Of Java audio in .mp3 format

@Sneha-0308 You had done the PR on the wrong branch. Please do the PR on Java branch. Currently it had been done on the main branch. Right now, there is no other way to go ahead without closing this PR. So, I am closing this PR, and please make a new PR to the java branch. If you have doubts, please tag me in discord, I will guide on how to do the PR perfectly. Peace ✌


Remove deprecated exprs

Fix for https://github.com/CartoDB/carto-vl/issues/696
TODO: replace removed expressions from the editor examples

:+1:


Activating device does not work

I have 2 devices for Spotify in my network. Both of them can be selected via the Spotify app and then the playback via the adapter works. However, when the devices go into sleep mode and I use the "useForPlayback" button functionality in the devices section of the adapter, I still get an error that "no active device can be found" when I start the playback via the adapter. When I then select the device manually in the Spotify app also the playback via the adapter works.
This is only reproducible when the device is sleeping, otherwise the useForPlayback works as expected.
Devices are both Soundtouch if it helps.
Is there any mechanism to wake up the devices via the adapter and thus simulating the functionality of the Spotify app?

I have noticed that it seems to work when the parameter play is set to true in the call
https://api.spotify.com/v1/me/player


[REVIEW] Portfolio and project

Portfolio link:
https://rathod-sahaab.github.io
Project:
repo:
https://github.com/elide-app
deploy
https://console.elide.me

Thanks for sharing :) I liked your profile personally maybe improve the contrast a bit and the language filter function is not working in the project section. Good job btw

Thanks for sharing :) I liked your profile personally maybe improve the contrast a bit and the language filter function is not working in the project section. Good job btw

Thanks for the feedback I made filtering functional ❤

Maybe consider using a rss feed or something like this to implement blog posts directly into your website
maybe add some padding at the bottom of the page in the skills and projects section

Thank you for sharing, we will review on a live stream soon 👍

Really nice design, just a few small things I noticed:

under skills don’t be negative about JavaScript,
Under skills there is are small typos in asynchronous, written, structure, separation
On the contact page the Twitter logo is blue but all others match your colour scheme

Link no longer works

Link no longer works
@eddiejaoude
https://abhay.rs


Add support for env argument

Eg. if you want to replace secrets in .condarc:
with:
  env:
    MY_SECRET: ${{ secrets.my_secret }}

I'm sure this already works...?


[Feature]: global value for split

Feature Request
It would be great if I could set the split value for all relevant nodes in one place. Maybe by passing it a context node?
Usually, while building a graph out, I want to keep it to like 2 test cases, but when that's done it would be nice to open the floodgates without having to change the split value in every place.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

We could allow a variable reference in the "Max" field, instead of just a hard-coded numeric value.


SaveBulkStateAsync() throws context deadline exceeded exception

Ask your question here
Hello,
I'm using Azure Cosmos DB state store and the SaveBulkStateAsync() to save bulk records, but an exception (context deadline exceeded) is thrown when the call takes more than 20 seconds(ish).
From cosmosdb.go code, it looks like there is a default timeout of 20 seconds.
All of the functions (Get, Set, Delete, ...) except BulkGet are using defaultTimeout to set the context timeout.

Is that the reason why my call fails?
Does that mean all the Azure Cosmos DB state store calls timeout after 20 seconds?
Is there any way to change this timeout?

Thank you!

@koumdev It looks to me like it's a hardcoded 20s timeout in the component itself.  I'd file an issue in dapr/components-contrib and ask these questions.  In any case, it's unlikely to be specific to the .NET SDK.

Closing this in favor of the components-contrib issue.


About flymake-flycheck

This plugin uses eglot and clangd to report the following errors and cannot be checked properly:
(fn REPORT-FN &rest _)] in ‘flymake--state’
error in process sentinel: Can’t find state for #[385 \300\242\203\0\302\303\301"\210\304\300\242!\210\300\305\240\210\302\306\301"\210\300\307\310p\311\301\312\305\313\314\301!&\240\210\315\300\242\316\317\301#"\207 [(#s(flycheck-syntax-check dwl.c c/c++-clang flycheck-c/c++-clang /home/xiliuya/git/)) c/c++-clang flymake-flycheck--debug interrupting defunct syntax check for %s flycheck-syntax-check-interrupt nil start syntax check for %s flycheck-syntax-check-new :buffer :checker :context :working-directory flycheck-compute-working-directory flycheck-syntax-check-start make-closure #[513 \302\303\301#\210\304\267\2021\0\300\305\306\307\310\301##\207\300\311\312\313\301""\314edB#\207\302\315\301#\207\302\316\301#\207\302\317\301#\207 [V0 V1 flymake-flycheck--debug received status %S from %s #s(hash-table size 4 test eq rehash-size 1.5 rehash-threshold 0.8125 purecopy t data (errored 12 finished 23 interrupted 37 suspicious 43)) :panic :explanation format Flycheck checker %s reported error %S mapcar apply-partially flymake-flycheck--translate-error :region checker %s reported being interrupted %S checker %s reported suspicious result %S unexpected status from checker %s: %S] 9

Comment out these lines:
init-flymake.el : 7-22 (when (maybe-require-package 'flymake-flycheck)

Sorry, I didn't quite understand what's wrong here. How would I reproduce this?

Sorry, I didn't quite understand what's wrong here. How would I reproduce this?

I added the configuration in init-local.el.
(add-hook 'c-mode-common-hook 'eglot-ensure)
Then close all the c files, restart emacs, open C file will pop up this error.

Sorry, I didn't quite understand what's wrong here. How would I reproduce this?

Is my configuration incorrect?
But even if I don't use eglot. If I don't comment out the above lines (flymake-flycheck).
Opening the c project *Flymake diagnostic gives an error similar to the following:
The test project is xv6-riscv
   1   9 error    (        'kernel/types.h' file not found [c/c++-clang]

Sorry, I didn't quite understand what's wrong here. How would I reproduce this?

I found that the value of flymake-diagnostic-functions is different
flymake-check does not open:
Its value is (eglot-flymake-backend)

When flymake-check is on:
Its value is
(eglot-flymake-backend #f(compiled-function
                          (report-fn &rest _)
                          #<bytecode 0x5319205130afeb4>))

Yeah, I guess it's possible that if some flycheck backends are enabled, then they will be used via flymake-flycheck alongside eglot. I'll have a think about how to tackle this. In the meantime you can probably add unwanted flycheck backends to flycheck-disabled-checkers.

Yeah, I guess it's possible that if some flycheck backends are enabled, then they will be used via flymake-flycheck alongside eglot. I'll have a think about how to tackle this. In the meantime you can probably add unwanted flycheck backends to flycheck-disabled-checkers.

It's very effective. Thank you for your help.
 (setq-default flycheck-disabled-checkers
                  (append (default-value 'flycheck-disabled-checkers)
                          '(c/c++-clang c/c++-gcc c/c++-cppcheck)))

Thanks for confirming that this helped you!

I also want to disable c/c++-clang and enable eglot-flymake-backend. I tried to put in init-local.el
(with-eval-after-load 'flycheck (add-to-list 'flycheck-disabled-checkers 'c/c++-clang))

as suggested in the link.
However, it does not work unless I delete (add-hook 'flymake-mode-hook 'flymake-flycheck-auto) in init-flymake.el and add it after (add-to-list 'flycheck-disabled-checkers 'c/c++-clang) in init-local.el.
I wonder if there is any way to reconcile without modifying anything in init-flymake.el. I want to keep things synchronized with you and avoid corruption from my changes.
Another things is that after I disable c/c++-clang in a dirty way, the backend eglot-flymake-backend cannot be enabled automatically. Executing M-x flymake-running-backends gives me nothing. I have to execute M-X eglot in the buffer manually and eglot checker got invoked. Is there suggestions to overcome this step and make things consistent with flymake-flycheck? I tried to add
(add-hook 'flymake-diagnostics-functions 'eglot-flymake-backend nil t)

and it does not work.
Many thanks!

@yszhang95 Thanks for the info. What do you mean by "the backend eglot-flymake-backend cannot be enabled automatically"? Do you mean when eglot is active in a project, and you open a new buffer, for example? I'm interested in how to reproduce this issue step-by-step specifically, because I'd like to fix it.

Thank you for the quick reply @purcell !
When I tried to prepare for a minimal example to reproduce, I found I have to include
(add-hook 'c++-ts-mode-hook 'eglot-ensure)
(add-hook 'c++-mode-hook 'eglot-ensure)

Sorry for the misleading question. I will update my last comment to avoid misleading others.
The remaining question is the first one, how to avoid corrupting your setup.
Best regards!


Errors while compiling OPAE-Intel-FPGA Drivers

Attempting on Ubuntu-21.10 LTS
Facing the below errors,kindly suggest any solution
/home/kn/opae-intel-fpga-driver-0.9.0/drivers/fpga/intel/fme-pr.c:326:28: error: macro "access_ok" passed 3 arguments, but takes just 2
326 |         port_pr.buffer_size))
|                            ^
In file included from ./include/linux/uaccess.h:11,
from /home/kn/opae-intel-fpga-driver-0.9.0/drivers/fpga/intel/fme-pr.c:25:
./arch/x86/include/asm/uaccess.h:69: note: macro "access_ok" defined here
69 | #define access_ok(addr, size)
|
/home/kn/opae-intel-fpga-driver-0.9.0/drivers/fpga/intel/fme-pr.c:325:7: error: ‘access_ok’ undeclared (first use in this function)
325 |  if (!access_ok(VERIFY_READ, port_pr.buffer_address,
|       ^~~~~~~~~
/home/kn/opae-intel-fpga-driver-0.9.0/drivers/fpga/intel/fme-pr.c:325:7: note: each undeclared identifier is reported only once for each function it appears in
make[2]: *** [scripts/Makefile.build:281: /home/kn/opae-intel-fpga-driver-0.9.0/drivers/fpga/intel/fme-pr.o] Error 1
make[1]: *** [Makefile:1879: /home/kn/opae-intel-fpga-driver-0.9.0] Error 2
make[1]: Leaving directory '/usr/src/linux-headers-5.13.0-30-generic'
make: *** [Makefile:38: all] Error 2

As mentioned in issue #2494, the opae-intel-fpga driver is no longer supported. Please use the latest DFL driver instead.


fix: add responsive styles to pagination for small screens

Ticket
Motivation and context
On small mobile screens the pagination would break into two lines. This PR adds some adjustments to pagination styles based on breakpoint.
Before

After

How to test
[Add a deep link and instructions how to verify the new behavior]

I checked with them in initial talk about it, but that would require different approach to mobile design of pagination (at least that was their idea). I think that is a stretch for now, since that's a lot of new listings to be added for 5 digit pagination.


Rewrite Dokka-generated markdown to include titles.

I tried rendering in jekyll mode but that doesn't look as nice with
Mkdocs.

Worth it!


We need a license

Before we can release anything, we need to choose a license. I like the MIT license or the BSD license, so that anyone can do whatever they want with the code.
@rbmetcalf Do we have any restrictions on the terms under which we can distribute code?

@rbmetcalf Can I add the MIT license to the project?


Add Grid Intensity CLI Docs

Add an initial set of documentation introducting and explaing the Grid Intensity CLI.
This PR will include:

[ ] Overview
[ ] Installation
[ ] Explainer: Information about providers
[ ] Tutorial: Getting grid intensity information for a region
[ ] Tutorial: Exporting grid intensity data

 Go CLI - Overview

thanks for this Fish! Can you @ message me when you'd like me to review this for merging in?
And is it ok to merge in myself if I'm happy?

@mrchrisadams this PR is ready for review. The first set of docs for the Grid Intensity CLI can be previewed at the below branch preview URL:
https://add-grid-intensity-cli-docs.developer-docs-64x.pages.dev/
No worries if you want to merge changes yourself, or if there's feedback you can leave it here.

hi @fershad  I'm happy with this - there's just one typo which I've highlighted, can you please merge this in after making that one change?


Creation of a Logo

Here's another issue, aimed at beginners.
Here I'm looking for a logo for the website. The name of the store would be 'Dynamic Technologies'.
And I'd like 'Hardware Data Recovery' to be highlighted somehow in the logo as well.
As usual, you're free to use any tools, any framework.
Time to get creative :)
Good luck!

if someone is not assigned , can you assign to me .

@IzacVrushabh This is an open running issue, you don't need to be assigned to work on it. I'm accepting multiple logos from everyone.


[AutoRelease] t2-redis-2022-08-31-22412(Do not merge)

https://github.com/Azure/sdk-release-request/issues/3130
Live test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1022

issue link:https://github.com/Azure/sdk-release-request/issues/3130


Execution failed for task ':shake_flutter:compileDebugKotlin'

Hi, shakebugs team.
Currently, I'm using the latest version of the shake_flutter package (16.2.1). However, I must update my Android project, including the Java version, to 17 because most dependencies must be updated in the pubspec file. The problem is that I get an error when I update. This is the error below:

Have you thought of updating the version to support Java 17?
Thanks for the package's support 👍

To continue this issue, I have made a fork to update the Java version to 17. Also, I set the dependencies values for Android and iOS to get the dependencies because I don't get the environment variables.
This is the fork link if anybody also needs it.

Thanks @FabianVarela , we'll definitely check this and your forked repo and probably include code in a new version.

Thanks @mstanic-shake


Issue 25352 dryrun testing section

Hey, I just made a Pull Request!
Added a new section to the documentation describing dry run functionality. Includes a guide for how to enable and add handling for dry run testing to a plugin with examples, further examples of how to write tests for dry run handling, and a description of how to run dry run testing via an API call with examples

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

resolves #25352

Hi @benjdlambert, can you give this one a look please?


Replace references to Rails with GraphQL

This replaces references to Rails with GraphQL in README.md, as they were likely a copy paste error.

Oh, I guess could copy it from somewhere 🤦♂thank you so much!


Technical question.

Hi, I have a problem that I want to solve. but I do not know if this library will help me! I have a serious data (training) that are similar to this:
http://prntscr.com/nqbyhf
I have more elements with that structure. and each one has an "associated performance" .. if it is a good set of data then the "performance" is positive .. if it is not a good data set then the performance is negative.
The idea is that by passing a new set of data the code will result in a set of data that "can be" a set with good performance or not.
For what I say I used this example http://prntscr.com/nqbz7j passing an "array" with all the data set that I have http://prntscr.com/nqbzhy and when I want to do the ".run ()" I pass this data to you. http://prntscr.com/nqbzoi
Now what I do not understand is because always ... the results are VERY SIMILAR OR EQUAL. I think they should be different .. because there may be a data set with bad configuration and a set of data with good configuration.
Look at the results: http://prntscr.com/nqc0cv

Yes it is: https://medium.com/@tech_fort/classifying-text-with-neural-networks-and-mimir-in-javascript-94c9de20c0ac


Welcome bot conflicts with pr labeler

Description

Welcome bot is not working as expected & seems to be conflicting with pr labeler.

@leecalcote  mentioned this issue in mesher-ci slack.

Expected Behavior

Welcome bot should work without any conflicts & independent of labeler.

@MukulMaheshwari it'll be good to keep an eye on this PR https://github.com/layer5io/meshery/pull/1595 that will soon merge to see if the welcome bot config is still conflicting with another config.

The issue was an invalid config.

Fixed in #1592

Much appreciated. It was painful not seeing new community members congratulated. :)


Upgrade jsonrpc

Upgrade jsonrpc beyond 14.0.5, it had some bugs fixed recently.

14.1 has been release: https://github.com/paritytech/jsonrpc/pull/545

The update is complete :tada: