Error in  load_from_checkpoint when LightningModule init contains 'hparams'

❓ Questions and Help
What is your question?
Just pulled master today, and load_from_checkpoint no longer works. I am wondering if this is a backwards compatibility issue, or I need to do something differently to load now? I've provided the HEAD commits of my local repo I'm installing from in the environment section below.
Error in loading
Here's the error I'm getting now without changing any code:
File "/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/core/saving.py", line 153, in load_from_checkpoint
    model = cls._load_model_state(checkpoint, strict=strict, **kwargs)
  File "/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/core/saving.py", line 193, in _load_model_state
    model = cls(**_cls_kwargs)
TypeError: __init__() missing 1 required positional argument: 'hparams'

The lightning module has always had a member variable hparams, so everything should have been saved (and indeed worked before this pull) in accordance with the docs on stable: https://pytorch-lightning.readthedocs.io/en/stable/hyperparameters.html#lightningmodule-hyperparameters
What's your environment?
❯ git rev-parse --short HEAD
1d3c7dc8    # DOES NOT WORK

❯ git rev-parse --short HEAD@{1}
90929fa4    # WORKS OKAY

can you put your LightningModule init code and load_from_checkpoint line here?

Hi Rohit,
Sure. Here's the init code:
class ProcessSystemBase(pl.LightningModule):

    def __init__(self, hparams: Namespace, builder: RecurrentBuilder) -> None:
        """ Initialize the Process module.

        Args:
            hparams             -- The hyperparameters for the experiment
            builder                -- The factory to build backbone components

        """
        super().__init__()

        ##----------------------------------------
        # Hack to combat lightning loading hparams as dicts
        # Refer issue #924
        if isinstance(hparams, dict):
            hparams = Namespace(**hparams)
        ##----------------------------------------
        self.hparams = hparams
        self.components = builder.init_components(hparams)

I have a custom factory object called builder that handles custom logic for initializing components but I am sure the details of it aren't relevant here. The conversion to Namespace is needed following #924 where the hparams type is not saved in the checkpoint.
And here is how I load it:
builder = RecurrentBuilder()
process = ProcessSystemBase.load_from_checkpoint(
        str(ckpt_path), builder=builder
)
process.freeze()

ckpt_path is a pathlib.Path object containing the filepath to the checkpoint object. This worked before my pull as mentioned with the rev-parse outputs in the original post.
Thanks for taking a look @rohitgr7

ah sorry, this issue totally slipped my mind. can you reproduce this issue with bug_report_model?Would be easier to check the issue :)

I was blocked on this so dove in and found the issue. It's essentially in these lines (also on master so I assume this applies to 1.0 as well) :
https://github.com/PyTorchLightning/pytorch-lightning/blob/01402e35948620e4ff77922ec30becf1f9b2d2fb/pytorch_lightning/core/saving.py#L189-L191
So you see here that when _cls_kwargs is updated, it doesn't maintain the rest of the keys as hparams. In my case above, the cls_init_args_name contains ['hparams', 'builder'] as expected. But after filtering, the kwargs only contains the key ['builder']. I fixed this locally by explicitly checking if cls_init_args_name contains hparams and passing in the rest of the keys as the hparams dict. Of course, ideally this needs to be converted to Namespace if that's how it was saved ( see #924 , which is still a problem anyways, my checkpoint didn't have the hparams type saved). So here is my inelegant fix (this is ugly, meant to debug, if you want me to issue a PR I will come up with a better solution):
    if not cls_spec.varkw:
        # filter kwargs according to class init unless it allows any argument via kwargs
        _cls_kwargs_filtered = {k: v for k, v in _cls_kwargs.items() if k in cls_init_args_name}
        if "hparams" in cls_init_args_name:
            _cls_hparam_kwargs = {k: v for k, v in _cls_kwargs.items() if k not in cls_init_args_name}
            _cls_kwargs_filtered["hparams"] = _cls_hparam_kwargs

    model = cls(**_cls_kwargs_filtered)

So here is my question @rohitgr7 , the docs suggest that assigning anything to hparams in init implies that it gets automatically saved. Am I mistaken in assuming that implies that it will also be loaded correctly and assigned to hparams? If so, I think the problem is that the loading code doesn't seem to handle this at all. The bug_report_model doesn't have an hparams in the init args, so you won't be able to see this problem there as is.

@chiragraman I have created a simple version of your workflow, can you please check this is what you are trying to do? If yes, I am not getting any error with this so maybe some backward compatibility issue is there in the new update.
https://gist.github.com/rohitgr7/f8637d73ed0840e771af8757ba8f34c5

@rohitgr7 thanks for this! I ran it and it matches the behavior you're seeing, so I went back and debugged a bit more. The only difference I found between my actual code and the simple version you created is that the value for hparams_name in my checkpoint is missing, while in the simple version it correctly returns the value hparams:
# My code:
(Pdb) checkpoint["hparams_name"]

# Simple setup
(Pdb) checkpoint["hparams_name"]
'hparams''

But this definitely makes things a lot clearer. Because there is no hparams value in the dictionary, the problem arises from here:
https://github.com/PyTorchLightning/pytorch-lightning/blob/01402e35948620e4ff77922ec30becf1f9b2d2fb/pytorch_lightning/core/saving.py#L181-L183
This is where the reassignment I'm doing in my debug fix above is supposed to happen. But that if check args_name fails  and the whole thing is skipped.  I have no idea currently why the key exists but the value is empty in the checkpoint. If this further isolation helps and triggers an epiphany do let me know :) Thanks for the help Rohit

@rohitgr7 Yo Rohit, I found the problem and can reproduce it with the bug report model. The issue is that saving hyperparameters to checkpoint fails for subclassed lightning modules.
The hparams_name is set by looking for ".hparams" in the class spec. This will obviously fail if your LightningModule is subclassed from a parent LightningModule that handles assigning of the hparams member variable in __init__. Consequently when the child is loaded from checkpoint the key for cls.CHECKPOINT_HYPER_PARAMS_NAME but no value.
I don't know if subclassing your custom LightningModule is not supported by design? I think sometimes the right encapsulation would require this, such as in my case, and most probably if someone is writing code with any intention for it to be used by other developers in a real world production setting. If it is supported, I think this part would have to be updated to find whether the .hparams has been assigned in a more elegant way:
https://github.com/PyTorchLightning/pytorch-lightning/blob/5c1eff351b035db5881d0cff81b1d9c9e150e2d0/pytorch_lightning/core/lightning.py#L1631-L1636
(maybe @williamFalcon can weigh in on the design goals question?)

So for a quick recap, here is the error in loading:
File "/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/core/saving.py", line 197, in _load_model_state
    model = cls(**_cls_kwargs_filtered)
TypeError: __init__() missing 1 required positional argument: 'hparams'

And here is the debug model code to reproduce it: https://gist.github.com/chiragraman/ff0e6b0239cb635df30093dd31ae3dce

@chiragraman which version of Lightning are you running your bug report model on?

@rohitgr7 Just pulled master yesterday, I'm updated to 1.0.1

I tried your code on both master and 1.0.1, but not getting any error. Am I missing something here?

Hmm, just ran it again and it crashed again. Here's the HEAD for my local lightning repo:
❯ git rev-parse --short HEAD
f967fbba

Hmm, if you add a breakpoint right before this,
https://github.com/PyTorchLightning/pytorch-lightning/blob/155f4e9aa5d6175f8d3db23d483154b065412cd0/pytorch_lightning/core/saving.py#L194
can you verify if the checkpoint contains any value for "hparams_name"? In the meanwhile let me pull master again and retry.

can you verify if the checkpoint contains any value for "hparams_name"

yes, it does hparams. Can you create a colab maybe? Would be better I guess to see that's hapenning here.

Hmm, just updated master so that my local HEAD is at 155f4e9a. Can confirm it still crashes. If it helps at all, here is the entire console dump. I also can't see how this would work given how .hparams doesn't exist in the lines parsed for ChildLitModel, this:
https://github.com/PyTorchLightning/pytorch-lightning/blob/5c1eff351b035db5881d0cff81b1d9c9e150e2d0/pytorch_lightning/core/lightning.py#L1635
has to fail and the function would return None.
GPU available: True, used: False
TPU available: False, using: 0 TPU cores
/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/utilities/distributed.py:45: UserWarning: GPU available but not used. Set the --gpus flag when calling the script.
  warnings.warn(*args, **kwargs)

  | Name | Type   | Params
--------------------------------
0 | l1   | Linear | 7 K
/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/utilities/distributed.py:45: UserWarning: The dataloader, val dataloader 0, does not have many workers which may be a bottleneck. Consider increasing the value of the `num_workers` argument` (try 8 which is the number of cpus on this machine) in the `DataLoader` init to improve performance.
  warnings.warn(*args, **kwargs)
/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/utilities/distributed.py:45: UserWarning: The dataloader, train dataloader, does not have many workers which may be a bottleneck. Consider increasing the value of the `num_workers` argument` (try 8 which is the number of cpus on this machine) in the `DataLoader` init to improve performance.
  warnings.warn(*args, **kwargs)
Epoch 0:  92%|████████████████████████▊  | 11/12 [00:00<00:00, 108.34it/s, loss=2.013, v_num=5]Epoch 0: val_loss reached 1.82315 (best 1.82315), saving model to /home/chirag/Projects/test/lightning_logs/epoch=0-val_loss=1.82.ckpt as top 3
Epoch 1:  83%|████████████████Epoch 1: val_loss reached 1.76090 (best 1.76090), saving model to /home/chirag/Projects/test/lightning_logs/epoch=1-val_loss=1.76.ckpt as top 3
Epoch 1: 100%|███████████████████████████| 12/12 [00:00<00:00, 131.93it/s, loss=1.860, v_num=5]
Traceback (most recent call last):
  File "lit.py", line 109, in <module>
    new_model = ChildLitModel.load_from_checkpoint(ckpt_path, another_param='something2')
  File "/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/core/saving.py", line 154, in load_from_checkpoint
    model = cls._load_model_state(checkpoint, strict=strict, **kwargs)
  File "/home/chirag/miniconda3/envs/ml/lib/python3.8/site-packages/pytorch_lightning/core/saving.py", line 194, in _load_model_state
    model = cls(**_cls_kwargs)
TypeError: __init__() missing 1 required positional argument: 'hparams'

@rohitgr7 I created a colab here: https://colab.research.google.com/drive/1HvWVVTK8j2Nj52qU4Q4YCyzOm0_aLQF3?usp=sharing#scrollTo=AAtq1hwSmjKe
But it works there, and I really don't know how for now :/

it's because of notebook env I think. I tried it as a python script and it's throwing the error but in a notebook it isn't. What's happening here haha  :sweat_smile:?

Phew, I was starting to lose my mind. Yeah, it works in that colab I shared. Haha I think the script behaviour is at least explainable looking at the code, I would expect the hparams value to be missing.

yep, true.

@rohitgr7 have you already started working on this? So adding a hparams argument to the bug report model will reproduce the issue?

@awaelchli I haven't started working on it since I don't know the best way to solve it.
@chiragraman has linked a code sample similar to bug_report_model to reproduce this issue along with some details on why this is happening https://github.com/PyTorchLightning/pytorch-lightning/issues/3998#issuecomment-710517824.
PS: don't run this code in a notebook since it won't throw any error.

I want to point out the following in your reproducible code that you posted:
class LitModel(pl.LightningModule):
    def __init__(self, hparams, another_param):
        super().__init__()

        # if isinstance(hparams, dict):
        #     hparams = Namespace(**hparams)

        # this is recommended Lightning way
        # it saves everything passed into __init__
        # and allows you to access it as self.myparam1, self.myparam2
        self.save_hyperparameters()

        # this is optional
        # only needed if you want to access hyperparameters via
        # self.hparams.myparam
        self.hparams = hparams

        # this is optional, if you call self.save_hyperparameters(), it's done for you
        self.another_param = another_param

        self.l1 = torch.nn.Linear(28 * 28, 10)

with save_hyperparameters(), this works smoothly,

@awaelchli you mean this in the context of the #924 /  #4333 discussion we're having right? The two lines you've commented out were just there for working around the Namespace issue. You can still reproduce the issue with the inheritance chain though?

Yes, and I commented out to show that with save_hyperparameters() function you don't need the workaround.

You can still reproduce the issue with the inheritance chain though?

No. Your child module has the same args as the parent one, so they are passed correctly. Only if you have different args in the child module, you need to call self.save_hyperparameters() there too.

Aaah okay makes sense. Using your recommended method avoids the offending code altogether here:
https://github.com/PyTorchLightning/pytorch-lightning/blob/5c1eff351b035db5881d0cff81b1d9c9e150e2d0/pytorch_lightning/core/lightning.py#L1631-L1636
So it makes sense that it works. However, the code above doesn't (and wouldn't) work for the setup in the reproducible example. If the recommended way is to explicitly save hyperparameters, then assigning to .hparams should altogether be dropped in my opinion, and this code above be removed to avoid unnecessary code bloat.

So in the spirit of offering the start of a solution instead of simply pointing out the problem, here is my opinion. The user code should be decoupled from the library code. If the way forward is to use save_hyperparameters() I think it would be more elegant to use it internally as well in the library based on a condition even if the user hasn't. We want to protect the usability at the user level that resulting from simply being able to assign to hparams.
The problem lies in what this condition should be. I'd contend that it's a little hacky to parse the class spec to see if the string ".hparams=" exists in there, which is doomed to fail the minute you use inheritance. If the same check needs to be upheld for some reason, one would have to walk up the inheritance chain to check it, which seems even more inelegant and might cause issues with complex subclassing patterns. So why is good old hasattr() not a good option here? (See the discussion here: https://hynek.me/articles/hasattr/

@awaelchli I was going to take a crack at points 1 and 2 above, I think the PR only updates the docs right? Wanted to confirm before proceeding, since incorporating the first two points would also solve #3998.

So I can only load a checkpoint from a model if i name the parameters after the checkpoint path, for example:
best_model_fold = model.load_from_checkpoint(CKPT_PATH, cfg= cfg, fold= fold)
and not
`


adds caveat section; mentions issue #10

adds caveat section which highlights #10 cypress-io/cypress/3090

Thanks @meinaart


Frontend - add more buttons

eg. On the side under the zoom buttons, some buttons labelled with "UserName"

Tapping on a user on the map will center the map on that user and automatically pan the map to follow them. I can add a menu that shows the current users on the map as well and add the same functionality for clicking on a user in that menu.

Tapping on a user on the map will center the map on that user

Yes, but if you don't see the users you need to "zoom out" and then "touch"

Frontend - add share location of user button: eg. share a geo: link that OsmAnd~ or Google Maps can open and we can "navigate to..." that user or whatever.
(from https://github.com/bilde2910/Hauk/issues/103#issuecomment-564972507)

There is now a user button in the top left corner that has this functionality. Let me know if you miss anything from it.

Look ok, thanks.

One note, don't list users that you can't yet see, eg. user was added to the group but they didn't submit a location yet

Done


DynamoDB provider support for WebIdentityRoleProvider

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
I've taken the public traefik Docker image (v1.7.34) and I'm trying to run it up on AWS EKS, configured to use the DynamoDB provider, using an IAM Role associated with a Kubernetes Service Account but I'm getting this error on start up:
Provider error: AccessDeniedException: User: arn:aws:sts::<redacted>:assumed-role/joe20210824175302473300000009/i-abcdefg123 is not authorized to perform: dynamodb:Scan on resource: arn:aws:dynamodb:us-west-2:<redacted>:table/traefik because no identity-based policy allows the dynamodb:Scan action
    status code: 400, request id: <redacted> time: 1.197095801s

The IAM role associated with the Kubernetes service account has an inline policy that allows "dynamodb:Scan":
  - effect: Allow
    actions:
      - "dynamodb:GetItem"
      - "dynamodb:Query"
      - "dynamodb:Scan"
    resources:
      - "arn:aws:dynamodb:us-west-2:<redacted>:table/traefik"

The Helm chart for my traefik deploy has the following Service Account definition:
---
apiVersion: v1
kind: ServiceAccount
metadata:
  annotations:
    eks.amazonaws.com/role-arn: "arn:aws:iam::<redacted>:role/traefik"

The AWS docs stipulate a minimum SDK version (v1.23.13 for Go, which is what traefik is written in), if we want to assume an IAM role via an OIDC web identity token file. It looks like traefik v1.7.34 is using Go v1.39.0, so that should be sufficient.
However, I think that the way the credential chain is constructed here means that AWS_WEB_IDENTITY_TOKEN_FILE is ignored.
This could be fixed by adding WebIdentityRoleProvider to the credentials chain, similar to what Hashicorp have done for Vault here: https://github.com/hashicorp/vault/pull/7738/files

Hello @pompeyjoe , we're not supporting the v1.x branch anymore as stated here which means we won't add any new features or fixes so I'm closing this proposal. Feel free to open a new one for v2 if the problem is still present, given the provider mechanism is very different.


Switch the Wichita options in the MSA dropdown

@marcesher Remember that time I said I fixed something but I actually didn't fix it at all? That was today.
Turns out the labels in the dropdown just needed to be switched.

Pressing the merge button on these PRs is the YOLOest thing I will do all day

I'm so sorry @marcesher


Issue serializing TagList to YAML

I'm in the process of upgrading an application to Rails 7.1.5, and am running

acts-as-taggable-on (12.0.0)
audited (5.8.0)
psych (5.2.0)

The audited gem is used to save a change history for models, which we're currently doing by serializing the data into YAML.
Given a model named Foo, that has a default tag list and is audited:
class Foo < ApplicationRecord
  acts_as_taggable_on :tags
  audited
end

If I try to save a change to the model that includes the tags, I get an exception:
Psych::DisallowedClass:
       Tried to dump unspecified class: Class

I did some digging, and I found that the issue is that Psych is starting to also validate the YAML on-dump and not just on-load.  As part of this, it sees that ActsAsTaggableOn::TagList is a subclass of Array and it tries to approve all the items in the array, as well as all the internal variables on the object.  When it does this, it finds :@parser is a reference to a class, and then throws the above exception.
I'm assuming that allowing Class would just allow any object, which doesn't seem desirable, so I'm looking at trying to get the tag list to either bypass this or somehow exclude :@parser when it runs through the dump process.  I noticed that earlier in the process, it checks if the object has an encode_with method, so I tried adding that to the class in an initializer (for now to get things going):
# config/initializers/acts_as_taggable_on.rb
module ActsAsTaggableOn
  class TagList
    def encode_with(coder)
      coder.represent_seq(nil, self)
    end
  end
end

This seems to work.  However, I'm a little fuzzy on serialization and the encode_with functionality, so I wanted to run it by you guys to see if this seems like an appropriate change to make to the class.  If it is, I can make a PR with the change.  If it isn't, I wanted to at least notify you about the issue.

For a quick workaround, I ended up having audited ignore tag_list
  acts_as_taggable_on :tags
  audited except: :tag_list


added zap property to android-studio

Added the zap property for android studio.
Got the info from: http://stackoverflow.com/questions/17625622/how-to-completely-uninstall-android-studio

Thanks for the zap stanza, could you remove the trailing whitespace on line 17, please?


Parallel RBF and Built-in CG solver

What has been done:
Make RBF sampler work in both serial and parallel mode.

Added functions to facilitate ghost cell collection and ghost cell matrix construction.

Removes the Eigen library dependency.

Established MUI built-in linear algebra namespace, i.e. mui::linalg, with sparse matrix system and conjugate gradient solver.
The conjugate gradient solver is based on Algorithm 1 of the following paper:
Igor Baratta, Chris Richardson, and Garth Wells. "Performance analysis of matrix-free conjugate gradient kernels using SYCL." International Workshop on OpenCL. 2022.
In addition to the implementation of the conjugate gradient solver based on the paper above, three preconditioners (Incomplete LU preconditioner, Incomplete Cholesky Preconditioner, Symmetric Successive Over-Relaxation Preconditioner) have also been implemented in the mui::linalg namespace. The conjugate gradient solver can be worked in both w/o preconditioner mode and with one of the preconditioner modes.
Add unit test code (a separate folder named “test”) to verify arithmetic operations of the sparse matrix, verify matrix file I/O and verify different combinations of preconditioner and CG by comparing calculated results with reference results.

Verifications:

The RBF sampler is verified by MUI demo 7-pseudo-diffusion. Both conservative mode and consistent mode have been tested.
The arithmetic operations of the mui::linalg sparse matrix system have been verified by the matrix.cpp code in the “test” folder. More specifically: matrix addition, matrix subtraction, matrix multiplication, matrix transpose and matrix I/O (includes print matrix to the console and matrix file I/O in CSV format).
The mui::linalg Conjugate Gradient solver has been verified with reference results by the conjugate_gradient.cpp code in the “test” folder. The following modes have been verified: TEST_0. CG Solver w/o Preconditioner & w/o Initial matrix Guess; TEST_1. CG solver with ILU Preconditioner & w/o Initial matrix Guess; TEST_2: CG solver with SSOR Preconditioner & w/o Initial matrix Guess; TEST_3: CG solver with IC Preconditioner & with Initial matrix Guess.

Future works:

Generalise ghost cell functions, so that to enable other samplers/algorithms to access and use it.
Since the RBF sampler is no longer special compared with other samplers, some pieces of code like below can be removed.

If define RBF:
    Do this;
else:
    Do that;

Sure, I'll withdraw this pull request.


[BUG] get all variants for single product by useCategorySearch()

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
the only information i got is from the current variant
Expected Behavior
i want all variants within a single product on the listing page (i have to show all variants by hovering the current product)
Steps To Reproduce

fetching data in FrontendNavigationPage.vue

const categoryResponse = await search(foreignKey.value, {
  withCmsAssociations: true,
  query: {
    ...route.query,
  },
});

iterating trough elements in CmsElementProductListing.vue
trying to access all variants for a single product ProductCard.vue
within options there is just the current one (array with one item only), children is null, parent is null ...

tried to get it by associations but still one entry in options and parent / children is null.

What am I doing wrong?
Environment
No response
Anything else?
No response

Due to performance reasons, showing the available options on product hover is a great case for async loading - and would be enough if the API responds really quick. Then I would try to use useProductSearch with combination of useProductConfigurator / useProduct to relatively easy extract the possible options.
The alternative option is to use the aggregations and take the possible options with all the listing elements (products) in just one request. @BrocksiNet had shown an example how to approach this: https://www.brocksi.net/blog/variants-selection-multi-page-or-single-page/#listing-with-all-options-for-products.
that might br the case for useCategorySearch which also allows to use a search criteria (including aggregations).

We also have an internal Issue with that NEXT-36467.


Spades: Better Nil defense by player to right of Nil bidder

Can you guys fix the spades bot that sits to the right of a NIL bidder? All they do is throw their highest card away to try to set NIL. This is with the game in hand as long as they make their bid. Fortunately I expect this and know to take as many extra books as I can knowing they’re not going to get theirs. They need to take into account the team bid especially when it gives us the win.

Please, please make note understand that while priority should be protecting nil that winning there books is also important


Features/disabled inputs

Coverage increased (+4.89%) to 100.0% when pulling 3672e9fa03a98bc08ec49e0e8396abba8a910b65 on features/disabled-inputs into 65acd060fe0f944102edccad58c2d636a923d3bb on master.


DualitySoftware.DSClock version <IP_ADDRESS>

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

The installer presents this error. Is there an installer switch that would skip this file, ignore this error, or otherwise allow the install to proceed?

Hi @afermar,
The installer presents this error. Is there an installer switch that would create this log in another location, ignore this error, or otherwise allow the install to proceed?

I tried with /LOG switch and the error doesn't appear.

/azp run

@wingetbot waivers Add Validation-Installation-Error


How to Linked Rankup to Pure perms

Can you tell me how to link pureperms and rankup? I have use 0.3.1 but dosnt work. I do /rankup than it still show previus rank

@IsraAtGamings here is a sample showing how to set a players rank and add perms. Also how to warp/tp a player.
---
groups:
  A:
    entrance:
    - s •§a{name} §bRanked Up To §eA
    - setuperm {name} essentials.warps.A
    - warp A {name}
    - setugroup {name} MVP
    exit: []
    perms: []
    members: []

I hope your problem is solved now. If so please do kindly close the issue.


When will the dataset be released?

When will the dataset be released? Thanks!

Please see issue #9 for temporary file server.

Hello. Currently, the "Image.zip" and "Material.zip" files are not available on the server's website. Could you please share them? My email is<EMAIL_ADDRESS>Thank you so much!

Hi there, please check here for the new server: http://mclab.ucsd.edu/OpenRooms_public/

Hi there, please check here for the new server: http://mclab.ucsd.edu/OpenRooms_public/

Hi, it is not available to open this url now.

Hi there, please check here for the new server: http://mclab.ucsd.edu/OpenRooms_public/

Hi, it is not available to open this url now.

Please try again.


[Bug] [workflow parameter] The workflow which pass parameter can't start

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
I follow the tutorial and try to pass parameter through the same workflow. and i found after configure it, the workflow can't be started. the workflow even not shown in "workflow instance" tab. but another workflow not pass parameter through workflow can start normally.
env:
jdk1.8
dolphinscheduler 3.2.x
postgresql
zookeeper 3.7.x
os: centos 7
What you expected to happen
the workflow which pass parameter through can start successfully.
How to reproduce
follow the tutorial with the shell part:
https://dolphinscheduler.apache.org/zh-cn/docs/3.2.1/guide/parameter/context
Anything else
No response
Version
3.2.x
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is there any error log in the master server? You can find the error message in the error command.

Is there any error log in the master server? You can find the error message in the error command.

no, there is no workflow instance run log which pass parameter between nodes.
for example:

the red workflow pass parameter through node but blue workflow not pass parameter, and only blue workflow can start and have workflow instance log when red workflow can't start.

Or can i find detail log in disk? I deploy the service with cluster in vitural machines.

Is there any error log in the master server? You can find the error message in the error command.

no, there is no workflow instance run log which pass parameter between nodes. for example:  the red workflow pass parameter through node but blue workflow not pass parameter, and only blue workflow can start and have workflow instance log when red workflow can't start.
Or can i find detail log in disk? I deploy the service with cluster in vitural machines.

This may caused by parse parameter error, you can find the log detail on master.log

Is there any error log in the master server? You can find the error message in the error command.

no, there is no workflow instance run log which pass parameter between nodes. for example:  the red workflow pass parameter through node but blue workflow not pass parameter, and only blue workflow can start and have workflow instance log when red workflow can't start.
Or can i find detail log in disk? I deploy the service with cluster in vitural machines.

This may caused by parse parameter error, you can find the log detail on master.log

found error log in master node as below:
Uploading error_log.txt…
looks like java problem

@ArkURL Please provide your task config, I cannot reproduce this problem, it's caused by the parameter parse error, this should be a bug.

Duplicated with https://github.com/apache/dolphinscheduler/issues/15676


Auto focus search box when typing

It would be cool if the search box would automatically focus when you start typing anywhere on the page.
Preferably this should be an option in the search bar settings.
Thanks in advance!

Thank you for the suggestion! I would love to allow this but right now, as far as I'm aware, the browser's do not allow the new tab page to take focus away from the address bar on load so this is unable to be done. Thanks for your support!


[HUDI-4908] Fixing flaky test by drastically reducing concurrency level (to avoid CI flakiness)

Change Logs
Fixing flaky test by drastically reducing concurrency level (to avoid CI flakiness)
Impact
**Risk level: None
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@yihua in one of my PRs it failed 2 times out of 3.
CI is stressing compute, so high concurrency in tests is prone to have tests stalled, and this particular one was failing b/c threads were held back for longer than the test was actually running eventually failing after test resources had already been reclaimed


Bug and Build with 1.6.13 and 1.6.10

[X ] I confirm that this is an issue rather than a question.

Bug report
Hi, I have my website with you theme, I use: npx @vuepress-reco/theme-cli init and instal 1x version in the past.
No problem, but today in another computer I install the web to check, and I have a problem with npm run build I think any package is updated and break anything. I install a clean vuepress roco web and I hace the same proble,
What is actually happening?
No problem when npm run dev, impossible to have a build version
✔ Client
Compiled successfully in 13.19s
✔ Server
Compiled successfully in 9.39s
wait Rendering static HTML...
[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

error Error rendering /404.html: false
undefined
[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

error Error rendering /tag/: false
undefined
error Error rendering /timeline/: false
undefined
error Error rendering /tag/tag1/: false
undefined
error Error rendering /tag/tag2/: false
undefined
error Error rendering /tag/tag3/: false
undefined
error Error rendering /tag/tag4/: false
undefined
error Error rendering /categories/category1/: false
undefined
error Error rendering /categories/category2/: false
undefined
[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

[Vue warn]: Error in setup: "Error: must be called in setup"
found in
--->

error Error rendering /categories/: false
undefined
error Error rendering /: false
undefined
error Error rendering /blogs/category1/2018/121501.html: false
undefined
error Error rendering /blogs/category1/2019/092101.html: false
undefined
error Error rendering /blogs/category2/2016/121501.html: false
undefined
error Error rendering /blogs/other/guide.html: false
undefined
error Error rendering /docs/theme-reco/: false
undefined
error Error rendering /docs/theme-reco/api.html: false
undefined
error Error rendering /blogs/category2/2017/092101.html: false
undefined
error Error rendering /docs/theme-reco/plugin.html: false
undefined
error Error rendering /docs/theme-reco/theme.html: false
undefined
Error: must be called in setup
at useInstance (node_modules/vuepress-theme-reco/helpers/composable.js:5:17)
at setup (node_modules/vuepress-theme-reco/layouts/404.vue:14:32)
at mergedSetupFn (node_modules/@vue/composition-api/dist/vue-composition-api.esm.js:1672:88)
at invokeWithErrorHandling (/home/joseluisgs/Proyectos/web/node_modules/vue/dist/vue.runtime.common.dev.js:3895:30)
at initSetup (/home/joseluisgs/Proyectos/web/node_modules/vue/dist/vue.runtime.common.dev.js:2313:29)
at initState (/home/joseluisgs/Proyectos/web/node_modules/vue/dist/vue.runtime.common.dev.js:5117:5)
at VueComponent.Vue._init (/home/joseluisgs/Proyectos/web/node_modules/vue/dist/vue.runtime.common.dev.js:5431:9)
at new VueComponent (/home/joseluisgs/Proyectos/web/node_modules/vue/dist/vue.runtime.common.dev.js:5566:18)
at createComponentInstanceForVnode (/home/joseluisgs/Proyectos/web/node_modules/vue-server-renderer/build.dev.js:1981:12)
at renderComponentInner (/home/joseluisgs/Proyectos/web/node_modules/vue-server-renderer/build.dev.js:7184:45)
at renderComponent (/home/joseluisgs/Proyectos/web/node_modules/vue-server-renderer/build.dev.js:7161:9)
at resolve (/home/joseluisgs/Proyectos/web/node_modules/vue-server-renderer/build.dev.js:7210:17)
Other relevant information
I try with a new version of vue press and older version. I have the same problem

Your OS:  Linux and mac
Node.js version: 16.5.1
VuePress version:  1.8.2
VuePress-theme-reco version: 1.6.10 and 1.6.13
Which package manager did you use for the install(npm/cnpm/yarn)? npm

这是来自QQ邮箱的假期自动回复邮件。

您好，我最近正在休假中，无法亲自回复您的邮件。我将在假期结束后，尽快给您回复。

Please en english?

I solve using my old package-lock
maybe it can help you
https://github.com/joseluisgs/joseluisgs/blob/master/web.back/packages/package-lock.json
https://github.com/joseluisgs/joseluisgs/blob/master/web.back/packages/package.json
Probably a dependece breaks the build mode
any idea?

Hello friends, please try to upgrade to v1.6.14

Hi, I will try it this monday :)

Thank you very much! I' sorry,
I haven't tried it yet.
But I guess everything will be great!
Thanks again

i have the same question, after upgrade to v1.6.14 all resolved

已收到，谢谢


[Feature/extensions] Restructured testExtensionsInitialize

Signed-off-by: Ryan Bogan<EMAIL_ADDRESS>Description
Fixes the gradle check failure for testExtensionsInitialize
Check List

[ X] New functionality includes testing.

[ X] All tests pass

[X ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #4029 (4b020e7) into feature/extensions (be1a605) will decrease coverage by 0.01%.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##             feature/extensions    #4029      +/-   ##
========================================================
- Coverage                 70.64%   70.63%   -0.02%
  Complexity                56884    56884
========================================================
  Files                      4584     4584
  Lines                    273654   273654
  Branches                  40105    40105
========================================================
- Hits                     193318   193284      -34
- Misses                    64085    64125      +40
+ Partials                  16251    16245       -6

Impacted Files
Coverage Δ

...n/indices/forcemerge/ForceMergeRequestBuilder.java
0.00% <0.00%> (-75.00%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
56.25% <0.00%> (-35.00%)
:arrow_down:

...arch/search/aggregations/pipeline/LinearModel.java
23.07% <0.00%> (-34.62%)
:arrow_down:

...ensearch/client/indices/DetailAnalyzeResponse.java
20.54% <0.00%> (-34.25%)
:arrow_down:

.../indices/forcemerge/TransportForceMergeAction.java
25.00% <0.00%> (-33.34%)
:arrow_down:

...ter/coordination/CoordinationStateTestCluster.java
73.78% <0.00%> (-23.18%)
:arrow_down:

...iscovery/ClusterManagerNotDiscoveredException.java
22.22% <0.00%> (-22.23%)
:arrow_down:

...opensearch/persistent/AllocatedPersistentTask.java
44.23% <0.00%> (-21.16%)
:arrow_down:

...ain/java/org/opensearch/geometry/MultiPolygon.java
80.00% <0.00%> (-20.00%)
:arrow_down:

...n/java/org/opensearch/test/rest/yaml/Features.java
60.00% <0.00%> (-20.00%)
:arrow_down:

... and 437 more

Help us with your feedback. Take ten seconds to tell us how you rate us.


Accordian does not collapse

https://pynecone.io/docs/library/disclosure/accordion

Great to hear from you Mike! Thanks for taking a look at our framework and for pointing this out. I forgot to add the allow_toggle method to be True here, which will allow the example to collapse. I'll make this default to True in our initial release to you don't have to manually add this every time in the future.

Resolved with new pynecone release.


PreventDuplicateClassMethodRule unrelated methods reported

PreventDuplicateClassMethodRule reports a wrong error in the following case with factory methods:
class a {
   public static function factory(): self {
      return new self();
   }
}

class b {
   public static function factory(): self {
      return new self();
   }
}

even if the code lines are identically in this case, since semantically self is a totally different thing this type of method should be ignored.
I guess there could be a similar problem with $this

There should a limit of lines that this rules ignores.
It does not work with types, that would be very performance expensive.
Any improvement is welcomed :)


Singularity PVS makes the distortion effect look bad

Description

Moving in/out of singulo PVS range makes the distortion effect suddenly pop in/out and it looks pretty bad as a result.
Presumably it just needs an increased PVS range?
Screenshots

https://streamable.com/yy6xty
Additional context

The video downplays how jarring it is.


[wait for #1366] [layer] Enable addition and multiout layer for V2 design

Update addition and multioutput layer for V2 design.
Add corresponding unittests.
Enable graph unittests which creates resnet like model for testing.
Enable correspoding resnet like add in models unittest.
Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1368. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

:octocat: cibot: @kparichay, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1368-202107072025570.22451901435852-8e26f19cfc2f09bede93988dce277e7877e89a76/.

I think it is great for now. But we need to start discussing about #1373 .


Add support for Fedora 41

This issue is created to track progress for the work to enable Fedora 41 support for .NET 8.0 and .NET 9.0
Product and Test updates

[ ] [All teams] Do a compatibility check for the product with the new OS
[x] ~[CoreCLR/CoreFX] Runtime / RID Support update~ Runtime does not need RID changes starting with 8.0.
[ ] [Infra] Ensure that the correct Docker images used by teams are built in the dotnet-buildtools-prereqs (if applicable)
[ ] [Infra] Replace docker tags used in builds across repos and branches
[ ] [CLI team] Update CI legs to latest updated Dockerfile
[ ] [All teams] Update test infrastructure to support the new version
[x] ~[Acquisition] Add support within nightly repos~
[x] ~[Acquisition] Deploy to master (GA) during next patch release cycle~

Release team:

[x]  Notify .NET Core teams regarding new OS release
[ ] Update the supported OS content on GitHub and marketing sites.

Document updates:

[ ] Add OS prerequisites to the dependencies document
[ ] Update distro install page

for .NET 8.0 and .NET 9.0

Given that we're excluding 6.0, runtime does not need to make any manual RID additions starting with .NET 8.0.

for .NET 8.0 and .NET 9.0

Given that we're excluding 6.0, runtime does not need to make any manual RID additions starting with .NET 8.0.

@carlossanlop Can we remove this item moving forward from add support template for all OS?

Yes, remove it, please and thank you! cc @rbhanda
Note: Only keep that item if for some reason we decide to add one more RID to 6.0 (that version still needs RIDs). Which probably won't happen as 6.0 is close to expiring.

@rbhanda my CTI team is including Fedora 41 in their manual testing matrix. This item is completed from my side


Submitting create project form with Enter key without mandatory fields defined

As expected, modal for creating projects disables the "Save" button if all mandatory fields are not defined.
However, after focusing on one of the input fields in the modal and just hitting the enter key, user is able to create a project without mandatory fields.
This results in a project without the mandatory project-id defined. This project can't be selected or deleted afterwards.
Here's a screenshot of the resulting project in the projects list:

@mgubaidullin I'll take this one...


Repositories without .git suffix are ignored

https:// URLs without .git are a ParserError.  GitHub and GitLab both support URLs without the .git (but GitLab emits a warning).

I haven't found a case where github or gitlab will actually provide the user with a copy pastable URL that doesn't already contain .git in the URL.  How would one encounter a URL from these two sources w/o .git?

Does this library actually only support parsing URLs provided by GitHub and GitLab? URLs without the .git suffix are valid git URLs.

PRs always welcome.


Make bits2fasm

Right now we use segprint2fasm to create fasm from a bitstream.  segprint was intended as a debugging utility and not to be used this way. Instead we should just make a dedicated bits2fasm utility and eliminate the intermediate step. I've recently cleaned up segprint, so this should be a lot easier

I've done some work on this in https://github.com/litghost/prjxray/tree/fasm_refactor
https://github.com/litghost/prjxray/blob/fasm_refactor/minitests/roi_harness/bit2fasm.sh

Talked to @litghost , there will be some conflicts but we'll review them after. sample output from the new tool: https://gist.github.com/mcmasterg/380d99efbb5bb5ac6fa6b5dbde167bcd
I copied over the formatting code, so I think it should be the same, but you never know

fixed


Dev drive feature not available on build 23451

Hi in https://learn.microsoft.com/en-US/windows/dev-drive/ It states that this feature requires Windows 11 dev channel. I am in dev channel build 23451, but I don't see any settings for dev drive. is that available now? if not, when will it be available?

Seems like available later this week in the blog post?

Closed, I think Microsoft publish the new one now


Support for Union-Types (Serde, Schemas, Avro-Values)

Difference between the Value::Union representation this PR and @kphelps 's one: Value::Union(variant_index: usize, variant_value: Box<Value>) instead of Value::Union(union_ref: UnionRef, variant_value: Box<Value>).
This option does not allow for creation of union-type instance from the inner variant-value "implicitly" (i.e. without knowing the variant-index able to carry that exact variant-data).
This should help a lot with Serde because this restriction allows "lossless" translations between Structs<->Avro-Values<->Avro-Binary in both directions.
Example of usage: https://gist.github.com/RGafiyatullin/4fec518e3cd8e3c6fe718ca976a4cdc5
Output: https://gist.github.com/RGafiyatullin/8aaf4c427bec5456edde2e0913dde472

Any movement on this? I have an incoming PR for Schema Compatibility that would gain a bit by having this PR in.

I would also benefit from having this work merged. Is there anything I can do to help?

There is also #60 implementing bits and pieces of the same feature in a slightly different way and they are both pretty old... (definitely because of the fault of us maintainers).
I'll need to budget some time to properly understand how serde handles enums (tonight I only managed to read the serialization half of the data format) and at the same time follow up with the authors to understand who is still interested in contributing.
@RGafiyatullin  are you still interested in contributing? Shall we try to revitalize this PR? I asked the same thing to @jdeschenes on #76 .

I believe that #110 landed the same feature of this PR only in a slightly different form. Closing for now, but please feel free to open a new one based on current master in case there is still something missing that you'd like to see added.


Replace nodejs icon with devfile-stack-icons link

What does this PR do?:
Replaces the broken icon for nodejs stack with the one added in the devfile-stack-icons repo.
Which issue(s) this PR fixes:
Fixes https://github.com/devfile/api/issues/1484
PR acceptance criteria:

[ ] Contributing guide
Have you read the devfile registry contributing guide and followed its instructions?
[ ] Test automation
Does this repository's tests pass with your changes?
[ ] Documentation
Does any documentation need to be updated with your changes?
[ ] Check Tools Provider
Have you tested the changes with existing tools, i.e. Odo, Che, Console? (See devfile registry contributing guide on how to test changes)

How to test changes / Special notes to the reviewer:

@BethGriggs could you please review the PR as you are the owner of the nodejs stack?

Merging as the PR is approved by @lholmquist (member of the team owning the stack)


continuous spinner when ng2-formio compiled on ios when app is offline

when ng2-formio rendering on an app that is compiled to ios platform, if the app goes offline, there is a continuous spinner, making it impossible to interact with the app.
due to 'loader-wrapper' class

This wrapper is overlapping other areas besides the form? I did make a change with the latest core renderer on how this loader works and thought I made it so that it would not overlap other areas on the page. Let me know otherwise since it may be behaving different in your app.

it just appears over the ng2-formio component. but the expected  behaviour is that forms loaded via raw JSON would still be visible etc... this behaviour also seems different from when compiled on android to apk, based on testing

Oh, I see... you are loading the JSON but the spinner never goes away? This does need to be fixed if that is the case.

yes. that's right. it's loading from JSON but spinner is still there...
Aside, when i am able, i will try to help you track down #90 --prod, but it is exceedingly opaque to debug. Aside from than that, i thought i would share my own prioiritized wish list of ng2-formio fixes :)
#94 (bad binding of data of radio/checkbox labels in datagrid)
#107 ( custom classes are not coming through ng2-formio)
#102 (- dual entries on datagrid, which is still broken but currently listed as a closed ticket)
#83 (custom events not emitted)

Should be fixed in RC18.


org.vcsreader:vcsreader:1.1.0 does not exist

When running the get_analysis_projects.groovy script I see the following:
Expected Behavior
Script runs successfully.
Current Behavior
org.codehaus.groovy.control.MultipleCompilationErrorsException: startup failed:
General error during conversion: Error grabbing Grapes -- [unresolved dependency: org.vcsreader#vcsreader;1.1.0: not found]

java.lang.RuntimeException: Error grabbing Grapes -- [unresolved dependency: org.vcsreader#vcsreader;1.1.0: not found]

Looking at mavenCentral there also does not seem to be the mentioned version but just v1.0.0: https://search.maven.org/artifact/org.vcsreader/vcsreader/1.0.0/jar
Steps to Reproduce (for bugs)
groovy scripts/get_analysis_projects.groovy ~/some/dir

Uh oh, good to know! i have patched the library here: https://bintray.com/arturbosch/code-analysis/vcs-reader

try just changing it to 1.0.0, it should still work?

I tried downgrading to v1.0.0 but that didn't really work for me because of issues with imports. But that could've been some other issue on my end.

yep, 1.0.0 misses VcsError, idk :X.
Please try:
@GrabResolver(name='detekt', root='http://dl.bintray.com/arturbosch/code-analysis/', m2Compatible='true')
@Grab('org.vcsreader:vcsreader:1.1.1')
import org.vcsreader.VcsProject
import org.vcsreader.vcs.VcsError
import org.vcsreader.vcs.git.GitVcsRoot


PHP 7.1 support

This PR introduces initial support for PHP 7.1 nullable (? syntax) return and parameter hints, as well as void return hint.
Implements #84
Implements #85
This last change can probably be reversed, since we don't really use `getType()` besides for testing, while consumers may actually be doing that...~~~ (fixed, BC retained)

TODOs:

 - [x] code review (ping @trowski, since you may want to adjust implementation on this)
 - [x] code review (ping @nikic, since you had the idea of just dropping `ReflectionType#__toString()`)
 - [x] figure out how to make this work both on 7.1 and on 7.0 (currently works only on 7.1)
 - [x] figure out why PHP throws `'Fatal error: Allowed memory size of 134217728 bytes exhausted (tried to allocate<PHONE_NUMBER>603911168 bytes) in /Users/ocramius/Documents/Projects/zend/zend-code/vendor/sebastian/global-state/src/Snapshot.php on line 329'` and `'Process finished with exit code 139 (interrupted by signal 11: SIGSEGV)'` when dealing with `(string)` casts of a `ReflectionType` representing a `?string` type

Note: changes will need to be reflected into:

 * https://github.com/doctrine/common/issues/731
 * https://github.com/Ocramius/ProxyManager/pull/327
 * https://github.com/Ocramius/ProxyManager/issues/314

@weierophinney can we drop PHP 5 at some point?

can we drop PHP 5 at some point?

Eventually, but not yet. 5.6 is still supported. Once that goes off active support, we can do a new minor version that bumps minimum to a v7 release.

Eventually, but not yet. 5.6 is still supported. Once that goes off active support, we can do a new minor version that bumps minimum to a v7 release.

I'll start ticking off the days.

Please don't drop the PHP 5.6 support too early. There are a lot of projects out there which won't be merged to PHP 7.x so quickly.
I would favour a new major release rather than a new minor release. Should be 4.x then to drop PHP 5.6 support. People tend to configure their composer.json files like this:
    "zendframework/zend-code": "^3.0",

and not like this:
    "zendframework/zend-code": "^3.0.0",

Jm2c

@RalfEggert this targets 4.0

Great! then I misunderstood Matthews comment...

What about support for these things in the Scanner classes?

@djmattyg007 didn't check them at all, sorry.

So is it safe to say at this point that the Scanner classes are no longer maintained?

No, i simply didn't t look at them, nor will do so in this PR. You can work on them basing your work on the test assets in this PR, if you want.

Are you still seeing the "allowed memory size" issue on the RC1 build?

I think that this is ready for review/merge.
Works on 5.6, 7.0, 7.1.

You mean you dropped 5.5. Which is perfectly acceptable for the next minor release. :smile:

Yep, typo, sorry :-)

Are you still seeing the "allowed memory size" issue on the RC1 build?

@nikic that's gone, thanks! :-)

@weierophinney implemented `iterable` support, removed useless docblock on `EmptyClass`

@weierophinney this is ready for merge :shipit:

Rebased, checked, ready. 👍
@weierophinney I'll deal with release notes and the actual release once this is through.

Needs to be delayed: reflection in 7.1 is still unstable/a mess. See https://github.com/php/php-src/pull/2136

@Ocramius As long as you use getName() in 7.1, everything is fine. The PR may only impact how __toString() behaves.

Yes, but I don't want to release something that may break in the next RC
anyway.
On 21 Sep 2016 15:25, "Nikita Popov"<EMAIL_ADDRESS>wrote:
@Ocramius https://github.com/Ocramius As long as you use getName() in
7.1, everything is fine. The PR may only impact how __toString() behaves.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/zendframework/zend-code/pull/87#issuecomment-248611339,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAJakNPUdXLiDDUqyRH7I81sQodST4s6ks5qsTBRgaJpZM4JouCM
.

Merging. The failure is caused by a CS check that is fixed in master, and will be fixed here when merging develop into master.


LB SSL termination

Feature flow in the UI @vincent99

user has to pre-upload the certificate to Rancher in order to use SSL on the Load balancer.
When create a load balancer, list of certificate + default certificate can be specified. Those certificates will be uploaded to the Load Balancer, and used by LB's https/ssl listeners.
We might want to put a check box against each listener identifying whether its an ssl type of listener. All ports with ssl checkbox on should be included to "io.rancher.loadbalancer.ssl.ports" label in format of integers separated by coma "port1,port2...port(n)" along with including them to the "ports" parameter itself.

Notes:
You can update the list of certificates for existing load balancer via loadbalancer.edit.
If no default certificate is specified, strict SNI rules will be applied on the backend
Rancher-compose syntax @ibuildthecloud

Similar to the UI, ssl ports should be listed in "io.rancher.loadbalancer.ssl.ports" label + in "ports" section itself
certificates get specified in "io.rancher.loadbalancer.ssl.certs" label in format of "cert_name, cert_name2"
default cert gets specified in "io.rancher.loadbalancer.default.cert" label in format of "cert_name"

Requires new version of agent-instance having 1.5 haproxy:
https://github.com/rancher/agent-instance/pull/6

@ibuildthecloud @vincent99 please help to review as I know both of you have SSL knowledge. Need closer review for SSL certificate validation section + feature flow in general.

@alena1108 Please update cattle-global.properties to rancher/agent-instance:v0.4.0  The image is now on DockerHub.

@ibuildthecloud updated cattle-global.properties, thanks


fix: spec/rails_helper: set expected RELEASE_VERSION

Test for health_controller expects RELEASE_VERSION to be set to 'VERSION_PROVIDED_ON_BUILD' - however, this is only done in Dockerfile.
Make the tests self-contained by setting adding it to the environment from within tests (if not already set).

Replacing #340 with a PR sent internally.


Question Answering&Named Entity Recognition: The rows and row headings are misaligned in the "Data Preview" table when dragging the scroll bar.

System Information (please complete the following information):
Windows OS: Windows-11-Enterprise-22H2
ML.Net Model Builder 2022: <IP_ADDRESS>2201 (Main Build)
Microsoft Visual Studio Enterprise: 2022(17.8.5)
.Net Core: 6.0,7.0,8.0
Describe the bug

On which step of the process did you run into an issue:
The rows and row headings are misaligned in the "Data Preview" table when dragging the scroll bar.

TestMatrix
https://github.com/dotnet/machinelearning-modelbuilder/files/14342564/NERTestdata.zip
To Reproduce
Steps to reproduce the behavior:

Select Create a new project from the Visual Studio start window.
Choose the C# Console App (.NET Core) project template.
Add model builder by right click on the project.
Select "Question Answering" or "Named Entity Recognition" scenario.
On the Data page, choose the data source, drag the scroll bar under the "Data Preview" table.
Sometimes the rows and row headings are misaligned.

Expected behavior
The rows and row headings should be aligned in the "Data Preview" table when dragging the scroll bar.
Screenshot:

Additional context:

This seems WPF DataGrid scrolling issues.
@zewditu should do more investigation


Update version for 4.1.1 release

4.1.2 in master branch

Ah, that's true. But this value is set incorrectly in the tagged release. Is there a way to fix that?

As I can see the 4.1.1 and 4.1.2 already released. What do you want to update?

Looks like the code changed since I posted this. Nothing to worry about then. Thanks!


Make ContentTransfer (), End () and Format() work even after go1.8

t0-5 fields of Result struct aren't used after go1.8 or later, but some codes depends on them. So, the above methods don't work go1.8 or later.
I fixed them.

Thanks

:+1:


Add who supports SV2

Until we get a confirmation from a few entities what you see on figma in this seciton is just a mockup and shouldnt' be implemeted until @moneyball confirms which entities are okay to go in there, reach out to him on BitcoinDesign Slack/SV2 Discord for confirmation.
Once you get confirmation from @moneyball reach out to Gene to make sure that area is ironed out, if so implement it 🚀

I'm in touch with all 5 and feeling reasonably optimistic about approvals.

Mining companies interested in supporting sv2:
Braiins - approved
Blockstream - approved
Galaxy - approved
Foundry - approved
Block - approved
Design-wise, the above list is expected to grow to dozens, hopefully many dozens, so ideally we know how the design will grow with that.
Companies funding sv2 project:
Spiral - approved
BitMEX - pending approved but expected (I'd say go ahead and include given the probability is high of approval)
Galaxy - approved
Foundry - approved

Block assets found here: https://block.xyz/mediakit

@pavlenex can you chase down SVG assets from each company? If you aren't already connected with them let me know and I can help.

@moneyball is Summer of bitcoin on the funder's list? I saw this mentioned.
I got all of the SVG logos already except block (png - or we can use the dancing gif)

Funding:
Summer of Bitcoin - approved

BitMEX - approved

PR is in https://github.com/stratum-mining/stratumprotocol.org/pull/34


Invoking completions in the middle of tag helper attribute results in empty completions window

Describe the bug:
In any other context, e.g. middle of html attribute, or middle of C# keyword or whatever else, when no completions are available completion window doesn't appear at all. So I think, it will be nice to not show it in razor context as well for feature parity reasons
Version used:
VS 2022 v17.3 Preview 2
To reproduce:
Manually invoke completions in the middle of tag helper attribute (ctrl + space)
Expected behavior:
No completion window appears
Actual behavior:
Completion window appears but says "No completions available"
Demo:

This is non-english VS localization, but you got the idea

Now that we have single server completion this is possible.

When you complete this item verify that it also fixes https://github.com/dotnet/razor-tooling/issues/6578, and re-open that issue if it does not.

This is fixed in 17.4 Preview


⚠ Bofrost.be has degraded performance

In b280fa5, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1498 ms

Resolved: Bofrost.be performance has improved in 3ce341e.


Release Toolium 2.1.0

Coverage remained the same at 90.033% when pulling f9d7019594f9b3d8916a1bb40fed5610d5902649 on release/2.1.0 into e0b996b329dfb3e81c83c4386b3f7fbed4e15d27 on master.


GH: Variable parameter Send/Receive nodes

Suggested by Jonathan from SOM, allow for multiple inputs to be created on the Send node, and expand the commit object on the Receive node.

PR is created and pending feedback. Anyway, I agree this is a necessary update so it could be merged in and released


Fix GraphQL invocation logic in Config DS

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

Project coverage is 29.17%. Comparing base (96a646a) to head (0d939ff).
Report is 33 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2123      +/-   ##
==========================================
+ Coverage   26.03%   29.17%   +3.14%
==========================================
  Files          68      118      +50
  Lines        4732    13871    +9139
  Branches     2426     2673     +247
==========================================
+ Hits         1232     4047    +2815
- Misses       3500     9693    +6193
- Partials        0      131     +131

Flag
Coverage Δ

adapter
27.63% <ø> (?)

config-deployer-service
26.03% <ø> (-0.01%)
:arrow_down:

idp-domain-service
80.21% <ø> (?)

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

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


Add section tortuosity feature to fst module.

Resolves #375.

@lidakanari Does this look OK?

See my previous comment:
I got a bit confused between section_length and path_distance definitions.


MacOS: Segmentation Fault on building MediaPipe's example C++ app

Please make sure that this is a build/installation issue and also refer to the troubleshooting documentation before raising any issues.
System information (Please provide as much relevant information as possible)

OS Platform and Distribution (e.g. Linux Ubuntu 16.04, Android 11, iOS 14.4): Installed on MacOS Monterey 12.3 Beta
Compiler version (e.g. gcc/g++ 8 /Apple clang version 12.0.0): Apple clang version 13.0.0 (clang-13<IP_ADDRESS>)
MediaPipe version: 0.8.9
Bazel version: 4.2.1
XCode and Tulsi versions (if iOS): Command Line Tool <IP_ADDRESS>.1.1638488800 installed via command line not App Store
OpenCV version (if running on desktop): 4.5.3

Describe the problem:
I've installed mediapipe and successfully run helloWorld. However, when I try building C++ command-line example apps running on CPU (https://google.github.io/mediapipe/getting_started/cpp.html), I always get the following error.
I've tried re-install everything and also tried install xcode from the app store but still get the error.
Provide the exact sequence of commands / steps that you executed before running into the problem:
bazel build -c opt --define MEDIAPIPE_DISABLE_GPU=1 mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu
Complete Logs:
Include Complete Log information or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached:
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'com_google_absl' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'com_google_benchmark' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'pybind11_bazel' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'com_google_protobuf' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'com_google_googletest' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'com_github_gflags_gflags' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'build_bazel_rules_apple' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'build_bazel_rules_swift' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'build_bazel_apple_support' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'xctestrunner' because it already exists.
DEBUG: /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/external/org_tensorflow/third_party/repo.bzl:122:14:
Warning: skipping import of repository 'pybind11' because it already exists.
WARNING: /Users/neufii/Documents/Work/mediapipe/mediapipe/framework/BUILD:54:24: in cc_library rule //mediapipe/framework:calculator_cc_proto: target '//mediapipe/framework:calculator_cc_proto' depends on deprecated target '@com_google_protobuf//:cc_wkt_protos': Only for backward compatibility. Do not use.
WARNING: /Users/neufii/Documents/Work/mediapipe/mediapipe/framework/tool/BUILD:182:24: in cc_library rule //mediapipe/framework/tool:field_data_cc_proto: target '//mediapipe/framework/tool:field_data_cc_proto' depends on deprecated target '@com_google_protobuf//:cc_wkt_protos': Only for backward compatibility. Do not use.
INFO: Analyzed target //mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu (0 packages loaded, 0 targets configured).
INFO: Found 1 target...
ERROR: /Users/neufii/Documents/Work/mediapipe/mediapipe/graphs/hand_tracking/subgraphs/BUILD:24:26: Executing genrule //mediapipe/graphs/hand_tracking/subgraphs:hand_renderer_cpu_inc failed: (Segmentation fault): sandbox-exec failed: error executing command
  (cd /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/sandbox/darwin-sandbox/1353/execroot/mediapipe && \
  exec env - \
    PATH=/Users/neufii/Library/Caches/bazelisk/downloads/bazelbuild/bazel-4.2.1-darwin-x86_64/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/local/bin:/opt/local/sbin:/Library/TeX/texbin:/opt/X11/bin:/Library/Frameworks/Mono.framework/Versions/Current/Commands \
    TMPDIR=/var/folders/yy/vtyc_g4x0w7dtpjywdb5xt0w0000gn/T/ \
  /usr/bin/sandbox-exec -f /private/var/tmp/_bazel_neufii/f5de0899693f5576c099856be410b430/sandbox/darwin-sandbox/1353/sandbox.sb /var/tmp/_bazel_neufii/install/d3bd16a8b22c859ba58f5dca04a9b1aa/process-wrapper '--timeout=0' '--kill_delay=15' /bin/bash -c 'source external/bazel_tools/tools/genrule/genrule-setup.sh; bazel-out/host/bin/mediapipe/framework/tool/encode_as_c_string "bazel-out/darwin-opt/bin/mediapipe/graphs/hand_tracking/subgraphs/hand_renderer_cpu.binarypb" > "bazel-out/darwin-opt/bin/mediapipe/graphs/hand_tracking/subgraphs/hand_renderer_cpu.inc"')
/bin/bash: line 1: 95387 Segmentation fault: 11  bazel-out/host/bin/mediapipe/framework/tool/encode_as_c_string "bazel-out/darwin-opt/bin/mediapipe/graphs/hand_tracking/subgraphs/hand_renderer_cpu.binarypb" > "bazel-out/darwin-opt/bin/mediapipe/graphs/hand_tracking/subgraphs/hand_renderer_cpu.inc"
Target //mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu failed to build
INFO: Elapsed time: 0.621s, Critical Path: 0.29s
INFO: 136 processes: 136 internal.
FAILED: Build did NOT complete successfully

Hi @neufii , can you provide steps to reproduce the issue.

Hi, I just followed the installation guide here(use option 1 for opencv).
After I successfully run Hello World Example (step 6), I just stay in the same folder (mediapipe) and follow the guide with option 1 (Running on CPU). I got the error above after running the command in step 1.

@neufii
Can you try with this
bazel clean --expunge
and retrying the build command.

@PrinceP Thank you for the suggestion. I've tried bazel celan --expunge followed by bazel build -c opt --define MEDIAPIPE_DISABLE_GPU=1 mediapipe/examples/desktop/hand_tracking:hand_tracking_cpu (also tried bazel clean` then build command) but they didn't work for my case 🥲

UPDATE: I've uninstalled command line tools and installed xcode from App Store. Still got the same error.

These steps are done ?
brew install opencv@3
brew uninstall --ignore-dependencies glog
Can you add --verbose_failures in your command also?

@PrinceP Yes, I already did those step and got the error. However, I just managed to solve my case.

Remove command line tools
Install XCode from the App Store and accept the license agreement
bazel clean --expunge

Thank you for the support :)


Fix scheduler bug on TPUs

TPUs should not be stepped n times like we do on GPUs. This will result in a severe performance degradation, and was the source of this issue https://github.com/huggingface/accelerate/issues/526
As a result, this PR adds a hard check if we step_with_optimizer to have it also perform the step if we are on the TPU.
Fixes https://github.com/huggingface/accelerate/issues/526

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

As discussed offline, this is not the right solution IMO. There should be no differences for schedulers steps in multi-GPUs and TPUs.


Add raw DB editing capabilities for tournaments

Because of how boltdb works we can't run the web API and the app at the same time. This is unfortunately a deal breaker for the web API, so we need something a bit more raw.

[ ] Add /:tournament/edit
[ ] Find a pluggable js JSON editor and plug it in
[ ] Add backend endpoints to accept JSON entry. Producer only.

https://github.com/jdorn/json-editor
https://github.com/alecthomas/jsonschema


Ensure that we use the dtype keyword in normalize_chunks

Fixes https://github.com/dask/dask/issues/4644

[ ] Tests added / passed
[ ] Passes flake8 dask

Thanks @mrocklin!

Thanks for the review!
On Fri, Mar 29, 2019 at 11:28 AM James Bourbeau<EMAIL_ADDRESS>wrote:

Thanks @mrocklin https://github.com/mrocklin!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/4646#issuecomment-478103214, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszAQty2Q1Lg2WMthvJJRetrzk-Xp7ks5vbltEgaJpZM4cRpv5
.


Update Confirmation fields for Code of Conduct, Photo/Video Form, and Liability Waiver to HackWITus versions

On the confirmation page there are links that go to a liability form, a photo waiver, and a code of conduct. We need to update these links to ones for our hackathon.

Was going through the MLH guidelines and noticed this, can one of you incorporate this as well?


Writing assignments is a lot of work

You have to create an ExpressionStatement, create a BinaryExpression, set the operator to =, create an Identifier and then finally set your value. Assignments are super common, so this should be simplified.

You don't have to create ExpressionStatement any more (edbe4fbb516eeaa2ce705acd7bd6345c9b038ea4) which helps with this issue


Fix Runtime warning on pushing multiple items into NavigationStackController (TCA only)

as per
https://pointfreecommunity.slack.com/archives/C04KQQ7NXHV/p1730815911553849

Hi @loongallday, thanks for taking the time to look into this, however your changes do break the test suite:
2024-11-14T20:31:17.8608910Z Failing tests:
2024-11-14T20:31:17.8610470Z 	NavigationPathTests.testAppend_UnrecognizedType()
2024-11-14T20:31:17.8611740Z 	NavigationPathTests.testDeepLink_UnrecognizedType()
2024-11-14T20:31:17.8613520Z 	NavigationPathTests.testDeepLink_UnrecognizedType()
2024-11-14T20:31:17.8632940Z 	NavigationPathTests.testPush_UnrecognizedType()
2024-11-14T20:31:17.8639810Z
2024-11-14T20:31:17.8639970Z ** TEST FAILED **

You can run the tests in the Examples project to see these failures for yourself. Would you be interested in looking into a fix?
Without looking into the issue very much I can definitely say that your change must be tweaked a bit. With the way the logic is now nothing is ever appending indices to invalidIndices. It is just always an empty index set. Even your if !invalidIndices.isEmpty check isn't doing anything. So this needs a bit more work.

Thanks, @mbrandonw I'll look into it but from basic inspecting I accidentally change the min iOS version to 15 since I wanna test with both version but forgot to change it back, will try that and run the test locally again :)

silly me, I put the the isEmtpy check in the wrong position facepalm

@loongallday One last question... would you happen to know how to write a test that triggers the runtime warning without this change? It would be nice to get some test coverage on this behavior.


Adding a missing link to serverless next.js docs

I'm getting into this project (great stuff, by the way!) and I've noticed a missing link in the docs so I figured - might as well add it 🎉

This is great! Thank you @tlakomy!


Inline parameters seems not to have precedence over local parameter file.

While deploying with latest az cli this command:
az deployment group validate -f t.bicep --parameters t.bicepparam environment=tst --resource-group rg-zm-esb-common-dev
The output of the variable is still the value from the bicepparam file and NOT of the inline parameter.

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

ID: cdf73225-d2e6-b23e-b355-fbb9d3fcdca3
Version Independent ID: fa832878-9821-f222-5f15-ad8646737a04
Content: Create parameters files for Bicep deployment - Azure Resource Manager
Content Source: articles/azure-resource-manager/bicep/parameter-files.md
Service: azure-resource-manager
Sub-service: bicep
GitHub Login: @mumian
Microsoft Alias: jgao

@rfontijn
Thanks for your feedback! We will investigate and update as appropriate.

@rfontijn, thank you for reporting it.
@mumian, could you please review this question and share your thoughts. Thank you

@rfontijn Unfortunately inline parameters are not supported with a bicepparam parameters file. We are working towards supporting this in a future Bicep release.

@rfontijn - I will add a note to the article.
#please-close

New-AzResourceGroupDeployment @CmdletParameters -Force -Verbose:$ …
|      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| Cannot retrieve the dynamic parameters for the cmdlet. Error parsing undefined value. Path '', line 1, position 1.
 Bit of an odd error message.

 That said, is there an issue to track the support/implementation of bicep param file with inline parameters?

I would also be interested to know where I can track progress on this issue.


Migrate pipeline

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Trino ingestion tests has not been passing in the upstream repo for a while.
pulling the delta in trino code also does not solve the problem.

lesson: always check the upstream tests result + look at the code differences + setup.py version differences, between this repo and the upstream, before deciding to jump all in to run the ingestion test locally.

oops, missed out hana_ingest

to squash all the commits as a single commit later

handled by another PR


RichText view mode trouble

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
version of the library: [1.13.0]
Expected / Desired Behavior / Question
No problem work in view mode
Observed Behavior
If youre try to open page with rich text editor in view mode it will not render complete. Need to click edit page then publish it, and only then it will display correctly.
Steps to Reproduce
1)Build webpart with rich text editor
2) pass dynamic isEditMode to props of Richtext Editor
3) Deploy it on page
2) Open page - there are no text editor. Click edit - it appear. click Publish - all ok. Reload page - here we go again

Facing the same issue , any updates to this ?

@hugoabernier any chance you can take a look at this?

@estruyf I can look at it next week.

Here's a quick hack if anyone needs this right now:

class SafeRichText extends RichText {
  public componentDidMount() {
    if (this.props.isEditMode) {
      super.componentDidMount();
    }
  }
}

Swap  in place of
Here's the original code:

I think the brace at line 140 needs to be moved down to line 149, then the problem should go away :)
Joel

@joeljeffery has fixed this, and the changes are applied in the latest beta version of 1.14.0. Please feel free to test it out. More information about how to use a beta version can be found here: https://sharepoint.github.io/sp-dev-fx-controls-react/beta/


Pluggable Auth: Minimal RBAC Authorization

as outlined in #628

Design Proposal
Goals

Maximum Flexibility with minimal effort for common cases
No one one has any permissions by default, all permissions need to be explicitly given
Asterisk (*) can be used to mean "all resources of this category"
Modelled after Kubernetes RBAC, because the user experience is great and users might already be familiar with the system

2-Step Configuration
We have two resources to configure Authorization:

A Role grants permissions to everyone who owns the role, for example ("get all things")
A RoleBinding assigns a role to a a user or group

Role
name: string
rules:
  - kinds: []string
    classes: []string
    verbs: []string

RoleBinding
name: string
roleName: string
subjects:
  - type: one of 'user', 'group'
     name: string

Exmaple configs
Example 1: Anonymous reads, but writing restricted
Goal: Anonymous users can read everything, admins can create/update/delete everything
Roles
name: read-all
rules:
  - kinds: ["*"]
    classes: ["*"]
    verbs: ["GET"]

name: crud-all
rules:
  - kinds: ["*"]
    classes: ["*"]
    verbs: ["*"]

Role-Bindings:
name: anonymous-read-all
roleName: read-all
subjects:
  - type: group
     name: anonymous

name: admin-crud-all
roleName: crud-all
subjects:
  - type: group
     name: admin

Example 2: Fruit Salad
This example shows that we have maximum flexibility, even this overly complex use-case would easily be possible:
Goal:

Anyone can read all Actions
Bob can read Bananas and read Apples
Remi can CRUD Oranges and read Bananas and Oranges
Micha can CRUD all actions

Roles
name: read-actions
rules:
  - kinds: ["action"]
    classes: ["*"]
    verbs: ["GET"]

name: banana-apple-read
rules:
  - kinds: ["thing"]
    classes: ["apple", "banana"]
    verbs: ["GET"]

name: orange-crud
rules:
  - kinds: ["thing"]
    classes: ["orange"]
    verbs: ["*"]

name: actions-crud
rules:
  - kinds: ["action"]
    classes: ["*"]
    verbs: ["*"]

Role-Bindings:
name: anonymous-read-actions
roleName: read-all
subjects:
  - type: group
     name: anonymous

name: banana-apple-readers
roleName: banana-apple-read
subjects:
  - type: user
     name<EMAIL_ADDRESS>  - type: user
     name<EMAIL_ADDRESS>
name: orange-cruder
roleName: orange-crud
subjects:
  - type: user
     name<EMAIL_ADDRESS>
name: actions-cruder
roleName: actions-crud
subjects:
  - type: user
     name<EMAIL_ADDRESS>
Feedback
@bobvanluijt @studiospindle Happy to hear your feedback on this RBAC Authorization proposal :)

@etiennedi
I don't know a lot about authorization&authentication, but the design looks very good. I took a quick look at Kubernetes' RBAC Authorization, and I think their and your design are very straightforward and clear!

@etiennedi should this one stay open?


Update webviewcontrol_capturepreviewtostreamasync_1169189879.md

Testing automation. Added whitespace.
Please do not merge.

@MancunianBandit : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


"File not found" error fixed for SQL inline migration

"File not found" error fixed for SQL inline migration
The __dirname name here calls for the working directory which is sql extension folder and from there we are navigating to scripts folder to get the required script

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.004%) to 41.76%

Files with Coverage Reduction
New Missed Lines
%

src/sql/workbench/services/connection/browser/connectionDialogWidget.ts
1
73.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.004%

Covered Lines:
30828

Relevant Lines:
69082

💛  - Coveralls


:copen opens quickfix window below lastwin instead of curwin

Steps to reproduce

Create a new split on the right
Jump to the left window
:copen
Quickfix window is opened below the right split

Expected behaviour
:copen should open the window below current window as it says here: https://github.com/vim/vim/blob/cdb7b4c50826df254d2e5ba8abd211e49b7a9784/src/quickfix.c#L4274
Version of Vim
VIM - Vi IMproved 8.1
Environment
N/A
Logs and stack traces
No response

Might be that I'm misunderstanding the expected behavior.
If it's not supposed to be opened under curwin, would it be possible to change the default behavior to open with WSP_BOT (instead of WSP_BELOW)?

The help states:
Normally the quickfix window is at the bottom of the screen.  If there are
vertical splits, it's at the bottom of the rightmost column of windows.

Is that not what is happening?  A comment in the code can easily be changed,  I
rather not change the actual behavior unless there is a good reason.

Yeah, that's what the OP is describing.
IOW: "T'ain't a bug, it's a feature."

Is that not what is happening?

Ah yes. The behavior is following the help text.
Makes sense.

I rather not change the actual behavior unless there is a good reason.

Opening qf window at the bottom right corner, by default, makes it seem like the quickfix list is somehow related to the window at bottom right corner. So opening it at the bottom of the editor seems more intuitive to me.
But it's your call... if this change is not desired, the issue can be closed.
Thanks.

You may want to use the locationlist window instead,  I think it should be opened below the currently active window instead.

Yeah, but locationlist is bound to a specific window.
If we want list that contains locations from various files across the codebase, quickfix list is the appropriate feature, right?
And since it's not bound to any specific window, (to me) it feels like bottom-of-the-editor would be better suited place to open it by default. Many plugins I use explicity executes :botright copen because of that. Would've been nice if :copen did that by default.
But again, it's the maintainers' call. If you think this shouldn't be changed, the issue can be closed.

If we want list that contains locations from various files across the codebase, quickfix list is the appropriate feature, right?

What has that to do with the other? I tend to use the locationlist, because it is more flexible and you can have more than one. But it doesn't depend on whether I want to visit files across the codebase.
But as has been said, I think changing the existing behaviour may surprise a lot of users, let's leave it as it. Also it should be possible to move the window around to a certain extend afterwards. So let's close it then.

On 2023-06-07, Munif Tanjim wrote:

Might be that I'm misunderstanding the expected behavior.
If it's not supposed to be opened under curwin, would it be possible to change
the default behavior to open with WSP_BOT (instead of WSP_BELOW)?

I didn't like the default behavior, either, so I added this to my
~/.vim/after/ftplugin/qf.vim.
" The following is taken from the QFEnter.vim plugin,
" http://vim.sourceforge.net/scripts/script.php?script_id=4778,
" by yssl.  I extracted and simplified the part I needed:  When
" jumping to items from the quickfix window, open them in the
" previous window instead of the window above the quickfix
" window.

nnoremap <buffer> <silent> <CR> :call QfEnter()<CR>
let b:undo_ftplugin .= "| nunmap <buffer> <CR>"

function!
If it's not supposed to be opened under curwin, would it be possible to change
the default behavior to open with WSP_BOT (instead of WSP_BELOW)?

I didn't like the default behavior, either, so I added this to my
~/.vim/after/ftplugin/qf.vim.
" The following is taken from the QFEnter.vim plugin,
" http://vim.sourceforge.net/scripts/script.php?script_id=4778,
" by yssl.  I extracted and simplified the part I needed:  When
" jumping to items from the quickfix window, open them in the
" previous window instead of the window above the quickfix
" window.

nnoremap <buffer> <silent> <CR> :call QfEnter()<CR>
let b:undo_ftplugin .= "| nunmap <buffer> <CR>"

function! QfEnter()
    let l:lnum = line('.')
    wincmd p
    try
        exe 'cc' l:lnum
        normal! zv
    catch /E37/
        if &errorbells
            normal \<Esc>               " Generate a bell
        endif
        echohl ErrorMsg
        echo matchstr(v:exception, ':\zs.*')
        echohl None
    endtry
endfunction

HTH,
Gary

Thank you for this.  It sometime annoys me when pressing Enter on QF
window opens the file in a different window than the one I previously
accessed.
But out of curiosity, this isn't what OP was after, is it?  Munif was
looking to change the default QF behavior by opening the QF window under
the current window.  Your function, Gary, opens the file in QF entry in
the previous window.  They are different asks, AFAI can tell.
--
Enan

Munif was looking to change the default QF behavior by opening the QF window under
the current window.

Just a small correction... not under the current window, but under the whole editor.
So, if there are existing windows A, B and C, I wanted :copen to do this by default:
---------
|   | B |
| A |---|
|   | C |
---------
|  QF   |
---------

Munif was looking to change the default QF behavior by opening the QF window under
the current window.

Yeah, I wanted it to open under current window initially (when creating the issue).
But later thought that under the whole editor makes more sense... so, if there are existing windows A, B and C, it makes more sense for :copen to do this by default:
---------
|   | B |
| A |---|
|   | C |
---------
|  QF   |
---------


[Bug][server] default worker group nodes may not sync

Describe the bug
This bug is also reported in https://github.com/apache/dolphinscheduler/issues/5528 which is not resolved yet though already closed. I ran into this bug when I deployed dolphinscheduler on k8s. I may have found the real cause of this bug, and give detailed reproduce steps as followed.
The bug is described as below.
After clicking run button and choose default worker group, tasks can not be dispatched forever.
Master node print error log which says fail to execute : %s due to no suitable worker.
To Reproduce
To reproduce, you can add Thread.sleep(60000) after line "load();" in method ServerNodeManager.afterPropertiesSet() to enlarge the timespan and increase the probability that this bug happens. And start master node first till it falls asleep, then start worker node.
Which version of Dolphin Scheduler:
-[1.3.6-release]
Requirement or improvement
From the reproduce steps, you can see the cause of this bug. It's because the map workerGroupNodes fail to sync with zookeeper. And that is because the listener registered too late. So maybe the solution is to switch the execute order of load() and listener registration.

After the Master/Worker is started, it will subscribe to the registry listener, and then pull information from the registry. Can you help me confirm that the current process is like this?

After the Master/Worker is started, it will subscribe to the registry listener, and then pull information from the registry. Can you help me confirm that the current process is like this?

Yes.


fix(dropdown): remove scroll addition to re-fix positioning

Same as the fix for dialog ... seems like improvements to position.service create conflicts with adding the window scroll

We are still getting this issue. Versions of carbon used:
"carbon-components": "^10.4.1", "carbon-components-angular": "^3.14.2"
Angular version is 8.1.x.
Dropdown menu is still positioned incorrectly when page is scrolled.

see https://github.com/IBM/carbon-components-angular/pull/695


Add end line for CHANGELOG.md

Codecov Report

Merging #13749 (c70e889) into master (3ec7ef9) will decrease coverage by 26.20%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #13749       +/-   ##
=============================================
- Coverage     43.72%   17.52%   -26.21%
+ Complexity     9074     2660     -6414
=============================================
  Files          1384     1384
  Lines         80313    80313
  Branches       9768     9768
=============================================
- Hits          35114    14071    -21043
- Misses        42269    65000    +22731
+ Partials       2930     1242     -1688

Impacted Files
Coverage Δ

...mon/src/main/java/alluxio/shell/CommandReturn.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...mon/src/main/java/alluxio/util/ExceptionUtils.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/wire/AlluxioProxyInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/alluxio/wire/AlluxioMasterInfo.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/job/meta/JobIdGenerator.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/src/main/java/alluxio/stress/BaseParameters.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...src/main/java/alluxio/client/UnderStorageType.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/underfs/options/ListOptions.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/worker/block/io/BlockReader.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...main/java/alluxio/worker/block/io/BlockWriter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 660 more

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 3ec7ef9...c70e889. Read the comment docs.

alluxio-bot, merge this please


Switch code formatter from Biome to Prettier

Close #24
discussion: https://github.com/svelte-docgen/svelte-docgen/pull/21#issuecomment-2540320847

In fact. This PR made me think if Biome is actually a right choice for this repository. Given that it doesn't support Svelte yet.
We could always switch back once it is supported, and some little slowness on CI caused by Prettier... I believe it would be tolerable.

@xeho91 We’re switching over to Prettier. It won’t impact the overall CI runtime, since Vitest takes more time.


Why is Simplify static in PublicApi?

@theimowski Is there a reason why Simplify is offered as static method of Dependencies and not as instance one like other processes?

I wanted to move away from Locate member of Dependencies which throws exception if dependencies file is not found. That was the easiest way to leave rest of the commands as they are now

You can use Create method, it doesn't throw.

But is it not what is intended for Simplify, can it function without dependencies file?

nope, that's why it calls PaketEnv.fromRootDirectory first which returns Failure if dep file not found

Solved?

whoa I can see some hard issue cleaning going on ;)
nope, it stayed that way - might need revision one day

/// Converts the current package dependency graph to the simplest dependency graph.
member this.Simplify(interactive : bool) =
    Utils.RunInLockedAccessMode(
        this.RootPath,
        fun () ->
            PaketEnv.fromRootDirectory this.RootDirectory
            >>= PaketEnv.ensureNotInStrictMode
            >>= Simplifier.simplify interactive
            |> returnOrFail
            |> Simplifier.updateEnvironment
    )


events: ["keyup"] in ie8

events: ["keyup"] in ie8 is not work, only blur can occur update model; Can solve?

Try using StackOverflow for questions.


Adding different types of Ciphers algorithms in Java-A-Z.

Adding different types of Ciphers algorithms in Java-A-Z.

Can I work on this issue?

@Danielinte  yes you can work on this app

@dubesar
What specific algorithms should it be? (symmetric/asymmetric...)
Where should be the section/directory for these algorithms?

Can I add algorithm code in this?

@DeepanshuGOYAL05 Sure,you can.

Hey,
I have added two Cipher Algorithms. Please review it and check pull
request!!
On Tue, Oct 6, 2020, 2:14 PM Kunal Mali<EMAIL_ADDRESS>wrote:

@DeepanshuGOYAL05 https://github.com/DeepanshuGOYAL05 Sure,you can.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dubesar/Java-A-Z/issues/377#issuecomment-704124114,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH6TG725XIUGY5N57PNSACTSJLKH7ANCNFSM4SEKNNUA
.


feature: run npm install with --production

Plugins, webapps and server updates should be installed
without development dependencies. Less to download and store and
a little less attack surface.
This requires a fair bit of testing and probably some beta cycles.

This was a brain fart, npm install xx doesn't install dev deps.