verrified

hi, just a question:
are the accounts verified, or only work for 2 days?

Not verified, although that's something I'm planning on.  Unfortunately I haven't had the time to implement it due to more pressing life issues.  If anyone can get that working, feel free to submit a pull request.  Otherwise I hopefully should be able to get something working in a few days.

I started working on a wrapper to make this happen. I added @jepayne1138 as a collaborator on the project to review and okay it.

If you use gmail, this script will open all emails in your inbox containing an activation link (original script found here)
/*
  Automatically click all "Verify your email" links in the welcome e-mail from
  Nintendo Pokémon Trainer Club's signup e-mails.

  How to use:
   1. Login to Gmail
   2. Go to https://script.google.com/
   3. Enter the code, save, run.
   4. Wait until fully completed, DO NOT LEAVE THE PAGE! (When finished, the text "running..." at the top will disappear.)
   5. Click View > Logs. At the bottom you'll see the total accounts verified.
   6. Enjoy
*/

function myFunction() {
  var verified = 0;

  var threads = GmailApp.getInboxThreads();
  Logger.log("Found " + threads.length + " threads.");

  threads.forEach(function(thread) {
    var messages = thread.getMessages();
    Logger.log("Found " + messages.length + " messages.");

    messages.forEach(function(msg) {
      if(msg.getSubject() === "Pokémon Trainer Club Activation") {
        var value = msg.getBody()
                       .match(/Verify your email/m);

        if(value) {
          var link = msg.getBody().match(/<a href="https:\/\/club.pokemon.com\/us\/pokemon-trainer-club\/activated\/([\w\d]+)"/);

          if(link) {
            var url = 'https://club.pokemon.com/us/pokemon-trainer-club/activated/' + link[1];

            var status = 0;

            while (status !== 200) {
              try {
                status = UrlFetchApp.fetch(url).getResponseCode();
              } catch (e) {
              }
            }

            Logger.log("[#] Verified (" + status + "): " + url);

            if(status == 200) { verified++; }
          }
        }
      }
    });
  });

  Logger.log("Completed " + verified + " verifications.");
}


Background tasks execution on iOS may look like evenly distributed across the day, but it is not guaranteed. And there's a bug when "Last checked at" executed before 18:00, but no other checking between 18-22, then scheduleNotificationIfNeeded() will still schedule a "no exposure detected notification" at 22:00 regardless of shouldNotifyEvenNoRisk settings.

Thank you for the feedback, it's really helpful!
A new version 1.6.1 is already submitted in app store.

Closed with https://github.com/ailabstw/social-distancing-ios/commit/317025d808a9c90c6525f4965229c646657418ad.

Here's another logic that may bring a "No exposure detected notification": when riskStatus changed from risky to clear. Related code here.
Was this be design? Shouldn't it respects shouldNotifyEvenNoRisk settings as well?

Yes! It should respects shouldNotifyEvenNoRisk as well.
Thank you for the feedback.

Closed with https://github.com/ailabstw/social-distancing-ios/commit/42c78e292a8cac57ce63d87b6a9eb77b1e668972 👏🏼


Add reconnect to Client

Got tired of implementing reconnects in my apps and figured the addon might as well support it. I added some new options to ClientOptions so you can enable/disable the reconnect option and the timing (defaults to on, 1s).
This seems to work fine with a reconnect time of 1s, but I get some weird crashes ~half the time when I lower it.  Maybe you've got a better idea of what's going on here?

Not sure why this is failing...the debugger shows context has a memory address/is not NULL, and the code only gets here after startThread() is called sooo.... ¯\_(ツ)_/¯

Looks good, thanks for this!
Not sure on the error. TBH there's something weird going on with ofThread development lately. Maybe open a separate issue for that?


[Python] Python bindings for the new C++ AzureFilesystem

Describe the enhancement requested
Add python bindings so that the new C++ AzureFilesystem can be used from Python code. abfs:// and abfss:// URIs should automatically select the AzureFilesystem.
Work on the C++ AzureFilesystem is tracked by https://github.com/apache/arrow/issues/18014
Component(s)
Python

My custom build of pyarrow actually has pybindings like this. I can probably make a PR for this within the next week or two.


use MSI version of wireshark for easier deployment and add information to manifest

…emAppId
Microsoft Reviewers: Open in CodeFlow

closed in favor of #1498


Create HAL link for Events with Type

Problem Description:
Concept pages needs a HAL link for Events with this Type.
Expected Behavior/Solution:
Once HAL link is created it will show Events with this type on the concept pages.
Requirements:
When visiting a concept page, an Event with this Type accordion will appear where available.
Needed for promotion:
If an item on the list is not needed, it should be crossed off but not removed.

[ ] Wireframe/Mockup - Mike
[ ] Committee discussions - Sarah
[ ] Feasibility/Team discussion - Sarah
[ ] Backend requirements - TBD
[ ] Frontend requirements- TBD
[ ] Questions
List of questions for discussions. Answers should be documented within the issue.

UAT/LUX Examples:
Once HAL link is created accordion "Events with this Type" will display:
https://lux-front-tst.collections.yale.edu/view/concept/88af90e3-d194-4bfb-aa8c-0998cdf7e679
Dependencies/Blocks:

Blocking: https://github.com/project-lux/lux-frontend/issues/107

Related Github Issues:
Related links:
~Wireframe/Mockup:~
~Place wireframe/mockup for the proposed solution at end of ticket.~

Closing as this issue is covered by #34

s


Add testcase for when 0 deposit is needed

Add test to make sure proposals still work when the deposit is configured to zero. There should be no need to set an allowance on the cw20 token.

Answered this question in the PR, existing tests cover this case, but could be worth it to write an explicit test to document the requirement https://github.com/JakeHartnell/cw-dao/pull/11#discussion_r702476520

This done.


chore: remove FF4J

Description

Remove FF4J

PR fixes following issue(s)
Fixes #24872

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Chore (housekeeping or task changes that don't impact user perception)

Testing

How Has This Been Tested?

[ ] Manual
[x] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

Summary by CodeRabbit

Refactor

Simplified the feature flag checking process, removing dependencies on external libraries.
Improved error handling in feature flag services for more robust operation.

Bug Fixes

Adjusted feature flag refresh logic to ensure accurate status representation.

Tests

Updated test cases to align with the new feature flag checking logic and error handling improvements.

Documentation

Removed outdated comments and documentation related to the old feature flag system.

/ok-to-test

/ok-to-test


Update building.md

An I remove the Debian instruction in building.md and add a link to the Debian8 instructions or not?

Thanks :smile:.


error 0x80030002 %1 could not be found

After changing the the .zip extension to use the file meta handler via the simple profile I am unable to add a tag to zip files.
The properties > details tab is changed as expected as is the details pane view.
However whenever I add a tag on either view and attempt to commit it via apply or save buttons I get an error pop up window with the following details,
An unexpected error is keeping you from applying properties to this folder. If you continue to recieve this error, you can use the error code to search for help with this problem.
Error 0x80030002 %1 could not be found.
EDIT: After some more playing about I've discovered that I can tag files on my C drive but not my mapped NAS drive. Also when I tag a file on the C drive and move it over to the NAS it loses the tag data. The NAS drive is using the NTFS file system so I'm unsure as to whats going on. Perhaps I'm misunderstanding some of the limits of Win 10's tagging system?
EDIT 2: Reading around this issue appears to be a NAS compatibility issue in regards to how though-roughly the NAS drive supports alternate data streams (ADS). I've therefore updated my NAS to the latest firmware (something I've avoided due to worry over issues but thankfully it went without a hitch). This made no change... but after reading the patch notes (a few years worth) I spotted a line which looked promising and it led me to the "Enable Samba VFS modules to enhance macOS compatibility" setting within my NAS settings.
This has allowed me somewhat to tag files on my NAS haha. Atm I'm getting a new error when I add a tag "Error 0x80030020: A share violation has occurred." but if I try again or skip that error it does appear to tag the file... though searching for it appears to crash my explorer window! A work in progress. I'll add another edit if I work through the remaining issues. Suffice it to say the problem isn't on the software end but rather lack lustre support for ADS on NAS drives.

You are right. Other users have found that some NAS drives have rather patchy NTFS implementations. The problem is missing support for built-in property storages in alternate streams. The Windows API allows such storages to be opened with a single call, but of course it will only work if the underlying firmware knows how to play the game. It sounds like you are on doing exactly the right things to try to find a solution.
Dijji

You guys are facing problems in other things, here's what's happening with me :-
I have been trying to send files to my phone, like transferring it from my laptop to phone's internal storage or SD card. And when I try to delete few stuffs of my phone, from my laptop, it shows "error 0x80030002 %1 not found".
GAAH! This thing is irritating! 💢 someone please help 😞


Auto comment doesn't always trigger

Describe the bug**

When pressing Enter on a line doc comment /// ... line Ionide should add ///  at each Enter. It seems to do it every other time

I didn't tested yet if this was a problem in Ionide or in VSCode as Ionide doesn't seems to do something special for this feature.
Machine info

OS: Windows
.NET SDK version: [e.g. 5.0.100] 5.0.101
Ionide version: [e.g 5.0.1, don't use descriptions like latest ] v5.4.0

For info, I created an issue on https://github.com/microsoft/vscode/issues/121125 because even when using an empty extension I have this problem.

This seems to be fixed in VSCode.


Speed up is_dir on entries returned by iterdir and glob

The is_dir check is fairly expensive, but at least for S3 and Azure when the entries were created as a result of the client's _list_dir method, you can tell for each entry whether it is a directory or a file and immediately set the result on the created CloudPath instance.
For example for the S3Client._list_dir, you could write something like:
            paginator = self.client.get_paginator("list_objects_v2")

            for result in paginator.paginate(
                Bucket=cloud_path.bucket, Prefix=prefix, Delimiter="/", MaxKeys=1000
            ):

                # sub directory names
                for result_prefix in result.get("CommonPrefixes", []):
                    path = S3Path(f"s3://{cloud_path.bucket}/{result_prefix.get('Prefix')}")
                    path._is_dir = True
                    yield path

                # files in the directory
                for result_key in result.get("Contents", []):
                    path = S3Path(f"s3://{cloud_path.bucket}/{result_key.get('Key')}")
                    path._is_dir = False
                    yield path

and modify S3Path.is_dir:
    def is_dir(self) -> bool:
        if self._is_dir is None:
            self._is_dir = self.client._is_file_or_dir(self) == "dir"
        return self._is_dir

This makes a HUGE performance difference if you need to call is_dir on the entries returned from iterdir or glob (in my case, when implementing a file dialog that works for cloud paths).
Not sure if this particular implementation is the best way to do this, but something like this is needed.

BTW, it probably would also be a good idea to cache other information returned in the directory entries that could be used to populate the stat() call, e.g. the file size and last modified time and perhaps the etag as well.

Note that methods that create an entry should make sure to fix the cached state appropriately.


How to force re-render when dynamically changing localization

I am using the hl property on the Recaptcha component, however, the component will not re render after it has been rendered once.
Any way we can force re rendering the component when changing the locale?
Thanks

https://developers.google.com/recaptcha/docs/display#js_api
grecaptcha.reset(
opt_widget_id
)
Resets the reCAPTCHA widget.
opt_widget_id
Optional widget ID, defaults to the first widget created if unspecified.

It doesn't work with me when I am user grecaptcha.reset() function!
Any other ideas
Thanks


prevent object id collision in hide_secrets

This fixes an issue related to object id reuse that can occur making the object identification cache fail.  Instead, this simplifies hide_secrets to always recurse and use setattr to always set the value based on the recursion.
Note, the object id reuse issue was seen in the events.filter_event_recurse development and this was the fix for the id reuse there.
Python documentation states:

id(object):
Return the “identity” of an object. This is an integer (or long integer)
which is guaranteed to be unique and constant for this object during its
lifetime. Two objects with non-overlapping lifetimes may have the same
id() value.

@ranweiler this should fix the build issues you're seeing.


[infra-netif] select infrastructure link among multiple candidates

This commit implements a general Infrastructure Link Selection that allows otbr-agent to select one network interface among multiple network interface candidates (specified by multiple -B arguments) as the infrastructure link.
The Infrastructure Link Selection uses following rules:

The network link in the most usable state is selected

Prefer up and running to up
Prefer up to down
Prefer down to invalid

Once an interface is selected, it's preferred if either is true:

The interface is still up and running
No other interface is up and running
The interface has been up and running within last 10 seconds

otbr-agent uses Infrastructure Link Selection to select the correct infrastructure link:

Run Infrastructure Link Selection when started to select the infrastructure link
Run Infrastructure Link Selection when network interface states change is detected. otbr-agent would quit if a different infrastructure link is selected, expecting the system to restart it to use the new infrastructure link.

Note:

The Infrastructure Link Selection mechanism is effectively disabled if less than two infrastructure link names are specified.
The current implementation only works for Linux platforms.

@bukepo @superwhd
Thanks for reviewing.
I have refactored the infra link evaluation implementation significantly in the last commit because of :

To fix the issue @bukepo has caught that multiple QueryInfraLinkState may return different states
To fix a bug that ILS does not react when an interface turn UP and RUNNING
To reduce calling to QueryInfraLinkState by running it only after interface state is changed
To reduce selection by only re-select when the states of an infra links did change.
To fix the logic issue in last running time evaluation.

Please take a look again. Thanks.

@wgtdkp test added in the 4th commit https://github.com/openthread/ot-br-posix/pull/1353/commits/ef2652f8b14660f76fa009f31236c765cab96a55

@samadDotDev any further comments?

@jwhui  If multi-network interfaces,  how to setup trel paramaters? just like trel://$ETH_INTF1  trel://$ETH_INTF？


Fix for hidden columns bug from main repo

Can we pull in this pull request from the main repo?
https://github.com/mbrn/material-table/pull/2655
Original issue here:
https://github.com/mbrn/material-table/issues/2654

@charlesc32 it was a quick fix so I just went ahead and added that line.
v2.2.2 has been published.

@oze4 Did you checked another PR, for the same issue?
https://github.com/mbrn/material-table/pull/2493
It looks like better fix for it.

@bva I would have to agree with you. v2.2.3 has been published.

Reverted changes made in v2.2.2
Made these changes


MMB: Fix raw text bug on flight services

closes #660

Could you please write tests for this fix?


Before Start of Upload Dynamic Action does not fire

Expected behavior
Dynamic Action "Before Start of File Upload" is suppose to fire before upload process, the idea is, to force the user to set an  item before the upload starts (e.g. to set a document category etc.)
Actual behavior
DA does not fire
Steps to reproduce the issue
Create Dropzone , add DA with event before start of file upload, Selection type: region, Affected elements: Dropzone region.
APEX version (4.2.6 / 5.0.3)
5.03
Used web server / version and platform (ORDS 3.0.3 / Tomcat 7 / Apache 2.4 / Linux x64)
ORDS 3
Used web browser / version and platform (Chrome 48 Mac / Firefox 44 Windows)
Firefox / IE

Hi @hairpin666
such an event does not exist for the Dropzone Plugin...
In the Readme are all available Events listed:
https://github.com/Dani3lSun/apex-plugin-dropzone#plugin-events

Oh, must have been the wrong plugin then :). Having this DA would be great though.


[PROPOSAL] function to pick subtable containing only keys from the specified list

There is tablex.index_by (tbl, idx) which returns list of values from specified keys. What about adding tablex.pick (tbl, keys_list) which would return a table with specified keys instead? Similar to JavaScript lodash.pick

I don't get it. If you specify the keys to get the keys, then you already have the keys before incoming that function, no?
If you want to return the values, then the current function already does that...


HiDPI cursors

There doesn't seem to be a way of creating/setting a HiDPI cursor

What is that?

Well, on OSX with a HiDPI display, when setting a 16x16 cursor for example, you end up with a 32x32 one instead (i.e. the cursor seems to be defined in window pixels, instead of framebuffer pixels)

Bump. Is there any work going on for this? If not, please let me know and I'll look into creating a PR for this, at least for macOS.


User Story: Landing Page

As a user i want a landing Page in order to choose which and what kind of board I open

[x] sidebar
[ ] sidebar functionallity
[ ] save recentboards
- [ ] local
- [ ] github
[ ] show recent boards page
- [ ] local
- [ ] github
[ ] open board page
[ ] new local board page
[ ] manage local boards page
[ ] reset user account page
[ ] athentication page

story to big will be splitted in short ones


links lead to advertisement page

I clicked on many of the styles listed, and got directed to an advertisement page. Impossible to get to the styles!

You forgot to mention which site you are on.

See https://github.com/ankit/stylebot/issues/352
There has been an exploit in the comments section for ALMOST TWO YEARS. Someone injected their own code and has been getting advertising clicks for maybe even longer.
The developer seems to have been aware of this since July 2014, when his "Security Researcher" pointed it out to him: https://github.com/ankit/stylebot/issues/340
I DO NOT ADVISE you use this extension, who knows what else has been  ignored for almost two years.

Somebody posted this <meta http-equiv="refresh" content="0;URL=http://www.clixsense.com/?68706xxx"> in the comments of popular styles. This leads to the immediate redirect.
Unfortunately there seems to be no solution - maybe hitting ESC fast enough before the comment gets executed.

Last time I checked there is no support for meta tag interception in Chrome and I wouldn't hold my breath for that to come anytime soon. Have been requested for ages.
For problematic sites use the trusty good old Firefox with NoScript, Adblock Plus and NoRedirect add-ons. A replacement for Stylebot in Firefox is Stylish.
Good tip is to surf problematic urls in Private mode.
Just tested in Firefox the url in #352 and no redirect.


EDU-920 batch room invitations

Closes https://educandu.atlassian.net/browse/EDU-920

Maybe we should put an info icon next to the input label, telling the user what separators they can use between email addresses.

Maybe we should put an info icon next to the input label, telling the user what separators they can use between email addresses.

Nice idea! Done.

Maybe we should put an info icon next to the input label, telling the user what separators they can use between email addresses.

Nice idea! Done.

👍 Cool
Should we in this case move it to the end of the label? Looks rather weird with the required mark.

Or maybe we can remove the requirement altogether and just not do anything when the user presses "Send" without any emails written?

Maybe we should put an info icon next to the input label, telling the user what separators they can use between email addresses.

Nice idea! Done.

+1 Cool Should we in this case move it to the end of the label? Looks rather weird with the required mark.
Or maybe we can remove the requirement altogether and just not do anything when the user presses "Send" without any emails written?

I actually did both as it looks best. Validation will kick in anyways when no email is specified.

Maybe we should put an info icon next to the input label, telling the user what separators they can use between email addresses.

Nice idea! Done.

+1 Cool Should we in this case move it to the end of the label? Looks rather weird with the required mark.
Or maybe we can remove the requirement altogether and just not do anything when the user presses "Send" without any emails written?

I actually did both as it looks best. Validation will kick in anyways when no email is specified.

Wunderbar!


Improve Apollo by detecting script with state

As seen on this site: https://www.eneba.com/
And it's documented in the docs: https://www.apollographql.com/docs/react/performance/server-side-rendering/#executing-queries-with-getdatafromtree

Thanks! I updated it to use dom instead of html.


Add support for matching on any search token

This PR adds support for search queries that can match on any search token, i.e. OR queries.
Search support for this feature was added in https://github.com/bvaughn/js-worker-search/pull/19 (version 1.4.0)

@bvaughn  Thanks again for the quick response in js-worker-search.  Thanks!

Thanks! :)
Published in 2.5.0


Defect: PO-Revision-Date being removed causes validation errors in 1.x release

We recently updated the edx-i18n-tools package in the Credentials IDA and it appears to be causing issues. We have temporarily pinned the version to the 0.9.x release (which resolved our CI issues).
It looks like the PO-Revision-Date attribute in the file (headers?) are being removed now. When validation is run using this package, it is flagging the missing PO-Revision-Date data as a warning, which causes the validation to fail with an error. Since the validation exits with an error code, it is causing issues in the Credentials CI.
I have a PR demonstrating the issue here: https://github.com/openedx/credentials/pull/2085/files.
The failing CI checks can be seen here: https://github.com/openedx/credentials/actions/runs/5588178826/jobs/10214553458?pr=2085
Example logs from the failing validate:
INFO:i18n.validate:

WARNING:i18n.validate:
en/LC_MESSAGES/djangojs-partial.po:7: warning: header field 'PO-Revision-Date' missing in header

INFO:i18n.execute:msgfmt -c -o /dev/null en/LC_MESSAGES/django.po
INFO:i18n.validate:

WARNING:i18n.validate:
en/LC_MESSAGES/django.po:7: warning: header field 'PO-Revision-Date' missing in header

INFO:i18n.execute:msgfmt -c -o /dev/null en/LC_MESSAGES/djangojs.po
INFO:i18n.validate:

WARNING:i18n.validate:
en/LC_MESSAGES/djangojs.po:7: warning: header field 'PO-Revision-Date' missing in header

INFO:i18n.execute:msgfmt -c -o /dev/null en/LC_MESSAGES/django-partial.po
INFO:i18n.validate:

WARNING:i18n.validate:
en/LC_MESSAGES/django-partial.po:7: warning: header field 'PO-Revision-Date' missing in header

INFO:i18n.execute:msgfmt -c -o /dev/null ro/LC_MESSAGES/django.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/ro/LC_MESSAGES/django.po
INFO:i18n.execute:msgfmt -c -o /dev/null ro/LC_MESSAGES/djangojs.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/ro/LC_MESSAGES/djangojs.po
INFO:i18n.execute:msgfmt -c -o /dev/null nb/LC_MESSAGES/django.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/nb/LC_MESSAGES/django.po
INFO:i18n.execute:msgfmt -c -o /dev/null nb/LC_MESSAGES/djangojs.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/nb/LC_MESSAGES/djangojs.po
INFO:i18n.execute:msgfmt -c -o /dev/null fa_IR/LC_MESSAGES/django.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/fa_IR/LC_MESSAGES/django.po
INFO:i18n.execute:msgfmt -c -o /dev/null fa_IR/LC_MESSAGES/djangojs.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/fa_IR/LC_MESSAGES/djangojs.po
INFO:i18n.execute:msgfmt -c -o /dev/null pl/LC_MESSAGES/django.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/pl/LC_MESSAGES/django.po
INFO:i18n.execute:msgfmt -c -o /dev/null pl/LC_MESSAGES/djangojs.po
INFO:i18n.validate:No problems found in /home/runner/work/credentials/credentials/credentials/conf/locale/pl/LC_MESSAGES/djangojs.po
make: *** [Makefile:168: validate_translations] Error 1
Error: Process completed with exit code 2.

Update as this came up again very recently.
The validate functionality in this library calls out to msgfmt: https://github.com/openedx/i18n-tools/blob/851c02f7a87d1ec146a2932413b43d58ba8923d0/i18n/validate.py#L76.
Specifically, the -c flag ensures that the format, headers, and domain directives pass. I think this is where we are running into issues. We're using msgfmt to check the headers, which I think expects PO-Revision-Date to be present?

https://github.com/openedx/i18n-tools/pull/129 seems to be the culprit. @shadinaif @OmarIthawi any thoughts?

Thanks for the extra eyes and your suggestions on this, @OmarIthawi.
Unfortunately, we can't follow your first suggestion -- the -c flag is coming from this library itself, not specified in Credentials.
The reason this is turning into a problem now is that we were told that upgrading to 1.x of i18n-tools is required for the Django 4.2 upgrade in the Open edX IDAs. We have Credentials pinned to a release of i18n-tools < 1.x, but now we're blocking the Django upgrade now. I
We can try and look at suggestion #2, but I expect that the i18n tool will just strip it out again?

We can try and look at suggestion no. 2, but I expect that the i18n tool will just strip it out again?

@justinhynes This behavior will be added to i18n_tool extract. That should pass the -c, we'll add a test case to ensure that whatever the i18n_tool extract generate passes the validate command. It makes sense that both commands are "on the same page" so to speak.
Again, apologies for breaking the workflow. no. 2 seems to be the way to go. So if others agree we can move forward and document this decision as soon as possible.
cc: @shadinaif and @brian-smith-tcril what do you think?

Thank you for the details @justinhynes . I was able to reproduce the error locally. The fix is on the way. I tried it and now quality_and_translations_tests job is passing successfully on my local machine
cc: @OmarIthawi @brian-smith-tcril

@justinhynes We've completed the fix, would you mind testing it before we cut a new release?

https://github.com/openedx/i18n-tools/pull/140

Hey @OmarIthawi,
I'll try to get to this today. Sorry for the slow response, I was out of commission for a few days due to COVID.

@justinhynes thanks for the heads up!
Get well soon!

I was able to try the changes first thing this morning and it looks good. No issues locally. Thanks!

Closed by https://github.com/openedx/i18n-tools/pull/140 and released in v1.3.0.


Reply Keyboard can have not more than 10 buttons

When I'm trying to add 11-th button to the Reply Keyboard, it stops sending messages.
In master/src/AsyncTelegram.cpp: void AsyncTelegram::sendMessage
It turned out that 512 is not enough:
if (keyboard.length() != 0) {
    DynamicJsonDocument doc(512);
    deserializeJson(doc, keyboard);
    JsonObject myKeyb = doc.as<JsonObject>();
    root["reply_markup"] = myKeyb;
}

If I change  DynamicJsonDocument doc(512); to  DynamicJsonDocument doc(1024); it works fine.
Maybe it should be changed to BUFFER_BIG, or 1024 is enough.
Have a good day.

Worked for me as well, thanks!

Worked for me as well, thanks!

Increased the size in release 1.0.9


Handle nil values from the tsm1 cursor correctly

Send nil values from the tsm1 cursor at the end of the cursor. After the
cursor reached tsm1, the nextAt() call would always return the default
value rather than a nil value.
Descending also didn't work correctly because the seeking functionality
for tsm1 iterators would always act like they were ascending instead of
descending when choosing which value to select. This resulted in very
strange output from the emitter since it couldn't figure out if it was
ascending or descending.
Fixes #6206.

@jsternberg could you add a test for this also?

One of the unit tests failed and tests this functionality. I've modified the test so it expects the correct value now. I'll see if I can add a test for the order by descending portion.

I added a test for the descending iterator. It's not ideal as I don't think the test belongs there, but it will work for making sure this specific issue doesn't happen again.

LGTM 👍


Update impress to version 2.0.3

Landed in https://github.com/metarhia/metarhia.com/commit/f07163f6d24277f546066888f581d5c8426933ce

Landed in https://github.com/metarhia/metarhia.com/commit/f07163f6d24277f546066888f581d5c8426933ce


Support 415

This is addressing #967.
The easiest way of supporting 415 (UnsupportedMediaType) is to return a special (singleton) exception from Decode.Dispatchable[CT, CNil] (the very last decoder in the list) so we can later match on that within a derived service.
Here is the proposed behavior:

By default (a standard Bootstrap), a body endpoint that sees an unknown media type will resolve into Throw(Error.NotParsed) (fail-fast) with an underlying cause of Decode.UnsupportedMediaType.
A Bootstrap option enableUnsupportedMediaType (false by default), would turn those errors into 415 responses. Note the default response remains 400.

I'm not entirely in love with encoding this logic in a special exception but it seems a very natural way to implement it.  I'm eager to hear what people think about this.
As a bonus, this PR fixes a bug with dispatchable decoders where content-type also contains a charset (i.e., application/json;charset=utf-8). The new logic only relies on media-type when dispatching to a decoder (as it should).

@sergeykolbasov Mind taking a quick look? I'm thinking to ship this today.

Sorry, I'm on vacation.
On Tue, 28 Aug 2018 at 19.10, Vladimir Kostyukov<EMAIL_ADDRESS>wrote:

@sergeykolbasov https://github.com/sergeykolbasov Mind taking a quick
look? I'm thinking to ship this today.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/finagle/finch/pull/980#issuecomment-416666189, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACNbQZ6rYtsPJAdJ8zyh8_EW-Wi-ZyPyks5uVXl0gaJpZM4WMSQW
.

Hey @sergeykolbasov, no worries (also, sorry to nag you). Feel free to take a look when you're back!

👍


Validation Error: Missing City / State

A validation error manifests when a venue does not have a City or State populated (example)
ValidationError: 2 validation errors for ArtistSetListResponse setlist -> 9 -> venue -> city -> state   field required (type=value_error.missing) setlist -> 10 -> venue -> city -> state   field required (type=value_error.missing)

this is fixed in version 0.4.0!


Filter Emby Server container properties

Per this discussion thread with the Emby Server devs, the server could return multiple formats for the container.   We only really care about MKV and MP4 container formats as those are used the most.  If neither of those are included, then we assume we are going to be Transcoding the files or using HLS streaming.
https://emby.media/community/index.php?/topic/65862-404-error-when-trying-to-direct-stream-some-mkvs-with-ac3-audio/
In the Emby Mappers from the emby-lib module make sure to check the container and only include static streaming for mkv or mp4.

Verified as working.


Commands do not work on my laptop

Good afternoon Ori,
I am Gonzalo Aguilar, doing a PhD on Robotics (Neural Networks) in Spain.
To be able to simulate my NN, I need to combine several programs. The programs to combine are:

UE4
ROS
PX4
AirSim

In the basis of my project i will try to do a simulation of my NN on a SITL on UE4, sensors simulated on PX4 and ROS. Later I will do a real test with a HITL using a real drone but my UE4 environment and finally a test on real environment.
To check compatibility my Hardware is:

Intel i7 7700 HQ
SSR 2050 Mb/s
Nvidia Gforce GTX 1070

My software is:

Ubuntu 16.04.03 LTS

I have already installed by my own and it is already working a combination of:

UE4.17.2
PX4
XBOX 360 Gamepad
AirSim

Now I will start explaining my main problems when I try to install DroneSimLab.

In these two images we can see:

the result of the docker images after following the process given on DroneSimLab
the result when I run the first demo given on Easy Start by running the commands
$  cd demos/px4_gazebo
$ ./runtmux_game_docker.sh

I do not really know what the problem is, but I am stucked on this since long time ago and I do not know how to go on.
I hope you can help me.
Thanks in advance.
Best regards.

Hello,
I was able to install every docker image except that of UE4. I will try it again and let you know.
I wonder if it is possible that I use my own UE4 project with your DroneSimLab App. Could you pleae inform me how to do it?
Thanks a lot.
Best regards.

Yes, you can use your project. follow carefully the wiki page link
especially the part that initializes the PROJECT_FILES_DIR variable. this is the parent folder of your project. This directory will be mapped inside the docker container to /project_files  folder.
The make sure you did it correctly after running the UnrealEngine container you can run:
cd /project_files
ls

you should see you project folder at the output

As for the unreal engine failure, you can attach the installation log file in:
/tmp/dbuild-unreal_engine_4.log

It sometimes doesn't run 100% the first time. It could take more than two runs sometimes in my experiences earlier this year.
There was also a 'ros_image_indigo' that has been removed from the build earlier this year.
You can see it showing in the repository column of the docker images example on the wiki 'Easy start'

I have run it multiple times just to check as you said ("could take more than two runs") but still the result is 'not existing the ros image'.

Maybe you need to reinstall the ros_image_kinetic. I had this issue. It did not install 100% although it showed as installed. To reinstall just kinetic, run '../build' from the kinetic directory.

I did as you indicated... as well I installed the ros_indigo image but as you can see there is no good result.
Besides, the "run_tmux_ros_px4.sh" is shown as not existing...

Hi,
Can you explain what is the ros_image_1 folder (under dockers)?
Did you create it?

I did not create it. Previously to installation I deleted all my docker images.
I think it must be included in the DroneSimLab.

well, it's not there. have a look at this folder:
docker folder

Good afternoon again,
I am trying to reconnect everything with another demo:

On the image you can see in the terminal top-left containers and images on my dockers, in the terminal to p-right the result when I run both ./run_image.sh and ./attach.sh. Finally center-down terminal you can see the result when I run the :
~/DroneSimLab/demos/px4_gazebo$ runtmux_full_docker.sh
Several procceses are open in window terminals top-right and center-down but i cannot see any result.

Good night again,
I had to unninstall the docker images and now when I am trying to re-install them again, this is the result with the ROS images:

After doing everything writen on your site, the result is shown, even after running the command you can see on the last line of the right terminal.
Can you please let me know if something has changed on the server?
Thanks a lot.
Best regards.

Maybe this helps:

I am rebuilding this image now. I'll keep you updated

Done.
I will try to run my UE4 project within the next days and inform you.
Thanks a lot.


Add Encrypted File Relationship to encrypted parts and models

As stated by Secure Content spec item 3.3,
"OPC parts that are encrypted MUST be associated with the package root through a Encrypted File relationship, and each SHOULD have an entry in the Key Store. If a custom OPC part is not referenced by a Encrypted File relationship it is assumed they are not encrypted and MUST NOT be any entry in the Key store associated to the file"
@martinweismann and @jordig100 I think this is one left over we forgot to add. Just have a question: the text mention package root. I just want to make sure that the relationship to be added is not on the package root model rels, rather at 'package root' rels (the sample provided makes it a bit confusing as external models aren't referenced from the package root rels).
If that's the case, we've the ability to add this relationship to the root model when we detect a part must be wrapped for encryption,
https://github.com/3MFConsortium/lib3mf/blob/8d4a912667520721f24619d9adb1b42558443a17/Source/Model/Writer/NMR_KeyStoreOpcPackageWriter.cpp#L140-L145
Another option would be to have it right before we finish writing the package
https://github.com/3MFConsortium/lib3mf/blob/8d4a912667520721f24619d9adb1b42558443a17/Source/Model/Writer/NMR_KeyStoreOpcPackageWriter.cpp#L162-L168
Let me now what you think.

Hi @feliperoos, this is already captured in the issue: https://github.com/3MFConsortium/spec_securecontent/issues/12
I think the asnwer is just to remove the "root" word from that sentence.
In the following link you an find the proposed change: https://github.com/3MFConsortium/spec_securecontent/blob/dev/3MF Secure Content.md#33-encrypted-file-relationship


fix: JSONPath Syntax in kubectl explain Documentation ([#47080])

This pull request addresses an inaccuracy in the JSONPath syntax explanation for the kubectl explain command in the Kubernetes website documentation.
Current Issue:
The current documentation incorrectly uses <type>.<fieldName>[.<fieldName>] as the JSONPath syntax for accessing fields within Kubernetes resources.
Proposed Fix:
This PR corrects the syntax to the following:
$.<type>.<fieldName>[.<fieldName>]

This change ensures an accurate representation of how to navigate nested structures within resources using JSONPath with kubectl explain.
Please review this pull request and consider merging it to improve the clarity of the kubectl explain documentation.
Thanks!

fix for issue #47080

/hold
Refer https://github.com/kubernetes/website/pull/47094#discussion_r1665916442
Instead of modifying the generated page directly, it would be more effective to regenerate the reference documentation incorporating upstream fixes.
Feel free to unhold this once the above actions has been taken.

/retitle Fix JSONPath syntax in kubectl explain documentation


asyncSequences: Add Timer AsyncSequence

Description
This PR brings a new Timer AsyncSequence. Timer is an async sequence that repeatedly emits the current date on the given interval, with the given priority.
Checklist

[x] this PR is based on the main branch and is up-to-date, if not please rebase your branch on the top of main
[x] the commits inside this PR have explicit commit messages
[x] unit tests cover the new feature or the bug fix
[x] the feature is documented in the README.md if it makes sense
[x] the CHANGELOG is up-to-date

Codecov Report

Merging #13 (ebb50a0) into main (c09db2c) will decrease coverage by 0.08%.
The diff coverage is 92.68%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   97.39%   97.30%   -0.09%
==========================================
  Files          46       48       +2
  Lines        2262     2303      +41
==========================================
+ Hits         2203     2241      +38
- Misses         59       62       +3

Impacted Files
Coverage Δ

Sources/AsyncSequences/AsyncSequences+Timer.swift
84.21% <84.21%> (ø)

...sts/AsyncSequences/AsyncSequences+TimerTests.swift
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Pre 1.13.0 CLI install likely broken for python 3.10 users

Describe the bug
Ran into https://github.com/ckan/ckanext-dcat/pull/213  while debugging https://github.com/dockstore/dockstore/pull/4958  build
In short, installation of the CLI dependencies is likely broken for Python 3.10 (possibly 3.9)
Should not be an issue in CLI 1.13 install since we're past the affected dependency version
To Reproduce
Steps to reproduce the behavior:
Use newer CircleCI machine image like this change https://github.com/dockstore/dockstore/pull/4958/commits/986097f302e6b94ca05c4ab84aaca2bb4faa1ea4 then try to install Dockstore CLI
Expected behavior
Backwards compatibility
┆Issue is synchronized with this Jira Story
┆fixVersions: Dockstore 1.12.x
┆friendlyId: DOCK-2187
┆sprint: 93- Ursula
┆taskType: Story

Confirmed that https://dockstore.org/quick-start has instructions for the fallback.


382 383 receipt form

Note: this is not ready to merge just yet, as we are awaiting some more specifications about fields and when this functionality should be available to the user.

What's the difference between "item" and "description"?

I was thinking that item would identify the Receipt - something like "Plane ticket" and description would explain the receipt further, if needed.
I guess both fields aren't necessary and we can take out the description if having just the item is sufficient

If we just have the one field, to me the name "description" feels more self-explanatory.

@dchukhin let me know when you've addressed these last two issues.

I think before we merge, it'd be a good idea to reduce this to a single migration. One way to do that would be to delete all the new migration files and then run makemigrations again.
Having fewer migrations will be a tad faster when running tests, and save a few files in the repo.

:+1:


Duration.bucketize

There's a bug somewhere because
[
  NEWGTS
  42 NaN NaN NaN '42' ADDVALUE
  bucketizer.last
  24 h
  'PT1M'
  3
]
DURATION.BUCKETIZE

returns a GTS with 1 bucket.

That is the wanted behaviour. See https://warp10.io/doc/DURATION.BUCKETIZE

I'm not sure this is the wanted behavior.
True there are 3 buckets, but all should be empty, look at
[
  NEWGTS
  42 NaN NaN NaN '42' ADDVALUE
  bucketizer.last
  24 h
  1 m
  3
]
BUCKETIZE

In the case of DURATION.BUCKETIZE the GTS has 1 value while BUCKETIZE returns 0 value.

Yes you are right, I'm looking into it.


ValueType debugging experience

Improve ValueType Debugger inspection by using a different enum for Object Bits

This change is

@Microsoft/chakra-developers Can someone look at this small change please ?


Create load_bits function with loadbits alias

This function should load the bits of a file into a dtype=='bool' numpy array.  The input parameters should be:
filename
number of bits to load (default = All)
starting bit (default = 0)
It should probably read the file a byte at a time rather than a short or int at a time so that we don't have to worry about endianness.

This should be ready for testing.


chore(deps): pin minor API version

Use ~ instead ^ range in API dependency to avoid that a semver minor API change is allowed.
Semver minor API changes usually need an update in SDK therefore automatically update to a new minor would break the API contract for the end user.
Refs: https://github.com/open-telemetry/opentelemetry-js-api/pull/123
Refs: https://github.com/open-telemetry/opentelemetry-js-api/issues/125

I'll bring the discussion in the thread in spec here because it's specific to JS I believe.

@obecny wrote: and after this change (why all this concern has been raised) -> open-telemetry/opentelemetry-js#2531 you might endup having 2 plugins that will depend on different api because one will be 1.0.x and other 1.1.x and because they will have ~ they will accept only pinned minor version

Two different installed API packages should not matter in this case because the globals will be shared, aren't they?

@Rauno56 Yes, I noticed meanwhile that I changed more then needed but found no time yet correct this. I had somehow in mind that this repo is "the SDK" but actually it's by far more.
I think to get into the most graceful setup we should even use ~1.0.0 or ^1.0.0 instead 1.0.3.
Will move the ticket into draft until I find some time to adapt it. If someone else what to take over feel free.

I definitely tend to lean towards that as well. What are the ways to enforce it?

At runtime the semver check in API should enforce it. Mismatching instrumentations are re-routed to noop. Clearly this will result in e.g. context loss but it should not result in application malfunction.
At install time NPM may complain if peerDependencies mismatch but I'tm not sure here.

@Rauno56 peer dependency will not install the api, you in yr my package@1 have to decide which api you want to install so you can still install api ver 1.1. I would like to be able to install api 1.3 and still expect the context manager to work fine - at least all functionalities that were available in ver of api 1.0.x should work fine. That would be at least my assumptions. So as long as I don't use any new functionality that were introduced in api 1.3 I would still expect this all should work just fine exactly as I used api 1.0.x.  If this is not true then why we are saying that we will provide a compatibility for major versions if that is not true ?

if multiple of your dependencies call for the same peer dependency it will pick the highest version which matches all ranges. If there is no match it throws an error. There is also a command line flag to override the error.

So should we then do something like this. Instead of returning noop implementation for things related to sdk, should we return original implementation and fill it with noop functions that are required - they could be taken from NoopImplementation from the api. So this way we could use for example SDK ver 1.1 and plugins ver 1.1, 1.2, 1.3 and still all would work fine but only new functionality would have noop. Instead of using the noop implementation for the whole object. It can be quite easy to implement more less like this (pseudo code)
ReturnedObjectFromSDK = MergeTwoObject(NOOP_OBJ_FROM_API, SDK_VER_1_1.REAL_OBJECT);
// if any noop function is being used show warning in console once only
// console.log("function newFeature is available from ver 1.2, fallback using noop implementation");

Do you think that would be better for end user instead of falling back to noop implementation for the whole object anyway ?
I just think that the base functionality should work fine with different minor version of api.

If the whole SDK becomes noop on a version mismatch then at least they know something is broken right away.

How do you want to know that ? Will you raise some warning or ?

IMO if an instrumentation requires API 1.2 and the user installs API 1.1 it should be an error at install time.

I don't think we should raise any error at all. That is main principle of otel, we should not harm the production. Imagine a simple thing you just want to release a new version you do npm i and you can't restart because it gave you an error. So what option you have return noop and warn ?.
In this PR the pinned version is done also for instrumentation, for me it is a main blocker as I think the instrumentation in ver 1.0 doesn't need to be updated for 3 years and you should still be able to use it (and it sho.
uld work fine) with api ver 1.99.x and sdk ver. 1.99.x
With regards to sdk I don't have a strong opinion if we should either have a noop for whole sdk if sdk is lower then api or only for certain functions. I still think it would be more convenient for a user to make it work fine then simply return noop or even raise an error. How we would tell user that something is wrong is another story and I would like to first figure out what we do noop object or noop for only missing functionality - still no strong opinion on any of these only trying to figure out best option for the user.

SDK could be ~ but I actually think explict range like >=1.0.0 <1.2.0 would be best that way we can make sure you get patch updates for all supported minor versions instead of just the latest one and SDK can support old APIs if needed.

In fact we should have only the upper bound
<1.2.0 only, by default the SDK 1.99 should still work fine with api 1.0.0 right, that is the promise we do ?
the lower should be always 1.0.0


Conversion error logs is not descriptive enough

We should probably add more context to every usage of ErrEmptyMessage. The error makes sense only if used as a top-level error. However, there's no instrument to prevent a developer from using it for included messages that might also be empty.
https://github.com/onflow/flow-go/pull/6571#issuecomment-2427894822

cc @peterargue


Add http agent and fix agent helper. (#42)

Example http agent usage:
pom_battle --agents simple::null,simple::null,http::localhost:8085,simple::null --render
PR is mostly a  copy & paste from docker agent and some fixes for broken agent selection CLI.
Would be nice to use it as a base for the docker agent to avoid duplicated code - but don't really have the time.

Mhhh - addressed cinjon's requests a while ago, rebased and squashed and forced push.
Will investigate later today why it does not show up.

OK - changes are in now. In future I will just add to the commits and count on commits being squashed on merge. Forced pushes are just evil.

Thanks! Agree re force pushes.


assert: async() and its resulting callback result in test suite stoppage

Hopefully I'm doing something wrong, but http://jsbin.com/votiye/ would seem to indicate that there's a hangup when using assert.async() and subsequently calling the resulting function.
Expected outcome:
Two tests are run.
Actual outcome:
The second test does not run.

Doing things the old-fashioned way (with start()) works: http://jsbin.com/geyifel/

OK. I see the error of my ways now. I must not use asyncTest() if I use assert.async().


fix: update email settings

Current Email should be in a <p> or <span>, or something just as uneditable.

The other issue will be resolved with https://github.com/freeCodeCamp/freeCodeCamp/pull/17669

Regarding the UI. I think I am not an expert at the moment. But will give it a quick try. I just added back the original views that we commented previously.


chore(FieldTheory/NormalClosure): remove a porting note

This will be made redundant by #6163

Even if it becomes redundant this is still a strict improvement for now.
bors merge


Laravel 5.5 branch

I'm fairly new to integrating vuejs to laravel so would love a 5.5 branch to play around with!

yes yes, will do. Keep visiting thanks.


aten._adaptive_avg_pool2d.default

Need to solved https://github.com/tenstorrent/tt-metal/issues/12151 or the ttnn.avg_pool2d cannot support case of output_size != [1, 1]
input = torch.randn([1, 512, 7, 7]).to(torch.bfloat16)
output_size = [7, 7]
adaptive_avg_pool2d = torch.ops.aten._adaptive_avg_pool2d.default(input, output_size)
print(adaptive_avg_pool2d.shape)
# torch.Size([1, 512, 7, 7])

Sorry the above comment is wrong, please ignore it
Here is the description of aten._adaptive_avg_pool2d.default:
aten._adaptive_avg_pool2d.default is not same with aten.avg_pool2d.default, but it can convert to aten.avg_pool2d.default by mapping aten._adaptive_avg_pool2d.default's output_size argument to aten.avg_pool2d.default's stride & size by
Stride = (input_size//output_size)
Kernel size = input_size - (output_size-1)*stride
Padding = 0

So if want to support lowering of aten._adaptive_avg_pool2d.default, then it needs either ttnn.adaptive_avg_pool2d op for direct converting or ttnn.avg_pool2d with above adjust
But seems there no ttnn.adaptive_avg_pool2d op now, and ttnn.avg_pool2d is not finished according to https://github.com/tenstorrent/tt-metal/issues/12151
Luckly, there's only one input variation appeared on models
Tensor<[1, 512, 7, 7]> self = ?, List[int] output_size = [7, 7]

And this input variation actually let aten._adaptive_avg_pool2d.default do nothing, so if we check these input varation type during lowering and just remove it, then we can say aten._adaptive_avg_pool2d.default is fully supported of the current scope of models


Loop closure messing up 3D map reconstruction

Hello and thanks for making LAMP available through this open source implementation.
I'm trying to test LAMP's inter-robot loop closure in simulation with a setup of 3 turtlebots exploring willowgarage using LAMP's default parameters. The 3D lidar map is quite accurate at the beginning of the exploration.

However after the first loop closure the map is badly reconstructed:

Do you know what might cause this issue ?
I believe there might be issues with the parameters of the Levenberg–Marquardt optimizer. I joined the output of the optimizer as well as the parameters of the simulation
.
Finally you can find an archive containing the logs of the simulation.
Thanks for your help !

Hi,
Thanks for trying out LAMP!
There's a few things that you can try:

Do the keyed scans make sense? Make sure that you are not down-sampling too much otherwise you would get bad loop closures. To tune the parameters related to this, take a look here or however you are filtering your point-cloud to create the keyed scans.
Do the output from the loop closures make sense? Take a look at the output topic of the loop closure module to make sure if the transforms are kind of what you expect. Otherwise you can tune the loop closure detection parameters
How is the outlier rejection (probably GNC) doing. There should be a terminal output like xx number of loop closures detected with xx inliers. If all the loop closures are accepted, there's probably something weird going on, and you can increase the strictness of GNC by tuning alpha in this file
In general, make sure that the book-keeping is done correctly. I.e., each robot should have a unique prefix such as what is defined here


[WIP]Use latest version azure go sdk and go-autorest

Update Azure-Go-SDK to 23.2.0
Update Go-Autorest to 11.2.8
The current build of Azure Go SDK is over 2 years old. It would be in the best interests to keep the Azure deps updated. Also, the querying of PowerStates requires us to use more recent. versions of the Go SDK (refer to #4908 )
Feedback is appreciated :). The changes worth noting can be seen in azure.go and azure_test.go. There are changes in the method signatures and type structure which I thought might be of interest to the reviewers.

Filed https://github.com/Azure/go-autorest/issues/353

cc @sylr @marratj @jannickfahlbusch @johscheuer who have recently contributed fixes and improvements to the Azure SD and might be able to test this PR once it is ready.

I can test this after this got merged :)
However, as #4719 just fixes a silenced error everything should be fine from this PR.

Just to make this clear, this PR is a work in progress. So it may go through significant changes. Reviews are always welcomed from everyone :)


question about firmware

Hello, is your firmware version open source? Because I didn't understand, probably due to my incompetence, how it's possible to view individual code files. Do you plan to publish the source code or, as I seem to understand, only the compiled versions? Thank you very much for your excellent work.

Hello,
Yes of course, it's full open source.
Take a look a this repo https://github.com/armel/uv-k5-firmware-custom and check branch.
Armel.

Thank you very much and sorry, I had searched for a long time but I really don't know why, I couldn't find the source files. Thanks again!


ci: report test results

Summary
Report test results in JUnit format to be consumed by the CI
Implementation details
Use https://www.npmjs.com/package/jest-junit
How to validate this change
The CI reports a JUnit trend report and also locally a file is created when running the tests.
Filename => junit.xml
Issue
Closes https://github.com/elastic/synthetics-recorder/issues/221

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2022-04-26T18:51:12.997+0000

Duration: 16 min 28 sec

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

Is this ready for review? I've tested locally and am seeing the expected files.

Is this ready for review? I've tested locally and am seeing the expected files.

It works locally, I'm now debugging what's the reason those files are not generated in the CI, maybe the docker agent might be the reason... still debugging

Finally, the junit step doesn't work with absolute paths, and the docker container uses them.
It's now finally reporting test results


Updates

Fixed an problem where integrator.f90.template assumed free protons in the reaction network.

Generalized prefactor and dens_exp calculation.

Added triple-alpha and triple-alpha_f90 examples.

Added a test-chapters example which uses one rate from each reaclib chapter, as a start for unit testing.

I just pushed 2 fixes to the original source:
-- got the notebook working again (it was broken from the last PR) + it now works with python3
-- for the examples/CNO/, there is now a file cno_rhs.py.benchmark -- this is the "correct" output.
Can you confirm that your changes here still give the correct output for CNO?  Just run cno.py and diff with the benchmark file
Can you add benchmark files for your nets too?  This will be the start of the unit testing -- we'll ensure that we always get the right answer for all subsequent pulls.

and yes, I should have done my changes as a PR too :(

My version does give the correct output, though not the same output. Mine reverses the order of the Y[in]*Y[im] multiplications to read Y[im]*Y[in].
I've added unit testing with nose, which the readme in pyreaclib/test explains. Because I had been working on this before today, I didn't use the .benchmark scheme. Instead, there is a directory pyreaclib/test/standard with the same subdirectory names as the examples directories. Each directory pyreaclib/test/standard/[example] contains the expected output from its corresponding example script, which prevents the pyreaclib/examples directories from becoming cumbersome, especially with the fortran examples which produce a lot of output files.
pyreaclib/test/test_reaclib.py runs the nose unit testing as follows:

Create a directory pyreaclib/test/runs/XYZ where XYZ is formed from the current date and time

Copy the contents of pyreaclib/examples/[example] to pyreaclib/test/runs/XYZ/[example] for each example

Run the example scripts in pyreaclib/test/runs/XYZ

Use diff to compare each file in pyreaclib/test/standard/[example] with its corresponding file in pyreaclib/test/runs/XYZ/[example], asserting the diff output is an empty string.

The contents of pyreaclib/test/runs will not be tracked by git.
Also, the 'chapters' example I added containing 11 nuclides and 11 rates, 1 rate from each Reaclib chapter, I have added to the unit testing after manually verifying the following about chapters_rhs.py:

All the reaclib fit parameters are read and written correctly

All the prefactors, exponents, etc in all the RHS terms are correct.

I just merged the upstream changes into this pull request and removed the .benchmark file. When I ran cno.py in a repo I cloned directly from pyreaclib/pyreaclib.git, I also see that the order of multiplication doesn't match what's in cno_rhs.py.benchmark so I suspect it came from an earlier git commit than the current one in pyreaclib/pyreaclib.git...

I was looking further into the benchmark file and it didn't come from 949bc17ffce5cc02fa799c0356e95ff233a1f22b either, which was the last commit before we merged in all my stuff. Running diff on cno_rhs.py between that commit and the latest in the current pull request, they match.

Fixed a formatting issue with the readme in pyreaclib/test


DRC display updates

Update the text for displaying Direct Reference Codes, and provide an actual working link to the code in VSAC.
NOTE: This PR should be merged before #3 so I can rebase that one, so the merge will work cleanly.

So glad we've replaced the UUIDs with VSAC relevant URLs.  Looks great!


skipToNextItem() produces PLAY_ACTIVITY error

When I press the button to skip songs, it consistently logs this error. It seems like it's a MusicKit issue because it seems to be plaguing all other Apple Music web players. It's not super consequential, other than it kinda glitches out and occasionally produces some audio hiccups. I just wondered if there's any way of avoiding this error, as I'm trying to use the MusitKit functionality on my own website.
Uncaught (in promise) PLAY_ACTIVITY: A play stop() method was called without a previous play() descriptor
at Function.MKError.playActivityError (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:25498)
at PlayActivity. (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:36354)
at https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:14473
at Object.next (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:14578)
at https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:13494
at new Promise ()
at __awaiter (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:13229)
at PlayActivity.stop (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:35849)
at PlayActivity. (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:33878)
at https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:14473
at Object.next (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:14578)
at https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:13494
at new Promise ()
at __awaiter (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:13229)
at PlayActivity.exit (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:33788)
at HTMLDocument. (https://js-cdn.music.apple.com/musickit/v1/musickit.js:1:163084)
at HTMLDocument.r (https://musi.sh/app.47281733.js:53:16771)

Yeah, this is very likely a MusicKit JS bug since I only call instance.player.skipToNextItem(); (https://github.com/zachomedia/apple-music-webplayer/blob/master/src/store/modules/musicKit.js#L428) in order to switch to the next track.
Have you tried opening a support request with Apple through https://feedbackassistant.apple.com/welcome?

Yeah, I've done that, so I'll reply when I get a response from them.

Apple has fixed the bug.


Move DMOD service images and service Python packaging image to Rocky Linux base

Move main stack service images and the py-sources image (on which the service images depend) to a base of Rocky Linux.  This removes the need for several pre-built wheels and simplifies the build overall, at the cost of slightly larger images.
These changes resolve some issues causing image builds to currently fail, as tracked in #425.

Is this ready to review or are you considering targeting rocky 9 after our conversation earlier, @robertbartel? For a little context, there is an ongoing discussion if NextGen needs to support GCC 8. This discussion centers around the absence of c++ std smart pointer types in GCC 8. Coincidentally, rocky linux 8's package repo ships GCC 8.5 where as rocky linux 9 ships GCC 11.

I am still considering moving the ngen images (and the others) to 9.1, but it turned out that 9.1 didn't help with the problem I was having. Since it isn't immediately helpful/necessary, I don't want to rush to make that change.
For now, let's move forward with 8.5 on this one.


is it possible that a client component query can used cached response from server component query?

Hello!, this is more of a cry for help rather than a issue. But I don't really know where else to ask
I currently have a nextjs + apollograph built upon the pages directory, but want to migrate to the new app directory with app routing. The issue I have is that I don't really find a way to use the ApolloNextAppProvider together with the registerApolloClient. The main issue: In the old way we did a serverside request called useFramework(), that had framework related global states and so on. Then on clientside we used the same request, but now fetched through the apollo-cache so that we didn't get unnecessary requests on.
Now I try to implement this in a very simple POC:
this is a server component that fetches the Framework-data, StartPage is a client side component. So everything down the line from StartPage will be clientside. How can i use the same query in a client component in order to use the cached response?
import React from 'react';
import { headers } from 'next/headers';

import { StartPage } from '@pages/sales/StartPage';
import { FrameworkDocument } from '@schema-types';
import { getClient } from '@lib/RSCClient';
const Home = async () => {
  const headersList = headers();
  const domain = headersList.get('host') || '';
  const fullUrl = headersList.get('referer') || '';
  const relativeUrl = fullUrl.split(domain)[1];
  const client = getClient();
  const { data, error } = await client.query({
    query: FrameworkDocument,
    variables: {
      url: relativeUrl,
    },
  });
  return <StartPage data={data} />;
};
export default Home;

I've tried using ApolloNextAppProvider in order to make the client requests work, but then I think I'll get two seperate clients, which I don't want, because the don't share same cache
Are a possible solution to store the framework data in a context? and wrap the context around everything in layout.tsx? or maybe pass data down from the server component?
### Tasks

I think I'll get two seperate clients, which I don't want, because the don't share same cache

Yes, and that will always be the case - RSC and SSR of Client Components (and later, the browser) run in different processes, sometimes even on different machines at different times. You can share nothing between them, and you should never use the same query in a RSC component and a Client Component. If the cache value updated, the static RSC content would not update while the Client Component would update the value - you'd get inconsistent UI.
That's why this is one of the first paragraphs of our README:

❗ We do handle "RSC" and "SSR" use cases as completely separate.
You should generally try not to have overlapping queries between the two, as all queries made in SSR can dynamically update in the browser as the cache updates (e.g. from a mutation or another query), but queries made in RSC will not be updated in the browser - for that purpose, the full page would need to rerender. As a result, any overlapping data would result in inconsistencies in your UI.
So decide for yourself, which queries you want to make in RSC and which in SSR, and don't have them overlap.

You can use <PreloadQuery in a RSC context to preload data that you want to later display in a Client component, but you should only use that for preloading - after that, you should consider that data as "client data" and never display it in a RSC component.

Thanks for the quick response! So never share data between RSC and Client Component only preload.
I have another question regarding the initialization of Apollo client. In our current codebase we have used this guide for our SSR logic: https://medium.com/@zhamdi/server-side-rendering-ssr-using-apollo-and-next-js-ac0b2e3ea461. We have a bunch of function like: initializeApollo(), useApollo(), createApolloClient() and I can't see how they'll fit in the new app router. How do I start to migrate all the apollo-functionality we've built using this guide? Or is this something that isn't necessary anymore? Or is this exactly what  does?

All of that isn't necessary anymore - it's essentially what this package does (although, due to the new streaming SSR model of the App router in a much more complicated fashion ;) ).

I believe everything here has been answered, so I'll close the issue :)
If you have any more questions in the future, feel free to open another issue!


Use StructOpt in tikv-ctl

Development Task
Refactor https://github.com/tikv/tikv/blob/master/cmd/tikv-ctl/src/main.rs by using structopt.

/assign

Current code with clap is clummy and crappy.

The new clap in master has adopted structopt; it's the future.


[master] JBPM-5217 - Link each element of the process model to its location in  the process documentation

(JS changes only)
@jlindop : adds a new button in the shape menu for each node in the process.
when the new documentation icon in shape menu is clicked, the tab changes from process modeling to process documentation and the page scrolls the location where this element is defined.
Note: for sequence flows we only show sequence flows in the docs that have either their name, documentation or conditionexpression properties set. If the shape menu docs button is clicked on a sequence flow which does not have at least one of these elements set, designer shows a notification telling the user that element does not exist in the docs and does not switch to the docs tab. if the sequence flow has at least one of these elements the same applies as for any other node type.

@jlindop amended per your code comment

+1


Support for extra-index-url

I am trying to install a custom plugin hosted on a private Gitlab instance.
pip supports --extra-index-url to specify additional package indexes.
llm does not appear to support this option:
Usage: llm install [OPTIONS] [PACKAGES]...
Try 'llm install --help' for help.

Error: No such option: --extra-index-url

Although, in the docs I see:

Use the llm install command (a thin wrapper around pip install) to install plugins in the correct environment

But I'm not sure if the option is meant to be omitted for other reasons I'm not privy to.

There's a workaround for the moment: you can install the llm-python plugin:
llm install llm-python

And then run pip like this:
llm python -m pip install ... --extra-index-url ...

Let me know if that works. I'm not sure if it's worth adding more options to llm install when this workaround exists.

That works great! Agreed, no need to add options with this workaround.
Might be worth throwing a link to this comment in the docs for future folks.


Server assets in Nuxt 3.1.1 not mounted according to docs

Environment
Nuxt project info:                                                               22:36:00

Operating System: Linux
Node Version:     v16.19.0
Nuxt Version:     3.1.1
Nitro Version:    2.1.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      typescript, nitro, app, alias, css, vite, runtimeConfig
Runtime Modules:  -
Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-u78ses?file=server/api/test.ts
Describe the bug
test.txt is not available with the key server:assets:test.txt according to https://nitro.unjs.io/guide/introduction/assets.
It is available with keys src:assets:test.txt and root:server:assets:test.txt.
Additional context
Perhaps it is more a bug related to Nitro than Nuxt, but upgrading to Nuxt 3 caused my project to fail due to undocumented changed behavior of server assets.
Logs
No response

@pi0 was that change intended? Also related: https://github.com/nuxt/nuxt/issues/18549

After further investigation, the server assets don't end up in the production bundle at all. So using the different keys only works in development build and not in the production build.

For now, you can do this:
// https://v3.nuxtjs.org/api/configuration/nuxt.config
export default defineNuxtConfig({
  nitro: {
    serverAssets: [
      {
        baseName: '',
        dir: 'assets',
      },
    ],
  },
});

https://stackblitz.com/edit/github-u78ses-2nplvr

This should be resolved in nitropack v2.1.1. which you can get by refreshing your lockfile or with npm nuxt upgrade --force.
Let me know if not and I'll reopen.

I had to use npx nuxi upgrade --force, but yes it works now. Thanks!

Hi guys,
I have a similar problem and wondering if I did something wrong maybe.
The server assets are not available after building in production (in dev everything works perfectly).
I tried with next:
Nuxt v3.4.3

Config:

  nitro: {
    serverAssets: [
      {
        baseName: '',
        dir: 'assets', // I tried different variations here
      },
    ],
  },
...

Files:
server\assets\templates\test.txt

Code:

const template = await useStorage().getItem('root:server:assets:templates:default.txt')

Tried with different settings (with or without specifying serverAssets), but cannot make it work yet on prod.
Reproduction is pretty simple. Can be used the one mentioned above (https://stackblitz.com/edit/github-u78ses-2nplvr) and make a build for prod.

Hello!
I'm sorry if I disturbed you.
That issue was resolved: https://github.com/nuxt/nuxt/discussions/20790
I don't get it clearly while reading the Nitro documentation - it was some confusion about naming assets in the config and about how to use it then. So, maybe there's a need to make some clarifications in the docs.


My Tasks - menu item

As a user
I want a place where all my tasks from all works are in one place
so that I can stay on top of my work
Acceptance Criteria
Scenario 1:
GIVEN I log into EPIC.track
WHEN I reach the landing page (my workplans)
THEN I see the My Tasks menu item under the My Workplans menu item
Screenshot 1

Scenario 2:
GIVEN I log into EPIC.track
WHEN I click on the My Tasks menu item
THEN I see the My Tasks Page with Task Grid
Screenshot 2

Validation Rules?
(If yes, list here)
Screenshot
(Link above will always be the most up-to-date version.)

link to the prototype - https://www.figma.com/file/XcYynwH7xrO2P30uqewaLS/For-Development?type=design&node-id=109-983&mode=design

Reports should still be bundled with insights.
My tasks is correct, but all works should be on it's own


Ignore chain id when using --dump-bytecode-as-base64 in sui move build

Description
Allow sui move build --dump-bytecode-as-base64 to not need to read the chain from client.yaml by adding a flag --ignore-chain. This allows building to proceed without a network connection or active environment, but it will not be able to automatically determine the addresses of its dependencies.
NB: --ignore-chain depends on dump-bytecode-as-base64, so it cannot be used on its own.
Should close #20458.
Test plan
Local test.
➜  first_package git:(main) ✗ sui move build --dump-bytecode-as-base64 --ignore-chain
INCLUDING DEPENDENCY Sui
INCLUDING DEPENDENCY MoveStdlib
BUILDING my_first_package
JSON_OUTPUT_HERE - replaced for brevity.
➜  first_package git:(main) ✗ sui move build --dump-bytecode-as-base64
Config file ["/Users/user/.sui/sui_config/client.yaml"] doesn't exist, do you want to connect to a Sui Full node server [y/N]?

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[x] CLI:  Added the flag --ignore-chain that works together with sui move build --dump-bytecode-as-base64 to bypass the need for a client.yaml file. This allows building to proceed without a network connection or active environment, but it will not be able to automatically determine the addresses of its dependencies. NB: --ignore-chain depends on --dump-bytecode-as-base64, so it cannot be used on its own.
[ ] Rust SDK:
[ ] REST API:

For backward compatibility, I believe the --use-chain option would be more suitable than the --ignore-chain option.
Additionally, I think having no option should ensure the same behavior as when it was originally developed.
I hope there will be an opportunity to consider this change.


Configuring with ENVs

As discussed I've made a few updates for providing some settings through environments.
All settings from auth.yml can be now provided with envs.

Config files shouldn't be part of the Docker image. They should be provided externally instead.
So example config files are copied while container building.

Please also run 'black' to fix the CI checks (see README.md).
see CI check log:
would reformat src/gateio_new_coins_announcements_bot/auth/gateio_auth.py
would reformat src/gateio_new_coins_announcements_bot/send_telegram.py

Please also run 'black' to fix the CI checks (see README.md).
see CI check log:
would reformat src/gateio_new_coins_announcements_bot/auth/gateio_auth.py
would reformat src/gateio_new_coins_announcements_bot/send_telegram.py

Done

We can add to the README an example of how to run the image with the config files mounted

We can add to the README an example of how to run the image with the config files mounted

It could be something like:
docker build -t gateio_cryptobot . && \
docker run --name crypto-bot -d --restart always \
     -v ~/gateio-bot/config.yml:/app/config.yml \
     -v ~/gateio-bot/old_coins.json:/app/old_coins.json \
     -v ~/gateio-bot/auth.yml:/app/auth/auth.yml \
     gateio_cryptobot

assuming config files are stored at ~/gateio-bot/


Support for "classic" ASP.NET

I was planning to migrate a couple of ASP.NET 4 projects to the new csproj format but this article says it's not supported.
Is it actually true? If so, it would be a pity since statistically the web project file is where most of our conflicts happen.
Also, current file size: 267 KB...

That's correct, and [@]DamianEdwards stated in a recent Community Standup that a migrate tool that would help with the process isn't here today but might come later. Nate's blog post is excellent. Since I don't see an issue for future tooling that would help you, you could close here and ask on https://github.com/dotnet/cli/issues/4874, since that issue has some recent activity pertaining to this.

A migration tool would be much above my expectations. I would be happy with knowing it is possible to run deploy an ASP.NET 4 application built with the new project.
I already managed to get it to build, it's mostly about packaging, I guess.

No ... it won't work. I can't point to the exact spot where Damian addresses it; but if you watch the Community Standup from March 14, he answers this question and comments on the possibility of tooling that would help migrate. https://www.youtube.com/watch?v=qlVE5iDjBIg&list=PL0M0zPgJ3HSftTAAHttA3JQU4vOjXFquF&index=1

@DamianEdwards mentions here https://youtu.be/c9Ub4uwjlbo?t=55m8s that Nick Craver did it.
I'm in the same boat as @Kralizek and migrating the msbuild first and move to ASP.NET Core is the best play here.
I don't expect full support. Of course more complex projects may not be able to migrate but some directions would help a lot.

@diegobfernandez are there any samples on how @NickCraver did it? I didn't find any ASP.NET 4 project based on the new SDKly project format and don't want to waste too much time, if somebody already found a nice solution...

Okay, something like that can be a start:
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net461</TargetFramework>
  </PropertyGroup>

  <PropertyGroup>
    <AppendTargetFrameworkToOutputPath>false</AppendTargetFrameworkToOutputPath>
    <OutputPath>bin\</OutputPath>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.AspNet.Mvc" Version="5.2.3" />
    <!-- ... -->
  </ItemGroup>

  <ItemGroup>
    <Reference Include="Microsoft.CSharp" />
    <Reference Include="System.Configuration" />
    <Reference Include="System.Web" />
  </ItemGroup>

  <ItemGroup>
    <Compile Update="Global.asax.cs">
      <DependentUpon>Global.asax</DependentUpon>
    </Compile>
  </ItemGroup>

  <ItemGroup>
    <Content Include="Web.config">
      <SubType>Designer</SubType>
    </Content>
    <Content Include="Web.*.config">
      <DependentUpon>Web.config</DependentUpon>
      <SubType>Designer</SubType>
    </Content>
  </ItemGroup>

  <Import Project="$(VSToolsPath)\WebApplications\Microsoft.WebApplication.targets" />

</Project>

In order to debug the project, using the "all-new" launch settings is possible:

Problem, so far: IntelliSense for Razor Views expects ASP.NET Core Views, which doesn't make much sense.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

Can we reopen this? Supporting pre-core applications should be somehow in the backlog

@Kralizek I think that it's best tracked by https://github.com/dotnet/cli/issues/4874, as this isn't really ASP.NET-specific.

@elion i agree, but there are also asp.net specific issues like the problem of razor validation assuming the core version of the SDK project file is used.
Also, it is to be validated how to properly execute and debug the application during a normal development cycle.

Ping... any movement on this?


Integer casting error

Some weird error happened when I run a sample script to predict parsnip encoding for PS1 data
import lcdata
import parsnip

ps1_model = parsnip.load_model('ps1')
ps1_data = lcdata.read_hdf5('data/ps1.h5')  # downloaded by lcdata_download_ps1 script

predictions = ps1_model.predict_dataset(ps1_data)

IndexError                                Traceback (most recent call last)
/var/folders/1d/bl0ng6jd7lq3p4j344gc98k00000gn/T/ipykernel_69482/1408101408.py in <module>
      5 ps1_data = lcdata.read_hdf5('data/ps1.h5')  # downloaded by lcdata_download_ps1 script
      6
----> 7 predictions = ps1_model.predict_dataset(ps1_data)

~/.virtualenvs/dr1-parsnip/lib/python3.8/site-packages/parsnip/parsnip.py in predict_dataset(self, dataset, augment)
   1236         for batch_lcs in loader:
   1237             # Run the data through the model.
-> 1238             result = self.forward(batch_lcs, to_numpy=True, sample=False)
   1239
   1240             # Pull out the reference time and reference scale. Note that if we are

~/.virtualenvs/dr1-parsnip/lib/python3.8/site-packages/parsnip/parsnip.py in forward(self, light_curves, sample, to_numpy)
    962
    963         # Decode the light curves
--> 964         model_spectra, model_flux = self.decode(
    965             encoding, ref_times, color, compare_data[:, 0], redshifts, band_indices
    966         )

~/.virtualenvs/dr1-parsnip/lib/python3.8/site-packages/parsnip/parsnip.py in decode(self, encoding, ref_times, color, times, redshifts, band_indices, amplitude)
    881
    882         # Figure out the weights for each band
--> 883         band_weights = self._calculate_band_weights(redshifts)
    884         num_batches = band_indices.shape[0]
    885         num_observations = band_indices.shape[1]

~/.virtualenvs/dr1-parsnip/lib/python3.8/site-packages/parsnip/parsnip.py in _calculate_band_weights(self, redshifts)
    318         remainders = flat_locs - int_locs
    319
--> 320         start = self.band_interpolate_weights[..., int_locs]
    321         end = self.band_interpolate_weights[..., int_locs + 1]
    322

IndexError: index -9223372036854775808 is out of bounds for dimension 0 with size 25485

pip freeze
appnope==0.1.2
argon2-cffi==21.1.0
astro-parsnip==1.0.1
astropy==4.3.1
attrs==21.2.0
backcall==0.2.0
bleach==4.1.0
certifi==2021.5.30
cffi==1.14.6
charset-normalizer==2.0.4
cycler==0.10.0
debugpy==1.4.3
decorator==5.0.9
defusedxml==0.7.1
entrypoints==0.3
extinction==0.4.6
h5py==3.4.0
idna==3.2
ipykernel==6.4.0
ipython==7.27.0
ipython-genutils==0.2.0
ipywidgets==7.6.4
jedi==0.18.0
Jinja2==3.0.1
joblib==1.0.1
jsonschema==3.2.0
jupyter==1.0.0
jupyter-client==7.0.2
jupyter-console==6.4.0
jupyter-core==4.7.1
jupyterlab-pygments==0.1.2
jupyterlab-widgets==1.0.1
kiwisolver==1.3.2
lcdata==1.0.0
lightgbm==3.2.1
MarkupSafe==2.0.1
matplotlib==3.4.3
matplotlib-inline==0.1.3
mistune==0.8.4
nbclient==0.5.4
nbconvert==6.1.0
nbformat==5.1.3
nest-asyncio==1.5.1
notebook==6.4.3
numexpr==2.7.3
numpy==1.21.2
packaging==21.0
pandocfilters==1.4.3
parso==0.8.2
pexpect==4.8.0
pickleshare==0.7.5
Pillow==8.3.2
prometheus-client==0.11.0
prompt-toolkit==3.0.20
ptyprocess==0.7.0
pycparser==2.20
pyerfa==2.0.0
Pygments==2.10.0
pyparsing==2.4.7
pyrsistent==0.18.0
python-dateutil==2.8.2
PyYAML==5.4.1
pyzmq==22.2.1
qtconsole==5.1.1
QtPy==1.11.0
requests==2.26.0
scikit-learn==0.24.2
scipy==1.7.1
Send2Trash==1.8.0
six==1.16.0
sncosmo==2.6.0
tables==3.6.1
terminado==0.12.1
testpath==0.5.0
threadpoolctl==2.2.0
torch==1.9.0
tornado==6.1
tqdm==4.62.2
traitlets==5.1.0
typing-extensions==<IP_ADDRESS>
urllib3==1.26.6
wcwidth==0.2.5
webencodings==0.5.1
widgetsnbextension==3.5.1

I found that PS1 dataset contains NaN redshift values, and NaN being casted to a long integer (here) is some weird number

Thanks for reporting this. parsnip currently requires the redshift to be known, and lcdata doesn't do any input data validation. Instead of using lcdata.read_hdf5('data/ps1.h5'), use parsnip.load_dataset('data/ps1.h5') instead. That will reject all of the light curves with NaN redshift values. I realize that this isn't documented anywhere right now. I'll update that, and add a more informative error message.

I released a new version of ParSNIP (v1.1.0) that should handle the situation that you ran into much more gracefully. Now if you try to run ParSNIP on a dataset that contains light curves without redshifts, it will prune those light curves from the dataset and output a warning suggesting that you use parsnip.load_dataset instead. This should resolve this issue.


[Feature Request if no]Is there a flag in the option to know the webpack object is created from webpack-dev-server?

Is there a flag in the option to know the webpack object is created from webpack-dev-server? If no, can I request this feature?

@webpack-bot move to webpack/webpack-dev-server