Remove unnecessary curation fields from .ttl file

I'm seeing what look like internal curation fields in the records display on Blackfynn.  In addition to the one I reported yesterday, I see "curation group" in the summary.  Please do a review of all internal fields and comment them out or do whatever is necessary to have them not display.

These properties are marked in sparc-methods.ttl via the ilxtr:curationInternal predicate. Closed via https://github.com/SciCrunch/NIF-Ontology/commit/1b0bb2e1adb401216acb2941e4b2ccd79a608386.


Documented the docstrings according to numpy style guide

Description
Documented the docstrings of functions according to numpy style guide
closes #36
Affected Dependencies
How has this been tested?

docstr-coverage
flake8

GIF Before

NA

GIF After

NA

Checklist

[x] I have followed the Contribution Guidelines.
[x] My changes do not edit/add any unrequired files.
[x] My changes are covered by tests.

@noviicee There seems to be a lot of unecessary edits due to styling, please only edit the code wherever needed only. We need docstrings only for functions and nothing else, check this for reference on how docstrings should be and read the issue description once.
You can revert the commit and make changes and push.

I have just edited the function docstrings and made the code according to pep8 style.
Do you want me to remove the pep8 styling?

@noviicee There seems to be a lot of unecessary edits due to styling, please only edit the code wherever needed only. We need docstrings only for functions and nothing else, check this for reference on how docstrings should be and read the issue description once.
You can revert the commit and make changes and push.

I have just edited the function docstrings and made the code according to pep8 style.
Do you want me to remove the pep8 styling?

Yes, see the example I linked above.

@noviicee There seems to be a lot of unecessary edits due to styling, please only edit the code wherever needed only. We need docstrings only for functions and nothing else, check this for reference on how docstrings should be and read the issue description once.
You can revert the commit and make changes and push.

I have just edited the function docstrings and made the code according to pep8 style.
Do you want me to remove the pep8 styling?

Yes, see the example I linked above.

Okay I'll remove the styling..btw the link forwards to the same page we are on now 😅

@noviicee There seems to be a lot of unecessary edits due to styling, please only edit the code wherever needed only. We need docstrings only for functions and nothing else, check this for reference on how docstrings should be and read the issue description once.
You can revert the commit and make changes and push.

I have just edited the function docstrings and made the code according to pep8 style.
Do you want me to remove the pep8 styling?

Yes, see the example I linked above.

Okay I'll remove the styling..btw the link forwards to the same page we are on now 😅

Oops sorry, just updated it

@noviicee There seems to be a lot of unecessary edits due to styling, please only edit the code wherever needed only. We need docstrings only for functions and nothing else, check this for reference on how docstrings should be and read the issue description once.
You can revert the commit and make changes and push.

I have just edited the function docstrings and made the code according to pep8 style.
Do you want me to remove the pep8 styling?

Yes, see the example I linked above.

Okay I'll remove the styling..btw the link forwards to the same page we are on now 😅

Oops sorry, just updated it

Thanks!

@Praful932 some functions do not have the docstrings written already, should I leave the description empty  in those cases and only edit the parameters and returns..?

@Praful932 some functions do not have the docstrings written already, should I leave the description empty in those cases and only edit the parameters and returns..?

You have to fill in that too, since that's the purpose of this issue

@Praful932 some functions do not have the docstrings written already, should I leave the description empty in those cases and only edit the parameters and returns..?

You have to fill in that too, since that's the purpose of this issue

Okay

@noviicee if your facing issues you can reach out to us on discord.

Good going, still needs some work
checks are failing :/
you have extra spaces and empty description of parameters

Yup. Will remove the extra whitespaces and add description of the parameters. Though I hope there isn't a problem with a test output statement line too long

When you have resolved the comments, you can notify us by clicking on request review
Thanks

Great Work!

Thanks a lot!


Table widget

Opinion: TableState should have ways to delete a certain row or even, all rows.
I am testing the Table widget to implement a file queue in my app.
Basically, I have a HashMap of files, and in TableListener::on_update() I'm simply updating the table to list all files in that map. Naturally, this map changes over time (and over user actions), but there is no way to delete row that correspond to files dropped from the map.
Adding TableState::remove_row(index) (and, optionally, TableState::remove_all_rows()) would suffice.

Additional discussion point
Currently, methods that modify the table headers / rows does not validate if the input matches with each others' column count. For example, make a table of 5 rows and 2 columns, then suddenly set the header to have just one column.

Yes, that is something I have seen, but I am not sure of how it should be handled here... Do you have an idea ?

Since the table header is what "defines" (or describes?) the data of a table, matching each row's column count to header's column count seems logical in this case?
How about:

Manipulating headers upon building a Table widget via Table::set_headers
=> stay as-is
Manipulating headers in the middle via TableState::set_headers
=> empty entire row set as there is no guarantee those rows' column count matches with new headers
Manipulating rows in any case
=>validate if each row contains exact number of columns to header's
(Might need to change method signatures to return Result)

For now, these are what floats in my mind.

If you want to try it feel free to do it and update your PR ;)

During my attempt to implement the above logic, I realized that it's not an easy job to decide among these two possible ways:

Make set_rows and add_row to return Result<(), TableError>:

Ok(()) when everything is sane
Err(TableError::HeaderRowColumnCountMismatch) when there is a column count discrepancy (i.e., header has n columns but the row has m columns)
Err(TableError::WithinRowsColumnCountMismatch) when rows have different column counts from each other (i.e., row x has n columns while row y has m columns)

For each case of a row having less or more columns than the header or other rows, do:

fill with an empty string literal ("") if this row has less columns
Ignore excess columns if this row has more columns

Which one do you think the best?

I think that the best way would be to let the app dev to choose what to do when there is a discrepancy, thus the first one is, in my opinion, the best way. And thanks for trying solving this issue, it is amazing ! :D


'Fees management' left side menu option hidden when using Metamask

Description:

Related issue(s):
Fixes #
Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Tested! :white_check_mark:


community-operators openshift-qiskit-operator (0.2.0)

Thanks submitting your Operator. Please check below list before you create your Pull Request.
New Submissions

[ ] Does your operator have nested directory structure?
[ ] Have you selected the Project Community Operator Submissions in your PR on the right-hand menu bar?
[ ] Are you familiar with our contribution guidelines?
[ ] Have you packaged and deployed your Operator for Operator Framework?

[ ] Have you tested your Operator in all supported installation modes?
[ ] Have you considered whether you want use semantic versioning order?
[x ] Is your submission signed?
[ x] Is operator icon set?

Updates to existing Operators

[x ] Did you create a ci.yaml file according to the update instructions?
[ x] Is your new CSV pointing to the previous version with the replaces property if you chose replaces-mode via the updateGraph property in ci.yaml?
[ x] Is your new CSV referenced in the appropriate channel defined in the package.yaml or annotations.yaml ?
[ ] Have you tested an update to your Operator when deployed via OLM?
[ ] Is your submission signed?

Your submission should not

[ x] Modify more than one operator
[ x] Modify an Operator you don't own
[ x] Rename an operator - please remove and add with a different name instead
[ x] Submit operators to both upstream-community-operators and community-operators at once
[ x] Modify any files outside the above mentioned folders
[ x] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[x ] Description about the managed Application and where to find more information
[ x] Features and capabilities of your Operator and how to use it
[ x] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[ ] Human readable name and 1-liner description about your Operator
[ ] Valid category name1
[ ] One of the pre-defined capability levels2
[ ] Links to the maintainer, source code and documentation
[ ] Example templates for all Custom Resource Definitions intended to be used
[ ] A quadratic logo

Remember that you can preview your CSV here.
--
1 If you feel your Operator does not fit any of the pre-defined categories, file an issue against this repo and explain your need
2 For more information see here

@blublinsky i see that you are running in replaces-mode and version 0.2.0 is not replacing 0.1.0. Please add replaces tag in cvs.
replaces: openshift-qiskit-operator.v0.1.0

/test deploy-operator-on-openshift

/test deploy-operator-on-openshift

@mvalarh, I did updates, can you, please, kick off test again

@mvalarh, I think its getting better, but there is still 1 error, which I do not really understand

/test all

/retest

@blublinsky there is problem with parsing of csv file, I see that mediatype: "image/png"  is aligned incorrectly

/test all

/test all

@blublinsky  metadata.annotations.containerImage value is not used in any container, please check.

/test all

/merge possible

@husky-parul Since you are contributor to this operator. Can you check at these changes if ok with you? You can also add you and others to reviewers list in ci.yaml. More info here

@mvalahtv @J0zi LGTM please merge

/test deploy-operator-on-openshift

/hold

/hold cancel

/test all

/merge possible

/retest

/test all

@J0zi , Change done, can you, please, approve and merge

/merge possible


occasional SyntaxError: Unexpected end of input, when creating pdf stream from string

Hi, I'm getting strange errors, occasionally, mostly on first tries (I'm doing retries of if it fails), my code:
return () => new Promise((res, rej) => {
        let html = `${htmlStart}${htmlBody}${htmlEnd}`;

        pdf.create(html).toStream(function (err, stream) {
            if (err) {
                rej(new Error("Fail converting html to pdf stream. html:" + htmlBody + " Error:" + err ));
            }

            res(stream);
        });
    });

html='<!DOCTYPE HTML>A new-'
throws SyntaxError: Unexpected end of input.

I'm returning a function that when called returns the promise (sometimes I won't need the function to be called)
using:
html-pdf: 1.5.0,
phantomjs 1.9.8

Having exactly same issue on Ubuntu, it looks it's related to this line -> https://github.com/marcbachmann/node-html-pdf/blob/master/lib/pdf.js#L128
When looking one line up, it's clear that data may turn as undefined (void 0), and making JSON.parse(undefined) throws SyntaxError: Unexpected end of input

@medikoo so what do you think? Why isn't there an exit code or error? That shouldn't happen.

@marcbachmann I'm not that familiar with internals of this module.
Still I've "fixed" it for my case by replacing:
data = JSON.parse(data);

with
data = data ? JSON.parse(data) : {};

Still it means that for some cases what goes into callback (which is few lines below) is in form of { filename: ... } and in some other (those which raised that error) it is {}. I can't say if it's ok or not.

@marcbachmann small correction, Buffer.concat(stdout).toString() resolves either to empty string, or plain whitespace. thendata when assigned with buf.trim() resolves to '', and its JSON.parse('') that throws SyntaxError: Unexpected end of input.
Anyway I still refer to version v1.5 (I have not yet tried v2.0)

Yes, doesn't matter. It's the same problem, the input is invalid.
Thanks for investigating.
On the render callback of the page inside phantom, the script doesn't check for an error code.
Maybe the page rendering fails somehow.

@marcbachmann good point, I take a look into that, and get back if I have some more info

@marcbachmann anyway what's in a code, is clearly a bug. There should be no JSON.parse called when we know that data is empty string. If it's indeed a result of some error, throwing some more meaningful message (instead of doing JSON.parse that can't work) would help.

Thanks. It's that one. https://github.com/marcbachmann/node-html-pdf/blob/master/lib/scripts/pdf_a4_portrait.js#L75

Hi! This error occurs because the event 'exit' using in the library. This event happens earlier than: child.stdout.on('data'), so in child.on('exit') variable stdout will be empty array and JSON.parse will fall.
I changed the use of the event "exit" to "close" and this error is not repeated.

I have the same issue, any fix for this?

I think the issue still persists because of the following line of code:
https://github.com/marcbachmann/node-html-pdf/blob/master/lib/scripts/pdf_a4_portrait.js#L70
It's totally logical to call exit after successfully rendering the PDF. It's also makes sense that a successful exit doesn't have any data to return to the parent process. This is where the problem lies because the parent process considers any response without data from the child process as an error (https://github.com/marcbachmann/node-html-pdf/blob/master/lib/pdf.js#L120), regardless if there were actually any errors thrown from the child process or whatever the value of the exit code is.
Here is a pull request from my fork:
https://github.com/marcbachmann/node-html-pdf/pull/418
I included comments there as to why I wrote it like that. Kindly take a look and lemme know if I missed anything.
Thanks.


Configuration checker window: first run does not display header image

Describe the bug
When using the new configuration checker window (#6286), the first time it displays (generally on package import) the header image does not appear.
To reproduce
Steps to reproduce the behavior:

Load the development branch repository
Create an asset package containing:

MixedRealityToolkit
MixedRealityToolkit.Services
MixedRealityToolkit.Providers
MixedRealityToolkit.SDK

Create a new Unity project
Import the package created in step 2

Expected behavior
Dialog to be fully populated, including the header image
Screenshots
If applicable, add screenshots to help explain your problem.

Your Setup (please complete the following information)

Unity 2018.4.11f1
MRTK development branch

This issue comes up when importing the foundation package, but the MRTK Project Configurator window also pops up twice.

This appears to be related to when Unity runs the configuration dialog and when the package import unpacks the image.

No longer seeing this issue.  Tested on 2.3.0 using Unity 2018.4.17f1


Remove use of feature(static_nobundle)

It's used in "rustc-dep-of-std" mode and prevents removal of this feature from rustc.
(No need to publish a new libc version right now, the rustc PR is still waiting on some other work being finished.)

r? @Amanieu
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit 7683ea2832373e71b989eb9a93855ca20bb2fae2 has been approved by Amanieu

:hourglass: Testing commit 7683ea2832373e71b989eb9a93855ca20bb2fae2 with merge<PHONE_NUMBER>b6a74c8bcf9cb407864e374143e6aa...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-11, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13
Approved by: Amanieu
Pushing<PHONE_NUMBER>b6a74c8bcf9cb407864e374143e6aa to master...


Tyleryasaka/add production issuer

Checklist:

[x] Code contains relevant tests for the problem you are solving
[x] Ensure all new and existing tests pass
[x] Update any relevant READMEs and docs
[x] Submit to the develop branch instead of master

Description:
Going forward, we have 3 separate issuers we can add to our ERC723/735 Origin Identity. One which works with our test signing key, one for our dev signing key, and one for our production signing key. The production issuer is added on test nets, the others are added locally and during tests.

I'm not a truffle ninja (yet), but I believe we can just run this as an additional migration without having to re-deploy everything.

Talked to @wanderingstan on Discord and we got the package-lock situation straightened out.
We do want to update the version here - all I'm doing is syncing the package-lock with the package. Those should always match.

This comment looks like what we need to do:
https://github.com/trufflesuite/truffle-migrate/issues/23#issuecomment-374645578

The results. @tyleryasaka does this look good to you, and can you test?
Rinkeby
~/Origin/origin-js/contracts$ npx truffle migrate --network rinkeby -f 4 --to 4
Using network 'rinkeby'.

Running migration: 4_add_sample_issuers.js
  Running step...
  ... 0xa4ba46b508ff19c181c4594f8fd1a1e073b7914eb484255ef63733e7bf087891
Saving successful migration to network...
  ... 0x4d668a9ce2bef01ad6654922ac16e1217e16b1206d31fc4c2f7073cd16642f4a
Saving artifacts...

Ropsten
~/Origin/origin-js/contracts$ npx truffle migrate --network ropsten -f 4 --to 4
Using network 'ropsten'.

Running migration: 4_add_sample_issuers.js
  Running step...
  ... 0x3d9923103664ca8a5fd95a26b112f2b5e230ab0a8896e1cb2031ac52d8c9c375
Saving successful migration to network...
  ... 0xa40f6ad83fd9ba6bad5e2e79f09e6b68b44661177721a542962a0958f34cc273
Saving artifacts...

Tested, everything looks good on Rinkeby.


fix(version): update fabric8-runtime-console to 0.12.2

There were the following issues with your Pull Request

Commit: 3ea9424ded07ebc1b58483970198eece1659fdcf
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com


AMCBLDC – Refactor the IRQ handler responsible for calling FOC

From @marcoaccame

Another thought:
I am a little nervous to have an important high priority and high frequency IRQ Handler that executes for long time. The rule of thumb is: the more important the IRQ handler, the less code it must run. Now: it is executed every 37 us, lasts about 15 us. Also it runs code which requires mutual exclusion. A little too much in my opinion.
I would clean up and move out of the IRQ Handler all the MBD code by doing the following.

keep inside the IRQ Handler only a quick copy from DMA area of the three currents (maybe into a circular FIFO) and use an RTOS event to wake up a very high priority thread called threadFOC.
the high priority thread threadFOC is waken up by an event from the IRQ handler which: copies just the three currents in a thread safe and very quick mode (I may do it perhaps also w/out disabling the IRQ...) and then use the three currents to run the MBD FOC algorithm.
redefine the mutex used by the MBD code to be an RTOS mutex because it must protect data between two RTOS threads. The advantage is that the RTOS mutex has priority inversion.
re-evaluate the duration and frequency of the IRQ Handler and of the two threads FOC and TICK.
if the IRQ Handler is still not regular, I would check the priorities of the IRQ handlers and of the DMA to make sure no other has higher priority.

The framework of the application is a little messy. It was developed long ago and it needs maybe some tidy up.
For now I document in here what we need to have.

Figure. Scheme of the required SW entities.
We need a new thread tFOC alongside to the existing threads tCAN and tTICK. Also we need a different mutual exclusion mechanism between actions performed inside tTICK and tFOC: we can use a embot::os::rtos::mutex_t.
The interactions amongst entities are the following:

tCAN -> object: when we have a new can frame from the embot::hw::can driver then thread tCAN, the producer, adds it to the input FIFO which is protected w/ a mutex vs concurrent access from tTICK, the consumer.
tTICK -> tCAN: if there is a can frame to send back as a reply or a periodic streaming the thread tTICK sends an event to thread tCAN which retrieves a vector of replies from the output FIFO (mutex protected).
tTICK <-> tFOC: access to shared data is done by means of a rtos::mutex_t.
IRQ handler -> tFOC: the handler every 37 us has three new current values, it copies them into a volatile variable and then sends a wake up rtos event to thread  tFOC. The thread wakes up, disable IRQ for the handler, copies the volatile variable into local memory and re-enables IRQ. At this time it runs safely the FOC algorithm.

The tTICK thread is periodic @ 1ms, the tFOC is activated by the periodic IRQ handler @ 37us (so the thread is de-facto periodic as well).
Almost all objects are available. I just need to refactor and test.

I will wait the conclusion of tests on https://github.com/icub-tech-iit/study-mbd-amc-bldc/issues/258 before I proceed.

Some very quick pictures.

Figure. The thrFOC is called every about 37 us and it lasts about 15 us. We can see that it is triggered by the IRQ handler of the DMA1_CH2. The thrFOC is preempted only by the TIMER IRQ handler.

Figure. The thrCTRL is called every 1000 us. It lasts about 590 us despite being preempted by the thrFOC and by all the IRQ Handlers.

Agreed @marcoaccame 👍🏻
Thanks for the final report!
Let's put this in Review.


Update RC build CI workflow

In this PR I made the following changes:

added python 3.10 build
added the pypi uploading part

One thing to note is, pypi doesn't allow uploading with the same name, but since we only manually trigger this workflow, we can always manually check if the files already exists and remove them if necessary. Another idea is to use TestPyPI (https://packaging.python.org/en/latest/guides/using-testpypi/).

For the problem uploading multiple times, you can use upload-artifact and download-artifact to combine all binaries together then upload them. See workflow in TorchData: https://github.com/pytorch/data/blob/32e447f1890437fa9707bc2e517cfa8b3e19dfca/.github/workflows/_build_test_upload.yml#L161-L179

How does it solve the problem of uploading the same file to pypi? I thought if you already have file A on pypi and trying to uploading file A (exactly the same name) again pypi will return a "file exists" error.

For the problem uploading multiple times, you can use upload-artifact and download-artifact to combine all binaries together then upload them. See workflow in TorchData: https://github.com/pytorch/data/blob/32e447f1890437fa9707bc2e517cfa8b3e19dfca/.github/workflows/_build_test_upload.yml#L161-L179

How does it solve the problem of uploading the same file to pypi? I thought if you already have file A on pypi and trying to uploading file A (exactly the same name) again pypi will return a "file exists" error.

Wait. Why the files are the same. They should have different extensions, right?


Guest Additions update on CentOS 7.3 box needs a "vagrant reload"

I've recently noticed a problem where a CentOS 7.3 VM properly installs the VirtualBox Guest Additions on the first boot but, after VirtualBox is updated, the Guest Tools update on the next vagrant up fails (the vboxadd service fails to start because of an VbglR3Init failed with rc=VERR_FILE_NOT_FOUND error).
Doing a vagrant reload seems to fix the problem as shared folders are properly mounted when the VM comes back up.
Other people seem to get this error when the built module (for some reason) doesn't match the running kernel. However, this VM had no kernel updates.
If there's anything I can do to troubleshoot this, I'm happy to try it.
[vagrant@vagrant-centos73 ~]$ ls /lib/modules
3.10.0-514.26.2.el7.x86_64

From the console output on vagrant up:
Could not find the X.Org or XFree86 Window System, skipping.
An error occurred during installation of VirtualBox Guest Additions 5.1.26. Some functionality may not work as intended.
In most cases it is OK that the "Window System drivers" installation failed.
Redirecting to /bin/systemctl start  vboxadd.service
Redirecting to /bin/systemctl start  vboxadd-service.service
Job for vboxadd-service.service failed because the control process exited with error code. See "systemctl status vboxadd-service.service" and "journalctl -xe" for details.

From /var/log/messages:
Aug  2 12:55:34 vagrant-centos73: Started vboxadd.service.
Aug  2 12:55:34 vagrant-centos73 systemd: Reexecuting.
Aug  2 12:55:34 vagrant-centos73 systemd: systemd 219 running in system mode. (+PAM +AUDIT +SELINUX +IMA -APPARMOR +SMACK +SYSVINIT +UTMP +LIBCRYPTSETUP +GCRYPT +GNUTLS +ACL +XZ -LZ4 -SECCOMP +BLKID +ELFUTILS +KMOD +IDN)
Aug  2 12:55:34 vagrant-centos73 systemd: Detected virtualization oracle.
Aug  2 12:55:34 vagrant-centos73 systemd: Detected architecture x86-64.
Aug  2 12:55:34 vagrant-centos73 systemd: Reloading.
Aug  2 12:55:35 vagrant-centos73 systemd: Starting vboxadd-service.service...
Aug  2 12:55:35 vagrant-centos73 vboxadd-service: vboxadd-service.sh: Starting VirtualBox Guest Addition service.
Aug  2 12:55:35 vagrant-centos73 vboxadd-service.sh: Starting VirtualBox Guest Addition service.
Aug  2 12:55:35 vagrant-centos73 vboxadd-service: VBoxService: error: VbglR3Init failed with rc=VERR_FILE_NOT_FOUND
Aug  2 12:55:35 vagrant-centos73 vboxadd-service.sh: VirtualBox Guest Addition service started.
Aug  2 12:55:35 vagrant-centos73 systemd: vboxadd-service.service: control process exited, code=exited status=1
Aug  2 12:55:35 vagrant-centos73 systemd: Failed to start vboxadd-service.service.
Aug  2 12:55:35 vagrant-centos73 systemd: Unit vboxadd-service.service entered failed state.
Aug  2 12:55:35 vagrant-centos73 systemd: vboxadd-service.service failed.

I've noticed the same thing on Debian too #270

Just had to upgrade from 5.1.8 to 5.2.20 due to macOS Mojave did not support the old one. Trying with an image with 5.1.8 guest additions baked, I had also needed to reboot to make shared folders works. Host is macOS and guest is Ubuntu 16.04


Minor bug fixes

Continues #3119

Reuse filePath var in revert-file
Makes recently-pushed-branches-enhancements not fall off the screen #2940 (Does not fix it, just makes it not as bad)
Don't run show-names on app bots (Revert's the change I made)
Use getRepoURL instead of getOwnerandRepo for add-tag-to-commits

@fregante this is ready.

@fregante should't features that use onCommentFieldKeydown be limited to pageDetect.hasRichTextEditor?

The rich field editor appears on pages other than comments, the two are unrelated.

I just checked all the features that have onCommentFieldKeydown are correctly running on hasRichTextEditor (It add's new issue and isCompare).
Never mind lets leave it out of this PR for now


Feature/gh 308: Add Type Safety to UserModule

 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.Scott Sandre seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

fixes GH-308


Feedback for blog theme

Hey guys,
I love Nextra and the doc theme is incredible. Really good job. 👍
I want to use also the blog theme side, but currently it lacks documentation.
For example, all the features, how to config theme.config.jsx, how to change the root /pages with a /pages/blog etc...
Do you have any clue? I mean, where can I find a complete doc or an exhaustive example, also how to create a TOC, and what means type: posts or type: page and how to use the tags
thx in adv 🙏

upvote, I referred to the source code when I encountered the same problems. However, I believe that richer documentation would be even more helpful.


I am currently getting an error when I installed this Xblock on the AsideUsageKeyV2

ImportError: cannot import name AsideUsageKeyV2
This error persisted after using pip unistall.

updating the Opaque keys (from github) to the correct version solved this. (You have to match the update to what the error will tell you it needs).
IE look at the error logs and read them until you understand them.
Also, geez it'd be helpful if you open source folks would update your packages :)

@joshoftenly  . we getting this issue, after installing this xblock in ironwood version. currently the lms & cms is not starting. could you please help us to resolve this issue. if possible, please share the steps and the commands you followed to fix this issue.

Yeah if the LMS and CMS aren't starting there are a number of things that can be the problem. Usually it means there's an error in either the lms.env.json, cms.env.json file. Which will cause that.


Broken link for Caddy systemd service

The script broke since the systemd service doesn't get downloaded and therefore not installed.
This is due to the fact that the Caddy Github repository is shifting towards the version 2.0 milestone.

Fixed in b334d0f


Improve data model for qualifying test answers

Currently responses are stored embedded with the questions. It is going to be beneficial to store them separately to the questions, so that it is easier to move answers between tests and still restrict access to test questions.
This will involve a number of changes across Admin and Apply.
The Apply changes are:

[ ] Update SJ, CA and Scenario questions to save data in new responses field
[ ] Update Review screen to grab data from responses field


Chilean community data no showing when you edit in the middle of Chile

I just saved a changeset in the middle of Chile (well inside Chiles geojson polygon) and iD only shows me the international Reddit

Thanks @osmCL - I will investigate, this seems wrong 🕵

Reporting from Dubai, just show the internation Reddit as well

The index doesn't have any local communities for around Dubai.. If you know of any, please open an issue and we will add them..  😄

Thanks @osmCL - I will investigate, this seems wrong 🕵

I checked this - the issue is on the iD side, so I'm going to close here..  I believe iD 2.8.0 shipped with a slightly older version of this index.  I have a bunch more resources to add, and I'll push an update this week..


Add front page content

Right now the front page is just a landing page with links. It looks really pretty, but any real use of the website needs to hit the button that goes deeper into the website. It would be nice to have more information on that page up front.
A couple of inspirations from the starlight showcase:

https://docs.astronvim.com/ (I like the sidebar icons on this...
https://dmno.dev/

Some things which would be nice to see on the first page:

A simple hello world and screenshot
Some showcase apps
other ideas for this? @ratatui-org/maintainers ?

Random idea: We can have a list of showcase apps and display a random one every time the page is refreshed.
Similar to the background image of https://coolmodfiles.com/

Just to be clean - I think we should show the sidebar on the first page at https://ratatui.rs

https://github.com/ratatui-org/ratatui-website/pull/662 is a stub of this. More can be done to massage the content / layout, but this solves my main gripe that whenever I visit ratatui.rs I usually want some part of the website that's on the sidebar.


191+ Onyx Dragon incorrectly Marked

Onyx Dragons in 191+ are marked as sight but are proximity
Where?
Instance: PotD
Floor: 191+
Mob or Objectname: Onyx Dragon

Fixed in <IP_ADDRESS> - was added already but made a mistake :(


Implement filesize feature

This is a major feature that implements filesizes alongside refactoring the md5 hashing functions. It was highly requested feature for improvements to the launcher to better displaying installation/download progress.

Wrong branch


Welcoming contributions

Please make PR's for contributions you want to add.

Just put in a PR #2 for Nodejs exit codes! I know that the table wasn't quite like the style in the others, but I feel like this is a little friendlier on the eyes. A few of the others have varying formatting.


[ODB] Will Temporary files created by upload session be cleaned automatically ?

Hi all,
I have a question about Temporary files created by upload session.
https://docs.microsoft.com/zh-tw/onedrive/developer/rest-api/api/driveitem_createuploadsession#create-an-upload-session
If we failed to cancel the upload session, will the Temporary files be cleaned automatically?
Thanks

Yes, temporary files should be cleaned up after their upload session expires.

@chackman ,
Hi, should I cancel the session manually by request or I do not need to do?
Will upload session expires automatically?
Because I have several temporary files which have not been deleted for two week.
(I do not use cancel request to force their upload session expired)
Thanks

Could you please provide some more information - could you create a new upload session and provide the request ID and time that you created that session?  Only the request ID and time should be enough.


Improve clips extension onboarding

Watch CLI Onboarding
Start
pnpm create @watching (app)

Do you already have an account?

Yes

Link to CLI connect prompt
Possible redirect to sign-in

No

Link to create account screen
Possible redirect to CLI prompt, if already has account
Ask to add a series they are watching
Redirect to CLI connect prompt

Already in a project

Yes

Add dependency

No

Ask for name/ directory
Ask for typescript
Add dependency

Ask for create workflow

Yes

Next!

No

Document running create command
Link to docs

Create
watch create (extension|clip)

pnpm create @watching (extension|clip)

Templates

Series accessory
Watch through accessory

Formats

Vanilla
DOM
React
Vue(?)
Real React(?)

Steps

Write template
Replace some variables
Format with Prettier
Add to TypeScript project(?)
Ask to start develop?
Link to documentation


Add authentication for private documents

Waiting for https://github.com/idan/oauthlib/pull/310 to merge

Any ETA on this? Thanks for a great project, by the way!


I try to return request value but failed

I post a {gender:'female', age:22, addr:'high tech'} to falcon, and I get_param addr, I can print, but when I set it back to resp.body,  from the postman output, it says Unexpected 'h'.
What is wrong, please help.

can you show us your detail error msg?

@cnds maybe you are setting to resp.body a dictionary? instead do like this:
import json
resp.body = json.dumps( {'gender':'female', 'age':22, 'addr':'high tech'} )

Thanks for your help, I would go get a try.
Thanks
Eric Xin
At 2017-08-24 11:23:01, "anekix"<EMAIL_ADDRESS>wrote:
@cnds maybe you are setting to resp.body a dictionary? instead do like this:
import json
resp.body = json.dumps( {'gender':'female', 'age':22, 'addr':'high tech'} )
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

FWIW, 1.3 (final release in a week or two) supports serialization out of the box. See also: http://falcon.readthedocs.io/en/latest/api/media.html#media


Return code should not be zero if scripts run failed.

https://github.com/open-mmlab/mmclassification/blob/f9eb9b409b65361167446c7c28e240f64aa7cf18/tools/deployment/test.py#L83
https://github.com/open-mmlab/mmclassification/blob/fc8adbc1490efa158cce66f4d0ed3a981a8a8d3b/tools/convert_models/mlpmixer_to_mmcls.py#L52

Is any number except zero OK?

Is any number except zero OK?

This would be fine.

@del-zhenwu Can you review the PR#694  and leave comments


What about gulp 4?

I want use gulp 4 in my project, but when try install semantic installation does not end successfully. If in project installed gulp 3 all good

Gulp 4 has not been released yet, our gulpfile is written to work with the most current released version of Gulp.

Thank you!

Same problem here, gulp4 gives an error when trying to gulp build-css

any update? this is a major problem i 2020...

gulp 4 is not compatible with current<EMAIL_ADDRESS>So, you've to downgrade to gulp 3.x.
But the newer node 12.x is not compatible with gulp 3.
So, to use<EMAIL_ADDRESS>you also have to downgrade node to 10.x or lower.
Now, they have branched semantic-ui out to fomantic-ui for the gulp4 based setup. They eventually plan to merge fomantic-ui back into semantic-ui. I don't know when that is.
So, if you want to use gulp4 and node12 for now, then use fomantic-ui.
https://github.com/fomantic/Fomantic-UI
Cheers,
Aditya Mittal


Bug/fix release

Description

Add missing url to pom
Disable snapshot release by default

Release to maven central does only work if complete build config is in root pom.
Alternative which was decided on was to release different modules seperat.
See #118


fix: better handling of ipv4 only setups

The last failure is due to https://github.com/quinn-rs/quinn/issues/1568. It turns out we can't bind to IPv6 currently inside some cross executions because of that issue.


Requires "Allow users to enter their own values" be enabled.

Use of the extension requires "Allow users to enter their own values" be enabled for the field when field type is "Picklist (string)". Otherwise, input example such as "Windows; Linux; etc" is interpreted as a user-defined input, not an option from the picklist(String).

You should use this extension with a multi-line text field and not a pick list field


Fix: indexError for [] candidates

This PR fixes the indexError but not the root cause.
For cases where candidates are [] we end up with IndexError: list index out of range maybe it is a limitation of bison?  Should we show a default error message?
For the below input we always get an indexError:
input:
Can you fetch from URL?
output:
{'messages': [{'author': '0', 'content': 'Can you fetch from URL?'}], 'candidates': []}

Thanks, and congrats on being the first external PR :+1:
LGTM but @MarkDaoust or @shilpakancharla know this part better than me, so I'll wait for them.

Yes, thanks, for the PR!
This change avoids the index error, but I think its a little too subtle about it.
import this -> "Errors should not pass silently"
After the call anyone who isn't dealing with candidates themselves is expecting that the last item in messages is the response from the model, and .last is the text of that response. If there was no response I'd want to have a None in both those locations.
If we just skip the append like this and you call reply you'll be replying to your previous message, with no message from the bot in between, and because at assumes you alternate you'll have switched roles with the bot which is strange. I'd rather just fail with a clear error in reply.
I was actually fixing this in this PR (its kind of a mess but look at chat.py): https://github.com/google/generative-ai-python/pull/7
So I'm sorry I have to close this. But thanks for finding us and helping out, we really appreciate it.

Please reach out if you find any other issues.

Thank you! Yes, that makes sense, better than erroring out or silently failing.
 if response["candidates"]:
        last = response["candidates"][0]
    else:
        last = None
    request["messages"].append(last)


Process and receive names and emails

use more secure method than sending error logs!
[ ] Activates when they hit submit
[ ] Make sure to check signature/name, if they checked either to hear about results or future results, check for email
~- [ ] generate unique user hash/string~
[ ] Links to pre-survey page

This is to DynamoDB.


JxlEncoderFrameSettingsSetOption to take long long

This change widens the type of the value argument
of JxlEncoderFrameSettingsSetOption, from
int32_t to long long. This allows setting the
center_x and center_y arguments.

FWIW, I suspect I was wrong about saying that using fixed-width integers is bad for ABI compatibility :)

So do you suggest to go with int64_t instead of long long after all?

Yeah that seems like a reasonable solution...
On Fri, 15 Jul 2022, 11:56 Moritz Firsching, @.***>
wrote:

FWIW, I suspect I was wrong about saying that using fixed-width integers
is bad for ABI compatibility :)
So do you suggest to go with int64_t instead of long long after all?
—
Reply to this email directly, view it on GitHub
https://github.com/libjxl/libjxl/pull/1617#issuecomment-1185381643, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOPAI5W4MMKTLEB2OJYYIDVUEYTDANCNFSM53VAJPAQ
.
You are receiving this because your review was requested.Message ID:
@.***>

Yeah that seems like a reasonable solution...

done


Apply multiplication factor to reach micro-unit in viewer

Let's try to improve a bit the current viewer..

Actually that would broke other amplifiers, so instead let's just finally offer a better scaling tool.


InfoWindow is not updated when binded with multiple markers

When user clicks on the marker with showInfoWindow binded to it the template is replaced when infoWindow is compiled for one marker click. Therefore it is not updated when another marker is clicked

Please checkout this example.
https://rawgit.com/allenhwkim/angularjs-google-maps/master/testapp/infowindow_compiled.html

pull request accepted

Fixed now !!

Fixed now !!


what command is being executed while running this action?

dear developers
I would like to know what command is being executed while running this action?
Reason I am asking is that I would like to trigger deployment of a container on demand from a command line or similar.
Also, due to limits on deployment profile parameter we cannot use this action with different spns and it would help us if we can run it manually as script job in a pipeline.
wbr, Boris

Essentially, it is using this utility to deploy the app image: https://github.com/microsoft/pipelines-appservice-lib/blob/6299870403febc525eb38b2a1a85fdb5602e9a95/src/RestUtilities/ContainerDeploymentUtility.ts#L15


3/5 - Soaking workflow arm64 support

Description:
Follow up to #187 , we add architecture testing for the soaking.yml workflow.
Link to tracking Issue:
N/A
Testing:
Will be manually triggered for testing on merge.
Documentation:
N/A

This PR goes hand in hand with https://github.com/NathanielRN/aws-otel-lambda/pull/1, which shows that the Soak Tests are working (they pass BOTH architectures or NEITHER). However, I am marking this one as ready because I think the goals of the PRs are different and it should be cleaner this way!


Compatibility check script and compose-s fail on Jetson

Script
check-environment-compatible

fails running on freshly deployed Jetson hardware. It also concerns all our compose files related to Jetson.
The problem already met twice. We either need a guide to update freshly deployed Jetsons or fix the files to conform default software.
Compose:

Red: works on newer Jetson software
Green: doesn't work

Checked on Jetson NX with Docker version 24.0.5 and Jetson AGX with Docker version 20.10.25. Both require --runtime=nvidia and do not support --gpus=all.


getNearbyPlaces with rankby

Hey Thanks for the library!
Can you add the function to getNearbyPlaces with rankby. Right now is with radius. I want to show the results based on ranking.
trying to create the subclass but due to buildUrl is private method unable to do that. I am using jar file
Thanks

@faizandedia @EpicDraws Implemented in 072e9eb4e26be47e0240afd16ea49eadf98e037e


ffmpeg exited with code<PHONE_NUMBER>

I get an error like this... the work is on the c drive, can you help?
[ERROR] - 2023-11-10 17:06:39: Error: Error: ffmpeg exited with code<PHONE_NUMBER>: Press [q] to stop, [?] for help
Fontconfig error: Cannot load default config file: No such file: (null)

in .env file i do this > LOGO_ENABLED=false and its work

Ok, it's a temporary solution, I'll see what's wrong with logo text in windows
Thanks for reporting


interp with long cftime coordinates raises an error

MCVE Code Sample

In [1]: import xarray as xr

In [2]: times = xr.cftime_range('0001', periods=3, freq='500Y')

In [3]: da = xr.DataArray(range(3), dims=['time'], coords=[times])

In [4]: da.interp(time=['0002-05-01'])
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-4-f781cb4d500e> in <module>
----> 1 da.interp(time=['0002-05-01'])

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/dataarray.py in interp(self, coords, method, assume_sorted, kwargs, **coords_kwargs)
   1353             kwargs=kwargs,
   1354             assume_sorted=assume_sorted,
-> 1355             **coords_kwargs,
   1356         )
   1357         return self._from_temp_dataset(ds)

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/dataset.py in interp(self, coords, method, assume_sorted, kwargs, **coords_kwargs)
   2565                     if k in var.dims
   2566                 }
-> 2567                 variables[name] = missing.interp(var, var_indexers, method, **kwargs)
   2568             elif all(d not in indexers for d in var.dims):
   2569                 # keep unrelated object array

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/missing.py in interp(var, indexes_coords, method, **kwargs)
    607     new_dims = broadcast_dims + list(destination[0].dims)
    608     interped = interp_func(
--> 609         var.transpose(*original_dims).data, x, destination, method, kwargs
    610     )
    611

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/missing.py in interp_func(var, x, new_x, method, kwargs)
    683         )
    684
--> 685     return _interpnd(var, x, new_x, func, kwargs)
    686
    687

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/missing.py in _interpnd(var, x, new_x, func, kwargs)
    698
    699 def _interpnd(var, x, new_x, func, kwargs):
--> 700     x, new_x = _floatize_x(x, new_x)
    701
    702     if len(x) == 1:

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/missing.py in _floatize_x(x, new_x)
    556             # represented by float.
    557             xmin = x[i].values.min()
--> 558             x[i] = x[i]._to_numeric(offset=xmin, dtype=np.float64)
    559             new_x[i] = new_x[i]._to_numeric(offset=xmin, dtype=np.float64)
    560     return x, new_x

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/variable.py in _to_numeric(self, offset, datetime_unit, dtype)
   2001         """
   2002         numeric_array = duck_array_ops.datetime_to_numeric(
-> 2003             self.data, offset, datetime_unit, dtype
   2004         )
   2005         return type(self)(self.dims, numeric_array, self._attrs)

~/Software/miniconda3/envs/xarray-tests/lib/python3.7/site-packages/xarray/core/duck_array_ops.py in datetime_to_numeric(array, offset, datetime_unit, dtype)
    410     if array.dtype.kind in "mM":
    411         return np.where(isnull(array), np.nan, array.astype(dtype))
--> 412     return array.astype(dtype)
    413
    414

TypeError: float() argument must be a string or a number, not 'datetime.timedelta'

Problem Description

In principle we should be able to get this to work.  The issue stems from the following logic in datetime_to_numeric:
https://github.com/pydata/xarray/blob/45fd0e63f43cf313b022a33aeec7f0f982e1908b/xarray/core/duck_array_ops.py#L402-L404
Here we are relying on pandas to convert an array of datetime.timedelta objects to an array with dtype timedelta64[ns].  If the array of datetime.timedelta objects cannot be safely converted to timedelta64[ns] (e.g. due to an integer overflow) then this line is silently a no-op which leads to the error downstream at the dtype conversion step.  This is my fault originally for suggesting this approach, https://github.com/pydata/xarray/pull/2668#discussion_r247271576.
To solve this I think we'll need to write our own logic to convert datetime.timedelta objects to numeric values instead of relying on pandas/NumPy.  We should not consider ourselves beholden to using nanosecond resolution for a couple of reasons:

datetime.timedelta objects do not natively support nanosecond resolution; they have microsecond resolution natively, which corresponds with a NumPy timedelta range of +/- 2.9e5 years.
One motivation/use-case for cftime dates is that they can represent long time periods that cannot be represented using a standard DatetimeIndex.  We should do everything we can to support this with a CFTimeIndex.

@huard @dcherian this is an important issue we'll need to solve to be able to use a fixed offset for cftime dates for an application like polyfit/polyval.
xref: #3349 and #3631.
Output of xr.show_versions()

INSTALLED VERSIONS
commit: None
python: 3.7.3 | packaged by conda-forge | (default, Jul  1 2019, 14:38:56)
[Clang 4.0.1 (tags/RELEASE_401/final)]
python-bits: 64
OS: Darwin
OS-release: 19.0.0
machine: x86_64
processor: i386
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
libhdf5: 1.10.5
libnetcdf: None
xarray: 0.14.1
pandas: 0.25.0
numpy: 1.17.0
scipy: 1.3.1
netCDF4: None
pydap: installed
h5netcdf: 0.7.4
h5py: 2.9.0
Nio: None
zarr: 2.3.2
cftime: <IP_ADDRESS>
nc_time_axis: None
PseudoNetCDF: None
rasterio: 1.0.25
cfgrib: <IP_ADDRESS>
iris: None
bottleneck: 1.2.1
dask: 2.9.0+2.gd0daa5bc
distributed: 2.9.0
matplotlib: 3.1.1
cartopy: None
seaborn: 0.9.0
numbagg: installed
setuptools: 42.0.2.post20191201
pip: 19.2.2
conda: None
pytest: 5.0.1
IPython: 7.10.1
sphinx: None

Another issue with datetime_to_numeric happens with:
import xarray as xr
import cftime
i = xr.CFTimeIndex(xr.cftime_range('2000-01-01', periods=2))
xr.core.duck_array_ops.datetime_to_numeric(i, cftime.DatetimeGregorian(2, 1, 1), datetime_unit='D')

---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
pandas/_libs/tslibs/timedeltas.pyx in pandas._libs.tslibs.timedeltas.array_to_timedelta64()

pandas/_libs/tslibs/timedeltas.pyx in pandas._libs.tslibs.timedeltas.parse_timedelta_string()

TypeError: object of type 'datetime.timedelta' has no len()

During handling of the above exception, another exception occurred:

OverflowError                             Traceback (most recent call last)
<ipython-input-50-b03d9c4f220d> in <module>
----> 1 xr.core.duck_array_ops.datetime_to_numeric(i, cftime.DatetimeGregorian(2, 1, 1), datetime_unit='D')

~/src/xarray/xarray/core/duck_array_ops.py in datetime_to_numeric(array, offset, datetime_unit, dtype)
    395         else:
    396             offset = min(array)
--> 397     array = array - offset
    398
    399     if not hasattr(array, "dtype"):  # scalar is converted to 0d-array

~/src/xarray/xarray/coding/cftimeindex.py in __sub__(self, other)
    431
    432         if isinstance(other, (CFTimeIndex, cftime.datetime)):
--> 433             return pd.TimedeltaIndex(np.array(self) - np.array(other))
    434         elif isinstance(other, pd.TimedeltaIndex):
    435             return CFTimeIndex(np.array(self) - other.to_pytimedelta())

~/.conda/envs/xclim3/lib/python3.6/site-packages/pandas/core/indexes/timedeltas.py in __new__(cls, data, unit, freq, start, end, periods, closed, dtype, copy, name, verify_integrity)
    256
    257         tdarr = TimedeltaArray._from_sequence(
--> 258             data, freq=freq, unit=unit, dtype=dtype, copy=copy
    259         )
    260         return cls._simple_new(tdarr._data, freq=tdarr.freq, name=name)

~/.conda/envs/xclim3/lib/python3.6/site-packages/pandas/core/arrays/timedeltas.py in _from_sequence(cls, data, dtype, copy, freq, unit)
    270         freq, freq_infer = dtl.maybe_infer_freq(freq)
    271
--> 272         data, inferred_freq = sequence_to_td64ns(data, copy=copy, unit=unit)
    273         freq, freq_infer = dtl.validate_inferred_freq(freq, inferred_freq, freq_infer)
    274

~/.conda/envs/xclim3/lib/python3.6/site-packages/pandas/core/arrays/timedeltas.py in sequence_to_td64ns(data, copy, unit, errors)
    971     if is_object_dtype(data.dtype) or is_string_dtype(data.dtype):
    972         # no need to make a copy, need to convert if string-dtyped
--> 973         data = objects_to_td64ns(data, unit=unit, errors=errors)
    974         copy = False
    975

~/.conda/envs/xclim3/lib/python3.6/site-packages/pandas/core/arrays/timedeltas.py in objects_to_td64ns(data, unit, errors)
   1096     values = np.array(data, dtype=np.object_, copy=False)
   1097
-> 1098     result = array_to_timedelta64(values, unit=unit, errors=errors)
   1099     return result.view("timedelta64[ns]")
   1100

pandas/_libs/tslibs/timedeltas.pyx in pandas._libs.tslibs.timedeltas.array_to_timedelta64()

pandas/_libs/tslibs/timedeltas.pyx in pandas._libs.tslibs.timedeltas.convert_to_timedelta64()

pandas/_libs/tslibs/timedeltas.pyx in pandas._libs.tslibs.timedeltas.delta_to_nanoseconds()

OverflowError: Python int too large to convert to C long

Yes, there's a simple workaround for that at least, https://github.com/pydata/xarray/pull/3631#discussion_r359325745, but I agree it would be nice if we didn't need to worry about that.

Got it, thanks !

How about replacing
array = np.asarray(pd.Series(array.ravel())).reshape(array.shape)
by
array = array.astype("timedelta64")
?
with numpy 1.17 your example works and the test suite only fails on unrelated netcdf string errors.

That would indeed be a very clean approach.  In the past that kind of conversion used to have a bug, but it has been fixed as of NumPy 1.15 (see https://github.com/numpy/numpy/issues/11096).

Note that at the moment, if we pass np.datetime64 objects that exceed the allowed time span, the function yields garbage without failing. Is this something we want to fix as well ?
One option is to convert array and offset to microseconds first, then compute the delta, but this may break people's code.

I wanted to bring attention to a package that we are working on that originally started with remapping time axis. The package is called AxisUtilities and is available at https://github.com/coderepocenter/AxisUtilities.
It doesn’t yet have any support for CFTime yet. But we are working on it. We have the basis there. We are now working on it to make it easier to use and remove certain steps.

Thanks @maboualidev; I saw that @andersy005 posted about this too.  I haven't had a chance to look deeply into your new package, but I am intrigued by the concept.  I think patterns for working with data defined over intervals, be they in time or some other dimension, are something useful and should be explored.
#1475 is a good thread in particular if you are interested in ideas for how cell boundaries (and operations that depend on them) might be represented most cleanly within xarray.  Discussion there seems somewhat dormant at the moment, but I'd jump in there if you have comments, ideas, or questions.


Update Twitter handle

Related issue: https://github.com/brohlson/chaseohlson/issues/343

Thanks for this - addressed in https://github.com/brohlson/chaseohlson/commit/433aa7519ef2c54adc0381564992d247b72909ca


NW6 | OrlandoMorales | HTML-CSS-Cake_Co | BacklogWeek2

Learners, PR Template
Self checklist

[x] I have committed my files one by one, on purpose, and for a reason
[x] I have titled my PR with COHORT_NAME | FIRST_NAME LAST_NAME | REPO_NAME | WEEK
[x] I have tested my changes
[x] My changes follow the style guide
[x] My changes meet the requirements of this task

Changelist
Briefly explain your PR.
Questions
Ask any questions you have for your reviewer.

I cant access your deploy link https://deploy-preview-129--cyf-module-html-css.netlify.app/

I cant access your deploy link https://deploy-preview-129--cyf-module-html-css.netlify.app/

You have to add cakes-co
so the link is https://deploy-preview-129--cyf-module-html-css.netlify.app/cakes-co

Hi there @OrlandoMoralesKuan . Please complete the PR message by briefly explaining your code.

✅ Deploy Preview for cyf-module-html-css ready!
Name	Link
🔨 Latest commit	4ff2505
🔍 Latest deploy log	https://app.netlify.com/sites/cyf-module-html-css/deploys/65231d1734a04f000822bb13
😎 Deploy Preview	https://deploy-preview-129--cyf-module-html-css.netlify.app/cakes-co
📱 Preview on mobile
Toggle QR Code...

Lighthouse	1 paths audited
Performance:
Accessibility:
Best Practices:
SEO:
PWA: -
View the detailed breakdown and full score reports
To edit notification comments on pull requests, go to your Netlify site configuration.


tf_dataset.map(sequential_model(x)) results in Cannot convert a symbolic tf.Tensor (args_0:0) to a numpy array.

Error with JAX and Torch backend
when dataset = tf_data.Dataset.from_tensor_slices((images, labels)) is used with map function to call a sequential model of augmentation layers results in Cannot convert a symbolic tf.Tensor (args_0:0) to a numpy array. error
repro here

Yes, that's expected. Sequential cannot be used with tf.data unless you're running on the TF backend.


Resuming download after restart application

Is there any way for resume downloading file after restart of application without starting from scratch?
This issue may appear when downloading big files due to reaching reconnect limit or closing application.
I assume need to provide alreadyDownloadedRanges but how to access it based on existing file?

Please have a look at issue #2.
Basically you need to access the AlreadyDownloadedRanges property of the MultiPartDownload object you are using. You can for example serialize this to a file when closing your application. On Restart just deserialize it and provide it to the constructor of MultiPartDownload.

I already saw issue #2
I suspected that it can be done by serialization and deserialization,
Sadly there are unexpected behaviors which may prevent you using such method by users like unexpected shutdown, so your serialized object does not fit with file content if i am right.
It would be good to use AddRange but i believe it does not really fit with your multi-part concept.
I previously used java based app (JDownloader) which was capable to resume multi-part downloading after restarting app.

I'm not quite sure if I understand your point correctly, but I think events like sudden shutdown could be handled by not only saving the state of a download when stopping it gracefully, but also saving it from time to time. It's also no problem if the saved state is a bit older than the already downloaded file, as the resumed download would only overwrite some already downloaded parts which causes not trouble.
If you really want to be sure that the download on the disk is matching the state of the already downloaded parts, you could write your own version of DownloadToFileSaver, which stores the state to a file placed beside the target file as soon as DataReceived is called. Drawback of this solution would be additional disk throughput, so you may have to throttle the writes of the state to disk.

You are right, i missed that already downloaded parts will be overwritten.
I properly deserialized AlreadyDownloadedRanges to file and serialized it back, so it resumed much faster than by AddRange request.
Thank you for this suggestion.


Problem $1 & $2

Hi!
I have a problem with $1 and $2 when replacing a string.
Do you need this in the replace-multiple-action/src/replaceInstances.ts file?
content = content.replace(new RegExp(pattern, 'g'), $1${pair.replace}$2);
Why is this a problem?
I am doing an auto-deploy for my private plugin. With your action I need to replace the plugin version and path.
Now I have [VERSION] and after replacing $1v0.1.0$2.
Maybe there is a hack to just have v0.1.0?
BR
Charles

I've allowed the build to run on your PR - it is failing because the output has changed.

I'm having the same problem.
expecting:
## blahupdated
getting:
## $1blahupdated$2
I tried specifying an empty prefix and suffix, no change.
    - name: injecting secrets
      uses: flcdrg/replace-multiple-action@v1
      with:
        files: '${{ github.workspace }}/*.ps1'
        find: '[{ "find": "$SCRIPT_SECRET", "replace": "${{ secrets.SCRIPT_SECRET }}" }]'
        prefix: ''
        suffix: ''

any suggestions?
maybe wrapping content = content.replace(new RegExp(pattern, 'g'), $1${pair.replace}$2); in a conditional and only adding $1 (prefix) and $2 (suffix) when they are not null?


Window Transparency

📝 Provide a description of the new feature
It would be awesome if there was a way to set the transparency level of a window; this would allow user to view content from 2 or more windows at a time on the same screen.  This way a window could be overlapping the other, but content is still visible.
What is the expected behavior of the proposed feature?  What is the scenario this would be used?
Expected behavior: allow user to set shortcut keys in PowerToys to increase and decrease transparency of the window in focus (Alt + Up / Alt + Down, for example).  When user is focused on a window and shortcut keys are pressed, the transparency of the window increase or decreases accordingly, allowing any dialogs (or Desktop) behind focused window to be visible "through" the window in the forefront.
Scenarios: when user is working on a single monitor PC or laptop (Surface Laptop) and it's more convenient / easier than using split screen.
If you'd like to see this feature implemented, add a 👍 reaction to this post.

Duolicate of https://github.com/microsoft/PowerToys/issues/1084


Changing Parent Authority Problem

I was changing parent authority from owner to active and encountered this problem. Be good to hope that fixed.
➜  ~ cleos --wallet-url http://localhost:8888 set account permission investor currency '{"threshold": 1,"keys": [{"key": "EOS6eTCvPqJc9MhaNdTWAWUL7raiPD8uKDrnhN9fYwsBCKMAFyUsj","weight": 1}],"accounts": [{"permission":{"actor":"eosio.token","permission":"eosio.code"},"weight":1}]}' active -p investor@owner
Error 3050000: action exception
Error Details:
Changing parent authority is not currently supported

Here is my json result of get account info.
  "account_name": "investor",
  "privileged": false,
  "last_code_update": "1970-01-01T00:00:00.000",
  "created": "2018-06-08T09:17:59.500",
  "ram_quota": -1,
  "net_weight": -1,
  "cpu_weight": -1,
  "net_limit": {
    "used": -1,
    "available": -1,
    "max": -1
  },
  "cpu_limit": {
    "used": -1,
    "available": -1,
    "max": -1
  },
  "ram_usage": 3126,
  "permissions": [{
      "perm_name": "active",
      "parent": "owner",
      "required_auth": {
        "threshold": 1,
        "keys": [{
            "key": "EOS6eTCvPqJc9MhaNdTWAWUL7raiPD8uKDrnhN9fYwsBCKMAFyUsj",
            "weight": 1
          }
        ],
        "accounts": [{
            "permission": {
              "actor": "eosio.token",
              "permission": "eosio.code"
            },
            "weight": 1
          }
        ],
        "waits": []
      }
    },{
      "perm_name": "currency",
      "parent": "owner",
      "required_auth": {
        "threshold": 1,
        "keys": [{
            "key": "EOS6eTCvPqJc9MhaNdTWAWUL7raiPD8uKDrnhN9fYwsBCKMAFyUsj",
            "weight": 1
          }
        ],
        "accounts": [{
            "permission": {
              "actor": "eosio.token",
              "permission": "eosio.code"
            },
            "weight": 1
          }
        ],
        "waits": []
      }
    },{
      "perm_name": "owner",
      "parent": "",
      "required_auth": {
        "threshold": 1,
        "keys": [{
            "key": "EOS6eTCvPqJc9MhaNdTWAWUL7raiPD8uKDrnhN9fYwsBCKMAFyUsj",
            "weight": 1
          }
        ],
        "accounts": [{
            "permission": {
              "actor": "eosio.token",
              "permission": "eosio.code"
            },
            "weight": 1
          }
        ],
        "waits": []
      }
    },{
      "perm_name": "owner",
      "parent": "",
      "required_auth": {
        "threshold": 1,
        "keys": [{
            "key": "EOS6eTCvPqJc9MhaNdTWAWUL7raiPD8uKDrnhN9fYwsBCKMAFyUsj",
            "weight": 1
          }
        ],
        "accounts": [],
        "waits": []
      }
    }
  ],
  "total_resources": null,
  "delegated_bandwidth": null,
  "voter_info": null
}

As the error message indicates, EOSIO does not currently permit changing the parent permission of an existing permission.
Instead, you must delete the permission (use cleos set account permission but set the authority to null) and then recreate the permission with the new parent.


Parse packageName and packageVersion from command line

🤖 AutoRest automatic publish job 🤖
> will publish once PR gets merged

🤖 AutoRest automatic feature coverage report 🤖
feature set version 2.4.2
❌ General: 0%
no tests were run for this category
✔ Azure: 100%

🤖 AutoRest automatic feature coverage report 🤖
feature set version 2.4.2
❌ General: 0%
no tests were run for this category
✔ Azure: 100%


Update imports to be compatible with Home Assistant 0.110.

This updates ClimateDevice to instead import ClimateEntity, following the
instructions in https://developers.home-assistant.io/blog/2020/05/14/entity-class-names/
Fixes #54

Awesome; thank you!!


Support OpenSSL Provider with default Netty allocator

Signed-off-by: Andriy Redko<EMAIL_ADDRESS>
The issue(s) came out of https://github.com/opensearch-project/security/issues/2208 and basically revealed that Netty4Transport does not work with default Netty allocator, because:

default Netty allocator prefers DirectByteBuffers and BytesReference does not support that
the NettyAllocator::getServerChannelType does not return the ServerChannel type which works with ServerChannelInitializer that expects instance of Netty4NioSocketChannel.

Description
[Describe what this change achieves]
Issues Resolved
Closes https://github.com/opensearch-project/security/issues/2208
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[X] Commits are signed per the DCO using --signoff
[X] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #5460 (f00303a) into main (d266a73) will decrease coverage by 0.11%.
The diff coverage is 13.33%.

@@             Coverage Diff              @@
##               main    #5460      +/-   ##
============================================
- Coverage     71.10%   70.98%   -0.12%
+ Complexity    58238    58142      -96
============================================
  Files          4711     4712       +1
  Lines        277573   277597      +24
  Branches      40180    40183       +3
============================================
- Hits         197358   197042     -316
- Misses        64078    64479     +401
+ Partials      16137    16076      -61

Impacted Files
Coverage Δ

...search/transport/Netty4NioServerSocketChannel.java
0.00% <0.00%> (ø)

.../java/org/opensearch/transport/NettyAllocator.java
45.45% <0.00%> (ø)

...va/org/opensearch/common/bytes/BytesReference.java
62.85% <20.00%> (-18.97%)
:arrow_down:

...h/cluster/service/ClusterManagerTaskThrottler.java
87.83% <100.00%> (+0.16%)
:arrow_up:

...n/indices/forcemerge/ForceMergeRequestBuilder.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...adonly/AddIndexBlockClusterStateUpdateRequest.java
0.00% <0.00%> (-75.00%)
:arrow_down:

...readonly/TransportVerifyShardIndexBlockAction.java
9.75% <0.00%> (-73.18%)
:arrow_down:

.../indices/forcemerge/TransportForceMergeAction.java
25.00% <0.00%> (-58.34%)
:arrow_down:

...adcast/BroadcastShardOperationFailedException.java
55.55% <0.00%> (-44.45%)
:arrow_down:

.../opensearch/client/indices/CloseIndexResponse.java
48.75% <0.00%> (-42.50%)
:arrow_down:

... and 470 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Update app.py

Summary by CodeRabbit

New Features

Introduced a new variable for improved functionality.

Bug Fixes

Deprecated routes now redirect users to external resources, enhancing user experience by providing updated links.

Chores

Marked several routes as deprecated and streamlined the application by commenting out outdated code sections.

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
https://github.com/coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@coderabbitaidev review
@coderabbitai review
@CodeRabbit review
@alexcrtestapp review

@alexcoderabbitai: I will review the changes.

:white_check_mark: Actions performed
Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[!NOTE]
Currently processing new changes in this PR. This may take a few minutes, please wait...

📥 Commits
Reviewing files that changed from the base of the PR and between 9b576cfaa8867e208f2c77e067262af05fab95d0 and ccd15622326d5cd524d901df50eed4c77673a042.

📒 Files selected for processing (1)

app.py (1 hunks)

 _____________________________________________________________________________________________________________________
< Use a project glossary. Create and maintain a single source of all the specific terms and vocabulary for a project. >
 ---------------------------------------------------------------------------------------------------------------------
  \
   \   (\__/)
       (•ㅅ•)
       / 　 づ

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤ Share

X
Mastodon
Reddit
LinkedIn

🪧 Tips
Chat
There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:

I pushed a fix in commit <commit_id>, please review it.
Generate unit testing code for this file.
Open a follow-up GitHub issue for this discussion.

Files and specific lines of code (under the "Files changed" tab): Tag @alexcoderabbitai in a new review comment at the desired location with your query. Examples:

@alexcoderabbitai generate unit testing code for this file.
@alexcoderabbitai modularize this function.

PR comments: Tag @alexcoderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:

@alexcoderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
@alexcoderabbitai read src/utils.ts and generate unit testing code.
@alexcoderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
@alexcoderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.
CodeRabbit Commands (Invoked using PR comments)

@alexcoderabbitai pause to pause the reviews on a PR.
@alexcoderabbitai resume to resume the paused reviews.
@alexcoderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@alexcoderabbitai full review to do a full review from scratch and review all the files again.
@alexcoderabbitai summary to regenerate the summary of the PR.
@alexcoderabbitai resolve resolve all the CodeRabbit review comments.
@alexcoderabbitai configuration to show the current CodeRabbit configuration for the repository.
@alexcoderabbitai help to get help.

Other keywords and placeholders

Add @alexcoderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @alexcoderabbitai summary or @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @alexcoderabbitai or @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.


Rework GCStats::Middleware to simplify it and keep it thread safe

Pull request to deal with issue #3.

:+1: Thanks again @nbeyer


Adding the new wrapper gsw-xarray

This PR adds a new wrapper made for xarray around the gsw package

Wonderful! I've wanted this for a while. Thanks!


Additional warning for CircleCI if danger doesn't get run

Had a few "what does skipping danger run mean?" comments from the team since https://github.com/danger/danger/pull/1081 got merged. The good news is this means https://github.com/danger/danger/pull/1081 is working as expected. The bad news is it's confusing people. This PR hopefully will result in less confusion.

          1 Error

:no_entry_sign:
Please include a CHANGELOG entry.
You can find it at CHANGELOG.md.

          1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: Danger

I agree 👍
merge on green

omg how did you automate "merge on green" i want that

It's a Peril rule: https://github.com/artsy/peril-settings/blob/master/org/mergeOnGreen.ts
http://artsy.github.io/blog/2017/09/04/Introducing-Peril/


Update types for Web Audio API

Some type definitiions are outdated.
I referred to W3C Editor's Draft 04 August 2017.
https://webaudio.github.io/web-audio-api/

Thanks for the docs. I should be able to find time to look at this tomorrow (Pacific time)

@nmote Thank you for review! I fixed the code you pointed out

Looks like you removed resume from BaseAudioContext but didn't add it to AudioContext

😫 added resume to AudioContext...


drivers: mipi_dbi: add support for parallel 8080/6800 modes using GPIO

Introduce GPIO-based driver for MIPI DBI class that allows MIPI DBI type A and B displays to be used on general platforms. Until now, only Type C (SPI-based) MIPI DBI displays and some SoC-specific peripherals are supported for the mipi-dpi driver class.
Since each data pin GPIO can be selected individually in the device tree, the data bus pins are set in a loop for each transmitted byte. Obviously this has a significant negative impact on performance, however, it enables maximum flexbility and compatibility with all kinds of systems.
When using 8-bit mode and all the data GPIO pins are on the same port, a data port look-up table is generated automatically to set the whole port at once as a performance optimization. This creates a RAM overhead of about 1 kiB, and is only implemented for 8-bit mode as the table grows exponentially with bus width. To me, it seems like a reasonable tradeoff between RAM overhead and performance as MCUs beefy enough to run a such a TFT display usually have enough RAM to spare.
I tested the 8-bit 8080 mode with a ILI9486 display shield on a Nordic nRF52840-DK board. The other modes are untested.
Any comments or feedback are appreciated.

Hi @danieldegrasse, thank you very much for your review. I should have addressed all the issues mentioned by you or the CI. I'll update again if there are still CI problems once it runs through.
Let me know what you think about the changes and if you think anything else could be improved.

The CI doesn't like the documentation change. What am I doing wrong?

It failed again with the new forced push in check-valid-pr. For some reason it checks  the old commit 30aa0d455262d643d0ab6e01f7db729ffc347d45 against the current HEAD 61130c72ed6756773981b55458e7d0fe3712c138, and that seems to fail because probably  headroom <= 1.  Maybe just rerunning helps?

It failed again with the new forced push in check-valid-pr. For some reason it checks the old commit 30aa0d4 against the current HEAD 61130c7, and that seems to fail because probably headroom <= 1. Maybe just rerunning helps?

That is odd, I haven't seen this check fail before. I'll investigate if it fails again there


Module Resolution: Planned Feature

Planning after some clean up, start working on module resolution, and other project scope items

Finish github online and file offline, bitbucket is next, private probably not possible(or too much trouble)


Update markdown.py

Type of changes

[x] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[ ] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[x] I accept that @willmcgugan may be pedantic in the code review.

Description
Fix the extra blank lines that appear when displaying tables:

After modified:

I can't reproduce this. Assuming fixed.

Here's a quick test which hopefully demonstrates this isn't fixed on master:
def test_table_with_empty_cells():
    """Test a table with empty cells is rendered without extra newlines above.
    Regression test for #3027 https://github.com/Textualize/rich/issues/3027
    """
    complete_table = Markdown(
        """\
| First Header  | Second Header |
| ------------- | ------------- |
| Content Cell  | Content Cell  |
| Content Cell  | Content Cell  |
"""
    )
    table_with_empty_cells = Markdown(
        """\
| First Header  |               |
| ------------- | ------------- |
| Content Cell  | Content Cell  |
|               | Content Cell  |
"""
    )
    result = len(render(table_with_empty_cells).splitlines())
    expected = len(render(complete_table).splitlines())
    assert result == expected

So it is empty cells that is the issue? Could have used that in description!

So it is empty cells that is the issue? Could have used that in description!

Sorry this isn't my PR and it has been several months. I agree this could have been clearer, this is the relevant issue:

#3027

Thanks. Fixed in main.


I can't use it with my hp notebook.

It sometimes doesn't work in my Hp notebook, The exact model is 450 G8，plz... what can i do with this issue

and this is my touchpad settings.

and the app settings

Do you think that this is a duplicate of #52 ?

Do you think that this is a duplicate of #52 ?

yeah, Our problem are similar. Thank you and I'll try to use the version 2.0.2


Consider all protein coding genes in the vicinity of a credible set to extract colocalisation neighbourhood features

As a developer, I want to include all protein-coding genes in colocalisation neighbourhood features, assuming an H4/CLPP of 0 for non-colocalised genes. This will create a more accurate baseline and boost feature importance for features in the neighbourhood.
Background
Colocalisation neighbourhood features calculate the average H4/CLPP for a credible set across genes to set a baseline. The genes that are considered to extract this metric are only those that are present in the colocalisation results.
To create a more accurate representation of what is the average value in the region (to then substract that to the local metric), we want to expand the set of considered genes to all protein coding genes. We will assume that their H4/CLPP is 0. This will push down the average metric for all genes, so that differences between the local and the neighbourhood metrics are not so extreme. We therefore expect to see a boost in the importance of these features.
Tasks

[ ] Update the logic in the neighbourhood
[ ] Measure effects on feature importance
[ ] Measure effects on average metrics

The implementation won't be so difficult as a result of the work in https://github.com/opentargets/issues/issues/3552, as gene_index is already a dependency of the colocalisation features.

As we discussed, this is just a change in the denominator that calculates the mean.
Suppose you have 10 protein-coding genes, but you only have two coloc results for two genes, e.g. h4_1=0.3, h4_2=0.4.
For now, the mean is calculated as E=(0.3+0.4)/2, but it should be E=(0.3+0.4)/protein coding=(0.3+0.4)/10


Try to not mix languages

It would be good if the comments were only in english for others to understand your code.

Thank you @cleudson


Resource Day and Week View

Is there a way for me to drag across multiple days in the day or week view with or without multiple resources? For example, if I drag down on a column from 1pm to 3pm, and then drag across the other columns horizontally, it will fill 1-3 on those columns. If it's not already implemented, where at in the code should this be implemented and I can make a pull request. Thanks.

stale


Cargo: version 0.2.0 -> 0.3.0

This branch prepares a 0.3.0 release, primarily to publish the rustls update (https://github.com/rustls/rustls-platform-verifier/pull/70)
Proposed release notes

Updates Rustls from 0.22 to 0.23.

Post-merge steps

[x] Generate Android Maven artifacts locally - N/A - no non-test changes made under android/
[x] Create and push Git tag - created v/0.3.0 tag
[x] cargo publish for each required crate, based on release steps - Published rustls-platform-verifier v0.3.0 at registry crates-io
[x] Create companion GitHub release - created v/0.3.0 release

Released!


Hydrogen wont import submodules in autocomplete

My Project folder
foo\

init.py
bar1.py
barz\

init.py
bar2a.py
bar2b.py

If I do "import foo" it runs fine, but none of the submodules show up in autocomplete

This needs more investigation. Anyone should feel free to investigate the source code for a fix or just more information


Add one constructor to AllColumns for hqltranslator usage

Add a constructor to AllColumns which only takes prefix as input. This was original supported in previous presto version.

Looks good.

BTW, this is not a very good commit message. The hql translator is internal and meaningless to anyone else following the project. A better message would've been "Add missing constructor to AllColumns" and in the details say "Allows callers to construct an AllColumns node with no position information"


Reading I04's GDA beamline_parameters currently errors as it tries to set a list as a float

When running Hyperion on I04, /hyperion/parameters/beamline_parameters.py reads GDA's
/i04-config/scripts/beamlineParameters and reads in some of the parameters as a float. This errors when these parameters are in a list format (eg the flux_predict_polynomial_coefficients parameters so we need a way to handle this.

We should:

Handle the list in that file
Log a warning if there's some other line in the file we can't handle but carry on anyway


Support for Kotlin 1.9 and 2.0 in flutter module template

Use case
Hi! My name is Reberth, and I'm a software engineer from XPInc, Brazil.
Our Android project is integrated with Flutter trough flutter module; it's a large native codebase project where we are migrating to 100% Flutter.
Context
The native side setup is built with Kotlin 2.0 and AGP 8.2.2 and we are doing well  using flutter 3.24.0, which uses Kotlin 1.7.10 and agp 7.3.0, defined from default variable templateKotlinGradlePluginVersion and templateAndroidGradlePluginVersionForModule respectively, variables used to generate the .android folder for every run of flutter pub get or flutter build aar.
Problem
We are bumping our code base Flutter (plugins and third-party libraries) to Kotlin 2.0 and AGP 8.2.2 (like the native project setup), but we are not able to build the aar after upgrading the dynatrace_flutter_plugin. Which is reproducing the following output:
e: file:///Users/xxx/.pub-cache/hosted/pub.dev/dynatrace_flutter_plugin-3.305.2/android/src/main/kotlin/com/dynatrace/android/agent/SettingsApiImpl.kt:32:21 This declaration needs opt-in. Its usage must be marked with<EMAIL_ADDRESS>or '@OptIn(kotlin.ExperimentalStdlibApi::class)'
e: file:///Users/xxxx/.pub-cache/hosted/pub.dev/dynatrace_flutter_plugin-3.305.2/android/src/main/kotlin/com/dynatrace/android/agent/SettingsApiImpl.kt:32:41 This declaration needs opt-in. Its usage must be marked with<EMAIL_ADDRESS>or '@OptIn(kotlin.ExperimentalStdlibApi::class)'
e: file:///Users/xxx/.pub-cache/hosted/pub.dev/dynatrace_flutter_plugin-3.305.2/android/src/main/kotlin/com/dynatrace/android/agent/SettingsApiImpl.kt:32:41 The feature "enum entries" is only available since language version 1.9

This error indicates that the build of aar is using a Kotlin version below than 1.9.0; that is true, because in Flutter 3.27.0 the Kotlin version for templates is 1.8.22.
Steps to reproduce
The error above is reproduced using flutter 3.27.0.

Create module: flutter create -t module sample_module
Add dynatrace plugin: cd sample_module && flutter pub add dynatrace_flutter_plugin
Build aar: flutter build aar

Note: The same occur using Flutter versions 3.24.0 and 3.24.3

Our proposals
Following the above scenario, we are requesting some of the following options to solve this problem:
1. Flexibility template options
Extend the functionality of pubspec yaml field module to support the definition of Kotlin, AGP, Gradle and the android.compileOptions.targetCompatibility/sourceCompatibility through YAML. Example:
name: sample_module
description: lorem epsum
version: 1.0.0+1

environment: ....

dependencies: ...

flutter:
   module:
        androidX: true
        androidPackage: br.com.sample_module
        ...
        kotlinVersion: 2.0.0
        agpVersion: 8.2.2
        gradleVresion: 8.2-all
        javaTargetCompatiblity: 17
        javaSourceCompatibility: 17

2. Create a toggle to prevent .android folder recreation
The Android setup has a note warning about the auto generated .android folder. It will be great if exists an toggle to turn off this auto-generation, so we can manually adjust the project and push to our repository.
3. Create a template that's supports kotlin 2.0 and AGP 8.x
Create a new template, adjusting setting up JAVA_17 and removing AndroidManifest.xml package field. Details here
Conclusion
I hope you guys understand our scenario and the steps to reproduce the error. Any doubts, touch me through e-mail<EMAIL_ADDRESS>Best Regards,
Reberth Kelvin
Mobile Software Engineer III | XPInc

cc @bkonyi


Need to confirm phone number twice

Problem
when 2FA is enabled with a phone number and you add the same phone to the recovery methods you receive 2 SMS with codes, one confirming adding a phone number and the second one for 2FA. Same for email.
Expected Behavior
When you creating an account and starting with phone number as a recovery and then adding the 2FA with the same phone number you have to confirm only once, so it should work the same.
Steps to reproduce

Go to /create page
Enter the account id
Activate seed phrase recovery
Enable 2FA with phone\email
Enable phone\email recovery method
You will receive 2 SMS to confirm

@kcole16 seems like very low priority, as during normal account creation you first setup recovery and only then 2FA. As I understand users can still complete the task as well (just slower).
@ihaid does this only happen with phone or e-mail as well?

@ihaid does this only happen with phone or e-mail as well?

@vgrichina email as well, if for 2FA you have a phone then it happens with email for recovery and vice versa.
And yes, the user still can complete the task just will be bothered by waiting for two letters/SMS in a row.

if for 2FA you have a phone then it happens with email for recovery and vice versa.

@ihaid  hmm, not sure I understand this correctly. If you use 2FA with a phone but setup e-mail recovery, then it is expected that you both need to verify code on phone (to confirm 2FA) and code from e-mail (to confirm e-mail).

@ihaid hmm, not sure I understand this correctly. If you use 2FA with a phone but setup e-mail recovery, then it is expected that you both need to verify code on phone (to confirm 2FA) and code from e-mail (to confirm e-mail).

Sorry, it was a typo. I edit the comment, now everything is correct.

@ihaid can you check if this issue was fixed?

@ihaid can you check if this issue was fixed?

Yes, still exists


[build] Avoid building too many extensions

This PR lets vllm only build one extension and reduces the build time by 75% (from ~5.5min to ~1.5min) based on my test.

Hi @ymwangg, sorry for the late review. Could you resolve the merge conflicts before I get into the review? No worries if you can't. We can do it ourselves if you can't.

@WoosukKwon sure, I've rebased.


feat: add hello world

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Updates orchestrator and web to use url instead of IP address or hostname

Closes #2142
This PR:

Updates the orchestrator and web server to take url arguments instead of host arguments.  The url arguments include the network protocol, such as HTTP or HTTPS.  Previously everything was hard-coded to always use HTTP.
Updates the network configuration file accordingly so that it also takes in url instead of hostname or IP address.

This PR does not:

Fix other tech debt issues in the web server and orchestrator

Key places to review:

orchestrator/lib.rs
web_server/lib.rs
examples/infra/mod.rs

I have to update some of the examples to use the URL instead , will make those changes and piggyback on this PR

I have to update some of the examples to use the URL instead , will make those changes and piggyback on this PR

Oh I might have just done this now!  Sorry, I didn't see your comment until now.

Works for me !
It might be better if we had the URL include the port (and then had the examples default to <IP_ADDRESS> with the specified port), but this is a step in the right direction

Yes, we should do this!  I'm going to do this now, since it will actually make fewer changes in the sequencer repo.  I'll switch everything to using the Url type instead of strings and u16s.


Review and Update the Administer section doc structure

Description:
Review and Update the Administer section doc structure

This issue was addressed when fixing another issue.


Efficient dependency detection for safe Java test acceleration

http://dl.acm.org/citation.cfm?id=2786823&CFID=725626965&CFTOKEN=17951443

This paper uses data of  popular java projects from open repository site, GitHub. The paper doesn't provide any link to dataset. . There  is not dataset available in this paper. A list of 921 Java projects are used in this paper. A email should be sent to the authors.


Missing execute permission for Linux build

Hi, the output for Linux builds does not contain the execute permission flag.

Can you provide this information?
The only way I can reproduce this issue is by building on Windows, which is likely trashing permissions from the Linux distributions of Electron. :/  On OS X and Linux the Linux executables seem to have proper permissions.

Sorry! This happens after grunt contrib copy job.


Map should work on IE

The map with all the spaces should work on IE Version 11, 11.0.9600.18537. If it's easy, map should also work on v.10 and maybe v.9.
Current Outcome: when opened with IE, the map is blank and all the spaces are listed in text format below the map.

The map is supported in Internet Explorer to version 10 as of #5

Oh no, something got re-adjusted and the spaces don't show up anymore! Can you fix this again, please?

Hmmm, the changes I made are still in the master branch. Are you running it locally or on the heroku instance? It seems that https://ananse-map.herokuapp.com/makermap/index hasn't been updated since these changes were made.
cc: @chinmayisk

Ah, that would explain it. Can you please update the heroku instance so I can email the Germans?

On Jan 19, 2017, at 8:46 AM, Will Newman<EMAIL_ADDRESS>wrote:
Hmmm, the changes I made are still in the master branch. Are you running it locally or on the heroku instance? It seems that https://ananse-map.herokuapp.com/makermap/index https://ananse-map.herokuapp.com/makermap/index hasn't been updated since these changes were made.
cc: @chinmayisk https://github.com/chinmayisk
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/AnanseGroup/map-of-innovation/issues/1#issuecomment-273779842, or mute the thread https://github.com/notifications/unsubscribe-auth/ASnZe_FM31aN30ylEV24YyUvd4-J-dm-ks5rT2kwgaJpZM4LgJob.


Capture fully resolved footnote backlinks. Extends #3084

If a footnote backlink has a fully resolved URL, not just a detailed same-page # link, it was not caught by the recent patch. The need for this has now been identified in the RSS feed for SixColors.com. (The original bug report was for the json feed, which has slightly different link formatting.)

Exactly what this does:
• In the footnote hover box, it will now hide a link if #fn appears anywhere in the link, not just at the start.
• In the footnote section itself, it will continue to capture all links that are page-relative but will also capture fully resolved links (containing a domain name at the start) if they have a page-location-target that contains #fn as well. In the wild chance that someone links to an external site, with a page anchor in the url, and that page anchor contains #fn then that link will fail to work. I say that's just tough. The much more likely scenario is the one we've encountered here, with a base url on the backlink, that doesn't match the canonical url for the article, and where the link should be treated as an on-page link still. This does that, with zero performance effects.


configure renovate

we should use renovate to help us manage updating dependencies. We can use the getting started page as the starting point and tweak it as we go

looks like the recommend way to use renovate on github is with the renovatebot github action. I will add that action to the repo