CypherQueryCorrector does not handle labels with :, such as biolink:Disease

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
I am following the doc: https://python.langchain.com/v0.2/docs/how_to/graph_mapping/
using a database where Neo4J labels have colon, for example, biolink:Disease, biolink:treats.
This seems to break the CypherQueryCorrector from langchain.chains.graph_qa.cypher_utils import.
The query is corrected to "" even when it is valid.
Error Message and Stack Trace (if applicable)
No response
Description
See above.
System Info
langchain==0.2.6
langchain-cli==0.0.25
langchain-community==0.2.6
langchain-core==0.2.11
langchain-experimental==0.0.62
langchain-text-splitters==0.2.2

It's using regex, so it seems this is an edge case that wasn't handled


ARO-9570: Add a controller to the ARO operator to lay down etc hosts machine config

Which issue this PR addresses:
ARO-9570

Fixes
What this PR does / why we need it:
This PR adds a MachineConfigs for the ARO operator to lay down:
99-master-aro-etc-hosts-gateway-domains
99-worker-aro-etc-hosts-gateway-domains
During the UDR installation incident we have gone back and forth on how resolve the root issue. The root cause is that the gateway domains are not populated for OpenShift images to be pulled and installed. These domains are IPs so we can pass them into /etc/hosts to resolve them before other DNS resolution options are available.

Test plan for issue:
I've some minimal smoke testing on the operator with this PR.

Is there any documentation that needs to be updated for this PR?

How do you know this will function as expected in production?

/azp run ci

/azp run e2e

No pipelines are associated with this pull request.

@lranjbar yea - kipp is right - can you move this to a branch under ARO-RP so we can run our pipelines? This is a result of security hardening... you can run make init-contrib to set up some local hooks on branch naming strategies, generally, <GITHUB_USERNAME>/<JIRA_ID><other-optional-text>.


update spring boot version

We should wait with merging this until spring boot is updated to use spring 6.1.14, because of the CVEs

We should wait with merging this until spring boot is updated to use spring 6.1.14, because of the CVEs

Spring boot 3.3.5 is released now which uses Spring Framework 6.1.14
--> https://github.com/spring-projects/spring-boot/releases/tag/v3.3.5
@twatzl could you update this PR please?

updated


New version: JetBrains.Rider.EAP version 241.13688.41

Created by 🥟 Dumplings in Run #3488.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run

This package installs and launches normally on a Windows 10 VM.


Update frameworks.md

Added documentation to state that F# is supported with Xamarin.

@willsam100 thanks, we'll take a look.


'And' keyword is not working, on commenting 'And' line the test works!

What did you see? 👀
And keyword is not working, on comment And line the test works!
What did you expect to see? 😎
And keyword should also work like other keywords
Which version are you using? 🎁

io.cucumber
cucumber-java
7.2.3
test

io.cucumber
cucumber-core
7.2.3

io.cucumber
cucumber-testng
7.2.3

## How could we reproduce it? 🔬

Use cucumber with java and testng

I would expect a parser to be lenient towards different space characters. However in this case the space is hardcoded into the keyword definition.
https://github.com/cucumber/common/blob/main/gherkin/gherkin-languages.json#L745

Maybe, a parser post-processing task that normalizes the step and removes/replaces the TABULATORs with SPACEs (and maybe removes duplicated SPACEs) would be a good idea !?
BUT: The question is how often does this syndrome occur ?

It is indeed weird; if we want to fix it I think we'd need to make the whitespace (regex?) part of the Dialect rather than building it into the parser, as I'm not sure if all other languages use the same spacers to separate words.
(It may be an idea to make the keyword separator (:) part of the dialect too)


Request: Thumbnail delay or multiple thumbnails

When the motion is triggered, the object causing the motion is only partially visible. So a delay, or multiple thumbnails at a given interval would be appreciated.
Thanks
FlorisJan

If you are talking about video motion before the trigger then this is what the buffer facility is for. Under camera settings set video buffer to say 4000. It then records into a RAM buffer all the time. When a motion trigger occurs then the RAM buffer is prepended onto the recording so you get a few seconds before the trigger occurs.
If you are talking about the recording stopping to soon then adjust the motion parameters so that recording needs a longer still period before a motion stop trigger is generated.
If you are talking about snapping an image then not a lot can be done for capturing before a motion trigger occurs. You could use a low frame rate video instead.

Thanks for your reponse Robert!
I would like for the thumbnail to be taken a short period after the motion trigger, not before. So the object is fully visible, not just the part causing the trigger. I can of course grab a copy of the jpg stream myself using python, but this is very cpu consuming.
FlorisJan

I want to be clear what you are trying to achieve here as I can interpret this 2 ways. First is where you want to capture an image and its corresponding thumbnail after a delay from a motion trigger. Second is where you are doing normal motion triggered video captures but you want the thumbnail associated with this to be sometime after a motion trigger is detected.
A potential solution in both cases could be achieved with a macro to delay the command sent as a result of the motion trigger.
So taking the first case if scheduler is sending an im command in response to a motion trigger then instead you send a sy command to execute a macro. The macro just sleeps for the required delay and then sends im to the PIPE.
In the second case you again substitute the ca 1 command by the macro which delays then issues the ca 1 command. Now by delaying the start of the video capture you may think that you lose some of the video but if you use the buffer then you are already including several seconds before the motion trigger occurred. You see this effect already if using the buffer because the thumbnail capture occurs several seconds into the captured video.

I meant the second one, sorry about my fuzzy explanation and thanks for answering both possibilities.
The solution sounds pretty good. Also taking into consideration my comment in thread #282, because by replacing the ca 1 command with a macro, which then sends the ca 1 command, the commands cannot collide in the FIFO.
Thanks, I'll let you know how it works.
Floris Jan

I've integrated the additional thumbnail into my python code so I can control exactly when it is 'taken'. Therefore I haven't tried the macro solution but I'm sure it would also work. Thanks again for your help.


Upgraded ~25 gems in Gemfile file

Upgraded roughly 25 gems in Gemfile file.

new upgrade PR - FYI

Should have asked before creating PR so pruning.

Al - I'm back around (travel), so did you not want to update the gems?

You say assume I am always updating the gems. You just have to ask for a PR.


next: Dropdown Menu Label Missing

Describe the bug
In the current version of bits-ui, there is a Label component for a visual label for a group of items.
In next, there is not this same component.
I'm having a hard time with dependencies in stack-blitz to get a repro created there so I've attached a repo instead. Essentially the label export doesn't exist on the next DropdownMenu object.
I'm not sure if this is intentional (I looked through issues and pull requests and didn't seem to find one that fit that this would to be removed, but I could've missed it), but I figured I'd start an issue just in case it wasn't.
Reproduction
https://github.com/JYLN/bits-ui-next-dropdown-label-repro
Logs
N/A

System Info
System:
  OS: Linux 5.15 Ubuntu 22.04.5 LTS 22.04.5 LTS (Jammy Jellyfish)
  CPU: (24) x64 AMD Ryzen 9 7900X 12-Core Processor
  Memory: 1.73 GB / 7.76 GB
  Container: Yes
  Shell: 5.8.1 - /usr/bin/zsh
Binaries:
  Node: 20.18.0 - ~/.nvm/versions/node/v20.18.0/bin/node
  npm: 10.8.2 - ~/.nvm/versions/node/v20.18.0/bin/npm
  bun: 1.1.30 - ~/.bun/bin/bun

Severity
annoyance

Hello, @JYLN
I think it's now called GroupHeading: https://next.bits-ui.com/docs/components/dropdown-menu#dropdownmenugroupheading

@shyakadavis Oh you're absolutely correct! That's what I get for not reading the docs thoroughly enough 🤦 Thank you for your reply and pointing that out to me! I'm going to close this issue.

Thanks for raising the issue and thanks for the clarification
I was banging my head for good few hours not knowing why My dropdown was not working.


Supported Archtecture

I use lspinstall on multiple architectures, obviously lspinstall only runs on x86. It would be nice if it didn't install x86 binaries if I am on a different architectures.
For now I just have, I am fine if this is the long term fix:
if system("uname -m") == "x86_64l\n"
 do lsp stuff.
endif

Some servers already support ARM. I don't have a device to test on. Feel free to make PRs for this.

See terraform and rust for examples

Yes, looks like terraform and rust distribute binaries.  I don't think the ones I look at do.  For example: clangd.  Darn

You could open an issue there, asking if they would be willing to also provide arm binaries in their releases. IIRC they are pretty responsive.
https://github.com/clangd/clangd/releases

Yeah.  There is an open ticket already: https://github.com/clangd/clangd/issues/514
Guess we can wait until this is complete.


add rtm cpu feature intrinsics

Blocked by https://github.com/rust-lang/rust/pull/60060
This does not include any HLE functions (xacquire/xrelease prefixed operations).
Let me know what needs changing.

Thanks! It's merged, but not yet in nightly AFAICT. I'll ping you.

Can you rebase on top of master?

I have already done so. verify signatures on master is broken on CI after #733.
---- verify_all_signatures stdout ----
failed to verify `_rdtsc`
  * failed to equate: `__int64` and PrimUnsigned(64) for _rdtsc
thread 'verify_all_signatures' panicked at 'assertion failed: all_valid', crates/stdsimd-verify/tests/x86-intel.rs:186:5

Oh I thought the fix for that already had landed!

Thank you @mtak- for working on this!


[CCXDEV-10828] Add sentry logging

Adding logging to Sentry.
Testing steps
I tested with:
pip3 install -r requirements.txt && pip3 install . && uvicorn insights_content_template_renderer.endpoints:app --log-config logging.yml

adding division_by_zero = 1 / 0 in the rendered_reports entrypoint with
export<EMAIL_ADDRESS>export SENTRY_ENVIRONMENT=dev
export SENTRY_CATCH_WARNINGS=true

and
curl -d<EMAIL_ADDRESS>-H "Content-Type: application/json" -X POST http://<IP_ADDRESS>:8000/v1/rendered_reports

but I don't see any issues in https://glitchtip.devshift.net/ccx/issues?project=93

done in https://github.com/RedHatInsights/insights-content-template-renderer/pull/103#pullrequestreview-2221396856


Add test framework and demo test

Please, ask if something is unclear with the test.

You've added tests to the src/utils/__tests__ directory.
Can we make the project tree be like that instead:
src/
|- core/
|- utils/
tests/
|- core/
|- utils/

Note about __tests__ directory and .test.js postfix:
testMatch [array]
available in Jest 19.0.0+
(default: [ '**/__tests__/**/*.js?(x)', '**/?(*.)(spec|test).js?(x)' ])

The glob patterns Jest uses to detect test files. By default it looks for .js and .jsx files inside of __tests__ folders, as well as any files with a suffix of .test or .spec (e.g. Component.test.js or Component.spec.js). It will also find files called test.js or spec.js.

Further reading
So yes, we can move test files anywhere in the project tree as long as they have .test.js or .spec.js postfix. I think we can leave it as it is for now, and refactor if we encounter any issues.
Ok?


[SMALLFIX]  Fixing spot instance launch scripts

A couple of bugs are stopping spot instances being correctly launched using current scripts.
This PR fixes them.

I just added a minor comment. Thanks!

Thanks for the reviews.. I totally misread the original code, which lead to a non-fix. (which actually somehow helped me to launch instances correctly once..) so I reverted that.
I think I do have it fixed now though, basically the ip is sometimes not assigned right away by aws, so we often get None as ip address from aws.

LGTM

+1 LGTM


Add support for Kubernetes v1.28 and v1.29

Description of the feature or enhancement:
Add support for Kubernetes v1.28 and v1.29
Use Case:
This would integrate the K8s latest release with cdk8s+
Other:

[ ] :wave: I may be able to implement this feature request
[x] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Can we have an update on this issue please maintainers? We already have Kubernetes 1.29 and cdk8s-plus is still at 1.27.

This makes me reluctant to use this project. I know Kubernetes releases come fast, but it has been 6 months since the release of 1.28 and there appears to be no one keeping this up to date. If there were to be an actual issue instead of just slow release cadence then I would not believe there to be much help resolving it or fixes.

any updates on this ? tagging owners to make sure they see this:
@iliapolo
@Chriscbr
@tabern

Are they released now?
https://github.com/cdk8s-team/cdk8s-plus/releases/tag/cdk8s-plus-28%2Fv2.0.0
https://github.com/cdk8s-team/cdk8s-plus/releases/tag/cdk8s-plus-29%2Fv2.0.0


Better message for encoder_truncate_gradient

Ran this (which I assume is a slightly out of date model):
THEANO_FLAGS=mode=FAST_RUN,floatX=float32,device=gpu,on_unused_input=warn python /fs/magni0/heafield/ro-en/nematus/nematus/translate.py -m /mnt/baldur0/rsennrich/wmt16_neural/ro-en/exp6/model.npz -k 12 -n -p 1 --suppress-unk

Get an error message:
Process Process-1:
Traceback (most recent call last):
  File "/usr/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
    self.run()
  File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
    self._target(*self._args, **self._kwargs)
  File "/fs/magni0/heafield/ro-en/nematus/nematus/translate.py", line 42, in translate_model
    f_init, f_next = build_sampler(tparams, option, use_noise, trng, return_alignment=return_alignment)
  File "/mnt/magni0/heafield/ro-en/nematus/nematus/nmt.py", line 393, in build_sampler
    x, ctx = build_encoder(tparams, options, trng, use_noise, x_mask=None, sampling=True)
  File "/mnt/magni0/heafield/ro-en/nematus/nematus/nmt.py", line 229, in build_encoder
    truncate_gradient=options['encoder_truncate_gradient'],
KeyError: 'encoder_truncate_gradient'

Note that this config does not mention encoder_truncate_gradient at all.

fixed. There's a separate place (compat.py) which we use to set default options for backward-compatibility.
Default options for old models may be different from the default options in the training script (for instance, the handling of dropout is different in newly trained models, but we keep support for old models in decoding).


Broken layout: debian.org

Current HEAD of master is unable to correctly capture the layout of https://debian.org with Chromium 80.0.3987.132 built on Debian 10.3, running on Debian 10.3, Python 3.7.3.

It's not expected to be able to perfectly reproduce every page, wget output often isn't perfect which is why it also saves the PDF, screenshot, and DOM dump separately.  Have you looked at all the outputs or is it just the wget one thats's broken?
If there's an error message that's ArchiveBox related and not caused by the origin, I'm happy to reopen the ticket.
If you need perfect fidelity archives I recommend checking out some of the alternatives here: https://github.com/pirate/ArchiveBox/wiki/Web-Archiving-Community#Web-Archiving-Projects

I don't understand why this issue is closed. A reasonably straightforward web site captured improperly (I don't know whether wget is to blame or anything else).
This bug report is for you to check why structure of the web site was not reproduced.
This is a legitimate problem that might have a solution. I'd appreciate if you could investigate and try to reproduce the problem instead of just closing the bug report.

I'm sorry but as a matter of policy I don't investigate archive fidelity issues when it's not an ArchiveBox bug (i.e. it's broken output from wget or Chrome).  I'd be chasing down hundreds of people's individual HTML/CSS archiving bugs if I did, I simply don't have the time to fix wget's bugs for them.  The ethos of archivebox is to provide multiple archive outputs for each site in order to cover the cases where an individual method fails.  We may also switch away from wget entirely in the future to https://github.com/rockdaboot/wget2 or Wpull.
You're welcome to attempt archiving the site yourself manually with wget, and if the issue is reproducible that way you can open an issue on the wget issue tracker to get it resolved on their side.
wget --no-verbose \
              --adjust-extension \
              --convert-links \
              --force-directories \
              --backup-converted \
              --span-hosts \
              --no-parent \
              -e robots=off \
              --restrict-file-names=windows \
              --timeout=60 \
              --warc-file=archive.warc \
              --page-requisites \
              --compression=auto \
              --no-check-certificate \
              --no-hsts \
              "https://debian.org"

If it's not reproducible with wget on its own, then it's indeed an ArchiveBox or config issue, and I'd be happy to reopen the ticket. (please provide the exact command you ran along with any config options you're using, and a screenshot of the broken output so I can investigate further)

Fair enough, thanks.
I tried the wget command that you've suggested and it captured the web site layout successfully.
Also 83197ef failed to detect page title (Failed: Unable to detect page title) while latest release 0.2.4 could capture the title correctly.
I'm just evaluating ArchiveBox so I tried to capture https://debian.org using command echo https://debian.org | archivebox without any additional config options.


Ability to Dump Application's Database

Is your feature request related to a problem? Please describe.
We have experienced sync issues due to malformed resources, caused by some references not properly set such as missing the resource type before the reference. Resources with such issues fail to sync since the referenced resources are not found on the server.
Identifying the cause of the failing sync has proven hard, especially when we are unable to reproduce the issue on our staging environment.
Describe the solution you'd like
To make troubleshooting possible for cases that are hard to reproduce, we would like a copy of the application's database to be downloadable to device storage for further investigations.
The dumped database file should be encrypted with a password that is defined by the app calling the database download SDK API.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Would you like to work on the issue?
Please state if this issue should be assigned to you or who you think could help to solve this issue.

@jingtang10  please feel free to review the PR On OpenSRP. I think we want to move that kind of functionality to the SDK

@jingtang10 We would like to revisit this issue and have the feature in the SDK. Please review the issue and share your thoughts.

cc @aditya-07 @MJ1998

We are happy to add this ability. Along with this there should also be a way for application to modify data before dump - so a plugin which application can implement and register.
Can you tell us how you are doing this in the PR linked above.
Would you work on this @qiarie  ?
Thanks.

@qiarie
We also wanted to understand the use-case more. Questions are around difference between staging and prod environment - and the possibility of replicating the scenario to troubleshoot the sync issue in prod environment ?
For example, one could upload all the logs to troubleshoot, no ?

Hello @MJ1998.

We are happy to add this ability. Along with this there should also be a way for application to modify data before dump - so a plugin which application can implement and register.

This is noted. Do you mind giving more information on there should also be a way for application to modify data before dump - so a plugin which application can implement and register.

Can you tell us how you are doing this in the PR linked above. Would you work on this @qiarie ? Thanks.

I will check with my team and share an update

@MJ1998

@qiarie We also wanted to understand the use-case more. Questions are around difference between staging and prod environment - and the possibility of replicating the scenario to troubleshoot the sync issue in prod environment ? For example, one could upload all the logs to troubleshoot, no ?

In our current use case and implementation, a user is able to initiate a raw database download to the file system.
For debug versions, the database is unencrypted and is saved as is to the file system.
For release versions, the app's database is encrypted. An empty database file is created. The encrypted database is read and its contents copied to the empty DB file.
In both cases, the database is zipped and a ZIP password set.

After internal discussions and careful evaluation, we've decided not to implement the database dump feature at this time. This decision aligns with our core principle of maintaining a secure, encrypted database environment. Additionally, we believe this feature may become less relevant as our sync capabilities mature.


⚠ ClamAV has degraded performance

In bc69143, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 77 ms

Resolved: ClamAV performance has improved in 3c0fb0e.


Set a value to undefined?

How do I set a value to undefined in a resource?
https://issues.jboss.org/browse/WFLY-3538
I've tried various versions of this:
  wildfly::resource { '/subsystem=ee/service=default-bindings':
    content   => {
      'datasource' => '',
    },
  }

  wildfly::resource { '/subsystem=ee/service=default-bindings':
    content   => {
      'datasource' => 'undefined',
    },
  }

  wildfly::resource { '/subsystem=ee/service=default-bindings':
    content   => {
      'datasource' => undef,
    },
  }

But none are actually setting the value to undefined, similar to  /subsystem=ee/service=default-bindings:undefine-attribute(name=datasource)

I also tried this:
wildfly::cli { 'Remove default datasource':
   command => '/subsystem=ee/service=default-bindings:undefine-attribute(name=datasource)',
   unless  => "(result == undefined) of /subsystem=ee/service=default-bindings:read-attribute(name=datasource)",
 }

However, that turns into 'resource' => 'undefined' for the check statement, yet the actual results from the CLI are:
[standalone@localhost:9990 /] /subsystem=ee/service=default-bindings:read-attribute(name=datasource)
{
    "outcome" => "success",
    "result" => undefined
}

I haven't figured out how to get it to check for undefined in the () check; is it documented somewhere?

The raw packet result (I sniffed lo to see what was going back and forth) gave us a JSON object:
{"outcome" : "success", "result : null}

Interesting.

wildfly::resource  is always removing undef values: https://github.com/biemond/biemond-wildfly/blob/master/manifests/resource.pp#L28. But it will probably not work with wildfly_resource either as I am not assembling undefined-attribute operations. 😞

wildfly::cli is not working cause API returns null instead of undefined as you pointed out in your last commentd.

Good news is that it's not hard to fix, just need to handle these different kinds of representations: undef, (Puppet), undefined (DMR) and null (JSON).
I'll work on this.

wildfly::cli is working for undefined comparisons now, you should be able to use the command you posted above.
Still need to change:

wildfly::resource to introduce a new parameter to allow attributes to be undefined.
wildfly_resource to convert undef to nil.
operation_request to convert undef to nil.

The changes above were made and it should be working right now, even for wildfly::resource using undefine_attributes => true.


Bring gateway-mfr-rs up-to-date

Related to: https://github.com/NebraLtd/hm-pyhelper/pull/148
To align with upstream, we need to upate gateway-mfr-rs, some parts of the api has changed.
We can make it generic like helium. For that we might have to introduce following:
SWARM_KEY_URI eg "ecc://i2c-1:96?slot=0"

This contains all the information that we might need for swarm key or any other purpose. eg. type, bus, address, key slot.
we can keep the old ECC_STORAGE_BUS for sometime.
Acceptance Criterion

Use latest gateway-mfr-rs 0.2.1

SWARM_KEY_URI format:
ecc:<dev>[:address][?slot=<slot>]

dev the device file name (usually begins with i2c or tty)
address the bus address (default 96)
slot the slot to use for key lookup/manipulation (default: 0)

Given that we bundle gateway_mfr with this repo, we should be able to make this fully transactional.

Related with https://github.com/NebraLtd/hm-pyhelper/pull/148

@posterzh can you recommend a way for us to validate that these changes are working in the fleet?

@marvinmarnold
Working diagnostics page is the obvious proof.

Animal name comes from the ECC public key. The updated get_public_keys_rust is used under the hood while getting the animal name.
"Ecc Detected: True" comes from updated detect_ecc() method.

If get_public_keys_rust() is broken while developing, even the diagnostics service couldn't start.

Thanks @posterzh. @KevinWassermann94 can you work with Wayne to confirm the diagnostics page is showing the correct public key for devices running the newest release?

@marvinmarnold @KevinWassermann94 The version of gateway_mfr_rs can only be checked in the SSH.
Running /opt/python-dependencies/hm_pyhelper/gateway_mfr --version in the diagnostics console should give gateway_mfr 0.2.1 as the response.

@posterzh Thanks Zhang, good work! This is working


check_dipole fails for dipole with orientation

Hi,
first of all: Thanks for the amazing library and all the work you put into making it :-)
Description:
I am facing some issues with non major aligned dipole sources. When I supply a azimuth and dip parameter as stated in the API reference check_dipole(...) fails, because it expects (x, y, z) instead of the supplied (x, y, z, azim, dip).
Code to reproduce:
import empymod
import numpy as np

sigma_h = np.r_[1, 0.55, 1]     # [S/m] Layer conductivities horizontal direction
sigma_v = np.r_[1, 0.18182, 1]  # [S/m] Layer conductivities vertical direction

depth = np.r_[5, 10] * 0.3048 # [m] Layer boundaries
aniso = sigma_v / sigma_h     # [ ] Layer anis.
res = 1 / sigma_h             # [Ohm m] layer resistivity

# Modelling parameters
verb = 0
comp = 'H_zz'
ab = 66         # source and receiver z-directed

# Tool Parameters
freq = 20000        # [Hz] tool operation frequency
z_0 = 1             # [m] tool position
L = 40 * (2.54/100) # [m] distance between source and receiver
azim = 0            # [°] horizontal deviation from x-axis, anti-clockwise
dip = 60            # [°] vertical deviation from xy-plane down/up-wards

# Spatial parameters
p_src = [0, 0, z_0,     azim, dip]
p_rec = [0, 0, z_0 - L, azim, dip]

inpdat = {'src': p_src,
          'rec': p_rec,
          'depth': depth,
          'freqtime': freq,
          'aniso': aniso,
          'ab': ab,
          'htarg': {
              'pts_per_dec': -1
              },
          'verb': verb}

fEM = empymod.dipole(**inpdat, res=res)
print(f'   {comp} (real, imag) {fEM}')

Proposed fix:
It seems like check_dipole(...) only expects (x,y,z). This function could be extended to also deal with (x,y,z,azim,dip). Whether or not this is the full scope of the issue is unclear to me at this point.

Hi @TobiasGlaubach .
dipole only expects (x, y, z) and the parameter ab which defines the source configuration. dipole only works for point dipoles, aligned with the major axis. What you want is bipole, not dipole.
Here the two descriptions:

dipole: https://empymod.readthedocs.io/en/stable/modeller.html#empymod.model.dipole
bipole: https://empymod.readthedocs.io/en/stable/modeller.html#empymod.model.bipole

I just saw now that you actually combined two things in your inpdat, azm/dip with ab. The former is for bipole, the latter for dipole.
Have a look at the docs, and please let me know if there is something that is not clear. E.g., if the distinction between dipole and bipole is not clear (the naming of them is probably a bit unfortunate to start with, but too late to change that I fear...). I am pretty sure the docs could be improved, so I am happy about any constructive feedback!

Just opened #101.
Your use-case works as expected. You provide a wrong src/rec-definition for dipole, it fails with a warning that source and receiver are wrongly defined.
However, if you change in your example empymod.dipole to empymod.bipole it works without any warnings etc - but, it should warn that there is an unused keyword ab provided. This has to be improved.
Thanks again!

That is actually already implemented. If you use empymod.bipole, and keep verb=1 (or anything > 0) then it would actually notify you
* WARNING :: Unused **kwargs: {'ab': 66}.
* WARNING :: Offsets < 0.001 m are set to 0.001 m!

So it works as expected for both, empymod.dipole (fails with a note on wrong src/rec definition) and empymod.bipole (warns about unused keyword).


Split utils/prepare to make TextModels only usage at least works without Node.js polyfills

This PR splits utils/prepare.ts into prepare-text.ts and prepare-image.ts.
Current problem
When using only TextModel,
import { TextModel } from "@visheratin/web-ai";

const result = await TextModel.create("grammar-t5-efficient-tiny", false)
console.log(result.elapsed)

ends up with a runtime error for example in Vite app like this.
path-arg.ts:1 Uncaught ReferenceError: process is not defined

This happens because when you import "@visheratin/web-ai";, exported modules other than TextModel that depend on jimp are also preprocessed, and fail without Node.js polyfills.
There are several ways to solve this issue, but the simplest way I came up with was to import "@visheratin/web-ai/dist/text"; directly.
But even this attempt utils/prepare.ts still depends on jimp and fails.
So with this PR, at least support to use only TextModels without any effort like seting up Node.js polyfills(which are completely useless at runtime).

UPDATE: I only use Vite these days, but I noticed that this happens no matter what bundler is used. The commit message is a remnant of this. 😓

Yes. 😃
In terms of usage, @visheratin/web-ai/text looks more natural(than /dist/text) if it's possible.
In terms of implementation, keep jimp isolated, at least until browser support becomes more better is good idea for better accessibility.

Thank you, too!


Added jsonb next to existing json check

This allows for jsonb fields to be returned as json_schema object.
Sidenote:
Possibly the value here could be an array.
There is no check for this in the existing code.

Could you add a test as well as add the DCO metadata? Thx

Are there any updates for this? Would love to ship it!

ping @fetishcoder

I'm going to close this due to inactivity. If you want to pursue this, please reopen.


Think about add limit parameter to your API points

Think about adding limit parameter to your API points or maybe I cant find how to add limit?

@EntroSolutions
Hi are you referring to the rate limit?
Please check the routes file
Route::group([
	'prefix' => '{prefix?}',
	'middleware' => [
		'throttle:60,1',
		Localization::class,
	],
], function () {

closed by lack of activity


[FEATURE REQUEST] Min/max values for numeric napari widgets should be specified

For QSpinBox and QDoubleSpinBox the minimum to maximum range is 0-99 and 0-99.99 respectively. We should allow acceptable ranges (or at least sensible ranges).
This has been done already for some as seen here https://github.com/quantumjot/btrack/blob/8e4da110c9662d7993f58753c1d7e7dd22e09a3a/btrack/napari/widgets/_motion.py#L20
It is not sufficient to only do it in pydantic. The widget would still allow you to go beyond that.

Some default ranges:
Max search radius 0-1000
Max lost 0-10
P(not track) 0-1
Accuracy 0.1-10
P 0-500
Q 0-500
R 0-500

There are others, but I'll close this once #376 is merged


BFUS 9.0.19 snapshot no more working after changing default settings

I just receive 2 new Yi Home BFUS 9.0.19.
Operations done were:

configure Yi Home app to find the firmware version. Testing Video : ok
donwload firmware y21ga_0.1.3.tgz, and uncompress it on a 32GB fresh FAT32 SD
boot, flash, it's a success !! snapshot works
enable "disable cloud", enable "recording without cloud", disable all RTSP and all onvif features, disable FTP and telnet
saving (not rebooting)
disable "switch on/off camera" and save
configure FTP upload (server, path, user/pass) and save
rebooting using "maintenance".
snapshot return a 0 file size

using ssh:
root@yi-4a58:~# cd /tmp/sd/yi-hack/bin
root@yi-4a58:/tmp/sd/yi-hack/bin# ./imggrabber
Starting program
Could not open file /tmp/iframe.idx
root@yi-4a58:/tmp/sd/yi-hack/bin# ls -l /tmp
drwxr-xr-x    2 root     root            40 Jan  7 18:41 app
lrwxrwxrwx    1 root     root            23 Jan  7 18:41 audio -> /home/app/audio_file/us
-rw-r--r--    1 root     root             2 Jan  7 18:43 cloud_init_finish
-rw-r--r--    1 root     root            21 Jan  7 18:41 dns1
-rw-r--r--    1 root     root             2 Jan  7 18:41 done
drwxr-xr-x    2 root     root            40 Jan  7 18:41 face_detect
-rw-r--r--    1 root     root          2886 Jan  7 19:07 ftppush.log
-rw-r--r--    1 root     root            12 Jan  7 18:41 gw1
-rw-r--r--    1 root     root             0 Jan  7 18:41 httpd.conf
drwxr-xr-x    2 root     root            40 Jan  7 18:41 ko
drwxr-xr-x    2 root     root            40 Jan  7 18:41 log
-rw-r--r--    1 root     root            14 Jan  7 18:41 mask
-rwxr-xr-x    1 root     root          2644 Jan  7 18:41 mmap.info
-rw-r--r--    1 root     root           116 Jan  7 18:41 resolv.conf
drwxr-xr-x    2 root     root            40 Jan  7 18:41 run
drwxr-xr-x    8 root     root         32768 Jan  7 18:43 sd
drwxr-xr-x    2 root     root            40 Jan  7 18:41 tools
drwxr-xr-x    3 root     root            60 Jan  7 18:41 var
-rwxrwxrwx    1 root     root           191 Jan  7 18:41 wpa_supplicant.conf

I flash the 2nd camera, and copy the /tmp/iframe.idx to the first but it fail with another error:
root@yi-4a58:/tmp/sd/yi-hack/bin# ./imggrabber
Starting program
Mapping file /dev/shm/fshare_frame_buf, size 1786156, to b6cce000
Encoding h264 frame
Starting decode
Decode frame
[h264 @ 0x13c8b50] No start code is found.
[h264 @ 0x13c8b50] Error splitting the input into NAL units.
Error decoding frame
Error encoding h264 frame

Then I reset to the default config, reboot, but no change.
I tune the second camera, and after the reboot both camera can't see snaphot (so it's not an issue with an SD card for example)
I reformat SD card from first camera, putting the same firmware, rebooting and SNAPSHOT is OK !!
So I guess there is a hidden dependency in the configuration.
I can help you to do more test if you want
Regards

Why do you switch off the cam?
I mean "switch on/off camera" checkbox.

Why do you switch off the cam?
I mean "switch on/off camera" checkbox.

Hi, In fact I just want to do some "explicit" snapshot from domoticz (just check that light is off for example). I don't need to detect motion or record anything. I enable automatic detection only when I'm on holiday.
From what I wrote yesterday, I'm not so sure because today I format both SD, reapply the same firmware, but snapshot continue to fail on both camera (snapshot, and RTSP video are dead, but camera config is very responsive with a goog wifi quality)

Hi, In fact I just want to do some "explicit" snapshot from domoticz (just check that light is off for example). I don't need to detect motion or record anything. I enable automatic detection only when I'm on holiday.
From what I wrote yesterday, I'm not so sure because today I format both SD, reapply the same firmware, but snapshot continue to fail on both camera (snapshot, and RTSP video are dead, but camera config is very responsive with a goog wifi quality)

About this error who generate or copy this missing file ?
root@(none):/tmp/sd/yi-hack/bin# ./imggrabber
Starting program
Could not open file /tmp/iframe.idx

About this error who generate or copy this missing file ?
root@(none):/tmp/sd/yi-hack/bin# ./imggrabber
Starting program
Could not open file /tmp/iframe.idx

After reinstalling and connecting using Yi app, camera appears off. So after put it online, I can again so some snapshot.
It's probably obvious to not be able to take snapshot if camera is off, but it's not the behaviour of the y25: camera off means no surveillance , but snapshot is possible. With the y21ga, if camera is off, surveillance and snapshot are disabled.
So I close the ticket

After reinstalling and connecting using Yi app, camera appears off. So after put it online, I can again so some snapshot.
It's probably obvious to not be able to take snapshot if camera is off, but it's not the behaviour of the y25: camera off means no surveillance , but snapshot is possible. With the y21ga, if camera is off, surveillance and snapshot are disabled.
So I close the ticket

Now I understand.
In the y25 cam, the stream is picked up directly from the kernel driver.
If you switch off the cam, the stream continues to run.
In the allwineer based cams, the stream is read from the fshare_frame_buffer file (filled by rmm process).
If you switch off the cam, probably the rmm process doesn't fill the file.

Now I understand.
In the y25 cam, the stream is picked up directly from the kernel driver.
If you switch off the cam, the stream continues to run.
In the allwineer based cams, the stream is read from the fshare_frame_buffer file (filled by rmm process).
If you switch off the cam, probably the rmm process doesn't fill the file.


Add PEP 570 support

This addresses https://github.com/tree-sitter/tree-sitter-python/issues/78.

Merged as #146


when will your release this function?

"Based on this analysis, we will subsequently publish a package that will allow you to analyse the Pauling rules for your own structures"...

We want to try your pauling rules test on our own cif files?
what is the easiest way to call your functions?

Also interested


Implement lane assignment for the bowling area

Multiple instances of bowling will share the same alley in-game. We should have a system that assigns lanes. This should be easy, but this will ensure our edge cases are covered.

Assign one lane to each instance
If an instance is disbanded (all players in the instance leave), the lane is freed up
If all lanes are full, a bowling instance cannot be created (go to another server, or figure out a better solution?)
Ensure lanes are properly reset
Ensure players are spawned in the correct area, and not blocking other players' cameras.

This is blocked by #4

Ideally we should just set these up as different gamemode instances

yeah each lane should be a gamemode instance with up to 6 players


Exiting with 0 after tf state push

A reference to the issue / Description of it
This is to implement further fix for the https://github.com/ministryofjustice/modernisation-platform/issues/5859 by letting the code exit with 0 after the errored tf state is pushed to the state file.
This means, that when the tf fails to save the state, it will try (once) to push the errored state to the state file, then it will exit with 0 regardless of whether the push command itself has failed or not.
The echo commands has also been added for easier debugging.
How does this PR fix the problem?
Evidence of a successful tf state push is here: https://github.com/ministryofjustice/modernisation-platform/actions/runs/7434337243/job/20229028807
Note, the pipeline files (exit 1) due to the original error. This change is to let it exit with 0, after the tf state push is run.
How has this been tested?
Run locally to test the correctness of the code and see the above comment for an example of a successful tf state push.
Deployment Plan / Instructions
Will this deployment impact the platform and / or services on it?
This is currently rolled out only to the scheduled baseline pipeline.
Checklist (check x in [ ] of list items)

[x] I have performed a self-review of my own code
[x] All checks have passed
[ ] I have made corresponding changes to the documentation
[ ] Plan and discussed how it should be deployed to PROD (If needed)

Additional comments (if any)
{Please write here}

This is not a good approach and the code exits before it gets to the state push, closing this one and going to follow on with further PRs.


[Bug]: Anonymous blocks can not be redebugged

ODC version
4.2.1
OB version
4.1.0
What happened?
while anonymous blocks redebugged，

Error：BadRequest exception, type=NullPointerException, message=AnonymousBlock can not be null for StartPLDebugReq
response info：
{
    "code": "BadRequest",
    "durationMillis": 27,
    "error": {
        "code": "BadRequest",
        "details": [
            {
                "code": "BadRequest",
                "message": "BadRequest exception, type=NullPointerException, message=AnonymousBlock can not be null for StartPLDebugReq",
                "target": null
            },
            {
                "code": "NullPointerException",
                "message": "AnonymousBlock can not be null for StartPLDebugReq",
                "target": null
            }
        ],
        "message": "错误的请求，详情：BadRequest exception, type=NullPointerException, message=AnonymousBlock can not be null for StartPLDebugReq"
    },
    "httpStatus": "BAD_REQUEST",
    "message": "错误的请求，详情：BadRequest exception, type=NullPointerException, message=AnonymousBlock can not be null for StartPLDebugReq",
    "requestId": "ITJPP3PSTE8HWEULR68S4V",
    "server": "sqaappecsv62s2011162217057.sa128",
    "successful": false,
    "timestamp":<PHONE_NUMBER>.954000000,
    "traceId": "2ff5a9d4ca1b4304"
}

What did you expect to happen?
re-enter the pl debug state
How can we reproduce it (as minimally and precisely as possible)?
Anything else we need to know?
Cloud

The parameter is missing anonymousBlock field, so the issue may should be fixed by frontend.

Fixed In Commit cbdee53be86e7702447bb38c18ae3066308ea4c4
------
晓康
Wed Sep 6 15:46:29 2023 +0800

PASS


Cherry-pick to earlgrey_es_sival: [sival, rv_core_ibex] test rv_core_ibex_isa_test in prod

Cherry-pick of #22876. This also include @HU90m's testplan changes which were not backported.

CI failures (aon_timer_wdog_lc_escalate_test_fpga_cw310_sival_rom_ext and pwrmgr_normal_sleep_all_reset_reqs_test_fpga_cw310_sival_rom_ext) are know to be flakey and unrelated to this change.


utils.get_input silently removed

Checklist

[x] Have you updated the plugin to the latest version on main branch?
[x] Have you checked the Breaking Changes issue?
[x] Have you read through :h nvim-surround to see if there might be any relevant information there?

Describe the bug
utils.get_input got removed without anything being posted to breaking changes or the docs being updated

@0xAdk I've added it to both the Breaking Changes issue, as well as updated the documentation to the latest version. Thanks for pointing this out!

feel free to re-open this issue if there's anything else I missed!

no clue how I would do that, but you did miss two places :p
https://github.com/kylechui/nvim-surround/blob/9e5c9ec48d0c44237d7bdcf7a030355524d56dba/doc/nvim-surround.txt#L267
https://github.com/kylechui/nvim-surround/blob/9e5c9ec48d0c44237d7bdcf7a030355524d56dba/doc/nvim-surround.txt#L282
btw while I have your attention would the plugin choosing to use vim.fn.input limit customization / using custom ui(s)?
I was under the impression that utils.get_input was used so it could be replaced with a different interface if the user wanted

Gah! Just fixed those two as well. Originally the plan was to have it so that vim.ui.input() could be used instead to allow for different interfaces, but it's an async function which causes all sorts of back-end problems. I really did try to use it, see #24, but was unsuccessful. I then later removed utils.get_input() because I was rearranging code and I wanted utils to be able to require config, but you can't have both files requiring each other simultaneously.
Also about re-opening issues, perhaps that's something that's only available on my end? Unsure :P

I think reintroducing the utils.get_input() function that users can call and optionally overwrite seems quite useful (see also the discussion in #89). About the circular dependency issue: can't you just pass the config table to the function in utils?

@smjonas I agree that it would be useful, but at the moment the code has:

utils requires config to access the buffer-local options (via config.get_opts())
config requires utils to access user input (via utils.get_input())

Could you be a bit more specific as how to refactor the code to circumvent this? The other option that could work is to have config.get_input() instead of utils.get_input(), but that seems sub-ideal IMO

Could you be a bit more specific as how to refactor the code to circumvent this? The other option that could work is to have config.get_input() instead of utils.get_input(), but that seems sub-ideal IMO

I meant that you could simply pass opts as an additional argument to functions like get_alias() which would make the dependency on the config module redundant.

I see; IMO it makes the most sense to have all functions refer to config in order to get information about how the user has configured nvim-surround, and as you said earlier, leave issues of error handling to the user. I think theoretically input could be queried in other ways, e.g. functional surrounds only care that they receive their table in a synchronous matter; it just so happens to be that vim.fn.input() is the simplest way to do so.

@smjonas Now that I think about it more, I don't think I'm that opposed to having a config.get_input(), but I think I need a strong(er) reason/some justification to proceed this way, to avoid pushing more Breaking Changes.

I actually think utils.get_input() would make more sense. This is how I imagined it:

config requires utils
utils should not require config

Instead, change:
M.get_delimiters = function(char, args)

to
M.get_delimiters = function(delimiters, char, args)

and call it from init.lua like this: local delimiters = utils.get_delimiters(config.get_opts().delimiters, ins_char, delim_args). Would that work for you?
Also regarding breaking changes: maybe you could support semantic versioning for your plugin? This way, users can pin their plugin version to a specific major version like version 1 if they want to. This allows them to deal with breaking changes in their plugins when they choose to instead of when they update their plugins. For each breaking change, you would simply increase the major version. I know there is kind of this hesitation to increase major versions quickly but that's how semver works ;)
Packer supports this via tags:
use {
  "kylechui/nvim-surround",
  tag = "1.*",
}

That's fine, I just thought this might be a way to circumvent the circular dependency issue. There are definitely other solutions as well :)

@smjonas I think I'm going to end up creating config.get_input(), along with other functions like config.get_selection() to simplify the user configuration process. TL;DR allowing users to use functions to determine what to delete, etc. is a bit of a pain, so I think exposing a configuration API is a better idea.


Extend field validators

Posibility to extend field custom validators on schema instance.

Coverage remained the same at 100.0% when pulling 59d7bc60b1670eaa16be829237b7384881f05fa9 on extendFieldValidators into 6032d15384e9ccc3801cf0fee9ae5aeb01789306 on master.


Bug 2083055: Update cluster name validation

Signed-off-by: Rastislav Wagner<EMAIL_ADDRESS>

@rawagner 2.4 release is "hibernated" and probably won't see any updates. Can we close this?


Update fundamentals-bio, ... to 1.0.2

Updates

io.7mind.izumi:fundamentals-bio
io.7mind.izumi:fundamentals-collections
io.7mind.izumi:fundamentals-functional
io.7mind.izumi:fundamentals-json-circe
io.7mind.izumi:fundamentals-platform
io.7mind.izumi:fundamentals-reflection
io.7mind.izumi:logstage-adapter-slf4j
io.7mind.izumi:logstage-core
io.7mind.izumi:sbt-izumi-deps

from 1.0.1 to 1.0.2.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.0.1).
You might want to review and update them manually.
idealingua-v1/idealingua-v1-core/src/main/scala/izumi/idealingua/model/publishing/manifests/TypeScriptBuildManifest.scala
idealingua-v1/idealingua-v1-runtime-rpc-typescript/src/npmjs/package.json

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.7mind.izumi" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #174.

Superseded by #174.


Fitting 'total water' profile

I have split my profiles into different peaks. When I want the total water, I add the areas from the different peaks. However, I can't fit this total water with your fitD function since it is not technically a 'profile'.
Is there a way to turn my list of total water contents into a profile so that I can fit it?

Try
profile.areas = your summed area list
profile.plot_area_list()

Actually, no, that won't work because it's going to re-make the areas every time. Let me fiddle with it.

pynams updated - you should be able to set the profile.areas directly and then it'll use those in the plotting command.

Great, thanks!
On Thu, Jan 26, 2017 at 10:52 AM, Elizabeth Ferriss <
<EMAIL_ADDRESS>wrote:

Actually, no, that won't work because it's going to re-make the areas
every time. Let me fiddle with it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EFerriss/Pynams/issues/10#issuecomment-275424652, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AU8EhBORkGfp8OijlmyCQR3E0VF-XhxKks5rWMFXgaJpZM4LuFkg
.

Let me know if it works, and if it does, you can close the issue.

Oh, it's not profile.plot_areas_list(), it's the usual plot command

It worked! Thanks!

It just occurred to me to add a caution: Because profile.plot_area_profile no longer automatically updates the profile.areas based on the current baselines, if you plot an area profile, change baselines, and then want to plot the new areas for the new baselines, you must first update the profile.areas by running profile.make_area_list()

I updated profile.get_baselines() to automatically run profile.make_area_list() so this problem doesn't come up and the areas are automatically updated.


_onAnnouncement crashes bot

For now, fix this. After that, add more error handling.
/home/chatbots/node_modules/steam-chat-bot/lib/chatBot.js:141
 function(groupID, headline) { that._onAnnouncement(that.maingroup, headLine);
                                                                    ^
ReferenceError: headLine is not defined
    at SteamClient.<anonymous> (/home/chatbots/node_modules/steam-chat-bot/lib/chatBot.js:141:108)
    at SteamClient.emit (events.js:110:17)
    at SteamClient.handlers.(anonymous function) (/home/chatbots/node_modules/steam-chat-bot/node_modules/steam/lib/handlers/friends.js:319:10)
    at SteamClient._netMsgReceived (/home/chatbots/node_modules/steam-chat-bot/node_modules/steam/lib/steam_client.js:106:26)
    at SteamClient.handlers.(anonymous function) (/home/chatbots/node_modules/steam-chat-bot/node_modules/steam/lib/steam_client.js:192:10)
    at SteamClient._netMsgReceived (/home/chatbots/node_modules/steam-chat-bot/node_modules/steam/lib/steam_client.js:106:26)
    at Connection.emit (events.js:107:17)
    at Connection._readPacket (/home/chatbots/node_modules/steam-chat-bot/node_modules/steam/lib/connection.js:50:8)
    at Connection.emit (events.js:104:17)
    at emitReadable_ (_stream_readable.js:424:10)

Fixed
https://github.com/Efreak/node-steam-chat-bot/commit/7c5332f3b43ce44f86921acbb0383afde808b59b


Public const should be in PascalCase, not camelCase

As evidenced by searching for public const in AspNetCore, the documentation incorrectly states that public const should be in camelCase.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 00bb2c0f-ca81-ae02-2dfa-d9ef8a6a9a97
Version Independent ID: de79bd37-59d4-cb32-f7d7-13c999d7ff68
Content: const keyword - C# Reference
Content Source: docs/csharp/language-reference/keywords/const.md
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Actually, only AspNetCore might not constitute a guideline, so this is a search for public const in DotNetCore as well.

Thanks for pointing this out @mycroes. Would you like to create a PR to address this?
The instructions are here: https://docs.microsoft.com/contribute/#quick-edits-to-existing-documents

Thanks for writing this up @mycroes
I've added it to our backlog, and the project for community contributions.
If you want to create the PR, as @NextTurn says, I'll help answer any questions you may have.


New version: Mp3tag.Mp3tag version 3.22

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This PR is broken. 3.22 has been pulled from the server and redirects to the download page. The current version is 3.22b released September 20.


Add lualine theme

Great! Thanks :) do you think this should go in autoload?

I don't think so. Lualine will search for the theme in the lua/lualine/themes/ directory of any installed plugin.

Ok! I don't use it so I didn't know


test-cases, travis: Add support for TAP reports

This commit adds test-cases support for the generation of TAP reports. It also takes
the oppurtunity to rationalize the options for specifying the output format.  There
is now one format option, --format, which accepts a number of different values;
html, tap, colour-text and text.  The default is colour-text.
This change requires a modification to the travis script that executes test-cases,
as the old formatting option used by travis.yml no longer exists.
Signed-off-by: Mark Ryan<EMAIL_ADDRESS>

Coverage decreased (-0.04%) to 66.523% when pulling afad4671944c3a03f87fabd4bef00f06f9f67903 on markdryan:master into 74c1e8d41bc4fceefa5b785513691a67d6544a54 on 01org:master.

Coverage decreased (-0.02%) to 66.545% when pulling a8efaf8a8fc94be8182890fa8ac595b1425a20d2 on markdryan:master into 74c1e8d41bc4fceefa5b785513691a67d6544a54 on 01org:master.

Coverage decreased (-0.08%) to 66.483% when pulling a8efaf8a8fc94be8182890fa8ac595b1425a20d2 on markdryan:master into 74c1e8d41bc4fceefa5b785513691a67d6544a54 on 01org:master.


Bots not buying weapons on fy_* maps

Describe the bug
As the title says: bots not buying weapons on fy_* maps.
Expected behavior
Bots should buy weapons on fy_ maps as on other normal maps if there is any buyzone they can use.
Additional context

If map have not spawned weapons, bots will just fight with pistols.
If you rename map by adding "aim_" prefix, as example, then bots will buy weapons.
I've uploaded map that I've tested: https://disk.yandex.ru/d/KfsGgVAiLEY9Ow. I also made some test graph file.
NOTE: it is for czero mod, if someone gonna test it in cstrike then you shoud download some czero wads. Game will request 'em

Video
Can't record a video, but made 2 demos: on fy_snowcave_cz and renamed variant (https://disk.yandex.ru/d/J7znCc-6tmx66g)
Environment details:

OS: Windows 10
Engine: vanilla (8684)
GameDLL: ReGameDLL_CS <IP_ADDRESS>6-dev
Metamod: 1.21p38
Bot Version: v4.2.637
Game mode: ?

AMX Mod X plugins:
 [  1] Admin Base              <IP_ADDRESS>94  AMXX Dev Team     pre-inst/admin.  running
 [  2] Admin Commands          <IP_ADDRESS>94  AMXX Dev Team     pre-inst/adminc  running
 [  3] Admin Help              <IP_ADDRESS>94  AMXX Dev Team     pre-inst/adminh  running
 [  4] Multi-Lingual System    <IP_ADDRESS>94  AMXX Dev Team     pre-inst/multil  running
 [  5] Menus Front-End         <IP_ADDRESS>94  AMXX Dev Team     pre-inst/menufr  running
 [  6] Players Menu            <IP_ADDRESS>94  AMXX Dev Team     pre-inst/plmenu  running
 [  7] Teleport Menu           <IP_ADDRESS>94  AMXX Dev Team     pre-inst/teleme  running
 [  8] Maps Menu               <IP_ADDRESS>94  AMXX Dev Team     pre-inst/mapsme  running
 [  9] Admin Chat              <IP_ADDRESS>94  AMXX Dev Team     pre-inst/adminc  running
 [ 10] Anti Flood              <IP_ADDRESS>94  AMXX Dev Team     pre-inst/antifl  running
 [ 11] Admin Votes             <IP_ADDRESS>94  AMXX Dev Team     pre-inst/adminv  running
 [ 12] NextMap                 <IP_ADDRESS>94  AMXX Dev Team     pre-inst/nextma  running
 [ 13] Nextmap Chooser         <IP_ADDRESS>94  AMXX Dev Team     pre-inst/mapcho  running
 [ 14] Pause Plugins           <IP_ADDRESS>94  AMXX Dev Team     pre-inst/pausec  running
 [ 15] Stats Configuration     <IP_ADDRESS>94  AMXX Dev Team     pre-inst/statsc  running
 [ 16] CS Stats Logging        <IP_ADDRESS>94  AMXX Dev Team     pre-inst/stats_  running
 [ 17] StatsX                  <IP_ADDRESS>94  AMXX Dev Team     pre-inst/statsx  running
 17 plugins, 17 running

Metamod plugins:
 [ 1] AMX Mod X        RUN   -    amxmodx_mm.dll    v<IP_ADDRESS>  ini   Start ANY
 [ 2] YaPB             RUN   -    yapb.dll          v4.2.637  ini   Chlvl ANY
 [ 3] FakeMeta         RUN   -    fakemeta_amxx.dl  v<IP_ADDRESS>  pl1   ANY   ANY
 [ 4] CSX              RUN   -    csx_amxx.dll      v<IP_ADDRESS>  pl1   ANY   ANY
 [ 5] Ham Sandwich     RUN   -    hamsandwich_amxx  v<IP_ADDRESS>  pl1   ANY   ANY
 [ 6] CStrike          RUN   -    cstrike_amxx.dll  v<IP_ADDRESS>  pl1   ANY   ANY
 [ 7] Engine           RUN   -    engine_amxx.dll   v<IP_ADDRESS>  pl1   ANY   ANY
 [ 8] Fun              RUN   -    fun_amxx.dll      v<IP_ADDRESS>  pl1   ANY   ANY
8 plugins, 8 running

If map have no func_buyzone entities then usually you can always buy within your spawn point.
If it is not prohibited by info_map_parameters entity

thank you


Points returning [ 0, 0, 0, 0]

Hi,
I was testing the script and I got the problem, that no matter where I would select the image to crop the get function would return the correct zoom, but the points would return only an array with zeros.
I tested on chrome, firefox both jQuery plugin and Vanilla.
Got it from the npm.

If I run this:
$('.demo-cropper').croppie('get')

on https://foliotek.github.io/Croppie, I get a correct list of points.
Can you be more detailed with your submission?  Have some code you can share?

I set the croppie on one function like this
croppie = $('#croppie_img').croppie({
                        url: image,
                        viewport: {
                            width: 150,
                            height: 150,
                            type: 'circle'
                        }
                    });

And I try to get the info on other function like this:
console.log($('#croppie_img').croppie('get').points)

croppie is a global variable, I tried using from this variable and setting it with a different name also.

By the way, I'm using an image that comes from the FileReader().
I guess this shouldn't be a problem, since I tried to console positions on the _get() function and it works just fine until I call por the $...croppie('get').

Not quite sure what you mean - is there anyway you can share a codepen/jsfiddle/jsbin that demonstrates the issue?

I'd don't think I'd be able to reproduce everything on the codepen, I sent the page to a server though.
The function that I use goes through some other functions...
You can check it happening on (http://ec2-52-67-147-248.sa-east-1.compute.amazonaws.com/testing/sd). You will need to login, just click on "Entrar" on the right top corner: username<EMAIL_ADDRESS>pasword: 123456
Then click on the profile picture to change it.

When I upload an and do $('#croppie_img').croppie('get') - I get points that aren't 0's.
Maybe it's something weird with the image you're using?

Ohhh, I just realized what is happening.
Seems that I was closing the modal window before collecting these informations...
Thanks for you help @thedustinsmith !


FECK-1905: Get the Scala apps on Swig

Accept an output directory in swig install

Are we missing a package.json version bump or does Lerna take care of that these days?

@RoryH yes Lerna will take care of that.

Lerna does it @RoryH


Adds a quickstart sample to cloud-kms.

To be referenced in
https://cloud.google.com/kms/docs/reference/libraries

Nice, thanks!

thanks!


Explore Scroll down navigation options to ease alt nav users

📆 Study info

Accessibility testing - July 2023 - Facilitator: @Lu-stack-art & @ClementineHahn
Name of the test :
Triage Board : link to the associated trello board

🚦 Issue severity : Low / Moderate / Severe ?
📝 What happened?
Describe what the participant encountered as a problem, barrier or event.
📗 User story
As a user, I need  to ....., so that....
📜 Recommendations
Researcher informed recommendation on possible actions to be taken
ℹ More info
Add here any relevant links or context that could help find a solution

After watching the video and doing some tests with voice control, it seems the user's AT was malfunctioning. I don't know what more we could explore or do to help when that type of situation happens.

Compatibility testing on fable, for GC DS webpage navigation. User is tasked to find "Radio Component"
Assistive tech is Screen Magnification
User found it difficult to tell where my focus is: multiple hover text boxes
While using hover text in the same boxes, in this case the 'Radio component' box, there are 3 different hover text boxes, and it is not only confusing but it is tough to use the hover text on mouseover. The title, in this case is a link, as illustrated by my bright pink text against the dark blue background, and then there are 2 sections, one starting with the words "Other names..." and then a section below it "A short set..." While magnified, and seeing only sections of this, and the hover text scroll/starting at the end all make this much harder than it needs to be.                                                                                Suggestions: If the title is the link, then make the other descriptions static, and all programmatically determinable using the OS and AT
Dev Notes- swap to card components which will resolve this issue

Compatibility testing on Fable, for GC DS webpage navigation
Assistive tech being Screen Magnification
https://trello.com/c/KZOaYEHI
User found it difficult to read text due to colour contrast:
Even while very magnified, the hover text appears very light white/cream as background, which makes it challenging to see. Consider what text looks like while inverted. The text and background are too close in colour range (white on cream) diluting any clear contrast between text and background.
Dev notes-work in progress, they will be replacing the overview component preview boxes with existing card component, and the Designers agree

Compatibility testing on Fable, for the GC DS webpage navigation. user is tasked to copy the code
Assistive tech is Screen Magnification
https://trello.com/c/m28QuNhx
Buttons and links: Other-does copy button work/display more clearly                                                                                Once clicking the “copy” button, there was no indication of that being copied onto the clipboard. Participant clicked 3 times to be sure…process unclear/no indicator os task being successful/complete.
possible suggestion (I think) : Some kind of popup that would be in focus, and remain visible long enough for me to make the magnification setting changes I needed to make. ideal to have indicator that code was copied correctly.
Dev Notes-button does say ‘copied’ versus copy. Devs will make button bigger to show better visibility
Designers notes-option could be to add lighter border to also show better visibility, or change the fill to a lighter colour...however, the easier fix is to place border around, (same colour as the text)


EC2 Instance Type Validation

Steps To Reproduce

Launch an EC2 instance with an invalid instance type such as t3.nano.
client.run_instances(InstanceType="t3.nano", ImageId="ami-00000000", MaxCount=1, MinCount=1, )

Run Describe Instances and see that the instance is launched with an invalid instance type.
{'Reservations': [{'Groups': [], 'Instances': [{'AmiLaunchIndex': 0, 'ImageId': 'ami-00000000', 'InstanceId': 'i-b4955ca3971326510', 'InstanceType': 't3.nano', ..............., 'StateReason': {'Code': '', 'Message': ''}, 'Tags': [], 'VirtualizationType': 'paravirtual'}], 'OwnerId': '123456789012', 'ReservationId': 'r-3e442b18'}], 'ResponseMetadata': {'RequestId': 'fdcdcab1-ae5c-489e-9c33-4637c5dda355', 'HTTPStatusCode': 200, 'HTTPHeaders': {'Content-Type': 'text/plain', 'server': 'amazon.com'}, 'RetryAttempts': 0}}

Referring to  'InstanceType': 't3.nano'

An InvalidParameterValueError should occur.

botocore.exceptions.ClientError: An error occurred (InvalidParameterValue) when calling the RunInstances operation: Invalid value 't3.nano' for InstanceType.

Only reproduced on python mocks so far, but could likely be reproduced on standalone server

fix for #1696

Thanks for opening; setting this as a feature request for instance type validation.

@spulec Thanks! I have opened PR on this here #1696


Linux Support ?

The script requires pywin32 which is a windows-only package, is there any alternative for Linux ?

I updated the script to include Linux support must be run as root

I updated the script to include Linux support must be run as root

Is it the only possible solution ? No way to run it in user mode ?

There is nota way to block the key switches at the user level that I know of.

There is not a way to block the key switches at the user level that I know of.

i have no idea, that's why i'm asking x)

I created a scrip using the X11 libraries to run this script without root privileges.


'Plonk sound' on keyboard input in macOS 10.12 Sierra Beta

I just noticed when turning up the sound on OSX 10.12, a 'key plonk' sound is played when a key is pressed in a minifb window (as if the the window isn't accepting keyboard input). It's the same sound that plays when you look at a text file with 'less', and try to scroll past the end when hitting the Space key, or when you go to "About this Mac" in the Apple menu, and try Cut/Copy/Paste in the about-window.
This reproduces in the minifb examples (e.g. the noise example), just run the sample and start pressing keys.
I don't know whether this also happens on older OSX versions. The complete version number of the 10.12 Beta is "10.12 Beta (16A254g)".

Found the issue. I will fix some more that I have pilled up and release a new version tonight/tomorrow :)

Thanks a lot!

Lets keep this one open until I submitted a fix :)

Alright. Try 0.8.3 and it should be fixed

Confirmed working. Thanks :)


Removed test from setup.py + Replaced all direct setup.py

This should resolved #403 and is a first step towards #400

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
1721

Relevant Lines:
1813

💛  - Coveralls


mailNickName field in Azure RBAC users and groups permissions report returns random ID

Describe the bug
When viewing the Azure RBAC users and groups permissions report. Some users report back with the correct mailNickName but some groups which appear to include spaces in the names report as a random ID instead of the readable names.
To Reproduce
Steps to reproduce the behavior:

Go to 'CCO PowerBi DashBoard'
Click on 'Azure RBAC Users and Groups Permissions Report'
Scroll down in the report to view the mailNickName column for human readable names instead of random ID's.

Expected behavior
The display name / name should be visible for for all users and groups that are assigned to the resources RBAC.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
PowerBI Version 2.98.882.0 64Bit

The API is returning the data in that way for some type of users:

You can add these fields from the RBAC Role Users and groups to have more details:


Update Cargo.toml

Add categories to -sys crate
Remove redundant keys (key = default-value)
Exclude .github dir from packaging

Do we not need the build script line anymore? Huh, if that's true then that's kind of nice.

Nope, build.rs is the default:

The default is "build.rs", which loads the script from a file named build.rs in the root of the package. Use build = "custom_build_name.rs" to specify a path to a different file or build = false to disable automatic detection of the build script.


starting new data structures

for this part:
len(the_list) returns the size of the list
print(the_list) returns what the display() method returns
do i need to adjust repr and len in the node function?

for this part:
len(the_list) returns the size of the list
print(the_list) returns what the display() method returns
do i need to adjust repr and len in the node function?

I don't see a function called node, could you restate the question?

Also update the readme!


Version updates

Updated spring-data-cosmosdb version to 2.2.0
Removed reactor netty override as it is not needed anymore with latest spring-framework and spring-data-commons versions.

Moved to PR in private repo.


Invalid commands typed in normal mode will be inserted when entering insert mode

In insert mode, if you type a character that isn't recognised as a normal mode command, then the next time insert mode is entered that character will be input.
Steps to reproduce
With the buffer
hello world

and cursor at h, in normal mode: type the sequence ]wi. You will end up in insert mode with the cursor before w:
hello ]world

It seems there can be one or more motion commands between typing the invalid command and entering insert mode. Or, eg. with the same starting state as above, type <space>a and end up with:
h ello world

Versions
Version 0.5.1-nightly (0.5.1567)
macOS 10.15.4

I can not reproduce this on arch linux latest master build

@kinglouie it could be that it's fixed on latest master? There hasn't been a nightly build in a couple of days. I will check it as soon as there's a new one.

Strange - I tried reproducing on master and some earlier builds (thought maybe https://github.com/onivim/oni2/pull/2574 could've impacted it) - but I'm not able to reproduce (tried on OSX 10.15.4 as well)
A couple questions that might help to narrow down:

Do you have any custom configuration or keybindings? Those might be needed to reproduce
What keyboard layout are you using?

@bryphe You're exactly right, it's an issue with some of my experimental.viml bindings. It seems that it doesn't like remapping sequences.
 I think when the first of the sequence matches but the rest doesn't, the matched characters are added to a buffer which is dumped on insert. (I noticed that these stray characters can also be printed when entering ex mode.)
Should I add an edit to this issue? Or close and open a new one? Are you accepting issues to do with the experimental viml?

Thanks for the details @baffalop !

 I think when the first of the sequence matches but the rest doesn't, the matched characters are added to a buffer which is dumped on insert. (I noticed that these stray characters can also be printed when entering ex mode.)

Definitely a bug - would you mind sharing a binding that reproduces here?

Should I add an edit to this issue? Or close and open a new one? Are you accepting issues to do with the experimental viml?

Thanks for checking - updated the title, we can use this to track. And definitely accepting issues - hoping to spend more time on the VimL support soon 👍

Ok, well for instance if I include the following mappings (to go to the beginning/end of a containing [] text object):
"experimental.viml": [
  "nnoremap [[ \"_ya[",
  "nnoremap ]] va[<Esc>"
]

Then entering [b will act as if [ was ignored and b was intepreted as back a word. Then [ will be inserted on next entering insert mode. Same or ]b.
In fact, no matter what is typed after [, the same behaviour obtains: [ is ignored, the following commands are received as normal, [ is dumped when entering insert mode.
Similarly, I could have a longer sequence, eg. nnoremap ysiw ciw()<Esc>P (a poor man's Surround) and on entering ys, y is ignored and s puts me in insert mode with the character under the cursor substituted with y.


Sent Time Corrections

What changed;

CLI now also accepts --firedrill/--no-firedrill options and has docs see snews_pt --help (defaults True)
snews_pt run-scenarios also has the firedrill option (default True)
both subscriber and publisher also displays the broker once (to be sure if we are using firedrill brokers)
When a message is published it creates a utcnow and appends to all messages 'sent_time' field.
This 'sent_time' is used to stamp the '_id'

This 'sent_time' is used to stamp the '_id' because it is always created whereas before we were using 'machine_time' and if nothing is passed this was None by default. Thus, all the messages from the same detector had the same id e.g. "4_CoincidenceTier_None" we did not notice this problem before but now that we have mongo it complains about messages having the exact same id. Replacing this with a utcnow timestamp fixes it. Accordingly, the tests had to be fixed. We also missed it there, all the test cases were testing for "_id"=="XX_CoincidenceTier_None". Now I just check the rest of the keys and not the timestamp. Furthermore, the snews_cs has some functions that were checking the 'sent_time', apparently we removed it at some point from pt but kept it in cs.
Now that we have a stamp when it is sent (if user also provides a machine_time with their timestamp) we can measure the latency between the user submitting and snews_pt publishing. As far as I see this is ~ms levels.
On the snews_cs side we have "received_time" and this can tell us the latency between the message being published, and after kafka flight it being received at Purdue.
So we have;
"neutrino_time"  - time of the neutrino burst
"machine_time" - the time experiment thinks they sent (optional)
"sent_time" - the time we stamp right before hop.Stream.write()
"received_time" - the time snews_cs reads it from the hop stream
Thus, we can infer several latencies.

@Storreslara @joesmolsky could you guys please review the commits?
I'm also going to make a small PR on the CS (to Sebastian's stats branch)

I also updated the requirements.txt because one package was missing and it is needed for the docs.
I updated some other packages, and made flexible dependencies (or whatever it is called when you set package~=x.y.z with a tilde). I set the same dependencies for both snews_pt and snews_cs so that both can be run with the same environment.
They don't have to be exactly the same, but the mutual package's versions have to be the same.

ipython actuall has a latest version 8.0.2 but for some reason, sphinx (readthedocs) can't find anything greater than 7.32.0 so I set it to this.


there is an option to increase quanity  when user is try to booked product in both themes

Bagisto Version

[ ] master branch

Steps To Reproduce

go to front page of any booking product

Actual Result

Expected Result

the same issue please check in  bliss theme at time of checkout  and in velocity there is option to  increase quanity at time of checkout

@singhgagan03 @Sanjaybhattwebkul ,
Same issue #5728


[BACK-81] Tests for the methods I implemented

Description of Issue
I had the task to implement tests for the method I coded. This includes Hackernews and OpenAI methods
How I implemented
I used console.log for most of the methods since the answers of chatgpt are not predictable and the contribtuor count is constantly changing. I only use propper testing for the categories.
Other
Not sure how much documentation is necessary for this

It was decided that we won't merge the tests with main


adding tracks to library

This is a feature request to be able to add a list of track IDs to a users' library, rather than a just a playlist.
I'm new to this project but may try to work on this if I can muster the time.
The current post endpoint is at: /addsongstolocker.
Examining the post request when a user adds a track to the library, this is what I can determine about the data being sent:
[[0, 2, 0] <--- I don't know what this means, I assume it's a code of some sort for the client status?
[null,
 [[<uuid1>, <track ID>], ...]
]

I don't know what uuid1 is, though. After some testing, it seems to be a track identifier that resets between sessions. Not sure if this is seen in other bits of the API.
Looks pretty easy to implement if the other bits of data are deciphered, which you guys might have a better idea about. Any tips?

It looks like you're referring to the webclient endpoint. Would it be ok to use the mobileclient endpoint instead?
We generally don't do any more development on the webclient because of the nasty data format you mentioned :grin:

ah, yeah, totally. I'll do some sniffing when I get home and see if I can decipher the data format, unless you have tips for spoofing mobile to look at the data format?

Oh, I meant that I thought this was already supported, but through the mobileclient (not the webclient).

oh, man, totally missed that method. sick! will make importing my rdio library much easier. this should be closed.

Haha, no problem, just wanted to make sure I understood (technically, it doesn't support a list of songs). Chalk it up to poorly-organized docs.

just a little tip -- I think one of the reasons why I missed it at first glance is because it references "all access", which afaik is no longer part of google music, so I may have passed it off as deprecated even though it is not

Wow, I didn't know they got rid of that branding, haha. I'll have to look into this at some point.

Just found that issue, will help to import my rdio collection as well!
@dcalacci did you come up with some script to do it?

Nevermind, I wrote one :)
https://gist.github.com/madx/b0a09ae013c628fe22b8f255f6b66fb4


expand extension actuator interface with migrate and restore

What this PR does / why we need it:
With this PR we extend the Extensions actuator interface with Migrate and Restore functionality.
Which issue(s) this PR fixes:
Part of # 1631
Special notes for your reviewer:
Release note:

Extend the Extensions actuator interface with Migrate and Restore

/kind/enhancement

/lgtm
/reviewed/do-not-merge
until<EMAIL_ADDRESS>is released, cc @ialidzhikov


Ensure @BeforeGroups gets called only once

Fixes #1108

@juherr Can you please help with reviewing this ?

LGTM after the little changes on form :+1:

@juherr Thanks for taking the time to help with the review. I have fixed all the review comments. Please help with merge.

:+1:
Ping @cbeust


Add Camille To Email Address Override Whitelist

This will help RE-Govuk test email-alert-api

I see there's a comment about keeping the list of users in sync with https://www.notifications.service.gov.uk/services/ecfc7e2f-5145-45a6-9413-5d9b6e813ea9/users — I'm not able to view that page, though. Is it in sync? :)


Build error : ‘attribute_error’ is not a member of ‘py’

Thanks for sharing these excellent work.
This strange error occured during pip installing, i found no info about it on google
ubuntu 20.04
gcc 9.4.0
cmake 3.21.6
ceres 2.2.0
python 3.9.7
Building log:
-- Generating done
-- Build files have been written to: /home/horry/pixel-perfect-sfm/build/temp.linux-x86_64-3.9
[  3%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/base/src/graph.cc.o
[  6%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/util/src/simple_logger.cc.o
[  9%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/util/src/threading.cc.o
[ 12%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/util/src/stringprintf.cc.o
[ 15%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/featurepatch.cc.o
[ 18%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/util/src/glog.cc.o
[ 21%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/featuremap.cc.o
[ 25%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/featureview.cc.o
[ 28%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/featuremanager.cc.o
[ 31%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/references.cc.o
[ 34%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/bundle_adjustment/src/bundle_adjustment_options.cc.o
[ 37%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/features/src/featureset.cc.o
[ 40%] Building CXX object pixsfm/CMakeFiles/pixsfm.dir/keypoint_adjustment/src/keypoint_adjustment_options.cc.o
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/references.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/references.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featuremap.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featuremap.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureview.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureview.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featurepatch.h:3,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featurepatch.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureset.h:3,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureset.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/keypoint_adjustment/src/keypoint_adjustment_options.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/keypoint_adjustment/src/keypoint_adjustment_options.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/bundle_adjustment/src/bundle_adjustment_options.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/bundle_adjustment/src/bundle_adjustment_options.cc:1:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featuremanager.h:2,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featuremanager.cc:7:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h: In lambda function:
/home/horry/pixel-perfect-sfm/pixsfm/_pixsfm/src/helpers.h:222:21: error: ‘attribute_error’ is not a member of ‘py’
222 |           throw py::attribute_error(ss.str());
|                     ^~~~~~~~~~~~~~~
In file included from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureview.cc:3:
/home/horry/pixel-perfect-sfm/pixsfm/util/src/memory.h: In function ‘long long int pixsfm::NumNonZerosJacobian(const ceres::Problem*)’:
/home/horry/pixel-perfect-sfm/pixsfm/util/src/memory.h:59:74: warning: ‘int ceres::Problem::ParameterBlockLocalSize(const double*) const’ is deprecated: LocalParameterizations are deprecated. Use ParameterBlockTangentSize instead. [-Wdeprecated-declarations]
59 |         num_active_params += problem->ParameterBlockLocalSize(param_block);
|                                                                          ^
In file included from /usr/local/include/ceres/ceres.h:66,
from /home/horry/pixel-perfect-sfm/pixsfm/base/src/interpolation.h:15,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featurepatch.h:20,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featuremap.h:24,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureview.h:23,
from /home/horry/pixel-perfect-sfm/pixsfm/features/src/featureview.cc:1:
/usr/local/include/ceres/problem.h:494:7: note: declared here
494 |   int ParameterBlockLocalSize(const double* values) const;
|       ^~~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:216：pixsfm/CMakeFiles/pixsfm.dir/features/src/references.cc.o] error 1
make[2]: *** waiting for unfinished work....
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:230：pixsfm/CMakeFiles/pixsfm.dir/keypoint_adjustment/src/keypoint_adjustment_options.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:146：pixsfm/CMakeFiles/pixsfm.dir/features/src/featurepatch.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:244：pixsfm/CMakeFiles/pixsfm.dir/bundle_adjustment/src/bundle_adjustment_options.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:160：pixsfm/CMakeFiles/pixsfm.dir/features/src/featuremap.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:188：pixsfm/CMakeFiles/pixsfm.dir/features/src/featureview.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:202：pixsfm/CMakeFiles/pixsfm.dir/features/src/featureset.cc.o] error 1
make[2]: *** [pixsfm/CMakeFiles/pixsfm.dir/build.make:174：pixsfm/CMakeFiles/pixsfm.dir/features/src/featuremanager.cc.o] error 1
make[1]: *** [CMakeFiles/Makefile2:421：pixsfm/CMakeFiles/pixsfm.dir/all] error 2
make: *** [Makefile:136：all] error 2
Traceback......etc.....
----------------------------------------
ERROR: Command errored out with exit status 1: /home/horry/anaconda3/envs/pytorch/bin/python3.9 -c 'import io, os, sys, setuptools, tokenize; sys.argv[0] = '"'"'/home/horry/pixel-perfect-sfm/setup.py'"'"'; file='"'"'/home/horry/pixel-perfect-sfm/setup.py'"'"';f = getattr(tokenize, '"'"'open'"'"', open)(file) if os.path.exists(file) else io.StringIO('"'"'from setuptools import setup; setup()'"'"');code = f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, file, '"'"'exec'"'"'))' develop --no-deps Check the logs for full command output.
Thanks for any advice

收到，谢谢

I solved this problem by building it in a simplified env


Hex Docs outdated for Bamboo.EmailPreviewPlug

Hi, I just noticed that the docs in hex refer to Bamboo.EmailPreviewPlug which doesn't work as it's Bamboo.SentEmailViewerPlug if I'm correct?
I can file a PR but I haven't found the docs anywhere to do it?

@mnussbaumer Thanks for reporting! I wonder if you are looking at the docs for an outdated version? The current docs are here.

@maymillerricci Looks fine! I honestly don't remember where I was looking at that time it's been a while
👍

Okay great! And sorry for the delay. 😄 I'm going to close this.


[BUG]still cannot use standalone app, cannot select input 2 ONLY

on many interfaces, you have to select input 2 as the sole input..
the standalone version of this app is unusable because it forces input 1 and 2 as a stereo configuration.
In other software, say Amplitube, I select input 2 for BOTH left and right inputs.
In your current setup, I cannot make a selection for the Left input, it defaults to 1.  And because of this, it doesn't work, at all.
This should be the highest priority than any other feature requests to just get the basic functionality working.

duplicate of #386

Please +1 #386. Closing this duplicate so that I have one place to focus.


Adds the functionality of unmounting storage using the mounted path.

Issue : https://github.com/redhat-developer/odo/issues/493
Signed-off-by: mdas<EMAIL_ADDRESS>

What would happen if we pass in: ./ or something that is not valid? Shouldn't we error out to the user?

@cdrage You mean at the start of the path?

@mik-dass Yes

@cdrage I think it might be complicated as there are many combinations which are possible

What would happen if we pass in: ./ or something that is not valid (or does not exist)? Shouldn't we error out to the user?

This should be treated as a storage name, and it should fail because no such storage exits.

@kadel Done

LGTM


add a release-note

Codecov Report

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

@@           Coverage Diff           @@
##           master     #129   +/-   ##
=======================================
  Coverage   95.81%   95.81%
=======================================
  Files           9        9
  Lines         716      716
=======================================
  Hits          686      686
  Misses         30       30

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 a38b5d7...f95896e. Read the comment docs.


export Mailin function instead of relying on globals

As part of the sendinblue API version 2 deprecation plan this repository is no longer supported. Please refer to its updated version 3: https://github.com/sendinblue/APIv3-nodejs-library


Removed npm-run-all and replaced script with concurrently.

Fix for #1407
====

Removed the npm-run-all package
Rewrote watch script to use concurrently

Thanks @tonyanziano. Looks good to me.
I have updated package-lock.json to revert 2 changes about "https -> http" on chalk/json5. There is a bug on NPM platform that unknowingly turned https://registry.npmjs.com/ links to http://registry.npmjs.com/ links. Not your fault.
https://github.com/npm/npm/issues/20719


flushOK is wrong in doCommit method when groupCommitRequest flushed 2 times

version:4.4.0
When nextOffset of groupCommitRequest  is in next mappedFile， it will excute mappedFileQueue.flush(0) 2 times，but flushOK is first flushed result（false），actually it mybe true after second flush. This cause producer get a error send result when a new mappedFile created. After 2 times flush loop, flushOK should be recheck to get real flush result.
show code:
org.apache.rocketmq.store.CommitLog.GroupCommitService#doCommit
                    for (GroupCommitRequest req : this.requestsRead) {
                        // There may be a message in the next file, so a maximum of
                        // two times the flush
                        boolean flushOK = false;
                        for (int i = 0; i < 2 && !flushOK; i++) {
                            flushOK = CommitLog.this.mappedFileQueue.getFlushedWhere() >= req.getNextOffset();

                            if (!flushOK) {
                                CommitLog.this.mappedFileQueue.flush(0);
                            }
                        }

                        req.wakeupCustomer(flushOK);
                    }

By adding log between flushed over and  wakeupCustomer, confirmed flushOK is false but CommitLog.this.mappedFileQueue.getFlushedWhere() > req.getNextOffset(), flushOK is wrong.

@panzhi33  git it