react.TypeScript.DefinitelyTyped not owned by DT

Hey All,
The nuget automation is currently failing up on AppVeyor with the following error:
NuGet.exe : Failed to process request. 'The specified API key is invalid or does not have permission to access the specified package.'.
At C:\projects\nugetautomation\CreatePackages.ps1:187 char:25
    +                         & $nuget push "$packageFolder.nupkg" -ApiKey $nugetApiKe ...
    +                         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (Failed to proce...ied package.'. :String) [], RemoteException
    + FullyQualifiedErrorId : NativeCommandError

The remote server returned an error: (403) Forbidden.

This error is because @wizzard0 claimed the react.TypeScript.DefinitelyTyped name on NuGet, which is unfortunate.
If @wizzard0 is listening, could you please rename your nuget package to something else (not using DT's convention) and then either add us as an owner to the package or, deleting yours completely so our nuget process can correctly publish to react.TypeScript.DefinitelyTyped?
Thanks

Some other packages claiming the DefinitelyTyped naming convention (but not ones we can publish to on NuGet)
https://www.nuget.org/packages/Sjcl.TypeScript.DefinitelyTyped/ @evgenus
https://www.nuget.org/packages/Jsbn.TypeScript.DefinitelyTyped/ @evgenus
https://www.nuget.org/packages/BigInt.TypeScript.DefinitelyTyped/ @evgenus
https://www.nuget.org/packages/TGrid.TypeScript.DefinitelyTyped/ @TesserisPro
@evgenus could you please consider renaming your NuGet packages to avoid conflicting with the same DT convention we've setup {project}.TypeScript.DefinitelyTyped? You could consider using something like BigInt.TypeScript.Evgenus (so it won't conflict with the DefinitelyTyped name?)
This would really be helpful and avoid confusion in the community. It looks like you may know @wizzard0 - would you mind reaching out to him about the React package? Thanks

Hi @staxmanade,
Initially, the packages are named like this for they to be discoverable on NuGet. I'm considering renaming to something like react.TypeScript.DefinitelyTyped.wizzard0, but that also requires patching our internal infrastructure (the react package in question is not compatible with DT one, so we can't just replace one with another).
Can this wait a bit while we finish our other project please?

Great, thanks for responding... We don't really have a choice to wait :) since we don't have permissions...
But yes, please do the work you need and let me know if you need any help cleaning things up.
For now, I'm excluding these packages explicitly from trying to publish, but when this project can reclaim the id's on NuGet, let me know so we can make sure the transition works out.
Appreciate you being willing to work through this!

@wizzard0 we are getting some flack for not having the DT typings properly represented in NuGet. Could you please make some traction at giving back the convention based DT NuGet name?

Hi all, I have just attempted to write a prototype app using React + TypeScript and my difficulties getting started have lead me to this issue. It would appear as though the current package owned by @wizzard0 has a bug whereby after installing the React.TypeScript.DefinitelyTyped package, no d.ts files are actually added to the project. (It appears to be an issue within the <files> section of the .nuspec file.)
I would be happy to follow up this specific issue within @wizzard0's project, but I think the main priority would be to relinquish the ownership of React.TypeScript.DefinitelyTyped so that the DT folks can release their package. I thought I would mention it here just to highlight the urgency since developers currently have no ability to access TypeScript definitions for React, either through the "unofficial" package, or the DT one.

What the DT team should do is submit a complaint to the NuGet team for violation of Intellectual Property, in that @wizzard0 is using the "DefinitelyTyped" name, but is not publishing on behalf of the DefinitelyTyped community.

I did send a complaint (not with that sort of language) but NuGet team responded it's a first-come-first-serve.
Would just be great if everyone who owns these packages could rename theirs and notify me so I can re-enable the proper DT version. https://github.com/DefinitelyTyped/NugetAutomation/blob/master/CreatePackages.ps1#L16

It's frustrating that there's effectively "squatter's rights" on NuGet but this is Open Source I guess.
@wizzard0 would you be able to rename your package please?  It's causing problems for other people and creating confusion.  It would be fantastic to resolve this.

Reaching out to all of the following who have also contributed to the other project in hopes they may know @wizzard0  - or be able to help push this forward.
Can any of you help out here? (@malbertife @jnetterf, @liammclennan, @Evgenus)
We'd really appreciate it.

hi guys, I restored my NuGet account and would be happy to transfer the package ownership - nuget does not allow me to relinquish a name so as not to break apps that might be using the packages, I can only add people with publishing rights. Who shall I add?

@wizzard0 that's fantastic! - NuGet has an option to "delete" a package - https://docs.nuget.org/create/deleting-packages I don't know if that will allow us to take it over going forward.
The last paragraph of that page says:

If you run into an exceptional situation where you think one of your packages must be deleted, this can be handled manually by the NuGet team. e.g. if there is a copyright infringement issue, or potentially harmful content, that could be a valid reason to delete it. You should submit a support request through NuGet Gallery in that case.

I wonder if we can reach out to NuGet to delete it for us?

@staxmanade i'd better just add you (or whoever mirrors DT to nuget) so my old versions would be persisted too

@wizzard0 I PM'd you my email address.

@staxmanade nuget requires an username, not email

added DefinitelyTyped

I don't know if there's a timing thing - but DefinitelyTyped still doesn't appear to have access (yet). At least the API key used for this account isn't yet working under react.TypeScript.DefinitelyTyped.

you need to log in interactively to DT and approve the transfer

Thanks @wizzard0 I found that an email was sent that I have to click a link to accept. (I did that for staxmanade - but haven't seen the email yet for DefinitelyTyped - I'll keep an eye out for it.

Just an update on this thread - I have not yet seen any emails to the DefinitelyTyped email (even after I tried changing it to a different email and removing/re-adding it to the package) - I've reached out to NuGet support to see where it could be hung in the process.

No response yet from support@nuget - just pinged them again.

Any updates?

@dberryalliance so far it's stuck somewhere around nuget support, I suppose

I just wanted to chime in to mention that I really appreciate the work you're doing on this issue, and I hope it comes to a swift conclusion!

Any update on this? Is the only alternative to grab the typings directly from DefinitelyTyped?

I believe yes - or npm


Accessible issue

Love love love this tabbing! It's the best one yet. Am using it for college.
However, there's an issue - it's not accessible friendly. I should've checked before implementing site-wide. Is there a way you can add accessible in there? I'll need tabbing and keyboard to work properly. Thank you.

Any progress on this?


[QoI] Improve diagnostics when accessing .init on a non-metatype

Resolves SR-3236 by falling through to existing logic when all candidates are inaccessible. Maybe not perfect, but better than the current state of affairs I think.

@swift-ci please smoke test macOS

@swift-ci please smoke test

cc @xedin @rudkx @jrose-apple who were involved with #5237.

LGTM!

I agree with @xedin. This shouldn't be an access control test; we should just reject this member access entirely.

(and the diagnostic shouldn't state anything about "access" or "inaccessible")

The main point of this PR was to avoid spewing a candidate list when some candidates are actually accessible, but wrong for other, diagnosable reasons (e.g. being a type member while the call site is trying to access an instance member).


chore: update repo metadata with codeowners

@anweshan, this should be updated to link to a GH team. Thanks!


Some details regarding generating NQ trainset for the reader model

Hi @AkariAsai. Thank you for this great work.
I'd like to understand more clearly how the NQ trainset for the reader model is generated.
On your comment, you said that you removed all the tables and list elements from the NQ's original preprocessed HTML data.
https://github.com/AkariAsai/learning_to_retrieve_reasoning_paths/issues/9#issuecomment-610714692
I'm curious how you handled the case where a list element contains an answer and a paragraph contains the list?  (like the following example)
https://github.com/google-research-datasets/natural-questions/blob/master/toy_example.md
eg.
Google was founded in 1998 By:

Larry

Sergey

Did you remove such examples?
Or did you just remove the 'list' tag so that the example is parsed like :
Google was founded in 1998 By: Larry Sergey

Addressing your main question,  we do not filter out the lists inside paragraphs, but we remove any HTML tags remaining in the context during our post-process. Thus, the example you mentioned above would be  Google was founded in 1998 By: Larry Sergey, but there might be some corner cases we missed.

Thank you for the quick response. It's of great help to me.
I'm also wondering if you may have applied additional filtering conditions (such as the length of answer tokens).

We do not apply additional filtering conditions based on the answer length. Our train/dev/test splits are from A Discrete Hard EM Approach for Weakly Supervised Question Answering's original repository, and in their preprocess script, the answers whose length is larger than five are filtered out, as in another previous work.

Thank you for the information!

No problem! Thank you for your interest in our work :)


403 doesn't trigger onError event

I am using form requests and when an action is not authorized I return false from authorize() method in the request.
This renders the dialog with the message 403 Not authorized, but only onFinish event triggers.
The problem is when using dialogs (shadcn ui), even though the 403 dialog renders above the dialog with form, the focus is still on the dialog behind (maening you can't  interact with 403 dialog until the one below is closed), and what I wanted to do with onError is to hide that dialog so the focus would be on 403.
But it seems none of the events is firing when there is any error response, other than validation errors (which returns 200 response and not an actual 422 Unprocessable Content error response). It seems onError triggers only when there is an error bag present in 200 response, and I think there is missing something to handle such response with proper events.

What i recommend to you is to handle the error in laravel's error handler and to manually set the error into the session so inertia.js will receive it, this way the onError will trigger.

I don't know what you mean. I use form request authorized method (which runs before validation) and which should return true or false, not an error or exception. Handling authorization manually would make the built-in features useless


Bug: Hiding either of Volume or Balance hides both from the UI

Disclaimer:
Please go through existing issues to avoid creating duplicates. If you found a similar issue please post a comment there. If you are uncertain your issue is related to others then create the issue. Obvious ignorance of this rule might get you blocked from posting new issues. Please tick the box below to show that you have followed the guideline.

[x] I have checked for a similar issue and sure it hasn't been reported before.

Describe the bug
When using the "Show/Hide Features" and hiding either of Volume or Balance, it hides both of them.
Steps to Reproduce
Steps to reproduce the behaviour (feel free to change the placeholder as you need):

Open eqmac settings
Click on Show/Hide Features
Untick either Volume or Balance
Come back to flyout and notice that both of the elements have hidden.

Expected behaviour
Only one of them should be hidden.
Setup information:

Audio device used for playback: Internal speakers
Audio transmission interface: idk, maybe system?
macOS Version: 10.12.6
eqMac Version 1.2.0
eqMac UI Version 2.1.1

Screenshots or Console.app logs

Thanks for reporting this and sorry for not replying in time, this bug has been fixed for a while now.
Closing this issue


Bug resizing

See video https://youtu.be/D2d57C_nK1s
I use demo code
var mc = $('#js-croppie');
mc.croppie({
    viewport: {
        width: 150,
        height: 150,
        type: 'circle'
    }
});
mc.croppie('bind', 'https://s3-us-west-2.amazonaws.com/s.cdpn.io/3/pie.png');

Why is this happening?

This happens after the bootstrap connection.css (Twitter Bootstrap 3)

Thanks for the info!
FYI, if you'd prefer to call mc.croppie('bind', {with parameters}); outside of your shown.bs.modal event, you can.  Then you can just call mc.croppie('bind') inside the callback to refresh croppie's location.


DBZ-3898 Skip excluded schema DDL events

https://issues.redhat.com/browse/DBZ-3898

@Naros, could you push that same change to @jpechane's 1.6.2 branch (https://github.com/debezium/debezium/pull/2641)? Thx! It doesn't apply cleanly, so some changes are needed.

@gunnarmorling @jpechane Pushed to PR #2641


Web - Clicking ToS before filling fields in connect bank account throws an error

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to workspaces and click on bank account
Click on connect to bank account
Click on the Tos checkbox with out filling routing and account number fields
Observe that please enter a valid routing number error is thrown
Uncheck the Tos check box
Fill the routing number filled with a correct routing number and check the ToS check box, observe that no please enter a valid account number error is not thrown

Expected Result:
Error should either not be thrown or should be consistent to both empty routing number and empty account number
Actual Result:
Error is thrown when checking Tos while routing and account number field are empty
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

Version Number: 1.3.37-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/93399543/e35eeed3-e87b-4bea-8497-e6f5a536ae4d
https://github.com/Expensify/App/assets/93399543/60810010-8c8d-41e2-9c08-58aa114a1a5d
Expensify/Expensify Issue URL:
Issue reported by: @sofoniasN
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1688640219628169
View all open jobs on GitHub

This looks like an intended behavior. You focused an input and then left in an invalid state. So an error is displayed under that input. Checking the checkbox is not crucial here.

@cubuspl42 the problem here is that the error is not consistent in the two fields, the routing number field and the account field. As mentioned in the expected result, either the error throwing should be consistent or no error should be sent when clicking ToS when both fields are not focused

@SofoniasN You're suggesting that there's some defined behavior here for checking the ToS checkbox, while I'm not aware of any such defined behavior.
What matters is that the input is unfocused. When a text input gets unfocused, and is left in an invalid state, an error is displayed under that input. This behavior is consistent between both the "Routing number" and the "Account number" text fields.
While I understand that someone might think that checking the ToS checkbox is somehow special, it's not. It's an input like any other. Only the "Continue" button is special in this matter.

The same thing happens if you start filling out the account number before the routing number, so basically, we just throw a field validation error if you start filling out a form in the wrong order. This is more of a feature request rather than a bug report, so I'm going to close.


Adding pointer typed exchange functions and a native "mint_atomic_t" integer

Only syntactic sugar, but useful. The pointer typed exchange functions facilitate unconditional consumer algorithms without casting to an integer type. The mint_atomic_t integer accommodates situations where fixed-sized operations are unfavorable. For example, on 64bit platforms, it's reasonably possible to have more than 2^32 objects in memory, where similar cannot be said on 32bit platforms(ignoring PAE). At the same time the CAS loop implementation of mint_atomic64_t methods could snowball contention through retries on heavily-contended points. Both considerations combined suggest the utility of a platform-sized integer, provided it is used with awareness of it's implicit limitations(a stipulation that applies to any type choice).
I'm aware an end user could easily roll the integer into their own code if need; this has the advantage of DRYing out that boilerplate for the community.

Hi Alexander, Mintomic is nearing the end of its life. I've basically moved all the atomic C code to Turf: https://github.com/preshing/turf/tree/master/turf/c
Turf includes C++ wrappers that provide additional flexibilty. Instantiating a turf::Atomicturf::uptr should achieve the same thing, syntactically, as you seem to have done here (an atomic pointer-sized integer).

That's neat, but incompatible with my current use case (a code base shared across kernel and user modes). In that C-only arena, I think mintomic still has a lot to offer.
However, I'll keep turf in mind for future user-mode only projects!


How to load trained SVM model?

I am not able to load the trained weights from .dat file to model.  Can you tell how can i load weights?

Please give more detail about your problem

I have trained the SVM model using code you have provided now to remove the overhead of training the model over whole dataset I am trying to save the trained weights into a .DAT file (same as we do with other Keras or Tensorflow models). But after Saving I am not able to use those weights. Can you share the solution to this problem?


当我运行stage2的命令时，显示错误连接openAI,这个是什么原因呢？

Loaded dataset size: 9009, top10 accuracy: 91.81, top1 accuracy: 86.54
Loaded dataset size: 5046, top10 accuracy: 79.83, top1 accuracy: 53.05
Working...            ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 0/5046 0:06:31/-:--:--(--s/iter)<class 'openai.error.APIConnectionError'> Error communicating with OpenAI
retrying...

OpenAI‘s API has been blocked. You need a proxy server for connection.


Revert "Merge pull request #237 from cisagov/172-design-vs-graphs"

This reverts commit a8fc488e3fefbafb71e9c0447c30fe4d3e976c03, reversing changes made to e8fe06d2e01212efa876027b67625bfaf5c29222.

🗣 Description
Revert changes in PR linked to #172
The branch was recreated named 172-design-vs-graphs.
💭 Motivation and context
Staging-cd started breaking once merged
Was working locally. Theory: there was an overload of data that caused the calls to take too long and went over the 29s limit for API gateway.
🧪 Testing

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All future TODOs are captured in issues, which are referenced
in code comments.
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] Tests have been added and/or modified to cover the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[x] Revert dependencies to default branches.
[x] Finalize version.

✅ Post-merge checklist

[x] Create a release.

[*]


⚠ Bofrost.fr has degraded performance

In 85618bd, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1418 ms

Resolved: Bofrost.fr performance has improved in f85f3d8 after 13 minutes.


GO:0043395 referred substance obsoleted in ChEBI

GO:0043395 refers to heparan sulfate proteoglycan but this was apparently obsoleted without successor:
id: GO:0043395
name: heparan sulfate proteoglycan binding
namespace: molecular_function
def: "Interacting selectively and non-covalently with a heparan sulfate proteogl
ycan, any proteoglycan containing heparan sulfate as the glycosaminoglycan carbo
hydrate unit." [ISBN:0198506732]
synonym: "heparin proteoglycan binding" RELATED []
intersection_of: GO:0005488 ! binding
intersection_of: has_input CHEBI:24499 ! heparan sulfate proteoglycan

Sorry, the chebi.obo from Nov-01 actually has the entry, so not your problem.


Issue during build

Hi,
I ran into the following issue while building the project using Maven.
Downloading from maven2: http://repo1.maven.org/maven2/edu/ncsu/codeseer/projects/1.0-SNAPSHOT/projects-1.0-SNAPSHOT.pom
...
[ERROR] Failed to execute goal on project code: Could not resolve dependencies for project edu.ncsu.codeseer:code:jar:1.0-SNAPSHOT: Failed to collect dependencies at edu.ncsu.codeseer:projects:jar:1.0-SNAPSHOT:
Failed to read artifact descriptor for edu.ncsu.codeseer:projects:jar:1.0-SNAPSHOT: Could not transfer artifact edu.ncsu.codeseer:projects:pom:1.0-SNAPSHOT from/to maven2 (http://repo1.maven.org/maven2): Transfe
r failed for http://repo1.maven.org/maven2/edu/ncsu/codeseer/projects/1.0-SNAPSHOT/projects-1.0-SNAPSHOT.pom 501 HTTPS Required -> [Help 1]
[ERROR]

I tried to use https instead of http, but the error persisted. I think the issue might be because the following URL is unreachable?
https://repo1.maven.org/maven2/edu/ncsu/
Noting that, I commented out the following in pom.xml file:
<!--<dependency>
    <groupId>edu.ncsu.codeseer</groupId>
    <artifactId>projects</artifactId>
    <version>1.0-SNAPSHOT</version>
    <!-<scope>system</scope>->
    <!-<systemPath>${project.basedir}/jars/projects-1.0-SNAPSHOT-jar-with-dependencies.jar</systemPath>->
</dependency>-->

After this change, the build was successful.
I am new to Maven and I am not sure if this is the right change. Can you please let me know?
Thanks,
Sridhar

Should we merge ICSE20 => master?

Hi George,
Thanks for the comments. I was able to run the code on the Example dataset.
There are a few comments that might be helpful to anyone in the future:

In Install.md, it is mentioned to run mvn clean install. But running this command directly would not be successful as you mentioned in your comments. I ran initialize.sh to get around this issue.
Python 3 is not supported. There is some syntax used in the code that has been deprecated by Python 3 (like is not). Hence, Python 2 is necessary. This becomes a hassle on the latest versions of Ubuntu where Python 2 is not supported anymore.

You can consider adding this to your documents.
BTW, great paper! Enjoyed reading it and loved the idea :)

Thank you Sridhar. Will update the ReadMe with your proposed changes :)


Incompatible services list is not complete. Products that are using "Network Container" do not support NSG Flow log

Incompatible services list is not complete. I have a discussion with Network Watcher Program Manager. Products that are using "Network Container" do not support NSG Flow log, but this is on the roadmap.
Product like
a. App service (with Regional Vnet Integration
b. Microsoft.DB  flexibleServers  (PostgreSQL.MySQL, MariaDB)
...

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

ID: bd699dac-d6e5-5a39-624f-2d3f77f92fca
Version Independent ID: c922bcd2-0778-a7af-4855-1bff6c9ce380
Content: Introduction to flow logging for NSGs - Azure Network Watcher
Content Source: articles/network-watcher/network-watcher-nsg-flow-logging-overview.md
Service: network-watcher
GitHub Login: @damendo
Microsoft Alias: damendo

@qliu95114 Thank you for sharing feedback. We are actively validating it. We will share updates soon.

@qliu95114 , We have assigned this to the content owner/author for review.
@damendo , for your review.

@damendo  any news

@Harsha-CS please confirm whether this is out in production or not?

#reassign: @Harsha-CS

@qliu95114 , App service v3 is not supported by App service v3 is supported.

@Harsha-CS does this mean App Service v3 SKUs support NSG flow logs?

@pl4nty i do not understand  "App service v3 is not supported by App service v3 is supported."  , looking for some clarification

@qliu95114 I do not understand either. But I might try testing this in my lab

@qliu95114 I've confirmed NSG flow logs are created for a vnet-integrated App Service B1 SKU. Do you know whether they're supported for Microsoft.DB/flexibleServers yet?

#reassign:halkazwini

@qliu95114 , @pl4nty,
Thank you for your feedback! I've just updated the list of incompatible Azure services.
#please-close

@halkazwini my testing shows vnet-integrated App Service v3 SKUs do produce NSG flow logs. likely implemented in the 9 months since this issue was opened. why are App Services listed as unsupported?

@pl4nty
App Service is not fully supported. The product group is evaluating its compatibility with NSG flow logs, I'll update once I hear from them.
Thank you!

@halkazwini thanks! Are the flexible servers in a similar state? I have customers who are pretty keen to use these logs, especially now that vnet flow logs are coming, but not if they're unsupported

You're welcome @pl4nty !
I know they're doing a compatibility evaluation. I'll check and get back to you.


Add Shadow Blocks to Mutator Inputs

As Per design principle from issue #68 Shadow blocks should be generated automatically when mutators create new inputs. Specifically all shapes blocks

but also including the vector math block when another input is added


Created tables for Operator, Client, Tool

Using FastAPI to create REST API for persistent operator interaction
Created Operator, Client, and Tool Table:
Operator:
  instructions: str
  title: str(32)

  clients: list[Client]
  tools: list[Tool]

 Client:
   client: str(32)
   temperature: float
   model: str(32)
   operator_id: foreign key
   operator: Operator

 Tool:
  operator_id: foreign key
  operator: Operator

Create and Read work on operator, but Update and Delete do not. CRUD not yet implemented for other objects.

tyty


Add PID motor control

link to a C++ implementation
https://gist.github.com/bradley219/5373998

No longer needed with the RMD-X7 motors as they have a PID and state estimation system built in.


Renamed objects are not deleted

Describe the bug
I'm not completely sure this is a bug but it's somehow unexpected behavior: When renaming an object (e.g. a pipeline), a new renamed object is created but the previous one is not deleted, so you end up having both the un-renamed and the renamed object in the Data Factory live mode. I think this can be handled by setting DeleteNotInSource to $true but it seems too much overhead just for a simple renaming, because you must make sure no other objects get accidentally deleted (which may be leveraging DeleteNotInSource = $false for a reason.
To Reproduce

Rename an object
$opt = New-AdfPublishOption
$opt.DeleteNotInSource = $false
$opt = New-AdfPublishOption \ $opt.Includes.Add("pipeline.Wait1", "")

 Publish-AdfV2FromJson -RootFolder "$RootFolder" `
    -ResourceGroupName "$ResourceGroupName" `
    -DataFactoryName "$DataFactoryName" `
    -Location "$Location" `
    -Option $opt

Expected behaviour
The declared object gets renamed (i.e. deleted and a new renamed one created)
Actual behaviour
The renamed object gets created, the un-renamed one persists.
Screenshots
If applicable, add screenshots to help explain your problem.
Version of PowerShell module of:

ver1.6.003

Provide all files to reproduce the issue

Attach at least log file (you can wipe out any potentially sensitive info)
Attach files of objects which are involved in the process

This is not a bug - it's a designed behaviour.
The only way how renamed objects can be handle is by deleting the old one.
To do this you must enable flag DeleteNotInSource = $true.
In repo, there is no tracking information about what has been renamed.


AMBARI-24646 'ambari-server setup-ldap' fails with AttributeError when master_key is not persisted

NOTE: this patch also includes fix for AMBARI-24663
What changes were proposed in this pull request?
AMBARI-24646
STR
Installed ambari-server and configured password encryption, but chose not to persist master key
[root@ctr ~]# ambari-server setup-security
Using python  /usr/bin/python
Security setup options...
===========================================================================
Choose one of the following options:
[1] Enable HTTPS for Ambari server.
[2] Encrypt passwords stored in ambari.properties file.
[3] Setup Ambari kerberos JAAS configuration.
[4] Setup truststore.
[5] Import certificate to truststore.
===========================================================================
Enter choice, (1-5): 2
Password encryption is enabled.
Do you want to reset Master Key? [y/n] (n): y
Master Key not persisted.
Enter current Master Key:
Enter new Master Key:
Re-enter master key:
Do you want to persist master key. If you choose not to persist, you need to provide the Master Key while starting the ambari server as an env variable named AMBARI_SECURITY_MASTER_KEY or the start will prompt for the master key. Persist [y/n] (y)? n
Adjusting ambari-server permissions and ownership...
Ambari Server 'setup-security' completed successfully.

Then export environment variable
export AMBARI_SECURITY_MASTER_KEY=hadoop
Thereafter ran the following:
Issue #1 - Gave AttributeError after accepting the 'Save settings' prompt, instead of asking for master key
[root@ctr ~]# ambari-server setup-ldap -v
Using python  /usr/bin/python
INFO: Loading properties from /etc/ambari-server/conf/ambari.properties
INFO: Loading properties from /etc/ambari-server/conf/ambari.properties
INFO: about to run command: ps -p 5596
INFO:
process_pid=12677
Please select the type of LDAP you want to use (AD, IPA, Generic LDAP):Generic LDAP
Primary LDAP Host (ldap.ambari.apache.org): ctr
Primary LDAP Port (389):
Secondary LDAP Host <Optional>:
Secondary LDAP Port <Optional>:
Use SSL [true/false] (false):
User object class (posixUser):
User ID attribute (uid):
Group object class (posixGroup):
Group name attribute (cn):
Group member attribute (memberUid):
Distinguished name attribute (dn):
Search Base (dc=ambari,dc=apache,dc=org): dc=apache,dc=org
Referral method [follow/ignore] (follow):
Bind anonymously [true/false] (false):
Bind DN (uid=ldapbind,cn=users,dc=ambari,dc=apache,dc=org): uid=hdfs,ou=people,ou=dev,dc=apache,dc=org
Enter Bind DN Password:
Confirm Bind DN Password:
Handling behavior for username collisions [convert/skip] for LDAP sync (skip):
Force lower-case user names [true/false]:
Results from LDAP are paginated when requested [true/false]:
====================
Review Settings
====================
Primary LDAP Host (ldap.ambari.apache.org):  ctr
Primary LDAP Port (389):  389
Use SSL [true/false] (false):  false
User object class (posixUser):  posixUser
User ID attribute (uid):  uid
Group object class (posixGroup):  posixGroup
Group name attribute (cn):  cn
Group member attribute (memberUid):  memberUid
Distinguished name attribute (dn):  dn
Search Base (dc=ambari,dc=apache,dc=org):  dc=apache,dc=org
Referral method [follow/ignore] (follow):  follow
Bind anonymously [true/false] (false):  false
Handling behavior for username collisions [convert/skip] for LDAP sync (skip):  skip
ambari.ldap.connectivity.bind_dn: uid=hdfs,ou=people,ou=dev,dc=apache,dc=org
ambari.ldap.connectivity.bind_password: *****
Save settings [y/n] (y)? y
INFO: Loading properties from /etc/ambari-server/conf/ambari.properties
Traceback (most recent call last):
File "/usr/sbin/ambari-server.py", line 1060, in <module>
mainBody()
File "/usr/sbin/ambari-server.py", line 1030, in mainBody
main(options, args, parser)
File "/usr/sbin/ambari-server.py", line 980, in main
action_obj.execute()
File "/usr/sbin/ambari-server.py", line 79, in execute
self.fn(*self.args, **self.kwargs)
File "/usr/lib/ambari-server/lib/ambari_server/setupSecurity.py", line 860, in setup_ldap
encrypted_passwd = encrypt_password(LDAP_MGR_PASSWORD_ALIAS, mgr_password, options)
File "/usr/lib/ambari-server/lib/ambari_server/serverConfiguration.py", line 858, in encrypt_password
return get_encrypted_password(alias, password, properties, options)
File "/usr/lib/ambari-server/lib/ambari_server/serverConfiguration.py", line 867, in get_encrypted_password
masterKey = get_original_master_key(properties, options)
File "/usr/lib/ambari-server/lib/ambari_server/serverConfiguration.py", line 1022, in get_original_master_key
if options is not None and options.master_key is not None and options.master_key:
AttributeError: Values instance has no attribute 'master_key'
[root@ctr ~]#

Issue #2 - Kept asking for Master key on the prompt, despite giving correct values
[root@ctr ~]# ambari-server setup
Using python  /usr/bin/python
Setup ambari-server
Checking SELinux...
WARNING: Could not run /usr/sbin/sestatus: OK
Customize user account for ambari-server daemon [y/n] (n)?
Adjusting ambari-server permissions and ownership...
Checking firewall status...
Checking JDK...
Do you want to change Oracle JDK [y/n] (n)?
Check JDK version for Ambari Server...
JDK version found: 8
Minimum JDK version is 8 for Ambari. Skipping to setup different JDK for Ambari Server.
Checking GPL software agreement...
Completing setup...
Configuring database...
Enter advanced database configuration [y/n] (n)?
Configuring database...
Enter current Master Key:
Default properties detected. Using built-in database.
Enter current Master Key:
Configuring ambari database...
Checking PostgreSQL...
Configuring local database...
Configuring PostgreSQL...
Backup for pg_hba found, reconfiguration not required
Creating schema and user...
done.
Creating tables...
done.
Enter current Master Key:
Enter current Master Key:
Enter current Master Key:

Issue #3 - Gave an incorrect master key this time and the shell kept on printing "ERROR: ERROR: Master key does not match." and kept scrolling the page
[root@ctr ~]# ambari-server setup
Using python  /usr/bin/python
Setup ambari-server
Checking SELinux...
WARNING: Could not run /usr/sbin/sestatus: OK
Customize user account for ambari-server daemon [y/n] (n)?
Adjusting ambari-server permissions and ownership...
Checking firewall status...
Checking JDK...
Do you want to change Oracle JDK [y/n] (n)?
Check JDK version for Ambari Server...
JDK version found: 8
Minimum JDK version is 8 for Ambari. Skipping to setup different JDK for Ambari Server.
Checking GPL software agreement...
Completing setup...
Configuring database...
Enter advanced database configuration [y/n] (n)?
Configuring database...
Enter current Master Key:
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
ERROR: ERROR: Master key does not match.
^C
Aborting ... Keyboard Interrupt.

Note/Workaround: The issues are seen when master key is not persisted as part of the initial password encryption step
AMBARI-24663
STR
Installed ambari-server and configured password encryption, but chosed not to persist a master key
[root@h002 ~]# ambari-server setup-security
Using python  /usr/bin/python
Security setup options...
===========================================================================
Choose one of the following options:
[1] Enable HTTPS for Ambari server.
[2] Encrypt passwords stored in ambari.properties file.
[3] Setup Ambari kerberos JAAS configuration.
[4] Setup truststore.
[5] Import certificate to truststore.
===========================================================================
Enter choice, (1-5): 2
Password encryption is enabled.
Do you want to reset Master Key? [y/n] (n): y
Master Key not persisted.
Enter current Master Key:
Enter new Master Key:
Re-enter master key:
Do you want to persist master key. If you choose not to persist, you need to provide the Master Key while starting the ambari server as an env variable named AMBARI_SECURITY_MASTER_KEY or the start will prompt for the master key. Persist [y/n] (y)? n
Adjusting ambari-server permissions and ownership...
Ambari Server 'setup-security' completed successfully.

Then export an environment variable
export AMBARI_SECURITY_MASTER_KEY=hadoop
ambari-server setup prompts to enter current master key
How was this patch tested?
live cluster check and unit tests run

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4012/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4022/
Test PASSed.


BuddyPress cat-generator-avatars

Great plugin, how about adding the code to allow cat-generator-avatars to work for BuddyPress profiles, which really makes a lot of use of avatars.

Version Information

PHP:  7
WordPress: Latest

Steps to Reproduce

install BuddyPress

What I Expected
BuddyPress profiles could use cat-generator-avatars
What Happened Instead
cat-generator avatars show in wp-admin but not on BuddyPress profiles

I never did such work before. I will take a look.

I've got the basic code for it actually, maybe I should do a pull request.

Great. Please do so.

ok that's done.
Notes/observations:

there's effectively some duplicated code as I added new functions for getting the url and the image tag as required by BuddyPress, but I didn't change the original filter_avatar to use the new functions
first attempt had issues as the various hooks were not allowing user to upload custom image and were always showing cat avatar even if an image had been uploaded, so various extra checks added for this
finally, I also had to change the original filter_avatar to check for user uploaded profile image and disable the gravatar check see #4

Published to Wordpress, please have a look.

Looking good so far, thanks, let's close this one!


[Idea: Webview] Enable injecting javascript into a WebView

This way we can take an existing web view and run custom javascript in it, as the WKWebview allows you to do via WKUserScript

Seems reasonable, can you submit a PR @oveddan?

Sure I can work on it.

Awesome :smile:

I think we would probably want to execute the JS via a direct function call, eg:
<WebView ref="myWebView" />
this.refs.myWebView.execute('alert("hello world")')
It would also be neat if there was some way to get data back via a callback function :smile:

Keep in mind that WKWebView in only available on iOS 8.0 and later, and React Native supports iOS 7.
Also WKWebView, as far as I know, is not stable enough to replace UIWebView yet, so it might be a better idea to make it a npm module that replaces the built-in RCTWebView.

Good points @tadeuzagallo!
If we want to integrate this into our standard UIWebView based WebView perhaps we can just use stringByEvaluatingJavaScriptFromString?

@tadeuzagallo yeah in an ideal world we would be able to build this around WKWebView, but it's only supported in IOS8, and it's not really production ready - for example, loading html from the local file system doesn't work, nor does leveraging the html5 appcache to make a page loaded from a remote server available offline.  Those two things two work with a standard WebView.
I'd want to follow a similar pattern to (WebViewJavascriptBridge)[https://github.com/marcuswestin/WebViewJavascriptBridge], which is built on top of a WebView, and as you suggested @brentvatne leverages stringByEvaluatingJavasScriptFromString here
My main concern is that this would add a lot of bloat to the RCTWebView - maybe this should be done as a separate github repo.  What do you think?

@oveddan - let's try it as a separate Github repo and see how it turns out, if doesn't end up being too large or if there turns out to be a lot of demand then it might be worth integrating into core

@oveddan - fyi, I just noticed #1191

+1
@brentvatne Any news on when it's going to be merged? #1191 is not merged yet so should I wait?

@brentvatne Thanks for noticing #1191 - I believe this issue will be resolved when that is merged.

@alinz - not sure when, it's a WIP. @oveddan - great, feel free to drop in there with comments to make sure it's appropriate for the use cases you foresee you will need :smile:

@brentvatne, I ended up porting one of the iOS javascript bridge to react-native. react-native-webview-bridge. I hope it can be useful for someone else.

Hi there! This issue is being closed because it has been inactive for a while.
But don't worry, it will live on with ProductPains! Check out it's new home: https://productpains.com/post/react-native/idea-webview-enable-injecting-javascript-into-a-webview


Rather strange feedback

Just putting this here in case it is useful.
This arose when allowing an alternative answer for a matrix question. The alternative answer gains the full 2 marks, but I do not know where it got 1.5 marks from.

Can you give me a link to the questoin on the editor, please?

It's part (c) of this question: https://numbas.mathcentre.ac.uk/question/121681/matrix-arithmetic-mixed-2/

Thanks! I've managed to reproduce it by setting a variable testing condition to get exactly the same expected answer as in your screenshot.
At the moment it looks like there's a bug to do with not fully clearing the feedback while marking alternatives, which is surprising, but I'm now at the end of my working week so I'll have to leave this until Monday.

Well, it took until Wednesday, but this is now fixed.
The code to apply feedback from an alternative didn't reset the part's credit to 0, so it was only reporting the amount awarded in addition to whatever the main part awarded.


Ability to extend post with extra fields and extra placeholders

We would like to add some properties to a blog post which should be manageable in the admin interface and in the web UI.
We currently have a monkeypatch to achieve this. The extra properties are added as an inline to the PostAdminForm, extra placeholders are auto created with signals.
It would be nice if there are some hooks to achieve this. eg:
def register_post_inline(inline)
def register_post_placeholder(model)
This enable any one to create a model and admin like:
class PostExtension(models.Model):
post = models.ForeignKey(Post, on_delete=models.CASCADE, related_name='extension')
some_field = models.CharField(max_length=10)
class PostExtensionInline(admin.TabularInline):
model = PostExtension
fields = ['some_field']
classes = ['collapse']
extra = 1
can_delete = False
def register_post_inline(inline)
{{ post.extension.some_field }}
And:
class PostPlaceholderExtension(models.Model):
post = models.OneToOneField(Post, on_delete=models.CASCADE, related_name='placeholders')
some_placeholder = PlaceholderField('some_placeholder')
register_post_placeholder(PostPlaceholderExtension)
{% render_placeholder post.placeholders.some_placeholder %}
@yakky What do you think? Are inlines a good solution to achieve this? Or do you have better ideas?

I am not against it, but I am a bit worried about the number of features this application is growing :joy:
Can't this already be achiedeved by creating external models pointing to the post and overriding the admin? :thinking:

@yakky The idea is not to include to many features, but giving programmers a generic ability to hook up features that should not be part of a generic blog application.
Some want to include images, some not.
Some want to include extra placeholder, some not.
We have someone that wants some fields which are very specific and should not be in a generic blog application. We already achieved the functionality with a monkeypatch and a signal. I also looked at overriding the admin form, but encountered some issues with the fieldsets. So the monkey patch with a inline seems the most straightforward way to achieve this. Thought it would be nice to offer this functionality in a more generic way.
I will write a proposal. If you think it is to much to maintain we will stick with the current solution.

@g3rb3n I got the idea to just implement hooks to make adding things easier, I was just wondering if it's achievable in a clean way with the current codebase or not, but I have not analyzed in great detail, yet
If you can write a detailed (even if brief) proposal, I think it would be an interesting addition

I have the required changes here:
https://github.com/g3rb3n/djangocms-blog/tree/feature/issue-569-post-extensions
Basically include inlines = [] in admin.PostAdmin
And the following:
def register_post_inline_extension(inline):
    """ Register a admin.Inline to the Post form.
        Model should have a post = models.OneToOneField(Post)
    """
    PostAdmin.inlines = type(PostAdmin.inlines)([inline]) + PostAdmin.inlines

def unregister_post_inline_extension(inline):
    """ Unregister a previously regsitered admin.Inline to the Post form.
    """
    PostAdmin.inlines.remove(inline)

def register_post_placeholder_extension(model):
    """ Register a model with placeholders to be auto created when a post is created.
        Model should have a post = models.OneToOneField(Post)
    """
    signals.post_save.connect(
        create_post_post_save(model), sender=Post, weak=False,
    )

def create_post_post_save(model):
    """ A wrapper for creating create_instance function for a specific model.
    """

    def create_instance(sender, instance, created, **kwargs):
        """ Create Model instance for every new Post.
        """
        if created:
            model.objects.create(post=instance)

    return create_instance

@g3rb3n thanks, now it's clearer :)
Generally speaking I think it's :+1:
I am thinking if we need two different code paths (inlines and onetoone), or we could just go with one and cover all use cases with that, just to make things easier to understand for the people who would need this feature.

I agree that a register hook should be easy to understand. There is a functional difference between the inline and the placeholder. The inline should appear in the form and only has to be created if specified. The placeholder is not in the admin and should always be created if a post is created.
It might be best to communicate this difference clearly in the docs. But maybe there is a clean way to cover both. Signals together with inlines do not work well and introduce edge cases. That should be avoided.
I will look into this!

@g3rb3n let me sit on this and think a possible unified solution
If I don't come up with anything, your implementation is :ok_hand:

I think we have no way around the two different models, but I suggest to slightly refactor the code.
The current placeholder_extension is not required to have a placeholder and it might register an inline itself, to allow adding normal fields and not exclusively placeholder fields, thus the functions might be adapted (pseudocode) this way:
def register_post_extension(inline):
    """ Register a admin.Inline to the Post form.
        Model should have a post = models.ForeignKey(Post) or post = models.OneToOneField(Post)
    """
    PostAdmin.inlines = type(PostAdmin.inlines)([inline]) + PostAdmin.inlines
    .if inline.model. has onetoone:
        signals.post_save.connect(
            create_post_post_save(model), sender=Post, weak=False,
        )

def unregister_post_inline_extension(inline):
    """ Unregister a previously regsitered admin.Inline to the Post form.
    """
    PostAdmin.inlines.remove(inline)
    .if inline.model. has onetoone:
        signals.post_save.disconnect(
            create_post_post_save
        )

This provides the same function for both, and you only need to decide if you want a OneToOne or a ForeignKey
What do you think?

The thing is though that inlines are handled by django and django automatically creates related instances when the admin user specifies the field. When a admin user leaves the field empty the placeholder is also not available nor createable.
The best would be to let django always create the related instance, also when u user does not specify the inline. Than we can also leave out the signals.
I will try this out soon.

@g3rb3n sorry, my pseudocode above probably was not  clear enough
fields with one-to-one must still be created via signal, but it can be handled in the same register function, by checking if the model linked to the inline has a OneToOneField to the post
By always adding the InlineAdmin whatever the key is, we also allow users to add fields on the model with the OneToOneField

So basically one register function and in the function the check on whether this instance should be created with a signal or not. That idea is clear.
There are two situations:

admin user specifies the admin inline -> object is created by django
admin does not specify the inline -> object with placeholder should be created with a signal

placeholder can always be null and are filled in later in the cms ui.
standard django fields can be mandatory, objects should only be created if a user specifies the inline instance.
Or am I missing something here?

It's correct, do you think it's doable? I think that inlines for OneToOneField are always created on save, even if blank, but I have not tested it

I have refactored the code so that there is just one register_extension function. I have also added unregister functionality.
Objects in inlines are not always created as far as I know. If left blank, this might be the expected behavior.
If someone wants to add just a placeholder an inline does not make sense, since placeholders are only visible in the CMS UI, not in the django admin UI. This is now supported, someone can create an OneToOne model and register this model as an extennsion.
I have created two PR's. One based upon the develop branch and one based upon the feature/issue-564-update-tooling branch.
If you have any comments, please let me know.

@g3rb3n awesome! I will review them quickly. #568 has already been merged on develop, BTW so we only need one

Fixed by #590


Implement consistent URL routes

Implement https://github.com/fastlane/ci/pull/207/files, resulting in https://github.com/fastlane/ci/blob/master/docs/UrlRoutes.md

The part of creating resources is missing I think! It'd be useful to add it in a showing revision 😁

@minuscorp can you elaborate ? :) I'm about to start this and I think I have an idea :)

If you take a look into project_controller you'll see that there is two endpoints to create a Project resource, it is a combination of both a get request to start the process and a post request with the final resource creation. This will happen (in a wide variety of ways) across the platform, we should try to normalize how we create resources (models), y'know, the C of CRUD 😁

@minuscorp got it :), of course I have a few ideas. I'm thinking we use convention over configuration for this.

The easiest the better, we want people to actually know how to contribute 😂

Alright, so @jcampbell05 is working on this card 👍

Because we change to a front-end application, so it might not be worth investing more time for now, it's more a nice to have.

@KrauseFx cool no worries, anything else I can do ?

Sure thing, lots of tasks in the pipeline, we tried to list them over here for the Beta milestone https://github.com/fastlane/ci/milestone/14. In particular the ones with the You can do this label are up for grab


Password Field Visibility in Enatega Multivendor Admin Dashboard Login Page

Description:
Upon accessing the Enatega Multivendor Admin Dashboard login page, I observed that when entering the password, the input is not being hidden, compromising the security of user credentials. This poses a potential security risk as sensitive information is visible to onlookers.
Steps to Reproduce:
Open the Enatega Multivendor Admin Dashboard login page.
Enter valid credentials in the respective fields.
Observe that the password entered is not masked or hidden.
Expected Behavior:
The password field should hide the entered characters, displaying them as asterisks or dots, to ensure the confidentiality and security of user passwords.
Actual Behavior:
The password entered in the login page is visible in plain text, raising security concerns.

Resolved!

Resolved!

waiting for publish

Resolved!

Still waiting for publish


Libgdx starts and reports an error on the Ubuntu platform

Please ensure you have given all the following requested information in your report.
Issue details
The system platform is popos, the compiler is vscode, and the Java version is 11. The following exceptions will be reported at runtime
Inconsistency detected by ld.so: dl-lookup.c: 105: check_ match: Assertion `version->filename == NULL || ! _ dl_ name_ match_ p (version->filename, map)' failed!

But I have no problem using the same environment under windows
Version of libGDX and/or relevant dependencies
Current master (version 1.10.0).
Stacktrace
WARNING: Illegal reflective access by org.lwjgl.LWJGLUtil$3 (file:/home/tyq/.gradle/caches/modules-2/files-2.1/org.lwjgl.lwjgl/lwjgl/2.9.3/3df168ac74e4a8c96562cdff24ad352e255bf89c/lwjgl-2.9.3.jar) to method java.lang.ClassLoader.findLibrary(java.lang.String)
WARNING: Illegal reflective access by org.lwjgl.LWJGLUtil$3 (file:/home/tyq/.gradle/caches/modules-2/files-2.1/org.lwjgl.lwjgl/lwjgl/2.9.3/3df168ac74e4a8c96562cdff24ad352e255bf89c/lwjgl-2.9.3.jar) to method java.lang.ClassLoader.findLibrary(java.lang.String)
Inconsistency detected by ld.so: dl-lookup.c: 105: check_match: Assertion `version->filename == NULL || ! _dl_name_match_p (version->filename, map)' failed!

Please select the affected platforms

[x] Linux

Either create a project with lwjgl3 or use java 8 from adoptium jdk.
Lwjgl2 is not updated anymore since years so it doesn't work with newer bundled java version in some linux repos.

OpenJDK 8 from the repositories also works (sudo apt install openjdk-8- then Tab for options). Since it's not clear how to install JDKs and JREs from Adoptium (they only have Windows and macOS installers). But yeah, newer versions of at least OpenJDK on Linux don't work with LWJGL2. The nightly release of gdx-setup and the stable release of gdx-liftoff use LWJGL3, or you can upgrade an existing project as linked above.

Thanks, Use ljwgl3 to solve problems


ORC-682: Upgrade to commons-lang3 3.11

What changes were proposed in this pull request?
Upgrade to commons-lang3.
Why are the changes needed?
This will bring the latest bug fixes.
How was this patch tested?
Pass the CI.

Thank you, @williamhyun !

Merged to master for Apache ORC 1.7.

Oh, it seems that we missed the code usage. I'll make a new PR to update it, @williamhyun .

Oh, it seems that we missed the code usage. I'll make a new PR to update it, @williamhyun .


WIP: Replace KDTree with a thin wrapper over cKDTree

Reference issue
First step towards gh-8923
What does this implement/fix?
This removes the KDTree python implementation in favor of cKDTree.
It turns out these are not actually 100% compatible, so instead of completely replacing KDTree, this makes KDTree a thin compatibility layer over a cKDTree.
Additional information
gh-12372 needs to be merged before this, so we aren't testing cKDTree against itself.
I've been searching for inconsistencies between the two APIs. So far I've found these:

KDTree has a different default leafsize.
cKDTreeNode is the only node type for cKDTree whereas KDTree had seperate leafnode and innernode classes. However, neither of these are a documented part of the KDTree api so maybe we can just drop these?
KDTree.query supports an undocumented query_all_neighbors mode by passing k=None. That is in effect just query_ball_point but it returns distances as well as indices.
There are several places where KDTree returns np.int_ and cKDTree returns np.intp. These are aliases on linux, but are different types on 64-bit windows.

cKDTreeNode is not really a node in the cKDTree. That would be struct ckdtreenode. It is a class that makes the struct ckdtreenode viewable from Python. They are actually created on the fly when traversed from Python and not when we build the tree. The main purpose of cKDTreeNode is for the testsuite to be able to verify the internal structure of cKDTree. With the way they are created they are e.g. not useful for traversing the tree from Python for the purpose of making custom search methods. This should perhaps be clarified in the docstring.

cKDTreeNode is not really a node in the cKDTree. That would be struct ckdtreenode. It is a class that makes the struct ckdtreenode viewable from Python.

I'm not sure I see why this is a meaningful distinction. If it exposes all of the relevant tree structure then it represents the same tree. Whether or not it's relevant to the native implementation is besides the point. You can't build your own trees from it, but I don't see what's stopping you from writing a query:
>>> t = cKDTree(np.random.rand(100, 4), leafsize=4)
>>> target_point = np.random.rand(t.m)
>>> node = t.tree
>>> while node.split_dim != -1:
...     if target_point[node.split_dim] > node.split:
...         node = node.greater
...     else:
...         node = node.lesser
>>> node.data_points
[[0.01195888 0.45111313 0.75893399 0.97949392]
 [0.1652289  0.05808805 0.43595002 0.76127174]
 [0.11536596 0.15262208 0.76160649 0.66025158]]

Nothing stops you from using them for a query, except it will be very ineffecient with respect to speed and memory. These nodes are created dynamically when you access the properties greater and lesser. Not only does it create a view of the node, it also collects the data associated with the node makes a contiguous ndarray associated with the node. One of the reasons for keeping KDTree was that it allowed subclassing in Python to be reasonably efficient. If we are going to replace KDTree we should change the behavior of cKDTreeNode into something more efficient. Right now it has an inefficient mechanism I wrote for the testsuite.

When someone touches cKDTree.tree from Python, the whole structure with cKDTreeNode objects should be created once. Only data_points should be kept as as dynamic property.

I will fix cKDTreeNode in a different PR and ping you when I am done.

These nodes are created dynamically when you access the properties greater and lesser.

Node creation actually seems to be very cheep:
In [1]: t = cKDTree(np.random.rand(100, 10))
In [2]: %timeit t.tree.greater
88.5 ns ± 1.77 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

That's about as fast as a line of python gets. ~4x the cost of a single attribute access, ~1.5x the cost of an empty function call.

Not only does it create a view of the node, it also collects the data associated with the node makes a contiguous ndarray associated with the node.

That's not true, the ndarrays are only materialised when the data_points or indices properties are accessed. These are quite slow since it builds the indices array by appending together each node's indices array recursively:
In [3]: %timeit t.tree.indices
25.6 µs ± 573 ns per loop (mean ± std. dev. of 7 runs, 10000 loops each)

That could certainly be sped up massively. However, it's functionality that KDTree.innernode doesn't offer at all. So, I don't see the argument that this is a pre-requisite to removing the python KDTree.

Node creation might be very cheap, depending on the size of the subtree it spawns. But using the nodes to traverse the tree is not, as we get node dynamic creations instead of attribute lookups. And in this attibute lookup, you get a whole subtree traversal to collect indices. Then on the next access to .greater or .lesser, the whole subtree is traversed again. And so it continues. Looking up .greater or .lesser thus have O(log n) complexity with n the size of the subtree. It should be O(1).

Looking up .greater or .lesser thus have O(log n) complexity with n the size of the subtree. It should be O(1).

I mean, just look at the code yourself. It's very clearly O(1) as all it does is read non-property constant attributes from the node and copy it the new cKDTreeNode instance:
https://github.com/scipy/scipy/blob/b2ce6b179b7419d43dce74f85d02c5c425991c99/scipy/spatial/ckdtree.pyx#L364
Yes, accessing indices and data_points on inner nodes is slow and O(n). But these attributes don't exist at all in KDTree.innernode:
https://github.com/scipy/scipy/blob/b2ce6b179b7419d43dce74f85d02c5c425991c99/scipy/spatial/kdtree.py#L271-L277

As "proof", here is a much deeper tree taking the exact same time to access node.greater:
In [24]: t = sp.cKDTree(np.random.rand(10**6, 4))

In [25]: %timeit t.tree.greater
83.8 ns ± 2.25 ns per loop (mean ± std. dev. of 7 runs, 10000000 loops each)

_setup() here is recursive: https://github.com/scipy/scipy/blob/master/scipy/spatial/ckdtree.pyx#L361 However this is called the first time .tree is accessed.
After that the access to the attribute is O(1), as the python surrogate objects are already created.
... there is a memorization in line 505, and we are thread safe probably due to GIL?

From the comments on the mailing list, it's okay to change the length of integer returned so I've removed that translation code. Less certainty about the NumPy scalars so lets keep it 100% compatible for now and we could always change it later down the road if there's a reason to.
With that decision, I think this is good to go.

I've been busy this month but would like to get this finished up soon. I plan to merge tomorrow if there are no objections, then I'll work on expanding the KDTree api to include the added features from cKDTree.

But if the plan is to retire (deprecate) KDTree, how would this help?

We are retiring the KDTree implementation. If you read through gh-8923, the aim is to make KDTree and cKDTree basically aliases of each other. After all, KDTree has the better name. Long term, users should be able to use KDTree everywhere and forget that there were ever two versions.

Added a section to the release notes.
https://github.com/scipy/scipy/wiki/Release-note-entries-for-SciPy-1.6.0/_compare/2a995a7d65a6c57403d79bf6812e973ebfc2fcf4...425aa67e442c894cbb831e851e7867dff71c662d

Yay! This was on the wish list for a long time. Thanks @peterbell10! And thanks @sturlamolden and @rainwoodman for reviewing!


Move to an offline-first architecture

Just curious and interested, are you planning to use Realm for implementing offline-first architecture? :D

@wotomas Yup, I was. But now I'm thinking of trying it out with the latest architecture components.

@esoxjem I see, that sounds interesting 👍 I haven't had time recently to look into it.. should try though 👍


Fixes and optimizations

This pull request:

Greatly reduces garbage during exports, improving export time.
Fixes error thrown with font_names with underscores
Fixes error when HTMLCanvasElement pattern is sent to C2S instance
Adds fill-rule support as reported in issue #62
Merges in opacity support for issue #53

I tried this patch because the fill=evenodd bug hit me hard.
I can confirm that the evenodd fix works.
Unfortunately I also have to report that some of the other optimisations broke other stuff, I get funny results that were previously ok.

Hey jods, what new issues are you noticing with this pull request?
On Sun, Jun 3, 2018, 11:44 AM jods<EMAIL_ADDRESS>wrote:

I tried this patch because the fill=evenodd bug hit me hard.
I can confirm that the evenodd fix works.
Unfortunately I also have to report that some of the other optimisations
broke other stuff, I get funny results that were previously ok.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/gliffy/canvas2svg/pull/66#issuecomment-394182503, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAGMvOnmhGKR0frkC3vQfK0acdwfu-Rxks5t5C6KgaJpZM4SjoX_
.

@mudcube in my document many lines end up at the origin after your patch, they were drawn correctly before.
Unfortunately it's real documents with around 10K calls to context, that I cannot easily share or reduce.
That said, some tests are currently falling, could that be related?
I can gladly retest my docs when they are fixed.

@jods4 The transformation issue should be sorted out, let me know if you're still running into the issue. There's still a couple failing tests, but part of that is due to slight formatting changes (ex. spacing), and doesn't actually affect what's visually exported.


Target single tag for compilation

Search terms

packageDocumentation
readme

Question
I document the entry point for a whole lot of modules (monorepo and single repo) with the @packageDocumentation tag in /src/index.ts.  The block looks something like:
/**
 * @packageDocumentation
 *
 * This module is a $FOO, it's useful for $BAR.
 *
 * These links might add some context
 *
 * @see http://example.com/hello
 * @see http://example.com/world
 *
 * @example
 *
 * Here's how you do $BAZ with it:
 *
 * ```js
 * // some code here
 * ```
 */

// implementation code here

The package documentation is always declared in it's own comment block and always in /src/index.ts.
I'd really like to include this information in the README so people can see it on npm/github but not have keep the two things in sync manually.  I already auto-generate READMEs based on the properties of the current repo so this would be a case of generating the markdown (preferably) or HTML and adding it to the README template.
Is there a way to have a typedoc plugin that can intercept the output of specific tags or to do this programatically somehow?

Plugins can modify comments:
// @ts-check
import td from "typedoc";

/** @param {td.Application} app */
export function load(app) {
    app.converter.on(td.Converter.EVENT_CREATE_DECLARATION, onCreateDeclaration);
}

/**
 * @param {td.Context} context
 * @param {td.DeclarationReflection} refl
 */
function onCreateDeclaration(context, refl) {
    const symbol = context.project.getSymbolFromReflection(refl);
    const decl = symbol?.getDeclarations()?.[0];
    if (!decl) return;

    if (refl.kindOf(td.ReflectionKind.Module)) {
        console.log("Created declaration for module", refl.getFullName(), "at", decl.getSourceFile().fileName);
        // create/update refl.comment
    }
}

That looks like a simpler solution than the one I came up with, but it doesn't seem to work for me.  Nothing seems to pass the refl.kindOf(td.ReflectionKind.Module) test in mono-repos or single-module repos.
I ended up listening for RendererEvent.END events and looking at each UrlMapping, then detecting if it relates to either a modules.html file (for single-module repos) or if the model.name of the mapping corresponds to a module name in the mono-repo.
It does feel a bit hacky though, I'd prefer a simpler solution if one is possible.

Oops, I completely forgot to consider projects with a single entry point. In that case, the root of the module will be the ProjectReflection.
// @ts-check
import td from "typedoc";

/** @param {td.Application} app */
export function load(app) {
    app.converter.on(td.Converter.EVENT_CREATE_DECLARATION, onCreateDeclaration);
    // The symbol won't be associated yet if listening to EVENT_BEGIN.
    app.converter.on(td.Converter.EVENT_END, onEnd);
}

/**
 * @param {td.Context} context
 */
function onEnd(context) {
    const symbol = context.project.getSymbolFromReflection(context.project);
    const decl = symbol?.getDeclarations()?.[0];
    if (!decl) return;

    console.log("Created project for module", context.project.getFullName(), "at", decl.getSourceFile().fileName);
}

/**
 * @param {td.Context} context
 * @param {td.DeclarationReflection} refl
 */
function onCreateDeclaration(context, refl) {
    const symbol = context.project.getSymbolFromReflection(refl);
    const decl = symbol?.getDeclarations()?.[0];
    if (!decl) return;

    if (refl.kindOf(td.ReflectionKind.Module)) {
        console.log("Created declaration for module", refl.getFullName(), "at", decl.getSourceFile().fileName);
        // create/update refl.comment
    }
}


Create project

Issue Number
🔒 Close #5
🔒 Close #6
New features

프로젝트 생성
Sign 프로비저닝은 개인화 (임시)
iOS 버전 14 고정
스토리보드 삭제
pod dependency 추가
screenshot(optional)

Review points

write here

Questions

write here

References

write here

Checklist

[x] Is the branch you are merging on correct?
[x] Do you comply with coding conventions?
[x] Are there any changes not related to PR?
[x] Has my code been self-reviewed?

LGTM


Report error: Error: The cursor position could not be read within a normal duration

a=$(jnv a.json )
Error: The cursor position could not be read within a normal duration

How can I get the filter data as output or copy it?

@lwx5924 Thanks! But it seems that performing such an operation is not possible, in my understanding. Please try running the filter prepared in jnv against jq like: jq 'something_filter' a.json.

Ok, thanks for your response. And thank you provide such tool. I will close this issue.

Ok, thanks for your response. And thank you provide such tool. I will close this issue.


[Autodiff] Runtime crash when attempting to compute a gradient when the program is built in Release mode.

Description
This appears to affect M1 machines only; I did not see this crash on an x86 Mac.
Steps to reproduce
The following program crashes at runtime when built in Release mode and run. For a command-line build, swift run -c release should be sufficient.
import _Differentiation
func r(a: Float) -> Float {for _ in 0 ..< 1 {}; return a}
func p() {gradient(at: 2, of: r)}
for _ in 0 ..< 2 {p()}

Expected behavior
The program should exit with an exit code of 0.
Environment

Swift compiler version info: This fails with Toolchains 2022-10-08a up until 2023-05-25a. Toolchains 2022-09-24a don't appear to exhibit this issue (I haven't narrowed down the date range between those two any further at this time).
Xcode version info: 14.2
Deployment target: M1. (Note: an x86 Mac does not appear to exhibit this crash, even with the 2023-05-25a toolchain).

Additional context
The for loop in line 2 does not do anything, but is required to replicate this issue.
Running the for loop in line 4 with a range of 0 ..< 1 instead of 0 ..< 2 causes the crash to no longer occur.

We've found some additional information:
Running the program (with swift run -c release) as a single file (outside of a package) does not cause the crash.
If one performs swift package init --type executable, then pastes the above code into the spot where one sees Hello World, then runs swift run -c release, then the program will crash.

This appears to be fixed with the 05-30a toolchain. I'll mark this as closed.


STACK TRACE unable to capture data

I can use it in the application, but I can't find the stack data in the collected event, click to view the stack and it shows below
0   unknown file       unknown method
What can I do to gather useful information?

Hi @fengzhonglingluandeduanfa - please could you write into<EMAIL_ADDRESS>quoting this GitHub issue, with a link to the event in your Bugsnag dashboard. We'd be happy to investigate more from there 👍

Is this the link you need?
https://app.bugsnag.com/dong-wan-xin-ke-ji-zhu-yan-jiu-kai-fa-you-xian-gong-si/comchat-ios/errors/62c3a5ce577210000ae6a551?filters[event.since]=30d&filters[error.status]=open&event_id=62d0caa80095457b6db20000


out_file plugin creates multiple gzipped file instead of a single file

Fluentd version - 0.12.35
OS - ubuntu:14.04
Configuration -
<source>
  @type forward
  port 24224
  bind <IP_ADDRESS>
</source>
<source>
  @type http
  port 8888
  bind <IP_ADDRESS>
  body_size_limit 32m
  keepalive_timeout 10s
</source>
<match **>
  type file
  path /var/log/test/logs
  format json
  time_slice_format %Y%m%d
  time_slice_wait 24h
  compress gzip
  include_tag_key true
  utc
  buffer_path /var/log/test/logs.*
</match>

Issue -
The above config is creating gzipped file every ~10min.
-rw-r--r-- 1 root root  256546 May  6 07:03 logs.20170506_0.log.gz
-rw-r--r-- 1 root root  260730 May  6 07:14 logs.20170506_1.log.gz
-rw-r--r-- 1 root root  261155 May  6 07:25 logs.20170506_2.log.gz
-rw-r--r-- 1 root root  258903 May  6 08:56 logs.20170506_10.log.gz
-rw-r--r-- 1 root root  282680 May  6 09:08 logs.20170506_11.log.gz
...
-rw-r--r-- 1 root root  261973 May  6 10:44 logs.20170506_19.log.gz

Is there any way to have a single file for each day ? Even setting time_slice_wait to 24h didn't help.

See: http://docs.fluentd.org/v0.12/articles/out_file#append

@repeatedly Thanks. Didn't read that part.


Support feature count

This works really well, thank you! It would be helpful to have a way of knowing the total feature count, in order to provide percentages to the user when processing files.
I don't know much about how Shapefiles work, but ogrinfo can provide this info instantly, so presumably it's stored in metadata and doesn't require manuallly counting.

Adding this commit https://github.com/mbostock/shapefile/compare/master...rubillionaire:shapefile:master for anyone else looking interested in this.  Small change from the above, using getUint32 instead of getUint16 to get the feature count, as referenced in this documentation


Ignore mkdocs site directory

mkdocs is rising as an alternative to Sphinx for project's documentation. The default build command puts the generated documentation in a site directory at the root of the project which should be ignored.

@ghisvail thanks!


Expand sliced to allow sliceable objects that don't use bool for emptiness

I was surprised when I tried to use sliced with a numpy array, and it failed.  The issue turned out to be that sliced calls bool on each slice to check if it's empty or not, so it knows when to stop.  By changing it to call len instead, we can use sliced with numpy arrays and other objects that don't use bool to indicate emptiness.
I've also added a test for sliced that uses a numpy array.  However this does add a dependency on numpy for our tests, so maybe we shouldn't keep this new test.

Merged in #373. Thanks!


Bluetooth: controller: Implements ISO Test Mode HCI commands

Implements the flollowing HCI commands:

HCI_LE_ISO_Transmit_Test
HCI_LE_ISO_Receive_Test
HCI_LE_ISO_Read_Test_Counters
HCI_LE_ISO_Test_End

Signed-off-by: Morten Priess<EMAIL_ADDRESS>

DNM until @cvinayak can take a look

@carlescufi, why is this DNM?

CI errors seem unrelated to my PR

CI errors seem unrelated to my PR

Indeed, seems to affect all PRs at the moment


--fix option removes escaped backticks in ES6 template strings

Original code:
const string = `string with \`back ticks\``;

"Fixed code"
const string = `string with `back ticks``;

<EMAIL_ADDRESS>

Would you mind running jscs --verbose, without the --fix flag and posting the result? Your .jscsrc config would also be helpful.

<EMAIL_ADDRESS>
You meant jscs, right?

I've narrowed down the conditions that cause this behaviour:
When the code is free of style errors, running jscs --verbose . returns nothing, and running jscs --fix . has no effect.
When I deliberately introduce an error, e.g. a trailing whitespace (tab character), then running jscs --verbose . results in:
disallowTrailingWhitespace: Illegal trailing whitespace at ./routes/creations.js :
   788 |
   789 |  if (!eligibleMeshAndArchiveFormatsRegex.test(format)) {
   790 |   return next(new httpErrors.BadRequest(`Invalid format value: \`${format}\``));
------------------------------------------------------------------------------------------^
   791 |  }
   792 |

1 code style error found.

Which is correct.
Then, running jscs --fix . fixes the whitespace, but it also strips the escaped backticks. More interestingly, it strips the backticks in several parts of the file (not just the line with the bad style).
I've attached my .jscsrc file as a .zip

<EMAIL_ADDRESS>

You meant jscs, right?

Derp. yes.

Having troubles opening that archive, would you mind posting your config to https://gist.github.com/

Not sure why the zip didn't work… in any case, here's the .jscsrc gist

Yeah, it is a babel (enabled when esnext option is used) issue :/, it removes the backslash - http://astexplorer.net/#/658X2gkWBg from tokens, whereas esprima (enabled when esnext is not used) do it correctly - http://astexplorer.net/#/aKoWHrenCG.
You can trying to remove esnext option from your config, esprima support most of the es6 features nowadays, but still might be incomplete.
Created an issue in the babel bug tracker - https://phabricator.babeljs.io/T6833.
@hzoo can you draw attention to this issue?
Since there is nothing we can do on our side, i'm closing it

Thanks!


Postgres fails inside transaction without initialization

Running an upsert command inside a transaction fails unless you first run it outside a transaction first.   psql version 9.4.5 and 9.5.4
Fails if we run inside a transaction first
require 'upsert/active_record_upsert'
Pet.transaction do
  Pet.upsert({name: 'Jerry'}, breed: 'beagle', created_at: Time.now, updated_at: Time.now)
end

Works correctly if we run outside a transaction first to create the postgres function
require 'upsert/active_record_upsert'
Pet.upsert({name: 'Jerry'}, breed: 'beagle', created_at: Time.now, updated_at: Time.now)

Pet.transaction do
  Pet.upsert({name: 'Jerry'}, breed: 'beagle', created_at: Time.now, updated_at: Time.now)
end

See minimal example...
https://github.com/abaldwin88/upsert_minimal
Is this a known issue?  In our use case we will never be running upsert outside of a transaction.
Is there a recommended work around?  We can add the CREATE OR REPLACE FUNCTION function in a migration.

I believe the issue revolves around the manner in which a missing function is detected... https://github.com/seamusabshere/upsert/blob/1d26591ca0236aa153c1d7be4dcc8eeb2121c15e/lib/upsert/merge_function/PG_Connection.rb#L22-L27
Catching PG::Error when inside a transaction means that the transaction has already been aborted on the database side but we continue to execute commands against it.
Not sure what a good solution would be.  Issuing a ROLLBACK or COMMIT wouldn't be appropriate.  Could possibly consider moving the exception catch into the query but that would impact performance.

@seamusabshere We could query the relevant DB and check for the existence of a function rather than checking for failure?

@abaldwin88 See #87 (once Travis checks everything off as being A-OK).  I'm working up a better solution, though, as there's a significant performance drop when using savepoints so liberally.

@pnomolos Excellent!
Right now I am creating the function before using upsert.  Not great but our use case is only a single upsert command (for now).
Looking forward to putting in a proper fix once your PR is merged - Thanks again.

Fixed via PR #87


[Bug] mage android is an invalid target

Describe the bug
The build target 'android'/'ios' do not work.
Unknown target specified: "android"

To Reproduce

Clone the repo
run mage android

Expected behavior
Since I'm running Linux I never expect iOS to work, so Android only for me.
Environment (please complete the following information):

OS: Linux  5.19.17-2-MANJARO #1 SMP PREEMPT_DYNAMIC Sun Nov 6 00:08:27 UTC 2022 x86_64 GNU/Linux
Go: 1.20

PS This log line reads iOS where it should read Android: https://github.com/TBD54566975/ssi-sdk/blob/main/magefile.go#L364

I've hacked a new build target in the mage file to get Android to build. That works but the build ultimately fails with:
❯ mage foo
Building Android - API Level: 23...
Building [crypto] package...
Building [did] package...
gomobile: /home/harm/go/bin/gobind -lang=go,java -outdir=/tmp/gomobile-work-4274079403 did failed: exit status 1
no exported names in the package "did"
no exported names in the package "did"
no exported names in the package "did"
no exported names in the package "did"

FATA[0004] Error building Android pkg: did               error="running \"gomobile bind -target android -androidapi 23 did\" failed with exit code 1"

Thanks @haarts that error looks correct. I'm wondering if it's best to remove the mage commands for ios and android in favor of our mobile SDK specific repo. What do you think @shamilovtim?

Thanks @haarts that error looks correct. I'm wondering if it's best to remove the mage commands for ios and android in favor of our mobile SDK specific repo. What do you think @shamilovtim?

That makes sense for now

https://github.com/TBD54566975/ssi-sdk/pull/324


Error when training with more than 60000 images

Hi!
I was trying to train the network with 79k images but it is showing this error
Length Error:  31
when I trained with 35k images the network was working fine. In this I have positive values for all the coordinates but in the 2nd set of images I have some negative values too . Is it because of this?
2041790.jpg -61,-461,-5,-262,-71,-252,59,-272,-18,-159,8,-333,19,-71,34,-189,32,-172,19,-113,87,-125,64,-52,67,-55,95,-83,103,-30,1
2041791.jpg -59,-461,0,-258,-68,-249,65,-267,-15,-151,9,-333,18,-71,36,-183,36,-168,25,-110,89,-121,68,-51,91,-48,96,-83,102,-13,1
2041792.jpg -60,-456,5,-252,-66,-246,75,-257,-13,-146,7,-322,27,-60,33,-189,38,-166,24,-111,89,-119,67,-51,92,-49,94,-79,103,-18,1
2041793.jpg -58,-457,5,-253,-66,-249,75,-257,-15,-150,6,-325,36,-55,32,-186,38,-166,24,-112,90,-118,67,-50,96,-51,97,-69,103,-19,1
2041794.jpg -58,-455,5,-255,-67,-253,75,-258,-15,-152,13,-297,35,-57,39,-158,36,-169,20,-115,89,-121,51,-47,98,-53,100,-28,106,-19,1
2041795.jpg -57,-452,7,-250,-62,-247,74,-254,-13,-148,3,-312,37,-57,5,-177,37,-167,21,-113,90,-119,54,-46,36,-54,79,-70,100,-35,1
2041796.jpg -52,-441,-33,-299,-93,-292,25,-306,-35,-185,69,-219,13,-98,69,-97,18,-195,7,-130,88,-140,40,-58,110,-56,75,-79,97,-31,1
2041797.jpg -56,-447,-30,-294,-85,-285,25,-302,-35,-185,70,-214,15,-98,58,-107,17,-193,2,-128,88,-142,35,-64,103,-62,82,-58,98,-31,1
2041798.jpg -57,-452,-1,-272,-74,-284,64,-261,-28,-178,50,-207,18,-75,34,-116,26,-170,14,-112,69,-108,52,-52,69,-53,108,-42,105,-28,1

I tried to change the tmp parameter in cpm.py file line 262
def sub(batch, idx):
           name = x.op.name
           tmp = x[batch, :, :, idx] * 255
           tmp = tf.expand_dims(tmp, axis = 2)
           tmp = tf.expand_dims(tmp, axis = 0)
           tf.summary.image(name + '-' + str(idx), tmp, max_outputs = 100)

even then it is showing the same error.
Could be please suggest anything regarding this?
Thanks in Advance!

Hi, have you solved the problem?

Hi yes, we can train it with more than 60k images. the error was due to some missing values in the dataset.


About model covariates

I was trying to update a reference scANVI model with a new query dataset. But apparently if the reference model has been trained with categorical covariates included this can't be done as of now.
Is there a way around this in the meantime? Re-training the reference model without the categorical covariate shouldn't be an option, as we've seen that including it greatly improves learning.
A minimal example that reproduces the behaviour:
import scvi
import scanpy as sc
import scarches as sca

adata = sc.read(
    "data/lung_atlas.h5ad",
    backup_url="https://figshare.com/ndownloader/files/24539942",
)
reference = adata[1:7001, :].copy()
query = adata[7002:10002, :].copy()

reference.obs["categorical_covariate"] = ['A'] * 3500 + ['B'] * 3500
query.obs["categorical_covariate"] = ['A'] * 1500 + ['B'] * 1500

# train reference model

reference.raw = reference  # keep full dimension safe
sc.pp.highly_variable_genes(
    reference,
    flavor="seurat_v3",
    n_top_genes=2000,
    layer="counts",
    batch_key="batch",
    subset=True
)

reference.obs_names_make_unique()

scvi.model.SCVI.setup_anndata(
    reference,
    layer="counts",
    batch_key="batch",
    continuous_covariate_keys=["percent.mito"],
    categorical_covariate_keys=["categorical_covariate"]
)

vae = scvi.model.SCVI(reference, n_layers=2, n_latent=30, gene_likelihood="nb")

vae.train(max_epochs=5)

query = query[:, reference.var[reference.var.highly_variable].index.to_list()].copy()

updated_model = scvi.model.SCVI.load_query_data(
    query,
    vae,
    freeze_dropout = True,
)

Full error
---------------------------------------------------------------------------
NotImplementedError                       Traceback (most recent call last)
Cell In [69], line 1
----> 1 updated_model = scvi.model.SCVI.load_query_data(
      2     query,
      3     vae,
      4     freeze_dropout = True,
      5 )

File conda_envs/scArches/lib/python3.10/site-packages/scvi/model/base/_archesmixin.py:113, in ArchesMixin.load_query_data(cls, adata, reference_model, inplace_subset_query_vars, use_gpu, unfrozen, freeze_dropout, freeze_expression, freeze_decoder_first_layer, freeze_batchnorm_encoder, freeze_batchnorm_decoder, freeze_classifier)
    110 adata_manager = model.get_anndata_manager(adata, required=True)
    112 if REGISTRY_KEYS.CAT_COVS_KEY in adata_manager.data_registry:
--> 113     raise NotImplementedError(
    114         "scArches currently does not support models with extra categorical covariates."
    115     )
    117 version_split = adata_manager.registry[_constants._SCVI_VERSION_KEY].split(".")
    118 if int(version_split[1]) < 8 and int(version_split[0]) == 0:

NotImplementedError: scArches currently does not support models with extra categorical covariates.

Same issue here, any updates?

I'm also having the same issue

@Koncopd


New copy button with starlight style

Summary
Updating the copy button with the starlight component to keep asymmetric style in the website.

Hi @ematipico I had to update to starlight v0.19 to use the code component also to use this new version required to update some peer dependencies. I'm still working on this just I have some questions.
This is a good way or should I discard updating to new versions?
packages/@biomejs/wasm-web is not longer there, there is a new location for this package?

@eyubkh you might want to rebase your PR. We added Starlight 0.19

@ematipico oh great I will close this pull request and open a different one, I have done it in a different branch.


<EMAIL_ADDRESS>compatibility

Hello,
I think you already know,
but I write you about this issue.
npm error While resolving<EMAIL_ADDRESS>npm error Found<EMAIL_ADDRESS>npm error node_modules/ol
npm error<EMAIL_ADDRESS>from the root project
npm error [...]
npm error Could not resolve dependency:
npm error peer<EMAIL_ADDRESS><10.3.0" from<EMAIL_ADDRESS>npm error node_modules/ol-mbtiles
npm error<EMAIL_ADDRESS>from the root project

Thank you,

Yes, I have a PR in the works, but since I was evicted from my home because of an extortion organized with the French police and some of the biggest companies in geospatial data, I am currently working from my van and I have limited access to electricity. I will try to merge it as soon as I can afford to do so.


Enable antialias

Im trying to test stereoscopic rendering with antialiasing enabled (In THREE.js). In the VREffect code i've change one line of code so predistored is set to true (to disable distortion effects in polyfill), but I'm still not able to get antialias to work. Any pointers to what else i should do? Antialias works when its not in presentation mode.
resolve( vrDisplay.requestPresent( [ { source: canvas, predistorted: true } ] ) );

Hey! Not sure about what's causing this. I think an important thing is that we support antialiasing in the native WebVR API @toji

This is a limitation of the polyfill and always will be for WebGL 1. The only surface that can be antialiased is the backbuffer, but the polyfill works by intercepting the draw calls and redirecting them to an intermediate surface. This means that we lose antialiasing on a technicality.
That won't be an issue for the native implementation.

Would it be possible to do FXAA on the intermediate buffer?


possibly undefined macros during autogen.sh

When I ran autogen.sh after building the depends, I got an error from a few "possibly undefined" macros. It suggested to include m4_pattern_allow, so I added the command into the file. When I ran it again, I got another error, saying that configure.ac is required. Even when resetting everything but depends back before I ran it for the first time, I still got the error. I have no idea why this is happening, but there seems to be no workaround to it.
Here's the errors:
with m4_pattern_allow:

without m4_pattern_allow:

I'm using the Debian distro with amd x64. This OS is being ran on an OracleVM virtual machine.

iirc, I'm using version 12 (Bookworm).
On Wed, Jul 26, 2023 at 3:03 PM Old Dip Tracker @.***>
wrote:

What version of Debian are you using?
—
Reply to this email directly, view it on GitHub
https://github.com/FACT0RN/FACT0RN/issues/31#issuecomment-1652338923,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVP26LFKV7IVGYVRJMU4AC3XSFSWZANCNFSM6AAAAAA2YWB7HY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do you have installed the latest versions of: pkg-config libtool? Historically this is usually the issue with undefined macros and configure.ac.

It appears that reinstalling pkg-config has solved the problem. Thanks for
the help!
On Wed, Jul 26, 2023 at 3:46 PM Escanor Liones @.***>
wrote:

Do you have installed the latest versions of: pkg-config libtool?
Historically this is usually the issue with undefined macros and
configure.ac.
—
Reply to this email directly, view it on GitHub
https://github.com/FACT0RN/FACT0RN/issues/31#issuecomment-1652398121,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVP26LAUXSLZAWRWRS5PFQLXSFX2HANCNFSM6AAAAAA2YWB7HY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Sure.


Fix: columns reset button a11y

Makes columns reset button accessible by keyboard.
https://uipath.atlassian.net/browse/PLT-35789

pls open up a PR for master as well.


Fix Google Music Manager (music-manager) pref pane

The app will copy the pref pane itself on first run (if it does not already exist), but then brew uninstall will not remove it.
By letting brew symlink the pref pane, it uninstalls properly.

There was a PR template that you needed to fill out. Please try again and fill it.

You may do so in this PR or a new one. Whatever you prefer.

Sorry, I commited and set the PR from the command using hub, so I did not see a template …
I took your hint though, the style issue is corrected.

[x] Commit message includes cask’s name (and new version, if applicable).
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Where is that PR template file located though, for future reference?

Located here. It should appear automatically, though.
It doesn’t appear now due to hub, but I think they are working on it.


Experience obelisk

Experience obelisk allows you to store all but only allows you to retrieve 6 lvls.  If you keep hitting store all/ retrieve all it keeps increasing the stored lvl.
Ender io 1.8.9-<IP_ADDRESS>
forge 1764

wrong project

Sorry Henry. where do I report


Add support for framework prefixes

It was already mentioned in #148 by @Mihailoff , I think it's a useful case to direct whole frameworks to specific paths instead of overriding specific modules on at a time.
{
    "extra": {
        "installer-paths-prefix": {
          "framework-folder": ["framework:wordpress"]
        }
    }

}

Sorry the original PR was flawed. I fixed the issue and amended the unit tests.

Closing as the implementation seems broken and it's way outdated/probably not needed anymore.


Add loading of shift-o output

This will be useful for debugging purposes when things don't work as expected

initial work here
https://github.com/dankilman/multiplex/tree/load

resolved by #19


Relative redirect is not behaving as expected.

Suppose you have the following page structure:
/app/routes/index.js
/app/routes/page1.js
/app/routes/page1/page2.js
And in page2.js, you have the following action:
export let action: Action = async () => {
  return redirect("..");
};

When that action is called, you would expect the browser to navigate from /page1/page2 to /page1. However, it is currently navigating all the way back to /.

Note: Earlier today, the app wouldn't even build with a relative redirect, with a very unhelpful Rollup "Unexpected token" error. I was able to resolve the error by removing that action altogether. Later, when I was testing it again to make this issue, I wasn't able to reproduce that error.

I just noticed an inconsistency with browser vs Remix behavior here. If there's no js on the page, the redirect('..') works as expected and goes from /page1/page2 to /page1. If there is js on the page, then it goes all the way back to /.

Yeah I just hit this problem too. The mismatch between the no-js and js experience is especially concerning, since it makes this impossible to work around. Some of our users will get the wrong result whichever way we go.

@oliverjam can you open a PR with a failing test case? 🙏 https://github.com/remix-run/remix/blob/dev/integration/bug-report-test.ts


6.2.10

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

[ v ] Does your manifest conform to the 1.5 schema?

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

Microsoft Reviewers: Open in CodeFlow

Can someone approve it?


GCP: Support for Subnetwork Resource with High Fidelity

This issue description is pretty much identical to #654 but for Subnetwork resource.
Subnetwork resource in GCP to be used for connecting different managed resources.
Part of #615
What seems to be the problem?
We need to manage the subnetwork for managed resources in GCP to have different resources be able to connect to each other. See https://github.com/crossplaneio/crossplane/blob/master/design/one-pager-resource-connectivity-mvp.md for connectivity architecture.
What could Crossplane do to help?
Have support for a new CRD specific to GCP's Subnetwork with high fidelity.
What does it look like when we're done?
A managed resource that we can connect different resources to each other in GCP. It should have all of the configurable options that GCP provides.
Todo

[ ] Create new group for networking if it doesn't exist.
[ ] Create GCP subnetwork CRD struct and its boilerplate.
[ ] Implement ExternalClient and ExternalConnecter interface functions in its generic managed reconciler.

Fixed in #699