Help Wanted: writer to summarize KEPs

KEP Writer
Responsibilities
Major Kubernetes features and other changes are implemented in the form of KEPs.  Our contributor community often struggles to keep up with the various KEPs in progress and their status.
This writer would canvass open KEPs, and write weekly blurbs of 50-100 words about 1-3 of them that are either new or have had recent significant activity.  Example:

KEP 3077: Contextual Logging: this recently-approved KEP will eliminate klog dependency, passing logs from components to a contextual logger instead, via a logr.Logger call passed into all functions.  This allows log output to be redirected without routing it through a centralized klog.  It will also improve logging unit tests.

Requirements
Writers will be familiar with Kubernetes, its features, and Kubernetes development in general.  They will have adequate English composition skills.  Writers will be able to search out KEPs with recent activity on their own, without needing explicit direction.

@jberkus I'm interested to volunteer for this. I've been working with Kubernetes everyday for the past 11 months at my job, and I've contributed to sig-docs as well before.

Reached out to you on Slack.

Is this issue still relevant?

@satyampsoni we have several contributors working on this now.  However, if you have specific KEPs you wanted to try summarizing, that would be great.  You can find us on #sig-contribex-comms in Slack


Switch to pyproject.toml for Python package

This is the Correct Python Packaging way of doing things now.

Resolved by https://github.com/replicate/cog/pull/843


Fix typo & false warning condition check in ChatClient.whisper()

Type: Bugfix

Fixes: #247

Thanks for spotting and fixing these!


Locks not telling the truth

From @zzottel on August 8, 2014 20:26
If on a thread that is completely open to the internet someone comments who has his channel privacy settings set to stricter options, the comment is shown with a lock.
In fact, though, this comment is not restricted and can be viewed by anyone on the internet. (Tested with a thread on parlementum.net which was at version 2014-08-07.760R+9a16955.)
Copied from original issue: friendica/red#558

From @zzottel on August 14, 2014 19:48
Fixed.


Teeth of Lotan : Fail to load all attributes

In game

In TQVault

Share.vault.zip

False alarm!
My game files were on "publicbeta" channel.
Tool and game are aligned on release channel.


pull request for e2e test at 11/08/2016 11:26:09

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

Details

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update italian language

@4gray
Please check and merge. Thanks.

Codecov Report
Base: 66.27% // Head: 66.27% // No change to project coverage :thumbsup:

Coverage data is based on head (a1bfaeb) compared to base (4ce405a).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##           electron     #229   +/-   ##
=========================================
  Coverage     66.27%   66.27%
=========================================
  Files            48       48
  Lines           925      925
  Branches         95       95
=========================================
  Hits            613      613
  Misses          283      283
  Partials         29       29

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Branch issue90 student num

Updating all Phone-related classes, methods, comments, and tests to StudentNumber
closes #90

Codecov Report

Merging #98 (9a53493) into master (046623a) will decrease coverage by 0.03%.
The diff coverage is 88.33%.

@@             Coverage Diff              @@
##             master      #98      +/-   ##
============================================
- Coverage     64.23%   64.19%   -0.04%
+ Complexity      455      454       -1
============================================
  Files            90       90
  Lines          1507     1511       +4
  Branches        160      160
============================================
+ Hits            968      970       +2
- Misses          490      491       +1
- Partials         49       50       +1

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
87.50% <ø> (ø)

.../java/seedu/address/model/util/SampleDataUtil.java
20.00% <ø> (ø)

src/main/java/seedu/address/ui/PersonCard.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/FindCommandParser.java
40.00% <25.00%> (ø)

...java/seedu/address/model/person/StudentNumber.java
80.00% <80.00%> (ø)

...c/main/java/seedu/address/model/person/Person.java
90.90% <85.71%> (ø)

...java/seedu/address/logic/commands/EditCommand.java
97.26% <100.00%> (+0.03%)
:arrow_up:

...a/seedu/address/logic/parser/AddCommandParser.java
93.33% <100.00%> (ø)

...ain/java/seedu/address/logic/parser/CliSyntax.java
85.71% <100.00%> (ø)

.../seedu/address/logic/parser/EditCommandParser.java
92.59% <100.00%> (+0.28%)
:arrow_up:

... and 4 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 046623a...9a53493. Read the comment docs.

Looks good to Merge, all issues have been fixed

LGTM!


Update for Prometheus Community

Migrate the codebase to the Prometheus Community

Add licenses headers
Add NOTICE file to record the copyright change.
Remove non-standard goreleaser config.
Remove non-standard .tool-versions config.
Add promu and Prometheus common Makefile setup.
Update Go module paths.

Small suggestions:

security.md should point to<EMAIL_ADDRESS>Use prometheus-community as org name in LICENSE.md

There will probably be a few more things to clean up. Just wanted to get the basic stuff done to make the build work.


WIP: Config dns in guest vms

What this PR does / why we need it:
This PR provides 2 features for the ipv6-masquerade PR #3112 :

read the DNS server IP address from the k8s-dns service, passing it via cloudInit to the VMs
configure the DNS server IP address of VM guests of running VMs
make the ip address setting of ipv6 addresses more robust

Special notes for your reviewer:
Reviewers should focus on the last 3 commits. If those are found good, they should move them to the #3112 PR.
The relevant commits are:

https://github.com/kubevirt/kubevirt/pull/3183/commits/49d27df9eddfb71f68d543c8ba76fdde8635dd9a
https://github.com/kubevirt/kubevirt/pull/3183/commits/0867760f26d2743cdd593d2ce1444d7e9b3ef1af
https://github.com/kubevirt/kubevirt/pull/3183/commits/f6bc9dddbb2a7825bdb3f8c90dc3dfadd44460d8

Release note:
NONE

/cc @AlonaKaplan @qinqon

Abandoning in favor of https://github.com/kubevirt/kubevirt/pull/3112/commits/77f372fb63cceada445774bc1161345d9de0d66c


Add Pricing Pages and Fix #60

This PR aims to add Pricing Pages in the App and Fixing #60
Please review this @fredfalcon, @souravbadami.
Thanks
https://user-images.githubusercontent.com/44541855/129634345-a7008db3-5043-4825-8619-f9f95a88daf7.mp4

This can be done with a simple carousel plugin, no need for 41 additional files

Was not able to find the exact corousel for this usecase. So created my own corousel. We could merge it as a diff library as well. Also some other files have features code as well, which is dependent on each other.
I was waiting for your review for that.


Setup Continuous Integration

We should research and identify a continuous integration solution for our spark jobs. This could be CircleCI, Travis, etc. Measure of success is running all unit tests when any git push occurs.

We currently have a basic CI setup on Travis. As of now, it just runs the tests, but we could also configure it to automatically publish new versions or do anything else that we can shell-script. We can deploy credentials to Travis by adding them to our config file and encrypting them with the Travis key.

Going to close this one off since the success criteria in the original item was fulfilled.


Fixing how project config is updated. Also, fixing issue in Optimizely.close()

Summary
This change fixes:

Optimizely.close() which currently checks that eventHandler and projectConfigManager are instances of Closeable. Changing that to check if they are instances of AutoCloseable.
ProjectConfig is refetched every time in getVariation. Using the passing in config itself.
ProjectConfig is fetched multiple times when getEnabledFeatures is called. Updating isFeatureEnabled.

Test plan

Unit tests.

I realize I have to fix 2 tests in OptimizelyTest.java. Following up with that.

Pull Request Test Coverage Report for Build 1091

11 of 12 (91.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 89.647%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-api/src/main/java/com/optimizely/ab/Optimizely.java
11
12
91.67%

Totals

Change from base Build 1080:
0.003%

Covered Lines:
3299

Relevant Lines:
3680

💛  - Coveralls

Pull Request Test Coverage Report for Build 1091

11 of 12 (91.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.003%) to 89.647%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

core-api/src/main/java/com/optimizely/ab/Optimizely.java
11
12
91.67%

Totals

Change from base Build 1080:
0.003%

Covered Lines:
3299

Relevant Lines:
3680

💛  - Coveralls


locationProvider.locationWillChange.sink  not called when start location udpates

locationProvider.locationWillChange.sink { loc in
// handleLocation(loc)
}
btw, locationProvider.location?.coordinate has instant values for current location.

Hi, could you check if the most recent change resolved your bug? Thank you!


Find os version for CentOS 6.5

@Rajesh-Rajagopal  Nice. Good work.

Thank You sir..
On Apr 27, 2016 6:44 PM, "Kishorekumar Neelamegam"<EMAIL_ADDRESS>wrote:

@Rajesh-Rajagopal https://github.com/Rajesh-Rajagopal Nice. Good work.
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/megamsys/libgo/pull/57#issuecomment-215077718

@rajthilakmca  Can you verify and merge this request ?


Issue739 spectro reverse rt

Main changes:

Several improvement to vos computation for non-spectrometers
BUGFIX: better interpolation of polygons in equivalent_aperture()
BUGFIX: more rbust root finding in pathological cases for reflections on crystals
BUGFIX: dlamb was overestimated by a factor ~2 from the LOS-approximation
BUGFIX: rocking curve now takes into account crystal to last aperture distance for LOS-approximation
BUGFIX: in some cases, reflection on a plane could lead to negative kk values that should be eliminated
Re-implemented: compute_diagnostic_etendue_los(numerical=True) for non-spectro
plot_diagnostic_vos() implemented for spectro and non-spectro
compute_diagnostic_solidangle_from_plane() operational for spectro and non-spectro
get_raytracing_from_pts() implemented, only used for spectrometers
compute_diagnostic_vos(replace_poly=bool) implemented to update vos polygons in cross-section
get_optics_reflections_ptsvect(isnorm=bool, fast=bool) implemented for faster spectro reflections
some refactoring / moving routines around
More debugging options (commented but available for later re-use)
tuto_real0.py updated to easily create test spectrometers (flat crystal, cylindircal crystal and non-spectro equivalent)

On the whole, etendue, taking into account rocking curve and dlamb, now agree within 30-40 % between the LOS approximation and a full VOS reverse ray-tracing (tested on a flat and on a cylindrical von hamos crystal)
Issues:
Fixes, in devel, issues

#739
#766

Examples:
Using code from tuto_real0.py, we can quickly create 2 spectrometers, with respectively a flat crystal and a cylindrical (Von Hamos) crystal
In [1]: import tofu as tf

In [2]: test = tf.tests.tests09_tutorials.tuto_real0

In [3]: conf, coll = test.main()

Now, we use the method to generate a 2d plane perpendicular to a LOS of reference of the camera, sample that 2d plane to compute the locally at each point, and using both a LOS  approximation and a VOS / reverse ray-tracing approach.
For a non-spectrometer diagnostic (detector + apertures), it allows to check to visualize the 2d map of solid angle associated to a single channel and see from which resolution of the plane sampling the numerical etendue matches the analytical etendue.
In [4]: dout = coll.compute_diagnostic_solidangle_from_plane(key_diag='d0', key_cam='bb1', res=[0.01, 0.01], config=conf, margin_perp=[0.01,0.02], margin_par=0.5)

With a sufficient resolution we get a converged integral => etendues match
In [5]: dout = coll.compute_diagnostic_solidangle_from_plane(key_diag='d0', key_cam='bb1', res=[0.001, 0.001], config=conf, margin_perp=[0.01,0.02], margin_par=0.5)

Note that even after convergence (sufficient resolution), the VOS and LOS etendue only match at the reference channel, that's because the etendue is dependent on the direction of observation and the integral on the plane is only equal to the etendue for the LOS perpendicular to that plane.
For a spectrometer, the routine will compute and compare:

the solid angle (step 1 of the comparison)

subtended by each camera pixel (through the slit and via the reflection on the crystal)
purely geometric, not taking into account the rocking curve

the solid angle convolved by the rocking curve  (step 2 of the comparison)

the LOS approcah uses an approximation based on the minimum between the typical angular width of the rocking curve and the angular width subtended by  pixel

the solid angle, convolved by the rocking curve, times the wavelength sampling step (dlamb), which gives the per-pixel quantity by which to multiply the LOS-integral at he reference wavelength to get the total photon flux per pixel.

This 3-steps comparison with increasing complexity was designed to facilitate debugging and understand where most of the differences come. They mostly come from how the 2 approaches model the rocking curve in the case of the flat crystal.
For the flat crystal:
In [4]: dout = coll.compute_diagnostic_solidangle_from_plane(key_diag='d03', key_cam='c0_camf', res=[0.0001, 0.005], config=conf, margin_perp=[0.01, 0.05], margin_par=13., n0=100, n1=100, res_lamb=0.02e-13)
		32507 / 32508 pts angmin dangmin: rock 2.36e-07 vs 1.65e-04 1/4 pixel

For the von hamos crystal, most of the difference comes from the ray-tracing (so probably from how the reflection of the slit is computed for the LOS-approach), later stages reduce the error.
In [5]: dout = coll.compute_diagnostic_solidangle_from_plane(key_diag='d02', key_cam='c0_cam', res=[0.0001, 0.005], config=conf, margin_perp=[0.01, 0.10], margin_par=13., n0=100, n1=100, res_lamb=0.02e-13
    ...: )
		18743 / 18748 pts angmin dangmin: rock 2.36e-07 vs 1.65e-04 1/4 pixel

Finally, the compute_diagnostic_vos() and plot_diagnostic_vos() methods samples a volumic mesh and computes the local solid angle (for non-spectro diags) convolved by the rocking curve (for spectro diags, it also stores extra quantities like the local cosine of the angle vs the toroidal direction).
For a non-spectrometer:
In [6]: dvos = coll.compute_diagnostic_vos('d0', key_mesh='m0', res_RZ=[0.01, 0.01], res_phi=0.001, visibility=False, store=False)

In [8]: dax = coll.plot_diagnostic_vos('d0', dvos=dvos, plot_config=conf)

For a spectrometer:
In [6]: dvos = coll.compute_diagnostic_vos('d03', key_mesh='m0', store=False, visibility=False, timing=False, verb=True, res_RZ=[0.05, 0.02], res_phi=0.0003, res_lamb=1e-15, res_rock_curve=25, debug=False,
   ...:  n0=10, n1=10)

Computing vos for diag 'd03':
Recommended res_lamb to ensure rocking curve overlap:
	- edge-edge: 	1.99e-14
	- MH-to-MH: 	3.27e-15
	- resolution: 	8.29e-16
	Provided: 	1.31e-15

	lamb.shape: (1248,)
	ang_rel.shape: (25,)
	iru.size: 31
	nRZ: 359

		30 / 31, 10 / 11, 407 / 408:  82 pts   	 dangmin: rock 3.94e-06 vs 1.72e-04 1/4 pixel

In [7]: dax = coll.plot_diagnostic_vos('d03', dvos=dvos)

@cjperks7 , @adriendaros , I'm putting you both as reviewers.
This is a big PR, so dont' feel like you have to go through everything, it's mostly to make sure you are informed of what is going on and you see what is possible / debugged / working.
Don't hesitate to try and reproduce some of these figures if you feel like digging a bit.

Bit of a sanity check, you started this exercise to have an improved calculation with the VOS method over the LOS method but now you're quoting that the agree within 30-40%. Wouldn't you only expect that to be in with an asymptotically long and narrow spectrometer? How are you making that comparison?

The comparison is done on the total photon flux per LOS length, assuming an uniform emissivity of 1 ph/s/m3/sr/AA.
You get a photon flux per LOS length (ph/s/m) by mutipliying that unity emissivity by a factor computed as:

LOS:  multiplying the etendue (taking into account the rocking curve) by dlamb (m2.sr x wavelrngth in m = sr.m2.AA)

VOS: integrating over a plane perpendicular to the LOS (surface => m2) the local solid angle (sr) convolved by the rocking curve (computed as a sum of independent rays traced to the camera), integrated over a large sample of wavelength (dlamb) => sr.m2.AA

This is kind of a generilzation of the definition of a LOS and etendue


Metadata v14 content

With the new scale-info features, many need to migrate and in general we should discuss and inform builders of it.

What versions of Substrate include this?
What changes in your nodes need to be made?
What changes in clients/apps need to be made?

Migration guide: https://gist.github.com/ascjones/0d81a4c44e84cacd9f714cd34a6de823

@ascjones could I ask you to add a few links/resources in this issue so we can make this content? :pray: Any key things we should point out?

Pro tip for easy grabbing of a local node ( or remote with RPC unsafe ports open?):
curl -H "Content-Type: application/json" -d '{ "id": 1, "jsonrpc": "2.0", "method": "state_getMetadata"}' http://localhost:9933

@ascjones - I still woudl love to see more formal content around this - are you able to help out here? :grinning:

https://docs.substrate.io/main-docs/build/application-dev/#metadata-system what we have so far
#494 related


Public Filters won't appear in some random occasions, until the page is manually refreshed

Even an auto-refresh (_ar keyword) doesn't make them re-appear.  The only solution is hitting F5 (refresh) or such.
Norm

Fixed in 0.17.1.


Add Scheduler and Task Metrics for SQL Server

What does this PR do?
Added collecting metrics from two additional tables:

sys.dm_os_schedulers
sys.dm_os_tasks

Their class names are SqlOsSchedulers & SqlOsTasks respectively.
SqlOsSchedulers

Tagged with session_id
Metrics: Link

SqlOsTasks

Tagged with session_id & scheduler_id
Metrics: Link

Motivation
Requested by customer
Issue AI-559
Additional Notes

These additional metrics are added by default and are not via custom metrics.
This PR does not fully address the customer's request as they would like metrics from custom queries which is not a supported feature at the moment.

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Rebased on the latest master branch (notably including the #7488 changes) and made a minor change to address @hithwen comment.
This change dramatically expands the number of metrics due to the high cardinality of the task and scheduler tags - from ~10 to ~200.


Fix the ElasticsearchStore key error

Change None to empty dict and add get metadata using function get to avoid key error.
Description
When there is no metadata in the Elasticsearch index, a KeyError: 'metadata' occurs during retrieval. If you try to use the get function to avoid this key error, a new error arises from Pydantic. ValidationError: 1 validation error for TextNode
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested?

[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

The error is just because is expected a dict by TextNode, and not a None.
I changed None to empty dict, it solves the problem.
traceback:
ValidationError: 1 validation error for TextNode
metadata
  Input should be a valid dictionary [type=dict_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.8/v/dict_type

The error is just because is expected a dict by TextNode, and not a None. I changed None to empty dict, it solves the problem.
traceback:
ValidationError: 1 validation error for TextNode
metadata
  Input should be a valid dictionary [type=dict_type, input_value=None, input_type=NoneType]
    For further information visit https://errors.pydantic.dev/2.8/v/dict_type

Got it, thanks! Just wanting to confirm if there was anything needed to be done on TextNode class itself. Your solution is the right one.

I had the same question.


how to Stream Search tool responses

Issue you'd like to raise.
search = SerpAPIWrapper()
tools = define_tools(llm_chain,search)
tools = [
Tool(
name="Search",
func=search.run,
description="useful for when you need to answer questions about current events",
# return_direct = True
),
]
prompt = CustomPromptTemplate(
template=template,
tools=tools,
input_variables=["input", "intermediate_steps", "chat_history"]
)
output_parser = CustomOutputParser()
llm = ChatOpenAI(temperature=0, model_name = "gpt-4-0613", streaming = True, callbacks = [MyCustomHandler(new_payload=new_payload)])
llm_chain = LLMChain(llm=llm, prompt=prompt)
agent = LLMSingleActionAgent(llm_chain=llm_chain, output_parser=output_parser, stop=["\nObservation:"], allowed_tools=tools)
agent_executor = AgentExecutor.from_agent_and_tools(agent=agent,
tools=tools,
verbose=True, memory = memory, early_stopping_method='generate')
Suggestion:
No response

This issue doesn't have any supporting information... Only an unformatted script.
What is the issue you're facing?


feat(pixels): add my new pixel (0,4)

Checklist

[ ] I ran npm test locally and it passed without errors.

[x] I only edited the _data/pixels.json file.

[x] I entered the username in the pixels.json that I'm also using to create this pull request.
[x] I acknowledge that all my contributions will be made under the project's license.

Hi I have made the changes only on the _data/pixels.json file why I am getting 1 failed while running the command npm run


Web-Console: Add show and copy actions to column tree

adds show and copy buttons in a pop over to the column tree menu in query view

Looks like the package-lock got out of sync you need to reinstall the packages and push

Thank you for addressing my feedback so vigorously. I am playing with this locally and it looks great. This is g2g after CI 👍


New read-mode:  api-read-only - 2.0

Change Description

New read-mode = api-read-only explained below.

p2p transactions are not processed.
Current blocks are not broadcast.
It is possible to sync from a nodeos running in this mode.

Resolves #8685

Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[ ] Documentation Additions
read-mode = api-read-only
In api-read-only mode database contains changes done up to the head block; only api transactions are speculatively executed. Similar to head read-mode in that chain api queries such as get_table_rows returns data as of the head block and does not include any speculative execution from any transactions either from p2p network or submitted by push_transaction or send_transaction.

This could use a better name.  I don't see how the api's are read-only in this mode.

@swatanabe-b1 point well taken, we decided on api

We have discussed this some more and have decided that api and the existing read-only are not really database read-modes. Plan on adding p2p-accept-transactions and  api-accept-transactions both defaulted to true.  The existing read-only will be deprecated in favor of specifying: read-mode = head, p2p-accept-transactions = false, and api-accept-transactions = false. For the v2.0.x & v1.8.x versions of this PR read-mode will indicate a warning message of deprecation and will automatically set p2p-accept-transactions = false, api-accept-transactions = false, and read-mode = head. The develop version of this PR will remove read-only mode.


docs(DataTable): add reference stories for row actions

Closes https://github.com/github/primer/issues/1816
Changelog
New

Add story for a single row-level action
Add story for two row-level actions
Add story for row-level actions in a menu

Changed

Make field optional for "virtual" columns like a sortable column which has no value in the data set
Update header from string to string | (() => React.ReactNode) to support custom rendering

Removed

@colebemis @mperrotti this is currently adding stories-only for these patterns just to see how far we could go with what we have built-in.
Curious if you all had any recommendations or ideas for ways to help abstract or make row-level actions easier for consumers to implement 👀

Is this space between the icon button and the edge of the table intentional?

Curious if you all had any recommendations or ideas for ways to help abstract or make row-level actions easier for consumers to implement 👀

Maybe we could export a component you can use in renderCell that makes it easy to render one or more actions?

@ericwbailey - should the row actions be in a role="toolbar" and only have one tab stop? Or should we keep them as separate unrelated buttons?

@colebemis seems like it's coming from auto-sizing of table cells from the table layout algorithm 😞
I'll defer this for after @mperrotti's work for column layout and see if it persists or if we need to target this case specifically through some kind of abstraction 👍

@mperrotti I'd love to make something simple but couldn't seem to land on the right abstraction 🤔 Curious if you have any ideas or places to take inspiration from that would help to simplify this.
One idea was to introduce primitives like:

RowAction
RowActionGroup (which would contain RowAction)
RowActionMenu, RowActionMenuItem

Where RowAction would ostensibly be a wrapper for IconButton and RowActionMenu would be a wrapper for ActionMenu. They didn't seem to add much value though.
Here's what some of the stories might look like with this in mind:
With Row Action
// Before
<IconButton
  aria-label={`Download: ${row.name}`}
  title={`Download: ${row.name}`}
  icon={DownloadIcon}
  variant="invisible"
  onClick={() => {
    action('Download')(row)
  }}
/>

// After
<RowAction
  rowHeader={row.name}
  action="Download"
  icon={DownloadIcon}
  variant="invisible"
  onClick={() => {
    action('Download')(row)
  }}
/>

With Row Actions
// Before
<>
  <IconButton
    aria-label={`Edit: ${row.name}`}
    title={`Edit: ${row.name}`}
    icon={PencilIcon}
    variant="invisible"
    onClick={() => {
      action('Edit')(row)
    }}
  />
  <IconButton
    aria-label={`Delete: ${row.name}`}
    title={`Delete: ${row.name}`}
    icon={TrashIcon}
    variant="invisible"
    onClick={() => {
      action('Delete')(row)
    }}
  />

// After
<RowActionGroup rowHeader={row.name}>
  <RowAction
    action="Edit"
    icon={PencilIcon}
    variant="invisible"
    onClick={() => {
      action('Edit')(row)
    }}
  />
  <RowAction
    action="Delete"
    icon={TrashIcon}
    variant="invisible"
    onClick={() => {
      action('Delete')(row)
    }}
  />
</RowActionGroup>

With Row Action Menu
// Before
<ActionMenu>
  <ActionMenu.Anchor>
    <IconButton
      aria-label={`Actions: ${row.name}`}
      title={`Actions: ${row.name}`}
      icon={KebabHorizontalIcon}
      variant="invisible"
    />
  </ActionMenu.Anchor>
  <ActionMenu.Overlay>
    <ActionList>
      <ActionList.Item
        onSelect={() => {
          action('Copy')(row)
        }}
      >
        Copy row
      </ActionList.Item>
      <ActionList.Item>Edit row</ActionList.Item>
      <ActionList.Item>Export row as CSV</ActionList.Item>
      <ActionList.Divider />
      <ActionList.Item variant="danger">Delete row</ActionList.Item>
    </ActionList>
  </ActionMenu.Overlay>
</ActionMenu>

// After
<RowActionMenu>
  <RowActionMenuItem onSelect={() => { /* ... */}>
    Copy Row
  </RowActionMenuItem>
  <RowActionMenuItem>Edit row</RowActionMenuItem>
  <RowActionMenuItem>Export row as CSV</RowActionMenuItem>
  <RowActionMenuItem.Divider />
  <RowActionMenuItem variant="danger">Delete row</RowActionMenuItem>
</RowActionMenu>

I like the RowActionGroup/RowAction/RowActionMenu abstraction, but I'm also ok with moving forward with the more verbose API for now.
Also just wanted to remind the group that we're not restricting consumers to icon-only buttons. It's just the preferred format to use when possible. There might be actions that can't easily be represented with an icon.


SAK-45370 Update calc q regex to avoid catastrophic backtracking

Convert non-greedy regular expression pattern for a calculated question
calculation to possessive version to avoid backtracking that can cause
stackoverflow errors or CPU load that brings down the system.

@davidpbauer I think is better to make the changes as indicated in the description. Thanks for worried about this problem!.

Making the changes in the description would require us to rewrite or update almost 18,000 calculated questions that we have locally. We'd also need to re-train all of our faculty to create their questions. I agree that the root of the problem is that we shouldn't be using regex to parse instructor-created html text, but if the main issue is CALCQ_CALCULATION_PATTERN then this change fixes that and we can move on without impacted every calculated question and every user of calculated questions.

Making the changes in the description would require us to rewrite or update almost 18,000 calculated questions that we have locally. We'd also need to re-train all of our faculty to create their questions. I agree that the root of the problem is that we shouldn't be using regex to parse instructor-created html text, but if the main issue is CALCQ_CALCULATION_PATTERN then this change fixes that and we can move on without impacted every calculated question and every user of calculated questions.

You don't need to rewrite or update any calculated question.

Making the changes in the description would require us to rewrite or update almost 18,000 calculated questions that we have locally. We'd also need to re-train all of our faculty to create their questions. I agree that the root of the problem is that we shouldn't be using regex to parse instructor-created html text, but if the main issue is CALCQ_CALCULATION_PATTERN then this change fixes that and we can move on without impacted every calculated question and every user of calculated questions.

You don't need to rewrite or update any calculated question.

Perhaps I didn't understand the changes described in the Jira description. I thought the suggestion was to change the characters used by instructors to designate a calculation in their question. For example, [[3*[pi]]]

This is some PhD-level regex mastery. Awesome!
This is also a great reason why we have unit tests for these expressions.

I've added a test for this change.


⚠ Bofrost.de has degraded performance

In b47004c, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1081 ms

Resolved: Bofrost.de performance has improved in e607e47.


Async task support?

It seems like asynchronous task support was removed in 3.0.0, but it is still mentioned in the docs. I don't see any references in the source to support this feature, except for an unreferenced case class in one of the REST API endpoints. (Search for statusCode to find it.)
To be more specific, if I add the "async": true flag to a job, it is simply removed from the job definition when I save.
I never had much luck with async tasks in 2.x, but if this feature is completely gone, it should be noted in the release notes and removed from the documentation. Just had a very frustrating experience with the parameter magically disappearing.

Yep, good point. I forgot to mention it in the release notes. I'll update accordingly.


please make it possible to hide Menu Bar

Describe the solution you'd like
Since most Windows laptops feature 16:9 display, there is no extra space (comparing to Mac 16:10 display) that can be easily sacrificed for Menu Bar. This makes a constant visible Manu Bar a super frustrating thing.
Could you please add a feature into settings to adjust Manu Bar position just as in Cairo Shell the Task Bar already can be changed to auto-hide
Also, it would be nice to add 2-3 different delay options : instant, 1 sec, 2 sec, because as Mac user I may confirm when you work with Web Browser and want to change the Tab, you hover the mouse at the top of the screen and Menu Bar apperas every time, it bothers. 1-2 secs delay could easily solve this issue

As a long-time KDE/Plasma user who's been using Windows again recently and is used to being able to customize the desktop shell quite a bit, I've been giving Cairo a shot to try and fix some of my grievances with the stock Explorer shell. It definitely bugs me that I cannot hide the top bar as I don't want a bar just for a systray and some folder shortcuts eating screen real estate. This is compounded by the fact that the taskbar can be hidden, so why not the menu bar as well?

Thanks for the feedback! I am currently working on adding this feature in #120. Closing this as a duplicate of #120.


gradle: support "discovery" of extra_settings.gradle

Automatically include modules from other repos cloned in the same directory as GearVRf. Needed for the internal build system.
GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Planning to merge on Thursday.


First CLaSP classes

Summary of the changes

This moves Razor onto the CLaSP Framework being developed over on dotnet/roslyn. That gives us access to the method of getting DocumentContext which Roslyn started using a Context object on the request.
This won't compile as I've just included the base classes and not updated any of their usage points. I'll send a couple PR's updating usages once this is merged into the feature branch.
Probably shouldn't merge this until the Microsoft.CommonLanguageServerProtocol.Framework package is available.

Begins fixing https://github.com/dotnet/razor-tooling/issues/6250.

LGTM! Just had a few clarifying questions. I also wonder if these changes will all play nicely with VS Code - there's nothing that stands out to me as concerning currently, but definitely something we should verify before merging

Agreed both that there's nothing which ought to cross the line into VSCode so long as we're following the LSP spec and that we should check anyway when the time comes.


make waterway=riverbank areas more blue

White or whiteish rendering is not making clear that it a water area, checking that requires clicking on it - and even then it may be not clear for newbies.

https://www.openstreetmap.org/edit?editor=id&node=4306153491#map=18/15.35375/73.99960
Compare with natural=water

With the next release of iD, anything tagged with waterway=riverbank will be flagged as depreciated with an option to upgrade to natural=water + water=river.

Yes, previously we've supported rendering deprecated tagging, but going forward we are using generic styling for deprecated features to encourage users to upgrade them.


[Event Change] Codeunit::"Copy Document Mgt." - Function TransfldsFromSalesToPurchLine - OnBeforeTransfldsFromSalesToPurchLine

We like to request the following Event:
Object: Codeunit::"Copy Document Mgt."
Function: TransfldsFromSalesToPurchLine
Publisher Event: OnBeforeTransfldsFromSalesToPurchLine
Reason: add an extra Parameter
Where is the event placed:
Event Code:
[IntegrationEvent(false, false)]
local procedure OnBeforeTransfldsFromSalesToPurchLine(var FromSalesLine: Record "Sales Line"; var ToPurchaseLine: Record "Purchase Line"; var IsHandled: Boolean)
begin
end;

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


Fix Magpie testing script to always find all test files

This pull request removes the include_dir! macro use which only works on compile time to use default rust file searching in order to always find all available tests when the testing suite is run instead of only updating which tests to run when magpie is compiled. Should be up to date with master as of starting this request.

This only works on Windows and not Linux/MacOS. Make sure you aren't using Windows path seperators (/) and try this


Concurrency and data corruption

With the concurrency setting in the configuration statsgod will setup multiple consumers for the relayChannel channel:
https://github.com/acquia/statsgod/blob/master/statsgod.go#L94
Each consumer maintains its own hash of metrics.  So how does this not cause data corruption when the same metric key is handled by more than one of the running consumers?  It looks like the individual flush cycles would end up overwriting the same metric from another consumer's flush cycle.

@jjneely thanks for the report! I believe that you are correct. The relay concurrency is a feature that we didn't end up using, but sadly did not get enough attention before going into master.
So, it's possible that the idea of concurrent relays is just fundamentally not going to work. Do you see any alternative, or do you think that we drop concurrent relays altogether?

In the branch I'm working on I removed the for loop there.  Otherwise, we need a common data structure, locking, and a new thread to handling flushing.
I'd much rather depend on the channel for synchronization there.  I haven't looked at how long we spend in various stages of the code so...

That sounds good to me. We will also want to make sure that we remove the configuration around config.Relay.Concurrency.


tea command doesn't execute python scripts

The README says tea can be used to run scripts, it it doesn't work.

➜  stable-diffusion git:(main) ls
LICENSE                           ldm
README.md                         main.py
Stable_Diffusion_v1_Model_Card.md models
assets                            notebook_helpers.py
configs                           scripts
data                              setup.py
environment.yaml
➜  stable-diffusion git:(main) tea setup.py
tea: command not found: setup.py
➜  stable-diffusion git:(main)

Try tea -X python setus.py. I'm not sure if it will work. This might work also. tea +python.org ./setup.py  The idea is one needs to tell tea what dependencies are needed. The first one does it with -X python, basically saying to add python's dependencies. The second one does it by specifying the dependency, +python.org.

tea setup.py should work yeah. Let me see if we accidentally broke this.

0.11.7 is being built. You should be able to upgrade with sh <(curl tea.xyz) in about 20 mins.
Please LMK how using it for stable-diffusion goes. My intent is to support python and pypa modules first class including pre-building any binaries for modules required, but my experience with Python is relatively limited so I'll need pointers. 🙏


Invalid Data Notification on Save/Export

Could have a notification for when a user tries to export/save a validated spreadsheet that has invalid fields. We don't want to prevent export, but this could be helpful for users who may have missed correcting a field by accident. Perhaps the notification could have a 'more info' option that would then list out the invalid cells.

@wwhsiao raised the possibility of not allowing export at all if the spreadsheet has invalid cells.
But I like the idea of having this notification on save.

Definitely allow save with errors.  Errors can take multiple work sessions to resolve completely.

To clarify, @wwhsiao raised the possibility of stopping export to... not save as...

Ah, of course!


add del to parallel set

As discussed in #65

:+1: big commit :)

@tomekwi thanks man :smile_cat:

:bell:

Checking this out now.

LGTM, waiting on readme tweak about keypaths

Oh well didn't realize your example utilizes the key path functionality. I think that is sufficient for now

<EMAIL_ADDRESS>


[BUG] The custom buttons do not receive tooltips

Hello,
First of all, thank you for your contribution.
The custom buttons do not receive tooltips, which breaks the consistency.
My suggestion would be:
Customization of Tooltips, Border Radius, Font, and Font Size in the Rich Text Editor
I'm using a rich text editor in my application, and I would like to know if it is possible to make the following customizations:
Tooltips Label for Each Button: I would like to customize the tooltips for each button, changing the text (label) to be compatible with the local language of the platform I am developing (in my case, to Portuguese).
Border Radius: Is it possible to customize the tooltip borders to a radius of 8 to align with the visual identity of my application?
Font and Font Size: Is there a way to customize the font family and font size of the editor's text to be consistent with the rest of my platform's visual identity?
Enable/Disable Tooltips: Is it possible to add an option to enable or disable the tooltips according to the user's preference?
Thank you in advance for your attention, and any help or guidance on how to achieve these customizations would be greatly appreciated.
Thanks!

Please Help
@anees4ever


Attach mapify.unitypackage to release instead of in the repository

mapify.unitypackage is a build artifact, so personally I think it should be an attachment to the release and not a committed file in the repository. This would also make it easier for users to find the right unitypackage for the Mapify version they use.

Is it not in the the releases too? I can see it here: https://github.com/Insprill/dv-mapify/releases/tag/v0.4.0
The latest one doesn't have a package attached however, I think it's because it didn't change anything from that release (just a few days apart)?


Does tflite_runtime support multiprocessing?

Hi,
Currently I want to speedup my code run yolov3-tiny for object detection on TPU by using multiprocessing or multithreading in python.
As I tried multithreaing, the performance when running more threads is worst than single thread. I also tried to use multiprocessing Pool but the code stop right before function call interpreter.invoke().
Have anyone had tried to multiprocessing inference before and give me some advices?
Here is my code:
def inference(interpreter, img):
    ...
    input_details, output_details, net_input_shape = get_interpreter_details(interpreter)
    img = np.expand_dims(img, 0)
    interpreter.set_tensor(input_details[0]['index'], img)
    # Stop right here
    interprete.invoke()
    #  Do not run the rest of the code

My environment:
Linux 16.04 (arm)
Coral TPU USB Accelarator
python3.5.2
tflite_runtime 2.1.0post
Also, in https://www.tensorflow.org/lite/performance/best_practices#tweak_the_number_of_threads  it is said that I can tweak the number of threads but it seems that this version of tflite_runtime doesn't support set_num_threads() function?
Thanks,

Hi @hamhochoi,
Python multiprocessing is hard to do with Tensorflow.  You can find some ideas here: https://stackoverflow.com/questions/36610290/tensorflow-and-multiprocessing-passing-sessions
If you have more than one edge TPU, you can pipeline the ML to speed up the inference time: https://coral.ai/docs/edgetpu/pipeline/
When TFlite runs on a CPU, it might be an advantage to run a number of threads to spread the computation over multiple CPU cores.  Running TFlite on the edge TPU does not benefit from multiple threads because there is only one ML core inside the edge TPU.

Thanks for your respond. I'll try your links.


Proof suggestions newness bool

might be used to put a "New" badge on "Add other identities"

the button is hardcoded "new" for now


Quantify the impact of iOS 13 on the Flutter package size

Apple's iOS 13 marketing references app size decreases
https://www.slashgear.com/ios-13-is-obsessed-with-iphone-speed-and-app-size-03578962/
It could hypothetically be related to the encrypting/compressing ordering.
Test and quantify its impact with Flutter package download size.

@goderbauer : while our current device lab hello_world size benchmark on iOS only executes some command like the one in https://flutter.dev/docs/perf/app-size#ios, I wonder if it's possible for us to setup a CI test that will go through the whole Apple signing/encrypting/compressing process, and get a final IPA size for either hello_world or flutter_gallery size?

I don't know if that's possible, maybe @digiter does?
For what it's worth, I was unable to see any size decrease with iOS 13 in Flutter apps.

If the whole process can be done in pure command line, then that's possible with the devicelab.

@goderbauer did you mean reported size from the App Store? i.e. our hypothesis about iOS 13's marketed app size decrease / reshuffled compress/encrypt order did not take place and didn't affect Flutter apps?

The size for Flutter apps reported on App Store connect didn't change with iOS 13 when I was looking at it a couple of weeks ago.
Since we have no way of quantifying this, I am going to close this.


Wha? Legend malfunction.

https://www.dropbox.com/s/a7nke19hdcud6bh/Screenshot 2015-09-24 16.02.38.png?dl=0

Is there a layer defined on the map here? Bug either way, but just want to be able to recreate quickly when I go in to fix it.

I believe this is now fixed along with #26. Both caused by the same change I made some time ago as part of performance improvements in the map view. This particular change was actually not central to the performance improvement (still works with Voltaire data, for example), so I've changed it back.


Not gracefully handling error codes with no message

During failed HRESULT code 0x8e5e04c0, throws additional exception.
c:\dev\src\sandbox\sandbox\sandbox.cpp(16): HRESULT result failure (0x8e5e04c0)
Exception thrown at 0x00007FFB06763FB8 in Sandbox.exe: Microsoft C++ exception: win32cpp::check_failed at memory location 0x00000031A19BF0C8.
C:\dev\src\win32cpp\src\lib\error.cpp(13): count result failure (317)
Exception thrown at 0x00007FFB06763FB8 in Sandbox.exe: Microsoft C++ exception: win32cpp::check_failed at memory location 0x00000031A19BC930.
unknown file: error: Unknown C++ exception thrown in the test fixture's constructor.
'Sandbox.exe' (Win32): Loaded 'C:\Windows\System32\kernel.appcore.dll'. Cannot find or open the PDB file.

Fixed in https://github.com/lordjeb/win32cpp/pull/12


Vundle appearing multiple times

I'm seeing vundle showing up a lot on the home page currently.

Yes, long time bug mentioned a few times already but with unfortunately no adequate measure to fix it :
the project is currently dead.

We're looking for a maintainer. Please email<EMAIL_ADDRESS>if you're interested.

If anyone wants to look into this, I wrote a report for a school project on this a year+ ago:
https://www.dropbox.com/s/24owxihtek7vau9/report.pdf?dl=0

@divad12 It is the great article!  I will look into it.

Thank you @Shougo! :)

This project is important for Vim/neovim.
I want to support it.
Unfotunately, I have not much experience/time for it.

I think the best is to create the organization for VimAwesome.
And look for the new maintainers from Vim/neovim users.

@divad12 Could you announce it in neovim/vim_dev MLs or Vim reddit page?

Absolutely, a GitHub organization is the way to go.

Happy to migrate to an org once we find a new maintainer.

Hi @divad12 I am trying to run the site locally, but I fail at step 6 and 7
(vim-awesome)vagrant@vagrant-box:~/vim-awesome$ make init_db
PYTHONPATH=. python db/init_db.py
Traceback (most recent call last):
  File "db/init_db.py", line 17, in <module>
    ensure_tables_and_indices()
  File "db/init_db.py", line 10, in ensure_tables_and_indices
    db.github_repos.PluginGithubRepos.ensure_table()
  File "/home/vagrant/vim-awesome/db/github_repos.py", line 68, in ensure_table
    cls.ensure_blacklisted_repos()
  File "/home/vagrant/vim-awesome/db/github_repos.py", line 80, in ensure_blacklisted_repos
    'is_blacklisted': True,
  File "/home/vagrant/vim-awesome/db/github_repos.py", line 105, in upsert_with_owner_repo
    db_repo = cls.get_with_owner_repo(repo['owner'], repo['repo_name'])
  File "/home/vagrant/vim-awesome/db/github_repos.py", line 91, in get_with_owner_repo
    return db.util.get_first(query)
  File "/home/vagrant/vim-awesome/db/util.py", line 13, in get_first
    results = list(query.limit(1).run(r_conn()))
  File "/home/vagrant/vim-awesome/vim-awesome/local/lib/python2.7/site-packages/rethinkdb/ast.py", line 75, in run
    return c._start(self, **global_optargs)
  File "/home/vagrant/vim-awesome/vim-awesome/local/lib/python2.7/site-packages/rethinkdb/net.py", line 212, in _start
    return self._send_query(query, global_optargs)
  File "/home/vagrant/vim-awesome/vim-awesome/local/lib/python2.7/site-packages/rethinkdb/net.py", line 313, in _send_query
    self._check_error_response(response, query.term)
  File "/home/vagrant/vim-awesome/vim-awesome/local/lib/python2.7/site-packages/rethinkdb/net.py", line 286, in _check_error_response
    raise RqlRuntimeError(message, term, frames)
rethinkdb.errors.RqlRuntimeError: Index `owner_repo` was not found on table `vim_awesome.plugin_github_repos`. in:
r.table('plugin_github_repos').get_all(['sstephenson', 'bats'], index='owner_repo').limit(1)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
make: *** [init_db] Error 1

(vim-awesome)vagrant@vagrant-box:~/vim-awesome$ rethinkdb restore -i vim_awesome rethinkdb_dump_2015-12-15.tar.gz
Unzipping archive file...
  Done (1 seconds)
Importing from directory...
importing with args: ['rethinkdb-import', '--connect', 'localhost:28015', '--directory', '/tmp/tmpnWOvFn', '--auth', '', '--clients', '8', '--import', 'vim_awesome']
Error: The following tables already exist, run with --force to import into the existing tables:
  vim_awesome.plugin_github_repos
  vim_awesome.plugins
  vim_awesome.tags
Error: rethinkdb-import failed

(vim-awesome)vagrant@vagrant-box:~/vim-awesome$ rethinkdb restore --force -i vim_awesome rethinkdb_dump_2015-12-15.tar.gz
Unzipping archive file...
  Done (1 seconds)
Importing from directory...
importing with args: ['rethinkdb-import', '--connect', 'localhost:28015', '--directory', '/tmp/tmp2BtAyl', '--auth', '', '--clients', '8', '--import', 'vim_awesome', '--force']
[                                        ]   0%
0 rows imported in 7 tables
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/plugin_github_repos.json
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/tags.json
ReQL error during 'create table': Connection is closed.
In file: /tmp/tmp2BtAyl/vim_awesome/github_repos.json
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/dotfiles_github_repos.json
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/plugins.json
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/plugin_repos.json
ReQL error during 'create table': Could not connect to localhost:28015. Error: [Errno 111] Connection refused
In file: /tmp/tmp2BtAyl/vim_awesome/submitted_plugins.json
Errors occurred during import
Error: rethinkdb-import failed

I haven't worked with rethinkdb before, so some pointers would be of immense help.

And besides that, do you already have any clue about the cause of the bug?

The server is currently running rethinkdb 1.16.1 – perhaps you have a newer version where something's changed? We should probably upgrade but I don't know what needs changing.

@spicyj ok cool, let me try that in the evening after work.

Happy to migrate to an org once we find a new maintainer.

:+1:
I will search a new maintainer.

I'm candidate for maintaining.

Closing as duplicate of #64.


HttpUrl#addPathSegment does not encode square brackets '[', ']

Was wondering if there are any plans on adding square brackets in list of characters that should be escaped in path segment. RFC3986 says these are allowed only in host part of the URL and I recently had web server returning 400 because "[" in path segment was not escaped.

This looks like we should be encoding https://stackoverflow.com/questions/40568/are-square-brackets-permitted-in-urls
Mentioned explicitly in https://datatracker.ietf.org/doc/html/rfc3986#section-3.2.2

Changing this is potentially breaking, so if we do it we should do it with 5.0.0.
Any fix should retain the unescaped characters if we receive them in addEncodedPath() or toHttpUrl().

@swankjesse should we do with 5.0?

Lets do it in 5.0.

Lets confirm that browsers do it, then do it ourselves in 5.0.


faq updates, resolves dawgpath-75

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.119%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
394

Relevant Lines:
554

💛  - Coveralls


No support for multi-valued ProductAttributeValue

There's this PR which attempts to resolve this issue: https://github.com/django-oscar/django-oscar/pull/879
I'm not sure it's the "best" way of fixing this, but I think we should have a way of making this work.
Right now, as a workaround, I'm just using a text field and entering in values comma-separated, which works OK, but obviously is somewhat brittle.
I might submit a PR here with a different solution.
Thanks,
John

I have same issue.
Currently there is support for multi-valued option in apps.dashboard.catalogue.forms... I started trying to implement it...
I can remove the unique_together from AbstractProductAttributeValue with an extended class (obviously the model then ought to have further validation, which looks like will be handled by #879)
But I can't add the MULTI_OPTION choice to TYPE_CHOICES on the AbstractProductAttribute model by extending it, because I would have to redefine the existing field.
As a minimal change I would like to see the MULTI_OPTION added to TYPE_CHOICES asap even if the proper validation etc is not in place, so we are not blocked from hacking up something for ourselves in the meantime.

Thanks for your input.
I think you're 100% right that we should at least add a MULTI_OPTION choice to TYPE_CHOICES because an application developer simply cannot modify it currently.
However, if it is possible to include a partially- or fully-featured implementation, we should do so.

I updated the changes as discussed in https://github.com/django-oscar/django-oscar/pull/1745#issuecomment-106035074
This seems like more reasonable behavior.
Also amended the commit so that I'm the author (was showing @codeinthehole as the author after I did the rebase) and squashed it into a single commit.
There are still no tests, but I would really appreciate some input before continuing.

Fixed in #2309.


Review and refactor slice architecture

Review the code structure and refactor it to slice architecture.
Acceptance criteria
1 - App running with updated code.

the missing images and sizing will be reviewed in #25


Discuss other sports

Kloppy started as a library for handling soccer event and tracking in a standardized way. There are many providers for just this data.
With the release of klopp-queue we introduced a algorithm agnostic way of searching for interesting event patterns. The expectation is this will grow to also operate on tracking data and combinations of event and tracking data.
This framework can be applied to event and tracking data for all kind of sports.
In this ticket I would like to collect information about other sports that can benefit from the framework, see what data is available and possible changed required to support the sport.

Hi, I maintain the betfairlightweight package so have a keen interest in live sports data, I have been following for a while and posted a few of your posts on our slack group (which you are welcome to join)
A recent advancement in tracking has been added to horse racing via TPD. I currently focus on the modelling during the live updates of this data but could see it lending itself to the inclusion in kloppy especially the searching functionality!
Happy to contribute and put you in contact with the devs at TPD as they have expressed an interest in being more involved in open source 😄


Community Building Ideas and Discussions

I think its about time we start thinking about building a more robust community around OSS University. I am really fond of this initiative, and I would like to see it become a very successful project. When it comes to learning online, I think one of the challenges that the MOOC revolution is facing is that online learners often times feel as if they are studying alone (despite the efforts on each platform to encourage discussions). Another issue is that many of the MOOCs being taken here are self-assessed. The problem with that is that students may not be receiving quality, frequent, and/or sufficient feedback. I think that building a community would begin to address these issues.
The last thing we want is for someone to start the program, get halfway through a single MOOC, think that the entire curriculum is too difficult or worse that they aren't smart enough, and quit.
I would suggest taking a look at how FreeCodeCamp went about building their community. I think that it has worked quite well so far. As I am typing this, 117,859  campers have joined FreeCodeCamp since they launched 362  days ago. How many actually complete the program, I do not know. But, I can say that I've stuck with the program in part because of how great I think the community is.
Right now we have nearly 1000 students signed up, and it would be a waste to let any of them fall by the wayside.
I notice that we currently have Gitter chat, a facebook page, and a subreddit. That's great. We could create additional gitter rooms for each section of the curriculum, perhaps begin working on a web site (I don't know if the ossu-ui project is going to be our website and mobile application), and a blog is an awesome way that students can share the materials they are finding helpful. We could also encourage local meetups as the number of students increase. These are just ideas to spark a discussion and keep in mind.

I agree with you @oharrison and I think the dev team is working on the technical side and they are inspired by project like FreeCodeCamp. Ossu-ui will be both. The blog to share document and resources is also planned :)
On the side non-technical, I hope there will be local meetups and support groups in the same time the community grow. Thanks for the spark :+1:

One (hypothetical) idea I'd like to throw around is being able to collaborate/share flashcards (in the form of JSON files) through GitHub (or some place to share files). Does OSS have something similar planned?
I'm currently developing and dog feeding a flashcard app (https://github.com/Dashed/grokdb) as a motivation for learning the go language. At the moment, I'm using it for computer science courses I'm taking at University of Toronto.

@oharrison I agree with you and I have one idea to add! In our website or blog (when we have one of them) we could add a section called "Community Spotlight" where can be posted from time to time (maybe weekly) a testimonial from a student or group of student (written by themselves, of course), projects that was developed and became trending, interviews, and so on. The amount of content we can place in this are big and I think that you got the idea. We can even add a button for "Submit your own story!".

@Dashed thank you for your suggestion. I will take a look at your flash card project. I might event send a pull request.
@bestknighter I like this idea. It good to see the stories of people who have struggled and "made it."

@oharrison thank you for your kind words.
I agree with all the suggestions that you guys said.
In fact, our reddit subchannel is a quite powerful way to show what we are doing, but we need adherence by the community showing their projects and discussing their progress in such channel.
If some of you have an interest in becoming an OSS community manager, just let us know.
We'd love to have more help in such position because keep people motivated is one of the core features that we need to offer and always improve, as well as you pointed, @oharrison .

Looking at the FreeCodeCamp model, I think there's one important thing to take away. When they first started, most of their tutorials were sourced from other sites. What really added value to FreeCodeCamp apart from the community and the tutorials was the list of solid project ideas that users have to complete.
While a lot of people here at OSS highly encourage project-based learning, it all seems to be along the lines of "Be creative, do whatever you want". There is definitely value in that. But a lot of people might not be able to find an idea that they're willing to pursue and stick with. Learners tend to have trouble finding ideas in the first place, or they might think of ideas that are too complicated for the stage of learning they're at. Which is why it really helps to be given a couple of concrete ideas to implement from scratch using skills you've learnt. From my personal experience with FCC, it really helped me with learning web dev.
Since everything here is nicely divided up into sections, I think we should add one project that students should do, per section.

@prashcr We have a link to some projects suggestions on the readme to the students. But reading your comment I noticed that we can add at the end of each category a list (or a link to a list) of those projects that the student can already accomplish. But I think that we should reinforce in those lists that the student must develop their creativity and start a project from the scratch and with his own project idea. Creativity is one of the most important skills nowadays and we should do our best to help everybody develop the creativity.

@bestknighter I don't think we need to reinforce the need to develop creativity, Creativity tends to flow naturally once students gain confidence in their abilities and that's what we should focus on IMO.
Ah, I didn't really notice those suggestions...:sweat_smile:
Which kind of leads back to the main point, that we need to make them more prominent.
The project suggestions are nice for further learning (after completing) and should remain there for that purpose. But for the actual learning process itself, we should narrow down a couple of compulsory projects. Otherwise students will just look at the list and get overwhelmed by the choices. This is one of the main philosophies at FreeCodeCamp, and it's been serving them quite well. Reduce choices where possible and keep the curriculum streamlined, with only the most important materials.
Learning to program can be a frustrating rabbit hole, and we shouldn't be making things harder for students.

@prashcr At least in my country, more than 80% of the population has a mindset that thinks that creativity is an innate skill.  Which is completely false, as you know. So, even when they get the abilities, the creativity doesn't flow because they don't let the creativity "born" in their minds. If the students can think of a good project idea by themselves and make it happen, this boosts their confidence more than give a project already ready and proved that works to them just implement. If they do just a simple and plain notepad program (even if they know the existence of Notepad++), if they come with the idea by themselves, it will have a batter impact if we give them a simpler version of Notepad++ for them to implement. We want to build creators, builders, inventors, not plain programmers. I'm saying this by my own life experience. People who know how to program gets jobs. People who can create, build, with the computation gets the best jobs. So, IMO, help students to be creative is something that we should do.
But I'm not saying that we should keep everything open and free and the students do whatever they want, when they want, how they want. Overwhelming in any area is a dangerous poison to learning. We must indeed address this issue but I think that we should do in another way. My suggestion, based in later messages here and this one, is the following:
At the end of each section, we should suggest just a few projects for the student do at the end of that section that is hard enough to exercise the knowledge learned but not too hard to avoid overwhelming. But always remembering them that being creative is important and thinking on a project idea (preferably as hard as the suggested ones) by themselves is desired, encouraged and it can be inspired by the suggested ones.

@bestknighter Hmm, in response to your Notepad example. I don't think it's fair to say that telling someone to make a Notepad clone doesn't help with creativity. Given the task of "Make a Notepad app", there are so many architectural, design, feature decisions that a student has to make, which involve heavy creative thinking. Just because we provide one or more user stories doesn't rob the task of creativity. In addition, students can still go out of their way and make the notepad do really cool things that weren't part of the spec.
Your suggestion sounds good. In addition, students could be required to do one of projects required and the 2nd one could be a choice between their own idea or another suggested project

@prashcr we've covered the point that you're talking about, "give students some project ideas". There are two repositories with this finality.
Now the only thing that we need to do is to create more project ideas and send for those repositories.
Students need to start to think by themselves, and for me is much better to invest time struggling in how to apply the knowledge acquired in a practical project than waste the same amount of time memorizing topics in order to take exams.
Yes, this is a totally disruptive approach, but I'm sure that this will lead us to produce better professionals, with balanced pragmatic and theoretical skills.

Any chance for the forum to be on Discourse rather than Google Groups? Example: https://discuss.reactjs.org/

@Dashed We already discussed it in a previous issue. We've chosen Reddit, instead. In that way, we don't need to spend money with servers.


korean translation typo revised & some improvement for polite speech

I noticed a typo in Korean translation, so I simply fixed it (line 1427)
Also, although most of the Korean translations are written in polite speech, few of them were written in informal speech, which were quite awkward. For consistency, I fixed them to polite speech.

Oh I wasn't aware of the polite vs informal speech, interesting to learn :D Thanks for the contribution! 🎉


TS+ESLint Slow in latest release, suspect scm history graph

From this thread: https://x.com/naoehtu/status/1820840108624425446
"Not sure what has happened to VSCode + TS + ESLint, but it is become incredibly slow."

The "Incoming/Outgoing" section has been removed from the main "Source Control" view.


Missing Ai tools on new installation

My actions before raising this issue

Read/searched the docs
Searched past issues

I just installed cvat based on Installation Guide and there is no ai tools button just like the webpage. My server version is 1.5. Should I do any extra steps?
Thanks!

what is the  ai tools button?

this button exist in online version but when I installed it on my computer I can not find it
https://imgur.com/a/V5XS4vZ

https://openvinotoolkit.github.io/cvat/docs/administration/advanced/installation_automatic_annotation/
I will close the issue. Don't hesitate to ask on our public gitter if you have more questions:  https://gitter.im/opencv-cvat/public

Thanks for the answer, @nmanovic .
Is it possible to run the Automatic annotation tools in the same machine where CVAT is running, this is, without the use of nuclio.
Thanks in advance!


Cypress web vitals will not work if credentials or special headers needed to visit the site.

Issue
Cypress web vitals will not work if credentials or special headers needed to visit the site.
cy.visit({
      url: 'https://www.someQaEnvironment.com/',
      headers: {
        "Accept-Encoding": "gzip, deflate"
      },
      auth: {
        username: 'qa',
        password: 'examplepass',
      }
 });

If these are not provided, ESocket Timeout error
Took a stab at a fix that worked for me locally so pr coming soon! 🎉
currently in a draft pr: #18
Setup:

node version: v16.14.2
cypress-web-vitals version: 2.0.0
"cypress-real-events": "^1.7.1",

Details

visitWithWebVitalsSnippet, runs cy.visit after intercept but loses auth and header context.
cy.intercept is not grabbing auth param.
cy.intercept gives different headers than are required by cy.visit

// cy.visit needs:
      headers: {
        "Accept-Encoding": "gzip, deflate"
      },

// cy.intercept gets:
      headers: {
        accept: "text/html,*/*"
        authorization: "Basic cWF1c2VyOlRlc3RlZDRxdWFsaXR5IQ=="
        connection: "keep-alive"
        cookie:  "some cookie"
        host: "www.example.com"
        user-agent: "some agent"
      }

^ because of ( 3) ) , cy.vitals is easy to fix
but cy.visit is not an easy fix so my draft pr #18  only fixes cy.vitals

happy to answer any questions, thanks!

Most cool, look forward to reviewing!


[PROPOSAL] allow passing signals between containers (--signals-to)

I have a logrotate_cont container, and an nginx_cont container; nginx running in the latter needs to be signalled by logrotate to re-open its logfiles. Currently, there is no sane way of doing this, save from giving the logrotate_cont container docker API acccess (which is not a good idea).
So here's an idea: just as files and ports can be linked/shared between containers, it would be useful to be able to make container A able to send signals to container B.
I propose a signals-to command-line parameter.
Why not --signals-from? Because we usually know where we want to send the signal, not necessarily who/what is going to send it (i.e. nginx running in my nginx_cont container has a set PID; logrotate's PID changes with each run).
The parameter would take one argument:
<container_id>(:<pid_or_process_name>(:<comma_separated_signals>))
So in my use-case I could use --signals-to nginx_cont, or if I want to be much more specific (and safe): --signals-to nginx_cont:nginx:USR1.
Or, if I wanted container A to send specific signals (say, USR1 and USR2 only) to any PID in container B: --signals-to cont_b::USR1,USR1
Question is, of course, if this is actually doable?

In this case I would say you should just use --pid=host so you can see the nginx in the process tree.
Closing since this is not really something we can do w/o sharing a pid namespace, which has been rejected in the past, and is completely possible by using the host's pid namespace.

For reference, it hasn't been rejected as a whole, see #10163, and this pull request.
Also, quoting the docker documentation: "--pid=host Set the PID mode for the container host: use the host's PID namespace inside the container. Note: the host mode gives the container full access to local PID and is therefore considered insecure."
But yes, --pid=<container> makes more sense (even if it gives a bit less fine-grained control, I guess).


Message in logs: Installer "Extract .zip/.tar.gz" cannot be used to install

I raised this in a slack thread but since it's not obvious that we have a solution yet I'm copying it here. There is a "JDK installation" role defined in the jenkins Global Tool Config which is intended to install JDK11 on the Azure nodes used for the Linux installer jobs, but it's causing warnings to be thrown in lots of other jobs:
Installer "Extract *.zip/*.tar.gz" cannot be used to install "jdk-11.0.13+8" on the node "<blah>"

While this was initially seen on the VPC jobs, the role definition in jenkins seems to be in place for various linux systems it's also affecting windows jobs e.g. create_installer_windows which is showing the same message (usually repeated six times in succession, repeated multiple times during the logs)

Also noted by @karianna in the sign_build job today.

@gdams Can we get some attention on this to clear it up please? It's not good to have these messages across the logs and we should understand why this is happening and endeavour to clear it up and you know how it was introduced. I believe that it originated from work you did for the installer at the end of 2021 and that you'd done some work to clear it up in some places but it still remains an issue in various jobs and you are best place to understand and resolve it.
In addition to the messages seen elsewhere I was working on a prototype jenkins job this week and the ppc64le system being used is failing with this error which seems likely be related given the version number that is specified in the message:

Building remotely on dockerhost-skytap-ubuntu2004-ppc64le-1 (build dockerHost dockerBuild linux docker ppc64le)FATAL: Failed to install https://github.com/adoptium/temurin11-binaries/releases/download/jdk-11.0.13%2B8/OpenJDK11U-jdk_ppc64le_linux_hotspot_11.0.13_8.tar.gz to /home/jenkins/tools/hudson.model.JDK/jdk-11.0.13_8

@gdams Can you clarify the implications if this is removed from the jenkins tools config please? Would we need to have JDK11 installed elsewhere on some of the machines? (And does it need to be 11?) Since this was for only one specific purpose it seems suboptimal to have it as something that is triggered for many jobs.
@steelhead31 Will the installer rewrite still require this install?

I dont believe it is used in the linux package install, unless there's some dependency Im not aware of.


Unable to install plugin after commit 8f18e823bda21c27cd7eb1eafd15c27ab3f50ad6

Hi,
The commit 8f18e823bda21c27cd7eb1eafd15c27ab3f50ad6:
deleted hardcoded support-v13 dependency
After this commit I am unable to install the plugin using plugman. It worked fine yesterday morning but after the commit I'm getting errors in regards to support-v13 library.
I already did a test case just to check this. I created a sample project with cordova and tried to add the plugin:
cordova create helloworld
cd helloworld
cordova platform add android
plugman install --platform android --project platforms/android/ --plugin https://github.com/phonegap-build/PushPlugin.git

The following error appear:
Fetching plugin "https://github.com/phonegap-build/PushPlugin.git" via git clone
Installing "com.phonegap.plugins.PushPlugin" for android
Error during processing of action! Attempting to revert...
Failed to install 'com.phonegap.plugins.PushPlugin':Error: Uh oh!
ENOENT, no such file or directory     '/Users/vfranchi/Projects/phonegap/android/helloworkd/platforms/android/local.properties'

Even after I loaded up Android Studio so it would create the local.properties file, I end up with an error about the support library.
Regards,
vfranchi

My bad, forgot to check my cordova and plugman version.
After npm update all worked fine. Sorry about that.


Add possibility to use YAML files for parameters

Everything's in the title. Also would be awesome to add compatibility with other data formats, e.g. JSON.

Closed in #12


No smooth animation on left sidebar menu arrows

When I expand an item on the left sidebar menu, I see the 90° rotation but it isn't smooth as it should be, regardless the demo https://adminlte.io/themes/AdminLTE/pages/examples/blank.html. I pass directly from > to v.
I just install last version of AdminLTE, I'm on version 2.3.11.
Am I missing something?

it sound like your browser doesn't support browser transition's

but if the live demo is working, mine should work on the same browser, no?

Sure it should work, then it could be that you missed some css classes in the html.
It's need the pull-right-container-class for the transition.
Check your code with this 😄 from the demo

             Examples

            ****

thanks again for your reply, but no, still nothing... I'm gonna start with a new fresh install to see what is going on.

Transition css (lines 547-549 in AdminLTE.css) was added to 2.4

I have the same problem... Someone can help me? My treeview:

        Notificações

I'm using the version 2.3.11.

The rotation transition of the arrow was added to version 2.4 which is the version in the demo. V2.4 is currently in Beta, feel free to to upgrade by following this guide:

I cl

Closed as the solution is to update.
Thank you everyone ;)


feat: support import expressions (#16)

Looks good to me! Thank you for your contribution. Closes #16

Published in v1.2.0.


Memory Leaks

To help mitigate memory leaks, listen to a store's "removed"` event (for this key) and turn off any storeit-value listeners. Currently, you are listening to a store's "modified" event for all of eternity, long after it may have been deleted.

You should also stop setting to the store when the value has been removed and maybe publishing a "removed" event yourself there chief.

Resolved by #10


Better messaging / instructions about Creative Mint

Seems that people are often confused about what is Creative Mint capable of, we need better instructions and error messages, i.e. -> wrong filetype etc.

ignore


Make use of window functions in SIZE to COUNT transformation

A possible optimization is to use a window function like the following instead of a correlated subquery count(distinct element.id) over (partition by parent.id, parent.parent.id...) but this only works/makes sense when the collection is fetched. This case is currently one that prevents using the count transformation and unnecessarily queries the table another time in the correlated subquery.
It should be possible to turn off this optimization and it should also only be activated when the underlying DBMS supports it obviously.
Since I doubt this is a common use case, this has very low priority.

Closing as the optimization is probably not worth it because such a usage is very rare.


Add "OAUTH2TokenUrl" and "OAuth2AuthzUrl" property key changes to IS 5.0.0 - 5.1.0 migration

In IS 500, token endpoint of the OpenIDConnect authenticator configuration is being persisted against 'OAUTH2TokenUrl' property key and authorization endpoint is being persisted against 'OAuth2AuthzUrl'. But in IS 510, these property keys are being changed to 'OAuth2TokenEPUrl' and 'OAuth2AuthzEPUrl' respectively. Migration with respect to these changed property key values has been missed in IS 500 to IS 510 migration. Thus, the UI will not populate the token endpoint and authorization endpoint values, as the values in the database are not persisted against the expected property key values.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


feat(themes): Updated Links

/merge

https://github.com/SallaApp/oauth2-merchant/pull/25 is been merged ✅

Merge by: @nabiloo19


Note about actually creating the folders :sweat_smile:

Embarrassingly, I tried starting it without creating the folder first.

Cheers


Add information how comments from PSL/AST are added to Prisma Client

https://www.prisma.io/docs/concepts/components/prisma-schema#comments
This section mentions how some comment make it to the AST, but there is no example how this is used. Instead there is a note that they are not used:

Note: /// comments are not present of the generated Prisma Client, but are present in the internal Data Model Meta Format (DMMF).

Per https://github.com/prisma/prisma/pull/10248 these are actually added to Prisma Client (please confirm @millsp), so we could use that as an example here (and this way also document that feature of Prisma Client generation)

Yes, comments are now visible on the property/column level and on the type level, but are not being reflected in prisma.<model>.

comments are now visible on the property/column level and on the type level, but are not being reflected in prisma..

Sorry, what exactly does that mean? Can you add examples?

I left a similar explanation here https://github.com/prisma/prisma/issues/5070#issuecomment-1066601301

I hope the example we produce for the docs will help me understand.

@millsp I think this section is sufficient on explaining how comments are displayed. WDYT? If so, we can close this issue.


Implement felt_div libfunc

Implement felt_div libfunc based on pre-refactor implementation

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

Comparison is base (1b4d1d5) 77.86% compared to head (0d876e4) 77.99%.

:exclamation: Current head 0d876e4 differs from pull request most recent head e5e74d7. Consider uploading reports for the commit e5e74d7 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #389      +/-   ##
==========================================
+ Coverage   77.86%   77.99%   +0.12%
==========================================
  Files         103      103
  Lines       26060    26209     +149
==========================================
+ Hits        20292    20442     +150
+ Misses       5768     5767       -1

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


Vulnerability count problems

I receive the following message during checking my code and I am wondering about the vulnerability count displayed in the summary.
[ERROR]
[ERROR] --------------------------------------------------------------
[ERROR] axis:axis:1.4  [VULNERABLE]
[ERROR] 3 known vulnerabilities, 2 affecting installed version
[ERROR]
[ERROR] [CVE-2012-5784]  Improper Input Validation
[ERROR] https://ossindex.net/resource/cve/353500
[ERROR] Apache Axis 1.4 and earlier, as used in PayPal Payments Pro, PayPal Mass                                                                                Pay, PayPal Transactional Information SOAP, the Java Message Service implementa                                                                                tion in Apache ActiveMQ, and other products, does not verify that the server hos                                                                                tname matches a domain name in the subject's Common Name (CN) or subjectAltName                                                                                field of the X.509 certificate, which allows man-in-the-middle attackers to spoo                                                                                f SSL servers via an arbitrary valid certificate.
[ERROR]
[ERROR] [CVE-2014-3596] The getCN function in Apache Axis 1.4 and earlier does n                                                                                ot properly verify that ...
[ERROR] https://ossindex.net/resource/cve/362463
[ERROR] The getCN function in Apache Axis 1.4 and earlier does not properly veri                                                                                fy that the server hostname matches a domain name in the subject's Common Name (                                                                                CN) or subjectAltName field of the X.509 certificate, which allows man-in-the-mi                                                                                ddle attackers to spoof SSL servers via a certificate with a subject that specif                                                                                ies a common name in a field that is not the CN field.  NOTE: this issue exists                                                                                because of an incomplete fix for CVE-2012-5784.
[ERROR]
[ERROR] --------------------------------------------------------------
[ERROR]
...
And in the end:
...
[ERROR] Failed to execute goal net.ossindex:ossindex-maven-plugin:2.1.3:audit (d                                                                                efault-cli) on project klaros-mylyn: 6 known vulnerabilities affecting project d                                                                                ependencies -> [Help 1]
Why is this resolved as 6 vulnerabilities? I would have expected a count of 2 here.

Please close and ignore. I missed 2 additional entries.


[RFC]: refactor blas/ext/base/ssort2ins to follow current project conventions

Description
This RFC proposes refactoring @stdlib/blas/ext/base/ssort2ins to follow current project conventions.
These conventions are outlined in more detail in https://github.com/stdlib-js/stdlib/issues/788. Namely, we want to migrate from C++ add-on interfaces to C add-on interfaces and then make various style and simplification changes.
Some example commits which perform most, if not all, of the requested changes include https://github.com/stdlib-js/stdlib/commit/d66d80cfcbab1da95cb5ae8cfae4a1fc4c5129d1 (saxpy),  https://github.com/stdlib-js/stdlib/commit/3c82fe7b6c0f9fd6ef02c86d6e2024d708ac8c14 (cswap), and https://github.com/stdlib-js/stdlib/commit/5b15c9675be434c0e11b1ec1e1f78d7b089abbcd (dasum).
Related Issues
Related issues #788, #1152.
Questions
No.
Other
No.
Checklist

[X] I have read and understood the Code of Conduct.
[X] Searched for existing issues and pull requests.
[X] The issue name begins with RFC:.

@Planeshifter I am done with the PR on blas/ext/base/ssumkbn2 and its under review, can I work on this issue?


Add GUI dataset description generator to software list

This PR introduces a new tool to be added under "Other Tools" section, a GUI form that generates dataset description json file

Thanks @tolik-g!


Add default for hm9000.port to make it optional

Thanks for contributing to the cloud_controller_ng. To speed up the process of reviewing your pull request please provide us with:

A short explanation of the proposed change:
Add defaults for hm9000.port

An explanation of the use cases your change solves
Makes hm9000 port optional when deploying CF without any hm9000 vms.

[x] I have viewed signed and have submitted the Contributor License Agreement

[x] I have made this pull request to the master branch

[x] I have run all the unit tests using bundle exec rake

[x] I have run CF Acceptance Tests on bosh lite

[#124374541]
Signed-off-by: Sandy Cash<EMAIL_ADDRESS>

Pulled it down, deployed, and ran tests.  Merged!


A thread deadlock occurred  when writing to SecAuditLog

A thread deadlock occurred  when writing to SecAuditLog
`220 bool SharedFiles::write(const std::string& fileName,
221     const std::string &msg, std::string *error) {
222     std::pair<msc_file_handler *, FILE *> a;
223     std::string lmsg = msg;
224     size_t wrote;
225     bool ret = true;
226
227     a = find_handler(fileName);
228     if (a.first == NULL) {
229         error->assign("file is not open: " + fileName);
230         return false;
231     }
232
233     pthread_mutex_lock(&a.first->lock);                                                                                234     wrote = fwrite(reinterpret_cast<const char *>(lmsg.c_str()), 1,
235         lmsg.size(), a.second);
236     if (wrote < msg.size()) {
237         error->assign("failed to write: " + fileName);
238         ret = false;
239     }
240     fflush(a.second);
241     pthread_mutex_unlock(&a.first->lock);
242
243     return ret;
244 }`
A deadlock occurred at line 233.

@zimmerle

Hi @xjjy1083,
Thanks for your report. Can you please provide more specific details on the ModSecurity version that this is happening? Does this behaviour also happens on the latest code from master?
Based on the output of the function names of your debugging session I believe you are talking about v3, but it help us a lot if you can tell us in detail and preferably follow the guidelines of the issue report template: https://github.com/SpiderLabs/ModSecurity/issues/new?template=bug-report-for-version-3-x.md
Thanks.

Closing this issue as it is missing feedback. There is indeed a critical section on the code, but the circumstances that lead this hypothetical deadlock are unknow.


Remove contract-repository contract

Change-type: major
Signed-off-by: Josh Bowling<EMAIL_ADDRESS>

@balena-ci I self certify!


Investigating improvements

nvm, will update in the next days..

Was just benching it on my machine 🎉 super nice.
When reading up on LRU caches, I came across ARC, which I was a reminded of a little with your two cache approach. I wonder if you looked at that in your thinking: http://dbs.uni-leipzig.de/file/ARC.pdf

no i didn't see that, i just had the idea, but it's pretty simple so
wouldn't be suprised if someone else tried something similar
On Sat, Dec 24, 2016 at 1:49 AM, Friedel Ziegelmayer <
<EMAIL_ADDRESS>wrote:

Was just benching it on my machine 🎉 super nice.
When reading up on LRU caches, I came across ARC, which I was a reminded
of a little with your two cache approach. I wonder if you looked at that in
your thinking: http://dbs.uni-leipzig.de/file/ARC.pdf
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/dominictarr/bench-lru/issues/1#issuecomment-269063449,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAP1LgE47h-r6dcEq8Za3ZkrsA4Mbcj9ks5rLHougaJpZM4LVGIX
.


Correct wrong utf8 content charset declaration

Thanks for the PR @lchanouha!
The tests seem to currently conflict with the existing code, would you have the chance of updating those as well?


refactor: pass the avatars urls as props for component reusability

Description
This PR refactors the AvatarCircles component to enhance its flexibility and maintainability by allowing dynamic avatar URLs to be passed as props.
This change makes the component more reusable and simplifies the code.
Changes Made
Added avatarUrls Prop

[x] Introduced a new prop avatarUrls which is an array of strings representing the URLs of the avatars.
[x] Updated the AvatarCirclesProps interface to include the avatarUrls prop.

Dynamic Rendering of Avatars

[x] Replaced the hardcoded avatar URLs with a .map() function to iterate over the avatarUrls array and render the images dynamically.
[x] Ensured each image has a unique key and appropriate alt text for accessibility.

Code Simplification

[x] Reduced repetition by eliminating hardcoded image elements.
-[x] Simplified the JSX structure, making it easier to read and maintain.

Usage Example
Updated the usage of the AvatarCircles component to pass the avatar URLs as props:

import AvatarCircles from "@/components/AvatarCircles";

const avatarUrls = [
  "https://avatars.githubusercontent.com/u/16860528",
  "https://avatars.githubusercontent.com/u/29210732",
  "https://avatars.githubusercontent.com/u/20110627",
];

function App() {
  return (
    <AvatarCircles
      numPeople={5}
      className="custom-class"
      avatarUrls={avatarUrls}
    />
  );
}

export default App;

Screenshots

Code

"use client";

import { cn } from "@/lib/utils";
import React from "react";

interface AvatarCirclesProps {
  className?: string;
  numPeople?: number;
  avatarUrls: string[];
}

const AvatarCircles = ({ numPeople, className, avatarUrls }: AvatarCirclesProps) => {
  return (
    <div className={cn("z-10 flex -space-x-4 rtl:space-x-reverse", className)}>
      {avatarUrls.map((url, index) => (
        <img
          key={index}
          className="h-10 w-10 rounded-full border-2 border-white dark:border-gray-800"
          src={url}
          width={40}
          height={40}
          alt={`Avatar ${index + 1}`}
        />
      ))}
      <a
        className="flex h-10 w-10 items-center justify-center rounded-full border-2 border-white bg-black text-center text-xs font-medium text-white hover:bg-gray-600 dark:border-gray-800 dark:bg-white dark:text-black"
        href=""
      >
        +{numPeople}
      </a>
    </div>
  );
};

export default AvatarCircles;

Addional Suggestions
I would suggest adding configuration files for NextJS that uses Image component instead of img

fixes #144