Tooltip displaying NaN instead of actual values for Lines

I am attempting to generating a plot of the population of a radioactive element over time and when I create a Pandas dataframe, called decay_data and then try to plot it using the following code snippet:
import bqplot as bq

# Code for creating decay_data removed
# decay_data['time'] is a Pandas series of floats
# decay_data['Tl-208'] and decay_data['Pb-208'] are Pandas series of ints

# Labels and scales for Axes
x_time = bq.LinearScale(min = 0, max=tmax)
y_number = bq.LinearScale(min = 0, max=NTl)
ax_x_time = bq.Axis(label='Time (s)', scale=x_time)
ax_y_number = bq.Axis(label='Number of atoms', scale=y_number, orientation='vertical')

# Define tooltip (not working)
def_tt_Tl = bq.Tooltip(fields=['x', 'y'], formats=['.1f', '3.0f'], labels=['Time', 'Tl-208'])
def_tt_Pb = bq.Tooltip(fields=['x', 'y'], formats=['.1f', '3.0f'], labels=['Time', 'Pb-208'])

# Define the lines
line_Tl = bq.Lines(x=decay_data['time'], y=decay_data['Tl-208'], scales={'x': x_time, 'y': y_number},
                   display_legend=True, colors=['red'], labels=['Tl-208'], tooltip=def_tt_Tl)
line_Pb = bq.Lines(x=decay_data['time'], y=decay_data['Pb-208'], scales={'x': x_time, 'y': y_number},
                   display_legend=True, colors=['blue'], labels=['Pb-208'], tooltip=def_tt_Pb)

# Crete and display Figure
fig_counts = bq.Figure(axes=[ax_x_time, ax_y_number], marks=[line_Tl, line_Pb],
                       legend_location='right', legend_style={'fill': 'white'},
                       title='Counts versus Time', background_style={'fill': 'black'},
                       layout={'width': '500px', 'min_height': '400px'})
display(fig_counts)

the tooltip just displays NaN for the values of time and counts instead of the actual x and y values of the Lines.  It DOES seem to work with Scatter, but not Lines.
Seems to be a bug based on what I read of the documentation on Tooltip in the code states "All the attributes  of the mark are accesible in the tooltip" and Lines is a Mark.  Furthermore the bqplot documentation shows Lines as having tooltip as a valid attribute.

As a hack work around, you can add both Lines and Scatter marks with the same x and y to the chart. Make the Scatter marks invisible (colors='0000aa00') and the tooltip hover on the Scatter marks. Under the hood, you'll get the info from the Scatter marks, but you'll only see the Lines.

Thanks, I will give it a try!

The documentation of bqplot.marks.Lines (marks.py)  states:
The fields which can be passed to the default tooltip are:
    name: label of the line
    index: index of the line being hovered on
    color: data attribute for the color of the line

So, no x or y fields.
However, if you add markers, for example line_Tl.marker = 'cross', then the tooltip correctly displays x and y values when hovering the markers (but still no value between the markers, i.e. NaN or nothing depending on the format).


Several severe bugs.

útraque
Itaque, Utique (uppercase)
Expecto  (uppercase)

These problems are related: all the special cases (uppercase, prefixes, suffixes etc.) are treated together.

Fixed.


Make custom parameter keys consistent with android

We have some issues with creating group dialogs across platforms.
Like, if group is created from android, it was not reflecting in iOS. From further investigation we could realize that in android and iOS using different keys from custom parameters for the system message after creating group.
See logs when receiving system message after creating group
Created From Android
[ChatService]: RCV: <message xmlns="jabber:client" type="headline"<EMAIL_ADDRESS><EMAIL_ADDRESS>id="5a97f79f154a8f8a030ccec9">
  <extraParams xmlns="jabber:client">
    <dialog_id>5a97f79fa0eb47308527cfeb</dialog_id>
    <dialog_type>2</dialog_type>
    <moduleIdentifier>SystemNotifications</moduleIdentifier>
    <dialog_name>ten</dialog_name>
    <notification_type>creating_dialog</notification_type>
    <occupants_ids>43678275,43678330,43678386,43679395</occupants_ids>
  </extraParams>
</message>

fCreated from rom iOS
[ChatService]: RCV: <message xmlns="jabber:client" type="headline"<EMAIL_ADDRESS><EMAIL_ADDRESS>id="5a97fe2a45ac9817430041b4">
  <body>jmathew انشأ مجموعة مع johny,telvin,ajinn</body>
  <extraParams xmlns="jabber:client">
    <moduleIdentifier>SystemNotifications</moduleIdentifier>
    <notification_type>1</notification_type>
    <dialog_id>5a97fe2aa0eb47394927d02a</dialog_id>
    <room_name>19</room_name>
    <room_updated_date>1519910442</room_updated_date>
    <type>2</type>
    <current_occupant_ids>43678275,43678330,43678386,43679395</current_occupant_ids>
    <date_sent>1519910442</date_sent>
  </extraParams>
</message>

Differences

Parameter
in android
in IOS

value for notification_type
creating_dialog
1

dialog type key
dialog_type
type

I could find inconsistency in the following codes.
https://github.com/QuickBlox/q-municate-services-ios/blob/3370000d584c46051077b99f81bf603bf8fe7682/QMChatService/QMChatService/Categories/QBChatMessage%2BQMCustomParameters.m#L34-L40
https://github.com/QuickBlox/quickblox-android-sdk/blob/69c6bae30daa550c9149abeebc825508c80ffb4f/sample-chat/src/main/java/com/quickblox/sample/chat/managers/DialogsManager.java#L26-L30

@jkmathew, Hello.
You are right. These keys are different.
Mentioned parameters are custom, so it depends on you how they will be named.
As for QMServices(iOS) they were initially designed for cross-platform application Q-municate and parameters are taken from our internal cross-platform specification.
As for Android, please take a look at this parameters and this class.
There are no inconsistencies in these parameters in Q-municate application either in Android, iOS or JS platforms.

Okay, Thank you for your clarification


Numbers support is limited

It seems that number to words support is limited only to bilions, after that the last digits with billion are used which results in wrong results:
e.g.
dig<PHONE_NUMBER>000000.words @dns.toys

should be
nine quadrillion ,
eight hundred seventy six trillion ,
five hundred forty three billion ,
two hundred ten million

however the output is
9876543210000000 = five hundred forty-three billion two hundred ten million"

Thanks for reporting this. Removed the lib and wrote a new algorithm for the conversion which now works up to the limit of int64 (~quintillion),


fix: change calls to "getSupportedChains()" to use "PluginManager.supportedChains"

Description
fix: change calls to getSupportedChains() to use PluginManager.supportedChains
supportedChains is a reactive state variable so it will cause re-renders when it changes
using adapters.getSupportedChains() isn't reactive.
Notice

[X] Have you followed the guidelines in our Contributing guide?
[X] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[X] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[ ] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)
No issue, but prevents an issue if we have a feature flag that toggles support for a chain adapter (such as adding a new chain like Osmosis).
Also changes the code to be consistent in how we check if a chain is supported
Risk
There shouldn't be any risk right now until we add a new chain plugin with a feature flag
Testing
That that the following load correctly:

Portfolio
Transaction history
Foxy
Yearn

Screenshots (if applicable)

@cjthompson as discussed, let's delete useChainAdapters hook for consistency and maintainability, now that usePlugins exposes the chainAdapterManager

@cjthompson as discussed, let's delete useChainAdapters hook for consistency and maintainability, now that usePlugins exposes the chainAdapterManager

I agree but I'm going to do that in a separate PR.


Add<EMAIL_ADDRESS>support

Add server side auth agent support see my related ssh2 PR

Should i provide more information about this ? Is there missing tests or something ? Please free to ask for refinement if you want !

Thanks, landed in 500e9014056104c57d331492efd50fe4ca66b042.


:bug: Fix OCI client configuration logic

This commit fixes the logic deciding to create a TLS client for OCI
charts.
The existing code was creating a non-TLS configured client if either the CA file
was unspecified or if insecureSkipTLSVerify was set to false. If a CA
file was specified then insecureSkipTLSVerify is always false, which meant
that a non-TLS client was always created, causing cert validation
failures if the OCI registry is served over TLS.
This commit changes the logic to create a non-TLS configured client if
both CA file is unset AND insecureSkipTLSVerify is false.

/cc @mboersma

Sorry I introduced this bug in #238 😞. @Jont828 @mboersma Would appreciate getting this merged and released ASAP if possible 🙏


[feat ⚡] Discord Server icon 🐸

Icon name
Discord
Use cases
Since discord icon is missing so, I think we should add that icon the
Design ideas
We can simply copy the discord logo as correctly have
Here is an image example

Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

#670


Phone Froze while in Snapchat and then Rebooted after previously randomly rebooting the night before

Yesterday my phone had rebooted overnight so I had to go back to Unc0ver and re-jailbreak. After about an hour of being back in a jailbroken state, my device froze and crashed (rebooted) once again.
Device (please complete the following information):
iOS 12.1.1
iPhone 8 Plus
Latest Unc0ver version and latest resources version
Place an "x" between the brackets if true:

[ ] this is a bug others will be able to reproduce
[ ] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[ ] this issue is present after a rootfs restore
[x ] this issue is present on the latest version of unc0ver

Logs
I installed C4shed today so if it happens again I can provide a log.

https://github.com/pwn20wndstuff/Undecimus/issues/927#issuecomment-482536993


Scale SVG to fit in container?

It seems to create a fixed width svg. If there isn't, how about adding an option to make it scale to the width of the container?

I'm having the same issue, the images it creates are too big and don't respect the width of the container in css

I managed to fix this issue like this:
$('svg').removeAttr('viewBox');

$('svg').each(function () {
    let svg = $(this)[0];
    let bbox = svg.getBBox();
    svg.setAttribute("viewBox", bbox.x + ' ' + bbox.y + ' ' + bbox.width + ' ' + bbox.height);
});

$('svg').css('width', '500px');
$('svg').css('height', 'auto');

of course this doesn't work if you run this before the SVGs have finished loading...


targetSdkVersion on Android is too low

Describe the bug
While upgrading to RN 0.60.5 I got this error while compiling my project:
Task :react-native-sensors:lint FAILED

TargetSdkVersion No Longer Supported
../../build.gradle:13: Google Play requires that apps target API level 26 or higher.

 10
 11     defaultConfig {
 12         minSdkVersion 16
 13         targetSdkVersion 22
            ^^^^^^^^^^^^^^^^^^^
 14         versionCode 1
 15         versionName "1.0"
 16         ndk {

As of the second half of 2018, Google Play requires that new apps and app updates target API level 26 or higher.

To Reproduce
Steps to reproduce the behavior:

Set up a project with RN 0.60.5 using react-native-sensors
cd android
`./gradlew build
See error

Expected behavior
Not having this error.
Desktop (please complete the following information):

React Native Version: 0.60.5
Device: N/A
Version: 5.3.0

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add precip unit conversion to CMIP6 cleaning

CMIP6 precip is in kg/m2/s, needs to be mm/day (e.g. multiply by 24 * 60 * 60). We should add this in to the standardize_gcm function, could be a flag similar to leapday removal.


Update recomended gcc version on installation instructions for Windows users

The yotta documentation about installating gcc should be updated and be consistent across multiple platforms (e.g. Linux section shows gcc 4.9; Windows section shows gcc 4.8 instead). We should keep updated the latest version known to work with all host OS.
http://docs.yottabuild.org/#installing-on-windows

PRs welcome :)
For the installation guide, this is the file to create a request against: https://github.com/ARMmbed/yotta/blob/master/docs/index.md

Created PR
https://github.com/ARMmbed/yotta/pull/357


Doesn't work anymore after the update

this hack always use to work but after the update it just stopped working.
Please fix.

Love how detailed this is. Includes what hack you're talking about, even! Lovely 👍

Sorry if I wasn’t describing it the best.
Basically what happens now (after the update) the get all blooks hack just doesn’t work but that twitter thing still pops up. Some of the other hacks work tho.

i have found some one else with working hacks

i have found some one else with working hacks

Where? I'd be happy if you could share the link!

@THlCK Check out the user @GooseterV's Blooket hacks!

Gsnipe1blookethacks / blookethack

just use token hack

@godlygamer123 Gsnipe1 didn't actually make them, he stole them from someone else.

skill issue L

Working cheats here: https://github.com/Minesraft2/Blooket-Cheats


Meta: Switch from Prettier to dprint for formatting

Are there any considerations about switching the formatter from Prettier to dprint? dprint is a plugins-based formatter, and it's written in Rust so it's fast.
But speed and performance isn't the most important factor, dprint is highly configurable while Prettier is opinionated. So, code styles can be easily customized to get satisfication.
The reason why I recommend dprint is that I've noticed you opened an issue about the bug of  formatting CSS selectors with comments in Prettier: https://github.com/prettier/prettier/issues/15356 . dprint doesn't have official plugins for formatting CSS at this moment, but here is a third-party one: Malva.
Malva is also highly configurable and customizable, and it provides more options than Prettier. Also, it works with comments better than Prettier. For the bug you caught in Prettier, Malva works well: Malva playground. Malva also fixes some other bugs in Prettier, such as https://github.com/prettier/prettier/issues/16232#issuecomment-2118609637 .
Feedback is welcomed.

Biome could replace both prettier and eslint :eyes:

Thank you but prettier is good enough for now.
Biome doesn't have compatibility with eslint plugins, which we have a lot of.

I gave it a try in https://github.com/refined-github/refined-github/commit/6210d1ee10a3a98344a0e00c779501a1e1567674 but I think it's a downgrade in a few spots, especially with comments being wrapped to the next line, or moved before the selector.
This was done for https://github.com/refined-github/refined-github/pull/7553

Quotes can be configured.

Would you be open to send a PR? Last week I had two issues:

:has(), :is() wrapping
comments position

If the first one can be fixed to maintain what we expect, we can probably figure something out with the comments positioning. Biome actually moved some comments before the comma and that made sense, for example.
- a, /* comment */
+ a /* comment */,
b {

}

But in the last dprint PR, the formatting in this case was changed to this instead, which isn't what we expect:
a,
/* comment */ b {

}

We're only using Prettier for CSS and GQL for now. I think GQL would still need Prettier because I don't see it (via dprint plugin)
https://dprint.dev/plugins/

Can you give me some examples about :has() and :is()? (expected results)
Recently I'm fixing comments problems.
dprint has a Biome plugin and Biome supports GraphQL, so we can format GraphQL indirectly.
dprint also has a Prettier plugin. If you don't like Biome, you can still use Prettier to format GraphQL. Compared to using Prettier directly, running Prettier in dprint will be faster because files will be formatted in parallel.

the expected result is what we have now, more or less. The wrong result is when multiple long selectors are on a single line inside is(), as shown in my last comment
yes GQL would need to use biome or prettier, I had also set biome for GQL without issues in my PR. Prettier would be preferred since it's part of our dependencies anyway (via XO)

Just enable "linebreakInPseudoParens" option, and the result of :is() and :where() can be better. playground

Malva v0.6.0 has been published with many comments improvements. Would you like to try it again? Don't forget to configure Malva as the dprint.json example below:
{
  "malva": {
    "quotes": "preferSingle",
    "blockSelectorLinebreak": "always",
    "linebreakInPseudoParens": true
  }
}

Can you give me some examples about :has() and :is()? (expected results)
Recently I'm fixing comments problems.
dprint has a Biome plugin and Biome supports GraphQL, so we can format GraphQL indirectly.
dprint also has a Prettier plugin. If you don't like Biome, you can still use Prettier to format GraphQL. Compared to using Prettier directly, running Prettier in dprint will be faster because files will be formatted in parallel.

I had a look at this comment unfortunatly it seems like graph ql formatter is unreleased for biome so dprint can't support as it's not released.


.T attribute evaluates to a pandas DataFrame, not a babypandas DataFrame

See title. This leads to weird issues in test cases. The only line involving .T in bpd.py is
self.T = _lift_to_pd(self._pd.T)

and _lift_to_pd seems to convert objects that are pandas DataFrames to babypandas DataFrames, but that doesn't seem to be happening here. @eldridgejm any ideas?

lift_to_pd is a higher order function which takes in a function and returns a wrapped version whose output is "lifted" to a BabyPandas object, when appropriate. If the item passed into lift_to_pd is not callable, it is immediately returned without modification.
In this case, self.T = _lift_to_pd(self._pd.T) wasn't working as expected because self._pd.T is not a function, but rather a pandas DataFrame. So _lift_to_pd just returned the original dataframe, unmodified.
This also results in a separate "bug" -- self.T is set once and does not change, even if the data frame is mutated. Of course, babypandas favors immutability over mutable methods, so this might never actually happen. Still, I moved .T and the other attributes to properties so that they are dynamic.


chore: set logger without mucking around with env vars

Env vars are global to a process. Setting them from within the tests is problematic because those tests are run concurrently in the same process.
tracing allows us to set a logger for the current stack scope using set_default. Instead of fighting over which test should initialize the logger, we can only set it for the current scope of each test.

What problem are you trying to fix?

What problem are you trying to fix?

I hit a panic when trying to use set_default in one of the tests because another test was already setting a global logger.
I figured that setting the env var and using a OnceCell is kind of a hack so I opened this PR as a way to only set a logger for the current test.

This is tailored to my workflow. My requirements are:

cargo test shows me log for the integration tests. It's enough for me to run cargo test flappy_ice_lite_state to get my reasonable defaults.
I can override using RUST_LOG like so: RUST_LOG=blaha=trace cargo test flappy_ice_lite_state

With 1, this is what I expect.
martin@nugget:~/dev/str0m$ cargo test flappy_ice_lite_state
    Finished test [unoptimized + debuginfo] target(s) in 0.06s
…
running 1 test
2024-04-24T04:41:07.790877Z  INFO str0m::ice_::agent: Add local candidate: Candidate(host=<IP_ADDRESS>:1000/udp prio=2130706175)
2024-04-24T04:41:07.790927Z  INFO str0m::ice_::agent: Add local candidate: Candidate(host=<IP_ADDRESS>:2000/udp prio=2130706175)
2024-04-24T04:41:07.790977Z DEBUG str0m::channel: Allocate channel id: ChannelId(0)
2024-04-24T04:41:07.791111Z DEBUG str0m::change::sdp: Create offer
2024-04-24T04:41:07.791117Z DEBUG str0m::change::sdp: Accept offer

This PR seem to not have any reasonable defaults.

cargo test shows me log for the integration tests. It's enough for me to run cargo test flappy_ice_lite_state to get my reasonable defaults.

cargo's default behaviour is to not show logs if the test is passing. To override that, you need to pass --nocapture. Taken that into account, running cargo test flappy_ice_lite_state -- --nocapture does indeed display the logs.
I did notice that RA didn't show me all usages of init_log and the guard was dropped too early (including for the test you mentioned) which is now fixed.
If you don't want the --nocapture behaviour then we can remove the with_test_writer from the logger setup. However, having passing tests be quiet by default is IMO kinda nice.

cargo's default behaviour is to not show logs if the test is passing. To override that, you need to pass --nocapture. Taken that into account, running cargo test flappy_ice_lite_state -- --nocapture does indeed display the logs.

Fully aware of all that. And no, I don't want to write --nocapture on every test run.

cargo's default behaviour is to not show logs if the test is passing. To override that, you need to pass --nocapture. Taken that into account, running cargo test flappy_ice_lite_state -- --nocapture does indeed display the logs.

Fully aware of all that. And no, I don't want to write --nocapture on every test run.

We can remove with_test_writer then!
I would like to understand the reasoning though. In what scenario do you need to see the output of a passing test?

I would like to understand the reasoning though. In what scenario do you need to see the output of a passing test?

Many times when I use the integration tests, it's to follow a flow of events happening. My use case is not necessarily a pass/fail check, it's figuring out what's going on using an integration test as reproduction.
This is different to unit tests, where there pretty much always is a pass/fail and I have no need to see logs for that.


Documentation of meeting on 7th October

Document meeting and add to meeting summary tool

https://docs.google.com/document/d/1v92Z6lyFniVRaIT_zJBxbeIdYMwIJfd3CEQxkaJr6q4/edit?tab=t.0


Resolve npm audit resolutions once and for all

This adds npm-force-resolutions as a dependency and automatically
runs it before installing something, to hopefully keep transitive
dependencies at a non-vulnerable version even if their parent
dependencies have not updated yet, even across multiple dependency
updates.
(Edit: adding it as a dependency doesn't work, since it runs before installing something - like itself. Ran with npx now and pinned the version there. Unfortunately that won't be recognised by dependabot, but I guess it's the best I can do for now.)
I'm not sure yet if a preinstall script will work for maintaining
it across dependabot updates and with npm ci runs, but I figured
we can at least try and see if it works.
Note that this would not remove the need to run npm audit fix or add a resolution every now and then, but at least once a resolution has been set, it should hopefully not need to be set again.

OK, seems like this isn't the solution either, since it breaks for people not using npm lockfiles...


Streaming category and sub-categories

HI,
On the first category (Streaming) If I click one of the subcategory? il will play all the videos on that category?  o will let me choose?

It will behave like the website (http://tv.jw.org/) and play all the videos for that category, but starting at the "broadcast time" so you may jump into the middle of a video.

Thanks


!ls invokes ls in shell, but !cd won't invoke cd in shell

Hi,
I understand you have already implemented .cd as the change directory function. But since cd is a shell command as well, would it be possible to have !cd invoke cd as well?
Also, could we have the auto-completion of commands or files when pressing tab key?

The reason .cd exists is because when you run !cd it runs in a subprocess which doesn't affect the working directory of the shell, just the subprocess. I can see the value of auto-completing other command names and files though. Marking as a feature request.


It is possible to send DTMF however, the delay is 3 - 4 seconds. Any idea what the cause is?

You might try adding in your dial plan a Set(TIMEOUT(digit)=1) before you enter SpeechBackground().


docs(instantsearch): keep autocomplete in sync with query during navigation

FX-1942
This PR updates autocomplete query to stay in sync with the one in InstantSearch (with routing enabled).
I also changed the templates from strings to html tagged templates, which allowed me to remove preact as a dependency.
We can push the change to the other flavours after we agree on the code for this one.
Preview →

Good find @FabienMotte, this happens because there is no initial pushState (as the state is already set in the URL), so the state isn't available from the popstate event at this time. I've changed the code to retrieve the query directly from instantsearch's helper.

Thanks @dhayab! 👏
Is the sandbox working on your side? Not sure why CodeSandbox is giving a syntax error: https://codesandbox.io/s/github/algolia/autocomplete/tree/docs/instantsearch-example-query-sync/examples/instantsearch


Threema Support

Feature Description
Would it be possible to include Threema to the supported services? Please see https://web.threema.ch
Motivation and Context

Mockups, Screenshots (if available):

just saw the issue 17 days ago. closed.

Scanning of Threema-QR does not work on iOS 10.3.3 app and FRANZ Version 5.0.0-beta.14 (5.0.0-beta.14.451).
The code is focused but nothing happens :(


Replace Hash with ChangeResolver::ResultSet in stack upgrade change resolver

Add a ChangeResolver::ResultSet class.

Removes the need for upgraded_services.select { |s| s.split('/')[0] == stack } type of filtering in report generation.
Changes things such as changes.added_stacks to changes.stacks.added

Complements #3185

Should this be milestoned for 1.5?

Should this be milestoned for 1.5?

Doesn't change functionality or fix any errors, so it's not that important.
Just a byproduct of trying to make sense of the original #3185 PR.


RFP - hidhide

Checklist

[x] The package I am requesting does not already exist on https://community.chocolatey.org/packages;
[x] There is no open issue for this package;
[x] The issue title starts with 'RFP - ';
[x] The download URL is public and not locked behind a paywall / login;

Package Details
Software project URL : https://github.com/ViGEm/HidHide
Direct download URL for the software / installer : https://github.com/ViGEm/HidHide/releases/download/v<IP_ADDRESS>/HidHide_1.2.98_x64.exe
Software summary / short description:  Gaming Input Peripherals Device Firewall for Windows.

I pick up this package :slightly_smiling_face:
I actually already have created a Chocolatey package which is currently waiting for the review of Moderators :wink:

This package has been reviewed and approved by @Windos :slightly_smiling_face:

Thanks for creating the package, @Xav83


Door/window sensor issues in zwave-js 10.0.4

Door/window sensors are very broken in zwave-js 10.0.4. They're fixed in zwave-js/node-zwave-js#5037, which is included in 10.1.0. Please release an update with #2643.
Related issues:
zwave-js/node-zwave-js#5027
hassio-addons/addon-zwavejs2mqtt#409

Released 7.2.0


Clean up Keychain tests

Simplify the nesting of do-catch blocks in the Keychain tests.

Travis CI's infrastructure for iOS builds has been at capacity for almost 20 hours, so I'm merging this without waiting for the CI tests to finish.


mixin error Server crashes upon startup

latest.log
Caused by: org.spongepowered.asm.mixin.transformer.throwables.MixinPreProcessorException: Attach error for forcecrawl.mixins.json:PlayerPose from mod forcecrawl during activity: [Transform -> Method updatePlayerPose(Lorg/spongepowered/asm/mixin/injection/callback/CallbackInfo;)V -> INVOKEVIRTUAL -> net/minecraft/class_304::method_1434:()Z]

If you install this mod on your client, it will work on a server. Currently, this mod doesn't have "official" server support. However, if you are on a server, this mod will still work, but it won't show you crawling. I may add this functionality eventually.

It doesn't play well on servers from what I've experienced
https://user-images.githubusercontent.com/77690119/216809888-6ea944e5-4bfc-4290-937f-db9d427743ab.mp4

Got it, I will make sure to add that server functionality as soon as possible.


chore: canary

Summary
DO NOT MERGE ME, canary test

Test Plan

Require Documentation?

[ ] No
[ ] Yes, the corresponding rspack-website PR is __

!canary


Removed console.log('data mode: array')

Removed console log message.

Closing this pull request as pull request https://github.com/ratiw/vuetable-2/pull/192 already removing the console.log('data mode:array').


Revert "[reflection-dump] Teach reflection dump how to dump protocols from the __swift5_protos section of a .o"

Reverts apple/swift#25617
Failure in rdar://51944326 seems to be due to this change. Reverting it to unblock the bots.
CC: @gottesmm

@shahmishal  could you merge this to unblock the bot? I don't have force merge access.

@swift-ci please smoke test


devops: Long branch names fail CI because GCP resource name is too long

Describe the issue or request
When I open a PR with a long branch name, CI fails with "Invalid value for field 'resource.name'".
Expected Behavior
CI passes on long branch names.
Current Behavior

ERROR: (gcloud.compute.instances.create-with-container) Could not fetch resource:

Invalid value for field 'resource.name': 'generate-checkpoints-testnet-label-cached-state-from-disk-32f315b'. Must be a match of regex '(?:a-z?)'

https://github.com/ZcashFoundation/zebra/actions/runs/6179715964/job/16776349421#step:11:78
Possible Solutions
Reduce the length of newer test IDs. (See the list below.)
Rejected Solution
Truncate the branch name to fit within the length limit. (I'm pretty sure this used to happen?)
Is this happening on PRs?
Yes
Is this happening on the main branch?
No, it has a short name

I can't find the specific message, but we onced talked about this issue and did not take further action on it, just informing that we had to keep the branch name short and without special characters to avoid this kind of issues (which also happens on other places, like docker build tags)
We could truncate the name, for certain, but we'd have to consider all places where this might happen.

I can't find the specific message, but we onced talked about this issue and did not take further action on it, just informing that we had to keep the branch name short and without special characters to avoid this kind of issues (which also happens on other places, like docker build tags)
We could truncate the name, for certain, but we'd have to consider all places where this might happen.

I think this is more of a problem now because the generate-checkpoints-testnet test ID is very long. So maybe we could fix this bug by making that name shorter instead?
The newer longest IDs are:

generate-checkpoints-testnet
generate-checkpoints-mainnet
full-sync-to-tip-testnet

Before they were added, the longest ID was:

sync-past-checkpoint

Here are some alternatives:

checkpoints-testnet
checkpoints-mainnet
full-sync-testnet

@gustavovalverde is this fixed in PR #7616, or does it need to wait until after that PR merges?


[CXF-7069] Finish JPACodeDataProvider implementation

This PR resolves the following issues :

shared em between all clients.
error when persisting BearerAccessToken (we annotated OidcUserSubject attributes with @Lob)

JPACodeDataProvider has now :

a resource-local entity manager version.
In this version, transactions are managed by JPACodeDataProvider.
a container-managed entity manager version (CMT).
In this version, transactions are managed by the container (i.e. Spring).

Tests have been added to check both Hibernate and OpenJPA compliance.
Bad news is that overall performance is not quite good.
Here's the number of request issued for basic OIDC / Oauth2 interactions :
|                                      |  Hibernate CMT | OpenJPA CMT      |
| Authorization code flow request      |     1          |      8           |
|                                      | S=1            | S=8              |
| Authentication                       |     1          |      8           |
|                                      | S=1            | S=8              |
| Grant                                |     9 (5 real) |     16           |
|                                      | S=1,I=7,U=1    | S=8,I=7,U=1      |
| Token Request                        |    15 (11 real)|     25           |
|                                      | S=2,I=9,U=1,D=3| S=11,I=9,U=1,D=4 |
| Token Introspection                  |    2           |     14           |
|                                      | S=2            | S=14             |

Closed as too old pr


Flights tab visibility

The Flights tab is only visible when the Blog is enabled.
Orange PI PC / Armbian | Apr 6, 2016 revision - adsb-receiver using MySQL

Fixed in commit eff75b058121b7363e6ac948101cecefb4bd8f6c.

Flights tab still visible when lite version is installed.
No option in portal to remove flight search box.


Don't print individual run timings

It's a lot of output you don't need most of the time but more importantly, when running the script from a TTY it can be relatively slow.

Hum, what about printing a single . for each iteration?

IMO the current approach is pretty good. Hasn't been a problem?

But we could add a --quieter option for Jean. Will accept that PR ;)


Bug in PawsxProcessor

Hi @sebastianruder,
There is no need to need to remove the first line of each file (the header) in the PawsxProcessor because it's already done during the download by the pawsx_preprocess() function.
We are losing the first pair of sentences for Pawsx files with the current code.

PR: https://github.com/google-research/xtreme/pull/71


AWS deprecation - CodeBuild AL2 AMI Upgrade

We are reaching out to you because you ran a CodeBuild build using a custom Docker image in the last 4 weeks. AWS CodeBuild is updating the operating system used for the CodeBuild's build fleet from Amazon Linux 1 to Amazon Linux 2, starting September 27, 2021. This update will also result in the default Docker storage driver to be updated, which may impact your custom Docker image’s ability to run in CodeBuild. Specifically, if your custom Docker image does not include the VOLUME parameter, your build will fail after we upgrade the default Docker storage driver. To ensure your custom Docker images can continue to run after the storage driver upgrade, please validate that your Dockerfile includes the following parameter: VOLUME /var/lib/docker. Please make this change to ensure your builds are unaffected during this update process. For an example, see [1].
CodeBuild customers who build custom Docker images on CodeBuild rely on a feature known as Docker-in-Docker. When Docker-in-Docker support was launched in 2013 [2], the VOLUME command was required for images supporting this feature. CodeBuild did not enforce the presence of this command due to the use of a storage driver that supported recursive usage and allowed custom images to run without the required VOLUME command. However, with the upgrade to Amazon Linux 2, CodeBuild will be upgrading the underlying Docker storage driver to use overlay2. The overlay2 driver does not support itself as a backing filesystem, meaning that Docker containers using overlay2 cannot run inside another Docker container using overlay2, unless the VOLUME command is present. Adding the VOLUME command to your custom Docker image’s Dockerfile will ensure your builds remain unaffected during and after this upgrade[3].
[1] https://github.com/jpetazzo/dind/blob/master/Dockerfile#L20
[2] https://www.docker.com/blog/docker-can-now-run-within-docker/
[3] https://docs.docker.com/storage/storagedriver/select-storage-driver/#supported-backing-filesystems
Assigning @pietroalbini as you're most familiar with our codebuild usage (I think just for releases today?). We should make sure that the images don't break, though it thankfully won't happen just before this upcoming release next week.

This shouldn't impact us, we're not relying on docker-in-docker.


fontist help and fontist --help should exit cleanly with return code 0

fontist --help exits with code 1. This is not ideal. Ideally it should exit with code 0 (cleanly). If the arguments are wrong (or whatever) then it should print the help text and exit with code 1.
Should exit code 0:
fontist --help

what happens right now:
jcbhmr@PIG-2016:~$ fontist --help
Commands:
  fontist cache SUBCOMMAND ...ARGS       # Manage fontist cache
  fontist config SUBCOMMAND ...ARGS      # Manage fontist config
  fontist create-formula URL             # Create a new formula with fonts from URL
  fontist fontconfig SUBCOMMAND ...ARGS  # Manage fontconfig
  fontist help [COMMAND]                 # Describe available commands or one specific command
  fontist import SUBCOMMAND ...ARGS      # Manage imports
  fontist install FONT                   # Install font
  fontist list [FONT]                    # List installation status of FONT or fonts in fontist
  fontist manifest-install MANIFEST      # Install fonts from MANIFEST (yaml)
  fontist manifest-locations MANIFEST    # Get locations of fonts from MANIFEST (yaml)
  fontist rebuild-index                  # Rebuild formula index (used by formulas maintainers)
  fontist repo SUBCOMMAND ...ARGS        # Manage custom repositories
  fontist status [FONT]                  # Show paths of FONT or all fonts
  fontist uninstall/remove FONT          # Uninstall font by font or formula
  fontist update                         # Update formulas

Options:
     [--preferred-family], [--no-preferred-family]  # Use Preferred Family when available
  q, [--quiet], [--no-quiet]                        # Hide all messages
  v, [--verbose], [--no-verbose]                    # Print debug messages
  c, [--no-cache], [--no-no-cache]                  # Avoid using cache during download
     [--formulas-path=FORMULAS_PATH]                # Path to formulas

jcbhmr@PIG-2016:~$ echo $?
1

Should exit code 1 and print help text withe error message:
fontist not-a-commmand
fontist blahhhhhhhhhhh
fontist ---------------------wassd

what happens right now: ✅👍
jcbhmr@PIG-2016:~$ fontist not-a-commmand
Could not find command "not-a-commmand".
jcbhmr@PIG-2016:~$ echo $?
1

im just drawing this "it should probably work this way" from my own intuition from using other clis. for concrete guides or documents there's articles like https://blog.developer.atlassian.com/10-design-principles-for-delightful-clis/ or https://clig.dev/#the-basics

@jcbhmr agree.
Currently fontist help also exits with 1, which should be 0.

I think we could use this module:

https://mattbrictson.com/blog/fixing-thor-cli-behavior


Improve the design of the Team section

Hello, I want to create a more attractive design for the team section including adding background color, icons and hover effect. Please assign this issue to me as a GSSOC-Exted contributor.

@vishanurag i would love to work on this one. please assign me this under the hacktoberfest tag


Fix bugs introduced by IBL fixes

Fixes issues created by https://github.com/godotengine/godot/pull/33668
Fixes: https://github.com/godotengine/godot/issues/33771

Thanks!


supFindUserAssocSet not working anymore on 19041

Hey there,
I recently applied some updates on my 19041 install and supFindUserAssocSet cannot find UserAssocSet_19041 pattern anymore.
After some researches, I figured out that shell32.dll got updated and that UserAssocSet_19042 pattern work just fine if used.
As such, I think that the code could be updated to try to find the other pattern as well if it cannot be found on 19041.

Hello, which exact version of Windows you have  (ver command) and that shell32 version too?

Hello, which exact version of Windows you have (ver command) and that shell32 version too?

19041.685 for Windows and 19041.662 for shell32.dll

Okay I will look on this.

Thanks for your work again and happy new year ^^

Thanks for your work again and happy new year ^^


Update cats-effect-laws to 3.3.13

Updates org.typelevel:cats-effect-laws from 2.5.5 to 3.3.13.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Applied Scalafix Migrations

org.typelevel:{cats-effect,cats-effect-laws}:3.0.0

github:typelevel/cats-effect/v3_0_0?sha=v3.0.0

org.typelevel:{cats-effect,cats-effect-laws}:3.3.0 (created no change)

github:typelevel/cats-effect/v3_3_0?sha=series/3.x

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

labels: test-library-update, early-semver-major, semver-spec-major, scalafix-migrations, commit-count:n:2

Superseded by #272.


intiial commit for query pipeline changes

Default document query execution context now used for

ChangeFeed queries
single partition queries (partitionkey, partitionKeyRangeId is present in options)
Non document queries

Errors out for aggregates on multiple partitions, with enable cross partition query set to true, but no "value" keyword present

Hits query plan endpoint for other scenarios to fetch query plan

Extended query info class for these functionalities

Fixed bug in routing map provider

Removed PoxyQueryExecutionContext and MultiCollectionQueryExecutionContext as they are no longer needed

Can one of the admins verify this patch?

Thanks for making this change :)


Bug in v1.1.0 : direction in subgraph generate TypeError: Cannot read properties of null (reading 'namespace')

Hi,
I've updated the library from 1.0.2 to 1.1.0 and an issue appeared when I build my code.
Expected Behavior
client compiled successfully
Current Behavior
TypeError: Cannot read properties of null (reading 'namespace')
at async Promise.all (index 0)
client compiled with 1 error
Possible Solution
After investigation and some testing, I managed to make it works by removing the direction line under subgraph.
Steps to Reproduce

project running on remark-mermaid-dataurl v1.1.0 & docusaurus v2
In mermaid md tag :

flowchart TB
subgraph subDummy
    direction TB
    foo[foo] -. can be .-> bar
end

Hi @pleyo, thanks for reporting this issue!
It should be fixed in remark-mermaid-dataurl 1.1.1. You may also need to run npm upgrade remark-mermaid-dataurl @mermaid-js/mermaid-cli to make sure that mermaid-cli is at least v8.10.2.
In version 1.1.0, I explicitly limited @mermaid-js/mermaid-cli version to be <=8.10.1,
since that upgrades to puppeteer v10.0.0, which drops Node v10 support.
Unfortunately, that version of mermaid-cli uses Mermaid v8.10.1, which doesn't support the direction option in flowchart subgraphs (that was only added in Mermaid 8.10.2).
However, it looks like although Node v10 is not officially supported by mermaid-cli 8.10.2+, it still actually works, so I've converted @mermaid-js/mermaid-cli to be ^8.9.2, like it was previously!


Docker Troubleshooting Tutorials - Review

Docker Orientation for VRX Competitors
Is Rocker used anywhere in the tutorials apart from the install?
Creating a Competitor Image

[x] Image Creation Overview

[x] Preparation
[x] Interactive process using docker commit
[x] Scripted process using a Dockerfile

[ ] Minimal Working Examples

[ ] VRX 2022 Starter Image
Still on ROS1
No visuals?

Troubleshooting trial runs

[ ] Troubleshooting Overview
[ ] Troubleshooting Prerequisites
[ ] Get basic debugging information
[ ] Examine a running container
[ ] Run your container manually
[ ] Run a trial manually

@M1chaelM I see, I got a little mixed up going through the link to the dockwater repo.

Code naming convention examples for "Run a trial manually" updated.


distutils was removed in Python 3.12

See https://peps.python.org/pep-0632/
This is blocking applications that depend on azure-identity to upgrade to Python 3.12.

Hey!
Please accept my apologies, but since there hasn't been any response or progress on this topic I'll now start to make some noise in the upstream azure-identity library. It has been known since a year (#118). I'm sorry about that.


Crash when open "incorrect" matroska.

Sample - https://yadi.sk/i/po7SrSNz3QKGBp
Quick fix - https://sourceforge.net/p/mpcbe/code/3161/

Thanks for sample file https://github.com/MediaArea/MediaInfoLib/issues/671

@Aleksoid1978 please confirm that this is fixed in master branch.

Confirm.


chore: upgrade to docker v23 and add new mocked funcs

fixed by https://github.com/go-vela/worker/pull/535


Multi output fixes

This fixes how pointer test and workspace switching work to fix most of the multi output issues present in sway.

The style on the debug stuff is also abhorrent, but I'll rewrite it anyway.

Danke


docs(registry): Further specify owner-remove response (#12056)

What does this PR try to resolve?
This fixes a documentation issue as mentioned in #12056.
How should we test and review this PR?
This is just a markdown file change, so it does not break anything.
Additional information
Let me know if you have any changes or suggestions or how to write the tests for markdown files as well!
Thanks

Since this documents the reality of how both canonical sides behave (crates.io, cargo), going ahead with this.
@bors r+

:pushpin: Commit 0c535fc329843c4492e27e51e203108b54948d7d has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 0c535fc329843c4492e27e51e203108b54948d7d with merge<PHONE_NUMBER>5ff50c31c4f84316456d0597b77562...

@bors r-
There are some issues with this PR.

:broken_heart: Test failed - checks-actions

We should probably move the current comment above the ˋ"ok": trueˋ line and add one above the msg field to warn that it is unused. Changes to use it can follow later. @Bhardwaj-Himanshu

Thanks @2ndDerivative, added some more changes as recommended!

Thanks!
@bors r+

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


Error occurred while scanning, please try again

When I click on learn FR or IR command I'm getting this error: Error occurred while scanning, please try again
Device Type: 0x653c - RM4 Pro
Can you tell me what I should do?

When I click on learn FR or IR command I'm getting this error: Error occurred while scanning, please try again Device Type: 0x653c - RM4 Pro Can you tell me what I should do?

do you solve this?


Added ALC292 layout-id 18 for Dell Precision M4800/M6800 by vanquybn

Tested fine on my Dell M4800

Added manually.


add programmes sort functionality

Added sort functionality for programmes.
Pls review and give me feedback.

Changes done.
If still not OK with whitespace / spaces/ tabs I mean can u pls provide some code style config file ?
I am using IntelijIdea 2016.2 and I am using their default code style.

@ancazapuc9 Merged it, but we get a test failure -- it seems the descending sort by scheduled_start doesn't work. Does it actually pass for you? I might have to revert this otherwise.

Hello,
For me it worked, I will retest
Anca

On 5 Dec 2016, at 17:31, Emils Solmanis<EMAIL_ADDRESS>wrote:
@ancazapuc9 https://github.com/ancazapuc9 Merged it, but we get a test failure -- it seems the descending sort by scheduled_start doesn't work. Does it actually pass for you? I might have to revert this otherwise.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mbst/glycerin/pull/14#issuecomment-264884347, or mute the thread https://github.com/notifications/unsubscribe-auth/AGmHqhOuckCOwfZwYaVU6WuQoRhQoi0Eks5rFC5EgaJpZM4LAZlY.

Found the pb, u can review it

On 6 Dec 2016, at 10:10, Anca Zapuc<EMAIL_ADDRESS>wrote:
Hello,
For me it worked, I will retest
Anca

On 5 Dec 2016, at 17:31, Emils Solmanis<EMAIL_ADDRESS><EMAIL_ADDRESS>wrote:
@ancazapuc9 https://github.com/ancazapuc9 Merged it, but we get a test failure -- it seems the descending sort by scheduled_start doesn't work. Does it actually pass for you? I might have to revert this otherwise.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/mbst/glycerin/pull/14#issuecomment-264884347, or mute the thread https://github.com/notifications/unsubscribe-auth/AGmHqhOuckCOwfZwYaVU6WuQoRhQoi0Eks5rFC5EgaJpZM4LAZlY.


Syntax highlighting

What's an IDE without syntax highlighting? Lol.
Assigning myself to implement syntax highlighting using SyntaxTextAreaFX.

Thanks for working on this! Feel free to make a pull request whenever you're done.
Also, I didn't get any email or notification from GitHub about this issue for some reason. Any idea how to fix that? Do I have to be watching my own repository?

Notifications are broken across all platforms. It’s been a time.

On Mar 25, 2019, at 2:33 PM, Tomer Aberbach<EMAIL_ADDRESS>wrote:
Thanks for working on this! Feel free to make a pull request whenever you're done.
Also, I didn't get any email or notification from GitHub about this issue for some reason. Any idea how to fix that? Do I have to be watching my own repository?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/TomerAberbach/mano-simulator/issues/3#issuecomment-476324879, or mute the thread https://github.com/notifications/unsubscribe-auth/AIs1-ACbICOJmxmUMqII6HXctLbntXOcks5vaRaEgaJpZM4cFM87.

So does that mean that watching my own repository won't fix the issue?

No guarantee, no. Fun, right? Same thing happens when you add contributors. No notification to the user.

On Mar 25, 2019, at 2:37 PM, Tomer Aberbach<EMAIL_ADDRESS>wrote:
So does that mean that watching my own repository won't fix the issue?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/TomerAberbach/mano-simulator/issues/3#issuecomment-476326215, or mute the thread https://github.com/notifications/unsubscribe-auth/AIs1-M2ILTjJ0Junz7SNvzOXt9QPzKmMks5vaRdfgaJpZM4cFM87.

Well I guess I'll watch the repository and hope.


Add PolicyActive condition.

Updates the policy reconciliation loop to set the PolicyActive condition to true when the policy status is updated to "active".
Test
You can test this manually or using the e2e test repository.
If you want to test it using e2e tests. You need to follow some steps:
# upload the container image somewhere
# update the makefile setting the container name and tag to be used
 make create-k8s-cluster install-kubewarden basic-e2e-test

I would also take advantage of this change to move this block:
https://github.com/kubewarden/kubewarden-controller/blob/8c8434c2fa25a88edf559de9c2bee2c4465770a5/controllers/policy_utils.go#L120-L133
before this block:
https://github.com/kubewarden/kubewarden-controller/blob/8c8434c2fa25a88edf559de9c2bee2c4465770a5/controllers/policy_utils.go#L107-L118
so that we don't wait for "uniquely reachable" in order to install the webhooks. The main reason is that we don't perform any kind of follow up later regarding the status of the pods to remove the webhook (which is fine, and I don't think we should do it). So I think we should not check this before we install the webhook either.
As a user, I can do:

Install policy
Wait for policy condition PolicyActive if I want -- this means that webhooks are reconciled, my policy is "active" in that the apiserver is starting to send requests to the webhook

When I change a policy setting, whatever that is:

Change policy
Wait for policy condition IsUniquelyReachable if I want


Remove redundant props, reuse types from FlatListProps + ListViewProps, export BigListRenderItemInfo

This PR:

remove duplicated props already included in ScrollViewProps
reuse types from FlatListProps and ListViewProps. Why? To keep up to date with React Native and also receive hints/comments in IDE

Add missing prop: controlItemRender
Export BigListRenderItemInfo (similar to Flatlist's ListRenderItemInfo)
Thanks!

Hi, thank you for your pull request is much appreciated!
Merged and will be released soon.


Initial support for ruby1.9

I'm not pretty sure about 1.9/Dockerfile.rhel7. Could someone please review it?

I appreciate the contribution but we don't intend to support older versions of ruby (the current RHEL release ships with 2.0 so that is our baseline) from this repository.  Ruby 1.9 is clearly phasing out of the community(https://semaphoreci.com/blog/2015/12/01/ruby-version-usage-in-commercial-projects-2015-edition.html) and support is ending(https://www.ruby-lang.org/en/news/2014/01/10/ruby-1-9-3-will-end-on-2015/) and we only have the resources to test/validate so many images/versions.
You're welcome to fork and maintain your own ruby1.9 image of course.

Totally understood. We have clients that need 1.9, even knowing it's gonna die soon.
May you link to my repo in your README.md so others can use it?

@caruccio sure, where would you like the link to point?

Here is fine: https://github.com/getupcloud/sti-ruby/
Thanks.

pull for adding the link: https://github.com/openshift/sti-ruby/pull/92


[small][Fixed The link routing issue]

Hey, thanks for this.
Can we get a little more information about what the change in behavior is? The original behavior, how you fixed it, and the new behavior?
Additionally, it appears your change has broken the typescript linting and tests. Those failures will need to be addressed as well before this can be reviewed.

Hey, thanks for this.
Can we get a little more information about what the change in behavior is? The original behavior, how you fixed it, and the new behavior?
Additionally, it appears your change has broken the typescript linting and tests. Those failures will need to be addressed as well before this can be reviewed.

Hey @jayofdoom!
@kindyluv has been working on #1144.  This is an issue focused on when someone clicks an annotation that is a URL, it will open a new tab in your browser.
I suggested that she create a simple POC using just a dummy react application so we can verify that this works without all the armada nonsense.
She has created https://github.com/kindyluv/LinkRouting which is the POC for testing this out.  I was able to build and test and it does in fact create a browser in a new window!

That's super cool, thanks for letting me know Kevin!


accuracy calculation is wrong

the accuracy function forget to take mean value of the matches of examples.

According to unit test, it seems to be designed to produce element-wise accuracy.


Dynamic confirm / preview screen where quotes are refreshed

See uniswap for example https://app.uniswap.org/swap

click review and wait a bit for it to refresh

quotes update on expiry even within the preview screen
@twblack88 / prodcut to take a first pass at this to make sure we dont want to modify the UI and how we can signal this to the user.
must be snappy! uniswap example again

After some discussion with engineering there may be some confusion, but to be absolutely certain commenting here to clear it up.
Quotes / rates should never refresh after a user has clicked the "Confirm and Trade" button.  There should be no "blind" updating of quotes, even if expired in the app. It is better to have a trade fail on chain than execute a swap during big market movements that ends up with unexpected pricing to the user.

Technical flow:
Input Stage (shows all available routes) [no wallet required]

Polling rates for all routes with best effort network fee estimation

Preview Stage (polling route picked) - [currently confirm screen] [no wallet required]

Still polling rate for single route with best effort network fee estimation
User clicks continue button

Button is Confirm if the wallet is connected -> go to Execution Screen
Button is Connect Wallet if no wallet is connected -> Button is Confirm -> go to Execution Screen

Execution Stage

If approval needed, rates continue to poll until approval is completed
If no approval is needed or approval has been completed, quotes start polling with accurate network fee estimation keeping the quote "fresh"
Sign and Broadcast Button (isLoading/Disabled while fetching new quote)

On button click - quote and network fees are locked in and polling stops, ensuring what is displayed on screen is as close to final on chain intention as possible

@shapeshift/product is thinking this through a bit and @NeOMakinG brought up that the swapper page is a bit bare. We're proposing a Landing stage. Its basically the swapper with marketing stuff beneath it pre-configured.
As a start, a dumb route with a more Superficial page to show functionality. Shows swapper the cwapper cycling between predefined quotes/pairs before inputs two ways to hit inputs:

cta to "swap/get started".
f you input a quote then it ingests that quote params. Boom you're dropped into input stage.

if this works we could also consider bringing that to the .com domain as an iframe but probably ahead of ourselves.

to fix your trouble check this solution click
maybe this will solve your problem.


Unable to specify an output directory with esbuild-plugin-meta-url

Hi!  The esbuild-plugin-meta-url is quite useful when moving from webpack to esbuild.  Unfortunately it seems to default to a behaviour of reproducing the relative path to the referenced files, and trying to recreate that path, where webpack would have renamed them (in some configurable fashion) and placed them in the output directory or a configurable assets directory.
The reason this is desireable is that, well, first, it avoids having weirdly long paths embedded in your bundle in the  case where the files came from somewhere deep in node_modules, but second, if the files happen to come from somewhere outside the working directory (as will be the case when using npm link), the plugin will build a directory somewhere else, possibly a place you don't want it to, or where you don't even have write permission.  For example, if you have index.js:
const img = new URL('../foo/logo.png', import.meta.url).href;
console.log(`URL is ${img}`);

and build.mjs:
import esbuild from 'esbuild';
import metaUrlPlugin from '@chialab/esbuild-plugin-meta-url';

await esbuild.build({
  entryPoints: ["index.js"],
  bundle: true,
  outdir: "dist",
  format: "esm",
  plugins: [
    metaUrlPlugin(),
  ],
});

Then, on running build.js, you will create these files:
$ node build.mjs
$ find . -mmin -1
./dist/index.js
./foo/logo.png

That's probably not what you want.  This may seem kind of contrived, but as mentioned earlier, it will happen in the case where you use npm link, and in any case it would be nice to be able to collect all your assets in a folder called, perhaps, dist/assets.

Hello @dhdaines! Thanks for reporting!
Could you check if the latest version of the plugin correctly saves the file in the output directory?

Hi! Sorry for the very late reply!  The latest version seemed to have the same behaviour when I tried it ... two months ago (sorry again) ... not sure if there has been another one since then?  I will have some time today or tomorrow to recheck this.


Add attention variant FastAttention from Performer

Self-note, need to create a requirements-gpu.txt file and maybe a "setup" directory with a install_reqs_gpu.sh script.
Just want to prevent developers from having to go back and forth when trying to guess deps.


Bug/3815/evaluation nan shown as number

Fixes #3815
Fixes #3838

We should show an error-icon if success is false. Additionally on hover there should appear an overlay, which shows the error-message

I re-worked the view. Now there is an icon shown, the error message is in an overlay like with the violated criteria. Additionally the criteria is shown, but only informative. There is no error styling applied there.
According the heatmap, I would propose to keep the "grey" state as semantically the evaluation neither failed or passed.


feat(ripple): initial mdInkRipple implementation

LGTM aside from a few minor comments. I would be good if one of @robertmesserle, @kara, or @hansl could also do a pass, though.

Found a few typos and the alpha version needs correcting, but once those are fixed, LGTM.

Looking forward to this being merged.

LGTM aside from the removing of "ink".


How to normalize the DMS_score

I noticed that there are some assays with very large labels, so I am wondering how to normalize the DMS_score.
Thanks.

Dear Jixian,
Assuming your question is within the context of supervised training on ProteinGym DMS assays, we typically standard scale DMS_score when training the various supervised baselines reported. In the zero-shot setting, all performance metrics are unaffected by scale.
Best,
Pascal

Dear Pascal,  following up on the previous question:

Since the range of the DMS_score varies across experiments, how is the cutoff defined for DMS_score_bin ?

Thanks, great work with the repo !
Antoine

Dear Antoine,
Thank you! We manually examine the DMS scores distribution and if it is nicely bi-modal we select the natural cutoff point as the threshold, otherwise we set it to the median value. The reference file specifies which approach is used in the DMS_binarization_method field.
Best,
Pascal

Dear Jixian,
I'm closing this issue as I believe the above addressed it. Please feel free to re-open if you have any further questions on this.
Best,
Pascal


Show in Explorer Issue ~ Atom 0.153.0 / Windows8.1 x64

Show in Esplorer open my User directory not the folder of my project.
sorry for my bad english.

i find an update on the atom.io the chocolatery version are not updated this resolve my issue.


Weather through Yahoo's API

This is just a placeholder to put progress there. I applied a few weeks back to get access to Yahoo!'s Weather API, and (quite understandably, after a bit of delay), got the approval today for Aerial !
I'll track progress here on the implementation of a new Weather info in Aerial.

Quick update, it's still not ready but getting there :

Should have a build ready soonish

Weather???  That would be cool!

So here's a first build, this is still to be improved but it's mostly usable:

There's a new Weather info, that's disabled by default. You have to enable it first. Then you need to type your City name at the bottom. After typing, press enter to validate the field. Then press the test button, you should see a new window pop after a bit :

You can play a bit with the textfield if needed. The lookup is done by Yahoo and usually returns something even if you type gibberish. At a later point I'll add an option to use Core Location to get your location, but that's not in this beta, and that path will likely crash if you change the setting from Manually Specify so don't ;)

You should get a box with the current conditions for your picked location. There's a lot more information I get from Yahoo! Weather, including some forecasts. I'll see how I can add that too as an option in a further build.
beta1 here : https://github.com/JohnCoates/Aerial/releases/download/v.1.8.3beta1/Aerial.saver.zip

I mistakenly updated the wrong version yesterday, so here's beta 2 with weather, sorry for those who tried beta1 : https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta2/Aerial.saver.zip

FWIW, I'm only getting Celsius vs. Fahrenheit and only for Cupertino, CA, as opposed to my location in neighboring Sunnyvale, CA.

FWIW, I'm only getting Celsius vs. Fahrenheit

I added the switch but forgot to code it 🙈 It's fixed for next beta !

and only for Cupertino, CA, as opposed to my location in neighboring Sunnyvale, CA.

Some cities didn't work it seem, including Sunnyvale, I think I fixed that too.
I should have a new beta with the fixes in a couple of hours, thanks for the report !

@r8ders2k This version includes the fixes I talked about above, let me know how that works : https://github.com/JohnCoates/Aerial/releases/download/v.1.8.3beta3/Aerial.saver.zip

@glouel It appears to be working, as Yahoo says it's 62º F and Aerial is also showing 62º F.  Thanks!  Aerial just keeping getting better and better!

This is amazing; thanks!
One bit of UI feedback - it wasn't clear to me that I needed to press enter after typing in my location. I entered a location, immediately hit 'Test', and, upon seeing location information for Paris (the placeholder / suggested text in the location field), I thought it was not working until I came here and saw your instructions.
Now, granted, I am a very dumb person, so take this feedback with a grain of salt ... and thanks again for this great new feature!

This is amazing; thanks!
One bit of UI feedback - it wasn't clear to me that I needed to press enter after typing in my location. I entered a location, immediately hit 'Test', and, upon seeing location information for Paris (the placeholder / suggested text in the location field), I thought it was not working until I came here and saw your instructions.
Now, granted, I am a very dumb person, so take this feedback with a grain of salt ... and thanks again for this great new feature!

Hi @SnarkNerd
Oh don't worry it's not you ! For some reason that's the default way Textfields work in Xcode, I had the same issue with other ones (like when you need to input a message etc) and solved it there back then (although I didn't remember how and just put the beta up without fixing it in a hurry ;)).
I'll fix it for the next beta because it's just not intuitive at all !

@SnarkNerd I fixed the textfield thing so it should be a lot more intuitive now. New version also fix the vertical alignment of the icon and the text so it looks a bit better.
https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta5/Aerial.saver.zip

@glouel thanks so much for following up on that! Wasn't necessary to inform me specifically, but I appreciate it. Thanks for making this great product; it's one of my favorite things about my Mac!

Thanks for your great work @glouel
I was wondering if there are flat weather icons available.
Noticing the sun is shining on the screensaver, although we are at night her in Germany right now. ;)

Yahoo Weather is showing this icon for Frankfurt am Main, DE
Icon<EMAIL_ADDRESS>
Any chance to customize this?
Thanks again for the time you are investing and your great work you do for the community.

@DJay-X
I'm using the icons that Yahoo provide as part of the API, as there are about 40 needed (way more than I could find in font awesome or something like that as there are so many "conditions" that they can return). Those icons are different than those that they use for their own site though (which are flat indeed and not approved for 3rd party use as far as I understand).
But ! They do provide night icons, which I'm not displaying on this beta right now. So in that case that would be this instead of your sun icon : https://s.yimg.com/zz/combo?a/i/us/nws/weather/gr/32n.png
This should be in next beta ;)

Just to add that Apple do provide official weather icons in SF Text:

which also came with all range of weights — from ultra thin to black. If we’re ever adopting flat icons, I’d say these could be the first choice.

I think those are Catalina only ? Could be an extra option, there seems to be many, although I'm not sure there are enough to map to every single one from Yahoo, there's a list here : https://developer.yahoo.com/weather/documentation.html#codes
There are actually 48 condition codes, I don't know if there's enough but worth having a look.

@rschiang
Quick update, SF Symbols looked super cool... except, of course, it's not even available on macOS, not even Catalina 🙈
You can export to SVG, but that's not natively supported either, so you need a lib just for that, or convert the SVGs to PNG. I'll try to see if there are other sets out there already, but this is disappointing that macOS was ignored again for this API.

@DJay-X @rschiang
I spent the whole afternoon on this, but I found a nice tool here to export SF Symbols to PDF : https://github.com/davedelong/sfsymbols
Took a while too to map the icons but there was enough to get it all mapped correctly to Yahoo's stuff, so here's how it look right now:

I've added a new option so you can choose which icon set you want, I put those as default as they look cooler in my opinion (and don't require a https request):

Couple of things, your Weather settings will be reset, and the symbols seem to be pretty thin. The tool I used above was supposed to export the symbols as "regular" but it does look thin to me. I'm getting to like it like that, but it maybe could have used a bit more weight (sadly with no native SF Symbols support I need to export the whole thing so I can't change it on the fly).
Also you should get night icons now too on both sets. Let me know how that goes:
https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta6/Aerial.saver.zip

Yay! You did it! Great job. And thank you so much. You have fulfilled my wish.
I agree the symbol is not that crisp, maybe because it is thin. But that's high-level whining. ;)
It look good and it fits perfect to the all-in-all design with the flat symbols.
Maybe you refine it over time but I see no existential need right now. You are fantastic and made my day. Thank you again. 😌🙏

AWESOME!

(I think my day/night detection algorithm may be a bit off. It was off by one hour here, I'm not sure how it is on other locales. 1hr after expected, I do get the correct night icons here)

https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta7/Aerial.saver.zip
There was a crash with some sunny conditions, as I had misnamed an icon in the code... This version fixes it, sorry about that :)

When I activate weather, the screen goes blank after a fraction of a second. I don't know what happened but it used to work on beta6. I attached today's part of the log (The whole thing is too big)
Many thanks
Hans
AerialLog.txt

Hi @hansw1204
So what you describe looks like a crash. Could you launch Console and look for a crash log ? It's usually called legacyScreensaver, or something that sounds like one.
Also, can you try pushing the test button from the UI, see if that crashes too ? That may be a quicker way to generate a crash log too.
Sorry about that in any case, hopefully I can fix it quickly !

I cannot find a Crash Log. When I try and test it in Preferences I get a hang and have to force quit Preferences.

After that, I cannot get into Aerial Sceensaver Options anymore and have to re-install a previous version.

Hmm. Could you attach your preference file here so I can give it a shot ?
It's /Users/glouel/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/com.JohnCoates.Aerial.plist

Here it is (renamed because otherwise not selctable)
com.JohnCoates.Aerial.plist.txt

Ok I'm super sorry, I gave you the wrong path :(
Could you give me ~/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Preferences/ByHost/com.JohnCoates.Aerial.{UUID}.plist instead ? That's the real preferences file for Aerial, sorry about the mixup I messed up !

No problem
com.JohnCoates.Aerial.FB891E04-96D2-5B69-81B2-DC1CF3CAF691.plist.txt

Ok, I tried it here and sadly I couldn't make it crash (it's possible the weather condition changed or something like that).
If you're willing to test again, I'm wondering since it seems you were using the color icons, could you try using the flat ones see if that crashes too ? The flat icons should be more stable right now I think in beta 7.
I'll give it another shot tomorrow see if I can reproduce again at a different time.

Same with flat icons. No rush. The weather is just icing on the cake

@hansw1204 and just to be clear, beta6 works fine for you ?

I tried but now it is not working either. I suspect it has nothing to do with Aerial but something on my system. I’ll check security things and firewalls etc.

I tried but now it is not working either. I suspect it has nothing to do with Aerial but something on my system. I’ll check security things and firewalls etc.

@hansw1204 I was thinking about your issue this morning, especially when  you mentioned that you had to force quit System Preferences after pressing test, so I did a quick check with no network. If you try the test button with no network, or the full screensaver with weather enabled, Aerial does not hang here, it fails gracefully (it doesn't show an error in the test thing, it should, I'll see about adding that) and just doesn't show the weather info at all.
So there are a couple of ways it could hang in my opinion:

you have an application level firewall (something like little snitch) that keeps the network connection open but doesn't let it go through (until you accept/deny). That may hang Aerial
you have a router level firewall that does that kind of thing (seems a bit weird)
you have a pi-hole or some equivalent dns filtering, mayyybe that could cause that too ?

I'd suggest double checking whatever firewall you have, FYI, the API call to yahoo is at this address : https://weather-ydn-yql.media.yahoo.com/forecastrss

Thanks, I'll check that. I was also able to get a crash report now (see attached)
ScreenSaverEngineCrash.txt

Thanks ! I would also suggest making sure in Updates, that you have all the auto updates disabled, just in case. That could also be what your firewall flags.
The crash log is intriguing but I have a hard time making sense of it. Let me know how it goes.

Checked with security measures off and it's the same thing. No worries, though, I can live without the weather display

Thanks for following up. Just to be clear, because your crash log gave me some doubts, the betas work as long as you don’t enable Weather right ?

@hansw1204 if you are willing I put up a new beta, I tried adding more debugging, and possibly a fix for some condition I didn't properly checked for. Let me know if it works better, I would suggest if you can, try the test button, and if it crash, send me the last few lines of the debugLog !
To all, I made the flat icons crispier, they are still the same thinness, which is regular. I'm still unsure on trying thicker icons, I did make a script to generate the icons though so while it still take a bit of effort, I can regenerate new icons easily.

Thank you so much Guillaume! 1.8.3b8 works now!
cheers,
Hans

Woo !
Could you please send me the log of pressing the test button ? I think that you'll always get day icons right now with my "fix", I'd love to understand how I messed up :) I think I probably have a locale issue with the date conversion, need to make sure I get it right thanks !

Here you go. I also attached a screenshot of the Test pop-up.
AerialLog.txt

Sorry, missed your earlier post. Yes, b7 worked as long as weather was not enabled.

Great ! I think I see what happens, you won't get night time icons until the next beta, again many thanks for sticking with the testing, that was super helpful to me.

Thank you for such an amazing utility and your stellar support! It's the only screensaver I use. Simply brilliant!

@glouel Love the new Icon. Looks perfect I think.
However maybe something to notice, here in Germany it's 21:42 and I geht the sun/cloudy icon. :)
Thanks again.

Log30042020_2142.txt

Hi @DJay-X !
I think my night calculation is off (on top of crashing earlier), I know it's off by one hour here somehow. You had sunset at 8h45 according to your log. Could you tell me if you have night right now or still not ?
I need to review the whole night thing, looks like I got it pretty wrong 🙈 Will try to have a look tomorrow thanks for the report !

I'll check this eventing at sunset and an hour after. I'll let you know. 😌 Thanks for your passion for perfection. 😉

@DJay-X please try again with beta9, I'm 99% sure I fixed it. Basically it was wrong because of daylight savings time. Icons should also be much sharper and there's a new icon battery indicator.
@hansw1204 could you give beta9 a try and attach the log from pressing the test button ? That would be helpful because I can't seem to make sense of the specific issue you had with night detection. Thanks !
https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta9/Aerial.saver.zip

Here you go.
AerialLog.txt

Here you go.
AerialLog.txt

That was quick, thanks a lot ! Looks like you still won't have night yet with this version and I'm even more puzzled.
Could I ask you if there's anything special with your Locale settings on your Mac ? Are you using default US settings or did you tweak things with hour format or something like that ?
I'll keep thinking about it, it's... weird. You are using Catalina by the way, correct ?

Ah, sorry, maybe it's because I use this option for time:

@hansw1204 That should be fine, because I'm not actually using those right now, I'm using the sunrise/sunset values that I get from Yahoo (because I can't always count on having sunrise/sunset from your location). I still haven't implemented the "Use current" location option available in Weather, so only the "Manually specify" works. And it looks like you are using it.
I was thinking more about your Language & Region settings in macOS's System Preference, those :

Did you tweak something in that panel ?

The only thing I have different is that I use Monday as the first day of the week instead of Sunday.
However, I have not selected "Manual Specific" but "Calculate From" for the sun set/sun rise determination

The only thing I have different is that I use Monday as the first day of the week instead of Sunday.
However, I have not selected "Manual Specific" but "Calculate From" for the sun set/sun rise determination

Sorry, I wasn't clear, whatever is in Aerial's Time tab should be irrelevant, I doubled checked and your settings here are fine, they are independent of what's happening.
What I meant with manually specify are the Weather settings, like this :

Please make sure you are on Manually Specify there :)
I also tried the monday thing but no luck, I have a couple more ideas, I'll think about it and get back to you ! Thanks !

Yes, on the Weather Tab I do have "Manually Specify" selected. Sorry about the confusion.

@glouel Now Aerial isn't starting at all. 🤷🏼♂ A Video is just to see for less than a second and then the screen is dark. Can't even open Screensaver-Settings.

@glouel Now Aerial isn't starting at all. 🤷🏼♂ A Video is just to see for less than a second and then the screen is dark. Can't even open Screensaver-Settings.

 Could you go into Console.app and look for a crash log (although those are not super useful since Catalina) ? Sorry about that, must have missed something again...

Hi. This morning, the screensaver is working again. But still couldn't open the settings.  The Debug from there is of course not showing a lot now. Sorry.
Seems to be something with "after sunset" or a special condition yesterday night? 🤔
The console log didn't show anything when I type legacyScreensaver.
Catalina, yes.
AerialLog.txt

@DJay-X that log might be super helpful, I'll try something here and let you know.
Regarding console, if you can still replicate, I would suggest opening console.app first, erase, then filter by "legacyScreen" (it's sometimes abbreviated to "legacyScreenSave" for some reason). Then launch SysPref, it should fill up way before it crashes. Thanks !

I will try this evening to reproduce the error and check the console log.
I don't have battery indicator enabled. iMac late 2012 here.

I will try this evening to reproduce the error and check the console log.
I don't have battery indicator enabled. iMac late 2012 here.

Thanks for your answer. I'm trying to harden pretty much everything I could think of right now, I'll have a new beta that hopefully will fail gracefully and not crash. Should take an hour or so, will ping you back.
iMac 2012 doesn't have retina so in the early betas the PDF icons probably looked pretty bad. I ended up booting my old 2011 MBA here and the weather icon look decent on it's non retina screen (the temperature looks less sharp than the icon somehow, not sure if it's in my head). The Yahoo logo looks terrible though as I'm using the retina version of the logo all the time ! I'll put that on my list.

@DJay-X I've hardened pretty much everything I saw in Weather, so hopefully you won't crash anymore and things will work smoothly from now on. You may see a wrench or a red square if something goes really wrong. I haven't fixed the yahoo logo though on non retina :)
@hansw1204 when you have some time, could you try again the Test button and attach the log ? I tried two new ideas, just in case.
https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta10/Aerial.saver.zip

Hi glouel,
I'm live in Fuzhou, Fujian, China. I enter true but still cannot get city info.

Fuzhou, Fujian, China

Hi @cxy5014 !
I just checked here and it works with "Fuzhou", "Fuzhou, China" and "Fuzhou, CN". So my guess is that maybe Yahoo's API is blocked in China. Any chance you could attach the debuglog ? This might give some information but I'm afraid the API may be blocked :(

01:15:13 : avInit .saver (0.0, 0.0, 296.0, 184.0) p: true o: true
01:15:13 : <AerialView: 0x7f867050d910> AerialView setup init (V1.8.3beta10) preview: true
01:15:13 : Time Management initialized
01:15:13 : Display Detection initialized
01:15:13 : Display Detection
01:15:13 : 1 display(s) detected
01:15:13 : 1cm = 49.21259916429594 points
01:15:13 : npass: dict [__C.NSDeviceDescriptionKey(_rawValue: NSScreenNumber):<PHONE_NUMBER>, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceIsScreen): YES, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceResolution): NSSize: {72, 72}, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceSize): NSSize: {1366, 768}, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceColorSpaceName): NSCalibratedRGBColorSpace, __C.NSDeviceDescriptionKey(_rawValue: NSDeviceBitsPerSample): 8]
01:15:13 :        bottomLeftFrame (0.0, 0.0, 1366.0, 768.0)
01:15:13 : src orig : (0.0, 0.0)
01:15:13 : left 0.0 below 0.0
01:15:13 : [id=2034333696, width=1366, height=768, bottomLeftFrame=(0.0, 0.0, 1366.0, 768.0), topRightCorner=(1366.0, 768.0), isMain=true, backingScaleFactor=1.0]
01:15:13 : (0.0, 0.0, 1366.0, 768.0)
01:15:13 : Display Detection Done
01:15:13 : Using : nil
01:15:13 : <AerialView: 0x7f867050d910> setting up player layer with bounds/frame: (0.0, 0.0, 296.0, 184.0) / (0.0, 0.0, 296.0, 184.0)
01:15:13 : Using dot workaround for video driver corruption
01:15:13 : Manifest init
01:15:13 : app support exists
01:15:13 : No customvideos.json at : /Users/XXX/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application Support/Aerial/customvideos.json
01:15:13 : isManifestCached 10 true
01:15:13 : isManifestCached 11 true
01:15:13 : isManifestCached 12 true
01:15:13 : isManifestCached 13 true
01:15:13 : 2020-05-02
01:15:13 : Interval : Optional(-90913.96394300461)
01:15:13 : No need to check for new videos
01:15:13 : Manifests files were not loaded in class
01:15:13 : Files were not already loaded in memory
01:15:13 : Manifests are cached on disk, loading
01:15:13 : Poi Strings Provider initialized
01:15:13 : Preferred languages : ["en-US", "zh-Hans-US"]
01:15:13 : Best matched language : en
01:15:13 : Defaulting to bundle
01:15:13 : path : /Library/Screen Savers/Aerial.saver/Contents/Resources/en.json
01:15:13 : Community JSON : 99 entries
01:15:13 : Using exiting customCacheDirectory : /Users/XXX/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application Support/Aerial
01:15:13 : cache to be used : Optional("/Users/XXX/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application Support/Aerial")
01:15:14 : Total videos processed : 99 callbacks : 0
01:15:14 : <AerialView: 0x7f867050d910> Setting player for all player layers in []
01:15:14 : initvideocache
01:15:15 : cached video file with length: 214607394
01:15:15 : <AerialView: 0x7f867050d910> playing video (OFFLINE MODE) : file:///Users/XXX
/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application%20Support/Aerial/comp_GMT026_363A_103NC_E1027_KOREA_JAPAN_NIGHT_v18_SDR_PS_20180907_SDR_2K_AVC.mov
01:15:15 : <AerialView: 0x7f867050d910> observing current item <AVPlayerItem: 0x600000fdd1c0, asset = <AVURLAsset: 0x600000dfc280, URL = file:///Users/bill/Library/Containers/com.apple.ScreenSaver.Engine.legacyScreenSaver/Data/Library/Application%20Support/Aerial/comp_GMT026_363A_103NC_E1027_KOREA_JAPAN_NIGHT_v18_SDR_PS_20180907_SDR_2K_AVC.mov>>
01:15:15 : <AerialView: 0x7f867050d910> observeValue Optional("readyForDisplay") false (0.0, 0.0, 296.0, 184.0)
01:15:15 : AssetLoaderDelegate deinit
01:15:15 : <AerialView: 0x7f867050d910> backing change 1.0 isDisabled: false frame: (0.0, 0.0, 296.0, 184.0) preview: true
01:15:15 : <AerialView: 0x7f867050d910> observeValue Optional("readyForDisplay") true (0.0, 0.0, 296.0, 184.0)
01:15:15 : start playback: (0.0, 0.0, 296.0, 184.0) (0.0, 0.0, 296.0, 184.0)
01:15:16 : <AerialView: 0x7f867050d910> startAnimation frame (0.0, 0.0, 296.0, 184.0) bounds (0.0, 0.0, 296.0, 184.0)
01:15:16 : restarting playback
01:15:16 : <AerialView: 0x7f867050d910> stopAnimation
01:15:16 : pwc init2
01:15:17 : Callback after manifest loading
01:15:17 : Looking for outdated files
01:15:17 : No old files found
01:15:18 : Aerial control panel V1.8.3beta10
01:15:18 : Cache size : 26.61 GB
01:15:18 : cvcinit2
01:15:18 : cvcawake
01:15:18 : wdl
01:15:43 : === YW: Starting locationMode

I already have an proxy enabled, but it seem not work.

@DJay-X soooo, trying the china location, I ended up finding exactly what your crash was. Good news is, beta10 would have fixed it, I had forgotten to include some night icons (cloud moon for example). But somehow some other icons that I had added weren't there. So after removing and readding everything, and testing them all, I'm 99.999% sure that every icon is here now in beta 11. I hate releasing 2 versions in a day but I think it's worth it in that case :
https://github.com/JohnCoates/Aerial/releases/download/v1.8.3beta11/Aerial.saver.zip
@cxy5014 reading your log there's one thing I can think of, please make sure you choose manually specify where I circled in red :

This may be the issue, the "Use current location" you picked doesn't work right now.

https://caiyunapp.com

@cxy5014 on top of what I said above, please try this URL : https://weather-ydn-yql.media.yahoo.com/forecastrss?location=sunnyvale,ca
Let me know what it says, it should say something like:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<yahoo:error xmlns:yahoo="http://www.yahooapis.com/v1/base.rng" xml:lang="en-US" yahoo:uri="http://yahoo.com">
yahoo:description
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:description>
yahoo:detail
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:detail>
</yahoo:error>

@DJay-X soooo, trying the china location, I ended up finding exactly what your crash was.

Wasn't me with the china location. I'm still in Germany.
But hey.. it's a small world after all. ;)
I was the one with the crash after sunset or at night on iMac late 2012.
With beta10 there was no crash anymore for now. Log attached. Maybe it gives you some help somehow anyway.
Now I download beta11 and will see if there is something else unexpected. But I have a good feeling. Icons look good in my older iMac. 👍
console_legacyScreen.txt
Debug_10.txt

Wasn't me with the china location. I'm still in Germany.
But hey.. it's a small world after all. ;)
I was the one with the crash after sunset or at night on iMac late 2012.

Yeah I know ! It's just that after trying that specific location from China, I ended up finding the missing icons (which I thought were there), and that very likely was the cause of your crashes yesterday ;) It was missing night cloud and night cloud+rain, you possibly had one of those conditions yesterday in Germany I guess.

With beta10 there was no crash anymore for now. Log attached. Maybe it gives you some help somehow anyway.
Now I download beta11 and will see if there is something else unexpected. But I have a good feeling. Icons look good in my older iMac. 👍

Awesome ! Finally getting there !

@cxy5014 on top of what I said above, please try this URL : https://weather-ydn-yql.media.yahoo.com/forecastrss?location=sunnyvale,ca
Let me know what it says, it should say something like:
This XML file does not appear to have any style information associated with it. The document tree is shown below.
<yahoo:error xmlns:yahoo="http://www.yahooapis.com/v1/base.rng" xml:lang="en-US" yahoo:uri="http://yahoo.com">
yahoo:description
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:description>
yahoo:detail
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:detail>
</yahoo:error>
Edit: It's also possible that your proxy work but Aerial ignores it. I need to have a further look at this, I had conflicting report about this.

This XML file does not appear to have any style information associated with it. The document tree is shown below.
<yahoo:error xmlns:yahoo="http://www.yahooapis.com/v1/base.rng" xml:lang="en-US" yahoo:uri="http://yahoo.com">
yahoo:description
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:description>
yahoo:detail
Please provide valid credentials. OAuth oauth_problem="OST_OAUTH_PARAMETER_ABSENT_ERROR", realm="yahooapis.com"
</yahoo:detail>
</yahoo:error>
This is my safari respond.

@cxy5014 Ok so that looks good (the credential thing is normal). I will add some more debugging info in the next build, to try and understand what happens on your case. I'll get back to you, but probably not before tomorrow.

Can I choose the API for my weather? I want to use a faster and more accurate one, in China.

@zcccccc1 right now I only support Yahoo (as it's the only "free" one that worked worldwide that I could find that allowed apps, although it has issues in China apparently).
If you know one that's free/fast for China (and accessible outside/hopefully documented in English so I can implement it at least), let me know and I'll see what I can do.

To all, I finally released 1.9.0 with weather. There are a couple of changes/polish that I made last minute :

The alignments of the icon blocks are now fixed, putting weather on the left side of the screen was also not particularly nice in betas. Things correctly align with each other as intended (this is true also for the battery indicator).

I've added the ability to use macOS location services instead of manually specifying a city, for the nomads out there. For this to work, you need both Location Services enabled on your mac, and give Aerial the permission to use it. Pressing the test button will take care of it, in Catalina you'll get a prompt to allow "legacyScreenSaver" to access your location. Location is checked with macOS at Aerial's launch.

For privacy reason, I've rounded up to 2 decimals the coordinates sent to Yahoo's API, which leaves a margin of error of 1km approximately. I think that's enough (this weather API doesn't have a very precise granularity from what I saw) but I'm willing to change that if needed.
I'll be closing this issue, feel free to open a new one if you find a bug, of if you have found another weather API that could be implemented for specific cases (China seems to be the biggest issue so far).
1.9.0 : https://github.com/JohnCoates/Aerial/releases/download/v1.9.0/Aerial.saver.zip
For the curious, I'm planning a few more things for 2.0.0 that I hope to get out before WWDC in a month or so. The focus will be mostly on the UI to enable/disable videos, and cache limits (with better defaults). I'll open another issue soon when I have a beta out ;)
Hopefully at WWDC we'll get a better screensaver API to fix all the issues introduced with Catalina !

Sorry for opening this up again, but is there a way to remove this annoying "powered by Yahoo" logo?

Sorry for opening this up again, but is there a way to remove this annoying "powered by Yahoo" logo?

Sorry, that's not possible, it's part of their attribution guidelines to be able to use the API, that's why it's in the preference panel too. I have no choice but to abide on that one to be able to use it. I made it as small as I could ;)
https://developer.yahoo.com/attribution/


Register a custom container host bind entry for Aspire containers

Rather than depending on host.docker.internal or host.podman.internal (which may not even be present in all containers), Aspire can bind its own custom domain to refer to the local host machine from the container. This change adds a new host.aspire.internal entry to containers created by Aspire and updates our default hostname logic to use that, regardless of the specific container runtime.
I've tested this with Docker Desktop and Podman Desktop and everything continues to work as expected (including host.docker.internal on Docker Desktop and host.containers.internal on Podman). The command being used is supported on docker-ce as well, so this should work for Linux users who wouldn't otherwise have a host.docker.internal entry.
Microsoft Reviewers: Open in CodeFlow

Inspiration for this came from comments on #4191

Did a bit of testing with docker-ce on WSL; it looks like in that scenario it has the same issue we've run into with podman, where it can't access ports on the local machine that are bound to <IP_ADDRESS>; it seems that only ports bound to <IP_ADDRESS> from the local machine are accessible. This does still improve the experience with Docker Desktop/Podman Desktop.

I like the idea of normalizing this for Aspire so we don't have to special-case Podman and Docker Desktop everywhere. I think we need to think about our DNS / networking story all up in the not too distant future.
For example should we ever get to the situation where each app host instance has its own network, or multiple networks to simulate network isolation? If that is the case would we want the host DNS name to be more dynamic (I don't know ... I'm just asking the question).

For containers, custom Docker networks (not the default bridge network) support some real cool features regarding DNS, since Docker acts as a DNS server for the individual containers. You're able to bind containers to a network and assign them specific DNS that will resolve to the container IP automatically.
For Windows processes, it gets a bit uglier, since DNS is a system level configuration. You either have to register a custom DNS server (which causes network issues on the host if your server isn't running) or you have to elevate to edit the hosts file to add/remove entries.

This feels like it should be handled by dcp. Dcp runs the container with the a custom host name and tells aspire what the host name is.

But! I see the appeal. Hmmmm

The proxy we have is a tcp proxy, very low level (bytes). Sure it could feed us data to use but I'm not sure it solves this issue.

Yeah that won't give us much value in terms of diagnosability.

I think this is an important work item for 9

Would love love love to see full reverse proxying be built into aspire. I see it as somewhat mandatory to complete the experience. You can't properly test things like CORS during development without real-like domain names and HTTPS.
The icing on the cake would be running some kind of built in DNS that I could then configure as an upstream DNS on my machine. Easier to do on Linux as dnsmasq is within reach. Windows might take a little more strain, but I'm sure you have people you can speak to for help 😉

We have to be a little careful about things that require DNS configuration changes on the workstation as developers sometimes work in environments where that isn't possible.

To be fair those issues always have existed. We just haven't plastered over them yet ;)

Would love love love to see full reverse proxying be built into aspire. I see it as somewhat mandatory to complete the experience. You can't properly test things like CORS during development without real-like domain names and HTTPS.
The icing on the cake would be running some kind of built in DNS that I could then configure as an upstream DNS on my machine. Easier to do on Linux as dnsmasq is within reach. Windows might take a little more strain, but I'm sure you have people you can speak to for help 😉

We have to be a little careful about things that require DNS configuration changes on the workstation as developers sometimes work in environments where that isn't possible.

@mitchdenny @atrauzzi  Take a look at #5508

Replacing this with #5584