Inline project methods

Sometime these methods are not inline under opt-level=z compile option, forcing inlining could be beneficial for performance and binary size.

Thanks for the PR!
It is not clear that it is always reasonable to do this, since a large number of inline attributes will affect compile time (e.g., see https://github.com/rust-lang/hashbrown/pull/119)
Also, if you want to fully inline project_replace without LTO, the inline attribute may also be needed for private types in pin-project-lite.
In any case, please provide specific data on what this will improve. (generated asm[^1], reproducible benchmarks, etc.)
[^1]: depend on the compiler version, but pin-project-lite is available on godbolt: https://godbolt.org/z/eTxGzb5E7

Here is my attempt at comparing the effect of inlining: https://github.com/EFanZh/pin-project-lite/tree/compare-inlining. (Sorry for not using godbolt, I couldn’t figure out how to patch pin-project-lite with a inlined version.)
Under opt-level="z", here are the assembly code generated before and after inlining:

Not inlined: https://github.com/EFanZh/pin-project-lite/blob/compare-inlining/compare-original-oz.s.
Inlined: https://github.com/EFanZh/pin-project-lite/blob/compare-inlining/compare-inline-oz.s.

Also, I have tested the effects of #[inline] with opt-level=3, and the result is the same:

Not inlined: https://github.com/EFanZh/pin-project-lite/blob/compare-inlining/compare-original-o3.s.
Inlined: https://github.com/EFanZh/pin-project-lite/blob/compare-inlining/compare-inlined-o3.s.

These examples are handcrafted by me, maybe it’s better to test on a real project. But I don’t know any projects uses pin-project-lite extensively, is there any suggestions?

I tested this patch on tokio-console with this branch: https://github.com/EFanZh/console/tree/compare-inlining-pin-project-methods. Here is the binary sizes comparison:
original release     8145408
patched release      8138240

original release -z  6450688
patched release -z   6426112

Currently, I have only tested binary size changes. @taiki-e do you need other tests? For example: runtime performance benchmarks?

Hi @taiki-e: Is there anything I can do to get this PR merged?

Sorry for the late reply. And thanks for the investigation. Assuming there is no impact on compile time, the effects of this all appear to be positive.
As for compile time, @nnethercote previously measured it by building async-std (https://github.com/taiki-e/pin-project-lite/pull/71). That said, I'm not sure if building the library is the right way to measure the impact of #[inline] on compile time. We may need to measure the compile time of binaries that use tokio, async-std, etc.

Again, I have tested the compile time of tokio-console using this script:
benchmark() {
    target_report=$1

    shift

    cargo clean
    cargo fetch "$@"
    cargo build --release --timings "$@"
    mv 'target/cargo-timings/cargo-timing.html' "$target_report"
}

patching_args=(
    --config 'patch.crates-io.pin-project-lite.git="https://github.com/EFanZh/pin-project-lite"' \
    --config 'patch.crates-io.pin-project-lite.rev="5ebf6921de62887572fa4e0477e725567de176df"'
)

for opt_level in '"z"' '3'; do
    for lto in 'off' 'thin' 'fat'; do
        for codegen_units in '16' '1'; do
            base_args=(
                --config "profile.release.opt-level=$opt_level"
                --config "profile.release.lto=\"$lto\""
                --config "profile.release.codegen-units=$codegen_units"
            )

            report_base_name="opt_level=${opt_level//\"/} lto=$lto codegen_units=$codegen_units"

            benchmark "$report_base_name - original.html" "${base_args[@]}"
            benchmark "$report_base_name - patched.html" "${base_args[@]}" "${patching_args[@]}"
        done
    done
done

Here is my test result:

Configuration
Original
Patched

opt_level=z lto=off codegen_units=16
51.7s
46.3s

opt_level=z lto=off codegen_units=1
67.7s
67.6s

opt_level=z lto=thin codegen_units=16
55.9s
55.9s

opt_level=z lto=thin codegen_units=1
76.2s
69.4s

opt_level=z lto=fat codegen_units=16
92.1s
91.7s

opt_level=z lto=fat codegen_units=1
97.2s
97.3s

opt_level=3 lto=off codegen_units=16
83.9s
83.4s

opt_level=3 lto=off codegen_units=1
103.9s
103.6s

opt_level=3 lto=thin codegen_units=16
68.6s
68.6s

opt_level=3 lto=thin codegen_units=1
115.2s
115.6s

opt_level=3 lto=fat codegen_units=16
140.2s
137.2s

opt_level=3 lto=fat codegen_units=1
141.9s
142.5s

I didn’t see significant compile time change for this change.

Published in v0.2.10. Thanks @EFanZh.


[FIX] Fix TypeError: invalid file: PosixPath

Hello,
When using Ubuntu 18.04 and Python 3.5.6, the following error was arises when calling show function:
  File "./app.py", line 6, in <module>
    report.build_report()
  File "/home/jvicente/myreport/report_class.py", line 66, in build_report
    doc.show()
  File "/home/jvicente/anaconda2/envs/qcreport/lib/python3.5/site-packages/handout/handout.py", line 102, in show
    with open(filename, 'w') as f:
TypeError: invalid file: PosixPath('output/index.html')

With this fixed, that problem is solved.

Could you try passing in str(path) instead of path to see if that's the issue, please?

Could you try passing in str(path) instead of path to see if that's the issue, please?

Like this?
>> path
'./output'
>> type(path)
<class 'str'>

path = str(self.path)
doc = handout.Handout(path, 'output')

  File "/home/jvicente/myreport/report_class.py", line 567, in build_report
    doc.show()
  File "/home/jvicente/anaconda2/envs/qcreport/lib/python3.5/site-packages/handout/handout.py", line 102, in show
    with open(filename, 'w') as f:
TypeError: invalid file: PosixPath('output/index.html')

Same error I'm afraid :(


improve error string if parse_binds is not set

When using rtic-syntax as a library and parsing the below
let (_app, _analysis) = crate::parse2(
    quote!(),
    quote!(
        mod app {
            #[task(binds = UART0)]
            fn foo(_: foo::Context) {}
        }
    ),
    Settings::default(),
)
.unwrap();

the user it met with a cryptic Error("unexpected argument"). What is not
obvious is that parse_binds in the Settings struct must be set if binds
are to be parsed.
This commit adds an improved error. Resolves #46.

@AfoHT Done.


Automated upload of data when on a WiFi network.

Original report by Brian Calder (Bitbucket: brian_r_calder, ).

Although the normal condition for the loggers is to be deployed in an environment where “always on” internet is scarce, there are some use cases where this might exist (e.g., bigger ships, yachts, etc.). In those environments, it would be beneficial to automatically upload the data files to a network endpoint as they are completed and closed, mainly so that their contents could be checked for validity before too long of a delay.
For bonus points, this facility also needs to be controllable by the volunteer such that it could be turned off under certain circumstances (e.g., the volunteer doesn’t want their current position broadcast in real time).  This would obviously interact with other controls on logging (e.g., so that the system could continue to log, but just not broadcast the data until later, or might stop logging completely for some time).
The “reporting/not reporting” status should be visible somehow to give the volunteer positive feedback on status.
The reporting might also be mediated by environmental parameters (e.g., geo-fenced location to enable transmission, but only if the system is moving at under 2kts, etc.)

Original comment by Brian Calder (Bitbucket: brian_r_calder, ).

Main request resolved in firmware 1.5.0, commit 3cf09437, tag "firmware-1.5.0".  Auxiliary "bonus points" moved to a separate issue.

Original comment by Brian Miles (Bitbucket: [Brian Miles](https://bitbucket.org/Brian Miles), ).

If the data are to be updated to S3, we need to handle authentication and authorization, key distribution, etc.


better way to support winning icon on large icons (aka 900x300, 1800x600, etc)

i later to give some proof

just do this
function onCreatePost()
     setProperty('iconP2.scale', scale value)
end

just do this
function onCreatePost()
     setProperty('iconP2.scale', scale value)
end

no i mean i fix large icon doesn't support winning icon based on some code from my fork pe 1.0

no i mean i fix large icon doesn't support winning icon based on some code from my fork pe 1.0

yeah
just make the icon the og size where it supports the winning icon, then do that

^


⚠ Bofrost.de has degraded performance

In ee1664b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1732 ms

Resolved: Bofrost.de performance has improved in ee49fa4 after 27 minutes.


[Improvement]: Add TableFormat information to the return value of the Catalog listTable method.

Why are the changes needed?
This PR is part of this issue #1061 .
In order for a catalog to support multiple formats, we need to include TableFormat information in the return value of the catalog's listTable interface. The code changes related to the #1061  are very large. To reduce the burden of review, some code refactoring has been proposed as a separate PR.
Brief change log

Change the return type of listTable from List<TableIdentifier> to List<TableIDWithFormat>
Add format to ServerTableIdentifier
Remove format from table_runtime
Change the input parameter of ServerTableDescriptor from ServerTableIdentifier to TableIdentifier.

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before making a pull request

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

LGTM


timout command issue

Type:

[x] Bug
[ ] Enhancement
[ ] Feature Request
[ ] Question

zplug --version: 2.4.1
zsh --version: zsh 5.3.1 (x86_64-unknown-linux-gnu)
uname -a: Linux overvince 4.9.8-1-ARCH #1 SMP PREEMPT Mon Feb 6 12:59:40 CET 2017 x86_64 GNU/Linux

Hi,
To be honest I don't know how could it be, but activating ZPLUG seems to mess up commands begining with a timout delay before aborting. That is not a hard issue for my day to day use of zsh, but there may be some other problems behind it, I don't know.
BTW, thanks for your amazing application.
Vincent

Minimal .zshrc
The .zshrc file:
export ZPLUG_HOME=$HOME/.config/zplug
source $ZPLUG_HOME/init.zsh
zplug load

alias mkdir='mkdir -p -v'
alias rm='timout 3 rm -Iv --one-file-system'

Steps to reproduce
Tests on terminal command line:

With ZPLUG

overvince% mkdir test/test
mkdir: created directory 'test'
mkdir: created directory 'test/test'
overvince% rm -r test
rm: remove 1 argument recursively? y
overvince% y
zsh: command not found: y

Without ZPLUG (3 first lines commented on .zshrc)

overvince% mkdir test/test
mkdir: created directory 'test'
mkdir: created directory 'test/test'
overvince% rm -r test
rm: remove 1 argument recursively? y
removed directory 'test/test'
removed directory 'test'

alias rm='rm -Iv --one-file-system' works as expected (recursively), with or without ZPLUG.

Sorry for late and thanks for reporting. But I cant reproduce your problem.
[vagrant@localhost]~% mkdir test/test
mkdir: created directory `test'
mkdir: created directory `test/test'
[vagrant@localhost]~% rm -r test
rm: remove all arguments recursively? y
removed directory: `test/test'
removed directory: `test'
[vagrant@localhost]~%


[GLUTEN-1817][CH] Fix: Initialize global thread pool at backend initalization

What changes were proposed in this pull request?
Refer to https://github.com/ClickHouse/ClickHouse/pull/11668, let's initialize GlobalThreadPool at process startup to avoid race.
BackendInitializerUtil::init is called when Dirver and Worker  initialize Plugin which is at process startup
(Fixes: #1817)
How was this patch tested?
Uisng existed UTs

LGTM


Minimum SDK version - 31

When I pull quickie into my project I get the following error:
The minCompileSdk (31) specified in a
dependency's AAR metadata (META-INF/com/android/build/gradle/aar-metadata.properties)
is greater than this module's compileSdkVersion (android-28).
The intro page indicates a minimum SDK of 21+
This looks to be due to the CameraView component.
I'm going to look to move to 31 - but I thought I'd mention it here.
Many thanks for your work so far.

Hi, thanks for reaching out. My readme is actually right. The minSDK version is 21, so devices back to Android 5.0 can run this library.
The issue that you are facing is because you don't use the latest compileSDK version in your project. This unfortunately prevents you from using all the latest AndroidX libraries like appcompat 1.4, lifecylce 2.4 and which this library depends on. There is nothing I could do (except downgrading dependencies which doesn't makes sense).
You should consider updating you compileSDK version or stick to an older version of this library for a little longer. :slightly_smiling_face:

OK that makes sense - thanks.


[BUG]: rio.Switcher transition_time of 0.0 breaks layout

Describe the bug
When setting the transition_time of rio.Switcher to 0.0, it causes the layout to break. This issue manifests as incorrect positioning or unexpected behavior in the layout of the components managed by rio.Switcher.
Expected Behavior
Setting the transition_time to 0.0 should result in an instantaneous switch between components without affecting the layout. The layout should remain consistent and correct, just as it would with a non-zero transition time.
Steps to Reproduce

Initialize rio.Switcher with a component or None.
Set the transition_time=0.0 of rio.Switcher.
Trigger a switch between the components.
Observe the layout behavior and note any issues.

Screenshots/Videos
https://github.com/rio-labs/rio/assets/41641225/7b2bb4cb-cd61-4153-817d-6694e9508433
Operating System
Windows
What browsers are you seeing the problem on?
Chrome, Safari, Edge
Browser version
No response
What device are you using?
Desktop
Additional context
No response

A quickfix with ~max(transition_time, 0.01) was added, but it doesn't solve the issue 😄


Gizmos aren't Z-sorted properly with transparent surfaces

Godot 3.0 alpha1 official
Windows 10 64 bits
nVidia geforce 940M
It depends on the viewing angle, gizmos get randomly drawn behind or in front of transparent surfaces. In the screenshot, the blue plane is slightly transparent:

Happened similarly on 2.2, AFAIR.

seen this problem, needs to be fixed, thanks for opening the issue!
On Fri, Jul 28, 2017 at 9:16 AM, Bojidar Marinov<EMAIL_ADDRESS>wrote:

Happened similarly on 2.2, AFAIR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/9935#issuecomment-318636904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z22QsdAGYMNcdMyodDcLD6QBl0tYlks5sSdEJgaJpZM4Ol-Dn
.

actually judging by your bug, i have no idea what's going on. If you can make a very simple test scene (in case it was not fixed already) i will give it a check

Also I am not sure if this is the right place to mention this, but the gizmos are also hidden by fog which is inconvenient sometimes.

@Zylann This should be properly working now (just fixed it), but please reopen if you see something different
@HummusSamurai This won't change. You can disable environment if you feel gizmos are inaccessible in the viewport menu.

Looks like it's fixed:

Sorry, I reopened the initial scene, the problem was never fixed...
I shot a video to better explain what happens:
https://www.youtube.com/watch?v=16nTnP02JIw&feature=youtu.be
My graphics card shown in the log:
GL 3.3 contextOpenGL ES 3.0 Renderer: GeForce 940M/PCIe/SSE2

Did you set the floor material as "on top"?
On Aug 29, 2017 6:29 PM, "Rémi Verschelde"<EMAIL_ADDRESS>wrote:

Reopened #9935 https://github.com/godotengine/godot/issues/9935.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/9935#event-1226522115, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AF-Z2-jTn0aysINxsWi17609ZD6Kqjv4ks5sdIKbgaJpZM4Ol-Dn
.

No, it's just a transparent quad

also if you can make a simple reproduction demo, that would be great

Well, I tried^^" but only repro in this project. I can try to simplify it

GizmoDrawOrder.zip
Simpler project

Sorry to popup again, but... it's better now, however some rogue gizmos still don't sort consistently:
(light arrow and cube bounding box)

looking at the commit.
there is probably just this missing for the light:
 -		gizmo_hl->set_flag(SpatialMaterial::FLAG_ONTOP, true);
 +		gizmo_hl->set_on_top_of_alpha();

Here is a MeshInstance with transparency flag. Node specific gizmos don't appear on top of it unless the render_priority of the material is -128. As seen in the picture, transformation gizmos render just fine.

Here is a MeshInstance that has a screen-reading shader. Again, can't see gizmos regardless of the depth.
shader_type spatial;
void fragment()
{
	ALBEDO = texture(SCREEN_TEXTURE, SCREEN_UV).rgb;
}

Sorry for digging up this old chestnut again, but the bug is still around in 4.0.2-stable. I also had it in 3.x. It's quite easily reproducible: just set transparency of a StandardMaterial3D to another value than TRANSPARENCY_DISABLED and observe gizmos disappearing. This is an example project featuring a CollisionShape3D gizmo disappearing in front of a box-shaped MeshInstance3D: gizmos_zordering_alpha.zip.

This cannot be resolved as per https://docs.godotengine.org/en/latest/tutorials/3d/3d_rendering_limitations.html#transparency-sorting.

Testing locally, I actually can't reproduce the issue anyway. The gizmo appears to sort fine

Testing locally, I actually can't reproduce the issue anyway. The gizmo appears to sort fine

In your second screenshot the CollisionShape3D gizmo isn't visible although being in front of the cube. I should have chosen a better albedo in my test project. Those red-blue-green "spatial manipulation gizmos" never disappeared in my case.
The vanishing gizmos are actually much more noticeable when environmental meshes (like a huge floor) have transparency enabled because then all the gizmos disappear. This way it's very hard to inspect and manipulate e.g. collision shapes or lights.

This cannot be resolved as per the 3D rendering limitations around transparency sorting.
Some workarounds have been proposed such as godotengine/godot-proposals#2138, but they come with downsides on their own. Order-independent transparency isn't something we can rely as a baseline feature for rendering the 3D editor, as it's very slow.

Thanks for the information. Although I don't fully understand why the red/green/blue manipulator gizmos are ordered correctly.

I see. So what you want is for all the gizmos to always appear on top? Right now they are sorted in the transparent pipeline. So if they are behind an object, they won't be visible. Is it correct that you would like them to be visible all the time, regardless of whether they are in front or behind an object?

I would expect them to be visible if they are in front of a geometry. I.e. if nothing is between the object/gizmo and the camera.

In my screenshot the physics shape is behind the cube and it is correctly sorted so only the portion sticking up above the cube is rendered.
So to clarify what you are reporting:

You are saying that gizmos (except the translate/scale/rotate) are always appearing behind transparent objects (even when they should be in rendered in front
You aren't suggesting that gizmos be always on top

Is that correct now?

You can already choose select node gizmos to appear always on top using the View > Gizmos menu and clicking on an gizmo until it has a "half open" eye icon. This means it uses the X-ray visibility mode. This will effectively resolve transparency sorting issues by not performing any sorting for the transparent object in the first place.

On my machine the gizmo is disappearing, regardless of which side of the cube it's on:

@clayjohn: ad 1) yes, ad 2) no, I suggest them being visible if they are not hidden by geometries and (as I now learned) being visible even when hidden when the "half open" eye is active for X-ray visibility mode.
@Calinou: the X-ray visibility mode of gizmos doesn't change the behavior in my gif


Error: destEnvironment is not defined

Hi there I think I found a bug in your useful tool.
https://github.com/jungvonmatt/contentful-migrations/blob/ba673e3c8b0a8a169dc0b7720606bb96de3a8c95/lib/content.js#L64
I think this is supposed to be destEnvironmentId ?

fixed in v5.5.1

Thanks again @DalkMania
If you find any other bugs feel free open more issues :D


Let UART write return some bytes were written

embedded_io::Write::write(buf) should not block until the full buffer was written. Instead, it should only block until at least one byte was written.
(See https://github.com/rp-rs/rp-hal/pull/837#issuecomment-2305371353)

Fixes https://github.com/rp-rs/rp-hal/issues/840


LimitInternalRecursion

Hi,
I would to try locally cors-anywhere for tests.
After install, the demo page work (http://myip:8080/demo.html), but each test with external URL fail with an apache error below, any idea about the root cause?
Request exceeded the limit of 10 internal redirects due to probable configuration error. Use 'LimitInternalRecursion' to increase the limit if necessary. Use 'LogLevel debug' to get a backtrace., referer: http://myip:8080/google.com
Same error with another URL, example http://myip:8080/http://www.africau.edu/images/default/sample.pdf
I tried to increase the LimitInternalRecursion value, but same result.
Thanks

This looks like an issue with your Apache configuration, independent of CORS Anywhere.


JBPM-4979 - Generated PDF is unreadable - update to itext lib version

same test as in https://github.com/droolsjbpm/jbpm-designer/commit/6e035d3a0ca5f6a97295ed3e335c47afc30c6e50
applies here.
This change is based on update to the itext version in the parent poms and update to reflect the api changes

depends on https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/pull/174 to be done first

Merged by https://github.com/droolsjbpm/jbpm-designer/commit/b921b6a6280856fa20f65c9deafacb10a6170a67


MKL: Update mkl docs

Decided to make bigger changes.

@tfboyd Please take a look at the proposed changes to the Performance Guide once PR is ready.


test: protect tests from bad random height

Description
This PR ensures that the random height is always greater than 1000.

@b00f  please fix the deadlock and race condition


Goldthorp/php city name

Does this include the UI change that displays the city in the property list?

No, I figured I'd keep this branch just for the php change and do the UI change after you've updated the live api. I thought it made sense to do the organizing the list by city thing as a separate transaction.


define exterior product (wedge)

Change-Id: I421dccbb1bdceb4bd57ff4714f662dee6540ec2b

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

:exclamation: No coverage uploaded for pull request base (I621e3d44d7e33b3809d6a66dfdafc8dc84cfbd63@2764838). Click here to learn what that means.

:exclamation: Current head e1e40b2 differs from pull request most recent head 3a0a640. Consider uploading reports for the commit 3a0a640 to get more accurate results

Additional details and impacted files
@@                             Coverage Diff                              @@
##             I621e3d44d7e33b3809d6a66dfdafc8dc84cfbd63      #35   +/-   ##
============================================================================
  Coverage                                             ?   97.40%
============================================================================
  Files                                                ?       13
  Lines                                                ?      154
  Branches                                             ?        4
============================================================================
  Hits                                                 ?      150
  Misses                                               ?        2
  Partials                                             ?        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Is this a correct comparison?

https://github.com/trinamic/TMC-API/blob/17d9adbc8cc3e3f2c7615ed194ad2518aa76fdae/tmc/ic/TMC429/TMC429.c#L334
if the first comparison is true the second one should be true as well so that one is never evaluated. I see the same comparison in the datasheet. Since pmul will be written to a 7-bit number shouldn't it be like so?:
if((0 >= pmul) && (pmul <= 127))

Hi,
The comparison in the code checks whether pmul is between 0 and 127 (both inclusive).
In the code, the check is correct. If the number is less than 0, the first condition fails (and the second one is not evaluated). If the number is greater than 127, the first condition succeeds, but then the second one fails.
Do note that the (0 <= pmul) comparison has the variable on the right hand side. It is equivalent to (pmul >= 0). Maybe this caused the confusion?

Do note that the (0 <= pmul) comparison has the variable on the right hand side
You are right. I missed that. Not very conventional to put the constant on the left, although I understand it is optically closer to a <= x <= b. I always have all warnings enabled so I never use assignment as comparison.


windows hardware developer->hardware dashboard API->update a shipping label.

[Enter feedback here]
i have a useing problem in windows hardware developer->hardware dashboard API->update a shipping label.
API URL:
https://manage.devcenter.microsoft.com/v2.0/my/hardware/products/{productID}/submissions/{submissionId}/shippingLabels/{shippingLabelId}
i want to konw how to write the request body.  i have already published a shipping label an its already finish. now i want to delete restrictedToAudiences info to publish it again. the following info is the request body:
{
"targetingInfo": {
"restrictedToAudiences": [
],
"businessJustification": "Business justification for updating shipping label"
}
}
but i got something wrong:
{
"error": {
"code": "invalidInput",
"message": "Validation Failed.",
"validationErrors": [
{
"target": "targetingInfo.chids",
"message": "At least one property is required for update."
},
{
"target": "targetingInfo.hardwareIds",
"message": "At least one property is required for update."
},
{
"target": "targetingInfo.restrictedToAudiences",
"message": "At least one property is required for update."
}
]
}
}
then i added some info about chid and hardwareId, but i still got this problem.

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

ID: 68b9e39f-a269-82bb-d44a-162ed96fab57
Version Independent ID: 03adbc66-9e14-11b0-407b-a2d4b09ee7b5
Content: Update a shipping label - Windows drivers
Content Source: windows-driver-docs-pr/dashboard/update-a-shipping-label.md
Product: windows-hardware
Technology: dashboard
GitHub Login: @anaharris-ms
Microsoft Alias: anaharris

@HaraldWang I am looking into this for you now. In the meantime, take a look at https://github.com/MicrosoftDocs/windows-driver-docs/issues/1438.  It's an issue from 2019 that seems to suggest that you should use Targeting instead of TargetingInfo. I'm not certain that will solve your problem, and so I will continue to look for more information. In the meantime, let me know if that solves your problem.

the solution above i will try later and i will give you the reply, thanks.

Closing out due to inactivity.  Please reopen if still needed.  Thanks for reaching out!

@anaharris-ms hi，sorry for long time with no reply, our project for some reason was delay. I tried the method mentioned in issue #1438.  the request and response like the picture below:

I just want to delete the value in field [restrictedToAudiences], and other infos remain unchanged.

Hi @HaraldWang ,
Sorry to hear you are still having trouble.

Anaharris
it looks like this was mistakenly left under MY COMPUTERS FEEDBACK SUBMISSION. I am the owning user. I think I'm listed as Sarahsmom. Those shipping labels....my shipping labels. What is the purpose of this complete life altering infiltration of any and all devices that I own. I,being a single Mother with no family except my nine year old. Who is just trying to life my life and provide for my child which you all at github and Microsoft make impossible. Is it just because you have been doing it for so long that the projects have more value than just   trial and error info?  There is a person behind the ip address. a face that goes with the limited admin rights user. What is this doing to my nine year old in the long game? this has been going on since before she was born. I can't even go to my kids parent portal to her school.  Just know that i write down every name I can. Yours just got added.


propagate laser pulse back and forth through the same crystal

This can be done with constant n2 (both longitudinally and radially).
Drift the pulse 50 cm to a mirror, then 50 cm back to the crystal.
What's new here is traversing the crystal slices in the opposite order.
Issue #185 needs to be completed first.
Completing this issue will like require modification of the Crystal and/or CrystalSlice classes.

This is now being tracked at https://github.com/radiasoft/rslaser/issues/10


Create a test for SystemJS support

There are a few issues around SystemJS support. A test is needed to confirm that SystemJS is working as expected.

Yes, in my project, the version 5.5.2 don't fix the path bug of operators.js

@jasonaden is this done? Does this still need done?

@benlesh I think this was done here. I'll close.


Need to reset pipeline when calculating features of training samples.

Otherwise the data at the end of one sample affects the features at the start of the next. Example data (w/ the accelerometer poses pipeline).
TrainingData.grt.txt

Done.
https://github.com/damellis/sensors/commit/dcfb17742a7dea76929e7fad18304d9737ad1b72


Remove status check from compliance checker

@pgmpablo157321 can we drop this check only for v4.1 since it was indeed a check that existed before so we can leave all previous rounds as is.


Search Page

The search page in the redesign should be more user friendly.

WAY too much blank space that is not being leveraged. The user has to scroll reach something usable.
The images are really small and the content is cut off when there is still a lot of space available.
There are 3 columns where 90% of the time the two from the right will be empty or near empty.

Suggestions

[ ] (css) The content should be primary, showing at the top before channels and downloaded content.
[ ] (css) There should be 1 column so that information can be fully on display, photo, title and substring of description.
[ ] (css) On high resolution screens we can probably have 2 columns which with a flex layout should be doable with a basis.
[ ] (js) Pagination is a must have for the redesign imo. Now that most searches return > 100 results the users will miss out on discovering a lot of content.

I made a few of the changes noted above. 1,2, and 3. Also added the first 3 lines of the claim description if available. #1522 is the PR.

closing 4th part of the issue for https://github.com/lbryio/lbry-app/issues/1261
Talked with @seanyesmunt he said this is not happening for the redesign release.


/usr/bin/ld: cannot find -lprofiler

tried to build and facing below error
[ 52%] Linking C executable picoquicdemo
/usr/bin/ld: cannot find -lprofiler: No such file or directory
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/picoquicdemo.dir/build.make:124: picoquicdemo] Error 1
make[1]: *** [CMakeFiles/Makefile2:147: CMakeFiles/picoquicdemo.dir/all] Error 2
make: *** [Makefile:91: all] Error 2

install libgoogle-perftools-dev


Year view support

I was wondering if it is possible to build a "year view" with this library.
I saw that you mentioned somewhere that it was being worked on back in sept 2016.
I was wondering if it is available now since that is exactly what I need

its not available yet.
I was thinking that i would have to create a new layout for it.
I do not want to clutter the existing framework.
I will do this when im free. But right now, I am very tied-up with multiple projects.
But i'll give this some thought soon

closing. Moving this to projects section -> https://github.com/patchthecode/JTAppleCalendar/projects

Enhancement coming.

Implemented.
Testing on branch -> YearView

merged on master branch

Hi @patchthecode , how we can use the year view? i couldn't find any clue on document!
Thanks.

It is not documented yet.
Never got the time to do it.
But it is running live in the sample project attached to github.
Can you try it and take a look at how to implement it?
It is just as simple as the regular calendar

If you have more questions, then ask

Hi @patchthecode, when I installed  library by pods it will be installed version 7.0.7! but YearView available in version 8, and with iOS 11.2 I faced minimum version error! what is the minimum version of iOS to use version 8?
You didn't put it on master branch?


How to convert an old application to Vert.x ? with mod-jersey / mod-cassandra

Hello,
I'm rewriting an old application that uses CXF and Cassandra. There are 3 layers working with Spring (core) : Web (cxf), service (with business rules in Java) and DAO (Cassandra).
I'd like to replace all this with Vert.x, using vertx-jersey for the Web layer and vertx-cassandra for the DAO layer.
Do you have recommandations / best practices ? Should I deployer verticles to replace each layer, for example ?
Thx,
Th.

If you can tolerate a pre-release version of vert.x, use vert.x 3 (currently 3.0.0-milestone3).  The dependency management is so much simpler than the module system in vert.x 2.
Use something to help with the async nature of vert.x.  I'm clearly biased towards vertx-when which is promise based, but if you prefer RxJava that's the most commonly used option in the vert.x community.  The important thing is to have something to help avoid callback hell and run on the vert.x context.

I have seen Spring mentioned in the google group, I personally avoid it and use HK2 or Guice for DI.
The question of how many verticles you should have is a difficult one to answer.  There is a cost and complexity to serializing your objects and sending them over the event bus between verticles.  Depending on the size of the application and how modular you need/want to be, I would probably start with one verticle as you start working with vert.x.  For a small project, I use a project structure like this:
parent
|_ core - contains POJO model objects, business logic, and interfaces
|_ cassandra - contains implementations of data access persistors that are Cassandra specific.  Depends on core.
|_ app - contains the main Verticle, Jersey resources, filters, etc.  This is the http REST layer and depends on core and cassandra.
Still keep each layer separated in different jars, but not necessarily separate Verticles per layer.  It's hard to generalize too much, but that's where I would start.
I tend to use the maven service factory to run the application, but you can also use the maven-shade plugin and run as a fat jar.
Hope this helps.

Not sure if you have seen vertx-cassandra now has vert.x helpers for the datastax mappers in the current SNAPSHOT build.
https://github.com/englishtown/vertx-cassandra/blob/develop/vertx-cassandra-mapping/README.md

Hello,
Thx for this great answer.
I'll start with Vert.x 3... Already in my pom ;-)
I am not sure to understand everything.The "core" maven module will contain my beans. The "cassandra" module will contain my DAO, and vertx-cassandra ? And the "app" module will contain the main class, and vertx-jersey.
By "main class" I mean a class with a "main()" method that deploys my verticles... I just copied what was proposed by vertx-jersey-webapp : https://github.com/NRaoV/vertx-jersey-webapp
public class MainVertx {

    private static final Vertx vertx = Vertx.vertx();

    public static void main(String[] args) {
        Context context = vertx.getOrCreateContext();
        JsonObject config = context.config();
        Properties props = CmdLineParser.getInstance().getCmdLineArgs(args);
        config.mergeIn(ConfigManager.loadConfig(props.getProperty(ServiceConstants.CONFIG_SOURCE)));
        //config.mergeIn(ConfigManager.loadConfig("my-ws-vertx/src/main/resources/config.json"));
        JsonObject jerseyConfig = config.getJsonObject(ServiceConstants.CONFIG_JERSEY);
        DeploymentOptions options = new DeploymentOptions();
        // options.setIsolationGroup("A");
        options.setConfig(jerseyConfig);

        // Specifying number of verticle instances, a rendre parametrable
        options.setInstances(16);

        // Jersey
        vertx.deployVerticle("java-hk2:" + JerseyVerticle.class.getCanonicalName(), options);

        // Service
        vertx.deployVerticle("com.mycorp.vertx.MonServiceVerticle");
    }
}

Just for now, I only have the web service part. I was thinking that the web service part, working with vertx-jersey, does not need so much CPU power. And the vertx-cassandra too... Am I wrong ? Therefore they could be on separated verticles...
@Path("/mywebservice")
@Produces(MediaType.APPLICATION_JSON)
@Consumes(MediaType.APPLICATION_JSON)
public class MyWebServiceVertx {

    @POST
    @Path("availability")
    @Produces(MediaType.APPLICATION_JSON)
    @Consumes(MediaType.APPLICATION_JSON)
    public AvaibilityResult getAvailability(AvailabilityCriteria availabilityCriteria) {
        AvaibilityResult result = foo(availabilityCriteria);
        return result;
    }
}

Here the class "MyWebServiceVertx" just receives the web service call, transforms the json to an object and then call the "foo()" method. In the "foo()" method, or instead of the "foo()" method call, I was thinking about calling other verticles with mod-rxvertx or vertx-when. But you suggest to perform the computation here directly?
public class MonServiceVerticle extends AbstractVerticle {

    @Override
    public void start() throws Exception {
        // ...
    }
}

By the way, do you have a full "pet shop" example, using both vert-jersey and vert-cassandra, somewhere?
Thx,
Th.

The example from NRaoV is running as an embedded vert.x instance.  I would recommend running as a pure vert.x application using either a fat jar or the vertx-maven-service-factory.  I personally use vertx-maven-service-factory for my applications.  My start script looks something like this:
vertx run service:com.englishtown.vertx.examples:vertx-example:1.0-SNAPSHOT -conf config.json -instances 4

If you run the vertx bash script without any arguments it will print the help.
The examples in vertx-jersey use fat jars with the maven-shade plugin.  See the README.md.  This means you can run them like this:
java -jar target/{artifactId}-{version}-fat.jar -conf src/test/resources/config.json

I don't have a full "pet shop" example, all of these packages have grown from internal needs.  I'll try to put together an example demonstrating the full stack I typically use.

Thx


Ideas for future SYSKA topics

From @TomKellyGenetics on September 1, 2015 23:18
Hi team, I thought this would be a good space to start planning ideas for SYSKA topics to gauge interest in various things we could do... and find volunteers for topics we'd like to see. Many of these packages I don't use very much but would like to learn, maybe giving a presentation would be a good push to try them out? Of course many of you are likely tobe experienced with some of these and likely more equipped to present it if you'd like to. While you're welcome to come up with your own topics for your presentations, I think it would be good to plan some in advance so people know what they're coming to, especially if we're eventually reaching out into the wider research community.
Topics I could do:
Data handling in R: data.table, bigmem, data-wrangling (reshape, tidyr, custom codes)
Statistics in R: linear models (although many others here could do a better job)
Installing and managing packages in R (CRAN, Bioconductor, GitHub, SourceForge)
Mathematica / Wolfram Alpha
Topics I would like to see:
R: ggplot(2), dplyr, and other Hadleyverse packages
R projects (with Git/Github)
LaTeX intro/recap (Authorea?)
Using with databases with R
Using Unix tools to manipulate files (awk, sed, grep, cat, head, tail, etc...)
Getting help: using Linux/R forums effectively
Handling sequence data
Topics I'd recommend for new(ish) students:
Accessing Servers in the terminal: ssh (alias, keys), scp, rsync - backups and running code
HPC in R or Python (local parallel, departmental servers, or NeSI)
NeSI: submitting and managing jobs with slurm
Copied from original issue: smilefreak/studyGroup#7

From @nickb- on September 1, 2015 23:29
Once my study life calms down, I'd be very happy to do something on R + databases. In particular, SQL Server 2016 will come with R built-in, providing a great platform for in-database analytics. Would be able to get my hands on an early release of SQL Server 2016 and give it a whirl.
Nick

From @smilefreak on September 2, 2015 2:27
Also, I will make a label for discussion threads so that we know it is a discussion.

From @smilefreak on September 2, 2015 2:27
Hi Tom, Nick
All those suggestions sound great! I think aswell as that we should try some interactive visualisation things, people really like those, and some papers now actually provide you with a way to publish interactive plots in the online version of a paper.
For R Shiny is the obvious choice.
But lot's more exist for instance in python we have http://stanford.edu/~mwaskom/software/seaborn/
Could we get a cancer genetics heatmap working that package, and once people are past the begineer stage in python it would be good to introduce them to the visualisation. Nothing will beat R in terms of
proper statistics but is important to keep you options open, as R is not the best tool for ever job.
Obviously expanding into a larger group we should be thinking of things like teach some C ( everyone should know some basic C), GPU programming, matlab, and nltk (more humanities here)

From @methylnick on September 7, 2015 20:27
On the thread of visualisation you should follow Martin Kryswinski's Blog (maker of Circos), if you haven't come across it already, the visualisation principles he preaches are very interesting and sound.  http://mkweb.bcgsc.ca/

From @smilefreak on September 2, 2015 8:37
Circos, yea forgot about that, I used it a little bit in my apple work, maybe pencil that in for a later date.
Will be good to learn it in enough detail to teach it also, that will be my benefit.

Yep, there should be plenty of interest in interactive graphics with R: shiny, ggvis, and d3.heatmap are all possible topics. Would take a bit more preparation though so maybe better to save these for a wider audience?
Could see if I can remember MATLAB but probably better to collaborate with Physics on that one. I'd also be interested in Python or MATLAB visualisations for networks if we can get someone to run through that. A CirCos demo would be cool too.

From @murraycadzow on October 8, 2015 20:36
Thinking of be able to encourage people whom might feel they don't have enough programming background to talk about software these are some ideas that could be accessible:
How people use resources that already exist such as:
genome browsers like ensembl or UCSC
1000 genomes
encode
various cancer datasets
plant datasets

From @methylnick on September 10, 2015 3:52
Kind of a SYSKA I would propose is the regulatory constraints around genomic testing. Happy to provide a high level overview on experience from Melbourne and a little with my organisation.  I can send a discussion paper along with it.


Add the ability to re-use existing types

Problem description
When designing reusable templates and prepare them with default values I often need to create user-defined types that represent the underlying already existing types. There is -as far as I know- no option to reuse the existing types.
For example: we want to use predefined sku, like t-shirt sizing, to set for a serverfarm, like:
File: skuselector.bicep
type skuType = { key: string, sku: { name:string, tier:string}}

@export()
type aspSku = 'free'|'nonprd'|'prd'

@export()
func selectSku(sku aspSku) skuType => first(filter(
  [
    { key: 'free',    sku: { name: 'F1', tier: 'Free', capacity: 1}}
    { key: 'nonprd',  sku: { name: 'B1', tier: 'Basic', capacity: 1}}
    { key: 'prd',     sku: { name: 'S1', tier: 'Standard', capacity: 1} }
  ],
  s => s.key == sku))!

File: serverfarm.bicep
import { selectSku, aspSku } from 'skuselector.bicep'

param location string
param appServicePlanName string
param sku aspSku

var selectedSku = selectSku(sku)

resource appServicePlan 'Microsoft.Web/serverfarms@2022-09-01' = {
  name: appServicePlanName
  location: location
  sku: selectedSku.sku
}

File: main.bicep
module appServicePlan 'serverfarm.bicep' = {
  name: 'myasp'
  params: {
    location: location
    appServicePlanName: myAppServicePlanName
    sku: 'nonprd'
  }
}

So in this simplified example we set the sku in the main.bicep to nonprd, which uses the pre-configured configuration and set the sku to name B1, tier Basic and a capacity of 1.
Desired solution, possible solution
It would be nice if we can reference existing types so we can replace this part:
type skuType = { key: string, sku: { name:string, tier:string}}
And replace it with something like:
type skuType = { key: string, sku: Microsoft.Web/serverfarms:types:SkuDescription@2022-09-01}
When we are able to reuse existing types we can also prepare parts of a resource like autoscale settings. In our environment we have a number of similar API's that we want to monitor likewise, but also want the flexibility to configure them t-shirt-size-wise, depending on the load or how critical the application is.

Duplicate of #9229


Fix ETag invalidation for posts#owed when reading a post

@posts isn't updated when users read posts, so we need to include the unread_views entry for users' browsers to correctly update the page's arrow colors when they read posts.

this doesn't solve it entirely - if you read a thread that has a lower updated_at than the latest one you've read then it's still an issue


Missing textures on certain items

In build 361, the Dimensional Transceiver, Enchanter, Killer Joe, and Zombie Generator are missing textures.

Fixed in build 363.


SSL host verification

This consolidates contributions from #69 and https://github.com/glaszig/logstash-logger/tree/ssl-context

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@058dd28). Click here to learn what that means.
The diff coverage is 94.36%.

@@            Coverage Diff            @@
##             master     #114   +/-   ##
=========================================
  Coverage          ?   91.67%
=========================================
  Files             ?       61
  Lines             ?     1537
  Branches          ?        0
=========================================
  Hits              ?     1409
  Misses            ?      128
  Partials          ?        0

Impacted Files
Coverage Δ

spec/device/tcp_spec.rb
100% <100%> (ø)

lib/logstash-logger/device/tcp.rb
88.23% <85.71%> (ø)

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 058dd28...2a41348. Read the comment docs.

@dwbutler Do you have any estimate for when this would merged?

Sorry been slammed lately. This feature is at the top of my priority list so it will definitely be the next thing to go out. Just need to find some time to go over this one more time and make any documentation changes (if needed).

Codecov Report

Merging #114 into master will increase coverage by 1.1%.
The diff coverage is 97.59%.

@@            Coverage Diff            @@
##           master     #114     +/-   ##
=========================================
+ Coverage   91.49%   92.59%   +1.1%
=========================================
  Files          61       61
  Lines        1469     1526     +57
=========================================
+ Hits         1344     1413     +69
+ Misses        125      113     -12

Impacted Files
Coverage Δ

spec/device/tcp_spec.rb
100% <100%> (ø)
:arrow_up:

lib/logstash-logger/device/tcp.rb
95.91% <94.44%> (-0.09%)
:arrow_down:

lib/logstash-logger/buffer.rb
71.42% <0%> (+1.53%)
:arrow_up:

lib/logstash-logger/device/base.rb
77.5% <0%> (+1.89%)
:arrow_up:

lib/logstash-logger/device/kinesis.rb
87.17% <0%> (+2.17%)
:arrow_up:

lib/logstash-logger/device/connectable.rb
83.07% <0%> (+4.81%)
:arrow_up:

lib/logstash-logger/silenced_logging.rb
97.14% <0%> (+5.25%)
:arrow_up:

lib/logstash-logger/formatter/cee_syslog.rb
100% <0%> (+7.69%)
:arrow_up:

lib/logstash-logger/configuration.rb
100% <0%> (+10.52%)
: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 058dd28...9f6c907. Read the comment docs.

Fix released in 0.23.0.


feat: enable max queue size for chromium and libreoffice

See #463

Codecov Report
Attention: 16 lines in your changes are missing coverage. Please review.

Comparison is base (975a9f5) 93.45% compared to head (2d71cb2) 93.03%.

:exclamation: Current head 2d71cb2 differs from pull request most recent head 0209696.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #804      +/-   ##
==========================================
- Coverage   93.45%   93.03%   -0.42%
==========================================
  Files          40       40
  Lines        3468     3490      +22
==========================================
+ Hits         3241     3247       +6
- Misses        143      157      +14
- Partials       84       86       +2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks @timgrohmann! Great PR!
I'll add the missing tests.

Closes #463.


Maybe we need to illustrate the meanings of 10004 and 10005 somewhere.

Maybe we need to illustrate the meanings of 10004 and 10005 somewhere.
Originally posted by @gaocegege in https://github.com/caicloud/temp-model-registry/pull/7

/cc @simon-cj


add possibility to execute concrete5 cli with shorter composer command

Old command: ./concrete5/bin/concrete5 list
New possibility: composer ccm list or composer ccm c5:info
I think ccm for the shortening of './concrete/bin/concrete5' is better than something like "c5". Otherwise, the command would look like this, which looks odd.
composer c5 c5:info

What about using the c5 command available at https://github.com/concrete5/cli ?
That way, you'll simply need to run c5 list or c5 c5:info for example...

Yes that would work to. It's even shorter! The thing is, it's not versioned by git.
I prefer to have all my project related commands versioned in course control, and composer scripts give me a convenient way of doing so.
I guess at the end it's a matter of personal taste.

The solution does't work  in all cases as expected. Short commands work fine. On long commands like c5 orm:schema-tool:update --update-sql the last argument is stripped.
So I close this in favor of the solution @mlocati pointed out.


AI-KG use case

I added the use case for the Artificial intelligence Knowledge Graph (AI-KG, http://w3id.org/aikg) by The Open University.

nice!! thanks


Platform-independent GitHub CI caching for local Maven repository

Caching the local Maven repository helps to save

time by improving CI build performance,
bandwidth by downloading fewer Maven artifacts.

As Maven artifacts - mostly JAR and POM files - are platform-independent, I think it would make sense to also cache them using a platform-independent cache key. I started analysing how the GitHub cache action works and found out that it yields different cache keys on Windows compared to the identical ones on Linux and MacOS. Due to the use of ${{ hashFiles('**/pom.xml') }}, the logical explanation for this is that the hash values for the POMs, being text files, are different due to the fact that on Windows they use CR+LF instead of LF line endings. See also:

https://github.com/actions/checkout/issues/135#issuecomment-889845025
https://github.com/actions/cache/issues/614
https://github.com/actions/runner/issues/498

We want to avoid forcing all developers cloning the repository to have git checkout use Unix mode for text files by means of a .gitattribute file. For now, a temporary workaround only affecting the GitHub workflow would be to add build step as suggested in https://github.com/actions/checkout/issues/135#issuecomment-602171132:
    steps:
      - name: Set git to use LF
        run: |
          git config --global core.autocrlf false
          git config --global core.eol lf

      - uses: actions/checkout@v2

Actually, on my local box I simply use core.autocrlf=input and no core.eol setting and seem to be fine. Either of the two variants would probably fix the cache key. Maybe when automatically creating commits (e.g. committing documentation like the Maven site automatically in the future or creating releases, bumping and tagging artifact versions), one of those options would be superior to the other.
The workaround should then be removed again in the future, as soon as either of the checkout, cache or runner actions provide a means to get platform-independent checkouts, cache keys or hash values. In either of the three layers, solutions would be possible.

After so many months and after all 3 issues I raised or commented on were utterly ignored or rejected, I think that any improvements in either of the actions is simply never going to happen, because nobody cares about wasting GitHub resources for unnecessry multiple caches. If every user had to pay, probably someone would be interested. Suggesting to use .gitattributes does not solve the problem, it is just a workaround bought et the expense of all developers cloning the repository, because they would all be affected by a default setting which is only meant to be applied for GitHub actions.
So, as sad as it is, my ugly workaround probably is here to stay. Sometimes, I cannot understand how jaded developers can be, simply ignoring real-world use cases, making silly suggestions, just in order to avoid 30 minutes of work for implementing and testing something which would hugely benefit cache users (speed!) and GirtHub (disk space!) alike.


cannot add custom node to cluster

Describe your issue here
centos 7.4 64bit
rancher 2.0.0-beta1
docker 1.12.6
Kubernetes v1.9.5-rancer1-1
network provider:Flannel
when add a custtom node to cluster,meet these error:
[controlPlane] Failed to bring up Control Plane: Failed to verify healthcheck: Service [kube-apiserver] is not healthy on host [<IP_ADDRESS>]. Response code: [403], response body: {"kind":"Status","apiVersion":"v1","metadata":{},"status":"Failure","message":"forbidden: User "system:anonymous" cannot get path "/healthz"","reason":"Forbidden","details":{},"code":403}

Useful
Info

Versions
Rancher v2.0.0-beta1 UI: v2.0.33

Access
local admin

Route
authenticated.cluster.index

2018-04-05 08:45:15.088951 N | etcdmain: the server is already initialized as member before, starting as etcd member...
2018-04-05 08:45:15.185070 I | etcdserver: recovered store from snapshot at index 50005
2018-04-05 08:45:15.429803 I | etcdserver: restarting member 6b174bdc9a972d0d in cluster f51adadd49b797b0 at commit index 57543

Those indicate that the host was not created from scratch/cleaned before adding it. Can you clean /var/lib/etcd on the host and try again?

I clean /var/lib/etcd, and reinstall rancher cluster. But the error is same:
Apr  6 00:30:23 manage01 journal: E0405 16:30:23.593697       1 reflector.go:205] k8s.io/kubernetes/pkg/client/informers/informers_generated/internalversion/factory.go:85: Failed to list *core.ResourceQuota: the server was unable to return a response in the time allotted, but may still be processing the request (get resourcequotas)
Apr  6 00:30:23 manage01 journal: E0405 16:30:23.593769       1 reflector.go:205] k8s.io/kubernetes/pkg/client/informers/informers_generated/internalversion/factory.go:85: Failed to list *core.ServiceAccount: the server was unable to return a response in the time allotted, but may still be processing the request (get serviceaccounts)
Apr  6 00:30:23 manage01 journal: E0405 16:30:23.593836       1 reflector.go:205] k8s.io/kubernetes/pkg/client/informers/informers_generated/internalversion/factory.go:85: Failed to list *rbac.Role: the server was unable to return a response in the time allotted, but may still be processing the request (get roles.rbac.authorization.k8s.io)
Apr  6 00:30:24 manage01 journal: E0405 16:30:24.553119       1 storage_rbac.go:175] unable to initialize clusterroles: the server was unable to return a response in the time allotted, but may still be processing the request (get clusterroles.rbac.authorization.k8s.io)
Apr  6 00:30:54 manage01 journal: I0405 16:30:54.553375       1 trace.go:76] Trace[497557028]: "Create /api/v1/namespaces" (started: 2018-04-05 16:30:24.551881216 +0000 UTC m=+65.280593338) (total time: 30.001286917s):
Apr  6 00:30:54 manage01 journal: Trace[497557028]: [30.001286917s] [30.001141308s] END
Apr  6 00:30:54 manage01 journal: E0405 16:30:54.554464       1 client_ca_hook.go:78] Timeout: request did not complete within allowed duration
Apr  6 00:30:59 manage01 journal: E0405 16:30:59.381254       1 streamwatcher.go:109] Unable to decode an event from the watch stream: json: cannot unmarshal string into Go struct field dynamicEvent.Object of type v3.NodeStatus
Apr  6 00:31:24 manage01 journal: E0405 16:31:24.555168       1 storage_rbac.go:175] unable to initialize clusterroles: the server was unable to return a response in the time allotted, but may still be processing the request (get clusterroles.rbac.authorization.k8s.io)
Apr  6 00:31:24 manage01 journal: F0405 16:31:24.555357       1 hooks.go:188] PostStartHook "rbac/bootstrap-roles" failed: unable to initialize roles: timed out waiting for the condition

Okay can you check etcd just in case using:
docker exec -t -e ETCDCTL_ENDPOINT=https://<IP_ADDRESS>:2379 etcd /bin/sh -c 'etcdctl --cert-file /etc/kubernetes/ssl/kube-etcd-192-168-137-130.pem --key-file /etc/kubernetes/ssl/kube-etcd-192-168-137-130-key.pem --ca-file /etc/kubernetes/ssl/kube-ca.pem cluster-health'

And the output of docker inspect --format='{{.Args}}' kube-apiserver
And can you share what the setup is, how many nodes with what roles, and the specifications of the node(s) and where you are running them?

I think it maybe the old files in /etc/kubernetes. So I clean the dir /etc/kubernetes and /var/lib/etcd，reinstall the cluster again, the error info change into:
[workerPlane] Failed to bring up Worker Plane: Failed to start [kubelet] container on host [<IP_ADDRESS>]: Error response from daemon: linux mounts: Path /var/lib/kubelet is mounted on / but it is not a shared mount.

1.I install my cluster,follow on the rancher'doc:
docker run -d --restart=unless-stopped -p 80:80 -p 443:443 rancher/server:v2.0.0-beta1
2.Before install cluster ,I use these cmd to clean the environment:
docker rm -f -v $(docker ps -aq)
docker volume rm $(docker volume ls)
rm -rf /var/lib/etcd/
rm -rf /etc/kubernetes
3.The cluster only have one node,the roles are default.

@wlcumt What is the MountFlags option? Is it an environment variable or is it an engine flag or something like that? I'm getting the same error on my local boot2docker vm and I am trying to do the same thing you did to fix it. Or maybe this flag is Centos/RHEL specific?


feat: Add stetho to project

Fixes #1138
Changes:
Added stetho to project.
(Also fixed indentation of some lines)

@iamareebjamal Should I create a separate PR to fix indentation or it's okay here?


bruno-cli: init at 1.34.2

Closes https://github.com/NixOS/nixpkgs/issues/335270
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354111

x86_64-linux

:white_check_mark: 1 package built:

bruno-cli

Em qui., 7 de nov. de 2024 10:44, Lucas Eduardo @.***>
escreveu:

nixpkgs-review result
Generated using nixpkgs-review https://github.com/Mic92/nixpkgs-review.
Command: nixpkgs-review pr 354111
x86_64-linux ✅ 1 package built:

bruno-cli

—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/354111#issuecomment-2462277539,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADXXO6HEA4OVCY4BENEF7S3Z7NVFNAVCNFSM6AAAAABRKA3NMSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRSGI3TONJTHE
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

@lucasew I published the PR before it was working to avoid duplicated work, but thanks for the testing nonetheless!
The MODULE_NOT_FOUND error is fixed now, but the packaging needs improvement. Even tho this is a CLI app, removing cairo, pango, ELECTRON_SKIP_BINARY_DOWNLOAD = 1; makes the build fail as buildNpmPackage downloads every dependency of all workspaces. I was thinking of using npmWorkspace = "packages/bruno-cli", but that doesn't seem to work, probably because it depends on other workspaces.

I took some inspiration from https://github.com/NixOS/nixpkgs/pull/354339 and further reduced the closure size from ~200M to ~126M.

Result of nixpkgs-review pr 354111 run on x86_64-darwin 1

1 package built:

bruno-cli

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354111

x86_64-linux

:white_check_mark: 1 package built:

bruno-cli


Specifying nulls

Is there a way to indicate a NULL value? The default Hive value is \N (capitalised). When I upload a CSV file containing that value, I simply get the value N.
According the the Hive DDL it is not possible to specify a SerDe and NULL terminator. Is there perhaps a class property for this SerDe that can be set (similar to how separator, quote and escape characters are set)? I could not find such a property in the source code.
It is vital that one can distinguish between NULL and empty values and it would be very helpful if this could be added as a feature.

I agree with @parautenbach . I have this issue currently. I don't really see the burden .. there needs to be some explicit way to specify what is NULL. Don't mind encoding NULLs as '\N' by default. Maybe this can be an option?

I meet this problem also.

+1

This bit me as well. I have an external table in AWS Athena using OpenCSVSerde. It contains an int column that can be null. If I try to query this column, Athena complains HIVE_BAD_DATA: Error parsing field value '' for field 0: For input string: ""
I tried setting the the table property serialization.null.format'='' but that did not help.

+1

+1

+1

Hey @ogrodnek - I know it's an old issue but are you by any chance planning to introduce such a change?

Hi @lkudyba ,
This serde code was contributed to Hive back in 2014, and is being maintained there.
See: https://cwiki.apache.org/confluence/display/Hive/CSV+Serde and https://issues.apache.org/jira/browse/HIVE-7777


chore: GCForms release v3.1.0

:robot: I have created a release beep boop
3.1.0 (2023-06-09)
Features

2FA Authentication through Email (#2133) (2cc568d)

Bug Fixes

cypress acceptable use page test (#2239) (83b04e2)

This PR was generated with Release Please. See documentation.

Closing as this was used as a test


netcore21 upgrade

anyone ones what happens in appveyor builds over there? @stevejgordon

@valdisiljuconoks I've noticed this issue has begun recently for our functional tests. I've spent a little time trying to figure out what changed but have yet to narrow it down. Locally the functional tests pass so I think it's an environment issue with AppVeyor.
@MisterJames I'm tempted to suggest we ignore functional test failures for some of the recent PRs and perhaps disable until we can resolve the failure?


Does it support TW (TW2) servers?

Does it support TW (TW2) servers?
I can't find my servers when i getting install.

Yes, it should – we just haven't updated the installer to accommodate for the new-ish servers in asia.
You can manually set TW2 by selecting any region with the installer and then changing the server value in ...\prod-toolkit\modules\plugin-config\config.json to "TW2"

THANKS!


Add Optional section to napari info with numba and triangle

References and relevant issues
We've added numba and triangle as optional depends.
Bugs or issues related to one of them could be hard to track because they don't show up in napari --info so need to request also pip show triangle
e.g. https://github.com/napari/napari/issues/6709
Description
Adds an Optional section to napari --info listing numba and triangle.

Question: should this be above OpenGL, closer to the other modules? or below it, like I did, since it's less important--hopefully.

should this be above OpenGL, closer to the other modules? or below it, like I did

I don't have a strong opinion on this. I think either is fine so I've approved as-is — the information is there either way so I don't think we should bikeshed this and we should merge and iterate in the future if people are getting tired from all the scrolling 😂


npm run build does not write to the build folder.

Steps I followed:

git clone the project
npm install
npm run build
tried to open build/index.html in the browser.

However, it works if I npm start and open localhost:3000 in the browser.
@mxstbr

Hey, thanks for taking the time to submit a bug report! Are you on the master branch? Also, what do you mean with "tried to open build/index.html in the browser.", what exactly did you do?

Yeah, on the master branch. In the README it says

so, I assumed that once I run npm run build I should be able to open the build/index.html file on Chrome and expect the static site locally without running npm run serve. Am I correct ?

Not quite.
When you run npm run build, your app will be compiled to the build folder in the root of the project. To see your production application in the browser, you have to run npm run serve or upload the contents of the build folder to a web server.


Make sure credential provider is set before activate completes

This change addresses a possible race condition when an "eager" onStartupFinished extension invokes a git clone but the github extension whose activation event is *, has not yet registered the Github CredentialsProvider. This results in a username/password auth flow which ultimately fails as Github does not support that auth anymore

@microsoft-github-policy-service agree

@vkryachko, sorry for not getting back to you on this until now. As I do not see any issues in VS Code linked to this pull request I was wondering if you could file one with the repro steps to illustrate the problem. Thank you very much in advance.


make configctl call "newip" public to cron

For Multi WAN with DHCP it might be good to have periodic renewal of the interface to fix error with stall routes ( documented in #1821 ).
It could also help in other situations e.g. buggy dhcp servers in the wan world to renew the address in given time ranges.

I thought we were talking about a workaround... ;)
maybe we don't need this here, you could always copy the file command to have it retained locally. this can also be solved by ifconfig up/down or maybe even probing the dhcp pid with an appropriate signal?

You mean creating a file e.g.
/usr/local/opnsense/service/conf/actions.d/actions_dhcpupdate.conf
Add the action with descripton and set in cron?
So this file wont be overwritten?

yes, exactly

Ok, I'll close this one, but I'd not be sad if this would also go into master some time 👍

we will find the root cause and fix that so this isn't needed anymore, I promise.

I'm testing right now the openvpn multi remote patch and now I'm in this unknown state. Motivated to troubleshoot or are you in weekend yet? :)


Pfstate docs

Codecov Report

Merging #7 (7be698e) into develop (c43c67d) will decrease coverage by 0.01%.
The diff coverage is 22.22%.

:exclamation: Current head 7be698e differs from pull request most recent head c18dda7. Consider uploading reports for the commit c18dda7 to get more accurate results

@@             Coverage Diff             @@
##           develop       #7      +/-   ##
===========================================
- Coverage    78.72%   78.71%   -0.02%
===========================================
  Files           53       53
  Lines         3948     3946       -2
===========================================
- Hits          3108     3106       -2
  Misses         840      840

Impacted Files
Coverage Δ

portfolyo/core/mixins/plot.py
14.75% <0.00%> (+0.11%)
:arrow_up:

portfolyo/core/pfstate/pfstate.py
53.76% <0.00%> (ø)

portfolyo/core/pfstate/enable_hedging.py
70.00% <50.00%> (-2.23%)
:arrow_down:

portfolyo/__init__.py
100.00% <100.00%> (ø)

portfolyo/tools/zones.py
90.66% <0.00%> (-0.13%)
:arrow_down:

portfolyo/tools/tests/test_frames.py
99.20% <0.00%> (-0.01%)
:arrow_down:

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 c43c67d...c18dda7. Read the comment docs.


Don't apply "smart indent" to parent symbols

Bug/issue #, if applicable: 86438798
Summary
This fixes a minor issue where the "smart indent" logic/styling is mistakenly being applied to declarations for Objective-C classes/structs/protocols/enums when it shouldn't be.
Here is an example of how an Objective-C class declaration is currently being rendered due to this bug:

Here is how it should look with the changes from this branch:

Testing
Steps:

Download/unzip this TestAtObjC.doccarchive.zip test fixture
Configure VUE_APP_DEV_SERVER_PROXY=/path/to/TestAtObjC.doccarchive and run npm run serve
Open http://localhost:8080/documentation/testatobjc/mytestclass?language=objc in the browser
Verify that the declaration is on a single line now

Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[X] Added tests
[X] Ran npm test, and it succeeded
[X] Updated documentation if necessary

@swift-ci test


Reorganize the tap result link aqavit_tapresult_link

Is your feature request related to a problem? Please describe.
Currently the marketplace test result link is designed per each binary (for example: "aqavit_results_link" : "https://github.com/adoptium/temurin17-binaries/releases/download/jdk-17.0.6%2B10/OpenJDK17U-jdk_x64_linux_hotspot_17.0.6_10.tap.zip",").  As discussed and implemented in Ensure we are pushing AQAvit TAP files to the binaries repository for each release there will only be one final AQAvit TAP file per release ( for example https://github.com/adoptium/temurin17-binaries/releases/download/jdk-17.0.6%2B10/AQAvitTapFiles.tar.gz), most often for now the tap file will be uploaded to the release repo after all tests are triaged. We'd like to update the api query https://api.adoptium.net/v3/assets/feature_releases/${version}/ga?page_size=50&vendor=eclipse to get the aqavit_tapresult_link if AQAvitTapFiles.tar.gz is available.
Describe the solution you'd like
With that enabled and PR like https://github.com/adoptium/api.adoptium.net/compare/marketplace...sophia-guo:marketplace?expand=1 Marketplace-data   can be auto updated by workflow https://github.com/adoptium/marketplace-data/blob/main/.github/workflows/temurin-updater.yml
Describe alternatives you've considered
Or any other solutions suggested, maybe the issue https://github.com/adoptium/api.adoptium.net/issues/543?
Additional context
Add any other context or screenshots about the feature request here.

@johnoliver @tellison any suggestion?

@sophia-guo wrt moving aqavit_results_link, I think the issue here will be, is the decision to do 1 tap file for all binaries in a release specifically an Adoptium decision or something that would be true for ALL marketplace vendors. For instance Dragonwell appears to show a tap file for each binary:
https://github.com/dragonwell-releng/dragonwell-adoptium/blob/master/json/8/jdk_8u345b01.json
So moving it to the release would prevent them from doing this. Although the current behaviour will result in redundant entries for Adoptiums releases, that is probably preferable to restricting other vendors ability to do per binary tap files.
WRT adding aqavit_results_link to the adoptium api, we can certainly do that, if you are certain that Adoptium will remain doing one tap file per release, then we can put it on the Release object in the Adoptium API. Are we sure that it will remain 1 per release, since once we add it to the Release object we will not be able to alter that till api v4.

@smlambert what's your suggestion regarding @johnoliver 's concerns https://github.com/adoptium/api.adoptium.net/issues/558#issuecomment-1496672801


fix: internal listeners infinite retry loop

Our block tracker implementation relies on the presence of listeners to establish whether the polling should be continued or stopped.
However, an internal listener is being added in the getLatestBlock method, which will count as the other external listeners and will prevent the instance from stopping fetching new blocks. This creates an infinite loop in case the network is unreachable.
This PR aims to fix this behavior by keeping track of internal listeners' references in order to exclude them from the listener's count
Fixes https://github.com/MetaMask/eth-block-tracker/issues/163

It would be good to cover this with tests


Grid descriptions in workflow + community sample image

This pr contains:
-An explanation  of the difference between grid and stepped grid in the sample workflow of grid object placement.
-Replacement of Vina's workflow image to a better image she requested.

@radumg please merge this PR to master

cc @vinarahimian , this PR merged the requested image


represent the staking reward ratio of the beneficiaries using basis points

Now we're using Fraction when setting the reward ratio for beneficiaries, which is less efficient comparing to basis points.

Fixed in #103


Find updates of the components that make Lokomotive

The output looks like this:
$ ./scripts/find-updates.sh
Component               Current Version     Latest Version
---------               ---------------     --------------
kubernetes                     v1.18.2"            v1.18.2
calico                         v3.13.3"            v3.13.3
etcd                            v3.4.7"             v3.4.7
cert-manager                    v0.14.2            v0.14.2
contour                          v1.3.0             v1.4.0
Dex                             v2.23.0            v2.23.0
external-dns                     2.21.2             2.22.0
gangway                          v3.2.0             v3.2.0
metallb              v0.8.3-2-gf653773b             v0.9.3
metrics-server                   2.11.1             2.11.1
openebs                           1.9.0              1.9.1

All inputs on how to make it configurable, non-repeatable are welcome.

@invidian added changes PTAL.

Can we get this moving, it seems very close to being ready :)

I'd like to suggest one exercise before updating a component to a new version. We check the github repository or the concerned component and check if there are any issues reported by the community in the new version before updating in Lokomotive.

Ping @surajssd


My parameter "-w" will be treated as for node-js

I build a simple program 'ss'; but my parameter "-w" will be treated as for node-js. How should I do?
$ bin/ss -w
Error: unrecognized flag -w
Try --help for options
Commands:
-w: Write a file to ceph. -w folder/filename -f full_file_path_name
-r: Read a file from ceph. -r folder/filename -f full_file_path_name
-l: List files of a folder. -l folder
-d: Delete a file in ceph. -d folder/filename

Hi! What's your cli opts looking like for invoking nexe?
Also, to clarify, are you saying that node.js is interpreting your flags instead of your program?

Try compiling with -f true in your nexe opts. i.e
nexe -i ./app.js -o ./out -f true -F iojs -r 3.0.0

with -f true is work! Thanks!


bin/* scripts required to be in azkaban root dir

Starting azkaban solo server from bin dir cause:
Exception in thread "main" java.io.IOException: Cannot find 'database.properties' file in sql/database.properties
Similarly starting web app from bin produce:
Caused by: java.lang.IllegalArgumentException: User xml file conf/azkaban-users.xml doesn't exist.
In my opinion, scripts should be executable from anywhere.

Bump. Just lost an hour to this yesterday. :-/

Please feel free to submit a pull request to fix this issue.

I have the same problem with solo instance.

Seems like you are trying to run azkaban-solo-start.sh run from /bin directory.
Try to execute bin/azkaban-solo-start.sh bin from your root directory.


Allow package to work with App Engine 'goapp' tool.

Google app Engine can't assamble *.s files, so any GAE app that uses this library won't be able to run tests for example, due to "no such tool 6a". Relevant discussion - https://github.com/btcsuite/fastsha256/pull/2 .
The fix for this issue is simple - it consists of adding one flag to all .s files:
// +build !appengine

Do you want to add this, and create a pull request?

Setting it up would probably take me more time than it would take you. The change affects one file: popcnt_amd64.s and it requires adding // +build !appengine at the start of the file. I can do the pull request if you want, but it would probably be faster for you to do it.

Ok, lmk if you see issues.
--
Will Fitzgerald
On February 11, 2015 at 2:52:12 PM, ThePiachu<EMAIL_ADDRESS>wrote:
Setting it up would probably take me more time than it would take you. The change affects one file: popcnt_amd64.s and it requires adding // +build !appengine at the start of the file. I can do the pull request if you want, but it would probably be faster for you to do it.
—
Reply to this email directly or view it on GitHub.


"selectcase" as one word causing problems

"SELECTCASE" and "ENDSELECT" as single keywords with no space are considered valid Fortran.  Some legacy code containing these single word uses were resulting in a number of diagnostic errors of "unexpected end of scope"

Thanks for the report. Yep, this was just a typo that should be fixed in master now. It will be included in the next patch (likely later today).


Update PyPI Package

Hi @JoshData,
I am interested in finishing the work @acxz did with getting this repo ready for python packaging in #267. I would like to push the latest repo to PyPI to allow users to install via pip.
The last time the package was updated was in 2016: https://test.pypi.org/project/united-states-congress/. I think this is a good time to release the repo to PyPI for people to use. The repo now has an easy to use command to run tasks from the command line, and the repo's modules can be imported in the REPL and other scripts users would like to write.
I have tested the workflow by packaging this repo locally and uploading it to test.pypi.org, and was able to install it successfully install and run it as well on my machine.
I am interested in maintaining this python package along with @acxz. What What actions do you need to take to make this happen? Do you have any suggestions about going about this?

One thing I'd want to see merged before releasing:

Command line usage instructions and help for the various tasks.

This is something I can work on.

Rel issue: https://github.com/unitedstates/congress/issues/188
cc: @illegalnumbers

The last time the package was updated was in 2016: https://test.pypi.org/project/united-states-congress/. I think this is a good time to release the repo to PyPI for people to use.

It has been pushed to the main pypi directory: https://pypi.org/project/united-states-congress/ by @illegalnumbers

What actions do you need to take to make this happen?

For us to update the united-states-congress package we need @illegalnumbers to hand over the maintainership of the pypi package over to you or me (or @JoshData assuming he wants to maintain the pypi package see prev comments)
If @illegalnumbers is unresponsive, filing a support ticket at https://github.com/pypa/pypi-support under PEP 514, allows us to reclaim the unmaintained package.
With this we can upload a newer version of the codebase. All of this does not require any commits to this repo.
However, for unitedstates/congress to "support" such a pypi package, @JoshData would have to write off on a PR that adds installing from pypi as an alternate or recommended install method (again concerns on maintainership burden arise)

Does every commit to the master branch constitute the version incrementing?

As of now the version is fixed no matter commit changes. While semantic versioning has been mentioned, @Andrew-Chen-Wang's suggestion of date versioning makes the most sense. I think the best approach to version and releasing to pypi would be "code dumps", i.e. when things look good enough just make a commit for the version change in setup.py and a tag on github releases with the date version then upload to pypi. (again maintainership burden) Notwithstanding that, just a simple point release scheme is good enough (as long as we don't exceed the current data, which I doubt will happen, so that we can fallback to the date versioning scheme whenever the project wants.

Command line usage instructions and help for the various tasks.

This is def a good one, but I think is orthogonal to a release on pypi.
A previous suggestion by you:

One side-effect however, when I call usc-run from somewhere other than the repo root, it creates the data/ and cache/ dirs in the directory where I called it. I feel like this is undesirable, and it should always download to some default location unless a download destination is provided.

This is another good issue that can be solved with the XDG spec, specifically $XDG_CACHE_HOME/usc for cache/. For data/ as that is an output I think default should be current directory and user should be able to specify output data/ dir with a flag like --data-output Again tho, it should be its own issue from this.
I believe the closure of this issue would be accomplished with decision of adding pip install united-states-congress on the README.md or not.

I've found that sharing of maintainership on PyPi is great, but I also recommend a GitHub workflow that creates a PyPi release when a maintainer releases a GitHub release. This helps when there's a team of maintainers who don't have much down time.

Great suggestion, such a change would be required for the closure of this issue.

@acxz Just a note but for an auto release bot that releases everyday, you can take a look at the auto changelog and release bot at cookiecutter django for inspiration (or copy pasta)

I do not recommend a release everyday at all. I don't this project undergoes that many changes to warrant such sophisticated methods.

I do not recommend a release everyday at all. I don't this project undergoes that many changes to warrant such sophisticated methods.

Agreed. Please do not make a release every day.


Fix: Dragging the hour hand at some point causes the pointerRotate to…

修改bug：某些时候拖动时针会导致pointerRotate大于1080，导致time>24，表盘显示异常。

@shianqi
V2.1.2 published and add you to contributors

@ecmadao 😘嗯呐~


refactor: Improving DateBone.singleValueFromClient()

Avoid rawValue and str()-conversion all the time.
This is a PR draft for now, as I want to test it first.

Ok thank you. I think we gonna merge this for now, but its not entirely tested.
Afterwards I would like to improve and generalize the test-suite so that several test cases on this bone can be created.

Afterwards I would like to improve and generalize the test-suite so that several test cases on this bone can be created.

Take a look at https://github.com/viur-framework/viur-core/pull/797


Attendee Stickiness Confusing

#454 creates a confusing UI situation. Say I do the following:

Begin the checkout process, getting past filling in attendee details
Get distracted from the computer for a bit. Let the cart timer run out.
Go back to the shop to actually do the checkout process. Add a pass to my cart and begin the checkout process.

I will get presented with this screen:

which, A. feels bad because it looks like an error and I don't think I did anything wrong and B. doesn't make it obvious what I'm supposed to do next. (Especially since I have never done item assignment before because I wasn't required to do it the first time through, just purchasing one item for one attendee.)

I would argue that this is a flaw that's always been there, which is now more obviously exposed. The item assignment ui is terrible and unintuitive and always has been. :-p

Yeah, but if a person is just purchasing on item for one attendee, they shouldn't have to assign anything, is my feeling about it.

I'm not saying #454 needs to be reverted, if that's what you're thinking. I just think this needs to work more smoothly and the process should (ideally) not bring me to completely new screens because I had a cart which expired previously.

(I opened this ticket because I encountered this UI and was confused for a minute and I can imagine that someone who, unlike me, doesn't understand how Dancerfly works would just give up at this point because it looks like there are two errors, no fields to correct, and no "next" button to move on with.)

Just to be clear, I agree that this would be very confusing. I guess all I mean to say is that I'd rather fix this by making the assignment ui more intuitive rather than with more special casing. Of we could get rid of the special casing we already have, that would be even better.

But that seems more complicated. Again, I think we should optimize for the standard case, which is someone checking out for one attendee. Forcing them to assign items when there is only one attendee is extra work and an additional potential point for a user to get confused.

It is a UX regression from what we had before.

(Extra work in the sense that it is arguably unnecessary. How many ways are there of assigning one item to one attendee? Why are we providing a screen making them do that?)

From a user:

I got almost to the end of registering myself (as a new, not-logged-in attendee), and then backed up to the screen where I entered my eMail address.  Then I hit save and continue.  This apparently added me as an attendee twice.  I don't see how to remove the duplicate.  Indeed the whole function of [the attendees view] wasn't clear to me at first.  Now that I think about it I can see that you are trying to create payment for multiple people, each of whom can order different goods and services.  Adding the duplicate attendee got me stuck.  Nothing to do but bail out, clear all cookies and start over.  Indeed, attempts to add a second attendee end in gridlock.  There's no way to enter item selections for the second and subsequent attendees.

I ran into a similar problem, where as an organizer I was testing my event to see what adding an attendee would feel like from a user perspective and now I'm stuck with a fake attendee called "My Friend". Maybe the easiest thing to do would be to allow people to easily delete attendees?


master and devel not aligned

master and devel branches are dangerously not aligned.
@davidetome Please always merge master in devel every time you push a new commit in master.

Instructions here: http://www.yarp.it/md__8github_CONTRIBUTING.html
Check the paragraph "This is the typical workflow to fix a bug in the master branch."

@randaz81 @robotology/icub-developers @robotology/icub-support-team
Originally, we did not create a master/devel branch on this repo (see https://github.com/robotology/icub-main/issues/360). We probably need to agree (in person) on how to deal with this and what is the policy for IIT robots.

I definitely agree to discuss this in person.

Just to give an example of the different workflows now in place @valegagge recently updated configuration files w.r.t. a change in the firmware only for iCubGenova02 and iCubGenova04 : https://github.com/robotology/robots-configuration/commit/ed89b3535a0c9b6e4cc5fb273afb13ee29596119 , as they are the only one currently on devel branch.

Another thing to discuss: should we tag this repo for releases, i.e. adding it to https://github.com/robotology/icub-main/blob/devel/admin/scripts/README_RELEASE#L54 ?

F2F discussions are indeed necessary.


Read tagPolicy from config

Closes #326

@geniass I think this was fixed by https://github.com/GoogleCloudPlatform/skaffold/pull/332. Feel free to reopen if it's not the case!

Looks like it fixes it. Thanks!


prizeItemRenderFunction breaks winning index

I wanted to change the width of roulette-pro-regular-prize-item which is default 205px. I ended up using the prizeItemRenderFunction to accomplish this, but now the prizeIndex doesn't line up how I would expect anymore. Is it impossible to change the width of each item, or did I do it incorrectly/miss a feature of prizeItemRenderFunction.
Here is an example of my prizeItemRenderFunction:

roulette implementation:

Hi, @Michael24566! Thank you so much for this issue.
At this moment the roulette defines its items width from a design plugin. So when you change width in prizeItemRenderFunction it will work incorrect. Of course this must be fixed and I will do this when I have time.
For now I can suggest you this solution:
Keep your prize items render function.
Add a prop designPlugin and specify your items width and height there like this:
<RoulettePro
...
     designPlugin={() => ({
          prizeItemWidth: 100,
          prizeItemHeight: 100,
      })}
...
/>

It should works. If it's not, please let me know.

TODO:
To fix the case when the roulette items width and height or one of them is changing in the prizeItemRenderFunction.
Issue: Currently the roulette defines its items width and height from a design plugin.

This seems to have fixed my original issue, thank you @IvanAdmaers. After applying the designPlugin it seems to have made the center delimiter disappear. Is there a property I can pass designPlugin to keep the delimiter?
Thanks again I'll mark this as closed soon since the original issue has been fixed.

I do apologize for this confusion. Please, add the topChildren key to the designPlugin returning options.
      <RoulettePro
...
        designPlugin={() => ({
          prizeItemWidth: 100,
          prizeItemHeight: 100,
          topChildren: (
            <div
              className="roulette-pro-regular-design-top horizontal"
            />
          )
        })}
      />

That did the trick, this was exactly what I was looking for. Thank you again @IvanAdmaers.

@Michael24566, I’m glad I could help you with this issue. Feel free to open a new one anytime. And I’ll fix the prize items width and height definition problem as soon as I have time.
Have a great week!

To whom it may concern.
At this moment, the solution described here is up to date. More details - https://github.com/IvanAdmaers/react-roulette-pro/issues/30#issuecomment-1567291557


Update 03-generating-metagraph-jars.md

I think I found a typo on the BaseImage instructions...
This...
mv your_project_directory/modules/data_l1/target/scala-2.13/my-project-data-l1-assembly-0.1.0-SNAPSHOT.jar data-l1/data-l1.jar
Should be...
mv your_project_directory/modules/data_l1/target/scala-2.13/my-project-data_l1-assembly-0.1.0-SNAPSHOT.jar data-l1/data-l1.jar
data-l1-assembly should be: data_l1-assembly (with the underscore).
I noticed at the completion of everything the file was named with the underscore and not the dash.

oh @IPadawans just realizing that this might be stale (from Feb. 10). Do you know if this has been addressed?


Replaces DeveloperError(s) with Check(s) in Geometry

Part of #4794

Just a reminder to remove the first two commits from this PR.

Should be fixed now

👍


Add layers into database when ES is down

This pull request enables registry to add layers into the database even Elasticsearch is down.

Coverage increased (+0.002%) to 99.543% when pulling 0def9a9be331d61c5630b9dc35131e6f50b23b1e on JorgeMartinezG:master into ffeb73c0af895d3ee660f22ada39e1672892b3d4 on boundlessgeo:master.


Find installed software from shell script

Description
In pr #717 , we parse a shell script into statements. We need to fiter the install command and extact what will be installed in the command.
To Do

Filter the install command.
Find what is being installed by the install command.

Background
Previous work #682.
Super Issues
#521

For a case mentioned in #744 .
apt-get install --no-install-recommends --no-install-suggests -y gnupg1 ca-certificates # INSTALL ca-certificates
...
...
apt-get remove --purge --auto-remove -y ca-certificates # REMOVE ca-certificates

ca-certificates should NOT be in the installed package list.
Besides, we may need to add remove to the remove subcommands of apt-get:
apt-get:
  install:
    - 'install' # subcommand to install
  remove:
    - 'purge' # subcommand to remove a package
  ignore: # list of subcommands that don't add or remove packages
    - 'update'
  packages: 'dpkg' # refer to base.yml's method of collection

Closing with PR #756 merged.


Why did you not use PyTorch transforms for augmentation?

There no specific reason, you can try it yourself though if you want.


open_gui should also save a .rst file in the temp folder

Right now there will occur an error in Ansys after using open_gui() as soon as there is an atempt to postprocess results. Thats becasue the result file is not found in the created temp-folder. Would be nice to add a copy of the jobname.rst file there. Maybe as an option via a bool parameter for open_gui()?

Fixed in #108


Gradle build configuration for VUE

Setup Gradle build configuration for building and packing VUE code into Webapp

Merged via PR #123

Implemented and merged.


Speed config

Can you add a config file to modify the pump speed

There is a config file to change the speed


trouble compiling websocketFrame

xcode can't seem to find where .getText() comes from (i also can't determine from where this is inherited).  Xcode also complains that the constructor for IO::ByteBuffer(buffer, size) doesn't match anything, although this is clearly not the case. I think i'm missing something when it comes to how i've inlcuded ofxIO. I've already attempted to rebuild the project using project generator.
xcode 6
oF 0.9.0
any hints would be much appreciated.

problem solved. i had been using an older version of ofxIO

+1 :) Yeah, been updating a bunch of stuff lately.