Lo L1A Science Counts CDF Creation

Change Summary
Overview
This PR adds attributes and dataset creation for Lo L1A Science Counts data product
New Dependencies
None
New Files
None
Deleted Files
None
Updated Files

imap_processing/lo/l1a/lo_cdf_attrs.py

added attributes for L1A science counts

imap_processing/lo/l1a/lo_l1a_write_cdfs.py

added dataset creation for science counts

Testing
Added units tests for data set data fields and cdf write file naming: imap_processing/tests/lo/test_lo_l1a_write_cdfs.py
Resulting CDF from unit test can be found here: https://lasp.colorado.edu/galaxy/pages/viewpage.action?pageId=197632025

This is not a comment that is specific to this PR, but this is the first time I am taking a look at how data is flowing from parsed packets to CDF files. This is certainly not meant to pick on you @sdhoyt .
Much of the code seems so formulaic and repeated that I wonder if it wouldn't be better to come up with a configuration file that captures all of the necessary details and a utility that uses that configuration to automatically define the CDF attributes and populate the xr.DataArray objects.

@subagonsouth That's a good idea. I've been thinking about how overwhelming this file is going to get as I need to add more attributes and have been trying to think of an alternative. Are you thinking something like a json file with the attributes that are unpacked to set ScienceAttrs()? For example:
sci_cnt_dataset["oxygen"] = xr.DataArray(
        np.concatenate([sci_cnt_data.OXYGEN for sci_cnt_data in sci_cnt]),
        dims="epoch",
        attrs=ScienceAttrs(**config["lo_oxygen"]).output(),
    )

I guess that I need a github tutorial. I want to be able to reply to this comment:

This is not a comment that is specific to this PR, but this is the first time I am taking a look at how data is flowing from parsed packets to CDF files. This is certainly not meant to pick on you @sdhoyt .
Much of the code seems so formulaic and repeated that I wonder if it wouldn't be better to come up with a configuration file that captures all of the necessary details and a utility that uses that configuration to automatically define the CDF attributes and populate the xr.DataArray objects.

@subagonsouth That's a good idea. I've been thinking about how overwhelming this file is going to get as I need to add more attributes and have been trying to think of an alternative. Are you thinking something like a json file with the attributes that are unpacked to set ScienceAttrs()? For example:
sci_cnt_dataset["oxygen"] = xr.DataArray(
        np.concatenate([sci_cnt_data.OXYGEN for sci_cnt_data in sci_cnt]),
        dims="epoch",
        attrs=ScienceAttrs(**config["lo_oxygen"]).output(),
    )

I was thinking even a level higher than that (if I was even thinking of something concrete, not just an abstract idea)
Something like perhaps a CSV file that defines all of the attributes in a CDF and links them to the dataclass in some way (ideally it would reference all the way back to the packet, but I see that things like decompression get in the way of that). For example:
name,validmin,validmax,catdesc,depend_0,display_type,fill_val,fieldname,format,var_type,units,label_axis,dims,data_class_field
oxygen,0,INT_FILLVAL,Oxygen Rate,epoch,time_series,INT_FILLVAL,Oxygen Rate,I12,data,seconds,oxygen rate,OXYGEN

Then
# Some pseudo code here
with read_csv('config.csv') as csv:
    for row in csv:
        attrs = ScienceAttrs(**get_science_attrs_from_row(row))
        data_array = xr.DataArray(
             np.concatenate([sci_cnt_data[row.data_class_field] for sci_cnt_data in sci_cnt]),
            dims=row.dims,
            attrs=attrs.output(),
        )

@subagonsouth, I remember seeing these yaml configurations at one point for an alternative way to do this. Is this similar to what you're suggesting?
https://github.com/HERMES-SOC/hermes_core/blob/640c23f9b636527a899970b665c1bdb6835bfd12/hermes_core/data/hermes_default_global_cdf_attrs_schema.yaml
https://github.com/HERMES-SOC/hermes_core/blob/640c23f9b636527a899970b665c1bdb6835bfd12/hermes_core/util/schema.py
Here is a related issue with some discussion as well: https://github.com/IMAP-Science-Operations-Center/imap_processing/issues/77

@subagonsouth, I remember seeing these yaml configurations at one point for an alternative way to do this. Is this similar to what you're suggesting?
https://github.com/HERMES-SOC/hermes_core/blob/640c23f9b636527a899970b665c1bdb6835bfd12/hermes_core/data/hermes_default_global_cdf_attrs_schema.yaml
https://github.com/HERMES-SOC/hermes_core/blob/640c23f9b636527a899970b665c1bdb6835bfd12/hermes_core/util/schema.py
Here is a related issue with some discussion as well: #77

@greglucas Good question. I certainly don't have something specific in mind. I find it advantageous to have product definitions all in one place, usually in a configuration file rather than in code. Based on the discussion in #77, I can see that you all have put lots of thought into this already. I am new to CDF and the tools being used to generate them, so I am probably the least qualified to make any decisions regarding implementation of this concept.
The HERMES-SOC approach looks useful for defining the products. It seems to me that it would be useful to add a piece that maps the dataclass(es) to the xr.DataSet. This would allow for generic code, common to all of the instruments, to do the work of populating the xr.DataSet.
All of this is outside the scope of this issue IMO.

Should we close this PR?


[BUG]: topP is not set from the config yaml of the openai provider

Checklist

[X] I've searched for similar issues and couldn't find anything matching
[X] I've included steps to reproduce the behavior

Affected Components

[X] K8sGPT (CLI)
[ ] K8sGPT Operator

K8sGPT Version
v0.3.29
Kubernetes Version
v1.27
Host OS and its Version
MacOS
Steps to reproduce

Create an openai provider with a topp different than 1
If you log the request, the parameter is hardcoded to 1 directly in the code
see line: https://github.com/k8sgpt-ai/k8sgpt/blob/6b38a56afbdaa8e0d8f025088a52d3022673ef9d/pkg/ai/openai.go#L40

Expected behaviour
As topP is a valid parameter from the openAI spec: https://platform.openai.com/docs/api-reference/chat/create#chat-create-top_p; a user should be able to set this parameter.
In addition, there are many openai compatible backends that could take advantage of a user defined topP.
Actual behaviour
topP should be set from the config file.
Additional Information
No response

I have opened a PR that should fix this issue, assuming it is considered a bug.
Please let me know, and feel free to discard the PR is this fix is not needed.


multiple v-if not work in root of template tag

In my code

    <test-component type="normal"></test-component>
    <test-component type="block"></test-component>

  <template id="template-test-component">

      test

Hello, World!

  </template>

This work in plunk, but on real project this code not work. v-if work only in div wrapper, example:
  <template id="template-test-component">

        test

Hello, World!

  </template>

But .wrapper screws up my layout.

A component is required to have one root element.
It looks like you actually want two different components here, like <block> and <normal>, then the v-if's won't be needed.
<block></block>
<normal></normal>

If type is dynamic you can use <component :is="type">
<component :is="type"></component>

Then the actual component being used will be whatever type is.

@crswll I have 2 views one component and not 2 different component. Therefore, your way is not the best ..

Closing as it'a duplicate of https://github.com/vuejs/vue/issues/4128, quote from Evan:

The problem here is that without code analysis of all your v-if expressions (in fact it's near impossible because it depends on your component's runtime state), there's no way to tell whether it will really return only one element.
You can get around this by either using a dynamic component (turning all conditional branches into components), or use a render function instead.

This might be improved in the future, where we just compile everything and warn you at runtime if more than 1 root node is returned.

@yyx990803 thx, i wait this 👍

i wait this too.

Couldn't this work with v-if + v-elif + v-else? Then only one part is guaranteed to be rendered.


Keyboard launcher

Summary of the new feature/enhancement
Problem: using the mouse slows down "keyboard users" (ie: developers) when they need to switch the project they are working on (ie: open the program to test their code, etc...)
Proposed solution: creation of a keyboard launcher that combine and enhance the Start menu search (that shows up when pressing the Win key and then start typing) with the run box (Win-R hotkey).
The keyboard launcher should

be activated by a customizable hotkey (possibly overriding Win-R)
display a well visible input box centered on screen (+1 if customizable in appearance/colors/font)
resolve file path (open in explorer or in command prompt) and commands (ie: all executable in PATH environment variable)
support creation of custom aliases (ie: "mywebmail" to open an URL with a non-default browser), also by importing an existing shortcut for later customization
aliases could also refer to ad hoc scripts/batch files which are not in the PATH environment variable
display command/alias icon if any once it knows which command/alias to run
support autocomplete extensively (for file path, commands, aliases, etc...) and display a drop down menu for MRU
recognize formulas and to the math (ie: typing "=1+1" should show "3", which is a bug to be fixed in next release :-) jk)
search the web (with customizable search engine/browser) when entered string begins with "?"

In short, it should be a "one shot" command line prompt

ahh Good old Google Desktop used to have such a feature. D'Conn'd i 2011.
We bringing it back?

It should be more like Executor. I'm a fan

closing this for #44 as the primary tracking


Refresh about Calico to include multiple dataplanes

Deploy preview for calico ready!
Built with commit 2b86dbde2096511bba7c093bd68040c2f4ae0077
https://deploy-preview-3877--calico.netlify.app

Deploy preview for calico ready!
Built with commit 28267cfb5d1a925e86361782146896c500b548a3
https://deploy-preview-3877--calico.netlify.app


RISC-V COMPLIANCE on SweRV EL2

Hi,
I have integrated a target for SweRV El2 on branch 1.0. The core is an industry-standard core and it is great if we have a target to run compliance on that.
If you guys are interested, I can send a PR for that.

Could you point at your proposed PR? I'm unclear exactly what you are proposing. Certainly we would like  to ensure that ACTs can be run easily on the SweRV core!

I'm not sure what you mean by target. Now that we are using riscof, the new
version of the ACT framework, the target directories have all been removed.
Do you mean you have defined the proper plugins and other support needed by
riscof?
On Wed, Jun 22, 2022 at 8:03 PM Kinza Qamar Zaman @.***>
wrote:

Hi @allenjbaum https://github.com/allenjbaum ,
I haven't sent any PR I'll send that out in few days.
The core itself has no compliance target. The ACT was integrated on
SwerVolf which is a EL2 based SoC.
—
Reply to this email directly, view it on GitHub
https://github.com/riscv-non-isa/riscv-arch-test/issues/261#issuecomment-1163870602,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHPXVJUAA2ITCHYG3XOSUOTVQPHWXANCNFSM5YRYAPOA
.
You are receiving this because you were mentioned.Message ID:
@.***>

No PR has been submitted, so closing this. Reopen if there is something actionable.


Warning: str_repeat(): Second argument has to be greater than or equal to 0 in... on Windows

I am having the same problem as https://github.com/thephpleague/climate/issues/9#issuecomment-98809057.
Opening a new issue because I think it's a separate problem.
The fix by @sergeizelenyi works for me too.

This should be fixed by #64 being merged, can you try out dev-master and report back here if it still isn't working, thanks

Just tested it on my project with the issue. Working for me 💓


APPS-1059 Use int64 for JobID

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

Comparison is base (3cb7f6f) 19.95% compared to head (9602775) 19.95%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #93   +/-   ##
=======================================
  Coverage   19.95%   19.95%
=======================================
  Files          28       28
  Lines        1298     1298
=======================================
  Hits          259      259
  Misses       1014     1014
  Partials       25       25

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


My results always return 1.

I just use the FASNet.py,run on Keras(Using TensorFlow backend) using the weights REPLAY-ftweights18.h5.I Pre-processing the input data ( find the face and cropped to a window sized 96 pixels ).But the result always return 1.Does anyone know why?

I do have found the fact that the uploaded model pretrained from replay/3dmad database will kill too much live faces(dont remember the ratio but quite high) on other databases, though they do work well on single database.
To verify this, you can change the inference code and just get predicted prob instead of the the calss. And than, print the value. I believe you will find the reason.
Cross-database validation still a big problem here so far as i know.
Personally, i choose other approaches to detect attack, since i dont think a cropped and resized face contain sufficient info to tell difference here. what is worse, in practical scene, you could not handle the uploaded image quality, and this has never been considered in most available database.

@cvtower I also found that the uploaded model pretrained from replay/3dmad database don't work well in other database, such as NUAA.
What's worse, I found that images with dark light are easy to be seen as real face, and images with bright light are always to be seen as attacks. Therefore, in 5105 real-face images from NUAA database, only 18% images are predicted as real face.
What can I do next? Maybe feeding more data into network? Or, just classifying one frame really works in the real world?

@Jason-xin  1 is attack.0 is real. I suggest you train your own model.I trained on replay-attack and the acc is 0.96.

https://github.com/OeslleLucena/FASNet/issues/14#issuecomment-397869741
I find predict_class always outputs 1 whatever the predict_proba's output is (0, 1).


List of thirdparty octopress 3 plugins?

Hi,
I have created a plugin for creating captioned images under octopress 3. It was mostly done as a learning experience, but on the off chance its useful to anyone else, is there a page I can add it to?
The plugin is  octopress-captioned-image.
Also, is there any objection to my publishing the gem with the octopress- prefix? Are you trying to reserve this for core octopress gems?
I'm also working on a gem for converting jekyll themes to octopress ink themes, but that is proving rather more complex. Will likely end up being more of a helper, requiring dev to tweak the output, especially where theme config files are concerned.
Thanks for all the great work on octopress 2 and looking forward to the release of v3.

Hi @ajesler. Thanks for posting this. I don't have a page for this yet but it would probably be a good idea to create a wiki on the octopress repo with a list of plugins. I'll set something up at the release.
That's a good question about the gem prefix. I think initially I expected that this would just be something I would use to mark things I'm building. It hadn't occurred to me (for some reason) that others would want to use it to signify that their plugins work with Octopress. The only concern that I have is that 3rd party plugins are obviously marked as such. GitHub makes it easy with user name-spaced repositories, but with ruby gems, that doesn't exist. There's not much I can do about that and the only real downside is that I don't get a name I want if someone else already has it. That's not too big a deal though.
If you have trouble getting an Ink theme to work, post an issue and ping me. I'm happy to help and I'm excited to see you're working on stuff. Thanks!

Would the prefix octopress-contrib- be acceptable? I think that is easy to see that the gem is for octopress and it is not official. The contrib prefix is used by other projects to denote an unofficial extension so should not be completely foreign to devs.

Why not just use octo- for 3rd party and octopress- for first party?

That is a much better idea.

I was just thinking about this independently arrived at the same conclusion as @oflannabhra. Totally, use octo- as a prefix if you like.


⚠ Jenkins Plugins Artifactory has degraded performance

In e5d9d37, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1459 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 71e3de1.


Why docker container name changed everytime?

I run  docker run -d -p 8080:8080 -v C:\Users\username\Dropbox\mywiki\:/var/lib/tiddlywiki mazzolino/tiddlywiki every restart computer
but everytime tiddlywiki-docker create a new continer, why doesn't it use already continer?

And everytime container name is unique

That's how Docker works. You could add --restart=always. Then the same container will be started over and over again automatically.

Ou yeah, I forgot about it.
I think this need added to README

You can also --name your container, which will prevent creating a new container with the same name. A stopped container can be started with docker start <name/id>

You can also --name your container, which will prevent creating a new container with the same name. A stopped container can be started with docker start <name/id>

Cool! Thanks!


Use symfony 2.7.x

Hello,
I would like to install this using composer global require sstalle/php7cc, instead of having to create a project for it.
This would allow me to run php7cc from anywhere.
Only problem is that it requires an old version of Symfony (2.5.x) which causes conflicts with other global packages that are requiring 2.7.x.
Could the dependency be updated safely here ? Or is there any specific reason 2.5.x is uses ?
Thanks

Hello, @nWidart. I've made symfony/console version constraint less strict. I think php7cc should coexist along with 2.7 just fine now. If it does not, please feel free to reopen the issue.
The reason 2.5 was used (and now 2.3) is that it required lower php version than 2.7.

hi. any chance of using symfony/console 3.0 or above? i'm having related problems similar to @nWidart.

Indeed, php7cc could require symfony/console with constraint ~2.3|~3.0 and we could update travis to make one build installing vendors with --prefer-lowest (although there is no tests for cli result I think).

thanks. keen to try it out but both my project and my global install require 3.0 or above

Alternatively you could also install it via the PHAR method (see the Readme) :wink:

composer.json was updated in the master branch. I think symfony/console 3.0 should not be a problem now.


Added rules adapter for kubernetes

@tomerGolany Please go through the prev code review comments - there are 2 items that are still open. Additionally, see an additional minor comment on new commits. Otherwise, this looks fine and can be merged.


Error integracion

Hello thanks for your amazing integration. When adding the integration it shows me this error:
Reintentando la configuración: Not received the expected result, cannot connect to hub <IP_ADDRESS> with id ST_xxxxxxxxxx. ('Expecting property name enclosed in double quotes: line 1 column 2 (char 1)',)
port: 1025
version: v0.1.10
port: 1025
version: v0.1.10
elro devices, those manufactured by siterwell, my devices have not been bought from elro but from siterwell.
sorry my english is not very good and i'm translating from google

Sorry, I had a problem with the wifi connection. it's already solved

I see that your setup is in Spanish. Currently only Dutch and English have a translation. I hope to solve that in the future.  Nice you solved your issue. Feel free to report any issues.

Added Spanish and Portuguese as languages

Muchas gracias por el excelente trabajo. Gracias a tu integración más dispositivos como los de siterwell son compatibles.
Buen trabajo
Thank you very much for the excellent work. Thanks to your integration, more devices such as siterwell are compatible.
Good job
Enviado desde mi iPhone
El 5 sept 2022, a las 20:35, Jan Bouwhuis @.***> escribió:

Added Spanish and Portuguese as languages
—
Reply to this email directly, view it on GitHubhttps://github.com/jbouwh/ha-elro-connects/issues/15#issuecomment-1237380930, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ARB4JW4DBHPZG5SOTQ3IAB3V4Y4QHANCNFSM6AAAAAAQE4GEWM.
You are receiving this because you modified the open/close state.Message ID: @.***>

Nice to hear that!


Any reason why rewards endpoints are not in the Beacon section?

All endpoints in the Beacon section begin with /eth/v{1,2}/beacon/...
All endpoints in the Builder section begin with /eth/v1/builder/...
All endpoints in the Config section begin with /eth/v1/config/...
...

I understand there is some exceptions for ValidatorRequiredApi and Experimental sections.
However, since all rewards endpoint are also prefixed by /eth/v1/beacon/, is there any reason why they are not stored in the Beacon section as well?
(@kevinbogner)

Purely because it's a clean grouping... they could be in the beacon section, but is there value?

The value would be to have all the endpoints in any <Xxx> section beginning with /eth/v{i}/xxx/..., which is currently the case for all sections, except for the Rewards section.
(And the ValidatorRequiredApi and Experimental sections as well, but all endpoints in these two sections are duplicate of other sections)

The /beacon endpoints are meant to be those that contain beacon chain structures.  Rewards are derived information, so should sit outside of this namespace.
(Also beacon endpoints are mandatory for support whereas I don't believe that rewards are, but that's secondary.)


fix status bug and add cluster parameter to ecs.py

the current implementation only allows the default cluster, but often it is necessary to use a specific cluster, so this commit adds cluster as a parameter
there was a bug in _track_tasks() by referencing the undefined variable status – this is fixed now

Afraid I can't help, I only fixed the docs on this repo.
On Fri, 23 Dec 2016 at 09:07, Mention Bot<EMAIL_ADDRESS>wrote:

@MarkusShepherd https://github.com/MarkusShepherd, thanks for your PR!
By analyzing the history of the files in this pull request, we identified
@jfeala https://github.com/jfeala and @adamchainz
https://github.com/adamchainz to be potential reviewers.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/spotify/luigi/pull/1970#issuecomment-268960113, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA0WCTgwETdF2_ef1o9AQgvU3mabY1RUks5rK48_gaJpZM4LUq28
.

@outlierbio


refactor: Replace react-bootstrap tabs with Antd tabs on Profile

SUMMARY
This PR re-reverts part of PR https://github.com/apache/incubator-superset/pull/11090 - it contains only changes to the Profile view, which were not affected by the issue https://github.com/apache/incubator-superset/issues/11192
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before/after:

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

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

@@            Coverage Diff             @@
##           master   #11211      +/-   ##
==========================================
- Coverage   61.49%   55.23%   -6.27%
==========================================
  Files         828      402     -426
  Lines       39145    13350   -25795
  Branches     3589     3393     -196
==========================================
- Hits        24073     7374   -16699
+ Misses      14892     5784    -9108
- Partials      180      192      +12

Flag
Coverage Δ

#cypress
55.23% <ø> (?)

#javascript
?

#python
?

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

Impacted Files
Coverage Δ

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/dashboard/util/getLayoutComponentFromChartId.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

.../src/dashboard/components/FilterIndicatorGroup.jsx
11.76% <0.00%> (-88.24%)
:arrow_down:

...c/explore/components/controls/withVerification.jsx
9.09% <0.00%> (-87.88%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rset-frontend/src/profile/components/Favorites.tsx
0.00% <0.00%> (-86.67%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

... and 719 more

Continue to review full report at Codecov.

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

@rusackas I did the changes that you requested. Frankly, I'm not sure why some of these files (like Modal and Dropdown) were a part of this PR - they were gone after I rebased to the newest master. I changed these hard-coded variables nevertheless 🙂


Allow type converters to be async

Is your feature request related to a problem? Please describe.
When querying alot of data, JSON decode can take a toll on the UI running in the main isolate.
Describe the solution you'd like
Changing type converters to allow returning a FutureOr<T> could be a non-breaking change
while enabling the usage of dedicated JSON decoding isolates inside of the converter.
Manually decoding the JSON after the query would mean we have to introduce extra code in the query functions,
as well as a new type that allows turning the already converted data from the database into a new type that now holds the json decoded data. This would take us further away from the ease of generated types and readable query functions.
This could be a possibly easier to implement alternative (if feasible) to #1872.

Changing type converters to allow returning a FutureOr<T> could be a non-breaking change

I like this idea, but it would probably still be breaking since we currently generate a synchronous factory to map rows to data classes (DataClass.fromData). That would have to become async (or at least a FutureOr but then we may as well make it a Future since users need to await it either way).
Perhaps there's a way to preserve backwards-compatibility here by only generating an async mapping method if an asynchronous type converter is used :thinking: But then we may need a special interface for those to detect their use.

Now I wonder how painful it would be to write converters as extends TypeConverter<Future<YourType>, String> and let drift generate a future in your data classes which you would have to await? It certainly introduces an additional bit of boilerplate, but perhaps it's an acceptable solution if you only need it for a few columns that are storing large json strings?

In my eyes, that would defeat the point entirely.
The reason we wanted asynchronous type converters was so that we can convert json inside of isolates.
Since converting it inside the drift isolates doesnt seem like a thing you would like to implement, it would be nice if we can do it ourselves in the type converters.
Of course we can already do this ourselves by just:

query our data normally, no json conversion with a type that uses strings
converts the json inside of the query function with an isolate
transfers all data from the original query type to a new type that is identical but has a datatype instead of a string field

the problem is just that this makes us write every single object twice and we have then arrived at a point where we are writing tons of boilerplate code.

That's a fair point, thanks for the feedback.
I've now spent a fair bit of time trying to implement async type converters, but it just appeared to be a bottomless bit of more stuff breaking the further I went:

Given that it may call async converters, ResultSetImplementation.map (the method going from a row as a map to a Dart object) would have to be asynchronous.
For the other direction, Insertable.toColumns() would have to be asynchronous too.
There are JsonTypeConverters which apply the conversion to json as well, so all of drift's builitn json serialization would have to become asynchronous as well. Nowadays I'm not too happy with json serialization being builtin to drift in the first place, but it's there now and I don't think it can really stop being synchronous.
On columns that have a type converter applied, the equalsValue method can be used to generate an == in SQL based on the mapped value. This would have to start being asynchronous too, which means that the entire query builder API would have to be async.

I'm ok with 1 and 2, but 3 and 4 are really causing problems and I haven't found a decent solution for them unfortunately.
One other approach that might work: Drift supports custom row classes, what if we allowed the mapping there to be asynchronous? For instance, you could then write:
class MyRowClass {
  // bunch of simple fields
  final MyComplexJsonObject json;

  static Future<MyRowClass> mapFromDb({
    /* other fields */,
    String json,
  }) async {
    // use `compute()` or another approach to decode json
   }
}

@UseDataClass(MyRowClass, constructor: 'mapFromDb')
class MyTable extends Table {
  TextColumn get json => text()(); // no type converter here, mapping done while loading
}

This solves the pain point of having to do two awaits, since drift would await the mapFromDb call internally after running a query. However, you'd have to write your own data class if it uses the large JSON field. Given that this should hopefully only be needed in few cases, do you think that could work? It's a tradeoff between the complexity of doing this in general and some convenience.

unfortunately this leads to the exact same problem mentioned above:
we have to write everything twice.
but I suppose this solution would be better integrated at least.
the problems youve mentioned seem pretty severe.
I think we would be okay with this as a solution.

Yeah, I agree it could ideally be more convenient to use but that's the tradeoff one has to make.
Support for async mappings to custom row classes is implemented in b9a605ed2510eacb6d53c47d6698a6720faa0c78. Let's see how that works out, if you have follow-up ideas based on that let's open a new issue.


Question about this line of code

https://github.com/jianfch/stable-ts/blame/3dfbd722a8edb606a2d819fee49ff9c5db4bf0f2/stable_whisper/result.py#L870
    def _split_segments(self, get_indices, args: list = None, *, lock: bool = False):
        if args is None:
            args = []
        no_words = False
        for i in reversed(range(0, len(self.segments))):
            no_words = not self.segments[i].has_words # <-- here
            indices = get_indices(self.segments[i], *args)
            if indices:

I was skimming through the code.
Was this line supposed to be a series disjunction like this?
# no_words = not self.segments[i].has_words
no_words = no_words or not self.segments[i].has_words

Yes. Thanks for pointing that out.


updated macvim-kaoriya (7.4,20160129)

Closes #1644.

Merged as 6762e2bff6caaaf1e32c36012519c01f31dad1d7.


es6 generations

overview
Right now it seems like all generations are converted to ES5, which is cool but it would be nice if the generations could be set to ES6 so we don't end up with a bunch of verbose ES5 polyfill when ES6 is supported.

closed by f79905f - might want to rip out babel + es5 support at some point.


Change parent for glutathione-regulated potassium exporter activity

GO:0015503 glutathione-regulated potassium exporter activity is an antiporter
currently under :
GO:0015291 secondary active transmembrane transporter activity
move to more specific parentage:
is_a GO:0015297 antiporter activity
The definition is consistent with this, so no change needed there.

#27495


Update README.md with usage info while streaming

OpenAI introduced a new chat parameter that allows you to get usage information at the end of a streamed response.  This updates the README with further instructions on how to accomplish this and the format of the response.
All Submissions:

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?

@codenamev -- amazing, I actually need this today, thank you!


chore: add migration docs for wp5

Closing this staled Draft PR as part of housekeeping.
Webpack 5 will not land right away in FC, and might be reconsidered along with alternatives by the end of the year.


Update Quickstart in RViz Tutorial Documentation and RViz Config argument name

Description
Fixes the rviz_tutorial flag in the "MoveIt Quickstart in Rviz" tutorial, as referenced in Issue 586. Added to declared launch arguments as well, since not setting it on the command line caused an error.
Original PR superseded by https://github.com/ros-planning/moveit2_tutorials/pull/635 since problem is mostly fixed in main. Edited to include small change to documentation and naming convention going forward; feel free to close if too minor a change.
Checklist

[X] Required by CI: Code is auto formatted using clang-format
[ ] While waiting for someone to review your request, please consider reviewing another open pull request to support the maintainers

@chancecardona , this looks like an issue of the documentation not correctly backporting to humble. On the main (rolling) branch of our docs, the rviz_tutorial argument is dropped. It might be a good idea to point out that there is an empty rviz config should they want one using:
ros2 launch moveit2_tutorials demo.launch.py rviz_config:=panda_moveit_config_demo_empty.rviz
I'm more in favour of teaching the user how to specify, modify, save, and select their own rviz config rather than use a rviz_tutorial flag for an empty rviz configuration. Which is discussed at the bottom of the tutorial in both the main and humble version of the docs

I'm more in favour of teaching the user how to specify, modify, save, and select their own rviz config rather than use a rviz_tutorial flag for an empty rviz configuration, as discussed at the bottom of the tutorial in both the main and humble version of the docs

Overall I agree with this. I think the rviz_tutorial flag is a bit redundant. Editing to update the docs for consistency instead and removing this variable (and the dangling references) unless anyone thinks it should be in humble.

@chancecardona , this looks like an issue of the documentation not correctly backporting to humble. On the main (rolling) branch of our docs, the rviz_tutorial argument is dropped. It might be a good idea to point out that there is an empty rviz config should they want one using:
ros2 launch moveit2_tutorials demo.launch.py rviz_config:=panda_moveit_config_demo_empty.rviz
I'm more in favour of teaching the user how to specify, modify, save, and select their own rviz config rather than use a rviz_tutorial flag for an empty rviz configuration, as discussed at the bottom of the tutorial in both the main and humble version of the docs

I'm also in favor of teaching the user how to run RViz independently of moveit and how to select their own config. I think that launching RViz in the same launch as moveit, controllers, joint state publishers and other manipulation specific nodes is an anti-pattern. Unlike in ROS 1, the order of launching in ROS 2 does not matter (there is no need for the roscore process like ROS 1 has). So it is up to the user to decide their preferred workflow including launching RViz once and leaving it running in between launching, stopping, re-building and re-launching manipulation related nodes. Also, moveit and manipulation logic will likely be running on a headless remote machine in a number of systems and launch architecture should reflect that.

@zflat I think your suggestions to discuss how launch Gazebo separately would be a great addition to the tutorials, but for the "MoveIt Quickstart in RViz" tutorial the steps should be as brief and easy as possible

Edited PR and created a new one with more relevant changes to humble. Feel free to close if this is not needed.

Closing in favor of https://github.com/ros-planning/moveit2_tutorials/pull/635.


[PT Run, Image Resizer] Fix for WPF DPI issue on .net core 3.1.19

Summary of the Pull Request
What is this about:
With the release of .net core 3.1.19, PowerToys Run and Image Resizer stopped adapting correctly to the zoom level set up on Windows Display settings.
What is include in the PR:
Adopts the workaround suggested in the corresponding WPF issue: https://github.com/dotnet/wpf/issues/5375#issuecomment-931451210
How does someone test / validate:
Users have validated the fix in the linked issue with a test build.
Quality Checklist

[x] Linked issue: #13221
[x] Communication: I've discussed this with core contributors in the issue.
[ ] Tests: Added/updated and all pass
[ ] Installer: Added/updated and all pass
[ ] Localization: All end user facing strings can be localized
[ ] Docs: Added/ updated
[x] Binaries: Any new files are added to WXS / YML

[x] No new binaries
[ ] YML for signing for new binaries
[ ] WXS for installer for new binaries

@jaimecbernardo does it make sense to revert this since PT is now on .NET6?

Doesn't seem to be causing issues for the time being. It'll get removed when we do the WinUI 3 refresh for ImageResizer most likely.


Added async example which makes multiple requests

Closes #301
Let me know if you'd like to see any changes.

This looks great, thanks!

Hello,
This example is very useful, but I think it could be expanded to make it better for real world use cases.
The additions I can think of are:

Dealing with multiple requests, rather than just two.
Using buffer_unordered to limit concurrency.
Being able to iterate through the results, without first collecting them into a Vec.
Being able to add new futures to the stream based on the results of previous requests.

I've found that there's very few useful futures examples currently which are adaptable by people who are not futures experts, so expanding this example would be very helpful.
Thanks


GraphQL support at Ballerina language level

This would be a great addition to retrieve data from data sources and expose them as POST-Query enabled services. - https://graphql.org/

@anupama-pathirage what's the status of this effort? Is anyone working on this feature? We have @scheung38 interested in knowing the current status of this feature on https://github.com/wso2/ballerina-integrator/issues/624


TypeError: "Cannot read properties of undefined (reading 'fact')" in command "fact"

Reported by .̔̏𝗠𝗲𝗽𝗵𝗶𝘀𝘁𝗼#5558<PHONE_NUMBER>25055775) with bot<PHONE_NUMBER>02710047
TypeError: Cannot read properties of undefined (reading 'fact')
at Message.run (/home/container/Commands/Fun/fact.js:13:40)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)

has been fixed at some point idk


Add stubbed-out query statements

@dgnorton branched some of his recent work of this branch, so this PR is no longer needed.


ENH: stats: Implement _munp() for the skewnorm distribution.

The skew-normal distribution has a moment-generating function from
which we can derive the noncentral moments.  This allows _munp(order, a),
the method of the rv_continuous class that computes the "standard" (i.e.
loc=0, scale=1) noncentral moment, to be implemented.  This is much
faster and more accurate than the default method, which uses
scipy.integrate.quad to compute the moments.

#16487 has been merged, if you rebase or merge with upstream you should be able to remove the pytest.mark.timeout mark here:
https://github.com/scipy/scipy/blob/fb0d2d2c8130963b7e49c580a8e261162f57f137/scipy/stats/tests/test_continuous_basic.py#L269

@larsoner wrote

you should be able to remove the pytest.mark.timeout mark

Done.  I left the code for handling custom marks in test_moments, with a comment about how it might be used again in the future.  It could all be removed instead, now that it is not being used; I'm fine with that option too.

Either way is fine by me. It'll be great just to see this come back green, it'll be a nice speedup!

It'll be great just to see this come back green...

I don't expect this latest commit to come back all green, because of a recurring unrelated failure in sparse.linalg that is happening in the Main prerelease_deps_coverage_64bit_blas job.

The Main prerelease_deps_coverage_64bit_blas job is passing now (thanks @rgommers!), but now the Azure job Main source_distribution is failing in all the PRs.

but now the Azure job Main source_distribution is failing in all the PRs.

already fixed by gh-16541

@tupui, thanks for the review!  I have responded to your suggestions, and after merging main to resolve conflicts, I push some updates that address your comments.
If/when this gets merged, the commits should be squashed into one commit.

FWIW and from 30 thousand feet, I'd welcome this sorts of scripts. They are a big help in "what are these magic coefficients and where did they come from" situations. Where to keep them, a folder named _precompute looks reasonable to me, it's a single ls-l away. A different easily discoverable place would work, too, though. Not sure what that would be?

But the code itself is not good for all that. It just make the boat heavier.... @mdhaber do you have an opinion on this?

Yes, this is one of the things I hoped you would comment on.  I confirmed these coefficients by writing my own code to generate them independently, so I don't think should also have to review line-by-line the code used by the author to generate them - especially if it's a script that itself writes Python code. I think it's is valuable code, but I don't think it belongs here in SciPy proper.
The lru_cache was another thing that concerned me, and so was the use of δ in the code. Both of those were removed, so that's good. I'm still not a fan of rewriting and - without comment - refactoring factorial2, but it seems to be correct, so I'll go ahead and merge.


startup.core overwritten

Full description
Updating ModularBOT via installer will overwrite existing startup.core
To Reproduce
Steps to reproduce the behavior:

Modify your startup.CORE
run modularBOT installer.
open startup.CORE
see it overwritten.

Expected behavior
startup.core is to be left untouched by installer.
Desktop:

OS: WINDOWS (All supported versions)

Workaround (until fixed next version)
Always backup your data prior to any updates. Next version will fix this issue via installer.

Fixed in release v2.0.503.1692


Generating MSI fails

Deployment framework : 5.1
VS 2012
BTS 2013
GenericBizTalkMSI.wxs
RedistComponentGroup.wxs
"C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\light.exe" "obj\Debug\GenericBizTalkMSI.wixobj" "obj\Debug\RedistComponentGroup.wixobj" -out "C:\Projects\US\GATEWAY\Gateway.Deployment\bin\Debug\Gateway-1.0.0.msi" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixUIExtension.dll" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixNetFxExtension.dll" -spdb
Windows Installer Xml Linker version 3.7.1224.0
Copyright (C) Outercurve Foundation. All rights reserved.
E:\delivery\Dev\wix37_public\src\ext\UIExtension\wixlib\LicenseAgreementDlg.wxs(32): error LGHT0103: The system cannot find the file 'License.rtf'. [C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj]
C:\Program Files (x86)\MSBuild\DeploymentFrameworkForBizTalk\5.0\BizTalkDeploymentFramework.WiXSetup.targets(251,5): error MSB3073: The command ""C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\light.exe" "obj\Debug\GenericBizTalkMSI.wixobj" "obj\Debug\RedistComponentGroup.wixobj" -out "C:\Projects\US\GATEWAY\Gateway.Deployment\bin\Debug\Gateway-1.0.0.msi" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixUIExtension.dll" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixNetFxExtension.dll" -spdb" exited with code 103. [C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj]
Done Building Project "C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj" (Installer target(s)) -- FAILED.
Build FAILED.
"C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj" (Installer target) (1) ->
(Installer target) ->
E:\delivery\Dev\wix37_public\src\ext\UIExtension\wixlib\LicenseAgreementDlg.wxs(32): error LGHT0103: The system cannot find the file 'License.rtf'. [C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj]
C:\Program Files (x86)\MSBuild\DeploymentFrameworkForBizTalk\5.0\BizTalkDeploymentFramework.WiXSetup.targets(251,5): error MSB3073: The command ""C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\light.exe" "obj\Debug\GenericBizTalkMSI.wixobj" "obj\Debug\RedistComponentGroup.wixobj" -out "C:\Projects\US\GATEWAY\Gateway.Deployment\bin\Debug\Gateway-1.0.0.msi" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixUIExtension.dll" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixNetFxExtension.dll" -spdb" exited with code 103. [C:\Projects\US\GATEWAY\Gateway.Deployment\Gateway.Deployment.btdfproj]
This work item was migrated from CodePlex
CodePlex work item ID: '10217'
Vote count: '1'

[tfabraham@6/26/2013]
The error indicates that License.rtf cannot be found.  That file must be in your Deployment project folder next to the .btdfproj.  It is included in every new project created with the Add New Project wizard.

[UnknownUser@7/10/2013]

[fdesim@1/14/2014]
Tom,
I'm getting the exact same error when trying to build an MSI on the TFS build server.  I have License.rtf in my Deployment project folder and have also placed it in the project build folder in TFS.
E:\delivery\Dev\wix37_public\src\ext\UIExtension\wixlib\LicenseAgreementDlg.wxs (32): The system cannot find the file 'License.rtf'.
C:\Program Files (x86)\MSBuild\DeploymentFrameworkForBizTalk\5.1\BizTalkDeploymentFramework.WiXSetup.targets (251): The command ""C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\light.exe" "obj\Debug\GenericBizTalkMSI.wixobj" "obj\Debug\RedistComponentGroup.wixobj" -out "C:\Builds\1\BizTalk\BTDFDemo1Test1\src\Test1.Deployment\bin\Debug\Test1-.msi" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixUIExtension.dll" -ext "C:\Program Files (x86)\Deployment Framework for BizTalk\5.1\Developer\WiX\WixNetFxExtension.dll" -spdb" exited with code 103.

[tfabraham@1/14/2014]
Please check your build's working folder on the build server to see if the License.rtf is being pulled from TFS and is in the same folder as the .btdfproj.

[fdesim@1/14/2014]
I added License.rtf to the project and verified that it is in the same folder in TFS as .btdfproj.  This gave me a new error that is referenced in this post:  https://biztalkdeployment.codeplex.com/discussions/218533.  I was able to get things working by setting SuppressMSIValidation=True in C:\Program Files (x86)\MSBuild\DeploymentFrameworkForBizTalk\5.1\BizTalkDeploymentFramework.WiXSetup.targets, as suggested in that post.

[tfabraham@1/14/2014]
OK great.  Yes, that is the way to solve the MSI validation error short of giving the TFS build account admin rights on the build server.

Issue closed by tfabraham with comment
Not a product issue
Reason closed
As Designed


REFACTOR: Implement full normalization with TermView.

I've compared the new approach with the old one and there are two comments labeled CHECK that highlight the differences. May need to double check those changes are valid.

bors r+

Whnfer when?

I'll try to finish that in the following days. But we'll see if that needs to use the TermView API.


doesnt work if min value = 1 and current value = 1

doesn't work if min value = 1 and current value = 1

Hey! I am going to take a look at this as soon as possible. Let me know if you find the source of the bug before me 😄

Are you using the swift2.3 branch or master? The latter works perfectly for me.

Had the same issue, though i wasn't using attributes inspector, just programmatically setting the stepper (using pod so I guess its from master). Had code below that caused the issue:
self.stepper.value = 1.0
self.stepper.stepValue = 1.0
self.stepper.minimumValue = 1.0
self.stepper.maximumValue = 50.0

I changed the order of how I was setting up the stepper and it seemed to do the trick:
self.stepper.stepValue = 1.0
self.stepper.minimumValue = 1.0
self.stepper.maximumValue = 50.0
self.stepper.value = 1.0

So I think you just value after you set the min/max of the stepper.

I was able to reproduce the bug. I think with my last commits on master it should be fixed now! I just released a new version on CocoaPods, please let me know if that fixes it for you 👍

@skibadawid Were you able to see if this issue is fixed in your case?

@BalestraPatrick yeah updated the pod and it works.

it doesn't work if min value = 1 and current value = 1 which is set in storyboard. It works when i set in code

@veluloganathan That's exactly what the example project does. I just checked it and it should work. Are you sure that the max value isn't 1? Please let me know of an example project that shows this behavior.


chore(tslint): remove component & directive prefix rules

As a note for this, our guide has different prefixes to show different concepts so this rule makes the lint to fail on those cases.
Rule is better for a concrete project than this site.

It appears that this was incorporated in some other way. Closing


"Game Over" dialogue

Game is missing a "game over" dialogue - will quit to terminal if player loses.
There should be some sort of text that appears instead, possibly using the same method as that which writes the intro story.
Related feature: a prompt that asks if the player would like to restart, and does so using a soft reset.

Closed, implemented!


A better example to show benefits of using the lib

It'd be great to compare formz approach with vanilla Form and TextFormFields approach. It's not clear at this moment.

its even better if we have example that also use essentials library like json_serializable

its even better if we have example that also use essentials library like json_serializable

To start, it would be good to see an example on how formz would be used in a Flutter application. Does it integrate with the Form widget or would it be used instead of it? If so, how?

Yes, absolutely. I'm not sure if we have enough documentation or examples to include this in a production application for Flutter right away. Would love to contribute to this project in the near future!


Chart version not updated to consume app release 3.2.0

Helm chart version needs a bump to consume traceId/spanId log updates released in 3.2.0 version. Any chance of quick revision to this file?
https://github.com/optimizely/optimizely-agent-helm/blob/main/Chart.yaml
appVersion: 3.2.0
version: 1.3.0

cc @pulak-opti

Hi @arjungangisetty
Helm chart is published with the latest Agent version.

Thanks for the swift update @pulak-opti


Publish macOS version as an app bundle

Describe the solution you'd like
Typically, macOS applications are distributed in an app bundle: https://docs.avaloniaui.net/docs/distribution-publishing/macos
Describe alternatives you've considered
As someone new to avalonia, I have no clue on how to properly integrate the current package in my desktop environement. It won't show up in LaunchPad. I can't pin it to the dock, etc...
Having to run it from the terminal every time is cumbersome.
EDIT: Bypassing security warning is also not obvious as the usual tricks do not apply.

Fixed in v19


Can't remove the ipv6 entry for localhost from /etc/hosts

For predictability, I'd like localhost to always resolve to <IP_ADDRESS>, and ip6-localhost to always resolve to ::1.
Unfortunately, the default /etc/hosts file contains these lines:
::1     localhost ip6-localhost ip6-loopback

I cannot edit the /etc/hosts file in the Dockerfile using something like RUN sed -i because I get a Resource Busy error -- this is, I gather, per design.
I also cannot edit the /etc/hosts file once the container has been created, again, because of the Resource Busy error. I'm less clear on whether this is by design.
But the problem remains: localhost will sometimes resolve as IP4 and sometimes resolve as IP6 which is inconsistent and causes problems with validation in my particular setup.
Expected behavior: docker build should generate a ::1 line without the bare localhost name.

Please file the issue in https://github.com/moby/moby.

@jwatte You can't use sed -i because sed then writes a temp file and tries to move it into place, but the /etc/hosts file in the container is a mounted-in file, not a normal file in a mounted directory.  Instead, you have to rewrite the file in place, which is not atomic and therefore "suboptimal" :-/


SUBMARINE-891. Remove namespace function arguments when creating resources

What is this PR for?
Currently when creating resources we pass namespace as function argument. However it is unnecessary since it can be read from submarine custom resource.
What type of PR is it?
[Refactoring]
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/SUBMARINE-891
How should this be tested?
Screenshots (if appropriate)
Questions:

Do the license files need updating? No
Are there breaking changes for older versions? No
Does this need new documentation? No

@kevin85421 Can you help me review this PR? Thank you.


"Expecting to find passwordCredentials in auth" on OVH Openstack Swift

I tried a buch of combinations of Tenant_ID, Tenant_Name, my username & password, but it appears that either OVH is expecting a passwort field but rclone claims to have an API key, or some naming is a bit confusing to me.

What is your rclone version (eg output from rclone -V)
1.32

Which OS you are using and how many bits (eg Windows 7, 64 bit)
Linux 64

Which cloud storage system are you using? (eg Google Drive)
OVH Object Storage (Openstack Swift)

The command you were trying to run (eg rclone copy /tmp remote:tmp)
config & copy

2016/07/27 18:32:12 rclone: Version "v1.32" starting with parameters ["rclone" "-v" "--dump-headers" "--dump-bodies" "copy" "/var/xxx" "ovh:xxx/"]
2016/07/27 18:32:12 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
2016/07/27 18:32:12 HTTP REQUEST
2016/07/27 18:32:12 POST /v2.0/tokens HTTP/1.1
Host: auth.cloud.ovh.net
User-Agent: rclone/v1.32
Content-Length: 144
Content-Type: application/json
Accept-Encoding: gzip

{"auth":{"RAX-KSKEY:apiKeyCredentials":{"username":"123abc567xy","apiKey":"ppppppppppppppp"},"tenantName":"1234567890123456"}}
2016/07/27 18:32:12 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
2016/07/27 18:32:12 <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
2016/07/27 18:32:12 HTTP RESPONSE
2016/07/27 18:32:12 HTTP/1.1 400 Bad Request
Connection: close
Content-Length: 246
Content-Type: application/json
Date: Wed, 27 Jul 2016 16:16:53 GMT
Vary: X-Auth-Token

{"error": {"message": "Expecting to find passwordCredentials in auth - the server could not comply with the request since it is either malformed or otherwise incorrect. The client is assumed to be in error.", "code": 400, "title": "Bad Request"}}

{"auth":{"passwordCredentials":{"username":"123abc567xy","password":"ppppppppppppppp"},"tenantName":"1234567890123456"}}
2016/07/27 18:32:12 >>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>
2016/07/27 18:32:12 <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
2016/07/27 18:32:12 HTTP RESPONSE
2016/07/27 18:32:12 HTTP/1.1 200 OK
Connection: close
Content-Length: 7075
Content-Type: application/json
Date: Wed, 27 Jul 2016 16:16:53 GMT
Vary: X-Auth-Token

{"access": {"token": {"issued_at": "2016-07-27T16:16:53.888292", "expires": "2016-07-28T16:16:53Z",...

The final call succeeds, but the overall operation still fails
2016/07/27 18:32:12 Failed to create file system for "ovh:xxx": Response didn't have storage url and auth token

FYI, OVH supplies the user with a "config" file like this:
#!/bin/bash

# To use an Openstack cloud you need to authenticate against keystone, which
# returns a **Token** and **Service Catalog**.  The catalog contains the
# endpoint for all services the user/tenant has access to - including nova,
# glance, keystone, swift.
#
# *NOTE*: Using the 2.0 *auth api* does not mean that compute api is 2.0.  We
# will use the 1.1 *compute api*
export OS_AUTH_URL=https://auth.cloud.ovh.net/v2.0

# With the addition of Keystone we have standardized on the term **tenant**
# as the entity that owns the resources.
export OS_TENANT_ID=ffffffffffffffffffffffffffffffff
export OS_TENANT_NAME="1234567890123456"

# In addition to the owning entity (tenant), openstack stores the entity
# performing the action as the **user**.
export OS_USERNAME="123abc567xy"

# With Keystone you pass the keystone password.
echo "Please enter your OpenStack Password: "
read -sr OS_PASSWORD_INPUT
export OS_PASSWORD=$OS_PASSWORD_INPUT

# If your configuration has multiple regions, we set that information here.
# OS_REGION_NAME is optional and only valid in certain environments.
export OS_REGION_NAME="SBG1"
# Don't leave a blank variable, unset it if it was empty
if [ -z "$OS_REGION_NAME" ]; then unset OS_REGION_NAME; fi

The swift library rclone uses  (also by me) tries the API key and the password to be backwards compatible with rackspace - so there should have been an attempt with the password just before the one you saw fail.  I'm not sure why this didn't get logged though, but I'm pretty sure it happens like this
2016/07/27 18:17:33 HTTP REQUEST
2016/07/27 18:17:33 POST /v2.0/tokens HTTP/1.1
Host: auth.storage.memset.com
User-Agent: rclone/v1.32
Content-Length: 101
Content-Type: application/json
Accept-Encoding: gzip

{"auth":{"passwordCredentials":{"username":"XXX","password":"XXXX"},"tenantName":"XXX"}}

rclone has worked in the past with OVH.
I think you want your config file ~/.rclone.conf to look like this for a standard openstack v2 auth (put the actual values in though!).  Unfortunately the swift library doesn't read auth from the environment - it really should now but it was written before that was a standard.
[ovh]
type = swift
user = $OS_USERNAME
key = $OS_PASSWORD
auth = $OS_AUTH_URL
tenant = $OS_TENANT_NAME

Sorry, overlooked the seemingly successful password call, edited the issue above.
This is the real error message:
2016/07/27 18:32:12 Failed to create file system for "ovh:xxx": Response didn't have storage url and auth token

I think I may have found the issue: I also set the region to "sbg1"
Deleting this from the config file made it work.

Glad you made it work! Fancy sending a pull request with a bit of OVH specific documentation?

Actually I'm going to add a bit to the docs
Rclone gives Failed to create file system: Response didn’t have storage storage url and auth token

This is most likely caused by forgetting to specify your tenant when setting up a swift remote.

Something like
This may also be caused by specifying the region when you shouldn't have (eg OVH).

What do you think of that?

This should be very helpful, yes.
Also the [ovh] config mapping of parameters from above.
Don't know if this is in the scope of this project, but this guide also was very helpful to set up the bucket and its credentials. http://blog-lrivallain.rhcloud.com/2015/06/30/ovh-cloud-object-storage-cyberduck/

Little update: OVH regions are more complicated than I first thought.
One set of credentials within one project will default to one specific region (for me, that's "GRA1", but other object stores may be configured under another region. If you want to access objects under that region, you will have to specify it, in all-caps, e.g. "SBG1".


Backup to minio

any step by step to create backup using minio?
Thanks
Edit: I install minio and add to backup setting but got error:
error listing backups: error listing backup volumes: Failed to execute: /var/lib/rancher/longhorn/engine-binaries/rancher-longhorn-engine-v0.5.0/longhorn [backup ls --volume-only http://minio-service-ext.default:20000], output Driver http is not supported! , stderr, time="2019-09-19T10:53:47Z" level=error msg="Driver http is not supported!" , error exit status 1

Hi, please check https://github.com/longhorn/longhorn/blob/master/docs/snapshot-backup.md#setup-a-local-testing-backupstore

Ah, I need a cup of coffe. That was writen in readme and I not see that.
Thank you very much.


wasm test: do not rely on dns resolution

Slightly change a test case to not rely on dns resolution.
My system has a slow dns and this test fails because every attempt to reach https://dummyurl takes between 3 and 5 seconds. The test case timeout is 10s, which gives room to approximately 2 attempts, not 5 as the test expects.
We could also increase the timeout, but I don't think it's the correct approach.
Changing the test case URL to an invalid one gives us the same result (i.e., testing a download failure) whithout relying on DNS resolution.

/cherry-pick release-1.16

@jwendell: new pull request created: #41673

In response to this:

/cherry-pick release-1.16

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Fix memory leaks #3829

Closes #3829

Merge after https://github.com/AlphaWallet/web3swift/pull/5 and #3821

watch wallet victor928.eth, after some time apps size increases over 1GB

watch wallet victor928.eth, after some time apps size increases over 1GB

😱

@oa-s but can you rename the commit message and PR title to be more descriptive? :)

Top 4 memory usage of the app running for 5 minutes:

Ethereum ABI
Kanna
AlamoFire
web3swift

I ran Instruments and found this leak. @oa-s @hboon

Also I found a 59 node retain cycle loop.

@eviltofu is it specific to this PR?

After https://github.com/AlphaWallet/web3swift/pull/8

Suggestion is resplace these filters with one filter

@oa-s can you do a timing test? Supposedly the filtering should be lazy and smart, as long as we use the filter() from Realm and not part of Swift's collections.

But probably a separate PR.

Is this PR good for review?

Is this PR good for review?

not yet

@hboon Bump


Documentation for preserve-content?

I scratched my head for quite some time till I found out about the preserve-content attribute on  templates to keep them untouched by polymer.
Is this somewhere in the docs? I couldn't find it. If it's missing, it's maybe something worth mentioning :wink:

+1
Would like to know if this feature is here to stay as well.

@kevinpschaaf Is this OK to document? If so, going to move it to the docs repo.

we would like to use this as well if its safe to do so

Yep, let's document.
Adding preserve-content to a template nested within a Polymer element's top-level template will:

prevent bindings from being parsed (and removed) from the template
prevent special handling of nested template's content fragment; specifically, the content fragment of a nested template is normally cached by Polymer and cleared, to be handed back to the template (as _content) prior to ready time.  This is done for a number of reasons including performance (avoids the cost of unnecessarily cloning elements in nested template contents) and to avoid HTMLTemplateElement bugs on certain platform/versions (older Chrome versions upgraded elements in nested templates when cloning outer templates, and Safari failed to deep-clone nested template contents).

This issue was moved to Polymer/docs#1462


API idea

Hey guys,
Really minor suggestion on API style. Is there ever going to be a case where a developer needs to call rerender(args.element) without supplying args.element? If not, could the API just provide a primed rerender() which can be invoked without supplying args.element? You could provide it as a third argument to render, then we don't need to import rerender either.
i.e. your example:
import { rerender } from "forgo";

function SimpleTimer(initialProps) {
  let seconds = 0; // Just a regular variable, no hooks!

  return {
    render(props, args) {
      setTimeout(() => {
        seconds++;
        rerender(args.element); // rerender
      }, 1000);

      return (

          {seconds} seconds have elapsed... {props.firstName}!

      );
    },
  };
}

Becomes:
function SimpleTimer(initialProps) {
  let seconds = 0; // Just a regular variable, no hooks!

  return {
    render(props, args, rerender) {
      setTimeout(() => {
        seconds++;
        rerender();
      }, 1000);

      return (

          {seconds} seconds have elapsed... {props.firstName}!

      );
    },
  };
}

Each component instance gets an args object that represents some stateful information pertaining to the instance. Such as the DOM element on which it is rendered, or its position in the list of components linked to the same DOM node. This is necessary while calling the rerender() function - and  that's what you're passing into rerender().

Now if by "primed", you mean that this information can be stored in the 'this' pointer somewhere - yes, that could be done. But so far we've avoided the 'this' pointer in our code. Functions just work on inputs provided explicitly. That's the coding style we've chosen for this project.

Now, we're API stable - there are already projects which depend on forgo. So there's a very high bar for changing the API. Unless there's a fundamental issue with it, we wouldn't be able to change it.

API extension (say add a more convenient API) also has a relatively high bar. We're very conscious of library size. The current API is extensible in many ways, and there are already a bunch of libraries which take advantage of this. So, the appropriate place to add an extension to the API would be in an external lib which consumes forgo.

I'll close this now due to (2) and (3). But feel free to continue the discussion - it helps us understand how forgo is being used, and what users want. Also, I'm not entirely sure I've addressed what you're trying to ask here - so please explain.

As an example of extending the API from outside, check the forgo-powertoys project. https://github.com/forgojs/forgo-powertoys
It introduces a renderElement() function which is sometimes convenient when integrating forgo with an existing app.

Ah, now I see what you mean - the code above brings clarity.
Can you post this as a separate issue (with the code example you've provided)? - I think the first three posts on this thread are offtopic.
We'll need to discuss it a bit and then take a call. I don't see it breaking anything, but we'd need to be sure if we're making the right call here.

Another option could be to tack it into args itself.
function SimpleTimer(initialProps) {
  let seconds = 0; // Just a regular variable, no hooks!

  return {
    // instead of render(props, args, rerender)
    render(props, { rerender }) {
      setTimeout(() => {
        seconds++;
        rerender();
      }, 1000);

      return (

          {seconds} seconds have elapsed... {props.firstName}!

      );
    },
  };
}

Continued: #29


getRealItem throws NullPointerException

I came to know about this when I tried to implement auto scroll.
Even I tried to implement auto scroll myself using a separate thread but at getRealItem it throws NullPointerException.
Below is the logcat of exception :
java.lang.NullPointerException
                               at com.gigamole.infinitecycleviewpager.InfiniteCycleManager.getRealItem(InfiniteCycleManager.java:370)
                               at com.gigamole.infinitecycleviewpager.HorizontalInfiniteCycleViewPager.getRealItem(HorizontalInfiniteCycleViewPager.java:226)
                               at in.sabstore.HomeActivity$5$1.run(HomeActivity.java:409)

+1

I had exactly the same question. This is a minor bug of this library. But you can avoid this simply by adding at least 3 items to PagerAdapter before you set it to HorizontalInfiniteCycleViewPager.
The reason why it crashes and throws a null pointer exception is,
in class com.gigamole.infinitecycleviewpager.InfiniteCycleManager
Line 313
public PagerAdapter setAdapter(final PagerAdapter adapter) {
// If adapter count bigger then 2 need to set InfiniteCyclePagerAdapter
if (adapter != null && adapter.getCount() >= MIN_CYCLE_COUNT) {
mItemCount = adapter.getCount();
mInfiniteCyclePagerAdapter = new InfiniteCyclePagerAdapter(adapter);
mInfiniteCyclePagerAdapter.setOnNotifyDataSetChangedListener(this);
return mInfiniteCyclePagerAdapter;
} else {
if (mInfiniteCyclePagerAdapter != null) {
mInfiniteCyclePagerAdapter.setOnNotifyDataSetChangedListener(null);
mInfiniteCyclePagerAdapter = null;
}
return adapter;
}
}
If you do not have 3(MIN_CYCLE_COUNT) or more items in adapter, the member mInfiniteCyclePagerAdapter will not be created. So when you invoke getRealItem() it crashes.


Bump nokogiri from 1.11.4 to 1.12.5

Bumps nokogiri from 1.11.4 to 1.12.5.

Release notes
Changelog
Commits

updated-dependencies:

dependency-name: nokogiri
dependency-type: indirect
...

Signed-off-by: dependabot[bot]<EMAIL_ADDRESS>

drag

drag
..
...


--kube-context param does not work inwerf helm-v3

Hi,
When kube-context is set inwerf helm-v3 upgrade command. Werf monitors the right context (that is set using --kube-context or WERF_KUBE_CONTEXT= env variable), but installs a helm package in a current context. So it's just waiting for release resources to become ready until timeout occurs.
How to reproduce:
❯ werf version
v1.1.23+fix6
❯ werf helm-v3 create test
Creating test
❯ werf helm-v3 upgrade \
    "test" \
    "./test/" \
    --timeout=10m \
    --history-max=10 \
    --wait=true \
    --install=true \
    --atomic=true \
    --kube-context="some_other_context"

┌ Waiting for release resources to become ready
│ ┌ Status progress
│ │ DEPLOYMENT                                             REPLICAS  AVAILABLE  UP-TO-DATE
│ │ test                                                   -         -          -
│ └ Status progress
│
│ ┌ Status progress
│ │ DEPLOYMENT                                             REPLICAS  AVAILABLE  UP-TO-DATE
│ │ test                                                   -         -          -
│ └ Status progress
│
│ ┌ Status progress
│ │ DEPLOYMENT                                             REPLICAS  AVAILABLE  UP-TO-DATE
│ │ test                                                   -         -          -
│ └ Status progress
^C│
│ Error: interrupted

I will suggest to try werf helm command in the v1.2 version

werf helm-v3 in the v1.1 version branch has been added experimentally when there were no v1.2 version. Currently there already is v1.2's werf helm command, which is the real deal, it uses helm 3 (and only helm 3) and contains all latest fixes.


Add social icons

Creating a draft PR because I'm struggling a bit--
I was having some issues with automatic linting that seem to be resolved when I fixed them manually, but the changing-color-on hover is no longer happening. It looks like the hover functionality stopped working when I introduced @media (prefers-reduced-motion); I'm not sure what the best practice is for fixing this?

You're almost there! When you add the prefers-reduced-motion media query what happens is that the styles in that block only get applied if a user has declared that they want reduced motion. In this case, the styles are a bit backwards because if a user has declared that they want reduced motion, these hover styles will be applied, which is kind of the opposite of what should be happening.
What you want to do is change that prefers-reduced-motion to prefers-reduced-motion: no-preference. This is the inverse of the default value, which is reduce and means those styles will be applied unless a user has asked for reduced motion. Check out the Mozilla docs for more information.

Adds styles for social media icons. Fixes #67

@bonniegee just noticed that on smaller styles (Large mobile and Small mobile) there's an underline peeking out. I think you need to add a text-decoration property to the links.

Looks good! If you tab through the component to test focus styles, you'll see that the focus outline is around the whole anchor element, not the span with the .social-icons__icon class. That is because there are some global focus styles in _base.scss that apply to all links and buttons. I'm guessing linting is forcing you to use both :hover and :focus together on lines 30-31, but we're only seeing those specific styles applied with the mouse hover.
I think it's fine to leave it as is, I just wanted to call attention to why hover and focus look different.


HSEARCH-3793 Document usage of the "exists" predicate on object fields

https://hibernate.atlassian.net/browse/HSEARCH-3793

Merged thanks!


feat(SPSTRAT-482): add pre-push for marketplacesvm

This commit allows the marketplacesvm-push-disk-images task to set the parameter nochannel on the wrapper to use the pre-push mode.
The pre-push mode will cause the VM images to be uploaded to the respective clouds but it won't fully publish on them. This is ideal to perform the "staging" tests before the final push.
It also fixes a naming issue with the task
collect-marketplacesvm-secret.
Refers to SPSTRAT-482
Describe your changes
Relevant Jira
Checklist before requesting a review

[x] I have marked as draft or added do not merge label if there's a dependency PR

If you want reviews on your draft PR, you can add reviewers or add the release-service-maintainers handle if you are unsure who to tag

[x] My commit message includes Signed-off-by: My name <email>
[x] I have bumped the task/pipeline version string and updated changelog in the relevant README
[x] I read CONTRIBUTING.MD and commit formatting

@mmalina PTAL again.
I believe I've adjusted everything you asked, please let me know if there's something else or if I did miss anything

@mmalina I've adjusted the test but I also rebased it and noticed some files have changed location, hopefully it's ok. Please let me know if there's something wrong with it

@mmalina I've adjusted the test but I also rebased it and noticed some files have changed location, hopefully it's ok. Please let me know if there's something wrong with it

You mean because of this, right? https://github.com/konflux-ci/release-service-catalog/pull/749
Yeah, that should be fine.

@mmalina feel free to merge it whenever it's ok for you

/ok-to-test


docs?

Can you give some documentation on for example which types of var document = new SiiDocument( typeof( AccessoryEngineData ) ); sii documents are available and how to parse the game.sii/info.sii from a savegame

CC @wilson212


Nougat multiwindow resize break shy behaviour

here it is

@roughike woah you are fast :) Thank you for the attention

woah you are fast :) Thank you for the attention

That's what she said!
(I'm sorry, but had to do it)

LOL

Here's what I've got so far:

The weird position is the library's fault. The translation gets somehow messed up when scrolling on multiwindow mode, and stays that way, leaving the part that should be under navbar also visible.
The navbar turning white is something really weird. The BottomBar doesn't overlay the navbar any way, and I even tried setting the default offset to way too high to verify this.

See the screenshot below, where the default offset is intentionally set to way too high:

I'll investigate this some more.

May it be an OS bug when you got softkeybar transparency active and resize a window?

I have a similar problem. How to solve it? I'm testing on the simulator Android 7.0, Nexus 5


Main Button Issue

I wired up the sketch on my arduino pro mini and it worked in the mockup, but then I needed to put the main button on pin 3. I got everything wired and it will boot up and the secondary button works, but the main one won't. I went back and checked the wiring and still can't figure it out.

A quick check and it won't recognize main button on any pin. It also does this weird thing when you hold down Aux to enter Config mode, it says "Config Mode: Volume" very quickly and then if try main button nothing happens. I checked the battery and it is fully charged.

Ok, so I found something out. I swapped the main button and aux(lockup) button in the program.
Original setup
Main    3
Aux      4
New setup
Main   4
Aux     3
When I upload the sketch and then run it, it is still showing Aux as being on pin 4, not 3, I think this is the problem. How can I force Eclipse to erase the old sketch on the pro mini and upload the new?

So I've even swapped the physical button that is known to work for the one for Main Button, and no matter what pin I put it on, it won't work. I did find out how to make sure changes are uploaded, so that works. But I'm dead in the water without the ability to turn on the saber. An interesting thing, only Pin 4 seems to work for me. I have finally assigned one of the buttons to pin 7 and the other to pin 4 and it works, but it screws up my PCB that I had made, but oh well....might want to look into that.

Hello
I was away this week and could not answer.
I don't own a Arduino Pro mini so I don't know how it is wired or which pin correspond to which one on the Nano.
Just a thought tho : how did you wire your button ? Pin X and (+ or -) ?

Pin is positive then negative to ground. Pin 7 and 4 worked but not 3 or 6, didn't test others
Sent from my iPhone
On Jun 6, 2016, at 9:01 AM, neskweek <notifications<EMAIL_ADDRESS>wrote:
Hello
I was away this week and could not answer.
I don't own a Arduino Pro mini so I don't know how it is wired or which pin correspond to which one on the Nano.
Just a thought tho : how did you wire your button ? Pin X and (+ or -) ?

You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHubhttps://github.com/neskweek/LightSaberOS/issues/13#issuecomment-224004422, or mute the threadhttps://github.com/notifications/unsubscribe/APZl3FGh8zOty0JzsORqC9Jq6MmJUbQjks5qJERTgaJpZM4IufrS.


Use pyproject.toml in Dockerfiles and infrastructure

#110 broke the Docker builds, this fixes them. Note that this is a merge from test-actions, which means we can see that the Docker build actions now pass.

I haven't got around to running this yet, but the changes look sensible to me and happy to merge.


ftp address failing in get_mirca

The Frankfurt ftp address used in get_mirca seems to be failing at the moment.
ftp://ftp.rz.uni-frankfurt.de/pub/uni-frankfurt/physische_geographie/hydrologie/public/data/MIRCA2000/harvested_area_grids/
The harvested area grids can still be downloaded from a link from here all in one (large) zip.
https://hessenbox-a10.rz.uni-frankfurt.de/dl/fiKPzNGoc9bsaFxuN865U9ws/harvested_area_grids.zip
I couldn't find any links to the ftp address maybe it is no longer supported ?
Do you know anything about this @adamhsparks ? Thankyou.

Sorry @AndySouth, I don't. I've poked around and discovered as much as I suspect that you have.
I'm still seeing the ftp server referenced as hosting the data from Uni Frankfurt pages, https://www.uni-frankfurt.de/45217988/GCWM, but the server can't be found.
That's frustrating if you have to spend the time to download everything instead of just the files necessary. Not to mention breaking reproducibility, the FTP server is referenced in several papers that I've found.

I've reached out to Felix Portmann regarding this.
Apparently the uni has changed policies and is shutting down FTP servers. So, no more FTP only the area grids in one large zip file as you've found.
I can rework the functionality to handle the large zip file if you'd like?

I'm having a discussion with the MIRCA2000 authors. We have a couple options that we're exploring, but should have a solution for this shortly.


Add H3ABioNet Open Learning Circles (OLC) project and people

Summary

H3ABioNet Open Learning Circles (OLC) has submitted two projects

Todo

[x] Add the projects
[x] Add people
[ ] Check if we need to add them a collaborators

Looks good!


Options in redux middlewares

This introduces a convenient way to provide options to redux middlewares.

Pull Request Test Coverage Report for Build 57

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 90.222%

Totals

Change from base Build 55:
1.4%

Covered Lines:
127

Relevant Lines:
131

💛  - Coveralls

Pull Request Test Coverage Report for Build 57

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 90.222%

Totals

Change from base Build 55:
1.4%

Covered Lines:
127

Relevant Lines:
131

💛  - Coveralls

Pull Request Test Coverage Report for Build 57

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 90.222%

Totals

Change from base Build 55:
1.4%

Covered Lines:
127

Relevant Lines:
131

💛  - Coveralls

Pull Request Test Coverage Report for Build 57

7 of 7 (100.0%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.4%) to 90.222%

Totals

Change from base Build 55:
1.4%

Covered Lines:
127

Relevant Lines:
131

💛  - Coveralls

Approved

Approved


Got an warning  "[!] Unable to read the license file LICENSE for the spec collection-view-layouts (0.1.2)"

Hello,
Thank you for the library. It works well on my project.
However, there is a minor error when running pod install. The warning is "[!] Unable to read the license file LICENSE for the spec collection-view-layouts (0.1.2)". As a result, it fails to achieve the license information in LICENSE file which is needed in my project.
I think the cause of the warning is the name of the license file, it should be LICENSE (without .txt). Otherwise, Cocoapods won't be able to read the file.
Thank you again.
Tony Hoang

Hello @tony-hoang999, this bug was fixed in version 0.2.2.
Please, try to update and feel free to reopen this issue if you get this warning.

Awesome. Thank you so much @plzen !


Error in text - password page

In following page there is this sentence : "Brute forcing an NTLM hash takes several orders of magnitude longer than brute forcing the NTLM hash of the same password."
I guess what was intended to write was : "Brute forcing an NTLM hash takes several orders of magnitude longer than brute forcing the LM hash of the same password."
https://docs.microsoft.com/en-us/previous-versions/tn-archive/cc875839(v=technet.10)?redirectedfrom=MSDN

@splintercelian "We're no longer updating this content regularly."
#please-close


:bug:Binary-Artifacts: no longer complain about ".bin" files

Those files most likely contain binary data used by tests for
example. It should be safe to remove this because executables
disguised as ".bin" files will still be caught and flagged by scorecard
before it even has a chance to look at extensions.
It should address https://github.com/ossf/scorecard/issues/1256

You also suggested using IsArchive() which seemed like a reasonable approach to me. Could you dump here your arguments in favor of it from a previous issue/PR?
@naveensrinivasan write this part of the code, let see what he thinks.

I think scorecard can switch to IsArchive once the library starts to recognize archives deterministically. As far as tell, if for example a file is a .deb package and an .ar archive at the same time filetype can return either .deb or .ar depending on the implementation details. Effectively that means that to catch deb packages scorecard still have to fall back to extensions.


Calling constructor from parent class gets you.. an instance of the parent class

SomeException: class {
  init: func (.message) { super(message) }
}

main: func {
  // throws an Exception, should err
  SomeException new(String, "you dun goofed") throw()
}

Yup.

@zhaihj, you were right :)


error occuring while downloading some songs

eg: https://www.jiosaavn.com/album/kabir-singh/kLG-OKbVmvM_

They've changed site's UI, that's why it no longer works. @monuyadav016, could you please fix it?

They've updated site's UI, that's why it no longer works. @monuyadav016, could you please fix it?

Thank you for reporting this. I'll look into fixing it soon.

@cs-mshah @faheembhati A small fix for downloading albums and playlist has been done. I'll have to figure out a fix for artist and individual songs. Download and update the scripts.

https://github.com/monuyadav016/Saavn-Downloader/issues/27#issuecomment-646726927


Read flag

I think Session::get should fail if the session has not been read yet, as it's a logic error. Further I'd like a read flag to be accessible to prevent double reads in multiple layers. We can't short circuit Session::read in case it has already been read, because some people might want to reread. We could also just add a reread flag to Session::read.

Implemented in master.


Exceeded token limit - Contacting the developer of metagpt

error.txt
It looks like the code encountered an error due to exceeding the token limit set by the model. This happened when attempting to generate a completion using the OpenAI API, specifically for the model that has a maximum context length of 8192 tokens. However, the request included 8298 tokens, which is beyond the allowed limit.
The error trace is showing that the problem occurred in an asynchronous call within a Python script that is part of a larger application, perhaps one that is simulating a software company environment where an "engineer" role is trying to generate code through a WriteCode action.
Here's a breakdown of the issue:

The write_code method in write_code.py is called to generate some code.
This method makes an asynchronous request to the OpenAI API to get a response.
The request fails because the combined length of the messages and the expected completion exceeds the model's token limit.
Tenacity, a library for retrying failed operations, attempts to retry the operation, but it keeps failing because the token limit issue is not resolved.
Eventually, Tenacity stops retrying and raises a RetryError, which is not caught, causing the whole application to crash.

To fix this issue, you would need to:

Reduce the size of the input prompt or the expected length of the completion so that the total number of tokens does not exceed the model's limit of 8192 tokens.
Catch and handle the InvalidRequestError exception to ensure the application does not crash, possibly by adjusting the input and retrying.
If the input cannot be reduced, consider breaking down the task into smaller parts that can be processed within the token limit and then combine the results.

If you're looking to adjust the script, it would involve catching the exception where the write_code method is called and then trimming the input. If you have access to the code, you can modify the relevant part of the script to handle this case. Otherwise, you may need to contact the developer of the application if you're not able to modify the code yourself.

Will modify this today check back Later thanks


After updated to python 3.11,   neovim can't find pynvim  even I already install pynvim

========================================================================

INFO: Platform: win
INFO: Nvim Version: NVIM v0.8.0-1210-gd367ed9b2

nvim: health#nvim#check
Configuration

OK: no issues found

Performance

OK: Build type: RelWithDebInfo

Remote Plugins

OK: Up to date

========================================================================
Python 3 provider (optional)

INFO: Using: g:python3_host_prog = "F:\Scoop\apps\python\current\python.exe"
INFO: Executable: F:\Scoop\apps\python\current\python.exe
ERROR: Command error (job=17, exit code 1): "F:\Scoop\apps\python\current\python.exe" -c "import sys; sys.path = [p for p in sys.path if p != """"]; import neovim; print(neovim.__file__)" (in 'C:\Users\unicorn')
stderr: Traceback (most recent call last):
File "", line 1, in
File "F:\Scoop\apps\python\current\Lib\site-packages\neovim_init_.py", line 5, in
import pynvim
File "F:\Scoop\apps\python\current\Lib\site-packages\pynvim_init_.py", line 11, in
from pynvim.msgpack_rpc import (ErrorResponse, child_session, socket_session,
File "F:\Scoop\apps\python\current\Lib\site-packages\pynvim\msgpack_rpc_init_.py", line 10, in
from pynvim.msgpack_rpc.session import ErrorResponse, Session
File "F:\Scoop\apps\python\current\Lib\site-packages\pynvim\msgpack_rpc\session.py", line 7, in
import greenlet
File "F:\Scoop\apps\python\current\Lib\site-packages\greenlet_init_.py", line 29, in
from ._greenlet import _C_API # pylint:disable=no-name-in-module
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ModuleNotFoundError: No module named 'greenlet._greenlet'
INFO: Python version: 3.11.0
INFO: pynvim version: unable to load neovim Python module
ERROR: pynvim is not installed.
Error: unable to load neovim Python module

ADVICE:

Run in shell: F:\Scoop\apps\python\current\python.exe -m pip install pynvim

WARNING: Could not contact PyPI to get latest version.
ERROR: HTTP request failed: error: curl error with https://pypi.python.org/pypi/pynvim/json: 56

========================================================================================
What I tried

python -m pip install pynvim   command can't fix this
following is output

 ﬘ —— python -m pip install pynvim
Requirement already satisfied: pynvim in f:\scoop\apps\python\current\lib\site-packages (0.4.3)
Requirement already satisfied: msgpack>=0.5.0 in f:\scoop\apps\python\current\lib\site-packages (from pynvim) (1.0.3)
Requirement already satisfied: greenlet in f:\scoop\apps\python\current\lib\site-packages (from pynvim) (1.1.2)
  unicorn   ~ 
 ﬘ ——

Try: python -m install -I --upgrade greenlet

thank you, but which prompt No module named install
following command worked ！ 😂
python -m pip install --upgrade  greenlet


Unable to handle failed pushes via APNS

When we send a push to Apple we need to detect failed response.
This logic exists in sly/notification-pusher/src/Sly/NotificationPusher/Adapter/Apns.php:
public function push(PushInterface $push) {
    // ...
    if (ServiceResponse::RESULT_OK === $this->response->getCode()) {
        $pushedDevices->add($device);
    }
    //...
    // Function returned devices which was successfully pushed
    return $pushedDevices;
}

But in the calling logic this response is never used (in sly/notification-pusher/src/Sly/NotificationPusher/PushManager.php):
public function push()
    {
        foreach ($this as $push) {
            $adapter = $push->getAdapter();
            $adapter->setEnvironment($this->getEnvironment());

            // As you can see it just casts $pushedDevices (see code above) to boolean value
            // which is kind of wrong, because we don't know if request failed
            if ($adapter->push($push)) {
                $push->pushed();
            }
        }

        return $this;
    }

What do you think?

@lezhnev74 have you tried to call adapter getResponse method?
Output for Android is
#response: array:5 [▼
    "multicast_id" =><PHONE_NUMBER>830258422
    "success" => 1
    "failure" => 1
    "canonical_ids" => 0
    "results" => array:2 [▼
      0 => array:1 [▼
        "error" => "NotRegistered"
      ]
      1 => array:1 [▼
        "message_id" => "0:1500286067905737%51282618f9fd7edf"
      ]
    ]
  ]

The first token was invalid, the second one was delivered.

It will be improved with an accomplishment of this issue: https://github.com/Ph3nol/NotificationPusher/issues/151

Closed by https://github.com/Ph3nol/NotificationPusher/issues/151
See https://github.com/Ph3nol/NotificationPusher/blob/master/doc/facades.md


674 | Longest Continuous Increasing Subsequence | C++

Title - C++ Solution for 674
What will change - Add new code
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

Can you assign to me please

@jDerp You wanna work on this?

[x] Assigning to @jDerp for now.


Update50

Update50 in terminal
failed to connect extension server on port 1337
restarting codespace did not resolve issue
ADD ISSUE DESCRIPTION HERE
Version: 1.85.2
Commit: 8b3775030ed1a69b13e4f4c628c612102e30a681
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
Embedder: codespaces

[F1] => run command extension.bisect.start

Assuming this relates to CS50 please close this issue and contact the course support channel.


is there any doc or paper that introduce the function of cImpPDController::CalcControlForces()?

i don't understand the function of cImpPDController::CalcControlForces(), any idears is appreciated. thanks.

I know DeepMimicCore.sln is to build DeepMimicCore.py, but what is the use of DeepMimic.sln?
Also, do I have to set the path of OpenGL, GLEW, freeGlut when building DeepMimicCore.py?
Can Anybody help me, please, btw, I'm using windows10.

I think it is related to PD controller, but I don't know the detail yet, maybe you can read the paper and share with us.

It is implementing a semi-implicit pd  controller from this paper:
https://www.cc.gatech.edu/~turk/my_papers/stable_pd.pdf
It is a lot more stable than a regular explicit pd controller. Though the code can be a bit hairy.

thanks both of you.


Added faq-accordion

Q
A

Bug fix?
no

New sample?
yes

Related issues?

What's in this Pull Request?
Added faq-accordion

Closing as I have done the pull request incorrectly. Will start over.


Eq 410 navigator

What is the context of this PR?
Removing the navigator and replacing it with the User Journey Manager. Also cleaning up the User action processor and moving the routing components into the routing engine.
How to review

Check out this branch
Check the application still works
Ensure all tests pass

[ ] Do all commits have sensible commit messages?
[ ] Is all new code unit tested?
[ ] Are there integration tests if needed?
[ ] Is there sufficient logging?
[ ] Is there documentation or is the code self-describing?

Who worked on the PR
@warrenbailey

Happy to merge this now?

Yep am now


Show number of connected peers

System_peers: Returns the list of peers from the [network_service::NetworkService] that are used to synchronize blocks. Returns, for each peer, their identity and best block number and hash.

System_peers: Returns the list of peers from the [network_service::NetworkService] that are used to synchronize blocks. Returns, for each peer, their identity and best block number and hash.

Unfortunately that method doesnt seem to be publicly available to using system_health RPC as per Substract Connect for now.


ArrayIndexOutOfBoundsException in GridScrollExtension.getColumnWidth

hello! I encountered an exception using the following code
    /**
     * resets columns widths to the widths calculated by Vaadin.
     */
    private void resetColumnWidths() {
        grid.getColumns().forEach(column -> {
            double columnWidth = gridScrollExtension.getColumnWidth(column.getId());
            if (columnWidth > 0) {
                column.setWidth(columnWidth);
            }
        });
    }

I have a grid and a gridScrollExtension.
when i use the resetColumnWidths method on a freshly created grid everything works as expected.
when i remove grid's columns and add other columns and invoke again the method resetColumnWidths the following ArrayIndexOutOfBoundsException might get raised
java.lang.ArrayIndexOutOfBoundsException: 17
	at org.vaadin.extension.gridscroll.GridScrollExtension.getColumnWidth(GridScrollExtension.java:275)
	at it.imc.vaadin.ui.utils.grid.DynaEditableGrid.lambda$resetColumnWidths$0(DynaEditableGrid.java:146)
	at java.util.ArrayList.forEach(ArrayList.java:1249)
	at java.util.Collections$UnmodifiableCollection.forEach(Collections.java:1080)
	at it.imc.vaadin.ui.utils.grid.DynaEditableGrid.resetColumnWidths(DynaEditableGrid.java:145)

Do you have any suggestions on the matter? Am i using the api in a wrong way? If there is a bug and you pinpoint me where and how to solve it i can try to create a pull request.
thank you for your time!

There is clearly a bug in the extension. The above or NPE can happen. The happens if there is column matching the id. NPE can happen if you you call the method with null. Will fix.

Fixed in 2.5.0. -1.0 returned if column not found.

thank you!


Consolidate modules

Consolidates model into kernel and base into inject - more to do
part of #701
Depends on #700

@bplommer do you mind if we close this PR in favour of #742?


Jitsi prompts to reenter password, password is forgotten, after connection timeout.

I discussed this on the Jitsi mailing lists and the link to that discussion is:
https://www.mail-archive.com/dev%40jitsi.org/msg01019.html
<EMAIL_ADDRESS>(Issue migrated from https://trac.jitsi.org/ticket/1290)

I am also affected by this bug which I believe is rather critical. I noted that the above links posted by jitsi-jenkins no longer work, I would appreciate if they were updated.

I also have this bug; it is a real hassle to deal with. Any chance someone could work on this?

Our organization is also very much affected by this behavior of jitsi.
Even thou this seems quite basic, it seriously affect the usability of jitsi.
My users are abandoning jitsi and uses external instant messaging services instead.
Please help make this critical.

hello,  this problem is more frequently meet on a long term use of Jitsi , it apears very frequently. please solve it.