[BUG] Sprintrace result is missing data

Describe the issue:
Some data is not visible when getting the results of the sprint race. Q1, Q2 and Q3 are obviously logical, but I am missing values for: Position, Gridpositions, Time, Status and Points:
Reproduce the code example:
>>> ff1.Cache.enable_cache('cache')
>>> year = 2022
>>> race = 'Imola'
>>> type = 'S'
>>> session = ff1.get_session(year,race,type)
>>> session.load()
core           INFO     Loading data for Emilia Romagna Grand Prix - Sprint [v2.2.4]
api            INFO     Using cached data for driver_info
api            INFO     Using cached data for timing_data
api            INFO     Using cached data for timing_app_data
core           INFO     Processing timing data...
api            INFO     Using cached data for session_status_data
core        WARNING     No lap data for driver 24
api            INFO     Using cached data for track_status_data
api            INFO     Using cached data for car_data
api            INFO     Using cached data for position_data
api            INFO     Using cached data for weather_data
core           INFO     Finished loading data for 20 drivers: ['1', '6', '10', '22', '18', '24', '44', '47', '63', '55', '5', '77', '11', '3', '14', '20', '4', '16', '31', '23']
>>> session.results
   DriverNumber BroadcastName Abbreviation         TeamName TeamColor  FirstName    LastName          FullName  Position  GridPosition  Q1  Q2  Q3 Time Status  Points
1             1  M VERSTAPPEN          VER  Red Bull Racing    1e5bc6        Max  Verstappen    Max Verstappen       0.0           0.0 NaT NaT NaT  NaT            0.0
6             6      N LATIFI          LAT         Williams    37bedd   Nicholas      Latifi   Nicholas Latifi       0.0           0.0 NaT NaT NaT  NaT            0.0
10           10       P GASLY          GAS       AlphaTauri    4e7c9b     Pierre       Gasly      Pierre Gasly       0.0           0.0 NaT NaT NaT  NaT            0.0
22           22     Y TSUNODA          TSU       AlphaTauri    4e7c9b       Yuki     Tsunoda      Yuki Tsunoda       0.0           0.0 NaT NaT NaT  NaT            0.0
18           18      L STROLL          STR     Aston Martin    2d826d      Lance      Stroll      Lance Stroll       0.0           0.0 NaT NaT NaT  NaT            0.0
24           24        G ZHOU          ZHO       Alfa Romeo    b12039     Guanyu        Zhou       Guanyu Zhou       0.0           0.0 NaT NaT NaT  NaT            0.0
44           44    L HAMILTON          HAM         Mercedes    6cd3bf      Lewis    Hamilton    Lewis Hamilton       0.0           0.0 NaT NaT NaT  NaT            0.0
47           47  M SCHUMACHER          MSC     Haas F1 Team    b6babd       Mick  Schumacher   Mick Schumacher       0.0           0.0 NaT NaT NaT  NaT            0.0
63           63     G RUSSELL          RUS         Mercedes    6cd3bf     George     Russell    George Russell       0.0           0.0 NaT NaT NaT  NaT            0.0
55           55       C SAINZ          SAI          Ferrari    ed1c24     Carlos       Sainz      Carlos Sainz       0.0           0.0 NaT NaT NaT  NaT            0.0
5             5      S VETTEL          VET     Aston Martin    2d826d  Sebastian      Vettel  Sebastian Vettel       0.0           0.0 NaT NaT NaT  NaT            0.0
77           77      V BOTTAS          BOT       Alfa Romeo    b12039   Valtteri      Bottas   Valtteri Bottas       0.0           0.0 NaT NaT NaT  NaT            0.0
11           11       S PEREZ          PER  Red Bull Racing    1e5bc6     Sergio       Perez      Sergio Perez       0.0           0.0 NaT NaT NaT  NaT            0.0
3             3   D RICCIARDO          RIC          McLaren    f58020     Daniel   Ricciardo  Daniel Ricciardo       0.0           0.0 NaT NaT NaT  NaT            0.0
14           14      F ALONSO          ALO           Alpine    2293d1   Fernando      Alonso   Fernando Alonso       0.0           0.0 NaT NaT NaT  NaT            0.0
20           20   K MAGNUSSEN          MAG     Haas F1 Team    b6babd      Kevin   Magnussen   Kevin Magnussen       0.0           0.0 NaT NaT NaT  NaT            0.0
4             4      L NORRIS          NOR          McLaren    f58020      Lando      Norris      Lando Norris       0.0           0.0 NaT NaT NaT  NaT            0.0
16           16     C LECLERC          LEC          Ferrari    ed1c24    Charles     Leclerc   Charles Leclerc       0.0           0.0 NaT NaT NaT  NaT            0.0
31           31        E OCON          OCO           Alpine    2293d1    Esteban        Ocon      Esteban Ocon       0.0           0.0 NaT NaT NaT  NaT            0.0
23           23       A ALBON          ALB         Williams    37bedd  Alexander       Albon   Alexander Albon       0.0           0.0 NaT NaT NaT  NaT            0.0

Error message:
none

Already fixed in master (44014c54b5c0a2d096ba7beb1cb1c862360bf3ea). Will be in the next release then.


Interpolate_na: Rework 'limit' argument documentation/implementation

What is your issue?
Currently, the 'limit' argument of interpolate_na shows some counterintuitive/undocumented behaviour.
Take the following example:
import xarray  as xr
import numpy as np
n=np.nan
da=xr.DataArray([n, n, n, 4, 5, n ,n ,n], dims=["y"])
da.interpolate_na('y', limit=1, fill_value='extrapolate')

This will produce the following result:
array([ 1., nan, nan,  4.,  5.,  6., nan, nan])

Two things are surprising, in my opinion:

The interpolated value 1 at the beginning is far from any of the given values
The filling is done only towards the 'right'. This asymmetric behaviour is not mentioned in the documentation.

Comparison to pandas
Similar behaviour can be created using pandas with the following arguments:
da=xr.DataArray([n, n, n, 4, 5, n ,n ,n], dims=["y"])
dap=da.to_pandas()
dap.interpolate(method='slinear', limit=1, limit_direction='forward', fill_value='extrapolate')

Output
y
0    NaN
1    NaN
2    NaN
3    4.0
4    5.0
5    6.0
6    NaN
7    NaN
dtype: float64

This is equivalent to the current xarray behaviour, except there is no 1 at the beginning.
Cause
Currently, the fill mask in xarray is implemented using a rolling window operation, where values outside the array are assumed to be valid (therefore the 1). See xarray.core.missing._get_valid_fill_mask
Possible Solutions
Boundary Issue
Concerning the 1 at the beginning: I think this should be considered a bug. It is likely not what you would expect if you specify a limit. As stated, pandas does not create it as well.
Asymmetric Filling
Concerning the asymmetric filling, I see two options:

No changes to the code, but mention in the documentation that (effectively), a forward-fill is done.
Make something similar to what pandas is doing. In pandas, there are two additional arguments controlling the limit behaviour: limit_direction is controlling the fill direction (left, right or both). limit_area effectively controls if we only do interpolation or allow for extrapolation as well.

What do you think?

Two comments if the pandas arguments are introduced:

If the pandas arguments are introduced, one might raise the question, whether the max_gap argument of xarray is still necessary, since pandas does not have such an argument. However, if the user does not want any interpolation if a gap is bigger than a specified length, this is currently not possible with pandas (actually, the max_gap feature is requested by the pandas community since a long time: pandas-dev/pandas#12187 )
Coordinate handling: Currently, limit is operating on indices, max_gap is operating on coordinates. Is there a deeper reason for this separation? Both operations limit the interpolation 'distance'. If not, similar to use_coordinates, one could introduce something like limit_use_coordinates. If True, limit as well as max_gap refer to numeric coordinates, otherwise to a linearly increasing index ([0,1,2,...]).

Do you have any thoughts on this?
I think with the following signature, the function will be backward compatible:
interpolate_na(dim, method='linear', use_coordinate=True, limit=None, limit_direction='forward', limit_area=None, limit_use_coordinate=False, max_gap=None)

Thanks for raising this issue @Ockenfuss and thoroughly documenting the behavior.

The interpolated value 1 at the beginning is far from any of the given values

Agreed that is surprising and seems like a bug!

The filling is done only towards the 'right'. This asymmetric behaviour is not mentioned in the documentation

A pull request would be most welcome @Ockenfuss - scoped to either just improve the documentation or with your preferred API change. We can discuss the preferred syntax in the pull request.

Thanks for the feedback @scottyhq! I used the time over christmas and tried to come up with an implementation, see #8577! As you suggested, I pointed out specific details to discuss in the PR.

After the Xarray meeting today (with @keewis and @dcherian) we discussed the preferred option for moving forward with cleaning up the limit argument. The concensus was to introduce a new keyword argument (e.g., cutoff) with the preferred default handling of limit, and to soft deprecate the old limit argument.

Some more results from the discussion:

This affects all filling methods: interpolate_na, fillna, ffill, bfill, ...
We want two functionalities: limit (or cutoff) to limit filling distance, max_gap (or ?) to avoid any filling
Both should work on coordinates or index
The mask is a valuable product by itself
We prefer a new API to avoid breaking changes

Here are my proposals:
Masking function Signature:
namask('t', limit=3, limit_coords=3h, max_gap=3, max_gap_coords=3h) #in line with pandas/current xarray
namask('t', limit=3, cutoff=3h, gap_limit=3, gap_cutoff=3h) #cutoff for coordinates, limit for index

For direction control:
namask(limit_direction=forward/backward/both, limit_area=outside/inside/None) #regardless of index or coordinate

Possible API:
Making it an object:
da.namask(...) #masking object
da.namask(...).fillna(123) #return dataarray
da.namask(...).mask #get mask only

Making it a method:
mask=da.namask(...)
da.fillna(123).where(~mask) #Invalid=True to match numpy behaviour?

How to deal with existing methods:

da.fillna(123) shall still work
If any exsiting limit, max_gap is used: Internally forward to new API, raise Deprecation Warning (but possibly support for very long/forever)

Anything else:
This almost feels like introducing numpy.ma in xarray...
This is beyond the scope of this Issue! But we may copy some of the behaviour from numpy, e.g. invalid values are True there.


RTL support for floating placeholder?

i changed the text alignment with isLTRLanguage = false or .textallignment = .right
both work for the the actual place holder but not the floating one, how can i manually change the alignment of the floating text?

This should be fixed in 3.1.0

Hi
I am facing the same problem.
Actually what i need is. I need my application in RTL. I have changed the appearance of my application from RTL.
Now i am using textfield with SkyFloatingLabelTextField,  and set its textAllignment to .right.
But when i use to run my app from Xcode it is showing the floating placeholder at the right.
and if i use to run app from device then it makes the floating placeholder at left position.
Please give me the reason.
Thanks

Thanks
I found the solution.


Problem with selected items

Hi,
i have some Problem with multiple tables. Hier ist my page:
[http://<IP_ADDRESS>/#/]
I have five tables. When I select an order (tr item), as well as the other tables show an item as selected. Can you helf me, please?
Login<EMAIL_ADDRESS>Passwort: stampmedia
My controller:
...
$scope.states = [     {name: "open", icon: "history"}     {name: "checking", icon: "comment-check-outline"}     {name: "processing", icon: "settings"}     {name: "shipping", icon: "truck"}     {name: "archived", icon: "archive"}   ]   $scope.selected = []
...
I tried to create a foreach loop, unfortunately without success(coffee script):
  selected = {} for key of $scope.states     selected[key] = [ ] $scope.selected = selected
and view:
`

{{orders[state.name].length}}

...
...
...

{{order.counter}}
{{order.Job_number}}
{{order.user.name}}
{{order.updated_at|date:'dd.MM.yyyy HH:mm:ss'}}

        <md-data-table-pagination md-limit="ordersPerPage" md-page="ordersPageNum" md-total="{{orders[state.name].length}}"></md-data-table-pagination>
        <md-progress-linear ng-if="loading" md-mode="indeterminate"></md-progress-linear>
        <md-progress-linear ng-if="progress" md-mode="determinate" value="{{progress}}"></md-progress-linear>
      </md-content>
    </md-tab-body>
  </md-tab>
</md-tabs>

`

I think you are using an old version of the data table that was known to have scoping issues. Can you update to a more recent version? The current version is v0.10.3, I think you are using v0.8.x.

Thank you for your message. I'll try it. Thanks again!


Using Already-Installed 7-Zip

I already have 7-Zip installed on a machine that I would like to install Scoop on. Is it possible to have Scoop use the version of 7-Zip that I already have installed?

scoop config 7ZIPEXTRACT_USE_EXTERNAL $true

Thanks. Does it automatically detect the path to 7-Zip, or do I need to supply that as well?

You must be add 7-Zip install path to system environment path.


Fix icon examples

Merge animated and non-animated icons into the same example
animatedIcons.py could be used to help testing different kind of animation class, but i remove it
It is there on the history, in case.

Closes #1623

There is not much changes in the icons.py one. I will remove the other one.

PR updated


Using multiple configurations confuses the sync engine

Templates from friend metadata are treated as desyncs by the sync engine. Generation is not affected so this is to some extent a cosmetic issue.

This has been fixed


[SPARK-29342][SQL] Make casting of string values to intervals case insensitive

What changes were proposed in this pull request?
In the PR, I propose to pass the Pattern.CASE_INSENSITIVE flag while compiling interval patterns in CalendarInterval. This makes casting string values to intervals case insensitive and tolerant to case of the interval, year(s), month(s), week(s), day(s), hour(s), minute(s), second(s), millisecond(s) and microsecond(s).
Also  I removed the fromCaseInsensitiveString method from CalendarInterval, and replaced its usage by regular fromString().
Why are the changes needed?
There are at least 2 reasons:

To maintain feature parity with PostgreSQL which is not sensitive to case:

 # select cast('10 Days' as INTERVAL);
 interval
----------
 10 days
(1 row)

Spark is tolerant to case of interval literals. Case insensitivity in casting should be convenient for Spark users.

spark-sql> SELECT INTERVAL 1 YEAR 1 WEEK;
interval 1 years 1 weeks

Does this PR introduce any user-facing change?
Yes, current implementation produces NULL for interval, year, ... microsecond that are not in lower case.
Before:
spark-sql> select cast('INTERVAL 10 DAYS' as INTERVAL);
NULL

After:
spark-sql> select cast('interval 10 days' as INTERVAL);
interval 1 weeks 3 days

How was this patch tested?

by new tests in CalendarIntervalSuite.java
new test in CastSuite

Test build #111729 has started for PR 26010 at commit 27b2bcd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16719/
Test PASSed.

Test build #111730 has started for PR 26010 at commit 47fbb22.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16720/
Test PASSed.

If we can make it sure that the negative test cases are covered (or not needed), this PR looks promising. Do you expect performance improvement, too?

fromCaseInsensitiveString is not only case insensitive but it is tolerant to missing of interval at the beginning of the input comparing to fromString(). I am looking to how to make the interval word optional in the pattern.

Test build #111729 has finished for PR 26010 at commit 27b2bcd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111729/
Test FAILed.

Test build #111730 has finished for PR 26010 at commit 47fbb22.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111730/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16725/
Test PASSed.

Test build #111736 has started for PR 26010 at commit f570749.

Test build #111737 has started for PR 26010 at commit cd679fc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16726/
Test PASSed.

Test build #111736 has finished for PR 26010 at commit f570749.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111736/
Test PASSed.

Test build #111737 has finished for PR 26010 at commit cd679fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111737/
Test PASSed.

@dongjoon-hyun I have restored the fromCaseInsensitiveString" function because it is not only case insensitive. Please, take a look at this.

ping @dongjoon-hyun

Retest this please.

Test build #111811 has started for PR 26010 at commit cd679fc.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16791/
Test PASSed.

Test build #111811 has finished for PR 26010 at commit cd679fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111811/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16794/
Test PASSed.

Test build #111814 has started for PR 26010 at commit c7e09a1.

Test build #111814 has finished for PR 26010 at commit c7e09a1.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111814/
Test FAILed.

jenkins, retest this, please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/16795/
Test PASSed.

Test build #111815 has started for PR 26010 at commit c7e09a1.

Test build #111815 has finished for PR 26010 at commit c7e09a1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/111815/
Test PASSed.

I removed the following from the PR description.
Also I renamed the fromCaseInsensitiveString method to fromStringWithOptionalPrefix because the difference between fromStringWithOptionalPrefix and fromString is in tolerance to missing of the interval prefix.


Pass along TLS info for Loggregator

[#106137298]
Signed-off-by: Nino Khodabandeh<EMAIL_ADDRESS>

Thanks, @fraenkel! Prioritizing.

Closing as a new pull request is coming.


[ca] Fix spurious CA server log messages

Only log messages about updating root CA in CA server if it's not the first time a cluster object has been set.
Fixes #2131

LGTM


Integrate El-Vira

From<EMAIL_ADDRESS>on September 12, 2011 13:39:22
Either as groovy code bundled as jar, OR port to 100% java
<a href="https://code.google.com/p/el-vira/" rel="nofollow">https://code.google.com/p/el-vira/

Add to Oort such that there is a release "foo-el.owl" for every ontology foo
Original issue: http://code.google.com/p/owltools/issues/detail?id=12

From<EMAIL_ADDRESS>on September 13, 2011 15:28:56

Ported the relevant part of el-vira to Java (~10 lines of code)
Added optional EL filter to the release pipeline and reasoner
Added GUI option

Status: Started

From<EMAIL_ADDRESS>on September 15, 2011 12:59:06
added test cases for ontology id and deprecation

From<EMAIL_ADDRESS>on September 22, 2011 14:26:25
keep this open as reminder: el-vira removes all deprecation annotations (datatype boolean is not EL)

From<EMAIL_ADDRESS>on October 22, 2013 13:26:20
Status: WontFix


Loading Key Vault values is attempted even if ConfigureKeyVault isn't called

Issue originally brought up in the docs repo: https://github.com/MicrosoftDocs/azure-docs/issues/49293
Repro:

Setup an App Config instance that contains a Key Vault reference.
Configure App Configuration without calling ConfigureKeyVault()

[assembly: FunctionsStartup(typeof(FunctionApp.Startup))]
namespace FunctionApp
{
    public class Startup : FunctionsStartup
    {
        public override void Configure(IFunctionsHostBuilder builder)
        {
            var config = new ConfigurationBuilder();

            config.AddEnvironmentVariables();
            TokenCredential credential = new DefaultAzureCredential();
            var endpoint = new Uri(Environment.GetEnvironmentVariable("AppConfigEndpoint"));

            //Setup Azure App Config
            config.AddAzureAppConfiguration(options =>
            {
                options.Connect(endpoint, credential);
            });

            builder.Services.Replace(
                ServiceDescriptor.Singleton(typeof(IConfiguration),
                config.Build()));
        }
    }
}

Expected Behavior: App Config provider won't try to load any results from Key Vault.
Actual Behavior: "Microsoft.Extensions.Configuration.AzureAppConfiguration.KeyVaultReferenceException: 'No key vault credential configured and no matching secret client could be found."
This is the same result as when ConfigureKeyVault() is called with a null credential. This means that if there are any Key Vault references in an App Config instance, all the apps using it will need to pass credentials to ConfigureKeyVault() even if they won't be using Key Vault references or use Select() statements to try to avoid loading any.

Hello @SamaraSoucy-MSFT
The appearance of this error, even though ConfigureKeyVault is not called, is by design. The secrets that cause the error are in the configuration set that is being queried by the application. If the application is allowed to run when this occurs it results in what we consider to be an undefined state since there is no clear expectation for what should be returned when one of the secrets is accessed by the app.
We can think more about this scenario to figure out a way that makes sense to limit secret resolution.
In the mean time, name-spacing settings with a prefix or placing settings into separate labels would be a way to solve this problem. This is also a best practice approach for isolating settings that are needed and not needed by the application.
As an example, the application can select only settings that start with "myApp/". Since the application depends on all these settings there shouldn't be any secrets, or if there are secrets it should cause the app to crash if it can't resolve them.
            config.AddAzureAppConfiguration(options =>
            {
                options.Connect(endpoint, credential);

                options.Select("myApp/*", LabelFilter.Null);
            });

The original thread from the customer was that, while using using labels and prefixes is definitely the preferred way to go, in reality not every team is going to be organized enough to follow a specific naming or labeling convention. All it would take is one misplaced reference to bring down an app not using Key Vault credentials.
Maybe at some point we could consider adding a configuration option to block key vault loading or state that they want the reference, not the Key Vault contents? That covers the scenario the customer brought up of a precautionary measure, as well as any where someone intentionally wants the reference instead of loading the content from Key Vault. The Azure portal is an example of the second scenario- interacting with the contents of the App Config data as opposed to what is in Key Vault.

Hi @SamaraSoucy-MSFT, please see the proposed solution in #209 and let us know if you have any concerns.


Includes of raylib-cpp.hpp generate warnings for "unused-function"

Hi @RobLoach. I've recently repointed my submodule for raylib-cpp to latest and I've noticed that the include of raylib-cpp.hpp will generate three compiler warnings for each include statement in reference to the functions LoadFileText GetDirectoryFiles and GetDroppedFiles as below:

The issue can be solved quite trivially by applying the [[maybe_unused]] hint to either the beginning of the function declaration, or just above as so:
[[maybe_unused]]
RLCPPAPI std::string LoadFileText(const std::string& fileName) {
    char* text = ::LoadFileText(fileName.c_str());
    std::string output(text);
    ::UnloadFileText((unsigned char*)text);
    return output;
}

[[maybe_unused]]
RLCPPAPI std::vector<std::string> GetDirectoryFiles(const std::string& dirPath) {
    int count;
    char** files = ::GetDirectoryFiles(dirPath.c_str(), &count);
    std::vector<std::string> output(files, files + count);
    ::ClearDirectoryFiles();
    return output;
}

[[maybe_unused]]
RLCPPAPI std::vector<std::string> GetDroppedFiles() {
    if (!::IsFileDropped()) {
        return std::vector<std::string>();
    }
    int count;
    char** files = ::GetDroppedFiles(&count);
    std::vector<std::string> output(files, files + count);
    ::ClearDroppedFiles();
    return output;
}

Let me know your thoughts and I can put in a PR for it!

Thanks! Looks like a simple addition. Strange that it issues warnings for those. They are used in the examples :shrug:

Perhaps CMake doesn't have -Wall on by default? Either way, I'll submit a PR for it.

Thanks for the fix up!


Support for GitHub App authentication

GitHub apps are more robust in that they are capable of granting read-only access to an organization.
Authentication should be provided by environment variable:

GITHUB_APP_KEY -  A one-line base64 encoded String of a PKCS1 or PKCS8 PEM formatted RSA private key.
GITHUB_APP_FILE - The path to a file with contents PKCS1 or PKCS8 PEM formatted RSA private key.

In both cases, the private key is plain and not password protected.  The private key will be used for both an encrypted cache to help reduce API requests and to generate time-gated API tokens to interact with GitHub both for cloning code and for querying repositories.
Cloning repositories:

Username: x-access-token (literal)
Password: a token generated by the app (varies)

The following is a sample git askpass shell script.
#!/bin/bash
case "$1" in
    Username*)
        exec echo "${GIT_USERNAME:-}"
        ;;
    Password*)
        exec echo "${GIT_PASSWORD:-}"
    ;;
esac

Perhaps cloneable should support a git askpass mode.  For example,
mkdir -p ~/.local/share/cloneable
curl -Lo ~/.local/share/cloneable/cloneable.jar https://some-download-url
echo -e '#!/bin/sh\njava -jar "'"$HOME"'"/.local/share/cloneable/cloneable.jar --git-askpass "$@"' > ~/.local/share/cloneable/cloneable.sh
chmod a+rx ~/.local/share/cloneable/cloneable.sh
GIT_ASKPASS="$HOME/.local/share/cloneable/cloneable.sh"
export GIT_ASKPASS

There should be a CLI option which prints the password to stdout for GIT_ASKPASS so that the askpass script reuses cached information as needed.


Update fs2-core, fs2-io, ... to 2.5.7

Updates

co.fs2:fs2-core
co.fs2:fs2-io
co.fs2:fs2-reactive-streams

from 2.5.6 to 2.5.7.
GitHub Release Notes - Changelog - 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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "co.fs2" } ]

labels: library-update, semver-patch

Superseded by #230.


Im making a plugin

Since discord invitation doesnt works, forum is dead and the wiki doesnt have the answer to my question, im here.
Im trying to make a RaidablePlots plugin, i already know how to get the plot owner, but how do i remove the protection of a plot?

add * as helper

true, thanks


Initial check in for iam custom role template

Fixed.
From: Gus Baratto<EMAIL_ADDRESS>Sent: July 25, 2018 6:30 PM
To: sourced/deploymentmanager-samples<EMAIL_ADDRESS>Cc: Vincent Pun<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [sourced/deploymentmanager-samples] Initial check in for iam custom role template (#17)
@tiadobatima1 commented on this pull request.

In community/faas/tests/fixtures/configs/iam-custom-role.yamlhttps://github.com/sourced/deploymentmanager-samples/pull/17#discussion_r205281746:

@@ -0,0 +1,32 @@

+# Example on how to use the iam-custom-role template
+#
+# Variables:
+#   ORGANIZATION_ID: The organization ID of your organization.
FAAS_ORGANIZATION_ID
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sourced/deploymentmanager-samples/pull/17#pullrequestreview-140523360, or mute the threadhttps://github.com/notifications/unsubscribe-auth/And1orYIb5CHw6a3ONgko1oz9FTUCXC1ks5uKPFogaJpZM4Vgxqe.


net/http/httputil: ReverseProxy does not pass unannounced trailers

What version of Go are you using (go version)?
go version devel +2d20ded584 Sat May 20 00:45:56 2017 +0000 linux/amd64
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/tristan/src/go"
GORACE=""
GOROOT="/home/tristan/src/go"
GOTOOLDIR="/home/tristan/src/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build356611585=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
What did you do?
Attempt to use grpc-go via a httputil.ReverseProxy. grpc.Server does not pre-anounce its trailers. ReverseProxy only passes on pre-anounced trailers.
https://play.golang.org/p/Lt7QPq9vmX
What did you expect to see?
All trailers from upstream should be passed on
What did you see instead?
Only trailers set in the Trailer header are passed on

With the above patch, both helloworld and routeguide grpc-go example will work via httputil.ReverseProxy

I think this might be a breaking change and that's the behavior for any handler/handlefunc
in Go and not only for httputil.ReverseProxy, unless I am mistaken.
I defer to @bradfitz for his view.

The runtime has explicit support for unannounced trailers (marking the headers with the http.TraolerPrefix prefix. It would seem odd if it were expected behaviour for tReversdProxy to drop them. Perhaps I'm missing your point?


how to create a navigation like this

Hello, I would like to create a navigation like this:

I just want to link the content or url in dropdown menu, not all navigation item.But the plugin require me to fill the two fields, url and relation to content type.How can I modify them? Thanks so much

Hi @Tiramisupxl,
we've taken an assumption that navigation is going to be build based on the existing content types, so each item will contain relation to the content type (and might be nested down) or external link. You've to think about building it more like a site structure.
In your case you've to create a content type that will be related with "Navigation Three - Submenu" item and then add couple of a child elements to it. If you want to group them by some attribute, you can use also the Audience content type. Up to you how you're going to manage it in the client app.
Hope I've explained it a bit.

Hi @Tiramisupxl,
we've taken an assumption that navigation is going to be build based on the existing content types, so each item will contain relation to the content type (and might be nested down) or external link. You've to think about building it more like a site structure.
In your case you've to create a content type that will be related with "Navigation Three - Submenu" item and then add couple of a child elements to it. If you want to group them by some attribute, you can use also the Audience content type. Up to you how you're going to manage it in the client app.
Hope I've explained it a bit.

Thank you for your replay. Your answer tells me a lot about the plugin and strapi.  I have found the solution from a youtube video someone taught how to make the menu in strapi. Anyway, thanks a lot!


CI on GitHub-hosted dockerized platforms sometimes fail due to "No space left on device" error

Examples where error is shown in log:
https://github.com/kuzudb/kuzu/actions/runs/6489330683/job/17627731842#step:17:566
https://github.com/kuzudb/kuzu/actions/runs/6489330683/job/17627731490#step:17:352
Sometimes the error is only shown in summary:
https://github.com/kuzudb/kuzu/actions/runs/6489330683

I'd wager that we create too many intermediate objects (along with arrow) that causes us to use too much space. Why it would be inconsistent is a mystery to me.

I did some test locally and I think rust debug build is taking the most space and it seems to be expected (https://users.rust-lang.org/t/why-are-debug-symbols-so-huge/81117, https://users.rust-lang.org/t/running-out-of-disk-space/10641/3). I think if we run the rust build in release mode, we will be able to fix this.

Yeah, rust uses a lot of space. Why were we ever building in debug mode?

I have no idea. I think @benjaminwinger set this up on CI and @russell-liu simply copied the scripts to GitHub hosted CI.

The inconsistency seems be due to that GitHub-hosted runners may be dispatched to different Azure VMs: https://github.com/actions/runner-images/issues/2840#issuecomment-791177163

Sometimes, image can have ~15-20 GB of free space and this volume can be changed without notice. We only guarantee that images contain at least 14 GB of free space.

Yeah, rust uses a lot of space. Why were we ever building in debug mode?

Debug mode is often faster for rust. It also has sometimes ended up catching some debug-only issues like assertions which otherwise get ignored, though I think what we need to do there is rewrite the ASAN job to fail on any test failure, rather than just when there is output in the asan log.

Does ASAN run in debug mode? IIRC, no

Why is debug mode often faster? Faster to build or run?

Faster to build. Release builds are usually much slower by comparison for rust than they are for C++. Though I'm not sure what the difference would be like for kuzu off-hand.
Evidently you changed it recently, but prior to 9ebc7f1cf6624f288a the asan job was in debug mode. We need something in CI that's testing in debug mode; otherwise we should remove the asserts or turn them into runtime exceptions which occur in release mode too.

Ah. I didn't realize that my changes changed the behaviour of asan. This explains a lot.
I don't really think assertions impose much of a performance penalty, and would welcome a change to make them always on. We already have KU_ASSERT.

After a bunch more discussion, my plan is to make rust build in release mode and enable asserts in CI by using a FORCE_ASSERT environment variable. To do this, every assert will be replaced by KU_ASSERT, which will be enabled if !defined(NDEBUG) || defined(FORCE_ASSERT), so, it will be enabled if we force it on, or we are in debug mode.


#8690 use utc interval when create pending segments

Fixes #8690.
Description
use utc interval when create pedding segments

Hi @liutang123, would you please resolve the conflict?


Login Error unable to create site

Category

[ ] Question
[ ] Typo
[x ] Bug
[ ] Additional article idea

I'm unable to add the training solution to my tenant. I tried a few days ago and got the same error. The behavior is that after selecting add to Tenant it flips to an authentication screen then just site there and the screen flashes. If I back out I'm presented with the following error message.
There was an unexpected issue. Please try again after a while.
IDX21323: RequireNonce is '[PII is hidden]'. OpenIdConnectProtocolValidationContext.Nonce was null, OpenIdConnectProtocol.ValidatedIdToken.Payload.Nonce was not null. The nonce cannot be validated. If you don't need to check the nonce, set OpenIdConnectProtocolValidator.RequireNonce to 'false'. Note if a 'nonce' is found it will be evaluated.

I should note that I am a global admin in the tenant and we have two factor authentication turned on.

Effective June 2019, the official guidance is to post all issues related to the PnP provisioning service to the following GitHub repository: https://github.com/SharePoint/sp-provisioning-service.
We've transferred your issue to the issue list for that repo. In the future, please ensure you use the https://github.com/SharePoint/sp-provisioning-service repo's issue list for all issues, questions & bug reports.


Support Secrets

Terraform CLI and Provider Versions
Terraform v1.5.5
Snowflake-Labs/snowflake 0.86.0
Use Cases or Problem Statement
I'd like to be able to manage snowflake secrets with terraform. This is not an option today.
Proposal
create new resource:
snowflake_secret
and
data resource:
snowflake_secret
https://docs.snowflake.com/en/sql-reference/sql/create-secret
How much impact is this issue causing?
Medium
Additional Information
No response

Hey @brandocomando. Thanks for reaching out to us.
We have supporting the missing GA resources on our roadmap: https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#our-roadmap. We will visit this topic then.
For now, you can leverage https://registry.terraform.io/providers/Snowflake-Labs/snowflake/latest/docs/resources/unsafe_execute resource. This is a dangerous resource so make sure you read the docs and limitations before using.

Hey @brandocomando 👋
We have resources for secrets ready for deployment, so they will be a part of the next release, which should happen tomorrow or somewhere around next week.

Closing due to inactivity.


Allow new version of pyparsing

jira2markdown pins pyparsing to < 3.0.0. Many new tools uses pyparsing > 3.0.0 and therefore creates pip conflicts if jira2markdown  is used.
If there are no breaks, allow pyparsing > 3.0.0

Hi @twodrops, thank you for the issue. It's possible to update pyparsing to the latest version, but there are a many changes compared to version 2. I can have a look and try to find out what should be updated at weekends. You can do it too if you have time, I will appreciate it.

@twodrops please check https://github.com/catcombo/jira2markdown/pull/12 and tell me if it works fine for you now. Once you do it, I will publish the new version to PyPI.

@catcombo ah. I am sorry. I am not not using jira2markdown directly. I am using https://github.com/useblocks/sphinx-needs-enterprise/ which uses jira2markdown and pulls in an old version of pyparsing which is incompatible in my setup. @danwos could you please do some quick test of sphinx-needs-enterprise to see if the features you need from jira2markdown works with this update.

@danwos Thanks for testing. @catcombo Could you please make a new release?

@danwos Thank you for testing @twodrops I published a new version to PyPI. I cleaned up the code a bit before merging but it should break something. Fell free to open a new issue if you'll find a new problem.


mobileDisabled is not used

There is a parameter mobileDisabled but it's not used anywhere in the code. Or did I miss something?
The code and doc would probably require clarification in regards to that parameter and forceDesktopMode, to clearly explain how to make it work on mobile.
For instance the doc mentions that mobileDisableddefaults to false, while it's true in the code. But anyhow this parameter is not used.

Hi @gotson ,
mobileDisabled was deprecated times ago.
Sorry, I need to fix it in the doc...
Anyway, in mobile devices the library is disabled by default. You could set forceDesktopMode = true to activate also on mobile devices.
Thanks

Thanks, that's what I figured out from the code. You can keep this issue for updating the doc then 😉

Doc updated! 🤟


Update README.md

make it clear that scan_interval is the number of seconds. Fixes #11.

Thanks for your modification!


Neuroglancer color hashing often does not produce distinguishable colors

Neurons with a similar color can get locked into cycles where even after many successive presses of the l key, they remain indistinguishable. I suspect that correlations in root id that occur because of the structure of ids in the pychunkedgraph introduce these similarities after the color hashing. It would be useful if either some additional care went into hashing to produce more distinct colors (e.g. add a random per-neuron offset that scrambles some of the PCG-derived correlations) or exploits the direct setting of colors to use pre-generated distinguishable colors (For example, using the color space sampling from something like seaborn husl colors)

If anybody wants to investigate if this correlation is real:
The hash function is based on MurmurHash3. While not suitable for cryptographic purposes, it should be sufficiently random(?):
https://github.com/seung-lab/neuroglancer/blob/958d7a5e56a4791b4871c5a4b94e46291cb22dc9/src/neuroglancer/gpu_hash/hash_function.ts#L17-L32
The segment color is generated here - c0 determines the hue:
https://github.com/seung-lab/neuroglancer/blob/958d7a5e56a4791b4871c5a4b94e46291cb22dc9/src/neuroglancer/segment_color.ts#L77-L84
First step could be to compute the hue for a large number of nearby UInt64 and check if it's uniformly distributed
Second pick a different seed and check if the hue changes are also uniformly distributed
If nothing weird is going on, some more ideas to improve distinguishability:

Don't make hues uniform - less blues, more greens
Add random textures in shader (stealing this idea from VAST), i.e. stripes, circles and grids of various size and rotation

Thanks for putting this info together! I'll try to look for root ids that tend to fall into this pattern as well.

I still haven't tried to implement the color function, but a few pairs of ids that seem to flock together:
864691135946809441,<PHONE_NUMBER>70906504
864691135994717610,<PHONE_NUMBER>66561524
864691135233287257,<PHONE_NUMBER>01548109


JavaScript scheduler framework is not being displayed in a page.

There's no scheduler written in Blazor as of the moment that satisfy my requirement and open source or free of use. So I decided to use Syncfusion scheduler (JavaScript Essential 1) written in JavaScript.  So now, when I load a certain Blazor component it doesn't load. However, if this is placed inside the _Host.cshtml

It just load everything. I know this is Blazor intended for Single Page Application and this JS scheduler is not for SPA. But is there a way a can still load or call the Scheduler when the component is loaded?

Hello @jhunexjun ... this sounds more like an issue for the product unit to address. Open it on the engineering repo :point_right: https://github.com/aspnet/AspNetCore/issues ...but be sure to search open and closed issues before you open a new issue to make sure that someone else hasn't already asked.


Safari security prevent usage

Fixed.
One image's link was unsecured due to //site.io/page/image.png. Changed into o https://site.io/page/image.png.


dist folder missing in 3.4.8 npm release

Describe the bug
The dist folder is missing in the 3.4.8 release on npm.
To Reproduce
Steps to reproduce the behavior:

Go to the npm code page for miyagi
There is no dist folder

Expected behavior
The release includes the dist folder.

Thanks… 🙈 Fixed in https://github.com/miyagi-dev/miyagi/releases/tag/core%2Fv3.4.9.


Move expectJobWithSuspendedAndNodeSelectors to test and remove util.go

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Cleanup VS code reporting issue and move expectJobWithSuspendedAndNodeSelectors to the test file.
Which issue(s) this PR fixes:

Relates to #3480
Special notes for your reviewer:
Does this PR introduce a user-facing change?

NONE

/approve
LGTM, leaving final tagging to @mbobrovskyi in case of conflicts with the other PR

@mszadkow Please resolve conflicts.

/lgtm


Improve Documentation

Hi,
Thanks for this repo and your works but I think it's need more documentation. Do you have any plan about it?

@cyclops24 Can you explain more what you need? What kind of documentation?

I was about to open an issue this very same subject. These are some topics that must be better documented:

What each module is intended for?
How to build release apk files?
A simple overview of how the app interacts with RC server
Technologies this app depends on and links to easy getting started

@cyclops24 I'm not really concerned about code documentation at this point. If that's what you were looking for when opened this issue, I might open a new one to address my request.
Thanks!

@matheusjardimb that would be an awesome starting point

then it would be amazing if someone would documents every single package of the app in detail, and by detail I mean something like:
MODULE APP

service contains all the bla bla bla

inside service, the directory internal contains all te bla bla bla which uses this library and bla bla bla connected to bla bla bla

shared contains all the bla bla bla

... bla bla bla
MODULE PERSISTENCE REALM

bla bla bla

@matheusjardimb I agree with you 👍  . Some tutorial and topics (like titles that you mention above) would be a good starting point.

This is already in progress/done for version 2.0.0.


Bump to Python 3.10.7

Is there an existing issue for this?

[X] I have searched the existing issues

Code improvement description
We are on Python 3.10.5, and we need to bump to 3.10.7.

Actually, closing in favor of https://github.com/dependabot/dependabot-core/issues/5653


[6.5.x] GUVNOR-2636: [Decision Table Spreadsheet] uploading new version causes error

See https://issues.jboss.org/browse/GUVNOR-2636

Jenkins please retest this.

Sweet @karreiro It's a +1 from me.. (when green).


Static assets result in IdentityUI project to fail publish, when the root is not very short

As the IdentityUI has been updated to avail of the new Static Assets feature, the publishing of a project referencing it results in quite long paths being used, which fails the publishing. See https://devdiv.visualstudio.com/DevDiv/_workitems/edit/935535 for more details
REPRO STEPS
File > New Preject > ASP.NET Core Web Application > Next > Give a name "Anc30_RazorWithRCL_Core_Ind", browse a new localtion "C:\Users***\source\repos\OpenExisting\30_preview7_0621" > ASP.NET Core 3.0 ~ .NET Core ~ Web Application with IndAuth > Create
Right click the projrect > Publish > Create New > Select exisitgg group/plan > Create a Database with new or exisitng SQL Server > Publish
RESULT
Error Message:
Unable to copy file "C:\Users*.nuget\packages\microsoft.aspnetcore.identity.ui\3.0.0-preview7.19320.1\staticwebassets\V4\Identity\lib\jquery-validation\dist\jquery.validate.min.js" to "C:\Users*\source\repos\OpenExisting\30_preview7_0621\Anc30_RazorWithRCLib_Core_Ind\Anc30_RazorWithRCLib_Core_Ind\obj\Release\netcoreapp3.0\PubTmp\Out\wwwroot_content\microsoftaspnetcoreidentityui\V4\Identity\lib\jquery-validation\dist\jquery.validate.min.js". The specified path, file name, or both are too long. The fully qualified file name must be less than 260 characters, and the directory name must be less than 248 characters.

We need to disable static web assets completely on the Identity UI package completely or take in https://github.com/aspnet/AspNetCore/pull/11029 which properly moves Identity UI into the static web assets model. (Which we either do now or not do until 5.0)
Also, from what I can see the publish output folder might be wrong in this case, as its pointing out to wwwroot_content, which suggests we are missing a path separator there.

More generally, users might run into path too long exceptions in nuget packages, I'm not sure if ultimately the only solution is to enable long paths on the windows registry to allow paths longer than 260 chars.

I did the work of migrating this and I have a PR out, I only need to fix helix tests for it and get a review from either Pranav or Haok.

@javiercn, please link the PR here.

Looks like the long file-path support will eventually become standard:

Source: https://docs.microsoft.com/en-us/windows/desktop/fileio/naming-a-file
Unfortunately, we can't rely on this and require developers to switch to that version of windows. So fixing this is the right thing to do.

https://github.com/aspnet/AspNetCore/pull/11029

@mkArtakMSFT Ultimately we can't do anything if the package path exceeds 260 chars for any given path on the package, as this depends on the user nuget package root path. We can try and have short paths, but that's it.

Agree, our abilities are completely limited here.

Fixed in https://github.com/aspnet/AspNetCore/commit/143c10169326ea3972fc90950a938c33640777e2


tests/suites/helpers.function missing #include <strings.h>

Description

Type: Bug
Priority: Minor

Bug
In tests/suites/helpers.function, strings.h is included only if defined(__unix__) || (defined(__APPLE__) && defined(__MACH__)).
Some files in the test suite use strcasecmp(), which is declared in strings.h.
Thus the build fails on cross-compiling for some embedded platforms.
Shouldn't strings.h be included in all cases?

@monniaux thank you for your question.
s you can see in this post, strings.h is from the BSD branch, and is not necessarily portable. I agree though that if we want the tests to be portable, we should probably have a wrapper for strcasecmp, for platforms without strings.h and \ or strcasecmp functionality, therefore marking as a bug.

ARM Internal Ref: IOTSSL-2772


koord-descheduler: evict pod by perferred groupverison

Signed-off-by: LeoLiuYan<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Find out the server if support eviction subresource and use preferred groupversion.
Ⅱ. Does this pull request fix one issue?
NONE
Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[✅] I have written necessary docs and comments
[✅] I have added necessary unit tests and integration tests
[✅] All checks passed in make test

/lgtm
Thanks @LeoLiuYan

/retest

/lgtm

/milestone v1.1


selectable fields in logrt

Sometimes don't want body etc

Calling this good enough with body skipping in c067e5a


AWS Internet of Things

http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-topicrule.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-thingprincipalattachment.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-thing.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-policyprincipalattachment.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-policy.html
http://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-resource-iot-certificate.html

Thanks!


Fix initialization from incompatible pointer type error in re_dump_bytecode

JerryScript-DCO-1.0-Signed-off-by: Akos Kiss<EMAIL_ADDRESS>

With make LOG=ON the build fails with
../../../jerry-core/parser/regexp/re-bytecode.c:242:41: error: initialization from incompatible pointer type [-Werror]
   re_compiled_code_t *compiled_code_p = bc_ctx_p->block_start_p;

This patch adds the missing explicit cast.

LGTM

LGTM


Update no-bad-latin.py

Add globbing to no-bad-latin.py to make ignoring easier.

Ehh, just going to merge this as I'm tired of seeing the blog posts give us a red x mark.


Serenity Update problem - addOption() and clearItems() methods on lookup editor are not accessible.

addOption() and clearItems() methods have been changed to protected in the latest update, is there an alternative to clear and add items to editor?
filter() on editor does not work.

They were always so


gl_FragData and WebGL extensions.

Did not find an easier way to detect fragData using SPIR-V specification, let me know if I missed something!
All common WebGL extensions should now work automatically.

Looks good to me.


Replace runfiles based Python tests with Java tests

This allows us to remove rules_python as a MODULE-level dependency.

There are still some .py tests remaining under //tests/integration_tests. Those files can be handled independently as they are part of self-container integration workspaces.
tests/integration_tests/example_workspaces/pkl_package/pkl_package_test_one.py
tests/integration_tests/example_workspaces/pkl_package/pkl_package_test_two.py
tests/integration_tests/example_workspaces/pkl_project/pkl_project_test.py


WebRTC causes webpage to become "Not Secure"

Which version are you using?
v1.5.0
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[X ] Linux arm64 standard
[ ] Linux arm64 Docker
[X ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[ ] Other (please describe)

Describe the issue
Hi
My site works with Full (strict) SSL/TLS through Cloudflare, with an origin and edge certificate installed. Now I have added a WebRTC stream and the site has gone from being 'Secure' to 'Not Secure'. The WebRTC stream is secured with a cert and key from open ssl (2048 & sha256), as per the README.
I am displaying the WebRTC stream inside an iframe. I have also tried with HLS and I have the same issue.
Any idea as to why adding the WebRTC stream causes the site to no longer be secure?
I can't link my site at this stage as it is still under development. I can create a safe page to demonstrate if required.
As a side note, I noticed that the WebRTC (and HLS) stream doesn't load through my domain unless I have viewed the stream locally first inside the browser and allowed the 'unsecure connection' to proceed. However I have to view this page over https (http doesn't work as expected).
Could this error be due to the self signed certificate used in the mediamtx webtrc encryption?
Thank you
Describe how to replicate the issue

start the server
go to domain

Did you attach the server logs?
no
Did you attach a network dump?
no

The problem was the browser didn’t like the self-signed certificate. Acquiring a certificate from a known CA fixed the problem.


no formatting when using self outside of impl

fn foo(self) {}

This code will be rejected to format, because
error: unexpected `self` argument in function
 --> src/lib.rs:1:8
  |
1 | fn foo(self) {}
  |        ^^^^ `self` is only valid as the first argument of an associated function

which is of course correct, but it shouldn't affect the formatting of the code, or am I missing something?

As rustfmt works on the AST, it is only applicable to code that is syntactically correct. It does not require the code to be semantically correct, however.
With respect to your code example, using self outside of impl block is a syntax error in rust, and hence is rejected by rustfmt.


SoapFault in Service.php line 218

Cannot process the message because the content type 'text/xml; charset=utf-8' was not the expected type 'application/soap+xml; charset=utf-8'.

+1 same error, but no more debug info

Try this
->options(['soap_defencoding' => 'UTF-8', 'soap_version'=> SOAP_1_2]);

Did you figure out this issue?
If not please re-open this issue.


Add Mandrill as a notification email provider

This PR would add mandrill as an email provider for the email notification plugin.
Right now it adds a hard dependency on the mandrill client (similar to how boto is a hard dependency), but I'd be willing to revisit that and make mandrill a soft dependency.

Thanks! Will take a look and get this merged in.

I think we should go ahead and make mandrill optional.
Something like:
try:
    import mandrill
except ImportError:
    pass

and
mandrill_requires = [
   'mandrill==1.0.57',
]

extras_require={
   'tests': tests_require,
   'docs': docs_require,
   'dev': dev_requires,
   'mandrill': mandrill_requires,
}

Sounds good. I'll update the PR

Closing this for now, feel free to re-open.


pdksync - MODULES-9692 - pdksync_1.12.0-0-g55d9ae2

MODULES-9692 - pdksync_1.12.0-0-g55d9ae2
pdk version: 1.12.0

@sheenaajay it looks like the acceptance tests are failing as they're running beaker in travis.


Season Icon possible Incompat with Open Parties and Claims

While on a Claimed chunk the Season text moves with the Info but the Icon doesn't
1.19.2 SeasonHUD 1.5.13
While on Forceloaded Chunk:

While on Claimed Chunk

Outside of Claims

Configs:
xaerominimap.txt
xaeroworldmap.txt
SeasonHUD is on Defaults

Thanks, I didn't account for the claim info overflowing to two lines if the text long enough.  Let me take a look and see if I can fix it for the next release

Cool, btw
Another thing I noticed is when you use the Xaero Minimap key to expand the minimap the icon also stays in place and overlaps with the map
I'll send a screenshot later

Whoops

Both of these should be fixed in next release. It should be out in a day or two, I just need to port it to the rest of the versions and test .


GitHub Enterprise Importer is currently unavailable

We are migrating repos from BitBucket to Github enterprise server and having issues while doing that.
Please find the error log and if someone can help us on this would be great
[2023-09-01 10:04:02] [DEBUG] RESPONSE (OK): {"data":{"createMigrationSource":null},"errors":[{"type":"SERVICE_UNAVAILABLE","path":["createMigrationSource"],"locations":[{"line":1,"column":109}],"message":"GitHub Enterprise Importer is currently unavailable. Please try again later."}]}
[2023-09-01 10:04:02] [ERROR] OctoshiftCLI.OctoshiftCliException: GitHub Enterprise Importer is currently unavailable. Please try again later.
at OctoshiftCLI.Services.GithubClient.EnsureSuccessGraphQLResponse(JObject response) in /home/rdoddi/gh-gei/src/Octoshift/Services/GithubClient.cs:line 255
at OctoshiftCLI.Services.GithubClient.PostGraphQLAsync(String url, Object body, Dictionary2 customHeaders) in /home/rdoddi/gh-gei/src/Octoshift/Services/GithubClient.cs:line 75 at OctoshiftCLI.Services.GithubApi.CreateBbsMigrationSource(String orgId) in /home/rdoddi/gh-gei/src/Octoshift/Services/GithubApi.cs:line 308 at OctoshiftCLI.BbsToGithub.Commands.MigrateRepo.MigrateRepoCommandHandler.ImportArchive(MigrateRepoCommandArgs args, String archiveUrl) in /home/rdoddi/gh-gei/src/bbs2gh/Commands/MigrateRepo/MigrateRepoCommandHandler.cs:line 208 at OctoshiftCLI.BbsToGithub.Commands.MigrateRepo.MigrateRepoCommandHandler.Handle(MigrateRepoCommandArgs args) in /home/rdoddi/gh-gei/src/bbs2gh/Commands/MigrateRepo/MigrateRepoCommandHandler.cs:line 102 at OctoshiftCLI.Extensions.CommandExtensions.RunHandler[TArgs,THandler](TArgs args, ServiceProvider sp, CommandBase2 command) in /home/rdoddi/gh-gei/src/Octoshift/Extensions/CommandExtensions.cs:line 65
at OctoshiftCLI.Extensions.CommandExtensions.<>c__DisplayClass1_0`3.<b__0>d.MoveNext() in /home/rdoddi/gh-gei/src/Octoshift/Extensions/CommandExtensions.cs:line 51
--- End of stack trace from previous location ---

For more information we are trying to call the graphql api, mutation end point createMigrationSource and getting the error
Current Github enterprise version that I am using is 3.9.3

@rakeshdoddi3646 Thanks for flagging this! I've just tried running a migratin myself, and it seems to be working okay for me. Can you try again now?

@timrogers thanks for responding o this can I know what  github enterprise server version you are using?


Updating webhook docs

Updates the webhook docs to point toward the docs for individual checks, and adds docs for common issues/version specific issues
Related to rancher/rancher#41493.

@martyav / @btat I've made the changes requested. I resolved the open conversations which I implemented, and left a couple open where I made slightly different changes than requested.

@KevinJoiner Hi, Kevin. Could you give this a technical review?


upgrade bettertouchtool to 1.47

Thank you for the contribution. It was merged directly as 947fdae206465c98ed91782b04a9876d28d3d504 to keep commit history cleaner. Your contribution is still credited to you.


Handle llama_index for newer .legacy.vector_stores

Handle llama_index for newer .legacy.vector_stores
Added a version check for llama_index version, and apply the use of .legacy.vector_stores on newer version than 0.10.0 Adjusted imports for version check.
Version check for llama_index
llama_index_version = pkg_resources.get_distribution("llama_index").version
if version.parse(llama_index_version) >= version.parse("0.10.0"):
from llama_index.legacy.vector_stores.chroma import ChromaVectorStore
else:
from llama_index.vector_stores.chroma import ChromaVectorStore

I'm at a loss, cannot find a lint-fix makefile in the project. I just added a condition to the file.
This is mysterious for me. Please advise.

Ugh, sorry @Xorba001 , I accidentally closed this by trying to fix the issue and force-pushing to the branch.
And I'm sorry about my mistaken advice last time. Basically, what you'd want to do is do pre-commit install in the opendevin repo. Then every time you make a commit it will automatically lint for you.


Make expression explanation more clear

@mikesjewett What do you think?

👍

Blync Deploy!
The following files were deployed

courses/javascript-primer/01-syntax/12-expressions/body.md


refactor: fix confusing impl of indexed_fragments

The reference to total_fragment_bitmap made no sense. This PR eliminates that, and also fixes very minor inefficiencies.

Codecov Report
Attention: Patch coverage is 86.66667% with 2 lines in your changes missing coverage. Please review.

Project coverage is 78.39%. Comparing base (536e73d) to head (5991d35).
Report is 7 commits behind head on main.

Files with missing lines
Patch %
Lines

rust/lance/src/index.rs
86.66%
0 Missing and 2 partials :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3047      +/-   ##
==========================================
+ Coverage   78.38%   78.39%   +0.01%
==========================================
  Files         240      240
  Lines       77122    78592    +1470
  Branches    77122    78592    +1470
==========================================
+ Hits        60449    61614    +1165
- Misses      13565    13862     +297
- Partials     3108     3116       +8

Flag
Coverage Δ

unittests
78.39% <86.66%> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Dehoisting members doesn't work with the new username system discord implemented

As the title says, using the manual dehoist command does not dehoist anyone anymore as it likely has to be with the new username system discord added. If there is a fix for this it would be helpful

This is due to the new 'display name' field, that isn't provided in the current version of the underlying library that Vortex uses. This is on-hold until some Vortex internals are ready to be updated.


Follow-up from "Resolve "Add an airflow route to the Meltano UI""

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/659
Originally created by @mbergeron on 2019-05-28 17:49:45

The following discussion from !602 should be addressed:

[ ] @derek-knox started a discussion:  (+2 comments)

The answer may be no, but do we want to have a global or otherwise reusable ref of port 5000? This is the second time I've seen it in this review (and I see three with a quick search in the src)

View 4 previous comments from the original issue on GitLab


Remove pty from the application

Intention

I'm submitting a feature request

As I do not feel comfortable to perform such surgery on Monterey, I am doing it only on the Prototyper, expecting that this information will help with performing the same action on. Monterey

Fixed for Prototyper - see here

Note for later:
Monterey will also need this removed:

plugin folder
https://github.com/monterey-framework/monterey/tree/master/app/src/plugins/terminal

plugin loading code
https://github.com/monterey-framework/monterey/blob/master/app/src/plugins/index.ts#L16

less style file
https://github.com/monterey-framework/monterey/blob/master/app/styles/less/layout/terminal.less

less loading/ref code
https://github.com/monterey-framework/monterey/blob/master/app/styles/less/layout/index.less#L13

Thanks for those steps @vegarringdal 👍
The postinstall task of npm install will now install the app dependecies as well


[PE-D][Tester D] Person's name does not accept theoretically valid names

Steps to reproduce:
add or edit but use a name with '/' within it.
I understand that this was clearly stated in the UG that only alphanumeric characters are allowed for the NAME format however many valid Indian Singaporean names contain 's/o' or 'd/o' e.g "Manjit Singh s/o Karamjit Singh" and as a financial advisor I believe that the full name of the client should be inputted

Labels: severity.Medium type.FeatureFlaw
original: sethteo/ped#2

Duplicate of #249


HTML tags cannot be used with commonmarker >= 0.18

commonmarker changed its default to be safe [1], [2], i.e. HTML tags can not be used.
There is :UNSAFE option to enable HTML tags, but I am not sure it is suitable to set it as default.
Maybe better to have user option to toggle it.
[1] https://github.com/gjtorikian/commonmarker/pull/81
[2] https://github.com/github/cmark-gfm/pull/123

Thanks for so fast fix and enhancement! I sent one PR https://github.com/okkez/redmine_common_mark/pull/6 to fix Gemfile.
I ❤ this plugin so much!


Missing numpy dependency at installation

I installed PyTorch on a fresh Python 3.6 installation (brew install python3), got an error and found out that numpy wasn't installed. I installed numpy and then things work. Shouldn't it be included among dependencies of the PyTorch package? (with the right version requirement, see #559?)
Minhs-MacBook-Pro:~ cumeo$ pip3 install --user http://download.pytorch.org/whl/torch-0.1.12.post2-cp36-cp36m-macosx_10_7_x86_64.whl
Collecting torch==0.1.12.post2 from http://download.pytorch.org/whl/torch-0.1.12.post2-cp36-cp36m-macosx_10_7_x86_64.whl
  Downloading http://download.pytorch.org/whl/torch-0.1.12.post2-cp36-cp36m-macosx_10_7_x86_64.whl (3.7MB)
    100% |████████████████████████████████| 3.7MB 1.6MB/s
Collecting pyyaml (from torch==0.1.12.post2)
Installing collected packages: pyyaml, torch
Successfully installed pyyaml-3.12 torch-0.1.12.post2
Minhs-MacBook-Pro:~ cumeo$ python3
Python 3.6.2 (default, Jul 17 2017, 16:44:45)
[GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
ImportError: numpy.core.multiarray failed to import
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/cumeo/Library/Python/3.6/lib/python/site-packages/torch/__init__.py", line 53, in <module>
    from torch._C import *
ImportError: numpy.core.multiarray failed to import
>>> import numpy
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'numpy'
>>>
Minhs-MacBook-Pro:~ cumeo$ pip3 install --user numpy
Collecting numpy
  Using cached numpy-1.13.1-cp36-cp36m-macosx_10_6_intel.macosx_10_9_intel.macosx_10_9_x86_64.macosx_10_10_intel.macosx_10_10_x86_64.whl
Installing collected packages: numpy
Successfully installed numpy-1.13.1
Minhs-MacBook-Pro:~ cumeo$ python3
Python 3.6.2 (default, Jul 17 2017, 16:44:45)
[GCC 4.2.1 Compatible Apple LLVM 8.1.0 (clang-802.0.42)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import torch
>>>

Perhaps you meant to file this at https://github.com/pytorch/pytorch/issues/new instead of as a TensorFlow issue?

I'm so sorry :">


Add StopWatch and Timer samples

The StopWatch state machine has two states; idle and timing.
When the machine is in state idle and receives;

idle - the machine transitions back to idle.
start - the machine records the current time and transitions to timing.
stop - the machine transitions back to idle.
exit - the machine exits.

When the machine is in state timing and receives;

stop - the machine records the current time, send the elapsed time to the sender, and transitions back to idle.

The state idle is the parent of timing, so when timing receives an idle, start, or exit message, it is handled by the parent state.
The Timer state machine takes a number of seconds as a parameter, and has the same two states; idle and timing of which the latter has no receivers and is the initial state. Upon entering  the state timing the elapsed time is compared with the seconds parameter, if elapsed is smaller the state will re-enter itself, else a timeup messages is send to the spawner.

Superseded by https://github.com/AletheiaWareLLC/Wink/pull/31


Default props can't be overridden by inline props

🐛 Bug Report
(A clear and concise description of what the bug is.)
To Reproduce

Press the toggle button in the demo. The square animates OK.
Press it again -> it animates "immediately". This is expected as we set default: { immediate: true } in the setStyle(...) call.
Press it yet again -> it animates "immediately". This is NOT expected as we set immediate: false in the setStyle(...) call. It works as expected if we use default: { immediate: false } and override the "default" props. However, I think the default props should be overridden by setting specific props in the function calls.

See https://codesandbox.io/s/quirky-goodall-6z3og?file=/src/App.js
Expected behavior
The default props should be overridden by "inline" props.
Link to repro (highly encouraged)
https://codesandbox.io/s/quirky-goodall-6z3og?file=/src/App.js
Environment

react-spring v9.0.0.-rc3
react v16.12

Hi @tomdohnal the issue is because there are no props with the name default.
This works fine.
if (toggle) {
  setStyle({
  transform: "translateX(0px)",
  immediate: true
  });
}
else {
  setStyle({
  transform: "translateX(200px)",
  immediate: false
  });
}

I'm using the v9 (9.0.0.-rc3) version of the library which introduces the concept of default props.
https://aleclarson.github.io/react-spring/v9/#Default-props

Ohh alright, my bad. I forgot you mentioned the environment you were running on.
But won't toggling the state of the default prop defeat the purpose of setting it as default?

Yeah, you probably wouldn't use it like that in a real-life scenario :)
However, the demo shows that if you set the default props within an imperative setStyle call and then try to override these default props in another setStyle call, it does NOT override them


[rm2fb] re-consider LD_PRELOAD for launchers

compiling binaries for usage in toltec testing channel requires using patchelf on them.
this is a strange situation since rM1 does not require rm2fb or necessarily have it installed, so a binary that has been patchelf'd is incompatible with rM1 that does not have toltec installed with rm2fb. i think this means that i need to supply people with two compilation instructions: one for toltec enabled rM1/rM2 and one for rM1 without toltec

actually, never mind me

I'm confused. If a package is in Toltec, it has a build recipe designed to produce binaries that are distributed as part of Toltec. If you build an app with a Makefile or similar build infrastructure directly from it's repo, it likely wont produce patchelf'd binaries. The patchelf'ing only happens for Toltec in a Toltec package recipe.

previously: all launchers had LD_PRELOAD enabled for them through a systemd override. any app launched through launcher will have rm2fb placed onto it, regardless if app comes from toltec or not.
now: launcher is patchelf and there is no global LD_PRELOAD. if the launcher wants to launch an app, it needs to setup LD_PRELOAD for that app or know that is has been patchelfed.
now: if i want to give someone a binary, the binary needs to be patchelf (if running on rM2) or it needs to not be patchelf (if running on non-toltec rM1). it is ok to give patchelf binary to rM1 with toltec and non-patchelf binary to rM2 without toltec.
as an example: remux either needs to be compiled with patchelf or not. if someone want to use my repo to compile remux and use it with toltec, they will need to use the patchelf version. if they have an rM1 and no toltec, they will need to use the non-patchelf. which do i choose by default? either way, some percentage of people will be confused when they didn't need to be

Please take everything below with a grain of salt. Maybe I'm still wrong in my understanding of your concern.
I think you should distribute a non-patchelf version of your launcher from your repo and let patchelf happen only in Toltec.
I think it is reasonable to expect that folks who aren't using Toltec will:
A.) Have an rM1 and not need a preload or rm2fb and will not want any patchelf or preload of any kind.
-or-
B.) Have an rM2 and have installed rm2fb manually and understand the need for both it and the shim/preload. Instructions for that should be kept with rm2fb in its repo with manual install instructions. For these users, you could detect the absence of Toltec and preload for them to be nice, or you could expect them to modify draft files.
Users who are using Toltec will have a package built automatically for their device that includes exactly what is needed for launchers and apps to work without tweaking.
The only case where there is an issue AFAICS is a user who has Toltec on an rM2, used it to install remux (which is therefore patchelf'd) and wants to use that launcher to launch a non-Toltec compiled and installed app.
What should really happen is non-Toltec apps shouldn't land in the same directory as Toltec apps and if you want to help out this user base, consider where the app is installed when deciding how to launch it. Because this isn't a sure thing, maybe you could ask opkg to elicit all of the files it has installed and LD_PRELOAD anything not belonging to one of its packages that originates from toltec. Arch has a pacman -Ql feature that could achieve this and I bet opkg does too.
We are already using /opt which is kind of unfortunate because ideally /opt/bin is where manually installed packages should go and package managers should target /usr/bin or /usr/local/bin. I understand why we chose to use /opt but maybe we should have made everything fall under /opt/toltec to avoid this conflict of directory purpose.
If I understand all of this correctly (which I admit I very likely don't), this exceptional usecase feels too narrow to justify losing the benefit of patchelf.

the issue is not just for launchers but for any app that is compiled outside of toltec. if someone wants to compile an app outside of toltec, it will not work unless they know to use patchelf or they install it into this special directory that is mentioned (but not yet implemented). this is what happened for zshelf: the person compiled it (to work with the new Qt) and were unable to launch it from any launcher in testing (or even their own custom compiled launcher if they had time to make one)
if i want to make an app for someone to test (without making it go through toltec channels), we will need to ask about their tablet every time (or publish two versions every time and tell them to pick the one that fits their tablet) or explain to them that it needs to go into a special directory

for now, i will try out a systemd override on ldpreload for remux only when rm2fb is installed

Please feel free to re-open if there are additional things to fix related to this issue.


Test on and support additional NICs

Original development was on Mellanox ConnectX-5. #40 adds support for Intel X710. We'd like to test Retina compilation/functionality on a few more common NICs.

I've recently shifted to a box using the Intel E810 and can confirm that #40/#54 also work on this NIC.

Removing the default flag for Mellanox ConnectX-5 in core/Cargo.toml would improve out of the box compilation. When the mlx5 dependencies aren't installed, this flag must be manually removed in order to get Retina to build. This was previously referenced in #1.
https://github.com/stanford-esrg/retina/blob/ad405d527b6d4d9ba9debff2ac726ba2fa3489e9/core/Cargo.toml#L53

This is great!! @sippejw could we add the server configurations you tested this on to the INSTALL.md documentation? (What CPU/OS were you using?)


SignalR NodeHttpClient module not found - Angular SSR

Describe the bug
This is the same exact as: https://github.com/aspnet/SignalR/issues/3254
When using SignalR with Angular in SSR it throws an error (using Angular7, template with CLI - based on .NET Angular official Template to be exact)
NodeInvocationException: Prerendering failed because of error: Error: Cannot find module './NodeHttpClient'

Additional context

@aspnet/signalr: v1.1.2 (latest)

After diagnosing some, basically NodeHttpClient, which is imported only on Node, is being imported relatively e.g.
// bundled code
var nodeHttpClientModule;
if (typeof XMLHttpRequest === "undefined") {
    // In order to ignore the dynamic require in webpack builds we need to do this magic
    // @ts-ignore: TS doesn't know about these names
    var requireFunc =  true ? require : undefined;
    nodeHttpClientModule = requireFunc("./NodeHttpClient");
}

SSR App is bundled (dist/main.js), hence this file does not exists relative to ./dist/main.js file at runtime
So i guess it should be either:

Somehow webpack bundles it if its node only
Imported also for browser, but will eventually gets treeshaked

@muratg This looks like it's worth patching.

@BrennanConroy could you take the initial look to see what a patch may look like?

If you would need a repro project which you can use to test, I should be able to provide (but would need some time to simplify it a bit)

Yes please, I tried out the dotnet new angular template and it seems to work fine.

Yes, thats because it does not bundle server (even prod build) and SSR its not enabled by default either (unless you configured it).
I will try to provide you the sample, probably in the coming 2days

I managed to get it up quickly
repo: https://github.com/sketch7/dotnet-ngx-sample/tree/signalr-ssr-error
To get started
git clone https://github.com/sketch7/dotnet-ngx-sample.git -b signalr-ssr-error
npm start

NOTE: I installed signalr via @ssv/signalr-client not signalr-client directly (which is our library), but all its doing at that point is creating a new Hub instance. Feel free to change it directly to @aspnet/signalr
Hope it helps, thanks!

Thanks for the repro it helped a lot!
After discussing with Andrew and Steve we're going to look at not trying to be clever and just including the import always. From a quick test it looks like this only adds a couple kb to the .js file.

Glad it was useful!
Yes sometimes simple is better, and there's a possibility that it gets treeshaked with the build (not sure how its used exactly tho, so might not be the case).

Thanks for pointing me to https://github.com/sketch7/signalr-client @stephenlautier. It's nice to see how other people have approached adding automatic reconnects to the JS client.

we're going to look at not trying to be clever and just including the import always. From a quick test it looks like this only adds a couple kb to the .js file.

I don't like adding a couple KB to the browser and webworker dists for no reason. I don't think either of these dists ever run on node.js. If you look at the the full NodeInvocationException, you can tell that the bundler used the cjs dist. I think always including the import in the cjs and esm dists is probably fine.
fail: Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware[1]
      An unhandled exception has occurred while executing the request.
Microsoft.AspNetCore.NodeServices.HostingModels.NodeInvocationException: Prerendering failed because of error: Error: Cannot find module './NodeHttpClient'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:613:15)
    at Function.Module._load (internal/modules/cjs/loader.js:539:25)
    at Module.require (internal/modules/cjs/loader.js:667:17)
    at require (internal/modules/cjs/helpers.js:20:18)
    at<EMAIL_ADDRESS>(C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:86548:28)
    at __webpack_require__ (C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:20:30)
    at<EMAIL_ADDRESS>(C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:89165:27)
    at __webpack_require__ (C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:20:30)
    at<EMAIL_ADDRESS>(C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:91187:73)
    at __webpack_require__ (C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx\ClientApp\dist-server\main.js:20:30)
Current directory is: C:\dev\halter73\dotnet-ngx-sample\Sketch7.DotnetNgx

   at Microsoft.AspNetCore.NodeServices.HostingModels.HttpNodeInstance.InvokeExportAsync[T](NodeInvocationInfo invocationInfo, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.NodeServices.HostingModels.OutOfProcessNodeInstance.InvokeExportAsync[T](CancellationToken cancellationToken, String moduleName, String exportNameOrNull, Object[] args)
   at Microsoft.AspNetCore.NodeServices.NodeServicesImpl.InvokeExportWithPossibleRetryAsync[T](String moduleName, String exportedFunctionName, Object[] args, Boolean allowRetry, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.NodeServices.NodeServicesImpl.InvokeExportWithPossibleRetryAsync[T](String moduleName, String exportedFunctionName, Object[] args, Boolean allowRetry, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Builder.SpaPrerenderingExtensions.<>c__DisplayClass0_0.<<UseSpaPrerendering>b__1>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.Routing.EndpointRoutingMiddleware.Invoke(HttpContext httpContext)
   at Microsoft.AspNetCore.StaticFiles.StaticFileMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
info: Microsoft.AspNetCore.Hosting.Internal.WebHost[2]

This has been merged.

Is this fixed? After updating to package 1.1.4, the Cannot find module './NodeHttpClient' but now I have this error:
NodeInvocationException: Prerendering failed because of error: Error: Cannot find module 'request' at Function.Module._resolveFilename (internal/modules/cjs/loader.js:580:15) at Function.Module._load (internal/modules/cjs/loader.js:506:25) at Module.require (internal/modules/cjs/loader.js:636:17) at require (internal/modules/cjs/helpers.js:20:18) at<EMAIL_ADDRESS>

File a new issue with the details of the problem you're seeing. We don't track closed issues.


pull request for e2e test at 11/20/2016 04:56:06

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

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


Add logic to validate enrollments of blockheader

This verifies the qualification of the enrollments included in the blockheader when verifying the block.
And for some grammatical modification, the parameter order of isInvalidEnrollmentReason was changed.
#205

@Geod24 @AndrejMitrovic  ping.


Update docusaurus.config.js for GA4

will need to wait a little until v2.4 of docusaurus to merge this https://github.com/facebook/docusaurus/issues/7221#issuecomment-1414099341

this should be ready to merge https://github.com/facebook/docusaurus/releases/tag/v2.4.0

hmm im still getting
[ERROR] ValidationError: "trackingID" must be a string

oh - looks like should use this instead: https://docusaurus.io/docs/api/plugins/@docusaurus/plugin-google-gtag


Update introduction.md

Fix typos in the Assignment Operators section.

Thanks for the quick turnaround on this!
Agreed on the revamp. I wanted to separate the simple fix from a larger change. I think there should be mention of operations in parentheses taking precedence over left-to-right order. The solution also seems to require std:ceil but doesn't cover floor vs ceiling rounding.
This exercise also explains using an apostrophe to separate digits which would have been better placed in an earlier exercise in the syllabus, Last Will, which uses that notation without any mention of it.

Thanks for the quick turnaround on this!

Thanks for opening a PR!

Agreed on the revamp. I wanted to separate the simple fix from a larger change. I think there should be mention of operations in parentheses taking precedence over left-to-right order. The solution also seems to require std:ceil but doesn't cover floor vs ceiling rounding.

I merged the fixes for the typo, there might be other places that have the same texts, I will try to correct those as well. Generally there is the introduction from the concept, which is reused in the about and again in each exercise that uses the concept. Three times the work. I would be happy to automate that bit with templates.

This exercise also explains using an apostrophe to separate digits which would have been better placed in an earlier exercise in the syllabus, Last Will, which uses that notation without any mention of it.

Good point. I was sure I had those in the right order, I have to check the config.json if there was a mix-up somewhere.

I merged the fixes for the typo, there might be other places that have the same texts, I will try to correct those as well.

I wasn't aware of the duplication - will keep it in mind for the future.
For this particular set of typos they also need fixing in:
exercises/concept/freelancer-rates/.docs/introduction.md
./concepts/numbers/about.md


Covered another branch with test

Coverage remained the same at 100.0% when pulling 056716bf3446d1addd9b04a240332aea8993af52 on moar-coverage into 187524235a04366a5b0344d203cacd19d4cb718d on master.


Carbon for ibm.com Patch Release Plan (v1.18.1)

April 22, 2021: Release Branch Created
April 22 - 27, 2021: Release Candidate(s) and testing
April 27: Full Patch Release

Web Components
Fixes

dependencies

changed @carbon/telemetry to be a dependency (#5898)

locale

use getLocale method to verify locale before passing into translation call (#5907)

React
Fixes

dependencies

changed @carbon/telemetry to be a dependency (#5871)

Styles
Fixes

dependencies

changed @carbon/telemetry to be a dependency (#5871)

Services
Fixes

locale

return locale with the fetch list call (#5899)

dependencies

changed @carbon/telemetry to be a dependency (#5871)

Utilities
Fixes

dependencies

changed @carbon/telemetry to be a dependency (#5871)

Housekeeping

settings

updating to v1.18.1

@RobertaJHahn release created: https://github.com/carbon-design-system/carbon-for-ibm-dotcom/releases/tag/v1.18.1


Error when using hover after diagnostic_jump

Describe the bug
Error detected while processing CursorMoved Autocommands for "<buffer=1>":
Error executing lua callback: /tmp/lazy/plugins/lspsaga.nvim/lua/lspsaga/util.lua:95: E31: No such mapping
stack traceback:
        [C]: in function 'nvim_buf_del_keymap'
        /tmp/lazy/plugins/lspsaga.nvim/lua/lspsaga/util.lua:95: in function 'delete_scroll_map'
        /tmp/lazy/plugins/lspsaga.nvim/lua/lspsaga/hover.lua:168: in function </tmp/lazy/plugins/lspsaga.nvim/lua/lspsaga/hover.lua:166>

Steps to reproduce

run :Lspsaga diagnostic_jump_next to jump to a diagnostic. you can create a syntax error if the file has no diagnostics
be sure not to move the cursor after running diagnostic_jump_next
run :Lspsaga hover. there should now be 2 floating windows drawn. the diagnostic window, and the hover window
move the cursor
see error

if you then use the hover function again, it will display a window and jump into it, as if you ran hover twice
Expected behavior
for there to not be an error, and possibly to close the diagnostic window before displaying the hover window. although that's a design choice
Neovim version (nvim -v)
0.9.0
lspsaga commit
58cbf44
Terminal name/version
Alacritty 0.11.0

that was fast. but now I'm getting this new error whenever I run hover no matter if after/before diagnostic_jump_next
Error executing Lua callback: /tmp/lazy/plugins/lspsaga.nvim/lua/lspsaga/hover.lua:326: module 'diagnostic' not found:
``

fixed.

thanks for the quick reply. but unfortunately i'm still seeing the original error when i try the steps above to reproduce

update and retry

awesome it works! thanks again!

@glepnir this issue seems to be happening again for me 🙁 : when I hover (using K) and then I want to close the hover (with K again), I'm getting the error

emm create a new issue with reproduce step.

@glepnir created #1451 ; I'd appreciate if you take a look
Thank you


Image to 3D point cloud?

How are you converting your 2D camera images to 3D point clouds?
You don't go into any detail about this in your paper or provide a reference for the method. I was wondering if you could either elaborate or point me to the code that deals with this method?

Depths could be directly converted into point clouds with camera parameters. It is not a method we propose or introduced by others, but one basic truth in 3D vision.
There are some online blog that introduce this, such as this.

I checked out your link & I'm not convinced you can calculate depth like that as equation 1 is not unique.


ISPN-6994 Explicit Subject impersonation

Overcomes the need to keep the Subject in a ThreadLocal and avoids the use of PrivilegedActions in user code

https://issues.jboss.org/browse/ISPN-6994

Still need to properly handle query permissions

Failures are unrelated

Hey @galderz, I removed the unneeded import

I have renamed as() to withSubject() which follows withFlags()

@tristantarrant The NoSuchMethodException's in org.infinispan.security.* appear to be related to your latest changes

I've fixed the SecureCacheTestDriver.

Those 2 failures look suspicious:
org.infinispan.server.test.configs.ExampleConfigsIT.testSSLHotRodConfig
org.infinispan.integrationtests.spring.boot.session.remote.RemoteSpringSessionTest.org.infinispan.integrationtests.spring.boot.session.remote.RemoteSpringSessionTest

Let me retrigger CI, maybe it happened by an accident.

ExampleConfigsIT.testSSLHotRodConfig seems to be failing for several builds, so I don't think that failure is related. However, I have tested RemoteSpringSessionTest locally and it fails on every run with a IndexOutOfBoundsException, so changes are required

I fixed the failing test.

Integrated, thanks @tristantarrant


Hover effect. SVG + Animation?

Hi there!
I try to do a hover effect directly with an animation from ae. I don´t know why, but it doesn´t work... (That what I have written). Could you help me?
`var animation = bodymovin.loadAnimation({
container: document.getElementById("logo").addEventListener("mouseover", function(){ anim.play(); }),
renderer: "svg",
loop: true,
autoplay: false,
prerender: true,
path: "json/logo.json"

})`
Could I a svg image (logo) in the html embed and render the animation (json data) as hover?
Thank you! :)

your animation instance is called "animation", have you tried doing animation.play() instead of anim.play() ?

Yes I did it, but with the same result... Maybe is it an animation´s problem?

Does putting these separate work:

var logo = document.getElementById("logo");
var animation = bodymovin.loadAnimation({
container: logo,
renderer: "svg",
loop: true,
autoplay: false,
prerender: true,
path: "json/logo.json"
});
logo.addEventListener("mouseover", function(){ anim.play(); });

Hey! It´s alive, thank you so much!
That´s what I wrote (maybe can it help someone ;)):
`var logo = document.getElementById("logo");
var animation = bodymovin.loadAnimation({
container: logo,
renderer: "svg",
loop: true,
autoplay: false,
prerender: true,
path: "json/logo.json"
});
logo.addEventListener("mouseenter", function () {
animation.play();
});
logo.addEventListener("mouseleave", function () {
animation.stop();
});`


SECURITY.md: delete

This is being replaced by https://github.com/Homebrew/.github/pull/58

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Review period will end on 2022-04-19 at 08:11:47 UTC.

Review period skipped due to critical label.