Build fails when versioning.enabled = false

Describe the bug
A clear and concise description of what the bug is.  Include the version(s) of DSpace where you've seen this problem. Link to examples if they are public.
DSpace 7.6
Building backend with disabled item versioning causes build to fail due to missing HandleIdentifierProvider in identifier-service.xml.
To Reproduce
Steps to reproduce the behavior:

set versioning.enabled = false in versioning.cfg
(optional) in identifier-service.xml comment out bean with id="org.dspace.identifier.HandleIdentifierProvider"
build backend

Expected behavior
The build should succeed.
Related work

I will provide a PR.

@tdonohue I'm not quite sure how to proceed with the unit tests. Integration tests run through just fine but the unit tests rely on versioning being enabled, so they fail miserably.
mvn install -DskipUnitTests=false fails and mvn test -DskipUnitTests=false '-Dtest=org.dspace.identifier.*Test' does not execute any tests.

@mirkoscherf : I'd recommend creating a draft PR with the changes you've made so that others can more easily see the failing unit test behavior.  It's difficult to help without understanding/seeing the errors.  It's possible that some individual tests need to have versioning.enabled=true, but in that scenario those tests could set that value in the test itself (see ConfigurationService.setProperty() and how it's used in other tests).  But, it's unclear to me which tests you reference, so it's difficult to provide advice.

PR is underway.
The failed tests look like this:
[ERROR] Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: 0.607 s <<< FAILURE! - in org.dspace.xmlworkflow.XmlWorkflowFactoryTest
[ERROR] org.dspace.xmlworkflow.XmlWorkflowFactoryTest  Time elapsed: 0.607 s  <<< ERROR!
org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.dspace.app.sherpa.submit.SHERPASubmitService' defined in file [F:\workspace\dspace7\src\dspace\dspace-api\target\testing\dspace\config\spring\api\sherpa.xml]: Cannot resolve reference to bean 'org.dspace.app.sherpa.submit.SHERPASubmitConfigurationService' while setting bean property 'configuration'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.dspace.app.sherpa.submit.SHERPASubmitConfigurationService' defined in file [F:\workspace\dspace7\src\dspace\dspace-api\target\testing\dspace\config\spring\api\sherpa.xml]: Cannot create inner bean 'org.dspace.app.sherpa.submit.MetadataValueISSNExtractor#743997e0' of type [org.dspace.app.sherpa.submit.MetadataValueISSNExtractor] while setting bean property 'issnItemExtractors' with key [0]; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating
bean with name 'org.dspace.app.sherpa.submit.MetadataValueISSNExtractor#743997e0': Unsatisfied dependency expressed through field 'itemService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.ItemServiceImpl#0': Unsatisfied dependency expressed through field 'handleService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.handle.HandleServiceImpl#0': Unsatisfied dependency expressed through field 'siteService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.SiteServiceImpl#0': Unsatisfied dependency expressed through field 'metadataValueService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.MetadataValueServiceImpl#0': Unsatisfied dependency
expressed through field 'authorizeService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.authorize.AuthorizeServiceImpl#0': Unsatisfied dependency expressed through field 'bitstreamService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.BitstreamServiceImpl#0': Unsatisfied dependency expressed through field 'relationshipService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.RelationshipServiceImpl#0': Unsatisfied dependency expressed through field 'entityTypeService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.EntityTypeServiceImpl#0': Unsatisfied dependency expressed through field 'groupService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.eperson.GroupServiceImpl#0': Unsatisfied dependency expressed through field 'collectionService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.CollectionServiceImpl#0': Unsatisfied dependency expressed through field 'resourcePolicyService'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.authorize.ResourcePolicyServiceImpl#0': Unsatisfied dependency expressed through field 'contentServiceFactory'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'contentServiceFactory': Unsatisfied dependency expressed through field 'dSpaceObjectServices'; nested exception is org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'org.dspace.content.CommunityServiceImpl#0': Unsatisfied dependency expressed through field 'identifierService'; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.dspace.identifier.service.IdentifierService' defined in file [F:\workspace\dspace7\src\dspace\dspace-api\target\testing\dspace\config\spring\api\identifier-service.xml]: Cannot resolve reference to bean 'org.dspace.identifier.HandleIdentifierProvider' while setting bean property 'providers' with key [0]; nested exception is org.springframework.beans.factory.BeanCreationException: Error creating bean with name 'org.dspace.identifier.HandleIdentifierProvider' defined in file [...\src\dspace\dspace-api\target\testing\dspace\config\spring\api\identifier-service.xml]: Invocation of init method failed;

nested exception is java.lang.RuntimeException: the org.dspace.identifier.VersionedHandleIdentifierProvider is enabled, but the versioning is disabled.

Current test run result
[ERROR] Tests run: 337, Failures: 0, Errors: 76, Skipped: 1

@mirkoscherf : Oh, I see.  Yes, that error is because the tests enable the "VersionedHandleIdentifierProvider" by default (which requires versioning.enabled=true).
The only way to solve this would be to:

Modify tests to always set versioning.enabled=true (regardless of your setting in dspace.cfg or local.cfg).  This would be done by adding the setting to the Test Environment local.cfg here: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/test/data/dspaceFolder/config/local.cfg
OR, locally, change your test environment to use the HandleIdentifierProvider.  That'd be done by modifying the identifier-service.xml used by the tests here: https://github.com/DSpace/DSpace/blob/main/dspace-api/src/test/data/dspaceFolder/config/spring/api/identifier-service.xml

If you choose this option, likely any tests which test for versioning will fail.  They'd need to be disabled individually.

So, basically, because these two Handle providers cannot both be enabled at the same time, the tests need to choose one.  Currently, by default they use VersionedHandleIdentifierProvider because there are tests that verify that versioning works properly.
Ideally, longer term, it'd obviously be better if the correct "HandleIdentifierProvider" was just selected based on the value of versioning.enabled.  But, I'm honestly not sure how easy that'd be to implement.

I believe part of the issue described in this ticket was a documentation error.  The Item Versioning docs were incorrect for how to properly disable versioning.  They have been corrected to note there are three steps:

Set versioning.enabled = false
Comment out the VersionedHandleIdentifierProvider in identifier-service.xml but ALSO REPLACE it with HandleIdentifierProvider
Removing versioning from list of default consumers

See docs at https://wiki.lyrasis.org/display/DSDOC7x/Item+Level+Versioning#ItemLevelVersioning-DisablingItemLevelVersioning

I followed the steps to disable versioning, including adding the HandleIdentifyProvider to [dspace]/config/spring/api/identifier-services.xml and removing versioning from the default.consumers list. But now I can't create new communities. The stack trace indicates there's an issue with casting in the HandleIdentityProvider.

2023-10-13 09:31:57,409 ERROR 2e874867-250d-4b44-8a71-12a185f9189b 3fe9dc00-0a90-4625-bce3-66ff24bd541e org.dspace.app.rest.exception.DSpaceApiExceptionControllerAdvice @ An exception has occurred (status:500)
java.lang.ClassCastException: class org.dspace.content.Community cannot be cast to class org.dspace.content.Item (org.dspace.content.Community and org.dspace.content.Item are in unnamed module of loader org.apache.catalina.loader.ParallelWebappClassLoader @340da44c)
at org.dspace.identifier.HandleIdentifierProvider.register(HandleIdentifierProvider.java:73) ~[dspace-api-7.6.1-SNAPSHOT.jar:7.6.1-SNAPSHOT]
at org.dspace.identifier.IdentifierServiceImpl.register(IdentifierServiceImpl.java:107) ~[dspace-api-7.6.1-SNAPSHOT.jar:7.6.1-SNAPSHOT]
at org.dspace.content.CommunityServiceImpl.create(CommunityServiceImpl.java:128) ~[dspace-api-7.6.1-SNAPSHOT.jar:7.6.1-SNAPSHOT]
at org.dspace.content.CommunityServiceImpl.create(CommunityServiceImpl.java:93) ~[dspace-api-7.6.1-SNAPSHOT.jar:7.6.1-SNAPSHOT]
at org.dspace.content.CommunityServiceImpl.create(CommunityServiceImpl.java:88) ~[dspace-api-7.6.1-SNAPSHOT.jar:7.6.1-SNAPSHOT]
at org.dspace.app.rest.repository.CommunityRestRepository.createCommunity(CommunityRestRepository.java:142) ~[classes/:7.6.1-SNAPSHOT]
at org.dspace.app.rest.repository.CommunityRestRepository.createAndReturn(CommunityRestRepository.java:123) ~[classes/:7.6.1-SNAPSHOT]
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[?:?]
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
at java.lang.reflect.Method.invoke(Method.java:568) ~[?:?]
at org.springframework.aop.support.AopUtils.invokeJoinpointUsingReflection(AopUtils.java:344) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.ReflectiveMethodInvocation.invokeJoinpoint(ReflectiveMethodInvocation.java:198) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:799) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:163) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:763) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.security.access.intercept.aopalliance.MethodSecurityInterceptor.invoke(MethodSecurityInterceptor.java:61) ~[spring-security-core-5.7.8.jar:5.7.8]
at org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:186) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.proceed(CglibAopProxy.java:763) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:708) ~[spring-aop-5.3.27.jar:5.3.27]
at org.dspace.app.rest.repository.CommunityRestRepository$$EnhancerBySpringCGLIB$$85138c3b.createAndReturn() ~[classes/:7.6.1-SNAPSHOT]
at org.dspace.app.rest.repository.DSpaceRestRepository.createAndReturn(DSpaceRestRepository.java:326) ~[classes/:7.6.1-SNAPSHOT]
at org.dspace.app.rest.repository.DSpaceRestRepository$$FastClassBySpringCGLIB$$615525f9.invoke() ~[classes/:7.6.1-SNAPSHOT]
at org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:218) ~[spring-core-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy.invokeMethod(CglibAopProxy.java:386) ~[spring-aop-5.3.27.jar:5.3.27]
at org.springframework.aop.framework.CglibAopProxy.access$000(CglibAopProxy.java:85) ~[spring-aop-5.3.27.jar:5.3.27]
...

@nwoodward : That sounds like this bug / fix: #9098.   It should now be fixed on the latest dspace-7_x code and main code, but it was only fixed in the last week.

@tdonohue Thanks! That's exactly what I needed. I wish I had seen that PR come in before disabling versioning on 14 DSpace repositories, but I guess this is what Friday afternoons are for.


error[E0446]: private type app::DeitiesState in public interface

error[E0446]: private type app::DeitiesState in public interface
--> src/app.rs:7:1
|
7 | #[component]
| -^^^^^^^^^^^
| |
| can't leak private type
| app::DeitiesState declared as private
error[E0446]: private type app::DeitiesStatus in public interface
my app layout is:

Issue resolved.


Optimize Memory Consumption in JPEG Decoding

We can optimize memory consumption in our JPEG decoding process. Currently, all MCUs are decoded at once and stored in the mcu_storage buffer, resulting in high memory use.
Instead, we could decode and render MCUs one by one. This approach would significantly reduce the required memory usage.
If the team agrees, I would like to handle this optimization.

Please go ahead :)


pyprakhar.py

Added Unique Pyramid Pattern of Digits..

created a program of unique number pattern in python please approve and merge it for my hactoberfest2021


Add Proper Image Ordering/Sorting for Derpibooru

This is being added posted as a feature request since it technically isn't a bug as everything is working exactly as it should.
As it is, you can't use the "Sort by" function found under the "Plus" options when searching for an image from the source Derpibooru. The reason for this, I believe, is due to the way Derpibooru searches for images. Unlike other boorus, it doesn't take namespaces into the search bar. Because of this, using the "Sort by" option, or any of the namespaces, for that matter, under the "Plus" options cause no results to be displayed for the Derpibooru source.

Sure, but note that (as far as I know), Derpibooru only supports sorting by score or by number of comments.
I will add order:score / order:score_asc / order:score_desc and order:comments / order:comments_asc / order:comments_desc support in the next version.

Sorting for Derpibooru no longer seems to work. Adding order:score makes it so that there's no results.

Indeed, it would seem that the sort: modifier is getting removed from Derpibooru searches. I'll push a fix.


Model Description
The Aspect Model describes the permanent characteristics of an individual IoT device
https://confluence.catena-x.net/display/ARTV/(CEC)+Data+Modeling+requ.+[F05]+IoT+Device+Onboarding
Relevant Standards

Example Data

Attribute   Name
Data Type
Unit or Enumeration   values
Data   description
Example
Standard/E-Class   Reference

catenaXId
String
Regular   Expression
The fully anonymous Catena-X ID   of the serialized part, valid for the Catena-X dataspace.
urn:uuid:7a6a8376-1783-4926-9be0-5d946622b2e2
RFC 4122 - A   Universally Unique IDentifier (UUID) URN Namespace (ietf.org)

IoT Device Manufacturer
String

Manufacturer of   the IoT Device

IoT Device Type
String

The   IoT Device Type
TRACK02839

Serial Number
String

The   serial Number of the IoT device
123-0740-3434-A

Device Owner BPNL
String

BPNL   of the device owner
BPNL00000003ABCD

MS1 Criteria

[ ] The proposed aspect model does not exist already
[ ] The proposed aspect model does not extend an existing aspect but introduces completely new functionality
[ ] The proposal references a Catena-X use case
[ ] Relevant standards are mentioned/linked
[ ] A first draft of the data structure is provided in an example

IDTA submodel has been checked by @Cvetanka-ZF but model does not fit to requirements. Also IDTA model is not yet released.
MS1 approved by modelling team.


Does this work with Django 1.8.2

Wonder if this has succesfully worked with Django 1.8.2 and if there are any issues regarding the code implementation and/or dependencies.

@Jzarecta I'm afraid we have never tried. My guts feeling is that this should probably work or at least be very little effort to get it working. I think we just need to rename the migrations folder into south_migrations and then initiate a new migrations folder which uses the core migrations module.
Then we need to document that south users need to specify that their old south-based projects should look for the south_migrations folder and everyone else who wants to upgrade needs to make sure that

they update to this latest version and run the migrations
now they disable south and switch to core migrations, there will only be one initial migration and they should run it with the --fake flag

@mbrochh @dkaufhold I haven't.

@mbrochh @Tyrdall me neither.
django-libs is ready for 1.8, right? Other than that I'd agree with it only being the migrations that need to be changed.

Ok sounds good. I think if no one else sends a pull request, I'd like to give the migrations thing a go sometime this week before Sunday.

@mbrochh Have you tried this in django 1.8? Or, has anyone else tried this in django 1.8 to get it successfully working?

Works with 0.7.2


Bulk Move Devices

I am trying to move a bunch of devices from one sub OU to another and GAM seems to say it cannot find the devices.  The error I get is below.  I have tried moving the device to the main OU but GAM still cannot find it.  Any Ideas?
C:\gam>gam update cros deviceserialnumber ou "suborgou"
updating deviceserialnumber (1 of 1)
Error 404: Resource Not Found: deviceserialnumber - notFound
Joe

This is an issue tracker for known bugs. Please discuss problems on the discussion group:
groups.google.com/group/google-apps-manager
Jay


Add maximum results per provider

Hello @fugkco ,
Is that possible to add in your next release an option to limit results per provider ?
Thanks a lot !

Hi @chicungunya, can you specify.
You want setting like "no more than N torrent for any provider"?
Or maybe just the overall limiter on torrents from jackett?
Or ability to set max for each provider separately? - don't think the last one is worth coding.
Please explain the Use-Case. For me it seems like a useless feature.


Support migrating runs from one viv instance to another

This is desired to make it easy to interoperate across multiple viv nodes.
Plan

specify S3 buckets (import/export) via env vars
specify auth keys for S3 buckets via env vars
if export env vars set: export to S3 automatically in the background after a run finishes
if import env vars set: background process runner will import newly-exported S3 items

will determine which ones need to be imported by looking up items added in the last $DURATION
and checking that list against existing runIds.

deal with runId conflicts:

create a new runId by inserting into runs_t first, and then adding the rest of the data in a
separate transaction
save the original runId as a metadata entry

for export format, use JSON version of any columns that have the given runId (notably this will not include the task_environments_t column for the run)

runs_t
agent_branches_t
trace_entries_t
agent_state_t
entry_comments_t
entry_tags_t
intermediate_scores_t
rating_labels_t
run_models_t
run_pauses_t

Per the inline mention, this will not export the task environment row for the run. In part this is because the container name would use the old runId (which would be confusing), but also generally there's not much in task_environments_t that's useful for analysis. The task's commit ID is specified there, and if that's useful then it can be added to the metadata map. But if this proves to have some undesirable side-effects, these rows can be exported/imported as well.

create a new runId by inserting into runs_t first, and then adding the rest of the data in a
separate transaction

I'm not sure a separate transaction is necessary. It could be fine to insert the row (which returns the run ID) then insert everything else, all in the same transaction.

Won't the k8s support mean that we can only have one viv instance (apart from dev instances) and then this feature won't be needed anymore?


Update CODEOWNERS file for Monitor

Add new GitHub team for feature crew working on Monitor data plane libraries
/cc: @joshfree

/check-enforcer override


Update monitoring-performance-by-using-the-query-store.md

the same change proposed and accepted in Update best-practice-with-the-query-store.md   , forcing a plan via sp_query_store_force_plan is not deterministic. in order to always have the forced plan we need a plan guide.

@baleng : Thanks for your contribution! The author, @MikeRayMSFT, has been notified to review your proposed change.

@baleng - I'm pulling this into the private repository to make a few additional changes. Thank you very much for the submission. It will be merged in a day or so.


Fix reading HTML via XmlReader API when root element is absent

Consider folowing snippet:
            string toParse;
            toParse = "
one

two

three
";
            var sr = new StringReader(toParse);
            var NULL = "NULL";

            using (var reader = new SgmlReader()
                {InputStream = sr, DocType = "HTML", IgnoreDtd = true})
            {
                while (!reader.EOF)
                {
                    reader.Read();
                    WriteLine($"[{reader.NodeType,10}] {reader.Name ?? NULL,5}: " +
                        $"{reader.Value ?? NULL}");
                }
            }

On version 1.8.12 SGMLReader fails to read all document since it comes to this section twice:
            if (this.Depth == 1)
            {
                if (this.m_rootCount == 1)
                {
                    // Hmmm, we found another root level tag, soooo, the only
                    // thing we can do to keep this a valid XML document is stop
                    this.m_state = State.Eof;
                    return false;
                }
                this.m_rootCount++;
            }

It only returns the first
 element and opening tag for second.
It was working in old good version 1.8.6, but I don't have a code of it, as on GitHub history starts from version 1.8.7 wich is already broken.
This commit allows to fix this behavior if user has set DocType to "HTML" explicitly.

Awesome, just the same problem as I'm having here, too. Hopefully it get's integrated and published to NuGet, soon.

I'm worried this change is too HTML centric, what about SgmlReader over other types of data, like OFX ?

@lovettchris I can not see how it affects OFX and other types. Why did you get this particular doubt?


Error: pull model manifest: Get

What is the issue?
Ollama is not able to pull the models.
ollama run moon dream
pulling manifest
Error: pull model manifest: Get "https://ollama.com/token?nonce=F_Rh4t6Jrv-EM0eRltrU-Q&scope=repository%!A(MISSING)library%!F(MISSING)moondream%!A(MISSING)pull&service=ollama.com&ts=1718102949": read tcp <IP_ADDRESS>:51154-><IP_ADDRESS>:443: read: connection reset by peer
OS
macOS
GPU
Apple
CPU
Apple
Ollama version
No response

Hi @funnyPhani. Do you happen to still have logs for this in ~/.ollama/logs/server.log

I had the same problem while trying to run ollama pull llama3.1:8b. Logs looked like this:
time=2024-11-21T14:45:50.689-08:00 level=INFO source=images.go:1020 msg="request failed: Get \"https://registry.ollama.ai/v2/library/llama3.1/manifests/8b\": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor"
  [GIN] 2024/11/21 - 14:45:50 | 200 |    5.217791ms |       <IP_ADDRESS> | POST     "/api/pull"
  [GIN] 2024/11/21 - 14:45:55 | 200 |      19.292µs |       <IP_ADDRESS> | HEAD     "/"
  time=2024-11-21T14:45:55.119-08:00 level=INFO source=images.go:1020 msg="request failed: Get \"https://registry.ollama.ai/v2/library/llama3.1/manifests/8b\": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor"
  [GIN] 2024/11/21 - 14:45:55 | 200 |    5.899459ms |       <IP_ADDRESS> | POST     "/api/pull"
  [GIN] 2024/11/21 - 14:46:55 | 200 |          34µs |       <IP_ADDRESS> | HEAD     "/"
  time=2024-11-21T14:46:55.591-08:00 level=INFO source=images.go:1020 msg="request failed: Get \"https://registry.ollama.ai/v2/library/llama3.1/manifests/8b\": dial tcp <IP_ADDRESS>:443: connect: bad file descriptor"
  [GIN] 2024/11/21 - 14:46:55 | 200 |   10.563084ms |       <IP_ADDRESS> | POST     "/api/pull"

This was while the Ollama GUI program was running in the taskbar. Very strangely if I killed the GUI program and just ran ollama server in one shell and run ollama pull llama3.1:8b in another, it worked just. ¯_(ツ)_/¯


Feature Request - support publishing to Gitlab Wiki?

Is your feature request related to a problem? Please describe.
No, arguably its for a whole new plugin!
Describe the solution you'd like
It would be cool to be able to publish a canvas or vault directly to a a Gitlab Wiki. Gitlab wiki's are nothing more than git repos that hold markdown, so it might be quite easy to just push the markdown up to the wiki repo.
The challenges might involve some regex work to make the rendered markdown gitlab-wiki compatible (e.g., link rewriting, etc), maybe some pandoc type stuff, etc...
Describe alternatives you've considered
I've not thought too much further about this,  but it could be done in a more simple way with a few shell scripts I guess.
Additional context
Nope

Thanks for your input, but I definitely agree with you that it's something for an entirely separate plugin.
It's not something I would consider trying to add to this package.
Also, in terms of publishing there is already the Obsidian Publish system, and I would suggest that building anything new should also be able to support more than Gitlab as a target destination.


Fix Hub v1 support

Description:
When insteon_local and insteon_plm were merged, support for the Insteon Hub version 1 is broken. This fixes that support.
Related issue (if applicable): fixes #16342
Pull request in home-assistant.github.io with documentation (if applicable): home-assistant/home-assistant.github.io#6211
Example entry for configuration.yaml (if applicable):
# Hub 2242 configuration variables
insteon:
  host: HOST
  ip_port: IP_PORT
  hub_version: 1

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[ ] New files were added to .coveragerc.

If the code does not interact with devices:

[ ] Tests have been added to verify that the new code works.

Can be merged when build passes.

@MartinHjelmare I have a release management question. Since 0.78.0b0 was just released, What does that mean for this fix? Does that mean it can be part of 0.78.0 or will it mean it has to wait until 0.79 or can it be included in 0.77.4 if one is released? Keep in mind this functionality was working prior to 0.77 for insteon_local users so I want to get them back up and running.  Appreciate all your help.

We can tag it for 0.78.


Volunteer Portal - Admin homepage

https://ncosd.github.io/food-pantry-app-prototype/volunteer/admin/homepage/

src/views/Home/vue was added.


[Bug] Optimal Aim Red Box completely covers screen when fighting/riding ender dragon

clip: https://medal.tv/games/minecraft/clips/ivXCPaPPR4Qc2kxPa/d1337ITtti6E?invite=cr-MSxGMUMsMTA1ODAwNzcs
version: quilt 1.20.6
mod version: v1.0

Fixed in this version
Here's a clip of the new behavior: https://streamable.com/e3fqfl


Input | Convert c# to xaml conversion

[x] Convert the functionalities to xaml file

Completed the Xaml conversion for input control.
Replaced existing tminput control with xaml converted control.


Can I use this pluign for my AR project?

I would like to display my IoT monitoring (web-based) kind of attach to an gameobject, is it possible?

I mean i will not use oculus just as AR app in andorid, i want to display my specific html page on my AR

You should be able to use it as long as you include the Oculus Plugin in your project. I think that it should work on mobile. You can read up here on the integration: https://developer.oculus.com/downloads/package/unity-integration/
Right now the plugin included in the demo project is not the newest one, however so you'll have to see if it works.

I have tried import the package, then just put the prefab called BrowserView but nothing happened when I play


bgpv2: Replace preflight reconciler with BGPRouterManager logic

Since the preflight reconciler is modifying instance data managed (created & deleted) by BGPRouterManager, it makes more sense to move its logic (detection if an instance needs to be re-created) into the manager (reconcileDiffV2).
This avoids duplication of code (getting of local ASN, router ID, local port from config, creating new instance) and makes Init() and Cleanup() reconciler methods work upon instance re-creation.
This is a pre-requisite for removing the global metadata map for reconcilers, but also fixes an issue with state notifications that currently stop working after instance re-create (state notification tracking is not reinitialized when a new instance is created by the preflight reconciler).

/test


Issue increments inconsistent

Most likely due to changing p.config.increment references. Should be an easy fix, just need the time to fix it... Referencing reports.ts as this is where reports are dealt with.

Fixed in this commit.


feat/conformance 004

I took a stab at conformance test case 004. I think I've managed to come up with a strategy that neatly tests all the cases without arbitrary sleeps etc.

I've updated the PR to use multiple "peer" nodes concurrently instead of restarting the node for every message.


Installing from pip install fidesctl==1.8.0 shows a dirty version

Bug Description
Installing fidesctl version 1.8.0 from pypi and then running fides --version shows a dirty tag

I've confirmed that the docker container for 1.8.0 has the same issue


fix: codegenning formatter in table jsx

Description of changes:

using formatter in codegenned table jsx

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

Codecov Report

Merging #579 (1c3f1a3) into tagged-release/forms-and-views (a83c18f) will decrease coverage by 0.11%.
The diff coverage is 79.41%.

@@                        Coverage Diff                         @@
##           tagged-release/forms-and-views     #579      +/-   ##
==================================================================
- Coverage                           89.58%   89.47%   -0.12%
==================================================================
  Files                                  83       84       +1
  Lines                                3035     3069      +34
  Branches                              814      824      +10
==================================================================
+ Hits                                 2719     2746      +27
- Misses                                301      306       +5
- Partials                               15       17       +2

Impacted Files
Coverage Δ

packages/codegen-ui/lib/types/string-format.ts
100.00% <ø> (ø)

...odegen-ui-react/lib/react-table-renderer-helper.ts
66.66% <66.66%> (ø)

...kages/codegen-ui-react/lib/react-table-renderer.ts
96.55% <93.75%> (-1.07%)
:arrow_down:

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 a83c18f...1c3f1a3. Read the comment docs.


Can't see the tracked pose as a topic

Hi,
I am trying to get tracked pose from cartographer. I am using ros melodic and ubuntu 18.04 where cartographer and cartographer ros binary packages are installed through apt. I am using a single 2d rplidar A1M8.
As suggested in this pr and here, if I set the parameter publish_tracked_pose = true, I get an error saying lua_parameter_dictionary.cc:410] Check failed: 1 == reference_counts_.count(key) (1 vs. 0) Key 'publish_tracked_pose' was used the wrong number of times. Here is my .lua file:
include "map_builder.lua"
include "trajectory_builder.lua"

options = {
  map_builder = MAP_BUILDER,
  trajectory_builder = TRAJECTORY_BUILDER,
  map_frame = "map",
  tracking_frame = "base_link",
  published_frame = "base_link",
  odom_frame = "odom",
  provide_odom_frame = true,
  publish_frame_projected_to_2d = false,
  use_odometry = false,
  use_nav_sat = false,
  use_landmarks = false,
  num_laser_scans = 1,
  num_multi_echo_laser_scans = 0,
  num_subdivisions_per_laser_scan = 10,
  num_point_clouds = 0,
  lookup_transform_timeout_sec = 0.2,
  submap_publish_period_sec = 0.3,

  publish_tracked_pose = true,

  pose_publish_period_sec = 5e-3,
  trajectory_publish_period_sec = 30e-3,
  rangefinder_sampling_ratio = 1.,
  odometry_sampling_ratio = 1.,
  fixed_frame_pose_sampling_ratio = 1.,
  imu_sampling_ratio = 1.,
  landmarks_sampling_ratio = 1.,
}

MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.num_accumulated_range_data = 7
TRAJECTORY_BUILDER_2D.min_range = 0.1
TRAJECTORY_BUILDER_2D.max_range = 10
TRAJECTORY_BUILDER_2D.use_imu_data = false
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.linear_search_window = 0.1
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.translation_delta_cost_weight = 0.1
TRAJECTORY_BUILDER_2D.real_time_correlative_scan_matcher.rotation_delta_cost_weight = 1e-1

return options

As suggested by some issues, I changed the parameter publish_tracked_pose = on, the error was gone but I can't see the tracked_pose topic published.
Here is my launch file:
<launch>
  <!-- <param name="robot_description" textfile="$(find cartographer_ros)/urdf/backpack_2d.urdf" />-->

 <!-- <node name="robot_state_publisher" pkg="robot_state_publisher" type="robot_state_publisher" /> -->

  <node pkg="tf" type="static_transform_publisher" name="base_link_to_lidar" args="0 0 0 0 0 0 base_link laser 100" />

  <node name="cartographer_node" pkg="cartographer_ros"
      type="cartographer_node" args="
          -configuration_directory $(find carto_mapper)/config
          -configuration_basename x2.lua"
      output="screen">
  </node>

  <node name="cartographer_occupancy_grid_node" pkg="cartographer_ros"
      type="cartographer_occupancy_grid_node" args="-resolution 0.05"
      output="screen" />

  <include file="$(find rplidar_ros)/launch/rplidar.launch" />
  <node name="rviz" pkg="rviz" type="rviz"/>
</launch>

Any help is appreciated.

I found that version 1.0.0 does not contain this feature, you may need to update the version.

i have question, 1.0.0 version, is latest. how can i update this version?


Document MySQL protocol databases

Describe the feature
There are mutliple databases that are described as using the MySQL wire protocol and/or syntax including [1]:

SingleStore
ClickHouse
TiDB
PlanetScale on Vitess
Dolt

There are probably others too. As we learn of them, it would be nice to document if any compatability testing has done against this adapter. Where applicable, redirect them to the specific adapter for the database.
Additional context
Some databases might not support crucial functionality. An example is TiDB's support for create table as ... select statements (CTAS) (#3839, #4754).

Hello @dbeatty10, FYI, we've developed a dbt-tidb adapter based on dbt-mysql. We've tested it against several TiDB versions and document the compatibility issues.
Thanks for your excellent work!


add a method to get the values yaml for testing / debug

not sure if there is such an option already but would be useful for testing / debugging - to be able to get the generated values yaml without actually creating it
ideally - should be able to get it by running the docker container directly with no env vars, so you could test it without deploying the provisioner api

@akariv


FT.SYNADD FT.SYNUPDATE FT.SYNDUMP support

Are you still working on this issue? @filipecosta90

Are you still working on this issue? @filipecosta90

haven't started yet. Appreciate if you want to push it :+1:

Got it, I will push it recently.

solved by #55 . Thanks @dengliming :+1:


How can I destroy instance of the menu ?

I need this function for responsivness, below X pixels i would like to destroy instance of mega menu, and use casual CSS Menu, or my own JS.

Use a media query and display none on the mega menu. Then display block on your custom menu.

What if I want to use the same menu for both mobile and desktop so doing a display none is not an option? I'm using a CMS and trying to be SEO conscious to not duplicate navigation links that would confuse Google's bots.
Are there any methods for destroying and re-initializing the megamenu js on resize? My event handlers won't function over top of the libraries.

I created a destroy function right below "setOptions" around line 825
destroy: function() {
$(this.menu).off('.accessible-megamenu');
}
Initiate
$("nav:first").accessibleMegaMenu(megamenu);
Remove
$('nav:first').data('plugin_accessibleMegaMenu').destroy();
Re-start
$('nav:first').data('plugin_accessibleMegaMenu').init();

@carlafranca Hi, Can you please elaborate further on how can use the above destroy function. I need to turn-off megamenu below certain resolution and bind different events(touch -tap/doubleTap)
Many thanks in advance.
Regards,
Mark W


Tensor tools conversion fails

Hello,
I am trying to understand how to convert my existing Pytorch models (.pth format) to the .ot format used in this crate. I understand that saving the pickled .pth files to .npz would be the way to go as an intermediate step.
To better understand the format of the npz to generate, I tried converting the existing .ot files given in the examples back to npz. The following fails with the latest version of master:
    let src_filename = "E:/files/resnet18.ot";
    let dst_filename = "E:/files/resnet18.npz";
    let tensors = if src_filename.ends_with(".npz") {
        tch::Tensor::read_npz(src_filename)?
    } else if src_filename.ends_with(".ot") {
        tch::Tensor::load_multi(src_filename)?
    } else {
        bail!("unhandled file {}", src_filename)
    };
    for (name, tensor) in tensors.iter() {
        println!("{}: {} {:?}", src_filename, name, tensor)
    }
    if dst_filename.ends_with(".npz") {
        tch::Tensor::write_npz(&tensors, dst_filename)?
    } else if dst_filename.ends_with(".ot") {
        tch::Tensor::save_multi(&tensors, dst_filename)?
    } else {
        bail!("unhandled file {}", dst_filename)
    };
    Ok(())

with Error: ErrorMessage { msg: "incoherent elt kind, Float != Uint8" }
Am I missing something?

Thanks for reporting this issue, I just pushed a fix for it.
You're right that the normal way to port model weights to rust is to write them to a npz file from python and then load them from rust and create an ot file. Note that the issue you reported was only happening when writing a npz file so should not be involved in this process.
Here is the simple python script that I used to export the weights for various computer vision models. I would then run cargo run -example tensor-tools cp resnet18.npz resnet18.ot.
Let me know if you run into any issue porting your weights from python to rust.
import numpy as np
import torch
import torchvision

m = torchvision.models.resnet18(pretrained=True)
nps = {}
for k, v in m.state_dict().items(): nps[k] = v.numpy()
np.savez('resnet18.npz', **nps)

Thank you again for the excellent package and the prompt feedback.


Add tests for action in a GitHub Wokflow

Ideally I would like to have some sort of automatic testing set up. Since it's quite time consuming to test this action manually :thinking:
Originally posted by @johanvanhelden in https://github.com/johanvanhelden/gha-clover-test-coverage-check/pull/4#issuecomment-689826575

I am starting to work this issue first.
https://github.com/johanvanhelden/gha-clover-test-coverage-check/tree/feature/tests


"Missing node type in status node. Cannot get workflow created with Argo <= 2.0....." after workflow controller restarted

Pre-requisites

[X] I have double-checked my configuration
[X] I can confirm the issues exists when I tested with :latest
[ ] I'd like to contribute the fix myself (see contributing guide)

What happened/what you expected to happen?
What happened:
(1) Workflow Controller POD experienced an OOMKilled by Kubernetes.
(2) The ongoing workflow status node becomes emptied in the "type" field.
(3) As a result, the argo watch reported:
"Missing node type in status node. Cannot get workflow created with Argo <= 2.0....."
What to expected:
(1) Argo watch can trace the on-going workflow as expected.
How to repeat the issue:
(1) Run a workflow and continue to watch it's status.
(2) Kill the workflow controller pod.
(3) See the captioned error pops out.
Version
v3.3.4
Paste a small workflow that reproduces the issue. We must be able to run the workflow; don't enter a workflows that uses private images.
A sample example long running workflow will do. Due to IP issue, cannot share the exact corporate workflow here.

Logs from the workflow controller
kubectl logs -n argo deploy/workflow-controller | grep ${workflow}

Logs from in your workflow's wait container
kubectl logs -n argo -c wait -l workflows.argoproj.io/workflow=${workflow},workflow.argoproj.io/phase!=Succeeded

(2) The ongoing workflow status node becomes emptied in the "type" field.

The current logic only updates the field and never empties it, so I would recommend trying with a more up-to-date version of Argo.
I've never seen this error before despite seeing a variety of OOMs as well, so this seems very rare as well.

https://github.com/argoproj/argo-workflows/blob/v3.5.11/cmd/argo/commands/common/get.go#L336

With no reproduction, no upvotes, and an old version of Argo, I'm going to close this as not planned at this time


Implement new flow for exchanging secrets with the auth lobby

Depends on https://github.com/fission-suite/auth-lobby/pull/72
Creates a small iframe to exchange the linking secrets with the auth lobby.
The auth lobby can signal that it supports this flow by providing the query parameter authoised=via-postmessage after redirecting back from redirectToLobby.

This PR can be tested on production at:

https://philipps-auth-test.fission.app/ The lobby part
https://dashboard-auth-test.fission.app/ An app with the new webnative change to support this

This also addresses #207


Fix line wrap predicate

Raw command line ends with \n, but after highlighting, it appends things like \x1B[0m to the end. And this makes rustyline thinks it should add a extra \n to the end of buffer. See https://github.com/nushell/nushell/issues/3431.
Since I'm not familiar with windows console, I didn't check if windows has the same problem.

Version 8.1.0 released


Heightmaps and Materials

Hey,  I'm using Russia and Poland to make a map but I am having issues with the Heightmaps and Materials,  are these meant to merge with the tool or do I have to manually add these to each map/location.
P.S Great too BTW

Hey sorry for late reply. Yeah heightmaps and marerials are meant to be merged by the tool. If you have any problems with this please contact me through Discord I'll try to help. Some legacy maps are not possible to be merged, but that shouldn't be a case with Poland map.


Include build instructions

There does not seem to be any documented way to reproducibly compile the source code.

Hi Nicolas,

The project was originally built by compiling the sources directly using javac.

A maven or other build script needs to be created.

Thanks & Regards,
Suresh
http://jambula.sourceforge.net : Lossless Photo Date Stamper

Sent: Wednesday, March 20, 2019 at 12:36 PM
From: "Nicolas Raoul"<EMAIL_ADDRESS>To: drogatkin/mediautil<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [drogatkin/mediautil] Include build instructions (#2)
There does not seem to be any documented way to reproducibly compile the source code.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Is this library on Maven?

@msuresh Writing the full javac command line in the README would be a great start. 🙂

@alamothe seems like it has been repackaged by google: https://central.sonatype.com/artifact/com.google.appengine/mediautil/0.2/overview
There is a maven suugestion in #4 to fix #3 and ultimately allow to publish in maven repository.


Added package.json for NPM

I added a package.json so the module can be installed as an NPM package with Browserify.
With this you can install the package with npm:
$ npm install git+https://github.com/sverrejoh/javascript-detect-element-resize

And then use the page with require from your Browserify project:
// Content of index.js
require("javascript-detect-element-resize");

Build into one bundle containing everything with Browserify:
browserify index.js > bundle.js

I didn't change the code at all, this is just metadata.
I think it would be nice if the package had its own namespace, which exported  addResizeListener and removeResizeListener, so you could use the code like this (instead of putting it on window):
var detectResize = require("javascript-detect-element-resize");
detectResize.addResizeListener(element, func);

But this requires some change in your code, and build step, so I didn't want to start on it.
Using Browserify you could have the canonical version as a Browserify/CommonJS module, and then just from that generate a package that works standalone, or with AMD/Bower.
https://github.com/substack/browserify-handbook#standalone
As I said, a little bit of changes, but I'd love to help if you're interested :).

+1. Exporting a separate object would really be useful in order to not pollute the global namespace. You could use https://github.com/umdjs/umd in order to export the correct format for all environments.


Revert "Retry agent settings 257.x 3263.x"

Reverts cloudfoundry/bosh-agent#102 due to the vSphere issues: https://github.com/cloudfoundry/bosh-agent/pull/102#issuecomment-258006409

Already reverted on master, closing this one.


vswhere.exe will not show installation of TestAgent

Despite the fact that the new test agent uses the same bootstrapper for installation as all other versions of visual studio, vswhere.exe will not show these installations.

I failed to look through the closed issues. I see how it should be used now.


Section <IP_ADDRESS>, Para 374 and 375

Paragraphs 374 and 375 appear to be unintentionally split.

Fixed in TLSWG-v3-editorial-fixes branch.


chore: add veracode scan actions

not signed but I verified that contributor is who he is :)

:tada: This PR is included in version 1.2.4-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


extract poh-config crate

Problem
solana_sdk::poh_config blocks moving out solana_sdk::genesis_config and imposes a solana_sdk dep on svm crates
Summary of Changes

Move to its own crate and re-export with deprecation
Make serde optional in the new crate
Replace unchecked_div_by_const with regular const division and a check for non-zero denominator, to avoid a solana-program dep

That would have been a breaking change

Isn't full a default feature?

It would be a breaking change for anyone building with default-features = false


[BUG] onClose not firing when the X on the popup form is triggered

I'm creating the form with .makePopup(url, options) and in the options passing 2 callback functions.

onSubmit - this one is working fine
onClose - this one is not working for me. I expect that it should fire when the "X" is clicked to close the popup form, but when clicking the "X" the form does indeed close, but the onClose function is not triggered.

@startupmasters I believe the onClose callback is called after the user has submitted the survey, not when the user clicks the X.
It would be great to have a callback when the user clicks 'X' 👍

Well, as per the documentation onClose should fire when the form is closed - "Callback function that will be executed once the typeform is closed."
When the user submits the form, the "onSubmit" function triggers normally, but "onClose" still does not, so it does not trigger neither when submitting, nor when closing the form. I believe this is aa big.

The issue is located in api.js of the package.
key: 'handleClose', on line 2621. the onClose() is triggered through this.props.onClose, when it should be triggered through this.props.options.onClose (because in the documentation, the callback is passed in the options object). The "onSubmit" is defined correctly on line 2661.
Changing this.props.onClose() to this.props.options.onClose() triggers the callback, BUT does not close the form itself.
One way to fix this would be to have the 'handleClose' key, fire both the internal 'onClose' (in the internal props) as well as the external 'onClose', passed in the options object when initializing. So the reworked key: 'handleClose' would be:
key: 'handleClose',     value: function handleClose() {       this.props.onClose();       this.props.options.onClose();     }   }

Hi! Thanks a lot for the bug report, we'll have a look at this shortly 👍

Hey, @startupmasters! Could you please provide a repo to reproduce the issue? I'm using the following code and everything works fine:
<html>

<head></head>

<body>
  <button id="openPopupButton">Open popup</button>
  <script src="https://embed.typeform.com/embed.js"></script>
  <script>
    const btn = document.querySelector('#openPopupButton')
    btn.addEventListener('click', openPopup)

    function openPopup() {
      const TF = window.typeformEmbed
      const popup = TF.makePopup(
        'https://admin.typeform.com/to/PlBzgL',
        {
          onClose: function () {
            console.log('Typeform is closed')
          }
        }
      )
      popup.open()
    }
  </script>
</body>

</html>

Also, are you consuming the API from the npm registry or just from the CDN?

Hey @startupmasters, did your problem get solved?

@startupmasters please feel free to ping, if there any other questions

Hello, sorry for the delay. Just checked back with the team and we're using an older version ^0.9.3 so that might have been the issue. I will double-check on the newer version and get back to you.
Thanks


Opening grid demo in android lasts for ever

Try to open this in android
http://vaadin.github.io/gwt-polymer-elements/demo/#vaadin/VaadinGridSample
Seems like device enters in a loop, but finally the grid is shown after 1,5min

It's a problem in row height recalculations when the grid is positioned as absolute, removing 'fit' class name it fixes. Fixed temporary in https://github.com/vaadin/gwt-polymer-elements/commit/8e53ac90eaa0f60595b84dbb1aee234a88ef464d

Fixed


Changed the safety belts.

The criterion I used here is: if it is on what you get if you just install Debian netinst (without selecting any package), it is fair game for the scripts.

I am resisting the itch to set up autoconf for bb-scripts right now.


Header support

What was changed

Updated proto files to communicate with RR (added header field)
Added getHeader() methods to Workflow and Activity facades
Added $header parameter to Workflow/Activity stub factories
Added passing for header values through Workflow/Activity executions
Some fixes in psalm and phpunit configs

Why?
Header supporting is required for interceptors implementation (#225)
Additional
RoadRunner 2.12.2 is required.
Note: target branch is not the master

This seems good. We have to make sure that we can properly update the SDK in relation to parent RR version.

We only expose headers to users via interceptors in our SDKs.
PHP shouldn't be special here.


remove nodegit dep

add mocha, use account for email & name
fix regex

@lexicalbits Can you give this a test on your box?


added disk monitoring

Hi, first of all, thank you for your continued interest in this project.
I'm having a couple of problems with this PR, nothing major, but I currently don't have the time to fix it myself. I could probably do it in the next few days, though.
My first problem is that MEDIA_DIR is created as a constant at compile time, including the USER environment variable. This means the binary will only work for the person who compiled it, and only on the same user. Also, this will fail to compile in certain environments. (nix, for example)
It seems that this constant is only used once, so it should be alright to just turn it into a variable in the function itself. I did this in my commit, so I hope you're okay with that solution.
My other problem is that the /media/run directory doesn't always exist, and when it doesn't, the program crashes, which it obviously shouldn't.
To address this, I think we'll have to turn mount_disk and drive_disk (or maybe just drive_disk) into Option<WatchDescriptor>. Since this might mean changes in a few more places, I don't want to do it at the moment, but as I said, if you don't want to fix this, I should be able to do it in a few days.
Overall, this is a great addition to the project, and thank you again for your contributions.

To address this, I think we'll have to turn mount_disk and drive_disk (or maybe just drive_disk) into Option
problem with the making like this if the mount_dir /media/run is not present then we wont be able to monitor mounts. Another thing i forgot to mention is that this monitor is for udisks2 (on arch) this will grant user to mount as a non root user but drive will be mounted to /media/run directory.

if you don't want to fix this, I should be able to do it in a few days
sure I will see if I can provide any workaround for this, may be an option


Don't clear addr on k8s service deletion

As described in #566, the k8s namer may become "stuck" so that load balancers receive no updates after a k8s service is deleted.
Removing linkerd's state-clearing code appears to resolve the problem.

lgtm -- as noted in #566, there's still some investigation to do; but this seems like a reasonable fix in the interim.


Add documentation for Chocolatey environment variables

Add documentation for Chocolatey environment variables

@pascalberger your changes have been merged, thanks for your contribution 👍


Separate Headscale log levels

Is your feature request related to a problem? Please describe.
Currently, the Headscale log level is tied to the global log level. This makes the development logs very noisy since the default log level is set to debug.
Describe the solution you'd like
Similar to FRP logs, Headscale should have its own environment variables that set its log level. It should default to error.

Unfortunately, tested this and it is not feasible because Headscale uses zerolog which is tied to logrus (the library we use). The consequence of that is that when we change the log level in logrus, it changes in zerolog as well.


Multiple Search and Replace at once.

I am looking for the feature to search multiple terms at replacing it with replacing terms. Here is the screenshot of that feature that explains everything.

And, there should be one replace button that replaces all search text with replacing text. I can add as many search and replace groups I want, which will replace all at once.

Sad to see this feature is closed.
I had similar needs and have to do it by script.
My need is to change on every files from a list of corresponding replace like this:
match1	replace-1
match2	other-replace
match3	something-else
...
matchX	xyz
Update in every files every couple: match by replace-1 and so on for every line.
This is not possible through more abstract search and replace because every line is different and more every replace can be drastically different from source.

Even i had stumbled into this kind of requirement. You have to also take care that bigger strings are replaced first as smaller strings might be prefix or suffix of bigger strings. Below is the Java code to just get it done. I know it is not convenient but it is a option if you are in a hurry and there are a lot of strings to be replaced.
https://github.com/overtomanu/java-util-code/blob/master/src/util/ReplaceListOfWordsInFile.java

Below is simple shell script which replaces strings in file "EnglishCols.txt" with strings from "FrenchCanadianColumns.txt" in the file "GenCSVFrenchCanadian.txt"
for i in $(seq $(cat EnglishCols.txt|wc -l)); do   toBeReplaced=$(head -$i EnglishCols.txt|tail -1);   replacementValue=$(head -$i FrenchCanadianColumns.txt|tail -1);   echo replacing \"$toBeReplaced\" by \"$replacementValue\"   sed -i '' "s|$toBeReplaced|$replacementValue|g" GenCSVFrenchCanadian.txt done


test: migrate rename_deps, replace, required_features and run to snapbox

What does this PR try to resolve?
Part of https://github.com/rust-lang/cargo/issues/14039.
Migrate following to snapbox:

tests/testsuite/rename_deps.rs
tests/testsuite/replace.rs
tests/testsuite/required_features.rs
tests/testsuite/run.rs

@bors r+

:pushpin: Commit 913b12b8d4cd4733436b30fceae702ee35118610 has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 913b12b8d4cd4733436b30fceae702ee35118610 with merge 28d213bae29ba7c8fe957d496bdd342acdc3ea8b...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 28d213bae29ba7c8fe957d496bdd342acdc3ea8b to master...


[review] Dealing with charges

Any thoughts on this one?
-At the end of section 7.1.1 on pages 15/16 the following statement appears: “Typical molecular dynamics engines are not set up to recognize this change, or at least not to correctly include contributions to the free energy from changing constraints/constraint length, so results for a transformation would usually be erroneous.”
This statement is a bit surprising since methods to calculate the contribution of the work done by constraint forces have been described a few times, e.g.:
-Van Gunsteren, W.F., In van Gunsteren W.F. and Weiner, P.K. (Eds.) Computer Simulation of Biomolecular Systems: Theoretical and Experimental Applications, ESCOM, Leiden, 1989, pp. 1-26.
-Pearlman, D.A., Kollman, P.A., The overlooked bond-stretching contribution in free energy perturbation calculations, J. Chem. Phys. 1991, 94(6), 4532-4545.
-Straatsma, T.P., Zacharias, M., McCammon, J.A., Holonomic constraint contribution to free energy differences from thermodynamic integration molecular dynamics simulations, Chem. Phys. Lett., 1992, 196(3-4), 297-302.
-Pearlman, D.A., Determining the contributions of constraints in free energy calculations: Development, characterization, and recommendations, J. Chem. Phys. 1993, 98(11), 8964-8957.
One implementation, based on integrating over the constraint forces, can for example be found in the MD code GROMOS. I suggest to mention the availability of both the theory and also practical implementations in the manuscript.
-In section 7.2.2, when discussing changes in net charge the “Double-system/single-box” setup described by Gapsys et al. may be mentioned. It is related to the alchemical ion approach but can be mentioned here for completeness and to help newcomers to the field in sorting the various approaches.
Gapsys, V., Michielssens, S., Peters, J. H., DeGroot, B. L., Leonov, H. Calculation of Binding Free Energies, In Andreas Kukol (ed.), Molecular Modeling of Proteins, Methods in Molecular Biology, vol. 1215, doi: 10.1007/978-1-4939-1465-4_9, Springer Science+Business Media New York 2015.
Also this:
I agree with the notes in 7.2.2. that no agreed upon way to handle changes in total charges is available. The authors might consider adding (if they agree) that whenever there is a change of total charge, direct comparison to experiment is meaningless, because in experiments the total charge is always conserved (otherwise the test tube would explode). One might even go so far as to say that any calculation with change in total charge is meaningless…which kind of solves the problem from an unexpected angle.

I'd add the references/make the suggested changes.
I also agree with the point on section 7.2.2; this is one reason why doing hydration free energies for ions is a bad idea.

(Note though that this point does not exactly apply to binding, when one is transferring the ligand from the binding site to solution, so there doesn't have to be a change in formal charge.)

Add the recent body of work of Oostenbrink and co-workers on charge corrections in binding free energy calculations. I don't think any of the charge correction approaches have been sufficiently tested  in depth to establish a best practice

@mrshirts should comment on the constraint issue, as he dealt with this iN GROMACS at one point/investigated it. He may be in a better position to write something on that.

Pinged Shirts on Slack.

I added some comments on section 7.1 on this.  GROMACS supports the constraint changes in TI, but not in potential energy differences (so not BAR or MBAR).  I qualified what does and doesn't support it, and told people to look at their user manuals to check.

Thanks you, I also added a couple more of the suggested references.


add issue comment github action workflow

add workflow file with an gh-action to perform issue comments on new created issues to inform catena-x-gh-teams members about this new issue.
based on the personal user prefered notification rule for the @mentioning parts you will recive a GitHub Notification or a Email that you were mentioned in this comment

Added public GitHub action:
https://github.com/marketplace/actions/create-or-update-comment
At the moment within Eclipse Tractus-X GitHub Organisaition there will be no

create and maintain custom tailored GitHub teams.

At this point its just a check if you can reference diffrent organisation GitHub teams to be notified.

Update:
Unfortunatly GitHub-Teams mentioning between different GitHub Organisations is not possible, so modified the comment section within this workflow to inform Issue Creator that we have recieved the issue and inform each member of the System Team personally over the comment section.


Cellranger module

scRNA-seq analysis tool

What's happening with this module? Is anyone actively working on it?


Issue292 Demodularize gist

@sa-bpelakh , I don't think it invalidates module imports.  I think if someone imports an un-versioned module other than gistCore,, e.g. gistAddress, that they'll get 9.2.0, and its imports closure (all 9.2.0).  Which is (and ever will be) the most recent version of gistAddress (and its imported modules).  Of course, this is why I think it is better to import a specific version of an ontology (use versionIRI), so you know exactly what concepts you are committing to.

@marksem I agree that imports should always use versioned IRIs; it doesn't make sense to import something when you don't know what you're getting. But (to state the obvious) what makes sense is not necessarily what happens in practice.
If someone imports unversioned gistCore and (redundantly) unversioned gistAddress, there is the potential for unresolvable clashes, since they'll get version 9.3.0 (or whatever) of gistCore, and 9.2.0 of gistAddress, and there may be conflicts between term definitions in the two versions. (See https://www.w3.org/2007/OWL/wiki/Imports#Clashing_versions). Note that you could say the same thing if they import, say, gistCore9.2.0 and gistAddress9.1.0, but this is specifically prohibited by the OWL spec: "The import closure of O should not contain ontologies O1 and O2 such that O1 and O2 are different ontology versions from the same ontology series."

Wonder if this would be a good time to switch to Turtle format. :)

@marksem Sorry, when you said "maybe now's the time" I thought you meant this release, not this PR. There is a separate issue #223 for this, which hasn't been decided on, and in any case I don't think this change should be included in this PR in order to keep things atomic. What do you think?

We've determined this will be a minor rather than major change, based on the fact that we encourage users to import versioned rather than unversioned IRIs.

@rjyounes said:

There is a separate issue #223 for [Turtle], which hasn't been decided on, and in any case I don't think this change should be included in this PR in order to keep things atomic. What do you think?

I agree. Good practice.

Fixes #292


[SMALLFIX] Test improvement in zk failure test

This prevents the logs from getting spammed by NPEs when we shut down the cluster without stopping the operation thread.

alluxio-bot, check this please

Merged build finished. Test PASSed.

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


Nuget Install Issues

Created a new ASP.net core web app (.net framework), targeting .net framework 4.6.2 and slack.webhooks 0.1.6.
Error in install:

error: Package ServiceStack.Text 3.9.71 is not compatible with net462 (.NETFramework,Version=v4.6.2). Package ServiceStack.Text 3.9.71 supports:
error:   - net35 (.NETFramework,Version=v3.5)
error:   - sl4 (Silverlight,Version=v4.0)
error:   - sl4-wp71 (Silverlight,Version=v4.0,Profile=WindowsPhone71)
error:   - sl5 (Silverlight,Version=v5.0)
error: One or more packages are incompatible with .NETFramework,Version=v4.6.2.

Also the 0.1.6 version fails on install in a class library targeting 4.6.2 - doesn't add a reference to slack web hooks assembly. Don't see the above error though.

Reverting to 0.1.5 fixes problem 2 above with .net standard and it not adding the reference to slack.webhooks. Doesn't help the asp.net core issue though.

Can you install RestSharp in a dotnet core project? If so, I may have a solution on another branch removing the ServiceStack.Text dependency.

Doesn't look like it. But there is a RestSharp.Core (https://www.nuget.org/packages/RestSharp.NetCore/)
IMO, I'd just use HttpClient in System.Http
Package RestSharp 104.4.0 is not compatible with netcoreapp1.1 (.NETCoreApp,Version=v1.1). Package RestSharp 104.4.0 supports:

net35 (.NETFramework,Version=v3.5)
net35-client (.NETFramework,Version=v3.5,Profile=Client)
net40 (.NETFramework,Version=v4.0)
net40-client (.NETFramework,Version=v4.0,Profile=Client)
sl4 (Silverlight,Version=v4.0)
sl4-wp71 (Silverlight,Version=v4.0,Profile=WindowsPhone71)
Package ServiceStack.Text 3.9.71 is not compatible with netcoreapp1.1 (.NETCoreApp,Version=v1.1). Package ServiceStack.Text 3.9.71 supports:
net35 (.NETFramework,Version=v3.5)
sl4 (Silverlight,Version=v4.0)
sl4-wp71 (Silverlight,Version=v4.0,Profile=WindowsPhone71)
sl5 (Silverlight,Version=v5.0)

I've published 1.0.0-rc1 which removes RestSharp completely and should support netcore if you'd like to give it a whirl?


Add optional typed base class for dynamic library API

The main benefit would be automatic method name completion and type validation provided by IDEs.
Usage would be something like this:

from robot.api import DynamicLibrary

class Library(DynamicLibrary):
    ...

The base class should have all mandatory and optional dynamic API methods with appropriate type information and documentation. The mandatory methods should raise NotImplementedError and optional should return dummy information that's equivalent to not having these methods at all. We could also add a separate base class for the hybrid API although it only has get_keyword_names.
This is so easy to implement that I believe it could be done already in RF 6.1. The hardest part is deciding what the class name should be and should it be imported directly from robot.api or somewhere like robot.api.libraries, robot.api.interface or robot.api.base. A separate submodule could make sense if we add other similar base classes.

We can also add a base class also for the hybrid API. It will be pretty simple because it has only one method.

API docs for these base classes is visible here:
https://robot-framework.readthedocs.io/en/master/autodoc/robot.api.html#module-robot.api.interfaces
Sphinx doesn't see type aliases that have been used in signatures, and thus doesn't show List[Name] or Optional[ArgumentSpec] but instead has List[str] and Optional[List[Union[str, Tuple[str], Tuple[str, Any]]]]. Fixing that seems to require so much configuration that I don't consider it worth the effort.

Some fixes and enhancements in 929055be62cf03d9ac44bd4ee02466a6cf7fefc2.
Some testing with Robot, Libdoc and Mypy was done using the following code, and after the aforementioned fixes everything worked fine. Running this with Robot or Libdoc as part of out acceptance tests wouldn't bring much benefits going forward, because it's very unlikely our possible changes would break them. Testing with Mypy could be useful, but hooking it up to our test system is likely too much work compared to benefits. Easier to run it manually if typing is changed.
from typing import Any, List, Optional

from robot.api.interfaces import (Arguments, Documentation, DynamicLibrary, Name,
                                  NamedArgs, PositArgs, Source, Tags, Types)

class DynLib(DynamicLibrary):

    def get_keyword_names(self) -> List[Name]:
        return ['kw']

    def run_keyword(self, name: Name, args: PositArgs, named: NamedArgs) -> Any:
        return 42

    def get_keyword_arguments(self, name: Name) -> Optional[Arguments]:
        return ['a', ('b', 2)]

    def get_keyword_documentation(self, name: Name) -> Optional[Documentation]:
        return 'the doc'

    def get_keyword_tags(self, name: Name) -> Optional[Tags]:
        return ['t', 'a', 'g', 's']

    def get_keyword_types(self, name: Name) -> Optional[Types]:
        return {'a': int, 'b': (int, float)}

    def get_keyword_source(self, name: Name) -> Optional[Source]:
        return ':12'

I consider this issue done, bu comments are still appreciated here or on Slack.


Replacement for VersionNumber

Looks like org.gradle.util.VersionNumber was deprecated and moved to an internal class in 7.1. Is there any replacement which plugin authors can use that will survive the switch to 8.0?
This class always came in super handy when working with various version numbers, so I'd be sad to see it go without any replacement.

There are no plans to provide a replacement API right now.
Your best option is to either 1) copy the class into the plugin or 2) find an external library that does something similar.
The problem with VersionNumber is that there is no single standard way to format a version. This class takes an approach that's like many version formats, but it was always limited to what was useful for our specific needs and wasn't general purpose.
It's very tempting to pass a String into this and get something out that's a "version", but there are no guarantees that it makes any sense unless you know how the version format should be interpreted. The thinking is that it's best to leave version parsing/interpretation up to the plugin that has the context about the version.

Is there a recommended workaround for this? Was planning to use VersionNumber to force a minimum version of log4j due to CVE-2021-44228 without having to bring in additional plugins just to parse a version string:
        resolutionStrategy.eachDependency {
            if (requested.group == "org.apache.logging.log4j" && VersionNumber.parse(requested.version) < VersionNumber.parse("2.15.0")) {
                useVersion("2.15.0")
                because("CVE-2021-44228")
            }
        }


Nudge causing reboots when running /usr/sbin/softwareupdate --download --all

I was playing around with Nudge on my personal computer and just saw it try to force log me out.
I've confirmed the same behavior within a terminal (no sudo)
/usr/sbin/softwareupdate --download --all
Software Update Tool
Finding available software
Downloading macOS Big Sur 11.2.3
Downloaded: macOS Big Sur 11.2.3
Attempting to quit apps: (
    "com.apple.Compressor"
)
Waiting for user to quit any relevant apps
Successfully quit all apps
Downloaded Pro Video Formats
Done.

what's strange is I thought /usr/sbin/softwareupdate --install --all --force was the trigger for this, which Nudge does not do.

It appears that the bug is caused by having an update that requires applications to quit. After manually installing Pro Video Formats, Nudge and /usr/sbin/softwareupdate --download --all no longer cause logout/reboots.
This is most certainly an Apple bug
Erik@eriks-macbook-pro ~ % /usr/sbin/softwareupdate --download --all
Software Update Tool
Finding available software
Downloading macOS Big Sur 11.2.3
Downloaded: macOS Big Sur 11.2.3
Attempting to quit apps: (
    "com.apple.Compressor"
)
Waiting for user to quit any relevant apps
Successfully quit all apps
Downloaded Pro Video Formats
Done.

Erik@eriks-macbook-pro ~ % /usr/sbin/softwareupdate --list --all
Software Update Tool
Finding available software
Software Update found the following new or updated software:
* Label: ProVideoFormats-2.2.1
	Title: Pro Video Formats, Version: 2.2.1, Size: 8175K, Recommended: YES,
* Label: macOS Big Sur 11.2.3-20D91
	Title: macOS Big Sur 11.2.3, Version: 11.2.3, Size: 2383629K, Recommended: YES, Action: restart,

Erik@eriks-macbook-pro ~ % softwareupdate --install ProVideoFormats-2.2.1
Software Update Tool
Finding available software
Attempting to quit apps: (
    "com.apple.Compressor"
)
Waiting for user to quit any relevant apps
Successfully quit all apps
Downloaded Pro Video Formats
Installing Pro Video Formats
Done with Pro Video Formats
Done.

Erik@eriks-macbook-pro ~ % /usr/sbin/softwareupdate --list --all
Software Update Tool
Finding available software
Software Update found the following new or updated software:
* Label: macOS Big Sur 11.2.3-20D91
	Title: macOS Big Sur 11.2.3, Version: 11.2.3, Size: 2383629K, Recommended: YES, Action: restart,

Erik@eriks-macbook-pro ~ % /usr/sbin/softwareupdate --download --all
Software Update Tool
Finding available software
Downloading macOS Big Sur 11.2.3
Downloaded: macOS Big Sur 11.2.3

https://openradar.appspot.com/radar?id=4987491098558464
How to replicate.

Spin up a VM running 11.0 -> 11.2.2
Install https://support.apple.com/kb/DL2020?viewlocale=en_US&locale=en_US
Open Terminal and run softwareupdate --list and confirm you have a Pro Video Format update and Big Sur update available
Open another terminal window and run man softwareupdate
Open Safari
Run softwareupdate --download --all

Once softwareupdate has finished downloading the updates, safari will be closed but your vi (man softwareupdate) session has prevented the force reboot
You can re-open Safari and rerun softwareupdate --download --all to reliably recreate the attempted force reboot
If you'd like to see the forced reboot, do not perform step 4

Nudge will no longer use softwareupdate until Apple fixes this: https://github.com/macadmins/nudge/commit/8ab838ca3b4975961cc10fc16858d18b128c1566

/usr/sbin/softwareupdate --download --all --background could work if nudge had an ability to run as root.

Hi @erikng, could we get this back into Nudge? This was fixed in macOS 11.4 beta 3 and is now in the GA release.

(Beta 3) Resolves an issue where softwareupdate -da would automatically install updates.

Yeah I'll get it back but currently on paternity leave. If someone wants to PR it I'll merge it.

Thanks @erikng! Hope you're enjoyed the time with family.

merged into development branch https://github.com/macadmins/nudge/commit/8b5c024cdd9b3a705834366aee9cf7796b89e4b6


Building no_std fails, even without any features

The README implies that a #![no_std] build should work if all the default features are disabled. There is even a build-nostd.sh in the repository.
However, I notice that script does not actually remove the standard library from the dependency tree. When I actually do that -- by compiling for an embedded target without a libstd -- it fails:
$ cargo build --no-default-features --target armv7a-none-eabi --jobs 1
   Compiling log v0.4.11
   Compiling bitflags v1.2.1
   Compiling cfg-if v0.1.10
error[E0463]: can't find crate for `std`
  |
  = note: the `armv7a-none-eabi` target may not be installed

error: aborting due to previous error

For more information about this error, try `rustc --explain E0463`.
error: could not compile `log`

To learn more, run the command again with --verbose.

It seems that even if this crate is #![no_std], its log crate dependency is not.
This crate seems necessary for a ton of debug!(...) prints and other things scattered throughout the code. It was added in da76625759068858b89c1a341f81d5b75d70022e, way back in 2017 (!) and looks rather difficult to remove.
I can hack around this myself, for my own purposes. But is there some possible upstream solution? If so, I would happily open a PR to add a CI job which builds and runs a really-no-kidding #![no_std] binary to prevent this from happening again.

Hmm, that is interesting.
The log crate itself suggests that it should be #![no-std]
https://github.com/rust-lang/log/blob/078e68fa1e27d3dd66ee524e0ef646cfb434c6f9/src/lib.rs#L273
#![cfg_attr(all(not(feature = "std"), not(test)), no_std)]

However, it seems that the argument --cfg feature="std" is passed to rustc when building the log crate, even when I set --no-default-features. Very strange.

You should be able to remove all debug stuff from a binary by using log crate compile time filters: https://docs.rs/log/0.4.11/log/#compile-time-filters
As for building failure if its the log crate causing problems maybe you should create an issue in that crate repo. I was trying some no_std builds in the past with a MIPS target and it was working but I don't remember details. Maybe log crate regressed since then. I must say I don't really use no_std regularly so there may be some issues and I will be happy to accept contributions improving this library for such use-case.

Well, this is interesting.
Apparently it's not your fault, and it's not the log crate's fault, but it's because of the log crate pulled in as a dev dependency.
I will close this, and consider a CI improvement PR in the future.


New Tab should launch in the original starting directory

Windows Terminal version
1.12.10982.0
Windows build number
10.0.19043.1706
Issue
I am stuck between two bad options for the starting directory option:

Launched From
Expected
Starting Directory .
Starting Directory %USERPROFILE%

"Open in Terminal" Context Menu
The folder you are currently in
✓
✓

Entering wt -d . in the explorer address bar
The folder you are currently in
✓
✓

Start menu / task bar shortcuts
$USERPROFILE directory
✘ System32
✓

Clicking new tab
The folder the first tab launched in
✓
✘ %USERPROFILE%

In short, the new tab behaviour works as expected when the "Use parent process directory" option is enabled, however I can't use this option because it breaks the start menu shortcuts (#11834 and #878)
Background
The use-case for this is that I often want to right click a folder to open in terminal, this opens in my default shell (e.g. WSL), but sometimes I actually want to be using a different shell in this directory (Powershell).
Or another example: I'm developing a project in C:\Users\jacob\projects\example, I want to right click open in terminal, and then have multiple tabs for running different tasks in the project

There's another option: You can use "Duplicate Tab" (Ctrl+Shift+D) to duplicate the current tab including its starting directory.

I like to use:

wt --window 0 new-tab -d .

to start a new tab, and have it be in the same directory that the first tab is in.

wt -w 0 nt -d .

is a shortcut for the same thing.  It's always in my command history, so I just use 'up-arrow' to find it.

Thanks @lhecker, but the problem is that I often don't want to duplicate the tab, since I would like to swap to a different shell.

@jacob-pro
I think I now I see what you mean.
If I have my profiles set to "Use parent process directory", things work well, except for the fact that invoking Terminal from the start menu now drops me into C:\Windows\System32.  (Which is rather nostalgic.)  Both  C:\Windows\System32  and  %USERPROFILE%   meet the criteria of "almost NEVER where I want to be", so I had never noticed that being dropped into C:\Windows\System32 was inconvenient, because %USERPROFILE% is just as inconvenient for me.  But I can see how System32 would be pretty irritating if you mostly needed to be in %USERPROFILE%.
However, aside from the above issue, I am able to launch a new tab into a shell of my choice, in a directory that makes sense for the current session, by using the following command (which is itself a bit inconvenient, admittedly):

wt --window 0 --profile 'Command Prompt'

Perhaps more convenient is that clicking the profile selection dropdown in Terminal and choosing a shell from there also drops me into the expected directory.  (Meaning the directory that this wt.exe window was originally launched in.)
In other words, by starting the session by right-clicking on the 'directory of the day' in Windows Explorer, and choosing "Open in Terminal", things will get set up such that all other tabs opened via the Terminal GUI controls in that particular window will open in that same directory.
Launching a different shell in the same directory via a command:

Launching a different shell in the same directory via the GUI:

Yep exactly @AltitudeApps if you have  "Use parent process directory" set then new tabs work as expected, and as a bonus you can also run wt.exe in your explorer address bar without the -d . hack.
IMO "Use parent process directory" is how it should work by default, because that is how all other shells on Windows - cmd.exe, powershell.exe normally work.
But as I already noted this breaks the start menu / task bar shortcuts, since they seem to default to System32. Normally (if this was any Win32 application) I would just create a custom shortcut with the right starting directory set, but this doesn't work properly for Windows Apps: https://github.com/microsoft/terminal/issues/12996#issuecomment-1148048802

But as I already noted this breaks the start menu / task bar shortcuts, since they seem to default to System32.

Honestly that's like 95% of why we added startingDirectory in the first place. It remains atrocious that applications launched by "PLM" (the "Package Lifetime Manager" or whatever) start in System32.
We tried to fix it at one point, but it turned out to be even worse than we thought. You can see the damage in Mike's comment here: https://github.com/microsoft/terminal/pull/3547#issue-521851468

@DHowett IMHO you should have merged that PR to open in the working directory by default, that makes almost everything work how it is expected by developers, and is consistent with CMD/Powershell, basically any other shell application.
Leave the Start Menu & Taskbar shortcuts opening in System32 problem to the Microsoft Windows team - since it is their silly problem to fix 😆  - All they need to do is give us the ability to modify the start menu shortcut to have a custom working directory, just like the shortcuts for powershell and cmd already do.

There's another option: You can use "Duplicate Tab" (Ctrl+Shift+D) to duplicate the current tab including its starting directory.

Hmm, I was expecting when I duplicated the tab for it to bring me to the current directory that I was in on that tab...

@michael-hawker I don't think that is technically viable, since Windows Terminal supports all sorts of shells (cmd, mingw bash, wsl etc.) and there is no way / standard API for them to communicate back what directory they are currently in

@jacob-pro interesting, there's a discussion about it for #10232, so I wonder what they're thinking there?

There actually totally is a standard way for the shell to tell the terminal about the CWD. There's no good way for the Terminal to ask, but the client app can tell the terminal. See Tutorial: Opening a tab/pane in the same directory in Windows Terminal

Ultimately, I don't think we're about to change the base behavior here. However, reading between the lines a little, the bit that gets me is:

Launched From
Expected
Starting Directory .
Starting Directory %USERPROFILE%

Clicking new tab
The folder the first tab launched in
✓
✘ %USERPROFILE%

That's probably a pretty unique workflow, tbh, but I think there's another request on the backlog that might address it: #445. With the proposal in that thread, you'd be able to rebind the action of the new tab button. So, you could change it from just newTab to a newTab action, with the startingDirectory changed to .:
"newTabButton":
{
  "command": { "action": "newTab", "startingDirectory": "." }
}

That's probably the way I'd go about solving this, at least. I'd go upvote that thread as a potential solution here.


USHIFT-1240: add runner for robot framework tests

I could not figure out how to make the Makefile let me run one test at
a time with valid arguments, so I moved most of the content to a wrapper
script to use instead.

/assign @pacevedom

/assign @ggiguash

@pacevedom I added the top-level Makefile target, as we discussed. I removed the Makefile in the test directory as part of that.
@ggiguash I added some setup instructions to explain the configuration file. Let me know if that addresses your concerns about the defaults.

This is LGTM for me. I'm delegating the final approval to @pacevedom

/override ci/prow/microshift-e2e-arm
/override
ci/prow/e2e-openshift-conformance-reduced-arm
Overriding both of these jobs since arm is flaky in AWS until we figure it out. Failures include unexpected restarts which render connection timeouts and refusals.

/override
ci/prow/e2e-openshift-conformance-reduced-arm

/override ci/prow/e2e-openshift-conformance-reduced-arm


Resolve normal urls

resolve urls like https://imgur.com/1UdA2uK to https://i.imgur.com/1UdA2uK.png
this can be supported by checking the open graph meta tags from the fetched page if the content-type is a HTML page

Closed in #11


ap.show_letter('1')

The function show_letter is not compatible in displaying the number 1 on the screen. Line 396 states that len(s)  > 1 when len(s) > 0 to make it compatible with all numbers. I found this issue at a Raspberrry Jam with a fellow GitHub user. All you need to do is change 1 to 0 to make it work.
Daniel Harrin age 12
babybigfox

Sorry - I got that wrong - I don't know how to fix it... Ignore what I said

The ap.show_letter error is still there when I display the number one. Any help?

This is not an issue. I am guessing babybigfox is not in possession of an Astro Pi HAT otherwise they would have just run ap.show_letter('1') and seen that it does indeed work.
Line 396 of the module is testing the string length (using len) that has been passed into the function, not the content of the string. It will only throw the ValueError exception, for example, if you run ap.show_letter('yo') where you're sending in a string of length two.

I have an Astro Pi hat and I found the issue with an organiser of the Jam when I tried to program the pi to count down from 9 to 0 which got stuck on 2 every time. Would you be able to suggest some coding to do the count down as I am new to Python and obviously need help!

Please post your code here and I'll take a look.

I have posted it...but it needs to be approved by a moderator before you can see it...

Turns out this was a bug! Well done for finding it =)
So in your code you used ap.show_message('1') as well as ap.show_letter('1'). The show message function has code in it to trim the white space from the back and front of the 5x7 text character so that long messages don't have huge gaps in them.
The internal _trim_whitespace function was incorrectly modifying the global copy of all the characters stored in _text_dict. So I've done a fix which means that the internal _get_char_pixels function that safeguards _text_dict always returns cloned copies of the 5x7 character. Then they can be trimmed by _trim_whitespace and discarded without it changing the original in _text_dict.

Thanks for that update but the pi now shows an error, all to do with show_letter, that pixel lists 'must have 64 elements'. Any help?

Thanks for that update but the pi now shows an error, all to do with show_letter, that pixel lists 'must have 64 elements'. Any help?

Thanks for the advice. I have don the updates and restarted my pi but it still shows the error
Traceback (most recent call last):
File "ReactionPi.py", line 182, in
menunum() # Start the whole program!
File "ReactionPi.py", line 84, in menunum
write()                                 # Enter pressed
File "ReactionPi.py", line 159, in write
ap.show_letter('1') #
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 378, in show_letter
self.set_pixels(coloured_pixels)
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 197, in set_pixels
raise ValueError('Pixel lists must have 64 elements')
ValueError: Pixel lists must have 64 elements
when it gets to the number 2 in show_letter 1 during the countdown in my function named write()
Anything wrong with my coding?

Thanks for the advice. I have don the updates and restarted my pi but it still shows the error
Traceback (most recent call last):
File "ReactionPi.py", line 182, in
menunum() # Start the whole program!
File "ReactionPi.py", line 84, in menunum
write()                                 # Enter pressed
File "ReactionPi.py", line 159, in write
ap.show_letter('1') #
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 378, in show_letter
self.set_pixels(coloured_pixels)
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 197, in set_pixels
raise ValueError('Pixel lists must have 64 elements')
ValueError: Pixel lists must have 64 elements
when it gets to the number 2 in show_letter 1 during the countdown in my function named write()
Anything wrong with my coding?

Thanks for the advice. I have don the updates and restarted my pi but it still shows the error
Traceback (most recent call last):
File "ReactionPi.py", line 182, in
menunum() # Start the whole program!
File "ReactionPi.py", line 84, in menunum
write()                                 # Enter pressed
File "ReactionPi.py", line 159, in write
ap.show_letter('1') #
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 378, in show_letter
self.set_pixels(coloured_pixels)
File "/usr/local/lib/python2.7/dist-packages/Astro_Pi_HAT-0.0.2-py2.7.egg/astro_pi/astro_pi.py", line 197, in set_pixels
raise ValueError('Pixel lists must have 64 elements')
ValueError: Pixel lists must have 64 elements
when it gets to the number 2 in show_letter 1 during the countdown in my function named write()
Anything wrong with my coding?

Please use the raspberry pi forums for technical support.

ok


SSL error when downloading ruby

Trying to run the installation script or manually git clone'ing and then running brew results in the same error:
$ brew -v
Error: Download failed: https://homebrew.bintray.com/bottles-portable/portable-ruby-2.0.0-p648.x86_64_linux.bottle.tar.gz
Error: Failed to install vendor Ruby.

It looks like although wget is happy to download the file, curl is throwing some kind of SSL error.  This is on a fully updated Ubuntu 14.04 machine.

I should say that I bootstrapped this by manually downloading the portable-ruby bottle to ~/.cache/Homebrew, then running brew install -s for each dependency of curl until I was able to just run brew install -s curl.  Now things are working okay, I assume because the TLS stack I'm using is now completely new (installing openssl has downloaded new certificate chains, my curl version is the latest 7.52 now, instead of the 7.47 that trusty comes with).

Thanks for the bug report, Elliot. I don't have a fix for you, thanks for the workaround. I'll leave this issue open, in case it affects other people.

I thought that I recalled an environment variable for either openssl or curl that allows it to use an insecure connection without giving an error similar to curl -k,--insecure but I'm not finding it now with a quick Google.

Maybe the solution is to switch to wget, of that works?

My suggested work around is:
curl -k -o ~/.cache/Homebrew/portable-ruby-2.0.0-p648.x86_64_linux.bottle.tar.gz https://homebrew.bintray.com/bottles-portable/portable-ruby-2.0.0-p648.x86_64_linux.bottle.tar.gz
brew -v


Implement the emp info endpoint

Implement the emp info endpoint. Address https://github.com/remind101/emp/issues/13.

Looks great! :+1:


fix: green text is lighter and easier to read on night mode

Pre-Submission Checklist

[X] Your pull request targets the staging branch of freeCodeCamp.
[X] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[X] You have only one commit (if not, squash them into one commit).
[X] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[X] Tested changes locally.
[X] Addressed currently open issue (replace #17501  with an issue no in next line)

Closes #17501
Description

Old night mode .green-text color

New night mode .green-text color

@renojvarghese
Looks better now..

@raisedadead I think nightmode could be a lot better. Would you be open to a redesign, including styles for the code editor?

I think nightmode could be a lot better. Would you be open to a redesign, including styles for the code editor?

YES! PLEASE!....
What you would want is to locally setup https://github.com/freeCodeCamp/learn

Also Contributors Chat room is where we hangout.


Cannot reproduce the results even with the same random seed

Thanks for sharing the code! The random seed in train_teacher.py seems not to work as every time run python train_teacher.py --exp_setting tran --teacher SAGE --dataset cora will generate different results even with the same seed. Accordingly, we cannot reproduce the exact same results as stated in the paper when running bash experiments/sage_cpf.sh. This seems a bug since the point of random seed is to reproduce the results. Could you please fix this?
The two different results with same random seed 0 (python train_teacher.py --exp_setting tran --teacher SAGE --dataset cora):

The results of bash experiments/sage_cpf.sh, which is different to the paper:

Dear meettyj,
Thank you for your interest in our work. What you pointed out is indeed a concern but I don't think it is a bug in our code and it doesn't affect the conclusion of our paper.
We have tried our best to guarantee reproducibility by fixing random seeds. However, there are certain parts of the code that are random but beyond our control. Our code is based on the DGL platform, and the SAGE implementation is adapted from the DGL official examples, which unfortunately involves sampling steps that produce different results even when we fix all the seeds (you can read more here https://github.com/dmlc/dgl/blob/master/examples/pytorch/ogb/ogbn-products/graphsage/main.py). Other models, like GCN, behave better and provide consistent results over runs when the seeds are fixed. However, we have to choose SAGE as the base GNN for our study because the other models won't scale on large graphs. Therefore, we repeat each experiment multiple times and report average performance with std to rule out this kind of concern.
I just reran the first line of code you showed on my end, i.e.
python train_teacher.py --exp_setting tran --teacher SAGE --dataset cora
Unfortunately, this gave me 0.8351 and 0.8230.
However, when I replaced SAGE with GCN, GAT, APPNP, and MLP. I got consistent results over multiple runs as below
GCN 0.8244
GAT 0.8370
APPNP 0.8412
MLP 0.5705
You are welcome to try our method with these models on your end to see if you get the same numbers, or you may include other models implemented by yourself.
Nevertheless, studying different GNN architectures is not the focus of our paper. The results you provided are already very close to ours. All of them only differ by less than 1% and the differences are within 0.5 std. This level of inconsistency shouldn't cause too much concern and doesn't affect our conclusion at all.
Please let me know whether your concern is addressed or not, and I am happy to discuss if you have further questions.
Best,
Shichang

Dear Shichang,
Thank you so much for your detailed answer and the well-written code! This makes sense to me if the randomness is coming from the sampling of SAGE. I will close this issue accordingly :)


AssetHub Kusama minting error (Nova wallet)

What happened?
An artist, Calta (tg: Calta99) has an issue while minting her collection on Statemine via her phone - using Nova wallet. On production.
Error message:

Please reproduce in steps
Create Classic (Collection on AssetHub Kusama)
Expected Behavior
mint the NFT without the error
What browsers are you seeing the problem on?
No response
At which address did you encounter bug?
No response
Are you logged in?
None
Which wallet you are using?
No response
Screenshots
added
Relevant log output
No response
Payment link for reward
https://kodadot.xyz/ksm/transfer/?target=<My_Kusama_Address_check_https://github.com/kodadot/nft-gallery/blob/main/CONTRIBUTING.md#creating-your-ksm-address>
Code of Conduct

[X] I agree to follow this project's Code of Conduct

She tried again and now she has this issue. She cannot get through the collection creation


⏫ Handle Null values for seedlots coming from Oracle

We are assuming some fields are non-null values because we do data validation on SPAR 2 before a seedlot can be submitted.
but these values can be null coming from oracle, we need to update the type definition on the front-end and make changes to handle null values
To reproduce this error, click on a seedlot that does not have a species code in the "my seedlots table"

done