[Feature Request]: Optional role to spawn pods from notebooks, used with Spark workloads

Feature description
The ServiceAccount used to run the notebooks does not have right to spawn Pods. This feature would be useful to directly launch Spark workloads from a notebook, which some teams are used to do in their current environments.
Details:

Spark supports kubernetes to directly launch workloads. You can target your spark-submit command to a kubernetes cluster, which will schedule executor pods, do the computation, and remove the pods when finished. The Spark instance running in the notebook acts as the driver for the workload.
This functionality requires rights to create/delete Pods and Services. However, the ServiceAccount under which the Notebook container is running does not have those rights.
The requested feature would add another Role to the notebook ServiceAccount, enabling those capabilities.
The drawback is that any notebook user would be able to create Pods in the namespace by invoking the kubernetes API, so this must be an admin-selectable feature.

Proposed implementation:

Create a Role with needed capabilities. This can be used as a base as it's exactly what the Spark operator is doing: https://github.com/opendatahub-io-contrib/spark-on-openshift/blob/main/spark-operator/spark-rbac.yaml
Create a toggle in admin to enable this role or not. As the drawback of the method is that people would be able to schedule Pods, admins must be able to allow this or not. The toggle would create/delete the associated RoleBinding. For DSP, this could be a per-project toggle.

Describe alternatives you've considered
No response
Anything else?
This would bring back some of the features we previously had in ODH with Spark notebooks, that customers are now lacking. Instead of spawning a pseudo-permanent Spark attached to a notebook, this would be more dynamic, with the Spark cluster being spawned when the command is submitted.

Hi @guimou is the service account you are mentioning notebook-controller-service-account?
If so, this would be an issue for either the kubeflow or odh-manifest repository, we don't control how the logic behind the notebook controller works and this repository should only hold issues related to the dashboard logic.
I would defer to @andrewballantyne either we keep this ticket or not.

I agree to some extent with @lucferbux -- The power of the Notebook Controller and any service account it has is not really within' the purview of our set of configurations. The closest we get to the Notebook Controller is by creating their custom Notebook CRs... which have a lifecycle we "sorta" know but don't interrupt or influence in anyway directly.
@VaishnaviHire / @LaVLaS I think this is still ODH related, but not Dashboard per se. Shall we move this issue over to another repo? If so, is that just odh-manifests or is there a better one? I suppose this could be a core kubeflow thing -- but I think this is a desire from our deployments and the permissions we grant to the service account that controls the pod logic. Open to suggestions where to move this to -- just don't think it's Dashboard.

I don't think our default functionality should allow any ODH user the ability to create a Pod in the notebook namespace, i think this request is to allow the notebook ServiceAccount the ability to create pods in the same way that we create the jupyter-nb-<USERNAME>-notebook-view rolebinding in notebookUtils

Discussed a bit with Landon and Vaishnavi on slack -- it's still my opinion this is outside the responsibilit of the Dashboard, but they have informed me the alternative is not great. Putting this on the backlog for now.

You can simply close this issue to clean up the list if you prefer. People have the recipe on how to do that by themselves should they wish to do so, it will do for the moment.

Sounds good, thanks @guimou


[mhlo] modify rng algrothim attr declare as enum

modify rng algrothim attr declare as enum

Thanks for the attribute enum change.
This rng op contrary to the other two doesn't have side effects/is stateless. It takes state and returns state but the op itself has no internal state or mutation given an input state it should produce the same output state so if one wants different values one has to thread the state around. Do you have an example where CSE is changing the result?

I will add custom builder function to fix compile error.
Consider the following IR if the RNG is really random, whether the result of the two random times is the same is what we expect? I am not particularly clear about the semantics of this Operation, if there are mistakes, thank you for correcting.
// RUN: mlir-hlo-opt %s -cse -allow-unregistered-dialect | FileCheck %s

func.func @rng_bit_generator(%arg0: tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>) {
  %0 = mhlo.constant dense<[10, 12]> : tensor<2xui64>
  %1, %2 = "mhlo.rng_bit_generator"(%0) {rng_algorithm = #mhlo<"algorithm DEFAULT">} : (tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>)
  %3, %4 = "mhlo.rng_bit_generator"(%0) {rng_algorithm = #mhlo<"algorithm DEFAULT">} : (tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>)
  func.return %1, %2, %3, %4 : tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>
}

I will add custom builder function to fix compile error.
Consider the following IR if the RNG is really random, whether the result of the two random times is the same is what we expect? I am not particularly clear about the semantics of this Operation, if there are mistakes, thank you for correcting.
// RUN: mlir-hlo-opt %s -cse -allow-unregistered-dialect | FileCheck %s

func.func @rng_bit_generator(%arg0: tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>) {
  %0 = mhlo.constant dense<[10, 12]> : tensor<2xui64>
  %1, %2 = "mhlo.rng_bit_generator"(%0) {rng_algorithm = #mhlo<"algorithm DEFAULT">} : (tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>)
  %3, %4 = "mhlo.rng_bit_generator"(%0) {rng_algorithm = #mhlo<"algorithm DEFAULT">} : (tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>)
  func.return %1, %2, %3, %4 : tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>
}

We will get:
module {
  func @rng_bit_generator(%arg0: tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>) {
    %0 = mhlo.constant dense<[10, 12]> : tensor<2xui64>
    %output_state, %output = "mhlo.rng_bit_generator"(%0) {rng_algorithm = #mhlo<"algorithm DEFAULT">} : (tensor<2xui64>) -> (tensor<2xui64>, tensor<10x12xui32>)
    return %output_state, %output, %output_state, %output : tensor<2xui64>, tensor<10x12xui32>, tensor<2xui64>, tensor<10x12xui32>
  }
}

We don't expect it to be different with same input state no:
"The output is guaranteed to be a deterministic function of the initial state but it is not guaranteed to be deterministic between backends and different compiler versions."
https://www.tensorflow.org/xla/operation_semantics#rngbitgenerator
If you wanted different numbers you have to use returned updated state from output of one as input to the other.

Nice, we may have to change the Python bindings too, if you are able to that would be great (JAX uses the Python bindings and I don't know off the top of my head if this will change the python build signature).
@hawkinsp for visibility.

Yes, I suspect this code will need an update:
https://github.com/google/jax/blob/39ac9faf61e98169d999d33418ca58cecaccc843/jax/_src/lax/lax.py#L4119
unless the bindings still accept i32 attrs in that place for now.

Yes, I will try to modify pybind.Thanks for your remind.

Note: you don't necessarily have to maintain backward compatibility for JAX specifically. There is a way we manage breaking changes to MHLO at the moment wrt to JAX. It requires two PRs:
a) a TF PR:

make your breaking MHLO change in the TF tree.
increment this version number in the same PR https://github.com/tensorflow/tensorflow/blob/1249b029f84c7424eb059710e6c1fb1dce713d31/tensorflow/compiler/xla/python/xla_client.py#L49

b) a JAX PR, in which you change the code that will be broken to look like this:
if jax._src.lib.mlir_api_version >= 1234:
  # new code
else
  # old code

and then we submit (b) before (a).
This process could probably be improved, but it does work. Also, it's not necessarily your obligation to do this unless you want to: we can also do the work as part of the TF PR merge process (one thing we do when merging a TF PR is running a number of downstream tests, which would catch this kind of issue and the person doing the merge would make the fix.)

+1 to what Peter said (nice to have but not blocker)

I see.  I will submit a PR for Jax to compatibily current mhlo api version. then modify and submit this PR. Thanks.

I have submitted the PR in Jax Repo.

Can you check tests:
//tensorflow/compiler/mlir/xla/tests:legalize-tf.mlir.test
//tensorflow/compiler/mlir/xla/tests/translate:export.mlir.test
//tensorflow/compiler/mlir/xla/tests/translate:import.hlotxt.test
?

Yes. I just run with command:
bazelisk test tensorflow/compiler/mlir:all --cache_test_results=no
and all pass, It seem some test no run.

I have resubmitted jax's patch, but it has not been merged yet.Integration testing may also be affected.

SG, I didn't see any of those failing here (I mentioned the other set yesterday and those you seem to have fixed but kokoro pending).

Thanks.

Some parameters still refer to std::string, I'm going to clean them all up in another patch.

I think you actually need to fix the std::string references before merging: they cannot appear in a C API.

cleanup std::string in attributes file other PR


passing front end libraries challenge without doing it

Describe the Issue
context - before on FCC site challenges (not Codepen) I would finish the course click off the screen and and the congrats box would disappear without clicking go to the next challenge, resulting in not registering completed. I just click the submit button and go to next challenge.
no Codepen account / save created for this as after the first one it had passed so I just didn't set one up. set up in VScode and copy paste into Codepen
I thought I had done about above for the front end challenges which involves completing the challenge on Codepen. on my 4th challenge, I had clicked off the FCC page to get just run the test one time and make sure I took a copy of the link. I had actually press the go to next challenge not realising that it was the 5th challenge and submitted and passed with out doing anything.
I assume that the link must be the same for all challenges and its not checking the code submitted. In my case would be blank anyway as i have no account
Affected Page
Front end libraries
Steps to Reproduce

Go to 'FCC front end libraries.'
Click on 'any challenge scroll to the bottom click the Code pen link copy link.'
Scroll down to 'bottom of the FCC front end challenge and submit '
No error just no work

Expected behavior
not to pass . i understand you don't that forcing a codepen account is not ideal.
Screenshots
No response
System

Device: PC
OS: Archlinux
Browser: Firefox
Version: 131.0.3

Additional context
No response

It relys on Academic Honesty, but also your cert could be removed if you don't supply links for working projects. With the new cert this will not be an issue anymore

thanks, just wanted to make you aware in case you weren't


[BUG] AzMon Exporter removes semantic attributes from telemetry

Library name and version
Azure.Monitor.OpenTelemetry.Exporter 1.3.0
Describe the bug
When an Activity is processed by the Azure.Monitor.OpenTelemetry.Exporter, any tags with keys in this list:
https://github.com/Azure/azure-sdk-for-net/blob/ede7138b0150024daa440819cc0008cd51f46bc1/sdk/monitor/Azure.Monitor.OpenTelemetry.Exporter/src/Internals/ActivityTagsProcessor.cs#L12C1-L69C1
are removed from the resulting telemetry. Some of the corresponding values are mapped to other AI telemetry fields, but many of them are just removed from the telemetry. For example, these fieldsare added by the GRPC request instrumentation  :

rpc.system
rpc.grpc.status_code
rpc.service
rpc.method

But then all of these fields except rpc.method are removed from the resulting telemetry in Application Insights, yielding the following output:

rpc.method

Expected behavior
Activity tags (particularly semantic tags) that are not mapped to other TelemetryItem fields show up in the TelemetryItem.Properties.
Actual behavior
All Activity tags in the s_semantics list are effectively stripped from the TelemetryItem. A few of them are mapped to TelemetryItem predefined values (good) but the others are dropped.
The only workaround is to purposefully instrument code with non-semantic tags, and/or copy the semantic values to non-semantic tags.
Reproduction Steps
This behavior is present for all types of Activity telemetry (request and dependency TelemetryItems in the exporter), but it's particularly noticeable with GRPC. Start with a GRPC sample app (eg Examples.GrpcCore.AspNetCore) and remove GrpcCore dependency, and add Azure.Monitor.OpenTelemetry.AspNetCore and enable the exporter, then view the output.
Environment
dotnet --info
.NET SDK:
Version:           8.0.302
Commit:            ef14e02af8
Workload version:  8.0.300-manifests.5273bb1c
MSBuild version:   17.10.4+10fbfbf2e
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.22631
OS Platform: Windows
RID:         win-x64
Base Path:   C:\Program Files\dotnet\sdk\8.0.302\
Host:
Version:      8.0.6
Architecture: x64
Commit:       3b8b000a0e
.NET SDKs installed:
6.0.423 [C:\Program Files\dotnet\sdk]
8.0.302 [C:\Program Files\dotnet\sdk]

Sounds like a bug, if tags are removed (without mapping to existing top level field in ApplicationInsights).

Thanks @johncrim for your efforts in reporting issues! Really appreciate it.
Thanks @TimothyMothra for the quick fixes.

Thank you @TimothyMothra !

Hi @cijothomas and @TimothyMothra - I think this should be re-opened. The fix only addresses  RPC/GRPC semantic attributes. There are a number of other attributes that are also stripped; and also can only be included in the output by using non-semantic tag names.
For example:

network.protocol.version
net.transport
enduser.role
enduser.scope

In short, I think none of the semantic attributes should be removed unless there is an explicit equivalent in the AI telemetry output.

Hi @johncrim, Thank you for all your work reporting issues.

none of the semantic attributes should be removed unless there is an explicit equivalent in the AI telemetry output.

Fully agree!
We initially expected to map all of these attributes. We've since struggled with some of OpenTelemetry's attributes stuck in Experimental. I'm seeing that quite a few of the attributes we expected to map are not currently used. I'm going to do a full audit of this list and get this corrected. I'm sorry for the trouble, but we really appreciate you.

I'm closing this issue. I believe we got the last of the attributes that were being dropped.
Our next beta release should be in early October.


Full tests replacing example tests in newly approved translations, making solutions visible in the example tests box

OK, this one is a mess:
Earlier today I saw a description of this happening in the discourse for this kata: https://www.codewars.com/kata/andys-coffee-addiction/discuss
And then when I solved the Python translation of this kata -- https://www.codewars.com/kata/simple-fun-number-62-draw-rectangle/train/python -- I saw that the full test cases were in the example tests box, including the solution code that the random tests run against.
Earlier today @nekokat did a CoffeeScript translation of my Resistor Color Codes, Part 2 kata, which I approved, and although the example tests in the Kumite are exactly as they should be, again, if you go to https://www.codewars.com/kata/resistor-color-codes-part-2/train/coffeescript you'll see that the full tests again including the solution code are visible in the Example Tests box, and clicking RUN EXAMPLES runs them.

I noticed this earlier and thought someone just messed up. This was a copypasta error on my part. I'm deploying a fix now.
Whatever translations were approved over the last day are going to need to be manually updated unfortunately :(

Actually I'm working on a script that will update the last few days automatically.


How to know database size in MB and/or each tables size in MB

Hi,
Is there a feature in DBeaver to know the database size of a mssql database? If yes, how can I do that?
Or is this a new feature request I can make?
Thank you

It is not supported for SQL Server. But it may be implemented as part of #810.


add env.close method

as mentioned here: https://github.com/openai/gym/issues/1056#issuecomment-471136134
close=True was removed a while ago, so this change just adds an 'env.close' method exactly the same as the way you were doing it, without removing the old way, so it's hopefully backwards compatible (let's test it!)

Thanks bionicles!


[Rus] Improved translation of build-a-survey-form

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for your contribution to the page! 👍
We're happy to accept these changes, and look forward to future contributions. 🎉


RTL_USER_PROCESS_PARAMETERS

Hello,
I was searching for RTL_USER_PROCESS_PARAMETERS in your project but can't find it. You did not implement it ?

It's there, it's just calling RtlUserProcessParameters. https://github.com/googleprojectzero/sandbox-attacksurface-analysis-tools/blob/master/NtApiDotNet/NtUserProcessParameters.cs#L83 in any case you shouldn't use that (unless you need to write something), you should instead call the GetUserProcessParameters method on an NtProcess object which returns an NtUserProcessParameters structure.


Override Services Address/Port

Description
This pr adds the possibility to override address and port for Liqo external services. It can be useful when using reverse proxies and/or NATs.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

[ ] Test A
[ ] Test B

/build

/test

/test

/test

/merge


Dashboard Grid Layout

In this change we're adding the ability for configurable widgets and a My Dashboard page.
The Dashboard grid uses a CSS Grid layout (defaults to 12 columns and rows). My hope is that this can also be a foundation for customizable (drag and drop and/or resizable widgets)
Monday Task

Thanks guys! Let's just merge this into #1587 to track in one PR


PR: Make pylint plugin find most local .pylintrc to given file and set cwd correctly

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

Supersedes PR #10751 by @tovogt . This should be ready to merge as-is, presuming tests pass on the CIs (as they all did locally).
Previously, the Pylint plugin (Static Code Analysis pane) would not find the correct .pylintrc configuration file (or any config file at all) when run, either in the working dir, the project dir or the home directory. Furthermore, the working directory was not set to match that of Spyder. With these improvements, which were originally implemented on my suggestion by @tovogt , the cwd is now set correctly, and more importantly the most proximate .pylintrc file is retrieved and used, smoothly integrated with Pylint's built-in search path:

.pylintrc (or pylintrc, pyproject.toml, or setup.cfg) or in the file's directory*
Any .pylintrc (etc.) in the Python module hierarchy at and above the file, at the closest level to the file*
.pylintrc (etc.) in Spyder's working directory*
.pylintrc (etc) in Spyder's active project directory (if any)*
.pylintrc (etc) named in the PYLINTRC environment variable
.pylintrc (etc) or .config/pylintrc in your home directory
/etc/pylintrc (on POSIX, obviously)

Denotes items added/modified on the search path relative to pylint's naive config

In addition, I've implemented a battery of comprehensive unit and integration tests for this functionality.
Issue(s) Resolved

Fixes #5345
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: CAM-Gerlach

@ccordoba12 @tovogt After a three hour battle with the Py2 and legacy CIs, this is finally ready for final review and merge.

@CAM-Gerlach, thanks a lot for your hard work on this one!
@dalthviz, please review it.

BTW, I realized I neglected to do so before, but for the record thanks @dalthviz and @goanpeca for your speedy reviews.

Thanks @goanpeca and @dalthviz for reviewing this and your suggestions!
@ccordoba12 ?

Thank you @CAM-Gerlach for your hard work on this one and @tovogt for the initial push to solve this problem. Great work guys!


Create a release

bitbang seems stable enough that users shouldn't have to compile from source

Wrong !
Don't work.

???

Hi.
Ticket already created, number #85
Le ven. 24 juin 2022 à 22:39, John Mcmaster @.***> a
écrit :

???
—
Reply to this email directly, view it on GitHub
https://github.com/JohnDMcMaster/open-tl866/issues/84#issuecomment-1165921481,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM5KBI5226HE27GYK7Z7XF3VQYMGJANCNFSM4M6N4Y7Q
.
You are receiving this because you commented.Message ID:
@.***>


Add to F-Droid

I would add the app also on F-Droid

This is impossible now - JustNotes uses Firebase to track analytics and for cloud backup feature.

it's planned to separate versions without google services, but now there is no alternative.

@jjewuz well if you want to fix it I would let it open :)

sad


nav [nfc]: Make navigation more cleanly testable, with WidgetRoute

As discussed when we added this first test of this kind: https://github.com/zulip/zulip-flutter/pull/249#discussion_r1289230559

Thanks, LGTM! Merged.


before_record_response DOES change real response

Versions:
python: v3.6
vcrpy: 1.11.1
Issue:
I tried using the before_record_response callback to scrub responses of sensitive auth info before they are written to disc. However, when I have a test that exercises a function that makes 2 requests, the first requesting an auth token, and the second using that auth token, I'm seeing that the actual response from the first request has been mutated.
There was an issue opened previously (https://github.com/kevin1024/vcrpy/issues/291), although a while ago, that asserted the opposite, and @kevin1024 seemed to agree and explain why the response wasn't/shouldn't be changed.
Repro:
I wrote the following script to reproduce the issue.
import json
import requests
import vcr

VCR = vcr.VCR(
    serializer='yaml',
    cassette_library_dir='cassettes',
    record_mode='once',
)

def filter_json_response_body(fields, replacement='[REDACTED]'):
    """This is a very naive scrubber, for demo purposes.
    """
    def before_record_callback(response):
        string_body = response['body']['string'].decode('utf8')
        body = json.loads(string_body)

        for field in fields:
            if field in body:
                body[field] = replacement

        response['body']['string'] = json.dumps(body).encode()
        return response

    return before_record_callback

with VCR.use_cassette(
    'it_mutates_the_response',
    before_record_response=filter_json_response_body(['text'])
) as cass:

    response = requests.get('http://numbersapi.com/42?json')
    print(response.json())

Here is my requirements.txt:
requests==2.18.4
vcrpy==1.11.1

Please let me know if I forgot to include anything!

I think there is a bug here
Here's the function:
def append(self, request, response):
    """Add a request, response pair to this cassette"""
    request = self._before_record_request(request)
    if not request:
        return
    response = self._before_record_response(response)
    if response is None:
        return
    self.data.append((request, response))
    self.dirty = True

It's ok that request gets run through _before_record_request and appended to self.data, because the request has already been made, and we just need to record it. But response is being assigned to the return value of _before_record_response, overwriting the original value, even though we want to return the original response.
I was able to deepcopy() the response in the above function and fix the issue. Tomorrow I can try whipping together a PR to fix this.


nox does not work when installed via pipx (Windows, Python 3.7.3)

There appears to be a problem with nox when run from via pipx:
>nox
nox > Running session test
nox > Creating virtualenv using python.exe in C:\Work\Scratch\n\.nox\test
nox > Command c:\users\pfmoore\.local\pipx\venvs\nox\scripts\python.exe -m virtualenv C:\Work\Scratch\n\.nox\test failed with exit code 100:
No pyvenv.cfg file
Using base prefix 'C:\\Users\\pfmoore\\AppData\\Local\\Programs\\Python\\Python37'
New python executable in C:\Work\Scratch\n\.nox\test\Scripts\python.exe
ERROR: The executable C:\Work\Scratch\n\.nox\test\Scripts\python.exe is not functioning
ERROR: It thinks sys.prefix is 'c:\\work\\scratch\\n' (should be 'c:\\work\\scratch\\n\\.nox\\test')
ERROR: virtualenv is not compatible with this system or executable
Note: some Windows users have reported this error when they installed Python for "Only this user" or have multiple versions of Python installed. Copying the appropriate PythonXX.dll to the virtualenv Scripts/ directory may fix this problem.

This seems to be related to mixing venv and virtualenv - see https://github.com/pypa/virtualenv/issues/1339. If so, it will probably affect other tools that use virtualenv, like tox, pew and pipenv.

Thanks for sharing. pipx exclusively uses venv. My limited testing with pipenv seemed to work fine, but good to know there might be some conflicts or issues.

No problem - I'm not sure where a fix would lie here, it seems to be a relatively new (and in this case Windows-specific) variation of a long-standing inconsistency between how virtualenv and venv work, so it may take some time to identify a suitable fix. In the meantime, knowing that there are potential issues with tools that use virtualenv might be as much as we can do.

Sounds good, I will add a note to the README saying that, so people don't have to search around the issues.

This is fixed now with the release of virtualenv 16.5.0. People with existing venvs created with pipx will need to ensure they get the new version of virtualenv into those venvs (and we can't do anything about tools that pin to older versions of virtualenv, of course) but there's no further need to track this here, IMO.
To be precise, if a user did (for example) pipx install nox and got the broken version of virtualenv, they will now need to run pipx runpip nox install --upgrade virtualenv to get the new version.


plugin no longer works

The github page has upgraded. This plugin doesn't work now.

Thanks for reporting the issue. I'm looking into it.

The issue has been fixed in 27dbdd9965284a50a5a43889c5cef886306c4a21. I've just published a new version that is being reviewed by the Google team. It should be live soon.

It's now live 🎉


update dependencies

/azp run

The EDS tests are failing because the EDS Client does not have StreamViews included. See: https://github.com/osisoft/sample-ocs-sample_libraries-python/pull/24

/azp run


Cequel::Record support

Hey guys, I did a bit of work to get this working for one of our projects. Is there any interest in my fleshing this out and PRing it at master?
https://github.com/orenmazor/rails_admin/commit/203a099454bb35943807a3dba4426916c9b0afab

Hi @orenmazor
I won't merge it (I cannot maintain this) but if you could make a plugin out of this, Cequel users would probably be pleased. If some work is needed in RailsAdmin internals, I'll gladly accept a PR.
Thanks a lot.

Hey, no worries. I actually abandoned it because it's fundamentally wrong to be making these kind of queries against cassandra. (aggregate stuff like select count(*) from blah cause all kinds of shitty performance)


Enhance a11y for SegmentedControl

path: stories/components/forms/segmented-control.stories.tsx
Description
1 accessibility violation was found in the SegmentedControl.
Target stories
isReadOnly
Expected the HTML found at $('#:r3o:') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('.css-1j77xsm[value="孫悟空"][data-checked=""]') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3o:-0') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3o: > .css-1j77xsm[value="ベジータ"]') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3o:-1') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('.css-1j77xsm[value="フリーザ"]') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3o:-2') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3q: > .css-1j77xsm[value="ベジータ"]') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Expected the HTML found at $('#:r3q:-1') to have no violations:

Received:
Elements must only use supported ARIA attributes (aria-allowed-attr)

Fix all of the following:
  ARIA attribute is not allowed: aria-readonly="true"

You can find more information on this issue here:
https://dequeuniversity.com/rules/axe/4.9/aria-allowed-attr?application=axeAPI

Problem Statement/Justification
A11y is essential to ensure quality when operating a package.
Additional Information
While it's ideal to cover all jobs, if there are too many jobs to work on, please create a PR within your capacity.
Any deficiencies not covered by the PR will be automatically created as issues.

@hirotomoyamada
can i work on this issue?

@GensIto
Hi, thanks for the comment!
Sorry, fixed this issue.


Update Generics.md

In order to prevent compiler errors
"error TS2564: Property 'hasMask' has no initializer and is not definitely assigned in the constructor.
Property 'nametag' has no initializer and is not definitely assigned in the constructor.
error TS2564: Property 'numLegs' has no initializer and is not definitely assigned in the constructor.
error TS2564: Property 'keeper' has no initializer and is not definitely assigned in the constructor.
"

Seems reasonable, we do assume strict everywhere. Can you please add spaces around the =s so it matches with the rest of the code 👍🏻


TRESA establish cost recovery process - BLOCKED

Goal Title
A unified process to track cost and execute recharges to projects using DSH
Description
A process in which a single TRESA point of contact deals with projects to request and pay for a DSH TRE
Definition of Done
The processes is signed off by all departments (REG, Finance, PMU) and a full cycle has been executed (request, cost sign off, recharge)
Details

RACI

Accountable team or person: RPM
Responsible team or person: TRESA
Informed and consulted people:

Estimated effort in total FTE: 0.3

Task and issue breakdown
Breakdown in specific tasks of maximum a month duration, tasks and issues may be added as necessary
Issues may span across repositories when necessary

issue alan-turing-institute/data-safe-haven-team#X
issue alan-turing-institute/data-safe-haven#X
issue alan-turing-institute/trusted-research
issue sa-tre/satre-specification#X
issue sa-tre/satre-team#X

Checklist

[x] This Story have been agreed with project members, it tackles prioritised work

[ ] If not: Label as ForPrioritisation so it discussed in the next monthly meeting, do not set a status

[x] I have filled in the Team Accountabe field
[x] I have included this Story in the agreed upon Milestone, set status as Planned

[ ] If not: set status as Backlog, to be Planned via weekly meetings or async discussions

[x] The work to be done is likely to span over a month
[ ] I have broken down the work in monthly issues and added them above
[x] I have labeled this issue according to its main project: SATRE, TRESA, CORE DSH (ELA) or other (please agree and create a new label if necessary)
[ ] If known: sum the total effort (points) of tracked issues and add it on the body
[x] Select a Story level of effort in the project fields

I think @jemrobinson may have closed this: https://github.com/alan-turing-institute/trusted-research/pull/215

I merged that PR, but it doesn't close the issue so I've reopened it.

mostly duplicated story, revisit TRESA stories and clean


Clip gradients option in contrib.layers.optimize_loss broken

Using the nightly build from 30 Mar 2016 trying to call:
train_op = tf.contrib.layers.optimize_loss(train_loss,
              global_step,
              0.002,
              "Adam",
              clip_gradients=10.0)

will throw:
TypeError: List of Tensors when single Tensor expected

I think opt.compute_gradients() returns a list of tuples not tensors so line 106 needs to replace gradients with [g[0] for g in gradients].

fixed by commit 3ab39c186d2a7e260023b68809963d0813566473


StakingMsg - any reason not to implement the other types?

create/edit validator  .. as well as Unjail from slashing module?
https://github.com/CosmWasm/cosmwasm/blob/632b8c23faa6d9b37f432ab0dda55ffe94524b2f/packages/std/src/results/cosmos_msg.rs#L72

Adding and maintaining the messages types is a good amount of work and it is not realistic that CosmWasm can keep up with all the message types Cosmos SDK or other blockchain modules are providing. So we have to be selective with what we add and know why we add messages (i.e. understand the use cases). The staking messages we have were added for writing staking derivatives. This also allows for all kinds of other scenarios where the contract is a delegator. What would be you use case for the message types you are asking for?


cURL install command support in Fedora

Laravel Version: 9.x.x
PHP Version: 8.1.x
Database Driver & Version:

Description:
The script that curl -s https://laravel.build/example-app | bash runs is not compatible with Fedora, every time is launched an error occurs. I was able to determine the cause and make it work. In the script when the /opt is binded to current dir in this line: -v "$(pwd)":/opt in Fedora (and I think every RedHat-based distro) because of SELinux the volume doesn't have write permissions which needs to be fixed by adding the 'Z' flag. Making that line to be like this: -v "$(pwd)":/opt:Z and that makes the error dissapear.
But to be honest, I'm not a Docker expert and I don't know if this solution is universal and can be added to the script without affect other distros that don't uses SELinux.
Steps To Reproduce:
Run curl -s https://laravel.build/example-app | bash in GNOME Terminal, mkdir() error appears.

@inietov could you please run the getenforce at the terminal and share the result?

Which Fedora version?


Mnemonic generation private key

Hello, how to generate the corresponding private key and address through mnemonic words?

Hello, how to generate the corresponding private key and address through mnemonic words?

Hi @developerjet, please have a look at this part of the README. It details how you can use AccountProvider to get an indexed Account derivated from a seed. (If needed, you can use BIP39 to construct a mnemonic and derivate a seed from it.) You can then access the account's private key through its serializedKey property. To encode the key as a base58Check (example), you can use SerializedKeyCoder from BIP32.
For addresses (derivated from an account), you can use AddressProvider. By using its addressAndPrivateKey function with an index as input, you can get both an address and its corresponding private key. To encode this key in a WIF-compressed format (example), you can use CompressedWIFKeyCoder from BIP32.


GoAccess 0.9.3 crashed by Signal 11

==2644== GoAccess 0.9.3 crashed by Signal 11
==2644==
==2644== VALUES AT CRASH POINT
==2644==
==2644== Line number: 29224396
==2644== Offset: 29224396
==2644== Invalid data: 284
==2644== Piping: 0
==2644== Response size:<PHONE_NUMBER> bytes
==2644==
==2644== STACK TRACE:
==2644==
==2644== 0 /usr/local/bin/goaccess(sigsegv_handler+0x12f) [0x40829f]
==2644== 1 /lib64/libc.so.6() [0x3e12a326a0]
==2644== 2 /usr/local/bin/goaccess() [0x41436a]
==2644== 3 /lib64/libc.so.6() [0x3e12a34ae4]
==2644== 4 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 5 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 6 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 7 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 8 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 9 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 10 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 11 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 12 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 13 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 14 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 15 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 16 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 17 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 18 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 19 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 20 /lib64/libc.so.6() [0x3e12a348c8]
==2644== 21 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 22 /lib64/libc.so.6() [0x3e12a348b8]
==2644== 23 /lib64/libc.so.6(qsort_r+0x29c) [0x3e12a34f1c]
==2644== 24 /usr/local/bin/goaccess(sort_raw_data+0x19) [0x4145d9]
==2644== 25 /usr/local/bin/goaccess(parse_raw_data+0x5a) [0x419e3a]
==2644== 26 /usr/local/bin/goaccess() [0x40c4b3]
==2644== 27 /usr/local/bin/goaccess(main+0x18a) [0x40cf6a]
==2644== 28 /lib64/libc.so.6(__libc_start_main+0xfd) [0x3e12a1ed5d]
==2644== 29 /usr/local/bin/goaccess() [0x4062c9]
==2644==
==2644== Please report it by opening an issue on GitHub:
==2644== https://github.com/allinurl/goaccess/issues

Server is centos 6
goaccess.conf
time-format %H:%M:%S
date-format %d/%b/%Y
log-format %h - %^[%d:%t %^] "%r" %s %b "%R" "%u"
config-dialog false
color-scheme 1
hl-header true
no-color false
no-column-names false
no-progress false
with-mouse false
no-csv-summary false
static-file .css
static-file .CSS
static-file .dae
static-file .DAE
static-file .eot
static-file .EOT
static-file .gif
static-file .GIF
static-file .ico
static-file .ICO
static-file .jpeg
static-file .JPEG
static-file .jpg
static-file .JPG
static-file .js
static-file .JS
static-file .map
static-file .MAP
static-file .mp3
static-file .MP3
static-file .pdf
static-file .PDF
static-file .png
static-file .PNG
static-file .svg
static-file .SVG
static-file .swf
static-file .SWF
static-file .ttf
static-file .TTF
static-file .txt
static-file .TXT
static-file .woff
static-file .WOFF

agent-list true
http-method true
http-protocol true
no-query-string false
no-term-resolver false
real-os true
with-output-resolver false
444-as-404 false
4xx-to-unique-count false
double-decode false
ignore-crawlers false
geoip-database /usr/share/GeoIP/GeoIP.dat
keep-db-files true
load-from-disk true
db-path /opt/goaccess-db/

A few questions for you.

Are you using the on-disk store?
Are you outputting to a terminal or a file? and did it break upon sorting a specific field?
Are you able to replicate this with a smaller subset of requests/hits? e.g., 1000 requests from your access log?

Thanks

Yes using on-disk store.
I had a bashed script to run script on each domain to output to a html file
From what I see only when database increase the problem appear. If I run first time or second time it runs ok but after database increase I got that error.
If I set load-from-disk false it is running ok.

I'm looking into this, but, could you please try to replicate it in gdb? and preferably with a smaller data set.
./configure --enable-debug --enable-utf8 --enable-geoip --enable-tcb=btree
# make
# gdb ./goaccess
// use args needed to replicate the issue
(gdb) set args -f access.log --keep-db-files --load-from-disk <enter>
(gdb) r <enter>
// after it crashes
(gdb) bt <enter>

Program received signal SIGSEGV, Segmentation fault.
0x000000000041480a in cmp_raw_num_desc ()
Missing separate debuginfos, use: debuginfo-install GeoIP-1.6.5-1.el6.x86_64 bzip2-libs-1.0.5-7.el6_0.x86_64 glibc-2.12-1.166.el6_7.1.x86_64 ncurses-libs-5.7-4.20090207.el6.x86_64 tokyocabinet-1.4.33-6.el6.x86_64 zlib-1.2.3-29.el6.x86_64
(gdb) bt
#0  0x000000000041480a in cmp_raw_num_desc ()
#1  0x0000003e12a34ae4 in msort_with_tmp () from /lib64/libc.so.6
#2  0x0000003e12a348c8 in msort_with_tmp () from /lib64/libc.so.6
#3  0x0000003e12a348c8 in msort_with_tmp () from /lib64/libc.so.6
#4  0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#5  0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#6  0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#7  0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#8  0x0000003e12a348c8 in msort_with_tmp () from /lib64/libc.so.6
#9  0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#10 0x0000003e12a348c8 in msort_with_tmp () from /lib64/libc.so.6
#11 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#12 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#13 0x0000003e12a348c8 in msort_with_tmp () from /lib64/libc.so.6
#14 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#15 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#16 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#17 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#18 0x0000003e12a348b8 in msort_with_tmp () from /lib64/libc.so.6
#19 0x0000003e12a34f1c in qsort_r () from /lib64/libc.so.6
#20 0x0000000000414a79 in sort_raw_data ()
#21 0x000000000041a5ba in parse_raw_data ()
#22 0x000000000040c683 in allocate_holder ()
#23 0x000000000040d13d in main ()

I've been trying to reproduce this with a similar environment, CentOS 6.7, 2.6.32-573 x86_64 and using the same config file. No crashes so far. I assume your database files were created on the same architecture as the one reading them.
Does it break with a much smaller data set? e.g., creating new database files with 1000 log lines and then appending another 1000 to it?
Can you please give more info on this issue? I mean how did you configure/execute it and what steps did you follow to reach here? This will help in reproducing this bug.
Also, If you could try v0.9.4, it would be great! Thanks!

I had tried with 0.9.4 it is same thing.
I have many domains on the server and I had used goaccess to create one html file for each of the domain
/usr/local/bin/goaccess --html-report-title=domain -f /var/log/nginx/domain.cache.log.1  >  /var/www/html/domain/index.html
The problem appear with large databse. for smaller datababe seem to work fine.

I'm starting to think this could be a data corruption on one of the database files. I just tested this with two data sets, first one a log of 50M lines and the second one with 30M, it didn't crash.
BTW, does this happen only when parsing the same log file or with any log? and have you tried deleting the db files and running everything again?
It would be great if you could post some additional data from gdb. However, you need to compile with debugging symbols. To do this you need to configure with --enable-debug:
# ./configure --enable-debug --enable-utf8 --enable-geoip --enable-tcb=btree
# make

then run it through gdb:
# gdb ./goaccess
(gdb) set args -f access.log --keep-db-files --load-from-disk <enter>
(gdb) r <enter>
// after it crashes
(gdb) bt <enter>
(gdb) info args <enter>
(gdb) info locals <enter>
 // select parse_raw_data frame #
(gdb) frame 21
(gdb) p raw_data->idx

Thanks!

@TempleNode I pushed a commit to ensure the raw data structure is not NULL upon sorting it.
Could you please pull the latest changes from upstream and see if that fixes the issue? Thanks.

Also, please make sure to specify a debug log file and post its content in here.
goaccess --debug-file=debug.log --html-report-title=domain -f /var/log/nginx/domain.cache.log.1

Closing this since I wasn't able to replicate it. Several changes have been made since v0.9.3.
Feel free to reopen it if needed. Thanks.


Release/0.5.1

Codecov Report

Merging #67 (1fe1ab2) into master (bb7d2cc) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 1fe1ab2 differs from pull request most recent head fc09c77. Consider uploading reports for the commit fc09c77 to get more accurate results

@@            Coverage Diff            @@
##             master      #67   +/-   ##
=========================================
  Coverage     96.41%   96.41%
  Complexity       86       86
=========================================
  Files            16       16
  Lines           363      363
  Branches         18       18
=========================================
  Hits            350      350
  Misses            2        2
  Partials         11       11

Continue to review full report at Codecov.

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


Create a docs folder and improve README

This PR improves the README by separating the setup and testing instructions and adding more language from the project abstract. The README is now in the docs folder, but Github will still display it on the front page of the repository. I have also added some images and a mkdocs.yml so we can publish this to ReadTheDocs.
Preview:
https://github.com/OpenGeoscience/uvdat/tree/update-readme

I have some suggested editorial changes. Reorganized a bit and added a few more details. I'll attach pdf with preview as well as patch file for consideration.
0001-Editorial-changes.patch.txt  (Remove the .txt extension -- I had to add it in order to get github to accept the upload)
UVDAT-Documentation.pdf

Note that if you use my changes, please fix the typo "representatitons" ==> "representations"

It appears that you added uvdat_flow.jpg but we don't use it (commented out). Was that on purpose or just an earlier version? My preference would be to leave the image out and remove the comment (line 12 in README.md).

@johnkit I added the uvdat flow diagram in 4222b7b, and part of your suggested changes comments out the image reference. I removed the image and the comment in 3dbeb10.


Web - Room - Visibility subtitle is wrong for public room

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

Action Performed:

Click on + icon
Choose new room option
Enter room name
Choose visibility as public and check subtitle below the dropdown (Anyone can find this room)
Click on create room button
Go to room
Click on header to see Details
Go to Settings and check text in Visibility section

Expected Result:
For public room text should be  Anyone can find this room
Actual Result:
Wrong text shows(People invited to this room can find it)
Workaround:
Unknown
Platforms:

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

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: <IP_ADDRESS>
Reproducible in staging?: yes
Reproducible in production?: yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
Issue reported by: @gadhiyamanan
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685526019865179
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Visibility subtitle is wrong for public room
What is the root cause of that problem?
no condition added for public room description
https://github.com/Expensify/App/blob/efb1463625ba784999dce58dec0faa9163e33e0b/src/pages/settings/Report/ReportSettingsPage.js#L204-L208
What changes do you think we should make in order to solve the problem?
use {this.props.translate(`newRoomPage.${this.props.report.visibility}Description`)} instead of below condition same as WorkspaceNewRoomPage.js
https://github.com/Expensify/App/blob/efb1463625ba784999dce58dec0faa9163e33e0b/src/pages/settings/Report/ReportSettingsPage.js#L204-L208
What alternative solutions did you explore? (Optional)
None

Yeah, I'm able to reproduce this one. Opening it up for proposal/reviews

Proposal
Please re-state the problem that we are trying to solve in this issue.
Visibility description is wrong for public room.
What is the root cause of that problem?
Current visibility description only either restricted or private.
<Text style={[styles.textLabelSupporting, styles.mt1]}>
        p.{this.props.report.visibility === CONST.REPORT.VISIBILITY.RESTRICTED
           ? this.props.translate('newRoomPage.restrictedDescription')
           : this.props.translate('newRoomPage.privateDescription')}
</Text>

What changes do you think we should make in order to solve the problem?
Solution 1: Use switch to get the correct description
visibilityDescription(visibility) {
        switch (visibility) {
            case CONST.REPORT.VISIBILITY.RESTRICTED:
                return this.props.translate('newRoomPage.restrictedDescription');
            case CONST.REPORT.VISIBILITY.PRIVATE:
                return this.props.translate('newRoomPage.privateDescription');
            default:
                return this.props.translate('newRoomPage.publicDescription');
        }
    }

<Text style={[styles.textLabelSupporting, styles.mt1]}>
     {this.visibilityDescription(this.props.report.visibility)}
</Text>

Solution 2: Use one more check condition
<Text style={[styles.textLabelSupporting, styles.mt1]}>
        {this.props.report.visibility === CONST.REPORT.VISIBILITY.RESTRICTED
                  ? this.props.translate('newRoomPage.restrictedDescription')
                     : (this.props.report.visibility === CONST.REPORT.VISIBILITY.PRIVATE ?
                               this.props.translate('newRoomPage.privateDescription')
                               : this.props.translate('newRoomPage.publicDescription')
                       )
       }
</Text>

What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Visibility subtitle is wrong for public room
What is the root cause of that problem?
There are no condition added for public room description.
https://github.com/Expensify/App/blob/79f18eca86c35fca8055e7036ca2d449ad34f85a/src/pages/settings/Report/ReportSettingsPage.js#L202-L206
What changes do you think we should make in order to solve the problem?
The easiest way is to change
{this.props.report.visibility === CONST.REPORT.VISIBILITY.RESTRICTED
         ? this.props.translate('newRoomPage.restrictedDescription')
         : this.props.translate('newRoomPage.privateDescription')}

to
{this.props.translate(`newRoomPage.${this.props.report.visibility}Description`)}

Contributor details
Your Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01697caeeda3411355

@abdulrahuman5196 thoughts on these proposals?

@JmillsExpensify  Thank you for checking up.
@gadhiyamanan's proposal here https://github.com/Expensify/App/issues/20127#issuecomment-1575100645 looks good. All other proposals present here are the same with minor variations so choosing the first proposal.
🎀👀🎀
C+ Reviewed
cc: @madmax330

Ok great, thanks! @madmax330 You good with this approach? If so, let's get the contributor assigned and put this in motion. 🙌🏼

Yeah looks good to me

@abdulrahuman5196 @madmax330 PR is ready for review

The PR that introduced the bug has been identified. Link to the PR:
The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment:
A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion:

Doesn't seem to be regression. It seems to be a miss from the original page implementation itself and we haven't noticed before the bug report - https://github.com/Expensify/App/pull/16910

Determine if we should create a regression test for this bug.
[@abdulrahuman5196] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

No. I don't think a regression test would be beneficial for this minor issue.

Gentle ping on payment processing for this issue @JmillsExpensify

bump @JmillsExpensify

Thanks for your patience, summarizing payouts for this issue:

Issue reporter: @gadhiyamanan $250
Contributor: @gadhiyamanan $1,500
Contributor+: @abdulrahuman5196 $1,500

Upwork job is here. I'll issue bonus on final payout.

@JmillsExpensify accepted, thanks!

Thank you @JmillsExpensify. Accepted the upwork offer.

Closing this, please comment if anything is outstanding.


[PoseNet] : Encountered unresolved custom op: PosenetDecoderOp.

Hello Team,
I am experimenting with all types of models with EdgeTPU to be used in a robot. I'm using C++ for the inference to make it really fast. I was able to build packages for Classification and Detection and I set onto Human Pose Estimation as it would be really helpful in detecting pedestrians on the fly. I wanted to understand the output tensor shape of posenet model and found it to be 340 * 170 *10 * 1.
While loading the model in the interpreter I guess I'm getting the the below error
Encountered unresolved custom op: PosenetDecoderOp.
So I did a bit of research and I found that I need to PosenetDecoderOp is a custom Operation that is not defined in the standard operation list of TFlite to be linked to the resolver.
I found the source code for the Decoding operation in the edgetpu repo
I'm currently using Cmake as the build tool and not Bazel for the package that I'm creating.
Is there any example on how to deal with this issue?

@Eashwar93 I think you'll just need to to build the decoder ops and register it similar to how we did with our native layer: https://github.com/google-coral/edgetpu/blob/master/src/cpp/basic/basic_engine_native.cc#L89

@Namburger Ah ok, thanks a lot, I will try to implement this and comeback again if I have some issues

@Namburger Works completely fine. Thanks once again for the strong support

@Namburger I tried writing a inference in C++ with posenet. The raw decoded output vector from the posenet that I received is a bunch of zeros to be precise a vector of 521 zeros.
I assure you that the input dimensions are correct and I did not get any build errors when I linked the Decode Op.
Any idea what I could possibly do wrong here. Let me know in-case if you want to inspect my Cmake file. I can share it if needed.

Which exact network are you running ? All zeros in out put may be correct (if no poses are detected).
Also, the output from the decoder network is 4 separate tensors, take a look at the example here https://github.com/google-coral/project-posenet
The outputs could be for example:
KeyPointCoordinates (10 poses, 17 key points, x and y)
name: poses
type: float32[1,10,17,2]
quantization: -10 ≤ q ≤ 10
location: 4
KeyPoint scores (10 poses, 17 key points)
name: poses:1
type: float32[1,10,17]
Pose scores (10 poses)
name: poses:2
type: float32[1,10]
Number of detected poses (one scalar)
name: poses:3
type: float32[1,1]
Now that's exactly 521 floats (10172 + 10*17+ 10 + 1)
Looks like your output vector is the buffer of these 4 tensors concatenated.
Looks like it works but there's a problem with the input image or the input image has no poses in it (in which case you'd get all zeros)

@mtyka Thanks for your detailed response. I tried running the following models
model1
model2
Actually I'm aware of the posenet  repo and understood how I need to post-process the buffer of the 4 tensors concatenated.
I can assure you that I'm using the same OpenCV Image pre-processing pipeline that I used for detection and classification of video stream which works. Image Pre-processing code snippets for your reference. image_width is set to 481 and image_height is set to 641 by default so that I can reshape it to 353 x 481 later on using OpenCV depending on the model I use.
edge::HumanPoseEngine engine(model_path, edgetpu_context, with_edgetpu);
	const auto& required_input_tensor_shape = engine.GetInputShape();

	cv::VideoCapture cam_frame(source);
	if(!cam_frame.isOpened())
	{
		std::cout << "Error opening camera \n";
	}
	cam_frame.set(cv::CAP_PROP_FPS, 30.0f);
	cam_frame.set(cv::CAP_PROP_FRAME_HEIGHT, image_height);
	cam_frame.set(cv::CAP_PROP_FRAME_WIDTH, image_width);

	cv::Mat frame;

	while(true)
	{
		cam_frame >> frame;
		const auto& input = edge::GetInputFromImage(frame,required_input_tensor_shape[1],required_input_tensor_shape[2],required_input_tensor_shape[3]);

GetInputFromImage function
std::vector<uint8_t> GetInputFromImage(cv::Mat input_frame,const int& width, const int& height, const int& channels)
	{
		cv::resize(input_frame, input_frame, cv::Size(width,height));
		cv::cvtColor(input_frame,input_frame,CV_BGR2RGB);
		cv::Mat flat = input_frame.reshape(1, input_frame.total()*input_frame.channels());
		std::vector<uint8_t> in_vec= input_frame.isContinuous()? flat : flat.clone();
		return in_vec;
        }

Do you see anything odd here in my Input preprocessing that might just lead to zeros or am I using wrong models? Please let me know.

I think you probably have width and height reversed. The input shape of the network is
1yx3 so 14816413 so height is 481 and width is 641

@mtyka Thanks a lot! It works. Is this convention[1,h,w,c] the same for all the models?
I'm able to visualize the keypoints in my camera stream. Also the the keypoints are in the (y,x) form which I totally forgot. Also its nice that the keypoint results are scaled according to the input size of the network.
I just ran into another problem. There is a offset of the keypoints in the x axis. Is there any offset correction that I need to perform? I the image size is exactly h=353 x w=481 as I used the model 353 x 481 model. The Y axis seems to be ok. I wondered if I flipped the frame, but I haven't as  I move the to the left the points move to the left as well and not to the right.
I followed your python code in the posenet repo and found this line.
I understand you are doing some offset and scaling but could not quite get what exactly is going on.
The scaling part I understand, I suppose that  I need to scale the keypoints by a ratio of the diplay size/network input size but I don't quite get the offset part if there exists any.
Below image for your reference.

Yes NHWC is a typical convention for neural nets. Note that normal images are actually also in this order even though we usually talk about x&y and not y&x.
The rescaling issue is because your input image (the image you attached) has a different aspect ratio than the network input, but the rescaling code likely preserves aspect ratio and probably adds black borders in stead of stretching the image. The keypoints y&x coordinates are relative to the actual input image that goes into the network.

@mtyka you are absolutely right. I thought I made sure that the image I'm displaying is the same as the image I have sent to the NN, but its not. Thanks again for your strong support. I will look into this further.
I should have debugged this by myself, sorry for the inconvenience.

@Eashwar93 I think you'll just need to to build the decoder ops and register it similar to how we did with our native layer: https://github.com/google-coral/edgetpu/blob/master/src/cpp/basic/basic_engine_native.cc#L89

Sorry, could you please explain how am I supposed to do that? Thank you!


[Action] Discuss TAG ENV at Open Source Summit EU in Vienna

The next Open Source Software Summit in Europe will take place in September (16-18 SEPTEMBER 2024
VIENNA, AUSTRIA). A few people from TAG have attended the conference in recent years. I expect a few will be there this year as well. It would be great to get the TAG people together at this event and represent TAG at the conference.
The OSS could serve as a milestone for the WG Green Reviews and overall TAG efforts (cc @catblade @mkorbi @guidemetothemoon). Depending on how many TAG Leads and TAG WG Leads attend the conference, we could also have some sort of off-site event.

https://events.linuxfoundation.org/open-source-summit-europe/

cc @patricia-cahill @claire-fletcher
cc @nikimanoledaki @AntonioDiTuri

Hi Leo! I will be there! It would be amazing to organize some off-site event the weekend before 14-15, maybe somewhere in the nature close by?

Are enough people going to have a meet up? Otherwise this might be unfortunately a closed issue as it is so close now.


如何使用LMFlow训练自己的私有模型？

我是一名计算机专业的学生，但也是一名机器学习的门外汉，我想咨询一下，我应该怎么应该如何使用LMFlow？如何利用LMFlow来训练自己的私有模型？用于训练的数据应该是什么格式的？是否有相关教程或者guide？

Thanks for your interest in LMFlow! To use LMFlow, you may first obtain a Linux server with GPU equipped. We recommend Ubuntu 20.04, so that the setup will be most compatible with our settings. Then, following the guide in setup, you can install the LMFlow package.
To have a first taste of LMFlow, you may try our Google Colab demo, which requires a Google Colab account. Google Colab provide free T4 GPUs for users, so you may directly run our chatbot (a rather weak version) on that.
After that, you may finetune your model via scripts/run_finetune.sh, which corresponds to this section of README. You are free to modify our shell script, mainly modifying the used model --model_name_or_path gpt2 to other models. The output models will be in output_models/finetune. You may load that model with ./scripts/run_chatbot.sh output/models/finetune to have a chat with the finetuned model.
To specifying the dataset, you may change this line to the path of your customized dataset /path-to-your-dataset/*.json. The data format of *.json is provided in dataset section of README, where text_only format can be used for training decoder-only models.
Our guide is available in README. You are also welcomed to join our community for more information. Thanks 😄

Thanks for providing more information! We have planned to support THUDM/chatglm-6b in this pull request, but haven't merged it yet. We will let you know once it is officially supported. Thanks! 🙏

Thanks for providing more information! We have planned to support THUDM/chatglm-6b in this pull request, but haven't merged it yet. We will let you know once it is officially supported. Thanks! 🙏

I have reviewed the Code about Chatglm supporting, i think the core  is to add the args about trust_remote_code=True
but i found the code only supported in inferencing not in finetuneing. And if i want fine-tune now, is it possible just add args about ``trust_remote_code=True`.

@fengsxy  @research4pan
I have a quick fix to support trust_remote_code=True. All you need is to add a new line (line 215, trust_remote_code = True) to file LMFlow/src/lmflow/models/hf_decoder_model.py . Check the capture for details.

Thanks for your meaningful contribution! We will add this to our pull requests soon @shizhediao  😄 👍

Hi,
We are happy that ChatGLM inference is supported in our framework.
Please try this script: https://github.com/OptimalScale/LMFlow/blob/main/scripts/run_chatbot_chatglm.sh

Closed this issue.
If you have any further questions, please re-open it or create a new issue.
Thank you!


File "/workspace/Hermes-Function-Calling/functioncall.py", line 126   :^^^^ SyntaxError: f-string: unmatched '('

line 126:    inference_logger.info(f"Here's the response from the function call: {tool_call.get**("name"**)}\n{function_response}")
The use of double quotes inside an f-string, which itself is delimited by double quotes causes an error. This causes Python to interpret the inner double quotes as the end of the string, leading to a syntax error.
To fix this:
Use single quotes inside the f-string if the string you're calling (e.g., the name in tool_call.get("name")) does not itself contain single quotes. This is a simple fix but might not always be suitable if the inner string could contain single quotes.
inference_logger.info(f"Here's the response from the function call: {tool_call.get('name')}\n{function_response}")
This would apply to code lines : 126, 129 and 132

hi henri, thank you for spotting it. i have updated the f-strings with single quotes.
https://github.com/NousResearch/Hermes-Function-Calling/commit/bd02a3ad352e98f05d9516ec8a9d37eb2ebeab2f


Added new term and the missing particle as pointed out by Paola Fixes…

… #14885
Hi @ukemi the new term should be ok, but I also changed the label of a related term as @paolaroncaglia suggested: "GO:0005041 low-density lipoprotein receptor activity should be corrected to low-density lipoprotein particle receptor activity". Could you please confirm that it is ok to merge this?
Thanks,
Barbara
cc @RLovering

oh no. I chose the wrong namespace... should I wait for the pending check to finish before editing this? Or can I edit and push again while this is running?

Shouldn't the namespace be molecular_function?

I have just corrected this. Yes.

But I have noticed another error that I made...  I put the pull request number after 'Fixes' instead of the issue number. Is there a way to edit this?

It shouldn't be a problem.

Ok, thank you @ukemi, I'll merge now.


A little confuse about "epoch_size".

Thanks for your great work!
But I'm a little confuse about epoch_size = 60000  # include flip * 2, aug * 4, batch * 16 in config.py
Could you explain about how to get 60000 a little? We know that COCO has about 150k person instances for training. Is there any connection between these two figures?

Sorry about that. It's a previous version of code but I forgot to delete the comments. The epoch_size means how many images are trained in one epoch. 60000 is an empirical number. Actually a smaller epoch_size is also acceptable, but in order to get more stable result we just trained longer.

Got it. Thanks for your explanation.


chore: release 1.5.2

:robot: I have created a release *beep* *boop*
1.5.2 (2021-05-24)
Bug Fixes

add annotation injection (157dbe6)
update helm version (b54d47a)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/GluuFederation/cloud-native-edition/releases/tag/v1.5.2 :sunflower:


Need help setting initial Date for SingleDatePicker

Hi!
I've seen in storybook that setting a "before-today" date for initial date in a SingleDatePicker is possible (here)

But I could not find the correct property to set the same behavoir.
I'm trying this
                <SingleDatePicker
                    showClearDate={true}
                    numberOfMonths={3}
                    enableOutsideDays={true}
                    isDayBlocked={() => false }
                    date={this.state.date} // momentPropTypes.momentObj or null
                    onDateChange={date => this.setState({ date })} // PropTypes.func.isRequired
                    focused={this.state.focused} // PropTypes.bool
                    onFocusChange={({ focused }) => this.setState({ focused })} // PropTypes.func.isRequired
                />

But the result is not the expected:

I've searched the documentation and try a lot of properties, but as I find a startDate prop in DateRangePicker, there is no equivalent in SingleDatePicker
Can somebody help with this? Maybe I'm missing something obvious. If this is the case, please, let me know.
Thanks

I think the initialVisibleMonth property might be what you're looking for? It expects a thunk ie (() => moment()) that returns a moment object representing the month you want to be first in your visible months.

JOKES: I think what you want is the isOutsideRange prop. If you look in the stories, https://github.com/airbnb/react-dates/blob/master/stories/SingleDatePicker_day.js#L25 is what you're looking for. You'd want to set isOutsideRange to () => false for this to work.

isDayBlocked is more intended for individual one-off days and isOutsideRange is intended for limiting the scope of what is selectable at all.


Reset button does not work when all steps are folded

Needs a fix in AWB, tracked in https://github.com/aiidalab/aiidalab-widgets-base/issues/343

Has been fixed in AWB, will be released in 2.0 version


[Issue] Hitting failed deploy due to az cli version upgrade prompt again

I was running azd up and it failed on deploy.  When I re-ran azd deploy I saw a familiar message and realized I'm hitting the issue again where the underlying az cli was outdated and there was an interactive prompt waiting for input that caused the command to fail.

We have almost removed Az CLI dependency. This should eventually go away once we release it in the upcoming release in Jan.

With the latest 0.5.0-beta.X release. Azd does not have Az CLI dependency. Closing it as fixed.


Fix adopt/adapt typo in user docs

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[X] Y

Closes: #429
2. What is the scope of this PR (e.g. component or file name):
Documentation.
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[X] Other

The docs current use the word "adopt" when it actually should be "adapt".
Adopt:
To become the owner or caretaker of.
To take and follow by choice or assent.
Adapt:
To make suitable to or fit for a specific use or situation.
This updates content, file names, and reference links to use adapt instead of adopt for documentation related to using KCL for a given purpose.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[X] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:
N

Sorry, it looks like when I was fixing the merge conflict with 519012c5b08b7b72a3b6bd377bc9c606dea39029 I accidentally reintroduced the error with the link using Jsonnet instead of jsonnet.


Invalid state doesn't work in IE9 and IE8

[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.
[x] I’ve searched for any related issues and avoided creating a duplicate issue.

What happened
Opened the test page in IE8 and IE9 could not see the invalid state.
What I expected to happen
See invalid state in older browsers.
Reproducing

Module name: control-input
Module version: v2.0.3

Attachments

Fixed by https://github.com/govau/uikit/pull/391
Tested in IE9 and IE8 on Windows 7


fix indent of template

Thanks Chris, I'll merge this in but I'm going to have a look through the rest of the snippets before putting out a release. I believe there was a problem where VS Code was interpreting 4 spaces as a tab too, which I'd like to take a look at.

@stuartpreston Awesome. Thanks for releasing this plugin. This is great work.
This was only a minor thing that came up while testing your plugin.


Introduce ArtifactResolutionQuery#forComponent(group, name, version)

So it can be used by kotlin-dsl in place of the internal DefaultModuleComponentIdentifier#newId.

Actually I just see you did. Let's wait for the pipeline to be complete.

Thanks, @melix.
I should have posted the link to the CI job here: https://builds.gradle.org/viewQueued.html?itemId=9655073&tab=queuedBuildOverviewTab

I'm an outsider to this, but it seems a little strange to give GAV-like components a special place in this interface. We have several other component identifiers that don't have group, name and version.
IOW, should this at least be forModuleComponent or there be a factory for creating these kinds of components somewhere else (DependencyHandler or something new)?

Thanks for the review, @big-guy.  We're adding the method to avoid introducing a factory and expose more API. I agree that Module instead of Component in the name would be clearer, my preference would be forModule, what do you think?

CI is green.

@melix, @big-guy, anything against the new name? (forComponent => forModule)

:+1:

Thanks @bamboo, I didn't really have enough context, but I thought it looked odd.  forModule sounds fine.


Username and password are not required parameters when creating a registry credentials

Version - V0.12.0
username and password are not required parameters when creating a registry credentials.Only email is the required parameter.
Should username and password also be made required parameters ? May be email could be an optional parameter?
When i try to log in to quay.io using web , I can log in using username/password or email/password. This authentication mechanism could be different for different providers.

@hibooboo2 Is there some case where you can login with just an email (no username and/or password)?

So you are right all three can be optional, as there is no spec for required implementation of docker authentication from what i have read. But I am not 100% sure.

More discussion in #921


ConvertFrom-Json: Add -DateKind parameter

PR Summary
Adds the -DateKind parameter to the ConvertFrom-Json that allows the caller to control how DateTime strings are converted into an object. The default behaviour is to create a DateTime value with the Kind being Unspecified if no TZ is set, Utc if the TZ Z is set, Local (after conversion) if an explicit TZ is set. This adds a Utc, Local to explicitly set the Kind as desired as well as a Offset and String value to create a DateTimeOffset or keep as a string.
PR Context
Provides the user the ability to create a DateTimeOffset value so that the underlying time zone information is preserved. Also gives the user the ability to keep it as a string or a specific DateTime with the Local or Utc kind.
Fixes: https://github.com/PowerShell/PowerShell/issues/13598
Docs PR for this change: https://github.com/MicrosoftDocs/PowerShell-Docs/pull/10737
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[x] Documentation needed

[x] Issue filed: https://github.com/MicrosoftDocs/PowerShell-Docs/pull/10737

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

@mklement0 Could please review and confirm this work as you expect?

Thanks, @iSazonov  - I have one small question (see above), but otherwise it looks great. Thank you for tackling this, @jborean93.

@jborean93 Thanks for your contribution!

Thanks for the review everyone!


Keyless run: teach VE to use board server's node proxy when registered

The mental model is that the board runs using the secrets of its board server. If the board is on bb.live, then it will use the secrets from bb.live. It the board is in browser storage (which is a kind of board server), it will use local secrets.
We likely also need a concept of secret fallback. The local store also acts as the fallback for secrets. If the secret wasn't found on the board server, it will just use the local secrets or ask user (interactive secrets). This serves the use case where I am working on a board that accesses some new API and I don't yet have its keys stored on the board server -- or when I don't want to share the secrets to the board server.
The secret fallback must work for both VE and board server runner.


There was a problem: Enter a valid email or mobile number

I am unable to login VIA [ip address]:3456.  I keep getting "There was a problem.  Enter a valid email or mobile number."  I am able to login directly to https://pitangui.amazon.com with no issues. Did something change on Amazon's side where this needs to be updated? It has been working great for weeks, and now I can't log in successfully.
Update: I was able to get it working with the cookie method. Not a perm solution, but will work for the short term.

Same here with login to layla.amazon.de. The Amazon account is ok. At least, I'm able to login to the alexa website.
In the change log of alexa-remote2, I found:
3.2.0 (2020-06-16)
(Apollon77) Update Cookie Library to allow Proxy Signup again after Amazon changes
That sounds like a hint, that Amazon really changed something quite recently.

I am unable to login VIA [ip address]:3456. I keep getting "There was a problem. Enter a valid email or mobile number." I am able to login directly to https://pitangui.amazon.com with no issues. Did something change on Amazon's side where this needs to be updated? It has been working great for weeks, and now I can't log in successfully.
Update: I was able to get it working with the cookie method. Not a perm solution, but will work for the short term.

Sorry, could you please tell me where to find the cookie method?

https://github.com/586837r/node-red-contrib-alexa-remote2/blob/master/get_cookie.md

Me too.  I am getting "Enter a valid email or mobile number."  After several attempts, Amazon forced me to change my password.

This is happening to me also on the UK amazon site. Had to use the cookie method.

same issue here in the UK :-(

https://github.com/586837r/node-red-contrib-alexa-remote2/blob/master/get_cookie.md

Can confirm that this method seems to work perfectly, so yep use cookie auth mode & all good

No luck here even with the cookie method. Do you need to copy the entire field beginning with Cookie: session-id=  ?

Same to Amazon DE, after update the package today

Hello,
I had the same issue... It didn't work with the german, the uk and the usa site. But then I tried the italian site and it worked. Maybe this can help you, too...
KR
Andy

Managed to get mine working using the US amazon.com site. Opened up Music and opened up alexa desktop in my pc. Asked alexa to play a radio station AC/DC. F12. last line of network codes.
Under Header there should be a cookie for playing that radio station. Copied this entire field into the node and started to work

well well - capturing a cookie works for me too
running with proxy and MFA not, regardless of the region i use for login.

Could somebody who has successfully authenticated by cookie post a detailed how-to, including exactly what steps to get the cookie, where it is located, which part of the cookie to copy, and the associated cookie settings in node-red?  No matter what I do, I am still asked to go to port 3456 to sign into Amazon, and that has resulted in my having to get 3 new Amazon passwords so far.

@tbclark3 I used Chrome and had to do this a couple of times before it worked, it sounds like you are having the same problem i was having.
Log into alexa.amazon as per the link above and then hit F12 then pick-  music video and books, go to TuneIn and then pick a local radio station and play the station on one of your devices. You should be able to get the cookie needed in the developer screen following the guide above.
You dont need Cookie: at the start just the info after.
In node red open the alexa routine node and then edit the account.
At Auth Method drop down to cookie and paste in the Cookie info you copied from the previous step and save then deploy.
You should see a green success beneath the node.
Hope this helps.

I have replaced cookie , but still i get please enter a valid email when i try to log in. Nothing is working. I have contacted Amazon, but just getting the run around. If i try email and password method i get a captcha, ut i have no way to see the captcha image.

Hi. Thanks for all the instructions above, cookie method seems to work. However, I am wondering how long is it valid, should it be changed too often, or is it working permanently (until Amazon will change something)?

Thanks @freqteqfpv.  It did help.  For the benefit of others who may be facing the same issue, my setup only worked after I deleted the existing alexa init node and created a new one.  Inserting the cookie into a new node worked flawlessly.  Like @kpmkpm, I am also wondering about the permanence of cookies.  Does it need to be refreshed?  Regenerated?

@tbclark3 happy it worked out for you, I'm not sure about the cookie question, It's one I hoped someone else could answer.

I have same issue and was not able to get it work with the grab of cookie from the browser...
I'm on amazon.it and I get the "Enter a valid email or mobile number" error when inserting credentials.
Is there any other way or a video on how to do it with the browser cookie grab?
Thanks

Just updated alexa-remote2 to 3.10.1 and worked fine immediately!
Thanks!

Also just updated to 3.10.1 and it now works

I daren't update or touch mine right now since I got the cookie working after many tries. I'll wait for the connection to break first before updating. Still going after 4 days or so now.

the update to 3.10.1 unfortunately didnt work for me. I used to use the cookie method but it only survives a few days

Does anyone have this working after upgrading to 3.10.3?  I have Node-Red running inside of Home Assistant, and I am still having the issue after upgrading. NR shows 3.10.3 version but no luck with proxy method :-(


Unable to login on Thruk

By default Thruk uses cookie based authentication and NagBar does not send a Authorization unless the server asks for a basic authentication.
It would be great to add an checkbox to force Authorization header usage.
Thanks a lot.

BTW I manage to force a basic authentication within HAProxy adding this line:
http-request auth realm "Thruk auth" unless { req.hdr(Authorization) -m found } { req.hdr(User-Agent) -m reg NagBar }

But still it would be better to add an option to force HTTP Auth in NagBar.


Error: spawn ENAMETOOLONG

Steps to reproduce
Tell us about your environment:
Puppeteer version: v3.3.0
Platform / OS version: win64
URLs (if applicable):
Node.js version: v14
What steps will reproduce the problem?

Launch browser with { headless: false }
Scrape some page content
await page.close(); await browser.close(); return data;
Result processing and then do same things from step 1

What is the expected result?
Running app 24/7, relaunch browser (1 or 2) every ~10 sec and handle results
What happens instead?
If app runs with 1 browser then after 2+ hours there is an error: Error: spawn ENAMETOOLONG. So the browser won't restart. If it's 2 browsers an error will occurs after 1+ hour.

I have the same problem when closing and starting browsers often. Only encountered this on windows.

I have the same problem when closing and starting browsers often. Only encountered this on windows.


Error in publications fetch with lens client 2.0.0-alpha.14

Describe the bug
An error occurs for the following query:
`
query Publications
{
"where": {
"withOpenActions": [
{
"address": "0xfF824aB2FCb6986e399B5Af922c4a20778a047A1",
"category": "COLLECT"
}
]
},
"orderBy": "LATEST",
"limit": "TwentyFive"
}
`
The error is:
{     "errors": [         {             "message": "[CanDecrypt] Publication metadata not found for 0x33-0x0261} - [getPublicationMetadata] Could not find metadata context for publication 0x33-0x0261 - fix the following issues\n· \"version\": Required - SHOULD NEVER HAPPEN!",             "locations": [                 {                     "line": 444,                     "column": 3                 }             ],             "path": [                 "result",                 "items",                 1,                 "operations",                 "canDecrypt"             ],             "extensions": {                 "code": "INTERNAL_SERVER_ERROR",                 "exception": {                     "apollo": false,                     "statusCode": 500,                     "stacktrace": [                         "Error: [CanDecrypt] Publication metadata not found for 0x33-0x0261} - [getPublicationMetadata] Could not find metadata context for publication 0x33-0x0261 - fix the following issues",                         "· \"version\": Required - SHOULD NEVER HAPPEN!",                         "    at PublicationOperationsFieldResolver.canDecrypt (/usr/src/app/packages/api-lens/lib/publication/resolvers/field-resolvers/publication-operations.js:191:19)",                         "    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)"                     ]                 }             }         }     ],     "data": null }
To Reproduce
Happens each time.
Expected behaviuor
No errors
Version of the packages you are using
lens client 2.0.0-alpha.14

This was an API issue which got resolved. Please reopen if experience this again.


[Feature] update contributing guidelines

Description

Update the guidelines to ask contributors to add descriptions while adding new subcategories.
As shown in the image below, it is vacant, which seems strange. Adding a small description at the same time of adding a new subcategory would save us the hassle of adding remaining descriptions.

Screenshots

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

Good idea @Anmol-Baranwal! :)


SOP: Clarify output format of CERTS output for RevokeKeyImpl, ChangeKeyPasswordImpl and ExtractCertImpl

If there are multiple keys, do we want to emit a single ASCII armored block containing all certificates, or multiple individual ASCII armored blocks, one for each certificate?

The way I'd read the specification, the intended way to deal with concatenated OpenPGP objects is to put them into a single ASCII armor block.
I unified PGPainless' behavior in this regard in https://github.com/pgpainless/pgpainless/commit/80cf1a7446b3db47a80d2ec6db33ecaac4f12b26


Separate indentation rules for attributes

I would like to indent all tags and text node with 2 spaces, but v-align my attributes. Like this:

  Some text...

    Some other text...

Currently this would throw an error, because class isn't indented with 6 spaces.

Just set indent-width to false. There really is no way for htmllint to detect which indentations are sensible and which aren't.

Is this a limitation of htmlparser2? Should I request  the feature there?

No, just a limitation of reality. htmllint could detect this specific case, but there are many other things you could potentially want to align and no way to determine which of those should be accepted by the linter. Trying to detect this case makes the linting rules more difficult to understand for a very narrow benefit.


CLOUDSTACK-9779 : Releasing secondary guest IP fails with error VM nic Ip x.x.x.x is mapped to load balancing rule

ISSUE
Releasing secondary guest IP fails with error VM nic Ip x.x.x.x is mapped to load balancing rule
REPRO STEPS
Create two isolated guest networks with same CIDR
Deploy VMs on both networks
Acquire secondary IP on NICs of both VMs and make sure they have the same value, user can input the IP address.
Configure Loadbalancing rule on one of the secondary IP address and try releasing the other secondary IP address.
The operation would fail
EXPECTED BEHAVIOR
Secondary IP address should be released if there are no LB rules associated with it.
ACTUAL BEHAVIOR
Not releasing secondary IP address even if there are no LB rules associated with it.

code LGTM

@niteshsarda I don't see any change in file createNetworkOffering.xml, but still there are many lines being removed and added. As there is not change in this file, please revert.
Also the marvin test case seems as the complete file was edited, even the license header. Please add only the changes that are relevant.

@ustcweizhou : Can you please check new PR for this issue and provide LGTM on that PR.
PR link :
https://github.com/apache/cloudstack/pull/1978

@SudharmaJain Can you please check new PR for this issue. Also, I have implemented the changes which you have suggested.
PR link :
#1978


Report generation changed to use React

The report generation was earlier done by using a html template that accepted the data in json format. This has been modified, where the web page has been split into multiple components which are rendered using React. Webpack has been used to bundle these components, so that it can be accessed by the template.

@harsha-salim I will check this PR, please gimme some time.

@harsha-salim Please run npm i and push the package-lock file.
build is failing due to https://github.com/npm/cli/issues/5113


Execute callback after outermost transaction commits

As a developer, I sometimes need to perform external actions when (and only when) the transaction commits and data actually lands in the database.
Examples:

Send email/notification after successful operation (e.g. user registration).
Delete external file after successfully deleting a database record referencing that file.

Similar feature in Django: https://docs.djangoproject.com/en/5.0/topics/db/transactions/#performing-actions-after-commit
I don't see how it's currently possible with Orchid. Any given ORM code could be running in a deeply nested transaction which may later abort due to unrelated circumstances that would happen further along the code execution path.
Also, as a developer, I want to place the code that performs the corresponding action immediately following the database operation, so that it's easier to support:
// This may be running inside a transaction, or may be not.
const user = await db.user.create({ ... })

// Only send email if and when the user is actually stored in the database.
await db.$afterCommitOrRunImmediately(() => sendEmail(user))

The "after commit" naming is not perfect as ideally the callback should run immediately if there is no pending transaction, otherwise it should postpone until the outermost transaction closes.
I believe multiple pending callbacks should be settled with Promise.allSettled(), but I'm not sure what should be thrown considering that multiple callbacks could fail? Perhaps the failed settle result should be wrapped into its own type of exception.

Oops. I discovered afterCreateCommit.
Let me test if it delays execution until the outermost transaction finishes. If it does, I will close the issue.

Let me know if it's works for you now.

Thanks! This more or less works. I noticed a few minor nuances (see above) but overall that's usable.
However, I still believe db.$afterCommit deserves a separate method on its own. Consider the scenario:
await db.$transaction(async () => {
  const user = await getUserFromSomewherePerhapsCreateOrMaybeSelectOrMaybeEvenFromCache()
  if (Math.random() < 0.5) {
    await updateUserBalanceOrMaybeCrash(user)
  }
  await db.$afterCommit(async () => {
    await sendEmail(user)
  })
})

I don't see how this can be accomplished with after commit hooks on queries.

Also, the current way of things is that db.$transaction awaits for the hooks and rejects as soon as the first of them fails. The other hooks finish in background.
This might be okay for now but somehow this doesn't feel right satisfying :). For instance, there is no easy way to tell a failed transaction (that has been rolled back) from a successful (committed) transaction with failed hooks, and multiple errors are lost. Also, I'm not convinced the whole transaction should reject at all, or even wait for hooks to complete.
I don't see a direct way for improve here, so these are just thoughts. At first I was thinking something like await db.$transaction(...).withHooks() returning something like { result, hooksResults } with the result of Promise.allSettled() for hooks, but that feels like overcomplication and is unlikely to be used by anyone. Probably how it works now is the least evil, and more advanced scenarios could be handled on user level.

These are all good points.
I guess db.$afterCommit could be added quite easily.

For instance, there is no easy way to tell a failed transaction (that has been rolled back) from a successful (committed) transaction with failed hooks, and multiple errors are lost.

That sounds bad, errors shouldn't be lost, but I'm either not sure what's best in this case.
You can create a yet another issue with more specific details about the use-case, and maybe I'll have some ideas.
I'm thinking about case when you created a record, but email sending failed, you can save email sending to a message queue to be processed later, but what if message queue failed, so you can end up with a created record but no side-effect.
After consulting with ChatGPT, they suggest "Outbox Pattern" and it makes sense: inside transaction you also save an intention to trigger side effects, so once transaction is committed, both your data and intentions were saved. Delete the intention record after doing the side effect. After the server suddenly crashes before performing side-effect, it will scan that table on startup and perform all side effects.
This is good for a bulletproof flow, I'm interested in finding best ways and practices for it, and would be cool if OrchidORM could assist with it.

because what if server suddenly crashes at error handling - side effects would be lost anyway

Agreed, that makes sense.
But then do we need to await for all the hooks to settle or have them fully detached from the main flow? Both approaches make sense in my thinking.

Another use case is allowing this pattern to work:
const selectObj = db.table.makeHelper(...) // reusable helper in a different module

const updatedObj = await selectObj(db.table).find(id).update({ ... }).afterUpdateCommit([], () => {
  console.log('actually updated obj', updatedObj) // crashes with:
  // Cannot access 'updatedBet' before initialization.
})

// Will rather need this:
await db.$afterCommit(() => {
  console.log('actually updated obj', updatedObj) // will not crash, updatedBet already initialized
})

afterUpdateCommit wants an explicit list of fields to pick, which is not always feasible to use when there are helpers that allow to reuse selectors, aliases or raw SQL subqueries.

@romeerez You mind if I try to come up with a PR? Any crash course guidance perhaps?

PR would be very welcome!
Yea, I can't keep up with issues.

Any crash course guidance perhaps?

OrchidORMs discord, ask any questions. Or here on github in discussions.
You did several PRs before, I guess you already know how to run it, how to run tests.
The code for transactions is in pqb in the file transaction.ts + in the file then.ts it's accessing transactions and calls some related callbacks.
orm has only transaction wrappers with $ sign.


Fix ocsp app exit code

If we run the ocsp command line app and the responder returns a
non-successful status code then the app should exit with a failure code.
Based on an original patch by Tatsuhiro Tsujikawa.
Fixes #2387

Pushed. Thanks.