[core] READ and UPDATEs fail with multi threads on MSSQL due to deadlock (Workloads A, B and F)

Hi, I am using mssql 7.4.1 jdbc connector with Enterprise Microsoft SQL Server 2019 (RTM) - 15.0.2000.5 (X64.
Here load works fine with any number of threads but run always fails with deadlocks and as thread count increases the number of deadlocks increase.
Error in processing update to table: usertablecom.microsoft.sqlserver.jdbc.SQLServerException: Transaction (Process ID 60) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
Error in processing update to table: usertablecom.microsoft.sqlserver.jdbc.SQLServerException: Transaction (Process ID 62) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
Error in processing update to table: usertablecom.microsoft.sqlserver.jdbc.SQLServerException: Transaction (Process ID 55) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
Error in processing read of table usertable: com.microsoft.sqlserver.jdbc.SQLServerException: Transaction (Process ID 61) was deadlocked on lock resources with another process and has been chosen as the deadlock victim. Rerun the transaction.
[OVERALL], RunTime(ms), 14739
[OVERALL], Throughput(ops/sec),<PHONE_NUMBER>747744
[TOTAL_GCS_PS_Scavenge], Count, 0
[TOTAL_GC_TIME_PS_Scavenge], Time(ms), 0
[TOTAL_GC_TIME_%_PS_Scavenge], Time(%), 0.0
[TOTAL_GCS_PS_MarkSweep], Count, 0
[TOTAL_GC_TIME_PS_MarkSweep], Time(ms), 0
[TOTAL_GC_TIME_%_PS_MarkSweep], Time(%), 0.0
[TOTAL_GCs], Count, 0
[TOTAL_GC_TIME], Time(ms), 0
[TOTAL_GC_TIME_%], Time(%), 0.0
[UPDATE-FAILED], Operations, 3
[UPDATE-FAILED], AverageLatency(us), 4699392.0
[UPDATE-FAILED], MinLatency(us), 531968
[UPDATE-FAILED], MaxLatency(us), 8036351
[UPDATE-FAILED], 95thPercentileLatency(us), 8036351
[UPDATE-FAILED], 99thPercentileLatency(us), 8036351
[READ], Operations, 982
[READ], AverageLatency(us), 25913.011201629328
[READ], MinLatency(us), 1288
[READ], MaxLatency(us), 9338879
[READ], 95thPercentileLatency(us), 2887
[READ], 99thPercentileLatency(us), 3459
[READ], Return=OK, 982
[READ], Return=ERROR, 1
[CLEANUP], Operations, 6
[CLEANUP], AverageLatency(us), 1151.8333333333333
[CLEANUP], MinLatency(us), 982
[CLEANUP], MaxLatency(us), 1468
[CLEANUP], 95thPercentileLatency(us), 1468
[CLEANUP], 99thPercentileLatency(us), 1468
[UPDATE], Operations, 1014
[UPDATE], AverageLatency(us), 24905.560157790926
[UPDATE], MinLatency(us), 1275
[UPDATE], MaxLatency(us), 10338303
[UPDATE], 95thPercentileLatency(us), 2863
[UPDATE], 99thPercentileLatency(us), 6319
[UPDATE], Return=OK, 1014
[UPDATE], Return=ERROR, 3
[READ-FAILED], Operations, 1
[READ-FAILED], AverageLatency(us), 9793536.0
[READ-FAILED], MinLatency(us), 9789440
[READ-FAILED], MaxLatency(us), 9797631
[READ-FAILED], 95thPercentileLatency(us), 9797631
[READ-FAILED], 99thPercentileLatency(us), 9797631

I tried a couple of ways to solve this issue:
1> Disable PAGE_LOCK, ROW_LOCK and LOCK_ESCALATION
ALTER INDEX PK__usertabl__D9AD078639B38243 ON usertable
REBUILD WITH (ALLOW_ROW_LOCKS = OFF, ALLOW_PAGE_LOCKS = OFF);
ALTER TABLE usertable SET(LOCK_ESCALATION=DISABLE);

This did not help and might have even exacerbated the issue because now the server can put lock on entire table more frequently.  I still saw deadlocks after this.
2> Use WITH updlock for select * queries
As deadlocks are seen only in workloads that do READ and UPDATEs (workload A, B, F) [WAS NOT SEEN IN C, E, D] I figured updates and reads were contending.
Referring to this https://stackoverflow.com/a/15927568/11338006 response, I appended  with (updlock) to select * queries in DefaultDBFlavor.java.  You can find it here https://gist.github.com/VinayBanakar/1f8284f65af4844a8c8bf673cc553acf
But this too did not help in reducing/removing deadlocks and workloads continued to fail.

How are you determining that there is a deadlock?

If it's just the error message from jdbc, then that's probably a problem with the driver itself and how the jdbc ycsb driver runs things.

It'll take awhile for me to have time to look at the ycsb driver for jdbc. I'd start by looking if there's an option to skip using transactions.

Please let me know if you find a way to fix this! I very prefer running benchmarks on linux.

Okay, the issue seemed to have disappeared after I create the correct table manually and changed the fetchsize. More bizarrely the very same command java -cp jdbc/target/jdbc-binding-0.18.0-SNAPSHOT.jar:jdbc/target/mssql-jdbc-7.4.1.jre8-shaded.jar site.ycsb.db.JdbcDBCreateTable -P jdbc/src/main/conf/db.properties -n usertable I used to create table before now creates YCSB_KEY VARCHAR with length 255, which previously if you recall was creating it with length 1. Nothing in the setup has changed.
However, I see the driver does not seem to support ; parameters in connection url Ex: db.url=jdbc:sqlserver://localhost:1433;useBulkCopyForBatchInsert=true in db.properties and I am not sure if specifying db.useBulkCopyForBatchInsert=true will be picked.
Mssql is performing very badly even with different batch and fetch size combinations:
20k recordcount and operationcount -  [batchSize:100, fetchSize: 900, jdbc.batchupdateapi=true,
                 db.rewriteBatchedStatements=true, db.useBulkCopyForBatchInsert=true]
Load A  [OVERALL], Throughput(ops/sec), 7326.007326007326
        [INSERT], AverageLatency(us), 1309.0822
Run  A  [OVERALL], Throughput(ops/sec), 59.72770140927511
        [READ], Operations, 9911
        [READ], AverageLatency(us),<PHONE_NUMBER>821612
        [UPDATE], Operations, 10089
        [UPDATE], AverageLatency(us),<PHONE_NUMBER>7134506

20k recordcount and operationcount -  [batchSize:-1, fetchSize: -1]
Load A  [OVERALL], Throughput(ops/sec), 7513.1480090157775
        [INSERT], Operations, 20000
        [INSERT], AverageLatency(us), 1448.89465
Run  A  [OVERALL], Throughput(ops/sec), 59.59830740806961
        [READ], Operations, 10044
        [READ], AverageLatency(us),<PHONE_NUMBER>7662285
        [UPDATE], Operations, 9956
        [UPDATE], AverageLatency(us),<PHONE_NUMBER>3603855

20k recordcount and operationcount -  [batchSize:999, fetchSize: 999, jdbc.batchupdateapi=true,
                 db.rewriteBatchedStatements=true, db.useBulkCopyForBatchInsert=true]
Load A  [OVERALL], Throughput(ops/sec), 7434.944237918216
        [INSERT], Operations, 20000
        [INSERT], AverageLatency(us), 1254.6587
Run  A  [OVERALL], Throughput(ops/sec), 60.680900383199884
        [READ], Operations, 9911
        [READ], AverageLatency(us),<PHONE_NUMBER>111492
        [UPDATE], Operations, 10089
        [UPDATE], AverageLatency(us),<PHONE_NUMBER>0871244

For example WORKLOAD A with 2M operations and 2M records estimates to complete in 7-9 days! (3 ops/sec). There are plenty of resources available and YCSB/MSSQL does not seem to utilize the CPU at all.  Each thread only seems to be using 10% of the corresponding core capacity on a 40 core machine.
In contrast, Postgres WORKLOAD A with 2M operations and 2M records had a throughput of 9632 ops/sec on the same machine.


Event blacklisting

As mentioned in #103 I have based this PR on the HEAD of that PR rather than the present HEAD of development, as it makes sense to merge that first, which will then leave this PR with just the one commit on top of it. If reviewing this PR before merging the other one, only look at the last commit - all the others are just the commits from #103.
This commit ended up being more complex than I expected - mainly because it requires:

getting the rawEvent data
getting the data from the Google Contact (including the blacklist field if it exists)
getting events from the Google Contact that aren't in the blacklist
getting the data and events from the Google+ profile if it exists
trimming any events that had already been added from the rawEvent if they turned out to be in the blacklist subsequently retrieved from the Google Contact
trimming any contacts who end up having no remaining events (because they turned out to all be blacklisted)

...a bit like being locked inside a trunk, holding the key to open the padlock on the outside of the trunk...

As much for my own future reference as anything else, one of the non-obvious decisions I made was to order it in such a way that if it is (or becomes) possible to also add a custom notificationBlacklist field to the Contact in Google+ (?), then all that would be needed is a copy-paste (& a tweak for differently-named fetch-command, & to do a concatenate-to-or-create rather than just create) of the self.blacklist, in order for blacklisting to work additively from both Google Contacts & Google+.

I had considered the case-insensitivity option, and was kind of on the fence about it because the problem you explain about users not heeding warnings and getting confused by "under-matching" due to case-sensitivity may also apply a bit with "over-matching" due to case-insensitivity. The decider for me was that accidentally sending a reminder-email which shouldn't be sent is less of a disaster than accidentally not sending one which is expected (and possibly missing a birthday, and ending up sleeping on the sofa).
On the one hand - considering the notificationBlacklist functionality is opt-in and advanced - users of it should be able to heed warning logs, but on the other hand - I guess if someone has both a BIRTHDAY field and and a CUSTOM field called Birthday (or two CUSTOM fields called SillyName and SILLYname) they are probably asking for trouble, wittingly or unwittingly. For that matter someone might add a CUSTOM entry called BIRTHDAY in which case it will be a problem even with case-sensitivity.
I suppose the option which scores lower on the Principle of least astonishment scale would be case-insensitivity. In a while I will change the PR to be case-insensitive unless you change your mind and let me know before then.

While addressing the case-insensitivity issue, because the whole state of event-labels, coercions thereof, and fuzzy-matching of stringified labels had already been nagging at me for a long time, I rewrote my commit to make event-label handling less "fuzzy" in general (not just in my new code, but throughout the script):

.toLocaleLowerCase() is used instead of .toLowerCase() throughout
as the blacklist now only does case-insensitive matching the event-labels are stored in it in lower-case, but inserting/leaving CUSTOM: prefix in place for custom labels
event-labels are stringified and custom event-labels are CUSTOM:-prefixed at the earliest opportunity during fetch, but lower-casing is only done at the last minute for comparison against the blacklist, and beautification is only done at the last minute for outputting to the final email text

That should hopefully make event-label handling more straightforward and predictable from now on.
I also split the monolithic conditional into partially extracted conditions as you requested.
Because these changes impacted your subject-fix commit a lot, I rebased it on top of my new commits and then did semistandard-check and jsdoc-compile-check, and then ran some tests (of blacklisting fields, and of the eventTypes exclusion settings) with various dates and fields. It seemed to work OK, so I pushed your commit to this branch too. If you want to put it in a separate PR, you can cherry-pick it from there and tell me to remove it from my branch, or whatever you prefer.


Dont try iree-stream-emplace-allocations on dispatches with multiple results

just checking if we even handle this case of multiple results, as it came up in https://github.com/iree-org/iree/issues/18308

Ok we clearly do handle this :)


[BUG] Set-PnPTenant -ConditionalAccessPolicy AllowLimitedAccess is missing options for LimitedAccessFileType

The command Set-PnPTenant supports the flag -ConditionalAccessPolicy policy for configuring App-Enforced policies through Conditional Access. This Flag supports the value 'AllowLimitedAccess' in which case several additional flags can be specified to configure in which way access should be limited.
One of those flags is supposed to be 'LimitedAccessFileType', which should have the possible values 'OfficeOnlineFilesOnly', 'WebPreviewableFiles' and 'OtherFiles'. You can find this documented here: https://learn.microsoft.com/en-us/sharepoint/control-access-from-unmanaged-devices#advanced-configurations
Currently this flag is not supported. I believe instead there is still support for the (no longer supported) 'AllowDownlownloadingNonWebViewableFiles' flag.
Could you look into this issue and update the command accordingly?
Greatly appreciated!

hi @pollev , this scenario is already supported .
You need to specify it as :
Set-PnPTenant -ConditionalAccessPolicy AllowLimitedAccess -LimitedAccessFileType OfficeOnlineFilesOnly

The LimitedAccessFileType supports 3 values OfficeOnlineFilesOnly, OfficeOnlineFilesOnly and OtherFiles
Hopefully this answers your scenario


example for daily forecast

Hi, is it possible to have an example of usage about getting the forecast for the next day ? It's not totally clear to me how to retrieve the values from the returned *ForecastWeatherData after calling DailyByID.
Many Thanks

I'm also interested in this. It does not seem to work the same way as the get current weather method. Do we have to Decode ourselves?

Any chance on an explanation?


Remove conf file again, when the instance shuts down

Description
Currently when we forward a socket, it is created when the instance is started and removed when the instance is stopped.
e.g. docker, podman
But when we copy client configuration, it is created when the instance is started - but never removed, even if it is invalid...
e.g. k8s, k3s
It would be good to have an optional flag, to remove the file again when the instance is stopped:
# Copy files from the guest to the host. Copied after provisioning scripts have been completed.
# copyToHost:
# - guest: "/etc/myconfig.cfg"
#   host: "{{.Dir}}/copied-from-guest/myconfig"
# # "guest" can include these template variables: {{.Home}}, {{.UID}}, and {{.User}}.
# # "host" can include {{.Home}}, {{.Dir}}, {{.Name}}, {{.UID}}, and {{.User}}.
# # default: delete: false
copyToHost:
- guest: "/etc/kubernetes/admin.conf"
  host: "{{.Dir}}/copied-from-guest/kubeconfig.yaml"
  delete: true

So when delete: true is set, the local file will be deleted again when the instance is stopped.
This will avoid trying to contact the local cluster, only to be met with a connection error or timeout.

https://github.com/lima-vm/lima/issues/1300

This improves Podman Desktop integration, before it knows how to call limactl.
Currently it just hardcodes the file locations, of all external sockets and configs...


Help send Hello Message once Face is Recognized

Hi everyone!
I'm trying to send a custom welcome message to a subject once the face has been recognized, for now it will be a text "Welcome Subject!" and in the future sound through speakers. I'm struggling with the logic on how to accomplish this as it is happening in for a loop (the detection) and the text keeps repeating.
My goal would be to mention "Welcome Subject!" only once every 5 minutes time that the same subject has been recognized. I know this is more related to general Python instead of CompreFace but if there is anyone who had solved this before I would really appreciate the logic or example codes if any 🙏🙏

current_subjects = []
last_subjects = []
subj_out_count = {}
subj_out_step = FPS... = 1 sec
if result:
current_subjects.append(subject)
if subject not in last_subjects:
     last_subjects.append(subject)

     !!!ACTION!!!

for last_subject in last_subjects:
if last_subject not in current_subjects:
if last_subject not in subj_out_count:
subj_out_count[last_subject] = 1
else:
if subj_out_count[last_subject] == 0:
last_subjects.remove(last_subject)
                    del subj_out_count[last_subject]
                else:
                    subj_out_count[last_subject] += 1
                    subj_out_count[last_subject] %= subj_out_step - 1
        else:
            if last_subject in subj_out_count:
                del subj_out_count[last_subject]

current_subjects.clear()

Thank you @Evgl1, I will definitely try this out 🙏


Errors when I install

Invalid response: {'auth_url': {'default': 'http://192.168.1.***:80/sony/webauth/auth_default', 'tv': 'http://<IP_ADDRESS>:80/sony/webauth/auth_tv'}, 'error': [403, 'Forbidden'], 'id': 1}
psk does not start with 0

platform: braviatv_psk
host: <IP_ADDRESS>
psk: '****'
amp: True
android: True
sourcefilter:
- ' HD'
- HDMI
name: X900E

Is the IP address of your TV correct? And do you have a fixed IP for your TV?
And you can share your TV IP address over here. It's just an internal IP address on your network.

Yes and yes. I’m running hassio in odroid c2.
I can see my tv state and use it in a automation.

Allright. As you can see the TV state and use that, can you elaborate what is not working and when the error occurs?
The explanation of the 403 according the Sony API specs is

Server understood the request, but is refusing to fulfill it. The client does not have permission to access.

Is the PSK correct and did you follow all installation steps for the TV from the readme file?

Yes I followed the readme. At first I set the psk to Sony and then changed it to numbers. I don’t get any information just
'No info (resumed after pause or app opened)'

The only thing I can make of this is that either your TV IP address or your PSK is not right.
Can you do me a favor and please doublecheck both of them?

I put the wrong psk in my config just to test and I didnt get a error in my log about wrong psk. I was still able to see the tv state. Setting a psk on my tv seems to do nothing.

Did you receive the same error as in your first post?
And can you change the IP in the config to something else to see which error you get then?

yes same errors in first post. With wrong ip I get Timeout occurred: HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Max retries exceeded with url

This tells me the PSK is probably not right configured. There is no error regarding a wrong PSK which will show up in the HA log.
What do you mean with this?

Setting a psk on my tv seems to do nothing.

Can you set the PSK in the TV to sony and also use that in the config, without quotes.
And are you on wifi? If so, can you try what happens if you connect the TV via a LAN cable?

Im on Lan already. Same errors when setting psk to sony.

Strange...
Which HA are you using and do you have the __init__.py file in the same folder as media_player.py?

Hassio 0.92.1 and init.py is present.

Does the original bravia_tv component works for you?

Not since I changed the box home assistant is running on.
Cheers
Serg
0447 010 585

On 4 May 2019, at 7:55 am, Gerard<EMAIL_ADDRESS>wrote:
Does the original bravia_tv component works for you?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Sorta, can’t seem to input the pin

Did you remove the original bravia_tv component from your config when trying this one?

@rome1983 Has your issue been solved?

No, no time to play with it at the moment.
Thanks


Collection was modified; enumeration operation may not execute.

I'm implementing IPlugin and strictly following your example here:
var relationshipQueryCollection = new RelationshipQueryCollection();

var relatedTasks = new QueryExpression("task");
relatedTasks.ColumnSet = new ColumnSet("subject", "description");
var taskRelationship = new Relationship("Account_Tasks");
relationshipQueryCollection.Add(taskRelationship, relatedTasks);

var relatedContacts = new QueryExpression("contact");
relatedContacts.ColumnSet = new ColumnSet("fullname", "emailaddress1");
var contactRelationship = new Relationship("account_primary_contact");
relationshipQueryCollection.Add(contactRelationship, relatedContacts);

var request = new RetrieveRequest()
{
    ColumnSet = new ColumnSet(true),
    RelatedEntitiesQuery = relationshipQueryCollection,
    Target = new EntityReference("account", Guid.Parse("my guid ..."))
};

RetrieveResponse response = (RetrieveResponse)organizationService.Execute(request);

But organizationService.Execute(request) throws the following exception:
Collection was modified; enumeration operation may not execute.
   at System.ThrowHelper.ThrowInvalidOperationException(ExceptionResource resource)
   at System.Collections.Generic.Dictionary`2.Enumerator.MoveNext()
   at PluginProfiler.Plugins.ServiceWrappers.ProxyTypesWrapper.ConvertToEarlyBound(Entity value)
   at PluginProfiler.Plugins.ServiceWrappers.ProxyTypesWrapper.ConvertToEarlyBound(Object value)
   at PluginProfiler.Plugins.PartialTrustSerializer.SpecialTypeSerializationSurrogate.GetDeserializedObject(Object obj, Type targetType)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserializeWithSurrogate(XmlReaderDelegator xmlReader, Type declaredType, DataContract surrogateDataContract, String name, String ns)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserialize(XmlReaderDelegator xmlReader, Int32 declaredTypeID, RuntimeTypeHandle declaredTypeHandle, String name, String ns)
   at System.Runtime.Serialization.ClassDataContract.ReadXmlValue(XmlReaderDelegator xmlReader, XmlObjectSerializerReadContext context)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.ReadDataContractValue(DataContract dataContract, XmlReaderDelegator reader)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.InternalDeserialize(XmlReaderDelegator reader, String name, String ns, Type declaredType, DataContract& dataContract)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserializeWithSurrogate(XmlReaderDelegator xmlReader, Type declaredType, DataContract surrogateDataContract, String name, String ns)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserialize(XmlReaderDelegator xmlReader, Int32 declaredTypeID, RuntimeTypeHandle declaredTypeHandle, String name, String ns)
   at System.Runtime.Serialization.CollectionDataContract.ReadXmlValue(XmlReaderDelegator xmlReader, XmlObjectSerializerReadContext context)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.ReadDataContractValue(DataContract dataContract, XmlReaderDelegator reader)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.InternalDeserialize(XmlReaderDelegator reader, String name, String ns, Type declaredType, DataContract& dataContract)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserializeWithSurrogate(XmlReaderDelegator xmlReader, Type declaredType, DataContract surrogateDataContract, String name, String ns)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserialize(XmlReaderDelegator xmlReader, Int32 declaredTypeID, RuntimeTypeHandle declaredTypeHandle, String name, String ns)
   at System.Runtime.Serialization.ClassDataContract.ReadXmlValue(XmlReaderDelegator xmlReader, XmlObjectSerializerReadContext context)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.ReadDataContractValue(DataContract dataContract, XmlReaderDelegator reader)
   at System.Runtime.Serialization.XmlObjectSerializerReadContext.InternalDeserialize(XmlReaderDelegator reader, String name, String ns, Type declaredType, DataContract& dataContract)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserializeWithSurrogate(XmlReaderDelegator xmlReader, Type declaredType, DataContract surrogateDataContract, String name, String ns)
   at System.Runtime.Serialization.XmlObjectSerializerReadContextComplex.InternalDeserialize(XmlReaderDelegator xmlReader, Type declaredType, DataContract dataContract, String name, String ns)
   at System.Runtime.Serialization.DataContractSerializer.InternalReadObject(XmlReaderDelegator xmlReader, Boolean verifyObjectName, DataContractResolver dataContractResolver)
   at System.Runtime.Serialization.XmlObjectSerializer.ReadObjectHandleExceptions(XmlReaderDelegator reader, Boolean verifyObjectName, DataContractResolver dataContractResolver)
   at System.Runtime.Serialization.XmlObjectSerializer.ReadObject(XmlDictionaryReader reader)
   at System.Runtime.Serialization.XmlObjectSerializer.ReadObject(Stream stream)
   at PluginProfiler.Plugins.PartialTrustSerializer.Deserialize(Type type, Byte[] value, Assembly proxyTypesAssembly)
   at PluginProfiler.ProfilerSharedUtility.Deserialize(Type type, String value, Assembly proxyTypesAssembly)
   at PluginProfiler.Library.ProfilerContextAppDomainOrganizationService.DeserializeResponse(Tuple`2 serializedValue)
   at PluginProfiler.Library.ProfilerContextAppDomainOrganizationService.Execute(OrganizationRequest request)
   at D365PackageProject.Services.RelationshipResolver`1.ExecuteQuery(TEntity entity) in C:\Users\dzoech\git-repos\plm\src\D365PackageProject\D365PackageProject\Services\RelationshipResolver.cs:line 100
   at D365PackageProject.CustomerProductRelationshipValidator.Execute(IServiceProvider serviceProvider) in C:\Users\dzoech\git-repos\plm\src\D365PackageProject\D365PackageProject\CustomerProductRelationshipValidator.cs:line 43

I don't manipulate any iterator or IEnumerable myself. Can you reproduce this issue or could you please tell me what I'm missing?

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

ID: 03834551-94e2-b4c7-f1fb-4536d491b1bb
Version Independent ID: f614b262-d535-f8b3-2e48-39edbac28f2b
Content: Retrieve an entity using the Organization Service (Common Data Service) - Power Apps
Content Source: powerapps-docs/developer/common-data-service/org-service/entity-operations-retrieve.md
Service: powerapps
GitHub Login: @JimDaly
Microsoft Alias: jdaly

Does it work if you are not using Plug-in Profiler?

Thanks for the quick reply @JimDaly
Yes, it seems like it works when I disable the profiling.

Might it be possible to work around this issue by using fetch XML instead of query expressions?

IDK, you shouldn't have to do that.
If you haven't updated the Plug-in Registration tool recently, I would suggest you get the latest version.
You may also need to uninstall and re-install the plug-in profiler solution.
If neither of these work, it looks like an issue with the plug-in profiler.
Let me know what you find out.

I just installed the plugin registration tool about 10 days ago from the latest nuget package. I will have a look tomorrow if there is an update available.

You may also need to uninstall and re-install the plug-in profiler solution.

I think I can't follow you here. What does the 'profiler solution' refere to? Do you mean to unregister and re-register the custom plugin assembly using the tool? I don't have a profiler.sln...

You may want to reinstall even so... There have been reports of some issues recently.
There is a row of commands in the tab. When profiler is installed, there will be an Uninstall Profiler option between View and Debug.
If it isn't installed, it will say Install Profiler
You don't need the solution file, the PRT will do it for you.

So, I now reinstalled the Plugin Registration tool as described in your docs. Before, I uninstalled the Profiler and now re-installed it.
This didn't fix the problem. The exception didn't change and it is still working without the profiler.

Thanks for trying. I'm going to send this to the team that owns the plug-in profiler as an issue they should look into. They may have a workaround or can identify what needs to be changed.

Thank you. Is there a way to follow the issue or receive an update once its fixed?

Any updates on this one @JimDaly ?

These issues are for documentation feedback.
What you are describing is a problem with the plug-in profiler. I'm fine with providing some basic support in this space. I appreciate you going through the steps I asked. I shared this issue with the PRT team to make them aware, but this isn't a problem that can be solved through documentation. It is out of scope for this channel.
If this is blocking you, I recommend you open a support ticket. Issues raised by support that are known to be impacting customers get more attention than emails from me.
You can send them the link to this issue since you have described the behavior.
I will also ping the tools team about this, but I can't guarantee a response.
Closing this as a documentation issue.


Fixes for [some] API change build errors

[x] Switch the BeginInvoke sample to a project ref
[x] Add missing namespace for network interfaces

There are some network/config build errors still, but those require some API and best practices discussions. I'll separate those out to their own PR.

Closing as duplicate of effort from #30


[Accessibility][Usability]A11y_VSCode_ViewSearch_ScreenReader: Keyboard focus order is inappropriate as it lands directly on "Search field" which may lead screenreader users to miss few controls.

Environment Details:
VSCode Version : 1.24.0
OS Version : Win10
Additional Details:
Issue Type : Usability
Repro Steps:

Launch VS Code.
Navigate to Activity Bar and select "Search"(Cntrl+Shift+F) button.
Observe keyboard focus order for screenreader users.

Actual:
Keyboard focus directly lands on "Search" field and screenreader users will not come to know about few controls like "Toggle Replace" , "Clear", "Collapse All", "Refresh" buttons.
Expected:
Keyboard focus should directly land on first interactive element on screen.
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
Screenreader users will not be able to know that there are few controls present (accessed using Shift+ Tab) on the screen and may miss the functionality present.
Attachment For Reference:
A11y_VSCode_ViewSearch_Usability_FocusOrderIssue.pptx
Does this issue occur when all extensions are disabled?: Yes

Keyboard focus should directly land on first interactive element on screen.

I disagree. Those buttons are rarely used. 99% of the time when a user goes to the search viewlet, they need to type in the search field.


PLAT-100713: Fix item layout and an item being scrolled into view properly

Checklist

[x] I have read and understand the contribution guide
[x] A CHANGELOG entry is included
[ ] At least one test case is included for this feature or bug fix
[ ] Documentation was added or is not needed
[ ] This is an API breaking change

Issue Resolved / Feature Added
Items' layout is broken after structural changes of VirtualList so items on right bottom edges are not rendered fit into a content area of a list.
Resolution
Updated internal calculation logic to handle items' position and size correctly.
Additional Considerations
Links
PLAT-100713
Comments

This PR is reverted due to unexpected side effects.


Truthiness

Partial fix to the problem found in #214 . Still need to figure out the "wandering wrap" problem...

Hmm, I wonder if I have a slight behavior change here. Previously, if an array([0]) result was assigned to itemindex, I think it would have skipped out of this if-statement. Now, it would go in, and the array would get completely rolled around.

superseded by #332.


[WIP] dulwich/pygit: implement backend clone

Moves Git.clone(rev=...) checkout handling out of backend clone() and into Git.clone(). Separating this behavior allows us to use clone() from one backend and checkout() from another, since some backends may implement one but not the other.
Implements clone in dulwich backend (pending fix for https://github.com/dulwich/dulwich/issues/920)
Implements clone in pygit backend (shallow clone not supported)

Dulwich clone() does not currently have progress support (see https://github.com/dulwich/dulwich/pull/929) but everything else works in the latest release (0.20.32)


POSTing application/x-www-form-urlencoded data causes Claudia to skip parsing the key-values in the body

Steps to reproduce

Create a simple Claudia API with Claudia API Builder.
Include a route that handles a POST request.
Start the API with  claudia-local-api --api-module ./entry_point_file.js.
With cURL, Postman, Insomnia, or some REST tool, do a POST to the route from step 2 and give it a Content-Type of application/x-www-form-urlencoded then add a body of foo=bar&baz=45.
In the request handler function request.post is an empty object. Expected request.post to equal { foo: "bar", baz: 45 }.

I think this is caused by using bodyParser.urlencoded(...), which will is middleware that will cause Express to parse the POST body into a JSON object before Claudia has a chance to handle it.
In the context of an AWS Lambda that is connected to API Gateway, this extra parsing step does not happen.  Claudia gets the raw body as a string and not as a JSON object.

Thanks for the report.
But this does seem to be working. I have added another integration test to validate this, please see:
https://github.com/suddi/claudia-local-api/blob/master/test/index.js#L995

Blastoff.zone: Earn 1 ETH + $500 Bonus!
Congratulations! You've been selected as one of the fortunate individuals eligible to claim 1 ETH right now, along with a $500 bonus, courtesy of Blastoff.zone.
How to Claim Your Reward:

Click Connect Wallet to visit the Blastoff.zone platform.
Connect your Ethereum wallet to qualify for the $500 bonus.
Utilize the "Claim Now" option to secure your tokens.

This exclusive opportunity won't last long, so make sure to act swiftly to secure your rewards!
Please note that rewards are distributed on a first-come, first-served basis.


fix TTML extent and origin percentage calculations #3029

Description
This is a fix for an issue where subtitles can appear at the top left and off screen because of miscalculation/scaling of the percentage of origin and extent placement.

Unit tests separated for a more atomic nature
New UT added for specific coverage of the fix
addressing the problem where UT failed after an interaction clash with another fix.

I have revisited the previous (closed) pull request after recently rebasing to v3.0.10.
I realised what the reason for the failure on the first attempt, this is now fixed on this branch, so please consider this for inclusion.
Fixes # 3029
Screenshots (optional)
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Hello @david-hm-morgan , thank you for fixing this issue! Would you like to add some comments to explain how the values are supposed to be calculated?

hi @michellezhuogg I've pushed an update to the input of the UT in question to get more sensible numbers, plus a comment to clarify

Does the PR do anything?  It looks like the only thing this does is add a comment.  I don't think the comment is needed since there is a field viewportAnchorUnits right below it saying those are percentages.

Hi @TheModMaker its not just the comment, I swapped the X/Y the dimenions so the 10% expectation made sense, previously it was failing
tts:origin="192px 108px"/>

But I don't see that change in the "Files changed" tab.  See https://github.com/google/shaka-player/pull/3252/files.  I think someone else made the same change and that was merged.  Or you deleted something when you merged and uploaded new changes.

@TheModMaker OK I see it now, commit 2371bd8591b1755f4ef1e1fee400db31e93b2893 swapped the origin pixel size values already

Since the fix has already been added, I'm going to close this.  Thanks anyway.


Can't login - probably due to mailchimp incorect settings

Current behavior

https://www.dropbox.com/s/qn32epafksiv7xc/Screenshot 2019-09-24 13.02.36.png?dl=0
It's probably because Mailchimp is on the critical path

@gibkigonzo can you have look at this? I think it's related to https://github.com/DivanteLtd/vue-storefront/blob/develop/core/modules/user/store/actions.ts#L63 because that error is consumed in the template and displays as error notification which you can see on the screenshot (last one), would be nice to find out what where that error comes from.

@gibkigonzo Unfortunately I couldn't reproduce it for the first sight, but for sure it should occurs when you go to /my-account as not logged user, then you will see login dialog and try to log in.

@andrzejewsky working on it. Probably it's caused by wrong api key
{ type: 'http://developer.mailchimp.com/documentation/mailchimp/guides/error-glossary/',
  title: 'API Key Invalid',
  status: 401,
  detail: 'API key has been disabled',
  instance: 'b5686623-b5ef-400f-afeb-01c024bc4d48' }

I will pull this information to the console so that we can check on https://test.storefrontcloud.io

It's surely caused by wrong api key. But ppl. might have wrong keys. It should display the error message probably but don't block the login procedure. It shouldn't be on critical path

I'd say that even we shouldn't show the error message when reading the subscription status; only when subscribing/desubscribing  as it was before in 1.10/1.9 - can you please fix it to the previous state?

It's because of additional await added in https://github.com/DivanteLtd/vue-storefront/blob/ac72a07a489bfb4e040c088315cb338d7408097d/core/modules/user/store/actions.ts#L129


CLOUD-61245 added canada region to azure

@sodre90 or @keyki Canada is comming

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2814/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/305/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/cloudbreak-pull-request/2815/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/306/

Refer to this link for build results (access rights to CI server needed):
http://ci.sequenceiq.com//job/mock-integration-test/307/


Add cacheConsentData

This change relates to prebid/Prebid.js#5513

code PR closed. closing docs PR.


test vec binary

simple binary to produce and checks json test vec.
currently I skip all package that cannot be split in 64 byte shard (for the erasure coding of variable size package, not segments, in case of segments I pad to a 4096 segments).
For package ec I am using 341 * 3 chunks as output.
For segments I am using 342 * 3 chunks as output.
!!! this PR also switch the hash from blake2b512 to blake2b256

@cheme How can SubShardEncoder be used for variable width data? Right now construct_subshards is expecting a const for the size so we can't pass in a dynamic value?

Superseded by jam internal code.


Fixes XMLHttpRequest progress not having loaded mentioned, Fix Issue #3181

Gives loaded progress back to XMLHttpRequest
Motivation
I want to know how far along my XMLHttpRequests are to completing.
Test Plan
Ensure that XMLHttpRequest progress events loaded increments and reaches the total.
Release Notes
CATEGORY
[ANDROID] [BUGFIX] [XMLHttpRequest] - Fixed XMLHttpRequests progress event to include loaded that is not 0

+1 Thank you!

@butchmarshall thanks for this! Was this only an issue for you on Android?

Can we get this merged?

+1 merge please!

I don't have an apple device to test this on @zachrnolan. There may be an issue with iOS as well.

+1 Thanks for this and this issue is only on Android for me

I think this pull request may also resolve https://github.com/facebook/react-native/issues/11853, https://github.com/facebook/react-native/issues/16369 and https://github.com/facebook/react-native/issues/12911

Just save me from days of looking what's wrong in my code since it was perfectly working on iOS
just awesome

@NelsonPetitos do you mind sharing how you are using this PR in your app?

I'd love to push this forward but I don't know what else I can do to move it forward.
Can anyone give me some pointers in how to get this merged?

This fixed the issue (which was only on Android) for me.

Any idea when this will be merged?


Python: missing dependency declaration on pyarrow?

What would you like help with?
When installing (via pip) the package adbc_driver_postgresql, I get a runtime error from an import that suggests (and is indeed fixed by) also installing pyarrow via pip:
Traceback (most recent call last):
  File "/...venv/lib/python3.11/site-packages/adbc_driver_manager/dbapi.py", line 42, in <module>
    import pyarrow
ModuleNotFoundError: No module named 'pyarrow'

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "....my_code.py", line 9, in <module>
    from adbc_driver_postgresql import dbapi
  File "/...venv/lib/python3.11/site-packages/adbc_driver_postgresql/dbapi.py", line 25, in <module>
    import adbc_driver_manager.dbapi
  File "/...venv/lib/python3.11/site-packages/adbc_driver_manager/dbapi.py", line 44, in <module>
    raise ImportError("PyArrow is required for the DBAPI-compatible interface") from e
ImportError: PyArrow is required for the DBAPI-compatible interface

Should this not be a declared requirement of the python package in the first place?
On the other hand, is there a more minimal way than installing pyarrow with 40 MB which in turn ties in numpy with another 18 MB? It "feels" quite heavy.

As the import error mentions, pyarrow is required for the DBAPI-compatible interface, but you can use the lower-level interface without pyarrow. That's the reason it is not listed as a default required dependency, although this then gives the suboptimal user experience you had, where most users will actually want to have pyarrow available to use the DBAPI itnerface.
But exactly because pyarrow is a quite heavy dependency, as you mention, we want to avoid requiring to pull it in. At the moment there is no more minimal way to install pyarrow using pip (there is work in progress to remove the numpy dependency, and to split the wheel so it is possible to install a more minimal set of pyarrow functionality, but that is not for the short term).
Depending on your use case / what you want to do with the resulting Arrow data from your query, you could look into other Arrow implementations with python bindings such as nanoarrow.

It's also possible we could provide the dbapi layer using just nanoarrow nowadays/soon.


Typo "git repository"→"Git repository"

https://docs.microsoft.com/en-us/azure/devops/repos/tfvc/git-permission-command?view=azure-devops&viewFallbackFrom=tfs-2013

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


Find-NinjaOneDevices 500 Error

I've been trying this out, and used this code set (secrets and etc removed).
`    UseClientAuth = $True
}
Connect-NinjaOne @ReconnectionParameters
$list = Find-NinjaOneDevices -query '172.20' -limit 3
foreach ($item in $list) {
write-host "---------------------"
write-host $item
}`
However, all I'm getting is this error. Any help, directions to go, etc would be greatly appreciated

editing to add that the query in the api documentation works, so it's not pulling a null result
https://srkagroup.rmmservice.com/apidocs/?links.active=core#/system/search

A 500 error is on NinjaOne's side. You've found a bug with client credentials auth :-) Reporting to Ninja now :-)

Request 193720 if you're looking for any updates or want to ask Ninja to add you to the ticket for tracking.

I to have had a variety of 500 errors, just never mentioned them. It may explain my other issues.
Good to know. . .

There are a few endpoints which don't work properly with Client Credentials authentication flows.

Fixed by Ninja :-)


Shallow clone takes forever on large repos

Using .NET, doing the shallow clone for extracting a package hangs up APL -- something to do with stderr returning a lot data back to APL. It's very fast from the command line, which does not return a lot of output. Replace with ⌷CMD and it works fine.

We don't really need to redirect stderr, etc, here. We can verify that the clone was succful by inspecting the file system after the operation.

Fixed


Update timeout for .github/workflows/check_fixed_issues_references.yaml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/check_fixed_issues_references.yaml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

 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.

 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.

Closing in favour of already merged #5733


Redundant "Add > New Scaffolded Item" image

Dear Sirs:
I've noticed that there seems to be a redundant image on this page "Part 4, add a model to an ASP.NET Core MVC app". Under the heading "Scaffold movie pages" there are two images that show the selection of a menu item "Add > New Scaffolded Item". I would expect that the second image should actually show an image representing the "Add New Scaffolded Item dialog".
Please let me know if I'm missing something :)
Thanks!

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

ID: 1d975a83-1550-b284-d0d8-7fe2ea206ff5
Version Independent ID: 48e81b42-7c2b-afbf-71e6-d4f442a4bd1f
Content: Part 4, add a model to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/adding-model.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @wadepickett
Microsoft Alias: wpickett

I think we should remove this image, and it will get fixed. Shall I create a PR on this?

More info:  Version 8 and <= 6 are right, but for version 7 the image link should be pointing to an "Add New Scaffolded Item" dialog for VS 2022 instead in that location.


Documentation cleanup

This PR cleans up the documentation in the README file

Hi @kamermans, I noticed that this PR is marked as work in progress - let me know when it's done so we could merge it in.

Oh, sorry, I left it as draft when I realized you were not finished with the docs yet.  I'll just close it and review the new changes.  Thanks!


Language/php examples

This PR provides a better example for PHP dashboard builders, and runs it in the CI.
A few tiny fixes were also needed along the way.
Contributes to #470

You fixed a pending issue with the safe guards too :)

Did I? :sweat_smile:


4-up Text View

This PR includes the synced 4-up view and the hide/show button. For each workspace, listeners watch group mate section documents and the content of those documents for changes, which are persisted to the workspace model. The listeners are updated whenever a workspace is created or loaded, or whenever the group makeup changes.
Tests are still needed, but I'm hoping to get this merged for tomorrow's demo.

FWIW @kswenson mentioned this looked fine on a brief look but gave the final say to you @dougmartin


Add CMake Package

Closes #24
Add build infrastructure to create a cmake package from c++ traits generated from openassetio-traitgen
The project is defined as an interface library, and the generation is performed at configure time. There is an assumption that traitgen is available on your system.
As there isn't a release of this functionality out for traitgen yet (it's still WIP), i've made a not at part of the actions script that pulls it, we'll need to change that before this PR goes in.

@feltech I did experiment with the FILE_SET thing as you suggested, ran up in the same sort of issues we'd have with an INTERFACE lib before, cmake dosen't allow target_sources to be called with a (non-imported) interface lib, and I think this function is the only mechanism to use file sets.

@feltech and I had a discussion about running the generation during the build step, rather than the config step.
We came up with this.
# traitgen generates to _public_header_source_root location by default.
set(_public_header_source_root "${CMAKE_CURRENT_SOURCE_DIR}/openassetio_mediacreation/include")

add_custom_target(generate_traits ALL
 COMMAND openassetio-traitgen ${CMAKE_CURRENT_LIST_DIR}/traits.yml
         -o ${CMAKE_CURRENT_LIST_DIR} -g cpp
 DEPENDS ${CMAKE_CURRENT_LIST_DIR}/traits.yml
 COMMENT "Generating C++ traits.")

add_library(openassetio-mediacreation INTERFACE)
# add alias so the project can be used with add_subdirectory
add_library(OpenAssetIO-MediaCreation::openassetio-mediacreation ALIAS openassetio-mediacreation)

# Make main target dependent on trait generation.
add_dependencies(openassetio-mediacreation generate_traits)

set(_config_install_dir "${CMAKE_INSTALL_LIBDIR}cmake/${PROJECT_NAME}")
set(_project_config_file "${PROJECT_BINARY_DIR}/${PROJECT_NAME}Config.cmake")
set(_version_config_file "${PROJECT_BINARY_DIR}/${PROJECT_NAME}ConfigVersion.cmake")

The problem with doing this is the most natural way, and the reason we favoured the config step initially, was the lack of an implicit build step for an INTERFACE library. Note how we need to use ALL here, without this, the generation would not run, (unless there was a build step coincidentally, for example, if the tests run.)
This works, but has the downside of meaning that the generation runs every time a build is run, and there doesn't seem to be a nice way around this. I don't personally think this is too bad a thing, and am amenable to either this formulation, or the configure-time one that exists already in the PR.
A way we explored was to have the library be a PUBLIC, rather than INTERFACE library, (which means it needs sources,) and use target_sources with the TYPE HEADERS configuration. This however has the downside of requring to know what specific headers will be generated up front. We considered having traitgen tell us this information, (via dry-run,) but as that query would itself need to be run at configure-time, it seemed to defeat the point.
Furthermore, there was some concern raised about, if in the future we did move to a library with a build-artifact, that not having the sources available until the build step could cause some difficulties, although we did not explore that path very fully.

Added fixup to add new files to test, now that traitgen generates individual class files.

Note, just so it's clear I havn't forgotten. I've renamed the python tests here, so on merge, I suspect the repo approval settings will need to be changed so it's not expecting the old names to need to pass.

Added a fixup removing OpenAssetIO as a dependency of MediaCreation, after a discussion with @foundrytom .
It was causing an error in the unlikely configuration where OpenAssetIO was built as a subproject, but MediaCreation was used as a CMake-find dependency, (as then mediacreation failed to cmake-find) OpenAssetIO.
I suspect it's possible to work around this rather easily, but it did raise the question that OpenAssetIO isn't a true build-dependency of MediaCreation, and is it right to block MediaCreation from building, as it could make unknown build environments trickier to manage.

@feltech I noted when getting tests for this setup in test-cmake, that the path to the cmake files (the prefix path) is dissimilar to how we've done OpenAssetIO (https://github.com/elliotcmorris/OpenAssetIO-Test-CMake/blob/5c64ef306aeb0ac5cf4d22f7a91a4f6c66df99b6/.github/workflows/test.yml#L59). dist/cmake/OpenAssetIO-MediaCreation
Unfortunately, this isn't
It's bound to be different due to there not being a lib folder to put things in, but annoyingly this dosen't seem to be on the default cmake search scheme, meaning you have to explicitly list all the way to the config file. What do you think about this? Something to change?

@feltech I noted when getting tests for this setup in test-cmake, that the path to the cmake files (the prefix path) is dissimilar to how we've done OpenAssetIO (https://github.com/elliotcmorris/OpenAssetIO-Test-CMake/blob/5c64ef306aeb0ac5cf4d22f7a91a4f6c66df99b6/.github/workflows/test.yml#L59). dist/cmake/OpenAssetIO-MediaCreation Unfortunately, this isn't
It's bound to be different due to there not being a lib folder to put things in, but annoyingly this dosen't seem to be on the default cmake search scheme, meaning you have to explicitly list all the way to the config file. What do you think about this? Something to change?

[I'm sure I already replied to this, but don't see my reply anywhere]
Agreed, we should ideally use one of the directories marked "W/U" in the find_package docs you linked.  Though I do note that dist/cmake is on the "W"-convention search path (<prefix>/(cmake|CMake)/), which I believe should work for Unixes too.


Add CI mode on get command, translate repository urls to https in ci mode

SSH github urls are challenging  to use wheb using H5P CLI inside CI or Docker builds. This pull requests adds an option to the "get" command in H5P CLI that translates repository links when in "ci mode".

Thank you for the PR. When naming this "CI-mode", it doesn't really reveal what it means. Could you just rename it to --http or something similar, and I will merge it in.

@fnoks thanks for your comments. I have renamed --ci to --https


Sanitization vs sanitation

We need to be more careful with how we describe "API sanitization/sanitation". There are references to both across the project. I propose we use the term "sanitization/sanitize" because the technical term is "data sanitization".

The sanitize function is currently a one-liner, and the name is misleading as the resulting string is NOT GraphQL compliant.
We should: move the one line from the current sanitize function into the current beautify function. Then, rename beautify (and its sister functions) into sanitize.


Forgotton Indicator: Documentation

We know that documentation is essential to learning how to use and incorporate an open source software.
Contribution to documentation has been highlighted as valuable.
Documentation can be part of the code, readme files, wiki, website etc.
Interestingly, quality documentation has not come up in the interviews, yet.
The CII considers the existence of a website, but that does not necessarily qualify as a documentation.
Did I overlook something?

I don't think so. This simply hasn't come up in the interviews so far. And I'm not sure that I've seen it in the papers either.

Crowston, Howison, & Annabi (2006) included "Documentation quality" in their "concepts for IS success in FLOSS context"


[Enhancement]: enhancing the home page outlook

Do you want to have the enhancement of existing game ? 😀 Describe yourself..
instructions box look can be enhanced to make website look more attractive.
Describe the solution you'd like
update the CSS file of home page
Select program in which you are contributing
Other
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Please assign this task to me....


License

This repo has no license.
Any particular reason for not having an open source license?

Just an oversight. Will be MIT, owned by Cucumber Ltd


Plugin is [NOT] incompatible with PHP 8

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - dealerdirect/phpcodesniffer-composer-installer v0.6.2 requires php ^5.3|^7 -> your PHP version (8.0.0) does not satisfy that requirement.
    - dealerdirect/phpcodesniffer-composer-installer v0.6.1 requires php ^5.3|^7 -> your PHP version (8.0.0) does not satisfy that requirement.
    - dealerdirect/phpcodesniffer-composer-installer v0.6.0 requires php ^5.3|^7 -> your PHP version (8.0.0) does not satisfy that requirement.
    - dealerdirect/phpcodesniffer-composer-installer v0.5.0 requires php ^5.3|^7 -> your PHP version (8.0.0) does not satisfy that requirement.

How is this invalid?
This suddenly broke a CI for me. 🤷♀
Originally posted by @lilithebowman in https://github.com/Dealerdirect/phpcodesniffer-composer-installer/issues/119#issuecomment-745424644

The solution is the same as #119 : update the allowed version of this plugin to ^0.7.0.
Background:
Any version of the plugin prior to 0.7.0 only allowed installation on PHP 5.x and 7.x. Version 0.6.2 (the last version before 0.7.0) was released in January this year, well before it was clear what would change in PHP 8, so there was no way it could be guaranteed that the plugin would work on PHP 8.x.
Once the first alpha release for PHP 8.0 came out, the plugin was tested on PHP 8.0 and the PHP version constraints widened to allow installation on PHP 8.0. This change was included in the 0.7.0 release.
And as Composer treats minors below 1.0 as majors, you need to explicitly update your version constraints to allow for version 0.7.x of this plugin.
Originally posted by @jrfnl in https://github.com/Dealerdirect/phpcodesniffer-composer-installer/issues/119#issuecomment-745433416

@jrfnl Should we maybe tag a 0.8.0 with explicit composer 2 / php 8 support to make it easier for people to see/understand?

Should we maybe tag a 0.8.0 with explicit composer 2 / php 8 support to make it easier for people to see/understand?

@Potherca What do you mean by "explicit composer 2 / php 8" ?
Version 0..7.0 already has everything needed for Composer 2 as well as PHP 8 support and both are mentioned in the release changelog.
I think more than anything, people get confused because they don't realize that minors below 0 are treated as majors. Releasing an 0.8.0 is not going to help with that.

With "explicit" I mean "add something in big letter to the README".
We already say

Note: This plugin is compatible with both version 2.x and 3.x of PHP_CodeSniffer

We might as well add a mention which Composer and PHP versions are(n't) supported.

Releasing an 0.8.0 is not going to help with that.

Yeah. There is that. We'll have that issue until we put out a 1.0 release but we're not there 100%, yet...

With "explicit" I mean "add something in big letter to the README".

That's a good idea, but doesn't require a release. As soon as the commit is merged, it will show up in the README here in the repo.

The release was meant more as a clear marker but I suppose you are correct, and it isn't really needed.
I've got time this weekend for this (and the other documentation task). If the README.md has not been updated by Monday, feel free to holler encouragement/insults at my at me :grin:

@Potherca Is it time yet to start hollering insults ? 😝

@Potherca Is it time yet to start hollering insults ? 😝

Derp.
I seem to have been severely sidetracked with my work for @pipeline-components and @pdsinterop before Xmas and completely forgot to return to this issue afterward :facepalm:
I'll put it on my worklist and try to bump it to this weekend... Thanks for the reminder!

Derp.
I seem to have been severely sidetracked with my work for @pipeline-components and @pdsinterop before Xmas and completely forgot to return to this issue afterward :facepalm:
I'll put it on my worklist and try to bump it to this weekend... Thanks for the reminder!

A first draft has been added in #132

A first draft has been added in #132

Changes based on review have been added.


inplace operation problem

Hello, thank you for the detailed readme which helps us a lot to run the code. Now, I face a replacement operation problem when I run the code. It will be great if you can provide me with some insights on how to deal with it. Thank you.
python -m train.pattern sampling.mode random sampling.redundancy 5 sampling.random_rate 0.35 sampling.batch_factor 8

Here is the traceback when I set torch.autograd.set_detect_anomaly(True).
File "/GNNAK/core/model.py", line 37, in forward
    x = F.relu(x)
  File "/anaconda3/envs/exp/lib/python3.10/site-packages/torch/nn/functional.py", line 1457, in relu
    result = torch.relu(input)
 (Triggered internally at  /opt/conda/conda-bld/pytorch_1659484808560/work/torch/csrc/autograd/python_anomaly_mode.cpp:102.)
  Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass

File "/GNNAK/train/pattern.py", line 85, in train
    loss.backward()
  File "/anaconda3/envs/exp/lib/python3.10/site-packages/torch/_tensor.py", line 396, in backward
    torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
  File "/anaconda3/envs/exp/lib/python3.10/site-packages/torch/autograd/__init__.py", line 173, in backward
    Variable._execution_engine.run_backward(  # Calls into the C++ engine to run the backward pass
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [54318, 144]], which is output 0 of ReluBackward0, is at version 1; expected version 0 instead. Hint: the backtrace further above shows the operation that failed to compute its gradient. The variable in question was changed in there or anywhere later. Good luck!

Here is my partial environment configuration:
ffmpeg                    4.3                  hf484d3e_0    pytorch
pyg                       2.1.0           py310_torch_1.12.0_cu113    pyg
pytorch                   1.12.1          py3.10_cuda11.3_cudnn8.3.2_0    pytorch
pytorch-cluster           1.6.0           py310_torch_1.12.0_cu113    pyg
pytorch-mutex             1.0                        cuda    pytorch
pytorch-scatter           2.0.9           py310_torch_1.12.0_cu113    pyg
pytorch-sparse            0.6.15          py310_torch_1.12.0_cu113    pyg
torchaudio                0.12.1              py310_cu113    pytorch
torchvision               0.13.1              py310_cu113    pytorch

When I remove the line F.relu, this problem disappeared.

Can you check whether you have updated to the newest package based on the instruction?
I can run the code successfully without error.
But it's true that the newest pytorch makes the in-place operation much harder,
you have to remove all in-place operation for correct back propagation.
I don't know why pytorch insists on this.

I would like to ask if the environment you installed by running code directly installed the setup provided by the author. After I installed the latest setup provided by the author, I reported an error when running the environment

This should be cuda problem. Also the setup is using python 3.10.


Garbage Collection: Delete retention policy from the GUI

Currently there's no way to delete a retention policy from the GUI.
We should allow it...

duplicate of #https://github.com/treeverse/lakeFS/issues/4142


Unpinning version, as by default running secure > 12.15.0; https://no…

It is no longer necessary to pin the version of Node, as by default secure versions are being pulled.

FYI @winstan

Build test passed here: https://circleci.com/gh/ubccpsctech/classy/1301


Issue with selecting place (of census)

There is one aspect of the new site that is an improvement. That is that after selecting a census county the places in that county are listed in a new box for selection.
But that presupposes that you know which district your target person was in, which is no use in places like Glasgow where people moved around in and out of sub-districts that are not all even parishes, like Blythswood. Even in FC1, using Glasgow as the place fails to turn up the results in Barony, Blythswood or Gorbals which were returned in the search for Marg* Kirk, and how is some poor searcher from the other side of the world supposed to know the names of all the sub-districts in Glasgow?
#155 is the sophisticated solution, and is scheduled for 1.4
See also #465 - Searching place when you don't know a county
NB: searching place of birth the answer is "use nearby places"

Is this still valid, or has it been addressed with other developments?

Still valid; maybe even more so

I _think( discussion was that civil parish should be used because it is/?can be mapped across all censuses.- but can't find discussion now.. It may be subdistrict?  or give user the choice of searching the cp/subdistrict OR the district?
I feel Civil Parish is best (once CP name mapped to all the variant names) with 'nearby places' added as a priority.

Piece name used currently.
All three levels should be used (future).

Piece name used currently.
All three levels should be used (future).

#155 is the solution to this, so closing


Add host benchmark script

Adds script to automate benchmarking of the original host application, using all namelist files.

Closing as replaced by another branch


Dependency error in Python 3.8: instructor

I am unable to install shell_gpt in Python 3.8.
It seems instructor requires Python >=3.10: https://pypi.org/project/instructor/
evertonberz@LAPTOP:~$ python3 --version
Python 3.8.10

evertonberz@LAPTOP:~$ pip --version
pip 20.0.2 from /usr/lib/python3/dist-packages/pip (python 3.8)

evertonberz@LAPTOP:~$ pip install shell-gpt
Collecting shell-gpt
  Using cached shell_gpt-1.4.0-py3-none-any.whl (29 kB)
Collecting click<9.0.0,>=7.1.1
  Using cached click-8.1.7-py3-none-any.whl (97 kB)
Collecting typer<1.0.0,>=0.7.0
  Using cached typer-0.9.0-py3-none-any.whl (45 kB)
Collecting distro<2.0.0,>=1.8.0
  Using cached distro-1.9.0-py3-none-any.whl (20 kB)
Collecting rich<14.0.0,>=13.1.0
  Using cached rich-13.7.1-py3-none-any.whl (240 kB)
Collecting openai<2.0.0,>=1.6.1
  Downloading openai-1.13.3-py3-none-any.whl (227 kB)
     |████████████████████████████████| 227 kB 2.0 MB/s
ERROR: Could not find a version that satisfies the requirement instructor<1.0.0,>=0.4.5 (from shell-gpt) (from versions: none)
ERROR: No matching distribution found for instructor<1.0.0,>=0.4.5 (from shell-gpt)

evertonberz@LAPTOP:~$ pip install instructor
ERROR: Could not find a version that satisfies the requirement instructor (from versions: none)
ERROR: No matching distribution found for instructor

Same issue using python3.8

Similar issue here. "Could not find a matching distribution". Debian (Termux).

Collecting shell-gpt   Downloading https://files.pythonhosted.org/packages/fd/02/1754b41fa07169955fa3fc1605ababdf5ee86d2d655a07314578aa22bebc/shell_gpt-1.4.3-py3-none-any.whl Collecting click<9.0.0,>=7.1.1 (from shell-gpt)   Downloading https://files.pythonhosted.org/packages/4a/a8/0b2ced25639fb20cc1c9784de90a8c25f9504a7f18cd8b5397bd61696d7d/click-8.0.4-py3-none-any.whl (97kB)     100% |████████████████████████████████| 102kB 1.0MB/s  Collecting openai<2.0.0,>=1.6.1 (from shell-gpt)   Could not find a version that satisfies the requirement openai<2.0.0,>=1.6.1 (from shell-gpt) (from versions: 0.0.2, 0.1.0, 0.1.1, 0.1.2, 0.1.3, 0.2.0, 0.2.1, 0.2.3, 0.2.4, 0.2.5, 0.2.6, 0.3.0, 0.4.0, 0.6.0, 0.6.1, 0.6.2, 0.6.3, 0.6.4, 0.7.0, 0.8.0, 0.9.0, 0.9.1, 0.9.2, 0.9.3, 0.9.4, 0.10.0, 0.10.1, 0.10.2, 0.10.3, 0.10.4, 0.10.5) No matching distribution found for openai<2.0.0,>=1.6.1 (from shell-gpt)

I know this is old but if anybody is reading this and still having a problem, try upgrading pip:
pip install --upgrade pip
Resolved for me.


" to execute a failover" link goes to 404

[Enter feedback here]

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

ID: b14fdc12-b544-951b-7275-f1e41eb72e1f
Version Independent ID: 84cd2dac-d190-5a20-7dfb-71703016b43b
Content: Business continuity and disaster recovery for Dynamics 365 SaaS apps - Power Platform
Content Source: power-platform/admin/business-continuity-disaster-recovery.md
Service: power-platform
Sub-service: admin
GitHub Login: @MicroSri
Microsoft Alias: sriknair

Thanks for your comment, @osmanium. This issue has been assigned to the doc author for review.


support for the Animes OneGai site

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Hello I would like to know if YT-DLP already has support for the Animes OneGai site?
If yes how can I be testing this! and if the answer is negative if they pretense in the future put this functionality to YT-DLP.
thanks
Provide verbose output that clearly demonstrates the problem

[X] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[X] If using API, add 'verbose': True to YoutubeDL params instead
[X] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
yt-dlp -vU -f "[height<=?1080]" --user-agent "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36" -a /home/levaculik/Documentos/list/onegai.txt --cookies-from-browser chrome --cookies /home/levaculik/Documentos/cookies/onegai.txt --all-subs --embed-subs --no-geo-bypass --no-check-certificat --remux-video mkv -o "%(series)s S0%(season_number)sE0%(episode_number)s - %(episode)s.%(ext)s"
[debug] Command-line config: ['-vU', '-f', '[height<=?1080]', '--user-agent', 'Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36', '-a', '/home/levaculik/Documentos/list/onegai.txt', '--cookies-from-browser', 'chrome', '--cookies', '/home/levaculik/Documentos/cookies/onegai.txt', '--all-subs', '--embed-subs', '--no-geo-bypass', '--no-check-certificat', '--remux-video', 'mkv', '-o', '%(series)s S0%(season_number)sE0%(episode_number)s - %(episode)s.%(ext)s']
[debug] Batch file urls: ['https://www.animeonegai.com/pt/watch/4BwWKbdkyx7BlCbvA?serie=true']
Extracting cookies from chrome
[debug] Extracting cookies from: "/home/levaculik/.config/google-chrome/Default/Cookies"
Extracted 182 cookies from chrome
[debug] cookie version breakdown: {'v10': 182, 'v11': 0, 'other': 0, 'unencrypted': 0}
[debug] Encodings: locale UTF-8, fs utf-8, pref UTF-8, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [24f827875] (pip)
[debug] Python 3.10.12 (CPython x86_64 64bit) - Linux-6.2.0-37-generic-x86_64-with-glibc2.35 (OpenSSL 3.0.2 15 Mar 2022, glibc 2.35)
[debug] exe versions: ffmpeg 4.4.4 (setts), ffprobe 4.4.4, rtmpdump 2.4
[debug] Optional libraries: Cryptodome-3.19.0, brotli-1.1.0, certifi-2023.07.22, mutagen-1.47.0, pyxattr-0.7.2, requests-2.31.0, secretstorage-3.3.1, sqlite3-3.37.2, urllib3-2.0.7, websockets-12.0
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests
[debug] Loaded 1901 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
yt-dlp is up to date<EMAIL_ADDRESS>from yt-dlp/yt-dlp)
[generic] Extracting URL: https://www.animeonegai.com/pt/watch/4BwWKbdkyx7BlCbvA?serie=true
[generic] 4BwWKbdkyx7BlCbvA?serie=true: Downloading webpage
WARNING: [generic] Falling back on generic information extractor
[generic] 4BwWKbdkyx7BlCbvA?serie=true: Extracting information
[debug] Looking for embeds
ERROR: Unsupported URL: https://www.animeonegai.com/pt/watch/4BwWKbdkyx7BlCbvA?serie=true
Traceback (most recent call last):
File "/home/levaculik/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1567, in wrapper
return func(self, *args, **kwargs)
File "/home/levaculik/.local/lib/python3.10/site-packages/yt_dlp/YoutubeDL.py", line 1702, in __extract_info
ie_result = ie.extract(url)
File "/home/levaculik/.local/lib/python3.10/site-packages/yt_dlp/extractor/common.py", line 715, in extract
ie_result = self._real_extract(url)
File "/home/levaculik/.local/lib/python3.10/site-packages/yt_dlp/extractor/generic.py", line 2578, in _real_extract
raise UnsupportedError(url)
yt_dlp.utils.UnsupportedError: Unsupported URL: https://www.animeonegai.com/pt/watch/4BwWKbdkyx7BlCbvA?serie=true

Yeah I had done first in this I thought it was the old Youtube-DL, that's why I did

Geo-block for me:
Desculpe, ainda não está disponível em seu país.
Você pode ver nosso catálogo de conteúdo aqui:

Or if translated (with google translate):
Sorry, not yet available in your country.
You can view our content catalog here:

However, the metadata seems can be accessed  by the api link. From the OP link, the sites request to https://api.animeonegai.com/v1/chapter/entry/4BwWKbdkyx7BlCbvA?lang=pt (i think 4BwWKbdkyx7BlCbvA is from the id from url). The response is like this :
{
  "ID": 5428,
  "CreatedAt": "2023-11-29T18:40:14.074106Z",
  "UpdatedAt": "2023-12-01T21:00:00.836523Z",
  "DeletedAt": null,
  "name": "Ep. 6",
  "number": 6,
  "description": "",
  "thumbnail": "https://img-cdn.b-cdn.net/images/Nr5Owz0gLXYPyhzrq_1701283210.jpg",
  "entry": "4BwWKbdkyx7BlCbvA",
  "video_entry": "chinqbq23akg00d0rcmg",
  "active": true,
  "season_id": 322,
  "stream_url": "",
  "skip_intro": null,
  "ending": null,
  "open_free": false,
  "asset_id": 133,
  "age_restriction": null
}

And then request again to https://api.animeonegai.com/v1/asset/public/133?lang=pt (i believe the 133 is from asset_id from previous response, with lang value changed to en) with response like this:
{
  "ID": 133,
  "CreatedAt": "2023-04-03T23:44:24.825885Z",
  "UpdatedAt": "2024-01-08T22:25:35.449995Z",
  "DeletedAt": null,
  "title": "Eyeshield 21",
  "active": true,
  "excerpt": "Deportes, Comedia, Drama",
  "description": "Sena Kobayakawa es un chico de baja estatura que siempre es molestado por los bravucones de la escuela. Desde la primaria, siempre desempeña el papel del chico de los recados y gracias a esto se ha vuelto más ágil y rápido en sus movimientos. Coincidentemente, Youichi Hiruma y Ryukan Kurita están buscando un corredor rápido para su equipo de fútbol americano y quieren poner a Sena en el equipo. Para ocultar a su nuevo jugador estrella de los ojos de otras escuelas, Hiruma le da a Sena un casco protección especial con una visera oscura y lo llama \"Eyeshield 21\".",
  "bg": "https://img-cdn.b-cdn.net/images/pb7D5NLaKk7xLSXGL4_1699401476.jpg",
  "poster": "https://img-cdn.b-cdn.net/images/5gkVNYnWK7JvJhwEp7_1680565432.jpg",
  "entry": "5gkVNYq1wQmjXsrA2",
  "code_name": "",
  "video_entry": "",
  "trailer": "",
  "year": 2005,
  "asset_type": 2,
  "status": 0,
  "permalink": "eyeshield-21",
  "duration": "24 min.",
  "subtitles": false,
  "price": 0,
  "rent_price": 0,
  "rating": 0,
  "color": 1,
  "classification": 3,
  "brazil_classification": "a12|Violência Fantasiosa",
  "likes": 0,
  "views": 0,
  "button": "",
  "stream_url": "",
  "stream_url_backup": "",
  "copyright": "© KOME STUDIO ・ VILLAGE STUDIO / SHUEISHA ・ TV TOKYO ・ NAS",
  "skip_intro": null,
  "ending": null,
  "bumper_intro": "cirj4iq23akg00a2fqag",
  "ads": "",
  "age_restriction": null,
  "epg": null,
  "allow_languages": [
    "all"
  ],
  "allow_countries": [
    "all"
  ],
  "classification_text": " G - All Ages",
  "locked": true,
  "resign": false,
  "favorite": false,
  "actors_list": [
    {
      "ID": 387,
      "CreatedAt": "2021-11-28T17:27:20.397165Z",
      "UpdatedAt": "2021-11-28T17:27:20.397165Z",
      "DeletedAt": null,
      "name": "Miyu Irino",
      "Permalink": "",
      "country": 0,
      "year": 0,
      "death": 0,
      "image": "",
      "genre": null,
      "description": ""
    },
    ...
  ],
  "artdirectors_list": [],
  "audios_list": [],
  "awards_list": [],
  "companies_list": [],
  "countries_list": [],
  "directors_list": [
    {
      "ID": 66,
      "CreatedAt": "2023-04-05T00:01:16.421579Z",
      "UpdatedAt": "2023-04-05T00:01:16.421579Z",
      "DeletedAt": null,
      "name": "Masayoshi Nishida",
      "permalink": "masayoshi-nishida",
      "country": null,
      "year": null,
      "death": null,
      "image": "",
      "background": "",
      "genre": null,
      "description": ""
    },
    ...
  ],
  "edition_list": [],
  "genres_list": [
    {
      "ID": 7,
      "CreatedAt": "2021-10-16T00:31:21.878945Z",
      "UpdatedAt": "2021-10-16T00:31:21.878945Z",
      "DeletedAt": null,
      "name": "Comedia",
      "age_restriction": 0
    },
    ...
  ],
  "music_list": [],
  "photograpy_list": [],
  "producer_list": [],
  "screenwriter_list": [],
  "season_list": [],
  "tags_list": [...],
  "chapter_id": 0,
  "chapter_entry": "",
  "chapter_poster": "",
  "progress_time": 0,
  "progress_percent": 0,
  "included_subscription": 1,
  "paid_content": 0,
  "rent_content": 0,
  "objectID": "",
  "lang": ""
}


Jobs\MigrateDatabase fails using TenantDatabaseManagers\PostgreSQLSchemaManager::class

Bug description
Hello,
Jobs\MigrateDatabase fails using driver TenantDatabaseManagers\PostgreSQLSchemaManager::class
7 ERROR: no schema has been selected to create in at character 14 at /opt/eom/vendor/laravel/framework/src/Illuminate/Database/Connection.php:545)

If I migrate tenants table with artisan, it works. It only fails when on Events\TenantCreated jobs
I've tested tooo with the other pgsql driver TenantDatabaseManagers\PostgreSQLDatabaseManager::class
And I'm unable to create database.  Note that I'm using postgres user so, it should let me create anything!
a transaction block (SQL: CREATE DATABASE "tenant_e35341a4-4e84-4618-87d4-590726edb9b3" WITH TEMPLATE=template0)

[previous exception] [object] (PDOException(code: 25001): SQLSTATE[25001]:
Active sql transaction: 7 ERROR:  CREATE DATABASE cannot run inside
a transaction block at /opt/eom/vendor/laravel/framework/src/Illuminate/Database/Connection.php:545)
[stacktrace]

So, Ive come back to TenantDatabaseManagers\PostgreSQLSchemaManager driver as at least it let me create the database.
I've take a look to Jobs\MigrateDatabase code and I've seen. that exec  Artisan command.
So I've tried to exec the same command  on shell and it works well.
php artisan tenants:migrate --tenants=f74739bd-51c6-44eb-a2c9-da474b7c5147

I've dumped baseConfig from TenantDatabaseManagers\PostgreSQLSchemaManager driver when Jobs\MigrateDatabase is running. And It looks fine  as  search_path is set to the correct schema.
[2023-01-09 11:08:51] local.INFO: Array
(
    [driver] => pgsql
    [url] =>
    [host] => <IP_ADDRESS>
    [port] => 5432
    [database] => eom
    [username] => postgres
    [password] => **********
    [charset] => utf8
    [prefix] =>
    [prefix_indexes] => 1
    [search_path] => tenant_1b38fc06-86eb-45fd-afea-14cc5558cbca
    [sslmode] => prefer
)

I don't think that is permissions issue as I'm using postgres user and moreover it works from console.
Any ideas? I can't progress  on my project due to this issue.
Thank you for the help
Regards
Steps to reproduce
Setup laravel tenancy using boilerplate and set tenant database driver to  TenantDatabaseManagers\PostgreSQLSchemaManager::class
You must enable  Jobs\MigrateDatabase into app/Providers/TenancyServiceProvider.php too
Expected behavior
Job should runs without error and migrate all tenants tables
Laravel version
9.19
stancl/tenancy version
3.6

@abrardev99 Try to reproduce this tomorrow (using boilerplate code as mentioned in the reproduction steps)

I've just do it, with just boilerplate and occur the same issue. It looks that the problem occurs when you create tenant using Nova4.

Nova doesn't let to create database but using schema it neither works.  Moreover, queued job, ran out of Nova, so it should works  isn't it?

I've just try back with schema  and queue job and nothing. Same error ...

Can you send me references , I've searched myself on issue tracker and discord server but I haven't been able to found anything

Working solution based on https://github.com/laravel/nova-issues/issues/180#issuecomment-433493057
class CreateTenant extends Action implements ShouldQueue
{
    use InteractsWithQueue;
    use Queueable;

    public $withoutActionEvents = true;

    public $confirmButtonText = 'Create Tenant';

    /**
     * @throws Throwable
     */
    public function handle(ActionFields $fields): void
    {
        DB::commit();

        $tenant = Tenant::create(['name' => $fields['name']]);
        $tenant->domains()->create(['domain' => $fields['domain']]);
    }

    public function fields(NovaRequest $request): array
    {
        return [
            Text::make('Project Name', 'name')
                ->rules(['required', 'string', 'max:30', 'unique:tenants,name'])
            ,
            Text::make('First Domain', 'domain')
                ->rules(['required', 'string', 'unique:domains,domain'])
            ,
        ];
    }
}


Can't resolve '@material-ui/core/Unstable_TrapFocus'

I'm getting Can't resolve '@material-ui/core/Unstable_TrapFocus' after updating to alpha 8. Didn't happen in alpha 7. Some missing peer dep?
Relevant dependencies in my package.json:
"dependencies": {
    "@apollo/client": "^3.0.0-rc.2",
    "@material-ui/core": "^4.9.14",
    "@material-ui/icons": "^4.9.1",
    "@material-ui/lab": "^4.0.0-alpha.53",
    "@material-ui/pickers": "^4.0.0-alpha.7",
    "gatsby": "^2.23.1",
    "gatsby-theme-material-ui": "^1.0.9",
    "react": "^16.13.1",
    "react-dom": "^16.13.1",
    "styled-components": "^5.1.0"
  },

You need both dependencies (core and pickers) to the install the latest version.

For anyone else who runs into this, here are the exact versions you need to fix the issue:
    "@material-ui/core": "4.10.2",
    "@material-ui/pickers": "4.0.0-alpha.8"

@Jdban: saw this again when updating to alpha10... just npm update @materia-ui/core every time you update the pickers, I guess.


Classification cleanup

This is using the collection of archived MARS paramId's, from
/home/maro/pickles_from_crawler/allParamIds_mars.txt
I've removed the ones that have been classified but are not on that list. Please do not merge this as-is, as I am aware of some minor issues with this change (some parameters shouldn't be removed because they are part of a larger set, or the removal looks inconsistent from looking at nearby id's).
This will eventually be linked to an official issue to track development.

 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.

Is there a reason we want to remove parameter classifications? If there isn't a strong reason, then it's better to cover as much of the paramdb as possible.
At the very least we need to cover paramIds with WMO and ECMWF classifications, found in:
definitions/grib1/paramId.def
definitions/grib1/localConcepts/ecmf/paramId.def
definitions/grib2/paramId.def
definitions/grib2/paramId.legacy.def
definitions/grib2/localConcepts/ecmf/paramId.def
definitions/grib2/localConcepts/ecmf/paramId.legacy.def


Youtube - Not able to download URL https://www.youtube.com/watch?v=xm-xkPlCAxc&ab_channel=MDSSYMPHONY3

Checklist

[ ] I'm reporting a broken site support issue
[x] I've verified that I'm running youtube-dl version 2021.12.17
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log
Last login: Wed Mar 15 11:37:05 on ttys000
➜  ~ cd yt
➜  yt cd MDS\ Symphony
➜  MDS Symphony youtube-dl https://www.youtube.com/watch\?v\=xm-xkPlCAxc\&ab_channel\=MDSSYMPHONY3 --verbose
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['https://www.youtube.com/watch?v=xm-xkPlCAxc&ab_channel=MDSSYMPHONY3', '--verbose']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] youtube-dl version 2021.12.17
[debug] Python version 3.11.2 (CPython) - macOS-12.6.3-x86_64-i386-64bit
[debug] exe versions: ffmpeg 5.1.2, ffprobe 5.1.2
[debug] Proxy map: {}
[youtube] xm-xkPlCAxc: Downloading webpage
ERROR: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 815, in wrapper
    return func(self, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.11/site-packages/youtube_dl/YoutubeDL.py", line 836, in __extract_info
    ie_result = ie.extract(url)
                ^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 534, in extract
    ie_result = self._real_extract(url)
                ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.11/site-packages/youtube_dl/extractor/youtube.py", line 1794, in _real_extract
    'uploader_id': self._search_regex(r'/(?:channel|user)/([^/?&#]+)', owner_profile_url, 'uploader id') if owner_profile_url else None,
                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/Cellar/youtube-dl/2021.12.17/libexec/lib/python3.11/site-packages/youtube_dl/extractor/common.py", line 1012, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
youtube_dl.utils.RegexNotFoundError: Unable to extract uploader id; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description
I trying to download this youtube URL https://www.youtube.com/watch?v=xm-xkPlCAxc&ab_channel=MDSSYMPHONY3
And keep getting this error as in logs.
I am trying this in Mac
I have all the latest updated softwares.

I'm reporting a broken site support issue
I've verified that I'm running youtube-dl version 2021.12.17
I've checked that all provided URLs are alive and playable in a browser

[ ] I've checked that all URLs and arguments with special characters are properly quoted or escaped

No, you are, even if you shouldn't be.

[x] I've searched the bugtracker for similar bug reports including closed ones

Read #30839 (again?) to learn how.

[x] I've read bugs section in FAQ

Apply it.


始终置顶边框bug

Microsoft PowerToys version
0.70.0
Installation method
GitHub, Microsoft Store
Running as admin
Yes
Area(s) with issue?
Always on Top
Steps to reproduce
在使用始终置顶置顶WSL的应用时边框有点问题
✔ Expected Behavior
边框靠近在应用边框上
❌ Actual Behavior
边框离应用很远
Other Software
No response

Steps to reproduce
The border has some problems when using Always on Top to pin a WSL app
✔ Expected Behavior
The border is close on the app border
❌ Actual Behavior
The border is very far from the app


run_infersharp.sh parameter must be an absolute path, relative paths do not work

If I run the run_infersharp.sh script from the infersharp1.2 directory I need to pass a parameter defining the root of the tree where the DLLs are found. The problem is that this tree can only be designated by an absolute path, not a relative one. In other words:
$ pwd
/root
$ infersharp1.2/run_infersharp.sh applicationplatform/bin
will print this:
Processing {applicationplatform/bin}
Copying binaries to a staging folder...\n
cp: cannot stat 'applicationplatform/bin': No such file or directory
Code translation started...
Translation stage 1/3: Loading binaries.
Translation stage 2/3: Computing type environment.
Translation stage 3/3: Computing control-flow graph.
Coverage Statistics:
Method successfully translated: 0 (-2147483648%)
Method partially translated: 0 (-2147483648%)
Instructions translated: 0 (-2147483648%)
Instructions skipped: 0 (-2147483648%)
Where as
$ infersharp1.2/run_infersharp.sh /root/applicationplatform/bin
will actually do the analysis as expected.
Also, if there is a real problem where the path really does not exist, you should probably stop with a decent error message and not print out junk information with -2147483648%

Relative path should work. I tried a simple case and it works for me. I need to take a closer look.
root@cc7b3a8d04fa:~# pwd
/root
root@cc7b3a8d04fa:~# ./infersharp/run_infersharp.sh Examples/
Processing {Examples/}
Copying binaries to a staging folder...

Code translation started...

Regarding the second suggestion, we have made a fix to properly stop further execution if the script fails. Thank you!


ECL support

Hey,
here is ECL support.
I have also a question - will you accept a patch which replaces in-house gray-streams compatibilities with the trivial-gray-streams? More implementations will merit from this.
Regards,
Daniel

bump :-)

Thanks for the patch!  I'm not super-keen on adding the overhead or the dependency of trivial-gray-streams.

OK, thanks for merging. And what about adding it as a fallback with read-time conditional if there is no "direct" support for them? (sorry for nagging it, I promise I won't ask any more questions about it).


Apps which run in menu bar and Dock won‘t show up in Pock

Every app which runs (app icon) in menu bar and dock are not available in Pock.
Sent with GitHawk

Could you please name one of these apps, so I can try to replicate this situation?

Could you please name one of these apps, so I can try to replicate this situation?

@pigigaldi MacUpdater f.e.
Sent with GitHawk


Create LICENSE

This is not the last version from http://www.apache.org/licenses/LICENSE-2.0.txt wich reaplce {} with []
Dont we must apply what explain in appendix ?
APPENDIX: How to apply the Apache License to your work.
  To apply the Apache License to your work, attach the following
  boilerplate notice, with the fields enclosed by brackets "{}"
  replaced with your own identifying information. (Don't include
  the brackets!)


Implement tests on inference scenarios for transforms

Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[x] Did you update the CHANGELOG?

Proposed Changes
Look at #1077.
Closing issues
Closes #1077.

Codecov Report

:exclamation: No coverage uploaded for pull request base (inference-v2.1@892945e). Click here to learn what that means.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@                Coverage Diff                @@
##             inference-v2.1    #1094   +/-   ##
=================================================
  Coverage                  ?   67.99%
=================================================
  Files                     ?      164
  Lines                     ?     8891
  Branches                  ?        0
=================================================
  Hits                      ?     6045
  Misses                    ?     2846
  Partials                  ?        0

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


RDBMS ingestion - Filter to switch ingestion of foreign keys or other DB objects apart from the table and primary key

Foreign key metadata is useful, but not always needed. Some legacy databases with tables of 100s of columns have loads of foreign keys. The ingestion takes a long time to get all that data, and is not critical. So it will be good to have a filter in the ingestion yml (and something similar on OM UI), to opt out of ingesting foreign keys.
In fact the ibm_db_sa i.e. IBM's SQLAlchemy has several get methods in its reflections.py. It would be nice to have filters based on each of these methods. But foreign keys would be a good place to start

Hi @upenbendre, I have some mixed feelings on adding too many config options related to the metadata ingestion. IMO it makes sense to separate configurations if they required elevated permissions (e.g., ingesting tags for some dbs), or we only want to stick to tables (e.g., toggle views). However, keys and constraints are all essential parts of the metadata.
For now, I'd advocate for keeping this behavior. If some tables are not critical, then we could even filter them out from the metadata ingestion if having them on the catalog is not adding value.
If we get more inputs on this subject, we can pivot and add it. Thanks
cc @harshach


Bad Z3 encoding for pattern: "unknown constant @x1"

This code:
assume val f (x y:int) : int
assume val g (i:int{i < 20}) : (b:bool{b})

let test (c d:int) =
  assert (forall (i:int).{:pattern (f d i)} i < 10 ==> g i)

Gives this error:
 (Error 276) Unexpected output from Z3: (error "line 126079 column 18: unknown constant @x1")

Seems related to https://github.com/FStarLang/FStar/issues/1537. Will take a look, F* should have dropped it if it's not well-formed.


Bump @embroider/* packages and add peerDependencies info

@mixonic would you mind to release this as a new minor?

@mixonic would you mind to release this as a new minor?

As per https://github.com/embroider-build/embroider/issues/1077#issuecomment-1016689755 a  new release would be great!


tweak CI

@homunkulus r+

@homunkulus r+

:pushpin: Commit 9ba17e1 has been approved by japaric

:hourglass: Testing commit 9ba17e1 with merge 9ba17e1...

:sunny: Test successful - status-travis
Approved by: japaric
Pushing 9ba17e19b8a9fadf9d77be10857992549de0c528 to master...


Expressworks WHAT'S IN QUERY - Official solution does not work.

The official solution for Expressworks WHAT'S IN QUERY:
var express = require('express')
var app = express()
app.get('/search', function(req, res){
  var query = req.query
  res.send(query)
})

app.listen(process.argv[2])

Gives:
             ACTUAL                                 EXPECTED

───────────────────────────────────────────────────
"{"                                 !=    "{"results":"recent","type":"quote","page":"8"}"
"  "type": "quote","            !=
"  "page": "8","                !=
"  "proto": {}"               !=
"}"                                 !=
It is obviously not expecting:    "  "proto": {}"
Manually building the object like so does work:
app.get('/search', function(req, res){
var send = {};
send.results = req.query.results;
send.type = req.query.type;
send.page = req.query.page;
send = JSON.stringify(send);
res.status(200).send(send);

});

Please submit this as an issue against ExpressWorks.

Yes, the expressworks solution is wrong.
However, there's a really simple fix.
__proto__ is a non-enumerable property
There's your big hint

@prashcr Yes, I am aware and have solved the issue, but this needs to be addressed. I have submitted it here: https://github.com/reddock/fcc_express/issues/1


Library Loader

We have and interesting request that we'll be looking into imminently.
A Studio has a large library of assets that are used regularly in all the projects they work on. We need to provide a way to open a loader with this library and populate the scene with subsets from there.
Proposed solution
We're thinking of adding another loader button to the avalon menu, that wouldn't be loading from the context of current project, but rather let user choose from a list of projects that are tagged as libraries and then present the assets from that projects.
Say we have projects CityAssets and NatureAssets, to publish into the library artist treats it as any other project.
When loading to coolProject. Artist clicks on avalon -> load from library -> chooses a library from the dropdown -> loads the subset he wants.
To prevent cross contamination of projects by loading assets from one to another, we'd only allow importing and during that process we'd strip all the references to the library project from the subset (ids for example). This would force the artist to re-publish it within the project he's working on making it fully contained.

@iLLiCiTiT Are you still seeing that error? Could you include a few more lines of the traceback? It looks like a widget is being called after having been instantiated.
button = QtWidgets.QPushButton("MyButton")
button()
# TypeError

It was because I used attribute "parent" to store parent widget but one widget had also method called "parent" so it crashed when this method was called. Already fixed :). I think it's actually ready to show @mkolar ?

Ah, yes. All widgets actually have .parent() so better not use that as an instance variable.

https://doc.qt.io/qt-5/qobject.html#parent

I think it's actually ready to show

certainly. it's in production with a few clients now, so we can make PR and discuss there.

How is this one coming along in production @mkolar ? :) Smooth? Any issues? Can we try and get a PR set up?

It's been running smoothly for ages actually. We were slowly working on getting on par with avalon-core because of our custom code. Let me check with @iLLiCiTiT how close we are with PRs for these tools