Embedded Run Keywords Unexpected Behavior

Try 1
If we run the follow robot, we could get:
Log  1
Log  2
Log  11
Log  22
And this is unexpected, "Log 1" and "Log 2" should not be logged, because "${condition}"!="yes"
*** Test Cases ***
Test Embedded Run Keywords
    Embedded Run Keywords

*** Keywords ***
Embedded Run Keywords
    [Arguments]                         ${condition}=no
     Run Keyword If                      0==0
    ...     Run Keywords
            ...     Run Keyword If      "${condition}"=="yes"
                    ...     Run Keywords
                            ...         Log  1
                            ...         AND  Log  2
            ...     AND  Log  11
            ...     AND  Log  22

Try 2
If we run the follow robot, we could get:
Log  2
Log  11
Log  22
And this is unexpected, "Log 2" should not be logged, because "${condition}"!="yes"
*** Test Cases ***
Test Embedded Run Keywords
    Embedded Run Keywords

*** Keywords ***
Embedded Run Keywords
    [Arguments]                         ${condition}=no
     Run Keyword If                      0==0
    ...     Run Keywords
            ...     Run Keyword If      "${condition}"=="yes"
                    ...     Run Keywords
                            ...         Log  1
                            ...         AND  Log  2
            ...     AND  Log  11
            ...     AND  Log  22

Try 3
If we run the follow robot with "${condition}"=="no", we could get error:
Test Embedded Run Keywords                                            | FAIL |
Keyword 'BuiltIn.Log' expected 1 to 5 arguments, got 0.
     Run Keywords
            ...     Run Keyword If      "${condition}"=="no"
                    ...     Run Keywords
                            ...         Log  1
                            ...         AND  Log  2
            ...     AND  Log  11
            ...     AND  Log  22

Everything seem to work as expected. AND always binds to the first Run Keywords. Indentation you use doesn't matter. To overcome this limitation, you need to create a user keyword where you use Run Keywords again.


is it wrong?

       // Sprite evaluation starts at the current OAM address and goes to the end of OAM (256 bytes)
        for (int i = _oamAddr; i < (256 - _oamAddr); i += 4)

hi!
can't understand this line
why (256 - _oamAddr)?
may be it should
for (int i = _oamAddr; i < 256; i += 4)

Hi there,
Yes, this appears to be a mistake on my part when I was implementing sprite evaluation. 13876b8816933d3c54f38560510cd62a997ddd31 fixes it.
Thanks for catching this bug!


Set Application Picture

Hi,I already set application name and now I wanted to know how to set application picture so it will output a rounded picture when using oauth2.
thanks.

You set the logo https://console.developer.google.com - go the APIs & Auth section then Consent screen, and you can enter and image URL there.


Reproducing PLDI'20 experimental results on two benchmarks

Dear authors, I am working on a research topic and I may need to use Regel in this new topic. I came across a intractable problem (may be my operation problems) when I attempted to call interactive.py on benchmark DeepRegex and SO for running experiments to check whether the actual results match the results shown in PLDI'20.
Specificly, after I prepared the required libraries of this repo, I entered:
python3 interactive.py --run_mode 1

Then I found that when Regel was synthesizing for the #62 and #50 (may be more) of benchmark DeepRegex, it reminded me to input two additional examples. According to my understanding, it means the examples already in the benchmark #62 and exp/interactive/deepregex/examples_cache/ are still not enough for synthesizing/distinguishing a final solution. I re-read the paper but found no more hints for it. So when running this controlled experiments, is it necessary for users to manually provide more examples so that one could finally obtain the overall results and compare it with experimental results in the PLDI'20 paper?
If it is convenient for you, please give a brief tutorial for guiding me (also other researchers) how to run the experiments correctly and quickly to get the experimental results in the PLDI'20 paper. Thank you very much!

In addition, if it is necessary for us to manually provide more examples in this interative mode to get the overall experimental results, could you provide the corresponding additional examples when you performed the experiments and then got the results in Fig. 14 (PLDI'20)? Thanks! If it is not convenient, I want to know if it is a heavy work to write these additional positive/negative examples.

In addition, if it is necessary for us to manually provide more examples in this interative mode to get the overall experimental results, could you provide the corresponding additional examples when you performed the experiments and then got the results in Fig. 14 (PLDI'20)? Thanks! If it is not convenient, I want to know if it is a heavy work to write these additional positive/negative examples.

I think now at least #62 and #50 should work correctly. Please see the most recent commits.

it means the examples already in benchmark #62 and exp/interactive/deepregex/examples_cache/
The examples in this folder are not guaranteed to be sufficient for the interactive mode to work without user inputs. I tried to keep the examples I used but it might not be the version that I used for the evaluation.

If it is convenient for you, please give a brief tutorial for guiding me (also other researchers) how to run the experiments correctly and quickly to get the experimental results in the PLDI'20 paper.
Unfortunately, I do not have the examples I used for producing Fig. 14. I would say that using whatever in the examples_cache should be the best try. Also, this code is the released version of the tool which might be different from the code I used for evaluation. The enumeration order might be different so you may need different examples. I haven't actually tried this version to reproduce the evaluation but I think it should be close.

I want to know if it is a heavy work to write these additional positive/negative examples.
It does require you to understand the DSL and interpret the natural language as intended and in some case some iteration of interaction. But overall imo it shouldn't be that hard.

I think now at least #62 and #50 should work correctly. Please see the most recent commits.

it means the examples already in benchmark #62 and exp/interactive/deepregex/examples_cache/
The examples in this folder are not guaranteed to be sufficient for the interactive mode to work without user inputs. I tried to keep the examples I used but it might not be the version that I used for the evaluation.

If it is convenient for you, please give a brief tutorial for guiding me (also other researchers) how to run the experiments correctly and quickly to get the experimental results in the PLDI'20 paper.
Unfortunately, I do not have the examples I used for producing Fig. 14. I would say that using whatever in the examples_cache should be the best try. Also, this code is the released version of the tool which might be different from the code I used for evaluation. The enumeration order might be different so you may need different examples. I haven't actually tried this version to reproduce the evaluation but I think it should be close.

I want to know if it is a heavy work to write these additional positive/negative examples.
It does require you to understand the DSL and interpret the natural language as intended and in some case some iteration of interaction. But overall imo it shouldn't be that hard.

Thank you for your answers, I will try again.

Thank you for your answers, I will try again.


clrDate does not override browser-native date picker

[x] bug
[ ] feature request
[ ] enhancement

Expected behavior

The date picker directive clrDate should show the Clarity date picker across supported browsers.
Actual behavior

According to v0.11 docs and component status the date picker component should be available. When I include an input with type date and the directive clrDate the native browser picker is shown, and the component does not render the Clarity version.
Reproduction of behavior

https://stackblitz.com/edit/clarity-light-theme-v11-5a1t3m

Environment details

Angular version: 5.0.0

Clarity version: 0.11

OS and version: Windows 10 Pro version 1709

Browser: Chrome 66.0.3359.181 | Firefox 60.0.1 | IE 11.431.16299.0 | Edge 41.16299.402.0

The date picker was introduced in 0.11.8, and your StackBlitz is using 0.11.1. If I update it to the latest 0.11, it works fine: https://stackblitz.com/edit/clarity-light-theme-v11-datepicker


Added DJ4E and CS50W courses in Education

Added Django for Everybody course, and Harvard's CS50W webdev course.

name: Add a new project
about: Suggest a new Django project for the Awesome Django list
title: "[NEW] Project Name"
labels: new project
assignees: ''
Project Information

Project Name:
Project URL:
Description: (Please provide a brief description of the project.)

Criteria
Please answer the following questions about the project you are submitting. This will help us evaluate if the project should be included in the Awesome Django list.

Is the project new?

[ ] Yes
[x] No

How long has the project been maintained?
(Please provide an approximate duration in months or years.)

How many releases has it had if it's a library or package?
(Please provide the number of releases, or link to the project's release history.)

Are you the author or are you submitting the project on behalf of a company?

[ ] I am the author
[ ] I am submitting on behalf of a company
[x] Other (please specify)

What makes it awesome?
(Please provide a brief explanation of why you believe this project is a valuable addition to the Awesome Django list.)
I've attended both courses, and I can see them as an invaluable resource for beginners in web development looking to learn Django.They cover not only Django, but also other topics and concepts helpful in webdev, that allow the students to have a complete view of what developing a web application in Django might look like.

Additional Information
Please provide any additional information that you believe is relevant to the project or its evaluation for inclusion in the Awesome Django list. This might include information about the project's documentation, test coverage, community support, or any unique features that set it apart from other Django projects.
(Please provide any relevant additional information here.)
Thank you for your submission!

Looks good! Thank you @pietrodeluca and @ulgens.


Please invite me to the GitHub Community Organization

Name
Arpit Mohapatra
Discord Username (if applicable)
Asuna#7535
Additional Context
Great community 🚀

Welcome to the Community!

Welcome to the Community!

thanks !!!


L.PM.setOptIn(true) causes tools to stop displaying

In HTML right after loading Geoman, I include .
This causes the Geoman tools to stop displaying.  Removing L.PM.setOptIn(true) fixes the issue.
Adding
map.on('pm:create', (e) => {
    e.layer.options.pmIgnore = false;
    L.PM.reInitLayer(e.layer);
});

before
map.pm.addControls({
  position: 'topleft',
});

does not fix the issue.  Am I missing something?
Also, it would be wonderful if it were possible to direct Geoman to ignore entire LayerGroups, not just single layers.
JSFiddle

Found the solution myself.  Clearly documented (of course...)
const map = L.map('map', { pmIgnore: false });
is required.


PCP: Feat accelerate load region

Which problem does this PR solve?
PCP #1830
What have you changed?

add regionsCache field in RegionSyncer
RegionSyncer cache regions in regionsCache when recieves SyncRegionResponse
add LoadRegionsCache func in RegionSyncer
call LoadRegionsCache when cluster loadClusterInfo

What are the type of the changes?

New feature : add follower region cache

How has this PR been tested?
unit test

Hi, @hrbeuyz24, Do you have any problems? You can ask us for a help.

Hi, @hrbeuyz24, Do you have any problems? You can ask us for help.

sorry, I need to prepare for the final exam recently.


Implement create_slice_grid

Fixes #67
Have to leave now, will finish this later today.
cc @martinschorb

Hey @martinschorb,
I have a first implementation now, but it is completely untested, and since there are many different code paths writing all tests and debugging will take some time and I can't do all of that today.
Can you please let me know for which source you need this exactly for the paper figure? Than I will make it work for that one today and cover the rest in the next few days.

Can you please let me know for which source you need this exactly for the paper figure?

that's tomo_38_hm. You could also leave out all transforms.
Alternatively we could do tomo38_lm including the fluorescence overlay. Then we would need the transforms from Figure2d.

that's tomo_38_hm. You could also leave out all transforms.
Alternatively we could do tomo38_lm including the fluorescence overlay. Then we would need the transforms from Figure2d.

Ok, I will give it a try after lunch and ping you once I have something.

Hey @martinschorb,
I have a first version that runs through and added a test view under slice-grid->slice-grid-tomo38 (on main directly).
The grid placement works, but the transformation I estimate seems to be much too small, and it shows each tomo at the same z position. I have to stop until 4 PM now, gonna have another look then.

Fyi @martinschorb I will probably merge this tomorrow without finishing the slice grid implementation, depending on whether we need it for the figure (might choose the continuous representation of the z-axis instead).
I still like the feature idea a lot and will implement it eventually, but I don't really have enough time right now to sit down and figure out all the transformation logic...
And I did a lot of refactoring here that I want to release asap because it significantly improves the package structure.


create new release

@open-homeautomation can you please create a new release soon?
We have a bit of dependency bug here:

miflora 0.5 depends on btlewrap>=0.0.4, so "at least version 0.0.4". Which was a bad idea in hindsight. We should have set it to == not >=.
In btlewrap 0.0.6 I changed the API a little bit, which triggers this bug: https://github.com/ChristianKuehnel/plantgateway/issues/40
So as everyone is now using the miflora 0.5 with the latest version of btlewrap, this combination breaks on the API change.
To fix that I just upgraded miflora to use btlewrap==0.0.8. So use exactly this version.

To resolve if for our users, we need to create a new miflora release. I'll then also create an update for HomeAssistant.
Can you please also push the version 0.5 tag to github?

Hello,
we have the same problem with the bt-mqtt-gateway. I look forward to your new maj. thank you for the work done.
Dans btlewrap 0.0.6, j'ai légèrement modifié l'API, ce qui a provoqué ce bogue: ChristianKuehnel / plantgateway # 40

Hello,
we have the same problem with the bt-mqtt-gateway. I look forward to your new maj. thank you for the work done.
Dans btlewrap 0.0.6, j'ai légèrement modifié l'API, ce qui a provoqué ce bogue: ChristianKuehnel / plantgateway # 40

Update bt-mqtt-gateway ok
https://github.com/zewelor/bt-mqtt-gateway/issues/110

0.6 now available in PyPi

Shouldn't have the 0.6 been tagged on the Git repo too?

@open-homeautomation can you please push your tags?


The menu tree on the left is really great. Can you open source it

The performance of this menu is very good. At the same time, it supports right-click menu and is beautiful enough. I really like it

This is an open source repository, so I am not sure what the issue is about.

Thanks for the praise, we put in a lot of effort into it. Unfortunately creating a component out of the list/tree view isn't in the nearby future for us. Sorry about that!


Add CMake build system (based on previous work from @mdsitton and @jcelerier) and CI

Hi @erikd,
thanks for creating this library. I wanted to create some Rust bindings and to simplify cross-platform compilation I continued the previous CMake attempts from @mdsitton and @jcelerier. I've also tried to setup some CI (Travis for Linux; Appveyor for Windows). Admittedly I've not really done much C development till now so please feel free to point out any problems with this PR.
https://travis-ci.org/Bobo1239/libsamplerate
https://ci.appveyor.com/project/Bobo1239/libsamplerate

This is a pretty large PR. Probably won't get time to have a proper look at it until the weekend.

hey, sorry, I had entirely forgot my PR. this looks pretty nice and complete ! But are you sure you want to support CMake versions that didn't even have PUBLIC / PRIVATE ? Are there still widely-used systems with such old version (that couldn't just install CMake's statically-built latest version) ?

@erikd No worries, take your time. :)
@jcelerier Tbh I was just trying to get everything here to become green (as they are now) and some of those platforms (they are Docker images) have CMake 2.8. I figured we aren't really using any new CMake features so it wouldn't hurt to support an older verson. But as I said in my opening post I don't have an overview over this ecosystem so you guys have to make the call whether it's worth to support old CMake versions.

libsndfile builds on travis with CMake min version of 3.1.3. Maybe figure out what that is doing.

Those builds aren't normal Travis instances; they are cross compiling and testing on many different Docker images (some of them using qemu internally?) and aren't maintained by me. (They are from cross which is ""Zero setup" cross compilation and "cross testing" of Rust crates"). If you guys don't think it's worth supporting such an old CMake version I have no problems reverting the last few changes. I would then just drop those few Travis instances or try to upgrade CMake in those Docker images.

Yeah, I don't think its worth supporting any CMake version older than the ones libsndfile uses.

that's interesting: these docker images have rust which is very recent, but don't have five-years old versions of CMake :p are they hosted somewhere on github ? it could be worth it to send PRs to them so that everyone doing such a work can at least leverage recent versions.

Reverted the last changes.
The docker images are here. They are using different Ubuntu base images (ranging from 12.04 to 16.04) so some of the older ones don't have CMake 3. I'll try to naively upgrade some of the Ubuntu versions but I'm expecting some issues as there's probably a reason that they are using different versions.

@Bobo1239 do you offer any build artifacts for libsamplerate?  If not, would the project entertain an automatic upload (e.g. github releases, etc?) to complement the cmake support?
I'm most interested in the win32 artifacts for multi-platform.  The other OSs (unix/linux/macos) have methods to fetch libsamplerate.  CMake drastically simplifies the MSVC++ setup and I'd like to add the 32-bit and 64-bit artifacts into the scoop.sh package manager.

@tresf I'll try to look into AppVeyor build artifacts this weekend.

@Bobo1239 good call on target_link_libraries.  a1ba332 now fixes building on non-posix environments.  Thanks!

@tresf Would this be enough for your use case?

@Bobo1239 will be needing the header file and the pkg-config (.pc) file.  libsndfile is a good example and here's how I've written the release into the scoop package manager.
Also, long-term, if there are no objections, it would be nice to have the file hosted at an unauthenticated, tagged location such as the github release area, if @erikd would entertain this.  AppVeyor is able to do this.

@tresf I guess it would make sense to setup the install target in CMake so we get that kind of structure for all platforms (also we're currently not generating a .pc file at all I think?)
I don't know when I'll have the time to try that out. If you need it rather sooner than later we can maybe try to land this PR first and let somebody else do that?

@Bobo1239 AFAIK .pc file should look like this:
prefix=
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/include

Name: samplerate
Description: An audio Sample Rate Conversion library
Requires:
Version: 0.1.8
Libs: -L${libdir} -lsamplerate
Libs.private: Ext/libsndfile.a
Cflags: -I${includedir}

CMake can configure this file like so (sample, I haven't tested it for syntax errors)
SET(prefix "")
SET(exe_prefix "\\${prefix}")
SET(libdir "\\${exec_prefix}/lib")
SET(includedir "\\${prefix}/include")
SET(VERSION "${PROJECT_VERSION}")
SET(LIBS "Ext/libsndfile.a")
CONFIGURE_FILE(samplerate.pc.in samplerate.pc @ONLY)

@tresf I guess it would make sense to setup the install target in CMake so we get that kind of structure for all platforms

I agree, although I can rearrange the files in the packaging utility if needed, that's what we have to do with fftw's for example.  Getting the output files is enough to start offering this package to win32 developers.

https://ci.appveyor.com/project/Bobo1239/libsamplerate/build/job/psucvyvat0tv6vye/artifacts Does this work?

@Bobo1239 Yes, it works quite well actually. :)
  -- Checking for module 'samplerate>=0.1.8'
+ --   Found samplerate, version 0.1.9

A working scoop.sh bucket is here.

Thank you all for your contribution!


[Feature]: use which-key's name as itemgroup

Similar Issues

[X] Before filing, I have searched for similar issues.

Description
When you register keys with which-key.nvim, you can add name which is displayed when you are in a group. For example, given this key:
			local keys = {
				x = {
					name = "Diagnostics",
					x = {"<CMD>TroubleToggle<CR>", "Toggle Trouble"},
					w = {"<CMD>Trouble workspace_diagnostics<CR>", "Workspace Diagnostics"},
					d = {"<CMD>Trouble document_diagnostics<CR>", "Document Diagnostics"},
					l = {"<CMD>Trouble loclist<CR>", "Location List"},
					q = {"<CMD>Trouble quickfix<CR>", "Quickfix"},
				}
			}
			local opts = {}

			wk.register(keys, opts)
			-- false if which-key.nvim handles binding them,
			-- set to true if you want legendary.nvim to handle binding
			-- the mappings; if not passed, true by default
			local handle_binding = false
			require('legendary.integrations.which-key').bind_whichkey( keys, opts, handle_binding)

I was hoping that this name would then show up in Legendary, but it doesn't:

I've tried adding itemgroup in a few spots, and not seeing it have actually add the itemgroup. I'm looking at these lines:
https://github.com/mrjones2014/legendary.nvim/blob/3008ed68c8f309ced07587684c8af53884791d35/lua/legendary/integrations/which-key.lua#L43
https://github.com/mrjones2014/legendary.nvim/blob/3008ed68c8f309ced07587684c8af53884791d35/lua/legendary/integrations/which-key.lua#L22
And it seems like adding it to the opts passed to require('legendary.integrations.which-key').bind_whichkey should end up in the items.

This should definitely be do-able. I don't personally use which-key so it might take me a bit to figure out.


500 instead of 404

When reaching for a non-existing image like
http://iiif.bdrc.io/image/v2/bdr:V23703_I1521::152106isfjgoeirf.jpg/full/full/0/default.jpg
the server returns a 500 error instead of a 404. The spec is quite clear that it should be a 404

this is fixed as of commit 859630d . However while this http://localhost:8080/image/v2/bdr:V29199_I5166::51660001.tif/info.json leads to 403 "Insufficient rights" locally (in dev), it returns 500 on ec2.

issue with 500 code instead of 403 has been fixed. http://iiif.bdrc.io/image/v2/bdr:V29199_I5166::51660001.tif/info.json

cool, thanks!


add pick_unused_port_range

This adds a way to select available free port within a range. Main intention is to enforce a range of port for more deterministic result.

@Dentosal would we be able to get this merged in?

I don't feel like this should be part of this crate. The users can just do the filtering themselves. Also, this pattern pushes towards an antipattern, where when multiple threads attempt to allocate ports from the same range, each of them tries all ports in order.

The users can just do the filtering themselves

Okay, but that seems a bit backwards/inefficient to have that happen after the library has already searched through a hardcoded (seemingly arbitrary) range of ports. Also, there is more hardcoded logic in there to fail out after arbitrarily 10 attempts and move to a whole separate method of port finding based on OS's availability, which, if a user were to 'just do the filtering themselves', could be completely outside the user's targeted range (same with the arbitrary 15000-25000 range).

Also, this pattern pushes towards an antipattern, where when multiple threads attempt to allocate ports from the same range, each of them tries all ports in order

I don't agree that this is an 'antipattern'. You're making an assumption that this library is thread safe as it is, which is not the case for the purpose you just stated. Is it not the case that, as the library is right now, it will attempt to select a port from a hardcoded range? The only (assumed) 'benefit' it has right now is that it randomly chooses a port, which makes the collision less likely to happen (which in my opinion is more of an 'antipattern'! Because the library is almost giving a false sense of collision security, but that's just not the case). Also, adding a form of determinism to this library seems like the opposite of an antipattern to me. Collisions are inevitable to happen by the nature of the fact that you would need to actually use the port before it would be considered in use by this library. To me, that naturally makes it the end-user's problem to handle the 'thread safety' of their port picking, since there isn't really anything this library can do about it.

I don't feel like this should be part of this crate

That perfectly within your right to determine, but I hope you might change your mind on this. I'd rather have this contributed back into this existing library than create a new package to handle this (node js style package hell).

Regarding the users doing the filtering themselves:
I noticed that is_free, fn is pub, so we could just iterate over a range ourselves and call that, which isn't all that bad. Maybe that's what you were suggesting. Still, I like the idea of having a fn do that logic for me, since it seems like a common requested use case. But if you decide that is now what you want for this crate, that is fine (I still don't agree with it being an antipattern, though).

For anyone else who wanted this functionality, I've created a crate openport that allows for this type of range port searching as the primary method. There is also a rand feature that you can enable if you still want the non-deterministic method of finding a port, however without that feature there are no external dependencies.


Feat: Condense Message History

Fix https://github.com/OpenDevin/OpenDevin/issues/1748
Adding https://github.com/OpenDevin/OpenDevin/pull/2937.

@CodiumAI-Agent /review

PR Reviewer Guide 🔍

⏱ Estimated effort to review: 4 🔵🔵🔵🔵⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Key issues to review
Missing HandlingThe parse method in action_parser.py now directly returns an AgentSummarizeAction if the response is of that type, but does not handle other potential actions that could be derived from AgentSummarizeAction. This might lead to unhandled cases if new subtypes of AgentSummarizeAction are introduced.
String ConcatenationIn the method action_to_str, string concatenation using '+' could lead to TypeError if the attributes like summarized_actions or summarized_observations are not strings. This could be mitigated by ensuring these attributes are always strings or using a different method of concatenation.
Error HandlingThe condense method raises SummarizeError if no messages are condensable or if there's only one message to summarize. However, it does not handle what should happen next, potentially leaving the system in an inconsistent state.
Token Limit CheckThe method is_over_token_limit checks if the token count exceeds the limit but includes a padding which might not be clear why it's needed. The use of padding should be justified or documented to avoid confusion.

@CodiumAI-Agent /ask
Give suggestions.

Ask❓
Give suggestions.
Answer:
Here are some suggestions for the changes introduced in this PR:

Documentation and Comments:

Ensure that all new methods and classes have comprehensive docstrings explaining their purpose, parameters, and return values. For example, the CondenserMixin class and its methods could benefit from more detailed explanations.
Add comments to complex logic sections to improve readability and maintainability. For instance, the loop in the condense method of CondenserMixin could use comments explaining the purpose of each step.

Error Handling:

In the step method of agenthub/codeact_agent/codeact_agent.py, consider logging the exception details when catching ContextWindowLimitExceededError to aid in debugging.
Ensure that all custom exceptions in opendevin/core/exceptions.py have meaningful messages that provide context about the error.

Code Consistency:

Ensure consistent use of single and double quotes throughout the codebase. For example, in opendevin/condenser/condenser.py, both single and double quotes are used interchangeably.
Maintain consistent formatting for method definitions and calls. For instance, in opendevin/llm/llm.py, the method get_token_count has inconsistent spacing around the parameters.

Performance Considerations:

In the condense method of CondenserMixin, consider optimizing the token counting logic to avoid recalculating token counts for the same messages multiple times.
Review the retry logic in opendevin/llm/llm.py to ensure it does not lead to excessive retries, which could impact performance.

Testing:

Ensure that new functionality, especially the summarization and condensing logic, is thoroughly tested. Add unit tests to cover various scenarios, including edge cases.
Consider adding integration tests to verify that the summarization and condensing work correctly within the broader system.

Logging:

Enhance logging to provide more context about the operations being performed. For example, in opendevin/condenser/condenser.py, log the number of tokens before and after summarization.
Ensure that sensitive information is not logged, especially when dealing with user inputs or model outputs.

Code Cleanup:

Remove commented-out code unless it serves a clear purpose. For example, the commented-out code in opendevin/condenser/condenser.py related to preserving the last N messages should be either implemented or removed.
Ensure that all TODO comments have corresponding tasks or issues created to track their completion.

By addressing these suggestions, the codebase will be more maintainable, performant, and easier to understand for future developers.

@CodiumAI-Agent /ask

Missing Handling
The parse method in action_parser.py now directly returns an AgentSummarizeAction if the response is of that type, but does not handle other potential actions that could be derived from AgentSummarizeAction. This might lead to unhandled cases if new subtypes of AgentSummarizeAction are introduced.

It will be handled if new subtypes are introduced. Remember Simple is better than complex.

String Concatenation
In the method action_to_str, string concatenation using '+' could lead to TypeError if the attributes like summarized_actions or summarized_observations are not strings. This could be mitigated by ensuring these attributes are always strings or using a different method of concatenation.

Both types are strings and remember mypy is happy.

Error Handling
The condense method raises SummarizeError if no messages are condensable or if there's only one message to summarize. However, it does not handle what should happen next, potentially leaving the system in an inconsistent state.

It may not be raised.

Token Limit Check
The method is_over_token_limit checks if the token count exceeds the limit but includes a padding which might not be clear why it's needed. The use of padding should be justified or documented to avoid confusion.

Yep, Quoting from Previous PR estimation of tokens to add to the prompt for the user-configured max token count

Ask❓

[Missing Handling](https://github.com/SmartManoj/Kevin/pull/21/files > The parse method in action_parser.py now directly returns an AgentSummarizeAction if the response is of that type, but does not handle other potential actions that could be derived from AgentSummarizeAction. This might lead to unhandled cases if new subtypes of AgentSummarizeAction are introduced. It will be handled if new subtypes are introduced. Remember Simple is better than complex.  > [String Concatenation](https://github.com/SmartManoj/Kevin/pull/21/files > In the method action_to_str, string concatenation using '+' could lead to TypeError if the attributes like summarized_actions or summarized_observations are not strings. This could be mitigated by ensuring these attributes are always strings or using a different method of concatenation. Both types are strings and remember mypy is happy.  > [Error Handling](https://github.com/SmartManoj/Kevin/pull/21/files > The condense method raises SummarizeError if no messages are condensable or if there's only one message to summarize. However, it does not handle what should happen next, potentially leaving the system in an inconsistent state. It may not be raised.  > [Token Limit Check](https://github.com/SmartManoj/Kevin/pull/21/files > The method is_over_token_limit checks if the token count exceeds the limit but includes a padding which might not be clear why it's needed. The use of padding should be justified or documented to avoid confusion. Yep, Quoting [from Previous PR](https://github.com/OpenDevin/OpenDevin/pull/2021/files

Answer:
Missing Handling
The parse method in action_parser.py now directly returns an AgentSummarizeAction if the response is of that type. While this is a straightforward approach, it does not account for potential future subtypes of AgentSummarizeAction. If new subtypes are introduced, they might not be handled correctly. To future-proof this, you could use a more flexible approach, such as:
def parse(self, response) -> Action:
    if isinstance(response, AgentSummarizeAction):
        return response
    # Handle other potential subtypes here
    action_str = self.parse_response(response)
    return self.parse_action(action_str)

This ensures that any new subtypes of AgentSummarizeAction will be handled appropriately.
String Concatenation
In the action_to_str method, string concatenation using + could lead to TypeError if summarized_actions or summarized_observations are not strings. Although you mentioned that these attributes are always strings and mypy is happy, it is still good practice to ensure type safety. You can use f-strings or the str() function to ensure that the attributes are treated as strings:
elif isinstance(action, AgentSummarizeAction):
    return (
        f'Summary of all Action and Observations till now. \n'
        f'Action: {str(action.summarized_actions)}\n'
        f'Observation: {str(action.summarized_observations)}'
    )

Error Handling
The condense method raises SummarizeError if no messages are condensable or if there's only one message to summarize. However, it does not specify what should happen next, potentially leaving the system in an inconsistent state. It would be beneficial to handle this exception to ensure the system remains consistent:
def condense(self, messages: list[Message]):
    try:
        # Existing condense logic
    except SummarizeError as e:
        logger.error(f'Condensation failed: {str(e)}')
        # Handle the error, e.g., by returning a default action or re-trying with different parameters
        return None

Token Limit Check
The is_over_token_limit method includes a padding of 512 tokens, which might not be immediately clear. Documenting the reason for this padding would help avoid confusion. For example:
def is_over_token_limit(self, messages: list[Message]) -> bool:
    """
    Estimates the token count of the given events using litellm tokenizer and returns True if over the max_input_tokens value.

    Parameters:
    - messages: List of messages to estimate the token count for.

    Returns:
    - Estimated token count.

    Note:
    - A padding of 512 tokens is added to account for potential discrepancies in token counting.
    """
    MAX_TOKEN_COUNT_PADDING = 512
    if not self.config.max_input_tokens:
        return False
    token_count = self.get_token_count(messages=messages) + MAX_TOKEN_COUNT_PADDING
    return token_count >= self.config.max_input_tokens

This documentation clarifies the purpose of the padding and helps maintainers understand its necessity.

@CodiumAI-Agent /ask

    # Handle other potential subtypes here
    action_str = self.parse_response(response)
    return self.parse_action(action_str)

Other subtypes of  Action are already handled after that.  What will be the new subtypes of AgentSummarizeAction? Even if it so, it will be inside the if branch.

f'Summary of all Action and Observations till now. \n'
        f'Action: {str(action.summarized_actions)}\n'

f is not needed in first.
no need to use str inside f-strings

However, it does not specify what should happen next, potentially leaving the system in an inconsistent state. It would be beneficial to handle this exception to ensure the system remains consistent:

need to check why, where, and when it will fail.

Ask❓

python > > action_str = self.parse_response(response) > return self.parse_action(action_str) >  Other subtypes of Action are already handled after that. What will be the new subtypes of AgentSummarizeAction? Even if it so, it will be inside the if branch. > python > f'Summary of all Action and Observations till now. n' > f'Action: {str(action.summarized_actions)}n' >  f is not needed in first. no need to use str inside f-strings > However, it does not specify what should happen next, potentially leaving the system in an inconsistent state. It would be beneficial to handle this exception to ensure the system remains consistent: need to check why, where, and when it will fail.

Answer:

action_str = self.parse_response(response)
return self.parse_action(action_str)

Other subtypes of Action are already handled after that. What will be the new subtypes of AgentSummarizeAction? Even if it so, it will be inside the if branch.

The new subtype introduced in this PR is AgentSummarizeAction. This action is specifically handled in the if branch before the parse_response method is called. If the response is an instance of AgentSummarizeAction, it is returned directly without further parsing. This ensures that the summarization action is processed correctly and not treated as a generic action string.

f'Summary of all Action and Observations till now. n'
f'Action: {str(action.summarized_actions)}n'

f is not needed in first. no need to use str inside f-strings

You are correct. The first f is unnecessary, and using str inside f-strings is redundant. The corrected code should be:
return (
    'Summary of all Action and Observations till now. \n'
    + 'Action: '
    + action.summarized_actions
    + '\nObservation: '
    + action.summarized_observations
)

However, it does not specify what should happen next, potentially leaving the system in an inconsistent state. It would be beneficial to handle this exception to ensure the system remains consistent: need to check why, where, and when it will fail.

To ensure the system remains consistent, it is crucial to handle exceptions that might arise during the summarization process. The SummarizeError and ContextWindowLimitExceededError exceptions are introduced to handle specific failure scenarios. Here's an example of how to handle these exceptions:

SummarizeError: This exception is raised when there are not enough messages to summarize. It should be caught and logged, and the system should proceed with the next available action or observation.

ContextWindowLimitExceededError: This exception is raised when the context window limit is exceeded, and memory cannot be condensed further. This should be caught, logged, and potentially trigger a fallback mechanism, such as notifying the user or attempting a different strategy.

Example of handling these exceptions:
try:
    response = self.llm.completion(
        messages=[message.model_dump() for message in messages],
        stop=[
            '</execute_ipython>',
            '</execute_bash>',
            '</execute_browse>',
        ],
        temperature=0.0,
        condense=True,
    )
except SummarizeError as e:
    logger.error(f'Summarization failed: {str(e)}')
    # Handle the error, e.g., by notifying the user or taking a fallback action
except ContextWindowLimitExceededError as e:
    logger.error(f'Context window limit exceeded: {str(e)}')
    # Handle the error, e.g., by notifying the user or taking a fallback action

By handling these exceptions, the system can maintain consistency and provide a better user experience even in failure scenarios.

Applied the suggestions in https://github.com/SmartManoj/Kevin/commit/7abe952853b6bfd958f2f817376aba3a900bfea3


Boostrap Utilities

Hi, I was wondering whether you planned on writing supporting code for bootstrap analysis, currently the lingam package is way too slow using inefficient code.
However, it remains a good example, please see the tutorial.
https://lingam.readthedocs.io/en/latest/tutorial/var.html

Thanks for suggesting this, we have put it on the list and will try to see the feasibility of it.


[1918] Fixed by allowing allocation to return current and previous al…

…location year
Context
Changes proposed in this pull request
Guidance to review
Checklist

[ ] Make sure all information from the Trello card is in here
[ ] Attach to Trello card
[ ] Rebased master
[ ] Cleaned commit history
[ ] Tested by running locally

 Allocations page 'tweak'


Linux support

On DS3T.exe launch it shows various errors (I think because it cannot read my GPU info) and when enabled I can't set my monitor's resolution when in game (which I could without the program).
It plays at a resolution considerably lower.
Tested on Proton-GE-5.4-1

@noaccOS Did you manage to get ultrawide working? Tried both this app and hex editor... Nothing worked.

@noaccOS Did you manage to get ultrawide working? Tried both this app and hex editor... Nothing worked.

Nope, I played on windows and I got banned... Now I play on Linux without the mod

This is one of the first places I found when I was trying to get ultrawide working, so gonna paste my protondb report here now that I have the "hex method" working in case others like me stumble here:

Proton 5.0-9
Distro: Arch Linux
Kernel: 5.8.5-arch1-1
RAM: 16 GB
GPU Driver: 4.6 Mesa 20.1.6
GPU: AMD Radeon RX 5700 XT
CPU: AMD Ryzen 5 2600X Six-Core

I got Ultrawide support working! Basically doing the "Hex Method", which is outlined here: https://steamcommunity.com/sharedfiles/filedetails/?id=668484112
Make sure you run the game once before you follow the guide, and set the game mode to windowed (if you don't/froget you can just edit the file to say WINDOW).
I actually made the GraphicsConfig.xml read-only too, but not sure if it was necessary.
I pieced together my resolution from these tables: https://www.wsgf.org/article/common-hex-values
000F 0000 4006
3840 x 1600
(The middle 4 values are always 0000 it seems)
The menus won't be ultrawide, but once you're ingame the window will expand like desired (and not stretch or show black bars)
Something very weird I experience is my GraphicsConfig.xml file having really weird artifacts/not-actually-display characters embedded in between every character. vim of course has no trouble with this and I could "cheat" the values outlined in the guide, but I imagine any other text editor wont' know how to read the characters (leafpad could only render a single <, for example).

(I know this doesn't further the scope of this ticket but I figure it's highly unlikely the dev cares about Linux users anyway lol)

@Lightsockie Thank you very much for your input. Finally figured it out. The problem was a strange encoding on my GraphicsConfig.xml. I removed it and created a new file with default settings copied from internet. Ultra wide is now working!


cherry-pick: lockedpool: avoid sensitive data in core files (Linux and FreeBSD) (#21)

Manual backport of bitcoin PR#18443 and bitcoin PR#15633.
Use madvise on Linux and FreeBSD to avoid sensitive data from secure_allocator to be written to swap and core-files.
https://github.com/bitcoin/bitcoin/pull/15600/commits/d831831822885717e9841f1ff67c19add566fa45
https://github.com/bitcoin/bitcoin/pull/18443/commits/f85203097f78d9daa1d35c4097a80beab31da2a4
(cherry picked from commit 7170b3ef04a0fd00f6fcd8212a7a0fc13e289e29)

Merging now because CI passes and already ACK since cherry-picked from #21


HADOOP-18817.Upgrade version of aws-java-sdk-bundle to 1.12.368 avoid verify error.

Description of PR
The compilation failed when I packaged through the maven command
mvn clean install -DskipTests -Dtar -Pdist -Pnative
report an error:
[WARNING]
Dependency convergence error for com.amazonaws:aws-java-sdk-simpleworkflow:1.12.367 paths to dependency are:
+-org.apache.hadoop:hadoop-aws:3.3.6
  +-com.amazonaws:aws-java-sdk-bundle:1.12.367
    +-com.amazonaws:aws-java-sdk:1.12.367
      +-com.amazonaws:aws-java-sdk-simpleworkflow:1.12.367

and

+-org.apache.hadoop:hadoop-aws:3.3.6
  +-com.amazonaws:aws-java-sdk-bundle:1.12.367
    +-com.amazonaws:aws-java-sdk:1.12.367
      +-com.amazonaws:aws-java-sdk-swf-libraries:1.11.22
        +-com.amazonaws:aws-java-sdk-simpleworkflow:1.11.22
[WARNING] Rule 0: org.apache.maven.plugins.enforcer.DependencyConvergence failed with message:
 Failed while enforcing releasability. See above detailed error message.

com.amazonaws:aws-java-sdk-swf-libraries are not required
How was this patch tested?
unit test.
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

-1 :x:
mvninstall
53m 23s
/branch-mvninstall-root.txt
root in trunk failed.

+1 :green_heart:
compile
0m 22s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 23s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
javadoc
0m 26s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 26s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
89m 46s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 15s

the patch passed

+1 :green_heart:
compile
0m 15s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 15s

the patch passed

+1 :green_heart:
compile
0m 14s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
0m 14s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 18s

the patch passed

+1 :green_heart:
javadoc
0m 17s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 16s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
40m 14s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 20s

hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 42s

The patch does not generate ASF License warnings.

136m 20s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5867

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux 96f952725026 4.15.0-212-generic #223-Ubuntu SMP Tue May 23 13:09:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / 538c37eb1c4d9a0c2ba354621a1a15d079a68a72

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/1/testReport/

Max. process+thread count
730 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/1/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

see discussion on the jira; really love to know how you see this even though I don't.
also: https://hadoop.apache.org/docs/current/hadoop-aws/tools/hadoop-aws/testing.html#The_submitter_of_any_patch_is_required_to_run_all_the_integration_tests_and_declare_which_S3_region.2Fimplementation_they_used.

Excluding aws-java-sdk-swf-libraries outright may be too radical.  Perhaps we can upgrade the aws-java-sdk-bundle version to 1.12.368 or later

i don't see this, nor does mavenrepository.
can you remove the bundle jar from your local maven repo? then rebuild being sure you are using the maven central repo only?

Excluding aws-java-sdk-swf-libraries outright may be too radical. Perhaps we can upgrade the aws-java-sdk-bundle version to 1.12.368 or later

the bundle is meant to have everything, with shaded dependencies. sometimes they add a new module and forget to add an excludes clause in the generated pom so even though it is in the jar, the old module still comes in. seen it more than once.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 42s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+1 :green_heart:
mvninstall
48m 35s

trunk passed

+1 :green_heart:
compile
0m 22s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
0m 23s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
mvnsite
0m 28s

trunk passed

+1 :green_heart:
javadoc
0m 28s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 25s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
86m 42s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 14s

the patch passed

+1 :green_heart:
compile
0m 15s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
0m 15s

the patch passed

+1 :green_heart:
compile
0m 15s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
0m 15s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 17s

the patch passed

+1 :green_heart:
javadoc
0m 15s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
0m 16s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
40m  5s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 19s

hadoop-project in the patch passed.

+1 :green_heart:
asflicense
0m 40s

The patch does not generate ASF License warnings.

133m  3s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/2/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5867

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint

uname
Linux 0b282fa23f10 4.15.0-212-generic #223-Ubuntu SMP Tue May 23 13:09:22 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / a34565f6284e09f99db9fcc0d248f2ffd3e57496

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/2/testReport/

Max. process+thread count
655 (vs. ulimit of 5500)

modules
C: hadoop-project U: hadoop-project

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/2/console

versions
git=2.25.1 maven=3.6.3

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 41s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  1s

codespell was not available.

+0 :ok:
detsecrets
0m  1s

detect-secrets was not available.

+0 :ok:
xmllint
0m  1s

xmllint was not available.

+0 :ok:
shelldocs
0m  1s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s

The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ trunk Compile Tests _

+0 :ok:
mvndep
16m  3s

Maven dependency ordering for branch

+1 :green_heart:
mvninstall
31m  8s

trunk passed

+1 :green_heart:
compile
17m 39s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
compile
16m  9s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
mvnsite
20m 17s

trunk passed

+1 :green_heart:
javadoc
8m 47s

trunk passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
7m 30s

trunk passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
49m 56s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 51s

Maven dependency ordering for patch

+1 :green_heart:
mvninstall
29m 49s

the patch passed

+1 :green_heart:
compile
16m 58s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javac
16m 58s

the patch passed

+1 :green_heart:
compile
16m 19s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
javac
16m 19s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
15m  0s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
javadoc
8m 41s

the patch passed with JDK Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1

+1 :green_heart:
javadoc
7m 32s

the patch passed with JDK Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

+1 :green_heart:
shadedclient
51m  8s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

-1 :x:
unit
756m  3s
/patch-unit-root.txt
root in the patch passed.

+1 :green_heart:
asflicense
2m  3s

The patch does not generate ASF License warnings.

1045m 23s

Reason
Tests

Failed junit tests
hadoop.hdfs.server.federation.router.TestRouterRPCMultipleDestinationMountTableResolver

hadoop.fs.http.server.TestHttpFSAccessControlled

hadoop.mapreduce.v2.TestUberAM

hadoop.mapreduce.v2.TestMRJobsWithProfiler

hadoop.mapreduce.v2.TestMRJobs

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/3/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/5867

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient codespell detsecrets xmllint shellcheck shelldocs

uname
Linux 810855e05769 4.15.0-213-generic #224-Ubuntu SMP Mon Jun 19 13:30:12 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
trunk / dcec887850490802a5244d9a15b6ac11dbb1d364

Default Java
Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Multi-JDK versions
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.19+7-post-Ubuntu-0ubuntu120.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_362-8u372-ga~us1-0ubuntu1~20.04-b09

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/3/testReport/

Max. process+thread count
3242 (vs. ulimit of 5500)

modules
C: hadoop-project . U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-5867/3/console

versions
git=2.25.1 maven=3.6.3 shellcheck=0.7.0

Powered by
Apache Yetus 0.14.0 https://yetus.apache.org

This message was automatically generated.

Thanks a lot steveloughran for the reminder; Unfortunately, my maven Central repository pom dependency on aws-java-sdk

@gp1314 didn't get the end of your comment.

My own repository aws-java-sdk-bundle was wrong causing the problem


fix: Line the edit and delete icons up

Fixes: #1847
Still not perfect but the icons line up better.  @riccardo-forina is actively modifying the look and feel - will rely on his changes for final appearance.

Yup - I basically made a separate span for the buttons so they would always
be kept as a pair (load mockdoc.adm and you'll see they get split).
On Mon, Mar 9, 2020 at 12:16 PM John Verhaeg<EMAIL_ADDRESS>wrote:

@jpav commented on this pull request.
So using Netlify and the Atlasmap Core link (the UI one never works for
me), I see this result:
[image: image]
https://user-images.githubusercontent.com/437085/76234213-68b91d80-61f7-11ea-9a9f-0314fb808272.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/atlasmap/atlasmap/pull/1854?email_source=notifications&email_token=AAWX4LLMWFTWU6AP73U5QM3RGUP7XA5CNFSM4LDICXDKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCYQ7ZXQ#pullrequestreview-371326174,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWX4LMKLYOQVWF5JZTNP23RGUP7XANCNFSM4LDICXDA
.


Error during NOR flash Winbond 2G-bit initialization (IDFGH-8666)

Answers checklist.

[x] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[ ] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[ ] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Hello,
I'm using esp-idf 4.4 and I have this error during the inizialization of Winbond: W25Q02JVTBIM(2G-bit):
E (1199) wl_flash: config(108): result = 0x00000102 E (1199) wear_levelling: wl_mount: config instance=0x00000000, result=0x102 E (1209) vfs_fat_spiflash: failed to mount wear levelling layer. result = 258
The initializzation code is:
`const spi_bus_config_t bus_config =
{
.mosi_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_MOSI,
.miso_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_MISO,
.sclk_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_CLK,
.quadwp_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_WP,
.quadhd_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_HOLD
};
const esp_flash_spi_device_config_t device_config =
{
.host_id = VSPI_HOST,
.cs_io_num = CONFIG_RCS_FLASH_DRIVER_W25Q_CS,
.io_mode = SPI_FLASH_DIO,
.speed = ESP_FLASH_40MHZ,
.cs_id = CS_ID
};
spi_bus_initialize(VSPI_HOST, &bus_config, 1);
esp_flash_t* ext_flash;
spi_bus_add_flash_device(&ext_flash, &device_config);
sp_flash_init(ext_flash);
uint32_t id;
esp_flash_read_id(ext_flash, &id);`
With the same code, I have no problem with the winbond: W25Q128JV(128M-bit).
Why with W25Q02JVTBIM(2G-bit) I have this error?
How can I solve it?
Thanks

Hello I solved the error, using this: https://github.com/espressif/esp-idf/pull/9381#issuecomment-1304428194, replacing the patch with:
if ((id & 0xFFFF) == 0x4020 || (id & 0xFFFF) == 0x7020) {
        *size = 0x4000000;
    } else if ((id & 0xFFFF) == 0x4021) {
        *size = 0x8000000;
    } else if ((id & 0xFFFF) == 0x7022) {
        *size = 0x10000000;
    } else {
        *size = 1 << (id & 0xFF);
    }


Issue 13593: tesseract-ocr/fuzzer-api: Undefined-shift in scaleGray2xLILineLow

OSS Fuzz reports an issue here:
scale1.c:2717:28: runtime error: left shift of 255 by 24 places cannot be represented in type 'l_int32' (aka 'int')
See ~~https://github.com/DanBloomberg/leptonica/blob/master/src/scale1.c#L2717~~
https://github.com/DanBloomberg/leptonica/blob/1.77.0/src/scale1.c#L2717.
@DanBloomberg, this is Leptonica code, so OSS Fuzz triggers a problem via the Tesseract API.

Looking at the code, I don't see the issue.
All the words are uint32, all the 8 bit pixel values are int32, but each is masked with 0xFF.
So the line in question is just a sum of 4 numbers, each <= 255, and then divided by 4.

Looks like it was fuzzed on 1.77.0, not the current head.  So the issue is really on line 2725.  And I do see the problem, something I'd have never thought about.
I believe the simple fix is to make all the sval* pixel values uint32.  Then when one is pushed << 24, it's still valid.  Will check this out.

That's right, it is currently using 1.77.0.

You can make a PR against google/oss-fuzz that adds you and Dan to auto_ccs in https://github.com/google/oss-fuzz/blob/master/projects/leptonica/project.yaml
But I'm not in charge of the leptonica project or oss-fuzz.
CC @Dor1s @jonathanmetzman

Fix is in at leptonica master.
Is there some place we should log this, or will the fuzzer eventually run on the fix and consider the issue cleared, or ...?

The oss-fuzz system will automatically detect the fix in the next build.

Really? It's in Leptonica, and I would have expected that an update of the configuration is needed. Maybe it can use Leptonica master instead of 1.77.0.

Right, it's using a static version now.. I'll make a PR against the tesseract-ocr oss-fuzz project to use Leptonica master.

@stweil I'll send out a PR to add you as a cc here: https://github.com/google/oss-fuzz/blob/master/projects/leptonica/project.yaml . What email should I use?

Thanks, you can use the same e-mail address as for tesseract-ocr.

Maybe it can use Leptonica master instead of 1.77.0.

Should tesseract also change dependency to the new/fixed version of leptonica?

Up to now there exists no Leptonica release with the fix (it's only in Git master).
So the question is how to get the fix into the major Linux distributions. I think this requires new Leptonica releases 1.74.5, 1.75.4, 1.76.1 and 1.77.1. Otherwise Debian and others would have to patch the code. CC @jbreiden as the maintainer.

@DanBloomberg : What is the  plan for 1.78.0 release?

No specific plan.  I can do it within the next three weeks.
AFAICT we do not need a release for this bug.  I haven't run the fuzzer, but I can't believe there is any memory corruption. The only thing the bug can do is make a few bad pixels in an 8 bpp grayscale image when that particular fast 2x upscaler is used.

You are right, it's not a security problem. Up to now OSS-Fuzz only detected one real security issue (#2298). But of course for OCR a few bad pixels might result in wrong text recognition, so it is good that this is fixed now.

OSS-Fuzz has verified the fix in Leptonica, so this issue can be closed. See details here.

actually I can't see it ... perhaps one needs to be on the auto_css list at
oss-fuzz/projects/tesseract-ocr  ?
but it's likely not important for me to be on it.
On Sun, Mar 10, 2019 at 10:39 AM Stefan Weil<EMAIL_ADDRESS>wrote:

OSS-Fuzz has verified the fix in Leptonica, so this issue can be closed.
See details here
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=13593.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/tesseract-ocr/tesseract/issues/2300#issuecomment-471327019,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AP6mLCV9OX_L9CPiM9UcBfWpyKt1wDyVks5vVUNVgaJpZM4blYXw
.

https://github.com/google/oss-fuzz/pull/2230 will add @stweil and @DanBloomberg to leptonica auto_ccs.


rotated bboxes and deepstream

Dear Contributors,
is it possible to use the provided .cu code to get rotated bboxes work with deepstream? I cannot find any references on this topic.
Cheers

As long as the built engine can load the shared object library correctly, so can deepstream


Feature request: open file through SFTP

Feature request: open/edit files with SFTP
建议SFTP中增加打开文件的功能、编辑文件的功能

schedule to next release

Fixed in v1.8.0


[GenEnum] Fix getBitMaskForNoPayloadElements.

Based on a patch from Joe Groff. This is only triggered in
lldb so I'll add a test there.
rdar://problem/51211938

@swift-ci test

I need to add a test in lldb but I want to make sure this doesn’t break anything in swift.

Build failed
Swift Test OS X Platform
Git Sha - aef830d4a4ceadfaac84771b8805d573c5016201

@swift-ci please test macos platform


Create a common infrastructure for testing Spark

Context
As part of the integration pulse to test Kubeflow<>Spark we will need to spin up some common infra to work together.
This could be either:

A VM, with necessary snaps (microk8s and spark-client)
EKS cluster

Then we'll also need to configure an S3 endpoint, that will be used from the Spark pods. My suggestion would be to go with a VM, as it can contain also spark snap in one place.
What needs to get done

Setup central environment that multiple folks can access (i.e. VM with ssh keys)
Install K8s and have the spark-client snap available
Add users to the common environment
Configure some S3, which Pods in the cluster can access

Definition of Done

An environment with K8s is deployed
The Spark credentials can be created in the Kubeflow user namespace
Pod in user namespace can put/pull files from S3

AWS VM has been created and shared across participants


Create phoneBR.js

Validates brazilian phone numbers (without international prefix).
Valida telefones brasileiros, fixos ou celulares, com ou sem DDD.
EX: 11 91234-1234 OU (011) 1234-1234

please add a few tests into https://github.com/jzaefferer/jquery-validation/blob/master/test/methods.js#L569 to support your new plugin

I would suggest changing this part of the regular expression (\(?[0-9]{2,3}\)?)? for this (\(0?[0-9]{2}\)|0?[0-9]{2})? this way a strange number such as (09999999-9999 will not be allowed and the national long distance digit 0 is also validated.

Closing because of missing tests. Feel free to add those, we can re-open again.


fix: navbar updates

📑 Description
Navbars in Flowbite.com have changed. Especially in mobile view.
This PR

tries to bring the compatibility with above changes
uses Frame to implement navbar components
adds few examples to navbar docs
some other non-navbar tweaks
Dropdown color and border fine tune
DropdownItem to be <a> and remove of tabindex

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed
[x] My pull request is based on the latest commit (not the npm version).

ℹ Additional Information

[ ] User menu dropdown

The position is wrong before and after click in a small screen.

[ ] Navbar with search

When you click the search icon, nothing happens.

[ ] Default mega menu

The ExampleDiv needs more height in mobile version.

[ ] Maga menu with icons

Same as above. It needs more height.

[ ] Full width dropdown

And in a normal screen, it breeds out of the ExampleDiv container. May you need to add relative to parent?
-[ ] Full width with CTA
Same as above.
And in a normal screen, it breeds out of the ExampleDiv container. May you need to add relative to parent?

[ ] Full width with image

Same as above.
And in a normal screen, it breeds out of the ExampleDiv container. May you need to add relative to parent?

[x]    User menu dropdown:  position is wrong before and after click in a small screen.
[ ]     Navbar with search: When you click the search icon, nothing happens.
What should happen? Nothing happens in previous version nor on Flowbite.com
[x]     Default mega menu: The ExampleDiv needs more height in mobile version.
[x]     Maga menu with icons: as above. It needs more height.
[ ]     Full width dropdown: And in a normal screen, it comes out of the ExampleDiv container. May you need to add relative to parent?
This is a popperjs workaround. There is already relative in the parent element and default strategy for positioning absolute makes the positioning shifted to be centered around the trigger element. To make it full screen I needed a strategy fixed. This works in full scree app, but does not respect parent element. I have no solution for today.

Corrected by popperjs modifier. No need to fixed strategy any more.

A11y problems with navbar page.
https://validator.unl.edu/?showsource=yes&showoutline=yes&doc=https%3A%2F%2Fflowbite-svelte.com%2Fcomponents%2Fnavbar#cl307c338


[BUG] Unable to install plasmo via pnpm

What happened?
I followed the docs and executed the following command pnpm create plasmo only to be met with the following errors.
pnpm create plasmo
🟣 Plasmo v0.86.3
🔴 The Browser Extension Framework
🟡 Extension name: demo-extension
🟡 Extension description: A basic Plasmo extension.
🟡 Author name: Plasmo Corp<EMAIL_ADDRESS>🔵 INFO   | Creating new project with popup
🔵 INFO   | Installing dependencies...
 WARN  1 deprecated subdependencies found<EMAIL_ADDRESS>Packages: +589
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 667, reused 588, downloaded 0, added 589, done
<EMAIL_ADDRESS>Running install script, failed in 7.2s
<EMAIL_ADDRESS>install$ (node install/libvips && node install/…
│ sharp: Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.1…
│ sharp: Installation error: Request timed out
│ sharp: Please see https://sharp.pixelplumbing.com/install for required dependenci…
└─ Failed in 7.2s at<EMAIL_ADDRESS> ELIFECYCLE  Command failed with exit code 1.
🟠 WARN   | pnpm install exited with non-zero code: 1
🔵 INFO   | Initializing git project...
🟢 DONE   | Your extension is ready in:  /Users/********/********/********/********/demo-extension

    To start hacking, run:

       cd demo-extension
       pnpm dev

    Visit https://docs.plasmo.com for documentation and more examples.

Even after explicitly installing sharp by following instructions from here I still get the same error.
When I run pnpm run in the project directory, unsurprisingly it throws the following error.
><EMAIL_ADDRESS>dev /Users/********/********/********/********/demo-extension
> plasmo dev

sh: plasmo: command not found
 ELIFECYCLE  Command failed.

Extra Info:
node version: v20.11.0
Version
Latest
What OS are you seeing the problem on?
MacOSX
What browsers are you seeing the problem on?
Chrome
Relevant log output
pnpm create plasmo
🟣 Plasmo v0.86.3
🔴 The Browser Extension Framework
🟡 Extension name: demo-extension
🟡 Extension description: A basic Plasmo extension.
🟡 Author name: Plasmo Corp<EMAIL_ADDRESS>🔵 INFO   | Creating new project with popup
🔵 INFO   | Installing dependencies...
 WARN  1 deprecated subdependencies found<EMAIL_ADDRESS>Packages: +589
++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
Progress: resolved 667, reused 588, downloaded 0, added 589, done
<EMAIL_ADDRESS>Running install script, failed in 7.2s
<EMAIL_ADDRESS>install$ (node install/libvips && node install/…
│ sharp: Downloading https://github.com/lovell/sharp-libvips/releases/download/v8.1…
│ sharp: Installation error: Request timed out
│ sharp: Please see https://sharp.pixelplumbing.com/install for required dependenci…
└─ Failed in 7.2s at<EMAIL_ADDRESS> ELIFECYCLE  Command failed with exit code 1.
🟠 WARN   | pnpm install exited with non-zero code: 1
🔵 INFO   | Initializing git project...
🟢 DONE   | Your extension is ready in:  /Users/********/********/********/********/demo-extension

    To start hacking, run:

       cd demo-extension
       pnpm dev

    Visit https://docs.plasmo.com for documentation and more examples.

(OPTIONAL) Contribution

[ ] I would like to fix this BUG via a PR

Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

Running pnpm add plasmo seems to install the plasmo binary, however pnpm dev still doesn't work.
❯ pnpm dev

><EMAIL_ADDRESS>dev /home/pavitra/Playground/chrome-ext-poc
> plasmo dev

Bus error (core dumped)
 ELIFECYCLE  Command failed with exit code 135.

I meet the same problem.

I use node v20 but failed. Using node 18 success.

May I ask which version you are using specifically? I have also encountered this issue

I use node v20 but failed. Using node 18 success.
May I ask which version you are using specifically? I have also encountered this issue

I use node v20 but failed. Using node 18 success.
May I ask which version you are using specifically? I have also encountered this issue
Finally, I used<EMAIL_ADDRESS>Solved the problem

i install python in my computer. it worked.

i install python in my computer. it worked.

Sorry, it seems unrelated to Python.


Multiple gVim dialog titles says "Vim 8.1"

It seems most dialog titles in gVim just says "Vim 8.1".
Like the Help > Find... dialog.
Would it not be better to add a descriptive title?

Do you mean that they should include patchlevel?

No.
It should say "Find" in the title of the find dialog (Help>Find...).
Just like it says "Font" in the title of the font dialog (Edit>Font...).

The dialog box is shown by the Vim script function inputdialog(). Currently it doesn't support setting the title.

:cry: ok

Adding an argument to inputdialog() might be an option, but it already has three arguments.
I think it's important to keep simplicity and also consistency with input().

From a users point of view its not consistent when some titles are "Vim 8.1" and some are not.


Crashing, regression with generic types

Crashing on 5f4511a30e808bb4f05badcfd9c5b69771e29701
Works fine on 2.1.0-dev.7.0
Sorry the repro is crazy. If I have time, I'll try to make it easier.

It looks like there is a missing substitution on a mixin application's superclass constructor.

Any updates on ETA on this?

Seeing this internally too with bleeding edge testing.


Zoo prefs

Description
@dr-darryl-wright integrated imitation/RLHF with Zooniverse:

added a ZooniverseQuerent(PrefCollectQuerent)
extended write_fragment_video function so that it can generate gifs
added a ZooniverseGatherer(PrefCollectGatherer)

Testing
Changes are not yet tested

Code changes were moved to the backflip project


Input PIN but no login

Issue description
When I type in my PIN, six digits does nothing, including not logging in.
If I type in wrong PIN counts as incorrect pin.
Firefly version
1.3.2
OS details
Windows 10 PRO
Can the issue be reliably reproduced?
Yes
Steps to reproduce the issue
1.open Firefly
2.click on wallet
3.input PIN
...
Expected behaviour
After following steps would expect to login.
Actual behaviour
No action after inputting pin program does nothing at all. Does not list as wrong PIN
Errors
no errors listed

Same issue for me

Chlarkin1 Do you still the same issue? I hope someone would help us with this. This is pretty worrying not to be able to handle my wallet for over a week.

Hi, I used Windows 10 system configuration restore to solve the problem. I took the computer back to the day that everything was working and firefly also started working again.
Raf
Inviato da iPhone

Il giorno 11 gen 2022, alle ore 12:27, juhrau @.***> ha scritto:

Chlarkin1 Do you still the same issue? I hope someone would help us with this. This is pretty worrying not to be able to handle my wallet for over a week.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.

I tried same but no change. Thank you anyway.

I got help from Discord. I created new account and used my .stronghold backup file and password. Now it's working.

Ok good!!
Hi
Raf
Inviato da iPhone

Il giorno 12 gen 2022, alle ore 13:37, juhrau @.***> ha scritto:

I got help from Discord. I created new account and used my .stronghold backup file and password. Now it's working.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you commented.

Closing issue due to inactivity, please upgrade to the latest app version and reopen the issue if the problem persists 🌷


Automated Package Upgrade

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


CAMEL-20218: Use a single Kafka consumer with assign() call

Use a single Kafka consumer with assign() call to consume all cached entries from the beginning of the Kafka topic.
Description
The existing implementation of KafkaIdempotentRepository may fail to restore the cached entries on application restart. This implementation is based on KafkaConsumer.subscribe() method and KafkaConsumer rebalancing. It navigates the Kafka consumer to the beginning of the Kafka topic using ConsumerRebalanceListener.onPartitionsAssigned() callback. However, the rebalancing occurs asynchronously. Therefore there is a chance, that first few calls to KafkaConsumer.poll() will be done before the consumer gets navigated to the beginning of the KafkaTopic. Hence, KafkaConsumer gets zero records from the Kafka topic and KafkaIdempotentRepository gets initialized with empty cache. With empty cache, KafkaIdempotentRepository re-consumes the entire input without any deduplication.
The proposed implementation does not rely on Kafka consumer groups at all. It has the only KafkaConsumer and explicitly assigns topic partitions using KafkaConsumer.assign() method. Actually KafkaIdempotentRepository needs only one KafkaConsumer - no need to balance the load across many consumers.
The proposed implementation does not need Kafka consumer groupId. The parameter groupId is still accepted by the public API, but ignored. All constructors and methods dealing with groupId are marked as deprecated.
There is no automated test for this change, because the problem can be reproduced only when restarting an application, and the Kafka topic should already have some cached entries. I have tested this fix manually on a real environment. The existing automated tests run successfully.
Target
main
Tracking
https://issues.apache.org/jira/browse/CAMEL-20218
Apache Camel coding standards and style

[ v] I checked that each commit in the pull request has a meaningful subject line and body.

[ v] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

Same modification, but for the 3.22.x branch: https://github.com/apache/camel/pull/12797

Thanks for your contribution @tashoyan


Release check should compare API review against all approved revisions.

This is an interim fix until we have packages versions available for all reviews so that reviews are compared against only corresponding semver ranges.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Notebook completion doesn't auto-open namespace when an open statement exists in a cell below

Repro Steps:
In a notebook, do the following:
Cell 1:
%%qsharp

//  invoke completion here and choose `DumpMachine`

Cell 2:
%%qsharp

open Microsoft.Quantum.Diagnostics;

Expected:
Cell 1:
%%qsharp

open Microsoft.Quantum.Diagnostics;
DumpMachine();

Actual:
Cell 1:
%%qsharp

DumpMachine();

The open statement is not inserted because the completion list thinks we already have an open for this namespace. But since it's in a cell further below, it doesn't apply. This causes DumpMachine to fail to be resolved.
Seen in b454828298fc2f505ce6de437167aacf58df3afd

Given that we can't control the execution order of cells, this is out of scope for us. This would entail the language service knowing the runtime state of the jupyter environment.


Implement Metanorma for OGC documents

Document types
Standards

Standard (with or without Compliance Suite)

Implementation Standard
Abstract Specification
Community Standard
Profile

Best Practices Document

Others

Engineering Report
Discussion Paper
Whitepaper
User Guide
Technical Committee Policy Directive
"Informative/Educational" (?), such as "OGC Reference Model 08-062r7"

Deliverable stages

SWG Work
OAB Review
OGC-NA Review
Public Review
Prepare for Approval
TC Approval to Vote
TC Vote
PC Vote

(Will seek clarification on document stage codes)
Versioning
A semantic versioning is necessary. e.g. 1.0, 1.1, 2.0, etc. Maybe this is the "edition"?
Example deliverables
Despite differences in appearance (now), all of these should be presented according to the standard "Standard" format.
Standard: WCS

Standard: WKT

Non-standard: Testbed

Non-standard: GeoPackage

Non-standard: Ref model

There was nothing I could find on the website that looked clearly like a specification of how to write documents, but there are clearly undocumented conventions, e.g. that there is a Conformance statement between Scope and References, that References are not called Normative References, and that there is no separate Bibliography. The standards, at least, seem to stick close to ISO style.
OGC are going to have to flesh out their information model.

@opoudjis from experience, having specifications for deliverables in standards bodies, is the exception not the norm.
We will have to help them "standardize" the conventions into an information model.

@opoudjis they have a specification and templates provided in this document:
https://portal.opengeospatial.org/files/?artifact_id=40077

OGC TC procedures (and document types): http://docs.opengeospatial.org/pol/05-020r24/05-020r24.html

I am modelling submission date as created-date, which is defined in the metanorma-standoc Readme as "The date on which the first version of the standard was created". This is not updated-date, "The date on which the current version of the standard was updated", nor circulated-date, "The date on which the unpublished standard was last circulated officially as a preprint. For standards, this is associated with the latest transition to a formally defined preparation stage, such as Working Draft or Committee Draft".
@ronaldtse Should I create a new "received-date", for when the document was first received by the standards body, or is created-date good enough, since noone cares about document being created until it is viewed by someone other than the author?

@opoudjis where's the "received-date" specified? I don't see them in the templates...

It doesn't exist yet. That's why I'm asking whether I should make it up, or use created-date.

I see, you mean the method of encoding the "submission date". Can't we just have a "submission-date"? "Submission" can mean so many things -- submission to a stage, submission to the body, submission by the editor...

... Which is precisely why we shouldn't use it. The dates should be clear in meaning if they can be.

Can we find out what OGC mean by it?

Do you want submitted-date, then? And if so, what is the formal definition of submitted-date? The date at which the document is received by the standards body for processing, with no further definition?

In the OGC TC process document, I can see the word “submit” being related to submitting an RFC (a proposal), submitting a “Candidate Standard” and a Full Standard.
I assume that’s the meaning of “submission date”, which is the date a document was processed into the OGC system. So maybe “received-date” is the closest.
We need to seek clarification on it.

https://github.com/opengeospatial/D012-MapML_Engineering_Report shows that the Engineering Report subgroup have a well-defined asciidoctor document header template, which I am making backward compatible with the Asciidoctor processor.
http://external.opengis.org/twiki_public/ERGuidance/WebHome does not have a pointer to this asciidoctor template. Please ensure that OGC provide all comparable asciidoctor templates for their different document types where they exist, with online locations for them (so the readme can reference them). OGC truly do have a document model, they just haven't surfaced it somewhere accessible.

Still need to see what asciidoc templates they already have in place; goes to establishing their document model, and would like our tool to be backward compatible with them.

What does “backwards compatible” mean in this context?

BTW notice that the latest D012 document uses a structure similar to ISO:

1 contains Foreword (amidst other things)
2 are normative references
3 are terms
bibliography is at the end

I was told they want to harmonize their structure with ISO since they often submit to ISO.

What does “backwards compatible” mean in this context?

They can use their current asciidoctor with our gem.

BTW notice that the latest D012 document uses a structure similar to ISO:

Oh,  a lot of their docs are, except that they usually inject a Conformance clause between Scope and References.

But (at least) most of their stuff:

Don't have the metadata document attributes
Have a non-conformant Foreword, don't have a Scope
Annexes are not coded as [appendix]

If we want to be backwards compatible, we will need to infer the document attributes (which is so fuzzy) and from titles.
It'll be easier if we have a new script (a one-time script) that specifically converts their old stuff to the new format?

OGC 10-177r2: Guidance to Editors and Authors of OGC Documents that use the OGC Standards document template.
OGC_Standard_Document_template_guidance_on_use..pdf

Work to go from Standards template:

Placement of preliminary elements in the document preface: Abstract, Keywords, Preface (= Foreword), Submitting Organizations, Submitters. Incorporation into Grammar.
Make Scope, Conformance, References, Terms mandatory for standards
Allow Normative References to be called References.
Indexes of figures, tables, and requirements (although perhaps OGC should be coding that, as an exercise: ISO used to require these, but it has presumably stopped doing so in practice.)

Done preliminary elements .

Done validation mandatory sections for standards. Done naming of references.

Thanks @opoudjis !


Put all lib/testing functions in @go. namespace

Decided that namespacing these functions was a good idea because it makes it clearer which assertions come from and depend upon the core library and those which don't.
Also includes some other minor improvements to existing tests.

Coverage remained the same at 95.154% when pulling 316cfc684060c604d4637fd1b96adcb399651809 on test-namespace into 0d5841b11a5243748b92a2d598869cc2d98ba0cb on master.


Potential speedup: Apply simplify() function to routes in views.py before passing to template

Shapely has a simplify() method that works on MultiLineStrings
It uses this algorithm https://en.wikipedia.org/wiki/Ramer–Douglas–Peucker_algorithm
worst case runtime O(n^2), best case O(n log n)
in tandem with #118

we actually have to use the GEOSGeometry version of the function to preserve data type, but it's the same algo https://docs.djangoproject.com/en/3.1/ref/contrib/gis/geos/#properties

I did some performance testing on the simplify function's tolerance parameter with each city -- it looks like the size in megabytes stays rather flat but rises more quickly around .0001, which incidentally is right where performance starts to become acceptable:

For now, I suggest keeping the tolerance parameter at .00005 (= 5e-5)(= five hundred-thosuandths).
Additional testing may be worthwhile, and we may want to set the parameter differently for different cities.


[QUESTION] How to update application?

The Docs does not describe how to update the application itself.
Is there a way to do so easily with keeping all the data/configs?
I dont know how long I already use this, so I dont even know which version I'm on as this nowhere gets shown..

Statusfy is a dependency, so you can see the version in your project package.json. To update it should be enough to update the same line in the dependencies to the desired version and running npm i, but make sure you test it after updating to see that all continues to work 👍


error when setting a key to json string

ERR Protocol error: expected '$', got '+'
I'm setting door_uuid to {"locked":true,"open":false}
(where uuid is a uuid)

false flag. two issues from my end:

I tried to set RESPValue.simpleString(.init(string: "val") instead of just "val"
apparently if I have a pubsub open, that needs its own redis connection and send commands have to happen on another connection.

@yspreen Great you found the underlying issue. If that issue isn't resolved please reopen this issue.


Fix issue with several graphs

Fix issue with several graphs when some don't contain bullets.
view issue

Merged. Thanks!


run in service or backend when install by command pip install mongo-connector

have no idea to let mongo-connector as a service or run at backend form pip installed.

You can't do it from a pip install, but you can do it from source:
https://github.com/mongodb-labs/mongo-connector/wiki/Installation#installing-as-a-linux-service
Note that this won't work on systems that use systemd.


chore: add comments and errors class

Description
Fix sonarqube issues.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Refactor/improvement

How Has This Been Tested?
unit test
Checklist

[ ] My commit message is conventional
[ ] I have run pre-commit on all files before creating the PR
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I have checked my code and corrected any misspellings

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


Fix header positioning

fixed issue caused by https://github.com/namespace-ee/react-calendar-timeline/pull/252


TODOs before public release?

[x]  reader macros

[x]  implement customizable reader macros
[x]  .\  eval at read time
[x]  _\  drop form
[x]  ` template quote
[x]  , unquote
[x]  ,@ splice
[x]  #\  gensym

[x]  set up Sphinx docs/tests.

[x]  can we doctest using the Hissp REPL?

[x]  setup.py for PyPI
[ ]  automated tests?
[x]  document compiler use

First Hissp preview release has been published!


Changing the title so that it matches the format of other books

@ehumph super minor change here - just doing this so it's consistent when we show it off in the symposium tomorrow :)


If I try to delete volumes that are being used, it stucks on removing

What kind of request is this (question/bug/enhancement/feature request):
bug
Steps to reproduce (least amount of steps as possible):
Launch redis from Catalog apps. By default it will claim 1x8GB disk.
Result:
Then if you try to delete its claimed volume from Volumes menu, it stucks like this:

Removing is logically right. If I delete redis from catalog apps, volume deletion succeeds.
IMHO it shouldn't try to delete volume that's being used or at least should give some warning.

@cjellick is it possible to fix it in backend?
The following comment is from @westlywright

I could be off base here but I think this should be a bug for the API. If the PVC is in a state where is can not be removed shouldn't the delete action fail? Or possibly the state could go to pending-removal? Possibly with a message from the api as to why it can not be removed? Throwing a warning at the user solves the immediate issue of why its not removed but that's just a bandaid.

This is probably an intentional "feature" of k8s but I don't think it needs to be exposed to the user.  IMO the API should remove the delete link on bound PVCs, but if you don't want to do that the UI can override canDelete on PVCs to provide the same behavior.

Ideally, we would fix this in the API, but let's fix it in the UI for now.


[Improvement] Use cache to improve read performance

What would you like to be improved?
In our usage scenario, such as spark, each executor will call get fileset when writing, which brings higher qps requirements on this interface. We found that cache is added to get catalog to improve performance. so we wonder if cache can be added to get fileset, or will this cause any consistency issues?
How should we improve?
Just like the above description, will adding cache in get fileset cause consistency problems? If so, how does get catalog avoid this problem?

@yuqi1129 appreciate it if you can explain this question


Uninstall hin for Sonoma Blocker is wrong

Uninstall hint must be customized for sonoma blocker. This is wrong:

To fully uninstall bigsurblocker, run the following (as root or with sudo):
current_user_uid=$( echo "show State:/Users/ConsoleUser" | scutil | awk '/UID :/ && ! /loginwindow/ { print $3 }' )

launchd_item_path="/Library/LaunchAgents/com.hjuutilainen.bigsurblocker.plist"
launchctl bootout gui/${current_user_uid} "${launchd_item_path}"

rm -f /Library/LaunchAgents/com.hjuutilainen.bigsurblocker.plist
rm -f /usr/local/bin/bigsurblocker

pkgutil --forget com.hjuutilainen.bigsurblocker

I think you might have skipped this line, before scrolling to the bottom:
“The rest of this README is from that original project:”
The uninstaller script is placed in the repo.


Association URLs

Thanks for creating this library!
I'm getting unexpected results from a query on an association. When I do a find on my Event class:
Event.find('1')
I get the expected result, and the API query URL is http://localhost:5000/events/1/
When I query for the related 'bookings' on an event like so:
let event = await Event.find('1')
let bookings = await event.bookings().load()

My bookings contains a collection of bookings, but it contains ALL bookings, not just ones for the associated event. When I look at the Chrome 'Network' tab I see that the URL for the bookings request is http://localhost:5000/bookings/.
I would have expected this query to be http://localhost:5000/events/1/bookings/.
Is there a configuration parameter I'm missing? Or is my API routing not configured in a manner that conforms to the standard that the library is expecting? I'm using Ruby on Rails for my back-end if that helps any.
Event.js
import MyResources from './MyResources'

class Event extends MyResources.Base {
  static define () {
    this.className = 'Event'
    this.attributes(
      'firstName',
      'lastName',
      'streetAddress',
      'city',
      'province',
      'country',
      'email',
      'phone'
    )
    this.hasMany('bookings')
  }
}

export default MyResources.createResource(Event)

Booking.js
import MyResources from './MyResources'

class Booking extends MyResources.Base {
  static define () {
    this.className = 'Booking'
    this.attributes('id', 'eventId', 'customerId')
    this.belongsTo('event')
  }
}

export default MyResources.createResource(Booking)

What is the full JSON response from the server when you request Event.find(1)