Could not flatten metadata whose ontology property only requires text

Actual Behavior:
UI displays the error page because the broker api is returning HTTP 500

Broker has error in logs:
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 16, in flatten
    self._flatten_entity(entity, object_key)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 32, in _flatten_entity
    self._flatten_object(content, row, parent_key=worksheet_name)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 66, in _flatten_object
    self._flatten_object(value, flattened_object, parent_key=full_key)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 70, in _flatten_object
    self._flatten_list(flattened_object, object, parent_key)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 74, in _flatten_list
    self._flatten_object_list(flattened_object, object, parent_key)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 84, in _flatten_object_list
    self._flatten_to_include_object_list_to_main_entity_worksheet(object, flattened_object, parent_key)
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 93, in _flatten_to_include_object_list_to_main_entity_worksheet
    flattened_object[f'{parent_key}.{key}'] = SCALAR_LIST_DELIMETER.join([elem[key] for elem in object])
  File "/Users/aaclan/dev/ait/repos/ingest-client/ingest/downloader/flattener.py", line 93, in <listcomp>
    flattened_object[f'{parent_key}.{key}'] = SCALAR_LIST_DELIMETER.join([elem[key] for elem in object])
KeyError: 'ontology'

Steps to replicate:

Restore prod db in the local database
Download spreadsheet of the Heart dataset in the production
http://localhost:4200/submissions/detail?uuid=20a871b4-5684-4630-96bd-146f94debf1c&project=ad98d3cd-26fb-4ee3-99c9-8a2ab085e737

Expected Behavior
The spreadsheet is downloaded. Only the text column has 2 values delimited by a pipe.

Notes:
The flattened needs to handle metadata json which has the ontology property value like:
"diseases": [
    {
        "text": "hypertension",
        "ontology": "MONDO:0005015",
        "ontology_label": "diabetes mellitus (disease)"
    },
    {
        "text": "Diabetes"
    }
],

sample prod metadata: https://api.ingest.dev.archive.data.humancellatlas.org/biomaterials/5f1b0ea3fe9c934c8b835c86
The diseases ontology schema is not requiring only the text and not the other ontology attributes :
https://schema.humancellatlas.org/module/ontology/5.3.8/disease_ontology

I tested the fix in dev and we don't have Service unavailable error anymore, but we still need to do a small fix as the || are still added into the cell's value if there is no value after them.
I attached a screenshoot from the spreadsheet.

Fixes:

https://github.com/ebi-ait/ingest-client/commit/6249cbc52c18ceae91f0649d2fa7297a8ef60970
https://github.com/ebi-ait/ingest-client/commit/c95e0794ae08e3d394229216cc8e0d6946531ba6

Tested it by the above mentioned project in production and I was able to download a spreadsheet without the service unavailable error and the spreadsheet looked fine.


JWSTSIAF-171 on OSS 8.1

This PR replaces #143 and is for JWSTSIAF-171
"Fixed typo in NRCA5_MASKLWB_F277W XSciRef"
It was determined that the next PRD release would be a DEVSOC release based on the previous OSS 8.1 delivery, which is PRDOPSSOC-028. As pysiaf's master and siaf-updates branch are currently up to date with PRDOPSSOC-027 which is for OSS 8.2, the old branch/PR (#143) built the new JWSTSIAF-171 change on top of the OSS 8.2 data rather than the OSS 8.1 data. For this reason, we have closed the old PR and this new PR has been opened to add the same changes on top of the OSS 8.1 data.
A note about this change:

The aperture that had the typo has different XDetRef, YDetRef, XSciSize, YSciSize, and YSciRef values in nircam_siaf_aperture_definition.txt between PRDOPSSOC 027 and 028. However, to keep track of these changes (both now and eventually when they will be merged with the OSS 8.2 data), only the XSciRef value (the one with the typo) has been changed - none of the other values have been altered.

Tagging @mgennaro who was the author of this change and the original PR

As the SIAF representative, I "approve" these changes (I can't actually be set as a reviewer because I created the PR). I've set Mario as the reviewer so he can check that this PR matches the previous, now-closed PR. We we also wait for the NIRCam team to approve these changes in the JIRA ticket before we merge anything

These changes were approved by the NIRCam SI team in a comment on the JIRA issue and can now be merged


merge from arlac77/template-github,arlac77/template-kronos-component

.github/workflows/ci.yml

chore(deps): add 15.0.1 (jobs.build.strategy.matrix.node-version)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #923   +/-   ##
=======================================
  Coverage   96.22%   96.22%
=======================================
  Files           2        2
  Lines          53       53
  Branches        2        2
=======================================
  Hits           51       51
  Misses          2        2

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 8631b98...745be6f. Read the comment docs.


matnwb cannot read output NWB file

A user reported difficulty reading an nwb file produced by the export function.

@bendichter - Should files exported here be readable with matnwb ?

@CBroz1 yes it should be. Would you be able to send me this file? @lawrence-mbf, do you have any idea what might be going on here?

@CBroz1 yes it should be. Would you be able to send me this file?

Of course. I'll send over slack. Thanks for looking into it!

@CBroz1 This unfortunately is a really old issue where MatNWB doesn't support anonymous type definitions: https://github.com/NeurodataWithoutBorders/matnwb/issues/43
Thankfully, I think this is localised in the waveform_mean VectorData object as we haven't seen the same elsewhere. A workaround could just be to make a waveform_mean_unit vectordata object if you really need it in MatNWB.

Thanks @lawrence-mbf for your insights!
@kabilar - I'm thinking we should make that edit so resulting files are more accessible. What do you think?

Thanks all.  Yes, let's make the update.  I couldn't find documentation on waveform_mean_unit so not sure how we would make this update.

@kabilar @CBroz1 It would really just be repeating that unit value as a vector column with the same number of rows. It is an ugly duplication of code and may one day throw warnings when validating but it will guarantee MatNWB support for now.


Maps ad events and properties

The properties and general logic look good to me here. Prateek can give you a more thorough code review for efficiency, etc. :)


[Feature Request] Account Switcher

An account switcher for Genshin Impact like this launcher.

Already discussed on #65

Already discussed on #65

Still disagree..
OP already provide similar open source HYV launcer with switch account support.

No, an account switcher doesn't necessarily mean that the authentication process needs to be done on Collapse's end. If you look at the repo I provided above, there seems to be a file saved locally that contains account information to allow account switching.
So the user actually logs in the account first (authentication process done by Hoyoverse) -> Then the user can choose to "save" the current logged in account (by saving a file or something similar) -> Then switch accounts by switching the file that contains account information.
Please seriously consider adding this because it's such an important feature that Hoyoverse's current launcher doesn't have.

The main reason why most official launchers don't have an account switcher is simple. You mainly use one account and done. It's rare for normal people to have multiple accounts, unless your purpose is to create an indefinite amount of accounts and reroll to get multiple 5* on first pull, which is petty imo, or helping a friend when they can't log in for a short amount of time. Also, mhy handles login in game, not on launcher. If it gets implemented in Collapse, the whole project will be at huge risk of being banned, and it's probably something all of us don't want.

This is my understanding of your points regarding why you think an account switcher feature should not be added:

Small use-case
You mentioned that most people "mainly use on account and done", this point can be largely biased based on one's point of view.  While we can't get the exact statistics of how many players in Genshin Impact play on more than 1 account, I can guarantee that the number is not small. Personally, my use-case is to play on my friends' accounts and sometimes do dailies for other people. Another major use-case that I know is to play on different servers, many players have already reached the end-game phase so they'd like to have a fresh restart. I strongly disagree with this point.

High risk feature
I agree with this, I indeed don't want to see an amazing project like Collapse to get shut down by Hoyoverse because of adding this feature. However, is it true that adding account switcher = Collapse shut down? First, we can look at this repo. It has 765 stars right now and the launcher existed for a very long period of time. While the fact that this launcher is not shut down yet doesn't necessarily mean that an account switcher is Okay, I think it's still reasonable to use it as a reference. (Not to mention that it even has a FPSUnlocker feature, which is clearly prohibited by Hoyoverse).

As author described in the last thread about this, its not a planned feature to be implemented in this launcher. A little read to Genshin's Legal FAQ  about this:

Can I put programs for updating miHoYo games on a website of my choice so others can download them?

No. To protect the security of our users' accounts, miHoYo strictly forbids users from making update programs available on unauthorized third-party platforms. miHoYo reserves the right to terminate any and all agreements with any users who are found to be in violation of this rule at its sole discretion. miHoYo reserves the right to refuse unauthorized third-party platforms from hosting, disseminating, or distributing miHoYo product-related programs for any reason and at any time.

While we do understand that this launcher already broke this point, adding more feature that touches their SDK (registry key for the account change is MIHOYOSDK_PROD_OVERSEA_x) will add more point towards their claim, if they decides to (try) strike this project.
We also have to remember that this project is not just launcher for Genshin, it also used to launch another HYV games such as HI3, which the main motivation for the author making this project. The game sometime suffers with cache issues and other inconveniences regarding the game that this launcher fix.
While I (and project maintainers I often talk to) do see the usefulness of this feature, its quite hard to come for the conclusion of implementing this feature or not, knowing the risk that it might have towards the project. Of course, the author might change his mind and re-think about this feature in the future.

Understood, I respect the project owner's decision.

Hi @seriaati, @tilsgee and all contributors who helped me on this,
First of all, thank you so much for your question in this issue.
I understand that you need this feature to be implemented. Actually, this is really easy for me to implement this.
But on the other side, I have a concern by accessing user's confidential data, means I may possibly compromise their data.
As part as this is also not planned and I believe that this launcher already crossed the line of miHoYo's T&Cs. We don't want to make this beyond the limit as you might already heard that most of the projects that took "as a result of reverse engineering" went gone (since this launcher is a result of that and some feature in this project uses it).
We are avoiding to bring this project into miHoYo's attention and we don't want to give this project the same fate.
I really appreciate your feedback on this launcher and thank you so much for trying this out.
But as I already said before,... we are sorry that we can't bring this feature alive to this launcher based on the reasons above.
Hope this could answer some of your questions.


Null Reference Crashes

When Nora Li from Civil Team and I were setting up Tuneup for Civil Team to consume, we were getting constant crashes which all point to null object references when launching Dynamo with TuneUp. After some debugging, I found there are multiple places which could cause such crash. Here are the fixes worked on my local system and a screen shot from Nora.

LGTM - do the tests pass?

@mjkkirschner Among three tests, the second passes while the other two failed at the ASM loading step during Dynamo initialization because I am using the RC for testing. Here is a screenshot of the extension working with results.


Development/schedule service unit tests

Unit tests for CreateScheduleAsync method were implemented. Here I decided to add one more helper method :
private void Initialize(CreateScheduleDto createScheduleDto, Mock<IThemeRepository> themeRepositoryMock = null,Mock<IMentorRepository> mentorRepositoryMock = null, Mock<IStudentGroupRepository> studentGroupRepositoryMock = null)
This method will be very comfortable to test other methods in service, because we can use only those mocks, that we need. I mean, to test CreateScheduleAsync, I must to use all repositories mocks, but for other methods we don't need this

This issue transferred to new branch


Vector not reponse "Hey Vector"

My vector cannot response "Hey Vector" command and after pressing the back button, I say something but do not get any response.
I checked on log, I saw some information

Thank you for doing great.

You may have bad microphones. Watch this video to get to the hidden menu and see what the values are for the mics
https://www.youtube.com/watch?v=hOt4iw17yn0

You may have bad microphones. Watch this video to get to the hidden menu and see what the values are for the mics https://www.youtube.com/watch?v=hOt4iw17yn0

Thank you for your response, I cannot go to hide the menu, I don't know the reason. But yesterday, I went go hide the menu and saw the microphone work.

You may have bad microphones. Watch this video to get to the hidden menu and see what the values are for the mics https://www.youtube.com/watch?v=hOt4iw17yn0

after teardown my vector and cleaning the cable, it working now. Thank you so much.


Pass provider to jwt callback

Summary of proposed feature
The callback for the jwt payload should have access to provider details.
Purpose of proposed feature
While I currently have access to the default token object and OAuthProfilewith an excerpt of data from the provider,
I have no way to determine which provider was used.
I want the JWT to include the name of the used provider for further usage.
Detail about proposed feature
I guess this could be done easily by adding provider as an additional argument to callbacks.jwt here:
https://github.com/iaincollins/next-auth/blob/3202ca1037e2e72ccc49cfd9e0d92922d5f0eda1/src/server/routes/callback.js#L73
Furthermore, the user, profile and account could also be passed(I assume at least one the three would be redundant, but didn't check the purpose of each). There might be usecases where it is interesting to store the access token or profile id as part of the jwt.

Hi there! This is already in progress and recently dropped in the current beta.
The latest documentation should cover everything. v3 will probably be live very soon.

@iaincollins Great! Although looking at the code, provider seems to not be included, is that part of the user or account object?

Yeah! Sorry I know it's not super obvious without some examples (there are still some gaps in tutorials) but the 'account' object contains the Provider ID (e.g. 'twitter'), Provider Account ID (e.g. '1234567') the provider type (e.g. 'oauth') and the Refresh Token and Access Token.
The profile object contains the actual raw OAuth Profile (name, location, username, etc.) for that provider.
The contents vary for non-OAuth providers (Email, Credentials) but are very similar and have logical substitutes.

@iaincollins Thanks, sounds like this exactly covers my neeeds.


Add name key to sources in Pipfiles

Newer versions of pipenv require any specified sources to be
explicitly name'd.
More context here: https://github.com/pypa/pipenv/discussions/5370#discussioncomment-3701061
This has been true since at least Sept 2022. Our version of pipenv is
from April of 2022, so it doesn't complain. But it will as soon as we upgrade.
Also, for sources that :dependabot: dynamically injects into the
Pipfile, they need a name. These sources are stripped from the final
Pipfile / Pipfile.lock during the FileUpdater#post_process_lockfile
method, so all we need is a placeholder name to placate pipenv.
Long term, we may want to add custom error handling to flag this
missing key as a Dependabot::DependencyFileNotResolvable error.
But I decided that was out of scope for now as this PR does not generate
the error... that will not happen until we upgrade to newer pipenv.
And even at that point, our first priority will be upgrading, and then
from there handling any new errors that start popping up.
And even then, most of the active users of pipenv are unlikely to see this
error because they're likely running a newer version of pipenv.

Likely there will be hash mismatches in the lockfiles, so they'll need regenerating... if so, check one or two to see what's actually changing... as no point in regenerating here using an older version of pipenv only to have to regenerate them again under the newer version of pipenv...

We got a report via a support ticket that :dependabot: opened a PR that included index: dependabot-inserted-index-0  in Pipfile.lock. So this PR introduced some bug...
Unfortunately, I can't repro and the affected user's repo is a private company so I can't access their repo and they are migrating to a different python package manager so decided it wasn't worthwhile for them to try to create a reproducible example.
The one observation they made:

the bad index seems to be added while updating the sub-dependencies of our private libs.

If anyone else encounters this, please feel free to file a new issue, preferably here in dependabot-core and linking to reproducible example and I'd be happy to take a look.

We are having the same issue but unfortunately also in a private repository :disappointed:
It looks like older versions of pipenv are happy with that new index, but certainly the newest version pipenv-2023.8.28 is not:
Successfully uninstalled pipenv-2023.2.4

..............................................................................................................................

Installing dependencies from Pipfile.lock (923e0b)...
--
125 | Unable to find dependabot-inserted-index-0 in sources, please check
126 | dependencies:

We also have private libraries, so not sure I'll be able to create a reproducible example.

Can you at least create a new issue describing what you're seeing? The pipenv maintainers are great folks and we have good lines of communication open with them, so maybe between all of us we can get it figured out...


Drop fax #s?

In #219 I suggested we drop fax numbers, but consensus was that we should keep them and Sunlight suggested they were going to keep them up to date.
Per #405, they may be out of date. I don't know if anyone is checking them. Sunlight obviously isn't anymore. Our scripts don't update the values. The only fax numbers we're now offering is for senators who were serving last Congress - we didn't pull forward fax numbers for re-elected members.
If no one chimes in to say they are taking responsibility for the fax numbers, I'd like to drop them.

With respect, absolutely not. 👎🏻👎🏻👎🏻
For effectiveness, the fax data is more important than the email forms.
The advice I read about contacting Congress is:

Phoning a district office is most effective, https://storify.com/editoremilye/i-worked-for-congress-for-six-year
Writing a district office is most effective, https://www.ted.com/talks/omar_ahmad_political_change_with_pen_and_paper
Writing or phoning the personal contact of the LA is most effective, like the lobbyists all do
Faxing a personalized letter is as good as a mail letter
Email the least effective, https://storify.com/editoremilye/i-worked-for-congress-for-six-year
Social media not effective at all

Personally, I choose to go with fax because it's as fast as an email and more effective, fax modems cost under $15 https://www.amazon.com/Internet-Modem-V-90-V-92-Sewell/, and my calling plan already covers all US long distance calls.
If I had more time I'd call the district offices. But I don't, so I fax rather than doing nothing.
It would be great if the EFF's https://democracy.io EFForg/democracy.io or Sunlight's contact-congress offered the range of phone/fax/email contact options.

I haven't checked all the fax numbers in the existing data, but the ones I have (numbers in the hundreds) have all been correct, so this looks like a useful, accurate dataset to me.

And per #405, these are all additions of fax data to the district offices, not corrections, so no, they are not out of date.

I strongly doubt that a faxed letter is better than a message sent through a webform (which is what I assume you mean by an email).
But that aside - If you're volunteering to do the work, then keeping the fax numbers is good with me. Will you submit a PR that ensures every legislator has an up to date fax number listed?

(As I see I mis-read #405. There are additions (to legislators-current.yaml), not changes.)

Yes I'll submit a PR for fax numbers.
Also, my argument rests on effectiveness, which I've based on what I read staffers say. If you known of any staffer assertions of the effectiveness of webforms/email versus letters, I'd appreciate reading it -- I'll adopt the means that makes the best use of my time.
Finally, would anyone be able to help how to submit a commit-specific PR to this repo with just the lookup script and fax number additions and corrections, if any? My repo is a clone-of-a-clone, which I did so that I can use @TheWalkers district data.
I see from stackoverflow it might involve voodoo like git reset HEAD^, but I don't really know.

Staffers tend to think that what happens in their office is what happens in every office, and you've probably heard from staffers in offices that happen to like faxes. I am confident we could find a few offices that throw faxes in the trash (metaphorically, since of course the fax is never printed). But I doubt we would find even one office that has a webform that doesn't store every message that goes into it. Whether any of this is effective or not has a lot more to do with other factors in any case (how many messages are sent, are there parallel advocacy efforts happening, opportunity costs for the constituent, ...).

Staffers tend to think that what happens in their office is what happens in every office, and you've probably heard from staffers in offices that happen to like faxes.

This would support the case to keep the fax data, along with the three basic options that legislators provide to contact them: phone numbers, fax numbers, and web forms.

Whether any of this is effective or not has a lot more to do with other factors in any case

Agreed. I would add the personalized nature of the contact to these factors. Faxing a personally written letter provides this option.

I just cross referenced independent congressional fax data with the fax data in committees-current.yaml.
Out of 538  checks, I only found 2 inaccuracies, and made two extra additions, one for the district data.
I'd say this is confirmation that the fax data is accurate and useful.
When it's useful to do this again, here's a quickie script:
import csv,fnmatch,os,re,yaml

with open('congress_fax_data.csv','r') as c:
    cdata = [[w.replace('\xa0','') for w in line] for line in csv.reader(c)]

with open(os.path.expanduser('~/Source/github/essandess/congress-legislators/legislators-current.yaml'),'r') as y:
   ydata = yaml.load(y,Loader=yaml.CLoader)

check = dict()
for line in cdata:
    try:
        cfax = line[-1]
        names = line[0].split(' ')[1:]
        for leg in (leg for leg in ydata if fnmatch.fnmatch(leg['name']['last'],names[-1]) and fnmatch.fnmatch(leg['name']['first'],names[0])):
            yfax = [term['fax'] for term in leg['terms'] if 'fax' in term]
            if cfax not in yfax: check[line[0]] = line[1]
    except:
        pass
for name in check: print('{}: Check fax {}.'.format(name,check[name]))

Relevant: Apps Make Pestering Congress So Easy That Politicians Can't Keep Up. This and related repo's and their authors are cited.

The consensus among activists and staffers alike is phone calls are better than email, and that showing up in person to district offices or town halls is better than phone calls. But email and social media make a difference too, as long as those communications are personalized.

This speaks to the importance of including both the fax data and the district data, which both facilitate personalized correspondence.

With respect, absolutely not. 👎🏻👎🏻👎🏻

+1000 to the idea of not dropping support for faxes.

I strongly doubt that a faxed letter is better than a message sent through a webform (which is what I assume you mean by an email).

Really?  Everyone I've ever talked to in any kind of office that has a fax machine checks their fax machine before their spam folder.

Really?

Really!
I'm unsubscribing from this thread. The only thing I care about is whether someone steps up to take responsibility for the data field and gets a PR merged with complete and accurate data for the 115th Congress.

@JoshData, Per my comment above, I have independently confirmed that all fax data for the 115th congress is accurate.
However, I do see from looking at the current term data using code in #428,
python congress_lookup.py -c Ways -T fax

that none (or little) of this data has been carried over to the current term.
Is this something that is done by hand? Or done by automated scraping? I would have expected the latter. Why doesn't this data appear automatically, and how should it be entered?

By the way, I have direct, personal, recent experience using this script with a fax modem that faxes to Congress are read and taken seriously.
Automated apps are important, but we now have this issue: Apps Make Pestering Congress So Easy That Politicians Can’t Keep Up.
Telephone, letter, and fax will always be of primary importance, and automated apps should include this information to provide the option.

@essandess Are you saying that you validated all the fax numbers from the 114th congress and you think that these can be automatically carried forward to the 115th congress terms for those members who currently have no fax number listed?  I'd be happy to do that.
Also, can you show how you are validating these so I can spot check them?

@essandess As @JoshData said, he unsubscribed from the thread. To both you and @apurvis, I just want to remind you that this dataset is maintained for free by people who have put hundreds or thousands of hours of human review over the last 4-5 years, and who have many more years of extensive experience working with Congress and its data. In @JoshData's case, at least 13 years. Feel free to bring up your own expertise and debate the value of fields, but I encourage you to take extra care in the words you choose and to treat the dataset maintainers' opinions as rooted in as much or more experience and intelligence as yours are.

Is this something that is done by hand? Or done by automated scraping? I would have expected the latter. Why doesn't this data appear automatically, and how should it be entered?

There is no central server where we run the scripts in the repo regularly, commits are always done by pull requests, and pull requests are always filed by hand. Automating the pipeline is less feasible than it may appear, as a) automating git behavior and pull requests in a safe way is non-trivial, and b) many fields get a lot of value out of human review before even filing the PR. Such is the reality of creating a high quality dataset of data gathered from dozens of different sources.
Thank you for your work at validating and improving our fax numbers, and demonstrating user interest in maintaining the field. I'm reviewing #428 and leaving comments, and I concur with @joelcollinsdc that we'd benefit from understanding your methodology for validating the fax data.

@joelcollinsdc @konklone
I've validated the fax data using two different methods:

Googled for online congressional fax lists generated independently of this project, parsed them, used the quickie script in my comment above to highlight potential errors, then googled by hand to check and correct any discrepancies.
I use the fax data to contact congress myself. I've haven't faxed the entire congress, but I've sent hundreds. I've only seen one or two errors that I corrected.

Furthermore, Congressional fax numbers are very rarely changed for all the obvious reasons: they're well advertised in print, local political organizations, web pages, civic groups, lobbying groups, and on and on.
I would presume no change in phone or fax contact information for a new legislative term unless there is explicit evidence for it.

Maybe having some real input from staffers in congressional offices would be helpful here, but my thoughts on the whole fax number situation is that they are likely to have more negative effects than potential positive effects.  I will guess that faxing is a pretty essential part of the "get help with a federal agency" process which is why member offices advertise them (especially the fax numbers associated with district offices, where casework happens).  I'm guessing here, but the volume of casework faxes must be much lower than the volume of constituent comments, but each request is very important and if these requests get lost in a high volume of abuse of the fax system, these constituents will suffer.
Also, I think just in general people facilitating communication to member offices should be trying to help member offices deal with the volume of traffic meaningfully.  Member offices are able to get the most metadata out of traffic that comes in through the CWC API (i'm guessing?), followed by webforms, followed by anything that get OCR'd, like snail mail and faxes.  So we should be good stewards of our role in enabling this process and try our best to advertise the best communication routes first.
Anyhow, I made a PR to pull the fax numbers forward.  I did not spot check more than a few.

Congressional contact information is necessarily public. The best anyone can do is to encourage everyone to use this information responsibly. Any abuse is the responsibility of the individual contacting the legislator, and staffers develop strong filters for listening to important constituent versus spam messages.
Real input from staffers is widely available. Staffers comment frequently in news reports and social media on effective ways to contact legislators. Here's just a few such comments from the past week:

A Low-Tech Guide to Becoming More Politically Active - The New York Times

Phone calls carry more weight than an email or a tweet, partly because a phone ringing incessantly is harder to ignore than a flooded inbox.

Call Congress, It's the Best Way to Be Heard. But DC's Phone Tech Is Broken | WIRED

it’s hard for staffers to figure out how much you actually care. “If it takes them five seconds to fill out a form that shoots out a form email,” Ellsworth says. “is that the same as someone who actually takes the time to call?” Oh, and that online petition you signed? You can bet nobody on Capitol Hill saw it, or cares.

calls “broke my BlackBerry. And it’s close to breaking my desktop.”

(github un-unsubscribes you when you are @-mentioned, so I am back)
@joelcollinsdc wrote:

I think just in general people facilitating communication to member offices should be trying to help member offices deal with the volume of traffic meaningfully.

I completely agree.
We have a responsibility when we publish information to be cognizant of its intended and unintended consequences and to not publish information that, on balance, does harm. (I'm not saying publishing fax numbers does harm, but we have a responsibility to consider how that might be the case.)
(Btw the NYTimes article that @essandess pasted above relies heavily on the reporter's conversation with ME.)

@JoshData Awesome accomplishments—thank you for all your hard work.

Closing as the fax numbers have been updated, thanks @essandess


Popper element is not aligned to trigger, off by 1px

CodeSandbox demo
https://stackblitz.com/edit/react-nyp2yj?file=src%2FApp.js
Clonable repo: https://github.com/OliverJAsh/popper-subpixel-issue
Steps to reproduce the problem
When we view this test case at odd viewport sizes e.g. 327px wide in this example, the popper element is not aligned to the trigger. It is off by ~1 px.

Zoomed in slightly:

Does https://github.com/popperjs/popper-core/pull/1400 fix it?

I just tested. It helps but it doesn't completely solve the issue.
Without #1400:

https://user-images.githubusercontent.com/921609/141235630-0ab1a544-0d5a-4a12-9c5e-6d641adc8137.mov
With #1400:

https://user-images.githubusercontent.com/921609/141235559-05a3eadc-8d51-4c93-ade3-63175fe7df08.mov

I wonder if using floor() for *-start placements, and ceil() for *-end* placements works in that function?

The following works for me:
function roundOffsetsByDPR({x, y}) {
  const win: Window = window;
  const dpr = win.devicePixelRatio || 1;

  // Don't round by DPR for variation placements
  if (variation) {
    return {
      x: round(x),
      y: round(y),
    };
  }

  return {
    x: round(x * dpr) / dpr,
    y: round(y * dpr) / dpr,
  };
}


Non-average Stress @ Top & Bottom Planes of Shell Elements

Hello,
I am currently promoting Pyansys in our company for fast screening of large structures to perform fatigue analysis. However I have two issues to resolve:
(1) I have not been able to extract the element stress (non-average) at top and bottom planes of shell elements. What "result.ElementStress(0)" provides is only the element stress at Mid-plane of shell elements through an array with 4 rows, which cannot be used for fatigue analysis. Is there any way to get the element stress through an array with 8 rows including 4 nodes at the top layer and 4 nodes at the bottom layer ?
(Note: Mid-plane results is indeed the average of top and bottom planes of shell elements. For a rectangular shell elements we have 4 top nodes and 4 bottom nodes, so the stress array should have 8 rows to capture the bending effect. This is how we traditionally extracted the results from ANSYS binary rst file.)
(2) We need to extract the component stresses for shell elements in the Element Coordinate System instead of the Global Coordinate System. Any suggestion?
Thanks in advance.
-Shobeir

Can you email me at<EMAIL_ADDRESS>

I'll work on this as soon as I get the rst file.  Please include a text output of the results that you'd like me to achieve like in #17.

Thank you. I will send you the ANSYS rst file and the expected text output shortly. I am working on it.

Alex,
I emailed you the ANSYS rst file (v18.2) and the expected outputs.
Please let me know if you need any other info from me.
Thanks,
-Shobeir

Results for SHELL181 elements now match ANSYS output in both the element coordinate system and the global coordinate system.  Updated in 0.32.4.  See the documentation for ElementStress.

Alex,
Thanks for the update. This is really a good improvement. I removed the previous version and downloaded the updated 0.32.4. version and successfully reinstalled it through Anaconda distributor by using "pip install pyansys" command.  However, I am still getting the previous results (mid-plane stress) and not the top and bottom surface stresses. Here is what I have
import numpy as np
import pyansys
result=pyansys.ResultReader('Test.rst')
estress, elem, enode= result.ElementStress(0,False)
When I try to change the coordinate system by typing
estress, elem, enode= result.ElementStress(0,False,True)
I get the following error:
TypeError: ElementStress() takes from 2 to 3 positional arguments but 4 were given
It seems that my pyansys has not been updated. Am I missing anything here?
Thanks,
-Shobeir

Alex,
Never mind about my previous comment. I resolved the installation issue.
Every thing works perfect now. Thank you so much.
Just a couple of suggestions for future considerations:
(1) Might be good to include an option to extract the stresses for certain element numbers and not for all the available elements.
(2) I guess the updated version works fine for all Shell elements and not only for Shell 181, is this correct? I can test it and let you know.
Thanks again.
Best,
-Shobeir

You can create a mask or loop through specific elements if you’d like only the stresses of those elements. I suppose there would be a speed up if you only wanted the stress of certain elements.  If it’s critical, let me know.
I only programmed it for SHELL181.  If you send me a rst file with a different shell type I can add it to the source.  I’d sure it won’t be much code on my side, but I like to have tests to support code and I don’t usually work with shells.


langserver: Extension to limit returned references

Seem like a reasonable extension?

Pagination support in general would make sense and I am pretty sure Microsoft will try to add something that works for all methods.
If you have limit you will likely want offset too.
Btw this reminds me that we forgot to prefix the workspace symbols parameter extension 😕

@felixfbecker there's still time to update that for workspace/symbols

@felixfbecker agreed. But in the interests of keeping it simple, I'd rather just have a limit with a large enough value that no user ever looks at that many. offsets makes life more difficult, so i'd prefer only adding that when we need it.
Also when microsoft adds pagination support to the protocol, we can just use that then. In the meantime we have very large responses being sent down the wire that the UI sometimes struggles to render on sourcegraph.com.
It may make more sense to do the Limit at another layer, but in general I don't like writing code in the proxy layer/etc which does special casing on the method.

Sounds good.

@keegancsmith please update the vendored code in the main repo

I'm passing context: { includeDeclaration: false, xlimit: 100 }, for provideReferences


Add second level: get moped

Upgrade bike to moped

@aryszka @vigneshshanmugam @w0rm @kolja Do you guys think this feature is +1? If so, let'sa dd a "Help Wanted" label.

I doubt this is feasible, because @kolja will have to draw a lot of images.


Missing cover art prevents movie add

Adding a movie with no cover art in IMDB causes the bot to error out. It returns with the wrong channel used for messages error.
Included is the relevant log section.
log snipett.txt
Example URL with missing artwork
https://www.imdb.com/title/tt29529729/

PR #85 should address the issue.


UnJS: Add abstraction to object / file storage (e.g. local, S3, R2, Vercel Blob)

Describe the feature
This is a feature request for the UnJS ecosystem but I am unsure where else to put it:
It would be amazing to have a library that abstracts / unifies the usage of object/file storages (like S3).
Possible drivers could be:

File system
AWS S3
Cloudflare R2
Vercel Blob

Benifits:

Easy to use like unjs/unstorage :)
Easy to transfer from one provider to another
Use different driver for local development than on production (e.g. file system on local, Vercel Blob on production)

Additional information

[ ] Would you be willing to help implement this feature?

I think unstorage is this very thing. It supports writing 'raw' content to FS/blob storage, etc.

Arent there then all the drivers for file storages missing? Like AWS S3, Cloudflare R2, Vercel Blob, etc
I always thought there is a huge difference between a KV store like Redis and a file store like S3

@MickL Unstorage combines all storage with a unified/simplified interface via drivers. There is an open tracker for S3, and R2 is supported too. Always feel free to open improvement suggestions to add more drivers to unjs/unstorage if feel something is missing 👍🏼


updated link and installation file

#23  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mAmineChniti and the rest of your teammates on  Graphite


Disappearing draggables in useDraggable example docs tab

When dragging some of the examples, the draggable item seems to disappear when dragged. It looks like it's going to the very bottom of its parent?
https://5fc05e08a4a65d0021ae0bf2-zditmhymjz.chromatic.com/?path=/docs/core-draggable-hooks-usedraggable--basic-setup

It's only in the "docs" tab, not the "canvas" tab
The first example works fine ("Basic Setup"), only subsequent examples seem to break (after "Drag Handle")
It may only be on the useDraggable examples, not the others

Loom video: https://www.loom.com/share/e23adf550caf4b84b6a5be4e217bfda7
Mac OS, Chrome
Just adding as I see 'em, let me know if this is a known issue or a dupe :)

Very odd, will investigate.


Autoconfig feature

Description

php-whois configuration for TLDs is centrally distributed, changes in whois servers' config have to be reported as issues, then committed to the config file, then redistributed via git/composer.
However, to find out a whois server name for a particular TLD, one can use the following shell script:
#!/bin/bash
tld=$1
# error checking omitted for demonstration purposes
dig +short CNAME $tld.whois-servers.net

Example

Implement a TldServer::getWhoisServer($tld) method that returns a server name using this command (use php system call for that)
Implement a TldServer::getTldList method to get the TLD list from the official site http://data.iana.org/TLD/tlds-alpha-by-domain.txt
Implement a updateConfig($tld) method (not sure which class it belongs to), that allows to query the server for an individual tld, or the whole list.
Make config file non-distributable, keep only sample file and possible parser-to-tld configuration

Wow! Very good feature.
But I can implement it later (after ~2 weeks)

Currently you can use workaround with:
$whois = Whois::create();
$servers = $whois->getServers() (\Iodev\Whois\Modules\Tld\TldModule ::getServers())
..find and remove bad TLDs from list..
..add to $servers properly configured (\Iodev\Whois\Modules\Tld\TldServer::from...)..
..set new server list for your Whois instanse..
$whois->setServers() (\Iodev\Whois\Modules\Tld\TldModule ::setServers())

Thanks for the response!
It is not something really urgent, and I doubt one can use the workaround on every whois query, as it is quite time-consuming. The suggestion was inspired by the idea that sometimes php-whois picks up a wrong server for a particular TLD, and central management for these issues

Your suggestion is OK.

and I doubt one can use the workaround on every whois query,
What you mean? Do you create every time for query new instance of Whois like this Whois::create()->querySomething() instead of single properly configured instance outside of code with queries?

I am using a single property, as in my quoted example:
$this->whois->doSomething();
What I mean is that config cannot be validated on every run, as it is too resource and time consuming. I suggest to have some sort of a config-cache, which can be updated or completely rebuilt anytime, leaving the TLD to server matching up to the end user, not maintaining it centrally any more.

I see. OK

php -r 'var_dump(dns_get_record("be.whois-servers.net"));'

dns_get_record($host,$type=DNS_ANY)
RFC8482 deprecates the DNS ANY query type.
I think it is better to use DNS_CNAME
var_dump(dns_get_record("be.whois-servers.net",DNS_CNAME));
array(1) {
  [0]=>
  array(5) {
    ["host"]=>
    string(20) "be.whois-servers.net"
    ["class"]=>
    string(2) "IN"
    ["ttl"]=>
    int(585)
    ["type"]=>
    string(5) "CNAME"
    ["target"]=>
    string(12) "whois.dns.be"
  }
}

Thanks!
Actually I used this case in dev branch (https://github.com/io-developer/php-whois/commits/tld_serverlist_updating)
But I was stuck with performance issue and some thoughts:

Some lazy background process needed because of very very long updating for each TLD (some not responding, not supporting, etc.)
Or needed on-fly updating when request to TLD happens at first time or after some intervals..

amphp and reactphp have a lot of functions for async and child process.
amphp/parallel
amphp/artax
reactphp

very excited to see this feature added to the master branch
since updating the list is time-consuming, would it make sense to keep a globally updated list somewhere that updates once a day so the rest of us can pull in through a simple get json request?

Actually loading tld list can be done in current version with minor overhead:
You could load list in any format and prepare data for $whois->setServers()
I don't like idea to host and update this custom server list in github, because it's needed to write some kind of bot and cron. Looks like reinventing the wheel because I've seen many lists like that in other repositories (and some of these are dead or not updating anymore) and it's not guarantee this list are actual and valid.


Can't add any slots to combat tracker

So I am not sure what I am doing wrong but I got the update and I can't add anyone to the combat tracker. I tried adding the npc to the tracker in the traditional way and nothing happened. I even made an new campaign just to see if any 3 party modules conflicting with the update. The only thing I didn't try was having a live player connected.
Is there a tutorial on how to run this combat tracker?

Known and fixed. We'll get a hotfix out soon. Thanks for the report, though.

https://github.com/StarWarsFoundryVTT/StarWarsFFG/pull/1335/files change, if you want to apply it yourself


#150433412 feedback-implementation

What does this PR do?

hide user secrete

Description of Task to be completed?
Not Applicable
How should this be manually tested?
Not Applicable
Any background context you want to provide?
Not Applicable
What are the relevant pivotal tracker stories?
Not Applicable
Screenshots (if appropriate)
Not Applicable
Questions
Not Applicable

Changes Unknown when pulling 385254c9b4e004e080e5d07132e3d0dbaee8c488 on chore/150433412/feedback-implementation into ** on staging**.


Use Card for nicer credit card UI

h/t
https://jessepollak.github.io/card/
https://github.com/jessepollak/card

I considered doing this in Liberapay but I decided to go with something simpler instead. The result is https://github.com/liberapay/liberapay.com/pull/679 and the creation of https://github.com/liberapay/payment-cards.js


enhancement Split period from auto collector

Proposed changes

Split period analyze from auto collector
Analyze table incrementally by default
Rename StatisticsAutoAnalyzer to StatisticsAutoCollector

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.39 seconds
stream load tsv:          546 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.7 seconds inserted 10000000 Rows, about 348K ops/s
storage size:<PHONE_NUMBER>6 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 49.86 seconds
stream load tsv:          534 seconds loaded<PHONE_NUMBER>9 Bytes, about 133 MB/s
stream load json:         21 seconds loaded<PHONE_NUMBER> Bytes, about 107 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>8 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 46.99 seconds
stream load tsv:          546 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.0 seconds inserted 10000000 Rows, about 344K ops/s
storage size:<PHONE_NUMBER>8 Bytes


Thank you

I couldn't find any other contact avenue here, but I just wanted to drop y'all a note to say very sincerely: thank you for building this.
I found your tool in a StackOverflow answer. I have a small TypeScript library that usually gets compiled and added to a different project. It has plenty of Jest tests that execute its code. But sometimes I just want to play around with a spike branch, and running ts-node and ts-node-esm was such a nightmare. I just wanted a simple way to run a .ts file locally to see what happens. This is exactly what I needed.
Go ahead and close out this issue, I just wanted to express my gratitude. You saved me hours of maintaining finicky config.

Best issue so far, I'm glad I helped you! 😉


Widget - Tapping a non main account email redirects to the main account inbox

Checklist

[X] I have used the search function to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

App version
6.803
Where did you get the app from?
Google Play
Android version
14
Device model
No response
Steps to reproduce

Status: the unified inbox is shown in the widget
Status: but the display of the unified inbox is not selected in the settings
Status: my main account inbox is empty (all my emails are in folders)
Tap from the widget an email which doesn't belong to the main account
The app is opening and shows the main account inbox instead of the email

Expected behavior
Tapping the email from the widget should open it in the app even if the display of the unified inbox is not selected.
Actual behavior
The app show the main account inbox (which is empty in my case).
Logs
No response

I reproduced the issue, widget needs a fix.


[Bug]: UC 28 Population of Region not Showing Region Name in Report Output

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Name
Anton
Description of the bug?
Name of Region should be displayed in the Report Output.
Version
Develop Branch
Steps to Reproduce the Error
Run Total Population of a Region Report

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Issue has been resolved. Prompt to enter region comes up, then output is shown based on the input.

Confirmation of print output. Region is displayed.


Linker Error after Updating to v2023_03_Release

I updated today to v2023_03_Release I tried to build my codes it is throwing a error in build.
Then I shifted to new workspace expecting something wrong with my workspace, But still error remains same.
Error
15:18:52 **** Build of configuration Default for project Hellow ****
make -r -j 8 --output-sync=target --no-print-directory
Loaded project.mk
/usr/bin/make -f C:/MaximSDK/Libraries/PeriphDrivers/libPeriphDriver.mk  lib BUILD_DIR=C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/softfp PROJ_CFLAGS="-Wall  -DMXC_ASSERT_ENABLE -DCAMERA_OV7692" PROJ_LDFLAGS="" MXC_OPTIMIZE_CFLAGS=-O2 MFLOAT_ABI=softfp DUAL_CORE= RISCV_CORE=
  CC    main.c
make[1]: *** No rule to make target 'c:\maximsdk\libraries\periphdrivers\source\aes\aes_key_revb_regs.h', needed by 'C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/softfp/aes_me17.o'.  Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [C:/MaximSDK/Libraries/PeriphDrivers/periphdriver.mk:96: C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/softfp/libPeriphDriver_softfp.a] Error 2
make: *** Waiting for unfinished jobs....
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/PushButton/pb.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/LED/led.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/Source/board.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/stdio.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/Camera/camera.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/Display/tft_ili9341.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78000/FTHR_RevA/../../../MiscDrivers/Camera/ov7692.c
"make -r -j 8 --output-sync=target --no-print-directory" terminated with exit code 2. Build might be incomplete.

I don't know why it is looking for this file which I don't have in my system
-- > c:\maximsdk\libraries\periphdrivers\source\aes\aes_key_revb_regs.h

After that
I changed the floating point settings
# Set hardware floating point acceleration.
# Options are:
# - hard
# - soft
# - softfp (default if MFLOAT_ABI is not set)
MFLOAT_ABI ?= hard

Now the errors increased
15:39:55 **** Incremental Build of configuration Default for project Hellow ****
make -r -j 8 --output-sync=target --no-print-directory
Loaded project.mk
/usr/bin/make -f C:/MaximSDK/Libraries/PeriphDrivers/libPeriphDriver.mk  lib BUILD_DIR=C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/hardfp PROJ_CFLAGS="-Wall  -DMXC_ASSERT_ENABLE -DCAMERA_OV7692" PROJ_LDFLAGS="" MXC_OPTIMIZE_CFLAGS=-O2 MFLOAT_ABI=hard DUAL_CORE= RISCV_CORE=
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/AES/aes_me17.c
make[1]: *** Waiting for unfinished jobs....
In file included from C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/AES/aes_me17.c:43:
c:\maximsdk\libraries\periphdrivers\source\aes\aes_revb.h:40:10: fatal error: aeskeys_revb_regs.h: No such file or directory
   40 | #include "aeskeys_revb_regs.h"
      |          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [C:/MaximSDK/Libraries/CMSIS/Device/Maxim/MAX78000/Source/GCC/gcc.mk:282: C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/hardfp/aes_me17.o] Error 1
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/AES/aes_revb.c
In file included from C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/AES/aes_revb.c:45:
c:\maximsdk\libraries\periphdrivers\source\aes\aes_revb.h:40:10: fatal error: aeskeys_revb_regs.h: No such file or directory
   40 | #include "aeskeys_revb_regs.h"
      |          ^~~~~~~~~~~~~~~~~~~~~
compilation terminated.
make[1]: *** [C:/MaximSDK/Libraries/CMSIS/Device/Maxim/MAX78000/Source/GCC/gcc.mk:282: C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/hardfp/aes_revb.o] Error 1
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/GPIO/gpio_common.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/GPIO/gpio_reva.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/GPIO/gpio_ai85.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/I2C/i2c_me17.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/I2S/i2s_ai85.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/I2C/i2c_reva.c
make: *** [C:/MaximSDK/Libraries/PeriphDrivers/periphdriver.mk:96: C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78000/hardfp/libPeriphDriver_hardfp.a] Error 2
"make -r -j 8 --output-sync=target --no-print-directory" terminated with exit code 2. Build might be incomplete.

15:39:58 Build Failed. 6 errors, 0 warnings. (took 2s.454ms)

Everything is working absolutely fine till yesterday.

Uninstalling everything and reinstalling solved the issue.

Hi @chaitanya-smartrotamac, this is most likely caused by an outdated peripheral driver library file.  If you had a project with an older library file but then attempted to build and link a newer project on top of it, you could get the object file mismatches shown above.
For future reference, you can run the make clean.periph command (or build task) and it will completely clean old peripheral driver library builds.  On the next build, it will be recreated from scratch.
Were you working from a copy of the Github repo, or a copy of the release installation?  The release is supposed to redistribute new peripheral driver library files to address this issue, and should have wiped the Libraries/PeriphDrivers/bin/ folders for each micro on the update.  Maybe there is a bug in our installer packaging.  I'll take a look

Replicated the issue.  Attempting to link a Hello_World project against the updated lib files.
A suspected bug in the installer packaging is under investigation.
For those who encounter this issue:  run make clean.periph or the build task to rebuild the Peripheral Library from scratch on the next build.
make -r -j 8 --output-sync=target --no-print-directory TARGET=MAX78002 BOARD=EvKit_V1 MAXIM_PATH=C:/MaximSDK MAKE=make PROJECT=Hello_World

Loaded project.mk
make -f C:/MaximSDK/Libraries/PeriphDrivers/libPeriphDriver.mk
 lib BUILD_DIR=C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78002/softfp PROJ_CFLAGS="-Wall  -DMXC_ASSERT_ENABLE -fdiagnostics-color=always -DTFT_NEWHAVEN -DFLIP_SCREEN -DSWAP_XY -DCAMERA_OV7692" PROJ_LDFLAGS="" MXC_OPTIMIZE_CFLAGS=-O2 MFLOAT_ABI=softfp
DUAL_CORE= RISCV_CORE=
  CC    main.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/LED/led.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/Source/board.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/PushButton/pb.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Touchscreen/tsc2046.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Camera/mipi_camera.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Camera/camera.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Display/tft_st7789v.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Camera/ov7692.c
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/Camera/sccb.c
  AS    C:/MaximSDK/Libraries/CMSIS/Device/Maxim/MAX78002/Source/GCC/startup_max78002.S
  CC    C:/MaximSDK/Libraries/CMSIS/Device/Maxim/MAX78002/Source/heap.c
  CC    C:/MaximSDK/Libraries/CMSIS/Device/Maxim/MAX78002/Source/system_max78002.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/mxc_assert.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/mxc_lock.c
make[1]: *** No rule to make target 'c:\maximsdk\libraries\periphdrivers\source\aes\aes_key_revb_regs.h', needed by 'C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78002/softfp/aes_ai87.o'.  Stop.
make[1]: *** Waiting for unfinished jobs....
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/mxc_delay.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/pins_ai87.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/nvic_table.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/ADC/adc_ai87.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/SYS/sys_ai87.c
  CC    C:/MaximSDK/Libraries/CMSIS/../PeriphDrivers/Source/ADC/adc_revb.c
make: *** [C:/MaximSDK/Libraries/PeriphDrivers/periphdriver.mk:96: C:/MaximSDK/Libraries/PeriphDrivers/bin/MAX78002/softfp/libPeriphDriver_softfp.a] Error 2
make: *** Waiting for unfinished jobs....
  CC    C:/MaximSDK/Libraries/Boards/MAX78002/EvKit_V1/../../../MiscDrivers/stdio.c

Note for maintainers:  As of https://github.com/Analog-Devices-MSDK/msdk/pull/457 this should not be an issue.  However, our Jenkins builders package the static libs with a separate method.

As of the June 2023 release this issue should be resolved.
Our installer now removes the problematic old dependency files left over from previous builds when it updates the PeriphDrivers component.  (https://github.com/Analog-Devices-MSDK/Packages/commit/a1bafdeec5ff584aeb15c916fe3bbe1388545758, https://github.com/Analog-Devices-MSDK/Packages/commit/c3b30f9f84922279331a25522f4ca8ab0814bf01)


[Chip] Control clickable prop irrespective of onClick Prop

The Clickable property becomes true if an onClick prop is present in the chip currently. It would be great to make clickable props determine the click-ability of the chip irrespective of onClick props

[ ] This is not a v0.x issue.
[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior
Click-ability of the chip should be determined only by clickable property
Current Behavior
It is also determined/overridden by onClick property. Though clickable is false, if onClick is present, Chip is always clickable
Steps to Reproduce
Though clickable is false, the chip is clickable because of presence of onClick prop
Link: https://codesandbox.io/s/38qj8xp0o1

Tech
Version

Material-UI
latest

React
latest

Browser
Chrome

@oliviertassinari  @mbrookes  I am picking up this issue. will raise a PR soon


Feat/connect new plan a lesson cms - LESQ-814

Description
Music year: {owa_music_year}

[x]  Add zod and types for new schema in OWA
[x]  Add GQL for new page
[x]  Pull data from sanity into page
[x] updates test and updates fixtures

How to test

Go to https://deploy-preview-2408--oak-web-application.netlify.thenational.academy/lesson-planning-new
Open console - you should see test page data coming through from sanity

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

:tada: This PR is included in version 1.428.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add perspective camera bindings (and test)

On top of #100

Codecov Report

Merging #106 (c976ab5) into master (7f03547) will increase coverage by 0.71%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #106      +/-   ##
==========================================
+ Coverage   73.33%   74.05%   +0.71%
==========================================
  Files          11       11
  Lines        1035     1056      +21
==========================================
+ Hits          759      782      +23
+ Misses        276      274       -2

Impacted Files
Coverage Δ

src/meshcat/commands.py
92.45% <100.00%> (ø)

src/meshcat/geometry.py
92.50% <100.00%> (+0.39%)
:arrow_up:

src/meshcat/tests/test_drawing.py
97.80% <100.00%> (+0.08%)
:arrow_up:

src/meshcat/animation.py
77.89% <0.00%> (+2.10%)
: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 7f03547...c976ab5. Read the comment docs.


React UI: Change the dynamic react key with static id

In this PR the dynamic react keys for the panels on the graph page are changed with static ID's. During refactoring I found another bug in parseParams which causes removing on the last panel. This is fixed as well. All other changes are just cosmetic changes and nothing functional is touched there. @juliusv

Cool, seems to work great, thanks! Just a few nit comments.

Thanks for the review @juliusv ! I think is ready to be merged.

:+1: Thanks!


CI: Have a check that published package will have entrypoint files in correct place

The problem of the 4.1.0 release was changed in the published dist structure. In order to prevent that, we could have simple check that builds the package and checks that the pronounced entry points are in place.

Related to https://github.com/ethersphere/bee-js/pull/696


Add users headers

When reading this doc I found it hard to figure out exactly what script I needed to run. I think these headers will help.

Docs Build status updates of commit 591f8e2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-5.2.0/install-az-ps.md
:white_check_mark:Succeeded
View (azps-5.2.0)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Reading hyperslabs fails

Example:
Prerequisites: Download hdf5 file from http://pklab.med.harvard.edu/velocyto/hgForebrainGlut/hgForebrainGlut.loom
var hdf5 = require('hdf5').hdf5;
var h5lt = require('hdf5').h5lt;
var Access = require('hdf5/lib/globals').Access;
var file = new hdf5.File('hgForebrainGlut.loom', Access.ACC_RDONLY);
var dim = file.getDatasetDimensions('matrix');
for (var i = 0; i < dim[0]; i++) {
  var buffer = h5lt.readDatasetAsBuffer(file.id, 'matrix', {
    start: [i, 0],
    stride: [1, 1],
    count: [1, dim[1]]
  });
}

Error:
HDF5-DIAG: Error detected in HDF5 (1.8.20) thread 0:
#000: H5S.c line 1382 in H5Screate_simple(): maxdims is smaller than dims
major: Invalid arguments to routine
minor: Bad value
HDF5-DIAG: Error detected in HDF5 (1.8.20) thread 0:
#000: H5Dio.c line 147 in H5Dread(): not a data space
major: Invalid arguments to routine
minor: Inappropriate type
HDF5-DIAG: Error detected in HDF5 (1.8.20) thread 0:
#000: H5S.c line 390 in H5Sclose(): not a dataspace
major: Invalid arguments to routine
minor: Inappropriate type
SyntaxError: failed to read dataset

Thanks.

This is a compressed dataset.  I've been working on getting compression capability.
I haven't tried a compressed dataset with hyperslabbing yet.  Studying this some more...

hdf5 forum tells me this is doable.
https://forum.hdfgroup.org/t/are-hyperslabs-and-compression-compatible/860
Looking for a c example. it may have to do with access property list for the reading step.  Looking...

Awesome! Thanks so much.
On Wed, Feb 28, 2018 at 1:00 PM rimmartin<EMAIL_ADDRESS>wrote:

hdf5 forum tells me this is doable.
https://forum.hdfgroup.org/t/are-hyperslabs-and-compression-compatible/860
Looking for a c example. it may have to do with access property list for
the reading step. Looking...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/HDF-NI/hdf5.node/issues/55#issuecomment-369326925,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE_pnzT_pAWl7B7lthz5C2LD2z03MqKlks5tZZRHgaJpZM4SSpCQ
.

Any updates on how to fix this issue? Thanks.

I'm getting back to things.  Had a software release at work that consumed a lot of time
I'll make a small c/c++ app to test this out a bit; it really should be no different

in  some versions of hdf5 H5S_UNLIMITED didn't work.  You're making my recaller kick in:-)

I think that was it! Must reveal itself when it is a compressed h5. Survey for more places an array should be used

All other H5Screate_simple's have the maximum_dims NULL. Will need to see if any should be filed in to allow H5S_UNLIMITED

@joshua-gould  I committed changes after applying @NINI1988 's fix to this one as well.

It works now! Thanks.


My Application Crashes

Hi,
I am using this plugin in MobileFirst Application (HTML, CSS and Javascript). When the show() method is being called, my application crashes. I am testing the application on Samsung S6 Edge and it has Android Marshmallow (6.0.1). When I checked the logs, it was giving an error Resource Not Found Exception
Please Help

Does it say which resource?
Make sure the plugin is copying all the resource files to the correct directories.  If the MobileFirst framework uses different resource directories, you may need to manually copy files to the correct location.
These are the resources and target directories from the plugin.xml
        <source-file src="res/android/drawable/ic_fingerprint_error.xml" target-dir="res/drawable" />
        <source-file src="res/android/drawable/ic_fingerprint_success.xml" target-dir="res/drawable" />
        <source-file src="res/android/drawable-hdpi/ic_fp_40px.png" target-dir="res/drawable-hdpi" />
        <source-file src="res/android/drawable-mdpi/ic_fp_40px.png" target-dir="res/drawable-mdpi" />
        <source-file src="res/android/drawable-nodpi/android_robot.png" target-dir="res/drawable-nodpi" />
        <source-file src="res/android/drawable-xhdpi/ic_fp_40px.png" target-dir="res/drawable-xhdpi" />
        <source-file src="res/android/drawable-xxhdpi/ic_fp_40px.png" target-dir="res/drawable-xxhdpi" />
        <source-file src="res/android/drawable-xxxhdpi/ic_fp_40px.png" target-dir="res/drawable-xxxhdpi" />
        <source-file src="res/android/layout/fingerprint_dialog_container.xml" target-dir="res/layout" />
        <source-file src="res/android/layout/fingerprint_dialog_content.xml" target-dir="res/layout" />
        <source-file src="res/android/values/fpauth-colors.xml" target-dir="res/values" />
        <source-file src="res/android/values/fpauth-strings.xml" target-dir="res/values" />
        <source-file src="res/android/values-es" target-dir="res" />

Thanks mjwheatley. Issue Resolved.
I was looking for the files resources which you have mentioned.


dont work, but no reason

Hi,
I'm having  a strange issue, ive already ran
rake db:mongoid:create_indexes

but when i try to use spacial functions (any of them, like geo_near, ;field.where([]) ), i always get this error ;
Mongo::OperationFailure: can't find special index: 2d for: { location: { $near: [ 37.761523, -122.423575 ] } }

here's my model
class Place

include Mongoid::Document
include Mongoid::Spacial::Document

field :name,type: String
field :street,type: String
field :postal_code,type: String
field :city,type: String
field :location, type: Array, spacial:true
has_many :parties

end

Dont know what to do , i've already used this gem in production with no trouble , but i can't find how this error is thrown.
Im using ruby-1.9.3, rails 3.1, mongo 2.0.2
thanks

ok no explanation , but when i create the index in the mongo-cli , it works well, the problem must be inside the rake task , tell me if you want to investigate ....


Improve SQL widget

The API is clunky and doesn't really work if we want to update a DF easily, let's rework the way is work.
We need an easy way to create a SQL instance out of a SQL string which generates an HTML form, the HTML form should rerun the sql if changes are applied.
When the result is available it should be super easy to render in whatever widget the resulting DF.

cc @jarutis

#438


callCabal2nix: Use host platform rather than build platform for evaluation of Cabal files

Motivation for this change
The if os(...) condition feature in Cabal refers to the host platform when it gets this information from GHC.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

cc @Ericson2314


patreon blender tutorial is broken

The link doesn't work anymore

The tutorial is no longer available, so I removed the link from the Readme.


sort dialogs without regards to leading non-word characters

It is confusing when ##thischannel comes before #somechannel because of
the double hash. Therefore sort after stripping leading non-word
characters.

I don't know if there is any place to test this. I have tried it locally and it works for me.

We currently don't have any frontend tests, so manual testing is sadly the only way to see if it works.

LGTM! 👍


Compilation Speed for relatively large application

Hi, I heard you on podcast rustocean. And I was fascinated that you took the signal route.  I really like what signal js doing.
Again I really like to use Rust on server and also on client. Had kind of good run with Yew. But until things go south at the compile time.
We are having relatively large application and compilation time take practically life time. And obviously there is no HMR. This makes designing so hard. You change one CSS and it will take too much time to reflect.
What is story with leptos? Means how it is in terms of compilation time? Specially when you are have too many files and relatively large application.

There isn't hot-module reloading, but there are dev servers that include live-reloading, including Trunk and our own cargo-leptos. Your CSS should update more or less immediately, assuming it's in a CSS file and not in a .rs file. Any changes to Rust files will require recompilation, and Rust being a relatively slow-to-compile compiled language, this is significantly slower than JavaScript.
I wouldn't expect any meaningful difference in compilation times between Leptos, Yew, and any other Rust framework for a project of similar complexity — this is mostly outside our control.


Can you provide more smooth animations with amazing effect.

Yes. Yes I can provide more smooth animations with amazing effect.

Thanks for that, i'm waiting...

Who, why are you making a PR from the gh-pages to the master?


terraformer plan azure: "panic: runtime error: invalid memory address or nil pointer dereference"

Terraformer v0.8.21, macOS Monterey 12.2.1 (but similar issue observed on a Windows 10 laptop as well)
$ az login
$ export ARM_SUBSCRIPTION_ID=<somesubscriptionid>
$ terraformer plan azure -R MYRESOURCEGROUP --resources=*

2022/06/02 23:08:13 Testing if Service Principal / Client Certificate is applicable for Authentication..
2022/06/02 23:08:13 Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
2022/06/02 23:08:13 Testing if Service Principal / Client Secret is applicable for Authentication..
2022/06/02 23:08:13 Testing if Managed Service Identity is applicable for Authentication..
2022/06/02 23:08:13 Testing if Obtaining a token from the Azure CLI is applicable for Authentication..
2022/06/02 23:08:13 Using Obtaining a token from the Azure CLI for Authentication
2022/06/02 23:08:13 Getting OAuth config for endpoint https://login.microsoftonline.com/ with  tenant
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x6bad4d3]

goroutine 1 [running]:
github.com/hashicorp/go-azure-helpers/authentication.Config.GetAuthorizationToken(...)
	github.com/hashicorp/go-azure-helpers@v0.10.0/authentication/config.go:123
github.com/GoogleCloudPlatform/terraformer/providers/azure.(*AzureProvider).getAuthorizer(0xc0005489a0)
	github.com/GoogleCloudPlatform/terraformer/providers/azure/azure_provider.go:91 +0x173
github.com/GoogleCloudPlatform/terraformer/providers/azure.(*AzureProvider).Init(0xc0005489a0, {0xc000a10ca0, 0x1, 0x102800a?})
	github.com/GoogleCloudPlatform/terraformer/providers/azure/azure_provider.go:105 +0x3f
github.com/GoogleCloudPlatform/terraformer/cmd.initOptionsAndWrapper({_, _}, {{0xc000a10c80, 0x1, 0x1}, {0x11e48aa0, 0x0, 0x0}, {0xad37714, 0x1e}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/import.go:112 +0x86
github.com/GoogleCloudPlatform/terraformer/cmd.Import({_, _}, {{0xc000a10c80, 0x1, 0x1}, {0x11e48aa0, 0x0, 0x0}, {0xad37714, 0x1e}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/import.go:85 +0xbf
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAzureImporter.func1(0xc001316dc0?, {0xac8fc10?, 0x3?, 0x3?})
	github.com/GoogleCloudPlatform/terraformer/cmd/provider_cmd_azure.go:30 +0xd2
github.com/spf13/cobra.(*Command).execute(0xc001316dc0, {0xc0012a1b00, 0x3, 0x3})
	github.com/spf13/cobra@v1.1.1/command.go:850 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc00113a2c0)
	github.com/spf13/cobra@v1.1.1/command.go:958 +0x39c
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
	github.com/GoogleCloudPlatform/terraformer/cmd/root.go:36 +0x1e
main.main()
	github.com/GoogleCloudPlatform/terraformer/main.go:39 +0x50

az cli version information:
$ az --version
azure-cli                         2.37.0

core                              2.37.0
telemetry                          1.0.6

Extensions:
azure-devops                      0.18.0

Dependencies:
msal                            1.18.0b1
azure-mgmt-resource             21.1.0b1

Same issue on WSL for Windows 10.
`$ az --version
azure-cli                         2.37.0
core                              2.37.0
telemetry                          1.0.6
Dependencies:
msal                            1.18.0b1
azure-mgmt-resource             21.1.0b1`
When running:
`terraformer plan azure --output hcl --resource-group $RESOURCEGROUPNAME --resources="*" --verbose
2022/06/06 20:54:09 Testing if Service Principal / Client Certificate is applicable for Authentication..
2022/06/06 20:54:09 Testing if Multi Tenant Service Principal / Client Secret is applicable for Authentication..
2022/06/06 20:54:09 Testing if Service Principal / Client Secret is applicable for Authentication..
2022/06/06 20:54:09 Testing if Managed Service Identity is applicable for Authentication..
2022/06/06 20:54:09 Testing if Obtaining a token from the Azure CLI is applicable for Authentication..
2022/06/06 20:54:09 Using Obtaining a token from the Azure CLI for Authentication
2022/06/06 20:54:09 Getting OAuth config for endpoint https://login.microsoftonline.com/ with  tenant
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x5faf3d3]
goroutine 1 [running]:
github.com/hashicorp/go-azure-helpers/authentication.Config.GetAuthorizationToken(...)
/home/runner/go/pkg/mod/github.com/hashicorp/go-azure-helpers@v0.10.0/authentication/config.go:123
github.com/GoogleCloudPlatform/terraformer/providers/azure.(*AzureProvider).getAuthorizer(0xc00044ea80)
/home/runner/work/terraformer/terraformer/providers/azure/azure_provider.go:91 +0x173
github.com/GoogleCloudPlatform/terraformer/providers/azure.(*AzureProvider).Init(0xc00044ea80, {0xc0000fcfb0, 0x1, 0x7f468582250a?})
/home/runner/work/terraformer/terraformer/providers/azure/azure_provider.go:105 +0x3f
github.com/GoogleCloudPlatform/terraformer/cmd.initOptionsAndWrapper({_, }, {{0xc0000fcfa0, 0x1, 0x1}, {0x11297e80, 0x0, 0x0}, {0xa1397f5, 0x1e}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:112 +0x86
github.com/GoogleCloudPlatform/terraformer/cmd.Import({, _}, {{0xc0000fcfa0, 0x1, 0x1}, {0x11297e80, 0x0, 0x0}, {0xa1397f5, 0x1e}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:85 +0xbf
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAzureImporter.func1(0xc0009302c0?, {0xa091888?, 0x6?, 0x6?})
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_azure.go:30 +0xd2
github.com/spf13/cobra.(*Command).execute(0xc0009302c0, {0xc000bc58c0, 0x6, 0x6})
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc000880580)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x39c
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
/home/runner/work/terraformer/terraformer/cmd/root.go:36 +0x1e
main.main()
/home/runner/work/terraformer/terraformer/main.go:39 +0x50`

Got the same issue, with macOS Monterrey 12.4 M1 pro.
Was this working fine in a previous release of terraformer?
Thanks to all

Confirmed the same issue on WIndows 11 Pro with terraform version v0.8.21
Any feedback musch appreciated!

Confirmed the same issue on my macOS Monterrey 12.4 with
terraformer v0.8.21
azure-cli 2.37.0

Same issue on my macOS 12.4 (M1 cpu)

I encountered this issue when I tried to run terraformer before terraform init. By this time, I had also build terraformer from source with all providers.

Looks like login over "az login" is broken :(
(or isn't compatible with 2FA or similar stuff)

i have tried install terraformer from source and looks like login is OK :)
(i have MacBook with intel CPU, i'm using (almost) latest terraform binary)

I have the same issue

I have the same issue on Terraformer import

same issue

Same issue when trying to run "~/terraformer import azure -r *"  using Terraformer v0.8.22 on Ubuntu 22.04.01
Here's the pertinent bit of output
      Getting OAuth config for endpoint https://login.microsoftonline.com/ with  tenant"
      Then "panic: runtime error: invalid memory address or nil pointer dereference
      [signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x6327315]

It would be nice to see this working
Cheers
-=A=-
.

Same issue when running terraformer import azuread --resources=group
It ran until it hit a group that has 43 characters in its name.  Not sure if that's what's causing it or not, but thought I'd mention that.
Here's what it output:
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x0 pc=0x10a1c8ae8]
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/azuread.(*GroupServiceGenerator).InitResources(0x1400018f7a0)
github.com/GoogleCloudPlatform/terraformer/providers/azuread/group.go:51 +0x98
github.com/GoogleCloudPlatform/terraformer/cmd.initServiceResources({, }, {, }, {{0x14000501860, 0x1, 0x1}, {0x1152b68d8, 0x0, 0x0}, ...}, ...)
github.com/GoogleCloudPlatform/terraformer/cmd/import.go:209 +0x1a4
github.com/GoogleCloudPlatform/terraformer/cmd.initAllServicesResources(, {{0x14000501860, 0x1, 0x1}, {0x1152b68d8, 0x0, 0x0}, {0x10c0dadd4, 0x1e}, {0x10c03d33e, ...}, ...}, ...)
github.com/GoogleCloudPlatform/terraformer/cmd/import.go:166 +0x12c
github.com/GoogleCloudPlatform/terraformer/cmd.Import({, _}, {{0x14000501860, 0x1, 0x1}, {0x1152b68d8, 0x0, 0x0}, {0x10c0dadd4, 0x1e}, ...}, ...)
github.com/GoogleCloudPlatform/terraformer/cmd/import.go:98 +0x17c
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAzureADImporter.func1(0x14001241600?, {0x10c02f4a5?, 0x1?, 0x1?})
github.com/GoogleCloudPlatform/terraformer/cmd/provider_cmd_azuread.go:30 +0xb0
github.com/spf13/cobra.(*Command).execute(0x14001241600, {0x14000501810, 0x1, 0x1})
github.com/spf13/cobra@v1.1.1/command.go:850 +0x4c4
github.com/spf13/cobra.(*Command).ExecuteC(0x14001198580)
github.com/spf13/cobra@v1.1.1/command.go:958 +0x34c
github.com/spf13/cobra.(*Command).Execute(...)
github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
github.com/GoogleCloudPlatform/terraformer/cmd/root.go:36 +0x20
main.main()
github.com/GoogleCloudPlatform/terraformer/main.go:39 +0x54

Is there a solution for the error. In my case it ran fine for weeks, but now I can't get it to complete.

from my last try, new version seems to be OKish.

Hi all, I installed yesterday and I have same error doing an import. Last version of terraformer and providers installed

Same error on Windows 10 with latest version of terraform, when trying to import aws resources="*". Getting below error
`panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x53f6f9f]
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.fromPort(...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:362
github.com/GoogleCloudPlatform/terraformer/providers/aws.baseRuleAttributes({, }, {0x0, 0xc002540e60, {0xc001d688b8, 0x1, 0x1}, {0x85ba2f0, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:164 +0x33f
github.com/GoogleCloudPlatform/terraformer/providers/aws.processRule({0x0, 0xc002540e60, {0xc001d688b8, 0x1, 0x1}, {0x85ba2f0, 0x0, 0x0}, {0x85ba2f0, 0x0, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:145 +0x718
github.com/GoogleCloudPlatform/terraformer/providers/aws.SecurityGenerator.createResources({{{{0x5fafb25, 0x2}, {0x0, 0x0, 0x0}, {0x5fb00f8, 0x3}, 0xc001ce0540, {0xc000dea6c0, 0x2, ...}, ...}}}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:84 +0x7cd
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*SecurityGenerator).InitResources(0xc00249e300)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:260 +0x3b0
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).InitResources(0xc0024829b0?)
/home/runner/work/terraformer/terraformer/providers/aws/aws_facade.go:61 +0x26
github.com/GoogleCloudPlatform/terraformer/cmd.initServiceResources({, }, {, }, {{0xc00098d800, 0x54, 0x80}, {0x85ba2f0, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:209 +0x21e
github.com/GoogleCloudPlatform/terraformer/cmd.initAllServicesResources(, {{0xc00098d800, 0x54, 0x80}, {0x85ba2f0, 0x0, 0x0}, {0x6021037, 0x1e}, {0x5fba256, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:166 +0x1a7
github.com/GoogleCloudPlatform/terraformer/cmd.Import({, _}, {{0xc00098d800, 0x54, 0x80}, {0x85ba2f0, 0x0, 0x0}, {0x6021037, 0x1e}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:98 +0x2a5
github.com/GoogleCloudPlatform/terraformer/cmd.importRegionResources({{0xc0002e0690, 0x1, 0x1}, {0x85ba2f0, 0x0, 0x0}, {0x6021037, 0x1e}, {0x5fba256, 0x9}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:121 +0x292
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAwsImporter.func1(0xc0001a62c0?, {0x5fb0cfc?, 0x5?, 0x5?})
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:58 +0x5ae
github.com/spf13/cobra.(*Command).execute(0xc0001a62c0, {0xc0002e85a0, 0x5, 0x5})
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc0003ad600)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x39d
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
/home/runner/work/terraformer/terraformer/cmd/root.go:36 +0x1e
main.main()
/home/runner/work/terraformer/terraformer/main.go:39 +0x50`

I am having the same issue,  I get this error o ky when trying to import aws security groups from a region. I was able to import all other resources fine.
I tried filtering to import just one security group and the same error again, has anyone tried to import individual resources and faced the same issue?

@Gopinath-Radhakrishnan ,
I have below code in sg.go
`func fromPort(ip types.IpPermission) int {
switch {
case *ip.IpProtocol == "icmp":
return -1
case *ip.IpProtocol == "-1":
return -1
case ip.FromPort != nil && *ip.FromPort > 0:
return int(*ip.FromPort)
default:
return 0
}
}
func toPort(ip types.IpPermission) int {
switch {
case *ip.IpProtocol == "icmp":
return -1
case *ip.IpProtocol == "-1":
return -1
case ip.ToPort != nil && *ip.ToPort > 0:
return int(*ip.ToPort)
default:
return 65536
}
}`
But still I am getting error while importing AWS SG.
`C:\Users\tapan.shah\Desktop\terraformer\terraformer>terraformer import aws --resources=sg  --regions=ap-northeast-1
2022/11/22 10:16:21 aws importing region ap-northeast-1
2022/11/22 10:16:24 aws importing... sg
panic: runtime error: invalid memory address or nil pointer dereference
[signal 0xc0000005 code=0x0 addr=0x0 pc=0x4ed6f9f]
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.fromPort(...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:362
github.com/GoogleCloudPlatform/terraformer/providers/aws.baseRuleAttributes({, }, {0x0, 0xc000c52410, {0xc000c1e0d8, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:164 +0x33f
github.com/GoogleCloudPlatform/terraformer/providers/aws.processRule({0x0, 0xc000c52410, {0xc000c1e0d8, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, {0x809a2f0, 0x0, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:145 +0x718
github.com/GoogleCloudPlatform/terraformer/providers/aws.SecurityGenerator.createResources({{{{0xc0008b5740, 0x2}, {0x0, 0x0, 0x0}, {0x5a900f8, 0x3}, 0xc001a067e0, {0x809a2f0, 0x0, ...}, ...}}}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:84 +0x7cd
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*SecurityGenerator).InitResources(0xc00089c120)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:260 +0x3b0
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).InitResources(0xc0003b6640?)
/home/runner/work/terraformer/terraformer/providers/aws/aws_facade.go:61 +0x26
github.com/GoogleCloudPlatform/terraformer/cmd.initServiceResources({, }, {, }, {{0xc000657830, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:209 +0x21e
github.com/GoogleCloudPlatform/terraformer/cmd.initAllServicesResources(, {{0xc000657830, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, {0x5b01037, 0x1e}, {0x5a9a256, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:166 +0x1a7
github.com/GoogleCloudPlatform/terraformer/cmd.Import({, _}, {{0xc000657830, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, {0x5b01037, 0x1e}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:98 +0x2a5
github.com/GoogleCloudPlatform/terraformer/cmd.importRegionResources({{0xc0006577f0, 0x1, 0x1}, {0x809a2f0, 0x0, 0x0}, {0x5b01037, 0x1e}, {0x5a9a256, 0x9}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:121 +0x292
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAwsImporter.func1(0xc000491080?, {0x5a90cfc?, 0x2?, 0x2?})
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:58 +0x5ae
github.com/spf13/cobra.(*Command).execute(0xc000491080, {0xc0008c44e0, 0x2, 0x2})
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc000490580)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x39d
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
/home/runner/work/terraformer/terraformer/cmd/root.go:36 +0x1e
main.main()
/home/runner/work/terraformer/terraformer/main.go:39 +0x50`

Facing the same issue. Can someone provide the solution if you were able to fix. Thanks

I fixed the problem by shifting to Terrafy as it's a lot easier to use, and
not only exports the existing state, it generates the .tf files as well
It could be better at exporting things, but it does what it says on the
tin, and it makes it easy to export an existing Azure infrastructure (or at
least as much as it can currently handle)
Probably not the solution most are looking for - but it's the best one for
me at this point in time
Cheers
-=A=-
.
On Tue, Jan 3, 2023 at 5:29 PM Chaitanya Boggavarapu <
@.***> wrote:

Facing the same issue. Can someone provide the solution if you were able
to fix. Thanks
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/terraformer/issues/1340#issuecomment-1370039426,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARAYJJM6RIERVMPZ4ZYRKVDWQROWPANCNFSM5XWH7CKA
.
You are receiving this because you commented.Message ID:
@.***>

@dbhaigh ,
I need solution for AWS infrastructure and Terrafy seems to be for Azure.

It is Azure-centric as it is being developed by Microsoft - great for me,
but unfortunately not for you
Hope the terraform devs sort their problems out as this issue has been
outstanding for a while now
On Wed, 4 Jan 2023, 19:38 tapanshah7891, @.***> wrote:

@dbhaigh https://github.com/dbhaigh ,
I need solution for AWS infrastructure and Terrafy seems to be for Azure.
—
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/terraformer/issues/1340#issuecomment-1371340637,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARAYJJNFXHTX2KKQXABC4DTWQXGU7ANCNFSM5XWH7CKA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Hi,
the problem for aws seems to be faced when we import sgs, i have 47 security group in tis account :
.....
.....
2023/01/23 16:51:11 aws done importing ses
2023/01/23 16:51:11 aws importing... sfn
2023/01/23 16:51:12 aws done importing sfn
2023/01/23 16:51:12 aws importing... sg
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x4d70abf]
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.fromPort(...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:362
github.com/GoogleCloudPlatform/terraformer/providers/aws.baseRuleAttributes({_, _}, {0x0, 0xc000b9a890, {0xc002238d98, 0x1, 0x1}, {0x7eff210, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:164 +0x33f
github.com/GoogleCloudPlatform/terraformer/providers/aws.processRule({0x0, 0xc000b9a890, {0xc002238d98, 0x1, 0x1}, {0x7eff210, 0x0, 0x0}, {0x7eff210, 0x0, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:145 +0x718
github.com/GoogleCloudPlatform/terraformer/providers/aws.SecurityGenerator.createResources({{{{0x59285be, 0x2}, {0x0, 0x0, 0x0}, {0x5928ade, 0x3}, 0xc002089f80, {0x7eff210, 0x0, ...}, ...}}}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:84 +0x7cd
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*SecurityGenerator).InitResources(0xc0020e8fc0)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:260 +0x3b0
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).InitResources(0xc0020b2500?)
/home/runner/work/terraformer/terraformer/providers/aws/aws_facade.go:61 +0x26

the problem for aws seems to be faced when we import sgs, i have 47 security group in tis account :

yes, got the same issue when importing security groups. I have 178 of them
`aws importing... sg
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x4d70abf]
goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.fromPort(...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:362
github.com/GoogleCloudPlatform/terraformer/providers/aws.baseRuleAttributes({, }, {0x0, 0xc000e88b10, {0x7eff210, 0x0, 0x0}, {0x7eff210, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:164 +0x33f
github.com/GoogleCloudPlatform/terraformer/providers/aws.processRule({0x0, 0xc000e88b10, {0x7eff210, 0x0, 0x0}, {0x7eff210, 0x0, 0x0}, {0x7eff210, 0x0, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:126 +0xa98
github.com/GoogleCloudPlatform/terraformer/providers/aws.SecurityGenerator.createResources({{{{0x59285be, 0x2}, {0x0, 0x0, 0x0}, {0x5928ade, 0x3}, 0xc00258d2c0, {0x7eff210, 0x0, ...}, ...}}}, ...)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:81 +0x6ed
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*SecurityGenerator).InitResources(0xc001e7d860)
/home/runner/work/terraformer/terraformer/providers/aws/sg.go:260 +0x3b0
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).InitResources(0xc002036550?)
/home/runner/work/terraformer/terraformer/providers/aws/aws_facade.go:61 +0x26
github.com/GoogleCloudPlatform/terraformer/cmd.initServiceResources({, }, {, }, {{0xc0006c5800, 0x54, 0x80}, {0x7eff210, 0x0, 0x0}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:209 +0x21e
github.com/GoogleCloudPlatform/terraformer/cmd.initAllServicesResources(, {{0xc0006c5800, 0x54, 0x80}, {0x7eff210, 0x0, 0x0}, {0x5998962, 0x1e}, {0x5932b62, ...}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:166 +0x1a7
github.com/GoogleCloudPlatform/terraformer/cmd.Import({, _}, {{0xc0006c5800, 0x54, 0x80}, {0x7eff210, 0x0, 0x0}, {0x5998962, 0x1e}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/import.go:98 +0x2a5
github.com/GoogleCloudPlatform/terraformer/cmd.importRegionResources({{0xc0003db2d0, 0x1, 0x1}, {0x7eff210, 0x0, 0x0}, {0x5998962, 0x1e}, {0x5932b62, 0x9}, ...}, ...)
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:121 +0x292
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAwsImporter.func1(0xc0001d8580?, {0x5929668?, 0x1?, 0x1?})
/home/runner/work/terraformer/terraformer/cmd/provider_cmd_aws.go:66 +0x13e
github.com/spf13/cobra.(*Command).execute(0xc0001d8580, {0xc0003db2b0, 0x1, 0x1})
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850 +0x67c
github.com/spf13/cobra.(*Command).ExecuteC(0xc000345080)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958 +0x39d
github.com/spf13/cobra.(*Command).Execute(...)
/home/runner/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
/home/runner/work/terraformer/terraformer/cmd/root.go:36 +0x1e
main.main()
/home/runner/work/terraformer/terraformer/main.go:39 +0x50
`

I am still facing issue with AWS SG import. Anyone got the solution around it?

Same issue with AWS SG import

Same Issue with AWS SG import

Still an issue with 0.8.24, using this on Windows 10 with the Azure provider.

Same problem importing aws sg, running on amazon linux


@itwin/oidc-signin-tool_v4.0.1

Rush and our desired release workflow do not fully work as intended. Until we replace rush #150 this cleanup is necessary.

prob gonna need rush change with an empty change comment and none as type to get this through

prob gonna need rush change with an empty change comment and none as type to get this through

Yep just pushed