Ansible distribution conditional stops me from executing on a 14.04 box.

Hi, i'm executing the role against a 14.04 box and got this:
TASK: [angstwad.docker_ubuntu | Fail if not a new release of Ubuntu] **********
fatal: [localhost] => error while evaluating conditional: ansible_distribution_version|version_compare(12.04, '<') or ansible_distribution_version|version_compare(12.10, '=')

I have use debug to print my box ansible_distribution version:
TASK: [debug var={{ansible_distribution_version}}] ****************************
ok: [localhost] => {
    "14.04": "14.04"
}

$>ansible --version
ansible 1.5.4

#43 duplicate


_ViewImport.cshtml changes do not immediately show in affected Razor files

Today when making changes in _ViewImport.cshtml changes are not reflected in the affected Razor files until a full re-parse occurs.
For instance when adding @addTagHelperPrefix "th:" TagHelpers that existed prior to the directive still show up in the editor until you start typing new HTML elements, new C# bits, delete anything, or close/reopen cshtml file.

b871172dd07bc0a65bfabd5cb0a76b97a463ca73


Add CloudFlare KV package

Free Plan

Related

https://github.com/Kikobeats/cloudflare-workers-kv

Close because HTTP API provided by CloudFlare Workers is too slow


FR: Add result getter to ServiceExtensionResponse for testing service extensions

Currently, it is not possible to test the output of a service extension handler because the ServiceExtensionResponse class has no public getters.
It would be helpful to have access to result in tests.

/cc @bkonyi

Up for review here.


Update page.cpp

Issue: Unsafe Type Conversion
Hello! This is Jason from Microsoft.
Direct type casting of PAGE_DIRECTORY and PAGE_DIRECTORY + PAGE_SIZE to (int *) may lead to undefined behavior if these values are incorrect or not properly aligned. Using such direct casts lacks type safety and may cause runtime errors.
Suggestion:
Instead of directly casting, use uintptr_t or a similar unsigned integer type for handling addresses. This ensures proper type safety and makes the code more robust.
Proposed Improvement:
uintptr_t page_directory_base = PAGE_DIRECTORY;
uintptr_t page_table_base = PAGE_DIRECTORY + PAGE_SIZE;
int *page_directory = (int *)page_directory_base;
int *page_table = (int *)page_table_base;

This change enhances code safety and readability by separating the raw address handling from pointer assignment.

Thank you for your consideration!


Fixes for #31, #32, #33

i think you need to update your PR, since "There are no new changes to show."

The tests are failing due to lint error: https://travis-ci.org/Trax-air/swagger-parser/jobs/178064560

Fix #31 #32 #33

Line 872 and 875 are too long, max length is 120

Good the tests are green :smile:
Can you add one test for each of your fix (eg for each of your issues) ?

I am not sure I am familiar with the test strategy. Do I need to modify some data (swagger) sample to introduce test cases that caused bugs? Or I can do that at UT level?

I think the best option is to call the functions you just fix with args that was failing before your test, and check that is working now. You can put this either in an already existing test case (function in this file: https://github.com/Trax-air/swagger-parser/blob/master/tests/test_swagger_parser.py) if one is already testing the function, or create a new test case.

we might want to consider some refactoring of the code to reduce the number of indentation levels and enhance readability

Sorry for the really long delay. I will check this soon :smiley:

Running into some errors which would be solved with this PR. Would be awesome to get this to master!

I resolved the conflict and kept the logging line for safety

@philippeluickx I'm merging this.
@cyprieng I'll fix the tests in a next PR to drop python2.6 support. I'll make a major version bump then


event webhook heiarchy does not show deferred event

Currently, the event webhook's event hierarchy does not display where the deferred event sits. The deferred event should be placed with delivered and bounced.

https://sendgrid.com/docs/images/event_1.png
https://sendgrid.com/docs/API_Reference/Webhooks/event.html

This has been rebuilt and updated!


Google Maps Options. Merge the default options with incoming

We are getting an error when we do not provide a zoom property and this will solve the problem.
ERROR TypeError: Cannot read property 'zoom' of null     at Bu (map.js:3)     at HTMLDivElement.<anonymous> (map.js:3)     at ZoneDelegate.invokeTask (zone-evergreen.js:391)     at Object.onInvokeTask (core.js:39680)     at ZoneDelegate.invokeTask (zone-evergreen.js:390)     at Zone.runTask (zone-evergreen.js:168)     at ZoneTask.invokeTask [as invoke] (zone-evergreen.js:465)     at invokeTask (zone-evergreen.js:1603)     at HTMLDivElement.globalZoneAwareCallback

Closing since this PR seems obsolete; @Yavorss feel free to open a new PR if you want to revisit.


Only lists .nammodels and not .nam

Only lists .nammodels and not .nam

That's strange.
Here it list all my *.nam files without issue. I didn't know that there are files out there with *.nammodel extension. Were do you get them?

Ah, now I know what you talk about, You mean the internal UI from Reaper. I didn't know that Reaper supports the mod file extension. I've fixed that now so that the internal Reaper UI shows nam files in the browser.

Great! I will try it out!

Itnow works with NAM files but IR file function lists *.audio and not .wav or actual audio files.

Ups, yes. Thanks for the hint. It's fixed now as well.


Option to remove '@' sign

Is there a way I can remove the at '@' sign in the mention span once we select an item in from the dropdown?

You can use CSS to hide it:
.ql-mention-at-sign {
  display:none;
}

Great, thank you!

Seems not to work - I don't see where that's defined.

You can set showDenotationChar to false.

Tried that - still shows the @
        mention: {
            allowedChars: /^[a-zA-Z0-9_]*$/,
            mentionDenotationChars: ["@"],
            showDenotationChar: false,
            source: function (searchTerm, renderList, mentionChar) {
                let values;

                if (mentionChar === "@") {
                    values = atValues;
                } else {
                    values = hashValues;
                }

                if (searchTerm.length === 0) {
                    renderList(values, searchTerm);
                } else {
                    const matches = [];
                    for (let i = 0; i < values.length; i++)
                        if (
                            ~values[i].value.toLowerCase().indexOf(searchTerm.toLowerCase())
                        )
                            matches.push(values[i]);
                    renderList(matches, searchTerm);
                }
            }
        },

I'm opening with quill.getModule('mention').openMenu('@')
This is what I get:


[BUG] Error message "Cannot redefine property: isVisible"

foundry 9.242
dnd5e 1.5.6
conditional visibility 0.3.1

patch the error on 0.3.2

This issue is closed, because the new series 0.4.X is just to different, checkout the last version of the module.


RHOAIENG-11299: Self signed cert configmap not being create in e2e tests

Description
Stops the failing of mounting the config map needed for self signed certs by creating it in setup when a value is given.
How Has This Been Tested?

Running e2e tests and seeing new config map mounted to the fetch tasks

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work

/lgtm
Looks good to me. Thanks @jackdelahunt

/retest

Rebase is needed

/lgtm


Create InstallationNotes

Instructions for a temporary fix allowing us to use Zeppelin to import large JSON files.

Thank you for this @valeriearoth


SetLastError as true for Symbolic and Hard Link native APIs

PR Summary
Ensure the native PInvoke call has SetLastError in the LibraryImport attribuute. We use the last error to determine the correct fully qualified error id.
PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[ ] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[ ] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

Could you please point a code where we use the last error? I don't see using GetLastWin32Error/GetLastError.

Could you please point a code where we use the last error? I don't see using GetLastWin32Error/GetLastError.

Yeah it's called here


I can't use your train file to generate the model. it looks like your code does not string like "nextword|:"

how to use bigram feature
like in CRF++ we write B in template file


separate blazegraph uploads from frontend returns

the idea would be to allow the main analysis -> display result to work even if blazegraph is down
main concern with implementing this is the use of spfyids

woops. meant to put this under /backend https://github.com/superphy/backend/issues/233


Upgrade rapidjson to master 2019-06-27

No, but while debugging an issue that I thought was related to this library I noticed that we hadn't updated it in a year or so, and there were a few improvements done to this codebase.


[Feature Request] Limit accidental "file opens" from Debug Screen File Stack

Description
The React Native Error Screen "The Red Screen of Death" has a feature where you can click on any of the items in the Call Stack to open the relevant file, for example, if I click on the top row of the File Stack below (employeeCreate) it will open employeeActions.js:

This is sometimes a very useful feature. What is NOT useful at all, is the number of times I have clicked in the Simulator to gain focus, and accidently opened a file I didn't want, which more often than not, opens up a massive file, and chokes my Text Editor to death (I am using Atom). If the file I accidently opened is large enough, Atom will freeze, sometimes long enough that I am forced to "Force Quit" and reboot the editor. This gets very frustrating after it happens a few times in a row.
The problem, as I see it, is that the hit space to open a file is just too large. By making the ENTIRE ROW clickable, you are making it too easy to accidentally open a file when you click on the Simulator to gain focus.
Reproduction
Throw any error from any file in your React Native project to produce the "Red Screen of Death" in the Simulator. Then, hover over any of the rows containing File references, and click anywhere on that row. A file will open:

Solution
Instead of making 90% of the simulator ready to launch a file, just make the filenames themselves capable of launching the file.
So you can turn this (where green is the "Safe Zone" to click without opening a file):

Into this (where the green is the "Safe Zone" to click without opening a file, and the pink is the click area for opening a file):

Overall I feel that I am going to need to gain focus on the Simulator more often than I am going to launch a file from the Simulator.

We're cutting down on the number of outstanding issues, in order to allow us to focus. I'm closing this issue because it has been open for over 60 days with no activity. If you think it should still be opened let us know why. PRs are always welcome.


Vectorized RNG

Addresses #4430
Only adds Uniform Random capability for now:

[x] SGMatrix
[x] SGVector
[x] Benchmark

Edit: Will add Normal random in a different PR.

@karlnapf I've added the task to benchmark - definitely intend to and will look into it after adding the SGVector version.

@iglesias maybe you also have comments

Nice work @saatvikshah1994, thank you.
I was wondering, would you like to experiment with concepts to implement what you have done here with traits and sfinae in sgvector and sgmatrix? If you're interested we can discuss further what would be the possible advantages and gains. This could be done in a separate branch and discussed in another pull request.

Will merge this tomorrow if no more input from @lisitsyn @vigsterkr thx!

Hold your horses plz needs some changes....

On 4 Jan 2019, at 17:22, Heiko Strathmann<EMAIL_ADDRESS>wrote:
Will merge this tomorrow if no more input from @lisitsyn @vigsterkr thx!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

Nice work @saatvikshah1994, thank you.
I was wondering, would you like to experiment with concepts to implement what you have done here with traits and sfinae in sgvector and sgmatrix? If you're interested we can discuss further what would be the possible advantages and gains. This could be done in a separate branch and discussed in another pull request.

Sure, I'm game! Do you want me to create an issue and we can discuss there? Or on IRC first?

@saatvikshah1994 thanks for the ping. The point beside the sees is that the whole design of the random class in shogun is flawed. This is why Viktor mentioned to not put any more effort into it. That said, we could of course still use your updates until we do a change, but the second point is that the uniform random number generator is not really used within shogun. This doesn't mean the PR is not useful. Quite the opposite is true, it is a really nice demonstration on how to gain speedups with vectorization as well as using better design patterns, so it is very useful for future reference. As said, for now, check the feature branch on how we can improve the random number generation in shogun in general. We should get the same ideas that you developed you in this PR into the feature branch and then put the efforts into pushing this forward rather than tuning deprecated designs.

@vigsterkr is this equivalent to what @theartful did?

Closing due as it is outdated and also we had some discussions whether we want to use this old design in the past


fix: update usage of SetGradient in Classic to how it is in Wrath and Dragonflight

SetGradient now expects TextureBase:SetGradient(orientation, minColor, maxColor) in Classic 1.14.4 as it is in Wrath and Dragonflight.

Thanks!

No problem. I love this addon 😊


Implement setting Keys and Certificates

Implement

Setting keys with certificate chains
Setting certificates
Store to file/stream

setting key and certificate implemented with https://github.com/KarlScheibelhofer/java-crypto-tools/commit/b8316e11a08e35acd05ccc7d276877d316b2e2c6


Typo in Azure disaster recovery table

In the matching table for disaster recovery in the first row the Paired Region "Australia Soutche(a)st" has a typo (a is missing).

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

ID: 1aa8c1f7-dc55-7129-95be-687a0b11a636
Version Independent ID: 4a09b58a-9fd2-69fc-0348-e6d4d0956a20
Content: Planning for an Azure File Sync deployment
Content Source: articles/storage/files/storage-sync-files-planning.md
Service: storage
GitHub Login: @wmgries
Microsoft Alias: wgries

@Neratio Thank you for the valuable feedback,we are investigating the issue.

I submitted the fix, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Adds the app tree view to the controller overview page

Description
It uses web components to render the tree - and only the tree - of applications. however, a more complex structure using pydantic has been put in place to deal with the other bits and pieces we want to render in the future (host and detector) easily. That will be used in when tackling #230.
As the nesting is unknown upfront, after checking with Pierre, the built of the tree in html is done in the backend using a recursive function.
Naturally, it uses dummy data until #229 is addressed once we have a working controller (after #203).
https://github.com/user-attachments/assets/3513c38f-d808-4d7d-a5de-79280d0e97c0
Fixes #228
Type of change

[ ] Documentation (non-breaking change that adds or improves the documentation)
[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (non-breaking, back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (whatever its nature)

Key checklist

[x] All tests pass (eg. python -m pytest)
[ ] The documentation builds and looks OK (eg. python -m sphinx -b html docs docs/build)
[x] Pre-commit hooks run successfully (eg. pre-commit run --all-files)

Further checks

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added or an issue has been opened to tackle that in the future. (Indicate issue here: # (issue))

Codecov Report
Attention: Patch coverage is 96.80000% with 4 lines in your changes missing coverage. Please review.

Project coverage is 85.46%. Comparing base (37e9449) to head (19b3c5f).
Report is 12 commits behind head on main.

Files with missing lines
Patch %
Lines

controller/controller_interface.py
69.23%
4 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #231      +/-   ##
==========================================
+ Coverage   82.00%   85.46%   +3.46%
==========================================
  Files          31       36       +5
  Lines         400      523     +123
==========================================
+ Hits          328      447     +119
- Misses         72       76       +4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Importing issue

Hello,
I have a problem importing this plug-in.

Which Plugin are you having an issue loading? Also how are you downloading the files from? I just tested loading all of the plugins downloaded from both the Clone or Download and Releases onto a console on V<IP_ADDRESS> and they all loaded correctly. Last I checked they also loaded in <IP_ADDRESS> correctly as well.
Usually if a plugin fails to load its logged in the Add Window >More >System Monitor. Can you send me any errors you are seeing there.

Thanks mate. I downloaded now with Clone and it works perfectly.
You are the best!
Thank you

Ok, Glad it's working.


generate vtk files

vtk files are generated in the GCEED part by this modification. It's much easier to make movies by vtk files when we use paraview. @uemoto1 san, I guess you are so busy. Is it possible for you to check this PR? If not, I'll ask other people.

I'll try to ask @tt-ims san.

@mn2007 san
I will check your pull request. Please wait for a moment.

@mn2007 san
I checked your pull request. Confirmed that vtk files are generated correctly. There is no problem to merge this pull request.


Pasting into the compiler options field gets overridden by the defaults

when we paste something in the "Compiler options" field, it will be overridden by the default settings. only manual typing works there.

Seems to no longer happen


Boyfriend doesn't play special animation

Hi,
so I wanted to make boyfriend play an animation when he hit a note but he's not playing it
Here's the code for a hit:

Here's the code for a miss: (boyfriend doesn't need to play an animation but I added it here if it has anything to do with it)

Here's the code for the note itself:

The dad character can play the animation. It is just boyfriend that can't play the animation.

Where is this code at?

I forgot to mention that it does play the normal animations.
With that i mean it plays the singLeft animation when a left arrow was hit

You're triggering the dodge animation before the normal singing animations, which cancels the dodge animation,
Move it to here:


Multiple issues with the CamSpy.

Portal rendering does not work correctly with the CamSpy, as they will be ignored if too close to the sides of the screen.

The CamSpy's rotation in the inventory is bugged, it will jitter and shake. In testing I saw this not happen with any other item but it may be a global issue.

https://github.com/fgsfdsfgs/perfect_dark/assets/130617241/029c9e44-aace-494b-9137-6c3655d4b3bd

Mouse look does not work with the CamSpy, you are stuck using the arrow keys to look left/right, hover up/down, and WASD to move it.

Mouse look does not work with the CamSpy, you are stuck using the arrow keys to look left/right, hover up/down, and WASD to move it.

Since the controls are not reference anywhere on the page, you don't know how long it took me to figure it out...
For those looking at this, the controls are:
WASD: C-buttons (Selection in menus)
Q: A Button
E: B Button
Left click: Z Button (Accept in menus)
Right click: R Button (Back in menus)
Mouse move: Control stick (ie. look around)
Arrow keys: Control stick
Enter/Return: Start Button
Escape/ESC: unfocus mouse from game
Hopefully controls custumization comes soon.

EyeSpy and the Slayer rockets should now support mouse control. The other issues still not fixed.

I've also had a crash when placing the Camspy. I'm unsure of the credentials to reproduce it at this very moment, but I feel it's related to the portal rendering issue (unloaded rooms when being placed?)

None of these issues occur on the most recent builds, so I'm closing this issue.


Update dependencies

Bumped sequelize version (high severity security alert)
Updated node version of Docker from carbon-alpine to 12.13-alpine
Removed Joi deprecated and unused/extra dependency

I am not sure if any extra upgrades (not just security deps) this repo needs.

Pull Request Test Coverage Report for Build 65

0 of 0   changed or added relevant lines in 0 files are covered.
13 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.4%) to 29.952%

Files with Coverage Reduction
New Missed Lines
%

src/Services/Database.js
13
0.0%

Totals

Change from base Build 62:
-0.4%

Covered Lines:
46

Relevant Lines:
138

💛  - Coveralls


WARNING : root : Unknown stanza interface: nick

Can someone tell me what this warning is? Everything seems to be working.
I am using the current code bas that is here on git hub.
This is what i see in docker:
DEBUG	 Writing ktop_asus43-rwrks .300259_nwws_processor.4905.txt
INFO	 message stanza rcvd from nwws-oi saying... KARX issues RWR valid 2024-12-30T03:00:00Z
DEBUG	 Writing karx_asus43-rwrarx.300259_nwws_processor.4906.txt
WARNING:root:Unknown stanza interface: nick
INFO	 message stanza rcvd from nwws-oi saying... KIND issues RWR valid 2024-12-30T03:00:00Z
DEBUG	 Writing kind_asus43-rwrin .300259_nwws_processor.4907.txt
INFO	 message stanza rcvd from nwws-oi saying... KIND issues Area Forecast Matrices (AFM) valid 2024-12-30T02:58:00Z
DEBUG	 Writing kind_fous53-afmind.300259_nwws_processor.4908.txt
WARNING:root:Unknown stanza interface: nick
INFO	 message stanza rcvd from nwws-oi saying... KMKX issues RWR valid 2024-12-30T03:00:00Z
DEBUG	 Writing kmkx_asus43-rwrwi .300259_nwws_processor.4909.txt
INFO	 message stanza rcvd from nwws-oi saying... KPBZ issues Point Forecast Matrices (PFM) valid 2024-12-30T02:59:00Z
DEBUG	 Writing kpbz_fous51-pfmpbz.300259_nwws_processor.4910.txt
WARNING:root:Unknown stanza interface: nick
INFO	 message stanza rcvd from nwws-oi saying... KGRB issues RWR valid 2024-12-30T03:00:00Z
DEBUG	 Writing kgrb_asus43-rwrgrb.300259_nwws_processor.4911.txt
WARNING:root:Unknown stanza interface: nick
WARNING:root:Unknown stanza interface: nick
WARNING:root:Unknown stanza interface: nick
INFO	 message stanza rcvd from nwws-oi saying... KAKQ issues Area Forecast Discussion (AFD) valid 2024-12-30T02:59:00Z
DEBUG	 Writing kakq_fxus61-afdakq.300300_nwws_processor.4912.txt

I believe it happens because I had an older version of the XMPP library zorn-v/xmpp in composer.json. I just pushed an update that appears to fix the issue. Just run composer update and try to run the script again.
Jim

Hi Jim, Thank you for the reply. I goof however, I should have said I 'm using the Python version and not the php, my fault. I'll give the PHP version a try.
If you're interested I am going to piping the output to mariadb and pulling the data with home assistant to make a better weather dashboard than home assistant provides.


Add initial setup / configuration UI

Almost the entire database is accessible through the gosst-http API, we just need to write a user interface that makes adding calibrations, linkages, setups, etc. possible without issuing HTTP request from the command line.

This required a proper authentication/authorization method, so instead of the static API token, we are now authenticating using JWT (still only on role though). In the meanwhile, the whole frontend has been rewritten using Mithril, which makes creating additional UI elements easier.


Tdr 508 save parent folder name

These changes add functions to add ParentFolder name to the Consignment table. This is done at the same stage in which we save files to the File table in order to keep the File and Consignment table in sync.
The changes include associated tests and a slight change to the test DB to add ChecksumMatches due to PR https://github.com/nationalarchives/tdr-consignment-api/pull/76  not being merged yet. May have to coordinate on which PR should be merged first.

Looks like you have a test failure on Jenkins.
14:48:33  [info] FileServiceSpec:
14:48:33  [info] createFile
14:48:33  [info] - should create a file given correct arguments *** FAILED ***
14:48:33  [info]   org.mockito.exceptions.verification.SmartNullPointerException: You have a NullPointerException here:
14:48:33  [info] -> at uk.gov.nationalarchives.tdr.api.service.FileService.addFile(FileService.scala:25)

Looks like you have a test failure on Jenkins.
14:48:33  [info] FileServiceSpec:
14:48:33  [info] createFile
14:48:33  [info] - should create a file given correct arguments *** FAILED ***
14:48:33  [info]   org.mockito.exceptions.verification.SmartNullPointerException: You have a NullPointerException here:
14:48:33  [info] -> at uk.gov.nationalarchives.tdr.api.service.FileService.addFile(FileService.scala:25)

I managed to fix this and a few ScalaStyle issues, should all be good now :+1:

Looks good, thanks! Is this OK to merge, or were you coordinating that with Sam because there's some overlap with #76?

Awesome, thanks! I'll give him a shout to check what would be best :smile:


Adjust caching to be less aggressive

Expected Behavior
When you save a link, subsequent saves from the same page should be treated as a unique save.
Actual Behavior
At present when you make a subsequent save you will get cached versions of recommendations and tag overflow.
Steps to Reproduce the Problem

Save a link (either via right click or browser action)
Add tags and observe recommendations
Save another link from the same page (so long as it is different than the original save)
Observe the cached data.

Specifications

Version: <IP_ADDRESS>
Browser: Chrome
Operating System: ALL

This has been addressed in the latest production release. https://github.com/Pocket/extension-save-to-pocket/pull/37


TAJO-1844: Eliminate explicit diamond expressions

Eliminate explicit diamond expressions in whole project.

+1 LGTM

Could you close this issue? I missed to add the close command to the commit log.
https://git-wip-us.apache.org/repos/asf?p=tajo.git;a=commit;h=aa699c70943f7563e77f5828683cd0399998a0f9

I find some expression was remained till now. so remove again.
I really sorry about your inconvenient caused by my mistake.

This ticket is already committed, so you should make another ticket on Jira for remaining issues.

@jihoonson Thanks for your reply. I'll do that.


luci-app-dhcp: 'srvhosts' tab added.

DHCP Help does not document srv-host, but they are available in dnsmasq.
This effectively implements SRV records within DNS.
Tested on 22.03.2
Signed-off-by: Paul Dee<EMAIL_ADDRESS>

Please use luci-mod-network:  as commit subject prefix. Also I think the additional record type tabs should be added after Hostnames as they'll be less commonly used.

Merged via 08a089740af0c9734169f6dc206beec09fe80324


adding dark mode toggle feature

many users prefer dark mode everywhere , this feature will enhance user experience .

okay so my plan is to :

I will place a simple toggle button inside the  to allow users to switch between light and dark modes.
Instead of using an external CSS file, I’ll apply inline styles directly within the HTML or through JavaScript. This will allow us to dynamically change the styles when toggling between modes.
I’ll write JavaScript code that checks the current mode (light or dark) and switches the inline styles accordingly.

the impact you can expect is :

The toggle button will allow users to switch between light and dark themes on the fly.
Since I'm not using external CSS, it will reduce dependency on additional files, but might result in a longer inline code block.
By embedding styles directly within JavaScript, it becomes easy to manage and implement without external dependencies, keeping the code base lightweight.

Steps for that will be

Add a Button: Insert a button in the HTML body.
Define Light and Dark Styles: Set inline styles in JavaScript for both light and dark modes.
Add Event Listener: The button will toggle between modes by switching the styles dynamically.

i bilieve that this will create a flexible and user-friendly theme-switching experience without external CSS.

@bitbroke website is already in dark mode (we've used dark gray or black color), your suggestion doesn't look unique.
It'd be great if you can suggest some new feature to add, like some beautiful UI's.


Add license

Hi there.
I was hoping you would add a FOSS license such as Apache 2.0 or MIT to the project. I'd like to use your Dockerfiles but cannot while they are unlicensed.
Thanks,
Nathan.

sorry for delay, way absent. ofcourse you can use it, will add license.


Edit: BT Sport 3 [UK]

Channel ID (required)
BTSport3.uk
Channel Name
BT Sport 3
Alternative Names
No response
Network
No response
Owners
No response
Country
UK
Subdivision
No response
City
No response
Broadcast Area
No response
Languages
No response
Categories
No response
NSFW
FALSE
Launched
No response
Closed
No response
Replaced By
No response
Website
https://www.tntsports.co.uk/
Logo
https://img01.products.bt.co.uk/content/dam/bt/portal/images/logos/tnt/TNTSports3_Logo_White_RGB_Logo.png
Notes
"BT Sport 3" has been rebranded and is now called "TNT Sports 3"

Please update your request:

the size of the logo should not exceed 512x512px

https://github.com/iptv-org/database/blob/master/CONTRIBUTING.md#channels

Changes made


Need safe span getter on TLVReader

From a review comment:

We should (followup, obviously) think about a better/safer "get the bytes" API on TLVReader, which takes a "buffer + size" and outputs "bytes and length".  Most simply, taking MutableByteSpan & or something.

Originally posted by @bzbarsky-apple in https://github.com/project-chip/connectedhomeip/pull/8929#discussion_r688087019

Done by TLVReader::Get()


Grails 7

Hi @bkoehm
Thank you for your contribution. I've updated the build.yml to include publishing the snapshot to https://oss.sonatype.org/content/repositories/snapshots/
See ya.


MODULE_TOP

For new S4 HANA tcodes how to get the MODULE_TOP & MODULE_TOP_DESC.
Kindly provide your answer at the earliest.

S4HANA will be re-designed and published to site s4hana.org (in progress)

Yes, you are correct, we need to add S4HANA support.
A new site at s4hana.org is being created and its still in progress.
The reason we have to create a new site is there are a lot of changes in S4


compact: Use sync-delay only for fresh blocks. Refactored halt, retry logic.

Currently, we have a similar issue as with prom 2.2.0 compaction.
< Critical fix >
For newly uploaded compacted blocks we need to assume strong consistency for object storage.
If not, we need to address it somehow with file locks or something like that. Basically, compaction is not safe to be planned without whole overview. (except fresh blocks - these can be safely excluded)
GCS has strong write-read consistency: https://cloud.google.com/storage/docs/consistency
S3 has weird consistency, but seems to be write-read for newly created objects:  https://codeburst.io/quick-explanation-of-the-s3-consistency-model-6c9f325e3f82
Signed-off-by: Bartek Plotka<EMAIL_ADDRESS>

@Bplotka: Can you please give an example scenario under which the sync delay causes overlapping blocks per #267?

AFAIK this is totally the same effect as in issue https://github.com/prometheus/prometheus/issues/3943
With prom 2.2.0 issue we were ignoring the newest block, but we wanted the freshest block. This caused wrong compaction as tested by my local tests here:  https://github.com/prometheus/tsdb/pull/299/files#diff-ac389a623f099fc727fc1f3edca6853fR291
I don't know how we missed that, but with sync-delay we introduced exactly same issue. We are ignoring newest blocks., so even these newly compacted for older time ranges. That resulted in compaction wrongly assuming that the block is not there, thus creating one without the recently created and when all of them are synced (after delay) we have overlapping blocks.
I even wrote repair job for that here: https://github.com/improbable-eng/thanos/pull/280 because I was required to repair that so many times until I found this.
This fix is the same as fix for 2.2.0 issue -> we ignore (delay) only fresh blocks. Also, I return strictly errors everywhere in our compaction. The outcome for all my issues so far, is that compaction needs to be aware of the exact state of the bucket for each compaction groups. It can never miss any block (even temporarly), because that leads to unexpected compactions, introducing overlaps and then missing data in worst cases. (before our overlaps check pre and post compaction).

I'm not sure that the object listing consistency and the sync delay we introduced are about quite the same thing.
A block consists of multiple objects and there are obviously no transactions. So you'll still see objects of the same block popping up one after another. The sync delay was meant to address this – and ideally also any eventual consistency S3 may have.
Overall though, that should just improve the general success rate but anything we do should ideally just deal with failures introduced if that didn't work. By retrying later and not messing up existing data.

I don't know how we missed that, but with sync-delay we introduced exactly same issue. We are ignoring newest blocks., so even these newly compacted for older time ranges. That resulted in compaction wrongly assuming that the block is not there, thus creating one without the recently created and when all of them are synced (after delay) we have overlapping blocks.

I don't follow. The compactor will only compact a certain range, if there's data at the start and end of it. So until we see the rightmost, fresh block, we would not attempt compacting the larger range it includes.
This should not trigger things as you described here. Am I missing something?

I'm not sure that the object listing consistency and the sync delay we introduced are about quite the same thing.
A block consists of multiple objects and there are obviously no transactions.

All true, but this is relevant for freshblocks only. Compacted ones are always uploaded sequentially to the sync, so we are safe on that (no need for transaction). However if we don't have strong consistency... It can still cause problems. But if I understand it right, S3 have it as well, if you are using file path for the fire time.

If you remove the sync delay from fresh blocks, we will attempt to compact them before all their files are properly uploaded.

Yes. That's why I don't change anything for freshblocks. See even my PR title: Use sync-delay only for fresh blocks. (:

I don't follow. The compactor will only compact a certain range, if there's data at the start and end of it. So until we see the rightmost, fresh block, we would not attempt compacting the larger range it includes.

You are right. But it can happen that rightmost is accessible (it's ulid is older than sync delay), but something we recently compacted, that is part of the certain range is not (ulid newer than freshblock and still within sync delay).

Compacted ones are always uploaded sequentially to the sync, so we are safe on that (no need for transaction)

That only holds true if you assume object listing consistent, which we aren't fully sure about in S3 AFAIK? Better to not assume it I'd say. Especially since there may be other backends that just implement the S3 API.
You can totally upload something, then go to the next compactor step but still don't have (all) the objects actually show up.

You are right. But it can happen that rightmost is accessible (it's ulid is older than sync delay), but something we recently compacted, that is part of the certain range is not (ulid newer than freshblock and still within sync delay).

Right, that makes sense.
Would it help if we sync everything but then skip a group if any of its blocks is too young?

Right, that makes sense.
Would it help if we sync everything but then skip a group if any of its blocks is too young?

That will work I think, with the risky that some compaction will be blocked forever if you put small min-block-size and larger or even only slightly smaller sync-delay.
However when I add up your previous statement:

That only holds true if you assume object listing consistent, which we aren't fully sure about in S3 AFAIK? Better to not assume it I'd say

"sync group + skip if there is too young" is not enough then. With eventual consistency, we are sure about nothing. I can imagine the situation where you upload compacted thing and even wait for 2h and still block not propagated and is not visible (theoretically still possible but maybe unlikely). So IMO we have few options:

Assume something. GCS has strong consistency. S3 IMO has as well, but we are not sure, so let's invest some time to dig it further. In worst case we can add code to make strong consistency happen (poll the block until it is listable? and fully uploaded?)

Set up some consistent storage then additionally (etcd?). We wanted to avoid external deps, but maybe they are not avoidable

Prepare for the worst and do optimistic compaction? Refactor compactor to be aware that we are not really sure that these blocks we have visible now are really ALL of them? It kind of works like that when we add support for vertical compactions (?). But it needs to be investigated more for other consequences.

What do you think? I feel like actually "1st" option might be the simplest now, but maybe the long-term goal
would be the 3rd option.

@mattbostock @fabxc  what we do with this?
I feel pretty confident with it - compactor with all my fixes in these PRs works pretty stably.
I added issue https://github.com/improbable-eng/thanos/issues/298 to continue our discussion related to consistency assumptions, but IMO this fix is critical and needs to be merged ASAP.


updated dependencies to latest possible for python 3.6

@chrisantonellis, I've updated all of the dependencies + changed few things beyond that. Let me know if changes make sense


Exception Microsoft.Quantum.Simulator.Runtime.dll

Hi,
I followed the instructions here
https://www.microsoft.com/net/download/linux-package-manager/ubuntu18-04/sdk-current
https://docs.microsoft.com/en-us/quantum/install-guide/command-line?view=qsharp-preview
but had issue on various linux versions (you can reproduce with ubuntu:18.04 or centos:7 from dockerhub) with the following error
Unhandled Exception: System.DllNotFoundException: Unable to load shared library 'Microsoft.Quantum.Simulator.Runtime.dll' or one of its dependencies. In order to help diagnose loading problems, consider setting the LD_DEBUG environment variable: libMicrosoft.Quan
tum.Simulator.Runtime.dll: cannot open shared object file: No such file or directory
   at Microsoft.Quantum.Simulation.Simulators.QuantumSimulator.Init()
   at Microsoft.Quantum.Simulation.Simulators.QuantumSimulator..ctor(Boolean throwOnReleasingQubitsNotInZeroState, Nullable`1 randomNumberGeneratorSeed, Boolean disableBorrowing)
   at Microsoft.Quantum.Examples.Teleportation.Program.Main(String[] args) in /Quantum/Samples/Teleportation/Program.cs:line 13

The following SO issue help me fixing it (https://stackoverflow.com/questions/49579153/unable-to-load-dll-microsoft-quantum-simulator-runtime-dll) by installing libgomp1
Maybe there is a way to improve the framework or mention this dependency somewhere in the doc.

Thank you for reporting this!
I've transferred this issue to the documentation repo https://github.com/MicrosoftDocs/quantum-docs-pr/issues/335, since it's not really something that can be addressed on the katas project level.


Add support to React Native

Any interest or thoughts about porting this over to react-native or making it universal?  Possibly by using something like react-native-web https://github.com/necolas/react-native-web
I'd be willing to help with the transition if the maintainers were game.

I think that would be a tough task, as there is a lot of really DOM specific code in here. It'd also be tough to maintain since we'd have to keep up to date with the interop layer, while only really using the web version. I do like the idea in general, but I don't think it makes sense right now.
Thanks for the offer to help though!

Hey - anyone ever give more thought to this / take a stab at implementing it?

https://github.com/llotheo/react-native-big-calendar


Remove some redundant checks

Hi, thanks for the pull request!
This library is following the recommendation to not commit a Cargo.lock file for libraries.  Can you explain why you committed it here?

I added Cargo.lock to cache target dir in github actions. But I guess that is not
necessary for small crate like this one. So it is removed.


[FEATURE] Add additional validations for role mappings

Is your feature request related to a problem?
Outcome of this PR https://github.com/opensearch-project/security/pull/3123.
Missing validations in role mappings:
 - users JSON property. There is no validation in place for the list of users that exist in the system. As a result, it is possible to add non-existing users to any role mapping.
 - hosts there are not clear rules around this property what it should contain.
It looks like hosts functionality has never been finished since OSD hasnt has possibility to add hosts.
What solution would you like?

Add check that users exist i the system
define clear rules how hosts should look like and add validation for this property.

What alternatives have you considered?
If we do not use hosts remove this property or re-implement it.

Linking related https://github.com/opensearch-project/security/issues/1852

Configuration validation can create some problems for us, in its current form we don't have a required order of declarations - this could create backwards capability trouble
Calling the APIs to create a user, role, and rolemapping the order doesn't matter.  However, if we start enforcing this it gets more interesting for our clients.  IMO that feels like a optional feature we could enable ?validateReference=true and then make it the default in a major version bump.


Whishlist from Numa.jl

List of enhancements in this library that would be useful in Numa.jl

[ ] export RefCell at the top level

This issue has become obsolete. Closing it.


fix: Fixing responsiveness of the contents in tools page

Description

Fixing the responsiveness of the content of tools page.
Earlier it was - Attached Image
After fixing -

Related issue(s)

#664

Idk why Lighthouse Ci linting has failed. Here's the working preview - https://deploy-preview-667--asyncapi-website.netlify.app/tools/generator

@helper-uttam Lighthouse failed as it was waiting for too long on preview generation. I rerun it.
can you please remove changes to package-lock.json from this PR?

@helper-uttam Lighthouse failed as it was waiting for too long on preview generation. I rerun it.
can you please remove changes to package-lock.json from this PR? And also have a look on check failing to validate PR title

Can you please guide me how to remove changes that I've done to package-lock.json , I've tried deleting it but it stills have conflicts.

@helper-uttam Lighthouse failed as it was waiting for too long on preview generation. I rerun it.
can you please remove changes to package-lock.json from this PR? And also have a look on check failing to validate PR title

Can you please guide me on how to remove changes that I've done to package-lock.json, I've tried deleting it but it still have conflicts.

And I've updated the title @derberg

@helper-uttam title is still not correct, have a look at logs of the failing check
regarding package-lock just take the master version and commit to your branch

@helper-uttam title is still not correct, have a look at logs of the failing check
regarding package-lock just take the master version and commit to your branch

thank you for the guidance :) Looking forward to work on other issues


Rerun from last checkpoint

It would be nice to have a simple way of telling parsl to run from the last checkpoint. At the moment the developer has to work out what the last checkpoint file was.

I've made this explicit since at the time the feature was developed it wasn't quite clear what modes users would use besides run from the last available checkpoint. Then there's the question of multiple workflows in the same folder and disambiguation of which checkpoint to use.

Pushing to 0.6.0 pending limited external interest.

Note from today's meeting with @djf604: this feature would be useful in his workflow.

What do we mean by 'last checkpoint'? Currently, this would mean checking for the last checkpoint in the rundir specified by the current script. There are various reasons a user might want to change rundirs-- those reasons may not always be coupled to where they want the cache to be. For me, since one of the exciting features of caching is that it would allow us to save work between runs, it does not make sense to couple the caching to the rundir. I would prefer the cache to be saved in a separate directory. We could enforce a(n adjustable) maximum cache size to control how far back is cached.

There's (slightly buggy) boilerplate code in the documentation to find the last checkpoint from the runinfo directory.
Most immediately, I think that should turn into a helper function inside parsl, so you can say something like:
dfk = parsl.DataFlowKernel(config=config, checkpointFiles = parsl.last_checkpoint())
and expect it to work (including if there are no checkpoints yet...)

The default should be for the current script as doing it for many scripts could be confusing.

I'm not totally convinced by this, but since it is still possible to manually share checkpoints between scripts, I'll go with the consensus view for now.


Feature Request: Better handling of old versions

It would be really nice if crates.io would not index old versions. If you're using an older version it would be great if the user had to click on the older version or search for it on crates.io. Google just always seem to prioritize older versions when I'm looking searching.
If that can't be done, it would be nice if users had an option to be redirected to the latest version of the create. Old versions are just of no use to me. If I'm reading the documentation, it's very likely that I've already updated my version to the latest.

I don't know enough about google here.
But: our sitemap only contains URLs that redirect to the latest version.
From what I understand of your request is that it could be related to the discussion in #1006 (which is unsolved currently)

Not sure, the problem is certainly google related. If I search for warp rust reply, I get

https://www.google.com/search?q=warp+rust+reply

That takes you to

https://docs.rs/warp/0.1.0/warp/reply/index.html

That's almost certainly not what your users want. They're going to want

https://docs.rs/warp/*/warp/reply/index.html

The problem is they may not notice that until after they're a bit late in the game.
Delisting urls that don't have /latest/ or /*/ in their name would seem like a reasonable solution. And allow people to -- from the latest -- read the docs on earlier versions.

I believe #1438 and the continous work of @jsha is the solution here.
We already have much improved search results where we see the latest page when the element still exists in it.
I'm closing this issue.

FWIW, I checked this specific example, and the results for "rust warp reply" on Google are currently:

https://docs.rs/warp/0.1.0/warp/reply/index.html
https://docs.rs/warp/latest/warp/reply/index.html

Not great; we want #2 to be #1, and #1 not to be listed.
I checked the Google Search Console, and Google has recently crawled both of these URLs, and sees the canonical link pointing from #1 to #2. It's just that Google is choosing to disregard the canonical link. Presumably the problem is some combination of:

The two pages are very different (due to the default-collapsed docblock on #1), so Google doesn't want to canonicalize one to the other.
#1 has a longer history of being crawled, so may be seen as more reputable.

I'm hopeful that over time, Google will establish a preference for the /latest/ URLs, since "longer history of being crawled" will be less relevant, but at least for now this is an example where canonicalization is not doing the trick. Still, I think it's reasonable to close this issue and keep tracking progress under #1438.


Build Failed (tried: master and tag: 0.7.2)

Tried to build the project but couldn't, what I did:
Build error in error.log below
#####################
git checkout 0.7.2
./autogen.sh
./configure > configure.log
make
#####################
Linux: Gentoo
error.log
configure.log

It appears you are missing libxpm, load it and you should no longer have a problem.  I suppose configure should error out when it is not present...

Build got further along after installing libXpm, after that got a link error:
adwm.c:6258: error: undefined reference to 'getappl'
adwm.c:5577: error: undefined reference to 'removeappl'
Got it to compile after installing startup-notification, which also seem to also be mandatory

I use the following dependencies ffor the Archlinux build:
libxft libxrandr libxinerama libxpm startup-notification libxcomposite libxdamage

I also add the following for default key bindings:
xdg-launch xde-ctools xde-menu xde-session


Issue with Cyrillic characters

Truncating Cyrillic characters does not work. Trying to truncate a mixed paragraph (cyr+lat), and the truncated text only shows the Latin characters. Upon clicking 'show more' the full text is displayed. Same goes if the full text is Cyrillic only: the truncated text shows nothing, and full text on 'show more'.

Thanks for catching it, I'll check it out.

Tnx for fixing this so quick.


[PQL] added hash functions(sha-1, sha-256, md5, etc)

Description
This PR added hash transform functions(sha-1, sha-256, md5, etc). This only works with bytes column
Upgrade Notes
Does this PR prevent a zero down-time upgrade? (Assume upgrade order: Controller, Broker, Server, Minion)

[ ] Yes (Please label as backward-incompat, and complete the section below on Release Notes)

Does this PR fix a zero-downtime upgrade introduced earlier?

[ ] Yes (Please label this as backward-incompat, and complete the section below on Release Notes)

Does this PR otherwise need attention when creating release notes? Things to consider:

New configuration options
Deprecation of configurations
Signature changes to public methods/interfaces
New plugins added or old plugins removed

[ ] Yes (Please label this PR as release-notes and complete the section on Release Notes)

Release Notes
If you have tagged this as either backward-incompat or release-notes,
you MUST add text here that you would like to see appear in release notes of the
next release.
If you have a series of commits adding or enabling a feature, then
add this section only in final commit that marks the feature completed.
Refer to earlier release notes to see examples of text
Documentation
If you have introduced a new feature or configuration, please add it to the documentation as well.
See https://docs.pinot.apache.org/developers/developers-and-contributors/update-document

Codecov Report

Merging #6218 into master will decrease coverage by 2.37%.
The diff coverage is 43.43%.

@@            Coverage Diff             @@
##           master    #6218      +/-   ##
==========================================
- Coverage   66.44%   64.07%   -2.38%
==========================================
  Files        1075     1239     +164
  Lines       54773    58756    +3983
  Branches     8168     8709     +541
==========================================
+ Hits        36396    37646    +1250
- Misses      15700    18363    +2663
- Partials     2677     2747      +70

Flag
Coverage Δ

unittests
64.07% <43.43%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <0.00%> (-79.32%)
:arrow_down:

...ot/broker/broker/AllowAllAccessControlFactory.java
71.42% <ø> (-28.58%)
:arrow_down:

.../helix/BrokerUserDefinedMessageHandlerFactory.java
33.96% <0.00%> (-32.71%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <0.00%> (+9.52%)
:arrow_up:

.../main/java/org/apache/pinot/client/Connection.java
35.55% <0.00%> (-13.29%)
:arrow_down:

...inot/client/JsonAsyncHttpPinotClientTransport.java
10.90% <0.00%> (-51.10%)
:arrow_down:

...not/common/assignment/InstancePartitionsUtils.java
73.80% <ø> (+0.63%)
:arrow_up:

.../apache/pinot/common/exception/QueryException.java
90.27% <ø> (+5.55%)
:arrow_up:

...pinot/common/function/AggregationFunctionType.java
98.27% <ø> (-1.73%)
:arrow_down:

... and 1075 more

Continue to review full report at Codecov.

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


Limit debugfile on SIGHUP

When reading the code and intent around -shrinkdebugfile, this omission seemed odd.

When reading the code and intent around -shrinkdebugfile, this omission seemed odd.

Thinking this could work if it doesn't necessarily have to happen at startup https://github.com/dogecoin/dogecoin/blob/1.14.7-dev/src/init.cpp#L470.
The file could shrink during any of these macros https://github.com/dogecoin/dogecoin/blob/1.14.7-dev/src/util.h#L77C29-L92 but it's only triggered by the SIGHUP signal. So, the user would need to send it via the terminal (e.g., using 'kill') while debugging, right?

We could do it that way, though I'm a little nervous about that. If I have access to your machine but not necessarily your configuration file, logs, wallet, etc then allowing me to send signals to your process to truncate your logs still gives me a nefarious mechanism to manipulate your Core experience.
"But Unky c, why would you give anyone shell access to a Core node even if you've locked down permissions?" That's a good question! That's also a risk. But it's not a strong reason to add a new risk, in my view.

To make it work a user would need to manually send the SIGHUP signal to the process as core doesn’t do this automatically.

Oh, I see what you mean. Yes, that's correct. Do you think this needs further documentation, or is there something else you're looking for in this PR?

Consider updating the -shrinkdebugfile help message here to reflect that shrinking occurs via the SIGHUP signal instead of client startup.

@edtubbs I've reviewed the code paths and updated the help text to indicate that shrinking happens on both startup and SIGHUP.

Ok, cool. I'm building this branch on Ubuntu 22.04 with x86_64. I'll test the switch and let you know how it goes.

Ok here's what I found.. I added some logging to the ShrinkDebugFile() and HandleSIGHUP(int) functions:
diff --git a/src/init.cpp b/src/init.cpp
index 75d6cf44a..94efe886c 100644
--- a/src/init.cpp
+++ b/src/init.cpp
@@ -282,6 +282,7 @@ void HandleSIGTERM(int)

 void HandleSIGHUP(int)
 {
+    LogPrintf("HandleSIGHUP(int) info, handler called.\n");
     fReopenDebugLog = true;
 }

diff --git a/src/util.cpp b/src/util.cpp
index c189b32f3..a1051c710 100644
--- a/src/util.cpp
+++ b/src/util.cpp
@@ -792,7 +792,11 @@ void ShrinkDebugFile()
         }
     }
     else if (file != NULL)
+    {
         fclose(file);
+        LogPrintf("ShrinkDebugFile() info, debug.log within history size.\n");
+    }
+
 }

Here's the output at startup, confirming that ShrinkDebugFile is called successfully during startup:
ed@ed-VirtualBox:/doge/dogecoin$ ./depends/x86_64-linux-gnu/bin/dogecoin-qt -datadir=./data -debug -debuglogfile=./data/debug.log -shrinkdebugfile -printpriority -printtoconsole

2024-07-11 19:32:31
...
2024-07-11 19:32:31 Dogecoin version v<IP_ADDRESS>-c1d8d616b-dirty
2024-07-11 19:32:31 InitParameterInteraction: parameter interaction: -whitelistforcerelay=1 -> setting -whitelistrelay=1
2024-07-11 19:32:31 Assuming ancestors of block e7d4577405223918491477db725a393bcfc349d8ee63b0a4fde23cbfbfd81dea have valid signatures.
2024-07-11 19:32:31 Prune configured to target 2861MiB on disk for block and undo files.
2024-07-11 19:32:31 Using RdRand as an additional entropy source
2024-07-11 19:32:31 ShrinkDebugFile() info, debug.log within history size.

On another run, when sending SIGHUP from another console, it logs the signal but ShrinkDebugFile() isn't called again:
ed@ed-VirtualBox:/doge/dogecoin$ ./depends/x86_64-linux-gnu/bin/dogecoin-qt -datadir=./data -debug -debuglogfile=./data/debug.log -shrinkdebugfile -printpriority -printtoconsole

2024-07-11 19:36:02
...
2024-07-11 19:36:02 Dogecoin version v<IP_ADDRESS>-c1d8d616b-dirty
2024-07-11 19:36:02 InitParameterInteraction: parameter interaction: -whitelistforcerelay=1 -> setting -whitelistrelay=1
2024-07-11 19:36:02 Assuming ancestors of block e7d4577405223918491477db725a393bcfc349d8ee63b0a4fde23cbfbfd81dea have valid signatures.
2024-07-11 19:36:02 Prune configured to target 2861MiB on disk for block and undo files.
2024-07-11 19:36:02 Using RdRand as an additional entropy source
2024-07-11 19:36:02 ShrinkDebugFile() info, debug.log within history size.
...
2024-07-11 19:36:03 Opened LevelDB successfully
2024-07-11 19:36:03 Using obfuscation key for /doge/dogecoin/./data/chainstate: a577229fd8dacfd9
2024-07-11 19:36:08 HandleSIGHUP(int) info, handler called.

Sending the SIGHUP signal from another console:
ed@ed-VirtualBox:~$ ps -a
    PID TTY          TIME CMD
  10538 tty2     00:00:00 gnome-session-b
  12470 pts/0    00:00:01 dogecoin-qt
  12478 pts/1    00:00:00 ps
ed@ed-VirtualBox:~$ kill -HUP 12470

So it makes sense to add ShrinkDebugFile() to the HandleSIGHUP(int) function. After adding that:
 void HandleSIGHUP(int)
 {
+    LogPrintf("HandleSIGHUP(int) info, handler called.\n");
+    ShrinkDebugFile();
     fReopenDebugLog = true;
 }

Then ShrinkDebugFile gets called upon sending the SIGHUP signal as expected:
ed@ed-VirtualBox:/doge/dogecoin$ ./depends/x86_64-linux-gnu/bin/dogecoin-qt -datadir=./data -debug -debuglogfile=./data/debug.log -shrinkdebugfile -printpriority -printtoconsole

2024-07-11 19:46:24
...
2024-07-11 19:46:24 Dogecoin version v<IP_ADDRESS>-c1d8d616b-dirty
2024-07-11 19:46:24 InitParameterInteraction: parameter interaction: -whitelistforcerelay=1 -> setting -whitelistrelay=1
2024-07-11 19:46:25 Assuming ancestors of block e7d4577405223918491477db725a393bcfc349d8ee63b0a4fde23cbfbfd81dea have valid signatures.
2024-07-11 19:46:25 Prune configured to target 2861MiB on disk for block and undo files.
2024-07-11 19:46:25 Using RdRand as an additional entropy source
2024-07-11 19:46:25 ShrinkDebugFile() info, debug.log within history size.
...
2024-07-11 19:46:25 Opened LevelDB successfully
2024-07-11 19:46:25 Using obfuscation key for /doge/dogecoin/./data/chainstate: a577229fd8dacfd9
2024-07-11 19:46:30 HandleSIGHUP(int) info, handler called.
2024-07-11 19:46:30 ShrinkDebugFile() info, debug.log within history size.

So it seems like ShrinkDebugFile needs to be added to HandleSIGHUP(int) so it gets called when receiving the signal as you expect.

I see what you're saying, but I'd like to do this a little differently. If we're okay with the shrinking not happening immediately on receiving the signal, I'd like to avoid doing any work within the signal handler (trying to be cognizant of threading here) and shrink the file on the next print to the debug log.
That's this code:
            // reopen the log file, if requested
            if (fReopenDebugLog) {
                fReopenDebugLog = false;

                if (GetBoolArg("-shrinkdebugfile", !fDebug)) {
                    ShrinkDebugFile();
                }
                fs::path pathDebug = GetDataDir() / "debug.log";
                if (fsbridge::freopen(pathDebug,"a",fileout) != NULL)
                    setbuf(fileout, NULL); // unbuffered
            }

https://github.com/dogecoin/dogecoin/blob/c1d8d616b59b48861213d994dee8a814d12d11f8/src/util.cpp#L324-L334
Does that make sense? So in your testing, you'd have to wait to see another debug line printed (maybe you can force one through an RPC command or wait for a new block or transaction), and then you'll see the file shrink.

Sure, we can shrink the file during the next debug log print with the mutex there for thread safety. Any reason printing to the console and the debug log have to be mutually exclusive checks?

I think they're exclusive because that's how the -printtoconsole CLI option is defined, so changing that might have further implications than they want here. I'm open to code suggestions otherwise, if you see good options.

Agreed, probably best not to change it since it's documented that way. Here's an updated test.. I've added logging with FileWriteStr after ShrinkDebugFile() is called:
diff --git a/src/util.cpp b/src/util.cpp
index c189b32f3..ae1502c58 100644
--- a/src/util.cpp
+++ b/src/util.cpp
@@ -327,6 +327,7 @@ int LogPrintStr(const std::string &str)
                 if (GetBoolArg("-shrinkdebugfile", !fDebug)) {
                     ShrinkDebugFile();
+                    FileWriteStr(LogTimestampStr("Called ShrinkDebugFile()\n"), fileout);
                 }
                 fs::path pathDebug = GetDataDir() / "debug.log";
                 if (fsbridge::freopen(pathDebug,"a",fileout) != NULL)

./depends/x86_64-linux-gnu/bin/dogecoin-qt -datadir=./data -debuglogfile=./data/debug.log -shrinkdebugfile -printpriority

ed@ed-VirtualBox:~$ ps -a
    PID TTY          TIME CMD
   1743 tty2     00:00:00 gnome-session-b
  21419 pts/0    00:00:02 dogecoin-qt
  21428 pts/1    00:00:00 ps
ed@ed-VirtualBox:~$ kill -HUP 21419

2024-07-15 21:10:17 init message: Rewinding blocks...
2024-07-15 21:10:17 Called ShrinkDebugFile()
2024-07-15 21:10:17 init message: Verifying blocks...
2024-07-15 21:10:17 Verifying last 6 blocks at level 3


Python: PUT endpoint not generated from .net swagger spec

What are you generating using Kiota, clients or plugins?
API Client/SDK
In what context or format are you using Kiota?
Windows executable
Client library/SDK language
Python
Describe the bug
I'm able to successfully do some POSTs and GETs with my kiota-generated python client.
I can't get PATCH to work properly (The JSONPatchDocument Operation models seem a bit messed up - but I'll leave that for another post) so I decided to just do a GET & PUT instead for now.
Unfortunately, the PUT endpoint seems to be completely missing - there is no builder function in myEntity.request_builder.py.
Let me know if you have any pointers - otherwise I'll RTFM again and make sure I'm not missing something.
Expected behavior
There should be a way to call my PUT endpoint for the target controller to create/update my entity.
How to reproduce
kiota generate -l python -c MyClientKiota -n client -d "...\MyApi\swagger.json" ^
-o ../../../shared/generated/my_client ^
--additional-data true ^
--include-path /MyEntity/** ^
--clean-output
Open API description file
Unable to provide (company IP) but I can paste an anonymized portion of the Swagger JSPN for the PUT endpoint if that helps - but there's nothing too special about it. In the controller it looks like:
        [HttpPut]
        [ProducesResponseType(StatusCodes.Status200OK)]
        [ProducesResponseType(StatusCodes.Status400BadRequest)]
        public async Task<ActionResult<MyEntity>> UpdateMyEntity(
            [FromBody][Required] MyEntity e,
            [FromQuery] bool anOption = false)
        {

Kiota Version
1.17.0+1eb16cd65853c17179e2dde3ae6098135deacf55
Latest Kiota version known to work for scenario above?(Not required)
No response
Known Workarounds
No response
Configuration
Windows 11 x64
Debug output
Click to expand log
```

</details>

### Other information

_No response_

Hi @thelazydogsback
Thank you for using kiota and for reaching out.
Can you please provide an anonymized OpenAPI description for the operation?

Thanks for the reply, @baywet .
Here you go:
{
  "x-generator": "NSwag v<IP_ADDRESS> (NJsonSchema v<IP_ADDRESS> (Newtonsoft.Json v<IP_ADDRESS>))",
  "openapi": "3.0.0",
  "info": {
    "title": "...",
    "version": "v1"
  },
  "paths": {
      "/MyEntities": {

      "put": {
        "tags": [
          "EEEs"
        ],
        "summary": "Creates or replaces a EEE",
        "description": "Sample request\n            \n            \t\tPUT\n            \t\t{...}",
        "operationId": "EEEs_UpdateEEE",
        "parameters": [
          {
            "name": "anOption",
            "in": "query",
            "schema": {
              "type": "boolean",
              "default": false
            },
            "x-position": 2
          }
        ],
        "requestBody": {
          "x-name": "EEE",
          "description": "EEE information",
          "content": {
            "application/json": {
              "schema": {
                "$ref": "#/components/schemas/EEE"
              }
            }
          },
          "required": true,
          "x-position": 1
        },
        "responses": {
          "200": {
            "description": "EEE information",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/EEE"
                }
              }
            }
          },
          "400": {
            "description": "",
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/ProblemDetails"
                }
              }
            }
          }
        }
      }
    },

I removed the --include to include all the endpoints (incl. ones I don't need) and noticed that other PUT endpoints are being generated.
I think I see the difference. If the path has an id defined in the path such as:
PUT /otherEntity/{id}' then it shows up, but if it is just: PUT /myEntity'
then it will not get generated.
I'm not sure why we're inconsistent here, but FWICT having id in the PUT is common but not required, and the identity of the new resource can just be part of the payload.

Thanks.  It looks like with no inclusions then it works. (Although I get about 80 files I don't need, which isn't the end of the world.) However, using both /MyEntity/** and /MyEntity/* (or just /MyEntity) will not include the endpoint.  I guess I can try excluding everything else.

Tried again -- looks like the double-star/no-star combo does work. (I had mistakenly used /MyEntity/ with the extra slash.)
All good on this one I think - thanks.


Clean up class name and self in calls to super()

PEP 3135 ^1 simplified the syntax for super() in Python 3.0 from:
super(ClassName, self)

to the following very simple and equivalent ^2 syntax:
super()

The current Keras codebase already requires Python 3+, and there's a mix of the two syntax formats above, sometimes both are used in a single file.
This change simplifies the entire code base by cleaning up the remaining explicit uses of the current class name and self and using the cleaner super() syntax everywhere consistently.
Since the new syntax is intended to be a shorthand for the old syntax, this change should have no semantic differences from before.

Thanks for the PR! Please replicate the fixes in the corresponding .md and .ipynb files.

Done.

Thanks for the PR! Please replicate the fixes in the corresponding .md and .ipynb files.

Done.


Can't Unlock Technomancy

I have Time discovered (pulsed a beacon harness using a button as you show in the video guide because I'm trying to do creative mode testing) as well as the three basic elements in case those are required as well. I place down a Detailed Crafter and place a bronze gear in each slot and nothing happens.
Using Crossroads 1.6.3 and Forge 2540

Looks like it's because I unregistered Crossroads gears from the OreDict. Not sure if that's something you want to change?

The unlocking technomancy recipe uses the oreDict bronzeGear, which means any mod's bronze gears should work. That also means removing crossroads bronze gears from the oreDict makes them not work for the unlocking. Working as intended.


Map one entity to many others

Let's say I have the following simplified classes:
public class Room
{
    [Key]
    public int RoomID { get; set; }
}

public class Teacher
{
    [Key]
    public int TeacherID { get; set; }
    public Room Room { get; set; }
}

public class Subject
{
    [Key]
    public int SubjectID { get; set; }
    public Room Room { get; set; }
}

And I have two Rooms Room r1 and Room r2, two Teachers  t1 and t2 and two Subject s1 and s2. I now want to assign r1 to t1, t2 and s1 and r2 to s2. I would do this by
using (var db = new DBContext())
{
    db.Add(r1);
    db.Add(r2);
    db.SaveChanges();

    var rooms = db.Room.ToList(); // rooms now contains both r1 and r2
    db.Add(new Teacher{ Room = r1; }); // t1
    db.Add(new Teacher{ Room = r1; }); // t2
    db.Add(new Subject{ Room = r1; }); // s1
    db.Add(new Subject{ Room = r2; }); // s2
    db.SaveChanges(); // Error shown below
}

The error I get is Inner Exception: "SQLite Error 19: 'FOREIGN KEY constraint failed'.". So I cannot do this. What is the proper declaration here, to achieve what I want?
What I want to happen: t1, t2 and s1 share all the same Room r1. When I change r1, I want the changes to be visible, when I next load t1, t2 and s1
Project Type: C# UWP
EF Core version: "Microsoft.EntityFrameworkCore.Sqlite": "1.0.0", "Microsoft.EntityFrameworkCore.Tools": "1.0.0-preview2-final",
Operating system: win10 x64 build 10586.494
Visual Studio version: VS2015.3

@minimalisticMe I am not able to reproduce what you are seeing with the code you posted. I have posted the complete console application I am using. Can you make appropriate changes to the application so that it throws the exception you are seeing?
public class Room
{
    [Key]
    public int RoomID { get; set; }
}

public class Teacher
{
    [Key]
    public int TeacherID { get; set; }
    public Room Room { get; set; }
}

public class Subject
{
    [Key]
    public int SubjectID { get; set; }
    public Room Room { get; set; }
}

public class DBContext : DbContext
{
    public DbSet<Room> Rooms { get; set; }
    public DbSet<Teacher> Teachers { get; set; }
    public DbSet<Subject> Subjects { get; set; }

    protected override void OnConfiguring(DbContextOptionsBuilder optionsBuilder)
        => optionsBuilder.UseSqlite("Data Source=RoomsDb");
}

public class Program
{
    public static void Main()
    {
        using (var db = new DBContext())
        {
            db.Database.EnsureDeleted();
            db.Database.EnsureCreated();

            var r1 = new Room();
            var r2 = new Room();

            db.Add(r1);
            db.Add(r2);
            db.SaveChanges();

            var rooms = db.Rooms.ToList(); // rooms now contains both r1 and r2
            db.Add(new Teacher
            {
                Room = r1
            }); // t1

            db.Add(new Teacher
            {
                Room = r1
            }); // t2

            db.Add(new Subject {Room = r1}); // s1
            db.Add(new Subject {Room = r2}); // s2

            db.SaveChanges(); // Error shown below
        }
    }
}

@ajcvickers please close this issue. I tried the code on a minimal example too and it worked for me as well.
For people in the future with similar sounding problems:
Turned out, I did some wrong referencing stuff in Teacher class. As I had a parent key, that did not get filled in, as I created an instance of Teacher on it's own instead over the parent.


⚠ Turnos 24x7 has degraded performance

In 269dd9f, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 877 ms

Resolved: Turnos 24x7 performance has improved in 5672d4e.


Yet another cant connect to P2P network + How to change protocol version?

Im attempting to run bitpay-insight against an altcoin base. It happily initially syncs using rpc calls, but it fails to connect to the p2p network and never updates any blocks until it restarts again.
I suspect i need to change the protocol version (its 7003 on this altcoin), but im not sure how. I did try export bitcoind_proto= 7003 but it doesnt seem to work.
Secondly, does anyone have any up to date instructions on installing? Do i need both insight and insight api? My understanding was insight would automatically pull insight-api.
Thanks in advance.

lol at best i connect to the p2p for a day or two.. at worst it takes restarting over and over every 30 seconds, and sometiems takes 20-50 tries to get it to connect and stay (for a few hours/days)
i see this issue all over github..

bitcoind is now running in the same process.


jsonpb: fix unmarshal tests to not compare using strings from proto.MarshalTextString

Current unmarshaling tests calls proto.MarshalTextString and compares the resulting string.  This is not a good comparison since MarshalTextString may not reflect the actual underlying data, e.g. nil vs empty slice.  It's better to just use proto.Equal.

Fyi.  I just realized that proto.Equal may not be the correct solution.  In the case of nil vs empty slice, proto.Equal will not detect the difference.  Anyways, we should still update the tests accordingly and not use MarshalTextString.

Closing this since jsonpb will be completely rewritten with its own set of tests.


Couldn't get real-time updates working in node

I'm trying to receive real-time updates through WebSocket. (node.js v10.4.1)
Do you see any mistakes in here? Callback to userFeed.subscribe is never fired.
 const gsc = gs.connect('x', 'y', '1')

  const userFeed = gsc.feed('user', 'bob')

  const sub = userFeed.subscribe(data => {
    console.log('RECV', data)
  })
  .then(data => {
    console.log('SUBSCRIBED')
  })
  .catch(reason => {
    debugger
  })

  userFeed.addActivity({
    actor: 'user:bob',
    status: 'Hello' + Math.random(),
    verb: 'status',
    object: '1'
  })
  .then(data => {
    console.log('SEND', data)
  })
  .catch(reason => {
    debugger
  })

I suggest moving userFeed.addActivity code inside the 'subscribed' callback; this will remove the add/subscribe race.

Also good to double check that the user feed group for this app has realtime enabled (you can check that on Stream's dashboard)

Ah, thanks a lot. Not sure what I was thinking :) adding the activity after subscription resolves works like a charm 👍


undefined variable should not be syntax error

assert(require"moocscript.core".loadstring("print(a)"))

[string "assert(require"moocscript.core".loadstring("p..."]:1: Error: undefined variable
File: _
Line: 1 (Pos: 6)
Source: print(a)
              ^

This is breaking change, as Lua's std loadstring is not so strict.
And this is in many cases just wrong, because it is just impossible to make such assumption not taking into account function's ENV.

assert(require"moocscript.core".loadstring("print(a)"))

[string "assert(require"moocscript.core".loadstring("p..."]:1: Error: undefined variable
File: _
Line: 1 (Pos: 6)
Source: print(a)
              ^

This is breaking change, as Lua's std loadstring is not so strict. And this is in many cases just wrong, because it is just impossible to make such assumption not taking into account function's ENV.

you are right, mooncake was very strict about global variable usage, when using core library's loadstring to transpile moocscript to Lua.
I think this is an advantage of mooncake, there will be less global variable pollution when code base becoming larger.
I try to figure out what's happening in this piece of code:
assert(require"moocscript.core".loadstring("print(a)"))

first of all, function call in this piece of code was wrong, and function call's normal form in mooncake should enclosed by parenthesis, otherwise parser will raise an error:

➜  /tmp moocscript -v
moocscript v0.8.20221204, LuaJIT 2.1.0-beta3
➜  /tmp cat a.mooc
assert(require"moocscript.core".loadstring("print(a)"))
➜  /tmp moocscript a.mooc
Error: invalid token 'string', when expected ')'
File: a.mooc
Line: 1 (Pos: 14)
Source: assert(require"moocscript.core".loadstring("print(a)"))
                      ^

the correct garmmar of this code:

assert(require("moocscript.core").loadstring("print(a)"))

and mooncake can transpile it to Lua, only treat "print(a)" as a string
➜  /tmp cat b.mooc
assert(require("moocscript.core").loadstring("print(a)"))
➜  /tmp moocscript -s b.mooc
assert(require("moocscript.core").loadstring("print(a)"))

3. when really running, core library's loadstring will raise an error for variable `a` undefined, it can not find variable `a`'s definition in source string all level scopes.
➜  /tmp moocscript b.mooc
~/rocks/bin/luajit: [string "b.mooc"]:1: Error: undefined variable
File: _
Line: 1 (Pos: 6)
Source: print(a)
              ^
stack traceback:
	[C]: in function 'assert'
	[string "b.mooc"]:1: in function 'f'
	...uarocks/rocks-5.1/mooncake/0.8.20221204-2/bin/moocscript:119: in function 'main'
	...uarocks/rocks-5.1/mooncake/0.8.20221204-2/bin/moocscript:243: in main chunk
	[C]: at 0x0104b67a70

4. you really know it will running well for bring pre-defined `a` from `setenv` or `_ENV`, you can using `export` keyword like
assert(require("moocscript.core").loadstring("export a; print(a)"))

4a. With print(a) it was only small example. In fact all the mooncake code meant to be run in custom feature-rich environment with dozens of predefined functions and variables. It is completely counterproductive to include "export" for all of them to each small piece of code.
Ya, nobody likes "global variable pollution", but we use special tools to prevent this (like glorious luacheck).
4b. Sure we use Luas loadstringfor Lua code. But the intent was to loadmooncake` code.
So I have no other idea how to deal with it, except asking you to add some option to skip 'undefined variable' checks.

4a. With print(a) it was only small example. In fact all the mooncake code meant to be run in custom feature-rich environment with dozens of predefined functions and variables. It is completely counterproductive to include "export" for all of them to each small piece of code. Ya, nobody likes "global variable pollution", but we use special tools to prevent this (like glorious luacheck).
4b. Sure we use Luas loadstringfor Lua code. But the intent was to loadmooncake` code.
So I have no other idea how to deal with it, except asking you to add some option to skip 'undefined variable' checks.

the mooncake only transpile .mooc source to Lua and only make sure all global variable in .mooc are exported before using

for you don't need local variable before define and using it

if you like using global variable more than local, export * in the file begining is enough, the compiler will not check this any more, further more, if you need project scope global variable, read source and change it for yourself

mooncake only transpile .mooc and will not affect your .lua source, you can require .mooc from .luaor require .lua form .mooc

you can transpile all your .mooc to .lua before runing it, and then no one will stop you using special tools to lint Lua code, like glorious luacheck

you know something typescript, right?

I think you can put your eyes on other mooncake's features, for using global variable directly is quit easy to by pass by tanspiler if you really want to

Unfortunately export * is completely irrelevant to my task, because I still want all the definitions to be local by default.
What I exactly need is the possibility to write in mooc (for example) a = b and get it transpiled into Lua as local a = b without checking whether b was defined before.
I need it because the function meant to be executed in rich environment, with a lot of variables predefined.
Mooncake just does not know anything about that target environment, because it is not in the source.

I think you can put your eyes on other mooncake's features, for using global variable directly is quit easy to by pass by tanspiler if you really want to

Sorry, I have not got what you meant.
Sure I have looked through docs, but not found any solution.

Sure I have looked through docs, but not found any solution.

Unfortunately I did not know why you want an undefined right-value, but mooncake is very strict to right-value to be defined first.
Sorry, may be I don't know what you want.
Sure I insist on you should read docs, including sources first for your task, which is completely irrelevant to me.
Whish the MIT lincesed sources will help you.

Unfortunately I did not know why you want an undefined right-value

Because it is actually defined, not in the source yourself, but in environment.
Consider such Lua example:
local some_predefined_function(x)
 -- ...
end
local ENV = {some_predefined_function=some_predefined_function}

local mooc = [[
  a = some_predefined_function(12345)
  -- ...
  re
]]

local mcore = require("mooncake.core")
local fn = assert(mcore.loadstring(mooc))
setfenv(f, env)
f()

Unfortunately I did not know why you want an undefined right-value

Because it is actually defined, not in the source yourself, but in environment. Consider such Lua example:
local ENV = {some_predefined_function=function (x)
 -- ...
end}

local mcore = require("mooncake.core")
local fn = assert(mcore.loadstring([[
  a = some_predefined_function(12345)
  -- ...
]]))
setfenv(fn, ENV)
f()

Ok，a n d

Unfortunately I did not know why you want an undefined right-value

Because it is actually defined, not in the source yourself, but in environment. Consider such Lua example:
local ENV = {some_predefined_function=function (x)
 -- ...
end}

local mcore = require("mooncake.core")
local fn = assert(mcore.loadstring([[
  a = some_predefined_function(12345)
  -- ...
]]))
setfenv(fn, ENV)
f()

Ok, consider mooncake source below, what would you transpile to Lua, I you do not know some_predefined_function was defined before:
a = some_predefined_function(12345) -- if a not export before ( in global variable list ), will be as 'local a = ...' in Lua
some_predefined_function = false -- local or global?, only the runing VM know, but trasnpiler don't know. If transpiler assume it as global, why assume 'a' before as global. Then, you need local all global variable, so that's Lua's view.

Unfortunately I did not know why you want an undefined right-value

Because it is actually defined, not in the source yourself, but in environment. Consider such Lua example:
local ENV = {some_predefined_function=function (x)
 -- ...
end}

local mcore = require("mooncake.core")
local fn = assert(mcore.loadstring([[
  a = some_predefined_function(12345)
  -- ...
]]))
setfenv(fn, ENV)
f()

you can export * in the very beginning of the loading string
local ENV = {some_predefined_function=function (x)
 -- ...
end}
setmetatable(ENV, { __index = _G })

local mcore = require("moocscript.core")
local fn = assert(mcore.loadstring('export *' .. [[
  a = some_predefined_function(12345)
]]))
setfenv(fn, ENV)
fn()

Unfortunately I did not know why you want an undefined right-value

Because it is actually defined, not in the source yourself, but in environment. Consider such Lua example:
local ENV = {some_predefined_function=function (x)
 -- ...
end}

local mcore = require("mooncake.core")
local fn = assert(mcore.loadstring([[
  a = some_predefined_function(12345)
  -- ...
]]))
setfenv(fn, ENV)
f()

you can export * in the very beginning of the loading string
local ENV = {some_predefined_function=function (x)
 -- ...
end}
setmetatable(ENV, { __index = _G })

local mcore = require("moocscript.core")
local fn = assert(mcore.loadstring('export *' .. [[
  a = some_predefined_function(12345)
]]))
setfenv(fn, ENV)
fn()

you can export * in the very beginning of the loading string

The problem is that export * makes all variables global by default, which is far from my intentions.
I still want variables to be local by default, as advertised in docs: https://github.com/lalawue/mooncake#features
Consider this example:
  export *
  fn x() {
    a = 1
    return a
  }
  a = 2
  x()
  print(a) --expect 2 but get 1

And using global by default rule I get environment polluted with unneeded identifiers

you can export * in the very beginning of the loading string

The problem is that export * makes all variables global by default, which is far from my intentions. I still want variables to be local by default, as advertised in docs: https://github.com/lalawue/mooncake#features Consider this example:
  export *
  fn x() {
    a = 1
    return a
  }
  a = 2
  x()
  print(a) --expect 2 but get 1

And using global by default rule I get environment polluted with unneeded identifiers

  export *
  fn x() {
    local a = 1
    return a
  }
  local a = 2
  x()
  print(a) -- 2

you can using like Lua this way, or using like moocscript the other way, you have the source, you know the rules.

you can export * in the very beginning of the loading string

The problem is that export * makes all variables global by default, which is far from my intentions. I still want variables to be local by default, as advertised in docs: https://github.com/lalawue/mooncake#features Consider this example:
  export *
  fn x() {
    a = 1
    return a
  }
  a = 2
  x()
  print(a) --expect 2 but get 1

And using global by default rule I get environment polluted with unneeded identifiers

  export *
  fn x() {
    local a = 1
    return a
  }
  local a = 2
  x()
  print(a) -- get 2

you can using like Lua this way, or using more strict the other way, anyway you have all the sources.


Add documentation around authentication headers in custom provider class

See #492 - add some documentation to indicate that when creating a custom Provider using AbstractProvider, it may be necessary to override the getAuthenticationHeaders() method.

Looks good. Thanks!


Feature: Identify repos with low or now activity when editing workspaces and repo insights

Suggested solution
When editing a workspaces or repo insight, it would be nice to see what projects have low or no activity so I can delete them.
https://opensauced.rewatch.com/video/ck8nfltvfrsxt1p5-help-delete-repos-from-workspaces

A compromise could be being able to search/filter the list based on the word "example" or similar.


Can't stop library from logging in Azure Databricks , It writes to much logs

Thanks for the awesome library but I am facing a small issue here I am using the library in Azure Databricks to write some logs
but the library for some reason changing the log level for the cluster machines to be DEBUG so it makes it write to many logs
like the image below so if you can provide a way to stop or change the log level it will be great
Edit:
I believe the issue is here https://github.com/fredysomy/pysonDB/blob/29c1d363aac33c8c6e332d27e3043f52a7f54f60/pysondb/db.py#L12

Hey @yehiaelsayed .
This log issue was solved in #10 .
you can turn off the logs using
a = db.getDb("test.json", log=False)  # the logs are turned off

Please try it out and check if this works , else ping me and i will find a remedy.
thanks for using this library.

@Adwaith-Rajesh

@fredysomy
Thank you so much for the quick response but this is not solving the issue
as I mention before this line below
https://github.com/fredysomy/pysonDB/blob/29c1d363aac33c8c6e332d27e3043f52a7f54f60/pysondb/db.py#L12
is changing the global configuration for the logging and when I try to change it again it is not effecting
so I overcome the issue with this workaround
import logging
logging.basicConfig(level=logging.ERROR)
from pysondb import db

I am setting the log level before it is being set in the library and it is working in this way

Can you join the discord disscusion group

Can you tell me the version you are using.
current version is v1.1.1
Without log

with log.(log=True)

forgive me if i did'nt get your point this day was stressfull.

@yehiaelsayed the package does not change the log level. It straight up disables logging when you use the log keyword.
So I don't think it's caused by pysondb.
If no logs are shown form other packages when you use log=False then it's an issue.

@yehiaelsayed PysonDB does changes the log level, if imported before other packages it does cause the effects you have mentioned. The workaround you specified is correct and that is exactly how I would use any other package that changes log level. I don't think there is anything to change in the PysonDB, and you also found a workaround for it. Trying to fix this issue is actually causing other issues as things that need to be logged are not logged and so on.

@yehiaelsayed , i will be adding this workaround  which you did to docs , if this happens to someone else in future.

this issue can be closed.

Closing this issue.

@fredysomy @Adwaith-Rajesh
sorry for troubling you guys, but setting log=False will stop the library logs but the case is this line
logging.basicConfig(level=logging.DEBUG) changing the configuration for the py4j logging so it keeps writing all logs from debug level  so it is the py4j logs, not pythondb logs (you can check the screenshot in the first message)
and I believe it will cause the same issue with anyone has a dependency on the same logging module because as per the
documentation logging.basicConfig(level=logging.DEBUG) this line apply the configuration on the logger root level
https://docs.python.org/3/library/logging.html#logging.basicConfig

so I believe you should use setLevel function on your logger level to avoid changing the root logger configuration
https://docs.python.org/3/library/logging.html#logging.Logger.setLevel

@yehiaelsayed will look into it. Thank for the suggestion.


New queries browser: DNU when receiving a single entity

When the new queries browsers receives a single entity, it considers it as a MooseGroup.
=> toSelect (instance of FamixEntity) DNU #first.
The old query browser used to include single entities in a MooseGroup (see MiQueriesBrowser>>#folllowAction).
Idk if it's the best solution.

Ah, sorry, the new browser does it too. It just sends #first before, to set the window title, resulting in a DNU for single entities


Add Botanist, Warden, Quartermaster and Chaplain

Adds YAML for these missing jobs. Also adds all of the ID card spritework courtesy of @SweptWasTaken and sprites for the chaplain's black jumpsuit from TGStation.

Does this add the spawn points for these jobs?

Does this add the spawn points for these jobs?

It does now. They're not mapped in though.

That's okay. Swept can take care of that when he does his next mapping PR

There literally is no space on the map for QM, Warden or Chaplain.


Test MI dashboard with http enabled

Description:
Test the MI dashboard with the https default enabled configuration, and make necessary changes.
Suggested Labels:

Suggested Assignees:

Affected Product Version:
OS, DB, other environment details and versions:
Steps to reproduce:
Related Issues:

Fixed with https://github.com/wso2/devstudio-tooling-ei/pull/1215


add params

begin to translate https://github.com/ONSAS/ONSAS/blob/master/sources/extractMethodParams.m#L21

Pull Request Test Coverage Report for Build 534164810

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-8.7%) to 45.783%

Totals

Change from base Build 534073536:
-8.7%

Covered Lines:
152

Relevant Lines:
332

💛  - Coveralls


[packages] Icon rounded outlined at the same time

Flutter is missing the option to make icons both rounded and outlined at the same time.
This option should exist because

In Figma material extension there is an option to make the icon rounded and outlined at the same time  (see fill=off)

I can do it under Material Symbols

https://fonts.google.com/icons?icon.style=Rounded
I may be missing how to use material symbols in Flutter.
I have tried
Icon(Icons.home_rounded, fill: 0.0) vs Icon(Icons.home_rounded, fill: 1.0)
and
Icon(Icons.home_outlined, fill: 0.0) vs Icon(Icons.home_outlined, fill: 1.0)
and
Icon(Icons.home, fill: 0.0) vs Icon(Icons.home, fill: 1.0)
flutter --version
Flutter 3.9.0-1.0.pre.164 • channel master • https://github.com/flutter/flutter.git
Framework • revision 9ba0d08ebc (4 days ago) • 2023-03-12 00:16:19 -0500
Engine • revision 57f7120ef9
Tools • Dart 3.0.0 (build 3.0.0-322.0.dev) • DevTools 2.22.2

Hi @guyluz11, material symbols are not yet supported in flutter. See  https://github.com/flutter/flutter/issues/102560
Closing from here as duplicate.
Kindly subscribe to the issue linked above for updates.
Thank you


remove navconfiguration: {}

What this PR does / why we need it:
remove navconfiguration: {}
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Release note:

@morvencao I create a PR in test-infra https://github.com/IBM/test-infra/pull/161 to enable image build for the release branch. Please review

/retest

/retest

/test build-operand-deployment-lifecycle-manager-s390x


Marker color customization on streaming updates

Use a different marker color (green) for streaming updates.

Fixed edd26fce63fc42c332003b9b1cf9504c4062b85f
Applied stream logic channel wise.


doc: clarify doc of HasProd

Thanks!

maintainer merge

Thanks :tada:
If CI passes, please remove the label awaiting-CI and merge this yourself, by adding a comment bors r+.
bors d+

bors merge


Support for PocketMine-MP version 3.23.1

I just want to be on the contributor list and I did not test this

I extensively tested it and it works!

very good

do you mean very poggers?

yes

hi me too please thanks


:sparkles: add support functions for a caching layer

This change adds cache helpers. The cache lives at ~/.cache/owid/nu/, and the cache duration is 3 hours.
By caching common commands, it means you can afford to do an expensive action at the start of a pipeline and then do filtering on the pipeline iteratively afterwards.
The basic pattern for any command that needs to cache its results is to wrap
the real work in a command like this:
def "my command" [--refresh] {
  let key = "my-cache-key" # XXX
  if ($refresh or (not (cache exists $key))) {
    (do the real work) # XXX
	   | cache tee $key
  } else {
    cache fetch $key
  }
}

By writing functions in this common style, you can reliably pass --refresh to do realtime iteration in the rare moments you need to.

Nice! @larsyencken I played a bit more with this and I think I managed to add a cache wrapper command that I think you wanted to have - I thought it might be nice to use this as owid cache KEY CMD ARGS - see the docs for examples.
If you think this works then please merge at your convenience :)

Nice! The cache wrapper works inline, though it doesn't seem to be able to replace the pattern that's at the top of the file.
I'll keep experimenting, but if you find a nice way to make cached versions of functions that's more readable than that top pattern, let me know!


Create front-end page

We need a page to search where do we eat and display the results too.
I see the front-end side like a one page application for now.
So maybe just one input to search and buttons to filter by categories directly.
The result will be displayed as a list maybe? With restaurant's name, distance, categories, price.
We will be able to order the listing by what we display.
Nothing is really decided and there is no data for now, so have fun.

Maybe @senzowayne will join us for that.