[Bug] New-DbaAgentJob -Force option drops schedule?

Attempting to use the New-DbaAgentJob.  the process I currently have is:

Create new operator 'New-DbaAgentOperator -Force'
Create new schedule 'New-DbaAgentSchedule -Force'
Create new job 'New-DbaAgentJob -Force'

If neither of the 3 exist, then it works without issue.
If however the job initially existed, the New-DbaAgentJob with -Force appears to drop the schedule too when it should not.
$connectedInstance = Connect-DbaInstance -SqlInstance $instance

New-DbaAgentOperator    -SqlInstance $connectedInstance -Operator $OperatorEmail -EmailAddress $OperatorEmail -WhatIf:$WhatIf -Verbose:$Verbose -Force -PagerDay EveryDay

$connectedInstance | New-DbaAgentSchedule -Schedule $AgentSchedule -StartDate $JobScheduleDate.ToString("yyyyMMdd") -StartTime $JobScheduleDate.ToString("HHmmss") -Force

$connectedInstance | New-DbaAgentJob -Job $AgentJob -Schedule $AgentSchedule -OwnerLogin "disabled_sa" -EmailOperator $OperatorEmail -EmailLevel "Always" -WhatIf:$WhatIf -Verbose:$Verbose -Force

Error I receive is:
WARNING: [16:28:15][New-DbaAgentJob] Something went wrong creating the job | The specified @name ('CHG________<EMAIL_ADDRESS>::') already exists.
Shouldn't the -Force flag recreate the job if it already exists?
Host used

[ ] powershell.exe
[X] ISE
[X] VS Code
[ ] Other (please specify)

Environmental information
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨
Please provide the output of the below script
PowerShell Version        : 5.1.18362.1171
dbatools latest installed : 1.0.141
Culture of OS             : en-US

@cmuncan, I recommend you remove your email address from the posting above (in the error message output).
Yes, "New-DbaAgentJob -Force" will drop the schedule when the existing job is removed. This is consistent with the default behavior in SSMS for unused schedules.
While troubleshooting this I noticed that pre-connected instances lead to caching issues with job schedules that have been deleted from the server. Even after calling Refresh() on the SMO server object it still held a reference to a deleted schedule. You'll see in the example code below I don't call Connect-DbaInstance prior to the other commands, but instead pass in the server info so that each command can obtain a fresh connection.
Example working code:
Option 1
$InstanceName = "localhost"
$OwnerLogin = "enter job owner here"
$OperatorEmail = "enter email address here"
$EmailLevel = "Always"
$WhatIf = $false
$Verbose = $true
$AgentScheduleName = "dbatools_test_schedule1"
$JobScheduleDate = "20210316"
$JobScheduleTime = "140000"
$AgentJobName = "dbatools_test_job1"

$newOperator = New-DbaAgentOperator -SqlInstance $InstanceName -Operator $OperatorEmail -EmailAddress $OperatorEmail -WhatIf:$WhatIf -Verbose:$Verbose -Force -PagerDay EveryDay
$newSchedule = New-DbaAgentSchedule -SqlInstance $InstanceName -Schedule $AgentScheduleName -StartDate $JobScheduleDate -StartTime $JobScheduleTime -Force
$newAgentJob = New-DbaAgentJob -SqlInstance $InstanceName -Job $AgentJobName -Schedule $AgentScheduleName -OwnerLogin $OwnerLogin -EmailOperator $OperatorEmail -EmailLevel $EmailLevel -WhatIf:$WhatIf -Verbose:$Verbose -Force

Option 2
$InstanceName = "localhost"
$OwnerLogin = "enter job owner here"
$OperatorEmail = "enter email address here"
$EmailLevel = "Always"
$WhatIf = $false
$Verbose = $true
$AgentScheduleName = "dbatools_test_schedule1"
$JobScheduleDate = "20210316"
$JobScheduleTime = "140000"
$AgentJobName = "dbatools_test_job1"

$newOperator = New-DbaAgentOperator -SqlInstance $InstanceName -Operator $OperatorEmail -EmailAddress $OperatorEmail -WhatIf:$WhatIf -Verbose:$Verbose -Force -PagerDay EveryDay
$schedule = Get-DbaAgentSchedule -SqlInstance $InstanceName -Schedule $AgentScheduleName
if ($null -eq $schedule) {
    $schedule = New-DbaAgentSchedule -SqlInstance $InstanceName -Schedule $AgentScheduleName -StartDate $JobScheduleDate -StartTime $JobScheduleTime -Force
}

$existingAgentJob = Get-DbaAgentJob -SqlInstance $InstanceName -Job $AgentJobName
if ($existingAgentJob) {
    $removedAgentJob = Remove-DbaAgentJob -SqlInstance $InstanceName -Job $AgentJobName -KeepUnusedSchedule -WhatIf:$WhatIf -Verbose:$Verbose
}
$newAgentJob = New-DbaAgentJob -SqlInstance $InstanceName -Job $AgentJobName -Schedule $AgentScheduleName -OwnerLogin $OwnerLogin -EmailOperator $OperatorEmail -EmailLevel $EmailLevel -WhatIf:$WhatIf -Verbose:$Verbose

@cmuncan, I recommend you remove your email address from the posting above (in the error message output).

Thank you I've removed it.  Oops

While troubleshooting this I noticed that pre-connected instances lead to caching issues with job schedules that have been deleted from the server. Even after calling Refresh() on the SMO server object it still held a reference to a deleted schedule. You'll see in the example code below I don't call Connect-DbaInstance prior to the other commands, but instead pass in the server info so that each command can obtain a fresh connection.

I noticed this also and that was my solution too to do 1 Connect-DbaInstance and then pass that to each call with -SqlInstance.

misunderstanding of how it all hangs together.  after it was explained.  fully understand now and closing the request


Support multiple parcels and node.js support in pre-warmed images.

Scripts including some commented out data. Please let me know if that data need to be removed.


Lib Refactoring

Description
Some of  current components in Sedna Lib  provide services independently based on different features. Each feature provides an independent interface and code structure, which is confusing. To better facilitate those use cases, We need to start thinking about decoupling some of our components, While not a specific action item for Sedna, believe this conversation is worthwhile.
Discussion so far:

Secondary development is difficult,  developers cannot replace with their customized processing logic. (e.g. pre-process, post-process, feature engineering, etc.).
Too many non-global variables defined in baseConfig is that since every function has access to these, it becomes increasingly hard to figure out which feature of sedna actually read and write these variables.
Right now Sedna Lib supports only Tensorflow, which possibly limits the number of possible users. If we can provide anyway to allowed extend other ML framework, it will benefit more developers.
The current feature  modules (incremental_learning 、 joint_inference、federated_learning, etc.) have classes scattered around in a generally unorganized mess，it throws users out of their rhythm when they go to learn 、use different modules.

Details
	During reviews I've seen a few areas where we can decouple concepts:

By using a  registration of class-factory functions to emulate virtual constructors, developers can invoke different components by change variables in the Config file.

class ClassFactory(object):
    """A Factory Class to manage all class need to register with config."""

    __registry__ = {}

    @classmethod
    def register(cls, type_name='common', alias=None):
        """Register class into registry.

        :param type_name: type_name: type name of class registry
        :param alias: alias of class name
        :return: wrapper
        """

        def wrapper(t_cls):
            """Register class with wrapper function.

            :param t_cls: class need to register
            :return: wrapper of t_cls
            """
            t_cls_name = alias if alias is not None else t_cls.__name__
            if type_name not in cls.__registry__:
                cls.__registry__[type_name] = {t_cls_name: t_cls}
            else:
                if t_cls_name in cls.__registry__:
                    raise ValueError(
                        "Cannot register duplicate class ({})".format(t_cls_name))
                cls.__registry__[type_name].update({t_cls_name: t_cls})

            return t_cls

        return wrapper

    @classmethod
    def register_cls(cls, t_cls, type_name='common', alias=None):
        """Register class with type name.

        :param t_cls: class need to register.
        :param type_name: type name.
        :param alias: class name.
        :return:
        """
        t_cls_name = alias if alias is not None else t_cls.__name__
        if type_name not in cls.__registry__:
            cls.__registry__[type_name] = {t_cls_name: t_cls}
        else:
            if t_cls_name in cls.__registry__:
                raise ValueError(
                    "Cannot register duplicate class ({})".format(t_cls_name))
            cls.__registry__[type_name].update({t_cls_name: t_cls})
        return t_cls

 	@classmethod
    def get_cls(cls, type_name, t_cls_name=None):
        """Get class and bind config to class.

        :param type_name: type name of class registry
        :param t_cls_name: class name
        :return:t_cls
        """
        if not cls.is_exists(type_name, t_cls_name):
            raise ValueError("can't find class type {} class name {} in class registry".format(type_name, t_cls_name))
        if t_cls_name is None:
            raise ValueError("can't find class. class type={}".format(type_name))
        t_cls = cls.__registry__.get(type_name).get(t_cls_name)
        return t_cls

Clean up and redesign the base Config class,  each feature  maintains it's specific variables, and ensures that developers can be manually updated the variables.
class ConfigSerializable(object):
    """Seriablizable config base class."""

    __original__value__ = None

    @property
    def __allattr__(self):
        attrs = filter(lambda attr: not (attr.startswith("__") or ismethod(getattr(self, attr))
                                         or isfunction(getattr(self, attr))), dir(self))
        return list(attrs)

    def update(self, **kwargs):
        for attr in self.__allattr__:
            if attr not in kwargs:
                continue
            setattr(self, attr, kwargs[attr])

    def to_json(self):
        """Serialize config to a dictionary. It will be very useful in distributed systems. """
        pass

    def __getitem__(self, item):
        return getattr(self, item, None)

    def get(self, item, default=""):
        return self.__getitem__(item) or default

    @classmethod
    def from_json(cls, data):
        """Restore config from a dictionary or a file."""
        pass

Refer to tools such as moxing、neuropod .etc, decouple the ML framework from the features of sedna, allows developers to choose their favorite framework.
Few frameworks which would be nice to support:

Scikit learn
Pytorch
MindSpore

Add a common file operation and a unified log-format in common module, use an abstract base class to standardize the  feature  modules' interface, and features are invoked by inheriting the base class.
Specifically, the goals are:

Clean up the base module's classes and package structure
Create a necessary amount of modules based on big features
Revise dependency structure for all the modules

Hi Joey
we need describe the positioning of sedna and lib firstly.

Hi Joey, it's very nice of you for seeing those problems, but I still have some questions:

Why does sedna need to support pre-processing and post-processing, while current frameworks are already  integrated with pre-processing function such as tf.keras.preprocessing?
I want to make sure this is just for sedna's code contributors instead of sedna's AI developer users?
It needs to point out what we can abstract from those machine learning frameworks.

Hi Joey, it's very nice of you for seeing those problems, but I still have some questions:

Why does sedna need to support pre-processing and post-processing, while current frameworks are already  integrated with pre-processing function such as tf.keras.preprocessing?
I want to make sure this is just for sedna's code contributors instead of sedna's AI developer users?
It needs to point out what we can abstract from those machine learning frameworks.

Thanks for reply!

Sedna has defined both pre-processing and post-processing in our model training and inference ( clean floder、start rest server 、LC report etc），We would like to be able to do something else between these jobs and model training/inference.
As a deployer, its too waste time to figuring out each variable in global variables, as we're not sure whether a variable is necessary for the feature we're interested in.
Yep! In fact, features of  sedna lib should be framework-agnostic, it should work natively with PyTorch, TensorFlow, and other. For example, model loading, dump_model, train, inference, data_input, and output  etc. should be consistent. Code contributors can develop new features based on the same style.

Rethinking after our Weekly Community Meeting [2021-04-29]，Contributor Experience has been discussing the lib refactoring of sedna and looking into how to make the lib more efficient and maintainable. To summarize the discussions:

Sedna formulate policies based on the variables defined in config，some of them should not be maintained by the deployer but by the developer ( hard_example_mind_algorithms、Aggregation_algorithm_in_Federated_learning etc.), We should make it clear to developers what input parameters they need to maintain.
As our lib is ML-framework-agnostic, developers can define their model (loading, dump_model, train, inference) call methods, and sedna invokes corresponding methods to implement functions when required. Of course, sedna can also consider performing operations according to the specified ML framework (train -> keras.model.fit) .
I previously thought that the interfaces of each feature should be unified, but because each feature has different objectives, it should be differentiated.
As a AI framework of Edge-cloud synergy, lib should open the interfaces for developers access , such as loss functions, optimization functions, and HEM/AGG algorithms etc.

This issue is a spot for further discussion.

Rethinking after our Weekly Community Meeting [2021-04-29]，Contributor Experience has been discussing the lib refactoring of sedna and looking into how to make the lib more efficient and maintainable. To summarize the discussions:

Sedna formulate policies based on the variables defined in config，some of them should not be maintained by the deployer but by the developer ( hard_example_mind_algorithms、Aggregation_algorithm_in_Federated_learning etc.), We should make it clear to developers what input parameters they need to maintain.
As our lib is ML-framework-agnostic, developers can define their model (loading, dump_model, train, inference) call methods, and sedna invokes corresponding methods to implement functions when required. Of course, sedna can also consider performing operations according to the specified ML framework (train -> keras.model.fit) .
I previously thought that the interfaces of each feature should be unified, but because each feature has different objectives, it should be differentiated.
As a AI framework of Edge-cloud synergy, lib should open the interfaces for developers access , such as loss functions, optimization functions, and HEM/AGG algorithms etc.

This issue is a spot for further discussion.

Examples are as follows:
`### start a api server（for example: rest server in join-inference and ws server in federated-learning）
def run_server():
job = FederatedLearning()
server = AggregationServer(
    servername=job.job_name,
    host=job.agg_ip,
    http_port=job.agg_port
)
server.start()

def fl_train():
fl_instance = FederatedLearning(model=CNNModel, # like keras.models
agg="FedAvg"  # define the agg algorithms
)
# load dataset. data pre-process
train_data, test_data = TxtDataParse(func=image_process)(
    train_dataset_url, test_dataset_url
)
x = np.array([tup[0] for tup in train_data])
y = np.array([tup[1] for tup in train_data])
test_x = np.array([tup[0] for tup in test_data])

model = fl_instance.train(x, y, # train ddataset
                  loss='FederatedLoss', # like keras.losses
                  metrics=categorical_accuracy, # keras.metrics
                  optimizer=Adam(learning_rate=0.01), # keras.optimizers
                 )
pred = model.inference(test_data, post_process=infer_rsl_process)
return pred

if name == 'main':
run_server()
`

fixed by #72
/close
feel free to reopen this


Repository tree binding issue

Hello
When we bind repository tree i am facing a problem. In your sample code the tree is binding first and after commitemessage and commitedate binding.
But the problem is if the tree content lots of record then the commitemessage and commitedate binding too much late.
I need to bind tree data with commitemesage in first load.
Code is following below
This the function for details calling. Is it possible to minimize the loop. Because the anchester count is 24777 for a repository.
And entries count is 10(for each page).
I want to filter the ancestors list with matching data for entries.
Please have a look.
private IEnumerable GetTreeModelsWithDetails(Commit commit, IEnumerable tree, string referenceName)
{
var ancestors = _repository.Commits.QueryBy(new CommitFilter { Since = commit, SortBy = CommitSortStrategies.Topological | CommitSortStrategies.Reverse }).ToList();
var entries = tree.ToList();
var result = new List();
        for (int i = 0; i < ancestors.Count && entries.Any(); i++)
        {
            var ancestor = ancestors[i];

            for (int j = 0; j < entries.Count; j++)
            {
                var entry = entries[j];
                var ancestorEntry = ancestor[entry.Path];
                if (ancestorEntry != null && entry.Target.Sha == ancestorEntry.Target.Sha)
                {
                    result.Add(CreateRepositoryDetailModel(entry, ancestor, referenceName));
                    entries[j] = null;
                }
            }

            entries.RemoveAll(entry => entry == null);
        }

        return result;
    }

@ayanbhowmick Can you try make a PR with the suggested changes, then it might be easier to see what changes you would like to have?


enhancement

This pull request aims to improve project reproducibility and fix some typos.
Changes proposed

🔧 exact versioning for requirements.txt

this allow other contributor to reproduce this project without focusing on which version of libraries to be used.

🎨 fix typos & wrap code into function

name-main idiom just a guard block for avoid unintended invoking script
we might reuse this search function in other use cases later.

📝 update README.md (Just update some typos)

Note to reviewers

It would be great to update the usage image. 😉
Thank you.

Code of Conduct

[x] By submitting this pull request, I confirm I've read and complied with the CoC 🖖

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

You can also join our Discord community.
Feel free to check out other cool repositories of the Thecyberworld.
Join the Thecyberworld GitHub Organisation by raising an issue (you will be sent an invitation).

Amazing work @Kawaeee


Ignore @Auth parameters in dropwizard

In dropwizard, the authenticated principal is passed to a resource method as a parameter annotated with @Auth, which should be ignored for documentation purposes. I am not yet sure how this could/should be done in a general manner by the doclet, but it would be nice.

Yea I agree this is very annoying especially when dealing with JSON Web token authentication.
Here's a screenshot:


Correction for raw affiliation School of Engineering and Computing, De Montfort University, Dubai, United Arab Emirates,

Correction needed for raw affiliation School of Engineering and Computing, De Montfort University, Dubai, United Arab Emirates,
raw_affiliation_name: School of Engineering and Computing, De Montfort University, Dubai, United Arab Emirates,
new_rors: 0312pnr83
previous_rors:
works_examples: W4390756936;W4366502700
contact: 153ce9d63f1fc19c320c282038ead7d1:624622 @ dmu.ac.uk

This issue was accepted and ingested by the OpenAlex team on 2024-12-19. The new affiliations should be visible within the next 7 days.


Feature request: add flash command

Hi,
I'd like to request a new command to be added "flash" that would allow to flash pending arrays data without committing a transaction.
We'd like to emulate insert and read inserted data in the same transaction and have a single commit at the end.
Currently there is a need to commit transaction after insert operation to force flushing pending arrays data from the client.

That should not be too hard to implement, so this is something I will consider.

Done. It will be available in the up-coming 3.0.1 release. If you are compiling yourself, download the latest master branch and compile.


"Update" and "new adapters" for multihost systems

Add feature for multihost systems.

created by ioBroker.info

not sure but is this related to a future request to collect slave information, too?
If not would it be possible to create a second instance on a slave host?

This adapter has been deprecated.
So most certainly this this issue will not be solved anymore


CloudVolumeRepository.java is causing issue in API project

below line in CloudVolumeRepository.java seems to be causing issues in API service start up.
Collection findByAttachInstancesIn(List attachInstances);
Error I was getting was 'property attach not found'. It seems to be Spring Data JPA related and I had to add this annotation to temporarily satisfy the build, not sure if the query result is same.
@Query(value = "{ 'attachInstances' : {$in: ?0 }}")
Anyone else has this issue?

that's odd!

Can you paste a little more of the stack trace?

reverted my change today to reproduce the issue and give you stack trace but now I do not see it. closing this for now and will watch out.


JAVA-5947 - Removed the large Geo Location File - GeoLite2-City.mmdb from code an…

…d it's references.

JAVA-5947 - Removed the large Geo Location File - GeoLite2-City.mmdb from code and updated the tests with comment to download file and run manual tests


Create a Pull Request Template for this repository

Please make yourself familiar with the types of issues and pages in this repository.
The pull request template will help identify related issues or other important factors before review.
Research may be neccessary. The final submission should be a working file that applies out the box.
There should be an option for:
Issue Related [] (if so, what issue #)
General Contribution[] (a)page (b)theme (c)debugging (e)enhancement
More information here:
https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/creating-a-pull-request-template-for-your-repository
Read the Docs.
Give a ⭐
Don't break anything. :)

Please assign it to me.

@Starkuser24 you are assigned.

Thank you @omicreativedev, I have made the PR accordingly


[T2A4][T16-A2]Xie Jihui

Ready for review

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.


Conversion to python3

This should be a working python3 port.

Would you like to take over the maintenance of the library?

Yes :-)

Great! Please give you PyPi username and I'll give you admin rights there. Also if you need any help how to run tests, etc. please let me know.

I contacted Arkadiusz how we should do this.

Hi
My pypi name is davsebamse
I am working getting the tests to run using python3.
On Fri, Dec 4, 2015 at 4:40 PM, Mikko Ohtamaa<EMAIL_ADDRESS>wrote:

I contacted Arkadiusz how we should do this.
—
Reply to this email directly or view it on GitHub
https://github.com/awahlig/skype4py/pull/58#issuecomment-161999084.

--
Master Wizz. David Askirk Fotel
Hvilebækvænge 32
3520 Farum
http://davsebamse.dk
Twitter: @davsebamse
Email<EMAIL_ADDRESS>Skype: davsebamse
12884901877 er et primtal

Thanks! Working on settings Skype4Py organization up.

@davsebamse You have now admin rights here https://github.com/Skype4Py/Skype4Py - can you do the pull request there.


Map properties based on other property (gRPC)

Is your feature request related to a problem? Please describe.
We are using Mapperly to map between hand-crafted objects and objects auto-generated from .proto files.
These auto-generated classes have one very special handling:
When a property is defined optional in the proto file, the C# property is actually not nullable (as we would have expected) but generates a HasX property.
Example:
message MyObject {
    optional string Name;
}

generates roughly this class
class MyObject
{
    public string name_;
    public string Name
    {
        get => name_ ?? "";
        set => _name = value ?? throw new ArgumentNullException(nameof(Name));
    }
    public bool HasName => _name != null;
}

So from the generated classes view, the public property ALWAYS has a value (in this case empty string, but the same concepts apply to things like int? double? and so on.
Now, where does this get me to:
If we were to simply let Mapperly generate the mappings, we would end up in a situation, where the initial null value is a empty string in the target object.
In earlier versions of mapperly we even saw the Exceptions in the generated proto C# code, but this was fixable through AllowNullPropertyAssignment = false (thanks for that btw).
Describe the solution you'd like
See next point, I'm honestly not sure about what solution I'd like. Really open to discussion here.
Describe alternatives you've considered

One option would be, that Mapperly "somehow" allows to reflect this logic like "if there is a HasX property for the property X and it returns bool, we check that. Would absolutely solve our usecase, but I do understand, that mapperly probably should / can not support specifics from other code generated codes (on the other hand, the gRPC implementation might be big enough to still consider it?)
Can we somehow emit custom code into the source generation? That would possibly allow us to implement this special logic ourself or even provide it as a integration nuget package between mapperly and gRPC for .NET
We understand that we can solve the problem by using https://mapperly.riok.app/docs/configuration/before-after-map/, but this would force us to implement a custom method for almost 90% of the classes and change 100's of properties after the initial mapping.
Also this feels very error-prone, as new properties might be added to the proto files and ppl miss to add them in the after map code.

Additional context

We would have loved to never have this problem and for the gRPC implementation to simply generate nullable properties for optional fields. There was quite some discussion on that matter over here https://github.com/protocolbuffers/protobuf/issues/9083 but it was eventually closed as the breaking changes can not be considered anymore.
We are aware that google offers proto Wrappers like StringValue or Int32Value which actually generate nullabe properties in C#. While this would solve the problem, it comes with two other problems

The proto files must be considered as is
Even if we could modify the proto files, not using the language defaults just because the code generation is not as we want it seems to be very bad practice.

So as said already, we are not sure what the best solution would. Very open to ideas and discussion!
Thanks,
Philipp

I completely understand your problem. We have already discussed whether Mapperly should provide a way to fix wrong or missing nullable annotations in 3rd party code in other issues / discussions, IMO the same applies here.

To keep the Mapperly implementation maintainable, it needs to focus on its main goals. Fixing the interpretation of wrong/missing nullable annotations or alternative semantics for expressing nullability is not one of Mapperly's tasks. Mapperly implements the handling of nullability using the standard C# nullable annotations.
Supporting Has* and Clear* would add a lot of complexity to Mapperly just to support one specific use case (namely protoc-generated source code, I don't know of any other library that uses Has* and Clear* notation for nullable properties). We try to keep the Mapperly implementation maintainable by focusing on its main goals.
AFAIK injecting custom code into the source generator at compile time is not possible in an efficient and reliable way with good DX. Let me know if you know of a way to achieve injecting custom code into a source generator while maintaining great performance and a nice DX 😊.

I feared a little that no real solution will be available, tough I can understand it.

I don't know of any other library that uses Has* and Clear* notation for nullable properties).
Microsoft is (*was) doing similar stuff in different areas, i.e. XmlSerialization with the *Specified notation or the ShouldSerialize* and Reset* notation like here https://learn.microsoft.com/en-us/dotnet/desktop/winforms/controls-design/how-to-designer-properties-shouldserialize-reset?view=netdesktop-8.0

In the end we might end up doing some (cached) reflection stuff for now in after maps

The problem is that before nullable reference types, there was no standard notation for how to handle this scenario. So there are a lot of different notations (you just made me aware of three; protoc, xml, WinForms, but I think there might be more). This would make the Mapperly implementation complex, while still not being able to support all scenarios.
I'm aware that the situation is far from optimal, but IMO the way to improve it is to adapt the nullable reference types in the libraries & code generators in question.
I'll close this for now, as we currently have no plans to improve the situation on Mapperly's end. We may revisit this if there is a lot of demand for it.


Parameter change does not make sound.

After updating to 1.4.1 I noticed that my copter doesn't beep when I change a parameter. Was that on purpose? I found it incredibly useful to confirm the parameter changes.

@LorenzMeier was this intentional? If not I'll look into fixing it.

@agribov yes this was an intentional change. QGC should let you know if a param change fails.


#2685 - Import Provincial Restrictions From SFAS to SIMS - Part 2 (e2e tests)

As a part of this PR, the following e2e tests were written:

Should save only one legacy restriction for the student if one or more restrictions are imported from SFAS which do not have a mapping to the SIMS restrictions.
Should not insert a restriction to SIMS student restrictions if the same restriction is imported from SFAS but is already active in SIMS, Should not insert a legacy restriction if a legacy restriction is already active in SIMS for the student, Should insert a restriction after mapping it to SIMS restriction if it is either inactive or is not at all present in the SIMS student restrictions, Should not insert a restriction if it has a restriction resolution date.

Screenshot:

Nice work @sh16011993 . Please have a look at the comments.

Thanks for making the changes @sh16011993 added more comments.


Copy Component

@richa I forgot to mention one detail missing, it's also required to include this component module in https://github.com/x-team/auto-ui/blob/master/src/index.js :)

@richa Is this ready for review, as i can see you're still committing things... If so, attach ready for review label pls or use WIP:  title prefix :)

@nicksp Sure I'll keep that in mind. Actually the PR was approved and after that I saw conflicts after latest merge. Today, I see another conflict here :)

thanks @richa .
If the only conflict is lib/auto-ui.js, don't worry about it - we can rebuild that file from scratch after merging.

Looks good, merging now.


liboauthcpp.cpp bool Client::getSignature(..., bool do_urlencode)

shouldn't getSignature have a do_urlencode?
now it always do
oAuthSignature = urlencode( base64Str );
in the end.. is that really correct?
what if I use it in a http header? .. then I guess it shouldn't be urlencoded?

Closing due to inactivity, if someone can pin down the issue and wants to reopen, feel free.


What is the best way to use NTP in this library?

Hello,
I managed to use NTP with the help of this other library:
https://github.com/arduino-libraries/NTPClient
rtc.setTime(timeClient.getEpochTime());

You don't need libraries on ESP32 to handle the NTP requests. The Arduino Framework (Espressiff), is built with this support and internal libraries. For example I did this eInk clock using it 3 years ago:
https://github.com/hpsaturn/m5coreink-ntp-clock


dnsmasq manage forwarder/address/host-record

Add task files to the dnsmasq role to manage forwarder, address and host-record config to the running dnsmasq server.
As a pull request owner and reviewers, I checked that:

[x] Appropriate testing is done and actually running
[x] Appropriate documentation exists and/or is up-to-date:

[x] README in the role
[x] Content of the docs/source is reflecting the changes

Hey @hjensas, thanks for the PR!
Since you have implemented forwarder and addresses configurations through new taskfiles and vars, care to remove dns.yml and its vars?

/approve


RFID Reader Dummy Functions

[ ] Create dummy functions
[ ] Add calls to dummy functions in demo file
[ ] Check that demo file runs

(1/2 hour)

@wecodewithmichaelmccloskey please review

Looks good


Remove azorius typechain

Please review and merge https://github.com/decentdao/fractal-interface/pull/1670 first.

@mudrila fixed both of your latest comments and pushed


Specify and/or upgrade KaTeX version

First, let me start off by stating how much I like both goldmark and this extension -- I am making a static site generator, and it's a blast thanks to this!
I ran into a small issue where KaTeX formulas seem to be rendered with the wrong font after migrating my blog. Looking at the diff between the previous version, I see that goldmark-qjs-katex renders the math with  whereas it would use  previously. A quick search suggests that this style has changed in some version of KaTeX.
This makes sense as I just included the same KaTeX stylesheet as I used before, without paying attention to the version. I would think that it would work if I include the right version of the KaTeX stylesheet. However, I'm not sure where I can find the version that is used in goldmark-qjs-katex. It might be good to specify it in the README so that people can include the right version.
It might also be desirable to upgrade the KaTeX version used. I'll give this a try.

I have two thoughts on this. One is I'm happy to merge the PR if you get the tests working. The tests are mostly to make sure goldmark handles switching between markdown and latex in a nice way, and making any modifications to the goldmark extension itself is where that might break: that's the case those tests are supposed to be helping you with. Updating KaTeX, it's just extra busywork. So even though gen_test.go is a generated thing it says not to edit, it might be easiest to just modify the test cases in there to use the new class name. If that works, that's fine.
In the worse case here it might turn out the test generator is relying on KaTeX behaviour that has changed in the new version and it may fail tests that should pass (the normalize function in the tests is meant to handle that kind of thing). But again my feeling is if that's the case, the easiest course of action will be a big find and replace to update the generated test cases
Second thing is I made this so I could throw it into Hugo and they've recently added the same functionality themselves using wasm instead of qjs. I don't know if this is more or less hassle than what you're trying here but what they're doing might be worth a look at

Thanks for your fast reply!
Regarding looking at Hugo: I have tried to use it before but didn't like it too much at the time. Since I have invested a bit in goldmark and the qjs-katex extension now, and they seem to work great for my purposes I don't intend to look at other options now.
I worked on https://github.com/graemephi/goldmark-qjs-katex/pull/9 a bit. It took a bit longer than expected but the tests pass now and I don't plan to make more changes to the PR unless you have comments.
I had to manually replace pandocs output for some cases. After manually replacing those testcases I don't see a change in the PR diff for them, so I figure these tests were only passing before because they were manually edited in gen_test.go as well. (Maybe the DO NOT EDIT message should go, or the failing tests should be moved to a different test file which is not autogenerated?).


Filter additional OS X files from zip.

closes #3401
Why was this change made? 🤔
Per @amyehodge
How was this change tested? 🤨
⚡ ⚠ If this change involves consuming from or writing to another service (or shared file system), run integration test create_object_h2_spec.rb and/or test manually in [stage|qa] environment, in addition to specs. ⚡
Unit
Does your change introduce accessibility violations? 🩺
⚡ ⚠ Please ensure this change does not introduce accessibility violations (at the WCAG A or AA conformance levels); if it does, include a rationale. See the Infrastructure accessibility guide for more detail. ⚡

@justinlittman Where does this work stand? Anything I can do to help as current FR?

@mjgiarlo I just undrafted, so ready for review.


⚠ T:Riza Api has degraded performance

In fbd98f0, T:Riza Api (https://infra.triza.dev/api) experienced degraded performance:

HTTP code: 200
Response time: 5591 ms

Resolved: T:Riza Api performance has improved in 0d572e0.


Add infinite function for floating point numbers

I think having this function could be good because using finite and nan to check infinity can add complexity to the code and is quite obscure to people unfamiliar with floating point numbers.

Thanks!


[ADD]: BASKET SIGN UP FORM PROJECT 💡

Close #2696
Checklist:

[x] I have mentioned the issue number in my Pull Request.
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have created a helpful and easy to understand README.md

Please merge this PR @TusharKesarwani, @pranjay-poddar, @Saurabh-kumar09 & @MohitGupta121.


New version: Tencent.TencentDocs version 3.1.9

[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?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.5 schema?

Microsoft Reviewers: Open in CodeFlow

Hi @SpecterShell,
I can't navigate from the PackageUrl to the InstallerUrl. This version is newer than the version on the ISV site. Is the link to this package in a different location?

It was obtained from their update API. You can find the script here: https://github.com/SpecterShell/Dumplings/tree/main/Tasks/Tencent.TencentDocs

Oh - need to be able to navigate from the PackageUrl to the InstallerUrl, and the PackageUrl has to be a full URL that supports the GET verb.

Oh - need to be able to navigate from the PackageUrl to the InstallerUrl, and the PackageUrl has to be a full URL that supports the GET verb.

The request to API for getting the InstallerUrl is POST, and the InstallerUrl obtained is GET.


HDDS-2078. Get/Renew DelegationToken NPE after HDDS-1909. Contributed…

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
1117
Docker mode activated.

_ Prechecks _

+1
dupname
0
No case conflicting files found.

+1
@author
0
The patch does not contain any @author tags.

-1
test4tests
0
The patch doesn't appear to include any new or modified tests.  Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1
mvninstall
30
hadoop-ozone in trunk failed.

-1
compile
23
hadoop-ozone in trunk failed.

+1
checkstyle
48
trunk passed

+1
mvnsite
0
trunk passed

+1
shadedclient
808
branch has no errors when building and testing our client artifacts.

-1
javadoc
17
hadoop-hdds in trunk failed.

-1
javadoc
18
hadoop-ozone in trunk failed.

0
spotbugs
148
Used deprecated FindBugs config; considering switching to SpotBugs.

-1
findbugs
25
hadoop-ozone in trunk failed.

_ Patch Compile Tests _

-1
mvninstall
32
hadoop-ozone in the patch failed.

-1
compile
25
hadoop-ozone in the patch failed.

-1
javac
25
hadoop-ozone in the patch failed.

+1
checkstyle
55
the patch passed

+1
mvnsite
0
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedclient
665
patch has no errors when building and testing our client artifacts.

-1
javadoc
18
hadoop-hdds in the patch failed.

-1
javadoc
18
hadoop-ozone in the patch failed.

-1
findbugs
27
hadoop-ozone in the patch failed.

_ Other Tests _

-1
unit
130
hadoop-hdds in the patch failed.

-1
unit
29
hadoop-ozone in the patch failed.

+1
asflicense
32
The patch does not generate ASF License warnings.

3815

Reason
Tests

Failed junit tests
hadoop.ozone.container.keyvalue.TestKeyValueContainer

hadoop.ozone.container.ozoneimpl.TestOzoneContainer

Subsystem
Report/Notes

Docker
Client=19.03.1 Server=19.03.1 base: https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/1444

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle

uname
Linux 67e4ed679fe3 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
personality/hadoop.sh

git revision
trunk / 6a9f7ca

Default Java
1.8.0_222

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/branch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/branch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/branch-javadoc-hadoop-hdds.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/branch-javadoc-hadoop-ozone.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/branch-findbugs-hadoop-ozone.txt

mvninstall
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-mvninstall-hadoop-ozone.txt

compile
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-compile-hadoop-ozone.txt

javac
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-compile-hadoop-ozone.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-javadoc-hadoop-hdds.txt

javadoc
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-javadoc-hadoop-ozone.txt

findbugs
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-findbugs-hadoop-ozone.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-unit-hadoop-hdds.txt

unit
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/artifact/out/patch-unit-hadoop-ozone.txt

Test Results
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/testReport/

Max. process+thread count
411 (vs. ulimit of 5500)

modules
C: hadoop-ozone/ozone-manager U: hadoop-ozone/ozone-manager

Console output
https://builds.apache.org/job/hadoop-multibranch/job/PR-1444/1/console

versions
git=2.7.4 maven=3.3.9 findbugs=3.1.0-RC1

Powered by
Apache Yetus 0.10.0 http://yetus.apache.org

This message was automatically generated.

@xiaoyuyao I think all these failures are due to the trunk issues. If you can retest, it might be a good idea.

/retest


Added new metatraffic attribute for entities [12534]

A new metatraffic attribute is included in Entities, which is true when related to topics used for sharing metatraffic data. This is useful for filtering by its value, being then able to hide/display entities used for metatraffic purposes in FastDDS-Monitor.

Corresponding PR for this feature in Fast-DDS-monitor:
https://github.com/eProsima/Fast-DDS-monitor/pull/122

It would be nice to add the Statistics metatraffic DataWriters. Currently the DataReaders are being seen in the graph (for example, launching two monitors) but not the DataWriters because they are being filtered.
Now that there is a way to hide them, it would be nice to be able to see them, in order to debug which endpoints has which statistics topics.


Typechecking not enabled by default is not clearly indicated in the docs

I'm new here, and I spent quite some time to figure out that typechecking is not enabled by default since version 1.23.
Please check out this discussion for what I think might be a common question for begginers:
https://stackoverflow.com/questions/72812645/how-to-activate-typescript-verification-of-types/72812958?noredirect=1#comment128609270_72812958
I couldn't easily find about the use of --check flag on the documentation.
So I started to spend a lot of time on how to create a tsconfig.json, then I started to wonder about errors in the linter VSCode is using, and sundelly hours have passed.
Thus I believe maybe that should be an important thing to stress for new learners..

Thanks, it seems docs got out of date on this topic. I'll make sure to update them next week.

Glad to be useful! Please let me know if I can contribute further anyhow.

@CavalcanteLucas if you want to contribute, it'd be great if these two sections from the blog posts could be incorporated into the manual:

https://deno.com/blog/v1.22#default-type-checking-behavior-has-been-updated
https://deno.com/blog/v1.23#no-type-checking-by-default

Of course, I'd be glad to do it!
Please allow me to read those posts carefully though.

Question:
Has the impact of the environment variable DENO_FUTURE_CHECK been changed?

Question:
Has the impact of the environment variable DENO_FUTURE_CHECK been changed because of these updates?

@CavalcanteLucas It no longer has an effect on deno run now that type-checking is disabled without the --check argument.

I believe I should close this Issue, as PR#357 has been merged.


Implement something for adding random points within a polygon

Created originally by @saleiva in https://github.com/Vizzuality/cartodb on 2011-11-05T21:38:51Z

Please re-consider implementing this. It would be wonderful to be able to visualise graduations of several  variables within a single polygon.
For inspiration on why CartoDB should do this, see, all, this, beauty:
https://www.flickr.com/photos/walkingsf/sets/72157624812674967/with/4982024006/
http://flowingdata.com/2010/12/16/mapping-demographics-of-every-block-and-city-in-america/
http://flowingdata.com/2010/09/20/race-and-ethnicity-by-mapped-by-block/
I love how districts can blend into each other, but at othertimes have surprisingly hard edges. It picks out social divides, heterogeneity and also gradients over space.
This should basically be a core visualisation in CartoDB. It's huge, and beautiful.
ps. Now imagine what it would mean for this to be animated


Error from listBlobFlatSegment

Package Name: @azure/storage-blob
Package Version: 10.5.0
Operating system: Linux
[X] nodejs

version: 10.15.0

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
I'm just getting started with the module (so it may be user error) and I have a basic test program trying to list blobs in a container but I get a node.js error when calling listBlobFlatSegment.
To Reproduce
Steps to reproduce the behavior:

Rename the attached file as aztest.js
Edit the file and fill in the credentials
Run "node aztest.js"

aztest.js.txt
Expected behavior
A list of blobs in the container. In my case, likely empty because I've never created one.
Screenshots
The error output I get is below.
Exception this._requestPolicyFactories[i].create is not a function
TypeError: this._requestPolicyFactories[i].create is not a function
at StorageClientContext.ServiceClient.sendRequest (/myproject/node_modules/@azure/ms-rest-js/dist/msRest.node.js:4389:64)
at StorageClientContext.ServiceClient.sendOperationRequest (/myproject/node_modules/@azure/ms-rest-js/dist/msRest.node.js:4514:27)
at Container.listBlobFlatSegment (/myproject/node_modules/@azure/storage-blob/dist/index.js:9022:28)
at ContainerURL. (/myproject/node_modules/@azure/storage-blob/dist/index.js:14804:61)
at step (/myproject/node_modules/tslib/tslib.js:136:27)
at Object.next (/myproject/node_modules/tslib/tslib.js:117:57)
at /myproject/node_modules/tslib/tslib.js:110:75
at new Promise ()
at Object.__awaiter (/myproject/node_modules/tslib/tslib.js:106:16)
at ContainerURL.listBlobFlatSegment (/myproject/node_modules/@azure/storage-blob/dist/index.js:14802:24)

Pasting the relevant code snippet here.
'use strict';

const Azure = require( '@azure/storage-blob' );
const credentials = {
    'sas_token': '...',
    'container_name': '...',
    'container_uri': '...',
    'region': '...',
    'account_name': '...'
};

async function go() {
    let cred = Azure.TokenCredential( credentials.sas_token );
    let pipeline = Azure.StorageURL.newPipeline( cred );
...

TokenCredential takes an OAuth token, not a SAS token. Here's some more information about Azure OAuth2 code flow.
We also realize the gap in our documentation and samples and have been working on improving them in the next preview version of the v12 libraries. See this work item for example.
Please let us know if you have further questions.

So given the credential properties that I have in that object, what Credential object should I be using? I can't use SharedKeyCredential since I don't have a key, and AnonymousCredential gives the same error as TokenCredential.

Excellent, that is working. Thank you


nuclei: 3.2.4 -> 3.2.5

Diff: https://github.com/projectdiscovery/nuclei/compare/v3.2.4...v3.2.5
Changelog: https://github.com/projectdiscovery/nuclei/releases/tag/v3.2.5
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 306778 run on x86_64-linux 1

1 package built:

nuclei


Documentation is not being published

Dokka is failing when parsing the Dashboard (useEffect, particularly). See: https://github.com/Doist/ffs/runs/4922080640

Reported upstream: https://github.com/Kotlin/dokka/issues/2327


dual trigger component did mount after server side render

Hi,
after server side render in the browser component event 'componentDidMount' is fired twice. I found this in widgets container after i moved data fetching to client only, the request for widget load is fired twice.
I have added this code:
componentDidMount() {
    console.log('mounted');
    if (!isLoaded(this.props)) {
        console.log((new Error).stack);
        return this.props.load();
    }
}

I also added console log of action type in one reducer to see actions triggered. After server-side render in browser console i see this:
@@redux/INIT

auth.js:16 @@redux/PROBE_UNKNOWN_ACTION_9.5.k.p.c.k.p.4.x.6.r

auth.js:16 @@INIT

auth.js:16 @@reduxReactRouter/routerDidChange

auth.js:16 @@reduxReactRouter/initRoutes

Widgets.js:37 mounted

Widgets.js:38 Error
    at Widgets.componentDidMount (Widgets.js:38)
    at Widgets.proxiedComponentDidMount [as componentDidMount] (createPrototypeProxy.js:61)
    at CallbackQueue.assign.notifyAll (CallbackQueue.js:65)
    at ReactReconcileTransaction.ON_DOM_READY_QUEUEING.close (ReactReconcileTransaction.js:81)
    at ReactReconcileTransaction.Mixin.closeAll (Transaction.js:202)
    at ReactReconcileTransaction.Mixin.perform (Transaction.js:149)
    at batchedMountComponentIntoNode (ReactMount.js:282)
    at ReactDefaultBatchingStrategyTransaction.Mixin.perform (Transaction.js:136)
    at Object.ReactDefaultBatchingStrategy.batchedUpdates (ReactDefaultBatchingStrategy.js:62)
    at Object.batchedUpdates (ReactUpdates.js:94)

auth.js:16 redux-example/widgets/LOAD

auth.js:16 @@reduxReactRouter/initRoutes

Widgets.js:37 mounted

Widgets.js:38 Error
    at Widgets.componentDidMount (Widgets.js:38)
    at Widgets.proxiedComponentDidMount [as componentDidMount] (createPrototypeProxy.js:61)
    at CallbackQueue.assign.notifyAll (CallbackQueue.js:65)
    at ReactReconcileTransaction.ON_DOM_READY_QUEUEING.close (ReactReconcileTransaction.js:81)
    at ReactReconcileTransaction.Mixin.closeAll (Transaction.js:202)
    at ReactReconcileTransaction.Mixin.perform (Transaction.js:149)
    at ReactUpdatesFlushTransaction.Mixin.perform (Transaction.js:136)
    at ReactUpdatesFlushTransaction.assign.perform (ReactUpdates.js:86)
    at Object.flushBatchedUpdates (ReactUpdates.js:147)
    at Object.wrapper [as flushBatchedUpdates] (ReactPerf.js:66)

auth.js:16 redux-example/widgets/LOAD

client.js?6a8d:53 [HMR] connected

client.js:30 Object {msg: "'Hello World!' from server"}

auth.js:16 redux-example/widgets/LOAD_SUCCESS

client.js:1036 GET http://localhost:8080/api/widget/load/param1/param2 500 (Internal Server Error)Request.end @ client.js:1036(anonymous function) @ ApiClient.js:40Promise @ es6.promise.js:197exports.(anonymous function).target.(anonymous function).function.target.(anonymous function).F @ $.export.js:30_this.(anonymous function) @ ApiClient.js:25promise @ widgets.js:92(anonymous function) @ clientMiddleware.js:15(anonymous function) @ historyMiddleware.js:24(anonymous function) @ replaceRoutesMiddleware.js:16(anonymous function) @ bindActionCreators.js:14componentDidMount @ Widgets.js:40proxiedComponentDidMount @ createPrototypeProxy.js:61assign.notifyAll @ CallbackQueue.js:65ON_DOM_READY_QUEUEING.close @ ReactReconcileTransaction.js:81Mixin.closeAll @ Transaction.js:202Mixin.perform @ Transaction.js:149Mixin.perform @ Transaction.js:136assign.perform @ ReactUpdates.js:86flushBatchedUpdates @ ReactUpdates.js:147wrapper @ ReactPerf.js:66Mixin.closeAll @ Transaction.js:202Mixin.perform @ Transaction.js:149ReactDefaultBatchingStrategy.batchedUpdates @ ReactDefaultBatchingStrategy.js:62enqueueUpdate @ ReactUpdates.js:176enqueueUpdate @ ReactUpdateQueue.js:24ReactUpdateQueue.enqueueElementInternal @ ReactUpdateQueue.js:252(anonymous function) @ ReactMount.js:425ReactMount.scrollMonitor @ ReactMount.js:413ReactMount._updateRootComponent @ ReactMount.js:424ReactMount._renderSubtreeIntoContainer @ ReactMount.js:523ReactMount.render @ ReactMount.js:570wrapper @ ReactPerf.js:66(anonymous function) @ client.js:63(anonymous function) @ client.js:71__webpack_require__ @ bootstrap 58dc4de70529c4edadcb:520fn @ bootstrap 58dc4de70529c4edadcb:75(anonymous function) @ bootstrap 58dc4de70529c4edadcb:543__webpack_require__ @ bootstrap 58dc4de70529c4edadcb:520(anonymous function) @ bootstrap 58dc4de70529c4edadcb:543(anonymous function) @ bootstrap 58dc4de70529c4edadcb:543

auth.js:16 redux-example/widgets/LOAD_FAIL

As you can see, after routerDidChange, multiple events are fired twice, even widget action LOAD with one result SUCCESS and one FAIL.
Have someone else this issue? Is there some solution? Thanks

Issue is visible in development environment

This has come up before, take a look at #429 for some possible solutions.

I use Crome Redux Extension https://chrome.google.com/webstore/detail/redux-devtools/lmhkpmbekcpmknklioeibfkpmmfibljd?hl=en ... May be it could be workaround?
It seems for me it is much cleaner way to integrate redux debug tools to the project

fixed with one of #429 possible solutions


Tests in datafusion-common failed

Describe the bug
After #7745, the tests in 'datafusion-common' didn't work.
To Reproduce
cargo test --package datafusion-common
Expected behavior
Pass all tests
Additional context
Related

#7745
#7930
#7929

I think this would be resolved by adding parquet into default feature of datafusion-common package.
[features]
default = ["parquet"]

I'm thoughtless to test every package individually. Maybe we should add some CI tests for packages?

I think this would be resolved by adding parquet into default feature of datafusion-common package.
[features]
default = ["parquet"]

I'm thoughtless to test every package individually. Maybe we should add some CI tests for packages?

I think so, it would be nice to test for packages. But I am not sure if it is necessary.

I'm thoughtless to test every package individually. Maybe we should add some CI tests for packages?

Perhaps we can extend this check:
https://github.com/apache/arrow-datafusion/blob/main/.github/workflows/rust.yml#L69


Use binary crate

cargo build --features vendored builds with prebiult repo's wx binary if specified for that target.
doesn't link currently Now it builds and run locally with help from @ancwrd1.
Adding GitHub Actions for these configuration will be another Issue and PR,

@ancwrd1 I think with this PR, you can use your vendored repo with --features use_binary flag.
And also it is able to use another binary crate by overriding concrete crates in consumer project's cargo configuration like:
[dependencies]
wx-x86_64-pc-windows-gnu = { git = "another/git/repo" }

This will support your suggested use case in #72? Any other feedbacks?
Thanks in advance.

Thanks for your work! I think the common feature name used by many popular crates is 'vendored' (or like 'vendored-wx', etc). For example it is used in native-tls, openssl, reqwest. But it's not too important.
I see that you have also added -msvc based binaries, that's nice! I will try it.

I think the common feature name used by many popular crates is 'vendored' (or like 'vendored-wx', etc).

OK, to rename to that.

I see that you have also added -msvc based binaries, that's nice! I will try it.

I'm afraid that it is not what you expected. The -msvc binary is wxWidgets official DLL distribution for MSVC which packaged in compatible way with your prebuilt crates. So, they are not static linked and requires you place DLLs in same directory of executables manually (at least, for now).
It's not hard to copy corresponding DLLs from -msvc to target build directory. I'll work on this in part of #34.

I'm afraid that it is not what you expected. The -msvc binary is wxWidgets official DLL distribution for MSVC which packaged in compatible way with your prebuilt crates. So, they are not static linked and requires you place DLLs in same directory of executables manually (at least, for now).

I've now added static manually built lean and mean MSVC repo as well:
https://github.com/ancwrd1/wx-x86_64-pc-windows-msvc.git
The following flags are needed for compilation: /std:c++14, /EHsc
/D_CRT_SECURE_NO_WARNINGS can be also specified to avoid warnings.

@ancwrd1 Thank you for providing static prebuilt crate.
I'll utilize that when I'll start writing unit tests for windows in future.


merge ufoLib into fontTools.ufoLib

Part of #1095

basically, I fetched from the upstream ufoLib's remote, moved files to Lib/fontTools, then removed all duplicate files (setup.py, etc.), and finally used git merge --allow-unrelated-histories to merge all git history from one repository to the other.
It's a lot of history, but the most relevant commits are the ones following https://github.com/fonttools/fonttools/pull/1335/commits/49c7bd1e1d08500c6ddb783df3ae61184982d89d, which is the last commit from ufoLib/master.

I've laso copied the *.rst files from the ufoLib/Documentation to the Doc/sources/ufoLib subfolder, but didn't also copy the Sphinx setup, since there is #1333 pending which overlaps with this.
I reorganized a few modules and moved them in other places where they make more sense, but kept shims that re-export all the symbols from the old module location, to make the transition easier.
These are:

ufoLib.etree => fontTools.misc.etree
ufoLib.plistlib => fontTools.misc.plistlib
ufoLib.pointPen => fontTools.pens.pointPen

Finally, I added extras_require to the setup.py script, to allow to specify the optional dependencies:
https://github.com/fonttools/fonttools/pull/1335/commits/783707644bfa0d92d924faea745e34f3c0555219
Currently, the optional dependencies are grouped under the following extras keys:
ufo, woff, lxml, unicode, interpolatable, symfont, type1
There is also a catch-all extra key, called simply all, that installs all the optional dependencies in one go.
Just a reminder: each of these strings can be added to the pip install fonttools[...] invocation (or when specifying fonttools as a dependency in one's setup.py). They go between the square brackets and are  separated by commas. E.g.:
pip install fonttools[ufo,lxml,unicode] will install fonttools plus all the optional dependencies needed to, in this case, read/write UFOs, using lxml (instead of ElementTree), and with unicodedata from current Unicode 11.

So, to make this clear, the core of fontTools (e.g. ttx, ttLib, etc.) stays dependency-free. But in order to use the new fontTools.ufoLib, one needs to install the fs module (aka pyfilesystem2), and the enum34 backport only if running under python2.7.
This is all specified under the [ufo] extra requirement, and the user only needs to remember to type pip install fonttools[ufo] iff they want to read/write UFOs with fontTools. Otherwise nothing changes for them.

Will update #1333 with the new docs and rebase source (incl. setup.py) on your new version here.  We can add the ufo extras requirement to the RTD settings file so that all of those new dependencies are pulled for remote doc builds.

@chrissimpkins thanks. let's first merge this then you can rebase on master

I will have time for a quick review mid-day tomorrow. Is that soon enough?

that's great, thank you :+1:

another legacy thing that I would love to get rid of, for the sake of consistency between lxml and ElementTree, is the custom xml declaration that uses double instead of single quotes.  Both lxml and ElementTree use single quotes:
<?xml version='1.0' encoding='UTF-8'?>

but in ufoLib we write it with double quotes because that's what it had always done.
This is the time to change these little annoying things.

@anthrotype I'm ok changing it, it's a one-time break to everyone's git histories. @miguelsousa you ok with such a thing? I know it's super annoying, but....

@anthrotype go ahead with all the cleanup/simplification/breakage. Now is the time.
After the merge, your UFO-breakage allowance will be freezed for 10 years 😄

Congrats @anthrotype!

Thanks to all the ufoLib contributors!


Automatically rename paired HTML/XML tag

A small but useful quality-of-life feature - when you rename one HTML/XML tag, automatically rename the paired HTML/XML tag. This functionality already exists in Visual Studio.
There is currently an extension for this, but it is buggy: https://marketplace.visualstudio.com/items?itemName=formulahendry.auto-rename-tag

duplicate of #47069


Allow tags to be added independently from filters

Is your feature request related to a problem? Please describe.
Well the problem stems from the fact that you can only add tags to requests as part of a filter. I would like to add a "requests" tag to all requests, but if I create a filter, I have to filter out some of the requests in order to add a tag.
Describe the solution you'd like
A separate section where you can add tags. Or at the very least, allow an "all" filter option, so that every request will go through that filter and then you can add the tag.
Describe alternatives you've considered
n/a
Additional context
n/a

Would this not make more sense to be part of the user profiles? Setting the default tags for each profile and then different tags are set for each user if needed, and can be overridden by filters?


lag test

@ANISH-GOTTAPU please break the PR if needed, but please note that we want following varieties:

Static LAG with raw traffic
Static LAG with BGP + traffic with auto mac learning
Dynamic LAG (LACP) with raw traffic
Dynamic LAG (LACP) with BGP + traffic with auto mac learning

Each LAG port should consist of 3 members (on both ends), min links should be 2 and we should validate the scenarios where traffic should flow correctly if one member goes down but does not flow correctly when two members go down.
There are other possible use cases, so let's review whatever LAG use cases we have as part of internal tests and log issues for them accordingly.

Stale PR


Update launching-the-exploit.md

All I had changed is adding a notice to update the system if Memory Pit fails. This is because I had run into this issue, updating solved the problem, and the problem had no info on this page. I just wanted to help out others who ran into this and had no idea what to do.

Memory Pit is compatible with all firmware versions, you probably just downloaded the wrong pit.bin (there's 3 different ones)

Memory Pit is compatible with all firmware versions, you probably just downloaded the wrong pit.bin (there's 3 different ones)

I don't see why this is necessary when Memory Pit is compatible with all versions. Also, system updates bad. Don't do it.
What we really should be adding here is double-checking your System Version and where to find it. Seems like that is missing out here. Ideally with a screenshot if possible.

I don't see why this is necessary when Memory Pit is compatible with all versions. Also, system updates bad. Don't do it.
What we really should be adding here is double-checking your System Version and where to find it. Seems like that is missing out here. Ideally with a screenshot if possible.

Put simply, I started on 1.4U. Starting Memory Pit (there appeared to be no region/version options, just pit.bin) by going to the SD card album froze the DSi before TWiLightMenu++ could load. After updating to 1.4.5U (which I knew from the start was a risky move), Memory Pit booted correctly. Maybe this problem is specific to me, but I wouldn’t know, for every other DSi I tried already had 1.4.5U.

Put simply, I started on 1.4U. Starting Memory Pit (there appeared to be no region/version options, just pit.bin) by going to the SD card album froze the DSi before TWiLightMenu++ could load. After updating to 1.4.5U (which I knew from the start was a risky move), Memory Pit booted correctly. Maybe this problem is specific to me, but I wouldn’t know, for every other DSi I tried already had 1.4.5U.

huh strange, I've personally used Memory Pit on a couple 1.4J consoles and it's well known to generally work on 1.4 so probably something odd with your console, if you got it used perhaps it had been downgraded incorrectly or something. It does occasionally have issues like that but usually either redownloading and recopying over the pit.bin or reformatting the SD card will fix it, so probably just got unlucky in some way, I'm actually pretty surprised updating fixed it though since 1.4 and 1.4.5 use the same version of memory pit.
(the three versions are https://dsi.cfw.guide/launching-the-exploit.html#memory-pit on the page fyi)

huh strange, I've personally used Memory Pit on a couple 1.4J consoles and it's well known to generally work on 1.4 so probably something odd with your console, if you got it used perhaps it had been downgraded incorrectly or something. It does occasionally have issues like that but usually either redownloading and recopying over the pit.bin or reformatting the SD card will fix it, so probably just got unlucky in some way, I'm actually pretty surprised updating fixed it though since 1.4 and 1.4.5 use the same version of memory pit.
(the three versions are https://dsi.cfw.guide/launching-the-exploit.html#memory-pit on the page fyi)

Something like this should actually be reported for bugs, I think. Obviously most of the testing is done on 1.4.5 since that's what most people have, so being able to test lower versions is a luxury that isn't easily achievable.
Either people just updated, or this is the first time this has happened, because I haven't heard this issue happen on a lower version at least in our Discord server.

Something like this should actually be reported for bugs, I think. Obviously most of the testing is done on 1.4.5 since that's what most people have, so being able to test lower versions is a luxury that isn't easily achievable.
Either people just updated, or this is the first time this has happened, because I haven't heard this issue happen on a lower version at least in our Discord server.


ServerMod small issue

Server won't start if there's no folder Logs inside servers/%serverid% folder. You need to manually create it.
I think it's better to create folder if not exist already.

redirect issue to Smod2 repo


Hotkeys work only for a while

The extension doesn't work correctly with the current version of Spotify Web Player.
When enabled (Manage Extensions / Spotify Web Player Hotkeys / Switch in the lower right corner) the extension works fine for a while, but after a few songs the player stops reacting to the hotkeys and the extension has to be manually disabled and reenabled to work again.
I refer only to those hotkeys:

Play/Pause media key
Previous media key
Next media key

I didn't try using the other ones.
OS: Ubuntu 20.04.3 LTS (64 bit)
Chrome: 97.0.4692.71 (Official Build) (64-bit)
Spotify Web Player Hotkeys: 0.6.1

Thanks for the bug report @sautin1 . I have trouble reproducing the issue though. What does "after a few songs" mean, do you have a more precise number of amount of time that one needs to wait for this to happen?
If not, may I ask that you open devtools (F12) at the beginning of a listening session (while the hotkeys are still working) and report back what the Console tab contains once the keys stop working?

@zopieux
I am pretty sure the hotkeys stop working starting from the moment the first Spotify ad pops up.
I suspect that the problem is related to another extension I am using - AdGuard Antibanner, which prevents websites from showing ads (or just skips them in case of Spotify). The issue is present with AdGuard enabled, hovewer I could not reproduce it after I disabled AdGuard.
See contents of the console in the log attached.
open.spotify.com-1642601759827.log

I see, thanks. Sadly it is unreasonable for me to debug or change the code to be compatible with the wide variety of random extensions people might install alongside SWPHK, that might break SWPHK for various (obscure) reasons. I'm gonna have to close this "won't fix".


TEST contribution

This is a practice contribution for a training module
Contributor guide pull request
Thanks for contributing to the Microsoft Learn contributor guide. Please read these instructions for processing your pull request (PR). Once you've read this text, delete it and write a brief description of the changes you're proposing.
Quality control

[ ] 1. Successful build with no warnings or errors: Review the build status to make sure all checks are green (Succeeded).

[ ] 2. #Sign-off: Once the PR is finalized and ready to be merged, indicate so by typing #sign-off in a new comment in the PR. Signing off means the document is ready for review and can be published at any time.

Merge and publish

All PRs to this repository are manually reviewed and merged.
Once all feedback on the PR is addressed, we'll merge the PR into the main branch.

Need help?
If you have an open PR, tag the PR reviewers in a comment with your question: @mikekinsman, @jehchow

#sign-off


docs-builder image: Only build from cilium/cilium

Most forks don't need to run this workflow. Check the repository owner and only build from cilium/cilium.

@learnitall could you approve https://github.com/cilium/cilium/actions/runs/6005524833/job/16288411316?pr=27757 🙏

/test

/test

yeah that makes sense, somehow it didn't occur to me that downstreams can just disable this action. closing 💥


Does MAPIE support multi-step regression? I am using a Tensorflow Model with KerasRegressor

I want to use MAPIE to get prediction intervals. This works for a single step output approach. However, when I want to look e.g. 10 timestamps into the future, I get the "ValueError: y should be a 1d array, got an array of shape (8190, 10) instead."
I am using LSTM for this example. When having a single step output, my model looks like this:
model = Sequential([layers.Input(shape=(X_train.shape[1], X_train.shape[2])),
layers.LSTM(64, return_sequences=True),
layers.Flatten(),
layers.Dense(32, activation='relu'),
layers.Dense(32, activation='relu'),
layers.Dense(1)])
For the multi step approach, I changed my last Dense layer parameter to the number of timestamps that I want to look into the future, in this case 10:
model = Sequential([layers.Input(shape=(X_train.shape[1], X_train.shape[2])),
layers.LSTM(64, return_sequences=True),
layers.Flatten(),
layers.Dense(32, activation='relu'),
layers.Dense(32, activation='relu'),
layers.Dense(10)])
But as mentioned, the "ValueError: y should be a 1d array, got an array of shape (8190, 10) instead." occurs.
The shape of my X is (8190, 32, 23) and the shape of y is(8190, 10)).

Hey @dani-vu,
That's something we have not yet implemented (see multi-output regression: #97). We only make for one time step ahead. Just to get more information about your issue, is this something that you would want to be implemented in MapieTimeSeriesRegressor?
For the moment, the alternative would be to iteratively do one prediction at a time.
Thank you,
Louis

Hey @LacombeLouis ,
thank you for the quick response. Yes, this is what I would want to be implemented in MapieTimeSeriesRegressor.
All the best,
Daniel


rfc2350: small fixes

use https for links
fix some typos
make a 1.5 release out of it

I was wondering why you're using the old/alternative gpg key in there, is it intended?

Thanks for the pull request. Not sure while the old PGP key is in the repo, maybe to keep existing links working.
The purpose of this repository is to provide an archive of the blog posts and whitepapers as we have written them. The remaining parts, e.g., keys, contact details, statistics, etc., should always be looked up on the live site at govcert.ch or the parent organizations website, currently ncsc.admin.ch.


MotorPair - motors not starting at same time

See https://forums.raspberrypi.com/viewtopic.php?p=2012025 for more information
Also see for possible solution https://github.com/RaspberryPiFoundation/python-build-hat/tree/motorset/buildhat , might need to look at firmware

Working on refactoring Motor classes and reducing delay of motor commands

In firmware?

In the library at the moment, although there's an odd issue i'm running into, that may need a firmware change

The following for two blue+white (typeid 49) motors reduces the delay between each starting for me -
#!/usr/bin/python3
import time

from buildhat import MotorPair

motors = MotorPair('A', 'B')
motors._leftmotor.plimit(1.0)
motors._leftmotor.bias(0.4)
motors._rightmotor.plimit(1.0)
motors._rightmotor.bias(0.4)

motors.start(-30, -30)
time.sleep(5)
motors.stop()

motors.start(30, 30)
time.sleep(5)
motors.stop()

I tried the plimit and bias methods, but I am still struggling with it. Do you know when this will be fixed? Thanks!


kinect2_bridge, 0Hz Publishing rate

Protonect gl, Protonect cl and Protonect cpu all work perfectly but roslaunch kinect2_bridge kinect2_bridge.launch fails to publish anything.
[ INFO] [1455104650.956339683]: [Kinect2Bridge::main] depth processing: ~-nanms (~-nanHz) publishing rate: ~0Hz
[ INFO] [1455104650.956483375]: [Kinect2Bridge::main] color processing: ~-nanms (~-nanHz) publishing rate: ~0Hz
[ INFO] [1455104653.956905909]: [Kinect2Bridge::main] depth processing: ~-nanms (~-nanHz) publishing rate: ~0Hz
[ INFO] [1455104653.957052427]: [Kinect2Bridge::main] color processing: ~-nanms (~-nanHz) publishing rate: ~0Hz

$ ldd Protonect | grep libfree
	libfreenect2.so.0.1 => /home/hassan/ext_src/libfreenect2/build/lib/libfreenect2.so.0.1 (0x00007fddb0dba000)

$ ldd ~/catkin_ws/devel/lib/kinect2_bridge/kinect2_bridge | grep libfree
	libfreenect2.so.0.1 => /usr/local/lib/libfreenect2.so.0.1 (0x00007f9ba7d0c000)
	libfreetype.so.6 => /usr/lib/x86_64-linux-gnu/libfreetype.so.6 (0x00007f9b9593d000)

This is my first time using iai_kinect2. Am I missing anything?

I do have the same problem here.
Protonect works fine on gl, cl and cpu modes but impossible to get any data from kinect2_bridge as it fails to publish any data.
Same Info message as above :
[ INFO] [1455184486.640053452]: [Kinect2Bridge::main] depth processing: ~-nanms (~-nanHz) publishing rate: ~0Hz
[ INFO] [1455184486.640273220]: [Kinect2Bridge::main] color processing: ~-nanms (~-nanHz) publishing rate: ~0Hz

Thank you for your help.

@xArkanEx I have got it working by rolling back libfreenect2 to 9f39b744a8fe89d4d86e38bb2c494303ab41ae86 and re-installing.

Thank you @haxan7  I will try it.
By investigating a bit I found that the  waitForNewFrame method of the  libfreenect2::SyncMultiFrameListener in the receiveFrames function of kinect2_bridge always returned false.
That's when i though the problem had more chance to come from libfreenect2 than from iai_kinect2.

I just tried out the current master from libfreenect2 and I can confirm the problem. I will look into it later, for now please use an older commit from libfreenect2.

I believe this issue is fixed in latest libfreenect2 master.


Add nodelocal DNS option as option in yaml

Enhancement
RKE issue: https://github.com/rancher/rke/issues/1050

The UI enhancement is validated in Rancher:master-head 8ede17d10
the following section shows when add/edit a cluster in YAML

The node-local DNS does not take effect in Rancher:master-head 8ede17d

add a new cluster in Rancher, edit it as YAML and fill the ip_address with <IP_ADDRESS>
the cluster is active, but there is NO node-local-dns DaemonSet
edit the cluster as YAML to add the following line to the dns section
provider: coredns
the cluster is active again, but the rke-nodelocal-addon-deploy-job is stuck in deploying, its pods shows the error:
The DaemonSet "node-local-dns" is invalid: spec.updateStrategy: Unsupported value: apps.DaemonSetUpdateStrategy{Type:"/v3/schemas/daemonSetUpdateStrategy", RollingUpdate:(*apps.RollingUpdateDaemonSet)(0xc00e4a5640)}: supported values: "RollingUpdate", "OnDelete"

Note:

If I added both the provider and the ip address when createing the cluster, the node-local DNS will work.

@jiaqiluo Can you retest this on current master-head (build 2444)

@superseb Can you add the PR that addresses this issue ?

https://github.com/rancher/rke/pull/1962

the enhancement is validated on master-2447-head 56d69e236
Steps:

add a new cluster(1 etcd+contrlplane, 3 workers) in Rancher, edit it as YAML and fill the ip_address with <IP_ADDRESS>

Results:

the cluster is active and all workloads are deployed successfully
confirm the --cluster-dns on the kubelet container should be set to designated address (need to ssh into nodes)

ubuntu@jiaqi-ec2-w1:~$ sudo docker inspect kubelet | grep -e  'cluster-dns'
            "--cluster-dns=<IP_ADDRESS>",
                "--cluster-dns=<IP_ADDRESS>",

confirm that the internal and external dns resolution works following the checsk on https://rancher.com/docs/rancher/v2.x/en/troubleshooting/dns/


Using V5.0.1 for Lumen - error at OAuth2ServerServiceProvider.php:111

So after reading some of the issus on here I decided to use the newest version of your library for my Lumen project. I'm completely new to Laravel and Lumen so please bear with me.
So after following this tutorial and also reading this I ran into this error:
exception 'ErrorException' with message 'explode() expects parameter 2 to be string, object given' in /Users/Patrick/dev/apitest/vendor/lucadegasperi/oauth2-server-laravel/src/OAuth2ServerServiceProvider.php:111

I then checked the tutorial of esbenp again and found out, that he used the version 4.0.2. Checking the code I found out, that in line 111
$grant->setVerifyCredentialsCallback($grantParams['callback']);

was replaced by
list($className, $method) = array_pad(explode('@', $grantParams['callback']), 2, 'verify');
$verifier = $app->make($className);
$grant->setVerifyCredentialsCallback([$verifier, $method]);

in 5.0.1.
After replacing this code with the old version again it all works like a charm. Any ideas on this?
PS: sidenote: In your wiki you still use "lucadegasperi/oauth2-server-laravel": "~4.1@dev", which returns a composer error! please fix!

First, if you're using Lumen you've to use the latest version of this library which is currently on 5.0. I've updated the docs.

After some more fiddling around I got it to work! :-)
The problem was, that in the tutorial the callback function for the password grant was defined directly in the config/oauth2.php file. This is what broke the whole thing. For a noob like me it took me sme time to figure that out. After solving it like shown here, it works now!
Maybe the part at line 111 can be solved somehow that both callback definitions work?!
Thanks for updating the docs!

Glad you solved it! Since 5.0 you can't use callbacks anymore. Sorry.

After reading what Twisterking did, I still dont solve it. I got this error
Argument 1 passed to League\OAuth2\Server\Grant\PasswordGrant::setVerifyCredentialsCallback() must be callable, array given
while it is being called in
oauth2-server-laravel\src\OAuth2ServerServiceProvider.php on line 124 and defined
Please somebody help me to solve this

Please open a new issue if you've any questions. Your comment will get lost and we wont be able to help you if you post on old closed issues.

okay I already did, sorry for spamming


TextEditor use 2gb memory for rendering a 10mb file

code for reproduce
use eframe::egui::{
    self,
    CentralPanel,
    TextEdit,
};

fn main() -> Result<(), eframe::Error,> {
    let options = eframe::NativeOptions {
        ..Default::default()
    };

    eframe::run_native(
        "editor big file test",
        options,
        Box::new(|_cc| Box::<MyApp,>::new(MyApp::new(),),),
    )
}

struct MyApp {
    text: String,
}

impl MyApp {
    fn new() -> Self {
        let bytes = include_bytes!("../10mb.png"); // a 10mb png file
        let encoding = detect_encoding(bytes,);
        let (str, _, _,) = encoding.decode(bytes,);
        let string: String = str.into();
        MyApp {
            text: string,
        }
    }
}

pub fn detect_encoding(text: &[u8],) -> &'static encoding_rs::Encoding {
    let len = text.len();
    let to = std::cmp::min(1000, len,);
    let mut encoding_detector = chardetng::EncodingDetector::new();
    encoding_detector.feed(&text[0..to], to == len,);
    encoding_detector.guess(None, true,)
}

impl eframe::App for MyApp {
    fn update(
        &mut self,
        ctx: &egui::Context,
        _frame: &mut eframe::Frame,
    ) {
        CentralPanel::default().show(ctx, |ui| {
            let code_editor = TextEdit::multiline(&mut self.text,)
                .code_editor()
                .frame(true,)
                .lock_focus(true,)
                .desired_width(f32::INFINITY,)
                .desired_rows(40,);
            let _output = code_editor.show(ui,);
        },);
    }
}

Here is a version of the new method that generates a big string without any dependencies, so it is easier to test:
    fn new() -> Self {
        let bytes = (0..1_000_000).flat_map(|_| (0u8..10));
        let string: String = bytes.map(|b| format!(" {b:02x}")).collect();
        MyApp { text: string }
    }

Anyway, I don't think this is a "bug". The file is just too big for an immediate UI to handle. It is trying to render the entire string as an image because it doesn't know where to start or stop, so the buffer ends up being huge. There would have to be an equivalent to the show_rows method for the text editor, so I guess this is more of a feature request?

@YgorSouza  I see, thanks for your explaination ;-)


Add MXNET_GPU_MEM_POOL_RESERVE option to reserve memory for kernel la…

@winstywang @precedenceguo Please have a look and see if this fixes the problem for rcnn.
MXNET_GPU_MEM_POOL_RESERVE sets the percentage of gpu mem that should be reserved.

I observe that the memory pool reserve is working so there is no reason for rcnn testing to fail due to oom.


EffectiveViewportChanged and InvalidateMeasure/InvalidateArrange

Describe the bug
I initially discovered this when trying to have both a Avalonia.Labs.Lottie and Avalonia.Controls.TreeDataGrid on the same view. Sometimes it would work, but most of the time the Lottie animation wouldn't show and the TreeDataGrid would break (unable to scroll, etc). I managed to distill the problem down into two example controls that cause the same behavior.
To Reproduce
using System;
using System.Diagnostics;
using Avalonia.Controls;
using Avalonia.Interactivity;
using Avalonia.Layout;

namespace Controls;

public class ExampleControl : Control
{

    protected override void OnLoaded(RoutedEventArgs e)
    {
        Debug.WriteLine("OnLoaded");
        base.OnLoaded(e);
        LayoutUpdated += OnLayoutUpdated;
        InvalidateMeasure();
    }

    private void OnLayoutUpdated(object? sender, EventArgs e)
    {
        Debug.WriteLine("OnLayoutUpdated");
    }

}

public class ExampleControlTwo : Control
{

    public ExampleControlTwo()
    {
        EffectiveViewportChanged += ExampleControlTwo_EffectiveViewportChanged;
    }

    private void ExampleControlTwo_EffectiveViewportChanged(object? sender, EffectiveViewportChangedEventArgs e)
    {
        Debug.WriteLine("ExampleControlTwo_EffectiveViewportChanged");
    }

}

Put only ExampleControl in a view and its OnLayoutUpdated method will execute as expected. Add ExampleControlTwo to the same view and the OnLayoutUpdated method in ExampleControl will no longer execute.
Expected behavior
The  OnLayoutUpdated method of ExampleControl should execute regardless of whether ExampleControlTwo is present in the same view.
Environment

OS: Windows 10
Avalonia-Version: 11.0.5-rc1

Additional context
I'm not yet familiar enough with the Avalonia source code to know what is causing this. I can see that adding to EffectiveViewportChanged adds a listener to the LayoutManager. This then affects the RaiseEffectiveViewportChanged method in the LayoutManager.

Can you please provide a minimal reproducible example project with an attached view?
Because I can't reproduce it with the two controls you mentioned. Test window XAML:
<Grid ColumnDefinitions="*,Auto,*">
  <controls:ExampleControl Grid.Column="0" />
  <GridSplitter Grid.Column="1" Background="Magenta" />
  <controls:ExampleControlTwo Grid.Column="2" />
</Grid>

Move the slider: the event is correctly raised and printed in the debug console.
LayoutUpdated should always be raised whenever a layout occurs: it's not conditional.
My first guess when reading your issue is that an uncaught exception happens somewhere with the controls you're using, crashing the layout.

It looks like you're right. I should have tested with a fresh project. It only seems to be occurring on views that are navigated to with a FluentAvalonia Frame specifically using DrillInNavigationTransitionInfo. I'll close this for now and investigate further, raising an issue with FluentAvalonia if necessary. Thanks for your help.

After digging deeper, this seems more like an Avalonia specific issue. The problem occurs when an animation is applied to the view containing both controls where ScaleTransform.ScaleXProperty or ScaleTransform.ScaleYProperty are set to 0 for the first keyframe.
MainWindow.axaml:
<Window xmlns="https://github.com/avaloniaui"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
        xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
        mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
        x:Class="AvaloniaApplication1.Views.MainWindow"
        Icon="/Assets/avalonia-logo.ico"
        Title="AvaloniaApplication1">
    <StackPanel>
        <ContentPresenter x:Name="TestContentPresenter" />
        <Button x:Name="TestButton">Test</Button>
    </StackPanel>
</Window>

MainWindow.axaml.cs:
using Avalonia;
using Avalonia.Controls;
using Avalonia.Animation;
using Avalonia.Animation.Easings;
using Avalonia.Media;
using Avalonia.Styling;
using System;
using System.Threading;
using Avalonia.Threading;

namespace AvaloniaApplication1.Views
{
    public partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
            TestButton.Click += TestButton_Click;
        }

        private void TestButton_Click(object? sender, Avalonia.Interactivity.RoutedEventArgs e)
        {
            TestContentPresenter.Content = new MainView();
            CancellationTokenSource cts = new();
            Dispatcher.UIThread.Post(async () =>
            {
                var animation = new Animation
                {
                    Easing = new SplineEasing(0.1, 0.9, 0.2, 1.0),
                    Children =
                        {
                            new KeyFrame
                            {
                                Setters =
                                {
                                    new Setter(Visual.OpacityProperty, 0.0),
                                    new Setter(ScaleTransform.ScaleXProperty, 0.0),
                                    new Setter(ScaleTransform.ScaleYProperty, 0.0)
                                },
                                Cue = new Cue(0d)
                            },
                            new KeyFrame
                            {
                                Setters =
                                {
                                    new Setter(Visual.OpacityProperty, 1.0),
                                    new Setter(ScaleTransform.ScaleXProperty, 1.0),
                                    new Setter(ScaleTransform.ScaleYProperty, 1.0)
                                },
                                Cue = new Cue(1d)
                            }
                        },
                    Duration = TimeSpan.FromSeconds(0.67),
                    FillMode = FillMode.Forward
                };
                await animation.RunAsync(TestContentPresenter, cts.Token);
            }, DispatcherPriority.Render);
        }
    }
}

MainView.axaml:
<UserControl xmlns="https://github.com/avaloniaui"
             xmlns:c="using:Controls"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:d="http://schemas.microsoft.com/expression/blend/2008"
             xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006"
             mc:Ignorable="d" d:DesignWidth="800" d:DesignHeight="450"
             x:Class="AvaloniaApplication1.Views.MainView"
             Foreground="White"
             Background="#171C2C">
    <StackPanel>
        <TextBlock Text="Welcome" HorizontalAlignment="Center" VerticalAlignment="Center"/>
        <c:ExampleControl Width="50" Height="50" />
        <c:ExampleControlTwo Width="50" Height="50" />
    </StackPanel>
</UserControl>

LayoutUpdated for ExampleControl isn't raised with the above code. However, if you change either ScaleXProperty or ScaleYProperty in the first keyframe to 0.01, it works.
Possibly related to https://github.com/AvaloniaUI/Avalonia/issues/12452

Thanks for providing a reproduction!
Indeed, this is exactly #12452, I can see a first chance System.InvalidOperationException: Transform is not invertible being thrown. (As also mentioned in that same issue, the exception is lost somewhere instead of being rethrown.)

Closing as a duplicate of #12452, which should be fixed by #13188.


Allow different risk profiles for nearby people

Is your feature request related to a situation you're trying to understand with microCOVID? Please describe.
I had a group outing with friends where one of them was tested positive for covid the day after we hung out. He was still negative when we hung out.
Only one of them had covid, but the others were still present, so I was confused whether to pick 1 person with covid, or the 2 others without covid. I settled on the former.
Describe the solution you'd like
Allow multiple risk profile selections, so:
How many people are within x feet for the below specified risk profile?
What's the risk profile?
Optional: Add people with a different risk profile
Loop with the same question.

Cool I didn't know this, thanks! Will keep in mind when I want accurate measurements. Though I marked him as "has covid", and since that's 1 million microCOVIDs, the rest don't really matter as much 😆


Panic SEGV in pkg/composer/composer.go:65

Actual Behavior
Panic SEGV in pkg/composer/composer.go:65
I note this panic in TestComposerCreateCmd, https://buildkite.com/ddev/macos-colima-vz/builds/1710#01925778-6aa1-439e-a5a2-800234aee613/15326-15327

panic: runtime error: invalid memory address or nil pointer dereference [recovered]
  | panic: runtime error: invalid memory address or nil pointer dereference
  | [signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x105300470]
  |
  | goroutine 737 [running]:
  | testing.tRunner.func1.2({0x1055ce140, 0x105bc7190})
  | /opt/homebrew/Cellar/go/1.23.2/libexec/src/testing/testing.go:1632 +0x1bc
  | testing.tRunner.func1()
  | /opt/homebrew/Cellar/go/1.23.2/libexec/src/testing/testing.go:1635 +0x334
  | panic({0x1055ce140?, 0x105bc7190?})
  | /opt/homebrew/Cellar/go/1.23.2/libexec/src/runtime/panic.go:785 +0x124
  | github.com/ddev/ddev/pkg/composer.(*Manifest).keyExists(...)
  | /Users/testbot/tmp/buildkite-agent/builds/tb-macos-arm64-6/ddev/macos-colima-vz/pkg/composer/composer.go:65
  | github.com/ddev/ddev/pkg/composer.(*Manifest).HasPostRootPackageInstallScript(...)
  | /Users/testbot/tmp/buildkite-agent/builds/tb-macos-arm64-6/ddev/macos-colima-vz/pkg/composer/composer.go:104
  | github.com/ddev/ddev/cmd/ddev/cmd.TestComposerCreateCmd(0x14000334ea0)
  | /Users/testbot/tmp/buildkite-agent/builds/tb-macos-arm64-6/ddev/macos-colima-vz/cmd/ddev/cmd/composer-create_test.go:222 +0x2280
  | testing.tRunner(0x14000334ea0, 0x1056a0cb0)
  | /opt/homebrew/Cellar/go/1.23.2/libexec/src/testing/testing.go:1690 +0xe4
  | created by testing.(*T).Run in goroutine 1
  | /opt/homebrew/Cellar/go/1.23.2/libexec/src/testing/testing.go:1743 +0x314
 ```

Same panic in https://buildkite.com/ddev/macos-lima/builds/1540#01925778-5d35-4f2c-858c-8153bd8e65e8

This may have been going on for some time.

### Steps To Reproduce

_No response_

### Anything else?

_No response_

Something changed in Composer 2.8.1, released today https://github.com/composer/composer/releases/tag/2.8.1
The panic is not the actual panic in DDEV, it's because assert is used instead of require:
https://github.com/ddev/ddev/blob/cccd95557898033d74c83bedea1b608ac6651345/cmd/ddev/cmd/composer-create_test.go#L147-L148
The actual error is https://buildkite.com/ddev/macos-colima-vz/builds/1710#01925778-6aa1-439e-a5a2-800234aee613/15049-15146
[RuntimeException]
2024-10-04 13:28:04 UTC	        	            	  Source path ".ddev/test-ddev-composer-create" is not found for package test
2024-10-04 13:28:04 UTC	        	            	  /ddev-composer-create

It is triggered when I pass a local --repository to ddev composer create.
AFAICT, it affected only our tests.
To reproduce:
mkdir composer-test && cd composer-test
ddev config --composer-version="2.8.1"
mkdir -p .ddev/test-ddev-composer-create && echo '{"name":"test/ddev-composer-create","version": "1.0.0"}' > .ddev/test-ddev-composer-create/composer.json
ddev composer create --repository='{"type": "path", "url": ".ddev/test-ddev-composer-create", "options": {"symlink": false}}' test/ddev-composer-create

The same works with Composer 2.8.0

Opened

https://github.com/composer/composer/issues/12150

The bad thing is that it broke all our tests, if it is not resolved upstream soon, the alternative is to create a new git repository and use it like this:
ddev composer create --repository '{"type":"vcs","url":"https://github.com/ddev/test-composer-create"}' ddev/test-composer-create

This will probably require adding a GitHub token to Composer config.

Yes, we can push the repo to packagist.org, and it will be available publicly, I leave it as our option.

I think you can do public just fine with static files on github too, but any way is fine.


NO-JIRA: chore: various dependency updates

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

Project coverage is 77.46%. Comparing base (f13ee72) to head (0676dac).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #588   +/-   ##
=======================================
  Coverage   77.46%   77.46%
=======================================
  Files          27       27
  Lines        1948     1948
=======================================
  Hits         1509     1509
  Misses        322      322
  Partials      117      117

/test precommit-check

/test precommit-check


[Feature Request] Config option to disable internal Tor

Had a few frustrating moments where Sparrow has dropped its connection to the electrum server, or my local Tor service (localhost:9050) has restarted and Sparrow seems to decide to start a Tor itself, which then ?conflicts with the already running Tor daemon. In the UI it' thens pretty fiddly trying to convince Sparrow to stop doing this by editing the connection back to using the proxy.
Best solution I've found so-far is quitting and editing ~/.sparrow/config to turn "useProxy": true, again; But ideally there'd be an option (config file only rather than UI would be fine) to disable Sparrow from ever trying to start Tor itself and just disconnect instead. My guess is Linux users with a tor daemon running are probably comfortable with editing a config file

Added in d84f3bf8. The config property autoSwitchProxy can be set to false in the Sparrow config file, which will cause Sparrow to not try attempting to start the internal Tor client on connection failure.