surprising output to ncdump -s with empty netcdf-4 file

I create an empty netcdf-4 file, and do an ncdump -s, and I get the following:
netcdf test_pioc_unlim_multiple_unlim_dim_3 {

// global attributes:
		:_NCProperties = "version=1|netcdflibversion=<IP_ADDRESS>|hdf5libversion=1.10.0" ;
		:_SuperblockVersion = 0 ;
		:_IsNetcdf4 = 0 ;
		:_Format = "netCDF-4" ;
}

I am surprised that _IsNtecdf4 = 0. Is that expected?
Also, is _SuperblockVersion 0 a valid version?
After I add a dimension, I get the following:
netcdf test_pioc_unlim_multiple_unlim_dim_3 {
dimensions:
	unlimited1 = UNLIMITED ; // (0 currently)

// global attributes:
		:_NCProperties = "version=1|netcdflibversion=<IP_ADDRESS>|hdf5libversion=1.10.0" ;
		:_SuperblockVersion = 0 ;
		:_IsNetcdf4 = 1 ;
		:_Format = "netCDF-4" ;
}

I am surprised that _IsNtecdf4 = 0. Is that expected?
yes, because no netcdf-4 specific informationis available.
The value 0 in this case means that  it looks lilke an HDF5 file
that was not created by netcdf.

Also, is _SuperblockVersion 0 a valid version?
You might do a hexdump on the file and see what value the
superblock varsion in the file has.

OK, thanks for the quick answer!

But wait a moment! The library should know this file was created with netCDF-4 from _NCproperties, right?

Not for older files that do not have _Ncproperties.
_isnetcdf4 is computed.

Look at the "provenance" section here.
http://www.unidata.ucar.edu/software/netcdf/docs/attribute_conventions.html

Ok thanks.


Add Annulus transform

@MikaelSlevinsky is there a way to do a Q&D transform for OPs in the annulus?

@dlfivefifty have you already integrated Mikael's transforms for the annulus or is this something I can do?

I haven't done anything....

I'll give it a go!


Add Support for local Modbus access on certain devices

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
On some devices there is a modbus interface which can be accessed on the local network.  This is a place holder for https://github.com/sHedC/python-masterthermconnect/issues/113
Describe the solution you'd like
Add Modbus support as an option
Describe alternatives you've considered
N/A
Additional context
N/A

Just wondered if this had progressed at all?


Support for Downloadable Google Fonts (ui-text-google-fonts)

Jetpack Compose allows you to download fonts from Google Fonts asynchronously in your app, rather than having to bundle them in the application.
https://developer.android.com/jetpack/compose/text/fonts#downloadable-fonts
This is done via the androidx.compose.ui:ui-text-google-fonts artifact, which exposes a GoogleFont.Provider, allowing developers to quickly add new fonts to their apps (without increasing the size of the initial app download).
Would be great to have support for these downloadable fonts in Compose Multiplatform as well.

Any updates on this? Thanks.

We don't have a plan to support it in the near future

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Remove pdepend dev-master dependency when 2.9.2 is released

PHPMD was broken with the match feature introduced in PHP 8.
The fix for that was present in pdepend's master branch, but not yet released (planned for 2.9.2).
705bc8573a834cf48955d34ab54f102153f53b2e added pdepend as a dev dependency directly to circumvent the problem in the meantime. However, this should be rolled back when pdepend's 2.9.2 will be released, to go back to a stable version.

This was fixed in 30cc1d0e2f1c797c74b0125be0a0ba465af1b18c.


load_weights will fail if shape mismatch

Fix for #10265
Ran keras tests, same results as origin/master:
31 failed, 549 passed, 98 skipped, 162 warnings, 2 error

It is reasonable, but I am concerned about performance degradation. If shape checking is performed for every weight, load_weights will be very slow.

Fair point, @taehoonlee. I timed it with and without my changes to load weights for ResNet50 with this gist: https://gist.github.com/AmirAlavi/ab8e3bdf3fca09c25ba808dd4cbdd48a
Time without my changes: 0.013439978134818375
Time with my changes:      0.013317499115131795
This was with timeit, with 10 repetitions of 100 calls (reported time is the minimum from the 10 runs, divided by 100). It's odd that it was faster, but I think the times are so similar that we can take this to mean that the performance hit is negligible if present at all. Please let me know what you think.

@taehoonlee, sorry, I had a mistake in my timing script (skip_mismatch was set to True, which caused the original code to also check each shape) which explains why the times were so similar. I had set it to True to get a particular model of mine to work, which had shape mismatches. I just changed it to skip_mismatch=False and here are the results:
Time without my changes: 0.013299283728003502
Time with my changes: 0.01338828501291573
It's slightly slower, but I think it's worth pointing out that in the current code, the different values of skip_mismatch cause different runtimes of load_weights in a non-intuitive way: if I "skip" things, I expect them to be faster, but load_weights is actually (slightly) slower because of the shape checking.
With this pull request, the runtimes will be the same regardless of the value of that flag, and we gain proper reporting of programming error.


fhir-persistence-schema CLI drop-admin does not clear out the fhir_admin schema

Describe the bug
when using the fhir-persistence-schema CLI tool to drop-admin the fhir_admin schema is not cleared out. the table in there still remains. and prevents the reuse of the DB for FHIR use.
Environment
Which version of IBM FHIR Server?
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
the use of --drop-admin in the fhir-persistence-schema CLI tool should clean out the fhir_admin schema
Additional context
Add any other context about the problem here.

now using fhir-persistence-schema cli to drop fhir_admin does allow you to reuse the schema for a new fhir server with the same fhir schema name.


Support Heroku-22 stack

Heroku-22 Stack has been released and it breaks the wkhtmltopdf-heroku gem with the following error message:

Failed to execute:
["/app/vendor/bundle/ruby/3.1.0/gems/wkhtmltopdf-heroku-<IP_ADDRESS>/bin/wkhtmltopdf-linux-amd64", "--page-size", "Letter", "file:////tmp/wicked_pdf20220622-6-3pxuf2.html", "/tmp/wicked_pdf_generated_file20220622-6-ko5r5s.pdf"]
Error: PDF could not be generated! Command Error: /app/vendor/bundle/ruby/3.1.0/gems/wkhtmltopdf-heroku-<IP_ADDRESS>/bin/wkhtmltopdf-linux-amd64: error while loading shared libraries: libssl.so.1.1: cannot open shared object file: No such file or directory

Heroku explains this with:

OpenSSL 3
Ubuntu 22.04 ships with OpenSSL 3.0. OpenSSL 1.1 is not available as a runtime library.

If you see OpenSSL/libssl related errors (for example libssl.so.1.1: cannot open shared object file, but there will be many other variations), you will need to update your dependencies and/or third-party buildpacks to a version that is compatible with OpenSSL 3, and ensure that dependencies or binaries you use are compiled against OpenSSL 3.

In addition, OpenSSL 3 drops default support for a number of legacy/insecure cryptographic algorithms. If you see TLS/SSL related failures connecting from your application to external services, it is likely that those servers are running outdated/insecure software or configurations, that will need to be fixed by the maintainers of those services.

@pallymore tell me if I can help by suggesting a PR 🙏

hmm this is tricky. I don't think that's something we can resolve in this repository - this gem simply ships the binary from https://github.com/wkhtmltopdf/wkhtmltopdf , unfortunately that repo is in a semi-abandoned state.
We could try to compile a binary from source with openssl 3 but it still might not work due to the breaking changes mentioned in the article 🤔

Looks like someone has already compiled a binary for Ubuntu 22: https://github.com/wkhtmltopdf/packaging/issues/114
testing that shortly.

Hi - I think it actually works - here's a test page:
https://wkhtmltopdf-demo.herokuapp.com/greetings/hello.pdf
could you test this on your end also? here's my test "app":
https://github.com/pallymore/wkhtmltopdf-test
could you try it with: https://github.com/pallymore/wkhtmltopdf-test/blob/main/Gemfile#L29
gem 'wkhtmltopdf-heroku',  git: 'https://github.com/rposborne/wkhtmltopdf-heroku.git', branch: '<IP_ADDRESS>-jammy'

@pallymore there's an official release for 22.04 published just now.

thanks - checking if those binaries work 🤔

Hi @ashkulz  - the latest release does not work on Heroku-22.
I'm getting the same libssl 1.1 error - for now i've reverted the binary back: https://github.com/rposborne/wkhtmltopdf-heroku/commits/<IP_ADDRESS>-jammy
@guillaumewrobel please let me know if that branch works for you:
gem 'wkhtmltopdf-heroku',  git: 'https://github.com/rposborne/wkhtmltopdf-heroku.git', branch: '<IP_ADDRESS>-jammy'

@pallymore yeah, there were some issues and I've deleted the release.

awesome, for now i've published the branch as version <IP_ADDRESS>.pre.jammy until an official binary is available.
https://rubygems.org/gems/wkhtmltopdf-heroku/versions/<IP_ADDRESS>.pre.jammy

@pallymore please check again with the latest build.

Hi @ashkulz  the latest release works! thank you!

Released: https://rubygems.org/gems/wkhtmltopdf-heroku/versions/<IP_ADDRESS>.pre.jammy
Thanks!
Is there going to be a Ubuntu 20 (Focal) AMD64 build?

@pallymore there is already a Ubuntu 20.04 build which was part of the original 0.12.6 release ... not sure what you mean? :thinking:

nvm, I thought this release is for <IP_ADDRESS> and the original release was <IP_ADDRESS> 😅 we are good here if they are built from the same source.
thank you again.

Yeah, the only changes in the 0.12.6.x branch is some fixes to get it working with newer compiler versions.

@toddsutt  I see you are using both wkhtmltopdf-binary and wkhtmltopdf-heroku
I'd check which one is being picked up by wicked_pdf
I'd recommend keeping wkhtmltopdf-binary in the :development and :test gem group, and only use wkhtmltopdf-heroku in production.

@pallymore thank you for the response! As you can see, I don't understand what these gems are doing, so I will give that a try!

@pallymore after your suggestion I get this error on heroku:
Command Error: /app/vendor/bundle/ruby/3.1.0/gems/bundler-2.3.10/lib/bundler/rubygems_integration.rb:319:in `block in replace_bin_path': can't find executable wkhtmltopdf for gem wkhtmltopdf-binary. wkhtmltopdf-binary is not currently included in the bundle, perhaps you meant to add it to your Gemfile? (Gem::Exception)

yea looks like it's still trying to find wkhtmltopdf-binary - I'd check where it is still referenced in the code.
what's in your wicked_pdf config file?

Do I need to do something to uninstall the binary gem from heroku after moving it to development? Like delete my Gemfile.lock?

Sorry for the late response - in regards to the lock file, yea you will need to run bundle install after making the gemfile changes, and commit the new lock file.
I'd search wkhtmltopdf-binary in the codebase to see if it's still being referenced somewhere - assuming you've set the environment to production on Heroku, only  wkhtmltopdf-heroku gem should be loaded.

also could you DM me on twitter we can continue there if you are still having issues: https://twitter.com/Yuruiology 👋🏻

@pallymore I finally figured it out! Btw I couldn't message you on twitter it said I was unable to send messages even after following.
I tried a whole lot of things but it ended up just being that I needed to add exe_path: Gem.bin_path('wkhtmltopdf-heroku', 'wkhtmltopdf-linux-amd64') to my WickedPdf.config for production environment.

great - I don't think exe_path is needed as this gem configures that for you automatically.
however I think I know the problem now - in your config you had:
WickedPdf.config = {
}

which overrides the whole config object - in the README file, it's recommended to extend the config instead
(see https://github.com/rposborne/wkhtmltopdf-heroku#when-using-with-wicked_pdf )
WickedPdf.config ||= {}
WickedPdf.config.merge!({
  # your extra configurations here
})

this should eliminate the needs of having to set an exe_path manually.

What is the solution I have the same problem

What is the solution I have the same problem

@nickyczeme for me it was what @pallymore last said - I simply needed to remove the WickedPdf.config = {} from my initializer


Slider Full Screen Image Carousel

How do i create a full screen image carousel?

use the property  :perPage="1" in your carousel, check the examples on the doc

any update on this?

As mentioned above, the :perPage=1 prop should do the trick


NetCDF file type metadata has wrong spatial reference and spatial coverage forms

The NetCDF file type in composite resource doesn't show the same metadata forms as that of NetCDF resource type (for spatial reference and spatial coverage metadata).
This makes the private composite resource unable to become a public resource when user needs to manually enter the required metadata. This issue can be tested with the NetCDF file in resource https://www.hydroshare.org/resource/25d864632a764ac3bb43fc35df7ff265/
The spatial coverage form should not include "longitude" and "latitude" fields. The spatial reference form should include "north extent", 'east extent', 'south extent', 'west extent' fields.
But after the user edits the metadata and clicks "update netCDF file" button, the two forms will show correctly.

@sblack-usu @pkdash is this still relevant or can be closed?

@sblack-usu @pkdash is this still relevant or can be closed?

It can be closed.  The other extent fields are displayed and the longitude/latitude are no longer displayed in netcdf aggregations.

It can be closed.  The other extent fields are displayed and the longitude/latitude are no longer displayed in netcdf aggregations.


Ask for help

I'm sorry to disturb you. I'm a newer. I try my best to study the code, but I don't know how to solve it. There may be two errors in the scripts.
1.When I ran the script , there generated 5 files(sgns.contexts.txt, sgns.contexts.bin, sgns.words.txt, sgns.words.bin, sgns.words.words).what's the last file? Why is sgns.words.words blank?Can you offer the source code of the word2vecf?
word2vecf/word2vecf  -train w2.sub/pairs -pow 0.75 -cvocab w2.sub/counts.contexts.vocab -wvocab w2.sub/counts.words.vocab -dumpcv w2.sub/sgns.contexts -output w2.sub/sgns.words -threads 10 -negative 15 -size 500;
2.the file of vecanalysis has no representations. If I want to get the alligned embeddings, how can I operate it. What's the perparameters to coney in the scripts of the seq_procrustes.py?
from vecanalysis.representations.representation_factory import create_representation
Thanks for your help.Thanks.@williamleif

Hmm, the way that I use the scripts is to run:
https://github.com/williamleif/histwords/sgns/runword2vec.py
(for word2vecf information see: https://bitbucket.org/omerlevy/hyperwords or use the makecorpus.py script in my package)
and then you can run:
https://github.com/williamleif/histwords/sgns/postprocesssgns.py
To post-process the word2vecf output into the format histwords uses.
Finally, you would run:
https://github.com/williamleif/histwords/seq_procrustes.py
On the output of "postprocesssgns.py".
Hope that is somewhat helpful!


SGH-1616: Add MessageSourceCustomizer

Resolves https://github.com/spring-cloud/spring-cloud-stream/issues/1616

LGTM - merged upstream.


Node-collector mounts for CNI binaries

What steps did you take and what happened:
Installed trivy-operator via: https://github.com/aquasecurity/trivy-operator/blob/main/deploy/static/trivy-operator.yaml
node-collector job is unable to be created with error in Event:
failed to generate container "e5a..." spec: failed to generate spec: failed to mkdir "/opt/cni/bin/": mkdir /opt/cni: read-only file system: CreateContainerError
What did you expect to happen:
node-collector runs without error
Anything else you would like to add:
It looks like this is where the cni binaries are for Cilium (the CNI in use), but I don't see a way to configure that for node-collector:
volumeMounts:
  - mountPath: /host/opt/cni/bin
    name: cni-path

[Miscellaneous information that will assist in solving the issue.]
Environment:

Trivy-Operator version (use trivy-operator version): 0.12.1
Kubernetes version (use kubectl version): 1.25.6-gke.1000
OS (macOS 10.15, Windows 10, Ubuntu 19.10 etc): GKE
CNI: Cilium (via DataplaneV2)

@timclifford @rwkarg I assume you mean that you want the the volume (on host) --> hostPath to be configurable , right?
example:
- name: opt-cni-bin
        hostPath:
          path: /opt/cni/bin/

@chen-keinan Yes, for my use case the calico CNI plugin is installed to a different location than the default '/opt/cni/bin', being able to change the mount path would fix this issue for me.


[WIP] use sphinx-click to autodoc cli

Opening as a WIP since the PR doesn't really improve on what we have currently. It's an option to autodoc the CLI now or in the future.

Pull Request Test Coverage Report for Build 118

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

Totals

Change from base Build 113:
0.0%

Covered Lines:
1123

Relevant Lines:
1266

💛  - Coveralls

Thanks for this suggestion!
I am not sure about this since we need the CLI --help in the README anyway. Since we are copying this automatically from the README now, I think we can pass on this idea.


toContainReact fails even though output matches

First off: I have tried searching the web to see if this was a common issue, but that does not seem to be the case. I assume this is the repository I should be posting this issue to -- if it is not, feel free to close it (and please direct me to the correct repository).
Now then!
I have written a recursive React component that I am writing tests for. For some reason, it seems that toContainReact fails even though the HTML output contains the expected elements.
I created an application that contains a minimum reproduction (at least as minimum as I could find) of this issue: Saser/tocontainreact-failing
When running the test in the above repo, I get the following output:
Expected
 to contain outerinner but it was not found.
HTML Output of
:

outerinner

Any idea as to what might be wrong?

it seems like the outputs are actually different? the actual component output seems to be wrapped in an additional div?

Hmm, maybe. Perhaps I haven't grokked everything about hos JSX works and how it interacts with these wrappers and test matchers.
Anyhow, I updated the test to not wrap the component in a
 and instead just render it directly: Saser/tocontainreact-failing @ 1a8ed51. The test is still failing, with this output:
Expected <Item> to contain outerinner but it was not found.
HTML Output of <Item>:
 outerinner

Oh, another note: if I "cheat" and just return the expected output directly, the test is passing.
const Item = () => outerinner;

Apologies, I haven't had time to dig into this. If you want to look into the code and figure out what's happening I would happily accept a PR!

Dug into this. Pretty sure there isn't an issue here. I tried to look at your repo, but looks like you deleted it. I'm going to close this for now. If you can help me dive into it sometime I'd love to figure out what's going on.


Unclear Metric Usage

The current system is difficult based on common structure and understanding of tensorflow to get working.
import tensorflow_addons as tfa
model.compile(
	loss=mean_squared_error,
	optimizer=Adadelta(),
	metrics=['mse', 'mae', tfa.metrics.CohenKappa, tfa.metrics.RSquare]
)

Relevant information

Are you willing to contribute it (yes/no): Yes I can add a working example when I figure it out.
Are you willing to maintain it going forward? (yes/no): No.
Is there a relevant academic paper? (if so, where): No
Is there already an implementation in another framework? (if so, where): No.
Was it part of tf.contrib? (if so, where): No.

Which API type would this fall under (layer, metric, optimizer, etc.)
Metric/Documentation.
Who will benefit with this feature?
People using tf-addons for the first time.

@vccabral @seanpmorgan , It would be great if you could please elaborate this issue, so that I could figure out if I can solve this one.
Thanks

Hi @vccabral could you explain a little more please. Are you looking for documentation that explains the  code snippet you posted, or does the code snippet you posted not work?

Hi @vccabral I'm still having a bit of an issue understanding what you're requesting here. IIUC you'd like to have better documentation about using tfa metrics with Keras?
We would be most happy to review a PR which adds more information to the doc string or a new tutorial for metrics

@seanpmorgan I was attempting to use TF addons with keras which was a bit more complicated than I originally thought it would be(exacerbated by the recent version update to TF2 on my part). Once I get a tutorial working with keras, I am happy to submit a PR to the tutorial's section.

Great! Looking forward to the PR. I'm going to close this issue for now... but feel free to re-open as a request for tutorial if you want to track it that way.


bug(homepage): ubuntu icon

ubuntu now use a more flat icon.
See https://mirrors.help and https://mirrors.zju.edu.cn/
Mentioned by @RalXYZ

#70


Syntax for pattern matching on lists

Right now, both of these work:
let head = fun lst => switch lst {
  | [] => failwith "Empty list"
  | [hd, ...tl] => hd
};

let head = fun lst => switch lst {
  | [] => failwith "Empty list"
  | hd::tl => hd
};

Is this by design? Since item1 :: item2 :: rest has been replaced by [item1, item2, ...rest], it seems like sticking to the latter syntax while pattern matching is the more consistent option.

Good call. We must have merely not deleted this parsing rule yet. Feel free to send a PR to remove it.

Hi!
Among all the syntax changes, this is the only one I did not like. What seems odd for me is that the "..." is used as well in records "{...a,  }". My suggestion is to use the "..." only in one of the contexts.
It would be interesting to know the motivations behind this syntax change.

@modlfo Good question!
When I first saw the syntax, I feel it is pretty intuitive. I think ... as "expand" operator.
In the case of [hd, ...tl], I interpret it as "expand list tl, add hd in front of it"
In the case of {...a, }, I interpret it as "expand record a, modify certain fields of it"

Hi @modlfo: It's good to hear you found the rest of the changes helpful/preferable. This is the syntax used by JavaScript for Array and object(record) "spreading". That syntactic pattern ... represents "that other thing, but with these things added".
The reasoning was that it doesn't matter too much what they look like as long as they have no syntactic pitfalls and were somewhat easy to learn.
I don't really love them, personally (I just find three dots ... a little annoying to type), but I find like that more people can instantly recognize them in my code even without having to learn a new language.
I like this better fwiw, do you agree?
let r = {myRecord | nextField: "hi"};
let a = [head | tl ];

Hi @yunxing that's exactly the case ([...tl, hd]) I was thinking that it can be confusing with ...
Since I'm more familiar with OCaml and F# I have a strong preference for the cons operator :: to create and pattern-match lists.  h :: t is more compact than [ h, ... t ].
@jordwalke using | in my opinion looks nicer and one has to type less. In addition, it's not possible  to write [ | t, h ] as in [...tl, hd]. In the case of records, {myRecord | nextField: "hi"} looks quite nice for me.

Currently I don't like the Prolog version [head | tl ] very much - at first glance it looks like a polymorphic variant from OCaml (BTW does Reason support polymorphic variants?).

@radekm We do support polymorphic variant with the same syntax. It actually took us a lot of efforts since there are many edge cases of syntax that we don't even know what the semantics are.
We chose not to document it in the basic syntax so we don't scare people away : )

that we don't even know what the semantics are.

@Yunxing is referring to "intersection types", which many people don't know even exist in OCaml. But we support parsing and print of them because we want to maintain a complete mapping of the entire language.

I would like to jump in and say that I like the ... spread syntax from JS the best in terms of readability.
That said, I prefer consistency over anything else. The ... syntax is consistent across lists and records.
But on the down-side, [...head, tail] being illegal is a little annoying. But that seems like a problem in OCaml itself, and so I'm not sure it can be solved with syntax.

I agree with @nmn, the consistency is important even at the cost of "more characters to type" when matching on [head, ...tail] instead of head::tail. Also [head | tail] is confusing coming from lower level languages where | is bit-wise OR.

@nmn, may I ask, is your background web/JavaScript development? Have you used other functional languages such as OCaml (I ask because it helps us keep track of which suggestions/preferences come from which "groups" of people). So far, quite predictably, people from JS web background resoundingly favor syntactic forms that resemble ES6 (I suppose this wasn't that difficult to predict).
fwiw, [...list, appendOne] could be made to represent List concatenation. Even [...list, insertOne, ...tail] could be made to work. This obviously couldn't work for pattern matching, but could work with expressions. The one downside is that it does encourage order n operations on lists.

My background is in the Javascript community. That said, I keep learning languages. I have some experience with Haskell, Elm, and Purescript.
So, while of course, I have my own bias, I'm trying to be objective whenever I comment on an issue.

@nmn. Cool, just trying to understand where you're coming from. I'm somewhat like you in that I try just about every new language.

I'm open to many of the proposals for pattern matching syntax, but I for one would not welcome pattern matching syntax that cannot be compiled efficiently.  "Efficiently" probably needs to be clarified, but I would say not more than a few branches (linear in the number of cases with a small constant), and preferably no allocation.

@jberdine We are in total agreement - the slower operations would not be pattern matching, they would be expressions that represent concatenation (exactly the same as if you had written List.concat ..).

After a discussion with @schmavery, we've reached the conclusion that even though ... isn't perfect, it's a good choice considering the tradeoffs (familiarity, power, visual representation of expression vs pattern matching, reasoning simplicity). That said, using it for both records and lists might not be necessary. We currently don't support [...rest, a] or [a, ...rest, b] because they're implicitly linear time. This makes ... act pretty differently on lists (differently compared to JS and compared to reason records). So I'd be in favor of bringing :: back for lists, which is nice and consistent across the board, and keeping ... for records.

That makes sense to me.

We currently don't support [...rest, a] or [a, ...rest, b] because they're implicitly linear time.

We could trivially for list concatenation, making them more consistent with the JavaScript syntax. The primary value of the ... syntax is just that they're immediately recognizable by the largest programming community in the world. I don't even particularly love them, but I love that just about anyone can immediately pick up on what they represent which means code I write will be understood by more people, and more people can benefit from OCaml more quickly.
When I was learning OCaml, it took me a little white to understand infix operators that weren't the traditional +/- etc because it wasn't obvious what the parsing precedence was:
myFunc a b :: myFunc c d

It took me a little while to understand that it was never parsed like:
myFunc a (b :: myFunc) c d

Now that I've been knee deep in the yacc parser for a while now, I understand fully, but I'm trying to put myself in the shoes of a noob. There's a ton of interest in Reason specifically because it makes an effort to appeal to a wider audience. If there's anything that's more familiar and we can tolerate it, we should opt for it, especially since we know we can always fix it later and auto-upgrade peoples' code using refmt. If the only thing accomplished by Reason is that we had gotten more people to use OCaml in its current exact form, but had to go through the process of casting a net with an approachable syntax and bringing it back closer over time - then it seems like a win to me.
We don't want to be familiar just for the sake of familiarity - there are some things that I think we should do just as ML has done, but this one isn't quite as clear so that makes me lean more towards widespread familiarity - we can always change our minds later.

@jordwalke thanks for explaining. ... everywhere does make it more readable. Time will tell if JS folks can't unlearn [...rest, a] ;p

"unlearning" differences in syntax is an argument for making different things look different.  This comes up a few places in Reason - ... in lists, switch for pattern matching both come to mind.

IIRC, Swift not only used the keyword switch, but even went as far as to require huge ugly "case" tokens before each match. IMHO that's taking it too far, but I do see the benefit of making things familiar even if it's a new concept. It's the difference between "here's an entirely new concept" and "switch statements are better than you are used to in other languages".

I'm closing this now as the concern of the original poster has been resolved. Feel free to reopen this one or continue discussing about what syntax should we use here.


Difficulty w/ ViewPager

I have a ViewPager that contains 2 pages. I set disableWhenHorizontalMove(true), and this seems to inhibit the invocation of checkCanDoRefresh(). The swiping action registers (checkCanDoRefresh()) infrequently and inconsistently when disableWhenHorizontalMove(true) is set. However, when I set disableWhenHorizontalMove(false), the swipe registers every single time but at the cost of shaky swiping between my ViewPager elements.
I poked around a bit in the source code, and identified the possible lines that might be causing this problem:
            if (mDisableWhenHorizontalMove && !mPreventForHorizontal && (Math.abs(offsetX) > mPagingTouchSlop || Math.abs(offsetX) > 3 * Math.abs(offsetY))) {
                if (mPtrIndicator.isInStartPosition()) {
                    mPreventForHorizontal = true;
                }
            }
            if (mPreventForHorizontal) {
                return dispatchTouchEventSupper(e);
            }

I think the tolerance for mPreventForHorizontal might be a bit too high.

Yes, you are right, :+1: , this's a problem. Do you have any idea to make this better?

I currently don't have time to look into it, but I'll try later.

这问题解决了吗，我也遇到这个问题了

This's not a real problem. It just a bit too sensitive.


Deepposekit-annotation

Traceback (most recent call last):
File "/Users/liuximac/deepposekit-data/datasets/fly/02.annotate.py", line 11, in
app.run()
File "/Users/liuximac/miniconda3/envs/tf/lib/python3.9/site-packages/deepposekit/annotate/gui/GUI.py", line 657, in run
self._hotkeys()
File "/Users/liuximac/miniconda3/envs/tf/lib/python3.9/site-packages/deepposekit/annotate/gui/Annotator.py", line 332, in _hotkeys
self._data()
File "/Users/liuximac/miniconda3/envs/tf/lib/python3.9/site-packages/deepposekit/annotate/gui/Annotator.py", line 316, in _data
self._save()
File "/Users/liuximac/miniconda3/envs/tf/lib/python3.9/site-packages/deepposekit/annotate/gui/Annotator.py", line 201, in _save
h5file["annotations"][self.image_idx] = self.skeleton.loc[
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "/Users/liuximac/miniconda3/envs/tf/lib/python3.9/site-packages/h5py/_hl/dataset.py", line 951, in setitem
self.id.write(mspace, fspace, val, mtype, dxpl=self._dxpl)
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py/h5d.pyx", line 232, in h5py.h5d.DatasetID.write
File "h5py/_proxy.pyx", line 114, in h5py._proxy.dset_rw
OSError: Can't write data (no write intent on file)

The image I marked can't be saved successfully? Why?


Update Proxy Version

Includes:

#7896
#7894
#8020
#8035
#8054

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Larger icon if state hidden

Is your feature request related to a problem? Please describe.
Icon size is a little small for older eyes.
Describe the solution you'd like
Can icon take up full height if state is hidden?
Describe alternatives you've considered
None

I understand your feature request, but I guess this is not for everyone, so I will not implement this by default.
That said there is an easy way to enlarge the icon size (per card).
Make sure you have installed the awesome Card mod plugin.
After that you add this to the card config in yaml:
  style: |
    ha-card {
      --mdc-icon-size: 3em !important;
    }

Left: 3em | Right 2.2em (default for this card)
Don't forget the !important part otherwise it won't work.
Downside is that you have to add this for every Slider button card.
Hope this helps you.

Thanks! How about a compact mode of half height instead? I find the sliders
quite thick for just lights/fans.
On Fri, 18 Jun, 2021, 10:20 pm MHA, @.***> wrote:

Closed #34 https://github.com/mattieha/slider-button-card/issues/34.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mattieha/slider-button-card/issues/34#event-4910781955,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHHBO46IHVULI2PF5HIEKCLTTN2MNANCNFSM462ONQ4A
.

@mouth4war I've made a compact mode:  #68 , looking forward about your feedback.

It's brilliant


Needs excel files.  [FEAT]:

What would you like to see?
A huge amount of info is loaded into Microsoft Excel files, yet you do not allow them in your document set.    That DRAMATICALLY reduces the value of this solution.  Solving that would increase your TAM greatly.

You can do this already. RAG on an excel is just a whole different use case but can be solved by document pinning or via agents. Agents are not supported yet.


GPflow on conda-forge

Right now the newest version is  1.3.0. Are there any plans of uploading newer versions to conda-forge? I would really appreciate it.

Hi @GuillemGSubies, I didn't even know we had a package on conda-forge!
Could you explain the value you see in a conda-forge package? We are already maintaining the pip package - you can simply do pip install gpflow (and this should work just fine in a conda environment, too). What added value does the conda forge bring? (It's a trade-off against the cost of maintaining it!)

In my company we use conda as a standard and we have a package that uses gpflow. So when trying to distribute it, not having a dependency available in conda, becomes a pain sometimes.
I did not know that you were not the maintainers of that conda-forge feedstock so I would understand if you cannot solve this issue.
Thanks you for the fast response

Hi @GuillemGSubies, it looks like the gpflow conda-forge https://github.com/conda-forge/gpflow-feedstock actually has auto-generated PRs for updating to current GPflow versions: unfortunately, it looks like no-one of the maintainers is addressing them. You might want to get in touch with the conda-forge admins to sort this out - I'm afraid there's nothing we can do about it as GPflow maintainers! Happy to chat further on gpflow.slack.com (invite link in README.md) if you want to discuss what you might need to do to get the conda-forge package updated.


Fix avatar radius problem on the new issue page

Close #31502
Related to #31419.
In this PR, the avatar width is set to 3em, but the height is not set, so the image is not squared.
When object-fit is set to contain, it can't maintain the radius of the image.
Result:

Wait it looks strange in my local:
before (main branch)

after(too big):

So my suggestion is

Ideally we should size avatars correctly inside the templates only and remove all the CSS that alters their size, it's unnecessary.

So my suggestion is
After:

This seems to be a better solution.
I have been updated.

Ideally we should size avatars correctly inside the templates only and remove all the CSS that alters their size, it's unnecessary.

@silverwind
I agree, how do you think about this PR, close it or merged it first as a quick fix and do that later.

Fix is fine to take as a quick fix. Ultimately it was a regression from https://github.com/go-gitea/gitea/pull/31419.


Update ENTRYPOINT/CMD table to agree with docs

- What I did
In the Dockerfile reference documentation, the table describing the interaction of ENTRYPOINT and CMD was not in agreement with what other parts of the document said. For example, this sentence https://github.com/docker/docker/blame/master/docs/reference/builder.md#L1084-L1085 states that CMD is ignored when the shell form of the ENTRYPOINT command is used but the table indicated otherwise. I tested myself which was correct, finding the table was incorrect, so I updated the table.
- How I did it
I edited the markdown in a text editor.
- How to verify it
You can create a Dockerfile with the following contents:
FROM busybox:latest
ENTRYPOINT /bin/echo p1_entry
CMD shell_cmd p1_cmd

build a container, run it, and see that p1_entry is output but shell_cmd p1_cmd is not.  Similar for an exec form of CMD.
- Description for the changelog
Dockerfile documentation more consistent and accurate
- A picture of a cute animal (not mandatory but encouraged)

Several other places in the document it states that when using the shell
form of ENTRYPOINT, CMD and command line arguments are ignored.  That is
accurate, this table was not.  It is now.
Signed-off-by: David Dooling<EMAIL_ADDRESS>

ping @sfsmithcha

I think you are right that this table is a little confusing. However I don't think that your update is correct.
The way I understand it, the column ENTRYPOINT exec_entry p1_entry is meant to show how the container executes a combination of that ENTRYPOINT with the CMD from the first column.
I could be wrong, but I'm not sure your example test accurate. I think shell_cmd  would need to be a valid command in busybox:
$ docker run -it --rm busybox
/ # shell_cmd
sh: shell_cmd: not found

cc: @cpuguy83 , @vdemeester , @mstanleyjones

@sfsmithcha you are correct that the table is meant to show how the container executes a combination of an ENTRYPOINT with a CMD. Unfortunately, the table is wrong. When a shell form of ENTRYPOINT is used, CMD is ignored, as stated several other places in the document. That is why my patch removes the specific text it does, as it contradicts the other places in the document and the actual behavior.
shell_cmd is not meant to be executed alone, it is meant as an argument provided to the ENTRYPOINT, which in my example is echo. It is meant to be a place holder for a command or text that someone would provide either in a CMD Dockerfile command or on the command-line. In either case, it is immaterial since it is ignored when the shell form of ENTRYPOINT is used.
If you create the Dockerfile I describe above and run it you will see that p1_entry is output but shell_cmd p1_cmd is not. If you prefer, replace shell_cmd with and actual command like ls and you'll see the same response.
$ docker version
Client:
 Version:      1.10.1
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   9e83765
 Built:        Fri Feb 12 22:11:40 UTC 2016
 OS/Arch:      darwin/amd64

Server:
 Version:      1.12.1
 API version:  1.24
 Go version:   go1.6.3
 Git commit:   f1e1b83
 Built:
 OS/Arch:      linux/amd64
$ cat Dockerfile
FROM busybox:latest
ENTRYPOINT /bin/echo p1_entry
CMD shell_cmd p1_cmd
$ docker build .
Sending build context to Docker daemon 2.048 kB
Step 1 : FROM busybox:latest
 ---> 2b8fd9751c4c
Step 2 : ENTRYPOINT /bin/echo p1_entry
 ---> Running in ecb0f279bc43
 ---> 5c380b4d10b6
Removing intermediate container ecb0f279bc43
Step 3 : CMD exec_cmd p1_cmd
 ---> Running in 46c4d9e020e9
 ---> 89d6415ec541
Removing intermediate container 46c4d9e020e9
Successfully built 89d6415ec541
$ docker run --rm 89d6415ec541
p1_entry
$ docker run --rm 89d6415ec541 foo bar
p1_entry
$ cat DockerfileExec
FROM busybox:latest
ENTRYPOINT /bin/echo p1_entry
CMD ["exec_cmd", "p1_cmd"]
$ docker build -f DockerfileExec  .
Sending build context to Docker daemon 3.072 kB
Step 1 : FROM busybox:latest
 ---> 2b8fd9751c4c
Step 2 : ENTRYPOINT /bin/echo p1_entry
 ---> Running in 701e0215e830
 ---> 102919a899bd
Removing intermediate container 701e0215e830
Step 3 : CMD exec_cmd p1_cmd
 ---> Running in 091e99cab0b6
 ---> 9bc311a02243
Removing intermediate container 091e99cab0b6
Successfully built 9bc311a02243
$ docker run --rm 9bc311a02243
p1_entry
$ docker run --rm 9bc311a02243 foo bar
p1_entry

@mstanleyjones PTAL

LGTM

Actually, italic is usually used for replaceable text. You'd probably want italic monotext. It's a little tricky in markdown, but use an underscore followed by the backtick followed by the phrase. Like this.

@SvenDowideit and @mstanleyjones you seem to be commenting on the format of the comments in the PR and not the actual PR, in which I changed no formatting and added no text. I only removed some erroneous text from the table.

@ddgenome nope - we're suggesting improving how the existing text could be improved.
sadly, I had an escaping fail - I meant to say put < and > around the non-functional examples

@SvenDowideit The desire to change how the table data is formatted is orthogonal to fixing the erroneous data in the table. Can this be merged in and an issue opened for your desired changes around formatting?


Use ellipsis when many versions returned for [ModrinthGameVersions]

Fixes #10346.
To make sure the ellipsis falls in the "middle", the list of game versions must be sorted. All the examples I've tried seem to return sorted results from the API, so I'm running under that assumption and have kept the implementation simple. We can always revisit later if need-be.

I think I'm fine with this. Do you have any thoughts on the slightly more involved solution I mentioned in https://github.com/badges/shields/issues/10346#issuecomment-2223139997 ?

I think I'm fine with this. Do you have any thoughts on the slightly more involved solution I mentioned in https://github.com/badges/shields/issues/10346#issuecomment-2223139997 ?

I figured I'd go for the simple solution first. If there's appetite from the community for a more fancy solution, I'm happy to refine things. :)


Install Grin when building with Stack

When we run stack build and then stack install, can we include grin in the build and install processes?
This would simplify the installation process.

This is already fixed, I had a old checkout.


Remove ES6 lambda feature occurrences

Hello guys,
Currently, the code contains ES6 lambda usage. Because of this, while using a bundler (e.g. Webpack) it's required to use "babel" for this particular lib only. Usually it's not required, so it would be good not to expose "node_modules" to "babel" at all.
Since the app already uses anonymous functions, there should not be any problems with the proposed change.

I guess we should introduce a build process during npm publish for users that need ES5 compatibility. Change is fine for now. Thanks.


Reduce nestiness of PreCloudInitIso method

What this PR does / why we need it: Reduces complexity/nestiness of PreCloudInitIso method by reverting if conditions.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): None
Special notes for your reviewer: There are no logic changes - this is strictly refactoring MR.
Release note:

NONE

/test all

/retest

/cc

/remove-lifecycle stale

@rthallisey could you take a look?

@xpivarc could you take a look to approve?

/remove-lifecycle stale


Please make all session videos downloadable

currently the videos are not downloadable, only the transcript is

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

ID: 3fe47949-15a1-6073-0ac7-91a072ee4ff5
Version Independent ID: f670a787-e68f-1216-8194-1472510c4525
Content: Governance, risk, and compliance in Azure - Azure Architecture Center
Content Source: docs/framework/security/governance.md
Service: architecture-center
Sub-service: cloud-design-principles
GitHub Login: @PageWriter-MSFT
Microsoft Alias: pnp

Thanks for reporting - this issue is under review.  This is a Microsoft Internal DevOps Bug ID AB#195018

@knssub, I verified internally and based on how the video's are hosted it is not possible enable Video download.


pretrained checkpoints.

Hello, I want to use your code as a video-based skeleton extraction tool in my work. Could you provide the pretrained checkpoint?
Thanks!

Hello @xiaochehe , have you tried --checkpoint=tshufflenetv2k30?
That downloads the pretrained checkpoint and uses it.

@svenkreiss I will try. Thank you so much!


Video recording with sound

Hi,
First, thank you for this great piece of software, works out of the box like a charm.
I have a particular use case where a videoconference should be recorded and I need help figuring out if it's even posible to achieve.
As the title suggests, I need to record the selenoid browser session with sound.
Doing this on a server with no sound card might be tricky.
From my research the following steps should be taken:

enable on the host OS snd-dummy (https://superuser.com/questions/344760/how-to-create-a-dummy-sound-card-device-in-linux-server)

on the browser machine:
- install alsa-utils and define /etc/asound.conf
- install pulseaudio and add user running chrome to audio group (I don't know if this step is                       really required if my system does not have a sound card)

ffmpeg should be compiled with alsa support and should pick up the sound from the dummy sound card added

From all this I managed to do 1 by creating a custom docker image raduburla/chrome which works well, i think, chrome finds the sound devices and 2, the OS finds the virtual sound card.
With 3 I am unable to modify the image that records video, I might be missing some docker knowledge here.
The videos recorded by selenoid are still without sound.
I am missing something or what I am trying to achieve is not even possible?
Any help appreciated, thank you.

@raduburla I think that's possible but you are the first person requesting this.

We are using ffmpeg to record video - you could create your own video recorder container using by modifying our build files: https://github.com/aerokube/selenoid-images/tree/master/selenium/video Selenoid allows to override video recorder image by specifying -video-recorder-image flag.
You could build you own browser images using ours as a base (in FROM instruction) and then install required software for Chrome. Then you just replace our images by yours in browsers.json and reload Selenoi configuration (or restart its container). snd-dummy is also not enabled in our images, not sure how to do this with Docker - probably vie enabling it on the host machine.

Hi,
First, thank you for this great piece of software, works out of the box like a charm.
I have a particular use case where a videoconference should be recorded and I need help figuring out if it's even posible to achieve.
As the title suggests, I need to record the selenoid browser session with sound.
Doing this on a server with no sound card might be tricky.
From my research the following steps should be taken:

enable on the host OS snd-dummy (https://superuser.com/questions/344760/how-to-create-a-dummy-sound-card-device-in-linux-server)
on the browser machine:

install alsa-utils and define /etc/asound.conf
install pulseaudio and add user running chrome to audio group (I don't know if this step is                       really required if my system does not have a sound card)

ffmpeg should be compiled with alsa support and should pick up the sound from the dummy sound card added

From all this I managed to do 1 by creating a custom docker image raduburla/chrome which works well, i think, chrome finds the sound devices and 2, the OS finds the virtual sound card.
With 3 I am unable to modify the image that records video, I might be missing some docker knowledge here.
The videos recorded by selenoid are still without sound.
I am missing something or what I am trying to achieve is not even possible?
Any help appreciated, thank you.

Hello, any progress on this? I also interested in how to record a video with sound. As I understand it is pretty straight forward to add additional flags for ffmpeg in video-recorder container but I'm not sure how to record get an internal audio channel in docker container.

@web1991t unfortunately no progress here. If you have resources to experiment - we could provide  any reasonable help.


Android - Workspace - After error window, the system redirects you to the LHN page

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

Action Performed:

Open the App
Login with any account
Go to Settings->Workspaсe
Kill the internet connection
Open any options of workspase and click "Connect bank account"
Close the window about error

Expected Result:
The system should redirect you to the previous page
Actual Result:
The system redirects you to the LHN page
Workaround:
Unknown
Platform:

Where is this issue occurring?

Android

Version Number: v1.2.12-0
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/194419353-9bf487eb-bb93-47c5-ab7b-fd7d1403e441.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

Closing this because we're working on a big refactor to the navigation code. The x button is going to be removed.


add definitions for angular-wizard

added type definitions for Angular Wizard 0.4.2 (https://github.com/mgonto/angular-wizard)

angular-wizard/angular-wizard.d.ts
check list

[ ] is collect naming convention?

https://www.npmjs.com/package/angular-wizard
http://bower.io/search/?q=angular-wizard
others?

[X] has a test file? (angular-wizard/angular-wizard-tests.ts or others)
[ ] pass the Travic-CI test?

@mjurisic thanks mate!


aggregate function performance improvement for large number of series

When using the aggregate function on ~5000 series with a 3-level hierarchy it takes around 20 minutes to complete on my system. It might be a great enhancement to speed up this function (if possible), especially considering the time it would take to aggregate into the millions of series.

That is great news, thanks @kdgutier!


BUG:Contact us page is not responsive.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The contact us page is not responsive.
Assign this task to me and I can make this page completely responsive suitable for various devices.

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok, assigning this to you

Hi @hustlerZzZ  thank you for assigning me but this issue is already completed now.


Add documentation for how to run tests with a fixed datagen seed [skip ci]

Relates to #9441. This adds documentation noting that the data generation is not deterministic between runs and how to override the data generation seed to make it deterministic.

build

build


Mqtt heartbeat has small bug

Only on startup a heartbeat is published.
Original code:
    if (mqttClient.connected())
    {
        if (mqttenabled == 1)
        {
            if ((unsigned)now() >= nextbeat)
            {
                mqtt_publish_heartbeat(now());
            }
            nextbeat = (unsigned)now() + interval;
        }
    }

This code will extend the nexbeat every time the loop passes, so now() will never be greater than nextbeat
Move the line to the if:
    if (mqttClient.connected())
    {
        if (mqttenabled == 1)
        {
            if ((unsigned)now() >= nextbeat)
            {
                mqtt_publish_heartbeat(now());
                nextbeat = (unsigned)now() + interval;
            }
        }
    }

This is already fixed on the dev branch, thank you for the feedback.


Option to include transaction CBOR in GET_ACCOUNT_INFO and GET_TRANSACTION

The GET_ACCOUNT_INFO endpoint, when the details parameter is set to txs, provides a paginated list of account transactions. With the addition of transaction CBOR in events from SUBSCRIBE_ADDRESS, extending GET_ACCOUNT_INFO and GET_TRANSACTION to support CBOR data would complete the integration of tx CBOR across all relevant wslink messages

Ah, sorry, I did miss this in the original feature request. This is of course needed for any discovery.

Done with #270 and #271


change the cursor from a blink line to the filled in ascii character because it looks nicer

blinky*

i don't know if i can do that, i could see though
i need to find the code for that in the first place

cause the actual character is some fancy css trick i think if i remember correctly

Im trying to find where that original code is and I'm having trouble but I found this https://www.amitmerchant.com/simple-blinking-cursor-animation-using-css/#:~:text=The first and foremost thing,after pseudo-element like so.

so this is funny

There IS no code for the cursor. Because it's the default blinky cursor.

https://stackoverflow.com/questions/2897155/get-cursor-position-in-characters-within-a-text-input-field
could use this, make another element that is positioned in the same spot, and then yknow
but then that may cause issues when it comes to overflow when you have a really long command and it scrolls, but I'll figure it out I hope

update it's literally almost impossible :sob:


Create custom message for Arduino IDE

Hardware description: ESP32
Version or commit hash: Humble

I tried following some tutorials but couldn't generate the required header file for the custom message for Arduino IDE. What is the proper way to create custom messages ?
Regards

ok, i found the answer myself:

Create a ros2 package named "my_custom_message" containing the .msg file using the commands:
1.a. ros2 pkg create --build-type ament_cmake my_custom_message
1.b. cd my_custom_message
1.c. mkdir msg
1.d. gedit msg/MyCustomMessage.msg
1.e. Put the required fields into the message file. E.g.
bool bool_test
byte byte_test
char char_test
float32 float32_test
float64 double_test
int8 int8_test
uint8 uint8_test
int16 int16_test
uint16 uint16_test
int32 int32_test
uint32 uint32_test
int64 int64_test
uint64 uint64_test
1.f. In the autogenerated CMakeLists.txt file you should add the following lines just before ament_package():

...
find_package(rosidl_default_generators REQUIRED)
rosidl_generate_interfaces(${PROJECT_NAME}
"msg/MyCustomMessage.msg"
)
...
1.g. In the autogenerated package.xml file you should add the following lines:
...
<build_depend>rosidl_default_generators</build_depend>
<exec_depend>rosidl_default_runtime</exec_depend>
<member_of_group>rosidl_interface_packages</member_of_group>
...

Put the package into "~/Arduino/libraries/micro_ros_arduino-2.0.7-humble/extras/library_generation/extra_packages" folder
Build it using the commands:
3.a. docker pull microros/micro_ros_static_library_builder:humble
3.b. docker run -it --rm -v $(pwd):/project --env MICROROS_LIBRARY_FOLDER=extras microros/micro_ros_static_library_builder:humble
We can view the generated message in "~/Arduino/libraries/micro_ros_arduino-2.0.7-humble/src"
Now, we can use the message within Arduino IDE as follows:
#include <my_custom_message/msg/my_custom_message.h>


CSV Import: Run "Get Account" Service Warning

Implement a warning that displays the number of students who have to run the Get Account service.
Although a warning is displayed, the class list should successfully update the database with the class list upload.

https://github.com/ubccpsctech/classy/pull/11

Check for duplicates precludes this... As soon as an instructor has a classlist with more than one run-getacct as CWL, their classlist will be rejected.
I'm OK with that.

That is a really good point. If you need the CWL duplicate check to be taken out in the near future, we can do that with one line of code.

Closed. This ticket logic conflicts with other priorities: https://github.com/ubccpsctech/classy/pull/11#issuecomment-510679136


tests: Add better ERC20 test suite

Description
Working on getting a package together to test the most common contracts (ERC20, ERC721, etc.) to make sure we've covered our bases. Starting with just ERC20. Adds a new common-contracts package.

bump on this @smartcontracts


Error when translation engine is bing and text contains quotes

When the translation engine is bing and the text contains a quote the request will fail.
bash trans -b -engine bing :fr -no-ansi "Life's but a walking shadow, a poor player"
[ERROR] Invalid request
[ERROR] You must specify a JSON array of Translate requests in the POST body
Same request is successful with google/yandex
bash trans -b -engine google :fr -no-ansi "Life's but a walking shadow, a poor player"
, Un pauvre joueur de vie mais une ombre de marche

Thanks for the report -- should be fixed now.


Thread Safe Motor Controller Driver

Made motor controller thread safe. Removed redundant structs and added error functionality to driver. Created bitmap for motor error

why no build

It seems to not build properly on here, but does it build locally?

BSP_CAN.C was an incorrect file, it should be called BSP_CAN.c with a lowercase C. There are BSP changes here that are made redundant by the changes that are in SendCarCAN PR, and some assertion stuff was merged into master that's also in here.


Provide Dead-Letter Queue monitoring/tooling

As of 4.6.0, Axon Framework has a Dead-Letter Queue.
Being able to monitor activities of this DLQ through some form of a UI would be very helpful.
Without this, users are inclined to check out the storage solution directly for this.
Although workable, having something out of the box on this UI would be nice.
As an added thought, we could think about implementing a basic retry mechanism for failed sequences within the DLQ.
Although I don't see this a part of the MVP, it would be a nice addition afterwards.

@Morlack I tested axon-open-admin 0.2.1 today on one of my Axon apps, especially because of the DLQ features it provides.
On my dashboard I see an event processor having a DLQ with one entry, but when I click it, I get redirected from /axon-admin to /dlq/MyEventProcessor but I cannot observe any backend calls and the page remains empty. When I refresh the page, I get 404_NOT_FOUND.
I would love to use it for DLQ administration, but is this feature already supposed to be completed?

Hello @stefanzilske , it's supposed to be completed but there must be a bug then! I will look into it, thank you for reporting it

Hello again @stefanzilske! I have taken a look and it seems I forgot to make the DLQ-page aware of the multiservice feature I introduced in the same release.
Please try again using 0.2.3 and let me know if it works now :)

Hey, it works better but not without issues. I opened #37 for that specific one (and created a PR to fix it) 😊.

Thanks for reporting back! I will close this issue as resolved


Swap testing vs non-testing servlet response

Fixes #2.

LGTM - I'm going to wait to merge this 'until later this afternoon when I can test & deploy it.
@tmatsuo if you wish to do this, I'm ok w/ it.

Friendly ping on this PR

LGTM

Takashi - did you also release it to maven central?


Dockerize project

Add necessary files for deployment with Docker and Docker Compose, and the instructions in the project's README

I could not make it work, and also I am proposing a single container production-ready version on PR #18


[W5] Update speaker name

Hello,
We had a request from our speaker Philip Resnik to change his name on the list of prerecorded talks (from Phil to Philip). His name is correct on the schedule.
Cheers,
Christos

@raosudha89 I can do this update


File type for "save" and "open" dialog boxes

Hi, I want to suggest a feature for a next VSCode version.
The current version of the software doesn't allow to filter the files by type, when we pick up a file to open it. We also can't select its extension when we save it (we are obliged to add the extension suffix to the filename).
So, I think it is a good idea to add this two functionalities to VSCode.

We are not showing file types in the dialog because this area is majorly broken in Electron on Windows and has enough bugs that our current way of not showing types is just better.

Ah ! I didn't know it. Indeed, it is problematic.

https://github.com/Microsoft/vscode/issues/1379

Not the same actually...

Splitting into https://github.com/Microsoft/vscode/issues/12248 and this one


fixes #74 skip body validation if body parser is not wired in the req…

…uest chain

link to https://github.com/networknt/light-rest-4j/issues/74

Documentation about this change https://github.com/networknt/light-doc/issues/48

PR to update light-codegen templates https://github.com/networknt/light-codegen/pull/246


Syntax highlighting for ThingTalk!

Pull Request Test Coverage Report for Build 178

5 of 5 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 54.389%

Totals

Change from base Build 174:
0.02%

Covered Lines:
4178

Relevant Lines:
7116

💛  - Coveralls


encode queryParam's key to avoid 400 error when queryParam is like ?p…

encode queryParam's key to avoid 400 error when queryParam is like ?params[name]=123&params[sex]=male

Make sure that:

[*] You have read the contribution guidelines.
[ ] You submit test cases (unit or integration tests) that back your changes.
[*] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #3334 (a35b6a5) into master (378283b) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3334      +/-   ##
============================================
- Coverage     65.40%   65.29%   -0.12%
+ Complexity     5204     5195       -9
============================================
  Files           746      746
  Lines         21020    21019       -1
  Branches       1919     1919
============================================
- Hits          13749    13725      -24
- Misses         5992     6013      +21
- Partials       1279     1281       +2

Impacted Files
Coverage Δ

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
63.82% <0.00%> (-8.52%)
:arrow_down:

.../plugin/grpc/loadbalance/AbstractLoadBalancer.java
70.65% <0.00%> (-7.61%)
:arrow_down:

.../logging/rocketmq/utils/LogCollectConfigUtils.java
71.11% <0.00%> (-4.45%)
:arrow_down:

...ng/rocketmq/rocketmq/RocketMQLogCollectClient.java
72.00% <0.00%> (-4.00%)
:arrow_down:

.../org/apache/shenyu/plugin/divide/DividePlugin.java
38.63% <0.00%> (-3.23%)
:arrow_down:

...henyu/admin/service/impl/UpstreamCheckService.java
70.14% <0.00%> (-1.50%)
:arrow_down:

...common/dto/convert/rule/impl/DivideRuleHandle.java
75.75% <0.00%> (-1.39%)
:arrow_down:

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 378283b...a35b6a5. Read the comment docs.

@vijayuan thanks for you contribution~


RxJS 7 missing as internal dependency

Hello, first of all, thank you so much for updating this library, it is so important to me!
One issue I stumbled upon: It assumes RxJS v7 to be present, as it uses lastValueFrom(). My project is running on RxJS 6.6 . I quickly resolved the issue for now by jumping into node_modules/angular2-smart-table, installing rxjs7 under an alias via  $ pnpm add rxjs7@npm:rxjs@7, and putting the missing function into fesm2022/angular2-smart-table.mjs.map via import { lastValueFrom } from 'rxjs7'
As a temporary fix, my app finally was building without errors, but unfortunately, the tables are not displayed yet. Is there a migration guide for people coming from the latest original ng2-smart-table? 🙏 The changelogs alone are not really telling me much, to be honest. I will try out this example I found though: https://stackblitz.com/edit/ngx-formly-ui-bootstrap-8whpda?file=src%2Fapp%2Fapp.component.ts
EDIT: Everything displaying for me, now. I forgot to change the template tags from <ng2-smart-table> to <angular2-smart-table>
Thank you very much for your help !


Add NewDot image preflight request caching

Problem
https://github.com/Expensify/App/pull/36560#issuecomment-1948219686
I believe I've identified the issue—the preflight response isn't being cached, resulting in its repeated execution with each image render. While the images themselves are cached, the browser is forced to reinitiate the preflight check due to the absence of caching headers. These headers are crucial as they instruct the browser on whether and how long to cache the preflight request.

a preflight request is being made even though the actual resource is cached

Consequently, in offline scenarios, the browser is unable to perform the preflight request, leading to a failure in loading all images.
Here are some visuals to illustrate the point:

Image caching example (Notice how the GET response for the image is cached):

Preflight request missing caching headers:

Solution
To fix the issue in offline usage, and omit doing preflight request for cached items we should add some cache to the preflight response as well (this is a backend task)
cc @trjExpensify since you're helping coordinate this effort as BugZero
cc @kidroca since the above P / S was written by you 🙏
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01d9de82c0447606ee
Upwork Job ID:<PHONE_NUMBER>561478144
Last Price Increase: 2024-02-28

I'll self assign to BZ this one as well.

PR is up for review.


Update README.md to use NuGet badge

Also fixing the casing of .NET

I see, you already have badges but in the bottom. Usually projects have them in the top. Would like to move them?

See f4c2425e43a5fc3f59ee7eeb18e7f61a9fd770e9
I prefer it below the title. Also, I prefer .Net over .NET 😉
Thanks anyway! 👍

You're welcome 🖖. Officially it's .NET, by the way ;)


Addition of a filter while selecting countries

What

User has to scroll down the list to select the desired country at first page after installation.

Solution
A search bar should be implemented , so that user can easily select the country from the given list.

Part of

#561

I really like the idea @bhattabhi013 .
One thing to note is that when the app is installed on a real device in most cases the country should be picked automatically so it isn't as bad as on the emulator.

When we implement the search feature we probably have to switch the source of the countries, just found this:
"World countries available in multiple languages, in CSV, JSON, PHP, SQL and XML formats, with associated alpha-2, alpha-3, and numeric codes as defined by the ISO 3166 standard, and with national flags included" https://stefangabos.github.io/world_countries/

Hi @bhattabhi013!
As @M123-dev pointed out the country is automatically detected (something like WidgetsBinding.instance?.window.locale.countryCode), and the notion of "changing countries" after onboarding is probably limited to power users.
Feel free to code this, but I wouldn't mind if you could find or create an issue with a higher priority.
@M123-dev I don't think either we need the world countries in different languages and with national flags.
The solution @jasmeet0817 found was to use OS data via iso_countries, we even had to upgrade our minimum Android version to 21 for that. I'm not in favor of additional country data that will need to be maintained and will take space - especially for a minor feature like country selector.

@bhattabhi013  are you working on this, just did something similar like this,
mind if I work on this one ?

Hi @AshAman999,
I'm working on some other issues and this issue is free to take.
Before working on anything, I will suggest confirming with mentors for country codes and their corresponding countries.

@teolemon and @bhattabhi013
Is this good to go with??
The font and the icon I will try to keep as the old one was trying to implement here,
And if could be assigned to the issue so that no one else wastes their efforts on the same task
https://user-images.githubusercontent.com/57723319/166095139-988ee283-cd76-4e60-bfa9-5c83bfe2a2a9.mp4

This looks good @AshAman999, I had something like this in my mind. Let's wait for other's opinions/ suggestions on it.

Heyy @AshAman999 looks great 💯

Great, will raise a pr soon 🍀
On Sun, 1 May 2022, 13:44 Marvin M, @.***> wrote:

Heyy @AshAman999 https://github.com/AshAman999 looks great 💯
—
Reply to this email directly, view it on GitHub
https://github.com/openfoodfacts/smooth-app/issues/1200#issuecomment-1114167600,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANYMTN3XY6VEBUABHPTXMZDVHY4MVANCNFSM5QSAUNTA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@M123-dev , did the changes as requested, looks like this for now


[dx12] allow view creation to fail

Sibling of #2922

The sibling got reviewed already, merging.
bors r=msiglreith


OpenXmlPowerTools.Body Constructor remove escaped characters

I'm opening a .dotx file and load it into a MemoryStream:
MemoryStream memoryStream = null; //this will hold our in memory Word doc

            using(Stream templateStream = File.Open(templatePath, FileMode.Open, FileAccess.Read))
            {
                memoryStream = new MemoryStream((int)templateStream.Length);
                templateStream.CopyTo(memoryStream);
                templateStream.Flush();
                memoryStream.Position = 0L; //reset position for reading
            }

After that I open the document as WordprocessingDocument and I start doing replacements looking for Merge Fields. I do the replacements in xmlDocXElement (see bellow)

 using (WordprocessingDocument docx = WordprocessingDocument.Open(memoryStream, true))
            {
                var mergeFields = docx.MainDocumentPart.Document.Body;

                XElement xmlDocXElement = XElement.Parse(docx.MainDocumentPart.Document.Body.OuterXml);

                    var dictionary = (IDictionary<string, object>)data;

                foreach (KeyValuePair<string,object> record in dictionary)
                {
                    ... //Replacement logic
                }
//Bug is about to happen
                docx.MainDocumentPart.Document.Body = new Body(xmlDocXElement.ToString());
//Bug happened
            }

Before the bug happens all the Special characters are encoded fine, as I use XElement.Parse before including them in the document. But for some reason after the Body() constructor some of the especial character like & are returned to their normal form?
& --> &
I tried adding & manually during the replacement process and all of them are replaced OK. I found only one Stack Overflow user with the same issue Go To Thread. It's a bit weird as it doesn't happen all the time so I can't really tell you what to do in order to simulate it.
Here is the row content before Body():
<w:r>
            <w:rPr>
              <w:rFonts w:cs="Arial" />
              <w:noProof />
              <w:szCs w:val="20" />
            </w:rPr>
            <w:t xmlns:w="http://schemas.openxmlformats.org/wordprocessingml/2006/main">Warwickshire Golf & Country Club North Woodloes,<w:br />Leek Wootton,<w:br />Warwick,<w:br />Warwickshire,<w:br />CV35 7QT</w:t>
          </w:r>

Here is the row content after Body():
<w:r>
                        <w:rPr>
                            <w:rFonts w:cs="Arial" />
                            <w:noProof />
                            <w:szCs w:val="20" />
                        </w:rPr>
                        <w:t xmlns:w="http://schemas.openxmlformats.org/wordprocessingml/2006/main">Warwickshire Golf & Country Club North Woodloes,<w:br />Leek Wootton,<w:br />Warwick,<w:br />Warwickshire,<w:br />CV35 7QT</w:t>
                    </w:r>

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


Change value of CompetencyJol to int

Is your feature request related to a problem?
#8598 introduced the CompetencyJol with Integer as data type for the stored value. But the value should always exist, if we only create the CompetencyJol when actually needed.
Describe the solution you'd like
Change the data type to int and only create the objects when they are actually needed.
Describe alternatives you've considered
Don't change it and accept all the null checks...
Additional context
No response

Can we simply do it in the corresponding branch?

We changed it to short in #8689


Implement Swing-Filter, and Slide-Filter

Implement Swing-Filter, and Slide-Filter linear approximation methods from the paper https://dl.acm.org/doi/abs/10.14778/1687627.1687645.

The swing filter method is already implemented and merged in the main branch.

Consider looking into if the internals of decompressSwing() and decompressSlide() overlap enough that it can be moved to a generic function called by decompressSwing() and decompressSlide() as discussed here.


servo center adjustment

It would be nice to be able to tweak the servo center slightly at runtime. Sometimes it's hard to center servos perfectly, especially field replacements.
It's not clear to me what the best solution is. We could modify the active mixer (mavlink ftp), new mavlink cmd, use params, etc. I think a simple param for each servo channel would get the job done, but correcting the mixer might make more sense?

Very good idea!

I've run into this issue so many times. Definitely a great idea!

I vote for exposing the mixer parameters to allow setting center and endpoints to provide the same functionality that one normally has with an RC transmitter.
What is "mavlink ftp"? Would you copy a new mixer into RAM, then invoke "mixer load"? That sounds pretty awkward to implement, since you'd have to edit a mixer file somewhere first.

The other piece of the mavlink ftp idea is a simple editor in QGC and one
button to transfer and load the new mixer.
On Mon, May 30, 2016 at 8:38 AM, Mark Whitehorn<EMAIL_ADDRESS>wrote:

I vote for exposing the mixer parameters to allow setting center and
endpoints to provide the same functionality that one normally has with
an RC transmitter.
What is "mavlink ftp"? Would you copy a new mixer into RAM, then invoke
"mixer load"? That sounds pretty awkward to implement, since you'd have to
edit a mixer file somewhere first.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/issues/4650#issuecomment-222483699, or mute
the thread
https://github.com/notifications/unsubscribe/AAFK6AKEqr9xKfkyeU2iT2hf_MnKG79Yks5qGtoxgaJpZM4In7PR
.

Yet another idea - onboard calibration.
With the basic mixer in place you could be walked through each output and asked to hold center and then each extreme.

It appears that the servo center point is implicitly the midpoint between pwm_min and pwm_max:
https://github.com/PX4/Firmware/blob/master/src/modules/systemlib/pwm_limit/pwm_limit.c#L181
That would make the center point dependent on the endpoints.

I think we should introduce parameters (like the reverse params) for this purpose. Best would be to use the mixer, but we should take one step at a time.

Once we have that in place we could add a simple setup screen in QGC using the metadata from the airframe.
https://github.com/PX4/Firmware/blob/master/ROMFS/px4fmu_common/init.d/2101_fw_AERT#L7

Let me handle that here. Your time is way better spent with CI. I know what I have to do here.

Sounds good to me!
Thanks Lorenz
On Mon, May 30, 2016 at 4:13 PM, Lorenz Meier<EMAIL_ADDRESS>wrote:

Let me handle that here. Your time is way better spent with CI. I know
what I have to do here.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PX4/Firmware/issues/4650#issuecomment-222549149, or mute
the thread
https://github.com/notifications/unsubscribe/AAFK6IOt8JDzqp-5UuFvwvZBbDFug4olks5qG0ThgaJpZM4In7PR
.

@bartosz-wawrzacz implementation here - https://github.com/PX4/Firmware/issues/4650

https://github.com/PX4/Firmware/pull/5720


the error in change  RPN model:conv5_3  to conv4_3

when I directly delete pool4 and conv5 layers, the error show as follows:

then, in lib/nets/network.py ,I modified feat_stride = 16 to feat_stride = 8
the train is run,but loss=nan ,  error as follows:

Do you have any solution for this error,thanks,I want to change RPN network for my own. @ShijunK

you may have to change a couple of other things as well. the current code supports feat stride 16. it also computes RoI with 1/16 and so on.

Thanks to your answer, but I can't find  where to modify ROI Steps. Could you tell me where to modify ROI to 1/8?  Thanks again! @endernewton

I modified self._feat_stride and  lr to solve this problem. It seems that it can work. but at the beginning, show a log as follows:

what's the meaning of it ?

Having the same problem here, and weirdly enough, at times it gives NaN, at times no. I also changed line 248 in config.py to
__C.DEDUP_BOXES = 1. / 16.

I have solved this problem. I change the learing rate of  lib/model/config.py  and  modify the lib/net/network feat_stride and if you change you input image size,you should also change the base_size of the lib/layer_uitls/generate_anchor.py. After these, it  worked. I hope these tips can help you.

So, as far as I can understand you changed:
self._feat_stride and self._feat_compress from 16 to 8 (in network.py)
base_size from 16 to 8 in generate_anchors.py
DEDUP_BOXES from 16 to 8 in config.py
lower learning rate in config.py
I have done the same too, but still getting NaNs. I am using some other dataset and pretrained net, so the problem might be somewhere else.
Thanks!


Add druntime Timer

This is an attempt to create an OS independent timer API in druntime. Nothing is set in stone with the PR and it is open for modifications in order make it better.
On Linux/Posix, the usual timer_create/timer_settime is used. I opted for an internal timer queue so far but it is also possible to create a new time timer_create and SIGEV_THREAD. Even if it is supposed to per process timer, the C library usually allows 1000 or so timers to be created.
On Windows, the only timer I could find with ok accuracy is CreateWaitableTimerEx with the CREATE_WAITABLE_TIMER_HIGH_RESOLUTION flag. TimeBeginPeriod(1) didn't work on my system and I'm not that keen on a system wide change. CREATE_WAITABLE_TIMER_HIGH_RESOLUTION is pretty recent addition and I don't know what happens when you try this out on an earlier Windows like Windows 7.
Another thing I'm not sure satisfied is how you must expose all the source files when you compile your program with druntime. Is there a better way to limit what the program can "see".
Unfortunately I'm not able to implement the timer for MacOS even if I would like to. I have left the MacOS implementation in a unimplemented stub status. If anyone wants to implement, it would be great.

What would you use this for?

For programs that needs a timer so that they don't need to reimplement a timer each timer. This is similar to C# threading.timer.

timer_create is just kinda terrible and i wouldn't use it for much of anything irl.

timer_create is just kinda terrible and i wouldn't use it for much of anything irl.

POSIX timer in Linux isn't that bad when it comes to accuracy. I'm not sure if there is any better.

Accuracy is a bad thing in a lot of applications, that's why Windows has the options it has. It'd be fairly important for an api to expose this requirement somehow too.
But I'm more thinking about using it. The posix thing calls a signal handler which severely limits its utility. The other option to call it from another thread is similarly problematic.
I'd much rather bless an event loop.

Accuracy is a bad thing in a lot of applications, that's why Windows has the options it has. It'd be fairly important for an api to expose this requirement somehow too.

I think you are referring to controlling the accuracy of the timers. Some timer OS interfaces has an accuracy parameters as well, this in order to be able to merge timeouts so that waking up the system from power save can be done more seldom. It is possible that such an accuracy parameter can be added to the timer interface. What the accuracy parameter really mean must also be defined.

I'd much rather bless an event loop.

I don't know what this event loop is. I have to be presented with a design in order to understand what it is supposed to do.


Fix gradle properties

Add wiremock_version and testcontainers_keycloak_version to gradle.properties when authenticationType is oauth2
Addresses the Could not get unknown property 'wiremock_version' issue reported by @mraible in https://github.com/jhipster/generator-jhipster-quarkus/issues/143#issue-761388190

@bhadreshdesai Thanks for your contribution!

Closing and reopening to restart CI.


Added LDAP-LTB, LDAP-PHPADMIN and updated version of ADOP-NGINX to 0.2.0

Added:

LDAP-LTB https://github.com/Accenture/adop-ldap-ltb
LDAP-PHPADMIN https://github.com/Accenture/adop-ldap-phpadmin

Updated:
ADOP-NGINX docker image version to 0.2.0 with new release notes page.

LGTM!


Docs fix

Fixes the current docs on rtd (you can check that it is actually working here).
@nikfilippas please review, but remember we want to keep any changes here as minimal as possible. Polishing the docs or including any API improvements is for later.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.004%) to 97.306%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
5815

Relevant Lines:
5976

💛  - Coveralls

Note that we also want to have docs for v2.7 (before official API switch), which were also broken. I've fixed them now in https://github.com/LSSTDESC/CCL/tree/v2.7.1, and pointed rtd's latest to it, so this is what it shows as latest until we tag the version in the current master. At that point we'll switch it back to master.

@damonge is this why I can't click to see the rest of the builds on RTD? The button on the bottom left is not clickable any more.

Okay, assuming this is a hotfix branch which really only exists for people to see some docs, overall it's fine. I browsed through them and there are still issues (formulas not showing correctly etc) but let's live with that for 2.7.1 for now, it's okay I guess.
One thing I'd like to point out is the following: because we restructured CCL there are so many new *.rst files. At the same time, these can all be auto-built using make html both locally, and on RTD. So, instead of polluting the repo storage & history with these files, why don't we remove them, add readthedocs/api to .gitignore, and then, simply call make html on RTD after the requirements have been installed.
There are some other issues with the RTD build as well, but we'll go through them on the Docs v3 PR.
For example, these two lines, install stuff that clash with one another, in 2 different ways. It's okay though, since the docs build, let's leave it like this for now, and we'll get back to it for v3.

The current build commands on RTD are convoluted and the docs will soon break once again if these are not addressed.
So, overall, what I propose is to just remove the *.rst files, update .gitignore, and replace this command
 python -m sphinx -T -E -b html -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html

with make html, which is why the makefile is there in the first place.

Also, help me understand your Slack announcement, what is the difference between the docs here and the docs on v2.7.1?

I don't know what you mean by
Replace this command
 python -m sphinx -T -E -b html -d _build/doctrees -D language=en . $READTHEDOCS_OUTPUT/html

with make html, which is why the makefile is there in the first place.

Also, help me understand your Slack announcement, what is the difference between the docs here and the docs on v2.7.1?

The docs on master show the new API. The docs on v2.7.1 show the docs of the currently released v2.7, which uses the v2 API. The docs from master are not useful, as most people install the latest release from pip/conda-forge.

The docs on v2.7.1 show the docs of the currently released v2.7, which uses the v2 API.

OK, but shouldn't there be a separate PR for those? Am I missing something?

I don't know what you mean by ...

There is a makefile in readthedocs/Makefile which automatically writes the *.rst files using sphinx-apidoc, and then builds the documentation using sphinx-build.
This allows us to remove the *.rst files from the repo completely, so that we don't rely on the developer manually rebuilding the *.rst files in every new PR we merge. This can all happen automatically on RTD.

I know all that, but I cannot touch the build commands on rtd. And as I said at the beginning, we don't need this. Repos store rst files routinely.

2.7.1: i can't make a PR to rewrite the commit history. This is just there for now so we have useful docs until we make the next release

I cannot touch the build commands on rtd

Right, I checked and we can in fact customize the commands, but I agree - let's not go this far.

This is just there for now so we have useful docs

I see! That's fine.

I'm merging it and I'll check if it pulls the ones from master now.

yes! finally they're back. Now we can point latest to master.

Ok, great. But no, I'll do that when we've released the last v2 version.


I can't load the chat screen, only this

BingGPT-0.3.6-win32-x64
OS：Windows 10  21H2

return to normal


[GUI Editor] Adjustable Canvas Size and Bug Fixes

The GUI Canvas can now be adjusted in real time.
Various bug fixes and improvements including:
-Allowing the specification of image source.
-Selection bugs with text
-Outlines over stack panel
https://github.com/BabylonJS/Babylon.js/issues/9870
https://github.com/BabylonJS/Babylon.js/issues/9819

just a lint issue


[Doc] Add list of winning solutions in data science competitions using XGBoost

Rerate #6173
Could you please have a moment to take a look at it?

Thanks!