Ditch CSS completely (use JS only) for "wall of logos"

Yesterday's hack didn't quite cut it, even inline styles (like max-width 225px) get lost in translation. So doubling down with a purely JS-based approach that injects the styles during onload(), onmouseover(), and onmouseout(). To recover my lost bet, $20 says it works this time.

I still think you should just resize the image files themselves - it'll make the page load faster as well.
Let's publish this and see what happens.

As discussed in email - let's try this first. I can resize as last resort.
The images, in their current "retina" (2x the needed resolution) form, are still just 400 KB all together.

Merging for publish this afternoon. @cosmosdarwin - if you could check the size around 3:50 today, that'd be great. Thanks!


Changes for policy server and agents

This branch was used to install and test development cluster today. Proved working. Suggest we merge it.

Ok


Clarify pipelines command-line instructions

https://kubeflow.org/docs/pipelines/build-pipeline/#install-the-kubeflow-pipelines-sdk
What does this even mean?
"Instead of using a notebook, you can choose to set up things yourself using the command line. But currently you can’t use the Python SDK to submit pipelines to a cluster, and you can’t build container images using the SDK. The DSL compiler works as usual."

/close


Problems with migrations

I get this when running rails neo4j:migrate locally:
Neo4j::Driver::Exceptions::DatabaseException: Could not initialize class org.eclipse.collections.impl.utility.LazyIterate

The neo4j-server is installed with:
rails neo4j:install[community-latest]

My Gemfile is:
gem 'activegraph', '~> 10.0.1'
gem 'neo4j-ruby-driver', '~> 1.7.0'

@johanandre sorry for the late response. That looks like a problem with neo4j server, not neo4j driver or activegraph


Mock API responses

What are you trying to achieve?
I'm using CodeceptJS to test a single page application that relies on an API for most of its features. In order to improve the isolation between the app and the API, I would like to intercept all the API calls performed by the app during the tests and return fake API responses.
The app I'm testing with CodeceptJS is being served on localhost, and Selenium uses Google Chrome to perform the tests.
I could think about two different solutions so far to solve this problem:

add some extra code in the app to check if it is the testing environment, then use some kind of library like Sinon.js to mock all API calls;

point the app to a fake API, that is going to return fake responses. This fake API could be a very simple express app used only during the CodeceptJS tests;

Although both solutions sound reasonable, I would like to ask the community if there are best practices regarding this isolation of the app and the API for testing purposes.
Cheers!

I think for your case you should follow the 2nd path.
You can add the fake API application to bootstrap/teardown hooks so it would be started with tests.
If you run acceptance test, you test the app from the outside, this way you should not rely on anything internal in it, like environment detection or sinon mocks. It should work just as it would for your users. The changes you can do with sinon are not a part of your API so this can be the really fragile solution. So yes, I'd choose the fake API.

Thanks for the answer, @DavertMik. I totally agree when you say that I shouldn't change the behavior of the app when running the acceptance tests, but mock the external services instead. I'm closing this issue since we agree that there is a very clear path to move forward in this regard.
Cheers!


Fix several invalid escapes

Future versions of Python will make invalid escapes in strings like '\s' a syntax error if the string isn't a raw string. This makes several docstrings in mpmath with backslashes raw. I didn't check but potentially some of these even had valid escapes, meaning things like \t or \n which would just produce the corresponding whitespace character in the parsed string.
These cause issues for upstream SymPy, as we have our tests set to fail if there are invalid escapes. See https://github.com/sympy/sympy/pull/15094.
I am also setting up testing of 3.6 and 3.7 in Travis, and testing of invalid escapes there so they won't reappear.

Worth noting here that the compileall test is a little non-user friendly. It doesn't stop when there is an error, so if the command fails, you may have to scroll up to see it. Unfortunately, it's the best way I know to actually test this.

I guess 3.7 doesn't actually work in Travis yet (I had thought that might be the case).

Thanks!


Doc-Coordinate Conversion: "Customize a coordinate format" does not include instructions on how to format DD/DDM/DMS (ArcGIS Pro)

For DD, DDM, and DMS formats, the user can customize the output, however, we don't have any content in this .pdf topic to explain how users can do this.
For example, using # and 0's to format decimal places.

Should also include content on how to customize the output of the other coordinate systems also.

@elinz I added a sentence explaining the difference between 0 and #, and specified MGRS letter formatting. I believe that we could include more content on how to customize the output of other coordinate systems, but the deeper I look into this the more I am confused at why our coordinate conversion tool doesn't mimic exactly how output coordinates are displayed in ArcGIS Pro.
I propose that, for post January release, we decide on whether we are going to change how the coordinate conversion tool formats coordinates to match that of arcgis pro, or whether we are going to keep the tool as is and add more extensive documentation that covers how we display output coordinate formats.
@dfoll @nfeuerstein fyi.

@Dbarnes1 The original intention of the CC tool was to meet a need for users who use different apps that use different systems and notations.  The original case had a user who had a dozen different applications running at the same time (including Desktop or Pro) and needed to copy/paste coordinates to and from all of these systems.  The CC AddIn allows the user to quickly copy/paste from system A, get the correct notation they need for system B, C, D, E, etc and copy/paste to these systems with only mouse clicks.

per my conversation with @nfeuerstein, we are going to move any further explanation of output coordinate display to post january release.

@Dbarnes1 @nfeuerstein Sounds good.

@topowright could you review the published documentation and see if you feel this is sufficiently documented now, if so please remove the "affects documentation" tag, if not we need to address this

closing. @topowright @lfunkhouser @NatalieCampos reviewed what exists in the documentation in the "customize coordinate" section. we came to consensus that the "affects documentation" aspect of this is taken care of. because the issue was just documentation based, closing because the documentation has been updated sufficiently


Can't search at the relations

I have model User and have relation with model Country by field country_id. How i can search and at the model User and Country

Hi @olegpy, have a look here. I hope it will help.


MAINT Removes GitPython dependency

This PR removes GitPython dependency and adds more unit tests to check the updated function.
Type

[ ] Bug Fix
[ ] Feature
[ ] Plugin
[x] Maintenance

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
GitPython was only used to read the remote url. This configuration is located in .git/config, which we can parse directly.
Tracking Issue
Towards flyteorg/flyte#4418

just to expand on my previous comment, here are the tests for url parsing in git itself: https://github.com/git/git/blob/77bd3ea9f54f1584147b594abc04c26ca516d987/t/t0110-urlmatch-normalization.sh. Parsing urls is itself a can of worms. Having basic support for ssh + https/http covers more than the 90% of the cases. We can always expand later if necessary.


Updated CocoaAsyncSocket version in pod spec

We have dependency to a more recent version of CocoaAsyncSocket (~>7.4.1) and that is conflicting with NHNetworktime as it prevents use of patches in CocoaAsyncSocket (<=7.4.1). This changes allows the use of patch releases for CocoaAsyncSocket

Thanks for your contribution


[main | latest] Issue with user search queries

every page after 3 i believe for me atleast has the last 9 or so user querys be the same every time. may be different on other devices, an example of this is on the bettermeower 1.5.0 branch. or the diference between https://api.meower.org/search/users?autoget&page=15&q= and https://api.meower.org/search/users?autoget&page=14&q=

Should be fixed in the CL4 port. De-prioritizing issue.

at this point this shouldn't be an issue, so unless you reproduce it on the current server, i'm going to close this


[Revamp] Continuous Integration

Links

Gutenberg material
Markdown files

Areas for improvement

[ ] fillmein

I'll do this.


Numba causes silent errors for mixed-component atmospheres

The new numba capability reproduces pyrads results identically when I only use HITRAN data from a single species.
However, numba appears to fail silently when using line data from multiple HITRAN species, which requires repeated calls to Absorption_Crosssections_HITRAN2016_numba. Test03.runaway_with_co2 contains an example of calculations which use both H2O and CO2 data. In this case the optical thicknesses computed using numba don't match the results without numba anymore.

Here is a minimal script that reproduces the error.
import numpy as np
import pyrads
from pyrads.Absorption_Crosssections_HITRAN2016 import getKappa_HITRAN
from pyrads.Absorption_Crosssections_HITRAN2016_numba import getKappa_HITRAN_numba
--
n0,n1,dn = 350.,400.,20.
n = np.arange(n0,n1,dn)
T = 300.
p = 1e5
pself = 1e3
--
kappa0 = getKappa_HITRAN(n,n0,n1,dn,"CO2", broadening="mixed",
press=p,press_self=pself,temp=T,
lineWid=25.,cutoff_option="fixed",remove_plinth=True)
kappa0_numba = getKappa_HITRAN_numba(n,n0,n1,dn,"CO2", broadening="mixed",
press=p,press_self=pself,temp=T,
lineWid=25.,cutoff_option="fixed",remove_plinth=True)
print( "kappaCO2 (no numba)=",kappa0 )
print( "kappaCO2 (numba)=",kappa0_numba )
print( "\n" )
--
kappa0 = getKappa_HITRAN(n,n0,n1,dn,"H2O", broadening="mixed",
press=p,press_self=pself,temp=T,
lineWid=25.,cutoff_option="fixed",remove_plinth=True)
kappa0_numba = getKappa_HITRAN_numba(n,n0,n1,dn,"H2O", broadening="mixed",
press=p,press_self=pself,temp=T,
lineWid=25.,cutoff_option="fixed",remove_plinth=True)
print( "kappaH2O (no numba)=",kappa0 )
print( "kappaH2O (numba)=",kappa0_numba )
print( "\n" )

Resulting output is:
(pyrads) daniels-MBP:PyRADS dkoll$ python test_hitran_numba.py
kappaCO2 (no numba)= [5.80495351e-05 3.91946490e-06 0.00000000e+00]
kappaCO2 (numba)= [5.80495351e-05 3.91946490e-06 0.00000000e+00]
kappaH2O (no numba)= [22725.4463143   7063.90460525     0.        ]
kappaH2O (numba)= [5.80495351e-05 3.91946490e-06 0.00000000e+00]

@AndrewWilliams3142: getKappa_HITRAN_numba seems to produce incorrect results when calling it for a second time with another gas species. Presumably an issue with numba's caching? Manually setting @jit(cache=False) doesn't fix things though.

Hi @ddbkoll ! I'll take a look now, it could well be something wrong with my numba interface (still getting used to it).
+1 for the tests!

@ddbkoll very weird! I thiiiink this might be to do with how numba deals with global instances, see here.

Global variables are treated as constants. The cache will remember the value in the global variable used at compilation. On cache load, the cached function will not rebind to the new value of the global variable.

One such global variable is molName! So I think that when numba compiles the function for the first time it's remembering this, which is why you always get the values for CO2 (or whatever else you ran first).
Absorption_Crosssections_HITRAN2016_numba.py clearly needs a bit more work! On this topic, I was going to ask, there are lots of functions in Absorption_Crosssections_HITRAN2016.py (and the numba version) which aren't used and just seem to be for plotting? If so, I think it might make sense to either remove these functions (to reduce clutter and make debugging easier) or put them in a separate file.

Interesting! Yes, the Absorption_Crosssections script is in urgent need of some rewriting and cleaning up. The code is essentially Ray Pierrehumbert's old PyTran script, and could probably be made much more efficient by leveraging modern Python capabilities like numpy. Many of the functions in there can, and should be, decluttered. If you have time to look at it, that'd be a great contribution!

Sweet! Yeah I recognized a few of the functions from PyTran actually. I'll start thinking about a clean-up this week! (Need to make some lists of what's being used)
Another question, do you use the stuff in ClimateGraphics/ClimateGraphics_MPL etc? As far as I can tell most of this is redundant now that matplotlib is so advanced anyways, so there might be an argument for removing it from the package.

ClimateGraphics should go. These are all pre-numpy/matplotlib legacy libraries.

In terms of rewriting Absorption_Crosssections: the important parts are loadSpectralLines() and computeAbsorption(). loadSpectralLines() reads in all the individual spectral lines from a hitran file, using a big for-loop. computeAbsorption() then loops over all the saved lines and adds up their contribution to the absorption crossection 'absGrid'.
Currently loadSpectralLines() is nice in that it requires minimal memory (reading one spectral line at a time), but the downside is that it appends that data to an ever-growing list. It might be possible to speed up loadSpectralLines() by first loading the entire hitran file into memory, then only keeping the subsection of lines we're actually interested in? One issue with this is that spectral files can become ridiculously large -- e.g., the full H2O line list from HITRAN2016 is only ~23MB, but other line lists can be >>100MB, so loading all lines isn't always an option.
computeAbsorption() then loops over the lines, calculates a lorentz line shape for each one, and adds the contribution of that line to the overall absorption grid. Since this only operates on line data that's already stored in memory, there might be a way of replacing the for-loop with numpy?
Alternatively, loadSpectralLines() and computeAbsorption() could be good targets for numba.

Interesting! Thanks for the overview!
Maybe I'm missing something here, but in DATA/HITRAN_DATA/HITRAN2016/ThermalOnly_0-5000cm.MainIsotopesOnly the files are all quite small? for example 01_hit16.par is only 5.3Mb? I'm a bit confused by these files though, I'm not entirely sure how to load the entire thing in haha

Another option could be to call loadSpectralLines() in the OpticalThickness calculation, before it loops over the temperatures/pressures. Currently it loads in the same spectral lines every iteration of the loop. I'm just putting together a PR now which should tidy up lots of the code


docs: provide a sample iam policy

Issue #, if available: N/A
Description of changes:
The AWS::Serverless::Function resource will give the proper IAM policies for the logger and tracing automatically but you need to provide an additional policy if you want to make use of the Metrics utility.
Checklist

[x] Meet tenets criteria
[ ] Update tests
[x] Update docs
[x] PR title follows conventional commit semantics

Breaking change checklist

RFC issue #:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #91 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           develop       #91   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           17        17
  Lines          474       474
  Branches        42        42
=========================================
  Hits           474       474

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 6db7263...65d340c. Read the comment docs.

Hey Joris!
Where/when would this policy be needed? The Powertools use EMF to create metrics, so as long as the function can emit logs, this should work.

@nmoutschen ah I see 🤔 I will retest I did not see any metrics until I added the policy.
I will get back to you on this one

I just checked on the permissions from the CreateOrder function in the aws-serverless-ecommerce-platform sample and it only has permissions for logs, X-Ray, API Gateway and DynamoDB.
By the way, have you decorated the handler with @metrics.log_metrics? It happened to me when I made the switch to Powertools for metrics. 😅

Right I added the decorator around the same time I added the IAM policy 😆 , I will close the issue!


DRAFT test WFLY-13894 Upgrade jboss-ejb-client to 4.0.35.Final

This is a DRAFT pull to test the upgrade of jboss-ejb-client from ejb-client-4.0.33 to ejb-client-4.0.35 (latest), to check if all CI jobs pass.

Restarted Linux SM - JDK 8 job, which previously failed with 100+ failures, now passed.

In that failed "Linux SM - JDK 8" job, the org.jboss.as.test.clustering.cluster.ejb.remote.TwoConnectorsEJBFailoverTestCase hung.  This caused the remaining 100+ tests to fail, as the server could not be started (because of the previous hung process).
This was the reason we rolled back this release.


Calibration refactor

This is a first stab at refactoring how the sensor calibration values are brought to the sensor drivers. For Snapdragon/QURT using the DriverFramework ioctl calls are not available and therefore uORB messages are used.
Note to self: the df_hmc5883 driver is not subscribing to the calbration topic yet.
@LorenzMeier Is this somewhat close to what you envisioned?

This is looking excellent. All what is needed now is to run an orb_check() in the sensor loop and copy the calibration data (my apologies if I missed that) and this should be good to go to hit master after testing.

@LorenzMeier how do you mean orb_check() in the sensor loop? Do you mean in sensors.cpp, or in the respective drivers?

In each driver. Best rate-limit it to 10 Hz or so (using a _last_cal_check variable and the hrt_absolute_time())

Ok, "in each driver" is done. Do you think thath rate limiting using hrt_absolute_time() is cheaper than orb_check every time?

Not sure about which one is more efficient - go with what is faster to implement.

This state seems to work on the Snapdragon and was also regression tested on a Pixhawk.

It looks overall good, but on non-NuttX devices doing two subsequent calibrations will give you wrong results. I do have an idea how to change things so it becomes easier and it works.

@LorenzMeier I've tested all calibrations with Pixhawk and did a quick hover in SITL. I consider this good to be merged.


Feat: Move notifications from core to modules to facilitate customization

Feature or Bugfix

Refactoring

Detail
As a rule of thumb, we encourage customization of modules while changes in core should be avoided when possible. notifications is a component initially in core which is only used by dataset_sharing. To facilitate customization of the notifications module and also to clearly see its dependencies we have:

Moved notifications code from core to modules as it is a reusable component that is not needed by any core component.
Moved dataset_sharing references inside dataset_sharing module and left notifications independent from any other module
Added depends_on in the dataset_sharing module to load notifications if the data_sharing module is imported.

TODO:

frontend changes in case notifications are not loaded
not for this PR, but as a general note, we should clean up deprecated ECS tasks

Relates

#785
#734

Security
Please answer the questions below briefly where applicable, or write N/A. Based on
OWASP 10.
N/A just refactoring
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@noah-paige and I discussed internally that this PR is just for reference and will stay in Draft. First the PR for #734 needs to be merged and then we will do the necessary changes which developed in this Draft PR.


how to start phantomjs service separately?

how to start phantomjs service separately?

http://docs.pyspider.org/en/latest/Command-Line/
http://docs.pyspider.org/en/latest/Deployment/


Move streamer out of NDS main

https://github.com/agencyenterprise/neural-data-simulator/issues/30
Introduction
User feedback: make it easier to mix and match components. Reorganize NDS into a library of components, exposing internal components with a user-friendly public API. Users should be able to create simulation scripts by importing the necessary components and connecting them together.
Changes

Move streamer extension out of NDS main folder and into a separate folder, analogous to the decoder extension
Refactor streamer settings into its own module

Open questions

[ ] Should we organize all the "extensions" together under an extensions folder?

[ ] Alternatively, are some of these extensions important enough to NDS to be moved into the main package?

Behavior
Code behavior should be the same as before.
Testing: ran run_closed_loop locally, and ran unit tests.

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

Files
Coverage Δ

...ural_data_simulator/scripts/post_install_config.py
98.14% <100.00%> (+0.03%)
:arrow_up:

... and 2 files with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.

Closing this one for now - I'll update this to follow the organization discussed in https://github.com/agencyenterprise/neural-data-simulator/issues/40, then re-open this PR.

After some more refactoring elsewhere, I think it would make the most sense merge this PR in first (i.e., refactor this out of core NDS), then move all the NDS extensions under the NDS main package. This will disentangle the 2 sets of changes, e.g. make it simpler to add in the import-linter.


Ensure only public/published resources are cataloged for discovery

Describe the feature you'd like and what it will do
A clear and concise description of what you want to happen.
Users submitting datasets through the CZNet Data Submission Portal may or may not make those datasets public/discoverable/published. In the automated functionality that copies metadata from the Data Submission Portal to the catalog database, only metadata for resources that are public, discoverable, or published with a DOI should be transferred to the catalog database.
Why is this feature important?
A short description of the importance of this feature and what it will help you achieve.
We don't want to allow discovery of or disclose metadata for private resources.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...].
Just need to verify that we are implementing appropriate safeguards to prevent metadata about private resources from ending up in the catalog database.
Additional context
Add any other context or screenshots about the feature request here.
NA

This needs a bit of design around it before I can implement. I have developed the ability to sync a submission to the search catalog, but I currently do not take into account whether a submission is public/private.
I think the simplest and most generalized solution is to simply loop through all the submissions daily and check the public endpoint for the submissions, then add/remove from the search database as appropriate.  This would also give us the opportunity to ensure the metadata is kept up to date.
The same function that checks for public/private of a submission could also check whether the metadata is up to date and could also be called as needed for triggering updates by the user manually.
I'm going to move forward with the daily job that checks submissions unless @horsburgh or someone else has a strong opinion against this solution.

@sblack-usu - I think your proposal kills two birds with one stone. A daily job would handle everything that is registered with the DSP nicely. If a resource meets any of the criteria for the public being able to view its metadata (a resources is public, published, discoverable), then it should get added to the catalog.  I realize that I'm using HydroShare's states of public, published, discoverable, but I'm guessing that anything in EarthChem and Zenodo can be mapped to those states.
A daily job should also verify that if the status of something changes from public to private, its metadata should be removed from the catalog. And, I like that a daily fetch would keep things up to date in the catalog with any changes that may have occurred in the repository.
There will still be a bunch of datasets in the catalog (Legacy CZO datasets) that are not registered with the DSP and so would not be regularly updated. We should address those as well. Our assumption is that the Legacy CZO datasets would change much less frequently (if ever), but it is possible.
The downside of a daily job is that a user who submits a dataset through the DSP or registers a dataset with the DSP probably has the expectation that the dataset will immediately appear in the catalog. Would it be possible to also trigger the immediate deposition of metadata to the catalog if you check the repository and find that the registered/submitted dataset is public, published, or discoverable? Alternatively, upon submission we could report to the user that it may take up to 24 hours for their dataset to appear in the catalog. We probably should have some messaging like this anyway to alert users that they need to share or publish their datasets if they want anyone to be able to see them (because we don't have access control functionality in the DSP).


Collapse blank lines after brackets

When collapsing a number of functions, the blank lines in between each function are not collapsed, so it ends up with a function per two lines, wasting half the screen space.
Could the collapse also include blank lines after the closing brace, so that it will end up showing a collapsed function per line without any blank lines in between?

I like this idea! Will add it to planned features.


Covert to typescript

Ideally this would use typescript.
I've attempted in the past but due to moving computers and not committing, it's been lost.

Complete and merged to master


Default path to ur_client_library urscript

PR changes default urscript that is sent to the robot when headless_mode is used.

That seems not to work for me.
ament_index_python.packages.PackageNotFoundError: "package 'ur_client_library' not found, searching: ['/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_simulation_gazebo', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_calibration', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_robot_driver', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_controllers', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_msgs', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_moveit_config', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_bringup', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_description', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ur_dashboard_msgs', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ros2_control_demos', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ros2_control_demo_bringup', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/rrbot_description', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ros2_control_test_nodes', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/ros2_control_demo_hardware', '/home/mauch/robot_folders/checkout/ur_ros2_galactic/colcon_ws/install/diffbot_description', '/opt/ros/galactic']"

However, the error might be on the client_library

With https://github.com/UniversalRobots/Universal_Robots_Client_Library/pull/131 this finally works.
However, I'll squash the two commits and change the commit message, as the script is not only used in headless mode.
@livanov93 would you like to make the changes or would you be fine with me doing them?


Changing Owners?

Is this repo changing owners? Why is everything in https://github.com/d5/tengo/pull/414/files/ec2849b7ea48512aca87cf69621d8f89ebf4113c..1dd22d7c63c704a00157747c6d08a80fcca16d7a changing the repo to github.com/sin3degrees/tengo/v2 ?

No, we're not. It was someone else's PR. I just closed it.

Thanks for the update. 👼


[Test] add dates to changelog

To make clear when particular versions have been released, please add a date to changelog entries.

Will be added going forward.


stm32/uart: add swap_rx_tx

Add an option to swap RX and TX. Tested on STM32U575ZITx on a board with swapped tx/rx pins.

CI is failing, I guess because it is some tests and examples are using uart without the swap bitfield in cr2. How can I fix this?

check here which uart versions have the swap bit, gate it with cfgs. For example, #[cfg(usart_v4)]` if only v4 has it.

I think this:
        #[allow(unused_variables)]
        let swap_rx_tx = false;

        #[cfg(any(usart_v3, usart_v4))]
        let swap_rx_tx = config.swap_rx_tx;

        if swap_rx_tx {
            let (rx, tx) = (tx, rx);
            rx.set_as_af(rx.af_num(), AFType::Input);
            tx.set_as_af(tx.af_num(), AFType::OutputPushPull);
        } else {
            rx.set_as_af(rx.af_num(), AFType::Input);
            tx.set_as_af(tx.af_num(), AFType::OutputPushPull);
        }

can be changed to this:
        cfg_if::cfg_if! {
            if #[cfg(any(usart_v3, usart_v4))] {
                if config.swap_rx_tx {
                    let (rx, tx) = (tx, rx);
                    rx.set_as_af(rx.af_num(), AFType::Input);
                    tx.set_as_af(tx.af_num(), AFType::OutputPushPull);
                } else {
                    rx.set_as_af(rx.af_num(), AFType::Input);
                    tx.set_as_af(tx.af_num(), AFType::OutputPushPull);
                }
            } else {
                rx.set_as_af(rx.af_num(), AFType::Input);
                tx.set_as_af(tx.af_num(), AFType::OutputPushPull);
            }
        }

There's a little more code repetition, but there are no warning to suppress and there's no runtime evaluation (even without compiler optimizations) of the flag when it is not available.


use plain bash instead of node because is too env dependent.

This is how I would love it to work:

On install create .ghooks or hooks folder or similar on project root
Have a plain bash file if [ -e "../../ghooks/$hookname" ] as hook in .git/hooks

This way we remove the need of any configurations file, we can use whatever language we want, we can have developer specific .env files to fix gittower alike GUI problems and all hooks are explicitly versioned and only in one place.
Also we could probably use symlinks in .git/hooks since it's the same file over and over.

Before forking I would love to understand if a PR is welcome, the way this works right now is not viable for my team. From the PM to the designer we all use git, and where everyone's node binary is unpredictable. Also most checks are better written in bash than node.

I got a prototype running with few lines
Every file in .git/hooks is a symlink to this, in this instance I symlinked .git/hooks/pre-commit
#!/bin/bash

FILENAME="$(basename "$0")"
HOOK_PATH="$(pwd)/hooks/$FILENAME"

echo "> checking for ./hooks/$FILENAME existence"

if [ -f "$HOOK_PATH" ]; then
  echo "> ./hooks/$FILENAME exists, running"
  echo ""
  $HOOK_PATH
fi

Then I have this in hooks/pre-commit
#!/bin/bash

# project specific check
if [ -f ./hooks/.env ]; then
  ./hooks/.env
else
  echo "> please create hooks/.env file"
  exit 1
fi

# project specific commands
node -v
echo "this is a test"

And hooks/.env is a git ignored file containing
. ~/.nvm/nvm.sh

I appreciate what you're trying to do and I think that it looks pretty neat. However I don't think that we're going to do anything like that for this project. We can assume the node environment because ghooks is distributed via npm. Also, it's quite easy to call a bash script from the package.json configuration and I do it all the time.
I would prefer to not make such a significant change to how this library works. So I recommend that you either fork this repo or create your own solution. Good luck!

@kentcdodds thanks for the reply. I will go ahead and hack something together :D
Just a note:

We can assume the node environment because ghooks is distributed via npm.

This is an unsafe assumption. If I do /my/custom/thing/npm install or I git commit through a different shell from what I used to install the project it won't work because the entry point is node. This is the case for git GUI like Tower and similar.
This said, I understand it is outside the scope of this module to fix this use case and I will go my own way :)

@kilianc of I npm install -g ghooks in windows, I do have node, I don't have bash... If you're using a custom npm client that doesn't use node, back for you.
As an aside, I do have and use Nash in windows... That said, there's more to node than just a Linux environment with bash, and this is an npm/node module...

Last comment from phone, forgive the spelling/grammar issues

@tracker1 not sure what you are referring to with this

@kilianc of I npm install -g ghooks in windows, I do have node, I don't have bash... If you're using a custom npm client that doesn't use node, back for you.

You missed my point about custom PATH with custom node installs if you use multiple versions of node on the same machine.

As an aside, I do have and use Nash in windows... That said, there's more to node than just a Linux environment with bash, and this is an npm/node module...

I see no issues supporting bat files or whatever format windows git hooks are written into.

this is an npm/node module

Kind of irrelevant if it doesn't actually allow me to use node with nvm. It's a tool for node and npm projects, being blindfolded and using node regardless even if it is obviously a road block is kinda silly.

turns out what I want is already out there: https://github.com/tarmolov/git-hooks-js

I am just gonna drop it here, I hacked together my own implementation https://github.com/kilianc/shared-git-hooks which is pretty much like husky but without configuration.
Main differences are:

entry point is a bash script
$PATH is frozen at the same value it was when installed
all scripts in .git/hooks are symlinks

Cool. Tweeted. :+1:

@kilianc Thanks, husky works great!
I was trying to use ghooks inside babun and ran into path translation issues, where somewhere along the line, /c/src/repo/git/hooks was getting turned into C:\cygdrive\c\src\repo\.git\hooks, causing breakage:
➜  repo git:(master) git checkout experimental
Switched to branch 'experimental'
Your branch is up-to-date with 'origin/experimental'.
module.js:341
    throw err;
    ^

Error: Cannot find module 'C:\cygdrive\c\src\repo\.git\hooks\post-checkout'
    at Function.Module._resolveFilename (module.js:339:15)
    at Function.Module._load (module.js:290:25)
    at Function.Module.runMain (module.js:447:10)
    at startup (node.js:141:18)
    at node.js:933

Husky doesn't suffer from this issue. I'm not claiming the error is on ghook's part, but it'd still be nice if whatever husky is doing to make it work with babun could somehow trickle into ghooks.


Add config for backpack_email_column

WHY
BEFORE - What was wrong? What was happening before this PR?
If developer set other column not called "email" php artisan route:list will fail, this is a small version of PR https://github.com/Laravel-Backpack/CRUD/pull/4623
AFTER - What is happening after this PR?
Developer can set any column as login and any as email, but if this last one is not called email will disable recovery password process (because Laravel is hardcode as email column)
HOW
How did you achieve that, in technical terms?
Update helper and add new check on recovery links
Is it a breaking change?
No
How can we test the before & after?
Change name to email column on user table and set it on base.php

I like how this looks! So small and clean, super easy to merge. @pxpm does this have your approval?
Cheers!

@tabacitu I assigned you because the only thing I had to point was a text change, and probably you are the better fit to review both me and @jorgetwgroup texts 👍
Cheers


Weird usage with Go modules

When using Go modules with the provided commands go.mod and go.sum need to be in the root folder, this makes the user opt-into this pattern.
When using run-commands with cwd this can be circumvented by I feel it should either be document or have the cwd set to the Go project (this would be preferable as Go checks folders upwards).
I can help with this if needed.

Hey there @fallion - thanks for all your input on this project. Really helpful!
As you can see I've published a few fixes up and I'd like to address this one too.
Can you tell if this fix works for you?
I've chosen to always strip off the root from the main file so that it doesn't break any existing apps.

Fix is published with version 1.6, let me know if you run into any issues!


Works great but conflicts with standard datepicker

This is working really well but I'm trying to build something where I have a standard date picker (one day only) as well as a range.  If I put the regular datepicker from angular-ui-bootstrap on the same page as this datepicker, the standard one stops working.  Behavior I'm seeing is a need to double-click in order to get the date buttons to highlight, and the ng-model for the standard date picker stops being updated.  I've tried initiating it with a value, but it gets stuck with that value.  If I initiate it with no value it appears to never pick up any value.  Not sure what's causing this but thought sharing this might help.

Could you post a sample? The directive shouldn't affect anything not marked 'datepicker'.

Also see #14.

I forked a plunkr from your original example and this actually fixed part of my original problem, so my original issue of the single date not registering to the ng-model must be an issue with the implementation in my app.  I will take a look at it.
I was able to reproduce the issue with #14 however where the dates from the multi-select picker show up as active on the single picker.
Plunkr here: http://plnkr.co/edit/jpyYlN?p=preview

I think this is related to #14. Let me know if you think I should re-open.


Add integration test for scraper demo

I also made a few minor corrections to the scraper demo doc

@sjezewski whenever you get a chance (or let me know if I should reassign)


enable mac security for fragmented mle message

Fixed DEV-1275
According to Thread spec 1.1.0, chapter 4.9:
MLE messages that fit within a single 802.15.4 frame MUST NOT use 802.15.4 security. MLE messages that require fragmentation by 6LoWPAN MUST use 802.15.4 security.

Current coverage is 69.42% (diff: 75.00%)

Merging #1202 into master will decrease coverage by 3.10%

@@             master      #1202   diff @@
==========================================
  Files           125        124     -1
  Lines         17218      17195    -23
  Methods        2433       2432     -1
  Messages          0          0
  Branches       2076       2073     -3
==========================================
- Hits          12487      11937   -550
- Misses         3951       4496   +545
+ Partials        780        762    -18

Powered by Codecov. Last update cabb885...7b1916d


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/base-ocp-vpc/ibm (source)
module
patch
3.31.0 -> 3.31.1

terraform-ibm-modules/cbr/ibm (source)
module
minor
1.26.0 -> 1.27.0

Release Notes

terraform-ibm-modules/terraform-ibm-base-ocp-vpc (terraform-ibm-modules/base-ocp-vpc/ibm)
v3.31.1
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cos/ibm to v8.11.14 (#527) (90b5c26)

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.27.0
Compare Source
Features

Enable traffic flow from SCC to COS is added (#525) (789366a)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 6.0.3 :tada:
The release is available on:

GitHub release
v6.0.3

Your semantic-release bot :package::rocket:


soc/arm/silabs/bg2x: support DFU

This PR contains contains following changes:

Enables flash driver for SiLabs EFR32BG22 and BG27 that was disabled by default.
Updates partition tables for Thunderboard EVB for those SoCs (removes scratch partition, increases size of MCUboot partition to 48K to fit recent MCUboot builds that are ~40KB), sets zephyr,code-partition in DTS.
Partition tables are moved from SoC DTS files to board DTS files, and this layout is aligned between BG22 and BG27.

With these changes, MCUboot & mcumgr can perform a DFU over UART.

In my opinion good idea. I am using a similar patch locally in my project with EFM32PG22
Might be worth to also add mcuboot-button0 = &button0; to device tree
If I remember correctly MCUBoot uses flash erases when swapping image 0 and 1 during DFU.
Flash erases are required to erase pages (8K on BG2X).
Because of that for me MCUBoot crashed when image partitions were not page aligned.
In you partition table this is not the case for 0x45000.
To test that you need firmware image support in the main image.
This way you can upload an image to slot 1 and mark it for testing.
MCUBoot should then crash during swap.
If this this does not crash please let me know

@kmeinhar
Indeed you're right! I overlooked the page size issue on BG22. I've re-tested it now with MCUboot and I see that although it does not crash per se, the flash driver reports erase errors due to flash areas being not aligned to the page size. I've pushed updated version of BG22 partition table and verified that with it I can upload & swap images with mcumgr several times in a row.
I've also added DT aliases in the third commit as you've proposed.


--tabColor and setTabColor should accept color names

Description of the new feature/enhancement
It'd be cool if --tabColor and setTabColor should accept color names in addition to hex-codes. Specifically, something like this would be accepted:
{ "command": { "action": "setTabColor", "color": "red" }, "keys": "" }

Proposed technical implementation details (optional)
Detecting if it's a name vs hex-code should be pretty straightforward: check if the leading character is a #.
#7578 may help detect if the color name actually exists?

If we’re going to do that, we should converge all color converters on the VT color parser. All of them. This will change the semantics of our color parser slightly. It’s the only right thing to do.

If we’re going to do that, we should converge all color converters on the VT color parser. All of them. This will change the semantics of our color parser slightly. It’s the only right thing to do.

Back then Dustin brought up the idea of "all the color we know about" in #7578. The XParseColor spec includes the # format which I think generally surpasses the plain hex format. So yeah this is currently possible using the functions added in #7578 .
You can just feed ColorFromXTermColor with a string and you'll have the color. See https://github.com/microsoft/terminal/blob/3181b6a5173b915e9f3d2fb44468072126d23881/src/types/ut_types/UtilsTests.cpp#L158

Back then Dustin brought up the idea of "all the color we know about" in #7578. The XParseColor spec includes the # format which I think generally surpasses the plain hex format. So yeah this is currently possible using the functions added in #7578 .
You can just feed ColorFromXTermColor with a string and you'll have the color. See https://github.com/microsoft/terminal/blob/3181b6a5173b915e9f3d2fb44468072126d23881/src/types/ut_types/UtilsTests.cpp#L158

Yeah, it'd also be nice if we could allow for things like "foreground": "red" in the JSON.

Yeah, it'd also be nice if we could allow for things like "foreground": "red" in the JSON.

The only downside is that #eee will turn into #e0e0e0 (X11 rules) instead of #eeeeee (CSS rules, current WT)

The only downside is that #eee will turn into #e0e0e0 (X11 rules) instead of #eeeeee (CSS rules, current WT)

Well, also that the color table will be copied into a third binary because of static linking but hey, we have a huge Utils problem already.

Well, also that the color table will be copied into a third binary because of static linking but hey, we have a huge Utils problem already.

I think we can prioritize CSS rules if the input is in the form of #eee in this new ColorFromString function. On the VT side, we keep the old ColorFromXTermColor. This way we'll have the best of the both worlds. (anyone noticed that this is actually Hannah Montana reference, anyone?...)

I think we can prioritize CSS rules if the input is in the form of #eee in this new ColorFromString function. On the VT side, we keep the old ColorFromXTermColor. This way we'll have the best of the both worlds. (anyone noticed that this is actually Hannah Montana reference, anyone?...)

If you're going to prioritize CSS rules for the hex formats, then you also need to consider whether you want to prioritize CSS rules for the color names too (for example, CSS green and X11 green are very different colors). Sooner or later someone is going to ask why a particular color is what it is. That's easy enough to explain if we can just say we're using CSS, or we're using X11. It's harder to justify if we've got a weird mix of the two.

If you're going to prioritize CSS rules for the hex formats, then you also need to consider whether you want to prioritize CSS rules for the color names too (for example, CSS green and X11 green are very different colors). Sooner or later someone is going to ask why a particular color is what it is. That's easy enough to explain if we can just say we're using CSS, or we're using X11. It's harder to justify if we've got a weird mix of the two.

That's actually a valid concern. Maybe we need to also introduce CSS colors and use them specifically for colors in JSON, etc.
获取 Outlook for Androidhttps://aka.ms/ghei36

From: James Holderness<EMAIL_ADDRESS>Sent: Tuesday, January 26, 2021 6:51:09 PM
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
If you're going to prioritize CSS rules for the hex formats, then you also need to consider whether you want to prioritize CSS rules for the color names too (for example, CSS green and X11 green are very different colors). Sooner or later someone is going to ask why a particular color is what it is. That's easy enough to explain if we can just say we're using CSS, or we're using X11. It's harder to justify if we've got a weird mix of the two.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-767462876, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL2ENJXX3JS5KIVDGPTS32NB3ANCNFSM4WSYX26Q.

That's actually a valid concern. Maybe we need to also introduce CSS colors and use them specifically for colors in JSON, etc.
获取 Outlook for Androidhttps://aka.ms/ghei36

From: James Holderness<EMAIL_ADDRESS>Sent: Tuesday, January 26, 2021 6:51:09 PM
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
If you're going to prioritize CSS rules for the hex formats, then you also need to consider whether you want to prioritize CSS rules for the color names too (for example, CSS green and X11 green are very different colors). Sooner or later someone is going to ask why a particular color is what it is. That's easy enough to explain if we can just say we're using CSS, or we're using X11. It's harder to justify if we've got a weird mix of the two.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-767462876, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL2ENJXX3JS5KIVDGPTS32NB3ANCNFSM4WSYX26Q.

I think that we should converge on the X11 color names and accept the breaking change (the loss of 4 bits of depth on the short-form hex code colors). @zadjii-msft what do you think?

I think that we should converge on the X11 color names and accept the breaking change (the loss of 4 bits of depth on the short-form hex code colors). @zadjii-msft what do you think?

I think if you're using #abc now as an alias for #aabbcc, and we break this by evaluating that now as #a0b0c0, and the workaround is "change the value to #aabbcc", then I think that's a perfectly reasonable story. People can deal with that for all of the 4 people using #abc style colors and who can tell the difference between #ffffff and #f0f0f0.

I think if you're using #abc now as an alias for #aabbcc, and we break this by evaluating that now as #a0b0c0, and the workaround is "change the value to #aabbcc", then I think that's a perfectly reasonable story. People can deal with that for all of the 4 people using #abc style colors and who can tell the difference between #ffffff and #f0f0f0.

Just so long as no one's expecting "red" to mean "the value of red from the color scheme". That's right out. Not doing that 😛

Just so long as no one's expecting "red" to mean "the value of red from the color scheme". That's right out. Not doing that 😛

no one's expecting "red" to mean "the value of red from the color scheme".

I'm not sure, man. Our users have proven to be...imaginative .

no one's expecting "red" to mean "the value of red from the color scheme".

I'm not sure, man. Our users have proven to be...imaginative .

I'm fine with converging on the X11 color names. I do believe most people are using #aabbcc instead of #abc (see, Github isn't picking this as a valid color). Let's face it, people who bother to use a customized hex code would probably be unsatisfied with simple colors like #abc.
Also I'm proposing to add support for rgb(111,  111, 111) format in addition to the X11 formats. This format is used widely on the web as we know it.

I'm fine with converging on the X11 color names. I do believe most people are using #aabbcc instead of #abc (see, Github isn't picking this as a valid color). Let's face it, people who bother to use a customized hex code would probably be unsatisfied with simple colors like #abc.
Also I'm proposing to add support for rgb(111,  111, 111) format in addition to the X11 formats. This format is used widely on the web as we know it.

Yea I'm cool with the rgb(12, 34, 56) version too

Yea I'm cool with the rgb(12, 34, 56) version too

Also I'm proposing to add support for rgb(111, 111, 111) format in addition to the X11 formats.

Can we please make sure that's not exposed in the OSC palette sequences though, unless it's one of the standard formats supported by other terminals.

Also I'm proposing to add support for rgb(111, 111, 111) format in addition to the X11 formats.

Can we please make sure that's not exposed in the OSC palette sequences though, unless it's one of the standard formats supported by other terminals.

Of course. It will be separate from existing X11 code.
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: James Holderness<EMAIL_ADDRESS>发送时间: Thursday, January 28, 2021 4:11:16 AM
收件人: microsoft/terminal<EMAIL_ADDRESS>抄送: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>主题: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
Also I'm proposing to add support for rgb(111, 111, 111) format in addition to the X11 formats.
Can we please make sure that's not exposed in the OSC palette sequences though, unless it's one of the standard formats supported by other terminals.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-768546798, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL2KCQV732OHCOTCAPTS4BXOJANCNFSM4WSYX26Q.

Of course. It will be separate from existing X11 code.
获取 Outlook for iOShttps://aka.ms/o0ukef

发件人: James Holderness<EMAIL_ADDRESS>发送时间: Thursday, January 28, 2021 4:11:16 AM
收件人: microsoft/terminal<EMAIL_ADDRESS>抄送: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>主题: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
Also I'm proposing to add support for rgb(111, 111, 111) format in addition to the X11 formats.
Can we please make sure that's not exposed in the OSC palette sequences though, unless it's one of the standard formats supported by other terminals.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-768546798, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL2KCQV732OHCOTCAPTS4BXOJANCNFSM4WSYX26Q.

I don't necessarily see the value in rgb(x,y,z). If we are working to reduce the number of individual color parsers that we have to support . . . we shouldn't do so and then turn around and add another. As the settings UI gets used more and more, I expect that the incidence of folks using the uncommon syntaxes will evaporate until it reaches 0.

I don't necessarily see the value in rgb(x,y,z). If we are working to reduce the number of individual color parsers that we have to support . . . we shouldn't do so and then turn around and add another. As the settings UI gets used more and more, I expect that the incidence of folks using the uncommon syntaxes will evaporate until it reaches 0.

I agree settings UI will reduce the usage of color values in settings.json. However, for command line usgae, i think we should support as many as formats. Hence the rgb idea.
获取 Outlook for Androidhttps://aka.ms/ghei36

From: Dustin L. Howett<EMAIL_ADDRESS>Sent: Thursday, January 28, 2021 5:54:58 AM
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
I don't necessarily see the value in rgb(x,y,z). If we are working to reduce the number of individual color parsers that we have to support . . . we shouldn't do so and then turn around and add another. As the settings UI gets used more and more, I expect that the incidence of folks using the uncommon syntaxes will evaporate until it reaches 0.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-768602350, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL7BFD4YV3LQCQG7KZTS4CDTFANCNFSM4WSYX26Q.

I agree settings UI will reduce the usage of color values in settings.json. However, for command line usgae, i think we should support as many as formats. Hence the rgb idea.
获取 Outlook for Androidhttps://aka.ms/ghei36

From: Dustin L. Howett<EMAIL_ADDRESS>Sent: Thursday, January 28, 2021 5:54:58 AM
To: microsoft/terminal<EMAIL_ADDRESS>Cc: Chester Liu<EMAIL_ADDRESS>Comment<EMAIL_ADDRESS>Subject: Re: [microsoft/terminal] --tabColor and setTabColor should accept color names (#8893)
I don't necessarily see the value in rgb(x,y,z). If we are working to reduce the number of individual color parsers that we have to support . . . we shouldn't do so and then turn around and add another. As the settings UI gets used more and more, I expect that the incidence of folks using the uncommon syntaxes will evaporate until it reaches 0.
―
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/terminal/issues/8893#issuecomment-768602350, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABD6BL7BFD4YV3LQCQG7KZTS4CDTFANCNFSM4WSYX26Q.


IMDB Search

Extension
https://www.raycast.com/ryan/imdb
Description
Hi,
I believe the search does not search for movie names in all languages.
It makes it hard to know how to search.
An example of a movie I cannot find:
l'immensità
Link in IMDB:
https://www.imdb.com/title/tt13051724/?ref_=fn_al_tt_1
Thanks for letting me know if there is something I need to do or if you can fix it?
thanks
Steps To Reproduce
See example
Current Behaviour
no results
Expected Behaviour
link to https://www.imdb.com/title/tt13051724/?ref_=fn_al_tt_1

@ryan-exe  I hope the issue is clear, would you mind having a look?
thank you so much!


[Linux] Add MimeType for appimage and metainfo file support

MimeType was missing from appimage packaging, it is similarly implemented as done for deb and rpm.
Metainfo file is used by all of the major apps and it tells about the metadata of the app used by appstores and flathub. Reference: #174
Pubspec.lock shoudn't be commited and it should be gitignored, I have deleted so that it is the correct behavior.

Please provide an example that you could add to examples/hello_wolrd.

@lijy91 Done! Please check now.

LGTM


[Snyk] Security upgrade io.streamnative:pulsar-client-original from <IP_ADDRESS> to <IP_ADDRESS>

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `maven` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

pom.xml

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Upgrade
Breaking Change
Exploit Maturity

539/1000  Why? Has a fix available, CVSS 6.5
Denial of Service (DoS) SNYK-JAVA-IONETTY-5725787
io.streamnative:pulsar-client-original:  <IP_ADDRESS> -> <IP_ADDRESS>
No
No Known Exploit

(*) Note that the real score may have changed since the PR was raised.
Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Denial of Service (DoS)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Add support for web output ($npm run pack:web).

2 seperate commits as I had to touch the renderer config ("app/src/renderer" to "app/src" business). Please let me know if that was a bad idea indeed.

It seems a little odd to have app/src/web/main.js, and then use app/src/renderer for the actual app. Why not just use an environment variable to describe the target platform?

The restructure looks great! I'll reopen the case once that branch is ready.
Just for the sake of learning though... is there any example of doing this via env vars in the source?
(Probably I'm missing something. The env var solution sounds worse to me as it would mean turning main.js into another template resource [like index.ejs] which further implicates things.)

I gave this setup a try with the simplified structure, but ran into an error.
With the default LandingPageView, UglifyJS throws an error because of some calls to Electron and Node versions in Versions.vue file. After removing those, it compiles just fine.
Just something to consider.
Thanks for the great work otherwise!

Web output has landed in the dev branch! Check #171 for more info. Closing.


Black border intermittently appearing at bottom of tree view

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Bug
In the process of dragging the vscode project explorer tab under the Light theme style, a black border sometimes appears, which is very annoying. May I ask what caused this, hope to fix it quickly, it is really frustrating. . . . Hope to fix it quickly. . .
Please see the attachment for specific performance as follows:
https://user-images.githubusercontent.com/54253524/132697041-efbb0ce2-8015-48af-9d1f-5429ddf6fce6.mp4
VS Code version: Code - Insiders 1.60.0-insider (e7d7e9a9348e6a8cc8c03f877d39cb72e5dfb1ff, 2021-09-01T10:40:58.134Z)
OS version: Windows_NT x64 10.0.19041
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz (4 x 2494)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
9.59GB (3.55GB free)

Process Argv

Screen Reader
no

VM
67%

https://user-images.githubusercontent.com/54253524/132696691-4f275770-4ac2-4740-8662-69216dce9de3.mp4

https://user-images.githubusercontent.com/54253524/132705031-4c6fb974-64ce-441f-a49b-e82dbec8dccd.mp4

This bug also exists in the default Dark theme style, which is very distracting. . .
https://user-images.githubusercontent.com/54253524/132788223-7ce9bdb1-fa60-4ff5-b496-c003fe2483eb.mp4
https://user-images.githubusercontent.com/54253524/132788523-ea3c462d-0ca5-49fd-8fa2-4c9ec9158159.mp4

Can you reproduce with zoom level 0?

"window.zoomLevel": 2,
"workbench.iconTheme": "material-icon-theme",
"workbench.colorTheme": "Visual Studio Dark",

The previous zoom level was: 2
The following demo is the zoom level I set to reproduce the bugs, for example: set the zoom level to 1, please see the specific demo as follows:
"window.zoomLevel": 1,
"workbench.iconTheme": "material-icon-theme",
"workbench.colorTheme": "Default Light+",
https://user-images.githubusercontent.com/54253524/132937723-f4df6b65-2c64-4b9d-beab-e2db1c2af001.mp4

https://user-images.githubusercontent.com/54253524/132937800-a132fde1-6e77-471e-b939-0d3a501a807c.mp4

Hello! Can this problem be fixed?

Can you reproduce with zoom level 0?

What's your current zoom level? Can you reproduce with zoom level 0?
This problem is very serious in version 1.60.2, which completely affects the consistent experience and distraction of the UI. Will you fix it? ? ?

Can you reproduce with zoom level 0?

"window.zoomLevel": 2,
"workbench.iconTheme": "material-icon-theme",
"workbench.colorTheme": "Visual Studio Dark",
The previous zoom level was: 2
The following demo is the zoom level I set to reproduce the bugs, for example: set the zoom level to 1, please see the specific demo as follows:
"window.zoomLevel": 1,
"workbench.iconTheme": "material-icon-theme",
"workbench.colorTheme": "Default Light+",
https://user-images.githubusercontent.com/54253524/132937723-f4df6b65-2c64-4b9d-beab-e2db1c2af001.mp4
https://user-images.githubusercontent.com/54253524/135702256-929fe76e-cf2b-466c-9a1d-4be46560471b.mp4

Can you reproduce with zoom level 0?

Hello, teacher!
If you confirm my needs, please reply, thank you very much!

What's your current zoom level? Can you reproduce with zoom level 0?

Hello, teacher!
If you confirm my needs, please reply, thank you very much!

Can you reproduce with zoom level 0?

Hello, teacher!
excuse me! Could this problem be fixed by the next milestone? Looking forward to your fix in the next milestone, thank you very much!

Hello, teacher!
excuse me! Could this problem be fixed by the next milestone? Looking forward to your fix in the next milestone, thank you very much!

https://user-images.githubusercontent.com/54253524/142089635-a2428efe-7366-4e07-976a-798a10959545.mp4

Can you reproduce with zoom level 0?

Hello, teacher!
VSCode is a great project that gives users the greatest autonomy from the very beginning, allowing users to personalize configurations according to their own habits, so it can fully surpass the long-born jetbrion series in just 5 years. The most important reason is that VSCode's open source thinking and allowing users to customize and configure VSCode development methods to the greatest extent can achieve today's achievements. I hope VSCode can continue to develop this way of thinking and development, focusing on every reasonable request of users and need. Thank you here!
vscode contributors
20220126

Hello, teacher!
VSCode is a great project that gives users the greatest autonomy from the very beginning, allowing users to personalize configurations according to their own habits, so it can fully surpass the long-born jetbrion series in just 5 years. The most important reason is that VSCode's open source thinking and allowing users to customize and configure VSCode development methods to the greatest extent can achieve today's achievements. I hope VSCode can continue to develop this way of thinking and development, focusing on every reasonable request of users and need. Thank you here!
vscode contributors
20220126

White lines appear on text selected with the mouse


Reduce assignment branch condition-size for get_awarded_review_score function

In get_awarded_review_score function in the file app/helpers/review_mapping_helper.rb , if condition will have conditions with simple variables instead of instance variable attributes calling.

Approved

Approved. Merge conflicts resolved. Ready to merge.


experiment-management: remove warning about no parameter validation

Per https://github.com/iterative/dvc/pull/6521
Covers feature change iterative/dvc#6521. Parameters will now be validated.
Related issue iterative/dvc#5477

This simply removes the warning about lack of parameter validation at the moment.

Do you mind marking this as a draft until https://github.com/iterative/dvc/issues/5477 is merged?

merged #2944 but this is still open

I think Restyled is having some issues (again)... After merging it to master, this PR showed conflicts with the formatted bits (should've been closed by that). And now that I solved the conflicts (used master versions) it created another PR with 0 changes (how do you even do that?): https://github.com/iterative/dvc.org/pull/2945

how do you even do that?

My master merge wasn't correct -- left an accidental trailing space in there. The Restyled PR changed it back to the already merged version in master. (That's how.)
Now I fixed the formatting here and there are no changes in this PR... 🤷 merging!


148 bug from the user

On line 364 of BeamSolver.cpp there is a new use of the Embarcadero method ReadBool()
To compile on linux, we need to create an g++ compatible wrapper for this method.
@robnagler or @garsuga  -- could you assist?

@garsuga lmk if you need help

It builds now but it needs to be tested. I am not sure where ReadBool gets used, @bruhwiler can you see if it runs fine for you?

The tests are failing now with something that seems unrelated to ReadBool

@skutsaev I am going to hand this back to you to fix whatever is failing in the tests.

It compiles and runs on my computer fine

@skutsaev this PR introduces a malloc issue:

tests/pkcli/beam_solver_test.py::test_beam_solver malloc(): corrupted top size
Fatal Python error: Aborted

Current thread 0x00007fa272e87740 (most recent call first):
  File "/home/runner/work/rshellweg/rshellweg/rshellweg/solver.py", line 22 in solve
  File "/home/runner/work/rshellweg/rshellweg/tests/pkcli/beam_solver_test.py", line 24 in test_beam_solver

Can you think of a reason why this would be?

Maybe there is some incorrect array creation, which Embarcadero treats fine? I'll try to investigate, but it's not an easy check without more details.

You can use our jupyter server to compile this. Visit sirepo.com/jupyter. Once logged in, start a terminal in JupyterLab then:
git clone https://github.com/radiasoft/rshellweg
cd rshellweg
pip install -e .
pykern test

If you need more help, let me know.

Unfortunately, python and Linux are beyond my expertise, so I will need to use embarcadero to look for the problem

If you aren't getting the error in Embarcadero, we'll debug further.

I don’t, but when I tried to compile with CodeGuard enabled (the feature to locate memory leaks), it crashed. So there might be something. I’ll check when I get to the computer


How to use xlsx-style in typescript file

Have been trying to make use of "Cell.s" in XLSX, but not able to.
Tried giving the styles format in xml and json, but with no success.
Since am not able to use style using xlsx, have asked for xlsx-style .d.ts file or
any way to include style using xlsx.
below is my code:
It would be very helpful if you are able to provide any example where i can give background colour in cell row or header using XLSX (in typescript file, please don't give for .js)
public exportAsExcelFile(json: any[], excelFileName: string): void {
const worksheet: XLSX.WorkSheet = XLSX.utils.json_to_sheet(json);
var wscols = [
{ width: 20 },
{ width: 15 },
{ width: 20 },
{ width: 15 },
{ width: 13 },
{ width: 15 },
{ width: 30 },
{ width: 15 }
];
worksheet['!cols'] = wscols;
worksheet['!rows'] = [{ hpx: 40 }];

const workbook: XLSX.WorkBook = { Sheets: { 'data': worksheet }, SheetNames: ['data'] };
const excelBuffer: any = XLSX.write(workbook, { bookType: 'xlsx', type: 'array', cellDates: true, cellStyles: true });
this.saveAsExcelFile(excelBuffer, excelFileName);

}
private saveAsExcelFile(buffer: any, fileName: string): void {
const data: Blob = new Blob([buffer], {
type: EXCEL_TYPE
});
FileSaver.saveAs(data, fileName + 'export' + new Date().getTime() + EXCEL_EXTENSION);
}

Also please don't give link for .d.ts file of XLSX, because i already have it.
need the same for XLSX-style

See https://github.com/SheetJS/js-xlsx/issues/1032 -- this is the repo for the xlsx project.


Update dataplane env vars in controlplane when dataplane service changes

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
The ControlPlane deployments are pointed to the correct DataPlane service via 2 different env vars: CONTROLLER_PUBLISH_SERVICE, and CONTROLLER_KONG_ADMIN_URL. These 2 vars contain the correct dataplane service address to talk to. With https://github.com/Kong/gateway-operator/pull/91, when the DataPlane service is deleted, it is recreated automatically by the dataplane-controller; the problem is that the service name is not static, but created through the generateName field. Hence, every time the DataPlaneservice is recreated by the dataplane-controller, the controlplane-controller should also update the ControlPlane deployment env vars.
Proposed Solution
We should trigger an update on the controlplane-controller every time the DataPlane service changes. To do so, we can watch the Dataplane services in the controlplane-controller and trigger a reconciliation loop every time an event occurs.
Additional information
No response
Acceptance Criteria

[ ] Every time the dataplane service is created, the dataplane service env vars are set in the controlplane deployment
[ ] Every time the dataplane service is deleted, the dataplane service env vars are unset from the controlplane deployment

Are we in position to actually work on this? It looks like the status of #91 is unclear until it's refactored around unrelated reconcile updates, and it doesn't look like it makes sense to handle this without #91 merged.
I opened https://github.com/Kong/gateway-operator/pull/130 into #91's branch since I figure the approach will be similar despite needs to refactor, and because we may as well get an answer on the environment update question.

Do we want to allow direct edits to the ControlPlane Deployment environment or require that you always update the ControlPlane?
To handle this we can either compare the entire environment from the current ControlPlane Deployment against the environment generated by the latest run of the ControlPlane reconciler and update the Deployment with the latter if they differ, or we can surgically find the publish service variable and update only that one.
Updating the whole thing wipes out any user changes made directly to the Deployment, but I think that's okay since we intend for them to edit the environment via the ControlPlane. It solves the issue of not having the latest value for every generated value in the environment, current and future alike. Surgical edits only fix publish service, and would maybe be a bit annoying to handle for individual keys if we need to update a lot (maybe--right now I've implemented it by looping over the desired environment and then have a nested loop over the actual environment, but we probably could build something that transforms envar slices to and from maps to have only a single loop).

Do we want to allow direct edits to the ControlPlane Deployment environment or require that you always update the ControlPlane?

I think that we should allow only updates on the controlplane object. The controlplane deployment is owned by the controlplane controller and it should be in charge of enforcing the expected spec.

To handle this we can either compare the entire environment from the current ControlPlane Deployment against the environment generated by the latest run of the ControlPlane reconciler and update the Deployment with the latter if they differ, or we can surgically find the publish service variable and update only that one.

I vote for wiping out every change made by the user on the deployment spec and enforcing the deployment spec as it is intended to be by the controller (the user can customize the controlplane object to perform controlplane deployment customizations). I think that only labels and annotations should be modifiable by the user.


Merge 2-7 updates to live

OPS Build status updates of commit 609f220:
:clock10: Pending: waiting for processors (8 builds ahead of you)
:warning: OPS Build is busy, click here to report any potential performance downgrade issues, or you could view OPS Portal for trend of build queue length.

  This is the expected behavior of the build system.

OPS Build status updates of commit 609f220:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

api/Access.Attachment.DblClick.md
:white_check_mark:Succeeded
View

api/Access.Attachment.Dirty.md
:white_check_mark:Succeeded
View

api/Access.Attachment.Enter.md
:white_check_mark:Succeeded
View

api/Access.Attachment.Exit.md
:white_check_mark:Succeeded
View

api/Access.Attachment.GotFocus.md
:white_check_mark:Succeeded
View

api/Access.Attachment.KeyPress.md
:white_check_mark:Succeeded
View

api/Access.Attachment.LostFocus.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnEnter.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnExit.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnGotFocus.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnKeyDown.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnKeyPress.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnKeyUp.md
:white_check_mark:Succeeded
View

api/Access.Attachment.OnLostFocus.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Action.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.AddColon.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.AfterUpdate(even).md
:white_check_mark:Succeeded

api/Access.BoundObjectFrame.AfterUpdate(property).md
:white_check_mark:Succeeded

api/Access.BoundObjectFrame.AfterUpdate-event.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.AfterUpdate-property.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Application.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.AutoActivate.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.AutoLabel.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BackColor.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BackShade.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BackStyle.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BackThemeColorIndex.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BackTint.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BeforeUpdate(even).md
:white_check_mark:Succeeded

api/Access.BoundObjectFrame.BeforeUpdate(property).md
:white_check_mark:Succeeded

api/Access.BoundObjectFrame.BeforeUpdate-event.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BeforeUpdate-property.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderColor.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderShade.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderStyle.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderThemeColorIndex.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderTint.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BorderWidth.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.BottomPadding.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Class.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Click.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ColumnHidden.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ColumnOrder.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ColumnWidth.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ControlSource.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ControlTipText.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.ControlType.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.Controls.md
:white_check_mark:Succeeded
View

api/Access.BoundObjectFrame.DblClick.md
:white_check_mark:Succeeded
View

This comment lists only the first 50 files in the pull request.
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.


Problem: Some users unable to upgrade to the latest version

The latest version of the Python driver, 0.5.0a1, is a "pre-release" (or at least that's what I heard; I couldn't find any indication of this fact on PyPI, i.e. at https://pypi.python.org/pypi/bigchaindb-driver ). That means that just doing:
$ pip install -U bigchaindb_driver

will not upgrade them to 0.5.0a1, because it's not a "release", it's a pre-release. They must do:
$ pip install bigchaindb_driver==0.5.0a1

To Do

Figure out how to get PyPI to indicate that it's a pre-release.
Update the docs (including those on PyPI) on how to install / upgrade the driver, to account for the case when the latest version is a pre-release.

Remove all mention of how to update the Python driver from the README.md because PyPI automatically takes care of that now.

But removing it from the readme is not helpful for people who just visit the GitHub repo.

But removing it from the readme is not helpful for people who just visit the GitHub repo.

Maybe the README.rst file could point to the PyPI page listing all releases and pre-releases, each of which has its own page with the correct instructions on how to install it, instructions which change over time!
https://pypi.org/project/bigchaindb-driver/#history

ok, I think this makes sense.


Add ignore std error for weird git push origin HEAD:master stderr f…

…or no reason with the msg To https://github.com/ciena-blueplanet/ember-cli-visual-acceptance.git.
#patch#
@sglanzer I don't know what else to do with this issue, https://travis-ci.org/ciena-blueplanet/ember-cli-visual-acceptance#L1436-L1442 ,  besides cloning the entire repo on travis and cd'ing into it so we're not detached. (I assume not being detached would fix this weird stderr but idk)


Elision with escape seq

Fixes #713.

Cc: @jhasse

Fixes #713.

I'm seeing some issues with this when using it on projects locally. Will debug and figure out what's missing and add a testcase for it.

What do you think about #1600?

Sure, that offers another frontend to be able to do what I want, but the thing is that today elision logic in ninja itself is broken with color (though this is also related to #1584). If #1600 is accepted, should color just be outright rejected from NINJA_STATUS? Or is it "basic" enough that having the support is worthwhile to to require some other frontend for color?

Good questions. We probably should also fix this in Ninja itself.
Btw: Did you have a look at my take on ANSI-aware elide_middle (in Python though)? https://github.com/jhasse/ja/blob/da264576a85d4d8b1de4517487bfe501b6337915/ja/native.py#L258

Yes, I had looked at it. I started the code from my old %0-based algorithm though. I should probably just try and redo this code from scratch (using that as a template) rather than trying to adapt the older logic which wasn't span-based.

Or have consider using my frontend, comes with other awesome features, too ;)

I had another go at it in #2428.


Signal 15?

My gitea log is filled with these, is this normal?

Gitea version? OS? Docker image?

Gitea version? OS? Docker image?

Sorry, I thought the Gitea version being in the screenshot would be enough.
Gitea Version: 1.12 Dev
OS: Docker Desktop for Windows
Yes, it's a Docker image, from the gitea/gitea repo

It's not normal no and gitea is constantly restarting for some reason.

If anyone needs any more information to help me sort this out, please just say what is needed; I'm not a techy type of person, so I have no idea where to start or what's needed.

@YouveGotMeowxy Can you please create an example in try.gitea.io? It's our public test site and it always runs the nigthtly build from Gitea.

So I think this is more likely to be a configuration issue.
@YouveGotMeowxy could you please post a suitably sanitised copy of your app.ini?
Please describe your setup of Gitea and where and how you've configured its directories including the log directories, and finally that you've confirmed the Gitea process owns all of the appropriate directories. The final thing to ensure is that you're definitely not trying to open Gitea on a port below 1024.

@zeripath Here's my docker compose for gitea:
version: '2'

services:

    gitea:
        image: 'gitea/gitea:latest'
        container_name: gitea
        environment:
            - TZ='America/Chicago'
            - PUID=197609
            - PGID=197121
        volumes:
            - 'my_gitea_data:/data'
        restart: unless-stopped
        networks:
            - default

networks:
    default:
        external:
            name: my-coding

I don't open any ports on the host.
Here's my app.ini
APP_NAME = Gitea: Git with a cup of tea
RUN_MODE = prod
RUN_USER = git

[repository]
ROOT = /data/git/repositories

[repository.local]
LOCAL_COPY_PATH = /data/gitea/tmp/local-repo

[repository.upload]
TEMP_PATH = /data/gitea/uploads
; Max number of files per upload. Defaults to 5
MAX_FILES = 500
[server]
APP_DATA_PATH    = /data/gitea
SSH_DOMAIN       = my.domain.com
HTTP_PORT        = 3000
ROOT_URL         = https://my.domain.com:4222/gitea
DISABLE_SSH      = false
SSH_PORT         = 22
LFS_CONTENT_PATH = /data/git/lfs
DOMAIN           = my.domain.com
LFS_START_SERVER = true
LFS_JWT_SECRET   = B4Y0nretretretretretretret
OFFLINE_MODE     = false

[database]
PATH     = /data/gitea/gitea.db
DB_TYPE  = sqlite3
HOST     = localhost:3306
NAME     = gitea
USER     = root
PASSWD   =
SSL_MODE = disable

[indexer]
ISSUE_INDEXER_PATH = /data/gitea/indexers/issues.bleve

[session]
PROVIDER_CONFIG = /data/gitea/sessions
PROVIDER        = file

[picture]
AVATAR_UPLOAD_PATH      = /data/gitea/avatars
DISABLE_GRAVATAR        = false
ENABLE_FEDERATED_AVATAR = true

[attachment]
PATH = /data/gitea/attachments

[log]
ROOT_PATH = /data/gitea/log
MODE      = file
LEVEL     = info

[security]
INSTALL_LOCK   = true
SECRET_KEY     = retetretrero97
INTERNAL_TOKEN = dsfsdfJoCSv0

[service]
DISABLE_REGISTRATION              = true
REQUIRE_SIGNIN_VIEW               = true
REGISTER_EMAIL_CONFIRM            = false
ENABLE_NOTIFY_MAIL                = false
ALLOW_ONLY_EXTERNAL_REGISTRATION  = false
ENABLE_CAPTCHA                    = false
DEFAULT_KEEP_EMAIL_PRIVATE        = false
DEFAULT_ALLOW_CREATE_ORGANIZATION = false
DEFAULT_ENABLE_TIMETRACKING       = true
NO_REPLY_ADDRESS                  = noreply.here.com

[oauth2]
JWT_SECRET = 4XYPNfAjTg

[mailer]
ENABLED = false

[openid]
ENABLE_OPENID_SIGNIN = true
ENABLE_OPENID_SIGNUP = false

[ui]
DEFAULT_THEME = arc-green

[metrics]
ENABLED = true

[i18n]
LANGS = en-US
NAMES = English

[i18n.datelang]
en-US = en

Other than that I haven't done anything special as far as changing anything else.

Ok, first switch your logging config to console and ramp it up to debug or even trace. Send the xorm logs to the console too with Xorm = console

@zeripath Tyvm for helping me out, I appreciate it beyond words. :)
So, I went to try to edit the app.ini to add your suggestions, and I saw that I'm getting a new error instead:

"Git module init failed: Failed to execute 'git config --global core.quotepath false': error: failed to write new configuration file /data/git/.gitconfig.lock"

After some Googling, I saw something similar in a Gogs issue thread, and one comment said he was getting the error due to running out of disk space. So I checked how much disk space was allocate to the Hyper-V machine for my Docker and it was using like 58.6/59GB. eek. So I changed from 59gb to 80 and restarted my windows machine.
I don't know whether or not my original problem is solved by that however, because I still can't even SSH into the gitea container due to the same error mentioned above. arrrgh
When I try to SSH into the container, (using Portainer) I get this:

And during that same time, doing a docker ps shows this:

I have read in some other cases, that simply deleting a .lock file can fix things like this, I don't know whether or not that would work in this case, but I can't even try it because I can't get into the container.  :/

Update:
After re-sizing my Hyper-V VM to 100 gb, a bunch of issues with docker were fixed, but I'm still having the same issue with Gitea stopping and starting repeatedly. After adding everything except the "xorm" flag to my app.ini (not sure which section to put it under?), my log now floods me with this:

The strange thing is I can still login to the Gitea page and navigate around as usual (while the container is briefly up; it goes up and down roughly every 30 seconds).
Any ideas?

You need to delete the broken queue files in your /data/queues

I deleted all of those folders, then started getting these:

Deleted THOSE folders, and now I think we're good! It's not restarting over and over anymore, and after a quick look-through it doesn't seem like I lost anything.
Thank you for the help! :D


Dead UI Link ETH about section

Overview
In the about section on the ETH account and asset page, there's a dead link pop up that appears when you hover over the underlined text, "smart contract platform."
References and additional details

go to app.shapeshift
log in with your wallet of choice
go to the ETH accounts or asset page.
go to the about section
hover over the underlined text, "smart contract platform"
noticed the dead link pop up that appears

Acceptance Criteria
Either the hyperlink should be added to the "smart contract platform" text in the about section or the dead link pop up should be removed. Not sure what the expected behavior should be here
Need By Date
No response
Screenshots/Mockups

Estimated effort
No response

@shapeshift/product Please have a look at the screenshots in the PR - This will do for now as far as fixing links goes, but we might want to iterate on this with our own in-house, improved description, as it's a bit outdated.

yeah I think we can remove the link. I'll have @decentralized-j take a pass at updating the copy there.


How to use youtube plugin in React Viewer

Summary
How to use youtube plugin in React Viewer
Version
"@toast-ui/react-editor": "^2.1.2"
Additional context
In 1.x, i use Editor.factory and set viewer value true to display content
    this.editor = new Editor.factory({
      el: document.querySelector('#viewContainer'),
      viewer: true,
      initialValue: this.state.markdown,
      exts: [
        'videoPlugin',
      ],
    })

After upgrade to 2.x, is there a recommended way to use video plugin in Viewer ?
ToastEditor.codeBlockManager.setReplacer only work in Editor

#899

#899

const videoPlugin = (editor) => {
  const { codeBlockManager } = Object.getPrototypeOf(editor).constructor
  codeBlockManager.setReplacer('youtube', function(sourceId) {
  ……
}

this way work in Editor and Viewer


Quick Sort

🚀 Feature
Quick Sort algorithm is one of the sorting algorithms used to sort elements. It is a Divide-and-Conquer algorithm. This is one of the fastest sorting algorithms.
Have you read the Contributing Guidelines on Pull Requests?
Yes.
Motivation
This is one of basic algorithm used to sort elements.

/assign


Improve acker pole placement doc

Hello,
This is a follow-up of #1004, still on pole placement, but now turning to acker docstring.
As a first step, this adds a "See Also" section to place and place varga.
However, I have two more ideas to extend this PR:

add reference to the Ackermann's formula. I've found § 2.3.5 in the "Linear Control Theory" book by F. W. Fairman, 1998, but I'm not sure it matches perfectly with what's implemented.
to the best of my (quite limited) knownledge, Ackermann's formula is for single input systems (and I got some linear algebra error on a test with a system with 2 inputs). If correct, I suggest to add a ValueError if B.shape[1] > 1.

coverage: 94.503%. remained the same
when pulling 3a06aed9c90585c9e0ddf5d0a49ce26d31ac6a7a on pierre-haessig:doc-acker
into 617809741b63daa1afacd0179678e7b53a87d06e on python-control:main.

Oups, I didn't see PR #1007 and pushed my own commit to fix place_varga doctring. Now I have to fix the conflict!

Now I have to fix the conflict!

Done!

@pierre-haessig Thanks!

Regarding the other two items in your opening post:

For widely known results like Ackermann's formula, I recommend to use an open access reference, if available. The book Feedback Systems can be used for this. Ackermann's formula is given in equation (7.21) in Chapter 7.
Ackermann's formula is only for single-input systems. This aligns with the similar acker function in Matlab. The function always raises an exception if the system has more than 1 input, but the error message may be confusing. We could add a check with ValueError as you suggest, or the documentation can add a note about single-input systems (or both). Feel free to create another pull request with it.

As an aside, I was not able to find a description of acker in the official Matlab documentation. Perhaps they removed it?

As an aside, I was not able to find a description of acker in the official Matlab documentation. Perhaps they removed it?

Neither did I. I only found an online copy of a unspecified Matlab version here: www.ece.northwestern.edu/local-apps/matlabhelp/toolbox/control/ref/acker.html (in the source code, the date 2002 is mentioned). I don't know when it was removed.


Update for latest Zig

Small updates for latest Zig updates:

https://github.com/ziglang/zig/pull/17623
https://github.com/ziglang/zig/pull/17392

Tested with zig build install and zig build test

This needs a Zig bump in flake.lock. Will work out how to do that

Thanks!


chore(utils): migrate block-nameify to TS and add unit tests

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Hey there! Can you take a moment and align the title with the contributing guidelines?

Hey there! Can you take a moment and align the title with the contributing guidelines?

Yeah sure, I wanted to ask: would this be a fix or feat ? It adds new unit tests and typings so it can be a feat, but can also be considered as a fix since it's improving existing code 🤔

it could be chore — it's migrating to typescript, not changing how it works
or it could be feat for the new tests

:sparkles: :tada: AWESOME! :tada: :sparkles:
Hi @Kroustille, thanks for this pull request and for contributing to the code-base for the first time. We are looking forward to more contributions from you in the future.
Cheers & happy contributing!


Bazel RBE Asan failure on test/core/end2end/h2_http_proxy_test@disappearing_server@poller=epoll1

https://source.cloud.google.com/results/invocations/db80efd1-4aa7-43e4-9903-1d9879263a86/targets
exited with error code 134

This looks like the same failure as #18840
Also seen in UBSAN: https://source.cloud.google.com/results/invocations/a821af9a-2b1d-4bae-ad2b-905e2d9f24c4/targets

Closing since it is a dup of #18840


fix: updated readme file

hey updated readme file check out

hey check this out having an issue regarding this PR?

@JayrajSingh65 Can you check the conflicts happening there!

@JayrajSingh65 Can you check the conflicts happening there!

Ok let me check


Hot word is not available, sample is depraced.

https://developers.google.com/assistant/sdk/guides/library/python/extend/handle-device-commands
here is depraced sample. Hot word option is not available.

Adding for those who are trying to enable hotword feature and avoid push enter to talk you can do these steps:
// upgrade google-assistant-library
$ python3 -m pip install --upgrade google-assistant-library
//Run the demo
google-assistant-demo --project-id PROJECT_ID --device-model-id DEVICE_MODEL
and then you can use "hey google or OK google" hotword, no keyentries needed.
You can also make it run at system startup which makes it great ;)

It says it's deprecated