{Branch Management} Merge release 2.53.1 back to dev

Related command

Description

Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

Branch Management

Currently release branch points to 1745a1f97a5139390df83b4fa3eb95dcb7011be4 (https://github.com/Azure/azure-cli/commits/release), which is on the upstream of https://github.com/zhoxing-ms/azure-cli/commits/merge_release_2.53.1_back_to_dev. This is the correct way of solving merge conflicts.
A kind reminder: Please remember to create a merge commit locally, instead of creating a squash merge commit on this web page.


Debug date validation (BREAD)

Hello,
I have a Date field that is required :

But when I tried to create a new entry, I had this error in the console, and nothing append :

I find that in the validation is the display_name that is used, so "Date" and not "date". So the validation was not working.
I think that would make more sense to use the field instead of the display_name and in the JS to use "name=" instead of "data-name=".
I'm a new user of voyager, so let me know if I'm wrong :)

This is more complex than simply changing those fields. Also, I swear we already fixed this... or at least have a PR open to do so.  @akazorg @emptynick, are either of you guys familiar with this?

Same or similar PRs that are currently open: #2191 #2198

Hi @fletch3555, I don't know about this.
I'll have more time next week, I'm quite busy atm.

I've sent a PR to fix this on nov 8th. Is there something I can do for the PR to be merged? There was a release a few days ago and the opportunity for this fix to be released was lost.

Thank's for the note @blitux, we've been busy. It should fixed soon.

The PR @blitux mentioned above was merged nearly a month ago, so I'm closing this one.


"Strong" text is no different from regular text

Describe the bug
Text marked with ... (obtained by **...** in Markdown in jupyter-book) looks just like regular text.
To Reproduce
Steps to reproduce the behavior:

Surround some text in a jupyter-book project with double asterisks.
Compile the project to a book.
Visit the generated website in the browser.
Notice that the marked text does not appear different from the surrounding text.
Inspect the text using the browser's developer tools.
Note that, while there are indeed ... tags around the text.

Expected behavior
Strong text should appear emphasized, typically using bold.
The screenshot below was generated from the following Markdown:
## Functions

**Definition:** A *function* is any method for taking a list of inputs and determining the corresponding output.

Environment (please complete the following information):

Python Version 3.7.3
Output of pip show jupyter-book says the version is 0.7.0b4.dev0

I transferred it 😄

Hmmm, I just tried this on the latest jupyter book and it worked fine for me, this is what I got:

can you try upgrading jupyter book pip install -U jupyter-book and re-building?
(also after you upgrade you should be able to do jupyter-book --version)

This did not solve the problem for me.
$ jupyter-book --version
Jupyter Book: 0.7.0
MyST-NB: 0.8.1
Sphinx Book Theme: 0.0.23
MyST-Parser: 0.8.1
Jupyter-Cache: 0.2.1

Hmmm, I am not sure what to tell you, it's hard to fix without being able to reproduce the bug. Could you post a link to a website you've built where this issue shows up?

Sure.
Page: https://nathancarter.github.io/MA346-course-notes/_build/html/chapter-2-mathematical-foundations.html
Repo: https://github.com/nathancarter/MA346-course-notes

hmm, could this be a browser issue? this is what I see:

Yep same here. @nathancarter a good way to check out the page styling is using the Firefox inspector. As you can see below the word is given the bolder styling.

Note it is also possible to add configuration to jupyter-book/sphinx to override the default CSS

OK that's helpful, but I have to confess I'm completely confused about something else now.
I took your suggestion, created a custom CSS file in my _static folder, as the jupyter-book docs say to do:
b, strong {
    font-weight: bold;
}

I then rebuilt the book and used the Chrome dev tools to verify that the ... tag now has its font weight set to bold due to my custom CSS file.
And yet the page looks exactly the same in Chrome, but looks correct in Firefox.
I'm at a loss.  I realize this is not a jupyter-book problem, but if you've got any thoughts based on your knowledge of the styles and tools involved, I'd love to hear!

Interestingly I get something similar with @nathancarter.
Here's a screenshot of an example:

Now, here's what the same text looks like when I remove the font-family Lato,sans-serif:

oh ho - maybe it's a fonts issue!

This definitely seems like progress, but I confess that I don't know what to do to solve my problem (which is to make it show up on all browsers).  Should I be changing the theme to use a different font?

well, it's tough to say because it sounds like it is dependent on the user's computer. I think one thing to try is make sure that the fonts used by sphinx-book-theme are on your computer. If not, then install them and see if that fixes it so we can pinpoint the issue. If SBT is using something non-standard then we should switch to whatever the standard is.

I'd want to be sure that anyone viewing my book sees the style as intended, so if there's even a possibility that any normal computer (such as my Mac) doesn't have the fonts to see the styles in my code, I'd want to use a different style.

For sure - we just need to pinpoint what is the style issue that's causing this problem, so that we know what to fix

This seems to have been fixed by https://github.com/pandas-dev/pydata-sphinx-theme/issues/222.

yep, we just need the pydata theme to release a new version, then bump its dependency in the SBT and we should be good-to-go

I believe this is now fixed in by pydata-sphinx-theme v0.4

Yes.  Somehow I thought I had replied somewhere and confirmed that this was fixed for me, but I can't find my reply right now.  Either way, thank you all--works great!

I use Github Action to deploy the website, and started having the same problem - bold texts are not different from regular ones. I do not specify versions in requirements.txt. On the other hand, NO problem arises if I open locally built html files. Just in case, I added the following in `requirements.txt' for Github Action
sphinx == 3.4.3
sphinxcontrib-serializinghtml == 1.1.4
sphinx-book-theme == 0.1.0

which are equivalent to the environment I used to locally create html files. But no difference.
When I open the Github Action-created website, it shows bold texts correctly but for less than half second, and then those texts turn quickly "regular".

I have the exact same issue. Local building is fine, building using Actions has several issues:

paragraph breaks removed
horizontal scroll bar at the bottom of the page
buttons to navigate to next page look different:

instead of

Repo: https://github.com/lucasdekam/nb2220-test
Page: https://lucasdekam.github.io/nb2220-test/index.html


An ability to get a parent prefix from an ip address using API

Issue type
[X] Feature request
[ ] Bug report
[ ] Documentation

Environment

Python version: 3.4.5
NetBox version: 2.2.6

Description
It is a continuation of NetBox-discussion https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/netbox-discuss/t7J_LL2jMcM/dOvZVqEYAQAJ
Would like to get the closest parent prefix for an IP address using API.
It would be useful for those that use a netbox with external tools. For example, I want to add a list of ip addresses to netbox from a core switch's arp table using API. For each ip address I must set a prefix length. If I could get a parent prefix using API than I could get a prefix length for this IP address.

If you have the ip with mask on CIDR form, just do the following request:
/api/ipam/prefixes/?parent=<IP_ADDRESS>%2F24

In my test it returns the parent prefix of <IP_ADDRESS>/24.

Yep, If I have the ip with mask, i can calculate the prefix and get it using API
But If I don't know a mask, I have to do something like this:
def get_prefix_length_for_ip(ip_address):
    for prefix_length in range(31, 0, -1):
        resp = requests.get(apiBaseUrl + '/ipam/prefixes/?parent=' + ip_address + '/' + str(prefix_length),
                            headers=headers).json()
        if resp['count'] > 0:
            return str(prefix_length)
    return None

This code gives a big load to the Netbox database that's why the function is executing slowly.

Just noticed that you said you didn't have the prefix length. But you do if you have ARP then you have a layer 3 interface and that interface has a mask set, right?

If you have the ip with mask on CIDR form, just do the following request:
/api/ipam/prefixes/?parent=<IP_ADDRESS>%2F24

In my test it returns the parent prefix of <IP_ADDRESS>/24.
Edit: Just noticed that you said you didn't have the prefix length. But you do, if you have ARP then you have a layer 3 interface and that interface has a mask set, right?

It appears that this is incorrect.
For those looking for this feature, the query param is actually "?contains=".


Updated and new tutorials

Many changes and additions, the major one removing the MNIST database so we're not pushing a large binary file.

landing problems.... trying another route.


Comment shortcut not working

The shorthand way of applying a comment - either single or multiple lines - used in other syntax highlighters is not functioning.
Using Sublime Text 3 on OSX, I am generally able to use [Command + /] to comment or uncomment a line (or group of lines) of code. This is not working which makes this plugin a bit cumbersome for debugging.

I've found that the Apache conf syntax works better than this plugin, even though it's not strictly correct.

same thing on Windows. I even tried to find out how to correct this, but can't seem to find...


Doc: pkgs{compressDrv,compressDrvWeb} improve clarity

Description of changes

Note: This doesnt get rendered into any of the manuals:
The only place where people can discover it currently is the code and https://noogle.dev/f/pkgs/compressDrv

Improves clarity
More consistent with the other docs.
Move the extensive example into an example block, which can be collapsed / is collapsed by default in nixpkgs manual.

cc: @motiejus @Ma27
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@motiejus Somwething i missed in the initial pr:
Why do you need ... ?
{ formats, compressors, ... }:
It is not possible to access the 'rest' arguments, and they are not forwarded to anything. Thus calling the function, with superflous arguments will not result in an error.

This looks great, but I can only judge on the surface. Is there a simple-ish way to generate something like https://noogle.dev/f/pkgs/compressDrv locally?

@motiejus Somwething i missed in the initial pr:
Why do you need ... ? { formats, compressors, ... }:
It is not possible to access the 'rest' arguments, and they are not forwarded to anything. Thus calling the function, with superfluous arguments will not result in an error.

This is a left-over from previous PR incarnations and can be removed. You can do it here (AFAIK), or I can do this once this PR is merged -- I let's avoid conflicts either way.

This looks great, but I can only judge on the surface. Is there a simple-ish way to generate something like https://noogle.dev/f/pkgs/compressDrv locally to review rendered changes?

No unfortunately Not. Usually you would just build the Manual. But this doesnt appear in the Manual yet.
Rendering this into the Manual is a Goal but a little complex, since we want a generic solution.
You could build noogle yourself though. This will Take a while If you do it the First time.
You can then Override the nixpkgs-master Input with your branch.
nix build GitHub:nix-community/noogle#ui --override-input nixpkgs-master {your url}

This is perfect!

nix build GitHub:nix-community/noogle#ui --override-input nixpkgs-master {your url}

Any flags I am missing?
[motiejus@vno1-gdrx:~/code/nixpkgs]$ nix build GitHub:nix-community/noogle#ui --override-input nixpkgs-master https://github.com/NixOS/nixpkgs
error: 'GitHub:nix-community/noogle#ui' is not a valid URL

[motiejus@vno1-gdrx:~/code/nixpkgs]$

Github needs to be lowercase. Got autocorrected in my Phone 😅

I'm getting
error: NAR hash mismatch in input 'https://registry.npmjs.org/@corex/deepmerge/-/deepmerge-4.0.43.tgz?narHash=sha256-GlUW2/0Z1wRF1RyrT/La9POas%2Bbv9x%2BTbbCQy9V3bW8%3D', expected 'sha256-GlUW2/0Z1wRF1RyrT/La9POas+bv9x+TbbCQy9V3bW8=' but got 'sha256-Cr1OYmR1+ozY0Hwny+oQ3BHvioS5+suPrMOujPio8+w='

Problem on master?!

Yep. Seems npm registry behavior changed. It'll be fixed anytime (minutes). We learn that npm sucks, and sometimes dependencies randomly change their hash ....

This seems to build for this PR:
nix build github:nix-community/noogle#ui --override-input nixpkgs-master github:hsjobeki/nixpkgs/doc/compressDrv

Both http://localhost:8000/f/pkgs/compressDrvWeb and http://localhost:8000/f/pkgs/compressDrv look great. Thanks!


Low variety output of randomdata.Paragraph()

randomdata.Paragraph()
Seems to output very often almost the same sentance .
It would be good if we could have a bigger variety.
For example when testing big data , with full text search,  having more words to scan would be nice

Agree wit this one. Will add some more paragraphs. Will most likely separate it out into a new json that contains only paragraphs (and quite a few of them)

I am working on a change that will help with this.

Fixed in #45


Support windows OpenJ9

The idea is to start an Attachment thread in a remote process and connect to a socket. Checking OpenJ9 process - find java/lang/J9VMInternals class. Quite a hackish way, even works with -Dcom.ibm.tools.attach.enable=no

Thank you for an interesting addition. I'll consider Windows OpenJ9 support for the next release.
I don't plan to merge the changes in the current form though, they'll need some refactoring first. It's probably better to move OpenJ9 and HotSpot specific parts to different files, and to avoid excessive duplication (e.g. translate_command, print_unescaped...)

Yes, I'm not good at c. In any case, I hope this will help you somehow


Icon

Subject: Icon
Description:
I added speed dial / shortcut to the browser so that I can quickly access this website.
Results:
The Couchers symbol or icon didn't appear. Instead a generic text icon with "Couchers.org" text was added.
Backend version: develop-ee29de23
Frontend version:
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/104.0.5112.97 Safari/537.36 OPR/90.0.4480.54
Page: https://couchers.org/
User: r_a (20434)

dupe https://github.com/Couchers-org/web-frontend/issues/302


Mocking with nock

When trying to mock the requests with nock i get the issue of doubling the uri.

Code is:
nock('https://api.etherscan.io')
  .get(`api`)
  .query({
    module: 'account',
    action: 'balance',
    tag: 'latest',
    address: '0x8d144e6277d0c38047f79df1dfc8b78994c6aa86',
    apiKey: 'YourApiKey'
  })
  .reply(200, responses.ethereumResponse);```

Do you have a example piece of code?

Hi. Yes. It's something like that.
var api = require('etherscan-api').init('YourApiKey');

class EthereumGateway {
    async getBalance(address) {
        let balance;

	balance = (await api.account.balance(address)).result

        return balance;
    }
}

ty
I will have a look.

this should behave differently now. Closing .


firebase_auth _auth.onAuthStateChanged.listen() should pass in an AuthResult

With the new api, shouldn't this method pass in an AuthResult instead of a FirebaseUser?

@collinjackson ?

On iOS the equivalent notification gets a user rather than an AuthResult so I think we should do the same.
typedef void (^FIRAuthStateDidChangeListenerBlock)(FIRAuth *_Nonnull,
                                                   FIRUser *_Nullable)

Here is my train of thought, correct me if I'm wrong:
Sign in and Creation now return an AuthResult instead of a FirebaseUser. I figure that this was changed in order to pass back AdditionalUserInfo since that is the only extra field in AuthResult right now.
    final AuthResult authResult = await _auth.signInWithEmailAndPassword(email: username, password: password);

    final AuthResult authResult = await _auth.createUserWithEmailAndPassword(email: username, password: password);

I would think that updates to this same data as a result of a state change via onAuthStateChanged would also result in an AuthResult in order to retrieve any updated AdditionalUserInfo (which is the extra information contained in the AuthResult). In the same way that signInWithEmailAndPassword returns an AuthResult now.
Currently, onAuthStateChanged is only signin/signout events, but I could see it also containing updated AdditionalUserInfo when that information changes as well. Otherwise, how would my app know that any AdditionalUserInfo has changed?
Hope this explains things a bit further.

@lookfirst
This issue has been moved to https://github.com/FirebaseExtended/flutterfire/issues/1088. Any further collaboration will be done there.


how to know what fields are required from inside a resolver

given the type:
type Node {
  id: String!
  description: String,
  description1: String
}

if I query for:
node {
      id,
      description1
    }

when this is handled by a resolver:
node(root, args, context, info) {
   return {id:'id', description: 'ok', description1:'ok1'}
}

how could I determine within the resolver that I didn't actually need description - ie hw can I know which fields are being asked for.  given that these fields may require additional overhead whch I can save by not needlessly fetching them...

ah ok got it to answer my own question one does:
Node: {
    description(node) {
      return new Promise((resolve) => {
        setTimeout(()=>
        resolve('the answer'), 1000);
      });
    },


Clarify examples for enterprise/core changefeeds on new Changefeed Example page

Fixes DOC-2503
Customer found the new layout of the changefeed examples page confusing. There was a feeling that the Cloud storage examples were hard to find.
Adjusted the layout to make Enterprise changefeeds more prominent in the filter placements. Added a sink breakdown to start with links to the respective sections and added link out to the Changefeed Sinks page to make clear that information on forming sink URIs can be found there.

@taroface Just bumping this up your inbox for when you return! Thanks

Hey @taroface, could you take a review of this when you have time? Thanks!


Fix Neo4jDatabase case sensitivity setting

For now, I explicitly upper-cased returned variables in internal queries so that it matches the results I'd automatically after running a query through Neo4jDatabase.
It turns out AbstractJdbcDatabase instances are not considered case-sensitive by default, which explains the implicit upper-casing.
Neo4jDatabase needs to override AbstractJdbcDatabase#setCaseSensitive and internal queries should go back to using conventional cases for variables.

Blocked by: https://github.com/liquibase/liquibase/issues/3071

https://github.com/liquibase/liquibase/issues/3071 is merged and has been released as part of 4.17.0


Implement "Mixed Content" check

Commit message:

This check ensures that resources embedded within HTML documents are not
requested using a protocol that degrades the security of the original
connection. This avoids "mixed content" warnings in browsers that
enforce consistent protocols, and it precludes the leaking of private
information in browsers that do not.

Resolves gh-542

This looks good! I’ll do a more in-depth review soon, but I‘m quite liking it so far.

Great, thanks! I'll hold off on moving that class variable till you get a chance to finish.

The style checks are failing. Apparently, they’re also failing on other parts of the code—I’ll fix it; ignore the style test results for now.

Can you rebase onto master? That should fix the style issues (at least the ones that are not your fault, heh).

Looks good otherwise!

Alrighty, I've rebased and incorporated your feedback in separate commits. I'm happy to squash when you think this is ready

Cool! OK to squash. I’ll properly test it out tomorrow and will merge it if it’s OK!

Excellent--all squashed

Thanks!

My pleasure :)


[material-ui] Uncontrolled TextField components fail to reset, and their floating labels don't return to default in a  with a reset button

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Steps to reproduce 🕹
Link to live example without using state: https://codesandbox.io/p/sandbox/cranky-chihiro-vc6pq4
Link to live example using state: https://codesandbox.io/p/sandbox/upbeat-joana-2n4dz9
Current behavior 😯

When using TextField component after clearing First TextField (First name), Pressing the reset button in a form overlaps default value of TextField with the placeholder (First name).

When using TextField component after entering some value in second TextField (Last name), Pressing the reset button in a form only reverts value of the TextField, but floating label (Last name) of TextField does not set back to its default position.

When using TextField component after selecting value in second TextField (Last name) from suggestions (previously entered values), Pressing the reset button in a form reverts value of the TextField and floating label returns to its default position while in above case it does not.

If I use TextField with state then it will work properly for all above cases.
Expected behavior 🤔
Value of the TextField will get reset to its default value and floating label (placeholder) also returns to its default position while clicking on the reset button when we are not using the state (for the first codesandbox).
Context 🔦
No response
Your environment 🌎

npx @mui/envinfo
  System:
    OS: Windows 10 10.0.19045
  Binaries:
    Node: 20.9.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 10.1.0 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: Not Found
    Edge: Chromium (120.0.2210.77)
  npmPackages:
    @emotion/react: ^11.11.1 => 11.11.1
    @emotion/styled: ^11.11.0 => 11.11.0
    @mui/base:  5.0.0-beta.28
    @mui/core-downloads-tracker:  5.15.1
    @mui/material: ^5.15.1 => 5.15.1
    @mui/private-theming:  5.15.1
    @mui/styled-engine:  5.15.1
    @mui/system:  5.15.1
    @mui/types:  7.2.11
    @mui/utils:  5.15.1
    @types/react: ^18.2.45 => 18.2.45
    react: ^18.2.0 => 18.2.0
    react-dom: ^18.2.0 => 18.2.0
    typescript: ^4.9.5 => 4.9.5

Hi @DiegoAndai if I have it controlled with a state variable, and I use value pointing to the state variable, and onChange to update the state variable, when I programmatically update an empty text box to have a value (by updating the state variable in an effect), the floating label doesn't float up. Instead it overlaps over the filled in value:

It seems the floating label will only float up and stay there when I interact with the input with my mouse and keyboard, not when the value is filled in programmatically. Seems like a weird bug. Is this related to this issue, is there an existing issue, or should I open a new issue?


Visit Contact Mode

There is need for more flexibility in the visit method & contact mode. It should be possible to state such options as "mode X or mode Y if X not possible"

[ ] @cupkes Propose change to USDM
[ ] Work with @jared-schreibman-cdisc to develop API
[ ] Add relation name or attribute names to CT


Added region tag for inclusion in c.g.c.

Intending to use on new standalone page for this module.
See cl/383472579.

Thanks for the PR! 🚀✅ Lint checks have passed.


add -sech_symmetric_key option to s_server

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Hey, @okeefeja, you should merge this when you're ready.


[test-framework] Merge from master

Planning daily merges from master to dev branches. This is first in series.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: wattli
Assign the PR to them by writing /assign @wattli in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
998818d5adc343eafa21754e54bd98389de8c3c8
link
/test istio-unit-tests

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

/lgtm

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nmittler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: wattli
Assign the PR to them by writing /assign @wattli in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


metrics-endpoint (interface: prometheus_scrape) is not implemented

Bug Description
Although the README states that metrics-endpoint is supposed, there is no metadata definition in the charm to use it.
https://github.com/canonical/grafana-agent-operator/blob/49abf15ffcde20d02961118f921c34565084a310/README.md#L48-L59
The downside of not having the implementation is that ceph-mon for example has to be scraped by prometheus so managing firewall policy, etc. is challenging than having grafana-agent with remote_write in the middle.
More context: https://bugs.launchpad.net/charm-ceph-mon/+bug/2041167
To Reproduce

deploy ceph-mon and grafana-agent charm in MAAS provider
relate those two

Environment
ceph-mon           17.2.6   active      3  ceph-mon                quincy/stable  185  no       Unit is ready and clustered
grafana-agent               active     13  grafana-agent           latest/edge     20  no

Relevant log output
$ juju integrate ceph-mon grafana-agent
ERROR cannot add relation "grafana-agent:juju-info ceph-mon:juju-info"
relation grafana-agent:juju-info ceph-mon:juju-info (already exists):

$ juju integrate ceph-mon grafana-agent:metrics-endpoint
ERROR application "grafana-agent" has no "metrics-endpoint" relation

Additional context
No response

Not sure which side needs work though. Looks like the grafana-agent charm has a completely new interface as cos_agent so it's either:

Document should be updated by removing the mention to metrics-endpoint, and every single charm out there should implement the cos_agent relation.
Implement metrics-endpoint relation in grafana-agent charm so existing charms with metrics-endpoint do not require modifications

I haven't looked into the detail of the difference of those two relations though.
https://github.com/canonical/grafana-agent-operator/blob/49abf15ffcde20d02961118f921c34565084a310/metadata.yaml#L38-L51

Hi Nobuto,
Thank you for the issue! Yes, the docs need to be updated. We quite recently split the two different charms from a prior monorepo.
There are still some documentation that needs an update as a result of the split. But you are correct, cos_agent is the interface that should be used by machine charms that wish to natively integrate with the grafana agent subordinate.

Closing this as cos-agent will be the only supported relation between principals and the GA subordinate. ceph-monnow also correctly implements the relation, thanks @sabaini!


Add mssql tools to PATH

Getting an error when trying to use the library with pyodbc, and believe its related to the tools not being added to PATH
  │ pyodbc.Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib
  │ 'ODBC Driver 18 for SQL Server' : file not found (0) (SQLDriverConnect)")

As per the docs the add to path steps appear to be missing, so proposing to add them

Didn't turn out to be the issue!


[Bug]: Python streaming LoadTests failing since July 8

What happened?
https://ci-beam.apache.org/job/beam_LoadTests_Python_ParDo_Dataflow_Streaming/
06:01:37                      [--input_options INPUT_OPTIONS] [--timeout_ms TIMEOUT_MS]
06:01:37                      [--iterations ITERATIONS]
06:01:37                      [--number_of_counter_operations NUMBER_OF_COUNTER_OPERATIONS]
06:01:37                      [--number_of_counters=0 NUMBER_OF_COUNTERS=0]
06:01:37 pardo_test.py: error: argument --number_of_counters=0: expected one argument

Issue Priority
Priority: 1
Issue Component
Component: build-system

Looks like this started with https://github.com/apache/beam/commit/20949a9d0842e1bd397a17cc0e49de100fa1b29b,

This is a gross one.
The problem is the space in the argument --experiments=use_runner_v2, shuffle_mode=appliance
shuffle_mode=appliance is being interpreted as a separate argument because there's a space and it's not quoted. Then after we parse the known arguments we're left with the unknown_args:
['--iterations=10', '--number_of_counter_operations=0', '--number_of_counters=0', 'shuffle_mode=appliance']

The logic interprets shuffle_mode=appliance as the argument for an option with name number_of_counters=0.
We can fix by changing use_runner_v2, shuffle_mode=appliance, use_runner_v2,shuffle_mode=appliance, or nmore generally by adding quoting logic in LoadTestsBuilder.groovy. We should also adjust the retain_unknown_options logic. It's bad that it assumes the arguments are consecutive.

cc: @robertwb


nova, glance need keystone, ...

when we merged jeblair's patch to only install services that are required it broke devstack.
This makes devstack glance/keystone/nova api run even if all aren't on the same box

To deal with this long term:
https://bugs.launchpad.net/devstack/+bug/891437

looks good for diablo, to help get multi-host installations working.

verified that this works in aio

lgtm

would be nice to eventually get a cleaner way to define deps - lgtm for now though


chore: set startup parameters to secure Log4j2

Checklist

[x] the individual contributor license agreement is signed
[x] commit message follows commit guidelines

Description of change

While uPortal and portlets are not susceptible to CVE-2021-44228, including the startup parameters to help guard custom code that may include Log4j2.

I don't understand this PR as the CVE doesn't affect uPortal and his portlets embedded. More when it's easy to upgrade a library version. In my mind, that's the responsibility of everyone to add these parameters if needed. Also this is needed only temporarily, until the upgrade is done as it's require only a lib upgrade or using Java 11.
So this shouldn't be merged and should stay as an example. Else the change are good if you need them.

I have already heard from two institutions that have Log4j in their uPortal customizations. So this would help them until they have time to fully upgrade. Yes, it would be up to them and this is an example, but I see no harm in adding this either. Also, there is talk that Log4j is not the only susceptible code.
I am fine not merging this, but I would prefer we lean on the side of caution.


update packaging.py

made changes on string function and on some operators :

Can one of the admins verify this patch?

yep :)

ok to test

Please use more descriptive titles; the fact that it's a change to packaging.py is obvious, but what does the change accomplish?  "update" is too general.


Footer adapter example RecylerView error

I've implemented the footer sample code and i am getting complaints from the RecylerView: " Cannot call this method in a scroll callback. Scroll callbacks mightbe run during a measure & layout pass where you cannot change theRecyclerView data. Any method call that might change the structureof the RecyclerView or the adapter contents should be postponed tothe next frame."
java.lang.IllegalStateException:
                                                                         at android.support.v7.widget.RecyclerView.assertNotInLayoutOrScroll(RecyclerView.java:2581)
                                                                         at android.support.v7.widget.RecyclerView$RecyclerViewDataObserver.onItemRangeInserted(RecyclerView.java:4951)
                                                                         at android.support.v7.widget.RecyclerView$AdapterDataObservable.notifyItemRangeInserted(RecyclerView.java:11383)
                                                                         at android.support.v7.widget.RecyclerView$Adapter.notifyItemRangeInserted(RecyclerView.java:6779)
                                                                         at com.mikepenz.fastadapter.FastAdapter.notifyAdapterItemRangeInserted(FastAdapter.java:1588)
                                                                         at com.mikepenz.fastadapter.adapters.ItemAdapter.add(ItemAdapter.java:366)
                                                                         at com.mikepenz.fastadapter.adapters.ItemAdapter.add(ItemAdapter.java:349)
                                                                         at ***.***.***.AbstractMainListFragment$1.onLoadMore(AbstractMainListFragment.java:47)
                                                                         at com.mikepenz.fastadapter_extensions.scroll.EndlessRecyclerOnScrollListener.onScrolled(EndlessRecyclerOnScrollListener.java:122)
                                                                         at android.support.v7.widget.RecyclerView.dispatchOnScrolled(RecyclerView.java:4618)
                                                                         at android.support.v7.widget.RecyclerView$ViewFlinger.run(RecyclerView.java:4776)
                                                                         at android.view.Choreographer$CallbackRecord.run(Choreographer.java:873)
                                                                         at android.view.Choreographer.doCallbacks(Choreographer.java:676)
                                                                         at android.view.Choreographer.doFrame(Choreographer.java:603)
                                                                         at android.view.Choreographer$FrameDisplayEventReceiver.run(Choreographer.java:859)
                                                                         at android.os.Handler.handleCallback(Handler.java:739)
                                                                         at android.os.Handler.dispatchMessage(Handler.java:95)
                                                                         at android.os.Looper.loop(Looper.java:168)
                                                                         at android.app.ActivityThread.main(ActivityThread.java:5845)
                                                                         at java.lang.reflect.Method.invoke(Native Method)
                                                                         at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:797)
                                                                         at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:687)

The Load more works as expected, but this is something that should be addressed in the sample?
The relevant code that triggers the error:
recyclerView.addOnScrollListener(new EndlessRecyclerOnScrollListener(footerAdapter) {
            @Override
            public void onLoadMore(final int currentPage) {
                footerAdapter.clear();
                footerAdapter.add(new ProgressItem().withEnabled(false));
});

Edit:  Just to note, wrapping the adapter clear and add methods in a Handler/Runnable eliminates the issue as it postpones the operation after onScroll has finished in the recyclerview,.

@engineers-bojan "wrapping the adapter clear and add methods in a Handler/Runnable" is the proper solution as also noted by the error message

Does the crash occur in the sample? If yes please open a PR

Not sure about the actual sample, just copied the relevant code parts in a project i am testing but probably happens there too. (i'll test it later and PR if i manage to get some time)

Thank you

Please report back. closing for now


Odd behavoir if net object in environment

Hi @dblodgett-usgs here is an odd one! If a user has an object called net in the global environment, it causes many hydroloom functions to fail.  Can't select columns that don't exist.
#> ✖ Column `id` doesn't exist.

Created on 2024-06-27 with reprex v2.1.0
Sometimes it fails with #> ✖ Column id doesn't exist., sometimes with #> ✖ Column toid doesn't exist. I cant find any rhyme or reason to that difference

Thanks for the report @mikejohnson51 -- this appears to be a bug in my use of exists -- I did not specify where parameter so it's looking in the global environment! Whoops!!!


Intended workflow with the name property of an article

Hi Paul, me again! 😅
I've been using v0.0.2 for a while, and figured out that the Nunjucks env only has a date filter added, under lib/render/index.js for parsing the indiekit.json configuration as well as the post templates (whether default or custom).
I've attempted publishing an article using Celestial and Indiekit v0.0.2. The expected path and permalink were configured to:
"path": "src/content/articles/{{ published | date('yyyy-MM-dd') }}-{{ name }}.md",
"url": "/articles/{{ published | date('yyyy/MM') }}/{{ name }}/"

What I didn't realize was, unlike a note, a user will write and expect the title of an article to show up exactly as he/she/they type it.
For On Micropub, they'll expect to see On Micropub. However, the path and the URL would ideally be slugified, leaving the name property untouched outside of this context. The idea then would be to use a slug filter for the name variable above, but not in the template when storing a, say, YAML front matter for title:
"path": "src/content/articles/{{ published | date('yyyy-MM-dd') }}-{{ name | slug }}.md",
"url": "/articles/{{ published | date('yyyy/MM') }}/{{ name | slug }}/"

title: {{ name }}

I am unsure if this is something that has been considered and taken care of in the upcoming v0.1.0 release. I tried to read through some of the newer code but unfortunately could not make as much sense of it as I'd have liked -- perhaps that will come with actual usage post a stable or beta 0.x release.
An alternate I considered and discarded was to use Nunjucks' built in filters: {{ name | lower | urlencode }}. This is somewhat acceptable for URLs but it would be very unfriendly for a user to see %20s in their file names.
Wondered what your thoughts were on this? Perhaps I am missing something obvious here?

Have you tried the following:
"path": "src/content/articles/{{ published | date('yyyy-MM-dd') }}-{{ slug }}.md",
"url": "/articles/{{ published | date('yyyy/MM') }}/{{ slug }}/"

slug takes its value from either a given slug property (slug, mp-slug), the slugified name property, or a 5-chartacre long random string. If you are publishing an article, you’ll have a name property, so it’ll be one of the first two. The same behaviour is used in v0.1.0 as well.

That works, thank you!
As far as I am aware, accepting a slug property for an article deviates from the spec. I could be wrong, though! Personally, however, I like this implementation. Having both title and slug offered to the user is just good, and I would argue even baseline, UX.
If this does deviate from the Micropub spec, maybe it's worth documenting this on the article wiki page + opening an issue on the spec repo, so this might eventually be a part of the spec.

The slug property does deviate from the spec, but some clients provided it as an option in the early days. Here’s Quill for example:

slug has been superseded by a namespaced value, mp-slug. I may do a little review to what clients are providing now, as would like to remove old and deprecated things like this.

Sorry, I didn't mean slug as in the property - but slug as in the concept. If this is already considered to be a valid microformats2/Micropub property for an article, I'll add it to the wiki.

The article post type uses h-entry schema, which doesn’t have a slug property (it does have u-url, which includes the slug of course). As a slug is only relevant when creating a post, that’s why Micropub offers the mp-slug extension. Should h-entry offer a property to store a slug value? I’m not so sure… but maybe I’ve not fully understood your question? 🤪

You're correct - this is why mp-slug is a Micropub property. I've just generally seen it used with notes, likes, replies, and so on -- but not with the article post type in particular. Maybe the mp-slug property has no such binding to any post type and can be used universally if seen to be appropriate. Inadequate documentation throwing me off, I guess. 🤷♀
I'll update the article wiki entry in this case, and close this issue. 😊

Ah, interesting. I hadn’t noticed that discrepancy… if you give Indiekit a slug, it’ll use it without question!

I've updated and added a new example for an article on the Micropub page, but I think I should probably also consult the IndieWeb folks on IRC.


write test harness in node, not php

You already have node as a requirement.  Writing the server side of the tests in javascript and node would make the test harness much easier to run.  I still haven't figured out how to get php working on mac os x.  A node based backend could be as easy as node tests/server.

You're right. I just don't have much time for this, and I'm not really used to nodejs at the time. If you have the time to migrate the tests, I'd be pleased :p

I'm closing it by lack of time to migrate PHP tests. If someone want to contribute, please reopen ;)


Supabase CLI just hangs with no response (supabase login, supabase init)

I'm having an issue using the Supabase CLI where it's just hanging forever without any output.  Here's what happened.

I installed the Supabase CLI w/ Scoop on windows
I used supabase login to set my access token (it worked initially)
I created my project folder, initialized git, and tried to use supabase init
I was told I had to have Docker running, which I didnt, so I installed Docker Desktop, updated WSL, and got Docker running
Now I'm unable to use supabase init or supabase login.  The commands display no output and they just hang requiring me to Ctrl+C out.

I've tried reinstalling supabase and restarting my computer.  I found Issue 819 which is  a similar issue but nothing there helped me to resolve this.
What can I do?  Is there some way I can logout?  supabase logout is not a valid command.  I'm thinking maybe if I could log out it might work but I have no idea how to do that.

Thanks for detailed steps to reproduce. I'm going to take a look at this today or tmr and get back to you.

I just want to add some more info.  I tried to use the Supabase CLI from another machine (which already had it & Docker installed as I was testing self-hosting a week ago on it) and I got the same result with supabase login and supabase init (hanging, no output).  I was using the scoop 1.48 version.
Additionally, I removed Supabase CLI from scoop and installed it into a new project folder using NPM.  This gave me version 1.49.4 of the Supbase CLI.  Using npx supabase login and npx supabase init once again hung without any output.

And now it randomly started working after another computer restart.  I have no idea what the cause of this was.  I restarted the computer before and it didn't fix it and I tried another computer and it didn't work so why it works now is befuddling.
I would suggest that the supabase CLI commands be more verbose with their output, so that if it does fail, you're not completely in the dark as to what's going on.

Yup adding more verbose logs would definitely help pinpoint such errors. It's surprising to me that supabase init hangs because its implementation is rather straightforward.

I had the same situation and it started working once I restarted. I think something weird was happening with my desktop manager and the keyring. I wasn't seeing the keyring prompt my password but once I restarted I started seeing it.

Same with scoop. I'm using windows 11.
both are asking for access token but I can't input anything.

I just realize you can input anything, paste or type, but it's not showing what you input. Paste token then login works on my end.

I typed the access token manually in the terminal, and that worked for me. However, as I was typing, there was no indication that I was actually typing anything.

I had the same issue and found the solution in this thread. Supabase tries to store the SUPABASE_ACCESS_TOKEN and SUPABASE_DB_PASSWORD in some system keychain (e.g. gnome), and hangs without response.
See if you can get debug output with --debug after exporting both your access token and db password with
export SUPABASE_ACCESS_TOKEN=...
export SUPABASE_DB_PASSWORD=...

Following Bewinxed's comment  sudo service gdm stop for killing the gnome desktop client.
With desktop client down and both variables exported, I was able to login and link.

Closing this as supabase login will now login through the browser by default.


modify libtextsort.gyp to add ".h .hpp" file to generated project

Generated header files are not included in generated project file, but we often need to scan code in header files when we finish project.

Thanks! Two requests:

Can you fill in our CLA at https://opensource.dropbox.com/cla/ ?
The support lib files have changed a bit. I like the use of find to locate them, and there are also header files directly under support-lib used by both the Java and ObjC libraries.

Alternative changes have already been merged which include headers in generated projects, so this diff is no longer needed.


Question: Builds C# in Docker or copies locally built binaries to Docker?

When running a command like dotnet publish --os linux --arch x64 -c Debug -p:PublishProfile=DefaultContainer is this building the code in Docker or is it building on the local machine and then the Docker build copies the binaries into the image?
The company I work for has been building C# code in Docker and something we wanted to move towards is auto generation of the Dockerfiles. I came across this project and it seemed to be along the lines of what we were looking for, but I noticed when running the above command the bin and obj directories were populated on my local machine. To me that suggests all of the building is occurring on the local machine and it is copying the built binaries into an image, but was hoping if anyone could offer clarification. Is my assumption correct?
Thanks for reading and appreciate any information to help clarify how this works!

That's exactly right - we build your app (targeting it at the specified runtime) and handle creating the container image file system layers and manifests/configuration required to run it purely in .Net. This is exactly what traditional container builders do, just without the additional requirement/capability of a Linux VM to handle RUN commands.
Regarding generating Dockerfiles, there is an open issue in this repo to add a target that would do that (either instead of building the container or in addition to). Would that be useful to you? What would you like to see in a generated Dockerfile?

This is exactly what traditional container builders do,

One big caveat here: if you have a RUN dotnet build step in a Dockerfile, you'll get the guaranteed-uniform consistent environment of the image you're running in. With SDK-integrated container builds, you are getting the SDK from the host, with less guarantees about always being the exact same version. That gives many benefits but may mean you choose to build in the Dockerfile if that level of locked-downness is important to you.

@Cjewett did you get the chance to dig into your needs more? Happy to continue the conversation.

@baronfel Sorry, hard to get time for open-source these days. Let me create the example now and I'll send it over with an explanation of why we do what we do.

@baronfel I updated the README here which describes how we have been thinking about this problem. I can go into more detail if needed and definitely would love to hear feedback.

@baronfel I think this ask is similar to https://github.com/dotnet/sdk-container-builds/issues/146 and wouldn't be opposed if you close this out in favor of that. Also if you have any guidance on where one would start to look at in code to step through and start understanding the internals I'm all ears.


When sprig, closing HTML tag in

Describe the bug
When sprig, closing HTML tag in

Rendered Result:

placeholder

placeholder

Versions

Plugin version:1.0.0-beta.16
Craft version:  <IP_ADDRESS>

What are you trying to do with this code? JavaScript code is generally placed inside of

Either way, you should try to avoid having  is not invalid HTML?

Just weighing in here. It's not an option to use <template> for Formie as part of our commitment to front-end support. It's not supported in IE10/11 for example.
I suppose as @bencroker mentions, its being modified by the DOM library, so it's probably an issue on their end.
Formie's repeater field uses  in its templates.

Ok reopening and I’ll see what can be done to make this work.

Fixed in https://github.com/putyourlightson/craft-sprig/commit/bf4a9ab7d073661d6d3b71601071a6bf02a72e64 and released in version 1.0.0-beta.17.


Basic shop functionality

The logic for an in-game shop could be based around the following:

A shop behavior that could be attached to an NPC, a room, or an item (for example, a vending machine). This would be configured with IDs of the items available, the quantity available (defaulting to infinite), and the cost.

A list command that would list all items available at the shop if applicable.

A buy command that would allow purchase if the player has enough currency. buy <item or #>

A sell command that would allow the player to sell one of their items for currency if the shop allows. A nice-to-have would be having the shop keep the item in its inventory, at least until reboot.

One prerequisite for this is currency.

OTOMH I'd say this could be done with a behavior like lootable where the config is just a list of items for sale and their price. But yeah, definitely need currency

Closed with a78e0feacf40abcc915571506c78419a9ab6244d


[UAccordion] Error on deleting items from Accordion Items Array

Environment

Operating System: Linux
Node Version:     v18.18.0
Nuxt Version:     3.8.0
CLI Version:      3.9.1
Nitro Version:    2.7.2
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      modules
Runtime Modules<EMAIL_ADDRESS>Build Modules:    -

Version
v2.7.2
Reproduction
https://stackblitz.com/edit/nuxt-ui-eus9qt?file=app.vue
Description
When deleting items from Item array. Getting errors on closeOthers.
vue.js?v=98314ebf:1628 Uncaught TypeError: Cannot read properties of undefined (reading 'closeOthers')
at Proxy.closeOthers (Accordion.vue:72:38)
at onClick (Accordion.vue:140:38)
at vue.js?v=98314ebf:9415:60
at callWithErrorHandling (vue.js?v=98314ebf:1565:18)
at callWithAsyncErrorHandling (vue.js?v=98314ebf:1573:17)
at callWithAsyncErrorHandling (vue.js?v=98314ebf:1583:17)
at HTMLDivElement.invoker (vue.js?v=98314ebf:9397:5)
Additional context
No response
Logs
No response

Issue was with click event bubbling to the top from the default item slot. It is working fine now.


Allow default content in content_tag in case of noscript/render failure

I would like to specify some default content that renders serverside within the content tag generated by the react_component helper. I've noticed that it defaults to calling content_tag with a nil body and created my own react_component_with_defaults helper like so:
def react_component_with_defaults(name, props = {}, options = {}, &block)
    tag = options.delete(:tag) || :div
    data = { data: { 'react-class' => name, 'react-props' => props.to_json } }

    content = block_given? ? capture(&block) : nil
    content_tag(tag, content, options.deep_merge(data))
end

This would theoretically allow me to pass a block with default content to render serverside as a fallback in the event of someone having js disabled or a mounting issue when the react js finally executes.
<%= react_component_with_defaults 'SomeComponent', { some_prop: 1 } do %>

default content here

<% end %>

Unfortunately the javascript side of webpacker-react prevents rendering if there is any innerHTML to the content tag (See relevant line)
Are there any arguments against adding this as a feature? Or is this just something that react wouldn't like with automatic hydration attempts? I'm not knowledgeable enough on react internals to know if this is a bad idea or not.

This comes up relatively often https://github.com/reactjs/react-rails/issues/901#issuecomment-412746677
https://github.com/reactjs/react-rails/issues/913#issuecomment-407677428
In general Ruby and JS contexts can't really share anything other than props. You could pass in child html though and dangerously render it to simulate react wrapping ruby content but not really recommended due to the dangerously set inner html call.

I don't think I was clear enough. I don't care about the content in the block surviving. I just want some default server rendered content that will display if the user has javascript disabled or in the event that the initial mount fails. Think of it like having a ..some serverside rendered version of the app.... My current solution is to use the noscript tag, but it doesn't kick in if there was some other javascript error before mounting.  If react mounts properly, it can destroy the current markup in the tag and render it's own tree. I'm fine with that, if that's how react would behave (I wasn't totally sure).

Ah yes it seems like I misunderstood what you were after, sorry for that.
You wonder if there's a nice way of doing it within the react component taking advantage of the lifecycle hooks. Such as have a default noscript render, then remove it on something like componentWillMount which only triggers when JS is mounting client-side.

I am open to have this feature implemented. With an optional block argument on the react_component helper we should not need to add a new helper (like your react_component_with_defaults).
One drawback is that if your pack file is loaded with async there might be a flash of content on page load, as the browser will display the HTML content, and our JS will remove the content from the DOM before calling ReactDOM.render.
A solutions would be to add a  tag around the block provided to react_component, so it will only be rendered if JS is disabled. Would this work for you?

I'm already using the  tag in conjunction with a react_component call for progressive enhancement purposes:
<%= react_component 'CommentFeedWithFancyInteractivity', comments: @comments %>

  <% @comments.each do |comment| %>

{comment.author}: {comment.body}

  <% end %>

This pretty much gets the job done. I think the only improvement would be in the super rare case that javascript is enabled, but somehow the mounting of the component failed. Maybe an error in the bootstrapping of the react component or app, or some browser incompatibility. It's in this case that the react version of the content never shows because of the error, and the noscript version of the content doesn't display because the browser has javascript enabled. I think supporting serverside rendering of the react components would also solve this unique case.
I wouldn't say that this is a defect or shortcoming of this gem at all. I only opened the issue because my brain thought "Wouldn't it be cool if it worked like this?", so I tried it.
I wouldn't put this high on any priority list since  gets 90% of the way. It could even be closed if you think it's too niche.


Error paloma

Paloma version 1.9.0, pigeon version 1.9.1, I run the node using snapshot but I keep getting an error
Started Paloma Node.
palomad.service: Main process exited, code=killed, status=9/KILL
palomad.service: Failed with result 'signal'.
sudo apt update
sudo apt install -y curl git jq lz4 build-essential unzip
bash <(curl -s "https://raw.githubusercontent.com/nodejumper-org/cosmos-scripts/master/utils/go_install.sh")
source .bash_profile
curl -L https://github.com/CosmWasm/wasmvm/releases/download/v1.4.0/libwasmvm.x86_64.so > libwasmvm.x86_64.so
sudo mv -f libwasmvm.x86_64.so /usr/lib/libwasmvm.x86_64.so
git clone https://github.com/palomachain/paloma.git
cd paloma
git checkout v1.9.0
make build
sudo mv ./build/palomad /usr/local/bin/palomad
NODE_MONIKER="ERN"
palomad config chain-id messenger
palomad init "$NODE_MONIKER" --chain-id messenger
wget -O ~/.paloma/config/genesis.json https://raw.githubusercontent.com/palomachain/mainnet/master/messenger/genesis.json
wget -O ~/.paloma/config/addrbook.json https://raw.githubusercontent.com/palomachain/mainnet/master/messenger/addrbook.json
curl https://snapshots.nodejumper.io/paloma/paloma_latest.tar.lz4 | lz4 -dc - | tar -xf - -C $HOME/.paloma
palomad start

I can see you're using v1.4.0 for your setup. The README actually still lists this, but we've rolled back to 1.3.0. I'm not convinced this is connected, but can you try falling changing your local lib to 1.3.0 again. Also make sure to clear the cache as instructed on the README.

Closing as fixed.


EEL DSL for a CLI shell

EEL DSL for a CLI shell

A Scala DSL for EEL commands.
The Scala REPL to be used as an interactive shell
Scala variables, loops and conditional statements can be combined with DSL commands to easily script tasks
A bootstrap shell script called eel-shell similar to spark-shell - will automatically import the eel DSL packages
Also automatically import OS packages to allow for OS commands to be run and combined with the DSL

Import
Import data with options from an EEL source to a sink
import from jdbc
with driver=blah,url=blah,sql=blah
to hive
with db=blah,table=blah
Import from jdbc
with driver=blah,url=blah,sql=blah
to Parquet
with path=blah
Export
Export data with options to an EEL sink from a source
export to hive
with db=blah,table=blah
from jdbc
with driver=blah,url=blah,sql=blah
export to hive
with db=blah,table=blah
from Parquet
with path=blah

Note allow a transform sub-command at the appropriate place to do custom transformations on the underlying rows in the frame.

More commands to follow....

DDL
Display the DDL from create table command for JDBC query:
ddl  from JDBC
with        driver=blah,
url=jdbc:blah,
sql=blah,
dialect=Parquet,
location=blah,
sql=blah
partitions=p1:string,p2:int

partitions are specified using the notation name:type

File Compaction
Reduce several small files into a single file for a folder or hive table.
Note for Parquet or other similar types the new file's schema should be the union of all schemas.
In addition you may have to add columns to the HiveMetasore.

compact Parquet path
compact Orc path
compact Hive dbName table


Log query/channel messages to disk

ZNC separates it out by day by default. I'm wondering how specific we should get with it. We'll likely do something like a custom format so peeps can have it however they want, with a default of per-day.

This will just be a simple writing lines to files thing, same as ZNC does. Config will likely be something like: enabled (bool), format (strong), root directory for logs (defaults to ./log if no choice is made).

This was part of https://github.com/goshuirc/bnc/issues/18


Refactor application services to use ApplicationServiceInvoker

Right now every service api endpoint is a bit of a big mess (switch cases with run blockings and so on. With a recent core update there is now a possibility to handle every request in a way simpler way.
Requirements:

Authorization should be handled by an application service decorator

There is no need to switch on request types as base request can be invoked via the application service invoker

I think we also have deviated from the suggested authorization requirements for some of the core endpoints, double-check if that’s the case or not.

Done in #38


_remove_control_recursively takes an extremely long time (or forever?)

This is a placeholder as I investigate and try to narrow down a test case.
I have a fairly complex app that has a container where I add and remove controls as the user moves through the app. (switching between content).
This became very very slow in some cases and in other cases never seems to end.  I added a print to the _remove_control_recursively function that shows the same controls being run through over and over (in one case, I have an icon that was in the function 120 times before I stopped it (I use id(control) to ensure I am looking at the same object.)
This is an example of an item on the screen, there are 7 of these on screen when this issue occurs.

(so seven of these are being removed, each one of these is a user control that contains 4 other user controls and 4 standard Flet controls, so the container has about 56 controls on, all being removed before new controls are added)

Code example to reproduce the issue:
(paste your program here)

Describe the results you received:

Describe the results you expected:

Additional information you deem important (e.g. issue happens only occasionally):
Flet version (pip show flet):
Name: flet
Version: 0.6.0.dev1292
Summary: Flet for Python - easily build interactive multi-platform apps in Python
Home-page:
Author: Appveyor Systems Inc.
Author-email<EMAIL_ADDRESS>License: Apache-2.0
Location: /Users/gregor.brandt/Documents/Startbridge/web-app2/fe_env/lib/python3.11/site-packages
Requires: flet-core, httpx, oauthlib, packaging, watchdog, websocket-client, websockets
Required-by:

Operating system:

Additional environment details:

commenting out
         for child in control._previous_children:
             print(f"recursive: previous_children: {control} {id(control)} child: {child} {id(child)}")
             removed_controls.extend(self._remove_control_recursively(index, child))

in the function makes things return to as they were in 0.4.2, everything works as I switch pages

Side note: looks like you need Badge control for your app: https://github.com/flet-dev/flet/issues/1264

Side note: looks like you need Badge control for your app: #1264

I wrote my own but a built in one is always better. 🙂


Server WARN

Warn: https://pastebin.com/ir8YR2Y5

This isn't related to ViaVersion nor something we can fix.
Looks like bad host name, read the error.


Windows iOS device provision

go-ios supports up to iOS 16 on any OS, currently the provider does not provision iOS devices on Windows so I should add this

To do when go-ios starts supporting iOS 17 on Windows


feat: support license

What's changed and what's your intention?
To use a license, users need to

store the license (JWT Token) in K8s Secret with key license
set the RisingWave CRD .license.secretName to the secret name

Checklist

[x] I have written the necessary docs and comments
[x] I have added necessary unit tests and integration tests

Refer to a related PR or issue link (optional)

https://github.com/risingwavelabs/risingwave/issues/17446
https://github.com/risingwavelabs/risingwave/pull/17396

@arkbriar make generate-all it will make such a diff. Is it expected or did I did anything wrong?

@arkbriar make generate-all it will make such a diff. Is it expected or did I did anything wrong?

Could you try remove the local bin dir and try again?


feat: support extra parameter in Logger messages

Issue #, if available: #253
Description of changes:

This PR adds support for extra parameter when logging messages across all levels. Any key/value added using extra parameter will be at the root level of the structured log.
Before this PR, all fields added in extra parameter are discarded when structuring logs.
UX
Add request_id field while logging a message
logger = Logger(service="payment")

fields = { "request_id": "1123" }
logger.info("Hello", extra=fields)

{
    "level": "INFO",
    "location": "<module>:5",
    "message": "Hello",
    "timestamp": "2021-01-12 14:08:12,357",
    "service": "payment",
    "sampling_rate": 0.0,
    "request_id": "1123"
}

FAQ
Q: What's the difference between structure_log and extra after this PR?
Keys added with structure_log will persist across multiple log messages while keys added via extra will only be available in a given log message operation.
Example - Persisting payment_id not request_id
from aws_lambda_powertools import Logger

logger = Logger(service="payment")
logger.structure_logs(append=True, payment_id="123456789")

try:
    booking_id = book_flight()
    logger.info("Flight booked successfully", extra={ "booking_id": booking_id})
except BookingReservationError:
    ...

logger.info("goodbye")

Note that payment_id remains in both log messages while booking_id is only available in the first message.
{
    "level": "INFO",
    "location": "<module>:5",
    "message": "Flight booked successfully",
    "timestamp": "2021-01-12 14:09:10,859",
    "service": "payment",
    "sampling_rate": 0.0,
    "payment_id": "123456789",
    "booking_id": "75edbad0-0857-4fc9-b547-6180e2f7959b"
},
{
    "level": "INFO",
    "location": "<module>:6",
    "message": "goodbye",
    "timestamp": "2021-01-12 14:09:10,860",
    "service": "payment",
    "sampling_rate": 0.0,
    "payment_id": "123456789"
}

Checklist

[x] Meet tenets criteria
[x] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

customer approvals: https://github.com/awslabs/aws-lambda-powertools-python/issues/253#issuecomment-758656236

customer approvals: https://github.com/awslabs/aws-lambda-powertools-python/issues/253#issuecomment-758656236


Generate /root/.astropy on the fly

Call astopy to force it to create /root/.astropy
Removed unused Dockerfile.

Theoretically this makes the computation time-dependent, as updates in the folder ".astropy" are not controlled by us. I propose an intermediate image that would optionally create this data if the local copy needs to be overridden.

But I don't see why this is a problem. When you create the image, you lock in .astropy and then you have fully reproducible system, since that image is tagged and versioned. The only think you need to specify is the image that you did analysis with.

One thing to consider is that it takes time to pull this file, so any tweak in the underlying image (cppyy) will lead to a reload... I guess this is acceptable.
On the other hand, can we keep an option to use a local copy of .astropy? It can even be coded in the Dockerfile, and of course it's even easier to have both.


Pr chartjs time options

Hi @gvreddy04
Here's another pull request that allows you to use time scales in the chart, provided a date adapter for Chart.js is loaded.

"The format of the analysis property sonar.login= is invalid", @gvreddy04 Could you assist in this one? Multiple PR are stuck on this.


Corrected variable name

changed sleep_time to wait_time on line 51. Should now match the argparser variable name.

Good find. Thanks for PR!


HTML Media Capture is at a Candidate Recommendation as of 09 May 2013. I would like this updated on ...

HTML Media Capture is at a Candidate Recommendation as of 09 May 2013. I would like this updated on caniuse and also renamed to 'HTML Media Capture'.
Includes MediaStreamTrack
http://www.w3.org/TR/mediacapture-streams/

This issue was imported from Google Moderator
Moderator votes: +10
Issue added by simzy39 on 2013-05-09
To vote this issue up or down, simply include +1 or -1 in your comment.

Spec: http://www.w3.org/TR/html-media-capture/

Includes MediaStreamTrack
http://www.w3.org/TR/mediacapture-streams/

That part is already present as http://caniuse.com/#feat=stream

+1 The CR spec is here by the way: https://www.w3.org/TR/2013/CR-html-media-capture-20130509/

+1

Now available at http://caniuse.com/#feat=html-media-capture

@Fyrd Updated 2019:
https://developer.mozilla.org/en-US/docs/Web/API/MediaStreamTrack


DeleteVersions

Implement MutableTree#DeleteVersions. This method will be used in the revised pruning strategy in the Cosmos SDK, where pruned versions will be deleted in "batches" as opposed to on a block-by-block basis.
@erikgrinaker feel free to keep this PR open while you tackle removing pruning from the IAVL as I know you'll also further optimize this method. Or you can also merge this and address optimizations later.
/cc @zmanian

That was just the result of a go mod tidy :)


Add option to raise exception on request failure

currently, if the request to fawry server fails, it will return FawryResponse object with the error data. I think it would be a good idea to have the option to raise an exception if the request fails and add the error to the exception object

will close this and open again if it is specifically requested


Loss NAN

When I train the border det in floder borderdet.dcnv2.x101.32x8d.fpn.coco.800size.2x. The loss become NAN after 100 660 iteration. How to fix it ?

This problem is fixed by launching another training process.


Adding new previews directory with files to generate previews

Created the "previews" subdirectory of "uranus_occs_earthbased"; it contains two Python scripts to generate the previews for the uranus_occs_earthbased bundleset as well as the font file used for the custom fonts in these previews.
Could not push directly to main due to the error "(protected branch hook declined)".

Is there a need for the extra previews directory? It seems like the code should just be in previews/uranus/whatever.py.

@mace-space created the directory rms-data-projects/previews/uranus_occs_earthbased/diagrams for her code and my new directory (rms-data-projects/previews/uranus_occs_earthbased/previews) is meant to mirror that structure. The repeat of the name preview is just because "preview" can be meant in the general sense (both diagrams and line plots) or in the specific sense (line plots only).

Got it. I had forgotten Mia's code was already in the same directory.

Mightn't it make more sense to have rms-data-projects/diagrams/uranus_occs_earthbased and rms-data-projects/previews/uranus_occs_earthbased?  It looks like that might be what we're set up to do in the pds4-holdings directory on production.

https://pds-rings.seti.org/pds4/diagrams/uranus_occs_earthbased

The layout of the source code does not need to match the layout of the bundle. Currently all programs that create either previews or diagrams for any volumes/bundles are under rms-data-projects/previews. For example rms-data-projects/previews/COCIRS contains scripts to produce both the previews and diagrams for those volumes. We could decide to rearrange all of the sources, but that doesn't seem necessary at the moment and could break existing code.

Okay


add combine method to middleware

I find it useful like the reducer combine method.

LGTM, thanks!

No, is I that I have to tanks you. 🍺


[Fix]: Responsive styles

Fix navbar, footer and button / form styles
This PR adds responsive layout styles for navbar (with search component in mobile menu), footer and the signup button in docs.
Tasks

[x] fix navbar layout on mobile

[x] add the search component in nav menu

[x] fix footer layout overflow on mobile
[x] fix button style which was large on mobile
[x] update tailwind package to support new feature eg. <div className="[&>*]:bg-white" />

Screenshots

Navbar (mobile)

Navbar (tablet)

Footer (mobile)

Button (in docs)

@olayway changes have been updated

@olayway changes have been updated


Fix title in React component example in readme

The <title> tag is generated by react-helmet. The example would generate a title tag within a title tag.

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

Just realized there is another PR for the exact same thing. Sorry about that!


Webapp for PyPortfolioOpt

I have created a webapp with a demo account and several features from PyPortfolioOpt. The repo is https://github.com/Originn/AdvancedPortOpt
Feel free to test it:)
Thanks again to Robert for allowing to share it.

It's great, but it should has no password for testing purposes.

@88d52bdba0366127fffca9dfa93895
You need to register to use it, its just a simple username and password, no email needed.

@BrandonKMLee I'll need to migrate the website from Heroku due to discontinuation of the free service, might as well streamlit it as well. I am not familiar with it, reading the docs currently. The issue currently that I'll need to find a free postgres hosting service and free redis service, or something similar to those. I'll have a look.

@Originn thanks for sharing! Small tip: you should include some screenshots in your Readme :)


Automatically insert a comma on Enter

In WebStorm, if you press Enter after curly brackets in JSON, a comma will automatically appear. It would be cool to implement this in VS Code
Example:

P.S.: Not sure if JSON is specific to this plugin, but either way it would be useful

For now, I personally use shift+enter keybind, which powered by Commands and Comma Enter extensions as its useful for absolutely all languages (e.g. objects in TypeScript). In JSON we can easily implement the same behavior as in WS by overriding Enter and Ctrl+Enter.

P.S.: Not sure if JSON is specific to this plugin

This extension was originally created to fix errors (diagnostics) in JSON files, but I'm going to add a lot of commands / on-type JSON-specific fixes (such as autoEscapeQuoteJson) so I think its perfect place for this request.

Not a bad solution, but there are problems. Comma Enter generally works everywhere, but in WebStorm the comma only works if the character next to ' or " or } or ]

In VS Code I now have this:

I just tested this on a release extension and it doesn't work for me on two devices. Everything is ok in development mode
@zardoy does this feature work for you?

Yes,

The problem is obvious: it just doesn't work with CRLF, however tests a bit harder to fix than I expected, will do within a few hours.

The problem is obvious: it just doesn't work with CRLF
Oh, okay, I'll fix it after work.

Oh, okay, I'll fix it after work.

Sorry, didn't see that, the simple was extremely simple, it was initially my failure as I didn't setup tests correctly (file in integration tests was always using LF, dispite of files.eol is set to auto)


Integrating qfit_residue into qfit_protein

Integrate specifying residues in qfit_protein so its only run on specific residues, similar to what qfit_residue currently does. This would eliminate the qfit_residue script altogether.

Completed.


[DR] Canonical-PB Memory Allocation Fix and Hybrid Basis and Tensor Basis Extensions

This DR is to fix an issue with the canonical bracket species taking up hybrid basis amounts of memory while only updating the associated serendipity basis of fixed poly order. Ideally, I intend this DR to bring the canonical-pb species inline with the other Vlasov species by:

Restoring the Hybrid Basis to the Canonical-PB species.
Add in tensor basis for the canonical-pb species.

The work for this has been branched from main, and is in the branch:can_pb_kernel_sel.
Note: Due to the extremely large kernel size, Serendipity 3x3v p1 is unavailable for can-pb species. For 3x3v, only p1 tensor will be supported.

I am not completely sure what this DR is supposed to be for. Is it so we have p=1 tensor basis in 3x3v? I agree we should not support hybrid basis in 3x3v. It would also be good to consider adding some logic for ignorable coordinates so we can do, for example, 2x3v and 1x3v sims.

It is so we have the p=1 tensor basis in 3x3v. For debugging and completeness the tensor will also be available in 1x1v, 2x2v p1/p2. This way, the canonical-pb updater will take the same convention as elsewhere for serendipity meaning hybrid basis and tensor meaning tensor basis. I've also considered the ignorable coordinates. I planned to do that in a second DR/PR following up on this one that extends these changes to include 1x3v, 2x3v.

Update:
This work has been completed and gives correct results and maintains energy conservation for 1x1v p1/p2 ser and p1 tensor, 2x2v p2 ser and p1 tensor, and 3x3v p1 tensor. 2x2v p1 ser (hybrid) has an issue that it does not conserve total number of particles or energy. I've gone through the project on nodes ctest and extended the tests to include 4x_p1/p2, 1x1v_hybrid, 2x2v_hybrid. While the results seems correct from the ctest, the error seems to indicate the hamiltonian is not being correctly projected into the continuous subspace.