AutoComplete - Cannot read property 'props' of undefined

with 0.14-rc2
I'm using an autocomplete component and getting the error below whenever I try to use that field.
Uncaught TypeError: Cannot read property 'props' of undefined
here in element.props
ReactElement.cloneElement = function (element, config, children) {
  var propName;

  // Original props are copied
  var props = assign({}, element.props);

component markup
        <AutoComplete
          showAllItems={true}
          dataSource={this.data.topics}
          searchText= "topics"
          floatingLabelText = "choose topic"
          onUpdateInput={(t) => {console.log(t); this.setState({input1: [t, t+t, t+t+t]});}}
          onNewRequest={(t) => {console.log('request:'+t);}}
        />

is there a way to set element?

I don't understand the the flow of code that leads to this bug. can you provide more details please?

I have the same error when passing array of objects instead of strings to dataSource property
const users = [
    {text: 'John Doe (john@doe.com)'},
    {text: 'Jack Johnson (jack@johnson.com)'}
];

<AutoComplete
          dataSource={users}
          filter={AutoComplete.caseInsensitive}/>

Just plain text works perfectly well
const users = [
    'John Doe (john@doe.com)',
    'Jack Johnson (jack@johnson.com)'
];

<AutoComplete
          dataSource={users}
          filter={AutoComplete.caseInsensitive}/>

v 0.14.2
I'm trying to create autocomplete items as objects:
const options = [];
    OwnersStore.getOwners().forEach((owner, key) => {
      options.push({text: owner.name, value: key});
    });
    return options;

But I'm getting the error:
Uncaught Invariant Violation: Element type is invalid: expected a string (for built-in components) or a class/function (for composite components) but got: undefined. Check the render method of `Menu`.

Look here to use objects : https://github.com/callemall/material-ui/pull/2286

@dcsan @dimasmith @Lopinopulos : Is this still an issue for you guys? If you were not able to resolve this using the above suggestion and need further help on this do let us know.
I am closing this issue for now. Feel free to leave a comment below, if you feel this needs further addressing.


new declaration api

https://heksher-py--22.org.readthedocs.build/en/22/

Codecov Report

Merging #22 (3bef12c) into master (5581dde) will decrease coverage by 2.09%.
The diff coverage is 91.48%.

@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
- Coverage   97.98%   95.88%   -2.10%
==========================================
  Files          13       13
  Lines         743      875     +132
==========================================
+ Hits          728      839     +111
- Misses         15       36      +21

Flag
Coverage Δ

unittests
95.88% <91.48%> (-2.10%)
:arrow_down:

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

Impacted Files
Coverage Δ

heksher/main_client.py
83.33% <50.00%> (ø)

heksher/clients/subclasses.py
90.35% <80.59%> (-4.95%)
:arrow_down:

heksher/clients/async_client.py
94.59% <90.56%> (-4.53%)
:arrow_down:

heksher/setting.py
98.05% <95.23%> (-1.95%)
:arrow_down:

heksher/setting_type.py
96.82% <96.29%> (+0.15%)
:arrow_up:

heksher/__init__.py
100.00% <100.00%> (ø)

heksher/_version.py
100.00% <100.00%> (ø)

heksher/clients/stub.py
100.00% <100.00%> (ø)

heksher/clients/thread_client.py
100.00% <100.00%> (ø)

heksher/clients/util.py
100.00% <100.00%> (+1.51%)
:arrow_up:

... and 5 more

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 5581dde...3bef12c. Read the comment docs.


Support version 1.1 of vhea table

The vhea table has a version 1.1 that changes the name and definition of the following fields:

ascender becomes vertTypoAscender
descender becomes vertTypoDescender
lineGap becomes vertTypoLineGap

https://docs.microsoft.com/en-us/typography/opentype/spec/vhea
The new names are not available in Lib/fontTools/ttLib/tables/_v_h_e_a.py yet.

See also #1714 and #1715 for how to implement it in a b/w compatible way.


bump bip174-liquid

this PR bumps bip174-liquid to 1.0.2
@tiero please review

You cam bump 1.0.3 @louisinger


Sort not work with parser: "babel-flow"

Here is my .prettierrc.json
{   "singleQuote": true,   "importOrder": ["^src/(.*)$", "^[./]"],   "importOrderSeparation": true,   "printWidth": 120,   "parser": "babel-flow" }
I try removing parser line then it works normally, but it won't sort with the parser: "babel-flow. Is this a bug or I was wrong somewhere?

@Yammoe Thanks the context. I need to check your use case.

@ayusharma  Is there any update on this? Thanks.

@Yammoe I'll check in this week. Sorry for the delays.

Hey @Yammoe the changes are merged and they will be released soon 👍

Thanks @byara . Waiting for it 😄

With the release of v2.0.1, this should be fixed. Either prettier will figure out the syntax correctly, or you can use experimentalBabelParserPluginsList in your .prettierrc to pass the correct parser name.
If there was still problems, you can of course re-open this issue.

Hei
I have just tested this in the current Version and I think it is not working any longer. I have created a Repository to demonstrate it, here the link https://github.com/wityan/prettier-import-example-error.
In src/index.jsx the common import should come before the test import.
Is there an error in my config or is something not working anymore?


[BUG] Datepicker and Timepicker not working properly when you have to scroll to get to them

I am building a website, and I cannot get neither the date nor the timepicker to work properly. There's a table that takes up most of the screen, and when you click on a row, the both pickers show up at the bottom (among other elements). But when you have to scroll to get to the pickers, they start behaving erratically. The datepicker's popup appears at different places at random, and the timepicker's modal shows up, but wherever you click, it always picks the same time (6 AM).
I am using Bootstrap 4. At first I thought it was an issue with the table, because if I deleted it, the pickers behaved normally, but after a lot of debugging, I noticed that the bug only occurs when the pickers are displayed at a place you have to scroll to, regardless of what's above them. To test that, I created a container, made several rows with the letter A, set the font to 20em, so that they pushed the pickers downwards. The pickers stopped working when they were somewhere you had to scroll to. Deleting the As one by one, from the inspector, as soon as the pickers became visible without scrolling, they immediately started behaving normally again (without even refreshing the page), and when you push them down again, the bug returns.
Any thoughts on what could be causing the issue? If you need more info or screencaps, let me know!
[EDIT]: bvn13's reported bug is the same as mine. To replicate it, you might want to try what I described above. My timepicker behaves exactly as his. This bug affects the datepicker too.
[EDIT 2]: You can replicate bvn13's bug in the example page you provided by adding a margin-top of 1500px and pushing down the picker in the inspector.

Thanks for the additional information. I'm looking at this. I will keep you posted.

We fixed that in the new development version that is going to be released as 1.9.4 later today.


Postsgres Error on character type on bulkUpdateAsync

System.ArgumentException: A PostgreSQL type with the name character(1) was not found in the database
at Npgsql.Internal.NpgsqlDatabaseInfo.GetPostgresTypeByName(String pgName)
at Npgsql.TypeMapping.ConnectorTypeMapper.ResolveByDataTypeName(String typeName, Boolean throwOnError)
at Npgsql.NpgsqlParameter.ResolveHandler(ConnectorTypeMapper typeMapper)
I am currently on version 6.3.7

Can you write test where the issue would be reproducible.

I have a property with the following configuration in the entity:
entity.Property(e => e.StoreType)
.IsRequired()
.HasMaxLength(1)
.IsUnicode(false)
.IsFixedLength()
.HasDefaultValueSql("('A')");

@borisdj
if (columnType.StartsWith("character varying")) // when MaxLength is defined
columnType = "character"; // 'character' is like 'string'
i guess if we just change the start with to character the issue should be resolved

Try now with v6.3.9
If StoreType doesn't have value you need to set BulkConfig with PropertiesToExclude = new List<string> { "StoreType" }


Flowchart.js github pages broken (ACE.js Dependency 404ing)

Hi. Looks like the website for this is broken. You currently can't use the code editors to preview what the SVG would look like. Seems at a quick glance in devtools that the ACE editor isn't accessible from your page anymore. Maybe something going on with the CDN.

This was corroborated by a user of a downstream project I created here: https://github.com/francoislaberge/diagrams/issues/3, he was seeing the same issue.

Can you submit a PR to the gh-pages branch?

should be fixed now, thx


feat(STONEINTG-415): optional-label-check should be skipped by default

This change will allow optional label check to be run only in case that the parameter: skip-optional is set to "false".

optional-label-check is skipped

Curious, what are the optional label checks that we are skipping/not skipping ?

Curious, what are the optional label checks that we are skipping/not skipping ?

The optional checks include the inherited labels and optional labels.

Looks good 👍


Build an API to receive files and output C2PA-injected images

Task Summary
📅 Due date: 24 Nov 2021
🎯 Success criteria: Build an API that allows backend to receive [ photos + JSON ] and inject C2PA with capture signatures in Capture assertion.
Links you'll need:

claim-tool binary and docs: https://github.com/starlinglab/organizing-private/issues/14#issuecomment-973704015
C2PA v0.8 verifier: https://verify.contentauthenticity.org/inspect
C2PA v0.8 spec: https://c2pa.org/public-draft/
Starling App (Android-only): https://www.dropbox.com/sh/23twqukw4mlk0lv/AACesyMPrH5cLtyEwXDesqcwa?dl=0

This is a dev build from @benhylau (same one used by Pablo)
@benhylau will document data format in #3

Numbers Data Integrity Assurance (DIA) API v3: https://dia-backend.numbersprotocol.io/api/v3/redoc/

This is what Starling Capture app currently talks to

The master issue for SCMP coordination: https://github.com/starlinglab/organizing-private/issues/22

To Do

[x] @benhylau to provide Starling Capture test files
[ ] spec API to receive and fetch
[x] add basic auth
[x] implement APIs in Python
[x] @benhylau to provide claim format for Capture assertion injection
[x] do an injection thru claim-tool
[ ] verify sending source files via POST and fetching C2PA-injected images thru GET

🤔 how we (e.g. SCMP) list directories?

I think we built the APIs for the job at the time. Future APIs can be scoped in new tickets. Closing!


chore: add .gitignore files

Sidenote: I think it's a bit unusual that the Makefile copies the binary into the top-level directory.

Sidenote: I think it's a bit unusual that the Makefile copies the
binary into the top-level directory.

I think it was mostly to have a convenient target that corresponds to
the actual file it creates and also make it easy to find. Probably not
very typical for Rust projects, though, since they are run mostly
through cargo anyway.
Is there a benefit to having multiple .gitignore files spread out
instead of placing everything in the root .gitignore? It seems easier to
read and edit if it is just a single file.

Yeah defining the gitignores as close to the ignored files as possible helps with maintainability when moving dirs around or removing dirs, you're less likely to end up with outdated or leftover gitignore entries.

looks good, thanks!


Improve Helm Chart

Pull Request Template
⚠ Before Submitting a PR, Please Review:

Please ensure that you have thoroughly read and understood the Contributing Docs before submitting your Pull Request.

⚠ Documentation Updates Notice:

Kindly note that documentation updates are managed in this repository: librechat.ai

Summary
For now, it's a draft to document my progress.
CC: @danny-avila, @iamNoah1
Done

Subcharts for Dependencies
Create RAG_API Subchart
Provide full getting started Guide

Todo

Move Guide to Docs
Add Rag_Api Guide
Keep Chart 1:1 Compatible to old Chart/ Document Changes with Version Change

Change Type
Please delete any irrelevant options.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[X] This change requires a documentation update
[ ] Translation update

Testing
Please describe your test process and include instructions so that we can reproduce your test. If there are any important variables for your testing configuration, list them here.
Test Configuration:
Checklist
Please delete any irrelevant options.

[ ] I have performed a self-review of my own code
[X] I have commented in any complex areas of my code
[ ] I have made pertinent documentation changes
[ ] I have written tests demonstrating that my changes are effective or that my feature works
[ ] Any changes dependent on mine have been merged and published in downstream modules.
[ ] A pull request for updating the documentation has been submitted.

@danny-avila @iamNoah1
I'm struggling a bid of choosing a way of the config syntax.
So in our chart for example we chose to place env config under librechat.configEnv and in the official chart its config.env.
No big deal, but everyone that uses the other chart has to manually migrate his config to the new format. Should I decide based on official/unofficial, user count of the chart, or make some dual-solution that just takes both config variants (maybe with an additional deprecation warning)?

Should I decide based on official/unofficial, user count of the chart, or make some dual-solution that just takes both config variants (maybe with an additional deprecation warning)?

I think this is pretty recent so whatever you choose shouldn’t be too disruptive. Maybe both just to be safe but I don’t think it’s totally necessary

@hofq I don't have a strong opinion here.

Thanks @iamNoah1 :) I will look into it if i find time to rebase that whole pr. There where some Changes to our chart repo in the past which also should be included here.
Additionally maybe putting renovate on the table for that PR would also be nice


Update sofaboot-sample-with-isolation

delete unnecessary configuration
fix java doc
update sofaboot-sample-with-isolation

Pull Request Test Coverage Report for Build 96

0 of 0 (NaN%) changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 27.871%

Totals

Change from base Build 94:
0.01%

Covered Lines:
500

Relevant Lines:
1794

💛  - Coveralls


HelpDot image request for QBD Expensify Classic articles

Please provide the following images in light mode and in .png format. Whoever completes this task will need a connection to QuickBooks Desktop. I do not have access to the QBD connection and therefore do not know how accurate or feasible a couple of these requests are. Please let me know if any of the steps below are inaccurate.
CONFIGURE QUICKBOOKS DESKTOP
Link to article: https://help.expensify.com/articles/expensify-classic/connections/quickbooks-desktop/Configure-Quickbooks-Desktop

In Expensify, go to Settings > Workspaces > [Workspace name] > Connections > highlight the Export button under the QuickBooks Desktop connection.

In another photo, highlight the Sync button.
In another photo, highlight the Import button.
In another photo, highlight the Advanced option/tab.
In a final photo, highlight the Autosync option.

QUICKBOOKS DESKTOP TROUBLESHOOTING
Link to article: https://help.expensify.com/articles/expensify-classic/connections/quickbooks-desktop/Quickbooks-Desktop-Troubleshooting

Right-click the Web Connector icon on your desktop and select Run as administrator.
In QBD, show the following steps in images:

Log in to QuickBooks Desktop as an Admin (in single-user mode).
Go to Edit > Preferences > Integrated Applications > Company Preferences.
Select the Web Connector and click Properties.
Make sure that the “Allow this application to login automatically” checkbox is selected and click OK.

In the Web Connector, show the following steps in images:

Check that the Last Status is “Ok”.
Check the Report Comments in Expensify to confirm that the report has been successfully exported to QuickBooks Desktop.

@jliexpensify - assigning you as a liaison for assisting design with getting a few QBD images for the Exfy Classic. since you handled the recent updates to QBD on HelpDot I figured i would assign this one to you to assist. tysm!

ah looks like Jli is OOO. @kadiealexander if you have some bandwidth over the next few days to assist with some QBD screenshots for these help docs, that would be lovely!

I need some screenshot help here as I don't have an account with QBD access. Is it possible to get the screenshots above in 1440x900 resolution. Then I can wrap them up into our screenshots template

In Expensify, go to Settings > Workspaces > [Workspace name] > Connections > highlight the Export button under the QuickBooks Desktop connection.

In another photo, highlight the Sync button.

In another photo, highlight the Coding button.

In another photo, highlight the Advanced option/tab.

In a final photo, highlight the Autosync option. - it's in the advanced tab above, I don't think it's useful to highlight it twice.
In QBD, show the following steps in images:
Log in to QuickBooks Desktop as an Admin (in single-user mode). - This isn't really something that's screenshottable.
Go to Edit > Preferences > Integrated Applications > Company Preferences.

Select the Web Connector and click Properties. Make sure that the “Allow this application to login automatically” checkbox is selected and click OK.

In the Web Connector, show the following steps in images:
Check that the Last Status is “Ok”.

Check the Report Comments in Expensify to confirm that the report has been successfully exported to QuickBooks Desktop.

Alright. Here's the screenshots. Please check and ensure they're all correct:
The zip file with mocks: qbd.zip

@dubielzyk-expensify these look great! @ren-jones do you need us to add them to the article?

Adding the images to HelpDot here. https://github.com/Expensify/App/pull/50702

Added images to the articles here.


Pass for check_undefined and added tests

Everything looks really good, except for scanner.py. I'm not sure what needs to happen there? It shouldn't be touched as far as I can see.

how are comments started in this language?

With a #, e.g:
# this is a comment
pub foo = 'hello, world'

oh, all good. my tests were wrong then. lemme change them

updated


IndexOpsMixin and its children should use its own sdf.

Codecov Report

Merging #989 into master will decrease coverage by 2.45%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #989      +/-   ##
==========================================
- Coverage   94.84%   92.39%   -2.46%
==========================================
  Files          34       34
  Lines        6519     6519
==========================================
- Hits         6183     6023     -160
- Misses        336      496     +160

Impacted Files
Coverage Δ

databricks/koalas/series.py
96.18% <100%> (-0.14%)
:arrow_down:

databricks/koalas/base.py
94.9% <100%> (ø)
:arrow_up:

databricks/koalas/indexing.py
94.38% <100%> (ø)
:arrow_up:

databricks/koalas/frame.py
93.84% <100%> (-2.42%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
24.54% <0%> (-72.73%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
78.04% <0%> (-7.32%)
:arrow_down:

databricks/conftest.py
91.48% <0%> (-6.39%)
:arrow_down:

databricks/koalas/groupby.py
89.34% <0%> (-2.05%)
:arrow_down:

... and 5 more

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 feca119...2b9da89. Read the comment docs.


Android - Tag - On disabling region sub tags, "required" is shown

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

Version Number: 1.4.80
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4602370
Issue reported by: Applause - Internal Team
Action Performed:
Pre-condition: Have tag enabled in new dot and in old dot upload dependent tag

Launch app
Go to Workspace settings
Tap tag-- state-- California
Edit name and tap save
Wait few seconds and note hmm not here shown
Tap app back button
Note state field , " Required " tag shown beside
Tap region
Select both sub classes and disable tag
Tap app back button and note "region" field

Expected Result:
After editing state sub tag hmm not here must not be shown. Since people must tag expense selected, "required" tag must be displayed beside "state" tag initially itself
After disabling region sub tags, "required" must not be shown beside region field
Actual Result:
After editing state sub tag hmm not here shown and "required" tag displayed beside "state" tag
After disabling region sub tags, "required" is shown beside region field
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/0e6cc143-a3c2-4686-a413-a789da955e85
https://github.com/Expensify/App/assets/78819774/ccc5dd80-0945-4d63-9bbb-095bead6be91
View all open jobs on GitHub

Hi @amyevans @marcochavezf I think you're both working on Tags, right?
Is the issue here that California56 isn't a real location, which is why it's throwing up a required error?

I'm not super familiar with tags, but I wonder why the page "hmm not here" page is shown? From the test steps looks like that's an expected behavior but probably it's causing the required error

I'm not able to reproduce the first bug:
https://github.com/Expensify/App/assets/7277067/c9a29918-e114-4cfa-9a59-cf8c99359f16
As for the second, I'm not quite sure what is going on. But I do know we don't support dependent tags in NewDot yet (slated for November release: https://github.com/Expensify/Expensify/issues/388722), so I suspect it might just be a side effect of that and producing unexpected behavior. For comparison, on OldDot I see this message:

Got it, thanks to both of you for checking this out! So it sounds like we can close this GH based off the fact that Tags in ND is still a work in progress?

Yeah I'm onboard with closing.


License

In the main pom.xml, there are two licenses indicated, WTFPL & BSD. However, on the GitHub home page of this project, at the bottom of the page, it says the code is licensed under WTFPL. So my question is, what is the applicable license? We have a concern in using this library because our code is Apache Licensed and WTFPL is not OSI approved, so there is a bit of a legal grey area. Please clarify whether this library is dual licensed under WTFPL & BSD or is it licensed only under WTFPL.
Thanks
Azeez

I tried to clarify this in pull #107

Fixed

@afkham pull #107 hasn't been merged, so I don't think this issue is fixed

Sorry, the PR hasn't been merged.

License is WTFPL.
I would even use a shorter version of it (if there was).
This library is published as an act of giving and generosity, from developers to developers. All the licensing governance mumble jumble does not concern me.
Please feel free to use it, and to contribute to the developers community in the same manner. One solution is to fork it and use another license.


Logging level: switched logic

Logging should only happen if the logging level is at least as high as the one set, before it was the other way around. E.g. when set to notice, notices, warning, errors, critical should be logged but info, debug and trace should not be.
Motivation and Context
Before this, one could not set the logging level to something meaningful, because one would miss the messages with the highest priority.
How Has This Been Tested?
Added a helper method wouldLog to LuminaLogger with the same logic as the logging methods. Added a unittest with the levels as example. Had to update the deployment target to 11.0 at two missing occasions for the tests to compile.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

Thanks @gerriet! I’m sorry it took me so long to come around to this - new job and all 😄
I think this is fine because I have a major re-design that I’m thinking of, using Combine and a new architecture. For now, I have no issues merging this.

Hey @dokun1, I figured that it had to do with your new job. Congratulations, btw. Looking forward to your redesign.


Created rfid_read_personal_data.ino

Sample code to read personal data that was written by "rfid_write_personal_data.ino" in "examples".

@ryand1011 Please add this example to travis build script config travis.yaml.

All the other examples failed, mine didn't however (??).

@Rotzbua Thanks for the request, I just updated it

@ryand1011 I have no idea why it suddenly fail. The script only fails with teensy build parameters, because it takes an old damaged library which was fixed in 2016.. as workaround I will remove temp remove teensy from build script


POST request with 400 response throws ConnectionError

I make a POST request to an internal API. When some preconditions are not met the API responds with 400.
Expected Result
Getting a response object with status_code == 400
Actual Result
requests throws excpetion with ConnectionError
System Information
$ python -m requests.help

{
  "chardet": {
    "version": "3.0.4"
  },
  "cryptography": {
    "version": ""
  },
  "idna": {
    "version": "2.6"
  },
  "implementation": {
    "name": "CPython",
    "version": "3.6.4"
  },
  "platform": {
    "release": "4.4.0-81-generic",
    "system": "Linux"
  },
  "pyOpenSSL": {
    "openssl_version": "",
    "version": null
  },
  "requests": {
    "version": "2.18.4"
  },
  "system_ssl": {
    "version": "20000000"
  },
  "urllib3": {
    "version": "1.22"
  },
  "using_pyopenssl": false
}

Additional information: This happens when the payload of the request was not received, yet. So it seems, requests can't handle this interruption.

Even more information: Only happens when data parameter is a file object created with open.

@sebastianbauer there isn't enough information here to properly debug this. At a minimum we'll need the stacktrace so we can identify what kind of ConnectionError you're seeing, and preferably you'd also include a snippet of code to reproduce the issue.
Is this reproducible on more servers than your own? If not, this is likely a behavior in the interaction of your two code bases which is causing an issue. In that case, we'd prefer this was followed up on StackOverflow which we have designated as a place for Requests questions/assistance inquiries. This issue tracker is specifically for reproducible defects in Requests. Thanks!

@nateprewitt Thanks for your comment. I investigated further and it actually is not an issue with requests itself, but some underlaying libs. It belongs to this one: https://bugs.python.org/issue25919
Sorry for the trouble.

No problem, thanks for checking back in! Closing since there's nothing left to address here.


[Data collection] Show message when draft is restored

A simple modal with text: "The app was closed during data collection. Your unsaved draft has been restored."
@amegantz for copy. Note that the app can be closed by the OS when in the background to conserve resources, or the user could have force quit the app, and we have no way to determine which happened.

I'd replace "draft" with "data," otherwise LGTM!
(Also I know "data" is technically plural, but it's basically singular now in common practice)

I'd replace "draft" with "data," otherwise LGTM!
(Also I know "data" is technically plural, but it's basically singular now in common practice)

SGTM! If we want to do this as a snackbar, how would we shorten this? Something like "App was closed. Unsaved data restored."?


Add 5 proposer test

This makes the multiproposer test take a -num-proposers, and run for 3 minutes. It also rewrites the prefix check to be tolerant of the many forks that happen during Ouroboros consensus. It does this by querying the "best path" (ie, list of state hashes from the tf root to tf best tip) for each node and ensuring they agree in some node in the tree. If ever a path becomes disjoint, the node has forked and the test fails.

seems the new shared prefix check isn't quite as stable -- shame. I'll investigate why...

it was less stable because of https://github.com/CodaProtocol/coda/pull/2011. This should be good to go now.

latest failures are best_path RPC getting interrupted when we restart the node.

These new commits need review:
https://github.com/CodaProtocol/coda/pull/1870/commits/00c63b99e97879444b421a8d2d2991e63373306e
https://github.com/CodaProtocol/coda/pull/1870/commits/8dd0552d7e8665fe5728af5f31dea076e3b227c4
https://github.com/CodaProtocol/coda/pull/1870/commits/1a071d34af0ba7fc6a3ca2cbaaaebc4bfa66bc01

Nits on 00c63b9 but looks good to me!


update_engine failure to update when passing through a proxy

Description
The update process used to work through proxies with environment variable "ALL_PROXY".  Today, we've realized that the update process have failed for all machines that are being the proxy.
This is what we used to inject into the update-engine.service through a dropins;
        - name: "proxy.conf"
          contents: |
            [Service]
            Environment="ALL_PROXY=<REDACTED>"

Impact
Machines behind a proxy aren't updating anymore.
Environment and steps to reproduce

Set-up: A Machine that does not have access to the internet directly
Task: Let locksmith try to update to the latest stable version
Error: observe the following message;

E0119 17:56:40.732053  1189 payload_processor.cc:487] prefs->GetInt64(kPrefsUpdateStateNextOperation, &next_operation) && next_operation != kUpdateStateOperationInvalid && next_operation > 0 failed.
E0119 17:58:48.966884  1189 postinstall_runner_action.cc:94] Postinst command failed with code: 1
E0119 17:58:48.974634  1189 update_attempter.cc:615] Update failed.
E0119 17:58:49.812515  1189 omaha_request_action.cc:626] HTTP reported success but Omaha reports an error.

Expected behavior
Get an updated machine on latest version of flatcar.
Additional information
We've tried to understand what happened, and we believe it's due to some change of the curl library being removed and another fetcher being used in the flatcar-postinst script.  The reason we believe so is that we were able to test the download which seems to be successful, but something fail after. We believe it might be linked to the following change ;
https://github.com/flatcar/update_engine/commit/43bb354c67c513ace66d414d9d4aec761eb24680
We'll keep on posting if we ever find the issue for sure.  Please let us know if you require anything else from us.

I'm no rust expert, and a library may blind-sight me, but I'm not seeing anything in there to handle proxy cases ;
https://github.com/flatcar/ue-rs/blob/trunk/src/bin/download_sysext.rs#L361

Correct, thanks for the bug report and testing Alpha! This case slipped through in the migration. Indeed we should add code to support the ALL_PROXY env var.

Thanks a lot for the confirmation @pothos ,  really appreciate the quickness !
May I suggest to look into supporting more than just the "ALL_PROXY" ? curl and wget have evolved to support many other variables (e.g. SOCKS_PROXY, NO_PROXY, HTTPS_PROXY, etc.).  As curl was used previously, I believe the customer base would expect the behaviour to be maintained, so preferable to get curl's quirks than other binaries.
I'm just saying this to avoid other customers to have problem as well but having a different implementation then us.  By the way, we did test most of those variables in trying to fix our issue. :)

By the way, we're not testing "alpha".  We're on "stable".  I don't think it changes anything, but I wanted the fact to be straight.

The ue-rs download_sysext helper also supports the needed proxy env vars when invoked directly.

Filed a PR and will test when the image build is ready: https://github.com/flatcar/update_engine/pull/37 https://github.com/flatcar/scripts/pull/1599

The fix is merged and will land in the next round of releases for all channels

FYI, the fix is required in the OS that does the update. If you don't have connectivity without the proxy, you can do an airgapped update with sudo flatcar-update -P … -E … where you pass the downloaded files from https://update.release.flatcar-linux.net/amd64-usr/

Report from the other thread:
I found one more problem on the way: Even with a newer flatcar-update (wget https://raw.githubusercontent.com/flatcar/init/flatcar-master/bin/flatcar-update) it won't work for old Stable or current LTS because the update_engine client doesn't hand the XML over.
A solution could be to jump through a release that doesn't require OEM payloads but will bring the proxy patch to unlock updating regularly. The next LTS bugfix release should work for that (Once available: sudo flatcar-update -V 3510.3.2).

Workaround is rolling out for Stable, Beta, Alpha - hope all is good now.

We were able to see the nodes update.  Some will update tomorrow, so I'll
close the ticket when we've done them all.
Regards,
Éric Lafontaine
On Wed, Feb 14, 2024 at 5:16 AM Kai Lüke @.***> wrote:

Workaround is rolling out for Stable, Beta, Alpha - hope all is good now.
—
Reply to this email directly, view it on GitHub
https://github.com/flatcar/Flatcar/issues/1326#issuecomment-1943456168,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXCYVDTQ4B43NWEAHRSBITYTSFJPAVCNFSM6AAAAABCCRKZU2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNBTGQ2TMMJWHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


which method is appropriate for writing GUI for steps

Hi
I need to write gui for new steps in webspoon and I know the default method is using the swt, but I want to know whether is it possible to use swing instead of swt ?
if yes, could you please tell me how , if no what is another option.
thanks for your consideration.

I haven't tried to use Swing in Spoon or in webSpoon before, but theoretically Swing should work in webSpoon.
You should try your Swing UIs in Spoon first (This doc may help you doing that), then you can try these UIs in webSpoon.
If they work in Spoon but they do not in webSpoon, please let me know. I'm happy to help you.
Having said, I'd recommend against using Swing in webSpoon due to

not common in Spoon hence not so many example code
a time-lag concern: when you open a dialog that has Swing UIs in webSpoon, you might feel some lag until the dialog to pop up.

Swing is written purely in Java unlike SWT. One of the main differences between Swing and SWT is where UI is rendered. In Swing UIs are rendered by Java, while in SWT they are rendered by native OS libraries (that's why you have different SWT jar for Windows, Mac OS X, Linux).
webSpoon uses RWT in place of SWT. RWT is a nearly-SWT-compatible UI widget for web app, meaning it takes a server-client architecture. RWT UIs are rendered in the client side in HTML and CSS based on the (meta)data sent from the server side.
I think that Swing UIs in webSpoon are rendered by Java in the server side, the rendered image is transferred via HTTP to the client side. You can imagine the data size of image is bigger than that of metadata. This would result in the time-lag when opening the dialog that has Swing UIs. I don't know how severe the impact would be. It might be negligibly small.

thanks for your answer and your consideration.


Third-party MP4Box.exe crashes when converting an MKV to MP4 with

When converting an MKV encoded by StaxRip (X265) and muxing m4a audio (if that matters), MP4Box.exe crashes. The Windows "Do you want to send more information..." dialog mentions the following files (attached) plus a dump file.
WER8556.tmp.appcompat.txt
StaxRip x64 <IP_ADDRESS> stable release shows:
Error Muxing using MP4Box 0.7.2-DEV-rev79 x64

Muxing using MP4Box 0.7.2-DEV-rev79 x64 failed with exit code: 255 (0xFF)

The exit code might be a system error code: The extended attributes are inconsistent.

The exit code might be a system error code: The extended attributes are inconsistent.

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
          Muxing using MP4Box 0.7.2-DEV-rev79 x64
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

C:\StaxRip\Apps\MP4Box\MP4Box.exe -add C:\Encode\IMG_5085_temp\IMG_5085_newa_out.hevc#video -add "C:\Encode\IMG_5085_temp\IMG_5085 ID1 Core Media Audio.m4a#audio:lang=eng:name=" -itags encoder="StaxRip <IP_ADDRESS>" -new C:\Encode\IMG_5085_newa.mp4

HEVC import - frame size 1536 x 864 at 29.970 FPS

StaxRip.ErrorAbortException: Muxing using MP4Box 0.7.2-DEV-rev79 x64 failed with exit code: 255 (0xFF)

The exit code might be a system error code: The extended attributes are inconsistent.

The exit code might be a system error code: The extended attributes are inconsistent.

-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
          Muxing using MP4Box 0.7.2-DEV-rev79 x64
-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-

C:\StaxRip\Apps\MP4Box\MP4Box.exe -add C:\Encode\IMG_5085_temp\IMG_5085_newa_out.hevc#video -add "C:\Encode\IMG_5085_temp\IMG_5085 ID1 Core Media Audio.m4a#audio:lang=eng:name=" -itags encoder="StaxRip <IP_ADDRESS>" -new C:\Encode\IMG_5085_newa.mp4

HEVC import - frame size 1536 x 864 at 29.970 FPS

   at StaxRip.Proc.Start() in D:\Projekte\VS\VB\StaxRip\General\Proc.vb:line 334
   at StaxRip.MP4Muxer.Mux() in D:\Projekte\VS\VB\StaxRip\General\Muxer.vb:line 294
   at StaxRip.GlobalClass.ProcessJob() in D:\Projekte\VS\VB\StaxRip\General\GlobalClass.vb:line 142

I'll need a full log file for it, ideally from build <IP_ADDRESS>.

No problem!
I'll see if I can build and use StaxRip <IP_ADDRESS>, which I assume is the current master, but for the time being here are two logs:
IMG_5085_newa_staxrip.log.txt is from the executable crash mentioned above
IMG_5091_new2_staxrip.log-recent-error-but-no-crash.txt is from a more recent attempt with a different video where the conversion did not succeed, but the MP4Box.exe process did not crash, at least not in a way that made Windows generate a dump.
IMG_5085_newa_staxrip.log.txt
IMG_5091_new2_staxrip.log-recent-error-but-no-crash.txt

Which audio and video encoders where used for encoding of the source?
Maybe you can upload a sample clip.


Update installing-core-osx.rst to use the latest runtime in usage examples and fix typo

Updated usage to use the latest version of the runtime and removed the '.' from the run command.

Corresponds to issue #48

Will resubmit when beta 8 comes out


Hubot not responding to ping on OSX

Installed hubot but it's not responding to Pinging on OSX. Message I get when I type "bin/hubot":
bin/hubot botname> [Sun May 22 2016 00:24:06 GMT-0400 (EDT)] WARNING Loading scripts from hubot-scripts.json is deprecated and will be removed in 3.0 (https://github.com/github/hubot-scripts/issues/1113) in favor of packages for each script.
Your hubot-scripts.json is empty, so you just need to remove it. [Sun May 22 2016 00:24:06 GMT-0400 (EDT)] ERROR hubot-heroku-alive included, but missing HUBOT_HEROKU_KEEPALIVE_URL. heroku config:set HUBOT_HEROKU_KEEPALIVE_URL=$(heroku apps:info -s  | grep web-url | cut -d= -f2) [Sun May 22 2016 00:24:06 GMT-0400 (EDT)] INFO hubot-redis-brain: Using default redis on localhost:6379

Message when I type "hubot help":
Hubot> [Sun May 22 2016 01:09:19 GMT-0400 (EDT)] WARNING Loading scripts from hubot-scripts.json is deprecated and will be removed in 3.0 (https://github.com/github/hubot-scripts/issues/1113) in favor of packages for each script.
The following scripts have known replacements. Follow the link for installation instructions, then remove it from hubot-scripts.json:
The following scripts don't have (known) replacements. You can try searching https://www.npmjs.com/ or http://github.com/search or your favorite search engine. You can copy the script into your local scripts directory, or consider creating a new package to maintain yourself. If you find a replacement or create a package yourself, please post on https://github.com/github/hubot-scripts/issues/1641:

hubot-shipit

You an also try updating hubot-scripts to get the latest list of replacements: npm install --save hubot-scripts@latest
[Sun May 22 2016 01:09:19 GMT-0400 (EDT)] ERROR Error loading scripts from npm package - Error: Cannot find module 'hubot-diagnostics'
at Function.Module._resolveFilename (module.js:440:15)
at Function.Module._load (module.js:388:25)
at Module.require (module.js:468:17)
at require (internal/module.js:20:19)
at Robot.loadExternalScripts (/usr/local/lib/node_modules/hubot/src/robot.coffee:401:11)
at /usr/local/lib/node_modules/hubot/bin/hubot:160:26
at FSReqWrap.readFileAfterClose [as oncomplete] (fs.js:439:3)

@bharadwajh sorry for the lack of response here. Hubot was unmaintained for a while, but we're back now. I assume you've moved on from this, but let me know if you're still having issues and want help working through it.


Timeevolution

Coverage decreased (-0.04%) to 98.08% when pulling 5aad429ca17dfb27190abe276be04f21fb4f11be on timeevolution into f0f4a20f308287241dcc1a5c3c8c6ec236699644 on master.

Coverage increased (+0.02%) to 98.127% when pulling 065b0496cf702811d8fb34f302bdcaef5a084399 on timeevolution into 47bd0fc05738670d1504825f934bb7f82beabb36 on master.