Support for a full equation of state for seawater (TEOS-10?)

@johncmarshall54 has suggested we switch to a full equation of state for seawater for LESbrary simulations as temperature and salinity profiles from state estimates may not be statically stable in Oceananigans when using a linear equation of state. The Roquet et al. (2015a) idealized nonlinear equation of state should be better. Ideally we would use the same equation of state as ECCO/SOSE I think, but TEOS-10 should be what we aim for I suppose.
I think we have a couple of options:

There's a Julia wrapper for the TEOS-10 C library but this probably won't work on the GPU: https://github.com/ax1ine/GSW.jl
A 6+52-term polynomial approximation to TEOS-10, accurate to ~0.1% and suitable for Boussinesq models, from Roquet et al. (2015b). We can code up a pure Julia version as there already exist Python, MATLAB, and Fortran implementations here: https://github.com/fabien-roquet/polyTEOS (Thank you @fabien-roquet!)

In implementing a pure Julia TEOS-10 equation of state we should make sure it can be shared between Oceananigans and CliMA Ocean (cc @blallen).
cc @jm-c @christophernhill @glwagner @rafferrari who might have suggestions or comments.
@leios Do you think evaluating a 52-term polynomial at every grid point will be problematic on the GPU? I imagine GPUs should be fast at evaluating polynomials but maybe we have to be smart about storing the coefficients in memory?
References:

Roquet et al. (2015a), "Defining a Simplified Yet “Realistic” Equation of State for Seawater", DOI: https://doi.org/10.1175/JPO-D-15-0080.1
Roquet et al. (2015b), "Accurate polynomial expressions for the density and specific volume of seawater using the TEOS-10 standard", DOI: https://doi.org/10.1016/j.ocemod.2015.04.002

I also think it’s a good idea to use a full equation of state for all simulations sooner rather than later. It’s simpler: we won’t have to report constants of linearization everywhere. And setting up simulations will be easier.
I’ll defer to the modelers for whether Roquet’s approximation is an acceptable model for TEOS-10.
This package is relevant and we should consider contributing to it rather than implementing an equation of state somewhere in the Clima ecosystem:
https://github.com/gher-ulg/PhysOcean.jl

In implementing a pure Julia TEOS-10 equation of state we should make sure it can be shared between Oceananigans and CliMA Ocean (cc @blallen).

:100: :100: :100: :100: :100: :100: :100: :100: :100: :100: :100: :100: :100:

This package is relevant and we should consider contributing to it rather than implementing an equation of state somewhere in the Clima ecosystem:
https://github.com/gher-ulg/PhysOcean.jl

:sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses: :sunglasses:
Whatever we do should be shared between Oceananigans and Clima. Either a custom implementation or using the PhysOcean.jl package is fine by me.

The  Roquet’s approximation is perfectly sufficient for Oceananigans, because it will never be used for global calculations where local approximations are an issue. However I agree with everybody else that it would be best to use the same EOS in Ocenanigans and Climate_Ocean. in that case we should adopt TEOS-10. Be warned that it is quite inefficient through. So we may be hit performance-wise. Hard to tell without trying.
Raffaele

On Mar 9, 2020, at 11:48 AM, Gregory L. Wagner<EMAIL_ADDRESS>wrote:
I also think it’s a good idea to use a full equation of state for all simulations sooner rather than later. It’s simpler: we won’t have to report constants of linearization everywhere. And setting up simulations will be easier.
I’ll defer to the modelers for whether Roquet’s approximation is an acceptable model for TEOS-10.
This package is relevant and we should consider contributing to it rather than implementing an equation of state somewhere in the Clima ecosystem:
https://github.com/gher-ulg/PhysOcean.jl https://github.com/gher-ulg/PhysOcean.jl
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/climate-machine/Oceananigans.jl/issues/692?email_source=notifications&email_token=AK24ROIBMZMYD77AGEZNM3DRGUMURA5CNFSM4LEKJAAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOHZCYQ#issuecomment-596611426, or unsubscribe https://github.com/notifications/unsubscribe-auth/AK24ROONT4RO4YCCF6BTIIDRGUMURANCNFSM4LEKJAAA.

What does J-M think?
John
On Mon, Mar 9, 2020, 3:07 PM rafferrari<EMAIL_ADDRESS>wrote:

The Roquet’s approximation is perfectly sufficient for Oceananigans,
because it will never be used for global calculations where local
approximations are an issue. However I agree with everybody else that it
would be best to use the same EOS in Ocenanigans and Climate_Ocean. in that
case we should adopt TEOS-10. Be warned that it is quite inefficient
through. So we may be hit performance-wise. Hard to tell without trying.
Raffaele

On Mar 9, 2020, at 11:48 AM, Gregory L. Wagner<EMAIL_ADDRESS>wrote:
I also think it’s a good idea to use a full equation of state for all
simulations sooner rather than later. It’s simpler: we won’t have to report
constants of linearization everywhere. And setting up simulations will be
easier.
I’ll defer to the modelers for whether Roquet’s approximation is an
acceptable model for TEOS-10.
This package is relevant and we should consider contributing to it
rather than implementing an equation of state somewhere in the Clima
ecosystem:
https://github.com/gher-ulg/PhysOcean.jl <
https://github.com/gher-ulg/PhysOcean.jl>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub <
https://github.com/climate-machine/Oceananigans.jl/issues/692?email_source=notifications&email_token=AK24ROIBMZMYD77AGEZNM3DRGUMURA5CNFSM4LEKJAAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOHZCYQ#issuecomment-596611426>,
or unsubscribe <
https://github.com/notifications/unsubscribe-auth/AK24ROONT4RO4YCCF6BTIIDRGUMURANCNFSM4LEKJAAA
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/climate-machine/Oceananigans.jl/issues/692?email_source=notifications&email_token=AKXUEQUJLIMSIC2HXXYDQG3RGU47RA5CNFSM4LEKJAAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIVI3Y#issuecomment-596726895,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKXUEQV7RCQMEU6MW3UZPZTRGU47RANCNFSM4LEKJAAA
.

What does J-M think?

Talked to @jm-c earlier today who agreed that the Roquet et al. (2015b) 52-term polynomial approximation to TEOS-10 is a good choice.
Although he did point out that ECCO and SOSE are probably using an older pre-TEOS-10 equation of state (from 1995 or 2003 based on the EOS-80 but for potential temperature I think?).

The  Roquet’s approximation is perfectly sufficient for Oceananigans, because it will never be used for global calculations where local approximations are an issue. However I agree with everybody else that it would be best to use the same EOS in Ocenanigans and Climate_Ocean. in that case we should adopt TEOS-10. Be warned that it is quite inefficient through. So we may be hit performance-wise. Hard to tell without trying.

Thanks for the feedback @rafferrari. I talked to @leios earlier today and we think it shouldn't be a problem on the GPU. It's just straight up number crunching so it might benefit from being run on a GPU. But we can make sure by doing a quick benchmark.

Ali,
This is a correct transcript of what I said earlier.
Cheers,
Jean-Michel
On Mon, Mar 09, 2020 at 07:21:21PM -0700, Ali Ramadhan wrote:

What does J-M think?

Talked to @jm-c earlier today who agreed that the Roquet et al. (2015b) 52-term polynomial approximation to TEOS-10 is a good choice.
Although he did point out that ECCO and SOSE are probably using an older pre-TEOS-10 equation of state (from 1995 or 2003 based on the EOS-80 but for potential temperature I think?).
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/climate-machine/Oceananigans.jl/issues/692#issuecomment-596867518

This is resolved as of v0.30.0!


Editor margins and form output with lists

Description
Per  Ticket #18996
List spacing isn't accurate between the editor and the form output.

Tested and LGTM!


OPENTHREAD_CONFIG_MLE_LINK_REQUEST_MARGIN_MIN does not seem to have any effect

No matter what value I configure, the parent is still far away, and the nearer cannot become the parent. The signal strength at the far end is -81dbm, and the signal strength at the near is -31dbm.
NRF52840+2401C
f45b588e119491d8fd365e584d210ab49e0aae6a

@Feng573795502 , I suspect the reason why your device is not attaching is unrelated to OPENTHREAD_CONFIG_MLE_LINK_REQUEST_MARGIN_MIN.
Can you provide more information as below?
Describe the bug A clear and concise description of what the bug is.
To Reproduce Information to reproduce the behavior, including:

Git commit id
IEEE 802.15.4 hardware platform
Build steps
Network topology

Console/log output If applicable, add console/log output to help explain your problem.
Additional context Add any other context about the problem here.

Closing stale issue.


Custom Service Call on Button Press

Apologies, I tried to troubleshoot this as much as  I could but I'm clearly missing something.
I have verirfied this service call works as I need:
service: media_player.volume_down
data: {}
target:
  entity_id: media_player.living_room

However, I can't seem to add this to the volume down button. I'm sure I'm just missing how it needs to be formatted but would love an example that shows this working. It's a really great functionality you've built in to allow custom buttons!
Here's what I've tried:
volumedown:
  service: media_player.volume_down
  data:
  target:
    entity_id: media_player.living_room

volumedown:
  service: media_player.volume_down
  data:
    data: {}
    target:
      entity_id: media_player.living_room

So the format for this card would be:
volumedown:
  service: media_player.volume_down
  data:
    entity_id: media_player.living_room

Thanks for your quick reply. I added that and it seems to still target the TV, not my Sonos soundbar (which is the media_player.living_room)
Here's what I have:
type: custom:polr-android-tv-remote-card
entity_id: remote.living_room_tv
apps:
  - disneyplus
  - prime
  - icon: mdi:youtube
    url: https://www.youtube.com
volumedown:
  service: media_player.volume_down
  data:
    entity_id: media_player.living_room

Hmm I'm not sure. I tried this with media_play entities I have and its working as expected. It may be worth double checking the formatting and spelling?

from my quick experiment it only works if you have both volume down and up defined.
Besides that, is media_player.living_room actually your sonos bar?

Thank you, solved!


[WIP] Lucene 10 SNAPSHOT upgrade

Apparently, deprecating Lucene's constant NO_MORE_ORDS wasn't just for fun on their side 😄.
In Lucene 10 stored set doc values wouldn't return -1 when they are "finished"... For example SingletonSortedSetDocValues would just always return the ord it has leading to a never-ending loop, others can end up throwing some exceptions...
I've tested the patched  DocValuesTextMultiValues and JoiningTextMultiValuesSource on main with Lucene 9 and it passed the TCK tests. I think it probably would be better to make these in the main, and let it be a part of Search 7.0.
Then as for the missing RegexpQueryFlag.COMPLEMENT counterpart in Lucene ... I've removed it from Lucene backend, but left it as is in the tests and Search enum until we decide how we want to treat that option in general...

Agreed. I created https://hibernate.atlassian.net/browse/HSEARCH-4939. Please send a PR for that?

Done: https://github.com/hibernate/hibernate-search/pull/3703 that's the PR 😃

with https://github.com/hibernate/hibernate-search/pull/3710 we wouldn't need an upgrade branch for lucene 10 (at least for now). closing.


Fix discriminator issues in storagecache 2020-03-01

MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I commit to follow the Breaking Change Policy of “no breaking changes
[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

[ ] Ensure to check this box if one of the following scenarios meet updates in the PR, so that label “WaitForARMFeedback” will be added automatically to involve ARM API Review. Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs, all “removals” and “adding a new property” no more require ARM API review.

Adding new API(s)
Adding a new API version
Adding a new service

[ ] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If there are following updates in the PR, ensure to request an approval from API Review Board as defined in the Breaking Change Policy.

[ ] Removing API(s) in stable version
[ ] Removing properties in stable version
[ ] Removing API version(s) in stable version
[ ] Updating API in stable version with Breaking Change Validation errors
[ ] Updating API(s) in preview over 1 year

Please follow the link to find more details on PR review process.

/azp run automation - sdk

Please do not merge this PR until the service team approved this. Thanks

/azp run automation - sdk

Can one of the admins verify this patch?

Looks good, thank you.

@ArcturusZhang and others
It's a bit concerning to see a breaking change in a stable version of Open API specs without any explanation. Would you be so kind as to explain why this change was made and what was the process of approving the breakage?

Hi @mikhailshilkov this breaking change is to revert one problematic breaking change introduced by #8606 .
In this PR, we nullified the problematic part - which is a breaking change when introduced in, and hence to get rid of it will cause another breaking change.
If we compare the content in this PR with a version before PR #8606, there should be no breaking changes.


[FEATURE] support duckdb

Is your feature request related to a problem? Please describe.
A situation arose where I wanted to use duckdb in local development.
I added duckdb as it does not seem to be currently supported.
Describe the solution you'd like
Please merge here
https://github.com/Datavault-UK/dbtvault/pull/170
I have confirmed that dbt seed and dbt run work.
AB#5369

I'm keen to try this with DuckDB. What is the holdup on the merge here? Is there something wrong with the code?

@adumicic Thanks for this question, you raise a really good point and a valid concern.
Our contribution process does not currently allow direct contributions via PRs due to our proprietary development and test environment. We have over 3000 tests which extensively test the AutomateDV macros with different patterns.
We are reluctant to approve PRs which have not gone through the test suite, however as public contributors do not have access to the test suite, it means we essentially cannot accept them.
We want to work towards having a more streamlined and publicly accessible contribution process, as it will benefit the community massively. I do appreciate this is awkward, and it is something we want to resolve sooner rather than later.
For now, leaving PRs open and making them visible to the community is what we are doing to compromise, rather than closing them.
For this particular PR, I did reach out to the author and let them know about the situation, and they were happy for us to leave it open in this way.

DuckDB is getting more and more useful and popular. I think it makes sense.

Of course it makes sense. Why wouldn't we want everything running in a python project, without depending on external connections?

+1 for duckdb.  I think it will be a great teaching tool to get people into datavault.


Could you please make the example runnable independenty?

As title, I'm trying to run it but it shows a bunch of errors I have to import the library according 1 by 1. I am trying this in a iex -S mix where beaver is installed as a dependency.

I also tried hexdoc article here, it doesn't work off REPL directly if I copy paste the code.

check out this PR: https://github.com/beaver-lodge/beaver/pull/197/files

Thank you!


Adds Selectable Difficulty modes of Easy, Normal & Hard

Adds difficulty mode to the top left of the UI - click to rotate between Easy, Normal and Hard
On Easy Mode, the valid word check only checks that five letters have been entered
Normal Mode also checks that the entry is a recognised word, as before
Hard Mode also checks that the previous correctly guessed letters are included in the current guess, and that all correctly guessed positions are included in that position in the current guess.

Note that it's possible to change difficulty level at any time, and the new level will apply to how the next and future guesses are validated.
Example of rejecting a guess because the letter 'U' is known to be in the answer, but is not in the guess

Not included in this PR are the settings modal dialogue, saving of the selected difficulty in local storage, or update to the info modal to explain the three modes.

PR for #6 - should link the issue

hey @Sam0ht! if you grab the changes from main, there is now a settings icon in the top left that opens a modal. that is where we should put this setting

Hi @Sam0ht! I made the UI for the dark mode toggle inside the settings modal. You can uncomment it and use it with the functionality you built.
/cc @octokatherine


Tray Commands Iconized [Feature request]

As tray support is coming (#35) I would like to suggest a very tiny, cosmetic enhancement- icons for tray commands Show and Exit, screen:

Perhaps tray commands should be iconized only when Settings - Application - Menu icons is enabled.

@Sigmanor
Thanks.


Add links to components of expense breakdown

Here we can add links to all the components of expense breakdown for better understanding of everyone.
There is a minor spelling error in hiring component in the end.

I would really like to work on this issue.

@sambhavgupta0705 @akshatnema I have applied the required changes. Please review.

@sambhavgupta0705 @akshatnema  I have implemented the required changes in this #2563 PR. Please review it.

Hey @sambhavgupta0705  @akshatnema @anshgoyalevil  can you please review my pr #2653 . i have added suitable links for each component


Datastore beta3?

Any plans to support beta3 of the datastore api?

There are currently no concrete plans. According to https://cloudplatform.googleblog.com/2016/04/Google-Cloud-Datastore-gets-faster-cross-platform-API.html the v1beta2 API will be turned down by end of September.

Thanks. I guess as long as we are able to use the new API before v2 ends we
should be good :)
On Mon, Apr 25, 2016, 4:58 AM Søren Gjesse<EMAIL_ADDRESS>wrote:

There are currently no concrete plans. According to
https://cloudplatform.googleblog.com/2016/04/Google-Cloud-Datastore-gets-faster-cross-platform-API.html
the v1beta2 API will be turned down by end of September.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/dart-lang/googleapis/issues/10#issuecomment-214216371

--
Sent from my phone

Is there any plans to get this updated to V1 final? Nervous to port over some code to Datastore because of this.

The datastore:v1 API has been added to package:googleapis some time ago.


Remove test file for HG_OOSTENDE

HG_OOSTENDE contains a stray test.csv file (with radar coverage data):
https://doi.org/10.5281/zenodo.3541812 that should be removed when republishing.

Removed in https://zenodo.org/record/3959038


Question: Is it converting varchar to nvarchar?

Working on MSSQL found that for some reason when a query is executed, it converts varchar to nvarchar. The column on the DB is varchar so we are losing some CPU converting nvarchar to varchar. Any reason why this happen?
Not sure, but it seems is here that happen that conversion: https://github.com/tgriesser/knex/blob/master/src/dialects/mssql/schema/columncompiler.js#L52

Code line linked there is not doing any conversion when queries are made. It just shows that when you are using knex schema building to create .varchar() column it actually creates nvarchar type of column. So no cycles are lost there. @Grimace1975 might know why it doesn't create varchar type.
Can you give some example code where you are experiencing varchar -> nvarchar conversion during normal queries?
Currently it seens that you can create varchar column only by using .specificType schema building method.

We have that case using Bookshelf:
this.where({
  VARCHAR_STRING: 'My varchar string'
 }).fetch({
   columns: ['ID', 'NAME', 'VARCHAR_STRING']
 })

Under the hood bookshelf execute something like that knex('TABLE_NAME').where.apply(knex('TABLE_NAME'), [{VARCHAR_STRING...}]);
And in the database we can see that SQL executed:
(@p0 int, @p1 nvarchar(9))select top (@p0) [ID], [NAME], [VARCHAR_STRING] from [TABLE_NAME] where [VARCHAR_STRING] = @p1

Pretty sure is something related to the driver, we can see it in the data type value list:

@HenriqueLimas Can you explain how we can make VarChar as the default for string?

Where or how can this be set in KNEX from MSSQL?
sql.map.register(String, sql.VarChar)
Can it be done in the initial config?
Since all my tables use VarChar and all parameters from Knex are NvarChar, all queries cause a table scan.

@stevesob You can use the type mapping API in node-mssql documented here.
Please consider opening new issues instead of commenting on issues that were closed years ago.

For anyone looking for how to do this:
db.client.driver.map.register(String, mssql.VarChar);

If someone is looking for a way to create a VARCHAR column in mssql, the best way is by using specificType - https://github.com/knex/knex/pull/4211/files

If someone is looking for a way to create a VARCHAR column in mssql, the best way is by using specificType - https://github.com/knex/knex/pull/4211/files


Auto-configure

Hello,
The auto-configure feature (#797) is a great idea. It would be useful if the exception option allowed for choosing an alternate value instead of just setting the rule to null. Even better would be if the number of errors for each possible alternate value were shown.
I have code for generating a .jscsrc file by automatically choosing the rule value that produces the lowest error count. It doesn't have any command line options yet. Would there be interest in integrating something like this as a built-in option for JSCS?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9605093-auto-configure?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Hey @ariutta. Thanks for contributing.

It would be useful if the exception option allowed for choosing an alternate value instead of just setting the rule to null.

I'm not seeing the benefit to allowing users to supply custom values in the autoconfig prompt. It's the same as replacing null with another value in the generated jscsrc. The goal of autoconfig, was to get the user up and running as quickly as possible. We also envision replacing the "fix it yourself" with "fix it automatically" so that you wouldn't have to worry about configuration.

Even better would be if the number of errors for each possible alternate value were shown.

This heavily depends on #480 to avoid having to modify rules and the autoconfigure code.

I have code for generating a .jscsrc file by automatically choosing the rule value that produces the lowest error count.

Great work here. Along with my previous comment, the route you've experimented with relies on us hardcoding the rule values in both the rule definition and the autoconfigure code.

Would there be interest in integrating something like this as a built-in option for JSCS?

Definitely, so long as it read the rule values from the rule's definition.

Great. I'll keep my eye on #480. When it's done, I'll see if I can get time to add this.


Upgrade to Webnative 0.35

Upgrade to to Webnative v0.35.
Changes:

webnative.app() has been replaced by webnative.program(). Which is now the sole entry point of webnative (no more initialise or permissionedApp either). The reason for the rename is to avoid confusion with Webnative apps, the ones you  create via the CLI or dashboard.
authed WAT session variable has been replaced by session.  I didn't want two variables representing roughly the same state so I removed authed
You can only add Uint8Arrays to the filesystem. That means that strings, files and blobs all have to be converted to Uint8Arrays first. I added one util function for that and strings are converted with TextEncoder/TextDecoder (or use the uint8arrays library).
I replaced the base64 data URIs in the gallery with object urls pointing to Blobs for performance. I assume that's why the file size was set to max 5MB? I increased it to 20 since it's much faster now.
I created a program store because this is the main interaction point to do something with your program. As opposed to using the webnative global. For example, before you'd call webnative.auth.isUsernameValid(), now you'd use program.auth.isUsernameValid(). Reasoning here is that global state has been removed from webnative, so all state lives in your created Program. I'm not entirely sure creating a store was the right approach here, since I'm not familiar with Svelte. Let me know if there's a better way to do this.
I made some adjustments to the type checking code in the gallery. Still not ideal, but safer.

Let me know if anything else needs explaining or if you have ideas to further improve any of the API.
Remaining issues:

[x] After linking another device, it still shows the non-authenticated view on the consumer. Refreshing the page fixes it, but it takes a few seconds.
[ ] Linking does not always work? Webnative issue? Needs more testing.

webnative.app() has been replaced by webnative.program()

Why is this necessary? Is it just because the signature of the method has changed, or is there a semantic reason why "program" is better than "app". TBH I think "app" is more generically understandable by developers.

webnative.app() has been replaced by webnative.program()

Why is this necessary? Is it just because the signature of the method has changed, or is there a semantic reason why "program" is better than "app". TBH I think "app" is more generically understandable by developers.

That came out of discussion with Boris who found the usage of app in this context confusing (correct me if I'm wrong Boris). I more or less agree. We have the app platform code as well in webnative to create Fission apps. I think we want to look at app cloning too in the future (at least we discussed that in the past). Meaning, we could have a tighter integration with apps. So to avoid confusion between the two concepts I opted for program. This also fits better technically since you can create multiple programs that represent the same app.

After linking another device, it still shows the non-authenticated view on the consumer. Refreshing the page fixes it, but it takes a few seconds.

I think the issue might be in loadAccount:
https://github.com/webnative-examples/webnative-app-template/blob/80a3b051be3f4c70c94c6fc9f97141d6b90e4780/src/lib/auth/account.ts#L79
Should this be updated with the session instead of authed?

Should this be updated with the session instead of authed?

That did the trick, thanks! Was expecting a type error for this one.

Issue I noticed: sometimes when you click on "photo gallery demo" in the menu, the app reloads entirely. Have you seen that @avivash @bgins ?

@avivash @bgins I figured out what the linking issue is, the linking producer is sometimes created multiple times.
Not sure why it happens though, something something subscription to session store?

It happens whenever I am redirect to the "delegate-account" page (eg. by clicking connect-device button)
If I'm already on that page and refresh, it works as intended.
Also happens on main branch, see console log at https://webnative-template.fission.app/

@avivash @bgins I figured out what the linking issue is, the linking producer is sometimes created multiple times. Not sure why it happens though, something something subscription to session store?

It happens whenever I am redirect to the "delegate-account" page (eg. by clicking connect-device button) If I'm already on that page and refresh, it works as intended.
Also happens on main branch, see console log at https://webnative-template.fission.app/

ahh interesting! I'll try to recreate/debug it 👀

@avivash @bgins I figured out what the linking issue is, the linking producer is sometimes created multiple times. Not sure why it happens though, something something subscription to session store?

It happens whenever I am redirect to the "delegate-account" page (eg. by clicking connect-device button) If I'm already on that page and refresh, it works as intended.
Also happens on main branch, see console log at https://webnative-template.fission.app/

ahh interesting! I'll try to recreate/debug it 👀

So it seems the page is double mounted exclusively when it is the slot of the SidebarNav component. Not quite sure why yet, but it's a starting point 🤔

@avivash @bgins I figured out what the linking issue is, the linking producer is sometimes created multiple times. Not sure why it happens though, something something subscription to session store?

It happens whenever I am redirect to the "delegate-account" page (eg. by clicking connect-device button) If I'm already on that page and refresh, it works as intended.
Also happens on main branch, see console log at https://webnative-template.fission.app/

ahh interesting! I'll try to recreate/debug it 👀

So it seems the page is double mounted exclusively when it is the slot of the SidebarNav component. Not quite sure why yet, but it's a starting point 🤔

@icidasset i've got a fix 👍🏼, will push it shortly

Issue I noticed: sometimes when you click on "photo gallery demo" in the menu, the app reloads entirely. Have you seen that @avivash @bgins ?

I'd never seen this before just now. It must been tied to a recent update. I'll take a look 👀

Issue I noticed: sometimes when you click on "photo gallery demo" in the menu, the app reloads entirely. Have you seen that @avivash @bgins ?

I'd never seen this before just now. It must been tied to a recent update. I'll take a look 👀 (though i'm having a hard time recreating it)

Okay, I think my fix for the delegate route re-mounting may have fixed this too because I can't recreate it anymore. I'll keep an eye on it though 👍🏼

Yup, seems to be all fixed! 🎉

Waiting on the official 0.35 release!


Make canonicalizer be able to strip session id params even if they are the first params in the query string. And add session id strip test.

This is to resolve https://webarchive.jira.com/browse/ARI-4749. Code has been reviewed by nlevitt.

The change to IAURLCanonicalizer.java is meant to ensure that if, transformations on the query string have completed the query is empty, there is not a ? added to the end of the url.


The plugin allows adding to the cart/buying the same product variation in different languages

The plug in allows you to add to the cart and buy the same product of the same variation in different language. At the end of the purchase the stock of that variation will show a negative number (even if you selected "not allowed back orders")
Reported by kamarena: https://wordpress.org/support/topic/woocommerce-26-27?replies=13#post-8563882

Solved in 1.0.0 Dev


What about conditional column widths for responsive layouts?

Instead of having to respecify every single tag you've set a column width on inside a media query, it would be cool to just re-specify the .column mixin if the screen goes below a certain size.
Right now we have to find every tag with a .column(x) mixin and set them to .column(12), like so:
@media screen and (max-width: 480px) {
   article {
      .column(12);
   }
   aside {
      .column(12);
   }
}

Would it be possible to do something equivalent to:
@media screen and (max-width: 480px) {
    .column(x) {
        width: 100%;
    }
}

This doesn't work, but I was wondering if there was some obscure LESS/SASS conditional logic that could somehow take care of this.

That wouldn't work because LESS is compiled into CSS before you flip your tablet. =)
Something that might work is using the media queries to specify the columns widths inside the column() mixin.

Using media queries inside the column mixin didn't work, although it sure was a cool idea.

I'll look into JS stuff. Thanks anyway.


[WIP] Add series data to JSON output

This adds an Enum constraint to the Series constructor, ensuring
that we can represent Series' arguments when creating the report.
This allows the user to inspect the report programatically and match
benchmark results against the benchmark input.
Note that this is a breaking change for series; however the JSON output changes should be backwards compatible (we only add field).
Example output:
    {
      "bench_data": [
        2,
        1
      ],
      "cycles": null,
      "measurements": null,
      "garbage_collections": null,
      "time_in_nanos":<PHONE_NUMBER>2967033,
      "bench_name": "n choose k:2:1",
      "alloc": null
    }

I'm not happy with the HashMap keys, but this is something we can change later when tackling #22 . Also the show instance for Series is a bit flaky, as it shows a String. Will submit a fix soon, but opening the PR already for feedback.

Split this into two PRs ( #27 and #28 ) and addressed comments.


How to Contribute to This Repository

Hello,
I’ve been learning various AI/ML-related algorithms recently, and my notes are quite similar to the content of your repository. Also this excellent work has helped me understand some of the algorithms, and I’d love to contribute by adding papers such as those on VAE, CLIP, BLIP, etc.
But before I start, I have a question on the implementation method. What tools did you use to generate the web pages for this repository? Are they hard-coded, or did you use a framework?
Looking forward to your response and contributing to this project!

slightly related, is there a contribution guideline about how we can contribute our own implementations? thanks!


relative excludes

- src
  - ./header.js
  - main.js <- browserify target this file - has requires ('./header.js') and require ('foo-bar')

now for browserify I explicity ask browserify to exclude ./header.js

# ./main.js

var headers = require ('./headers.js') # browserify correctly ignores, YAY ! :)
var foo_bar = require ('foo-bar')

so lets look at foo-bar
foo-bar directory

- node_modules
   - foo-bar
     - dist
       - header.js
       - main.js <- entry file has require ('./header.js')

browserify unfortunately also ignores ./header.js in foo bar :(
@substack - what do ?
cheers !

Browserify's exclude attempts to deal with both paths (what you want) and the actual arguments to require (what you're doing now). Instead of excluding ./header.js, try excluding the path to the file.
Excluding/externalizing by require argument is useful in cases where you want to prevent requiring npm modules, e.g. to create a separate bundle with third party code that won't be changing.

@MellowMelon
yes that is exactly what I am trying to achieve ! - multiple bundles
if you do
exclude:'C:/working/src/header.js'
you also need to in your source code do something like this
require (''C:/working/src/header.js'')

something I want to avoid.

Don't exclude the absolute path; exclude the relative path from the basedir (which defaults to the directory from which you ran browserify). You do not need to change the require arguments.

that is exactly what I was doing :( unfortunately it seems browserify does a string match for excludes.

It does both. Check your path again.


(feat) wrap docker run_command in bash

CLA signed by all contributors.

The solution for this is going to be to add a new shell_command option to the docker transport. When set all commands on the target will be wrapped in the shell_command

The new config options should be added to https://github.com/puppetlabs/bolt/blob/master/pre-docs/bolt_configuration_options.md#docker-transport-configuration-options

Can you update the commit message to include the ticket number.

@adreyer Done


common/gpu/amd: use modesetting driver by default

Description of changes
Follows upstream nixpkgs, see https://github.com/NixOS/nixpkgs/pull/218437
Things done

[x] Tested the changes in your own NixOS Configuration
[x] Tested the changes end-to-end by using your fork of nixos-hardware and
importing it via <nixos-hardware> or Flake input

bors merge

hardware.amdgpu.loadInInitrd still defaults to true, is there still a benefit in having amdgpu in initrd if modesetting is being used?

Yes, the kernel module and the DDX are different things.


Added hint to JS increment challenge

Pre-Submission Checklist

[x] Your pull request targets the staging branch of FreeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm run test-challenges. Use git commit --amend to amend any fixes.

Type of Change

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

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (Closes #XXXX): Closes

Closes #7611
Description

Added hint text and link to Increment a Number with JavaScript challenge.

LGTM :+1:

:sparkles: :sparkles: LGTM. :sparkles: :sparkles:
Thanks @DusanSacha


Refactor/styled components

Created a very simple demo to be used after the first styled components session. (see below)

Moved the 2 old sessions to styled components 2 (where we now talk about props).

Reduced styled components 1 demo (start and end)

Created a new styled components 2 demo (start and end)

react-styled-components/card-conversion
📄 Rendered Exercise
🔎 Review
Review locally by running this command in your Terminal:
npx ghcd@latest neuefische/web-exercises/tree/refactor/styled-components/sessions/react-styled-components/card-conversion -i

✨ Copyable Markdown for Curriculum
Solve this challenge locally by running this command in your Terminal:

```
npx ghcd@latest neuefische/web-exercises/tree/main/sessions/react-styled-components/card-conversion -i
```

TODO. I would still like to add a basic challenge where they style a Link component from nextjs.


Update license format

Avi let me know that the official test of the UPL and the approved short format has changed.

Just find and replace in my IDE ;)

... but I also updated the Checkstyle rule so I know I got all the files.


[Event Request] Codeunit 7307 "Whse.-Activity-Register" - OnCodeOnBeforeWhseActivLineIsEmpty

Hi, would it be possible to get an event OnCodeOnBeforeWhseActivLineIsEmpty in codeunit 7307 (Whse.-Activity-Register), like this:
local procedure "Code"()
    var
        OldWhseActivLine: Record "Warehouse Activity Line";
        TempWhseActivLineToReserve: Record "Warehouse Activity Line" temporary;
        TempWhseActivityLineGrouped: Record "Warehouse Activity Line" temporary;
        SkipDelete: Boolean;
        IsHandled: Boolean;
    begin
        OnBeforeCode(WhseActivLine);

        with WhseActivHeader do begin
            WhseActivLine.SetRange("Activity Type", WhseActivLine."Activity Type");
            WhseActivLine.SetRange("No.", WhseActivLine."No.");
            WhseActivLine.SetFilter("Qty. to Handle (Base)", '<>0');
            IsHandled := false;
            OnCodeOnBeforeWhseActivLineIsEmpty(WhseActivLine,IsHandled);
            if not IsHandled then
              if WhseActivLine.IsEmpty then
                  Error(Text003);
            CheckWhseItemTrkgLine(WhseActivLine);
            ...
    end;

[IntegrationEvent(false, false)]
local procedure OnCodeOnBeforeWhseActivLineIsEmpty(var WarehouseActivityLine: Record "Warehouse Activity Line"; var IsHandled: Boolean)
begin
end;

Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Feature/opal support

I took your latest master branch and made the tests with Opal. There is still the issue that opal does not support mutable strings (it is a transpiler to javascript which lacks this support).
I changed the immutable string issues and now all tests pass if run through opal. There was also a difference when handling arbitrary classes. to_json behaves slightly different between ruby and opal with respect of options and indentation.
I never used mutable strings in ruby so all my ruby code worked well with Opal.
This PR replaces PR #13. I would really appreciate if you could consicer merge this.

Could you please make a decision if you intend merge this pull request. Due to the recent changes it now has new merge conflicts.
If you do not intend to merge, it is ok for me, but then I need to continue with my own fork.

I finally got around to perf testing. I appreciate your pull request, for it allowed me to easily identify which areas needed to be fixed. However, I wanted some of the code to be a little different, so I decided to make my own modifications to make it work for Opal.
Commit b72faf3128 fixes the string mutation issue and supports Opal. I've released this as v0.8.1.

It is fine wit me. Thanks for adding the Opal support. I will give it a try in my app.

BTW, the impact of moving to non-mutating was negligible; about a 0.7% increase in performance time on large (50k) JSON formatting, over a variety of options.


The 'isFromCart' parameter was added when creating an order

Codecov Report

Merging #210 (56df47b) into master (3110960) will increase coverage by 0.23%.
The diff coverage is 50.00%.

:exclamation: Current head 56df47b differs from pull request most recent head d77c03f. Consider uploading reports for the commit d77c03f to get more accurate results

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##             master     #210      +/-   ##
============================================
+ Coverage     31.37%   31.61%   +0.23%
- Complexity     2652     2656       +4
============================================
  Files           115      115
  Lines          8015     8028      +13
============================================
+ Hits           2515     2538      +23
+ Misses         5500     5490      -10

Impacted Files
Coverage Δ

retailcrm/retailcrm.php
32.21% <ø> (ø)

retailcrm/lib/RetailcrmExport.php
22.37% <50.00%> (+1.50%)
:arrow_up:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


fs-uae: 2.8.3 -> 2.8.4

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae --help got 0 exit code
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae --version and found version 2.8.4
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae --help and found version 2.8.4
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae-device-helper -h got 0 exit code
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae-device-helper --help got 0 exit code
ran /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4/bin/fs-uae-device-helper help got 0 exit code
found 2.8.4 with grep in /nix/store/4n2cnrh2bq910wk4rlk1v7swjwhgq9jq-fs-uae-2.8.4
directory tree listing: https://gist.github.com/6996324bf1e34d7108f74089af68c981

cc @AndersonTorres for review

@GrahamcOfBorg build fsuae

No attempt on x86_64-darwin
The following builds were skipped because they don't evaluate on x86_64-darwin: fsuae
No log is available.

No attempt on aarch64-linux
The following builds were skipped because they don't evaluate on aarch64-linux: fsuae
No log is available.

Success on x86_64-linux (full log)
Attempted: fsuae
Partial log (click to expand)
make[1]: Leaving directory '/build/fs-uae-2.8.4'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4
shrinking /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4/bin/fs-uae
shrinking /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4/bin/fs-uae-device-helper
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4/bin
patching script interpreter paths in /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4
checking for references to /build in /nix/store/bdk975vvbq5jamw2dsvp88d01s5v70r0-fs-uae-2.8.4...


Angular treeview multiple nodes expanding at a time

How we asynchronously load children in angular

I'm sorry, can you explain the question a bit more on what you are trying to do?

I'm sorry, can you explain the question a bit more on what you are trying to do?


Err : Babel plugin 'react-optimized-image/plugin' not installed or this component could not be recognized by it.

I am also getting this error. I don't know why. @heet-vakharia have you found a solution yet?

no bro

@ohansFavour @heet-vakharia
hey guys, did you found a solution?

@ohansFavour @heet-vakharia
hey guys, did you found a solution?

Nope
On Tue, 12 Jan, 2021, 6:28 pm media-maven<EMAIL_ADDRESS>wrote:

@ohansFavour https://github.com/ohansFavour @heet-vakharia
https://github.com/heet-vakharia
hey guys, did you found a solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyrilwanner/react-optimized-image/issues/19#issuecomment-758638816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGALUKPQ6O3UJ6MWQL2PI3SZRBPRANCNFSM4UFI7H4A
.

Nope
On Tue, 12 Jan, 2021, 6:28 pm media-maven<EMAIL_ADDRESS>wrote:

@ohansFavour https://github.com/ohansFavour @heet-vakharia
https://github.com/heet-vakharia
hey guys, did you found a solution?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cyrilwanner/react-optimized-image/issues/19#issuecomment-758638816,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMGALUKPQ6O3UJ6MWQL2PI3SZRBPRANCNFSM4UFI7H4A
.

Same issue

Same issue

Same issue here

Same issue here

Any fix?

Any fix?

I'm using v2 for now. I have it working with these plugins: imagemin-mozjpeg, imagemin-optipng, imagemin-svgo, responsive-loader, sharp and webp-loader.

I'm using v2 for now. I have it working with these plugins: imagemin-mozjpeg, imagemin-optipng, imagemin-svgo, responsive-loader, sharp and webp-loader.

looks like it works fine with local images but not when the source is an external url

looks like it works fine with local images but not when the source is an external url

Same for me - the lib does not work as expected at first install :( which is sad

I'm using v2 for now. I have it working with these plugins: imagemin-mozjpeg, imagemin-optipng, imagemin-svgo, responsive-loader, sharp and webp-loader.

@bensbits91 could you please describe what's v2?

@t1gor I'm actually using next-optimized-images with a next.js project: https://github.com/cyrilwanner/next-optimized-images. Version 3 has similar issues, so I reverted to version 2, which is working well. I'm not sure if there's a version 2 of react-optimized-image

Allright, thanks.
In case someone is still intrested, I managed to get it to work without the lib at al:

Added customize-cra
Add config-overrides.js

const {override, addWebpackModuleRule} = require('customize-cra');

module.exports = override(
	addWebpackModuleRule({
		test: /\.(png|jpe?g|webp|tiff?)$/i,
		use: [
			'file-loader',
			{
				loader: 'webpack-image-resize-loader',
				options: {
					width: 1000,
				},
			},
		],
	}),
);

In components code, use it's params in way:

import StartupRocketMan from 'Assets/Images/startup.png?width=305&height=337&quality=85';

where width, height & quality are the params of the webpack loader. Check the docs for options: https://github.com/Calvin-LL/webpack-image-resize-loader#options
Hope that helps someone.

@cyrilwanner I have been able to diagnose at least one case in which this bug occurs.
If the require call is performed in a higher component then passed down to another component where the Img component is, the srcRaw attribute is not built. The require call must be sent directly to the Img.
This can be problematic if you want to create a component which receives an image to render as an argument, and performs several other functions besides rendering the .
An example:
export const ImageWithCaption = ({image, caption}) => {
    return (<>

      {caption}
    </>);
};

export const App = () => {
    return (

      <ImageWithCaption caption="This is a bird." image={require('./images/bird.png')} />

);
};

Here, the Babel plugin (which searches specifically for  components and reinterprets their src properties) is unable to apply the special data to the Img and the previously cited error occurs.
It is useful to know that this is the cause of the problem, but the result is that the component is highly restrictive in the places it can be used and reused.

@MasterEric thanks for those details - I was just trying to add the v3 for the first time to my app and got that error. I have that exact case, where a component receives the image src as a prop. Hopefully this can be fixed 🤞

I also need this fixed, I need to dynamically render Img with sources get from MDX but no matter what I try I get this error

On my side, I'm getting this error when I'm conditionally calling the image src file

It's frustrating and it needs to be fixed, but in the meantime, you could write this as a workaround:
If you were passing the ImgSrc data in some variable to your Img render function, such as
import LogoA from './images/logo.jpg';
const images = [LogoA, LogoB, ...];

// and rendering later:
<MyComponent src={images[0]} />

const MyComponent = (props) =>

you will need to replace that with
import LogoA from './images/logo.jpg';
const images = [(p) => , (p) => , ...];

// and rendering later:
<MyComponent img={images[0]} />

const MyComponent = (props) => <props.img alt="xyz" className="abc" />

but I agree we need that babel plugin fixed :-D

Hi,
I am also getting the above error has anyone figured out a clear solution for this?

Hi,
I have the same problem/

Same problem here and still no solution

Hi,
me too  have the same problem. Is there any fixes for that?

If you created your react app via create-react-app, you might have to rewire it to override the default bablerc  https://www.npmjs.com/package/react-app-rewired

same problem here.. how i wish there is noted as a known issue.. i could have save me time in the setup and then just facing this issue

Same Error

Facing same error, any luck with the solution?

Hey people! Found the solution.
https://youtu.be/3dzLIy8SPJg
I've fully followed the instructions from here and it now works just fine
the problem was the app wasn't reading the .babelrc file and that's the reason it had been showing the error "plugin not installed".


store: update goroutine leak check

The code this check was based on changed lately and we've been getting
some occurrences of a goroutine leak when running unit tests.
Update the code to match golang upstream 1, which waits a bit for
goroutines to finish, in the hope of getting rid of those issues.

Unrelated classic TestFetch failure on Semaphore.

lgtm on green


camera is missing in 0.5.0

it appears that camera module is being commented in 0.5.0:
//pub mod camera;
https://docs.rs/crate/ggez-goodies/0.5.0/source/src/lib.rs

I finally managed to make it work after directly selecting repo and branch in .toml file
ggez-goodies = { git="https://github.com/ggez/ggez-goodies", branch = "master"}

Yeah because it broke with the 0.4 -> 0.5 transition, and it didn't seem terribly useful in the first place.


Make login look nice.

The IRC examples looked great!
<klange> Makerimages: Also, I assert that the login screen looks naked without a background box        for the inputs: http://i.imgur.com/XHlQjZ6.png
<klange> Perhaps you would like the 50% transparent box rather than the 75% transparent box: http://i.imgur.com/x0mJouI.png

DUPE #63 !

Oh right... SO that's where I saw that....


Add Breakpoint option in tools

Description
It will be awesome if a breakpoint in tools include in next release of proxyman windows

The breakpoint is one of the most used features on Proxyman for mac. We definitely support it soon @coolerspeex 👍


Add onaccept handler to websocket rule

This gives an opportunity to deny websocket connection upgrade based on HTTP request.

Thank you for the PR. This is a nice addition.


bpf: revert changes to metrics directions contants

This commit partially reverts commit 38ab8f0 ("bpf: add support for
SERVICE metrics") by bringing the METRIC_INGRESS and METRIC_EGRESS
constants back to their original values.
In addition to that, it introduces the ct_to_metrics_dir helper which
allows to convert a CT direction into the corresponding metric one.
This mitigates any upgrade/downgrade impact caused by having keys with 2
different formats on the metrics map.
Fixes: #14175

test-me-please

test-me-please


Add categories for analysis options

nothing big, but @xavijam can you review?
@andrewxhill FYI, related to what we talked about the other day

:+1:

Frontend tests passing, so merging


Fix bug navigating away from settings page

Description
There was a bug that would crash the page if you opened a connection's settings in the Settings page, then attempted to navigate back to a normal page.
This was cause by svelte's animation attempting to close the connection settings pane before navigating.
This PR adds |local to the animation, which solves the issue.
Also adds a border to the db settings panel
Before

After

Checklist

[x] For UI or styling changes, I have added a screenshot or gif showing before & after
[x] I have added a changeset

Definitely

@csjh ok sweet I'll add that in here too

I'm also getting an error by default when navigating to the sql console - happens in a fresh install as well:


exllama v2: problem with Mixtral-8x7B GPTQ

System Info
GPU: A6000 48 GB
Image: ghcr.io/huggingface/text-generation-inference:1.3.2
Model: TheBloke/Mixtral-8x7B-Instruct-v0.1-GPTQ & TheBloke/Mixtral-SlimOrca-8x7B-GPTQ
Running docker run --gpus '"device=0"' ghcr.io/huggingface/text-generation-inference:1.3.2 --model-id TheBloke/Mixtral-8x7B-Instruct-v0.1-GPTQ --num-shard 1 --max-input-length 3696 --max-total-tokens 4096 --max-batch-prefill-tokens 4096 --max-best-of 1 --quantize gptq throws an error:
/opt/conda/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py:602: UserWarning: You are using a Backend <class 'text_generation_server.utils.dist.FakeGroup'> as a ProcessGroup. This usage is deprecated since PyTorch 2.0. Please use a public API of PyTorch Distributed instead.
  warnings.warn(
Traceback (most recent call last):

  File "/opt/conda/bin/text-generation-server", line 8, in <module>
    sys.exit(app())

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/cli.py", line 89, in serve
    server.serve(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 215, in serve
    asyncio.run(

  File "/opt/conda/lib/python3.10/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)

  File "/opt/conda/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
    return future.result()

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/server.py", line 161, in serve_inner
    model = get_model(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/__init__.py", line 310, in get_model
    return FlashMixtral(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_mixtral.py", line 21, in __init__
    super(FlashMixtral, self).__init__(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/flash_mistral.py", line 333, in __init__
    model = model_cls(config, weights)

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 820, in __init__
    self.model = MixtralModel(config, weights)

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 757, in __init__
    [

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 758, in <listcomp>
    MixtralLayer(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 692, in __init__
    self.self_attn = MixtralAttention(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 232, in __init__
    self.query_key_value = load_attention(config, prefix, weights)

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 125, in load_attention
    return _load_gqa(config, prefix, weights)

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/models/custom_modeling/flash_mixtral_modeling.py", line 158, in _load_gqa
    get_linear(weight, bias=None, quantize=config.quantize)

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/utils/layers.py", line 330, in get_linear
    linear = ExllamaQuantLinear(

  File "/opt/conda/lib/python3.10/site-packages/text_generation_server/utils/gptq/exllamav2.py", line 145, in __init__
    assert qzeros.shape == (

AssertionError

This seems to be a problem with exllama kernels v2: It is possible to start the GPTQ Version by explicitly setting EXLLAMA_VERSION=1, i.e. docker run --gpus '"device=0"' --shm-size 1g -e EXLLAMA_VERSION=1 ghcr.io/huggingface/text-generation-inference:1.3.2 --model-id TheBloke/Mixtral-8x7B-Instruct-v0.1-GPTQ --num-shard 1 --max-input-length 3696 --max-total-tokens 4096 --max-batch-prefill-tokens 4096 --max-best-of 1 --quantize gptq.
Information

[X] Docker
[ ] The CLI directly

Tasks

[X] An officially supported command
[X] My own modifications

Reproduction
Run docker run --gpus '"device=0"' ghcr.io/huggingface/text-generation-inference:1.3.2 --model-id TheBloke/Mixtral-8x7B-Instruct-v0.1-GPTQ --num-shard 1 --max-input-length 3696 --max-total-tokens 4096 --max-batch-prefill-tokens 4096 --max-best-of 1 --quantize gptq
Expected behavior
Works with exllama v2.

Maybe related to https://github.com/huggingface/text-generation-inference/issues/1355, attempting the same with sharding.

I don't believe TGI supports Mixtral yet.
There are added parameters - like number of chosen experts - that would need to be added to the code. Probably at a more abstracted level than Exllama.
I also don't see Mixtral mentioned in the exllamav2 repo

@RonanKMcGovern https://huggingface.co/blog/mixtral#using-text-generation-inference
TGI definitely has base support for Mixtral. I expect the issue is on the quantization/exllama side.

@nathan-az , yes, sorry. I meant TGI using GPTQ/exllama quantization.
TGI works with bnb nf4 and also eetq and also bf16.

I am encountering the same error, with the same cards. Sharding is working.

Having the same issue when running: TheBloke/Mixtral-8x7B-Instruct-v0.1-GPTQ


[DFP-182] Add JSON loader

Add support for loading JSON and newline-delimited JSON files into a view
Refactor CSV loader to share most of its functionality with the JSON loader
Expand CSV tests a bit to include files without headers and using non-comma delimiters

Should have mentioned that I'm explicitly not bumping the version, since I have another PR incoming soon. Not a big deal if someone else bumps it, I just don't need any of this functionality separate from that other PR.


Particle.js works well, but not in the correct div

Hi Everyone ! :smile_cat:
I'm currently working with particles.js (Short Documentation : https://github.com/VincentGarreau/particles.js/), on a one page template with consequently a few different section. So I'm trying to implement this js library in my website.
I know I use the lib correctly because it works well on my first div, on my first website page. (cf website screenshot: https://www.noelshack.com/2019-07-3-1550019179-first-div-first-page-no-problem.png (Yes, it's very light but you can see it haha) and the screenshot of the code : https://www.noelshack.com/2019-07-3-1550020026-first-div-first-page-no-problem-code.png).
It definitely works well. So, the problem is coming now.
When I try to do the same thing on an other div later in the code (on the background that a submission form) (cf screenshot : https://www.noelshack.com/2019-07-3-1550020687-div-problem-code.png https://www.noelshack.com/2019-07-3-1550020679-div-problem-website1.png), the module particle.js is printed at the bottom of my website page (cf screenshot : https://www.noelshack.com/2019-07-3-1550020758-div-problem-website2.png). It should be print in the background of the wanted div, as my first div (cf second screenshot), not at the bottom :thinking:
(Btw, yes the id name "particles-js1" is different than the first name id "particles-js" in the first screenshot, only because by deduction of my previous problems in this code, I think you can't call 2 times the same script "particle-js" in the same document) Anyway, that's not the point.
If someone can help me to fix this issue and help me understand why the lib particle.js is loaded at the bottom and not in the correct div background, I'll be very grateful.
Thank you in advance guys !

Anybody ?

Same for me, only working as the first div and seems not able to be added as background.

it should work if you set the css to position: fixed;
when i had it set to absolute i had tons of problems of it not rendering correctly

Same here, How do you fix the rendering issue?

Based on @ClintOxx 's comment, this worked for me:
#particles-js {
	width: 100%;
	height: 100%;
	background-size: cover;
	background-position: 50% 50%;
	position: fixed;
	z-index: 0;
}


Use shorter date format for updated at

Before:
Wed, 27 May 2020 17:57:14 +0000

After:
2020-05-27

(Note that I haven't tested it, but that's what it should look like.)
This PR was split out from #1248.

Looks like a to_string() might be needed or something like that?


how to build hloc-cpp?

I do not actively support the C++ implementation, but you can try to build it. We use Catkin, and the dependencies are mentioned in package.xml. Note that the C++ version does not support GPU feature matching.

@Skydes thank you for your reply


Stride parameter incorrect usage in PackPlanarWords

Stride must not be multiplied with sizeof(cmsUInt16Number)

should be fixed now, thanks


fix: [DirectoryWatcher] - Folders are case sensitive on Linux

PR Details
For some reason Path is transformed to lowercase, but on Linux distros it causes a runtime error - "The directory name .... does not exist" during game runtime.
Types of changes

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

Checklist

[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[ ] I have built and run the editor to try this change out.

Wouldn't it make sense to remove the ToLowerInvariant call entirely ? If it works on linux it will work on windows since it is the more lenient of the two ? Or do we have some specific handling per os here that breaks this assumption ?

                if (needProjectReload == false
                    && ((e.ChangeType == FileEventChangeType.Deleted || e.ChangeType == FileEventChangeType.Renamed || e.ChangeType == FileEventChangeType.Created)
                    && Path.GetExtension(changedFile)?.ToLowerInvariant() == ".cs"
                    && changedFileDirectoryName.StartsWith(directoryName, StringComparison.OrdinalIgnoreCase)))
                {
                    needProjectReload = true;
                }

would break other parts that have path comparisons

That example would not break on windows afaict though, on linux it may if you have two directory whose name differs only in character casing. What would break on windows is string comparison without IgnoreCase specified.

on linux it needs  a ordinal comparison, on windows it doesnt matter

What, then, do you finally suggest? I would like to avoid regression thus I added a IsLinux condition to not break anything

Okay, probably best to leave this for later, can you add a small comment above those lines mentioning that it might be safe to remove that check but to investigate that further on windows ?

Yeah, taking small steps 😅 Alright , I've added a TODO comment as you suggested.

Thanks !


Error executing ./tagui samples/1_yahoo chrome - fixed in cutting edge version

Warning: count(): Parameter must be an array or an object that implements Countable in /Users/awsw/Desktop/tagui/src/tagui_parse.php on line 31
Warning: count(): Parameter must be an array or an object that implements Countable in /Users/awsw/Desktop/tagui/src/tagui_parse.php on line 32
Warning: count(): Parameter must be an array or an object that implements Countable in /Users/awsw/Desktop/tagui/src/tagui_parse.php on line 34
ERROR - automation aborted due to above

https://github.com/kelaberetiv/TagUI/issues/597#issuecomment-546177231
This is due to the OS using php>=7.2. One solution is to use php<=7.1.

Adding on, alternatively, there is a fix implemented in TagUI cutting edge release. To use it, follow the steps in setup section for overwriting existing packaged installation with the cutting edge release codebase - https://github.com/kelaberetiv/TagUI#set-up
The packaged release for this fix is planned for release later this month.

Closing now but do feel free to reply if there is any problem with the fix.

Hi,
My next issue :
awsw@Andrews-MacBook-Pro src % pwd
/Users/awsw/desktop/Tagui-master/src
awsw@Andrews-MacBook-Pro src % ./tagui samples/1_yahoo chrome
./tagui: line 327: /usr/local/bin/casperjs: No such file or directory
awsw@Andrews-MacBook-Pro src % php -v
PHP 7.3.9 (cli) (built: Nov  9 2019 08:08:13) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.9, Copyright (c) 1998-2018 Zend Technologies
awsw@Andrews-MacBook-Pro src %

Google Chrome is up to date
Version 79.0.3945.88 (Official Build) (64-bit)

Hi @andrewwoo0902 can you raise a new issue with this info so that it's easy to add on troubleshooting info and also for other users to benefit from your issue? Thanks!
It could be because you are downloading master.zip directly. Check the Set Up section to make sure that you are downloading the packaged release for your OS.


no errors are found if the project name matches exclude_files

I tested Deptrac and named the folder test_deptrac, as a result,
whatever errors were actually, Deptrac did not find them. After
I renamed the folder, Deptrac started finding errors.
This seems like a very unpleasant bug, because of which you can
waste a lot of time and still not understand why errors are not found.
Config:
paths:
  - ./
exclude_files:
  - '#.*test.*#'
layers:
- name: A
  collectors:
  - type: className
    regex: .*A.*
- name: B
  collectors:
  - type: className
    regex: .*B.*

It seems both issues (with #599) are somehow related to the FileResolver::resolve and Finder from Symfony.
$finder = (new Finder())
            ->in($configuration->getPaths())
            ->name('*.php')
            ->files()
            ->followLinks()
            ->ignoreUnreadableDirs(true)
            ->ignoreVCS(true)
            ->notPath($configuration->getExcludeFiles());

From your configuration, it seems correct that deptrac will ignore the errors. You specifically requested it in the configuration file:
https://regex101.com/r/A0aPrP/1

From your configuration, it seems correct that deptrac will ignore the errors. You specifically requested it in the configuration file:
https://regex101.com/r/A0aPrP/1

This seems like a strange approach, since I wanted to exclude the
folder with tests, but Deptrac excluded the entire folder with the project,
although I run the analysis in the folder with the project, that is, it should
not matter what the name of this folder is.
Imagine that if I have a folder with name with the substring test in the path
to the project (not even project folder), is it really normal that there will be no errors?

Maybe you are looking for something like \/test\/.* or test\/.*? You can always use the debug command to see what classes are currently matched.

If you provide an example directory structure I might be able to help you with the configuration.

Maybe you are looking for something like \/test\/.* or test\/.*? You can always use the debug command to see what classes are currently matched.

It seems with \/test\/.*  everything works as expected, thanks.
I drew attention to it, as I could not understand why Deptrac does not
give mistakes for a long time, but since this is an expected behavior,
then thanks for explained.

Yeah, this is partially what the debug commands are for. To ensure that your matching patterns for both files and layers are as expected.

Thanks I'll know.


Remove Dependency on NodeJS

What was done

Removed bignum dependency and replaced with jsbn
Include browser-compatible buffer library
Created alias functions on jsbn.BigInteger so minimal code changes were required

Functions like sub --> subtract, eq --> equals, etc

All tests pass so it should be safe to use!

Woops. Didn't mean to PR on this project. Was supposed to be internal


New context menu option that can ask you to select the board before saving.

I would like to request a new feature. Adding a context menu option, similar to the current option that asks the user to save the selected thing to which board and list, rather than directly saving to settings all the time.
There may be sometime reason to save to another board or another list. But because of settings it always saves to the same board and list. It would be great

Yeah this would be a cool feature. I imagine that the user could add new board/list combinations in the options menu, in some sort of list. Clicking the extension button would use the first one in the list. All the other combinations would only show up in the right-click menu. The combinations could be reordered in the options dialog with drag-and-drop.
I won't spend time on this for now, since I don't have the time and it's mostly outside of what this extension was intended for. Feel free to create a pull request though!


0.98.3 - "impl/async_query.hpp" file not found

In General
Hello, I have updated via cocoa pods to the latest version of the library (0.98.3), but when I try to build the project I'm getting a compile error, some files are not being found :/ . Any ideas ??
PS: Attaching info below
PS 2: It works up to Realm version 0.98.2 (confirmed)
Expected Results
Being able to use the latest version or the library
Actual Results
Getting a compile error
"impl/async_query.hpp" file not found on file "async_query.cpp"
Steps to Reproduce
Just updated the pods
( "pod update"
Installing Realm 0.98.3 (was 0.97.1)
Installing RealmSwift 0.98.3 (was 0.97.1)
)
and then build the project (I've tried cleaning the project and erasing the derived data).
Code Sample
On source file : async_query.cpp
Line : 19  ( #include "impl/async_query.hpp" )
Version of Realm and Tooling
ProductName:	Mac OS X
ProductVersion:	10.11.3
BuildVersion:	15D21

/Applications/Xcode.app/Contents/Developer
Xcode 7.2
Build version 7C68

/usr/local/bin/pod
0.39.0
Realm (0.98.3)
RealmSwift (0.98.3)

/bin/bash
GNU bash, version 3.2.57(1)-release (x86_64-apple-darwin15)

/usr/bin/git
git version 2.5.4 (Apple Git-61)

Please try to clean your build folder, reset your CocoaPods integration (rm -rf Pods) and clear the CocoaPods cache. (rm -rf ~/Library/Caches/CocoaPods)

Amazing! Cleaning the CocoaPods cache (rm -rf ~/Library/Caches/CocoaPods) did the trick !
Thanks!
Marking as closed

just wanted to add, I had to remove the Pods directory also to get the file to be redownloaded/seen


Update install_cpython.sh

from https://github.com/pytorch/builder/commit/000114fc93c9bc2817e0c4cd1bac2988609bf029

@ethanwee1 This change was applicable for py3.13. We are not planning to support 3.13 in rocm6.2 builder branch.


Sorbet wrongly flags Array.sample with no n, random argument

Input
https://sorbet.run/#def main
  a %3D [1%2C 4]
  a.sample(random%3A Random%3A%3Anew())%0Aend
def main
  a = [1, 4]
  a.sample(random: Random::new())
end

Observed output
editor.rb:3: Expected Integer but found {random: Random} for argument arg0 https://srb.help/7002
     3 |  a.sample(random: Random::new())
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    https://github.com/sorbet/sorbet/tree/master/rbi/core/array.rbi#L1832: Method Array#sample (overload.1) has specified arg0 as Integer
    1832 |        arg0: Integer,
                  ^^^^
  Got {random: Random} originating from:
    editor.rb:3:
     3 |  a.sample(random: Random::new())
                   ^^^^^^^^^^^^^^^^^^^^^
Errors: 1

Expected behavior
This should pass, it's one of the documented signatures.

https://sorbet.org/docs/faq#it-looks-like-sorbet-s-types-for-the-stdlib-are-wrong

This was fixed in https://github.com/sorbet/sorbet/pull/2430


Add hints to contact form

Add a script that scans the message typed into the contact form for
common problem descriptions and suggests how the user could resolve the
problem himself instead of messaging us. See the Demo.
At the moment I'm not sure how to test this, since it's purely an UI thing and we don't have any automated UI testing integrated in our pipeline.
TODO:

[ ] Run the tests and see them pass
[ ] Rebase your branch on top of develop

@sebschrader very cool, didn't know about embedded JSON. But Locale can't be serialized as JSON, so I converted it to string first (since the only thing we need is the country code).

The JSON <script> tag won't block HTML parsing/rendering as a JavaScript <script> tag would, which must be executed immediately at the location it was specified in the HTML stream, because some evil genius could use document.write to add HTML right there with JavaScript.

I cannot test this ATM due to #354.


Dragon conversion completely non-functional

Threw this into a big pack to test out, the dragon fireball conversion wasn't working so I went to test it on its own and yeah, it doesn't work in general. I'm on 1.20.1 Fabric 0.14.22 with version 1.1.0 of this mod.
First thing I thought to look for is the tags, while the ore replaceables was loaded, dragon_made_ores does not exist in the mod folder. Unfortunately, that didn't fix my issue, and the mod still doesn't convert any obsidian into the ore.
I've attached a log but it's basically empty. I test mods with Mod Menu, Jade, EMI, StackDeobfuscator, Better Log4j Config, YACL and Dashloader. Afaik none of these should be interacting with your mod's ability to alter the world.
latest.log
At some point while testing I actually got a crash but I have no idea what caused it. The game crashed as soon as one of the fireballs hit the obsidian. I have Carpet, Neruina and Collective loaded here but again, I don't suspect these are the reason at all.
crash-2023-08-27_04.05.54-server.txt
Unfortunately I no longer have the log that preceded the crash.

Yes I've encountered this error as well, we've hopefully fixed it in the 1.1.1 patch.


"Azure Data Lake Storage REST interface" link gives 404

The link points to this URL which results in a 404 error - https://docs.microsoft.com/en-us/rest/api/datalakestorage/
I think the correct link is this https://docs.microsoft.com/en-us/rest/api/storageservices/data-lake-storage-gen2

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

ID: 4ca909b0-2d80-ca90-52d4-c3d7958040d9
Version Independent ID: b5c888da-ff2e-7fc2-3c36-e0eb22132932
Content: The Azure Blob Filesystem driver for Azure Data Lake Storage Gen2 Preview
Content Source: articles/storage/data-lake-storage/abfs-driver.md
Service: storage
GitHub Login: @jamesbak
Microsoft Alias: jamesbak

@arsenvlad Thanks for reporting! I will work to fix it.


how to get errored records from bulk index

Please use the following questions as a guideline to help me answer
your issue/question without further inquiry. Thank you.
Which version of Elastic are you using?
[ ] elastic.v2 (for Elasticsearch 1.x)
[ ] elastic.v3 (for Elasticsearch 2.x)
[x ] elastic.v5 (for Elasticsearch 5.x)
Please describe the expected behavior
This is just a question not a bug. When doing a bulk index how do you know which records failed? I.e. is it possible to get the records out of bulkResponse.Failed() ? Also, ifs there a way to force failure for testing?
Please describe the actual behavior
Any steps to reproduce the behavior?

Elasticsearch returns errors that failed in an array. The BulkResponse.Errors flag indicates whether any errors happened at all. The BulkResponse.Failed() will return all items that had an error. You can use the Id property to get a back-reference to the original document, retry or do whatever is appropriate for your application.
You should probably not test Elasticsearch and its bulk operation capabilities but the service that uses it. You could e.g. do that by defining an interface and provide two implementations: One for a live environment and one (mocked) implementation that provides callbacks for your testing code.


feat: As a user, I want to see the tutorial UI

What does this PR do?
Related ticket
Fixes #575
Type of change

[ ] Chore
[ ] Bug fix
[X] New feature
[ ] Documentation update

Check list

[ ] Unit tests for any smart contract change
[ ] Contracts and functions are documented

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 93.63%. Comparing base (8502ea4) to head (2bbaedb).
Report is 2 commits behind head on dev.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #576   +/-   ##
=======================================
  Coverage   93.63%   93.63%
=======================================
  Files          20       20
  Lines         330      330
  Branches       76       76
=======================================
  Hits          309      309
  Misses         10       10
  Partials       11       11

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Can this model do qa tasks？

I find that the model seems to have a hard time output options, my prompt is：
Detect and report the timestamp of the video frame that semantically matches the given textual query "puts the bread on the plate ".
Choose the correct option to the following question:
(0) 33.59402815508409, (1) 37.721428155084084, (2) 39.995228155084085, (3) 50.98892815508409, (4) 55.96002815508409,
Best Option: (

Hi, thanks for your interest.
I think you should do some prompt engineering, like:
text_input = "In the context of a culinary event where someone 'puts the bread on the plate', determine the closest time from the following options:"\
"(A) 33.5 seconds; (B) 37.7 seconds; (C) 39.9 seconds; (D) 50.9 seconds; (E) 55.9 seconds."\
"Please select the correct option by responding with the corresponding letter (A)-(E). Avoid outputting anything other than the designated letters."

The output is:
The closest time is (B) 37.7 seconds.

@leexinhao Hi, we found that reduce lora_alpha from 32 to 20 during inference can be very helpful for qa tasks. See https://github.com/RenShuhuai-Andy/TimeChat/blob/master/docs/FAQ.md#3-how-to-better-instruct-the-model-to-perform-qa-or-other-specialized-tasks


Fix Style/ClassAndModuleChildren for nested classes with explicit superclass

Fixes https://github.com/bbatsov/rubocop/issues/3112

Thanks!


feat: add support for timeZone parameter

This PR adds support to pass through the timeZone parameter into IntlProvider.
This could be necessary for users using the Storybook with snapshot testing, where the time zone can affect how things are rendered. In that situation, it's better to fix the time zone to a constant value independent of the computer it's running on.

I plan on adding another commit to this to update the readme before marking it as ready.

@danielhep Seems like a simple thing. Can you finish the README update so I can merge and release?


In simple test React app, query returns only "Promise {}"

My app is extremely simple, for no reason other than to test it out. However, no matter how I format the query, nothing is returned.
import React, { useState, useEffect } from 'react';
import Dexie from "dexie";

import './App.css';

const db = new Dexie("yawndb");
db.version(1).stores({
	tasks: "++id, data"
})

function App() {
	const [field, setField] = useState("");
	const [incoming, setIncoming] = useState("");

	const doInput = e => {
		setField(e.target.value)
		// console.log(e.target.value);
	}

	const getTest = async e => {
		e.preventDefault();
		const giving = await db.tasks.put({ data: field})
		const getting = await db.tasks.where("data").startsWith("b").sortBy("data");
		console.log(25, field, giving, getting);
	}

	useEffect(() => {
		async function getdata() {await db.tasks.where("data").startsWith("b").sortBy("data");}
		// const getdata = async () => {await db.tasks.where("data").startsWith("b").sortBy("data");}
		// const getdata = async () => {await db.tasks.where("id").equals(17);}
		// console.log(28,JSON.stringify(getdata(),null,2));
		console.log(34,getdata());
	},[incoming])

	return (
		<div className="App">
			<form onSubmit={getTest}>
				<input onChange={doInput} id="testfield" type="text" placeholder=" " name="field" value={field} />
			</form>
		</div>
	);
}

export default App;

That which is shown here is my last attempts of several dozen to get the output. I've no idea what I tried before hand.
I can see from the Dev Tools > IndexedDB that the data is making it into the store. . .
yet the query results:

The the console line that begins "25" is from getTest. The number, a state variable, the return from the put, and the return from the query (empty array).
What am I missing?

Ok well . . . just like it always works, walk away and give it a minute (or a good night sleep) and the solution hits you in the face like an exwife.
I wasn't setting a state variable, so the data was there, but not being presented to the useEffect() rerender.
Solution:
inside of getTest() before the conlog I added
setIncoming(getting)
and then in the useEffect() i added
console.log(34, incoming)
So the big lesson is that useEffect() should not be used for querying AND displaying the resulting data.
I will make this code presentable and submit a PR for addition to your examples. I'm using React Hook syntax, of which I could find no examples.


Added skip-inverse-transform tag for Imputer and HampelFilter

Reference Issues/PRs
#744
What does this implement/fix? Explain your changes.
Skip inverse_transform for HampelFilter and Imputer in TransformedTargetForecaster

Martin Walter<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md

@mloning seems there is an issue with brew install libomp, unrelated to my PR.

Martin Walter<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md

@mloning seems there is an issue with brew install libomp, unrelated to my PR.
Martin Walter<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md

seems libomp is back working. Will fix the other issues then.

ready to merge :)

Martin Walter<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of Daimler TSS GmbH.
https://github.com/Daimler/daimler-foss/blob/master/LEGAL_IMPRINT.md


🐛 Preview stretched / zoomed-in on android

What's happening?
I'm creating a new issue with the same topic as this one : https://github.com/mrousavy/react-native-vision-camera/issues/2583
because on samsung device, taken picture is still zoomed in / stretched in my case (ratio 1 and 0.625)
I have issues with square and the ratio used in the example app.
I have made a small example to have side by side preview and taken picture (preview on the left)
With back camera

With front camera

Reproduceable Code
Here is a repo to reproduce the issue
https://github.com/mlecoq/VisionCameraPreviewIssue
Relevant log output
1142  1142 E cdsprpcd: vendor/qcom/proprietary/adsprpc/src/fastrpc_apps_user.c:2707: Error 0x200: open_dev (-1) failed for domain 3 (errno Connection refused)
03-25 17:13:45.447  1142  1142 E cdsprpcd: vendor/qcom/proprietary/adsprpc/src/fastrpc_apps_user.c:1215: Error 0x200: remote_handle_open failed for adsp_default_listener (errno Success)
03-25 17:13:45.447  1142  1142 I cdsprpcd: vendor/qcom/proprietary/adsprpc/src/rpcmem_android.c:191: rpcmem_deinit_internal: closed ION fd 5
03-25 17:13:45.448  1142  1142 I cdsprpcd: vendor/qcom/proprietary/adsprpc/src/fastrpc_apps_user.c:2756: fastrpc_apps_user_deinit done
03-25 17:13:45.449  1142  1142 E cdsprpcd: vendor/qcom/proprietary/adsprpc/src/cdsprpcd.c:47:cdsp daemon will restart after 100ms...
03-25 17:13:45.454   823  1388 E CamX    : [ERROR][STATS_AF ] af_alg_util.cpp:480: af_alg_util_get_fovc_factor() [Jake] pSettings = 0xe776f984
03-25 17:13:45.466  1184  1184 E cnss-daemon: Failed to get sock name Socket operation on non-socket
03-25 17:13:45.474   823  1396 I CHI     : [SS_INFO]: [CHI         ]: chxusecase.cpp: ExecuteCaptureRequest: 2401: [CaptureRequest] stream = 0xeae66dd8 streamType = 0 streamFormat = 0x22 streamWidth = 1280 streamHeight = 960
03-25 17:13:45.474   823  1396 E CHI     : [SS_ERR ]: [CHI         ]: chxusecase.cpp: ExecuteCaptureRequest: 2447: ExecuteCaptureRequest: pMetaData is NULL
03-25 17:13:45.474   823  1396 I CHI     : [SS_INFO]: [CHI         ]: chxusecase.cpp: ExecuteCaptureRequest: 2456: Default usecase capture request for frame 22016 with 1 output buffers
03-25 17:13:45.487  1184  1184 E cnss-daemon: Failed to get sock name Socket operation on non-socket
03-25 17:13:45.506  1184  1184 E cnss-daemon: Failed to get sock name Socket operation on non-socket
03-25 17:13:45.527 22427 25346 I CameraView.takePhoto: Successfully captured 2976 x 2976 photo!
03-25 17:13:45.529 22427  2538 I ReactNativeJS: /data/user/0/com.visioncamerapreviewissue/cache/mrousavy-9182797906249792109.jpg

Camera Device
{
  "formats": [],
  "sensorOrientation": "landscape-left",
  "hardwareLevel": "full",
  "maxZoom": 8,
  "minZoom": 1,
  "maxExposure": 20,
  "supportsLowLightBoost": false,
  "neutralZoom": 1,
  "physicalDevices": [
    "wide-angle-camera"
  ],
  "supportsFocus": true,
  "supportsRawCapture": false,
  "isMultiCam": false,
  "minFocusDistance": 0,
  "minExposure": -20,
  "name": "0 (BACK) androidx.camera.camera2",
  "hasFlash": false,
  "hasTorch": false,
  "position": "back",
  "id": "0"
}

Device
Samsung Galaxy Tab S5e | SM-T720
VisionCamera Version
4.0.0-beta.10
Can you reproduce this issue in the VisionCamera Example app?
Yes, with some modifications, see my comment below
Additional information

[ ] I am using Expo
[ ] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

I think it's more visible because I'm using a tablet but I also see small differences on my redmi

I'm also experiencing this issue while testing 4.0.0-beta.11 on Galaxy A01

can you guys reproduce this on the example app in v4 branch?

can you guys reproduce this on the example app in v4 branch?

I have just tested with example app. First I didn't reproduce the issue (even if I have a small preview with a ratio of 0.625), but then I have tried to have a similar display as my example : preview and result on the same screen. And now I reproduce the issue, particularly when I remove frame Processor
Here is the CameraPage I use in the example app on v4 branch to reproduce the issue:
https://gist.github.com/mlecoq/94ffbfb4803cddea58860e379d9a5eb6.js

Maybe I am not using correctly react-native-vision-camera but everything is fine on iOS so I was expecting a similar behavior on android

Sorry you all. After more testing, I realized I was setting the wrong photoAspectRatio.

@mlecoq how about you - are you using different photo sizes?

@mrousavy in the example I made, I use as aspectRatio the same one as used to display preview and taken picture
In the example of react-native-vision-camera I have seen that used aspect-ratio depends on screen size, so I have tried with it (see gist here https://github.com/mrousavy/react-native-vision-camera/issues/2690#issuecomment-2028490665) but I have still the issue

The capture is good, on the preview I have an elongated face

Yea because photo size doesn't have anything to do with the video stream.
I think what you're seeing is all expected.

Do you mean that there is an issue on how I use  ? Because I don't see this preview issue on iOS with the same code and on my Redmi neither.

what also surprise me (but there is maybe a technical limitation), is that given photoAspectRatio is not applied on taking a picture.
For example I have set a ratio of 0.625 (like preview in the capture below) and the taken picture has a width of 720 and a height of 1280, so a ratio of 0.5625 (it's why we see black stripes on the captures image)
So what you see with preview is not what you get. For the moment I mitigate this by cropping the image (to have expected ratio) but the result is zoomed in.

when I use screen aspect ratio, the result looks worse (taken picture has a width of 720 and a height of 1600)

I am wondering if this https://developer.android.com/media/camera/camerax/configuration?hl=en#crop could be used to solve this issue

Any solution about this issue?

Hi everyone,
I'm also facing the same preview stretched issue on stock android device, for other device and for ios it is working fine.

Did you found any solution?

Switching to beta version as mention in [https://github.com/mrousavy/react-native-vision-camera/issues/2583] solved my issue.

Switching to beta version solved my issue.

you're welcome 👍

Try this hack if you don't wanna install the beta version.

Hey - I think this issue has been fixed in VisionCamera 4.0.0. 🥳
Please try V4 and let me know if you still experience this issue;

if not, please consider 💖 sponsoring me on GitHub 💖 to support the development of VisionCamera and thank me for my time spent on fixing bugs and building new features.
if you still see this issue, please comment and we can re-open this. But please update your native logs with the native (Xcode/Android Studio) logs from running VisionCamera V4 so I can investigate this.


getExtent is not a function

ol-source-wmts dose not work for me
I use this
<ol-tile-layer>
  <ol-source-wmts
    layer="satImage"
    :projection="{
      code: 'EPSG:4490',
      extent: [-180, -90, 180, 90],
      global: true,
      units: 'degrees',
    }"
    url="http://xxx"
  ></ol-source-wmts>
</ol-tile-layer>

or this
<ol-tile-layer>
  <ol-source-wmts
    layer="satImage"
    projection="EPSG:4490"
    url="http://xxx"
  ></ol-source-wmts>
</ol-tile-layer>

Both do not work. And before this code , I had use this
<ol-projection-register
    projectionName="EPSG:4490"
    :projectionExtent="[-180, -90, 180, 90]"
    projectionDef='GEOGCS["China Geodetic Coordinate System 2000",DATUM["China_2000",SPHEROID["CGCS2000",6378137,298.257222101,AUTHORITY["EPSG","1024"]],AUTHORITY["EPSG","1043"]],PRIMEM["Greenwich",0,AUTHORITY["EPSG","8901"]],UNIT["degree",0.0174532925199433,AUTHORITY["EPSG","9122"]],AUTHORITY["EPSG","4490"]]'
  />

In Chrome I got this Error
Uncaught (in promise) TypeError: get$3(...).getExtent is not a function

ok, maybe I fix it.
I try to add this in Projection.vue,then it work
import { addProjection } from 'ol/proj'
addProjection(projection)


Fix service wrapper init for metrics

Some components like ActivityTaskHandler are not using metrics wrapped service client. That's the reason we emit part of operation metrics. Fix this by init wrapped service client at worker init.

Coverage decreased (-0.03%) to 68.231% when pulling 9ef49dc023db86027faaad01ebdd82cdc34646f9 on serviceMetrics into 59d07340413212fe62387413c1f62d783cac34c9 on master.


feat: add MapView.clearData

Fixes: #2797
Unfortunately it's seems that CacheManger was removed from early iOS sdk.
There is a new v10 only method clearData see https://docs.mapbox.com/ios/maps/api/10.12.2/Classes/MapboxMap.html#/s:10MapboxMaps0A3MapC9clearData10completionyys5Error_pSgc_tF
https://github.com/mapbox/mapbox-maps-ios/commit/5e79f92d061172998b583e00967a71d3c4c93d1d
https://docs.mapbox.com/ios/maps/api/10.0.0-beta.20/Extensions/CacheManager.html#/s:So15MBMCacheManagerC10MapboxMapsE17clearAmbientCacheyyys6ResultOyyts5Error_pGcF

@mfazekas I think it would be useful to be able to use clearData without needing a map reference. Similar to MapboxOfflineManager.resetDatabase(). I think this is possible as clearData are exposed statically on both iOS & Android.

Ah sorry I missed that little detail from the static variant. Ok then it might be more effect to ensure the right options.
I as looking for an alternative for invalidateAmbientCache to basically remove the cached tile urls on-demand as this was available statically pre-v10 I thought clearData would be the correct way of doing so.


where is the config file?

Do you want me to deploy it again? If there's some demand I can do it.

I'm beginner in programing and I want to run your bot on my own herouku account can you help me with some information about  how to configure the bot and run it

config.js includes some configuration variables about the bot. Sure, you can. I'll add a configuration file here, where you can replace the placeholder values with your own values. Other than I think it's good to go. However, it's buggy and there are no tests, so beware while running this. I will also recommend you to check some bots written by other developers for code inspiration :+1:

if it is buggy can you add some comments like
//put your token here


Terrible scaling

Terrible scaling for 4k

Same here.
Also unable to grab a column to resize.

Same here on both

Would be great to have a setting to change the overall font & line sizing.
I'd love to have the Log on the bottom of Atom. A bit like PHPStorm does it.
But the overall sizing is just waaay to huge.


F1, F2, F3, F4 has stopped working

This worked before OS X 10.10 Yosemite.

Confirmed in macOS 10.15 Catalina


Setup on Windows

On a windows workstation with VSCode installed what are the dependencies for this?
Does cfn-lint have to be installed on Windows or can it be installed using PIP in the MINGW64 shell?
It would be useful to understand how this extension calls into cfn-lint.

so I went through this morning.  It has to be installed via pip.  I installed using python though (https://www.python.org/downloads/windows/) so it was integrated with my standard powershell and my path ended up having cfn-lint available so it just worked.

Closing as this is explained here.


Difference with Earth2-mip

Hi,
Thanks for putting together all these fantastic AI weather models.
I'm wondering what's the difference between Earth2-mip and Earth2-studio. Will the Earth2-mip still be maintained alongside with Earth2-studio?

TLDR; Earth2Studio is bring-your-own model, bring-your-own data, bring-your-own workflow. Earth2MIP is primarily bring-your-own model.
To elaborate
The philosophy behind Earth2Studio is to provide building blocks for users to build their own workflows. In this package we focus on the following building blocks: data sources, model wrappers, IO handling, common utilities. Earth2MIP is primarily targeted at providing specific workflows, usually with specific metrics or plotting scripts associated with those workflows.
Earth2Studio is actively maintained and has ongoing software engineering support.


ENH: add BIDS output version checker

started as https://github.com/poldracklab/fmriprep/issues/1526, but should be extendible to any package producing BIDS compliant outputs

this is all set

I based this on the assumption the dataset_description.json would already be in that pipeline's output directory, but could add a secondary check if desired

Yeah, this is fine. We can add features in the future.