Style/MultilineHashBraceLayout still flagging single lines

Hello,
I previously reported the issue with the Multiline cops flagging single line statements as false positive.
This is still happening with the Style/MultilineHashBraceLayout Cop.

Expected behavior
Do not flag single line statements as offense when using the EnforcedStyle: new_line option.
Actual behavior
Single lines are flagged as violation.
Steps to reproduce the problem

Enable the Cop Style/MultilineHashBraceLayout
use the configuration: EnforcedStyle: new_line
run rubocop.

The following code for example get's flagged:
expect(subject.cache_dir).to eq cache_dir

let(:cache_dir) { Rails.root.join('tmp/uploads') }

VCR.configure do |c|
  c.filter_sensitive_data('<BVDWEBPASSWORD>') { settings.bureau_van_dijk.web_service_password }
end

RuboCop version
Include the output of rubocop -V:
$ bundle exec rubocop -V
0.40.0 (using Parser <IP_ADDRESS>, running on ruby 2.2.5 x86_64-darwin15)

@NekoNova: I can not reproduce this using 0.41.1. Can you confirm it has been fixed on that version?

I cannot run that version due other bug with the. Rubyversion file
On 3 Jul 2016 08:46, "Ted Johansson"<EMAIL_ADDRESS>wrote:

@NekoNova https://github.com/NekoNova: I can not reproduce this using
0.41.1. Can you confirm it has been fixed on that version?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bbatsov/rubocop/issues/3256#issuecomment-230138632,
or mute the thread
https://github.com/notifications/unsubscribe/AAsWhUTHfNU8OYTn7X2uiMKTNVFbn3DLks5qR1q7gaJpZM4JAEQY
.

I've upgraded Rubocop to 0.41.2, but I'm still running into issues with this Cop.
Right now I's throwing me the following error:
Closing method call brace must be on the line after the last argument.
          { ...
          ^

The offending code is the following:
def generate_some_objects(options = {})
  before :each do
     create_one_object(
        MyObject.new,
        {
            option_1: 'a',
            option_2: 'b',
            option_3: 'c'
         }.merge(options)
      )
  end
end

This code should be accepted when using the new_line configuration option.

The following snippet in our specs is not accepted either:
    it 'should create the process' do
      expect(expected_builder).to(
        receive(:new).with(
          command_name: :test,
          user_id: 1,
          customer_id: 1,
          parent_process_id: 1,
          command_attributes: { attribute_for_command1: 2, attribute_for_command2: 3 }
        ).and_return(builder)
      )
      expect(builder).to receive(:build).and_return(process)
      expect(subject.execute(options)).to eq process
    end

spec/support/background_example_groups.rb:54:9: C: Closing method call brace must be on the line after the last argument.
        receive(:new).with( ...
        ^^^^^^^^^^^^^^^^^^^

First of all it's the Style/MultilineMethodCallBraceLayout you're talking about.
I think its behavior, as far as detecting offenses is concerned, is consistent with the description in config/default.yml
Style/MultilineMethodCallBraceLayout:
  EnforcedStyle: symmetrical
  SupportedStyles:
    # symmetrical: closing brace is positioned in same way as opening brace
    # new_line: closing brace is always on a new line
    # same_line: closing brace is always on the same line as last argument
    - symmetrical
    - new_line
    - same_line

Note: "closing brace is always on a new line".
The problem is that the cop highlights the wrong thing. Since all the messages talk about what's wrong with the closing bracket (or brace as this cop calls it), this is what we should highlight.
Then we get reporting like this:
/tmp/3256.rb:17:22: C: Closing method call brace must be on the line after the last argument.
      }.merge(options)
                     ^
/tmp/3256.rb:30:25: C: Closing method call brace must be on the line after the last argument.
    ).and_return(builder)
                        ^

I'll submit a PR soon.

that makes more sense.


Allow setting the MAVEN_MIRROR_URL

Questions have been asked if our S2I builders can support MAVEN_MIRROR_URL.
It seems the official RH Java S2I builder (registry.access.redhat.com/redhat-openjdk-18/openjdk18-openshift) does indeed. But setting it would be a very manual process that would require manual intervention each time a Booster was launched.
So perhaps we can actually add this to the Launch application itself.
One option would be to add it to the wizard (in the advanced setting section for example). It would be less manual work but it would still require the user to know about mirror urls.
The second option would be to add it to the template we use to create the Launch application. This assumes that the Maven mirror is part and parcel of the chosen OpenShift server. It also means that the list of possible OpenShift servers needs to be extended with an (optional) mirror url.
In both cases the mirror url is very runtime-specific. It can be used for the Spring Boot, WIldfly Swarm and Vert.x runtimes, but not for the NodeJS runtime. But one could imagine that NodeJS might have it's own NPM mirrors that people would like to use. So the chosen solution should take into account that at some point we'd might need several mirror urls.

I would like to second this enhancement request.
This would make it easier to use Nexus as cache for all new applications by default. And IMHO, to make this really usable there needs to be optional CR field for the default MAVEN_MIRROR_URL value, instead of requiring every user to input it.


Build Error [ src/main.zig:66:60: error: unused capture |err| ] with Zig Version 0.9.0

Following the readme instructions to build, Its failing with error: unused capture on Main.zig Line:66
~/ $  zig version
~/ $  0.9.0
~/ $  cd csv2json
~/csv2json $  zig build -Drelease-fast
		    /home/user/csv2json/src/main.zig:66:60: error: unused capture
		            bufSize = std.fmt.parseUnsigned(u32, b, 10) catch |err| {
		                                                               ^
		    csv2json...The following command exited with error code 1:
		    /usr/bin/zig build-exe /home/user/csv2json/src/main.zig -OReleaseFast --cache-dir /home/user/csv2json/zig-cache --global-cache-dir /home/user/.cache/zig --name csv2json --pkg-begin clap /home/user/csv2json/lib/zig-clap/clap.zig --pkg-end --pkg-begin csv /home/user/csv2json/lib/zig-csv/src/main.zig --pkg-end --enable-cache
		    error: the following build command failed with exit code 1:
		    /home/user/csv2json/zig-cache/o/0cf20af84c42b68b4c79e2a929c982e5/build /usr/bin/zig /home/user/csv2json /home/user/csv2json/zig-cache /home/user/.cache/zig -Drelease-fast

I'd compiled the app with v0.8.1. Looks like the language has changed between that and v0.9.0. Please feel free to send a PR if you're able to fix this, or I'll look into this when I get a chance.

I'd compiled the app with v0.8.1. Looks like the language has changed between that and v0.9.0. Please feel free to send a PR if you're able to fix this, or I'll look into this when I get a chance.

PR Sent


jog_coord does not behave as I thought it would

Describe the bug
I'm trying to get the robot to grasp an upright optical element by bracing the claw against the "lip" and moving vertically upward (with no rotation of the claw).
My understanding based on https://docs.elephantrobotics.com/docs/pdf/myCobot-en.pdf is that the coordinates of the head of the robot are encoded as [x, y, z, r_x, r_y, r_z]. In order to move vertically upward, I thought I would simply jog coordinate z, and the other coordinates would remain fixed (i.e., no rotation or lateral movement).
However, when I send jog_coord(3, 1, 1), (which should be telling it to increase it's z-axis coordinate at a speed of 1), the head rotates while it moves upward.
https://github.com/elephantrobotics/pymycobot/assets/173008148/4fcc8382-d8be-4b99-b384-6b1f0d0ff624
https://github.com/elephantrobotics/pymycobot/assets/173008148/ca92a2f5-ea40-4637-80dd-4f48dda8a8ab
To Reproduce
Steps to reproduce the behavior:

Lay the robot arm out flat on the ground
Send jog_coord(3, 1, 1)
Send  jog_coord(3, 0, 1)
See that the gripper rotates as the arm moves upwards.

Expected behavior
I expect the head of the joint to hold the same rotational and x-y coordinate (visually stay steady) as the arm moves up in the vertical direction.
Screenshots
Videos are attached.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

OS: Ubuntu 20.04, as comes with Raspberry Pi
Browser Firefox
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Please use myStudio to upgrade the atom firmware to the latest version 7.0-beta and try again


Wiki update tracker action

Changes:

Added Wiki Update Tracker action
Added Awagi to CODEOWNERS file for the /fr/ translation of the wiki

Wiki Update Tracker
Awagi's Wiki Update Tracker is a translation tool to keep track of changes to all of the wiki's English files.
Features

Watches all wiki files for changes on master branch
When an English wiki file is changed, creates a GitHub issue showing the changes

Action
Problem

The action needs permission to read the code and create GitHub issues.
The default GITHUB_TOKEN secret provides unnecessary permissions, like the ability to write GitHub packages.
The wiki is deployed using GitHub's package system, and we don't want to pass this token to any external apps that would be able to release their own packages.

Solution

Instead of using the GITHUB_TOKEN, this action uses a custom GitHub App I've added to this repo that only has permission to read the code and read/write issues.
The private key and app ID of this app is stored in the SECRETS for this repo, and passed to the marketplace app Actions App Token to get a custom token for the WUT to use.
The WUT then uses this token to gain all of the permissions that the GitHub App has (read code, read/write issues).

Thanks to Awagi for writing the action file for this, I only tested it.

Merging this because it worked when I tested it, and BSMG Staff doesn't seem to mind.


Default Cargo.toml template provide help for completing the metadata

Descriptio
This is new Cargo.toml example.
[package]
name = "foo"
version = "0.1.0"
authors = ["k-nasa"]
documentation = ""
homepage = ""
repository = ""
readme = ""
categories = []
keywords = []
license = ""
edition = "2018"

[dependencies]

# See more https://doc.rust-lang.org/cargo/reference/manifest.html

Motivation
close: https://github.com/rust-lang/cargo/issues/6845

r? @alexcrichton
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks for the PR! I would personally prefer to not do this as I feel it clutters up the template with a bit too much noise, but perhaps we could add a comment by default with a link to the reference?

@alexcrichton   I fixed it ! How is it like this?
I'm wondering if I write a link on head or write it tail.

Since the intention was to teach about the metadata in the [package] section, perhaps this could be just after that section and look like:
# See more keys and their definitions at https://...

I changed it. It will be like this.
[package]
name = "hogehoge"
version = "0.1.0"
authors = ["k-nasa<EMAIL_ADDRESS>edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]

Ok thanks @k-nasa!
Since this affects the default template which has pretty high visibility I'd like to check in with other folks as well before merging...
@rfcbot fcp merge

I like the "just add a comment" version much better. :+1:

Succeeded in rebase.
@ehuss @joshtriplett I'm sorry to do miscellaneous things

@bors: r+

:pushpin: Commit f781d2df86c4215f1a4ec36772b9ee68e6b5d206 has been approved by alexcrichton

:hourglass: Testing commit f781d2df86c4215f1a4ec36772b9ee68e6b5d206 with merge 7eb9e6ca529d0a551a74d3f5bc774c8d5a075682...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: alexcrichton
Pushing 7eb9e6ca529d0a551a74d3f5bc774c8d5a075682 to master...


some fixtures need renaming

commit a113b8550dc683d8d507b9e618bc438805476d19 introduced some test fixtures but they don't have the .fixture.js extension.

cc @outsideris


Iso time

Change time format to ISO format.  This makes the TimeStuff class obsolete.  Times can now be set with datetime.utcnow().isoformat().
Version also changes from 1.1.0 to 1.1.1

Your commits from yesterday started to fail the tests;
https://github.com/SNEWS2/SNEWS_Publishing_Tools/actions
could you have a look at the cause

I have updated the format checker to include warnings with the logs.
Also did some fixes to the time checking logic and fixed some typos in the test function times.

Thanks Joe it looks better!
I can try to test it tomorrow, latest next Monday and merge it.
Please let me know if you need me to do it faster

You can also debug these via snews_pt run-scenarios it shows already these error messages without having to interact with cs


validation_epoch_end behavior with DDP

I might be misunderstanding how PL works, but when using DDP my validation_epoch_end argument still contains batches from single GPUs, and I thought they would be collated from all GPUs.
E.g. My validation dataset has 888 images, but when I validate on 8 GPUs (batch size of 1), I only get 111 batches in validation_epoch_end.
If that's correct, how can I produce metrics that combine information from all GPUs?

 validation_step operates on a single batch of data from the validation set.
validation_epoch_end is called at the end of the validation epoch with the outputs of all validation steps.
So I believe your problem lies at validation_step. Can you show us your validation step, ideally the whole model?

I asked around and apparently that is the intended behaviour right now, i.e. validation_epoch_end is per-process, and we cannot access global information for metrics or logging. I was able to solve this by doing the reduce_all myself, with something like this:
metrics[key] = metrics[key].to('cuda:{}'.format(self.trainer.proc_rank))
dist.all_reduce(metrics[key])
metrics[key] /= self.trainer.world_size

Not sure why I had to explicitly cast the tensors to their processes.

Also want native support of this ability. such as adding an argument to validation_epoch_end , when it's set to True, then metrics returned by validation_epoch_end  will automatically be all_reduce and logged, instead of only metrics on each process

I am using the latest version 1.2.2 and I get the same behavior.
@williamFalcon I see this was fixed in #2029. But I did not find how to make validation_epoch_end receive batches from all gpus.
Could someone please give me a hint what do I have to do to fix this?


deleted failed

What happened

What you expected to happen
can delete task
How to reproduce
see above
Environment
No response
InLong version
1.12.0
InLong Component
InLong Manager
Are you willing to submit PR?

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

Code of Conduct

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

From the screenshot, it can be seen that there is a stream in test_group_1646 that has not been deleted. You need to delete the stream in the group first in order to successfully delete the group.


ui-auto-monkey script would stop when app go to background

Hello,
I face a problem that if tap some button app would go to background, then script stop runing.
For example tap on 3rd login button. How to i solve it?

I had same problem while I use auto-monkey doing test. If the monkey taps out of the target app, the instruments will hang over there and nothing can do. Is there a way to detect the thread runs to different app? I notice there is a demo: https://www.youtube.com/watch?v=y5PZGVbLHtI&feature=youtu.be
In this demo, he could detect and stop the monkey, but I don't have idea how did he make it.

Currently I use a workaround to overcome this situation. Because we use Monkey as our stress test to try if there is any crash for current version. So I write a script to launch the Monkey for at most 10 mins one time, and set the script in LaunchAgent to run it every 10 mins. In this way to relaunch Monkey and make sure it can run through the whole weekend. But if possible I still would like to know how to detect while it runs to different app.

Find another better workaround. Please refer link as below:
http://developer.appcelerator.com/question/40081/ios4---make-app-not-run-in-background
Config the Info.plist file to enable "UIApplicationExitsOnSuspend", in this way if the target app is hidden to background the app will exit right away, so it would cause instruments to be terminated.
But in this way the log will show "2015-05-19 08:51:29 +0000 Fail: The target application appears to have died", which might be confused with real crash. Still have to find a way to detect the status and log it.

@JohnnyWuMigme @jonathanpenn refer to my Project， https://github.com/vigossjjj/CrashMonkey4IOS

Is running with Guided Access on an option? Your script could pause and allow you to enter Guided Access manually. You can also enter programmatically (via objective-c code or swift) if you have a managed device. That requires adding some code to your application and creating a managed iOS device (Meraki is one provider).


HttpStatusIOException from BTCEMarketDataService.getOrderBook()

This exception surfaces about once per hour for BTCE.
si.mazi.rescu.HttpStatusIOException: HTTP status code was not OK: 400
	at si.mazi.rescu.ResponseReader.read(ResponseReader.java:106)
	at si.mazi.rescu.RestInvocationHandler.mapInvocationResult(RestInvocationHandler.java:137)
	at si.mazi.rescu.RestInvocationHandler.receiveAndMap(RestInvocationHandler.java:125)
	at si.mazi.rescu.RestInvocationHandler.invoke(RestInvocationHandler.java:104)
	at com.sun.proxy.$Proxy12.getDepth(Unknown Source)
	at com.xeiam.xchange.btce.v3.service.polling.BTCEMarketDataServiceRaw.getBTCEDepth(BTCEMarketDataServiceRaw.java:56)
	at com.xeiam.xchange.btce.v3.service.polling.BTCEMarketDataService.getOrderBook(BTCEMarketDataService.java:62)

The actual call is:
Object[] args = {20};
BTCEMarketDataService.getOrderBook(CurrencyPair.BTC_USD, args);

For BTCE, args[0] is the depth of the OrderBook that you want.
I am calling this once per 1000ms.  The only thing I can think of is that this exception hits potentially when the book is less than 20 deep?    For BTCE though, I find that highly unlikely.
Any suggestions are greatly appreciated.
-Bryan

Did you try calling it with other argument, say new int[]{5}?
Did you enable trace logging for rescu and check the actual HTTP call?
400 is Bad request, so maybe you're right about the order book being too shallow,
have you tried contacting BTC-e support?

@bryan4887 Any luck with this?

Hey @rafalkrupinski I haven't had a chance to follow up on this yet, but I certainly will update everyone when I get a chance.  Thanks again for the suggestions!


Flagging wayfinders in our map

Is there a way to flag wayfinders in our map so that we can quickly find the ones we want to review a few more times?

wayfinders?

Yeah, that. I was thinking about a map while I was typing that.

I am in favor of this as well -- at one point in a gitter room I had suggested making it possible for people to tag the challenges with topics so they could return to review examples on a particular topic ... :)

YES Please!!

I'd chime in on perhaps adding a way to easily go back and review topics / lessons like a "previous" and "next" button to just take you sequentially back and forth through recent lessons.  Another thought might be to provide helpful links to the lessons that are most immediately applicable to the lesson at hand.  But I think that the "previous" and "next" would go a long way to helping navigate the lessons because they are usually in a good sequence, and just looking at the most recent ones might help to jog a person's memory or give a good hint about what they should do.


New features

Additions and changes;

PlaceholderAPI version has been upgraded from 2.11.3 to 2.11.6.
Added instance to the main plugin class.
Added 2 new classes.
Changes were made on config.yml.
When players enter fields with names, a message has been set to be sent to players as actionbar or title.
Added black-list to block some fields.
Of course this feature is set to disable :)

Sorry for the wait. Thank your for the PR! Had no idea action bars on entering weren't a feature already built into WorldGuard.


bug-fix/iresearch-error-propagation

Use arangodb::Result to pass errors up to the caller. Before the most functions returned a bool value only. Making it necessary to throw in order to let the user know what is wrong.
This PR will eventually fix: https://github.com/arangodb/backlog/issues/559

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/4516/


Issue using seeding script and refs-to-seed with hashes that have no seeds

So I'm currently trying to set up a gateway to pin a larger list of content.
Using make seems to work OK, but the script hangs because I'm running into content that I don't think is seeded at all.
It would be nice to be able to identify which hashes are hanging, in order that we could either find them and seed them, or remove them from the original list.
Because the script is recursively seeding hashes, I'm unable to find which is the parent hash of the hashes that are hanging.
I'm not sure what the best way to approach fixing this is, perhaps a timeout in the seeding script or in ipfs pin?
Being able to pin recursively would also be useful.

Yeah, a couple of people have already come across that (myself included). I think @jbenet already fixed some of it, but obviously there's still some issues. From memory, the problematic hash was
# ipfs-geoip
/ipfs/QmQQ3BUpPjgYiTdhp4H9YWSCtoFXs8t91njhpvXNNLd3yB

Ah, OK. so ipfs pin -r does work in this case. That helps a little bit.
As I said, I have my own list of hashes to seed, they aren't the ipfs ones.
but I can start working manually and using pin -r  to try and identify it

still running into pretty big problems with this.
it's a big challenge because there isn't really any feedback in the UI.
Ideally, it'd be great to say: ipfs pin -r [listofhashes], and then see something like:
adding 42 hashes, 237 recursive hashes
followed by a progress bar for each hash.
The issue is that i'm running into timeouts and the command is stalling. I don't know which hash is causing the issue, and I'm having to use trial and error to add each one by one.

Hey @sroerick -- that sounds very bad, sorry, let's get your problems fixed.

Ideally, it'd be great to say: ipfs pin -r [listofhashes], and then see something like:
adding 42 hashes, 237 recursive hashes
followed by a progress bar for each hash.

this sounds great. agreed on making the UX better. maybe open an issue in go-ipfs with a detail description of the UX that you would love?

The issue is that i'm running into timeouts and the command is stalling. I don't know which hash is causing the issue, and I'm having to use trial and error to add each one by one.

One thing i often do is run ipfs refs -r <hash> beforehand. just to see them stream in. i also run bsdash. see https://github.com/jbenet/node-bsdash

:+1: better pinning ux

One thing i often do is run ipfs refs -r  beforehand. just to see them stream in. i also run bsdash. see https://github.com/jbenet/node-bsdash

If you do it, it might be good to mention it somewhere in the program mans or incorporate your flow directly.

agreed. we really should just improve the pinning UX overall, and not have to do refs.

thanks @jbenet,
opened https://github.com/ipfs/go-ipfs/issues/1711

Please reopen if there's anything actionable here, from an infrastructure point-of-view


Begin to rework integration tests

This improves the test harness in the following ways:

[x] uses libtest-mimic to dynamically construct a rustc --test-style test harness

[x] integration tests are now automatically discovered
[x] examples tests are run in parallel, at the same time as the integration tests

[ ] integration tests now use the CLI rather than reaching into the internals of Fathom
[ ] the cursed diagnostic lexer has been removed

Seems like in order to make progress on decoupling the tests from the internals I might want some simple JSON serialization of the compiler diagnostics in order to correlate the line //~ error: ... directives with the compiler output, without having to match on the pretty-printed output. This might be a candidate for inclusion in codespan-reporting, but it could initially be done in codespan-cli. Not sure I want to do this in this PR.

Haven't got everything finished, but I'm gonna merge this as is, because I think it's an improvement. I was also experimenting with TOML-based syntax for test directives in Pikelet, which would avoid me having to come up with something myself: https://github.com/pikelet-lang/pikelet/pull/258


ProviderChecks() always silently choose default choice

The minecraft AUR package requires java-runtime, which is provided in the repo by both jre7-openjdk and jre8-openjdk.
The ProvidersCheck() prompt is expected to kick off and offer the choice of the binary deps, as originally designed in #77. However, the default choice is chosen automatically, and later on makepkg -s also choose the default first choice due to the use of the --noconfirm flag.
The issue is located in ProvidersCheck(), with the repoprovidersconflictingpkgs not correctly set and the subsequent code that "install provider packages and repo conflicting packages that makepkg --noconfirm cannot handle" unable to work as expected.
I am not sure when this regression has been introduced, but it's been possibly done a long time ago. Since the depedency tree is not broken, it stayed undetected.
The issue is tightly related to the use of pactree that should be removed anyway (#575), so an overhaul is needed here.
Default provider choice silently chosen:
$ pacaur -S minecraft
:: Package minecraft not found in repositories, trying AUR...
:: resolving dependencies...
:: looking for inter-conflicts...

AUR Packages  (1)            Old Version     New Version

aur/minecraft                                latest-26

Repo Packages (3)            Old Version     New Version     Download Size

extra/java-runtime-common                    2-2                  0.00 MiB
extra/jre7-openjdk                           7.u131_2.6.9-1       0.15 MiB
extra/jre7-openjdk-headless                  7.u131_2.6.9-1      42.32 MiB

Repo Download Size:   42.48 MiB
Repo Installed Size:  78.08 MiB

:: Proceed with installation? [Y/n] n

Expected: Provider prompt appears and allows to choose the package that provides java-runtime.

Fixed in c6b7fdb98de063326f6e659f8691197893e4d5a8.
$ pacaur -S minecraft
:: Package minecraft not found in repositories, trying AUR...
:: resolving dependencies...
:: There are 2 providers available for java-runtime:
   0) jre7-openjdk (extra)
   1) jre8-openjdk (extra)

Enter a number (default=0): 1
:: looking for inter-conflicts...

AUR Packages  (1)            Old Version  New Version

aur/minecraft                             latest-26

Repo Packages (3)            Old Version  New Version  Download Size

extra/java-runtime-common                 2-2               0.00 MiB
extra/jre8-openjdk                        8.u131-1          0.16 MiB
extra/jre8-openjdk-headless               8.u131-1         25.84 MiB

Repo Download Size:   26.00 MiB
Repo Installed Size:  95.55 MiB

:: Proceed with installation? [Y/n] n


Online icon doesn't reflect if online on mobile or not

The status icons are always the same circular shape instead of distinctive shapes, just colored differently according to the online status. This is the case everywhere in the UI. The only place where the actual online status, including shape and mobile indicator, is shown, is in the DM header.
There have been instances where the mobile icon does show up in the UI, but that seems to be a bug instead of intentional behavior, as it only shows up if the online status of someone changes while viewing the list they are contained in, or if the said person's online status is loaded after the theme is, for example when scrolling the list.

Discord's statuses are very very limiting and are a huge pain to deal with. And the mobile indicators are are bug that we cannot fix as Discord doesn't update the element that we rely on.

Would there be a workaround for me to revert to Discord's default status icons or would that involve changing underlying code that breaks other things like positioning etc.? I pretty much rely on Discord showing me at a glance on my DM list, if someone is online on their PC or their phone. It isn't a huge deal if it doesn't work but it would be nice to have the option.


Update gel4y-enc.php

i agree with the changes you gave in the file "gel4y-enc.php" but not in "gel4y.php" because the original content should remain for the next development process. Thank you for your participation


Debian Stretch no longer supported

New as of June 2022:
When developing and calling vagrant-spk vm up, the virtual machine's apt update fails to find some packages (I get a bunch of Error 404 s) because debian stretch (via vm box debian/contrib-stretch64) is not supported anymore. I think they took down some essential repos.
I'm trying out bumping to debian/contrib-buster64 and debian/bullseye64. There is no contrib-bullseye64 option.

Just use bullseye64, contrib- isn't required starting with that release.

Yeah I tried just dropping in bullseye64, but it didn't work out of the box. Buster did though, so I'll make a pm for that.

What issue did you hit with bullseye64?

Looks like /opt/app/ didn't mount correctly. I've been trying to get virtual box guest additions versions from my host to work with the ones on the virtual machine for a bit.
From the logs
Probably not a problem
[default] A Virtualbox Guest Additions installation was found but no tools to rebuild or start them.
The guest tries to update the additions
Installing Virtualbox Guest Additions 6.1.34 - guest version is 6.0.0
These versions may be compatible
An error occurred during installation of VirtualBox Guest Additions 6.1.34. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
Unmounting Virtualbox Guest Additions ISO from: /mnt
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.0
VBoxService inside the vm claims: 6.1.34
Going on, assuming VBoxService is correct...
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.0
VBoxService inside the vm claims: 6.1.34
Going on, assuming VBoxService is correct...
Got different reports about installed GuestAdditions version:
Virtualbox on your host claims:   6.0.0
VBoxService inside the vm claims: 6.1.34
Going on, assuming VBoxService is correct...
Restarting VM to apply changes...

And finally there are a few different errors about /opt/app/ not being mounted, such as
    default: /opt/app/.sandstorm/setup.sh: line 26: npm: command not found
The SSH command responded with a non-zero exit status. Vagrant
assumes that this means the command failed. The output for this command
should be in the log above. Please read the output to determine what
went wrong.

Just connecting this issue with a PM for updating to Debian Bullseye (which is more recent than Buster).
https://github.com/mnutt/davros/pull/149


ValueError when running "makemigrations" or "migrate" Stable/1.9.x

When i try to migrate or makemigrations i get this error:
ValueError: ModelState.fields cannot refer to a model class - "regiao.to" does. Use a string reference instead.

It is strange because i don't have any to='something' parameter.
I'm not copying my models.py here because it's more than 2000 lines.
I'm using django 1.9 and these packages:
(and django-adminactions incorporated in the project without pip for translation porpouses)
Django==1.9
# pyKML==0.1.0
pymysql==0.6.1
wheel==0.22.0
openpyxl==1.8.3
unidecode==0.04.14
requests==2.3.0
gpxpy==0.9.8
geocoder==1.10.1 # git+https://github.com/DenisCarriere/geocoder.git#egg=geocoder
suds==0.4
gunicorn==19.4.5
newrelic==<IP_ADDRESS>
MySQL-python==1.2.5
dj-static==0.0.6
dj-database-url==0.3.0
restless==2.0.1
sqlparse==0.1.15
darts.util.lru==0.5
redis==2.7.6
mock==1.3.0
django-solo==1.1.0
django-test-without-migrations==0.4
django-nested-admin
django-modeladmin-reorder==0.2
django-rosetta

six==1.10.0
ratelim==0.1.6
argparse==1.2.1
click==6.3
decorator==4.0.9
funcsigs==0.4
pbr==1.8.1

xlsxwriter
xmltodict

numpy

Has anyone ever had an error like this?

@vitorcunha1995, please use one of our support channels (see sidebar) to get help.


BepInEx does not discover plugin

I've followed the instructions but the plugin does not load and the BepInEx startup log says "0 plugins to load" (and "0 patcher plugins loaded")

+1, using bepinex6.0

same issue, downloaded another plugin and can confirm it works. seems to be that bepinex is looking for a folder named com.xxx.xxx containing the files we download here, with the dll following the same naming scheme.

I have tested this and seems to work as normal without the need to create a folder for the plugin

Does this seem to be fixed with the latest build?

Yep, 1.4 wasn't available for me a couple days ago but it's fixed the issue. Thought that just updating to a new BepInEx version was the fix for 1.3.1

All right. I guess updating the meta library for the plugin did the job, but it works now, so I'll close this


'unexpected key "mod2.block1.bn1.bn.weight" in state_dict'

Hi, I want to employ the WideResNet38, InPlace-ABN for scene parsing task.
I just make some modifications, firstly I add this line to ensure the ABN is set right,
from modules import InPlaceABNSync
ABN = functools.partial(InPlaceABNSync)

Then we load the parameters like this,
       deeplab = get_segmentation_model(args.method, num_classes=args.num_classes)
       #args.restore_from =  './pretrain_model/wide_resnet38_ipabn_lr_256.pth.tar'
        saved_state_dict = torch.load(args.restore_from)
        saved_state_dict = saved_state_dict['state_dict']
        new_params = deeplab.state_dict().copy()
        for i in saved_state_dict:
            i_parts = i.split('.')
            if not i_parts[1]=='classifier':
                new_params['.'.join(i_parts[1:])] = saved_state_dict[i]

We excute  the  print(new_params['mod2.block1.bn1.bn.weight']) and get the right output.
Thus we have loaded the pretrained parameters in to the array new_params.
Then we excute ,
 deeplab.load_state_dict(new_params)
It just informed me that

'unexpected key "mod2.block1.bn1.bn.weight" in state_dict'

This is a little strange for me as I have checked that the parameters have been loaded successfully to the  new_params.

@PkuRainBow print the parameters' names of deeplab so maybe you figure out why the keys you provide are wrong.

@rotabulo I print the parameters' name of deeplab and find that there exist no key with name  'mod2.block1.bn1.bn.weight'
But the check point you provide contains the parameter with key name both 'mod2.block1.bn1.weight' and 'mod2.block1.bn1.bn.weight'.
So I am wondering whether it is related to the problem of your checkpoint?

@PkuRainBow I checked the checkpoint files and they are correct, i.e. the key you mention is not there. I suspect you are using an older version of the checkpoint.

@rotabulo Thanks!  I have solved this problem by using the new checkpoint.


NHibernate: map collection of enums using string and store as nvarchar

Could you please help me with my issue?
http://stackoverflow.com/questions/39527990/nhibernate-map-collection-of-enums-using-string-and-store-as-nvarchar

@FSou1 seems I'm late - you've already found the answer. By the way, Europlan started using NHibernate? I thought they used EF exclusively

@chester89, nice to meet you here, m8 :) last time i was at europlan they used dapper ;)

@FSou1 ah, yes
2016-09-16 16:08 GMT+03:00 FSou1<EMAIL_ADDRESS>
@chester89 https://github.com/chester89, nice to meet you here, m8 :)
last time i was at europlan they used dapper ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jagregory/fluent-nhibernate/issues/359#issuecomment-247596052,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALnfKbGPyzErQIe5Gyf4_HMIZi71KJ8ks5qqpTVgaJpZM4J-x2M
.

--
Yours faithfully,
Gleb


Question about ECCN of Bouncy Castle

Does anyone know about the Export Control Classification Number (ECCN) of Bouncy Castle?

Thanks to the mail from David Hook, the ECCN is 5D992.c now


-1 => 65535

I just ran across this oddity:
$ echo 'a=-1' | shrinko8 - - --minify --count  #good
tokens: 3 0%
chars: 4 0%
compressed: 13 0%
pico-8 cartridge // http://www.pico-8.com
version 41
__lua__
a=-1

$ echo 'a=-1' | shrinko8 - - --minify --no-minify-tokens --count  #what?
tokens: 3 0%
chars: 7 0%
compressed: 16 0%
pico-8 cartridge // http://www.pico-8.com
version 41
__lua__
a=65535

Oh, right - a recent regression due to constant propagation
I had a fix for this in my local git, so I pushed it.


Support rendering direction on vertices (stop sign, traffic_signals, etc)

[x] #3003
[x] #3815
[x] #4271
[x] #4455
[x] #4013
[x] #3610
[x] update walkthrough
[x] update presets
[x] fix turn restrictions editor
(updated: scope creep)

Something I threw together today - I still might adjust the marker style a bit, and I'm not 100% sure if it handles all of the tag cases correctly.
highway=stop with direction=forward / direction=backward

This is what it does for stop=all or direction=both

An arrow is kinda confusing to me. Is that the direction of travel or the direction the sign would be facing?

An arrow is kinda confusing to me. Is that the direction of travel or the direction the sign would be facing?

Yeah, this is a hard problem.  Currently, the arrowhead faces whatever the direction tag says: "Forward" or "Backward" - so that's "direction of travel".
Even those words are not great - I don't think an untrained mapper would really be able to understand this concept, or even why stop signs (and even moreso traffic signals) would need a direction.  The help text is not very helpful either.

I'd definitely welcome some ideas on how to make this easier.

Some other design thoughts.. whatever we do needs to:

have good contrast on top of everything
be distinct from both the oneway markers and the streetlevel photo view cones
look good very close to or under the vertex (even when the vertex has a selection halo)

@iandees I do kind of like the viewfield a little better, what do you think?

Yea, that makes more sense I think. I was also going to suggest something like that but with a flat line perpendicular to the incoming way representing the top of the sign/signal.

So this functionality is turning out to be pretty cool.  I have directions rendered for a bunch of things now (street signs, cameras, etc).
The main issue remaining is that a point with directions disappears when dragged.

I know why - it's because vertices aren't displayed during a drag, which is also #3003.

This PR is pretty wild... Here's the overview:

I wanted to show directions on vertices and put some view cones around them to do that (#3815). This is useful for mapping street signs and traffic signals.
Then I wanted to put the view cones on standalone nodes too, to support surveillance cameras and stuff that is not on a way.
To make that look good, the nodes needed to render like vertices (circles), not markers.  We already do this in wireframe mode, so I thought "no big deal".
This worked ok, but then you couldn't drag the nodes around because we don't render vertices while dragging (#3003).
So I made the vertices visible during dragging.  (also rotating, moving, etc)
Now with vertices being rendered, having text drawn on top of them while the user dragged them around was a big issue, so I had to fix the label placement. (#4271).
  I needed to get much more strict about which things can be targets and where the pointer events can go (only to certain vertices and segments along a way).
So I added invisible touch targets to control which things can listen to pointer events, and a debugging mode to make them visible and see what's going on.
These touch targets are circles around the vertices and MultiLineString geojson segments along the ways, and the targets allowed me to again say where it is ok for a line to loop back on itself (#4455)
If we can draw touch targets, we might as well also draw "nope" targets (which are parts along the line or vertices that the user specifically can't interact with.
This let me prevent the user from drawing self-intersecting polygons or lines that return back on themselves (#4013).
Then I realized we didn't need a lot of complicated drawing code anymore, which used temporary nodes and segments to get around the problems of targeting the way that the user was drawing. (2be62ff is pretty wild)
Then I realized that if we can draw targets for connecting and for not connecting, we can have the not-connecting targets do useful things like change the cursor and put the line back where it was.
I made it transitionable, because it looks cool:

Anyway that's what is new.  It will merge soon!
Some things that we can do later:

Use the nope targets to prevent users from joining things that should not join (e.g. power lines with roads) or from dragging things where they should not go (e.g. a junction that zig-zags across several roads)
Not having the temporary drawing objects anymore simplifies the draw code a lot!  This will make it easier to do square building drawing mode.  (yes one of my next priorities for early 2018)

Great work, @bhousel! I've been following along on your commits and this has gotten wild. What a great way to end the year. Happy Holidays! 🍰 🎉

🎉

Hi,
I like the new viewfield for direction but aren't forward and backward reversed?

I like the new viewfield for direction but aren't forward and backward reversed?

I don't think they are reversed.  For example, a stop sign that affects forward travel will be pointed backwards so the cars can see it:

But if you found a situation that we are rendering incorrectly, please open an issue and me know!

2018-02-20 11:49 GMT-03:00 Bryan Housel<EMAIL_ADDRESS>
I like the new viewfield for direction but aren't forward and backward
reversed?
I don't think they are reversed. For example, a stop sign that affects
forward travel will be pointed backwards so the cars can see it:

Thanks for your quick answer. Your explanation is clear but the value used
for direction and the viewfield shown confuse me. If the stop sign points
backwards relatively to the way direction and I'm mapping its direction
(where it faces to), why tag it with 'forward' as in this example and not
'backward'?
In your example, the viewfield looks backwards (I think), what is OK, after
all the stop sign faces backwards, but the value for direction is
'forward'. What am I missing?
-- Fidelis Assis

If the stop sign points
backwards relatively to the way direction and I'm mapping its direction
(where it faces to), why tag it with 'forward' as in this example and not
'backward'?

Oh yeah, I left out an important detail in that screenshot above.  The direction=forward/backward tag works relative to the direction that the way is drawn, not the direction of the sign.  A funny thing about these tags - if you reverse the way, iD also needs to (and does) reverse the direction=* tag.
See:

https://wiki.openstreetmap.org/wiki/Key:direction#Forward_and_backward
https://wiki.openstreetmap.org/wiki/Tag:highway=stop#Tagging_minor_road_stops

The main reason that I started rendering these viewfields is because it's nearly impossible (especially for casual mappers) to understand how the direction apply to a way.

2018-02-20 14:32 GMT-03:00 Bryan Housel<EMAIL_ADDRESS>
If the stop sign points
backwards relatively to the way direction and I'm mapping its direction
(where it faces to), why tag it with 'forward' as in this example and not
'backward'?
Oh yeah, I left out an important detail in that screenshot above. The
direction=forward/backward tag works relative to the direction that the
way is drawn, not the direction of the sign.

OK, I understand that. Just to make myself clearer, let's consider a oneway
road with a stop sign (or a traffic calming - that was my initial
motivation, I was confused when checking a few tagged with forward). On a
oneway road, all stop signs and traffic calmings face backwards because the
drawing direction is the same as the traffic flow (perhaps a few old
exceptions). So, I was thinking they, if tagged with direction (unnecessary
for one-way roads anyway, also for two-way roads when affect both
directions), the value should be backward. But the viewfield in this case
shows the opposite direction.
In the example below, both traffic calmings were tagged with forward,
probably because the mapper wanted to align the viewfield correctly. Again,
in this case the direction isn't necessary.
Ex:
https://www.openstreetmap.org/edit?node=349338170#map=19/-23.02126/-43.44967
I extract speedcams for GPS from OSM. If 'forward' is the right value in
these cases I'll have to change how I calculate direction when it matters
(ex: traffic calming or speed sensor acting on a single direction on a
two-way road).
A funny thing about these tags - if you reverse the way, iD also needs to

(and does) reverse the direction=* tag.

Nice. Does it also reverse if the value is in degrees?

See:

https://wiki.openstreetmap.org/wiki/Key:direction#
Forward_and_backward
https://wiki.openstreetmap.org/wiki/Tag:highway=stop#
Tagging_minor_road_stops

The main reason that I started rendering these viewfields is because it's
nearly impossible (especially for casual mappers) to understand how the
direction apply to a way.

Yes, I think it's a very helpful facility. Thanks for adding it!
-- Fidelis


Configure/Makefile: Perl error in FIPSMODULENAME declaration

Not sure whether this classifies as an OpenSSL (documentation) issue or just a question about my Perl installation. But since others may encounter the same problem, I 'll just post it as a bug report here.
After configuring (./Configure --debug)  my local copy (which is at commit 52c6c12c1cad6f1046b34f4139d1aa3e967a5530), my Makefile contains the following erroneous line:
FIPSMODULENAME=Program fragment delivered error ``Can't locate object method "basename" via package "platform" at Configurations/unix-Makefile.tmpl line 96.''

which was generated from
https://github.com/openssl/openssl/blob/52c6c12c1cad6f1046b34f4139d1aa3e967a5530/Configurations/unix-Makefile.tmpl#L90-L96
Obviously, the basename function is missing somehow. What am I missing?
My Perl version
msp@msppc:~/src/openssl$ perl --version

This is perl 5, version 30, subversion 3 (v5.30.3) built for x86_64-linux
(with 30 registered patches, see perl -V for more detail)

Copyright 1987-2020, Larry Wall

Perl may be copied only under the terms of either the Artistic License or the
GNU General Public License, which may be found in the Perl 5 source kit.

Complete documentation for Perl, including FAQ lists, should be found on
this system using "man perl" or "perldoc perl".  If you have access to the
Internet, point your browser at http://www.perl.org/, the Perl Home Page.

My Text::Template version (via Gentoo Linux Package Manager)
msp@msppc:~/src/openssl$ emerge -pv Text-Template

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R    ] dev-perl/Text-Template-1.530.0::gentoo  USE="-test" 0 KiB

Total: 1 package (1 reinstall), Size of downloads: 0 KiB

There should be a use File::basename; in that perl nugget or an earlier one.  The obvious fix is to simply add it there

But if it's just missing in the template why don't you get the error?

I see a use File::Basename (with a capital F further down):
https://github.com/openssl/openssl/blob/52c6c12c1cad6f1046b34f4139d1aa3e967a5530/Configurations/unix-Makefile.tmpl#L1246

But if it's just missing in the template why don't you get the error?

Dunno.  Maybe I have without noticing it? ....  Ah, yeah, I can see it.  I just haven't tried installing for a while, so it went unnoticed.
Anyway, the perl nuggets follow lexical order, so a use File::basename; later does no good.  Move it up or duplicate it, whichever you feel is best (perl doesn't care about duplicates).

Hmmm... I tried this, first with a capital File::Basename, then with File::basename
diff --git a/Configurations/unix-Makefile.tmpl b/Configurations/unix-Makefile.tmpl
index 342e46d24d..8d71c12e44 100644
--- a/Configurations/unix-Makefile.tmpl
+++ b/Configurations/unix-Makefile.tmpl
@@ -87,7 +87,8 @@ MODULES={- join(" \\\n" . ' ' x 8,
                 fill_lines(" ", $COLUMNS - 8,
                            map { platform->dso($_) }
                            @{$unified_info{modules}})) -}
-FIPSMODULENAME={- # We do some extra checking here, as there should be only one
+FIPSMODULENAME={- use File::basename;
+                                 # We do some extra checking here, as there should be only one
                   my @fipsmodules =
                       grep { !$unified_info{attributes}->{modules}->{$_}->{noinst}
                              && $unified_info{attributes}->{modules}->{$_}->{fips} }

and both attempts failed:

FIPSMODULENAME=Program fragment delivered error ``Can't locate object method "basename" via package "platform" at Configurations/unix-Makefile.tmpl line 97.''

FIPSMODULENAME=Program fragment delivered error ``Can't locate File/basename.pm in @INC (you may need to install the File::basename module) (@INC contains: . /home/msp/src/openssl/Configurations /home/msp/src/openssl/util/perl /home/msp/perl5/lib/perl5/5.30.3/x86_64-linux /home/msp/perl5/lib/perl5/5.30.3 /home/msp/perl5/lib/perl5/x86_64-linux /home/msp/perl5/lib/perl5 /etc/perl /usr/local/lib64/perl5/5.30.3/x86_64-linux /usr/local/lib64/perl5/5.30.3 /usr/lib64/perl5/vendor_perl/5.30.3/x86_64-linux /usr/lib64/perl5/vendor_perl/5.30.3 /usr/local/lib64/perl5 /usr/lib64/perl5/vendor_perl/5.30.1/x86_64-linux /usr/lib64/perl5/vendor_perl/5.30.1 /usr/lib64/perl5/vendor_perl/5.24.3 /usr/lib64/perl5/vendor_perl/5.24.1 /usr/lib64/perl5/vendor_perl/5.22.3 /usr/lib64/perl5/vendor_perl/5.22.2 /usr/lib64/perl5/vendor_perl/5.20.2 /usr/lib64/perl5/vendor_perl/5.20.1 /usr/lib64/perl5/vendor_perl/5.18.2 /usr/lib64/perl5/vendor_perl/5.16.3 /usr/lib64/perl5/vendor_perl /usr/lib64/perl5/5.30.3/x86_64-linux /usr/lib64/perl5/5.30.3 /home/msp/perl5/lib/perl5/5.30.1/x86_64-linux /home/msp/perl5/lib/perl5/5.30.1 /home/msp/perl5/lib/perl5/5.24.3 /home/msp/perl5/lib/perl5/5.24.1 /home/msp/perl5/lib/perl5/5.22.3 /home/msp/perl5/lib/perl5/5.22.2 /home/msp/perl5/lib/perl5/5.20.2 /home/msp/perl5/lib/perl5/5.20.1 /home/msp/perl5/lib/perl5/5.18.2 /home/msp/perl5/lib/perl5/5.16.3 /home/msp/src/openssl/external/perl/Text-Template-1.56/lib) at Configurations/unix-Makefile.tmpl line 90.
BEGIN failed--compilation aborted at Configurations/unix-Makefile.tmpl line 90.''

I guess, I'll better just wait till you come up with a PR ;-)

BTW: wouldn't it be better it ./Configure would break on this kind of errors, instead of writing garbage into the makefile`?

If should be File::Basename (capital B), and is a core perl 5 module since the beginning of time (well, since perl 5.3.7, so very early).  I have no idea how gentoo handles perl modules...  I know that in RPM based Linuxes, you have to install perl-core rather than perl (the latter is very bare bones), so that's beyond what I can do in a PR.

BTW: wouldn't it be better it ./Configure would break on this kind of errors, instead of writing garbage into the makefile?

I agree.  Strictly speaking, it's util/dofile.pl that does the work, and while it does catch some errors, there are errors, like this one, that escapes the mechanism, and I haven't figured out why.

If should be File::Basename (capital B), and is a core perl 5 module

The module exists and I can load without problem, if I do it standalone.
use File::Basename;

print basename "/foo/bar/baz"

I rather suspect that the problem has something to do with the
join(" ", map { basename **platform**->dso($_) } @fipsmodules)

snippet, because the error message talks about loading "basename" via the "platform" package.
``Can't locate object method "basename" via package "platform" at Configurations/unix-Makefile.tmpl line 97.''

FIPSMODULENAME={- use File::Basename;
                  # We do some extra checking here, as there should be only one
                  my @fipsmodules =
                      grep { !$unified_info{attributes}->{modules}->{$_}->{noinst}
                             && $unified_info{attributes}->{modules}->{$_}->{fips} }
                      @{$unified_info{modules}};
                  die "More that one FIPS module" if scalar @fipsmodules > 1;
                  join(" ", map { basename platform->dso($_) } @fipsmodules) -}

This is one of those where perl can be...  mysterious, even for me.  This has to do with how it parses statements in certain blocks (map blocks, for example), which can sometimes seem heuristic more than anything else.
If you change basename platform->dso($_) to basename(platform->dso($_)), you'll probably see a better result.

This was fixed by #14320


Promote bluez, libicall, and libell to base repo

Merge Checklist

All boxes should be checked before merging the PR (just tick any boxes which don't apply to this PR)

[x] The toolchain has been rebuilt successfully (or no changes were made to it)
[x] The toolchain/worker package manifests are up-to-date
[x] Any updated packages successfully build (or no packages were changed)
[x] Packages depending on static components modified in this PR (Golang, *-static subpackages, etc.) have had their Release tag incremented.
[x] Package tests (%check section) have been verified with RUN_CHECK=y for existing SPEC files, or added to new SPEC files
[x] All package sources are available
[x] cgmanifest files are up-to-date and sorted (./cgmanifest.json, ./toolkit/tools/cgmanifest.json, ./toolkit/scripts/toolchain/cgmanifest.json, .github/workflows/cgmanifest.json)
[x] LICENSE-MAP files are up-to-date (./SPECS/LICENSES-AND-NOTICES/data/licenses.json, ./SPECS/LICENSES-AND-NOTICES/LICENSES-MAP.md, ./SPECS/LICENSES-AND-NOTICES/LICENSE-EXCEPTIONS.PHOTON)
[x] All source files have up-to-date hashes in the *.signatures.json files
[x] sudo make go-tidy-all and sudo make go-test-coverage pass
[x] Documentation has been updated to match any changes to the build system
[x] Ready to merge

Summary

Move packages bluez, libical, and libell from SPECS-EXTENDED to SPECS folder.
Change Log

Moved directories to the SPECS folder
Bump release number

Does this affect the toolchain?

NO
Associated issues

#xxxx

Test Methodology

Local build

Stealing the standard promotion review spiel from @PawelWMS Pawel Winogrodzki FTE:
Specs from SPECS-EXTENDED rarely can be simply moved into SPECS without additional work as they have a lower bar in terms of test, spec structure, linting, etc.
Please make sure to now:

Lint the spec.
Make sure the ptests (inside the %check section) run successfully.
Remove all references to other distros (if any). In most cases we tend to keep the parts of the spec, which were enabled for Fedora and remove the other ones.
Remove the GPG keyring and signatures - we rely on the signatures.json files.
Make sure all automated PR checks pass (hard to miss, you probably won't be able to merge before the required ones are resolved).

One more item, which came to my mind right now: make sure the packages have all of their run-time dependencies satisfied.

Please re-base the change against the main branch - we don't merge directly into 2.0 except during our release process.

Rebasing these changes with a PR based on main instead of 2.0


Release 1.7.0

Adding UI Designs for Users and Groups

Following should be fixed -

Proper alignment in html and ts files.
data in html should come from ts files using angular data binding. It would be easier later to integrate with actual data.

Not valid


ODH 2.18.2 Release Tracker

Links to tracker issues for components planning updates in this.  Please update you manifests before
Release Date:
Comment Template for automation
#Release#
- component_name|release-branch/release-tag|release-notes

List of component_name:

[ ] modelregistry
[x] notebook-controller
[x] notebooks
[x] codeflare
[x] ray
[x] kueue
[x] trainingoperator
[x] trustyai
[ ] odh-dashboard
[x] data-science-pipelines-operator
[x] model-mesh
[x] kserve
[x] model-mesh-serving
[x] odh-model-controller

upstream MR: https://github.com/kubeflow/model-registry/releases/tag/v0.2.7-alpha
midstream MR: https://github.com/opendatahub-io/model-registry/releases/tag/v0.2.7
MR Python client: https://pypi.org/project/model-registry/0.2.7a1/
Model Registry Operator: https://github.com/opendatahub-io/model-registry-operator/releases/tag/v0.2.7

#Release#
modelregistry | https://github.com/opendatahub-io/model-registry-operator/tree/release/v0.2.7 | https://github.com/opendatahub-io/model-registry-operator/releases/tag/v0.2.7

#Release#

odh-dashboard | https://github.com/opendatahub-io/odh-dashboard/tree/v2.26.2-odh-release | https://github.com/opendatahub-io/odh-dashboard/releases/tag/v2.26.2-odh

#Release#

codeflare|https://github.com/opendatahub-io/codeflare-operator/tree/v1.8.0-odh | https://github.com/opendatahub-io/codeflare-operator/releases/tag/v1.8.0-odh
ray|https://github.com/opendatahub-io/kuberay/tree/v1.1.1-08-14-2024 | https://github.com/opendatahub-io/kuberay/releases/tag/v1.1.1-08-14-2024
kueue|https://github.com/opendatahub-io/kueue/tree/v0.7.0-odh-1 | https://github.com/opendatahub-io/kueue/releases/tag/v0.7.0-odh-1
trainingoperator|https://github.com/opendatahub-io/training-operator/tree/v1.8.0-odh-1 | https://github.com/opendatahub-io/training-operator/releases/tag/v1.8.0-odh-1

#Release#

notebooks| https://github.com/opendatahub-io/notebooks/tree/v1.22.0 | https://github.com/opendatahub-io/notebooks/releases/tag/v1.22.0
notebook-controller| https://github.com/opendatahub-io/kubeflow/tree/v1.7.0-15 | https://github.com/opendatahub-io/kubeflow/releases/tag/v1.7.0-15

#Release#

Kserve | https://github.com/opendatahub-io/kserve/tree/v<IP_ADDRESS> | https://github.com/opendatahub-io/kserve/releases/tag/v<IP_ADDRESS>
Model Mesh | https://github.com/opendatahub-io/modelmesh-serving/tree/v<IP_ADDRESS>-rc0 | https://github.com/opendatahub-io/modelmesh-serving/releases/tag/v<IP_ADDRESS>-rc0
ODH Model Controller | https://github.com/opendatahub-io/odh-model-controller/tree/v<IP_ADDRESS> | https://github.com/opendatahub-io/odh-model-controller/releases/tag/v<IP_ADDRESS>

#Release#
data-science-pipelines-operator | https://github.com/opendatahub-io/data-science-pipelines-operator/tree/v2.5.x | https://github.com/opendatahub-io/data-science-pipelines-operator/releases/tag/v2.5.0

#Release#
trustyai | https://github.com/trustyai-explainability/trustyai-service-operator/tree/release/1.26.0 | https://github.com/trustyai-explainability/trustyai-service-operator/releases/tag/v1.26.0

https://github.com/redhat-openshift-ecosystem/community-operators-prod/pull/5228


fix ci

Attempts to fix CI. Supersedes https://github.com/async-rs/async-std/pull/622. Thanks!

yay, test pass!


Dev

@JuliaRegistrator register

Comments on pull requests will not trigger Registrator, as it is disabled. Please try commenting on a commit or issue.


Explorer: Render Source Code

we can support adding an optional URL to package metadata and display the URL in the explorer.
Context: https://mysten-labs.slack.com/archives/C02DRAL32GN/p1652820425013529

Chatted with @tnowacki offline:

we don't want to embed the GitHub URL in the package metadata because URLs can be long and will increase the size of onchain data
Instead we will store a hash of the compiled module onchain. And then the publisher can visit the Explorer, and upload the source code, where the Explorer backend will re-compile the package and make sure the hashes match.

Duplicate of #1212


It says "Keep Trying" but the code is correct as far as I know

Challenge use-hex-code-to-mix-colors has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

I am red!

I am green!

I am dodger blue!

I am orange!

    ```
It says "Keep Trying" but the code is correct as far as I know. It doesn't accept the dodger blue hexadecimal code as correct.
![screenshot 27](https://cloud.githubusercontent.com/assets/20484432/18385017/45cabdf4-768e-11e6-840c-fac99cb05c11.png)

having same issue

This is a known issue (https://github.com/FreeCodeCamp/FreeCodeCamp/issues/10523). Closing as duplicate. Happy coding!


pipenv --support errors when trying to detect installed python versions

Issue description
Whilst submitting  #3113 I discovered that pipenv --support errors out whilst discovering python instalations.
My guess is that it's either related to pyenv; or something funky in my path.
Expected result
A listing of python versions that have been installed.
Actual result
Based on the comment on 3113 I tried against the latest update-vendor branch. (ignore the --prefix ... that's from another bug)
$ pip install --prefix=/home/opal/.local/ -e git+https://github.com/pypa/pipenv.git@update-vendor#egg=pipenv
Obtaining pipenv from git+https://github.com/pypa/pipenv.git@update-vendor#egg=pipenv
  Cloning https://github.com/pypa/pipenv.git (to revision update-vendor) to ./src/pipenv
Branch 'update-vendor' set up to track remote branch 'update-vendor' from 'origin'.
Switched to a new branch 'update-vendor'
  Installing build dependencies ... done
Requirement already satisfied: pip>=9.0.1 in /home/opal/.local/lib/python3.6/site-packages (from pipenv) (18.1)
Requirement already satisfied: certifi in /home/opal/.local/lib/python3.6/site-packages (from pipenv) (2018.10.15)
Requirement already satisfied: setuptools>=36.2.1 in /home/opal/.local/lib/python3.6/site-packages (from pipenv) (40.4.3)
Requirement already satisfied: virtualenv-clone>=0.2.5 in /home/opal/.local/lib/python3.6/site-packages (from pipenv) (0.4.0)
Requirement already satisfied: virtualenv in /home/opal/.local/lib/python3.6/site-packages (from pipenv) (16.0.0)
Installing collected packages: pipenv
  Found existing installation: pipenv 2018.10.14.dev0
    Uninstalling pipenv-2018.10.14.dev0:
      Successfully uninstalled pipenv-2018.10.14.dev0
  Running setup.py develop for pipenv
Successfully installed pipenv
$ ( cd ./src/pipenv; git rev-parse HEAD )
912d9c5e9c0f72a69ec2ad2e4b204f4f9495d71c
$ pipenv --support
(snip ... see below)

Pipenv version: '2018.10.14.dev0'
Pipenv location: '/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv'
Python location: '/usr/bin/python3'
Python installations found:
Traceback (most recent call last):
  File "/home/opal/.local/bin/pipenv", line 11, in <module>
    load_entry_point('pipenv', 'console_scripts', 'pipenv')()
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 764, in __call__
    return self.main(*args, **kwargs)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 717, in main
    rv = self.invoke(ctx)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 1114, in invoke
    return Command.invoke(self, ctx)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 956, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/decorators.py", line 64, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/core.py", line 555, in invoke
    return callback(*args, **kwargs)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/click/decorators.py", line 17, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/cli/command.py", line 142, in cli
    get_pipenv_diagnostics()
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/help.py", line 33, in get_pipenv_diagnostics
    python_paths = finder.find_all_python_versions()
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/pythonfinder/pythonfinder.py", line 135, in find_all_python_versions
    major=major, minor=minor, patch=patch, pre=pre, dev=dev, arch=arch, name=name
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/pythonfinder/models/path.py", line 268, in find_all_python_versions
    values = list(self.get_pythons(sub_finder))
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/pythonfinder/models/path.py", line 219, in get_pythons
    reverse=True
  File "/home/opal/piwik3/piwik-release-build/src/pipenv/pipenv/vendor/pythonfinder/models/path.py", line 217, in <genexpr>
    (p for p in self._filter_paths(finder) if p.is_python),
AttributeError: 'list' object has no attribute 'is_python'

Thank you so much for testing this. I finally got the rest of the broken things fixed after a full week of debugging, but I didn’t want to cut a release that is still broken!

Would you mind pulling and trying again?

The latest code works exactly as intended:

Pipenv version: '2018.10.14.dev0'
Pipenv location: '/home/opal/temp/src/pipenv/pipenv'
Python location: '/usr/bin/python3'
Python installations found:

3.6.5: /home/opal/.pyenv/shims/python3.6
3.6.5: /home/opal/.pyenv/shims/python3.6m
3.6.5: /home/opal/.pyenv/shims/python3
3.6.5: /usr/bin/python3
3.6.5: /usr/bin/python3.6m
2.7.15: /home/opal/.pyenv/shims/python
2.7.15: /usr/bin/python

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.6.5',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '4.18.15-1-default',
 'platform_system': 'Linux',
 'platform_version': '#1 SMP PREEMPT Thu Oct 18 08:56:17 UTC 2018 (5a53676)',
 'python_full_version': '3.6.5',
 'python_version': '3.6',
 'sys_platform': 'linux'}

(snip)


Improve error message around record initializers

Category
Other
Describe the feature you'd like to request
Improve the errors around record initializes in ASTNode<Option<cst::Primary>>::to_expr_or_special
Describe alternatives you've considered
Leave the errors as they are
Additional context
No response
Is this something that you'd be interested in working on?

[X] 👋 I may be able to implement this feature request
[ ] ⚠ This feature might incur a breaking change

Can we have an example of an error message that needs improving, and an input that triggers it?  (I don't doubt the messages are bad, but an example will help focus the issue)

One example,
permit(principal, action, resource) when { {a:}};
  × failed to parse policy set
  ╰─▶ unexpected token `}`
   ╭─[<stdin>:1:1]
 1 │ permit(principal, action, resource) when { {a:}};
   ·                                               ┬
   ·                                               ╰── expected `!`, `(`, `-`, `[`, `{`, `action`, `context`, `else`, `false`, `forbid`, `has`, identifier, `if`, `in`, `is`, `like`, number, template slot, `permit`, `principal`, `?principal`, `resource`, `?resource`, string literal, `then`, `true`, `unless`, or `when`
   ╰────

The list of expected tokens is simply too long.

Some weird errors come from the fact that the attribute Name is actually parsed as an expression.
For instance, here it claims that context isn't a valid attribute name (but it actualy is).
jkastner@dev % cedar check-parse <<<'permit(principal, action, resource) when { {(context): 1} };'
  × failed to parse policy set
  ╰─▶ not a valid attribute name: `context`
   ╭────
 1 │ permit(principal, action, resource) when { {(context): 1} };
   ·                                              ───────
   ╰────
  help: attribute names can either be identifiers or string literals

Error:   × record literal has invalid attributes
   ╭────
 1 │ permit(principal, action, resource) when { {(context): 1} };
   ·                                            ──────────────
   ╰────

Or an unfortunately long expected token list when trying to parse the attribute name. It should expect an identifier, but the expected tokens are based on the expression grammar.
jkastner@dev % cedar  ./target/debug/cedar check-parse <<<'permit(principal, action, resource) when { {a: 1,'
  × failed to parse policy set
  ╰─▶ unexpected end of input
   ╭─[<stdin>:1:50]
 1 │ permit(principal, action, resource) when { {a: 1,
   ·                                                  ▲
   ·                                                  ╰── expected `!`, `(`, `-`, `[`, `{`, `false`, identifier, `if`, number, `?principal`, `?resource`, string literal, or `true`
   ╰────


Closes: #1

Fix validation name into validations.py file.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it


Web Portal: Fix status display in jobs table

Introduced in #691
convertState is used to render status column, which I copied as getHumanizedJobStateString by mistake.

Coverage decreased (-0.5%) to 50.09% when pulling 8bf5bfba75714ee8fac138491599c58b16d2db30 on qixcheng/web-portal-fix-job-status into 389652b2926f20e2397975daa6a920fffd80652a on master.

Coverage decreased (-0.5%) to 50.09% when pulling 8bf5bfba75714ee8fac138491599c58b16d2db30 on qixcheng/web-portal-fix-job-status into 389652b2926f20e2397975daa6a920fffd80652a on master.

Coverage decreased (-0.5%) to 50.09% when pulling 8bf5bfba75714ee8fac138491599c58b16d2db30 on qixcheng/web-portal-fix-job-status into 389652b2926f20e2397975daa6a920fffd80652a on master.


Collapsible comments

+/- to show/hide comment thread.
After 5 levels, link to a new page instead of displaying entire thread.
http://demcra.com/r/test/1000020_Inception_Test

done


[BEAM-9846] Clean up references to unused native BQ source/sink experiment in Dataflow

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @yifanzou

Run Java PreCommit


Documentation site doesn't use Ember or a Service Worker

If you had a gh-pages for it, I would like to at least look and see how hard it would be to see do both of these things and then PR.

Just realized that it's in the doc directory. Will take a look.

I'll keep it simple for now. Might make it a service worker some time.


Install using the Python terminal provided by VS Code if it is configured

Use terminal provided by “Python: Create Terminal”, provided by https://github.com/Microsoft/vscode-python.
If not available, perhaps prompt the user to perform a manual install?

:tada: This issue has been resolved in version 0.85.0 :tada:
The release is available on:

v0.85.0
GitHub release

Your semantic-release bot :package::rocket:


Remove superfluous english sentence from german translation

Probably a left-over from translating.

yeah definitely -- thanks for the catch!

just pushed this to prod, should see the update after a refresh

Thanks, looks good!


[HOLD Mentions v2] Support mentions in editing comments

We want to update the edit comment flow to support editing/adding mentions. This does not mean adding support for the auto-complete component. Just parsing the mentions in both directions. This means that when someone goes to edit a comment with a user or report mention we should:

convert that from html to markdown so that it can be edited
perform the proper markdown to html conversion when the edited comment is saved so that if mentions have been added/removed we properly send that to the API

This is covered in the mentions v2 design doc here.
HOLD for https://github.com/Expensify/App/issues/39549

Held on https://github.com/Expensify/App/issues/39549

Sounds good. Just FYI, the back-end for this hasn't been done yet.

@puneetlath any update on the backend? I think my PR works as expected but would like to get confirmation from your side :)

Not yet! Will probably be a few more days before I can start the back-end. Are you ready for a review?

Yup, already marked PR as ready for review, but I wonder if the backend is a blocker for this 🤔 If not I'll notify the reviewer :)

Yup, already marked PR as ready for review, but I wonder if the backend is a blocker for this 🤔 If not I'll notify the reviewer :)

@puneetlath kind bump :)

Does it work without the back-end? Like when you refresh the page, is it still shown as mentions?

@puneetlath Yup, you can see it here or in the PR description :)
https://github.com/Expensify/App/assets/61014013/70ac925c-460d-443a-8bed-7ae3d58b3d25

@war-in are you aware of some "Cannot read properties of undefined (reading 'accountIDToName')" errors ? I believe a few people are seeing this, not sure exactly the flow

Putting all the details in slack here https://expensify.slack.com/archives/C04878MDF34/p1715699038416249 please @war-jump there if you can, as this is breaking the app in production

@puneetlath @Beamanator mentioned issue will be resolved in this PR. Can I mark mine as ready for review? I've shown here that it works without the backend

Yes, sounds good for me. Let's get it in review.

this looks like it can be closed at this point

Hi @MitchExpensify
Sorry for not updating here 🙏 We are working on the PR closing this issue here (updates were posted there)

@puneetlath I think we can close it now since #40565 was merged :)

@rlinoz mind confirming that we're all done here?

Let's wait until it gets deployed to prod, but also we might have a case we didn't account for https://github.com/Expensify/App/issues/42521#issuecomment-2173848054

@eh2077 was the C+

Payment summary:

$250 C+ @eh2077 (Upwork)

Checklist

[x] [@eh2077] Please propose regression test steps to ensure the new feature will work correctly on production in further releases. This is a new feature and I don't think we need a regression test here

@eh2077 Upwork job created here: https://www.upwork.com/jobs/~012066a1fb40a4a923, offer sent!

Paid and contract ended


Only seeing one entity

Hi,
I recently installed v1.1.0 via Hacs. Did the setup with my DSN, u/p and I only see one "Entity" with the status and nothing else. I have no matching devices, however.
I didn't see anything on the documentation as to when the "one device with multiple attributes" screencap on your README would appear.
https://i.imgur.com/8ymGBkc.png

Click on that Ecowater entity that you now have.  It should bring up a window showing that it is Online.  Now click on the Attributes dropdown below the History and it will bring up a list of attributes of that entity.
https://imgur.com/a/HTrr3rx

Good afternoon!
I am having a problem with getting my entity to work. It keeps stating it is unavailable in the status. I can successfully log in to their website using the username/password. Any ideas?

Answered by @mblauser.
Thanks 👍

I have kind of the same problem since updating to the latest version.
I had created a card where I had a view on all attributes/entities
Since the updates all entities are unknown: https://imgur.com/a/bZXPiXK
However when I follow the procedure mentioned above all attributes are available.
Any insights?

I am still getting unavailable, I have removed and added back multiple times.  Any ideas on why it is not connecting online?


Document criterias for changes / additions / removals

Should align with the mission statement (#7) and align with existing users and as a rule of thumb not break them – just more accurately describe them.
A way to gradually introduce more breaking changes, to allow the ecosystem to evolve, should also be documented.
One example of such a one is the no-var rule that was introduced as error in standardjs 16.0.0 and then relaxed to warn in 16.0.1: https://github.com/standard/standard/issues/633#issuecomment-719097551 (It is still kept as warn in neostandard for now, to ease the transition to neostandard)
One additional criteria should be to document the reason for having the rule, see #19
It may be that we want to do this process through a neostandard/rfcs repository rather than directly through this repo? Not sure which would be the best though.

Fixed by #94


Avoid propagating BindingKind::Global and BindingKind::Nonlocal

Summary
This PR fixes a small quirk in the semantic model. Typically, when we see an import, like import foo, we create a BindingKind::Importation for it. However, if foo has been declared as a global, then we propagate the kind forward. So given:
global foo

import foo

We'd create two bindings for foo, both with type global.
This was originally borrowed from Pyflakes, and it exists to help avoid false-positives like:
def f():
    global foo

    # Don't mark `foo` as "assigned but unused"! It's a global!
    foo = 1

This PR removes that behavior, and instead tracks "Does this binding refer to a global?" as a flag. This is much cleaner, since it means we don't "lose" the identity of various bindings.
As a very strange example of why this matters, consider:
def foo():
    global Member

    from module import Member

    x: Member = 1

Member is only used in a typing context, so we should flag it and say "move it to a TYPE_CHECKING block". However, when we go to analyze from module import Member, it has BindingKind::Global. So we don't even know that it's an import!

From the ecosystem checks:

airflow/configuration.py:1513:13: F841 [*] Local variable FERNET_KEY is assigned to but never used

Through https://github.com/apache/airflow/blob/bdfebad5c9491234a78453856bd8c3baac98f75e/airflow/configuration.py#L1805 and seeing AIRFLOW__CORE__FERNET_KEY else in the code it looks like they are using that module level argument elsewhere through metaprogramming. It's a bit of a problem because there's no __all__, but marking this as an unused local variable is wrong.
Otherwise the ecosystem checks improve.


sip auto register

Now, I find baresip can't auto register. I use wireshark to capture sip package, I can't find any sip register package in 30 minutes.

In my memory, baresip can auto register. It seems a bug in one update. My barsip version is 0.5.7.
Do I need to invoke register api periodicity?

Hi @cloud-jxy,
Would you please post the content of ~/.baresip/account and ~/.baresip/config?

hi @cloud-jxy
can you please test with latest version of libre and baresip from GIT ?
please include the debug log output.
Alfred

please re-open if you can provide more information.

I don't use ~/.baresip/account. I  invoke  ua_alloc to register acount. The config file is default, auto generated.
int ua_alloc(struct ua **uap, const char *aor)
ua_alloc(NULL, aor);
aor likes sip:1122:test1234@<IP_ADDRESS>;audio_codecs=pcma. 1122 is account, test1234 is password.


Quick fixes for multiple required arguments could be more useful

void example() {
  // https://dart.dev/tools/diagnostic-messages#missing_required_argument
  Listener();
}

class Listener {
  final void Function() onA;
  final void Function(String) onB;
  final void Function(String, String) onC;

  Listener({
    required this.onA,
    required this.onB,
    required this.onC,
  });
}

... produces the following quick-fixes:

I would have assumed Add all required arguments was an available quick fix, but it was not.

When dealing with a class with a lot of named parameters - like widgets, I find myself copying and pasting other instances of that widget and filling in the differences. As error prone as that is, it feels 10x faster than trying to correctly autocomplete and quickfix a new instance.


Adding all the django configurations

basically using the configurations that Ido added before...

@TalReinfeld
Please rebase push --force


Allow SymbolicUtils v2

Tests pass locally, though it doesn't seem tested because it didn't catch the similarterm stuff.


Document openssh patches

There are quite a few patches in the Dockerfile applied to openssh before it is compiled.
Could these patches, their source and purpose be documented, please? As the bastion server security is very important, any changes to the openssh implementation should be documented so it can be audited.
Also, would the current patch set prevent upgrading the Dockerfile to use the current openssh version V_7_6_P1  instead of V_7_4_P1 which is used currently?
Thank you!

@marji good points. We should document what those are needed/wanted.
For now, here's an explanation:
We use this so we have SSH_ORIGINAL_COMMAND available during pam auth.
https://github.com/cloudposse/bastion/blob/master/patches/openssh/original-command.diff
We use this to hide the SSH version so it's not announced to port-scanners.
https://github.com/cloudposse/bastion/blob/master/patches/openssh/obfuscate-version.diff
All the rest of the patches are from the original alpine image, here:
https://git.alpinelinux.org/cgit/aports/tree/main/openssh
Upgrading is hopefully straightforward, but we haven't tried yet. Patches might need to be regenerated. If you give it a go and submit PR, we'll gladly accept it! Thanks
-Erik


Sort @deprecated completions lower than others

Fixes #43796

There's "something" in VSCode that's still re-ordering the suggestions. Debugging https://github.com/microsoft/vscode/tree/main/extensions/typescript-language-features shows that the server response looks correct:

But the suggestions shown to the UI differs, b is not at bottom:

May we add more behavior about the order of completion item?
We have https://github.com/microsoft/TypeScript/pull/40079 sometime ago.
It's seems very helpful for user experience.
BTW: No need to do in this PR.

CC @uniqueiniquity @jessetrinity

There seems to be another PR #43880 doing this same thing

Ah, do we just create overriding PRs like that 😬