is there any way to repost a user's posts ?

@osmanmmz Did you find a way to get users media to repost?

when get "feedResult.getImage_versions2()" it returns a json it have a link witch contains jpg or mp4. get the link and download than post like its your content.
try (BufferedInputStream in = new BufferedInputStream(new URL(link).openStream());
FileOutputStream fileOutputStream =new FileOutputStream("file.mp4"/or jpg/)) {
byte dataBuffer[] = new byte[1024];
int bytesRead;
while ((bytesRead = in.read(dataBuffer, 0, 1024)) != -1) {
fileOutputStream.write(dataBuffer, 0, bytesRead);
}
} catch (IOException e) {
e.printStackTrace();
}

@osmanmmz if you colud take a look in my project above I appreciate

@krismorte thanks i will.


Skip 32bit platform warning test after node 8.9.0

As asnyc stack traces will not be turned on by default after node 8.9.0,
The warning message of 'async stack trace is not available on 32 bit
platform' no longer emitted. We will not emit this message after node
8.9.0.

I think the phrase As asnyc stack traces will not be turned on by default after node 8.9.0... should be changed to As asnyc stack traces will not be turned on by default starting with node 8.9.0 because async stack traces are not turned on by default in 8.9.0 but the original phrasing makes it sound like it is not until 8.9.0 that that change occurs.

Also the title Skip 32bit platform warning test after node 8.9.0 would probably be better written as Skip 32bit platform warning starting with node 8.9.0.


Not compatible with Visual Studio 2017?

Hi,
I'm trying to complete Lab 1. The only available install of VS is 2017. When installed (with the Windows 8 SDK extension), a universal app does not seem to be possible -the WindowsPreview.Kinect reference is grayed out (along with all the other Kinect references).  If selected, when building the solution an error is presented -something along the lines of "WindowsPreview.Kinect" references  for "Microsoft.VCLibs Version 12" which references "Visual C++ 2013 Runtime Package for Windows Universal Version 12".  While version 14 is available to reference -I cannot find an extension SDK to install that makes 12 available; and referencing 14 does not resolve the error.
Is anyone else having this problem? Is VS 2017 simply not compatible?!
Thank you.

These labs were written for Visual Studio 2013 and Windows 8.1.
There is some work required to bring them up to date with Windows 10 and VS 2017.
If you (or anyone) wants to help contribute back to update the labs that would be great.

I'd be happy to update the tutorial -but I need some help getting started.  I don't know if I'm waiting on library updates for UWP app development, or additional SDKs. Or anything else for that matter.

Visual Studio 2013 Community Update 5 is still available. I had the same unresolvable problem as you MiMqz so I'm currently downloading and installing it to see if I get any further. Ive managed to get all the SDK V2 code examples working in Windows 10/ VS 2017 but a fair bit of mucking around updating them, it didnt happen automatically at first . Also finally Windows 10 prompted me to go into "developer" mode which previously I gather, the package manifest grants granular permissions to hardware resources like camera and microphones. Open Source drivers for the Kinect had no luck but this is a big learning curve after decades out of IT, and just a little VB development. Avoided C++ like the plague.

I meet the similar problem,the lab_project can not open in  Windows 10/ VS 2017 .Can someone help solve this problem?

Hello i am having the same issue

Hi everyone. I updated some of the tutorials from this repository. They are working on Visual Studio 2019 as of today August 2021. I also created the step-wise tutorials.
The link to the repository
Cheers,


Width Issues

Hello,
I looked through the previous issues but didn't see any issues similar to what I'm experiencing so I thought I would create one.  Please feel free to close/destroy this ticket if it's something that has already been brought up.
Boostrap version: 2.3.2
Bootstrap-switch version: 3.3.2
Issue: It appears the auto width is not getting set correct:

Sometimes there's white space, more often with the smaller ones, and sometimes it's not auto scaling to the proper width.  There's /is/ a chance the CSS from other plugins might be messing with the overall CSS (I've checked the elements in both Firefox and Chrome to no avail), so if there is any further information I can provide, please let me know.

mis-matched bootstrap version is probably the issue.

I'll keep investigating, but pulling from master, there's specific files for CSS, but the js files appear to be bootstrap version independent.  If I discover anything, I'll let you guys know.  Thanks for the update. :)

Same here! It works ok but the width appears bad! plz help!

It may be an issue with the size of the container they are in as well. does it happen if you put them in a wide open space?

Solved!
It works updating bootstrap and jquery plugins to the current version. I was using jquery 1.7.
If i put it on a wide open space it happens the same. Thanks for your help!

I've discovered that if I use the bootstrap3 CSS file with 2.3, the switches work as intended.

I also had this exact problem, I updated my jQuery version from 2.0.3 to 2.1.3 -> Solved

+1 raphjutras, updating jQuery to 2.1.3 solves the width display issue.


Fix for case where there are no search results

In the case that there are no results there is not a next page token
and so previous to this change the get_next_page_token
method would result in the exception "undefined method `[]' for
nil:NilClass"

Thanks :)


Was this service designed for reverse proxy only?

Was this service designed for reverse proxy only?
I can think of a way to use it in as a forward proxy but couple things are not really sitting in my head around the design of the software.

The design oscillated between using http.Handler and http.RoundTripper, which is probably the key difference. I assume you are wanting to use it as a transport layer for a http.Client or something similar?
I'd really like to include a RoundTripper or Transport specifically for this. At the minute it could be done with the httptest.ResponseRecorder I believe. Happy to accept and collaborate on pull requests.

I did't had much time to read the whole software but what I have tried is a defined proxy for curl\wget\firefox without the reverse proxy parts of the code and also with this:
http://paste.ngtech.co.il/p15wshitz
I have encountered an unexpected issue and I wanted to first verify that I am using the tool properly and in the way it was supposed to be used.


version 4.0 does not work at all

Even samples does not work, requires crdentials ini file. I store credentials in the database so do not use ini files.

reading guide.


Not committing the right offset when closing or pausing the ConsumerGroupStream

Bug Report
I use ConsumerGroupStream with async.queue. And I have a number of threads defined when adding items to the queue. I pause ConsumerGroupStream when 200 items are consumed and queued. Then when the queue is drained I call resume() on the ConsumerGroupStream.
For example if I have 349 Items evented. I consume 200 pause() add to the queue. When queue is drained consume the next 149. When finished close() the consumerGroupStream and destroy it. Then on demand dynamically create the ConsumerGroupStream I receive the same 149 that I already consumed. If I continue doing so I continue receiving the 149 again and again.
Environment

Node version: 5.0.0
Kafka-node version:
Kafka version:

For specific cases also provide

Number of Brokers: 3
Number partitions for topic: 1

const consumerGroupOptions = {
  'kafkaHost': config.get('kafka.kafkaHost'), // connect directly to kafka broker (instantiates a KafkaClient)
  'ssl': true,
  'sslOptions': certsUtils.createSslOptions(),
  'groupId': config.get('kafka.groupId'),
  'sessionTimeout': config.get('kafka.sessionTimeout'),
  'protocol': ['roundrobin'],
  'fromOffset': 'latest', // default
  'commitOffsetsOnFirstJoin': true,
};

this.consumerGroupStream = new ConsumerGroupStream(Object.assign(consumerGroupOptions), [topic]);
this.consumerGroupStream.on('data', this.onMessage);
this.consumerGroupStream.on('error', this.onError);

// when some event happens this function is called to close and destroy the consumer group stream
stopConsumer(correlationId) {
    if(this.consumerGroupStream) {
      this.consumerGroupStream.close(() => {
        logger.info('Batch Kafka Consumer is closed', {}, 'BatchKafkaConsumer.stopConsumer', correlationId);
        this.consumerGroupStream.destroy();
        this.consumerGroupStream = null;
      });
    }
  }

I found out that this happens when you dynamically initialize your ConsumerGroupStream. I tried closing the ConsumerGroupStream before creating a new one. I also tried manually commit the offset.
Even if I commit the offset I still see the same message with the same offset come back again! when the consumerGroupStream is created.


[BUG - BlackMythWukong ]Unable to load blueprint mod

Branch or Release
The Tag for the Release you are using, or the commit you used if you built from source yourself.
https://www.nexusmods.com/blackmythwukong/mods/19
Game and Engine Version
Game where the bug occurred, and the UE engine version of the game.
Game: Black Myth Wukong
UE engine: https://github.com/kboykboy2/UnrealEngine/
Describe the bug
A clear and concise description of what the bug is.
Unable to load the blueprint mod packaged by this engine
Mods directory
A zip of your Mods folder if you've installed any mods that don't come with UE4SS by default
To Reproduce
Steps to reproduce the behavior:

Launch game
Attempt UHT Dump
See error in log

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots, UE4SS Log, and .dmp file
If applicable, add screenshots to help explain your problem, and upload the log and .dmp files located in the Win64 directory.
error.txt

Desktop (please complete the following information):

OS: [e.g. Win10, Win11]
W11 64x

Additional context
Add any other context about the problem here.

This is unrelated to the base UE4SS repo and likely user error in packaging.

This is unrelated to the base UE4SS repo and likely user error in packaging.

Is this an engine issue or a project issue? The packaging settings are the same as using the official engine and project normally

This is unrelated to the base UE4SS repo and likely user error in packaging.

The error comes from the packing process.
But here's how we could improve the error message when AssetData is nil.
https://github.com/UE4SS-RE/RE-UE4SS/blob/e123730ef514e7e65698881e5f7124d43e3ff94f/assets/Mods/BPModLoaderMod/Scripts/main.lua#L210-L213

Yes, the error message is a leftover from the old script that I forgot to fix.  Regardless, this discussion should not be happening on this repo, it should be on nexus.


accumulo-client properties like "ssl.truststore.password" do not get found inside a CredentialProvider if one is provided

Describe the bug
When you setup a CredentialProvider JCEKS file with an alias "ssl.truststore.password" for use in the accumulo-client, then Accumulo will not be able to find and retrieve that value from the JCEKS. I got a bit lost, but I think what was happening was that the ClientConfConverter class was only ever looking for the server property "rpc.javax.net.ssl.trustStorePassword" and not the client-specific value. So long as I put "rpc.javax.net.ssl.trustStorePassword" as an alias into the JCEKs file I didn't have an issue.
To Reproduce
Steps to reproduce the behavior (or a link to an example repository that reproduces the problem):

Setup an accumulo-client.properties with basic info but importantly ssl enabled, a credential provider specified, and a ssl.truststore.path

#Whatever you have for instance.name and instance.zookeepers in accumulo.properties
instance.name=instance
instance.zookeepers=zookeeper:2181
#When defined, Accumulo will try to retrieve matching properties out of the jceks
general.security.credential.provider.paths=jceks://file/path/to/accumulo/conf/accumulo.jceks
auth.principal=accumulo
auth.type=password
auth.token=password
#SSL Configuration
ssl.enabled=true
ssl.truststore.path=/path/to/accumulo/conf/ssl/truststore.jks

Launch Accumulo itself with SSL enabled, then try to connect to Accumulo shell with the above accumulo-client.properties
See error

 [root@accumulo scripts]# accumulo shell --config-file accumulo-client.properties
2024-05-23T13:27:21,382 [clients.ClientServiceThriftClient] WARN : Failed to find an available server in the list of servers: [ssl:accumulo:9997 (120000)]

With java debugging enabled you will see that it is giving an empty password to the truststore. And if you debug on line 172 of ClientConfConverter.java and click play a few times to see the properties it's looking for, you will find that it is only searching the for properties like "rpc.javax.net.ssl.trustStorePassword"
Expected behavior
Entering Client properties should work in the CredentialProvider lookup in the same way that we are able to use the Client properties in the accumulo-client.properties file. I'm unsure if that's feasible if the list of properties supported by Credentialprovider is defined by Hadoop and not Accumulo.
In which case, another (easier) solution would be to update the list of properties in the documentation to include a caveat about CredentialProviders and how the property name conversion thing means you need to put the server property into the JCEKS. For example, in the Description column of "ssl.truststore.password" you could include "Note: if protecting this property in a CredentialProvider specified in 'general.security.credential.provider.paths', then you must use the property "rpc.javax.net.ssl.trustStorePassword" inside of the CredentialProvider."
Semi-related when it comes to properties documentation, but I think there are a few server properties you do need to specify in accumulo-client.properties that aren't documented on the client properties page. For example, if Accumulo is using TLSv1.3, then the accumulo-client.properties file has to specify "rpc.ssl.client.protocol=TLSv1.3" though that property is only listed in the server properties page, otherwise you get infinite
"2024-05-23T13:34:54,194 [transport.TIOStreamTransport] WARN : Error closing output stream.
java.net.SocketException: Connection or outbound has closed
" stack traces. You have to put the property in both server and client property files if the value isn't the default TLSv1.2, or else "accumulo-cluster stop" will throw a similar error. Oh, and currently the client properties don't mention that general.security.credential.provider.paths is a supported property either, though I'm very grateful that it is supported still.

You are correct that the Hadoop CredentialProvider mechanism is not used for the SSL properties.
However, I am not sure about implementing that feature.  I'm not very familiar with all the CredentialProvider mechanisms, but the one that I've interacted with (jceks file used for tests that I recently updated in #4580) isn't that secure. It is just a Java keystore that doesn't have a passphrase.. which is not much better than a plaintext file. Perhaps CredentialProvider has implementations that are more secure, or there's a way to securely pass a passphrase for the jceks file itself, but I'm not familiar with those methods at the moment.
By contract, the whole point of these SSL properties is to allow you to store the relevant SSL/TLS certificates into a keystore. In other words, it seems to serve a similar purpose. It sounds like you're wanting to store the passphrase for the keystore holding the certs in a separate keystore holding the passphrase. But then, is that second keystore going to be encrypted? If so, where are you going to store the passphrase for that second keystore? Wouldn't you just have to pass that to CredentialProvider (assuming it even supports that, which as I said above, I'm not sure about)?
I'm sure I have room to learn some more about how the CredentialProvider works, but without more information, it currently seems to me that it's turtles all the way down and I don't see much point in adding code to support adding another layer. If there is a way to securely unlock an encrypted keystore without a keystore passphrase needing to be stored insecurely, perhaps the JSSE system properties (which these client properties are intended to mirror) support something more naturally, without using Hadoop's CredentialProvider in an insecure way? I'm not sure the answer, but it seems more worth pursuing that than it does adding an extra insecure layer.

@gwynlionhart Which version of Accumulo are you using? And did this work in a previous version?

I previously was using Accumulo 1.10.2. At the time my accumulo.conf had the same-named properties as my server's accumulo-site.xml, so the JCEKS files for each had the same alias names for both server and client JCEKS files (the client had fewer properties of course, not needing the root user's password for example).
When I upgraded to 2.1.2, I moved the client side from accumulo.conf to accumulo-client.properties and tried updating the property names in line with what the docs defined. Though in my original post aboveI did note that some properties are omitted from the documentation that ended up required for me, but that's a different issue. I also eventually stopped using accumulo-client.properties due to #4573 and the issue with CredentialProviderTokens, but will go back to the properties file when 2.1.3 releases with that fix.
So when I updated to using the accumulo-client.properties, I moved to using ssl.truststore.password and updated my JCEKS keystore to have that alias, but then I encountered issues arising from the Accumulo client not finding the truststore password during initialization. When I reverted the JCEKS file's alias for that entry back to "rpc.javax.net.ssl.trustStorePassword", then the client initialized correctly


Cronjob delete issues

Cronjob delete still have cronjob running(although not visible),as when i add a job with same name again,after delete,first time the method setup for execution in cronjob code,is called as many number of times,as time elapsed after cronjob was deleted divided by time duration bet ween cronjobs.Ex- if job interval is 20 seconds and i adda job with same name after deleting it,after 40 seconds it will excute first time for 2 times.If i add job after more than 120 s it will execute first time for 6 times.

@TestFirebaseLogin Does this still happen on master?

Duplicate of #416


git salt state trying to use brew from incorrect location

The git state is trying to run brew from arch -arm64 /opt/homebrew/bin/brew install git when the new versions of brew are installed in /usr/local/bin/brew

Changes have landed in the salt states repository.


[Wizard] Needs validation hook

We're validating the step's contents via an API call, so the "next" trigger is happening as a callback. There are other products needing dynamic/async based roadblocking and next step triggering.
Specifics to come.

@jbarngr ^

@kevinparkerson @interactivellama Thoughts? This is a blocker for both MobilePush and GroupConnect.

Why can't you cancel/suppress the actionclicked event and then call the next method when validation is successful?

@kjaenicke ^

@mbeard Providing a callback at least gives the developer a method to internally override the default functionality, without having to completely override all functionality built in the wizard. My main reason for this is the ability to easily prevent going to the next step while waiting on a server side response when saving draft states between each step.

Canceling the event it's really completely overriding the entire wizard.  It's not really doing anything other than preventing the next method from being called internally...so you can call subsequently call it when you need to - you can see that here
Presumably, your already doing some logic on step change to determine what step you are on (to know if you need to do validation at all, or which route you need to call to validate if you have validation on multiple steps.  This would just be one additional snippet to cancel the event.
With a callback, I think you'd have to handle both success conditions (calling next), in addition to error conditions (remaining on the page), and we'd have to deal with the callback not returning at all (if there was some sort of scripting error).  I guess what I'm saying is that I don't see the callback as any more elegant than solution we have to support this now.

@jbarngr Does that help?

I would like the wizard to keep track of a boolean state that it checks to make sure that the next function can initiate the .next function or that can be checked in the next function and pause the step process.

Guys, can we all get in a room for a few minutes to whiteboard as a team?

*"we all" meaning, whoever needs to be involved?

A meeting is on the books.
Scott Williams
Sr. Front-End Engineer, UX ArchitectureUXDS, Salesforce Marketing Cloud
On Fri, Jun 5, 2015 at 2:01 PM, Nick Kirkes<EMAIL_ADDRESS>wrote:

*"we all" meaning, whoever needs to be involved?
—
Reply to this email directly or view it on GitHub
https://github.com/ExactTarget/fuelux/issues/1320#issuecomment-109381764
.

Conversation has been transferred to #1347.


How to improve the odemotry accuracy？ I found this method is not robust，when I change the camera and image(1280*720). But I want to apply this method to the actual scene , please help me, thanks

Make sure you provide true calibration data in params.yaml file. There are several methods to increase accuracy. We have mentioned them in the paper.

Thank you for your reply. I tested this algorithm on the emulator (Carla), so the camera parameters should be correct.I will read this paper further and try to improve the accuracy of the algorithm

this python version， I fixed some operational issues。

Greate @wuyuanmm
Thanks for making the Python wrapper. I will add this to the Readme so that more users can find it.


Setting dark theme in _config.yml appears to be ignored

Using minima, setting the theme to dark inside of _config.yml appears to be getting ignored:
minima:
  skin: dark

The other configuration appears to be getting pulled in correctly, so it's not an issue of _config.yml being ignored entirely. It's specifically the theme value that has no effect.
To replicate, see: https://github.com/joshbuker/gsd-tools/tree/2269b5ba3a054844eaaf19598a6870a6245f658b/website
The issue happens both when running locally and when deployed via Cloudflare pages.

I have the same Issue, both with a local and a Github Pages deployment.

For folks looking to use dark theme before 3.0 gets released, update your files as follows:
_config.yml
# Disable using the latest Minima release
#theme: minima
# And instead pull straight from GitHub via jekyll remote theme
remote_theme: jekyll/minima

plugins:
  - jekyll-seo-tag
  - jekyll-feed
  - jekyll-remote-feed

minima:
  skin: dark

Gemfile
group :jekyll_plugins do
  gem 'jekyll-seo-tag'
  gem 'jekyll-feed'
  gem 'jekyll-remote-theme'
end


Doc - Describe cairo_run using Rustdoc comments (three slashes: ///)

In an effort of having fully documented code, we need our cairo runner to be documented.
Please proceed to comment above the pub fn cairo_run declaration in an equivalent way as pub fn compile is documented.

Hi, can I work on this issue?

Sure thing!


Unicode::Collate does not have Hebrew locale

The Unicode::Collate package that biber uses is missing the Hebrew locale, Unicode/Collate/Locale/he.pl. I am using version 2.7 of biber.

Biber 2.8 develpment version has updated dependency on Unicode::Collate 1.19 which includes Hebrew support. You can try the 2.8 DEV version from sourceforge - to use this, you will need biblatex dev version 3.8 also on Sourceforge.


minecraft:foil item reference page is not listed in TOC

minecraft:foil item component page is not listed in TOC.
https://learn.microsoft.com/en-us/minecraft/creator/reference/content/itemreference/examples/itemcomponents/minecraft_foil
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 167f0357-2973-795c-c167-f7eca2ee0f8f
Version Independent ID: 167f0357-2973-795c-c167-f7eca2ee0f8f
Content: Item Documentation - minecraft:foil
Content Source: creator/Reference/Content/ItemReference/Examples/ItemComponents/minecraft_foil.md
Product: gaming
GitHub Login: @mammerla
Microsoft Alias: v-jillheaden

Fixed! https://github.com/MicrosoftDocs/minecraft-creator-pr/pull/806


Scope and time scheduling of the meeting for overview of scope of research projects - 2024#1 (issue #958 related)

The goal is to schedule the meeting in Teams calendar on 2024.10.10 at 20:15.

Meeting has been scheduled.


test: add bucket lifecycle testcase

I have locally passed this testcase.
[root@c49 s3-tests]# S3TEST_CONF=test.conf ./virtualenv/bin/nosetests -vs s3tests.functional.test_s3:test_bucket_lifecycle
Deleting buckets on alt
Cleaning buckets from connection alt prefix 'yan-rl78cczcrsce9mtacgbeufxo4-'.
Deleting buckets on main
Cleaning buckets from connection main prefix 'yan-rl78cczcrsce9mtacgbeufxo4-'.
Done with cleanup of test buckets.
s3tests.functional.test_s3.test_bucket_lifecycle ... ok
Deleting buckets on alt
Cleaning buckets from connection alt prefix 'yan-rl78cczcrsce9mtacgbeufxo4-'.
Deleting buckets on main
Cleaning buckets from connection main prefix 'yan-rl78cczcrsce9mtacgbeufxo4-'.
prefix= yan-rl78cczcrsce9mtacgbeufxo4-
Cleaning bucket <Bucket: yan-rl78cczcrsce9mtacgbeufxo4-1>
Done with cleanup of test buckets.

----------------------------------------------------------------------
Ran 1 test in 0.912s

OK
[root@c49 s3-tests]#

Signed-off-by: Yan Jun<EMAIL_ADDRESS>

@yehudasa Could you take a look at this?

@Yan-waller is this test still relevant? (we addedd a few lifecycle tests), if so can you rebase it?

@yehudasa since you have added lifecycle tests, I closed this one.


replace print statements with logging

Closes #12
@JanOdijk, could you verify this doesn't break your local workflow (or MWE-annotator)

Dag Ben,
Ik kijk hier volgende week naar als ik terug ben van LREC-COLING in Turijn.
Jan
From: Ben Bonfil @.>
Sent: donderdag 23 mei 2024 09:54
To: UUDigitalHumanitieslab/mwe-query @.>
Cc: Odijk, J.E.J.M. (Jan) @.>; Review requested @.>
Subject: Re: [UUDigitalHumanitieslab/mwe-query] replace print statements with logging (PR #13)
CAUTION: This email originated from outside of Utrecht University. Do not click links or open attachments unless you recognize the sender and know the content is safe.
@bbonfhttps://github.com/bbonf requested your review on: #13https://github.com/UUDigitalHumanitieslab/mwe-query/pull/13 replace print statements with logging.
—
Reply to this email directly, view it on GitHubhttps://github.com/UUDigitalHumanitieslab/mwe-query/pull/13#event-12905048734, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF3FJGSWQPYZJDHOM2JCCP3ZDWOABAVCNFSM6AAAAABIFC3PWOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHEYDKMBUHA3TGNA.
You are receiving this because your review was requested.Message ID<EMAIL_ADDRESS>


Added method to reshape dims of network.

This PR implements two methods, match_in_dims and match_out_dims that allows splitting edges to match a target dimension. This is primarily meant to be used with ptrace as we need to reshape the outer nodes such that the network has the appropriate dimensions.
Note that _match_edges_by_split is a very similar function but with different scope. The new _match_dimensions is meant to be used with unary operators that required a change in the dimension (ptrace) whereas _match_edges_by_split is meant to be used with binary operators (matmul). This is because for  _match_dimensions (unary version) we do not allow merging two edge dimension. However, such operation can also be achieved in _match_edges_by_split (binary version) by just splitting the edges as two list of edges is available.
The reason to not allow here in _match_dimensions to merge edges is because this is a relatively complex operation that will require contracting nodes. I am assuming that it is best to do the contraction of the full network in a single step such that an optimized path is used.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

38 of 39 (97.44%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 91.643%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/qutip_tensornetwork/core/data/network.py
38
39
97.44%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
318

Relevant Lines:
347

💛  - Coveralls

Thank you for the review Simon!

I changed so that match_in_dims and match_out_dims return a new Network instead of performing an inplace modification. @hodgestar, could you take a look at the latest changes?


  Make the code procedural.
Modification:
Remove looping structure and grow the array by 1.5x instead of 2x for a less more conservative growth.  Also, when doubling the size, it can quickly get larger than the largest signed Integer size.

honestly I am not sure if this is any better as the ByteBuffer[] is shared in a ThreadLocal anyway.
Let the others have a look as well.

@netty-bot  test this please

test failed during other PR.

Well, thank you for your interest.  I appreciate how active you all are in reviewing PRs. :)
So, this submission is inspired by OpenJDK ArrayList.

With the current implementation, if the size is 2GB + 1 Byte, allocation will fail because doubling the size will overflow the array size
Less code
More conservative array growth, efficient use of space.  Instead of doubling the array size, this solution multiplies by 1.5x and even then, if that one action isn't big enough, it only allocates enough space to meet the requested size.

http://hg.openjdk.java.net/jdk8/jdk8/jdk/file/tip/src/share/classes/java/util/ArrayList.java#l237
Definitely a minor suggestion, but I noticed there are a few different places in Netty that has an expanding buffer, and as I recall, there are some difference in implementation.  I wanted to gauge interest with this one example before making all dynamic buffer allocation consistent.

OK.  Can close PR.  Thanks.


bug: Controller being called twice, and nav-bar disappears

Type: bug
Platform: all
When caching is disabled controllers are being initialized twice.
see also this issue: https://github.com/driftyco/ionic/issues/3620#issuecomment-106057528
and this one: http://forum.ionicframework.com/t/hidden-nav-bar-using-state-go-with-disabled-caching-beta-14/17686
pluckr:
http://plnkr.co/edit/X7zM82OvlUQvLjZq5au8?p=preview1
When clicking the refresh button, the controller is executed twice, and the nav-bar disappears

Greetings @manmetplok!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

View caching will be overhauled in 2 but looking at 1.3 for this one

@manmetplok I've forked your Plunk and provided a workaround for you http://plnkr.co/edit/mrv7KCiM4AzkuG6i9ElK?p=preview
It's located in the controller.js file

Hi, I too am having this issue where nav-bar disappears and controller is being called twise when caching is disabled for the application. I saw this was marked for  milestone 1.3 and upgraded my ionic at v1.3 but this is still reproducible.
Can you please confirm if this is supposed to be fixed in v1.3?

@ksgupta Not v1.3

Hello all! Thanks for opening an issue with us! I am going to close this one for now as their seems to be a workaround that @0x1ad2 posted. Thanks for using Ionic!

So there is no plan to fix this for version 1? We need to just manually call it in the afterEnter event?

I realised that this workaround has issues when you want to override the ion-nav-bar with custom ion-nav-buttons for some pages!
The ion-nav-bar defined in the index.html is used and the overrides in the page's ion-view do not modify the nav bar. Is there any workaround or more robust solution to this problem?
I know your target and focus is v2 but leaving issues open for the old but still wide-used version is not good.
Thanks

So, like @lambrospetrou said, the workaround doesn't work every time. It would be great if this issue gets re-opened.

Ok guys, I've worked on a pure Javascript solution, for my situation, and here it is. Hope it can help someone.
My situation: like what @lambrospetrou said, the workaround wasn't working if you had the ion-nav-bar overrided in some template, and I didn't wanted to use jQuery.
$scope.$on('$ionicView.enter', function(e) {
    $timeout(function() {
      showHeader();
    }, 1000);

    function showHeader() {
      // Having the nav-bar in your template, set an ID to it.
      var header = document.getElementById("header_id");
      if (header.classList) {
        if (header.classList.contains('hide')) {
          header.classList.remove('hide');
        }
      }
    }

  });


v2: vagrant destroy not implemented?

Issue Type

Bug report

Molecule and Ansible details
ansible --version
<IP_ADDRESS>
molecule --version
<IP_ADDRESS>rc6

Molecule installation method: pip
Ansible installation method: OS package

Desired Behaviour
In the end of molecule test in the destroy.yml phase, molecule actually calls vagrant destroy -f
Actual Behaviour (Bug report only)
The next test run is against an already provisioned machine instead of a fresh one.
I'm not even sure how Molecule v2 handles teardown, it seems to set a state: destroy that apparently is just silently ignored.

This might have been reported too soon, it seems like I inadvertently deployed to my locally running lxd instance rather than the one in the VM and thus was seeing already provisioned containers.

Ok, verified, it actually works, my bad. :-(


Import initialization methods from the C++ API

We should import initialization methods from the C++ API.
There are functional features only available this way.
https://github.com/antimatter15/tesseract-rs/issues/28
Whilst we mostly use the C API, we currently import a couple constants from the C++ API. See this PR for the code that currently imports from a C++ header:
https://github.com/antimatter15/tesseract-rs/pull/9

I haven't been able to find a rust-bindgen example for a C++ method that takes a std::vector<std::string>. It sounds like it might be quite a problem because of inline methods on those std types. I found a recommendation in this rust-bindgen issue https://github.com/rust-lang/rust-bindgen/issues/738#issuecomment-664011855 to use the cxx crate instead, but that seems like a pretty big addition/change to tesseract-sys


Allow users to explicitly define environment in conda.yaml / environment.yaml file

Let users manage their production-time dependencies explicitly through an optional conda.yaml file.
If the file is not present, it would then fall back into the current behaviour.

I believe this is what I was talking about in #219, I assume this functionality now exists? Would be great to document so we can close #219 too.


Playground: new screen - build Repositories > Code using new layout approach

First pass:

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


Remove dropRight and takeRight from Iterator

Fixes #131.
Note that I kept the grouped and sliding methods of Iterator even though they also use buffers. Since these methods do exist in 2.12 we have to discuss before we remove them…

Looksreasonable.  I made some comments but didn't do a formal review.


Unable to create Azure solution in VS Code in MacOS

Repro steps:

Create new Azure Function
C#

Action: azureFunctions.createNewProject
Error type: Error
Error Message: Could not find template with language "C#", runtime "~2", and id "Azure.Function.CSharp.IotHubTrigger.2.x".
Version: 0.17.1
OS: darwin
Product: Visual Studio Code
Product Version: 1.36.0
Language: en

Call Stack
Function.<anonymous> extension.bundle.js:164:253583
o extension.bundle.js:164:252718

@avinashsharma5

can you check the Azure Functions output window and copy the logs from there

Do you have the dotnet cli installed and on your path?
Does it work if you do a different template, like http trigger?


feat: send screen above the fold

Type of change (Remove other not matching type)

New feature (non-breaking change which adds functionality)

Describe the changes you have made in this PR -

Removed the card style from the send screen as it uses a lot of space.
Smaller avatar.
Visually hide scrollbar as in most extensions.

Screenshots of the changes (If any) -
Before:

After:

great!
need to check which other prompts had it. (e.g. my make invoice prompt #776)

looks better without the scrollbar and the smaller icon. I found the colors before (light grey and dark grey) made the text a bit more legible.

@MoritzKa how about this?
I wanted the the titles to match the color of our input labels. (same as amount & comment)

Yes. Looks good!


3 new terms for anastomosis during sprouting angiogenesis, fixes #13690

OWLAPI parse errors are incredibly hard to read...
We'll look into making these more targeted but in the interim the protocol is to find the part of the report that says OBOFormatOWLAPIParser
like here:
--------------------------------------------------------------------------------
Parser: org.semanticweb.owlapi.oboformat.OBOFormatOWLAPIParser@2c3dec30
    Stack trace:
LINENO: 475107 - Unexpected tag id in term frame.
LINE: id: GO:0120079        org.semanticweb.owlapi.oboformat.OBOFormatOWLAPIParser.parse(OBOFormatOWLAPIParser.java:60)

ok so we found the message but I have no idea why it's complaining since 'id' is obviously valid

Ah, something odd has gone wrong with how the new terms were inserted into the file, creating stanzas without a [Term] header. I have never seen this before in many many git commits, very odd..... will look further

I was working on this in a different branch that I started last week, but when the pull request for that one indicated conflicts. Looks like I made a mistake when I resolved the conflicts. Thanks for finding that.

OK, that's good to know, @kltm were trying to figure the different ways this could happen with a normal commit!


Separation of concerns

I've been looking at some separation as I feel the App class is a bit of a mixup of several methods that do not have anything to do with each other.
App now acts as:

container
part of the router (adding routes)
is the core/kernel middleware layer
contains the application runner method
handles outputting the final response
contains several 'helper' method like halt() and subRequest()

I think it would do good to separate some of this functionality.
There is a WIP pull request that handles the contains part. To separate some more functions I tried some things.
As spoken of in another pull request separating the outputting from the app:
https://github.com/slimphp/Slim/compare/develop...JoeBengalen:feature-separate-core?expand=1
Also I tried separated the core/kernel middleware layer:
https://github.com/slimphp/Slim/compare/develop...JoeBengalen:feature-separate-core?expand=1
Note this compare also contains the changes from the above one.
I know passing the container as a dependency may not be ideal, but  you get the idea.
Let me know how you think about separating some functionality from the App class.
@codeguy

Maybe we can split this concerns into trait like for the middleware stack and the callable resolver.

I agree with this idea in principle, and would like to see this as a long term goal. I think this should be tagged for a 4.0 release as it is a major structural change and we really need to push out 3.0 sooner than later.

Disagree.

@r3wt Mind explaining why?


NFCDemo crashed on device but ok with Simulator

I integrated the library into my application using Carthage, and I want to use NFCDemo in my app. It runs fine on the simulator, but it throws an error on a real device. I have tried to set ASN1Kit in Link Binary with Library as Optional,but it still didn't work.
Error:
dyld[92622]: Symbol not found: (_$s7ASN1Kit16ObjectIdentifierV2eeoiySbAC_ACtFZ)   Referenced from: '/private/var/containers/Bundle/Application/C0ACE973-2220-4174-81F4-D715537E1332/Apotheke.app/Frameworks/HealthCardAccess.framework/HealthCardAccess'   Expected in: '/private/var/containers/Bundle/Application/C0ACE973-2220-4174-81F4-D715537E1332/Apotheke.app/Frameworks/ASN1Kit.framework/ASN1Kit'

It might be a caching problem of some sort.
Can you provide a (minimal) project on GitHub that reproduces the bug?
In any case we plan to provide a Package.swift file for this project within this or next week. That will eliminate the need for Carthage as well

It might be a caching problem of some sort. Can you provide a (minimal) project on GitHub that reproduces the bug?
In any case we plan to provide a Package.swift file for this project within this or next week. That will eliminate the need for Carthage as well

Thanks for reply. I just created a hello world project and by using Carthage with "github "gematik/ref-openHealthCardKit" ~> 5.0" to drag xcframeworks into this project. Here is the link https://github.com/caijunjieNick/OpenHealthCardFrameworkTest.git of project for clone. It still crashed on real device.

Thank you.
We could reproduce the problem but weren't able to figure out, what's causing the bug exactly. We include the framework into our own app (see https://github.com/gematik/E-Rezept-App-iOS/blob/6c7a7b752f96715c96f985c1e3a1384ca82cb494/Cartfile), and it has been working for us thus far.
As announced, we will add a Package.swift file to this project that should resolve the problem. (We were not able to reproduce the bug when we included the framework via SPM.)
We opt publish the Package.swift within this week.

Hello, it works finally. I really appreciate it:)

Good to hear that.

Hello, may i ask, wheter i can get the personal information from the eGK with nfc by using the library?


Implement API endpoints for users to update their profile fields, such as name, bio, location, etc.

I need to add a new field to the user schema to reflect the professional status of users. Then I need to add routes to allow users to update specific fields for their user data

This issue was successfully implemented and tested with #18


Couldn't open file: data_linemod_gt/02/all.txt

Sorry for bothering you. I run the file gt_single_object.py ,then create the folder data_linemod_gt, I run train_single.sh, then  I got the following information:

Total BFLOPS 65.290
Loading weights from darknet53.conv.74...
seen 64
Done!
Learning Rate: 0.001, Momentum: 0.9, Decay: 0.0005
Couldn't open file: data_linemod_gt/02/all.txt

What should I do?

the 02 sequence is actually a multiple object sequence, probably it's due to this issue

Sorry for bothering you. I run the file gt_single_object.py ,then create the folder data_linemod_gt, I run train_single.sh, then I got the following information:

Total BFLOPS 65.290
Loading weights from darknet53.conv.74...
seen 64
Done!
Learning Rate: 0.001, Momentum: 0.9, Decay: 0.0005
Couldn't open file: data_linemod_gt/02/all.txt

What should I do?

Hello,
Did run this code successfully?
I load sixd data return a error when I get 'obj_bb' from 'gt.yml'.
If you solved this problem, can you give me some suggesions?


DCPower Example does not work with single channel SMUs

Bug Report
The default pin map for the DCPower example has two pins mapped to two channels. If I run the example with only Pin1 in the pin control, I get the following error:
nidcpower.errors.DriverError: -1074135008: IVI: Unknown channel or repeated capability name.

Channel Name: 1

After removing the second pin, the example still fails to execute with the following error:
2023-01-20 14:22:30,486 ERROR: Exception calling application: StatusCode.UNKNOWN: Stream removed
Traceback (most recent call last):
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\grpc\_server.py", line 444, in _call_behavior
    response_or_iterator = behavior(argument, context)
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\ni_measurementlink_service\_internal\grpc_servicer.py", line 195, in Measure
    output_value = self.measure_function(**mapping_by_variable_name)
  File "measurement.py", line 88, in measure
    session = stack.enter_context(_create_nidcpower_session(session_info))
  File "measurement.py", line 182, in _create_nidcpower_session
    return nidcpower.Session(resource_name=session_info.resource_name, **session_kwargs)
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\nidcpower\session.py", line 6888, in __init__
    self._interpreter.set_session_handle(self._fancy_initialize(resource_name, channels, reset, options, independent_channels))
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\nidcpower\session.py", line 7196, in _fancy_initialize
    return self._initialize_with_independent_channels(resource_string, reset, option_string)
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\nidcpower\session.py", line 7623, in _initialize_with_independent_channels
    vi = self._interpreter.initialize_with_independent_channels(resource_name, reset, option_string)
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\nidcpower\_grpc_stub_interpreter.py", line 310, in initialize_with_independent_channels
    response = self._invoke(
  File "C:\Users\Administrator\Desktop\measurementlink-python-main (1)\measurementlink-python-main\examples\nidcpower_source_dc_voltage\.venv\lib\site-packages\nidcpower\_grpc_stub_interpreter.py", line 64, in _invoke
    raise errors.RpcError(grpc_error, error_message) from None
nidcpower.errors.RpcError: StatusCode.UNKNOWN: Stream removed

In the instruments section of the pin map editor, changing Number of Channels to 1 results in the same error message.
I tested this using a PXIe-4139
Repro or Code Sample
Run example DCPower measurement using a 1 channel SMU.
Expected Behavior
Example should run correctly.
Current Behavior
Error message when running the example. Example still errors after removing all but 1 Pin/Channel.
Possible Solution
N/A
Context
Testing examples with real hardware.
Your Environment
Windows 10 device with physical single channel SMU (PXIe-4139)
Dev build of MeasurementLink
Dev build of measurementlink-python
AB#2275162

Example - it's OK if it doesn't work in all cases.

I think we should understand the root cause before making a decision whether to close this. Based on @DelpireNI 's screenshots, I would expect the measurement to use channel 0, not channel 1.


Introduce Create Conversation Route

Is your feature request related to a problem? Please describe.
As an Airy user having previously imported contacts (https://github.com/airyhq/airy/issues/2293),
I want to be able to create a conversation for source types that support this,
so that I can send a message to the relevant contact to initiate the conversation
Describe the solution you'd like

We could add a new POST /conversations.create route e.g. with a payload such as:

{
  contact_id: "UUID",
  channel_id: "UUID"
}

returning HTTP 201 and the conversation_id in case the source type of the channel_id supports initiation of a conversation by somebody other than a contact
or returning HTTP 400 in case the source type of the channel_id doesn't support this

Then we could simply send a message to the relevant conversation using the existing route /messages.send

Describe alternatives you've considered
We could experience "empty" conversations without any messages following the approach as suggested which generally shouldn't be a problem, but we need to make sure all clients expect and are able to handle this by rendering empty states, etc..
As an alternative we could change the POST /messages.send route in a way it would take {contact_id: UUID, channel_id: UUID} as an alternative to conversation_id.
Additional context
We could extend the API already in a way that conversations with more than one contact/participants can be created as we expect future sources to go beyond the current restriction inherited from the sources we currently support.
An alternative version would be:
POST /conversations.create
with payload:
{
  channel_id: "UUID",
  contact_ids: [ "UUID1", "UUID2" ]
}

It's important to note that conversations are a materialized view of messages in Airy. Therefore you cannot create a conversation without sending the first message. Respecting this rule has the added benefit of preventing the creation of orphaned conversations without any messages (an issue in the previous platform).
Furthermore, we should not make this API depend on contacts, because they are not part of the messaging system and because each contact can have multiple messaging sources, which makes disambiguation in this endpoint tricky.
Therefore I would propose that we allow users to either provide the conversationId or the sourceConversationId when sending a message.
POST /messages.send
{
  "source_conversation_id": "+49 1234567", // or an email address
  "channel_id": "<airy uuid>",
  "message": {} // regular message body
}

I can relate to the restriction to materialize conversations from messages, so thank you @chrismatix for explaining this.
I think the source_conversation_id property can be ambiguous as we usually can't influence the way these are created on the level of the source, so having this property for a phone number, an email address, etc. looks a bit strange to me.
The suggested design would also not fulfill the part if the requirement "having previously imported contacts" but actually extend it in a way to support use cases where no contact information has previously been imported, which might be a valid case.
Making it not dependent on contacts is actually a fair point, but that doesn't mean that previously imported contacts can't be referenced still.
What do you think of sth like this?
POST /messages.send
  "recipient": { "phone": "+14151234567" }, // or "email"<EMAIL_ADDRESS>  "contact_id": "<airy uuid>", // optional value
  "channel_id": "<airy uuid>",
  "message": {} // regular message body

We should be able to get the source type and sender from channel_id, so this design even would work for Facebook e.g. recipient: {id: "PSID"} given that even getting the PSID is equivalent to a receiving a conversation_id from the source. So I could send messages without looking up the Airy conversation_id as long as I know a unique identifier on the level of the source and the channel_id.

I think the source_conversation_id property can be ambiguous as we usually can't influence the way these are created on the level of the source, so having this property for a phone number, an email address, etc. looks a bit strange to me because the actual conversation_id on the level of the source can still be  instead of +49151123456789.

I don't quite understand this point. The ambiguity is resolved by the channel_id which defines the source and thus the interpretation of the source_conversation_id. Those SCIds are by our own definition represented by a single string, so there is no need to nest them in a recipient field.

The suggested design would also not fulfill the part of the requirement "having previously imported contacts" but actually extend it in a way to support use cases where no contact information has previously been imported, which might be a valid case.

It's true, but this case is much stronger since it fits the messaging sources without the need for the concept of sources.

Making it not dependent on contacts is actually a fair point, but that doesn't mean that previously imported contacts can't be referenced still.

Agreed and I am not unfavourable to the suggestion. However in an effort to strictly separate the concepts I would propose adding an endpoint to the planned contacts app #2293
POST /contacts.sendMessage
{
  "contact_id": "<airy uuid>",
  "channel_id": "<airy uuid>",
  "message": {} // regular message body
}

This route looks up the contact's source_conversation_id given the messaging source of the channel and in turn calls the aforeproposed /messages.send endpoint. This way we provide both functionalities and cleanly separate contacts and messaging.
If you are ok with this design I would create a separate /contacts.sendMessage ticket 🙏

I just meant we shouldn't call a property source_conversation_id when the actual conversation_id on the level of the source could be something totally different (e.g. a UUID and not a phone number), as it might be confusing to people. E.g. when you start a conversation on Twilio by sending a messages you just use their Send Messages endpoint with the payload {"to": "+491234567", "from": "+123456789", "body": "some-text" } and then they assign as conversation ID. That seems to be case for most sources that support the relevant use case so we can't possible know what the source_conversation_id will be before sending the message and creating the conversation on the level of the sources. So I would use a name that is more generic.
E.g. in supporting the use case you describe, we could even do sth. like:
POST /messages.send
{
  "to": "+14151234567", // or "to"<EMAIL_ADDRESS>  "contact_id": "<airy uuid>", // optional value
  "channel_id": "<airy uuid>",
  "message": {} // regular message body
}

I would actually prefer that design which is rather close to what you suggested before to introducing another route with the payload as referenced above.
We still lack consistency in naming routes at the moment (we mix camelCase, snake case and names separated by dots). Also having two routes that send messages that are called POST /messages.send for conversations and POST /contacts.sendMessage seems to inconsistent to me.
In case you wanted to separate both use cases which adds complexity for the developer, a consistent way would be to do sth. like POST /conversations.messages.send and POST /contacts.messages.send.

I just meant we shouldn't call a property source_conversation_id when the actual conversation_id on the level of the source could be something totally different (e.g. a UUID and not a phone number), as it might be confusing to people. E.g. when you start a conversation on Twilio by sending a messages you just use their Send Messages endpoint with the payload {"to": "+491234567", "from": "+123456789", "body": "some-text" } and then they assign as conversation ID. That seems to be case for most sources that support the relevant use case so we can't possible know what the source_conversation_id will be before sending the message and creating the conversation on the level of the sources. So I would use a name that is more generic.

@steffh Thanks for the explanation! In the Twilio case, the source_conversation_id is the to phone number. For every source that supports starting a new conversation without previous messages from the contact, there has to be some sort of identifier (like a phone number or an email) that we can use to send messages.
Following our offline conversation however I can understand why source_conversation_id would looks surprising to users. Hence I propose source_recipient_id. (💡 By convention we prefix all ids not managed by Airy, but by sources with source_)
contacts.sendMessage
We realized that while this endpoint seems convenient, a contact could have multiple identities within a given source (i.e. Facebook page scoped user ids or phone numbers). Therefore even a <channel id, contact id> tuple is insufficiently unambiguous to receive messages.
Conclusion
We favor this extension of the message send endpoint.
POST /messages.send
{
  "source_recipient_id": "+49 1234567", // or an email address
  "channel_id": "<airy uuid>",
  "message": {} // regular message body
}

@chrismatix Thank you & I couldn't agree more.

@steffh I will edit the ticket body now to reflect that.


Pick up latest STS w/ DacFX bump (#26056)

Pick up latest STS w/ DacFX bump (#26056)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 41.712%

Totals

Change from base Build<PHONE_NUMBER>3:
0.0%

Covered Lines:
30720

Relevant Lines:
68916

💛  - Coveralls


How to Force SSL on a domain: Wordpress

Steps Taken So far
From https://cloud.google.com/appengine/docs/flexible/php/securing-custom-domains-with-ssl

I have a custom domain.
i have verified a managed certificate has been provisioned
I have upgraded to managed SSL certificates automatically. Thanks Google
I have created a htaccess that Forces SSL

My Problem

After running gcloud app deploy my site goes to http instead of https.

Answered in https://github.com/GoogleCloudPlatform/php-docs-samples/issues/489


Update InvariantValue type so that the value field cannot be reassigned once set.

https://trello.com/c/HED9eq2A/82-consider-immutability-of-the-invariant-types

Thanks, this looks good to me :+1:


Codegen overloaded function declarations

TODO:

[ ] handle declare fn
[ ] refactor TaggedTemplateLiteral so that we can update the Throws field

Codecov Report
Attention: Patch coverage is 90.34091% with 17 lines in your changes missing coverage. Please review.

Project coverage is 83.99%. Comparing base (2dba351) to head (dc6f1b7).

Files
Patch %
Lines

src/Escalier.Codegen/Codegen.fs
86.40%
17 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #352      +/-   ##
==========================================
+ Coverage   83.75%   83.99%   +0.23%
==========================================
  Files          22       22
  Lines       12156    12313     +157
  Branches     1733     1755      +22
==========================================
+ Hits        10181    10342     +161
+ Misses       1975     1971       -4

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


Printable directory

All on one page (no pagination)
Need to deal with bootstrap print quirkiness: http://stackoverflow.com/questions/20243767/printing-page-with-bootstrap-3/20244712#20244712
Respect filters
Change 'create' button to just a + and put invite as an option under it
Add new print button

UPDATE

Add album/table view to lens
Table shows all on one page always
Clicking print in album mode loads everyone in hidden div with table tags (these print better it seems)
Table view should print fine without any funkiness

Originally created by Tom Smyth at 2016-11-06 15:26, migrated from redmine-#5371

Priority changed from 3 to 4

By Tom Smyth at 2016-11-06 20:53

Priority changed from 4 to 2

By Tom Smyth at 2017-01-19 18:20

Priority changed from 2 to 1

By Tom Smyth at 2017-05-23 12:06

Description updated (diff)

By Tom Smyth at 2017-06-01 12:55

Estimate set to 2.00 h

By Tom Smyth at 2017-06-01 12:58

Description updated (diff)

Estimate changed from 2.00 h to 5.00 h

By Tom Smyth at 2017-06-02 02:04

Status changed from New to Closed

By Tom Smyth at 2017-06-06 13:54

Version set to 4.1

By Tom Smyth at 2017-06-07 02:20


Extremely large large models cause BabylonReactNative demo app to crash.

Very large and complicated meshes will not load in the BabylonReactNative demo app.
Steps to repro:
Try to load:
https://raw.githubusercontent.com/RonnieAshlock/TestAssets/master/GLB/CherryPicker.glb
in a BabylonReactNative demo app.

I could not repro the crash. Can you please push your branch with the changes to load the .glb so I can test it?

Hi, Cedric. Ryan helped me push up my branch. Here is the URL:
https://github.com/RonnieAshlock/BabylonReactNative/tree/ModelTests
You should be able to just launch BabylonReactNative and repro the issue as I've set up SampleScene.ts to load the GLB that causes the app to crash.
Thanks for taking a look at this.

Hi Ronnie, I've tested your branch on iOS(device+sim) and it worked fine. I got the mesh displayed properly in AR mode as well.

Ronnie is right! It's running out of memory.
The react http download tries to allocate 250Mb

And the allocation profile

I think you can repro the issue just by trying to dowload the file without BabylonNative.
I don't know why it makes a so big allocation for a 80Mb glb.

@ryantrem @Alex-MSFT Can one of you take a look? We think this is React Native specific.

React Native's XMLHttpRequest takes the byte buffer, converts it to a base64 string, passes that string (by copy) to JavaScript, and finally it is converted in JavaScript back to a byte buffer. This results in too much memory pressure and leads to the crash. This has been addressed by allowing Babylon.js to use Babylon Native's XMLHttpRequest implementation instead of React Native's. See these PRs for details:

https://github.com/BabylonJS/Babylon.js/pull/9148
https://github.com/BabylonJS/BabylonNative/pull/442
https://github.com/BabylonJS/BabylonReactNative/pull/94


Inconsistent behavior with Emulate Prepares

Repro script:
http://pastebin.com/CWHAi74z
Current output:
SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
NO EXCEPTION?
Expected output:
SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
SQLSTATE[HY093]: Invalid parameter number: parameter was not defined

This is the right script:
http://pastebin.com/fDgrsy7d

@david-garcia-garcia: We believe we closed this issue with our latest release https://github.com/Azure/msphpsql/releases. Can you help us validate?

This is not fixed. The script still outputs:

SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
NO EXCEPTION?

When it should output:

SQLSTATE[HY093]: Invalid parameter number: parameter was not defined
SQLSTATE[HY093]: Invalid parameter number: parameter was not defined


Swap $timeout for setTimeout or nothing at all

I think this could be a significant perf gain if we could get rid of running runExpressions in a $timeout... https://github.com/formly-js/angular-formly/blob/master/src/directives/formly-field.js#L61

This may require a breaking change...

Just for context... runExpressions is a function that's added to all of your fields which is in charge of running all of your expressionProperties. You can invoke it yourself, but most of the time it's just invoked by the parent formlyForm (here).
Because expressionProperties require the $viewValue and the $modelValue, I remember having trouble with getting those values at the time I was developing it. Wrapping the function in a $timeout seemed to fix the problem. However, things have changed and it may not be necessary to do this anymore. I'm crossing my fingers.
If it is still necessary to execute expressionProperties on the "next tick" of the event loop, the next best thing I could do is replace $timeout with setTimeout. Right now, every one of your expressionProperties causes a $digest to fire because it's using $timeout. This is a major performance drag. However, if I switch to setTimeout then I fear that we may have a situation where things don't get updated as you would expect and you'd have to invoke $digest yourself (I don't want to force you to do this). This is why I would release this in a beta and ask that a lot of people please upgrade to try it out in your apps.
Feedback is appreciated.

May want to check out evalAsync/applyAsync

Oh! Yeah! Why didn't I think of that earlier. Thanks for the tip. That's probably exactly what I need.


Misc typo

Validates whether a text can be encoded. The validation is based at a regular expression accoring to
should be
Validates whether a text can be encoded. The validation is based at a regular expression according to

Can you share more context, repro steps? Where do you get this message?

See: https://github.com/microsoft/ALAppExtensions/blob/a598cfc8f86aa68891983abe5225e8030bb340f8/Modules/System/Barcode/src/Barcode Font Provider/BarcodeFontEncoder.Interface.al#L21

Thanks :-) We'll get that fixed.

These are both from a598cfc8f8
ALAppExtensions/Modules/System/Barcode/src/Barcode Font Provider/BarcodeFontEncoder.Interface.al
https://github.com/microsoft/ALAppExtensions/blob/a598cfc8f86aa68891983abe5225e8030bb340f8/Modules/System/Barcode/src/Barcode Font Provider/BarcodeFontEncoder.Interface.al#L21

Comments for
procedure IsValidInput(InputText: Text; var BarcodeEncodeSettings: Record "Barcode Encode Settings"): Boolean;

Actually,
Validates whether a text can be encoded. The validation is based at a regular expression according to
should be
Validates whether a text can be encoded. The validation is based on a regular expression according to

Just above are comments for
procedure EncodeFont(InputText: Text; var BarcodeEncodeSettings: Record "Barcode Encode Settings"): Text;

Encodes a input text to a barcode font.
should be
Encodes an input text to a barcode font.
Thanks

As I was checking out #12922, I noticed another misc typo in the
documentation for the previous method. I added a comment there pertaining
to the typo.
Thanks
On Tue, Jun 22, 2021 at 4:45 AM Jesper Schulz-Wedde <
@.***> wrote:

Closed #12922 https://github.com/microsoft/ALAppExtensions/issues/12922.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/ALAppExtensions/issues/12922#event-4921277014,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHO3XBPPZN3RA3U5HI56QALTUBEQ5ANCNFSM4664VGVQ
.

I noticed and fixed the at -> on, when I was correcting the typo. That one is already being processed in our build system :-) The a -> an I didn't spot :-S
We should get a spell checker on our code comments. Currently, you need to enable it in VSCode yourself. Clearly, not all developers are doing so...

How do you enable Spell Checking for document comments in VS Code?
Thanks
On Tue, Jun 22, 2021 at 8:27 AM Jesper Schulz-Wedde <
@.***> wrote:

I noticed and fixed the at -> on, when I was correcting the typo. That one
is already being processed in our build system :-) The a -> an I didn't
spot :-S
We should get a spell checker on our code comments. Currently, you need to
enable it in VSCode yourself. Clearly, not all developers are doing so...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/ALAppExtensions/issues/12922#issuecomment-865940324,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHO3XBN6JIRHVOZ3CLABQ4LTUB6UVANCNFSM4664VGVQ
.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


How to hide port usage

Hello i am a noob please can you explain how can i use Knock to hide port usage, thank you

Hey, I'm not sure what you are asking. If you don't understand the port knocking in general, just google it.

for example, I am using ssh on 22 port, so can i hide that 22 port so if hacker will scan me she or he will not  see that port

Yeah, you can use port knocking for that. Google it.

i know but every tutorial is on linux i need it on windows can you help?

Sorry, I can't. I'm not a Windows user so I wouldn't be able to, even if I wanted to put my time in it.

ok thanks


Can see messages in subscription of bus service but can't retrieve

I can see there are a number of messages in a subscription both in Azure portal and when using message_count method of that subscription. But I can't retrieve any of them with receive_subscription_message. Those messages are supposed to be unlock already. Does anyone know what could be the reason for this behavior?
Also, I need to delete those messages. Is the only way with Python API  to use receive_subscription_message? Does Message time to live parameter in Azure portal help with this?

I'm unfortunately not a specialist in SB, but it seems to me the last time I saw that, was a DLQ:
https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-dead-letter-queues
You can try SBE to figure that out:
https://github.com/paolosalvatori/ServiceBusExplorer

Thanks for suggesting SBE. I checked and they are actually Dead Letter Queue messages. I wonder if you have plan to implement methods to count active and DLQ messages separately?

DLQ is a regular queue, it's just that the name is a system one. There is a staticmethod format_dead_letter_queue_name to build that name for you. You can then use a DLQ with exactly the same method.
https://github.com/Azure/azure-sdk-for-python/blob/master/azure-servicebus/azure/servicebus/servicebusservice.py#L157-L160

Ah nice. Thank you for your support. I close this (not really) issue here.


Duplicate guidance

When institutions have customised a funder template and added example answers, the funder guidance duplicates. This only seems to happen when example answers are present.

Deployed in v.0.3.2. Closing.


Add Async support with new API version 2021-05-01-preview

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

Can you please create the PR against the AzureArcForKubernetes:k8s-extension/public. This way, we release from this branch instead of directly from yours

Also, this way we have internal review before opening it up for review against upstream

k8s

Closing this, as I've created a PR for our Internal review


Fetch upstream

Codecov Report

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

@@            Coverage Diff            @@
##             master      #13   +/-   ##
=========================================
  Coverage          ?   26.13%
=========================================
  Files             ?       67
  Lines             ?     2238
  Branches          ?        0
=========================================
  Hits              ?      585
  Misses            ?     1653
  Partials          ?        0

Continue to review full report at Codecov.

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


Feature: Kotlin Support

Is your feature request related to a problem? Please describe.
Currently, Kotlin code is not supported.
Describe the solution you'd like
Support Kotlin code & KDoc.
Describe alternatives you've considered
Not adding it.
Additional context
Related: #99

Boost priority

Boost priority in our backlog through Polar.sh. Higher pledge, higher priority.
Minimum pledge by user/organization is $5, minimum amount for boost is $30.
View all issues with pledges.
We receive the funds once the issue is completed and confirmed by you.
Features with the insiders label are released to sponsors first, and tied to a funding goal.

Thanks for the request @solonovamax! You mention #99 in the additional context, could you elaborate on why this is related to the Java handler?

You mention #99 in the additional context, could you elaborate on why this is related to the Java handler?

kotlin & java are both closely related as they run on the jvm platform. (well, kotlin can also be multiplatform, however it is primarily used for the jvm platform)
kotlin's documentation tool, dokka, also generates documentation that is quite similar to what javadoc generates. you'd even ingest its packages/classes the same way as you would with javadoc.


Password confirmation

Add prop and validation function to PasswordField to compare it with another PasswordField for confirmation.

Better let app handle that and keep our collection of elements as simple as possible. Will make a demo to illustrate what can be done with handling custom form events.


An SubscriptionGroup that has been deleted leads to a pile of messages

Rocketmq Version
V4_5_0
topic  status
bash-4.4# ./mqadmin statsAll -n <IP_ADDRESS>:9876 <IP_ADDRESS>:9876  -t sms-engine-whevt-push-sms
OpenJDK 64-Bit Server VM warning: ignoring option PermSize=128m; support was removed in 8.0
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=128m; support was removed in 8.0
#Topic                            #Consumer Group                  #Accumulation      #InTPS     #OutTPS   #InMsg24Hour  #OutMsg24Hour
sms-engine-whevt-push-sms         sms-engine-whevt-push-sms-yunpian         1674        0.00        0.00             19              0
sms-engine-whevt-push-sms         sms-engine-whevt-push-sms                   0        0.00        0.00             19             19

I already deleted the   SubGroup  sms-engine-whevt-push-sms-yunpian

consumerStatus
bash-4.4# ./mqadmin consumerStatus -n <IP_ADDRESS>:9876 <IP_ADDRESS>:9876  -g sms-engine-whevt-push-sms-yunpian
OpenJDK 64-Bit Server VM warning: ignoring option PermSize=128m; support was removed in 8.0
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=128m; support was removed in 8.0
org.apache.rocketmq.tools.command.SubCommandException: ConsumerStatusSubCommand command failed
	at org.apache.rocketmq.tools.command.consumer.ConsumerStatusSubCommand.execute(ConsumerStatusSubCommand.java:134)
	at org.apache.rocketmq.tools.command.MQAdminStartup.main0(MQAdminStartup.java:135)
	at org.apache.rocketmq.tools.command.MQAdminStartup.main(MQAdminStartup.java:86)
Caused by: org.apache.rocketmq.client.exception.MQBrokerException: CODE: 206  DESC: the consumer group[sms-engine-whevt-push-sms-yunpian] not online
For more information, please visit the url, http://rocketmq.apache.org/docs/faq/
	at org.apache.rocketmq.client.impl.MQClientAPIImpl.getConsumerConnectionList(MQClientAPIImpl.java:1113)
	at org.apache.rocketmq.tools.admin.DefaultMQAdminExtImpl.examineConsumerConnectionInfo(DefaultMQAdminExtImpl.java:308)
	at org.apache.rocketmq.tools.admin.DefaultMQAdminExt.examineConsumerConnectionInfo(DefaultMQAdminExt.java:235)
	at org.apache.rocketmq.tools.command.consumer.ConsumerStatusSubCommand.execute(ConsumerStatusSubCommand.java:79)
	... 2 more

how can i delete  group sms-engine-whevt-push-sms-yunpian  form topic sms-engine-whevt-push-sms

The master's node maybe more than 1, and you only delete 1 in your cluster.

when i delete the  SubGroup , cluster repot that  i have delete  SubGroup from all the master node
bash-4.4# ./mqadmin deleteSubGroup -n <IP_ADDRESS>:9876 <IP_ADDRESS>:9876 -c broker -g sms-engine-whevt-push-sms
OpenJDK 64-Bit Server VM warning: ignoring option PermSize=128m; support was removed in 8.0
OpenJDK 64-Bit Server VM warning: ignoring option MaxPermSize=128m; support was removed in 8.0
delete subscription group [sms-engine-whevt-push-sms] from broker [<IP_ADDRESS>:10911] in cluster [broker] success.
delete subscription group [sms-engine-whevt-push-sms] from broker [<IP_ADDRESS>:10911] in cluster [broker] success.
delete subscription group [sms-engine-whevt-push-sms] from broker [<IP_ADDRESS>:10911] in cluster [broker] success.
delete topic [%RETRY%sms-engine-whevt-push-sms] from cluster [broker] success.
delete topic [%RETRY%sms-engine-whevt-push-sms] from NameServer success.
delete topic [%DLQ%sms-engine-whevt-push-sms] from cluster [broker] success.
delete topic [%DLQ%sms-engine-whevt-push-sms] from NameServer success.

@ccctask You can try to add -r true to deleteSubGroup command. In this way, broker will clean the offset table.


made a small grammar fix

[x] I have read and agreed to the RethinkDB Contributor License Agreement http://rethinkdb.com/community/cla/

Description
Just a small grammar fix, if I find anymore problems, I will fix those too :smile:

Thanks.


How to show a small portion of previous and next image?

Hi @zvonicek
Thanks for the awesome library. I have a query as to how to show a small portion of the previous and next image? Below is the example.

No. Not yet

Hi, this is unfortunately not possible yet. It was discussed before in #117. I'd love to see this in the project, but currently does not have resources to implement it. If you manage to solve it I'd be happy if you share the result.


Issue 15595 - Interface cast on collection may fail

This solves an issue where the GC collection cycle would take out a vtable pointer before another destructor tried to access it when calling destroy() on the object. This covered 99.9% of the remaining causes of segmentation faults in my server and was unpredictable.

Please add a test case.

So you want to make casting of an interface that belongs to an already destroyed object more forgiving?
Not sure about that, you might not generally access other objects in finalizers b/c finalization is deliberately unordered.

My issue was when the TLSClient object is destroyed by the GC in certain circumstances. It has say, a Unique!Hash member object which may be an implementation of SHA512 and allocated on the GC. So, once the GC collects TLSClient, the destructor of Unique!Hash will call destroy which fails in this cast. I still need to find a simple way to reproduce this..

You're not using that crappy Unique, are you?
In this case Unique is not the real owner of the object (b/c it's also allocated on the GC) and must not destroy it in a finalizer.
According to the docs Unique.release is your friend.

Yes, a customization of unique in memutils. I use it on manually managed allocations, but high-level objects are allowed to be managed in the interface. For obvious reasons, I need determinate lifetimes for these high-level objects when working with TLS.

Have you been able to resolve your problem?

It would require a rewrite that moves everything out of the GC. I think it would be simpler if the cast forgave null pointers

It would silence a bug in your code and hide a race condition once we parallelize finalization.

In that case, the .destroy() call would need to be thread-safe to be compatible with GC parallelized collection shouldn't it?

An alternative fix for me is to check in Unique.~this whether bool rt_isInGC() returns true before calling destroy.
Here is the implementation in lifetime.d :
bool g_in_gc;

extern (C) bool rt_isInGC() {
	return g_in_gc;
}
extern (C) void rt_finalizeFromGC(void* p, size_t size, uint attr)
{
	g_in_gc = true;
	scope(exit) g_in_gc = false;

This will be solved by using the gc_inFinalizers call from the GC SpinLock pull request


Replace gemfile for front-end translations with the underlying Jed library

As part of moving all the assets to Webpack, the gem "gettext_i18n_rails_js" has to be replaced with Jed library directly. A separated issue to deal with this will allow progressing on the ticket #587

move JS constants to hidden input elements on the page so that the regular GetText can be used for translation. Then update JS to pull in the values when needed.

picking this one up @jollopre as we want to get translation files out to a few people. Will add the constants to the views/layouts/application.html.erb file and tie them in via the lib/assets/constants.js

can't test in UI. closing issue.


Moldy Bread doesn't work on Rundeck v2.10.8

After setting a Job to use the plugin for notifications, after I click save, here's the error that's generated:
Error processing GroovyPageView: Error executing tag <g:render>: Error executing tag <g:form>: Error executing tag <g:render>: Error executing tag <g:render>: Error executing tag <g:render>: Error executing tag <g:render>: Error executing tag <g:render>: failed to lazily initialize a collection of role: rundeck.Option.values, could not initialize proxy - no Session
Anything you'd like me to try?

also on 3.3.10 not working
Caused by: java.net.MalformedURLException: no protocol:  WebHook URL: $webhockurl


CVS API shows "uri" null

Our programmer that is trying to use the CVS API with Dataverse reported to me that the URI is still not displaying through the API. She has tested this yesterday after the new release. Please have a look at this and see what is causing the issue. It should be displayed now, since it is finally working on production.
Not sure who to assign to this, so I am basically assigning you all.

What is the API endpoint she is trying, please? Just to check directly the
requesting API.
On Thu, Mar 9, 2023 at 9:35 AM MajaDolinar @.***> wrote:

Our programmer that is trying to use the CVS API with Dataverse reported
to me that the URI is still not displaying through the API. She has tested
this yesterday after the new release. Please have a look at this and see
what is causing the issue. It should be displayed now, since it is finally
working on production.
Not sure who to assign to this, so I am basically assigning you all.
—
Reply to this email directly, view it on GitHub
https://github.com/cessda/cessda.cvs.two/issues/559, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB23PTTLU46INU3YZDFJB7DW3GI33ANCNFSM6AAAAAAVUY7ZSI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Just tried it via the Swagger docs (https://api.tech.cessda.eu/)
and ran these queries, which returned results:
https://vocabularies.cessda.eu/v2/codes/TopicClassification/4.0/en
https://vocabularies.cessda.eu/v2/search/codes?agency=CESSDA&lang=en&query=Economics&size=20&vocab=TopicClassification

So we are talking about the URI to the individual codes?
On Thu, Mar 9, 2023 at 9:46 AM John Shepherdson @.***>
wrote:

Just tried it via the Swagger docs (https://api.tech.cessda.eu/)
and ran these queries, which returned results:
https://vocabularies.cessda.eu/v2/codes/TopicClassification/4.0/en
https://vocabularies.cessda.eu/v2/search/codes?agency=CESSDA&lang=en&query=Economics&size=20&vocab=TopicClassification
—
Reply to this email directly, view it on GitHub
https://github.com/cessda/cessda.cvs.two/issues/559#issuecomment-1461580623,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB23PTQ4FAYJ3PFBJTUEXOLW3GKGPANCNFSM6AAAAAAVUY7ZSI
.
You are receiving this because you commented.Message ID: <cessda/cessda.
@.***>

@Stifo @pakoselo @john-shepherdson I think Her complaint was not because it does not return anything. The "uri" fields in the "Results" array show "null". See below
{   "@context": {     "skos": "http://www.w3.org/2004/02/skos/core#",     "owl": "http://www.w3.org/2002/07/owl#",     "uri": "@id",     "type": "@type",     "onki": "http://schema.onki.fi/onki#",     "results": {       "@id": "onki:results",       "@container": "@list"     },     "versionInfo": "skos:versionInfo",     "notation": "skos:notation",     "prefLabel": "skos:prefLabel",     "definition": "skos:definition",     "@language": "en"   },   "versionInfo": "4.2.1",   "results": [     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics",       "prefLabel": "ECONOMICS",       "lang": "en",       "vocab": "TopicClassification"     },     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics.EconomicConditionsAndIndicators",       "prefLabel": "Economic conditions and indicators",       "definition": "Economic indicators indicate in which direction an economy is headed and include, for example, GDP growth rate, employment, inflation and interest rate, consumer and investor activity, household debt. Use note: This topic is also used for economic conditions of individuals and households which are surveyed or registered.",       "lang": "en",       "vocab": "TopicClassification"     },     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics.EconomicSystemsAndDevelopment",       "prefLabel": "Economic systems and development",       "definition": "Data covering capitalist, socialist and other economic systems and data on, for example, international development programs.",       "lang": "en",       "vocab": "TopicClassification"     },     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics.EconomicPolicyPublicExpenditureAndRevenue",       "prefLabel": "Economic policy,  public expenditure and revenue",       "definition": "Data on economic policy, including budget and fiscal policy, public expenditure and public revenue.",       "lang": "en",       "vocab": "TopicClassification"     },     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics.IncomePropertyAndInvestmentSaving",       "prefLabel": "Income, property and investment/saving",       "definition": "Data on the accumulation of personal assets, including income, property, investments and savings.",       "lang": "en",       "vocab": "TopicClassification"     },     {       "uri": null,       "type": [         "skos:Concept"       ],       "notation": "Economics.ConsumptionAndConsumerBehaviour",       "prefLabel": "Consumption and consumer behaviour",       "definition": "Refers to the act of acquiring and/or using something to meet the needs and wants of a person, community or society. Data in this category include patterns of consumer behaviour, changes in them, and data on consumer ethics. For example, survey data on how often people buy certain items; when and where they shop; whether ethical considerations affect their consumer choices.",       "lang": "en",       "vocab": "TopicClassification"     }   ] }
I will send a copy of the production DB anyway.

@Joshua-cessda-admin I got it, but the question is why are there differences between staging and production?

Yes @pakoselo you are right. there are differences in the results. Let me know what we miss so we can correct it in the database.
However, I think the way we apply changes to the DB is not sustainable. We need to codify the changes we want to see in the DB. We should talk more about this in the next sprint meeting. @MajaDolinar Please can you add this agenda?

@Joshocan added to the notes for the Sprint

the problem with null uris is due to a bug fixed by #526 when uris were not generated. i'll prepare similar sql script as in #484 to fix it. basically, uris for the migrated codes need to be generated and it will be ok.

The following script re-generates URIs for published concepts with null URIs:
SET @SQL_SAFE_UPDATES=@@SQL_SAFE_UPDATES;
SET SQL_SAFE_UPDATES = 0;
update concept as act_concept
    left join concept as prev_concept on
        act_concept.previous_concept = prev_concept.id
    left join version as act_version on
        act_concept.version_id = act_version.id
    left join version as prev_version on
        prev_concept.version_id = prev_version.id
set
    act_concept.uri = case
        when act_concept.uri is null then
            regexp_replace(
                prev_concept.uri,
                regexp_replace(
                    prev_version.number,
                    '([0-9]+)\.([0-9]+)(\.[0-9]+)?',
                    '$1\\\\.$2(\\\\.[0-9]+)?'
                ),
                act_version.number
            )
        else act_concept.uri
    end
where
    act_concept.uri is null
    and
    act_version.status = 'PUBLISHED'
    and
    prev_version.status = 'PUBLISHED'
    and
    prev_concept.uri is not null
;
SET SQL_SAFE_UPDATES = @SQL_SAFE_UPDATES;

This should be included in the next release. Please execute the prepared code - should not be pushed to 3.3.0.
@john-shepherdson @Joshua-cessda-admin

@matthew-morris-cessda needs to be done as part of tomorrow's release

This I cannot test, so hopefully it will be working with the 3.1.0 release.

DB fix applied

Apparently, the URI is still null. @matthew-morris-cessda have you run the fix? Please check this ASAP

Apparently, the URI is still null. @matthew-morris-cessda have you run the fix? Please check this ASAP

@MajaDolinar can you please provide as with CV, which URIs are nulls? I have etsted few of them and they aren't nulls.

I applied the DB fix yesterday when I closed the issue

Hi!
For example, https://vocabularies.cessda.eu/v2/search/codes?agency=CESSDA&lang=en&query=Economics&size=20&vocab=TopicClassification shows uri: null
meanwhile, it's fine on dev: https://vocabularies-dev.cessda.eu/v2/search/codes?agency=CESSDA&lang=en&query=Economics&size=20&vocab=TopicClassification

I applied the DB fix yesterday when I closed the issue

@matthew-morris-cessda where there any errors during update?

@matthew-morris-cessda, @MajaDolinar @zibertg  apparently there is probably another type of error. If you look into https://vocabularies.cessda.eu/vocabulary/TopicClassification you can see that the latest version is 4.2.2 and actually there is no version labeled 4.2.1 (this is what this query: https://vocabularies.cessda.eu/v2/search/codes?agency=CESSDA&lang=en&query=Economics&size=20&vocab=TopicClassification returns). So there is completely different problem with the database. @matthew-morris-cessda can you please provide us (me and @Stifo ) latest dump of the production database? I assume the source code is same on dev and production sites, the problem must be in the database.

I've attached a copy of the latest production database
Vocabularies Service_CVS Database Export (Issue 559) - 2023-04-05 (13_24_43).zip

@matthew-morris-cessda please, try running reindexing from the admin maintenance page. it seems, that the cause of the problem is an old index.

@Stifo Re-indexing is completed in CVS production.

thanks @Joshocan. problem solved.


About pretrain the YOLOV++ models

Please ask is to pre-train YOLOX by ImageNet VID and DET at the same time, and then fine-tune the model by ImageNet VID to get the final

Yep, the pipeline of YOLOV++ is the same as YOLOV. The differences lie in the model arc.


Tidy things up.

Make methods which can be static methods static methods.
Mark some unused args/kwargs with an underscore
Simplify the logic of some if statements.
Use less broad exceptions where possible
Simplify the creation of lists (i.e. don't initialize and then append, instead initialize with the value which would be appended anyway).
Add a missing parent constructor call to a class.
Add missing doc comments.

fixes #

Checklist:

[x] Add tests that demonstrate the correct behavior of the change. Tests should fail without the change.
[x] Add or update relevant docs, in the docs folder and in code.
[ ] Add an entry in CHANGES.rst summarizing the change and linking to the issue.
[x] Add .. versionchanged:: entries in any relevant code docs.
[x] Run pre-commit hooks and fix any issues.
[x] Run pytest and tox, no tests failed.

Make methods which can be static methods static methods.
Mark some unused args/kwargs with an underscore

Is this really a good idea? To me it feels noisy to make methods static just because they can be, as long as they are always called on an instance..

I think it makes things clearer – but I'm happy to revert the changes if you think that's better.

There are a few good changes in here, so I'll try to extract those. But for the most part I'm going to roll them back. See review comments for discussion.
PyCharm's inspections tend to be pretty noisy by default. I get that it can be annoying to see a yellow instead of a green status, but in the end it's pretty inconsequential. I turn off a lot of the inspections. The various tools we run in pre-commit will catch the important things.