fix: stop CRP controller from overwriting CRP annotations when reconciling CRPs of the PickN placement type

Description of your changes
This PR fixes an issue where the CRP controller would overwrite CRP object annotations when reconciling CRPs of the PickN placement; when this issues occurs, the CRP object would lose its CRP generation annotation, which would fail all future reconciliation loops.
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested

[x] Unit tests
[x] Integration tests
[x] E2E tests

Special notes for your reviewer

Merged. If there's any concern, please let me know.


fix: clean up SelectMenu Scrolling

The SelectMenu had several odd behaviors:

The trigger button had no max size and would wrap text, causing inconvenience.
Giving each menuItem a unique random uuid created difficulty because of the birthday paradox. Changed to use refs instead in much cleaner form.
The popper could shift left/right if the size of the trigger button changed.
offset sometimes wasn't perfect.

Note: this does not fix the scroll the whole page on center item errors.

Coverage increased (+0.2%) to 87.573% when pulling c26099538e9473eb2316c9400456e05946787af4 on fix--cleanup-SelectMenu-Scrolling into f9996ece7ff486de85192b9e3b7b40ee68a25f8e on master.

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update controlplane.rb to 1.5.8

Thank you for the contribution. It needed some fixes, so they were made in 1ea862b074dc91de4629818b4bcc733bf54745b2. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

Thanks for fixing


ODR + Pint integration

When trying to run the ODR algorithm on numpy arrays with pint units, the ODR algorithm "strips" the units away from the result.
It would be nice if the ODR algorithm could play nice with pint and output the parameters with respective units.
Reproducing code example:
import pint
import numpy as np
from scipy.odr import Model, RealData, ODR

ureg = pint.UnitRegistry()

def parabolic(a, x):
    return a[0] + a[1] * x + a[2] * x ** 2

a = np.array([10, 0, 1])
x = np.arange(0, 10)
xerr = np.random.normal(scale=0.1, size=10) * ureg.sec
y = parabolic(a, x)
yerr = np.random.normal(scale=0.01, size=10) * ureg.sec

model = Model(fcn=parabolic)
real_data = RealData(
    x=x * ureg.sec, y=y * ureg.meter, sx=xerr * ureg.sec, sy=yerr * ureg.meter
)
odr = ODR(data=real_data, model=model, beta0=np.ones(shape=3))
output = odr.run()

a = output.beta
chi2 = output.sum_square
print("a=", a)
print("chi2=", chi2)

Running this code presents the following error message:
C:\Users\saroa\repositories\sandbox\venv\lib\site-packages\numpy\core\_asarray.py:85: UnitStrippedWarning: The unit of the quantity is stripped when downcasting to ndarray.
  return array(a, dtype, copy=False, order=order)

Version information:

Pint - 0.11
Numpy - 1.18.2
Scipy - 1.4.1
Python - 3.7.2

I think Pint will have that problem with many functions in scipy and other packages. Code that's not units-aware may or may not work, that's going to be accidental either way.
Pint is numpy and pandas aware to a large extent, but once you have plain Python code + compiled extensions, it just can't be expected to work.
@hgrecco I assume you get this kind of question a lot, is there a generic answer to point to somewhere?

This indeed a problem. I described this in a numpy issue a few years ago (see https://github.com/numpy/numpy/issues/4072). At that time, the response was that "np.asanyarray() allows only strict subclasses". This (and similar) functions are used extensively in numpy/scipy code and they efectively erase the units.
This is why I developed wraps which allows me to convert any code into a units aware code without loosing units nor performance. It works really nice but it requires some extra code.
I am not sure if this has changed. NEP0-18 has been a great step in the right direction and we are already using it in many pint and build a small infrastructure that makes it very easy to add new supported functions.
From the very beginning we have correctly supported a large number of numpy functions in "the right way" (i.e. numpy.cos(1*ureg.meter) raise an exception. I would be very happy to improve our connection to Numpy and Scipy.

Hey guys, thank you for the very elaborated answer!
It seems like I have a fundamental misconception in my code: I don't know the units of the beta parameters in advance. I was thinking that Pint would conclude the units of the beta parameters from context, but it is not possible ATM. Even if ODR didn't strip out the units, the only way the beta output would contain units is if you specifically indicate the units in the beta0 variable.
Therefore, this is no more than wishful thinking for now. If there's a way you think I can help make that happen, I'll be glad to help!

This is why I developed wraps which allows me to convert any code into a units aware code without loosing units nor performance.

That does look nice.

I am not sure if this has changed. NEP0-18 has been a great step in the right direction and we are already using it in many pint and build a small infrastructure that makes it very easy to add new supported functions.

For pure Python code in SciPy we should at some point bypass asarray/asanyarray for inputs that support array protocols. That doesn't help when going into compiled code though, because there NumPy array are expected - so wraps is probably still the solution there.


Developers views Management

Created the following views

View all the developers
View a single developer
Create a developer
Add/remove to the developer

Kindly don't put attention to the navbar on top  just for testing THank you


Consistent config format between

It might make sense to unify the configuration format with klipper and moonraker.
The current config format is not unified with klipper, which might lead to user input errors and undesired behaviour.
timelapse {
heigth = 0
enabled = true
}
The config in example for the timelapse module uses curved brackets, tabulation and the ability to disable a featur by setting a
"enabled = true" flag.
With klipper and moonraker, a feature is enabled by having it present in the config in square brackets, i.e.:
[display_status]
[pause_resume]
The disabling of a feature is simply omitting it from the config alltogether.
Parameters are set to 0 per default, and unless entered into the config are inactive, as in
[stepper_x]
gear_ratio: 5:1
Unless gear ratio is present in the config, it is set to 0 and not used.
Referring to the initial example with the timelapse module of the bot,
it should look like this:
[timelapse]
#height: 0.2
It should be considered, that should this change be implemented, it will break backwards compatibility with existing installations. Nevertheless, in my opinion, it is urgently needed to make this change while the userbase is small, and the available features are minimal.
Input from @zztopper as well as @suglosta and @RushOnline is more than welcome.

Seems legit, I completely agree with @aka13-404.
We already had a conversation with @nlef on this, but he said he's used to this format and finds it convenient as he is a Java developer.
Also, this format makes any custom config additions and\or changes very hard to support in code (I tried once to add a section for gcode macros running).

Ok. I'll rework config to be more like python configs & klipper/moonraker configuration logic.

The following is a preview of the suggested config change:
[bot]
server: "localhost"
chat_id: "68266244"
bot_token: "1826261904:AAF1207YSq14OzlcECqultI59uvIH2rhVHc"
light_device: "chamber_led"
debug: true

[camera]
host: "http://<IP_ADDRESS>:8080/stream.mp4"
flipVertically: false
flipHorisontally: false
videoDuration: 125
light_control_timeout: 2

[progress_notification]
percent: 5
heigth: 5
time: 5
min_delay_between_notifications: 60

[timelapse]
basedir: "/mnt/stationary/timelapseusb"
cleanup: true
heigth: 0.2
time: 5
target_fps: 30

[telegram_ui]
hidden_methods: "/gif",
enabled_macros: "MANUAL_BED_LEVEL"

@aka13-404 @zztopper maybe this should be closed? new config format was introduced in release 1.0.0


Open and fill is showing item in 1password

Diagnostic
Workflow version
2022.20
Alfred version
5.0.6
macOS version
13.1
Architecture
arm64
Preferences
Default
1Password version
8.9.10
op version
2.7.1
Launchd job
Loaded: -	0	com.alfredapp.vitor.1password
Installed
Debugger output
[11:15:54.409] 1Password[Script Filter] Processing complete
[11:15:54.467] 1Password[Script Filter] Passing output '' to Conditional
[11:15:54.468] 1Password[Conditional] Processing complete
[11:15:54.469] 1Password[Conditional] Passing output '' to Arg and Vars
[11:15:54.469] 1Password[Arg and Vars] Processing complete
[11:15:54.469] 1Password[Arg and Vars] Passing output 'Open and Fill' to Run Script
[11:15:54.470] 1Password[Run Script] Processing complete
[11:15:54.471] 1Password[Run Script] Passing output 'openandfill' to Conditional
[11:15:54.471] 1Password[Conditional] Processing complete
[11:15:54.471] 1Password[Conditional] Passing output 'openandfill' to Conditional
[11:15:54.472] 1Password[Conditional] Processing complete
[11:15:54.472] 1Password[Conditional] Passing output 'openandfill' to Open URL

Terminal output
n/a

More details
Apologies as I am not super familiar with Alfred workflows.
After upgrading both Alfred and this workflow to the latest this morning (5.0.6 & 2022.20 respectively, the behaviour of the workflow changed.  Instead of opening the URL of items in the default browser, it started opening the item in 1password itself.
Goofing around in the Alfred debugger, I isolated it to the If no URL available, view in 1Password step.  (https://github.com/alfredapp/1password-workflow/blob/f480528ad5eb544a21c1aae6faccb0f96bdb6361/Workflow/info.plist#L1813-L1824 I believe)
The default is to check if var.url is false and if so open and fill.  This seems funny to me.  I would think you would  want the opposite.  If the value is not empty.  And is false will check for emptiness, but is true does not check for non-emptiness (only 1, yes, true)
So I tried matches regex == .+ (could probably be improved) and this brought back the behaviour I was expecting -- that openandfill launches the url in the default browser.
I tried reseting, reinstalling, rebooting etc first.  Did my workflow get goofed, or did I misunderstand something, or is this a bug?
Thanks!

Right you are! I did some tests after 2022.19 and must’ve forgotten to change back. Could you check 2022.21?

I did a clean slate remove / install from https://github.com/alfredapp/1password-workflow/releases/tag/2022.21 (I didn't see it on the gallery yet) and I can confirm it is now working as expected again.
Thanks for the super quick reply and plugin!  I appreciate it!

I didn't see it on the gallery yet

By design! So you could test.

Thanks for the super quick reply and plugin! I appreciate it!

Thank you as well for the thorough and fast feedback. Have a great week!


VS2019 Port

Hi, Could you please specify the port for Visual Studio 2019 Remote Debugger?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 75c19e4f-4623-c462-e399-008a85ea1523
Version Independent ID: bc0b82bb-acc8-7438-7b18-5165de39d847
Content: Remote Debugger Port Assignments - Visual Studio (Windows)
Content Source: docs/debugger/remote-debugger-port-assignments.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@paulovila -- thank you for your question.  The port is 4024.  I am updating this article.

Thank you for your feedback.  I have updated the documentation and it should be live by tomorrow.


Rename master -> main.

Motivation
Use better terminology.
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

@bors-libra r+

:pushpin: Commit a63452e has been approved by metajack

:hourglass: Testing commit a63452e508693d1c893bb9d9d757115710416c07 with merge 0969652e6e2e314981219618890e289697e06057...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: metajack
Pushing 0969652e6e2e314981219618890e289697e06057 to master...


Gamemode

"gamemode (spectator|survival)": "allow" does not let me change gamemode. The /gamemode works, but no sub commands show or work.

Merging these related issues, so closing as a duplicate of #37 🙂


Change/add to current functions;  BlueBackPixel.java

Changed servo functions so wait is optional, if used, only waits until servo reaches position +5 ms. Implemented change, added middle/right programs. All need tuning on field

@gwarby The following documentation specs that getPosition() simply returns the last commanded position.  I think your idea is superior to the manual wait times, but I'm not sure servos have a way to wait for "not busy" or "position ~= x".
ServoImpl:getPosition()


update gcloud and go gae sdks to latest

@jprobinson, this seems to fix @brianfoshee 's issue regarding apparently duplicate app.ymls.
Is there a reason they were pinned at these versions? Or is it just because there's no download link for the latest one?

this is deployed to jrynyt2/drone-gae:latest

ok. I can't merge


Is it possible to query another database rather than the default one?

In my application, I have multiple IndexedDB databases, so a interesting feature is the ability to switch my database.
I attempted to write
USE "rxdb-dexie-eva_notebook--0--blocks";

but it didn't work.

IndexedDB storage itself has namespace feature but.. for now JS package does not support it.
I think that the custom namespace functionality can fit your needs, would it be right approach?
Then, I consider to add namespace parameter for loadIndexedDB function.

It works for me. Thank you!

It works for me. Thank you!

Ok that's simple task so I'll let you know after the work

I've managed to integrate GlueSQL with my open-source computational notebook. This  could serve as an example.
It could potentially be a playground for GlueSQL users.

@sanchezcarlosjr
npm i<EMAIL_ADDRESS>
can you try 0.14.4-rc.1 version? then you can use namspace when you call loadIndexedDB.
because it may take some time to release 0.15 so I've released with rc tag.

It works. Thank you. I'm closing the issue.


Feature: Display even amount of recommended files in homepage

What feature or improvement do you think would benefit Files?
A tiny thing that just bugs me in the back of my head, and that's that the recommended items appearing under Quick Access in the homepage aren't always even.

that one extra folder named "Assignment" appears

Even when resizing the window, the cards don't reorganise.

Requirements

Limit the cards to display only an even amount of recommended items

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.4541
Comments
No response

I understand that it can be a bit annoying when the items under Quick Access aren't even. However, the number of displayed items depends not only on the recent folders (which happens to be an even number), but also on the pinned folders. To maintain an even number of items, I recommend having an even number of pinned folders. If we were to change the behavior to display only three recent folders, it would introduce inconsistency and limit functionality.


Pin nightly version to un-break coverage

This PR just pins the nightly version to September 2nd, as that was the last working version before code coverage started failing again.
We should be able to revert this change in the near future once the issue works itself out.
Test Plan
See a green :heavy_check_mark: beside the coverage CI job.

I also see
fail_ci_if_error: true
Maybe setting it to false is all that we need? Would be nicer to have nightly auto update and just not get coverage while its broken.

From what I understand, the fail_ci_if_error: true just makes the job fail if the there is an issue uploading the report. Right now, the job is failing because of the Generate code coverage step so I'm not sure if it would help.

Codecov Report

Merging #1095 (721939e) into main (fefe902) will increase coverage by 0.10%.
The diff coverage is 66.06%.

@@            Coverage Diff             @@
##             main    #1095      +/-   ##
==========================================
+ Coverage   64.46%   64.56%   +0.10%
==========================================
  Files         123      125       +2
  Lines       24387    25102     +715
==========================================
+ Hits        15720    16206     +486
- Misses       8667     8896     +229

Give that a coverage report has been included here, doesn't this mean it's working...?

It is, but I have a better fix in #1096


Added Deep Learning Courses

Closes #680

please also add in all course json file

please create another pr , there is a conflict


integration cannot be loaded

Hello,
I am running HA on a raspbianOS. I have installed the bluetooth dependencies.
I have added this integration successfully via HACS.
But when trying to add it in HA I get this issue: "Config flow could not be loaded"
Which I can only dismiss by clicking OK.
If I try to add it again I get an endless loop of "Adding integration..." (even after 10min it does not disappear).
I have also added the additional logs in configuration.yaml. But all I got was this message:
2021-04-30 12:11:52 ERROR (MainThread) [homeassistant.config_entries] Error occurred loading configuration flow for integration awox: No module named 'Crypto'

had similar issue yesterday in my new test env. Looks like some requirements where changed in the core.
Next release should fix this.


Chrisda to Main

Docs Build status updates of commit 0c7fc94:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MailUser.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 924f280:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-AntiPhishPolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-MailUser.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-TenantAllowBlockListItems.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Created Inline time picker

Closes https://github.com/EnsembleUI/ensemble/issues/1549

closing as a wrong widget was implemented.


[Blog Post - WG App Delivery] Platform Engineering in 2024 - Industry Trends and Emerging Focus. An holistic proposal for Internal Developer Platforms - Platform Engineering ++changed internal platform to internale developer platform in the para…

Hi all! after some weeks and the super useful feedback from a lot of people (thanks!) here is the PR for the blog post discussed in the issue https://github.com/cncf/tag-app-delivery/issues/586
I hope that I added all file correctly :-)
Thanks for the support
Note: I deleted the previous PR #695 because I haven't signed the push

@giulioroggero I've read through the post and I like the overall direction. The library metaphor doesn't quite make it for me, but that may just be my opinion.
I do like the way you are pushing to bring more to the table with more than just infrastructure provisioning and app deployment.
Maybe you could have a first line be something along the lines of this to complement the title: "Should internal platforms be limited to self-service infrastructure provisioning and app deployment?"

@dstengle good suggestion


ssh_config does not contain all nodes created by k2

I created a k2 cluster using the default config created by
./up.sh --generate

I noticed that the generated ssh_config does not all of the nodes created by k2
rodin:k2 dwat$ grep Host\  ~/.kraken/dew/ssh_config
Host etcd-1
Host etcd-2
Host etcd-3
Host etcd-4
Host clusterNodes-1
Host clusterNodes-2
Host clusterNodes-3
rodin:k2 dwat$

Was this a conscious design choice? Or is it a bug? While I agree that in principle there should be no reason to log on to the etcEvent or master nodes, in practice it can be very useful to be able to do so for debugging purposes.

in my case it container only 2of special Nodes.
I get the config with default config which generated by k2.
Host specialNodes-1
  HostName ec2-54-245-20-219.us-west-2.compute.amazonaws.com
  Port 22
  User core
  IdentityFile /root/.ssh/id_rsa
Host specialNodes-2
  HostName ec2-54-200-233-94.us-west-2.compute.amazonaws.com
  Port 22
  User core
  IdentityFile /root/.ssh/id_rsa

and in my second try it only contains ETCDs
I think there are some problem when generating ssh_config

I only see the etcd, etcdEvents, clusterNodes, specialNodes
Where are the Masters, and APIServers? (or whatever setup we are using for the control plane?)

two issues:

master nodes aren't being found (unsure why)
etcd nodes aren't being found (they are no longer autoscaled so the filter is wrong.

will continue to investigate this tomorrow.

PR available ^^


Update README.md

Add build status

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


How to present the FUIPhoneAuth without showing the welcome screen with the 'Sign in with phone' button?

When I present the authViewController, it displays a screen with a button to 'Sign in with phone'. Is it possible to bypass this screen, and go straight to the 'Enter Phone Number' screen?
Specs:

Swift
iOS version: 10.3
Firebase iOS version: 4.0.2
FirebaseUI version: 4.0.0

let authUI = FUIAuth.defaultAuthUI()
// You need to adopt a FUIAuthDelegate protocol to receive callback
authUI?.delegate = self
authUI?.isSignInWithEmailHidden = true

let providers: [FUIAuthProvider] = [
    FUIPhoneAuth(authUI:FUIAuth.defaultAuthUI()!)
]

authUI?.providers = providers

let authViewController = authUI?.authViewController()

self.present(authViewController!, animated: true) { }

@pete183 Yes, here is my objective-c code snippet:
    FUIAuth *auth = [FUIAuth defaultAuthUI];
    auth.delegate = self;

    FUIPhoneAuth *phoneAuth = [[FUIPhoneAuth alloc] initWithAuthUI:auth];
    auth.providers = @[phoneAuth];
    [phoneAuth signInWithPresentingViewController:viewController];

Swift
let auth = FUIAuth.defaultAuthUI()
auth?.delegate = self
let phoneAuth = FUIPhoneAuth(authUI: auth!)
auth?.providers = [phoneAuth]
phoneAuth.signIn(withPresenting: self)

update for swift 3.1
let auth = FUIAuth.defaultAuthUI()             auth?.delegate = self as? FUIAuthDelegate             let phoneAuth = FUIPhoneAuth(authUI: auth!)             auth?.providers = [phoneAuth]             phoneAuth.signIn(withPresenting: self)


[Fix] logging: replacement non utf-8 characters

Description
logInternalError print ech_call error message, if there are non-utf-8 characters, the log file will be corrupted
Changes include

[x] Bugfix (non-breaking change that solves an issue)

Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels

Testing

[x] I have tested this code with the official test suite

Codecov Report

Merging #355 (cca76a9) into dev (00cac94) will increase coverage by 0.02%.
The diff coverage is 0.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##              dev     #355      +/-   ##
==========================================
+ Coverage   47.64%   47.67%   +0.02%
==========================================
  Files         136      136
  Lines       20614    20618       +4
==========================================
+ Hits         9822     9829       +7
- Misses       9961     9962       +1
+ Partials      831      827       -4

Impacted Files
Coverage Δ

jsonrpc/dispatcher.go
56.85% <0.00%> (-0.68%)
:arrow_down:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


how to use main menu

Hello guys ,
I followed a tutorial showing how to create a mod but it's very old and the code for the menu did not work
` var menulist = new List();
var button = new MenuButton("Example button 1");
button.Activated += (sender, args) => { UI.Notify("This is an example"); };
menulist.Add(button);
    button = new MenuButton("Example button 2");
    button.Activated += (sender, args) => { UI.Notify("This is an example 2"); }; // either use this or a function call
    menulist.Add(button);

    var menutoggle = new MenuToggle("This is a toggle", "Your description here");
    menutoggle.Changed += (sender, args) => { UI.Notify("You toggled me"); };
    menulist.Add(button);
    Menu Active_Menu = new Menu("Example menu", menulist.ToArray()); // if you store the menu inside a var you can edit it
    View.AddMenu(Active_Menu);`

i hope someone provide me with the new way to do it or an example

Thats very old code and the menu support was removed from this project. There is an alternative here that you should probable check out
https://github.com/Guad/NativeUI
It has a wiki to help get you started as well

thank you very much zorg93 this is what I'm looking for :)
I really appreciate your answer


Supporting multiple definitions

The usage for the module appears to be in a single container context
resource "aws_ecs_task_definition" "this" {
  container_definitions = "${module.container_definition.json}"
   ...
}

The change in this PR makes it possible to allow usage for multiple definitions, but a side effect is that the module would instead return the first definition and the list assignment would need to happen in the resource usage.
resource "aws_ecs_task_definition" "this" {
  container_definitions = "[${module.container_definition.json},${module.container_definition2.json}]"
  ...
}

An alternative to this would be adding another output for something like json_as_single_value which wouldn't impact the usage.

@stephencoe I think this is an interesting suggestion.

An alternative to this would be adding another output for something like json_as_single_value which wouldn't impact the usage.

I think I'm leaning towards this solution. We could store the single value as a local. Then present the two diferent outputs. Let's preserve the behavior of json and maybe add json_map which is the map inside the list.

I agree that preserving the behaviour is better, I don't like that the regex is duplicated. It would be possible to not do that by using an input variable but I think I like that idea less

I don't like that the regex is duplicated.

I don't think we need to duplicate the regex.

simplify the container_definition to a map (without list)
json encode / normalize and store in a local called json_map
change json output to value = "[${local.json_map}]"
change json_map output to value = "${local.json}"

Am I on point?

thanks for the feedback, completely missed the obvious path


date-histogram-* plugins are not working

They return a 500 response code.

date-histogram-* plugins should be generally working now w the exception that trying to request a specific number of bins from date-histogram-num-bins gives a 500. im closing this and opening a separate issue for that, since i suspect the priority for that falls below profiling for performance.

I'm still getting 500s:
curl 'https://dfalke-b.clinepidb.org/eda-data/apps/pass/visualizations/date-histogram-bin-width' \
  -H 'Content-Type: application/json' \
  --data '{"studyId":"GEMSCC0002-1","filters":[],"config":{"outputEntityId":"EUPATH_0000096","valueSpec":"count","xAxisVariable":{"entityId":"EUPATH_0000096","variableId":"EFO_0004950"}}}'

I'm reopening. Feel free to close if I'm missing something :)

the rserve container was failing to build, and so wasnt reflecting my recent changes. this is now resolved and im re-closing the ticket


Rate productivity upon timer completion

Describe the feature or enhancement you'd like
The current implementation of DownTimer has no way to measure productivity beyond timers. If we add a mechanism for "rating" productivity levels during a timer, we can better understand productivity habits.
When a timer expires, we should give the user a way to score their productivity. This can be sync'd like everything else. It can then be used as a weight in analytics.

This is currently in development on a local branch.
A snackbar appears after timer expiry, prompting for a rating.

Closed by #31


Examples

@ocots please check unified versions of examples (no more notebooks, single version for basic.jl and goddard) and merge

Are you ok if I put the jl files into the doc?
I need a new release of ctdirect then I will do a new release of OC without direct shooting and if you are ok I will put the examples into the doc.

Sure, but then wouldn't it be better to have md files (for Documenter.jl) instead of jl ones? I actually hesitated but wanted to put forward the possibility to run the code directly. And this kind of jl file that includes markdown docstrings is well rendered on, e.g., Pluto ^1 (or using some vscode extensions...)

Are you ok if I put the jl files into the doc?
I need a new release of ctdirect then I will do a new release of OC without direct shooting and if you are ok I will put the examples into the doc.

To put them on the doc, I will have to change them to md files. This seems the most usual to do for the moment. The jl files are for concrete applications. Not doc.

Right now I need to have them running (Documenter.jl will also run them but well...) So please keep them as jl files and merge. See also this post

I do it by the end of the week.


Update http4s-ember-client to 0.23.24

About this PR
📦 Updates org.http4s:http4s-ember-client from 0.23.12 to 0.23.24
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
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!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "http4s-ember-client" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.http4s", artifactId = "http4s-ember-client" }
}]

labels: library-update, early-semver-minor, semver-spec-patch, version-scheme:early-semver, commit-count:1

Superseded by #379.


Azure Front Door setup/configuration process has changed

The process to setup and configure Azure Front Door has changed, however the steps and screens in the documentation does not reflect that.

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

ID: 9c4a65cd-dae6-898a-514b-5bb9238f58a3
Version Independent ID: 9c2f7824-efed-7108-7c1e-eb2b57f3c0d6
Content: Enable Azure AD B2C custom domains - Azure AD B2C
Content Source: articles/active-directory-b2c/custom-domain.md
Service: active-directory
Sub-service: b2c
GitHub Login: @kengaderdus
Microsoft Alias: kengaderdus

@schalkvanwyk Thank you for your feedback . We will engage the content team and update the thread.

@schalkvanwyk
Thank you for your feedback and patience. We're already tracking this issue internally, and we're in the process of updating this document to capture the new changes. For now, I'll close this issue, and will comment back once the document is updated. #please-close


enhancement: return to actual page in agenda list

I have an agenda that is so long that it needs 2 pages.

click on an agenda item on page 2 to modify it
click now on the button "back to overview

result: page 1 of the agenda is displayed and not page 2 where the item appears
I found out that it would be better to return to the actual page where the item appears in the agenda because one often has the task to modify items one after each other. If you disagree, feel free to close this as WONTFIX.

This is not fixed in OS2.2beta2

@donovaly It is fixed in 2.2b2 - I can confirm it. Maybe you have to remove your browser cache?
Please try again:
Create >25 agenda items. Go to 2nd page in agenda list view. Open an agenda item (of 2nd page) in detail view. Not go "Back to overview". => You see the 2nd page of the agenda list view - NOT the first page.
So for me it works as expected. Can you confirm it?

Maybe you have to remove your browser cache?

Now it works. Maybe it was indeed only something in the cache.


Run CI against VFX Ref 2018

Although we don't officially support VFX Reference 2018, our codebase seems to still build against it, so I suggest we keep this test for the time being. When we deprecate support for Houdini 18.0 later this year, we can jump to VFX Reference 2020 as our minimum supported version - which will make Boost 1.70 our minimum version.

The whole point of the boost change was so you can have disable_checks OFF for H18... your previous builds were failing due to the older CMake version so you should be able to revert that change?

My untested assumption was that there were other old deps in VFX 2018 that would require disabling checks, such as TBB 2017. I'll revert this change and see if it still passes.

The whole point of the boost change was so you can have disable_checks OFF for H18... your previous builds were failing due to the older CMake version so you should be able to revert that change?

My untested assumption was that there were other old deps in VFX 2018 that would require disabling checks, such as TBB 2017. I'll revert this change and see if it still passes.

Nope, failed on attempting to pull in TBB 2017. Un-re-reverted.

Nope, failed on attempting to pull in TBB 2017. Un-re-reverted.

Oh I misunderstood what you are doing - I thought it would be better to explicitly tell our CMake to build against Houdini 18 boost on the VFX19 platform. But this is okay too.


BLEU4/CIDEr

I'm sorry to bother you again, professor. As for the two indicators BLEU4 and CIDEr of Dense Video Caption task in the paper, how can I get these two results？

Hi, youcan try the following command.
cd PDVC/densevid_eval3
python evaluate2018.py -v -s path/to/results.json

Hi, youcan try the following command.
cd PDVC/densevid_eval3
python evaluate2018.py -v -s path/to/results.json

Hi, youcan try the following command.
cd PDVC/densevid_eval3
python evaluate2018.py -v -s path/to/results.json

I'm sorry, professor, I don't seem to understand you.
I modified your code, then trained the model, and tested my trained model using 'eval.py'.
In the generated 'val.log' file: METEOR, Recall, Precision, soda_c, para_Bleu_1-4, para_METEOR, para_ROUGE_L, para_CIDEr.
No file to generate result.json.
Can BLEU4/CIDEr be calculated directly using 'evaluate2018.py'? Don't I need to change the code while training the model?

To find the results json, you do not need to modify the code. After training, I believe you could find some JSON files in save/**/prediction/num*_epoch*_rerank*.json. These files are the generated results at each epoch.
After running eval.py, you can also find a json file in save/** which stores the generated captions of the best checkpoint.
Use evaluate2018.py to evaluate these files, you will get the score of all types of metrics.

thank you very much, professor~

To find the result json, you do not need to modify the code. After training, I believe you could find some JSON files in save/**/prediction/num*_epoch*_rerank*.json. These files are the generated results at each epoch.
After running eval.py, you can also find a json file in save/**/*.json which stores the generated captions of the best checkpoint.
Use evaluate2018.py to evaluate these files, you will get the score of all types of metrics.

thank you very much, professor~


Changing color of a tile

I want to change the color (or blur it) of a tile. Is there any function which takes the tile number (the relative tile number) as arguments and color/fade/blur that particular tile.

No, you would need to use a custom BitmapProvider, which gets the Tile instance (including row and column number).  There is an overview in the readme.

I will have to sue third party library in that case like Picasso, right?

no, you can do anything you want, all that matters is that getBitmap returns a bitmap.  You can use any code you want in that implementation.


Add tests with Entra ID auth, for non-OpenAI chat and embeddings models

Description

Add tests with Entra ID auth, for non-OpenAI chat and embeddings models. Previously we only had Entra ID tests for OpenAI models.
Refactor test code. Split into separate files, based on which client is used (chat completions, text embeddings or image embeddings)
Forgot to do this in previous PR: Update CHANGELOG.md to mention the new ImageEmbeddings client.

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-ai-inference


Standard Names and identifiers

In the source the canonical standard name, i.e. the term which is defined by the CF Convention, is used as a record identifier. This worked because of the guaranteed uniqueness of these terms. The import has replaced the ID with an opaque string and thus lost the canonical form of the standard name.
To recover this, I suggest replacing the record label value with the old record identifier.

updated on https://<IP_ADDRESS>/standardname/


Clones head way outside enclosing DIV

My table is in a bootstrap grid:

PMT test

             ...

             ...

And with this directive I get:

As you can see, the thead is rendered above the h1 that precedes the table. And its not even fixed.

Move h1 header to the top and try..
`

PMT test

             ...

             ...

`

That's not valid bootstrap code. I bagged it and found another directive that works.


number protocol issues

[ ] table design. function pointer vs table offset https://github.com/RustPython/RustPython/pull/4615#discussion_r1123553379
[ ] unary ops handling. https://github.com/RustPython/RustPython/pull/4615#discussion_r1131191733
[ ] test_collections regression related to slot selection https://github.com/RustPython/RustPython/pull/4615/files#diff-a2a882635e20e8e9439f090d33513502a7af648286efebfa4e4bc37270c44784R264
[x] heaptype concerns https://github.com/RustPython/RustPython/pull/4615#issuecomment-1462515684
[ ] __builtins__.pow is still not using number protocol
[ ] Remove #[pymethod] from number types' binary ops
[ ] Remove PyNumber assumption from PyNumberMethods

cc @xiaozhiyan @qingshi163
part of #3244

per discuss #4720 don't put number slots into HeapTypeExt is better and simplier.


PF-1694: create/clone workspace property

create the workspace property

clone the workspace with property

multiple entries add

Are we sure we want to override the properties on clone? I would've expected them to be merged with the source workspace. I'm actually surprised we want to specify them on clone at all. I thought the intent of the ticket was to just clone the existing properties to the new workspace.


Peer Review - Hazard Analysis: FEMA recommended actions

Artifact Under Review
FEMA - Recommended actions
Team Number for Team Doing the Review
Team -27
Description of Issue
Upon reviewing the FMEA tables, I noticed that some of the recommended actions, particularly those related to data security, could benefit from further elaboration to ensure effective hazard mitigation. For instance, the recommendation to "Improve cybersecurity" is quite broad and would be more actionable if it specified techniques such as implementing multi-factor authentication, conducting regular security audits, deploying intrusion detection systems, and ensuring timely software updates and patch management

@Tan630


Train my own dataset

Thank you for your excellent work
I have a question about how to train my own dataset. Can I train my own dataset with this code? What do I need to change in the code?

Of course, you can generate datasets from the data directory and scripts directory first, and then modify the config.py file.

Thanks for your reply!


Download Copied Images.

Allow users to download the copied images from the clipboard. Currently, users cannot download those images, so the save image feature is not used efficiently.

Copied images can now be downloaded


Text-based/Disassembly search

Need search / replace for both hex and disassembly (replace for disassembly requires #27)

The strings window also desperately needs a search option.

Search is implemented, but replace is not.  Also, implement text searching in linear view.

Making this ticket specifically targeting finding of text in the current BinaryVIew

Added in 1.1.1422


 My hypothesis is that we'll get way less of them if we can at least eliminate the ones with errors around line length in prose.

Test plan
N/A

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6299   +/-   ##
=======================================
  Coverage   63.54%   63.54%
=======================================
  Files         221      221
  Lines        8502     8502
  Branches        3        4    +1
=======================================
  Hits         5403     5403
  Misses       3098     3098
  Partials        1        1

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 3d127ff...a569cc1. Read the comment docs.

this'll look awesome on the contribution graph

Blazing good


The label for a has many association should be a link to a filtered list view

A very useful feature for our team is if a user clicks the label of a has many relation, it should bring up the list view of the associated model filtered by the foreign key. The user can then filter, sort or view attributes of the related models.
eg. If Author 1 has many Posts. If you click the Posts it takes you to the Post List View and puts a filter for Author is exactly 1.
I was able to make this work decently well locally with the following but was unsure how to limit it to just the show view:
RailsAdmin::Config::Fields::Types::HasManyAssociation.register_instance_option :label do
  (@label ||= {})[::I18n.locale] ||= abstract_model.model.human_attribute_name association.name
  begin
    bindings[:view].link_to(
      @label[::I18n.locale],
      bindings[:view].rails_admin.index_path(
        model_name: association.klass,
        query: '',
        f: {
          abstract_model.model_name.demodulize.underscore => {
            '1' => {
              o: 'is',
              v: bindings[:model].id
            }
          }
        },
        utf8: '✓'
      ),
      style: 'color:#FFFFFF;',
      class: 'pjax'
    )
  rescue
    @label
  end
end

+1

Please open a PR if anybody find this useful.


[data-client] Add metrics to data client

Motivation
This PR adds metric counters to diem-data-client to keep track the following:

the number of requests sent (per request type and total)
the number of success responses (per request type and total)
the number of error responses (per request type and total)
the number of peer selection errors (per request type and total)
request latencies (per request type)

Have you read the Contributing Guidelines on pull requests?
Yes.
Test Plan
No test plan is required. Re-ran existing tests to make sure this PR does not fail existing tests.
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/diem/diem/tree/main/developers.diem.com, and link to your PR here.)
If targeting a release branch, please fill the below out as well

Justification and breaking nature (who does it affect? validators, full nodes, tooling, operators, AOS, etc.)
Comprehensive test results that demonstrate the fix working and not breaking existing workflows.
Why we must have it for V1 launch.
What workarounds and alternative we have if we do not push the PR.

Hi @JoshLind , I'm looking for some initial feedback on the PR. Could you please give me some feedback when you have time to take a look? Thank you! 😃

Thanks for the feedback @JoshLind . I tried to address/apply all the nits. Please let me know if there is anything else you'd like me to fix. Thank you! ❤

/land

Thanks @khiemngo! LGTM! 🚀

Oops, sorry I just realize that I did not get the official approval from @JoshLind before landing. (I saw this comment and landed instead. I'll look more carefully next time 😃 ). I hope this won't be a problem...

@khiemngo, I think I did approve 🤔 Otherwise, how would you land? Either way, it's good to go 😄


SNPE for iid

hi there
just curious why we cannot have SNPE for all iid sample
like SNLE or even traditional MLE？
thank you

The basic problem
During training, you only have 1 x for every theta (because you simulate only once for each theta). So, the neural network learns to approximate p(theta | x) (with x being a single x here).
Now, after training, you want to have p(theta | x_1, ..., x_N), but the neural net was never trained for many x.
How does SNLE do it
SNLE has the same "issue", it is trained on pairs of theta and a single corresponding x. The neural net then approximates p(x | theta). However, one can neatly multiply these terms to obtain p(x1,...xN | theta) = p(x1 | theta)*...*p(xN | theta) to obtain the joint likelihood and plug it into MCMC.
Workaround for SNPE
The only way of having iid trials with SNPE is to simulate N times for each theta (N being the number of trials). You can then concatenate the N simulation outputs for each theta and, after training, also concatenate the iid dataset into a 1D vector. Alternatively, instead of concatenating the simulation outputs, you can also use the permuatation-invariant embedding, but this is not released yet so you will have to rely on the bleeding edge github version of sbi.
Either way, this works, but has the disadvantage that one has to simulate N times for each theta.
Hope this helps
Michael

Dear Michael,
Thank you a lot for your detail explanation. As you mentioned, I also come up with the first way that simulation N iid for each theta, then concatenate to 1D vector. I tested the example in tutorial and it works!
However, I am not sure if this may lose some information: e.g.
if for each trial the return x is a vector with length m
then after concatenation it would be a 1-D vector with length Nm. How the neural network could know that it is N trials with output length m, but not N/2 trials with output length 2m (or other combination)?
I am just wondering if this would be helpful to arrange that to a matrix with N*m so that the neural network could learn more easily?
Thank you.
Best wishes
Tim

Dear Tim, good to hear that this works.
In general, the neural network does not know if it is N trials of length m or N/2 with length 2m. However, as for the amount of "information" the two scenarios are identical and thus the neural network should eventually converge to the same thing.
However, as you already mention, there can be differences in how easy it is to learn for the neural network. Therefore, I would recommend using the PermutationInvariatEmbedding as follows:

Install github version of sbi

git clone https://github.com/mackelab/sbi.git
cd sbi
pip install -e .

Use the permutation invariant embedding:

from sbi.neural_nets.embedding_nets import PermutationInvariantEmbedding, FCEmbedding
from sbi.utils import posterior_nn

single_trial_hidden_dim = 100
single_trial_net = FCEmbedding(input_dim=dimensionality_of_x, output_dim=single_trial_hidden_dim)
embedding_net = PermutationInvariantEmbedding(single_trial_net, trial_net_output_dim=single_trial_hidden_dim)
net = posterior_nn("maf", embedding_net=embedding_net)
inference = SNPE(prior, density_estimator=net)

The PermutationInvariantEmbedding explicitly handles each trial separately in the single_trial_net and then combines the trials.

Thank you, and I will try this!
Just wonder if there is any plan or timeline that this would be released? Thank you!

We are trying to make it happen this week (Friday), but no guarantees.

Thank you, I am trying it.
Am I correct that the simulated data for training, x, also need to be in 3d?
e.g. if I did 20000 simulations, with 100 trials and 10 columns in each trial, then x should be in the form
(20000,100,10)?
I am following the example using simulate_for_sbi and it automatically convert that to 20000, 100*10, and got an error.

Hi! You are right, the shape should be (20000,100,10), see also here.
And indeed, simulate_for_sbi will break things -- please do not use simulate_for_sbi for now, we will fix this soon. prepare_for_sbi and simulate_for_sbi are only for convenience, they do not have to be used.

Hm, maybe try x_oo = x[:1, :] such that the batch dimension gets preserved?

yes, this is working. thanks!

For SNPE we are enabling iid inference by treating the trial dimension as an additional data dimension. Thus, the simulator needs to take of this additional dimension as well, e.g., it needs to return (num_trials, dim_single_trial_data) for a single simulation.
Accordingly, when passing x_o for sampling the posterior, one needs to pass it with a leading batch dimension, e.g., (1, num_trials, dim_single_trial_data). Otherwise it is unclear whether it's a multiple trials, or a multi-dimensional x.
Maybe I am missing something, but if the simulator returns (num_trials, dim_single_trial_data) (or (1, num_trials, dim_single_trial_data) after calling process_simulator, then simulate_for_sbi should work just fine.

Hi there,
Thank you for suggestions. I tried both, but the simulate_for_sbi returns n_simulation, num_trials*dim_single_trial_data.

Hi Tim,
I found the problem, it's in process_simulator where we wrap a user simulator that doesn't return a batch dimension into one that does by looping over the batch. That's where we squeeze the data into one dimension.
I fixed it and will be in main soon.
For now I recommend you prepare your simulator for sbi yourself, e.g., by making sure it always returns a batch dim: for data of shape (10, 2), it should return (1, 10, 2) even for a single simulation.

should be fixed with #781 . feel free to re-open if not.


Elements not freed correctly

When running the simulation, the amount of elements increases even when the amount of objects doesn't.

Bug is caused somewhere in tryCollapseBranch().

It was caused by not freeing the elements of the child when collapsing it into one node.


B1

rejected:B1
It looks like you are trying to submit B1, however it seems like the following submission criteria are problematic:

Logs are missing
We’ve thus marked B1 as rejected, however if this is a mistake, please upload the log summary and we can re-review.

confirmed:ID2
account1:4V1vBoLNKy8japTmFzwdzy7mvoUor5SHrm6T62FShMnDKVtvpx
account2:4X8sxnwWzi7AUQxm1TdNkmJfk9itizD2yLgWHTSzXMYXk6RGok
PublicDataAttributes:Verified
DeviceMake&Model:Verified

confirmed:ID2
account1:4V1vBoLNKy8japTmFzwdzy7mvoUor5SHrm6T62FShMnDKVtvpx
account2:4X8sxnwWzi7AUQxm1TdNkmJfk9itizD2yLgWHTSzXMYXk6RGok
PublicDataAttributes:Verified
DeviceMake&Model:Verified

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:missing

confirmed:ID3
DocumentType:verified
Nationality:verified
DeviceDetails:verified
Experience:verified
Screenshot:missing

rejected:ID4
reason:We cannot fully verify this submission. It looks like you are trying to submit ID4, however, it seems like we cannot find the transaction between your account and Concordium Test Account.

rejected:ID4
reason:We cannot fully verify this submission. It looks like you are trying to submit ID4, however, it seems like we cannot find the transaction between your account and Concordium Test Account.

confirmed:B4
address:4V1vBoLNKy8japTmFzwdzy7mvoUor5SHrm6T62FShMnDKVtvpx
baker:518

confirmed:B4
address:4V1vBoLNKy8japTmFzwdzy7mvoUor5SHrm6T62FShMnDKVtvpx
baker:518


Add the ability to select custom themes in the settings dialog

Description
This loads themes (any .json file) from the /Themes directory on startup and lists those in the settings page.
The custom themes in that directory will have a Custom: prefix to show that they're custom.
In the settingsfile, we save the name of the theme (e.g. 'forsen', no more info).
When the theme is updated or on Chatterino startup, we find the best matching theme with that name, whether it's a built in theme or a custom theme.
There's currently no mechanism for making a theme that has the same name as a built in theme, and they still show up in the menu. IMO we should just not allow duplicate names for simplicity's sake.
There's a few things left to do, and a few things to discuss before taking this step, even though I still deem it to be "pre-production" even when this PR is merged in.
The changelog entry is left as a Dev entry on purpose since I don't think this is a feature that should be fully announced until there's more testing & better UX for users.

It'd be great if there was hot reloading of the current theme when the JSON file is modified. This would make creating themes and trying out different colors easy (compared to what I imagine is having to restart Chatterino after every change).
There appears to be a Qt class that would make detecting the file change easy (in theory, barring os-specific annoyances): https://doc.qt.io/qt-6/qfilesystemwatcher.html. I haven't dug through the Theme code that much though, so it's possible that this is more difficult than it sounds.

It'd be great if there was hot reloading of the current theme when the JSON file is modified. This would make creating themes and trying out different colors easy (compared to what I imagine is having to restart Chatterino after every change).

I would like that type of functionality in a V2/V3 kind of thing instead of in the initial release.
Initially I'd like this in a nightly release without any commitments to how/where the themes are located so it can be tested, then we look into specifics for how themes are worked on by users.
I could see a world where theme creation isn't tested in Chatterino itself, but rather a web UI. Or there's a /reload-theme command to do the reloading.


Update Mavic 3 class to C1

Updated Mavic 3 classification - C1.

Hello Andrew, thanks for your PR. Much appreciated! :)


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/60674

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/72192

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/76134

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/76989

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83187

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/87810

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/94926


How to add JSON tags for parameters?

Problem statement
I am writing a swagger spec for designing API's. I am using this swagger spec to generate go client code by using "swagger generate client" command.  I have defined parameters for API's but generated code is not showing up the json tags.
Swagger specification
sample.yaml
paths:
   /user/create:
          get:
                description: "some description"
                tags: "user"
                parameters:
                      -
                       x-go-name:"UserName"
                       description: "some description"
                       name: "username"
                       in: "query"
                       required: true
                       type: "string"

Steps to reproduce

swagger generate client -f sample.yaml -t <client_path>

This shows the following struct in *_parameters.go file:
type GetUserCreateParams struct {
// HTTP Request Object

HTTPRequest *http.Request `json:"-"`

/* Some desc
Required: true
In: query
*/
Username string
}

In the above-generated code, I am expecting the username parameter to be generated with json tag. Am I missing something in the swagger spec?
Environment
swagger version: 0.18.0
go version: go 1.12
OS:   Ubnutu 14.04

No, query and path parameters do not lead to json tag generation.
Tags are generated with body params, responses and model definitions.

Thanks, @fredbi. Could you please clarify why json tags are not generated for a path and query params?
Because if backend service is expecting a json tag then it would be really easy to map with actual field. Now in the current scenario, I have to manually map the json tags with actual field.

These params are not passed in the HTTP request body, but in the URL. There is no json unmarshalling to be considered here, and there is no "backend expecting a json tag".


Change to fix the ValueError at the split

If your folder that you were hashing, had files or folders with a space in their name, then this line of code will throw an error.
The code fixes it here by specifying the double space, and then a maxsplit=1 count to ensure that it only does the first double space, so if a folder or a file has a double space in the name they don't get split as well.
When using this in testing, this the error and I was unable to make another error with this.

Good catch, thank you!


Remove PUSHL Instruction

I think this would simplify the Instruction interface quite well.  However, to make it work we would need some way to encode labels.
Potentially one option is to merge PUSHL into PUSH by adding a bool operand indicating whether or not it is pushing a jump destination.

PUSHL is really a piece of syntactic sugar (see #21)


Add omitempty tag for slices

Otherwise marshalling/unmarshalling CRDs with webhooks results in error
as it serializes these struct fields as "null".
The KeystoneAPI "keystone" is invalid: spec.networkAttachmentDefinitions: Invalid value: "null": spec.networkAttachmentDefinitions in body must be of type array: "null"

Looks like the default does not work as it's treated as string?
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openstack-k8s-operators_placement-operator/133/pull-ci-openstack-k8s-operators-placement-operator-master-functional/1627975037304705024

I think I understand it better now.
Either the defaulting webhook specially or having two webhooks defaulting + validation causing the failure without omitempty. If I remove the defaulting webhook then the validating webhook did not cause the null issue alone.
So I think that there is multiple deserialization and serialization happens (at least one per webhook type) and the originally absent field get converted from json to a go value [] or nil then serialized again to json but now to fieldname: null instead of as an absent field. And that causing the next deserialization (that happen to be the validation logic) see that null value for something that cannot be null.
With some googling I found https://github.com/longhorn/longhorn/issues/3352 that give me an idea of // +nullable marker. I was able to make the null problem disappear either with omitempty or // +nullable . The omitempty handles this at the serialization step by not adding the field with [] or nil value to the json after the first webhook run. The // +nullable handles this as the deserialization level by accepting the null value as well at the second webhook run. I don't see any pro / contra for either solution right now. So I guess this is OK to land.


Unification Segfault

v1.23.40
$ ./tpl
?- C = C, C = [C].
   C = [[[[[[[[[[[[[[[[[[[[[[[[[...]]]]]]]]]]]]]]]]]]]]]]]]]]].

?- C = [C], C = C.
Segmentation fault (core dumped)
$

v1.23.42, defect corrected!


pure-export fallback data source may not match primary data source

The make target local/mongo_as_unvalidated_nmdc_database.yaml runs a command called pure-export, passing it several CLI options, including:

--client-base-url https://api.microbiomedata.org
--mongo-host localhost
--mongo-port 27777

A user could have localhost port 27777 mapped to a different Mongo database than the one used by the web server at https://api.microbiomedata.org. In that case, when the pure-export command (i.e. whatever runs when that command is issued) uses a Mongo client, the Mongo database it is accessing would differ from the one it would be accessing when using HTTP.
Here's a message the command showed when I ran it:
Attempting to get collection stats from functional_annotation_agg
estimated_document_count = 11816893
Attempting to get 200000 documents from nmdcschema/functional_annotation_agg in pages of 200000.
warning: 500 Server Error: Internal Server Error for url: https://api.microbiomedata.org/nmdcschema/functional_annotation_agg?max_page_size=200000
warning: FastAPI request to nmdcschema/functional_annotation_agg appears to have failed. Trying as a PyMongo query.

I want to emphasize the final line: warning: FastAPI request to nmdcschema/functional_annotation_agg appears to have failed. Trying as a PyMongo query.
This implies to me that the underlying code first tried to fetch data via HTTP (which involved one Mongo database) and then was going to try via a Mongo client (which would involve a different Mongo database).

Yeah, pure-export needs a lot of work! I already have a few related issues that I will add soon. @brynnz22 and I have had some  conversations about the local/mongo_as_unvalidated_nmdc_database.yaml target.
Maybe there should be subcommands for

just getting are report about the intersection of the Database slots and the MongoDB collection names, plus the
downloading with a pure API approach
downloading with a pure MongoDB approach

Or do you think that doing away with the hybrid/fallback solution is a bad idea @eecavanna ?
Do you have suggestions for breaking improvements to pure-export into small commits and then prioritizing them? I would love to work on this with you.
We could also include @mbthornton-lbl because he wrote an API-only dumper with fewer configuration options that only returns records (from multiple collections) if they have some path back to a named Study. We worked on that together to launch as a command line tool () and to take some of its configuration from local/.env. I would like to either integrate that with pure-export or at least to make the launching and configuration more similar.


chore: fix homepage link

This sets the homepage field in the package.json of some packages to our actual, nice URL.

What about the other packages? Shouldn't be this link set/updated there as well? For example in the components/package.json

I'll create a follow up ticket to align all package.json files on this and other fields.


kcptun go客户端适用于安卓手机吗？

golang 1.5可以在安卓上运行，不知可以实现安卓运行不？

try linux arm

cross compile it, it might work. with great probability

https://godoc.org/golang.org/x/mobile
maybe you can add a GUI to android app.

i am working on this,once the warper apk is done, i will open source it.

shadowsocks releases 3.0.1 is support kcptun.
https://github.com/shadowsocks/shadowsocks-android/releases

@shutup @bubbless SS自带的Kcptun功能还不稳定，希望有一个更好的可替代品。


Added usage sample

Q
A

content fix?
yes

New article?
no

Related issues?
n/a

What's in this Pull Request?
Added sample on how to use the functionality

@KoenZomers,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Hi Koen,
getting these in now, but these files will be automatically unfortunately overridden when we have them updated, since they are automatically generated. Getting sample usage for each of them would though make lot of sense, so thanks for the suggestion. We'll work on seeing if we can get that included in alternative ways in future.


added:keyboard shortcut to Drafts link tooltip

Fixes #22131
Fixes: added keyboard shortcut to drafts link tooltip

Screenshots and screen captures:

Self-review checklist

[X] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[X] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[X] Each commit is a coherent idea.
[X] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[X] Visual appearance of the changes.
[X] Responsiveness and internationalization.
[X] Strings and tooltips.
[X] End-to-end functionality of buttons, interactions and flows.
[X] Corner cases, error conditions, and easily imagined bugs.

@aparna-bhatt please review the commit message guidelines and update your commit message accordingly. Thanks!

Otherwise looks good to me. @garg3133 do you want to take a look at the code?

@aparna-bhatt please review the commit message guidelines and update your commit message accordingly. Thanks!

OK I will change the commit message.
Thanks.

Also please squash all your commits into a single commit. You can read more about squashing commits from here.

Sorry for the mistake I will correct it!
Thanks!

@garg3133 I have changed the commit message and corrected the class-name.
Thank you !

@alya I have squashed the commits.
Thank you!

@timabbott small change reviewed by @garg3133 and myself, ready for you.

Merged, after fixing a typo in the commit message, thanks @aparna-bhatt!


Added string matching algo: Boyer moore and finite automat

Added string matching algo: Boyer moore and finite automat

@Anushkam09 this PR includes other codes too along with add test cases too.

@UTSAVS26 The other algorithms were already there in the file, I added boyer moore and finite automata in the similar format. Adding the test cases in some time!

@UTSAVS26 The other algorithms were already there in the file, I added boyer moore and finite automata in the similar format. Adding the test cases in some time!

So wait until the previous PR get merged otherwise it will raise merge conflicts I am closing this one.


Firebase config not found on deployment

firebase-config.json is in gitignore. To avoid committing the config file to the repo, set the key, value pairs inside the config as environmental variables in Heroku and create a file at runtime. Example in comments

Fixed


Add podcast episode WAYAHDYGH Sam

Sharing my edit of my interview w/ Sam! The sound at the beginning seems a little off and generally seems kind of quiet? What do you guys think? (Please be brutal w/ the audio feedback. This is my first time editing and I'm not sure if I fully have the hang of it yet.)
Thanks to @SamRozen for doing this with me! Let me know if you're good to go on this from a content perspective? (By morning Thurs, 3/18 if you can!)

Listened to the first ~5m and scanned a bit; sounding nice! I agree that the vocal tracks are a bit on the quiet side. I think you could bump your intro track by 3 - 4db and both your side and sam's side of the interview by 2 - 3db (that's a rough guess, feel free to tinker with it). Not sure if you've already done this, but I tend to aim for my audio to fall in the -18db to -6db range on audacity. Something like this:

My two cents are that yeah that first bit is a little quiet over the music but once the interview gets rolling I think it sounds great! 🙌

wohoo sounds great! merging!


Add French translation

Thank you for #7! This MR adds a French translation.

Awesome! That was quick. I'll add the language dropdown and cut a new release today.


Add location name to screen

Hi John,
First, I love this screen saver for my Mac, it catches everyones' eye and I pass it on to them. It would be cool to see the name, in some nondistractive way, of the location on the screen when the screen saver is on. So when viewing a screen from China it would display "China" in one of the corners either the entire time or just for several seconds once it starts.
Thanks for creating this package!
Cheers,
Jason

Closing as duplicate of #58 .
There is a potential implementation waiting to be merged.  PR #285


Can't install on OSX 10.11.6 via homebrew

Hello,
I have trouble installing unused on OSX 10.11.6 via Homebrew, installing via Stack is working.
Here's what I get:
2016-08-16 10:25:49 +0200

cabal
install
--jobs=4
--max-backjumps=100000
--prefix=/usr/local/Cellar/unused/<IP_ADDRESS>

clang: warning: -Wl,-headerpad_max_install_names: 'linker' input unused
clang: warning: argument unused during compilation: '-L/usr/local/lib'
clang: warning: argument unused during compilation: '-L/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries'
Resolving dependencies...
Notice: installing into a sandbox located at
/private/tmp/unused-20160816-11091-79s5cn/unused-<IP_ADDRESS>/.cabal-sandbox
Configuring unused-<IP_ADDRESS>...
Building unused-<IP_ADDRESS>...
Failed to install unused-<IP_ADDRESS>
Build log ( /private/tmp/unused-20160816-11091-79s5cn/unused-<IP_ADDRESS>/.cabal-sandbox/logs/unused-<IP_ADDRESS>.log ):
cabal: Entering directory '.'
Configuring unused-<IP_ADDRESS>...
clang: warning: -Wl,-headerpad_max_install_names: 'linker' input unused
clang: warning: argument unused during compilation: '-L/usr/local/lib'
clang: warning: argument unused during compilation: '-L/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries'
Building unused-<IP_ADDRESS>...
Preprocessing library unused-<IP_ADDRESS>...
[ 1 of 45] Compiling Unused.CLI.ProgressIndicator.Types ( src/Unused/CLI/ProgressIndicator/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/ProgressIndicator/Types.o )
[ 2 of 45] Compiling Unused.CLI.Util  ( src/Unused/CLI/Util.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Util.o )
[ 3 of 45] Compiling Unused.CLI.Views.AnalysisHeader ( src/Unused/CLI/Views/AnalysisHeader.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/AnalysisHeader.o )
[ 4 of 45] Compiling Unused.CLI.Views.GitSHAsHeader ( src/Unused/CLI/Views/GitSHAsHeader.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/GitSHAsHeader.o )
[ 5 of 45] Compiling Unused.CLI.Views.NoResultsFound ( src/Unused/CLI/Views/NoResultsFound.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/NoResultsFound.o )
[ 6 of 45] Compiling Unused.CLI.Views.Error ( src/Unused/CLI/Views/Error.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/Error.o )
[ 7 of 45] Compiling Unused.CLI.ProgressIndicator.Internal ( src/Unused/CLI/ProgressIndicator/Internal.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/ProgressIndicator/Internal.o )
[ 8 of 45] Compiling Unused.CLI.ProgressIndicator ( src/Unused/CLI/ProgressIndicator.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/ProgressIndicator.o )
[ 9 of 45] Compiling Unused.Cache.FindArgsFromIgnoredPaths ( src/Unused/Cache/FindArgsFromIgnoredPaths.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Cache/FindArgsFromIgnoredPaths.o )
[10 of 45] Compiling Paths_unused     ( dist/dist-sandbox-3f92c8f5/build/autogen/Paths_unused.hs, dist/dist-sandbox-3f92c8f5/build/Paths_unused.o )
[11 of 45] Compiling Unused.Regex     ( src/Unused/Regex.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Regex.o )
[12 of 45] Compiling Unused.Util      ( src/Unused/Util.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Util.o )
[13 of 45] Compiling Unused.Projection.Transform ( src/Unused/Projection/Transform.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Projection/Transform.o )
[14 of 45] Compiling Unused.Projection ( src/Unused/Projection.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Projection.o )
[15 of 45] Compiling Unused.ResultsClassifier.Types ( src/Unused/ResultsClassifier/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/ResultsClassifier/Types.o )
[16 of 45] Compiling Unused.ResultsClassifier.Config ( src/Unused/ResultsClassifier/Config.hs, dist/dist-sandbox-3f92c8f5/build/Unused/ResultsClassifier/Config.o )
[17 of 45] Compiling Unused.ResultsClassifier ( src/Unused/ResultsClassifier.hs, dist/dist-sandbox-3f92c8f5/build/Unused/ResultsClassifier.o )
[18 of 45] Compiling Unused.CLI.Views.InvalidConfigError ( src/Unused/CLI/Views/InvalidConfigError.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/InvalidConfigError.o )
[19 of 45] Compiling Unused.Cache.DirectoryFingerprint ( src/Unused/Cache/DirectoryFingerprint.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Cache/DirectoryFingerprint.o )
[20 of 45] Compiling Unused.Cache     ( src/Unused/Cache.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Cache.o )
[21 of 45] Compiling Unused.CLI.Views.FingerprintError ( src/Unused/CLI/Views/FingerprintError.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/FingerprintError.o )
[22 of 45] Compiling Unused.TagsSource ( src/Unused/TagsSource.hs, dist/dist-sandbox-3f92c8f5/build/Unused/TagsSource.o )
[23 of 45] Compiling Unused.CLI.Views.MissingTagsFileError ( src/Unused/CLI/Views/MissingTagsFileError.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/MissingTagsFileError.o )
[24 of 45] Compiling Unused.Types     ( src/Unused/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Types.o )
[25 of 45] Compiling Unused.Aliases   ( src/Unused/Aliases.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Aliases.o )
[26 of 45] Compiling Unused.GitContext ( src/Unused/GitContext.hs, dist/dist-sandbox-3f92c8f5/build/Unused/GitContext.o )
[27 of 45] Compiling Unused.ResponseFilter ( src/Unused/ResponseFilter.hs, dist/dist-sandbox-3f92c8f5/build/Unused/ResponseFilter.o )
[28 of 45] Compiling Unused.LikelihoodCalculator ( src/Unused/LikelihoodCalculator.hs, dist/dist-sandbox-3f92c8f5/build/Unused/LikelihoodCalculator.o )
[29 of 45] Compiling Unused.Grouping.Types ( src/Unused/Grouping/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Grouping/Types.o )
[30 of 45] Compiling Unused.Grouping.Internal ( src/Unused/Grouping/Internal.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Grouping/Internal.o )
[31 of 45] Compiling Unused.Grouping  ( src/Unused/Grouping.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Grouping.o )
[32 of 45] Compiling Unused.CLI.Views.SearchResult.ColumnFormatter ( src/Unused/CLI/Views/SearchResult/ColumnFormatter.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult/ColumnFormatter.o )
[33 of 45] Compiling Unused.CLI.Views.SearchResult.Types ( src/Unused/CLI/Views/SearchResult/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult/Types.o )
[34 of 45] Compiling Unused.CLI.Views.SearchResult.Internal ( src/Unused/CLI/Views/SearchResult/Internal.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult/Internal.o )
[35 of 45] Compiling Unused.CLI.Views.SearchResult.ListResult ( src/Unused/CLI/Views/SearchResult/ListResult.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult/ListResult.o )
[36 of 45] Compiling Unused.CLI.Views.SearchResult.TableResult ( src/Unused/CLI/Views/SearchResult/TableResult.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult/TableResult.o )
[37 of 45] Compiling Unused.CLI.Views.SearchResult ( src/Unused/CLI/Views/SearchResult.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views/SearchResult.o )
[38 of 45] Compiling Unused.CLI.Views ( src/Unused/CLI/Views.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Views.o )
[39 of 45] Compiling Unused.CLI.GitContext ( src/Unused/CLI/GitContext.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/GitContext.o )
[40 of 45] Compiling Unused.TermSearch.Types ( src/Unused/TermSearch/Types.hs, dist/dist-sandbox-3f92c8f5/build/Unused/TermSearch/Types.o )
[41 of 45] Compiling Unused.TermSearch.Internal ( src/Unused/TermSearch/Internal.hs, dist/dist-sandbox-3f92c8f5/build/Unused/TermSearch/Internal.o )
[42 of 45] Compiling Unused.TermSearch ( src/Unused/TermSearch.hs, dist/dist-sandbox-3f92c8f5/build/Unused/TermSearch.o )
[43 of 45] Compiling Unused.Parser    ( src/Unused/Parser.hs, dist/dist-sandbox-3f92c8f5/build/Unused/Parser.o )
[44 of 45] Compiling Unused.CLI.Search ( src/Unused/CLI/Search.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI/Search.o )
[45 of 45] Compiling Unused.CLI       ( src/Unused/CLI.hs, dist/dist-sandbox-3f92c8f5/build/Unused/CLI.o )
Preprocessing executable 'unused' for unused-<IP_ADDRESS>...
[1 of 2] Compiling App              ( app/App.hs, dist/dist-sandbox-3f92c8f5/build/unused/unused-tmp/App.o )
[2 of 2] Compiling Main             ( app/Main.hs, dist/dist-sandbox-3f92c8f5/build/unused/unused-tmp/Main.o )

app/Main.hs:26:39: error:
    • Variable not in scope: (<>) :: ParserInfo a -> InfoMod a16 -> t7
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:26:53: error:
    • Variable not in scope: (<>) :: t7 -> InfoMod a17 -> ParserInfo a
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:45:9: error:
    • Variable not in scope:
        (<>) :: Parser SearchRunner -> Mod f14 a14 -> t6
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:46:9: error:
    • Variable not in scope:
        (<>) :: t6 -> Mod f15 a15 -> Parser SearchRunner
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:51:5: error:
    • Variable not in scope: (<>) :: Parser Bool -> Mod f12 a12 -> t5
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:52:5: error:
    • Variable not in scope: (<>) :: t5 -> Mod f13 a13 -> Parser Bool
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:66:5: error:
    • Variable not in scope: (<>) :: Parser String -> Mod f10 a10 -> t4
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:67:5: error:
    • Variable not in scope: (<>) :: t4 -> Mod f11 a11 -> Parser String
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:72:5: error:
    • Variable not in scope: (<>) :: Parser Bool -> Mod f8 a8 -> t3
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:73:5: error:
    • Variable not in scope: (<>) :: t3 -> Mod f9 a9 -> Parser Bool
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:78:5: error:
    • Variable not in scope: (<>) :: Parser [String] -> Mod f6 a6 -> t2
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:79:5: error:
    • Variable not in scope: (<>) :: t2 -> Mod f7 a7 -> Parser [String]
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:97:5: error:
    • Variable not in scope: (<>) :: Parser String -> Mod f4 a4 -> t1
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:98:5: error:
    • Variable not in scope: (<>) :: t1 -> Mod f5 a5 -> Parser String
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:103:5: error:
    • Variable not in scope: (<>) :: Parser Bool -> Mod f2 a2 -> t0
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:104:5: error:
    • Variable not in scope: (<>) :: t0 -> Mod f3 a3 -> Parser Bool
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:109:5: error:
    • Variable not in scope:
        (<>) :: Parser Bool -> Mod f1 a1 -> Parser Bool
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:117:9: error:
    • Variable not in scope:
        (<>) :: Parser (Maybe String) -> Mod f0 a0 -> t
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)
cabal: Leaving directory '.'
cabal: Error: some packages failed to install:
unused-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

HOMEBREW_VERSION: 0.9.9
ORIGIN: https://github.com/Homebrew/brew
HEAD: b630ab3d1717c1a436610a1542c71ebaadcf3bc0
Last commit: 25 hours ago
Core tap ORIGIN: https://github.com/Homebrew/homebrew-core
Core tap HEAD: 1e1be4517826366a608065d4823320aa8fa6d252
Core tap last commit: 3 hours ago
HOMEBREW_PREFIX: /usr/local
HOMEBREW_REPOSITORY: /usr/local
HOMEBREW_CELLAR: /usr/local/Cellar
HOMEBREW_BOTTLE_DOMAIN: https://homebrew.bintray.com
CPU: quad-core 64-bit broadwell
Homebrew Ruby: 2.0.0-p648
GCC-4.2: build 0
Clang: 7.3 build 703
Git: 2.8.2 => /usr/local/bin/git
Perl: /usr/bin/perl
Python: /usr/bin/python
Ruby: /usr/bin/ruby => /System/Library/Frameworks/Ruby.framework/Versions/2.0/usr/bin/ruby
Java: 1.8.0_77
OS X: 10.11.6-x86_64
Xcode: 7.3.1
CLT: N/A
X11: N/A

HOMEBREW_CC: clang
HOMEBREW_CXX: clang++
SDKROOT: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk
MAKEFLAGS: -j4
CMAKE_PREFIX_PATH: /usr/local
CMAKE_INCLUDE_PATH: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/libxml2:/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/usr/include/apache2:/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Headers
CMAKE_LIBRARY_PATH: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks/OpenGL.framework/Versions/Current/Libraries
CMAKE_FRAMEWORK_PATH: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk/System/Library/Frameworks
MACOSX_DEPLOYMENT_TARGET: 10.11
PKG_CONFIG_LIBDIR: /usr/lib/pkgconfig:/usr/local/Library/Homebrew/os/mac/pkgconfig/10.11
HOMEBREW_SDKROOT: /Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX10.11.sdk
ACLOCAL_PATH: /usr/local/share/aclocal
PATH: /private/tmp/unused-20160816-11091-79s5cn/unused-<IP_ADDRESS>/.cabal-sandbox/bin:/usr/local/Library/Homebrew/shims/super:/usr/local/opt/ghc/bin:/usr/local/opt/cabal-install/bin:/usr/local/opt/the_silver_searcher/bin:/usr/bin:/bin:/usr/sbin:/sbin

I am having a similar issue on 10.10.5:
$ brew info unused
joshuaclayton/formulae/unused: stable <IP_ADDRESS>, HEAD
A command line tool to identify unused code.
https://github.com/joshuaclayton/unused
Not installed
From: https://github.com/joshuaclayton/homebrew-formulae/blob/master/Formula/unused.rb
==> Dependencies
Build: ghc ✔, cabal-install ✔, ag ✔

$ brew install unused
==> Auto-updated Homebrew!
Updated 2 taps (caskroom/cask, homebrew/core).
==> New Formulae
jxrlib
==> Installing unused from joshuaclayton/formulae
==> Downloading https://github.com/joshuaclayton/unused/archive/v<IP_ADDRESS>.tar.gz
Already downloaded: /Users/gerhard/Library/Caches/Homebrew/unused-<IP_ADDRESS>.tar.gz
==> cabal sandbox init
==> cabal update
==> cabal install --jobs=8 --max-backjumps=100000 --only-dependencies
==> cabal install --jobs=8 --max-backjumps=100000 --prefix=/usr/local/Cellar/unused/<IP_ADDRESS>
Last 15 lines from /Users/gerhard/Library/Logs/Homebrew/unused/04.cabal:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:117:9: error:
    • Variable not in scope:
        (<>) :: Parser (Maybe String) -> Mod f0 a0 -> t
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)
cabal: Leaving directory '.'
cabal: Error: some packages failed to install:
unused-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/joshuaclayton/homebrew-formulae/issues

Looks like a workaround is to install unused via Stack:
brew install haskell-stack
stack update
stack install unused

@julienXX @gerhard Interesting. Did you previously have Haskell installed? It seems to be that Data.Monoid.<> isn't available (it was introduced in base-4.5.0).
I'll do a bit of research in the meantime to see how best to handle this.

@joshuaclayton I had GHC and Stack previously installed yes.

@julienXX @gerhard can you report what ghc-pkg list base returns for you?

@joshuaclayton
λ ghc-pkg list base
/usr/local/Cellar/ghc/8.0.1/lib/ghc-8.0.1/package.conf.d
    base-<IP_ADDRESS>

@julienXX hmm, that makes it more interesting. I'm not sure Homebrew's using the same version when installing, and I'm not sure how best to check either.
I might have to do something along the lines of https://github.com/Rufflewind/wigner-symbols/blob/b389caaaf5f0aa0243d817d13fd1b87c17ad33be/src/Common.hs#L14-L20 to be safe. I'll ask at work about how best to handle this, I'm still relatively new to Haskell so there may be something else I'm missing here. I'd like to avoid recommending Stack installation if at all possible, based on the assumption that most developers won't have that environment set up.

@joshuaclayton I tried to remove GHC & Stack and brew install again but no luck.

@julienXX thanks for taking the time to do that work. I'm thinking optional imports might be the best bet; once I have something, I'll try to hash out a way you could test it (as I've not found a way to reproduce) and we can go from there. It'll probably take me a day or two to line everything up.
Thanks for your patience and help with this!

Sure! Feel free to ping me if you need additional testing.

@julienXX one last question - what does brew --version give you back?

@joshuaclayton
λ brew --version
Homebrew 0.9.9 (git revision 48c4; last commit 2016-08-18)
Homebrew/homebrew-core (git revision d20c; last commit 2016-08-18)

@joshuaclayton I get the same error as @gerhard on install:
~ % brew install unused
==> Installing unused from joshuaclayton/formulae
==> Downloading https://github.com/joshuaclayton/unused/archive/v<IP_ADDRESS>.tar.gz
Already downloaded: /Library/Caches/Homebrew/unused-<IP_ADDRESS>.tar.gz
==> cabal sandbox init
==> cabal update
==> cabal install --jobs=8 --max-backjumps=100000 --only-dependencies
==> cabal install --jobs=8 --max-backjumps=100000 --prefix=/usr/local/Cellar/unused/<IP_ADDRESS>
Last 15 lines from /Users/davemoore/Library/Logs/Homebrew/unused/04.cabal:
    Not in scope: ‘<>’
    Perhaps you meant one of these:
      ‘<$>’ (imported from Options.Applicative),
      ‘<*>’ (imported from Options.Applicative),
      ‘*>’ (imported from Options.Applicative)

app/Main.hs:117:9:
    Not in scope: ‘<>’
    Perhaps you meant one of these:
      ‘<$>’ (imported from Options.Applicative),
      ‘<*>’ (imported from Options.Applicative),
      ‘*>’ (imported from Options.Applicative)
cabal: Error: some packages failed to install:
unused-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

Output from ghc-pkg list base:
~ % ghc-pkg list base
WARNING: cache is out of date: /usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/package.conf.d/package.cache
ghc will see an old view of this package db. Use 'ghc-pkg recache' to fix.
/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/package.conf.d
   base-<IP_ADDRESS>
~ % ghc-pkg recache
~ % ghc-pkg list base
/usr/local/Cellar/ghc/7.10.3b/lib/ghc-7.10.3/package.conf.d
   base-<IP_ADDRESS>

On OSX 10.11.3

$ ghc-pkg list base
/usr/local/Cellar/ghc/8.0.1/lib/ghc-8.0.1/package.conf.d
    base-<IP_ADDRESS>
$ brew --version
Homebrew 0.9.9 (git revision 5c7c9; last commit 2016-08-18)
Homebrew/homebrew-core (git revision 1479; last commit 2016-08-18)

Is there a reason for ctags-5.8_1? universal-ctags works better for me. Happy to open a new issue if it's a distraction.

Same problems here.
macOS 10.11.6
~ > brew install unused
==> Installing unused from joshuaclayton/formulae
==> Downloading https://github.com/joshuaclayton/unused/archive/v<IP_ADDRESS>.tar.gz
Already downloaded: /Users/kevin/Library/Caches/Homebrew/unused-<IP_ADDRESS>.tar.gz
==> cabal sandbox init
==> cabal update
==> cabal install --jobs=8 --max-backjumps=100000 --only-dependencies
==> cabal install --jobs=8 --max-backjumps=100000 --prefix=/usr/local/Cellar/unused/<IP_ADDRESS>
Last 15 lines from /Users/kevin/Library/Logs/Homebrew/unused/04.cabal:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)

app/Main.hs:117:9: error:
    • Variable not in scope:
        (<>) :: Parser (Maybe String) -> Mod f0 a0 -> t
    • Perhaps you meant one of these:
        ‘<$>’ (imported from Options.Applicative),
        ‘<*>’ (imported from Options.Applicative),
        ‘*>’ (imported from Options.Applicative)
cabal: Leaving directory '.'
cabal: Error: some packages failed to install:
unused-<IP_ADDRESS> failed during the building phase. The exception was:
ExitFailure 1

READ THIS: https://git.io/brew-troubleshooting
If reporting this issue please do so at (not Homebrew/brew):
  https://github.com/joshuaclayton/homebrew-formulae/issues

~ > ghc-pkg list base
/usr/local/Cellar/ghc/8.0.1/lib/ghc-8.0.1/package.conf.d
    base-<IP_ADDRESS>
~ > brew --version
Homebrew 0.9.9 (git revision a5dac5; last commit 2016-08-18)
Homebrew/homebrew-core (git revision 1611b; last commit 2016-08-18)

All,
Thanks for the feedback here. I was able to verify failure when installing via Homebrew, and I've also verified a fix that worked for me.
To test:
brew update
brew install joshuaclayton/formulae/unused --HEAD

This will install via head, which is currently pointing to #69. I've confirmed locally that this builds correctly. I'd love for you all to try this and let me know if it addresses the issue.

@joshuaclayton it's worked for me, thanks!

@joshuaclayton works for me :)

@joshuaclayton worked for me, thanks for the quick turnaround.

@joshuaclayton worked for me as well!

All, I've released version <IP_ADDRESS> and updated the Homebrew formulae.
To install a fresh copy, run:
brew update
brew install unused

To upgrade:
brew update
brew upgrade unused

Please let me know if you run into any issues installing, and happy ✂ !


doom/upgrade Error building org-mode

I haven't noticed any misbehaviour in usage yet, but running doom/upgrade today I was met with this error:
      x There was an unexpected error
        Message: File is missing
        Data: (file-missing . "/home/jordan/.emacs.d/.local/straight/build-28.0.50/org-mode/org-loaddefs.el")
        Backtrace:
          (comp-el-to-eln-filename "/home/jordan/.emacs.d/.local/straight/build-28.0
          (file-name-nondirectory (comp-el-to-eln-filename file))
          (concat comp-native-version-dir "/" (file-name-nondirectory (comp-el-to-el
          (doom--eln-file-name "/home/jordan/.emacs.d/.local/straight/build-28.0.50/
          (setq eln-name (doom--eln-file-name file))
          (while (consp --cl-var--) (setq file (car --cl-var--)) (setq eln-name (doo
          (let* ((--cl-var-- doom--eln-output-expected) (file nil) (eln-name nil) (e
          (doom--write-missing-eln-errors)
          (progn (doom--compile-site-packages) (doom--wait-for-compile-jobs) (doom--
          (if built (progn (doom--compile-site-packages) (doom--wait-for-compile-job
      ! Extended backtrace logged to .emacs.d/.local/doom.error.log

Comint exited abnormally with code 255 at Fri Apr 23 21:28:27

System information:
https://pastebin.com/KGhMpti7

I cannot reproduce this. Please update Doom (at least e8d51f2be) and native-comp and try again.

Try upgrading doom manually:cd ~/.emacs.d
git pull
doom sync -up

Try deleting your local org-mode repo: rm -rf ~/.emacs.d/.local/straight/repos/org-mode then doom sync again.

Until I have leads to further suggest this is a Doom issue and not a local one I will close this issue, but feel free to follow up here.

I had the same problem, but with another error message was different.

@MarkusSagen And the contents of that error message was...?


Bug fix in .travis.yml

Coverage remained the same at 57.872% when pulling 7ec27607abea67946a11f68a6a3ea15b710e2cbc on develop into fea88a2e0a62215f4d8afaeae89bfd90f51066ec on master.

Coverage remained the same at 57.872% when pulling 7ec27607abea67946a11f68a6a3ea15b710e2cbc on develop into fea88a2e0a62215f4d8afaeae89bfd90f51066ec on master.


Remove the precision adjustment on the ContourBuilder coordinate creation.

Fixes the issue where parts of holes are placed outside of their shell following precision modifications.

Hey @michaz, might need your input on this. Thanks!