Waypoint: Trigger Click Events With jQuery -- fuzzy wording

I noticed some of the wording for this waypoint is a bit confusing.
The Issue lies with the following paragraph:
Let's make our "Get Message" button change the text of the element with the class message.
The request is to modify a button with the id "getMessage" as opposed to a class "message". The wording might also cause somebody to think there is some css involved with the code (i.e. they are expected to change the button text with a css class). There are also elements in the code with the class "message" in it.

@StevePaytosh thanks for posting this issue!
Bind the click event to the button with the ID of getMessage and make our "Get Message" button (which is the button with the ID of getMessage) change the text of the element with the class message.
Which means clicking the button with ID getMessage should change the text of the div with class message.
Closing this issue. Feel free to reopen if you still experience problems, thanks.

Please review the Guidelines for Contributing, thank you!.


Cannot find module '@azure/msal-angular' or its corresponding type declarations. Angular 16

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
3.0.0
Wrapper Library
MSAL Angular (@azure/msal-angular)
Wrapper Library Version
3.0.0
Public or Confidential Client?
Public
Description
Cannot find module '@azure/msal-angular' or its corresponding type declarations.
The module '@azure/msal-angular' or its corresponding type declarations could not be located. Despite having installed<EMAIL_ADDRESS>I'm encountering difficulties in utilizing the library. Refer to the image below for more context. It's important to note that I'm utilizing the library with Angular 16, having previously used version 3.0.0-beta.1, which functioned as anticipated.

Error Message
No response
Msal Logs
No response
MSAL Configuration
import {
  MsalBroadcastService,
  MsalGuard,
  MsalInterceptor,
  MsalModule,
  MsalRedirectComponent,
  MsalService,
  MSAL_GUARD_CONFIG,
  MSAL_INSTANCE,
  MSAL_INTERCEPTOR_CONFIG
} from '@azure/msal-angular';

Relevant Code Snippets
import { MsalGuard } from '@azure/msal-angular';

Reproduction Steps

download any angular 16 project
configured whatever is required for msal
then run npm install<EMAIL_ADDRESS><EMAIL_ADDRESS>npm start

Expected Behavior
Application should run.. but receiving below error

Identity Provider
Azure AD / MSA
Browsers Affected (Select all that apply)
Chrome, Internet Explorer
Regression
<EMAIL_ADDRESS><EMAIL_ADDRESS>Source
External (Customer)

content of node_modules/@azure/msal-angular seems to have just src and no dist || other build results

we are having the same issue.check here #6310

Please link this bug with #6310, so that i can follow the same whatever the status change would notified me.

@theviveik1 As others have stated above, there has been an error in our release process for msal-angular, and we are currently working on a fix. Please follow #6310, where the issue was first raised, and where updates will be posted.
Closing as duplicate of #6310.


Changing display format to a more compact form.

Hi,
I am new to Rust, and open source contribution, so please bear with me..
When debugging arrays of complex numbers i find the formatting to be unnecessarily long, making it hard to see what is going on in the data. See the example below.
I therefore propose to change the debug and display formatting to a+bi, e.g. 1+2i.
Note that in the example below, Github doesn't wrap the line.
stderr:
thread 'main' panicked at 'assertion failed: `(left == right)`
  left: `[Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }, Complex { re: 1, im: 0 }]`,
 right: `[Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }, Complex { re: 0, im: 0 }]`', src/lib.rs:13:1
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Erik.

Display should already have that compact form, but there's a reasonable case to be made for Debug as well. We did make that change for BigInt and BigUint in rust-num/num-bigint#195, but it's somewhat debatable whether that should be considered a breaking change. In that case, I was getting ready for a semver bump anyway.

Yes, I see now.
I appreciate the feedback!
Erik.

Has there been any progress on this topic?
When testing/debugging complex matrix functions, assert_eq statements print complex numbers in their Debug representation. This causes headaches due to the unnecessarily verbose output of matrices. I am happy to put up a PR, if you would like.


HTML and minor bugfixes

Fixes:

excessive spaces
errant tags
escaping
browse context

Pull Request Test Coverage Report for Build 336

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.9%) to 83.86%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

browse/controllers/abs_page/init.py
1
2
50.0%

Files with Coverage Reduction
New Missed Lines
%

browse/controllers/abs_page/init.py
1
81.29%

Totals

Change from base Build 332:
1.9%

Covered Lines:
3024

Relevant Lines:
3606

💛  - Coveralls


Subscriptions success page design and copy

We should have a nice page to redirect the user afer they successfully subscribed to a tier.
A few comments:

We have access to the backer name, email and the subscription tier they subscribed to
If they are new user to Polar, an account is automatically created for them and they can now login using a magic link. We have a similar upsell for pledges:

Authenticated user should not see this upsell. What should we show, do for them?

Good idea - I can take a stab at that!


Location of the Background Task Inspector

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-compose-verify-background-work?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-7-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-verify-background-work#3
Specify the language of the codelab if it is not English:
In which task and step of the codelab can this issue be found?
Step 4 - Tag and update the UI based on Work status
Describe the problem
The location of the Background Task Inspector is not mentioned when first introduced. (View > Tool Windows > App Inspection, then "Background Task Inspector" tab)
Alternatively, a tip to check "Debug WorkManager with Background Task Inspector" or move  "Debug WorkManager with Background Task Inspector" before Step 4 would resolve the issue.

ddd


fixed an issue where trafaret>=2.0.0 used.

fixes: https://github.com/mariocesar/pytest-requests/issues/1

@mariocesar fyi

Thank you!


Doesn't work with amd

Got the following error when trying to install via phpstorm terminal
gh-foreach unsupported for windows-amd64. Open an issue: `gh issue create -R mtoohey31/gh-foreach -t'Support windows-amd64'`

I'm trying to run...
git rm -r *.nrd
git add *.ber
git commit -m "changing extension"
git push

That's strange, because the latest release includes a build for windows-amd64. Do you get this error when running gh extension install mtoohey31/gh-foreach or does it show up at another point?

I got it in git-bash also.  I also tried installed the .exe and that didn't change anything

Hmm ok, I'll have to do some testing with Windows, I'm not at my computer right now though. If you have a second, could you check if the same thing happens with the same commands in powershell instead of git-bash?

powershell was the first place.  Thats what phpstorm uses.

Ok, I'm not able to reproduce this under windows on my system, which is also amd64. I do have one idea though, is your githib cli up to date? I believe support for precompiled extensions such as this one is a relatively recent feature.

I have
 C:\windows\system32>git --version
git version 2.34.0.windows.1

I JUST updated to 2.34.1
Can't be wsl2 or anything like that can it?

Can you post the output of gh --version?

Does the same issue occur when you update to the latest version?

yep

The extension doesn't seem to be installed now, can you run:
gh extension remove foreach # it's ok if this command fails
gh extension install mtoohey31/gh-foreach

ok, now it shows up.. tyvm..
Microsoft Windows [Version 10.0.19043.1415]
(c) Microsoft Corporation. All rights reserved.

C:\windows\system32>gh extension remove foreach
no extension found: "C:\\Users\\troy\\AppData\\Local\\GitHub CLI\\extensions\\gh-foreach"

C:\windows\system32>gh extension install mtoohey31/gh-foreach
✓ Installed extension mtoohey31/gh-foreach

C:\windows\system32>gh foreach
execute commands across multiple github repositories

Usage:
  gh-foreach [command]

Available Commands:
  clean       remove cache
  completion  generate the autocompletion script for the specified shell
  help        Help about any command
  run         execute a command

Flags:
  -h, --help   help for gh-foreach

Use "gh-foreach [command] --help" for more information about a command.

C:\windows\system32>

Np, glad I could help, I'll add a note about the minimum required GitHub cCLIversion to the README.


Publish new release to PyPI

Could we get a new release published with #38 included?

Hey @tomv564, we're trying to add support for MyPy in Spyder through the python-language-server and we're running into troubles because the current release imports the future package in Python 3.
I noticed that a fix was already submitted and merged in March (PR #38), but there hasn't been a release yet. So please let us know if this package is unmaintained, to decide if we need move it to the  python-language-server (preserving its history of commits, of course) and maintain it there.

@tomv564, it seems you've been a bit off the grid during the last months on Github. So i hope you don't mind we take over maintenance of your work.
If we don't hear from you in the next hours, we'll move this plugin to the python-language-server and won't load it there if it's found by setuptools. Thanks a lot for your work on this!

Any news on an update or fork?

No, there aren't. We're trying to fix some maintenance issues with the PyLS before proceeding with this.


Whitelist sparkol.com since there is some unknown compression error

Awesome bot seems to have some issues with one link by sparkol.com to go through the checks. This patch simplifies this and whitelists this particular domain instead.

Oh, it's --white-list. Can you amend?

Ah, yes. There was a typo --white-list is the correct one. Thank you :)


Jupyter notebook playbook for lab seeding

Build a Jupyter notebook to document and automate the steps involved in creating (seeding) a new lab.

Jupyter notebook available now.


Update pylance shipping version to take latest fixes

we r just about to release again in a few minutes

3 approvals, my lowest rating ever


Set the ID of a PC's worn armor as a roll option

I need it for a thing :)

Wouldn't the same be useful for the equipped shield?
On Tue, Oct 25, 2022, 13:32 Shark that walks like a man <
@.***> wrote:

I need it for a thing :)
You can view, comment on, or merge this pull request online at:
https://github.com/foundryvtt/pf2e/pull/4553
Commit Summary

c13cb46
https://github.com/foundryvtt/pf2e/pull/4553/commits/c13cb46bc04bb4ee6fe85c62f52e0d325505cdb2
Set the ID of a PC's worn armor as a roll option

File Changes
(1 file https://github.com/foundryvtt/pf2e/pull/4553/files)

M src/module/item/armor/document.ts
https://github.com/foundryvtt/pf2e/pull/4553/files#diff-022a1ec48c769d32c325190dd9bf0adf3c275a531a6a51f186241adeb5096f0a
(2)

Patch Links:

https://github.com/foundryvtt/pf2e/pull/4553.patch
https://github.com/foundryvtt/pf2e/pull/4553.diff

—
Reply to this email directly, view it on GitHub
https://github.com/foundryvtt/pf2e/pull/4553, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA7FAZGFU2637CU5DXNFULLWE6ZM5ANCNFSM6AAAAAARN2PJ7Y
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I can't think of any uses off the top of my head, but maybe.


Don't compile test target when you package

Is your feature request related to a problem? Please describe.
I'd like to be able to use Scala 3 and produce a native image via Scala Native. While my code can do that, the testing framework I'm using isn't published for Scala Native. When trying to publish my image it won't publish because of a test dependency.
Describe the solution you'd like
I'm not sure this behavior is expected, and I'm not 100% sure if this even needs to be, but I think it'd be nice to not have the package command rely on the test compilation. This would unlock more code bases to be able to produce a native image for example like I'm trying to.
Describe alternatives you've considered
Move my code into different directories and try to only point the package command towards them.

That looks like an oversight after some recent refactoring, we should fix that.


The download version is 1.0,but the cocaPods is 0.7

“ buttonsLayout” is not support in 0.7

@vikmeup any update on it?


Don't sanitize payload for storage

Reverts the changes of commit 520e19ab44065622a7a47c22d60ccd70de0e68bc. Setting changes to default values are no longer removed from the Firebase payload, avoiding the bug described in #1183. Passes E2E tests but I don't have a means to test Firebase network requests locally - @mfix22 can you share a deployment link, or test yourself if that's easier.
Closes #1183, although another change is required on backend to avoid storing default values.

@adghayes I moved the sanitization to the backend, which uses admin.firestore.FieldValue.delete() to remove all fields that are the defaults
If you want to test it out, here is the staging deployment: https://carbon-mfix22-carbon-app.vercel.app

I can't seem to save a snippet with this deployment, but I confirmed that the outbound POST requests now contain fields that need to be reset to defaults. It's a significant but I think fine amount of additional data over the wire, and it should fix our problem.

I can't seem to save a snippet with this deployment

Ahh yes, there is a bug in my deployment. Will fix that before merging here.

Link above should be working now 🤞

thanks @mfix22. with the deployment fixed, I was able to confirm that the bug described in #1183 is fixed by this patch. As long as you're OK with the larger payloads (which I think are fine), this is looking good


Feature Request: control Relay2 not only by the price but by current/expected PV Yield

Is it possible to modify switch for Relay (2 preferably) to close and open not only based on negative price or below average price, but according to current energy balance?
If there is enough excess energy for heatpump or other usage.
Maybe in config node setup how much energy is needed for devices, controled by relay.
If there is enough excess for past 10 minutes for example and there is expected excess predicted, close relay. If excess drops and stays dropped for like 10 or less minutes and no better prediction, open relay.
I have never worked in NodeRED - started with dynamicESS last week. But i have tried to do it simply for my heatpump by using counter. (counter decrease is faster then increase on purpose). But it can be done better - maybe batch of last 10 minutes data + PV Yield estimation for example?
Adding my example which is active on my site - working with Buderus (IVT) heatpump. Consumption set to 3100 watts (hard number in one function) - relay 2 is connected to External input on heatpump. External input is configured as PV system on heatpump. When relay2 is closed, water comfort level is increased, temperature is also increased by 2 degrees for the next working point in heatpump queue.

relay2.zip

One of the pitfalls is that this will work against the algorithms predictions and decisions. Which the existing example (switching on when the price is low) also does. At the moment we are not actively putting more effort into that.
As you are just starting with Node-RED, have you seen the examples on https://github.com/victronenergy/node-red-contrib-victron/wiki/Example-Flows? They might give you some ideas on how to do this.

One of the pitfalls is that this will work against the algorithms predictions and decisions. Which the existing example (switching on when the price is low) also does. At the moment we are not actively putting more effort into that.
As you are just starting with Node-RED, have you seen the examples on https://github.com/victronenergy/node-red-contrib-victron/wiki/Example-Flows? They might give you some ideas on how to do this.

@dirkjanfaber Ok thanks. I just thought that for static installations it would make sense. Thats why I have attached my simple working example to your flow. just used output from negative price node to add additional nodes.
I am aware it can go against prediction and decision, but as I have mentioned in other comment I think you are doing recalculation every 10 or 15 minutes. I am not sure if recalculation is strictly only for future or if it takes past values to project for target values like SOC in the end of day/price period. I have though that in case of past values, consumption is known, expected and real PV Yield also and probably target endpoint for the day (or for the period of published dynamic prices). So based on configured values for node and putting real PV Y instead of predicted should let recalculation do it dynamically.
For the consumption change it is just normal operation even without relay(wash machine, laundry machine etc.) When relay closes, it will be some time consumption will be higher, so in that case algo can take it as new expected consumption level for defined timeframe (!5 minutes). Then recalculation starts again.
I have checked examples, but I was strugling with specifics: creating custom properties ie combine 2 or more messages into one, having time values back as array and remove oldest value each tick and recalculate average from rest + new one, etc.


Playername and UUID problem

Hello.
I test the plugin with MMOProfiles Proxy mode. In this plugin every profile will have different uuid and uuid are really change after the profile selected.
Problem with ServerVariables is when adding svar to player targeting wrong uuid. This is not a bug of this plugin I'm not asking to fix.
I'm only asking small feature; use uuid instead of playername with everywhere in the plugin. We can use MMOProfiles %mmoprofiles_current_profile_uuid% instead of player name to fix for us.
Thank you.

It seems you have multiple files of ServerVariables for the same user, that must not happen or you will experience this bug.

It seems you have multiple files of ServerVariables for the same user, that must not happen or you will experience this bug.

When using MMOProfiles this will going to happen all the time because every profile of user is a different uuid. Player select a class when join the lobby server and teleport to play server after select a profile. When join a play server player will have different uuid and every profile has is different uuid.

Can you support this please ?

That is a serious incompatibility issue of MMOProfiles. All plugins work by a player name with the corresponding uuid. The uuid is unique, so it's not possible to have multiple uuids for the same user.


Added chores endpoint to add new chore to database (POST /api/Chores/Flat)

Pull Request Information
Closes #195
What is the PR Type: Feature

Describe the added/fixed behaviour:
Added an endpoint (POST /api/Chores/Flat) to add a new chore to the database per the API description in the wiki..

Checklist

[x] Pull request has suitable title and labels, adequately described and associated with a github issue
[x] Branch name follows correct convention (e.g. feature/#[id]-some-feature)
[x] Branch builds and executes with no errors
[x] Test cases has been generated for the feature and test cases pass
[ ] Updated relevant wiki/documentation

Additional Information

Can't get the CI to run, despite that it builds locally. Does not provide any error messages when it fails.

Looks good, while it is functionally correct there is something important to consider here. Ryan and I were having a discussion earlier as to how we would link the assignee to the chore.
On the frontend the user shouldn't have to decide the assignee based on IDs. They should be provided a list of flatmates to make their decision. Or we could take the easy route and just have them type out the asignees name. Either is fine, but we need to make sure there is the functionality from the flats side of things to return all flatmates to the frontend.
Perhaps we should have a meeting to discuss this.

What if we get a list of users using /api/Flat/getMembers then display that to the user on the frontend, and send back the selected ID


Add hunllef-helper v1.0.0

Adds the Hunllef Helper plugin. The plugin is similar in use to the gauntlet counter. It assists the user by informing them about the Hunllefs attacks. In order to comply with the Jagex third party plugin rules, the plugin does not use any game state information. The only interaction it has with the game is to automatically hide the plugin panel when a player is not in The Gauntlet area. This behavior equals the already existing zulrah-helper.

It looks like you need to clear the scheduled executor on plugin shutdown. It's declared here, and later defined and started here, but is not explicitly stopped on plugin shutdown.

@Nightfirecat
Nice catch, I missed that one. I have applied the change.


Is it possible modify query in initialize on event 'fetching'?

if I want to add some extra column e.g.:
 initialize: function() {
    self = this
     this.on('fetching', function(model, attrs, options) {
      self.query(function(qb) {
        return qb.leftJoin(...).groupBy(...).count(...);
      });
    });
  }

this does not work, how can I fix it? Thanks

The fetching event provides a options.query, which is equivalent to the qb object given to the Model.query callback.
Correct code:
  initialize: function () {
    this.on('fetching', function (model, attrs, options) {
    options.query
      .leftJoin(/*...*/)
      .groupBy(/*...*/)
      .count(/*...*/);
  }

thanks it works,
it is sad, that no mention about it in doc :(

You could always make a PR against the docs.


Network changes do not take effect without manual reboot

Changes to the network are not reflected immediately, but need a reboot.
Reason might be a wrong command in network.cgi, trying to call "/etc/init/network" - which does not exist. But my tcl/tk is not good enough to actually understand if cp_network.cgi calls network.cgi at all or if it is a leftover from old times.
Steps to reproduce the behavior

Change something in the network settings
Save
Nothing has actually changed in the active network settings.

Expected behavior
Changes to the network settings should be reflected immediately through restarting the network (e.g. /etc/init.d/S40network restart).
The best solution would actually be to test out the new settings and revert if simple checks (ping gateway, etc.pp) fail.
Elsewise it should at least give a popup saying something like "Reboot needed for the changes to take effect. Reboot now?"
Screenshots
Not needed
System information:

RaspberryMatic Version: <IP_ADDRESS>01226
Used Hardware: RP3
Used HomeMatic RF-Module: HB-RF-USB

Additional context
Needs to be fixed upstream imho.

Where in the official eQ3 CCU documentation is it stated that network configuration changes should not require any reboot? I think it is perfectly normal and common sense that upon changing the network settings a reboot is required.

Where in the official eQ3 CCU documentation is it stated that network configuration changes should not require any reboot? I think it is perfectly normal and common sense that upon changing the network settings a reboot is required.

I never said it's in the documentation. Actually it would make me really wonder if bugs oder enhancement requests would already be in the documentation...
And honestly, I think most people would expect it to work that way, no matter what the documentation says. In the end it's common sense. If I change something, it is changed. If for some reason the change cannot be activated online, a good software at least tells me about that fact and asks me if I want to reboot.
Following your argumentation - why does the RM ask me for a reboot after changing the certificate?

I never said it's in the documentation. Actually it would make me really wonder if bugs oder enhancement requests would already be in the documentation...
And honestly, I think most people would expect it to work that way, no matter what the documentation says. In the end it's common sense. If I change something, it is changed. If for some reason the change cannot be activated online, a good software at least tells me about that fact and asks me if I want to reboot.
Following your argumentation - why does the RM ask me for a reboot after changing the certificate?

Simply, because the CCU3 is doing this and we are still using a good bunch of upstream code borrowed from eQ3. But if you feel that something should be changed in this area, feel free to submit a pull request with the necessary changes.

Simply, because the CCU3 is doing this and we are still using a good bunch of upstream code borrowed from eQ3. But if you feel that something should be changed in this area, feel free to submit a pull request with the necessary changes.

I know, but if I remember correctly it was you who said: if you want something changed in OCCU, better post it here, because no one really cares about issues at OCCU. ;)
And as I already said, I do not really know tcl/tk good. So a pull request would rather be of questionable quality. I could give it a try though - but then I would have to do the pull req against OCCU, or not?

I know, but if I remember correctly it was you who said: if you want something changed in OCCU, better post it here, because no one really cares about issues at OCCU. ;)
And as I already said, I do not really know tcl/tk good. So a pull request would rather be of questionable quality. I could give it a try though - but then I would have to do the pull req against OCCU, or not?


[Service Bus] Exception Handling review

This PR is mainly for https://github.com/Azure/azure-sdk-for-python/issues/10647
Changes made to exception handling

Removed exception class InvalidHanlderState.
Added ServiceBusAuthenticationError. Now there are both ServiceBusAuthenticationError and ServiceBusAuthorizationError for two scenarios.
There is no way to detect whether an entity exists or not in the uamqp library. So ServiceBusConnectionError will be raised.
Added exception class MessageError for trying to send a problematic Message. It can be mapped to some of uamqp's exception class.
errors.MessageAccepted,
errors.MessageAlreadySettled,
errors.MessageModified,
errors.MessageRejected,
errors.MessageReleased,
errors.MessageContentTooLarge,
send() raises MessageSendFailed when the underlying uamqp.MessageException is caught. Otherwise it doesn't raise a SendFailedException. So send() and receive() have same exceptions for other underlying uamqp exceptions.
Added type validation to send()

Deferred. Not in this PR

Shared connection exception handling. When multiple receivers/senders share a connection, which is disconnected, the receivers/senders shouldn't try to reconnect separately. Rebuild a single connection is good enough. We can implement this together with implementing shared connection.

Need more discussions during my review. Not in this PR

AutoLockRenewFailed and AutoLockRenewTimeout might be removed. A separate discusion.
SettleMessageFailed wraps all exceptions. This might be able to breakdown to more specific exceptions like send failure / receive failure.

Other changes
_is_live -> _check_live
_can_run -> _check_session

/azp run python - servicebus - tests

/azp run python - servicebus - tests

aha, don't forget about updating the changelog 😃


MWPW-137282 - Action item block layout options

The spacing and width of action items nested within the scroller and section grid are not currently correct for the action-icons 3-5 up design examples. This is due to the way the grid template items are styled in both places. Adding some options to make the action icons design possible.

Added .center style on section grid in desktop that will justify grid content center
Added default width on the action item to match designs and provide a function that applies item width conditional logic
Added center content style in action scroller for 5 or less items (also considers the established action item width logic)
Added Action scroller hide mask attribute to handle edge case where mask was obscuring content when justified center
Removed an unused style in section-metadata.css (center) and also removed unneeded grid style overrides in icon-block.css

Resolves: MWPW-137282
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/sarchibeque/justify-action-item?martech=off
After: https://sarchibeque-mwpw-137282-center-actions--milo--adobecom.hlx.page/drafts/sarchibeque/justify-action-item?martech=off

@ryanmparrish here are my other testing files in case they are helpful, thanks for reviewing!
All action block variants:
https://sarchibeque-mwpw-137282-center-actions--milo--adobecom.hlx.page/drafts/sarchibeque/actions-support-slider
Icon blocks using the center variant in section:
https://sarchibeque-mwpw-137282-center-actions--milo--adobecom.hlx.page/drafts/sarchibeque/icons-centered-grid

Let's socialize this a bit more to GWP to see if everyone is ok with how this corrections affects existing content.

@Sartxi  - One thing that I did notice, the tablet view doesn't use justify-items:center.
I see in the task that it states 'desktop' so it might be fine, but do we need to check w/ consonant or Figma on those? Generally when a section.center is used its a broad stroke across all viewports, so it might be worth a quick check.

This is leading to some issues https://main--homepage--adobecom.hlx.page/uk/homepage/index-loggedout?milolibs=stage

yep, It definitely seems to be coming from this branch .
https://stage--homepage--adobecom.hlx.page/uk/homepage/index-loggedout?milolibs=sarchibeque-mwpw-137282-center-actions

@Sartxi you will need to re-raise this PR once the issue is fixed


python3Packages.python-language-server: disable when python>=3.9

Motivation for this change
The package is not supported on anything newer than 3.8 yet.
https://github.com/palantir/python-language-server/issues/896#issuecomment-752790868
https://github.com/NixOS/nixpkgs/pull/121370#issuecomment-830583762
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Except python39Packages.parso builds just fine.

PS: I'd be happy if all software written by Palantir was marked as broken.

Except python39Packages.parso builds just fine.

But python-language-server is, according to aforementioned issue, not compatible with parso 0.8.

python38Packages.python-language-server builds and tests just fine
python39Packages.python-language-server runs into test issues, as mentioned in the op

To be blunt: Fuck Palantir.

Probably want to avoid dialogue like this in public, we now have a github reference to one of their tickets with this PR and all of it's contents :)

I don't see a problem with voicing a political opinion here, especially since we did not let that get in the way of technical decisions.

After trying pyright today, I am going to step back from python-language-server... https://github.com/NixOS/nixpkgs/pull/129168
Not sure if we should remove this package or not.


Get-MgPlaceAsRoom don't retrieve resources of type "Workspace"

When i try to use the cmdlet Get-MgPlaceAsRoom, are retrieved only the resources of type "room".
But i have in the Exchange both resources of type "room" and "workspace".
On the other hand, when i use the cmdlet Get-MGUserMailboxSetting, both the resources of type "room" and "workspace" have the property UserPurpose set to "room".
Could it be an API improvement?
https://learn.microsoft.com/en-us/graph/api/place-list?view=graph-rest-1.0&tabs=powershell

Thanks for following up on this.
Workspace derived type is missing in the service metadata for place entity in beta.
As a workaround, you can use Invoke-MgGraphRequest -Uri "https://graph.microsoft.com/beta/places/microsoft.graph.workspace" to retrieve the resources. See API reference doc for more details - https://learn.microsoft.com/en-us/graph/api/place-list?view=graph-rest-beta&tabs=http#example-2-list-all-the-workspaces-defined-in-the-tenant.
I've transferred the issue to the metadata repo for visibility.
<EntitySet Name="places" EntityType="microsoft.graph.place">
  <Annotation Term="Org.OData.Validation.V1.DerivedTypeConstraint" xmlns:edm="http://docs.oasis-open.org/odata/ns/edm">
    <Collection>
      <String>microsoft.graph.room</String>
      <String>microsoft.graph.roomList</String>
      <String>microsoft.graph.workspace</String> <!-- This is currently missing in beta -->
    </Collection>
  </Annotation>
</EntitySet>


Enhancement - Draw a line over a pdf to delimitate a column

Hi!
1st of all, thank you for Tabula, a software that saves my life everyday.
One question :
Could it be useful/possible to implement a function that would allow us to draw a vertical line on every page, on top of the selected area, in order to delimitate columns on documents where columns are not well marked.
It would help Tabula to structure the document.
Thanks for your help
Hervé (from Paris France)

Hi Hervé,
Glad to hear Tabula is useful to you! We've thought about implementing a feature for Tabula like you describe. We've never found time to do it, though tabula-extractor, Tabula's command-line counterpart, supports this feature. We'll keep in mind your feedback when choosing what new features to add.


Computer configuration

Suggested new feature or improvement
希望安装环境的时候可以选择安装的位置
Scenario
比如在安装之前设置好安装路径,或者在设置里面直接设置好
Additional details
No response

Bing translate: Computer configuration
If you want to install the environment, you can choose the location where you want to install it
For example, set the installation path before installation, or set it directly in the settings


Is this incompatible with SE/A3?

I got it installed on windows 10/anaconda 3.7.5.  Just trying simple commands like axi goto 30 30 send the arm carrening to its full limts out until the motors reached their limit and started skipping.  Also noticed that the pen up/down range is fairly limited.
Wondering if there is some kind of setup or configuration that needs done?  I'm using a new SE/A3 that has worked well with the default EMS-software.

Oh dear, I realized that the parameters it takes are inches....   No wonder it shot off the edge.


Document APRILTAG_SRC environment variable

It's due to #2. We need to make it explicit on README.

Close it due to #2.


Can not build on Windows with "sm-raw-window-handle" feature

cargo build --no-default-features --features "sm-raw-window-handle"
results in:
error[E0432]: unresolved import `raw_window_handle::RawWindowHandle::Xlib`
   --> surfman\src\platform\windows\wgl\connection.rs:139:13
    |
139 |         use raw_window_handle::RawWindowHandle::Xlib;
    |             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ no `Xlib` in `RawWindowHandle`

error[E0560]: struct `platform::windows::wgl::surface::NativeWidget` has no field named `window`
   --> surfman\src\platform\windows\wgl\connection.rs:143:17
    |
143 |                 window: handle.window,
    |                 ^^^^^^ `platform::windows::wgl::surface::NativeWidget` does not have this field
    |
    = note: available fields are: `window_handle`

That looks like a copy-paste error from ec40477110386e54192b9367aa32195944d0700e. It should be using the Windows variant from https://docs.rs/raw-window-handle/0.3.3/raw_window_handle/enum.RawWindowHandle.html instead.


RTL Support

Describe the bug
Notion should have support for RTL languages like Arabic where the text is Right to Left, instead of LTR.
To Reproduce
Node version: latest
Notion JS library version: latest
Steps to reproduce the behavior:
Write anything in Arabic
Expected behavior
Text should appear on the right of the screen.
Screenshots

Closing this since not related to API or SDK. I'd recommend looking at the Notion Help Page for assistance on the Notion product.


[BUG][haystack] Output messages is a string rather than a list of messages, causing pheonix to crash - this is using the quickstart

          Output messages is a string rather than a list of messages, causing pheonix to crash - this is using the quickstart

Originally posted by @mikeldking in https://github.com/Arize-ai/openinference/issues/524#issuecomment-2274279088

@21ShisodeParth marking as high - most people will hit this issue using haystack with chat completions


Signal handlers and method visibility in Kotlin

In Spine, different kinds of signal handlers should have different visibility levels. For example, event @Subscribe-rs should be package-private (which, among other reasons, eliminates the need to type an extra keyword in Java). However, package-private access level is inaccessible in Kotlin. This causes runtime warnings upon bounded context construction:
WARNING: SignatureMismatch{unmetCriterion=ACCESS_MODIFIER, message=The access modifier of `io.spine.protodata.ProtoSourceFileProjection.on$compiler(class io.spine.protodata.EnteredOneofGroup)` method is `public`. We recommend it to be `package-private`. Refer to the `Subscribe` annotation docs for details.}

It is, of course, possible to hack Kotlin into making the methods package-private, but this just doesn't seem to be a very user-friendly, way, not to mention the lack of real need in making those methods visible.
This is something to look into in the nearest future. My instinct is to allow making all handler methods private, so that nobody mistakenly calls them directly. On the other hand, typing "private" for each of them is a bit tedious and might impair readability.

We now also allow Kotlin internal visibility.


B2 Challenge

concordium-testnet-node-4f85be334186c64fe4ed6cfb5ec746b7ac1dba10e4cbced034a55c782c35ad76.log
https://dashboard.testnet.concordium.com/node/4c09a914cc5a1463
Baker Id = 435
Logs will Be submitted after 30 days

confirmed:B2

confirmed:B2


[Idea] HTML5 JavaScript Editor

Hey guys,
I'm thinking of creating a new plugin for JavaScript Bootstrap, the bootstrap-editor.js! I'm developing in a separate repository and then implement in future. (https://github.com/formaweb/bootstrap-editor).

Like it? Thanks.

+1 :D

I don't think this is something we'll make part of the main toolkit, but definitely something we can add to our wiki's list of addons and such. Let me know when that's ready for linking with a separate issue—good work!


Launchrctl support for actions and plugin commands aliases

Note that this PR also contains changes of https://github.com/launchrctl/launchr/pull/43/files for simplicity


cross-platform: buildable

Configured for cross-platform (at least Windows and Linux) successful build (and debugging).

Fixed by commit: 6022e14


Add option to set target of Export html:a button.

I need it to open new tab to create pdf of filtered rows.

Could you please fix travis? https://travis-ci.org/github/contributte/datagrid/jobs/725164052
thx

Travis fixed.

Thanks! :)

@hlavacz Uuh, one more thing - could you add that pls to docs? :) https://github.com/contributte/datagrid/tree/master/.docs

Sorry, I am not familier with github as I should be... I pushed commit of doc update on mine fork but I am afraid you can't see it after you aprooved this pull request right? So I need to create a new pull request to commit of doc file? I cant see any other option..

Yes, a new pull request is needed. If you won't be able to work it out, you can send it to me here and I will edit the master branch manually, np.

Edit manualy Export.md is probebly easest for both, Few words only.. :) Line 25
You can tell whether to use ajax or not (`->setAjax()`). Or a button title (`->setTitle('Title')`). Or target (`->setTarget('_blank')`).

Thanks 👍
https://github.com/contributte/datagrid/releases/tag/v6.2.28


Add 'getMatrix' method into 'Quaternion' class

Added this very useful method (http://irrlicht.sourceforge.net/docu/quaternion_8h_source.html#l00349)

Please note that this library uses the Hamiltonian convention as described here:
https://dev.px4.io/en/contribute/notation.html

@xahon
As @bresch correctly pointed out the library uses Hamiltonian convention and needs to be consistent.
Also you don't need to reimplement the quaternion to rotation matrix calculation but could rather use the existing method similar to this untested example:
/*
 * Useful Description like:
 * Build transformation matrix from quaternion and position coordinate vector to ...
 *
 * [ R(q)  center ]
 * [  0      1    ]
 *
 * Source (http://irrlicht.sourceforge.net/docu/quaternion_8h_source.html#l00349)
 */
SquareMatrix<Type, 4> transformationMatrix(const Vector3 center)
{
    const Quaternion &q = *this;
    SquareMatrix<Type, 4> T; // initialize result matrix with all elements zero

    // place the 3x3 rotation matrix built from q in the top left corner of T
    T.set<3, 3>(Dcmf(q), 0, 0);
    // place 3x1 vector center in the top right corner of T
    T.set<3, 1>(center, 3, 0);
    // place a 1 casted to the used type of the template in the lower right corner of T
    T(3, 3) = Type(1);

    return T;
}

Feel free to reopen if you follow up further on adding this functionality.


Get detailed news

I am wondering, should I get the whole article of the page?

It is difficult to identify this issue, but we do not have a license when taking the article, it is related to the legal issue.


[staging-next] firefox: take LLVM tools from llvmPackages.llvm

Since 37194a325de0 llvmPackages*.bintools is a bintools-wrapper. Thus it
only contains a wrapper for as and ld. This change makes sense, but
causes regressions like this one. Since we need the LLVM tools
specifically opting to use llvmPackages.llvm instead of the alternative
which would be using the passed through “raw” bintools derivation
llvmPackages.bintools.bintools.

Untested, but verified all paths in firefox-unwrapped.makeFlags are now correct.

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

Off the top of your head, do you understand some of the other llvm parts in firefox?
{
  # Force the use of lld and other llvm tools for LTO
  llvmPackages = llvmPackages0.override {
    bootBintoolsNoLibc = null;
    bootBintools = null;
  };
}

That snippet forces clangUseLLVM to use the llvm bintools instead of whatever stdenv.cc.bintools is (which usually would be GNU binutils). This ensures that ${buildStdenv.cc.bintools}/bin/ld is a linker that actually can deal with clang's LTO.
Note that instead of llvmPackages.llvm in my change I could've used llvmPackages.bintools-unwrapped, buildStdenv.cc.bintools.bintools or even buildStdenv.cc.libllvm.out.
Since we have a common llvmPackages for all of those it doesn't really matter, but it may be cleaner to take the bintools from buildStdenv actually. Might amend this later, gonna eat first!

For reference, this is how the bintools used for clangUseLLVM is chosen: If null is passed as bootBintools, then we use the LLVM bintools including lld as linker, otherwise whatever is passed in from the outside.
https://github.com/NixOS/nixpkgs/blob/acb7a605e21db44bc6e2fa7b11e1cdfbe082f4c0/pkgs/development/compilers/llvm/11/default.nix#L52-L59
The default is pkgs.bintools{,NoLibc} which I don't think will differ from stdenv.cc.bintools usually.
https://github.com/NixOS/nixpkgs/blob/acb7a605e21db44bc6e2fa7b11e1cdfbe082f4c0/pkgs/top-level/all-packages.nix#L12738-L12767
The intention behind these changes is to make useLLVM more flexible: You can now either have useLLVM with lld like it used to be or use other linkers (cctools, bfd, …).

Since we are using the correct binutils actually, we should probably just set AR=llvm-ar etc. since they are correctly added to PATH…
Would make such a change on staging though since I'm not sure if there may be any sideeffects. This seems like the safer fix.

Okay using buildStdenv.cc.bintools.bintools now — seems nicer overall.

Not exactly relevant to this PR but for what it is worth, the buildStdenv abstraction exists because Firefox used to be built with GCC (and still can be if you disable LTO). Considering upstream only has CI testing on LLVM it would be best if we could move forward to removing that abstraction and just always have Firefox built with the LLVM stdenv. LTO or not.

Since we are using the correct binutils actually, we should probably just set AR=llvm-ar etc. since they are correctly added to PATH…

I am not so sure if the conditional makeFlags behind ltoSupport are still needed or if Mozilla has fixed their build system to resolve them. I have not tested it but you likely can just remove them entirely and Firefox should still build. Which I am in favor of to make the Nix expression more concise.

You bring up valid points, but I think this is the lowest-impact change we can make which has the least chance of breaking it in a different way which is what I am looking for in a staging-next PR. I'd say we do the other changes on master next and maybe backport them to 21.05.
Confirmed this fixes the build on x86_64-linux at least:
$ nix-store -r /nix/store/yy5r4b1xk5v6x4lq3gnvc5ahby5r1wva-firefox-88.0.1.drv
/nix/store/xcxhfncn3rbk2qj7jxcsg3cgz8pzgmbz-firefox-88.0.1

was about to merge :). Anyway, thanks!


[Bug] Cannot run "Train a PyTorch model on Fashion MNIST with CPUs on Kubernetes" doc

Search before asking

[X] I searched the issues and found no similar issues.

KubeRay Component
ray-operator
What happened + What you expected to happen
Tested on Linux x86-64 machine.
Following the steps in this doc https://docs.ray.io/en/latest/cluster/kubernetes/examples/mnist-training-example.html, when I execute kubectl logs -f rayjob-pytorch-mnist-nxmj2, the log of the pod shows
2024-08-23 02:54:24,095    INFO cli.py:39 -- Job submission server address: http://rayjob-pytorch-mnist-raycluster-qcgfz-head-svc.default.svc.cluster.local:8265
2024-08-23 02:54:24,405    SUCC cli.py:63 -- -------------------------------------------------------
2024-08-23 02:54:24,405    SUCC cli.py:64 -- Job 'rayjob-pytorch-mnist-d78q2' submitted successfully
2024-08-23 02:54:24,405    SUCC cli.py:65 -- -------------------------------------------------------
2024-08-23 02:54:24,405    INFO cli.py:289 -- Next steps
2024-08-23 02:54:24,405    INFO cli.py:290 -- Query the logs of the job:
2024-08-23 02:54:24,405    INFO cli.py:292 -- ray job logs rayjob-pytorch-mnist-d78q2
2024-08-23 02:54:24,405    INFO cli.py:294 -- Query the status of the job:
2024-08-23 02:54:24,405    INFO cli.py:296 -- ray job status rayjob-pytorch-mnist-d78q2
2024-08-23 02:54:24,405    INFO cli.py:298 -- Request the job to be stopped:
2024-08-23 02:54:24,405    INFO cli.py:300 -- ray job stop rayjob-pytorch-mnist-d78q2
2024-08-23 02:54:24,407    INFO cli.py:307 -- Tailing logs until the job exits (disable with --no-wait):
2024-08-23 02:54:24,248    INFO job_manager.py:531 -- Runtime env is setting up.
2024-08-23 02:56:14,938    INFO worker.py:1456 -- Using address <IP_ADDRESS>:6379 set in the environment variable RAY_ADDRESS
2024-08-23 02:56:14,938    INFO worker.py:1596 -- Connecting to existing Ray cluster at address: <IP_ADDRESS>:6379...
2024-08-23 02:56:14,941    INFO worker.py:1772 -- Connected to Ray cluster. View the dashboard at <IP_ADDRESS>:8265
2024-08-23 02:56:14,987    INFO tune.py:253 -- Initializing Ray automatically. For cluster usage or custom Ray initialization, call `ray.init(...)` before `<FrameworkTrainer>(...)`.

View detailed results here: /home/ray/ray_results/TorchTrainer_2024-08-23_02-56-14
To visualize your results with TensorBoard, run: `tensorboard --logdir /tmp/ray/session_2024-08-23_02-54-00_437051_1/artifacts/2024-08-23_02-56-14/TorchTrainer_2024-08-23_02-56-14/driver_artifacts`
(autoscaler +1s) Tip: use `ray status` to view detailed cluster status. To disable these messages, set RAY_SCHEDULER_EVENTS=0.
(autoscaler +1s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +36s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +1m11s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +1m46s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +2m21s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +2m56s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +3m31s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +4m6s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +4m41s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +5m17s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +5m52s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +6m27s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +7m2s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +7m37s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +8m12s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +8m47s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +9m22s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +9m57s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +10m32s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +11m7s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +11m42s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.
(autoscaler +12m18s) Error: No available node types can fulfill resource request {'CPU': 3.0}. Add suitable node types to this cluster to resolve this issue.

But I am sure that I have enough CPU. The following is the output of kubectl describe nodes
Name:               kind-control-plane
Roles:              control-plane
Labels:             beta.kubernetes.io/arch=amd64
                    beta.kubernetes.io/os=linux
                    kubernetes.io/arch=amd64
                    kubernetes.io/hostname=kind-control-plane
                    kubernetes.io/os=linux
                    node-role.kubernetes.io/control-plane=
Annotations:        kubeadm.alpha.kubernetes.io/cri-socket: unix:///run/containerd/containerd.sock
                    node.alpha.kubernetes.io/ttl: 0
                    volumes.kubernetes.io/controller-managed-attach-detach: true
CreationTimestamp:  Fri, 23 Aug 2024 17:51:27 +0800
Taints:             <none>
Unschedulable:      false
Lease:
  HolderIdentity:  kind-control-plane
  AcquireTime:     <unset>
  RenewTime:       Fri, 23 Aug 2024 18:13:57 +0800
Conditions:
  Type             Status  LastHeartbeatTime                 LastTransitionTime                Reason                       Message
  ----             ------  -----------------                 ------------------                ------                       -------
  MemoryPressure   False   Fri, 23 Aug 2024 18:09:21 +0800   Fri, 23 Aug 2024 17:51:25 +0800   KubeletHasSufficientMemory   kubelet has sufficient memory available
  DiskPressure     False   Fri, 23 Aug 2024 18:09:21 +0800   Fri, 23 Aug 2024 17:51:25 +0800   KubeletHasNoDiskPressure     kubelet has no disk pressure
  PIDPressure      False   Fri, 23 Aug 2024 18:09:21 +0800   Fri, 23 Aug 2024 17:51:25 +0800   KubeletHasSufficientPID      kubelet has sufficient PID available
  Ready            True    Fri, 23 Aug 2024 18:09:21 +0800   Fri, 23 Aug 2024 17:51:45 +0800   KubeletReady                 kubelet is posting ready status
Addresses:
  InternalIP:  <IP_ADDRESS>
  Hostname:    kind-control-plane
Capacity:
  cpu:                24
  ephemeral-storage:  976274Mi
  hugepages-1Gi:      0
  hugepages-2Mi:      0
  memory:             32597756Ki
  pods:               110
Allocatable:
  cpu:                24
  ephemeral-storage:  976274Mi
  hugepages-1Gi:      0
  hugepages-2Mi:      0
  memory:             32597756Ki
  pods:               110
System Info:
  Machine ID:                 daeb1519b4b140b58c523842b45a9149
  System UUID:                b5f85f95-abd1-41f4-9179-5713e3011492
  Boot ID:                    a8ebdb3c-f1fe-4c9e-97b4-4b97c07b729f
  Kernel Version:             6.8.0-41-generic
  OS Image:                   Ubuntu 22.04.1 LTS
  Operating System:           linux
  Architecture:               amd64
  Container Runtime Version:  containerd://1.6.12
  Kubelet Version:            v1.26.0
  Kube-Proxy Version:         v1.26.0
PodCIDR:                      <IP_ADDRESS>/24
PodCIDRs:                     <IP_ADDRESS>/24
ProviderID:                   kind://docker/kind/kind-control-plane
Non-terminated Pods:          (14 in total)
  Namespace                   Name                                                              CPU Requests  CPU Limits  Memory Requests  Memory Limits  Age
  ---------                   ----                                                              ------------  ----------  ---------------  -------------  ---
  default                     kuberay-operator-8c6bdd449-gsl6f                                  100m (0%)     100m (0%)   512Mi (1%)       512Mi (1%)     21m
  default                     rayjob-pytorch-mnist-qp6p9                                        500m (2%)     1 (4%)      200Mi (0%)       1Gi (3%)       19m
  default                     rayjob-pytorch-mnist-raycluster-qcgfz-head-xx757                  2 (8%)        2 (8%)      4Gi (12%)        4Gi (12%)      20m
  default                     rayjob-pytorch-mnist-raycluster-qcgfz-small-group-worker-562xz    2 (8%)        2 (8%)      4Gi (12%)        4Gi (12%)      20m
  default                     rayjob-pytorch-mnist-raycluster-qcgfz-small-group-worker-h9zt9    2 (8%)        2 (8%)      4Gi (12%)        4Gi (12%)      20m
  kube-system                 coredns-787d4945fb-4dgrs                                          100m (0%)     0 (0%)      70Mi (0%)        170Mi (0%)     22m
  kube-system                 coredns-787d4945fb-t9vmq                                          100m (0%)     0 (0%)      70Mi (0%)        170Mi (0%)     22m
  kube-system                 etcd-kind-control-plane                                           100m (0%)     0 (0%)      100Mi (0%)       0 (0%)         22m
  kube-system                 kindnet-bhp4w                                                     100m (0%)     100m (0%)   50Mi (0%)        50Mi (0%)      22m
  kube-system                 kube-apiserver-kind-control-plane                                 250m (1%)     0 (0%)      0 (0%)           0 (0%)         22m
  kube-system                 kube-controller-manager-kind-control-plane                        200m (0%)     0 (0%)      0 (0%)           0 (0%)         22m
  kube-system                 kube-proxy-gcmmb                                                  0 (0%)        0 (0%)      0 (0%)           0 (0%)         22m
  kube-system                 kube-scheduler-kind-control-plane                                 100m (0%)     0 (0%)      0 (0%)           0 (0%)         22m
  local-path-storage          local-path-provisioner-c8855d4bb-xl2rs                            0 (0%)        0 (0%)      0 (0%)           0 (0%)         22m
Allocated resources:
  (Total limits may be over 100 percent, i.e., overcommitted.)
  Resource           Requests       Limits
  --------           --------       ------
  cpu                7550m (31%)    7200m (30%)
  memory             13290Mi (41%)  14214Mi (44%)
  ephemeral-storage  0 (0%)         0 (0%)
  hugepages-1Gi      0 (0%)         0 (0%)
  hugepages-2Mi      0 (0%)         0 (0%)
Events:
  Type    Reason                   Age   From             Message
  ----    ------                   ----  ----             -------
  Normal  Starting                 22m   kube-proxy
  Normal  Starting                 22m   kubelet          Starting kubelet.
  Normal  NodeAllocatableEnforced  22m   kubelet          Updated Node Allocatable limit across pods
  Normal  NodeHasSufficientMemory  22m   kubelet          Node kind-control-plane status is now: NodeHasSufficientMemory
  Normal  NodeHasNoDiskPressure    22m   kubelet          Node kind-control-plane status is now: NodeHasNoDiskPressure
  Normal  NodeHasSufficientPID     22m   kubelet          Node kind-control-plane status is now: NodeHasSufficientPID
  Normal  RegisteredNode           22m   node-controller  Node kind-control-plane event: Registered Node kind-control-plane in Controller
  Normal  NodeReady                22m   kubelet          Node kind-control-plane status is now: NodeReady

Reproduction script
Following the steps in the doc can reproduce this error.
Anything else
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

The PyTorch examle runs 4 workers with 2 CPUs each. For some reason you only have 3 workers on your kind cluster

The example references 4 here: https://github.com/ray-project/kuberay/blob/master/ray-operator/config/samples/pytorch-mnist/ray-job.pytorch-mnist.yaml#L43
So weird how yours is only running 2. Can you share the full RayJob output in your cluster?

Related issues: https://github.com/ray-project/ray/issues/47316


Request for paper or report

Thanks for your brilliant work and sharing. I am confused of the idea of this repo, how could I read your paper or report? Looking forward to your reply.

Thank you for your interest and kind words!  The camera-ready version of the paper is currently being prepared and is not available yet.  In the meantime, you might find the preliminary draft on OpenReview useful for understanding our work.  You can access it here: https://openreview.net/pdf?id=nt9YEGfXf3.  Please feel free to reach out if you have any further questions or need more information.

ok, Thanks!

the access is restricted

+1 seems openreview link is a restricted link

+1. I'm interested in this work. But access to https://openreview.net/pdf?id=nt9YEGfXf3 is restricted. If you can provide a new link it would be great. Thanks! @XiaolongTang23

+1 openreview cannot download the work

I'm really sorry. I made a mistake. I have just submitted the article to Arxiv, and it will be available soon.


[INLONG-8464][Sort] Add JDBC connector on Flink 1.15

[INLONG-8464][Sort] Add JDBC connector on Flink 1.15
Fixes #8464
Motivation
Add JDBC connector on Flink 1.15
Modifications

create new modules for connectors-v1.15 and connector-jdbc-v1.15
Support audit metrics for sort-connector-jdbc-1.15

Verifying this change
(Please pick either of the following options)

[x] This change is a trivial rework/code cleanup without any test coverage.

[ ] This change is already covered by existing tests, such as:
(please describe tests)

[ ] This change added tests and can be verified as follows:

Documentation

Does this pull request introduce a new feature? (no)

Please add clickhouse support as well https://inlong.apache.org/zh-CN/docs/quick_start/data_ingestion/file_pulsar_clickhouse_example


adding className for styling

Adding a class name to the display area for styling purposes. /cc @rgbkrk

Great, good idea. Makes me wonder if we should put up a style guide or doc on our classes for people to use this.

Went ahead and shipped as v0.0.2


Add exchange parameter to backtesting.py wrapper

 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.


newspaper.fulltext AttributeError

AttributeError: 'NoneType' object has no attribute 'xpath'
Repro with python3:

import requests
import newspaper
resp = requests.get("https://capitalandgrowth.org/questions/1250/hair-salon-appointments-what-is-the-best-exit-inte.html")
newspaper.fulltext(resp.text)

File "/usr/local/lib/python3.7/site-packages/newspaper/api.py", line 91, in fulltext
top_node = extractor.post_cleanup(top_node)
File "/usr/local/lib/python3.7/site-packages/newspaper/extractors.py", line 1040, in post_cleanup
node = self.add_siblings(top_node)
File "/usr/local/lib/python3.7/site-packages/newspaper/extractors.py", line 869, in add_siblings
baseline_score_siblings_para = self.get_siblings_score(top_node)
File "/usr/local/lib/python3.7/site-packages/newspaper/extractors.py", line 926, in get_siblings_score
nodes_to_check = self.parser.getElementsByTag(top_node, tag='p')
File "/usr/local/lib/python3.7/site-packages/newspaper/parsers.py", line 123, in getElementsByTag
elems = node.xpath(selector, namespaces=NS)
AttributeError: 'NoneType' object has no attribute 'xpath'

I didn't have this problem when running my codes. I'm facing the same attribute error since yesterday.

I get the same with Python 3.7.

I am having the same issue with very simple html pages:
$ python
Python 3.7.0 (default, Jun 28 2018, 07:39:16)
>>> import requests; import newspaper
>>> from newspaper import fulltext
>>> html = requests.get('http://localhost/.../articles/group2_2.html').text
>>> html
"\n\n
Hi this is a fake article
\n\n
Some random text
\n\n\nto 3\n\n\n"
>>> fulltext(html)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/mypath../lib/python3.7/site-packages/newspaper/api.py", line 91, in fulltext
    top_node = extractor.post_cleanup(top_node)
  File "/mypath../lib/python3.7/site-packages/newspaper/extractors.py", line 1040, in post_cleanup
    node = self.add_siblings(top_node)
  File "/mypath../lib/python3.7/site-packages/newspaper/extractors.py", line 869, in add_siblings
    baseline_score_siblings_para = self.get_siblings_score(top_node)
  File "/mypath../lib/python3.7/site-packages/newspaper/extractors.py", line 926, in get_siblings_score
    nodes_to_check = self.parser.getElementsByTag(top_node, tag='p')
  File "/mypath../lib/python3.7/site-packages/newspaper/parsers.py", line 123, in getElementsByTag
    elems = node.xpath(selector, namespaces=NS)
AttributeError: 'NoneType' object has no attribute 'xpath'

For some reason it does work for another, very similar fake article:
>>> html = requests.get('http://localhost/.../articles/fake_article1.html').text
>>> fulltext(html)
'Hi this is a fake article 1\n\nAnd it is about Basketball'
>>> html
"
Hi this is a fake article 1
\n\n
And it is about Basketball
\n\nLink to Fake Article 2\n"
>>>

Thanks for finding the bug guys, really appreciate this!

Hi @reactionhashs, I checked your last code with a few URLs in articles.txt. The error you are facing is because your requested some URL's articles don't contain the Full Body text in the P, PRE, and TD tags, while it's present in the DIV tag. So, a possible solution for you currently might be to ignore the URLs by using exception handling in python until @codelucas accepts the above-proposed solution.

Seems like this div tag is not the issue. I had my scraper built with newspaper and I'm using fulltext() to get the content of the url like below gives me error AttributeError: 'NoneType' object has no attribute 'xpath'
from newspaper import fulltext
import cloudscraper

url = 'http://www.pharmafile.com/news/544464/novartis-reveals-hard-hitting-five-year-data-its-gene-therapy-zolgensma-spinal-muscular-'

scraper = cloudscraper.create_scraper()

ht = scraper.get(url)
txt = ht.text
print(fulltext(txt))

But the above piece of code works fine for this url "https://www.eurekalert.org/pub_releases/2020-03/dci-nrs031720.php" though the content is present in  tags succeeding the  tag.
Please help in resolving this issue. If there's already a resolution on this please guide me through that. This issue is priority in my current work and I'm trying this since a week but nothing really helps.
Thanks in advance.

Same issue for me, still no workaround?

I am experiencing the same issue, as @shashank7596 mentioned, even though the texts are written in div, it raises this AttributeError.
Any suggestions?


chore: add sourceip field

Summary of changes
Asana Ticket: [extra] 🚲 Log Trike sequence breaks
Modifies ocs_saver to ignore fields that aren't part of the schema by using mask instead of create from superstruct (https://github.com/ianstormtaylor/superstruct).
Adds a test to validate that sourceip (and, in turn, any additional field) doesn't break the core processing loop.

I think OCS Saver is not following the spec; adding new fields should not cause it to break. Instead, the new fields should be ignored.

agree with @paulswartz. OCS Saver is not following JSON Schema: https://json-schema.org/understanding-json-schema/reference/object.html#additional-properties

The additionalProperties keyword is used to control the handling of extra stuff, that is, properties whose names are not listed in the properties keyword or match any of the regular expressions in the patternProperties keyword. By default any additional properties are allowed.

(emphasis mine)
@nlwstein, could you please make sure OCS Saver ignores unknown fields and add tests for this?

Sure! It's not a huge deal but will require modifying how the existing tests work slightly.

agree with @paulswartz. OCS Saver is not following JSON Schema: https://github.com/mbta/schemas/blob/b9703c1cac6305b1cb33546c2e25f846559ed346/schemas/com.mbta.ocs.raw_message.json
Per https://json-schema.org/understanding-json-schema/reference/object.html#additional-properties:

The additionalProperties keyword is used to control the handling of extra stuff, that is, properties whose names are not listed in the properties keyword or match any of the regular expressions in the patternProperties keyword. By default any additional properties are allowed.

(emphasis mine)
We're not (currently) setting "additionalProperties": false for data. @nlwstein, could you please make sure OCS Saver ignores unknown fields and add tests for this?

@bklebe I force-pushed a new version of this that just makes OCS Saver ignore extra fields.


Map merge aggregation function

Hi, I'd like to submit this new aggregation function that merges map<K, V>s. The examples below show what it can do.
This new map_merge(map<K, V>) reuses the logic in MapAggregation.combine(..) by delegating to it.
Example 1:
select map_merge(foo), map_merge(bar) from (
 select map(array['a', 'b'], array[1000, 2000]) as foo, cast(null as map<varchar, double>) as bar
 union
 select map(array['c', 'd'], array[3000, 4000]) as foo, cast(null as map<varchar, double>) as bar
);

Example 2:
select map_merge(foo), map_merge(bar) from (
 select cast(null as map<varchar, bigint>) as foo,
        cast(null as map<bigint, double>) as bar
 union
 select map(array['a', 'b'], array[1000, 2000]) as foo,
        map(array[33], array[3300.33]) as bar
 union
 select map(array['c', 'd'], array[3000, 4000]) as foo,
        map(array[44, 55], array[4400.44, 5500.55]) as bar
 union
 select map(array['e', 'f', 'g', 'h', 'i'], array[50, 60, 70, 80, 90]) as foo,
        cast(null as map<bigint, double>) as bar
 union
 select map(array['j', 'k', 'a', 'b', 'n', 'o', 'p'], array[200, 300, -45, -55, 600, 700, 800]) as foo,
        cast(null as map<bigint, double>) as bar
 union
 select map(array['c', 'd'], array[90000, 120000]) as foo,
        map(array[44, 55], array[1100.0, 2200.0]) as bar
);

Example 3:
with
part_a as (
 select map(array['a', 'b'], array[1000, 2000]) as foo, 'homer' as first_name, 'simpson' as last_name
),
part_b as (
 select map(array['c', 'd'], array[3000, 4000]) as foo, 'marge' as first_name, 'simpson' as last_name
),
part_c as (
 select map(array['a', 'd'], array[12, 3]) as foo, 'moe' as first_name, 'szyslak' as last_name
),
part_d as (
 select cast(null as map<varchar, bigint>) as foo, 'bart' as first_name, 'simpson' as last_name
)
select last_name, map_merge(foo) from (
 select * from part_a
 union
 select * from part_b
 union
 select * from part_c
 union
 select * from part_d
)
group by last_name;

Example 4:
with
part_a as (
 select cast(null as map<varchar, bigint>) as foo, 'homer' as first_name, 'simpson' as last_name
),
part_b as (
 select cast(null as map<varchar, bigint>) as foo, 'marge' as first_name, 'simpson' as last_name
),
part_c as (
 select cast(null as map<varchar, bigint>) as foo, 'moe' as first_name, 'szyslak' as last_name
),
part_d as (
 select cast(null as map<varchar, bigint>) as foo, 'bart' as first_name, 'simpson' as last_name
)
select last_name, map_merge(foo) from (
 select * from part_a
 union
 select * from part_b
 union
 select * from part_c
 union
 select * from part_d
)
group by last_name;

@martint Please have a look at the latest code with unit tests. Thanks!

We discussed and decided to call this map_union.

@dain Done (map_union). Have a look.

The pull request is rather old and I'm sorry for not having merged it earlier.
Given the age, I decided to just make the changes needed myself and put up a separate pull request (instead of going back and forth with comments) #5282.
Thank you for your contribution.

Finally :-)
Thanks.


[Enhancement] Choose which relationship to use when nesting

As of today (Archi 4.6) when in a view you nest A inside B, but some relationships exist in the model to justify this nesting, Archi choose one of these relationship randomly (well, I guess this is the first one found, but I don't know on which criteria this one is the first one ;-)
On good enhancement would be to:

Show the list of existing relationships (if any) so that the use can choose the right one, but...
Add an option for the user to create a new one if none of the existing ones match the intended semantic (and in this case show the usual dialog which is visible when no relationship exist)

Not sure what you mean. At what point does this happen?


Updated the version puppeteer

Purpose
Attempt to resolve circle ci build failures on the pdf service
Approach
Upgrade the base docker image and align the package.json versions to match
During testing of the image upgrade I was able to create a failure by running multiple builds in quick succession, I add the --no-cache option to the build step and have since seen no further errors - may not be the root, but the time difference seemed minimal so the stability trade off seemed worth while
Checklist

[x] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

PDF generation  looks fine.


How reference script PlutusV2.txOutReferenceScript corresponds to Babbage era TxOutCompactRefScript?

Please can you help me to understand how reference scripts relate from Plutus V2 to cardano-ledger's Babbage Era?
I'm trying to convert Plutus V2 to Babbage Era Tx for testing purposes.
The TxOutCompactRefScript expects TxOut to contain the full script
as it type goes:
  | TxOutCompactRefScript
       {-# UNPACK #-} !(CompactAddr (Crypto era))
       !(CompactForm (Core.Value era))
       !(Datum era)
       !(Core.Script era)

So it contains full script and it seems that we don't win in TxSize except that it's in the UTxO set arounnd TX.
But when I try to convert from Plutus V2 it has ScriptHash in txOutReferenceScript.
Should I calculate the map of ScriptHash to Script for conversion?

It seems that PlutusV2 TxOut does not correspond to Babbage era TxOut.

I'm sorry, I don't understand the question.

trying to convert Plutus V2 to Babbage Era Tx

Are you trying to use a V2 script as a refernce script?

So it contains full script and it seems that we don't win in TxSize except that it's in the UTxO set arounnd TX.

The point of reference scripts is that you can later use them via reference inputs. This way, if you want to use the script multiple times, you only need to pay the cost of putting it in a transaction once. See CIP-33.

PlutusV2 TxOut does not correspond to Babbage era TxOut

What do you mean by a "PlutusV2 TxOut"?
I'm going to close this issue, since we use them to track work in the ledger repo. Questions like this are better asked on our stack exchange..


Can't open anymore the Vienna App after I have added a subscription

The Vienna App continues to crash, do you guys have any idea about why it's crashing? i have attached the crash report:
Vienna_2017-05-07-133307_Giannis-MacBook-Pro.crash.zip
Both buttons lead to crash of the app. If there is some way to save my subscribtions before reinstalling the App.

The crash report is not very specific, unfortunately. It does not show where Vienna is at fault. Can you remember what you did before the crash occurred?

I was adding an rss to a group folder. After that i pressed the arrow to expand that folder, but it didn't expand(i am sure that folder had different rss inside except the one i was adding). Then i pressed again the arrow - > the group folder disappeared from the list and the crash occurs.

Did you try removing the preferences file ?
~/Library/Preferences/uk.co.opencommunity.vienna2.plist
This might also help : http://forums.cocoaforge.com/viewtopic.php?t=26313&p=137346#p137346

I have realised what is the error. If you try to add this feed ( http://www.cleverhans.io/feed.xml ) in a group folder it crashes. I have checked the feed and it doesn't contain anything, so i guess this is the reason why Vienna App crashes. Is there a check if we have added a valid feed or not?

The culprit is the empty feed name in the database 😈

Temporary workaround : replacing the (NULL) value with something sensible.

So if i understand correctly the feed name is stored in the foldername field of the DB. I think it's essential to check that a field it's not null before doing any operation, so the user is informed that is trying to add an invalid feed.

Yup I can work on it if someone else isn't already

Please do !

@josh64x2 thanks for your time ;)

Fixed in current stable branch

Closed at version 3.1.12

Thanks both @lapcat , @barijaona for this fix!

You’re welcome.


feat(Code): Code can now officially delegate its comparison

Using two new kinds of function:

func(t *td.T, arg)
func(assert, require *td.T, arg)

this way the usage of *td.T methods is secure. Note that these
functions do not return anything.
At the same time, using a *td.T instance as Cmp* first parameter
allows to inherit its configuration.
td.Cmp(td.Require(t), got, 42)

is the same as:
td.Require(t).Cmp(got, 42)

@Alkorin you should like this...


Rust analyzer panics when used in Emacs: assertion failed: c.num == crate_map.len() as u32

rust-analyzer version: rust-analyzer db2a7087b 2021-12-13 stable
rustc version: rustc 1.58.1 (db9d1b20b 2022-01-20)
When opening a new project, Emacs fails to communicate with rust-analyzer, and I get a panic in the rust analyzer stderr:
thread '<unnamed>' panicked at 'assertion failed: c.num == crate_map.len() as u32', src/tools/rls/rls-analysis/src/lowering.rs:140:13

rc/tools/rls/rls-analysis/src/lowering.rs: this is a panic coming from rls, not rust-analyzer. So for some reason your emacs seems to run rls instead of/with rust-analyzer

Oh you're right... It probably remained there after I tried switching back and forth between rls and RA. However the fact remains that RA doesn't work for me. I'm trying to spot an error message or artifact I can attach here...

I will open a new issue with more specific details. Thanks in the mean time


clangd wants to replace . with -> but it results in .->

Problem
In C you can access the variables of a struct with ->
clangd is able to complete variables even if instead . is typed.
It deletes the . and replaces it with -> on completion.
When using nvim-compe the . doesn't get deleted.
Instead the completion results in .->
Info
NVIM v0.5.0-dev+1073-gb894fbcbe
clangd version 11.0.1
Minimal init.vim
call plug#begin($XDG_DATA_HOME.'/nvim/plugins')
    Plug 'hrsh7th/nvim-compe'
    Plug 'neovim/nvim-lspconfig'
call plug#end()

let g:compe = {}
let g:compe.source = {}
let g:compe.source.nvim_lsp = v:true
lua require 'lspconfig'.clangd.setup{}
se completeopt=menu,menuone,noselect

Minimal bug.c
struct Foo {
   int foo;
};

int main() {
   struct Foo *foo;
   foo;
}

Reproduce

Create bug.c
Place cursor between foo and ; like foo|;
Type . to trigger completion
Select first completion with <C-n>

The line with the completion looks now like
foo.->foo;

Instead it should look like
foo->foo;

@astier If you invoke compe#confirm when the line is foo.->foo|, the compe will replace the line to foo->foo correctly. So it is not bug of nvim-compe, I think.

https://user-images.githubusercontent.com/51877647/107307651-a71b2480-6ab9-11eb-8a22-a39907c78445.mp4
You need to invoke compe#confirm it, I think. Because this doesn't happen to me, it only happens if I didn't press <CR> which I've remapped to compe#confirm

Oh... coc supports it...
Icthink it relates to #153
At least, you can fix by compe#confirm

compe#confirm works. With lexima I also have to call call lexima#set_default_rules() to make it work.

regression...

I've confirmed this problem is solved.

not working... (too difficult...)

It maybe works perfectly.

What a rollercoaster. :)

@hrsh7th is the best maintainer I ever seen, he doesn't let things slide easily :laughing: . Also, very good at creating great tools :smile:

Ah, I did not mean anything negative. Just enjoyed following this issue. I have nothing but huge admiration for @hrsh7th and all his work. Outstanding! Entire nvim community and what is happening around it these days is breath taking.

@cloggier :laughing: :laughing: I know you didn't mean anything, I was kinda agreeing with you :laughing: then I mentioned what great job @hrsh7th does.

I will work if interest it.
This issue (and #153) is really interesting about how VSCode/coc.nvim works.
I felt it again that coc.nvim is great while implementing this.

After revisiting compe after a while I tried the same example as posted above with the minimal init.vim and bug.c however I have the same issue again. Is it supposed to work now?

Can we reopen this?

I think we can't fix it at the moment...

I think we can't dot-repeatable it... (because this completion item uses textEdit.

Eventually I will open a new issue for this. Thanks for the great work.


Change CAP descriptor formatting.

Having the two conversion types in the middle becomes really annoying after a while

CAP descriptor now carries only one conversion type


Parse drafts as well

Hello, sorry for late response!
I'm not sure if this a good idea exactaly like you coded.
Maybe we can add an includeDrafts parameter to the settings to do this.
I was long time not touching this package, but I will update it and publish a new release soon.
I will include this idea.
Thanks for coontributing!

No problem. I'll leave it on you, thank you.


create a project/milestone

i need to track the progress made on implementing the trucky api.

created the project, now i just need to check what has already been done


Help me understand the logic between GitPitch

As an exercice, I'm trying to build a GitPitch-like application with Laravel, but I can't figure out how the application works (never used Java before 😞).
From the file structure, I'm supossing the framework you're using it's a MVC framework, but I'm stuck there.
It would be very helpful if you could explain me the request flow that happens when I request, for example, this repo's presentation.

Hi @m1guelpf
GitPitch has been designed to follow standard Play Framework conventions. So to begin, in order to understand the layout of the project itself I recommend the following article, Anatomy of a Play Application.
If you really want to understand how it all works you will have a lot to learn. And undoubtedly the best place to start is with the official Play Framework documentation.
More generally, I'd imagine you would be better off embracing Laravel conventions in your design and approach for whatever exercise you are working on. Honestly, GitPitch functionality is probably not a great candidate for a learning exercise given it's inherent complexity. Far too complex for me to explain here.
Best of luck with all things Laravel. I've heard great things about that framework.

@gitpitch I've managed to understand everything but the asset organization. Could you explain what assets could be pulled from npm, what assets have been modified, what assets have been created and why you have like 3 different versions of some assets?
Thank you!

GitPitch works with many versions of the same assets to support distinct use cases and features at runtime. I'm not familiar with npm or how it manages assets.


Update product-permission-analytics.md

Removing step that mentions the Configuration tab. There is not Configuration tab.

Learn Build status updates of commit a0cef98:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error: GitBranchDeletedOrForcePushed]

Cannot sync git repo to specified commit because ref +refs/pull/101435/merge has been deleted or has been force pushed.
 If this is pull request, please ensure it is mergeable (no merge conflict) and open (not closed).

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Send one camera to multiple Android TV's

Is it possible to send one camera screen shot to multiple Android TV's?

Is it possible to send one camera screen shot to multiple Android TV's?

Yep.  You would need to change the environment variable for R2ATV_IP_ADDRESS to an array and then encapsulate the sendNotification method to loop through the array to send a notification to each TV.  Maybe something like this?
async function sendNotification(title, message, imageFile, exitAfter = false) {
        tvIpAddress.foreach(function(entry) {
                const options = {
                    method: "POST",
                    url: "http://" + entry + ":7979/notify",
                    port: 7979,
                    headers: {
                        "Content-Type": "multipart/form-data"
                    },
                    formData: {
                        "duration": displayTime,
                        "position": 0,
                        "title": title,
                        "titleColor": "#0066cc",
                        "titleSize": 20,
                        "message": message,
                        "messageColor": "#000000",
                        "messageSize": 14,
                        "backgroundColor": "#ffffff",
                        "image": (imageFile == '') ? "" : fs.createReadStream(__dirname + '/' + imageFile),
                        "imageWidth": 640
                    }
                }

                // Fire off POST message to PiPup with 'request'
                request(options, function(err, res, body) {
                    if (err) {
                        console.log(`[ERROR] Error sending notification: ${title} - ${message}`)
                        console.log(err)
                        process.exitCode = 1
                    } else {
                        console.log(`Sent notification successfully: ${title} - ${message}`)
                    }
                    if (exitAfter) process.exit()
                })
            }
        }

Great thanks for the quick response.
On Feb 1, 2021, at 1:14 PM, Steven Flesch <notifications<EMAIL_ADDRESS>wrote:
Is it possible to send one camera screen shot to multiple Android TV's?
Yep. You would need to change the environment variable for R2ATV_IP_ADDRESS to an array and then encapsulate the sendNotification method to loop through the array to send a notification to each TV. Maybe something like this?
async function sendNotification(title, message, imageFile, exitAfter = false) {
tvIpAddress.foreach(function(entry) {
const options = {
method: "POST",
url: "http://" + entry + ":7979/notify",
port: 7979,
headers: {
"Content-Type": "multipart/form-data"
},
formData: {
"duration": displayTime,
"position": 0,
"title": title,
"titleColor": "#0066cc",
"titleSize": 20,
"message": message,
"messageColor": "#000000",
"messageSize": 14,
"backgroundColor": "#ffffff",
"image": (imageFile == '') ? "" : fs.createReadStream(__dirname + '/' + imageFile),
"imageWidth": 640
}
}
            // Fire off POST message to PiPup with 'request'
            request(options, function(err, res, body) {
                if (err) {
                    console.log(`[ERROR] Error sending notification: ${title} - ${message}`)
                    console.log(err)
                    process.exitCode = 1
                } else {
                    console.log(`Sent notification successfully: ${title} - ${message}`)
                }
                if (exitAfter) process.exit()
            })
        }
    }

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fstevenflesch%2Fring-to-android-tv%2Fissues%2F11%23issuecomment-771051996&data=04|01||42bdccccc9284f986be908d8c6dd3175|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637478000603027841|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=7dAJpLaSq2Y05J0mdkCmLeh7hEBC%2FCfXv%2FD%2BDxfg4TA%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAECZ3QASC6L3WOI3X7EMZKTS43VPVANCNFSM4W5G2OXA&data=04|01||42bdccccc9284f986be908d8c6dd3175|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637478000603037800|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=3v1ByVW1gM47UuUikvCkUoTMsneGwS0jsWqr%2FRkoXvc%3D&reserved=0.


CLI is not very helpful when arguments are missing.

Release
Operating System
Cause

next
Windows & OSX & Linux)
Code

Context
When the user issues a CLI command but forgets one of the required arguments, the CLI prints the entire help string and then exits.
It would be more helpful to the user if the CLI could instead point out which arguments are missing.
Steps to Reproduce
Consider the following command:
$ cardano-wallet address list 2512a00e9653fe49a44a5886202e24d77eeb998f

The final positional parameter is the wallet-id parameter.
Now suppose we omit that parameter, and instead issue the following command:
$ cardano-wallet address list

Expected behavior
The CLI gives the user feedback that is specific and relevant, perhaps similar to:
$ cardano-wallet address list

Error: missing parameter <wallet-id>.

Usage: cardano-wallet address list [--port=INT] [--state=STRING] <wallet-id>

Actual behavior
The CLI prints out the entire help string, with no specific help to identify what went wrong:
See more

$ cardano-wallet address list

Cardano Wallet CLI.

The CLI is a proxy to the wallet server, which is required for most
commands. Commands are turned into corresponding API calls, and submitted
to an up-and-running server. Some commands do not require an active server
and can be run "offline". (e.g. 'generate mnemonic')

    ⚠  Options are positional (--a --b is not equivalent to --b --a) ! ⚠

Usage:
  cardano-wallet server [--network=STRING] [--port=INT] [--bridge-port=INT]
  cardano-wallet mnemonic generate [--size=INT]
  cardano-wallet wallet list [--port=INT]
  cardano-wallet wallet create [--port=INT] <name> [--address-pool-gap=INT]
  cardano-wallet wallet get [--port=INT] <wallet-id>
  cardano-wallet wallet update [--port=INT] <wallet-id> --name=STRING
  cardano-wallet wallet delete [--port=INT] <wallet-id>
  cardano-wallet transaction create [--port=INT] <wallet-id> --payment=PAYMENT...
  cardano-wallet address list [--port=INT] [--state=STRING] <wallet-id>
  cardano-wallet -h | --help
  cardano-wallet --version

Options:
  --port <INT>                port used for serving the wallet API [default: 8090]
  --bridge-port <INT>         port used for communicating with the http-bridge [default: 8080]
  --address-pool-gap <INT>    number of unused consecutive addresses to keep track of [default: 20]
  --size <INT>                number of mnemonic words to generate [default: 15]
  --payment <PAYMENT>         address to send to and amount to send separated by @: '<amount>@<address>'
  --network <STRING>          testnet, staging, or mainnet [default: testnet]
  --state <STRING>            address state: either used or unused

Examples:
  # Create a transaction and send 22 lovelace from wallet-id to specified addres
  cardano-wallet transaction create \
    2512a00e9653fe49a44a5886202e24d77eeb998f \
    --payment<EMAIL_ADDRESS>

Resolution Plan

PR

Number
Base

#?
develop

QA

I remember there was a similar point about showing CLI help (along with some other minor improvements to the CLI) on https://github.com/input-output-hk/cardano-wallet/wiki/Blackboard and we even discussed it on one of the meetings. I don't see it anymore however, I don't remember if it was supposed to land as a ticket (but I don't see ticket either) (@KtorZ?)

This is now on our roadmap. As a consequence, I'll close this ticket and point to that roadmap instead. This is a known issue that we'll eventually tackle :+1:


Add windows support

using netsh.exe wlan show networks mode=Bssid
if you can help please attach sample output (as text no images) as a comment.

I'm interested in Windows support.
Here is the output of that command, (I modified some MAC addresses and SSIDs for privacy reasons but didn't modify the format).
Notice that it contains the output of all two enabled Wifi Interfaces. One has a better antenna than the other.

I just noticed that the paste expired, did you download it before it expired?

I made a new scan, here.
Nothing is stopping you from adding windows support now :)

I don't have the time, but I'll be adding a contributors.md so others can step in to help.


IPython indentation issue

When a Python terminal is open with IPython running, and you select a code block and select "execute in Python console", it adds additional tabs in the pasted text before executing and returns syntax error.
This happens on Mac, not sure about Windows.

Can you verify you don't have the issue if you are not running IPython? If so that would suggest IPython is injecting the extra indentation.

Yes, specifically ipython. Obviously something is up however as PyCharm can send code to iPython with no issue. Perhaps there should be a “send to iPython” command that correctly adjusts indentation?
Getting this feature correct I believe is important to get the scientific computing (and perhaps also data science) community onboard with VSCode and Python.

@Moretizzle

select "execute in Python console",

There is no such command in the extension.

Please could you provide some sample code
Please could you provide the exact steps used to replicate this issue.
For instance when do you run ipython, is this after you execute some code, or before?
How are sending the python code to the terminal (what menu or command are you using)?

Closing this issue has It has been over a month since we requested some information.
Please feel free to re-open the issue with the requested information.