[BUG] org.opensearch.cache.store.disk.EhCacheDiskCacheTests.classMethod is flaky

Describe the bug
The test case org.opensearch.cache.store.disk.EhCacheDiskCacheTests.classMethod is flaky:
com.carrotsearch.randomizedtesting.ThreadLeakError: There are still zombie threads that couldn't be terminated:
   1) Thread[id=81, name=MappedByteBufferSource Async Flush Thread, state=WAITING, group=TGRP-EhCacheDiskCacheTests]
        at java.base/jdk.internal.misc.Unsafe.park(Native Method)
        at java.base/java.util.concurrent.locks.LockSupport.park(LockSupport.java:371)
        at java.base/java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionNode.block(AbstractQueuedSynchronizer.java:519)
        at java.base/java.util.concurrent.ForkJoinPool.unmanagedBlock(ForkJoinPool.java:3780)
        at java.base/java.util.concurrent.ForkJoinPool.managedBlock(ForkJoinPool.java:3725)
        at java.base/java.util.concurrent.locks.AbstractQueuedSynchronizer$ConditionObject.await(AbstractQueuedSynchronizer.java:1707)
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$DelayedWorkQueue.take(ScheduledThreadPoolExecutor.java:1170)
        at java.base/java.util.concurrent.ScheduledThreadPoolExecutor$DelayedWorkQueue.take(ScheduledThreadPoolExecutor.java:899)
        at java.base/java.util.concurrent.ThreadPoolExecutor.getTask(ThreadPoolExecutor.java:1070)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1130)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642)
        at java.base/java.lang.Thread.run(Thread.java:1583)

Related component
Storage:Remote
To Reproduce
./gradlew ':plugins:cache-ehcache:test' --tests "org.opensearch.cache.store.disk.EhCacheDiskCacheTests" -Dtests.seed=4D5D98F56511FEC6

Expected behavior
The test must always pass
Additional Details
Plugins
Please list all plugins currently enabled.
Screenshots
Standard
Host/Environment (please complete the following information):

CI

Additional context

https://build.ci.opensearch.org/job/gradle-check/35254/testReport/junit/org.opensearch.cache.store.disk/EhCacheDiskCacheTests/classMethod/

@sgup432 please take a look, may be caused by https://github.com/opensearch-project/OpenSearch/pull/12734

[Triage - attendees 1 2 3 4 5 6]
Fixed by #12764


add new oxidation states for Cl, Br, I, Xe, Po, At

Also changes the allowed value 2->3 for I
These all regularly show up as hypervalent species, might as well make it less painful to use them.

I wished to this myself but I did not dare to :-)
I think you also want to do it for Cl then.

 good point. It's there

I'm going to add the relevant entry to the ReleaseNotes in a separate commit that I'm doing in preparation for the upcoming release.

@ptosco look ok?

@greglandrum LGTM, just merged.

What's the best way to flag molecules that have unusual valences? This change breaks a lot of our molecule generation code because now we get species like C#Cl that parse just fine even though we clearly don't want to be generating stuff like that...
I suppose we could check for halogen valences explicitly, but I'm wondering if there's a way to globally adjust the sanitization to use a "preferred" valence or set of valences?

@skearnes you're absolutely right. We overshot on this one and need to at least partially back the changes out. At the very least the extra valence states for Cl should probably be removed. Anyone who wants/needs to parse those can still use the usual RDKit approach for handling hypervalent species: triggering the sanitization manually.

Reverting the changes for Cl and Br is probably the best compromise solution, as C#I is less likely to appear in standard generative chemistry workflows. I hadn't thought of this unintended consequence.

Note: partially reverted by #3545

Note: partially reverted by #3545


I have two  questions about the turtlebot view and pedestrain

1、In the robot view, the above and below of the image are black, this may influence the learning,can I adjust it?
2、I want to import some Pedestrains in the scene, then how can I drive the Pedestrain with the social-forces model ORCA？
Is there any tutorial for these two questions？

Hello, which operating system did you use to install igibson? Which version? thank you

ubuntu20.04, igibson = 2.1.0

@THU-Bohr ubuntu20.04 and igibson=2.1.0

@Gengshucheng , for the robot issue, try setting it to be hidden https://github.com/StanfordVL/iGibson/blob/master/igibson/render/mesh_renderer/mesh_renderer_settings.py#L27
The best way to set this parameter will depend on how you create your environment
For the use of simulated humans, I defer to some of my colleagues, I haven't use them much
Thanks!

(in case it was not clear, the black stripes is the robot. these parts of the robot are visible in the images from the camera of the real robot because of its position, but if you are not planning to do sim2real, you may make the robot to be hidden for the camera with the rendering option I mentioned)

@roberto-martinmartin  thanks，It's useful for me.

@Gengshucheng For simulating pedestrians with ORCA, you can refer to this file:
https://github.com/StanfordVL/iGibson/blob/cvpr21_challenge/gibson2/tasks/social_nav_random_task.py
Make sure you install ORCA first: https://github.com/sybrenstuvel/Python-RVO2


No rewards for staking pool delegation

To make slot length and epoch size configurable in plutip, module Cluster.hs was brought (copy-pasted) from cardano-wallet local cluster framework into plutip. After this was done receiving rewards for staking pool delegation stopped working.
Changes in Cluster.hs are minimal and should not affect rewards. But if cluster launch performed with original Cluster.hs rewards starts working again. It's not yet clear what is broken.
Branch forked from master, where cluster launch reverted back to use module from cardano-wallet and rewards working:
 plutip-debug
https://github.com/mlabs-haskell/plutip/commit/8e26b47cef5b51d97929d16ef6e68239c7cc5d0b
Where it breaks in master:
dc5c9440fe2a66d533f383bd6f6477ad764da1f9 - works
650e796dffc1ac502311260a56f6f67f2cf80dcf - does not

This is breaking due to epochLength. slotLength can be freely configured provided epochLength remains at 80

For context: here's a skipped test that reproduces this:
https://github.com/Plutonomicon/cardano-transaction-lib/blob/0576feabc1c7ec03eceb0d65a09fec4b26939322/test/Plutip/Staking.purs#L380


Undo/history and Multi Language

The multi-language support is causing issues with undo. We've had page's plugins all be deleted because of an edit on some other language variant for a page only for undo to delete the first language's content.
The consistency of the language system has me baffled.

@doctormo any further help would be appreciated, we cannot reproduce it this far. Would you be able to provide video captures?

Hello,
I'm another contributor of the Inkscape website.
Here are two videos:

https://files.cloud.orange.fr/cloudUpDown/V06.00.08/UpDownCloud/downloadFileAnonymous?fileId=__upload%2Fdjango-cms-undo-bug-long.ogv&shareToken=SjMxQKt86u1077019c7c
https://files.cloud.orange.fr/cloudUpDown/V06.00.08/UpDownCloud/downloadFileAnonymous?fileId=__upload%2Fdjango-cms-undo-bug.ogv&shareToken=SjMxQKt86u1077019c7c

I launched randomly the first video and had fun with a local copy of the Inkscape website around Martin's procedure. I create an English page and a French translation, then I add a text plugin into it at the same time. (I also added a Spanish translation, but it is useless.) Then I click on History > Undo on the French translation, this doesn't change the French translation and makes the text plugin disappear in the English one. I didn't publish any page.
The second video is more focused on the procedure: I create a page in English and a French translation, I save a text plugin into each of them still at the same time. Again, I forgot to publish and I did History > Undo on the French translation and the result, again, is the French page unchanged, and the English text plugin not removed but empty (well, I discovered it by publishing the English page — it's a mess at the end).
Note: please copy the videos elsewhere if you need archives; I used my personal cloud to store them and I probably won't keep these files more than a year.

Please admire how the settings in our shield plugin disappeared thanks to a click on ‘Revert to Live’ from a translator after he had wrongly put his translation on the English plugin:
http://i.imgur.com/V4JixQG.png
Saved state on a local instance:
http://i.imgur.com/sKKUPgY.png
It seems the data cannot be restored by an additional revert.
Thanks for marking this bug as a blocker, hope it'll be fixed soon.

This might be related to #5722 as the plugin uses a reverse foreignkey collection for tabs. While this is covered in copy_relations, reversion doesn't know anything about it at all.

This was one of the shortcomings of the old revision system.
It wasn't multilingual so when undoing changes in one language, it would undo on all.
This has been addressed by the undo/redo functionality of djangocms-history.

@czpython : sorry for the off-topic question: is djangocms-history a replacement for the removed history / revision in django-cms proper? (We don't have a page history anymore in django 1.8.17, and no undo/redo, but it would be nice to have it back.)

Hello @Moini,
djangocms-history adds back the undo/redo functionality to the cms but its not a direct replacement of revisions because you can't recover deleted pages or go back in history more than 24h.

Thank you, @czpython ! Good to know, I hope we can include it :)


Fix up error message when compilation references are missing

Follow up to https://github.com/aspnet/Razor/pull/2405. The current error message isn't very accurate


Possible logic error for exported targets

In the following code chunk, there is a possibility a target is found but explicitly disabled. For example, I don't think you would append openmp if ENABLE_OPENMP=OFF despite OPENMP_FOUND=ON. Perhaps change OR to AND?
https://github.com/LLNL/axom/blob/9410ed3db64ca1ab33b6053d8fd3e452c20bb562/src/cmake/AxomConfig.cmake#L165C23-L165C23
set(_optional_targets cli11 fmt hdf5 lua openmp sol sparsehash)
foreach(_tar ${_optional_targets})
    string(TOUPPER ${_tar} _upper_tar)
    if(ENABLE_${_upper_tar} OR ${_upper_tar}_FOUND)
        list(APPEND _axom_exported_targets ${_tar})
    endif()
endforeach()

This shouldn't have openmp in it. We purposefully do not make any targets depend on openmp due to the issue with generator expressions. This will change soon with the new BLT macro.
This loop handles multiple generated variables between TPLs and internal builtin TPLs. The TPLs set the _FOUND variables and without verifying it, I think openmp was the only one with an ENABLE_ variable. This shouldn't cause an issue though because we are not setting OPENMP_FOUND and CMake's FindOpenMP.cmake sets OpenMP_FOUND.
Did you hit a bug that was showing a problem that made you look at this?

@white238 One of the TPLs I was linking against had -fopenmp, which is what caused the error. The loop makes sense the way it is, since otherwise I wouldn't have noticed my bug.


fix(ci): fix flakiness by inlining html & css

I was eating empanadas and working on the overlay, which prompted me to look at the portal spec. A part of my brain said:
"You know, I've never seen this test flake."
And another part responded:
"Of course it doesn't flake, it doesn't have a tem-".
"Oh."
So I've fixed the flakiness by inlining all the templates and CSS. After I did this, one of the checkbox tests was actually failing, so I fixed that too.
This also has the pleasant side effect of making the tests in all browsers significantly faster.
Also, I removed firefox beta because all it does is make the SauceLabs optional task take forever to finish failing.

R: @hansl @robertmesserle @kara

I don't see any change for karma to inline the changes. Did I miss a commit?

Never mind I just saw you ran it as part of the Travis script. Lgtm.

First of all, NICE!
Second, LGTM!


feat: extCore'

Moved from Mathlib.CategoryTheory.Basic in order to be more moveable.
Corresponding mathlib4 pr needed.

Corresponding mathlib4 pr: https://github.com/leanprover-community/mathlib4/pull/5329

I don't think this is appropriate for std. Defining a thunk for tagging purposes in mathlib seems just fine to me.

see suggestions here: https://github.com/leanprover-community/mathlib4/pull/5157#discussion_r1233705472

ok


blacklist on mikrotik

Hi, it working nice thanks for this one.
Have another, is possible to send IPs from http://host/fail2ban back on router to addresslist?

Have done it so ignore this one pls, closing


Optimize Tact's runtime stdlib functions

For instance, __tact_create_address in src/generator/writers/writeStdlib.ts contains the following:
var b = begin_cell();
b = b.store_uint(2, 2);
b = b.store_uint(0, 1);
...

It may be more gas efficient to combine the two stores of constants into one:
var b = begin_cell();
b = b.store_uint(4, 3);
...

@jeshecdom you might want to look into this at some point, the example I provided is sort of an instance of things constant propagation analysis might resolve


[S] Populate content (further education)

[ ] populate copy into MDX files for each page
[ ] set up config required for ResearchNavigation (i.e. pills) to work

"Overview" in the breadcrumbs should be "Life stage overview"
-The back button is missing on the 'Life stage overview'. There is a button to go back to the previous life stage on all 'life stage overview' pages
-Space between next up button and footnotes is a bit too large on life stage overview page
-Graph missing on 'Post-16 educational trajectories' page
-Space between the breadcrumbs, contents, and title is a bit tight on mobile. Same on al research pages
-Spacing between some paragraphs bit large as well.

Some comments above. @carinapope13 Did you have any others?
@cameronramsay @tkeburia


Update displays_panel.cpp

Signed-off-by: ahcorde<EMAIL_ADDRESS>This PR is related with PR in RVIZ 1. https://github.com/ros-visualization/rviz/pull/1623
The progress bar dialog box is added to ensure that the duplication process can be completed properly and prevent other events from being responded to.

This pull request has been mentioned on ROS Discourse. There might be relevant details there:
https://discourse.ros.org/t/ros-2-tsc-meeting-minutes-2021-9-16/22372/1

Linux
Linux-aarch64
macOS
Windows


[Additional Parameter Request] Codeunit 7301 "Whse. Jnl.-Register Line" - OnDeleteFromBinContentOnAfterSetFiltersForBinContent

Hi team,
In Codeunit 7301 "Whse. Jnl.-Register Line" procedure DeleteFromBinContent we would like to change some code to work with custom functionality. For that the event OnDeleteFromBinContentOnAfterSetFiltersForBinContent would be good if some additional parameters were added to be able to copy the functionality of standard code completely:

[IntegrationEvent(false, false)]
local procedure OnDeleteFromBinContentOnAfterSetFiltersForBinContent(var BinContent: Record "Bin Content"; WarehouseEntry: Record "Warehouse Entry"; var WhseJnlLine: Record "Warehouse Journal Line"; var WhseReg: Record "Warehouse Register"; var WhseEntryNo: Integer; var IsHandled: Boolean)

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.


Overwrite the existing record

Better to add override parameter in create function instead forcing all the use cases. keep default behavior as override =false.
And application can decide based on guard type override is required or not.

Better to add override parameter in create function instead forcing all the use cases. keep default behavior as override =false. And application can decide based on guard type override is required or not.

Sure but, do we have any use case for that? If so, can you please elaborate?
OR In general, are you referring to support that in the libguard::create() interface?

Better to add override parameter in create function instead forcing all the use cases. keep default behavior as override =false. And application can decide based on guard type override is required or not.

Sure but, do we have any use case for that? If so, can you please elaborate?
OR In general, are you referring to support that in the libguard::create() interface?
Because I am wondering that overwrite logic present in the libguard and application does not know that like which types will support overwrite (Present or Future)

Libguard is used by guard tool/production code. we don't want to enforce some special rules in libguard, this is kind of more application specific requirement.

Better to add override parameter in create function instead forcing all the use cases. keep default behavior as override =false. And application can decide based on guard type override is required or not.

Sure but, do we have any use case for that? If so, can you please elaborate?
OR In general, are you referring to support that in the libguard::create() interface?
Because I am wondering that overwrite logic present in the libguard and application does not know that like which types will support overwrite (Present or Future)

Libguard is used by guard tool/production code. we don't want to enforce some special rules in libguard, this is kind of more application specific requirement.

As discussed: By default, the libguard::create() interface overwrite the existing guard record if meets a certain condition and If the caller doesn't want to perform override, they can pass the overwriteRecord flag with false


selection for 3d volume

Here is the PR for 3d volume selection, some improvements would be:

[ ] Implement selection for multiple visual rather than single one (for both 3D scatter plot & volume rendering viewer)
[x] The ellipse shape selection is much slower than others, I don't know why this happens.
[x] The Glue toolbar at the top window doesn't always show, the point selection icon is still missing.
[ ] Many error info for the improper empty visual data set, I will open a new issue for this.

Great! I'll try this out soon :)

Thanks @PennyQ! I can look later at how to deal with selecting multiple volumes.


Avoid redundant metadata reads in ZarrArrayWrapper

Modify ZarrArrayWrapper to avoid re-reading metadata each time data is read from the array.

[x] Closes #8290
[x] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Thanks @headtr1ck!

You should add an entry in whats-new!

Great, this has been done.

And as a bonus you could add a benchmark, but that's really optional (seems like there are only netcdf Io benchmarks right now).

I didn't do this. I'll try to add a note to #8290 of a before/after scenario, to illustrate the impact of the reduced requests.


Specs for Dashboard

Who is this PR for?
Developers and maintainers
What problem does this PR fix?
Dashboard specs are currently just shallow smoke tests
What does this PR do?
Tests entire component contract for each dashboard component
Automated Testing
Does the PR touch an area of the codebase that needs more specs? More examples within the specs? More scenarios covered by specs?

[x] Author improved specs for code in need of better test coverage

@alexsoble , this should be all set as well. I'm using enzyme to test the render output and functions of each of the dashboard components.

@alexsoble Thanks for the check. I incorporated your suggestions so this should be ready for another look when you have the time.

@edavidsonsawyer 👍 👍 Thanks for adding this test suite, shipping now! 🚢 🚢
One other code improvement I think we should make before we add new features is bringing the JS module style in line with the direction we're using for our other classes.
The tardies dashboard uses React.createClass, but the rest of our JS is moving toward class ClassName extends React.Component. We're also changing the format of our filenames to match create-react-app styles: FileName.js and FileName.test.js instead of file_name.jsx and file_name_spec.jsx. Check out #1467 for an example.
I think you're really well positioned to do this refactoring since you know these classes intimately and have been in the weeds of writing them over the last couple of weeks. Let me know if that sounds like a good plan to you / what questions you have!


Release 65 Scarlet Rosefinch

@fblundun - this is all looks good! (Sorry, have only just got round to reviewing). Minor things:

Is there a reason why the DynamoDB table has id lowercase versus JSON all caps? Seems a bit idiosyncratic.
Your tests have a mix of blah: {} and blah {} in your embedded hocons

Thanks Alex, I've changed the embedded hocons and lowercased "json".

Nice one! I'll push the updated Kinesis apps to Bintray now and then let @jbeemster know that the "json" key is now lowercase.


Remove spurious nils in multierror

The intention of #1 was to allow the simple pattern:
for ... {
  errs = multierror.Append(errs, foo())
}

However, albeit correct in terms of err != nil,
the resulting multierror was ugly because it contained a lot of nil since
it would have appended those nils once the multierror has been created
for the first not-nil error.

+1


Where should I enter the order data in the database?

where I must enter the data in the database, in "Ds_Merchant_MerchantURL"
or in "Ds_Merchant_UrlOK"?

I'm not sure what you mean with "enter data in the database" but... Ds_Merchan_UrlOK is the place that you should put all your code with actions according with the result (update database, send email, etc). Ds_Merchant_MerchantURL is only informative.

I'm not sure what you mean with "enter data in the database" but... Ds_Merchan_UrlOK is the place that you should put all your code with actions according with the result (update database, send email, etc). Ds_Merchant_MerchantURL is only informative.


[STAC] resolves relative asset path

ref: https://github.com/developmentseed/titiler/issues/36#issue-639793489
from @fabricebrito

This feature would ease our setup as the STAC catalogues we use move from a location to another and this is why we use relative hrefs also for the asset paths. For instance pystac resolves the the full href with e.g.

item.assets['nbr'].get_absolute_href()

it should append here https://github.com/cogeotiff/rio-tiler/blob/ce156bc5aaa52c579ee0891998eef7a97e44469c/rio_tiler/io/stac.py#L184-L190

Is there progress on this issue?

@emmanuelmathot no, mostly because I have no idea how to do it 🤷♂, I've looked at pystac but got lost!

No problem, I modified the STACReader to use pystac. PR#313


Error in src/index.js

if (Animated.ScrollView !== props.scrollableComponent) {
      this.ScrollableComponent = Animated.createAnimatedComponent(
        props.scrollableComponent,
      );
    }

in your render func you have <this.ScrollableComponent>.... on return, but it's defined only in a conditional function what is in contructor, it must be at least like this:
this.ScrollableComponent = Animated.createAnimatedComponent(
      Animated.ScrollView,
    );
if (Animated.ScrollView !== props.scrollableComponent) {
      this.ScrollableComponent = Animated.createAnimatedComponent(
        props.scrollableComponent,
      );
    }

or change default props key from 'scrollableComponent' to 'ScrollableComponent

Hi, @440459012, thanks for the issue. You right, I forgot about it.
Already fixed in v1.5.3

cool, thanks


Fix !!!! being interpreted as a !S2! command and getting stuck

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

What is the current behavior? (You can also link to an open issue here)
Using ! in dialogue, it is possible to trigger the !S2! speed commands unintentionally, causing missing dialogue, random characters, or the game to get stuck on 1 textbox. Reported on discord.
https://discord.com/channels/554713715442712616/570925559346102273/751042234761085079

What is the new behavior (if this is a feature change)?
If "!" check fails, it prints the character normally as part of a mini else.
Might fix some other slightly weird text cases, but be on the look out for more.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Users must update their ejected engine.

Other information:

Good spot! Thanks


Search function will not return a SearchResult

I have configured an eodag instance having as a provider ONDA DIAS and also the respective username and password.
The objective is to search and download a product by its ID (The ID is S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916)
I have written the following simple code in order to achieve the above.
Code
...
    try:
        product: SearchResult= dag.search(id=product_id)
    except Exception as err:
        print(f"Failed to collect '{product_id}' product.", err)
...

The search function finds 1 result(s) on provider 'onda' but it doesnt return an SearchResult successfully.
Output
python core/core.py --provider onda  --product_id S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916
2022-09-12 19:13:05,923-15s eodag.config                     [INFO    ] (config           ) Loading user configuration from: /home/vagrant/Desktop/JRC_Repos/data-downloaders/eodag_download/eodag/core/config/eodag.yml
2022-09-12 19:13:06,356-15s eodag.core                       [DEBUG   ] (core             ) Opening product types index in /home/vagrant/.config/eodag/.index
2022-09-12 19:13:06,383-15s eodag.core                       [INFO    ] (core             ) Locations configuration loaded from /home/vagrant/Desktop/JRC_Repos/data-downloaders/eodag_download/eodag/core/config/locations.yml
2022-09-12 19:13:08,108-15s eodag.core                       [INFO    ] (core             ) Searching product with id 'S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916' on provider: onda
2022-09-12 19:13:08,108-15s eodag.core                       [DEBUG   ] (core             ) Using plugin class for search: ODataV4Search
2022-09-12 19:13:08,109-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Getting genric provider product type definition parameters for None
2022-09-12 19:13:08,109-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Getting genric provider product type definition parameters for None
2022-09-12 19:13:08,110-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Building the query string that will be used for search
2022-09-12 19:13:08,110-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Retrieving queryable metadata from metadata_mapping
2022-09-12 19:13:08,111-15s eodag.plugins.search.qssearch    [INFO    ] (qssearch         ) Sending search request: https://catalogue.onda-dias.eu/dias-catalogue/Products?$format=json&$search=%22S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916%22
2022-09-12 19:13:08,819-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Sending metadata request: https://catalogue.onda-dias.eu/dias-catalogue/Products(de250e38-6e13-44cc-ba07-fc5411658ab7)/Metadata
2022-09-12 19:13:09,169-15s eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Adapting 1 plugin results to eodag product representation
2022-09-12 19:13:09,251-15s eodag.core                       [INFO    ] (core             ) Found 1 result(s) on provider 'onda'
Failed to collect 'S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916' product.

Environment:

Python version: 3.6.9
EODAG version: 2.5.2

Additional context
The metadata request is successfull (in the browser)
https://catalogue.onda-dias.eu/dias-catalogue/Products(de250e38-6e13-44cc-ba07-fc5411658ab7)/Metadata
As well as the request for the product:
https://catalogue.onda-dias.eu/dias-catalogue/Products?$format=json&$search=%22S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916%22

Hello @iliion ,
in your example, the error is not displayed, so I do not know what can raise the error....
However, I can suggest you to:

check that SearchResult is imported (from eodag import SearchResult)
even if your example won't fail, it stores a tuple (search_result, total_count) in the variable product. You'd better unpack search_result and total_count in two variables (search() method returns 2 variables, unlike search_all():

product_id = "S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916"
try:
    search_result, total_count = dag.search(id=product_id)
except Exception as err:
    print(f"Failed to collect '{product_id}' product.", err)
print(search_result)

2022-09-14 17:38:24,996 eodag.core                       [INFO    ] (core             ) Searching product with id 'S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916' on provider: onda
2022-09-14 17:38:25,005 eodag.plugins.search.qssearch    [INFO    ] (qssearch         ) Sending search request: https://catalogue.onda-dias.eu/dias-catalogue/Products?$format=json&$search=%22S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916%22
2022-09-14 17:38:26,002 eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Sending metadata request: https://catalogue.onda-dias.eu/dias-catalogue/Products(de250e38-6e13-44cc-ba07-fc5411658ab7)/Metadata
2022-09-14 17:38:26,476 eodag.plugins.search.qssearch    [DEBUG   ] (qssearch         ) Adapting 1 plugin results to eodag product representation
2022-09-14 17:38:26,587 eodag.core                       [INFO    ] (core             ) Found 1 result(s) on provider 'onda'
[EOProduct(id=S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916, provider=onda)]

I also removed type annotation as it is not supported for tuple unpacking

Hello @sbrunato
SearchResult was already imported correctly. --> from eodag import SearchResult
Nevertheless I copied your code and I get the same result. It is also true that no error is raised. It just fails to collect the Product.
Im quite puzzled.
For the record I already have another function implemented like this:
products: SearchResult = dag.serch_all(**search_criteria, items_per_page=100)
The latter is successfull

can you try this please ?
import traceback

product_id = "S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916"
try:
    search_result, total_count = dag.search(id=product_id)
    print(search_result)
except Exception:
    print(traceback.format_exc())

Then post the output. Thanks

I get this error
2022-09-20 08:27:35,380-15s eodag.config                     [INFO    ] Loading user configuration from: /home/vagrant/Desktop/JRC_Repos/data-downloaders/eodag_download/eodag/core/config/eodag.yml
2022-09-20 08:27:35,890-15s eodag.core                       [INFO    ] Locations configuration loaded from /home/vagrant/Desktop/JRC_Repos/data-downloaders/eodag_download/eodag/core/config/locations.yml
2022-09-20 08:27:37,761-15s eodag.core                       [INFO    ] Searching product with id 'S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916' on provider: onda
2022-09-20 08:27:37,764-15s eodag.plugins.search.qssearch    [INFO    ] Sending search request: https://catalogue.onda-dias.eu/dias-catalogue/Products?$format=json&$search=%22S2B_MSIL1C_20170717T153909_N0205_R011_T19TCJ_20170717T154916%22
2022-09-20 08:27:45,543-15s eodag.core                       [INFO    ] Found 1 result(s) on provider 'onda'
Traceback (most recent call last):
  File "core/core.py", line 49, in search_by_id
    product, total_count = dag.search(id=product_id)
  File "/home/vagrant/.local/lib/python3.6/site-packages/eodag/api/core.py", line 591, in search
    return self._search_by_id(search_kwargs.pop("id"), **search_kwargs)
  File "/home/vagrant/.local/lib/python3.6/site-packages/eodag/api/core.py", line 853, in _search_by_id
    guesses = self.guess_product_type(**results[0].properties)
  File "/home/vagrant/.local/lib/python3.6/site-packages/eodag/api/core.py", line 467, in guess_product_type
    raise NoMatchingProductType()
eodag.utils.exceptions.NoMatchingProductType

Thanks for the traceback. Very useful

You're welcome.
Also, you should update eodag, as this issue has bee fixed in #380 (v2.4.0 release).
Or if you cannot update, specify the product type while searching: dag.search(id=product_id, productType="S2_MSI_L1C")

Ok that solved it. Thaks for your support.
Cheers! :)


Weights ignored?

I've updated from 1.0.4 to 1.1.1 and apparently my setup with adding data-pagefind-weight="2" to different elements does not work anymore. My top results in the search are pages that should be low in priority since I gave them weights of 0.01. Did the version change something or is it just my machine?

👋 @thebrightsideofmathematics
Yes, the v1.1.0 release changed much of the ranking algorithm. You can see the release notes for this here: https://github.com/CloudCannon/pagefind/releases/tag/v1.1.0
I wouldn't expect the weighting to be quite as dramatically different in your case though — do you have a site you can share and I can take a look?

I tried to create a minimal example but including to few pages didn't show the behaviour I noticed before. Indeed, in this case, 1.0.4 and 1.1.1 show the same sorting.


Fix LaTeX rendering issue in docstring of lbfgs in aliases for optimi…

Fix LaTeX rendering issue in docstring of lbfgs in aliases for optimizers file alias.py

Thank you! The error in the lbfgs docstring is not solved by this pr.
The real error stems from using mbox. One can use text instead. This error appears both in lbfgs and scale_by_lbfgs.
If that's ok with you, I'd like to merge a PR that sweeps over current doc errors.
I've not made a pass on trailing spaces as you did, so if you want you can keep this PR that fixes these errors (once the aforementioned PR is merged).

Of course, no problem!


[UBS Admin Cabinet][Orders] Admin can not change order status from “Не вивезли”/"На маршруті"/"Привезе сам" to "Скасовано"

Environment: Windows 10, Google Chrome 106.0
Reproducible: always
Build found: 24.10.2022
Preconditions
1.User is signed in to  Greencity as admin
2.  In Order page Order with status “Не вивезли”/"На маршруті"/"Привезе сам" is present in table
Steps to reproduce

Click on pencil icon
Change status from “Не вивезли”/"На маршруті"/"Привезе сам" to "Скасовано"
Choose any reason radiobutton
Click on "Додати" button
Go to DB to check order status / refresh page /open orders details page

Actual result
Status is not updated.
Expected result
Status should be changed.
Additional information
Not reproducible on order details page
User story and test case links
User story #2901
Test case 2338 2339 2340
Labels to be added
"Bug", Priority ("pri: "), Severity ("severity:"), Type (" "Functional")
Video
https://user-images.githubusercontent.com/112310945/197626711-52ac5529-478c-4aee-bb11-391cc7f832b3.mp4

Need to fix on back-end as status will not change upon request.
Endpoint: https://greencity-ubs.testgreencity.ga/ubs/management/changingOrder

This is not a backend bug. Each status has a list of statuses to which it can be changed.
For example, the NOT_TAKEN_OUT status can be replaced only by the ADJUSTMENT, and NOT_TAKEN_OUT statuses.
ON_THE_ROUTE status can be replaced only by DONE and NOT_TAKEN_OUT statuses, but not by others.
In case of an incorrect status change, we return a list of order IDs that were not processed and their change is irrelevant.


[add-resource] New Theme

I would like to add a new theme to the Grav Repository.
Here are the project details:
https://github.com/CaribThemes/grav-theme-saba
Thank you,
MATsxm

done


🐛 namespace names should valid RFC 1123 DNS labels

What type of PR is this?
/kind bug
What this PR does / why we need it:
As mentioned here, https://kubernetes.io/docs/concepts/overview/working-with-objects/namespaces/#namespaces-and-dns, all namespace names must be valid RFC 1123 DNS labels.

@christopherhein  PTAL, thanks.

Out of curiosity, what is the difference between these two values?

Out of curiosity, what is the difference between these two values?

// DNS1123SubdomainMaxLength is a subdomain's max length in DNS (RFC 1123)
const DNS1123SubdomainMaxLength int = 253

// DNS1123LabelMaxLength is a label's max length in DNS (RFC 1123)
const DNS1123LabelMaxLength int = 63

Is DNS1123SubdomainMaxLength changed? I clearly remember this should be 64, otherwise the code is wrong.

I'm not sure about this. But I found the latest definition, it is indeed the case. Because I recently discovered that there was too long namespace that was not synchronized, and then I found the reason here.
https://github.com/kubernetes/apimachinery/blob/master/pkg/util/validation/validation.go#L209

Did you change your kubernetes version locally?

Did you change your kubernetes version locally?

no. and I checked the historical version of k8s.io/apimachinery, it should always be that value.

cool. Thanks for fixing the long-term bug then.
/lgtm
/approve


Update node versions

Removed 4.0, 4.1 and iojs (it's a part of node) and added 4 and 5.

Thanks!


Initialize libsodium with global constructor

This frees the user of sodiumoxide from remembering to call sodiumoxide::init(), which we deprecate for now.
NOTE: maybe we shouldn't deprecate this function and instead keep it so that a caller can use it to make sure that initialization hasn't somehow failed? This should be discussed before merging this pull-request.
We could panic in the constructor instead, but that feels a bit scary since it runs pre-main, and we don't have any guarantees about the state of rust when that happend (I might be wrong here, please correct me if I am).

Nice!

Pull Request Test Coverage Report for Build 904

0 of 0   changed or added relevant lines in 0 files are covered.
29 unchanged lines in 9 files lost coverage.
Overall coverage decreased (-0.1%) to 94.914%

Files with Coverage Reduction
New Missed Lines
%

randombytes.rs
1
95.24%

crypto/aead/aead_macros.rs
1
99.38%

crypto/secretbox/xsalsa20poly1305.rs
1
99.4%

crypto/pwhash/scryptsalsa208sha256.rs
2
94.81%

crypto/kx/x25519blake2b.rs
2
98.11%

crypto/stream/stream_macros.rs
3
97.52%

crypto/box_/curve25519xsalsa20poly1305.rs
4
99.03%

lib.rs
5
37.5%

crypto/secretstream/secretstream_macros.rs
10
90.35%

Totals

Change from base Build 900:
-0.1%

Covered Lines:
2874

Relevant Lines:
3028

💛  - Coveralls

bors: r+

By the way, how about making it optional via features and disabling it by default to keep the original behavior?


Hide grid-toolbar and s-QuickSearchBar in Grid

Please, how can I hide these items (grid-toolbar and s-QuickSearchBar)?
With a template xyzGrid.Template.html?
Thank you

in your grid constructor
use jQuery to play with html.

Overraide createToolbar and createQuickFilters
protected createToolbar(): void {
}

protected createQuickFilters(): void {
}

Thank you.
As well createQuickSearchInput()

@Estrusco cool deal man!
Any idea how do I hide bottom navigation of the grid (it's always just a bunch of lines, so no need with heavy navigation buttons

@ga5tan if you talk about pager, you can override usePager in your xyzGrid.ts
        protected usePager(): boolean {
            return false;
        }

@Estrusco MAN! you are on fire. Spot on. BIG THANKS
Since I have u on the line, can u please suggest, how to replace (override) Save button with fancier Submit button at the bottom of the page (under all the entry forms)

btw where do u get info like this. Is there any other documentation besides Developers Guide?

You may try to edit the following file
.\Views\Templates\EntityDialog.Template.html
It will change all your dialog look.

@dfaruque  thx, I checked that, but there is not much.
and no mention of the button.
for now I guess I will just try to hide the save button and put another one with service call at the bottom.


Can't build plugin on Android

Any help would be appreciate !!
The project builds and runs fine with this plugin, however when I add the plugin it fails.
This error happened recently (In the last week or so), the project runs fine in ios but the google maps android plugin keeps failing
The error
BUILD FAILED in 3s
Unable to apply changes on device: emulator-5554. Error is: Failed to build plugin nativescript-google-maps-sdk :
Error: Command ./gradlew failed with exit code 1 Error output:
/Users/home/.gradle/caches/transforms-1/files-1.1/core-1.0.0.aar/0184a27fd8c513aa4b55e0481e42cdbe/res/values/values.xml:133:5-70: AAPT: error: resource android:attr/fontVariationSettings not found.
/Users/home/.gradle/caches/transforms-1/files-1.1/core-1.0.0.aar/0184a27fd8c513aa4b55e0481e42cdbe/res/values/values.xml:133:5-70: AAPT: error: resource android:attr/ttcIndex not found.
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':verifyReleaseResources'.

com.android.ide.common.process.ProcessException: Failed to execute aapt

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 3s

Also getting an error if I switch the version of tns-android:
spawn: ./gradlew "-p" "/Users/home/Frontend/apps/mobile/platforms/tempPlugin/nativescript_google_maps_sdk" "assembleRelease" "-PcompileSdk=android-27" "-PbuildToolsVersion=27.0.3" "-PsupportVersion=26.0.0-alpha1"

I just started having this issue as well as of today. I believe it's the first time I removed the platforms folder / other project "clean" steps since reinstalling OpenJDK and possibly getting a new version of Gradle. I have not come up with a solution yet but will keep you posted if I do.
If anyone else knows what could be the issue, any help would be greatly appreciated!

I solved issue by commenting Lines for  fontVariationSettings and ttcIndex in "/Users/home/.gradle/caches/transforms1/files1.1/core1.0.0.aar/0184a27fd8c513aa4b55e0481e42cdbe/res/values/values.xml" File
It works for me.

Switching to using Android Studio instead of android-sdk (and thus changing $ANDROID_HOME, reinstalling Build Tools and other things through sdkmanager) solved the issue for me.

@samtheprogram Could you elaborate on the steps you did, I'm currently also using Android Studio to manage my SDK, I've also reinstalled Android Studio several times, would be nice to come up with exact steps on how you managed to solve it since I think it would help alot of other people.

Also what do you set your $ANDROID_HOME to

@amobarak I'm not sure the exact steps, but what I do know is that once I re-installed Android Studio and my desired libraries I was able to build again. Here is roughly what I think I did:

Remove android-sdk if it's installed: brew cask uninstall android-sdk
Install Android Studio.
In Android Studio's SDK Manager, take note of the value for "Android SDK Location". For me, it is /Users/sam/Library/Android/sdk. This should be what your $ANDROID_HOME is set to. This is visible in the screenshot below.
In Android Studio's SDK Manager, I installed Android 7.1.1 Nougat (Platform, Sources, and Google Play Intel images as seen in the screenshot below) along with the same packages for Android 8.1 as I need those for my AVD simulators / for testing.
Update your $ANDROID_HOME value based on step 3. On Linux, you might need to log in & log out for your bash_profile / bashrc / etc. changes to take effect. On Mac I just need to close and reopen the terminal. Check by running echo $ANDROID_HOME.
Run the SDK Manager CLI command as is stated in the NativeScript Advanced Setup document. For me, this is:

$ANDROID_HOME/tools/bin/sdkmanager "tools" "emulator" "platform-tools" "platforms;android-28" "build-tools;28.0.3" "extras;android;m2repository" "extras;google;m2repository"

Here's a screenshot for what I install for local emulators (step 4), which will probably differ from your own, and the value for the SDK locaiton / ANDROID_HOME (step 3):


[Help] It's possible to use this plugin without using with typescript?

Describe the problem you confuse
I'm want to use vue and electron, i'm not big fan of Vite (but that is minor of my problem), but also i don't want Typescript.  But why i need to learn Typescript in order to use this plugin?
I know that Typescript have their advantage but come on, no one need to be force to learn Typescript, and i'm feeling this is case.
There any other way to handle this?

You can use vite-plugin-electeron alone, it doesn't require you to use js or ts, it depends on waht you prefer to use.

That is truth, but the only difference is in
tsconfig.json just add allowJs set it true.
and we continuing using typescrypt. Btw, thanks anyway.


[Docs] Compile tests error in Installing TVM chapters

Thanks for participating in the TVM community! We use https://discuss.tvm.ai for any general usage questions and discussions. The issue tracker is used for actionable items such as feature proposals discussion, roadmaps, and bug tracking.  You are always welcomed to post on the forum first :smile_cat:
Issues that are inactive for a period of time may get closed. We adopt this policy so that we won't lose track of actionable issues that may fall at the bottom of the pile. Feel free to reopen a new one if you feel there is an additional problem that needs attention when an old one gets closed.
Documentation Title & Type
Include the title of the document (e.g. "Getting Started with TVM"), and the type of documentation (e.g. user docs, developer docs, tutorials)
Installing TVM -> Install from Source -> Enable C++ Tests
Additions/Changes Requested
If an RFC/discuss post exists, link it here.
Otherwise, specify what actions should be taken to provide additional clarity/readability/reproducibility to the document. Include code snippets from the previous documentation if applicable.
when I use ./tests/scripts/task_cpp_unittest.sh command to compile C++ tests, but get errors below:
lzlang@YH-202006201754:~/code/tvm$ ./tests/scripts/task_cpp_unittest.sh
+ '[' 0 -gt 0 ']'
+ '[' -n '' ']'
+ BUILD_DIR=build
+ source tests/scripts/setup-pytest-env.sh
++ set +u
++ [[ ! -z '' ]]
++ export 'PYTEST_ADDOPTS=-s -vv '
++ PYTEST_ADDOPTS='-s -vv '
++ set -ux
+++ pwd
++ export TVM_PATH=/home/lzlang/code/tvm
++ TVM_PATH=/home/lzlang/code/tvm
++ export PYTHONPATH=/home/lzlang/code/tvm/python
++ PYTHONPATH=/home/lzlang/code/tvm/python
++ export TVM_PYTEST_RESULT_DIR=/home/lzlang/code/tvm/build/pytest-results
++ TVM_PYTEST_RESULT_DIR=/home/lzlang/code/tvm/build/pytest-results
++ mkdir -p /home/lzlang/code/tvm/build/pytest-results
++ pytest_errors=()
++ trap cleanup 0
+ export LD_LIBRARY_PATH=lib:
+ LD_LIBRARY_PATH=lib:
++ pwd
+ export VTA_HW_PATH=/home/lzlang/code/tvm/3rdparty/vta-hw
+ VTA_HW_PATH=/home/lzlang/code/tvm/3rdparty/vta-hw
+ export TVM_BIND_THREADS=0
+ TVM_BIND_THREADS=0
+ export OMP_NUM_THREADS=1
+ OMP_NUM_THREADS=1
+ python3 tests/scripts/task_build.py --sccache-bucket tvm-sccache-prod --cmake-target cpptest --build-dir build
[tests/scripts/task_build.py:63 INFO] Not using sccache, reason: 'sccache' executable not found
[ci/scripts/cmd_utils.py:56 INFO] + cmake -GNinja -DCMAKE_BUILD_TYPE=RelWithDebInfo ..
CMake Error: Error: generator : Ninja
Does not match the generator used previously: Unix Makefiles
Either remove the CMakeCache.txt file and CMakeFiles directory or choose a different binary directory.
Traceback (most recent call last):
  File "tests/scripts/task_build.py", line 79, in <module>
    sh.run("cmake -GNinja -DCMAKE_BUILD_TYPE=RelWithDebInfo ..", cwd=build_dir)
  File "/home/lzlang/code/tvm/ci/scripts/cmd_utils.py", line 66, in run
    return subprocess.run(cmd, **defaults)
  File "/usr/lib/python3.8/subprocess.py", line 516, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command 'cmake -GNinja -DCMAKE_BUILD_TYPE=RelWithDebInfo ..' returned non-zero exit status 1.
+ cleanup
+ set +x

command make cpptest works well.
Triage
Please refer to the list of label tags linked above to find the relevant tags and add them here in a bullet format (example below).

needs-triage

please refer to the cmake error output
CMake Error: Error: generator : Ninja
Does not match the generator used previously: Unix Makefiles
Either remove the CMakeCache.txt file and CMakeFiles directory or choose a different binary directory.


Standard helpers for use outside Ember

Just wondering if there are any plans to bundle a set of standard HTMLBars helpers (if, each, etc) so that it can be comfortably used outside Ember?
I've been really impressed with using HTMLBars so far, and I'd love it if there were a minimal-but-batteries-included core package for smaller apps that don't justify the whole Ember experience. Are there plans to continue developing HTMLBars as a standalone library with documentation etc (like Handlebars, I guess), or will it eventually just be subsumed into the Ember core?
I'd be happy to help if there's anything I can do to further this – I've tried to extract out a few of the core helpers from Ember's ember-htmlbars package but a) it's hard not to end up yanking out the whole kitchen sink with them, and b) a lot of the behavior seems to be radically different when used outside Ember (e.g. inline if helper within an attribute seems to throw errors, or return "undefined" instead of "", etc), so I'm not sure whether this is the right approach to take. I'd be grateful for any advice as to whether it's better to carry on with this or just to start from scratch and write a whole new set of helpers.
Background: I'm currently working on a backend Node app that allows user-uploaded templates, so Handlebars/Mustache seems like the only safe route. Page content and styling need to be fully user-customizable, so I've hooked up a client-side HTMLBars preview where the user can live-edit colors, fonts etc and have the page instantly rendered exactly like it will on the server. Due to HTMLBars, this now works incredibly smoothly – thanks guys!

@timkendrick I am a big fan of this idea. I've wanted to include the base operators in htmlbars core for a while (and, or, not, etc.). A new version of glimmer will soon be arriving so I've been waiting until then.

@mmun that sounds amazing! My main concern was that HTMLBars would only exist to serve Ember's needs, but it's a great library and I'd love to see it become something in its own right - I really feel it's got mass appeal. Looking forward to seeing how things progress!


Init JETI docs

Hey @dtbuchholz, this has changed substantially since you last reviewed (many thanks to your feedback!) Not just the docs, but the underlying library. Let me know what you think of the changes.

@awmuncy nice, sgtm! I can give it another go and see if anything comes up in my testing but looks solid.

Oh! You'll need to pull the branch off which this PR is built: https://github.com/tablelandnetwork/jeti/pull/55. The skip function isn't in the staging branch.


Nvidia Tesla M60

Hello,
I would like to inquire whether the Nvidia Tesla M60 is compatible with Ollama's code.
Can someone please provide information or insights regarding this compatibility?
Thank you!

That Compute Capability of that card is 5.2.  Support for 5.2 was just merged this past weekend and so I'd expect it to show up in the next release. I'd guess that would happen in the next week or two.

Hi @nejib1, have you tested it out? I am considering getting a M40 or M60 card if it is significantly faster than CPUs for running Ollama.

I went on an ancient GPU buying spree in 2022 and ended up with a K80 and M60.  The K80 isn't great because it's compute capability 3.7 (only recently working, but you have to build ollama from source). The M60 is newer, but is in many ways weaker than the K80 (only 1 GPU, and only 8GB RAM).
The king of cut-rate GPU's right now has got to be the P40, which you can get on ebay for $200. It's a bit faster than an Nvidia T4 or RTX 3060, but the killer is it has 24GB RAM.  It doesn't support float16 however (rather, it does, but is immensely slow) so any code that can leverage float16 or Tensor cores would be much faster on a more modern GPU.  But at a cost less than a 3060...

@orlyandico Very informative. Thank you!!
I have a 16GB 4060Ti (around $600) on my PC. It's imo the best modern Nvidia GPU with enough VRAM and an okay-ish performance for people on a budget.
I want to build a cheap always-on server that can run some LLM workloads. The P40 looks like a great option. My only other concern is its power consumption... If it's gonna add $50 to my monthly electricity bill, I would rather get another 4060Ti.

Hi @nejib1, have you tested it out? I am considering getting a M40 or M60 card if it is significantly faster than CPUs for running Ollama.

Hello,
I bought a new one RTX A4000, it's  a bad idea to work with old GPU..

Nvidia Tesla M60 should be supported in the current release of Ollama. Closing this, but please reopen and post server logs ~/.ollama/logs if you have issues.


findIndexImpl ffi question

hi, I have a little quesion on the code .
exports["findIndexImpl"] = [](const boxed& just_) -> boxed {
	return  [=](const boxed& nothing_) -> boxed {
		return [=](const boxed& f_) -> boxed {
			return [=](const boxed& xs_) -> boxed {
				const auto& xs = unbox<array_t>(xs_);
				int len = xs.size();
				for (auto i = 0; i < len; i++) {

					if (unbox<bool> (f_(boxed(xs[i])) ))
					{
						return just_(i);
					}
				};
				return nothing_;
			};
		};
	};
};

if (unbox<bool> (f_(boxed(xs[i])) ))
this line of code , I can also write it as
if (unbox<bool> (f_(xs[i]) ))
which can compile and run without problem. (windows vs2017 with c++ 14 standard)
So it's suggest that  the compiler can convert  xs[i]  to boxed(xs[i]) ,
am I right? and why was that?
thank you for your time!

Yes, that's correct. Since xs is an array of boxed, then x[i] is already a boxed.

thank you ~


Add info on new list item from 10.2.1 - moved from GUI - Webpanel to

Find the correct listname and how to use

Solved by #592


Self-hosted runners for non-organization repositories

I was hoping to be able to use an M1 Mini I have with Github Actions instead of turning to something like BuildKite, and I notice that although Github only supports runner groups for organizations, self-hosted runners are technically supported for personal repositories without an organization.
I tried (just to see if by some chance it would work) setting the organization slug to my Github username, but no dice. Is there any way support for personal repositories could be added for adding self-hosted runners?

Relevant API docs for self-hosted runner operations for standalone repositories here. The app used to authenticate and grab a registration token for the ephemeral runners needs the administrator permission for the target repository as opposed to the self-runner perms for organizations, but otherwise the ephemeral registration flow and webhooks are about the same.
Would it be a lot of work to add the option to do self-hosted runners for standalone repos for the provisioner? I'd happily take a crack at it, but I'm not familiar with Swift :) If you guys don't have the bandwidth though, I can take a stab


spec: Require newer sos

Get the fix for https://bugzilla.redhat.com/2095263, already available
in CentOS Stream 8.
Change-Id: I8438c119ff2f6696b9f5470ee5b33b18ff8f17ac
Signed-off-by: Yedidyah Bar David<EMAIL_ADDRESS>Fixes issue # (delete if not relevant)
Changes introduced with this PR

Are you the owner of the code you are sending in, or do you have permission of the owner?
[y/n]

This is for rhv bz 2094786. There, since RHV is not going to upgrade to RHEL 8.7, we require sos-4.2-20. On oVirt, since we use CentOS Stream 8, we can require the currently-latest version there, sos-4.3-2.


Unable to move windows about

Hi,
Environment Setup
Latest updates performed - yes
On a fresh installation of Ubuntu desktop 16.04 and following these instructions...
https://askubuntu.com/questions/592537/can-i-access-ubuntu-from-windows-remotely
I am unable to move any windows on XRPD session from Windows. Whenever I try to select a window its just beeps?
How do I fix this?
Thank you
James

Ubuntu Xenial ships a xrdp version too old: 0.6.1
0.7 was released in 2009, so your version is 10 years old!
Please upgrade to a newer version before reporting bugs

Try newer version.


Bump version (pyqt rebuild)

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Thanks @jschueller!


Update testing

@jcockhren i am unable to see why the tests are failing.  They are passing locally


chore: make render

:tada: This PR is included in version 1.15.11-beta.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix broken FB_REFERENCE_IMAGE_DIR preprocessor macro

A change was made that converted the bulk of what was previously a macro, to actual code. The problem with that, was it moved the tricky use of FB_REFERENCE_IMAGE_DIR out of the macro, so setting FB_REFERENCE_IMAGE_DIR in your preprocessor macros yourself no longer worked.
It used to work by replacing instances of FB_REFERENCE_IMAGE_DIR in the preprocessor step with the one you could set in your preprocessor macros, but that doesn't seem to work unless you're in a #define.
Fixes #209

Updated the original commit with documentation.

Bump

@nscoding Can this be merged?

Could someone merge it, please

Thank you @nscoding! Could you please do a new release in Cocoapods? The last one is rather dated at the moment.

@garnett change your Podfile to use :git until they make a new version


Add .NET Fx -> Core breaking change for ComputeSignature

Fixes #17003

LGTM, but the API ref for Core still says the silent parameter is not used -- is there an API docs PR to fix that?

Yes, here. Could you take a look at it?

Thanks for the review, Tom.


Fixes #353: Resolves error handling in Jupternotebook.

As discussed in the issue #353 when it runs on Jupyter the program silently returns nothing. This PR make sure if anything went wrong, it catches the right exception on the output.

Thank you so much! Sorry we were buried with readying the 0.1 "pythonic" release recently. The new release should address this problem, but if you see if come back up please re-open this or post an issue :)


Make HTML5Rocks support canonical URLs of our existing content

I know this should be on H5R technically, but I wanted to track it here so everyone knows the work is getting done.
This will allow us to at first set a canonical url for the content that points to this site, that means search should index on this instead....  In the future I will do a full redirect... I am doing it like this because we also need to support https in the future and we need canonical urls for that migration.

https://github.com/html5rocks/www.html5rocks.com/pull/1452


Only define "str_ends_with" when it's not already defined by PHP

PHP8 already includes a definition for the function "str_ends_with". To handle Wordpress installations running PHP8+, the plugin should only define that function if it is not already defined.
Without this change, WordPress gives an error when activating the plugin.

Good catch, thanks!

Hi @tomlagier,
I found your plugin very useful, thank you for it!
But I still have this issue after adding the package using the following command:
npm install wordpress-shortcode-webpack-plugin
Npm installed version 0.0.17, and it doesn't include your fix for some reason.
Could you please kindly republish the package? Or make this fix available through npm in a different way.

Sure thing, I'll make sure this is published tonight.

Published v0.0.18 including this change.

Hi @tomlagier,
Thank you, successfully updated from the Npm Registry and can confirm that everything works like a charm!


remove max file size check in openExistingOrNew

The current implementation of logrotate.openExistingOrNew has multiple checks to verify if there is an memory overflow on the existing log file and rotates it if true.
This code in particular checks the current log file size when logger.size is not yet set. (which defaults l.size to 0)
The max function always returns a number less than filesize + writeLen resulting in the rotate function being called. This behavior is seen when maxBytes is set to -1.
This results in nerdctl container logs failing to persist in the same log file as each new write rotates the logs and overwrites the older log file.
nerdctl issue

Thanks @coderbirju, could you please add a test to control this behavior!

Updated the comments and added a test case for the openExistingOrNew function. PTAL. Thanks

Updated the comments and added a test case for the openExistingOrNew function. PTAL. Thanks

Thank. Is it possible to make test with MaxBytes equals to -1 to test the behavior that you mentionned in the description ?


On main view with all providers, add authenticated badge indicator

Since one can be logged in/authenticated to multiple providers, it might be nice to show an Authenticated status in each provider box, section.
You might be able to reuse
https://github.com/redwoodjs/playground-auth/blob/main/web/src/components/Badge/Badge.js
but wrap it in the AuthProvider needed:
<AuthProvider client={supabase} type="supabase">
But that would require creating a client per box too, which is fine.
The "not logged in" or unauthenticated would be a nice call-to-action to then try to sign in/up.

I'm happy to take this on if y'all need help!

**Idea:** AuthResults _(to display results in a separate container)_

Didn't want to open a new issue for this but just worked on an idea for AuthResults to display in a separate container.

@WizardSource Thanks! That could definitely help readability.
We have 2 helpers to let you know auth is working ok: the results abs a refresh token poll. Both could use some love
If you push a PR we can review


Logrus logger stops logging and the service hangs when used with syslog hooks

I am using golang version 1.8 and logrus package to log to local syslog on my system but I am seeing that after sometime logging stops and the service hangs. Is this a known issue?
Here is my logger init method:
import (
"encoding/json"
"fmt"
logger "github.com/Sirupsen/logrus"
logrus_syslog "github.com/Sirupsen/logrus/hooks/syslog"
)
func init() {
customFormatter := new(logTextFormatter)
logger.SetFormatter(customFormatter)
logger.SetLevel(supportedLogLevels[strings.ToLower(Config.LogLevel)])
logger.SetOutput(ioutil.Discard)
hook, _ := logrus_syslog.NewSyslogHook("", "", priority, "")
logger.AddHook(hook)
}
Any pointers to fix this will be greatly appreciated?

@sirupsen is this a known issue or there is something obviously wrong in my logger init?

Never heard of this before. What steps have you taken at debugging this?

@sirupsen Can we merge @mauricio fix for the deadlock in logger issue stated above as we are seeing this in our production environments?

@rparulek the fix was already merged.

@sirupsen What specific version of logrus should we be using? Can we use master or some tag needs to be used for a stable version?
@mauricio I incorporated your fix locally in my vendor library for logrus, but my service still hangs in some situations while logging to rsyslog. Any particular logs that would help debug further?
Thanks!

sorry tagged a new version. v1.0.5 has this.

Hi,
I just updated to v1.0.5 and it seems that this change introduced a deadlock in my case.
I have a hook that writes a log itself, and it's hanging at the entry on the mutex within fireHooks() method.


how to mapping the PCA shape components to Apple 51 blendshapes?

Does there any instructions on how to convert?

I tried to build this conversion. But our model reconstructed from the East Asian faces is quite different with Apple's face model in the nose, eyes, etc. There is still some problems in this conversion. I will continue to try to solve this problem.

@LizhenWangT that is promising! Are u trying using Apple's standared face model to fit FaceVerse's version? It seems once we have a dividable 52 shapes, and it's semantic meaning, we can mimick apple's behavior.

Yes, that's just what I want to do. I'm trying to create the 52 FaceVerse shapes which is in the same expression with Apple's standard 52 blendshapes.

@LizhenWangT I am hoping for your impressive result!

@jinfagang This has been solved in version 2!

@LizhenWangT  Thanks your work， could you provide the related code about  converting PCA shape components to Apple 51 blendshapes？

@LizhenWangT Thanks your work， could you provide the related code about converting PCA shape components to Apple 51 blendshapes？

You can refer to this issue. It's more like a manul job rahter than coding.


[Hunt Mob] Planning

task project
Hunt Mob chain
Task description
Plan and identify what we need.
created by
James

Analysis:
To succesfully hunt the mobs, we need to

Track them
Determine our distance to them
Complete appropriate task against them if able*
Return to previous task

if not overriden by higher task
Design:  This is also visible in the HuntMobsChain file

Here is the main pseudocode body of hunt mobs
        in terms of priority, this task superscedes mob defence unless we are below half health

        mobDefence, etc (Whatever makes sense.  These tasks INTERRUPT hunt mob)

        Top priority tasks (in order):
            pause/stop if not in a game
            pause/stop if HuntMobs settings is set to false TODO: Add a hunt mobs setting
            pause if we are hungry and need to eat.
            pause if VERY dangerous mobs are nearby TODO: Determine what is "very dangerous".
            *important*
            pause if we are not doing a task (we should only be hunting mobs we need. This may be tough, so to start,
            lets have it always be active if a task is running).

        Lower priorities:
            Hunt rarest items first.
                - ENDERMAN
                - SHEEP
                    IF we have sheers
                        THEN shear sheep
                        IF we need food
                            THEN kill sheep
                        ELSE
                            let it be
                        FI
                    FI
            IF we need food
                THEN hunt animals (call food chain)
            FI

METHOD: Check if we should run
this is a beta idea, but we should only run if we are going to execute a task that requires a certain material. At the moment, if we are executing a task, we are always running. We will revisit this in the Second Design Phase.
Further helper methods need not be defined as the "how" of how to do these is used in other modules. Simply look at mob defense and food chains for any "how to help".


[Spark Load]fix min_value of GlobalDict will be negative number in spark load

Proposed changes
The row_number() function in spark returns an integer type value.
It will cause two problems in Spark Load :
case 1:  loading a large amount of data at one time causes row_number() overflow.
When the cardinality of the columns to be encoded in the data imported at one time is more than 2.1 billion, row_number will return a negative number.
case 2:  loading data by many times causes the maximum dict_value  in the global dictionary to exceed Integer, but we do not cast it to bigint.

For case 1, I think it's a design flaw that causes a bottleneck on one-time loading and case 1 has relatively few scenarios, which can be solved by importing in multiple batches in the short term.
For case 2,  it will be solved by this pr.
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (Yes/No/I Don't know)
Has unit tests been added: (Yes/No/No Need)
Has document been added or modified: (Yes/No/No Need)
Does it need to update dependencies: (Yes/No)
Are there any changes that cannot be rolled back: (Yes/No)

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

@wangshuo128  Is there any window function like row_num return-type is Long in spark?

@wangshuo128 Is there any window function like row_num return-type is Long in spark?

The data type of Spark builtin function row_number is Integer, please see
https://github.com/apache/spark/blob/2349f74866ae1b365b5e4e0ec8a58c4f7f06885c/sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/windowExpressions.scala#L564
If you want to have a long type row_number function, you could implement a UDAF yourself.


silenceForm required labels

I was not able to find something like this in the configuration.
It would be very useful to be able to select required labels that must be filled in when adding a silence (e.g. team).
Something like:
silenceForm:
  required:
    - name:
      value:
      value_re:

You could potentially use ACLs for that:
  - action: requireMatcher
    reason: db users must pass team=db
    scope:
      groups:
        - db
    matchers:
      required:
        - name: team
          value: db

I have the following ACL, but even if i specify the label team karma prevents adding the silence.
rules:
  - action: requireMatcher
    reason: must specify label team
    matchers:
      required:
        - name: team
          value: .+
          isRegex: true

DEBU[0028] [dev] Proxy request /proxy/alertmanager/dev/api/v2/silences
DEBU[0028] ACL 0: isAllowed=false err=silence blocked by ACL rule: must specify label team
WARN[0028] [dev] proxy request 'POST /proxy/alertmanager/dev/api/v2/silences' was blocked by ACL rule: silence blocked by ACL rule: must specify label team

Right, that's because value is the exact value that's required in the matcher, not a regex rule to match value against.
I need look into it to refresh my memory, but might just add name_re and value_re to the matchers:required to allow regexes there.

https://github.com/prymitive/karma/blob/master/docs/ACLs.md#require-everyone-to-always-specify-team-matcher-in-silences


FanDuel MLB: Only a handful of lineups generated

Hi there,
I'm trying to test this tool (It's amazing by the way) to generate 150 FanDuel MLB lineups. I am feeding it players from a .csv that has 121 players and their projections. I'm not restricting it with any rules. Literally just here's the players, optimize top 150. So the most basic usage possible. But it will only deliver 75 lineups and then it errors with "Unable to build lineup." There should be way more than enough players to generate 150 lineups. I'll attach my .csv resource and paste the code below. Am I doing something wrong?
dfinfopath = 'path to .csv reference'
optimizer = get_optimizer(Site.FANDUEL, Sport.BASEBALL)
optimizer.load_players_from_csv(dfinfopath)
count = 0
for lineup in optimizer.optimize(150):
count = count + 1
print(count)
print(lineup)
fd_reference.zip

I been trying to figure this out since April 1.  FanDuel is breaking for me constantly for Baseball only.  I believe the problem has to do with multiple positions available.  All the other modules have no problem with the multiple positions, but FanDuel keeps throwing errors after a few lineups.
This is where the magic stops happening:
495         players_with_positions = link_players_with_positions(players, positions)
496         for player, position in players_with_positions.items():
497             lineup.append(LineupPlayer(player, position.name, used_fppg=context.players_used_fppg.get(player)))
players_with_positions undefined, global link_players_with_positions = , players = [Frankie Montas P (OAK), Christian Yelich OF (MIL), Keston Hiura 1B/2B (MIL), Aaron Judge OF (NYY), Mike Trout OF (LAA), Austin Meadows OF (TB), Anthony Rendon 3B (LAA), Kevin Kiermaier OF (TB), Bo Bichette SS (TOR)], positions = [<pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>, <pydfs_lineup_optimizer.settings.LineupPosition object>]
Been banging my head against the wall trying to figure out who or what is causing this and no luck yet.

Interesting. I thought it might be the multiple positions too, but I am getting some players in my generated lineups who are eligible at multiple positions. But maybe that can still happen while it's messing up the results. I'm not sure and kind of lost.

Same here.  It works until it doesn't than throws the error.  The CSV file always populates, but I have never reached 150.  Usually dies between 10-40 lineups.  It handles the multiple positions for some, but the players_with_positions error shows undefined.
My thought was maybe it chooses Player X to be a C/1B, but then it fails when player X is OF.  It also fails every time if you set exposure in FanDuel.Baseball.  You can't generate a lineup if you set min or max exposure in FanDuel baseball, so maybe the key is somewhere there too.

Ok I have no idea what happened but I just ran the exact same minimalistic code with the exact same .csv file and it gave me a full 150. What in the world? What would be dynamically changing throughout the day? @bigboyz see if yours works now?

No dice.  Same errors for me.  What slate are you using on FanDuel?  I'm failing with #56543 the main for tonight at 7:05PM EST

Yes I'm running main slate too. I did upgrade my pydfs_lineup_optimmizer module this morning. Maybe there was some caching issue that didn't clear itself until later in the day?

Ahhh...I didn't see a new version was released yesterday.  Going to try that.  I'll update later today when I got it downloaded.

It was a bug with multi positions for FD MLB, I fixed it and released new version 3.4.2

Ahhh...I didn't see a new version was released yesterday. Going to try that. I'll update later today when I got it downloaded.

@bigboyz  were you able to get this resolved? On latest code and seem to still be having an issue here with this.

I am also having trouble with this but can't seem to pin down the issue other than it seems to be happening when there are players with more than two positions. for example when a player has C/1B/2B as a position the optimizer may create a set of lineups but stop before it should with no explanation. But if I remove all the players with three positions then it will calculate the appropriate set of lineups

@DimaKudosh it looks like this is still an issue, even in the latest version.
It seems to mostly be an issue when the player has three eligible positions, and it looks like it's bugging out during the link_players_with_positions part. It also looks like C1B and SS positions are causing the issue. Can't confirm this 100%.
If you trim all players to 2 positions, the issue settles down a lot but still fails on a few. If you trim down to a single position per player, the issue disappears.
"LineupOptimizerException: Unable to build lineup”
Appreciate any help you can provide.

@bigboyz @mbencho were you guys able to get this resolved? Having same issue.


Allow to configure a single column name

Some transformers only make sense with a single column (e.g. map). However, the configuration asks for a columns list.
It should be possible to indicate a column with a single iteme like: column: mycol.
This would be shorter and easier to type/read.

Implemented in PR #54 .


Download still old version

Hi guys,
here you describe to install this plugin :
"AntiCopyPaster requires IntelliJ IDEA of version 2022.2 to work. To install the plugin:
Download the pre-built version of the plugin from here;"
But it's still the old version for Intellij 2021.3 and not as written 2022.2 ?
There are another download link or other possibility to get ?
Kind regards
Holger

Hi @hschenke,
Thanks you! We have updated the link to the plugin in README: link.

Thanks @onewhl it works now 👍


Bootstrap Error In Mailerlite

Hi...I was having issues with mailerlite.  When I went to inspect the Java Console, everything that came up was an error. They tried to assist me, but they could not and told me to contact you guys.  Here's the error that I'm receiving
"https://stackpath.bootstrapcdn.com/bootstrap/4.1.1/css/bootstrap.min.css" please help.  Thank you!

This is a URL, not an error. What kind of error do you have?
BTW BootstrapCDN is not recommended anymore, you may try to use JsDelivr instead<EMAIL_ADDRESS>for the said file.

This is a URL, not an error. What kind of error do you have?
BTW BootstrapCDN is not recommended anymore, you may try to use JsDelivr instead<EMAIL_ADDRESS>for the said file.

I know it's a URL, however, this is what I was told to send to you from Mailerlite. Here's a screenshot of the Java Console.

I know it's a URL, however, this is what I was told to send to you from Mailerlite. Here's a screenshot of the Java Console.

didn't mean to post it three times...

didn't mean to post it three times...

Could be that Stackpath just is no more basically? Try moving your CDN links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing the problem.

Could be that Stackpath just is no more basically? Try moving your CDN links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing the problem.

How do I move my CDN links?
ᐧ
On Tue, Jan 26, 2021 at 12:18 PM Mark Otto<EMAIL_ADDRESS>wrote:

Could be that Stackpath just is no more basically? Try moving your CDN
links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing
the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/32900#issuecomment-767768544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGH642FIOOD2YF4PFDKPIATS34IQHANCNFSM4WSYGI7Q
.

How do I move my CDN links?
ᐧ
On Tue, Jan 26, 2021 at 12:18 PM Mark Otto<EMAIL_ADDRESS>wrote:

Could be that Stackpath just is no more basically? Try moving your CDN
links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing
the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/32900#issuecomment-767768544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGH642FIOOD2YF4PFDKPIATS34IQHANCNFSM4WSYGI7Q
.

I'm just trying to use my email marketing system without issues.
ᐧ
On Tue, Jan 26, 2021 at 12:22 PM Kreda McCullough<EMAIL_ADDRESS>wrote:

How do I move my CDN links?
ᐧ
On Tue, Jan 26, 2021 at 12:18 PM Mark Otto<EMAIL_ADDRESS>wrote:

Could be that Stackpath just is no more basically? Try moving your CDN
links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing
the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/32900#issuecomment-767768544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGH642FIOOD2YF4PFDKPIATS34IQHANCNFSM4WSYGI7Q
.

I'm just trying to use my email marketing system without issues.
ᐧ
On Tue, Jan 26, 2021 at 12:22 PM Kreda McCullough<EMAIL_ADDRESS>wrote:

How do I move my CDN links?
ᐧ
On Tue, Jan 26, 2021 at 12:18 PM Mark Otto<EMAIL_ADDRESS>wrote:

Could be that Stackpath just is no more basically? Try moving your CDN
links to jsDelivr instead—BootstrapCDN is essentially no more.
Beyond that, we can't help debug further without a live demo reproducing
the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twbs/bootstrap/issues/32900#issuecomment-767768544,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGH642FIOOD2YF4PFDKPIATS34IQHANCNFSM4WSYGI7Q
.

@KredaMcCullough You may simply change the URL you have with the one I gave<EMAIL_ADDRESS>This should work fine.

@KredaMcCullough You may simply change the URL you have with the one I gave<EMAIL_ADDRESS>This should work fine.


Google Sign-In now returning error popup_closed_by_user

Our Electron desktop application has been using Google Sign-In for months now without any issues. But all of a sudden, starting today, Google Sign-In from our Electron desktop app started returning the "popup_closed_by_user" error. The login pop-up properly renders and you can successfully login to your Google account, but when the pop-up closes, instead of it properly logging in the user, it returns the popup_closed_by_user error.
Google Sign-In continues to work fine for our website from web browsers like Chrome, Safari, and Firefox, but no longer works from our Electron desktop app.
It appears there has been some kind of regression in the Google Sign-In service?
Our google auth code for reference:
var googleAuth

gapi.load('auth2', function() {
    googleAuth = gapi.auth2.init({
        client_id: 'XXX',
        scope: 'email profile',
        fetch_basic_profile: false
    })
})

function beginGoogleAuth() {
    googleAuth.grantOfflineAccess({
        prompt: 'select_account',
        redirect_uri: 'postmessage'
    }).then(function(response) {
        document.getElementById('login-google-auth-code').value = response.code
        document.getElementById('login-form').submit()
    })
}

I'm encountering the exact same issue with Electron. Working fine on Chrome.

@grant looks like there was a regression in Google Sign-in recently. Seems somewhat similar to https://github.com/google/google-api-javascript-client/issues/473 in that it is only affecting certain clients like Electron, though the error in this case is different. Any chance you could ping the team to see if anything changed?

Same exact issue with Electron starting on Feb 6th. No changes were made to our end.

I do not seem to see recent changes to the gapi client.
Perhaps this is related: https://github.com/google/google-api-javascript-client/issues/405
Can you provide more details in the request and why Electron is making the request fail?

Just like to chip in here and say we're experiencing the popup_closed_by_user error failure also in our React project - appears to be affecting a small % of users across a range of devices (Chrome, FireFox, Safari, Safari Mobile)

Hi,
We do have the same issue. popup_closed_by_user error in our React/Electron project.
The same code works fine in Chrome, Safari and Firefox.

Hello,
Same for our project, React app running in electron gives popup_closed_by_user error. Web version is fine.
We use react-google-login and I was thinking maybe it has the issue, so I tried with simplest page with just google's suggested button and result is the same - working in browser, not working in electron.

Hi, Also can report that this is no longer working for our electron app, but is working fine in any browser.

Any updates here? @grant

@JorgenPhi
I'm not sure why this is happening for this set of applications. I've stated that I believe there's no change in gapi that would cause this.
If there a specific something specific I can do, LMK. However, I cannot debug other projects that use this project like Electron.

@grant
Of course you can not debug other projects, bug as for @JorgenPhi, there were no modifications on the electron app.

@grant
Perhaps some more info will help:
I too am seeing this in my electron project. With no changes in our code, no changes to the electron base, our google authentication fail began failing and only giving a message of "popup_closed_by_user". We regressed against prior releases, also known to be previously working and found them to all fail the exact same way.
It is unlikely that this change is anything other than a change to the auth api given the steps we took to verify the issue and the google authentication being the only common factor.
Was something change in policies? Is this a part/byproduct of the g+ shutdown (we are in the process of removing g+ for auth and profile info)?
Our project uses electron-prebuilt and was tested against 3.x and 4.x builds. All fail with the same error.
Apologies for the callout, we have thousands of users affected by this and having some support would be great.

This may be related. Or not at all. Timeline coincides.
https://developers.google.com/terms/api-services-user-data-policy

Any update about this issue? I'm also having the same problem

Note: I do not work on this library. I just work at Google and help all over.

I have notified the Google team this issue.
My unofficial guess is that G+ login/changes may be related.
I will report the answer I receive as soon as I get one from the gapi team.

If what @bodhi3032 mentioned is true, then migrating to the alternative oauth scopes may help.
https://developers.google.com/identity/sign-in/web/quick-migration-guide

@SatoshiOoO My app has always been using the alternative scopes and wasn't using the G+ scopes nor the G+ specific APIs, but instead the generic Google Sign-In APIs which haven't been deprecated. This issue is also only affecting our Electron app, but not our website login which continues to work fine.
The issue may definitely still be the result of an inadvertent change made alongside the G+ login/changes, but not likely due to lack of a migration needed on our side.

I'd generally agree with @sachinrekhi that this is not g+ shutdown related, as our web (which also uses g+) still works.
Also if you check G's own docs, you'll find there appears to have issues as well: https://developers.google.com/identity/protocols/OAuth2UserAgent
Click the demo link, you'll see the auth is busted there.
So i think this is a policy issue, perhaps G itself didn't realize the full ramifications of the last updates. My guess is it may have to do with this (from the policy update link i posted earlier):

All permission requests must accurately represent the identity of the application that seeks access to user data. If you have obtained authorized client credentials to access Google API Services, keep these credentials confidential.

Whoever initially set this up on our project did so for web, and indeed the electron app follows the browser flows but using electron generated windows, though it does handle via the website (anyone else find this stuff overly convoluted at times?). I am wondering if perhaps something in this flow is failing the above criteria of  accurately representing the app.
Which brings back to the gapi side of things: The broken auth on G's docs page at least gives an error that is useful(ish). perhaps gapi needs an update to do the same, rather than the generic error we are all receiving.
My next step is to look at migrating to the Desktop Flow for oauth. I'm curious if others affected here used the web browser flow or desktop, or some variation thereof?
Best of luck everyone, I will post back my results when I have some.

@bodhi3032 I’m using the web browser flow in both our web client and Electron client. Can you link to the docs for the desktop flow?

OK, good news, looks like they fixed the problem, now it works as expected with no changes on our side. Check your apps ;-)

I can confirm it's working now in our Electron app.

Same for us. Great news !!

Same, Thanks for the fix!!

I can also confirm that Google login is once again working in our Electron app without any required changes. Thanks so much!

While it's nice that all is working again, the fact it went down and there was no acknowledgement of the issue or report of findings is troublesome.
I for one will be looking to adapt user flows to mitigate such outages, but the trade off is more friction in the sign up process.
Keeping the devs who use such platforms in the loop is important in maintaining trust and building community support.
But hurrah? i guess..

I think it came back. I'm experiencing the same issue in Brave browser, but not in Chrome not Firefox.
Seems suspicious...

I am also getting this Google login error in Chrome now, but only on one of my machines. Doesn't seem to matter if I'm incognito or what profile I'm using. But on other browsers and Chromium on my other laptop, it works fine.

The error is present on Version 78.0.3904.108 (Official Build) (64-bit)

OK. I was hitting this issue and realized the bug might be in the original code.
Anyone care to verify this address the error?
Original:
var googleAuth

gapi.load('auth2', function() {
    googleAuth = gapi.auth2.init({
        client_id: 'XXX',
        scope: 'email profile',
        fetch_basic_profile: false
    })
})

function beginGoogleAuth() {
    googleAuth.grantOfflineAccess({
        prompt: 'select_account',
        redirect_uri: 'postmessage'
    }).then(function(response) {
        document.getElementById('login-google-auth-code').value = response.code
        document.getElementById('login-form').submit()
    })
}

Fix is to use the .catch()  NOT a try{}catch around the call.
var googleAuth

gapi.load('auth2', function() {
    googleAuth = gapi.auth2.init({
        client_id: 'XXX',
        scope: 'email profile',
        fetch_basic_profile: false
    })
})

function beginGoogleAuth() {
    googleAuth.grantOfflineAccess({
        prompt: 'select_account',
        redirect_uri: 'postmessage'
    }).then(function(response) {
        document.getElementById('login-google-auth-code').value = response.code
        document.getElementById('login-form').submit()
    }).catch(function(error) {
      console.log(error)
    })
}

Hi All,
I am facing similar issue, any help would be really helpful. I am a junior developer and I might be doing it wrong.
I have a react app, when run locally everything works fine but when I build the project using "yarn build" and move the contents of build folder to aws s3 bucket and enable static hosting, it fails and says "popup_closed_by_user".
I have made sure all the necessary URL have been whitelisted in the oauth credentials side. But still it says that s3 bucker URL is not whitelisted.
Thanks in advance.

Seems to be happening again. Same symptoms as listed previously. Everything works fine in chrome, but is busted in electron.

Hey, I'm also facing the same issue, it works in my machine but not working in my colleague's machine, please help me to fix this.

For those still struggling with the popup_closed_by_user issue on Electron, just to let you know that in order to fix this issue, you need to set the BrowserWindow.webPreferences.nativeWindowOpen property to true (this is required from Electron v8.2.4) on the window which hosts the Google sign-in button.

It also happens in React App all browsers with me

@edouardmercier thank you so much! That fixed the issue for us on electron!

I'm also facing the  same issue with Electron. Working fine on Chrome.

For those still struggling with the popup_closed_by_user issue on Electron, just to let you know that in order to fix this issue, you need to set the BrowserWindow.webPreferences.nativeWindowOpen property to true (this is required from Electron v8.2.4) on the window which hosts the Google sign-in button.

I can verify this works, thank you for this 👍

I'm getting this error even in chrome if I login in incognito tab. I'm using the following code
`
.............
..............
function onSuccess(result) {
console.log(result);
    authenticate(result.code)
        .then(res => {
            token = res.data.token;
            console.log(res);
        })
        .catch(console.log);
}

function authenticate(code) {
    return axios.post('http://localhost:8080/auth/google', JSON.stringify({code}));
}
.............

function onClickSignIn() {
auth2.grantOfflineAccess()
.then(onSuccess)
.catch(onFailure);
}
`

Facing the same issue in react app in all browser. Someone please help.

For authentication/authorization issues please note that gapi.auth2 has been deprecated and replaced with Google Identity Services. See https://developers.googleblog.com/2021/08/gsi-jsweb-deprecation.html for more information. If you have questions related to authentication/authorization please look at the associated documentation or post questions on Stack Overflow with the google-oauth tag.

You can use google loign npm. Module and just pass clientId and secret id
On Tue, 17 May, 2022, 10:05 Sai Swaroop Bedamatta, @.***>
wrote:

Facing the same issue in react app in all browser. Someone please help.
—
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1128401074,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMUTJQSTSUDXU6AOR7PNDTTVKMOXNANCNFSM4GWJQF4A
.
You are receiving this because you commented.Message ID:
@.***>

Are you taking about “https://www.npmjs.com/package/react-google-login”
this one ??
On Sat, 21 May 2022 at 1:21 PM, Neeraj @.***> wrote:

You can use google loign npm. Module and just pass clientId and secret id
On Tue, 17 May, 2022, 10:05 Sai Swaroop Bedamatta, @.***>
wrote:

Facing the same issue in react app in all browser. Someone please help.
—
Reply to this email directly, view it on GitHub
<
https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1128401074
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AMUTJQSTSUDXU6AOR7PNDTTVKMOXNANCNFSM4GWJQF4A
.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1133558788,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AI3TIWJ6OPYULSNEUSDMDILVLCIXNANCNFSM4GWJQF4A
.
You are receiving this because you commented.Message ID:
@.***>

--
Sai Swaroop Bedamatta

Can you please tell me google login issue you are facing in development
mode or production mode.
If you are facing in development mode/production mode , it means you  need
to set in
webPresences:{
nodeIntegration:true,
nativeWindowOpen:true,
webSecurity:false
}
in electron.js file
If again it is not fixed then you need implement through deeplinking
By the use of electron.shell
Thanks and Regards
Neeraj Maurya
MERN Stack Developer
On Tue, 24 May, 2022, 15:02 Neeraj Moriya, @.***> wrote:

Yes, it is.
On Sat, 21 May, 2022, 15:42 Sai Swaroop Bedamatta, <
@.***> wrote:

Are you taking about “https://www.npmjs.com/package/react-google-login”
this one ??
On Sat, 21 May 2022 at 1:21 PM, Neeraj @.***> wrote:

You can use google loign npm. Module and just pass clientId and secret
id
On Tue, 17 May, 2022, 10:05 Sai Swaroop Bedamatta, @.***>
wrote:

Facing the same issue in react app in all browser. Someone please
help.
—
Reply to this email directly, view it on GitHub
<

https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1128401074

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AMUTJQSTSUDXU6AOR7PNDTTVKMOXNANCNFSM4GWJQF4A

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1133558788
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AI3TIWJ6OPYULSNEUSDMDILVLCIXNANCNFSM4GWJQF4A
.
You are receiving this because you commented.Message ID:
@.***>

--
Sai Swaroop Bedamatta
—
Reply to this email directly, view it on GitHub
https://github.com/google/google-api-javascript-client/issues/506#issuecomment-1133594088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMUTJQVP2ULDXD5GFCIV32TVLCZHXANCNFSM4GWJQF4A
.
You are receiving this because you commented.Message ID:
@.***>


Feature Request: Dynamic list of Management Groups

Configure a dynamic list of management groups.
The code would be here and here.

This is no longer needed. The Management group name is now derived from the service connection.


enable ebpf connection tracking by default

Currently the ebpf connection tracking from #2135 needs to be enabled explicitly with --probe.ebpf.connections=true. Once we have gained some confidence in the correctness and stability of this feature, we should enable it by default.

We should probably fix this https://github.com/weaveworks/tcptracer-bpf/issues/10 first though

https://github.com/weaveworks/tcptracer-bpf/issues/10 is fixed now

Then I think we can finally go ahead enable it by default.
It won't be released until 1.5.0 though. I am cutting the 1.4.0 release soon (in 1 or 2 days) and I find it premature without further testing (i.e. having it enabled in master for a while).
We need to be careful about the integration tests when enabling eBPF by default. They should verify the fall back and old behaviour (/proc connection walking). So, we may need to incorporate a couple of integration tests and/or explicitly disable eBPF in a few in the existing ones


Add integration test for proxy

Maybe add an assertion to check that the proxy is actually used. I must admit it's not easy!
An other way of doing this is to run the proxy as a go routines of the test binary (with https://github.com/elazarl/goproxy for example). It removes the extra container, makes it portable to Windows and you can check that it is used. (as you want, your PR is totally fine. We can improve later)

Maybe add an assertion to check that the proxy is actually used. I must admit it's not easy!

@guillaumerose assertion in the integration test?

An other way of doing this is to run the proxy as a go routines of the test binary (with https://github.com/elazarl/goproxy for example). It removes the extra container, makes it portable to Windows and you can check that it is used. (as you want, your PR is totally fine. We can improve later)

Yes this is the module I also looked and can be used as as daemon process, something we used to do in minishift, Also I think we do need the docker for Mac/windows installed by default and use docker instead of podman so all those tests can be executed on the win/mac side also.

assertion in the integration test?

Yes, in this case, it could be an assertion on squid logs. Checking the traffic.

Running goproxy code in a goroutine inside the test process may be simpler/more flexible than going through an external container image.

/retest


Create a mock redux store for front-end unit tests

FOI project has a pattern for this

Hey team! Please add your planning poker estimate with Zenhub @djnunez-aot @saravanpa-aot @tiagograf-AOT @VineetBala-AOT

Closing this since we don't use the store enough for having units