Cluster API with IPAM & InCluster compatibility

It looks like we are trying to solve the same issues related to IP address management at vSphere using Kubernetes Cluster API.
Is this component working with CAPV v1.5.1 and Cluster API 1.3.1 ?
Given the following template with reference to the cluster IP pool using the new feature "addressesFromPools" in CAPV 1.5.0:

apiVersion: infrastructure.cluster.x-k8s.io/v1beta1
kind: VSphereMachineTemplate
metadata:
  name: vmware-test-controlplane
spec:
  template:
    spec:
      cloneMode: linkedClone
      datacenter: dc1
      datastore: datastore5
      diskGiB: 10
      folder: vmware-test
      memoryMiB: 8192
      network:
        devices:
        - dhcp4: false
          dhcp6: false
          gateway4: <IP_ADDRESS>
          addressesFromPools:
          - apiGroup: ipam.cluster.x-k8s.io
            kind: InClusterIPPool
            name: vmware-test-controlplane
          networkName: VM Network
     ...

and the IPAM:
apiVersion: ipam.cluster.x-k8s.io/v1alpha1
kind: InClusterIPPool
metadata:
  name: vmware-test-controlplane
spec:
  start: <IP_ADDRESS>
  end: <IP_ADDRESS>
  prefix: 24
  gateway: <IP_ADDRESS>

From those configuration we do get IPAddresses and IPAddressclaims as expected, however, the IP are not propagated to the virtual machine.
I wonder if cluster-api-ipam-provider-in-cluster was written before the release of CAPV 1.5.0 and thereby not being compatible with the new CAPV feature.
Note, we are using Talos for bootstrap & control plane, however, that should not affect IPAM.

This is the first provider implementation for the new CAPI IPAM contract, and supposed to serve as a reference implementation that providers can use for testing. The intention is that it works with any provider that implements the IPAM contract.
VMware used this project to test their implementation, and they are working on e2e tests using it: kubernetes-sigs/cluster-api-provider-vsphere#1667
Since it's a relatively new addition to CAPV, there might still be bugs. Have you opened an issue for CAPV already?
cc @tylerschultz

Thanks for the info and great work. I can see that the guest-info metadata actually contains the correct network configuration. I will look at why Talos is not picking up that metadata.

@ErikLundJensen Please do let us know if you get to the bottom of what's going sideways, we'd be curious to know.

The issue is Talos' integration with VMware where the Talos v1.3 has no implementation of reading metadata from guestinfo.


Function in function to read JSON

Hi,
Abell seams pretty cool but I can't find a way to have a function in an other function to read "complex" JSON
Exemple
I have a JSON file
[
	{
		"title":"Blabla",
		"tags": ["internet", "ethic"],
		"images":["blabla.png"],
		"date":"août 2019",
		"sources":{
			"url":"https://github.com/",
			"description":"Github website"
		}
	}
       {...}
]

And I want to make a .map() in another .map() to add tags and sources information to a page. I try different stuff but it's look like Abell don't want to make that
Is there a way to do it ?
Thanks in advance
Have a nice day 🌞

Hi, We have to use ${} inside strings since everything inside double curly brackets is JavaScript
So you can do something like:
{{
  data.map(obj => `
    Tags:
   ${
        obj.tags
          .map(tag => `<span>${tag}</span>`)
          .join('')
     }
  `)
}}

This should render
Tags:
internetethic

I hope this answers the question. Thank you and have a great day! Let me know if you meant something else

Thanks a lot !
That's what I was looking for !


Support InternVL2's quant now?

Support InternVL2's gptq quant now?

Hii,
Currently, AutoGPTQ does not support InternVL2’s GPTQ quantization. AutoGPTQ primarily focuses on language models and does not natively support many visual language models like InternVL2 or MOLMO.

Hii, 嗨， Currently, AutoGPTQ does not support InternVL2’s GPTQ quantization. AutoGPTQ primarily focuses on language models and does not natively support many visual language models like InternVL2 or MOLMO.目前，AutoGPTQ 不支持 InternVL2 的 GPTQ 量化。AutoGPTQ 主要关注语言模型，本身不支持许多视觉语言模型，如 InternVL2 或 MOLMO。

Thank you for your reply.


v2.9.2 jplayer don't work with zepto.js v1.1.6,please help me

Uncaught TypeError: Cannot read property 'animate' of undefined
#my source:#
`

瑞莱宝

play

play
stop

mute
max volume

repeat
full screen

Update Required
                To play the media you will need to either update your browser to a recent version or update your Flash plugin.

`

#247
shortly: don't use  smoothPlayBar: true


Control vwc-text-field font size

The problem
On iOS - when using the vwc-text-field component the browser zoom-in on the vwc-text-field and distort the whole page layout
The solution
enabling font size control on the vwc-text-field component
Application/website
web chat
Business priority
Medium Priority = upcoming release but is not pressing
What time frame would this ideally be needed by (if applicable)
No response
Examples
No response
Code of Conduct

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

@TamirVonage not sure we need to expose the font size control, but need to change it to al least 16px in mobile to solve this problem.

@rachelbt No objection there. We just need any solution.

This behaviour affect all vonage and is inherited from the Volta parity AFAIK. Making a change to the base & body font size will need to be addressed by design.
Note: this is not a blocker for the frontier web chat nor is text field issue alone. It is a typical browser behaviour for when font size is less than 16 within inputs (in certain viewports).
@TamirVonage would u please mention designers involved here? And whom specifically from design this request comes from? We'll need a bit of research

Yamit is involved in this project but I doubt she has a github account

access to this repo and content is publicly available. it's ok to discuss further on JIRA if need be
@Vonage/vivid-design can u please mention this task to Yamit and assign?

will be resolved by redefining tokens for different platforms


Roll Flutter from 5b3e42afd019 to 36fa6c4d9081 (2 revisions)

https://github.com/flutter/flutter/compare/5b3e42afd019...36fa6c4d9081
2022-03-18<EMAIL_ADDRESS>Fix wrong documentation: There is no LeaderLayer._lastOffset anymore (flutter/flutter#100300)
2022-03-18<EMAIL_ADDRESS>Roll Engine from 89b16c065c9d to a99d51f0c128 (2 revisions) (flutter/flutter#100391)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-plugins
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Plugins: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Support for globbing of inputs

It would be nice to be able to just add a directory and have all the inputs be included. For instance, if I use dotnet pack to generate a package, I don't necessarily know the full name. If I could then run dotnet SignClient.dll zip --config [path] -i Package.*.nupkg that would be really nice.

Thinking more about this, what should it do if there's more than one match? There's an output file option too, should it be allowed only if there's a single matching input?
The workaround to this is that the script uses the full name or for multiple files, can put it in a zip file and send that.
Also, if a filter is needed, there's no current way of mapping that to a file; it'd be the same filter for all inputs.
How would you expect it to work?


Allow psr/container ^1.1

After trying to implements idea from https://github.com/thecodingmachine/graphqlite/discussions/601 i realize that just allowing  psr/container ^1.1 work perfectly.

Great news!

hey @oojacoboo , could you make a release with this ? thanks in advance :)

@fezfez https://github.com/thecodingmachine/graphqlite/releases/tag/v6.2.3


Hitting unreachable in websocket's parse_message (version 0.2.2)

Apparently hitting unreachable!() in parse_message of websocket is possible. Last night I had the following panic:
thread '<unnamed>' panicked at 'internal error: entered unreachable code', /home/kr/.cargo/registry/src/github.com-1ecc6299db9ec823/bitmex-0.2.2/src/websocket.rs:102:14
This was in a long running session. I am not sure what they sent.

Hi @kr011 the issue was fixed in 44548a31. I'll release a new version soon.

Thank you.

@kr011 maybe you can first have a try on the master branch. I want to include tokio 1.0 in the new release however it is blocked by hyper-tls https://github.com/hyperium/hyper-tls/issues/76.

@dovahcrow what's in master doesn't seem to be usable yet. I get this as soon as I start it.
itMEXWsMessage::Info: InfoMessage { info: "Welcome to the BitMEX Realtime API.", version: 2020-12-16T22:50:54Z, timestamp: 2020-12-27T19:34:11.742Z, docs: "https://www.bitmex.com/app/wsAPI", limit: Limit { remaining: 35 } }
thread '<unnamed>' panicked at 'internal error: entered unreachable code: Cannot deserialize message from BitMEX: '{"success":true,"request":{"args":["***",***,"***"],"op":"authKeyExpires"}}'', /home/kr/.cargo/git/checkouts/bitmex-rs-2861a3a714a639fb/af8602e/src/websocket/mod.rs:123:27

Looks like the response to the WS Auth message. I just copy/pasted from the examples:
let expires = (Utc::now() + Duration::seconds(30)).timestamp();
ws_client.send(Command::authenticate(expires as u64)).await?;

@kr011 I pushed a new fix. You can have a try to see if that solves the issue.

@dovahcrow that fixed it. Thanks. I am running it now. I will let you know if I run into any bugs. Cheers!

@kr011 Glad to hear that!


Fixed toolbar at the top of the page

It has proven near to impossible to fix a toolbar/header at the top of the page and have the grid sit underneath this header.  I want the header pinned at the top while the grid is scrollable underneath it.
I've tried various combinations of divs, flex settings and a couple of other hacks but nothing works.  Once the grid overflows the window, the entire page is scrollable and the header scrolls off the top when scrolling.
I'm missing some combination of property settings or something.  I'm trying to build a dashboard and I prefer to have the responsiveness of the grid turned off so the grid items don't resize.  The user would use vertical and horizontal scrolling to view the grid items if they are off the page.  To keep things clean I want the grid items to be fixed with no dragging or resizing and then let the user enter a Edit mode that allows them to move things around and resize them, change the dashboard name, etc and then save the changes.
But as I say in my initial assertion, I can't create a pinned header above the grid such that it doesn't scroll off the page when the grid overflows its container.
Suggestions?  Anything may help.

Put the grid in a  element where the div has got a margin at the top and bottom, for a header and footer, respectively. The height/width should be set to auto, to match the size of the grid. This div element should not be fixed and the margins should at least match the size of the header and footer.
Could you add a plunkr in case you don't manage to figure it out?

I not sure I followed all the instructions properly but I still can't figure it out.
plnkr here

This should be the fix: plnkr.
Please keep in mind that this sounds more like implementing css than an actual issue with the angular2-grid package - I may be mistaken but this would not be the appropriate place to post this issue.


ZLC - CRC resources page with links

Looks like the resources page can handle external links. Categories are defaulted to the standard array but can be overridden in the facility config file.

Rejected by staff


Document/Images Usage page: 'ManyToOneRel' object has no attribute 'verbose_name’

Issue Summary
Users cannot view the usage details of documents or images (where they’re being used) used at least once in any multiple Parentalkey levels models from the Wagtail admin site.
When images/documents are used in certain parts of the website, and users try to get the usage information on where those images/documents are being used at, the following error occurs at trying to get the verbose_name from the right child field (determined through the reference index):
AttributeError at /admin/documents/usage/1/
'ManyToOneRel' object has no attribute 'verbose_name’

This seems to be a problem in referencing the verbose_name of the correct child field, likely stemming from the multiple ParentalKey level nested structure of the ClusterableModel/Orderable models in the example (Resource → ResourceGroup → ResourceItem, etc.).
Affected Code:
https://github.com/wagtail/wagtail/blob/ef9375c47578045914c0fa4727388ba01e488354/wagtail/models/reference_index.py#L651-L653
Current Expected: ManyToOneRel → Child Field with verbose_name (Image/Document)
Actual: ManyToOneRel → ManyToOneRel → Child Field with verbose_name (Image/Document)
Ultimately, we might be looking at a recursive fix for multi-level ManyToOneRel fields.
Video demonstrating the issue (Steps 6 to 10 in Steps to Reproduce below):
https://www.loom.com/share/8dade8f9e25e44c38b4a863c870d8098?sid=bf96dd4a-0b96-4413-8e89-47e201b88808
Steps to Reproduce

Start a fresh Wagtail project following the tutorial at https://docs.wagtail.org/en/stable/getting_started/tutorial.html up until the Start the server step.

Replace the contents of mysite/home/models.py with the following code:
from django.db import models

from wagtail.models import Orderable
from wagtail.documents import get_document_model_string
from wagtail.admin.panels import FieldPanel, InlinePanel

from modelcluster.fields import ParentalKey
from modelcluster.models import ClusterableModel

class ResourceItem(Orderable):
    resource_group = ParentalKey(
        "home.ResourceGroup", related_name="item"
    )

    document = models.ForeignKey(
        get_document_model_string(),
        null=True,
        on_delete=models.SET_NULL,
    )

    panels = [
        FieldPanel("document"),
    ]

class ResourceGroup(ClusterableModel, models.Model):
    resource = ParentalKey("home.Resource", related_name="group")

    panels = [
        InlinePanel("item", label="Resource")
    ]

class Resource(ClusterableModel, models.Model):
    pass

Create a wagtail_hooks.py file at the mysite/home directory with the following contents:
from wagtail.admin.panels import InlinePanel
from wagtail.snippets.models import register_snippet
from wagtail.snippets.views.snippets import SnippetViewSet

from .models import Resource

class ResourceViewSet(SnippetViewSet):
    model = Resource
    panels = [
        InlinePanel("group", label=" Resource Group")
    ]

register_snippet(ResourceViewSet)

On your terminal, run python manage.py makemigrations then python manage.py migrate from the project root directory (eg. ../mysite)

Browse and login to the Wagtail Admin site at localhost:8000/admin with your admin account.

Using the sidebar, navigate to Snippets → Resources.

Perform the following: Add resource → Add resource group → Add resource (Under Resource Group 1) → Choose a document (Upload any file) → Save

Once the new Resource object is created, using the sidebar, navigate to Documents and select the document you just uploaded.

On the right/bottom of the page, under Usage, select Used 1 time

The following error should occur:
AttributeError at /admin/documents/usage/1/

'ManyToOneRel' object has no attribute 'verbose_name'

Expected Outcome:
No errors occur, and the usage page of the selected document can be viewed, with the correct verbose_name of field references listed under the Field column. e.g. Resource group → Resource item → Document

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: Yes

Technical details

Python version: 3.8.14
Django version: 3.2.23
Wagtail version: 5.2.2
Browser version: Screenshots are on Firefox, but this should occur on any browsers.

Working on this
I'm keen to work on this issue once it has been accepted.

After working on the issue for some time, I've had some luck with a potential recursive fix:
if isinstance(field, models.ManyToOneRel):
    label = f"{capfirst(field.related_model._meta.verbose_name)}"
    idx = 2
    child_field = field.related_model._meta.get_field(model_path_components[idx])
    while isinstance(child_field, models.ManyToOneRel):
        label += f" → {capfirst(child_field.related_model._meta.verbose_name)}"
        idx += 2
        child_field = child_field.related_model._meta.get_field(model_path_components[idx])
    label += f" → {capfirst(child_field.verbose_name)}"
    return label

This fix should also fix an initial issue where the first field level's verbose_name is excluded from the returned string.
For example, for a field reference of ResourceItem → Document within a ResourceGroup object:
Current result: Document
This fix's result: Resource item → Document
While this potential fix addresses the reported issue in this case, more testing is required for the other cases. I welcome any feedback or insights from the community to ensure the solution's robustness.

I have also had a similar problem with nested ManyToOneRel.
In our scenario the relationship we have is ManyToOneRel → ManyToOneRel with no final object with a 'verbose_name'
I like what @terrsoshi has suggested as a solution but we would need accomodate for no 'verbose_name' being present for our case

This solution works perfectly for the problem I have been having.
Would you be comfortable opening a PR?

I have created a PR for this issue at #11808.


Remove hardcoded path prefixes for game assets

What could be improved
Flame has hard coded prefixes for loading game assets that are not well documented on the website, and should be replaced with relative paths using the application root bundle.
// Prefix flame adds when loading images
assets/images/

// Prefix flame_tiled adds when loading tiles
assets/tiles/

The documentation is not clear what directories flame is expecting developers to use, and makes it difficult to load game assets correctly:
https://docs.flame-engine.org/main/flame/structure.html
Why should this be improved
The hardcoded path prefixes are not well documented and re-organizing assets for large projects is time consuming and costly. 60% of the time, using a path to load assets works every time.
Any risks?
It will take time to implement.
More information

Example implementation:
final mapComponent = await TiledComponent.load(
        'Assets.maps.finalworld.path', Vector2(16, 16));
    add(mapComponent);

This is a documentation issue if anything, if you want to use a different base path for the caches it is already possible.

The hardcoded path prefixes are not well documented

This is quite a strong statement 😅. I'd say it is just missing the part about assets/tiles. Other than that, the docs clearly mentions where flame expects a certain type of assets by default.
And it also specifies this

If you want to change this structure, this is possible by using the prefix parameter and creating your instances of AssetsCache, Images, and AudioCache, instead of using the global ones provided by Flame.


500 Internal Server Error - After upgrade to 3.1

Describe the bug
I used the docker instructions from the github page. https://github.com/danielbrendel/hortusfox-web?tab=readme-ov-file#installation but when i tried to access the app, username and password was ok, but once i click Login it gave me this error.
Screenshots and Logs
Screenshot
Logs
Software:

HortusFox version 3.1
PHP Version: PHP 8.2.0
MySQL version: Ver 15.1 Distrib 10.5.23-MariaDB
Docker version: docker-compose version 1.29.2

Hi,
The error is caused because there is at least one recent log entry in the
log table associated with a non existing user.
A workaround would be to identify the user id and clear all log entries of
that user. Or clear the entire log table.
You could also manually edit the /app/models/LogModel.php file and validate
that $log_user is not null at line 51.
Those are the only solutions right now until I add a fix.
Sorry for the inconvenience. If you need help with that workarounds feel
free to join the discord and msg me there.
Kind Regards
chiruleonard @.***> schrieb am Fr., 24. Mai 2024, 15:25:

Assigned #213 https://github.com/danielbrendel/hortusfox-web/issues/213
to @danielbrendel https://github.com/danielbrendel.
—
Reply to this email directly, view it on GitHub
https://github.com/danielbrendel/hortusfox-web/issues/213#event-12925087213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN7WQA3WRTIM3EXQJFVDUBLZD45V7AVCNFSM6AAAAABIHSV3LGVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHEZDKMBYG4ZDCMY
.
You are receiving this because you were assigned.Message ID:
@.***>

Hi,
The error is caused because there is at least one recent log entry in the
log table associated with a non existing user.
A workaround would be to identify the user id and clear all log entries of
that user. Or clear the entire log table.
You could also manually edit the /app/models/LogModel.php file and validate
that $log_user is not null at line 51.
Those are the only solutions right now until I add a fix.
Sorry for the inconvenience. If you need help with that workarounds feel
free to join the discord and msg me there.
Kind Regards


optimization on database IN

This PR will remove all str join but use a sql builder and reduce unused codes.

LGTM

Current coverage is 3.03% (diff: 0.00%)

Merging #152 into master will not change coverage

@@            master      #152   diff @@
========================================
  Files           33        33
  Lines         8096      8096
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           246       246
  Misses        7830      7830
  Partials        20        20

Powered by Codecov. Last update 9bf28a2...ade6d4a

LGTM


Uncomment Task

Challenge Uncomment HTML has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/50.0.2661.94 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
I am being asked to delete a trailing comment tag that is not ther
My code:

Hello World

CatPhotoApp

Hello Paragraph

@codesnsuch
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please consult the Help Chat Room for any assistance to Coding.
Happy Coding!

I'm experiencing the same issue and the default code has,  which has been brought to my understanding as incorrect syntax


Pinned columns in multiple views for the same table cause performance issues in some cases

v1.16.*, 1.17.1
Steps to reproduce:

Open SPGI
Pin Id and Structure
Open filters
Add 10-15 more views with the same table (e.g. clone current view), each having filters and pinned columns
Check performance for one of the following:

Add one more vew
Add another table viewer to the current view
Open 'order and hide columns' or similar dialog
Open column selection dropdown for some viewer

Expected results: Pinned columns should not affect performance significantly.
Actual results: The most noticeable thing is that any new table (added to the current view or one more view) is rendered much slower compared to the case when there are no pinned columns in the opened views. Column selection dropdowns and dialogs for various viewers are also rendered slowly.

PowerGrid 1.2.5

PowerGrid 1.3.0 DG 1.18.5
Data: SPGI

Pin Id and Structure
Clone current view (View > Layout > Clone) - Id and Structure columns are not shown in the clone

This issue has been mirrored in Jira: https://reddata.atlassian.net/browse/GROK-15375

PowerGrid 1.3.0,  DG 1.18.5


Change Claculator Button Color

Do something that will add value, changing colour is not very big thing.


[sdk-node] add HostDetector to the list of default resource detectors

HostDetector is not part of the default list of resoruce detectors when intantiating NodeSDK.
I propose we add it to the default list alongside the Environment and Process detectors.

as discussed offline, I can take this one 😄


neovim: support visualizing debug logs inside vscode

At the moment, to visualize the neovim log when debugging, we need to show the content of a file outside of vscode, for instance in the terminal.
There is a way to actually open a terminal automatically and visualize the logs directly in the terminal using the following:
.vscode\tasks.json:
    {
      "label": "neovim logs",
      "type": "shell",
      "windows": {
        "command": "powershell",
        "args": [
          "(New-Object -ComObject WScript.Shell).Run(\"\"\"powershell Get-Content '${workspaceFolder}/packages/cursorless-neovim/out/nvim_node.log' -wait -tail 30\"\"\", 1, $false)"
        ]
      },
      "group": "build"
    },

I'm not using it atm because I don't find it very nice, but this is a possibility in case we want to improve the workflow.
Note that the above is Windows only so this will need to be ported for macos or Linux.
also note that before neovim 5.1.0, the logs were actually inside vscode due to a bug that were not redirecting them into the custom logging system. see https://github.com/neovim/node-client/issues/329#issuecomment-2009676355 so it may actually be possible to use vscode debug console alternatively writing them to a file.

Since it works for Linux and OSX, I tried again for Windows but encoutering a few problems.

I need to use shell type instead of process as otherwise tail command is not found when trying to execute it. As long as it also works for OSX/Linux, we should be fine (cc @pokey / @fidgetingbits)
vscode does not automatically attach anymore when debugging (I am actually wondering if it attaches for you @pokey and @fidgetingbits with that task, since @fidgetingbits mentioned having problems attaching in https://github.com/cursorless-dev/cursorless/issues/2446)
I need to use a path of the format C/path/to/ instead of C:\path\to\ due to using git bash shell as otherwise the file is not found. This could be solved by using a script that convert the path then execute the command

NOTE: isBackground: true does show that the task is a background task afaict (it shows neither a "tick" indicating a finished task nor a "loop" indicating a non-finished task)
diff --git a/.vscode/tasks.json b/.vscode/tasks.json
index 7bdda96f..645e1d4e 100644
--- a/.vscode/tasks.json
+++ b/.vscode/tasks.json
@@ -247,7 +247,8 @@
     },
     {
       "label": "Neovim: Show logs",
-      "type": "process",
+      "type": "shell",
+      "isBackground": true,
       "osx": {
         "command": "tail",
         "args": [
@@ -263,9 +264,11 @@
         ]
       },
       "windows": {
-        // NOTE: We don't have a way on Windows atm due to command with argument inside Run() not working
-        // so we need to show logs outside of vscode (see #2454)
-        "command": "echo"
+        "command": "tail",
+        "args": [
+          "-f",
+          "/C/path/to/cursorless/packages/cursorless-neovim/out/nvim_node.log"
+        ]
       }
     },

@pokey With your changes from https://github.com/cursorless-dev/cursorless/pull/2256/commits/56aec571a4930e6423cf17ebcb57e822690b7c18
I get this error on Windows:
 *  Executing task in folder cursorless_fork: C:\path\to\cursorless\tail -f C:\path\to\cursorless/packages/cursorless-neovim/out/nvim_node.log

 *  The terminal process failed to launch: Path to shell executable "C:\path\to\cursorless\tail" does not exist.

It works with this change:
diff --git a/.vscode/tasks.json b/.vscode/tasks.json
index e706c6f9..cf5e2230 100644
--- a/.vscode/tasks.json
+++ b/.vscode/tasks.json
@@ -269,7 +269,7 @@
     },
     {
       "label": "Neovim: Show logs",
-      "type": "process",
+      "type": "shell",
       "command": "packages/cursorless-neovim/scripts/show-logs.sh",
       "problemMatcher": [],
       "isBackground": true,

I haven't pushed as didn't want to break it for you but can you check and update if it works for you too?

ok works for Pokey on OSX and for me on Windows after changing with shell so only remaining to test is Linux if you want to give it a go @fidgetingbits and close this issue.

note that this is not an ideal solution, as it still requires running the task separately, so we might want to keep this issue open to track that, but I don't feel strongly

Ya, we only need to open it once, and then it is revealed each time we debug so it is pretty good like that imho so I would personally mark it as done. But happy too if you think it is worth improving later

Do we have a separate issue to track the long-term solution of using proper console.log once neovim improves their node support? If so I'd def be happy to close this one

This logging works on Linux if I run the task separately yes, I see the logs in the vscode output window; it no longer runs automatically when running the neovim tests. Also the node debugger no longer automatically attaches for me again.
If this isn't what you're asking for, tell me exactly what to test :D

@fidgetingbits sorry so you're saying debugger is still not working for you with the latest changes?

I guess define not working? I have to manually attach to the node process with the latest changes in order for the tests to start running, but otherwise it works.

I guess define not working? I have to manually attach to the node process with the latest changes in order for the tests to start running, but otherwise it works.

Weird. Auto-attach is working when debugging for Pokey (OSX) and me (Windows) with the latest changes. I am wondering if your debugging does not work because of something else on Linux.

@fidgetingbits  I don't see why debugging should not work as normal for you. The logging task is separate from the build process now. Maybe try to see if it auto attaches if you previously don't run the task to start the logs. My guess is it won't auto attach either?

Not sure. When it worked for me it didn't work for you both :D I can do some more tests, since the devshell recently messed up so maybe has something to do with it still


The value of the slider is Infinity,-Infinity, the maximum value is -Infinity, the value of this slider can not be greater than the maximum value . Computed Properties

Hello. Im trying to use this component as a stateless vue component and in that effort i can only use computed properties.
when i want to pass a computed prop to the slider it fails.
Sorry i cant repro but i will leave the snippet  here
Component
<template>

      Monto Pagado

            <vue-slider v-bind="sliderOptions" v-model="sliderValue"/>

</template>

Main page where i render
<template>

      <range-component :entities="mapEntities"/>

</template>

I failed to reproduce your error. https://jsfiddle.net/2xy72dod/444/
I read your code, there are some problems.

The value of the slider should not be less than the minimum value
No matter how slider slides, sliderValue will not change, because the value returned by the set has nothing to do with the value of the component

Ok. Sorry for the inconvenciences. I will close this issue


Questions about privacy.resistFingerprinting

Why is Firefox spoofing the time zone? The IP address reveals your city (or even your neighborhood in some cases).  So a weird time zone is actually bad for privacy.

I'm running a fairly common resolution. Why doesn't Firefox start maximized? How does a weird size window help?

Is it better to let Firefox handle the canvas or we should install CanvasBlocker? What are the recommended CanvasBlocker 0.4.5c settings?

see #350 section on canvas in the first post. If you are using RFP (privacy.resistFingerprinting) then it's important that you have the same FP as all the other RFP users - so that means use their canvas blocking (which is a 10x10 white canvas).  If a site absolutely requires your real canvas (I'm not an expert, eg maybe banks or sites you log into frequently [ie they know it's you anyway] so as to not get your account flagged, or maybe a web game), then you can whitelist the site in CB as well.

The resolution spoofed is using the inner browser window measurements (because anything else would screw up web page layouts etc). All measurements are returned as your actual inner browser windows ones - that's

browser inner window width+height
browser width+height (I think, been a while)
screen resolution
available screen resolution

If you go maximized, then you cannot control your inner browser window - it is instead controlled by other factors, such as your 1. available screen res and 2. browser chrome elements.
Available screen res can differ - such as in windows, the position of the task bar could be left-right, or top-bottom. The height/width of the task bar can differ as well.
Browser chrome elements can differ. All of which affect the inner browser window measurements - such as the extra drag space, displaying the menu bar, sidebar, toolbar. Also the theme can make changes, such as compact vs default. Even flicking the find bar open changes the metric.
So despite having a common screen res, there are too many variables for a consistent result. Also, not everyone in the RFP set would be maximized. The only solution to encompass everyone is to not allow maximized on startup, and to set a standard size.

Because FP'ing as a science makes assumptions. It's looking purely at FP'ing and entropy. So all users need to be on one of the most common timezones (i.e there are 24 timezones, but the online population is not evenly distributed).

Clearly if you are giving away your IP, then you've possibly already lost the battle (depends on your threat model). TBB doesn't have to worry about this, in the sense that they are using TOR. Firefox users can use VPNs etc. The assumption is that you are already hiding your location.

Here's an article on how entropy works in the real world : https://www.eff.org/deeplinks/2010/01/primer-information-theory-and-privacy

"The assumption is [..]" very clear, IMO? A huge, over-elaborate, question about the basics - in the support (issues), doesn't really help anyone? xD
Anyway, Pants, thanks for the entropy URL, it's a great read..:)

The IP address reveals your city

resistFingerprinting mainly enables uplifted Tor Browser patches. Timezone may not reveal additional information if and only if all the users have their geoip-located timezones matching the ones revealed by the browser. The reality is different. Someone uses Tor, someone uses a VPN, someone's geoip points to another city or even country systematically. These all contain some information, which can be used to reduce anonimity of users, including Tor ones.

Thanks for the explanation, Pants!
I still think spoofing the time zone is bad unless you have access to IPs in that particular time zone. Which won't be the case for most Firefox users.

Running Firefox in a window is only marginally better than a maximized Firefox window according to Panopticlick.
What's the best option to start Firefox maximized? AutoHotkey?

Running Firefox in a window is only marginally better than a maximized Firefox window according to Panopticlick.

You can't take most of those metrics seriously, although at least resolutions are relatively stable, eg compared to say user agents. First of all, it is looking at data gained over a long time, rather than real world, right now. Because you cannot really hide you are using Firefox, you could look at the metrics for FF users - https://hardware.metrics.mozilla.com/ - which is gathered every couple of weeks. However, that is just screen resolution. Panopticon is also only looking at your screen res. RFP is looking at multiple measurements (see above post).
At the end of the day, you need to be the same as RFP.

What's the best option to start Firefox maximized? AutoHotkey?

 Unfortunately it didn't work for Firefox Portable 60.0.2

If you want to start Firefox in as big a window as possible, then use the two hidden prefs
/* 4502: set new window sizes to round to hundreds (FF55+) [SETUP]
 * [NOTE] Width will round down to multiples of 200s and height to 100s, to fit your screen.
 * The override values are a starting point to round from if you want some control
 * [1] https://bugzilla.mozilla.org/1330882
 * [2] https://hardware.metrics.mozilla.com/ ***/
   // user_pref("privacy.window.maxInnerWidth", 1600); // (hidden pref)
   // user_pref("privacy.window.maxInnerHeight", 900); // (hidden pref)

I use mine to start at 1400x800, and then I use Window Resizer to quickly flick to 1366x768. Note: I only use one window, open opening links in a new one will use a 1400x800 (actually its 1400x799 due to bugs when you show your toolbar). Also the position of FF within my screen is remembered, so if I do use another window, I make sure the first one opened is always the last closed.

I removed all the presets in Window Resizer and set my own one. That 1380x871 is the browser measurements, which gives me my inner window of 1366x768 (compact theme, extra drag space, toolbar enabled)
Not saying this is ideal for you, but at least you can FF to open as large as possible


fix doc link version

Coverage increased (+0.004%) to 80.975% when pulling fcd31baa3e0a6a671aef3813c51cad8657325eb5 on pongad:fixup into 05d5811fd62a1e4aadc366f081b0d6b33834c0b3 on GoogleCloudPlatform:master.

https://github.com/pongad/google-cloud-java/blob/fcd31baa3e0a6a671aef3813c51cad8657325eb5/pom.xml#L96 also needs to be updated to 0.1.7

-2; replacing with https://github.com/GoogleCloudPlatform/google-cloud-java/pull/1913 .


Schema file requirements

Loosening the requirements of attributes within the file block. Keeping file.relative_path as both contractual and required, but removing file.absolute_path and file.checksum_md5. These attributes can only exist when file exists on local disk. However, use cases such as usage within aggregation service may not involve exporting the file to the disk at all. In these cases, the file.relative_path will still be kept as it serves as an important identifier for a specific data object.

[ ] Confirm with SSDL/REP
[ ] Confirm with Sumo

Hei!
Replying on behalf of REP/SSDL.
At a first glance:

Removal of checksum_md5. We are using the checksum and propagating all the way to the surface-api. So would have to investigate the impact of this removal a little more and do the corresponding changes. Might be breaking for us.

Removal of absolute_path. This property is not in use by us. One question though, about the downloading of files and existence on local disk - will this impact the loading of files from blob storage?

We currently have the checksum in file.checksum_md5, but also in _sumo.blob_md5. I'm not sure if the version in _sumo.blob_md5 comes from the uploader, or is added by Sumo. It may be useful to have a checksum that is added by the source, which we can then check against the checksum that the blob store computes and attches to the blob. If _sumo.blob_md5comes from the source, then file.checksum_md5 is redundant and can be dropped.

We currently have the checksum in file.checksum_md5, but also in _sumo.blob_md5. I'm not sure if the version in _sumo.blob_md5 comes from the uploader, or is added by Sumo. It may be useful to have a checksum that is added by the source, which we can then check against the checksum that the blob store computes and attches to the blob. If _sumo.blob_md5comes from the source, then file.checksum_md5 is redundant and can be dropped.

If the _sumo.blob_md5 is a "duplicate" of sorts, it would not require much effort on our end to remap to this, I believe.

It’s not an exact duplicate; file.checksum_md5 is a hexadecimal representation of a 128-bit number, while _sumo.blob_md5 is a base64 encoding ogf the 128-bit number. So, same value, different encodings.
Example:
    "_source": {
      "file": {
        "checksum_md5": "48ebec435e821e93505e8bafc7a238cb"
      },
      "_sumo": {
        "blob_md5": "SOvsQ16CHpNQXouvx6I4yw=="
      }
    },

Another difference is that _sumo.blob_md5 is not currently searchable, but that be changed by an update to the ElasticSearch index mapping.

On 21 Feb 2022, at 09:13, ek-bouvet @.*** @.***>> wrote:
We currently have the checksum in file.checksum_md5, but also in _sumo.blob_md5. I'm not sure if the version in _sumo.blob_md5 comes from the uploader, or is added by Sumo. It may be useful to have a checksum that is added by the source, which we can then check against the checksum that the blob store computes and attches to the blob. If _sumo.blob_md5comes from the source, then file.checksum_md5 is redundant and can be dropped.
If the _sumo.blob_md5 is a "duplicate" of sorts, it would not require much effort on our end to remap to this, I believe.
—
Reply to this email directly, view it on GitHub https://github.com/equinor/fmu-dataio/pull/191#issuecomment-1046582435, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAIDQOWEEVAFBXH6YTQBQ5TU4HX3NANCNFSM5O2RGC7A.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.

Sounds like an OK (?) option here is to keep file.checksum_md5 but change the definition of it slightly. Instead of it being a hash of the saved file, it is a hash of the instance before it is stored to a file. My hunch is that this should be trivial to change on the FMU-side. That would result in only the absolute_path not being present in some cases.
The reason this comes up now is because of the bulk aggregation service, which is a context where we don't want to save any files to a disk local to the computation, but upload the instance directly to Sumo. E.g. for a surface. Currently we don't make the file.checksum_md5 until the file has been saved to local disk.

Also, there is perhaps a difference between contractual and required. E.g. a required attribute will always be present (metadata not valid without). A contractual attribute may not always be present, but the definition of it will not be changed without due process. But we may not have a clear enough contract for the definition of contractual 🤔

Sounds like an OK (?) option here is to keep file.checksum_md5 but change the definition of it slightly. Instead of it being a hash of the saved file, it is a hash of the instance before it is stored to a file. My hunch is that this should be trivial to change on the FMU-side. That would result in only the absolute_path not being present in some cases.
The reason this comes up now is because of the bulk aggregation service, which is a context where we don't want to save any files to a disk local to the computation, but upload the instance directly to Sumo. E.g. for a surface. Currently we don't make the file.checksum_md5 until the file has been saved to local disk.

This would work well on our end, at least 👍

file.checksum_md5 and file.relative_path are required, file.absolute_path is not required.
Changing the definition of contractual attributes to be specific to these two sub-attributes, rather than the whole file block.


Add initial BreachRx pack.

Pull Request to show code diff between master and breachrx_xsoar_app.

@cdimino - Nice work!
A few comments:

Make sure that you add an Author image.
Add missing files to Pack root, see Pack List for the full list. I believe you're missing .pack-ignore and .secrets-ignore.
Remove pack_metadata.json.lock.
The integration image is too large. Please resize it to fit the standard size.
Please create a README for the Playbook. Follow Creating Documentation for more info how to automatically generate it.

After these changes, I believe we're ready to create a PR into https://github.com/demisto/content. Make sure to sync this repo's master branch with demisto/content master branch.
Please read through https://xsoar.pan.dev/docs/contributing/contributing#before-opening-a-pull-request and https://xsoar.pan.dev/docs/contributing/conventions.
I will be notified when the PR is opened. FYI, our PR template includes a few CI/CD jobs that will run UTs/coverage and validation on the added content items. It's likely that some problems will be exposed during the builds. Don't worry though, we'll make sure to assist in unblocking them :)


Fix 3 bugs

Fixes 3 bugs. Note that this includes one of the bugs that was already found in https://github.com/mallorbc/whisper_mic/pull/66 by @oreshytko.

Thanks!


Parsing Envelope

Hi,
Is there a way to parse raw bytes &[8u], Vec<u8> to Envelope struct?
Thanks.
With respect,
Felix P.

The envelope function should be what you need. However, it is pub(crate), i.e., not exposed via a public API. Can you tell me a bit about your use case? Trying to understand if it would make sense to expose something like use imap_codec::nom::envelope.

Actually, I like the idea of having an "ecosystem" of composable nom functions. Thus, I would be happy to expose (all) parsers via imap_codec::nom::*. Do you like to make that PR? :-) I will be at home in some hours and can review the PR (or commit something myself).

I'm using SQLite to store mail messages for my IMAP server. Currently I store envelope information as data blob inside this database. I don't know, maybe I should store all envelope fields as separate columns. That way I would just construct new Envelope struct from separate SLQ columns.
Please commit yourself, if its not a problem for you, I'll be out for a weekend.
Thanks.

Ok, before I do anything, I want to ask.
Do you want new pub mod nom with re-exports of functions from mod parse or do you want to replace mod parse with mod nom?
Creating a separate mod nom would allow us to aggregate all [from bytes to struct] functions into one place, while keeping mod parse structure intact. Not sure how to nicely put [from struct to bytes] function (aka. .encode(writer)) to mod nom.

I think Decode trait would be the best option. (Easiest to understand for new users)

Hey @FelixPodint,
Sorry, I didn't had too much time to look into it. Thanks for taking care!

Do you want new pub mod nom with re-exports of functions from mod parse or do you want to replace mod parse with mod nom?

I think mod parse should stay for now.

Creating a separate mod nom would allow us to aggregate all [from bytes to struct] functions into one place, while keeping mod parse structure intact.

Yes. I also prefer this solution. But with a minor change, maybe. As far as I know, crates that use types from a dependency in their public API reexport the dependency under the same name. Thus, I changed my mind, and think that we should reexport all parsers via imap_codec::internal:: or something like that. This way, we would not block a reexport of the nom dependency.

Not sure how to nicely put [from struct to bytes] function (aka. .encode(writer)) to mod nom.

Can you elaborate? Do we have to change that?

Agree. I also prefer this solution. But maybe with a minor change: as far as I know, crates that use types from a dependency in their public API may reexport the dependency under the same name. Thus, I changed my mind, and think that we should reexport all parsers via imap_codec::internal:: (or something like that) and not via imap_codec::nom::. This way, we would not block a reexport of the nom dependency.

Maybe pub use ::nom as nom_nom could work for re-exporting nom crate, but I agree using different name than nom is better idea.

Not sure how to nicely put [from struct to bytes] function (aka. .encode(writer)) to mod nom.

Can you elaborate? Do we have to change that?

OK lets stop and think for a moment.

What I want to have is a unified way to got from struct, like Envelope, to raw bytes and vice versa.
Currently we have parse module, which has functions to convert raw bytes to structs, and we have an Encode trait, which allows conversions from structs to raw bytes. So all of the most difficult parts are already done.
Maybe it's possible to have Decode trait, which mirrors Encode?

Is the proposal of a Decode trait directly relevant to the change to expose the parsers?
From my view we can first expose the parsers via an internal module, such that one can already use use imap_codec::internal::envelope::envelope.
Second step, maybe in a separate issue?, would be to discuss how to unify the Encode and Decode API. I see your point and it seems desirable to have Encode and Decode. But I have some more thoughts on that we should discuss separately. What do you think?

Agree, got a little to exited.
I will make a PR for re-exports.

Hehe, no worries :-)


links and resources is ... underwhelming

I was looking for a list of tools and got to https://bids-standard.github.io/bids-starter-kit/links.html which seems to be really "basic".
Should point to BIDS-Apps, various other currently existing, possible get rid of outdated/no longer developed like https://github.com/robertoostenveld/bids-tools (with @robertoostenveld blessing)
or may be there is already some other location collating those, then just point there instead of having this mediocre smth.

Agreed to have a single place for them all.
Either have eveyrything on the website or everything on the starter kit.
Feel like the starter kit you give more flexibility.

Will close this one as the bids starter kit will be merged into the main BIDS website.


Missing use_lspsaga in README

Hi, I just want to mention that at least one config option is not shown in the README file, namely 'use_lspsaga' (i don't know about the others).
Took me a little to figure this out, maybe you could add it to the list of options there?

It was removed from readme as lspsaga is deprecated.

There is a maintained fork of lspsaga which I am using. Also you did not remove the use_lspsaga option from the doc/lsp_signature.txt. If you consider leaving it in there you should also leave it in the README IMO.

In fact, if you enable use_lspsaga, you will be promoted with a warning message.
I remove the option in doc as you suggested


Moving a vue file creates an Include entry in tsconfig.json - why?

Describe the bug
when I move a .vue file from one directory in my source tree to another directory in the source tree, VueDX modifies my tsconfig.json file by putting an entry in the 'include' section that points to the file in the new location.
Why does it do this and is there a way to make it stop? This is incredibly annoying because it clutters up the tsconfig.json file if I miss the fact that it changed and don't revert it. It is also incredibly annoying because every time I move files, I have to find the tsconfig.json file and either not save the open buffer or revert the file - both are annoying operations involving a popup dialog.
To Reproduce
Steps to reproduce the behavior:
take any project based on typescript using vue.
Move a .vue file from one directory to another directory where both are under /src
What you will see is that tsconfig.json gets modified as above
Expected behavior
tsconfig.json shouldn't be touched. There is no value in the change that was made to the file.
Info (please complete the following information):

OS windows 10
IDE VS Code
TypeScript Version 4.3.2
Vue Version 2.6.13
Other versions (e.g. Nuxt, Vue Router etc.)

This is a side effect from monkey patching tsserver APIs.
I am working on an alternate much cleaner implementation (#240) which would not have such issues.


serial: fixes, more tests and docstrings.

 Issue by jfng
Wednesday Jan 01, 2020 at 20:39 GMT
Originally opened as https://github.com/m-labs/nmigen-stdio/pull/3

jfng included the following code: https://github.com/m-labs/nmigen-stdio/pull/3/commits

Superseded by #5.


hook-lang

https://github.com/fabiosvm/hook-lang

Lol I like that language! ;) I could do a Pull Request, what do you think, @ChessMax?

PR will be awesome. Otherwise I will add it to the list myself soon. Thanks 👍

PR #322

Merged. Thanks!

Thank you!


Getting error in configuring default binder while configuring with Multiple binders

@muhammad-usman-confiz commented on Fri Nov 01 2019
Using spring cloud stream binders, I am trying to connect with multiple azure event hub namespaces. This configuration works fine with single namespace but for multiple namespaces, I am getting following exception on startup.
Caused by: java.lang.IllegalStateException: A default binder has been requested, but there is more than one binder available for 'org.springframework.cloud.stream.messaging.DirectWithAttributesChannel' : kafka2,kafka1, and no default binder has been set.
	at org.springframework.cloud.stream.binder.DefaultBinderFactory.doGetBinder(DefaultBinderFactory.java:181) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binder.DefaultBinderFactory.getBinder(DefaultBinderFactory.java:132) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binding.BindingService.getBinder(BindingService.java:339) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binding.BindingService.bindProducer(BindingService.java:234) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binding.AbstractBindableProxyFactory.createAndBindOutputs(AbstractBindableProxyFactory.java:136) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binding.OutputBindingLifecycle.doStartWithBindable(OutputBindingLifecycle.java:58) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at java.util.LinkedHashMap$LinkedValues.forEach(LinkedHashMap.java:608) ~[na:1.8.0_221]
	at org.springframework.cloud.stream.binding.AbstractBindingLifecycle.start(AbstractBindingLifecycle.java:57) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.cloud.stream.binding.OutputBindingLifecycle.start(OutputBindingLifecycle.java:34) ~[spring-cloud-stream-3.0.0.RC1.jar:3.0.0.RC1]
	at org.springframework.context.support.DefaultLifecycleProcessor.doStart(DefaultLifecycleProcessor.java:182) ~[spring-context-5.2.0.RELEASE.jar:5.2.0.RELEASE]
	... 14 common frames omitted

Here is my .yml configuration
spring:
  cloud:
    stream:
      kafka:
        binder:
          brokers: broker1.servicebus.windows.net:9093
          configuration:
            sasl.jaas.config: "..."
            sasl.mechanism: PLAIN
            security.protocol: SASL_SSL
      dynamicDestinations: dev-event-hub,prod-event-hub
      bindings:
        input-1:
          destination: dev-event-hub,prod-event-hub
          binder: kafka1
          group: $Default
        input-2:
          destination: dev-event-hub,prod-event-hub
          binder: kafka2
          group: $Default
      binders:
        kafka1:
          type: kafka
          environment:
            spring:
              cloud:
                stream:
                  Kafka:
                    binder:
                      brokers: broker2.servicebus.windows.net:9093
                      configuration:
                        sasl.jaas.config: "..."
                        sasl.mechanism: PLAIN
                        security.protocol: SASL_SSL
                  dynamicDestinations: dev-event-hub,prod-event-hub
        kafka2:
          type: kafka
          environment:
            spring:
              cloud:
                stream:
                  Kafka:
                    binder:
                      brokers: broker3.servicebus.windows.net:9093
                      configuration:
                        sasl.jaas.config: "..."
                        sasl.mechanism: PLAIN
                        security.protocol: SASL_SSL
                  dynamicDestinations: dev-event-hub,prod-event-hub

@muhammad-usman-confiz Your configuration looks right. Can you remove that initial configuration for binders where you define the brokers since you already provide that as part of multi-binder configuration? If it still doesn't work, please provide a small sample for us to test and make the brokers local in the sample. Did you try this sample?

@sobychacko Due to project deadline I used various other ways to connect with multiple event hub namespaces (multiple Kafka clusters) and ended up on spring-Kafka with totally customized solutions.
After your reply, I tried again using the spring-cloud-stream to reproduce this issue but now without default Kafka binder, so this solution is working fine.
I think I should explore more on my side whether this issue was due to my configuration or any other reason.
Thanks

@muhammad-usman-confiz Good to hear that it got resolved when you remove that default configuration. Feel free to share the repo if you are still having any issues and then update here. For the time being, I am closing this issue.


fix get the openapi interface that contains namespace information for deleted items

What's the purpose of this PR
fix get the openapi interface that contains namespace information for deleted items.
Which issue(s) this PR fixes:
Fixes #4593
Brief changelog
fix get the openapi interface that contains namespace information for deleted items
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Read the Contributing Guide before making this pull request.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Write necessary unit tests to verify the code.
[x] Run mvn clean test to make sure this pull request doesn't break anything.
[x] Update the CHANGES log.

There are many places that consume the result of com.ctrip.framework.apollo.portal.service.NamespaceService#transformNamespace2BO, e.g. com.ctrip.framework.apollo.openapi.server.service.ServerNamespaceOpenApiService and com.ctrip.framework.apollo.portal.controller.ConfigsExportController.
However, only the controllers for the namespace UI need the deleted items, e.g. com.ctrip.framework.apollo.portal.controller.NamespaceController.
One way to fix the issue is to filter the deleted items in places that we don't need them, or we may add a flag and let the caller decide whether to include the deleted items or not, e.g. private NamespaceBO transformNamespace2BO(Env env, NamespaceDTO namespace, boolean includeDeletedItems).
I think the latter might be better. What do you think?

There are many places that consume the result of com.ctrip.framework.apollo.portal.service.NamespaceService#transformNamespace2BO, e.g. com.ctrip.framework.apollo.openapi.server.service.ServerNamespaceOpenApiService and com.ctrip.framework.apollo.portal.controller.ConfigsExportController. However, only the controllers for the namespace UI need the deleted items, e.g. com.ctrip.framework.apollo.portal.controller.NamespaceController.
One way to fix the issue is to filter the deleted items in places that we don't need them, or we may add a flag and let the caller decide whether to include the deleted items or not, e.g. private NamespaceBO transformNamespace2BO(Env env, NamespaceDTO namespace, boolean includeDeletedItems). I think the latter might be better. What do you think?
I agree with your opinion, but I found that the export namespace config file contains deleted items, this' right?

I agree with your opinion, but I found that the export namespace config file contains deleted items, this' right?

I think the export namespace config file should not contain deleted items, which means we also need to add the includeDeletedItems flag to com.ctrip.framework.apollo.portal.service.NamespaceService#findNamespaceBOs.

I think com.ctrip.framework.apollo.openapi.server.service.ServerNamespaceOpenApiService#getNamespaces should also exclude deleted items?


Change logrotate free space check script to read free space correctly

Previously we were reading space used as free space.
https://bugzilla.redhat.com/show_bug.cgi?id=1294991

@gtanzillo

Looks good :+1:


Improve User seeder

Refactor UserFactory to a Class
Use Laravel/Prompts to ask for user input in UserSeeder

Thanks for showing your interest in our project and sending your pull request.
Your pull request accepted.
There was an issue which is fixed in commit 6461ac4b
Issue was that password was not being hashed before calling the user factory. It has been fixed by adding below lilnes in UserSeeder.php
//
use Illuminate\Support\Facades\Hash;
//

//
$password =  Hash::make($password);
//

So the UserSeeder.php now looks like this.
<?php

namespace Database\Seeders;

use App\User;
use Illuminate\Database\Seeder;
use Illuminate\Support\Facades\Hash;

use function Laravel\Prompts\text;

class UserSeeder extends Seeder
{
    /**
     * Run the database seeds.
     *
     * @return void
     */
    public function run()
    {
        /* Ask for name, email and password from terminal. */
        $name = text('Name:', required: true);
        $email = text('Email:', required: true);
        $password = text('Password:', required: true);
        $password =  Hash::make($password);

        User::factory()->create(compact('name', 'email', 'password'));
    }
}

Thanks for your pull request again. Hope you continue to contribute to this project.

Thanks for merging this PR and for fixing the Password Hashing,


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-yml-archive-non-versioning-archive/pull/86 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-yml-archive-non-versioning-archive.git

Docs Build status updates of commit d0280f3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit d0280f3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

docs/TOC.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level1/article3-1-3.yml
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-1.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-2.md
:white_check_mark:Succeeded
View

docs/to-be-archive/level2/article3-2-3.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


sails lift --prod does not link assets specified in tasks/pipeline.js

My tasks/pipeline.js in my sails application looks like this
'''
// CSS files to inject in order
//
// (if you're using LESS with the built-in default config, you'll want
//  to change assets/styles/importer.less instead.)
var cssFilesToInject = [
'styles/**/*.css'
];
// Client-side javascript files to inject in order
// (uses Grunt-style wildcard/glob/splat expressions)
var jsFilesToInject = [
// Load sails.io before everything else
'js/dependencies/sails.io.js',
'js/jquery.js',
'js/jquery/jquery.validate.min.js',
// Dependencies like jQuery, or Angular are brought in here
'js/dependencies/**/*.js',
// All of the rest of your client-side js files
// will be injected here in no particular order.
'js/**/*.js'
];
'''
However, when I run sails lift --prod, the grunt task is not linking the css and js assets specified in the above pipeline.js file.

Try:
// CSS files to inject in order
// (if you're using LESS with the built-in default config, you'll want
// to change assets/styles/importer.less instead.)
var cssFilesToInject = [
'/styles/*/.css'
];

// Client-side javascript files to inject in order
// (uses Grunt-style wildcard/glob/splat expressions)
var jsFilesToInject = [

// Load sails.io before everything else
'/js/dependencies/sails.io.js',
'/js/jquery.js',
'/js/jquery/jquery.validate.min.js',
// Dependencies like jQuery, or Angular are brought in here
'/js/dependencies/*/.js',

// All of the rest of your client-side js files
// will be injected here in no particular order.
'/js/*/.js'
];

@metzuda no, those ** are there for a reason--they make sure all subdirectories are traversed.
@codesdk what exactly do you mean by "not linked"?  In production mode, Sails doesn't copy individual files to your public folder--it minifies and concatenates them into production.min.js and production.min.css.


Make the config folder XDG compatible.

Please respect freedesktop standards.
https://www.freedesktop.org/wiki/Software/xdg-user-dirs/
https://specifications.freedesktop.org/basedir-spec/basedir-spec-latest.html

Hi, thanks for raising this.

The XDG specification recommends using .config/<program name> and .cache/<program name> as its folder structure. This means that the ~/.node-red is violating that specification, potentially leading to errors with some system-integrated tools that do management, cleanup or further integration of those files.

@DarkPark ok - can you give examples of tools that affects? We've not had any issues reported and is a pretty major change to make given how long we've used ~/.node-red and how much collateral now exists on that assumption.

I'm new with Node RED so can't list the affected tools.
But take a look in your /home/USER/.config/ dir. For the past years a lot of programs moved their configs to this dir to keep user home dir clean. It's a global tendency so it would be nice to think this option over.

looking in my home dir I have 45  .xxx application dirs (inc .node-red) and 9 in .config
apps in home dir include small app providers like aws wireshark wine ssh npm docker dropbox atom
apps in .config include errr git as the only big name I recognise...
not sure there is a compelling push in this direction - and does this apply to Windows ?

I can see 45 dirs in .config and 34 in home root.
But in general it's a good idea to keep home clean and tidy. Besides it's about standardization.
I don't know about Windows. Not sure it's covered by this standard.

program.profile = program.profile ||
    path.join(process.env.XDG_CONFIG_HOME || path.join(process.env.HOME, '.config'), pkgInfo.name);

something like this will work ok both on XDG compatible and not systems

I like this idea.  A suggestion for lessening the impact may be:

Detect ~/.node-red.
If found, move it into e.g. ~/.config/node-red, and force=symlink ~/.node-red to ~/.config/node-red
If not found, just use ~/.config/node-red.

Given how node_modules works, I don't see any way to use the XDG cache or data dir effectively at this point, unless we rely on symlinks again--which is risky given npm's handling thereof.
For Windows users, there's no equivalent concept, so no change is needed here.

There are several approaches, but moving user files is not a good option, someone could have a reason to keep it in the home. Just use XDG by default and fallback to home for compatibility. I'm not actually using node-red, but seeing it created a dir in my home I'm happy to give a +1 for this issue.


Potential overflow in crowdsale participation with indivisible tokens

CMPTransaction::logicHelper_CrowdsaleParticipation:
if (!isPropertyDivisible(property)) { // <-- the tokens "invested"
    nValue = nValue * 1e8; // <-- nValue is uint64_t
}

The cut-off is<PHONE_NUMBER>8, which is the largest number where nValue * 100000000 <= INT64_MAX, and sending more than<PHONE_NUMBER>8 indivisible tokens either causes an assertion violation or undefined behavior.
To be more specific:

Amount
Master Core 0.0.9
Omni Core 0.0.10

0 to<PHONE_NUMBER>8
OK
OK

92233720369 to<PHONE_NUMBER>37
OK
assertion violation

184467440738 or more
undefined
undefined

The assertion violation is caused by the conversion of uint64_t to int64_t, which results in negative numbers in that range (rejected via ConvertTo256).
The results for numbers higher than<PHONE_NUMBER>38 are undefined, meaning: it's up to the compiler implementation how it's dealt with. On my system it seems to be 0, but it can really be anything between 0 and MAX.
The positive:

no new tokens can be created out of the blue
we can easily mirror the exact behavior of 0.0.9

The negative:

a buyer may not get the tokens he or she paid for

It's notable, even without overflow, that<PHONE_NUMBER>8 is the maximum number of divisible tokens that are "counted" anyway..
Potential fix:
int64_t adjusted = 0;

if (nValue > (INT64_MAX/100000000)) { // ><PHONE_NUMBER>8
    adjusted = INT64_MAX; // clip to max. allowed amount
} else {
    adjusted = nValue * 100000000;
}

edit: calculateFractional (for missed tokens) must be adjusted for the fix.
This is potentially consensus breaking.

Nice, thanks for this dude :)
Why don't we do the inflation of amounts once we're in uint256?  We can avoid the modification of nvalue and overflow for INT64 should no longer be a concern?
With a simple patch such as https://github.com/zathras-crypto/omnicore/commit/33f311496e9e333e59bbf6b1330523a6758932ce, perhaps we may address this issue, #248 and #249 together?
Note - this is just calculateFundraiser, I haven't looked at calculateFractional yet...
I attempted to replicate your regtest with the patch:
omni_gettransaction af270a05abccd89c4ed7e554cf09c94da3cccad8f35851748f052f0b24e6844b
{
"txid" : "af270a05abccd89c4ed7e554cf09c94da3cccad8f35851748f052f0b24e6844b",
"fee" : "0.00000256",
"sendingaddress" : "n144os5tJMmfVP8SvRJD5TSZph7HMd7P7J",
"referenceaddress" : "msDqmnX8jpTRJdBevjqN5UQqfXUV9WU5aK",
"ismine" : true,
"version" : 0,
"type_int" : 0,
"type" : "Crowdsale Purchase",
"propertyid" : 3,
"divisible" : false,
"amount" : "1",
"purchasedpropertyid" : 4,
"purchasedpropertyname" : "Z_TestProperty",
"purchasedpropertydivisible" : false,
"purchasedtokens" : "1",
"issuertokens" : "0",
"valid" : true,
"blockhash" : "026a81dfeb9223dbf5ee5168556b60f01aa9066ee4820fbe4ee0dcf75d48252f",
"blocktime" :<PHONE_NUMBER>,
"block" : 165,
"confirmations" : 1
}

omni_getcrowdsale 4 true
{
"propertyid" : 4,
"name" : "Z_TestProperty",
"active" : true,
"issuer" : "msDqmnX8jpTRJdBevjqN5UQqfXUV9WU5aK",
"propertyiddesired" : 3,
"tokensperunit" : "1",
"earlybonus" : 0,
"percenttoissuer" : 0,
"starttime" :<PHONE_NUMBER>,
"deadline" :<PHONE_NUMBER>,
"tokensissued" : "1",
"addedissuertokens" : "0",
"amountraised" : "1",
"participanttransactions" : [
{
"txid" : "af270a05abccd89c4ed7e554cf09c94da3cccad8f35851748f052f0b24e6844b",
"amountsent" : "1",
"participanttokens" : "1",
"issuertokens" : "0"
}
]
}

That's pretty nice actually, and it even removes the artificial limit. Can you think of any downsides?

Note - this is just calculateFundraiser, I haven't looked at calculateFractional yet...

I'm very sure calculateFractional should be overhauled, because it basically re-runs all crowdsale participations to calculate totals.. as double. :/
But there is good news: unless I overlook something, we may almost completely remove it, because we already sumed all values.
int64_t missedTokens = crowdsale.getIssuerCreated() - (crowdsale.getUserCreated() * [bonus]);

Then another point I was wondering about: the "missed tokens" are only calculated, if a crowdsale expires, but not in case the crowdsale is manually closed. But it should, I assume?


Bitcoin currency support

Please add support for the Bitcoin currency. It does not have an official ISO 4217 currency code yet, but XBT is commonly used.

For inspiration: there's a patch I use to do it for GnuCash desktop.
It doesn't add the maximum precision allowed by Bitcoin, but works well enough in practice.

@chrisberkhout Thanks for the patch! However, simply adding the currency won't be enough. There's still the problem reported in #635, to be solved by finishing the migration from Currency to Commodity.

Fixed in develop.


Various minor install/usage errors

I am attempting to install ReadMe-Generator and encountered the following issues while following the README:

On install step 2, running npm install -g repopack I get the following error:

npm ERR! code EACCES
npm ERR! syscall mkdir
npm ERR! path /usr/local/lib/node_modules
npm ERR! errno -13
npm ERR! Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules'
npm ERR!  [Error: EACCES: permission denied, mkdir '/usr/local/lib/node_modules'] {
npm ERR!   errno: -13,
npm ERR!   code: 'EACCES',
npm ERR!   syscall: 'mkdir',
npm ERR!   path: '/usr/local/lib/node_modules'
npm ERR! }
npm ERR!
npm ERR! The operation was rejected by your operating system.
npm ERR! It is likely you do not have the permissions to access this file as the current user
npm ERR!
npm ERR! If you believe this might be a permissions issue, please double-check the
npm ERR! permissions of the file and its containing directories, or try running
npm ERR! the command again as root/Administrator.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/david/.npm/_logs/2024-10-24T17_28_38_731Z-debug-0.log

I suggest updating the step to either indicate running as sudo or only require a non-global install (the latter being my personal preference, but a bit more work)

On usage step 2, running readme-generator fails with the following error:
readme-generator: command not found

It looks like we're missing an install step pip install .

After installing readme-generator, running it fails with the following error:

Traceback (most recent call last):
  File "/home/david/Projects/ReadMe-Generator/venv/bin/readme-generator", line 5, in <module>
    from readme_generator.__main__ import main
  File "/home/david/Projects/ReadMe-Generator/venv/lib/python3.11/site-packages/readme_generator/__main__.py", line 3, in <module>
    from .cli import main
  File "/home/david/Projects/ReadMe-Generator/venv/lib/python3.11/site-packages/readme_generator/cli.py", line 8, in <module>
    from .generator import ReadmeGenerator
  File "/home/david/Projects/ReadMe-Generator/venv/lib/python3.11/site-packages/readme_generator/generator.py", line 11, in <module>
    from . import logger
ImportError: cannot import name 'logger' from 'readme_generator' (/home/david/Projects/ReadMe-Generator/venv/lib/python3.11/site-packages/readme_generator/__init__.py)

It looks like you are missing the __init__.py file in the src/logger directory needed to make pip install pick up that package. When I add an init file with from .logger import * it works. (You could also move logger/logger.py to logger/init.py)

Thanks for raising this issue! @davidn
I've pushed some changes to address them. Please let me know if this resolves it for you!!

Install works perfectly now!  When I run I get the following error:
Traceback (most recent call last):
  File "/home/david/Projects/ReadMe-Generator/.venv/bin/readme-generator", line 5, in <module>
    from readme_generator.__main__ import main
  File "/home/david/Projects/ReadMe-Generator/.venv/lib/python3.11/site-packages/readme_generator/__main__.py", line 3, in <module>
    from .cli import main
  File "/home/david/Projects/ReadMe-Generator/.venv/lib/python3.11/site-packages/readme_generator/cli.py", line 8, in <module>
    from .generator import ReadmeGenerator
  File "/home/david/Projects/ReadMe-Generator/.venv/lib/python3.11/site-packages/readme_generator/generator.py", line 11, in <module>
    from . import logger
  File "/home/david/Projects/ReadMe-Generator/.venv/lib/python3.11/site-packages/readme_generator/logger/__init__.py", line 1, in <module>
    from .logger import error, success, warning, get_progress, console
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ImportError: cannot import name 'console' from 'readme_generator.logger.logger' (/home/david/Projects/ReadMe-Generator/.venv/lib/python3.11/site-packages/readme_generator/logger/logger.py)

I think to fix this 1) change the __init__.py line to from .logger import error, success, warning, get_progress, logger and 2) change utils.py line 23 to logger.logger.console.print("npm install -g repopack") so that you are using the console attribute on the logger object instead of trying to find console in the module itself (which doesn't exist).

@davidn I've added some changes to reduce the install friction and updated the install instructions. Let me know if that resolves the issue here!


Error:tensorflow.python.framework.errors_impl.NotFoundError: libtensorflow_framework.so.1: cannot open shared object file: No such file or directory

Hi author!I'sorry to bother you but i encountered a problem when import dirt.my environment is as follow: ubuntu18.04 tensorflow-gpu=2.1 python=2.7
Traceback (most recent call last): File "infer_single.py", line 9, in from model.octopus import Octopus File "/home/chensien/octopus/model/octopus.py", line 16, in from render.render_layer import RenderLayer File "/home/chensien/octopus/render/render_layer.py", line 1, in from render import render_colored_batch File "/home/chensien/octopus/render/render.py", line 1, in import dirt File "/home/chensien/anaconda3/envs/octopus/lib/python2.7/site-packages/dirt/init.py", line 2, in from .rasterise_ops import rasterise, rasterise_batch, rasterise_deferred, rasterise_batch_deferred File "/home/chensien/anaconda3/envs/octopus/lib/python2.7/site-packages/dirt/rasterise_ops.py", line 6, in _rasterise_module = tf.load_op_library(_lib_path + '/librasterise.so') File "/home/chensien/anaconda3/envs/octopus/lib/python2.7/site-packages/tensorflow_core/python/framework/load_library.py", line 58, in load_op_library lib_handle = py_tf.TF_LoadLibrary(library_filename) tensorflow.python.framework.errors_impl.NotFoundError: libtensorflow_framework.so.1: cannot open shared object file: No such file or directory
i check the the path "/home/chensien/anaconda3/envs/octopus/lib/python2.7/site-packages/dirt" and libtensorflow_framework.so.1 is exists.I'd be pretty appreciate if u offer some advice!thx!

It looks like you somehow built DIRT using a different version of tensorflow (1.x), and then installed it in a conda env with tensorflow 2.1. Did you do a development-mode install (i.e. cmake followed by pip install -e .)? If so, delete the cmake build folder and rebuild, making sure your octopus conda env is activated beforehand.

thanks for replying.Actually i didn't do a development-mode.I create a new conda env, install in development mode and then i get another problem:
Traceback (most recent call last):  File "square_test.py", line 4, in <module>    import dirt  File "/home/chensien/octopus/dirt/dirt/__init__.py", line 2, in <module>    from .rasterise_ops import rasterise, rasterise_batch, rasterise_deferred, rasterise_batch_deferred  File "/home/chensien/octopus/dirt/dirt/rasterise_ops.py", line 6, in <module>    _rasterise_module = tf.load_op_library(_lib_path + '/librasterise.so')  File "/home/chensien/anaconda3/envs/octopus/lib/python2.7/site-packages/tensorflow_core/python/framework/load_library.py", line 57, in load_op_library    lib_handle = py_tf.TF_LoadLibrary(library_filename)tensorflow.python.framework.errors_impl.NotFoundError: /home/chensien/octopus/dirt/dirt/librasterise.so: undefined symbol: _ZN10tensorflow7strings8internal9CatPiecesB5cxx11ESt16initializer_listIN4absl11string_viewEE

gcc 5.4 on Ubuntu 18.04 is an unusual combination that I cannot test easily. The default system gcc version for Ubuntu 18.04 (i.e. gcc 7.4) seems to work correctly, combined with the python & tf versions you said (py27 / tf210, both from conda/main).
In your original install, without development mode, if you still have the conda env, could you do conda list and paste the results? It looks like some tensorflow 1 & 2 got mixed up somehow.

I already removed the previous conda env.As you said,there are different version of tensorflow in the conda env.After checking my cuda version,i think problem is the mismatch of cuda version and tensorflow instead of gcc.So i reinstalled DIRT with tf1.12,and it works^ ^  Much thx for your  help !


nilesoftShel not showing in Install Section

Describe the bug
App existed inside application.json not found on GUI
To Reproduce
Steps to reproduce the behavior:

Go to 'Install'
Click on 'Search and search nilesoft'
Not found

Expected behavior
I was looking for nilesoftshel a right-click alternative. Searched on  winutil not found. So I was going to create a PR. But found out Its already listed inside application.json but not showing on GUI.
Screenshots

Thanks @SHR1SHAK
This is because the "content" is the name used to display it in the utility, in this case "Shell (Expanded Context Menu)".
I will make a PR to make it use it's actual Name.

Sure


Fix override condition for sized dealloc with gcc 6

Apparently gcc 6 defines __cpp_aligned_new in C++14 mode, however no std::align_val_t is available there for obvious reasons.

Ah strange, if it defines _cpp_aligned_new is should have std::align_val_t right? ah well.

Looks like it defines it unconditionally, regardless whether we're in C++14/11 mode or C++17 mode.


dynamicly load imageData

Hey there!
Great component so far. I only have one question:
I get my imageData from the server and I put it into the imageData (v-model). But there is no image shown.
Maybe I missed it but is there a way to provide the imageData and show it without the user selecting an image first?
Thank you very much!

Same question.
I did this way: if image received from server, then I would render v-img with image src.
If not: show v-image-input. On v-img @click just show input and hide image. Not the best option, but it works.

I did this way: if image received from server, then I would render v-img with image src.

How did you get v-img to render the pure imageData? Or did you pass the path as src?

I did this way: if image received from server, then I would render v-img with image src.

How did you get v-img to render the pure imageData? Or did you pass the path as src?

Ah okay I fixed this using:  v-bind:src="'data:image/jpeg;base64,'+imageData
But are you able to use the v-image-input features like zoom / orientation-switch and stuff using v-img?
Thank you in advance!

But are you able to use the v-image-input features like zoom / orientation-switch and stuff using v-img?

If user starts interacting with the image, I show them input. You can add the edit button when hovering over v-img and switch to v-image-input  after clicking this button.

If user starts interacting with the image, I show them input. You can add the edit button when hovering over v-img and switch to v-image-input after clicking this button.

Thanks for the fast answer! But after checking whats behind the prop inputData I figured I just just the same "trick" as with v-img and now I ended up with this:
<v-image-input        v-model="imageData"        :image-quality="0.85"        clearable        image-format="jpeg"    />
and adding 'data:image/jpeg;base64' to my server response like this:
this.imageData = 'data:image/jpeg;base64,' + response.data;'
This those the job for me so far! Now I can load images from the server and the user can open images manualy by clearing the viewer and use it in the standard way

Thanks for the idea, I'll do the same

Hi.
Yes. Value accepts an image data url.
Here you find the syntax: https://developer.mozilla.org/docs/Web/HTTP/Basics_of_HTTP/Data_URIs


Allow edit on the textbox and can get either left or right box value

how do I allow edit on the diffcontrol box and can get either left or right box value in the code?

The controls in Menees.Diffs.Windows.Forms don't support editing. They're all read-only (i.e., view-only) controls, so the value in a control will never change from what you initialize it with.


Improve mobile compatibility

This PR fixes 2 issues that have been bugging me for a while when viewing docs on mobile:

user-select: none: for some reason clicking on some buttons selected it, this just disabled it :)
Other changes: A bit of moving around to prevent scrolling while one of the sidebars is open (navegation and TOC).

I am nowhere proficient at HTML and CSS and this solution I came up with seems hacky, but I cant find a better way that doesnt break the site in one way or another. If there is a better suggestion, I will make sure to test and update the PR to match that!

I'm gonna go ahead and close this out, since it looks like a decent chunk of these improvements have been made, either as literally the same change or via alternatives. :)
Thanks again @davfsa for this PR! ^>^


ranger: new package at 1.9.3

Description of package
ranger is a console file manager with VI key bindings. It provides a minimalistic and nice curses interface with a view on the directory hierarchy. It ships with rifle, a file launcher that is good at automatically finding out which program to use for what file type.

[x] I agree to maintain this package.
[x] I agree to notify @dylanaraps if I can no longer maintain this package.

Existing package

[ ] I am the maintainer of this package.

Yes for now It is, read https://github.com/ranger/ranger/issues/1892
Whithout mandoc ranger crash, It was 1st build. But it has been reported and will be corrected in the next release

wait wait... problem ...

Ok this error occurs in the variable case TERM=screen256-color ... in all other cases, that is: TERM = xterm or similar there are no problems, the fix will be released in the next release 1.9.4
let me know if I have to release it as it is now or if you want to add the fix in post install or if I have to wait for the official release of 1.9.4

patch apply. now work.


Plugin finds unbalanced less than (<) operator (C++)

For the code below
for(is = 0; is < sl + 1; is++)
the plugin finds unbalanced bracket at the less than operator. Is this intended for C++ source files?

The plugin was built to check large pieces of sql files, but it can used to check html/xml files, and other other file formats.
It is not inteded to check source files.
I am going to add parametrization on which bracket the plugin should check.
Thanks!

Did you already have any luck with parametrization of this superb plugin?
There are some other plugins which can do the same, but somehow I prefer using this one.

Not yet, really, but I will soon, I promise :)

I fix the plugin as promised.
The plugin manager is not updated yet, but uninstalling-reinstalling the plugin it will be updated to version 1.2 (despite plugin manager say it's version 1.0)

Excellent work! Thank you very much. :)

Works like a charm.
Thanks mate!


This Extension Wrecked My Setup, Cannot check in anything now

What's this issue all about?
I installed this add-in, and now git on my entire workstation is broken. I cannot check in anything without getting very vague error messages.
Expected Behaviour
I expect that git works on my workstation after installing this extension!
Actual Behaviour
I cannot check in code for anything now without getting "Unknown error" from git. This happens in VSCode, GitKraken, VS, everything.
Error is "Failed to Execute Git"
This has wrecked my workstation, and as a result I cannot check in a single thing, thus costing me an entire day (and it's still not fixed). DO NOT INSTALL THIS EXTEMSION!!!
Software Info
VSCode version - v.latest
GitMerger version - v.latest
Operating System - Win11

This extension only executes git commands behind the scenes, so what you are describing doesn't make any sense.
Furthermore, you are the first user after ~180K downloads who report this behavior.


Benchmark results?

Hey,
really like the project!
I am really interested in the performance of this this LLM vs the official Reprover.
Did you consider running a lean-dojo benchmark? If so, where can we find the results?

Hi @josojo the baseline model achieved 51.2% validation and 50.1% test on the lean-dojo random split. The numbers vary based on the search algorithm and whether we count initialization failures. The numbers above used an attempt with beam search with size 32 for up to 100 iterations, followed by an attempt with 32 temperature 0.5 samples for up to 100 iterations, and a 10 minute timeout for the whole search. Using only the beam attempt got 48.8%/47.6% validation/test, so quite similar to the non-retrieval model from lean dojo.
We should have a more precise writeup later this month, along with proofsearch/evaluation code so that others can evaluate their models using the same evaluation setup.


Data error: (Stanford) MARC::XMLParseError: XML parsing error: Input is not proper UTF-8, indicate encoding !Bytes: 0xA0 0x4D 0x75 0x73

Stanford's provided some records with invalid UTF-8 characters in it (and, an empty subfield key...).
Backtrace
line 42 of [PROJECT_ROOT]/app/services/marc_record_service.rb: block in each
line 100 of [PROJECT_ROOT]/app/services/marc_record_service.rb: block in with_reader
line 99 of [PROJECT_ROOT]/app/services/marc_record_service.rb: with_reader

View full backtrace and more info at honeybadger.io

I've reported one record for remediation, but there's probably more...


Update CD pipeline to used trusted publishers

Fixes #11727
Status
ready
Description
With this PR, we move away from PyPi user/token authentication in our CD pipeline. Finally adopting (actually enabling) PyPI 2FA together with trusted publisher registration in the PyPi projects.
In addition, I have updated the version of a couple of GitHub actions.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
None
External dependencies / deployment changes
It does depend on configuring trusted publisher in all of the WMCore PyPi projects (under the account we use for publishing packages to PyPi).
Some references are:
https://docs.pypi.org/trusted-publishers/
and
https://packaging.python.org/en/latest/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/

Even though I have these changes tested in a playground repository, it could be that something will still fail, as workflows configuration are different.
I have just configured a trusted publisher in PyPi for all the projects currently defined in our CD pipeline, which I list in here as well:
        target: [wmagent, wmagent-devtools, wmcore, reqmon, reqmgr2, global-workqueue, acdcserver, reqmgr2ms-unmerged,
                 reqmgr2ms-output, reqmgr2ms-pileup, reqmgr2ms-rulecleaner, reqmgr2ms-transferor, reqmgr2ms-monitor]

And now, I am gonna give it a try without even bugging people for code review. Any feedback is welcomed though.

Wohooo, it works!
https://github.com/dmwm/WMCore/actions/runs/10287262120


Add link to bookmarklet back on Chrome version of homepage

@rdhyee Asked for it!

Closed with https://github.com/hypothesis/wordpress-theme-hypothesis/pull/31


[WIP] Sky color

Ports some of the Sky-Logic from the old REGoth repository. This PR adds a component to the GameWorld, which calculates the sky color depending on the daytime using the Gothic 2 sky colors.
To test this, one can modify GameClock from this:
constexpr float CLOCK_SPEED_FACTOR  = SECONDS_IN_A_DAY/6000;

to
constexpr float CLOCK_SPEED_FACTOR  = SECONDS_IN_A_DAY/60;

for 60 second days.
[WIP]: Color is not applied to any meshes now, only the clear color is modified.
Using an indoor-world like OLDMINE.ZEN works for testing, but you need to peek through the wall with your camera to see the effect.
Also #35 should be merged first.

Doesn't work for me. The sky stays blue in G1 and G2, no matter the world. It seems you forgot to commit the part where the Sky component is added to a scene object.

Hmm, I remember amending that to the commit. I can't check what went wrong currently though. If you want, you can simply add the Sky component to the game world. Nothing else is required.

When manually adding the component to the game world, the sky color does seem to change according to the time. Did you figure out what went wrong?

Works.


Expanded documentation for DataSketches aggregators.

Originally written by @AlexanderSaydakov in druid-io/druid-io.github.io#448. I also added redirects and updated links to point to the new datasketches-extension.html landing page for the extension, rather than to the old page about theta sketches.

Please note that the tuple sketch module is still in a pull request under review, not in master

Got it. In that case these should be tied to the same release, so we don't release the docs without the feature.

Is #5148 the tuple sketch?

yes, thanks

Thanks @AlexanderSaydakov - I updated the milestone on #5148.


Write tests

Packages need automated tests that run on npm test. Typecheck is good but not enough. Just need to make sure that the tests are not packaged on npm publish.

Created xpt package to achieve this. Simple error throwing framework which all packages in this repository will use for testing.

SMath has now incorporated xpt. May revise. Will set a precedent for all other packages.

xpt package now renamed to exray


[xcodeproj] LLDB search paths O(N XcodeRuns) growth

I noticed the LLDB search paths have an O(N XcodeRuns) growth rate. In other words, the growth rate is proportional to the number of times I run the app. The 2 pieces of evidence are in the settings, and I noticed LLDB performing several stat operations on a signle file at an increasing rate.
This was within a rules_ios project, however I'm not sure it's related to rules_ios. Tested on
Xcode 12.1
Build version 12A7403

debug-file-search-paths (file-list) =
  [0]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [1]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [2]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [3]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [4]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [5]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [6]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator
target.default-arch (arch) =
target.detach-on-error (boolean) = true
target.disable-aslr (boolean) = true
target.disable-stdio (boolean) = false
target.x86-disassembly-flavor (enum) = default
target.display-expression-in-crashlogs (boolean) = false
target.display-recognized-arguments (boolean) = false
target.display-runtime-support-values (boolean) = false
target.enable-synthetic-value (boolean) = true
env-vars (dictionary of strings) =
target.error-path (file) =
exec-search-paths (file-list) =
  [0]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [1]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [2]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [3]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [4]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [5]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [6]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator  [7]: /Users/jmarino/Library/Developer/Xcode/DerivedData/bazel-project-hjacvhupxduqpdamhfgrlcfihgcv/Build/Products/Debug-iphonesimulator/

Worth trying on a non-bazel project and compare the settings? Also were you using one of the test apps in the repo or is this an internal app? Looking for a way to repro myself.

Also is this with the VFS stuff on? Looking at the description  makes me think if it could be related to bad in-memory state

@thiagohmcruz this was VFS on and off on an internal app. I'd be interested to see if the debugger slows down in certain conditions after doing several builds and runs.
Perhaps we can build an performance suite on the C++ or pyhton API. The LLDB integration test https://github.com/llvm/llvm-project/blob/b87e6c6a2ba97a8b5c876298e4bf74a775f6ee30/lldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py may be of help here to integrate this into the performance suite.


Add header information to ga2sam

(This issue is part of a trio of closely related issues #705, #706 and #707 that would be well suited to a group of people working together on.)
The ga2sam utility converts the results of a reads/search request to the server into SAM/BAM format using pysam. This utility is roughtly functional, but needs some work to fill out the rough corners. One area that hasn't been worked on is writing an appropriate header.
To try this out ga2sam, run (assuming you are running a server with the example data)
$ python ga2sam_dev.py http://localhost:8000 MWtnLXAzLXN1YnNldDpIRzAwMDk2OlNSUjA2MjYzNQ== --referenceId=R1JDaDM4LXN1YnNldDoz | less -S

The header is just some fabricated data, and we want to fix this.
Edit ga4gh/converters.py and find the _getHeader function. We need to generate a header that conforms to what pysam expects and compatible with the SAM spec.
We want to add reference headers at a minimum. Use the information in self._reference (see the client API docs for details) to add some SQ headers.

Related https://github.com/ga4gh/ga4gh-server/issues/1457

Closing, future converter development can be seen here https://github.com/BD2KGenomics/ga4gh-integration/issues/67