chore: cache members, mappable members, get attribute and optimize `CollectionInfoBu…

Cached GetMembers, GetAccessibleMappableMembers and optimized CollectionInfoBuilder
Description
Modified WellKnownTypes to cache more members, mappable members and attributes. Added a couple of checks to CollectionInfoBuilder to speed up common paths.

Added cached methods to WellKnownTypes for GetAttributes, GetAllMembers and GetAllAccessibleMappableMembers
CollectionTypeInfo

Converted CollectionTypeInfo to a struct
Removed duplicate IImmutableQueue and added ImmutableSortedDictionary
Added check to HasValidAddMethod for types with known Add methods, otherwise all none None types return false. None types are manually checked for Add methods.
IsCountKnown returns false for IEnumerable and true for all none None types. None types are manually checked for length properties.
Added a massive switch expression to GetImplementedCollectionTypes, it uses the known CollectionTypeInfo to get the implemented types. None types are manually checked for implemented types. Could be moved into _collectionTypeInfos.

Added a special check for string to return IEnumerable, because string isn't actually a "collection" type but does implement IEnumerable and is marked None by GetCollectionTypeInfo. It would iterate thrrought every possible collection type causing a considerable slow down.

Moved GetEnumeratedType into BuildCollectionInfos. This is a quick check to ensure that both the source and target types are actually enumerable, saving a lot of wasted time.

I kept the slightly modified GetAllMembersand GetAccessibleMappableMembers (they call WellKnownTypes) in SymbolExtensions. Should I remove them and update all the call sites to use the WellKnownTypes version?
Benchmarks
Optimized

Method
Mean
Error
StdDev
Gen0
Gen1
Allocated

Compile
1.699 ms
0.0322 ms
0.0317 ms
85.9375
19.5313
180.45 KB

LargeCompile
54.299 ms
0.9632 ms
0.9460 ms
1500.0000
375.0000
9344.24 KB

Original

Method
Mean
Error
StdDev
Gen0
Gen1
Allocated

Compile
3.569 ms
0.2264 ms
0.6641 ms
171.8750
31.2500
311.38 KB

LargeCompile
100.170 ms
4.6670 ms
13.3905 ms
2200.0000
800.0000
13395.01 KB

Helps #72
Checklist

[x] The existing code style is followed
[x] The commit message follows our guidelines
[x] Performed a self-review of my code
[x] Hard-to-understand areas of my code are commented
[x] The documentation is updated (as applicable)
[x] Unit tests are added/updated
[x] Integration tests are added/updated (as applicable, especially if feature/bug depends on roslyn or framework version in use)

@TimothyMakkison thank you for this PR. Could you split it up into several smaller PR's to make them easier to review? This also makes sure you get prompter feedback and keeps the enhancements isolated. Additionally the performance impact of each change can be measured more easily. Proposed PR's:

Caching of members
CollectionTypeInfo general optimisations (struct, Immutable duplicate / sorted dict, GetEnumeratedType in BuildCollectionInfos)
CollectionTypeInfo GetImplementedCollectionTypes

I'd split GetImplementedCollectionTypes and the general CollectionTypeInfo optimisations as I'm not sure if the massive type switch is worth it, would be nice if we could measure the performance impact on that separately and therefore create an isolated PR.
On a first glance the enhancements look promising 😊  I think I'd remove the member methods from SymbolExtensions and move them to the well known types class. Not sure if well known types is still an appropriate name then or if it should be renamed to just Types or similar.

Good idea, I'll split this pr.

I'm not sure if the massive type switch is worth it,

It was probably the most impactful change. Iirc Mapperly spent 30% of the time calling GetImplementedCollections. It would also get the type symbol for every possible collection, degrading most use cases where they only simple collections.
I can remove the switch and move it into CollectionInfos as another property

@TimothyMakkison oh okay, I didn't expect that. I think I'd keep the current switch as it is easier to read 🤔
Doesn't this deactivate the feature detection? Eg. IReadOnlySet is only available for .NET 5+ and not for .NET Standard 2.0, this was detected before, but isn't in the new code. Not sure if this is an actual problem for Mapperly...

@TimothyMakkison oh okay, I didn't expect that.

Yeah, I was a little surprised.

Doesn't this deactivate the feature detection? Eg. IReadOnlySet is only available for .NET 5+ and not for .NET Standard 2.0, this was detected before, but isn't in the new code. Not sure if this is an actual problem for Mapperly...

It could be an issue if Mapperly added a non backward compatible mapper for types implementing IReadOnlySet.


chore(proto): Update storage proto.lock

Description
Update storage proto.lock file.
The following commands are used:
cd proto/storage
../../.gotools/bin/protolock commit

Checklist

[ ] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
No testing was performed.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

What's the plan on how to keep this thing up to date to begin with?  particularly regarding new objects.

One general comment, not directly related to this PR:
I think it would be good to have the proto.lock file updated once per release before (or just after) branching. This would ensure the backward compatibility checks actually check changes in proto files do not break backward compatibility.


Task - Edit TaxonName - Error message for synonym name

I have a family with is treated as a synonym of another family, all children were moved to the valid family.

I did not see a pop up message to move children to the valid taxon and related section
I am getting an error message (see the picture).

the taxon in question is in 3i database
https://sfg.taxonworks.org/tasks/nomenclature/new_taxon_name?taxon_name_id=408911

Fixed, @proceps . The manage synonyms section should be visible now for FamilyGroups too. Let me know if the result you see there are correct.


Photino.HelloPhotino.AdvancedNET crashes with undefined symbol: basic_ostringstream

On Ubuntu 18.04, Photino.HelloPhotino.AdvancedNET sample (and others), clicking the "Call .NET via text passed to a memory buffer" button causes an app crash. The Photino window disappears and in the terminal the app prints:
/tmp/photino.Samples/Photino.HelloPhotino.AdvancedNET/bin/Debug/net5.0/HelloPhotino.AdvancedNET: symbol lookup error:
/tmp/photino.Samples/Photino.HelloPhotino.AdvancedNET/bin/Debug/net5.0/runtimes/linux-x64/native/Photino.Native.so:
undefined symbol: _ZNSt7__cxx1119basic_ostringstreamIcSt11char_traitsIcESaIcEEC1Ev

Demangling the undefined symbol:
std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()

We haven't seen that issue come up on our end, and neither has any other Linux user reported it.
Probably needs some more investigation, maybe differences in framework or OS installations?

This problem we believe should no longer be there in the newly released Photino v2 version. Please give it a try and let us know if you see anything unexpected.

I just installed .NET SDK 5.0.202 on Ubuntu 18.04.6, and then installed Photino using:
dotnet new -i TryPhotino.VSCode.Project.Templates
I then did
dotnet new photinoapp -o MyPhotinoApp
cd MyPhotinoApp
dotnet run
and I saw the same issue reported above.
For reference, here's the trace of the run from the terminal:
mikel@mars:~/Laboratory/photino/MyPhotinoApp$ dotnet run
Photino.NET: "Photino".SetTitle(Photino for .NET Demo App)
Photino.NET: "Photino for .NET Demo App".SetUseOsDefaultSize(False)
Photino.NET: "Photino for .NET Demo App".SetSize({Width=600, Height=400})
Photino.NET: "Photino for .NET Demo App".Center()
Photino.NET: "Photino for .NET Demo App".SetResizable(False)
Photino.NET: "Photino for .NET Demo App".Load(wwwroot/index.html)
Photino.NET: "Photino for .NET Demo App".Load(file:///home/mikel/Laboratory/photino/MyPhotinoApp/wwwroot/index.html)
Photino.NET: "Photino for .NET Demo App".SendWebMessage(Received message: "Hi .NET! 🤖")
/home/mikel/Laboratory/photino/MyPhotinoApp/bin/Debug/net5.0/HelloPhotino.NET: symbol lookup error: /home/mikel/Laboratory/photino/MyPhotinoApp/bin/Debug/net5.0/runtimes/linux-x64/native/Photino.Native.so: undefined symbol: _ZNSt7__cxx1119basic_ostringstreamIcSt11char_traitsIcESaIcEEC1Ev


set-output deprecation

While using this I got this warning in the log:
Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

This can be fixed by bumping the project's @actions/core package dependency to 1.10.0, which migrates the commands from the deprecated environment variable-based implementation to the new "environment file"-based implementation:
https://github.com/actions/toolkit/pull/1178

Fixed in the latest release. Apologies for the late fix.


Support sequence of keys with same modifiers

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

20 of 35 (57.14%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-8.7%) to 90.789%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/test-utils.ts
8
23
34.78%

Totals

Change from base Build<PHONE_NUMBER>:
-8.7%

Covered Lines:
175

Relevant Lines:
191

💛  - Coveralls


This builds for EL6+ and Fedora 26 (at least).

There's a bug yet with EL6 not recognizing %{_tmpfilesdir} I'll have to fix.

%{_tmpfilesdir} is a systemd thing, so it's not present in EL6.

https://github.com/flowerysong/OpenARC/commit/824cc0f5b74e37a709755cee9f4846bc03958d43 is what I think this change should look like. It maintains the propagation of @VERSION@ from autotools, doesn't try to point at a git snapshot (Source0 needs to be updated to a URL once we have an actual release, but this is still alpha software), and cleans up the now-unused configure bits.
Ideally the systemd configuration should be moved into the main build system as an option instead of sequestered in the RPM spec, but that can be done as a separate set of changes when someone has the time.

Packaging within Linux distros is dark arts to me, so I rely on the community for guidance.  Does this PR in its current form have consensus to proceed?

I'll accept @flowerysong's suggestions.  Are we going to reuse group 'mail'
or use a new group 'openarc' ?  The spec has parts each way which is broken.
On Wed, Aug 16, 2017 at 4:52 PM, Murray S. Kucherawy <
<EMAIL_ADDRESS>wrote:

Packaging within Linux distros is dark arts to me, so I rely on the
community for guidance. Does this PR in its current form have consensus to
proceed?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mskucherawy/OpenARC/pull/51#issuecomment-322910112,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAqDqjI2X0YkjRGflHRZp0dV2LJIpfujks5sY2SRgaJpZM4OgsUx
.

I merged @flowerysong 's branch into mine, accepting your suggestions.  I reviewed the groups usage, and believe we are good.  The user openarc gets added to group mail, but is running as group openarc.  If we want to drop adding to group mail, we could, but at least this is consistent with opendmarc packaging as-is.  @mskucherawy I believe this is safe to merge now.

@mskucherawy As no one else has spoken up with suggestions or objections, we appear to have reached consensus.

I appear to have accepted something that conflicts with this.  Can you rebase and resubmit?

#63 contained these changes with a cleaner history so this pull request can just be closed.


include a example to create a create log_configuration

please include a example to use a log_configuration,  I wasn't able to figure out how  pass to module

@rafilkmp3 https://github.com/cloudposse/terraform-aws-ecs-container-definition/blob/dea2f1cefaa4702d9b318dda889a5f64407abae7/examples/complete/fixtures.us-east-2.tfvars#L43

@rafilkmp3 Closing this unless that log_configuration example provided above wasn't good enough. Please reopen if that example doesn't satisfy you.


[5.8] Maintain default db connection on migrate call with db flag

This pull request attempts to address issue https://github.com/laravel/framework/issues/28253. The main problem is quoted below from this issue:

The problem is that when Artisan is running any database related commands, it sets the option passed with'--database' flag as the framework's default connection. I guess that it was not meant to call such artisan commands somewhere in between while the framework is processing the request (like in my case).

To remedy this, the PR creates a new restoreDefaultConnection method. If the DB_CONNECTION is set in the .env file, it resets the default connection to this instance after a migration is completed.
All of the tests in the automated test suite passed, except the ones that were skipped, console output below:
OK, but incomplete, skipped, or risky tests!
Tests: 4093, Assertions: 9492, Skipped: 87.

P.S. Not sure if it is appropriate to use the env helper in this case. If this is considered a bad practice, I apologize.

Please add a test.

Please use config() instead of env(): https://laravel.com/docs/configuration#configuration-caching
We should also cover migrate:reset and db:seed.
With migrate:rollback, the connection is not restored if the migrations table is empty.
/cc @deleugpn

Theoretically, someone could do this:
// DB_CONNECTION=mysql

DB::setDefaultConnection('pgsql');

Artisan::call('migrate', ['--database' => 'sqlite']);

dd(DB::getDefaultConnection()); // mysql

Here, config() wouldn't work. Not sure, whether we should cover this edge case.

I think solving this problem properly requires some digging and breaking changes. I never invested time on trying it because I assumed the amount of changes would outweight the benefit of fixing it Taylor would just reject a significant change with small benefit.
My work around works fine, but if this is becoming more and more a problem for more people, maybe I can try to come up with a fix that would work similarly to my workaround but from within the core.

Does this even solve the problem though? You are assuming they want to go back to the default, as opposed to the "previous"?
i.e. imagine this scenario

Point to Database Default
Point to Database Other
Run Migration
Now pointing to Database Default

Can we capture the "current/existing" DB connection as the migration starts, then set it back to that at the end?

That's exactly what we do, when we run migrations from an other command.

I am closing this pull request because it lacks sufficient explanation, tests, or both. It is difficult for us to merge pull requests without these things because the change may introduce breaking changes to the framework.
Feel free to re-submit your change with a thorough explanation of the feature and tests - integration tests are preferred over unit tests. Please include it's benefit to end users; the reasons it does not break any existing features; how it makes building web applications easier, etc.
Thanks!


integration: Switches juju to 3.0 for integration tests

The latest release of the postgresql-k8s charm is supported by juju 3.0 or newer.
The charmed-kubernetes/actions-operator@main action install juju 2.9 by default.

Heads up: I'd recommend jumping straight to 3.1 as that'll be the longest supported release until 4.0 some time next year 😄

Thanks, updated to 3.1. :)

Give me a shout if you need any assistance with the migration. Looks like you'll need to move to a strict MicroK8s track like 1.27-strict/stable or similar 🙂

I think this is failing because the juju Python dependency is pinned to a low version that isn't compatible with 3.1
juju==<IP_ADDRESS> should get you there

I think this is failing because the juju Python dependency is pinned to a low version that isn't compatible with 3.1
juju==<IP_ADDRESS> should get you there (in the tox.ini)

Awesome, seems that did the trick. Thanks!


encoding of time and time_bounds differs in compute_ann_mean results for decode_time=True

If I use xr.open_dataset with decode_times=True (the default) to open a dataset ds, then the values of both ds.time and ds[tb_name] are converted to cftime objects (tb_name=ds.time.attrs['bounds']). If I execute
ds_ann = esmlab.climatology.compute_ann_mean(ds),
then the values of ds.time are also cftime objects, but the values of ds[tb_name] are not.
Is this difference intended? I find it confusing.

@klindsay28, We've made a couple of changes in the last two weeks. And as far I can tell, the issue you pointed out is among some of the inconsistencies we addressed.

From this line ds_ann = esmlab.climatology.compute_ann_mean(ds),  I  can tell that you are not using the master branch of esmlab. If you are using Python 3+, do you mind installing esmlab from master branch
pip install git+https://github.com/NCAR/esmlab.git

or by cloning the repo and using pip install -e .?

To avoid confusion, we also changed the nomenclature of functions that used to be in the climatology.py module. There's an ongoing discussion in https://github.com/NCAR/esmlab/issues/55#issuecomment-479689907 about this nomenclature issue. Feel free to chime in.

To compute, the annual mean, you will need to use:
ds_ann = esmlab.resample(ds, freq='ann)

instead of
ds_ann = esmlab.climatology.compute_ann_mean(ds)

Sorry for the inconveniences and confusion!

@andersy005, in the thread at https://github.com/NCAR/esmlab/issues/55#issuecomment-479689907, you describe using esmlab.compute_ann_mean(ds), but here you are suggesting esmlab.resample(ds, freq='ann'). From an esmlab point of view, is there a reason to prefer one of these over the other?

That was before we made the final conclusion last Thursday. When @kmpaul, @matt-long, @jukent and I met, we recognized that to avoid unnecessary code duplication and confusion to the user, the functions could be put into three main categories:

resample
climatology
anomaly

https://github.com/NCAR/esmlab/blob/master/esmlab/core.py

When I update to the latest esmlab and replace esmlab.climatology.compute_ann_mean(ds) with esmlab.resample(ds, freq='ann'), I now get the following error message from xarray:

NotImplementedError: Resample is currently not supported along a dimension indexed by a CFTimeIndex.  For certain kinds of downsampling it may be possible to work around this by converting your time index to a DatetimeIndex using CFTimeIndex.to_datetimeindex.  Use caution when doing this however, because switching to a DatetimeIndex from a CFTimeIndex with a non-standard calendar entails a change in the calendar type, which could lead to subtle and silent errors.

The error is due to an old version of xarray. It seems like your xarray version is <0.12. You will need to upgrade to the latest version of xarray with:
pip install xarray --upgrade

esmlab's requirements.txt contains the line

xarray>=0.11.2

Is this incorrect?
I have 0.11.3, the most recent version of xarray that is available in conda.

Okay, I updated xarray to 0.12.1 using pip, and am past the NotImplementedError listed above, and am on the latest esmlab, updating ds_ann=esmlab.climatology.compute_ann_mean(ds) to ds_ann=esmlab.resample(freq='ann').
However, I still am seeing the behavior of this issue that the values of ds_ann.time are cftime objects, while the values of ds[tb_name] are not. It doesn't look like the recent developments address this.

@klindsay28, when I executed the code below, I can confirm that my time and time_bound are both decoded:
In [1]: import esmlab

In [2]: ds = esmlab.datasets.open_dataset('cesm_cice_daily')

In [3]: ds
Out[3]:
<xarray.Dataset>
Dimensions:      (d2: 2, nc: 5, ni: 6, nj: 6, nkbio: 5, nkice: 8, nksnow: 3, nvertices: 4, time: 365)
Coordinates:
    TLON         (nj, ni) float32 ...
    TLAT         (nj, ni) float32 ...
    ULON         (nj, ni) float32 ...
    ULAT         (nj, ni) float32 ...
    NCAT         (nc) float32 ...
  * time         (time) object 0061-01-02 00:00:00 ... 0062-01-01 00:00:00
Dimensions without coordinates: d2, nc, ni, nj, nkbio, nkice, nksnow, nvertices
Data variables:
    VGRDi        (nkice) float32 ...
    VGRDs        (nksnow) float32 ...

    HTN          (nj, ni) float32 ...
    HTE          (nj, ni) float32 ...
    ANGLE        (nj, ni) float32 ...

Attributes:
    title:             b.e21.B1850.f09_g17.CMIP6-piControl.001
    contents:          Diagnostic and Prognostic Variables
    source:            Los Alamos Sea Ice Model (CICE) Version 5
    time_period_freq:  day_1
    model_doi_url:     https://doi.org/10.5065/D67H1H0V
    comment:           All years have exactly 365 days
    comment2:          File written on model date 00610102
    comment3:          seconds elapsed into model date:      0
    conventions:       CF-1.0
    history:           This dataset was created on 2018-08-12 at 13:23
    io_flavor:         io_pio
    Extracted_from:    /gpfs/fs1/p/cesm/pcwg/timeseries-cmip6/b.e21.B1850.f09...

In [4]: ds.time_bounds
Out[4]:
<xarray.DataArray 'time_bounds' (time: 365, d2: 2)>
array([[cftime.DatetimeNoLeap(61, 1, 1, 0, 0, 0, 0, 5, 1),
        cftime.DatetimeNoLeap(61, 1, 2, 0, 0, 0, 0, 6, 2)],
       [cftime.DatetimeNoLeap(61, 1, 2, 0, 0, 0, 0, 6, 2),
        cftime.DatetimeNoLeap(61, 1, 3, 0, 0, 0, 0, 0, 3)],
       [cftime.DatetimeNoLeap(61, 1, 3, 0, 0, 0, 0, 0, 3),
        cftime.DatetimeNoLeap(61, 1, 4, 0, 0, 0, 0, 1, 4)],
       ...,
       [cftime.DatetimeNoLeap(61, 12, 29, 0, 0, 0, 0, 3, 363),
        cftime.DatetimeNoLeap(61, 12, 30, 0, 0, 0, 0, 4, 364)],
       [cftime.DatetimeNoLeap(61, 12, 30, 0, 0, 0, 0, 4, 364),
        cftime.DatetimeNoLeap(61, 12, 31, 0, 0, 0, 0, 5, 365)],
       [cftime.DatetimeNoLeap(61, 12, 31, 0, 0, 0, 0, 5, 365),
        cftime.DatetimeNoLeap(62, 1, 1, 0, 0, 0, 0, 6, 1)]], dtype=object)
Coordinates:
  * time     (time) object 0061-01-02 00:00:00 ... 0062-01-01 00:00:00
Dimensions without coordinates: d2
Attributes:
    long_name:  boundaries for time-averaging interval

In [5]: ds.time
Out[5]:
<xarray.DataArray 'time' (time: 365)>
array([cftime.DatetimeNoLeap(61, 1, 2, 0, 0, 0, 0, 6, 2),
       cftime.DatetimeNoLeap(61, 1, 3, 0, 0, 0, 0, 0, 3),
       cftime.DatetimeNoLeap(61, 1, 4, 0, 0, 0, 0, 1, 4), ...,
       cftime.DatetimeNoLeap(61, 12, 30, 0, 0, 0, 0, 4, 364),
       cftime.DatetimeNoLeap(61, 12, 31, 0, 0, 0, 0, 5, 365),
       cftime.DatetimeNoLeap(62, 1, 1, 0, 0, 0, 0, 6, 1)], dtype=object)
Coordinates:
  * time     (time) object 0061-01-02 00:00:00 ... 0062-01-01 00:00:00
Attributes:
    long_name:  model time
    bounds:     time_bounds

In [6]: ds_ann = esmlab.resample(ds, freq='ann')
/Users/abanihi/opt/miniconda3/envs/dev/lib/python3.6/site-packages/xarray/core/nanops.py:159: RuntimeWarning: Mean of empty slice
  return np.nanmean(a, axis=axis, dtype=dtype)

In [7]: ds_ann
Out[7]:
<xarray.Dataset>
Dimensions:      (d2: 2, nc: 5, ni: 6, nj: 6, nkbio: 5, nkice: 8, nksnow: 3, nvertices: 4, time: 1)
Coordinates:
  * time         (time) object 0061-07-02 12:00:00
    TLON         (nj, ni) float32 320.5625 321.6875 ... 325.0625 326.1875
    TLAT         (nj, ni) float32 -79.22052 -79.22052 ... -76.54944 -76.54944
    ULON         (nj, ni) float32 321.125 322.25 323.375 ... 325.625 326.75
    ULAT         (nj, ni) float32 -78.952896 -78.952896 ... -76.28169 -76.28169
    NCAT         (nc) float32 0.6445072 1.3914335 2.4701793 4.567288 100000000.0
Dimensions without coordinates: d2, nc, ni, nj, nkbio, nkice, nksnow, nvertices
Data variables:
    time_bounds  (time, d2) float64 2.19e+04 2.226e+04
    aicen_d      (time, nc, nj, ni) float64 nan nan nan ... 7.933e-05 8.486e-05
    VGRDi        (nkice) float32 1.0 2.0 3.0 4.0 5.0 6.0 7.0 8.0
    VGRDs        (nksnow) float32 1.0 2.0 3.0
    VGRDb        (nkbio) float32 1.0 2.0 3.0 4.0 5.0
    tmask        (nj, ni) float32 0.0 0.0 0.0 0.0 0.0 ... 1.0 1.0 1.0 1.0 1.0
    tarea        (nj, ni) float32<PHONE_NUMBER>.0<PHONE_NUMBER>.0 ...<PHONE_NUMBER>.0
    uarea        (nj, ni) float32<PHONE_NUMBER>.0<PHONE_NUMBER>.0 ...<PHONE_NUMBER>.0
    dxt          (nj, ni) float32 23968.484 23968.484 ... 29094.156 29094.156
    dyt          (nj, ni) float32 59395.453 59395.453 ... 59395.453 59395.453
    dxu          (nj, ni) float32 23966.3 23966.3 ... 29661.271 29661.271
    dyu          (nj, ni) float32 59395.453 59395.453 ... 59395.453 59395.453
    HTN          (nj, ni) float32 23966.3 23966.3 ... 29661.271 29661.271
    HTE          (nj, ni) float32 59395.453 59395.453 ... 59395.453 59395.453
    ANGLE        (nj, ni) float32 0.0 0.0 0.0 0.0 0.0 ... 0.0 0.0 0.0 0.0 0.0
    ANGLET       (nj, ni) float32 0.0 0.0 0.0 0.0 0.0 ... 0.0 0.0 0.0 0.0 0.0
    lont_bounds  (nj, ni, nvertices) float32 320.0 321.125 ... 326.75 325.625
    latt_bounds  (nj, ni, nvertices) float32 -79.48714 -79.48714 ... -76.28169
    lonu_bounds  (nj, ni, nvertices) float32 320.5625 321.6875 ... 326.1875
    latu_bounds  (nj, ni, nvertices) float32 -79.22052 -79.22052 ... -76.01522
Attributes:
    history:  \n2019-04-08 16:48:00.562852 esmlab.resample(<DATASET>, freq="a...

In [8]: ds_ann.time
Out[8]:
<xarray.DataArray 'time' (time: 1)>
array([cftime.DatetimeNoLeap(61, 7, 2, 12, 0, 0, 0, 5, 183)], dtype=object)
Coordinates:
  * time     (time) object 0061-07-02 12:00:00
Attributes:
    long_name:  model time
    bounds:     time_bounds

In [9]: ds_ann.time_bounds
Out[9]:
<xarray.DataArray 'time_bounds' (time: 1, d2: 2)>
array([[21900., 22265.]])
Coordinates:
  * time     (time) object 0061-07-02 12:00:00
Dimensions without coordinates: d2
Attributes:
    long_name:  boundaries for time-averaging interval

Can you post a small snippet of your computation here for debugging purposes?

@klindsay28, never mind.. I was quick to jump to a conclusion. You are absolutely right
In [4]: ds.time_bounds
Out[4]:
<xarray.DataArray 'time_bounds' (time: 365, d2: 2)>
array([[cftime.DatetimeNoLeap(61, 1, 1, 0, 0, 0, 0, 5, 1),
        cftime.DatetimeNoLeap(61, 1, 2, 0, 0, 0, 0, 6, 2)],
       [cftime.DatetimeNoLeap(61, 1, 2, 0, 0, 0, 0, 6, 2),
        cftime.DatetimeNoLeap(61, 1, 3, 0, 0, 0, 0, 0, 3)],
       [cftime.DatetimeNoLeap(61, 1, 3, 0, 0, 0, 0, 0, 3),
        cftime.DatetimeNoLeap(61, 1, 4, 0, 0, 0, 0, 1, 4)],
       ...,
       [cftime.DatetimeNoLeap(61, 12, 29, 0, 0, 0, 0, 3, 363),
        cftime.DatetimeNoLeap(61, 12, 30, 0, 0, 0, 0, 4, 364)],
       [cftime.DatetimeNoLeap(61, 12, 30, 0, 0, 0, 0, 4, 364),
        cftime.DatetimeNoLeap(61, 12, 31, 0, 0, 0, 0, 5, 365)],
       [cftime.DatetimeNoLeap(61, 12, 31, 0, 0, 0, 0, 5, 365),
        cftime.DatetimeNoLeap(62, 1, 1, 0, 0, 0, 0, 6, 1)]], dtype=object)
Coordinates:
  * time     (time) object 0061-01-02 00:00:00 ... 0062-01-01 00:00:00
Dimensions without coordinates: d2
Attributes:
    long_name:  boundaries for time-averaging interval

In [9]: ds_ann.time_bounds
Out[9]:
<xarray.DataArray 'time_bounds' (time: 1, d2: 2)>
array([[21900., 22265.]])

I will fix this by tomorrow


add get file contents and index

extend Drive tool to pull contents for LLM
https://developers.google.com/drive/api/guides/manage-downloads

Closing in favor of Google Drive Tool


Add dataset cards

@streino Agreed. We'll need to iterate on the design later anyway. ATM I'm just bootstrapping stuff with an acceptable look.


⚠ Bofrost.fr has degraded performance

In 940dae1, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1383 ms

Resolved: Bofrost.fr performance has improved in 28f03b9 after 6 minutes.


[Automation] Generate Fluent Lite from newrelic#package-2024-01-01

[Automation] Generate Fluent Lite from newrelic#package-2024-01-01

/azp run java - newrelicobservability - ci

API change check
APIView has identified API level changes in this PR and created following API reviews.
com.azure.resourcemanager:azure-resourcemanager-newrelicobservability


Added SVG version of MNTL Logo for AssetMantle

Added SVG version of MNTL Logo for AssetMantle

will resubmit with a better SVG file.


Revert "release 9.9.40"

Reverts F5Networks/f5-openstack-agent#1762

Do we need to close this pr?


How to integrate f7 router with react-router

[ ] bug
[ ] enhancement
[x] feature-discussion (RFC)

Expected Behavior
I want to integrate f7 router with react-router, but i am not sure is it possible?

This question is far more outside of this repo


Implement event driven logic as decorator

Currently this logic is encapsulated into all adapters. Implementing it as a decorator would leave it transparent, but make it optional (currently it is not). Also would make the code cleaner. It would probably require some minor structural change in the code.
The point is that decoupling event handling from adapters would make this a plain adapter package, where event layer is an extra on top of it.
What do you think?

I support the proposal...particularly the last sentence, I prefer a plain adapter package too.

+1

@egeloen ping
Also, this would make #64 invalid

Sorry for the delay, I'm a little bit busy currently. Regarding the proposition, I have mostly making it mandatory in order to solve common use cases for all adapters in the exact same way. For example, it is difficult to handle the redirect for all adapters with the same behavior for all of them.
Anyway, If everyone agrees to move it as decorator, I'm :+1: too.

@egeloen IMO that's why PSR7 will be an awesome thing when it gets ready. You will be able to do things the same way.
Anyway, this is something that bugs me for a long time now. This is why I originally started my own adapter package, but I ended up here when you removed events being mandatory. So I am so much :+1: for having it as a decorator.

Will this be part of any near release?


Improve before-index performance

Currently it just uses srange to do insertion but can be done more efficiently.

Also make a benchmark comparing before-index against srange to show improvement.


[Application Graph API] Permissions enforcement

Overview of feature request
As a Radius user, I am only allowed to retrieve application graph data for which I have permission to view through the cloud provider credentials I had set up through the rad CLI and any relevant RBAC checks done by the Radius control plane. These permission checks should be enforced in both the application graph API as well as the rad CLI (e.g. rad app connections) This ensures that security and confidentiality policies within my organization can be enforced.
Acceptance criteria

Upon making a request to the Applicaiton Graph API, the user receives a response containing only the application graph data for which they have permissions.

Additional context
related to #6900
AB#10640

ado item is closed


Extended stack trace in REPL?

For example -- I get an error like this:
MysqlDataTruncation Data truncation: Data too long for column 'X' at row 1  com.mysql.jdbc.MysqlIO.checkErrorPacket (MysqlIO.java:4206)

However, the normal stack trace and error message includes the table, etc.
How is this done with proto-repl?

@shoxter in general in Clojure you can always get the latest exception by calling *e , see https://clojuredocs.org/clojure.core/*e
In general people tend to use a stacktrace library like clojure.repl clojure.stacktrace, clj-stacktrace or aviso/pretty (off the top of my head). If you dont want to have to choose among all of those simply call (clojure.repl/pst) since you dont need any extra dependencies ;)

Thank you very much. (println *e) solved my issue.
I appreciate your help friend.


pkg/scheduled_test/test/unittest_compatibility_test failing on command line js engines

This is failing on jsshell and d8 after r33042
Marking this as high priority since I don't know if this has other implications on the unittest framework.

This issue has been moved to dart-lang/test#245.


Consolidate multiple implementations of JavaScript evaluation

I believe we have multiple implements of javaScript evalutation in the codebase:

constant folding in the minifier
evaluation of enum values in the transformer
a bit of evaluation in linter/tree_shaking

There may be more in downstream tools.

Should I re-open my const eval PR?

Should I re-open my const eval PR?

That's a whole runtime evaluation, but goal is to detangle the constant evaluation mess 😢

Started moving constant evaluation code in https://github.com/oxc-project/oxc/pull/6549
Will continue the move.


Adding new coinbase

I wanted to change the coinbase on coda daemon witth the following command:
coda daemon -coinbase-receiver 4vsRCVR4HPZcKRehTPWYVhAaRiKFTPkCb6wqPoLj3g4GA4mPwCGbWh6Wyb8FpmPLRVYKjMq7XHccGyonYsiPFnqDR2bhuyEvUXpxdGEn7z6J92xQxtUmfXTzHjWMLh3JKiHqAbUsqsiYgA4b
I get the following output:
2020-01-25 19:10:44 UTC [Info] Coda daemon is booting up; built with commit "1dffa7cc1438d3e39eb7182b812e882e4a94a7d7" on branch "master"
2020-01-25 19:10:44 UTC [Info] Daemon will expire at "2024-12-10 14:00:00-07:00"
2020-01-25 19:10:44 UTC [Info] Booting may take several seconds, please wait
2020-01-25 19:10:44 UTC [Info] Looking for the genesis tar file "coda_genesis_1dffa7cc_5f69e1300e9b0392.tar.gz"
2020-01-25 19:10:44 UTC [Fatal] Error loading the genesis ledger from $dir: $error
	dir: "/home/ilhan/.coda-config/coda_genesis_1dffa7cc_5f69e1300e9b0392/ledger"
	error: "(Failure\n \"IO error: While lock file: /home/ilhan/.coda-config/coda_genesis_1dffa7cc_5f69e1300e9b0392/ledger/LOCK: Resource temporarily unavailable\")"

This is my systemd file:
[Unit]
Description=Coda Daemon Service
After=network.target
StartLimitIntervalSec=60
StartLimitBurst=3

[Service]
EnvironmentFile=-/etc/default/coda-config
Type=simple
Restart=always
RestartSec=5
User=ilhan
ExecStart=/usr/local/bin/coda daemon $DAEMON_OPTIONS $DAEMON_PEERS $WALLET $SNARK $SNARK_FEE
ExecStop=/usr/bin/pkill -f 'coda daemon'

[Install]
WantedBy=multi-user.target

Closing this issue as it is stale for a long time.


Display risk calculated at on risk cards (DEV)

To prevent confusion, the timestamp shown is changed to the one from the in-app calculation.

obsolete


Do setup ESLint required to use this plugin ?

Error:
./src/App.js
  Line 7:  'pug' is not defined  no-undef

Search for the keywords to learn more about each error.

Create-react-app : 1.5.2
Babel-plugin-transform-react-pug: ^6.0.0

Nvm after eject and reinstall eslint. Then clear all error by eslint such as : React (unsed-var), fetch/pug (not-defined). It's worked. React docs or babel plugin transform react docs should have mention about if error detect by eslint it won't compile. I leave my repo bellow as reference

@PhmNgocNghia hey, we've already mentioned this in the doc: https://github.com/pugjs/babel-plugin-transform-react-pug#eslint-integration
Let me know if it requires more detailed explanation.


Added maximum volume parameter that lets users set up to 1000% volume.

They need a permission however otherwise it defaults to 100%.
I made this because I just wanted to have the freedom to go over 100%. I originally modified it just for myself but I decided to add permissions for it and a config option and everything, then fork and make a pull request it so it could be in the main repo.
Basically, if a user/group has "AllowHigherVolume = yes", then when they adjust the volume the maximum limit is what the user sets in the config (default: 100%). So if someone set the MaxVolume option to 500, they could type for example "!volume 500" if they have the permission. However, if someone has "AllowHigherVolume = no", then they're limited to 100% no matter what.
Log:
Set player max volume
setting up permissions, config option
Working, adding permissions
Permissions working, adding config check
Config error handling handled correctly

Bobe?

https://github.com/Just-Some-Bots/MusicBot/pull/861

@KingGold171 I'm not sure what you're trying to say, It's just for making the bot ridiculously loud pretty much, anything higher than 100% (in the bot) distorts.

Pr's to master will always be closed. Please use the review branch

@MattBSG Yeah sorry about that, I just merged it all to review, it's set correctly now.


How to enable conversation commands? Just started using this bot. TIA

Please don't do this as a GitHub issue. Join the Discord Server and we'll help you out.


json/decode on empty string should return nil

json/decode on an empty string currently throws an error:
decode error at position 0: unexpected end of source

I think it should instead return nil.

I disagree.


Vulnerability roundup 79: libxslt-1.1.33: 1 advisory

search, files

[ ] CVE-2019-18197 CVSSv3=9.8 (nixos-19.03)

Scanned versions: nixos-19.03: d1dff0bcd9f. May contain false positives.

See also: #73665


Task - Benchmark API request timing

Description
Use clockwork to help improve site performance.

Duplicated by #5569


Add switches for disabling TempFileCleanup.bat, SFC and the Repair File Extensions Stage

Hey there.
Firstly, thanks to vocatus and the community for a great tool. I've spent the last day or so going through it in detail and it looks incredibly promising. However, I've since noticed a few things I'd like to prevent from running; these are the System File Checker, the execution of TempFileCleanup.bat, and the Repair File Extensions step.
Looking into the source code of TempFileCleanup.bat tells me that some things it's doing are a little too aggressive for my liking, from deleting browser temp files and useful logs, to deleting default Windows 7 backgrounds, to emptying search caches and the Recycle Bin.
Likewise with Repairing File Extensions; I really don't want to undo years of assigning extensions to certain programs by having everything restored to factory settings, especially when I'm certain nothing about the extensions are broken. This again feels more aggressive than it needs to be, especially without a switch to disable the behaviour.
I've been unable to find switches in the documentation to disable the above steps. Is it possible that these can be added to Tron?
For now, I'm making do with editing the batch files to remove/alter the respective stages directly, but this is a short-term solution and it would be really great to be maintain a single copy of Tron and use command-line switches to disable any of the above behaviours as needed.
Thank you.

Why do you want to disable SFC?  SFC should never hurt anything/change settings/etc.

Hi @Kaos-Industries, thanks for the thoughts.
For now I'm going to leave everything as-is, since Tron is intended for system rescue and not as a regular maintenance tool. In a rescue environment it makes more sense to reset file extensions (frequently hijacked), run SFC, and wipe cache/recycle bin/etc.
Of course you're welcome to tweak your copy of Tron so that it doesn't perform those actions; crack open any of the scripts and disable or delete whatever you don't like.

Tron is useful for more than just rescuing system though, it is good for any Windows system that needs a performance boost from a good cleaning (a.k.a. almost all of them. ;)


[Enhancement] check storage medium consistency when strict mode is active

allowing storage medium check when strict mode is active
What type of PR is this:

[ ] BugFix
[ ] Feature
[X] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[ ] I have checked the version labels which the pr will be auto-backported to the target branch

[ ] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE Incremental Coverage Report]
:disappointed: fail : 11 / 18 (61.11%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/ClusterLoadStatistic.java
0
3
00.00%
[215, 216, 217]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
10
14
71.43%
[1155, 1156, 1164, 1165]

:large_blue_circle:
com/starrocks/system/SystemInfoService.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 11 / 18 (61.11%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/ClusterLoadStatistic.java
0
3
00.00%
[215, 216, 217]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
10
14
71.43%
[1155, 1156, 1164, 1165]

:large_blue_circle:
com/starrocks/system/SystemInfoService.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)

[FE Incremental Coverage Report]
:disappointed: fail : 12 / 17 (70.59%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/clone/ClusterLoadStatistic.java
1
6
16.67%
[168, 170, 171, 173, 217]

:large_blue_circle:
com/starrocks/clone/TabletScheduler.java
11
11
100.00%
[]

[BE Incremental Coverage Report]
:heart_eyes: pass : 0 / 0 (0%)


Added a vertical Navbar to the left side

@EshanTrivedi21 Please merge my pull-request.

Hey @Aishaa-26 nice work,
but the sidebar menu and Main content is overlapping.

Suggesting you to reduce the width from 250px to 200px for #cssmenu and #cssmenu > ul

and you can also work on changing it for the dark mode.

Sure will solve this....Thanks for the input.


provider url has a uniqueness validation so the factory should reflect that

url is delegated: https://github.com/ManageIQ/manageiq/blob/master/app/models/provider.rb#L17 which has a uniqueness validation: https://github.com/ManageIQ/manageiq/blob/master/app/models/endpoint.rb#L9 so maybe the factory shouldn't be setting ones that won't be unique?
@miq-bot assign @jrafanie

👍

Awesome


[Extensions] Visualizer missing namespace count

Current Behavior

Contributor Guides and Handbook

📚 Meshery Build and Release Strategy
🎨 UI Design For Meshery Extensions Figma in Figma (open invite)
📚 Meshery Extensibility Providers
⌨ Meshery API Docs
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

This issue has been fixed.


Assertion failure in -[MBProgressHUD initWithView:], Pods/MBProgressHUD/MBProgressHUD.m:212

i have a error in line 212
0x0000000183aaedc8  + 148
1   libobjc.A.dylib                     0x0000000183113f80 objc_exception_throw + 56
2   CoreFoundation                      0x0000000183aaec80  + 0
3   Foundation                          0x0000000184434154  + 112
4   YouPinHui                           0x00000001009efef0 -[MBProgressHUD initWithView:] + 268
5   YouPinHui                           0x00000001009eef50 +[MBProgressHUD showHUDAddedTo:animated:] + 100

What exactly is the exception message?

Closing due to inactivity.

Thank you very much. I've solved it, because I added it to nil view


Auto-controlled props emit spurious warning when set to undefined (missing undefined check).

Steps
For any auto-controlled property x, set both x and defaultX to undefined. For example, open and value in Dropdown.
Expected Result
These values should be treated as if they are undefined/do not exist.
Actual Result
Warning: Dropdown prop "open" is auto controlled. Specify either defaultOpen or open, but not both.
Version
0.68.0
Testcase
http://codepen.io/anon/pen/xdGPxz?editors=1111
Suggested Fix
Here in AutoControlledComponent.js, replace:
  if (defaultPropName in this.props && prop in this.props) { ... }

by
  if (_.has(this.props, defaultPropName)
      && !_.isUndefined(this.props[defaultPropName])
      && _.has(this.props, prop)
      && !_.isUndefined(this.props[props]) {
    ...
  }

In case anyone is wondering, "why on earth would you set open and defaultOpen to undefined?"...
The answer is interop with other languages targeting JS. For example, using ScalaJS with a Props facade type. When constructing an instance of Props, all fields must be provided/default to js.undefined.

At first glance, I think this will still pass tests and be backward compatible.  In fact, we likely should've been treating undefined as not present to begin with.
Let's get a PR and see what happens.  Note, the logic can be reduced to this as we only care that they are undefined, not that they are present in the objects at all:
if (!_.isUndefined(this.props[defaultPropName]) && !_.isUndefined(this.props[prop]) {
  ...
}

Hey @levithomason and @DylanLukes I am looking to contribute. I am guessing this issue is still open and is needing to be done, so I would love to help!


How can I not show the summoner spell in europe layout?

I don't want to show the summoner spell in europe layout.
Is there any script like simple layout I can edit to do this?

You can set spellsEnabled to false in backend\config.json and save the file. The UI reflects that change without reloading, so you can show/hide spells whenever needed


Reference script and material...?

I'm not a Medium member so I can't comment there.
I created the script and now you're just running straight away.

We’ll add the call to the server in GetColor() later
Once we’ve added the code we should see a field in the Inspector to add the Material. Let’s add the CubeMat to that field

Where how we added the code? I have edited the .cs file with VS Code.
What now?

Hi @dalu sorry for the delay in replying
When you got to that point in the Medium article did you also create the ColorRandomizer.cs file and add this code block to it?
using UnityEngine;

public class ColorRandomizer : MonoBehaviour {

    public Material material;

    private readonly Color _defaultColor = Color.red;

    public void ChangeColor() {
        material.color = GetColor(material.color);
    }

    //We'll use this method to get a color from the server later
    private Color GetColor(Color currentColor) {
        Debug.Log("Got a request to change the color");
        return _defaultColor;
    }
}

You need to then attach that script to an Empty GameObject
Once you do that you should see a field in the inspector to add the Material

You're not writing that anywhere in the post.
You're writing:

The script will need a reference to the Material that we created earlier and a public method for changing the color that we’ll link with the Randomize Button’s OnClick handler

Add the code below to the ColorRandomizer.cs you just created above

In the code above you’ll notice that all we’re doing in GetColor() outputting a message to Debug.Log() and setting the colour to Red. Don’t worry we’ll add the call to the server later

Now save ColorRandomizer.cs and you should see a field in the Inspector to add the Material. Let’s add the CubeMat to that field

And then you're already testing. No mention about any GameObject or whatever, but all of a sudden there's a Randomizer in the left panel
So now I did:

create empty GameObject
rename to Randomizer
attach ColorRandomizer script
attach CubeMat to script
in the onClick of the Button I click + and reference the ColorRandomizer script, also set "Runtime Only" "No function" and below is the ColorRandomizer reference

Ok the linking the onClick handler with something was explained in this video:
https://www.youtube.com/watch?v=TYzdhiRiKd0
But you know what would be really interesting?
Doing the same with the EntitySystem. Since ECS has data (struct types) and data processors that should get interesting with grpc.
Anyhow, thanks for the tutorial and response.


Enable workflow item deletion without publishing it (DS-3926)

The whole issue is described here:
https://jira.duraspace.org/browse/DS-3926

Closing and opening to restart tests

@santit96, it's been a while since you submitted, but a belated thanks for the pr
I am having some difficulty actually testing the functionality described, I haven't used the workflow process very much
could you explain how to use the described feature in this PR?

Thanks @santit96
I don't know why but I don't get that option, I am pretty sure I have done everything correctly, I am building your branch with the two commits and using docker-compose.

these are the only, and standard options I get.
Is it possible this branch is missing a commit?

@santit96  should there perhaps be something in either
https://github.com/DSpace/DSpace/blob/dspace-6_x/dspace-xmlui/src/main/resources/aspects/Workflow/workflow.js
or
https://github.com/DSpace/DSpace/blob/dspace-6_x/dspace-xmlui/src/main/resources/aspects/XMLWorkflow/xmlworkflow.js
?

Or possibly, rather
https://github.com/DSpace/DSpace/blob/dspace-6_x/dspace-xmlui/src/main/java/org/dspace/app/xmlui/aspect/workflow/PerformTaskStep.java

@J4bbi I'm pretty sure that i have pushed all the commits, but i made this PR long ago so i'll check just in case!

@santit96, can you also confirm if this PR is intended for configurable xmlworkflow or the basic workflow?
see: https://wiki.lyrasis.org/display/DSDOC6x/Configurable+Workflow

It is for configurable xmlworkflow

I checked the feature again with the branch of this PR (also checked that i have the same commits this PR has), and everything works fine for me. I seems to me that there is a permissions issue. Perhaps the problem is that the the group assigned to WF2 Accept/Reject/Edit Metadata Step does not have DELETE permissions over the items, you can try giving DELETE permissions over the collection to that particular admin.

@santit96 and @J4bbi :  I'd recommend this PR be updated to go against main and DSpace 7.x.  It looks, to me, like it's adding a new feature to Configurable Workflow (the ability to delete a workflowitem while it's in workflow) and XMLUI.  Since this feature doesn't exist in DSpace 7.0, adding it into 6.x this late is potentially problematic unless it's also added to 7.x.
Therefore, I'd like to ask that you consider adding this into 7.x instead of 6.x.  In the 7.0 platform, it is possible for Administrators in the system to delete any item in Workflow (which would solve the DS-3926 ticket in a different manner).  But, I don't believe it's possible yet for reviewers to delete items currently in workflow.


Small fixes found during testing

Can be closed?

Now that I'm no longer actively testing this integration, closing this convenience issue.


Fix specifying options in (mis)Optimization

The long argument name options specified to objdump require
two dashes '--'.

Fixes https://github.com/rust-embedded/discovery/issues/284 .

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @jamesmunns (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Did a git grep objdump and seemed that Chapter 9.2 NOP also required a -- in the options.
This is following up fixes from https://github.com/rust-embedded/discovery/pull/281 which fixed Chapter 7.5 Type safe manipulation.


All entra m365group commands should accept displayName option

Closes #6147

Thank you @MartinM85 we'll try to review it ASAP

Hi @MartinM85, could you resolve the merge conflicts by rebasing with the latest main, please?

@milanholemans Some commands are using groupName instead of groupDisplayName. What's the correct option name? Still groupDisplayName?

We've renamed a bunch of commands from groupDisplayName to groupName https://github.com/pnp/cli-microsoft365/issues/5616
So groupName is the option name we should use.

@MartinM85 may I kindly ask you to rebase the PR again before I may proceed with the review.
@milanholemans if we in this PR we are changing the groupDisplayName to groupName like in docs/docs/cmd/entra/m365group/m365group-conversation-post-list.mdx isn't this breaking change and this should be a pr-major as we may only do that now with the v10 release?

@MartinM85 may I kindly ask you to rebase the PR again before I may proceed with the review. @milanholemans if we in this PR we are changing the groupDisplayName to groupName like in docs/docs/cmd/entra/m365group/m365group-conversation-post-list.mdx isn't this breaking change and this should be a pr-major as we may only do that now with the v10 release?

That wasn't really the purpose of the issue, but the commands should be aligned indeed. You are right.

@Adam-it Hope I've rebased everything correctly

That wasn't really the purpose of the issue, but the commands should be aligned indeed. You are right.

Ok I will add pr-major label to this PR since now it is a breaking change

Hi @Adam-it, I had a typo in dispalyName.

Hi @Adam-it, I had a typo in dispalyName.

Thanks for the quick turn around. I will try to make more reviews this week and for sure will retake this one as well

Read to merge 🚀

Merged manually. Thank you for your awesome work! You Rock !🤩
@MartinM85 I noticed that you have been doing a looooot of awesome stuff in CLI repo 🚀. I was wondering if you would like to present some of your work, what you added to the CLI, or how you go about it, or why you do it, or how you use CLI.. etc to the community on one of the PnP Community calls?

@Adam-it I can try to present some of my work on the PnP Community calls, but I can't say now what exactly. :)

@Adam-it I can try to present some of my work on the PnP Community calls, but I can't say now what exactly. :)

that's awesome to hear 🤩. You may more than one demo if you want 😉. There are plenty of awesome stuff you did that may be presented 👏
May I kindly ask you to fill out the form so that we may set a demo spot for you
https://forms.office.com/pages/responsepage.aspx?id=v4j5cvGGr0GRqy180BHbR8ke1rGfE-VNsUHrnMWCrL5UNjhIVEc4VERQUDNYOTM4MDdKN1ZLWFVBMSQlQCN0PWcu


Add GlobalMaxPool3D layer

Currently, the support for GlobalMaxPool3D is missing and it would be great if we could add support for this layer as well. The desired PR addressing this issue should include:

Implementation of layer class named as GlobalMaxPool3D (you can take inspiration from implementation of GlobalMaxPool1D as reference)
Documentation of layer.
Test cases
Support for export of layer to JSON (see ModelSaver.kt)
Support for import of layer from JSON (see ModelLoader.kt)

Also, if needed, you can take a look at Keras documentation for this layer. NOTE: for the moment, there is no need to add support for "data format" (i.e. channels last vs. channels first) in your PR; you can simply assume the channels are always in the last dimension.

So, are you going to create PR by yourself or we will mark it with the "good first issue"?


[AutoPR cognitiveservices/data-plane/ComputerVision] ComputerVision add tag.hint, make scalars non-nullable

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/3259

Can one of the admins verify this patch?

This PR has been merged into https://github.com/Azure/azure-sdk-for-java/pull/2120


Optimize performance of maps

This PR optimizes the compiler's use of objects as maps as follows:

Require the host to be ECMAScript 5 compliant and create maps using Object.create(null). This ensures maps have no prototype (and thus no inherited members).
Apply the delete operator to every newly created map object. This causes V8 to put the object in "dictionary mode" and disables creation of hidden classes which are very expensive for objects that are constantly changing shape.
Skip hasOwnProperty calls on property lookups. They're no longer necessary when objects are known to have no prototype.

The PR introduces a new MapLike<T> type that represents a regular object used as a lookup (e.g. an object literal). A MapLike<T> requires use of hasOwnProperty to determine whether a particular property was inherited from the prototype.
The Map<T> type now has a brand and Map<T> instances can only be created by calling createMap<T>() (which in turn calls Object.create(null) and puts the object into dictionary mode).
Performance impact on the Monaco real world code base is as follows:

15% overall speed increase (6.7s vs. 7.9s).
19% less memory consumed (211Mb vs. 261Mb).

Definitely a worthwhile optimization, provided we're ok with dropping support for ECMAScript 3 hosts.

:+1:, logged #10278

@ahejlsberg #10298 is an alternative to this and is a port to master of what I was working on for the transforms branch. It has the same improvements in speed and memory usage as this PR but also keeps support for ES3.

👍 following the discussion in today's design meeting. I've closed #10298 in favor of this. I'll follow up with a separate PR for the isolated cases that should also be switched from MapLike to Map.

from this PR , my project got lots of improvement on compile time and memory used .
before :
after:


Timestamps in dataset change to incorrect format after training model.

Hey guys,
I'm currently training a model with historical data for a single stock. I'm using the quickstart tutorial in the AutoGluon Docs.
The model trains successfully, however the timestamps for my predictions change to much earlier timestamps. The first timestamp in my data is in the year 2005, however this function shows the first timestamp being 1900:
predictions = predictor.predict(train_data)
predictions.head()

Also, the database is supposed to show timestamps in daily intervals, however the intervals turn to every second.
I should mention that I have set ignore_time_index=True since the data has a lot of missing days, and I don't know whether this could have anything to do with the issue. Also, since the dataset didn't come with any ID column, I manually created one and set an ID.
EDIT: Hey, I just read this in the documentation for when I use ignore_time_index:
In this case, the predictor will completely ignore the timestamps in train_data, and the predictions made by the model will have a dummy timestamp index with frequency equal to 1 second. Also, the seasonality will be disabled for models like as ETS and ARIMA.
How to account for this in my prediction and ensure I get a correctly formatted output?
Thanks!
Thanks!

Since the training data doesn't have regularly sampled timestamps, TimeSeriesPredictor doesn't have a way of knowing what timestamps to assign to the prediction (what if the next prediction_length timestamps also have some missing values because of weekends/holidays?).
You can try forward-filling the training data to make sure that it has a regular frequency:
def forward_fill_missing(ts: TimeSeriesDataFrame, freq="D") -> TimeSeriesDataFrame:
    original_index = ts.index.get_level_values("timestamp")
    start = original_index[0]
    end = original_index[-1]
    filled_index = pd.date_range(start=start, end=end, freq=freq, name="timestamp")
    return ts.droplevel("item_id").reindex(filled_index, method="ffill")

df = df.groupby("item_id").apply(forward_fill_missing)

Thanks for the quick reply!
If I do this, will I need to set ignore_time_index=True?
Also, I'm currently using your code, however I'm getting an error:
KeyError: 'item_id.
For reference, my current dataset (before applying the forward_fill_missing looks like this:

I figured out how to this. I was using tf as a variable instead of changing it to what I had set. Thank you so much!


404 Contribution Guidelines not found

Description

[ ] The link on the README.md for contribution guidelines returns 404
[ ] There is no CONTRIBUTING.md file in the repo.

Link to Reproduction (or Detailed Explanation)
https://github.com/chakra-ui/ark/blob/main/CONTRIBUTING.md
Steps to Reproduce
This is the line in README.md - https://github.com/chakra-ui/ark/blob/5b1b0d50e8b192c0a0df3d26f3c1483476fbbb27/README.md?plain=1#L63
Ark UI Version
1.0.1
Framework

[ ] React
[ ] Solid
[ ] Vue

Browser
No response
Additional Information
No response

@MathurAditya724
Thanks for pointing out the missing CONTRIBUTING.md. I've added a base version. Feel free to contribute. Any help is appreciated


Additional Focus Report Changes

Focus report updated and additional versions for review created
Files Changed
home\views.py
home\urls.py
home\templates\reports\ProjectFocusTopicInfo.html
home\templates\reports\ProjectPartnerInfo.html
UnoCPI\sqlfiles.py
diff options available
projectInfo_old.html - original report
projectInfo.html - Focus area only with revised query
projectTopicInfo.html - Focus and Topic details with mission counts tied to projectmissionsubcategory
primaryFocusTopicInfo.html - Focus and Topic details with missions counts tied to projectmission

merged


How do we want to allow extending "parts of the page"?

Right now, I've taken the approach of having a Jinja2 block that "wraps" an include of a template file named elements/{whatever}.html. The wrapper tags have classes that are directly used in the skeleton.css file.
This is what it looks like right now:
https://github.com/pradyunsg/sphinx-basic-ng/blob/678a10d27891ae1df58c584d0a55775204308cff/src/sphinx_basic_ng/theme/basic-ng/page.html#L36-L40

that seems reasonable to me 👍 perhaps we should close this issue once this is documented?

This now has a placeholder in the documentation, and likely doesn't need a dedicated issue to track filling that placeholder. :)


[BitwiseCopyable] Add a marker protocol.

Behind the BitwiseCopyable experimental feature flag.

@swift-ci please test

@swift-ci please test

@swift-ci please test

@atrick You wrote

we'll probably need to make it available under a production feature before libraries, like swift-collections, can start building on top of it.

To do that, it seems that we'd need to make the BitwiseCopyable protocol be defined in production stdlibs.
At the moment, this patch is keying off the asserts-ness of the stdlib build to decide whether to include the protocol, in order to avoid adding it to production stdlibs.
The options that come to mind are

live for now with a new public protocol in the stdlib which will eventually get proposed
add an underscore to the protocol
add an underscored attribute to start with

The last of these seems unfortunate.
@atrick , @kavon Any preference among these options or other options that I'm missing?

Now using a different option which makes the protocol useable by clients regardless of the compiler's asserts-ness but with future availability:
#if $BitwiseCopyable
@available(SwiftStdlib 5.11, *)
@_marker public protocol BitwiseCopyable {}
#endif

Now
#if $BitwiseCopyable
@_marker public protocol _BitwiseCopyable {}
#endif

@swift-ci please test


ErrorCode: integrated_windows_auth_not_supported_managed_user

I am trying to use the credential provider from PowerShell via the Package Management Cmdlets. The authentication keeps failing and it generates personal access token with vso.drop_write scope for every attempt.
Repro Steps:

Install credential manager via provided PowerShell
Register my Azure Artifacts provider
Find package

./installcredprovider.ps1
Register-PackageSource -Name $artifactFeedName -Location $artifactFeed -ProviderName NuGet -SkipValidate
Find-Package -Name $artifactName -Source $artifactFeedName

Error Message is repeated.
After every attempt to sign in it creates a new personal access token and fails.
   at Microsoft.IdentityModel.Clients.ActiveDirectory.Internal.Flows.AcquireTokenIWAHandler.PreTokenRequestAsync()
   at Microsoft.IdentityModel.Clients.ActiveDirectory.Internal.Flows.AcquireTokenHandlerBase.RunAsync()
   at Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContext.AcquireTokenCommonAsync(String resource, String clientId, IntegratedWindowsAuthInput iwaInput)
   at Microsoft.IdentityModel.Clients.ActiveDirectory.AuthenticationContextIntegratedAuthExtensions.AcquireTokenAsync(AuthenticationContext ctx, String resource, String clientId, UserCredential userCredential)
   at NuGetCredentialProvider.CredentialProviders.Vsts.AdalTokenProvider.AcquireTokenWithWindowsIntegratedAuth(CancellationToken cancellationToken) in E:\A\_work\911\s\CredentialProvider.Microsoft\CredentialProviders\Vsts\AdalTokenProvider.cs:line 122
   at NuGetCredentialProvider.CredentialProviders.Vsts.WindowsIntegratedAuthBearerTokenProvider.GetTokenAsync(Uri uri, CancellationToken cancellationToken) in E:\A\_work\911\s\CredentialProvider.Microsoft\CredentialProviders\Vsts\BearerTokenProviders.cs:line 58
   at NuGetCredentialProvider.CredentialProviders.Vsts.VstsCredentialProvider.HandleRequestAsync(GetAuthenticationCredentialsRequest request, CancellationToken cancellationToken) in E:\A\_work\911\s\CredentialProvider.Microsoft\CredentialProviders\Vsts\VstsCredentialProvider.cs:line 117
        ErrorCode: integrated_windows_auth_not_supported_managed_user
[Minimal] [CredentialProvider]DeviceFlow<EMAIL_ADDRESS>[Minimal] [CredentialProvider]ATTENTION: User interaction required.

    **********************************************************************

    To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code HMKX9YF4Y to authenticate.

    **********************************************************************

System details
Windows 10 Version 2004
Credential Provider Version: <IP_ADDRESS>
PackageManagement Module Version 1.4.7
Package Provider NuGet Version <IP_ADDRESS>
My Azure AD account is cloud only. Also the account is a guest in the target tenant of Azure DevOps instance.

The way the credential provider works is that it first tries Windows integrated auth and if that doesn't work it'll fall back to interactive auth. In your case it's device flow. Just to clarify, did you try following the device flow instructions to authenticate interactively? Did that fail too?

When I finish the device flow the module creates a personal access token in Azure DevOps and then the same error is repeated and it ask me again for device flow. This results in yet another personal access token being created by the module.

Hey Guys,
Was this issue solved. I'm experiencing the same problem as well.

Hello,
I have looked into this issue once again and have tried to reproduce it locally after following your steps and set up. After testing with a newer nuget version, I haven't been able to reproduce it. If you have any more information you would like to provide, or if the issue persists after updating to a newer version of nuget, feel free to reach out. Until then this issue will remain closed, thanks!


Torque and Job Arrays - ScaleUp

We use a fair number of job arrays with torque and noticed that cfnCluster wasn't scaling up (as expected, at least) with the defined thresholds.  Traced it down to publish_pending:
[centos@ip-10-0-0-30 ~]$ qstat -i | tail -n+6 | awk '{total = total+ $7}END{print total}'
26
[centos@ip-10-0-0-30 ~]$ qstat -it | tail -n+6 | awk '{total = total+ $7}END{print total}'
618
I'm not sure if this is intended behavior, but we locally patched publish_pending:20 with the addition of the -t flag for qstat.

This is intended behaviour and we'll update or you can submit a pull request against the develop branch.

I'll put together a pull request tonight or tomorrow for you.  Thanks!

All the PRs have been included in cfncluster-1.2.1


Unified API Client.Close()

The Shutdown and Draining docs page for Go suggests using sentry.Flush where other SDKs use Close.
Then it says:

After shutdown the client cannot be used any more so make sure to only do that right before you shut down the application.

That is incorrect docs, since it doesn't match what Flush does.
We should implement Close and fix the docs to use it.
As seen in recent issues, Flush is often not the answer and is being misused.

Agree, I missed that. Close is basically Flush + enabled = false

Will be added in https://github.com/getsentry/sentry-go/pull/894.


ios本地客户端组件实现

小弟现在想在ios本地实现一个自定义组件给服务端调用,但是看了系统的组件实现,里边有协议,自己实现的话,还引用不到协议,直接把组件定义到weex文件夹还会报错,求组这种问题大概解决方向应该怎么弄呢`万分感谢

@slidoooor  可以具体描述一下么，没打理解， 哪个组件的啥协议..

@acton393  就是想自定义一个搜索栏,但是看weex源码定义的控件例如textfield之类的都遵守了协议,,就有点乱..

hi @slidoooor  weex 源码里面input (系统组件textfield) 应该没有实现啥协议，你如果需要自定义一个搜索栏组件，就直接继承WXComponent 就可以，不需要实现啥协议

This PR/issue doesn't receive response since Weex migrated to Apache two years ago and this repository is no longer active yet.
Feel free to ask question in new repository and thanks for your contribution.

This issue is outdated for a long time and will be closed now. You can create a new one if you still have questions.


Update Jenkins

Hi, how can I update Jenkins? It shows me a message that is a new version available.

Hi @chg1995
you need to update the Jenkins Dockerfile and build the image again.
https://github.com/marcelbirkner/docker-ci-tool-stack/blob/master/jenkins/Dockerfile
Greetings,
Marcel

Thank you very much for your quick response.
Greetings.


Graceful uploading error handling

For now application doesn't handle backend fail cases and just responds with 500 error code, which is not convenient. So maybe it would be nice to handle those exceptions in app and return 413 in case of size validation, or 400 when ArgumentError is thrown. It'd improve semantics and probably help handling errors on frontend.

Better error status codes would be nice, I agree. I'd love a PR for this.


Refine  cn document, test=document_preview

Refine cn document of Normal, Uniform, iou_similarity, affine_channel.


Update webdrivers gem to fix feature test errors

We need to do this every time there's a Chrome update, apparently. I'd be great if we didn't have to do this.

Coverage remained the same at 57.41% when pulling 955ca0bfed69a2aa06e3ef58106550c73eae8f5d on update-webdrivers into e9063925ed0e4bb45cee4fb9d6457465c50d3420 on develop.


Getting a timeout

docker run -v $HOME/desktop/sandbox/mmock/config -p 8082:8082 -p 8083:8083 jordimartin/mmock
Above is the line I'm using to launch mmock with the config files provided in this repo for testing. This is what I'm seeing when it is launched:
MMock v 2.0.2

		.---. .---.
               :     : o   :    me want request!
           _..-:   o :     :-.._    /
       .-''  '  `---' `---' "   ``-.
     .'   "   '  "  .    "  . '  "  `.
    :   '.---.,,.,...,.,.,.,..---.  ' ;
    `. " `.                     .' " .'
     `.  '`.                   .' ' .'
      `.    `-._           _.-' "  .'  .----.
        `. "    '"--...--"'  . ' .'  .'  o   `.
        .'`-._'    " .     " _.-'`. :       o  :
      .'      ```--.....--'''    ' `:_ o       :
    .'    "     '         "     "   ; `.;";";";'
   ;         '       "       '     . ; .' ; ; ;
  ;     '         '       '   "    .'      .-'
  '  "     "   '      "           "    _.-'

   ____    __
  / __/___/ /  ___
 / _// __/ _ \/ _ \
/___/\__/_//_/\___/ v3.2.1
High performance, minimalist Go web framework
https://echo.labstack.com
____________________________________O/_______
                                    O\
⇨ http server started on <IP_ADDRESS>:8082
2017/06/30 18:53:47 Getting external IP
2017/06/30 18:53:47 HTTP Server running at <IP_ADDRESS>:8083
2017/06/30 18:53:47 HTTPS Server running at <IP_ADDRESS>:8084
2017/06/30 18:53:47 Console running at <IP_ADDRESS>:8082
2017/06/30 18:53:47 File monitor started

However, when I ping these endpoints I get a timeout/server unable to send response.
I know this is a vague question, but I'm unsure of where I went wrong (the absolute path is where the json files are and the correct docker image is being served), so any help would be appreciated. Thanks for a great api testing tool.

Update: Due to my config it said it was running on <IP_ADDRESS> but was actually being served on <IP_ADDRESS>.


Azure sphere VS code error

Hi. Today i am getting a new error while debugging the Azure Sample code in VS code. I dont know whether the VS code is causing the error or the CMake. Let me know what i can do to fix this.
I tried reinstalling Cmake extensions and also tried previous versions of VS code.  Thank you.
We are in a hurry because a lot of Azure devices are with customers and we need help as soon as possible.

@nishithpoojary
It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

@nishithpoojary It would be great if you could add a link to the documentation you are following for these steps? This would help us redirect the issue to the appropriate team. Thanks!

I follow the Azure Samples from this link. https://github.com/Azure/azure-sphere-samples/tree/main/Samples/AzureIoT
SDK, CMAKE and Ninja are downloaded from this link
https://learn.microsoft.com/en-us/azure-sphere/install/install-sdk?view=azure-sphere-integrated&pivots=vs-code
I am using the latest 24.03 Azure SDK.
VS code version is 1.92.0
Using latest CMake and CMake Tools extension in VS code.

@nishithpoojary
Thank you for bringing this to our attention.
I've delegated this to content author, who will review it and offer their insightful opinions.

Hello @nishithpoojary, has the issue been officially addressed by Microsoft? I had the same problem and suspect it was caused by the Azure Sphere plugin for VsCode not working. If there's any good news would be greatly appreciated! Thank you very much.

Hello @nishithpoojary, has the issue been officially addressed by Microsoft? I had the same problem and suspect it was caused by the Azure Sphere plugin for VsCode not working. If there's any good news would be greatly appreciated! Thank you very much.

Hi @Fan-aodong
I have also raised the issue in the microsoft Q and A forum.
https://learn.microsoft.com/en-us/answers/questions/1856577/azure-sphere-vs-code-error?comment=question#newest-question-comment
They are suspecting that some recent changes in VS code may be causing this. Waiting for their response.

Looking into it the error seems to be caused by a change to the node engine disallowing spawn calls to .cmd or .bat files without the shell=true arg. Downgrading vscode to a version that does not have this change to node (1.88.0) resolves the problem for me.

@nishithpoojary @TPavanBalaji
This issue has been resolved. Azure Sphere VS Code Error. Please consider closing it.


Chunk Corruption - EnderIO / Obelisk

Issue Description:
Chunk corruption when using Experience Obelisk with Experience Hopper. I apologize for not having more information regarding it on which it is specifically, but I originally thought I had corrupted my chunk from using Refined Storage so I let it go, wiped the chunk, rebuilt everything. Then this time I separated the chunks, rebooted the server, and only the chunk using the Obelisk and Experience Hopper were effected and my storage system chunk was fine. You can generally tell in advance the chunk is corrupted due to the world will not save and will hang when trying to either shut down or using save-all in prior to closing the server.
What happens:
Chunk corruption, world is unable to start up without using MCASelector to delete specific chunks.
What you expected to happen:
The world to load as normal
Steps to reproduce:

Create an grinder that collects EXP, in my case, I had a skeleton spawner feeding into a XP Vacuum with a Obelisk
Let it run for a while, unfortunately unsure how much experience had collected but it would've been 40+ levels.
Restart the world, the world will fail to start without using MCASelector to delete the chunk

Affected Versions (Do not use "latest"):

EnderIO: EnderIO-1.20.1-6.0.25-alpha
Minecraft: 1.20.1
Forge: 1.20.1-47.2.21
SpongeForge? no
Optifine? no
Server

Your most recent log file where the issue was present:
[pastebin/gist/etc link here]

pls use the other issue tracker here https://github.com/Team-EnderIO/EnderIO


Story: Environment

[ ] #20
[ ] #21
[ ] #22

Belongs to Epic #2

As a user, I want to be able to interact with the environment around me in a way that makes sense so that I can immerse myself in the game.


Remove the HealthCheckRunner

With the Procrastinate shell now in place we can remove the HealthCheckRunner completely.

I'm having second thoughts. A simple callable that tells you that 1) the app is configured correctly and 2) the DB is configured correctly might still be interesting as a smoke test. But we should make it just that, and remove the part on counting tasks.


Remove function advice around codeAction callback.

Old code action objects are now cleared (https://github.com/emacs-lsp/lsp-mode/commit/85442531315e826922362d0d545ddf9fc8653346), so the advice is no longer needed.
Plus, the callback function for codeAction is a dynamically generated closure, so lsp--text-document-code-action-callback no longer exists.

Awesome! are you reworking codeaction support upstream? cause its another thing i think it would make sense to do in lsp-ui. Maybe code actions with ido selection in lsp-mode, then ivy/helm in lsp-ui?

@topisani if the code actions just default to completing-read, then basic ivy/helm support should come largely for free. (lsp-ui would probably still want to implement richer code action wrappers, though.)

Awesome! are you reworking codeaction support upstream?

Sort of. See https://github.com/emacs-lsp/lsp-mode/commit/cf7c3b6648e0d7f95e156e6e756423c4f0afc133


Don't use deprecated gem option

ruby-2.4.6 - #adjusting #shebangs for (gem irb erb ri rdoc testrb rake).
Install of ruby-2.4.6 - #complete
Ruby was built without documentation, to build it run: rvm docs generate-ri
Using /u/pfmanager/.rvm/gems/ruby-2.4.6
ERROR:  While executing gem ... (OptionParser::InvalidOption)
    invalid option: --no-ri

-N(--no-document) instead --no-ri --no-rdoc.

Thanks!


kwargs instead of args

Usage of kwargs seems to be better than the current usage of args..

This will change the default behavior of thop. Could you list some examples that requires kwargs for the net input?

Hi, for example it could be much easier to use thop working with transformers. Here u may see the forward function of BART model.
So user may pass not all parameters, e.g. input_ids, attention_mask and past_key_values.


Provide support for declaring repositories on a per-artifact basis

Expected Behavior
User should be able to declare which repository an artifact should come from. Then gradle won't search all the repositories in order; it will only search the given repository for the artifact.
Current Behavior
Every repository is searched for every artifact.
Context
Often a repository will be added to a gradle configuration to resolve just one artifact in the build. It would be nice to specify that repository inline with the dependency that needs it. This adds clarity for everyone involved, and should shorten build times because Ivy will not have to query that repository for every other dependency.
This was originally tracked in GRADLE-1066, opened by Chris Beams.

Closing as duplicate of https://github.com/gradle/gradle/issues/1369.


homebrew install instructions have changed

It should be:
/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"

Check https://brew.sh

Thanks!
Feel free to fix things yourself too. :)


Intelligent branching order

Consider:

The free degree of each r/c.
The EPB related subgraphs adjacent to r/c, for example their size, the number of adjacent trees or the number of internal tree edges that are adjacent.
The distance of each vertex to side R or B, optionally weighting by the size of each side.
Whether or not the current optimal solution assigns this vertex to the larger or smaller side.

Added in b2929fb173, but still needs evaluation.

Disappointing results, so reverted. Closing.


Added emulation of a dice throw

Used that in a few games :)

Great, @codepo8. Hope to see more and more snippets from you 👍


Moneris support

I see a few libraries that seem to support Moneris.
https://github.com/kbahchevanov/omnipay-moneris
https://github.com/darksaboteur/omnipay-moneris
Should these be added in to the main account, are the fine to use?

Yes, they can be added in.  Encourage one of the owners to submit a PR here.


Register proxy if dashboard is dead

Ok. If proxy exist in cluster (configured etc) and dashboard stop. Proxy can't re register in cluster, no matter use I etcd or filesystem. Maybe I was made not right cluster, but ft or ha can't exist without auto re register proxy. One more thing if I running set/get command I getting error: router is not online if use sentinel and stopped dashboard.

@3JIou Hi, dashboard has single point failure under this version. Dashboard is the maintainer of zk/etcd in our architecture. Without dashboard, you can't register/unregister proxy or migrate slots. But in my future plan, I'll try to rewrite the dashboard, to remove the single point failure, and embed all the sentinel's features.

The answer of the 2nd question, the router: is not online was cased by the failure of registration process. Please make sure the dashboard is alive when you try to restart the proxies. Good lock!!

This single point failure will be removed in Codis4.x.
Thanks.


Custom text in DateDisplay fields

Is there a way to add some custom text to the From and To fields above the calendar? Here is an example:
https://ibb.co/WzQTDj8

Maybe you can try with css pseudo elements
Try to implement this on your stylings
span.rdrDateInput.rdrDateDisplayItem::before {
content: 'To:';
position: absolute;
display: inline-block;
width: 185px;
color: black;
font-size: 1.8rem;
}
span.rdrDateInput.rdrDateDisplayItem.rdrDateDisplayItemActive::before {
content: 'From:';
}


Supporting Maven and Gradle Wrapper in JFrog Xray audit-gradle and audit-mvn

Is your feature request related to a problem? Please describe.
I am trying to execute JFrog Cli with the command
jfrog audit-gradle
I got and error below:
[Error] exec: "gradle": executable file not found in $PATH
It seems the jfrog cli relies on gradle on the path which may be installed as a system wide gradle installation.
Describe the solution you'd like to see
I prefer the jfrog cli for audit-gradle to support gradle wrapper. This can be extended to maven wrapper as well. Our environment doesn't have system wide installation of Gradle/Maven in the system $PATH.

Thank you for sharing this requirement with us @e88z4!
We've added this task to our work backlog, and will update this issue once the work on this starts.

@e88z4
Actually, using the Gradle wrapper in the audit-gradle command is already supported.
All you need to do is to create a minimal configuration file in:
<workspace>/.jfrog/projects/gradle.yaml
or
<user-home>/.jfrog/projects/gradle.yaml.
The JFrog CLI generates this file by running one of the following commands:
# Option 1: Non interactive - local configuration
jf gradle-config --use-wrapper

# Option 2: Non interactive - global configuration
jf gradle-config --use-wrapepr --global

# Interactive
jf gradle-config

You can commit this file or create it every time in your CI pipeline.
If you wish, you also have the option to resolve the dependencies in the audit commands from Artifactory. See all options of the gradle-config command under Running Gradle Builds. See also audit-gradle documentation and this blog post.
Notwithstanding, using the Maven wrapper is not yet supported, but will be in the future.
Please let me know if that helped.

Hi @yahavi ,
The Gradle wrapper instruction works.
Can the ticket be open to fully support Maven wrapper?

Definitely @e88z4.
We will use this issue and update it once we get to supporting the maven wrapper.
Thanks for feedback!

The jf gradle-config should also support the --uses-plugin switch, to add usePlugin: true into the generated .jfrog/projects/gradle.yaml file.
This command also doesn't work with the jfrog-azure-devops-extension, when attempting to use the JfrogCliV2@1 task, as the task adds flags which are not supported by the jf gradle-config command. jf gradle-config also can't be easily ran from a generic script task, as it's not in the $PATH.

Sorry, I mixed things up. Yes, jf gradle-config does support it. The problem is that there doesn't seem to be a way to run  jf gradle-config within an Azure DevOps JFrog task.
The JFrog Gradle task is just fine, but I'm interested in the JFrog Audit (for Gradle projects). Committing .jfrog/projects/gradle.yaml into the project is a clumsy solution. Right now I have a shell script task in Azure Devops to create this file with cat and here-documents.

I was able to workaround this by modifying the path to include the wrapper dist bin folder (before whatever other installation you have already if any) after running the wrapper once to initialize the install. Sub optimal for sure but unblocked my current task at least.
./gradlew -Dscan.tag.ScaScanPrep
echo "export PATH=\"$(find $HOME/.gradle/wrapper/dists -type d -regex ".*/bin" | head -1):\$PATH\"" >> $BASH_ENV

$BASH_ENV being a path to a script that CircleCi will run at the start of each run step so its useful for holding onto vars/functions/etc.

FYI, not sure if this is a separate issue but jf will routinely ignore the maven wrapper setting even when set on the CLI as well as in config. Logs from a GitHub Actions workflow below:
  cp .jfrog/projects/maven.yaml $HOME/.jfrog/projects/
  export PATH=$HOME/.local/bin:$PATH
  jf mvn-config --global --use-wrapper
  jf audit --mvn \
    --user $JFROG_USERNAME \
    --password $JFROG_PASSWORD \
    --url https://$JFROG_URL/xray \
    --watches $WATCHES \
    --format table \
    --extended-table
env:
    JAVA_HOME: /home/runner/_work/_tool/Java_Temurin-Hotspot_jdk/17.0.11-9/x64
    JFROG_USERNAME: ***
    JFROG_PASSWORD: ***
    JFROG_CLI_LOG_LEVEL: DEBUG
16:21:12 [Debug] JFrog CLI version: 2.58.2
16:21:12 [Debug] OS/Arch: linux/amd64
16:21:16 [Info] Running SCA scan for maven vulnerable dependencies in /home/runner/_work/artifactory/artifactory/tests/e2e/maven directory...
16:21:16 [Info] Calculating Maven dependencies...
16:21:16 [Debug] Using resolver config from /home/runner/_work/artifactory/artifactory/tests/e2e/maven/.jfrog/projects/maven.yaml
16:21:16 [Debug] Preparing to read the config file /home/runner/_work/artifactory/artifactory/tests/e2e/maven/.jfrog/projects/maven.yaml
16:21:16 [Debug] Analytics metrics are disabled, skipping sending update event request to XSC
16:21:16 [Debug] Sending an error report to JFrog analytics...
16:21:16 [Debug] Sending HTTP GET request to: https://<redacted>/xsc/api/v1/system/version
16:21:16 [Debug] failed to check availability of Xsc service:server response: 404 Not Found
404 page not found

Reporting to JFrog analytics is skipped...
16:21:16 [Info] Trace ID for JFrog Platform logs: e0d096f6bf504532
Error: 6 [Error] audit command in '/home/runner/_work/artifactory/artifactory/tests/e2e/maven' failed:
failed while building 'maven' dependency tree:
failed running command 'mvn org.apache.maven.plugins:maven-install-plugin:3.1.1:install-file -Dfile=/tmp/jfrog.cli.temp.-1723220476-1905448768/maven-dep-tree.jar -B': exec: "mvn": executable file not found in $PATH

Running
 jf audit -h

Say that
--use-wrapper                    [Default: true] Set to false if you wish to not use the gradle or maven wrapper.

Which is a lie because our ci have the same issue as @coopernetes  jf audit doesn't use wrapper.
I even could not run jf mvn-config --use-wrapper on our ci because it run into interactive mode and asked me this question


Do not load after/syntax/css.vim 2 times for less

runtime! syntax/css.vim already loads after/syntax/css.vim because after is in &rtp. This change prevents after/syntax/css.vim to be loaded two times for less filetype if it is present in user vimruntime directory

did you contact the maintainer? It's seems to be maintained here: https://github.com/genoma/vim-less so ping @genoma

So you're periodically fetching changes from external repositories by looking at "URL" comment in syntax files?

No, we cannot afford to hunt new runtime files all over the internet every now and then and in addition we do not now, if the current status of the runtime file are ready to be included (or just a dev/test/experimental version). Therefore it's the maintainers responsibility to periodically send updates, whenever he thinks the runtime file is stable.
See also https://github.com/vim/vim/blob/master/CONTRIBUTING.md#syntax-indent-and-other-runtime-files

understood https://github.com/genoma/vim-less/pull/6


Trying to insert a 'NoReturn' inlay hint prefixes it with 'sys'

Have a function that throws an exception:
import sys

def raises():
    raise ValueError('This is a test')

Double click on the -> NoReturn: inlay hint
Expected result:
import sys
from typing import NoReturn

def raises() -> NoReturn:
    raise ValueError('This is a test')

Actual result:
import sys

def raises() -> sys.NoReturn:
    raise ValueError('This is a test')

I briefly looked into this issue and found that it happens because when typeStringGenerator tries to retrieve existing module info from import statements, it finds NoReturn in the symbol table of sys.__init__.pyi.
I suspect that since NoReturn is imported and used in __init__.py, it is considered as part of the module symbols during binding. I still need take a closer look to find the root cause. Same bug also repros for other modules that import NoReturn such as os or the example below.
# file1.py
from typing import NoReturn

# file2.py
import file1

def raises() -> file1.NoReturn:
    raise ValueError('This is a test')

ya, we might need to see if symbol it found is alias symbol, and if it is, resolve it and if it is resolved to stdlib symbol, we use it from stdlib if not, use as it is.


Pinning failure better error

Here’s another take at fixing #3165 which, unlike #3191, should not introduce a security flaw!
I will explain what I did and why in greater details later but I’m already submitting this pull request to see if all tests pass on Travis.
I’ll also create a better error with a full userInfo dictionary, so please don’t merge this yet.

I tried something with 3e01f58c419fd573ecbe7b9eb8c9d71098f57c6a. The documentation is lacking for now.

Can you rebase this with master to get Xcode 8 coverage?

I rebased, but this is stil missing documentation, so I wouldn’t merge yet. Also, I think there might be some overlap with #3616.

Should I punt this from the 3.2.0 release train? Would like to release it this week sometime.

Current coverage is 86.77% (diff: 85.07%)

Merging #3425 into master will decrease coverage by 0.71%

@@             master      #3425   diff @@
==========================================
  Files            45         44     -1
  Lines          6224       6109   -115
  Methods        1099       1086    -13
  Messages          0          0
  Branches        414        405     -9
==========================================
- Hits           5445       5301   -144
- Misses          776        805    +29
  Partials          3          3

Powered by Codecov. Last update 4429921...4501acb

I'm almost done writing the documentation. But it's not covered by tests at all for now. 😕

I added documentation in 9310feb and I’m happy with it. It still needs some more tests though. I'd like to implement a custom authentication challenge handler that demonstrate how to use it for client certificates, but I think it’s going to be tough!

Any chance this will get fixed? That "cancelled" error is a real pain.

Unfortunately this PR is too complex to be merged with the amount of work it would take to validate the behavior.

That's a shame, I invested so much time in it to have proper error handling. 🙁

For future reference, the easier way to produce per-task security errors would be to drop the implementation of URLSession:didReceiveChallenge:completionHandler: altogether and rely on the URLSession:task:didReceiveChallenge:completionHandler: method. That way all auth challenges are handled on a per-task basis. If the NSURLSession level method is implemented at all it will receive all trust challenges.

If you want you can reengineer the patch on top of current master and feel free to drop the backward compatibility concerns for the major release. If you do want to do this, you can build off of the afnetworking4 branch that's up right now.


[MXNET-134] Don't do imperative bulk when profiling aggregate stats

https://issues.apache.org/jira/browse/MXNET-134
Description
When profiling aggregate stats, ImperativeBulk() will group many ops together, making the aggregate statistics nonsensical and useless.
This change keeps the bulk size as zero when profiling aggregate stats, so that operator statistics are kept separate.  This has a slight performance hit, although recording aggregate stats already has a slight performance hit, so it isn't introducing anything new in that area.  This is explained in the python description:
def set_config(**kwargs):
    """Set up the configure of profiler (only accepts keyword arguments).

    Parameters
    ----------
    ...
    aggregate_stats : boolean,
        whether to maintain aggregate stats in memory for console
        dump.  Has some negative performance impact.
    """
   ...

Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[x] Changes are complete (i.e. I finished coding on this PR)
[x] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[x] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[ ] Feature1, tests, (and when applicable, API doc)
[ ] Feature2, tests, (and when applicable, API doc)

Comments

If this change is a backward incompatible change, why must this change be made.
Interesting edge cases to note here

How about making engine->bulk_size always 0 when aggregate is running?

the option can be turned on and off at will. I think calling it from Profiler::SetConfig() would be weird

Plus, this way it saves it and resores

I was thinking about this last night and I think I can come up with a more elegant solution. However, I would prefer not to directly overwrite the bulk size variable because:

The whole save/restore thing the way it is done now doesn't seem very thread-safe (or multi-graph safe) since there's only one engine, but possibly many graphs, so there seems to be some disconnect here that may need to be dealt with one day (although it's currently not a problem).  Please correct me if I am wrong here.
If profiler sets bulk size to zero, then saves the old value and something else wants to set the bulk size (ie from python), it will both overwrite the 0 set by the profiler and also, when the aggregate is turned off, it will replace the older pre-python-set value.  Because of this, you won;t be able to toggle aggregate or profiling without possibly affecting the deterministic behavior of the intended bulk size value

Oh, I see -- Engine::bulk_size() is a function...

@cjolivier01 this PR seems to cause issues with MKL-DNN build, is CI done for this. thanks.

what problem with mkldnn build?

does not crash if built with make -j(nproc) USE_MKLDNN=1 USE_PROFILER=1 ;
crashes if built with make -j(nproc) USE_MKLDNN=1

i see you have a PR to disable profiler option, please ignore above.


Use native message box instead of QMessageBox

QMessageBox have rendering problems on Linux with Wayland (at least on GNOME).

Here are the problem on GNOME with Wayland:

Just notice this bug happened only on High-DPI screen.

I cannot reproduce on Qt 6.5 anymore so I assumed it is already fixed.


Check if reinforcement data is saved in userProfile

[] Dan give us a user profile for Anissa

@dan-chop-ris did we fix this issue?

Yes, reinforcement data is saved in user profile.


Publish API docs

This is a tracking issue for publishing API docs based on inline comments in the TypeScript source. API docs will probably be published at http://orbitjs.com/api

Yes, sir, please!

Wow. It's almost 2019 and API docs are still missing!

Well it's now 2019 and still no API docs. Any ETA?

Any ETA?

Very soon.
I'll be releasing the first beta of 0.16 this weekend. 0.16 has much improved typings, which makes TypeDoc happy and fairly straightforward. The only piece that's missing is composing all the API docs for the different packages nicely in a single UI.
If anyone knows has a good example of TypeDoc-generated docs for multiple packages merged nicely with a common search and (perhaps) TOC, please let me know.
Regardless, I'll publish the API docs earlier rather than waiting for a perfect unification.

Hey y'all. We're currently  evaluating orbitjs and really needed some api docs, so I went ahead and generated some: https://orbitjs-docs.now.sh
There are some remaining issues, like picking the correct readme and module order, but at least there's something to peruse now.
@dgeb Would you like a PR for this?

@dgeb Would you like a PR for this?

Yes, please! Looks like a good start. Thanks so much, @brumm

@brumm awesome, thanks!

I'm pleased to say that API docs have finally been published, starting with v0.17. The announcement is here.