Feature: introduce CompoundFeatures

Alternative branch to https://github.com/FZJ-INM1-BDA/siibra-python/pull/441 based on feedback received on the dev meeting dated 18.09.2023 and further discussions between @xgui3783 and I.
fts = siibra.features.get(siibra.get_region('julich 2.9', "hoc1 left"), 'bigbrain')
for f in fts:
    print(f.name)

[x] Make RegionalBOLD compoundable.
[ ] RegionalConnectivity's subjects property is obsolete with F-Tract. A new property name to refer to the files is required.
[ ] Add AnatomoFunctionalConnectivity class to digest F-tract data.
[ ] CompoundFeature.data should produce average data?
[ ] CompoundFeature.plot() that can plot individuals and the average
[ ] What other features should be compounded? Suggestions:

CellbodyStainedSection

[ ] Update e2e and unit tests
[ ] Update examples
[ ] Should we add attributes and dataframe property for all features which then would allow getting instance dataframe listing all with attributes to be filtered by the user
EDT: added a to-do list to polish/improve/add more functionalities. @dickscheid and @xgui3783 please add more or comment.

Codecov Report

Merging #479 (ae962de) into main (5a0e6b3) will increase coverage by 0.42%.
Report is 131 commits behind head on main.
The diff coverage is 28.12%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
@@            Coverage Diff             @@
##             main     #479      +/-   ##
==========================================
+ Coverage   36.81%   37.24%   +0.42%
==========================================
  Files          61       61
  Lines        5421     5831     +410
==========================================
+ Hits         1996     2172     +176
- Misses       3425     3659     +234

Files Changed
Coverage Δ

siibra/configuration/configuration.py
40.90% <0.00%> (-1.45%)
:arrow_down:

siibra/core/atlas.py
85.71% <ø> (+1.23%)
:arrow_up:

siibra/livequeries/ebrains.py
25.35% <0.00%> (+0.35%)
:arrow_up:

siibra/locations/boundingbox.py
19.76% <ø> (-0.13%)
:arrow_down:

siibra/locations/pointset.py
31.64% <0.00%> (ø)

siibra/volumes/neuroglancer.py
21.42% <0.00%> (ø)

siibra/volumes/volume.py
50.29% <0.00%> (ø)

siibra/features/tabular/cortical_profile.py
26.43% <3.70%> (-4.23%)
:arrow_down:

siibra/configuration/factory.py
21.12% <3.84%> (-0.86%)
:arrow_down:

...a/features/tabular/receptor_density_fingerprint.py
26.19% <4.54%> (-2.02%)
:arrow_down:

... and 31 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

For now, my only comment is that compound_key does not need to be there, and access to subfeatures (for now) will rely exclusively on integer index.
To refine it further (maybe in the second commit, once this model is approved), CompoundFeature should have a attributes public dataframe, which should allow the subfeatures to be queried arbitarily, in addition to integer access

I have pushed https://github.com/FZJ-INM1-BDA/siibra-python/pull/479/commits/0651cb13c6a9892efab622368b71b16aa5d58ba9 to resolve this. Let me know what you think.
Also allowed compounding functional connectivity with 'paradigm' as well, https://github.com/FZJ-INM1-BDA/siibra-python/pull/479/commits/e411250c7104e4256e47bad4f36bf913a298b8dc

(okok, I know we went back and forth about this, but I swear this is/might be the last thought on this)
regarding
    @property
    @abstractmethod
    def _groupby_attrs(self) -> List[str]:
        """Attributes used to reduce the list of features to CompoundFeatures."""
        raise NotImplementedError

    @property
    def _groupby_key(self):
        """
        Attribute key-value pairs used to reduce the list of features to
        CompoundFeatures based on groupby property. Can be overriden if need be.
        """
        return tuple((attr, self.attributes[attr]) for attr in self._groupby_attrs)

in Compoundable would it make sense if we just remove  _groupby_attrs and make _groupby_key abstract property instead?
classes subclassing Compoundable, instead of definine a list of str, to indirectly influence the group by key, can directly generate the key.
for e.g.
class RegionalTimeseriesActivity(tabular.Tabular, Compoundable):
    # ... trimmed for brevity

    # current
    @property
    def _groupby_attrs(self) -> List[str]:
        return ["class", "modality", "cohort"]

    # proposed
    @property
    def _groupby_key(self):
        return (self.modality, self,cohort)

in Compoundable would it make sense if we just remove _groupby_attrs and make _groupby_key abstract property instead?

Hmm we certainly can. I will just need to adjust the sorting part of the code. I'd think it will make your job harder but if it works for you, why not.

I saw that sorting ignores groupby key
I wonder if we need/should do any sorting at all, since we allow any arbitary objects to be a part of attribute value. they may or may not be sortable.

I saw that sorting ignores groupby key
I wonder if we need/should do any sorting at all, since we allow any arbitary objects to be a part of attribute value. they may or may not be sortable.

If we want reproducible results, we need sorting. This is especially important for Bigbrain intensity profiles.
(I pushed a commit which now takes a look at gorubby_key instead.)

edit: also, genuine question, what happens when user calls compound_feature_instance.[append|extend]? the subfeature will no longer be sorted after all

I'd expect that the user would expect the append and extend behave similar to the list's. From reproducability standpoint, the compund feature constructor must ensure that but unless we change append and extend to update, I think it behaves as expected.
But I also do not really see a use case for append/extend. Maybe you can clarify how do you envison it once more?

why do we need to add inst.__class__.name to groupby key? I thought modality should do it already?
on a separate note, the fact that Bigbrain intensity profiles return list of different feature/order should be the issue addressed how Bigbrain intensity profiles generate the list. we should address it there, not here.

I'd expect that the user would expect the append and extend behave similar to the list's. From reproducability standpoint, the compund feature constructor must ensure that but unless we change append and extend to update, I think it behaves as expected.

If we expect the compound feature to behave similar to list, the following code would potentially fail, no?
features = [foo1, foo2]
compound_feature = CompoundFeature(features)
assert features [0] is compound_feature [0]

But I also do not really see a use case for append/extend. Maybe you can clarify how do you envison it once more?

if we allow index access to subfeatures, I would expect CompoundFeature instance is (in some way) an extension to list. In this narrative, extend and append makes sense.
But I suppose we could assume it is more similar to an extension to tuple instead, which is immutable.

I've already checked this. It returns ordered. when set applied, there is a kind of ordering happening.

Am I understanding correct that:
feature = BigBrainIntensity.match (returns same order every time) ->
list(set(features)) (order becomes is randomised and unpredictable)
?

I've already checked this. It returns ordered. when set applied, there is a kind of ordering happening.

Am I understanding correct that:
feature = BigBrainIntensity.match (returns same order every time) ->
list(set(features)) (order becomes is randomised and unpredictable)
?

yep

I've already checked this. It returns ordered. when set applied, there is a kind of ordering happening.

Am I understanding correct that:
feature = BigBrainIntensity.match (returns same order every time) ->
list(set(features)) (order becomes is randomised and unpredictable)
?

yep

then we should either apply sorting right after list(set(features)), or investigate why list(set(features)) produces unstable order.
compound feature is not the place for this.
in particular https://github.com/FZJ-INM1-BDA/siibra-python/pull/479#issuecomment-1729118597

list(set(features)) (order becomes is randomised and unpredictable)

indeed, set (and in theory dict) is unordered.
I would argue that we cannot use list(set) to remove duplicates.
We will need to use for loop IMO.

To prevent an accidental clash of modality names and also give the flexibility to just group by class.
For example grouping two types of MRI modalities.
Or say later a new feature is introduced but the modality is the same as another since modality string does not always match class name.

yea, I am not against this. Just found it a little strange.

I've already checked this. It returns ordered. when set applied, there is a kind of ordering happening.

Am I understanding correct that:
feature = BigBrainIntensity.match (returns same order every time) ->
list(set(features)) (order becomes is randomised and unpredictable)
?

yep

then we should either apply sorting right after list(set(features)), or investigate why list(set(features)) produces unstable order.
compound feature is not the place for this.
in particular #479 (comment)

I disagree. This would require that all features have sorting mechanism implemented and we sorting will cause extra overhead for features this is unnecessary for.
Also, if sorting is not made, the dataframe for connectivity results in a complete mix which is extremely unhandy. If the user does not have compound_key they should at least have a rule they can rely on in the dataframe to get the index

I disagree. This would require that all features have sorting mechanism implemented and we sorting will cause extra overhead for features this is unnecessary for.

For the record, I am arguing that we shouldn't do any sorting.

Also, if sorting is not made, the dataframe for connectivity results in a complete mix which is extremely unhandy. If the user does not have compound_key they should at least have a rule they can rely on in the dataframe to get the index

the user has access to compound_feature.dataframe, and any number way of filtering the feature according to the fields provided by the dataframe. (later, if we include the feature reference in the dataframe, they could even order it in whichever way they desired)

list(set(features)) (order becomes is randomised and unpredictable)

indeed, set (and in theory dict) is unordered.
I would argue that we cannot use list(set) to remove duplicates.
We will need to use for loop IMO.

In theory, after the changes I made to the parse_feature_type, there should not be any duplicates anymore. I am just not sure about all live queries. (Need to look into Allen)

In theory, after the changes I made to the parse_feature_type, there should not be any duplicates anymore. I am just not sure about all live queries. (Need to look into Allen)

we need to remove list(set), and implement deduping in a more deterministic manner. (even if we no longer need to dedup here, but for future applications , we should note to not use list(set)) to dedup

@xgui3783 what do you think of following __get_item__
def __getitem__(self, index_or_query: Union[int, Dict[str, Any]]):
    """
    Get the subfeature corresponding to an integer index or key-value pairs
    matched with the queryable attributes and subfeature values.
    """
    if isinstance(index_or_query, int):
        return self.subfeatures[index_or_query]
    assert isinstance(index_or_query, dict)
    if all(k for k in index_or_query.keys() if k in self.query_attrs):
        query_expr = " and ".join(
            f"`{k}` == '{val}'" for k, val in index_or_query.items()
        )
        result = self.dataframe.query(query_expr)
        if len(result) == 1:
            return self[int(result.index[0])]
        if len(result) > 1:
            logger.info(
                f"Multiple features matched '{index_or_query}'."
                "Returning a CompoundFeature of the matches."
            )
            features = [self[i] for i in result.index]
            return CompoundFeature(features, self._queryconcept)
    raise KeyError(f"Could not identify a subfeature with '{index_or_query}'")

(The test won't be able to pass as it needs to target a tag. But creating a tag would mean it would also fail as 0.4a77 does not have a way to build AnatomoFunctionalConnectivity. This is the caveat of the tagged connection design. Hence, confirm the tests passes on the branch, then merge to main, then release configuration, then re-run tests)


raft_client: connect to other stores using peer_address (#5569)

Signed-off-by: disksing<EMAIL_ADDRESS>
What have you changed?
We plan to provide a heterogeneous engine with tikv, which may need to use different addresses to provide kv and raft services. Now kvproto has an optional peer_address to support this.
The corresponding tikv modification is: use peer_address when connecting other nodes to send raft messages. (for backward compatibility: if peer_address is not set, address is still used)
What is the type of the changes?

New feature (a change which adds functionality)

How is the PR tested?

Unit test

Does this PR affect documentation (docs) or should it be mentioned in the release notes?
No
Does this PR affect tidb-ansible?
No
Refer to a related PR or issue link (optional)
cherry-pick: #5569

/run-all-tests

/run-all-tests

/run-all-tests

/merge


some question about model and the result

Hi Axel,
I have some question  for  the repo

i read the OmniAnomaly code, i found it use 25/55 as the out_dim  for the MSL and SMAP,  i also open the MSL data, only the   first dimension has value, so can i think the MSL and SMAP  is just univariate time series, but almost paper say the dataset  is multivariate, it's  make me confuse.
i use the repo as the baseline for my research, i found you replace the decoder from VAE to GRU, so did you try the original VAE for the decoder , i do some experiment  but i can't  achieve  the result  in the original  paper,  so if you try VAE can  achieve  the result in original  paper

I will appreciate it if you can reply as soon as possible.

Hello!

MSL and SMAP has one so-called telemetry value, and then the rest are one-hot encoded values. I don't know if OmniAnomaly tries to forecast/reconstruct the one-hot encoded values, but to us that makes little sense. Therefore, we use all features as input (making it a multivariate), but only forecast/reconstruct the telemetry value.
Yes, we talk about this in #2.

Hi @axeloh, if I have 4 columns, each column represent sensor data, and anomaly is based on the 4 features. Can I reconstruct all the 4 columns with your model ?
If yes, can you please show me what I need to  modify ?


Option for WSL2

I saw your utility supports installing WSL. I just wanted to suggest also offering the option to install WSL2. This is the updated version and it's more efficient and robust. It mounts both your Linux subsystem AND your Windows OS on the same hypervisor.
https://pureinfotech.com/install-windows-subsystem-linux-2-windows-10/

The install process is the same, it's how you setup your Linux instance inside WSL that determines if it's WSL1 or 2


T236 removal

As we discussed, we will allow the removal of some redundant theorems if they are not needed for some other purpose (like shortcircuiting in deduction chains) and they are not referenced in mathse or mo.
So remove T236 (weakly local compact + Lindelof ==> hemicompact) as it is implied by T53 and T5.
See https://topology.pi-base.org/spaces?q=weakly locally compact%2Blindelof%2B~hemicompact
(T236 was used in https://math.stackexchange.com/questions/4566624, but I edited the post to not refer to it.)
Also added a better justification for https://topology.pi-base.org/theorems/T000053 and an additional reference to show the equivalence between the definitions of https://topology.pi-base.org/properties/P000025.

I have a technical question, what does this part of T53 mean?

converse:

T000008
T000005
T000237
T000122

I have a technical question, what does this part of T53 mean?

It's the list of theorems that are displayed in the "Converse" tab of https://topology.pi-base.org/theorems/T000053.  We use that when the implication is an equivalence.


Unable to issue certificate batch using Metamask (for UAT)

Hi OpenCerts team,
We are unable to issue certificate batch using Metamask as the page is not responsive after click on Issue button,
This is for UAT but user needs to do UAT urgently.
Please advise.
Thanks.

Can you open the developer console and see whether an error is fired?

Hi,
Attached is the image from dev console FYR.

FYI, we used this Metamask account to published to UAT before

can it be sent via email? as it is used by SEAB for UAT and we dont want everyone to see it

I don't get your point?

Document store is a public info (hard to guess/find)
Merle root is just a hash with not data at all
it's a test environment

document store is store address?

Yes

Store address:    0xeDe1B6Fc03f1a9C6905C93a2fceb06E19624a55E
Merkle Root:      0x693f7aaeddb224bdc902b4c584b369e3072dacdc006c37079a5e3aba826ddfa9

From https://ropsten.etherscan.io/address/0xeDe1B6Fc03f1a9C6905C93a2fceb06E19624a55E#readContract I can see that 0x693f7aaeddb224bdc902b4c584b369e3072dacdc006c37079a5e3aba826ddfa9 has already been issued

the other day we tried, but the cert is not valid. can i check when it is issued?
possible to provide date and time?

so I used https://ropsten.etherscan.io/address/0xeDe1B6Fc03f1a9C6905C93a2fceb06E19624a55E#readContract
and getIssuedBlock function
which gives me 5279052
which is https://ropsten.etherscan.io/block/5279052
which has been mined 461 days ago

Transaction hash => https://ropsten.etherscan.io/tx/0x863b6d41e1b1d34a4a6fa01c0df963d18392e4c0414f6da07e3855f3c3bef4b3

This file has only been published this week. I attached the opencert file here for your reference. it can't be viewed online at https://dev.opencerts.io/
881.zip

those two certs are under the merkleRoot of 0x920ea6e7e5e49986b1dfa696658ff7316eb35ac83ee27bf4c21dde0628b426a3 which has not been issued, where did 0x693f7aaeddb224bdc902b4c584b369e3072dacdc006c37079a5e3aba826ddfa9 come from?


Add cloudbees logo and info

assuming this referred to using jenkins, if so, unnecessary as GH-actions more powerful solution and at least here was using travis ci which replaced that need.


Extending some classes with labels

Thanks a lot for your help. I will also backport this to wicket 6 branch

Sounds good! Thanks.


ndarray constructor behavior mismatches when strides is given

Description
As title.
To Reproduce
>>> import numpy as np
>>> a = np.ndarray((1,), strides=(8,), dtype=np.float64)  # this works
>>>
>>> import cupy as cp
>>> a = cp.ndarray((1,), strides=(8,), dtype=cp.float64)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "cupy/_core/core.pyx", line 136, in cupy._core.core.ndarray.__new__
  File "cupy/_core/core.pyx", line 213, in cupy._core.core._ndarray_base._init
ValueError: memptr is required if strides is given.

Installation
Source (pip install cupy)
Environment
No response
Additional Information
No response

Maybe NumPy no longer raises in recent releases, so the old comment https://github.com/cupy/cupy/pull/1611#discussion_r218683200 is no longer applicable?


update alpine 3.13.1 -> 3.13.2

Update alpine version, mainly to solve openssl vulns.

I wonder if it is worthwhile moving away from alpine and using a distroless / scratch container for GoStatsd?
There is nothing within alpine itself that it really needs.
Can I also ask that you raise an issue for the docker image as well?

I've raised the issue: https://github.com/atlassian/gostatsd/issues/390

I am going to reject this PR in favour of the issue created since it will reduce the number of times it is needed to bump alpine to the latest fixed version.


Update xml-encryption due to ejs vulnerability

Update xml-encryption due to ejs vulnerability via remote code execution: https://securitylab.github.com/advisories/GHSL-2021-021-tj-ejs/

should we cover this as well? https://github.com/krakenjs/passport-saml-encrypted/security/dependabot


Add "bach-build-run.jsh"

Based on Bach, the Java Shell Builder.

Also, why can't I push changes to your fork of my repo? Is that not commonly possible?

That's not possible by default. Unless invited to collaborate.

Ah, that explains it - I only did this with people that had all access to the same repo.

 You should really branch and not do feature development on master. :P

I shouldn't grant everybody write access... 😉

Fixed "java" execution via https://github.com/sormuras/bach/commit/54762347e3a71859e954f7bff5ae8f3bfa39dd7a
Just press play on tape... (i.e. remove local target folder)

I think including Travis overdraws a Hello World's complexity budget. :sink:

Back to the original 3 commits. Have fun!

Does the bach-build-run.jsh work for you now?

You're not one to let go, are you? 😆


NoMethod error on accessor for id for belongs_to association

I'm trying to use activemodel-associations to create a series of associations to build a query.  However,
it appears that it's not building the expected methods, or at least those I expect..
The code is basically:
class YmmYear < ActiveRecord::Base
  validates :year, presence: true
  def name
    self.year
  end
end
class Ymm
  include ActiveModel::Model
  include ActiveModel::Associations
  attr_accessor :ymm
  belongs_to :ymm_year
  def [](attr)
    self.send(attr)
  end
  def []=(attr, value)
    self.send("#{attr}=", value)
  end
end
class Admin::YmmsController < ApplicationController
  before_action :check_if_admin
  def new
    @ymm = Ymm.new
    @ymm.ymm_year_id = 1126
  end
end

The assignment to @ymm.ymm_year_id fails with NoMethod error.
undefined method `ymm_year_id=' for #<Ymm:0x9c549d8 @association_cache={}>

The methods to which @ymm responds that include "ymm_year" are:
autosave_associated_records_for_ymm_year
ymm_year
ymm_year=
build_ymm_year
create_ymm_year
create_ymm_year!

A classic ActiveRecord belongs_to assocation would create the accessor ymm_year_id, would it not?
Or, am I misunderstanding this?
Thanks.

When I update the model to add this line, the Rails form helpers begin to work:
  attr_accessor :ymm, :ymm_year_id

Yes, Your way is right.
belongs_to needs foreign key attribute.


chore: add drop-checker page

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context

[x] Related: https://github.com/kodadot/ops-internal/issues/1768

Scr?

looks good, where can i access the page?

looks good, where can i access the page?

Demo: https://deploy-preview-10578--koda-canary.netlify.app/drop-checker/ahp/158

so <url>/drop-checker/<prefix>/<id>

just a thought, what if we added ❌ or ✅ if the onchain and database data was === similar to what we do in the code-cheker

added 👍🏻

small darkmoda stuff
can't see the text here

fixed


Fix #426

Fixes #426 (using a regular cast, of course).

@Adrian-Samoticha thanks for the review!


Incomplete step by step to GPO Domain controller policy

I realize your section https://learn.microsoft.com/en-us/defender-for-identity/configure-windows-event-collection#event-id-8004
has incomplete step by step written.

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

ID: b032a29d-2d3a-d737-28fb-3e6632a667e2
Version Independent ID: b75dd591-4cd7-259a-1fe7-a581d6e356c9
Content: Configure Windows Event collection Microsoft Defender for Identity
Content Source: ATPDocs/configure-windows-event-collection.md
Service: microsoft-defender-for-identity
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

@SabrinaKay - Thanks for the helpful feedback. We've updated the section to be clearer (should be live soon).


Format all scopes supported by clang-format

Furthermore this allows the user to configure format on save per scope.

This is great and something that should be in this project, However 2 things: I think there should still be a boolean option to turn on/off format on save totally. And also the options for scopes should be boolean toggles for each language instead of stings, most users won't know what "scope.cpp" and the other atom api for scopes means.

I cleaned up the menu close to what I described above and pulled it into master though #24

Nice, definitely more user friendly than my solution. I wanted to keep it simple and flexible, therefore the array config option.


Updating readme.md to insist on having postgresql-devel package before running make

Without the postgresql-devel package, running make command fails the build, so adding this info to documentation

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Service Manager UI

add UI for Service Offerings (Marketplace) and for Service Instances

/hold

/unhold


How can I set up next build?

I want to set up i18n for this landingpage, but this requires next build and a node.js server -> I set the build command to next build but it doesn't work, so what should I do?

And I also would like to add the google tag manager, but I am failing - I am really new to next.js so I am really sorry :c


Create unlines function

Create the convenience function unlines, which works as an opposite to lines.  The name is taken from Haskell, which has both functions, lines and unlines, even though it has other joining function such as intercalate.

Great! Thanks!

Now released 0.4.0 with your improvement ;)

Cool!
On Mar 30, 2015 7:24 PM, "Andrey Antukh"<EMAIL_ADDRESS>wrote:

Now released 0.4.0 with your improvement ;)
—
Reply to this email directly or view it on GitHub
https://github.com/funcool/cuerdas/pull/9#issuecomment-87760164.


[Bug] /posts returns string for userId instead of number

https://jsonplaceholder.typicode.com/posts
/*snip*/
  {
    "userId": 10,
    "id": 100,
    "title": "at nam consequatur ea labore ea harum",
    "body": "cupiditate quo est a modi nesciunt soluta\nipsa voluptas error itaque dicta in\nautem qui minus magnam et distinctio eum\naccusamus ratione error aut"
  },
  {
    "title": "foo",
    "body": "bar",
    "userId": "4", //Should be `4`, not `"4"`
    "id": 101
  }
]

@AnyhowStep How is this affecting your testing?  There are a number of ways to get around this, without them having to make changes.  If you have a case where this is causing a problem, post a snippet of code and I can show you how to get around it.

Hi,
Thanks for the issue. There shouldn't be a 101st item when doing a GET on https://jsonplaceholder.typicode.com/posts. If there is, it's a bug in JSONPlaceholder.
I've checked and I get 100:
...
    "userId": 10,
    "id": 99,
    "title": "temporibus sit alias delectus eligendi possimus magni",
    "body": "quo deleniti praesentium dicta non quod\naut est molestias\nmolestias et officia quis nihil\nitaque dolorem quia"
  },
  {
    "userId": 10,
    "id": 100,
    "title": "at nam consequatur ea labore ea harum",
    "body": "cupiditate quo est a modi nesciunt soluta\nipsa voluptas error itaque dicta in\nautem qui minus magnam et distinctio eum\naccusamus ratione error aut"
  }
]

Are you still getting 101 items? Is it random?

@typicode Hitting the above entrypoint you supplied, the last row in the result is indeed a 101

if it returns 101 again, please let me know :) (seeing also 100 items at the moment)


Fix creating dependency-injected condition

correct Array type check
correct calculating Array's last index

Coverage remained the same at 96.269% when pulling 2a3cdab45c41585d02489ec44d73ec08ce73638f on jayellsbrg:patch-1 into e10fb3bf74e46c8bfba604505625e5d870639c9f on aleross:master.

Is there a reason why this PR is not being merged? We need this fix, otherwise conditions are not working. Thanks.


Add atomic_schema as repository_field.

This change is

@jgoclawski review please.

Comments from Reviewable

Reviewed 3 of 3 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Reviewed 3 of 4 files at r4, 10 of 10 files at r5.
Review status: all files reviewed at latest revision, 5 unresolved discussions, some commit checks failed.

flask_jsonapi/nested/nested_resource_repositories.py, line 1 at r5 (raw file):

import flask_jsonapi.nested.nested_resources

inconsistent imports:
from flask_jsonapi.nested import nested_resourcesnxn

flask_jsonapi/nested_resource_repositories.py, line 8 at r4 (raw file):

NotImplementedProperty('schema')

'nested_schema' as argument

flask_jsonapi/nested_resource_repositories.py, line 10 at r4 (raw file):

nested_schema

this should be a required argument, no? if yes, then the "if" below is not required.

flask_jsonapi/nested_resource_repositories.py, line 24 at r4 (raw file):

get_nested_schema_kwargs

how about "get_list_view_kwargs" ?

flask_jsonapi/nested_resource_repositories.py, line 25 at r4 (raw file):

super()

self

Comments from Reviewable

Reviewed 2 of 10 files at r5, 8 of 10 files at r6, 9 of 9 files at r7.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Minify JS files

It's needed to minify JS files in Gulp tasks js and js-vendor after Babel. To do this, gulp-uglify plugin can be used.

I'd like to solve it.

It's all yours!


Use RegisterReadOnly interface, not RegisterData, in views

This migrates the views to use the RegisterReadOnly interface instead
of RegisterData.  It also adds getRegisterName() and
getRegisterMetadata() to that interface.
My idea here is that we can start to migrate other things to the
RegisterReadOnly interface - like getting the custodian and the
branding.
This also will help with the work I'm doing towards multiple registers
in one app server.

Just a very general point - when using MVC frameworks, I've always assumed that the objects to be passed into a view should be 'beans' as opposed to 'services'. I had a quick look at the DropWizard docs and in their simple example a View class just wraps a Domain class.
The implementation of RegisterReadOnly which gets passed to the views is PostgresRegister which is a mixture of both. The 'service' methods are not being called in the view but it would be possible to do so. Is that a good thing. When we go to multi-tenancy will the Register Metadata for the request get looked up in the view?

Yeah, I'm not sure either.  I think we're in a funny situation because we have a single View that gets rendered to different serialisations, which have different requirements.
In particular, right now you have to provide a Registry and a Branding to each subclass of AttributionView when it's created, and you get these by calling the GOV.UK organisations API.  But they only get used by the HTML representation - not by any of the others - so we're paying the price of accessing the API even when we don't need to.
We could save this cost by making the View responsible for getting the data, on-demand, from the API, so that the API call is only made when the Registry/Branding is needed.
So.. while in principle I agree that views should just be bags of data; I also disagree.  And I think some of these problems would go away if we moved away from using Dropwizard for HTML/CSS/JS stuff.

going to go away and think about what I've done


code crashes

VSCode Version:
OS Version:

Version: 1.41.1
Commit: 26076a4de974ead31f97692a0d32f90d735645c0
Date: 2019-12-18T15:04:31.999Z
Electron: 6.1.5
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Linux x64 3.10.0-514.26.2.el7.x86_64
Steps to Reproduce:

Does this issue occur when all extensions are disabled?: Yes
Insider build also crashes.
[main 2020-01-31T22:25:10.455Z] args: {
_: [],
diff: false,
add: false,
goto: false,
'new-window': false,
'reuse-window': false,
wait: false,
version: false,
help: false,
telemetry: false,
'list-extensions': false,
'show-versions': false,
verbose: true,
log: 'trace',
status: false,
'prof-startup': false,
'disable-extensions': true,
'disable-gpu': true,
'extension-development-confirm-save': false,
logExtensionHostCommunication: false,
'skip-getting-started': false,
'skip-release-notes': false,
'sticky-quickopen': false,
'disable-restore-windows': false,
'disable-telemetry': false,
'disable-updates': false,
'disable-crash-reporter': false,
'disable-user-env-probe': false,
'skip-add-to-recently-opened': false,
'unity-launch': false,
'open-url': false,
'file-write': false,
'file-chmod': false,
'driver-verbose': false,
force: false,
trace: false,
'force-user-env': false,
'no-proxy-server': false,
nolazy: false,
'force-renderer-accessibility': false
}
Server response: <?xml version="1.0" encoding="UTF-8"
[main 2020-01-31T22:26:43.867Z] [VS Code]: detected unresponsive
Gtk-Message: GtkDialog mapped without a transient parent. This is discouraged.
[main 2020-01-31T22:26:43.891Z] telemetry/windowerror { type: 1 }
[main 2020-01-31T22:26:46.487Z] Lifecycle#window.on('closed') - window ID 1
[main 2020-01-31T22:26:46.487Z] Lifecycle#onWillShutdown.fire()
[main 2020-01-31T22:26:46.491Z] [storage state.vscdb] close()
[main 2020-01-31T22:26:46.493Z] Lifecycle#app.on(window-all-closed)
[main 2020-01-31T22:26:46.493Z] Lifecycle#app.on(before-quit)
[main 2020-01-31T22:26:46.493Z] Lifecycle#onBeforeShutdown.fire()
[main 2020-01-31T22:26:46.497Z] Lifecycle#app.on(will-quit)
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib64/libthread_db.so.1".
Core was generated by /home/xxxxxx/.local/VSCode-linux-x64/code --type=renderer --disable-color-cor'. Program terminated with signal SIGILL, Illegal instruction. #0  0x00002aba9bfc5192 in ?? () warning: File "/tools/oss/packages/x86_64-rhel6/gcc/9.1.0/lib64/libstdc++.so.6.0.26-gdb.py" auto-loading has been declined by your auto-load safe-path' set to "$debugdir:$datadir/auto-load".
To enable execution of this file add
add-auto-load-safe-path /tools/oss/packages/x86_64-rhel6/gcc/9.1.0/lib64/libstdc++.so.6.0.26-gdb.py
line to your configuration file "/home/xxxxxx/.gdbinit".
To completely disable this security protection add
set auto-load safe-path /
line to your configuration file "/home/xxxxxx/.gdbinit".
For more information about this security protection see the
"Auto-loading safe path" section in the GDB manual.  E.g., run from the shell:
info "(gdb)Auto-loading safe path"
(gdb) bt
#0  0x00002aba9bfc5192 in ?? ()
#1  0x00000895fcc18000 in ?? ()
#2  0x00000895fcc9cbc0 in ?? ()
#3  0x00002ababc9b2470 in ?? ()
#4  0x00002aba99194bf5 in ?? ()
#5  0x00000895fcc18000 in ?? ()
#6  0x00000895fcc18000 in ?? ()
#7  0x00002aba97e8b58c in ?? ()
#8  0x00000895fcc18000 in ?? ()
#9  0x00002ababc9b2460 in ?? ()
#10 0x00000895fcc18000 in ?? ()
#11 0x00000895fcc9caf8 in ?? ()
#12 0x00000895fcc9caf8 in ?? ()
#13 0x00000895fcc9cbb0 in ?? ()
#14 0x00000895fcc9cbb8 in ?? ()
#15 0x00002ababc9b2520 in ?? ()
#16 0x00002aba991a191f in v8::Object::Set(v8::Localv8::Context, v8::Localv8::Value, v8::Localv8::Value) ()
Backtrace stopped: previous frame inner to this frame (corrupt stack?)

/needsMoreInfo

@twen2000 Give us your code so we can help

/needsMoreInfo

/needsMoreInfo

@sportshead What extra info do you need? The crash happens consistently during the editing(it seems especially after switching workspace). I don't see this issue in Mac though.

Try giving some of the code or steps to reproduce so we can help

It seems the crash happens when I have multiple root directories under a same workspace. I have 3 workspaces which two of them have multiple root directories. Since some directories have very large # of files, I have the following setting to disable file watches.(I am not root, so I can't increase fs.inotify.max_user_watches
"files.watcherExclude": {
    "**/**": true,
}

After I placed all my directories into a single directory which is a single root of a workspace, I haven't run into the crash after a few days.

@twen2000 is this still an issue with latest insiders https://code.visualstudio.com/insiders ?


Change code to AliasCode for Route, Frequency and Unit

See JIRA ticket 517
Change code to aliascode class for:

AgentAdministration.route
AgentAdministration.frequency
Quantity.unit

Only model and API changes needed.

Item
Ticket

Model
#326

CT
Not applicable

API
#325

IG
?

Examples
#346

Reviewed all subtickets: 1 question remaining for example files => accounted for quanty unit related to SubjectEnrollment?


Dependency API Issues

Type of issue: API concern
Impact: API modification
Development Phase: request
Summary
There are some concerns that the merged Dependency API (#1079) may be insufficient for all use cases, notably Aspects (https://github.com/freechipsproject/chisel3/pull/1077/) and repeated application of the a sequence of phases (https://github.com/ucb-bar/barstools/commit/787740).
I'm marking this as Milestone 1.2.0 so that either this is either blocking on 1.2.0 and this gets fixed or it's bumped back to 1.3.0 and we back out #1079.
Roughly, there's a couple of use cases where you would like to run the same Transform multiple times with different arguments/annotations.
Detailed Examples
Aspects
$toBePopulated
Nested Group and Dedup
As a Chisel user (driving towards a tape-out), I would like to be able to recursively group components into modules.
Put differently, this means that a user wants to run firrtl.transforms.GroupComponents multiple times back-to-back. This also means that the user wants to run firrtl.transforms.Dedup between groupings. (Equivalently, this is back-to-back runs of firrtl.transforms.GroupAndDedup.) This is complicated by the fact that each run of GroupComponents needs to have a different set of GroupAnnotations to get the correct ordering.
Currently, there is no way for the Dependency API to represent such an ordering because prerequisites and dependents are of type Seq[Class[A]] meaning there exists no way to define dependencies between objects.
Proposed Solutions/Partial Solutions
Proposed ideas are summarized below. Note: these are sometimes overlapping and sometimes mutually exclusive.
Dependencies Specified as Objects
Currently, the Dependency API specifies dependencies as classes. This could be converted to objects.
DependencyManager Inputs are Objects
The Dependency API could stay as classes, but the inputs to a DependencyManager, "targets" and "current state", could be objects.
There could be an optional guarantee that the ordering of the objects specified must be preserved.
DependencyManager Supports Objects and Classes
Define targets as both objects and classes.
Define a "Mapping" Annotation
Some of these ideas need a way of mapping specific annotations to specific transforms. This would require defining some function that splits/mangles/duplicates the AnnotationSeq so that the requested annotations are given to a specific instance of a transform.
Non-Solution (Stacked Annotations)
Alternatively, this is working as intended and the Dependency API should stay as specifying dependencies as classes. This would then mean that for users who want object dependencies, they need to hide this inside a Dependency API-compliant class.
One non-solution a user could implement is to use a stack of annotations. Annotations can be packed into a "stacks" of Seq[AnnotationSeq, Seq[AnnotationSeq]]. This stack results in repeated application of a Transform with dependency resolution between applications until the stack is empty.
Discussion
2019-07-10
Participants: Adam, Schuyler
Differences of Aspect Ordering
Considering a design (D) and two aspects (A1 and A2), does the ordering in which the aspects are applied matter? Or, is aspect "application" commutative? Further or: are the following equivalent?
(D + A1) + A2
(D + A2) + A1

If the aspect is an injecting aspect (adding some Chisel code), then no, due to last connect semantics. Consider an aspect that sets a signal to zero and an aspect that sets that same signal to one. Due to last connect semantics, the later applied aspect overwrites the earlier applied aspect.
Annotation Binding
Consider the following annotation sequence and sequence of transforms:
val annos: AnnotationSeq = Seq(Z(1), X, Z(2), Y)
val transforms: Seq[Transform] = Seq(TZ, TX, TZ, TY)

If a user wants to send Z(1) to the first TZ and the second Z(2) to the second TZ, how would you do this?
After some back and forth, the tentative conclusion is that this is not allowable. A user should have one application of TZ and an annotation that contains the merged version of Z(1) and Z(2).
Consider another example of two annotations that set the initialization of a register target to a specific value:
/* An annotation that will set the initial value of target to value */
case class SetTo(target: Target, value: Int)

/* Two annotations, the first sets foo to one, the second sets foo to zero */
val annos: AnnotationSeq = Seq(SetTo(foo, 1), SetTo(foo, 0))

Here, if one of these is passed to one resolving annotation and the other to a second instance, then the order in which the transforms runs determines the final value of the register foo.
If these two annotations are both emitted by Chisel, then there needs to be some way to resolve what the user intent was. @azidar proposed the concept of merging annotations. Here annotations define how they should be merged together and a Phase does this (similar to how annotations are updated based on their specified renaming rules). In effect, annotation merging is a means of coalescing multiple annotation input arguments to a transform down to one object.
This removes the need for the Dependency API to consider ordering multiple instances of the same class.

After additional discussion on this, the (current) conclusion is that:

Ordering of transform instances is out of scope of the dependency manager.
If a transform writer wants to implement their first transform in such a way that the transform calls multiple instances of some other second transform, it's their job to both use a packed annotation that contains enough information for the first transform to know how to run instances of the *second transform.

This likely pushes towards a refactor for one multi-target annotation per transform as opposed to scattered single-target annotations.
The caveat here is that neither I nor @azidar were able to come up with a motivating use case that required being able to express instance orderings and could not be handled in some other way (e.g., packed annotations).
Closing this (and unblocking 1.2.0) pending ideas on further use cases.


Fix removal of duplicate schemas

Bug fix / Enhancement
This PR contains removal of duplicate schema entries.

Thanks for the contribution. The changes are ok but could you ensure our DCO check passes by following the guidelines here

Hello @Yadunund I made the appropriate changes for the DCO, I believe its ready to be merged. Thank you!

Got it. No worries, I made the necessary changes, it should now merge to main without conflicts.

Looks like DCO is failing now 😅

hehe it seems that an older commit sha: 48e78cb, is causing that. Anyways I will create a fresh PR from a new branch: #294


fix: hybriddatamanager model validation errors

Remove empty "body" for RESPONSE_SCHEMA_NOT_IN_SPEC errors

Automation for azure-sdk-for-go

Automation for azure-sdk-for-python

Can one of the admins verify this patch?


Add support to execute tests in workerd

Motivation
I'm already developing for the workers-rs platform (workerd)

About 1000 test templates running with wasm-bindgen-test and firefox
Almost 400 in a single crate
In the heavier setup they take almost 60h to complete on a single core
Workers-rs only has about 50 tests in JS/TS running in vitest

I need to

Run tests on workerd - for specific functionality
Better test output - 400 random organised tests in a crate are hard to check
Distribute the load between multiple executing instances
Code Coverage

I already have some support and I plan to tackle this problems as much as possible, I'm highly motived and self reliant.
But I believe that your guidance is quite important:

To maintain compatibility with your API
To say how you want things done so that its a smooth process for all
To support the provided test runner in your test runner like you support Firefox/Chrome/Safari/Node if you want
To point me to the relevant information and to relevant steps in the process

Proposed Solution
The expected roadmap should be something like:

Create an initial workerd test runner
Execute in compatibility mode with other platforms
Integrate with your test runner
Execute with workerd specific functionality
Add support for running tests in parallel inside the test runner
Improve test output
Analyse if its possible to add code coverage
Integrate wasm-bindgen-test with nextest
Add support for cargo-mutants

Workerd is started and configured by Miniflare, I believe it will be a test runner in JS/TS, that I'm guessing will be opaque like the other testing platforms.
Although some kind of streaming of the output could be nice.
Alternatives
I have discussed some details of the strategy with Miniflare lead dev several times, it seems reasonable and recently I was no longer told to wait, that I could start moving forward.
The integration of the runner with your test runner, would be something nice to have, I don't mind working on a more general solution that also allows other test runners.
Additional Context
Back on #3510
@daxpedda said:

Apologies, you are right your case is different, I didn't pay enough attention.
I think the same philosophy applies to the test runner as well. The preference here it not to have to cover every use case there is in wasm-bindgen. So as you said, CF should provide their own test runner.
That said, if the wasm-bindgen test runner can be adjusted to provide a general way for other runtimes/platforms to inject what they need to get things to work, that would be great as well. That would obviously require some design work, but considering the lack of time from maintainers this has to be done in small steps that can be easily discussed and reviewed.

@hamza1311 said:

using wasm-bindgen produced test files to run the tests with your test runner using workerd is a bit of a different use case.

That is a use case I would love to support. There's also a point to be made of running wasm-bindgen output in non-node.js environments like Deno or Bun. I think this PR is out of scope for this discussion. Can you create a new issue for this so we can discuss how to handle this? I think we can take inspiration from the Jasmine testing framework in the JS ecosystem

3. Distribute the load between multiple executing instances

wasm-bindgen-test should mimic Cargo test as much as possible, parallelizing tests even more is done in Cargo test externally, e.g. nextest. So the goal would be to figure out how we can offer integration with wasm-bindgen-test for external test runners (or figure out what the current issue is).

4. Code Coverage

Code coverage is being tracked in https://github.com/rustwasm/wasm-bindgen/issues/2276 and there was already a working implementation in https://github.com/rustwasm/wasm-bindgen/pull/3782, which basically just needed a rebase.
It would be good however if somebody took a look at how to fix llvm-cov to find the debug information, see https://github.com/rustwasm/wasm-bindgen/pull/3782#issuecomment-1913110704.

The integration of the runner with your test runner, would be something nice to have, I don't mind working on a more general solution that also allows other test runners.

This is the path we want go down. A proposal how this would be tackled and some design work would be whats needed here.
I'm happy to discuss and look at any more specific proposals.

@daxpedda I gave a look to nextest, it's indeed relevant and in end I would like to support it also, but the missing piece seems to be in nextest.
I tested it and got the message:
info: for the target platform, using target runner `wasm-bindgen-test-runner` defined by `target.wasm32-unknown-unknown.runner` within `/Users/alexander/Developer/rust/.cargo/config.toml`
error: creating test list failed

Caused by:
  for `core_application_handler`, line 'no tests to run!' did not end with the string ': test' or ': benchmark'
full output:
no tests to run!

I'm guessing its best to add support for workerd first.
But recognising its relevance, I'm taking it in consideration, its documentation says:

By default, cargo test uses this execution model:
each test binary is run serially, and binaries are responsible for running individual tests in parallel.

A cargo-nextest run has two separate phases:
The list phase. cargo-nextest first builds all test binaries with cargo test --no-run, then queries those binaries to produce a list of all tests within them.
The run phase. cargo-nextest then executes each individual test in a separate process, in parallel. It then collects, displays and aggregates results for each individual test.

With cargo test its up to the binary to run individual tests in parallel, so in that model it would make sense for the test-runner to run tests in parallel launching multiple vms and distributing the load.
To support the cargo-nextest model its trickier because a simpler version would imply starting a vm, running a single test and return.
Starting a vm about 400 times for a single crate can be a bit expensive I guess.
So i'm guessing, that we need to have a server that manages all the executions and gets called by multiple clients, that is able to work in both modes.
I'm going to update my roadmap to include nextest and cargo-mutants.

With cargo test its up to the binary to run individual tests in parallel, so in that model it would make sense for the test-runner to run tests in parallel launching multiple vms and distributing the load.
To support the cargo-nextest model its trickier because a simpler version would imply starting a vm, running a single test and return.

I'm not following where VMs come into play here. Seeing the error you posted, it seems to me the issue in nextest is that the output by wasm-bindgen-test isn't the same as cargo test, which is why it is unable to compile a list of all tests to then run them on its own.
This is a general issue in wasm-bindgen-test, but should fix compatibility with nextest as well.
Let me know if I missed anything here!

@daxpedda Well if the problem is with wasm-bindgen-test that isn't providing the proper test list, I can start with that, no problem.
I need to learn that part anyway.

Regarding the execution, nextest invokes the binary once for each test, that would imply starting the browser / node / workerd, run the test and exit, in a simple direct implementation.

@daxpedda Well if the problem is with wasm-bindgen-test that isn't providing the proper test list, I can start with that, no problem.

I think that's a good step forward: aligning the output of wasm-bindgen-test to cargo test.

Regarding the execution, nextest invokes the binary once for each test, that would imply starting the browser / node / workerd, run the test and exit, in a simple direct implementation.

This might be a bit too much overhead for parallelizing tests with wasm-bindgen-test, starting the browser consumes way more resources compared to starting single tests with cargo test. We might end up either adjusting nextest to account for that or actually consider implementing some level of parallelization in wasm-bindgen-test indeed.
In any case, this is something to ponder upon later when we actually have some data.

@daxpedda Regarding supporting cargo nextest, its still work in progress but I created PR #3920, so that we can discuss current status and address some issues I'm having. Thanks!


Trying to add a death animation that is not the default BF death animation to my custom character.
I made their json, with the death animation's sprites, bla bla bla.. The .png for the character also has the death animation, all of it. firstDeath, deathLoop, deathConfirm. Also added the script for custom death animations in the scripts folder. Spelled it all right in the script.. even with all that, whenever i select the custom character and die with them, the custom death animation still doesn't play, and it just plays the default BF death animation.
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
No, only touched the mods folder for adding songs, character jsons, and pngs.

what version are you using? if your using 0.7 and newer then theres a data tab in chart editor where u can do it easily.

what version are you using? if your using 0.7 and newer then theres a data tab in chart editor where u can do it easily.

I'm using 0.5.1

0.5.1 is outdated, and is no longer supported

oh i see


Bazel Build fails with missing input file error @grpc

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No. Stock script is used.
OS Platform and Distribution: CentOS Linux release 7.2.1511.
TensorFlow installed from : Source.
Bazel version: 0.4.5
CUDA/cuDNN version: 7.5
Exact command to reproduce:
./bazel build --config=opt --config=cuda --spawn_strategy=standalone --genrule_strategy=standalone //tensorflow/tools/pip_package:build_pip_package

Describe the problem
I am installing Tensorflow 1.1 on a system where I do not have root priviliges. Upon running Bazel command (mentioned above), I get the following error -
missing input file '@grpc//:include/grpc++/impl/codegen/string_ref.h'.
I need to include the path of gprc to the Bazel. However, I do not know how to find the location of these files and how to refer them to Bazel. I have tried to find these files in the filesystem and bazel downloads, but couldnt find them.
Another insteresting thing that is happening is -
Everytime I run the bazel command again, it gives me different error each time ( However, they all are related to "@grpc//: ". Some of the other errors I got were -
ERROR: missing input file '@grpc//:include/grpc++/impl/service_type.h'.
ERROR: missing input file '@grpc//:LICENSE'.
ERROR: missing input file '@grpc//:include/grpc++/server_builder.h'.
etc.
Error Log
ERROR: missing input file '@grpc//:include/grpc++/impl/codegen/string_ref.h'. ERROR: /home_dir/xxxxx/nn/installation/tensorflow/tensorflow/core/BUILD:1308:1: C++ compilation of rule '//tensorflow/core:lib_hash_crc32c_accelerate_internal' failed: crosstool_wrapper_driver_is_not_gcc failed: error executing command external/local_config_cuda/crosstool/clang/bin/crosstool_wrapper_driver_is_not_gcc -U_FORTIFY_SOURCE '-D_FORTIFY_SOURCE=1' -fstack-protector -fPIE -Wall -Wunused-but-set-parameter ... (remaining 43 argument(s) skipped): com.google.devtools.build.lib.shell.AbnormalTerminationException: Process terminated by signal 15. ERROR: /home_dir/xxxxx/.cache/bazel/_bazel_xxxxx/feba725fe17da6f4082aa7bd94139644/external/grpc/BUILD.bazel:1664:1: @grpc//:grpc++_unsecure: missing input file '@grpc//:include/grpc++/impl/codegen/string_ref.h'. Target //tensorflow/tools/pip_package:build_pip_package failed to build Use --verbose_failures to see the command lines of failed build steps. ERROR: /home_dir/xxxxx/.cache/bazel/_bazel_xxxxx/feba725fe17da6f4082aa7bd94139644/external/grpc/BUILD.bazel:1664:1 1 input file(s) do not exist. INFO: Elapsed time: 13.184s, Critical Path: 2.25s

Bazel clean solved this issue.
Thanks.


Feat/monthly radolan aggregation

Needs https://github.com/technologiestiftung/giessdenkiez-de-postgres-api/pull/268

Closed in favor of https://github.com/technologiestiftung/giessdenkiez-de-dwd-harvester/pull/132


iOS mightier_amp unable to load backing tracks

Several issues identified in current iOS release:

Adding tracks is not possible without an Apple Music subscription as +icon for adding songs to Tracks only opens an empty Apple Music Playlist.  There is currently no option available to browse for locally stored backing tracks.
With an active Apple Music subscription, selecting a song from an Apple Music Playlist fails to save the song into the Track list. The Track list remains empty.

Edit: Apple Playlist working in iOS 15.7.3 (older iPhone) not IOS 16.4.1a on iPhone and iPad. Still unable to browse for Local files.

The current version can load tracks from the Media Library only - that is when you sync tracks from your computer, which will appear in the Apple Music app. I didn't know that the tracks from the Apple Music subscription will appear as well. Those are unplayable in Mightier Amp due to DRM, but there's a way to hide them, which I'll do in the next update.
I don't know why the newer iOS versions don't let you pick from tracks that you synced manually. Unfortunately, my iPhone is too old and can't update to the latest iOS.
I'll add browsing for tracks using a file picker in the next update. It will let you add not only local tracks but tracks from iCloud, Google Drive, and any other cloud services that you have installed.
All this comes from the fact I'm new to iOS and not familiar with how people add their music. I thought most users sync their music with the media library.

I thought most users sync their music with the media library.Synching requires access to a pc/Mac running iTunes. Without Apple sub, with Win 11 on a very old i3 laptop with 2 gb mem, you need to connect iPad/iPhone to laptop, copy the files to desktop, open iTunes and drag them into mighty amp app. It’s quite tedious on an old clunker.With backing tracks already on iPhone/iPad the ideal solution would be file picker and then add directly mighty plug. I used to save my own music to Apple media library but when Apple Music sub ran out it empties the media library including your own music files. My old iPhone 6s Plus on iOS 15.7.3 can save tracks from music library but it is wanting to update to 15.7.5 so hopefully this doesn’t break the track saving. My iPad and iPhone 12 mini can’t save tracks from music library. iOS 16.4 . Anyway to fix this? Hopefully the file picker helps with saving local media.I didn't know that the tracks from the Apple Music subscription will appear as well. Those are unplayable in Mightier Amp due to DRM, but there's a way to hide them, which I'll do in the next update.The Apple Music files saved in the mightier amp app on my 6s actually do play. Tracks only show up with an active sub so I would expect them to play. No active sub and media library is empty, so nothing to load. I don’t think you need to hide it, just need to be able to choose from either Apple Music or Files.Here’s how it looks on just chords track selector. Files gives you file access and Apple Music accesses media library.Sent from my iPadOn 16 May 2023, at 8:23 pm, tuntorius @.***> wrote:
The current version can load tracks from the Media Library only - that is when you sync tracks from your computer, which will appear in the Apple Music app. I didn't know that the tracks from the Apple Music subscription will appear as well. Those are unplayable in Mightier Amp due to DRM, but there's a way to hide them, which I'll do in the next update.
I don't know why the newer iOS versions don't let you pick from tracks that you synced manually. Unfortunately, my iPhone is too old and can't update to the latest iOS.
I'll add browsing for tracks using a file picker in the next update. It will let you add not only local tracks but tracks from iCloud, Google Drive, and any other cloud services that you have installed.
All this comes from the fact I'm new to iOS and not familiar with how people add their music. I thought most users sync their music with the media library.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@tuntorius  Any progress on a fix/update for iOS? Currently it’s unable to load ANY backing tracks on iOS 16. Your app is so much better than the Nux version that I use it regardless of backing track issue (and use Bluetooth for BT’s), but it would be really nice to be able to have them on the app. Happy to test any beta via TestFlight.

Latest beta 1.0.1 (39)
Still unable to access tracks. The previous version was able to load Apple Library tracks on an earlier iOS version (ios15) phone, but not iOS 16 iPad/iPhone.
Unfortunately the current beta not longer accesses tracks on either version of iOS.

Beta 1.0.2 the file picker switching to track edit and back to file picker is no longer crashing app.

Beta 1.0.2.(42)
Connects to MP3 Pro but Drums no longer working. Other features working
Connects to Mighty BT 20 but incorrectly reads fx settings for the selected amp style.

Beta 1.0.2 (43)
MP3 track edit and drums working again.
Unable to connect to Mighty BT 20 amp.


Feature request : Add support for Openrouter, cohere, anthropic ,together, llamacpp and Google Gemini Api

Hello and how are you.
in future development can you add

Openrouter,
cohere,
anthropic
,together ai,
llamacpp and
Google Gemini Api
as part of engine and module for translation ?
thanks.

hi @tukangcode, sounds fantastic! I am adding them to roadmap.


winit: upgrade to 0.23.0 / move back upstream!

I also threw in some fixes to prevent wasm deps from being compiled on linux platforms. On linux, we're now down from 311 dependencies to 266!

Nice! This makes my heart happy. 😄


feat(cmdline): allow configuring separate cmdline preset

This PR does two things:

allow configuring a different keymap for the cmdline
added a default preset more closely matching the default Neovim cmdline keymaps

Closes #514

Thinking about this more, specifically for the cmdline, users will probably want to customize other config options as well.
Like auto_insert etc, so maybe a better approach (but way more code changes) would be to allow a complete different config for the cmdline.

This PR is indeed not enough.
Personally I'd want to have auto_insert enabled, but with no preselect and use <tab> / <S-Tab> to cycle between results.
Not sure what the best approach is.

so maybe a better approach (but way more code changes) would be to allow a complete different config for the cmdline.

That would make the entire config dynamic which would make this much more complicated. I'd like to avoid this if possible

Personally I'd want to have auto_insert enabled, but with no preselect and use <tab> / <S-Tab> to cycle between results.

So you'd like auto_insert in default mode and preselect in cmdline mode just to clarify?

I think so? :)
I don't have anything configured regarding preselect and auto_insert right now for blink.
I want to achieve a similar setup as with the regular cmdline, as in:
When I type T, the menu opens like this:

<tab> then cycles forward (<s-tab> backwards) and inserts the items when doing so, so you can press <cr> anytime to exeucte

These are the options I set:
vim.opt.completeopt = "menu,menuone,noselect"

The screenshots are Noice obviously, but that behaves exactly as the normal cmdline regarding completion.

So you'd like auto_insert in default mode and preselect in cmdline mode just to clarify?

Maybe make blink.cmp.select_next() take an extra argument of type same as blink.cmp.config.completion.list.selection ?
So we can set keymaps like this:
['<Tab>'] = {
function(cmp)
    if vim.fn.getcmdtype() == '' then
        return cmp.select_next({select_behaviour = 'preselect'})
    end
    return cmp.select() -- selection_behaviour defaults to `auto_insert`
end }

@the-fuckin-nobody though about doing it that way initially, but in my case, the first entry would already be pre-selected, so you can't use <tab> to next&insert then.

@the-fuckin-nobody though about doing it that way initially, but in my case, the first entry would already be pre-selected, so you can't use <tab> to next&insert then.

If I am understanding it correctly, is this the desired behaviour?
https://github.com/user-attachments/assets/51bbf44e-e2e3-4819-9bfe-af9622999d49

Yep, exactly.

Yep, exactly.

Then, it is actually doable with the current state of the plugin, by configuring the tab  keymap like this like this:
["<Tab>"] = {
				function(cmp)
					if vim.fn.getcmdtype() ~= "" then
						return cmp.select_next()
					elseif cmp.snippet_active() then
						return cmp.accept()
					end
				end,
				"snippet_forward",
				"fallback",
			},

But in my case I would prefer to have selection behaviour as preselct for normal completion menu and auto_insert for cmdline completion menu.

But in my case I would prefer to have selection behaviour as preselct for normal completion menu and auto_insert for cmdline completion menu.

Yep same here

Yep same here

I just found this line while skimming through the codebase , but skip_auto_insert has not been implemented yet. Implementing this may achieve the purpose

any progress about handling the 'special' auto_insert for cmdline?

any progress about handling the 'special' auto_insert for cmdline?

That should be done in another PR I guess cuz its unrelated to this PR(cmdline keymaps).

Yep, I'm still working out how I want to solve it

Yep, I'm still working out how I want to solve it

Take a break bro not everything must be implemented right away!
PS: You're awesome

any progress about handling the 'special' auto_insert for cmdline?

for those who find here and want some temp fix:
here is a snippet💖:
(remember to set <CR> to {} )
init = function()
        -- FIX: Temporary for not being able to set auto_insert for cmdline
        local orig_list_selection = nil
        vim.api.nvim_create_autocmd("CmdlineEnter", {
            callback = function()
                local list = require "blink.cmp.completion.list"
                orig_list_selection = list.config.selection
                list.config.selection = "auto_insert"
            end,
        })
        vim.api.nvim_create_autocmd("CmdlineLeave", {
            callback = function()
                if orig_list_selection then
                    local list = require "blink.cmp.completion.list"
                    list.config.selection = orig_list_selection
                end
            end,
        })
    end,

ref https://github.com/Parsifa1/nvim/blob/4ce2ab13d428f3efa6ae269bff030f033f59dceb/lua/plugins/lsp/blink.lua


heartbeat gm is maybe out of tune

heartbeat gm is maybe out of tune. AKWF is on D, not C...
We need to test/investigate this.

fixed


refactor: use CSS Modules for toggle styles

Breaking Changes:

Former react-toggle classes have been removed in favor of css module classes (with hash). This may break some custom CSS selectors you have if you customized the toggle with CSS classes.

Motivation
Resolves #5586
Although this would require changes to styles of sites with toggle customization, it is a better approach that is consistent with our other components.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Preview
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

Thanks, LGTM 👍


Cannot import "exports": { "module": ... } without type=module configured packages, instead we could get an error: Cannot use import statement outside a module

Reproduction
Quick: Go to https://stackblitz.com/edit/remix-run-remix-cqepq2?file=app%2Froutes%2F_index.tsx
Or see https://github.com/remix-run/remix/pull/9071

Initialize remix plain project.
Install<EMAIL_ADDRESS>(Don't worry! It's tiny library, fully zero dependency, and I confirmed this is caused by some conditions with just rewriting its package.json)
Add import statement for 'use-immer'

The package which causes this problem has these conditions:

has no type: module in package.json
has exports: { ".": { import: "something" }} in package.json

The package.json in<EMAIL_ADDRESS>you can see like this:
{
  "name": "use-immer",
  "version": "0.9.0",
  "description": "Use immer with React hooks",
  "main": "dist/use-immer.js",
  "umd:main": "dist/use-immer.umd.js",
  "module": "dist/use-immer.module.js",
  "jsnext:main": "dist/use-immer.module.js",
  "react-native": "dist/use-immer.module.js",
  "typings": "dist/index.d.ts",
  "source": "src/index.ts",
  "exports": {
    ".": {
      "require": "./dist/use-immer.js",
      "import": "./dist/use-immer.module.js",
      "types": "./dist/index.d.ts"
    }
  },
  "sideEffects": false,
  "scripts": { omitted.. },
  "repository": { omitted.. },
  "files": [ omitted.. ],
  "keywords": [ omitted.. ],
  "author": "Michel Weststrate",
  "license": "MIT",
  "bugs": {
    "url": "https://github.com/mweststrate/use-immer/issues"
  },
  "homepage": "https://github.com/mweststrate/use-immer#readme",
  "peerDependencies": { omitted.. },
  "devDependencies": { omitted.. }
}

And we can confirm this satisfies the condition.
Actually, if I add "type=module" to this, and restart remix run, it succeeds to import.

Extra contexts
related:

https://github.com/remix-run/remix/discussions/6168
https://github.com/remix-run/remix/discussions/9029

This could be the bug of vite. I can create issue instead on vite, but I'm not sure for now.
The most important part of this issue is I got small reproduction for these issues (currently discussions).
~I'm also making PR for Option 1 after submitting this issue.~ Done.
By the way, you may think, such packages should have type: module in their package.json. Actually, import from these packages from esm does not work, and multi-support is actually not possible by native level. Even if type=module is added, now, it cannot be require-able.
Sadly, this is complicated situation. Node.js, TypeScript and Webpack treats exports as their own way. Webpack says as https://webpack.js.org/guides/package-exports/#support .
Thanks.
System Info
System:
    OS: Linux 5.19 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (8) x64 11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz
    Memory: 7.87 GB / 15.36 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 20.11.0 - /usr/bin/node
    npm: 10.2.4 - /usr/bin/npm
    bun: 1.0.29 - ~/.bun/bin/bun
  Browsers:
    Chrome: 114.0.5735.133
    Chromium: 122.0.6261.128
  npmPackages:
    @remix-run/dev: ^2.8.1 => 2.8.1
    @remix-run/node: ^2.8.1 => 2.8.1
    @remix-run/react: ^2.8.1 => 2.8.1
    @remix-run/serve: ^2.8.1 => 2.8.1
    vite: ^5.1.0 => 5.1.6

Used Package Manager
npm
Expected Behavior
Not get error and just can import it as ESM module as usual.
Actual Behavior
Got an error:
Cannot use import statement outside a module
    at internalCompileFunction (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:180:1005)
    at wrapSafe (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:54:14101)
    at Module._compile (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:54:14512)
    at Module._extensions..js (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:54:15550)
    at Module.load (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:54:13457)
    at Module._load (https://remixrunremixcqepq2-01ee.w-credentialless-staticblitz.com/blitz.810981ba.js:54:10535
Click outside, press Esc key, or fix the code to dismiss.
You can also disable this overlay by setting server.hmr.overlay to false in vite.config.ts.

We have the workaround for this, just with importing by relative path:
- import { useImmer } from 'use-immer';
+ import { useImmer } from '../../node_modules/use-immer';

Is there any update on this issue? I am experiencing it with v2.9.2.
@remix-run/dev: 2.9.2
@remix-run/node: 2.9.2
@remix-run/react: 2.9.2
@remix-run/serve: 2.9.2

I've been running into the same issue with several packages such as react-content-loader and the workaround mentioned above seems to work for my case too.

I encountered the same issue when using chartjs-react.
Following the suggestions from the original poster and this comment, I tried the following two solutions, both of which resolved the issue:

Add { "type": "module" } to lib's package.json file
Move the relevant code to a .client.tsx file

Encountered the same when trying to reexport some gsap defaults.
// app/lib/gsap.ts
import { gsap } from "gsap";
import { useGSAP } from "@gsap/react";

import { ScrollTrigger } from "gsap/ScrollTrigger";
import { TextPlugin } from "gsap/TextPlugin";

gsap.registerPlugin(useGSAP, ScrollTrigger, TextPlugin);

gsap.defaults({
  ease: "power2.out",
  duration: 0.5,
});

export { gsap, useGSAP };

Upon import it shows the error message.
// app/routes/_index.tsx
...
import { useGSAP, gsap } from "@/lib/gsap"; // ← Throws the error
// ↓ This works!
import { gsap } from "gsap";
import { useGSAP } from "@gsap/react";
...


[Question] Is there a possibility to skip "Try it out" button?

I have a recent version of springfox-swagger-ui and springfox-swagger2 (2.9.2)
After click on specific endpoint element

I want to not must click on "Try it out" button

But instead to have Execute button available to click - just after clicking on endpoint element

Is this possible?

Super :D, do you mind closing this issue?

I can't :), either you or @dilipkrish can.

Ow right - it's my first issue on github :D
So for now I close the topic - maybe when this'll be added in the future in swagger-ui I will create other to support that. Thanks


'az ad group member add' does not return any result

Is your feature request related to a problem? Please describe.
Most az cli commands that I've used so far return some sort of json object on success.  I've been using the presence of a valid json object to check for successful completion of the command in my automation scripts.  However, az ad group member add does not return anything even when the command is successful.
Describe the solution you'd like
It would be nice if this command would return a json object on success to be consistent with other commands.

add to S169

@egeyer, thanks for the feedback.
However, this is the design of AD Graph REST API Add group members to return 204 No Content:

Status Code
Description

204
No Content. Indicates success. No response body is returned.

If you run the command with --debug, you can see the actual REST request:
> az ad group member add -g testgroup --member-id 0d504196-1423-4569-9a6e-15149656f0ee --debug

msrest.http_logger : Request URL: 'https://graph.windows.net/54826b22-38d6-4fb2-bad9-b7b93a3e9c5a/groups/2cde37aa-6fd0-42ad-849a-33d06c0da145/$links/members?api-version=1.6'
msrest.http_logger : Request method: 'POST'
msrest.http_logger : Request headers:
msrest.http_logger :     'Content-Type': 'application/json; charset=utf-8'
msrest.http_logger :     'accept-language': 'en-US'
msrest.http_logger :     'Accept': 'application/json'
...
msrest.http_logger : Request body:
msrest.http_logger : {"url": "https://graph.windows.net/54826b22-38d6-4fb2-bad9-b7b93a3e9c5a/directoryObjects/0d504196-1423-4569-9a6e-15149656f0ee"}

msrest.http_logger : Response status: 204
msrest.http_logger : Response headers:
msrest.http_logger :     'Cache-Control': 'no-cache'
msrest.http_logger :     'Pragma': 'no-cache'
...
msrest.http_logger : Response content:
msrest.http_logger :

MS Graph API Add member also has this behavior:

Response
If successful, this method returns a 204 No Content response code. It does not return anything in the response body.

You may check the exit code to determine whether the command is successful.
# bash
echo $?

# cmd
echo %ERRORLEVEL%

# pwsh
echo $LastExitCode

You may also use az ad group member check:
az ad group member check -g testgroup --member-id 0d504196-1423-4569-9a6e-15149656f0ee

{
  "odata.metadata": "https://graph.windows.net/54826b22-38d6-4fb2-bad9-b7b93a3e9c5a/$metadata#Edm.Boolean",
  "value": true
}


Create spsjm.txt

Link to the education plan where you can see the programming subject
http://www.spsjm.eu/wp-content/uploads/2017/09/ucebne_plany/UP_2694M_IST_2017-2018-2.-4.rocnik.pdf
or see the page of our "Information and network technologies specialisation"
http://www.spsjm.eu/studium/studijne-odbory-2/informacne-a-sietove-technologie-2694-6/
On a screenshot email adresses of our students with the spsjm.eu domain

Thank you. Regarding age, we should be ok, as our students are typically aged 15-19.


Can I disable Flights autoloader and use composer instead?

Hello,
is there a way to disable autoloading inside Flight and Use composer for autoloading instead? I need Flight to go through composers autoloader to be able to use it with goaop. Is there a way to do so?

Alternatively, you could call spl_autoload_functions(), search for Flight's autoloader and unregister it from class loaders stack or move it down.

Seems addressed.


Test Istio release "Istio 1.8.5"

Istio recently (2021-04-15 19:59:29 +0000 UTC) released "Istio 1.8.5". Let's test it :rocket:.

/close
Started testing 1.9.x by https://github.com/knative-sandbox/net-istio/pull/597 as 1.8.x will be EOL next month https://istio.io/latest/about/supported-releases/.


Fix jenkins failure: /.git/index.lock': File exists.

clean up untracked files, prevent git cmd from failing

Coverage decreased (-0.01%) to 51.785% when pulling c80d5a708e4a8d561bc4d633e8e852f9599ca06c on hao/fix_jenkins_git_lock into 3a81107ab246cf8cfd0a638e40fb1dcb6a9a5521 on master.

Coverage decreased (-0.01%) to 51.785% when pulling c80d5a708e4a8d561bc4d633e8e852f9599ca06c on hao/fix_jenkins_git_lock into 3a81107ab246cf8cfd0a638e40fb1dcb6a9a5521 on master.

Why git reset to another branch? Does git checkout --force work?

get reset to exactly the same branch, in order to avoid conflict.
reset mean regardless any local changes, so it would work. Still don't know why git pull fails.


Move NodeStatus update to subresources

I started to do it today, but I didn't manage to finish it.

Adding missing team label.

This is marked P1 and 1.0, but IIUC requires an API change. Am I correct? If so, I assume we're not actually going to do it for 1.0?

The API change would be additive, not break compatibility.
Feel free to reprioritize if you think this won't improve performance or simplify node controller and/or kubelet status posts.

FYI: I don't believe it'll have any performance impact. Even when using subresources we send "whole" objects though the network, so that we'll be able to check objects version id. I see it only as a cleanup for now.

e2e is green


Added #31: notifier plugins with tests.

#31

Coverage decreased (-0.8%) to 96.591% when pulling e57b92d8a30e99227a5541f65a63cf61ae8666c9 on DSilence:feature/notifiers into bc59a360f562d48f204ab9e0d4c0157a5f603bb7 on werwolfby:develop.

Current coverage is 96.59%

Merging #32 into develop will decrease coverage by -0.80% as of bc5b000

@@            develop     #32   diff @@
=======================================
  Files            30      36     +6
  Stmts          2416    2640   +224
  Branches          0       0
  Methods           0       0
=======================================
+ Hit            2353    2550   +197
  Partial           0       0
- Missed           63      90    +27

Review entire Coverage Diff as of bc5b000

Powered by Codecov. Updated on successful CI builds.

Coverage remained the same at 100.0% when pulling e247ea7fb20dd9fc17db6fba16cf7fc8e6372e95 on DSilence:feature/notifiers into ba1c3cb05881757d911ca6a26918ef2e07a4fae7 on werwolfby:develop.

Rebased with latest changes, added rest interface and UI. Also upgraded tests to 100%


[WIP] Doc site as a theme designer and sharing app

This PR is exploring the ability for the doc site to become a theme designer.  It would also enable users to save, share, and spin off of other user's themes.
See the RFC #2939

Codecov Report

Merging #2938 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2938   +/-   ##
=======================================
  Coverage   99.96%   99.96%
=======================================
  Files         163      163
  Lines        2742     2742
=======================================
  Hits         2741     2741
  Misses          1        1

Continue to review full report at Codecov.

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

This is now blocked.  The issue is that in browser LESS import statements do not resolve variables the same as when running in node.
In order to support this, we'll need to either:

move to CSS in JS and compile the JS on changes (preferred)
precompiled a single doc site friendly LESS file that includes all components and variables in order

I vote number 1.  It is future proof and lets us do lots of cool things with persisted themes.  It is also way faster to compiled than LESS.
Number 2 is a smaller short term step for us, but it doesn't lead anywhere great.  I do not think we would be able to provide a very good experience to users in the editor nor in how themes are created and edited.


[TextField] Handle Chrome autofill

[ X ] I have followed (at least) the PR section of the contributing guide.

Fixes https://github.com/mui-org/material-ui/issues/14427 as per given fix by @oliviertassinari

The changes should solve two structural issues: 1. the JavaScript change event was fired before we listen to it. 2. The input value is hidden for security reasons. What browser issues do you have in mind?

What browser issues do you have in mind?

That was my question. It is not obvious what issue is solved here. Only theoretical issues that have no confirmed reproduction.

Only theoretical issues that have no confirmed reproduction.

@croraf I believe that we have a reproduction for 2 different issues and confirmation for fixes with these changes. The issues are:

Somebody might write in a input before it gets hydrated. It can be reproduced on the /examples/ssr/ example with a set timeout in the hydration logic
Google might fire a change event while hiding the input value. It can be reproduced with https://github.com/mui-org/material-ui/issues/14427#issuecomment-530503051 on Browserstack Windows 10 Chrome latest.

To be clear: A reproduction is code + environment.

A user writes in an input before we listen to the event. This can be reproduced on the /examples/ssr/ example with a set timeout in the hydration logic.

What is the use case for hydrating after a timeout? Seems like a strawman to me.

Chrome autofill the input before React hydrates the from. This can be reproduced with #14427 (comment).

As I said no example in that issue has a confirmable reproduction. It might reproduce for you but without knowing the environment you cannot know if this is a library or environment issue.

Chrome doesn't fire a change event. This can be reproduced in reproduced #14427 (comment) with BrowserStack Windows 10, Chrome latest

BrowserStack is not a reliable source for reproduction. If it was we would not have tabable buttons in safari.

What is the use case for hydrating after a timeout?

For instance, you might want to wait for an optional polyfill to be loaded, over the network, before hydrating your DOM. I had the very use case at Onepixel, I was frustrated Next.js has no support for it.

As I said no example in that issue has a confirmable reproduction.

Then we can call it a nice side effect of fixing the previous issue.

BrowserStack is not a reliable source for the reproduction

Don't worry, we have enough users to report if these fixes don't cut it. I don't think that we should wait to get the perfect reproduction, we might never have it. I think that we should move fast by sacrificing the scope of the problems we solve, without sacrificing the code quality so we can learn from "production". In this case, we have an opportunity to try these changes. Worse case 1. we still have this bug, people provide a better reproduction, worse case 2. we solved the bug, but we remove the logic by mistake in the future, people report the regression. In any case, the changes are direct application of people reported solution to the solution @croraf and I have seen a positive impact on. Things are imperfect and will always be :).

Well thanks for adding technical debt that we need to refactor.

@eps1lon I'm sorry you feel this way. I understand that you have done great progress in the past simplifying the logic of this component and that you want to keep it in a great state. I will personally make sure this new logic only remains if proven valuable. Meaning:

If React solves the related issues https://github.com/facebook/react/issues/1159, https://github.com/facebook/react/issues/12955, investigate if we can remove the patch (as far as I know, these issues have been present since the first version of React).
If we still get autocomplete issue report, investigate if these changes are really relevant, look for better solutions.

But yes, the more problems we solve, the more complex the codebase is and the more inertia we have. Sometimes, it's better not to solve problems reported by our users. I don't think it's one of these cases.
Assuming that we want to solve the problem. I don't think that we went down a quick patch road. It should be resilient.

this update makes me change something to still have my transparent background for my input:
Separate css
/* Chrome hack for auto set password ;) */
@-webkit-keyframes autofill {
    to {
        color: white;
        background: transparent;
    }
}

Adaptation in theming, to still have this transparent background and not a grey-blue one
MuiInputBase: {
            input: {
                '&:-webkit-autofill': {
                    animationName: 'autofill',
                    animationFillMode: 'both',
                    animationDuration: '0s'
                  },
            }
        }

@jkeruzec CSS injection order issue?


Rebuild 0.8.3

This PR rebuilds 0.8.3 to address failed CI (network timeout).

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


Arch packaging fix

Trying to follow the instructions here: https://github.com/input-output-hk/haskell.nix/issues/229

Evidence this works: https://office.runtimeverification.com/jenkins/blue/organizations/jenkins/k/detail/PR-1877/2/pipeline


FIX #58 broken link in the README

Fixes https://github.com/bendrucker/angular-credit-cards/issues/58

Thank you!


Request for new feature

It would be nice to have a button for switching to RAW mode or JSON mode the content of the page.
Thanks

Great idea! How/Where would you like this button to be?

Like this picture.
The buttons appears when the mouse over the screen.

That looks awesome, do you want to go ahead an do the PR? Or can you wait till I get some time and do it. :+1:

I have no time for this moment : i work for a node project and i learn also laravel 5 framework.
So you have time to do this but it will be great for me to have this update soon as possible :-)

@visualight It's done. Restart your safari for it to update. I'd also recommend this other plugin also built by me JSONAce.


free command doesn't work when spark3-preview write data into alluxio-2.1.1 path mounted on ceph

Alluxio Version:
2.1.1
Describe the bug
alluxio can not auto free ram, and "alluxio fs free /" didn't work
To Reproduce

mount ceph 14.2.2 by s3 gateway 2.0.2
use spark-3.0.0-preview write about 30G data into step 1's dir
job finished in success
alluxio can not auto free ram, and "alluxio fs free /" only made ram to about 60%
alluxio worker's log as follows:
2020-01-31 21:29:41,821 DEBUG AsyncBlockRemover - 587202560 is being removed. Current queue size is 140.
2020-01-31 21:29:41,821 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-7 was cleaning block 301989889, exception is /mnt/ramdisk/alluxioworker/301989889.
2020-01-31 21:29:41,821 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-5 was cleaning block 469762048, exception is /mnt/ramdisk/alluxioworker/469762048.
2020-01-31 21:29:41,821 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=469762049, location=any dir, any tier, any medium type
2020-01-31 21:29:41,821 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-2 was cleaning block 301989888, exception is /mnt/ramdisk/alluxioworker/301989888.
2020-01-31 21:29:41,821 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=335544321, location=any dir, any tier, any medium type
2020-01-31 21:29:41,821 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=33554433, location=any dir, any tier, any medium type
2020-01-31 21:29:41,827 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-9 was cleaning block 469762049, exception is /mnt/ramdisk/alluxioworker/469762049.
2020-01-31 21:29:41,827 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-3 was cleaning block 335544321, exception is /mnt/ramdisk/alluxioworker/335544321.
2020-01-31 21:29:41,827 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=167772161, location=any dir, any tier, any medium type
2020-01-31 21:29:41,827 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=503316481, location=any dir, any tier, any medium type
2020-01-31 21:29:41,827 DEBUG TieredBlockStore - removeBlock: sessionId=-4, blockId=402653185, location=any dir, any tier, any medium type
2020-01-31 21:29:41,827 WARN  AsyncBlockRemover - IOException occurred while block-removal-service-8 was cleaning block 33554433, exception is /mnt/ramdisk/alluxioworker/33554433.
2020-01-31 21:29:41

Assigned to myself.

Can you try to directly delete the file like rm /mnt/ramdisk/alluxioworker/469762049 on that machine and see if that works? @JySongWithZhangCe

/ping jiacheliu3

fixed via #12643


question regarding SetRequestURIBytes, SetHostBytes and SetScheme etc

if i have already provided full "http://example.com/hello", do i still need to set sethost or setscheme or anything else?
  ctx.Request.SetRequestURIBytes(reqURI)

still need to set below or no need?
  ctx.Request.SetHostBytes(reqHost)
  ctx.Request.URI().SetScheme("http")

with ref to this https://github.com/cloudwego/hertz/issues/446

No setup required.

ok thx!


Toward Denotational Semantics

The aim of this issue is to present the plan to construct
a denotational concurrency semantics based on the formalism of
stable event structures, and to track the progress in achieving this goal.
I'll present the current vision of this plan in a series of steps.
Each major step then will be accompanied by a separate issue
where we'll be able to discuss technical details of a specific problem.

[ ]  Stable Event Structures.
Define stable event structures and their configurations.
Define the binary causality relation on configuration and prove its properties.
This is the most straightforward task since we only need
to mechanize existing simple theory in Coq [1,2].

[ ] Synchronization Algebra.
The parallel product on labeled stable event structures is
defined with respect to synchronization algebra.
This algebra decides which events from the parallel process should synchronize.
We need to define the synchronization algebra and prove its properties.
At this point, we'll need to adjust the theory given in the textbooks for our needs
(more details will be posted in the separate issue).

[ ] Combinators on Stable Event Structures.
Define the following combinators on stable event structures:
empty structure, unit structure, sum (conflict), sequential product, parallel product.
Prove their properties (e.g. associativity, commutativity, etc).
Again, we'll need to adjust the theory here.

[ ] Correspondance between Prime and Stable Event Structures.
Use the standard theory [2] to construct an isomorphism between
configurations of prime and stable event structures.

[ ] Denotation Semantics of a simple concurrent language.
Define simple imperative language with the parallel composition (IMP)
which has the following constructs: atomic instructions (read/writes/updates),
conditional (if), sequential composition (;), parallel composition (||).
Give a denotational semantics in terms of stable event structures.

Bonus Track!

[ ] Refinement Relation and Observational Equivalence.
Define refinement relation on stable event structures.
Define observational refinement on program syntax (via operational semantics).
Prove full abstraction (i.e. equivalence of two refinement relations).
Use the refinement relation to prove soundness of several program transformations [3].

[ ] Fixpoint constructions on Stable Event Structures.
Prove that stabel event structures form cpo (chain-complete partial order).
Use it to define fixpoint operator on stable event structures.
Use the fixpoint to give semantics to loops and recursion in the programming language.
In the process, we will have to also mechanize the theory of cpos
and Kleene fixed-point theorem [4] (or try to find ready to use Coq library).

RFC @volodeyka @dmitromikh @anton-trunov @anlun
[1] Winskel G. Event structures //Advanced Course on Petri Nets. – Springer, Berlin, Heidelberg, 1986. – С. 325-392.
link
[2] Winskel G. Event structure semantics for CCS and related languages //International Colloquium on Automata, Languages, and Programming. – Springer, Berlin, Heidelberg, 1982. – С. 561-576.
link
[3] Dodds M., Batty M., Gotsman A. Compositional verification of compiler optimisations on relaxed memory //European Symposium on Programming. – Springer, Cham, 2018. – С. 1027-1055.
link
[4] Bertot Y., Komendantsky V. Fixed point semantics and partial recursion in Coq //Proceedings of the 10th international ACM SIGPLAN conference on Principles and practice of declarative programming. – 2008. – С. 89-96.
link

I've published an RFC w.r.t. synchronization algebras #59

I've published an RFC w.r.t. synchronization algebras #59

An RFC w.r.t. to stable event structures encoding #60

An RFC w.r.t. to stable event structures encoding #60


fixes blank tiles due to unsupported content

This is a draft of a solution.

What I'd like to do is have a central list of supported type strings ( 'image/jpg' ); passing this list allows for future filtering. Would it be better for supported types be passed or require()d?

Also, this function could be renamed or duplicated to GetAllMatchingChannelClaims, GetAllSupportedChannelClaims, or just GetChannelClaims. I don't think it makes sense to duplicate it just to get at currently unsupported content.

Merging to get a fix in ASAP, will make a clean-up task.


Reducer typings

Hi, i am learning typescript and redux and cannot understand how to deal with new typings of Redux.
Right now i am trying to complete this tutorial
At this moment i am on "Creating Store" phase with this simple code below:
store/configureStore.ts
import {createStore, Reducer} from 'redux';
import rootReducer from '../reducers';

export default function configureStore(initialState) {
    const store = createStore(rootReducer, initialState);

    if (module.hot) {
        module.hot.accept('../reducers', () => {
            const nextRootReducer=require('../reducers');
            store.replaceReducer(nextRootReducer);
        })
    }

    return store;
}

index.tsx
import * as React from 'react';
import {render} from 'react-dom';
import {Provider} from 'react-redux';
import App from './containers/App';
import configureStore from './store/configureStore';

const store = configureStore({});
render(
    <Provider store={store}>
        <App />
    </Provider>,
    document.getElementById('root')
);

reducers/index.ts
import {combineReducers} from "redux/index";

import user from './user';

const rootReducer = combineReducers({
    user
});

export default rootReducer;

//reducers/user.ts
const initialState = {
    name: 'Аноним'
};

export default function user(state = initialState,
                                action = {type: '', payload: null}) {
    return state
}

When i trying to compile it webpack messages me about this error:

ERROR in ./src/store/configureStore.ts
(10,34): error TS2345: Argument of type '{}' is not assignable to parameter of type '<A extends Action>(state: any, action: A) => any'.
  Type '{}' provides no match for the signature '<A extends Action>(state: any, action: A): any'

@aikoven @use-strict @Igorbek Can you offer any advice? Thanks!

In this block:
const nextRootReducer=require('../reducers');
store.replaceReducer(nextRootReducer);

require returns module of type {} as it have no enough typings to infer it properly. Then you try to pass {} which is not compatible to type Reducer<any>. If you know that you require returns an appropriate reducer then you could cast it to the type Reducer<any>.

Thanks for help.


clarify details of transfer learning: which layers were frozen, which were re-initialized

it's not just the last layer that was trained!
only feature layers were frozen
just the last layer was re-initialized

done in revision


Log state/inferences during play

Useful for debugging purposes.
To enhance this, some BattleState objects would have to return additional data describing inferences in better detail, e.g. PossibilityClass#narrow()/#remove() returning a list of removed keys.

Obsoleted by #321.