:sparkles: Update logging of closed RPC errors

Solves https://github.com/konveyor/analyzer-lsp/issues/500

Thanks for your comment @shawn-hurley
In the call method, should I add it here: https://github.com/konveyor/analyzer-lsp/blob/main/jsonrpc2_v2/conn.go#L354 ? and here for Notify: https://github.com/konveyor/analyzer-lsp/blob/main/jsonrpc2_v2/conn.go#L288?
And one more question, should I do the same for the jsonrpc2 (v1) package?

Yes those locations look correct to me
and yes if you can add it in v1 as well that would be amazing

@abrugaro, we are experiencing some issues with the CI system, so while this is a low-risk change that I would normally merge on green, this might take a little longer to get in. I am sorry

@abrugaro, we are experiencing some issues with the CI system, so while this is a low-risk change that I would normally merge on green, this might take a little longer to get in. I am sorry

Sure, its my first contribution to this project so I prefer to wait to be sure that everything is ok before merging :smile:

Thank you @abrugaro this is a really helpful change!


[DC-1854] Add the list of sandbox table names to the RaceEthnicityRecordSuppression cleaning rule

Implemented get_sandbox_tablenames() for RaceEthnicityRecordSuppression and updated the integration test to use this function.

@ksdkalluri It looks like this issue is already solved by your following PR
https://github.com/all-of-us/curation/pull/1080/files
I think I should just close this PR without merging or anything. What do you think?


Handle CNI uninstall corner case in chained mode

Corner case handling - for CNI plugin in chained mode, we are adding and removing a section from a JSON array in a (shared, node-level) config file.
We read the file into memory, excise our config, and overwrite the result back out to the file.
99% of the time this is fine - but if we are chained with another node agent doing something similar, the other node agent can read/excise/write the file in between the time when we read it in and write it back out - in that case we end up writing invalid node config because we don't recheck before writing.
So, do a simple Stat check immediately before write to make sure the file hasn't changed since we initially read it. If it has, discard and retry.
This is clearly just best-effort, the actual Real, Correct way to solve this is in the CNI plugin spec itself, e.g. with something like: https://github.com/containernetworking/cni/issues/928

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
dd3c2efed14d8b269dfbfdf87fb2db7a39bb382c
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d0447d76496a8d6a59dbcc7730272b813b7abf39
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@bleggett: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
d0447d76496a8d6a59dbcc7730272b813b7abf39
link
true
/test lint

gencheck_istio
d0447d76496a8d6a59dbcc7730272b813b7abf39
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

I'm raising https://github.com/istio/istio/issues/46848 so the underlying issue isn't lost, and closing this PR (since by itself it's not an effective fix)


Data Table where a click in a row shows extra data about selected row

Hello Everyone,
I need to show in a Data Table where the data are getting to through of PHP and MariaDB. After, when I click in any row, it shows extra data about the row selected and two buttons in the section just to right of the data table.
Is it possible?
Regards friends

Due to some neglect in my part, many issues weren't addressed for some time. Some of these issues are no longer relevant since a solution might have been found. Therefore, we decided to close all issues that are over a month old. However, if the issue still persists and you'd like help, then by all means repost your issue and we will consider it again. In the mean time, we will close this post.
Thanks for your understanding and we apologize for any inconvenience <3.


adds new SlackClientError and ResponseParseError types

Summary
Adding new error types to describe errors.
SlackClientError is the base class for all errors thrown from this package. It provides a generic default message if none is provided.
ResponseParseError is an error for Web API responses which cannot be parsed as JSON. Its message includes the response body and the message for the exception it wraps.
I attempted to follow best practices according to this guide.
I could spend some time writing tests, but I wanted to get this PR submitted for code review ASAP.
FYI: The new error types are in a new namespace called .exceptions. The existing custom error types SlackConnectionError and SlackLoginError should be moved under this namespace, but cannot be moved without bumping the major version number. This is noted in a comment. Upon acceptance of this PR, we should make an issue to track this issue and mark it as semver:major.
Requirements (place an x in each [ ])

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.

Forcing a merge with the decreased test coverage. We haven't come up with a good way to mock a response in our test runner, and we don't want to wait on that to get this change out.


Symfony 5.1.7

Coverage increased (+0.2%) to 97.194% when pulling 0b4a3fe31d458358075e519c6f3a7da3f918e798 on symfony-5.1.7 into 7dab2b32acc67c04088524db21016a0a215f9efa on master.


pass timestamp as header and a GET arg to /values

details: https://github.com/Sentenai/Corona/issues/312

https://github.com/Sentenai/py-sentenai/pull/56


Add PostgreSQL tests

Evaluate the effect of bit size using PostgreSQL
@srikanthallu , does this reflect how you performed the data size evaluations with Postgres?

See this notebook in particular: https://github.com/ROVI-org/storage-requirement-estimation/blob/postgres/evaluate-postgresql/0_store-in-postgres.ipynb


Autogenerated schemas not quite as fully-featured as hand-written ones

Hi,
The new schema autogeneration stuff is very interesting.  Meanwhile we have been writing schemas by hand: so I am now trying to generate something that is equivalent.
I'm having trouble with a couple of things:

I don't see a way to perform extra validation, in particular I'd like to be able to insist that a string match some pattern:

   mySpecialString:
      description: A string that matches a regular expression
      type: string
      pattern: "^[^:,]+:[^:,]+$"

https://github.com/GREsau/schemars/issues/12 looks relevant: but also long inactive.

I don't see a way to insert funky stuff like x-kubernetes-int-or-string: true.  Eg I currently have a field:

    /// Field that can be integer or string in the k8s object
    #[serde(deserialize_with = "string_or_number")]
    pub my_special_field: u16,

which makes use of this.
Am I missing a way to achieve these things?
Alternatively, WIBNI to be disable the schema generation.  If I'm going to have to write it by hand anyway, why force me to generate it?
Thanks!

I figured out that I can add a pattern, by hand-implementing JsonSchema.  x-kubernetes-int-or-string: true currently still has me beaten, though.

Ooh, I can do that too with a hand-crafted implementation.  It's all a bit verbose, but do-able.  Sorry for the noise!

Schema generation does not fully support declarative validation rules yet, but some of them are supported. I summarized the current state here: https://github.com/clux/kube-rs/issues/129#issuecomment-749463718
It also cannot generate structural schema for more complex types (any_of, all_of, one_of). This requires transforming the Open API v3 schema into something Kubernetes specific.

@dimbleby : I am happy that you've found a workaround, but we also think you have a point here about people potentially being forced to use a new dependency that they might not need. So, to clarify this issue for future readers; in the next version, kube-derive will have a default-enabled schema feature that you can opt out of to manually generate the required schema. See https://github.com/clux/kube-rs/blob/9e0cd42cd2d645b51a6451fa369660d73591a91f/examples/crd_derive_no_schema.rs#L14-L45  for details. Until the 0.45.0 is out, this will be accessible from master.


Update setup.py

fixed "requirement" typo

@nicko170 Could you review it please?

Thanks @netdevadm. Merged.


Store version map in cache on success (instead of failure) to retrieve it.

Fixes #144.
I'm not sure how to add a test for this since the tests for sourceGateway assume git.

roflcopter 🤣 🤣 🤣 🤣 🤣
good catch, thank you 😄
so, for tests...yeah. this certainly does highlight why we'd want to test more than just the git source gateway - this specific behavior is one that differs because of how the different maybeSource impls work for different vcses, and now hey look! it's the one we missed.
i think the easiest (relatively speaking) thing to do for tests would be:

rename testSourceGateway to testGitSourceGateway
just copy the whole func body into a new testHgSourceGateway
adapt values from testHgSourceInteractions into the new test

hg and bzr work the same way with respect to this particular bug, so i think just adding the one is should be adequate for now.

Are there a common set of behavioral invariants across all implementations that we could test for? I'm at a disadvantage being new to the code base.

Between git, bzr, and hg, this is - unless I'm forgetting something - the only behavioral difference at the level of the sourceGateway. The difference is a product of relying on git ls-remote as the "ping" mechanism to determine an upstream repository's existence - it would be very wasteful (another network request later) to ask for a version list and discard it. bzr and hg, OTOH, separate pinging from listing versions.
svn is something else, but we really don't even properly support that yet, so best to leave it out of discussion.
sourceGateway itself is a slapdash FSM, with require() being the mechanism for effecting transitions. The rigorous-but-brute-force approach to testing it would probably be enumerating the possible transition space (it's not TOO huge) and testing each. That, or a smarter reduction of it, would be nice to have, but is a task for another PR that doesn't have a crucial bugfix :smile:. The contents of testSourceGateway is my basic attempt at covering most of the transitions, which is why I think duplicating it for hg would be adequate.

This is actually causing larger issues than I'd realized - I misdiagnosed a different problem as this one last week. Do you think you'll have time to tackle additional tests anytime soon? If not, I understand, but I'd like to merge this and we can do more tests in a follow-up.

I might have time to work on adding tests this weekend, but can't promise I'll have them working in the next few days. If you merge this before then I'll just open a new PR to add the tests.

@ChrisHines great, I'll merge now then. thank you!


Option to only check for non-existent params in pydocs, rather than params missing in pydocs

I am hoping for a config option along the lines of --require-doc-for-all-args (where the default/current behavior is true). I would like the option to allow a pydoc that only documents some of the params. This would give more flexibility to the developer.
If possible, it would also be great if there was an option to not require return docs. Maybe --require-return-section
Thanks!

Hi @JessieSteckling , I'd be very hesitant to add any config options that don't promote good coding style.  IMHO, only documenting certain params is not a good coding style.  In fact, part of the reason why people want to use pydoclint or darglint is to ensure all input args and return args are documented.
If you really want to achieve that, I think using noqa to bypass specific violations is the way to go.

Closing this issue.


SEVERE: Error while invoking read-locked operation

Hi, I would like to ask what could be wrong on our setup. When I use the redis + tomcat +nginx. I don't get any error regarding Locktemplate but when we try to change the nginx and replace it with AWS Applcation Load balancer the Locktemplate shows in our logs.
Example Stacktrace:
Caused by: java.lang.ClassCastException: java.lang.StackTraceElement cannot be cast to java.lang.String 	at java.io.ObjectInputStream.readTypeString(ObjectInputStream.java:1612) 	at java.io.ObjectStreamClass.readNonProxy(ObjectStreamClass.java:719) 	at java.io.ObjectInputStream.readClassDescriptor(ObjectInputStream.java:882) 	at java.io.ObjectInputStream.readNonProxyDesc(ObjectInputStream.java:1815) 	at java.io.ObjectInputStream.readClassDesc(ObjectInputStream.java:1713) 	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:2000) 	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1535) 	at java.io.ObjectInputStream.defaultReadFields(ObjectInputStream.java:2245) 	at java.io.ObjectInputStream.readSerialData(ObjectInputStream.java:2169) 	at java.io.ObjectInputStream.readOrdinaryObject(ObjectInputStream.java:2027) 	at java.io.ObjectInputStream.readObject0(ObjectInputStream.java:1535) 	at java.io.ObjectInputStream.readObject(ObjectInputStream.java:422) 	at org.apache.catalina.session.StandardSession.doReadObject(StandardSession.java:1611) 	at org.apache.catalina.session.StandardSession.readObjectData(StandardSession.java:1077) 	at com.gopivotal.manager.SessionSerializationUtils.deserialize(SessionSerializationUtils.java:81) 	at com.gopivotal.manager.redis.RedisStore$22.invoke(RedisStore.java:442) 	at com.gopivotal.manager.redis.RedisStore$22.invoke(RedisStore.java:436) 	at com.gopivotal.manager.LockTemplate.withReadLock(LockTemplate.java:59) 	... 24 more
Tomcat Version: 8.5.27
redis-store-1.3.2.RELEASE.jar
redis-3.2.10-2.el7.x86_64
Here's my context.xml.
<Valve className="com.gopivotal.manager.SessionFlushValve" />   <Manager className="org.apache.catalina.session.PersistentManager">     <Store className="com.gopivotal.manager.redis.RedisStore"                          host="<IP_ADDRESS>"                           port="6379"/>   </Manager>
I've already saw  some old issues regarding Locktemplate but none of them worked for me
(#48 )
(#20)
Any thoughts will be much appreciated.

My bad,  found out by one of our developer that some objects in the checkout session weren't serialized that's why we are getting an error regarding read-locked. Closing this issue now.


[.NET SDK] Improve pt-BR choice translation

Increase option to "yes" and "no" choices for pt-BR

@tomferreira Thank you for contributing.


New option to install Vault via HashiCorp Linux Repo

Introduce a new variable vault_install_hashi_repo which we can toggle to install Vault via HashiCorp Linux repository.
More info about this new repo can be found in this announcement: https://www.hashicorp.com/blog/announcing-the-hashicorp-linux-repository

This could be a bit more consise using the generic package: module to install after configuring yum, dnf and apt with their specifics.

So I did try out using the package: module, but it doesn't seem to support installing a specific version of a package: https://docs.ansible.com/ansible/2.8/modules/package_module.html
That's why I'm using yum and apt separately here

Thanks for this contribution!


Do not pack libraries into vendor.js in development mode

Please add options to not pack libraries into one vendor.js bundle during development because it is hard to read console.log out put with the debug information from libraries.

@casufi You could use source maps, or make your own configuration of webpack.


#653 Detection of multilanguage choices only works if first choice list is multilanguage

Closes #653
Why is this the best possible solution? Were any other approaches considered?
There are 2 main problem areas addressed: the translation warning, and the detection of translations for triggering multi-language (itext) output behaviour,
This changes the missing translations check in xls2json to use column headers instead of the first row. To avoid re-iterating the sheet data, the existing dealias function is modified to track and return the headers as well as the sheet data.
This changes multi-language detection by doing so only in the Survey object. For the Question sub-types, Survey sets the relevant info with new fields, so that the Questions don't need to re-check. For choices, the detection is per itemset (it could have looked across all itemsets but then untranslated itemsets would've had to be mangled into translations).
What are the regression risks?
Since there are some new fields on the Survey and Question classes, external library users that pass in JSON without these may experience errors or inconsistent behaviour. There is a file json_form_schema.json but it seems to be not consistently updated, and I don't know if it's accurate - there doesn't seem to be a pyxform way to generate it or use it to validate.
Does this change require updates to documentation? If so, please file an issue here and include the link below.
Probably not, it's all internal representation.
Before submitting this PR, please make sure you have:

[x] included test cases for core behavior and edge cases in tests
[x] run nosetests and verified all tests pass
[x] run black pyxform tests to format code
[x] verified that any code or assets from external sources are properly credited in comments

There's a small recommendation and a question about risk inline but I'm going to merge now so we can get it onto staging asap.


rustorm supports mysql

...I think. The example code mentions a "with-mysql" feature:
https://crates.io/crates/rustorm
Currently the summary only says:
If you’d like to use an ORM there is:
* Diesel (Postgres / Mysql / Sqlite)
* Rustorm (Postgres / Sqlite)

Rustorm itself has mentioned mysql support on its changelog. Here's the link:
https://github.com/ivanceras/rustorm/blob/master/Changelog.md#0150

Thanks for an update!


Added route to get win rate with game id.

Added a route in routes/api/games.js to get the win rate from a given game id.

That's awesome! I'll try to implement UI part today and deploy so you could see result of your work in action. Thanx!

Yea will be waiting for that.😀😀😀

It's alive) Great feature, thanx again

O great to see it in action!!!!
On Mon, Oct 5, 2020, 8:37 PM DavidGo<EMAIL_ADDRESS>wrote:

It's alive) Great feature, thanx again
[image: image]
https://user-images.githubusercontent.com/32278933/95096988-9fc3c300-0735-11eb-9f74-7363e324e855.png
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/DavidGolodetsky/GeekScore-2/pull/55#issuecomment-703694130,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKO6BNJQJHNHMCP2EBOORUTSJHOMNANCNFSM4SESJXUQ
.


Remove param Enabled

The Enabled param for crosschain is no longer used anywhere and should be removed from the codebase

Hey @lumtis , Can I work on it?

Sure, assigning it to you. Thanks!


Use posix spawn and report abnormal termination

This avoids command injection or other weird problems due to double expansion of shell metacharacters.

Code review was adequate.


Compose Example doesent work on Windows 10

https://docs.docker.com/compose/gettingstarted/
If i try the example and run docker-compose up i get always this error message:
PS C:\Development\flask-app> docker-compose up
Starting flask-app_web_1   ... done
Starting flask-app_redis_1 ... done
Attaching to flask-app_redis_1, flask-app_web_1
redis_1  | 1:C 04 Feb 2021 20:12:32.209 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
redis_1  | 1:C 04 Feb 2021 20:12:32.210 # Redis version=6.0.10, bits=64, commit=00000000, modified=0, pid=1, just started
redis_1  | 1:C 04 Feb 2021 20:12:32.210 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
redis_1  | 1:M 04 Feb 2021 20:12:32.212 * Running mode=standalone, port=6379.
redis_1  | 1:M 04 Feb 2021 20:12:32.212 # Server initialized
redis_1  | 1:M 04 Feb 2021 20:12:32.212 # WARNING overcommit_memory is set to 0! Background save may fail under low memory condition. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.
redis_1  | 1:M 04 Feb 2021 20:12:32.212 # WARNING you have Transparent Huge Pages (THP) support enabled in your kernel. This will create latency and memory usage issues with Redis. To fix this issue run the command 'echo madvise > /sys/kernel/mm/transparent_hugepage/enabled' as root, and add it to your /etc/rc.local in order to retain the setting after a reboot. Redis must be restarted after THP is disabled (set to 'madvise' or 'never').
redis_1  | 1:M 04 Feb 2021 20:12:32.213 * Ready to accept connections
web_1    |  * Serving Flask app "main.py"
web_1    |  * Environment: production
web_1    |    WARNING: This is a development server. Do not use it in a production deployment.
web_1    |    Use a production WSGI server instead.
web_1    |  * Debug mode: off
web_1    | Usage: flask run [OPTIONS]
web_1    |
web_1    | Error: Could not import "main".
flask-app_web_1 exited with code 2

Its also weird that he says: serving Flask App "main.py". But my file is called àpp.py.
Dockerfile:
FROM python:3.7-alpine
WORKDIR /code
ENV FLASK_APP=app.py
ENV FLASK_RUN_HOST=<IP_ADDRESS>
RUN apk add --no-cache gcc musl-dev linux-headers
COPY requirements.txt requirements.txt
RUN pip install -r requirements.txt
EXPOSE 5000
COPY . .
CMD ["flask", "run"]

docker-compose.yml:
version: "3.9"
services:
  web:
    build: .
    ports:
      - "5000:5000"

  redis:
    image: "redis:alpine"

app.py:
import time

import redis
from flask import Flask

app = Flask(__name__)
cache = redis.Redis(host='redis', port=6379)

def get_hit_count():
    retries = 5
    while True:
        try:
            return cache.incr('hits')
        except redis.exceptions.ConnectionError as exc:
            if retries == 0:
                raise exc
            retries -= 1
            time.sleep(0.5)

@app.route('/')
def hello():
    count = get_hit_count()
    return 'Hello World! I have been seen {} times.\n'.format(count)

And in VSCode he says also:

Thanks for sharing this issue. As this issue is old, some context may have changed. I'm unable to reproduce the issue when following the docs and it appears to be specific to your environment. If you changed "main.py" to "app.py" in your dockerfile after running docker-compose up, make sure you remove the old images and volumes.
As this is not an issue in the documentation, I'm closing this issue. If you need further support, you can try the Docker Community Forums.


Produce GLOWS L1B CDFs

Steps

[ ] Produce DE CDFs
[ ] Produce Histogram CDFs
[x] Transform L1A data to L1B data
[x] Create L1B attributes
[x] Create validation tests

Tickets

#889
#201
#895
#897

Remaining work is blocked by lacking SPICE tools, see #897


Adding scripts for CCEval prompt crafting for RAG experiments

Issue #, if available: https://github.com/amazon-science/cceval/issues/14
Description of changes: Adding python scripts in the directory prompt_builder to form CCEval prompt for RAG experiments.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@wasiahmad Thanks so much for uploading the scripts used to generate data.
I noticed that raw-data zip has a slightly different structure than assumed in augment_with_cfc.py
#root_dir = os.path.join(repository_root, lang, repo_name)
root_dir = os.path.join(repository_root, repo_name)


Sends a proof presentation - Wallet credential identifier

POST: /present-proof/records/{pres_ex_id}/send-presentation
above api request requires: "Wallet credential identifier"
but doesn't look like there is  "Wallet credential identifier" anywhere.
here is the response from http://localhost:9001/present-proof/records/aa7f4604-7f27-442f-b36b-8d3db3ef459c/credentials:
[
  {
    "cred_info": {
      "referent": "string",
      "attrs": {
        "firstName": "Alice",
        "score": "9/10",
        "lastName": "Jones"
      },
      "schema_id": "3avoBCqDMFHFaKUHug9s8W:2:issue_schema_test1:1.0",
      "cred_def_id": "3avoBCqDMFHFaKUHug9s8W:3:CL:56740:default",
      "rev_reg_id": null,
      "cred_rev_id": null
    },
    "interval": null,
    "presentation_referents": [
      "additionalProp1"
    ]
  }
]

I think you need to use the referent attribute as the "Wallet credential identifier". Not sure why those have different names
See https://github.com/hyperledger/aries-rfcs/blob/master/features/0037-present-proof/README.md#referent for more information on referent

gotcha. it works. thank you!
do you want me to keep issue open so that naming can be corrected to avoid confusion?

Better yet -- put in a PR.  :-)

You are likely to find these terms used interchangeably throughout the indy-sphere for historical reasons.
I don't remember exactly but I think they introduced 'credential identifier' to disambiguate from 'referent' in a proof request, after they changed 'uuid' to 'referent' in the proof request.
The referent in an RFC-37 proof proposal can tie multiple items together into one credential for presentation, or allow for their presentation via distinct credentials. Hence the RFC-37 referent acts as an index of the wallet referent within the proof proposal.


Fix: Remove default glob from shell.test (fixes #529)

Regression from #492

LGTM. Thanks for the fix. @gyandeeps did you find any other commands that had a similar mistake? If not, I can merge this tomorrow.

This s the only I was sure about.

I can't find any other regressions. This should be great, thanks for catching this!

@nfischer When is the plan to make a release with this fix? Just wanted to know. Thanks

I need to take a look at our recent commits, but I think we can get this out as a patch release pretty soon. I also want to push out some plugin-related features to begin dogfooding them, but I should be wrapping those up soon. We can probably get this out within the week.


Have the following issue when trying to install

npm WARN deprecated<EMAIL_ADDRESS>please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue
npm WARN deprecated<EMAIL_ADDRESS>This is a stub types definition for commander (https://github.com/tj/commander.js). commander provides its own type definitions, so you don't need @types/commander installed!
npm WARN deprecated<EMAIL_ADDRESS>Deprecated in favour of eslint-config-wikimedia. -- https://phabricator.wikimedia.org/T118941
npm WARN deprecated<EMAIL_ADDRESS>Package no longer supported. Contact<EMAIL_ADDRESS>for more info.
npm ERR! Windows_NT 10.0.17134
npm ERR! argv "C:\Users\a\AppData\Local\atom\app-1.30.0\resources\app\apm\bin\node.exe" "C:\Users\a\AppData\Local\atom\app-1.30.0\resources\app\apm\node_modules\npm\bin\npm-cli.js" "--globalconfig" "C:\Users\a\.atom\.apm\.apmrc" "--userconfig" "C:\Users\a\.atom\.apmrc" "install" "C:\Users\a\AppData\Local\Temp\d-11883-17912-705rqs.jwzc15rk9\package.tgz" "--runtime=electron" "--target=2.0.5" "--arch=x64" "--global-style"
npm ERR! node v6.9.5
npm ERR! npm  v3.10.10
npm ERR! code EREADFILE
npm ERR! Error extracting C:\Users\a.atom.apm\marko\4.13.3\package.tgz archive: ENOENT: no such file or directory, open 'C:\Users\a.atom.apm\marko\4.13.3\package.tgz'
npm ERR!
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!     C:\Users\a\AppData\Local\Temp\apm-install-dir-11883-17912-270ehe.v4d39mgqfr\npm-debug.log
(WINDOWS 10 MACHINE)

I just published v0.33.1.  For the time being, marko-prettyprint has been removed as there is an issue with the marko dependency when downloading.


hope to support environment variable for prometheus config

Proposal
i'd like to use environment fill prometheus config, but current not support.
Use Case
use pod-name to fill the external_label

Dupe of #2357.

@mojo-zd   Starting Prometheus 2.27, we will be able to substitute env variables in external labels if you use --enable-feature=expand-external-labels
This can be tested  with the main docker image.


esp_event.h: No such file or directory (IDFGH-8032)

I'm trying to link my project (idf 5) with esp_modem but I'm getting the following error:
... managed_components/espressif__esp_modem/include/cxx_include/esp_modem_primitives.hpp:17:10: fatal error: esp_event.h: No such file or directory
idf_component.yml:
## IDF Component Manager Manifest File
dependencies:
  espressif/esp_modem: "==0.1.18"
  espressif/coap: "==4.3.1-rc.1"
  ## Required IDF version
  idf:
    version: ">=4.4.0"
  # # Put list of dependencies here
  # # For components maintained by Espressif:
  # component: "~1.0.0"
  # # For 3rd party components:
  # username/component: ">=1.0.0,<2.0.0"
  # username2/component2:
  #   version: "~1.0.0"
  #   # For transient dependencies `public` flag can be set.
  #   # `public` flag doesn't have an effect dependencies of the `main` component.
  #   # All dependencies of `main` are public by default.
  #   public: true

Main CMakeLists.txt
idf_component_register(SRCS "bt.c" "main.c"
                    INCLUDE_DIRS "." REQUIRES esp_event driver nvs_flash bt esp_timer GSM)

target_compile_options(${COMPONENT_LIB} PRIVATE "-Wno-format" "-Wno-implicit-function-declaration" "-Wno-cpp" "-Wno-unused-variable")

Can someone help me?

The fix was recently committed by @suren-gabrielyan-espressif in https://github.com/espressif/esp-protocols/commit/6e4e4fab1d2c6c63c796a81b5dce33d4d33892e4 but it seems the new release of esp-modem hasn't been made yet. We'll try to release 0.1.19 with the fix soon!

Thanks for reporting, the issue seems to have been fixed when the https://github.com/espressif/esp-protocols/pull/112 gets merged, feel free to reopen.

The IDF version is 5.0.1


[Bug] RefreshInterval is too long if no metadata has been retrieved (IDX20803)

Which version of Microsoft.IdentityModel are you using?
Latest - problem is present in dev branch at the time of writing.
Where is the issue?

[x ] M.IM.Protocols

Is this a new or an existing app?
Irrelevant, issue is present in current dev branch.
Repro

Have the initial ConfigurationManager::GetConfigurationAsync call fail for whatever reason (e.g. transient network issue)
Make any REST call to the service

Authentication will fail then with IDX20803: Unable to obtain configuration from: ... for the duration of RefreshInterval (5 minutes by default).
Expected behavior
When no AAD configuration is currently present in ConfigurationManager (_currentConfiguration), AAD configuration should be retrieved even if RefreshInterval has not yet passed
Actual behavior
When no AAD configuration is currently present, the SDK will throw IDX20803 for the duration RefreshInterval.
Possible solution
Either ignore RefreshInterval if no AAD configuration is currently present, or provide a new configuration option such as MissingConfigurationRefreshInterval that would default to maybe 5 seconds.
Additional context / logs / screenshots / links to code
Upon failure, the sync timeout is set to RefreshInterval: https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/900dc266687d24f96c78cd9af3ac085461f3bba0/src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs#L150
Therefore, until RefreshInterval passes, all calls will fail immediately: https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/900dc266687d24f96c78cd9af3ac085461f3bba0/src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs#L162
Workaround
Could we just set RefreshInterval to be something like 5 seconds? What is the worst case scenario here, we make an AAD config call every 5 seconds? On it's face it doesn't seem like a huge issue...

@ohadschn BaseConfigurationManager.RefreshInterval allows you to set the value.
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/900dc266687d24f96c78cd9af3ac085461f3bba0/src/Microsoft.IdentityModel.Tokens/BaseConfigurationManager.cs#L120
Would this work for you?

As a workaround, yes, but presumably there is a reason the default value is 5 minutes as opposed to 5 seconds no?

@ohadschn We adopted a bootstrap refresh interval when no AAD configuration is currently present in ConfigurationManager. The new timer with exponential backoff for failing to fetch metadata at bootstrap til we get the first config.
Would this work for you?
PR: https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/pull/2052

@ciaozhang thank you, it looks good.
My only suggestion would be to have a limit lower than RefreshInterval for the case where _currentConfiguration is null.
So perhaps you'd define a MaxBootstrapRefreshInterval with a lower default than RefreshInterval (say 1 minute) and then instead of if (_bootstrapRefreshInterval < RefreshInterval) ... you would have something like if (_bootstrapRefreshInterval < Math.Min(MaxBootstrapRefreshInterval, RefreshInterval)) ...

We believe the exponential backoff(start at 1 second) is better option for the case where _currentConfiguration is null.

@ciaozhang I'm not suggesting you should get rid of the exponential backoff, just use a lower maximum for it.
Keep everything as is, just change if (_bootstrapRefreshInterval < RefreshInterval) ... to if (_bootstrapRefreshInterval < Math.Min(MaxBootstrapRefreshInterval, RefreshInterval)) ...
BTW I can't reopen issues...

@ohadschn  The bootstrapRefreshInterval won't be longer than the RefreshInterval if you take a look of our implementation.
https://github.com/AzureAD/azure-activedirectory-identitymodel-extensions-for-dotnet/blob/5068c6eaf1c87cf3fac7fb7411bbaee7ded8baa4/src/Microsoft.IdentityModel.Protocols/Configuration/ConfigurationManager.cs#L181


Attempt method in class TokenGuard

Laravel Version: 7.27.0
PHP Version: 7.3.13
Database Driver & Version: Mysql 5.7.28

Description:
There's isn't an attempt method on Illuminate\Auth\TokenGuard, Therefore we have to have a parallel auth guard in \config\auth.php for each driver session, token. In-order to authenticate a user with session & then creating a token for the user.
I'll explain with some code.
In config\auth.php
'web' => [
    'driver' => 'session',
    'provider' => 'users',
],

'api' => [
    'driver' => 'token',
    'provider' => 'users',
    'hash' => false,
],

if (! $token = auth()->guard('web')->attempt($credentials)) {
    return response()->json(['error' => 'Unauthorized'], 401);
}

$token = auth()
    ->guard('web')
    ->user()
    ->createToken( 'token-name');

If only there was an attempt method on Illuminate\Auth\TokenGuard so we could do something like following:
if (! $token = auth()->guard('api')->attempt($credentials)) {
    return response()->json(['error' => 'Unauthorized'], 401);
}

$token = auth()
    ->guard('api')
    ->user()
    ->createToken( 'token-name');

Why?
This i'm thinking for the scenario where there's isn't any web app, There's just Laravel backend running so we can skip declaring guard for a session driver & we don't have to
    'guards' => [

         'api' => [
             'driver' => 'token',
             'provider' => 'users',
             'hash' => false,
        ],
    ],

PS: If this is dumb, I'm honestly sorry for taking up your time.

Sounds like a feature request. Feel free to pr something.


Event flags should auto-detect overflow state

Currently, this is hard coded to 3 events. Flags should check whether or not they can be added without overflowing.


Mk32 Glowup

Improved the Mk32 sprite, fixed erroneous pixels on the katana sheath sprite, added a new pistol gunshot for the Universal.

About the PR
Media
It looks like this now I guess:

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: TJohnson

tweak: The experimental Mk32 design has been upgraded to be more ergonomic and quieter, for the shooter's convenience.

Seems like a pretty simple PR. The RSI validation error isn't to do with any of my files, and I'm not sure what the build and test debug failure is all about.

RSI validator will have to get fixed on master

sprites and sounds are fine in game


Please add credit to kmp-viewmodel

Hi, I have noticed some code in meteor-viewmodel is similar to my library kmp-viewmodel(https://github.com/hoc081098/kmp-viewmodel/tree/master/viewmodel).
Could you please add credits/links to my library?
Thanks 🙏

I noticed the same thing ! .

I noticed the same thing ! .

Same to me. I noticed the same thing !. Please leave the credit

Please leave the credit

🙏

Hey, @hoc081098
Thanks for your attention! Sure, I'll added your your library as credit :)
In the future, I'll change meteor-viewmodel code.
@hoangchungk53qx1 @olololoe110399 @dungngminh @olololoe110399 Thank you all very much for your attention!


Tundra2.1 segfaults on linux

Did a clean checkout, and clean build.
Ubuntu 32bit, natty.
brett@natty32:~/naali/bin$ gdb Tundra
GNU gdb (Ubuntu/Linaro 7.2-1ubuntu11) 7.2
Copyright (C) 2010 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later http://gnu.org/licenses/gpl.html
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.  Type "show copying"
and "show warranty" for details.
This GDB was configured as "i686-linux-gnu".
For bug reporting instructions, please see:
http://www.gnu.org/software/gdb/bugs/...
Reading symbols from /home/brett/naali/bin/Tundra...done.
(gdb) run
Starting program: /home/brett/naali/bin/Tundra
[Thread debugging using libthread_db enabled]
Starting up Tundra.

Working directory: /home/brett/naali/bin
Startup options:
'--accept_unknown_http_sources'
'--accept_unknown_local_sources'
'--fpslimit' '60'
'--hide_benign_ogre_messages'
'--nofilewatcher'
Config directory: /home/brett/.Tundra/configuration/
[New Thread 0xb7138b70 (LWP 16669)]
[New Thread 0xb67ffb70 (LWP 16670)]
Asset cache directory: /home/brett/.Tundra/assetcache/
[New Thread 0xb5c63b70 (LWP 16671)]
Opened default OpenAL playback device.
[New Thread 0xb1461b70 (LWP 16672)]
Loading plugin 'OgreRenderingModule'.
Loading plugin 'EnvironmentModule'.
Loading plugin 'PhysicsModule'.
Loading plugin 'TundraProtocolModule'.
Loading plugin 'JavascriptModule'.
[New Thread 0xb0a8bb70 (LWP 16674)]
Loading plugin 'AssetModule'.
Loading plugin 'AvatarModule'.
Loading plugin 'ECEditorModule'.
Loading plugin 'SkyXHydrax'.
Loading plugin 'OgreAssetEditorModule'.
Loading plugin 'DebugStatsModule'.
Renderer: Initializing Ogre
--* OGRE Initialising
--* Version 1.7.3 (Cthugha)
Renderer: Using OpenGL Rendering Subsystem

Program received signal SIGSEGV, Segmentation fault.
0x03175565 in std::basic_string<char, std::char_traits, std::allocator >::basic_string(std::basic_string<char, std::char_traits, std::allocator > const&) () from /usr/lib/i386-linux-gnu/libstdc++.so.6

works for me on 64bit natty -- do you Cg enabled in plugins-unix.cfg? Supposing you are using the andrewfenn ppa or otherwise have built Ogre with cg.
if you don't have cg, perhaps need to disable skyx-hydrax? i have cg and skyx at least works.

works for many now, supposedly the issue is gone


kconfig: fix dt_node_has_prop

The Kconfig function "dt_node_has_prop" was using label as its
parameter, where other functions use either chosen or path.
The documentation says that the parameter is path, so this patch
makes the function as documentation says and as other functions
in the file.
Signed-off-by: Michał Barnaś<EMAIL_ADDRESS>

Using label instead of path was implemented as purpose, as this is more convenient.

I understand that, but it's not consistent with rest of the functions in the file. I would rather make this function as the rest, to operate on the paths, and make another functions that operate on labels but with name that suggests the difference (x_by_label).

Ok, then please do.

Note that thanks to dtc checks, we're sure that label is unique and can be used safely. Forcing to use path instead of label is the contrary of an improvement.

The functions that are using the paths, can be provided with aliases, since the devicetree lib checks if path starts with "/", if not, it looks for aliases. This will also guarantee the uniqueness.

If labels, which are there anyway, are sufficient to provide what we need, I don't think this is a wise choice to add aliases just to satisfy the requirements of poorly designed kconfig functions. We should aim at simplicity whenever it is possible.

Also, please don't change a function implementation if you don't update the users.

How is this done? In changelog or something?

I mean in tree users at least, for which grepping the code should be sufficient.
Then, I'm not sure if this API is exposed to out of tree users, but in which case, a deprecation message or warning should be considered. For this specific case of Kconfig API, I'm not sure how to proceed though.

Ok, then please do.

Done

If labels, which are there anyway, are sufficient to provide what we need, I don't think this is a wise choice to add aliases just to satisfy the requirements of poorly designed kconfig functions. We should aim at simplicity whenever it is possible.

I agree with you.

I mean in tree users at least, for which grepping the code should be sufficient. Then, I'm not sure if this API is exposed to out of tree users, but in which case, a deprecation message or warning should be considered. For this specific case of Kconfig API, I'm not sure how to proceed though.

Right, changed the usage to nodelabel function.

Thanks for this work and taking my remarks into account. 2 minor comments, otherwise LGTM.

Fixed, thanks for your review :)


Investigate use of JSON schema meta schema

available here: https://json-schema.org/specification-links.html

only used as guidance


disableWebsockets option - using wrapped sockjs instead

Allows node-red to work in environments that do not support websockets, using sockjs.

Coverage decreased (-0.2%) to 85.967% when pulling 248ae1dd12e3dbdf81dbd54c0c1a68a6be97c5c4 on ozomer:master into 96a0dbea2defa0eb24f1f8072aa312e740f1498a on node-red:master.

Coverage decreased (-0.3%) to 85.863% when pulling c1c01f27e4e9ede0422bea10d805bd473d110c86 on ozomer:master into 96a0dbea2defa0eb24f1f8072aa312e740f1498a on node-red:master.


Error retrieving available resources - Invalid or missing authorization in header (401)

Hi
Since October the integration stopped working.
Had this happen before and just started over with al the config and project on Mercedes API.
But now it doesn't seem to be working.
Created a new token via the script with the -t flag => OK
Can execute the -r flag but nothing seems to happen.
when executing the -s flag i get following error:
Error retrieving available resources - Invalid or missing authorization in header (401)
Error retrieving available resources
Error retrieving resource list.
Error initializing resources
So i can refresh the token but it doesn't seem to be valid when doing requests.

My bad, was creating the token with the old scripts.
Make sure you use the new scripts when creating the first token.
for some reason mercedes still let's you use the old way to create and refresh but not to use the token for requests.


nsjail: 2.2 -> 2.7

Motivation for this change
Package update
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Success on aarch64-linux (full log)
Attempted: nsjail
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7
shrinking /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7/bin/nsjail
gzipping man pages under /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7/share/man/
strip is /nix/store/428gs2z4b8f9byvghzlpbjwjb3a7jwxx-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7/bin
patching script interpreter paths in /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7
checking for references to /build in /nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7...
/nix/store/36j6551f7xggf5aw77sfkwcidwgy1ppx-nsjail-2.7

Success on x86_64-linux (full log)
Attempted: nsjail
Partial log (click to expand)
installing
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7
shrinking /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7/bin/nsjail
gzipping man pages under /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7/share/man/
strip is /nix/store/dxf1m7dhc4qb655bdljc1fsd74v1nag3-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7/bin
patching script interpreter paths in /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7
checking for references to /build in /nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7...
/nix/store/614i2ffhcfdaxb3g84q94kbjin8nwkix-nsjail-2.7

cc @bosu for a test/review

LGTM, thanks.


help with Language Problem

Hello, Italian and Arabic and Persian language not working. blank page show when select this languages.
the global.po file also have problem when open it with poedit software. it say duplicate string and gettext error
who can help, please?


fix: add ns param for removeAttribute

Thank you for the PR!
Can you run cargo fmt on your branch?
Other than that, I think things look good.

Looks great! Thank you!


How can I add a keypress event that selects current item and closes select box?

I would like the Return key to select the currently selected item and close the select box.
I tried adding this at line 1088:
'ng-keyup="if($event.which === 13) syncItems( item, $event, $index )"' +
But I can't get this to trigger..
Any ideas on how  I can do this?

Further to this I just noticed that actually space bar already does exactly what I want!
So I would like to change the Keypress Id that triggers the save and close event from Space Bar to the Return key.
Also I would like to add the down arrow to the list of keys that activate the control. When tabbing from the previous field, I would like to click down arrow to open this, currently the return key does this job.
So can you point me in the right direction of changing these keycodes? I have tried searching through the code and I can't find the Spacebar or Return key events anywhere..! :)

Hi,
I actually try to rely on whatever default browser implementation for most of the things. That's why you don't see return or spacebar keys (13 or 32) keys in the code.
For example, since it's actually just a button, pressing return or spacebar on the button will toggle the dropdown.
Another example is on the selection item. I actually use a checkbox to handle the keyboard listener, so spacebar will select/deselect an item.
I personally against the down arrow to activate the control since it's not a standard, but if you need it, you'll need to add your own key listener. Just add new keys in the keyboardListener function.


v1 tag is not pointing to the latest v1 version

It would be useful that the v1 tag would always point to the latest v1.x version. That way we could use algolia/algoliasearch-crawler-github-actions@v1 in Github Actions workflow.

I believe a potentially better way to solve this would be to make v1 a branch instead of a tag. Then when releases/changes are made, make changes directly on v1 and optionally tag it with a 3-component semver.


[Query Fuzz] Query panic - influxql.Expr is *influxql.Wildcard, not *influxql.VarRef

This issue was found by fuzzing the database with a query generator tool. It may be possible to simplify the query to help identify the root cause.
Dataset
$ curl https://s3.amazonaws.com/noaa.water-database/NOAA_data.txt -o NOAA_data.txt
$ influx -import -path=NOAA_data.txt

Query
QUERY: SELECT    COUNT(*) +   MIN("pH") *   MEDIAN("index")  AS "fA" FROM ( SELECT    DISTINCT("index")  AS "_" FROM  "average_temperature" LIMIT 6162) ORDER BY time ASC

Result
Server returns a panic response.
See attached trace.
1488219733564080130-panic.txt

Further examples which I believe may be the same panic:
1488219744271203574-panic.txt
1488219749799180959-panic.txt
1488219783949433866-panic.txt
1488219784274789674-panic.txt
1488219858286334718-panic.txt
1488219959854317737-panic.txt
1488219975159021313-panic.txt
1488219975938728265-panic.txt
1488219980131072766-panic.txt
1488220045879950882-panic.txt
1488220069229540178-freeze.txt
1488220162935703106-panic.txt
1488220199616613808-panic.txt
1488220244142216837-freeze.txt
1488220309346886646-freeze.txt
1488220347155331031-freeze.txt
1488220393906083238-panic.txt
1488220401760049354-panic.txt
1488220432585038269-panic.txt
1488220515165036047-panic.txt
1488220528324224756-panic.txt
1488220576028736525-panic.txt
1488220606956366191-panic.txt
1488220703491897190-panic.txt
1488220751761977833-panic.txt
1488220754828598389-panic.txt
1488220819956786864-panic.txt
1488220879570619784-panic.txt
1488221029481144191-panic.txt
1488221122832762639-panic.txt
1488221141435341877-panic.txt
1488221334192078618-panic.txt
1488221335963530507-panic.txt
1488221355513478176-panic.txt
1488221396639559914-panic.txt
1488221416315065597-panic.txt
1488221446470839843-panic.txt
1488221465124526802-panic.txt
1488221642213625422-panic.txt

The reason this seems to happen is because the wildcard expansion code just outright ignores any binary expression.
The fix isn't that simple. The main problem is that when calls are substituted, they're given an alternative name so they don't all just show up as the function name like mean. They do this by setting the alias attribute on the field that gets created.
Unfortunately, the only way to do this reliably would be to recurse through the binary expression and return a list of new expressions. But the Field type isn't an expression and is treated differently (only at the top level) so I'm not yet sure how to get that iteration process working yet.
I'll look at it and see what I can come up with though.

@e-dard I'm just going to prevent the panic and say this is currently impossible. Since it panicked before a change that now forbids it, I'm pretty certain nobody was using this successfully.


Fixed issue commits pagination

The pagination always show the first 10 commits.

It seems like this PR is broken? At least in only shows that a font is changed and I doubt that has something to do with pagination - @ozielguimaraes Can you find the original changes?

I pushed the same fixes previously. This is why the diff show nothing relevant.
The woff2 line in csproj is already there : https://github.com/jakubgarfield/Bonobo-Git-Server/blob/master/Bonobo.Git.Server/Bonobo.Git.Server.csproj#L533


[azdev-ify] Module discovery and usage for /core

It's not easy to figure out what are the modules I should be using - especially for hosts.
e.g., In /host

we have 11 biceps for appservice
4 for ACA (container-app.bicep vs. container-apps.bicep - what's the diff?)
3 for functions - more straightforward - a base and 2 lang-specific.
1 for staticwebapp

Consider:
Scenario 1 - I just want to use what's in /core
All I need is to know is what parameters to pass to the modules, I may not need to know what's in the modules but I need to know what each module is for and what is the contract.
Scenario 2 - I want to modify what's in /core and create my own private collection
For this scenario, developers do need to understand what's in the modules figure out how to customize/add own logics/conventions.

We could have files in root of /host be ones we intend people to use directly - but it gets iffy b/c they could use them directly in certain situations.


Add a context menu to the active tab that allows adding an annotation to the selected element.

Currently, you can enter "record mode" in order to select elements on the page, and add an annotation to them. However, we should also add a browser context menu that allows you to right-click an element of the page and add an annotation to it. Regardless if you're in record mode or not

It appears like the context menu API doesn't provide a reference to the selected DOM element. So this experience isn't possible.


Split Loudness computation according to ISO 532-1 method 1 and method 2

Currently the script to compute Loudness according to ISO 532-1 method 1 (stationary signal) and method (time varying signals) are mixed in the same folder. It can be confusing.
Will be created:

One folder for method 1 with a main function called comp_loudness_zwicker_stat() that takes third octave band spectrum as an input
One folder for method 2 with a main function called comp_loudness_zwicker_time() that takes time signal as an input
Main function called comp_loudness to call the different Loudness method according to an input parameter

Check my fork https://github.com/mglesser/MoSQITo to follow the development

Finally not pertinent


Improved GpuArrowEvalPythonExec

This gets GpuArrowEvalPythonExec to the point that I think it can be on by default.  From my testing with this the performance is either better than the original version (for fast python UDFs) or is as good (for slow UDFs simulated using a sleep) but with much lower CPU utilization so it is likely that that it would end up being faster because there would be less CPU contention.
I did fix one bug in the existing code to make this work.  In SpillableColumnarBatch if we try to create the batch from an batch of GpuColumnVectorFromBuffer I messed up the reference counting and things were being deleted before they should be, which ended up resulting in NullPointerExceptions instead of the double free or other similar error messages that are simpler to debug.

build

build


Is there anyway to interact and trade with a villager?

You might find better support in the Discord server: https://discord.gg/7PYY9x4JAq


documentation has bad example code

In the "Exit Codes and Exceptions" section of the documentation -https://github.com/amoffat/sh/blob/develop/docs/source/sections/exit_codes.rst#exit-codes--exceptions
there is the first example code
output = ls("/")
print(output.exit_code) # should be 0

This code does not work as described - when run in the terminal it returns an exception. It seems the way to make it work as described is -
ls("/", _return_cmd=True)
print(output.exit_code) # should be 0

this seems to include a larger issue, where the object returns is a string instead of a RunningComand object.. a fix for this may also be planned, but for now maybe consider fixing the docs?

Thanks for the report!
This was the only instance in the docs that needed fixing, AFAICS.
Commands return strings by default since sh 2.0.0, so only the docs needed fixing.


Agent DatabaseQuery action run

Context:
We can configure an agent with action = "Database Query", from this PR: https://github.com/dust-tt/dust/pull/2820
This one is about handling this new action when we run the agent.
This is the new Dust app: https://dust.tt/w/78bda07b39/a/e0c5993d65
(Shamelessly copy-pasted from @fontanierh's experiments 🙇🏻♀ )
Scope:
For now, "DatabaseQuery" action is open only to Dust so we can iterate on it after shipping this version -> Once merged I'll ping Duncid for reco.
Deployment:
Requires running init_db in front_edge.
Demo:
https://dust4ai.slack.com/archives/C050SM8NSPK/p1702459827197579?thread_ts=1702459806.291499&cid=C050SM8NSPK
Review:
Review with hidden whitespaces is advised: https://github.com/dust-tt/dust/pull/2851/files?diff=split&w=1

(btw the "run dust app" uses the input dataset's schema to generate inputs)

Indeed I'm not passing the context to the Dust app, I'll add some commits!
There was some sort of history but only from the generation step


Issue#896 Workflow steps with non-existant output artifact path will succeed

Issue: https://github.com/argoproj/argo/issues/897
Solution: Added new element "optional" in Artifact. The default is false.  This flag will make artifact as optional and existence check will be ignored if input/output artifact has optional=true.
Output Artifact ( optional=true ):
Artifact existence check will be ignored during the save artifact in destination and continued workflow
Input Artifact ( optional=true ):
Artifact exist check will be ignored during load artifact from source and continued workflow

I have experience with system that has implemented optional inputs twice, but both times the design was bad.
There needs to be a way to distinguish between two different causes of the lack of argument for an optional input:

The input is not connected to any upstream (at any point of the upstream hierarchy).
The input is connected, but some task in the upstream chain has failed. I expect that in this case the downstream tasks are not executed even when the input is optional.

Typical scenario where some bad designs fail:
There is a template with 5 optional input so that the consumer can connect 1 to 5 upstreams without changing the template.
Let's say that 4 inputs have upstream connections.
One of the 4 inputs have upstream failure, so the data is missing.
I expect that the template does not start (not all 4 connected inputs have argument values; even though the inputs are optional, they're connected and become required).
With some problematic designs, the template starts with only 3 out of 4 arguments.


vue-build docs page not working

do you mean the readme?
Can you give more details?

My apologies, This link  on the vue-cli readme

vue build
Use vue-cli as a zero-configuration development tool for your Vue apps and component, check out the docs.

Where are the docs now? The page not working


Vortex Staging Folder

Hello!
I have been using your tool for vortex for quite a while. Ever since you switched to vortex-linux nothing works anymore.
For example, I have my games and mods all on the same main internal drive. I do not use the SD Card slot. I set the games location such as Skyrim to the game path Z:\Home\deck.local\steam\steamapps\common\Skyrim Special Edition\ thats just me typing out the path I believe it is however no matter what game it is, or what path I set vortex always refuses to use ANY deployment mode unless I somehow created a Staging folder at Z:\Home\Vortex-Mods\ which is of course impossible because thats the user folder where you have deck or the other folder. But for whatever reason vortex refuses to see any other path as apart of the same drive the game is on.
At one point I did get it to deploy but it would only use Symlink deployment and that didn't work, no files got deployed and every single mod was always redundent. So basically I've had zero luck with this working even slightly. Vortex opens, mods download, but no deployment.

Hmm disregard that. I found you linked games to the J: drive. Which does fix the staging folder issue for Skyrim. However and I know theres technically no support for Witcher 3 but any mods I add get downloaded and installed. Folders are created for the mod in the games directory however no files are actually ever deployed. It only allows me to use Symlink which is probably the issue, but annoying as every single mod is always Redundant due to no files actually being linked or anything.

I had the same issue, but for skyrim. Where did you end up pointing the staging folder to in order to fix it? UNder my J folder when selecting the staging folder doesnt have skyrim. It only has proton builds and steamlinuxruntime

I had the same issue, but for skyrim. Where did you end up pointing the staging folder to in order to fix it? UNder my J drive when selecting the staging folder doesnt have skyrim. It only has proton builds and steamlinuxruntime

You probably installed Skyrim to your Microsoft card not your Internal drive. J: points to internal drive. E: points to your external micro and card

Where did you end up pointing the staging folder to in order to fix it?

Vortex's suggested path should always work.

Where did you end up pointing the staging folder to in order to fix it?

Vortex's suggested path should always work.

Only if when you selected where the game directory is that you used the E: or J: directory.
In my experience if you actually go to the path of the game through Z:etc instead of using the J: drive it bugs out. Vortex forces you to try to put the staging folder in Z:\home\vortex staging which of course it’ll throw an error for not allowing you to do that because it doesn’t have permissions to create a folder there.
But it won’t recognize anything inside of Z:\home\deck\ as the same “drive” for whatever reason.

Where did you end up pointing the staging folder to in order to fix it?

Vortex's suggested path should always work.

Only if when you selected where the game directory is that you used the E: or J: directory.
In my experience if you actually go to the path of the game through Z:etc instead of using the J: drive it bugs out. Vortex forces you to try to put the staging folder in Z:\home\vortex staging which of course it’ll throw an error for not allowing you to do that because it doesn’t have permissions to create a folder there.
But it won’t recognize anything inside of Z:\home\deck\ as the same “drive” for whatever reason.

Not quite the issue i had, the error it gave me was that the staging folder had to be on the same drive as the game install.(Which it points to my regular steam deck, steam apps folder. Not a virtualized one for some reason) And when i try pointing the staging folder to that steam deck native one, it gives me that error that they're not on the same drive. I haven't tested your solution yet though

Where did you end up pointing the staging folder to in order to fix it?

Vortex's suggested path should always work.

Only if when you selected where the game directory is that you used the E: or J: directory.
In my experience if you actually go to the path of the game through Z:etc instead of using the J: drive it bugs out. Vortex forces you to try to put the staging folder in Z:\home\vortex staging which of course it’ll throw an error for not allowing you to do that because it doesn’t have permissions to create a folder there.
But it won’t recognize anything inside of Z:\home\deck\ as the same “drive” for whatever reason.

Not quite the issue i had, the error it gave me was that the staging folder had to be on the same drive as the game install.(Which it points to my regular steam deck, steam apps folder. Not a virtualized one for some reason) And when i try pointing the staging folder to that steam deck native one, it gives me that error that they're not on the same drive. I haven't tested your solution yet though

That’s exactly what I just said. You must have the games directory pointing to Z:\home\deck.local\shared\steam\steamapps\common\Skyrim Special Edition\
Which is incorrect. If the game directory is pointed to the actual directory in the Z: drive vortex bugs out and says the staging folder must be in Z:\home\ which it cannot due. Putting the staging folder using the Z: letter which you have to if the games directory is Z it’ll tell you that the staging folder is not on the same drive.
to fix this change the games directory to J:\Skyrim Special Edition\
Then use Vortexe suggested path for the staging folder which will probably be J:\Vortex-Mods

That’s exactly what I just said. You must have the games directory pointing to Z:\home\deck.local\shared\steam\steamapps\common\Skyrim Special Edition\
Which is incorrect. If the game directory is pointed to the actual directory in the Z: drive vortex bugs out and says the staging folder must be in Z:\home\ which it cannot due. Putting the staging folder using the Z: letter which you have to if the games directory is Z it’ll tell you that the staging folder is not on the same drive.
to fix this change the games directory to J:\Skyrim Special Edition\
Then use Vortexe suggested path for the staging folder which will probably be J:\Vortex-Mods

I have skyrim installed on an external drive, but going to my E drive has nothing. Its completely empty. Do i have to install Skyrim in desktop mode? Re do the Pikdun set up and uninstall vortex(i already did that once since this issue has arised) I had mods working earlier.

J: and K: are the right drives.

J: and K: are the right drives.

Thank you, it was in my K drive. I got hardlink deployment working now

J: and K: are the right drives.

Do have a question.
I ended up using DeckMTP to just transfer all the Wither III mods which was annoying but works. I tried using STL vortex as well as your Vortex-Linux to install mods for Witcher 3. With your vortex Linux it only gave me the option for Symlinks which didn’t work. It never created any symlinks and said that all the mods were redundant well because it didn’t even actually put any files there lol.
With STL, it allowed me to do Hardlink, Symlink, and the experimental Move deploy. However all of those failed or had the same issue.
It would seem for whatever reason when it comes to Witcher 3 Vortex doesn’t have permissions or something to deploy the files in any kind of way. Skyrim works fine, haven’t tried any other games but just wondering if there’s anything to be done for this.

Not sure, not familiar with Witcher 3 modding.
Entirely possible Vortex is doing something silly, though.

Not sure, not familiar with Witcher 3 modding.
Entirely possible Vortex is doing something silly, though.

Hmm, yeah Witcher 3 is a little different. Basically deploys games into the Mods folder, sets some config files on the Bin folder, and also has a Script merger to merge script files that overlap eachother from mods.

Sounds like the original issue was solved.
I'll close this and make a placeholder ticket for tracking the Witcher 3 issues.


[Universal bots] Fixing adaptive card response type for adaptiveCard/action

The response type value is incorrect in the response format for adaptiveCard/action. Fixing the same with this change

Docs Build status updates of commit 65ae867:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/Sequential-Workflows.md
:white_check_mark:Succeeded
View

msteams-platform/task-modules-and-cards/cards/Universal-actions-for-adaptive-cards/User-Specific-Views.md
:white_check_mark:Succeeded
View

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


Implement Excel_Files and Excel_Connection

This task is automatically imported from the old Task Issue Board and it was originally created by James Dunkerley.
Original issue is here.

Allow connection to a collection of Excel files:
Excel_Options.Excel_Files path:(File|Text) recursive:Boolean=False patterns:(Vector Text)=[”*.xlsx”, ”*.xlsm”, ”*.xls”]

Ability to connect to a single workbook or a folder of workbooks.
Users can provide either a file or folder path.
The user can choose to scan subfolders using the recursive option.
By default, it will look at standard Excel extensions but user can set patterns as they wish.

The Connection API would be mapped as follows:

Database ==> Excel filename.
Schema ==> Not used - Nothing.
Table ==> Worksheet name or range name
The query function will return an Unsupported_Operation error.
The read method will not support SQL but will allow:

Table names.
Range names.
Range address.
Otherwise an Illegal_Argument error should be returned.

Blockers:
#183578491 blocked

Separating off the directory connection work from this. But a new Excel_Workbook covers the other behvaiors.


DHCPv6 Vlan priority

Certain ISP's, Orange France for example use a VLAN, it is a requirement that the dhcpv6 packets are sent with a different priority, in Orange Frances case it is 6.
This PR allows the user to select whether or not to enable the setting of the VLAN priority for dhcpv6 packets and the value to use.
The is then used in the filter.lib.inc to add the extra set-prio value when the fillter is created.
This PR requires the the sysctl net.link.vlan.mtag_pcp be set to 1.

Maybe it's better here to have a "do not modify" default setting (empty) and otherwise set 0-7 :)

Indeed... Now why didn't I think of that. :)

@marjohn56 thanks, I've pulled this in and will rework as suggested

https://github.com/opnsense/core/commit/ba725881f895db5908dee0ac36c07748229af917
https://github.com/opnsense/core/commit/617dcaee88c11eedcd91e9f783048257bf2db566
Now if we have the same for dhclient on IPv4 we are almost done :)


1.14.4 Does not reflect twitch chat ingame

Hey your 1.14.4 version of the mod does not get messages ingame from the twitch chat but you can send messages to twitch from the ingame chat fine.

Which version of the mod are you using?

I'm going to close this issue because it's been 3 months and you haven't responded. If you are down to reply anytime just reopen it.

I'm going to close this issue because it's been 3 months and you haven't responded. If you are down to reply anytime just reopen it.


Bug: IMartenQueryable doesn't work well with Projection.

Spot the difference:

The above screenshot queries the MemberKycVerificationStep with the Marten's Linq IQueryable implementation.

This one uses System.Linq AsEnumerable().
This is a new issue spun from my last issue when I decided to try out alternatives.

@kacey90 Hey, you've really got to provide more detail about this for us to have any possible chance of addressing your issue. I can't even see what your queries were. And can you please paste in code and/or exception messages instead of screen shots next time? That would be much more helpful.

@jeremydmiller Forgive me.
var results = await _querySession
    .Query<MemberKycVerificationStep>()
    .Where(x => x.MemberId == request.UserId)
    .Select(x => new MemberKycVerificationStepDto
    {
        KycVerificationStepId = x.KycVerificationStepId,
        Name = x.KycVerificationStep.Name,
        DateUpdated = x.DateUpdated,
        Status = x.Status,
        MemberId = x.MemberId
    })
    .ToPagedListAsync(request.PageIndex, request.PageSize, cancellationToken);
return await Result<IPagedList<MemberKycVerificationStepDto>>.SuccessAsync(results);

The above snippet returns Name of MemberKycVerificationStepDto as null as shown in the first screenshot.
    .Query<MemberKycVerificationStep>()
    .Where(x => x.MemberId == request.UserId)
    .AsEnumerable()
    .Select(x => new MemberKycVerificationStepDto
    {
        KycVerificationStepId = x.KycVerificationStepId,
        Name = x.KycVerificationStep.Name,
        DateUpdated = x.DateUpdated,
        Status = x.Status,
        MemberId = x.MemberId
    })
    .ToList();

This on the other hand, works for my requirement where the Name parameter of MemberKycVerificationStepDto gets the expected value as displayed in second screenshot.
I hope it's clear enough.

@kacey90 I'm sorry, I really just don't have enough bandwidth to parse out exactly what it is you're trying to say isn't working. Can you do this through a failing test maybe?
At a guess, I'm not sure if         Name = x.KycVerificationStep.Name, would work as a deeply nested member. I think so, but not sure. Might need a real reproduction to help with this. As a workaround, you can always call ToList() before Select() so that it's done in memory.

And sorry, but trying to repro this, I get a "works on my box"


Solution-Checker two robots carrying same shelf

The a-domain solution checker is not detecting the following case:
Instance:
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
%Grid size X: 2
%Grid size Y: 2
%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
#program base.

%init
init(object(node, 2), value(at, (2, 1))).
init(object(node, 4), value(at, (2, 2))).
init(object(node, 3), value(at, (1, 2))).
init(object(node, 1), value(at, (1, 1))).
init(object(pickingStation,1), value(at,(1,1))).
init(object(robot,1), value(at,(1,2))).
init(object(robot,2), value(at,(2,1))).
init(object(shelf,1), value(at,(2,2))).
init(object(product,1),value(on,(1,1))).
init(object(order,1),value(pickingStation,1)).init(object(order,1),value(line,(1,1))).

Plan:
occurs(object(robot,1),action(move,(1,0)),1).
occurs(object(robot,1),action(pickup,()),2).
occurs(object(robot,2),action(move,(0,1)),3).
occurs(object(robot,1),action(move,(-1,0)),3).
occurs(object(robot,2),action(pickup,()),4).
occurs(object(robot,1),action(move,(0,-1)),4).
occurs(object(robot,2),action(putdown,()),5).
occurs(object(robot,1),action(deliver,(1,1,1)),5).

no mistake is detected even though the same shelf is picked up twice.
when you add the statement #show holds(object(robot, R), value(carries, S), T): holds(object(robot, R), value(carries, S), T).
You get
holds(object(robot,1),value(carries,1),2) holds(object(robot,1),value(carries,1),3) holds(object(robot,1),value(carries,1),4) holds(object(robot,2),value(carries,1),4) holds(object(robot,1),value(carries,1),5)

At timestep 4 robot 1 and 2 are carrying the shelf 1.
There is missing a mechanism for updating the shelf positions after pickup and putdown.

Thanks for pointing this out! This was caused by a typo, fixed in d54a0e2.


Bump manifests for v0.6.0-alpha.0

Release note:
NONE

/test all

/override pull-cert-manager-chart

Chart version on upstream/release-0.6 : v0.6.0-dev.4
New chart version:  v0.6.0-alpha.0
Linting 'contrib/charts/cert-manager/Chart.yaml'...
ERROR: Chart version not ok. Needs a version bump.

Seems like our chart verify script doesn't like this version bump - manually merging.


Add minimal OpenQASM dialect support in MLIR

Context:
Adding a native representation of the OpenQASM standard in MLIR.
Description of the Change:

Add the OpenQASM dialect with support for three operations:

Qubit allocation
RY gate
CNOT gate

Add support for a translation pass to produce a program in OpenQASM format from the IR.

Benefits:
Having native OpenQASM support in MLIR will, for example, help facilitate the translation of an OpenQASM text file into and out of MLIR, enable transformations on OpenQASM programs, and more generally improve inter-operability between tools.
Possible Drawbacks:
Only limited support for now.
Related GitHub Issues:
Resolves #873

This is my initial attempt at resolving issue #873. At the moment, the OpenQASM dialect I have implemented is not "aware" of what any of the operations do (qubit allocation, RY gate and CNOT gate); I have assumed this was not required for the issue at hand. The MLIR -> OpenQASM translation pass simply walks through the IR operations and prints to the console the equivalent OpenQASM statements (as suggested in the original issue description). However, because of the way the MLIR tools are implemented, this also prints the original IR after it, since the operations were not modified in any way. I'm curious if there's a solution to this.

Hi @joeycarter, thank you for the submission! I will have a look at it shortly :)

Nice work!! I don't really have any comments so this is great as is 🎉

However, because of the way the MLIR tools are implemented, this also prints the original IR after it, since the operations were not modified in any way. I'm curious if there's a solution to this.

Good point, this is just how the opt tool works I think. If you really wanted to you could probably output the openqasm to a different stream, and then redirect the regular opt output to /dev/null. But I think its fine.

Thanks @dime10! In that case I'll leave it as is.


Add support for Rocky linux distribution

Doesn't work for me:
[root@scw-elastic-brahmagupta ~]# dnf install -y wireguard-dkms
Copr repo for wireguard owned by jdoss                                                                     13 kB/s | 3.3 kB     00:00
Error:
 Problem: conflicting requests
  - nothing provides dkms needed by wireguard-dkms-1:1.0.20210606-2.el8.noarch
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

Doesn't work for me:
[root@scw-elastic-brahmagupta ~]# dnf install -y wireguard-dkms
Copr repo for wireguard owned by jdoss                                                                     13 kB/s | 3.3 kB     00:00
Error:
 Problem: conflicting requests
  - nothing provides dkms needed by wireguard-dkms-1:1.0.20210606-2.el8.noarch
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

you need dkms to install wireguard if you have a lxc vps this is not possible you need a kvm vps

I've managed to run installation script on a rocky linux AWS EC2 istance with no problem at all.
it works like a charm.
Rocky is bug to bug compatible with CentOS 8 so the script does the OS check without issues, as in rocky's /etc/ dir you have the "centos-release" file.
the only error that did absolutely not affect installation or performance was an error raised by line 147 on the installation script.
Rocky version i tested was 8.4, and the error states:
"[[: 8.4: syntax error: invalid arithmetic operator (error token is ".4""

That would be this line:
elif [[ ${OS} == 'rocky' ]]; then

This PR still doesn't work for me:
+ dnf install -y wireguard-dkms
Copr repo for wireguard owned by jdoss                                                            12 kB/s | 3.3 kB     00:00
Error:
 Problem: conflicting requests
  - nothing provides dkms needed by wireguard-dkms-1:1.0.20210606-2.el8.noarch
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

[root@scw-confident-shtern ~]# cat /etc/os-release
NAME="Rocky Linux"
VERSION="8.4 (Green Obsidian)"
ID="rocky"
ID_LIKE="rhel fedora"
VERSION_ID="8.4"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Rocky Linux 8.4 (Green Obsidian)"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:rocky:rocky:8.4:GA"
HOME_URL="https://rockylinux.org/"
BUG_REPORT_URL="https://bugs.rockylinux.org/"
ROCKY_SUPPORT_PRODUCT="Rocky Linux"
ROCKY_SUPPORT_PRODUCT_VERSION="8"

This PR still doesn't work for me:
+ dnf install -y wireguard-dkms
Copr repo for wireguard owned by jdoss                                                            12 kB/s | 3.3 kB     00:00
Error:
 Problem: conflicting requests
  - nothing provides dkms needed by wireguard-dkms-1:1.0.20210606-2.el8.noarch
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

[root@scw-confident-shtern ~]# cat /etc/os-release
NAME="Rocky Linux"
VERSION="8.4 (Green Obsidian)"
ID="rocky"
ID_LIKE="rhel fedora"
VERSION_ID="8.4"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Rocky Linux 8.4 (Green Obsidian)"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:rocky:rocky:8.4:GA"
HOME_URL="https://rockylinux.org/"
BUG_REPORT_URL="https://bugs.rockylinux.org/"
ROCKY_SUPPORT_PRODUCT="Rocky Linux"
ROCKY_SUPPORT_PRODUCT_VERSION="8"

[rocky@ip-172-31-11-255 ~]$ cat /etc/os-release NAME="Rocky Linux" VERSION="8.4 (Green Obsidian)" ID="rocky" ID_LIKE="rhel centos fedora" VERSION_ID="8.4" PLATFORM_ID="platform:el8" PRETTY_NAME="Rocky Linux 8.4 (Green Obsidian)" ANSI_COLOR="0;32" CPE_NAME="cpe:/o:rocky:rocky:8.4:GA" HOME_URL="https://rockylinux.org/" BUG_REPORT_URL="https://bugs.rockylinux.org/" ROCKY_SUPPORT_PRODUCT="Rocky Linux" ROCKY_SUPPORT_PRODUCT_VERSION="8"
rocky@ip-172-31-11-255 ~]$ uname -a Linux ip-172-31-11-255.us-east-2.compute.internal 4.18.0-305.3.1.el8_4.x86_64 #1 SMP Thu Jun 17 07:52:48 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

This PR still doesn't work for me:
+ dnf install -y wireguard-dkms
Copr repo for wireguard owned by jdoss                                                            12 kB/s | 3.3 kB     00:00
Error:
 Problem: conflicting requests
  - nothing provides dkms needed by wireguard-dkms-1:1.0.20210606-2.el8.noarch
(try to add '--skip-broken' to skip uninstallable packages or '--nobest' to use not only best candidate packages)

[root@scw-confident-shtern ~]# cat /etc/os-release
NAME="Rocky Linux"
VERSION="8.4 (Green Obsidian)"
ID="rocky"
ID_LIKE="rhel fedora"
VERSION_ID="8.4"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Rocky Linux 8.4 (Green Obsidian)"
ANSI_COLOR="0;32"
CPE_NAME="cpe:/o:rocky:rocky:8.4:GA"
HOME_URL="https://rockylinux.org/"
BUG_REPORT_URL="https://bugs.rockylinux.org/"
ROCKY_SUPPORT_PRODUCT="Rocky Linux"
ROCKY_SUPPORT_PRODUCT_VERSION="8"

@angristan  Sorry! with the first comment i meant with your script: "angristan/wireguard-install", not this PR one.
Being Rocky 1:1 with CentOS, there is no need to add "rocky linux support" as is already supported by original release.
I'm new here... apologize me! @

Oh, ok, I see. thanks


Remove blocked bot crawler from v5

Tracking for before we go stable with v5—we'll need to update robots.txt.

Adding to both v5-stable and v5-beta in case we do what we did with v4 and put the beta docs on the main site.

For reference https://github.com/twbs/bootstrap/commit/723cd8d12bdd48f389234b5fc40c3b1789807b4d#diff-d7f6c1ca03bd330a561f8e674d577ee8
We'll need to revert all those changes when we are ready to move the v5 docs to the main repo.

I want to highlight the common issue I am facing.
The recent version of Bootstrap CSS as specified in this link is either behaving badly or not working properly.
Please pay some attention to this.

HI
I want to highlight the issue I am facing with the current CSS link on the Bootstrap website, which is as given below:

Using this NO CSS files are loaded onto the browser, I have checked this on Google Chrome and Microsoft Edge.
Please look into this matter.

Sorry to bother, just wanted to ask is there any schedule for v5 release date?

Sorry to bother, just wanted to ask is there any schedule for v5 release date?

https://github.com/twbs/release


Modules that import .ts files must be imported in nuxt.config.ts

Environment

Operating System: Windows_NT
Node Version:     v16.17.0
Nuxt Version:     3.1.2
Nitro Version:    2.1.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      -
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/nuxt-starter-jvrsel?file=nuxt.config.ts
Describe the bug
Hi :wave: ,
I ran into an issue with my module. It need to dynamically import .ts files within the module.
Setting the module as a string in modules within nuxt.config.ts does not work when the module tries to import a ts file with await import(/*   */).
The only workaround is to directly import the module within the nuxt.config and add it into the modules array.
Why is there a difference between declaring the module with the module name and declaring the module by directly importing it ?
Additional context
No response
Logs
Unknown file extension ".ts" for ".........."

If you check in the module author guide and the playground in the modules repo, you can just do import it without an extension:
https://github.com/nuxt/starter/blob/module/playground/nuxt.config.ts

Maybe did i badly explain my issue :joy: . Did you check my reproduction?
What i'm saying is that within my module here, i'm doing a dynamic import (without file extension) by using await import(/* */) but if my module is declared with its package name thourhg a string :

// https://v3.nuxtjs.org/api/configuration/nuxt.config
export default defineNuxtConfig({
  modules: [
     'nuxt-runtime-compiler', // this don't work
  ],
});

It won't work. This also triggers a
[ERR_UNSUPPORTED_ESM_URL_SCHEME]: Only file and data URLs are supported by the default ESM loader. On Windows, absolute paths must be valid file:// URLs. on windows even if i use filePathToURL then thePath.href
However directly importing it allows the module to import TS files
import nuxtRuntimeCompiler from 'nuxt-runtime-compiler';

// https://v3.nuxtjs.org/api/configuration/nuxt.config
export default defineNuxtConfig({
  modules: [
    nuxtRuntimeCompiler, // this works,
   //  'nuxt-runtime-compiler', // this don't
  ],
});

Without digging into it too deeply, I believe this is because it uses jiti to pass the module when it's a string, which has a slightly different way of resolving than the node runtime.
Not sure of the exact fix, might need to include the .ts.
Saying that, not exactly sure why you're trying to support build time config from app.config. Seems like you should just use the nuxt.config or am I missing something

We shouldn't support importing from a runtime file like ~/.nuxt/app.config into a build-time module. I'm more than happy to help with your particular module, separately.
As far as this issue is concerned, as long as your module is .ts it can import other .ts files. Otherwise we assume it's a built file and we use native ESM loading.


Repository is massive (> 70 MB)

Problem description
Cloning the repository takes a long time. I don't see why this repo is any larger than 10 MB. This slow download adds friction for contributors.
Steps to reproduce
 ➤ git clone git@github.com:callemall/material-ui.git
Cloning into 'material-ui'...
remote: Counting objects: 52693, done.
remote: Compressing objects: 100% (2/2), done.
Receiving objects:  90% (47878/52693), 64.71 MiB | 91.00 KiB/s

@matthewoates It's probably because of the gh-pages branch. I suggest you ( and probably others reading this to only clone master or next )
git clone -b master --single-branch https://github.com/callemall/material-ui.git

That's a very good point.
I'm wondering how practical it's to ask users to clone a single branch when starting contributing. Wouldn't it be better to move the docs builds history into a different repository?

Would it be better to move the docs builds history into a different repository? Yeah it would :+1: But I think the tooling will need a lot of updating.

I have noticed that we output the source map. Removing them would be a huge space saving.

I'm wondering how practical it's to ask users to clone a single branch when starting contributing.

git clone --depth 1 ?

$ git clone https://github.com/callemall/material-ui.git
Cloning into 'material-ui'...
remote: Counting objects: 65552, done.
remote: Compressing objects: 100% (49/49), done.
remote: Total 65552 (delta 19), reused 0 (delta 0), pack-reused 65501
Receiving objects: 100% (65552/65552), 81.07 MiB | 675.00 KiB/s, done.
Resolving deltas: 100% (42589/42589), done.
Checking out files: 100% (1669/1669), done.

$ du -ks material-ui
94640	material-ui

$ git clone --single-branch https://github.com/callemall/material-ui.git
Cloning into 'material-ui'...
remote: Counting objects: 50889, done.
remote: Compressing objects: 100% (4/4), done.
remote: Total 50889 (delta 0), reused 4 (delta 0), pack-reused 50885
Receiving objects: 100% (50889/50889), 15.69 MiB | 1.03 MiB/s, done.
Resolving deltas: 100% (33028/33028), done.
Checking out files: 100% (1669/1669), done.

$ du -ks material-ui
27548	material-ui

git clone --depth 1 https://github.com/callemall/material-ui.git
Cloning into 'material-ui'...
remote: Counting objects: 1872, done.
remote: Compressing objects: 100% (1627/1627), done.
remote: Total 1872 (delta 213), reused 1436 (delta 167), pack-reused 0
Receiving objects: 100% (1872/1872), 2.83 MiB | 509.00 KiB/s, done.
Resolving deltas: 100% (213/213), done.
Checking out files: 100% (1669/1669), done.

$ du -ks material-ui
12788	material-ui

However the wheels are in motion to move the gh-pages branch out of the material-ui repo, which will knock a chunk off the full repo download.

Okay, the move gh-pages move was done:
git clone https://github.com/callemall/material-ui.git
Cloning into 'material-ui'...
remote: Counting objects: 64219, done.
remote: Compressing objects: 100% (3/3), done.
remote: Total 64219 (delta 0), reused 0 (delta 0), pack-reused 64216
Receiving objects: 100% (64219/64219), 19.62 MiB | 523.00 KiB/s, done.
Resolving deltas: 100% (41780/41780), done.
Checking out files: 100% (1669/1669), done.

du -ks material-ui
31684	material-ui

(30.9MB down from 92.4MB yesterday).
I'd still recommend the use of git clone --depth 1 for the smallest possible download, but at least the full download is much lighter now.

Great!


[Improvement]: Invalid JSON in response example

What can we improve?
The JSON example in Event Format, is not valid JSON.
This makes the response confusing.
Validation tool: https://jsonformatter.curiousconcept.com/#

Which page needs an update?
https://www.docs.developers.amplitude.com/analytics/apis/dashboard-rest-api/#event-format

Thanks for reporting @philippb -- it should be fixed on the site any minute now.


Refactor muts away, enhance ci builds

refactor away unnecessary &mut self from ipfs::Ipfs blockstore calls
enable building of examples and tests in the same phase
require Cargo.lock to be up to date cargo build --locked

This is split and extended from #147, where I'll be dropping the first commit soon-ish.

oh well, forgot my fixup commit ... I'll let the CI run complete/fail before rebasing.

Forgot to start by cargo fmt as the master has a few issues at the moment.

And a transient test failure in the readme-doctest.

If this is ok, I think we should rebase and merge or just merge.


openjfx11: 11.0.3+1 -> 11.0.11+1, openjdk{11,12,13,14}: enableJavaFX by default

Description of changes

Continuation of https://github.com/NixOS/nixpkgs/pull/203958.
Upgrading openjfx11 to the latest version available seems to allow WebKit to be build with python3. Since now it works again, we can re-enable openjfx11 in all openjdk versions that depend on it.
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@emmanuelrosa @GaetanLepage Both bisq-desktop and pdfsam-basic seems to work with the changes in this PR (didn't mess with both too much, but at least the interface opens fine).

Moving this back to draft just to test one thing.

Moving this back to draft just to test one thing.

My test didn't work, reverting back as it was.
Open for review/merge again.

Seems to be 2019-04-13 (11.0.3+1) → 2021-01-27 (11.0.11+1)
https://openjdk.org/groups/vulnerability/advisories/
Particularly this merge request should help with the following advisories:

https://openjdk.org/groups/vulnerability/advisories/2020-04-14
https://openjdk.org/groups/vulnerability/advisories/2020-07-14

Potentially not patched:

https://openjdk.org/groups/vulnerability/advisories/2021-10-19 (CVE-2021-3517 and CVE-2021-3522)

Seems to be 2019-04-13 (11.0.3+1) → 2021-01-27 (11.0.11+1)
Log: https://hg.openjdk.java.net/openjfx/11-dev/rt

https://openjdk.org/groups/vulnerability/advisories/
Particularly this merge request should help with the following advisories:

https://openjdk.org/groups/vulnerability/advisories/2020-04-14
https://openjdk.org/groups/vulnerability/advisories/2020-07-14

Potentially not patched:

https://openjdk.org/groups/vulnerability/advisories/2021-10-19 (CVE-2021-3517 and CVE-2021-3522)

There is the openjdk/jfx11u repository that is still receiving updates. However changing to it broke the build, and I am not sure if I am that interested in fixing it.
I kinda of want to switch all our OpenJDK related builds to the support repositories since we are probably dangerous out of date.

Going to merge as is. I will open an issue later about the whole situation in our OpenJDK derivations, but this is not related to #201859 so just to unblock it I am merging.

Yeah, separate seemed better. Thanks for this update. :3

Afaik since Java 11 JavaFX is not packaged with the jdk/jre anyways. I wonder if it would make more sense remove the inclusion from jdk package as well as it reflects how JavaFX is handled nowadays as a seperated library.

Afaik since Java 11 JavaFX is not packaged with the jdk/jre anyways. I wonder if it would make more sense remove the inclusion from jdk package as well as it reflects how JavaFX is handled nowadays as a seperated library.

Maybe we should disable by default for all OpenJDK releases them. This also has the advantage of avoiding security issues bought by the WebKit from OpenJFX. Individual derivations can override it.
However, we need to do so in a way that is properly documented in the release notes.


Release type on Windows

This is how I build SQLiteCpp on macOS and Ubuntu:
cmake -DCMAKE_BUILD_TYPE=Release -DSQLITECPP_BUILD_EXAMPLES=OFF -DSQLITECPP_BUILD_TESTS=OFF ..
cmake --build . --config Release

This builds in RELEASE mode, however, on Windows it seems its discards CMAKE_BUILD_TYPE and instead of RELEASE, DEBUG is built.
Am i missing something here? How do i build it on RELEASE on Windows?
-- CMake version: 3.16.2
-- Project version: 2.99
-- CMAKE_CXX_COMPILER 'C:/Program Files (x86)/Microsoft Visual Studio/2019/Enterprise/VC/Tools/MSVC/14.24.28314/bin/Hostx64/x64/cl.exe' 'MSVC' '19.24.28314.0'
-- CMAKE_CXX_FLAGS                '/DWIN32 /D_WINDOWS /W3 /GR /EHsc'
-- CMAKE_CXX_FLAGS_DEBUG          '/MDd /Zi /Ob0 /Od /RTC1'
-- CMAKE_CXX_FLAGS_RELEASE        '/MD /O2 /Ob2 /DNDEBUG'
-- CMAKE_CXX_FLAGS_RELWITHDEBINFO '/MD /Zi /O2 /Ob1 /DNDEBUG'
-- CMAKE_CXX_FLAGS_MINSIZEREL     '/MD /O1 /Ob1 /DNDEBUG'
-- Compile sqlite3 from source in subdirectory
-- Found PythonInterp: C:/hostedtoolcache/windows/Python/3.6.8/x64/python.exe (found version "3.6.8")
-- Could NOT find cppcheck
-- SQLITECPP_RUN_DOXYGEN OFF
-- SQLITECPP_BUILD_EXAMPLES OFF
-- SQLITECPP_BUILD_TESTS OFF
-- Configuring done
-- Generating done
CMake Warning:
  Manually-specified variables were not used by the project:

    CMAKE_BUILD_TYPE

Ahhh! my main folder was Release. This contains debug files and this folder has subfolder called Release. This folder contains the release binaries. I don't know why this is different on windows.
Sorry for misleading issue. Just ignore this. Closing...


Create Hello_World_Hect.py

hello world program

You can checkout our Idea-Arca, the project idea bank for any languages, for open-source contributions.
The PR from this repo is being accepted in hacktoberfest2023
Happy Contributing season!
Keep up the good work!


fix: added reasonable default proxy-buffer-size

Which problem does the PR fix?

As @Ben-Sheppard  noted in #889 the default proxy-buffer-size can sometimes cause issues in our web components. This has led to some issues internally as well as a few support tickets and ask-distribution questions. @jonathanlukas @plungu  thank you for your efforts on those.
What's in this PR?
This PR adds a default proxy-buffer-size to our default annotations as an attempt to warn users that if they want to use their own ingress, hopefully they will make sure to set this annotation. And if they happen to use ingress-nginx, it'll hopefully work out of the box.

Checklist
Please make sure to follow our Contributing Guide.

Before opening the PR:

[x] There is no other open pull request for the same update/change.
[x] The commits follow our Commit Guidelines.
[x] Tests for charts are added (if needed).
[x] The main Helm chart and sub-chart are updated (if needed).
[x] In-repo documentation are updated (if needed).

After opening the PR:

[x] Did you sign our CLA (Contributor License Agreement)? It will show once you open the PR.
[x] Did all checks/tests pass in the PR?

@jessesimpson36, you probably just need to run make go.test-golden-updated in the repo's root to update the golden file.

I've quickly updated the golden files to merge the PR.

Thanks, sorry I fell off of this topic!