Input with left and right attached labels

One thing I'm trying to do which seems to not be possible with SUI, is to add both a left and right attached label to an input. Currently only left, or right is possible unless I'm missing something. See here:
http://jsfiddle.net/nz8smghx/1/
I've tried playing around with various classes but it all ends up much the same. One label is fine, the other label is facing the wrong way. I completely understand why it's happening, just would be useful to have the option of adding both

Under what circumstances would you need to markup an input on both sides?

I'm trying to implement a form the kind of reads like a sentence in the way it is filled out, and want to use labels/dropdowns on both sides. For example
[input][days after/days before]     [the tasks][input][start/end]

Here the second input has the label attached to the left simply describing that the input is a selection of tasks, and the right of it contains a dropdown to specify the start, or the end of those tasks. It's quite hard to explain but hopefully this makes a little bit of sense?

That makes sense, let me look into it.

@pareeohnos

the second input has the label attached to the left simply describing that the input is a selection of tasks, and the right of it contains a dropdown to specify the start, or the end of those tasks

As you explain, the "right label" it is in reality an action. So, if I didn't misunderstood, we can do the following to get what you describe:

The label

    <input type="text" placeholder="The input">

The action

Start

End

Live example: https://jsbin.com/soxeze/5/edit?html,output

@maturano yeah that's right. Sure I tried that but that's spot on. What about the other way round, where the left side is a dropdown and the right side is the label?

Well ...left action right labeled: https://jsbin.com/sonuhulapo/1/edit?html,output

what the hell! I swear I tried these :( well guess that sorts my issue out then. Thanks :)

Glad to help :smile:

@jlukic Besides of that case (label/input/action), and as far I understand, for this issue it have to implement a both sides class to the input (as non English speaker, I don't know if there is "one word" to describe that)

 Left

  <input type="text" placeholder="the input">

 Right

Because I was thinking in the possibility of move the "position" class to the label:

 Left

  <input type="text" placeholder="the input">

 Right

While I wait for comments, how do you do for highlight the classes in the examples of docs?

It highlights either any classes matching the title of the example, or the data-class (comma separated list) value on
, put two word variations before one word ones in the class list, or it won't match.

Position classes should be on input since there is no previous sibling selector in CSS. There's no way to say "the input before the label with class right".
There's probably a better name than both sides attached I'll think about it.

Added with PR tagged above


Issues with ImageStack support

Observed these issues while modernizing the Gerrymandering examples.
This is all with rasterize=True and working around a HoloViews bug (see https://github.com/holoviz/holoviews/issues/6187):

When setting aggregator=ds.count_cat('cat'), plot options like width are not applied. The issue is there https://github.com/holoviz/hvplot/blob/ce7af4039f9de5c455fa5d517e0ef8759b59f3bb/hvplot/converter.py#L1432, the code sets the element type to be Image in that case and apply the options to the Image elements only, but there's none so they are dropped.
I think I'd expect rasterize=True, by='cat' and rasterize=True, aggregator=ds.count_cat('cat') to create the same ImageStack plot.
When setting rasterize=True, by='cat' the colorbar looks a little weird to me

import datashader as ds
import numpy as np
import pandas as pd
import hvplot.pandas

num=10000

dists = {cat: pd.DataFrame(dict([('x',np.random.normal(x,s,num)),
                                 ('y',np.random.normal(y,s,num)),
                                 ('val',val),
                                 ('cat',cat)]))
         for x,  y,  s,  val, cat in
         [(  2,  2, 0.03, 10, "d1"),
          (  2, -2, 0.10, 20, "d2"),
          ( -2, -2, 0.50, 30, "d3"),
          ( -2,  2, 1.00, 40, "d4"),
          (  0,  0, 3.00, 50, "d5")] }

df = pd.concat(dists,ignore_index=True)
df["cat"]=df["cat"].astype("category")

p = df.hvplot.scatter('x', 'y', rasterize=True, aggregator=ds.count_cat('cat'), width=800)
p

p2 = df.hvplot.scatter('x', 'y', rasterize=True, by='cat', width=800)
p2

When setting rasterize=True, by='cat' the colorbar looks a little weird to me

This is the colorbar of an ImageStack.

When setting rasterize=True, by='cat' the colorbar looks a little weird to me

This is the colorbar of an ImageStack.

Does the HoloViews doc explain how to understand this colorbar? My first impression was that it's a bug.

Nope. I'm not a big fan of it either. But I don't think there is a nice way to represent multiple-dimensional colors in a single colorbar. Even if you have multiple colorbars, some colors could overlap on different, making it hard to see what bar a color belongs to. I see it as alpha values for the colors.
An explanation, if any, could be found in Bokeh. An example of it being used in pure Bokeh can be found here: https://github.com/bokeh/bokeh/pull/12356


[do-not-merge]: add periodic reporter

@bharathkkb - This is probably the best module for the required template: https://github.com/GoogleCloudPlatform/cloud-foundation-toolkit/tree/master/infra/terraform/modules/lint_file

Thanks @apeabody! Will add it there once I have it working.

@apeabody this is ready for a review. I will add it via TF in a follow up once we have it soaked for a bit.


Minimal tests

Took a tiny bit, but the CI tests are now up and running. Additionally

overhauled README
added uninstaller for Windows
aligned functionality of Windows and Linux/WSL/macOS iunstallers

Fantastic work, @pantaray! This should make it easier to detect defects in the included packages.


Problem With MDI Container

Hello,
Thanks in Advance
I am facing one problem as i listed below .Please suggest me solution.
When I write Code Like below then child form doesn't open
FrmTest.MdiParent = Me
FrmTest.Show()
But child opens
if i removes mdiParent Property from Code then it's working properly.

I have the same issue!

Ran into this issue as well. Setting MetroForm's IsMdiContainer to true results in bad drawing artifacts, rendering it essentially useless.

NuGet package
install MetroFramework-<IP_ADDRESS> and MetroModernUi


Update: Replace deprecated web maps with new maps in Open Map (URL)

See common-samples/issues/2832 for details.
This PR replaces the 2 deprecated portal items used in Open Map (URL) sample. Other changes

Removed 1 screenshot
Cleared redirect field in metadata
Replaced the thumbnails in assets folder

Interesting. I didn't noticed that. I took them from the thumbnail images of the portal items:

https://arcgis.com/home/item.html?id=5be0bc3ee36c4e058f7b3cebc21c74e6
https://arcgis.com/home/item.html?id=064f2e898b094a17b84e4a4cd5e5f549

Let me recapture these images.

The new thumbnails are taken from the original authoritative sources

World Terrestrial Ecosystems: https://www.arcgis.com/home/item.html?id=926a206393ec40a590d8caf29ae9a93e
Recent Hurricanes, Cyclones and Typhoons: https://www.arcgis.com/home/item.html?id=adfe292a67f8471a9d8230ef93294414


Fixes #1008 by downloading the Qiskit Metapackage into a tmp folder

…nd getting the tags from that folder

Summary
Fixes #1008
Details and comments
git tag for translated docs returns an empty list in "Qiskit Metapackage Version" because translated docs are in  qiskit-community/qiskit-translations. Instead of doing that, a new function has been aded, It downloads the git repo into a temporary folder and then the git tag command is executed so it always return the latest version list from the official repo.

 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.


Support for alerts

Hi,
I've been playing with the Terraform provider you created and I'm interested in extending it (and therefore this lib) to support managing alerts, since we use Librato's alerting component a lot at Contentful. Do you have any pointers for me to get started, or know of any existing efforts to implement this?
Thanks!

😢

Hi! Sorry, I missed your first comment, my GitHub inbox had filled up quite a bit. I'm not really working much on this library right now, other things have taken priority, but if you want to work on a pull request for alerts I'd be happy to review it. It'd probably be best to look at the files like spaces.go or metrics.go and base the alerts API on that.


[BUG] - ref issue when migrating to react 19

NextUI Version
0.0.0-canary-20241216115515
Describe the bug
Using the merged version which is supposed to migrate to React 19, I get some ref mismatchs here and there.
E.g. in this component which worked fine with React 18:

Does anyone know the fix for this?
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Expected it to behave as in react 18
Expected behavior
Expected it to behave as in react 18
Screenshots or Videos
No response
Operating System Version
macOS
Browser
Chrome

please provide a sandbox instead of a screenshot.


Doc: Fix a broken link

Fix link at https://github.com/ApolloAuto/apollo/blob/master/docs/technical_tutorial/apollo_3.5_technical_tutorial.md
Broken link:
Software architecture and algorithms -> Apollo 3.5 software architecture

 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.

Fix link at https://github.com/ApolloAuto/apollo/blob/master/docs/technical_tutorial/apollo_3.5_technical_tutorial.md
Broken link:
Software architecture and algorithms -> Apollo 3.5 software architecture

Thanks for your PR. As the r3.5.0 branch was released already long ago, I personally think the better solution would be to fix the links on the master branch.

It is now fixed in this commit, 5ac9b8070669c4c66a74895b00e0569b69686481.
Thank you.


Revert "Change the ballerina command to jballerina in vscode extension"

Reverts ballerina-platform/ballerina-lang#15958

Codecov Report

:exclamation: No coverage uploaded for pull request base (jballerina@fed58b3). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             jballerina   #16205   +/-   ##
=============================================
  Coverage              ?   15.26%
  Complexity            ?     3517
=============================================
  Files                 ?     1472
  Lines                 ?    71372
  Branches              ?     8552
=============================================
  Hits                  ?    10896
  Misses                ?    59491
  Partials              ?      985

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 fed58b3...81be105. Read the comment docs.

Closing this as it requires few other changes.


 The problem is that we are not using await for invokeMapMethod in the MethodChannelCamera.initializeCamera method:
https://github.com/flutter/plugins/blob/9aca084f4e02b8a887130e3a8cced87fb218cbf0/packages/camera/camera_platform_interface/lib/src/method_channel/method_channel_camera.dart#L118-L132
The solution to is to asynchronously catch PlatformException and passing it to _completer.
Fix https://github.com/flutter/flutter/issues/96405
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@bparrishMines Could you do the initial review here?


Compare submitted licenses with the rejected as well as yet not approved licenses.

To Check if the submitted licenses matches with that of the once that are already rejected or yet not approved licenses by the legal team.

@goneall @techytushar I have tested it and it is working on my machine I will attach the SC but still I would encourage to test it, just a note for the development it will only work for the issues you have created as the license request you create will be present on the local database but it won't be an issue in the production. Here's the SC

Looks good to me.
@techytushar Let me if you think this is OK to merge.

@techytushar @goneall I have updated the branch with the requested changes. Please have a look. Thanks

Great - I'll go ahead and merge


Incomplete resolve of refs (Including in NSwag tests)

Hello, I am trying to implement a code generator for AsyncAPI in the manner of NSwag, and I have encountered a problem with link resolution.
If the main specification (e.g., asyncapi.yaml) uses an external file (e.g., common.yaml), and this file contains links to elements within itself, then the links are not fully resolved.
I also noticed a similar issue in the NSwag test - ExternalReferenceTests.When_file_contains_response_reference_to_another_file_it_is_loaded. This test uses the files response-reference.json  and common.json. If we add a check on the response scheme, for example on the number of properties of the response, then the test will fail
Example:
Assert.NotEmpty(document.Paths.First().Value.Values.First().Responses.First().Value.ActualResponse.ActualResponse.Schema.ActualProperties);
Steps to reproduce:

Add to the ExternalReferenceTests.When_file_contains_response_reference_to_another_file_it_is_loaded a check for the number of properties in the response schema Assert.NotEmpty(document.Paths.First().Value.Values.First().Responses.First().Value.ActualResponse.ActualResponse.Schema.ActualProperties);
Run test

I don't understand something about how the NJsonSchema works, or is it a bug?

Experiencing the same issue as described here as of 12/05/2024


[plugin] binding does not recognise adat companion

This statement throws a runtime error "missing adat companion". The reason is that the plugin does not recognise
this form and does not set the companion in the binding.
val userMode = autoInstance(sidebarState)?.userMode

Workaround:
val userMode = autoInstance(sidebarState)

if (userMode?.mode == SidebarUserMode.Closed) {

Write unit tests and they work, so I'll close this one.


fix: config.toml and app.toml overwrites

Closes: #XXX
What is the purpose of the change
This code is currently really odd due to being in between the previous and currently expected behavior.
Previously, we only overwrote the config/app values if the value matched a specific value. We now just overwrite regaurdless (unless a specific flag is provided to not overwrite).
This PR standardizes the logic between the two methods, specifically addressing:

Single quote matching (https://github.com/osmosis-labs/osmosis/pull/8056)
Configs with special permissions being overwritten with the default config.toml

Testing and Verifying
Tested the following:

Initialization still behaves as intended
Making a custom change to one of the overwrite values and then removing write permissions prevents the file from being overwritten, but gracefully logs and continues

Special case of wasm config simulation_gas_limit is handled (had to special case logic due to requirement of it being commented out as a valid value)

Summary by CodeRabbit

New Features

Introduced custom configuration options for better control over application behavior.

Improvements

Enhanced the configuration overwrite mechanism to prevent unintended changes.

Documentation

Updated the CHANGELOG with details on recent improvements and features.

Going to convert to draft, want to make a couple more improvements


Implement /tryfix feature that auto fixes links that has redirections

Assume you have a file called A.md which is renamed to B.md and a redirection is added. If there are links that refers to A.md, they should be updated to B.md. All that will be needed to fix this is commenting on the PR with /tryfix and wait for an automated commit. This will require a new yml file:
name: MSDocs build verifier - tryfix
on:
  issue_comment:
    types: [created]

jobs:
  tryfix:
    if: github.event.issue.pull_request != '' && contains(github.event.comment.body, '/tryfix')
    name: Try fix build
    runs-on: ubuntu-latest
    env:
      IS_TRY_FIX: true # differentiates /tryfix from the validation-only run.
      GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
    steps:
    - name: Checkout the repository
      uses: actions/checkout@a81bbbf8298c0fa03ea29cdc473d45769f953675 #@v2
    - name: Checkout Pull Request
      env:
        GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
      run: |
        PR_URL="${{ github.event.issue.pull_request.url }}"
        PR_NUM=${PR_URL##*/}
        echo "Checking out from PR #$PR_NUM based on URL: $PR_URL"
        hub pr checkout $PR_NUM
    - name: Tryfix
      uses: Youssef1313/docs-actions/actions/docs-verifier@docs-actions

    - name: Push changes
      run: |
          git config --global user.name github-actions
          git config --global user.email<EMAIL_ADDRESS>          git commit -am "Automated: Fix links"
          git push

@gewarren @BillWagner @IEvangelist Can we merge this into a feature branch instead of main? Then we can test it in a dummy PR in dotnet/docs without breaking anything (in case this is breaking something)

This looks good to me, but I don't think that having it in a branch would work - I thought we disabled GitHub Actions on feature branches, but I could be wrong.

Recently, a change was tested in https://github.com/dotnet/docs/pull/27387 via a branch in this repository. Has something changed after that?

This looks good to me, but I don't think that having it in a branch would work - I thought we disabled GitHub Actions on feature branches, but I could be wrong.

Recently, a change was tested in dotnet/docs#27387 via a branch in this repository. Has something changed after that?

Ah, yes. You're right let's do the test then.

@IEvangelist Can you create the branch? Thanks!

@IEvangelist Can you create the branch? Thanks!

Hey @Youssef1313, sure - see here: https://github.com/dotnet/docs/tree/youssef-testing

@IEvangelist The needed branch is in this repo not dotnet/docs

@IEvangelist The needed branch is in this repo not dotnet/docs

Oops, I'm sorry. I thought you meant for over in docs - here you go. https://github.com/dotnet/docs-tools/tree/youssef-testing

No problem! Rebased the PR on the new branch. Thanks.

@IEvangelist Anything left to merge this?


Tests segfault

[probcomp-4  pc/crosscat]$ py.test
================================================================== test session starts ===================================================================
platform linux2 -- Python 2.7.6 -- pytest-2.5.1
plugins: flaky
collecting 0 items / 4 errorsSegmentation fault
[probcomp-4  pc/crosscat]$

Should be fixed now.  Try again?

I just noticed this was waiting on me... :-p
So running bare py.test still segfaults for me on both probcomp-4 and osx. But that's not the recommended thing to do. We should be doing ./check.sh instead, and there's likely to be a meaningful difference. So let's just call bare py.test unsupported. Does ./check.sh work?
On probcomp-4, yes.  Or rather, appears to. There's a part after the test that just spews output without indicating whether that output is good or bad.
On osx yes kinda, but no. That part after the test just outright fails, and leaves me with no confidence that everything worked as it should:
========================== 82 passed in 0.81 seconds ===========================
g++ -c src/Cluster.cpp -o obj/Cluster.o -pg -O2  -Wall -Werror -std=c++03 -lm -Iinclude/CrossCat
clang: error: -lm: 'linker' input unused
make: *** [obj/Cluster.o] Error 1

Date: Tue, 22 Dec 2015 11:39:17 -0800
From: Gregory Marton<EMAIL_ADDRESS>So running bare py.test still segfaults for me on both probcomp-4
and osx. But that's not the recommended thing to do. We should be
doing ./check.sh instead, and there's likely to be a meaningful
difference. So let's just call bare py.test unsupported. Does
./check.sh work?
Running py.test ought to work if you have installed Crosscat, and in
particular the same version of Crosscat.  All that check.sh does
beyond running py.test is (a) make a clean build and then (b) set
PYTHONPATH.
On probcomp-4, yes.  Or rather, appears to. There's a part after
the test that just spews output without indicating whether that
output is good or bad.  On osx yes kinda, but no. That part after
the test just outright fails, and leaves me with no confidence that
everything worked as it should:
========================== 82 passed in 0.81 seconds ===========================
g++ -c src/Cluster.cpp -o obj/Cluster.o -pg -O2  -Wall -Werror -std=c++03 -lm -Iinclude/CrossCat
clang: error: -lm: 'linker' input unused
make: *** [obj/Cluster.o] Error 1
The makefile needs to be rewritten.  Among many other problems it has,
the .cpp -> .o rule should not pass -lm to the C++ compiler.  That
part should be easy to fix, though.

d00380f

Confirmed fix on a fresh OS X build.


Crash After Joining Nether

Modpack: Roguelike Adventures and Dungeons
Version: 1.2.9
Minecraft Version: 1.12.2
Forge: <IP_ADDRESS>38
NetherEx: 1.12.2-2.0.10
I am playing on the R.A.D. modpack and it pointed me to  NetherEx as a possible cause of the crash and linked to hastebin for the crash report.
This was my first time entering the Nether, I was able to join. Mined some soul sand and then my shovel hit "gloomy netherack", at which point it crashed.
Brief of crash report:
---- Minecraft Crash Report ----
// I blame LogicTechCorp.

Time: 2019-08-24 23:19:58 BST
Description: Unexpected error

java.lang.NullPointerException
    at logictechcorp.netherex.block.BlockNetherrack.onBlockClicked(BlockNetherrack.java:61)
    at net.minecraft.client.multiplayer.PlayerControllerMP.clickBlock(PlayerControllerMP.java:242)
    at net.minecraft.client.multiplayer.PlayerControllerMP.onPlayerDamageBlock(PlayerControllerMP.java:334)
    at net.minecraft.client.Minecraft.sendClickBlockToController(Minecraft.java:1480)
    at net.minecraft.client.Minecraft.processKeyBinds(Minecraft.java:2268)
    at net.minecraft.client.Minecraft.runTickKeyboard(Minecraft.java:2020)
    at net.minecraft.client.Minecraft.runTick(Minecraft.java:1808)
    at net.minecraft.client.Minecraft.runGameLoop(Minecraft.java:1098)
    at net.minecraft.client.Minecraft.run(Minecraft.java:3942)
    at net.minecraft.client.main.Main.main(SourceFile:123)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke(Method.java:497)
    at net.minecraft.launchwrapper.Launch.launch(Launch.java:135)
    at net.minecraft.launchwrapper.Launch.main(Launch.java:28)

This has been fixed in the latest version.


Update templates access and remove template decorator

Summary:
Replacement of how to access the templates in order to encourage the user to use qml.Template instead of qml.templates.Template.
Also remove the template decorator in the demo tutorial_quantum_metrology.py.

@antalszava I'll double check it :)


use the demo script，GPU OOM

use the demo script，
if you have more images, such as 2000 images,
raise GPU OOM
can anyone help solve the problem?
thanks

@AnddyWang
Hello,
How about run demo.py with --batch_size 32?
Default batch_size is 192 and it could raise OOM.


Webhook open_basedir restriction /dev/null

The undocumented feature of webhooks for gitlab is quite nice.
Allthough we had some trouble with open_basedir restriction:
Warning: fopen(): open_basedir restriction in effect. File(/dev/null) is not within the allowed path(s)

In a nutshell, I could not figure out where the output is redirected to dev/null. Maybe someone can help and optimize that, so the php.ini does not have to be updated.
The exception above is thrown in AbstractWebhook -> getResponse. Unfortunately the exception is not returned in https://github.com/ludofleury/satisfy/blob/bc3ecffe447927582f1b5609a6541fc78e305713/src/Playbloom/Satisfy/Webhook/AbstractWebhook.php#L44
This could get implemented like this:
 throw new ServiceUnavailableHttpException(null, $exception->getMessage());

@rabauss are You sure there is no /dev/null in Your satis configurations?

I could not find any setting to dev/null

I see, UnixPipes tries to open blackhole file as command output, when it is disabled.
https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Process/Pipes/UnixPipes.php#L49
@rabauss please check latest master, should be fixed.

Thanks, for the fix!
For our scenario with our own gitlab it's working.


Span of the error "The body might complete normally" makes writing null-safe closures annoying to write

I've run into this situation several times recently with a null-safety-enabled side project I'm writing for fun.  Short repro:
void f(int Function() callback) {}

void g() {
  f(() {
    // <-- Cursor is here
  });
}

This code has an error because the closure passed to f has an inferred return type of non-nullable int, but there's no return statement in its body.  The analyzer correctly reports this error as error: The body might complete normally, causing 'null' to be returned, but the return type is a potentially non-nullable type..
The problem is, the span of the error covers the entire closure body.  Which means that as I'm trying to write the code for the closure, everything I type is covered in red underlines.  Worse yet, because of the way IntelliJ updates errors when typing occurs, every time I type something, the red underlines go away briefly and then re-appear.  I've trained myself over the years to pause every time I see a red underline appear so I can fix the error the analyzer caught, so this behavior is triggering that habit and I'm getting very distracted trying to code.
I've taken to working around the problem by adding throw 'TODO'; to the bottom of the closure when I start writing it; then I delete this line when I'm done.
It would be better if we could restrict the span of the error to something smaller.  Perhaps the leading { of the closure body, or the => if it's an expression-bodied closure.
Note that this is only a problem for closures.  The span of the error is reasonable if the problematic code is a method or top level function, e.g.:
int f() { // Span of the error just covers the `f`
  // ...
}

@pq @bwilkerson FYI


Restarting docker causes space-cloud containers to not work anymore

The frst time I setup my local space-cloud environment, I closed docker down docker as it was eating my machine's resources. Later on when I started it again, stuff started to break and had to rebuild my entire space-cloud + DB environment again.
Apparently, in the tutorial, it says to use the
docker inspect Command to get the IP of the DB we use. However, I suppose this IP is volatile, and upon relaunching docker, new IPs are assigned to docker.

What if for the initial setup, there would be an option to deploy a DB container first alongside giving the DB url? That way space-cloud can manage the local DB as well

Coming in v0.17.0


Go executables fail when CONFIG_ARM64_64K_PAGES=y

Hi,
I've tried out the merge.arm64 branch (06c5ac294452a4f1a5c22bd91f65eb0e3664fe94) on a system running 64K pages. Unfortunately I get regular crashes when running go executables. Switching to 4K pages, made the problems go away. From a cursory glance at the go code, I think there are some 4K assumptions baked into go that need tweaking.
An example failure:
$ ./go
unexpected fault address 0x5f2220
fatal error: fault
[signal 0xb code=0x2 addr=0x5f2220 pc=0x5f2220]

goroutine 1 [running, locked to thread]:
runtime.throw(0x7574f0, 0x5)
	/home/steven/tmp/go/src/runtime/panic.go:543 +0x108 fp=0x4208085b20 sp=0x4208085b00
runtime.sigpanic()
	/home/steven/tmp/go/src/runtime/sigpanic_unix.go:27 +0x470 fp=0x4208085b70 sp=0x4208085b20
crypto/sha512.init()
	?:0 fp=0x4208085b80 sp=0x4208085b80
crypto/ecdsa.init()
	/home/steven/tmp/go/src/crypto/ecdsa/ecdsa.go:246 +0x88 fp=0x4208085ba0 sp=0x4208085b80
crypto/x509.init()
	/home/steven/tmp/go/src/crypto/x509/x509.go:1922 +0xb4 fp=0x4208085be0 sp=0x4208085ba0
crypto/tls.init()
	/home/steven/tmp/go/src/crypto/tls/tls.go:271 +0x98 fp=0x4208085c30 sp=0x4208085be0
net/http.init()
	/home/steven/tmp/go/src/net/http/transport.go:1281 +0xcc fp=0x4208085e80 sp=0x4208085c30
main.init()
	/home/steven/tmp/go/src/cmd/go/zdefaultcc.go:6 +0xd0 fp=0x4208085f80 sp=0x4208085e80
runtime.main()
	/home/steven/tmp/go/src/runtime/proc.go:83 +0x294 fp=0x4208085fd0 sp=0x4208085f80
runtime.goexit()
	/home/steven/tmp/go/src/runtime/asm_arm64.s:891 +0x4 fp=0x4208085fd0 sp=0x4208085fd0

Cheers,
Steve

Have a look at the _PageSize constant in the runtime package, you can change that to 64k and rebuild.

On 12 Mar 2015, at 22:30, stevecapperlinaro<EMAIL_ADDRESS>wrote:
Hi,
I've tried out the merge.arm64 branch (06c5ac2) on a system running 64K pages. Unfortunately I get regular crashes when running go executables. Switching to 4K pages, made the problems go away. From a cursory glance at the go code, I think there are some 4K assumptions baked into go that need tweaking.
An example failure:
$ ./go
unexpected fault address 0x5f2220
fatal error: fault
[signal 0xb code=0x2 addr=0x5f2220 pc=0x5f2220]
goroutine 1 [running, locked to thread]:
runtime.throw(0x7574f0, 0x5)
/home/steven/tmp/go/src/runtime/panic.go:543 +0x108 fp=0x4208085b20 sp=0x4208085b00
runtime.sigpanic()
/home/steven/tmp/go/src/runtime/sigpanic_unix.go:27 +0x470 fp=0x4208085b70 sp=0x4208085b20
crypto/sha512.init()
?:0 fp=0x4208085b80 sp=0x4208085b80
crypto/ecdsa.init()
/home/steven/tmp/go/src/crypto/ecdsa/ecdsa.go:246 +0x88 fp=0x4208085ba0 sp=0x4208085b80
crypto/x509.init()
/home/steven/tmp/go/src/crypto/x509/x509.go:1922 +0xb4 fp=0x4208085be0 sp=0x4208085ba0
crypto/tls.init()
/home/steven/tmp/go/src/crypto/tls/tls.go:271 +0x98 fp=0x4208085c30 sp=0x4208085be0
net/http.init()
/home/steven/tmp/go/src/net/http/transport.go:1281 +0xcc fp=0x4208085e80 sp=0x4208085c30
main.init()
/home/steven/tmp/go/src/cmd/go/zdefaultcc.go:6 +0xd0 fp=0x4208085f80 sp=0x4208085e80
runtime.main()
/home/steven/tmp/go/src/runtime/proc.go:83 +0x294 fp=0x4208085fd0 sp=0x4208085f80
runtime.goexit()
/home/steven/tmp/go/src/runtime/asm_arm64.s:891 +0x4 fp=0x4208085fd0 sp=0x4208085fd0
Cheers,
Steve
—
Reply to this email directly or view it on GitHub.

Unfortunately Go doesn't work very well with non-4k pages. I am not sure if what Dave mentioned is enough, but you can try.
The plan is, of course, to eventually make Go work well with non-4k pages. I will switch my machine to a kernel with 64k pages.

I think I've fixed this for my systems running 64KB pages with pull request #148 . Please can an adult sanity check this :-). (I'm not a go expert).


Removing /typings/ directory from .npmignore file

Description
This is a one line change to remove the /typings/ directory from the .npmignore file. This change is necessary in order to have that directory and its contents published to NPM with the  npm publish command.
How Has This Been Tested?
This change does not introduce any code changes.
Checklist:

[x] I have added a Signed-off-by.
[x] I have either added documentation to cover my changes or this change requires no new documentation.
[x] I have either added unit tests to cover my changes or this change requires no new tests.
[x] I have run golint and have fixed valid warnings in code I have added or modified. This tool generates false positives so you may choose to ignore some warnings. The goal is clean, consistent, and readable code.

Signed-off-by: Anna D Derbakova<EMAIL_ADDRESS>

LGTM, thanks


Check if it is an image

Hey,
grat package, it is however coming with a big security hole. You need to implement a check that makes sure that it is indeed an image, if not reject it, don't even store it.
Best regards

Thank you for your suggestion. Will check it ASAP.
--
Muhammad Zamroni
Sent with Airmail
On 2 Februari 2016 at 00.05.27, TheTechnoMan<EMAIL_ADDRESS>wrote:
Hey,
grat package, it is however coming with a big security hole. You need to implement a check that makes sure that it is indeed an image, if not reject it, don't even store it.
Best regards
—
Reply to this email directly or view it on GitHub.

Here's the code I implemented:
`$info = getimagesize($_FILES['image']['tmp_name']);
            if ($info === FALSE) {
                $imageError = true;
            }

            if (($info[2] !== IMAGETYPE_GIF) && ($info[2] !== IMAGETYPE_JPEG) && ($info[2] !== IMAGETYPE_PNG)) {
                $imageError = true;
            }

If isset $imageError then reject :)

It willl be great if you could make a PR
--
Muhammad Zamroni
Sent with Airmail
On 2 Februari 2016 at 00.58.14, TheTechnoMan<EMAIL_ADDRESS>wrote:
Here's the code I implemented:
`$info = getimagesize($_FILES['image']['tmp_name']);
if ($info === FALSE) {
$imageError = true;
}
        if (($info[2] !== IMAGETYPE_GIF) && ($info[2] !== IMAGETYPE_JPEG) && ($info[2] !== IMAGETYPE_PNG)) {
            $imageError = true;
        }

If isset $imageError then reject :)
—
Reply to this email directly or view it on GitHub.


Required indicator is not shown

Lookup Field has setRequiredIndicatorVisible() method available, but it has no effect.
Expected behavior is as in Combo Box

Required fields are marked with an indicator next to the label, and become invalid if left empty after having been focused.

Invalid state can be rendered and error message works as expected.

Fixed in release 23.3.5


Expose ts.matchFiles as public API to make implementing readDirectory easier.

Problem:
When implementing a custom host that virtualizes the file system, the readDirectory method is cumbersome to implement in a way that matches tsc behavior.
If you look at the hosts within typescript, the matchFiles method was added to unify behavior with readDirectory between hosts.
While matchFiles is exported, it isn't in the type definition files so I'm assuming it is "private" API.
Either the host should be able to provide a getFileSystemEntries(path: string) => { files: string[], directories: string[] } to get a readDirectory implementation or expose matchFiles.
I'm working on improving the broccoli-typescript-compiler broccoli plugin to have more parity with tsc and VS Code, but I need to virtualize the FS, since broccoli builds from tmp directory and I want the node_modules/@types discovery, tsconfig extends, etc all to work, I need to pretend the tmp dir is inside the project.
Thanks

Our policy has been to only expose APIs that represent language constructs/concepts; helpers and utilities are kept private to limit the API surface area.
This specific API falls under the utilities category, and it kinda falls between two abstraction levels (System, and CompilerHost). We have generally responded to similar requests by asking about duplicating the code, specially for small helpers, would that be an option?
If understand correctly, you are implementing System, and not not using the existing ts.sys, if so, can you use ts.sys?

@mhegazy you cannot make a host that reasonably supports globbing that virtualizes the file system, look at your own examples of hosts, ts.sys doesn't work here because it doesn't allow the file system to be virtualized.
The globing is somewhat unique to TypeScript, it doesn't quite align with other node glob libraries so if you want to ensure your host behaves similar to tsc would require using private API or copying and pasting a large amount of code.
I need to virtualize the file system to fit into how broccoli build system works in tmp dirs, but give errors and resolve config and node_modules from the project dir.

@mhegazy I generally understand the policy regarding API surface but there is a massive amount of useful things in this codebase that is private API, stuff that would make for some amazing static analysis like the flow, container, bindings, imports, exports.  Also, non of the code generation stuff is exposed, no plugins etc.   Exposing this stuff could make for some wickedly cool satellite projects.  I understand not wanting to back yourself into a corner with the API but sometimes worse is better.

The code is open source, and has always been, so to be clear we do not intend to hide anything.
The main issue is that more API surface area means more promises to not break API shape and semantics, and we really hate to break ppl. What complicates matters is we have certain use cases, for APIs, these are the things we test for, other use cases could be broken with no notification.
For the emitter, we have been working on exposing these, see https://github.com/Microsoft/TypeScript/issues/13765, https://github.com/Microsoft/TypeScript/issues/13764 and https://github.com/Microsoft/TypeScript/issues/13763. The printer API is already in, see https://github.com/Microsoft/TypeScript/pull/13761, and the factory is in review, see https://github.com/Microsoft/TypeScript/pull/13825.
For other APIs, please let us know what APIs you want to use, and how you intend to use them, and we would be happy to expose them. we just need to make sure we understand how users are using the API to make sure we know what changes we can and what we can not make in the future.
As for the original issue.. if you really, really, really need it, I suppose we can expose it.

+1, I am working on trying to find all the source files associated with a tsconfig.json and using the existing sys.readDirectory isn't an option because it doesn't work in a virtualized file system.
It would be nice to reuse the code in ts.matchFiles.

For now I am just relying on the fact it is exported and hoping it won't break but I consider readDirectory to be a poor contract for a host to be able to provide and hope to ensure the include/exclude function the way people expect.

@mhegazy can this be looked at again? I've encountered the exact same need.
I have my own implementation for an in-memory file system which I would like to use for tests and caching. Implementing a ParseConfigHost or LanguageServiceHost is easy up to the point where you have to provide a custom readDirectory. I order to get the exact same behavior of TypeScript, I shamelessly augmented the types:
import * as ts from 'typescript'
declare module 'typescript' {
    function matchFiles(
        path: string, extensions: ReadonlyArray<string>, excludes: ReadonlyArray<string>,
        includes: ReadonlyArray<string>, useCaseSensitiveFileNames: boolean, currentDirectory: string,
        depth: number | undefined, getFileSystemEntries: (path: string) => FileSystemEntries): string[]

    export interface FileSystemEntries {
        files: ReadonlyArray<string>
        directories: ReadonlyArray<string>
    }

    export interface JsonSourceFile extends ts.SourceFile {
        parseDiagnostics: ts.Diagnostic[]
    }
    export function getNewLineCharacter(options: ts.CompilerOptions | ts.PrinterOptions): string
}

This can obviously break at any given future TS release without me knowing at build time.
Also, even with matchFiles() exposed, I had to implement:
    function getFileSystemEntries(path: string): ts.FileSystemEntries {
        const files: string[] = []
        const directories: string[] = []
        const entries = syncFs.listDirectorySync(path)
        for (const entry of entries) {
            const node = syncFs.getStatsSync(syncFs.path.join(path, entry))
            if (node.isFile()) {
                files.push(entry)
            } else if (node.isDirectory()) {
                directories.push(entry)
            }
        }
        return {files, directories}
    }

The parseDiagnostics field and getNewLineCharacter() helper are two additional symbols I found useful when creating a custom language service.

@AviVahl I'm trying to do the same thing and just encountered readDirectory myself.  My goal is to publish an in-memory implementation of ts.System for use in browsers demos, experiments, etc.  Do you have a link to what you're working on?

@cspotcode https://github.com/AviVahl/ts-tools/blob/master/packages/typescript-service/src/create-host.ts

I'm currently having to implement a readDirectory etc and I was really hoping this had been addressed 3 years ago when brought up.


Update Mapbox-GL to v1.13.0

Description
Update from v1.11.1 to v1.13.0.
Changelog: https://github.com/mapbox/mapbox-gl-js/releases

The compact attribution icon is no longer visible on mobile (next to the scale).
It's probably related to the change discussed in https://github.com/mapbox/mapbox-gl-js/pull/9991 that may conflict with our css overrides.

@amatissart indeed! It passed through my check. Thanks!


brfs require error

Hi.
I'm using this module with pdfmake and browserify in my setup. I am getting a require error on the brfs module.

Maybe the soultion in #471 helps you?


Showing select box next to each other on MsEdge

I've searched everywhere but no luck to find a solution.
The issue is showing the select box next to each other on MsEdge browser though everything is good with Google Chrome.
This is how it looks on Chrome.

And, this is how it looks on Edge.

In my JS I have this:
$('#myList').selectpicker({            noneSelectedText: 'Choose one or more types'     });
In my HTML, I have this:

I've tried both load options dynamically and write the options statically. But, neither is working. I am wondering if any of you have a solution to this issue.

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case (you can use our preconfigured Plunker), and report back with:

your link
Bootstrap version
bootstrap-select version
specific browser and OS details.

I've later found that it was conflicting with jquery-steps. The issue is solved after I've moved the select input box out of the step wizard form. It would be great if there's a way to resolve such conflicting issue. Thank you.

@MenAtWeb Without seeing an example or having any of the other information I mentioned in my previous message, I'm unable to determine if this is something that could be fixed from within bootstrap-select or not. If you can get me an example I'd be happy to take a look.


flow computation details

In the code IFNet_m.py or IFNet.py, there is this line for flow computation in the IFBlock forward():
flow = tmp[:, :4] * scale * 2
What is the purpose of multiplication to scale * 2?

Thanks for the explanation, that is clear.
I have another question related to flow scaling for arbitrary interpolation. I found two different implementations in your code.
I found this in the inference_video.py file:
  if n == 1:
      return [middle]

  first_half = make_inference(I0, middle, n=n//2) # the floor division // rounds the result down to the nearest whole number
  second_half = make_inference(middle, I1, n=n//2)
  if n%2:
      return [*first_half, middle, *second_half]
  else:
      return [*first_half, *second_half]

which is one way of performing interpolation and arbitrary rates, but this approach is suitable only for rates that are powers of 2 since n equals to 2**args.exp-1.
Another implementation that I found in the forward method of IFNet in the IFNet_m.py file is the following:
timestep = (x[:, :1].clone() * 0 + 1) * timestep
and then, this 'timestep' is used as additional input to the IFBlocks:
flow_d, mask_d = stu[i](torch.cat((img0, img1, timestep, warped_img0, warped_img1, mask), 1), flow, scale=scale[I])
However, I don't see explicitly that it was used this way in your paper, and I am not sure what is the reason to perform it this way.
I experimented by myself adding the parameter 't' for arbitrary interpolation. I first tried to add it for the interpolation equation (equation 1 in your paper). This interpolation is implemented in the following way:
merged[i] = merged[i][0] * mask_list[i] + merged[i][1] * (1 - mask_list[I])
and I modified it to:
merged[i] = merged[i][0] * mask_list[i] * (1 - t) + merged[i][1] * (1 - mask_list[i]) * t
according to the simple 1D piecewise linear interpolation. Of course, that didn't yield any good results, since we cannot achieve arbitrary interpolation by simply scaling the intermediate densities which are the results of the backward warping.
After that, I added scaling 't' before applying warping in the following way:
warped_img0 = warp(img0, flow[:, :2] * (1 - t)) # Ft->0
warped_img1 = warp(img1, flow[:, 2:4] * t) # Ft-> 1

that resulted in better performance. But still, I am not sure if this is the correct way to perform arbitrary interpolation.
I want to ask you, in case RIFE supports arbitrary interpolation at any rates which are not necessarily powers of two, how did you perform this?

I directly input t into the convolutional network, and the model can achieve good results. The recursive implementation is an early demo version. During training, t has some randomness, not limited to powers of two.

I see, thanks for explanations. And do I understand correctly that this is implemented as the following (the forward method of IFNet in the IFNet_m.py file):
timestep = (x[:, :1].clone() * 0 + 1) * timestep
...
flow_d, mask_d = stu[i](torch.cat((img0, img1, timestep, warped_img0, warped_img1, mask), 1), flow, scale=scale[I])

?


Code Review for [data-model] | RoR-Blog App

Hello Dear Code Reviewers Team 👋,
I would like to ask for your kindly help to do the 🔍 Code Review 👀 for my project.
The following requirements 📌 were fulfilled this time:
- 🎯 General requirements

[X] Make sure that there are no linter errors.

🟢 Linters for rubocop and stylelint were installed following the RoR linters guide.
🟢 rubocop ran and no offenses were detected.

[X] Make sure that you used correct flow git-flow.

🟢 The dev branch was created from main.
🟢 Then the dev branch was set as default branch.
🟢 The initialize-data-model branch was created from dev to follow the git-flow.

[X] Make sure that you documented your work in a professional way.

🟢 A README.md file was added to the project.

- 🎯 Project requirements

[X] Initialize your project with Git.

🟢 A new repo ror-blog was created at GitHub and then cloned locally.

[X] Create a new app. Give it a meaningful name.

🟢 RoR-Blog App was created.

[X] Make sure that your project has Postgres database set up.

🟢 When the App was created, the option --database=postgresql was used.

[X] Build your project schema.

🟢 A schema.rb was generated to configure our database.
[X] Create and run the necessary migration files.

🟢 Create**** migrations were created to set all the tables.
🟢 Modify**** migrations were created to set null and default values on tables.
🟢 Add****Ref migrations were created to add the foreign keys and indexes to the tables.

[X] Table and column names should match the ERD diagram.
Note: photo for users table should be a link to an image.

🟢 All column names match the ERD Diagram
🟢 photo column at users table is string to store a link.

[X] Foreign keys should be included.

🟢 All FK that reference another table are included.
[X] Pay attention that in posts table the column holding the foreign key to the users table should be named author_id

🟢 author_id references id at users table.

[X] All columns that are foreign keys should have a corresponding index.

🟢 An INDEX was created for all FK columns.

I really thank you for your support. 🙏
Cheers and Happy Reviewing! 👀
Luis 🙋♂

Thank you @skyv26.


'type not with @model' can't be returned as a multi-auth query result.

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
I was planning to provide non-authorized access to the API using @auth multi-auth directives.
But the "type" which is not annotated with @model cannot be returned as a query result, inspite of the query was fully authorized by @auth. For Instance, "type S3Object" cannot be returned with error message
GraphQL error: Not Authorized to access bucket on type S3Object
GraphQL error: Not Authorized to access region on type S3Object
GraphQL error: Not Authorized to access key on type S3Object

To Reproduce
My extracted schema code
type Class
  @model
  @searchable
  @auth(rules: [{ allow: private, provider: userPools }, { allow: public, provider: apiKey, operations: [read] }])

  {

  id: ID!
  title: String!
  host: User! @connection(name: "ClassHostUser", sortField: "createdAt")
  ...
  }

type User @auth(rules: [{ allow: private, provider: userPools }, { allow: public, provider: apiKey, operations: [read]  }]) @model {
  id: String!
  name: String
  intro: String
  picture: S3Object
}

type S3Object {
  bucket: String!
  region: String!
  key: String!
}

query GetClass($id: ID!) {
  getClass(id: $id) {
    __typename
    id
    title
    host {
      id
      name
      intro
      picture {
        bucket
        region
        key
      }
    }
  }
}

Expected behavior
S3Object should be returned...
but not because it's not authorized to public access and I already know that I can't annotate @auth to 'type S3Object' because it's not annotated with @model.
Additional context
amplify-cli: 3.15.0

You can add the AppSync Authorization Modes directly in your schema.graphql like so:
type S3Object @aws_iam @aws_cognito_user_pools {
  bucket: String!
  region: String!
  key: String!
}

You can add the AppSync Authorization Modes directly in your schema.graphql like so:
type S3Object @aws_iam @aws_cognito_user_pools {
  bucket: String!
  region: String!
  key: String!
}

it works!.. indeed not @aws_iam but @aws_api_key, but it feels so weird to me that I have to use @auth directives because I can't use @aws_api_key @aws_cognito_user_pools directly in schema.graphql with older versions of amplify-cli and it's not documented at all, isn't it?

@rarira This is based on the change in the newer versions of the cli which support multi-auth.
Closing this issue, feel free to comment on this thread if you are stuck on this.

@SwaySway
It should be documented in the @Auth section.
Maybe as a tip for simple types.


My code won't submit

Challenge wherefore-art-thou has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/52.0.2743.116 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.

     function whatIsInAName(objArr, objSource) {
 // What's in a name?
 var arr = [];
 // Only change code below this line
 var objects = [];
 for(i=0; i<objArr.length; i++){
   var obj = objArr[i];
   var objProps = Object.keys(obj);
   var objSourceProps = Object.keys(objSource);
   var countInstance = 0;
   for(j=0; j<objSourceProps.length; j++){  //for each objSourceProp
     var objSourceProp = objSourceProps[j];

     for(k=0;k<objProps.length;k++){ //go through each key in the obj
       var objProp = objProps[k];

       if(objSourceProp==objProp){ //increament count if they're same
         countInstance++;
         if(countInstance == objSourceProps.length){
           objects.push(obj);
         }
         break;
       }

     }
   }
 }

if(objects!==null){
  for(i=0;i<objects.length;i++){
    var object = objects[i];
    var objectKeys = Object.keys(object);
    var objectSourceKeys = Object.keys(objSource);
    var count = 0;
    for(j=0;j<objectSourceKeys.length;j++){
      var objectSourceKey = objectSourceKeys[j];
      for(k=0;k<objectKeys.length;k++){
        var objectKey = objectKeys[k];
        if(object[objectSourceKey]==objSource[objectSourceKey]){
          count++;
          if(count == objectSourceKeys.length){
            arr.push(object);
            break;
          }
        }
      }
    }
  }
}

 // Only change code above this line
 return arr;
}
whatIsInAName([{ first: "Romeo", last: "Montague" }, { first: "Mercutio", last: null }, { first: "Tybalt", last: "Capulet" }], { last: "Capulet" });

My code won't submit even though it is correct. It just pops a submit link on the bottom right but when I've clicked on the submit, it won't submit still. I don't know what is happening.
When I checked the console, it says TypeError: c is undefined and a bunch of other link after the Stack trace: key

Please use the Help chat room for getting challenge related help. Happy Coding!


Add Loader Icon in all buttons while submitting

Currently I just disable the button, it will be nice if we can improve by adding the loader also.
Based on #43, you can use Loader using icon and combined with css animate from tailwind:
e.g: <Loader2 className="animate-spin" />

Hi @mazipan, Do I need to get it assigned first OR shall I raise my PR directly.?

No need, but if you want, just create a draft PR first to reserve the ticket


PyKOB.config.py: Doesn't process 'Station' or 'Wire' values correctly from the configuration file

The config.py module processes setting the Station and Wire values into the preference file, but when reading it back it always processes as None.

I tested locally. Please comment with your testing if it doesn't work.


RE3 Crashes whenever I install

I switched to the older build for RE3 but reframework instantly crashes the game. This only happens when I install reframework as without it, RE3 works fine as an fresh install.

Need more info. Post your re2_framework_log.txt

[2022-06-29 23:11:47.313] [REFramework] [error] Exception occurred: c0000005
[2022-06-29 23:11:47.313] [REFramework] [error] RIP: 7ffb536c9c85
[2022-06-29 23:11:47.313] [REFramework] [error] RSP: c9667af480
[2022-06-29 23:11:47.313] [REFramework] [error] RCX: 7ff6b2e7e950
[2022-06-29 23:11:47.313] [REFramework] [error] RDX: 0
[2022-06-29 23:11:47.313] [REFramework] [error] R8: 0
[2022-06-29 23:11:47.313] [REFramework] [error] R9: 40
[2022-06-29 23:11:47.313] [REFramework] [error] R10: 0
[2022-06-29 23:11:47.313] [REFramework] [error] R11: c9667af3f0
[2022-06-29 23:11:47.313] [REFramework] [error] R12: 100000001b3
[2022-06-29 23:11:47.313] [REFramework] [error] R13: ffffffffffffffff
[2022-06-29 23:11:47.313] [REFramework] [error] R14: 7ff6b2e7e950
[2022-06-29 23:11:47.313] [REFramework] [error] R15: 0
[2022-06-29 23:11:47.313] [REFramework] [error] RAX: 0
[2022-06-29 23:11:47.313] [REFramework] [error] RBX: c9667af688
[2022-06-29 23:11:47.313] [REFramework] [error] RBP: c9667af580
[2022-06-29 23:11:47.313] [REFramework] [error] RSI: 7ff6b4973310
[2022-06-29 23:11:47.313] [REFramework] [error] RDI: 0
[2022-06-29 23:11:47.313] [REFramework] [error] EFLAGS: 10256
[2022-06-29 23:11:47.313] [REFramework] [error] CS: 33
[2022-06-29 23:11:47.313] [REFramework] [error] DS: 2b
[2022-06-29 23:11:47.313] [REFramework] [error] ES: 2b
[2022-06-29 23:11:47.313] [REFramework] [error] FS: 53
[2022-06-29 23:11:47.313] [REFramework] [error] GS: 2b
[2022-06-29 23:11:47.313] [REFramework] [error] SS: 2b
[2022-06-29 23:11:47.313] [REFramework] [error] Module: 7ffb53460000 D:\SteamLibrary\steamapps\common\RE3\DINPUT8.dll
[2022-06-29 23:11:47.314] [REFramework] [error] Attempting to write dump to D:\SteamLibrary\steamapps\common\RE3\reframework_crash.dmp

The entire thing was too long

You should still be able to post the entire thing. Just drag the .txt file into the window.

I tried both the old and new version. Ill try it with the new build.

I got it to work and Im not sure how. RE2 didnt have this same issue even though I was using the same one Installed the day I installed the RE3 one.


Moved retsSession to instance scope

Multiple RetsClient instances may co-exist now

Great, looks like this will prevent issues when using 2 simultaneous instances of rets-client that access different servers.  Thanks!

Thanks, I also incorporated into version 1 (v1.1.23)
On Wed, Jul 8, 2015 at 8:20 AM, jessecstewart<EMAIL_ADDRESS>wrote:

Multiple RetsClient instances may co-exist now
You can view, comment on, or merge this pull request online at:
https://github.com/sbruno81/rets-client/pull/8
Commit Summary

Moved retsSession to instance scope

File Changes

M lib/metadata.js
https://github.com/sbruno81/rets-client/pull/8/files#diff-0 (33)

Patch Links:

https://github.com/sbruno81/rets-client/pull/8.patch
https://github.com/sbruno81/rets-client/pull/8.diff

—
Reply to this email directly or view it on GitHub
https://github.com/sbruno81/rets-client/pull/8.


Mod Options panel has no scroll bar

The Mod Options panel has no visible scroll bar, even if the options extend past the bottom of the panel. This happens with script mods with a lot of options, such as Randomizer. Scrolling with the mouse wheel still works.
I believe this is being caused by the combination flag ImGuiWindowFlags_NoDecoration on the ImGui window:
https://github.com/spelunky-fyi/Playlunky/blob/08628db9c53a6125afb920f88542953a1951c815/source/playlunky/mod/mod_manager.cpp#L1026-L1029
That flag is redundant with the others, since NoTitleBar, NoResize, and NoCollapse are already explicitly set on the window, so all it serves to do is add NoScrollbar.

Enabled scrollbars and other window controls in #51


iPadOS: Korean input isn't combined properly

OS Version: iPadOS 14.3
Version: 1.53.0-insider
Commit: 32b28f6f8f9dfe36efad102cc460e14a84ba756a
Date: 2021-01-19T06:08:51.817Z
Browser: Mozilla/5.0 (iPad; CPU OS 14_3 like Mac OS X) AppleWebKit/605.1.15 (KHTML, like Gecko) CriOS/87.0.4280.77 Mobile/15E148 Safari/604.1
Originally reported in code-server: https://github.com/cdr/code-server/issues/1314
Reproduce

Add Korean keyboard under General > Keyboard > Keyboards > Add New Keyboard > Korean
Open code-server
Open file (e.g. markdown file)
Switch to Korean keyboard
Start typing: ㄱㅏ

Expected
가
It works properly on VSCode + macOS.
Version: 1.52.1
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:30:02.420Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin x64 20.2.0
Actual
ㄱ가
But not on Codespaces/code-server + macOS
https://user-images.githubusercontent.com/3806031/105103759-58f0a380-5a6e-11eb-892c-ccf84e63b6c1.MP4

Does this issue occur when all extensions are disabled?: Yes

I could not reproduce on Safari:
https://user-images.githubusercontent.com/5047891/105535686-76807000-5cef-11eb-9df6-cc8a68ba9fa8.mp4

I could not reproduce on Safari:
https://user-images.githubusercontent.com/5047891/105535686-76807000-5cef-11eb-9df6-cc8a68ba9fa8.mp4

@alexdima I was using Chrome (shouldn't matter though, right?)
Is there anything more I can do to help.

@alexdima I was using Chrome (shouldn't matter though, right?)
Is there anything more I can do to help.

I am having the same problem when using mobile Safari.

Same with me. Any progress here?

I am fairly confident that this was caused by us writing to the textarea (changing its value) during the compositionstart event. In other words, I think the fix for #138095 will also fix this issue. I would be very thankful if you could verify that the issue no longer reproduces using tomorrow's Insiders that should be released in 9-10 hrs.

I would be very thankful if you could verify that the issue no longer reproduces using tomorrow's Insiders that should be released in 9-10 hrs.

Just confirmed, it is working as expected on https://insiders.vscode.dev
Video
https://user-images.githubusercontent.com/3806031/144130880-08b55b11-3c85-4c3d-8f95-740ebbf83ed9.MP4
Thanks for fixing this!

Thank you very much for verifying!


Fixed broken link

Category

[x] Content fix
[ ] New article
Related issues: fixes #2544

What's in this Pull Request?
Fixed broken msdn link

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/excel-services-error-codes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/excel-services-error-codes.md
:white_check_mark:Succeeded
View

docs/general-development/how-to-use-the-subcode-property-to-capture-error-codes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/general-development/excel-services-error-codes.md
:white_check_mark:Succeeded
View

docs/general-development/how-to-use-the-subcode-property-to-capture-error-codes.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add Dockerfile

I just discovered KMonad and found it awesome. Thank you very much for sharing this fine piece of software.
Since I wanted to try the latest master branch, I had to go for building. I'm on Ubuntu Bionic and I didn't feel like to install more software just to build it. So I went for building under Docker. I've created a Dockerfile based on Haskell 9 on Debian Buster which just adds source files and runs build instructions.
With just this file, I've been able to get a binary freshly built from source by just doing this in the Dockerfile directory:
# Build a Docker image which will contain the binary.
$ docker build -t kmonad-builder .
# Spin up a Docker container from the built image, which will copy the
# built binary to a directory bind-mounted to the host's current dir.
$ docker run --rm -it -v ${PWD}:/host/ kmonad-builder bash -c 'cp -vp /root/.local/bin/kmonad /host/'
# Clean up build image.
$ docker rmi kmonad-builder

And... ta-da! I found in my current dir a kmonad binary ready to use (actually, I'm using this since a couple days ago without problems, so far). It builds also on your keycode-refactor branch, but I've not used that binary.
As an added bonus, you can build Docker images directly from public repo URLs, whithout even needing to clone (mine used as example here, in case you want to copy/paste for testing):
$ docker build -t kmonad-builder github.com/pataquets/kmonad.git

Now, about the image:
First and foremost, since I was struggling to find an up-to-date binary for my system and I was able to get it easily using Docker, as I do with other software, I wanted to share it in case it can be of any help to others. You only need Docker installed, since the build happens isolated with all the build toolchain already installed in the building container. No other software needed in the host. This is way easier, tidier and safer than having to install build deps and build on your system. Looks like the built artifacts availability of the project could benefit of some love.
This Dockerfile can even be used in a hack/build/test development workflow (it's optimized to take the most advantage possible of Docker's build cache).
It also might come handy later when you get to do some CI, as I've seen in #264.
Otherwise, as it is now, it does not fits the 'use Docker to run the app' use-case as it's usually done in Docker. Hooking into host's uinput/udev/etc from inside a Docker container would be a first for me to sort out (I've accomplished using host's FUSE and disk/audio devices from Docker containers in the past, thou). Also, the built artifact weighs 3.85 Gb (!), but this can be solved, if necessary. But anyway, I feel it is a useful enough tool worth contributing, and also my way of giving back to this awesome software. I would be more than happy to submit a documentation PR, once we sort out if this is OK.
I can be of any further help, do not hesitate to ask. I wish you all the best of health.

@slotThe : Sure! I'll add the docs. A couple questions:

Most of build rationale/docs will be a shortened version of the issue summary.
I'm assuming I should add docs to doc/Readme in the building section.
Not sure, but I guess binary is statically linked.
As for Docker usage, since I'm a long time user, I'm unsure about what's not obvious about it. Since you're playing the not-familiar-with-Docker user role, which is the main target here, would you mind in ellaborating what's not clear, so I can keep it in mind?

Thanks for the prompt review.

* Most of build rationale/docs will be a shortened version of the issue summary.

That's totally fine—don't have to write things twice when there's already quite a good guide written down somewhere

* I'm assuming I should add docs to doc/Readme in the building section.

Yes I would put it around here to the other "how to compile" sections.

* Not sure, but I guess binary is statically linked.

It probably is, but it's most likely linked against glibc.  People running other libc's (like musl, for example) will probably be able to run the executable.  I'm not sure if this is worth mentioning though; I would imagine no one sees "Debian Buster Docker image" and expects that to be linked against musl.

* As for Docker usage, since I'm a long time user, I'm unsure about what's not obvious about it. Since you're playing the not-familiar-with-Docker user role, which is the main target here, would you mind in ellaborating what's not clear, so I can keep it in mind?

I'm honestly not sure if I'm in a position to judge this.  I'm less the "new to Docker and wants to install kmonad with it" user and more the "has never used Docker and doesn't plan on doing so for the forseeable future" :)  But really, it was just what to actually execute to get the image to build things, i.e. especially this part:

With just this file, I've been able to get a binary freshly built from source by just doing this in the Dockerfile directory:
# Build a Docker image which will contain the binary.
$ docker build -t kmonad-builder .
# Spin up a Docker container from the built image, which will copy the
# built binary to a directory bind-mounted to the host's current dir.
$ docker run --rm -it -v ${PWD}:/host/ kmonad-builder bash -c 'cp -vp /root/.local/bin/kmonad /host/'
# Clean up build image.
$ docker rmi kmonad-builder

And... ta-da! I found in my current dir a kmonad binary ready to use (actually, I'm using this since a couple days ago without problems, so far). It builds also on your keycode-refactor branch, but I've not used that binary.
As an added bonus, you can build Docker images directly from public repo URLs, whithout even needing to clone (mine used as example here, in case you want to copy/paste for testing):
$ docker build -t kmonad-builder github.com/pataquets/kmonad.git

Telling users that this is really only for building and one then needs to copy over the executable would also be valuable I think.

@slotThe: Docs added, I hope they're good enough.
I've checked my Docker-built binary for linking:
$ ldd `which kmonad`
        linux-vdso.so.1 (0x00007ffceeead000)
        libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f1ae0315000)
        librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007f1ae010d000)
        libutil.so.1 => /lib/x86_64-linux-gnu/libutil.so.1 (0x00007f1adff0a000)
        libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f1adfd06000)
        libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f1adfae7000)
        libgmp.so.10 => /usr/lib/x86_64-linux-gnu/libgmp.so.10 (0x00007f1adf866000)
        libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f1adf475000)
        /lib64/ld-linux-x86-64.so.2 (0x00007f1ae06b3000)

I've digged a bit to learn about static compilation for maybe using it here. I've seen that it can be done with nix, which could be installed for building. But, if I was given the choice, ideally it would be just one more Make/Cabal/etc target, which would be passed as a build argument using Docker's ARG statement.

@slotThe: Docs added, I hope they're good enough.

Looks fine to me!

I've digged a bit to learn about static compilation for maybe using it here. I've seen that it can be done with nix, which could be installed for building. But, if I was given the choice, ideally it would be just one more Make/Cabal/etc target, which would be passed as a build argument using Docker's ARG statement.

If you want to do this in this PR then feel free to.  Don't feel pressured though, I would be happy to merge it as-is and perhaps if someone want to add this later on they can do it.

I would like to do it in another PR when/if the need arises. I expect it to be quite involving to achieve, so it would be a separate issue on its own.
I'm about to lose bandwidth due to holidays and I also would like to watch for feedback in case someone reports.

Sure; thanks!

@slotThe: May I suggest you to merge/cherry pick this into keyboard-refactori branch to enable others to easily test that branch. I've just tried and it built successfully and running it without noticing any problem.
I've been reviewing some issues and this might lower the friction for some interested parties who otherwise may not take the plunge to test it due to building complexity.


neon-cli login should perform an operation to ensure connectivity

@granth7 ran into a possible issue last week where he was given a cluster login file for a new cluster, logged into the cluster but was subsequently unable to perform any operations, apparently due to misconfigured node IP addresses (or something).
The client should perform an operation like kubectl get pods or something during login to verify that cluster operations can be actually be performed.

DONE


Fix field names in generated json statement

Fixes: #1128
To keep backwards compatibility with previous struct versions (e.g, github.com/in-toto/in-toto-golang/in_toto) we have to use protojson.Marshal function to transform the struct to its json representation so the tags defined in the intoto.Statement are follow and applied to the resulting string.

Changes
When getting the payload object we check if it is an instance of proto.Message to use the proper Marshall function.

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[ ] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

/cherry-pick release-v0.21.x

/approve

/lgtm

/kind bug


Added a bengali book on C++

Fixes #7463
What does this PR do?
Added a bengali book on C++ to help readers
Description
A bengali book on C++ programming to help users code their way to greatness. A free and wholesome ebook for users in the form of a downloadable PDF.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

In general, we do not link to file sharing sites, or file hosting sites such as google docs. or dropbox.


Remove extra "add" in sentence

Signed-off-by: Tomás Mota<EMAIL_ADDRESS>small doc fix

Thank you for contributing during Hacktoberfest 🍂! You can claim the Backstage Hacktoberfest Holopin at https://bck.st/hacktoberfest-holopin 🙌🏻


Sound not correct after loading init preset

Tested using the current "next" branch. (was testing the fix for issue #32)
Tested with Hive 2 on M1 Mac with Monterey in Bitwig 4.4.3.
When loading the init preset, not all parameters are set to the correct values.
Example using Hive:

load preset "Detuned Punk" (preset doesn't really matter, but sound issue is quite obvious with that one)
now right-click on Hive's main data display and choose "Init"
play a note, sounds wrong
choose the Init preset once more, now the sound will be correct

i wonder if you have parameters with the id bit set > 2^31? We fixed that a week or so ago and it might cause what you mention.

i wonder if you have parameters with the id bit set > 2^31? We fixed that a week or so ago and it might cause what you mention.

I've checked that, all is correct. I suspect that this has been solved for now - feel free to re open, when it can be replicated.

Did some tests and this seems to be fixed indeed. Thanks a bunch.


Add acl permissions support

Add ability to configure ACL permissions
Right now, permissions are hard-coded to 'public-read'.  This change adds a new prop to DropzoneS3Uploader called aclPermissions.  The value is a string.  Any canned aws ACL should be valid, we are using 'private' for our purposes. http://docs.aws.amazon.com/AmazonS3/latest/dev/acl-overview.html#canned-acl.
Also, we are specifying the directory for babel in our build commands.

Any updates on this? Would be great add this functionality

Hi Founderlab,
We are looking for a solution to use private permission as well, and spent many hours figuring out why it isn't working before finding this PR.  I'd imagine many other users would find this functionality super useful.
K


Fix standalone Citadel yaml file

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

Needs approval from an approver in each of these files:

install/OWNERS

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

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

Test name
Commit
Details
Rerun command

prow/istio-unit-tests.sh
4813d8595db118c4d80e14f3a1c013751b92055e
link
/test istio-unit-tests

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

/approve
/lgtm

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

Needs approval from an approver in each of these files:

install/OWNERS [GregHanson]

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

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

Needs approval from an approver in each of these files:

install/OWNERS [GregHanson,wattli]

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


No LICENSE file provided.

Headers refer to a file named "LICENSE" which is not included in the distribution.  We (Disney) cannot use open source projects unless there is clear and complete licensing information.
Thanks!

thanks -- fixed. :)


Infection Rate Reset Prior if Normalized Input Cases is Less than 1

This PR addresses issue #
Please Check

[ ] Are the JSON schemas updated?
[ ] Are tests passing?

Closed because outdated. Replaced by https://github.com/covid-projections/covid-data-model/pull/1130.


handle case when node.fetch returns None

This PR fixes an exception thrown when a reader returns None
#2222

Ah, that's how None coming instead of []. Thanks!
Will backport it to 1.1.x and prepare release notes of 1.1.2, now we have more then enough patches for it.

Codecov Report

Merging #2223 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2223      +/-   ##
==========================================
+ Coverage   79.47%   79.47%   +<.01%
==========================================
  Files          85       85
  Lines        8842     8845       +3
  Branches     1888     1889       +1
==========================================
+ Hits         7027     7030       +3
  Misses       1550     1550
  Partials      265      265

Impacted Files
Coverage Δ

webapp/graphite/finders/utils.py
96.42% <100%> (+0.2%)
:arrow_up:

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 a01dbe9...f275242. Read the comment docs.


🚸 Expose PydanticUndefinedType

Change Summary
The goal here is to expose the PydanticUndefinedType. I'm unsure if this is the best way to do it.
The motivation is that FastAPI wants the type of PydanticUndefined, and I can't get that since PydanticUndefined is an object(). See FastAPI code for more details.
Checklist

[X] Unit tests for the changes exist
[X] Documentation reflects the changes where applicable
[ ] Pydantic tests pass with this pydantic-core (except for expected changes)
[X] My PR is ready to review, please add a comment including the phrase "please review" to assign reviewers

Selected Reviewer: @davidhewitt

please review

Codecov Report

Merging #693 (7bb97b9) into main (4c2a8f8) will increase coverage by 0.00%.
The diff coverage is 100.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.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #693   +/-   ##
=======================================
  Coverage   93.72%   93.72%
=======================================
  Files          99       99
  Lines       13804    13805    +1
  Branches       25       25
=======================================
+ Hits        12938    12939    +1
  Misses        860      860
  Partials        6        6

Impacted Files
Coverage Δ

pydantic_core/__init__.py
81.81% <ø> (ø)

src/argument_markers.rs
86.88% <100.00%> (ø)

src/lib.rs
100.00% <100.00%> (ø)

src/validators/model.rs
98.41% <100.00%> (ø)

src/validators/with_default.rs
99.10% <100.00%> (ø)

Continue to review full report in Codecov by Sentry.

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

Ha, my first PR in rust. 😎


Onnx Conversion for Faster-RCNN

Has pytorch2onnx.py ever been used to convert a standard Faster-RCNN model? While I  understand it's still an experimental feature, I thought I could use pytorch2onnx to convert my Faster-RCNN model but it did not work.

To provide more context, these are the things I've tried:
I tried running pytorch2onnx.py however this creates an invalid graph and fails the onnx check (onnx.checker.check_model(onnx_model)) due to two Constant nodes containing the device name ('cpu'). The same error as in #2299. As stated in #2299, I was able to create a valid graph by removing these two nodes.
However, I was not able to load this onnx model. I tried using onnxruntime and caffe2 onnx backend.

Loading with onnxruntime fails with Fail: [ONNXRuntimeError] : 1 : FAIL : Fatal error: ATen is not a registered function/op
Loading with Caffe2 Onnx backend (opset 9) fails with Don't know how to translate op roi_align which I guess makes sense since RoiAlign support was added opset 10 onwards.

I had also tried #1386 which did allow me to create a onnx model and run inference. However, the model was giving the same outputs for different inputs.

Pls. try with the latest mmcv and mmdetection. Reopen it if you still have any issue.


Area Chair console does not show AC Recommendation scores

See the screenshot. I would expect that "AC Recommendation:" would display the given score (which is called "Overall Assessment" in the meta review), for example
"AC Recommendation: 3" or better "AC Recommendation: 3 (There are major points that may be revised)"

This is not yet implemented, but is on our improvement list. I'm closing the issue here since we want to keep these issues focused on the website.


HUD in bristle woods rift

The hover for standard cheese has the quantities showing on next line, and spilling over bottom.
Looks normal when turning off HUD for that area

Fixed in v0.31.0


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-09-04)
Features

first release (1dcf327)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


omnisharp-atom toggle

i install atom via brew cask then install packages like omnisharp-atom and autocomple-plus. then when i toggle the omnisharp-atom i have this error.
"Omnisharp Atom needs the package language-csharp (version ^0.3.0) to be installed. You have an older version 0.5.0."
Note: language-csharp plugin is already install when i install the atom.

This message is from code in the omnisharp-atom package because Atom doesn't have the concept of package dependencies like this. Specifically here:
https://github.com/OmniSharp/omnisharp-atom/blob/master/lib/omnisharp-atom/dependency-checker.coffee
Please file a bug on the omnisharp-atom package for this.


Fixed CRDs owned by GitOps Operator in the clusterserviceversion mani…

What type of PR is this?

Uncomment only one  /kind line, and delete the rest.
For example, > /kind bug would simply become: /kind bug

/kind bug

What does this PR do / why we need it:
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes https://issues.redhat.com/browse/GITOPS-4513
Manual cherry-pick of https://github.com/redhat-developer/gitops-operator/pull/678
Test acceptance criteria:

[ ] Unit Test
[ ] E2E Test

How to test changes / Special notes to the reviewer:

/lgtm
/approve


Why choose 6 residual blocks in the generator but not 9 as in CycleGAN?

Hi, does anyone know why the BeautyGAN authors choose 6 residual blocks in the generator but not 9 as in CycleGAN because the training images are 256*256?
I also find that some unofficial implementations based on tensorflow are using 9 blocks. For example, https://github.com/baldFemale/beautyGAN-tf-Implement
It's really weird!

Have you figured it out why?

Do u know the difference between BeautyGAN and CycleGAN? If i want to train ,which one should i choose


script fails after retries

no kudos are given, script basically times out after retries.
error started showing up recently, maybe this week?

In general, it would be very helpful

before raising an issue: See if you had updated the fork.
when raising an issue: include some basic details, such as which line is specifically timing out (can be found in the logs) and some hints to help others to reproduce your issue.

Closing this issue for now due to inactivity. Feel free to reopen to add any details requested above.