Add ability to easily switch between TaskWarrior databases

Feature request culled from #233
An approach like this would allow pretty easy switching between TaskWarrior databases:

A configuration option to specify 'shortcuts' to various task databases
A new action that would allow a user to enter a shortcut name, which would then reload VIT with the task database the shortcut refers to.

Should be pretty efficient, for the following reasons:

The action would be bound to a key, just hit the keyboard shortcut and enter the alias. Both history and tab completion would be supported for alias name
I believe you could also map additional keyboard shortcuts to do a direct switch to a particular task database.

Only somewhat related, but it would be nice if vit could accept a command line parameter specifying data folder to use.
That could be used to feed task with e.g. a default rc.data.location override, or possibly setting the TASKDATA envvar appropriately beforehand.
Alternatively allow passthrough command line param, which could e.g. also be used to specify alternate taskrc location (but there could be dragons)
Anyway, thanks for the good work!

I haven't tried it, but do things work as you want if you export TASKDATA to the location you want before calling vit?

Yes, that variable is used by TaskWarrior (and indirectly by vit) once it's set/exported, and things work as expected
You can use task rc.data.location:alternateDataLocationDirectory rc:alternateTaskrcConfFile ... or set associated TASKDATA and TASKRC environment variables.
See "CONFIGURATION FILE AND OVERRIDE OPTIONS" section in task(1) manpage for details.

@fdela I think your idea should probably be filed as a separate feature request.


L515 (RGB) Dynamic Calibration Tool

Issue Description
For the D400 the tool the Dynamic Calibration Tool (specifically Intel.Realsense.CustomRW.exe) exists to read and write custom calibrations onto the device. I was wondering if the same tool or an equivalent is available for the L515 RGB Camera.
Best,
Faaux

Hi @Faaux
The tool that you reference is only for the D400 product line.  It does not extend to the L500 product line.  Also to complete an answer to your question, there is no equivalent tool for L515.

Thanks for the quick answer. I would like to ask another question then (if I should open a new issue for this please tell me).
Context
My goal is to calibrate a D435 to a L515. I then want to either transform the D435s left IR channel to match the L515 depth channel or the other way round. The extrinsic calibration of this is already done using 3d3d pose estimation of the point clouds. As I want to use the IR image for segmentation of the scene I'd like to map the pixels to the more accurate L515 depth values.
Current approach
The current approach was to map the IR image to the L515s RGB sensor and then use the internal align between the RGB to Depth of the L515 (calibration via OpenCV of IR ot L515s RGB). For this I would like to get the L515s RGB intrinsics as well as the extrinsics between the L515s RGB sensor and the depth sensor.
Questions

Is this a sensible approach?
What would you recommend?
Is there a way to get the intrinsics and extrinsics of the L515?
Are there specialized functions to project the point cloud back down to the image in librealsense (essentially rs2_project_point_to_pixelfor a whole cloud)?


Model's idField ignored

Steps to reproduce
This is the first time I'm using feathersjs and vuex so apologies if I've misunderstood the documentation.
I have a bikes service that interacts with my backend service. I'm able to fetch my data fine but I'm struggling to patch it back to the server.
I have the following idField specified on my bikes model:
...
const servicePath = 'bikes'
const servicePlugin = makeServicePlugin({
  Model: Bike,
  idField: 'bikeId',
  debug: true,
  service: feathersClient.service(servicePath),
  servicePath,
})
...

On the server the bikeId field is also specified:
export default function (app: Application) {
  const options = {
    Model: createModel(app),
    paginate: app.get('paginate'),
    id: 'bikeId',
  };
  ...

After a successful useFind the state for bikes shows that the correct idField is set in $store.state.bikes.idField but that field isn't used as the id, instead it is using the _id field from mongo on the server:
{
  "ids": [
    "5e9acedc2909ac001aa7ab85",
    "5e9acedc2909ac001aa7ab87",
    "5e9acedc2909ac001aa7ab88",
    "5e9acedc2909ac001aa7ab89",
    "5e9acedc2909ac001aa7ab86",
    "5e9acedc2909ac001aa7ab8b",
    "5e9acedc2909ac001aa7ab8a",
    "5e9acedc2909ac001aa7ab8c"
  ],
  "keyedById": {
    "5e9acedc2909ac001aa7ab85": {
      "stravaAthleteId": 2164652,
      "bikeId": "b2680865",
      "order": 1,
      "_id": "5e9acedc2909ac001aa7ab85",
      "bikeName": "Canyon Dude",
      "resourceState": 2,
      "totalDistance": 3046948,
      "updatedAt": "2020-04-20T07:41:45.262Z",
      "createdAt": "2020-04-19T17:35:35.849Z"
    }
...

When I use the model's save method the id used for the patch is the long mongo id which the server ignores as it isn't using that id to match records, it is expecting the bikeId field's value instead. If I log the incoming requests on the server in the patch method, I can see the id is not the correct one, for example: Incoming patch with id:  5e9acedc2909ac001aa7ab8c
I'm using the rest client on a Nativescript built Android app (currently just in simulator). Connections between server and client are working fine.
Expected behavior
I would expect the model's save method to use the value of the configured idField.
Actual behavior
Instead the model's save method uses the _id field as the id which doesn't match what the server is expecting.
System configuration
Tell us about the applicable parts of your setup.
Module versions:
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
NodeJS version: v12.16.1
Operating System: Linux
Browser Version: Frontend is using Nativescript/Android
React Native Version: N/A. Using Vue and Nativescript
Module Loader: Webpack

I had a poke around in the code and I think my issue is related to there being a _id field in the result sent back from the server. In getId the comment says it searches in the following order:
Get the id from a record in this order:
  1. id
  2. _id
  3. the `idField`

In after hooks on both the server and client I've removed the _id field but now when I use the save method on the client it sends create instead of patch. I will continue investigating.

Oh I should mention that after removing the _id property, in the client store the fields are now correctly indexed by bikeId:
{
  "ids": [
    "b2680865",
    "b2910542",
    "b6675209",
    "b4062849",
    "b6114126",
    "b2084322",
    "b4578164",
    "b5113219"
  ],
  "keyedById": {
    "b2680865": {
      "stravaAthleteId": 2164652,
      "bikeId": "b2680865",
      "order": 0,
      "bikeName": "Canyon Dude",
      "resourceState": 2,
      "totalDistance": 3046948,
      "updatedAt": "2020-04-20T07:41:45.262Z",
      "createdAt": "2020-04-19T17:35:35.849Z"
    },
...

If I define my model with an explicit idField it seems to work:
class Bike extends BaseModel {
  constructor(data, options) {
    super(data, options)
  }
  static modelName = 'Bike'
  static idField = 'bikeId' // <-- Added explicit idField here
}

I tried adding idField to the options in the constructor but that seems to get ignored.
If adding a static property is the solution I'm happy with that but I didn't come across it in the docs. Do you think this is correct or did I miss something while reading the docs?

Your issue is related the function getId.
Like stated in the comment:

Get the id from a record in this order:

id
_id
the idField

The solution would be easy, check with the idField first.
But maybe @marshallswain had a good reason to get it in that order.
If it's ok for you Marshal, I'll submit a PR, but if there is people that rely on this behavior,
this could be a breaking change, no?

This is a big.  Please file a PR and I’ll do a release.
Sent from my iPhone

On Apr 26, 2020, at 3:24 PM, J3m5<EMAIL_ADDRESS>wrote:

Your issue is related the function getId.
Like stated in the comment:
Get the id from a record in this order:
id
_id
the idField
The solution would be easy, check with the idField first.
But maybe @marshallswain had a good reason to get it in that order.
If it's ok for you Marshal, I'll submit a PR, but if there is people that rely on this behavior,
this could be a breaking change, no?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

PR submitted.
You might want to check on this PR too.


quiz

Nice, almost there. I see some syntax errors, but I can also see that you get the idea. Watch that syntax on strings, for instance: GET /drums/1 to drums#show' should be get /drums/:id to 'drums#show'


Interpreter: Remove kill Query

Signed-off-by: ywqzzy<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #4567
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/merge

/rebuild

/run-integration-test


Digital Clock

Description: In this project we will make a Digital clock that will display the current time in Hours, Minutes and second. This
clock display the time as per Local time zone. We will use HTML and CSS to design this Clock, and for Updating the current time we will use JavaScript Date Object.

2d18fea)
tech stack  :html ,css  ,JavaScript
Are you contributing under any open source program?
yes, Kharagpur winter of code.

Hello @Paarisha1 ....Thank you for showing your interest to work on this issue but we already have this project as a part of Metaverse. Please try to contribute on a different issue.


Template code does not run

Problem encountered on https://dotnet.microsoft.com/en-us/learn/aspnet/blazor-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.
Updated VS to 17.8.6
Created a new project using the 'Blazor Web App' project template.

Click Run button. Project doesn't run correctly.

Sorry, this seems to be a duplicate of #9088. Hopefully my additional information helps.

Adding the following line to Program.cs seems to magically create the missing CSS file:
StaticWebAssetsLoader.UseStaticWebAssets(builder.Environment, builder.Configuration);
After adding this, it seems to run OK, though I still get the WebSocket connection failure... not sure if that's expected or not.

@danroth27 can you help with this one?

@mikeries Can you share your project with us that reproduces the issue? Please include the bin and obj folders.
Would it also be possible for you to collect a binlog when building the problematic project? Instructions for collecting a binlog are here: https://learn.microsoft.com/visualstudio/ide/msbuild-logs?view=vs-2022#provide-msbuild-binary-logs-for-investigation.
The websocket error means that the app is unable to connect back to Visual Studio to enable certain developer productivity features, like hot reload and auto browser refresh. Is there anything on your machine that might be blocking that local connection?


Update npm with most recent module versions

The most recent module versions are not downloadable via npm. Please provide them!

+1

Please add to NPM.  This is last package that I use that I can only get in bower.

+1

Can I add some weight to this please? The versions are all there so it's literally a 10 second job to publish.

1.0.0 has been published


Would it be possible to reformat dates to ISO standards?

Many datasets are flagged with "Recorded date invalid" on GBIF because the dates provided don't follow the ISO standards. See this example: https://www.gbif.org/occurrence/1890646124
In the IPT people can choose the date format when uploading a spreadsheet or CSV file.

Would it be possible to reformat these dates to the ISO standards when the Archive is generated?
I thought the IPT already did that but when I tried to test it, it didn't seem to be the case. I think it would help a lot of publishers.

@ManonGros Also Interpretation supports MDY, but you need to specify that manually by adding machine tag in registry:
name: default_date_format
value one of following: ISO, MDY or DMY

It would be best if the IPT dealt with this to provide a good quality DWCA.
It can either convert the dates from the datasource-specified date format to ISO format, or it could use the dateFormat property of Darwin Core Archive: https://dwc.tdwg.org/text/#221-attributes -- though we currently ignore that attribute during interpretation; I think last time I looked no-one had ever used it!

I think we can move the issue to pipelines project, we already have a mechanism for date formats, we can choose one of the preferable date formats on the interpretation side based on DWCA date format field

I think it would be good to have it handled in the IPT (too). I don't think many publishers know about the dateFormat property of Darwin Core Archive (I have never seen it used either).
It would help more people to have the conversion based on the datasource-specified date format in the IPT.

Fixing it at source (in the IPT) means we have a better DWCA, that others can interpret without workarounds. I'd prefer that than doing this in pipelines.


Creating timing log decorator

Description

As the timing log allows for custom messages, it handles cases where a parameter identified by position in the log is passed as a keyword param and vice versa.

Closing this, as it would require some more work for this to be adapted into the code.


Not showing up in Extension Pack query

Hey @felixfbecker - wondering why this won't show Extension Packs under the category? Perhaps it wasn't published after you added that category.
I want to feature it on the VS Code blog if you want to republish soon.
(Look at Categories below... I expect to see Extension Packs)

Maybe its a bug on our side?

Thanks for the heads-up, published!
Awesome you want to feature this, if there is anything I can help with let me know!


cargo metadata supports artifact dependencies

What does this PR try to resolve?
After this PR, cargo metadata will contain information of artifact dependencies in both packages and resolve.nodes.deps.dep_kinds. The implementation follows this comment https://github.com/rust-lang/cargo/pull/9992#issuecomment-964538002.
fixes #10607
How should we test and review this PR?
Raise some design decisions here:

For field resolve.nodes.deps.dep_kinds.compile_target, if the artifact dependency is specified as { …, target = "target" }, it will show as "target": "<target>" since Cargo doesn't know the exact target triple at that time.
It is forward-compatible with multidep. When Cargo sees multidep, it just appends a new dep_kind onto the array resolve.nodes.deps.dep_kinds.
resolve.nodes.deps.dep_kinds.name is deprecated. If a dependency doesn't have any lib target, it will show as empty string, e.g. "name": "".

Some integration tests:

metadata::workspace_metadata_with_dependencies_and_resolve reflects the change.
metadata::cargo_metadata_with_invalid_artifact_deps verifies syntax error of artifact

Additional information
I might revisit https://github.com/rust-lang/cargo/issues/10407 after this to do clean-ups.

Thanks! Sorry again about my confusion.
@bors r+

:pushpin: Commit d02e36cea10f703c7c39761f1484511304554910 has been approved by ehuss
It is now in the queue for this repository.

:hourglass: Testing commit d02e36cea10f703c7c39761f1484511304554910 with merge f0b6f81a78aba8ae2b1b6ba6c9dbed9f67c0a4f8...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing f0b6f81a78aba8ae2b1b6ba6c9dbed9f67c0a4f8 to master...


[Bug] listreceivedbyaddress includes send addresses

I was using this command:
./dash-cli listreceivedbyaddress 0 true
to generate a list of unused addresses (0 confirmations and 0 amount) to assign to users for payment processing until I discovered that send addresses (external addresses where Dash was sent) are included in the list. This is a legacy bug from Bitcoin Core that is described here:
https://bitcointalk.org/index.php?topic=13011.0
discussed here:
https://www.dash.org/forum/threads/bug-listreceivedbyaddress-includes-send-addresses.16294/
@UdjinM6 observed:

Ah, I see what's going on! They list all addresses saved in wallet's address book BUT address book also stores addresses you sent to a) with some label attached to it or b) via payment request. You can check these addresses in "File" -> "Sending addresses...". I'm not sure if that's a feature or a bug but given the fact that amount is always "0" and no actual txes are listed I guess it's indeed a bug

Hi Jim,
It looks like this was fixed in #1664. Can you confirm?

Fixed in #1664, closing.


Add Apple M1 support to lodestar

Motivation

Description
The core issue is hardware accelarated sse4_crc32 dependency that is suppose to fallback to
javascript implementation but it doesn't since nodejs crashes while loading bindings on M1 chip.
I noticed this issue https://github.com/ashi009/node-fast-crc32c/issues/21#issue-609766808 which actually claims that this lib written in rust even has better performances (library author has no intention on merging this change). I've forked that repo and replaced lib and released under chainsafe org.
I've also updated that dependency in our fork of node-snappy-stream. Hopefully, at some point, somebody (khm google) will force those authors to maintain those "official" libraries.
Tests in all repos are passing and I'm successfully syncing mainnet.

Closes #2475
Steps to test or reproduce

Awesome! Thanks


Add CI to check that requisite information is provided when new target is defined.

          We should also add some CI that checks that all the requisite things are present when a new host is defined, this shouldn't be too tricky.

Originally posted by @hmacdope in https://github.com/choderalab/asapdiscovery/issues/563#issuecomment-1788311198

can just add a test that loops through the allowed TargetTags, runs a few tests in dataviz etc.


mismatch result of triviaqa when using vllm and hf

Hi, I'm trying reproduce the result of llama2-7B on triviaqa benchmark, I use --model two vllm and hf, but the results between them had a big gap. vllm is 0.615 and hf is 0.0701.

Though the n_shot is a little different, I think it's not the main reason.

Could just be the few-shot? You should re-run vllm with the same num_fewshot for a proper comparison.

Based on the output prints, it looks like HF is zero-shot while vLLM is 1-shot, in which case, a discrepancy along these lines is not super surprising (though if there is a discrepancy when number of shots is equal that is a likely bug or divergence.)

Based on the output prints, it looks like HF is zero-shot while vLLM is 1-shot, in which case, a discrepancy along these lines is not super surprising (though if there is a discrepancy when number of shots is equal that is a likely bug or divergence.)

The vllm result is lower though! But yeah, I checked it and the 0-shot vllm result is ~0.52. Don't think it is the stop sequences though. From what I could tell they decode each token per generation and check if it matches the string.

@learner-crapy do you have the generations/output files from these runs saved by chance?

@haileyschoelkopf I didn't save the output files, just took the screenshot of the final result.

@haileyschoelkopf @baberabb I'm sorry for the inconsistent of the num_fewshot of vllm and hf, I re-run hf command with num_fewshot=1, then I get the result 0.0869, a big gap compared with vllm result(0.6155), here is the screenshot.

@learner-crapy no problem! I was looking into it!
@haileyschoelkopf I've attached the zero-shot outputs. Looks like it is partly due to the stop sequences as they HF generations are pretty monosyllabic and the vllm not so much. But some generations are completely different as well.
Uploading Archive.zip…

New zero-shot result on HF is
hf (pretrained=meta-llama/Llama-2-7b-hf), gen_kwargs: (None), limit: None, num_fewshot: None, batch_size: auto
| Tasks  |Version|     Filter      |n-shot|  Metric   |Value |   |Stderr|
|--------|-------|-----------------|-----:|-----------|-----:|---|-----:|
|triviaqa|Yaml   |remove_whitespace|     0|exact_match|0.5247|±  |0.0037|


Enhancement: Use HTTP 2.0 for OpenAI requests

What features would you like to see added?
Should hopefully improve perf slightly. Right now requests to my LiteLLM (aka custom OpenAI API) endpoint are all HTTP/1.1 from what I see. =)
More details
See https://github.com/BerriAI/litellm/issues/3533#issue-2286515074 for example code in Python. Not entirely sure how to do it with node.
Which components are impacted by your request?
General, Endpoints
Pictures
No response
Code of Conduct

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

https://github.com/openai/openai-node?tab=readme-ov-file#customizing-the-fetch-client

gotcha that's an easy implementation, thanks!

@Manouchehri I see, there's a little more to configure to enable h2 by default
https://github.com/nodejs/undici/issues/399


Antec Core HS screen orientation is wrong

Describe the bug
I am trying to install Bazzite on my Antec Core HS as a dual boot on a Windows 11 partition but the screen is in the wrong orientation and cuts off part of the display

What did you expect to happen?
I expected the screen to be landscape and scaled to fit my screen
Output of rpm-ostree status
No response
Hardware
Antec Core HS
Extra information or context
I previously completed the installation but was unable to change the display settings. I tried installing Fedora Kinoite instead but ran into errors so am resetting Windows on my device, therefore I can't run rpm-ostree status. In the download Bazzite screen I chose other handheld PC -> KDE

Under hardware, is that the DMI name from the command in our issue template?

The DMI name is SLIDE
I was able to get it working this time and changed the settings to get the correct orientation.
In Steam there are no buttons mapped to the Xbox buttons Home and Share - should I contact Handheld Daemon about that?

Hello,
I'm no longer using bazzite on my device so I can't confirm if it's still
relevant.
Thanks
Claire
On Tue, 26 Nov 2024 at 16:08, dosubot[bot] @.***> wrote:

Hi, @clairearmstrong https://github.com/clairearmstrong. I'm Dosu
https://dosu.dev, and I'm helping the bazzite team manage their
backlog. I'm marking this issue as stale.
Issue Summary:

You reported an incorrect screen orientation on your Antec Core HS
device when dual booting Bazzite.
You provided the DMI name "SLIDE" to help address the issue.
You managed to correct the screen orientation yourself.
KyleGospo plans to open a PR to officially resolve the orientation
issue.
An issue with unmapped Xbox buttons in Steam was noted, with a
suggestion to report it to Handheld Daemon.

Next Steps:

Please let us know if this issue is still relevant to the latest
version of Bazzite by commenting here.
If there is no further activity, this issue will be automatically
closed in 14 days.

Thank you for your understanding and contribution!
—
Reply to this email directly, view it on GitHub
https://github.com/ublue-os/bazzite/issues/1554#issuecomment-2501264841,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD772B3LP2R7NG7IMBBFZWL2CSMHRAVCNFSM6AAAAABNF43BF2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMBRGI3DIOBUGE
.
You are receiving this because you were mentioned.Message ID:
@.***>


Large yaw drift while stationary

During testing i would frequently encounter the critical warning: Preflight fail: yaw estimate error. After looking a bit at the yaw while the imu is stationary, this is quite understandable. The yaw looks fairly stable until an initial disturbance, after which is drifts significantly. I have played around with the EKF parameters and not have any success. This problem is quite similar to #11847, but i am trying to use the magnometer to constrain the drift. The problem presents itself similarly regardless of the value of EKF2_MAG_TYPE.

Any suggestions, I would expecet it to be a hardware issue, but the IMU and MAG values seem normal. No GPS is used, though i wouldnt expect that to cause these types of issues.

I suggest that you set SDLOG_MODE to create a log from boot and then paste the log here. The plot above is not really enough for me to grasp what is going on.
Also, a picture of your setup might help.

Hi thanks ill attavh a log file here where i initially disturb at around 15 seconds and leave it stationary for the next 4 minutes and then disturbe it again. The external disturbances are fairly easy to recognize.
I have tried to narrow down the problem a bit so will attach some more plots here.
It seems like the problem has somthing to do woth the fusion of the IMU data as you can see the HIGHRES_IMU drifts quite a bit after a disturbance, while the SCALED_IMU gyro values look to be quite accurate.
These plots are with all the default values in the EKF2, but i have tried tweeking the time constants and process noise variables without really producing any different results.
Here is the plot of the HIGHRES_IMU

and here is the SCALED_IMU data for bith IMU's

Another image where i try to ilustrate the problem. 4 external disturbances were applied
t1: small motion with no absolute rotation
t2: Small rotation of the MAV
t3: Larger rotation of MAV
t4: High frequency oscillation of yaw.

During startup all values are stable as well as after a long period the yaw will eventually stabilize.
log_79_2019-12-16-17-01-40.ulg.tar.gz

Please upload the log file to https://logs.px4.io, thanks.

https://logs.px4.io/plot_app?log=45acb2b4-448b-4b20-92e5-d9f087e30d4e

Thanks for the log @aalexjo. That's interesting and I don't think I understand what is going on.
@priseborough would you mind having a quick look at the above log to see if something springs to mind? Thanks.

@aalexjo Do you have a picture of your setup? Are you sure that the mag rotation is correct?

Unfortunately i dont have a very revealing picture of my setup handy, nor the drone. But the mag rotation might be onto something.  I have de-cased the pixhawk 4 mini and softmounted the IMU directly on the flight controller. This involved rotating the IMU unit 180 degrees and mounting it on the bottom of the FCU and mounting this setup so that the IMU was mounted in the standard orientation. I¨ve assumed that the mag was co-located on this unit, but if that is not the case then the  IMU would be flipped 180 degrees in roll compared to the mag which might explain the failure to estimate the bias. I am not sure if this is the case though, i will do some tests again when i have the equipment in front of me.
In the meantime can anyone confirm that this might be the problem?


Add marathon-style constraints for cassandra nodes

Removes prior placement_strategy option entirely, as it was effectively a no-op anyway: could never behave as anything other than NODE, due to the matching port across nodes
Note that constraints must be of the form 'a:b,x:y:z', NOT the json array form. This is due to how the cassandra service currently handles configurations. See integration tests for examples.

LGTM!


on_close() takes 2 positional arguments but 4 were given

Hi ! Thanks for this very helpful library!
I use websockets inside python classes. And when I want to use the "on_close" method, I have this error :
on_close() takes 2 positional arguments but 4 were given
This is my code :
def on_close(self, ws):
    print("Closed")

self.node_ws = websocket.WebSocketApp(self.node_ws_url,
    on_message=self.on_message,
    on_close=self.on_close,
    on_error=self.on_error,
    on_open=self.on_open)
self.node_ws.run_forever()

Maybe the problem went from the self argument?
Thanks for your reply!
Eliot

You are referring to another library. You filed this issue against the wrong repository.


i am not getting custom notification that app is background and closed

Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.

If you are filing an issue about FCM in the background make sure to read #4 and #89 first!

For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: _____
Android OS version: _____
Google Play Services version: _____
Firebase/Play Services SDK version: _____

Step 3: Describe the problem:
Steps to reproduce:

Observed Results:

What happened?  This could be a description, logcat output, etc.

Expected Results:

What did you expect to happen?

Relevant Code:
// TODO(you): code here to reproduce the problem

@Reenaefgh in the issue template you can see that this is discussed in #4 and #89.  As you did not provide any indication that your issue is different, I am going to close this and direct you there.


Add new command /usage to get last 30 days api usage in $

New command "/usage".
Calls the non documented API dashboard/billing API to get the usage, in dollars, of the last 30 days.
Example response:
Usage in the last 30 days: 0.42 $

Still WIP. I'm not sure if the value taken is the cost in dollars.

It seems to be a reasonable option for user to monitor the cost. @flynnoct  Do you want to add this new command? Or maybe you can add it in the logging system, simply add a command named 'usage' in Telegram feels user-unfriendly.


feat: medici 5 (with cache)

also: mongoose 6 (with types)

[x]  still need to be done: reactivate unit tests

@krtk6160 @sandipndev  this branch is also ready to be merged. can you work on preparing the upgrade of the helm chart for mongodb?

@dolcalmi btw, I had deactivate those tests, they will need to be reactivated, ie: test/unit/services/ledger/domain/entry-builder.spec.ts.skip

good to merge for me


Endpoint type merge

Merge current master into Dominic Wörner's PR#589 (which became #634) endpoint-type work en route to merging back into master, unit test coverage

Merges current master into https://github.com/hyperledger/aries-cloudagent-python/pull/634 in preparation for merge of this (endpoint-type) work back into aca-py master.

Thanks!

Is there a need to allow for provisioning with different endpoint types? Since only endpoint_type==EndpointType.ENDPOINT affects the wallet, I am guessing that there is no such need?

This could make sense. In particular if you want to do all ledger operations during provisioning

Is there a need to allow for provisioning with different endpoint types? Since only endpoint_type==EndpointType.ENDPOINT affects the wallet, I am guessing that there is no such need?

This could make sense. In particular if you want to do all ledger operations during provisioning

I think that could be separate PR though. This one introduces a significant change on its own. I can get on it once it's merged.


Some corrections to the Metrics API YAML and MD file

Description

Urgency

[ ] Blocker
[ x] High
[ ] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

@darkonie @philipnrmn This PR fixes the Metrics YAML so that it includes tags and also fixes the red ERROR that was showing up at the bottom. Let me know if you see any issues with these changes. This renders very well for me locally. I will also submit a PR against dcos-metrics and try to sync these two YAML sources. Thanks.

We are going to go ahead and merge this so that we can get these fixes up on the dcos.io and EE sites. These fixes are primarily formatting-related, plus the correction to the path. However, should you have any comments on this PR post-merge, please feel free to add them and I will address. Thanks!

LGTM


HBASE-22484 Javadoc Warnings: Fix warnings coming due to @result tag i…

…n TestCoprocessorWhitelistMasterObserver

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
45
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
0
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1
mvninstall
277
master passed

+1
compile
59
master passed

+1
checkstyle
75
master passed

+1
shadedjars
281
branch has no errors when building our shaded downstream artifacts.

+1
findbugs
223
master passed

+1
javadoc
37
master passed

_ Patch Compile Tests _

+1
mvninstall
259
the patch passed

+1
compile
58
the patch passed

+1
javac
58
the patch passed

+1
checkstyle
79
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
289
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
1050
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.0.3 3.1.2.

+1
findbugs
182
the patch passed

+1
javadoc
34
the patch passed

_ Other Tests _

-1
unit
16935
hbase-server in the patch failed.

+1
asflicense
39
The patch does not generate ASF License warnings.

20276

Reason
Tests

Failed junit tests
hadoop.hbase.client.TestFromClientSide

hadoop.hbase.client.TestFromClientSide3

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-256/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/256

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 0ab802694ff4 3.13.0-153-generic #203-Ubuntu SMP Thu Jun 14 08:52:28 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / b4dc778c3b

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-256/1/artifact/out/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-256/1/testReport/

Max. process+thread count
4955 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-256/1/console

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.


GitSavvy: submit last failed git command as issue

For debugging purposes (particularly for those submitting bugs), store the stdin and stdout of the most recent failed git operation.  Add command to open web browser with URL along the these lines:
https://github.com/divmain/GitSavvy/issues/new?title=failed%20git%20command%20X&body=some%0Ainput%0Aand%0Aoutput

Unable to automatically open new GitHub issue with log text, due to query param length limits.


result's ptr is not correct for some numbers

This fails:
string_view s = "0e0";
auto err = detail::charconv::from_chars( s.begin(), s.end(), d );
BOOST_ASSERT( err.ptr == s.end() );

Same for e.g. 0e00000000000 and 0e1.

This looks to be fixed by the change to fast_float.


Update logo to better quality

Haha thanks, but I actually wanted the "logo" to be like it currently is. It took too much space in the readme and I might actually remove the "logo" from there in the near future when the new website and documentation is finished. On the website will be a place for a higher resolution "logo" ;)


Fix the wrong type of Invul for withers

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

npm package (@latest dist-tag)
Visual Studio Marketplace
GitHub release

Your semantic-release bot :package::rocket:


[release/1.6] Add stable ABI support in windows platform matcher + update hcsshim tag

This PR does the following:

Update hcsshim tag to 0.9.10 + go mod tidy/vendor

Uses the function introduced by https://github.com/microsoft/hcsshim/pull/1843 to invoke stable ABI compliant function in windows platform matcher. (In this PR, this change is limited to platforms/default_windows.go file. platform/default_windows_test.go file has some new tests added).
Exact platform match is required for OS Versions < WS2022. The exact version match is not needed in versions > WS2022 due to stable ABI compliance.

@jterry75 @dmcgowan @dcantah  could you take a look when you have some time please? Thanks a lot!

@kiashok Do we want this in 1.7 also?

@kiashok Do we want this in 1.7 also?

Yes we do. I already have a PR out in containerd/main which are hitting a race condition in transfer service which are unrelated to any of the windows platform matcher code changes being made. That is the reason I submitted a separate PR to containerd/1.6 as it does not have transfer service code.

@kiashok ack

@jterry75 @dmcgowan @mikebrow could you take a look when you have some time please?

LGTM

Hi @mikebrow , could this PR be merged? There are two sign offs and no open questions. Thanks!

LGTM

Hi @mikebrow , could this PR be merged? There are two sign offs and no open questions. Thanks!

Hi @kevpar , could you please help merge this PR if everything looks good to you? Thanks!

@kiashok will you be preparing the cherry pick for 1.7

@kiashok will you be preparing the cherry pick for 1.7

Thanks for merging the PR @mikebrow . Yes I will be submitting changes for 1.7 as well (currently blocked by the discussion in this PR https://github.com/containerd/containerd/pull/8903 )


Travis Checks not running!

We have made several pull requests over the last hour and there is no indication that the Travis checks are running.

Seems that they are slow, but have recovered.

Sorry - we need to re-open this ticket.
The Travis checks don't seem to be starting now.

I think it's a transient Travis problem: https://www.traviscistatus.com

We are currently seeing a backlog of public/open source builds on all our infrastructures. Apologies for the inconveniences.

Okay, thanks for the update and Link, @balhoff


Act on SEO Audit

I pulled out SEO audit tasks into a private board here. @cgettings, I'm opening up a branch hotfix-SEO-audit to start to work on these changes. I'm finding a few things that confuse me, so might tag you in here for some conversation and thoughts about them. strikethroughs for things that are NA.
Meta

[x] Pages without titles
Pages with multiple titles
Pages with duplicate titles
[x] Duplicate meta descriptions

High priority

[ ] Incorrect region/language codes
[x] Duplicate titles
Content exact duplicates (exclusively things on IndicatorPublic not /beta)
[x] 4xx error
[x] Broken internal links
[x] Broken external links
Broken internal images

medium priority

[x] H1 missing

low priority

[x] Image missing alt
[ ] URL multiple slashes

@cgettings - re: the SEO audit's findings on "incorrect regions."  ML reported:

Incorrect Language & Region Codes
Description: URLs with hreflang annotations that contain invalid language (in ISO 639-1 format) and or optional regional (in ISO 3166-1 Alpha 2 format) values. Unsupported hreflang values can be viewed in the lower window 'URL Details' pane with an 'invalid' status.
How To Fix: Update hreflang annotations to include valid language and regional values, so they can be used for geotargeting the right audiences appropriately.

It flagged a bunch of pages with translations (see file here).
I think the problem is that we use cn for Chinese, but the standard is zh.
cn comes from .Language.Lang in head.html (see below), so I assume that Hugo is saying that Chinese = cn. Is this an issue, do you think, or just the file extensions we've used? Eg, index.cn.md instead of index.zh.md? Plus, in config.toml, we specify languages.cn (also mentioned in the Gruntfile, which we'd have to edit).
{{/*  content translations  */}}

{{- if .IsTranslated -}}
	{{- range .Translations -}}
	<link rel="alternate" hreflang="{{ .Language.Lang }}" href="{{ .RelPermalink }}" title="{{ .Language.LanguageName }}">
	{{- end -}}
{{- end -}}

Anyway: I think the play is to update the translated filenames, update config, and update the Gruntfile. Before I make these changes, wanted to see if this sounds right to you, or if you know anything else about this.

Fixed broken links, but the report flagged https://a858-nycnotify.nyc.gov/notifynyc/ (in the footer) as a problem - though it (now) goes to a real page.

re: languages -
One strange thing is the performance of that if .IsTranslated block.
For example, the home page doesn't have translations, but in head:
<link rel="alternate" hreflang="es" href="https://a816-dohbesp.nyc.gov/IndicatorPublic/beta/es/" title="Español"><link rel="alternate" hreflang="cn" href="https://a816-dohbesp.nyc.gov/IndicatorPublic/beta/cn/" title="中文">

but, a similar block on data-stories/single.html correctly runs .IsTranslated, so I'm a bit confused. Going to dig further.

Taking a look at Google Search Console, I think there's some work we can do in instances where we have pages that do hashing (like the HVI and AQE - and even the Data Explorer?) to set the canonical. But I'm not 100% clear on this so don't want to go off half-cocked (yet).


Add numazu-vtc.ac.jp domain

Numazu Vocational Training Center
Website: http://numazu-vtc.ac.jp/

@yasunobk Could you please clarify how long a typical IT-realted course in this training center takes? Is it a month, 3 months, 12 months?

@philipto We have a 2-year InformationTechnology course.
Application Guidlines Page: http://www.numazu-vtc.ac.jp/nyukou/index.html
Unfortunately, we don't have English pages.

@yasunobk Thank you for the clarification. Pull request merged.


Mapping platform 'redhat' to 'rhel' for redhat platform

Default yum repo of mariadb on http://yum.mariadb.org/ uses 'rhel' for redhat platform, for example: http://yum.mariadb.org/10.0/rhel7-amd64/ but ohai returns node['platform'] as 'redhat'. Needs to do the mapping in repository.rb
Fix some rubocop issues.

Everything works, thx to you for this PR !


Changing chunk size, requires re-training?

Hi, I desire in changing the "action_chunk_size" do you think it may require a massive re-training?
Thanks!

The weights, except for x_pos_embed, can be loaded as usual. Changing the action_chunk_size may require a bit more training than usual, but it shouldn’t necessitate a full retraining. However, it’s recommended to fine-tune the model for a bit longer with the new configuration to ensure the best possible performance.


[Fix] Query Hangs if Connection is Closed Mid-Query

See the test for how to trigger this.

Cancelled the integration tests CI manually, since they wouldn't have finished.
Side note @gwynne: probably good to have a time limit in CIs.

Side note @gwynne: probably good to have a time limit in CIs.

The default timeout is 1 hour, IIRC.

the test I added as an example hangs 100% of time for me.

the test I added as an example hangs 100% of time for me.
Hmm you're right. I thought i tested that properly and it turned out like i said, but apparently not.
I did try to remove the like you did with just closing the channel before hand, but i thought that didn't work out to get the hang.

ahh hmm CI is failing with Fatal error: SWIFT TASK CONTINUATION MISUSE: listen(_:) tried to resume its continuation more than once, throwing ioOnClosedChannel! need to investigate.

I reckon I didn't add a unit test for "each" modified function. Should I?

I reckon I didn't add a unit test for "each" modified function. Should I?

Yes, please!


RTPS: auto generate yaml file

The RTPS yaml file has to be manually updated for each new uorb message, which is a bit annoying because

This causes CI failures when you forget to update it
This causes many merge conflicts

Can it be automatically generated?

This has been discussed before and the answer is no, because you want to keep consistency on the ID's that are set to each message. The same way you don't automatically generate new ID's for the Mavlink messages.

What about a unique hash of the message name, or of the message content?

What about a unique hash of the message name, or of the message content?

That was also spoken and an 8-bit hash would result in collisions. And it can't be higher (to include the content), because the ID field has limited size (0-255).

That is unfortunate. We are already approaching the limit of 150 IDs reserved for non-multitopic messages, are there plans for when that limit is reached?

That is unfortunate. We are already approaching the limit of 150 IDs reserved for non-multitopic messages, are there plans for when that limit is reached?

That limit can be increased. At least for now at development stage we can do that. When the microRTPS starts to be deployed in end systems, then that will be an issue.

then that will be an issue.

Sure. MAVLink 1 had the same problem: it was designed with a 255 messages limit, which probably seemed plenty at the time, but was reached since.
We are maybe going further than necessary to solve a mild inconvenience :-). There may be a better solution than a unique hash. But assuming we want a hash, and thus need a larger ID field, is that possible? Is the 8-bit limit imposed by the RTPS protocol, or by PX4's implementation?

We are maybe going further than necessary to solve a mild inconvenience :-). There may be a better solution than a unique hash. But assuming we want a hash, and thus need a larger ID field, is that possible? Is the 8-bit limit imposed by the RTPS protocol, or by PX4's implementation?

As far as I know that's a protocol limitation (specific of the microCDR implementation).

@jlecoeur but I don't actually see ourselves requiring more than 256 msgs for internal consumption and onboard communication. A scaleble system must also keep a certain level of consistency and simplicity on the way it exchanges it's data, and have control on what messages are needed and not. Sometimes, it just makes sense to add a specific field to an existent message than just create a new message for the same purpose.

Absolutely, that brings the question wether all uORB messages should be mapped to RTPS, or just a selected, stable, few.

Absolutely, that brings the question wether all uORB messages should be mapped to RTPS, or just a selected, stable, few.

Not all will be. But we are not at the stage of doing that selection, at least for now.

There is a significant difference to MAVLInk in that we’re controlling both sides here (forced by tight integration). We will also need some sort of filter to ensure security and safety of the system for inbound and outbound messages. Otherwise a debut message in ROS can bring the flight controller down via a DDS-like action.

We will also need some sort of filter to ensure security and safety of the system for inbound and outbound messages. Otherwise a debut message in ROS can bring the flight controller down via a DDS-like action.

@LorenzMeier we may explore the security options that Fast-RTPS offers for that same purpose. FastRTPS implements security at three levels: access control of entities, encryption of data and authentication of remote participants.


[BUG] Actions for projects visible to users

Describe the bug
Buttons to edit / suspend / delete / add users allowed to a regular user of a project. Should be hidden unless the uesr is a project admin

Critical added, while no functionality is broken users shouldn't see button they're not authorised to press/see.

cc @Pearl1594 does your project-roles PR cover this - let's discuss thanks.

With @Pearl1594 - as discussed please embed the two components and use this.$set see the Instance Tab component for example

Fixed in https://github.com/apache/cloudstack-primate/pull/382


Update blogs.json

Added lucasvandongen.dev

Thanks Lucas!
I couldn't find the blog on your site, is there one? I also notice there's no data in the RSS feed over and above the structural XML.
I guess it's fine to go in the index if that's intentional, it's just not a situation I've seen before. Most (if not all) of the traffic this site will drive is through blog posts in the RSS feed as the RSS feeds form the basis of several aggregation sites.

Whoops, it wasn't showing any of them. Now it's working.
I don't use any frameworks, just a bunch of PHP includes, so the set up is a bit simple. The home page does contain a list of articles. I could make a /blog/ directory in the future, but at the moment I would also blog about anything else than iOS / macOS / Swift so people could then filter on category.

Ahh, I see the blog posts now. They were pretty well hidden 😂
Thanks Lucas!


Systemd persistence

Can one of the admins verify this patch?

Yes sorry my bad.  Pushed this out very quickly.
On Sat, Apr 22, 2017, 8:34 AM Florin Sarbu<EMAIL_ADDRESS>wrote:

@floion commented on this pull request.
In meta-resin-common/recipes-core/systemd/systemd_%.bbappend
https://github.com/resin-os/meta-resin/pull/650#discussion_r112804032:

@@ -37,6 +38,12 @@ do_install_append() {
install -d -m 0755 ${D}/${sysconfdir}/systemd/system.conf.d
install -m 0644 ${WORKDIR}/watchdog.conf ${D}/${sysconfdir}/systemd/system.conf.d

if ${@bb.utils.contains('DISTRO_FEATURES','development-image','false','true',d)}; then

   # enable watchdog

Wrong comment I assume?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/resin-os/meta-resin/pull/650#pullrequestreview-34151752,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGPpeNEJnrFwqKMh6UPnRDZ438QZH3iks5ryZ9dgaJpZM4NEFT2
.

Could you add a CHANGELOG entry?

According to an internal talk, we do not require CHANGELOG for external contributions. Ignore that comment please.

Sorry, what are you waiting on?
On Mon, May 1, 2017 at 11:54 AM, Andrei Gherzan<EMAIL_ADDRESS>wrote:

@agherzan commented on this pull request.
In meta-resin-common/recipes-core/systemd/systemd/coredump.conf
https://github.com/resin-os/meta-resin/pull/650#discussion_r114113346:

@@ -0,0 +1,2 @@
+[Coredump]
+Storage=none

Friendly ping on this one @linux4kix https://github.com/linux4kix
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/resin-os/meta-resin/pull/650#discussion_r114113346,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGPpclESZ9MfwizMsxiQFJnDwRS7dqgks5r1avmgaJpZM4NEFT2
.

Merged here with some changes: https://github.com/resin-os/meta-resin/pull/662


Trying to run in command-line window

When opening the command line window with q: the following error is printed:
Error executing vim.schedule lua callback: .../.config/nvim/plugged/diagnostic-nvim/lua/diagnostic.lua:74: Vim(lexpr):E11: Invalid in command-line window; <CR> executes, CTRL-C quits
diagnostic-nvim is being loaded using on_attach in my configuration, but this error occurs even when I haven't loaded a file yet.
diagnostic-nvim version is latest from git.
nvim version:
NVIM v0.5.0-523-g04a0486c6
Build type: Debug
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -g -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fno-common -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DMIN_LOG_LEVEL=1 -I/home/gecko/build/neovim/build/config -I/home/gecko/build/neovim/src -I/home/gecko/build/neovim/.deps/usr/include -I/usr/include -I/home/gecko/build/neovim/build/src/nvim/auto -I/home/gecko/build/neovim/build/include
Compiled by gecko@mortred

Features: +acl +iconv +tui
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/share/nvim"

Run :checkhealth for more info

It should be fixed now, please update and checked:)

Works, thank you!


OmniSharp log should display version of OmniSharp that is running

This will help diagnose issues.

This should be handled with https://github.com/OmniSharp/omnisharp-roslyn/issues/757.


new Paper-only event player opens sign

What's the reason for this event? Did somebody request it? What reason would you use this event that other events don't cover?

What's the reason for this event? Did somebody request it? What reason would you use this event that other events don't cover?

oh. i forgot to mention. this event requested from here https://discord.com/channels/315163488085475337/1136954189646401557


embed image / background image

would be a cool customization feature

+1

+1

this is a minimalist module, this is not in my consideration.


Fuse Mountoptions

It should be possible to specify fuse mountoptions like allow_other

Already implemented, will be released in next milestone.


[BUG] Error: nextjs-node-loader: Module did not self-register: robotjs

Describe the bug
A clear and concise description of what the bug is.
robotjs did compiled for node successfully and at the first page compilation it works flawlessly but as i change something in my react sourcecode (server or client side), page recompiles and throws the error multiple times
Error: nextjs-node-loader:
Error: Module did not self-register: '.../root_directory/.next/server/robotjs.node'.
...
at async Server.requestListener (.../root_directory/node_modules/next/dist/server/lib/start-server.js:141:13) {
  digest: '246830862',
  page: '/0/0'
}

as i resave the next.config.js (apparently the whole project restarts) and everything starts to function again until the next hot reload
Which version of Next.js?
"next": "^14.2.11",
Which version of nextjs-node-loader?
"nextjs-node-loader": "^1.1.5",
To Reproduce
Steps to reproduce the behavior or even better. Share me a link to repository with reproducable:
see the bug description

thanks for help! your reply was so quick <3
p.s. in case somebody waisted 26 hours on debugging as well... my solution was to downgrade next from 14.2.11 to 14.2.3

I'm sorry I didn't respond at time, but I don't appreciate the sarcastic tone in your last reply.
You have to realize that I'm the sole maintainer of this repository and that you might not get an instant reply.
I try my best to keep up with issues, but with the volume of notifications and my other responsibilities, some things can get missed. I appreciate your patience and thank you for sharing your workaround.
I also don't know when I'll have time to take a better look, you're welcome to contribute to this repository.

I just want to add that I had the exact same issue with multiple modules (keytar, msnodesqlv8, etc.), downgrading to next 14.2.3 (didn't try other versions) fixed this issue, seems that something changed in the intervening versions that breaks nextjs-node-loader
Just wanted to add a big thank you to @amarjanica for maintaining the repo

You're welcome, @lsafd. I'll try to make time by the end of the week to look into this and I'll report here what I find. If anyone else wants to give it a try in the meantime, feel free to jump in

I believe this is related to #45 , but never got a response back. Something in webpack changed. What previously worked as:
    config.module.rules.push({
      test: /\.node$/,
      use: [
        {
          loader: 'nextjs-node-loader',
          options: {
            flags: os.constants.dlopen.RTLD_NOW,
            outputPath: config.output.path
          },
        },
      ],
    });

is now a
 config.module.rules.push({
      test: /.+.node$/ig,
      use: [
        {
          loader: 'nextjs-node-loader',
          options: {
            flags: os.constants.dlopen.RTLD_NOW,
            outputPath: config.output.path
          },
        },
      ],
    });

Note the regex pattern change. Please let me know if it works for you @lsafd and @antonbrams

@lsafd @antonbrams try the 1.1.8 version. I don't know exactly what was the issue, but bumped all the dependencies and updated my example/next-14. when you run it, go to /api/robotjs, you should get a mouse position response. also added next-14 example build to the ci, to avoid "it works on my machine".
dunno, maybe the dependencies bump helped

@amarjanica Let me know if I should create a new issue, but I am still having issues with msnodesqlv8. Here is a repo that should make recreating it easy https://github.com/lsafd/nextjs-node-loader-msnodesqlv8 (just load / on different versions of next).
This has taken me down a bit of a rabbit hole. I will say this is an area where I have little expertise, but I tried the following https://github.com/TimelordUK/todo-with-nextjs_msnodesqlv8 which works fine (without even using nextjs-node-loader). I then tried to copy the setup with the app router where doing a require or import failed. Could you explain / link me to some materials about why this is the case? I find this topic very interesting but am struggling to find documentation for changes.

Thanks for sharing the repo. Little background on this loader:
This is a plugin which calls process.dlopen, which is a low-level function in Node.js used to load and dynamically link native modules. Plugin was adapted from https://github.com/webpack-contrib/node-loader because a solution is specific to nextjs and how it handles assets.
Key is to get process.dlopen work with the correct path to the native module.
You got it, this is broken in 14.2.14.
It's hard to pinpoint where's the issue, but when debugging it, take a look at the require part in .next/server/app/api/your_route_that_requests_native_module/route.js.
In previous versions, path to native module was correct.
In >14.2.14, loader works up to a point that it puts .node modules in .next/server/chunks and I can also console.log the correct require path to native modules. But the route.js require path is wrong.
Currently I have zero idea except that I'm getting suspicious that some other webpack loader is causing this.
Zero idea

hey @lsafd and @antonbrams, solution is to add:
  experimental: {
    serverComponentsExternalPackages: ["yournodepackage"],
  },

A similar issue was https://github.com/yisibl/resvg-js/issues/315#issuecomment-2395408012
I guess there's no longer need for this loader.

Works for me! Appreciate the help.

I'm closing the issue as completed. Solution for a module built with neon bindings is slightly different. Instructions are in Readme.md. Thanks for reporting the issue


Map Guide text

Draft language for Map Guide pop up for Ebola visualizations

@ksewalk
Welcome to the Global.health Ebola map!
These geospatial data visualizations allow you to explore data for the 2022 Ebola outbreak in Uganda. Further expansion is possible and our team will continue to track cases through the first 100 days.
Country View: Explore data at the country level.
Regional View: Explore data at the sub-national level.

@aimeehan1 looks good!

@maciej-zarzeczny re-assigned to you for implementation of the text above!

@ksewalk @aimeehan1 I've added the text, it's now deployed on DEV: https://dev-map.ebola.global.health


Added support for go mods

The import will change to go.bug.st/serial, the old one (go.bug.st/serial.v1) will continue to work but will not receive further updates.
Fix #49

Manually merged.


Bug with key words?

Hi! I have this language.yaml

command:
  setSpawnpoint: "spawnpoint"
  teleport: "telepor"
  toggleMsg:
    on: "message enable"
    off: "message disable"

When i try get command.toggleMsg.on or command.toggleMsg.off i have null (in this case Missing language key - [command, toggleMsg, off]).
Code
configuration.get().node("command", "toggleMsg", "off").getString("&cMissing language key - &e" + Arrays.toString(key))

It is bug or not?

This is probably due to the yaml spec treating 'on' and 'off' as boolean values in 1.1. It's a known quirk.


Update scala-library to 2.13.7

Updates org.scala-lang:scala-library from 2.13.5 to 2.13.7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (2.13.5).
You might want to review and update them manually.
.github/workflows/ci.yml

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

labels: library-update, early-semver-patch, semver-spec-patch, old-version-remains

Superseded by #91.


Nested session

I am looking for a nice way to switch from a local and a remote session (the remote session is typically nested in the local). Prefix + prefix is a bit annoying!
Here are some examples, gist.
Do you plan to add such features?
Thanks

Hey @guhur 👋
No plan for such features but I'll think about it.
I have many other devs in flight that I want to merge first though.
I'll keep this issue open in the meantime.

Ok great! I am working usually on three different remote servers, so being able to jump easily from one session to the other one would be a real time saver for me!

Well in the meantime you can still define your own bindings in your ~/.tmux.conf.local copy

Do you recommend using a different prefix for the nested tmux session?

Do you recommend using a different prefix for the nested tmux session?

Hello @ajing 👋
I personally hit the prefix key twice when I want the nested session to receive the prefix

Hello @ajing 👋
I personally hit the prefix key twice when I want the nested session to receive the prefix

I just use different prefix keys, but if you don't like that approach you could try using the tmux-suspend plugin.
If you use the same prefix, load the plugin locally (not on the remote instance) then simply press F8 and suspend actions on the local instance.


[container-runtime-security-falco] Update lab

What type of PR is this?
/kind bug
/kind content
Any specific area of the project related to this PR?
/area lab: container runtime security with falco (falco)
What this PR does / why we need it:
Updates existing Katacoda content that was outdated and not working properly.
Signed-off-by: Pablo Musa<EMAIL_ADDRESS>

@pmusa: The label(s) area/lab:, area/container, area/runtime, area/security, area/with, area/falco, area/(falco) cannot be applied, because the repository doesn't have them.

In response to this:

What type of PR is this?
/kind bug
/kind content
Any specific area of the project related to this PR?
/area lab: container runtime security with falco (falco)
What this PR does / why we need it:
Updates existing Katacoda content that was outdated and not working properly.
Signed-off-by: Pablo Musa<EMAIL_ADDRESS>
Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: pmusa
To complete the pull request process, please assign developer-guy after the PR has been reviewed.
You can assign the PR to them by writing /assign @developer-guy in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

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

Welcome @pmusa! It looks like this is your first PR to falcosecurity/katacoda-scenarios 🎉

[APPROVALNOTIFIER] This PR is APPROVED
Approval requirements bypassed by manually added approval.
This pull-request has been approved by: pmusa
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

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

LGTM, thanks for the PR @pmusa.

@developer-guy I have checked the changes included in this PR and it lgtm. The biggest change is that now this lab uses the latest falco_rules.yaml for its practice. It also fixes a number of contents of the lab that made it unusable or outdated.
If you are fine with this changes too, can you approve? Thank you!

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: pabloopez, pmusa
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [pabloopez]

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


Publicly readable feature list?

I don't want to give my personal information to two different third-party companies (the forum website and one of its two unsavory login providers) just to find out what's even in this mod. I hope you'll consider putting at least a minimal feature overview somewhere that everyone can read it.

I did not switch to Twitch and that probably deleted all of my content on the forums. Not sure why it wants you to log in when ít's likely dead, anyway. You can check out this archived version of the thread, though. Some features are also detailed on the FTB Wiki page on it.

Oh nice, that helps. Thank you!


Fix Crash on Sort By Message

Closes: #1591
Adds the Available type back into the computeMessage function!

Hopefully this PR resolves, but FYI running latest v2 this morning with make cluster-dev navigating to Flux Runtime view results in the same error message, after this it persists across ALL tables and you have to clear your cache before it will work again

Duplicate of #1645 - closing


How to show inline form as table?

How to show inline form as table?

 How to show inline form as table?


No audio and sporadic playback

I did a stock install with this and I get absolutely no audio via HDMI. I tried two different TVs and multiple HDMI cables. Also noticed when playing some videos through YouTube it takes a while and sometimes doesn't even play and I have to reboot.
This is with the latest July 24th image. I will test a previous one just in case.

There is an issue with the HDMI connector, using a USB-C to HDMI cable resolved this. Seems like the USB-C connector for video is getting more love than the HDMI :)


2.0.0-rc.3 - unknown fields in alertmanager config: static_config"

I am aware of how to fix this problem, but these are the settings in the "Default" configuration.
What did you do?
wget https://github.com/prometheus/prometheus/releases/download/v2.0.0-rc.3/prometheus-2.0.0-rc.3.linux-amd64.tar.gz
tar xf prometheus-2.0.0-rc.3.linux-amd64.tar.gz
cd prometheus-2.0.0-rc.3.linux-amd64
./prometheus --config.file="prometheus.yml"

What did you see instead? Under which circumstances?
level=info ts=2017-11-08T06:30:18.281802915Z caller=main.go:215 msg="Starting Prometheus" version="(version=2.0.0-rc.3, branch=HEAD, revision=8a9b32d0eb5be05a8066492e8a75c18b90e2d6ff)"
level=info ts=2017-11-08T06:30:18.281898923Z caller=main.go:216 build_context="(go=go1.9.2, user=root@f25dd60e77b3, date=20171104-20:31:56)"
level=info ts=2017-11-08T06:30:18.281946227Z caller=main.go:217 host_details="(Linux 4.13.11-1-ARCH #1 SMP PREEMPT Thu Nov 2 10:25:56 CET 2017 x86_64 XXX (none))"
level=info ts=2017-11-08T06:30:18.284090913Z caller=web.go:380 component=web msg="Start listening for connections" address=<IP_ADDRESS>:9090
level=info ts=2017-11-08T06:30:18.284091223Z caller=main.go:314 msg="Starting TSDB"
level=info ts=2017-11-08T06:30:18.284124087Z caller=targetmanager.go:69 component="target manager" msg="Starting target manager..."
level=info ts=2017-11-08T06:30:18.29104953Z caller=main.go:326 msg="TSDB started"
level=info ts=2017-11-08T06:30:18.29111071Z caller=main.go:394 msg="Loading configuration file" filename=prometheus.yml
level=error ts=2017-11-08T06:30:18.291462737Z caller=main.go:356 msg="Error loading config" err="couldn't load configuration (--config.file=prometheus.yml): unknown fields in alertmanager config: static_config"

Environment

System information:
Linux 4.13.11-1-ARCH x86_64

Prometheus version:
2.0.0-rc.3

Prometheus configuration file: Default!

# my global config
global:
  scrape_interval:     15s # Set the scrape interval to every 15 seconds. Default is every 1 minute.
  evaluation_interval: 15s # Evaluate rules every 15 seconds. The default is every 1 minute.
  # scrape_timeout is set to the global default (10s).

  # Attach these labels to any time series or alerts when communicating with
  # external systems (federation, remote storage, Alertmanager).
  external_labels:
      monitor: 'codelab-monitor'

# Alertmanager configuration
alerting:
  alertmanagers:
  - static_config:
    - targets:
      # - alertmanager:9093

# Load rules once and periodically evaluate them according to the global 'evaluation_interval'.
rule_files:
  # - "first_rules.yml"
  # - "second_rules.yml"

# A scrape configuration containing exactly one endpoint to scrape:
# Here it's Prometheus itself.
scrape_configs:
  # The job name is added as a label `job=<job_name>` to any timeseries scraped from this config.
  - job_name: 'prometheus'

    # metrics_path defaults to '/metrics'
    # scheme defaults to 'http'.

    static_configs:
      - targets: ['localhost:9090']

Fixed in #3415 already.


Unable to use XamlReader.Load unless type is already used in a XAML file

Describe the bug
It's not possible to use XamlReader to parse XAML in an application if the types being parsed aren't already included in a XAML file that was compiled into the app.
This may seem like a very niche case but it is very problematic for some testing and library scenarios.
Steps to reproduce the bug

Create a new "Blank App, Packaged (WinUI 3 in Desktop)" and call it XamlReaderRepo
Update all NuGet packages to the latest versions (v1.1.0 of Microsoft.WindowsAppSDK)
Add this class to the project:

    public class MyCoolElement : FrameworkElement
    {
    }

In MainWindow.xaml.cs, replace the event handler with the following:

        private void myButton_Click(object sender, RoutedEventArgs e)
        {
            var pg = Microsoft.UI.Xaml.Markup.XamlReader.Load(@"<Page
                    xmlns=""http://schemas.microsoft.com/winfx/2006/xaml/presentation""
                    xmlns:x=""http://schemas.microsoft.com/winfx/2006/xaml""
                    xmlns:local=""using:XamlReaderRepo"">
                    <local:MyCoolElement />
                </Page>") as FrameworkElement;
            myButton.Content = pg.ToString();
        }

Run the app, click the button, see it throws a XamlParseException.
In MainWindow.xaml, add the following immediately under the Button but above the closing StackPanel tag:

<local:MyCoolElement />

Run the app again, click the button, see the text changes.

Expected behavior
It should be possible to parse a type from a string without having to use it in a XAML file first.
I assume this is failing as some type information is being generated at compile time for everything that is in a .xaml file. Is there a way to trigger this for types not defined in xaml and compiled into the assembly?
Is there some metadata or attributes that can be added to a type definition to enable this? Or some way of specifying namespaces, or assemblies that contain types that the code might want to load as a XAML string.
Based on https://github.com/microsoft/microsoft-ui-xaml/issues/4161#issuecomment-777911468 I thought it would be possible to put a [Windows.UI.Xaml.Data.Bindable] attribute on MyCoolElement to enable the expected behavior it does not.
Screenshots
No response
NuGet package version
WinUI 3 - Windows App SDK 1.0.3
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
Issue discovered as part of work for Windows Community Toolkit. ;)
Possibly also connected to https://github.com/microsoft/microsoft-ui-xaml/issues/6299

I would imagine this is currently by design as xaml compiler needs to generate xaml type info for non-framework types (such as anything under local:). Perhaps something C#-specific could be done to reify the xamltypeinfo at runtime but I don't believe anything like this exists today. @RealTommyKlein to keep me honest.

.

This is still reproducible on:

v1.6.241114003
v1.7.241114004-experimental1


Is there any open source code for tsbs? 请问是否有tsbs的开源代码?

Describe your question
Hello,
I discovered that TSBS is used for performance testing on https://docs.opengemini.org/zh/guide/introduction/performance.html. However, it seems that the specific code for using TSBS for testing is not provided. Is it because TSBS can be directly used for testing without any modifications to the code? If modifications are necessary, is there any open-source code available for these tests?
Thank you!
您好
我在 https://docs.opengemini.org/zh/guide/introduction/performance.html
发现了有使用tsbs作为性能测试, 但好像没找到使用tsbs的测试代码?
请问是因为可以直接使用tsbs来测试而不需要改代码吗?
如果需要改代码 ,是否有开源测试的代码?
谢谢!

The openGemini is compatible with the InfluxDB syntax and APIs. Therefore, we do not develop the tsbs test tool of the openGemini. Instead, we use the tsbs test program of the InfluxDB to perform the performance test.
openGemini 兼容 InfluxDB 语法和接口API，因此我们并没有单独开发openGemini的tsbs测试工具，而是直接使用Influxdb的tsbs测试程序进行的性能测试。

好的 谢谢


Updated Mediatype description for pubmatic Adapter

Pubmatic Adapter Supports Banner, video and native. Updated the media_types in documentation.

@pm-shashank-jain - is this related to an existing Prebid.js PR or is it functionality that already exists and you're just catching the docs up to the code?

Pubmatic Adapter already supports all three media types. I've just updated the docs for the existing code.


[BUG]: EXPO 빌드 관련

Is there an existing issue for this?

[X] I have searched the existing issues

Version of @mj-studio/react-native-naver-map
2.1.1
Version of react-native
expo 51.0.26
What os are you seeing the problem on?
Android
What device types are you seeing the problem on?
Simulator
What architecture types of react native are you seeing the problem on?
Old Architecture(Bridge)
Version of device(android API, iOS OS version, etc...)
Android 34
Expo App

[X] I am using Expo

What happened?
expo생성 후 해당 라이브러리 적용 "RNCNaverMapView" was not found in the UIManager. 에러가 나타나고 있습니다.
Relevant a package.json.
{
  "name": "golden-expo-app",
  "main": "expo-router/entry",
  "version": "1.0.0",
  "scripts": {
    "start": "expo start",
    "reset-project": "node ./scripts/reset-project.js",
    "android": "expo start --android",
    "ios": "expo start --ios",
    "web": "expo start --web",
    "test": "jest --watchAll",
    "lint": "expo lint"
  },
  "jest": {
    "preset": "jest-expo"
  },
  "dependencies": {
    "@expo/vector-icons": "^14.0.2",
    "@mj-studio/react-native-naver-map": "^2.1.1",
    "@react-navigation/native": "^6.0.2",
    "expo": "~51.0.26",
    "expo-build-properties": "~0.12.5",
    "expo-constants": "~16.0.2",
    "expo-font": "~12.0.9",
    "expo-linking": "~6.3.1",
    "expo-location": "~17.0.1",
    "expo-router": "~3.5.21",
    "expo-splash-screen": "~0.27.5",
    "expo-status-bar": "~1.12.1",
    "expo-system-ui": "~3.0.7",
    "expo-web-browser": "~13.0.3",
    "golden-expo-app": "file:",
    "react": "18.2.0",
    "react-dom": "18.2.0",
    "react-native": "0.74.5",
    "react-native-gesture-handler": "~2.16.1",
    "react-native-reanimated": "~3.10.1",
    "react-native-safe-area-context": "4.10.5",
    "react-native-screens": "3.31.1",
    "react-native-web": "~0.19.10"
  },
  "devDependencies": {
    "@babel/core": "^7.20.0",
    "@types/jest": "^29.5.12",
    "@types/react": "~18.2.45",
    "@types/react-test-renderer": "^18.0.7",
    "jest": "^29.2.1",
    "jest-expo": "~51.0.3",
    "react-test-renderer": "18.2.0",
    "typescript": "~5.3.3"
  },
  "private": true
}

Relevant log output
ERROR  Invariant Violation: requireNativeComponent: "RNCNaverMapView" was not found in the UIManager.

This error is located at:
    in RNCNaverMapView
    in Unknown (created by Index)
    in RCTView (created by View)
    in View (created by Index)
    in Index
    in Unknown (created by Route(index))
    in Suspense (created by Route(index))
    in Route (created by Route(index))
    in Route(index) (created by SceneView)
    in StaticContainer
    in EnsureSingleNavigator (created by SceneView)
    in SceneView (created by SceneView)
    in RCTView (created by View)
    in View (created by DebugContainer)
    in DebugContainer (created by MaybeNestedStack)
    in MaybeNestedStack (created by SceneView)
    in RCTView (created by View)
    in View (created by SceneView)
    in RNSScreen (created by Animated(Anonymous))
    in Animated(Anonymous) (created by InnerScreen)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by InnerScreen)
    in InnerScreen (created by Screen)
    in Screen (created by SceneView)
    in SceneView (created by NativeStackViewInner)
    in Suspender (created by Freeze)
    in Suspense (created by Freeze)
    in Freeze (created by DelayedFreeze)
    in DelayedFreeze (created by ScreenStack)
    in RNSScreenStack (created by ScreenStack)
    in ScreenStack (created by NativeStackViewInner)
    in NativeStackViewInner (created by NativeStackView)
    in RCTView (created by View)
    in View (created by SafeAreaProviderCompat)
    in SafeAreaProviderCompat (created by NativeStackView)
    in NativeStackView (created by NativeStackNavigator)
    in PreventRemoveProvider (created by NavigationContent)
    in NavigationContent
    in Unknown (created by NativeStackNavigator)
    in NativeStackNavigator
    in Unknown (created by RootLayout)
    in RootLayout
    in Unknown (created by Route())
    in Suspense (created by Route())
    in Route (created by Route())
    in Route() (created by ContextNavigator)
    in RNCSafeAreaProvider (created by SafeAreaProvider)
    in SafeAreaProvider (created by wrapper)
    in wrapper (created by ContextNavigator)
    in EnsureSingleNavigator
    in BaseNavigationContainer
    in ThemeProvider
    in NavigationContainerInner (created by ContextNavigator)
    in ContextNavigator (created by ExpoRoot)
    in ExpoRoot (created by App)
    in App (created by ErrorOverlay)
    in ErrorToastContainer (created by ErrorOverlay)
    in ErrorOverlay (created by withDevTools(ErrorOverlay))
    in withDevTools(ErrorOverlay)
    in RCTView (created by View)
    in View (created by AppContainer)
    in RCTView (created by View)
    in View (created by AppContainer)
    in AppContainer
    in main(RootComponent), js engine: hermes

Reproducible Sample Repository
실행 테스트라 없습니다.
Code of Conduct

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

라이브러리 설치 후 development build 만드셨나요??

답변 감사드립니다~!


[AIRFLOW-2445] Allow templating in kubernetes operator

Make sure you have checked all steps below.
JIRA

[ ] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-XXX
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[ ] Here are some details about my PR, including screenshots of any UI changes:

Tests

[ ] My PR adds the following unit tests OR does not need testing for this extremely good reason:

Commits

[ ] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[ ] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[ ] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

Codecov Report

Merging #3338 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3338   +/-   ##
=======================================
  Coverage   76.11%   76.11%
=======================================
  Files         197      197
  Lines       14741    14741
=======================================
  Hits        11220    11220
  Misses       3521     3521

Continue to review full report at Codecov.

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

@asfgit why not included this in 1.10 branch?


requirements: Remove mypy.txt and update doc on mypy install.

CC: @gnprice

@rht I don't think this change makes sense, since we will very likely end up upgrading to a Git version of mypy again in the next couple months and needing to undo it.  At least, that's what we've ended up doing with every other past mypy release.

Make sense

On second thought: even with the Git version, the content of mypy.txt will still be one-line anyway.

Yes, but we the docs/mypy.md instructions are better this way.


WFLY-17957 EJB timer schedule increment 0 should be considered as single value

https://issues.redhat.com/browse/WFLY-17957

The failed 1 test looks unrelated to this changes:
org.jboss.as.test.manualmode.ejb.client.outbound.connection.security.ElytronRemoteOutboundConnectionTestCase.testOverridingElytronDefaultContextWithSSLRemoting
WFLYRMT0004: Address already in use [::1]:54448
Caused by: java.net.BindException: Address already in use


Implement status UI for eUPF

I'm as a eUPF user is able to see current system status in order to be sure that eupf up and running.
Acceptance criteria:

[x] Status UI screen implemented
[x] UI presented to team


Add response headers

New features. Parse header comment and add to response.
Add a comment with header in the same format as the success tag.
// @Header 200 {string} Location "entity/10"
It is possible to add multiple headers to the same response code.
Header comment must be defined after success tag.
Example:
// @Success 200 {string} string	"hello"
// @Header 200 {string} Location "/entity/1"
// @Header 200 {string} Token "qwerty"

will generate
  responses:
        "200":
          description: hello
          headers:
            Location:
              description: /entity/1
              type: string
            Token:
              description: qwerty
              type: string
          schema:
            type: string

@ihippik Could you help add doc for this new feature?

Sorry, forgot. Added documentation.


Prometheus failing to start as it cannot validate alertmanager url

What did you do?
prometheus -alertmanager.url=http://alertmanager-infra--0-4-1--mtq3ndm0odqxnw.service.us-east-1.consul:9093 -config.file=/etc/prometheus/prometheus.yml -log.level=debug
What did you expect to see?
Prometheus should start
What did you see instead? Under which circumstances?
ERRO[0000] invalid Alertmanager URL: http://alertmanager-infra--0-4-1--mtq3ndm0odqxnw.service.us-east-1.consul  source=main.go:64
Environment
Docker Using phusion base image . Running docker with prometheus in net=host mode

System information:
uname -srm
Linux 4.4.0-31-generic x86_64

Prometheus version:
prometheus -version
prometheus, version 1.0.1 (branch: master, revision: be40190)
build user:       root@e881b289ce76
build date:       20160722-19:54:46
go version:       go1.6.2

Alertmanager version:

Prometheus configuration file:

global:
  scrape_interval:     15s # By default, scrape targets every 15 seconds.
  evaluation_interval: 15s # By default, scrape targets every 15 seconds.
  # scrape_timeout is set to the global default (10s).

  # The labels to add to any time series or alerts when communicating with
  # external systems (federation, remote storage, Alertmanager).
  external_labels:
    apptype: 'infra'
    datacenter: 'us-east-1'

rule_files:
  - /data/alert_rules/*.rules

# A list of scrape configurations.
scrape_configs:

  - job_name: 'all-consul'
    scrape_interval: 10s
    scrape_timeout:  10s
    consul_sd_configs:
      - server: 'localhost:8500'
        datacenter: us-east-1
        services: []
    relabel_configs:
      - source_labels: [__meta_consul_service]
        target_label: job
        # Use the instance ID as the instance label
      - source_labels: ['__address__']
        regex: '^(.+):.+'
        target_label: __address__
        replacement: '${1}:9100'
      - source_labels: [__meta_consul_node]
        target_label: instance

Alertmanager configuration file:

insert configuration here (if relevant to the issue)

Logs:

ERRO[0000] invalid Alertmanager URL: http://alertmanager-infra--0-4-1--mtq3ndm0odqxnw.service.us-east-1.consul  source=main.go:64

I see yall are using github.com/asaskevich/govalidator
I did the following as a test
checkurl.go

package main

import (
  valid "github.com/asaskevich/govalidator"
)

func main() {
	println(valid.IsURL(`http://alertmanager-infra--0-4-1--mtq3ndm0odqxnw.service.us-east-1.consul:9093`))
}

When i run it
go run checkurl.go
true

Why is prometheus failing to start up even when the url is valid?

Updating of the govalidator version fixes this issue. Fix in #2023.

Thanx @fabxc .
Qs: Will there be a version to download from https://prometheus.io/download/ or someplace or do i need to pull from master and build ?


Add disableScrim to Dialog

/release-pr

Can we target this PR to main please, putting it back on the original branch will still make reviewing harder than if it wasn't

Can we target this PR to main please, putting it back on the original branch will still make reviewing harder than if it wasn't

@joshwooding so move everything back to labs ?


Internal 'drv' rule left, when should be replaced by alias

The grammar below illustrates the issue.
from lark import Lark

grammar = r'''
    start: s1 | s2
    s1: s2
    s2: a bc

    a: "a" -> id1
    bc: b c -> id2
    b: "b" -> id3
    c: "c" -> id4
    '''

value = 'abc'
parser = Lark(grammar, parser='earley', ambiguity='explicit', keep_all_tokens=True)
tree = parser.parse(value)

print(tree)
print(tree.pretty())

Tree(_ambig, [Tree(start, [Tree(s2, [Tree(id1, [Token(A, 'a')]), Tree(id2, [Tree(drv, [Token(B, 'b')]), Tree(drv, [Token(C, 'c')])])])]), Tree(start, [Tree(s1, [Tree(s2, [Tree(id1, [Token(A, 'a')]), Tree(id2, [Tree(drv, [Token(B, 'b')]), Tree(drv, [Token(C, 'c')])])])])])])
_ambig
  start
    s2
      id1	a
      id2
        b	b
        c	c
  start
    s1
      s2
        id1	a
        id2
          b	b
          c	c

[It may look a little pathological :), but it is designed to capture just the issue. The real grammar is much larger, and needs both the aliases and the ambiguity.]

Thanks for the bug report. I will look into it soon.

Works on master


1.1.1: tooltip prop types fix

Coverage remained the same at 82.767% when pulling 3e0c86f2d2a9a3163c9279f3f9e7b8e0a064e8c4 on dev into 08d99e8c9d6aa1b1db0dd5ed05e757573bfd90d7 on master.


example

Please an  example of usage it is a bit blurry.
thanks

I think that is a little late but i explain this package for anyone who need,
you do not need any usage example, u only have to register package routes. and package will handle all that you need.


only

See rule #1

I did this purposefully. Yalu isn't a beta. The name is the only thing what makes it a beta.
Yalu isn't unstable. Random reboots are something you will see on every new 64bit jailbreak and it's not a big deal. Literally the only issue, which is not that big

You misunderstand the purpose of this site. It's supposed to be user friendly. The Yalu website is not user friendly.

Firstly, the website is called canijailbreak not cansomenoobjailbreak
Clicking a download button is definitely not user friendly ^\s

By following that logic, perhaps Apple.com should be wemakeiphonesandipadsandmacsandsellthemandalsohaveicloud.com

Seriously, the download button is there, you click it, done. What's not user-friendly? Also the website doesn't really matter IMO. If someone like Pangu releases a super-stable untethered jailbreak for 10.3-11.0, and the website isn't user-friendly, you won't add it just because of that? ...

@cj123 if you still aren't convinced, make a new section on the site, name it "beta-jailbreaks" or "not-fully stable jailbreaks" and add a big caption:
"These jailbreaks may have bugs and random reboots. Use at your own risk"


Typo in title in dropdown menu

Hi Team,
I am from the Azure CXP team and just spotted an issue with the title of this article that shows in the dropdown menu on the left. I believe it should say "Explore Azure SQL Resources" instead of "Explorer Azure SQL Resources".

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 1c4f0e47-7264-41fa-bf83-55240e6a0e79
Version Independent ID: a933d5da-0579-cd95-1051-95398e605563
Content: Explore Azure SQL resources with Azure Resource Explorer - Azure Data Studio
Content Source: docs/azure-data-studio/azure-resource-explorer.md
Product: azure-data-studio
GitHub Login: @yanancai
Microsoft Alias: yanacai

@acmds -- thank you for spotting this.  I am updating the table of contents.


openpgp: PromptFunction infinite loop

ReadMessage PromptFunction is called forever.
If PromptFunction returns the correct key the first time, then decryption succeeds. If, however, you return the correct key any time after the first time then decryption will never succeed and PromptFunction gets called forever.

I am happy to tackle this if no one else is working on it. You may assign it to me.

@marete, feel free to send a cl (changelist); http://golang.org/doc/contribute.html

AndeweVos: Please download and test this CL, which should fix the problem: https://go-review.googlesource.com/2570

@marete and @bradfitz, I think @agl fixed this issue with https://go-review.googlesource.com/#/c/14038 but since in the commit message of that CL the magic word Fixes golang/go#9315 wasn't used but instead used Fixes #9315 yet openpgp project belongs under golang/x/crypto, this issue was not auto-closed nor referenced here. This also means that CL https://go-review.googlesource.com/2570 may not be needed either.
Please get the latest @AndrewVos and verify.

Ping!


lnwire: hard reject positive inbound fees set over RPC

In https://github.com/lightningnetwork/lnd/pull/6703, we'll add initial support for inbound fees. Due to deployment considerations, only inbound fees should be used, as they're backwards compatible. An unupgraded sender will end up missing out on the discount, but the payment will go through if negative fees are used. If positive fees are used, then the forwarding node may reject a payment from an unupgraded sender. Additionally, in order for the sender to be able to properly retry, they'll also need the inbound fee policy in addition to the outbound fee policy. lnd isn't yet able to send both policies until https://github.com/lightningnetwork/lnd/pull/6967 is merged.
To ensure the base feature minimized disruption in the network, we should also modify the RPC code to reject positive inbound fees. The CLI code will already do this.

The CLI update should indeed prevent an ad-hoc unintentional positive inbound fee from being set.
But for applications that are using the API and to which inbound fee support is added specifically, this safe guard is perhaps a bit too much hand-holding? The down side of the API restriction is that even if someone would want to try experimenting with positive inbound fees, they cannot do it and need to wait for and upgrade to a future version of lnd. Enabling experimentation was the main goal of the inbound fee change, so I'd argue to keep the option open on the API level.
#6967 isn't just relevant for positive inbound fees, but also for negative inbound fees that are made less negative.

If external tools have implemented incoming fees, unintentional use can also occur via the API. Imho this is also more likely as many users do their policy settings with tools like lndg or charge-lnd.
I suggest that positive inbound charges are disabled by default, but the user has the option to enable the experimental use via the configuration.

If external tools have implemented incoming fees, unintentional use can also occur via the API. Imho this is also more likely as many users do their policy settings with tools like lndg or charge-lnd.

Yeah this captures my fear: users see the value, set it to somethign positive, then wonder why all forwards are being/failed rejected.

I just meant to say that someone developing a 3rd party tool and using the api is probably more aware of the consequences of using positive inbound fees and will make a sensible decision how to offer the functionality to the tool's users. Where as with a casual cli user, the risk of making an unintentional change might be higher. Although setting a positive fee and seeing traffic go away also isn't the end of the world either.
How about adding a bool flag to the api call allow_positive_inbound_fee?

@joostjager I started already Pull Request #8627. I think it is better to have an activation flag for the positive fees in lnd.conf. Then it is disabled per default and the users who want to play around with this feature can enable it.

Ah didn't notice that one. Why do you think it is better?

Maybe the flag on the API level is more flexible and people don't need to restart LND. Tho I need regarding LND's history it feels like adding a config flag is more in line with the previous design choices, for example accept-keysend, protocol.wumbo-channels etc., but I am open to both approaches.

The reason is that we also have to think about the users of 3rd party tools. It would be quite easy for a developer of the tool to implement a logic that sets a boolean flag to true if the inbound fee is greater than 0. But then it is up to the user to set the right sign when configuring the tool. I believe that many users of a 3rd party tool would confuse fee and discount at the beginning and unintentionally set a positive one, although they actually only want a discount.
In my opinion, the config option represents an additional hurdle and the warnings in the sample will hopefully motivate the user to look more closely at the topic.

The reason is that we also have to think about the users of 3rd party tools. It would be quite easy for a developer of the tool to implement a logic that sets a boolean flag to true if the inbound fee is greater than 0. But then it is up to the user to set the right sign when configuring the tool. I believe that many users of a 3rd party tool would confuse fee and discount at the beginning and unintentionally set a positive one, although they actually only want a discount. In my opinion, the config option represents an additional hurdle and the warnings in the sample will hopefully motivate the user to look more closely at the topic.

this is a great point. I think this either has to be an opt-in config setting, or perhaps even a separate API tucked under devrpc until the functionality is widespread across the network.


bug(mat-slider): Mat slider unstable values when sliding and when release click

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
mat slider with range (even without range) acts weird. When sliding thumb with mouse click and dragging, it shows the actual values properly but when you release the mouse button it changes its value abnormally. I attached a gif that demonstrates problem.
Reproduction
                    <div class="w-full px-4">
                        <mat-slider class="mt-2 w-full mx-0" min="0" [max]="maxHoleSize"  disableRipple="true">
                            <input formControlName="minHoleSize" matSliderStartThumb>
                            <input formControlName="maxHoleSize" matSliderEndThumb>
                        </mat-slider>
                    </div>

just use this snippet with tailwindcss classes.
Expected Behavior
when you release the mouse, (as you can follow with starting and ending of ripple effects on gif) you'll see the anomaly.
Actual Behavior

Environment

Angular:15.0.0
CDK/Material:15.0.0
Browser(s):firefox, safari
Operating System (e.g. Windows, macOS, Ubuntu): macos

Could you try this on the latest patch. I believe this issue has been fixed

Yes it works now, I should have checked that in the first place, sorry for your time. thanks