Plugin manager doesn't handle case where charm deployed with config options set

The charm should handle from the get-go the possibility to deploy the charm with config options set and without the opensearch service being up.

@Mehdi-Bendriss indeed, I've opened a PR to shuffle the checks on plugin manager.

Doing some digging @Mehdi-Bendriss
Back in the day, the decision to wait for the OpenSearch cluster was because we needed: (1) to cover the case some plugins manage things via API calls; (2) to know the opensearch version; and (3) we use the /_cluster/_settings to load the default settings. So, we needed the cluster up and running.
I am breaking this up into:

Any plugin that needs to manage things via API call should check the health of the cluster using check_plugin_manager_health
Moving opensearch_distro.version to load the workload_version file we have present instead of an API call
We will waive the need of loading the default settings if this particular unit is powered down: which makes sense, in this moment we can do any config changes as we will eventually powered it back up later

That still frees the config_changed to just call plugin_manager.run() before everything is set, as the run() method changes hard configuration only.

That is going to demand some attention on OpenSearchKeystore. We have now to account to the case where the keystore was not yet created, because the 1st start is not finished. In this case, we need to save the keystore password for later and pass it to the opensearch at startup time instead of leaving the opensearch to manage that.

Whilst I agree we should deal with plugin_manager configuration independent of the cluster being ready or not, I did quite some digging into this issue and I've found out we are stuck on an endless loop of: opensearch-peers-changed hook issued > calls deferred config-changed > changes the content of peer databag > gets deferred > retriggers a new peers-changed

Now, this is caused by a change that happens within opensearch_peers_relation_changed, at the deferred config-changed, the peer databag starts with:
'deployment-description': '{"config": {"cluster_name": "backup-test", "init_hold": false, "roles": ["cluster_manager"], "data_temperature": null}, "start": "start-with-provided-roles", "pending_directives": [], "typ": "main-orchestrator", "app": "main", "state": {"value": "active", "message": ""}, "promotion_time":<PHONE_NUMBER>.931757}'

And finishes with:
'deployment-description': '{"config": {"cluster_name": "backup-test", "init_hold": false, "roles": ["cluster_manager"], "data_temperature": null}, "start": "start-with-provided-roles", "pending_directives": [], "typ": "main-orchestrator", "app": "main", "state": {"value": "active", "message": ""}, "promotion_time":<PHONE_NUMBER>.7171}'

That is caused by this: https://github.com/canonical/opensearch-operator/blob/46bc4a8f46905228a576c97d29de1572dc141bdd/lib/charms/opensearch/v0/models.py#L200
Which wrongly resets the promotion_time.
Full stack trace:
  /var/lib/juju/agents/unit-main-0/charm/src/charm.py(264)<module>()
-> main(OpenSearchOperatorCharm)
  /var/lib/juju/agents/unit-main-0/charm/venv/ops/main.py(544)main()
-> manager.run()
  /var/lib/juju/agents/unit-main-0/charm/venv/ops/main.py(520)run()
-> self._emit()
  /var/lib/juju/agents/unit-main-0/charm/venv/ops/main.py(506)_emit()
-> self.framework.reemit()
  /var/lib/juju/agents/unit-main-0/charm/venv/ops/framework.py(859)reemit()
-> self._reemit()
  /var/lib/juju/agents/unit-main-0/charm/venv/ops/framework.py(939)_reemit()
-> custom_handler(event)
  /var/lib/juju/agents/unit-main-0/charm/lib/charms/opensearch/v0/opensearch_base_charm.py(617)_on_config_changed()
-> previous_deployment_desc = self.opensearch_peer_cm.deployment_desc()
  /var/lib/juju/agents/unit-main-0/charm/lib/charms/opensearch/v0/opensearch_peer_clusters.py(323)deployment_desc()
-> return DeploymentDescription.from_dict(current_deployment_desc)
  /var/lib/juju/agents/unit-main-0/charm/lib/charms/opensearch/v0/models.py(39)from_dict()
-> return cls(**input_dict)
  /var/lib/juju/agents/unit-main-0/charm/venv/pydantic/main.py(339)__init__()
-> values, fields_set, validation_error = validate_model(__pydantic_self__.__class__, data)
  /var/lib/juju/agents/unit-main-0/charm/venv/pydantic/main.py(1100)validate_model()
-> values = validator(cls_, values)
> /var/lib/juju/agents/unit-main-0/charm/lib/charms/opensearch/v0/models.py(202)set_promotion_time()
-> if values["typ"] == DeploymentType.MAIN_ORCHESTRATOR:


[UX][Publisher] Users tend to miss adding the resource path before implementation step

Description:
Users tend to miss on clicking add button before going to implementation stage.

Suggestion:

Suggested Labels:
UX Improvement
Affected Product Version:
2.5.0

This has been fixed in latest versions of APIM.


Tablib 3.7.0 support?

tablib 3.7.0 has been released a few days ago. Shall we update our dependency to use it?

Yes, now would be a good time to update because release 4.2 will be ready soon.
Thanks

Ah I see that the requirement has already been updated in https://github.com/django-import-export/django-import-export/pull/1923 so no worries.


Continuity is not active (MBP early 2011, OS X 10.11.2, Trust bt4 dongle)

I have installed CAT 2.2.3 and when I reboot the MBP the following message appears:
"ERROR: Continuity is not active.
Continuity is no active. This might be due to an system update. Do you want to open Continuity Tool to reenable it?" "Dismiss?" or "Open Continuity Activation Tool"
Whatever option I choose, it does not solve the problem.
. Handoff is not working
. Airplay and airdrop are working fine
Does anyone have a suggestion? thanks

Whats the result of running CAT and selecting option 2 (System Diagnostics) ? Also please run "ioreg -l | grep LMPVersion" and post the results.


Decode transaction messages because they may have HTML Entities

@aldo-expensify please review
Details
We were original showing the text here, but now that the IOU message may contain encoded HTML entities after this PR: https://github.com/Expensify/Web-Expensify/pull/36598, we should be decoding them in the front-end.
Fixed Issues

$ https://github.com/Expensify/App/issues/15576
Tests/QA Steps

Open the App
Create new money request with currency without special symbol, but with abbreviation only: UAH, AFN, AMD etc
check IOU details
Make sure that there is no nbsp; or anything like that in the messages for the transactions

Offline tests

N/A

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@aimane-chnaif @thienlnam One of you needs to copy/paste the Reviewer Checklist from here into a new comment on this PR and complete it. If you have the K2 extension, you can simply click: [this button]

@aimane-chnaif are you around to the reviewer checklist?

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[ ] MacOS / Chrome / Safari
[ ] MacOS / Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is correct English and approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@thienlnam do you want to divide the platforms? I can give you a hand with collecting the screenshots

@aldo-expensify If you could please help with android and desktop that would be amazing!

Provided desktop, ios native, android native

Team work makes the dream work baby

Oh I left a message in the issue, this was on purpose because a contributor was fixing the front-end here https://github.com/Expensify/App/pull/15340. We warned a few bug reporters this morning about this but we didn't warn Applause, that was my fault 🤦♀

🚀 Cherry-picked to staging by https://github.com/aldo-expensify in version: 1.2.77-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

@yuwenmemon @aldo-expensify I think this will cause a regression.
Ultimately the best fix here is to do nothing and decode the text in the backend, there is a conversation about this here. Due to test complexity I think we won't fix completely it in the backend (at least not for now).

Anyway, the regression I think of is if the message is: hi

Current behaviour: shows hi
After this PR: shows hi

I don't think this is intended, is it? If not then the fix would be:
Str.htmlDecode(new ExpensiMark().htmlToText(this.props.action.message[0].html));

Or we can just Str.htmlDecode(this.props.action.message[0].text) This is wrong because we are decoding a plaintext, but that plaintext is not pure plaintext (it contains html entities due to a backend bug as mentioned ^)

Tyler added a decode in the backend here: https://github.com/Expensify/Web-Expensify/pull/36642 (not deployed yet)
Would be good to check if maybe that made this PR unnecessary.

@aldo-expensify I think it's added only for notifications, no?

@aldo-expensify I think it's added only for notifications, no?

Hmm no, it was added to a function that converts the html to text for all cases I think.

This was for all IOU cases, not only notifications, Tyler's PR doesn't cover IOU messages. I think our code was really messy in different parts and it wasn't consistent. Some actions would decode, other ones wouldn't, some front-end would encore, other parts wouldn't...

ah cool, then it should cover IOU message as well, I don't think there is anything special about those. Let's wait for the PR to be deployed and test again, maybe this PR is not required after all

🚀 Deployed to production by https://github.com/yuwenmemon in version: 1.2.77-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


let people choose domain-prefix by themselves

hi developers, may I ask you to let people choose domain-prefix by themselves ?
just like Scihub X Now

and maybe it would be better to let people manage url prefixes by themselves, then they could be able to add urls such as：

https://standard--template--construct-org.ipns.dweb.link/#/nexus_science/doi:
https://standard--template--construct-org.ipns.dweb.link/#/nexus_books/cid:
https://zh.annas-archive.org/doi
https://zh.annas-archive.org/md5
https://sci-hub.se/

thanks~

I don't have plans to support anything besides STC in this extension. As for nexus_books by CID, I don't see a compelling use case.

Thanks for replying, however, you don't see any useful case, then you imagine it's nonexistent. Maybe it's better to expand your perception over STC.

I am working on integration with nexus using summa, which is not something that scihub x now will support.
I'm not against sites other than STC, they are not within the scope of this extension.

Anyway, happy to see this extension will be used by many people, its useful. Thanks your work again.


[BottomNavigation] Allow null child

Prior to this commit, the BottomNavigation component would
fail to render if it contains a null child:
TypeError: Cannot read property 'props' of null

This is because the component tries to 'preprocess' every
child, and expects every child to have props (see preparedChildren).
There are however, cases where the child is null, and an error will
be thrown in such cases.
It may be null in cases where a user would like a navigation item
to be 'hidden' if certain conditions are not fulfilled (rather
than being disabled).
This bug is not a problem in the v1-beta branch, but the fix
has not been applied to master (as of this commit, v0.19.x).

Closes #8859

@MrSaints Thanks!


Please provide an example of all JSON result files

It would be useful to show the structure of the JSON output from these tests, so people would know how to interpret them.

+100 to this

can't agree more

Does /bin/create_charts.py help?


Clickhouse Approx/Distinct Counts

User using this package will be able to use Clickhouse functions such has uniq, uniqExact, uniqTheta, uniqCombined, UniqCombined64 & uniqHLL12 to get Approx/Exact counts, this will be a optimized alternative for COUNT(DISTINCT var)
Ref : https://clickhouse.com/docs/en/sql-reference/aggregate-functions/reference/uniq

Thanks. Could you add some basic tests for these functions？

Thanks. Could you add some basic tests for these functions？

Welcome. I have updated my PR with required basic test cases.

Hi, @jayvynl can you please re-start the build job?

Hi, @jayvynl can you please re-start the build job?

Sure!
I think I'd better add a contributing guide.

Hi, @jayvynl can you please re-start the build job?

Sure!
I think I'd better add a contributing guide.

Yes, that will be great! @jayvynl

Can you merge the PR?
@jayvynl

Hey, observed that coverage dropped by 0.2% in last build. In commit (https://github.com/jayvynl/django-clickhouse-backend/pull/66/commits/d99dadf6eb3c4194d84c33609dfa3a8fdf442f4e) I have improved AggregatesTestCase. I request you to please re-start the build @jayvynl

Hey, observed that coverage dropped by 0.2% in last build. In commit (https://github.com/jayvynl/django-clickhouse-backend/pull/66/commits/d99dadf6eb3c4194d84c33609dfa3a8fdf442f4e) I have improved AggregatesTestCase. I request you to please re-start the build @jayvynl

Great.


I1 contact matrix

Closes #1.
Create contact matrix class using as guidelines:

for content: https://github.com/Como-DTC-Collaboration/comoDTC/blob/i19-contact-matrices/R/get_contact_matrix.R
for style: https://github.com/SABS-R3-Epidemiology/branchpro/tree/main/branchpro

Codecov Report

Merging #4 (35443ef) into main (096898d) will increase coverage by 95.65%.
The diff coverage is 95.34%.

@@            Coverage Diff             @@
##            main       #4       +/-   ##
==========================================
+ Coverage   0.00%   95.65%   +95.65%
==========================================
  Files          2        3        +1
  Lines          4       46       +42
==========================================
+ Hits           0       44       +44
+ Misses         4        2        -2

Impacted Files
Coverage Δ

epimodels/version_info.py
100.00% <ø> (+100.00%)
:arrow_up:

epimodels/_contact_matrix.py
95.12% <95.12%> (ø)

epimodels/__init__.py
100.00% <100.00%> (+100.00%)
: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 096898d...35443ef. Read the comment docs.


Bug: Game doesn't sync with decimal BPM songs

Describe your bug here.
I am making a 8-bit song called Challenger with a bpm of 112.5 or in Pico-8, speed 16. While charting this song, I noticed the song was desyncing with the notes. I used Pico-8 to make the song and charted the song based on the source song file.
The song file and the chart are in the zip file:
Challenger.zip
Command Prompt/Terminal logs (if existing)
No response
Are you modding a build from source or with Lua?

[ ] Lua
[X] Source

What is your build target?

[X] Windows
[ ] Linux
[ ] Mac
[ ] HTML5
[ ] Flash/Air-based target
[ ] Neko, HashLink, or other build system

Did you edit anything in this build? If so, mention or summarize your changes.
I added winning icons and note movement stuff.

It randomly desyncs mid-song

Lol I don't have the correct bpm :/


Only discover remote services if we need them.

At the moment remote service discovery is always triggered when a central device connects.
However nothing will actually happen in the BLE_GATTC_EVT_PRIM_SRVC_DISC_RSP logic if _numRemoteServices is zero:
case BLE_GATTC_EVT_PRIM_SRVC_DISC_RSP:
    ...
    for (int j = 0; j < this->_numRemoteServices; j++) {

_numRemoteServices is only non-zero if the end user called BLEPeripheral::addRemoteAttribute(...) at least once with a remote attribute of type BLETypeService.
So I've added a one line check to just call sd_ble_gattc_primary_services_discover(...) if _numRemoteServices is greater than zero.

@george-hawkins awesome thanks!


Don't present Withdrawn notice

The withdrawn notice is set as a result of a call to the /unpublish endpoint so it is no longer necessary to present it with the content item.

Want to include the republish to remove from published items too?

@fofr I've added a data migration to republish case studies. The other migrated formats 'should' have already updated the content store correctly and the remaining formats will be republished over the next few weeks.

@bevanloon Yes... I think it does. I'll open another PR for that so we can get this merged and unblock @fofr .


[Feature]: Add default value for axes argument in add_missing_bounds()

Is your feature request related to a problem?
I think from v0.6.0 add_missing_bounds started requiring parameters.
I liked its previous usage, e.g., ds.bounds.add_missing_bounds(), which automatically loops through the Dataset’s axes and attempts to adds bounds to its coordinates if they don’t exist. (By the way, the documentation is still written in this way).
Describe the solution you'd like
I wonder if add_missing_bounds can be operated in two optional ways:
(1) ds.bounds.add_missing_bounds(): loop through axes and fill missed bounds
(2) ds.bounds.add_missing_bounds("X"): fill missed bounds for "X" axis
Describe alternatives you've considered
No response
Additional context
(added to the post)
Some PMP  and obs4MIP codes have implemented add_missing_bounds() routines but with xcdat update those codes needs changes, and if ds.bounds.add_missing_bounds() could be still used, it is going to be very efficient to update xcdat version in PMP dependency.
@acordonez, @gleckler1, @bosup any thoughts?

Hey @lee1043, sorry for this issue. In v0.6.0 add_missing_bounds() was updated to include the axes argument to align with the changes to open_dataset() and open_mfdataset(). Both of these APIs accept a list of axes (e.g., add_bounds = ["X", "Y"]), which get passed to add_missing_bounds() under the hood.
To avoid this breaking change, I should have set a default value similar to open_dataset() and open_mfdataset(). I do think it is a good idea for the user to explicitly define which axes they want missing bounds added, because not all users might want to add missing bounds for all axes if they don't want or need to.
In your case, if you want to add bounds for all axes, you can do this:
# Constant variable to add missing bounds to.
AXES_BNDS = ["X", "Y", "T", "Z"]

# Update all of these lines to this:
add_missing_bounds(AXES_BNDS)
```.

Is it possible to add a default positional argument if the user doesn't pass in an argument? I think it would make sense to default this to ["T", "X", "Y"] (X and Y should have been added on open, so if a user is calling this, they likely want time bounds; we shouldn't add vertical bounds because this is a rarer case).

Is it possible to add a default positional argument if the user doesn't pass in an argument? I think it would make sense to default this to ["T", "X", "Y"] (X and Y should have been added on open, so if a user is calling this, they likely want time bounds; we shouldn't add vertical bounds because this is a rarer case).

I actually thought about this over the weekend too. This sounds reasonable to me.
It just involves adding a default value and fixing a few tests. If somebody has time to open up a PR before me that would be great! Otherwise I'll get to it soon.

This might involve updating this line to:
def add_missing_bounds(self, axes: Optional[List[CFAxisKey]] = ['X', 'Y', 'T']) -> xr.Dataset:  # noqa: C901

[I didn't test this, but I think it is on the right track. Tom notes we'd need to fix tests. Maybe that noqa statement would need an update, too].
@lee1043 – would this be sufficient top address this issue? Does Z need to be included (I think you'd get bounds, but it would be unclear if they are the bounds you want)?
If this works – would you be willing to give this fix a shot in a PR (@acordonez, @gleckler1, @bosup)?

I got rid of my other errors using:
    def add_missing_bounds(  # noqa: C901
        self, axes: List[CFAxisKey] = ["X", "Y", "T"]  # noqa: C901
    ) -> xr.Dataset:  # noqa: C901

I'm getting some failures when I run pytest but how do I tell what the failures actually are?

@acordonez – thanks for working on this – I don't see failures (usually red X marks, I think) in that screenshot, but if there are failures I think the last couple lines should point to a htmlcov folder (or something like that), which you can open in your web browser and it highlights the failed lines in red. Reach out on if it would be helpful to chat via WebEx.

@pochedls Ok thanks it sounds like I was misreading the test outputs then.

I'm getting some failures when I run pytest but how do I tell what the failures actually are?

The test results marked with a lower-case "x" denotes "xfail" (expected to fail), so no worries here. The console should also output the coverage report, in addition to the htmlcov/index.html file.


Ensure SSH access and LB backend pool for Master Node replacement

Which issue this PR addresses:
Fixes: https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/14539058/
Fixes: https://msazure.visualstudio.com/AzureRedHatOpenShift/_workitems/edit/16195191/
What this PR does / why we need it:
This PR will fix SSH to the new node did not work after the master node replacement  and a PUCM was performed to reconcile the NIC and restore ssh access (as PUCM will call fixssh to do that reconciliation). Also this PR  fixes the issue whether the new NIC  is not being added to Internal LoadBalancer API Backend Address Pool after the master is replaced even after running the Admin Update or PUCM.
Test plan for issue:
Added unit tests. Also tested manually by replacing the master nodes in test clusters.
<!--
How did you test that this PR works?

Added new Go Mock Testcases.

### Is there any documentation that needs to be updated for this PR?

https://msazure.visualstudio.com/AzureRedHatOpenShift/_wiki/wikis/ARO.wiki/253699/Replace-a-Master-Node-VM

/azp run e2e

/azp run ci


Syntax error related to bug reported on community.

https://github.com/vanderbilt-redcap/fhir-services-module/blob/a07d5f338abab96e1597ed792af54809e5735495/SchemaParser.php#L306
https://community.projectredcap.org/questions/122411/error-when-enabling-fhir-services-module-redcap-v1.html?childToView=122861#comment-122861

This has been fixed in version 2.0.4.  Thanks very much for identifying this @dr01d3r!


Running in virtual machine supported?

I'm trying to test the library under linux, but whenever I run the program that works without issues in Windows I end up with this kind of error saying that the gpu-process crashes
Initially I had a libX11.so not found, so I installed libx11-dev, but the segfault remained
pi@pi-VirtualBox:~/Downloads/linux-x64$ ./crossplatdemo
2020-03-07 15:54:52.235 [INFO] Running Linux chromium 77.0.3865.120
On CefGlue child process launch arguments:
/proc/self/exe --type=gpu-process --field-trial-handle=8883176290764582883,17956091608203660837,131072 --disable-features=OutOfBlinkCors --no-sandbox --locales-dir-path=/home/pi/Downloads/linux-x64/locales --log-file=logs\chromely.cef_20200307.log --log-severity=info --resources-dir-path=/home/pi/Downloads/linux-x64/ --lang=en-US --default-encoding=utf-8 --allow-file-access-from-files --allow-universal-access-from-files --disable-web-security --ignore-certificate-errors
2020-03-07 15:54:56.748 [INFO] Running Linux chromium 77.0.3865.120

DevTools listening on ws://<IP_ADDRESS>:20480/devtools/browser/d89227bb-7026-4581-a6ee-64c402dcb411
[0307/155457.575608:ERROR:sandbox_linux.cc(369)] InitializeSandbox() called with multiple threads in process gpu-process.
On CefGlue child process launch arguments:
/proc/self/exe --type=utility --field-trial-handle=8883176290764582883,17956091608203660837,131072 --disable-features=OutOfBlinkCors --lang=en-US --service-sandbox-type=network --no-sandbox --locales-dir-path=/home/pi/Downloads/linux-x64/locales --log-file=logs\chromely.cef_20200307.log --log-severity=info --resources-dir-path=/home/pi/Downloads/linux-x64/ --lang=en-US --default-encoding=utf-8 --allow-file-access-from-files --allow-universal-access-from-files --disable-web-security --ignore-certificate-errors
2020-03-07 15:54:58.365 [INFO] Running Linux chromium 77.0.3865.120
2020-03-07 15:54:58.429 [WARNING] Request Code: 1	Request Type: X_CreateWindow	X11 Error: BadRequest (invalid request code or no such operation)
Segmentation fault
2020-03-07 15:54:58.514 [INFO] Sub process executes successfully with code: 0

is it possible it's due to the fact that I'm running it in a virtual machine?

@SirePi yes, you can run Chromely in VMs. I have used VMs and DigitalOcean droplets to test Chromely apps.
I won't go as far as saying we support Chromely on VMs because that could mean a whole lot of things. There are variations of distros out there and different ways to access them via VMs.
Yes, there is a possibility that the issue you are seeing is VM related or missing libs. It is like X11 is failing to actually create a window for gtk.
CEF itself seems to be running as expected:
2020-03-07 15:54:58.514 [INFO] Sub process executes successfully with code: 0

What distro/version are you running? Personally I have tested on Linux - Lint Mint Xfce (Ubuntu 18.04.3 LTS), Linux Mint 19.3 Tricia, Ubuntu 19.10 (x64
You can try a fresh setup of your VM. Or try a different distro/version of Linux.
Please ensure that you have gtk3 libs installed.
Also check https://github.com/chromelyapps/Chromely/issues/163. @snairza may also be able to help as he recently had a closely related issue.

I'll try with an ubuntu.. I was trying it on ElementaryOS 5.1

Can confirm that on Ubuntu 18.04 it works properly after installing libx11-dev 👍

I have confirmed that 18.04 works, however I seem to have a problem in
16.04 where the Cromely browser starts up but no content is rendered - only
a white screen.
On Sun, Mar 8, 2020 at 1:47 PM SirePi<EMAIL_ADDRESS>wrote:

Can confirm that on Ubuntu 18.04 it works properly after installing
libx11-dev 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chromelyapps/Chromely/issues/167?email_source=notifications&email_token=AB2AE5XJBYZM2SQPNQ3NLZLRGOAV3A5CNFSM4LDQRUHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOET2RA#issuecomment-596196676,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AB2AE5Q7JXO6WOARGYZ4HSLRGOAV3ANCNFSM4LDQRUHA
.

@snairza Installed 16.04 - does not work at all
Installed updates AND libx11-dev
Now it works on my VirtualBox VM

I cloned the project again and can confirm it now works! Thanks guys!


Enable experimental windows build for OneAPI [Intel Discrete GPUs]

Hello, since the support for Intel build is already present in the main directory, I think it would be helpful for users to have some directions on building Ollama from source for Intel discrete graphics cards.
I am creating this pull request for enabling this. I have,

Corrected compiler type in gen_windows.ps1
Wrote build instructions for windows
Tested on Windows 11 with ARC A770 graphics card

Let me know if anyone can test this on their machine following build instructions as I have written. Also, if this gets merged, I will open a new pull request for Linux OneAPI build instructions.
Thanks,
Akash

@zhewang1-intc Let me know what you think of this. I know that you already have a branch for enabling support for iGPUs.

@dhiltgen Does this complement or repeat with your branch here #4876 ?

I have been attempting to get this working on windows.
the go generate ./.. returns
"skipping oneapi generation step"
Despite all my attempts and everything checking out properly from that i can tell at least, it will not seem to actually complete that step to compile the runners for oneapi, it just skips it every single time

Hey, I tried this a while ago on an older main branch. Let me pull the latest one and try again.
Will report back.

Hey, I tried this a while ago on an older main branch. Let me pull the latest one and try again.
Will report back.

Thank you I appreciate it
i think i am still on 0.3.10 or so, i forget which but i am a bit behind since i managed to get avx-512 and kv quantisation working i stopped updating since there were some api changes and other things that remove per model k/v configuration which is necessary for embedding models since they seem to break unless run on f16.
I would love to get this working, especially if somehow i could load embedding models to the intel gpu and chat models to nvidia or mix them to distribute and have more vram available.
I only have a A310 4GB but an extra 4gb of higher f16 performance vram would be great since i am only working with a tesla p4 8gb

Tested myself on Arc A750, the performance is way more better than vulkan backend.
There are some known issue while building proccess:

In windows, icx can't be recognize as clang based compiler automatically by ccache.
ggerganov/llama.cpp#9976
When oneapi env is active, cpu build will failed.
ggerganov/llama.cpp#9983

If both PR get merged into llama.cpp, build for oneapi will be less painful.

@AncientMystic disable ccache can solve your problem.


Remove Loggerithm as the repo has been archived

Repo has been archived
https://github.com/honghaoz/Loggerithm

@ricsantos Thank you.


DetailLevel String or Repeatable?

When I set $getItem->DetailLevel to a string I get:
Fatal error: Uncaught DTS\eBaySDK\Exceptions\InvalidPropertyTypeException: Invalid property type provided for DetailLevel. Expected DTS\eBaySDK\Types\RepeatableType but got string in vendor\dts\ebay-sdk-php\src\Types\BaseType.php:383.
When I set it to a RepeatableType I get:
Fatal error: Uncaught DTS\eBaySDK\Exceptions\InvalidPropertyTypeException: Invalid property type provided for DetailLevel. Expected string but got DTS\eBaySDK\Types\RepeatableType in vendor\dts\ebay-sdk-php\src\Types\BaseType.php:497
Please help!

What about showing some code or shall we guess what you are really doing here? 😏

Well that would make life too easy. You are just supposed to assume I am doing everything correctly 😜
`$requestGetItem->RequesterCredentials = new Types\CustomSecurityHeaderType();
$requestGetItem->RequesterCredentials->eBayAuthToken = $credentials['production']['authToken'];
$requestGetItem->IncludeItemCompatibilityList = true;
$requestGetItem->IncludeItemSpecifics = true;
$requestGetItem->DetailLevel = Enums\DetailLevelCodeType::C_RETURN_ALL;
$requestGetItem->ItemID = $item->ItemID;
$responseGetItem = $service->getItem($requestGetItem);`

what happens if you try with $requestGetItem->DetailLevel = ['ReturnAll']; ?

 Should all repeatables be passed as an array?
Thank you for your help!


"Decorated type deeper than introspection query" error with nested lists

We see the following errors when using nested lists:
Error: Decorated type deeper than introspection query.
    at n (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:23637)
    at n (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:23879)
    at n (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:23733)
    at n (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:23879)
    at r (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:24425)
    at http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:26609
    at http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:11:8480
    at Array.reduce (native)
    at keyValMap (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:11:8444)
    at f (http://cdn.jsdelivr.net/graphiql/0.7.1/graphiql.min.js:15:26484)

Here's the example code that can cause this issue. Graphene introspection doesn't report any errors.
import graphene, json

class Animal(graphene.Interface):
   id = graphene.String()

class Dog(graphene.ObjectType):
   class Meta:
      interfaces = (Animal, )
   breed = graphene.String()

class Cat(graphene.ObjectType):
   class Meta:
      interfaces = (Animal, )
   category = graphene.String()

class Query(graphene.ObjectType):
  animals = graphene.NonNull(graphene.List(graphene.NonNull(graphene.List(graphene.NonNull(Animal)))))
  def resolve_animals(self, args, context, info):
    return[[Dog(id=1, breed='Golden Retriever'),
            Dog(id=2, breed='German Shepard')],
           [Cat(id=11, category ='Tiger'),
            Cat(id=12, category='House Cat')]]

schema = graphene.Schema(query=Query, types=[Dog, Cat, ])

Errors disappear if we remove the "required" or NonNull decorators. Any suggestions on how to fix these errors?

issue appears to be due to graphiql version set to 0.7.1 in render_graphiql.py - I was able to fix the issue by changing this to 0.10.1 in my local copy.


Support custom component types via CLI extension

Describe the feature
currently the CLI extension is limited to consuming RethComponents which includes the regular txpool.
Ideally we support embedding custom txpool types (and eventually more, like CL Payload, etc..) via the extension trait.
this needs to be plugged into the RethNodeCommandConfig.
I think we another trait that defines types
like
trait RethComponents {
  TxPool: ....,
  Payload: ....,
  ....
}

then RethNodeCommandConfig becomes RethNodeCommandConfig<T: RethComponents>
and we it as another associated type to:
https://github.com/paradigmxyz/reth/blob/b93b909edeee439cdafab2adf77a060b3b4e8b53/bin/reth/src/cli/ext.rs#L25-L25
I kinda expected this to some degree, because we were already using the RethCliExt trait which only has a single associated type
we then can do something like:
pub trait RethCliExt {

    type Node: RethNodeCommandExt<Self::Components>;

    type Components: RethComponents;
}

trait RethComponents {
  type Txpool: TransactionPool + ...;

}

naming TBD.
Additional context
No response

Hi @mattsse I can take this.

I think this is the direction we want to go with, so I appreciate if you could start drafting the RethComponents trait.

Thought about this for a bit, and had a few notes, lmk if this makes sense:

If we have a custom Payload type, then it seems like we would need an abstraction that the PayloadBuilderHandle would implement. Otherwise PayloadBuilderHandle would still return the concrete BuiltPayload type instead of the custom type. Maybe instead the custom type should just be a PayloadBuilder trait, which has an associated Payload type.
Would it even make sense to impl RethNodeCommandConfig<T> for Struct, for multiple T types? If not then an associated type seems more appropriate


fix bug: corrigido o encoding para o tratamento de stderr

dos subprocessos da classe Convert.
close #35

Codecov Report

Merging #36 (7eb5a18) into master (3b42a6d) will not change coverage.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master     #36   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files           5       5
  Lines         511     512    +1
======================================
- Misses        511     512    +1

Impacted Files
Coverage Δ

conversor_divisor/convert.py
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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


test_trainer.py update

This PR enables some tests in test_trainer.py that were commented because the SDK was not supporting them before.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


navbar flashes mobile before isMobile is called

An issue on desktop where the the navbar flashes its mobile mode before the navbar is officially mounted. I believe this is a result of icluding the navbar in the persistent layout, per https://transitionlink.tylerbarnes.ca/docs/installation/ -- my suspicion is that persistent layouts are pre-rendered under conditions in which the mobile setting is true.
it's only a minor graphical issue but I would like to fix it soon. I can also see that it affects behaviors on other sites that use transitionlink https://www.hiddenwork.ca/

looks like a gatsby bug, though I don't think it's reported. i'll submit an issue to gatsby soon https://stackoverflow.com/questions/58608523/gatsby-react-conditional-rendering-based-on-window-innerwidth-misbehaving
ive hacked a fix with in https://github.com/mwhatters/solo-website/pull/10


Python version pinning

If I installed environment with the environment below (python is not pinned)
name: test_coiled
channels:
- conda-forge
dependencies:
- python
- coiled-runtime=0.0.3

Then python 3.10 is installed on my machine (MacOS). However, later coiled asks for python 3.9 version.
Steps to reproduce:

save the above yaml as environment.yml

conda env create -f environment.yml
conda activate test_coiled
python --version

The above prints 3.10.4.

To elaborate on the coiled error message:
from coiled.v2 import Cluster
cluster = Cluster(n_workers=1, account="sultanorazbayev")

I got the error ServerError: Software environment coiled/default-py310 not found
Specifying 3.9 using this works, but will give inconsistent python versions on client/workers:
cluster = Cluster(n_workers=1, account="sultanorazbayev", software="coiled/default-py39")

@SultanOrazbayev Thank you for reporting this!.
@jrbourbeau Should we pin the python version to be < 3.10 in the recipe? and probably patch the release in the feedstock?

@SultanOrazbayev we just fixed this in the feedstock, not sure how long it takes to kick in, but probably in a couple of hours, this shouldn't be a problem anymore.

Interesting, I just reproduced the same issue (python installed is still 3.10.4)... (it's not a big one, though, just wanted to let you know).

@SultanOrazbayev
Mmh I'm not able to reproduce what you say. I just upgrade my python to 3.10.4 on the base environment, then create a new environment installing the coiled-runtime and it installed python 3.9.12
My guess is you might have a cached version of the coiled-runtime package which is probably causing this.

That's very odd, I just tried it on another computer (Ubuntu) where I did not have coiled previously and still able to reproduce. I am using conda-forge channel, could that be an issue?

I tried adding coiled as a higher priority channel, but the original issue is still reproduced.
However, I just wanted to report it, it's not a blocking issue for me.

@SultanOrazbayev Thank you for reporting it.  You shouldn't use the coiled channel, as that is an older version of the runtime and we are not maintaining it, except for nightly builds.
conda-forge is the way to go, I'm curious why it's not working for you, @jrbourbeau do you know what could be happening here?

Mmh I'm not able to reproduce what you say ... create a new environment installing the coiled-runtime and it installed python 3.9.12

Same here. I tried with both conda and mamba and with both python=3.9.12 was installed.
@SultanOrazbayev can you paste the output of conda info? By chance, do you have any extra channels in your .condarc file?

Below is the content of conda info:
     active environment : None
       user config file : ~/.condarc
 populated config files : ~/.condarc
          conda version : 4.12.0
    conda-build version : not installed
         python version : 3.8.5.final.0
       virtual packages : __osx=10.16=0
                          __unix=0=0
                          __archspec=1=x86_64
       base environment : ~/miniconda  (writable)
      conda av data dir : ~/miniconda/etc/conda
  conda av metadata url : None
           channel URLs : https://repo.anaconda.com/pkgs/main/osx-64
                          https://repo.anaconda.com/pkgs/main/noarch
                          https://repo.anaconda.com/pkgs/r/osx-64
                          https://repo.anaconda.com/pkgs/r/noarch
          package cache : ~/miniconda/pkgs
                          ~/.conda/pkgs
       envs directories : ~/miniconda/envs
                          ~/.conda/envs
               platform : osx-64
             user-agent : conda/4.12.0 requests/2.27.1 CPython/3.8.5 Darwin/21.4.0 OSX/10.16
                UID:GID : <removed (not sure if needed)>
             netrc file : None
           offline mode : False

This is inside ~/.condarc (don't recall changing these options manually, so might be the defaults):
always_yes: true
quiet: true
changeps1: false

Not sure if this applies to conda, but does it use some regional mirrors? If not, then perhaps there's some caching done by my ISP...


Option to save last used source in incognito

Describe your suggested feature
I activate the incognito mode everytime I browse for new manga and I need to inspect them before saving in library, so the history doesn't get clogged with unsaved stuff.
The incognito mode though doesn't save the last used source, which is a bit uncomfortable.
I suggest adding the option to save the last used source in incognito mode. Can add this in the security settings.
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] I have updated the app to version 1.12.1.
[X] I will fill out all of the requested information in this form.

That's a weird way to use incognito mode. Would be confused.
Try pin it then?

This is the only useful use case for me with incognito, because I don't show my stuff to anyone around me, neither I am ashamed of that.
I already have many pinned sources which I use to multi search non-lewd content, and so if I want to pick the last source used I must scroll down or type it's name.
I believe someone else too uses Incognito mode to browse new manga and not clog the history up.

because I'm going to implement the filter there, with that you can have more than just pin


[New Domain] : <Gogoanime.ee>

What feature should be added to this source?
Gogoanime source has moved to the new domain gogoanime.ee, please add it in the source settings.

No worries it redirect to the new domain in the app

none of the attached domains work anymore. these two are the new ones.
gogoanime.tel
gogoanime.ar

Resolved by #1693


Fix configuration example

fontTargetDir and cssBundleDir are not arrays

You're absolutely correct. Thanks for the fix, much appreciated!


ORC-1432: Add MacOS 13 GitHub Action Job

This is a backport of #1511

This is resolved via https://github.com/apache/orc/commit/db5ecfcbc75537fcf45aa76996090c7216e51085


Improve auto scroll experience for chat messages

Auto scroll to new message should happen in the following scenarios

[x] Whenever user sends a message
[x] Whenever user receives a message and he is currently at the bottom of the message list

Auto scroll to new message should not happen in the following scenarios

[ ] User is reading old messages(i.e, is not scrolled to the bottom of chat window). In this case we can show an indicator which says X New messages which when clicked can scroll to the first unread message.

hello everyone i am intrested in contributing to this project so can anyone help me in setting up this project in my local machine as it is my first open source project.

Hi @loneWarrior581,

hello everyone i am intrested in contributing to this project so can anyone help me in setting up this project in my local machine as it is my first open source project.

Here are the instructions to setup the project:
https://github.com/muke1908/chat-e2ee#for-developers

@muke1908  Sir I have set up the project on my computer and facing issues on how to bypass the front page to reach the chat section as the private link is not able to generate. Iwould be very keen if I would be solving some issues to make my first contribution.

@loneWarrior581 Looks like you have not configured the google captcha. Please go through this https://github.com/muke1908/chat-e2ee/wiki/Captcha
Instead of this issue, you start picking an issue from good-first-issue

Help needed
@muke1908  sir I am getting an error while setting up it says : MONGO DB ERROR! although i have given the mongo db uri. And also what are the values to be given in the field CHAT_LINK_DOMAIN=' ' and MONGO_DB_NAME=' '. Also I am not able to find the location of IMAGE_BB_API_KEY to be given .

Help needed
@muke1908 sir I am getting an error while setting up it says : MONGO DB ERROR! although i have given the mongo db uri. And also what are the values to be given in the field CHAT_LINK_DOMAIN=' ' and MONGO_DB_NAME=' '. Also I am not able to find the location of IMAGE_BB_API_KEY to be given .

Hi @loneWarrior581
Get IMAGE_BB_API_KEY from https://api.imgbb.com/.
For mongodb, you have to create a database, that you have to assign to MONGO_DB_NAME.
What error are you getting anyway?

@muke1908  Sir I am not able to find the directory where the variable IMAGE_BB_API_KEY is located. And also how to make it possible to make instances so that they can communicate within the localhost for the testing purpose in our local machine

@muke1908 Sir I am not able to find the directory where the variable IMAGE_BB_API_KEY is located. And also how to make it possible to make instances so that they can communicate within the localhost for the testing purpose in our local machine

@loneWarrior581 IMAGE_BB_API_KEY is in './env` check sample
Run npm install, then run npm run dev.

@DarkSouL11 hey! Can I work on this?

@AudreyKj Yeah sure. @muke1908 I believe this is not picked by anyone yet

ok!

Hey, can anybody tell me how to approach this? if no one is working I'll give a try.

Hii.. Can I get assigned to this role?
I would love to contribute

Sure, go ahead.
On Thu, Sep 16, 2021 at 5:12 PM amreekminocha @.***>
wrote:

Hii.. Can I get assigned to this role?
I would love to contribute
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/muke1908/chat-e2ee/issues/77#issuecomment-920991239,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AE23SBJTX4IKXAW6EMHJZJ3UCICPFANCNFSM4QAEXHMQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

[ ] User is reading old messages(i.e, is not scrolled to the bottom of chat window). In this case we can show an indicator which says X New messages which when clicked can scroll to the first unread message.

I think I can fix it

Is anyone working on this? If not can I give a try?

@JasmikDobariya Sure, please go ahead!

i can fix scroll issue chack i am send pull request

@muke1908 i fixed scroll issue! check my pull request if any problem in my code send me mail

Hi, I met an issue, the "New Messages" pops up both when sending messages continuously and receiving messages. It’s quite annoying. Could you please check and optimize it?

Now cliking the ‘New messages’ will jumps to the bottom. Can someone improve it so it jumps to the first unread message?


Update translations.json with Italian translations

Description

Checklist

[✔] I have read and followed the Contributing Guide
[✔] I have read and accept the Code of Conduct (TLDR: just behave in a friendly manner)
[✔] My code follows the general style of this project (explained here)

Thank you a lot for submitting the translation :)


add 0.8.1 release notes

Current coverage is 70.94% (diff: 100%)

No coverage report found for master at 9063ae8.

Powered by Codecov. Last update 9063ae8...628d138

+1


test(module:card): added missing tests

PR Checklist
Please check if your PR fulfills the following requirements:

[✔] The commit message follows our guidelines: https://github.com/NG-ZORRO/ng-zorro-antd/blob/master/CONTRIBUTING.md#commit
[✔] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Application (the showcase website) / infrastructure changes
[✔] Other... Please describe: missing unit tests for the Card Component

What is the current behavior?
What is the new behavior?

Before
After

Statements
96.96%
100%

Branches
100%
100%

Functions
85.71%
100%

Lines
96.87%
100%

Does this PR introduce a breaking change?

[ ] Yes
[✔] No

Other information

@Laffery
thank you for checking my PR, your rrequested changes are made.

@Laffery
I have made the requested changes 👍


Bug using SNOBFIT in combination with UCCSD (VQE) / misleading function signature of optimizer SNOBFIT

When running a VQE algorithm with ansatz UCCSD the following error message shows up:
[...]
 File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit_nature\algorithms\pes_samplers\bopes_sampler.py", line 175, in sample
    self._raw_results = self._run_points(points)
  File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit_nature\algorithms\pes_samplers\bopes_sampler.py", line 201, in _run_points
    raw_result = self._run_single_point(point)  # dict of results
  File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit_nature\algorithms\pes_samplers\bopes_sampler.py", line 259, in _run_single_point
    result = self._state_solver.solve(self._problem, aux_ops_current_step)
  File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit_nature\algorithms\ground_state_solvers\ground_state_eigensolver.py", line 91, in solve
    raw_mes_result = self._solver.compute_minimum_eigenvalue(main_operator, aux_ops)  # type: ignore
  File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit\algorithms\minimum_eigen_solvers\vqe.py", line 530, in compute_minimum_eigenvalue
    opt_result = self.optimizer.minimize(
  File "C:\Users\poc\Anaconda3\envs\qiskit\lib\site-packages\qiskit\algorithms\optimizers\snobfit.py", line 96, in minimize
    if abs(theta) > bounds[idx][0]:
TypeError: '>' not supported between instances of 'float' and 'NoneType'

In case of ansatz EfficientSU2 the code runs fine, only with UCCSD I get the error.
I tracked down the error to be caused by missing parameter bounds. The function signature of the optimizer's minimize function (``) suggests that None is a valid value. But the code never handles None-type input (neither None nor the nested list/tuple with None values).
So, I propose one of the following changes:

Change the signature of the minimize method (still not easy to read for a user)
Catch the Nones in the input bounds and add a more verbose error message that optimizer SNOBFIT requires parameter bounds
Artificially add bounds for the optimizer. Basically the parameter can only be varied from 0 to 2pi so one could catch the None input and change it to these bounds. Though I'm not aware of the side effects this approach might have... Here it might be useful to ask also someone from qiskit nature whether it could be resolved within the UCCSD ansatz (though it would not solve the initial problem with the misleading signature/missing error message).

Any fixes (or suggestions therefore) are highly appreciated.

Thanks for reporting this!
If SNOBFIT requires bounds then I think the optimizer should raise an error if they are not passed. The interface is dictated by the Optimizer class so we can't really change that, and adding default bounds is very restrictive (e.g. [0,2pi] is not valid as soon as the circuit contains e.g. a gate like Ry(0.0001 * theta)).

@Cryoris Based on your comment I added a more concise error message in #8638
This resolves the problem at hand in the SNOBFIT code but actually there is still an open question: When I use ansatz EfficientSU2, all parameter bounds are set to (-pi, pi) allowing proper usage of optimizer SNOBFIT. On the other hand with ansatz UCCSD all parameter bounds are set to (None, None).
So, maybe someone with more insight into the ansatzes might have a look at it and can tell why those two ansatzes treat SNOBFIT so differently.


do a checkup on all documentation

review all documentation to check if there is no spelling issue, subject missing, or incomprehension

All the libs documentation, Install and Readme has been checked
#170

parser node documentation was updated in #137


S3 PutBucketPolicy - Unable to Grant Read-Only Permission to an Anonymous User

in the example bucket policies listed here: http://docs.aws.amazon.com/AmazonS3/latest/dev/example-bucket-policies.html It specifies you can put a principal like the following:
"Principal": "*"
The SDK has AllUsers, but this creates a principal like the following:
"Principal": { "AWS": "*" }
There is currently no way to create a principal specifying only the anonymous user through the sdk. Policy.FromJson can parse this kind of principal into an empty list. When trying to create a policy with an empty list for Principal the .toJson doesn't put a principal in the string, which throws an error when attempting to Put it.
The main difference is that if we put an explicit Deny in the policy for "*", our IAM user can still make requests against that bucket. If { "AWS": "*" } is used, our IAM user is included in those users who are rejected.

This has been addressed in the referenced commit, and was included in yesterday's release.
The change adds a static Anonymous Principal instance which should serialize as "Principal" : "*'", as well as making the documentation a little more clear and adding some constants for federated and service principals.


background mode with geolocation plugin

Hi, I was trying to activate the geolocation to the user location after X minutes, but when I locked the phone or put the app on the background it only get the position twice, then it stops working. I already use disableweboptimization and disablebatteryoptimization.
 this.backgroundMode.enable();
      this.backgroundMode.disableWebViewOptimizations();
      console.log('BACKGROUND');
      console.log(this.backgroundMode.isEnabled());
      this.backgroundMode.on('activate').subscribe(r => {
        this.backgroundMode.disableWebViewOptimizations();
        this.backgroundMode.disableBatteryOptimizations();
        // console.log('ONACTIVATE');
        // console.log('STATUS BACKLOCATION');
        this.intervalId = interval(10000);
        this.intervalSub = this.intervalId.subscribe(() => {
          // console.log('PRUEBA PRUEBA');
          this.backgroundMode.wakeUp();
          this.geolocation.getCurrentPosition().then(r => {
            // console.log('UBICACIONI');
            console.log(r.coords.latitude);

And when I unlock the app it resumes the geolocation for the times I was getting the location.

could you solve it?
Same problem, I did the test in android 7 and 9 and everything worked fine, in android 10 when the background mode enters the geolocation plugin stops working, there is no error code

I have a solution, in android 9 or lower the gps works in background mode, but in android 10 it does not, what you should do is go to settings-app & notifications-you select your app, select permissions, select location, 3 options will appear by default this is the one Option of allow only while using the app, you must select allow all time and once your application is done, the gps will work with background mode, in the documentation it is indicated that the previous steps must be indicated to the user
https://developer.android.com/training/location/permissions#background

hope it helps, please test on two different devices with android 10, test nokia 4.2, huawei p30 ligth

I tried this, but I think sometimes is not working when I build , the geolocation stop working.

I tried this, but I think sometimes is not working when I build , the geolocation stop working.

you have these permissions in your config.xml

I use it and I have no problem, I even request permission to allow all the time from the app. check the permissions and tell me to help you

you have these permissions in your config.xml

I use it and I have no problem, I even request permission to allow all the time from the app. check the permissions and tell me to help you

@alexisqc92 I faced exactly the same problem now, did you manage to find a solution?

@alexisqc92  @i-metlyuk
Hello, if I could solve it, what problem do you have exactly?
Sometimes although the permissions are in the config it does not work and they have to be added directly in the android manifest in the platform / android folder
In android 10 it does not ask for the permissions in execution, I recommend using the android permission plugin requesting permissions.
I recommend in android 10 to check if you have the permissions, if you do not have redirect the user to the settings so that he selects allow all the time
I currently have it working on android 9 or inderior, android 10 and 11
It also works for me on IOS
If you provide snippets of your code it would be great to help you more: D
add android guidelines on permissions and location
https://developer.android.com/training/location/permissions#request-background-location

@alicarlo
Hi
I'm trying to get geolocation in the background mode using this code:
turnBackgroundTaskOn: function () {
            let $this = this;
            $this.backgroundMode.on('activate', function () {
                $this.backgroundMode.disableWebViewOptimizations();
                clearInterval($this.sendDataTaskInDefaultMode);
                clearInterval($this.sendDataTaskInForegroundMode);
                $this.sendDataTaskInBackgroundMode = setInterval(() => {
                    console.log('trying to create request');
                    $this.getAllData().then(()=>{
                        $this.sendData().then(()=>{
                            console.log('Data sent in background mode');
                        });
                    });
                }, $this.datePeriodForChecking);
            });
        },

$this.getAllData() includes the code for geolocation:
getGeolocation: function () {
            let $this = this;

            return new Promise(function (resolve, reject) {
                cordova.plugins.diagnostic.isLocationAvailable(function (available) {
                    console.log('gps is avaivable')
                    if (available) {
                        navigator.geolocation.getCurrentPosition((geolocation) => {
                            $this.geolocationLatitude = geolocation.coords.latitude;
                            $this.geolocationLongitude = geolocation.coords.longitude;
                            resolve();
                        }, (error) => {
                            $this.geolocationLatitude = null;
                            $this.geolocationLongitude = null;
                            resolve();
                        }, { timeout: 5000 });
                    } else {
                        $this.geolocationLatitude = null;
                        $this.geolocationLongitude = null;
                        resolve();
                    }

                }, function (error) {
                    $this.geolocationLatitude = null;
                    $this.geolocationLongitude = null;
                    resolve();
                });
            });
        },

My AndroidManifest.xml has those geolocation permissions:
<uses-permission android:name="android.permission.ACCESS_BACKGROUND_LOCATION" />
<uses-permission android:name="android.permission.ACCESS_COARSE_LOCATION" />
<uses-permission android:name="android.permission.ACCESS_FINE_LOCATION" />

Therefore, my application has the "Allow all the time" geolocation option enabled
When I switch to another application or turn off the screen, the code in the background mode continues to work, I receive all the necessary data (device model, android version, ip and etc.) and send it.
But after ~40 seconds, I can no longer receive geolocation data.
It looks like If geolocation service is turned off. And if we'll check this code in console, it will return undefined:
navigator.geolocation.getCurrentPosition((res)=>{
            lconsole.log(res)
})

I hope this is comprehensive information.
And, yeah, the platform is Android 10.
Thanks in advance.

hello @i-metlyuk
I did a background demo with gps and local notification, I add the url of the repository, do the test and tell me if it works for you, in case it works for you we will review your code again, I hope it will be of help to you and others, I'm waiting to see if it works for you: D
If you like the solution give me a like and a star in the repository
https://github.com/alicarlo/backGroundMode-Demo-Ali/tree/master
If you have android 10 or higher remember to enable allow all the time in settings that is for the gps to work in the background


Add more options for tree-tagger hook

Codecov Report

Merging #452 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #452      +/-   ##
==========================================
- Coverage   64.96%   64.95%   -0.02%
==========================================
  Files          77       77
  Lines        7128     7128
==========================================
- Hits         4631     4630       -1
- Misses       2497     2498       +1

Impacted Files
Coverage Δ

onmt/utils/Dict.lua
90.34% <0%> (-0.69%)
:arrow_down:

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 dce6a16...77d9ea4. Read the comment docs.

thanks for the PR - can you please update the doc (https://github.com/OpenNMT/OpenNMT/blob/master/hooks/README.md). thanks!

Thanks @panosk !


Spring Boot 3 Compatibility

Auto configuration is currently not working in projects using Spring Boot 3, because the spring.factories file is no longer supported (source).
There might be more issues, but I did not get past auto configuration.
Are there any plans to add support for Spring Boot 3?

Yes, there is a plan. Will create an issue. But might take some time, because of our current bandwidth. So contribution is welcome :)

now it is supported! closing the issue


Hide comment box upload message

What
Closes #1992
Notes
@bfred-it mentioned here that

the !important is required now because they have a utility class on it

Oops, just realized that I merged master into my branch, and that some of the css were moved into their own files. I'll revert and fix it 😄


Alpha of SpriteContainer ignored?

I am fading a SpriteContainer's alpha property with TweenJS. By listening to the change event, I can see that the value of alpha changes over time. However, the sprites inside the container are not affected by the change at all. This is roughly what I'm doing:
hero.alpha = 1;

createjs
  .Tween
  .get(hero)
  .to({alpha: 1}, 1000)
  .addEventListener("change", () => console.log(hero.alpha))

If I perform the tween directly on any of the container's children, the fading happens as it should. I don't think this is the expected behaviour but maybe I'm missing something?!

I think this might be a bug in the old SpriteContainer. We deprecated its usage, and recommend trying out the new StageGL. It is ready, it just needs docs and demo polish, which we have been trying to get to recently. You can get it from the StageGL-beta branch. It is both more performant, and also way easier to use (just swap your stage).
Cheers,

@lannymcnie Ok, thanks for the input. I'm in the middle of a project with my students and it's probably not the best time to switch (especially if the documentation is not ready yet)... unless the JSDoc that I saw inside the StageGL object is published somewhere?
By the way, it might be a good idea to update the SpriteStage and SpriteContainer documentation to say that they are now deprecated ;-)
Thanks.
P.S. StageGL is a much better name than SpriteStage (imo).

Does StageGL also require the use of a single sprite sheet?

No, StageGL is a rewrite to remove those limitations. You can literally swap out the regular stage for a StageGL, and any supported content will work. This version has a much more efficient pipeline, and is able to use as many textures as are supported by the GPU.
Not supported: Vectors and text (unless cached)
It is advisable to still work things into spritesheets where possible (eg, a lot of small caches are not helpful to the GPU).

@lannymcnie Excellent! Can't wait to try it out with my students. It should be easier this way. I'll wait for an official and documented release though. Thanks for the excellent work. Cheers!

Closing this since SpriteContainer is officially removed.


Please release 0.7.0 on crates.io

Hello, I'm a developer that's interested in using TiKV as a backend of storage.
But current build of client-rust keeping on fail and merely changing the dependency of it to git repo will require you to release a 0.7.0 version.
Would you kindly publish a new version?

Let's @tikv/maintainers to take a look.

Now it's released and published.


Allow to zoom and pan when the image is opened

This issue is a:

Feature request

Description
It would be nice if we could actually zoom and move around the image after it is opened, right now if the image is super big, this component actually makes it smaller by making it fit on the screen.
In other words, allow to zoom more, after clicking on the image

@searleser97
Thank you for opening an issue! There are a number of other React components out there that do this already, and here's an example:
https://www.npmjs.com/package/react-zoom-pan-pinch
This project was originally based on the Medium.com-style implementation, and pan + zoom would blow out the scope of this project further than I want to support.


Add missing pipeline binaries struct entry

VkPipelineBinaryHandlesInfoKHR contains a dynamic array and must be handled differently.
Currently any attempt to call vkCreatePipelineBinariesKHR would assert in AddStructGroupEntry.

CI gfxreconstruct build queued with queue ID 278597.

CI gfxreconstruct build # 5065 running.

CI gfxreconstruct build queued with queue ID 278617.

CI gfxreconstruct build # 5066 running.

CI gfxreconstruct build # 5066 passed.

Hey, @MarkY-LunarG , do you want a chance to review this or can Ziga merge?

LGTM.


Fix search_path after move to extension

The search_path should be set by connector not extension.
Reenable tests here: https://github.com/timescale/promscale/pull/1132/files#r821157544

I mean 2 here. I think we should modify the database's default search path as before when the connector is doing the initial installation. I don't see much of a downside to doing that. What is your concern?

We discussed this and came to the following conclusion:

Set search path on initial extension install
If search path cannot be set, log a warning to promscale's log, but do not fail. The warning should contain the exact SQL required to be run by DB owner to set the search path.

We recently discussed this again, and decided to revert on the original idea. We will continue to put Promscale's public schemas (those without an _ prefix) on the search_path during extension install. This ensures that all of the promscale views/tables/functions are available to users. We will document this in the installation notes.

is this done with https://github.com/timescale/promscale_extension/pull/198? @JamesGuthrie should issue be closed?

I still need to document this with the install instructions.


Refactors returning edits on anonymous buffers create new anonymous buffers, confuses TSServer

const _unused = {
    method() {
        [|this.doThing();|]
    }
}

Request refactorings, and extract to a function at the top-level.
Expected: no crash
Actual: I'm not sure on the specific version, but in the last 9 days or so we started seeing the following crash:
Response received: getApplicableRefactors (7). Request took 5 ms. Success: false . Message: Error processing request. Debug Failure. Expected 146 <= 145
Error: Debug Failure. Expected 146 <= 145
    at assertDiagnosticLocation (node_modules/typescript/lib/tsserver.js:15099:22)
    at Object.createFileDiagnostic (node_modules/typescript/lib/tsserver.js:19847:9)
    at getRangeToExtract (node_modules/typescript/lib/tsserver.js:159474:42)
    at Object.getRefactorActionsToExtractSymbol [as getAvailableActions] (node_modules/typescript/lib/tsserver.js:159273:38)
    at node_modules/typescript/lib/tsserver.js:149548:30
    at getIterator (node_modules/typescript/lib/tsserver.js:562:23)
    at Object.next (node_modules/typescript/lib/tsserver.js:557:35)
    at Object.arrayFrom (node_modules/typescript/lib/tsserver.js:1309:88)
    at Object.getApplicableRefactors (node_modules/typescript/lib/tsserver.js:149544:23)
    at Object.getApplicableRefactors (node_modules/typescript/lib/tsserver.js:165354:32)
    at IpcIOSession.Session.getApplicableRefactors (node_modules/typescript/lib/tsserver.js:176715:53)
    at Session.handlers.ts.Map.ts.getEntries._a.<computed> (node_modules/typescript/lib/tsserver.js:175191:61)
    at node_modules/typescript/lib/tsserver.js:177103:88
    at IpcIOSession.Session.executeWithRequestId (node_modules/typescript/lib/tsserver.js:177094:28)
    at IpcIOSession.Session.executeCommand (node_modules/typescript/lib/tsserver.js:177103:33)
    at IpcIOSession.Session.onMessage (node_modules/typescript/lib/tsserver.js:177129:35)
    at process.<anonymous> (node_modules/typescript/lib/tsserver.js:179764:31)
    at process.emit (node:events:390:28)
    at emit (node:internal/child_process:917:12)
    at processTicksAndRejections (node:internal/process/task_queues:84:21)

Possible introduced with 10c8c1c2cce90256dc84597034c18b391a8b189f

Yeah, I believe it didn't repro on a >2 week old nightly, but I can't 100% confirm.

This bug has been fixed in the latest release of VS Code Insiders!
@DanielRosenwasser, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version 15253f1528573fc8f84e956b21af4bad57f38c07 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!


War conversion

Output in /common/history/diplomatic_plays/99_converter.txt
Each entry should have the following syntax:
DIPLOMATIC_PLAYS = {
	c:[Attacker tag] = {
		create_diplomatic_play = {
			name = [localized string]

			target_state = s:[STATE_NAME].region_state:[ABC]

			war = yes

			type = [casus belli]

			add_war_goal = {
				holder = c:[Defender tag]
				type = [war_goal]
				target_state = s:[STATE_NAME].region_state:[ABC]
			}
		}
	}
}

Casus belli is mapped through war_goal_map.txt configurable.
[war_goal] is mapped through defense_war_goal_map.txt
No idea yet how to use target_state or how to trigger co-belligerants

Include a fronter option to toggle if wars are converted or not, even in a good conversion this can get messy. Also makes less sense for people time skipping their games from 1821->1836.


Add carrier name next to tracking code and implement tracking URL for supported carriers

What is the motivation for adding/enhancing this feature?
Enhance the order tracking information by displaying the carrier name next to the tracking code and providing a tracking URL for supported carriers in the order look up page.
Requirements:

Append the carrier name to the tracking code in the following format
Tracking Code: 532281899 - CityExpress
The tracking URL, at least for the carriers that provide this feature (like C807).

Additional information
Clickup ticket for reference: https://app.clickup.com/t/86cvzt9cc

Verified in release v2.10.0 and found that the tracking URL for C807 is not clickable. Hence, reopening this issue.

Verified and working fine in release v2.11.4. Hence, closing this issue.
https://jam.dev/c/9deb4624-5545-4d8e-b4ab-55f8df208a1b


[FR] - Add input utxo validation to transaction build cmd

Internal/External
Internal   if an IOHK staff member.
Area
Plutus    Related to Plutus Scripts (Alonzo).
Describe the feature you'd like
The new transaction build cmd doesn't validate whether --tx-in utxo exists, it just says (where I was using a non-existing utxo and trying to send 5ada and it thought fee was 0.163785ada)
"The net balance of the transaction is negative: Lovelace (-5163785) lovelace."
So, it treats the utxo as value 0 instead of explicitly saying what's up. It may not be so obvious with complex transactions, or if the utxo gets spent it'd be good to know.
Describe alternatives you've considered
Would be good to error with list of all the --tx-in utxos that don't exist.
Additional context / screenshots
n/a

The balance check is done in the following places:

https://github.com/input-output-hk/cardano-node/blob/aab244a42ab9a1e79f82c34a8e8f11faf1fa8a57/cardano-api/src/Cardano/Api/Fees.hs#L816
https://github.com/input-output-hk/cardano-node/blob/aab244a42ab9a1e79f82c34a8e8f11faf1fa8a57/cardano-api/src/Cardano/Api/Fees.hs#L820

the correct validation is done for the --tx-out

Proven #3151 fixes this. Additional validation may error like:
Command failed: transaction build  Error: The following tx input(s) were not present in the UTxO: fac4b496fc50d0fc15cc9dd60ae1f0e411154311144d020334dfaa0522fef92a#1
fac4b496fc50d0fc15cc9dd60ae1f0e411154311144d020334dfaa0522fef92a#2


Allow to edit social sharing texts from the settings panel

Shouldn't be taken from environment variables:
socialshare : {
    siteName : env.SOCIALSHARE_SITE_NAME,
    siteDescription : env.SOCIALSHARE_SITE_DESCRIPTION,
    image : env.SOCIALSHARE_IMAGE,
    domain : env.SOCIALSHARE_DOMAIN,
    twitter : {
      username : env.SOCIALSHARE_TWITTER_USERNAME
    }
  },

These belong to the config for now...


Duplicate seeds across threads in RandomState with std feature?

https://github.com/orlp/foldhash/blob/4f19fe6c349603824b381d375cd9f591c8371f98/src/seed.rs#L35-L42
This looks like it will produce the same sequence of seeds on each thread. Am I missing something?
This doesn't seem ideal, at least the non-std version below appears to try to avoid duplicate seeds across threads.

Hmm, I think you're right, I should change that.


Not able to set attribute name to tooltipster="top"

Looks like Webflow has maybe removed the ability to include '=' signs in attribute names?

"Hello, please and thank you" are appreciated words.
Thank I don't know what you are talking about, this doesn't sound like a Tooltipster issue at all.
Thank you.

Apologies! Didn't mean to come off rude at all - I was just trying to help!
However, I can see that the issue is specifically with Webflow and perhaps would have been better raised here: https://webflow.com/website/Tooltips-for-Webflow

No worries :) I only blame Stackoverflow for creating a generation of developers that skip these things lol.
Yes, please raise the issue to Webflow, thank you.


encryptionKey isn't actually a secret key

see https://github.com/facebook/conceal#entity-and-keys

Hi @vonovak
So, I didn't get it. I'm using the same example as facebook usage example.
It is just a wrapper for shared prefs :)


added my homework 3

@ebeshero can I make this pull request

@Janman813 Yes you can!

@Janman813 I have merged your pull request and deleted your remote branch. YOU right now still have the local branch on your computer. You should now delete that branch locally.
Switch to your main branch on your computer: git checkout main
Do a git pull to get the fresh updates on the main branch (including your freshly merged files from your branch0.
Then, delete your branch with git branch -d yourBranch


Poor user and dev experience in Identity email confirmation

From @natelaff on December 30, 2015 19:42
There are two issues using the default template for identity when using email confirmation.
Following the guide here: http://docs.asp.net/en/latest/security/authentication/accconfirm.html
Poor user experience: When you click "Register" it sends the email off, but just redirects the user to the Home controller. Redirecting to something else to indicate they have to confirm their account would be a better example. Perhaps the default template should have such a view OOTB.
Poor dev experience: When logging in with an external account (lets say Facebook) if the account doesn't exist you get thrown into AccountController.ExternalLoginConfirmation, great. This screen requires an email, so you complete the registration then you're logged in but never shoots off the email confirmation link here. Perhaps a commented out section in this action to deal with this would improve the dev experience here.
Copied from original issue: aspnet/Templates#365

From @rustd on January 5, 2016 5:4
Thank you for your suggestion. We will look into improving this.

From @Ponant on March 6, 2017 9:28
When will this be addressed? It is one year old. A related issue https://github.com/aspnet/Templates/issues/832 .

This issue was moved to aspnet/templating#124

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain tonydrake.com has exceeded the max emails per hour (100/100 (100%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o3.sgmail.github.com ([<IP_ADDRESS>]:38645)
by web240.dnchosting.com with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89)
(envelope-from<EMAIL_ADDRESS>id 1e77AA-003Z9x-0S
for<EMAIL_ADDRESS>Tue, 24 Oct 2017 16:55:08 -0500
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=7w2SqBMUifOT950oTGSaZ4mAQto=; b=Afc2rpYWrw6Ru5dt
Wh6xqTDS/sfQj9ZbnFKB0QpHNTdbXmhrLy1c+vY1XTEKq/1STbV5aJ2t+z0qoA8w
zi8IUStcBl9OLEUuYZ9wxJfOqPnEm/WUPwj0XBFtsUuNwOCGeEB+EyP7V+2TthvH
q0TEhBSHx/ie/aboLF03YcoTRx4=
Received: by filter0990p1mdw1.sendgrid.net with SMTP id filter0990p1mdw1-8593-59EFB690-5
2017-10-24 21:54:24.253421751 +0000 UTC
Received: from github-smtp2b-ext-cp1-prd.iad.github.net (github-smtp2b-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0010p1iad1.sendgrid.net (SG) with ESMTP id MRWe1k2rRbiQ68A6Q5urEw
for<EMAIL_ADDRESS>Tue, 24 Oct 2017 21:54:24.193 +0000 (UTC)
Date: Tue, 24 Oct 2017 21:54:24 +0000 (UTC)
From: Eilon Lipton<EMAIL_ADDRESS>Reply-To: aspnet/templating<EMAIL_ADDRESS>To: aspnet/templating<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: [aspnet/templating] Poor user and dev experience in Identity email
confirmation (#125)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_59efb6901672c_c763fb5e0ae2f3834052c";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: Eilon
X-GitHub-Recipient: tonydrake
X-GitHub-Reason: subscribed
List-ID: aspnet/templating <templating.aspnet.github.com>
List-Archive: https://github.com/aspnet/templating
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/ABtKPnabkga55RDBWeYqClmtCZiB4K4yks5svlyQgaJpZM4QFJaT
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: VzzGqkY9w9rHVIaUFM0gblt5mMHwjysQbnAPllhCkoWPo9Z3zbOpxc3o1nI6kNcFCOw8VZh++ZreLO
e2gwKldI45N48nNDdt8BplpvJhIVl3V57z+Qdmn7mNcmr2jvLB+yrr9qHRrD9lq1O2mrKI7wDwsmSh
jvJpaS85apKllwQCBhFevd97h9WRR8TlkRjrdXWlsD9P6/ZashkvPO9XXnujyAh9deT3Q+lw9aSvSl
k=
X-Spam-Status: No, score=1.2
X-Spam-Score: 12
X-Spam-Bar: +
X-Ham-Report: Spam detection software, running on the system "spamd2.dnchosting.com", has
identified this incoming email as possible spam.  The original message
has been attached to this so you can view it (if it isn't spam) or label
similar future email.  If you have any questions, see
the administrator of that system for details.
Content preview:  From @natelaff on December 30, 2015 19:42 There are two
issues using the default template for identity when using email confirmation.
Following the guide here: http://docs.asp.net/en/latest/security/authentication/accconfirm.html
[...]
Content analysis details:   (1.2 points, 10.0 required)
pts rule name              description

-1.1 RP_MATCHES_RCVD        Envelope sender domain matches handover relay domain
2.2 HTML_IMAGE_ONLY_32     BODY: HTML: images with 2800-3200 bytes of words
0.0 HTML_MESSAGE           BODY: HTML included in message
0.0 T_DKIM_INVALID         DKIM-Signature header exists but is not valid
X-Spam-Flag: NO
----==_mimepart_59efb6901672c_c763fb5e0ae2f3834052c
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
From @natelaff on December 30, 2015 19:42
There are two issues using the default template for identity when using email confirmation.
Following the guide here: http://docs.asp.net/en/latest/security/authentication/accconfirm.html
Poor user experience: When you click "Register" it sends the email off, but just redirects the user to the Home controller. Redirecting to something else to indicate they have to confirm their account would be a better example. Perhaps the default template should have such a view OOTB.
Poor dev experience: When logging in with an external account (lets say Facebook) if the account doesn't exist you get thrown into AccountController.ExternalLoginConfirmation, great. This screen requires an email, so you complete the registration then you're logged in but never shoots off the email confirmation link here. Perhaps a commented out section in this action to deal with this would improve the dev experience here.
Copied from original issue: aspnet/Templates#365
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/templating/issues/125
----==_mimepart_59efb6901672c_c763fb5e0ae2f3834052c
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
From @natelaff on December 30, 2015 19:42
There are two issues using the default template for identity when using email confirmation.
Following the guide here: http://docs.asp.net/en/latest/security/authentication/accconfirm.html
Poor user experience: When you click "Register" it sends the email off, but just redirects the user to the Home controller. Redirecting to something else to indicate they have to confirm their account would be a better example. Perhaps the default template should have such a view OOTB.
Poor dev experience: When logging in with an external account (lets say Facebook) if the account doesn't exist you get thrown into AccountController.ExternalLoginConfirmation, great. This screen requires an email, so you complete the registration then you're logged in but never shoots off the email confirmation link here. Perhaps a commented out section in this action to deal with this would improve the dev experience here.
Copied from original issue: aspnet/Templates#365
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_59efb6901672c_c763fb5e0ae2f3834052c--

Oops, dup of https://github.com/aspnet/templating/issues/124.


Purge build plans on CI server

After the due date of an exercise, it would be good, if Artemis can delete all student build plans on the CI server without deleting data in the Artemis database so that statistics and results are still available. Another possibility would be to manually start this purge mechanism.
The repositories in the VCS don't need to be deleted, so that student could theoretically resume their exercise (in this case only the build plan would be recreated).
To allow for better statistics, Artemis should store the following CI date in its database:

Results of each test case (this would require a change of the object model / database)
maybe: build log (but this should not be interesting)

important: in case of build output files (e.g. jars in release management exercises), the build outputs would be lost, but in this case students could resume the exercise and create the build output again

I think @krusche implemented this in 3c97e3ab, d760683a and 95ba5204.

Yes. We can close it.


"it's" means "it is"

A pet peeve, sorry.

@bukzor No reason to be sorry, I really appreciate this. Wrong is wrong.


Enable Headless support

Currently the gltf-viewer supports a faux headless mode where the window is made invisible while it's still running with the need of some kind a Xserver backend requirement. Is it possible to enable proper headless support so that the entire screenshot generation process could be run in an automated environment without the need of X .

I've tried to do proper headless support, but some trouble with the framebuffer (it remained black). I think it should work with some further debugging, there's probably only a tiny error somewhere.
The code is still there, you could try to get it working (I'd add a link to it, but I'm currently on mobile).
Otherwise I might try again, but I'm not sure when I'll find the time.

okay thanks @bwasty I'll look into it .

Figured it out -> #14 (a gl::Viewport call was missing)
NOTE: I'm not entirely sure it will really work without an X server now. If not, there's always xvfb...

To summarize #14 - 'real' headless mode now works fine on macOS, but not on Linux. It would require quite a bit of further debugging and possibly fixes to glutin to get working.
But #15 should provide a decent workaround 😃

Created an issue for glutin: https://github.com/tomaka/glutin/issues/988

@bwasty It seems the headless issue was fixed in the glutin library on version 0.21.0.  However, there seems to be lots of breaking changes  that are not trivial to apply to gltf-viewer.
The PR that fixed the issue itself: https://github.com/rust-windowing/glutin/pull/1125