Adding more than one object to PFRelation in PFObject subclass  "Operation is invalid after previous operation."

Using LocalDataStore
Subclass of PFObject is defined like this:

//PList.h
@interface PList : PFObject<PFSubclassing>
+ (NSString *)parseClassName;

@property (retain) NSString *name;
@property (retain) PFRelation *notes;
@end

//PList.m
@implementation PList
@dynamic name;
@synthesize notes = _notes;
+ (void)load {
    [self registerSubclass];
}
+ (NSString *)parseClassName {
    return @"PList";
}

- (PFRelation *)notes{
    if(_notes == nil) {
        _notes = [self relationForKey:@"notes"];
    }
    return _notes;
}
- (void)setNotes:(PFRelation *)notes{
    _notes = notes;
}
@end

//----------
Used in code like this:
PNote *note = [PNote Object];
note.name = @"mynote";

PList list = ....
 PFRelation *relation = [list notes];
[relation addObject:note];
 [list saveInBackground];

Adding one 'note'-object goes without problems, I can see the object appearing on the dashboard. But when adding a second 'note' object an exception is thrown: [NSException raise:NSInternalInconsistencyException format:@"Operation is invalid after previous operation."];
-> object in relation is not added

Hey @meoz, thanks for the report!
Let's look into this together:

Are you calling into .notes = ... at any give point in time?
Is there any chance notes can be anything else other than PFRelation at any given point in time?
Can you post a full stack trace when the exception is being thrown? (it is most likely inside PFRelationOperation, where if you could print out and paste here the value of oldValue argument that would be just amazing.

Most likely what happens here is that you are trying to set the relation field to a PFRelation, which is unsupported use case at this point (we could support it eventually).
Another solution that might help here is to not to use synthesized property for your relation, but rather organize it into something that looks like this:
@interface PList : PFObject<PFSubclassing>
+ (NSString *)parseClassName;

@property (retain) NSString *name;
@property (nonatomic, strong, readonly) PFRelation *notes;
@end

//PList.m
@implementation PList
@dynamic name;
@dynamic notes;
+ (void)load {
    [self registerSubclass];
}
+ (NSString *)parseClassName {
    return @"PList";
}

- (PFRelation *)notes {
    return [self relationForKey:@"notes"];
}
@end

It would better describe your intention here, compared to mutable relation.
Let me know if any of this helps.

Hi @nlutsenko  thanks for replying (sorry for  the late reaction)
For some reason it appears that after calling 'saveEventually' on the subclassed PFObject the PFRelation property 'notes' becomes a PFRelationOperation which will cause the subsequent NSInternalInconsistencyException. I cant't find any place where I use the notes property for anything else than adding 'child' objects which the PFRelation is pointing to (ASQTask in my code).
This is the method I wrote to add objects to the relation:
- (void)addNotesObject:(PTask *)note toList:(PList*)list
{
    [list unpinInBackgroundWithBlock:^(BOOL succeeded, NSError * _Nullable error) {
        if (succeeded)
        {
            PFRelation *relation = [list notes];

            //add to list.notes PFRelation
            //notes = "<PFRelation: 0x7fd653830c90, 0x7fd651747580.notes -> ASQTask>";

            [relation addObject:note];

            [list saveEventually:^(BOOL succeeded, NSError * _Nullable error) {

                //here list.notes became PFRelationOperation
                //notes = "PFRelationOperation<ASQTask> add:{(\n    <ASQTask: 0x7fd65162a860, objectId: sggBDRwAoE, localId: (null)>\n)} remove:{(\n)}";

                [list pinInBackgroundWithBlock:^(BOOL succeeded, NSError * _Nullable error) {
                    if (succeeded)
                    {
                        NSLog(@"saved && pinned");
                    }
                }];

            }];
        }
    }];
}

When I would add a second object, it'll crash on the following method in PFFieldOperation.m
Value of oldValue:  FRelationOperation<ASQTask> add:{(     <ASQTask: 0x7fd65162a860, objectId: sggBDRwAoE, localId: (null)> {....
Value of key: notes
- (id)applyToValue:(id)oldValue forKey:(NSString *)key {
    PFRelation *relation = nil;
    if (!oldValue) {
        relation = [PFRelation relationWithTargetClass:self.targetClass];
    } else if ([oldValue isKindOfClass:[PFRelation class]]) {
        relation = oldValue;
        if (self.targetClass) {
            if (relation.targetClass) {
                PFParameterAssert([relation.targetClass isEqualToString:targetClass],
                                  @"Related object object must be of class %@, but %@ was passed in",
                                  relation.targetClass, self.targetClass);
            } else {
                relation.targetClass = self.targetClass;
            }
        }
    } else {
        [NSException raise:NSInternalInconsistencyException format:@"Operation is invalid after previous operation."];
        return nil;
    }

    for (PFObject *object in self.relationsToAdd) {
        [relation _addKnownObject:object];
    }
    for (PFObject *object in self.relationsToRemove) {
        [relation _removeKnownObject:object];
    }

    return relation;
}

Also doing this:
PFQuery *query = [[self.list notes] query]; NSArray *items = [query findObjects];
will crash on the following method:
`

(BFTask *)_findObjectsAsyncInRelation:(PFRelation *)relation
ofObject:(PFObject *)parentObject
forQueryState:(PFQueryState *)queryState
withCancellationToken:(BFCancellationToken *)cancellationToken
user:(PFUser *)user {
return [[super findObjectsAsyncForQueryState:queryState
withCancellationToken:cancellationToken
user:user] continueWithSuccessBlock:^id(BFTask *fetchTask) {
  NSArray *objects = fetchTask.result;
  for (PFObject *object in objects) {
      [relation _addKnownObject:object];
  }

  return [[_offlineStore updateDataForObjectAsync:parentObject] continueWithBlock:^id(BFTask *task) {
      // Roll-forward the result of find task instead of a result of update task.
      return fetchTask;
  } cancellationToken:cancellationToken];

} cancellationToken:cancellationToken];
}
Stacktrace:
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[PFRelationOperation _addKnownObject:]: unrecognized selector sent to instance 0x7ff6e0ccbbf0'
*** First throw call stack:
(
0   CoreFoundation                      0x000000010c22bd85 __exceptionPreprocess + 165
1   libobjc.A.dylib                     0x000000010bc76deb objc_exception_throw + 48
2   CoreFoundation                      0x000000010c234d3d -[NSObject(NSObject) doesNotRecognizeSelector:] + 205
3   CoreFoundation                      0x000000010c17acfa forwarding + 970
4   CoreFoundation                      0x000000010c17a8a8 _CF_forwarding_prep_0 + 120
5   Notes_parseServer                   0x00000001066e1ee0 __106-[PFOfflineQueryController _findObjectsAsyncInRelation:ofObject:forQueryState:withCancellationToken:user:]_block_invoke + 432
6   Notes_parseServer                   0x0000000106563170 __62-[BFTask continueWithExecutor:successBlock:cancellationToken:]_block_invoke + 160
7   Notes_parseServer                   0x00000001065624f0 __55-[BFTask continueWithExecutor:block:cancellationToken:]_block_invoke + 112
8   Notes_parseServer                   0x000000010655d8be __29+[BFExecutor defaultExecutor]_block_invoke_2 + 190
9   Notes_parseServer                   0x000000010655e26e -[BFExecutor execute:] + 94
....`

I want to add that not synthesizing the property and doing this:
- (PFRelation *)notes {
    return [self relationForKey:@"notes"];
}

dit not help. Still crashing on the same locations.
Thanks for any help on this.

It seems to problem is not related to using subclasses or localDataStore. Apparantely the problem occurs when  a beforeSave cloud function returns response.success() instead of response.success(request.object). When using the former on an object that  contains a PFRelation, that PFRelation get's screwed up on the client side (becomes PFRelationOperation).

I started getting this error message after migrating to heroku. I am adding a single PFUser to the relation "blocked" off the User class. I even tried a save before the add just in case.
oldValue is
PFRelationOperation<_User> add:{(
    <PFUser: 0x7fd5051256f0, objectId: GkDlCuA92g, localId: (null)> {
}
)} remove:{(
)}

Is this issue specific to parse-server?

Well I am using the parse-server (2.2.9) in my heroku app, but I sense that it has more to do with the iOS SDK than the server component.

@kokernutz Cool, thanks.
I'm encountering this bug after switching to parse-server (2.2.7), also on Heroku. The relation code in our app that's causing the PFRelationOperation thing hasn't changed in a while. I'm pretty sure the same code was working on Parse.com.
Has anyone encountered this using the iOS SDK with Parse.com?

@nlutsenko, I have also encountered the same issue after migrating parse server(2.2.9) on Heroku. I was trying to add some objects to the PFRelation Object but it automatically converted into the PFRelationOperation.
let ratedRelation = currentUser?.relationForKey("ratedImages")
        ratedRelation?.addObject(imagePFObject!)
        currentUser?.saveInBackground()

In the above code, I have a one to many relation between User and RatedImages and trying to add new ratedImages into relation of the current user.
I have also tried to save the imagePFObject before adding into the PFRelation.
Also, The about code is working fine with parse.com.

Any update on this issue. We are observing the same and the migration to Parse Server is blocked due to this issue.

I'm also having this issue trying to recreate it now

Any updates on this issue? Anybody else observing the same?

@nlutsenko, I was trying to catch the actual issue and following are the details which I have found so far,

Tried upgrading parse-server to 2.2.11 and still have the problem.

Upgraded to parse-server 2.2.12 and still have the problem.

Any update on this issue?

OK, not sure which of the following three did the trick, but this is all of a sudden working for me again, albeit with one caveat.

Updated to parse-server 2.2.13
Deleted all Installations
Deleted all Sessions

The caveat is the following code no longer filters out blocked users. (Have yet to dig in.)
query = PFUser.query()
let blocked = PFUser.currentUser()!.relationForKey("blocked").query()
query!.whereKey("objectId", doesNotMatchKey: "objectId", inQuery: blocked)

We upgraded to 2.2.14 and did not face this issue.
We are still testing it, and update here if we face any issues.


Greedy quantifiers

Currently Odinson quantifiers are neither greedy nor lazy.  The greedy v. lazy distinction is, however, essential moving forward.  Consider a naive rule fragment for detecting an NP:
[tag=/N.*/]+

The current behavior is to return all (or all tiled?) possible matches, rather than the longest.  This clutters the state and is counter-intuitive to the user considering the syntax and standard meaning of the + quantifier.

This has been fixed for a while now.


ceph-iscsi: rename group iscsi_gws

Let's try to avoid using dashes as testinfra needs to be able to read
the groups.
Typically, with iscsi-gws we can't add a marker for these iscsi nodes,
using an underscore fixes the issue.
Signed-off-by: Sébastien Han<EMAIL_ADDRESS>(cherry picked from commit 20c8065e48b6ea3027669bd5fa2f79779eac570a)

jenkins test pipeline

jenkins test luminous-ansible2.4-centos7_cluster


configure webserver

nginx or caddy

deny all .env and .git paths
username.hackclub.app -> ~/pub/username.hackclub.app static or proxy, can be updated by the config daemon
tls certs for *.hackclub.app and custom domains (caddy should do this automatically)

Obsoleted by new plan involving 2 caddy instances. #11


Drill into viz elements with anchor tags

Feature Description

You click on a bar of a bar graph, a title of a column, or a scatter chart point, and it navigates you through to another page where you can locate relevant information about the item.

Goal of Feature

Ability to navigate intuitively through Evidence to get more detail.
Has a similar goal to a filter, but should:

Be faster - drill page is prebuilt
Inspire more confidence - it feels more intuitively designed, and it doesnt feel like you might have ended up somewhere unintended

This looks like it would be achievable within echarts by using events (echarts docs)
At first glance, some things I think we'd need to figure out are:

Whether the links should apply only to series, or also to x-axis values

E.g., on a stacked bar chart showing sales by country by year, clicking on an individual box on that chart could either bring you to a page for the country you clicked, or to a page corresponding to the country and year

Syntax for adding clickable links to the chart and setting the appropriate granularity (series only, series + x, etc.)
Would we need to ensure that the links available on the chart are all represented by pages in the project?
If/how to allow for non-standard url paths (e.g., we could set it up to always start from the current page, but that wouldn't allow you to customize where the link is taking you)
How to make the links discoverable to users in the chart


Grouping the mobilenetv2_coco models

@maringeo this PR groups together the COCO models and contains metadata populated TFLite models.
Cc: @khanhlvg

@maringeo the purpose is to group together the models belonging to the same checkpoints and also update those models with metadata population. I will work on the PR to address your comments. Thank you.

@maringeo the purpose is to group together the models belonging to the same checkpoints and also update those models with metadata population. I will work on the PR to address your comments. Thank you.

Thanks @sayakpaul for the explanation, I have two follow-up questions:

do you want to group these models in a collection on tfhub.dev (similar to https://tfhub.dev/google/collections/efficientnet/1) or is this grouping purely affecting the directory structure of the MD files in the GitHub repo?
if the underlying .tflite files are changed to contain metadata, we should publish these as version 2 of the current models, since the TF Hub asset files (.tflite, .tfjs, saved_model.pb) are immutable (for backwards compatibility reasons)

@maringeo the grouping similar to this: https://tfhub.dev/google/magenta/arbitrary-image-stylization-v1-256/2. Would you suggest to keep the v1 models (i.e. the nonpopulated ones) as they are?

@maringeo the grouping similar to this: https://tfhub.dev/google/magenta/arbitrary-image-stylization-v1-256/2. Would you suggest to keep the v1 models (i.e. the nonpopulated ones) as they are?

All clear, thanks! In that case, the Lite MD files should reference the same Placeholder MD as parent-model. I see that you've already done this in the PR.
Regarding keeping the v1 models - I think we should not change anything in the v1 MD content. File-organization wise, I think it is a good idea to keep the v1 and v2 model MDs under the same parent directory, so if you add the v2 files in a separate directory, could you also move the v1 versions there?
One final suggestion - it's often a good idea to outline in the MD description what changed between model versions. This makes it easier for TF Hub users to pick the right version for their use case. Can you add to the v2 MDs that the difference between v1 and v2 models is the extra TFLite metadata?

One final suggestion - it's often a good idea to outline in the MD description what changed between model versions. This makes it easier for TF Hub users to pick the right version for their use case. Can you add to the v2 MDs that the difference between v1 and v2 models is the extra TFLite metadata?

@maringeo I think I have made a note about that already. If that is not clear please let me know.
Another issue when I was creating the v2 models I did not know about the grouping capability. So, I created all the models separately. For example, this one and this one  are basically exported from the same checkpoint. This PR groups them under one roof while only keeping their v2 variant.

Thanks, I saw the note. Also, just fyi, since today, tfhub.dev renders TF Lite metadata within the model page. You can see an example at https://tfhub.dev/sayakpaul/lite-model/arbitrary-image-stylization-inceptionv3/dr/predict/1
Overall the plan sounds good. I think that we should also modify the v1 Lite MDs to point to the new Placeholder MD files that is used for the grouping. In this way, https://tfhub.dev/sayakpaul/lite-model/deeplabv3-mobilenetv2-int8/1/default/1 would still resolve correctly. What do you think?

Fair enough. That sounds pretty. The metadata preview is so cool. @maringeo

@maringeo I have addressed your feedback. Added the following note:

If you are looking for metadata populated models you should refer here.

I hope the URL (when the model would be up on Hub) is correct.

Hi @sayakpaul , the models are live on https://tfhub.dev/sayakpaul/lite-model/mobilenetv2-coco/dr/1 (also the TF Lite metadata renders correctly at the bottom of the model pages).
This PR has been merged in 64d8c58b52e329f8d5dbd0ee4dd13848779d8cc2.

@khanhlvg FYI.


Fullscreen Script Error

Installed v2.0 via the Godot Engine AssetLib tab.
I'm trying to implement the fullscreen setting as explained in the docs. I've added the setting as below in the screen shot.

I have then setup the fullscreen.gd script on that setting:
extends Node
# value: bool
#	Whether window is fullscreen or not
# window_script_path: String
#	Path to the logic script that's used for changing window size
# window_setting_index: int/String
#	The index of the window size setting in the settings list

func main(value: Dictionary) -> void:
	OS.window_fullscreen = value["value"]

	## Execute the window size logic again, in case the player changed the
	# window size while the window was at fullscreen.
	if value["value"] == false:
		var script_resource: Script = load(value["window_script_path"])
		var script_instance: Object = script_resource.new()
		var current = ggsManager.settings_data[str(value["window_setting_index"])]["current"]
		script_instance.main(current)

	OS.center_window()

And added the control to the screen with the correct setting index.
However when I run the scene I am getting the following error:

Invalid get index 'window_script_path' (on base: 'Dictionary').

Removing the if block in the above function removed the issue but I just thought I'd call it out as I don't see any mention of window_script_path on the How to Use page

Every logic script uses a sort of "structure" to work properly, as mentioned on the wiki.
The comment in the script explains the structure. So the dictionary you pass to the fullscreen.gd script must have three keys:

value which is a boolean value, which is whether fullscreen is active or not.
window_script_path is the path that you're going to use for resizing the window size. This can be either windows_size.gd or window_resolution.gd, depending on which one you decided to use.
window_setting_index is the index of the setting that handles window size.

Here's an example:

In the above example, window_script_path points to the absolute path of window_scale.gd script, because that's the script that 01. window_size setting uses. And window_setting_index is 1 because the index of window_size is 1.

Thanks for getting back to me.
I don't see any mention of window_script_path on the How to Use page, but I probably missed it.
Happy to close this then :+1:


⚠ Bofrost.be has degraded performance

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

HTTP code: 200
Response time: 2510 ms

Resolved: Bofrost.be performance has improved in 5c101e4 after 16 minutes.


Enabling authentication with the API

It is necessary to allow app authors to authenticate against their API so that the calls can be properly authorised.
To do that, I'll want to provide a hook where the request can be modified before it goes to the server. This will be the point where consumers will add the Authorization header among other things.

Closed by #139


360 prompt virus

应该给360提bug？

应该给360提bug？

我看行

那儿下载的

呵呵，这是检测到源码内容了。

那儿下载的

就一个 download zip动作

If this is your personal computer,please do not use 360.You are a developers you can solve computer problems by yourself.

use git clone

do NOT developing your project when some AV is on. Or you will LOSE some codes at save... delete your codes without alert. easy like ringing finger. Thanos of AV world and maybe released from DC.    ----devDpool xD a joke

Please kill 360safe.exe, and run git clone to get the code.

do NOT developing your project when some AV is on. Or you will LOSE some codes at save... delete your codes without alert. easy like ringing finger. Thanos of AV world and maybe released from DC. ----devDpool xD a joke

AV is not exist.. haha

do NOT developing your project when some AV is on. Or you will LOSE some codes at save... delete your codes without alert. easy like ringing finger. Thanos of AV world and maybe released from DC. ----devDpool xD a joke

AV is not exist.. haha

"Anti-Virus" I means. 不是说那个AV熊迪~

I used git clone and received the same virus warning for tools\openps.cmd

This is simply a false positive, exclude the programs from your Antivirus and report it as a false positive to your Antivirus provider.

It is a virus itself.

You just need uninstall 360 software and revoke their cert.
https://github.com/chengr28/RevokeChinaCerts

Your overactive virus software is not a bug in our code, sorry.

Then uninstall 360 virus.

Do not use that, 360 is the biggest virus on Windows platform.

Pour some water into your computer.


Rename EVM to Chain and relocate in code

From a brief gitter chat.
The EVM object should be renamed to Chain and relocated to evm.chain.Chain.

Fixed in #46 and #56


Keep a consistent sorts for the top left menus between ENU and non-ENU

Storage Explorer Version: 1.10.1
Build: 20190926.6
Branch: master
Platform/OS: Windows 10/ Ubuntu Linux 18.04/Mac OS High Sierra
Language: Chinese(zh-CN)
Architecture: ia32/x64
Regression From: Not a regression
Steps to reproduce:

Launch Storage Explorer using the command 'StorageExplorer.exe --lang=zh-CN'.
Check the top left menus sort.

Expect Experience:
The localized menu 'Help' is sorted as the last one like ENU.

Actual Experience:
The localized menu 'Preview' is sorted as the last one.

Fix merged into master.


Add playbook to deploy elk

The new elk tooling has been released and is now production ready. This
playbook ensures were using the new elk tooling for all of our
deployments automatically without additional user interactions.
Signed-off-by: Kevin Carter<EMAIL_ADDRESS>(cherry picked from commit 256c2b0)
Issue: RO-4397

Thank you for submitting a pull request to this project!
Once all check jobs have passed and the pull request has received a
sufficient number of reviews, the repository's gate jobs can be triggered by
adding a comment of :shipit: to this pull request.
When the gate jobs have completed successfully, this pull request will get
merged automatically.


WFLY-10099 ClusteredJPA2LCTestCase fails intermittently

https://issues.jboss.org/browse/WFLY-10099

It has been mentioned / "proposed" in the past.
Ideally we want to allow cache providers to make these kind of swaps
themselves though
On Fri, Apr 13, 2018 at 9:00 AM Scott Marlow<EMAIL_ADDRESS>wrote:

@scottmarlow commented on this pull request.
In
testsuite/integration/clustering/src/test/java/org/jboss/as/test/clustering/cluster/jpa2lc/DummyEntity.java
https://github.com/wildfly/wildfly/pull/11122#discussion_r181397254:

@Entity
@Cacheable
+@Cache(usage = CacheConcurrencyStrategy.TRANSACTIONAL)

Since this test doesn't modify data, I think READ_ONLY could be used
instead of TRANSACTIONAL (some doc on different modes are in
https://docs.jboss.org/hibernate/orm/5.1/userguide/html_single/chapters/caching/Caching.html
).
@sebersole https://github.com/sebersole did we ever try to get a
javax.persistence.CacheConcurrencyStrategy added to the JPA spec? I don't
recall that coming up in the past.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/wildfly/wildfly/pull/11122#pullrequestreview-112021587,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAOUEwatxUuuAfQthBQR2_KpEXyOFjmcks5toK95gaJpZM4TS3eI
.


A few updates/fixes

While working on new features in the dashboard I've found some bugs and wanted some new features:

Add filter bar to exports
Add a "snug" variant to tables
Make z-index on panels lower so tooltips & popovers work
Fix currency symbol in text fields without a label

🌲


Crash entering isolate when clearing handle (possibly during shutdown)

Mirror issue for https://fuchsia.atlassian.net/browse/FL-273; conversation should occur there, please.

This issue looks stale.


Resolved dialogue context not found and added feature select tab and click command on dialog

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Other (updates to documentation, formatting, etc.)

Description

Added a check for dialog on Select Tab and click command, if dialog is active click on dialog ribbon button or select tab on dialog else on main page. Also changed the xpath for dialog context as it is not unique to all environment
Issues addressed

#1091
#1092
All submissions:

[x] My code follows the code style of this project.
[ ] Do existing samples that are effected by this change still run?
[ ] I have added samples for new functionality.
[ ] I raise detailed error messages when possible.
[x] My code does not rely on labels that have the option to be hidden.

Which browsers was this tested on?

[x] Chrome
[ ] Firefox
[ ] IE
[ ] Edge

@atear10 - Could you refresh this branch of yours with the latest from develop? It looks like you have some commandBar oriented changes that haven't sync'd from changes merged to develop yesterday.
Also, there are a couple of of xpath changes with using the '.' expression that I want to make sure don't impact existing methods. Comments included on the PR to confirm.


feat: workflow for canary releases

Description
Adds a workflow for doing canary releases. I think this works.

So, how does it work? What would be the workflow in order to public an rc?


Error: abstract def must be implemented for methods with block argument

Hi there.
This code works with Crystal 0.35.1
abstract class Base
  abstract def with_type(type : String = "default", &block)
end

class Adapter < Base
  def with_type(type : String = "default", &block)
    yield type
  end
end

adapter = Adapter.new
adapter.with_type("string") do |type|
  puts type
end

But on Crystal nightly is not compiled:
 2 | abstract def with_type(type : String = "default", &block)
                  ^--------
Error: abstract `def Base#with_type(type : String = "default", &block)` must be implemented by Adapter

Can you check if #9634 fixes this? If not, something in #9585 is probably causing this regression.

Note that yield type is making the block accept an argument, but the original method doesn't have block arguments. The message should probably mention this, but the compiler is working fine.

That is, you should either be doing yield or writing &block : String -> in both cases

That is, that the code compiles in 0.35.1 is actually a bug that is now "fixed".

@bcardiff #9634 doesn't fix this.
Thanks, @asterite for the explanation. &block : String -> is working for me.
Closing.

Let's keep this open because the error message is very confusing.


pull request for e2e test at 11/06/2016 01:26:03

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

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


[Feature Request] Upgrading dependency

What is your enhancement?
first I downloaded the folder containing all the files from github at the following link https://github.com/creativetimofficial/soft-ui-dashboard-react
when I tried npm install. a message appeared unable to resolve dependency tree.
npm ERR! ERESOLVE code
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! When resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from root project
npm ERR!
npm ERR! Unable to resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>|| ^17.0.0" from<EMAIL_ADDRESS>npm ERR! node_modules/react-chartjs-2
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Fix upstream dependency conflicts, or try again
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept incorrect (and potentially broken) dependency resolution.

this is due to a conflict in the react-chartjs-2 dependency with react 18. please update the dependency. in hopes of fixing the issue.

I am facing the same issue now. How did you solve it?

I am facing the same issue now. How did you solve it?

just like this
npm install --force

I am facing the same issue now. How did you solve it?

just like this npm install --force

I tried this and I was able to install the modules. However, when I start the server using npm start head head to localhost:3000 nothing loads in my browser.


10.0.17763.5830 x86&x64

please help me.

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/e699fefcc5663a3b8a8ae5b3b8f659daa0b9f32a

Now only x64

please i need help, I am facing the same issue and not sure what to do with the termsrv.zip file. All help will be appreciated

@opeyemienitan solution is posted in the 3rd comment of this issue.
For support, contact Telegram chat (link in Readme).

need file .ini please


Where is 'Statement'?

I am using parser to get an ast which returns a vector of statements but i cant find what statement is and what methods are available on it

Statement is a enum. you can find it's definition here https://github.com/sqlparser-rs/sqlparser-rs/blob/main/src/ast/mod.rs#L687

Thanks

@poonai how to make statement implement serialize type to convert to json?


On python 2.7.x the usage looks funny

A few lines from the usage:
usage: ./rusthon.py [python files] [markdown files] [tar file] [--release] [--run=] [--data=]
[tar file] is the optional name of the output tar that contains the build
()
source files, transpiled source, and output binaries.
()
--release generates optimized output without extra runtime checks
()
The () are because "print" is a statement in 2.7.x and a function in 3.x. If rusthon requires 3.x maybe a check for that would be helpful. If it doesn't, consider adding "from future import print_function"

https://github.com/rusthon/Rusthon/commit/c6cf411774a22dd6ea096cbcc7fdd7d405a4155a


Configure auto-shutdown when creating Virtual Machine

I have automated the launching of Azure Virtual Machines with python sdk. However I have not been able to found out how to configure auto-shutdown. This can easily be done when launching a VM from the Azure Portal. Am I missing something or is this feature not yet implemented in the python sdk?
Here is my code:

from azure.mgmt import compute

compute_client = compute.ComputeManagementClient(credential, subscription_id)
compute_client.virtual_machines.begin_create_or_update(
        resource_group,
        vm_name,
        {
            "location": location,
            "storage_profile": {
                "image_reference": {
                    "id": IMAGE_ID
                },
                "os_disk": {
                    "caching": "ReadWrite",
                    "managed_disk": {"storage_account_type": "StandardSSD_LRS"},
                    "name": f"{vm_name}osdisk",
                    "create_option": "FromImage",
                },
            },
            "hardware_profile": {"vm_size": MY_INSTANCE_TYPE},
            "os_profile": {
                "computer_name": vm_name,
                "admin_username": MY_USER,
                "linux_configuration": {
                    "disable_password_authentication": True,
                    "ssh": {
                        "public_keys": [
                            {
                                "path": MY_PATH,
                                "key_data":MY_KEY,
                            }
                        ]
                    },
                },
                "custom_data": custom_data,
            },
            "network_profile": {"network_interfaces": [{"id": MY_NI_ID}]},
        },
    )

`

Thank you for the feedback @edumugi . We will investigate and get back to you asap.

Hi @edumugi Python SDK is generated with swagger automatically. Related swagger is here https://github.com/Azure/azure-rest-api-specs/blob/main/specification/compute/resource-manager/Microsoft.Compute/ComputeRP/stable/2023-03-01/virtualMachine.json#L3031 and I don't find similar feature. So please open issue in https://github.com/Azure/azure-rest-api-specs/issues and mark with label feature request. Once swagger is fixed, python SDK will release new version to support it.


Contributors ORCIDs not retained on GBIF

~2~ 3 of the 8 contributors of the ESAS dataset have their ORCID recorded in IMIS:

Eric Stienen
Desmet Peter
Nicolas Vanermen

That ORCID is not retained in the contributor information on GBIF: https://www.gbif.org/dataset/3470d506-e667-4e3f-b178-819669684c05#contacts The email is not retained either, but reverts to the email of the institute.
Would it be possible to retain ORCIDs on GBIF?

@rubenpp7 Maybe the ORCID of Nicolas can also be added to IMIS: https://orcid.org/0000-0003-0156-357X

Hi @peterdesmet , working on it, I'll let you know when it is fixed

Hi @peterdesmet , we have a fix now at the IMIS side, those changes will be pushed to the EurOBIS IPT (and therefore to GBIF) in the next EurOBIS data harvest cycle next month


fix signature_backend_choices()

signature_backend_choices() was returning an empty list, because it was referencing django_anysign.settings.ANYSIGN['BACKENDS'], which is empty, rather than Django's runtime settings.

@smcoll the tests are in failure, can you fix your PR please ?

i don't think this approach will work...


Fix for obj?.[expr] optional chaining syntax

Description

[✅ ] Source branch in your fork has meaningful name (not master)

Fixes Issue: #1801
Before Merge Checklist
These items can be completed after PR is created.
(Check any items that are not applicable (NA) for this PR)

[✅ ] JavaScript implementation
[✅ ] Python implementation (NA if HTML beautifier)
[✅] Added Tests to data file(s)
[NA ] Added command-line option(s) (NA if
[NA] README.md documents new feature/option(s)

@mayankshouche
Doh, this was a duplicate of #1727 which as fixed by #1779. Fix released in 1.11.0.
If you have time to fix #1794 (it will look similar to what was done in #1779), that would be fantastic.

@mayankshouche
Doh, this was a duplicate of #1727 which as fixed by #1779. Fix released in 1.11.0.
If you have time to fix #1794 (it will look similar to what was done in #1780), that would be fantastic.


Assert that fish configuration is enabled if any user has fish as their shell

Description of changes
Add an assertion that makes it impossible to use pkgs.fish as a shell if programs.fish.enable isn't also true, otherwise the login shell will miss all the useful nix things in its $PATH.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Because that's implicit behaviour.
Also, it is quite hard to get right. If the fish package gets overridden by a user, that mechanism would fail (most likely) and that would mean that nixpkgs behaves differently suddenly although you would expect it to behave equally.
It is way less error prone to require that extra configuration line instead of doing something that only works half the time, IMO.

@ofborg test xxh

@ofborg test shadow

I fucked up, the commit is not formatted correctly.

@farcaller For next time (my bad mostly), please format it according to nixos/users-groups: ... and squash probably the commits. :)

Can we please revisit this PR. It causes issues (at least on my install) when I set program."shell".enable in home-manager.

Could you provide more details about your issue?
I tested this PR with Home-Manager for zsh and I didn't have particular
issue?
Le sam. 25 mars 2023 à 10:44, Florian @.***> a écrit :

Can we please revisit this PR. It causes issues (at least on my install)
when I set program."shell".enable in home-manager.
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/211603#issuecomment-1483780220,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMZRAKYGMIHNCNOXH5NGDW5247ZANCNFSM6AAAAAAUARLREE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I am using users.users."gador".shell = pkgs.zsh and use programs.zsh.enable = true in my home-manger config.
My home-manager config is imported as:
modules = [
[...]

home-manager.nixosModules.home-manager
        {
          home-manager.useGlobalPkgs = true;
          home-manager.useUserPackages = true;
          home-manager.users = userconfig;
        }
]

where userconfig is:
userconfig =  {
        "gador" = {
          imports = [
            nixvim.homeManagerModules.nixvim
            ../modules/userconfig/gador.nix
          ];
        };
};

And the gador.nix is further seperated, but contains the programs.zsh.enable = true part.
Also I never had an issue with zsh not working or not finding stuff in the NIXpath..

This is because you set programs.zsh.enable at the home level rather than
at the system level.
I think there are definitely stuff that are lost if you're not doing it at
the system level too, like completions paths.
In the mean time, adding the programs.zsh.enable at the system level would
fix this, but we need to investigate with home manager to see what makes
the most sense in this scenario.
Le sam. 25 mars 2023 à 12:00, Florian @.***> a écrit :

I am using users.users."gador".shell = pkgs.zsh and use programs.zsh.enable
= true in my home-manger config.
My home-manager config is imported as:
modules = [
[...]
home-manager.nixosModules.home-manager
{
home-manager.useGlobalPkgs = true;
home-manager.useUserPackages = true;
home-manager.users = userconfig;
}
]
where userconfig is:
userconfig =  {
"gador" = {
imports = [
nixvim.homeManagerModules.nixvim
../modules/userconfig/gador.nix
];
};
};
And the gador.nix is further seperated, but contains the programs.zsh.enable
= true part.
Also I never had an issue with zsh not working or not finding stuff in
the NIXpath..
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/211603#issuecomment-1483794848,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMZRCJZGJNQVHG3FOVJHDW53F6TANCNFSM6AAAAAAUARLREE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Taking zsh example:

System-level: https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/programs/zsh/zsh.nix#L168-L300
Home-level: https://github.com/nix-community/home-manager/blob/master/modules/programs/zsh.nix#L431-L616

I feel like the two options are probably complementary and it seems like "home enabled -> system enabled" ; it's not enough to just pass pkgs.zsh to users.users."gador".shell = pkgs.zsh; even if it works fine for many reasons at the moment.
@gador So I feel like this is working as intended and reducing previous wrong configurations in that space.
But if you feel like we should enable home-only configuration of shells with only passing the package at the system-level, I am interested in the arguments. It makes sense to me if you don't want to have all the /etc files around and probably some system environment variables modification. But I wonder what happens if you do sudo -E -u anotheruser -s in that case and all the stuff related to moving to another user while keeping the same shell.
@farcaller We should probably have a follow-up in form of a release notes to inform users about this change which can be surprised, would you send such a PR when you have time? (if we do agree on the intents regarding the current problem).

I have quite a lot customizations of my zsh config for my user. It wouldn't make sense to have that at the system level. I fell like this could be a common usecase: Every user should be able to customize their shell.
If I do sudo -E -u root -s I have my normal user zsh shell as root.
Just out of curiosity: What would happen, if I don't pass users.users."gador".shell = pkgs.zsh; and only use the home-manager config?

I have quite a lot customizations of my zsh config for my user. It wouldn't make sense to have that at the system level. I fell like this could be a common usecase: Every user should be able to customize their shell.

I'm not asking you to disable the zsh customization for your user, I am asking to enable zsh system files rather than just having home-level files. They will provide defaults for others users (which does not exist in your case).
Every user can customize their shell with this option (this is what I am doing, I am also running a lot of customization at the home-level).

If I do sudo -E -u root -s I have my normal user zsh shell as root. Just out of curiosity: What would happen, if I don't pass users.users."gador".shell = pkgs.zsh; and only use the home-manager config?

If you do this, then, your entry in /etc/shadow or /etc/passwd won't have the zsh shell configured, you will not have zsh as a default shell in some situations (maybe not all of them).
There's a lot of subtleties related to login shells, interactive shells, etc. when it comes to UNIX heritage.

That's definitely a good argument to perform a revert of this PR or at
least transform it in a warning.
Or expose a new option "I know what I'm doing".
Le dim. 26 mars 2023 à 09:55, gilice @.***> a écrit :

This is an issue for me, too. The zsh config programs.zsh.enable does is
ridiculously slow (makes shell startup about 250ms on my potato pc), so I
don't use that.
I know what I am doing, and am willing to take the risk for performance
.How can I disable this?
—
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/211603#issuecomment-1484023568,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACMZRH73JNJBTAHQLCDLYLW5776XANCNFSM6AAAAAAUARLREE
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Hm, the message was deleted. I don't know what to do about this.

@farcaller is on it.

I drafted #224116 to add a workaround and make a release note.

Adding some context for issues if systemd-level programs.${shell}.enable = false:

For fish, environment variables generated by systemd-level modules will not be set.  See https://github.com/nix-community/home-manager/pull/3610#issuecomment-1436113813 for some links.
For zsh, before https://github.com/NixOS/nixpkgs/pull/217205, environment variables set in ~/.zshenv were overridden by system-level ones.  That PR fixes this issue.  However, there are other issues.  See proposal 2 for more information.


polaris endpoint not accessible out side of client machine

Polaris Bug Report
Description of the bug
polaris endpoint not accessible out side of client machine. i get the following error trying to access it outside of the machine:
Bad Request - Invalid Hostname
HTTP Error 400. The request hostname is invalid.

Expected behavior
Hello world should be the return, as it is the return on the client machine

Hi @mazad01, on the current version in PowerShell gallery you would have to be running as admin for it to be listening for external requests.
In #175 we added support for binding to a specific hostname and port which means you can listen on external interfaces without running as admin although it will error out if you have not reserved the url for powershell.exe ahead of time.

Hi, I am running Polaris in an admin context of ps_ise

Can you give any more detail on your environment to help us reproduce?

Sure:

I'm using a vanilla build of Windows Server 2016 Standard (10.0.14393)
PSVersion: 5.1.14393.2636
Using the latest polaris version
Windows Firewall is completely turned off
Running the following in an Administrator instance of Powershell_ISE

New-PolarisGetRoute -Path "/helloworld" -Scriptblock {
    $Response.Send('Hello World!');
}

Start-Polaris

On the local machine, i can visit that endpoint through I.E (NOTE: it does not work via invoke-restmethod on the local machine. I'll open a separate issue for that later)
On a remote machine, I cant hit the endpoint. However, a netstat on the polaris box shows that the remote machine initiated a connection that is on TIME_WAIT

Hmm... odd.
So let's say your $Env:Computername is MyPC01 and you are running the above script and then going to another PC and running:
Invoke-RestMethod "http://MyPC01:8080/helloworld"

results in no response. Correct?
After running the sample script you gave there can you check the output of the following command:
(Get-Polaris).Listener

The Prefixes property should be {http://+:8080/}

yes thats correct. and prefixes property is empty:
PS C:\WINDOWS\system32> (Get-Polaris).Listener

AuthenticationSchemeSelectorDelegate :
ExtendedProtectionSelectorDelegate   :
AuthenticationSchemes                : Anonymous
ExtendedProtectionPolicy             : ProtectionScenario=TransportSelected; PolicyEnforcement=Never; CustomChannelBinding=<null>; ServiceNames=<null>
DefaultServiceNames                  : {HTTP/localhost, HTTP/mypc01.ad.com}
Realm                                :
TimeoutManager                       :
IsListening                          : False
IgnoreWriteExceptions                : True
UnsafeConnectionNtlmAuthentication   : False
Prefixes                             :

This is the statement we're using to detect if you're an admin on the box:
[System.Environment]::OSVersion.Platform -ne [System.PlatformID]::Win32NT -or
            ([System.Environment]::OSVersion.Platform -eq [System.PlatformID]::Win32NT -and
                ([System.Security.Principal.WindowsPrincipal]::new([System.Security.Principal.WindowsIdentity]::GetCurrent())).IsInRole([System.Security.Principal.WindowsBuiltInRole]::Administrator))

It should evaluate to true
Then it will run essentially the following code if you want to try it just plain vanilla and see if you're getting any error
$ListenerPrefix = "Http"
$Listener = [System.Net.HttpListener]::new()
$Listener.Prefixes.Add("http://+:8080/")

The first statement does evaluate to true. and i reran my code, and see this under prefixes:
Prefixes                             : {http://localhost:8080/}

question is how do i get the hostname added to the listener attribute of get-polaris? i tried running the command above, and same local host prefix

alright i manually modified this line with the actual hostname of the machine https://github.com/PowerShell/Polaris/blob/a0117afb7308273d869167af2b3d8be4f5ed8fe4/lib/Polaris.Class.ps1#L296
and i see it in (get-polaris).listener. i was also able to access it through the remote machine. question now becomes how does that attr get populated?

it seems -hostname is required in start-polaris, that might be it

Ah yeah that could be it. Did you install from the Github repo or from PowerShell gallery?

github repo

Okay gotcha, yeah we haven't updated the docs yet since the last gallery release. We just merged in a change for that. Sorry I completely missed asking that.
# Respond to all names on the designated port
Start-Polaris -Hostname "+"

# Listen on all unreserved names
Start-Polaris -Hostname "*"

# Listen on the computername of the PC
Start-Polaris -Hostname $Env:Computername

# Listen on a specific IP address
Start-Polaris -Hostname "<IP_ADDRESS>"

# Listen for requests to a specific URL
Start-Polaris -Hostname "myawesomesite.mydomain.com"

*Note: Anything other than "localhost" which is the default will either require running as admin or pre-registering the prefix for use with a specific user by running netsh as admin.
netsh http add urlacl url=http://+:1233/ user=Chris-PC\Chris

Awesome, thanks for the help!


@conda-forge-admin add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/meds-feedstock#11.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/meds-feedstock#11.


Move Python release Move archetype-net-release.yml into the public repo, Except for signing stage. Add dependsOn: PublishPackage for PublishDocs and UpdatePackageVersion deployments. Tested with Azure.Template.1.0.2-preview.7 release Included version changes to Azure.Templatetage to public repo

Move archetype-python-release.yml into the public repo
Add dependsOn: PublishPackage for PublishDocs and UpdatePackageVersion deployments.

Can one of the admins verify this patch?


Need support for hot SSL certificate reload

We need to be able to reload SSL keys while the server is running.
Environment Details

Helidon Version: 1.4.8
Helidon SE
JDK version: 1.8
OS: Linux OEL7
Docker version (if applicable):

Problem Description
Currently, if we modify the certificate the application will have to be restarted. We have a requirement where there will be multiple clients connecting to the application and each will have their own certificate. In this situation, restarting the application is not feasible as that will require coordination between multiple clients. Would it be possible for Helidon to allow SSL certificates to be reloaded at runtime.
Steps to reproduce
Nothing to reproduce. This is an enhancement request.

This is about mutual TLS - e.g. the requirement is to modify trust store of the server, not the private key.

I will just add a clarification note here, this creates completely new SSLContext and that means, it reloads basically everything.  When WebServerTls is created, the new SSLContext instance  is created based on the provided values and that includes private key, trust store etc.


WIN64 doesn't support multiple return values

Ints are returned in RAX, floats in XMM0. Small structs are returned in RAX.
Would it be okay to just bail out in machinize?
(*MIR_get_error_func (ctx)) (MIR_call_op_error,  "Windows x86-64 does not support multiple return values.");
Or should it happen somewhere else?

Ints are returned in RAX, floats in XMM0. Small structs are returned in RAX.

Fortunately, there are no structures on MIR-level.  It is a responsibility of MIR provider to implement structure passing according to ABI.

Would it be okay to just bail out in machinize?

Yes, at this stage it is ok.  In the future I'd like to extend ABI to implement any number of return values.  This will remove machine-dependency for return value passing.

(*MIR_get_error_func (ctx)) (MIR_call_op_error, "Windows x86-64 does not support multiple return values.");
Or should it happen somewhere else?

The place is ok.  But besides this place, I believe the same should be done in file mir-x86_64.c.  The generated code here is used for implementing FFI for the interpreter.  This file also provides C interface to the interpreter and provides a call wrapper which is used by the interpreter and the generator.  This code most probably should be adapted to WIN64 too.
Implementing ABI is a lot of work.  I appreciate what are you doing even if it were only part of ABI implementation.

Implementing ABI is a lot of work. I appreciate what are you doing even if it were only part of ABI implementation.
I see it as a learning experience. I'll get to mir-x86_64.c eventually.


Clicking "cancel" on the break lease confirmation dialog releases the leases

Storage Explorer Version: 1.5.0
Platform/OS Version: Windows 10/ MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32
Build Number: 20181117.12
Commit: 78a0d1ab
Regression From: Previous release 1.5.0
Steps to Reproduce:

Open one blob container which contains multiple blobs.
Acquire lease for all the blobs.
Right click all the leased blobs then check the context menu.

Expected Experience:
'Break Lease' is disabled.
Actual Experience:
'Break Lease' is enabled.

@v-xuanzh Does it actually work if you click to Break Lease?

Actually, It only works well for the first blob when trying to release the lease for multiple blobs.

@kfasick Actually in our test build for 1.6.0 (20181129.17) multi-select to release lease does work for all selected blobs. However, we only ask for confirmation on the first blob.  Users need to be careful so leases are not unintentionally released.

@v-xuanzh In build 20181129.17, multi-select to release leases does work for all selected blobs. However, we only ask for confirmation on the first blob.

Hi @jinglouMSFT In fact, clicking 'Cancel' directly on the confirmation dialog will release leases of all the leased items when selecting multiple items

@kfasick , I can confirm the next version of stg exp will return multi select lease breakage. Sorry for the nerf! 😄


Full Static not generating dynamic route if linked from a component in the layout

Versions

nuxt: v2.13.0
node: v12.18.1

Reproduction

<template>
	<div>
		<Navigation />
		<nuxt />
		<Footer />
	</div>
</template>

<script>
import Navigation from '@/components/Navigation'
import Footer from '@/components/Footer'

export default {
	components: { Navigation, Footer }
}
</script>

Steps to reproduce
Add a navigation component to layout. Link to a page that is using a dynamic route to generate. For example /mvp using a dynamic route page _pageUID.vue.
What is Expected?
Generate /mvp folder along with the other static routes.
What is actually happening?
On nuxt build && nuxt export the /mvp folder does not get generated. If I add the link anywhere else on another page (not a component that is used only in the Navigation) the /mvp folder does actually get generated.

Nuxt has no problem in finding links in subcomponents. I just tested this to make sure of it.
Could you provide a reproduction sample on codesandbox?
Also what is content of Navigation component?


remove unnecessary code

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+
Thanks!

:pushpin: Commit b6aac11 has been approved by alexcrichton

:hourglass: Testing commit b6aac11 with merge 00616da...

:broken_heart: Test failed - status-appveyor

@bors: retry

:hourglass: Testing commit b6aac11 with merge fdfdb5f...

:sunny: Test successful - status-appveyor, status-travis
Approved by: alexcrichton
Pushing fdfdb5f393c39ef5899625b344489b6567aaaa28 to master...


Add tests to order system and refine some functionality

TODO: implement count increasing for duplicate item added to order.

Conflicts are resolved, good to merge now.


Docs: Link to pill from introduction page is broken

Docs: Link to pill from introduction page is broken
When navigating to the pill element from the introduction page -> elements showcase seems like the link is broken
To Reproduce
Steps to reproduce the behavior:

Go to https://papanasi.js.org/?path=/docs/📗documentation-introduction--page
Click on the pill 'Preview Element button'
See error

Expected behavior
Page should render pill docs https://papanasi.js.org/?path=/docs/🧩elements-components-pill--pill

[x] Will add a fix

Thanks for the change :)


Implement Array.of method

The algorithm is based on ECMA-262 v6, <IP_ADDRESS>
JerryScript-DCO-1.0-Signed-off-by: Adam Szilagyi<EMAIL_ADDRESS>

I think we havbe a constructor which constructs an array from a list of elements. Can't we use it?

@zherczeg The constructor always creates an array instance, however this method is generic.

@rerobika, @zherczeg We could do something like:
if (!ecma_is_constructor (this_arg))
{
  return ecma_op_create_array_object (arguments_list_p, arguments_list_len, false);
}

/* Do generic construct. */

I did the requested changes above.


fix attachment ENABLED (#4064)

Back port from #4064

fix attachment ENABLED
change wrong name on app.ini.sample

make L-G-T-M work


removed Makefile

This pull request is now managed with CodeReviewHub. Refresh.

:shipit:

Is it a test pull request or you are really going to remove it?
It looks like the makefile maybe useful


Switch from less to scss

To upgrade bootstrap for the CSS we would need to switch from less to scss. It looks doable, but if anyone has opinions, or experience, please comment.

Superseded by #868.


More buttons

Is there a reason why only so few buttons are added? I'm assuming it's because they are the center ones, but I'd say add a few more - a bit of the third row at least.

I'm going to work on this in the future.

@LudwikJaniuk I have pushed a new update - v1.3 - which now allows you to change the shortcut of any of the letter keys. Either restart DevKeyboard and follow the auto-update instructions, or go to my releases page and download version 1.3.

And perfect! I'm clossing the issue.


Memory detection v2

This is a complete rewrite of the memory issue detection algorithm.

Instead of looking at the memory usage, it looks directly at how much the event loop was blocked due to garbage collection.
The previous implementation did look at event look blocking to some extent. However, it was not a direct measurement, it was only valid for node.js < 8, and only looked at events related to the old space.
The new version can detect issues with both old and new space. In recent node versions, the old space GC is actually so good that it is rarely an issue.
The new version indirectly handles highly parallelized gc, by looking at actual durations which will be less for highly parallelized gc.
The new version, also detects issues with short but very frequent blocking gc. The scavenge event, only takes about 5ms but if it runs 100 times over a second that is still a big issue.

Along with the new algorithm, is a new internal document that describes the model in details and contains the following table that I spent quite a while compiling.

Task
parallel
Incremental
Cocurrent
heap space
Appropiate time
Description
Trace event

Scavenge
√ (v10+)

new
~ 5 ms
Cleanup nursery
V8.GCScavenger

Marking
√ (v10+)
√ (v4+)
√ (v8+)
old

Finding live objects
V8.GCIncrementalMarking

Finalize Marking

old

V8.GCIncrementalMarkingFinalize

Sweeping

√ (v4+)
old

Reclaming dead space

Compaction + Update
√ (v6+)

old
~ 10 ms
Defragmenting memory
V8.GCFinalizeMC

Array buffer freeing

√ (v10+)
unknown

@goto-bus-stop Could you review this?

@goto-bus-stop Thanks. I merged the PR with the changes you suggested.


nix-shell failing on mac

When I invoke the command nix-shell on Mac (MacOS Ventura 13.2.1) I get compilation errors around some API_AVAILABLE/API_UNAVAILABLE code. Does this mean anything to you? I can't figure out which part is not working.
Output attached in file nix-shell-output.txt

Was this introduced, merging #175?

Was this introduced, merging #175?

I tried again with commit https://github.com/elementsproject/peerswap/commit/2e2177e947e220de73002df7bdcc8bd97dd354f7
Build does work there, so I guess it's introduced in that PR, yes.
Script ends with the following in this commit, but no error sprees.
found 0 vulnerabilities
linking bin 'mmdc'
patching script interpreter paths in<EMAIL_ADDRESS>installPhase completed in 44 seconds
post-installation fixup
rewriting symlink /nix/store/nwi1k60928bvk19mq3v2iyqq24hwn1fw-_at_mermaid-js_slash_mermaid-cli-9.2.2/bin to be relative to /nix/store/nwi1k60928bvk19mq3v2iyqq24hwn1fw-_at_mermaid-js_slash_mermaid-cli-9.2.2
patching script interpreter paths in /nix/store/nwi1k60928bvk19mq3v2iyqq24hwn1fw-_at_mermaid-js_slash_mermaid-cli-9.2.2
bash: ./contrib/startup_regtest.sh: line 50: syntax error near unexpected token `else'
bash: ./contrib/startup_regtest.sh: line 50: `    else'
bash: setup_alias: command not found

Sorry for the late response, I got trapped in a nixpkg rabbit hole. It seems that the bitcoin v24.0.1 nix package fails to build on darwin (at least x86_64). #178 reverts to v24.0 which seems to work for me.
Could you give it a try and report if the PR also works for you (besides the dup3 issue)

Could you give it a try and report if the PR also works for you

Works for me!


Dijkstra Algorithm in c++

This issue is about [Algo/DS Name/Question Name](link to resource for the Algo/DS/Question)

[ ] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[ ] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[ ] I will send my PR only after I'm assigned to this issue by any maintainer

[ ] Assign this issue to me (I want to work on this)

[ ] I ensure that I am not already assigned to 2 or more issues

Can you Assign this to me.


Tutorial Indicator and other unused Dependencies removed

#262 (Part 3)
I have removed the Tutorial Indicator Dependency and other dependencies.

@opticod Let's wait for this PR. I tried your way -
Remove first compile fileTree(include: ['*.jar'], dir: 'libs') then try to remove each. but again app was running fine when axml.jar was removed.
I think, I have to look at Gradle lint tool to find unused dependencies. I will soon commit changes.

@opticod
compile files('libs/core-<IP_ADDRESS>.jar')
compile files('libs/prov-<IP_ADDRESS>.jar')
There not used anywhere in project. I am removing these.
However, library is used for security purpose (crytography). Any idea why it was included ?
However, for others , only compile files('libs/zipsigner-lib-1.17.jar') is unused which also I am removing.

Seems like we removed necessary libraries. APK creation is failing. Will fix this.

Added via 5cbae12


Function not defined error

Hi! First of all, thanks for this awesome bridge - really useful tool)))
I am configuring my templates trying to replace GeneratorURL host with my company public domain and whatever function I tried to use (stripDomain, stripPort, replace, reReplaceAll) it always  throws an error like function "<funcName>" not defined. Here is a simple expression from one of my templates:
{{ if (.GeneratorURL) }}👉[Alert details]({{ replace .GeneratorURL "http://vmalert-vm-victoria-metrics-k8s-stack-599dbcd7db-mfb76:8080" "https://vmalert.mycompany.com" }})  {{- end }}

Is it even possible to use those functions in templates?

Hey there, @cyxou - sorry for the slow reply!
Yes, those look like they should work fine. I think this is caused by not including the Prometheus functions in the very first template rendering attempt. I've tweaked it and am adding a new release, but can you test and let me know if it works as you have it configured?
If it doesn't, I'd like to hear more about how your configuration is structured such as file locations/contents, debug logs, etc.


Fix Import/Export tools not running with JAVA 10

One dependency required in the two tools is the package javax.xml.soap, which is no longer available from Java 9. So we need to add this dependency through Gradle, in order for users who use Java 9 or higher version to get them running properly.

Go for merge! :+1:


RFC:  Need support for ... RETURNS SETOF <type> and ... RETURNS TABLE (...)

We need to support Postgres' SRF api.

SETOF is already supported, TABLE needs work!

+1 for supporting this.  Using RETURNS TABLE lets the data types be coded into the function definition.  Using RETURNS SETOF often involves creating custom types.  Many of these types would only be used for a single SETOF function.

This got merged in PR #326

Looks like the docs need updating https://github.com/tcdi/plrust/blob/main/doc/src/spi.md#complex-return-types


Pull Test Content From oscal-demo-content

We could consider publishing the oscal-content folder as a Maven artifact with a different classifier (perhaps default-content) that could be pulled along with the app jar as a default/testing artifact.
We could consider pulling from oscal-demo-content as a default/testing artifact.

This was partially done with the maven submodules in #34, but we likely want to consider an approach similar to #62 to dynamically download content from oscal-demo-content.

Did you have a general plan in mind for the implementation here? We can download a ZIP of the repo but I am not entirely sure how we'd go about extracting it.
I also feel like using git submodules for that purpose might be a little messy but possibly another option?

We could use maven-antrun-plugin for unzipping.
I have had issues in the past with keeping 'latest' git submodules up to date in the past, but that was quite a while ago.

Turns out there's an unpack option to the download plugin we've already used elsewhere! Had to dig a bit to find the parameters that are available.


Create RType and rtype(). Move parse and eval_string to functions.

This is a response to a request from @Ilia-Kosenkov to mirror the SEXPTYPE enum in Rust.
We introduce the RType enum and the method Robj::rtype.
As a side-effect, move parse and eval_string to functions and lose the Robj:: prefix.

Since you"re calling it RType and not SEXPType, would it make sense to also use human readable values? Something like RType::Character, RType::Integer, RType::Real, RType::Closure, RType::Pairlist, etc.

That would certainly be more readable.

@clauswilke, @andy-thomason,
I am all for readable ({cpp11} - style) names.

The names mostly match the corresponding wrappers.
I may also make an RAny enum which converts to/from any of the wrappers.

Ok. I'll do this.
On Sun, Feb 14, 2021 at 6:38 PM Claus Wilke<EMAIL_ADDRESS>wrote:

@clauswilke commented on this pull request.
In extendr-api/src/lib.rs
https://github.com/extendr/extendr/pull/177#discussion_r575847376:

Character,

Logical,
Integer,
Real,
Complex,
String,
Dot,
Any,
List,
Expression,
Bytecode,
ExternalPtr,
WeakRef,
Raw,
S4,
Unknown,

Could you add comments here for each stating which SEXPTYPE it corresponds
to? Like so?
Logical,  // LGLSXP
Integer,  // INTSXP
Real,     // REALSXP
etc?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/extendr/extendr/pull/177#pullrequestreview-590073452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAL36XGGYMZRWVRHAVYQUH3S7AKC7ANCNFSM4XTHAHKQ
.


Type attribute stripped from input tags

app/src/components/list-control-search.html:
...<input type="text" value.bind="filter & debounce:1000" placeholder='ie: "name: intterra"' />...
app/dist/components/list-control-search.html:
...<input type="text" value.bind="filter & debounce:1000" placeholder='ie: "name: intterra"' />...
app/dist/app-build.js:
...<input value.bind="filter & debounce:1000" placeholder=\'ie: "name: intterra"\'>...
Everything else in the bundle is working great. I feel like I'm taking crazy pills:)

Are you using the jspm bundling system? If so, it has an HTML minifier. My guess is that the default settings remove the type if it is a type=text because that would be the default. You can change that. @ahmedshuhel Can you confirm?

Here is the default list of options we use for minifying html. Only  removeEmptyAttributes: true, is closer to this bug. But, it should only remove empty attributes and not the type="text" as this is not empty. I'll have to dig into html minifier docs to see if they enables this stripping by default or not. In the mean time, as @EisenbergEffect suggested you can override any defaults if you want by using options.htmlminopts : { all_options_that_html_minifier_support }
Here is a list of options html-minifier supoorts. https://github.com/kangax/html-minifier#options-quick-reference

@jdposthuma @EisenbergEffect only removeRedundantAttributes option can cause this. According to html-minifier docs it's false by default and we haven't enabled it either.  @jdposthuma Could you try explicitly disabling this in options.htmlminopts and see how it goes?

Please note that html minification actually never happens in version 0.4 due to https://github.com/aurelia/bundler/pull/146.

Added removeRedundantAttributes: false  default option.


How to enable auto_complete as a filter for AJAX source

Nick,
I am trying to implement an auto_complete filter from an ajax source. I am using the newest version of yadcf and DataTables so I am having difficulty getting your past answers to work. Specifically, do I need to include enable_auto_complete anywhere?
I tried changing the filter_type from "text" to "auto_complete" but that didn't work.
What else do I have to edit to enable auto_complete from an ajax source?
I can provide a js fiddle to show my syntax/files. Thanks for the continued help.
Edit, Also, do I need to include a text_data_delimiter?

You mean that you want to set a the source attribute of the auto complete plugin? Like in the following examples? N#1 / N#2 ?
You know that you can populate your autocmplete filter by using the data of yadcf and if not it will get populated automatically by yadcf.
Please describe your scenario.

I have a dataTable with yadcf filters. Tables are being drawn to an html page from mysql tables via an AJAX file. I want to have the auto-complete filter use the same data that the column has. Is there a simple edit if the auto-complete has the same values as the column if its filtering?
Thanks for the explanation.

It should work automatically,  please post your datatables and yadcf setup

Those files are thousands of lines long, which sections are you interested in? Alternatively, i have posted a link to my source code if you don't mind looking at that. Your help is always appreciated.
https://www.dropbox.com/sh/sl099mnccavco5i/AABko2JrED4xogtPFds153-Ya?dl=0

In server side mode you must return additional data in your json for each yadcf column, that attribute should be called yadcf_data_0 / yadcf_data_1 / etc... look at the showcase page - server side setup - read all the comments on that page you can see all the relevant stuff (server response) in developer tools of chrome in the network tab.

The autocomplete filter on the link you provided for the server set up does not work.

You are right , I messed it up in 0.8.9.beta.23 , will fix it in the following days, untill then you cna use 0.8.9.beta.22


implement generator for single height pitch matched nand2 gate

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@11bca36). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   64.68%
=========================================
  Files             ?       24
  Lines             ?     2115
  Branches          ?        0
=========================================
  Hits              ?     1368
  Misses            ?      747
  Partials          ?        0

Continue to review full report at Codecov.

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


crypto: add support for verifying EIP712 messages

For UsernameProof messages, users can either set an fname or an ENS domain they own
For ENS domain UsernameProofs particularly, the signature they send for the proof is an EIP-712 signature made from the Ethereum wallet which owns the domain (and is a verified address to their FC user).
Currently we don't have the foundation set up for handling EIP-712 signatures, which needs to happen so we can verify UsernameProof messages

I thought the fname registry was still off-chain?
Regardless, I'd be happy to take this one on @haardikk21

AFAIK yes it is still offchain, but ENS UsernameProofs are possible I believe.

This might be a good reference for 712 signature verifications: https://github.com/farcasterxyz/hub-monorepo/blob/main/packages/core/src/crypto/eip712.ts
@gregfromstl, fyi: i would need this in message validations (#19) for validating the "verifications". maybe we should discuss the API or exactly follow hubble implementation.

Okay sounds good. If you have a specific API you'd like me to follow let me know, otherwise I'll follow Hubble's. I'm hoping to get this done by our call Sunday

Lets follow the hubble API.


Update

@daden-ms,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Add 'E' keybinding (exec shell) to services panel

Addresses https://github.com/jesseduffield/lazydocker/issues/385

@jesseduffield Thanks for taking the time to check this out and answer me.


 I chose the GPU version (torch 1.8.1+cu111). But, then in the dev-only steps, this GPU version is immediately uninstalled and replaced with the CPU version (torch-1.8.0+cpu). This is because of:
https://github.com/ivadomed/ivadomed/blob/6c56f91c614faf43567d558acbad349b6565c1fa/requirements_dev.txt#L1-L2
Tagging @dyt811 as this is tied to the changes made in https://github.com/ivadomed/ivadomed/pull/757. Is the intention that devs only use the CPU version of PyTorch?

Tagging @dyt811 as this is tied to the changes made in #757. Is the intention that devs only use the CPU version of PyTorch?

That was the original intention when we always assume user clone down the repo. I think since we broke down that multi step process, we can probably remove the lines referenced and assume CPU version was installed first already and if needed, GPU installed manually after that before the dev instructions are  carried out.

Hi @joshuacwnewton and @dyt811 👋
As discussed in #728, and in light of #951 #861 #757 I'm trying to work on a fix. IMHO installation is an important aspect. I would definitely need a little help to ensure if the fix works cross-platform. Please let me know if there are any other issues that I have missed and should refer them as well.

@kanishk16 the most recent important change was introduced in https://github.com/ivadomed/ivadomed/pull/969
The original intention was to use the requirements_dev.txt as a SINGLE step process to setup development and all dependencies including Torch. I think probably the quickest solution currently is to simply remove the requirement.txt reference in Line 2 and keep the current installation instruction as it covers very wide base already.
#951 and #757 are already mostly resolved in #969 so I don't think they pose much of an issue.
#861 is challenging MAINLY because across OS, and across setups (CPU vs GPU), ivadomed cannot have a unified automated installation process. i.e. ivadomed is not capable at installation time to automatically infer whether the user has compatible GPU/driver/etc necessary to install the right torch version (cuda 10 vs 11 etc). If there is a UNIFIEID pip install torch that works for all OS/platform, we can adopt it but this problem is upstream at torch level. The best example is to look at how torch installation screen is done:

Notice how across OS, across CUDA version, the pip commands needs to be modified. This is further complicated by the venv choice.
From our perspective, if we can somehow reduce the CUDA supported version (right now we only support 11) or drop CPU version installation support completely (e.g. Mac, since almost no GPU support there), the installation instructions can be much more streamlined.
Another alternative is to use a script that has system level access and does a lot of assumption checking and validation before pulling down and installing the right component (essentially automating the installation instructions we currently have) but those typically do not exist  at conda/pip level and needs a wrapper like FSL or SCT does.


Asking about SVDSolver

when you train LCM_svd, you set svd_solver like,
svd_solver = SVDSolver(args.N, noise_scheduler.config.sigma_min, noise_scheduler.config.sigma_max, 7,0.7, 1.6)
why you change training timestep to args.N (for example 40),
can I get advice about the  SVDSolver ..?

The SVD is built with EDM framework, which is a continuous diffusion format. Yet the consistency distillation requires us to set discrete timesteps for distillation. You might refer to https://github.com/luosiallen/latent-consistency-model/blob/main/LCM_Training_Script/consistency_distillation/train_lcm_distill_sd_wds.py if you are building things with DDPM framework.


Added setReadOnly method

For one of my projects, I wanted a read-only log view, so I needed to expose the setReadOnly editor method. Implementation is straightforward.

Sorry about that. I have no idea how I overlooked that before creating the pull request. Should be fixed with my latest commit.

No problem @microtherion!
I can't tell you how happy I am that others are using this library :)


Make the progress bar representative of actual progress

This change goes most of the way towards a progress bar that reflects the actual progress of XMLHttpRequest and not simply and pseudo-progress bar.
It uses the Content-Length header to determine the total length and that could cause the progress bar to fail if the server does not send the header.
I may be wrong, but I believe this addresses #58.
This modified code with a real progress bar can be seen on my website.

Thanks for this. But having thought about representing actual progress a bit more in the meantime (and having not updated the home page that talks about it, sorry) I think a real progress bar would actually be detrimental for web pages. The progress indicator is here to indicate that the page is loading, and most of the time we’re waiting for a webpage it’s because we haven’t received the first byte yet (latency), not because it’s not completed (bandwidth). With a real progress bar, the bar would appear stalled most of the time, which look like the website just stopped working.
This would be useful only for big files downloaded through InstantClick, but I advise against doing that, as gaining 300ms out of a 3s download isn’t worth it.
So there’s no use for it, I think. Thanks for a proof of concept though. I tested it on your webpage and yeah, it reinforces my point: tracking a page progress won’t be an useful bit of feedback in the vast majority of cases.
It doesn’t address #58, the problem described there is that the progress bar (introduced in 3.0) makes pages appear slower when they display instantly. Something that should get fixed in the next release.

I noted the same. That's probably the best thing to do. :thumbsup: Feel free to use this if you ever want to.


Disable 'Restart tablet' button on followers

On node's tablets tab, there is a button to restart each tablet, including followers

(btw it has 2 hovers for some reason)
One might expect that this button will restart the follower, however it will call tablets?KillTabletID=72075186224103527, restarting the leader tablet.
AFAIK there is no way to restart a specific follower, so the best fix is probably to simply remove the button for followers

Remove two popups
On hover display message "it's impossible to restart a follower"


Kick SubCommand

[x] This PR actually submits something - don't use this system as a messaging service!
[x] This PR isn't duplicated - you can check if it is by scanning the small list - link is on the navigation bar for this repository.
[x] This PR includes appropriate markdown for sections - e.g. code blocks for suggested code.
[x] This PR description and comments in code is understandable - feel free to use your native language to write if you are not comfortable with English.
[x] This PR has a single branch for itself in your fork - don't just commit to the master branch of your repository!
[x] This PR has comments written in clear English - please don't write unreadable comments just for your eyes.

Pull Request description
This PR adds /p kick  to kick players from your plots.

Optional information
I just translated it for eng + deu.

I updated everything you wanted from me. :3


“a.b = c ” does not work in the list

I created a file that reads: a: [ {b.c = 1} ] , When I executed hocon:load/1, I got it: {ok,#{<<"a">> => [#{<<"b">> => 1}]}} , It looks like c is missing
When I change the contents of the file to: a: {b.c = 1} and executed hocon:load/1 again, I got the correct result: {ok,#{<<"a">> => #{<<"b">> => #{<<"c">> => 1}}}}

fixed in 0.9.3


sagemaker use reserved instances

cc @achalddave

Merging to unblock for now but can you make this a flag?


test: refactor mock setup in maven plugin tests

Refactoring required by changes in vaadin/flow#20465.

The Flow change will be backported to 24.5 and 24.4, so probably this PR should also be cherry picked.


Feature request: alert when paper in my library is replaced with new revision

It would be nice to receive an alert of some form when the authors upload a new revision of the paper i have previously saved in my library.

(Point of information: It looks like libraries are sorted by date of most recent revision, so one nice thing is that an updated paper will jump to the top of your library.)


buildPursPackage: fix compilation/test execution

summary
This PR fixes the compilation/test execution of all frontend clients.
some background:
Compile all required sources and execute unit tests during the checkPhase.
The checkPhase is now a required argument to buildPursPackage. It
would be just always execute npm run test but this fails in cases
where spago test is used which (always) tries to download remote files.
Notes

The scb-client is not currently being built by hydra so #2597 should be merged before this one
Once scb-client is actually built by hydra it will fail because of some naming conflict. So we also need to wait for that.

Pre-submit checklist:

Branch

[x] Commit sequence broadly makes sense
[x] Key commits have useful messages
[x] Relevant tickets are mentioned in commit messages

PR

[x] Self-reviewed the diff
[x] Useful pull request description
[x] Reviewer requested

If you updated any cabal files or added Haskell packages:

[ ] $(nix-build default.nix -A plutus.updateMaterialized) to update the materialized Nix files
[ ] Update hie-*.yaml files if needed

If you changed any Haskell files:

[ ] $(nix-shell shell.nix --run fix-stylish-haskell) to fix any formatting issues

If you changed any Purescript files:

[ ] $(nix-shell shell.nix --run fix-purty) to fix any formatting issues

Pre-merge checklist:

[x] Someone approved it
[x] Commits have useful messages
[ ] Review clarifications made it into the code
[ ] History is moderately tidy; or going to squash-merge

I just rebased on top of master now that scb has been fixed and hydra is actually building it. Looking good locally. All test suites are being executed and are succeeding.

I just rebased on top of master now that scb has been fixed and hydra is actually building it. Looking good locally. All test suites are being executed and are succeeding.

@shmish111 you mean aside from https://github.com/input-output-hk/plutus/pull/2599/files#diff-db24ed8bb37d6b394bdb2a451c426e71a322daccb374e7eac7b7b9e6109279e5R40 ? As a comment in the git commit or where should I put it?

@shmish111 you mean aside from https://github.com/input-output-hk/plutus/pull/2599/files#diff-db24ed8bb37d6b394bdb2a451c426e71a322daccb374e7eac7b7b9e6109279e5R40 ? As a comment in the git commit or where should I put it?


Why doesn't theme re-exporting work?

The docs show two approaches how to import and set a theme:
1. (e.g. here):
import { themes } from 'mdx-deck';
export const theme = themes.dark;

2. (e.g. here):
import { dark } from 'mdx-deck/themes';
export const theme = dark;

I'm wondering why this doesn't work:
export { dark as theme } from 'mdx-deck/themes';

It's even in the homepage's GIF but I guess it's no longer supported. Out of curiosity, what's the technical reason? I'd prefer this one-liner 😄.

This is currently a limitation with MDX, which I believe will be resolved in a future version


TextureFrame modulate won't work if aspect ratio is keep aspect or keep aspect centered

Operating system or device - Godot version:
Linux Mint 18.1 - based on Ubuntu 16.04
Godot version 2.1.1
Issue description (what happened, and what was expected):
Changing the modulate color on inspector not affecting the TextureFrame if it's stretch mode is Keep Aspect or Keep Aspect Centered
Steps to reproduce:
Add a TextureFrame node, change its modulate color. Change the stretch mode to every option and see the modulate color not working for Keep Aspect and Keep Aspect Centered modes.

Two points

The bug is only present in 2.1.1, as 3.0 has removed the modulate from TextureFrame, and readded it into CanvasItem.
The bug comes from the lack of ,modulate here: https://github.com/godotengine/godot/blob/d5e2b59c84d8cc60dd042c1567cd1d74a654dc6e/scene/gui/texture_frame.cpp#L83

fixed by a7ec7dc.
2.1.3-rc1 got this fix.


Nodes - It is possible to make a node parent/child itself

It is possible to make a node child or parent of its own.
If a node is his own child/parent the LoadProperty children load his own and you have the same effect with LoadProperty parent.
Update
$node.ParentId = $node.Id;
$svc.Core.UpdateObject($node);
$svc.Core.SaveChanges();

Result
Type          : biz.dfch.CS.Appclusive.Api.Core.Node
Parameters    : {}
ParentId      : 790
Id            : 790
Tid           : 11111111-1111-1111-1111-111111111111
Name          : TestNode Child
Description   : TestNode used in test
CreatedBy     : WIN-8A036G6JVPJ\Administrator
ModifiedBy    : WIN-8A036G6JVPJ\Administrator
Created       : 17.11.2015 16:01:51 +01:00
Modified      : 17.11.2015 16:01:51 +01:00
RowVersion    : {0, 0, 0, 0...}
Parent        : biz.dfch.CS.Appclusive.Api.Core.Node
Children      : {TestNode Child}
IncomingLinks : {}
OutgoingLinks : {}

duplicates dfch/biz.dfch.CS.Appclusive#118

this is still an issue, when using the following:
$svc.Core.SetLink($node, "Parent", $node);
$updateResult = $Svc.Core.SaveChanges();
it doesn't throw error, the parent Id of the node becomes the same as its Id and the node cannot be deleted (unless another node becomes its parentId)


Add Object method mocking functionality

Purpose
Object method functionality is implemented in this PR
Fixes #22095
Approach
Iterate through type definitions to verify if the function exists in the Object and it the return types are compatible
During BIR generation, the callTerminator of the object method we are mocking is modified.
isVirtual is set to false and the Object reference is removed from the arguments
Check List

[X] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

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

shall we send another PR to master as well.

I created a PR for master : https://github.com/ballerina-platform/ballerina-lang/pull/22348

Codecov Report

Merging #22366 into ballerina-1.2.x will not change coverage by %.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           ballerina-1.2.x   #22366   +/-   ##
================================================
  Coverage            14.59%   14.59%
================================================
  Files                   51       51
  Lines                 1411     1411
  Branches               219      219
================================================
  Hits                   206      206
  Misses                1189     1189
  Partials                16       16

Continue to review full report at Codecov.

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


Bug Fix bf error change the murmurhash to crc32 in regression test p2

Proposed changes
Bug Fix bf error change the murmurhash to crc32 in regression test p2 cause join result with string type result error.

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 42.09 seconds
stream load tsv:          456 seconds loaded<PHONE_NUMBER>9 Bytes, about 156 MB/s
stream load json:         22 seconds loaded<PHONE_NUMBER> Bytes, about 102 MB/s
stream load orc:          58 seconds loaded<PHONE_NUMBER> Bytes, about 18 MB/s
stream load parquet:          29 seconds loaded 861443392 Bytes, about 28 MB/s
insert into select:          66.9 seconds inserted 10000000 Rows, about 149K ops/s
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230626071549_clickbench_pr_167712.html


Map face onto another matched face.

What i'm trying to acheive using this library is to match a captured face into another captured face in a video. Sort of a face-swapping thing like https://www.jibjab.com/ . Is there a way to use the landmarks to map one face into another?
So far I'm using extractFaces() to get a rectangular image of a face, and then drawing that in a canvas using the FaceDetection coordinates, but that's not a very good solution since it's jittery and doesn't always work

The library doesn't provide any utility for that, you would have to come up with you own logic to cut out the face based on the landmark points from your canvas.


Property of object without explicit initialisation in constructor does not become 'reactive'

I assign an array of Task objects to Vue.data. When an instance property of Task is not being set in/by constructor, Vue will not attach setter/getter functions. Is this by design?
In the following case, show will  not be 'reactive':
class Task
     name: ''
     isRunning: false
     show: true

 constructor: (@name, @isRunning = false) ->

Below, show is being set explicitely in the constructor and will be 'reactive'
class Task
    name: ''
    isRunning: false
    show: true

    constructor: (@name, @isRunning = false, @show = true) ->

Vue creation:
class ViewModel
    search: ''

    constructor: (@view, @tasks, controller) ->
        allowUnsafeNewFunction =>
            @vm = new Vue
                el: @view
                data: {
                    'tasks': @tasks # Array of Tasks
                    'search': @search
                }
                methods: controller
                watch: {
                    'search': @searchTasks
                }

NB, local search property is not being set explicitely in in/by constructor either, but is reactive nontheless.

Vue observer ignores prototype properties, this is by design.

But why then is search reactive. It is not set either...

Because it's initialized in data. Please read http://vuejs.org/guide/reactivity.html


canutils: fix a few debug prints

Summary
Fix a few issues on the debug prints, and also some nxstyle issues
Impact
No impact
Testing
Compile-tested only

Looks like this will collide with https://github.com/apache/incubator-nuttx-apps/pull/940
..will update this once that is merged.

Looks like this will collide with #940 ..will update this once that is merged.

Look like the PR has to rebase with the new master:(

Rebased


RoadmapView: use header instead of Text

Hadn't realised that a header was supported.
Without a header, I couldn't get the header text to scroll, and so ended up with a custom collapsible (but non-standard) solution.

Done


DPDV-8269: Fix failing tests

There hasn't been any update for 4 months and meanwhile it got broken - https://github.com/scalyr/opentelemetry-exporter-dataset/actions/workflows/release-image.yaml?query=branch%3Amain - and it's spamming our slack channel.
This PR is changing few things:

Update subtrees to the latest versions, so that we are using the latest versions. make subtrees-pull
 git push

Fixes all the problems.

To make it simpler in the future, I have written steps, that I have taken into this document - https://sentinelone.atlassian.net/wiki/spaces/DATA/pages/3660055031/Updating+opentelemetry-exporter-dataset
Because of the subtrees, there are thousands of changed files. :/

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

Project coverage is 97.25%. Comparing base (199a47f) to head (bb027ae).
Report is 69 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main      #60      +/-   ##
==========================================
+ Coverage   97.01%   97.25%   +0.24%
==========================================
  Files           5        5
  Lines         469      400      -69
==========================================
- Hits          455      389      -66
+ Misses         11        8       -3
  Partials        3        3

see 5 files with indirect coverage changes

Thanks for fixing this.


Custom Select — has-danger icon is broken

has-danger Icon for custom select is broken. See image

This is because the background size is already set in .custom-select

Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin or CodePen and report back with your link, Bootstrap version, and specific browser and OS details.

Hello,
Just created a live demo. https://codepen.io/surjithctly/pen/OmdwQv
Thank you.

This has already been discussed somewhere on an other issue. The icons aren't meant for select inputs

So, either the icon should be removed from select element or should be adjusted.  I guess it won't be a problem to adjust that for custom select because the default arrow is already hidden.

It is #19852.


Multiple DbContexts, wrong DbContextOptions being injected.

I have multiple modules that run code on start up of my ASP.NET 5 RC 1 application. Each module needs to register its own DbContext. The modules are not aware of eachother. They run the following code to register their paritcular DbContext:-

  services.AddEntityFramework()
             .AddSqlServer()
             .AddDbContext<GcDbContext>(options => options.UseSqlServer(connectionString));

What I notice, when I place a break point in the constructor of a particular DbContext instance, is that the wrong DbContextOptions are being passed in:
 public GcDbContext(DbContextOptions options, ILogger<GcDbContext> logger)
        : base(options)
        {
            Logger = logger;
            // The options here are ones registered for a different DbContext!
        }

I need to have the constructor for unit testing purposes.
Where am I going wrong!? :)

@dazinator you want to use the generic options type:
 public GcDbContext(DbContextOptions<GcDbContext> options, ILogger<GcDbContext> logger)
        : base(options)

BTW we have improved the exception message in RC2 to give more info on this situation

Ahh bingo. Cheers!


feat(nargo): include short git commit in cli version output

Related issue(s)
Resolves #720
Description
Summary of changes
I've added some macros which append the current git hash onto the version returned by clap so we now get the output:
$ nargo --version
nargo 0.1.0 (git:d4154364)

If there are uncommitted changes to the repo then the git hash is suffixed with -dirty to alert us that it's not running the exact code from that commit.
At the same time, I've made clap always pull the version from cargo.toml so we don't need to manually keep this in sync.
Dependency additions / changes
The following dependencies are added to nargo:

git_version as this allows us to get the git hash in a non-hacky way which handles incremental compilation correctly.
const_format in order to format the cargo version and git hash together.

Test additions / changes

Checklist

[x] I have tested the changes locally.
[x] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[x] I have linked this PR to the issue(s) that it resolves.
[x] I have reviewed the changes on GitHub, line by line.
[x] I have ensured all changes are covered in the description.

Additional context

If there are uncommitted changes to the repo then the git hash is suffixed with -dirty to alert us that it's not running the exact code from that commit.

Can you explain this? If changes are not committed, then obviously it is not part of the commit, so I don't understand.

Yes but if I check out a particular commit, make a bunch of changes to the code, and then build nargo without committing those changes then nargo would still report that same commit hash despite it no longer corresponding to the code being run.


Doc (Updates) : Added Weekly Milestones for Competency Passbook

Project
Competency Passbook
Changes
Added weekly milestones for Competency Passbook : Private Blockchain Network, API Gateway Server and ReactJS Frontend.
Notes
None

Changes LGTM @Ansh-Sarkar.
Can you squash your commits now?

@Shruti3004 apologies for the late reply. Have squashed the commits. Thank you.