Single process restart fixes

Description:
Fixes:

startup on Docker (ignore setpgid error)
restart when using a pidfile
restart when started with python -m homeassistant

Related issue (if applicable): #1793
Checklist:
If the code does not interact with devices:

[ ] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[ ] Tests have been added to verify that the new code works.

Think this looks good 👍

🐬

@jaharkes Fix is working!


catkin can not be find when run colcon build in compile.sh

Hi,
I am interested in your excellent work. However, when I run ./deploy/build_docker.sh it throws an error:
--- stderr: catkin_simple
CMake Deprecation Warning at CMakeLists.txt:1 (cmake_minimum_required):
  Compatibility with CMake < 2.8.12 will be removed from a future version of
  CMake.

  Update the VERSION argument <min> value or use a ...<max> suffix to tell
  CMake that the project does not need compatibility with older versions.

CMake Error at CMakeLists.txt:4 (find_package):
  By not providing "Findcatkin.cmake" in CMAKE_MODULE_PATH this project has
  asked CMake to find a package configuration file provided by "catkin", but
  CMake did not find one.

  Could not find a package configuration file provided by "catkin" with any
  of the following names:

    catkinConfig.cmake
    catkin-config.cmake

  Add the installation prefix of "catkin" to CMAKE_PREFIX_PATH or set
  "catkin_DIR" to a directory containing one of the above files.  If "catkin"
  provides a separate development package or SDK, be sure it has been
  installed.

I try to install catkin-tools in docker manually but it still didn't work.
I search that it is because the package has been created for ROS1, but runs on ROS2.
Waiting for your reply!

Yes, catkin_simple is a deprecated dependency from the ROS1 days that we don't need anymore.
I removed it from the repo and added some fixes to the deployment scripts in #36.
Should be fine now.


Prompt changed based on when dspy.settings.configure(lm=lm) is executed.

If you configure the LM (dspy.settings.configure(lm=lm)) before initializing your modules you get a different prompt to if you configure your LM after initializing your modules.
    dspy.settings.configure(lm=lm)
    pot = ChainOfThought(BasicQA)

Adds "reasoning" to the output signature.
    pot = ChainOfThought(BasicQA)
    dspy.settings.configure(lm=lm)

Adds "rationale" to the output signature.
Think its because the dspy.settings.lm is referenced in ChainOfThought.__init__:
        # Add "rationale" field to the output signature.
        if isinstance(dspy.settings.lm, dspy.LM):
            extended_signature = signature.prepend("reasoning", rationale_type, type_=str)
        else:
            extended_signature = signature.prepend("rationale", rationale_type, type_=str)

Maybe something to flag in migration docs or just to be aware of before this logic is removed when dsp.LM is removed.

Very interesting, thanks for the catch @mikeedjones ! Good reason to migrate fully in 2.6 soon

Since this could be fairly annoying, I'm thinking the following: if we don't release 2.6 (which deletes old clients) within a very short time, we should make them only work if dspy.configure(legacy=True) before any DSPy usage. Once legacy is turned on, it cannot be turned off.
That way, we can greatly simplify the if statement causing this issue.

As far as I can tell this only appears in ChainOfThought and Predict - could we have a ChainOfThoughtv1 or something for the change over period?

After migrating from version 2.4 to 2.5, I noticed that the “rationale” field has been replaced by the “reasoning” field. Additionally, the content of this new field consistently displays the message, “Not supplied for this particular example.” This change is affecting the results.

@Nasreddine Can you share more about your LM and your modules? Thank you!

@okhat sure!
Task: converting row text into structured xml.
model: gpt4o-mini
Module: ChainOfThought
Optimizer: BootstrapFewShot
Reasoning results for the same example in the compiled program :
DSPy 2.4
  "rationale": "produce the XML. We will start by structuring the preface section according to the Akoma Ntoso (AKN) format. The key components we need to include are the long title, the document type, the document number, and the document date. We will also ensure to format any italicized text correctly and include the relevant references where applicable.\n\nHere\u2019s the XML representation based on the provided text and document number:",
DSPy 2.5
"reasoning": "Not supplied for this particular example.",

@okhat I’m also testing with advanced optimizers, but I’m still getting the same results.
       "rationale": "Not supplied for this particular example.",

I believe this issue is due to the new prompt template in the dspy.LM class. I’m experiencing the same problem and am awaiting an explanation for this new behavior.

I was initially impressed because of how nice and clean it looked, until I re-ran my bootstrapfewshotwithrandomsearch optimizer, which produced no reasoning.

@Nasreddine @satyaloka93 Until we explore this behavior, in your optimizers, set max_labeled_demos=2. The default of 16 confuses some models.

@okhat This is still happening sadly, even with max_labled_demos=2 (and 1). My max_bootstrapped_demos are 8-10 depending. This is on 2.5.31 now. I see this message is coming from dspy/adapters/json_adapter.py and dspy/adapters/chat_adapter.py.
def format_fields(self, signature, values, role):
        fields_with_values = {
            FieldInfoWithName(name=field_name, info=field_info): values.get(
                field_name, "Not supplied for this particular example."
            )
            for field_name, field_info in signature.fields.items()
            if field_name in values
        }

So it's failing to get field_name, but why?

@satyaloka93 or @Nasreddine are you able to share a small task where this happens with a specific LM? That will help us a lot in debugging the issue!

@okhat Here's my optimizer code with signature/module I'm using:
'''
gemma2 loaded locally with llama-server (llama.cpp)
'''
os.environ['OPENAI_API_KEY'] = "sk-xxxx"
os.environ['OPENAI_API_BASE'] = "http://localhost:8080/v1/"
turbo = dspy.LM('gpt-3.5-turbo')
dspy.settings.configure(lm=turbo)
'''
Create dataset for DSpy - language translations done by gpt4o
'''
examples = []
for index,row in df.iterrows():
examples.append(Example(message=row['Message'],translation=row['gpt-4o']))
trainset = [x.with_inputs('message') for x in examples[100:200]]
devset = [x.with_inputs('message') for x in examples[200:250]]
'''
Metric is cosine sim
'''
def sentence_similarity(sentence1, sentence2, model_name="sentence-transformers/all-mpnet-base-v2"):
....do similarity...
def similarity(example, pred, trace=None):
...call sim function,write some logging output
'''
Main translation signature and module
'''
class Translation(dspy.Signature):
"""Translate foreign language to English, or repeat if not necessary"""
message = dspy.InputField(desc="message for translation")
translation = dspy.OutputField(desc="translation in English, or repeated message")
class TranslationModule(dspy.Module):
def init(self):
super().init()
    self.generate_answer = dspy.ChainOfThought(Translation)

def forward(self, message):
    prediction = self.generate_answer(message=message)
    return dspy.Prediction(translation=prediction.translation)

'''
Optimizer
'''
fewshot_optimizer = BootstrapFewShotWithRandomSearch(metric=similarity,
max_bootstrapped_demos=8,
max_labeled_demos=2,
num_candidate_programs=2,
num_threads=2)
compiled_qa = fewshot_optimizer.compile(student=TranslationModule(),
trainset=trainset,valset=devset

I just ran this quickly this morning, example of the program afterwards:

[2024-11-20T06:35:06.774488]

System message:

Your input fields are:
1. `message` (str): message for translation

Your output fields are:
1. `reasoning` (str)
2. `translation` (str): translation in English, or repeated message

All interactions will be structured in the following way, with the appropriate values filled in.

[[ ## message ## ]]
{message}

[[ ## reasoning ## ]]
{reasoning}

[[ ## translation ## ]]
{translation}

[[ ## completed ## ]]

In adhering to this structure, your objective is:
        Translate foreign language to English, or repeat if not necessary

User message:

This is an example of the task, though some input or output fields are not supplied.

[[ ## message ## ]]
nerezus Ващето тут Пинч обсуждается а не СП2......... Дело твое.......Че хоч то и ставь.......

Respond with the corresponding output fields, starting with the field `[[ ## reasoning ## ]]`, then `[[ ## translation ## ]]`, and then ending with the marker for `[[ ## completed ## ]]`.

Assistant message:

[[ ## reasoning ## ]]
Not supplied for this particular example.

[[ ## translation ## ]]
Actually, Pinch is being discussed here, not SP2......... It's your business....... Do whatever you want.......

[[ ## completed ## ]]

User message:

This is an example of the task, though some input or output fields are not supplied.

[[ ## message ## ]]
Universal1337 V2 unpacked //Index// Steam Produkt Keys Network Passes Paypal Ebay AIM ICQ Miranda Trillian MSN Messenger AOL Instant Messenger Windows Messenger (In Windows XP und Vista) Windows Live Messenger (In Windows XP und Vista) Yahoo Messenger Google Talk GAIM (Pidgin) Windows Mail Outlook Express Microsoft Outlook IncrediMail Eudora Netscape Mozilla Firefox Internet Explorer Group Mail Free PC Infos :zns5: Скачать|Download

Respond with the corresponding output fields, starting with the field `[[ ## reasoning ## ]]`, then `[[ ## translation ## ]]`, and then ending with the marker for `[[ ## completed ## ]]`.

Assistant message:

[[ ## reasoning ## ]]
Not supplied for this particular example.

[[ ## translation ## ]]
Universal1337 V2 unpacked //Index// Steam Product Keys Network Passes Paypal Ebay AIM ICQ Miranda Trillian MSN Messenger AOL Instant Messenger Windows Messenger (In Windows XP and Vista) Windows Live Messenger (In Windows XP and Vista) Yahoo Messenger Google Talk GAIM (Pidgin) Windows Mail Outlook Express Microsoft Outlook IncrediMail Eudora Netscape Mozilla Firefox Internet Explorer Group Mail Free PC Info :zns5: Скачать|Download

[[ ## completed ## ]]

Any thoughts on this @okhat? It is a pretty simple program I've used with numerous llms, since 2023! Confounding how this new adapter is somehow failing.

I am even able to get great structured output with the old typed predictors and pydantic model ouputs with these llms and 2.4.

Marking as resolved for @satyaloka93 by setting max_labeled_demos=0


navigator.mediaDevices is not available in insecure contexts

lib Update Request

Configuration Check

My compilation target is esnext and my lib is ['esnext', 'dom'].
Missing / Incorrect Definition
navigator.mediaDevices (and potentially everything that doesn't exist in insecure contexts) should be an optional property in the interface.
Documentation Link
Accoding to the spec:
partial interface Navigator {
  [SameObject, SecureContext] readonly attribute MediaDevices mediaDevices;
};

The meaning of SecureContext:

... it indicates that the construct is exposed only within a secure context.

Context: We blew up our production because we trusted DOM typings :shrug:


Implement csr_matrix.getitem for single element

I implemented getitem method for single element.
related to #36

Duplicated indeces are common for CSR format and cusparse routines work for them properly?

Actually currently cupy.sparse does not support duplicated indices. We need to implement sum_duplicates and is implemented in #326.

I updated it.

Actually currently cupy.sparse does not support duplicated indices. We need to implement sum_duplicates and is implemented in #326.

I got it.

Would you check the CI result?

Please fix to pass flake8.

Please merge master after #347 is merged to master so that this PR passes CI completely.


Support Incremental Annotation Processing in Gradle 4.7+

Support Incremental Annotation Processing in Gradle 4.7+
https://docs.gradle.org/nightly/userguide/java_plugin.html#sec:incremental_annotation_processing

Implemented and will be part of the upcoming 1.1.11 release.  Any help testing the change is appreciated.

Included in 1.1.11 release.


Observability metrics for VS and Upstream will continue to report after object is removed

Gloo Edge Version
1.11.x (latest stable)
Kubernetes Version
1.21.x
Describe the bug
After adding metrics via helm values as described in https://github.com/solo-io/gloo/issues/3964, it was observed that upstreams or virtual services that are removed/delete will continue to report their last status.  Would expect the metrics to stop after the upstream or virtual service is deleted.
Steps to reproduce the bug
Enabled VS and upstream metrics by setting the following helm-values.yaml file
gloo:
  settings:
    kubeResourceOverride:
      spec:
        observabilityOptions:
          configStatusMetricLabels:
            Upstream.v1.gloo.solo.io:
              labelToPath:
                name: '{.metadata.name}'
                namespace: '{.metadata.namespace}'
            VirtualService.v1.gateway.solo.io:
              labelToPath:
                name: '{.metadata.name}'
                namespace: '{.metadata.namespace}'

Add an upstream or VS.  Observe the metrics in prometheus.  Delete the upstream and/or VS. Metric will continue (either with 0/healthy or 1/unhealthy).
Current workaround for a deleted VS restart the gateway deployment and for a deleted upstream restart the gloo deployment.
Expected Behavior
When a VS or upstream is removed the associated metric would cease.
Additional Context
No response

Related metrics:
validation_gateway_solo_io_virtual_service_config_status
validation_gateway_solo_io_upstream_config_status

Restarting the Gloo pod also seems to remove the deleted VS in 1.18.0.


should use "ng" service name for all angular related tasks

image that that when we need to npm install.
If we have serve and test services like currently, we have to update both.
If we have only one, update only one is preferable.

so the workflow will be like:
to start serving:
$ docker-compose up -d && docker-compose logs -f

to start unit testing:
$ docker-compose exec ng /bin/bash
$ npm run test

to install new npm package:
$ docker-compose exec ng /bin/bash
$ npm install taxer --save

It's more time saving, using 1 container is enough for developing. We will revise this later with big apps.


No commit option in .pronto.yml? (also, crashes unless local master exists)

(Didn't see this brought up elsewhere. And from what I can tell, behavior is the same in master as my installed version of 0.7.1.)
Essentially, I think it'd be useful if you could set pronto to check against origin/master rather than master. There are a couple of reasons for this:

not all workflows require having a local master branch (e.g. even for small changes, we make a feature branch, push it upstream, and create a PR)
when working on a feature, it's easy to not be aware that master is out of date with origin/master, and if you run pronto after merging origin/master into your feature branch, you'll end up with spurious results since the diff will include recent merges as well as your own code.

Obviously, this is fixed by always passing -c origin/master to pronto. But I think it'd be convenient if this could be configured in .pronto.yml. (Sorry if that already exists -- I looked at the docs and code but didn't see a way to configure the default commit, and tried adding commit: to .pronto.yml to no effect.
Also, Pronto crashes when it can't find the commit it's looking for with
...pronto-0.7.1/lib/pronto/git/repository.rb:72:in `merge_base': Revspec 'foo/bar' not found. (Rugged::ReferenceError)

Of course, there's no way to recover from this error, but it might be nice to show a friendlier error message. For instance, fat-fingering the argument to -c will trigger this, and I wouldn't consider that to be "exceptional," really.
I'm willing to make PRs for these changes if they're in fact desired.

I'm willing to make PRs for these changes if they're in fact desired.

@johncip yes, go for it! 😄 Sorry for a late reply 🙇.
Please post an update if you intend to do it, so we wouldn't end up in a situation where we both are doing the same work 😄.

Ok!

In the meantime, below is an example of a workaround I use to get the commit in .git/hooks/pre-push.
This git hook asks pronto to run Rubocop (thanks to pronto-rubocop) before pushing to the remote repository.
Note the last line retrieves the name of the base branch git name-rev --name-only HEAD:
pronto run -c $(git name-rev --name-only HEAD) --exit-code
My understanding is that HEAD denotes the main/base branch checked out by default when cloning the repository. It is most often main in recent repositories or master in older ones. Then, in the local repository HEAD is a symbolic ref to the base branch.
The actual value of HEAD of course is configured in the repository, for instance in Github:
https://github.com/USERNAME/REPO/settings/branches.
#!/bin/sh

# To reinstall this script in the same or another git repo run:
# curl -sSL https://gist.githubusercontent.com/elliotthilaire/cef91754bb296d67e776/raw/pre-push > .git/hooks/pre-push; chmod +x .git/hooks/pre-push

# check that pronto is installed
#  Source: http://elliotthilaire.net/gem-pronto-and-git-hooks/
#
hash pronto 2>/dev/null || {
  echo >&2 "Pronto is not installed. Install with 'gem install pronto pronto-rubocop'";
  echo >&2 "Find other Pronto runners at https://github.com/mmozuras/pronto#runners";
  exit 0;
}

# Run pronto
pronto run -c $(git name-rev --name-only HEAD) --exit-code


Weak Encryption detected by Fortify scans

In our current version this is the file with issues. pdfviewer/build/pdf.worker.js line 7744, line 7745, and line 7763
In the latest version I just pulled down hoping that this was fixed and it isn't, this is the file now: pdf.js-master\src\core\crypto.js line 23 line 24, and line 41
Summary
The call to ARCFourCipher_encryptBlock() at pdf.worker.js line 7763 uses a weak encryption algorithm that cannot guarantee the confidentiality of sensitive data.The identified call uses a weak encryption algorithm that cannot guarantee the confidentiality of sensitive data.
Explanation
Antiquated encryption algorithms such as DES no longer provide sufficient protection for use with sensitive data. Encryption algorithms rely on key size as one of the primary mechanisms to ensure cryptographic strength. Cryptographic strength is often measured by the time and computational power needed to generate a valid key. Advances in computing power have made it possible to obtain small encryption keys in a reasonable amount of time. For example, the 56-bit key used in DES posed a significant computational hurdle in the 1970's when the algorithm was first developed, but today DES can be cracked in less than a day using commonly available equipment.
Instance ID: 76DBA5DF23B75AFD4D7D14B21B49F543
Primary Rule ID: C8EDE160-7B48-42C6-8D4B-051B35E47BD9
Standards and Best Practices
OWASP 2013
A6 - Sensitive Data Exposure
OWASP 2014 Mobile
M6 - Broken Cryptography
PCI 3.2
6.5.3 - Insecure Cryptographic Storage
DISA STIG 4.1
APSC-DV-002010 CAT II
APSC-DV-002040 CAT II
FISMA
MP
This is coming up as a critical issue and we need to upgrade to a version that is corrected with current standards of encryption asap. This will be a issue from more people real soon as they get these scans on the software.

PDF.js does not produce PDFs with weak encryption. So the issue looks unrelated.
The issue does not provide/document expected behavior of PDF.js in case if user wants to open PDF with weak encryption. Closing as incomplete.

Please look at this again... If you look at the files in question and looked at the encryption methods you are using and read the rest of the original post you will see that you are using a deprecated encryption methods. I am asking for you to update your encryption methods to the latest standards.
Thanks,

I am asking for you to update your encryption methods to the latest standards.

@lmcguffee I'm not sure what do you mean by "pdate your encryption methods to the latest standards". In this case you just asking to stop using encryption and not to follow PDF32000 specification. This means legacy PDF will not be shown for end users. Is it expected behavior?

If you read this link and look at the code in the line mentioned you might understand what I'm meaning.
https://security.stackexchange.com/questions/26765/what-are-the-differences-between-the-arcfour-arcfour128-and-arcfour256-ciphers
The ARCfourCipher is what you are using and it is deprecated.

The ARCfourCipher is what you are using and it is deprecated.

Yes, I understand this part. What I'm asking from you what will be the expected behavior for the viewer if PDF document encrypted using RC4? What you are asking is to remove RC4 support, which mean PDF.js will no longer be capable to read the legacy PDFs.

hum... guess that isn't feasible from your prospective. At some point we just can't support vulnerable apps. We will have to move to a different app that isn't vulnerable. At some point more and more of the people using this app will have to do the same. So I'd look at your user base and determine what is best for all. Even Microsoft says we are not supporting older systems.  If the legacy apps are vulnerable then can you realistically support that? I'm hoping that you are looking to move forward. Let me know what you figure out. For now you can close this.

At some point we just can't support vulnerable apps. We will have to move to a different app that isn't vulnerable.

Any PDF viewer contains code RC4 -- it will not be possible to remove, e.g. you can observe it in Chrome https://pdfium.googlesource.com/pdfium/+/master/core/fdrm/crypto/fx_crypt.cpp#156 . It also can be found in Reader and Mac OSX Preview.

At some point more and more of the people using this app will have to do the same.

It's their choice. As alternative, we provide source code as open source, so people can easily to remove parts they don't like and deploy custom version of PDF.js instead.

If the legacy apps are vulnerable then can you realistically support that?

There is no proof that PDF.js is legacy or vulnerable. You provided proof of existence of ARCFour in our code base, but you did not provide proof that PDF.js uses this algorithm to expose sensitive data that is not already accessible by other means. PDFs are generated by other software which means, which means the sensitive data can be exposed without PDF.js.

Let me know what you figure out.

@lmcguffee I tried contacting Fortify about the false detection. But they require to be a paid customer (?) to provide any support. It will be helpful for the purpose of this report to provide details why they decided to flag PDF.js product as dangerous. So we cannot provide additional information on our side.

I received response from Fortify, which somewhat confirms my comments above. To make the response complete, I should probably to be clear in following areas:

Mozilla's PDF.js, demo viewer or pdfjs-dist as it is packaged, does not encrypt any customers PDF.
PDF.js (or any other viewers) needs to use RC4 to check if user has particular rights and/or if user provided correct password for some PDF (see the PDF32000 standard specification).
Good portion of PDFs are encrypted or protected by RC4, so disabling or removing this algorithm will prevent a PDF viewer from opening these PDFs.
Fortify only detects presence of antiquated encryption algorithms, but it does not detect how algorithm is used, whether it is disabled, or a condition when it's invoked; so the Fortify is merely provides a warning about these algorithms presence. The decision how a particular threat is mitigated falls on its users -- it can be looking into how the algorithm is used, removing it or just not support of PDF viewing.

@yurydelendik  - Any update on Fortify Technical Support case number #00273796 ?

Hello, we have this finding as well with our Fortify results. Is there a way to get a copy of this .js file that will ignore this functionality?

Hi, was anyone able to find a solution to this problem?


"edit-schema" permission should take database into account

Such that users can be granted permission to edit schemas in one database but not in another.
Also needed for this issue:

#31

This also affects this page, which should only display databases you are allowed to edit the schema of:


1.14 compatibility

Naive fix for 1.14.
Nether portal detection seems to work, even though we no longer use TravelAgent (removed in 1.14 Bukkit by Spigot).
Returning from the End seems to throw you out of a nether portal.

Not needed any longer.


(BUG?) Energy Upgrades no longer reduce operating power costs for Electrolytic Separators

Issue description:
The Energy Upgrade no longer reduces the operating power costs for the Electrolytic Separator, it only increases the internal power reserves that the machine has available.
Steps to reproduce:

Place down Electrolytic Separator, wire it with power and pipe water into it.
Watch and note operating power cost.
Put a full stack of 8 Energy Upgrades into it, then go back to power panel on the machine and watch as the operating power cost for the machine does NOT decrease.

Version (make sure you are on the latest version before reporting):
Forge: <IP_ADDRESS>73
Mekanism: 1.12.2-<IP_ADDRESS>9
Other relevant version: Mekanism Generators 1.12.2-<IP_ADDRESS>9
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

Not so much a bug as a balancing thing. Making hydrogen then burning hydrogen used to be a net gain from what I understand.

They've never been meant to reduce usage in the separator, as ding so would allow for a net energy gain, thus violating thermodynamics.


(open-payments): list incoming payments

Add functionality to list incoming payments in the the open payments client: https://docs.openpayments.guide/reference/list-incoming-payments

Refer to https://github.com/interledger/rafiki/issues/817 for similar functionality


HOST_VISIBLE always ending up with HOST_COHERENT enabled without explicitly defining it

My apologies in advance in case this is/was answered in the docs (Or if it's something basic that i've overlooked).
I have implemented UBOs with VMA and benchmarking various implementations as to how memory is sent to the GPU.
(Specifically the impact of the HOST_COHERENT memory flag)
Now, i define UBOs like this:
VkBufferCreateInfo bufferInfo = {};
bufferInfo.sType = VK_STRUCTURE_TYPE_BUFFER_CREATE_INFO;
bufferInfo.size = bufferCreateInfo.size; //bufferCreateInfo is an external struct passed into this function

VmaAllocationCreateInfo allocInfo = {};

bufferInfo.usage = VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT; //uniform buffer usage
allocInfo.requiredFlags = VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT;

VmaAllocationInfo allocationInfo = {};
VkBuffer buffer;
VmaAllocation allocation;

VkResult res = vmaCreateBuffer(this->allocator, &bufferInfo, &allocInfo, &buffer, &allocation, &allocationInfo);

(memory mapping happens later with vmaMapMemory)
The thing is that no matter if i set the "requiredFlags" as
allocInfo.requiredFlags = VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT;
or
allocInfo.requiredFlags = VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT | VK_MEMORY_PROPERTY_HOST_COHERENT_BIT;
the memory allocated via VMA always ends up with the "HOST_COHERENT" bit enabled. (Which i wasn't able to figure out/trace through the source code. From my understanding, only enabling the "VK_MEMORY_PROPERTY_HOST_VISIBLE_BIT" should not enable HOST_COHERENT as this is a seperate flag altogether.)
I validated this by checking the buffer with:
VkMemoryPropertyFlags memPropFlags;
vmaGetAllocationMemoryProperties(allocator, allocation, &memPropFlags);
if ((memPropFlags & VK_MEMORY_PROPERTY_HOST_COHERENT_BIT) > 0)
{
	assert(false);
}

as well as the general behaviour of the Data passed in to the GPU. ( even without flushing the mapped memory of this buffer with vmaFlushAllocation, the data is visible for the GPU without any issues/flushing/cach invalidation being needed.)
Checked the Buffer in Renderdoc too and host coherent is also defined there.
Is this intended behaviour? (Or what is more likely an error or my part?) Are there some specific circumstances where HOST_COHERENT is automatically enabled? (like with the VK_BUFFER_USAGE_UNIFORM_BUFFER_BIT?)
Testing this on an Nvidia Geforce GTX 1060 (6GB)

Do you actually have memory types which are host visible but not coherent? On my RTX3060M on Linux there are no such memory types. Everything that is host visible is also coherent.

If you want to find the logic for selection of the memory type, check function FindMemoryPreferences: https://github.com/GPUOpen-LibrariesAndSDKs/VulkanMemoryAllocator/blob/master/include/vk_mem_alloc.h#L3653 HOST_COHERENT flag is not required there.
You are right that a memory type without this flag, if one is available, may offer better performance. However, it seems that currently all 3 PC GPU vendors (AMD, NVIDIA, Intel) expose all memory types that are HOST_VISIBLE as also having HOST_COHERENT flag set, at least on Windows. Please use vulkaninfo command-line tool to inspect memory types available on your platform and see if this is still true.

Oh, so the combination of HOST_VISIBLE and HOST_COHORENT has to be a specific memory type that is supported by the Driver? I thought that HOST_COHERENT is optional (as in if it's supported you can enable it to avoid manually flushing the memory but don't have to.)
FindMemoryPreferences was one of the functions i was stepping through in my last debugging session but wasn't able to catch HOST_COHERENT anywhere. (Though i might've overlooked something there. Have to check again.) I assume the driver isn't automatically flagging the HOST_VISIBLE allocation as HOST_COHERENT after the fact?
 I thought that HOST_COHERENT is optional (as in if it's supported you can enable it to avoid manually flushing the memory but don't have to.)

This is correct. Please read more about device memory in Vulkan - chapter 11.2 of the specification:
https://registry.khronos.org/vulkan/specs/1.3-extensions/html/chap11.html#memory-device
Unlike in Direct3D 12, Vulkan has a flexibility in memory types that a particular Physical Device exposes. When allocating memory, you need to choose one of the available memory types. They have some flags that describe their capabilities and performance. These are not flags that you enable. You can also check this article:
https://asawicki.info/news_1740_vulkan_memory_types_on_pc_and_how_to_use_them
As you can see on the listing you quoted, on your GPU all memory types that are HOST_VISIBLE are also HOST_COHERENT.
Automatic choice of the best memory type for your Image or Buffer is one of the main features of VMA. I recommend you to use VMA_MEMORY_USAGE_AUTO (plus VMA_ALLOCATION_CREATE_HOST_ACCESS_SEQUENTIAL_WRITE_BIT or VMA_ALLOCATION_CREATE_HOST_ACCESS_RANDOM_BIT, if you need to map it) and forget about all this complexity.

Alright, that clears it then. Should've done more research on this topic.
Will do as you suggested. Thank you.


⚠ Bofrost.de has degraded performance

In b66e7fe, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1690 ms

Resolved: Bofrost.de performance has improved in 22b5a93.


As nav for skips elements

When i use as nav for its skips one slide, nothing happens, then when i click again its scrolls to the next slide. Here is my code:
https://codesandbox.io/s/ko7rwj89r5

Fixed by this: https://github.com/akiran/react-slick/pull/1423/commits/0e348f665c22a13b5d285f283fb3b2369e8fc128

I was just about to open this issue. Thanks for posting the answer.

Can this issue now be closed? Something should be said in the docs about using at least 0.24.0 if you want to use nav for. I had the same issue and thought it was my code

@gcosgreave-us  U can use solution I posted earlier, it ain't implemented in library itself.


reduce recommended bcrypt cost to the lowest allowed value

Two objectives:

Reduce thundering-herd effects on server restart (a cost of 4 should be
approximately 1 millisecond of CPU time per reconnecting client)
Speed up mobile reattach as much as possible (see also #1420)

notes mostly to self for future bcrypt-cost considerations:

since each session runs as a separate goroutine, would this let the bcrypting be split across the available cpu cores automagically?
bit of a slowdown in terms of thundering-herd reconnections is fine, but the sort of slowdowns we have right now is pretty crazy so this sort of change prolly makes sense.
if we do look at auth-cookies in future so that mobile clients can totally bypass the need for bcrypted weird stuff, look at how resume creates and distributes its similar thing because there's been a lot of discussion around that and it's pretty much the same concept, but shorter-lived than you might expect mobile clients' auth cookies to last.
at some point it may be worth looking at having a system where when the cost is changed here, every re-auth checks the cost of the stored hash and then re-hashes it with the new cost, so over time a server admin can (eventually, as everyone reconnects) reduce or increase costs across all user accounts automagically by just changing the cost in the config file. same as how we upgrade old creds to the new cred system automagically as we can.

notes mostly to self for future bcrypt-cost considerations:

since each session runs as a separate goroutine, would this let the bcrypting be split across the available cpu cores automagically?
bit of a slowdown in terms of thundering-herd reconnections is fine, but the sort of slowdowns we have right now is pretty crazy so this sort of change prolly makes sense.
if we do look at auth-cookies in future so that mobile clients can totally bypass the need for bcrypted weird stuff, look at how resume creates and distributes its similar thing because there's been a lot of discussion around that and it's pretty much the same concept, but shorter-lived than you might expect mobile clients' auth cookies to last.
at some point it may be worth looking at having a system where when the cost is changed here, every re-auth checks the cost of the stored hash and then re-hashes it with the new cost, so over time a server admin can (eventually, as everyone reconnects) reduce or increase costs across all user accounts automagically by just changing the cost in the config file. same as how we upgrade old creds to the new cred system automagically as we can.

Correct, every individual check is sequential but multiple checks are parallelizable.

Correct, every individual check is sequential but multiple checks are parallelizable.

The design subtleties in resume, relative to a normal webapp with cookies, come from the "account-less" design --- if we were issuing temporary passwords / cookies in the reattach context, we wouldn't have that problem because clients could supply their account names as part of the handshake. (I'm thinking it might be easiest to have them prepend a ! to their username to let us know they're trying a cookie, then use SASL PLAIN with !username as the username and <cookie> as the password. Then we could compare the cookies directly with subtle.ConstantTimeCompare.)

The design subtleties in resume, relative to a normal webapp with cookies, come from the "account-less" design --- if we were issuing temporary passwords / cookies in the reattach context, we wouldn't have that problem because clients could supply their account names as part of the handshake. (I'm thinking it might be easiest to have them prepend a ! to their username to let us know they're trying a cookie, then use SASL PLAIN with !username as the username and <cookie> as the password. Then we could compare the cookies directly with subtle.ConstantTimeCompare.)


Modification to TID handling and new tests to cover TID and timeout

We found an issue where sometimes receiving packets would fail following a packet timing out. This commit makes the TID handling more consistent - the rx state TID is no longer incremented when a packet is succesfully decoded/fails - previously it would be incremented to the "next expected" TID.
Tests have been added to cover various TID/timeout scenarios.

@mike7c2 would you be able to address the issues I described, or should I do that?

I wonder if we could also apply these changes to the uavcan-v1.0 branch?

Addressed in #142.


Aider not recognized as command in Windows cmd prompt

Hi,
I'm trying to follow the getting started instructions. After installing aider and setting my open ai key, I try to run aider hello.js
But, at this step, I get a message indicating that aider isn't recognized as a command.
I'm on a Windows machine.
Thanks!

I got the same issue, I already installed several times, assisted with ChatGPT4 and still cant make it run.

hey , did you run "pip install aider-chat" ? Try running that and see once you activate the conda environment you are working in.

Hi,
Yes, I ran "pip install aider-chat" each time I tried the installation process.  Thanks!

Thanks for trying aider!
Have you reviewed the installation instructions?
https://aider.chat/docs/install.html

Feel free to join us in the discord if you need help debugging your aider installation.
https://discord.com/channels/1131200896827654144/1133060264825208942
I'm going to close this issue for now, but feel free to reopen it or file another issue if there's more you would like to discuss.

I have the same issue. Followed the installation instructions, but I get the following error when I try to use it:
'aider' is not recognized as an internal or external command,
operable program or batch file.

@jazib450 are you sure you did pip install aider-chat not pip install aider

Yes

Unfortunately that means there is a problem with your local python environment. That's pretty tough for me to help debug.

I also face the same problem. This message appears when I try to find out which model I have
"λ aider --models openai/
'aider' is not recognized as an internal or external command,
operable program or batch file."

Did you follow the install instructions? In particular did you pip install aider-chat not just aider?
https://aider.chat/docs/install.html
You can try running python -m aider.main as well.

I have followed the steps
First, I opened Smdr and went to the file containing the project I wanted to work on and try Aider
then I run this command line
pip install aider-chat
The downloading process completed without any problems
Then I exported the API of Gemini Pro
using this command
setx OPEAN_API_KEY AIzaSyAVpkoVk6TvtMk1nOY4KsvPnNZcXTprG_4
SUCCESS: Specified value was saved.
Then you write this command line knowing which models are available to it and the API works correctly.
aider --models openai/
But the problem appears to me like this
'aider' is not recognized as an internal or external command,
operable program or batch file.

Looks like you did everything right. Something is wrong with your local python/shell environment.
Did you try python -m aider.main ?

If you don't mind, can you contact me via discord to make a call to try to get it working?
this is my name in the discord
jabersaed

You're welcome to join our discord:
https://discord.gg/Tv2uQnR88V
See the docs for info on using models:
https://aider.chat/docs/llms.html
https://aider.chat/docs/llms/gemini.html

I have tried every possible way to try to fix this error, but it still appears. I hope I tried to solve it

I run this command and it is install without any errors
python -m pip install aider-chat
but when i run this
python -m aider.main
that what i get
λ python -m aider.main
Newer version v0.39.0 is available. To upgrade, run:
C:\Users\User\AppData\Local\Microsoft\WindowsApps\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\python.exe -mpip install --upgrade aider-chat
Model gpt-4o: Missing these environment variables:

OPENAI_API_KEY
https://aider.chat/docs/llms/warnings.html
Model gpt-3.5-turbo: Missing these environment variables:
OPENAI_API_KEY
https://aider.chat/docs/llms/warnings.html
Aider v0.38.0
Models: gpt-4o with diff edit format, weak model gpt-3.5-turbo
Git repo: .git with 184 files
Repo-map: using 1024 tokens
Use /help to see in-chat commands, run with --help to see cmd line args

I would use it with gemini
but when i run this command
aider --model gemini/gemini-1.5-pro-latest
that what i get
λ aider --model gemini/gemini-1.5-pro-latest
'aider' is not recognized as an internal or external command,
operable program or batch file.

I don't know how this happened but I fixed it and it works fine
I'm glad I can now try it and explore it
For those facing such a problem, do the following
Update Pipe to the latest version:
python -m pip install --upgrade pip
In the process of installing Adar, use this line
py -m pip install aider-chat
instead of:
pip install aider-chat
To run it you can use this command
python -m aider.main --model gpt-3.5turbo --browser
You can only change the model as you wish by changing its name and do not forget to add the API for module
as ex
export OPENAI_API_KEY= # Mac/Linux
setx   OPENAI_API_KEY  # Windows
export GEMINI_API_KEY= # Mac/Linux
setx   GEMINI_API_KEY  # Windows
It was really tiring and stressful. It took me two days to try to run it, but it wouldn't work, but I'm glad I discovered and learned a lot of things.

I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.

Took a bit of fluffing about back-and-forth.
Also using Windows and VS Code, this is my final happy code
python -m aider.main --sonnet --browser --anthropic-api-key sk...

I would recommend adding or mentioning this python-m aider.main thing in the install documentation.

Good idea, I've updated the docs to mention python -m aider.
I'm going to close this issue for now, but feel free to add a comment here and I will re-open or file a new issue any time.


Use versioneer for automated versioning

This PR adds versioneer for automatically setting the correct library version based on the latest git tag.

if the latest commit also is referenced by a tag, the tag is taken as version number, i.e. 0.7.5.
if the latest commit has no tag, the main version will point to the latest tag, and then the number of commits after the tag and include the commit hash, i.e. 0.7.5+1.gb6a164d.
if there are unstaged changes in the repository, dirty will be added, i.e. 0.7.5+1.gb6a164d.dirty

After versioneer has been installed once, one does not need it as requirement. All necessary code is included in versioneer.py and bimmer_connected/_version.py.
Although not tested with Github Actions, it should work flawlessly as well. If executed on a repository with a tag on the last commit (like we do when creating releases), running python setup.py sdist bdist_wheel (such as in the Action) creates the correctly versioned archives.

Closing this in favor of #173 as pbr was used before and has less dependencies at runtime.


Remove or fix Display language

Change the display language don't work, can't get the extension in norwegian sadly KEKW

Thanks for reporting, already registered as #26.


Multiple square waveform with different frequencies?

I'd like to generate a couple of square waveforms with different frequencies on different pins(duty cycle is 50%).
e.g.

20kHz on GPIO26
30kHz on GPIO19

I am using the following code to generate, but the waveform on GPIO19 isn't 30kHz at all:
import time

import pigpio

pi = pigpio.pi()

conf = {        # pin: Hz
    26: 20000,
    19: 30000
}

for pin, freq in conf.items():
    pi.set_mode(pin, pigpio.OUTPUT)
    pulse_width = int(1. / freq * 1000000) // 2  # us
    actual_frequency = int(1000000 / pulse_width / 2)
    square = [
        pigpio.pulse(1 << pin, 0, pulse_width),
        pigpio.pulse(0, 1 << pin, pulse_width)
    ]
    pi.wave_add_generic(square)
    print(f"Waveform on GPIO{pin}, {freq}(actually {actual_frequency})Hz")

wave_id = pi.wave_create()
if wave_id >= 0:
    print(f"Wave ID: {wave_id}")
    pi.wave_send_repeat(wave_id)

while True:
    try:
        time.sleep(.1)
    except KeyboardInterrupt:
        break

if wave_id >= 0:
    pi.wave_tx_stop()
    pi.wave_delete(wave_id)

pi.stop()

Result:
Waveform on GPIO26, 20000(actually 20000)Hz
Waveform on GPIO19, 30000(actually 31250)Hz
Wave ID: 2

Waveform on GPIO19(should be 31.25kHz, but 20kHz actually):

Is it impossible to generate different waveforms on multiple pins?

Yes, if you repeat with a period that is an integral multiple of the two of the two periods you want to generate.  In this case try placing two cycles of 20KHz and three cycles of 30KHz within a wave repeating at 10 KHz.

Thanks for your reply, @guymcswain

Yes, if you repeat with a period that is an integral multiple of the two of the two periods you want to generate. In this case try placing two cycles of 20KHz and three cycles of 30KHz within a wave repeating at 10 KHz.

Could you share a code snippet to generate the 10kHz period?


All checks have failed

All checks have failed after merging the pull request.
I guessed it should be related with project settings.
refer to this ticket - https://discuss.circleci.com/t/git-github-com-permission-denied-publickey-on-repo-that-previously-worked/37763/4

If you have received<EMAIL_ADDRESS>Permission denied (publickey).” Please click “project settings” button, than “SSH keys”. Click to remove deploy key and than click “add deploy key” once again. Than you can click on “${project} deploy key”, it will navigate you to github. Your new deploy key should be visible on github project page.

After that checkout will succeed.

@squat Can you help on? Thanks.

Ack. I find it pretty bizarre that the key would suddenly become invalid. In any case, following these instructions fixed the issue; checking out code from circle CI is working once again and CI works :)
I think we would be well served moving to GitHub actions at some point to make the integration with CI better.

Thanks @squat for your help to make ci works.

Thanks @squat for your help to make ci works.


Fix typo

Closes: #434

open source/open-source is generally hyphenated for an adjective. So line 260/280 should probably be left as is.


request a new release

👋 it has been more than two years since last release, and it would be good to cut a new release (the HEAD build has some issue for me as in https://github.com/iheanyi/bandcamp-dl/issues/232).
Seeing some regression run failure for 0.0.15
/opt/homebrew/Cellar/bandcamp-dl/0.0.15_3/libexec/lib/python3.12/site-packages/bandcamp_dl/utils/unicode_slugify.py:84: SyntaxWarning: invalid escape sequence '\s'
  new = re.sub('[%s\s]+' % space_replacement, space_replacement, new)
Traceback (most recent call last):
  File "/opt/homebrew/Cellar/bandcamp-dl/0.0.15_3/bin/bandcamp-dl", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/opt/homebrew/Cellar/bandcamp-dl/0.0.15_3/libexec/lib/python3.12/site-packages/bandcamp_dl/__main__.py", line 76, in main
    config = init_config(arguments)
             ^^^^^^^^^^^^^^^^^^^^^^
  File "/opt/homebrew/Cellar/bandcamp-dl/0.0.15_3/libexec/lib/python3.12/site-packages/bandcamp_dl/utils/config.py", line 61, in init_config
    elif first_run is False:
         ^^^^^^^^^
UnboundLocalError: cannot access local variable 'first_run' where it is not associated with a value

relates to https://github.com/Homebrew/homebrew-core/pull/183094

Soon, it was completely broken for a while and recently went through some big changes. As of now it works and one of the most pressing issues is fixed so its usable but I want to comb through the issues and see if there is any low hanging fruit and ensure that the recent changes work for everyone (I don't have the time personally to test across all OSes)

Yay, my bandcamp-dl is fixed! Thanks guys, everyone who put work into this

FYI, looks like v0.0.16 is missing from PyPI due to the publish job failing: https://github.com/iheanyi/bandcamp-dl/actions/runs/10818854657/job/30015403909

FYI, looks like v0.0.16 is missing from PyPI due to the publish job failing: https://github.com/iheanyi/bandcamp-dl/actions/runs/10818854657/job/30015403909

It's never worked due to the fact that while I can create actions I can't edit repo settings to add secrets and such for credentials to push releases to PyPI as I'm not the owner just the maintainer.
I can manually do it though and will soon after fixing some minor things, in the mean time you can pip install the .whl file in the tagged release, just update urllib3 to the latest version after.

Done, current PyPI release has only one minor change over tagged release relating to urllib3 version to avoid dependency issues.


User-Agent is localized

Hi, I see that User-Agent in Manager https://github.com/Alamofire/Alamofire/blob/master/Source/Manager.swift#L64 is localized
because of this
let os = NSProcessInfo.processInfo().operatingSystemVersionString

The operating system version string is human readable, localized, and is appropriate for displaying to the user. This string is not appropriate for parsing.

is this good?

Great question @onmyway133! We'll have to do some investigation here and get back to you.

cc @jshier, @AnthonyMDev, @kcharwood

Probably not a good idea, as Apple's docs explicitly call out not using the vale for parsing. Manually creating the version string from operatingSystemVersion seems like a better solution.

I agree @jshier. I'll work on putting together a fix.

We've discovered this as well in our own internal attempts to set headers w/ the current OS version. Has anyone had success actually getting NSProcessInfo.operatingSystemVersionString to output a string with a different locale?  I've tried using Xcode scheme language options and the launch arguments (-AppleLocale and -AppleLanguages) to no avail.  Of course I also tried changing my Mac and iOS simulator language & region settings with no luck.

This is my temp solution
public static let osInfo: (version: String, build: String) = {
    let version = UIDevice.currentDevice().systemVersion

    var build = ""
    let os = NSProcessInfo.processInfo().operatingSystemVersionString
    let regex = try? NSRegularExpression(pattern: "\\(.*\\)", options: [])
    if let range = regex?.firstMatchInString(os, options: [], range: NSRange(location: 0, length: os.characters.count))?.range {
      let parts = (os as NSString).substringWithRange(range)
        .replace("(", with: "")
        .replace(")", with: "")
        .split(" ")

      if parts.count == 2 {
        build = parts[1]
      }
    }

    return (version: version, build: build)
  }()

I'm going to close this issue out since @jshier has put together a fix in #1292. Please redirect all future comments to #1292.
Cheers. 🍻


wrong examples for allOf in arrays

What happens?
Currently AllOf schemas get listed as separate array items in the request body example.
Similar existing issue (fixed) but for responses https://github.com/scalar/scalar/issues/2137

What did you expect to happen?
AllOf schema properties are combined in request body examples for arrays
OpenAPI Document
https://sandbox.scalar.com/e/44Rod

Fixed in #3655


Port to F411 ?

Hi,
You've mentioned in our previous conversation that you were planning to port to the Black Pill. Any idea when that might happen?
I'm starting to run out of flash space with -O0 flag, which forced me to switch to -Og, which in turn makes debugging much more difficult. I've tried switching over to F411 headers, but it seems you've made some modifications to the blue pill headers so the code doesn't readily translate.

Cool, I'm really curious what lib you end up with. Is it your own creation you've mentioned before? Hopefully it'll have some HAL-like macros to reduce the need for bit-banging somewhat. Also, I took a look at Kvasir git, it's been dead since 2017...
As for debugging, I'm just trying to debug UI code, nothing time-critical. It's just frustrating without -O0, half the lines and vars are optimized-out. I found a workaround for now, I just disable the code that it not relevant to what I'm debugging and I get just enough room. F411 would solve that properly, but for now it's working well enough.
And I took your advice and ordered an Owon scope (I know you suggested a logic analyzer, but this looked more fun). Should be interesting to play with.

An unrelated question: how do you envision switching ratios on the fly would work? I'm trying to implement it in feed mode, where I don't really care if I miss steps, but simply calling gear::configure() seems to confuse the algorithm and it stops step generation for a while. I tried adding encoder::update_channels() right after, and even tried disabling tim1 while updating all that, but it didn't seem to help much.

Hi. First off, despite my intentions I unfortunately didn't have any time work on the project this week so no updates on the code yet.
I am glad you got a scope :)
Yeah, Kvasir is dead, that's one of the motivations. Imho the API is pretty good but there are some things that need improving. Especially on the code generator part to compensate for the low quality SVD files from vendors like ST.
This is indeed a low level API so I get the need for a higher API but for certain applications (like this where we want to use the hardware to it's fullest) this is good enough. It's definitely better then the error prone manual bit manipulations to read/write the registers.

An unrelated question: how do you envision switching ratios on the fly would work? I'm trying to implement it in feed mode, where I don't really care if I miss steps, but simply calling gear::configure() seems to confuse the algorithm and it stops step generation for a while. I tried adding encoder::update_channels() right after, and even tried disabling tim1 while updating all that, but it didn't seem to help much.

I am surprised. I recall that part working without any issues in my tests (should be on the video as well?). Are you sure the pulse ratio you calculate is valid? I will make sure to test this once I get back into it.

I think I've fixed the on-the-fly gear change issue, at least somewhat. Instead of changing gear::state immediately when a different ration is selected I save it into a temp var and apply it on the next interrupt. That way any potential timing issues are avoided (perhaps an interrupt between encoder::get_count() and actual refresh of gear::state, or between assignments to state::N , D and err).
Changes work fairly smoothly now, but there's still some sort of a timing glitch. Once in a while when changing on the fly the stepper would just stop completely and it would take a few seconds and a couple of gear changes to revive. I'll try to dig in some more into it and I'll post a video of it happening tomorrow.
And, of course, one new issue, this one more of a limitation: I found that with a direct-drive stepper and 4096ppr encoder, the stepper is really limited to 8x micro-stepping. At 16x it wouldn't be able to do even a 10tpi thread. Do you think it's possible to extend the algorithm to generate more than one step per encoder pulse? Maybe interpolate based on current speed? It wouldn't be exact, but should still be fairly smooth.
Also, I've created a fork to keep my changes somewhat organized (https://github.com/akunadze/Didge), in case you'd like to see. Most of them are UI-related, but there's some control flow reordering that you might find useful.

Found the cause of the issue: step_gen::set_delay(phase_delay(encoder_pulse_duration::last_duration(), range.next.error));
I'm using a rickety old dc motor to drive the encoder for testing and its RPM is not very stable. I think what happens is the delayed steps collide with next steps in sequence, especially in the face of ratios changing on the fly. I removed the delay and it's much more reliable, though obviously the steps are not as smooth.
I also ran into resource exhaustion while trying out 16x microstepping. Somewhere around 0.040in/rev feed and 700rpm UI stopped updating and steps began to falter. Upping the RPM resulted in the stall happening at lower ratios. Black pill would really be nice here...

Hi there. I've been busy with the library that I've mentioned so could not respond sooner :(
I think turning off the optimizations might be culprit here. With libraries like Kvasir, the optimizer is essential to eliminate the 'fluff'. Interrupt service routine is sensitive to the runtime / code size. If the next jump comes while we are still in the ISR the main thread will never have time to do anything for instance or lower pri interrupts will not be handled. Please give -Os a shot.
Re: gear changes - I think it's a good idea to have an intermediate storage for the gear status like you mention. You may still run into the case where you are midway through the temp struct update and the ISR gets executed. This needs an atomic operation to do it without any glitches. This is on my must have list.
Re: tpi limitation. Well hopefully with the switch to the black pill, there will be more 3 timers to work with so yes I think it's definitely possible.

Yes, I realized I had -O0 flag on and since tried -O2 and -Os. It did improve things, I was able to get to 0.1in/rev feed, but there was still an occasional glitch at lower feeds. Looking at it in the scope I noticed that sometimes the step signal remains high for a long time, and sometimes stays long for a while. Could it be that something is masking the interrupts? According to the docs, the timer needs them to work reliably. Either way, there's definitely a timing issue there somewhere, it's just easier to hit with optimization turned off.

@akunadze  Sorry not related but are you by any chance Alex Kunadze who developed CuteFTP? 😄

@WildPoYo Haha, yes that's me. I'm surprised anyone remembers that one, it's been a few decades :)

@akunadze It was the first ever FTP tool I used back in my high school times. Which is almost two decades ago :)

@prototypicall , any news on the port, and perhaps on moving off Kvasir?
I've been trimming down/refactoring Didge to get familiar with the system, and will try to migrate to direct register manipulation to remove the Kvasir dependency (mostly to make it easier to port/combine with additional logic), but if there's something already done in this space, I'd love to take advantage of that as well.
My plan is to use one STM32 to power the main synchronization, while operating as an I2C slave. A separate unit will be the master, and interface with any displays, pendants and so on.

@oyvindkinsey Sorry about the delay. I had to unfortunately put the 'moving off of Kvasir' to a long halt. Lost my SSD drive to a corruption last year with significant portion of the new library and some additional projects for generate SVD files from the ST reference manuals with it. Was really disappointing. Anyway, I am not sure when I will be able to get back to this project - hopefully in a not so distant future but don't know yet.

No worries, I already ported it to CMCIS  and have everything working.
Almost done with the i2c slave implementation now, where a master will be
able to read rpm, set the ratio, set limits, and initiate rapids etc.
Fun starter project for the stm32 :D
On Thu, Jan 5, 2023, 10:01 PM prototypicall @.***>
wrote:

@oyvindkinsey https://github.com/oyvindkinsey Sorry about the delay. I
had to unfortunately put the 'moving off of Kvasir' to a long halt. Lost my
SSD drive to a corruption last year with significant portion of the new
library and some additional projects for generate SVD files from the ST
reference manuals with it. Was really disappointing. Anyway, I am not sure
when I will be able to get back to this project - hopefully in a not so
distant future but don't know yet.
—
Reply to this email directly, view it on GitHub
https://github.com/prototypicall/Didge/issues/5#issuecomment-1373179912,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAA2MARRHKTDGEF4OIMRF33WQ6YNDANCNFSM4XS6EM5A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Does Influxdb support mutil-datadir?

A volume having little space that Influxdb store using, The follow-up data need to write to the other volume, Does Influxdb support multi-datadir?
Thanks.

@Tancen thanks for the issue. this is not supported. You would need to change the data-dir configuration to write to a new location and it wouldn't know about the old one.


Hubspot - Data recipes

Additional views and sample queries to the Hubspot data source

All look good in terms of syntax, some of these have potential to cause disk full errors when ran on large amounts of data. One example being deals_consolidated when ran on a hubspot_deals_properties table with 77 million rows and hubspot_deals with 1.2 million rows.


Crash when opening Preferences

Basic information

Your Linux distribution and its version: Ubuntu 18.04

Your desktop environment: Unity

Nuvola Runtime version: 4.14.6

The web app you use and its version: Google Calendar script 1.3.2

Describe the bug
When I click on Preferences in the setting menu, the app crashes.
Terminal output:
Runner: [CRITICAL Δ05s741856us DioriteGlib] drt_variant_utils_to_strv: assertion 'variant != NULL' failed
Runner: [CRITICAL Δ000108us DioriteGlib] Application will not function properly.
Runner: [CRITICAL Δ000061us DioriteGlib] drt_variant_utils_to_strv: assertion 'variant != NULL' failed
Runner: [CRITICAL Δ000048us DioriteGlib] Application will not function properly.
Worker: [DEBUG Δ04s438064us DioriteGlib] DuplexChannel.vala:502: Channel (0) has been closed.
Worker: [DEBUG Δ000269us DioriteGlib] DuplexChannel.vala:328: The channel has already been closed

...
Steps to reproduce the behavior

Open app
Click on menu button (upper right corner)
Select Preferences.

Just updated flatpaks; same issue
Nuvola Apps - Genuine flatpak build
Version 4.15.0
Revision 0-g06de715
Diorite 4.15.0
WebKitGTK 2.22.4
Chromium 70.0.3538.77
libsoup 2.64.2

Hello @sebawild. Thanks for reporting the issue. The fix is being built right now.

Could you test the latest stable build? It contains Nuvola 4.15.0 and Diorite 4.15.1.

I installed the update and the preferences show up now.
Thanks, that was amazingly fast!

Released with Nuvola 4.16.0, more info: Nuvola 4.16 Brings Libre.FM Scrobbling, Ivoox Service, and more


Identify testing frameworks

[x]  Meet with tech team to identify how to set up testing frameworks
[x] Identify language for testing framework
[x] Identify testing tools

Assignee:
Yefim

The testing framework will be written in the Java Language, using a Maven packager, TestNG as the testing engine, and Selenium/Rest Assured as the Automation Engine. Github Runners will be the remote VM to run the tests for the Pipeline.


precomputed annotation format related ID lookup

I'm wondering if there is a path forward to allow users to select segments based on interactions with pre computed annotations.
The present precomputed annotation format doesn't store the relationships with the annotation data, so it's hard for the UI to allow users to identify and select related segments based on interacting the the annotations. The related ID index allows for an efficient way to query all the annotations associated with a related segment, but not which related segments are associated with an annotation.  This presently works for local annotations.
The simplest solution I think would be to simply encode the relationships along with the other properties in a "v2" format.  I don't know if you have other/better ideas.  The present v1 approach is highly data duplicative, and has the advantage of being a fixed length of bytes per annotation and this would break that convention.

Yes, currently you have to make a separate request to the by_id index per annotation in order to retrieve the list of all related segments, because only the by_id index stores that information.  It would be reasonable to store the relationship data in the other indices if there were a use for it, but that would indeed be a format change.
Regarding a v2 format, there are a few thoughts I had on that:

It may make sense to use Parquet or similar arrow-related format for encoding each chunk rather than the custom binary format currently used, but I have not investigated that too much.  I don't think Parquet is particularly suitable for representing an entire index, but I could be mistaken.
It would be nice to allow indices to be defined on arbitrary (ordered) combinations of geometry, relationships, properties.
It would be nice if there were an existing database format that could be leveraged (i.e. designed to be read directly without a server over high-latency storage, can be written via batch process also without a server) but unfortunately I don't think there is.
OCDBT could be used in place of precomputed sharded format, as that would allow ordered indices over arbitrary strings rather than just hash indices over uint64 values.

I realized that part of my confusion was i had a bug in #522 which was not writing the by_id index right, so the related segments were not showing up for my layers.  I fixed that, so thank you for clarifying so i realized my bug.  Comments on a v2 format make sense and I whole heartedly agree.  We've been discussing what format to write old versions of materialized data from CAVE to and many of these same issues came up in that discussion.


Open low equal

Screener related to open low equal should be introduced
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
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.

@Saj250 - Please look for menu option X > 12 > 32 (Intraday Breakout/Breakdown setup). It already has Open=low setup.
See today's results, for example:


Delete compilation.tar.gz after it's been extracted

The pipeline agents started failing today due to disk space running out. This deletes the compilation.tar.gz after it's been extracted to clear up some space.

How big is the zip? Just wondering if this is small enough that we'll hit this again before long if we add some more dependencies (especially STS stuff)

It's 736MB, so it takes up a good chunk of the 10GB that's available on the agents


QR & Image printing very slow

I have:

[x] searched open and closed issues for duplicates

Hey guys,
I have an Epson TM88-III hooked up to a raspberry pi through serial connection (baudrate 9600).
Everything works fine, but the printing of images & QR codes (which are basically images in non-native mode) is very slow with me. Also, I can hear the printer printing the image line by line, with quite some time in between, which I hope is not expected behaviour.
At first I thought this might have to do with time needed for data transfer, but even a 300byte image (which should take <0,5 second to transfer @ 9600 baud) gets printed 'line by line' rather slowly.
I have made a short video how a QR code currently prints with me:
https://dl.dropboxusercontent.com/u/4651651/2017-08-15 16.05.26.mov
Is this expected behaviour? Is there any way to make QRs/images print faster/smoothly?
I am planning on implementing this in a production environment, and I know for sure the local employees will go mad within a day if every receipt is slowed down by the QR printing :)
(PS1: Barcodes are printing quickly & smootly, as they are produced on the printer itself. I have tried setting the flag native=True with QR generation, which does not seem to work.)
(PS2: I also found this thread which describes the same problem, just with a different library. Any thoughts on this? https://github.com/Simonefardella/escposprinter/issues/4)

So, after some thorough testing I did get a bit further.
It seems that printing QR/images goes alright if I make sure the image is a simple bitmap (no compression or greyscales), and no bigger than approximately 100*100 pixels.
Then if I use high_density_vertical/horizontal=False the image comes out at reasonable size, at OK speed (@9600 baud)

Hi,
which version of the library are you using? We have sped up the processing in the past, but you probably already have that version.
Depending on your processing power it might take a short moment. To time this better you could use the dummy-printer to print into memory. Then you definitely can rule out the transfer.
I haven't tested the library recently on a system in the league of Raspis so I am not sure what is normal, but we had a thread once where iirc @mike42 timed the execution of the single parts of the image-printing

Hi Patrick,
Thanks for the reply!
So I've tried the preprocessing with the dummy printer, but this actually doesn't help anything.
It seems now the problem has ceased now I switched to pure 1-bit bitmap images (no greyscale!) which are no larger than 100*100 pixels.
Thanks though!

OK, feel free to write again, if you have any further issues.


Service Section

I've fixed the Issue.
It's now responsive.
992 px > 3 Columns
768 px > 2 columns
576 px < 1 column
Also, I added a new hover effect on class .responsive.

@AliReza1083 Your branch has merge conflicts. Kindly fix them.
__
  Summary: :hammer_and_wrench: This code needs a fix  |  Tags: Not reusable

@Susmita-Dey
is it fixed?

@Susmita-Dey is it fixed?

Yes
__
  Summary: :question: I have a question

will it be acceptable?

@AliReza1083 Issue no?
I can't give you points without this
__
  Summary: :question: I have a question

Sorry, it's my first time collaborating here, and I don't know what do you mean with issue no?

 Link it here in comment.
__
  Tags: Reusable

Issue #250


TODO list

[ ] Hyperlink doesn’t work in inside of  tags

https://dev-onejun.github.com/about

339158f

the link, my_github, is not working.

Previous TODO list
Because I will manage the TODO list via Issue, I append the previous TODO list history in README.md
https://github.com/dev-onejun/dev-onejun.github.io/commits/master/README.md

[ ] delete Study/Github Study Repo Category
[ ] add Review/Conference Category

[ ] write about lablup conference and DDC 2023

[ ] merge Project Category into Review/Project

[ ] change 404.html with more description

"The page you want may move to another category. Please use the search function in this blog"

[ ] update sitemap.xml due to https://support.google.com/webmasters/answer/7440203#soft_404.

it reports automatically as Google by e-mail


The tool fails for wasmer and doesn't give much information

reproduce as follows:
whubbs@whubbs1 ~ $ git clone git@github.com:wasmerio/wasmer
Cloning into 'wasmer'...
remote: Enumerating objects: 22, done.
remote: Counting objects: 100% (22/22), done.
remote: Compressing objects: 100% (22/22), done.
remote: Total 70416 (delta 5), reused 6 (delta 0), pack-reused 70394
Receiving objects: 100% (70416/70416), 57.83 MiB | 7.92 MiB/s, done.
Resolving deltas: 100% (56455/56455), done.
whubbs@whubbs1 ~ $ cd wasmer
/home/whubbs/wasmer
whubbs@whubbs1 wasmer $ git checkout 0.11.0
...
Note: switching to '0.11.0'.

...

HEAD is now at 8f50dab0f Trying to improve release process
whubbs@whubbs1 wasmer $ cargo-ebuild ebuild
failed to parse manifest at `/home/whubbs/wasmer/Cargo.toml`
whubbs@whubbs1 wasmer $

Any help would be appreciated.
Thanks,
William

That's actually an internal error coming from Cargo itself. This crate uses an older version of Cargo and it's likely related to that. I'm in the process of switching a lot of usage over to cargo_toml and cargo_metadata. I've been burned before by cargo_metadata removing data necessary but I'll attempt to trust it again.

So I've created a branch https://github.com/cardoe/cargo-ebuild/tree/change-up-the-parse which might do a better job. Any chance you can give it a whirl? I'm on slow, spotty internet until after the new year so I cannot test until then.

So I gave the branch a try because I faced a similar problem and it does produce an ebuild.
Haven't tested if the wasmer build would compile but it did create a valid ebuild.


[VS Code] Failed to create a collaboration session. An error occurred while sending the request.

@fanyikoukouchi Thanks for reporting the issue. We could use a bit more information to help you out.

What OS are you running?  If Linux, what distro?
Could you run the "Live Share: Export Logs" command and either drag-and-drop them here or email them to<EMAIL_ADDRESS>


Update configuration.md

Without a specific package the sample doesn't compile. The solution is only found in one of the comments. Cost me one hour too so I assume it won't be the last time hits this.

@sebastienros,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@sebastienros  sorry about that. Removed obtrusive NOTE and rewrote as
Note: This extension method requires the Microsoft.Extensions.Options.ConfigurationExtensions NuGet package.

Nothing to be sorry about. Thank you for keeping it consistent.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Rick Anderson<EMAIL_ADDRESS>Sent: Saturday, May 6, 2017 11:43:45 AM
To: aspnet/Docs
Cc: Sebastien Ros; Mention
Subject: Re: [aspnet/Docs] Update configuration.md (#3307)
@sebastienroshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fsebastienros&data=02|01|sebastien.ros%40microsoft.com|49e44530e0cf4952bbf108d494afd4e7|72f988bf86f141af91ab2d7cd011db47|1|0|636296930298166836&sdata=BtF%2FOjbCd72P1oKX0Q6Hr7lKLcALUvC%2B6T%2BWRF1ZJJc%3D&reserved=0 sorry about that. Removed obtrusive NOTE and rewrote as
Note: This extension method requires the Microsoft.Extensions.Options.ConfigurationExtensions NuGet package.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Faspnet%2FDocs%2Fpull%2F3307%23issuecomment-299658564&data=02|01|sebastien.ros%40microsoft.com|49e44530e0cf4952bbf108d494afd4e7|72f988bf86f141af91ab2d7cd011db47|1|0|636296930298176849&sdata=s5OaLb13eQRBO5LcuMTHMKlHVV5L95UXuhoNVf%2BXee4%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABHJ7abgjXWZGULp1uzKO9A_VJH1f_gCks5r3L9hgaJpZM4NRDhM&data=02|01|sebastien.ros%40microsoft.com|49e44530e0cf4952bbf108d494afd4e7|72f988bf86f141af91ab2d7cd011db47|1|0|636296930298176849&sdata=Q6wkegO7uR%2FwRPyKZa4LE%2FlvMBlgag%2FDXKZAgFTrfYg%3D&reserved=0.


Minimum Subsequence in Non-Increasing Order | C++

Minimum Subsequence in Non-Increasing Order -
what will change -
This program will obtain a subsequence of a minimum size of an array whose sum of elements is strictly greater than the sum of the non-included elements.
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code

Programming Language
Please add/delete options that are not relevant.

[x] C++

Self Check

Ask for issue assignment before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Happy Coding!

Please assign this issue to me.


Make db init command independent of container ID

docker-composer up -d names the koel container something like docker-koel_koel_1_f9319c3876b6. To the db init command work for all setups, I changed it to use docker-composer directly, where docker-compose will figure out the actual container id.

Didn't know this was a thing. Thanks!


Add java args to 2.0.2

Signed-off-by: Sayali Gaikawad<EMAIL_ADDRESS>Description
Even though we are not releasing this version Dashboards branch is failing due to lack of OpenSearch snapshot.
https://build.ci.opensearch.org/job/distribution-build-opensearch/6054/consoleFull
Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@gaiksaya can you also add to others like 2.2.1?


App does not list user's collections in create/admin

I helped someone I know who is an artist / graphic designer setup a Kusama account that has the following public key SS58 address on Kusama network D7xZ4QxGmtim6CP1XKxpFhbaCbuUJLg6TCPmTuFwk34a47S for a friend who is an artist / graphic designer and they added some KSM to pay transaction fees.
They then successfully created a collection at kodadot.xyz by using the menu "Create > Classic > Collection" approach.
The collection appears successfully when they go to menu "Explore > Collections", at this link https://kodadot.xyz/rmrk/collection/182147F2EAE72A963B-LART
But there are issues:

If they go to kodadot.xyz/rmrk/admin (Admin panel) with that same account loaded using Polkadot.js Extension, it says "You have no collections".
If they try to add an NFT to that collection with their account already loaded with the Polkadot.js extension, by going to kodadot.xyz/rmrk/create/ with menu "Create > Classic > NFT", and selecting the "NFT" tab, it shows their account is loaded, and then they click "Select a collection", but instead of showing their collection so they can select it, it just displays "--" in the dropdown menu.

Why isn't it associating the new collection that was created successfully with their account so they can create NFTs for that collection and view it in their profile? Why does it only show that their account is associated with that collection when you view it in the Kodadot explorer?
Note that they are using the latest Brave browser and the latest version of the Polkadot.js Extension, as both were installed on 3rd April 2022.

Thanks for the issue!
Both cases are using the same query, so the problem would be somewhere there.
It is also possible that subquery is timing out these requests


Add geolocalizer cloud function version 1

add geolocalizer cloud function based on Sasan code
fix some bugs
set environment variable to read API_KEY
How to test function
Go to function geolocalizer_test ->  test function -> input following request

{
"uri":"https://raw.githubusercontent.com/spatial-computing/map-ocr-ground-truth/master/maps/1920-1.png"
}

@sasantavakkol @yaoyichi FYI, btw, seems I can not add reviewers


Link Insert Incorrect Placement

I was testing this and found when selecting a word in the content area to convert into a hyper link, the word is copied and inserted again at the beginning of the content area and turned into a link instead of converting the pre-selected word as a link. This also happens when trying to insert an image in a specific spot, it goes straight to the start.
e.g if you try and make 'fox' a hyper link

The quick brown fox jumps over the lazy dog

it becomes

fox The quick brown fox jumps over the lazy dog

Thanks for reporting this.
However, this seems to be more of an issue with the rich text editor. What version where you testing?

Thank you for your reply, I have to apologise, now that you mention the version, I realised I was testing with an older version 4.0.16, but now that I've switched to the latest 4.4.1, its all working fine.
I did run the chrome browser debugger on the tinymce 4.0.16 whilst trying to establish what was going wrong and it seems that during the link generation, tinymce repaints the screen and deselects the chosen word, which caused it to fail the link insert.
This is a really good plugin and a definite improvement upon the original inspiration grid manager.
Thanks again


[3rd Party Feeds] Card Assignment Flow

Part of the Third Party Workspace Feeds project
Main issue: https://github.com/Expensify/Expensify/issues/394008
Doc section: Doc Link
Project:
Feature Description

[ ] Card Assignment Flow  (@koko57)
[ ] Card List (@koko57)
[ ] Card Details (@narefyev91)

I'm working with this Robert on this project, but I don't need to be assigned to every frontend new feature issue, as the work here is just regular BZ management. Going to revert back to the chosen auto-assignee!

@jliexpensify , as an fyi @narefyev91 and @koko57 are working on this, and as soon as they comment, you'll be able to assign them! Setting to daily as this is deliverable by September 9th, 2024

I'm Nicolay from Callstack - will work on this issue

I will work on the Card Assignment Flow and Card List

Thanks, assigned both of you

I'm starting working on Card Assignement flow, tomorrow I'll be focusing mainly on this issue.

Hi! I'm going to help @koko57 with Card List implementation here

Still working on Card Assigment Flow, I'm halfway through the process.

TY for the update!

Updates: Continue to work on the Cards List page implementation. I think I should be able to open the PR tomorrow if nothing unexpected comes up.

Assigned @allroundexperts as they're going to review PRs here

Most of the work is done for Card Assignement Flow, but I still have a few things to do there. I'm not sure if I'll be able to open the PR today, but I'll do my best to open it no later than tomorrow.

Updates:

I've prepared the Draft PR and now preparing screenshots. I've noticed some UI issues on iOS, and I'm going to fix it tomorrow morning.

The Company Cards List PR is ready for the review

PR for Card Assignment Flow is opened https://github.com/Expensify/App/pull/48278 although I need to test it on the mobile devices. I think it would be good to merge https://github.com/Expensify/App/pull/48176 first

@allgandalf can you add you Upworks name to your GH profile please?

@joekaufmanexpensify @yuwenmemon can you confirm these amounts are correct for PR reviews?

C+: @allgandalf $250
C+: @allroundexperts $250 (ND payment)

Yeah, I think that is the correct payments for the issue. We should issue them after all PRs on the issue are merged, and have been in production for 7 days.

It looks like we might be waiting on https://github.com/Expensify/App/pull/48278 to hit Prod.
Just a heads up - I am OOO from the 3rd to 14th. I'll keep this assigned to myself though, I don't think any payments will need to be done during this period.

Upworks job and Payment Summary here

Invited @allgandalf

@allgandalf - is a checklist needed?

Paid and job closed!

$250 approved for @allroundexperts


Varlamore Patch

This PR is intended on adding most Varlamore additions such as world map additions, agility shortcuts, regions for Discord rich presence, farming tracking and item stats.
I hope this is not too many additions at once. Please let me know if I missed something or amendments are preferred, e.g. squashing commits.
Leaving a note:
The dungeons inside Cam Torum have the same region IDs as the city, presumably they are just on different planes. The current implementation for Discord regions does not allow duplicate region IDs, so the dungeons were omitted.

Omitted regions
  DUNGEON_NEYPOTZLI_ANCIENT_PRISON("Neypotzli Ancient Prison", DiscordAreaType.DUNGEONS, 5525),
  DUNGEON_NEYPOTZLI_ANCIENT_SHRINE("Neypotzli Ancient Shrine", DiscordAreaType.DUNGEONS, 6037),
  DUNGEON_NEYPOTZLI_ANTECHAMBER("Neypotzli Antechamber", DiscordAreaType.DUNGEONS, 5781),
  DUNGEON_NEYPOTZLI_EARTHBOUND_CAVERN("Neypotzli Earthbound Cavern", DiscordAreaType.DUNGEONS, 5527),
  DUNGEON_NEYPOTZLI_STREAMBOUND_CAVERN("Neypotzli Streambound Cavern", DiscordAreaType.DUNGEONS, 6039),
  DUNGEON_NEYPOTZLI_THE_BLOOD_MOON("Neypotzli The Blood Moon", DiscordAreaType.DUNGEONS, 5526),
  DUNGEON_NEYPOTZLI_THE_BLUE_MOON("Neypotzli The Blue Moon", DiscordAreaType.DUNGEONS, 5783),
  DUNGEON_NEYPOTZLI_THE_ECLIPSE_MOON("Neypotzli The Eclipse Moon", DiscordAreaType.DUNGEONS, 6038),

I've cherry-picked (with fixes and adaptations where needed) many of these commits, though there are still a few I would like to review--mostly because I haven't had time to actually interact with the new content and unlock access to all the new areas so can't confirm with certainty every data point--from this PR, namely:

c3b5778 (#17588) (discord regions)
33842fb (#17588) (moth item stats)

The dungeons inside Cam Torum have the same region IDs as the city, presumably they are just on different planes. The current implementation for Discord regions does not allow duplicate region IDs, so the dungeons were omitted.

This does not seem to be true at all. The room containing the monolith appears to be region 22, 150 (5782), and the others similarly are not overlapping the city. The world map location dev tool is not always accurate--particularly for dungeon locations--so the Location dev tool, or shell, should be used to verify them instead.

Discord areas have been added in 6b793fc0e5095c7f63eeeda4ae091d691e23c7b3, and the item stat changes are being reviewed separately in #17607, so there is nothing more to be reviewed from this PR. Thanks for the help with all these changes.


ToPrimitive bound on Iterator impl for Range and related traits.

See this line which references rust-lang/rust#10414, now closed. The modern Range trait in the stdlib uses an unstable Step trait to generate its size_hint; perhaps something similar would work for this crate?

Well, core can get away with that because they only support iterating Range with the primitive integer types, which can privately implement Step. For num-iter, we would need a public (stable) trait that external users can implement to make this Range work for their types.

Yes, agreed, it would need to be stable. But that seems more flexible than requiring ToPrimitive.


test(tags): add tests for tags

Current coverage is 10.31% (diff: 100%)

Merging #197 into master will increase coverage by 1.47%

@@            master       #197   diff @@
=========================================
  Files           49         49
  Lines         3460       3460
  Methods          0          0
  Messages         0          0
  Branches         0          0
=========================================
+ Hits           306        357    +51
+ Misses        3115       3057    -58
- Partials        39         46     +7

Powered by Codecov. Last update 272b1cd...1e97965


PVS-Studio: fixed vulnerabilities

V3095 The 'searchLocation' object was used before it was verified against null. Check lines: 170, 178. Microsoft.Build.Tasks Resolver.cs 170
V3095 The 'searchLocation' object was used before it was verified against null. Check lines: 249, 264. Microsoft.Build.Tasks Resolver.cs 249
V3095 The 'assemblyName' object was used before it was verified against null. Check lines: 176, 194. Microsoft.Build.Tasks Resolver.cs 176
V3095 The 'e' object was used before it was verified against null. Check lines: 165, 170. MSBuild InitializationException.cs 165
V3095  ->  CWE-476 (Medium) NULL Pointer Dereference

@dotnet-bot test Ubuntu16.04 Build for CoreCLR please


8x super-resolution reconstruction

Hello, I need to conduct a comparative experiment and hope to perform 8x super-resolution reconstruction.
What modifications do I need to make to ensure smooth operation?
I will cite your literature.

Sorry for the late reply and thank you for your interest.
If you need my 8x model, let me know. I will send you and let you know some code modification.

Thank you very much, I really need your 8x model and hope to get your support.

I hope the code and the models for 8x below help you. These are unofficial codes.
https://www.dropbox.com/scl/fi/ttw1jn97t0ndccdtkhug8/FxSR-PD-OOE-UNet-F_VGG12345-L_BTMap_LPIPS_MSE-8x.zip?rlkey=p4zejymkw3p2bcgu7t3pcz1x9&dl=0

Thank you very much for your code. After I run the project, I will report any problems to you.
Thank you very much for your code. After I run the project, I will report any problems to you.
 黄大胜
@.***

------------------ 原始邮件 ------------------
发件人:                                                                                "seungho-snu/SROOE"                                                                                @.>;
发送时间: 2023年12月24日(星期天) 下午5:34
@.>;
抄送: "<EMAIL_ADDRESS>主题: Re: [seungho-snu/SROOE] 8x super-resolution reconstruction (Issue #16)
I hope the code and the models for 8x below help you. These are unofficial codes.
https://www.dropbox.com/scl/fi/ttw1jn97t0ndccdtkhug8/FxSR-PD-OOE-UNet-F_VGG12345-L_BTMap_LPIPS_MSE-8x.zip?rlkey=p4zejymkw3p2bcgu7t3pcz1x9&dl=0
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Receive Parts from Orders

I created an order with a part. I submitted that order and am now trying to receive that part.
I click the receive item, select the qty and stock location and click 'submit' but the dialog box remains. I get a: (I clicked submit 2 times)
[02/Jan/2020 01:29:30] "POST /order/purchase-order/1/receive HTTP/1.1" 200 2673
[02/Jan/2020 01:29:33] "POST /order/purchase-order/1/receive HTTP/1.1" 200 2564

from the server. When I close the dialog box and refresh the item has still not been received.

Interestingly I just tried to receive partial. So out of my 1000 parts on order I only received 100 and that registered.

I thought it might be an issue with the quantity. So I created tests with 100 and 10 parts and both were only able to receive total_qty-1

I just for some reason tried to receive more than was ordered and that also works. So with a PO of 1000 you can only receive:
>0
<1000
>1000

I have been scrubbing around the code but I have no idea where I should be looking.

@a7hybnj2 this is interesting! Can you capture the POST data being sent to the server (and the JSON response also)?
You can do this in the developer interface in your browser. e.g. in Firefox:

Yes, i am working on that now. i just got a new computer so gotta  install from scratch.

Okay.... I don't really know what I am doing but I think this is what you asked for.
{"JSON":{"form_valid":false,"success":"Items marked as received","title":"Receive Parts","html_form":"
\n\n\n\n\n\n
\n\n\n\n\n\n\nReceive outstanding parts for PO test-order - new order\n\n<form method='post' action='' class='js-modal-form' enctype='multipart/form-data'>\n    <input type=\"hidden\" name=\"csrfmiddlewaretoken\" value=\"AGIYZAfPNsVvSbRcfzCyZJAvuGb9ImLOl6h5PypEtStjldHpYlxFMfqdNqOigMOy\">\n    \n\n    Parts\n
Select parts to receive against this order.
\n\n
\n
\n            Part	\n            Order Code	\n            On Order	\n            Received	\n            Receive	\n
\n        \n
\n            \n            \n                \n\n
\n    \n        \n    \n
\n                Resistor\n            	\n            resistor-sku	\n            \n            1000.00000	\n            0.00000	\n            \n
\n
\n                        <input class='numberinput' type='number' min='0' value='1000.00000' name='line-1'/>\n
\n
\n            	\n            \n                <button class='btn btn-default btn-remove' onClick=\"removeOrderRowFromOrderWizard()\" id='del_item_1' title='Remove line' type='button'>\n                    \n                </button>\n            	\n
\n        \n
\n\n    \n\n<input type=\"hidden\" name=\"csrfmiddlewaretoken\" value=\"AGIYZAfPNsVvSbRcfzCyZJAvuGb9ImLOl6h5PypEtStjldHpYlxFMfqdNqOigMOy\">
 \n                Location*
 <select name=\"location\" class=\"select\" required id=\"id_location\"> <option value=\"\">---------</option> <option value=\"1\" selected> Junk Drawer - the junk drawer</option>\n\n</select>
Receive parts to this location

\n\n</form>\n\n\n\n\n"},"Response Payload":{"EDITOR_CONFIG":{"text":"{\"form_valid\": false, \"success\": \"Items marked as received\", \"title\": \"Receive Parts\", \"html_form\": \"
\\n\\n\\n\\n\\n\\n
\\n\\n\\n\\n\\n\\n\\nReceive outstanding parts for PO test-order - new order\\n\\n<form method='post' action='' class='js-modal-form' enctype='multipart/form-data'>\\n    <input type=\\\"hidden\\\" name=\\\"csrfmiddlewaretoken\\\" value=\\\"AGIYZAfPNsVvSbRcfzCyZJAvuGb9ImLOl6h5PypEtStjldHpYlxFMfqdNqOigMOy\\\">\\n    \\n\\n    Parts\\n
Select parts to receive against this order.
\\n\\n
\\n
\\n            Part	\\n            Order Code	\\n            On Order	\\n            Received	\\n            Receive	\\n
\\n        \\n
\\n            \\n            \\n                \\n\\n
\\n    \\n        \\n    \\n
\\n                Resistor\\n            	\\n            resistor-sku	\\n            \\n            1000.00000	\\n            0.00000	\\n            \\n
\\n
\\n                        <input class='numberinput' type='number' min='0' value='1000.00000' name='line-1'/>\\n
\\n
\\n            	\\n            \\n                <button class='btn btn-default btn-remove' onClick=\\\"removeOrderRowFromOrderWizard()\\\" id='del_item_1' title='Remove line' type='button'>\\n                    \\n                </button>\\n            	\\n
\\n        \\n
\\n\\n    \\n\\n<input type=\\\"hidden\\\" name=\\\"csrfmiddlewaretoken\\\" value=\\\"AGIYZAfPNsVvSbRcfzCyZJAvuGb9ImLOl6h5PypEtStjldHpYlxFMfqdNqOigMOy\\\">
 \\n                Location*
 <select name=\\\"location\\\" class=\\\"select\\\" required id=\\\"id_location\\\"> <option value=\\\"\\\">---------</option> <option value=\\\"1\\\" selected> Junk Drawer - the junk drawer</option>\\n\\n</select>
Receive parts to this location

\\n\\n</form>\\n\\n\\n\\n\\n\"}","mode":"application/json"}}}

If that isn't it let me know and I can try again.

I guess I could also zip my Inventree and upload that. I am just in a test instance currently. Is the database and everything stored within the inventree directory?

Ah, I have managed to recreate this. Fixed in https://github.com/inventree/InvenTree/pull/605

@SchrodingersGat YES! You are awesome! Like I said that was the issue that was preventing me from putting this into use at home.

Glad we could get it sorted out! Please keep the reviews / queries coming - it really helps get these issues fixed.


Does TensorRT support sequence_bias?

HuggingFace's generation supports an argument, sequence_bias, that allows you to up- or down-weight particular sequences of token IDs along with an argument renormalize_logits to renormalize the logins before sampling.
I am just wondering if a similar effect can be achieved with TensorRT? I don't see an exact mapping to these arguments -- apologies if I am missing them and thanks in advance!
Likely related to https://github.com/NVIDIA/TensorRT-LLM/pull/459 (however I don't see an implementation of SequenceBiasLogitsProcessor there. I could try to PR it when 459 is merged?)

TRT LLM does not support such feature. You could take a try and create PR.
Since TRT LLM is not a fully open source project, when you create PR, we will merge the changes back to internal repo first, then update to github and add you into the contribution list.