Create subclasses of Model for Water/Energy/Nutrient cycles

At the moment the framework only focusses on the water cycle, but we aim to include at least the nutrient cycle. Once one other cycle is included in the framework, and the two cycles are able to communicate, adding any additional cycle is likely to be trivial.
At this stage, I think the best way to achieve this is by subclassing the Model class into WaterModel (then NutrientModel, etc.). Each will take one component for each part of the terrestrial water cycle. But their inward/outward transfers will need to be specific to the cycle considered (i.e. compliant with the interface of the given cycle). This is likely to be done by renaming current SurfaceLayerComponent/SubSurfaceComponent/OpenWaterComponent to WaterSurfaceLayerComponent etc. and by creating additional ones for NutrientSurfaceLayerComponent etc. They should all be able to directly be subclasses of Component, as in currently the case for the water cycle components. The superclass Model should retain the Clock and the Coupler.
For now water and energy cycles are likely to be considered together, because they are very intertwined in JULES at the moment, and it would require substantial efforts to isolate them (if possible at all). But, by bringing an additional cycle in the framework, most of the challenges will already have been addressed and, separating water and energy cycles would only require to remove energy fluxes from the water cycle interface + create a new energy cycle + breakdown actual science components to comply with the new interfaces.

One important decision to make here is whether we allow components of other cycles than water+energy to run on different temporal/spatial resolution of the water+energy cycle.
For now, we don't see a use case where different resolutions would be required for different cycles.

Starting development of separate Nutrient subclasses. I'm seeing this as the master-thread that collates together the various tasks and issues that will need to be completed to successfully add nutrients separate nutrient components to unifhy.
An initial, but temporary, solution was to integrate the nutrient components within the existing Surface, Subsurface and Openwater Components which are considered the 'water energy' Components. This means that the necessary transfers between Components are already implemented and the main remaining task is to separate these out into separate 'Nutrient' Components.
To ensure integrity of the code and minimise the chance of breaking stuff, I will follow the 'test-driven-development' idea which requires tests for the code to be developed before the code itself. As comprehensive tests have already been designed for unifhy, I see the first step as editing these tests to expect 3 additional Components - the NutrientSurface, NutrientSubsurface and NutrientOpenwater Components. This will cause the tests to fail, with subsequent development just enough to cause the tests to pass and no more. Once this has been achieved I can start following the 'test-driven-development' idea properly and implement much smaller changes to the tests and subsequent development.
To that end the first issue is to catalogue and then implement all the changes that need to be made to the tests to support the 3 additional components. See issue... for this.

Great to see you have started on this. Let me know if there is anything I can do to help.

Will do! I've been quietly working a lot on it the last few weeks, mostly just deep-diving into the code and getting to grips with how it's structured and where the changes are most likely to be made.

Happy to help as well if you have questions/need code reviews. :-)

Thanks Thibault!
At the moment I think feedback on the approach I'm taking and stuff I'm planning to do (when I get to it) would be most useful. I intend to document what I intend to do here first before implementing anything. But extra code reviewers certainly can't hurt too!

Test updates (finally!) complete. Issue #93 closed.
Next step is to analyse which of these changes will result in the errors when the full framework is run and make changes to the framework to allow the tests to pass.

[ ] Calls to unifhy.Model() need to accept nutrient components as an additional three arguments
[ ] unifhy/model.py needs to have nutrientsurfacelayer, nutrientsubsurface and nutrientopenwater layers defined as classes
[ ] TBC

Now to check and commit after a break...

Documentation also needs updating: #99

Generic list of changes to be made for future Component additions to the framework:
unifhy.model

[x] Add XxxxxYyyyyComponent to the from .component import statement
[x] Add

xxxxxyyyyy: XxxxxYyyyyComponent object
The Component responsible for ...

to the docstring of the __init__  function

[x] __init__ needs to accept the extra component as an extra argument
[x] __init__: A call to _process_component_type needs to be added for the new component
[x] _check_components_plugging: Add the new Component to the loop
[x] identifier.setter: Add a line for the new Component, following the syntax used for the other Components
[x] __str__: Add the Component info as for the other Components
[x] from_config:  add an import_module statement for the new Component as for the other Components
[x] And add it to the return statement of the from_config method similarly
[x] to_config: Add the new Components in the return statement as the other Components
[x] initialise_transfers_from_dump: Add new Components to Compass, Clock and Exchanger calls, as they are for the other Components
[x] spin_up: Add get_spin_up_timedomain call, add main_??_td lines/assignments and in two places add self.xxxxxyyyyy.timedomain lines/assignments
[x] _initialise: Add self.xxxxxyyyyy.initialse_(...) lines like for the other modules
[x] _run: Add the new Component to the Compass, Clock and Exchanger sections as for the other Components
[x] Add run_xxxxxyyyyy to the loop over clock, then add a dump line in the if dumping: section, then add a run_xxxxxyyyyy block for the new Component, like the other Components
[x] _finalise: Add a self.xxxxxyyyyy.finalise_() line like for the other Components
[x] resume: Add the new Component to the loop to initialise component states and records, and to the remaining time domain calculation loop

unifhy.component

[x] In the Component Class, change the documentation of the category getter to include nutrients, i.e. "part of the water*/nutrients* cycle"
[x] Add NutrientSurfaceLayerComponent, NutrientSubSurfaceComponent and NutrientOpenWaterComponent Classes, following the structure of the pre-existing Components, and moving the nutrients entries in their _input and _output_info dictionaries to their respective nutrients Components (i.e. from SurfaceLayerComponent to NutrientSurfaceLayerComponent, SubSurfaceComponent to NutrientSubSurfaceComponent, OpenWaterComponent to NutrientOpenWaterComponent)

XxxxxYyyyy is the name of the class of the Component in the unifhy/component.py and xxxxxyyyyy is the name given to the component by the _category attribute of the class.
Elsewhere

[x] Calls anywhere to unifhy.Model() need to accept the new Component as an additional argument

Applicable to tests updates only:

[x] Add

nutrientsurfacelayer: tests.components.nutrientsurfacelayer.dummy
nutrientsubsurface: tests.components.nutrientsubsurface.dummy
nutrientopenwater: tests.components.nutrientopenwater.dummy

to the docstring of the from_yaml method


Add time for next skill drain tick

Title

#699 has this, but is yet to be merged.

Duplicate of #288


The meaning of num_feature_levels.

Thanks for your great and detailed code!
In dab_deformable_detr.py, is this part about dealing with the insufficient channel of features (for Deformable DETR)?

https://github.com/IDEA-opensource/DAB-DETR/blob/309f6ad92af7a62d7732c1bdf1e0c7a69a7bdaef/models/dab_deformable_detr/dab_deformable_detr.py#L169-L181

It seems that the last channel of features is put to backbone[1] repeatedly. (channel? or something else?)
Could you please help me understand that?

Yes, it is. By default, num_feature_levels=4 but 3 different scales are extracted from the backbone. Hence it will 2x upsample the highest feature map(C4) as the 4th feature level.


[cuda-api-wrappers] new port

Upstream: https://github.com/eyalroz/cuda-api-wrappers
Related PR:  https://github.com/microsoft/vcpkg/pull/30888
Thanks to @eladm-ultrawis

[x] Changes comply with the maintainer guide
[x] The name of the port matches an existing name for this component on https://repology.org/ if possible, and/or is strongly associated with that component on search engines.
[x] Optional dependencies are resolved in exactly one way. For example, if the component is built with CMake, all find_package calls are REQUIRED, are satisfied by vcpkg.json's declared dependencies, or disabled with CMAKE_DISABLE_FIND_PACKAGE_Xxx
[x] The versioning scheme in vcpkg.json matches what upstream says.
[x] The license declaration in vcpkg.json matches what upstream says.
[x] The installed as the "copyright" file matches what upstream says.
[x] The source code of the component installed comes from an authoritative source.
[x] The generated "usage text" is accurate. See adding-usage for context.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is in the new port's versions file.
[x] Only one version is added to each modified port's versions file.

CMAKE_CUDA_ARCHITECTURES is None on CI, but its valid on local  :crying_cat_face:

Note: I will be converting your PR to draft status. When you respond, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

I'm not experienced with vcpkg, but I will note that I see an attempt to "find CUDA", but don't see a check for whether NVIDIA driver is installed. These two things are generally separate, i.e. you can have the CUDA toolkit without a driver and vice-versa. So perhaps that check is necessary as well. Or not, I can't say for sure.

@Adela0814 Than you for the reivew. I fxed algiment.
@eyalroz
find_program(NVIDIA_SMI "nvidia-smi")

if(NOT NVIDIA_SMI)
    message(STATUS "NVIDIA driver not found")
else()
    execute_process(COMMAND ${NVIDIA_SMI} --query-gpu=driver_version --format=csv,noheader
                    OUTPUT_VARIABLE NVIDIA_DRIVER_VERSION
                    OUTPUT_STRIP_TRAILING_WHITESPACE)
    message(STATUS "NVIDIA Driver Version: ${NVIDIA_DRIVER_VERSION}")
endif()

This snipper woudl work. I gues

@eyalroz Driver checking make CI failure,
I understand your concern. But I don't think you should put in a nvidia driver check.
Why not just assume that the developer PC has both?

@dg0yt  Thanks a lot!

@eyalroz Driver check causes CI failure.

Really? A machine doesn't need to have an actual GPU in order to have the driver.

I understand your concern. But I don't think we should put in a nvidia driver check. Why not just assume that the developer PC has both? Of course, as you say, the driver should work without the toolkit, and vice versa. To break it down like that, I think it would be too much to write all the features in the vcpkg port depending on the mode.

Well, the library is header-only - and thus doesn't really require anything else; but to you want to build something with the wrapper library e.g. its example programs - you have to have both the toolkit and the driver.
In the library's CMakeLists, I require both the CUDA runtime library and the driver. Although the toolkit-provided stub is sufficient to satisfy that ... Anyway, I guess it's fine the way you have it now.

@eyalroz These are the CI log when I tried to put driver check in CMakeLists.txt
Linux
https://dev.azure.com/vcpkg/c1ee48cb-0df2-4ab3-8384-b1df5a79fe53/_apis/build/builds/97109/artifacts?artifactName=failure logs for x64-linux&api-version=7.1&%24format=zip
Windows
https://dev.azure.com/vcpkg/c1ee48cb-0df2-4ab3-8384-b1df5a79fe53/_apis/build/builds/97109/artifacts?artifactName=failure logs for x64-windows&api-version=7.1&%24format=zip
find_program(NVIDIA_SMI "nvidia-smi")

if(NOT NVIDIA_SMI)
    message(FATAL_ERROR "NVIDIA driver not found")
else()

Above driver check works fine on local machine (both Windows and Linux)
But did not work on the CI machine
-- Using source at /mnt/vcpkg-ci/buildtrees/cuda-api-wrappers/src/v0.6.6-e9c3c9be44.clean
CMake Error at ports/cuda-api-wrappers/portfile.cmake:13 (message):
  NVIDIA driver not found
Call Stack (most recent call first):
  scripts/ports.cmake:170 (include)

@eyalroz I mie
I think it is just path problem, I will figure it out.

Usage tested pass on x64-windows.

Thanks :)

Thank you for the reviews.


kubeflow-fairing-presubmit integration tests always running timeout

/kind bug
What steps did you take and what happened:
[A clear and concise description of what the bug is.]
kubeflow-fairing-presubmit integration tests always running timeout
What did you expect to happen:
kubeflow-fairing-presubmit integration  should be successfully.
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Your build timed out. Use the [--timeout=DURATION] flag to change the timeout threshold.
ERROR: (gcloud.builds.submit) build eabff2c0-6398-4984-887b-9786cb624ac8 completed with status "TIMEOUT"

failed to create fsnotify watcher: too many open files

@karthikv2k @abhi-g I tried to find the root course, unlucky I did not get that, I guess that caused by CI/CD env problems, since there is some message failed to create fsnotify watcher: too many open files  sometimes, any suggestion for this? or could you please take a look? Thanks.

Not sure if this is root couse, but seems the fs.inotify.max_user_instances = 128,  that's small, but we cannot update that in the docker container...
sysctl: setting key "fs.inotify.max_user_instances": Read-only file system

Thanks Jin Chi for investigating. I'll try to get further help.
On Fri, Aug 9, 2019 at 1:35 AM Jin Chi He<EMAIL_ADDRESS>wrote:

Not sure if this is root couse, but seems the fs.inotify.max_user_instances
= 128, that's small, but we cannot update that in the docker container...
sysctl: setting key "fs.inotify.max_user_instances": Read-only file system
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/fairing/issues/342?email_source=notifications&email_token=ACZ2UZRJXMWH3BCUVARTYDDQDUT47A5CNFSM4IKQQRFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD36AMRY#issuecomment-519833159,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACZ2UZXO6JA6BNPKTURARPDQDUT47ANCNFSM4IKQQRFA
.


Should the binary_sensor be On or Off ?

Thanks for the integration
A query?
Both of my binary sensors 'binary_sensor.dvla_xxxxxxx_motstatus' & 'binary_sensor.dvla_xxxxxxx_taxstatus' are showing as Off.
Before the recent updates they showed as On.
The car is taxed & MOT'd
The MOT Status is showing as Valid and the Tax Status is showing as Taxed
I've tried removing the integration from HACS and re-downloading it, etc, but they remain in the Off state.
I don't have any errors in the logs except for:
Detected that custom integration 'dvla' calls `async_add_job`, which is deprecated and will be removed in Home Assistant 2025.4; Please review https://developers.home-assistant.io/blog/2024/03/13/deprecate_add_run_job for replacement options at custom_components/dvla/sensor.py, line 137: self.hass.async_add_job(self.async_remove()), please create a bug report at https://github.com/jampez77/DVLA-Vehicle-Enquiry-Service/issues
which I don't think is relevant.
Any suggestions?

Thanks for bringing this to my attention, I will look into this one I return from my holidays

I've noticed the same thing, I suspect this is related to 9f336a7e833530fd5e3c9d0e883c09f0f63ebe44, because update from cordinator sets self._state, but is_on reads self._attr_is_on

Hope you are having a restful holiday ... you are far too dedicated replying while you are on holiday!!
I see the same thing
sensor.dvla__motstatus shows as "Valid"
binary_sensor.dvla__motstatus is "off" however the sensor friendly name is "MOT Valid" so I would have expected it to be "on" as opposed to "off"

Hope you are having a restful holiday ... you are far too dedicated replying while you are on holiday!!
Thanks, i had a lovely time and most importantly the children loved it :)

This should be fixed now with 2024.9.4

I've applied the latest release
and it's all working again,
Thank you

Excellent news, thanks!


Enable all protocols supported by upstream hyper via ALPN by default

Disclaimer: I didn’t look at the code from this crate for too long.
This crate appears to be pretty much the most hands free way to get https support with hyper. However, a caveat is that this does not do any protocol negotiation and thus hyper is forced to always use http1.1.
I think it should, by default, do most of this kind of setup for the user without any additional effort on the user end.

Isn't the problem that native-tls doesn't support ALPN? (Because it's only implemented on a few of the underlying native implementations).

Correct, native-tls doesn't have ALPN support, and won't until all the backends have support added (see https://github.com/sfackler/rust-native-tls/pull/113).

Do we want to reopen this given that native-tls added ALPN support now? https://github.com/sfackler/rust-native-tls/blob/master/CHANGELOG.md#v027 The documentation around the new feature seems to be rather lackluster though, the alpn flag is only mentioned in the changelog and not on the docs.rs or in the README.

Do we want to reopen this given that native-tls added ALPN support now? https://github.com/sfackler/rust-native-tls/blob/master/CHANGELOG.md#v027 The documentation around the new feature seems to be rather lackluster though, the alpn flag is only mentioned in the changelog and not on the docs.rs or in the README.


Update guide.md

Improved the guide wording to more accurately reflect the code.

Thanks for contributing!


fix handling of request exceptions in client.js

When a network or internal exception is thrown when making the http request via needle (for example when mocking via unit tests using nock), it would throw a TypeError in Client.handleErrors() because reason.options is not set for those error (reason is the caught error object):
TypeError: Cannot read property 'method' of undefined

This PR fixes it by adding the necessary structure upon errors inside rp().
Included a unit test that uncovers the problem: https://github.com/apache/openwhisk-client-js/pull/196/files#diff-1a5d07059384adab4362f4ed2eed98e2R26

Codecov Report

Merging #196 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #196      +/-   ##
==========================================
+ Coverage   99.27%   99.27%   +<.01%
==========================================
  Files          14       14
  Lines         411      414       +3
==========================================
+ Hits          408      411       +3
  Misses          3        3

Impacted Files
Coverage Δ

lib/client.js
98.88% <100%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


How to install as LXC container?

Hi,
Not particularly a feature request, but it isn't a bug at all; it's more a documentation request:
Could you please help in how can I install Obsidian Remote as an LXC container instead of using docker?
Thank you for the help.

I am not using LXC, but looking around you can use a docker container in there with this command. Let me know how it goes.
lxc-create obsidian-remote-lxc -t oci -- --url oci://ghcr.io/sytone/obsidian-remote:latest


Orchestrate a Local Cluster with Kubernetes Doc Update

Re: Orchestrate a Local Cluster with Kubernetes
Issue Description
In the step to decommission a node, include a note to issue decommission command against a node and include the node port.
Suggested Resolution
kubectl exec -it cockroachdb-3 -- ./cockroach node decommission --self --certs-dir cockroach-certs --host=cockroachdb-3.cockroachdb.default:26258

This is now resolved.


Optimize generating UUIDs from strings

I'm building a driver for Neo4j and wanted to check on the overhead of generating UUIDs from strings since it's common to store UUIDs as strings in Neo4j. While it was still really good (1M UUIDs/sec), I noticed that UUID incurred a bunch of heap allocations for hardcoded arrays and intermediate strings, so I wanted to see if it could be improved.
In total, generating a UUID from a 36-char string involved creating 2 intermediate arrays and 32 intermediate strings (each hex pair in the hardcoded array has one within the loop and another in string_has_hex_pair_at!). Moving these to use stack-based values instead of heap, I was able to get a 4x performance boost.
➜  Code crystal build --release benchmark_optimized_uuid.cr
➜  Code ./benchmark_optimized_uuid
59c87908-4c45-4a94-b45f-2d338c2b2e17
59c87908-4c45-4a94-b45f-2d338c2b2e17
   string   1.03M (970.15ns) (± 0.44%)  688 B/op   4.11× slower
optimized   4.23M (236.14ns) (± 0.59%)    0 B/op        fastest

Benchmark code

There is even more t for improvement.
The conversion of hex pairs is duplicatesdin the main method and string_has_hex_pair_at!. The helper method should directly return the byte value if it is successfully read.

Awesome! I don't know how those hardcoded arrays passed a previous review.
By the way, for those who wondered about a use case for to_i and friends from a slice instead of a string, here's one :-)

(though the current code will probably be faster than to_u8 on a slice because it goes straight to the point, but this is just a guess)

Or, since strings are immutable, maybe String#[offset, length] could return a sub String with a pointer to the parent String allocation, instead of always allocate/copy? It would be simpler than going through a Slice.

@ysbaddaden The problem is that a String knows its bytesize and utf-8 size, and that's right at the beginning of a String. Another idea would be to have String be a sort of a union (as in C union) where it can be an inlined string or a string reference to some other string. But then the problem with that is taking a small substring from a huge string and never being able to free the huge string...
I think for performance it's better to go to Slice with to_slice.

to_i and friends from a slice instead of a string

This is actually kinda where I was trying to go with this, but I figured I'd go with the smaller change. :-)

The conversion of hex pairs is duplicatesdin the main method and string_has_hex_pair_at!. The helper method should directly return the byte value if it is successfully read.

I found so many roadblocks trying to do that the first time (basically, I felt I had to choose between two potential nil values with to_u8? or a different error with to_u8) but it turns out this does indeed double performance again — we're now up to 8x.
➜  Code crystal build --release benchmark_optimized_uuid.cr && ./benchmark_optimized_uuid
df817978-7772-49fb-8b2f-6293a8e212db
df817978-7772-49fb-8b2f-6293a8e212db
   string   1.04M ( 961.5ns) (± 0.37%)  688 B/op   8.19× slower
optimized   8.52M (117.33ns) (± 0.46%)    0 B/op        fastest

Another optimization could be to work directly with the return value from string.to_slice instead of string. For example doing string[i] will check if the string is ascii-only and then index fast into it, otherwise it will have to go utf-8 char per char to check that. Most probably the string will be ascii-only, but why do the check all the time if we can skip that? (the only problem is that you'll have a byte, but you can turn that into a char with chr, or maybe it's faster not do use to_u8?, but at that point the performance gain won't be much, I think)

Or, since strings are immutable, maybe String#[offset, length] could return a sub String with a pointer to the parent String allocation, instead of always allocate/copy? It would be simpler than going through a Slice.

@ysbaddaden One of the main downsides with this is that it silently can use up a lot of memory, if the substrings have a longer life time than the parent string. Java used to have substrings behave like you suggests here but moved away from it.


Standardise deprecations in CMS4 for other core modules

Follow on card from https://github.com/silverstripe/silverstripe-framework/issues/10500
Process deprecations in CMS 4 for all core modules except framework
AC

[ ] Code with a @deprecated annotation throw a Deprecation::notice()
[ ] Code throwing a Deprecation::notice() is annotated with a @deprecated
[ ] Normalise the @deprecated annotation ... within reason.

Implementation lessons from https://github.com/silverstripe/silverstripe-framework/issues/10500

Use http://github.com/emteknetnz/silverstripe-deprecator
This will highlight deprecation messages that need to be manually changed - docblock has precedence over Deprecation::notice()
Undo code-writing commit, manually update messages, commit, then do 2nd commit containing code-writing

Related

https://github.com/silverstripe/silverstripe-assets/issues/521

PRs

https://github.com/silverstripe/silverstripe-admin/pull/1384
https://github.com/silverstripe/silverstripe-asset-admin/pull/1302
https://github.com/silverstripe/silverstripe-assets/pull/523
https://github.com/silverstripe/silverstripe-behat-extension/pull/228
https://github.com/silverstripe/silverstripe-campaign-admin/pull/246
https://github.com/silverstripe/silverstripe-cms/pull/2791
https://github.com/silverstripe/silverstripe-config/pull/70
https://github.com/silverstripe/recipe-plugin/pull/27
https://github.com/silverstripe/vendor-plugin/pull/60
https://github.com/silverstripe/silverstripe-versioned/pull/375
https://github.com/silverstripe/silverstripe-versioned-admin/pull/259
https://github.com/silverstripe/silverstripe-elemental/pull/1016

@emteknetnz
Now that the version numbers aren't actually being checked, it would be tidiest if we changed the 4.0.1 deprecation noticed back to 4.0.0 - that said, there's very little benefit in that beyond pedantic correctness so I won't actually ask you to make this change, just pointing out that it's a possibility now in case you want to do it.

PRs are merged


Service Request by [Emilia Camp] on inability to save demographics, after edits

[ ] Qualtrics Service Request Form was received from User
[ ] e-mail acknowledging that User's Qualtrics Service Request Form was sent
[ ] Problem NOT RESOLVED
[ ] Problem PARTIALLY RESOLVED
[ ] Problem FULLY RESOLVED and Issue is Closed
[ ] Qualtrics Post-Service Request Survey was sent to User

Use comments feature (below) to document:

A clear and concise description of the solution.

A clear and concise description of any alternative solutions or features you've considered.

Any additional context, such as screenshots about the problem.

From: Camp, Emilia<EMAIL_ADDRESS>Monday, April 11, 2022 at 1:33 PMTo: Jessica Neufeld<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>David Lounsbury<EMAIL_ADDRESS>Nicholas Bove<EMAIL_ADDRESS>RE: Impt/Timely: Suffolk IPA Tracker...
CAUTION: This email comes from an external source; the attachments and/or links may compromise our secure environment. Do not open or click on suspicious emails. Please click on the “Phish Alert” button on the top right of the Outlook dashboard to report any suspicious emails.
Yes.  I believe this was the same issue I reported a while ago regarding demographic.Fast Track I can’t delete -1 under gender Emilia Camp, MBASenior Business Intelligence Suffolk County Dept. of Information Technology725 Veterans Memorial Highway, Bldg. 50Hauppauge, NY 11788-0099PH:<PHONE_NUMBER><EMAIL_ADDRESS>From: Jessica Neufeld<EMAIL_ADDRESS>Sent: Monday, April 11, 2022 1:02 PMTo: Camp, Emilia<EMAIL_ADDRESS>Donrie Purcell<EMAIL_ADDRESS>David Lounsbury<EMAIL_ADDRESS>Nicholas Bove<EMAIL_ADDRESS>Re: Impt/Timely: Suffolk IPA Tracker... ATTENTION: This email came from an external source. Do not open attachments or click on links from unknown senders or unexpected emails.Hi Emilia, Thanks for bringing this to our attention.  We will work on troubleshooting the issue and get back to you.  Did you notice this issue with any other IPAs or just E2?  Thanks,Jessica On Mon, Apr 11, 2022 at 11:27 AM Camp, Emilia<EMAIL_ADDRESS>wrote:Hi Donrie I encountered another issue with the tracker.  I tried to edit the demographic for E2 gender but it won’t allow me to save.  The error is The demographic totals do not match the total value.  It matches the total value of 11 (7+4)   Emilia Camp, MBASenior Business Intelligence Suffolk County Dept. of Information Technology725 Veterans Memorial Highway, Bldg. 50Hauppauge, NY 11788-0099PH:<PHONE_NUMBER><EMAIL_ADDRESS>Camp, Emilia <[Emilia.Camp@suffolkcountyny.gov](mailto:Emilia.Camp@suffolkcountyny.gov)>
Date: Monday, April 11, 2022 at 1:33 PM
To: Jessica Neufeld
Cc: Donrie Purcell , David Lounsbury , Nicholas Bove
Subject: RE: Impt/Timely: Suffolk IPA Tracker...
CAUTION: This email comes from an external source; the attachments and/or links may compromise our secure environment. Do not open or click on suspicious emails. Please click on the “Phish Alert” button on the top right of the Outlook dashboard to report any suspicious emails.
Yes.  I believe this was the same issue I reported a while ago regarding demographic.
Fast Track I can’t delete -1 under gender
Emilia Camp, MBA
Senior Business Intelligence
Suffolk County Dept. of Information Technology
725 Veterans Memorial Highway, Bldg. 50
Hauppauge, NY 11788-0099
PH:<PHONE_NUMBER>
<EMAIL_ADDRESS>From: Jessica Neufeld<EMAIL_ADDRESS>Sent: Monday, April 11, 2022 1:02 PM
To: Camp, Emilia<EMAIL_ADDRESS>Cc: Donrie Purcell<EMAIL_ADDRESS>David Lounsbury<EMAIL_ADDRESS>Nicholas Bove<EMAIL_ADDRESS>Subject: Re: Impt/Timely: Suffolk IPA Tracker...
ATTENTION: This email came from an external source. Do not open attachments or click on links from unknown senders or unexpected emails.
Hi Emilia,
Thanks for bringing this to our attention.
We will work on troubleshooting the issue and get back to you.
Did you notice this issue with any other IPAs or just E2?
Thanks,
Jessica
On Mon, Apr 11, 2022 at 11:27 AM Camp, Emilia<EMAIL_ADDRESS>wrote:
Hi Donrie
I encountered another issue with the tracker.  I tried to edit the demographic for E2 gender but it won’t allow me to save.  The error is The demographic totals do not match the total value.
It matches the total value of 11 (7+4)
Emilia Camp, MBA
Senior Business Intelligence
Suffolk County Dept. of Information Technology
725 Veterans Memorial Highway, Bldg. 50
Hauppauge, NY 11788-0099
PH:<PHONE_NUMBER>
<EMAIL_ADDRESS>

Hi Jeff
I am now able to enter

@jbazinet1111
Donrie and I both trouble shooting this. It seems that edits to the summary values are not being saved.
Can you check this out?
Is it a record locking issue, maybe?
Best,
DL

I have identified the issue and am working on a resolution. Expect on later today.

Thank you Jeff!


Do not break on error as this is not thread safe

Following the reversion of PR18 which contained a handful of different fixes I have now broken it down into in smaller individual PR's to allow each fix to be discussed and committed individually.
This PR has been raised to trigger discussion as to a more appropriate action and (for now) simply comments out the 'break' statement in base.py on _error_callback. I have done this as it is not thread safe and meant that threads would hang (break) and the main code is not aware, this in turn appears like the thread has hung (but no error messages are presented to the user).
It may be that there is a more appropriate solution? I am happy to alter/add to this PR further if people feel necessary.
Further information on this issue can be seen here: http://axotron.se/blog/problems-with-python-xbee-2-2-3-package/

@hansmosh This is the proposed fix for the break on error issue. I don't know if you had a more appropriate solution in mind?

The addition of the error_callback was a small addition that I made when I first took over this project. Originally, exceptions besides ThreadQuitException were uncaught which would un-gracefully exit the thread. My intention was to keep the same behavior of exiting the thread but allow the user to react to that with the error_callback.
https://github.com/nioinnovation/python-xbee/commit/73dffd17a8cdc0c13d502f3bfc7f0bd3703c5a78
For my use case, that made sense as it meant I needed to re-initialize my XBee. What behavior are you seeing? Are you seeing an exception that you are want to be caught but still let the thread continue to run?

Hi Chris,
Thanks for getting back to me. The particular issue for me was when the xbee received a packet type it did not recognise (in my case it was the xA1 route-record-indicator).  This caused the code to barf the exception and then break. Also see URL link example.
I would have expected more normal behaviour upon receiving a unrecognised packet would be to throw error message but continue running. Otherwise you run the risk of the code dying each time some weird/new packet type comes about.
In my case the xA1 packet issue was fixed by adding it to the Zigbee dictionary file (so for my specific needs it is OK), but I still think the issue still remains.
Thanks again for this great library!
Jim

On 7 Sep 2016, at 01:00, Chris Brackert<EMAIL_ADDRESS>wrote:
The addition of the error_callback was a small addition that I made when I first took over this project. Originally, exceptions besides ThreadQuitException were uncaught which would un-gracefully exit the thread. My intention was to keep the same behavior of exiting the thread but allow the user to react to that with the error_callback.
73dffd1
For my use case, that made sense as it meant I needed to re-initialize my XBee. What behavior are you seeing? Are you seeing an exception that you are want to be caught but still let the thread continue to run?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Got it. Why don't you re-push this commit with the break deleted instead of commented out. Go ahead and include a link to that blog post in the commit message if you want a reference to it somewhere.

Thanks Chris,
I removed the commented out #break line and I see you have now merged it in :-) Thank you.


Insulated screwdriver handle

Wera 817 VDE Kraftform handle

Features

Kraftform VDE holder
Suitable for Kraftform Kompakt VDE blades

Facts
Length: 175 mm
Weight: 2100 g
Country of origin: CZ
EAN:<PHONE_NUMBER>359
Links

Buy on ChadsToolbox for $12

WERA<PHONE_NUMBER>1 KRAFTFORM KOMPAKT 62I PH 2 X 154 MM INTER-CHANGEABLE BLADE (PHILLIPS) FOR KK VDE

Buy on ChadsToolbox for $5.00

WERA KRAFTFORM KOMPAKT 60I 1.0 X 5.5 X 154 MM INTER-CHANGEABLE BLADE (SLOTTED) FOR KK VDE

Buy on ChadsToolbox for $5.00


[Bug]: setcontent times out.

Minimal, reproducible example
const puppeteer = require('puppeteer');
const fs = require('fs');

async function generatePDF(content, outputPath) {
    const browser = await puppeteer.launch();
    const page = await browser.newPage();
    await page.setContent(content);
const.log("it reaches here...");
    const pdf = await page.pdf({ format: 'A4' }); // this crashes in the latest
console.log("but not here...");
    await browser.close();
    fs.writeFileSync(outputPath, pdf);
}

(async () => {
    // Generate two PDFs
    await generatePDF('
First PDF
', 'first.pdf');
    await generatePDF('
Second PDF
', 'second.pdf');

    console.log('PDFs generated');
})();

Error string
async timeout
Bug behavior

[ ] Flaky
[X] PDF

Background
No response
Expectation
it should create a pdf as it does in "puppeteer": "^22.3.0",
Reality
the app times out after 30000ms
Puppeteer configuration file (if used)
No response
Puppeteer version
22.9.0
Node version
20.11.1
Package manager
npm
Package manager version
10.5.0
Operating system
Windows

Probably duplicate of https://github.com/puppeteer/puppeteer/issues/12470


This repo is private, yet it is linked from the public docs!

The public documentation at https://os.mbed.com/docs/latest/tutorials/quick-start-offline.html ...
has a link to https://github.com/ARMmbed/mbed-cli-windows-installer/releases, which is private.
Thus, 99% of customers cannot access it!
Windows

Install Mbed CLI
Download and run the Mbed CLI Windows .exe installer

cc @BlackstoneEngineering  @AnotherButler

This repo is public. Closing.


Switch Option to only display current system findings when moving from system to system.

Would it be possible to have a switch option to only display the current systems findings, I know you try and refresh the current system findings, but once moved to a new system, the option to refresh the list with only any new findings, all previous one removed.
Thanks.

The original observatory actually had a "clear list on new system" option which hasn't been implemented yet since the rewrite. It will have to be implemented for Core on a per-plugin basis, but I can accept the issue here and make it happen for Explorer at the very least.

Hello, would absolutely love this functionality. Having Explorer show only the system I'm currently in would be awesome. :)


feat(preset-mini): support Functionality transition-[...]

close: #1303

Refactor the transition parse logic, see #3924


OCPVE-677: chore: separate filter and lvm package from vgmanager

Part of the findings of vgmanager testing. This separates vgmanager into 3 packages:

lvm, now contains ALL calls to LVM in the codebase
filter, now contains our filter methods and gets included from vgmanager
vgmanager, now only calls to lvm and filter in the correct places

Also the previously raw command executor calls for thinpool creation/extension were moved into separate methods in lvm package so that the perogative to contain all lvm calls still remains.

/test lvm-operator-e2e-aws-sno

/test lvm-operator-e2e-aws-sno

/test lvm-operator-e2e-aws-sno

I loved this!
/lgtm
/approve

/test lvm-operator-e2e-aws-sno

/test lvm-operator-e2e-aws

Codecov Report

Merging #410 (e0a0dd1) into main (8da92ee) will decrease coverage by 1.54%.
Report is 6 commits behind head on main.
The diff coverage is 10.71%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #410      +/-   ##
==========================================
- Coverage   55.01%   53.48%   -1.54%
==========================================
  Files          27       27
  Lines        2341     2468     +127
==========================================
+ Hits         1288     1320      +32
- Misses        965     1062      +97
+ Partials       88       86       -2

Files Changed
Coverage Δ

pkg/filter/filter.go
61.22% <0.00%> (ø)

pkg/lvm/lvm.go
38.25% <0.00%> (ø)

pkg/vgmanager/status.go
0.00% <0.00%> (ø)

pkg/vgmanager/vgmanager_controller.go
11.41% <5.00%> (+1.91%)
:arrow_up:

pkg/vgmanager/devices.go
75.78% <33.33%> (-0.60%)
:arrow_down:

... and 1 file with indirect coverage changes


Add Culture of Insight blogdown site

I recently rebuilt the website using blogdown and have been blogging about data analysis and visualisation using R so thought you might want to include!
Thanks.

Nice looking site, thanks for the submission!

thanks Mark!


Icons break, when the token is refreshed in a hidden iframe

Bug, feature request, or proposal:
Sometimes we see for the icon text (for instance add or file_download) instead of the actual icon. This seems to happen after the oidc-client library triggers an invisible iframe refresh with the identity server loaded, to reload a new JWT.
Icons looked fine before, suddenly some of them are replaced with text + elements get duplicated. After resizing the page slightly or even just clicking somewhere, everything looks normal again.
What is the expected behavior?
For it to work normally all the time. Example how a page looks correctly:

What is the current behavior?
Sometimes (very hard to reproduce) it suddenly looks like this:

after slight resizing of the window or even just clicking somewhere, it looks correctly again.
What are the steps to reproduce?
We had some ideas about library dependencies and not cleaned up node_modules folders, based on which we thought we could reproduce it, and thought we fixed it, but it keeps coming back, and now we don't have any idea how to reproduce it anymore.
If it happens, it seems to be only happening after the iframe for JWT token refresh is reloaded (we are using the library oidc-client) but just because the iframe was reloaded (happens every 5 minutes) doesn't mean, it will break. This is what such a reload looks like roughly:

This one broke it icons, but it looks the same if it doesn't break them.
Which versions of Angular, Material, OS, TypeScript, browsers are affected?

angular version 5.2.11
material version 5.2.5
OS: observed it on mac and windows
TypeScript version 2.6.2
browsers: observed it on latest firefox and latest chrome

Is there anything else we should know?
Maybe this is a known issue with icon fonts in general, which has a very easy fix, we just don't even know what to look for online / how to describe it.

If you're seeing the text instead of the icons, it means either:

The icon font is not loaded, or
The icon font css class isn't being applied to the mat-icon element

Are you able to determine which of those is occurring?

@jelbourn as you can see, when the screen breaks, the elements with the broken icons get duplicated and the last element has the correct icon so ...

the icon font is not loaded - I don't think this is possible, just clicking on the screen fixes it immediately (seemingly nothing happens according to the developer tools in chrome or firefox)
the icon font css class isn't being applied - this might be some hint. Maybe the issue isn't the icon isn't displayed, but the actual issue is that those elemets get duplicated.

It can happen that elements get duplicated multiple times (while this bug is getting worse and worse) until the user clicks on the screen. At this time only the last element is rendered correctly. All other elements are rendered without the specific styles being applied. I also just noticed on the screenshot I provided, that the broken Export button is in black, while the correct one is orange. So it doesn't seem to be just the icons, but the whole CSS breaking. - Thx for the hint, we never noticed that before, because we were so fixated on the icons.
So probably the questions should change more to:

Would you have any idea what could cause some elements to be duplicated?

If you're getting just regular elements duplicated, it doesn't seem like anything related to Angular Material. I'd suggest looking at whether some async operation in your app is escaping the Angular zone.

@jelbourn yeah sry, we discussed it internally, and we do have a problem with the oidc library and e2e tests as well, because it seems to be constantly running an async loop in the background which means when our protractor tests are waiting for angular to complete they never execute.
Therefore your hint with the async operations and Angular zone seems a reasonable explanation (although really hard to verify - at least for me).
Thank you


Greaseweazle writes floppies correctly (?) but on real hardware (Amiga) there are errors

Hello
I had a situation several times when Greaseweazle wrote a disk from an .adf file but no error was displayed. After inserting this disk into a real Amiga and checking in X-COPY there are errors on the disk (6 and 8). After changing to a working disk it is OK. It seems that the program and/or the device does not verify the correctness of the write. When I connect Greaseweazle to the WinUAE emulator it recognizes errors on the disk. Is it possible to set Greaseweazle in such a way that when the disk is damaged the program verifies errors on the disk?
My equipment is Greaseweazle F1 Blue Pill. The program I use to write is Greaseweazle 1.20 + GreaseweazleGUI.

Writes are verified. You should see "all tracks verified" at the end of the write. Possibly one of your drives is slightly out of alignment. You could also try reading one of these disks back to adf on Greaseweazle after observing an issue on Amiga, and see whether Greaseweazle can read it or not.

The message "all tracks verified" appears every time, even with damaged disks.
It is probably not out of adjustment, because I have already checked 3 different disk drives. Besides, if the heads were set incorrectly, there would be no errors on the WinUAE emulator in X-COPY, but there are errors.
The same thing happens when formatting a diskette. It looks as if the Greaseweazle 1.20 program formatted without verification (the formatting speed is high, as if without verification). Additionally, I started the G-COPY program and it already recognizes errors when formatting.

It does verify. What command do you execute to "format" a disk?

The command that is executed is:
"C:\Amiga\Greaseweazle\greaseweazle-1.20\gw.exe" erase
If there is any other command to format a diskette with verification, let me know.

In short, if you "gw write" an ADF from the command line it will be verified unless you explicitly specify --no-verify. If you have concerns over GreaseweazleGUI behaviour please raise them with the gui's maintainer.

But this is not a GUI issue. Without the GUI I type "gw.exe erase" in cmd and it is exactly the same. It formats but does not see errors. On the same device on G-COPY or in the WinUAE emulator under X-COPY the errors are visible.


let's add --oneline option to the rewrite command

Now, the rewrite command produces a pretty-printed formula. Sometimes it's more desirable (especially for tests) to get it in one line. Let's add --oneline option, making it print the entire formula in one line.

@deemp please, help

@yegor256, please provide some examples of single-line output.

@yegor256, is this output OK?
{ ⟦ c ↦ Φ.org.eolang.float ( as-bytes ↦ Φ.org.eolang.bytes ( Δ ⤍ 40-39-00-00-00-00-00-00 ) ), result ↦ ξ.c.times ( x ↦ ⟦ Δ ⤍ 3F-FC-CC-CC-CC-CC-CC-CD ⟧ ) .plus ( x ↦ ⟦ Δ ⤍ 40-40-00-00-00-00-00-00 ⟧ ), λ ⤍ Package ⟧ }

I'll merge https://github.com/objectionary/eo-phi-normalizer/pull/588 and make a release, and we can discuss further changes then.

@deemp yes, looks good

@deemp works fine in 2.2.2, thanks!


CRI Logging Driver

Fixes #246, #213
Replaces #224
Proposed Changes
This PR introduces a CRI logging driver as an alternative solution to #213. This driver outputs docker logs in CRI formats. Docker has dual logging capability that will take care of also outputting the logs in JSON format.
The driver creates a CRI logger to output CRI formatted logs to the location specified by kubelet. This logger will exist within cri-dockerd. i.e. cri-dockerd will handle it's current GRPC messages as well as the logger messages from dockerd
The driver will need to be installed with cri-dockerd and specified in cri-dockerd's code when it creates a container along with the kube file location. The kube file location is passed to the driver to eliminated the need to create a kube client within the driver when it already exists in cri-dockerd.
Changes Still Needed

Build and install the driver with cri-dockerd
The driver will now run as a part of cri-dockerd. My thoughts are to have cri-dockerd tell dockerd it is installed
using something like https://docs.docker.com/engine/api/v1.43/#tag/Plugin/operation/PluginPull.
Change cri-dockerd to specify the driver and kube path when creating a container
Create documentation

Docker has dual logging capability that will take care of also outputting the logs in JSON format.

I've tried dual logging before, using the fluentd driver, and for dual logging docker used the "local" driver as a "dual logging cache" instead of the json-file driver. This broke kubectl logs, since it was looking for the json-file logs which the "local" driver does not produce. Meanwhile, docker logs was still functional, since it was apparently able to work with the "local" driver.

@zmedico The current logging code for cri-dockerd creates a symlink to the json log file created by docker in the log file location that is expected/designated by kubectl. This code would have to stay in place for the dual logging to work. My guess is that the fluentd driver does not create this symlink and therefor kubectl cannot find the log file when it tries to read the logs.

@zmedico The current logging code for cri-dockerd creates a symlink to the json log file created by docker in the log file location that is expected/designated by kubectl.

Yes, I've found the createContainerLogSymlink function in cri-dockerd/core/logs.go, and from the cri-dockerd debug log it seems that the path variable refers to a json-file style path that does not exist when the fluentd log driver is used. Ultimately it logs an Unsupported logging driver by CRI message since the IsCRISupportedLogDriver function returns false for any driver except json-file. The only log file that actually exists for the container is named container-cached.log and is a different format that docker's local logging driver uses for dual-logging (decoder found in moby/daemon/logger/local/read.go).

This code would have to stay in place for the dual logging to work. My guess is that the fluentd driver does not create this symlink and therefor kubectl cannot find the log file when it tries to read the logs.

My fear is that using your criLogDriver will give similar results, since docker will still use its local logging driver for dual-logging, and the createContainerLogSymlink will behave approximately the same as it does with the fluentd driver.
The "fallback" method that you mention in https://github.com/Mirantis/cri-dockerd/issues/246#issue-1925115428 would avoid this problem, since docker would continue to use the json-file log driver which is apparently the only log driver compatible with the createContainerLogSymlink function.


Handshake failure with PM2 cluster mode

Hi,
Is this supposed to solve the handshake problem with multiple instances?
Sometimes it is connected at first try and some other times it just keep trying and it may establish connection after few tries...
VM5983:35 WebSocket connection to 'ws://localhost:3000/socket.io/?EIO=3&transport=websocket&sid=Fpn-vkHqf6k6fypKAAAE' failed: Connection closed before receiving a handshake response

Thanks.

No, this module only allow broadcasting between nodes.

@darrachequesne   Can you guide me on this? So far no luck in solving it and really don't want to use sticky session solution.
I need to have cluster + socket.io, but hand shakes fails.

You have to have sticky sessions, so handshake requests get routed to the same socket process. If not, then the handshake will fail.

@tectual there has been some work in that direction (https://github.com/socketio/engine.io/pull/416), but it's a work in progress.
The thing is that the handshake is not the only step that might fails in a multi-node configuration. The messages waiting to be sent to the client must also be shared, since the client can reach any node in polling mode. And the data received from the client must be redirected to the server "owning" the client, where the callbacks are registered.

@jreeter I thought sticky sessions are not generally good (Or let's say not the best way to load balance). I hoped I can avoid using it. But seems thats the only option left out.
Thanks @jreeter and @darrachequesne.


Fill CS-03 Positions

Request CS-03 pools
Conduct review and create short list
Review short list with Gab
Conduct informal interviews.
Engage resource.

Objectives:
Find

Strategists / Innovators
Cloud Design and DevOps/Pipeline Experts

Objectives:
Find

Strategists / Innovators
Cloud Design and DevOps/Pipeline Experts

Engaged James Santos


🚀 Feature: implement high fidelity designs

make our site look pretty
Use Case

Proposed Solution

[ ] :wave: I may be able to implement this feature request

This is a :rocket: Feature Request

Closing because switching backend to contentful


TopShelf will not start when there are space characters in its install path

I'm on a Windows 10 environment
Steps to reproduce:

Copy your TopShelf based service to a folder somewhere with no space characters in the path and observe that the service starts fine.
Copy your TopShelf based service to a sub folder under either 'C:\Program Files' or 'C:\Program Files (x86)' (or any other folder with a space in the path for that matter)
On cmd prompt:

C:\Program Files (x86)\Your Service Folder> <Your Service Host>.exe install
C:\Program Files (x86)\Your Service Folder> <Your Service Host>.exe start

Result:
Topshelf.Hosts.StartHost Error: 0 : The service failed to start., System.InvalidOperationException: Cannot start service  on computer '.'. ---> System.ComponentModel.Win32Exception: The service did not respond to the start or control request in a timely fashion
--- End of inner exception stack trace ---
at System.ServiceProcess.ServiceController.Start(String[] args)
at System.ServiceProcess.ServiceController.Start()
at Topshelf.Runtime.Windows.WindowsHostEnvironment.StartService(String serviceName, TimeSpan startTimeOut)
at Topshelf.Hosts.StartHost.Run()

Why this closed? How you resolve this issue?


Another way that numbers don't add up on gene search in beta

Here's the search I did without using the filter:

It returned 11,279 genome features (also called GENES by almost everybody in the world):

When I clicked on that 11,279 genome features (aka GENES) it takes me to a table that says there are 11,460 genome features (i.e. GENES):

So I ask you...which number is correct?

Rats!  This one was supposed to be the final salvo!  Sorry...Its like when the percussion guy drops the cymbals 30 seconds before he's supposed to play them.

I'm not sure why the numbers don't match yet, but you also seem to be of the opinion that we should not be calling things Genomic Features.  I'm fine if we want to change that, but I thought that was discussed and debated years ago and I have no opinion on that other than what we call them should be consistent throughout the site.

Maybe, the number discrepancy came from the query "feature_type:!source" in the global search. I am more leaning toward removing source features. @mshukla1 what do you think?

I think you are probably correct about global search excluding source features and feature list including them.

fixed on beta.


This bookmarklet is still very relevant because Slack's preference to turn off wysiwyg breaks copy pasting with markup preserved

Hi @kfahy , thanks a lot for putting this together.
As of now, the readme of the project says:

The contents of this repository and the remainder of this README are left intact for reference, but should not be used going forward.

However, turns out, it's not quite the case at least for me personally: even though Slack has implemented a preference to turn off wysiwyg, which is cool, but, they broke copypasting with markup preserved!
E.g. I select a text with markup, I copy it, I paste it to the Slack's input, and all the markup is gone. It used to be preserved, and I personally use it a lot.
But the good news is, your bookmarklet does save the day again. With this bookmarklet applied, I select a text with markup, I copy it, I paste it to the Slack's input, and the markup is preserved!
So, just wanted to say that it's probably worth mentioning in the readme.
Thanks again!

Keyboard shortcuts don't work in the new simple editor, e.g. ⌘B to make text bold.
Old editor can still be enabled with this script and that makes it still very much useful.


Fix Hall sensors input

Looks like hall sersor input is wrong.

[x] Check if rotating sequence is correct
[x] Check if interrupts are working correct
[ ] Make hall input scheme on pcb

Sometimes appear behavior like this  (0 = A-B, 1 = A-C, so on).
Looks like problem is in shitty breadboard, not in code.

Hall sensors connection scheme.


Unable to complete chain rescan: MsgTx.BtcDecode

Background
2020-05-18 20:12:46.639 [INF] LTND: Waiting for chain backend to finish sync, start_height=630894
2020-05-18 20:12:47.590 [DBG] LNWL: Chain backend synced to tip!
2020-05-18 20:12:47.622 [INF] LNWL: Started rescan from block<PHONE_NUMBER>00000000091dba4418995d092305179439b5b340b409791e43224d (height 626971) for 44 addresses
2020-05-18 20:12:47.672 [ERR] LNWL: Unable to complete chain rescan: MsgTx.BtcDecode: witness tx but flag byte is 00
When I initially got this error I updated from version: 0.8.0-beta commit=v0.8.0-beta-211-g4d7a151b4763f5893c91c4f6b49316391db17f73 to 0.10.0-beta but updating didn't fix it.
I then tried a rescan using dropwtxmgr and after several hours the rescan failed with the same error at the same height.
Your environment

0.10.0-beta commit=v0.10.0-beta-137-g4d3bd6f178e541b1c99ea05b4e16607e129fc499
Linux ATC 4.15.0-72-generic Ubuntu SMP Tue Nov 26 12:20:02 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
bitcoind

Steps to reproduce
$ lnd
Expected behaviour
lnd starts and syncs to chain and graph, connects to peers, channels open
Actual behaviour
Lnd fails to rescan and sync with backend. No channels open. No peers available.

What version of bitcoind are you using?

From the error, it looks like we're expecting a witness transaction, but then there ends up being no witness data.

v<IP_ADDRESS>-g2472733a24a9364e4c6233ccd04166a26a68cc65

What version of bitcoind are you using?
@Roasbeef  Thanks for the response! I was using v<IP_ADDRESS> and just updated to v0.19.1 and now I get this:

[ERR] LNWL: Unable to complete chain rescan: -1: Block not found on disk

not sure yet about the witness decoding, but

[ERR] LNWL: Unable to complete chain rescan: -1: Block not found on disk

is your node pruned?

not sure yet about the witness decoding, but

[ERR] LNWL: Unable to complete chain rescan: -1: Block not found on disk

is your node pruned?

No

It looks to be a corrupted/missing block.
$ bitcoin-cli getblock "00000000000000000002ea23db68d5a523bcf9cd293d9f1e6cfc60d372d58f09"
error code: -1
error message:
Block not found on disk
Doing a -reindex on my bitcoind instance now and closing this as it's not an LND issue.
Thanks! @Roasbeef @cfromknecht


Enable leader election in manager

Signed-off-by: Chuck Ha<EMAIL_ADDRESS>What this PR does / why we need it:
This PR allows the manager to work with with the default --enable-leader-election flag that kubebuilder has added to the Deployment YAML.
Special notes for your reviewer:
Release note:
Enable highly available manager

/lgtm


Bump to version 0.7.0

Since at least https://github.com/containers/oci-spec-rs/commit/ad7b406a0dc627b6c4ec226c1b3163cd6083afc5 broke semver.

@cgwalters do we need this? We have our https://github.com/containers/oci-spec-rs/blob/main/hack/release script which should take care of bumping the version.


Adding methods to locate folders and files in VS installs

VC build needs to be able to use toolsets installed by older VS versions (native multitargeting). Before VS2017 the vc targets locations were written in the registry. To figure out Vs2017 install locations we have to call the installer API.  There also might be several instances on the machine and we need to be able to use a toolset installed in any of the instances.
Here is how the added functions are going to be used in cpp props:
<PropertyGroup>
    <_VCTargetPath15Folders>$([Microsoft.Build.Utilities.ToolLocationHelper]::GetFoldersInVSInstallsAsString('15.0', '16.0', 'Common7\IDE\VC\VCTargets'))</_VCTargetPath15Folders>
    <_VCTargetsPathForToolset Condition="'$(_VCTargetPath15Folders)' != ''">$([Microsoft.Build.Utilities.ToolLocationHelper]::FindRootFolderWhereAllFilesExist($(_VCTargetPath15Folders), $(_RelativeToolsetFiles)))</_VCTargetsPathForToolset>
  </PropertyGroup>

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

Would it make better sense to merge these two APIs into a single one like ResolveFilesAgainstVSInstances which gets a set of relative files and returns the set normalized against the first VS instance which contains all of them?

I am planning to use FindRootFolderWhereAllFilesExist for 14 and older VS versions  as well as additional paths specified for a project (new feature in Dev16), so I'd like to have it separately from VsInstalls method.

I thought about unittests, but I could not figure out how I actually do anything meaningful there. Can you help?

Or are you saying to add a test for Unix and make sure an empty list is returned?


reloading data - RuntimeError: self.samples is missing, please run self.export_posterior() first

[x] I have confirmed this bug exists on the latest version of cell2location. See https://github.com/BayraktarLab/cell2location#installation
[x] I follow the instructions from the scvi-tools tutorial.

I try to import my model and h5ad that was already trained, it is able to import correctly but cannot be used for downstream analysis. For example, even when I attempt to plot QC, i return an error:
RuntimeError                              Traceback (most recent call last)
[<ipython-input-11-bee3b2ef217e>](https://localhost:8080/#) in <module>
----> 1 mod.plot_QC()

[/usr/local/lib/python3.7/dist-packages/cell2location/models/base/_pyro_mixin.py](https://localhost:8080/#) in plot_QC(self, summary_name, use_n_obs)
    516
    517         if getattr(self, "samples", False) is False:
--> 518             raise RuntimeError("self.samples is missing, please run self.export_posterior() first")
    519         if use_n_obs is not None:
    520             ind_x = np.random.choice(

RuntimeError: self.samples is missing, please run self.export_posterior() first

What could be happening?

For most of the downstream analysis, you don't need to load the scvi-tools model - only the anndata object. However, to run mod.plot_QC() you need to do exactly what the error asks: please run self.export_posterior() first.

Ok thank you. My other issue I am running into is the ability to do NCEM. When I run:
'''# Compute expected expression per cell type
expected_dict = mod.module.model.compute_expected_per_cell_type(
mod.samples["post_sample_q05"], mod.adata_manager
)
Add to anndata layers
for i, n in enumerate(mod.factor_names_):
adata_vis.layers[n] = expected_dict['mu'][i]
Save anndata object with results
adata_file = f"{run_name}/sp.h5ad"
adata_vis.write(adata_file)
adata_file
'''
I get the error:
RegressionBackgroundDetectionTech model with the following params:
n_factors: 23
n_batch: 1
Training status: Trained
AttributeError                            Traceback (most recent call last)
 in
1 # Compute expected expression per cell type
----> 2 expected_dict = mod.module.model.compute_expected_per_cell_type(
3     mod.samples["post_sample_q05"], mod.adata_manager
4 )
5
1 frames
/usr/local/lib/python3.7/dist-packages/torch/nn/modules/module.py in getattr(self, name)
1206                 return modules[name]
1207         raise AttributeError("'{}' object has no attribute '{}'".format(
-> 1208             type(self).name, name))
1209
1210     def setattr(self, name: str, value: Union[Tensor, 'Module']) -> None:
AttributeError: 'RegressionBackgroundDetectionTechPyroModel' object has no attribute 'compute_expected_per_cell_type'
I was also getting a similar error last night saying cell2location has no attribute "samples"

To generate cell-type specific RNA count predictions as below:
expected_dict = mod.module.model.compute_expected_per_cell_type(
mod.samples["post_sample_q05"], mod.adata_manager
)

you indeed need to runmod.export_posterior() after loading the model.

AttributeError: 'RegressionBackgroundDetectionTechPyroModel' object has no attribute 'compute_expected_per_cell_type'

Please make sure that you are loading the cell2location model using anndata object for Visium data - not the regression model with Visium anndata.

Thank you this worked well!


Fixing demo issues

Fixing issues with popovers and adding missing bootstrap styles. Let's see if it will work this time around.

Hey! What do you know, it works:-) Still a small issue with caching rule's descriptions, but I'll fix it later.

Also looks like a bit of a styling issue as you hover over the rules.
But still, looks great!

Checkboxes jumping on hover you mean? That looked like bootstrap core
issue. Couldn't figure it out, but I'll take another look.
On Mar 8, 2015 1:12 PM, "Nicholas C. Zakas"<EMAIL_ADDRESS>wrote:

Also looks like a bit of a styling issue as you hover over the rules.
But still, looks great!
—
Reply to this email directly or view it on GitHub
https://github.com/eslint/eslint.github.io/pull/29#issuecomment-77761978
.


Missing albanian langauge

There is no option for albanian language on the "Language of the name unknown" mission. Please add the option to input albanian as a language.
Thanks in advance

Dublicate https://github.com/kort/kort-native/issues/48
Guys! Could you add more information about 'how' add more languages to https://github.com/kort/kort-core?

Thanks for your suggestion. Kort translation is based on Transifex.
Kort development has stalled end of 2017 and there are currently not maintainer nor dev resources available on this project. Around October 2018 there will be some shiny activities in order to resurrect this nice game experience.
So, if you have some spare time, you can head over to https://www.transifex.com/odi/kort/ and start translating right now. Transifex has nice docs. and help pages.

Closing this issue because it relates to another resource. You can still reopen it if you have questions while translating.


Задокументировать работу с popups

:tada: This issue has been resolved in version 7.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Integration branch: Update Script Error

Symphony Fatal Database Error
No database selected
An error occurred while attempting to execute the following query
SELECT SQL_CACHE t1.name, t2.page, t2.delegate, t2.callback FROM `extensions` as t1 INNER JOIN `extensions_delegates` as t2 ON t1.id = t2.extension_id WHERE t1.status = 'enabled'
Backtrace:
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/toolkit/class.mysql.php:461] MySQL->__error();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/toolkit/class.mysql.php:612] MySQL->query();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/toolkit/class.extensionmanager.php:55] MySQL->fetch();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.symphony.php:227] ExtensionManager->__construct();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.symphony.php:122] Symphony->initialiseExtensionManager();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.frontend.php:45] Symphony->__construct();
[/Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.frontend.php:33] Frontend->__construct();
[/Users/designermonkey/Projects/Symphony/install-test/update.php:124] Frontend::instance();
Database Query Log:
SET character_set_connection = 'utf8', character_set_database = 'utf8', character_set_server = 'utf8'; [0.0001]
SET CHARACTER SET 'utf8'; [0.0001]

The above occurs at running the script.
Afterwards, the backend throws this error
Catchable fatal error: Argument 1 passed to DateTimeObj::setSettings() must be an array, null given, called in /Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.symphony.php on line 111 and defined in /Users/designermonkey/Projects/Symphony/install-test/symphony/lib/core/class.datetimeobj.php on line 37
Call Stack
#	Time	Memory	Function	Location
1	0.0002	337320	{main}( )	../index.php:0
2	0.0011	360428	renderer( )	../index.php:25
3	0.0141	2356844	Administration::instance( )	../index.php:13
4	0.0141	2357276	Symphony->__construct( )	../class.administration.php:55
5	0.0145	2363484	DateTimeObj::setSettings( )	../class.symphony.php:111

You don't have a config.php :)


python311Packages.intake-parquet: 0.2.3 -> 0.3.0

Diff: https://github.com/intake/intake-parquet/compare/0.2.3...0.3.0
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 235021 run on x86_64-linux 1

4 packages built:

python310Packages.intake
python310Packages.intake-parquet
python311Packages.intake
python311Packages.intake-parquet


Fix SQL Server 2017 error during migration: "There is already an obje…

"There is already an object named 'user_group' in the database."
by simply renaming the index name

Where is that user_group object coming from?

That's a good question!
I did search inside all the sys objects of SQL Server and didn't find anything. Probably an existing index name linked to the Security in SQL Server?
the error happens when installing Rainlab.User during the migration on a clean oc install.

@chrisvidal is the database completely clean too when installing?

after OC installation,. SQL Server 2017 has created some objects here and there. I am not an expert on SQL Server at all, so I am not sure what it is creating. But there seem to have some security stuffs, some sys objects, etc...

@chrisvidal I've just done a quick search on the SQL Server docs and there's nothing to indicate that user_group is in any way a special term, so it seems more likely that perhaps your database is not clean.

I did some research as well and didn't find anything.
The db was all brand new.
we installed Rainlab.Blog to test the installation of group_concat for SQL Server 2017 and it was successful,
then we installed Rainlab.User and find this migration error.

The only think I can think of is if SQL Server 2017 has a global database namespace for combined primary key indexes since the backend user system also has a combined primary key index with the same columns provided as input. In most other DB drivers however since those are in different tables it shouldn't cause an issue. https://github.com/octobercms/october/blob/develop/modules/backend/database/migrations/2013_10_01_000003_Db_Backend_Users_Groups.php#L14.
If that's the case, then we might need to make some changes to the SQL Server grammar so that doesn't cause this issue.
@chrisvidal can you create a new empty database, and this time before you run any of the migrations just make this PR's change to the migration file for the backend that I linked above and then see if adding the user plugin without this PR's changes works?

Ooo that would make total sense @LukeTowers
Alright I will give it a try, I will have access remotely to the server only tomorrow.

@LukeTowers as I am using the wizard installer, when is the good time to make the change in the migration file?

@chrisvidal just delete the database and recreate it after the installer has finished, and then run php artisan october:up after making changes to the migration file

@LukeTowers you were right, after updating the backend user group migration, the Rainlab User plugin installation is smooth again.

@chrisvidal Classic Microsoft :(.
@LukeTowers given the above, I think changing the primary key group name should be fine. It shouldn't affect anyone who's already installed the plugin without any issues - we'll just have to note it down as a comment if we ever have to migrate that table further that the index might be called one of two names. Off the top of my head, I don't think there's anything specific in the grammar that we can change.

Hmm, according to https://docs.microsoft.com/en-us/sql/t-sql/statements/create-index-transact-sql?redirectedfrom=MSDN&view=sql-server-2017 index names only have to be unique within the table or view, but this issue is happening even though both indexes exist within separate tables. According to https://nakulvachhrajani.com/2016/07/11/0376-sql-server-msg-2714-there-is-already-an-object-named-tablename-in-the-database/, this might be an issue with Deferred Name Resolution if both queries were being processed at the same time, but AFAIK they are run separately within the database.
@chrisvidal can you try installing October into a clean database as one action (with no modifications) and then installing the RainLab.User plugin as a second separate action?
(Note for the future, the command is defined in Illuminate\Database\Schema\Blueprint->indexCommand())

Hmm, according to https://docs.microsoft.com/en-us/sql/t-sql/statements/create-index-transact-sql?redirectedfrom=MSDN&view=sql-server-2017 index names only have to be unique within the table or view, but this issue is happening even though both indexes exist within separate tables. According to https://nakulvachhrajani.com/2016/07/11/0376-sql-server-msg-2714-there-is-already-an-object-named-tablename-in-the-database/, this might be an issue with Deferred Name Resolution if both queries were being processed at the same time, but AFAIK they are run separately within the database.

Reminding that we are using SQL Server 2017 on Windows 2019 here.

@chrisvidal can you try installing October into a clean database as one action (with no modifications) and then installing the RainLab.User plugin as a second separate action?

That is the process that we followed during the 1st installation and that raised the SQL State exception from SQL Server.

Reminding that we are using SQL Server 2017 on Windows 2019 here.

That's what the docs link is for is it not?

That is the process that we followed during the 1st installation and that raised the SQL State exception from SQL Server.

Hmm, that most likely rules out the deferred name resolution then.
Is there any non-standard configuration being used that would cause the index names to not be scoped to the table they apply to?

Reminding that we are using SQL Server 2017 on Windows 2019 here.

That's what the docs link is for is it not?

yes, I was referring to the article dated from 2016, I was wondering if SQL 2917 was out already,

That is the process that we followed during the 1st installation and that raised the SQL State exception from SQL Server.

Hmm, that most likely rules out the deferred name resolution then.
Is there any non-standard configuration being used that would cause the index names to not be scoped to the table they apply to?

I have no idea, the server is set up by another hosting vendor that I do not have access to for now.

Closing as it has been over a month since any activity on this occurred and we are trying to figure out what issues are still relevant. If this is still something that you would like to see through to fruition please respond and we can get the ball rolling.

Will come back to that in about one month.

Thanks mate!


java file first line blank when using java package

command:
.\jsonschema2pojo -s .\config.json -p com.demo -T JSON -t config

produce:
first line is blank

package com.demo;

import com.fasterxml.jackson.annotation.JsonInclude;
import com.fasterxml.jackson.annotation.JsonProperty;
import com.fasterxml.jackson.annotation.JsonPropertyOrder;

how to resolve this?

Sorry, we rely on the Sun/Oracle CodeModel library to generate these files, we don't really have fine-grained control over this. If it's really bothering you you'll have to process them to remove leading newlines yourself. I suggest though that you don't worry about micro-managing this output.


Copy existing Sinopia Resource

Adds a new Copy link to the Editor tab:

Clicking on the Copy link creates a new message that the resource is copied:

When a pre-existing (or saved) resource is loaded into the editor tab:

Clicking on the Copy link removes the resourceURI from state and the old URI is displayed in the tab:

Getting testing errors in __tests__/components/editor/property/InputValue.test.jsx and __tests__/components/editor/property/PropertyPanel.test.js that also are present in master branch.
Would close #1361

Also, I think you might need a few test tweaks as well.


City not parsed correctly from google in some cases

Hi there,
Can we include 'postal_town' as a fallback value for city/city_long, when 'locality' is not returned from Google? For example, 'Ullared' is not a 'locality' but a 'postal_town'. See below:

Similarly Southampton here:

Thanks!

Good catch, it looks like this issue would only happen in the UK, I don't think any other countries have postal_town as an attribute.
Let me know if it works, it should be fixed now.


Fixed collection and category behat tests

tasks:

[x] test coverage
[ ] gather feedback for my changes
[x] submit changes to the documentation

informations:

q
a

Fixed tickets
none

BC breaks
none

Documentation PR
https://github.com/sulu-io/sulu-docs/pull/26

@dantleech Can you have a look?

ok with me. we need to make better Behat scenarios generally, but that is out-of-scope here.


Cleaned up demo sections

Proposed changes
Simplified and cleaned up demo sections.
Changed images to the new color scheme.
Removed redundant information and redirected to corresponding deployment section.
There might be many files changed due to update in links.
Fixes
If it fixes a bug or resolves a feature request, be sure to link to that issue.
Types of changes
What types of changes does your code introduce? (A breaking change is a fix or feature that would cause existing functionality and APIs to not work as expected.)
Put an x in the box that applies

[ ] Non-breaking fix (non-breaking change which fixes an issue)
[ ] Breaking fix (breaking change which fixes an issue)
[ ] Non-breaking feature (non-breaking change which adds functionality)
[ ] Breaking feature (breaking change which adds functionality)
[ ] Refactor (non-breaking change which changes implementation)
[ ] Messy (mixture of the above - requires explanation!)

Checklist
Put an x in the boxes that apply.

[ ] I have read the CONTRIBUTING doc
[ ] I am making a pull request against the main branch (left side). Also you should start your branch off our main.
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have locally run services that could be impacted and they do not present failures derived from my changes

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

Linters seems working now.


check package of zero-byte files

the libzip docs package on linux is filled with dozens of zero-byte files but it failed.  this is likely a bug somewhere.

this is a ravenadm issue, it extracts as expected with xrvn.

sorry, it is rvn, but not rvn-format.
Disk space required to install these packages: 9 MiB
[01/10]  |  |  |  |` autoselect-ssl~single~std~3                           [ok]
[02/10]  |  |  |` libressl~primary~std~3.8.4_1                             [ok]
[03/10]  |  |  |` xz~primary~std~5.6.2,1                                   [ok]
[04/10]  |  |  |` zlib~primary~std~1.3.1                                   [ok]
[05/10]  |  |  |` bzip2~primary~std~1.0.8_1                                [ok]
[06/10]  |  |` libzip~primary~std~1.11.1                                   [ok]
[07/10]  |` libzip~tools~std~1.11.1                                        [ok]
[08/10]  |` libzip~docs~std~1.11.1                                         FAIL
Installation failure detected! Remaining steps skipped.

fixed with new rvn-format version: https://github.com/Ravenports/rvn-format/releases/tag/v3.9


Introduce factories for the seeds

Introduce factories for the seeds

integrated fishery which provides clean and elegant solution for creating model factories easily (for seeding, testing, etc.)
created a PoC with the Person seeds to showcase the factories idea
minor refactoring to the seeds scripts

Note: This is just a proposal for an enhancement and a PoC applied to only one of the models. In case this approach is accepted, I'll proceed and create factories for the other models, so we'll end up with a clean interface for creating demo data for both seeds and tests.
// It's simple like that
const randomPerson = personFactory.build();

// Also, we can build up multiple items
const randomPersons = personFactory.buildList(5);

// We can provide custom values for some of the fields
const adminPerson = personFactory.build({
  firstName: 'Admin',
  lastName: 'Dev',
  email<EMAIL_ADDRESS>})

// It can also handle relations between models..

https://github.com/thoughtbot/fishery

Progress update:
Created more factories to be used for data seeds:

created factories for City, Country, CampaignType, Company, InfoRequest, Supporter, and Coordinator
refactor the essential seeds to use factories and simplify seeding
update seeds for the dev data to use factories (partially)

Note: Thank you for your approval @kachar, I plan to create factories for the rest of the existing models (used in the seeds) as well, but I left the most complex to be the last ones. Feel free to merge this (unless you want to wait for more approvals), so we don't pile up huge PRs with a lot of changes, because it will become difficult to review. I can create new PRs when I'm ready with the changes for the other models.

Great improvement @valkirilov with the factory pattern!

PS: Tests are failing here in the PR (maybe because of the forked repo?), but it's all green on my local:

Regarding the failing tests during github action - looking at the logs it seems you also need to commit the updated yarn.lock file after adding the fishery lib.

Regarding the failing tests during github action - looking at the logs it seems you also need to commit the updated yarn.lock file after adding the fishery lib.

Looking at the changelog it seems that the yarn.lock is updated properly and committed as part of this PR (or at least it contains a section with info related to fishery)

Regarding the failing tests during github action - looking at the logs it seems you also need to commit the updated yarn.lock file after adding the fishery lib.

Looking at the changelog it seems that the yarn.lock is updated properly and committed as part of this PR (or at least it contains a section with info related to fishery)

Well, it looks there is still an uncommitted change that is reported by the test run (see screenshot). May be you added fishery and committed the yarn.lock (which was ok), then removed the carret ^ from packages.json manually, but then didn't commit/push the yarn.lock again (which is where the diff comes from).

Mmm good point. Maybe that was the case :thinking:
Just updated again the yarn.lock file, but for some reason it's still failing :disappointed:


added Devlin's ssh key

Try rebasing and fixing the syntax errors.
https://github.com/bryanhonof/pxl-infra/actions/runs/9109024871/job/25040995297?pr=53


2022051101

Removals only
Closes #2963
Closes #4107
Closes #4608
Closes #4908
Closes #4932
Closes #4931 Mistake in #961
Closes #4926 Mistake in #937

Hello Team,
Will my cases will be closed and removed from deny list please?
Please tell the time please. You just mistakenly reported it and now I have to follow with you everyday to remove it .It is very disappointing actually .
Regards
Mo


Upgrade ramen to latest snapshotter crd and enforce VGS name length limit

This PR includes the following updates:

Update Ramen to use latest CRD from external-snapshotter

Integrated the latest CRD from external-snapshotter.
Updated dependencies and relevant configuration files.
Changed VolumeSnapshotRefList in VolumeGroupSnapshotStatus to PVCVolumeSnapshotRefList, which allows to map a VolumeSnapshot to the application PVC.

Ensure VolumeGroupSnapshot name contains at most 63 characters:

Added validation to ensure that the VolumeGroupSnapshot (VGS) name complies with the Kubernetes resource name limit of 63 characters.

LGTM, requeued the unit test run before merge.


Refactoring

DESC

clean up potentially critical code parts

TODOs

[x] write down todos according to risks

handled in #128 // duplicated


runtime: optionally (reliably) avoid netpoller

The gVisor project implements a user-space Kernel, and its implementation performance-sensitive, which forces a manual avoidance of the netpoller by avoiding certain APIs.
It would be nice to automate and enforce this avoidance, either by exposing some API that could be use to assert in a test that the netpoller has never been used, or by exposing a build tag that would guarantee that the netpoller is inactive. As of this writing it seems concretely that we want to avoid ever incrementing netpollWaiters.
cc @iangudger @nlacasse @prattmic @amscanne

This seems like the beginnings of a proposal but there’s no concrete next steps.
What changes would you like to be made specifically?
Thanks
@randall77 @ianlancetaylor

Do you want gVisor to never use the net poller full stop, or does this need to apply only to certain operations within gVisor?
The whole point of the netpoller is to be more efficient (particularly, in the # of OS threads needed) than just blocking an OS thread on each read/write. I'm curious in what circumstances netpoll needs to be avoided. Maybe we could solve that problem instead of this one.

We never want gVisor to use netpoll, full stop. One way of doing this that we have discussed is adding some way to detecting if it was ever used to the runtime and then running all of our tests and failing any which use netpoll.
golang.org/cl/78915 has more context on why netpoll is a problem.

Could you just check runtime.netpollInited? Or does that get set despite netpoll never being used?

golang.org/cl/78915 has more context on why netpoll is a problem.

That CL has been merged for a long time now. I don't see any comments there about other issues besides the one that was fixed in the CL.

Could you just check runtime.netpollInited? Or does that get set despite netpoll never being used?

Correct. runtime.netpollInited gets initialized when files are created with os.File regardless of whether netpoll is actually used.

golang.org/cl/78915 has more context on why netpoll is a problem.

That CL has been merged for a long time now. I don't see any comments there about other issues besides the one that was fixed in the CL.

The benefits documented in the CL are only if netpoll is not in use.

Correct. runtime.netpollInited gets initialized when files are created with os.File regardless of whether netpoll is actually used.

Bummer.

The benefits documented in the CL are only if netpoll is not in use.

So you want the 12% improvement to CPU usage that this CL provides? But you only get that 12% if you never use netpoll? Or are you interested in the 0.5% latency improvement?
How close is your app to those benchmarks?  They are really corner case benchmarks, with lots of very quick trips into and out of the poller/channel ops/scheduler, with no work on top of that.

It is mostly the CPU usage. gVisor is used in high-density environments. gVisor's CPU usage is much higher than Linux and we are currently looking into other options for reducing CPU usage as well.
Latency is important too though. We measure latency in nanoseconds and shaving even a few nanoseconds in a hot path can be a win for us.

How close is your app to those benchmarks? They are really corner case benchmarks, with lots of very quick trips into and out of the poller/channel ops/scheduler, with no work on top of that.

At the time golang.org/cl/78915 was written, it reduced total runtime of a Tensorflow model training benchmark running inside gVisor by 5% (and total CPU usage by 10%).
Tensorflow can be extremely futex heavy, as it coordinates very small units of work (size depends on the model) on a threadpool, where workers contend on resources. When an application calls futex inside gVisor and it actually blocks, that ultimately becomes a wait on a channel. Since new work is likely to be available very soon, this application becomes very sensitive to overall latency and CPU usage of the Go scheduler.
https://github.com/google/gvisor/issues/205 is a similar situation. In general, I think overall scheduler improvements (such as making netpoll cheaper) for these cases would be a possible alternative to an explicit API.

Ideally we'd like to prevent the netpoller from ever running, but I'd be happy just with a way to check whether the netpoller has run.
Here two proposals:

Add a new "implementation" of netpoll that just panics, and put that behind a nonetpoll build tag.

Add a boolean that is set to true whenever netpoll is called. Users that want to avoid netpoll could link against this boolean and check that it is false (likely inside tests).

Are there preferences/objections, or better alternatives to this?

From my perspective this is so special purpose that it's hard to get excited about having to maintain some publicly visible API for it.  I'm pretty skeptical that it would ever have more than one user.
We've talked about having some sort of runtime package stats access (#15490).  Perhaps we could make sure that those stats include some data on use the netpoller.  Then your tests could use that.


@Ignore frequently failing FaultToleranceMicrometerIntegrationTestCase

Tracked by https://issues.redhat.com/browse/WFLY-18162

Thanks @pferraro


add package: uni-vga

Unicode VGA font

Could you add license and platforms to meta.attributes?

Updated.

Manually merged, thanks!


[feature request] add net and event modules

I don't know how many native functions I need to add in order to connect to some TCP ports.
we need some demos like the ffi module, thanks.

we need some demos like the ffi module, thanks.

Are you referring to this FFI module?

I don't know how many native functions I need to add in order to connect to some TCP ports.

You would likely want to find a C or C++ library that handles networking. If you don't already have one in your firmware, I can recommend trying Mongoose. You can then integrate with the Mongoose API using something like the Microvium FFI example library.
Depending on your level of experience, you could wrap Mongoose into a Microvium library for others to use as well. Similar to how the Microvium FFI example library is able to produce C++ code at compile time and then link to it at runtime, a Mongoose wrapper library can "generate" the Mongoose C++ code (i.e. just fs.writeFileSync to dump the Mongoose files to disk) as well as all the JavaScript-to-C++ bindings. I'll probably do this at some point if nobody else does, but since it doesn't require any additional engine capabilities, it's a library that in theory anyone can write (and I'm happy to advise and help with the design).
Having said that, even if you just do basic integration for your own project, I'm happy to help. Just let me know where you get stuck.

@radiocom, it's not really clear what the right abstraction is for your use case, without knowing what it is.  Generally, anything that has raw access to TCP sockets needs to do buffering of messages and parsing.  This is likely to stress the Microvium heap quite a bit.  Exposing higher-level network protocols requires a lot less work in the JavaScript, but more on the integration side.  For example, something like MQTT might be exposed as a simple callback for handling an event and a function for publishing to a specific node, so a small amount of JavaScript on the device can drive a lot of policy and not require you to reimplement an efficient C/C++ MQTT client library in interpreted JavaScript.


Why baseURL in axios.js if you have one in main.js

Thank you for this great template. It's very helpful. I connected this template to my backend server and have a question - Why I need this in axios.js:
https://github.com/ClintOxx/vuetify-admin-dashboard/blob/ba249b7580d39a930592d7a5e0cc80f656bdd286/src/plugins/axios.js#L5
when there is one exists in line28 in main.js?

Looks like plugins/axios.js is not imported but is there just for developers who prefer all plugins and plugin config grouped under plugins/.  Thus, is redundant but left for exemplary purposes.

@jblachly is correct, i left it there for people who prefer their set up that way

@ClintOxx You can close this issue !!


Add -bin suffix to genearted tarball

In generate release tarballs it states we are creating alluxio-version-bin.tar.gz

@calvinjia  update docs where this file name (e.g., in download url) is referenced?

@apc999 This is the name we have always been using, it must have been changed sometime since the last release.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/4097/
Test FAILed.

alluxio-bot, merge this please.

alluxio-bot, cherry-pick this to branch-2.0 please.


Update Get-WsusUpdate.md

Docs Build status updates of commit e3b088f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/updateservices/Get-WsusUpdate.md
:white_check_mark:Succeeded
View (winserver2012-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

@tiburd This one has been approved by the reviewers and can be merged. Thanks.

@tiburd This one has been approved by the reviewers and can be merged. Thanks.

Docs Build status updates of commit a464509:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2012-ps/updateservices/Get-WsusUpdate.md
:white_check_mark:Succeeded
View (winserver2012-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


noti: add module

Add module for https://github.com/variadico/noti

@rycee thanks, I've made the changes you've suggested

Great, it looks good! I've rebased to master in 3f34bf4465b367c6eb4bc109349e75638e64ae52. Thanks for the contribution 👍


ARU Org Join Request

Your Name
Dean Makoni
Membership Type
Member
Role
Postgraduate
Supervisor
Paul Amayo
Teams
Spatial Intelligence
Collaborator Repositories
No response
Special Requests
No response
Code of Conduct

[X] I agree

Invite sent. There is no "Spatial Intelligence" Team currently. @paul-amayo can add you to any of his teams if you require something else.


[BEAM-7372] cleanup py2 codepath from runners/dataflow and runners/interactive

cleanup following codes from runners/dataflow and runners/interactive

usage of future package
usage of past package
unnecessary builtins import

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python PreCommit

R: @tvalentyn

LGTM, thank you


Updated validations.py pythons script

…ting with non letters.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

Just added a regex to ensure that the initial character is a letter