Performance on i5 better then on latest i9

I have very basic test and very strange results.
console.time("ROTATE-RESIZE")
sharp('./photo.JPG')
  .rotate(90)
  .resize(200)
  .toBuffer()
    .then(()=> {
      console.timeEnd("ROTATE-RESIZE")
    })

On Macbook Pro Mojave ( Core i5-7267U, 8RAM, Iris VGA) execution time is ~93ms
While on Desktop Ubuntu 18.4 (Core i9-7944X, 16Ram , M2 SSD, Geforce 1050) execution time is ~121ms
I bought this monster PC to speed up resizing, but in reality I got 20%  performance decrease.
Do you have any explanation ?
Thank you.

Both running Node v10.13

test/bench/parallel.js  results
Macbook:
1 parallel calls: fastest=59ms slowest=59ms mean=59ms
1 parallel calls: fastest=20ms slowest=20ms mean=20ms
{ queue: 0, process: 2 }
2 parallel calls: fastest=32ms slowest=33ms mean=32.5ms
4 parallel calls: fastest=50ms slowest=52ms mean=51.5ms
{ queue: 0, process: 8 }
8 parallel calls: fastest=103ms slowest=96ms mean=91.125ms
{ queue: 0, process: 16 }
{ queue: 0, process: 13 }
16 parallel calls: fastest=118ms slowest=191ms mean=164.625ms
{ queue: 0, process: 32 }
{ queue: 0, process: 32 }
{ queue: 0, process: 22 }
32 parallel calls: fastest=230ms slowest=401ms mean=337.90625ms
{ queue: 4, process: 60 }
{ queue: 0, process: 64 }
{ queue: 0, process: 63 }
{ queue: 0, process: 51 }
{ queue: 0, process: 45 }
{ queue: 0, process: 26 }
64 parallel calls: fastest=192ms slowest=820ms mean=674.359375ms
{ queue: 0, process: 0 }

Ubuntu
1 parallel calls: fastest=56ms slowest=56ms mean=56ms
1 parallel calls: fastest=27ms slowest=27ms mean=27ms
{ queue: 0, process: 2 }
2 parallel calls: fastest=31ms slowest=42ms mean=36.5ms
4 parallel calls: fastest=33ms slowest=42ms mean=37.25ms
{ queue: 0, process: 8 }
8 parallel calls: fastest=40ms slowest=49ms mean=44.125ms
16 parallel calls: fastest=56ms slowest=71ms mean=63.8125ms
{ queue: 0, process: 32 }
{ queue: 0, process: 32 }
32 parallel calls: fastest=127ms slowest=159ms mean=148.40625ms
{ queue: 0, process: 64 }
{ queue: 0, process: 64 }
{ queue: 0, process: 64 }
64 parallel calls: fastest=259ms slowest=313ms mean=304.3125ms
{ queue: 0, process: 0 }

Hello, is this using the latest sharp? Are the prebuilt libvips binaries being used or could vips have been compiled/installed globally? If globally, were the same compiler flags used?
Are you able to run the full set of benchmark tests on each? http://sharp.pixelplumbing.com/en/stable/performance/#running-the-benchmark-test
The newer CPU has a higher clock speed, more cores and a larger CPU cache so I'd expect it to be a bit faster, although you may be approaching the I/O limits of libjpeg.

is this using the latest sharp?

YES v0.21

Are the prebuilt libvips binaries ...?

I simply installed latest sharp on both machines. I did not compile/install manually any libraries ( libvips , vips ). Just "npm install sharp"

Are you able to run the full set of benchmark tests on each?

No it is failing on both machines here -> "cd test/bench > npm install" with
npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild` npm ERR! Exit status 1

test/bench/parallel.js is the only test I can run

approaching the I/O limits

I'm using this SSD -> Samsung 970 Pro SSD review: The fastest M.2 NVMe drive yet

There are a few prerequisites that need to be installed for the benchmark test dependencies to compile correctly.
http://sharp.pixelplumbing.com/en/stable/performance/#benchmark-test-prerequisites
I should probably create a Docker image to make these easier to run.

False alarm. One file processing make no sense.
Once I run it on 590 different files,  each applied unique set of modifications( scale, rotate, combine, extend, extract) -  the result on Ubuntu machine was 4.1 times faster. ( 6.81min vs 1.66min)
More than happy.
Thank you for assistance and the package itself.
Do you have donations wallet for "Sharp" ?

Glad you worked it out. If you'd like to donate something, please do so to a homelessness charity local to you.


Rebuild to fix broken upload

Somehow #13 uploaded the same build number as already existed, so this is just a rebuild to correct that.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The build section contained an unexpected subsection name. error_overlinking is not a valid subsection name.
The build section contained an unexpected subsection name. error_overdepending is not a valid subsection name.


Load Software Products into Database

Do we use CSV2RDF or do load it over SPARQL?

There are 22 software products in the endpoint right now:
SELECT ?s {?s a hito:SoftwareProduct.}
s
http://hitontology.eu/ontology/Bahmni
http://hitontology.eu/ontology/GnuHealth
http://hitontology.eu/ontology/Orthanc
http://hitontology.eu/ontology/AGFA_PACS_IMPAX
http://hitontology.eu/ontology/AGFA_RIS_Elefante
http://hitontology.eu/ontology/Abbott_i-Stat
http://hitontology.eu/ontology/CSC_MedChart
http://hitontology.eu/ontology/CernerMilleniumPowerOrders
http://hitontology.eu/ontology/Cerner_FirstNet
http://hitontology.eu/ontology/Cerner_Soarian_CPOE_platform
http://hitontology.eu/ontology/Eclipsys_Continuum_2000_CDS
http://hitontology.eu/ontology/GE_Patchspecd_PACS
http://hitontology.eu/ontology/HNA_Millenium
http://hitontology.eu/ontology/Medicator
http://hitontology.eu/ontology/Metavision
http://hitontology.eu/ontology/PIK
http://hitontology.eu/ontology/SAP_R3_Business_Process_Control_System_software
http://hitontology.eu/ontology/Sysmex_XS_1000i_haematology_analyser
http://hitontology.eu/ontology/TeleDICOM
http://hitontology.eu/ontology/Theriak
http://hitontology.eu/ontology/eChasqui
http://hitontology.eu/ontology/e_Doc

Checking the first ones if they are as tables on Google Drive but there is no table for:

hna_millenium
AGFA PACS IMPAX
AGFA RIS Elefante
Abbott i-Stat
CSC Med Chart (last one checked)

errors on insert due to foreign key constraints https://github.com/hitontology/database/issues/9

Done with fb8208acbfffc1eb8f778ca6650ac923c09e937a

Citations fehlen noch:

[] fill table "citation"

ERROR:  insert or update on table "citation" violates foreign key constraint "citation_classified_suffix_fkey"
DETAIL:  Key (classified_suffix)=(SupportForCommunicationBetweenOrganizations) is not present in table "classified".
Importing swp_has_child from https://hitontology.eu/sparql
DELETE 0
INSERT 0 1
Importing swp_has_interoperabilitystandard from https://hitontology.eu/sparql
DELETE 0
INSERT 0 5
Importing swp_has_language from https://hitontology.eu/sparql
DELETE 0
INSERT 0 3
Importing swp_has_license from https://hitontology.eu/sparql
DELETE 0
ERROR:  insert or update on table "swp_has_license" violates foreign key constraint "swp_has_license_license_suffix_fkey"
DETAIL:  Key (license_suffix)=(GNU_General_Public_License) is not present in table "license".
Importing swp_has_operatingsystem from https://hitontology.eu/sparql
DELETE 0
INSERT 0 9
Importing swp_has_programminglanguage from https://hitontology.eu/sparql

[x] (GNU_General_Public_License) is not present in table "license".
https://github.com/hitontology/csv2rdf/commit/d9ea14c8656781fde2148f7afd1a500e4e463ca8
https://github.com/hitontology/csv2rdf/commit/364322bff626794e1520221ef1eb3c21fa842a7b
https://github.com/hitontology/ontology/commit/1fe3357737534fba003cb25eaf327f16ea83a361
https://github.com/hitontology/ontology/commit/a5d43cbffa2c93cba592d7dfc0bf81652605fa36
https://github.com/hitontology/ontology/commit/f040828a523d5fa1bea13a8536cda8658f57194b
https://github.com/hitontology/ontology/commit/d750ad9ef7377eab3dda229d23284261a6ab54c7

[ ] (SupportForCommunicationBetweenOrganizations) is not present in table "classified"

All errors are fixed.

[ ] fill table "classified_has_child"


[NEW ICON] InertiaJS

Skill Name
InertiaJS
Why?
InertiaJS is a popular framework for Laravel to connect it to a JS frontend without an API.
Reference Image

(This is the logo as an svg, but it is white, so you can't see it when using light mode)

Up

Hey @TheBlckbird , I've submitted a PR on the newest version of the project to add these since the maintainer hasn't shown up in months, you can check the repo here, and the PR https://github.com/LelouchFR/skill-icons/pull/96


Add a fake devshell for impure packages

This should make issues like https://github.com/nix-community/dream2nix/issues/272 less common

A default should probably also be added so when users attempt to run nix build . the warning also displays

Adding a default package allows the user to run both nix build . and  nix develop and receive a warning.

The case where impureFakeDerivations is empty still needs to be handled.

This commit also modifies how the testing of the examples works, only running a check if impurities have been resolved, without this the default check would be run which errors as designed with the new behavior.
Good chance this is the wrong way to do this, might have been no need to modify these tests.


is this project dead?

The title is pretty much self-explanatory. So are we done?
I mean it still works fine as it is but would be nice to know.

This project is not dead.
Vaadin Router will be an integral part of the future version of Vaadin framework. However, it has arrived first and has not been getting much attention lately, waiting for the other parts to come together.
Your have a very fair point though. Right now Router has a bunch of open issues and no commits since November 2018. The team has been focusing on other things with the intention to come back to building the future Vaadin–including this router–later this year when the priorities so permit.
As this is a Vaadin Labs project at the moment, we would be happy to hear any feedback you might have related to your experience with Vaadin Router. What works well for you? What is lacking?


Issue with transferring world to container.

Want to say thank you for your work! I have no issues with the setup but one thing. And that is bringing over any world that is not being generated by the container itself. After bringing over any .db .fwl files from my local to the container, I seem to always get a "version" error and that it can not successfully load the world. Only when I let the container load the world, will it then work. If I can get assistance on how to solve this, would be wonderful. Running on headless Ubuntu if that matters. Thank you!

I am having this same issue when bringing in a world file from a friend's Windows PC. I am running in Docker on Ubuntu 22.04.
Question - do we only need to transfer over the .db file from within the worlds directory? Or does it need to be all contents within the worlds folder? Also, I notice the container generates a worlds_local directory rather than using the existing config/worlds directory (containing the .db file) that was mounted into the container.

Just ran into this scenario and finally figured it out. If you manually run the docker commands, you specify the data and config volumes ($HOME/valheim-server/…). If you use the systemd service, that service is looking in /home/valheim and /opt/valheim. You’ll need to edit the service file to match the full path(s) (pwd is your friend), and then restart the service.


Additions to IPA Resource Model

[x] Add “Industry Company Name” under Project Name, under Project Details - Is this free-form text or pick list? Is it required? If free form text what is the maximum length? What is the placeholder text?

[x] To the SAP FMO Assessment, add “SAP FMO” to the term “Approval Status”

[x] To IM Process, add one more node called “Impact Mitigation Final Report (IMFR)”

[x] Under this new node, add four items:

[x]  “IMFR Number”
[x] add “Impact Mitigation Final Report” as an upload document box
[x] add “FMO Comments on Impact Mitigation Final Report”
[x] add “IMFR - FMO Approval Status”

[x] To SAP Process, add one more node called “Site Assessment Final Report (SAFR)”

[x] Under this new node, add four items:

[x]  “SAFR Number”
[x] add “Site Assessment Final Report” as an upload document box
[x] add “FMO Comments on Site Assessment Final Report”
[x] add “SAFR - FMO Approval Status”

“Industry Company Name”
-required field
-maximum length - some will be greater than 40 char, e.g. = NorthRiver Midstream NEBC Connector GP Inc.

pick-list with ability to add new as this will evolve over time - will need to make a ticket to build this
placeholder text "Name of the company responsible for the project" looks good

-no other new fields are required in case a project does not have these components

@LeannePyle  - If Industry Company Name is a controlled list, we should manage this in the Contributors resource model, with another role.
Also, we're still missing placeholder text for the text fields. Am I correct in assuming that the FMO comments should be rich text?

Agreed about Contributors, but can we discuss tomorrow
Yes to use rich text for FMO comments

At the meeting on June 26 it was decided to leave the Industry Company Name as a text field as there are not often repeat companies. There are certain circumstances that there may not be an industry company so left as non-mandatory with a maximum length of 80 characters.


The tests test_cjose_concatkdf_derive_* fail on big endian architectures like s390x, powerpc, ppc64, sparc64, ...:
check_concatkdf.c:159:F:concatkdf:test_cjose_concatkdf_derive_simple:0: Assertion 'derived==expected' failed: keylen==%, derived==0x20, expected==0x566eb950
check_concatkdf.c:190:F:concatkdf:test_cjose_concatkdf_derive_ikm:0: Assertion 'derived==expected' failed: keylen==%, derived==0x20, expected==0x566ebb80
check_concatkdf.c:224:F:concatkdf:test_cjose_concatkdf_derive_moreinfo:0: Assertion 'derived==expected' failed: keylen==%, derived==0x20, expected==0x566ebd50

Maybe this is something similar to #38 where some bad casts were the source of the problem that go unnoticed on little endian, but make it fail on big endian.
However, I did not find something at a quick glance (except from the fact that &err gets used in both function calls there, is not nulled inbetween and is not checked - maybe that would help debugging).
https://github.com/cisco/cjose/blob/1250eff10fa178937aea1924887d114c8ba943c2/test/check_concatkdf.c#L149-L150
More details and complete build logs can be found on the respective Debian Buildd page:
https://buildd.debian.org/status/package.php?p=cjose
There is also a Debian bug report tracking the issue:
https://bugs.debian.org/890907

@moschlar Re-reviewing the code with #38 in mind, the possible problem points I see are:

Applying a 32-bit value to a byte buffer (including some pointer arithmetic) https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L17
More pointer arithmetic (via function calls) https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L91
More direct pointer arithmetic while appending successive digests https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L153

As with last time, we don't have a big-endian environment to troubleshoot and test on, so any assistance with or access to such an environment would be valuable.

Applying a 32-bit value to a byte buffer (including some pointer arithmetic)

Ah yes, it seems to be called with value of anything but uint32_t:

size_t:

https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L35
https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L95

int:

https://github.com/cisco/cjose/blob/b122181f5537e928f8e78afd33f96a35bdefa67f/src/concatkdf.c#L140

I'll see if I can somehow access such an environment to test it (cc @chrism).
Could you maybe think of a simple way to fix these three occurences?

The Debian architecture builds are now working, too! :-)


Match not processed at all

Match in question: https://www.opendota.com/matches/3344000233
Shows up on profile as no hero, 48 years ago etc.

Looks fine now?

I used request a parse on the match id and it fixed it.  Not sure what caused it in the first place though.

Did you just enable public match data?

Nope, haven't touched any of my dota settings.  The match I played yesterday was processed fine, and the ones I played after it today where processed fine.  Just seems that the match data never got processed (processed as in the stage before parsing), but it had the match listed with the match id on my profile.

Hmm, if you see it again don't request a parse and let me know, I'll try to investigate the issue


when using array input as data, is there a way to indicate byte layout?

For instance, I have a JPEG array that has layout [r,g,b,r,g,b,etc] rather than [r,g,b,a,r,g,b,a,etc] and it seems to be throwing rgbquant off quite a bit. Is there a way to tell it which channels live on which indices?

there's no way to do this and i'm not sure if the extra code to account for varying layouts in every function & bitshift is worth it - it would bloat the code for sure and likely impact perf quite a bit for all use-cases. your best bet is to just do the alpha insertion ahead of time.

fair enough


Fix typo in Exchange.java JavaDoc

Thanks

Well, I have to thank you, that was blazing fast feedback :)

We love contributions at Apache Camel :-D


Feature: Add support for some Validation checks

It may not be enough to cast the incoming ENV value to an Elixir data type using Dotenvy.Transformer.to!/2: sometimes the app will break if the value is wrong.  For example: a negative number used for a database port or a random quote from Shakespeare when a subdomain was expected.  In both cases, the basic Elixir data type was correct (an integer and a string, respectively), but the app would still break because the values are unusable.  Adding validation checks would help strengthen the contract between the app and its environment.
Basic validations:

number ranges (e.g. gt (i.e. >), gte, between, etc.)
enum or in (only one of set of possible values is allowed)
regex: the (string) value must match the given regular expression
URL: the (string) value must be a valid URL (ugh... don't try a regex for this)

Rather than trying to fit this into the Dotenvy.env!/2 or Dotenvy.env!/3 functions, it might be clearer to pipe output, e.g.
import Dotenvy

config_value: env!("SOME_ENV", :string) |> match!(~r/^a-z{2, 10}$/),
x: env!("X", :string) |> in!(["a", "b", "c"]),
y: env!("SOME_ENV", :string) |> matches!(~r/^a-z{2, 10}$/),

that way it would be possible to chain together multiple rules if complex criteria exist.
If this does belong as an input to env!/2 or env!/3, that messes with the simplicity of the functions because there's no room for options when arity 3 is already taken.

I would like to be able to pass a function as the type.
Basically: def to!(str, fun) when is_function(fun, 1), do: fun.(str)

I like that idea!

https://github.com/fireproofsocks/dotenvy/pull/6


Extra next call in raw_scan

There will be always an extra next call if any key is about to be returned.
https://github.com/tikv/tikv/blob/fd24cc9e3b43f77f0dceba5bf20f7b4f9e9dae44/src/storage/raw/store.rs#L175-L192

/assign @JayiceZ


Fix to stdout_logfile_maxbytes values

ONLY suffix multipliers like “KB”, “MB”, and “GB” can be used in the value.

Thank you for the fix!


How to get a dynamic clock icon for idle_inhibitor so icon reflects true time.

Ok so my setup revolves around using idle_inhibitor as my clock icon and not on the clock itself. but my problem is that i want to make it so that. if the time is lets say 10pm then it would display as 10pm icon. instead of some random one. in both activated and not active states. i cant figure out what to do

Since its been a week and no comment so I will be closing this. 😢


Handle timeout in data receiver and amend connection closing

The origin of these changes is an attempt to find solution/workaround to a problem in an environment with Deye SUN-8K-SG04P3-EU inverter and a logger with firmware version LSW3_15_FFFF_1.0.9E when using Solarman Home Assistant integration. See my comments in the discussion in home_assistant_solarman issue #523.
This PR contains three group of changes (commits):

Add more debug logging (that was used to investigate the problem in the environment)
Add better handling of timeout and queue empty exceptions
Fix socket closing

Some of these changes might make sense also on the async side, but I have no feasible environment to test those and therefore I did not touch the async side at all.

There are linter findings (too long lines, lazy style logging text formatting, ...) @jmccrohan If you wish, I could try to address them in a separate PR.

This was my first PR for this component. How do you deal with version numberin etc. Should it be part of the PR, or done separately?

Hi,
Good job. Thank you. Looking at the comments in the said discussion, the descriptors leak seem to be caused by a combination of datalogger firmware bug / bad wi-fi connection and issue in pysolarmanv5. Adding a generic handler except Exception as e:... below the TimeoutError хандлинг should improve the situation even further. Another improvement can be added here
https://github.com/jmccrohan/pysolarmanv5/blob/fb5169eba4aff03b7cf6c9c0389ca15440f5ad8d/pysolarmanv5/pysolarmanv5.py#L344
by removing the self._reader_thr.is_alive() check and calling socket.close() in try/except block all the time on reconnect.
As for the async part, I use the aysnc version extensively and I don't see FD leaks there.

I pushed fixes. I have them running in my system atm.

I pushed fixes. I have them running in my system atm.

Six hours so far successful. One time out failure which was recovered.

There has been one failure in 24 hours. Related logs:
2024-04-18 08:05:59.690 WARNING (MainThread) [homeassistant.helpers.entity] Update of sensor.solarman_pv1_power is taking over 10 seconds
2024-04-18 08:06:04.730 DEBUG (SyncWorker_11) [custom_components.solarman.solarman] Got exception when receiving frame:
2024-04-18 08:06:04.730 DEBUG (SyncWorker_11) [custom_components.solarman.solarman] Disconnecting from server after queue empty exception
2024-04-18 08:06:04.731 INFO (SyncWorker_11) [custom_components.solarman.solarman] Disconnecting from solarman data logger <IP_ADDRESS>:8899 connected via ('<IP_ADDRESS>', 33762)
2024-04-18 08:06:24.733 WARNING (SyncWorker_11) [custom_components.solarman.solarman] Querying [3-89 (0x0003-0x0059)] failed, [1] retry attempts left
2024-04-18 08:06:24.734 INFO (SyncWorker_11) [custom_components.solarman.solarman] Connecting to solarman data logger <IP_ADDRESS>:8899
2024-04-18 08:06:39.751 DEBUG (SyncWorker_11) [custom_components.solarman.solarman] Socket creation failed: timed out
2024-04-18 08:06:39.756 DEBUG (SyncWorker_11) [custom_components.solarman.solarman] Got NoSocketAvailable exception (No socket available) at ['  File "/config/custom_components/solarman/solarman.py", line 97, in get_statistics\n    self.connect_to_server()\n', '  File "/config/custom_components/solarman/solarman.py", line 43, in connect_to_server\n    self._modbus = PySolarmanV5(self._host, self._serial, port=self._port, mb_slave_id=self._mb_slaveid, logger=log, auto_reconnect=True, socket_timeout=15)\n                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^\n', '  File "/usr/local/lib/python3.12/site-packages/pysolarmanv5/pysolarmanv5.py", line 120, in __init__\n    self._socket_setup(kwargs.get("socket"), kwargs.get("auto_reconnect", False))\n', '  File "/usr/local/lib/python3.12/site-packages/pysolarmanv5/pysolarmanv5.py", line 446, in _socket_setup\n    raise NoSocketAvailableError("No socket available")\n']
2024-04-18 08:06:39.756 WARNING (SyncWorker_11) [custom_components.solarman.solarman] Querying [3-89 (0x0003-0x0059)] failed, [0] retry attempts left
2024-04-18 08:06:39.756 WARNING (SyncWorker_11) [custom_components.solarman.solarman] Querying registers [3-89 (0x0003-0x0059)] failed, aborting.

In the next update round in 10 min connection succeeds again normally.

Thanks @sofkaski for identifying this and for creating the PR. Happy to accept any linter fixes as a separate PR.
@githubDante thanks for reviewing.
Merged.


FAQ page is fugly

One thing we'd discussed is adding a left-hand side menu with shorter question titles derived from a new front matter field (or possibly the filename though that might not be as nice)


Expose parse_s3_url() directly

I find this function extremely useful, but a little tricky to use without an instantiated S3Hook. This exposes parse_s3_url as a staticmethod and gently deprecates the leading underscore.

:+1:  Thanks for the PR.


Add sequential sketch merging to MSQ

Current Implementation
In the current MSQ implementation, each worker maintains a ClusterByStatisticsCollector. If we are clustering by time, one sketch is maintained for each time chunk. During the merge, we sent the entire ClusterByStatisticsCollector from the worker to the controller. These are merged together, downsampling if memory taken is too much and partitions are generated from this.
Potential Improvements
Taking all worker sketches into controller memory is an unneeded step, which leads to downsampling of sketches and generating partitions outside the targeted weight. Since we have a primary partitioning on time, we only need to maintain the sketch from all workers for a particular time partition, generate the partition and remove the sketches. This is the sequential sketch merging approach.
Since this increases time taken however, this has been moved to a separate WorkerSketchFetcher. This will use an executor service to avoid blocking the main controller thread. This will not remove the current functionality, it will only uses sequential merging under cases that it is likely to provide greater benefits. For small sketches, it continues to fetch the entire sketch for the speed.
Key changes

Add a new phase to controller: MERGING_STATISTICS and necessary transitions
Change worker to now send a WorkerReport to the controller, which contains information necessary to fetch sketches later.
Change the sketch fetching a pull from the controller instead of a push from the worker.
Add WorkerSketchFetcher to fetch sketches in the background.

Context changes

Add a new parameter clusterStatisticsMergeMode. It controls whether to parallel or sequential merging of worker sketches. Can be PARALLEL, SEQUENTIAL or AUTO. On AUTO tries to find the best approach based on number of workers and size of input rows.

Release note

Removes an API on ControllerChatHandler /keyStatistics/{queryId}/{stageNumber}/{workerNumber}
Adds an API on ControllerChatHandler /partialKeyStatisticsInformation/{queryId}/{stageNumber}/{workerNumber} that accepts an object that is deserializable to PartialKeyStatisticsInformation.

Key changed/added classes in this PR

ClusterByStatisticsCollector
WorkerClient
ControllerClient
StageDefinition
WorkerSketchFetcher
ControllerStageTracker
ClusterByStatisticsWorkerReport

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Thanks for the contribution @adarshsanjeev.


How to fetch data from json local file

Please guide me how to use this framework and add a function where instead of using server processing, source data as json local file instead using fetch method without the server processing. thanks

This is currently not supported.


Specify jsbeautify settings via the Visual Studio Code settings

Operating System (+Version): OSX 10.11.6
VS Code Version: 1.9.1
beautify Version: 0.8.0

Provide the settings you use:
(VS Code workspace and user settings, .jsbeautifyrc, .editorconfig)
eg.:
VS Code:
// Place your settings in this file to overwrite the default settings
{
    "vsicons.projectDetection.disableDetect": true,
    "editor.scrollBeyondLastLine": false,
    "editor.rulers": [
        100
    ],
    "files.insertFinalNewline": true,
    "npm-intellisense.scanDevDependencies": true,
    "npm-intellisense.importLinebreak": "",
    "auto-close-tag.activationOnLanguage": [
        "html",
        "xml",
        "php",
        "javascriptreact",
        "vue"
    ]
}

.jsbeautifyrc

None

Is it possible to specify actual jsbeautify settings (as they would be within .jsbeautifyrc) but actually in the Visual Studio Code settings.json for this extension? If it is I can't seem to get it to work. If this option doesn't exist would it be possible to add that? I almost feel like beautify.config setting could be either a string for a path or an object for actual config settings, so for example:
{
  "js": {
    "indent_size": 5
  },
  "css": {
    "indent_size": 4
  },
  "html": {
    "indent_size": 3
  },
  "eol": "\r\n",
  "indent_with_tabs": false,
  "indent_size": 2
}

I am not sure but from quickly looking at the code it seems like it may not be too much work. Just adding additional check for config detection and then probably refactoring some of the setup code once we have config settings?
Any thoughts? I can try and do a PR if this is acceptable for the design / vision of the extension.
Thanks.

Good point. And the new beautify.config setting definitely makes this easier. Perhaps @IgorNovozhilov would like to have a crack at this, it may even be a better option for the team sharing functionally you were after?
I'm happy to implement this, but probably won't have time in the next few weeks.

Do you like it #104 :smile:

By the way, I'm more comfortable file .jsbeautifyrc, so as it can be used in different IDE.
Settings inside the VSCode, can only be used for the VSCode.

But, the idea is to support greater flexibility, I like it :+1:

PR applied and version 0.8.1 published.

Anyone have an example of this working? I tried to set properties other than indent_size but they don't seem to take.
This is what I have in my settings file...
"beautify.options": {         "indent_size": 4,         "html": {             "end_with_newline": true,             "js": {                 "indent_size": 2             },             "css": {                 "indent_size": 2             }         },         "css": {             "indent_size": 4,             "newline_between_rules": true,             "end_with_newline": true,             "selector_separator_newline": true         },         "js": {             "preserve-newlines": true         }     }


React Native NativeEventEmitter is not working in Release apk

Description
I'm using the react native's event emitter (NativeEventEmitter) to send events from native to JS. This thing is working absolutely fine on the DEBUG version. I'm able to send events with data from the native side to JS, but this is not working in the final RELEASE APK!
Basically, I'm trying to send a String from the native side to JS
React Native version:
System:
OS: Linux 5.4 Ubuntu 20.04.1 LTS (Focal Fossa)
CPU: (8) x64 Intel(R) Core(TM) i7-8665U CPU @ 1.90GHz
Memory: 816.23 MB / 15.27 GB
Shell: 5.8 - /usr/bin/zsh
Binaries:
Node: 10.14.2 - ~/.asdf/shims/node
Yarn: Not Found
npm: 6.4.1 - ~/.asdf/shims/npm
Watchman: Not Found
SDKs:
Android SDK:
API Levels: 28, 29, 30
Build Tools: 28.0.3, 29.0.2, 30.0.2
System Images: android-30 | Google APIs Intel x86 Atom
Android NDK: Not Found
IDEs:
Android Studio: Not Found
Languages:
Java: 1.8.0_275 - /usr/bin/javac
Python: Not Found
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.3 => 0.63.3
npmGlobalPackages:
react-native: Not Found
phone: Samsung a71
model numbe: SM-A715F
Steps To Reproduce

Send an event with some data from the native side using event emitter.
Add listener in JS side and try to fetch the results from this native event.
create Release APK and inatall it
Works fine in DEBUG and RELEASE build variants, but now try generating an APK(RELEASE)
We don't get events data in release APK!

Expected Results
I'm expecting that my listener at the JS side should listen to the event and get data!
Snack, code example, screenshot, or link to a repository:
JAVA:
`public void sendName() {
WritableMap resultData = new WritableNativeMap();
resultData.putString("name", "Example Name");
sendEvent("EventName_Name",resultData);
}
public void sendEvent(String eventName, @Nullable WritableMap params) {
getReactApplicationContext().getJSModule(RCTNativeAppEventEmitter.class).emit(eventName, params);
}`

it doesn't matter if I send with params or null

JS:
`import {NativeEventEmitter, NativeModules} from 'react-native';
const nameModule = NativeModules.RNNameModule;
const nameManagerEmitter = new NativeEventEmitter(nameModule);
nameManagerEmitter.addListener("EventName_Name", (data) => { console.log(data) });`
the addListener noes not "listen" to the emit action in release APK.

You must use RCTDeviceEventEmitter.class, instead of RCTNativeAppEventEmitter.class. Please see https://reactnative.dev/docs/native-modules-android#sending-events-to-javascript.

You must use RCTDeviceEventEmitter.class, instead of RCTNativeAppEventEmitter.class. Please see https://reactnative.dev/docs/native-modules-android#sending-events-to-javascript.


[layer] Bug fixes for layers

This patch adds bug fixes for layers:

bn layer support in-place is corrected.
flatten layer support in-place is corrected.
pooling layer padding related corrections.

Signed-off-by: Parichay Kapoor<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1577. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.


Tab Title Text not displaying

I followed the guide for version 2.0, and every works besides the text and padding of the icon. The icon for each tab is too large and no title text is being displayed at all. The icons I used followed the standards in the documentation.

I'm having the same issue.
Logcat shows this message:
requestLayout() improperly called by android.support.v7.widget.AppCompatTextView{ac20783 V.ED..... ......ID 44,138-171,168 #7f0f0086 app:id/bb_bottom_bar_title} during layout: running second layout pass

Having this issue also. You guys fixed it?

Could you provide some sample code on how you have the tabs and BottomBar setup?

I have also not displayed under the buttons.
<com.roughike.bottombar.BottomBar         android:id="@+id/bottomBar"         android:layout_width="match_parent"         android:layout_height="60dp"         android:layout_alignParentBottom="true"         app:bb_tabXmlResource="@xml/bottombar_tabs" />
<com.roughike.bottombar.BottomBar         android:id="@+id/bottomBar"         android:layout_width="match_parent"         android:layout_height="60dp"         android:layout_alignParentBottom="true"         app:bb_tabXmlResource="@xml/bottombar_tabs" />

any idea when this bug could be fixed?

Facing same issue. The activity layout file is as follows:

<?xml version="1.0" encoding="utf-8"?>
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    tools:context="com.ms.droid.yipyap.activity.MainActivity">
    <android.support.v7.widget.Toolbar
        android:id="@+id/toolbar"
        android:layout_width="match_parent"
        android:layout_height="?attr/actionBarSize"
        android:background="?attr/colorPrimary"
        android:elevation="4dp"
<EMAIL_ADDRESS><EMAIL_ADDRESS>/>

    <FrameLayout
        android:id="@+id/frame_container"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        android:layout_below="@id/toolbar"
        android:layout_above="@+id/bottomBar" >
    </FrameLayout>

    <com.roughike.bottombar.BottomBar
        android:id="@+id/bottomBar"
        android:layout_width="match_parent"
        android:layout_height="60dp"
        android:layout_alignParentBottom="true"
        app:bb_tabXmlResource="@xml/bottombar_tabs" />

</RelativeLayout>

Tabs XML:
<tabs>
    <tab
        id="@+id/tab_home"
        icon="@drawable/ic_home"
        title="Home" />
    <tab
        id="@+id/tab_chat"
        icon="@drawable/ic_chat"
        title="Chats" />
    <tab
        id="@+id/tab_yipyap"
        icon="@drawable/ic_yipyap"
        title="My YipYaps" />
</tabs>

BottomBar injected in the view:
    @BindView(R.id.bottomBar)
    BottomBar bottomBar;

Creating icons through  Android asset studio generator solves the issue. Using Android Studio Asset Importer is not working.

me too not working with android asset studio generator

confirming the same issue

face the same problem, just use Android Asset Studio and make sure choose two paddings to 0 (there are two paddings to choose,please see it clearly)

@ZXIAOY is right, setting the two paddings to 0 worked for me. The text is now visible.

@ZXIAOY your solution did worked, but still the icons are too small, and i need to enlarge them to match the design, if there's any way to custom the library itself to fit larger icons?

in this condition image height should not be greater than 32dp.


Distinct method is not documented

How you guys would expect a distinct to be written?
I can't see any viable method and the only solution I can think of is a small hack to parse the returned sql and split immediately after the SELECT statement.
Not elegant, but it can work I guess.
Any idea is welcome

Query.from_("abc").select("foo").distinct()
'SELECT DISTINCT "foo" FROM "abc"'

Worth updating the main guide then, because I based my comment from that.

Hmm makes sense, I'll leave the ticket open with a documentation label.

@gl3nn I created a new README.rst for this issue but couldn't push my changes to a branch I made (403 error).  I was able to authenticate with Gitub using my browser credentials so not sure of the issue.  ANy ideas?


Liceul Teoretic Grigore Moisil Timisoara  -  Grigore Moisil Highschool Timisoara

@Gabriel1961 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

URL: http://info.tm.edu.ro/
Location: http://info.tm.edu.ro/contact/
Address: Str. Ghirlandei Nr.4, Timişoara, Timiș Romania
On Sun, Oct 16, 2022, 13:28 Philip Torchinsky @.***>
wrote:

@Gabriel1961 https://github.com/Gabriel1961 Please provide us with:

the school official website URL
the school street address, including city and country
an URL of a page on the official website where the school offers an
IT-related long-term (>1 year) course
a proof, which shows that the school recognizes the domain you are
submitting as an official email domain for the students.

—
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/15356#issuecomment-1279940659,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ASCY75B3QTNTTHE2HDT6KIDWDPKGLANCNFSM6AAAAAARBRD6KU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@Gabriel1961  Thank you for the clarifications. Pull request merged.


opentracing-spring-cloud-starter 0.2.0

You  versions 0.x.y are meant to be used with Spring Boot 1.5，but opentracing-spring-jaeger-cloud-starter 0.2.2 is use opentracing-spring-cloud-starter 0.2.0，opentracing-spring-cloud-starter 0.2.0 is use Spring Boot 2.x ！！！！！！！

What do you think? Change the document

Please mark Versions 1.x.y of the library are meant to target Spring Boot 2.x while versions 0.x.y are meant to be used with Spring Boot 1.5， But 0.2.2 is use  opentracing-spring-cloud-starter 0.2.0 is use Spring Boot 2.x  ！！！！！！

This is definitely something that we should do, thanks for pointing that out @wintermin.
Would you like to submit a PR for that? If you don't have the time, I'll probably get to it in a few days

@wintermin same question will happen to opentracing-spring-jaeger-web-starter?


Inconsistent behavior in evaluation of factory definitions

Laravel Version: 8.17.2
PHP Version: 7.4.9
Database Driver & Version: not related.

Description:
According to the documentation of the factories:

If the relationship's columns depend on the factory that defines it you may provide a callback which accepts the evaluated attribute array:
public function definition()
{
    return [
        'user_id' => User::factory(),
        'user_type' => function (array $attributes) {
            return User::find($attributes['user_id'])->type;
        },
        'title' => $this->faker->title,
        'content' => $this->faker->paragraph,
    ];
}

That's true only key of the relationship is before the dependent attribute. for example:
   return [
         'user_id' => User::factory(),
         'content' => function(array $attrs) {
               return "A POST FOR USER" . $attrs['user_id'] ;
         },
     ];

in this example it'll work and $attrs['user_id'] will be some kind of a key (some integer for example).
but if we reverse the order of the keys:
   return [
          'content' => function(array $attrs) {
               return "A POST FOR USER" . $attrs['user_id'];
         },
         'user_id' => User::factory(),
     ];

here while running the 'content' closure, $attrs['user_id'] will have the value of User::factory(). this is not consistent and probably will cause trouble to developers just like it caused to me.
If the behaviour is intended I think this should go in the documentation. OR it can be repaired by expanding the attributes Factory.php#L407 on two passes, the first pass evaluates the relationships like in Factory.php#L413 then other types of attributes will be evaluated in the second pass.

If you want to improve this behavior feel free to send in a PR.


ID Claim 055: Random Mission Selection

Mod ID: 055
Mod Name: Random Mission Selection
Mod Author: Darth Pointer

Added


[BUG][cert-csi] :  "--attr" of ephemeral-volume performance test doesn't support properties file

Bug Description
when use attributes file to "--attr" of ephemeral-volume performance test , I see the script panic.   but same function test of "--attr" support attributes file.  and the doc also show attributes file as example. it didn't  note user that performance test can't use it.
to run performance,  there only have to attach key/value to "--attr"
./cert-csi test ephemeral-volume  --driver csi-powerstore.dellemc.com --fs-type ext4 --pods 5 --pod-name wrephe --attr arrayId=PS6096ea15a0b0 --attr protocol=ISCSI --attr size=20Gi  --driver-ns csi-powerstore  -ns csi-powerstore
[2023-08-03 10:28:26]  INFO Starting cert-csi; ver. 0.8.1
[2023-08-03 10:28:26]  INFO Using EVENT observer type
[2023-08-03 10:28:26]  INFO Using default config
[2023-08-03 10:28:26]  INFO Successfully loaded config. Host: https://<IP_ADDRESS>:6443
[2023-08-03 10:28:26]  INFO Created new KubeClient
[2023-08-03 10:28:26]  INFO Creating 5 pods, each with 1 volumes
... ...
function test:
cat ephemeral-config.properties
arrayId=PS6096ea15a0b0
protocol=ISCSI
size=20Gi
[root@registry ~]# ./cert-csi functional-test ephemeral-volume --driver csi-powerstore.dellemc.com --fs-type ext4 --attr ephemeral-config.properties --pods 5 --pod-name wrephe
[2023-08-03 09:55:08]  INFO Starting cert-csi; ver. 0.8.1
[2023-08-03 09:55:08]  INFO Using EVENT observer type
[2023-08-03 09:55:08]  INFO Using default config
[2023-08-03 09:55:08]  INFO Successfully loaded config. Host: https://<IP_ADDRESS>:6443
[2023-08-03 09:55:09]  INFO Created new KubeClient
[2023-08-03 09:55:09]  INFO Creating 5 pods, each with 1 volumes
... ...
Logs
]# ./cert-csi test ephemeral-volume  --driver csi-powerstore.dellemc.com --fs-type ext4 --pods 5 --pod-name wrephe --attr ephemeral-config.properties  --driver-ns csi-powerstore  -ns csi-powerstore
[2023-08-03 10:04:11]  INFO Starting cert-csi; ver. 0.8.1
[2023-08-03 10:04:11] ERROR Parameter ephemeral-config.properties incorrect.
panic: runtime error: index out of range [1] with length 1
goroutine 1 [running]:
cert-csi/pkg/cmd.getEphemeralCreationCommand.func1(0xc0000f4dc0)
/root/cert-csi/pkg/cmd/testcmd.go:960 +0x49c
github.com/urfave/cli.HandleAction({0x2143c00?, 0x26746f0?}, 0x10?)
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/app.go:523 +0x50
github.com/urfave/cli.Command.Run({{0x25548a8, 0x10}, {0x254546e, 0x9}, {0x0, 0x0, 0x0}, {0x2565c09, 0x18}, {0x0, ...}, ...}, ...)
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/command.go:174 +0x67b
github.com/urfave/cli.(*App).RunAsSubcommand(0xc0004de540, 0xc0000f4b00)
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/app.go:404 +0xe87
github.com/urfave/cli.Command.startApp({{0x2539a1a, 0x4}, {0x0, 0x0}, {0x0, 0x0, 0x0}, {0x2552c8e, 0xf}, {0x0, ...}, ...}, ...)
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/command.go:373 +0xb7f
github.com/urfave/cli.Command.Run({{0x2539a1a, 0x4}, {0x0, 0x0}, {0x0, 0x0, 0x0}, {0x2552c8e, 0xf}, {0x0, ...}, ...}, ...)
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/command.go:102 +0x865
github.com/urfave/cli.(*App).Run(0xc0004de1c0, {0xc000130000, 0x11, 0x12})
/root/go/pkg/mod/github.com/urfave/cli@v1.22.2/app.go:276 +0xbc7
main.main()
/root/cert-csi/cmd/cert-csi/main.go:95 +0xa85
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
create attributes file firstly and use it in ephemeral-volume performance test
./cert-csi test ephemeral-volume  --driver csi-powerstore.dellemc.com --fs-type ext4 --pods 5 --pod-name wrephe --attr ephemeral-config.properties  --driver-ns csi-powerstore  -ns csi-powerstore
Expected Behavior
the performance test of ephemeral-volume case could support attribute file same as function test.
CSM Driver(s)
CSI powerstore
Installation Type
helm 3.0
Container Storage Modules Enabled
No response
Container Orchestrator
K8s 1.24.4
Operating System
debian 11

Hi @vincent1chen , Thanks for reporting this issue.
Based on the documentation, it appears that only the "functional-test" command accepts a file name as an argument for the "attribute" attribute.

./cert-csi functional-test ephemeral-volume --driver csi-powerstore.dellemc.com --fs-type NFS --pods 5 --pod-name wrephe --attr  ephemeral-config.properties  -ns csi-powerstore
I have successfully executed the above commands, and it seems like they also worked for you, as you mentioned in your initial statement.
Now, regarding the command you provided, I tested it out, and I encountered the same issue you mentioned. After investigation, I found that the "cert-csi" tool expects users to pass arguments in the exact format you specified, and it seems you have correctly identified the root cause of the issue as the test suite is not expecting user to pass the file name.
Before proceeding with fixing this problem, I'd like to clarify your requirement:
Do you want to pass the file name as "--attr" argument only, or do you also want to support both formats?
Since I need to assess the consequences of accommodating both formats in the "certify" command, I would like to understand the potential impact of this change.

@adarsh-dell ,  i just realize both test case should support same argument format. otherwise it's easy to confuse user.  regarding to the format,   I am ok for both.  if there has problem to code it,  I prefer to file name.

Hi @vincent1chen We appreciate your input.
After consulting with others, we've determined that using the "fileName" option for both the "test" and "function-test" commands is more straightforward. Therefore, we're actively working on a solution to maintain consistent argument formatting across both suites.
Regards,
Adarsh

Hi @vincent1chen I hope this is what you are exaclty looking for?
./cert-csi test ephemeral-volume --driver csi-powerstore.dellem          c.com --fs-type ext4 --attr attr-file --pods 5 --pod-name wrephe

@adarsh-dell , yes that is it.  you fix it:-) thanks

Closing since the fix is as per the expectations and will be available from CSM v1.8.0 release.


Release 0.1.0 to crates.io

Do an initial release! :ship:

https://crates.io/crates/stm32f7xx-hal
v0.1.0 is out! :tada:


Add the option to start Xwayland only when a client connects

Fixes #849

Yeah, GTK apps seem to always connect to the X socket, even if they could use wayland (and GDK_BACKEND=wayland is set).

The xwayland_ready listener seems to have existed because previously the DISPLAY envvar was only set after Xwayland was started.

I wonder why we had a xwayland_ready listener in the first place, everything seems to work correctly without it…

In case you wanted to wait to run startup commands until after X was ready

Addressed the comments. I will make a PR for sway.

Thanks!


Add useGamepad hook

This PR sets up a useGamepad hook that provides both a raw and mapped gamepad. This was a first pass at an idea for the API, so feedback and ideas would be appreciated!
Docs page: https://threlte-git-fork-michealparks-usegamepad-threlte.vercel.app/docs/reference/extras/use-gamepad
Right now this only supports mapping for classic gamepads, if we end up liking this design then as a follow up I'll add the standard-xr mapping to this hook and remove the @threlte/xr useGamepad hook since this'll cover it much better :)
Resolves https://github.com/threlte/threlte/issues/605

Can we make { mapping: 'none' } the default and rename mapped? If you see mapped` out of context, it's maybe a bit too generic.

Good point, and agreed. Any thoughts on a good name? Maybe: mappedGamepad or standardGamepad?

What do the touch events react to?

These are really cool. Some controllers will emit data if a button is pressed slightly, or "touched". If I recall correctly the quest 2 controllers support this and you can observe just by touching and not pressing the a or b buttons. Others like the PS5 controller will report this for the triggers if they're only slightly pulled.

Do you think it's a good idea to implement "change" events for the axes?

Yes! Great idea. I'll add this 👍

Do you think it makes sense to implement catch-all events?

I also think this makes sense. My intuition would be to keep both and to normalize the event object:
gamepad.on('press', (e) => {
  e.type // 'press'
  e.button // 'clusterBottom'
})

gamepad.leftTrigger.on('down', (e) => {
  e.type // 'press'
  e.button // 'leftTrigger'
})

Think this makes sense?

@grischaerbe I just finished changes based on feedback; I definitely like this version more 🎉 let me know what you think!

@grischaerbe updates completed! Unless you notice anything, I think this should be ready!


Recent Client Update Breaks Theme

hi its me again the new discord canary / development update literally broke everything

I'll have to wait before I do anything. Have to wait and make sure Discord won't change every class again as well as wait for the same change to be pushed to other builds.

I came here to see what was going on, thanks for keeping this theme maintained, it's very much appreciated.

mario likes this theme

I'm going to wait a bit before doing anything, just to be sure that Discord doesn't break multiple things again where I have to update almost the entire theme again. Also, only canary users are affect currently, which probably isn't a majority of users. I would like to wait til these changes are out on other builds as well.

all good! thanks!

from the Powercord Discord channel:

Many of my plugins and themes are broken
Discord released an update that broke many plugins and re-rolled many class names, which broke many themes. The solution is to wait for an update from the plugin and theme devs.

New update broke the theme on Stable and PTB too it seems

works on stable for me

stable is broken on my end as well

reeeeeee
I'll get to updating it sometime this week

UPDATE: I'm currently in the process of fixing it, it'll either be ready by later today or tomorrow.

im going to try to fix some stuff in about an hour and a half, ill update if I can get it working 100%

Feel free to, but I planned on changing some things (since I'm redoing the entire theme).

im going to try to fix some stuff in about an hour and a half, ill update if I can get it working 100%

Feel free to, but I planned on changing some things (since I'm redoing the entire theme).

oh god
well I'm excited to see the technical v3 of it

It'll be labeled as v3 yea, but don't expect too much to change besides minor color adjustments.

if you're desperate for a working amoledcord and can't wait for v3 just git clone https://github.com/RadNotRed/amoled-cord and it should work

It'll be labeled as v3 yea, but don't expect too much to change besides minor color adjustments.
Also, all the color issues you've reported will be fixed in this version.

sounds good , thanks

broken on stable release now. rip

broken on stable release now. rip

wait for v3 should be done soon

@LuckFire Idk how much you plan on changing in said V3 but I have updated the theme to the new classes, want me to make a PR or are you already working on it?

@LuckFire Idk how much you plan on changing in said V3 but I have updated the theme to the new classes, want me to make a PR or are you already working on it?

could u upload it thanks

could u upload it thanks

I have a fork of the theme

UPDATE: I'm currently in the process of fixing it, it'll either be ready by later today or tomorrow.

Thank you very much, your work is greatly appreciated! <3

@LuckFire Idk how much you plan on changing in said V3 but I have updated the theme to the new classes, want me to make a PR or are you already working on it?

Appreciate it, but I'm already working on it and almost have it done. I apologize for taking so long, since this is something I do whenever I have motivation to actually do it. I'll have the stuff pushed out by today.

@LuckFire Idk how much you plan on changing in said V3 but I have updated the theme to the new classes, want me to make a PR or are you already working on it?

Appreciate it, but I'm already working on it and almost have it done. I apologize for taking so long, since this is something I do whenever I have motivation to actually do it. I'll have the stuff pushed out by today.

ur good bro, take your time

pushed, give it a bit to update.

how do you install it for better_discord  ?
do i have to compile it myself ?

how do you install it for better_discord ? do i have to compile it myself ?

it should automatically just update if you have it, unless you have a version older than 2.0.0, then you can download it here

😄 Looks good. Everything seems to be spot on.

The theme colors everything properly but when I use the "Hide member list" custom CSS with the theme -- the member list when hovered over becomes transparent.

/* Hide member list */
:root {
--hover-animation-speed: 0.35s;
--seperator-color: var(--seperator-color);
}
.membersWrap-3NUR2t {
min-width: unset;
width: 50px;
justify-content: right;
}
.members-3WRCEx {transition: transform var(--hover-animation-speed);}
.membersWrap-3NUR2t:hover .members-3WRCEx {transform: translateX(-190px);}
.membersWrap-3NUR2t .membersGroup-2eiWxl::after {
content: '';
width: 30px;
height: 2px;
border-radius: 50px;
background: unset;
background-color: var(--seperator-color);
position: absolute;
left: -36px;
top: 29px;
margin-left: 0px;
}
.membersWrap-3NUR2t .members-3WRCEx .member-2gU6Ar {
background-color: unset;
margin-left: 0px;
transition: margin-left var(--hover-animation-speed);
}
.membersWrap-3NUR2t:hover .members-3WRCEx .member-2gU6Ar {margin-left: 8px;}
.membersWrap-3NUR2t:hover .membersGroup-2eiWxl {transform: translateX(-6px);}
.membersWrap-3NUR2t .membersGroup-2eiWxl {
cursor: default;
transform: translateX(45px);
transition: transform var(--hover-animation-speed);
overflow: visible;
}

as you can see, there are still some things that have not been changed. I appreciate you for maintaining this still, it is a must have addition.

thread closed. pls make your own issues instead of spamming my inbox :(

replying to this so you all know that i've seen these issues, and ill push something out soon to fix and adjust things

yo @LuckFire how long will it take to fix this broken theme?

yo @LuckFire how long will it take to fix this broken theme?

it is quite literally fixed, redownload if it's not working still.

yo @LuckFire how long will it take to fix this broken theme?

it is quite literally fixed, redownload if it's not working still.

thank man redownloaded it and its fixed

@LuckFire can you add me on discord? My discord tag is  Jay Mehta#9292
i have a suggestion for you if you like then add it

STOP THIS THREAD IS CLOSED

@LuckFire can you add me on discord? My discord tag is Jay Mehta#9292
i have a suggestion for you if you like then add it

I suggest creating another issue or just dming me on Discord, my DMs are open if you're in the powercord or bd server. Im closing this now to prevent further replies.


WIP: api: add endpoint for accepting a document

cc @joshsadler here's what I was thinking of for an endpoint to accept a document. Thoughts?
Demo: https://editor.swagger.io/?url=https://raw.githubusercontent.com/adamdecaf/customers/document-accept-route/openapi.yaml

Codecov Report

Merging #90 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #90   +/-   ##
=======================================
  Coverage   60.88%   60.88%
=======================================
  Files          31       31
  Lines        1828     1828
=======================================
  Hits         1113     1113
  Misses        508      508
  Partials      207      207

@adamdecaf this looks good to me. The only thing that I got caught on was whether a PUT should require a request body. As Long as the service handles no content length, it should be fine!


Prepare release 20.30.2

Summary of changes
Checklist

[x] Relevant changes are reflected in CHANGES.rst
[ ] Added or changed code is covered by tests
[ ] Documentation has been updated if necessary
[ ] Changed code does not contain any breaking changes (or this is a major version change)

Wrong release version due to a typo :x


Can we use HttpClient Outgoing Middleware with this?

I want to use outgoing middleware
It would be nice to be able to provide my own HttpClient and attach my own middleware.

Looks like adding my own IAlphaVantageService is the best option?

Yes, implementing your own AlphaVantageService is the best. Bit of a pain though that BuildUrl() lives in AlphaVantageService. I might look at moving that somewhere more re-usable.

That would be very helpful if possible


[Change request to #3274]

Description of a Change request:
Remove duplicate save place check box from more options filter component
Reason for Change:
We don't need more than one saved places checkbox
Acceptance Criteria

duplicate save place check box is removed


initial (very simple) PR template

... encouraging contributors to write at least one sentence on why/what changes are being made.
Related to Netflix/titus-control-plane#298

Pull Request Test Coverage Report for Build 1342

0 of 0   changed or added relevant lines in 0 files are covered.
348 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.2%) to 22.925%

Files with Coverage Reduction
New Missed Lines
%

api/netflix/titus/agent.pb.go
1
17.55%

executor/drivers/titusdriver.go
2
62.5%

executor/mock/jobrunner.go
37
0.0%

executor/runner/runner.go
87
54.87%

executor/runtime/docker/docker.go
221
2.24%

Totals

Change from base Build 1338:
-0.2%

Covered Lines:
2058

Relevant Lines:
8977

💛  - Coveralls


Add the Ability to Specify the Service Name/Version in the Debug Config

This PR contains an update where the new start() method can be called on the object returned from require('@google/cloud-debug) to start the debugger while specifying a configuration.
Additionally, the configuration can now specify custom values for the app's service name and service version.
This PR replaces PR #166

@GoogleCloudPlatform/node-team PTAL
This PR replaces PR #166, which I closed to avoid confusion.

@GoogleCloudPlatform/node-team
Everybody on @GoogleCloudPlatform/node-team is watching the cloud-debug-nodejs and cloud-trace-nodejs repositories already, and therefore gets notifications. Is it okay with y'all if we cut back on using @GoogleCloudPlatform/node-team to explicitly re-notify everybody?

Could you update the title to reflect that this introduces the start function/configuration flow as well?


VS 2022?

Any Plans for VS 2022 support?

Yes it is planned and work has started!

Grab the new release! 🚀

thx


Break some JS circular dependencies.

Talk to @showell or @timabbott before working on this ticket.
This is a list of dependencies we can try to break in our JS code.
ping @tommyip
activity.js
    activity.process_loaded_messages
        message_fetch.js

compose.js
    compose.stream_name
        composebox_typeahead.js

    compose.send_message_success
        echo.js

    compose.finish
        composebox_typeahead.js

    compose.transmit_message
        echo.js

    compose.mark_rendered_content_disparity
        echo.js

    compose.empty_topic_placeholder
        message_edit.js
        message_store.js

    compose.snapshot_message
        drafts.js

    compose.autosize_textarea
        drafts.js

    compose.report_as_received
        echo.js
        message_util.js

echo.js
    echo.apply_markdown
        drafts.js

hashchange.js
    hashchange.exit_modal
        subs.js

    hashchange.save_narrow
        narrow.js

    hashchange.operators_to_hash
        narrow.js

message_edit.js
    message_edit.get_editability
        message_list.js

message_events.js
    message_events.insert_new_messages
        echo.js

modals.js
    modals.close_modal
        drafts.js

narrow.js
    narrow.topic
        topic_list.js

    narrow.activate
        topic_list.js

    narrow.show_empty_narrow_message
        message_fetch.js

    narrow.stream
        navigate.js

    narrow.huddle_with_uri
        activity.js

    narrow.narrowed_to_pms
        typing.js

    narrow.operators
        typing.js

    narrow.by_stream_uri
        stream_data.js

    narrow.filter
        pm_list.js

    narrow.by_subject
        notifications.js

    narrow.public_operators
        message_fetch.js

    narrow.active
        echo.js
        message_fetch.js
        pm_list.js
        unread.js

    narrow.by_stream_subject_uri
        topic_list.js

    narrow.pm_with_uri
        activity.js
        pm_list.js

    narrow.by
        activity.js
        navigate.js

navigate.js
    navigate.scroll_to_selected
        resize.js

notifications.js
    notifications.update_pm_count
        unread_ui.js

    notifications.close_notification
        unread_ops.js

    notifications.update_title_count
        unread_ui.js

    notifications.redraw_title
        narrow.js

    notifications.window_has_focus
        unread_ops.js

    notifications.clear_compose_notifications
        compose.js
        narrow.js

pm_list.js
    pm_list.update_private_messages
        echo.js
        message_fetch.js

popovers.js
    popovers.set_userlist_placement
        resize.js

    popovers.hide_all
        echo.js
        pm_list.js
        resize.js
        stream_list.js

    popovers.hide_reactions_popover
        reactions.js

reload.js
    reload.initiate
        channel.js
        compose.js
        server_events.js

    reload.is_in_progress
        compose.js
        server_events.js

    reload.is_pending
        compose.js

search.js
    search.update_button_visibility
        narrow.js

server_events.js
    server_events.home_view_loaded
        message_fetch.js

    server_events.force_get_events
        tutorial.js

    server_events.assert_get_events_running
        compose.js

    server_events.restart_get_events
        compose.js
        socket.js

stream_events.js
    stream_events.mark_unsubscribed
        subs.js

    stream_events.mark_subscribed
        subs.js

stream_list.js
    stream_list.get_stream_li
        navigate.js

    stream_list.update_dom_with_unread_counts
        unread_ui.js

    stream_list.update_streams_sidebar
        message_fetch.js

subs.js
    subs.set_color
        stream_color.js

    subs.check_stream_existence
        compose.js

    subs.invite_user_to_stream
        compose.js

tab_bar.js
    tab_bar.colorize_tab_bar
        stream_color.js

tutorial.js
    tutorial.defer
        message_fetch.js

    tutorial.is_running
        message_fetch.js

ui.js
    ui.hide_loading_more_messages_indicator
        message_fetch.js
        narrow.js

    ui.show_local_message_arrived
        echo.js

    ui.show_message_failed
        echo.js

    ui.show_loading_more_messages_indicator
        message_fetch.js
        narrow.js

    ui.actively_scrolling
        narrow.js

    ui.replace_emoji_with_text
        notifications.js

    ui.show_failed_message_success
        echo.js

unread_ops.js
    unread_ops.process_visible
        notifications.js

Wow, can't wait for this to be done 😄

You can use this tool to generate the report at the top of this issue:
https://github.com/zulip/zulip/blob/36b29a966ebfcdf9c57dc3358c0dc6fa08a47ef0/tools/js-dep-visualizer.py
As we fix stuff, the tool will start to generate different output, and there's some possibility that as we break some dependencies, the tool may make different decisions about the future roadmap than it originally did.  This could be for various reasons--for example, we may break a dependency by splitting out code, but that may introduce new dependencies, that while hopefully more innocuous, will change the way the roadmap algorithm "sees" the call graph.

I'm closing this, as early efforts to break dependencies showed it to be a pretty large problem, and we probably just need to accept that any future asset-building tools will need to be able to work with circular dependencies.  (Some of our circular dependencies are symptoms of bad code organization, but not all of them.)


[Bug]: Beta doorbell streaming is perfect but…

What happened?
Since the stable version was not working for me, I tried the 1.1.1beta2 and the streaming works like a charm, the only issue is when I leave the streaming it breaks homebridge and it has to restart:

Device Type
Doorbell (Supported)
Plugin Version
beta (Best Effort)
HomeBridge Version
v1.3.x (Supported)
NodeJS Version
v16 (Supported)
Operating System
Raspbian
Relevant log output
No response

Same here

Duplicate of https://github.com/samemory/homebridge-eufy-security/issues/262


string(184) "Failed without error message: wkhtmltoimage "test123.html" "\AppData\Local\Temp\tmpB072.tmp.png" (Exit code: 1)"

getting this issue while creating image. I am on windows string(184) "Failed without error message: wkhtmltoimage "test123.html" "\AppData\Local\Temp\tmpB072.tmp.png" (Exit code: 1)"

I can not help you without any (short) code example. Your problem sounds like you forgot to configure the path to wkhtmltopdf.exe. See the second paragraph in the README:

For Windows systems make sure to set the path to wkhtmltopdf.exe in the binary option. Alternatively you can add the wkhtmltopdf "bin" directory to the system PATH variable to allow wkhtmltopdf command available to Windows CMD.

do we also need to add this .exe file in linux server?

You need to install wkhtmltopdf. But all  this is mentioned in the README. Please understand that this is not a support forum but for reporting bugs.


App works with localhost or loopback IP but not with device IP

Bug description
I have my Laravel 9 installation with tenancy inside xampp/htdocs. It worked fine when I used API URL with localhost or loopback IP, but it threw 404 when I used my device's IP address.
http://localhost/myapp/public/api/users It worked fine and returned me all the users
but
http://<IP_ADDRESS>/myapp/public/api/users gave me 404
I simply tested the case with only one php file test.php and http://<IP_ADDRESS>/testapp/test.php worked fine.
I figured out that this error is due to my tenancy configuration. In the config/tenancy.php file there is list of central domains where it has been stated in the comment that those central domains are only relevant if I am using domain or subdomain to identify tenants. But I am using request data to identify tenant and it is still preventing me from accessing the app from other URLs except those listed there.
Old config
/**
 * The list of domains hosting your central app.
 *
 * Only relevant if you're using the domain or subdomain identification middleware.
 */
'central_domains' => [
    '<IP_ADDRESS>',
    'localhost',
],

Fix
I simply added my local device IP to that list of central domains, and it worked. Also, I found that I can use an empty string to allow all domains.
New config
/**
 * The list of domains hosting your central app.
 *
 * Only relevant if you're using the domain or subdomain identification middleware.
 */
'central_domains' => [
    '<IP_ADDRESS>',
    'localhost',
    '',
],

Is this the expected behaviour?
Steps to reproduce
Simply browse the tenancy enabled site with local device IP instead of localhost or loopback IP.
Expected behavior
The site should open no matter which URL I used among localhost, loopback, and my local network IP.
Laravel version
9.6.0
stancl/tenancy version
3.5.5

If you're using an IP address, you can't be using domain identification.
And if you're not using domain identification, the central_domains config has no effect.

Exactly. But what I said is a bit different. I am not using the IP address for identification. I am using request data for tenant identification.
The issue has nothing to do with tenant identification instead I am not being able to access the application using my local IP address. I can use localhost and <IP_ADDRESS> but not my local IP. But after adding my local IP or a blank string to the list of central domains I was able to access the IP.

I'm not sure why the config would have an effect here then? Are you using the PreventAccessFromCentralDomains middleware or something like that?
It doesn't seem right that central_domains would do anything with request data identification.

No, I am not using any other middleware than InitializeTenancyByRequestData and CheckTenantForMaintenanceMode. You can test this behavior by putting a tenant enabled app inside server root and trying to access with local IP.
Surprisingly, the requests don't even show up in the middleware. I created a Logger middleware to check incoming requests and logged the requests inside public/index.php and the Logger middleware, the requests were reaching to index.php but not to the middleware when using IP address.
I could see the request in the middleware while accessing via http://localhost/myapp/public/api/users and http://<IP_ADDRESS>/myapp/public/api/users but not via http://<IP_ADDRESS>/myapp/public/api/users until I added <IP_ADDRESS> into the central_domains.

That's strange. I looked into the code and central_domains doesn't appear to be used in anything related to request data identification but for some reason the request data test also overrides central_domains.
I'll keep this issue closed right now since it isn't technically a bug in any described behavior, but if you could debug this further and see what's causing this, I'm happy to reopen and fix it.
Thanks!

I think we shouldn't map routes in RouteServiceProvider if I am not using domain or subdomain identifiers. In the documentation, there is instruction to map routes where we are specifically using central domains while mapping.
I think we can leave RouteServiceProvider as it is if we are using request data identifier. Am I right?

I think so, yeah.

Thanks. I request you to confirm this and update the docs so that it is clear for everyone. l


Simplify module/symbol filter definition format, impl

Update the filter rule format and implementation to be simpler and user-predictable. In particular, we remove an accidental dependence of rule application on hash map iteration order.
Example of the new rule format (with explanation inline):
[
    // Track any .exe, but only functions whose symbols match some regex (ignoring the rest).
    {
        "module": ".exe$",
        "include": [
            "^main$",
            "test_input$",
            "read_all$"
        ]
    },
    // Track `libabc.so`, `libabc.so.1`, &c, excluding only those symbols whose
    // name contains the string `asan` or starts with `_private` (tracking the rest).
    {
        "module": "libabc.so",
        "exclude": [
            "asan",
            "^_private.*"
        ]
    },
    // Track the entire module `libxyz.so`, and all its symbols.
    // By default, this would be redundant, except for the next rule.
    {
        "module": "libxyz.so",
        "include": true
    },
    // If a module does not match any of the above, do not track it.
    // This means we won't even try to parse backing file.
    // This effectively inverts our default behavior.
    {
        "module": ".*",
        "include": false
    }
]

Note that our JSON parsing does not actually accept comment syntax.

include and exclude can be either a bool or a list of regexes?

include and exclude can be either a bool or a list of regexes?

Correct. It isn't hard to change that, if we want.

Add new doc that explains the coverage tracking rule format, with worked examples.


add guardrails to open long & short preventing double max

in our fuzz tests we should prevent allowing two max longs/shorts in row
lets put a check in the interface before calling rust funcs; if bond & share are at certain levels (e.g. close to equal for max long) then we directly return FixedPoint(0) without bothering with the rust func. Similar for max short
we should also catch neg interest in crash reporting and see if it's because max long was too big (calc_max has noise, this results from two max trades in a row)

we discussed this a bit, and the most robust solution would be to add guardrails to the rust sdk. that may still be something we want to do, but it's a bigger task, since the logic needs to be mirrored in solidity. we have a lot more flexibility in addressing this issue in python, like running a mock trade at the end.

A shortcut was added to max long in https://github.com/delvtech/hyperdrive-rs/pull/5
A similar one should be added to max short.


examples/terraform/modules: Add outputs.tf with kubeconfig

Enables module users to use the cluster's kubeconfig in their own, add-on infrastructure declarations, such as adding diskless workers.

I rely on this and know its working.


OpenSource : 4.4 Explain everything related to Linux Foundation Mentorship Programs

Description :

Explain everything related to Linux Foundation Mentorship Programs

Note :

Changes should be made inside the Open_Source/ directory & Opensource branch.
Issue only for GWOC'21 contributors.
Issue will be assigned on a first come first serve basis, 1 Issue  ==  1 PR == 1 Contributor
For Contributor please don't create the same issue again, before creating any issue do visit OpenSource Milestone
Doc name should same as the issue title

Must Follow : Contributing Guidelines & Code of Conduct before start Contributing.
Happy Contributing 🙌

Can I try with this?

Can I try with this?

Sure @Harshil-Jani , Happy Documenting 🎉🎉


Not able to find file "data/item_ids"

I tried to run the main.py script. However, I encounter the following error.
   124 def remap_data(
    125     input_data,
    126     polyvore_version="nondisjoint",
   (...)
    129     generate_per_outfit=0,
    130 ):
--> 132     with open(data_path + "item_ids", "r") as fp:
    133         cols = json.load(fp)
    135     category_x_itemID = (
    136         item_metadata[cols]
    137         .transpose()
    138         .reset_index()
    139         .rename({"index": "item_id"}, axis=1)
    140     )

FileNotFoundError: [Errno 2] No such file or directory: 'data/item_ids'

The file is item_ids is not in the polyvore data. Where can I find this file?

Thank you for raising this issue. I have now added the "data/item_ids" file. Please let me know if you face any trouble with the file.

It fixes the issue. Thank you!


Give external-dns service port a name handle

What this PR does / why we need it:
The externaldns service port needs a name in order for prometheus-operator to be able to scrape from it. Give the port a name.
No-one listed in Chart.yaml to not sure who to @-mention?
Signed-off-by: Joe Hohertz<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test
/approve
/lgtm


Does not gracefully handle when Shopify returns 404

The application crashes instead of simply printing out the error. When a 404 happens, still raise an error but include detailed information (error code, etc.)
/cc @stevebosworth

Is this still an issue? Do you have any replication steps @stevebosworth?

I have not seen it in a while so it's possible this is resolved.  I'm not sure how you'd trigger a 404 to check tho

I found it. PR will be going up soon.


Put reason in for rejecting PIREP

Is your feature request related to a problem? Please describe.
When my team rejects PIREPs, it sends an email saying their pirep has been rejected, but doesn't actually give them a reason why - forcing us to email them again
Describe the solution you'd like
I would like to make it so when you click reject on a PIREP, it comes up with a textbox where you can input the reason for rejection, so that it emails them saying their pirep has been rejected and puts the reason in the same email.
Additional context
Screenshot of what it did in PHPVMS5

There is a field for comments, if that gets emailed (just realized it doesn't), would that be good enough? Or should there be a dedicated thing for admin comments that gets passed along with an acceptance/rejection?

I personally think it would be better to have both of those. I'd like it to have the rejection comment only so they don't have to get two emails (comment added and pirep rejected) - I feel it would be better just to include it all in one email.
Obviously that may be a stretch so if you can only get the comment one working that will be fine :)

That makes sense, just checking. I think I'll need to do both anyway. I'm just trying to prioritize

Thanks again, keep up the great work :)

Just some notes for myself, I'll try do this for the next dev

Add popup for reason when rejection button is clicked (plain browser prompt() is ok for now, or modal with #onclose for OK)
DB field rejection_reason NULL DEFAULT '' (table, model, validations)
Add rejection_reason into email templates

I'm currently had this modification in phpvms\resources\views\notifications\mail\pirep\rejected.blade.php
Your PIREP has been rejected
 @if($pirep->comments->count() > 0)     ## Comments     @foreach($pirep->comments as $comment)       - {{ $comment->comment }}     @endforeach   @endif
@component('mail::button', ['url' => route('frontend.pireps.show', [$pirep->id])])


TST: check internal Categorical

[x] closes #13076
[x] tests added / passed
[x] passes git diff upstream/master | flake8 --diff

thanks @sinhrks


Spawn host objects works via messages just like clients

instead of having a special case for host objects,  spawn them like client/server objects via messages

did you guys test this one in ummorpg and usurvival?

@vis2k wait for the "PassedTestRoutine"  label please

:tada: This PR is included in version 5.1.6 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Detect detached symbols

There has been a lot of user research and feedback from designers saying that detecting the presence of "detached symbols" will be a super useful rule to have.
So this means detecting when a user has clicked "Detach from symbol" on a symbol instance, and converted it to a plain group, breaking the link with the symbol master. This can be a major problem in large organisations that require strict adherence to design systems using libraries and symbols.
This issue is to discuss how to implement a rule that achieves this. This is tricky because, as far as I know, once a symbol has been detached and converted into a group there is no record in the file JSON that the group was ever previously a symbol instance.
One idea has been to update Sketch to record a note in the group's userInfo property that it was previously a symbol instance ...
Or is there another way?

The change to support this from the sketch app has been merged. Any Symbol or Style that is detached will leave a userInfo entry on the layer.
The userinfo entry will be keyed by 'com.sketch.detach' and will contain information about what was detached.


(22.11) openrazer module doesn't make user service available

Describe the bug
The service file for the openrazer-daemon.service is created (checked via ls /nix/store), but not executed. It can also not be found by systemctl.
Steps To Reproduce
Steps to reproduce the behavior:

hardware.openrazer.enable = true in config

I really don't know what else could lead to this behaviour
Expected behavior

daemon is started on graphical-session.target
i can start, stop, ... the daemon via systemctl --user ...

Additional context
$ systemctl --user status openrazer-daemon.service
Unit openrazer-daemon.service could not be found.

When i manually copy the exec line from the service file that is being created, i can start the daemon (which also indicates that the daemon is not started by sysd, as you would get "unable to lock on the pidfile" otherwise.
Please tell me if something like this was fixed some time ago, as i have last updated my systems flake.lock one month ago (updates always take ages due to my network connection).
EDIT: updated to latest 22.11 revision, problem persists
whats weird, is that all other user services that i am aware of seem to be functioning properly, openrazer is the only bad fruit here
Metadata
(Flake system, no channels)
 - system: `"x86_64-linux"`
 - host os: `Linux 6.2.0, NixOS, 22.11 (Raccoon), 22.11.20230307.a7cc819`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.1`

Update: still not working as expected in 23.05. Using service exec line to run the service works without any issues (the file is created, just not used by systemd).
Checked journalctl because this is getting on my nerves again and this is what came up:
journalctl -b 0 | grep openrazer
Okt 10 13:21:29 snowstorm systemd[1714]: openrazer-daemon.service: Failed to open /home/paul/.config/systemd/user/openrazer-daemon.service: No such file or directory

so it seems like running the service/making it run on default.target works, it just doesn't use the derivations output to run it?

Update: still not working as expected in 23.05. Using service exec line to run the service works without any issues (the file is created, just not used by systemd). Checked journalctl because this is getting on my nerves again and this is what came up:
journalctl -b 0 | grep openrazer
Okt 10 13:21:29 snowstorm systemd[1714]: openrazer-daemon.service: Failed to open /home/paul/.config/systemd/user/openrazer-daemon.service: No such file or directory

so it seems like running the service/making it run on default.target works, it just doesn't use the derivations output to run it?

Do you have hardware.openrazer.users = [ "youruser" ]; somewhere on your conifg?
https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/hardware/openrazer.nix#L98

Yes, i am using the name from users.users.<name>
hardware.openrazer.users = [ "paul" "luise" ];

(forgot to include that in the question, sorry)

ok found the issue, for some reason i do not understand there was always a symlink to a previous generations' output in the .config/systemd directory, that way it looked legit to me but i just didn't know that it was actually pointing to nothing.
Solved now :)


Track slack improvements opportunities

First Draft and iteration of this document was created on April 25th, 2022
https://docs.google.com/document/d/1LxTxBjfiwtizJskJ_LRns5-wnHPFUdrCaqLhSmFhgms/edit?usp=sharing
This is an ongoing discussion. CC Admins as of May 2, 2022 are considering moving team activity to Discord to incorporate Dework.


add journal reference

Adding a CITATION.bib to the top level directory is relatively common in the Julia ecosystem.

Maybe it would be good to add links to the citation file from the relevant README.md and index.md sections?

I don't think so. We already list the relevant bibtex entries in the README and the docs explicitly, which humans will find easily. From my point of view, CITATIONS.bib is more written for bots etc., so we don't need this additional level of indirection.

OK, sounds reasonable.


AWS Lambda 'recursive loop protection' might consider the lambda function a problem

I have a variant of this code that has been working well for several years.
In the last 24h I got an email from AWS saying that AWS Lambda is introducing 'recursive loop detection'.

Starting July 5, 2023, AWS Lambda is launching recursive loop detection, a capability that automatically detects and stops certain recursive invocations. With this launch, Lambda will stop invocations for functions that utilize supported SDK versions [1] and are part of a recursive loop composed of the underlying Lambda function and Amazon SQS queue and/or Amazon SNS topic after 16 recursive calls.
To prevent potential disruption to your account, we have turned off this new feature for the accounts in the regions listed in this notification because we previously detected that you have one or more Lambda functions in the xxxxx Region that are being invoked in a recursive loop with other AWS resources. These recursive loops were detected some time during the last three months. Recursive invocations within these accounts will not be stopped.
[1] https://docs.aws.amazon.com/lambda/latest/dg/invocation-recursion.html

Code such as this draining example will be considered by AWS as a recursive loop because it publishes a message to the same SNS topic that launches the lambda function as per here.
You might want to update the example code with a suitable mitigation (perhaps some sort of decreasing counter?)

We have the same issue. Would be cool to see an update of this lambda function from this project.


Improve keyboard friendliness of history in Reference List

Testing https://github.com/Microsoft/vscode/issues/64227.

search for references
give editor focus
run References: Clear from the command palette
Cmd+0 to focus references list
=> I have to use tab to get to the correct history entry to re-run. I'd like to use arrow keys to navigate or even F4 as this is what I would use in this view anyways. When navigating the history, the focus should stay in the view.

we have done that, the history is now normal tree/list entries


Captcha Problem

Running osx Yosemite  10.10.3
Dependencies installed.
Tried many times with different usernames, passwords, email and Gmail accounts.
When account seems to be aproven, "captcha.zip" is not generated.
When proces is "UNAUTHORIZED", "captcha.zip" is generated, but when decompressing  "captcha.zip.cpgz" is generated and decompresing it will give "captcha2.zip" and so on. I'm guessing captcha is not well generated. Tried with 3 unarchivers.
Any suggestions?
 If there is a ghost in a image means 1, if not 0
The result should be like the following one: 110000101
After completion, the zip file will be deleted automatically.
Result:

@LanchaBasualdo try with latest update

Same problem.

running php 5.6.9
php 5.3 really necessary?

looks like it's an issue with the bearer

doesn't look like it's getting a bearer token from google before registration and it's probably needed

one sec

yep, i suspected right. i'll push a fix now

cc3020acf485b3cdf406410429e194d52a8d4c8c

@teknogeek
Thanks but still

"but still" what? it works fine now
On Jun 11, 2015 3:10 PM, "LanchaBasualdo"<EMAIL_ADDRESS>wrote:

@teknogeek https://github.com/teknogeek
Thanks but still
—
Reply to this email directly or view it on GitHub
https://github.com/mgp25/SC-API/issues/131#issuecomment-111245078.

@teknogeek Sorry, typing error.
Thanks, working know

Still have the same problem


Check errors if we can't annotate a secret

Now we ignore errors from setResourceMigrated function, which can lead to unpredictable behavior in runtime. To prevent this we add the standard error checking in the function.

/lgtm
/approve
There is only a marshalling error. This will never happen. So I think this merely cosmetics.


[Feature request] Ability to chose which build events trigger an email notification

When subscribed to email notifications I receive an email for all build events (STARTED, FAILED, SUCCEED, etc).
I would like to select which build events should trigger an email notification.

The Amplify UI team would love to have this feature get prioritized on the roadmap. It would be very helpful for limiting noise in automated CD pipelines while allowing us to specifically be notified of build failures. Thanks, Hosting team!

I had the same need to receive only notifications for FAILED deployment builds. You can do it by editing the AWS Event Bridge rule associated with your Amplify app.
See snippet below
{
  "detail": {
    "appId": ["<<my amplify app id>>"],
    "jobStatus": ["SUCCEED", "FAILED", "STARTED"] // Remove `"SUCCEED"` and `"STARTED"` here.
  },
  "detail-type": ["Amplify Deployment Status Change"],
  "source": ["aws.amplify"]
}

Credit https://github.com/aws-amplify/amplify-hosting/issues/271#issuecomment-866849726


Broken Dockerhub tag structure

The tag structure on the public dockerhub repo seems completely hosed. Many recent versions have recently gone missing are missing and latest links to a version published over a year ago.

@rhyas I've updated the latest tag to match v1.12.2. That was an oversight on my part when I pushed v1.12.2 a few weeks ago.
Sounds like we have two separate issues:

no automation guaranteeing latest matches the actual latest build
no automation guaranteeing a docker image is pushed for every release of kubectl

If you're interested in addressing these, pull requests welcome.


FeatureLayer.query(geometryFilter=

Hello,
Can someone please help to verify the use of the 'geometryFilter' and perhaps provide a few input examples. I can't seem to get the query to honor this parameter.
"{'xmin': 321200, 'ymin': 5529500, 'xmax': 322200, 'ymax': 5530500, 'spatialReference': {'wkid': 26911, 'latestWkid': 26911}}"
OR
321200, 5529500, 322200, 5530500
OR
as arcpy.Extent object
Thanks,

@orlando67 – I’ll take a look today and get back to you.
Andrew

@orlando67
Below is a working example using geometryFilter object and a query on a feature layer:
 envelope = arcrest.common.geometry.Envelope(xmin=-125.4,
                                           ymin=35.2,
                                           xmax=-118.7,
                                           ymax=43,
                                          wkid=4326)
gf = arcrest.common.filters.GeometryFilter(geomObject=envelope)
url = "http://sampleserver1.arcgisonline.com/ArcGIS/rest/services/Specialty/ESRI_StateCityHighway_USA/MapServer/1"
fl = arcrest.ags.FeatureLayer(url=url, initialize=True)

print fl.query(geometryFilter=gf, where="1=1", returnCountOnly=True) # count is 3

Hope this helps.
Andrew

Thanks a million, works like a charm!


how to upload binary to ESP?

I was wondering, is suppose the travis build be available to download or how exactly should I do in order to upload the binary to the ESP? (btw my travis looks is working fine)

To answer my own question, if you run the code via travis, you can upload the binary using the site transfer.sh.