Feature Request: Log filtering

We're getting more of our engineers using Tilt and an early piece of feedback was that it would be nice to be able to filter out log messages.
We have a few spammy messages from third party packages and being able to omit them through a regex or similar would be very powerful.

Interesting! Are these build logs (like warnings from third-party packages) or runtime logs (like some debug noise from a library you're calling into)?

The example I was sent was runtime logs from a library.


fix --watch not finding any files to execute; closes #3748

...missing parameter.

appveyor is killing me today

BTW...
$ DEBUG=* /var/tmp/mocha-ya/bin/_mocha --require coffee-script/register --extension="coffee"
  mocha:cli:cli entered main with raw args [ '--require',
  'coffee-script/register',
  '--extension=coffee' ] +0ms
  mocha:cli:options no config found in /var/tmp/sample-node-typescript-module/package.json +0ms
  mocha:cli:options No mocha.opts found at ./test/mocha.opts +48ms
  mocha:cli:run:helpers loaded require "coffee-script/register" +0ms
  mocha:cli:run post-yargs config { _: [],
  diff: true,
  require: [ 'coffee-script/register' ],
  r: [ 'coffee-script/register' ],
  extension: [ 'coffee', 'js' ],
  'watch-extensions': [ 'coffee', 'js' ],       // <= Note this and its camelCase version are different...
  watchExtensions: [ 'coffee' ],
  opts: false,
  reporter: 'spec',
  R: 'spec',
  slow: 75,
  s: 75,
  timeout: 2000,
  t: 2000,
  timeouts: 2000,
  ui: 'bdd',
  u: 'bdd',
  spec: [ 'test/' ],
  config: false,
  package: false,
  '$0': 'mocha' } +0ms
...

camelcase args are thrown out at some point in there before we get to _mocha


Find new maintainers

As I'm no longer using the library (and Scala as well), and I can't arrange the time for reviewing PRs, managing updates, and publishing uap-scala. I would like to hand off some of these responsibilities to new maintainers.
If you're using the library and would be interested in talking about taking on maintenance responsibilities, please either reply here.

As @thyandrecardoso is interested in maintaining this repository. I close this issue.
For transparency, JIRA permission grant log


The action items 'Open' and 'Open in Portal' are not localized for one app configuration under the Quick Access node

Storage Explorer Version: 1.28.0-dev
Build Number: 20221203.4
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 22.04/MacOS Monterey 12.6 (Apple M1 Pro)
Reproduce Languages: All
Architecture: ia32/x64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Launch Storage Explorer -> Open 'Settings' -> 'Application -> Regional Settings'.
Select '中文（简体）' -> Restart Storage Explorer.
Expand the 'Application Configurations' node.
Select one app configuration -> Pin it to the Quick Access node.
Right click the QA app configuration.
Check whether all action items are localized.

Expected Experience
All action items are localized.
Actual Experience
The action items 'Open' and 'Open in Portal' are not localized.

Additional Context
Remove the azure account that associate with the app configuration.
The action item 'View error...' is not localized.

Verified this issue on the build main 20230117.7 with the new app config extension package.
Fixed on all languages. So, we are going to close it.


error on update 2.2.5-->3.1.1

hi there,
i´ve tried to update to 3.1.1 on my win10 system:
`$ ./iobroker upgrade mihome-vacuum
Update mihome-vacuum from @2.2.5 to @3.1.1
NPM version: 6.4.1
npm install<EMAIL_ADDRESS>--loglevel error (System call)
npm ERR! path C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll
npm ERR! code EPERM
npm ERR! errno -4048
npm ERR! syscall unlink
npm ERR! Error: EPERM: operation not permitted, unlink 'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll'
npm ERR!  { [Error: EPERM: operation not permitted, unlink 'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll']
npm ERR!   cause:
npm ERR!    { Error: EPERM: operation not permitted, unlink 'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll'
npm ERR!      errno: -4048,
npm ERR!      code: 'EPERM',
npm ERR!      syscall: 'unlink',
npm ERR!      path:
npm ERR!       'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll' },
npm ERR!   stack:
npm ERR!    'Error: EPERM: operation not permitted, unlink 'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll'',
npm ERR!   errno: -4048,
npm ERR!   code: 'EPERM',
npm ERR!   syscall: 'unlink',
npm ERR!   path:
npm ERR!    'C:\ioBroker\node_modules\iobroker.mihome-vacuum\node_modules\canvas\build\Release\libbrotlicommon.dll',
npm ERR!   parent: 'iobroker.mihome-vacuum' }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It's possible that the file was already in use (by a text editor or antivirus),
npm ERR! or that you lack permissions to access it.
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator (though this is not recommended).
npm ERR!      C:\WINDOWS\system32\config\systemprofile\AppData\Roaming\npm-cache_logs\2021-04-26T15_21_43_397Z-debug.log
ERROR: host.KELLER Cannot install<EMAIL_ADDRESS>4294963248
ERROR: process exited with code 25`

This is a permission error  "EPERM" . Best remove the mentioned files manually and try again

is now working?


You need to enable JavaScript to run this app.

When dashboard is set behind the reverse proxy I get the following message when I visit the page:
You need to enable JavaScript to run this app.

Should I change anything with the proxy config to make it work?

I'm seeing the same behavior, and I'm hitting the container directly over port 8080 - also in docker. I had used a pervious version and it worked fine with this same configuration, I don't know what changed.

I think I may have solved this issue...
In the package.json you need to add the below
Under the base {} add
  "proxy" : "http://<IP_ADDRESS>:8080",
  "homepage": ".",

The for the scripts update http-server:app
    "http-server:app": "http-server ./build --proxy http://<IP_ADDRESS>:8080 --port 3000",

This seems to have solved it.
You will need to build the docker image your self locally.
Using the below, I was able to test it and it worked
docker build -t test:1 .

Then running the app
 docker run -v $(pwd)/data:/data -p 8080:8080 test:1

@userbradley Could you make a PR?

@biboc why can't you?

No time for the moment but that would help people using it with docker :)


Feature request: Vault Harvest History

Please put each vault's harvest info/history or link to it right into each Vault on the UI.
Needs history data

Implemented in #265. Shipped in #273.


How can I set popup list in the center of the screen.

I want to show dropdown list as below image

How can I achieve this?

@sagark1510
Try adjustFrame and the style.


Hash-shuffle for dask.dataframe needs task solution

Some merge operations use a pure hash join, rather than a sorted index join.  We implemented a task-based sorted-index join in https://github.com/dask/dask/pull/1186 as well as a shuffle for dask.bag.  However we failed to implement a hash-join, which is generally about the same as the dask.bag shuffle.

In the mean time this bug can be avoided by using non-shuffle groupbys like df.groupby(df.x).y.sum() or by explicitly setting the index ahead of time.

Fixed


chore: version bump to fix vulnerabilities

Bump outdated versions to current ones to fix vulnerabilities. Deprecation is not yet merged to master. Code should be secure before deprecation.

@magrusch Thanks a lot for this PR. LGTM


MediaError static fields uses Doubles and does not have defined constants

We cannot switch on MediaError as it has doubles for the static fields, instead of ints.
Also the actual values are not assigned as constants, which also means it cannot be used in a switch statement.
https://developer.mozilla.org/en-US/docs/Web/API/MediaError

Contribution is welcome here.
This can be easily fixed by

modifying dom/integer_entities.txt for listing these members as integer entities.
Mark these members as constant in google/closure-compiler/blob/master/externs/browser/html5.js

@jDramaix This issue is now completed and can be closed

Thanks!


Fixed underlines getting clipped in Firefox. Fixed font rendering thi…

Fixed font rendering thicker in Firefox
Chrome

Firefox

Fixed underlines being clipped in Firefox
Any component using the .list__link class was affected, for example 'footer', 'bare list', 'dashed list' etc, due to overflow-hidden being applied.
Chrome

Firefox

 Issue #851: Typography is rendering differently in Firefox than it is in Chrome

 Issue #1363: Link underlines have not been correctly applied


Ability to get a course grade for a single student regardless of whether course grades are released

Required for #247.
In dynamically updating a student's course grade when a score changes, the current endpoint returns 'null' if the course grades are configured to be not released to students.  It'd be great to have the ability to pull back the latest course grade for a single student, also formatted accordingly to the Gradebook settings.

Checked out the service code for this. There is a hard check in there to not do anything if its not released. However we can probably bypass that with a role check. We'll need this bypassed for both instructor and TA correct?

Yep that sounds right - the course grade column is visible to both instructors and TAs so that makes sense.

This issue was moved to sakaiproject/sakai#1383


Correcting an argument typo and renaming folder SQL VM

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

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@tjprescott I have a doubt. One of the tests is failing with:
The following files should be added to doc_source_map.json:
src/command_modules/azure-cli-sqlvm/azure/cli/command_modules/sqlvm/_help.py
Once I run it locally, a lot of command modules have the same error. How should I unblock this test? What do I need to add? Thanks :)

Looks like you need to update the doc source mapping for your path changes.

@yareyes
https://github.com/Azure/azure-cli/blob/dev/doc/sphinx/azhelpgen/doc_source_map.json
You need to add the path to your _help.py file here.


Incorrect input shape of Convolution2D

Hi,
From the documentation at https://keras.io/layers/convolutional/. They suggest that the input for the convolution should follow the order: channels, rows, cols. However, when I run the example codes:
model = Sequential()
model.add(Convolution2D(64, 3, 3, border_mode='valid', input_shape=(3, 256, 256)))
print model.output_shape # output (None, 1, 254, 64)
I get (None, 1, 254, 64) instead of (None, 64, 254, 254). When I switch input shape to: rows, cols, channels, I got the correct result.
model = Sequential()
model.add(Convolution2D(64, 3, 3, border_mode='valid', input_shape=(256, 256, 3)))
print model.output_shape # output (None, 254, 254, 64)
I use Keras version: 1.1.2 (updated from master branch).

input_shape depends on K.image_dim_ordering() that you are using(can be found in ~/.keras/keras.json), you can check here. I think in your case image_dim_ordering is tf.
Hope it helps.


"First elected"

Currently, "first elected" shows when the person was first elected to congress. Jeff expressed that for the congressional seniority sort, it should instead use when the person was first elected to their current chamber. This would probably require rejiggering either the old or new db schemas.

Is this a page we want to have persons or 'members' (i.e. congress-icpsr
pairs)?
On Wed, Sep 14, 2016 at 10:24 PM, Aaron Rudkin<EMAIL_ADDRESS>wrote:

Currently, "first elected" shows when the person was first elected to
congress. Jeff expressed that for the congressional seniority sort, it
should instead use when the person was first elected to their current
chamber. This would probably require rejiggering either the old or new db
schemas.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/JeffreyBLewis/WebVoteView/issues/19, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AS6QixJ9ZITA-QkXzFNSRXopmz554IDfks5qqNacgaJpZM4J9ght
.

This is not really a page at all. It's just a property of several pages (i.e. congress listings, party listings). Currently both pages use "distinct" sortings (persons) even on the old schema, so you don't get duplicates of the same person. The data that drives this is the congresses field on voteview_members in the old schema. Congresses is a list of lists, each list being a continuous run of congresses someone is in. The sort value is the begin period of the first set of congresses. We would need to convert these 2-tuples into 3-tuples to add a chamber, and then the sort code would check for that.
Sort either of these pages by seniority.
http://<IP_ADDRESS>/congress/senate
http://<IP_ADDRESS>/parties/555
Jeff noted that Chuck Schumer sorts incorrectly on seniority in this wiki article:
https://github.com/JeffreyBLewis/WebVoteView/wiki/Things-to-add-or-fix

It is possible to get duplicates, for instance Ben Campbell in S104, who changed party mid-congress. This may be the desired behavior, though, since he has two different ICPSRs and nominate scores during that congress.
In this particular case, he changed party in his first congress, so the duplicates are adjacent, being a freshman in both cases. However, in the general case, if a legislator changes party in a congress after their first, however, they'll appear twice and separately in the list. This could result in a user missing the member they're looking for, because they find the pre-change member when they meant the post-change person, or they might not realize there are two at all.
Should the unit of first-elected be the icpsr or the person? @JeffreyBLewis

Interesting.  I would think first elected should refer to the DNA not the
party-DNA.  On the other hand, maybe it should refer to the
person-chamber.  That is, the question is when where you first elected the
office that you hold so that Chuck Schumer doesn't look like a long-time
Senator when he moves from the House.
Best,
Jeff
On Thu, Sep 15, 2016 at 10:27 AM, adamboche<EMAIL_ADDRESS>wrote:

It is possible to get duplicates, for instance Ben Campbell in S104, who
changed party mid-congress. This may be the desired behavior, though, since
he has two different ICPSRs and nominate scores during that congress.
In this particular case, he changed party in his first congress, so the
duplicates are adjacent, being a freshman in both cases. However, in the
general case, if a legislator changes party in a congress after their
first, however, they'll appear twice and separately in the list. This could
result in a user missing the member they're looking for, because they find
the pre-change member when they meant the post-change person, or they might
not realize there are two at all.
Should the unit of first-elected be the icpsr or the person?
@JeffreyBLewis https://github.com/JeffreyBLewis
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JeffreyBLewis/WebVoteView/issues/19#issuecomment-247394904,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEinZVHH5jzys2N-kOY1qd_fxwiBqurJks5qqYALgaJpZM4J9ght
.

--
Jeffrey B. Lewis
Professor and Chair
Department of Political Science
University of California, Los Angeles
BOX 951472, 4289A Bunche Hall
Los Angeles, CA 90095-1472
President
The Society for Political Methodology

RE: "This could result in a user missing the member they're looking for, because they find the pre-change member when they meant the post-change person, or they might not realize there are two at all." we do have a safeguard against this--member pages will helpfully direct you to previous or subsequent incarnations of the same person. So they might miss on their first click, but we don't dead end them.
(Not that this distracts from the broader point about the feature!)

This ended up being a little easier than I thought. It's only in the old schema, but basically I gave each ICPSR a "electedSenate" or "electedHouse" field as appropriate, and the seniority ranking on the congress pages now sorts by chamber seniority.
Schumer now sorts in the late 90s rather than the early 80s.
Not done:

[ ] Tracking seniority across party switching
[ ] the "Elected" text under members still says their first date of election.
[ ] Seniority for people who are elected, defeated/retire, and then are elected again still calculates to their earlier date--so if/when Russ Feingold is re-elected, he'll sort as 1993 rather than 2017.

I think longer term on the new schema the correct approach is to retain buckets of continuous service for both the combined chambers and the individual chambers;  and then for the nth congress, look at the beginning of the relevant chamber bucket to get seniority.


feat: add FeaturedProductsCarousel to 404 page

What/Why?
Add the FeaturedProductsCarousel to the 404 page
Depends on:

https://github.com/bigcommerce/catalyst/pull/1812

Testing

Code looks good but how come the carousel doesn't have an appropriate content width?

@migueloller Good question. Looks like even on the vibes component page there's no content width: https://vibes.site/docs/soul/page-not-found
I'm wondering if this was intentional for the design 🤔

Code looks good but how come the carousel doesn't have an appropriate content width?

@migueloller Good question. Looks like even on the vibes component page there's no content width: https://vibes.site/docs/soul/page-not-found
I'm wondering if this was intentional for the design 🤔

It does seem to have one


Keystroke data storage

As a user, I want to record a piece of music starting from when I press a "Start Recording" Button, either on the webapp or physical board. This will start storing button-press information into a consistent data structure.

[ ] Frontend: Clickable start button that enables this recording
[ ] Frontend: Clickable stop button for terminating recording
[ ] Backend: Convert key data into raw file (MIDI or other)
[ ] Database: Table for storing information
[x] Raspberry Pi: Transmit button info & timestamps

Resources:

"Hello World" in MusicXML

React piano installations:

npm install --global yarn
yarn add react-piano

Video: https://www.youtube.com/watch?v=040BKtnDdg0
Code: https://github.com/OneLoneCoder/Javidx9/blob/master/PixelGameEngine/SmallerProjects/OneLoneCoder_PGE_MIDI.cpp

This feature only works if something was actually recorded, when I test it without the Pi's the data is not updated, so it will either not download anything or re-download the last thing that was downloaded.


feat(sms): ability to set test phone numbers

Test phone numbers does not require provider to be set. This gives a way to easily develop your apps and
sign in flow without being charged or using trial credits. The SMS template is logged with the verification code, so developers can also preview them.
Before submitting this PR:
Checklist

[x] No breaking changes
[x] Tests pass
[x] New features have new tests
[x] Documentation is updated

Breaking changes
Avoid breaking changes and regressions. If you feel it is unavoidable, make it explicit in your PR comment so we can review it and see how to handle it.
Tests

please make sure your changes pass the current tests (Use the make test or the make watch command).
if you are introducing a new feature, please write as much tests as possible.

Documentation
Please make sure the documentation is updated accordingly, in particular:

Workflows. Workflows are Mermaid sequence diagrams
Schema. The schema in a Mermaid ER diagram
Environment variables. Please adjust the .env.example file accordingly
OpenApi specifications. We are using inline JSDoc annotations

Conventional commits
Versioning and changelog are generated following conventional commits. Please do your best to follow the convention, but don't worry about it too much. We will be most likely moving away to changesets soon.

@plmercereau do I need to to something regarding these changesets?

If interested interested in learning changesets we could set a call tomorrow morning if you like. But don't bother otherwise, I'll do it for you
Sorry for the delay!

@plmercereau whenever you can have time, please check it. I will be grateful to have this, so I can stop wasting some SMS credits while developing.


Feature/parse rust filenames correctly

depends on https://github.com/mikepenz/action-junit-report/pull/359
Splitting line number parsing and file name parsing as the file name parsing happens at an usual place -- I am not sure whether this should happen somewhere else
When cargo2junit, the filenames are not encoded in the xml, i.e. parse
them analogue to line number parsing.

@kesselborn thank you very much for the PRs I'll have a look at them in the next few days.


Soft deletion of aspect

Toady we support soft deletion of entities using aspects like Status.
WDYT on soft deletion of aspect ?
Say, today I fill up aspect DatasetProperties for one dataset D1. And, tomorrow I want to remove aspect DatasetProperties for dataset D1 ( in other words, this aspect is no more valid for D1)
How do we model this?

My original thought is to have the aspect owner add custom fields (say removed) in their aspects and self interpret it. However, I do see value having a consistent way to represent this. One method I can think of is to use a sentinel value (e.g. empty JSON {}) to represent deleted aspect. This should be fairly easy to implement such logic in LocalDAO but may require storage-system-specific logics.

Another RFC? :P

+1 on RFC (Naga, lol)
This is interesting. Some initial thoughts about it. I am working on getting away from soft deletion for secondary indices. If an aspect is soft deleted, we need to think whether the relationship derived from it needs to hard or soft deleted.  Also, status itself is an aspect, need to clarify what does it mean if Status aspect is soft deleted? Lastly, should the derived aspect that is built on top of other aspects be marked as deleted if one of the underlying aspect is deleted?

I think there are some cases for querying soft deleted entities from secondary indexes, but it probably isn't the 90% use case.
Example: "Find all my removed upstream datasets" (good indication my pipeline is broken)
Another example maybe "find all datasets with inactive owners". Granted, "inactive" users (left the company) are not the exact same thing as "removed" datasets, but it's sort of the same, right? Some bit of the metadata.

I'm not saying those are good use cases / enough to justify only soft deletion, I'm just playing devil's advocate.

we need to think whether the relationship derived from it needs to hard or soft deleted

I believe yes. We should invalidate the relationships in graph . Relations exist only when the aspect is present.  If the aspect powering the relations is invalid, then the relations created as a result of this aspect should be invalid too.
How we invalidate those relations, whether soft or hard deletes depends on the use case.  I believe @jplaisted  is right on the point, individual use cases drive hard/soft deletions in the graph relations (or secondary indices)
Or, we could take a generic stance and say we support only soft deletions. But, that can mean bloating up the indices.

Lastly, should the derived aspect that is built on top of other aspects be marked as deleted if one of the underlying aspect is deleted?

No, I don't think so. Derived value should change based on the status of the underlying aspect. My derived aspect can be referring to more than one aspects and the logic of how I compute the derived value can account for (in)activeness of the underlying aspects.

Marking this as feature request.

Curious to understand why one would want to purge a particular aspect, but not an entity?
If its because the data is incorrect, it should suffice to emit a new aspect version that represents a cleanse, right? (ie a new ownership aspect with an empty owners array)
In what cases does this approach not suffice?

If its because the data is incorrect, it should suffice to emit a new aspect version that represents a cleanse, right? (ie a new ownership aspect with an empty owners array)

Think of a case where I don't know the correct owners here. Hence, I want to invalidate all previously said metadata.
GMA has never been designed keeping such use cases. But, we have seen increasing number of such use cases where one wants to invalidate the previously said metadata. Fortunately, we have a way to invalidate an entity via Status aspect. But there is no way to invalidate further at the aspect level.

We are now managing feature requests through our new portal to improve visibility & prioritize work into our public roadmap. If this request is still relevant, please create a new issue so we can gauge current interest from the Community & prioritize work accordingly.
We'll close out this issue in the meantime. Feel free to reach out to me on DataHub Slack with any questions!


Linux Execution Issues

Hello! I'll preface this by saying I'm still somewhat new to Linux so my apologies if the answer to this is obvious.
I've been able to launch the Linux version of DDTO if I'm in the directory containing the [DokiTakeover] executable by using [./DokiTakeover].
However, if I try to run it from any other directory like in these examples:
While in assets: ../DokiTakeover
While in the directory above [Doki.Doki.Takeover.Linux]: /Doki.Doki.Takeover.Linux/DokiTakeover
I get the error:
Error : Could not load module lime@lime_application_create__prime
I'd imagine this probably has more to do with the Haxe Engine then your mod in specific and I can still run the mod, but I nest directories to organize files so it's a bit of a pain to navigate to that folder every time instead of using a .desktop file.
If anyone has any advice that would be appreciated. Thanks!
I'm running:
Linux Mint 20.3 x86_64
bash 5.0.17

The game's assets are stored inside the asset folder, so when the executable is moved elsewhere it'll try looking for an assets folder in its directory with no luck.
I'm pretty sure you can make a .desktop file that points to the path of the executable.

Seems as though I was misunderstood. I will try to provide clarification.
I did not rearrange or remove any files after extracting [Doki.Doki.Takeover.Linux.tar.gz].
When I typed:
However, if I try to run it from any other directory like in these examples:
While in assets: ../DokiTakeover
While in the directory above [Doki.Doki.Takeover.Linux]: /Doki.Doki.Takeover.Linux/DokiTakeover
I was not trying to say that I moved the [DokiTakeover] executable to the [assets] directory or anywhere else, but instead that I navigated to the [assets] directory in the terminal and then used the bash shortcut ".." to look in the directory above [assets] and execute [DokiTakeover]. The purpose of this was to try to executing [DokiTakeover] outside of its directory and instead providing a file path there.
My apologies for the confusion. (It's also entirely possible I misunderstood what you typed as well to which case I am sorry for the trouble!)
I have done further testing with a newly downloaded and extracted copy of [Doki.Doki.Takeover.Linux.tar.gz].
The testing and results are as follows:
A new copy of [Doki.Doki.Takeover.Linux.tar.gz] is downloaded to /home/(user)/doki
[Doki.Doki.Takeover.Linux.tar.gz] is extracted in /home/(user)/doki
I run navigate to /home/(user)/doki/Doki.Doki.Takeover.Linux in a terminal and run sudo chmod 777 DokiTakeover
DokiTakeover is set to [Allow executing file as program]
Double clicking the program displays an error for [Unknown file type]
Navigating to /home/(user)/doki/Doki.Doki.Takeover.Linux in the terminal and running ./DokiTakeover runs the game as normal
Navigating to /home/(user)/doki in the terminal and running Doki.Doki.Takeover.Linux/DokiTakeover results in the error:
[Error : Could not load module lime@lime_application_create__prime]
Navigating to /home/(user)/doki in the terminal and running ./Doki.Doki.Takeover.Linux/DokiTakeover results in the error:
[Error : Could not load module lime@lime_application_create__prime]
Navigating to any other directory and providing a file path to execute [DokiTakeover] results in the error:
[Error : Could not load module lime@lime_application_create__prime]

I would like to humbly request that this issue be reconsidered based on my second post @ActualMandM

It sounds like an issue related to Lime itself, which means there's nothing that we can really do on the FNF side of things.


Release azure-cli-dls 0.0.5

Reviewers, please verify the following for this PR:

[ ] The PR title has format Release <component-name> <version>.
[ ] setup.py has been modified with the same version as in the PR title.
[ ] HISTORY.rst has been modified with appropriate release notes.
[ ] If releasing azure-cli or azure-cli-core, __version__ defined in __init__.py should also be modified to match.

Started releasing azure-cli-dls 0.0.5...

Release of 'azure-cli-dls' with version '0.0.5' successful. View at https://pypi.python.org/pypi/azure-cli-dls/0.0.5.

GitHub release created. https://github.com/Azure/azure-cli/releases.

Codecov Report

Merging #3221 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #3221   +/-   ##
=======================================
  Coverage   70.35%   70.35%
=======================================
  Files         381      381
  Lines       24777    24777
  Branches     3785     3785
=======================================
  Hits        17431    17431
- Misses       6237     6240    +3
+ Partials     1109     1106    -3

Impacted Files
Coverage Δ

src/azure-cli-core/azure/cli/core/util.py
68.61% <0%> (ø)
:arrow_up:

...dback/azure/cli/command_modules/feedback/custom.py
34.69% <0%> (ø)
:arrow_up:

...-cli-role/azure/cli/command_modules/role/custom.py
35.74% <0%> (ø)
:arrow_up:

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 39a8fcd...f9a2b5e. Read the comment docs.


Child Nodes Implementation

Hi devs, I'm Nishan. Right now I'm using this amazing library and thank you guys for developing this. But I have a small issue, I need to implement child nodes for each node. So is there any way to do that or is there any possible way to access the full node? I want to add child nodes same as nodes. The child node should be a normal node and I want to add that child nodes programmatically. Thank you.

View

#423

Thank you


Android Basics: Coroutines

name: Android Basics Unit 4 issue
about: Report problems with the Android Basics Unit 4 codelabs
title: 'Android Basics Unit 4 issue:'
labels: ''
assignees: ''

URL of codelab
https://developer.android.com/codelabs/basic-android-kotlin-training-introduction-coroutines#3
In which task and step of the codelab can this issue be found?
in provided example:
fun CoroutineScope.launch {
    context: CoroutineContext = EmptyCoroutineContext,
    start: CoroutineStart = CoroutineStart.DEFAULT,
    block: suspend CoroutineScope.() -> Unit
}

Describe the problem
I guess it should be:
fun CoroutineScope.launch(
    context: CoroutineContext = EmptyCoroutineContext,
    start: CoroutineStart = CoroutineStart.DEFAULT,
    block: suspend CoroutineScope.() -> Unit
)

@xinneph
Thank you for reaching out to us. Your concern regarding the code provided in the codelab was correct. The actions were taken for updating it:

Happy coding!


Regex needs to be corrected

\( should be \\( I think, otherwise it is implemented as an incorrect Python string escape.
https://github.com/TranslatorSRI/Babel/blob/8b1a23dedc7523b339130ed0c3626dcc50b54abe/src/datahandlers/kegg.py#L153

Fixed in PR https://github.com/TranslatorSRI/Babel/pull/273, closing.


HTTP Error 429

Hi Team,
I get the following error. Kindly help.
ERROR: Unable to download webpage: HTTP Error 429: Too Many Requests (caused by HTTPError()); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; see  https://yt-dl.org/update  on how to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Command run is:
youtube-dl -o 'tmp/LZWszPv_cAw.%(ext)s' --extract-audio --audio-format flac --prefer-ffmpeg https://www.youtube.com/watch?v=LZWszPv_cAw
youtube-dl version = 2019.09.28

Changed the ip of the server, solved the issue. I guess youtube had blocked the server.


fix(config): switch master branch name to main

Related Ticket(s)
{{Provide url(s) to the related ticket(s) that this pull request addresses}}
Description
{{Add a human-readable description / detail summary of what the PR is changing and any details around how and why}}
{{If applicable, include a screenshot indicating an example or examples of what the PR is changing in the application}}
Changelog
New

{{new thing}}

Changed

{{changed thing}}

Removed

{{removed thing}}

Thanks @annawen1 !


Narrow traceback duration

aka "Traceback Duration"
Outcome: Only send exposures for dates that users are contagious.
As a user, I want to know if I have come in contact with someone who is contagious, so I can take the right precautions.

Link to Figma mockups: https://www.figma.com/file/0pLawavG1fnBTXSYn4pdKZ/Exposure-notification?node-id=10721%3A20478
Screenshots for reference. Refer to Figma to inspect designs.

Focused input + character chunking. Allow users to input chars and spaces. Allow pasting. Auto format if users do not enter the spaces.

Secondary button always goes back to step 2 (Radio screen)

 Implement traceback duration

Based on the conversation from yesterday we've also moved the "prefer not to say" option out of the date picker.  It's now a button.

Noting:
Yesterday we added the radio buttons - we'll need to work on ensuring the are accessible (today).

Might help with context for that button and draw attention to the text.

I brought this issue in a thread in the interaction design channel. The two buttons together reinforces the idea of having a choice. Also, by separating the buttons, people might skip reading below the primary button altogether.
All good on the "Prefer not to say" button.
Make sure we have an empty option in the date-picker so we don't force a default date.

updating with content changes @timarney @smcmurtry
View flow using the same link as before. Here are images for reference!

With the flow:

 Create error messaging for traceback duration

@smcmurtry has started testing on this.

Closing notes will be added to the PR https://github.com/cds-snc/covid-alert-app/pull/1109


If possible, do not use FlagValues.FlagDict.  Or use FlagValues._flags instead.

This is a followup for https://github.com/GoogleCloudPlatform/PerfKitBenchmarker/pull/1175/commits/4b6c2dbeac5a2fc2b46686a78fd65da468631ca7

LGTM. @tedsta, can you check this out too?

Kindly ping? @tedsta


Get invalid option

Is there a way to get an invalid option, like an option that doesn't exist in array of cag_option?
while (cag_option_fetch(&context)) {
  ident = cag_option_get(&context);
  switch (ident) {
  case 'h':
    printf("%s\n", "Usage: program [-h]");
    exit(EXIT_SUCCESS);
    break;
  default:
    // here's where it handle invalid option; below is an example
    printf("Error: invalid option '%s'\n", cag_option_get_invalid(&context));
    exit(EXIT_FAILURE);
    break;
  }
}

Hi @UrNightmaree ! ident will be set to '?' if an option is not known.
However, there is currently no API to get the letter/name of the invalid option. You could remember the index from the context before fetching, but that's not ideal. It would actually be fairly easy to implement here for long names and here for access letters by storing the info in the context. But it's currently not there.

Does argv[context->index] will get the invalid option?


Pod is not available in cocoapods repo

Hi,
I've noticed that this pod is not available in the master repo for cocoapods. It is a bit confusing since it's counterpart - ContentfulDeliveryAPI is available through master repo of cocoapods.
Another thing I've noticed when added source declaration - I'm seeing a warning of duplicated specs when installing pods:
[!] Found multiple specifications for `Result (0.6-beta.1)`:
- HOMEFOLDER/.cocoapods/repos/contentful/Result/0.6-beta.1/Result.podspec.json
- HOMEFOLDER/.cocoapods/repos/master/Specs/8/e/e/Result/0.6-beta.1/Result.podspec.json

I've checked files and the spec you have in https://github.com/contentful/CocoaPodsSpecs/tree/master/Result/0.6-beta.1 is identical to what is published in master repo.

Hey @maxkomarychev,
Thanks for bringing this to my attention. I'll get this published to the main cocoapods repo ASAP.

Hey @maxkomarychev, a quick update—I will need to update the Realm dependency to make the Pod ready for publishing. This shouldn't take long, but may cause a delay of a day or 2 if there are unforeseeable roadblocks.

Hi @maxkomarychev
This repo is now published to Cocoapods, so it should no longer be necessary to add an extra line to your Podfile to source an additional specs repo.
The package name has changed and to add this repo to your project you will now just need the following in your Podfile:
pod "ContentfulPersistenceObjC"

A necessary step to publishing was updating Realm from v0.98.0 to v2.5.0. If you use Realm, please let me know if you encounter any regressions.


Ratios displaying as float

I'm new to Clojure development, but ratios don't seem to be working properly in my proto-repl. In lein, 1/2 returns 1/2, but in proto-repl 1/2 returns 0.5.

this is a duplicate of #77

Check out the details on #77 to understand why it's happening. You can disable automatic pretty printing to stop that in the REPL.


Expose plate-level comments in sample plating and gdna extraction interfaces

Original issue title: need to expose plate-level comments in plating interface

Greg says: need this for sample plating and gdna extraction

I'm working through this issue at the moment. If I understand correctly, the piece that seems to be missing is a textbox to add plate-level notes in the plating interface i.e. along with the spreadsheet, etc. And in addition, it would be useful if these notes could be displayed in the Extract gDNA interface.
I believe the last part is already there, for example in this plate I added notes via de Python API and those were correctly loaded when I accessed that plate:

I want to make sure that I am understand what we are going for, because these notes we care about (for this issue) are plate-level, not sample/well-level right?

You have highlighted a key point here that we may need to discuss more.  Based on my understanding of the current code in gdna_extraction_process.GDNAExtractionProcessHandler.get, the notes collected/shown in that interface are actually NOT (despite what they appear to be) notes attached to the gDNA plate--they are notes attached to the gDNA extraction process.  We already have pretty good support in the code api (and a few places in the interface) for inputting notes on a process--as I recall, we added that earlier in the launch push so that users could indicate why they had to redo quantification processes, and it wasn't hard to make it a bit more generalized.
What this issue requested as written, was plate notes, which are supported in the database but as far as I've found not much supported yet in the code api or gui ... there is notes getter/setter on the Plate object, but notes can't be passed as part of the create call for Plate or anything.
A really, really good question would be whether the need is definitely for notes on the Plate object or whether notes on the process that generated the Plate object (and that basically appear associated with the Plate object, as shown in the example you gave) would be good enough.  If process notes are good enough, our lives will be easier ...
@tanaes , since you created the issue, could you weigh in on the use case(s) driving it, so we can determine whether process notes would do?

My understanding is that notes associated with the plating process are what
we want here, and indeed I think that should be a natural extension of the
generic Process.notes function. I don't see any reason to have a notes
column attached the Plates table per se -- in actuality these plates are
all being created in a process, and it's that process for which we're
wanting to record notes.
On Wed, Jul 25, 2018 at 2:25 PM Yoshiki Vázquez Baeza <
<EMAIL_ADDRESS>wrote:

A really, really good question would be whether the need is definitely for
notes on a Plate object or whether notes on the process that generated a
particular Plate object (and that basically appear associated with the
Plate object, as shown in the example you gave) would be good enough.
You are right, I hadn't even begun to consider those annotations. Was too
focused on the sample and plate-level notes. As I think about this, a
comprehensive description of the notes associated with an object would
likely be a better fit for the "explore interface" in #77
https://github.com/jdereus/labman/issues/77.
I would indeed appreciate @tanaes https://github.com/tanaes' input on
this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jdereus/labman/issues/242#issuecomment-407901117, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6JAAJniJxdNEc29SGbrfocszeEGk2bks5uKOJcgaJpZM4UMo7F
.

@tanaes  cool!  I think the reason that plates themselves have notes is in case there is something one needs to say about a plate that is not associated with any particular process, but with the persistent physical article of the plate.  For example, "This plate was discarded because we found mouse poop in it"  :)

Oh interesting... I guess that begs the question of where and when one
would enter that information. 😅 But sounds good for now!
On Wed, Jul 25, 2018 at 3:24 PM Amanda Birmingham<EMAIL_ADDRESS>wrote:

@ElDeveloper https://github.com/ElDeveloper given @tanaes
https://github.com/tanaes input that it is ok to use the notes
functionality on the process level, rather than having to expose the
plate-level notes, I think (hope!) this issue becomes a good deal smaller
in scope.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jdereus/labman/issues/242#issuecomment-407915914, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AH6JADzuGKZ_vDo62ztFk3fs3XfgJH0nks5uKPAmgaJpZM4UMo7F
.

Yes, it certainly does!  As noted above, right now that field really isn't available anywhere through the gui that I can see, and has a pretty small footprint in the api.  For now, I think if one wanted to actually put in a plate-level comment, it would mean finding one's friendly neighborhood database admin and having them modify the notes field directly in the db.  Oh, well ... altering that is a task for another day!

@ElDeveloper given @tanaes 's input that it is ok to use the notes functionality on the process level, rather than having to expose the plate-level notes, I think (hope!) this issue becomes a good deal smaller in scope.

Indeed, thanks both for the help clarifying this. I'll get back to the drawing board 👍


Update description label value

Not 100% sure this is what was intended, but it fixes the crash
Here's the gist model
    createdAt = "2015-03-23 15:44:43 +0000";
    descriptionText = "";
    files =     (
    );
    gistID = 969;
    htmlURL = nil;
    jsonURL = "https://api.github.com/gists/969f2df7166ec8f11969";
    public = 1;
    updatedAt = nil;
    user = nil;

Close #21

Uh this was also in https://github.com/RestKit/RKGist/pull/18


not able to change the fontSize, i would appreciate any fix, thanks in advance

not able to overide fontsize with messageStyle prop, i would appreciate any fix, thanks in advance

Hey @rajesh221b , If you look at https://github.com/sidevesh/react-native-snackbar-component/blob/7b0875668a45ac0af106506af72ae8f1355f437d/index.js#L212 fontSize is one of the properties that can't be overriden by the messageStyle prop, you can supply your own render method to textMessage to get better control over this.


Replace topological sort

[ ] I have added the tests to cover my changes.
[x] I have updated the documentation and CHANGELOG accordingly.
[x] I have read the CONTRIBUTING document.

Maintain the following data structures:

A "pending_deps" dictionary that tracks for each node the number of
parents that have yet to complete. This is initialized to each node's
in_degree.
A "tasks_queue" containing nodes that are ready to run. This is
initialized with the parentless nodes (the first layer in the previous
"topological sort").

While not all tasks have been processed, the main thread (running
_run_planned_workflow) fetches tasks from the tasks_queue and
submits them to the execution Thread Pool. When each task completes, its
children's pending_deps is decremented by 1; when a child's
pending_deps reaches 0, the child is pushed to the tasks_queue.
Handle task failures and cancellations by pushing a dummy
node_id (currently -1) to the tasks_queue.
Closes #651

@cjao this is decreasing test coverage by 12%. Can you update the tests so that it isn't decreasing coverage? Seems like the offending modules are app.py and transport.py

That's odd. This patch touches neither app.py nor transport.py.

It's removing this call to get_topologically_sorted_graph()
https://github.com/AgnostiqHQ/covalent/pull/700/files#diff-a99ca6e887d5a6439efcda540e9e2661cdafa9e3a92109f2ac9c390299f794fbL284
I suspect there's some test that was covering that function via the call you removed... that would explain the drop in coverage for transport.py. Is the function spaghetti code now (can it be removed), or does it just need its own unit test?

get_topologically_sorted_graph() is no longer used with this patch. Let me try removing that function.

The remaining references to get_topologically_sorted_graph are:
* `Lattice.check_constraint_specific_sum; Covalent doesn't current support consumable constraints and this code can be safely deleted.

* `Result._write_dispatch_to_python_file`; this appears to reconstructs the workflow source from the Transport Graph. This should be modified to not use  `get_topologically_sorted_graph`.

are you going to update these calls in this branch or should we make new issues for this?

I am inclined to make new issues for them.

thanks @cjao, please make the new issues and link them back to this PR

@cjao @venkatBala is this ready to go? I see no activity on it in over a week

To my knowledge the implementation is correct. However, the details will change once we move from a thread pool back to a single asyncio event loop for managing a workflow (so as to support async exercutors). I would suggest that @venkatBala wait until that change is merged and then make the necessary adjustments to this PR. The implementation will simplify since locks would no longer be needed to avoid concurrency issues.

@venkatBala @cjao what's the status on this? is it good to go?


Add ryzen-ppd: A Power Profiles Daemon driver for AMD Ryzen

Add package ryzen-ppd and add needed nixos module
Things done

Add package

Add programs.ryzen-ppd

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage

[ ] Tested basic functionality of all binary files (usually in ./result/bin/)

24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[x] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

the default settings of the project are not general purpose, I just realized I was accidentally overclocking my laptop. Will rework to move default config to example, and add warning.


Brightness Slider Non Functional

Device Model
iPhone 12 Pro, A14
iOS Version
17.0
Bootstrap Version
1.2.7
Serotonin Version
No response
What happened
When bootstrapping on this version immediately I can't change my brightness. This also happens on previous versions but it's not immediate I think.
Log output
No response
Acknowledgement of README and FAQ

[X] I have read both the README and the FAQ.

I know someone had this issue using Bootstrap and HaruUiX (or something like that I forget the name of the tweak). When he uninstalled HaruUiX it started working again

I know someone had this issue using Bootstrap and HaruUiX (or something like that I forget the name of the tweak). When he uninstalled HaruUiX it started working again

I haven't even heard of HaruUiX


How to enable  multi-GPU training

Thank you for your interest in our work. In this work, I didn't use multi-GPU training since the network is lightweight.
If you want to use multi-GPU training, you can check the usage of "nn.DataParallel" in line 76 of the training code.

thanks for your share


Update live with current master

@BillWagner ok to publish the new topics?

All these topics can go live.  I'll merge once the build finishes.


BUG: Extrinsic/intrinsic swapped in documentation for spatial.transform.Rotation.from_euler() ?

Describe your issue.
The documentation for spatial.transform.Rotation.from_euler() states that a seq with uppercase letters represents an intrinsic rotation, while a seq with lowercase letters represents an extrinsic rotation. Is it possible that this is swapped? (Or that I have a fundamental misunderstanding of extrinsic vs intrinsic rotations?)
As far as I can tell, this concern was first raised in this comment, which provided a simple test case.
A gist with a different check is provided here: https://gist.github.com/whophil/c809692ceb8d5b4a22b70edc103dd87c, but arrives at the same conclusion.
Reproducing Code Example
see gist

Error message
n/a

SciPy/NumPy/Python version information
1.9.3 1.23.5 sys.version_info(major=3, minor=11, micro=0, releaselevel='final', serial=0)

I am wondering if the issue here could be the order in which the angles are provided to the from_euler call. The following piece of code does not raise and it seems to me that (yaw * pitch * roll).as_matrix() replicates the assembly of the R matrix in the example you provide.
import numpy as np
from scipy.spatial.transform import Rotation as R

assert np.allclose(
    R.from_euler("xyz", [15, 10, 5], degrees=True).as_matrix(),
    R.from_euler("ZYX", [5, 10, 15], degrees=True).as_matrix(),
)

roll = R.from_euler("X", 15, degrees=True)
pitch = R.from_euler("Y", 10, degrees=True)
yaw = R.from_euler("Z", 5, degrees=True)

assert np.allclose(
    (yaw * pitch * roll).as_matrix(),
    R.from_euler("ZYX", [5, 10, 15], degrees=True).as_matrix(),
)

I agree with @Patol75, the first rotation matrix from
https://en.wikipedia.org/w/index.php?title=Rotation_matrix&oldid=1122062121#General_rotations
represents an intrinsic rotation around Z, then Y, then X.
That wikipedia section is confusing for several reasons[^1].
Instead, I suggest these slightly better resources:

http://flothesof.github.io/intrinsic-extrinsic-rotations-euler-angles.html#TL;DR
https://www.mauriciopoppe.com/notes/computer-graphics/transformation-matrices/rotation/euler-angles/
https://en.wikipedia.org/wiki/Davenport_chained_rotations#Conversion_to_extrinsic_rotations
https://math.stackexchange.com/questions/1137745/proof-of-the-extrinsic-to-intrinsic-rotation-transform/3314025#3314025

The summary is that you appear to have mixed up intrinsic and extrinsic rotations.
For extrinsic rotations, the order of composition is reversed.
For intrinsic rotations, the order of composition is forward (left-to-right).
From what I understand, this comes from a second reversal of the ordering needed to express the intrinsically-rotated vector in the static "world" coordinate frame.
The from_euler methods to take arguments which specify a rotation $R$
composed of elementary rotations $R_1(α), R_2(β), R_3(γ)$ for angles {α, β, γ}.
See the following:
In [1]: import numpy as np

In [2]: from sympy import symbols, Matrix, sin, cos

In [3]: theta, alpha, beta, gamma = symbols("theta, alpha, beta, gamma")

In [4]: rot_z = Matrix([
   ...: [cos(theta), -sin(theta), 0],
   ...: [sin(theta), cos(theta), 0],
   ...: [0, 0, 1],
   ...: ])

In [5]: rot_y = Matrix([
   ...: [cos(theta), 0, sin(theta)],
   ...: [0, 1, 0],
   ...: [-sin(theta), 0, cos(theta)],
   ...: ])

In [6]: rot_x = Matrix([
   ...: [1, 0, 0],
   ...: [0, cos(theta), -sin(theta)],
   ...: [0, sin(theta), cos(theta)],
   ...: ])

In [7]: R = rot_z.subs(theta, gamma) * rot_y.subs(theta, beta) * rot_x.subs(theta, alpha)

In [8]: alpha_val = np.radians(5)

In [9]: beta_val = np.radians(10)

In [10]: gamma_val = np.radians(15)

In [11]: mat_symbolic_truth = R.subs(alpha, alpha_val).subs(beta, beta_val).subs(gamma, gamma_val)

In [12]: from scipy.spatial.transform import Rotation

In [13]: mat = Rotation.from_euler("ZYX", [gamma_val, beta_val, alpha_val]).as_matrix()

In [14]: np.allclose(mat, np.array(mat_symbolic_truth, dtype=float))
Out[14]: True

In [15]: mat_extrinsic = Rotation.from_euler("xyz", [alpha_val, beta_val, gamma_val]).as_matrix()

In [16]: np.allclose(mat_extrinsic, np.array(mat_symbolic_truth, dtype=float))
Out[16]: True

Other permutations don't work:
In [17]: mat_err1 = Rotation.from_euler("XYZ", [alpha_val, beta_val, gamma_val]).as_matrix()

In [18]: np.allclose(mat_err1, np.array(mat_symbolic_truth, dtype=float))
Out[18]: False

In [19]: mat_err2 = Rotation.from_euler("XYZ", [gamma_val, beta_val, alpha_val]).as_matrix()

In [20]: np.allclose(mat_err2, np.array(mat_symbolic_truth, dtype=float))
Out[20]: False

In [21]: mat_err3 = Rotation.from_euler("ZYX", [alpha_val, beta_val, gamma_val]).as_matrix()

In [22]: np.allclose(mat_err3, np.array(mat_symbolic_truth, dtype=float))
Out[22]: False

In [23]: mat_extrinsic_err1 = Rotation.from_euler("zyx", [alpha_val, beta_val, gamma_val]).as_matrix()

In [24]: np.allclose(mat_extrinsic_err1, np.array(mat_symbolic_truth, dtype=float))
Out[24]: False

In [25]: mat_extrinsic_err2 = Rotation.from_euler("zyx", [gamma_val, beta_val, alpha_val]).as_matrix()

In [26]: np.allclose(mat_extrinsic_err2, np.array(mat_symbolic_truth, dtype=float))
Out[26]: False

In [27]: mat_extrinsic_err3 = Rotation.from_euler("xyz", [gamma_val, beta_val, alpha_val]).as_matrix()

In [28]: np.allclose(mat_extrinsic_err3, np.array(mat_symbolic_truth, dtype=float))
Out[28]: False

To be certain, I checked the implementation which is found in
https://github.com/scipy/scipy/blob/main/scipy/spatial/transform/_rotation.pyx,
where the Rotation.from_euler method is implemented in terms of quaternion composition
using _elementary_quat_compose, the definition of which is:
cdef double[:, :] _elementary_quat_compose(
    const uchar[:] seq, const double[:, :] angles, bint intrinsic=False
):
    cdef double[:, :] result = _make_elementary_quat(seq[0], angles[:, 0])
    cdef Py_ssize_t seq_len = seq.shape[0]

    for idx in range(1, seq_len):
        if intrinsic:
            result = _compose_quat(
                result,
                _make_elementary_quat(seq[idx], angles[:, idx]))
        else:
            result = _compose_quat(
                _make_elementary_quat(seq[idx], angles[:, idx]),
                result)
    return result

So, intrinsic rotations are composed in forward order, extrinsic rotations in reverse order.
I was also confused by this at first, so the documentation could certainly clarify this better.
[^1]: The reasons:
- In aeronautics, the angles are usually ordered by the right-handed roll-pitch-yaw system,
so the order of α, β, γ would be reversed.
- The first and second matrices, both denoted by R in that section,
are not equivalent according to the definitions.
- The last paragraph is unclear and the way I read it only applies to the second R,
for the first R, the "order of operations" is left-to-right.

Friendly ping @scottshambaugh do you have a view here?

(I don't use Euler angles day-to-day so had to dig in here) - I believe @adigitoleo is right in that the "order of operations" is flipped for extrinsic and intrinsic rotations. To add one more example to the pile:
from scipy.spatial.transform import Rotation as R

phi = 30; theta = 20; omega = 45
rx = R.from_euler("xyz", [0, 0, omega], degrees=True)
ry = R.from_euler("xyz", [0, theta, 0], degrees=True)
rz = R.from_euler("xyz", [phi, 0, 0], degrees=True)

r3 = R.from_euler("xyz", [phi, theta, omega], degrees=True)
print('extrinsic xyz')
print((rx.as_matrix() @ ry.as_matrix() @ rz.as_matrix()))  # note the order
print(r3.as_matrix())

r3 = R.from_euler("XYZ", [phi, theta, omega], degrees=True)
print('intrinsic XYZ')
print((rz.as_matrix() @ ry.as_matrix() @ rx.as_matrix()))  # note the order
print(r3.as_matrix())

extrinsic xyz
[[ 0.66446302 -0.49145005  0.5629971 ]
 [ 0.66446302  0.73329482 -0.14410968]
 [-0.34202014  0.46984631  0.81379768]]
[[ 0.66446302 -0.49145005  0.5629971 ]
 [ 0.66446302  0.73329482 -0.14410968]
 [-0.34202014  0.46984631  0.81379768]]

intrinsic XYZ
[[ 0.66446302 -0.66446302  0.34202014]
 [ 0.73329482  0.49145005 -0.46984631]
 [ 0.14410968  0.5629971   0.81379768]]
[[ 0.66446302 -0.66446302  0.34202014]
 [ 0.73329482  0.49145005 -0.46984631]
 [ 0.14410968  0.5629971   0.81379768]]

The docs are definitely not clear here and could use some explanation. That explanation should also include why this is set up the way it is, with an example use case that shows the utility of the inversion. It's too late to change the convention and break backwards compatibility.
@RichardLee0211 your results are due to this inversion of the multiplication order. Your plot should read $r_2 = R^z_\omega R^y_\theta$, and then the results make sense.

(I don't use Euler angles day-to-day so had to dig in here) - I believe @adigitoleo is right in that the "order of operations" is flipped for extrinsic and intrinsic rotations. To add one more example to the pile:
from scipy.spatial.transform import Rotation as R

phi = 30; theta = 20; omega = 45
rx = rX = R.from_euler("xyz", [phi, 0, 0], degrees=True)
ry = rY = R.from_euler("xyz", [0, theta, 0], degrees=True)
rz = rZ = R.from_euler("xyz", [0, 0, omega], degrees=True)

r3 = R.from_euler("xyz", [phi, theta, omega], degrees=True)
print('extrinsic xyz')
# note the order - rx is applied first to a vector, then ry, then rz
print((rz.as_matrix() @ ry.as_matrix() @ rx.as_matrix()))
print(r3.as_matrix(), '\n')

r3 = R.from_euler("XYZ", [phi, theta, omega], degrees=True)
print('intrinsic XYZ')
# note the order - rZ is applied first to a vector, then rY, then rX
print((rX.as_matrix() @ rY.as_matrix() @ rZ.as_matrix()))
print(r3.as_matrix())

extrinsic xyz
[[ 0.66446302 -0.49145005  0.5629971 ]
 [ 0.66446302  0.73329482 -0.14410968]
 [-0.34202014  0.46984631  0.81379768]]
[[ 0.66446302 -0.49145005  0.5629971 ]
 [ 0.66446302  0.73329482 -0.14410968]
 [-0.34202014  0.46984631  0.81379768]]

intrinsic XYZ
[[ 0.66446302 -0.66446302  0.34202014]
 [ 0.73329482  0.49145005 -0.46984631]
 [ 0.14410968  0.5629971   0.81379768]]
[[ 0.66446302 -0.66446302  0.34202014]
 [ 0.73329482  0.49145005 -0.46984631]
 [ 0.14410968  0.5629971   0.81379768]]

The docs are definitely not clear here and could use some explanation. That explanation should also include why this is set up the way it is, with an example use case that shows the utility of the inversion. It's too late to change the convention and break backwards compatibility.
@RichardLee0211 your results are as expected, and due to this inversion of the multiplication order. Your plot should read $r_2 = R^z_\omega R^y_\theta$, and then the results make sense.

I was just looking at this and my interpretation is that extrinsic/intrinsic aren't mixed up but the sequence is applied from right to left mirroring how rotation matrices are applied. It wasn't what I was expecting but makes sense in a way—it should be documented better at any rate.
rotation = Rotation.from_euler("XYZ", [90, 90, 0], degrees=True)
vector = [1, 0, 0]
rotation.apply(vector)
[0., 1., 0.]
rotation_1 = Rotation.from_euler("Z", 0, degrees=True)
rotation_2 = Rotation.from_euler("Y", 90, degrees=True)
rotation_3 = Rotation.from_euler("X", 90, degrees=True)
(rotation_3 * rotation_2 * rotation_1).apply(vector)
Also [0., 1., 0.]
rotation_3.apply(rotation_2.apply(rotation_1.apply(vector)))
Also [0., 1., 0.] as well

@adbugger @larsoner @nmayorov @rgommers it looks like you worked on #8945 which introduced Rotation.from_euler(). I know it's been awhile but do you have any insight about what the order of seq is supposed to mean and how the documentation for intrinsic vs extrinsic rotations should be fixed?

Looking over the code from my original implementation, I think I can clarify a few points here:

Rotation.as_matrix returns the final rotation matrix which must be pre-multiplied to a vector to perform the rotation
Rotation.from_euler(seq, angles) describes a sequence of rotations around the given axes with the given angles:

Rotation.from_euler("xyz", [a, b, c]) represents a series of extrinsic rotations, first around x-axis with angle a, then around y-axis with angle b, and finally around z-axis with angle c. Since the rotations are all extrinsic, the frame of reference (x,y,z axes) remains constant throughout.
Rotation.from_euler("XYZ", [a,b,c]) represents a series of intrinsic rotations. First rotation is around x-axis with angle a. Second rotation is around y'-axis with angle b. This y'-axis is body centered and is actually the y-axis (from external coordinate frame) with the first rotation applied. The final third rotation is around z''-axis with angle c. This z''-axis is again body-centered and is actually the z-axis (from external coordinate frame) with the first and second rotations applied.

If R_x, R_y, R_z represent extrinsic rotations around x, y, z with specific angles, then Rotation.from_euler("xyz", angles) is the result of applying R_x, R_y, R_z in sequence, i.e. R_1 = R_z * R_y * R_x.
Similarly Rotation.from_euler("XYZ", angles) is the result of applying 3 consecutive intrinsic rotations R_X, R_Y, R_Z in sequence, where the X,Y,Z axes are in the body centred frame. The resultant rotation matrix R_2 = R_Z * R_Y * R_X. Please note that since the axes are changing R_X, R_Y and R_Z matrices are not equivalent in general to R_x, R_y, R_z matrices.
In both cases "xyz" and "XYZ" the sequence of rotations is the same. First along x-axis, then y-axis, then z-axis.
I think the source of the "order reversal" confusion happens because mathematically R_2 is equivalent to R_1 with the order switched. R_Z * R_Y * R_X = R_x * R_y * R_z. The full rotation (sequence of intrinsic rotations) is equivalent to the simplified external rotations with the sequence switched.
@evbernardes has a better and more beautiful explanation of the math here https://github.com/scikit-image/scikit-image/pull/5263#issuecomment-1422357031
Looking at the gist:

A gist with a different check is provided here: https://gist.github.com/whophil/c809692ceb8d5b4a22b70edc103dd87c, but arrives at the same conclusion.

@whophil I think a comparison between your gist and the linked issue comment will be illuminating. Specifically in the gist, the same matrices cannot be used for both intrinsic and extrinsic rotations. For extrinsic rotations the coordinate axes are not changing, but for intrinsic rotations the coordinate axes do change so to express the final coordinates in the extrinsic frame after an intrinsic rotation requires a different rotation matrix. It also explains why the rotation composition in the gist is applicable for external but not internal rotations.
Apologies for the long comment. I do believe evbernardes' linked comment clarifies all of this in a much neater fashion.
Finally, the relationship between internal and external rotations is also tested here:
https://github.com/scipy/scipy/blob/d55cb95282b5cfab381e373d3c127630c7e915a0/scipy/spatial/transform/tests/test_rotation.py#L558-L565
In summary:

The rotations are always in the order specified by seq. The confusion arises from the fact that while the resultant of intrinsic rotations is the same as extrinsic with order switched, the individual rotations in each are not the same.
I am not sure what documentation changes are exactly required here as I think the docs are already correct. However an explanation similar to https://github.com/scikit-image/scikit-image/pull/5263#issuecomment-1422357031 would clear things up quite a bit.

For what it's worth, I also banged my head against the wall until I did the math myself, as I couldn't find a good explanation on this "order reversal" (which is really not a reversal in convention, just a mathematical quirk).
So yeah, +1 for maybe adding an explanation to the docs.

Thanks for that explanation.
I think I've figured what was tripping me up and some ideas for improving the documentation. I was working with a coordinate transforming/passive rotation but Rotation represents an active rotation, so I needed to invert the rotation to make it agree with my textbook:
rotation_i_was_expecting = Rotation.from_euler("ZYX", [yaw, pitch, roll]).inv()

Rotation representing an active rotation is only mentioned in from_quat() and as_quat() (and kind of in apply()) and I think it would be helpful to briefly mention that in every from/as method. I also think it would help to add a paragraph in the class-level documentation about how for a rotation that transforms coordinates (passive) rather than rotates actual position (active) the Rotation object needs to be inverted or apply()'s inverse parameter needs to be true.


Visual issues when scrolling - Reported by: @parasharrajat

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

Action Performed:

Navigate to a conversation that has a long history.
Scroll upwards and downward in a conversation quickly

Expected Result:
There should not be any visual issues.
Actual Result:
There are several visual issues shown:

Messages are not loading consistently
Messages are loading from bottom to top (should be the other way)
Blank page is also seen in the conversation

Workaround:
No need, visual issue.
Platform:

Where is this issue Confirmed?

Web

Version Number: 1.1.16-2
Reproducible in staging?: Yes
Reproducible in production?: No
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://user-images.githubusercontent.com/44479856/143066584-9914e3ae-375c-4e9e-92a8-a54a52b19fc0.mp4

Expensify/Expensify Issue URL:
Issue reported by: @parasharrajat
Slack conversations:

https://expensify.slack.com/archives/C01GTK53T8Q/p1637599329355400
https://expensify.slack.com/archives/C01GTK53T8Q/p1637599274354300
https://expensify.slack.com/archives/C01GTK53T8Q/p1637599100353300

View all open jobs on GitHub

Triggered auto assignment to @cead22 (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

:wave: Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open StagingDeployCash deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:

Identify the pull request that introduced this issue and revert it.
Find someone who can quickly fix the issue.
Fix the issue yourself.

Triggered auto assignment to @jasperhuangg (Demolition), see https://stackoverflow.com/c/expensify/questions/8099 for more details.

cc: @thienlnam

Not sure why this was closed, its not on staging yet

@isagoico the fix was CP'd to staging. Can you have this retested?

Retest for this was a pass! 🎉 Checking it off.

Am I eligible for reporting bonus here :sweat_smile: ?

Yup! Let me get someone assigned to take care of that!

Triggered auto assignment to @michaelhaxhiu (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

@michaelhaxhiu can you get @parasharrajat paid for reporting this issue? Thanks!

Hm, the External label typically is applied before the job is posted and completed. Did Rajat propose and fix the issue? Or did he report it, and it was fixed internally?
A little confused where is the upwork link for this job?

Rajat reported the issue, and since it was a deployblocker I fixed it in the interest of speed. He is only due the reporting bonus

since it was a deployblocker I fixed it in the interest of speed.

Ah!! Deployblockercash - missed that label, thanks for the context 🙇 . These often move swifty and totally makes sense. Rajat is to be compensated $250 then?

https://www.upwork.com/jobs/~01c796785099359c5e

Rajat is to be compensated $250 then?

Yup, as long as the reporting bonus is still $250!

Paid, all done here.


Opt Avoid copy in set_num_rows implementation

Proposed changes
Issue Number: close #xxx
if the column has no extra reference, we could assume_mutable and just resize it. no need to copy.

run buildall

run buildall


[Feature request] Command line option to ignore tests failures when the exit code is computed

When one or more tests fail at the moment the bats command exits with an error code of 1. If there is a syntax error in the bats file, the behaviour is the same.
The feature would consist of a flag (for example "-i|--ignore-failures"), so that the exit code is 0 even if some tests fail, allowing the caller to differenciate between abnormal behaviour (exit code > 0) and tests failures (exit code = 0, but number of failures > 0 as printed in stdout).

Note:
For the moment in order to achieve the same goal I have to use the following wrapper script:
#!/bin/bash
TEMP_FILE=$(mktemp)
bats tests.bats 2>/dev/null 1>"$TEMP_FILE"
RETVAL=$?
if [ $RETVAL -ne 0 ]
then
  if [ $(wc -l "$TEMP_FILE") -gt 0 ]
  then
    exit 1 # Abnormal error (probably a syntax error)!
  else
    exit 2 # Some tests have failed...
  fi
fi

This project is no longer being actively maintained by its original auther (see https://github.com/sstephenson/bats/issues/236).
You might want to migrate to the community maintained fork bats-core and report your issue there.


show at least a warning, if a workflow would be triggered, but is filtered out because a dependency will be not

as there is no other sign - so you can get easily confused

well it should be a warning in the linter errors ... so it wont hurt you if you use it as feature.


You need to jump or moving boxes and try to survive as long as you can.

How to play your game?

W is jump and L is replay.

Code
Check off the items that are true.

[ Check] The game was made using the Sprig editor.
[Check ] The game is placed in the in the /games directory.
[Check ] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[ Check] The game runs without errors.
[ Check] The name of the file/game contains only alphanumeric characters, -s, or _s.
[ Check] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file. Example: sokoban_plus.js -> sokoban_plus.png.

Thanks for PR!

Good game! were you inspired by any games?

Hello,Thanks for checking out my game so quickly, and to answers your question yes I was inspired by games like Google dinosaur endless and games of the like.Thanks you so much and have a fantastic day,Elijah Schneider On Dec 19, 2022, at 3:39 PM, Lucas @.***> wrote:
Good game! were you inspired by any games?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Enable CD for git-automerger plugin

I made a fix of git-automerger plugin but it seems is no longer published via Jenkins CI. I tried manual publishing, but my credentials for https://repo.jenkins-ci.org/ are not working anymore, so I suppose CD is the straightforward route to publish a fixed version of the plugin.
Reviewer checklist (not for requesters!)

[ ] Check this if newly added person also needs to be given merge permission to the GitHub repo (please @ the people/person with their GitHub username in this issue as well). If needed, it can be done using an IRC Bot command
[ ] Check that the $pluginId Developers team has Admin permissions while granting the access.
[ ] In the case of plugin adoption, ensure that the Jenkins Jira default assignee is either removed or changed to the new maintainer.
[ ] If security contacts are changed (this includes add/remove), ping the security officer (currently @daniel-beck) in this pull request. If an email contact is changed, wait for approval from the security officer.

There are IRC Bot commands for it

so I suppose CD is the straightforward route to publish a fixed version of the plugin.

yup :D


Fix some type convert scalar functions

Since we have added the native BigDecimal support, change the BigDecimal - bytes conversion to directly take and return BigDecimal
Change base64 encode/decode to be symmetric (both input and output are bytes). There are also the toBase64/fromBase64 which are the string version
Fix the missing BigDecimal support in the ScalarTransformFunctionWrapper

Codecov Report

Merging #9509 (3a86e85) into master (aade773) will decrease coverage by 6.22%.
The diff coverage is 76.27%.

@@             Coverage Diff              @@
##             master    #9509      +/-   ##
============================================
- Coverage     69.97%   63.74%   -6.23%
- Complexity     4871     5173     +302
============================================
  Files          1921     1868      -53
  Lines        102398   100095    -2303
  Branches      15538    15274     -264
============================================
- Hits          71649    63804    -7845
- Misses        25687    31605    +5918
+ Partials       5062     4686     -376

Flag
Coverage Δ

integration1
?

integration2
?

unittests1
67.29% <76.27%> (-0.02%)
:arrow_down:

unittests2
15.59% <0.00%> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...sform/function/ScalarTransformFunctionWrapper.java
81.19% <74.54%> (+6.79%)
:arrow_up:

...n/function/scalar/DataTypeConversionFunctions.java
71.42% <100.00%> (+19.04%)
:arrow_up:

...va/org/apache/pinot/core/routing/RoutingTable.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/org/apache/pinot/common/config/NettyConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/org/apache/pinot/common/metrics/MinionMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/apache/pinot/common/metrics/ControllerMeter.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/BrokerQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../apache/pinot/common/metrics/MinionQueryPhase.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...ache/pinot/server/access/AccessControlFactory.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...he/pinot/common/messages/SegmentReloadMessage.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 432 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


amendment misunderstood lock server and results server address

In fact, results server and lock server are function of the paddles, written pulpito is misleading, although pulpito and paddles always installed on one machine.

Can one of the admins verify this patch?

@zhouyuan

@zhouyuan 看下这个PR呗，谢谢。#_#


⚠ Digital Ocean - https://status.digitalocean.com/ has degraded performance

In c7101b2, Digital Ocean - https://status.digitalocean.com/ (https://status.digitalocean.com/api/v2/status.json) experienced degraded performance:

HTTP code: 200
Response time: 183 ms

Resolved: Digital Ocean - https://status.digitalocean.com/ performance has improved in a10920f.


"title" is auto-added to custom components when nested but not top-level

I have a custom field editor for a multi-select dropdown (react-select) and with a somewhat recent change to react-jsonschema-form lib, the titles are no longer showing up on their own for properties at the top-most level. However, when nested fields are using the same editor, the titles are added. My solution originally was to add the label to the custom component, which fixes the top-most level, but then the nested ones now have double titles.

Would you mind trying to provide a reproducible example, ideally a jsfiddle?

Everything works great! thanks for the speedy resolution!


SSH: Add git-perl to have git-add--interactive available

I tend to use git add -p and this is only included in git-perl
Also see https://pkgs.alpinelinux.org/contents?branch=edge&name=git-perl&arch=armhf&repo=main

Hi @lazyfrosch,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

what are the size differences of the image with this?

@pvizeli Since CI doesn't actually build the image on PR, I did it manually:
homeassistant/armhf-addon-ssh                                 with-git-interactive   a72ce074e858        2 seconds ago       176MB
homeassistant/armhf-addon-ssh                                 latest                 a4ad1d1af135        4 weeks ago         148MB

So about 28 MB.

Alternatively we could introduce some configuration to run custom commands on startup of the container.

pvizeli Since CI doesn't actually build the image on PR, I did it manually:

If you rebase now @lazyfrosch it actually does ;)

At least its green! :D

If you scroll to the bottom of the build logs, it also does a docker images so we get the size as well

Merge or not? :wink:
Could we at least add some installation hooks to the config, so you can install or do certain things on starting the addon?

Make an option to install packages on start, so we cover multiple use case


TypeError: Class constructor Enquirer cannot be invoked without 'new'

I keep getting this error
let res = await prompt ({
type:			'input',
name:			'url',
message:		Please give me a url to start scanning
}
    )
    let res2 = await prompt ({
    type:			  'password',
    name:			  'webhook',
    message:		  `Which webhook would you like the images to be sent? `

})

let res = await prompt ({
^
TypeError: Class constructor Enquirer cannot be invoked without 'new'

let res = await prompt ({
^


Hostconfig and Spack package: Where should we draw the line

The context
In the Spack package, we can change a lot of parameters to configure the build.
All these parameters are not transposed to the host-config file because it would make them too specific, and naming them (for example) would be a pain.
Instead, a host-config file aims at configuring the toolchain, and how the projects adapts to it.
In practices:
Everything "host-config" goes into the hostconfig function:
https://github.com/LLNL/Umpire/blob/7e4f2f3c30fc3470b6f3fa2808b837dee2c47939/scripts/uberenv/packages/umpire/package.py#L119
We find there the flags, the compilers paths, the cuda info, etc.
Everything "build-config" goes into the cmake_args function:
https://github.com/LLNL/Umpire/blob/7e4f2f3c30fc3470b6f3fa2808b837dee2c47939/scripts/uberenv/packages/umpire/package.py#L255
We find there the languages support, tests mode, openmp option, etc.
The issue
I wanted to point this out to the team because in order to manually build the project, picking a host config may not be enough, unless one wants to rely on the defaults value of the project (if they are set). Instead, one may have to specify some variables on command line.
Also some parameters may be misplaced:

"deviceconst" may move from hostconfig to cmake_args,
"openmp" may move from cmake_args to hostconfig.

Solution
I would appreciate in someone could review the different parameters and discuss with me if it should be placed in the hostconfig, or specified on command line (typically).
Takaway
A host-config file does not suffice to reproduce a Spack build, although I may have said it was.

I would like a build to reproducible entirely from a CMake "host-config". I believe this is what we originally discussed. To make this work we need a low overhead way of generating these files "on demand" via uberenv. If there is a way to cache the spack clone I think that would be fine.

I see 2 different points in your answer:

All information for a build should go in the host-config (including build_type, omp, API support).
Need for a cached spack, (especially in CI I guess).

Yes, that's right. Is this achievable?

First point is easy, it’s mostly a "strategic" decision :), I’m almost there (with the hash in the host-config name).
Second point will need a bit more work. I’ll keep you posted.


Fix #65

qt_list_str_widget.py:
When there is a change to the current row in the list widget, the selected item
needs to be updated
DictManager.py:
Check that an item exists before popping it off the list
Replace tabs with spaces

I'm not going to make you re-submit this a third time. But for my sanity, could you please separate out tabs->spaces changes from everything else in a separate commit?

Will do.
On 7/30/15 10:42 AM, Blake Johnson wrote:

I'm not going to make you re-submit this a third time. But for my
sanity, could you please separate out tabs->spaces changes from
everything else in a separate commit?
—
Reply to this email directly or view it on GitHub
https://github.com/BBN-Q/PyQLab/pull/67#issuecomment-126351975.


Problem with sphinx-copybutton

Problem
When copying snippets including prompts, and something is wrong on the indentation side, copy doesn't work properly.
Examples

https://crate.io/docs/crate/howtos/en/latest/getting-started/generate-time-series/cli.html#set-up-cratedb

Hi Matthias,
thanks for the report. Our focus was that doctest-style multiline SQL statements with proper continuation symbols work flawlessly. For example, https://crate.io/docs/crate/reference/en/5.0/general/ddl/partitioned-tables.html#information-schema.
However, something obviously goes wrong on certain multiline console snippets.
With kind regards,
Andreas.

Unable to rebuild crate-howtos to confirm this will be resolved now.
-- https://github.com/crate/crate-howtos/pull/310#issuecomment-1192448703

All the terminal-based code blocks on https://crate.io/docs/crate/howtos/en/latest/getting-started/generate-time-series/cli.html should work now, copy/paste-wise.
Another flaw with the SQL DDL block should be mitigated by https://github.com/crate/crate-howtos/pull/311.

thanks @amotl, I'll close this for now since I don't have any other errors like that :)


Rename plugin methods

This is for issue: https://github.com/infinitered/ignite/issues/636
Renamed Methods:
addComponentExample -> addPluginComponentExample
addScreenExamples ->addPluginScreenExamples
removeComponentExample -> emovePluginComponentExample
removeScreenExamples -> removePluginScreenExamples
Left old methods in place that call the new methods and print
deprecation warnings

Sure thing!

@skellock I only see an instance of ExampleRegistry.addPluginExample in the doc, and that isn't what we're changing here ... what are you seeing?

I'm dumb.

Ya, I saw addPluginExample and i somehow thought that was it.  I'm sorry.


PP-5168 Feature flag for Capture process using SQS

maybe document that it will control the behaviour when reading from the queue, rather than writing?

maybe document that it will control the behaviour when reading from the queue, rather than writing?

@heathd We may need to use it when writing to the queue too i.e., when writing to SQS, ignore any exceptions if feature flag is not enabled. When enabled, abort the payment and return 500 for exceptions.


Web service signature description!

It would be VERY useful to specify, either in this page or another related to AutomatedML, that the webservice signature is :
{
"data": [
[
# list of values of fields in alphabetical order (no key)
]
]
}
I had to connect the container to find out the order of features was not the same than in the scoring script of the model.

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

ID: d2fcbcc1-0824-1dba-f5ae-6cf5c97a9d2c
Version Independent ID: 0de15ea0-6def-bf7e-2e68-ae7d434d2d4b
Content: What is automated ML / automl - Azure Machine Learning
Content Source: articles/machine-learning/service/concept-automated-ml.md
Service: machine-learning
Sub-service: core
GitHub Login: @nacharya1
Microsoft Alias: nilesha

Hello @macamhi , thank you for reaching out. We will look into this issue and get back to you shortly.

Hello @macamhi, thank you for your feedback. Did you use the auto-generated scoring script or did you upload a scoring file?

I just wanted to call the web service from a web application, in my case a Flow from PowerApps.
By the way, the sample scoring script didn’t have the same order of parameters than the web service.
Envoyé de mon mobile
Le 26 sept. 2019 à 21:20, GiftA-MSFT <notifications<EMAIL_ADDRESS>a écrit :
Hello @macamhihttps://github.com/macamhi, thank you for your feedback. Did you use the auto-generated scoring script or did you upload a scoring file?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/azure-docs/issues/39590?email_source=notifications&email_token=AHCZOWFR2KMKWGOOPCPT2NLQLUDPDA5CNFSM4I2ZBPJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7WVUJI#issuecomment-535648805, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AHCZOWFGNSFJGXTWQOTJLRTQLUDPDANCNFSM4I2ZBPJA.

@macamhi thank you for clarifying. Does this document (where it says "important") provide the information you're looking for? Are you saying this is not the case as outlined in the documet? If this is not the information you're looking for, can you please provide other document(s) you're following or referencing?

No it doesn’t, it actually misguided me as it suggests the fields are to be in there same order as the scoring script. But, as already indicated below, in the case of a model generated and deployed through Automated Machine Learning, I got a type error when trying to call the web service with data structure as in the sample scoring script generated for the model, and found out by analyzing the container (swagger.json file) that the order of fields in the web service signature was changed to alphabetic order of keys!!!!
So the documentation is WRONG in the case of Automated ML deployment.

@macamhi, thank you so much for the information.
@aashishb, can you please verify if it's an expected behavior for the web service signature of Automated ML to change to alphabetical order? If so, can we specify that information in the document for clarity?

@Blackmist

Thanks for the feedback @macamhi
I'm working with the developers to figure out if this is a bug or expected behavior. If it's expected, we'll get the behavior documented. If it's a bug, we'll add a note to the documentation about it until the bug is fixed.
I'll post back here once I have more information on this.

Welcome, happy it finally made it through.

May I add that it'd be useful to have the Swagger API return a description of the webservice on a particular verb? But that's more a feature request, not sure where I should post it.

@macamhi you can share feedback and/or vote for feature requests on the Azure Machine Learning user voice forum. Thanks.

Hi @macamhi can you provide more details on how you used AutoML to train the model? The engineering team wants to try and reproduce the problem. Was this trained using a notebook or through the UI? Thanks.

I can provide details in private, but I used the full Automated ML path : creation and run of an experiment through Automated ML, selection of the best model in the results, then click on Deploy model .

Thanks @macamhi that might be enough info. I'll see if the engineers need anything else to go on.

Thanks again @macamhi . There's a bug open with engineering and I'm waiting on their feedback on what changes to make to the document. They may contact you directly on GitHub if they require more information.
I'm closing this doc issue as any update is waiting on research from the engineering team now. Please let me know if you have any questions on this.
#please-close

Hey there, just to +1 on this. We've had a similar issue with the web service signature descriptions for forecasting. It would be super helpful to understand the input expected, as it took a couple of days + help on Stackoverflow to deduce this.

@hollyemblem  can you send me a link to the stackoverflow thread? I'd like to add it to the bug for the engineering team.

Sure thing: https://stackoverflow.com/questions/58377444/swagger-json-example-json-for-forecast-model-doesnt-seem-to-return-predictions


Add latency diagnostics for RPC events

Add latency diagnostics to track the time it takes to generate and fetch images, primarily motivated by wanting to understand end to end latency and consider improvements.

Codecov Report

Merging #494 (fef93ef) into main (1ff64a3) will increase coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #494      +/-   ##
==========================================
+ Coverage   91.89%   91.95%   +0.05%
==========================================
  Files          18       18
  Lines        2023     2038      +15
==========================================
+ Hits         1859     1874      +15
  Misses        164      164

Impacted Files
Coverage Δ

google_nest_sdm/diagnostics.py
96.92% <100.00%> (+0.76%)
:arrow_up:

google_nest_sdm/traits.py
97.67% <100.00%> (+0.11%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Can you provide an example of UserController?

In the HandlerGroups section of the docs you describe a clean way to map api implementations, but don't provide an example of what UserController's functions might look like.
Intellij suggests that they should be (Context) -> Unit, however simple functions implemented along those lines don't seem to work. An example in the docs would be super helpful.
Thanks

You have an example here: https://github.com/javalin/javalin-samples/blob/main/javalin5/javalin-auth-example/src/main/kotlin/UserController.kt

Ah alright, they're static. Makes sense, though I'll stick with an alternate code style. How about updating the docs with an example?

Not sure what I was doing, but great, that works.
Question: is there a more canonical way to add path variants that I am doing with "bah" below? I quite like the scoping of Foo, whereby I make all of its resources inside the path block; I can't do that with Bah, because there are two separate paths.
server.apply {
            routes {
                path("api") {
                    path("foo") {
                        val foo = FooResource(FooService())
                        get(foo::getAll, *Role.AUTHENTICATED)
                    }
                    val bah = BahResource(BahService())
                    path("bah") {
                        get(bah::getAll, *Role.AUTHENTICATED)
                        post(bah::create, *Role.AUTHENTICATED)
                    }
                    path("bah/{id}") {
                        get(bah::getById, *Role.AUTHENTICATED)
                    }
                }
            }
        }

Not sure what I was doing, but that works - thanks.
Do you have a solution for this following issue? I like the nesting that the controller approach to setting up routes provides, e.g. with bah here. What I don't like about this is that knowledge of "id" is spread between where routes are now split and the handler (i.e. BahResource.getById).
I can push down relevant path blocks into each resource (e.g. BahResource could be responsible for the path("bah") block in a setup function, and FooResource could call path("foo") in its own setup function), but that feels kind of dirty, because there's nothing enforcing the context in which they're called.
server.apply {
            routes {
                path("api") {

                    path("bah") {
                        val bah = BahResource(BahService())
                        get(bah::getAll, *Role.AUTHENTICATED)
                        post(bah::create, *Role.AUTHENTICATED)

                        path("bah/{id}") {
                            get(bah::getById, *Role.AUTHENTICATED)
                        }
                    }
                 }
            }
        }


Failed to load config for navigator

I was trying to install and when i ran nvim for the first time it throwed this error.
Failed to run `config` for navigator.lua

/Users/sajan.kumar/.config/nvim/lua/modules/lang/config.lua:275: module 'navigator' not found:
	no field package.preload['navigator']
cache_loader: module navigator not found
cache_loader_lib: module navigator not found
	no file './navigator.lua'
	no file '/opt/homebrew/share/luajit-2.1/navigator.lua'
	no file '/usr/local/share/lua/5.1/navigator.lua'
	no file '/usr/local/share/lua/5.1/navigator/init.lua'
	no file '/opt/homebrew/share/lua/5.1/navigator.lua'
	no file '/opt/homebrew/share/lua/5.1/navigator/init.lua'
	no file '/Users/sajan.kumar/.luarocks/share/lua/5.1/navigator/init.lua'
	no file '/Users/sajan.kumar/.luarocks/share/lua/5.1/navigator.lua'
	no file './navigator.so'
	no file '/usr/local/lib/lua/5.1/navigator.so'
	no file '/opt/homebrew/lib/lua/5.1/navigator.so'
	no file '/usr/local/lib/lua/5.1/loadall.so'

# stacktrace:
  - nvim/lua/modules/lang/config.lua:275 _in_ **config**

Same here

It may related to dev setting. Please check if the latest version works thanks.


Portal does not allow for contextualization

There are some cases where a portal needs additional context information. For example:
globalPortal myTexture \t ->
  scene [ mesh (material { map: myTexture }), mesh (material { aoMap: myTexture })]

If the closure myTexture has a payload ie giveToParent { id: me, parent }, there is no way to further specialize this to indicate that myTexture should be a map in one instance and an aoMap in another one.
One way to address this could be to add an extra argument to giveNewParent that allows for specialization.

Fixed in 0.0.2.


How can I dynamically adjust the animation speed in code?

I would like to dynamically adjust the animation speed in code. How can I achieve this? Thank you.

+1

Temporary workaround:
Write your own CustomRiveAnimationView, modify createRenderer() method, return your custom RiveArtboardRender.
Modify RiveArtboardRender#advance() method:
super.advance(elapsed * 2.0) means play animation with double speed.


How to get openset accuracy?

Thanks for sharing your work. May I ask how to calculate openset accuracy? Because, before calculating openset accuracy, it has
probs, preds = F.softmax(self.total_logits.detach(), dim=1).max(dim=1)

so that open_threshold doesn't work.
https://github.com/zhmiao/OpenLongTailRecognition-OLTR/blob/fb6203c8e1cfa7ed5bf9b1acf581800d04f92c4a/run_networks.py#L296

Hello @Qoo95 , the openset accuracy is calculated here: https://github.com/zhmiao/OpenLongTailRecognition-OLTR/blob/master/run_networks.py#L297
Open threashold won't be used if main argument test_open is not True.
Does that make sense?

It has been 26 days since last comment. I will close this issue now. If you have any more questions, please reopen this issue. Thanks.

@zhmiao
so if probs < self.training_opt['open_threshold'] is equal to true, this input image is regard as an open set input (novel class)?


⚠ ClamAV has degraded performance

In 71bc075, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 72 ms

Resolved: ClamAV performance has improved in 6d2c2ee after 536 days, 12 hours, 8 minutes.


Topic name configurable through fluent API

We wanted to set the topic name per environment, so that we can use the same AWS account in all our environments.

@SamHWhale Can you please update your PR before I can merge it?
PR #89 that was merged earlier has caused conflicts with yours.

@SamHWhale, have a look at the other PR #96 , we are proposing a broader approach than just allowing to control topic name. We have the same requirement - we wanted to use same AWS acount for all of our environments. We wanted to control both the queu names and topic names.

Queue pattern - [env]-[instance]-[servicename]
Topic patern - [env]-[instance]-[messagename]

The way to configure that would be to provide a custom INamingStrategy implementation, below would create topic and queue prefixed with environment.
public class BusNamingConvention : IBusNamingConvention
{
    private string environment = "testEnvironment";
    public string GetTopicName(string topicName, string messageType)
    {
         return string.Format("{0}-{1}", environment, messageType);
    }

    public string GetQueueName(string queueName, string messageType)
    {
         return string.Format("{0}-{1}", "testEvironment", queueName);
    }
}

Then use it like that:
     var busNamingConvetion = new BusNamingConvention("myEnvironment");
     CreateMeABus
                .InRegion(awsSettings.Regions)
                .WithSerialisationFactory(busConfigurator.ConfigureSerialisationFactory())
                .WithNamingStrategy(() => busNamingConvention);

Hope this will work for you.

Updated from JustEat/Master @dariusdamalakas, @payman81 decide if he wants to accept this. I'm happy to use either method. I didn't see the need to extend the queue name as there was a method to do this on the FluentApi already.

@SamHWhale I've merged #96 which allows you to provide a naming strategy for topics and queues. Therefore, I'll close this PR.